Create an initial version of the bot

It is required to modify echo-bot and get the initial version of the bot from it, which should:

Have a basic set of commands
Know how to work with a database
Additional features for convenient administration

First of all we need to implement signing up in our bot.
Just a simple "Hello put ur Name 'n' Surname pls".
Later we will support dialog to create game.

And move bot.py file to src/bot and add some tests to test. Talking about tests I hope you will do something. Maybe it is a good idea to look through the source code of python-telegram-bot. I am more than sure that they do have tests there.

I look at file src/bot/main.py and start crying because of a long file. Maybe we should split functions that create games into one file, functions that allow users to sign up for a game into another and so on. We also should leave conversation handler and the main part of the bot in src/bot/main.py


qs.reports.metrics() and qs.reports.full() give different results using same data

So far I found cumulative returns, time in market, average drawdown and average drawdown days to be different but there could be others.
You can reproduce this issue with the following code:
version: quantstats                0.0.62

np.random.seed(0)
ts = pd.Series(np.random.randint(1, high=4, size=12), index=pd.date_range(start='2021-05-01', periods=12))

qs.reports.full(ts, compounded=True)
                           Strategy
-------------------------  -----------------
Start Period               2021-05-01
End Period                 2021-05-12
Risk-Free Rate             0.0%
Time in Market             67.0%

Cumulative Return          200.0%
CAGR﹪<PHONE_NUMBER>313.54%

Sharpe                     6.62
Prob. Sharpe Ratio         95.99%
Smart Sharpe               3.4

qs.reports.metrics(ts, compounded=True)
                    Strategy
------------------  -----------------
Start Period        2021-05-01
End Period          2021-05-12
Risk-Free Rate      0.0%
Time in Market      100.0%

Cumulative Return   11,059,100.00%
CAGR﹪<PHONE_NUMBER>313.54%

Sharpe              6.62
Prob. Sharpe Ratio  95.59%
Sortino             20.04

I have been finding that the call to:
metrics = qs.reports.metrics(returns, display=False, sep=False, mode="full")
Gives you values that are rounded to aggressively. Max Drawdown for example comes back as -0.08 if the drawdown was -8.38% for example. I have been unable to find any option that would change the output values that are not rounded. You can pass internal=True like:
metrics = qs.reports.metrics(returns, display=False, sep=False, mode="full", internal=True)
This will return strings of numbers used in the HTML report. So, for Max Drawdown for example you would get back "-8.3%", which is better than -0.08. But since this is a string, also not exactly what you want.
You could use the qs.stats method to return un-rounded values, but that is less elegant as you need to know all the key names for each stat. For instance, you could call:
mdd = qs.stats.max_drawdown()
This will get you the full value, which you can then round / multiply how you want. I was hoping I could get the the qs.metrics method to return all the un-rounded values.

Check out https://github.com/Lumiwealth/quantstats_lumi, which is being updated regularly. We are a fork of this library that is being maintained by Lumiwealth


Problem with multipart-upload and content-length

I get the following error when i try to upload large files to S3.
Smaller files that does not require multipart works fine.
Tho when i remote the 'ContentLength' option (AwsS3Adapter.php upload function) before upload, the multipart upload works fine.
I have tried with different servers and different s3 endpoints (eu-central-1 and eu-west-1) with the same result.
I started getting these errors about a week ago.
Not sure what to make of this :-)
Any clues?
Error: An exception occurred while uploading parts to a multipart upload. The following parts had errors:

Part 2: Error executing "UploadPart" on "https://xxxx.s3.eu-central-1.amazonaws.com/D.pdf?partNumber=2&uploadId=1fDClNAW1P21lsnYkl3RjnqPG3nj9926JJicbgufX_Vy8wmB5gyJcMgRgmMgJFc.qER6ZlhkSrV_VqmLl1QMtUBzJ0Awh3UTYBVMpeQ7Y4E-"; AWS HTTP error: Client error: PUT https://xxxx.s3.eu-central-1.amazonaws.com/D.pdf?partNumber=2&uploadId=1fDClNAW1P21lsnYkl3RjnqPG3nj9926JJicbgufX_Vy8wmB5gyJcMgRgmMgJFc.qER6ZlhkSrV_VqmLl1QMtUBzJ0Awh3UTYBVMpeQ7Y4E- resulted in a 400 Bad Request response:

RequestTimeoutYour socket connection to the server w (truncated...)
RequestTimeout (client): Your socket connection to the server was not read from or written to within the timeout period. Idle connections will be closed. -
RequestTimeoutYour socket connection to the server was not read from or written to within the timeout period. Idle connections will be closed.D4FBBC6ED5B2ADDCJqgxslgdNEBoVO27VV88++pbZID3IMOJxkI7eNDPGEFG1cLRL5bzwab3gb4hgfBCMj33YSkFrQM=

Part 3: Error executing "UploadPart" on "https://xxxx.s3.eu-central-1.amazonaws.com/D.pdf?partNumber=3&uploadId=1fDClNAW1P21lsnYkl3RjnqPG3nj9926JJicbgufX_Vy8wmB5gyJcMgRgmMgJFc.qER6ZlhkSrV_VqmLl1QMtUBzJ0Awh3UTYBVMpeQ7Y4E-"; AWS HTTP error: Client error: PUT https://xxxx.s3.eu-central-1.amazonaws.com/D.pdf?partNumber=3&uploadId=1fDClNAW1P21lsnYkl3RjnqPG3nj9926JJicbgufX_Vy8wmB5gyJcMgRgmMgJFc.qER6ZlhkSrV_VqmLl1QMtUBzJ0Awh3UTYBVMpeQ7Y4E- resulted in a 400 Bad Request response:

RequestTimeoutYour socket connection to the server w (truncated...)
RequestTimeout (client): Your socket connection to the server was not read from or written to within the timeout period. Idle connections will be closed. -
RequestTimeoutYour socket connection to the server was not read from or written to within the timeout period. Idle connections will be closed.BA0FF6BAE0D104EFM7aZrrJieX2ceZZ708HjzQ65PJaB43ckHAQgxAp/5GfteC+BXYliYD6DE1q+O4u2WK1Mujf4Gss=

Part 1: Error executing "UploadPart" on "https://xxxx.s3.eu-central-1.amazonaws.com/D.pdf?partNumber=1&uploadId=1fDClNAW1P21lsnYkl3RjnqPG3nj9926JJicbgufX_Vy8wmB5gyJcMgRgmMgJFc.qER6ZlhkSrV_VqmLl1QMtUBzJ0Awh3UTYBVMpeQ7Y4E-"; AWS HTTP error: Client error: PUT https://xxxx.s3.eu-central-1.amazonaws.com/D.pdf?partNumber=1&uploadId=1fDClNAW1P21lsnYkl3RjnqPG3nj9926JJicbgufX_Vy8wmB5gyJcMgRgmMgJFc.qER6ZlhkSrV_VqmLl1QMtUBzJ0Awh3UTYBVMpeQ7Y4E- resulted in a 400 Bad Request response:

RequestTimeoutYour socket connection to the server w (truncated...)
RequestTimeout (client): Your socket connection to the server was not read from or written to within the timeout period. Idle connections will be closed. -
RequestTimeoutYour socket connection to the server was not read from or written to within the timeout period. Idle connections will be closed.B00513569508D224AY7mfYZop7AF24ZWIOmiQ2YUoOO0IArQ9zMNWymkck93VNUzm/m+j/vdOlIO0IT1Dp2HVgXpIeE=

Part 4: Error executing "UploadPart" on "https://xxxx.s3.eu-central-1.amazonaws.com/D.pdf?partNumber=4&uploadId=1fDClNAW1P21lsnYkl3RjnqPG3nj9926JJicbgufX_Vy8wmB5gyJcMgRgmMgJFc.qER6ZlhkSrV_VqmLl1QMtUBzJ0Awh3UTYBVMpeQ7Y4E-"; AWS HTTP error: Client error: PUT https://xxxx.s3.eu-central-1.amazonaws.com/D.pdf?partNumber=4&uploadId=1fDClNAW1P21lsnYkl3RjnqPG3nj9926JJicbgufX_Vy8wmB5gyJcMgRgmMgJFc.qER6ZlhkSrV_VqmLl1QMtUBzJ0Awh3UTYBVMpeQ7Y4E- resulted in a 400 Bad Request response:

RequestTimeoutYour socket connection to the server w (truncated...)
RequestTimeout (client): Your socket connection to the server was not read from or written to within the timeout period. Idle connections will be closed. -
RequestTimeoutYour socket connection to the server was not read from or written to within the timeout period. Idle connections will be closed.CF889DA5E56141E9nDOKv2aXBRzeKxy+m8tPt5C9STDN+hTnpacpNsVqr/iymTYgfkDqFrpvLEEZqpRyQaagxTMl8cM=
#0 [internal function]: Aws\Multipart\AbstractUploadManager->Aws\Multipart{closure}()
#1 /Users/kalle/Workspace/xxxx/app/Vendor/guzzlehttp/promises/src/Coroutine.php(122): Generator->send(NULL)
#2 /Users/kalle/Workspace/xxxx/app/Vendor/guzzlehttp/promises/src/Promise.php(203): GuzzleHttp\Promise\Coroutine->_handleSuccess(NULL)
#3 /Users/kalle/Workspace/xxxx/app/Vendor/guzzlehttp/promises/src/Promise.php(156): GuzzleHttp\Promise\Promise::callHandler(1, NULL, Array)
#4 /Users/kalle/Workspace/xxxx/app/Vendor/guzzlehttp/promises/src/TaskQueue.php(47): GuzzleHttp\Promise\Promise::GuzzleHttp\Promise{closure}()
#5 /Users/kalle/Workspace/xxxx/app/Vendor/guzzlehttp/guzzle/src/Handler/CurlMultiHandler.php(96): GuzzleHttp\Promise\TaskQueue->run()
#6 /Users/kalle/Workspace/xxxx/app/Vendor/guzzlehttp/guzzle/src/Handler/CurlMultiHandler.php(123): GuzzleHttp\Handler\CurlMultiHandler->tick()
#7 /Users/kalle/Workspace/xxxx/app/Vendor/guzzlehttp/promises/src/Promise.php(246): GuzzleHttp\Handler\CurlMultiHandler->execute(true)
#8 /Users/kalle/Workspace/xxxx/app/Vendor/guzzlehttp/promises/src/Promise.php(223): GuzzleHttp\Promise\Promise->invokeWaitFn()
#9 /Users/kalle/Workspace/xxxx/app/Vendor/guzzlehttp/promises/src/Promise.php(267): GuzzleHttp\Promise\Promise->waitIfPending()
#10 /Users/kalle/Workspace/xxxx/app/Vendor/guzzlehttp/promises/src/Promise.php(225): GuzzleHttp\Promise\Promise->invokeWaitList()
#11 /Users/kalle/Workspace/xxxx/app/Vendor/guzzlehttp/promises/src/Promise.php(267): GuzzleHttp\Promise\Promise->waitIfPending()
#12 /Users/kalle/Workspace/xxxx/app/Vendor/guzzlehttp/promises/src/Promise.php(225): GuzzleHttp\Promise\Promise->invokeWaitList()
#13 /Users/kalle/Workspace/xxxx/app/Vendor/guzzlehttp/promises/src/Promise.php(62): GuzzleHttp\Promise\Promise->waitIfPending()
#14 /Users/kalle/Workspace/xxxx/app/Vendor/guzzlehttp/promises/src/Coroutine.php(65): GuzzleHttp\Promise\Promise->wait()
#15 /Users/kalle/Workspace/xxxx/app/Vendor/guzzlehttp/promises/src/Promise.php(246): GuzzleHttp\Promise\Coroutine->GuzzleHttp\Promise{closure}(true)
#16 /Users/kalle/Workspace/xxxx/app/Vendor/guzzlehttp/promises/src/Promise.php(223): GuzzleHttp\Promise\Promise->invokeWaitFn()
#17 /Users/kalle/Workspace/xxxx/app/Vendor/guzzlehttp/promises/src/Promise.php(267): GuzzleHttp\Promise\Promise->waitIfPending()
#18 /Users/kalle/Workspace/xxxx/app/Vendor/guzzlehttp/promises/src/Promise.php(225): GuzzleHttp\Promise\Promise->invokeWaitList()
#19 /Users/kalle/Workspace/xxxx/app/Vendor/guzzlehttp/promises/src/Promise.php(62): GuzzleHttp\Promise\Promise->waitIfPending()
#20 /Users/kalle/Workspace/xxxx/app/Vendor/aws/aws-sdk-php/src/S3/S3ClientTrait.php(30): GuzzleHttp\Promise\Promise->wait()
#21 /Users/kalle/Workspace/xxxx/app/Vendor/league/flysystem-aws-s3-v3/src/AwsS3Adapter.php(587): Aws\S3\S3Client->upload('xxxx', 'D.pdf', Resource id #192, 'public-read', Array)
#22 /Users/kalle/Workspace/xxxx/app/Vendor/league/flysystem-aws-s3-v3/src/AwsS3Adapter.php(368): League\Flysystem\AwsS3v3\AwsS3Adapter->upload('D.pdf', Resource id #192, Object(League\Flysystem\Config))
#23 /Users/kalle/Workspace/xxxx/app/Vendor/league/flysystem/src/Filesystem.php(87): League\Flysystem\AwsS3v3\AwsS3Adapter->writeStream('D.pdf', Resource id #192, Object(League\Flysystem\Config))
#24 /Users/kalle/Workspace/xxxx/app/Vendor/league/flysystem/src/MountManager.php(213): League\Flysystem\Filesystem->writeStream('D.pdf', Resource id #192, Object(League\Flysystem\Config))
#25 /Users/kalle/Workspace/xxxx/app/Console/Command/BackboneShell.php(46): League\Flysystem\MountManager->copy('D.pdf', 'D.pdf')
#26 /Users/kalle/Workspace/xxxx/lib/Cake/Console/Shell.php(458): BackboneShell->backup()
#27 /Users/kalle/Workspace/xxxx/lib/Cake/Console/ShellDispatcher.php(219): Shell->runCommand('backup', Array)
#28 /Users/kalle/Workspace/xxxx/lib/Cake/Console/ShellDispatcher.php(66): ShellDispatcher->dispatch()
#29 /Users/kalle/Workspace/xxxx/app/Console/cake.php(47): ShellDispatcher::run(Array)
#30 {main}

