bug: Main process exited, code=exited, status=2/INVALIDARGUMENT

Pigeon service crashes sometimes with this error and than starts itself:
Jul 22 21:12:19 vmd94125.contaboserver.net pigeon[19154]: wrapped error: unrecoverable error
Jul 22 21:12:19 vmd94125.contaboserver.net pigeon[19154]: goroutine 1 [running]:
Jul 22 21:12:19 vmd94125.contaboserver.net pigeon[19154]: main.main()
Jul 22 21:12:19 vmd94125.contaboserver.net pigeon[19154]:         /app/cmd/pigeon/main.go:22 +0x85
Jul 22 21:12:19 vmd94125.contaboserver.net systemd[1]: pigeond.service: Main process exited, code=exited, status=2/INVALIDARGUMENT
Jul 22 21:12:19 vmd94125.contaboserver.net systemd[1]: pigeond.service: Failed with result 'exit-code'.
Jul 22 21:12:25 vmd94125.contaboserver.net systemd[1]: pigeond.service: Scheduled restart job, restart counter is at 21.
Jul 22 21:12:25 vmd94125.contaboserver.net systemd[1]: Stopped Pigeon daemon.
Jul 22 21:12:25 vmd94125.contaboserver.net systemd[1]: Started Pigeon daemon.
Jul 22 21:12:25 vmd94125.contaboserver.net pigeon[19653]: chirp chirp

@mesahin001 what version of palomad and what version pigeon are you running?

@mesahin001 what version of palomad and what version pigeon are you running?

palomad version: v0.3.0-alpha
pigeon version: App version: v0.3.1 - Build commit hash: 624fe0ba69c23ee379963087e7f985c635a76ea7

I was able to repro this as well with the following version:
palomad version: v0.3.0-alpha
pigeon version: App version: v0.3.1 - Build commit hash: 624fe0ba69c23ee379963087e7f985c635a76ea7
Jul 23 06:33:29 vultr pigeon[76426]: error 97: insufficient funds for transfer
Jul 23 06:33:29 vultr pigeon[76426]: error 98: insufficient funds for transfer
Jul 23 06:33:29 vultr pigeon[76426]: error 99: insufficient funds for transfer
Jul 23 06:33:29 vultr pigeon[76426]: error 100: insufficient funds for transfer
Jul 23 06:33:29 vultr pigeon[76426]: wrapped error: unrecoverable error
Jul 23 06:33:29 vultr pigeon[76426]: goroutine 1 [running]:
Jul 23 06:33:29 vultr pigeon[76426]: main.main()
Jul 23 06:33:29 vultr pigeon[76426]:         /app/cmd/pigeon/main.go:22 +0x85
Jul 23 06:33:29 vultr systemd[1]: pigeond.service: Main process exited, code=exited, status=2/INVALIDARGUMENT
Jul 23 06:33:29 vultr systemd[1]: pigeond.service: Failed with result 'exit-code'.
Jul 23 06:33:29 vultr systemd[1]: pigeond.service: Consumed 6min 29.346s CPU time.
Jul 23 06:33:34 vultr systemd[1]: pigeond.service: Scheduled restart job, restart counter is at 64.
Jul 23 06:33:34 vultr systemd[1]: Stopped Pigeon daemon.
Jul 23 06:33:34 vultr systemd[1]: pigeond.service: Consumed 6min 29.346s CPU time.
Jul 23 06:33:34 vultr systemd[1]: Started Pigeon daemon.
Jul 23 06:33:34 vultr pigeon[81912]: *chirp chirp*
Jul 23 06:33:34 vultr pigeon[81912]: time="2022-07-23T06:33:34Z" level=info msg="starting relayer" func="github.com/palomachain/pigeon/relayer.(*Relayer).Start" file="/app/relayer/start.go:21"
Jul 23 06:33:45 vultr pigeon[81912]: time="2022-07-23T06:33:45Z" level=info msg="relayer loop" func="github.com/palomachain/pigeon/relayer.(*Relayer).Start" file="/app/relayer/start.go:49"
Jul 23 06:33:45 vultr pigeon[81912]: Error: post failed: Post "http://localhost:26657": read tcp [::1]:43544->[::1]:26657: read: connection reset by peer
Jul 23 06:33:45 vultr pigeon[81912]: panic: post failed: Post "http://localhost:26657": read tcp [::1]:43544->[::1]:26657: read: connection reset by peer
Jul 23 06:33:45 vultr pigeon[81912]: goroutine 1 [running]:
Jul 23 06:33:45 vultr pigeon[81912]: main.main()
Jul 23 06:33:45 vultr pigeon[81912]:         /app/cmd/pigeon/main.go:22 +0x85
Jul 23 06:33:45 vultr systemd[1]: pigeond.service: Main process exited, code=exited, status=2/INVALIDARGUMENT
Jul 23 06:33:45 vultr systemd[1]: pigeond.service: Failed with result 'exit-code'.
Jul 23 06:33:45 vultr systemd[1]: pigeond.service: Consumed 1.438s CPU time.
Jul 23 06:33:50 vultr systemd[1]: pigeond.service: Scheduled restart job, restart counter is at 65.
Jul 23 06:33:50 vultr systemd[1]: Stopped Pigeon daemon.
Jul 23 06:33:50 vultr systemd[1]: pigeond.service: Consumed 1.438s CPU time.
Jul 23 06:33:50 vultr systemd[1]: Started Pigeon daemon.
Jul 23 06:33:51 vultr pigeon[81921]: *chirp chirp*P

Pigeon service crashes sometimes with this error and than starts itself:
Jul 22 21:12:19 vmd94125.contaboserver.net pigeon[19154]: wrapped error: unrecoverable error Jul 22 21:12:19 vmd94125.contaboserver.net pigeon[19154]: goroutine 1 [running]: Jul 22 21:12:19 vmd94125.contaboserver.net pigeon[19154]: main.main() Jul 22 21:12:19 vmd94125.contaboserver.net pigeon[19154]: /app/cmd/pigeon/main.go:22 +0x85 Jul 22 21:12:19 vmd94125.contaboserver.net systemd[1]: pigeond.service: Main process exited, code=exited, status=2/INVALIDARGUMENT Jul 22 21:12:19 vmd94125.contaboserver.net systemd[1]: pigeond.service: Failed with result 'exit-code'. Jul 22 21:12:25 vmd94125.contaboserver.net systemd[1]: pigeond.service: Scheduled restart job, restart counter is at 21. Jul 22 21:12:25 vmd94125.contaboserver.net systemd[1]: Stopped Pigeon daemon. Jul 22 21:12:25 vmd94125.contaboserver.net systemd[1]: Started Pigeon daemon. Jul 22 21:12:25 vmd94125.contaboserver.net pigeon[19653]: chirp chirp

Could you please start your pigeon with "LOG_LEVEL=debug" and print a bit more lines before the "unrecoverable" error comes to screen?

cc @mesahin001

pigeon_log.txt

we have moved on from this error - closing


docs: remove module-metadata.json and commitlint.config.js

:tada: This PR is included in version 1.3.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


No changelog on F-Droid page

Why is there no option to see the changelog before updating?

Is this more of an issue to raise with F-Droid directly, or is there something we could do to improve this?
FYI, we post all the releases and change logs to https://github.com/duckduckgo/Android/releases if you ever need to check what's new before installing from anywhere, including F-Droid.

Is this more of an issue to raise with F-Droid directly, or is there something we could do to improve this?
FYI, we post all the releases and change logs to https://github.com/duckduckgo/Android/releases if you ever need to check what's new before installing from anywhere, including F-Droid.

"Bug fixes and other improvements"
This is not a changelog.
Why not provide information?


Add support for plot(y,keywords)

There is a convenient function plot(y,format) already, which however
doesn't allow to set additional parameters like labels, so add a similar
function that supports keywords.

Thanks for contributing!


replaceRevive: Use switch/case for what's interesting in replacer.

As Greg suggests, at
https://github.com/zulip/zulip-mobile/pull/4348#discussion_r548764896.
Also add some comments / jsdoc.

Thanks for the review! I just pushed a new revision with that commit-message change.

Thanks -- merged!


Main

ADD ISSUE DESCRIPTION HERE
Version: 1.70.1
Commit: 6d9b74a70ca9c7733b29f0456fd8195364076dda
User Agent: Mozilla/5.0 (Linux; Android 10; SAMSUNG SM-G960U) AppleWebKit/537.36 (KHTML, like Gecko) SamsungBrowser/18.0 Chrome/99.0.4844.88 Mobile Safari/537.36
Embedder: github.dev

/needsMoreInfo

Thanks for creating this issue! We figured it's missing some basic information or in some other way doesn't follow our issue reporting guidelines. Please take the time to review these and update the issue.
Happy Coding!


I can't found fontconfig.txt file. and more question.

I would like to use this program because I want to do a Japanese-Korean translation.
I am not good at English. Thank you for your understanding.
First, let me explain the current situation.

I'm trying to translate "Spice and Wolf: My Year With Holo"

I used the "tool extract" command, The work has been completed successfully.
[extract, out_IMG, repack] folders, [bin_output.txt, dat_output.txt, wsb_analysis.txt, and wsb_output.txt] files were extracted.

I already have a table file used by CrystalTile2 and an Icfont12.NFTR file that I have already created.

Now, let me ask you a question.

I would like to know what you wrote in readme.md about fontconfig.txt. For me, the fontconfig.txt file was not extracted. Can you explain in more detail?

Can I use the Icfont12.NFTR file that I have already modified?

Is the method I use appropriate?
-1. create data folder, put the holo.nds file in it.
-2. Open the CMD window, move to the path where the tool.exe file is located, and input command
-3. modify wsb_output.txt, out_IMG files....etc
-4. Rename the out_IMG folder to work_IMG, also renamed *_output.txt files to *_input.txt (bin_output.txt/dat_output.txt/wsb_output.txt)
-5. Open the CMD window, move to the path where the tool.exe file is located, and input command

An error has occurred. What to do?
F:\GameTranslate\SpiceWolfNDS\work>tool repack
OokamiTranslation version 1.5.2
Repacking BIN from data/bin_input.txt ...
Done! Translation is at 0.26%
Applying armips patch ...
[ERROR] Command error C:\Users\SNOWYE~1\AppData\Local\Temp_MEI158762\armips.exe C:\Users\SNOWYE~1\AppData\Local\Temp_MEI158762\bin_patch.asm
Done!
Repacking DAT from data/dat_input.txt ...
5%|████▏                                                                              | 1/20 [00:00<00:00, 34.58it/s]
Traceback (most recent call last):
File "tool.py", line 122, in
File "lib\site-packages\click\core.py", line 829, in call
File "lib\site-packages\click\core.py", line 782, in main
File "lib\site-packages\click\core.py", line 1259, in invoke
File "lib\site-packages\click\core.py", line 1066, in invoke
File "lib\site-packages\click\core.py", line 610, in invoke
File "tool.py", line 62, in repack
File "repack_dat.py", line 68, in run
File "lib\site-packages\hacktools\common.py", line 185, in writeUInt
struct.error: argument out of range
[15116] Failed to execute script tool

I would like to know what you wrote in readme.md about fontconfig.txt. For me, the fontconfig.txt file was not extracted. Can you explain in more detail?

The fontconfig.txt file just takes a list of characters and their replacement, for example:
“={
‘=}

This will replace “ with { and ‘ with } while writing strings.
If you keep all the characters in ASCII space, this is preferred since it will work better with the VWF and automatic wordwrapping.

Can I use the Icfont12.NFTR file that I have already modified?

Yes, you need to put it in a "replace" folder, with the full path being: data/replace/data/font/lcfont12.NFTR

Is the method I use appropriate?

This is correct. You will also need subtitle files for the opening and special message in the data folder, I've attached the English ones here: subfiles.zip