@kallealberg Could you tell me how did you solve this problem? as I'm facing it as well for just 160Mb files.

Updating composer fixed the issue
  - Updating spatie/laravel-backup (4.18.0 => 4.18.1): Downloading (100%)
  - Updating aws/aws-sdk-php (3.31.2 => 3.31.3): Downloading (100%)


fix: support for Fibers

Continuation of #99. Closes #46, closes #99.

I don't understand why fiber tests are failing in Docker containers but not when running directly in GHA runners. Adjusting the stack size with ulimit and setting -fstack-split seems to have no effect. Also, I don't manage to reproduce the issue locally (macOS + M1) or in Docker (tests are always green).
Alpine/musl doesn't look to be the problem here as fails also tests on Debian in Docker.
Do you have any idea of what can be the difference on @withinboredom?

I'll have to take a gander in the morning. At least for me, it looks like Docker has gone insane...
 => [internal] load .dockerignore                                                                                0.1s
 => => transferring context: 133B                                                                                0.0s
 => [internal] load build definition from Dockerfile                                                                                0.0s
 => => transferring dockerfile: 2.29kB                                                                                0.0s
 => CANCELED [internal] load metadata for docker.io/library/php-base:latest                                                                                0.5s
 => CANCELED [internal] load metadata for docker.io/mlocati/php-extension-installer:latest                                                                                0.5s
 => ERROR [internal] load metadata for docker.io/library/golang-base:latest                                                                                0.5s
 => [auth] library/php-base:pull token for registry-1.docker.io                                                                                0.0s
------
 > [internal] load metadata for docker.io/library/golang-base:latest:
------
Dockerfile:6
--------------------
   4 |
   5 |     COPY --from=golang-base /usr/local/go/bin/go /usr/local/bin/go
   6 | >>> COPY --from=golang-base /usr/local/go /usr/local/go
   7 |
   8 |     # This is required to link the FrankenPHP binary to the PHP binary
--------------------
ERROR: failed to solve: golang-base: pull access denied, repository does not exist or may require authorization: server message: insufficient_scope: authorization failed

I'm an idiot and forgot docker bake is a thing.

So @dunglas, I was digging around php-src to start that blog post :) and I saw this in the 8.3 notes:

Custom Fiber implementations have to initialize EG(stack_limit) and
EG(stack_base).

Maybe frankenphp needs to implement a go-specific fiber implementation?

Internals: SAPIs that may execute PHP in alternative stacks have to set EG(stack_limit) and EG(stack_base)

From: https://github.com/php/php-src/pull/9104
That PR is amazing information about how the stack works.  This assumes contiguous stacks.

Emphasis mine. I don't think the stack will be contiguous with Go involved.

Thanks for the pointer, this looks interesting! Passing --disable-fiber-asm doesn't help.
Regarding contiguous stacks, I'm not sure that Go will be a problem because FrankenPHP creates the threads itself, they aren't the same threads as the one started by Go.

they aren't the same threads as the one started by Go.

Yep, saw that as well. I wonder if we're running into the same situation as https://github.com/golang/go/issues/39411#issuecomment-639863069 and the PHP fiber implementation is somehow messing the main stack?

I added zend_call_stack_init(); in frankenphp_execute_script (from Zend/zend_call_stack.h) as an experiment. It went from passing 30-40% of the time to passing 70-80% of the time. There's probably a hint somewhere in that code path as to what is going on. Maybe. (this is PHP 8.3 specific, that code doesn't exist in 8.2)
7/10 vs. 4/10

One thing sticks out to me:
goroutine 120 [runnable, locked to thread]:
runtime.gcTrigger.test({0x0?, 0x0?, 0x0?})
        /usr/local/go/src/runtime/mgc.go:547 +0xfb fp=0xc000b27990 sp=0xc000b27988 pc=0x41ed7b
runtime.mallocgc(0x4000, 0x788020, 0x1)
        /usr/local/go/src/runtime/malloc.go:1171 +0x768 fp=0xc000b279f8 sp=0xc000b27990 pc=0x412f48
runtime.makeslice(0xac9120?, 0x788060?, 0xc000b27a68?)
        /usr/local/go/src/runtime/slice.go:103 +0x52 fp=0xc000b27a20 sp=0xc000b279f8 pc=0x454212
github.com/dunglas/frankenphp.go_read_post(0x70ad45?, 0x409b46?, 0x4000)
        /go/src/app/frankenphp.go:590 +0x87 fp=0xc000b27b48 sp=0xc000b27a20 pc=0x7079e7
_cgoexp_411032355cef_go_read_post(0x7f348abe9ee0)
        _cgo_gotypes.go:985 +0x2c fp=0xc000b27b70 sp=0xc000b27b48 pc=0x70ae0c
runtime.cgocallbackg1(0x70ade0, 0xc000b27d20?, 0x0)
        /usr/local/go/src/runtime/cgocall.go:315 +0x2b1 fp=0xc000b27c38 sp=0xc000b27b70 pc=0x409a51
runtime.cgocallbackg(0xc000482340?, 0x300000002?, 0xc000482340?)
        /usr/local/go/src/runtime/cgocall.go:234 +0x109 fp=0xc000b27cc8 sp=0xc000b27c38 pc=0x409709
runtime.cgocallbackg(0x70ade0, 0x7f348abe9ee0, 0x0)
        <autogenerated>:1 +0x2f fp=0xc000b27cf0 sp=0xc000b27cc8 pc=0x47382f
runtime.cgocallback(0x409585, 0x759b20, 0xc000b27d80)
        /usr/local/go/src/runtime/asm_amd64.s:998 +0xb4 fp=0xc000b27d18 sp=0xc000b27cf0 pc=0x4710d4
runtime.systemstack_switch()
        /usr/local/go/src/runtime/asm_amd64.s:463 fp=0xc000b27d20 sp=0xc000b27d18 pc=0x46f0e0
runtime.cgocall(0x759b20, 0xc000b27d80)
        /usr/local/go/src/runtime/cgocall.go:167 +0x85 fp=0xc000b27d58 sp=0xc000b27d20 pc=0x409585
github.com/dunglas/frankenphp._Cfunc_frankenphp_execute_script(0x7ef0bc0a35d0)
        _cgo_gotypes.go:770 +0x48 fp=0xc000b27d80 sp=0xc000b27d58 pc=0x7047e8
github.com/dunglas/frankenphp.go_execute_script(0xc000482301?)
        /go/src/app/frankenphp.go:461 +0x207 fp=0xc000b27e20 sp=0xc000b27d80 pc=0x7067a7
_cgoexp_411032355cef_go_execute_script(0x7f4987ffea50?)
        _cgo_gotypes.go:925 +0x1c fp=0xc000b27e38 sp=0xc000b27e20 pc=0x70abdc
runtime.cgocallbackg1(0x70abc0, 0xc000b27fe0?, 0x0)
        /usr/local/go/src/runtime/cgocall.go:315 +0x2b1 fp=0xc000b27f00 sp=0xc000b27e38 pc=0x409a51
runtime.cgocallbackg(0x0?, 0x0?, 0x0?)
        /usr/local/go/src/runtime/cgocall.go:234 +0x109 fp=0xc000b27f90 sp=0xc000b27f00 pc=0x409709
runtime.cgocallbackg(0x70abc0, 0x7f348abf0980, 0x0)
        <autogenerated>:1 +0x2f fp=0xc000b27fb8 sp=0xc000b27f90 pc=0x47382f
runtime.cgocallback(0x0, 0x0, 0x0)
        /usr/local/go/src/runtime/asm_amd64.s:998 +0xb4 fp=0xc000b27fe0 sp=0xc000b27fb8 pc=0x4710d4
runtime.goexit()
        /usr/local/go/src/runtime/asm_amd64.s:1598 +0x1 fp=0xc000b27fe8 sp=0xc000b27fe0 pc=0x471301

Is Go doing GC while executing a script? Am I reading that right?

GC is a red herring. I disabled it, and analysed a bunch of stack traces. There doesn't seem to really be much in common which makes me think this is on PHP's side, not Go's.
Oh, and zend_call_stack_init is also a red herring. After more rigorous testing, it is clear that I was just "lucky".

There's some asm in Go that switches to g0 upon returning from a cgo callback. I suspect that is where the culprit is. However, I cannot reproduce the crash in the debugger. Still trying.

From cgocall.go:
// The call from C is on gp.m's g0 stack, so we must ensure
// that we stay on that M. We have to do this before calling
// exitsyscall, since it would otherwise be free to move us to
// a different M. The call to unlockOSThread is in unwindm.

hmmmm....

I've managed to figure out how to reproduce it reliably while a debugger is running. Yet stepping through the code is enough to work around the issue. This is certainly a race condition:

Reboot the computer.
Startup debugging of frankenphp
Make a request to localhost
See the crash within the first 1-5 requests
Goto (1)

I've narrowed down the issue to somewhere in _cgo_gotypes.go, which appears to be a generated file and not checked into source control. The issue appears to possibly be a bit of funky memory from PHP. Although that isn't certain yet. This is a very slippery bug.