An error has occurred. What to do?

There are a couple of lines that are exported incorrectly from the dat file, and you need to remove them from the dat input file. I believe they all start with L.
If they're the only lines for a file (like pricegossip.dat) you should remove the file line altogether.
Doing that and adding the subtitle files should do the trick.

Thank you for your answer!

There are a couple of lines that are exported incorrectly from the dat file, and you need to remove them from the dat input file. I believe they all start with L.

|(%d → %d)=>

These types of strings are fine. Is it right? (Translation without (%d → %d), %s....etc.)

Another error occurred.
Done! Translation is at 0.00%
Repacking WSB from data/wsb_input.txt ...
  0%|                                                                                | 0/264 [00:00<?, ?it/s]
Traceback (most recent call last):
  File "tool.py", line 122, in <module>
    common.cli()
  File "lib\site-packages\click\core.py", line 829, in __call__
  File "lib\site-packages\click\core.py", line 782, in main
  File "lib\site-packages\click\core.py", line 1259, in invoke
  File "lib\site-packages\click\core.py", line 1066, in invoke
  File "lib\site-packages\click\core.py", line 610, in invoke
  File "tool.py", line 65, in repack
    repack_wsb.run(firstgame)
  File "repack_wsb.py", line 92, in run
    newlen = game.writeShiftJIS(f, newsjis, b1 == 0x95, False, 0, encoding, firstgame)
  File "game.py", line 91, in writeShiftJIS
    f.write(c.encode(encoding))
UnicodeEncodeError: 'shift_jis' codec can't encode character '\uc548' in position 0: illegal multibyte sequence
[2828] Failed to execute script tool

Does this program only support English and Japanese?
I think "shift_jis codec" does not support Korean.

I have already added all Korean characters to fontconfig.txt.
like this:

The file is backwards, should be the other way around, for example:
가=亜
각=唖

And so on

I just changed all txt files to utf-8 and completed the test.
Everything works fine! Thanks!
And... I have one more question.
When the "product description" window appears, can I ignore the length limit of the line?
ex)
JP: 瓜 - 水分たっぷりの大きな瓜。疲労|回復に効果があるとされる食材。
EN: Oriental melon
KR: 참외
瓜(2byte) <-> 참외 (4byte)
From what I tested, it seems to be automatic expanding script space, right?

There are different limitations and workarounds depending on the file you're translating
bin_input: longer lines are moved to different locations of ROM so there's plenty of space for these.
wsb_input: the script and pointers are all rewritten to fit the text so no limits here. This also includes turning a single textbox into multiple ones with >>
dat_input: the files here all have different limits, but there should be plenty of space to fit them, and you should get an error if they don't fit.

Thank you for making such a great tool!
more people can enjoy the game of Spice and wolf in their language!
And thank you so much for your quick response!


Inlude new files in registry of tables dynamically

As ROAPI User
I would like to point to folder containing files (JSON/Parque/etc)
and be able to include new files (as in added after ROAPI start) in query results.

Streaming update is definitely something I wanted to add to roapi as well :)
@jbremmer how are you deploying roapi with corresponding files today?

Option 3 seems rather hard to implement. I expect it needs some form of support from the underlying storage system. I believe both S3 and Azure have something like that, but we probably need a separate implementation for each of those, and there is non-trivial machinery involved, at least in azure
Wrt the other two options:
It also depends on the both the storage format and the table format. For cloud storage, repeatedly updating file listings (parquet, ...) may become expensive, whereas keeping up to date with a delta table is relatively cheap.
As this piece of functionality is required for my use case, I will try to look into this, maybe next weekend.

Having operational endpoint to refresh specific folder, or simply point to new file would be nice workaround too.
@houqp I have bunch of parquet files on local file system (about to move to S3).

Option 3 seems rather hard to implement. I expect it needs some form of support from the underlying storage system. I believe both S3 and Azure have something like that, but we probably need a separate implementation for each of those, and there is non-trivial machinery involved, at least in azure

Yes, this requires support from the underlying storage system or table format. s3 event notification for aws, inotify for local fs, etc. For table formats like delta tables, we can also fall back to simple polling of latest version.

Just for reference, delta has a config (at spark session level), spark.databricks.delta.stalenessLimit which represents a time unit with maximum allowed staleness. Since this is per session, it would be conceptually close to the 1. (query parameter maxAge) above.

@brycefisher the partition support you mentioned got merged into datafusion recently (see https://github.com/apache/arrow-datafusion/pull/1141), although right now it only works for local filesystem. But we should be able to get this support for free once other object stores are added to datafusion.

Regarding Option 3, perhaps ROAPI could expose a REST API endpoint to allow an external system to trigger a refresh. This would allow you to automatically refresh any backing storage going forward without having to ship inotify, sns, pub/sub, eventhub integrations from day one.

I think push based update is also a good feature to add, thanks for the suggestion :)
So to summarize, here are the features we could add to support data refresh in different scenarios:

an additional query parameter, say, maxAge that triggers roapi to rescan / reload the files
a configuration option that defines some sort of update interval
subscribe to table source update event if supported by the table/source type
expose to endpoint/webhook to allow external system to trigger refresh

an additional query parameter, say, maxAge that triggers roapi to rescan / reload the files
a configuration option that defines some sort of update interval
subscribe to table source update event if supported by the table/source type
expose to endpoint/webhook to allow external system to trigger refresh
I here again
Did any of the above four options has implemented? :)

is under review at https://github.com/roapi/roapi/pull/154. The other options haven't been implemented yet, but PRs welcome.

thx bro :)

Thanks to @zemelLeong , we now have option 4 fully implemented with a solid infrastructure to support runtime table updates. option 1, 2 and 3 should be pretty trivia to implement now if we reuse the same infrastructure.

Option 1 is now implemented. Try adding this to your roapi command: -r 3600

I tried -r 3600 with cli, I got error. I use v0.11.3
RUST_BACKTRACE=full ./roapi -r 3600 -c roapi.yaml -d
thread 'main' panicked at /root/.cargo/registry/src/index.crates.io-6f17d22bba15001f/clap_builder-4.5.2/src/parser/error.rs:32:9:
Mismatch between definition and access of `reload-interval`. Could not downcast to TypeId { t:<PHONE_NUMBER>03461539246944466681743340709 }, need to downcast to TypeId { t:<PHONE_NUMBER>67627837999251560468041260107 }