Requires a custom build of PHP, but I can no longer reproduce it all locally...
./configure $conf_vars
echo #define ZEND_FIBER_UCONTEXT 1 >> main/php_config.h
make -j

ZEND_FIBER_UCONTEXT  uses a different method for managing the stack that seems to work just fine with Go.

Also, there's an epic blog post 🔜 ... it needs a lot of cleaning up, which I hope to get to in the nearish future.

@withinboredom thanks for your in-depth research on this topic, I very much appreciate it!! Unfortunately, I don't think that defining ZEND_FIBER_UCONTEXT is enough, because it's already what this PR does and tests are still failing (at least with Docker): https://github.com/dunglas/frankenphp/pull/171/files#diff-dd2c0eb6ea5cfc6c4bd4eac30934e2d5746747af48fef6da689e85b752f39557R95-R96
Passing the --disable-fiber-asm flag defines ZEND_FIBER_UCONTEXT: https://github.com/php/php-src/blob/php-8.2.9/configure.ac#L1317

:sigh: ok, I have another lead, but that was the most promising.

Funny enough, today I did run into it with the asm disabled locally (outside of tests).
Anyway, I think I know what is going on. At least I'm fairly certain. This bug should affect ALL cgo programs, if I'm actually understanding things correctly.
Essentially the issue is that we're passing a pointer from C to Go that contains pointers to Go structures... in morestack, it calls newstack which makes a copy of the stack and rewrites all pointers. When we're using Fibers, this causes the stack to grow upon reentry to Go, which causes it to rewrite all pointers.
Here's the thing. These pointers are owned by C structs, not Go structs. Thus they crash quite spectacularly when they don't get rewritten by newstack.
I think this can be fixed by having a hash in Go that the C structs then reference (main context / current context). Essentially:
// Go Land
$contexts = ['main' => $ctx, 0 => $otherctx, 2 => ... ];

// C Land
{
  go_call_thingy('main', 0);
}

// Go Land
go_call_thingy($main, $current) {
  $main = $contexts[$main];
  $current = $contexts[$current];
}

or something like that. Here's the thing, once a few requests are served, the stack stays pretty stable, thus that is why it is hard to reproduce on 'real servers' vs. these tests which keep the stack really small and thus easy to grow, especially the g0 stack.
The moral of the story is that I think we cannot reference Go structs from C structs or vice-versa.

Thanks for sharing your findings, this looks promising!
Maybe are we violating the pointers passation rules in some way?
Do you have already identifier the offending pointers? Maybe could be avoid passing them from Go to C, or ping them?
Also, we should set GODEBUG=cgocheck=2 in tests to see if we get better errors.

Do you have already identifier the offending pointers? Maybe could be avoid passing them from Go to C, or ping them?

Ah, yep. I guess I accidentally hinted to them. It's the HTTP contexts from Caddy which are stored in the current/main contexts in Frankenphp's C code.

I just reviewed the code again, and I don't think that we store any Go pointer in C memory.
We use runtime/cgo.Handle (which is designed to not have to pass Go pointers to C).
Do you have any GDB backtrace? (I still cannot reproduce locally, most likely because my CPU an Apple M1 which seems unaffected by this bug).

I went down quite a rabbit hole trying to find a bad pointer.
Handy bit of C:
static void debug_log(char *data) {
	pid_t id = gettid();
	fprintf(stderr, "tid %d: %s", id, data);
}

after commenting out sections, hardcoding values from Go, and adding debugging statements everywhere, I realized that I was getting a crash before a thread could be reused. After adding debugging code in thpool, this turned out to not be true.
as far as I can tell, the crash almost always happens between the call (cgo asm) to go_add_header from frankenphp_send_headers. It doesn't matter what you pass to it, whether it is an invalid pointer or not, whether it is junk length or junk data. Just simply making the call breaks everything.
then I realized that was not the issue, after a git stash and commenting out the go_header* funcs. No, I also had to remove the call to go_ub_write. Once those were commented out, there was no more crashing.
Obviously, this is not ideal. However, it appears that simply outputting something from a PHP fiber causes Go to crash in the assembly code. Sometimes. In between the C and the actual entry of the Go program. I have no idea why, I cannot catch it while debugging.

This is beyond my expertise to resolve now, as I'm not intimately familiar with Go and assembly. Here's a minimum diff to get tests to run on this branch (though they still fail due to not outputting anything) without crashing, uncommenting either one of those functions will cause the crash when running tests.
Apparently, this may be amd64 specific. I can reproduce this in an ubuntu 22.04 VM without fail, or a docker container. On my baremetal Linux laptop (pop-os 22.04), I cannot reproduce it reliably (if at all).
Now I'm wondering if this is ubuntu specific... but I'll have to test that later, it's 1am.
cc: @krakjoe

@withinboredom here is why: https://github.com/golang/go/issues/62130#issuecomment-1684431260
We'll likely have to patch Go to add support for Fibers.

I wonder if we can do this in C.

detect we are in a fiber context
reset the stack back to what Go expects
call go
on return, reset the stack to what PHP expects
resume program

Huh, dunno how I missed this, but zend_timeout_handler gets called from a syscall to sleep, even from Go. So, as soon as Go sleeps for any reason, it tries to call zend_timeout_handler... this causes a crash, instantly.

Regarding zend_timeout_handler, are you sure that you compiled PHP with --enable-zend-max-execution-timers? This feature prevents this issue and we have tests for this.

are you sure that you compiled PHP with --enable-zend-max-execution-timers?

Double-checked and verified. Running 8.3-dev fwiw.

are you sure that you compiled PHP with --enable-zend-max-execution-timers

Double-checked and verified. Running 8.3-dev fwiw.

In fact, building with --disable-zend-max-execution-timers causes tests to pass with fibers. It seems there is a bug with that feature?

Doing this:
static void disable_timers() {
	#ifdef ZEND_MAX_EXECUTION_TIMERS
		// unset default timeout
		// TODO: add support for max_input_time
		zend_unset_timeout();
	#endif
}

static void restart_timers() {
	#ifdef ZEND_MAX_EXECUTION_TIMERS
		// Reset default timeout
		// TODO: add support for max_input_time
		zend_set_timeout(INI_INT("max_execution_time"), 0);
	#endif
}

and wrapping calls to go that happen during scripts also seems to work. It'd be handy if zend_unset_timeout returned the seconds remaining so we could zend_set_timeout with the correct time.

Turns out to be that PHP isn't installing the signal handlers properly. I'm opening a PR over there to get it fixed in 8.2/8.3 -- well, hopefully. I've never opened a PR over there.

I can assist you and review if you need any help (it's probably related to changes I've done to PHP). Feel free to contact me in DM!

Turns out to be that PHP isn't installing the signal handlers properly.

I was going to ask if you'd tried toggling whether ZEND_SIGNALS was (en|dis)abled.

Fixed by #1137.


Stop depending on rules_pkg through the federation

See discussion on https://github.com/bazelbuild/bazel-federation/issues/117

cc @aiuto

Thanks. I don't have write access, please do the merge.


better CMake packaging, add forestclaw.pc generation

waiting on:

[x] https://github.com/cburstedde/libsc/pull/33
[x] https://github.com/cburstedde/p4est/pull/92

Scott merged into develop


[Proposal] Add tolerations attribute under spec.job attribute

The same attribute which exists under jobManager and taskManager should also exist under job attribute.
In our current case, the pod won't be allocated to any node unless this 'tolerations' attribute is specified. Currently we're using some webhook just to add this attribute. Adding it in the CRD would be a more elegant solution.

Hi @Archangelix! adding that makes total sense. Let me know if you want to take a stab at it.

Hi @Archangelix I went ahead and add it, coz I was actually needing it myself 😄


Add additional data to uploader function?

Is your feature request related to a problem? Please describe.
I'm not even sure if this is exists already or not, but I can't figure out from reading the docs how I might add additional data to the xhrUploader. I'd like to add a simple {user_id: user.id}.
Describe the solution you'd like
Similar to the axios.js library, I'd like to be able to add additional data to the uploader to be sent to my API endpoint.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
I currently have something along the lines of:
this.uppload = new Uppload({
    customClass: 'avatar-upload',
    lang: en,
    uploader: xhrUploader({
        endpoint: `${apiUrl}/update_avatar`,
        method: 'POST',
        responseKey: 'avatar_url',
        settingsFunction: xmlHttp => {
            xmlHttp.setRequestHeader('Accept', 'application/json');
            xmlHttp.setRequestHeader('Authorization', 'Bearer ' + this.currentUser.token);
        },
   }),
});

If this ability does already exist, where can I find that in the docs?
Thanks!

@jimmiejackson414 You can simply add custom headers to accomplish this, for example:
this.uppload = new Uppload({
    customClass: 'avatar-upload',
    lang: en,
    uploader: xhrUploader({
        endpoint: `${apiUrl}/update_avatar`,
        method: 'POST',
        responseKey: 'avatar_url',
        settingsFunction: xmlHttp => {
            xmlHttp.setRequestHeader('Accept', 'application/json');
            xmlHttp.setRequestHeader('Authorization', 'Bearer ' + this.currentUser.token);
            xmlHttp.setRequestHeader('user_id', user_id);
        },
   }),
});

Then just grab the headers from the HTTP request on the server side/your API endpoint and you should be good to go.

Thanks for opening this issue, @jimmiejackson414, and thanks @sigma-technology for the solution. It makes a lot of sense to use xhrUploader and configure it with additional data in headers.
If you don't want to do that, however, you can write your own uploader from scratch, for example like this based on our Fetch uploader:
const uploader = () => {
  return file =>
    new Promise((resolve, reject) => {
      const formData = new FormData();
      formData.append("file", file);
      formData.append("user_id", user.id);
      window
        .fetch(
          "https://example.com/endpoint",
           {
              method: "POST",
              body: formData
           }
        )
        .then(response => {
          if (!response.ok) throw new Error("errors.response_not_ok");
          return response.json();
        })
        .then(json => resolve(json.url))
        .catch(() => reject("errors.response_not_ok"));
    });
};

new Uppload({ uploader });

For more information about creating a custom controller, read this article: Writing a custom controller

Ahh well I feel slightly like an idiot now haha. I definitely misunderstood the docs. To be honest, it might be helpful to add these examples to the docs? I'll go ahead and close, and thanks to both of you for your help!


Create Recipe-App Models

Task #1 - Create the Models by looking at the ERD diagram below.

Create the models for User, Food, Recipe-Food, Inventory, Inventory-Food, and Recipe.

Create a model using rails g model MODEL_NAME

Make migrations for all tables in your database

command: rails db: migrate.

Alter the migration table by adding the associations.

Feature Implemented


[Feature 13.2] ui/passenger/PurchaseTicketPage.java

ticket sale of $0 should now be allowed
ticket sale should be limited to max 12 tickets

I meant not instead of "now"


Add a config for ECS deployment

What this PR does / why we need it:
Which issue(s) this PR fixes:
Ref https://github.com/pipe-cd/pipe/issues/1568
Does this PR introduce a user-facing change?:

NONE

@nghialv
I'm ready for your review.

@nghialv
I'm ready for your review again!

/approve cancel

Thank you.
/lgtm

Thank you!
/approve


Web user still "kind of" logged in after session expires

Your Environment

verdaccio version: v5.10.0
node version: v16.15.0
package manager: npm 8.5.5 (but bug is in website)
os: verdaccio runs on linux, user runs chrome version 101 on MacOS
platform: Website

NOTE: the Local Storage in chrome still contains username, darkMode, and token

Describe the bug
After the web session expires for a logged-in user (after ~60 minutes) the user is still shown in the user-dropdown but has no access to packages (we use access: $authenticated). In order to view the packages again the user has to log-out and then log-in again.
To Reproduce

Start verdaccio with access: $authenticated configuration
Log into web server as a valid user
wait until session expires (~60 minutes)
reload page:

No Packages are shown: Correct
User is still shown (e.g., "Hi joerg.rech"): Intended?
To log-in the user has to log-out and then log-in again: Inconvenient

Expected behavior
I expected that the user can directly log-in without having to log-out first. Either by showing a menuitem to log-in in the user dropdown (e.g., where the log-out menuitem is) or by directly showing  the log-in modal dialog.
Screenshots, server logs, package manager log
N/A
Configuration File (cat ~/.config/verdaccio/config.yaml)
I don't think this will help - but I can provide it if you need it
Environment information
Verdaccio is running on AWS ec2 t2.micro instance using pm2 verdaccio behind a load balancer using a certificate to support https.
Environment Info:(node:25430) [LRU_CACHE_OPTION_maxAge] DeprecationWarning: The maxAge option is deprecated. Please use options.ttl instead.
(Use `node --trace-deprecation ...` to show where the warning was created)

  System:
    OS: Linux 5.10 Amazon Linux 2
    CPU: (1) x64 Intel(R) Xeon(R) CPU E5-2676 v3 @ 2.40GHz
  Binaries:
    npm: 8.5.5 - ~/.nvm/versions/node/v16.15.0/bin/npm
  npmGlobalPackages:
    verdaccio: 5.10.0

Debugging output
N/A for web?

I ran into the same exact behaviour today. Verdaccio 5.8.0 on an GCP VM instance running Debian 10.

Hello, what kind of authentication plugin do you use? @joergrech

Hey, I'm using the default one.

I managed to reproduce the error.
Environment

verdaccio version: v5.13.1

node version: v16.15.1

package manager: npm 8.11.0

os: debian 11

**platform:** Website

Config
storage: ./storage
auth:
  htpasswd:
    file: ./htpasswd
uplinks:
  npmjs:
    url: https://registry.npmjs.org/
packages:
  "@*/*":
    access: $authenticated
    publish: $authenticated
    proxy: npmjs
  "**":
    proxy: npmjs
    access: $all
    publish: $authenticated
log: { type: stdout, format: pretty, level: http }

I published packag @test/npm-package-1 and npm-package-1. One hour after logging in, package @test/npm-package-1 was not visible.
Corresponding log from verdaccio:
 http --- <IP_ADDRESS> requested 'GET /'
 http --- 200, user: null(<IP_ADDRESS>), req: 'GET /', bytes: 0/0
 http --- <IP_ADDRESS> requested 'GET /-/verdaccio/data/packages'
 http --- 200, user: null(<IP_ADDRESS>), req: 'GET /-/verdaccio/data/packages', bytes: 0/0
 info --- auth/allow_action: access granted to: undefined
 info --- allowed access for npm-package-1
 error--- forbidden access for @test/npm-package-1. Error: authorization required to access package @test/npm-package-1

I logged out and then logged in. Package @test/npm-package-1 was now visible. Log:
 http --- <IP_ADDRESS> requested 'GET /'
 http --- 200, user: null(<IP_ADDRESS>), req: 'GET /', bytes: 0/0
 http --- <IP_ADDRESS> requested 'GET /-/verdaccio/data/packages'
 http --- 200, user: null(<IP_ADDRESS>), req: 'GET /-/verdaccio/data/packages', bytes: 0/0
 info --- auth/allow_action: access granted to: undefined
 info --- allowed access for npm-package-1
 info --- auth/allow_action: access granted to: undefined
 info --- allowed access for @test/npm-package-1

I have a feeling of déjà vu, you should take a look at my thread here: https://github.com/verdaccio/verdaccio/discussions/3059
Let me know if you found something helpful.

I have a feeling of déjà vu, you should take a look at my thread here: #3059
Let me know if you found something helpful.

Thank you for the info. I think I found the cause.
Looks like the web UI shows "logged in" status when values token and username are present in the local storage of the browser (https://developer.mozilla.org/en-US/docs/Web/API/Window/localStorage), which has no expiration time and is cleaned only by logging out.

Following the login, the token expires after 1 hour (https://verdaccio.org/docs/configuration/#security), but it is still present in the local storage until the user logs out.

@juanpicado, can you take a quick look at this?

@juanpicado, can you take a quick look at this?

No much time this week sorry, maybe over the next weeks or whenever I find some time next days, no guarantee.

Any update on this issue? I can still reproduce it in the latest release.

Just FYI, we ended up setting the token expiration duration to 3 months (default is 1 hour). It's not ideal but it made our lives much easier. See https://github.com/verdaccio/verdaccio/discussions/3059#discussioncomment-2406364 for detailed informations.

I am also still experiencing this issue in the latest release.
Any updates on this?

The issue is reproducible some time ago. I just write here to keep live this issue.


Hostname on xtensa esp-idf esp32s3

Hi,
I need to use hostname_max and the function gethostname for xtensa-esp32s3-espidf but they're not supported for this target.
Can anyone give me any suggestion or help?
Thanks

PRs are welcome here! Take a look at https://github.com/rust-lang/libc/blob/1ffe0bde929a43ce3d25a8f08b9f95bde8d3e5d8/CONTRIBUTING.md, it should be pretty easy to add the API on those platforms.

There is no written guide for this target specifically that i am aware off. Though the target ( and all other -espidf targets ) are using a version of newlib that is directly integrated in the underlying esp-idf system. Depending on the version of the esp chip there are parts of this newlib library directly inside the ROM.
Here you can find the in tree version of newlib that is used in ESP_IDF version 5.2.2.
What actually lives inside the ROM's is tracked inside this repository
We have one outstanding PR that does a bit of cleanup inside libc with respect to correct constants. There you see for example how the rust libc site and the esp_idf site can be checked against each other.
If you have any target specific questions all the maintainers are also available in our esp-rs matrix channel  found here


adding graphical and letter symbol

Added 2 new models

GraphicalSymbolDesignation
LetterSymbolDesignation

closes #30

Thanks @HassanAkbar ! Please feel free to merge.


Should reserve moves have several uses (like items)?

Should reserve moves have several uses in 1 turn?

No, that would be way too overpowered.


facter -p stuck on plesk servers with a sendmail-wrapper-postfix

Hello,
The puppet agent is stuck on plesk 9.x servers with centos 7 because
sendmail -d0.4 -ODontProbeInterfaces=true -bv root 2>/dev/null
Doesn't exit
Quick fix is maybe a
Facter.add(:sendmail_version, :timeout => 5) do
With regards,
Richard

I suspect the main problem here is the wrapper reading unconditionally from STDIN and ignoring all command line arguments.

I have updated the fact code using a timeout. Hopefully this fixes your issue.
I will prepare a new module release on the forge including the fix.


[BUG] default GitHub branch name is hardcoded to "master"

Bug Template
Describe the bug
When using the GitHub integration from MakeCode, the new repo defaults to the master branch. GitHub changed their default branch name for new repos to main recently, so I'd expect either the default to be main or to have a way to specify what branch gets used
To Reproduce
Steps to reproduce the behavior:

Go to https://makecode.microbit.org/#editor
Create a project
Click on the GitHub integration icon and save it to a GitHub repo
Open the project on https://www.github.com
Notice that the default branch is master

Expected behavior
I'd expect the GItHub default of main to be used
Desktop (please complete the following information):

OS: macOS 11.6
Browser: Chrome
Version: 94.0.4606.81

Additional context
Add any other context about the problem here.

Looks like a duplicate of #3452

@eanders-ms did we implement this?


Test tweaks

Resolves issues 2 through 4 of #210.
Happy to make any adjustments that you would like!

Hi Eddie, I believe the next time you run the CI pipeline, the lint-ancient ancient job will fail. You will have to upgrade the lein-cloverage to 1.2.1 in project.clj.

@anthony-khong Thanks for the review. I have addressed the PR comment and rebased on master so that the lein-cloverage version bump is in the history. Let me know if there is anything else you would like me to do.

Thanks again @erp12. This is awesome!


connection error with create-react-app

Is this a bug report?
Yes.

Did you try recovering your dependencies?

My project folder is empty because it fails to create react app.
Which terms did you search for in User Guide?

"Connection error", "create-react-app not working/failing"
Environment

Environment Info:
System:
OS: Windows 10
CPU: (8) x64 Intel(R) Core(TM) i7-7700HQ CPU @ 2.80GHz
Binaries:
Node: 11.13.0 - C:\Program Files\nodejs\node.EXE
Yarn: 1.9.4 - C:\Program Files (x86)\Yarn\bin\yarn.CMD
npm: 6.7.0 - C:\Program Files\nodejs\npm.CMD
Browsers:
Edge: 42.17134.1.0
Internet Explorer: 11.0.17134.1
npmPackages:
react: Not Found
react-dom: Not Found
react-scripts: Not Found
npmGlobalPackages:
create-react-app: Not Found
Steps to Reproduce

(Write your steps here:)

Run create-react-app project-name OR npm init react-app project-name OR npx create-react-app project-name OR yarn create react-app project-name

Expected Behavior

I expected a new react app would be created as usual.
Actual Behavior
I get a connection error even though:
a) my connection is working properly
b) I tried turning firewalls off
c) I tried uninstalling create-react-app and installing it again, clearing cache, uninstalling and reinstalling yarn. Still same connection error.
(Write what happened. Please add screenshots!)

Reproducible Demo

I don't have any code, I'm failing at create-react-app.

Fixed. After disabling all firewalls, turning Windows defender off and shutting down Malwarebytes, I noticed there was a background process of Malware bytes I couldn't shut down, so I removed it from Startup, restarted my computer and now it worked. Not sure which one of those things worked - I had tried the firewall and Windows Defender, but I might not have turned off the correct firewall before (I just noticed my network was incorrectly set to public network so I might have overlooked that).


#125 Add codacy coverage reporting

Closes #125

Current status: I have a problem with codacy API key. Need to encrypt it in Travis properly.

@netme You can have a look at pazuzu-registry where I already did this integration. Hope this helps.

@fmueller , thanks I will check.

Things to do after merge:

[ ] Remove a temporary API key from Codacy
[ ] Save encrypted key to build against master

👍


Make "Registration" button take proper action

Pending PR #82
The registration button needs to:

Start the registration flow to OpenShift Online Starter through RHD
Be linked into Segment tracking to track the action (Step 3 of https://segment.com/docs/sources/website/analytics.js/quickstart/)

PR #82 done, ready for this to be scheduled.

I think this can be closed?


E2-1628 refactor request class part 1

yeah, let's rename the old class to RequestOld
On Mon, Oct 29, 2018 at 12:18 PM Grace Gu<EMAIL_ADDRESS>wrote:

@gg4race commented on this pull request.
In AndroidSDKCore/src/main/java/com/leanplum/internal/LPRequest.java
https://github.com/Leanplum/Leanplum-Android-SDK/pull/276#discussion_r228814026
:

software distributed under the License is distributed on an

"AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY

KIND, either express or implied.  See the License for the

specific language governing permissions and limitations

under the License.

*/

+package com.leanplum.internal;
+
+import com.leanplum.Leanplum;
+
+import org.json.JSONObject;
+
+import java.util.Map;
+
+public class LPRequest implements LPRequesting {
There are a lot of package private references to things inside the Request
class. Do you think it's worth the effort to make these changes? Or I could
name the old Request class as RequestOld?
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/Leanplum/Leanplum-Android-SDK/pull/276#discussion_r228814026,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAudLFJOuYqzVUDhuuH9HrpIga27oj7rks5upqSvgaJpZM4X7Xzn
.

--
Thanks
Mayank


[WIP] Add fly-curl command.

I use cf curl a bunch and would find a similar command for fly useful. For example, until there's time to implement #129, this would give users a quick way to list teams from the cli. If this sounds useful, I'll fill in integration tests.

So I have a general philosophical unease with this.
I used to be the anchor of the CLI team and am now the anchor of the Concourse team and I very strongly believe that cf curl was used as a crutch because the CLI did not implement things that people needed it to.
And now, the CF CLI team is having to go back to the drawing board and extract all of the use cases that people are silently using cf curl to accomplish.
I would much prefer to not fall down this same trap and instead have a more design-centered feedback cycle for fly.

I agree that using cf curl, fly curl, etc as a crutch to put off necessary features would be a problem. But I'm also not sure that fly needs to support every single endpoint from the moment it exists, or to offer a one-to-one correspondence with the api. And even if cf and uaac guaranteed full api support, I expect that I would still use those commands, since they're useful for writing and debugging applications that consume the cf and uaa apis.
So I also think that fly users should never have to use a command like fly curl, but I do think it's useful to have the option if users want it.

WDYT @chendrix ?

@chendrix I'm surprised that there isn't a fly curl or fly api command (https://github.com/concourse/concourse/issues/1078). As a product owner, yes, you'd like to support most use cases with dedicated commands.
But from a end-perspective it's hard not to hear: "You're out of luck because we've not implemented 'X', please file an issue and wait 4 months...."
I don't think that's the message you want to send, is it?
Cheers, Peter

@chendrix: we're always impressed with how much the concourse team is able to accomplish with a small team and appreciate the work you all do. My hope with this patch was to take just a little bit of pressure off you by making it easier for users to interact with the api without having to ask questions in slack or file feature requests. And, selfishly, I have found cf curl to be really useful even when the cf cli exposes the features I'm using, like I was saying earlier--it's a convenient way to understand the api when developing scripts or applications that consume it. But if this feature doesn't offer value to other concourse users, and/or if it's going to add too much to your maintenance burden, we won't be heartbroken if you close this.

Thanks @jmcarp, you will notice I haven't closed this PR, I still agree that there are merits to adding it. I personally haven't devoted the time to think about my reservations, the implications, and weighed adding/not adding it with the team.
We are also undergoing some new processes around product management these days so figuring out how adding this PR fits into that new process is still evolving.
It might not feel satisfactory right now, but I implore you to give us a bit more time. I have a ton of respect for the 18F team, we added a bunch of your PRs recently, but this one is a bit meatier of a PR from a product standpoint.

Thanks for the updates, @chendrix. My comment did indeed read as lacking empathy for your work. Sorry. :/
I would like a way to have fly handle my api authentication so I could, indeed, explore the API or implement calls into concourse that haven't yet made their way into a fully supported subcommand.
Since so many of us in this space are used to something like cf curl or knife raw (Chef), then an equivalent command in fly seems (to me) like a natural use case.
Good luck with making a call on this one. I'll be a happy Concourse user regardless. Cheers, Peter

@topherbullock and I had a situation where this would have been useful while pairing. That being said, we were able to getting around not having it.
Since the team has not committed to supporting a public facing API, I'm against this. With cf curl you are able to look at the capi docs to know what endpoints can be used and are supported. We don't have that. And in the vein of "I don't want a bunch of support tickets saying, 'I used fly curl but not it is broken in the latest release, HALP!?" I'd don't want that situation.
I'm not for this PR, selfishly. 😛

The issue the original PR (concourse/concourse#1072) mentions is closed because another PR added that specific functionality. Are there any other compelling uses for this? Or is it just a giant stopgap for "I want X feature in fly" or "I want to poke at the APIs"? I also don't feel great about having an http client interface to the ATC on the CLI, and taking a path of documenting all the APIs.
I'd be open to being compelled if there's a burning use case that this serves, but for now I'm going to close this off in favour of improving the CLI UX, rather than bloating (yes, its a small PR, but not conceptually small in product impact) with a feature that will lead us down a dark path of supporting an API any other HTTP requests folks want to start making.


OllaMail - An email client powered by Ollama

Hi Ollama Team,
Thank you for your time and effort in making sure that the Ollama API is always exceptional when a new version is released.
With that said, I'd like to inform you of an email client that reads and sends email using Ollama. I'll be making a version native to Windows available and wanted to know if you would like to add it to the README.md Community Integrations. It's called OllaMail. See git repo: https://github.com/perpendicularai/OllaMail
I have attached an image for your perusal.

@perpendicularai this is awesome! Thank you for sharing! Would it be possible to add it to the README.md via a pull request? That way folks can find the project as a community integration! I'll close this for now.

All done!


05-05 Pipeline pulls from main, add a line to swap to dev

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior
Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

Closed with https://github.com/rh-aiservices-bu/parasol-insurance/pull/34


HBASE-20226: Parallelize region manifest deletes (#2159)

We observed this delete call to be a bottleneck for table with lots of
regions. Patch attempts to parallelize them.
Signed-off-by: Andrew Purtell<EMAIL_ADDRESS>(cherry picked from commit f07f30ae24e6d0e9151bb76aebf78928ec9572e3)

branch-1 precommit is broken, I ran the full test suite in a private jenkins environment. Aside from the usual flakes, no test failures.


Update s3 bindings

The S3 library has made a few fixes to the retry logic. Updating the bindings
accordingly.
Signed-off-by: Stephen J Day<EMAIL_ADDRESS>

LGTM

:cake: LGTM


+r Prevents Validation in Containerised RO Environment.

A couple weeks ago a bump in validation was made (Great work! Holy jeez!) and the below function was added:
https://github.com/dennis6p/adaptive-cards-py/blob/9b0611e7664e6a94ca4d5a92e04751dfc7bc75da/adaptive_cards/validation.py#L516
    def __read_schema_file(self) -> dict[str, Any]:
        with open(
            Path(__file__)
            .parent.joinpath("schemas")
            .joinpath(f"schema-{self.__schema_version}.json"),
            "+r",
            encoding="utf-8",
        ) as f:  # pylint: disable=C0103
            return json.load(f)

https://github.com/dennis6p/adaptive-cards-py/commit/14fdab48502316f5dc158a198f5a5c3793a7e357#diff-7b521b205014b14b49f5feda10e253306aebcda4a4e8173d88b8ac8d91b2100fR516
That +r implies read and write when only read is needed. On RO filesystems (such as with cloud functions or some containers), this causes the below error:
OSError: [Errno 30] Read-only file system: '/layers/google.python.pip/pip/lib/python3.10/site-packages/adaptive_cards/schemas/schema-1.5.json
Please adjust +r to simply r unless there is a need to write in this read function.

Hey @YorithTheDreamer,
thanks for reporting the issue. You're right. Read-only would be sufficient in that case. I will take care on it and change it for the next release. I try to be quick with this one. :)

@YorithTheDreamer The issue is fixed. Thanks a lot for pointing me right to the affected line of code!
Hopefully, this will make it running smooth again on your side with read-only systems. Please let me know, when there are other issues!

Thank you for the rapid response and PR! I'd love to have PR permissions to help contribute some clean up here or there to save you time.

That would be really cool. You can already by just forking and creating a PR from there. I would appreciate your contribution! :)


EXPIRED JOB - OMNICONVERT - UI/UX Team Lead

ENVIRONMENT: production
URL: https://solr.peviitor.ro
Browser: Microsoft Edge
Device: Laptop
OS: Windows 10
STEPS TO REPRODUCE:

Open URL in the browser
Go to "OMNICONVERT" Company
Open the "OMNICONVERT" website
Check the validity of the "UI/UX Team Lead" job

EXPECTED RESULTS:
The job "UI/UX Team Lead"  is valid
ACTUAL RESULTS:
The job "UI/UX Team Lead" is not on the list of open jobs

Scraper dose not extract the jobs correctly.

@TheMrM fix it!!!


Routes not working with app component

Hi There!
I am using the vue-webpack-boilerplate and I was testing the vue-router today and noticed that when I try to use the component tag (   ) such as:
<!DOCTYPE html>

    <meta charset="utf-8">
    <title>VueJS Starter</title>

     <app></app>

I get the following errors:
[vue-router] <router-view> can only be used inside a router-enabled app.
[vue-router] v-link can only be used inside a router-enabled app.
But if I use a regular element with an id of the component   on my index.html in the body element it works just fine.
Is it possible to use component tags with the router? I'd like to because without the component tags it appears that vue-devtools in Chrome will not work properly without them.
thanks!

Sure you can. I am doing it this way:
{
import App from 'components/App'
...
router.start({
components: { App },
store,
}, 'body');

@sqal are you having the v-links and router-view inside the body element then and not the  component?

No no, all my template is inside App component and my html looks just like yours ;)

@sqal Gotcha! For some reason i still get the following errors:
[vue-router] <router-view> can only be used inside a router-enabled app.
[vue-router] v-link can only be used inside a router-enabled app.
Here is my setup:
main.js
    import Vue from 'vue'
    import App from './App'
    import VueRouter from 'vue-router'
    import VueResource from 'vue-resource'
    Vue.use(VueResource)
    Vue.use(VueRouter)

    /* eslint-disable no-new */
    new Vue({
      el: 'body',
      components: { App }
    })

    var router = new VueRouter({
      hashbang: false,
      history: true,
      linkActiveClass: 'active-class'
    })

    // Define some components
    var Foo = Vue.extend({
      template: '
This is foo!
'
    })

    var Bar = Vue.extend({
      template: '
This is bar!
'
    })

    /* Route Map */
    router.map({
      '/home': {
        component: Foo
      },
      'about': {
        component: Bar
      }
    })

    /* Route Redirects */
    router.redirect({
      '*': '/home'
    })

    /* Bootstrap routes to the main component */
    router.start({
      components: { App }
    }, 'body')

index.html
<!DOCTYPE html>

    <meta charset="utf-8">
    <title>VueJS Starter</title>

    <app></app>

App.vue

    new Vue({
      el: 'body',
      components: { App }
    })

You don't need to initialize Vue instance here (and you shouldn't) becouse you vue-router creates it for you in router.start. So just get rid of this part and all should work fine.

@sqal Understood! That worked perfectly. Thank you so much!

@sqal I have got following errors:
[Vue warn]: Cannot find element: #app
here is my code:
main.js
`
import Vue from 'vue'
import VueResource from 'vue-resource'
import VueRouter from 'vue-router'
import App from './App'
import Register from './components/Register'
import Login from './components/Login'
Vue.use(VueResource)
Vue.use(VueRouter)
const router = new VueRouter()
router.map({
'/register': {
component: Register
},
'/login': {
component: Login
}
})
router.start(App, '#app')
`
App.vue
`

REGISTER
LOGIN

`

@xwjgo Please use our forum to ask questions.


Feat/explorer client

@OmarElawady good changes! Well done :ship:

Still woking on zdb password encryption

@LeeSmet
Pushed the zdb password encryption part (verified by matching it with the js-sdk one with the same nonce)


kubeconfig: create .kube dir if it doesn't exist

Fixes issue that if you don't have ~/.kube dir on local machine it will fail copying kubeconfig from the remote.

Can we please add a test that checks this both happy and sad?


Feature request simulation of camera in gazebo

ISSUE TEMPLATE ver. 0.2.0

Which TurtleBot3 you have?

[ x] Burger
[ ] Waffle
[ ] Waffle Pi

Which SBC(Single Board Computer) is working on TurtleBot3?

[ x] Raspberry Pi 3
[ ] Intel Joule 570x
[ ] etc (PLEASE, WRITE DOWN YOUR SBC HERE)

Which OS you installed in SBC?

[ x] Ubuntu MATE 16.04 or later
[ ] Raspbian
[ ] etc (PLEASE, WRITE DOWN YOUR OS)

Which OS you installed in Remote PC?

[ x] Ubuntu 16.04 LTS (Xenial Xerus)
[ ] Ubuntu 18.04 LTS (Bionic Beaver)
[ ] Linux Mint 18.x
[ ] etc (PLEASE, WRITE DOWN YOUR OS)

Write down software version and firmware version

Software version: [x.x.x]
Firmware version: [x.x.x]

Hello i would like to simulate the raspberry camera of the turtlebot in gazebo unfortunately i dont now where to put the necessary code in the urdf files.
Even a camera similar to the raspberry pi camera would be of great help.
I think i would have to manipulate the two file turtlebot3_burger.gazebo.xacro  and turtlebot3_burger.urdf.xacro. But appending the with the following code didnt work out for me:
In turtlebot3_burger.urdf.xacro:

-0.118317 0.002195 0.004205 -0 -0 -3.12703

0.1

1
0
0
1
0
1

0 0 0 0 -0 0

0
0

1.3962634

800
800
R8G8B8

0.02
300

1
30
1

true
0.0
iris/camera
image_raw
camera_info
camera_link
0.07
0.0
0.0
0.0
0.0
0.0

and in turtlebot3_burger.gazebo.xacro:

0.2
true

0.0
camera_ir
/camera/depth/image_raw
/camera/depth/camera_info
/camera/depth/image_raw
/camera/depth/camera_info
/camera/depth/points
camera_link
0.05
0
0
0
0
0
0
0
0
0
0

Following the guide https://github.com/richardw05/mybot_ws/blob/master/src/mybot_description/urdf/mybot.gazebo i was able to create a camera for my turtlebot simulation

Dear florianspy,
Sorry for the late response. Please open another issue if you have any other additional questions.
Good luck with work,
Ryan


Centralized published dependency resolution

Previously, ivy, maven, and gmm all used separate logic to determine how a given dependency is resolved before publishing. This commit adds a VariantDependencyResolver and VariantDependencyResolverFactory, which all 3 metadata types now delegate to. This effecively allows publication dependency resolution to be pluggable. Now, variant dependency coordinate resolution can be implemented in a metadata-agnostic manner. This allows us to simplify the upcoming changes to imlpement variant-level precision for publication dependency resolution.

@bot-gradle test this

I've triggered the following builds for you. Click here to see all build failures.

PullRequestFeedback build

Has this been run through smoke tests yet?

It should have during test this, which it passed. I'll be running again after QFL finishes

@bot-gradle test this

I've triggered the following builds for you. Click here to see all build failures.

PullRequestFeedback build

Your PR is queued. See the queue page for details.

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


[Bug]: Usage data collection  is getting reset after RHODS upgrade

Is there an existing issue for this?

[X] I have searched the existing issues

Deploy type
OpenDataHub core version (eg. v1.6.0)
Version
Current Behavior
The usage data collection setting is getting reset after the upgrade
Expected Behavior
Usage data collection seeting should not reset
Steps To Reproduce

Launch RHODS dashboard
go to Settings->Cluster setting -> uncheck the Allow collection of usage data under Usage data collection
Save Changes
Upgrade the cluster to latest build

RHODS-11791(Downstream issue)
Workaround (if any)
No response
What browsers are you seeing the problem on?
Firefox, Chrome
Anything else
There was old issue that seemed to have been misplaced and I couldn't locate it yesterday in jira. That's why I decided to open a new one.

@DaoDaoNoCode If you would like to include this issue in the 1.34 Release Notes, please provide the text for the "Known issues" section.

@andrewballantyne If you would like to include this issue in the 1.34 Release Notes, please provide the text for the "Known issues" section.

@bredamc I'm not quite sure how this happens... I have not seen this issue myself, so I'm unsure what to put as a release note.
I think there is something funky happening here... I don't think this is as clear cut as the bug states it is.
There appears to be a release note in the Jira when it was first logged. Maybe go with that?

Thanks @andrewballantyne  -- this issue was actually already documented in 1.33 as RHODS-11791, so I've just updated that section to add a link to this Git issue.

This is still happening when upgrading OpenShift AI 2.4 > 2.5rc2

@jgarciao This seems very odd... We need to figure out what's happening here, because an "upgrade" shouldn't touch any of this unless some other component is over-reaching. I need to see what the setting state is in and ask for more output before and after an upgrade. This field can't just be "reset" without someone intentionally doing something 😕

https://issues.redhat.com/browse/RHOAIENG-1205


Add Etherchain to the extension

This merge request is to add etherchain

I just saw my last pull request and the comment you made i will modify the code ! to modify the url with https://www.etherchain.org/api/gasnow and i will create a new PR


Dany

https://github.com/openid/AppAuth-Android/blob/master/library%2Fjava%2Fnet%2Fopenid%2Fappauth%2FAppAuthConfiguration.java#L105

@Dannyss18, what is this?


ObjectSchema.extend Does not return a schema instance

Prerequisites

[X] I have written a descriptive issue title
[X] I have searched existing issues to ensure the bug has not already been reported

Fastify version
4.x.x
Plugin version
4.2.1
Node.js version
18.0
Operating system
Linux
Operating system version (i.e. 20.04, 11.3, 10)
Manaro 5.10.218 x86
Description
Once you call extend on a schema, the function chain is broken because the extend function returns a plain object rather than an instance of the schema being extended. This breaks the normal usage of the builder pattern implemented by this library.
Link to code that reproduces the bug
https://github.com/fastify/fluent-json-schema/blob/6db9f0ba5bbf49aa266bc44dfb82a8f0c04b7c71/src/ObjectSchema.js#L350
Expected Behavior
One should be able to continue chaining function calls such as without or only, etc after calling extend.
It should be safe to use the schema factory function found on the schema to build out a new instance of the same type.

Thanks for reporting!
Can you provide steps to reproduce? We often need a reproducible example, e.g. some code that allows someone else to recreate your problem by just copying and pasting it. If it involves more than a couple of different file, create a new repository on GitHub and add a link to that.

var fluent = require('fluent-json-schema')
var base = fluent
  .object()
  .prop('one', fluent.number())
  .prop('two', fluent.number())

var extended = fluent.object()
  .prop('three', fluent.number())
  .prop('four', fluent.number())
  .extend(base)

var alternate = extended.without(['one'])

var alternate = extended.without(['one'])
                         ^

TypeError: extended.without is not a function
    at Object.<anonymous> (/home/esatterwhite/dev/js/logdna/auth-provider-service/error.js:12:26)
    at Module._compile (node:internal/modules/cjs/loader:1369:14)
    at Module._extensions..js (node:internal/modules/cjs/loader:1427:10)
    at Module.load (node:internal/modules/cjs/loader:1206:32)
    at Module._load (node:internal/modules/cjs/loader:1022:12)
    at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:135:12)
    at node:internal/main/run_main_module:28:49

Node.js v20.12.2

Thanks for reporting! Would you like to send a Pull Request to address this issue? Remember to add unit tests.


Stylesheet is not loaded

when I visit the website balletboetiek.css is not loaded

The gallery is not loading the stylesheet because it searches for it relative to the current page


 This still needs testing on both SNP and TDX, as I'm not very confident I've got the right bits here.

Once we have this, I wonder if we should make scan() auto-acknowledge the interrupt rather than have the split, and rely on rewind in more paths.

Once we have this, I wonder if we should make scan() auto-acknowledge the interrupt rather than have the split, and rely on rewind in more paths.
Maybe? I'd be hesitant just because rewinding requires pulling in the offloaded state, and doing that too often could hurt perf right?


Add rule for unknown url parameters

Closes https://github.com/toddledev/toddle-internal/issues/1970

You still have to add the rule to the problems.worker.ts file

Lol thank you 😅


Please can we get a GUI for this tool?

This is a very useful tool and not all of us are geeks, please can we just use the tool without running vscode or terminal... Just copy layers into the program, set some parameters with basic or advanced Ui and export to JPG feature too

Excellent idea, and also a time-consuming one ;) I don't say no, but I also can't say if and when I could start work on something like that.

many people are in need of the GUI... even the geeks will appreciate a basic Ui or a web app.


Kubernetes provider PDF does not contain versioned resources

The Kubernetes provider docs PDF does not contain versioned resources like kubernetes_job_v1. As of version v2.7.0 resources will begin to be versioned.

Or the issue for me is at least the file in the dropbox for Kubernetes hasn't been updated since 8/5/20, 10:18:01 PM.


Display CII best practice badge in README

Fixes #989
I believe we now meet all the criteria. Feel free to double check my answers: https://bestpractices.coreinfrastructure.org/en/projects/4173

Codecov Report

Merging #1618 (dcaeb73) into master (9d3d10b) will decrease coverage by 22.90%.
The diff coverage is 19.59%.

@@             Coverage Diff             @@
##           master    #1618       +/-   ##
===========================================
- Coverage   63.31%   40.41%   -22.91%
===========================================
  Files         170      106       -64
  Lines       14250    12907     -1343
===========================================
- Hits         9023     5216     -3807
- Misses       4292     7220     +2928
+ Partials      935      471      -464

Flag
Coverage Δ

kind-e2e-tests
?

unit-tests
40.41% <19.59%> (-0.87%)
:arrow_down:

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

Impacted Files
Coverage Δ

cmd/antrea-agent/agent.go
0.00% <0.00%> (ø)

cmd/antrea-agent/options.go
21.69% <0.00%> (+0.97%)
:arrow_up:

pkg/agent/agent.go
12.20% <0.00%> (-36.51%)
:arrow_down:

pkg/agent/agent_linux.go
0.00% <0.00%> (-100.00%)
:arrow_down:

pkg/agent/cniserver/pod_configuration.go
34.52% <0.00%> (-19.24%)
:arrow_down:

pkg/agent/config/node_config.go
0.00% <0.00%> (-100.00%)
:arrow_down:

...gent/controller/noderoute/node_route_controller.go
32.20% <ø> (-14.26%)
:arrow_down:

.../agent/flowexporter/connections/conntrack_linux.go
28.37% <0.00%> (-0.79%)
:arrow_down:

pkg/agent/proxy/proxier_linux.go
0.00% <0.00%> (-25.00%)
:arrow_down:

pkg/agent/querier/querier.go
66.66% <0.00%> (-21.50%)
:arrow_down:

... and 155 more

/skip-all


fix(PeriphDrivers): Fix dma_revb & nvic-table.c warnings

RevB DMA driver had incorrect pointer syntax. NVIC driver had an extern inside a function. These generates errors in CI workflows on third-party repositories which use later version of ARM GCC, even though the files aren't necessarily used by a build. In particular, this is blocking a PR merge due to failing CI on Adafruit's CircuitPython. Since the fixes are simple, it makes sense to just get rid of the warnings here.
https://github.com/adafruit/circuitpython/pull/9667
Description

Corrected pointer syntax in dma_revb.c. A couple functions pass srcdst as a pointer instead of a struct. In these cases a '->' instead of a '.' must be used to access members.
Moved __isr_vector extern outside of NVIC_SetRAM in nvic_table.c. Later versions of GCC create warnings when externs are placed inside of a function.
Added "void" argument to a function prototype in dma_revb.h

I don't know why the DMA RevB drivers are included in the SDK when they're not used anywhere... and especially since no development has been done on them since around ~2020, I'm not surprised there were warnings in them. They don't follow the convention for hardware revision files either.
I think it's safe to delete these drivers entirely.

Removed DMA RevB drivers in latest commit


Tensorflow models

Hello, thank you for this code, i try to test imageDepthEstimation.py
i  did not find the Tensorflow models, can you please give me the link to download it .
thank you

Check the readme:
https://github.com/ibaiGorordo/HITNET-Stereo-Depth-estimation#tensorflow-models

thank you for your answer, i checked the readme but i didn't found the models

Check the .sh

i have this error with this command :ERROR: Error [WinError 2] Le fichier spécifié est introuvable while executing command git version ERROR: Cannot find command 'git' - do you have 'git' installed and in your PATH?

I see, you need to install git to do that. You can either try to install git and retry it.
Or, if you know where pafy is installed, you can search for backend_youtube_dl.py in your computer and comment the dislike line #self._dislikes = self._ydl_info.get('dislike_count', 0) as in: https://github.com/mps-youtube/pafy/pull/305/files

There seems to be some extra text (Skip to text) at the beginning of backend_youtube_dl.py. Make sure that the only difference is that the dislike count line is commented.


Update README.md

Fix reference to BFTaskCompletionSource

Nice catch! Thanks!
Will merge in after CLA Signed badge appears from FB Bot. :grin:

Here it comes...
@peymano please accept the CLA, so I can merge your changes in :)