stack backtrace:
   0:          0x21f0404 - std::backtrace_rs::backtrace::libunwind::trace::hcc3ffcf602aeb3f9
                               at /rustc/878c8a2a62d49ca5c454547ad67290a1df746cb5/library/std/src/../../backtrace/src/backtrace/libunwind.rs:104:5
   1:          0x21f0404 - std::backtrace_rs::backtrace::trace_unsynchronized::h4d11f1856594b3de
                               at /rustc/878c8a2a62d49ca5c454547ad67290a1df746cb5/library/std/src/../../backtrace/src/backtrace/mod.rs:66:5
   2:          0x21f0404 - std::sys_common::backtrace::_print_fmt::h56e05411a88a9113
                               at /rustc/878c8a2a62d49ca5c454547ad67290a1df746cb5/library/std/src/sys_common/backtrace.rs:68:5
   3:          0x21f0404 - <std::sys_common::backtrace::_print::DisplayBacktrace as core::fmt::Display>::fmt::h477649c552655cf7
                               at /rustc/878c8a2a62d49ca5c454547ad67290a1df746cb5/library/std/src/sys_common/backtrace.rs:44:22
   4:           0xf2b748 - core::fmt::rt::Argument::fmt::h7c30fefcde82879a
                               at /rustc/878c8a2a62d49ca5c454547ad67290a1df746cb5/library/core/src/fmt/rt.rs:142:9
   5:           0xf2b748 - core::fmt::write::h285c07517a6cc718
                               at /rustc/878c8a2a62d49ca5c454547ad67290a1df746cb5/library/core/src/fmt/mod.rs:1120:17
   6:          0x21c4838 - std::io::Write::write_fmt::h51ef3a306ca7bdee
                               at /rustc/878c8a2a62d49ca5c454547ad67290a1df746cb5/library/std/src/io/mod.rs:1846:15
   7:          0x21f1744 - std::sys_common::backtrace::_print::h6e0667f7ddd71146
                               at /rustc/878c8a2a62d49ca5c454547ad67290a1df746cb5/library/std/src/sys_common/backtrace.rs:47:5
   8:          0x21f1744 - std::sys_common::backtrace::print::h28fd6df394d2cf3d
                               at /rustc/878c8a2a62d49ca5c454547ad67290a1df746cb5/library/std/src/sys_common/backtrace.rs:34:9
   9:          0x21f1008 - std::panicking::default_hook::{{closure}}::h4817439a46a688e4
                               at /rustc/878c8a2a62d49ca5c454547ad67290a1df746cb5/library/std/src/panicking.rs:272:22
  10:          0x21f1fa8 - std::panicking::default_hook::hdae94c34a045410c
                               at /rustc/878c8a2a62d49ca5c454547ad67290a1df746cb5/library/std/src/panicking.rs:292:9
  11:          0x21f1fa8 - std::panicking::rust_panic_with_hook::h92499ddc3613d759
                               at /rustc/878c8a2a62d49ca5c454547ad67290a1df746cb5/library/std/src/panicking.rs:779:13
  12:          0x21f1a80 - std::panicking::begin_panic_handler::{{closure}}::h39b7e0b1891a13bc
                               at /rustc/878c8a2a62d49ca5c454547ad67290a1df746cb5/library/std/src/panicking.rs:657:13
  13:          0x21f19f0 - std::sys_common::backtrace::__rust_end_short_backtrace::h197ffd1b729fcb5d
                               at /rustc/878c8a2a62d49ca5c454547ad67290a1df746cb5/library/std/src/sys_common/backtrace.rs:171:18
  14:          0x21f19e8 - rust_begin_unwind
                               at /rustc/878c8a2a62d49ca5c454547ad67290a1df746cb5/library/std/src/panicking.rs:645:5
  15:           0x43a0a0 - core::panicking::panic_fmt::h6a52dd3ef190e5c7
                               at /rustc/878c8a2a62d49ca5c454547ad67290a1df746cb5/library/core/src/panicking.rs:72:14
  16:          0x1e7c5bc - roapi::config::get_configuration::h77884271ac4503f5
  17:           0x7fb4a4 - roapi::main::{{closure}}::h65cb304f36bb1924
  18:           0x7f7a2c - roapi::main::hf8ece59c33f0fcdc
  19:           0x5de790 - std::sys_common::backtrace::__rust_begin_short_backtrace::h1949da3a922dd346
  20:           0x5e0694 - std::rt::lang_start::{{closure}}::h10b75704edabce31
  21:           0x7fe4f4 - main```


Detect npm client in SummaryReporter

Summary
Fixes https://github.com/facebook/jest/issues/3262
Test plan
$ npm test
Snapshot Summary
 › 1 obsolete snapshot found, run with `npm test -- -u` to remove them.

$ yarn test
Snapshot Summary
 › 1 obsolete snapshot found, run with `yarn test -- -u` to remove them.

Awesome, thanks for fixing those small things :)

Codecov Report

Merging #3263 into master will decrease coverage by 0.65%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #3263      +/-   ##
==========================================
- Coverage   64.25%   63.59%   -0.66%
==========================================
  Files         175      175
  Lines        6435     6434       -1
  Branches        4        4
==========================================
- Hits         4135     4092      -43
- Misses       2299     2341      +42
  Partials        1        1

Impacted Files
Coverage Δ

packages/jest-cli/src/reporters/SummaryReporter.js
26.66% <0%> (-0.46%)
:arrow_down:

packages/jest-message-util/src/index.js
0% <0%> (-100%)
:arrow_down:

packages/jest-matcher-utils/src/index.js
70.27% <0%> (-28.38%)
:arrow_down:

packages/jest-message-util/src/messages.js
72.36% <0%> (-14.48%)
:arrow_down:

.../jest-message-util/src/separateMessageFromStack.js
80% <0%> (-10%)
:arrow_down:

packages/jest-mock/src/index.js
89.54% <0%> (-2.28%)
:arrow_down:

...s/jest-editor-support/src/parsers/BabylonParser.js
98.93% <0%> (-0.02%)
:arrow_down:

packages/jest-cli/src/watch.js
64.61% <0%> (ø)
:arrow_up:

packages/jest-config/src/normalize.js
89.54% <0%> (ø)
:arrow_up:

...est-test-typescript-parser/src/TypeScriptParser.js
100% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

Lovely!


[owners] Better link to status message

For AMP this might be an OK link
https://github.com/ampproject/amphtml/blob/master/GOVERNANCE.md#owners
But we should probably have a dedicated doc in https://github.com/ampproject/amphtml/tree/master/contributing

I'm not clear on what the "status message" is referring to

@erwinmombay Can you add some context on this? I'm not clear on what needs to be updated

Closing as obsolete/duplicate of #282


Add boba to toolbox

@maxwofford fix your icon!

This wasn't supposed to be merged yet! I'm still deciding on the right icon to use

AH! Sorry, was cleaning and figured it was ready.


IndexError: index out of range in self

From now on, we recommend using our discussion forum (https://github.com/rusty1s/pytorch_geometric/discussions) for general questions.
❓ Questions & Help

I am unable to fix this error.data_list_1 is list in which I have Data type object with edge_attr , edge_index, pos and y. List having only 2 Data object. After calling my Net module , I got output for 1st data in list but for 2nd got index error.
Simple.ipynb - Colaboratory.pdf

Please remove the cached=True option from the GCNConv.

Thank you very much sir. Its solved.


Add ente-auth extension

Description
Ente Auth TOTP generator to display TOTP codes. This does not use any API exposed from the upstream project. You have to import your secrets using the import subcommand built into the extension. See screenshots below.
Screencast

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

This could be the replacement for my Authy extension. Thank you for your work. Cant wait to test it.

Love this.
Code looks good to me

@pernielsentikaer I appreciate the feedback. Any else I should change besides error handling?

I don't think so but @S33G mentions a few things too

@S33G do you mind checking again 🙂

The updated changes look good to me! 🚀


Added my drumKit project.

Can you please merge it using the HacktoberFest-accepted label?

Sorry, can you please, I am new to github?
Do I need to do the whole process again?

@bhaveshlohana


take shouldn't do abort inside a data read

This change fixes problem mentioned here: https://github.com/dominictarr/pull-stream/issues/26#issuecomment-138457876
Don't abort the source stream unless you are gonna cb end. if it's data, you should wait for one more read to abort. This allows you to cb the error, if one occurs during the abort.

@regular this is what i'm thinking

Yes, I see, that looks good. In an attempt to summarise (--verbosity=3):
Rules for implementing read in a through stream:
1) if called with true (meaning downstream, i.e. the sink has enough data)

just forward the exact read() call to the upstream chain to inform everyone.
switch to a new mode, where any attempt to read() is answered with cb(true)

2) if instead we have enough data

initiate an abort by calling read(true) (causing the aforementioned behaviour (1) upstream)

make sure to pass any potential err (failure during abort) down the downstream chain
if all goes well, err will just be the value true
both cases will cause the behaviour (3) described below downstream

3) if, during a read(null, cb) from upstream (i.e. the Source), our callback is called with err !== null (meaning upstream, i.e. the Source, is either exhausted or has a problem)

forward that exact callback towards the downstream chain (towards Sink) to inform everybody.
switch to a new mode, where any attempt to read() is answered with cb(err)  (the err we received)

In none of the above cases data is flowing!
4) If data is flowing (normal operation: read(null, cb) causes cb(null, data)

forward data downstream (towards the Sink)
do none of the above!

There either is data flowing (4) OR you have the error/abort cases (1-3), never both.
Did I miss anything?
And now, excuse me, I have to go fix some code in pull-generate :)

yeah, that sounds about right. I got a bit confused in places though so i'm gonna merge this and add what you wrote (more or less) to the documentation.

merged into 2.28.2


How to print or save idtoke,access token and refresh token after authentication?

Hi,
i am using this method to authenticate
u = Cognito('your-user-pool-id','your-client-id',
username='bob')
u.authenticate(password='bobs-password')
It's not giving any error,but after that it's not printing anything,not even return value of u.authenticate()
How to print idtoken,access token and refresh token?

I think you can access it by u.id_token, u.access_token, u.refresh_token

Hi below snippet will work for you
u = Cognito('your-user-pool-id','your-client-id',
username='bob')
res = u.authenticate(password='bobs-password')
print res.__dict__


runner: gracefully fail if failing to get url path

@evilstreak

@all-contributors please add @sirupsen for code


Cache text property in TextInput

This request aims to speedup calls of text property by caching it.
Attribute _lines_flags isn't converted to ListProperty as it is always set before _lines property and they work together to produce text. Converting it to property can introduce bugs because text value will get rebuild first on _lines_flags and then on _lines changes, making a space for possible bug between two rebuilds.
Internal calls self._get_text(encode=False) are replaced with self.text because self.text will now return cached value which is built with encode=False.

@tito @matham Can you review this pull request?

I'm not familiar with the textinput code. @akshayaurora is probably the person to review this.


chore: remove write_to_firestore

Description
Removes write_to_firestore magic function
Integration tests
Adapter to test:

postgres
bigquery

Python version to test:

3.8
3.9

Tests ran earlier before the action that updated the lockfile: https://github.com/fal-ai/fal/actions/runs/3901548260


Response consumtion is broken

Calling Response:serialize raises the following
luncheon/response.lua:275: attempt to get length of a nil value (field 'body')
calling the iterator from Response:iter raises the following
luncheon/utils.lua:66: bad argument #1 to 'find' (string expected, got nil)

This is due to a previous assumption that stream based requests and responses are not consumed via serialization which isn't great ergonomics


Where are the example files?

Where can one find the files listed in README.MD:
app-with-list-view.js
app-with-tab-view.js
app-with-vmodel.js

Fixed in #9. Closing


[CLN] Deprecate blockfile get_at_index()

Description of changes
Summarize the changes made by this PR.

Improvements & Bug fixes

...

New functionality

...

Test plan
How are these changes tested?

[ ] Tests pass locally with pytest for python, yarn test for js, cargo test for rust

Documentation Changes
Are all docstrings for user-facing APIs updated if required? Do we need to make documentation changes in the docs repository?

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#3332  👈 (View in Graphite)
#3331
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Merge activity

Dec 20, 1:45 PM EST: A user started a stack merge that includes this pull request via Graphite.


Fix hist1d interface

Add an edge case to convert 1d hists. This is needed for the CES fit.

Can you give an example file for testing?

@dachengx Here is the test that I did locally:
import pytest
import numpy as np
import multihist as mh
import h5py
from inference_interface import template_to_multihist, multihist_to_template

def test_template_to_multihist_1d():
    # Create a sample 1D histogram
    bins = np.linspace(0, 10, 11)
    hist = mh.Hist1d(bins=bins)
    hist.histogram = np.random.rand(10)

    # Save the histogram to a template file
    file_name = "test_template_1d.h5"
    multihist_to_template([hist], file_name, histogram_names=["hist"], hist_to_store=mh.Hist1d)

    # Load the template file and compare the loaded histogram
    loaded_hist = template_to_multihist(file_name, hist_name="hist", hist_to_read=mh.Hist1d)
    assert np.allclose(loaded_hist.histogram, hist.histogram)
    assert np.allclose(loaded_hist.bin_edges, hist.bin_edges)

def test_template_to_multihist_dd():
    # Create a sample multi-dimensional histogram
    bins = [np.linspace(0, 10, 11), np.linspace(0, 5, 6)]
    hist = mh.Histdd(bins=bins)
    hist.histogram = np.random.rand(10, 5)

    # Save the histogram to a template file
    file_name = "test_template_dd.h5"
    multihist_to_template([hist], file_name, histogram_names=["hist"], hist_to_store=mh.Histdd)

    # Load the template file and compare the loaded histogram
    loaded_hist = template_to_multihist(file_name, hist_name="hist", hist_to_read=mh.Histdd)
    assert np.allclose(loaded_hist.histogram, hist.histogram)
    assert all(np.allclose(b1, b2) for b1, b2 in zip(loaded_hist.bin_edges, hist.bin_edges))

def test_multihist_to_template_1d():
    # Create a sample 1D histogram
    bins = np.linspace(0, 10, 11)
    hist = mh.Hist1d(bins=bins)
    hist.histogram = np.random.rand(10)

    # Save the histogram to a template file
    file_name = "test_template_1d.h5"
    multihist_to_template([hist], file_name, histogram_names=["hist"], hist_to_store=mh.Hist1d)

    # Load the template file and compare the loaded histogram
    with h5py.File(file_name, "r") as f:
        assert np.allclose(f["templates/hist"][:], hist.histogram)
        assert np.allclose(f["bins/0"][:], hist.bin_edges)

def test_multihist_to_template_dd():
    # Create a sample multi-dimensional histogram
    bins = [np.linspace(0, 10, 11), np.linspace(0, 5, 6)]
    hist = mh.Histdd(bins=bins)
    hist.histogram = np.random.rand(10, 5)

    # Save the histogram to a template file
    file_name = "test_template_dd.h5"
    multihist_to_template([hist], file_name, histogram_names=["hist"], hist_to_store=mh.Histdd)

    # Load the template file and compare the loaded histogram
    with h5py.File(file_name, "r") as f:
        assert np.allclose(f["templates/hist"][:], hist.histogram)
        assert all(np.allclose(f[f"bins/{i}"][:], b) for i, b in enumerate(hist.bin_edges))

I'm a little hesitate to make a test folder for this repo though. It's probably an overkill. What do you think?

Oh sorry, for the tests, let's keep it like this. Thank you very much!!! @yuema137


Updating to 2.9.1 caused failure in deliver step

New Issue Checklist

[x] Updated fastlane to the latest version
[x] I have read the Contribution Guidelines

Issue Description
When using 2.29.0, I didn't have any screenshots or metadata, and everything still worked. After updating to 2.29.1, I get a chomp error, but the script continues to run. At the end, I don't have a binary pushed to the store.
The initial command that's run: fastlane release scheme:"$FOLDERNAME"
It looks like the loop for TRADE_REPRESENTATIVE_CONTACT_INFORMATION_VALUES is still running, event though there aren't any defined in the options?
I was able to update the deliver command in the fastfile to this:     deliver(force: true, skip_screenshots:true, skip_metadata: true)  to get it working.
Complete output when running fastlane, including the stack trace and command used

You can use: --capture_output as the last commandline argument to get that collected for you

the script is run through a lane, but the relevant failures for deliver are here:
[11:26:20]: Login successful

+-----------------------+---------------------------------------------------------------------------------------------------------------+
|                                                        deliver 2.29.1 Summary                                                         |
+-----------------------+---------------------------------------------------------------------------------------------------------------+
| force                 | true                                                                                |
| ipa                   | /Users/membersfirst/Code/Cordova/Projects/Stonebridge Country Club/platforms/ios/Stonebridge Country Club.ipa |
| screenshots_path      | ./fastlane/screenshots                                                                                |
| metadata_path         | ./fastlane/metadata                                                                                |
| app_version           | 1.1.2                                                                                |
| username              |<EMAIL_ADDRESS>                                                                                |
| app_identifier        | com.membersfirst.stonebridgecountryclub                                                                       |
| edit_live             | false                                                                                |
| platform              | ios                                                                                |
| skip_binary_upload    | false                                                                                |
| skip_screenshots      | false                                                                                |
| skip_metadata         | false                                                                                |
| submit_for_review     | false                                                                                |
| automatic_release     | false                                                                                |
| dev_portal_team_id    | M7P78RNDNU                                                                                |
| overwrite_screenshots | false                                                                                |
+-----------------------+---------------------------------------------------------------------------------------------------------------+

[11:26:21]: Making sure the latest version on iTunes Connect matches '1.1.2' from the ipa file...
[11:26:22]: '1.1.2' is the latest version on iTunes Connect
+--------------------+------------------------------------------------------------------------------------------------------------------------+
|                                                                Lane Context                                                                 |
+--------------------+------------------------------------------------------------------------------------------------------------------------+
| DEFAULT_PLATFORM   | ios                                                                                |
| PLATFORM_NAME      | ios                                                                                |
| LANE_NAME          | ios release                                                                                |
| IPA_OUTPUT_PATH    | /Users/membersfirst/Code/Cordova/Projects/Stonebridge Country Club/platforms/ios/Stonebridge Country Club.ipa          |
| DSYM_OUTPUT_PATH   | /Users/membersfirst/Code/Cordova/Projects/Stonebridge Country Club/platforms/ios/Stonebridge Country Club.app.dSYM.zip |
| XCODEBUILD_ARCHIVE | /Users/membersfirst/Library/Developer/Xcode/Archives/2017-05-11/Stonebridge Country Club 2017-05-11 11.25.54.xcarchive |
+--------------------+------------------------------------------------------------------------------------------------------------------------+
[11:26:24]: undefined method `chomp' for nil:NilClass

+------+-------------------------------------+-------------+
|                     fastlane summary                     |
+------+-------------------------------------+-------------+
| Step | Action                              | Time (in s) |
+------+-------------------------------------+-------------+
| 1    | Verifying required fastlane version | 0           |
| 2    | default_platform                    | 0           |
| 3    | gym                                 | 25          |
| 💥   | deliver                             | 5           |
+------+-------------------------------------+-------------+

[11:26:24]: fastlane finished with errors

Looking for related GitHub issues on fastlane/fastlane...

➡  set_trade_representative_contact_information: undefined method `chomp' for nil:NilClass
    https://github.com/fastlane/fastlane/issues/9140 [open] 0 💬
    4 minutes ago

➡  how to get XCode user defined settings in a lane
    https://github.com/fastlane/fastlane/issues/6795 [closed] 11 💬
    30 Jan 2017

🔗  You can ⌘ + double-click on links to open them directly in your browser.
/Users/membersfirst/.rvm/gems/ruby-2.3.0@global/gems/fastlane-2.29.1/deliver/lib/deliver/upload_metadata.rb:253:in `block in set_trade_representative_contact_information': [!] undefined method `chomp' for nil:NilClass (NoMethodError)
	from /Users/membersfirst/.rvm/gems/ruby-2.3.0@global/gems/fastlane-2.29.1/deliver/lib/deliver/upload_metadata.rb:252:in `each'
	from /Users/membersfirst/.rvm/gems/ruby-2.3.0@global/gems/fastlane-2.29.1/deliver/lib/deliver/upload_metadata.rb:252:in `set_trade_representative_contact_information'
	from /Users/membersfirst/.rvm/gems/ruby-2.3.0@global/gems/fastlane-2.29.1/deliver/lib/deliver/upload_metadata.rb:115:in `upload'
	from /Users/membersfirst/.rvm/gems/ruby-2.3.0@global/gems/fastlane-2.29.1/deliver/lib/deliver/runner.rb:62:in `upload_metadata'
	from /Users/membersfirst/.rvm/gems/ruby-2.3.0@global/gems/fastlane-2.29.1/deliver/lib/deliver/runner.rb:21:in `run'
	from /Users/membersfirst/.rvm/gems/ruby-2.3.0@global/gems/fastlane-2.29.1/fastlane/lib/fastlane/actions/deliver.rb:16:in `run'
	from /Users/membersfirst/.rvm/gems/ruby-2.3.0@global/gems/fastlane-2.29.1/fastlane/lib/fastlane/runner.rb:252:in `block (2 levels) in execute_action'
	from /Users/membersfirst/.rvm/gems/ruby-2.3.0@global/gems/fastlane-2.29.1/fastlane/lib/fastlane/actions/actions_helper.rb:50:in `execute_action'
	from /Users/membersfirst/.rvm/gems/ruby-2.3.0@global/gems/fastlane-2.29.1/fastlane/lib/fastlane/runner.rb:230:in `block in execute_action'
	from /Users/membersfirst/.rvm/gems/ruby-2.3.0@global/gems/fastlane-2.29.1/fastlane/lib/fastlane/runner.rb:226:in `chdir'
	from /Users/membersfirst/.rvm/gems/ruby-2.3.0@global/gems/fastlane-2.29.1/fastlane/lib/fastlane/runner.rb:226:in `execute_action'
	from /Users/membersfirst/.rvm/gems/ruby-2.3.0@global/gems/fastlane-2.29.1/fastlane/lib/fastlane/runner.rb:148:in `trigger_action_by_name'
	from /Users/membersfirst/.rvm/gems/ruby-2.3.0@global/gems/fastlane-2.29.1/fastlane/lib/fastlane/fast_file.rb:146:in `method_missing'
	from Fastfile:49:in `block (2 levels) in parsing_binding'
	from /Users/membersfirst/.rvm/gems/ruby-2.3.0@global/gems/fastlane-2.29.1/fastlane/lib/fastlane/lane.rb:33:in `call'
	from /Users/membersfirst/.rvm/gems/ruby-2.3.0@global/gems/fastlane-2.29.1/fastlane/lib/fastlane/runner.rb:49:in `block in execute'
	from /Users/membersfirst/.rvm/gems/ruby-2.3.0@global/gems/fastlane-2.29.1/fastlane/lib/fastlane/runner.rb:45:in `chdir'
	from /Users/membersfirst/.rvm/gems/ruby-2.3.0@global/gems/fastlane-2.29.1/fastlane/lib/fastlane/runner.rb:45:in `execute'
	from /Users/membersfirst/.rvm/gems/ruby-2.3.0@global/gems/fastlane-2.29.1/fastlane/lib/fastlane/lane_manager.rb:52:in `cruise_lane'
	from /Users/membersfirst/.rvm/gems/ruby-2.3.0@global/gems/fastlane-2.29.1/fastlane/lib/fastlane/command_line_handler.rb:30:in `handle'
	from /Users/membersfirst/.rvm/gems/ruby-2.3.0@global/gems/fastlane-2.29.1/fastlane/lib/fastlane/commands_generator.rb:104:in `block (2 levels) in run'
	from /Users/membersfirst/.rvm/gems/ruby-2.3.0@global/gems/commander-fastlane-4.4.4/lib/commander/command.rb:178:in `call'
	from /Users/membersfirst/.rvm/gems/ruby-2.3.0@global/gems/commander-fastlane-4.4.4/lib/commander/command.rb:153:in `run'
	from /Users/membersfirst/.rvm/gems/ruby-2.3.0@global/gems/commander-fastlane-4.4.4/lib/commander/runner.rb:476:in `run_active_command'
	from /Users/membersfirst/.rvm/gems/ruby-2.3.0@global/gems/fastlane-2.29.1/fastlane_core/lib/fastlane_core/ui/fastlane_runner.rb:40:in `run!'
	from /Users/membersfirst/.rvm/gems/ruby-2.3.0@global/gems/commander-fastlane-4.4.4/lib/commander/delegates.rb:15:in `run!'
	from /Users/membersfirst/.rvm/gems/ruby-2.3.0@global/gems/fastlane-2.29.1/fastlane/lib/fastlane/commands_generator.rb:303:in `run'
	from /Users/membersfirst/.rvm/gems/ruby-2.3.0@global/gems/fastlane-2.29.1/fastlane/lib/fastlane/commands_generator.rb:42:in `start'
	from /Users/membersfirst/.rvm/gems/ruby-2.3.0@global/gems/fastlane-2.29.1/fastlane/lib/fastlane/cli_tools_distributor.rb:66:in `take_off'
	from /Users/membersfirst/.rvm/gems/ruby-2.3.0@global/gems/fastlane-2.29.1/bin/fastlane:20:in `<top (required)>'
	from /Users/membersfirst/.rvm/rubies/ruby-2.3.0/bin/fastlane:23:in `load'
	from /Users/membersfirst/.rvm/rubies/ruby-2.3.0/bin/fastlane:23:in `<main>'
	from /Users/membersfirst/.rvm/gems/ruby-2.3.0@global/bin/ruby_executable_hooks:15:in `eval'
	from /Users/membersfirst/.rvm/gems/ruby-2.3.0@global/bin/ruby_executable_hooks:15:in `<main>'

Environment
Please run fastlane env and copy the output below. This will help us help you :+1:
If you used --capture_output option please remove this block - as it is already included there.
✅ fastlane environment ✅
Stack

Key
Value

OS
10.12.4

Ruby
2.3.0

Bundler?
true

Git
git version 2.11.0 (Apple Git-81)

Installation Source
~/.rvm/rubies/ruby-2.3.0/bin/fastlane

Host
Mac OS X 10.12.4 (16E195)

Ruby Lib Dir
~/.rvm/rubies/ruby-2.3.0/lib

OpenSSL Version
OpenSSL 1.0.2j  26 Sep 2016

Is contained
false

Is homebrew
false

Is installed via Fabric.app
false

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

Xcode Version
8.3.2

System Locale

Variable
Value

LANG
en_US.UTF-8
✅

LC_ALL

LANGUAGE

fastlane files:
`./Fastfile`
# Customise this file, documentation can be found here:
# https://github.com/fastlane/fastlane/tree/master/fastlane/docs
# All available actions: https://github.com/fastlane/fastlane/blob/master/fastlane/docs/Actions.md
# can also be listed using the `fastlane actions` command

# Change the syntax highlighting to Ruby
# All lines starting with a # are ignored when running `fastlane`

# If you want to automatically update fastlane if a new version is available:
# update_fastlane

# This is the minimum version number required.
# Update this, if you use features of a newer version
fastlane_version "1.106.0"

default_platform :ios

platform :ios do
  before_all do
    # ENV["SLACK_URL"] = "https://hooks.slack.com/services/..."
    ENV["GIT_URL"] = "https://bitbucket.org/MF_Development/certificates"

    app_identifier = CredentialsManager::AppfileConfig.try_fetch_value(:app_identifier)
    username = CredentialsManager::AppfileConfig.try_fetch_value(:apple_id)
    team_id = CredentialsManager::AppfileConfig.try_fetch_value(:team_id)

  end

  desc "Runs all the tests"
  lane :test do
    scan
  end

  desc "Submit a new Beta Build to Apple TestFlight"
  desc "This will also make sure the profile is up to date"
  lane :beta do
    # match(type: "appstore") # more information: https://codesigning.guide
    gym # Build your app - more options available
    pilot

    # sh "your_script.sh"
    # You can also use other beta testing services here (run `fastlane actions`)
  end

  desc "Deploy a new version to the App Store"
  lane :release do |options|
    # snapshot
    gym(scheme: options[:scheme])
    deliver(force: true)
    # frameit
  end

  desc "Add app to iTunes Connect and Dev Portal and create certificates"
  lane :prepare do |options|
    produce(
      app_name: options[:app_name]
    )
    pem(
      generate_p12: true,
      p12_password: "M3mF1rst",
      save_private_key: true,
      output_path: "certificates"
    )
    pem(
      development: true,
      generate_p12: true,
      p12_password: "M3mF1rst",
      save_private_key: true,
      output_path: "certificates"
    )
    match(
      type: "appstore",
      git_url: ENV["GIT_URL"],
      force_for_new_devices: true
    )
    match(
      type: "development",
      git_url: ENV["GIT_URL"],
      force_for_new_devices: true
    )
  end

  # You can define as many lanes as you want

  after_all do |lane|
    # This block is called, only if the executed lane was successful

    # slack(
    #   message: "Successfully deployed new App Update."
    # )
  end

  error do |lane, exception|
    # slack(
    #   message: exception.message,
    #   success: false
    # )
  end