Yay!


missing libpbcopper

Operating system
Which operating system and version are you using?
CentOS 6.10
Package name
Which package / tool is causing the problem? Which version are you using, use tool --version. Have you updated to the latest version conda update package?
pb-assembly 0.0.2-py27_0 bioconda
Conda environment
What is the result of conda list? (Try to paste that between triple backticks.)

I installed pb-assembly using bioconda and was able to run pb-falcon but pb-unzip fails at 4-polish/cns-output step. After investigating further it appears the error actually happened at quiver-run phase where variantCaller failed due to missing libpbcopper.so.4.2, error pasted below.
Error message
Paste the error message / stack.
variantCaller --algorithm=arrow -x 5 -X 120 -q 20 -j 24 -r ../../../quiver-split/refs/000000F/ref.fa aln-000000F.bam            -o cns.fasta.gz -o cns.fastq.gz --minConfidence 0 -o cns.vcf
libpbcopper.so.0.4.2: cannot open shared object file: No such file or directory
Traceback (most recent call last):
File "/scicomp/home/bun3/.local/lib/python2.7/site-packages/pbcommand/cli/core.py", line 138, in _pacbio_main_runner
return_code = exe_main_func(*args, **kwargs)
File "/scicomp/home/bun3/.local/lib/python2.7/site-packages/GenomicConsensus-2.3.2-py2.7.egg/GenomicConsensus/main.py", line 340, in args_runner
return tr.main()
File "/scicomp/home/bun3/.local/lib/python2.7/site-packages/GenomicConsensus-2.3.2-py2.7.egg/GenomicConsensus/main.py", line 269, in main
self._algorithm = self._algorithmByName(options.algorithm, peekFile)
File "/scicomp/home/bun3/.local/lib/python2.7/site-packages/GenomicConsensus-2.3.2-py2.7.egg/GenomicConsensus/main.py", line 57, in _algorithmByName
from GenomicConsensus.arrow import arrow
File "/scicomp/home/bun3/.local/lib/python2.7/site-packages/GenomicConsensus-2.3.2-py2.7.egg/GenomicConsensus/arrow/init.py", line 4, in
from . import utils
File "/scicomp/home/bun3/.local/lib/python2.7/site-packages/GenomicConsensus-2.3.2-py2.7.egg/GenomicConsensus/arrow/utils.py", line 11, in
import ConsensusCore2 as cc
File "/scicomp/home/bun3/.local/lib/python2.7/site-packages/ConsensusCore2.py", line 17, in
_ConsensusCore2 = swig_import_helper()
File "/scicomp/home/bun3/.local/lib/python2.7/site-packages/ConsensusCore2.py", line 16, in swig_import_helper
return importlib.import_module('_ConsensusCore2')
File "/scicomp/groups/OID/NCEZID/DSR/BCFB/usr/bin/miniconda2/denovo_asm/lib/python2.7/importlib/init.py", line 37, in import_module
import(name)
ImportError: libpbcopper.so.0.4.2: cannot open shared object file: No such file or directory
To Reproduce
Steps to reproduce the behavior. Providing a minimal test dataset on which we can reproduce the behavior will generally lead to quicker turnaround time!
This was run on the Exanple Test data greg200k-sv2
Expected behavior
A clear and concise description of what you expected to happen.
I expect the test data to finish running