end

# More information about multiple platforms in fastlane: https://github.com/fastlane/fastlane/blob/master/fastlane/docs/Platforms.md
# All available actions: https://github.com/fastlane/fastlane/blob/master/fastlane/docs/Actions.md

# fastlane reports which actions are used
# No personal data is recorded. Learn more at https://github.com/fastlane/enhancer

`./Appfile`
app_identifier "com.membersfirst.neptunecove" # The bundle identifier of your app
apple_id<EMAIL_ADDRESS># Your Apple email address
team_id "M7P78RNDNU"  # Developer Portal Team ID

# you can even provide different app identifiers, Apple IDs and team names per lane:
# More information: https://github.com/fastlane/fastlane/blob/master/fastlane/docs/Appfile.md

fastlane gems

Gem
Version
Update-Status

fastlane
2.29.1
✅ Up-To-Date

match
1.0.0
✅ Up-To-Date

pem
2.0.0
✅ Up-To-Date

Loaded fastlane plugins:
No plugins Loaded
Loaded gems

Gem
Version

did_you_mean
1.0.0

executable-hooks
1.3.2

bundler-unload
1.0.2

rubygems-bundler
1.4.4

bundler
1.13.6

io-console
0.4.5

CFPropertyList
2.3.5

public_suffix
2.0.5

addressable
2.5.1

babosa
1.0.2

claide
1.0.1

colored
1.2

colored2
3.1.2

highline
1.7.8

commander-fastlane
4.4.4

unf_ext
<IP_ADDRESS>

unf
0.1.4

domain_name
0.5.20170404

dotenv
2.2.1

excon
0.55.0

multipart-post
2.0.0

faraday
0.12.1

http-cookie
1.0.3

faraday-cookie_jar
0.0.6

faraday_middleware
<IP_ADDRESS>

fastimage
2.1.0

gh_inspector
1.0.3

jwt
1.5.6

little-plugger
1.1.4

multi_json
1.12.1

logging
2.2.2

memoist
0.15.0

os
0.9.6

signet
0.7.3

googleauth
0.5.1

httpclient
2.8.3

hurley
0.2

mime-types-data
3.2016.0521

mime-types
3.1

uber
0.0.15

representable
2.3.0

retriable
2.1.0

google-api-client
0.9.28

json
2.1.0

mini_magick
4.5.1

multi_xml
0.6.0

plist
3.3.0

rubyzip
1.2.1

security
0.1.3

slack-notifier
1.5.1

terminal-notifier
1.7.1

unicode-display_width
1.1.3

terminal-table
1.7.3

tty-screen
0.5.0

word_wrap
1.0.0

nanaimo
0.2.3

xcodeproj
1.4.4

rouge
2.0.7

xcpretty
0.2.7

xcpretty-travis-formatter
0.0.4

generated on: 2017-05-11

submitted the same time as another issue. Closing in favor of https://github.com/fastlane/fastlane/issues/9140


Rename to g:loaded_fern from g:fern_loaded

This is mini update PR.
In fern.vim, g:fern_loaded was renamed to g:loaded_fern in fern.vim ... https://github.com/lambdalisue/fern.vim/commit/aad432d2c3247fd4ffc0481a7b45eb5d8483e171.
This PR is following the commit.

that commit seems invalid, I cannot see it. I am also wondering, if it makes sense to use this:
if (get(g:, 'fern_loaded', 0) ||  get(g:, 'loaded_fern', 0)) && get(g:, 'airline#extensions#fern#enabled', 1)

This way it should still be working, even if the user is using an older fern version.

Sorry, I have pasted the wrong url, the corrent url is https://github.com/lambdalisue/fern.vim/commit/ad432d2c3247fd4ffc0481a7b45eb5d8483e1710
I've fixed my code with your advice. Please check.

I'm closing this PR. If you have a problem, open it again or create this again.


Feature: Templates for common page types

Thought it would be good to start a discussion about what page templates the prototype kit could include. For example, this question template seems really useful.
Most of the other templates I can think of are already in examples, so we might want to also discuss if it's worth having them as templates as well/instead.

@sanjaypoyzer this doesn't seem to have attracted suggestions - do you think we could gather this info elsewhere, then record here, or as separate issues? I agree with the gist of this - gathering user needs for page templates in the kit.

Yeah, if there's any other way to collect suggestions we could do that. I'll throw it into Slack and see if gets any more love.

you could try the digital service design mailing list too

There seems to be some interest in the DWP dashboard recently – as various people, including myself, asked for it in the past weeks. Not though sure if this justifies to make it a template just yet.

(But if so, it needs quite some cleanup as the mobile version breaks in the current version. For our version, for the Registers Design Authority, we made some changes and fixed that.)

interesting example, as I'd say it's not useful for a prototype, but there is need for it more widely

Okay, here is a thing I need as a template: An accordion. Although not popular when it comes to accessibility, there are some applications within GOV.UK and I’d probably have to use one now, too. Any chance you might add that? Or is it already in and I overlooked it?

@martin-jordan I don't think we have an agreed accordion pattern yet. Showing the need for one and gathering research on a good pattern would be the next steps.

@martin-jordan Oof, controversial!
There is a view that accordions are generally a sign of bad information architecture. I'm not sure I'm fully in that camp, but I think it's definitely something to be wary of.
In any case, that's probably a discussion for the elements repo.

I know, Sanjay! :/
After Louise just highlighted this accordion in here services programme update, I thought it might be worth a try :)  https://www.gov.uk/service-manual/agile-delivery
Also, am happy to show you my little challenge – maybe you have a better idea how to avoid even using an accordion.


fix: map-guided model curations

Description of the issue:

The MetabolicAtlas website presents beautifully-drawn maps of human metabolism based on humanGEM
During map generation process, some inconsistent cases related to subSystems, compartment and others are noticed, and will be tabulated here for validation and further modification

Current feature/value/output:

Here are a list of reactions whose subSystems need to be checked out based on map analysis:
ALAALACNc
ARACHFATPc
ARACHDFATPtc
CYSAMOe
C4x
C6COAt
CHOLESTle
FMNALKPle
GALFUCGALACGLCGAL14ACGLCGALGLUSIDEtg
GDA1tn
GLYC2P3Pc
GLYGLYCNc
GLYLEUHYDROc
GLYPROPRO1c
GLYSARCNc
HC02220c
HMR_0006
HMR_0007
HMR_0008
HMR_0015
HMR_0016
HMR_0017
HMR_1392
HMR_1919
HMR_2141
HMR_3964
HMR_4184
HMR_7568
LEUGLYHYc
LEULEULAPc
LGNCFATPtc
LINOFATPtc
OCDCAFATPc
OLEICFATPtc
PALFATPtc
PNTEHe
PROGLYPRO1c
PROSTGI2c
PTPATe
SAMHISTAe
TTDCAFATPc
r1146
ATPasel
HMR_2585
HMR_7569

I hereby confirm that I have:

[X] Checked that a similar issue does not exist already

From the original list, the following reactions are no longer presented in reactions.tsv, meaning they have thus been removed from the model: CHOLESTle.
The remaining list is:

MA ID
original ID
Current subsystem

MAR06861
ALAALACNc
Isolated

MAR07807
ARACHFATPc
Transport reactions

MAR09870
ARACHDFATPtc
Transport reactions

MAR07817
CYSAMOe
Transport reactions

MAR04678
C4x
Transport reactions

MAR04722
C6COAt
Transport reactions

MAR08969
FMNALKPle
Transport reactions

MAR08282
GALFUCGALACGLCGAL14ACGLCGALGLUSIDEtg
Transport reactions

MAR12119
GDA1tn
Transport reactions

MAR09958
GLYC2P3Pc
Transport reactions

MAR06898
GLYGLYCNc
Transport reactions

MAR11411
GLYLEUHYDROc
Transport reactions

MAR11414
GLYPROPRO1c
Transport reactions

MAR06915
GLYSARCNc
Transport reactions

MAR10447
HC02220c
Transport reactions

MAR00006
HMR_0006
Transport reactions

MAR00007
HMR_0007
Transport reactions

MAR00008
HMR_0008
Transport reactions

MAR00015
HMR_0015
Transport reactions

MAR00016
HMR_0016
Transport reactions

MAR00017
HMR_0017
Transport reactions

MAR01392
HMR_1392
Transport reactions

MAR01919
HMR_1919
Transport reactions

MAR02141
HMR_2141
Transport reactions

MAR03964
HMR_3964
Transport reactions

MAR04184
HMR_4184
Transport reactions

MAR07568
HMR_7568
Transport reactions

MAR08979
LEUGLYHYc
Transport reactions

MAR07149
LEULEULAPc
Transport reactions

MAR09894
LGNCFATPtc
Transport reactions

MAR07152
LINOFATPtc
Transport reactions

MAR08983
OCDCAFATPc
Transport reactions

MAR07156
OLEICFATPtc
Transport reactions

MAR07157
PALFATPtc
Transport reactions

MAR08986
PNTEHe
Transport reactions

MAR07193
PROGLYPRO1c
Transport reactions

MAR10395
PROSTGI2c
Transport reactions

MAR08990
PTPATe
Transport reactions

MAR11777
SAMHISTAe
Transport reactions

MAR08996
TTDCAFATPc
Transport reactions

MAR02302
r1146
Cholesterol metabolism

MAR07799
ATPasel
Transport reactions

MAR02585
HMR_2585
Transport reactions

MAR07569
HMR_7569
Transport reactions

What would be the suggested plan to resolve the current issue for these reactions?

@mihai-sysbio thanks for updating the table. This issue probably can be closed.
PS - was the 2D map disabled on Met Atlas?


Is it possible to use without npm?

Is there a CDN link that can be used directly in browser?

you can always use https://unpkg.com.
Regular<EMAIL_ADDRESS>and minified<EMAIL_ADDRESS>


:pencil: Added MIT License

TLDR:
Anyone can use this software, make modifications, redistribute, etc. but we are covered by:

No liability
No warranty

Check this against GNUv3 and Apache but I think this is the best suited for us. Check here regarding licenses: https://www.freecodecamp.org/news/how-open-source-licenses-work-and-how-to-add-them-to-your-projects-34310c3cf94/

I don't mind. MIT it is then.
On 03 April 2020 09:21:48 SAST, David Baker Effendi<EMAIL_ADDRESS>wrote:

Check this against GNUv3 and Apache but I think this is the best suited
for us. Check here regarding licenses:
https://www.freecodecamp.org/news/how-open-source-licenses-work-and-how-to-add-them-to-your-projects-34310c3cf94/
--
You are receiving this because your review was requested.
Reply to this email directly or view it on GitHub:
https://github.com/Skippy404/C45DiscordBot/pull/30#issuecomment-608273749

--
Sent from my Android device with K-9 Mail. Please excuse my brevity.


feat: branch and PR support, switching UI

Towards https://github.com/giantswarm/giantswarm/issues/25147
Summary
Add branch/PR support and switching UI.

Basic support for switching between branches (only GitHub backend r/n)
Query parameter ?branch=

overrides config (eg. from config.yml)
for creating edit links for files on branches

Automatically create PRs when trying to publish on protected branches (eg. main)

Note: Re-tested creation of new commits on existing branch and creation of commit on a new branch+PR after all suggestions from review were applied.

OK, prettier also ran.


Ruby 3.1 compatibility (Psych 4 breaking change)

Hello,
rake translation:sync rake task fails with:  Psych::BadAlias: Unknown alias: male_first_name under Ruby 3.1.1.
YAML::load calls across sources need to be updated.
Example adjustment (if you want to support older and newer Psych with same gem version):
read_options = RUBY_VERSION >= "3.1.0" ? { :aliases => true } : {}
configurations = YAML.load(File.read(file_path, **read_options)

Thank you.

Hello @saboter,
We don't seem to use the aliases option in this gem, could you please tell us where this error comes from?
From one of our dependency maybe?

Hello @MichaelHoste,
this is the exception thrown by rake translation:sync with backtrace:
Psych::BadAlias: Unknown alias: male_first_name
/.rvm/gems/ruby-3.1.1/gems/translation-1.30/lib/translation_io/client/sync_operation/create_yaml_pot_file_step.rb:17:in `block in run'
/.rvm/gems/ruby-3.1.1/gems/translation-1.30/lib/translation_io/client/sync_operation/create_yaml_pot_file_step.rb:15:in `each'
/.rvm/gems/ruby-3.1.1/gems/translation-1.30/lib/translation_io/client/sync_operation/create_yaml_pot_file_step.rb:15:in `run'
/.rvm/gems/ruby-3.1.1/gems/translation-1.30/lib/translation_io/client/sync_operation.rb:39:in `run'
/.rvm/gems/ruby-3.1.1/gems/translation-1.30/lib/translation_io/client.rb:19:in `sync'
/.rvm/gems/ruby-3.1.1/gems/translation-1.30/lib/translation_io/tasks.rb:23:in `block (2 levels) in <main>'

I have used different exception thrown by other gem (yet caused by analogical problem) in the initial description.
Sorry for that (have already changed in the first post).

Don't worry, I found the real issue here: https://stackoverflow.com/a/71192990/1243212
male_first_name comes from faker that uses YAML aliases, issue here: https://github.com/faker-ruby/faker/issues/2330
That was fixed in Rails like this for the locales: https://github.com/ruby-i18n/i18n/pull/569/files
And like this for the configuration file: https://github.com/rails/rails/commit/179d0a1f474ada02e0030ac3bd062fc653765dbe
We'll release something similar as soon as possible, thank you for the warning 🙂

It should be fixed on the master branch, could you please confirm?
If it works, we'll quickly release a new version.

Version 1.31 is released with this fix.
Thank you for alerting us to this problem, it is appreciated.


[Bug] Code coverage error: must supply CC_TEST_REPORTER_ID var or pass -r flag

This error showed up in PR #745, from an external contributor.
Run<EMAIL_ADDRESS>/home/runner/work/pdr-backend/pdr-backend/cc-reporter before-build
ℹ 'coverageCommand' not set, so skipping building coverage report!
/home/runner/work/pdr-backend/pdr-backend/cc-reporter after-build --exit-code 0
Error: you must supply a CC_TEST_REPORTER_ID ENV variable or pass it via the -r flag

Full log:
https://github.com/oceanprotocol/pdr-backend/actions/runs/8175039266/job/22351907806?pr=745

From @calina-c :
It seems an external contributor does not have access to the repo secrets.
tests have successfully ran, but the coverage secret for codeclimate is not available, in order to publish the result


Exams: Rerout tutors when clicking on exam in student view

Checklist
General

[ ] I tested all changes and their related features with all corresponding user types on a test server.
[ ] This is a small issue that I tested locally and was confirmed by another developer on a test server.
[ ] Language: I followed the guidelines for inclusive, diversity-sensitive, and appreciative language.
[ ] I chose a title conforming to the naming conventions for pull requests.

Server

[ ] I followed the coding and design guidelines.
[ ] I added multiple integration tests (Spring) related to the features (with a high test coverage).
[ ] I added pre-authorization annotations according to the guidelines and checked the course groups for all new REST Calls (security).
[ ] I implemented the changes with a good performance and prevented too many database calls.
[ ] I documented the Java code using JavaDoc style.

Client

[ ] I followed the coding and design guidelines and ensured that the layout is responsive.
[ ] Following the theming guidelines, I specified colors only in the theming variable files and checked that the changes look consistent in both the light and the dark theme.
[ ] I added multiple integration tests (Jest) related to the features (with a high test coverage), while following the test guidelines.
[ ] I added authorities to all new routes and checked the course groups for displaying navigation elements (links, buttons).
[ ] I documented the TypeScript code using JSDoc style.
[ ] I added multiple screenshots/screencasts of my UI changes.
[ ] I translated all newly inserted strings into English and German.

Changes affecting Programming Exercises

[ ] I tested all changes and their related features with all corresponding user types on Test Server 1 (Atlassian Suite).
[ ] I tested all changes and their related features with all corresponding user types on Test Server 2 (Jenkins and Gitlab).

Motivation and Context

Description

Steps for Testing

Prerequisites:

1 Instructor
2 Students
1 Programming Exercise with Complaints enabled

Log in to Artemis
Navigate to Course Administration
...

Exam Mode Testing

Prerequisites:

1 Instructor
2 Students
1 Exam with a Programming Exercise

Log in to Artemis
Participate in the exam as a student
Make sure that the UI of the programming exercise in the exam mode stays unchanged. You can use the exam mode documentation as reference.
...

Review Progress

Code Review

[ ] Review 1
[ ] Review 2

Manual Tests

[ ] Test 1
[ ] Test 2

Exam Mode Test

[ ] Test

Test Coverage

Screenshots

Codecov Report

Merging #6240 (83cc3a0) into develop (b5be3ab) will decrease coverage by 0.01%.
The diff coverage is 0.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@              Coverage Diff              @@
##             develop    #6240      +/-   ##
=============================================
- Coverage      80.27%   80.27%   -0.01%
  Complexity     12909    12909
=============================================
  Files           2286     2286
  Lines          88692    88694       +2
  Branches       13103    13104       +1
=============================================
+ Hits           71199    71200       +1
- Misses          9744     9745       +1
  Partials        7749     7749

Impacted Files
Coverage Δ

...course-exam-detail/course-exam-detail.component.ts
75.00% <0.00%> (-2.28%)
:arrow_down:

...idebar/conversation-selection-sidebar.component.ts
89.33% <0.00%> (+0.66%)
:arrow_up:

Flag
Coverage Δ

client
76.43% <0.00%> (-0.01%)
:arrow_down:

server
84.55% <ø> (ø)

Continue to review full report at Codecov.

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

Better implemented in #6286


Add field comments support

Adds comments to the object type fields.

#1  👈
master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @vaclavbohac and the rest of your teammates on  Graphite


Net Worth Report is not calculating net worth correctly

The Net Worth Report is not calculating as it should (assets - debts). In this screenshot, though my account balances show $5,500.00 in debt and $3741.65 in assets, the report shows the net worth as $3852.00.

I tried a bunch of different calculations and it appears that it is adding total Assets (3741.65) and absolute value of the absolute value of the Total Activity for February (-83.35).

In my personal budget which doesn't have a screenshot here. The assets and liabilities appear to be added together rather than subtracted.

That's really odd. I'll have a look and see what I can do.

I think I might know what's going on here but I can't reproduce with my test data. To give me a dataset to work with, could you run this in your console and post what you get back? If you want we can do this in a non-public place like email.
(function() {
  ynab.YNABSharedLib.getBudgetViewModel_AllAccountTransactionsViewModel().then(function (transactionsViewModel) {
    var transactionDisplayItems = transactionsViewModel.get('visibleTransactionDisplayItems')

    var transactions = transactionDisplayItems.filter(function(transaction) {
      return transaction.get('displayItemType') == "transaction";
    });

    // Sort the transactions by date. They usually are already, but let's not depend on that:
    transactions.sort(function (a, b) {
      return a.get('date')._internalUTCMoment._d - b.get('date')._internalUTCMoment._d;
    });

    console.log(JSON.stringify(transactions.map(function(transaction) {
      return {
        date: transaction.get('date')._internalUTCMoment._d,
        formattedDate: ynab.YNABSharedLib.dateFormatter.formatDate(transaction.get('date')._internalUTCMoment._d, 'MMM YYYY'),
        account: transaction.getAccountName(),
        amount: transaction.getAmount()
      };
    })));
  });
})();

What does is pull your transactions with the following info on each:

Date
Formatted date (should look like Jan 2016 for example)
Account name
Amount (YNAB stores these 1000 too big for cents and such, that's normal)

So for example when I run this with my test budget I get:
[{"date":"2016-01-03T00:00:00.000Z","formattedDate":"Jan 2016","account":"Test","amount":-5000},{"date":"2016-01-06T00:00:00.000Z","formattedDate":"Jan 2016","account":"Credit Card","amount":0},{"date":"2016-01-10T00:00:00.000Z","formattedDate":"Jan 2016","account":"Test","amount":-5000},{"date":"2016-01-10T00:00:00.000Z","formattedDate":"Jan 2016","account":"Credit Card","amount":5000},{"date":"2016-01-11T00:00:00.000Z","formattedDate":"Jan 2016","account":"ing","amount":0}]

That will let me run this through the code on my side with your data so I can make sure I've fixed it.

I think the issue was two fold. I was putting the wrong label on months by one, and I was also not handling the last month's worth of data correctly.
In my test budget I put one transaction in Dec 2015 and one in Jan 2016. This reproduced the issue. I then applied the two fixes in the commit above and it worked for me.
If you could give me your data I'll make sure it works in your case, or if you want to check out current master and build, then try it on your side I'll be able to do another release to fix this for everyone.

Thanks. Here is my data from my test budget where I found the error.
[{"date":"2016-01-31T00:00:00.000Z","formattedDate":"Jan 2016","account":"Checking","amount":-175000},{"date":"2016-01-31T00:00:00.000Z","formattedDate":"Jan 2016","account":"Checking","amount":4000000},{"date":"2016-02-03T00:00:00.000Z","formattedDate":"Feb 2016","account":"Credit Card","amount":-500000},{"date":"2016-02-03T00:00:00.000Z","formattedDate":"Feb 2016","account":"Credit Card","amount":-26000},{"date":"2016-02-08T00:00:00.000Z","formattedDate":"Feb 2016","account":"Checking","amount":-57350},{"date":"2016-02-09T00:00:00.000Z","formattedDate":"Feb 2016","account":"Credit Card 2","amount":-5000000},{"date":"2016-02-09T00:00:00.000Z","formattedDate":"Feb 2016","account":"Checking","amount":-26000},{"date":"2016-02-09T00:00:00.000Z","formattedDate":"Feb 2016","account":"Credit Card","amount":26000},{"date":"2016-02-13T00:00:00.000Z","formattedDate":"Feb 2016","account":"Credit Card","amount":-25000}]

Thanks so much for the quick response.
This is what I see now with your test data and those two fixes applied:

I did some calculations in a spreadsheet with your numbers and I think at least for your case I've got it right now.

Ok, these changes are released in v0.3.8. Could you verify on your side as to whether I've fixed this or not?

Sounds like this is fixed now based on the feedback on Reddit. I'm going to go ahead and close this issue. Please let me know if you see anything out of the ordinary on these now.

It looks great now. Thanks! I'm glad you were able to fix it.


Unknown penalty (--unkpen) has no effect on the generated text

🐛 Bug
While using fairseq-generate to generate text in the target language for a translation task, setting the --unkpen value to even large numbers like 1e5, 1e6, etc. does not change the hypotheses. As a result I am unable to remove the <<unk>> tokens using this, as was suggested in #481 .

Expected behavior
The number of <<unk>> tokens decreases as the unknown penalty increases.
Environment

fairseq Version (e.g., 1.0 or master): 0.9.0
PyTorch Version (e.g., 1.0): 1.2.0
OS (e.g., Linux): Linux (Ubuntu 18.04.2)
How you installed fairseq (pip, source): pip
Build command you used (if compiling from source):
Python version: 3.6.8
CUDA/cuDNN version: 10.1
GPU models and configuration: GeForce GTX 1080 Ti
Any other relevant information:

Additional context

I found out a bug in my code that probably isn't caused by the --unkpen. Thus I am closing this.


Add a Pirate version of the Code of Conduct

Love it! I'll merge and revise a little in a subsequent commit. ;-)
Awesome.


chore: upgrade openapi dependency to feat/r17

Is the PR ready?

LGTM!

Add GroupIDmap & ImsDataRepository service api with not implement response.

Overall, it looks good to me.


[FEATURE REQUEST] Go to Macro Config Window from Macro Home Window (Hotlink)

Hello @chhoumann !!!!!!!
This is a simple FR related to QoL :)
I have almost 400 Macros Choices and many times during the year I have to change/modify my Macros due to new plugins and new core features from Obsidian. We are always improving :)
This way, I really need a way to easily open the Configure window of a specific Macro Choice without having to look for the Macro inside the Macro Manager window
📸 A button here