Your version of CC2 is massively outdated. Have you tried running conda update --all? Remember to also update your bioconda channel priorities, as these have changed recently.

just try reinstalling pb-assembly and pb-falcon, using explicit version specifiers.

ok tried that .. and here is what happens.
if i try to run it as is. I get the same missing  library.
if i install pbcopper, the varintCaller seg faults.

$ ldd miniconda2/lib/python2.7/site-packages/_ConsensusCore2.so
	linux-vdso.so.1 =>  (0x00007ffc6d38d000)
	libpython2.7.so.1.0 => /home/dseifert/miniconda2/lib/python2.7/site-packages/../../libpython2.7.so.1.0 (0x00007f4379b92000)
	libstdc++.so.6 => /home/dseifert/miniconda2/lib/python2.7/site-packages/../../libstdc++.so.6 (0x00007f4379857000)
	libm.so.6 => /lib64/libm.so.6 (0x00007f4379546000)
	libgcc_s.so.1 => /home/dseifert/miniconda2/lib/python2.7/site-packages/../../libgcc_s.so.1 (0x00007f4379334000)
	libpthread.so.0 => /lib64/libpthread.so.0 (0x00007f4379117000)
	libc.so.6 => /lib64/libc.so.6 (0x00007f4378d56000)
	libdl.so.2 => /lib64/libdl.so.2 (0x00007f4378b52000)
	libutil.so.1 => /lib64/libutil.so.1 (0x00007f437894e000)
	/lib64/ld-linux-x86-64.so.2 (0x0000559bc6289000)

CC2 does not link to pbcopper anymore. But here's a red flag: your normal bioconda installation seems to be in /scicomp/groups/OID/NCEZID/DSR/BCFB/usr/bin/miniconda2/, yet you're loading pbcommand from /scicomp/home/bun3/.local/. Have you mucked with user-site dir? See my reply in https://github.com/PacificBiosciences/pbbioconda/issues/6#issuecomment-419661454 for a fix.

No feedback, I assume this is fixed with the hint I have in the previous comment.


$250 Workspace - Checkmark icon not displayed after copying email

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel! https://www.upwork.com/jobs/~01228c64f5fadb4c43

Action Performed:

Launch the app
Log in with expensifail account
Select any Workspace
Go to Capture Receipts and Click on Copy icon by the<EMAIL_ADDRESS>Go to Manage your bill and Click on Copy icon by the<EMAIL_ADDRESS>
Expected Result:
Checkmark should be displayed after copying email.
Actual Result:
Empty space displayed after copying email
Workaround:
None needed. Visual issue.
Platform:

Where is this issue occurring?

Android

Version Number: 1.1.7-22
Reproducible in staging?: Yes
Reproducible in production?: No (New feature)
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation

https://user-images.githubusercontent.com/44479856/137426775-80765abc-4eda-4ff9-aab4-c1af3c8fc887.mp4
Expensify/Expensify Issue URL:
Issue reported by: Applause
Slack conversation:
View all open jobs on GitHub