📸 To Access here

Thanks for reading this !!!!! I hope you like my suggestion 🙏 🙏 🙏
Have a great day 🌞

Hey @FelipeRearden!
I think this would be a fantastic QoL upgrade, so I went ahead and added it.
Here are the new features:

Hope you find this implementation useful!

This is absolutely spectacular @chhoumann !!!!!!
It's much more smart than my FR, it's amazing how easy to edit AND create new MACRO from the home screen !!!!!
I am gonna save so much time :)
Thank you very much @chhoumann  and have a great day  ☀☀☀

Great to hear that! Thank you for checking it out. :)


Fix modal SASS when $spacer is of the pixels unit

Setting the $spacer variable to a pixels unit breaks the compilation due to a calculation that's happening in the modal SASS:
SassError: 8px and 0.25rem have incompatible units.
    ╷
131 │   padding: $modal-inner-padding - $modal-footer-margin-between * .5;
    │            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    ╵
  node_modules\bootstrap\scss\_modal.scss 131:12  @import
  .scss\bootstrap.scss 29:9                       @import

It's actually line 132 now (click).
Since it's half the size of the spacer, might as well calculate the size.

Thanks for the PR and your involvement!
I don't think this should be done, here's why:

this is not consistent across variables (and even the modal component itself) since for example the next variable remains .5rem.
we use rem for sizing, as a convention, and thus don't check for units and don't prepare to mixing units either.
you can easily change $modal-inner-padding to use a px value, which would solve your issue and make your sizing more consistent too.

IMHO there's no need to change the default values, and I strongly recommend you either change the other variable or use rem across your codebase.
I'm closing this for now, but feel free to comment back here if you have anything else to share—we may reconsider this if needed.

Thanks for your quick reply, appreciate it!
I noticed that the next variable was .5rem too, but that one compiled fine (which is a bad reason not to have changed it in my PR now that I think about it). But now that we're talking about it, shouldn't it be relative to the $spacer variable too?
What I've done in my PR is something I see in other places too:
#L582, #L1072, #L1131, #L1219, #L1282, #L1448
But it seems to be missing in other places that are about the same type of variable (paddings/margins):
#L1472, #L1495, #L1496, #L1525 and others.
Changing the $spacer value will be ignored by other variables that are about paddings and margins too.
Is there a specific reason for it, or am I misunderstanding?
One last note, which sounds conceited but I totally don't mean it in that way:
Changing $modal-inner-padding to a px value isn't going to help; it's still going to break compilation since it's being used in a calculation later with the rem variable $modal-footer-margin-between. Setting that variable to a px would fix it though.
Thanks again for your in-depth reply and recommendations, it helps me learn to make the most out of Bootstrap!


Add LGTM code quality badges

Hi there!
I thought you might be interested in adding these code quality badges to your project. They will indicate a very high code quality to potential users and contributors.
To get an idea of the analyses reflected by these grades, check the alerts discovered by LGTM.
N.B.: I am on the team behind LGTM.com, I'd appreciate your feedback on this initiative, whether you're interested or not, if you find time to drop me a line. Thanks.

Codecov Report

Merging #146 into dev will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev     #146   +/-   ##
=======================================
  Coverage   88.05%   88.05%
=======================================
  Files         135      135
  Lines        2403     2403
  Branches      279      279
=======================================
  Hits         2116     2116
  Misses        176      176
  Partials      111      111

Continue to review full report at Codecov.

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

Can one of the admins verify this patch?


Add Telegram "World" Channel (English)

... before someone is not joining because german only

Looks like this is done and the issue can be closed


kromogo cluster_age_days calculation

I think you have the wrong calculation for cluster_age_days using:
          - name: cluster_age_days
            query: round((time() - max(kube_node_created{kubernetes_node=~"k8s-[0-9]+"}) ) / 86400)
            suffix: "d"

The max(kube_node_created) would return the value of your newest node member, not your oldest.  Thus it returns the number of days since you last added a node.  If you want the cluster age in days, then use the min() to get the timestamp of the oldest cluster node.
When I use round((time() - min(kube_node_created)) / 86400) I get 681 which lines up to the age of my oldest cluster node:
$ k get nodes
NAME    STATUS   ROLES                       AGE    VERSION
k3s01   Ready    control-plane,etcd,master   680d   v1.28.5+k3s1
k3s02   Ready    control-plane,etcd,master   654d   v1.28.5+k3s1
k3s03   Ready    control-plane,etcd,master   661d   v1.28.5+k3s1
k3s04   Ready    worker                      427d   v1.28.5+k3s1
k3s05   Ready    worker                      427d   v1.28.5+k3s1
k3s06   Ready    worker                      354d   v1.28.5+k3s1

if I use max() then I get 355 which is the age of the newest node.

Thanks for reporting, fixed!


Should we add the ability to create an instance of Cardoteka?

This will simplify the use of the library by a whole one step! I can't foresee or observe any negative consequences of such a decision so far.

If Cardoteka becomes base class Cardoteka, then this will require the use of 'base', 'final' or 'sealed' for all descendant classes. Which is a big change and is subject to raising the major version.


workflows: add documentation preview

Clone flatcar/flatcar-website and checkout on main branch
Sed the config to target the tested flatcar-docs branch
Build and upload the website

I will try.

@joaquimrocha Could you please try again?


Create a /help command

I propose to implement a /help chat command to display all the chat command available with WFRP set of rules.
Then we will discuss later how to help people to find out the /help command.

In the file chatMessage.js, there is a lot of simple equals (ie. if (command[0] == "/table")). Is there a reason to do that instead of the triple equals that check the type (ie. if (command[0] === "/table")) ?
If there is a reason is it the way you have to code in Foundry VTT js framework ?
If not, I will change all the "==" by "===" in the file.

Hi,
At this point, the result of the commande /help does this :

I can add a few things :

an example for each command
a link to the wiki page (https://github.com/CatoThe1stElder/WFRP-4th-Edition-FoundryVTT/wiki/Chat-Commands)

I'd like to have your opinion.
what do you think ? It's ok like that, more stuff, less stuff ?

Examples would be great, but it's looking good so far! Also yeah there's no reason behind me using == over ===


Updated for PHP 8. Unit tests passed.

On branch updated-for-php8
Changes to be committed:
modified:   composer.json

We can not limit ourselves to a php 8 minimum requirement just yet, same as with tightenco/collect. The SDK should work without these upgraded.
Thanks for your contribution.


Update PilotAssistant.netkan

Fix override, no leading v (thankfully).

LGTM, merging with thanks!


Extend CodeShovel to capture method annotations

resolves #1
Extend CodeShovel to capture methods annotation such as @Override, @Test, @SuppressedWarnings etc.

@ishtiaque05 Can you pull develop into this and add a stub in PythonFunction so it builds? I'll fill it out :)

Also, do you have a test case for this? Just a single one in the java oracles would be super helpful I think.

@ishtiaque05 Can you pull develop into this and add a stub in PythonFunction so it builds? I'll fill it out :)

Sure no problem :)

Also, do you have a test case for this? Just a single one in the java oracles would be super helpful I think.

Yes, I will write some as soon as I can execute mvn test on my laptop. Apparently, my 8 GB ram is not enough to execute all of the CodeShovel tests. So I would not know if my code breaks anything or not but I can surely write some separate test cases :)

Also, do you have a test case for this? Just a single one in the java oracles would be super helpful I think.

Yes, I will write some as soon as I can execute mvn test on my laptop. Apparently, my 8 GB ram is not enough to execute all of the CodeShovel tests. So I would not know if my code breaks anything or not but I can surely write some separate test cases :)

Haha well if you want I can test for you. Just give me a test case and I can run it :)


Added chocolateyUninstall.ps1 file for Firefox

I made the chocolateyUninstall.ps1 for Firefox choco package, please review. I tested the package with the new file using my local folder/repository and the uninstalling pocedure is now silent. I also tested through Ansible (that needs Firefox to be silently uninstalled otherwise it does not work) and it works too.

In the registry I could find only the path to the firefox.exe file. How would I get to the helper.txt path through the registry?

Weird, I am going to call no dice on this one.

The only reason Chocolatey can't automatically uninstall it is that it is not able to detect the installer type (yet).

So this is interesting...

FirefoxInstall.exe: installer: 7-Zip(1.0); overlay: 7-zip Installer data(-)

Unzipped there is a setup file:

/FirefoxInstall/setup.exe: installer: Nullsoft Scriptable Install System

Yep, I found the location of the Uninstall string in the registry. Sorry but I just started with powershell, so.. how would I change my script to get to uninstall location from the registry?

@sparkvilla first choco upgrade chocolatey -pre and then run choco new firefox and take a look at the uninstall script that is generated to adjust this one.

@ferventcoder I have used the template you suggested to make up a second version of chocolateyuninstall.ps1 file. It works but Firefox is not uninstalled silently (things that was achieved with the script of this pull request). Something wrong. How can I upload the new script so that you can have a look at it? or, Shell I open a new pull request?

The script generated for uninstall is a template result which still requires some editing. The idea is to look at that to get an idea how you can adjust this PR to use the registry instead. You still need to make the adjustments to it for the silent arguments.

You can always paste script text into these messages as another alternative way of looking at what you have questions about.

` # If this is an exe, change installerType and silentArgs
$ErrorActionPreference = 'Stop'; # stop on all errors
$packageName = 'firefox'
$softwareName = 'Mozilla Firefox' #part or all of the Display Name as you see it in Programs and Features. It should be enough to be unique
$installerType = 'EXE'
#$silentArgs = '/qn /norestart'
https://msdn.microsoft.com/en-us/library/aa376931(v=vs.85).aspx
$validExitCodes = @(0, 3010, 1605, 1614, 1641)
if ($installerType -ne 'MSI') {
The below is somewhat naive and built for EXE installers
Uncomment matching EXE type (sorted by most to least common)
$silentArgs = '/S'           # NSIS
#$silentArgs = '/VERYSILENT /SUPPRESSMSGBOXES /NORESTART /SP-' # Inno Setup
#$silentArgs = '/s'           # InstallShield
#$silentArgs = '/s /v"/qn"' # InstallShield with MSI
#$silentArgs = '/s'           # Wise InstallMaster
#$silentArgs = '-s'           # Squirrel
#$silentArgs = '-q'           # Install4j
#$silentArgs = '-s -u'        # Ghost
Note that some installers, in addition to the silentArgs above, may also need assistance of AHK to achieve silence.
$silentArgs = 'none'             # none; make silent with input macro script like AutoHotKey (AHK)
#       https://chocolatey.org/packages/autohotkey.portable
$validExitCodes = @(0)
}
$uninstalled = $false
$local_key     = 'HKCU:\Software\Microsoft\Windows\CurrentVersion\Uninstall*'
$machine_key   = 'HKLM:\SOFTWARE\Microsoft\Windows\CurrentVersion\Uninstall*'
$machine_key6432 = 'HKLM:\SOFTWARE\Wow6432Node\Microsoft\Windows\CurrentVersion\Uninstall*'
$key = Get-ItemProperty -Path @($machine_key6432,$machine_key, $local_key)                         -ErrorAction SilentlyContinue
| ? { $_.DisplayName -like "$softwareName" }
if ($key.Count -eq 1) {
$key | % {
$file = "$($_.UninstallString)"
if ($installerType -eq 'MSI') {
  # The Product Code GUID is all that should be passed for MSI, and very
  # FIRST, because it comes directly after /x, which is already set in the
  # Uninstall-ChocolateyPackage msiargs (facepalm).
  $silentArgs = "$($_.PSChildName) $silentArgs"

  # Don't pass anything for file, it is ignored for msi (facepalm number 2)
  # Alternatively if you need to pass a path to an msi, determine that and
  # use it instead of the above in silentArgs, still very first
  $file = ''
}

Uninstall-ChocolateyPackage -PackageName $packageName `
                            -FileType $installerType `
                            -SilentArgs "$silentArgs" `
                            -ValidExitCodes $validExitCodes `
                            -File "$file"

}
} elseif ($key.Count -eq 0) {
Write-Warning "$packageName has already been uninstalled by other means."
} elseif ($key.Count -gt 1) {
Write-Warning "$key.Count matches found!"
Write-Warning "To prevent accidental data loss, no programs will be uninstalled."
Write-Warning "Please alert package maintainer the following keys were matched:"
$key | % {Write-Warning "- $_.DisplayName"}
} `

@sparkvilla @ferventcoder I just wanted to touch base on this...
Is there still work required here?

@sparkvilla pinging again about this...
Is there still work required?

@gep13 @ferventcoder  @sparkvilla  Taking a look at this now will be back with a PR soon as work permits.

I have found a solution for @sparkvilla problem; This borrows some code from the original install file. Created a new PR #262

Closing this in favour of new PR: https://github.com/chocolatey/chocolatey-coreteampackages/pull/262


clear conditional fields when unselected

Are you submitting a bug report or a feature request?
bug
What is the current behavior?
upon selecting a field that conditionally renders more fields, changing that initial field should remove the conditional field values.
See the example link. Click hasEmail and type some email. Now unclick hasEmail. the email value is saved in the form despite it no longer existing.
What is the expected behavior?
when a field doesn't exist in redux-form it is removed from the state
Sandbox Link
https://redux-form.com/7.1.1/examples/selectingformvalues/
What's your environment?
redux-form 7.1.1
Other information

That is a quite controversial behaviour of Redux Form...
There are folks that defend one may want to include unmounted fields in the submission, others don't.
Since this is pretty much the case of the link you mentioned, I'll just say that this case can be fixed by nullifying the other field when a change happens.
I'm closing this issue for now, but feel free to continue the discussion!


Bump version to 0.2.0

In preparation for release add a changelog entry and bump the version number.

Tagged and published.
cc @DanGould

I'm a bit confused about the upgrade, it seemed on an incomplete draft you wanted someone to check whether or not the soon-to-be finished port was a character-for-character copy from rust-bitcoin, but then it got merged.
Did I misunderstand your request? Glad this was published in any case

Oh I just assumed that no one would do it, I also realised that this crate claims to be a copy right from the start so any user would likely want to verify that them self anyways and this patch does not change that.

I also put a bunch more effort into the v2 stuff so it was good to go, IMO.


fix(postgraphql): try getting error code from error.statusCode before falling back to 500

The HTTP error handler was looking in error.status for the status code to return to the client, but some error codes were being set in error.statusCode.
Fixes half the error reported in #287 where the JWT error throws a 500 rather than a 403.

Looks good to me. Although I agree that we should try to be more consistent about this…


Add league fractal dependency. #8991

Issue Reference

Add league fractal dependency. #8991

@suraj-webkul Please check the PR again, as there are some conflicts which need to be fixed.

@devansh-webkul sir,
All conflicts have been resolved. please verify the changes.


Updated UTF to spin up a cluster, interrogate it, then spin it down

Description

Added library code to manage Docker interactions
Added library code to spin up and manage an ElasticSearch Cluster and its Nodes
Added unit tests for the library code
Updated the UTF workflow to spin up a cluster of 3 nodes, query the cluster state, then spin down the cluster
Updated the run_utf.py script to kick off the workflow
Deprecated the "resume" functionality as it was becoming cumbersome to keep up-to-date and seems to offer limited value
Bare-bones test_config.json parsing

Issues Resolved

https://github.com/opensearch-project/opensearch-migrations/issues/6
https://github.com/opensearch-project/opensearch-migrations/issues/8

Testing
Added unit tests:
(.venv) chelma@3c22fba4e266 upgrades % python -m pytest tests/
================================================================================ test session starts =================================================================================
platform darwin -- Python 3.10.6, pytest-7.2.0, pluggy-1.0.0
rootdir: /Users/chelma/workspace/OSMigration/opensearch-migrations/upgrades
collected 56 items

tests/test_cluster.py ......                                                                                [ 10%]
tests/test_docker_framework_client.py .............                                                                                [ 33%]
tests/test_exception_base.py ...                                                                                [ 39%]
tests/test_framework_runner.py ......                                                                                [ 50%]
tests/test_framework_state.py ...                                                                                [ 55%]
tests/test_framework_step.py ....                                                                                [ 62%]
tests/test_logging_wrangler.py .                                                                                [ 64%]
tests/test_node.py .......                                                                                [ 76%]
tests/test_shell_interactions.py .......                                                                                [ 89%]
tests/test_test_config_wrangling.py .....                                                                                [ 98%]
tests/test_workspace_wrangler.py .                                                                                [100%]

================================================================================= 56 passed in 1.79s =================================================================================

Also, tested workflow using the UTF kickoff script.  My test_config.json looks like this:
{
    "source_docker_image": "docker.elastic.co/elasticsearch/elasticsearch-oss:7.10.2"
}

The script and its output looks like this:
(.venv) chelma@3c22fba4e266 upgrades % ./run_utf.py --test_config /tmp/test_config.json
[FrameworkRunner] Running Step: LoadTestConfig
[LoadTestConfig] Loading test config file...
[LoadTestConfig] Loaded test config file successfully
[FrameworkRunner] Step Succeeded: LoadTestConfig
[FrameworkRunner] Running Step: BootstrapDocker
[BootstrapDocker] Checking if Docker is installed and available...
[BootstrapDocker] Docker appears to be installed and available
[BootstrapDocker] Ensuring the Docker image docker.elastic.co/elasticsearch/elasticsearch-oss:7.10.2 is available either locally or remotely...
[BootstrapDocker] Docker image docker.elastic.co/elasticsearch/elasticsearch-oss:7.10.2 is available
[FrameworkRunner] Step Succeeded: BootstrapDocker
[FrameworkRunner] Running Step: StartSourceCluster
[StartSourceCluster] Creating source cluster...
[StartSourceCluster] Waiting up to 30 sec for cluster to be active...
Node source-cluster-node-1 is now active
Node source-cluster-node-3 is now active
Node source-cluster-node-2 is now active
[StartSourceCluster] Cluster source-cluster is active
[FrameworkRunner] Step Succeeded: StartSourceCluster
[FrameworkRunner] Running Step: TestSourceCluster
[TestSourceCluster] ip         heap.percent ram.percent cpu load_1m load_5m load_15m node.role master name
<IP_ADDRESS>           31          59  68    2.15    0.51     0.24 dimr      -      source-cluster-node-1
<IP_ADDRESS>           10          59  58    2.15    0.51     0.24 dimr      -      source-cluster-node-3
<IP_ADDRESS>           12          59  58    2.15    0.51     0.24 dimr      *      source-cluster-node-2
[FrameworkRunner] Step Succeeded: TestSourceCluster
[FrameworkRunner] Running Step: StopSourceCluster
[StopSourceCluster] Stopping cluster source-cluster...
[StopSourceCluster] Cleaning up underlying resources for cluster source-cluster...
[FrameworkRunner] Step Succeeded: StopSourceCluster
[FrameworkRunner] Ran through all steps successfully
[FrameworkRunner] Saving application state to file...
[FrameworkRunner] Application state saved
[FrameworkRunner] Application state saved to: /tmp/utf/state-file
[FrameworkRunner] Full run details logged to: /tmp/utf/logs/run.log.2022-12-12_12_28_20

Check List

[X] New functionality includes testing

[X] All tests pass, including unit test, integration test and doctest

[ ] New functionality has been documented
[X] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

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

@@           Coverage Diff           @@
##             main      #40   +/-   ##
=======================================
  Coverage        ?   88.27%
=======================================
  Files           ?       19
  Lines           ?      580
  Branches        ?       50
=======================================
  Hits            ?      512
  Misses          ?       62
  Partials        ?        6

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


expose OQS_PERMIT_UNSUPPORTED_ARCHITECTURE, for example as cargo feature

I can't build my project for x86 with a musl libc due to https://github.com/open-quantum-safe/liboqs/issues/1442 , and as I see there is no way to trigger the aforementioned CMake option using the rust wrapper, and due to https://github.com/open-quantum-safe/liboqs-rust/issues/190 I can also not substitute the version of liboqs used externally :disappointed:

I'm not sure if it's not better to just wait for a fix for #190 to land upstream...

Euh right, that's the issue I meant to reference


Dust limit too large when trying Turbo channel

Problem:
When attempting to accept a Turbo channel, "Dust limit too high" message appears for a while and opening fails. The message specifies that the dust limit is 2730, while max is 1000.
I have set dust limit to 546, yet the problem persists. It's unclear whether BLW is rejecting the channel or Eclair. I suppose it's Eclair.

This is a known issue which used to affect channels of low capacity (below 273 000 SAT), but it was fixed in last 0.3-130 update. I guess you're using an outdated BLW version, updating it should fix an issue.
What is going on: both Eclair and BLW provide their own dust limits, Eclair's is 546 SAT by default while BLW's was 2730 SAT (546 SAT now). When opening a channel there's a so called unspendable reserve which can not be lower than 1% of channel capacity AND can not be lower than dust limit. So when proposed channel capacity was below 273 000 SAT Eclair was rejecting it because BLW's dust was higher than minimal allowed reserve.
With latest update minimal incoming channel capacity is 54 600 SAT.

Ah, thanks, that helped! Now I'm getting "They are funder and cannot afford fees message".

Probably remote funder tries to push too much money on BLW side.
An example: total channel capacity is 100 000 SAT and remote funder pushes 95 000 on BLW side. Current on-chain feerates would require 6000 SAT for a commitment transaction. Fees are paid by remote funder and they only left 5000 SAT to themselves, so channel gets rejected.

Figured it out - local amount was equal to push. I raised it by 20000 sats and it works. It'd be helpful if you know how to exactly calculate minimum value instead of guessing.

LOL, GitHub lag, thanks! :)


Rename loadContext to Context?

It feels weird doing loadContext.user wouldn't it make more sense if it was just context.user?

Don't want to confuse it with react's context.

But ... yeah I could pretty easily be persuaded to just call it context.

I think it's pretty clear that it's not from react since you have to pass it in to loadPropsOnServer. It feels so weird accessing it this way haha


Support in RLMArray for basic types (int, strings, etc)

It'd be awesome if RLMArray supported the types RLMObject does (or just plain NSArray/Swift Array types for basic types), for use cases like this:
{
"id": "1234",
"name": "test",
"tags": ["one", "two", "three"]
}

Closing as a duplicate of https://github.com/realm/realm-cocoa/issues/1120.