Triggered auto assignment to @MonilBhavsar (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

Confirmed, it's occurring on Android only

Issue reproducible during KI retests.

Working on it

Some of the investigations
What's the Issue
On Android, after pressing on clipboard icon, the checkmark icon is rendered outside of the screen.
Looking at the Component tree, SVG Component is at same level as that of Text component, whereas in iOS SVG is wrapped inside Text component.
Looking at the SVG properties in Android, the pivot property resets the x and y coordinates causing this issue
https://user-images.githubusercontent.com/32012005/141307947-8c1d90b2-579e-45d4-a5e7-f1d015f70cd5.mov
In iOS, SVG is wrapped inside Text component not causing the issue
https://user-images.githubusercontent.com/32012005/141307979-65dc82b2-c382-4eae-8708-9392b609b737.mov
thread https://expensify.slack.com/archives/C03TQ48KC/p1635317496207600

@MonilBhavsar I'm still a bit confused why we can't just replace the outer Text component here with a Pressable or TouchableOpacity. We should be able to figure out the necessary styles to get it to render in the correct position. Can you show what you've tried so far?

I see now why we can't just replace the outer Text component, and it's because the entire CopyTextToClipboard component is rendered inside a parent Text component.

I've done a bit more digging and I think what's most curious in this case is that the SVG image displays in the correct location when the page first loads, but when we change to a different SVG the calculated position changes.
Another thing I've found is that React Native has a separate native component for images nested within text: https://github.com/facebook/react-native/commit/a0268a7bfc8000b5297d2b50f81e000d1f479c76
I'm not sure, but I'm starting to think this bug might need to be fixed in react-native-svg, but the primary maintainer is unlikely to be able to address it. So I'd suggest that our next step should be to keep digging and create a detailed issue in react-native-svg explaining the problem and see if anyone in the community has ideas.
We can also consider making this issue external to leverage our own contributor community.

Triggered auto assignment to @laurenreidexpensify (External), see https://stackoverflow.com/c/expensify/questions/8582 for more details.

Thanks @roryabraham for looking into it. Making it external to get views from other contributors.

Oh gosh sorry this got buried when I was OOO - adding to Upwork now

https://www.upwork.com/jobs/~01228c64f5fadb4c43

Triggered auto assignment to @Luke9389 (Exported), see https://stackoverflow.com/c/expensify/questions/7972 for more details.

@laurenreidexpensify I believe https://github.com/Expensify/App/pull/6271 address it...

I checked the PR and the root cause of this issue is gone via wrapping the icon in View thus #6271 indirectly solves this and thus we can close this as soon as that PR is merged or let QA run it through test cycles and it will be automatically closed.

@parasharrajat So I fixed this one? When I saw this happening I though it was my fault 😄
@laurenreidexpensify if it fixes, do I get paid?

if it fixes, do I get paid?

In my case, it depends if you had to do some extra work. But seems you just came to know today that you fixed it eventually.. :sweat_smile:

@sidferreira you'll get paid for the scope of #6271 - I don't see any additional code pushed directly connected to this issue knowingly, so we would likely close this without further reimbursement.

@laurenreidexpensify well, worth trying :D Weird that I didn't see this issue (even tho it was created with #5783 which I fixed)

Ok, so I think we can just go ahead and close this then. @laurenreidexpensify do you agree?

Let's go with @parasharrajat's suggestion

close this as soon as that PR is merged or let QA run it through test cycles and it will be automatically closed.

i've removed help wanted and placed it on hold and we can wait for 6271 to be merged and take it from there

6271 is still in review, so no update yet

6271 is still on staging and hasn't been deployed yet

Issue not reproducible during KI retests. (First Week)

Triggered auto assignment to @bfitzexpensify (External), see https://stackoverflow.com/c/expensify/questions/8582 for more details.

@bfitzexpensify i'm OOO quite a bit over next week moving house, so handing this one off while I"m away thanks!

7 days since https://github.com/Expensify/App/pull/6271 (to fix https://github.com/Expensify/App/issues/5873) was deployed,  with no regressions. Given this also wasn't reproducible during KI tests, I think we're good to close this out now.


Fatal error: Class 'Google_Service' not found

Error:
Fatal error: Class 'Google_Service' not found in /###/Google/Service/Analytics.php on line 32

Make sure you have setup the include path properly (https://developers.google.com/api-client-library/php/start/installation) and included the autoloader. Should be handled for you if using composer.


Cannot use WSL.exe to invoke Linux commands in PowerShell

Version
Microsoft Windows [Version 10.0.19043.1165]
WSL Version

[X] WSL 2
[ ] WSL 1

Kernel Version
5.10.16
Distro Version
Ubuntu 20.04
Other Software
No response
Repro Steps

Open Windows terminal
Open pwsh tab
Run wsl.exe echo 'hello'

Expected Behavior
It should work and get hello back
Actual Behavior
No result returned.
When I open powershell as admin it works, but do not want to be running powershell as admin(least privilege).
There is no mention in documentation I believe that you need to be running as admin
Diagnostic Logs
n\a

I have same issue.

Able to reproduce.

WT 1.12.2931.0
PS 5.1.22000.282
Windows 22000.282

/logs

I have same issue.

My issue is resolved.    From Add & Remove Features, selected Windows Subsystem for Linux and clicked Uninstall, then rebooted.     Then retried wsl.exe echo 'hello', which worked!   Must have been some type of installation / update conflict.

My default distribution had been set to  docker-desktop-data  which resulted from installing docker desktop.
I swithed back to ubuntu as default and working now: wsl --distribution Ubuntu-20.04 --user xxx
wsl --list --verbose
  NAME                   STATE           VERSION
* Ubuntu-20.04           Running         2
  docker-desktop-data    Stopped         2
  docker-desktop         Stopped         2

wsl.exe echo 'hello'
hello

Will follow up with the docker folks to see if they can make a better error message if one of their distros gets set to the default.

FYI there was no error message, it just returned.  Powershell $Error was empty


[HUDI-8447] Claiming RFC-83 for Hudi data is ordered by primary key

Change Logs
The process of sorting Hudi's base file and log file data by primary key is as follows:

During the delta commit phase, the data is sorted.
During the compaction phase, the data is sorted and merged using merge sort.

Delta Commit: The batch data written is sorted by primary key
Compaction: Compaction with Merge Sort Implementation link
Impact
Sorting the base file and log file data in Hudi offers the following benefits:

Enhancing Hudi Reader Performance: After sorting the data, merge sort can be used to read the data, thereby avoiding the large memory usage or disk IO overhead associated with Map-based methods.
Improving Compaction Performance: Sorted data can also utilize merge sort during the merging process, reducing the large memory usage and disk IO overhead required by Map-based methods.
Supporting MDT Introduction of Primary Key Index: Sorted data facilitates the introduction of spare index similar to those in ClickHouse, thus improving the query efficiency of Hudi Reader.

Performance Comparison After Introducing Ordered Hudi Data (Including Compaction with Merge Sort and Log Compaction with Merge Sort), Primary Key Index and Secondary Index: Performance Comparison link
Risk level (write none, low medium or high below)
medium
Documentation Update
none
Contributor's checklist

[x] Read through contributor's guide
[x] Change Logs and Impact were stated clearly
[x] Adequate tests were added if applicable
[x] CI passed

the PR design content has been submitted to https://github.com/apache/hudi/pull/11793


Breaks posh-git file completion

There should be something like this before (or after) printing the prompt.
$Global:GitStatus = Get-GitStatus

Without this, the file statuses are never updated in posh-git and so autocompletion for files (e.g. "git add ....") doesn't work.
e.g. try:
echo foo > foo
git add <TAB>

This should complete to foo automatically.

Good find. I hardly ever add files manually so I didn't notice this yet. I'll see if I can fix this this evening.

Thanks!

Yw!


deps: update mbta_metro to minor version 1

Summary of changes
Updates mbta/mbta_metro to the version which references mbta/rider-design-system which references the design tokens created in Figma by the design system team.
While individual components are still being finalized, this gives us a mechanism for pushing updates to them via the design system. However, the button component from Metro is already updated with this PR as well.
Asana Ticket: "Get trip suggestions" button should match dotcom primary button styling
The updated <.button> component definition with styling via this Tailwind plugin definition.

It works!!


Crosshair not appearing after installing imglab.exe

Hi Davisking,
I compiled the imglab software on my computer using Visual Studio and created an imglab.exe file. The software opens up properly when I run  the imglab -c command with a .xml file and  dataset path from the command window, but the cross-hairs for drawing bounding boxes are not working or appearing on the GUI while I press the shift key. I have tried using the arrow keys to surf through the image directory and that works. No error messages appear.
The version of Dlib that I am using is dlib-19.13, I downloaded it directly from http://dlib.net/. I am working on Windows 10, 64 bit. The C and CXX compiler I used are both the MSVC  version 19.0.24210.0.

I have no idea.  It works fine for me with the same setup as you describe.
You probably just have an old copy by accident.

I just downloaded the most recent version of dlib from your github page. I recompiled dlib using the same method and the issue was fixed! Thank you!


Things like invite/go to hideout/say "ty" dont work

Pretty much title. Loving the overlay, used it for 4-5 leagues, but now it is just not working. What happened?

Having the same issue, would like to know if there is a fix

Problem: My buttons (invite/trade/kick) don't work.
Fix: Don't run PoE as admin. If that didn't help use alternative launcher (run it as admin!). Keep in mind Steam by default runs PoE as admin, so you'll most likely need the alternative launcher to compensate.
This fixed the problem for me.

Hey, this is from troubleshooting:

Problem: Buttons don't work (if you use poe trade macro)
Fix: Uncheck "Put tooltip results on clipboard" in the iteminfo tab in poe trade macro.

Are you using Trade Macro? I'm using MercuryTrade like always, and it still works for me with no changes.


Script ignores my imput for temp files delete...

First I want to thank you for the script 😁
Okay. I ran the Sophia script version 5.3 on Windows 10 20H2 and when it comes to temp files delete, if I skip OR delete and then skip (when some files are locked by some other app - script said) it just circles back to beggining of temp files deletion even when I confirm S for skip, it says that it is skipped, but I could not proceed foward.

Thank you)
I've already fixed this yesterday and uploaded a new vesion.
Redownload the archive!
https://github.com/farag2/Windows-10-Sophia-Script/issues/101#issuecomment-744385383

Thank you 😁

All OK?

Will try it in next 12 hours and I will let you know 😁


The README requires significant and ongoing updates as it is currently outdated.

I have updated the README by adding more details and removing any references to a storehouse.

I have updated the README by adding more details and removing any references to a storehouse.

The README has been updated.


Option to keep full name in {@link} output

Search Terms
link
Problem
We have many links in our documentation, some which link to members on the same page, others which link to other pages.
{@link foo}
{@link Bar.baz}
Currently, the Bar.baz link is shortened to just baz in the output, which is less informative when linking to another class. It would be nice if there was an option to allow {@link} tags to keep the full name in output.
This also doesn't appear to be easy to change with a plugin since the InlineDisplayTag.text is set in https://github.com/TypeStrong/typedoc/blob/master/src/lib/converter/comments/linkResolver.ts. That means it's hard to determine the original link text if you wanted to get the full name from the InlineDisplayTag.target.
Maybe it could be done with a plugin that appended a '|' to all links including '.'? Massaging input like that doesn't seem great though.
Suggested Solution
Add an option, maybe 'fullNameLinks', that keeps the full name if the @link tag includes '.' or '#'.

For now, I've worked around this like so:
export class MyPlugin {
  initialize(app: Application) {
    // retain full link names
    app.converter.on(
      Converter.EVENT_RESOLVE_END,
      (context: Context) => {
        const project = context.project;
        for (const reflection of Object.values(project.reflections)) {
          if (reflection.comment) {
            MyPlugin.expandLinks(reflection.comment, reflection);
          }
        }
      }
    );
  }

  /**
   * Save full friendly name as tsLinkText so we can retain full name links.
   * This prevents links like "Foo.bar" from rendering as just "bar".
   * Only applies to links that include "." or "#", not local links.
   * Does not apply to links including "|", which indicates another name is supplied.
   * @param comment
   * @param reflection
   */
  static expandLinks(comment: Comment, reflection: Reflection) {
    function mapParts(parts: CommentDisplayPart[]) {
      return parts.flatMap((part) => {
        if (part.kind === "inline-tag" &&
            (part.tag === "@link" || part.tag === "@linkcode" || part.tag === "@linkplain") &&
            (part.text.includes('.') || part.text.includes('#')) &&
            !part.text.includes('|') &&
            part.target instanceof ReflectionSymbolId) {
          const tsTarget = reflection.project.getReflectionFromSymbolId(part.target);
          if (tsTarget) part.tsLinkText = tsTarget.getFriendlyFullName();
        }
        return part;
      });
    }

    comment.summary = mapParts(comment.summary);
    for (const tag of comment.blockTags) {
      tag.content = mapParts(tag.content);
    }
  }
}

This assumes the links are well-formed and useTsLinkResolution is being used.

It also assumes that all @links are within the same project if running with --entryPointStrategy packages
I think this is a reasonable option to add, maybe also as default behavior...


[Bug] App not showing anything when runner is kid

On app start when a kid's number is entered the app shows just black screen.
After app restart:

Profile screen all fields are empty
Barcode shows just white screen.

Results, events and donate screens seem to be OK

This will be solved with the next release this week.


Article index order

Hi
I would like to know if it is possible to sort the articles in reverse chronological order (older on top) but with the article index in descending order.
Example output would be something like:
"3" "Aug 31" "Article name"
"2" "Sep 01" "Article name"
"1" "Sep 02" "Article name"
currently it shows like so:
"251" "Aug 31" "Article name"
"252" "Sep 01" "Article name"
"253" "Sep 02" "Article name"
If not possible I would like to make it a feature request.
In my opinion it makes more sense and is more readable and easier to use (if you have it in that date order) because it makes it easier to see how many unread articles you have left.

Hi,
It's not possible, no. Furthermore, I believe your argument for adding such a feature is flawed in two ways.

The exact number of unread articles is already displayed in the articlelist title  by default (see "Format Strings" section in the doc.)
Change in indexing doesn't help if you mix read and unread items, or if you're at the end of the feed full of unread articles.

I believe using %u in the articlelist title will solve your problem.

I do have the unread number in the article title but I usually only mark them as read once I've eye browsed through the whole article list, and even if I did mark them as read one by one, you still have to lose concentration by finding it in the feed title. And the article index number as is, doesn't really do much in quickly telling you where you are.
For example.
I have feeds that fill with many articles very fast, so when I'm browsing through pages of them from up to down, would be nice to know where I am.
I guess what I was wishing for was the exact opposite of the "article-sort-order date-desc".
As for number 2, I usually go through articles from older to new, so I don't really have read articles in the middle of unread ones.
Not a big deal of course, I would just prefer to go up to down instead of the opposite (which is what I'm actually using because of this) as it would be faster to go through article titles and I could actually use the N keybind (not really usable if you use date-desc because it marks it as read and then goes one down, instead of one up).

you still have to lose concentration by finding it in the feed title

You can customize the title for the count to stand out. Put it at the very beginning, perhaps?
My issue with your request is that you have a problem and want a custom solution where you already have means to reconfigure Newsbeuter to behave near-perfectly.
Please, set article-sort-order to date-asc, change articlelist title for unread count to stand out, and use N.

I guess I'll stick with that. Thanks for your time.


Extract track-specific test instructions from config/exercise_readme.go.tmpl

Each track needs a file that contains track-specific instructions on how to manually run the tests. The contents of this document are only presented to the student when using the CLI. This file lives at exercises/shared/.docs/tests.md. You almost certainly already have this information, but need to move it to the correct place.
For v2 tracks, this information was (usually) included in the readme template found at config/exercise_readme.go.tmpl. As such, tracks can extract the test instructions from the config/exercise_readme.go.tmpl file to the exercises/shared/.docs/tests.md file.
See https://github.com/exercism/csharp/pull/1557/files for an example PR.
Tracking
https://github.com/exercism/v3-launch/issues/51

I'll take a closer look at https://github.com/exercism/haxe/blob/main/docs/TESTS.md. Including buddy, we're expecting the datetime library for gigasecond and meetup to be installed so that should be added. I'll also add a note about changing xit to it to run a test. For extra credit, I'll see if there's a way we can run all pending tests at once or a particular describe section.


Create tests for Promise export

Will add them whenever I have time to 👍 https://github.com/zeit/micro/pull/336

This was fixed.