Improve Github Login

When logging in to Github in VS Code (Desktop), this website is shown:

Chrome prompts to launch VS Code Insiders and then VS Code Insiders prompts to call into the extension.
I think this could be improved.
When logging in to Facebook in Spotify, they don't do that.
Instead, they show this page:

And when clicking on "Continue to the App", I'm magically signed in in Spotify.
Behind the scenes, spotify opens an http server and when clicking on "Continue to the App", they send a request to http://<IP_ADDRESS>:4382/login?code=.... This works nicely and without ugly prompts.
Also, I think the design of the github page in the screenshot could use some love.
The authorization token could be hidden by default and only shown when the user clicks on some "Didn't work?" button - for me, it works all the time.

Chrome prompts to launch VS Code Insiders and then VS Code Insiders prompts to call into the extension.

I think a good first step of this issue is to get rid of VS Code prompting to call into the extension. We're in complete control of this flow so I don't see why we need to notify here.

Sign in to GitHub doesn't even work for me at all on Ubuntu 20.04 with latest Firefox.

It takes me to that page screenshotted above.
When I click the browser prompt to continue to VS Code, nothing happens in VS Code except that the palette shows "Error" without any message.
I can not paste my auth code manually, because in VS Code there is no such message "Signing into github.com..." to click on. Nothing.
Atempting to do this over and over simply leads to the same result, never being able to log in.

I think we can close this now. The flow is quite different. We do still use the Chrome redirection, but only because that is the best option. Using the http server is hard because in Remote scenarios, the GitHub Authentication extension may be running on the remote side.
We do fallback to the http server if the Chrome redirection doesn't work ... and we fallback to Device Code flow if that doesn't work... and there are notifications to get you through this.


Fix rendering issues

My PR https://github.com/IMRCLab/crazyswarm2/pull/511 had some small mistakes in there that didn't render hyperlinks and the list properly. This should fix it

Seems like the checks pass now


Prevent Laravel API from processing the token if it comes in the URL as querystring

Laravel Version: 5.3
PHP Version: 7.1
Database Driver & Version: pgsql

Description:
Before all, sorry if this is not the place for asking but I'm trying to prevent laravel from processing the token if it comes in the url as querystring and I'm not being able to do it.
This is my config/cors.php file:
<?php

return [
   'defaults' => [
       'supportsCredentials' => false,
       'allowedOrigins' => [],
       'allowedHeaders' => [],
       'allowedMethods' => [],
       'exposedHeaders' => [],
       'maxAge' => 0,
       'hosts' => [],
   ],

   'paths' => [
       'v1/*' => [
           'allowedOrigins' => ['*'],
           'allowedHeaders' => [
               'Origin',
               'Content-Type',
               'Accept',
               'Authorization',
               'X-Request-With'
           ],
           'allowedMethods' => ['GET', 'POST', 'PUT', 'PATCH', 'DELETE'],
           'exposedHeaders' => ['Authorization'],
           'maxAge' => 3600,
       ],
   ],
];

And here you what I want to avoid:

And this is the only behavior that I want:

Sorry again if this is not the place but I'm trying to figure this out since last week and I think I'm not even close. Thank you for your time!

You can use the forums to ask questions, I'm sure someone will be able to help you there: https://laracasts.com/discuss

Hi @themsaid, I already did last week, here https://laracasts.com/discuss/channels/laravel/prevent-laravel-api-from-processing-the-token-if-it-comes-in-the-url-as-querystring and here http://stackoverflow.com/questions/43764919/prevent-laravel-api-from-processing-the-token-if-it-comes-in-the-url-as-querystr , if you know the answer you will get my bounty in stackoverflow :+1:

you can join is in the slack channel and throw your question there so we can discuss it and find a solution for you if forums don't work out

@Dylan-DPC that would be nice! could you invite me? or tell me the slack team name, please.

@Dylan-DPC I'm in :+1:


Perform cluster bookeeping tasks before determining retry

Just because a block is not going to be retried, does not mean we should not process topology updates & redirections in response to errors, I think.
Before this patch is merged:

We catch an exception,
We see if we have retry_count remaining,
And if so, update cluster topology,
And then retry

After this patch is merged:

We catch an exception,
We update cluster topology (e.g. processing MOVED).
If we have retry_count remaining,
Then we retry

This is important for cluster transaction support because we will always do user-provided transaction blocks with retry_count: 0 (we don't know if the provided block is safe to call again); but, we need to make sure we still update the topology so that if the user retries themselves, it will actually work.

Linter error is not related this pull request. We can ignore them at this time.

Thank you!


Simple update to help with docker-machine

Just thought it would be helpful, to others, now that docker-machine is out and we can use it on OS X instead of Boot2Docker

@aglahe
Merged. Thank you for your feedback.


WIP: Authentication

When I run Bosun without auth, I see my username and a log out drop down, even thought I am not logged in. This probably should not be visible in no auth mode:

When running with a self-signed cert, the logs get spammed repeatedly with:
$ go run main.go -c bosun.toml -w -r -q -skiplast -n
2016/12/08 12:57:10 info: main.go:149: readonly relay at http://<IP_ADDRESS>:63606 to http://ny-tsdb02:4242
2016/12/08 12:57:10 info: web.go:224: tsdb host: <IP_ADDRESS>:63606
2016/12/08 12:57:10 info: web.go:234: bosun web listening https on: :8080
2016/12/08 12:57:10 info: main.go:277: watching *.go in .
2016/12/08 12:57:10 info: main.go:277: watching *.html in web/static/templates
2016/12/08 12:57:10 info: main.go:277: watching *.ts in web/static/js
2016/12/08 12:57:25 error: queue.go:117: Post https://localhost:8080/api/put: x509: certificate signed by unknown authority

not sure why it is doing that with the -r flag anyways?

Actually, that error may be a problem regardless:
2016/12/13 12:16:28 error: queue.go:117: Post https://localhost:8080/api/put: x509: certificate is valid for bosun, not localhost

Ah yes.  I'm thinking we should just call the method directly if we are doing the in-proc mode. Perhaps a func in collect/metadata that can be swapped out to replace the http call altogether with a simple function call to the appropriate handler.
but datapoints really should be going out through tsdbrelay anyway.

Mixed content due to jquery: Mixed Content: The page at 'https://bosun:8080/login/' was loaded over HTTPS, but requested an insecure script 'http://code.jquery.com/jquery-3.1.1.min.js'. This request has been blocked; the content must be served over HTTPS.

well that's just sloppy.

Couple things regarding the bosun self metrics issue:

Is there some other communication channel we could use other than the HTTP route for the self metrics so we don't have to do this auth thing against localhost?
Unless you use influx or OpenTSDB, these metrics can't go anywhere. Might be good to wrap them under an option anyways


Test case for partial page no flush.

Description of changes:
Test that page wasn't flushed if it's not full.
Also, name the wrapper function the original function's name so Nosetests in PyCharm could recognize the method. Example: https://jmp.sh/H5LYUD9

@Johnius can you create an issue for this? Thanks.


Implement ToString trait for records

This implements the ToString trait for every record that contains a Name<'a>.

Merged. Thanks!


Save 4 bytes on every ArgInfo by deleting rebind

This PR moves rebind information from ArgInfo to Method to optimize memory usage and align with better modeling of inference logic. Previously, every ArgInfo instance stored a 4-byte rebind field even though only block arguments needed it. Now, we store this as an 8-byte blockRebind field on Method where it logically belongs, since every method has exactly one block argument. While this increases the Method size from 136 to 144 bytes, I'm hoping that this would actually result in aggregate memory savings by removing the redundant fields from all of the ArgInfo instances. I really wanted to find a way to implement this without increasing the size of Method, but from what I understand, we need a dedicated spot to hold the block's rebind (distinct from ArgInfo) that round-trips in the Method's pickle/unpickle code—though I may be missing some nuance here about how we could achieve this differently. Without storing this information somewhere that gets serialized, we'd lose rebind details for block arguments.
Motivation
Fixes #6629
Test plan
This does not include any new tests, but I did update the bind_class_of expectation test. If there were a regression or change that somehow stores rebind anywhere else, then the resolver/bind_class_of test would fail because the updated test expects the rebind to be on the Method. The infer/rebind test already covers method rebinding, block argument rebinding, and nested block rebinding.

I had a feeling something was unusual about the memory allocation for ArgInfo, especially because when I previously adjusted ArgInfo to reduce its size to 28 bytes, the CheckSize assertion immediately failed. However, I didn’t fully understand why this was happening. I was under the impression that the assertion was enforcing an 8-byte alignment even if ArgInfo itself didn’t explicitly need it. So basically, I was mostly optimistic that removing rebind from every ArgInfo could result in aggregate memory savings, given that there are generally far more arguments than methods, but I now see that alignment constraints negate these savings.
I understand now why this change isn't feasible as-is, but I'm struggling to see the path forward. The rebind on ArgInfo and Method can't simply be merged because block arguments need to track their rebind independently—blocks can capture and rebind self differently from the containing method. For example, relying on the single existing rebind on Method has led to Sorbet losing rebind information in the inference phase, particularly around method binding and block types. This manifests as expectation test errors (infer/rebind.rb.cfg-text.exp) like the following:
@@ -106,7 +106,7 @@
 method ::B#only_on_B {

 bb0[firstDead=2]():
-    <self>: B = cast(<self>: NilClass, B);
+    <self>: C = cast(<self>: NilClass, C);
     <finalReturn>: T.noreturn = return <returnMethodTemp>$2: NilClass
     <unconditional> -> bb1

Do you think there's a practical way to maintain separate rebind tracking for blocks without increasing Method's size?


SuperTuxKart segfaults with GPU enabled

Environment
Windows build number: Win 11 2200.258
Your Distribution version: Ubuntu 20.04 LTS
Your WSLg version: WSLg 1.0.26

Steps to reproduce
install supertuxkart on linux
sudo apt install -y supertuxkart
Then run.
supertuxkart
WSL logs:

Attach WSLg logs from  /mnt/wslg

You can access the wslg logs using explorer at: \\wsl$\<Distro-Name>\mnt\wslg (e.g.: \\wsl$\Ubuntu-20.04\mnt\wslg)

puseaudio.log

W: [pulseaudio] authkey.c: Failed to open cookie file '/home/wslg/.config/pulse/cookie': No such file or directory
W: [pulseaudio] authkey.c: Failed to load authentication key '/home/wslg/.config/pulse/cookie': No such file or directory
W: [pulseaudio] authkey.c: Failed to open cookie file '/home/wslg/.pulse-cookie': No such file or directory
W: [pulseaudio] authkey.c: Failed to load authentication key '/home/wslg/.pulse-cookie': No such file or directory
E: [pulseaudio] module-console-kit.c: GetSessionsForUnixUser() call failed: org.freedesktop.DBus.Error.ServiceUnknown: The name org.freedesktop.ConsoleKit was not provided by any .service files
E: [pulseaudio] module.c: Failed to load module "module-console-kit" (argument: ""): initialization failed.
E: [rdp-sink] module-rdp-sink.c: RDP Sink - Trying to connect to /mnt/wslg/PulseAudioRDPSink
E: [rdp-sink] module-rdp-sink.c: Connected failed
E: [rdp-sink] module-rdp-sink.c: RDP Sink - Trying to connect to /mnt/wslg/PulseAudioRDPSink
E: [rdp-sink] module-rdp-sink.c: RDP Sink - Connected to fd 12

weston.log

[22:08:13.689] weston 9.0.0
               https://wayland.freedesktop.org
               Bug reports to: https://gitlab.freedesktop.org/wayland/weston/issues/
               Build: 9.0.0-52-gb49e1942
[22:08:13.689] Command line: /usr/bin/weston --backend=rdp-backend.so --xwayland --shell=rdprail-shell.so --logger-scopes=log,rdp-backend,rdprail-shell --log=/mnt/wslg/weston.log
[22:08:13.689] OS: Linux, <IP_ADDRESS>-microsoft-standard-WSL2, #1 SMP Wed Aug 25 23:20:18 UTC 2021, x86_64
[22:08:13.689] Using config file '/home/wslg/.config/weston.ini'
[22:08:13.690] Output repaint window is 7 ms maximum.
[22:08:13.691] Loading module '/usr/lib/libweston-9/rdp-backend.so'
[22:08:13.697] using FreeRDP version 2.3.2
Date: 2021-10-15 CDT
[22:08:13.697] RDP backend: WESTON_RDP_DEBUG_LEVEL: 3
[22:08:13.697] RDP backend: WESTON_RDP_MONITOR_REFRESH_RATE: 60000
[22:08:13.697] RDP backend: timer resolution tv_sec:0 tv_nsec:1
[22:08:13.697] RDP backend: Environment dump - start
[22:08:13.697]   WSL_DISTRO_NAME=
[22:08:13.697]   WSL2_USER_PROFILE=C:\Users\Aaditya Murthy
[22:08:13.697]   WSL2_SHARED_MEMORY_OB_DIRECTORY=WSL\12F45B53-BECC-48CB-B1F9-D10C50AAD5F5\wslg
[22:08:13.697]   PWD=/
[22:08:13.697]   NAME=Aadi-PC
[22:08:13.697]   WSL_FEATURE_FLAGS=7
[22:08:13.697]   WSL2_INSTALL_PATH=C:\Program Files\WindowsApps\MicrosoftCorporationII.WindowsSubsystemforLinux_<IP_ADDRESS>_x64__8wekyb3d8bbwe
[22:08:13.697]   SHLVL=1
[22:08:13.697]   WSL2_GUI_APPS_ENABLED=1
[22:08:13.697]   WSL2_DISTRO_NAME=Ubuntu
[22:08:13.697]   WSL2_VM_ID=12F45B53-BECC-48CB-B1F9-D10C50AAD5F5
[22:08:13.697]   _=/usr/bin/WSLGd
[22:08:13.698]   HOME=/home/wslg
[22:08:13.698]   USER=wslg
[22:08:13.698]   LOGNAME=wslg
[22:08:13.698]   SHELL=/bin/bash
[22:08:13.698]   PATH=/usr/sbin:/usr/bin:/sbin:/bin:/usr/games
[22:08:13.698]   XDG_RUNTIME_DIR=/mnt/wslg/runtime-dir
[22:08:13.698]   WAYLAND_DISPLAY=wayland-0
[22:08:13.698]   DISPLAY=:0
[22:08:13.698]   XCURSOR_PATH=/usr/share/icons
[22:08:13.698]   XCURSOR_THEME=whiteglass
[22:08:13.698]   XCURSOR_SIZE=16
[22:08:13.698]   PULSE_AUDIO_RDP_SINK=/mnt/wslg/PulseAudioRDPSink
[22:08:13.698]   PULSE_AUDIO_RDP_SOURCE=/mnt/wslg/PulseAudioRDPSource
[22:08:13.698]   USE_VSOCK=3
[22:08:13.698]   WSL2_DEFAULT_APP_ICON=/usr/share/icons/wsl/linux.png
[22:08:13.698]   WSL2_DEFAULT_APP_OVERLAY_ICON=/usr/share/icons/wsl/linux.png
[22:08:13.698]   WESTON_DISABLE_ABSTRACT_FD=1
[22:08:13.698]   WSL2_SHARED_MEMORY_MOUNT_POINT=/mnt/shared_memory
[22:08:13.698]   WESTON_CONFIG_FILE=/home/wslg/.config/weston.ini
[22:08:13.698] RDP backend: Environment dump - end
[22:08:13.698] Using external fd for incoming connections: 3
[22:08:13.698] Default head is being added
[22:08:13.698] Registered plugin API 'weston_rdprail_api_v1' of size 56
[22:08:13.698] RDPAPPLIST_MODULEDIR is set to /usr/lib/rdpapplist
[22:08:13.699] RDP backend: use_rdpapplist = 1
[22:08:13.703] RDP backend: use_gfxredir = 1
[22:08:13.703] RDP backend: enable_hi_dpi_support = 1
[22:08:13.703] RDP backend: enable_fractional_hi_dpi_support = 0
[22:08:13.703] RDP backend: debug_desktop_scaling_factor = 0
[22:08:13.703] RDP backend: enable_window_zorder_sync = 1
[22:08:13.703] RDP backend: enable_distro_name_title = 1
[22:08:13.703] RDP backend: enable_copy_warning_title = 0
[22:08:13.703] Using VSOCK for incoming connections: 3
[22:08:13.704] Registered plugin API 'weston_rdp_output_api_v1' of size 16
[22:08:13.704] Head attaching: rdp-0, index:0, is_primary: 1
[22:08:13.704] get_config: attached head [0]: make:(null), mode:(null), name:rdp-0, (0x557d7e268770)
[22:08:13.704] get_config: attached head [0]: x:0, y:0, width:0, height:0
[22:08:13.704] set_size: attached head [0]: make:weston, mode:rdp, name:rdp-0, (0x557d7e268770)
[22:08:13.704] set_size: attached head [0]: x:0, y:0, width:0, height:0
[22:08:13.705] Output 'rdp-0' enabled with head(s) rdp-0
[22:08:13.705] Compositor capabilities:

versions.txt

WSLg ( x86_64 ): 1.0.29+Branch.main.Sha.2e87e194b507f0ced0b08f715096a35cf9f51498
Mariner: VERSION="1.0.20210224"
FreeRDP: b05321cd4e6a862aef76163a69db4e1910245736
weston: b49e1942ac6242c9dd2118fbd7072e1692f49208
pulseaudio: 2f0f0b8c3872780f15e275fc12899f4564f01bd5
mesa:

Expected behavior
Should open a playable version of SuperTuxKart.
Actual behavior
Window open briefly, segfaults, and then closes, with the following output
[info   ] [FileManager]: Creating directory(ies) '/home/aaditya/.config/supertuxkart/config-0.10/'
[info   ] FileManager: Creating directory '/home/aaditya/.config/supertuxkart/'.
[info   ] FileManager: Creating directory '/home/aaditya/.config/supertuxkart/config-0.10/'.
[info   ] [FileManager]: Creating directory(ies) '/home/aaditya/.local/share'
[info   ] FileManager: Creating directory '/home/aaditya/.local/'.
[info   ] FileManager: Creating directory '/home/aaditya/.local/share/'.
[info   ] FileManager: Creating directory '/home/aaditya/.local/share/supertuxkart'.
[info   ] FileManager: Creating directory '/home/aaditya/.local/share/supertuxkart/addons/'.
[info   ] FileManager: Creating directory '/home/aaditya/.local/share/supertuxkart/addons/icons/'.
[info   ] FileManager: Creating directory '/home/aaditya/.local/share/supertuxkart/addons/tmp/'.
[info   ] FileManager: Creating directory '/home/aaditya/.local/share/supertuxkart/screenshots/'.
[info   ] FileManager: Creating directory '/home/aaditya/.local/share/supertuxkart/replay/'.
[info   ] FileManager: Creating directory '/home/aaditya/.cache/supertuxkart'.
[info   ] FileManager: Creating directory '/home/aaditya/.cache/supertuxkart/cached-textures/'.
[info   ] FileManager: Creating directory '/home/aaditya/.local/share/supertuxkart/grandprix/'.
[verbose  ] main: Error messages and other text output will be logged to /home/aaditya/.config/supertuxkart/config-0.10/stdout.log.
[info   ] UserConfig: Could not read user config file '/home/aaditya/.config/supertuxkart/config-0.10/config.xml'.  A new file will be created.
[info   ] [FileManager]: Data files will be fetched from: '/usr/share/games/supertuxkart/data/'
[info   ] [FileManager]: User directory is '/home/aaditya/.config/supertuxkart/config-0.10/'.
[info   ] [FileManager]: Addons files will be stored in '/home/aaditya/.local/share/supertuxkart/addons/'.
[info   ] [FileManager]: Screenshots will be stored in '/home/aaditya/.local/share/supertuxkart/screenshots/'.
[info   ] [FileManager]: User-defined grand prix will be stored in '/home/aaditya/.local/share/supertuxkart/grandprix/'.
[info   ] [FileManager]: Asset 0 will be loaded from '/usr/share/games/supertuxkart/data/challenges/'.
[info   ] [FileManager]: Asset 1 will be loaded from '/usr/share/games/supertuxkart/data/gfx/'.
[info   ] [FileManager]: Asset 2 will be loaded from '/usr/share/games/supertuxkart/data/grandprix/'.
[info   ] [FileManager]: Asset 3 will be loaded from '/usr/share/games/supertuxkart/data/gui/icons/'.
[info   ] [FileManager]: Asset 4 will be loaded from '/usr/share/games/supertuxkart/data/gui/screens/'.
[info   ] [FileManager]: Asset 5 will be loaded from '/usr/share/games/supertuxkart/data/gui/dialogs/'.
[info   ] [FileManager]: Asset 6 will be loaded from '/usr/share/games/supertuxkart/data/replay/'.
[info   ] [FileManager]: Asset 7 will be loaded from '/usr/share/games/supertuxkart/data/shaders/'.
[info   ] [FileManager]: Asset 8 will be loaded from '/usr/share/games/supertuxkart/data/skins/'.
[info   ] [FileManager]: Asset 9 will be loaded from '/usr/share/games/supertuxkart/data/ttf/'.
[info   ] [FileManager]: Asset 10 will be loaded from '/usr/share/games/supertuxkart/data/po/'.
[info   ] [FileManager]: Asset 11 will be loaded from '/usr/share/games/supertuxkart/data/library/'.
[info   ] [FileManager]: Asset 12 will be loaded from '/usr/share/games/supertuxkart/data/models/'.
[info   ] [FileManager]: Asset 13 will be loaded from '/usr/share/games/supertuxkart/data/music/'.
[info   ] [FileManager]: Asset 14 will be loaded from '/usr/share/games/supertuxkart/data/sfx/'.
[info   ] [FileManager]: Asset 15 will be loaded from '/usr/share/games/supertuxkart/data/textures/'.
[info   ] [FileManager]: Asset 16 will be loaded from '/usr/share/games/supertuxkart/data/tracks/'.
[verbose  ] translation: Env var LANGUAGE = 'C.UTF-8'.
[warn   ] Translation: Unsupported language 'C.UTF-8'
[info   ] ServerConfig: Could not read server config file '/home/aaditya/.config/supertuxkart/config-0.10/server_config.xml'. A new file will be created.
..:: Antarctica Rendering Engine 2.0 ::..

Segmentation fault

I got this same bug with a different application called Octave. Octave is basically open source version of MATLAB. Whenever I generated a plot and interacted with it, I also generated a segfault. However, this error only occurred when the GPU was on.

FWIW if you install supertuxkart as a snap (systemd-genie env here), it works

perhaps this @hideyukn88 ?
https://gist.github.com/djfdyuruiry/6720faa3f9fc59bfdf6284ee1f41f950


Crash server on error in CreateAndStartVisibilityServer.

What would you like to be added:
Use os.Exit(1) on error in CreateAndStartVisibilityServer.
Why is this needed:
The current silencing of the error is not the best.
https://github.com/kubernetes-sigs/kueue/blob/dac07e65a3be053dfac3d21a3fade4c4991a6995/pkg/visibility/server.go#L51-L53
Some users may need it, Kueue will start for them but not work as expected.
Completion requirements:
This enhancement requires the following artifacts:

[ ] Design doc
[ ] API change
[ ] Docs update

The artifacts should be linked in subsequent comments.

+1 I like crashing (calling exit).
The only consideration from me is when users cannot configure visibility correctly, and they don't care about it, they would like to still be able to run Kueue. However, I expect we will get this kind of feedback before graduating to stable, and until then such users will have an option to disable by the feature gate.
cc @PBundyra

Have you seen this error happening in the wild? I wonder what are common scenarios that could lead to it.
Nevertheless, I think it's safer to crash.

I didn't see it

/assign


Rewrite SA1025 code fix to use more efficient text manipulation

The performance of the SA1025 code fix is already good, but it would be even better if it performed the fixes using direct TextChange operations in the same way that the SA1028 code fix already does.

Fixing this!

Marking wontfix based on the performance evaluation in #2209.


Accept to parse sources not ending with '\n'

Allows the source to be parsed to not end in a endline (\n), as the stdlib ast module does.
The goal is to have a pythonparser as near as possible to a drop-in replacement of ast.
This and other compatibility PRs will allows less changes on https://github.com/google/grumpy/pull/216, for example.

Hmm, I'm not sure if this is the right fix. I'll need to look at the root cause of the issue first.

This is not the right fix. Specifically, if someone passes a Buffer without a trailing \n, parsing will fail just like before.

I've reviewed #3 and determined that this is not a bug.


--standard option

Is it possible to load any phpcs.xml containing a ruleset?

coc-php-cs-fixer is a coc extension that integrates with the php-cs-fixer and laravel/pint tools. coc-php-cs-fixer does not support PHP_CodeSniffer (phpcs).
Does the option "--standard" exist in php-cs-fixer or laravel/pint? I couldn't find it.

I couldn't find it either, I likely misunderstood the tool and thought it should have the same options that phpcs.
I'll create my own rules based on the phpcs.xml file. Thanks


Allow package signing

fpm seems to support package signing, it'd be sweet if pkgr would let me do that too. I know it's possible to sign packages after production, but it'd be nice to have that built into this. I suspect it wouldn't be TOO difficult since it's part of fpm.
Just an idea!

I would accept a PR for this, but will close the issue for now.


Create new DirmonEntry from an existing instance.

It can be quite complex to set all the custom values when creating a new dirmon entry. Copying from an existing entry will reduce error when most of the fields are the same.

I'm working on it

I've found how to add an Action column in Enabled Dirmon Entries Page.

I found out that if I wanted to show the copy button, I had to add the name button in

access_policy.rb

I found out that if I wanted to show the copy button, I had to add the name button in access_policy.rb

I'm confused about this part and still cannot create my copy_dirmon_entry_path method. I was advised to review Rails basic tutorial, so I'm doing this now.

I just created the copy page showing the detail of the selected Dirmon Entry.

I just sent my PR about this new process. https://github.com/rocketjob/rocketjob_mission_control/pull/65

Fixed with https://github.com/rocketjob/rocketjob_mission_control/pull/65.
Thank you @Elro1118


MB: See if Automated Ships allows automated flagships to be recoverable even if they shouldn't be

This is apparently working as intended.
fleet_flagship_recoverable is actually for whether the flagship is ALWAYS recoverable. The default, false, means normal vanilla recovery behavior.
I have renamed the variable to fleet_flagship_alwaysRecoverable (backwards compatible) to be less confusing.


Privacy Mode not working properly for some Reports

Defective ToolKit Feature Name: Privacy Mode
Expected Behavior

activate privacy mode
all currency values will be obscured until hovered over
show currency value in reports while hovering over a category

Actual Behavior

activate privacy mode
net worth and spending report shows currency values in graph
net worth and spending report shows obscured values in graph even if hovered over
(it is not possible to place the mouse cursor directly over the number but one would expect hovering the donut piece would be enough)

Steps to Reproduce the Behavior

activate privacy mode in extension settings
activate privacy mode on budget page
show reports

Export of Toolkit Settings (Bottom Left of Settings Page)
[{"key":"accountTransactionSearch","value":false},{"key":"accountsClearSelection","value":false},{"key":"accountsDisplayDensity","value":"0"},{"key":"accountsEmphasizedOutflows","value":false},{"key":"accountsRowHeight","value":"0"},{"key":"accountsSelectedTotal","value":true},{"key":"accountsStripedRows","value":false},{"key":"activityTransactionLink","value":true},{"key":"betterScrollbars","value":"0"},{"key":"budgetBalanceToZero","value":true},{"key":"budgetProgressBars","value":"0"},{"key":"budgetQuickSwitch","value":false},{"key":"budgetRowsHeight","value":"0"},{"key":"calendarFirstDay","value":"1"},{"key":"categoryActivityPopupWidth","value":"0"},{"key":"changeEnterBehavior","value":false},{"key":"checkCreditBalances","value":true},{"key":"checkNumbers","value":false},{"key":"collapseExpandBudgetGroups","value":false},{"key":"collapseSideMenu","value":true},{"key":"colourBlindMode","value":false},{"key":"compactIncomeVsExpense","value":true},{"key":"currentMonthIndicator","value":true},{"key":"daysOfBuffering","value":true},{"key":"daysOfBufferingHistoryLookup","value":"3"},{"key":"editButtonPosition","value":"0"},{"key":"enableRetroCalculator","value":true},{"key":"enlargeCategoriesDropdown","value":true},{"key":"enterInRegisterNow","value":true},{"key":"goalIndicator","value":false},{"key":"goalIndicatorWarningColor","value":false},{"key":"googleFontsSelector","value":"0"},{"key":"hideAOM","value":false},{"key":"highlightNegativesNegative","value":false},{"key":"importNotification","value":false},{"key":"incomeFromLastMonth","value":"0"},{"key":"l10n","value":"0"},{"key":"largerClickableIcons","value":false},{"key":"monthlyNotesPopupWidth","value":"0"},{"key":"options.dark-mode","value":false},{"key":"pacing","value":"0"},{"key":"popupCalculator","value":false},{"key":"printingImprovements","value":true},{"key":"privacyMode","value":"2"},{"key":"reconciledTextColor","value":"4"},{"key":"removePositiveHighlight","value":false},{"key":"removeZeroCategories","value":true},{"key":"reports","value":false},{"key":"resizeInspector","value":true},{"key":"rightClickToEdit","value":true},{"key":"runningBalance","value":false},{"key":"seamlessBudgetHeader","value":false},{"key":"showIntercom","value":true},{"key":"splitKeyboardShortcut","value":true},{"key":"splitTransactionAutoAdjust","value":true},{"key":"squareNegativeMode","value":false},{"key":"swapClearedFlagged","value":false},{"key":"toBeBudgetedWarning","value":false},{"key":"toggleSplits","value":true},{"key":"toggleTransactionFilters","value":"1"},{"key":"warnOnQuickBudget","value":true}]

System Settings
ToolKit Version: 0.7.7
Operating System (Mac/Windows/Etc): Mac El Capitan 10.11.6
Browser (Chrome/Firefox/Safari): Vivaldi
Browser Version: 	1.4.589.38 (Chrome/53.0.2785.148)
Other Extensions: 1Password, Ghostery, Adblock, Trello

I'm having some trouble with this one...I think either @falkencreative or @egens will have to take this one.

This issue should be fixed or improved in the next release, where I've made some adjustments to how this feature works on both the YNAB reports and the Toolkit reports. I unfortunately can't blur the chart numbers (a limitation of CSS styling and how those charts are put together), but I've been able to hide and show them on hover, without the blur effect. I've also been able to remove the blur from graph popups.

Closing this issue now that my improvements have been integrated. Please open a new issue if you are still having trouble or want to discuss this further.


Issue 11: Rename package to io.pravega.conenctors.presto

Signed-off-by: Flavio Junqueira (fpj<EMAIL_ADDRESS>
Renames all packages to be prefixed with io.pravega.connectors.presto.

Fixes #11

@chipmaurer I have been to start the coordinator with the changes in this PR:
2021-03-16T17:53:52.819+0100	INFO	main	com.facebook.presto.metadata.StaticCatalogStore	-- Loading catalog properties etc/catalog/pravega.properties --
2021-03-16T17:53:52.824+0100	INFO	main	com.facebook.presto.metadata.StaticCatalogStore	-- Loading catalog pravega --
2021-03-16T17:53:53.286+0100	INFO	main	Bootstrap	PROPERTY                       DEFAULT      RUNTIME                DESCRIPTION
2021-03-16T17:53:53.286+0100	INFO	main	Bootstrap	pravega.controller             ----         tcp://localhost:9090
2021-03-16T17:53:53.286+0100	INFO	main	Bootstrap	pravega.hide-internal-columns  true         true
2021-03-16T17:53:53.286+0100	INFO	main	Bootstrap	pravega.schema-registry        ----         http://localhost:9092
2021-03-16T17:53:53.286+0100	INFO	main	Bootstrap	pravega.table-description-dir  etc/pravega  etc/pravega
2021-03-16T17:53:54.103+0100	INFO	main	com.facebook.presto.metadata.StaticCatalogStore	-- Added catalog pravega using connector pravega --
2021-03-16T17:53:54.114+0100	INFO	main	com.facebook.presto.security.AccessControlManager	-- Loading system access control --
2021-03-16T17:53:54.115+0100	INFO	main	com.facebook.presto.security.AccessControlManager	-- Loaded system access control allow-all --
2021-03-16T17:53:54.115+0100	INFO	main	com.facebook.presto.storage.TempStorageManager	-- Loading temp storage --
2021-03-16T17:53:54.118+0100	INFO	main	com.facebook.presto.storage.TempStorageManager	-- Loaded temp storage local --
2021-03-16T17:53:54.120+0100	INFO	main	com.facebook.presto.server.PrestoServer	======== SERVER STARTED ========

Is there anything else you think I should be trying?

Yes, it appears that with this PR merge: https://github.com/pravega/presto-connector/pull/14 that I cannot start the server from intelliJ.  Previously, 'build install' copied built jar and dependent jars to build/install/pravega.  Now, only the .gz file is being created, so running from intellij is not possible.

I'm not sure why you are referencing PR #14 here, they are independent PRs. Do you have any other comments on this PR?

OK, I'll ask it this way.  Can you run the server from IntelliJ with these changes?

I'm running from command line, I'm following the instructions in:
https://prestodb.io/docs/current/installation/deployment.html


other: update slf4j-simple from 1.7.30 to 1.7.31

Updates org.slf4j:slf4j-simple from 1.7.30 to 1.7.31.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.slf4j", artifactId = "slf4j-simple" } ]

labels: library-update, semver-patch

Codecov Report

Merging #2 (98bc2de) into master (b9e7ea0) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master       #2   +/-   ##
=======================================
  Coverage   98.11%   98.11%
=======================================
  Files           5        5
  Lines          53       53
  Branches       15       15
=======================================
  Hits           52       52
  Misses          1        1

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 b9e7ea0...98bc2de. Read the comment docs.


React Flow - Maximum update depth exceeded when moving nodes around

What platform were you using when you found the bug?
React Flow, Mac OS, Google Chrome
Live code example
No response
Describe the Bug
Hello!
I have a bug with React Flow.
I have created a flow, where you can add nodes, edit their data, delete, etc.
Sometimes, I get the problem "Maximum update depth exceeded."
This does not happen every time, but if I move them around very fast, select and deselect, etc, than this happen.
I do not know what is the root cause of this, and implementing a sandbox will be hard since I have a lot of logic going on.
I will appreciate any idea you have of might can cause this.
Here is a part of my code:
  const [nodes, setNodes, onNodesChange] = useNodesState(initialNodes)
  const [edges, setEdges, onEdgesChange] = useEdgesState(initialEdges)

  const onSelectionChange = useCallback((elements: any) => {
    const nodes = elements?.nodes
    if (nodes && nodes.length > 0 && nodes[0]?.type !== 'initialNode') {
      const { selected, ...rest } = nodes[0]
      dispatch(SET_SELECTED_NODE(rest))
    } else {
      dispatch(SET_SELECTED_NODE({}))
    }
  }, [])

  const handleNodesChange = (changes: NodeChange[]) => {
    const nextChanges = changes.reduce((acc, change) => {
      // Disable backspace / delete key
      if (change.type === 'remove') {
        return acc
      }
      // all other change types are just put into the next changes arr
      return [...acc, change]
    }, [] as NodeChange[])

    // apply the changes we kept
    onNodesChange(nextChanges)
  }

  const handleEdgesChange = (changes: EdgeChange[]) => {
    const nextChanges = changes.reduce((acc, change) => {
      // Disable backspace / delete key
      if (change.type === 'remove') {
        return acc
      }
      // all other change types are just put into the next changes arr
      return [...acc, change]
    }, [] as EdgeChange[])

    // apply the changes we kept
    onEdgesChange(nextChanges)
  }

  useEffect(() => {
    // This uses dagre to position the nodes correctly
    if (nodes.length > 0 && edges.length > 0) {
      const { nodes: layoutedNodes, edges: layoutedEdges } = getLayoutedElements(nodes, edges)
      setNodes(layoutedNodes)
      setEdges(layoutedEdges)
    }
  }, [])

  return (
    <ReactFlow
      nodes={nodes}
      edges={edges}
      nodeTypes={nodeTypes}
      onNodesChange={handleNodesChange}
      onEdgesChange={handleEdgesChange}
      onSelectionChange={onSelectionChange}
      onLoad={() => setTimeout(() => fitView(), 0)}
      >
      <CustomControls />
      <Background />
    </ReactFlow>
  )

Thank you very much! :)
Steps to reproduce the bug or issue
Moving the nodes around fast, selecting & deselecting while moving the nodes, etc.
Expected behavior
No errors.
Screenshots or Videos
No response
Additional context
No response

// Disable backspace / delete key
if (change.type === 'remove') {
  return acc
}

Not necessarily specific to the issue at hand but you can also just use <ReactFlow deleteKeyCode={null} ...> to disable the delete key.

Very hard to help without a reproduction in this case.

I could imagine that wrapping handleNodesChange and handleEdgesChange with useCallback fixes this. If not, please re-open with a codesandbox https://new.reactflow.dev


Change Styling / Font-Size

I have tried and tried and I cannot seem to make any changes to the font sizes, card height or any other styling. I wish this could be done to fit the card better om smaller areas.
Any chance I am missing specificly how this can be done?

I forgot to mention that I am in love with the look and function of the card itself. My wife has explicitly asked me to keep it on our dashboard in the hallway but its taking quite the realestate to have.

I second that...it would be great to "squeeze" card a bit...


Replace current image implementation with eleventy-img

Eleventy has an official image plugin:
https://github.com/11ty/eleventy-img
It would be cool to ditch the current implementation in favour of the official plugin, as well as implement an include template for images.

We need:

Specify alt description
Specify caption
Support Retina displays
AVIF, WebP, JPEG formats for each image
Responsive sizes for each image (for example, 600px/auto for mobile devices and 1200px/auto for bigger devices)

Captions relate to figures, not images.
Retina images support might happen at some point via the Eleventy plugin itself.


Workspace view is not able to show directories (and their contents).

Acorn app with commit id - https://github.com/gptscript-ai/desktop/commit/1eeb6f0161e8eee00f8aa1a7ebf2780dee3a8672
Steps to reproduce the problem:

Start a conversation with Default Assistant
Change workspace to say - /dir1 using manage workspace and confirm to restart scripts.
View workspace set using  manage workspace.

Notice that only files present in this directory are listed.
Directories are not listed.
Expected Behavior:
Directories should be listed and user should have the ability to expand the directory structure to see all the files in this directory and any sub directories.

With latest electron builds , we are not able to show the files that get added using "Add knowledge" option in "Manage Workspace" directory since they get added in knowledge directory under workspace directory.

 % ls /Users/sangeethahariharan/Library/Application\ Support/acorn/Acorn/threads/o47enc/workspace/knowledge
MyFeb12.pdf	TravelPlan.pdf


[18.09] fix buildbot

Motivation for this change
#56053
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

cc @nand0p @ryansydnor @lopsided98

Assuming that we also merge #56200 into release-19.03, this will be fixed anyway.

@dotlambda How? This is for 18.09 and I don't see anything relevant in #56200 anyway.
I think we should just upgrade sqlalchemy_migrate, rather than creating a new package just for Buildbot. The root cause of the Buildbot test failures is actually a SQLite update (which was backported to 18.09), which broke sqlalchemy_migrate. sqlalchemy_migrate 0.12.0 contains fixes to make it compatible with the latest version of SQLite. Buildbot's comprehensive tests happened to catch the issue at build time, but any package that uses sqlalchemy_migrate 0.11.0 with SQLite is likely to break at runtime.

@lopsided98 I think you are right. Looking at the changes between 0.11.0 and 0.12.0 they look like just bugfixes: https://gist.github.com/veprbl/b8c50d01343157507990a30d183927e2

Sorry, I mixed up 18.09 and 19.03.

This broke sqlalchemy_migrate:

https://hydra.nixos.org/job/nixos/release-18.09/nixpkgs.python27Packages.sqlalchemy_migrate.x86_64-linux
https://hydra.nixos.org/build/89918316

Those tests failed randomly on Hydra, so I disabled them for release-18.03 in https://github.com/NixOS/nixpkgs/commit/31e932c0d410a4d4a31c6737c25c0557dea1c72c
It would be nice if maintainers fixed that properly on master and future release-19.03. Example of this problem on trunk is:
https://hydra.nixos.org/build/83000720
cc @makefu

@veprbl is sqlalchemy_migrate also broken on 19.03 in version 0.12.0? It seems that some version update broke the tests (again).
I can have a look at it.

@makefu I don't know for sure. There were only two evaluations for 19.03 so far: https://hydra.nixos.org/job/nixos/release-19.03/nixpkgs.python27Packages.sqlalchemy_migrate.x86_64-linux
There were more evaluations for master ("trunk" on Hydra) so that's why I checked there.

Last successful build 2018-10-13 17:57:56 This build 2018-10-18 14:29:23
It is broken since almost 5 months and not really a "random fail", right?

I think this really means "Previous successful build". You can find this build in the list of trunk evaluations:
https://hydra.nixos.org/job/nixpkgs/trunk/python27Packages.sqlalchemy_migrate.x86_64-linux/all

If not on release-19.03, it might be broken on staging-19.03.

I'm also not sure if this is caused by version update. Looking at different number of repetition of the output:
${PYTHON:-python} -m subunit.run discover -t ./ .  --load-list /build/tmpy7vZIL
running=OS_STDOUT_CAPTURE=${OS_STDOUT_CAPTURE:-1} \
OS_STDERR_CAPTURE=${OS_STDERR_CAPTURE:-1} \
OS_TEST_TIMEOUT=${OS_TEST_TIMEOUT:-60} \

I get much fewer of these on my machines and during successful builds on Hydra.
I think this must be a some kind of timeout problem, maybe we could increase those.

so this means that sqlalchemy_migrate in version 0.12.0 looks fine up until now. 0.11.0 tests only broke once (hence "random fail" i guess) in october 2018. TBH i really do not know what happened in this build with the shutil thingy.

so this means that sqlalchemy_migrate in version 0.12.0 looks fine up until now

I already mentioned that 0.12.0 broke on release-18.09 in the same way https://hydra.nixos.org/build/89918316

well that sucks , the test case which is failing is different but both times a shutil copy step fails which seems to be used to prepare a test environment in /tmp/.
I am unsure where this error could come from.


Sense license can be 1 per mesh

Sense consists of

dashboard
catalog
slo

If any of these exist in a mesh, it counts as 1 license of Sense

This is addressed in catalog-count but not yet in kubetctl-count


federation-plugin: toSubGraphSchema is not a function

i followed the guide on https://pothos-graphql.dev/docs/plugins/federation and i get

export const schema = builder.toSubGraphSchema({
                              ^
TypeError: builder_1.builder.toSubGraphSchema is not a function

its a bit odd, the types say that this function exist, but the builder does not have this function.
i made sure i added the plugin:
export const builder = new SchemaBuilder<SchemaTypes>({
  plugins: [PrismaPlugin, PrismaUtils, DirectivePlugin, FederationPlugin],

  prisma: {
    client: prisma,
    filterConnectionTotalCount: true,
  },
});

i also tried to remove the other plugins to see whether there are some incompatibilities.
versions i use:

additionaly, all other methods attached by this plugin do not work, like schema.selection
seems like this is not executed: https://github.com/hayes/pothos/blob/main/packages/plugin-federation/src/schema-builder.ts

Nothing obvious here, but it looks like your error is coming from some sort of compiled/generated/bundled code.  It's possible what ever tool you are using to build your app is somehow eliminating excluding the part of the plugin that defines that function.  The function is defined here: https://github.com/hayes/pothos/blob/main/packages/plugin-federation/src/schema-builder.ts and is imported from the plugins index file.

Nothing obvious here, but it looks like your error is coming from some sort of compiled/generated/bundled code. It's possible what ever tool you are using to build your app is somehow eliminating excluding the part of the plugin that defines that function. The function is defined here: https://github.com/hayes/pothos/blob/main/packages/plugin-federation/src/schema-builder.ts and is imported from the plugins index file.

its a freshly created project with just ts-node to run the ts code directly on node 18. I also tried bundling it with esbuild and then run the compiled js with the same result. Both environments may tree-shake out the code as it relies on a side effect by importing the file you linked. Bundlers usulay should not tree-shake away code with potential side effects though (unless you mark your package as side-effect free) and in my case there isn't even a bundler running (not sure though how ts-node handles this internally)

what is smelly is that https://github.com/hayes/pothos/blob/main/packages/plugin-federation/src/index.ts only imports a specific export from https://github.com/hayes/pothos/blob/main/packages/plugin-federation/src/schema-builder.ts
import { entityMapping } from './schema-builder'
but this file has other exports (which are never imported anywhere as well as side-effects.
other plugins do a import "./schema-builder" to explicitly import it for its side-effects.
i advice to remove any export from this file and move them somwhere else and import it for its side effects. E.g. if you would no longer need entityMapping, you would no longer import the file at all

All plugins use the same pattern for extending Pothos classes.  Do the other plugins you are using successfully add methods?
It might be worth checking if the federation plugin is somehow referencing a different copy of Pothos.
Are you using cjs or esm as the target module type for your app?

All plugins use the same pattern for extending Pothos classes. Do the other plugins you are using successfully add methods?
It might be worth checking if the federation plugin is somehow referencing a different copy of Pothos.
Are you using cjs or esm as the target module type for your app?

ts-config:
{
  "compilerOptions": {
    "esModuleInterop": true,
    "strict": true
  }
}

while writing this i checked the typescript version, and it seems i used a relativly ancient one (4.5.4).
I now use 5.0.4 and the error is gone!
i am not sure though why 4.5.4 tree-shakes that away though...

Nice, glad you got it figured out.  Good catch on the import.  Agree that should be updated to pull named imports from somewhere else, I'll get that updated later today

published a new version of several packages that avoid importing from files that have side effects.  Hopefully that will prevent anyone from running into this in the future.  Thanks for the report, but also happy this is not an issue in newer typescript versions.


Drop JSON dependency

All major Ruby implementations such as MRI, JRuby, and Rubinius ship with JSON. Many gems including Rails already stepped away from it, and it is no longer necessary to depend on it.

@yuki24 Thanks - good idea!

Released as 1.0.1 - sorry for the delay :)


Branch update ugd gv2

In general,

changed instances of people/persons to client, changed instances of AddressBook to FAST.
Added usage scenarios for each command.
Added more to the intro section.
Added hyperlinks to refer to each section. Removed duplicated sections.
Improved language used at the intro portion to be more user-centric

Codecov Report

Merging #126 (aad03ee) into master (3a7ab29) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #126   +/-   ##
=========================================
  Coverage     71.95%   71.95%
  Complexity      650      650
=========================================
  Files           102      102
  Lines          2104     2104
  Branches        249      249
=========================================
  Hits           1514     1514
  Misses          528      528
  Partials         62       62

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 3a7ab29...aad03ee. Read the comment docs.


The image builded with this Dockerfile doesn't work.

I have cloned this repo, and builded a image with this Dockerfile. And I ran a container with this image and a flasky application container. When I input Chinese sentence in flasky blog, there was Internal Error.
I have checked the option files in Mysql Docs, the option file should be in the /etc/mysql/ directory.
I have checked the charset applications in Mysql Docs . collation-server=utf8_unicode_ci in .cnf file should be 'collation-server=utf8_general_ci'.
After I modified these errors, I build and run the containers again, I can input Chinese sentence correctly.

Thank you for this comment @franklili3
The issues may be due to the fact this image was created 3 years ag.
I will take the suggested changes into account as soon as I find a bit of free time (hopefully very soon !)


Fix OOM in LS

Purpose
Fixes #https://github.com/ballerina-platform/ballerina-lang/issues/36527

@azinneera, Until we come up with a proper solution as mention in https://github.com/ballerina-platform/ballerina-lang/issues/37242, we have to change several project API tests. Can you please check those?


[Question] Generator can't hold on to power?

Something I have noticed is while the Generator has an internal storage for power after a bit that power seems to simply discharge from the buffer. In my setup I got 2 EnderIO basic capacitors, some machines and a screen controller from RFtools. The screen controller is the only one which constantly drain power (about 14 RF/t). I managed to make a pretty good crystal which gave a lot of power and not that much radiation (around 6300 according to the radiation probe). It quickly filled up my system so I turned it off. There was still over 300k RF in the generator when I turned it off. So things went along and it slowly drained (according to the RFtools power observer thingy). But then suddenly after a few minutes it started dropping way faster. But nothing had changed in the meantime. The machines were off, capacitors full and the screen controller didn't drain that much (like 1000 RF per second). So I wonder if this is an intended feature or a bug?
EDIT: So I used the RF power system tool multimeter thingy and strangely all it said was that 16-14 RF/t was drained but in fact 100-300 RF was disappearing from the generator per second.  (For some reason the power in EnderIO conduits fluctuate some but only like +/- 1 but I don't think that's the issue?)
EDIT 2: Huh. After it reached 124,2k it stopped free falling and went back to a slow decline.

This is not something that can happen. The generator keeps its power forever unless something connected to the generator is draining power

I can try to disconnect it from the power network and see if it is still happening. Then report back to you. If it keeps happening then I'll know it's an issue with the reactor/version I'm using (I'm on the DireWolf20 pack for 1.10) and not me kicking up a machine which I forgot about. I'll try it.

Thanks for your reply. At least I know it's not intended behavior. So either a bug or me being forgetful (which is possible)


template structure

figure out json structure to represent existing templates in csv. UI will hydrate this to create template picker for market creation page.

@bthaile Added to MC Template Epic

this is complete


Update webjars-play to 2.8.13

Updates org.webjars:webjars-play from 2.6.3 to 2.8.13.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.webjars", artifactId = "webjars-play" } ]

labels: library-update, early-semver-minor, semver-spec-minor, commit-count:1

Superseded by #103.


Increase touch area of download button

Addresses https://openedx.atlassian.net/browse/MA-1988
Not the most elegant solution but definitely better. I tried looking at TouchDelegate but it seems a bit complicated (and overkill) compared to this solution.
@1zaman @miankhalid

Just curious, have you RTL tested your solution ?

Have not. I haven't done any RTL testing :/

This layout is RTL supportive. Lemme check if the change affects it somehow.

Other than the nit :+1:


Prepare to call jit from block machine.

This PR performs preliminary preparations in the block machine so that it will be able to JIT-compile and evaluate lookups into this machine given a certain combination of "known inputs".

ready for review again?

LGTM, but has conflicts.


fix query predicate bug for NotEqual expression type

Any chance we can get this? Looks useful to me!

Hi @mdrakib - first off, thanks for opening this, I'm assuming this is query generation for numeric !=? can you add a test to show what it's fixing?
thanks!

Hi @slorello89
Thanks for your reply, I have added a test coverage, please review.


Enhancements to the ICS3 TLA+ spec

Summary
Following the work on #58 there are several issues that were raised and may deserve further consideration in a separate PR.
For instance:

removing the latest height from the client record, cf. @milosevic suggestion.
introduce malicious env. behavior, captured separately in #70
refine the level of abstraction we want to target, cf @milosevic suggestion.

Problem Definition

Proposal

For Admin Use

[ ] Not duplicate issue
[ ] Appropriate labels applied
[ ] Appropriate contributors tagged
[ ] Contributor assigned/self-assigned

Another sub-issue that we should consider tackling: alignment with the VDD traceability proposal. Concretely, TLA+ operators should be tagged, for instance.

Some of these issues overlap with #89.


Fix accessibilityRole in 0.73-stable

Please select one of the following

[ ] I am removing an existing difference between facebook/react-native and microsoft/react-native-macos :thumbsup:
[ ] I am cherry-picking a change from Facebook's react-native into microsoft/react-native-macos :thumbsup:
[x] I am making a fix / change for the macOS implementation of react-native
[ ] I am making a change required for Microsoft usage of react-native

Summary:
During the merge for 0.73, accessibilityRole was accidentally put behind an #if block removing it from macOS's implementation.  This simply restores the functionality of the prop to what it was in 0.72, but more changes are forthcoming to refactor and support the new role prop.
Changelog:
[MACOS] [FIXED] - Fixed accessibilityRole
Test Plan:
See before/after of Accessibility Inspector for accessibilityRole="button" changing from AXUnknown to AXButton

Thanks for the PR! There’s issues with CI so I might have to force merge it. Can you also prepare the 0.73 version so I can merge both?

Thanks for the PR! There’s issues with CI so I might have to force merge it. Can you also prepare the 0.73 version so I can merge both?

This is the 0.73 PR. Did you mean main?

Thanks for the PR! There’s issues with CI so I might have to force merge it. Can you also prepare the 0.73 version so I can merge both?

This is the 0.73 PR. Did you mean main?

Oh yes, just noticed. Yes, main please. Also for future reference we try to prefix PRs to one of the stable branches with "[0.73] PR title" just so that it's easier to distinguish!

Thanks for the PR! There’s issues with CI so I might have to force merge it. Can you also prepare the 0.73 version so I can merge both?

This is the 0.73 PR. Did you mean main?

Oh yes, just noticed. Yes, main please. Also for future reference we try to prefix PRs to one of the stable branches with "[0.73] PR title" just so that it's easier to distinguish!

Yea, I just noticed that looking at other PRs. Noted 😅. I'll get the main one out shortly.


#25553 Locked

Checklist

[x] I'm asking a question
[x] I've looked through the README and FAQ for similar questions
[x] I've searched the bugtracker for similar questions including closed ones

Question

Hi, I'm wondering why issue #25553 is locked and if there are any updates on it? I can't upvote or respond to participate in the normal way.
Thank you!

Because of spam/useless +1 noise.


[RESOLVED] Interface eth0(): High error rate (>2 for 5m)

Problem started at 13:51:29 on 2023.08.11
Problem name: Interface eth0(): High error rate (>2 for 5m)
Host: S100-Primario
Severity: Warning
Operational data: errors in: 3, errors out: 0
Original problem ID: 35288
Event details in Zabbix: http://<IP_ADDRESS>/tr_events.php?triggerid=23483&eventid=35288

Problem has been resolved in 12m 0s at 14:03:29 on 2023.08.11
Problem name: Interface eth0(): High error rate (>2 for 5m)
Host: S100-Primario
Severity: Warning
Original problem ID: 35288


Drop UI components from OONI API that are duplicated in OONI Explorer

There are several bits of UI which currently exist in OONI API which are going to be integrated into the new OONI Explorer.
As soon as the new OONI Explorer is released we should drop all the duplicate bits of code from OONI API and consider most (if not all) of the UI bits from OONI API inside of Explorer (an API is an Application Programmer Interface and it's not meant to have a GUI beyond that needed for documentation).
Things I think we could drop are:

[ ] The https://api.ooni.io/stats page. We have this now on the landing page of Explorer
[ ] All the https://api.ooni.io/files listing. We should support consulting the measurements inside of OONI Explorer and there is no need to duplicate this functionality.

We could also investigate remove complexity around documentation pages. Possibly save them into static pages and commit them into git.


Fix error UnicodeDecodeError in idf_tools.py

Fix error decode() by adding utf-8, just below install.sh with the error.
$ ./install
Installing ESP-IDF tools
Installing tools: xtensa-esp32-elf, esp32ulp-elf, openocd-esp32
Skipping<EMAIL_ADDRESS>(already installed)
Traceback (most recent call last):
  File "/data/sdk/esp/esp-idf/tools/idf_tools.py", line 1317, in <module>
    main(sys.argv[1:])
  File "/data/sdk/esp/esp-idf/tools/idf_tools.py", line 1313, in main
    action_func(args)
  File "/data/sdk/esp/esp-idf/tools/idf_tools.py", line 1098, in action_install
    tool_obj.find_installed_versions()
  File "/data/sdk/esp/esp-idf/tools/idf_tools.py", line 503, in find_installed_versions
    ver_str = self.check_version(self.get_export_paths(version))
  File "/data/sdk/esp/esp-idf/tools/idf_tools.py", line 445, in check_version
    in_str = version_cmd_result.decode()
UnicodeDecodeError: 'ascii' codec can't decode byte 0xc3 in position 110: ordinal not in range(128)

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

Hi @cleitonbueno. Thank you for your contribution but this was reported a while ago (https://github.com/espressif/esp-idf/issues/3841) and we've already fixed it but not yet published.
This will be fixed in master and in v4.0 ASAP. Thank you for your patience and understanding.

Hi @cleitonbueno. Thank you for your contribution but this was reported a while ago (#3841) and we've already fixed it but not yet published.
This will be fixed in master and in v4.0 ASAP. Thank you for your patience and understanding.

Okay, I hope I can help more.


Lookup dragging key when in dragging mode

Fixes bug introduced in 22916e9 where dragging of floating windows was messed up.

Will test this in a bit.


Use SecureRandom for avoiding the port conflicts

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (66db890) 40.16% compared to head (8f0cc6e) 40.15%.

Additional details and impacted files
@@              Coverage Diff              @@
##             develop     #219      +/-   ##
=============================================
- Coverage      40.16%   40.15%   -0.02%
  Complexity      1186     1186
=============================================
  Files            246      246
  Lines           9714     9714
  Branches         928      928
=============================================
- Hits            3902     3901       -1
  Misses          5544     5544
- Partials         268      269       +1

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Return user networking mode as default on macOS and linux & Remove unused flags from goproxy integration test

Fixes: Issue #4335
Solution/Idea
This PR addresses two issues I encountered while running integration tests on my local machine

The default networking mode returned should be based on the OS
goproxy test is defining and parsing unused flags which are causing a post test-run panic

Proposed changes

Removed the unused flags
Modified the function for fetching the default networking mode

Testing
make GINKGO_OPTS="--ginkgo.label-filter='openshift-preset'" BUNDLE_PATH="--bundle-path=~/.crc/cache/crc_vfkit_4.16.7_arm64.crcbundle" PULL_SECRET_PATH="--pull-secret-path=~/Downloads/pull-secret.txt" integration

No panics now..
% make GINKGO_OPTS="--ginkgo.label-filter='goproxy'" BUNDLE_PATH="--bundle-path=/Users/anath/.crc/cache/crc_vfkit_4.16.7_arm64.crcbundle" PULL_SECRET_PATH="--pull-secret-path=/Users/anath/Downloads/pull-secret" integration
Running Suite: Integration - /Users/anath/work/github.com/crc-org/crc/test/integration
======================================================================================
Random Seed:<PHONE_NUMBER>

Will run 7 of 36 specs
•••••••SSSSSSSSSSSSSSSSSSSSSSSSSSSSS

Ran 7 of 36 Specs in 953.907 seconds
SUCCESS! -- 7 Passed | 0 Failed | 0 Pending | 29 Skipped
PASS
ok      github.com/crc-org/crc/v2/test/integration      955.429s

@vyasgun need to run make fmt; the CI tests are currently failing because of:
pkg/crc/network/types.go:5: File is not `goimports`-ed (goimports)
	"github.com/crc-org/crc/v2/pkg/crc/logging"
make: *** [cross-lint] Error 1

Thanks @anjannath ! For some reason, make lint fails on my system with the following panics:
gvyas@Gunjans-MacBook-Pro crc % make lint
"/Users/gvyas/work/crc/tools/bin"/golangci-lint run
ERRO [linters_context/goanalysis] fact_purity: panic during analysis: interface conversion: interface {} is nil, not *buildir.IR, goroutine 12222 [running]:
runtime/debug.Stack()
        /opt/homebrew/Cellar/go/1.23.0/libexec/src/runtime/debug/stack.go:26 +0x64
github.com/golangci/golangci-lint/pkg/goanalysis.(*action).analyzeSafe.func1()
        /Users/gvyas/work/crc/tools/vendor/github.com/golangci/golangci-lint/pkg/goanalysis/runner_action.go:105 +0x4c
panic({0x101b30000?, 0x1400601d1a0?})
        /opt/homebrew/Cellar/go/1.23.0/libexec/src/runtime/panic.go:785 +0x124
honnef.co/go/tools/analysis/facts/purity.purity(0x1400c802380)
        /Users/gvyas/work/crc/tools/vendor/honnef.co/go/tools/analysis/facts/purity/purity.go:109 +0x270
github.com/golangci/golangci-lint/pkg/goanalysis.(*action).analyze(0x14004249d10)
        /Users/gvyas/work/crc/tools/vendor/github.com/golangci/golangci-lint/pkg/goanalysis/runner_action.go:191 +0x8ac
github.com/golangci/golangci-lint/pkg/goanalysis.(*action).analyzeSafe.func2()
        /Users/gvyas/work/crc/tools/vendor/github.com/golangci/golangci-lint/pkg/goanalysis/runner_action.go:113 +0x20
github.com/golangci/golangci-lint/pkg/timeutils.(*Stopwatch).TrackStage(0x1400282c2d0, {0x10173ebc3, 0xb}, 0x1400370af30)
        /Users/gvyas/work/crc/tools/vendor/github.com/golangci/golangci-lint/pkg/timeutils/stopwatch.go:111 +0x44
github.com/golangci/golangci-lint/pkg/goanalysis.(*action).analyzeSafe(0x1010948e0?)
        /Users/gvyas/work/crc/tools/vendor/github.com/golangci/golangci-lint/pkg/goanalysis/runner_action.go:112 +0x70
github.com/golangci/golangci-lint/pkg/goanalysis.(*loadingPackage).analyze.func2(0x14004249d10)
        /Users/gvyas/work/crc/tools/vendor/github.com/golangci/golangci-lint/pkg/goanalysis/runner_loadingpackage.go:80 +0xac
created by github.com/golangci/golangci-lint/pkg/goanalysis.(*loadingPackage).analyze in goroutine 3677
        /Users/gvyas/work/crc/tools/vendor/github.com/golangci/golangci-lint/pkg/goanalysis/runner_loadingpackage.go:75 +0x174
ERRO [linters_context/goanalysis] nilness: panic during analysis: interface conversion: interface {} is nil, not *buildir.IR, goroutine 12218 [running]:
runtime/debug.Stack()
        /opt/homebrew/Cellar/go/1.23.0/libexec/src/runtime/debug/stack.go:26 +0x64
github.com/golangci/golangci-lint/pkg/goanalysis.(*action).analyzeSafe.func1()
        /Users/gvyas/work/crc/tools/vendor/github.com/golangci/golangci-lint/pkg/goanalysis/runner_action.go:105 +0x4c
panic({0x101b30000?, 0x140060441e0?})
        /opt/homebrew/Cellar/go/1.23.0/libexec/src/runtime/panic.go:785 +0x124
honnef.co/go/tools/analysis/facts/nilness.run(0x1400c98a0e0)
        /Users/gvyas/work/crc/tools/vendor/honnef.co/go/tools/analysis/facts/nilness/nilness.go:65 +0x244
github.com/golangci/golangci-lint/pkg/goanalysis.(*action).analyze(0x140043b65a0)
        /Users/gvyas/work/crc/tools/vendor/github.com/golangci/golangci-lint/pkg/goanalysis/runner_action.go:191 +0x8ac
github.com/golangci/golangci-lint/pkg/goanalysis.(*action).analyzeSafe.func2()
        /Users/gvyas/work/crc/tools/vendor/github.com/golangci/golangci-lint/pkg/goanalysis/runner_action.go:113 +0x20
github.com/golangci/golangci-lint/pkg/timeutils.(*Stopwatch).TrackStage(0x1400282c2d0, {0x1016ecf00, 0x7}, 0x1400326f730)
        /Users/gvyas/work/crc/tools/vendor/github.com/golangci/golangci-lint/pkg/timeutils/stopwatch.go:111 +0x44
github.com/golangci/golangci-lint/pkg/goanalysis.(*action).analyzeSafe(0x1010948e0?)
        /Users/gvyas/work/crc/tools/vendor/github.com/golangci/golangci-lint/pkg/goanalysis/runner_action.go:112 +0x70
github.com/golangci/golangci-lint/pkg/goanalysis.(*loadingPackage).analyze.func2(0x140043b65a0)
        /Users/gvyas/work/crc/tools/vendor/github.com/golangci/golangci-lint/pkg/goanalysis/runner_loadingpackage.go:80 +0xac
created by github.com/golangci/golangci-lint/pkg/goanalysis.(*loadingPackage).analyze in goroutine 3677
        /Users/gvyas/work/crc/tools/vendor/github.com/golangci/golangci-lint/pkg/goanalysis/runner_loadingpackage.go:75 +0x174
ERRO [linters_context/goanalysis] typedness: panic during analysis: interface conversion: interface {} is nil, not *buildir.IR, goroutine 12220 [running]:
runtime/debug.Stack()

Figuring out what's wrong. I've updated the PR with the change that was mentioned in the logs so hope the CI tests pass

Thanks @anjannath ! For some reason, make lint fails on my system with the following panics:

This can happen when golangci-lint is built with a go version older than the one installed on your system. You can try removing  ./tools/bin/golangci-lint and running make lint again.


Unable to use case_when function properly

I am trying to use case_when function and able to replace the values where conditions are met but as per the CASE statement, we have to specify the ELSE part which verticapy allows to mention within other parameters. The issue is that, if I want to use other values unchanged I have to specify the column name as it is like we did in SQL case statement but vertica is automatically using /' before and after when creating its SQL thus throwing error.

Hi @SMoazzamR,
Have you tried setting the auto_quote parameter to False in order to enter pure SQL ? In this case if you want a string as a condition you'll have to write "'value'".

Thanks, Issue is resolved if I want to use the value of column in ELSE part then I have to use like others="column1" and in conditions={'a<b':"'hardcode value'"}


Debug flag support for different Intel compiler versions

ESMA_cmake debug flags used with Intel compilers include unitialized variables check -check uninit.
https://github.com/GEOS-ESM/ESMA_cmake/blob/c2e6fc5c9903624a443f96b3a50c62f22aad5a3d/Intel.cmake#L58
This works with ifort19 but causes an error with ifort18 when building MAPL, as reported by a GCHPctm user (https://github.com/geoschem/GCHPctm/issues/35). The error message is:
[ 15%] Building Fortran object src/MAPL/MAPL_cfio_r4/CMakeFiles/MAPL_cfio_r4.dir/ESMF_CFIOBaseMod.f.o
ifort: error #10236: File not found:  'uninit'
make[2]: *** [src/MAPL/MAPL_cfio_r4/CMakeFiles/MAPL_cfio_r4.dir/ESMF_CFIOBaseMod.f.o] Error 1
make[2]: *** Deleting file `src/MAPL/MAPL_cfio_r4/CMakeFiles/MAPL_cfio_r4.dir/ESMF_CFIOBaseMod.f.o'
make[1]: *** [src/MAPL/MAPL_cfio_r4/CMakeFiles/MAPL_cfio_r4.dir/all] Error 2
make: *** [all] Error 2
Changing the flag -check,uninit fixes the problem, but then makes it incompatible with ifort19.
Do you have plans to allow for version-specific flags in ESMA_cmake to fix issues like this?

Hmm.  I thought -check unit was supported for quite some time.

Looks like I misread the original poster's suggested fix. It is actually this: I recommend changing -check bounds -check uninit to -check bounds,uninit

The error must be more subtle.   I've confirmed that on our cluster Intel 17, 18 and 19 all support check unit.  (Further, we routinely build GEOS using Intel 18 and the same Intel.cmake.
Could the user provide us with (a) the output from the cmake config step and (b) the log for a serial build using VERBOSE=1?    I want to see what command and flags is actually being issued to the compiler.

@lizziel Our messages crossed each other.    I'd still like to see what is actually going wrong.   I know there are parts of the model where @mathomp4 had to do some fairly ugly things to get compiler flags in the right order, but ... this is in MAPL.  (And again, the existing logic works for us.)
Regardless of any bugs, I defer to @mathomp4 as to whether the concatenate form for the compiler flags is easier for us to work with or not.

I reproduced the issue on my end using the original flags in Intel.cmake.  I am using the following libraries:
Intel/18.0.5
OpenMPI/4.0.1
CMake/3.16.1
Here are the logs:
configure.txt
compile.txt
compile_verbose.txt

Ah the error is happening in pFlogger.  That's an important clue, as we build pFlogger separately at this end.    I'll have to dig down to see how the esma flags override the ones in pFlogger.   And it begs the question as to where the bug really is in any event.

@lizziel   Looking closer,  I see that your logs are insufficent.   compile.txt was parallel which mixed the error messages, so that I can't see which files actually had the error.  Maybe it is pFlogger, maybe it was an earlier file in the log.    And the compile_verbose.txt does not show any errors.  Just stops at 33%.   Maybe you hit an upload limit on GitHub?    If you put the file on our cluster, I can look at it there.

@tclune, right, my build is parallel so the error message does not necessarily point to pflogger. I did not hit an upload limit - the verbose option when sending stdout and stderr to log file prints the error message to screen instead. Is there a different data stream being used? For the non-verbose build it was short enough to copy to file. For the verbose, not so much.

@lizziel   I don't know where the modified flags are coming from in our example.    Our Intel.cmake  has
... -check bounds -check uninit ...

But your reproducer has
... -check bounds uninit ...

So some layer is modifying the flags and getting it wrong.   This does not happen with GEOS so I suspect the modifications are happening on your end.    Happy to accept a PR on our end that resolves this, so long as it does not break GEOS.

Interesting. I'll see if I can figure out where that is coming from. @LiamBindle, any ideas?

@lizziel @tclune Sorry I didn't see this until now. Thanks for @ me Lizzie.
The problem is the compiler option de-duplicator. CMake runs a step where it removes duplicate flags to avoid passing the same compiler option many times. This is avoided in GEOS because GEOS-ESM/ESMA_cmake sets compiler options like
set (CMAKE_Fortran_FLAGS_DEBUG   "${GEOS_Fortran_FLAGS_DEBUG}" CACHE STRING "Debug Fortran flags" FORCE )

and the de-duplicator does not run on CMAKE_Fortran_FLAGS_<CONFIG>. The de-duplicator runs on options specified via target properties.
In GCHP we set compiler options like so
separate_arguments(GEOS_Fortran_FLAGS_<CONFIG> UNIX_COMMAND ${GEOS_Fortran_FLAGS_RELEASE})

and then add ${GEOS_Fortran_FLAGS_<CONFIG>} to targets via the target_compile_options() command. In GCHP we do this to de-couple the compiler arguments of MAPL and GEOS-Chem since CMAKE_Fortran_FLAGS_<CONFIG> is a (directory-down) global variable.

@lizziel I think this will be fixed by wrapping multiword arguments with "SHELL:word1 word2 [...]". For example, -check bounds should be "SHELL:-check bounds" and -check uninit should be "SHELL:-check uninit". This tells the de-duplicator that -check bounds and -check uninit are arguments, not -check, bounds, -check, and uninit which results in the second -check getting dropped.

This is actually the same thing that's blocking #57. The holdup for setting comiler options via target properties is that compiler options have to have a consistent ordering in GEOS (for bitwise zero-diff requirement). So for the time being this is GCHP specific.
Note: I think we could actually get consistent option ordering in GEOS via target properties by wrapping the entire contents of CMAKE_Fortran_FLAGS_<CONFIG> with SHELL:. That should cause the de-duplicator to treat the entire string as one argument, and preserve its current ordering. Perhaps that's something I could explore further.

@LiamBindle I am looking at changing the Intel Debug flags to just use -check all. As long as GEOS runs with it, it's probably a better option anyway!

That would be great. I will try it and see if it works on our end.

Well the answer is nope, at least for binarytile.x. Died right out of the gate!
forrtl: warning (406): fort: (1): In call to I/O Read routine, an array temporary was created for argument #5

Image              PC                Routine            Line        Source
binarytile.x<PHONE_NUMBER>40B710  Unknown               Unknown  Unknown
binarytile.x<PHONE_NUMBER>4051E5  MAIN__                     55  binarytile.F90
binarytile.x<PHONE_NUMBER>402A6E  Unknown               Unknown  Unknown
libc-2.22.so       00002AAAAB40D725  __libc_start_main     Unknown  Unknown
binarytile.x<PHONE_NUMBER>402969  Unknown               Unknown  Unknown

also:
forrtl: warning (406): fort: (1): In call to I/O Read routine, an array temporary was created for argument #3

On the plus side, at least I got the Intel traceback.

@mathomp4 was there any actual error messages?   The warnings are harmless.

That's too bad since it does indeed fix the issue on our end. I'm now running to see what kind of errors get detected with the catchall. The drawback is it is now super slow.

@lizziel Don't give up just yet.   We might be able to get this to work with a reasonable investment of effort.

@mathomp4 was there any actual error messages? The warnings are harmless.

Nope. In fact, it might be "fun" to debug. I just tried running binarytile.x on the command line and it got stuck in essentially a:
10 print warning
20 print traceback
30 goto 10

loop and pegged a processor. I killed it after about 5-10 seconds and in that time it went through about 5000 iterations of the traceback print. Hmm.
So, I took a "pre-made" tile.bin from another run and moved it over to avoid this and then I get an actual error:
forrtl: error (63): output conversion error, unit -5, file Internal Formatted Write
Image              PC                Routine            Line        Source
GEOSgcm.x<PHONE_NUMBER>7482AE  Unknown               Unknown  Unknown
GEOSgcm.x<PHONE_NUMBER>7A0AE2  Unknown               Unknown  Unknown
GEOSgcm.x          000000000B3C0FB5  fvdycorecubed_gri        1596  DynCore_GridCompMod.F90
libesmf_fullylink  00002AAACA1C57CB  _ZN5ESMCI6FTable1     Unknown  Unknown
libesmf_fullylink  00002AAACA1C92BF  ESMCI_FTableCallE     Unknown  Unknown
libesmf_fullylink  00002AAACA6A6195  _ZN5ESMCI2VM5ente     Unknown  Unknown
libesmf_fullylink  00002AAACA1C6E7A  c_esmc_ftablecall     Unknown  Unknown
libesmf_fullylink  00002AAACA1C6A2E  _ZN5ESMCI6FTable1     Unknown  Unknown
libesmf_fullylink  00002AAACA1C6765  c_esmc_setservice     Unknown  Unknown
libesmf_fullylink  00002AAACAAC5C44  esmf_gridcompmod_     Unknown  Unknown
GEOSgcm.x<PHONE_NUMBER>0753E9  mapl_genericmod_m        4560  MAPL_Generic.F90
GEOSgcm.x<PHONE_NUMBER>07647F  mapl_genericmod_m        4595  MAPL_Generic.F90
GEOSgcm.x<PHONE_NUMBER>5D28FE  geos_superdyngrid         149  GEOS_SuperdynGridComp.F90

but that doesn't seem to stop the model. It goes on and starts throwing tracebacks with every warning! Some of which are:
$ rg forrtl 1day.avoid_binarytile.1x6.out  | sort | uniq
forrtl: error (63): output conversion error, unit -5, file Internal Formatted Write
forrtl: warning (406): fort: (1): In call to CELL_CENTER2, an array temporary was created for argument #1
forrtl: warning (406): fort: (1): In call to CELL_CENTER2, an array temporary was created for argument #2
forrtl: warning (406): fort: (1): In call to CELL_CENTER2, an array temporary was created for argument #3
forrtl: warning (406): fort: (1): In call to CELL_CENTER2, an array temporary was created for argument #4
forrtl: warning (406): fort: (1): In call to CELL_CENTER2, an array temporary was created for argument #5
forrtl: warning (406): fort: (1): In call to GET_SYMMETRY, an array temporary was created for argument #1
forrtl: warning (406): fort: (1): In call to GET_SYMMETRY, an array temporary was created for argument #2
forrtl: warning (406): fort: (1): In call to MPI_GROUP_INCL, an array temporary was created for argument #3

I wonder if this is a bug or something in Intel 18? I'll try with Intel 19.1.2 and see what happens.

Yes, the traceback per warning is rather excessive. My run failed fairly early since I did not have the AdvCore_GridCompMod.F90 fix (https://github.com/GEOS-ESM/FVdycoreCubed_GridComp/issues/71) and my log was already at 1G.
@LiamBindle commented on a way to fix this in GCHPctm issue https://github.com/geoschem/GCHPctm/issues/37.

I believe those temporary array warnings can be suppressed with -check,noarg_temp_created.

Yes - that would suppress the warning, but ... might bring us right back to the original problem.
sigh

Well, for now I will try -check all,noarg_temp_created.
Note I did try setting: setenv FOR_DISABLE_STACK_TRACE 1 which did indeed suppress the stack traces...but it still spitout an amazing number of forrtl messages: 1539012 before I hit Ctrl-C.
And this was using 6 processes.
And it only got through about 1.5 time steps of our model.
I think we make a lot of temp arrays...

Doing the comma-separated list works for me for both ifort18 and ifort19. I still get a few warnings about temporary arrays, but only about 30. It then dies with this, presumably an entirely separate issue I'm looking into so not really relevant here:

stdout:
DYNAMICSRun                                    891

stderr (only one thread shown here:
pe=00002 FAIL at line=00067    GetPointer.H                             <status=41>
pe=00002 FAIL at line=01829    MAPL_Generic.F90                         <needs informative message>
pe=00002 FAIL at line=00454    GCHP_GridCompMod.F90                     <status=1>
pe=00002 FAIL at line=01829    MAPL_Generic.F90                         <needs informative message>
pe=00002 FAIL at line=01087    MAPL_CapGridComp.F90                     <status=1>
pe=00002 FAIL at line=01013    MAPL_CapGridComp.F90                     <status=1>
pe=00002 FAIL at line=00733    MAPL_CapGridComp.F90                     <status=1>
pe=00002 FAIL at line=00863    MAPL_CapGridComp.F90                     <status=1>
pe=00002 FAIL at line=00324    MAPL_Cap.F90                             <status=1>
pe=00002 FAIL at line=00198    MAPL_Cap.F90                             <status=1>
pe=00002 FAIL at line=00157    MAPL_Cap.F90                             <status=1>
pe=00002 FAIL at line=00131    MAPL_Cap.F90                             <status=1>
pe=00002 FAIL at line=00029    GCHPctm.F90                              <status=1>

This should be closed by the release of v3.1.4 (:pie:!) which uses the new -check all,noarg_temp_created


[Question]:  How can I run this offline? Can't get past Sign in

Describe your problem
I want to run this on an air gapped system. It's a requirement for my project.  Can I still use ragflow?

It seems like the log in just creates a local account.


Anchor tag is not coming when using adaptivecard lib in angular 4 application

Details
Explain your issue. If you just have a question, please post on StackOverflow instead.
I am using adaptivecard lib with directline to display the cards. However I am not getting the anchor tag. It is working on other channels. e.g. "text": "ER-13052"

@sanjay2880 a couple of requests:

Could you update the bug title to be something other than the placeholder text from the bug-filing template :)
Also do you have the sample JSON here that you are working with?

@ckkashyap heads up on this one in case you have some context at all for us..

@shalinijoshi19 @ckkashyap
Got the fix for the issue. I am using adaptive cards in our angular4 application. We need to also add "markdown-it" as dependency which was not there.
"npm i markdown-it --save"
Upon adding the dependency it was working as expected. Please add in the readme or documentation which would help others

Thanks for letting us know! markdown-it should be an optional dependency, could you provide any more details on what exactly went wrong before it was added?

`

Adaptive Cards Example


test corner cases/overall functionality of "quick join"

quick join is a complex function, with several special cases, like when no game is available. This should be well tested

I have tested the "Quick Join" functionality extensively (corner cases etc) and have not found any abnormalities. Therefore, I will close this ticket.


Clearing Netlify CMS cache in branch deploys

I'm having in issue with my netlify CMS cache not clearing when making changes in a branch set up with a build preview.
I can manually clear 'cache and redeploy' from within the netlify admin panel but I'd like the option to automate clearing the cache or to not cache content at all between deploys.
I've seen some solutions outlined here, specifically appending && rm -rf /path/to/cache to the build command but I'm unable to find the exact path to the cache - some insight on this would be helpful.
Thanks.

Hi @svvimming, this repo is intended for Decap CMS related issues.
For Netlify build related issues I suggest to reach out in the forums: https://answers.netlify.com/
If you want to clear cache of the admin panel, just hard refresh your browser.
Closing the issue, and please comment if you have any other questions.


Type error after starting pocketbase

Hey, after running the serve command the types.d.ts file is created. This file has a type error in line 17706. Can someone please have a look and if necessary fix it?

I'll consider updating the generated types with the next release, but this file is not required for PocketBase to work and you can ignore it.

Okay thanks :) I am using a TypedPocketBase and when building the compiler complained, that the generated types have an error.

I don't know what TypedPocketBase is but the above file is intended to be used only by your editors LSP when working with JS pb_hooks and when referenced manually in your hook file as shown in https://pocketbase.io/docs/js-overview/#typescript-declarations-and-code-completion.

I meant this from the js-skd
TypedPocketBase

Again, the above file has no relation to the JS SDK or your own custom types. It is used only as ambient TS declarations for the JS pb_hooks.
If for some reason your TS config is complaining about it, you can try to exclude it (probably you have a wildcard pattern somewhere defined that will traverse all directoris with to look for .ts/.d.ts files).

Yes excluding the files worked, thanks

The generated TS file was updated in master and will be available with the next minor release sometime later next week.
But again, the above file can be safely ignored since it is just a helper to provide ambient TS declarations (eg. autocomplete, docs on hover, etc.) when PocketBase is used as framework with the JS pb_hooks.


RBS signatures

I've noticed that jwt doesn't ship with rbs signatures yet. Is this something the project would like to have going forward?

Could you describe the issue a little bit more. What are these "RBS" signatures?

https://www.ruby-lang.org/en/news/2020/12/25/ruby-3-0-0-released/
Ctrl+F "RBS"

This looks interesting. Maybe in the future.
This isn't going to be a priority at the moment.
I am putting this on my watch list of the new things to learn.

Nice. The intent of the question was to know whether you'd be interested in me contributing the sigs and perhaps dialing it in CI.

Ah, lol.  I thought it was a cryptographic signature algorithm I've never heard of. Now it makes sense :)

I think that it could bring value to describe the public methods of the gem like this. The challenge maybe is to figure out what the signature actually is :). There is a lot of history.
Would appreciate a little something related to this. Maybe starting with the JWT module and decode and encode methods.
Have not bumped into this for any gems I'm familiar with. Are there some examples out there how existing gems have adapted/implemented something like this?

I added RBS for jwt to gem_rbs_collection.
https://github.com/ruby/gem_rbs_collection/pull/243
Currently, this doesn't cover all signature and some parts are untyped.
(Of course, it would be preferable if the gem itself includes RBS)

Think i'm going to close this one. Lets keep the RBS signatures separated.

@anakinj I see a v3 is in the works. Are you also planning to update the rbs sign in the gem rbs collection repo?

Unfortunately I don't have the capacity to keep them updated. If someone is willing to keep the collection signatures in shape that would be great.


Issue with Pipes

Am currently using Pipes to format my currencies within the render function , though it works perfectly on screen but when i export as PDF or Excel , the value before i applied pipes are getting displayed

I take it you found the export function in the configuration @Liahus1996 which is the way to handle how to export the field?


Support for Laravel 6.x

Please merge @bubba-h57 , we'd also like to use Laravel ^6.2 :-)

Sorry for the day, I was out dealing with some personal issues the past couple of days.
Laravel 6 moved to SemVer for its framework. Let's change, and simplify the version ranges?
I think we could break out a new major release and move to:
"illuminate/support": "^5.4|^6.0"
"illuminate/console": "^5.4|^6.0"

It has been stated that no more releases for Laravel 5 are expected, and this way, we will not have to deal with updates for every little release.
@vintagesucks if you wouldn't mind modifying your PR, I would love to merge that today and get a new 2.0 release out. Thanks for your assistance on this!

@bubba-h57 I modified my PR (please not that I force-pushed to keep it clean.)


Implement Sigprocmask 2 and 3

This fixes #263 by implementing SIG_UNBLOCK (2) and SIG_SETMASK (3)
We now get an error on Syscall 608, which I'll create an issue for.

Have you checked on the PS4 kernel to see if it does the same thing? Also please don't include the commits from the other PR.

Welp, new PR time.


It shows errors in the pipeline, but they are not reflected in the platform

A general error is being presented in the evaluation since this morning, running an oas for the platform https://apisecurity.io/tools/audit/ gives 100 but for the pipeline it gives 79 even so it does not show issues

Closed for inactivity


[Discussion] Datatype of field 'count' in matrix

A general question: How strict is the usage of the datatypes you have defined? In you documentation on readthedocs it is written that the datatype of matrix['count'] is a int32 (http://cooler.readthedocs.io/en/latest/schema.html#contact-matrix). You can manipulate this by setting cooler._writer.COUNT_DTYPE  to another datatype, but this is non-standard behaviour. The usage of int32 as a datatype is a bit problematic because after a correction of a transformation of the data to obs/exp or pearson matrix we need float values and not integers. For this case you have the field 'weight' in bins and the idea is to apply this weights to the int counts.
I see here two issues: 1. It is a slow down factor to apply every time the correction factors instead of storing the corrected values directly as a float. Moreover, if you have e.g. a Pearson matrix you need to calculate first the weight factors which is not that easy and needs additional computing time. Especially for high resolution matrices.
2.The files you provide on the ftp server show that you do not stick strictly to your own definition. I checked a few files and they have floating values for matrix['count'] set and even worse the correction factors stored in 'bin['weight']' were present, but were all NaN.
To make a long story short: Wouldn't it be better to change the default data type of matrix['count'] to float32?

How strict is the usage of the datatypes you have defined?

Not particularly strict at all! Especially for the count column.

You can manipulate this by setting cooler._writer.COUNT_DTYPE

This used to be the case. There is actually a dtypes option to explicitly specify data types in cooler.io.create. It isn't documented very clearly, and I've been working on replacing it with columns and dtype arguments that mimics the behavior of pandas.read_csv. This will allow overriding standard column dtypes and declaring non-standard columns.

The usage of int32 as a datatype is a bit problematic because after a correction of a transformation of the data to obs/exp or pearson matrix we need float values and not integers.

Yes, indeed! The rationale for making count integer by default was for the goal of losslessly storing the raw counts and applying transformations on top of them (e.g. storing multiple balancing vectors). Of course that's not always the case, and we definitely intend to support floating point as an alternative dtype for count, which can be done.
Note that Cooler was also designed to support multiple value columns, so one could even store count, balanced and obsexp values next to one another, or do something even more exotic like grouping counts by strand orientation!

For this case you have the field 'weight' in bins and the idea is to apply this weights to the int counts.

We plan on also storing "expected" vectors in a standard way in the file. The delay has come mostly from debating how to deal with the fact that we sometimes partition expected by chromosome arm rather than by chromosome. That also complicates the transformation.

The files you provide on the ftp server show that you do not stick strictly to your own definition.

Yikes! I'll take a look at that. If you remember which ones were problematic, could you email me?

Wouldn't it be better to change the default data type of matrix['count'] to float32

I wouldn't change the default at this point, especially since we almost never stray from it (It is also suggested by the name "count"). However, I do think it should be trivially easy to set a different dtype.
One thing to possibly consider is making even the "count" column strictly optional. It is already possible to query alternative value columns using Cooler.pixels(field='myfield').


add featuregate for MachineConfigNodes

This adds a featuegate for the work on openshift/api#1596 which I will then bring into the MCO. Keeping this separate as it can merge before the types get into the API.

/lgtm


compound keys

I hear that compound keys are desired (on import)
dat import -d dataset /path/to/file.csv --key=city,state,zip

Concatenated using a separator, and sorted internally in case they're given in scrambled order
cc @talos

:+1:
What do you mean by "scrambled order"?

@talos city,state,zip should be the same as zip,city,state

Got it.  For the sake of uniqueness it doesn't matter, but I guess I don't see the advantage of overriding what could be the user's choice in order.  Certainly for primary keys in MySQL or Postgres, the order of a compound key could be chosen so that it could aid in joins.  While this might not matter now for dat, in the future perhaps it could, and this behavior could be problematic.
Your choice -- just glad to see this on the roadmap!

Ah good point. Perhaps we should make it auto order now for simplicity's
sake and add an option later to force the given order
On Tuesday, August 4, 2015, john krauss<EMAIL_ADDRESS>wrote:

Got it. For the sake of uniqueness it doesn't matter, but I guess I don't
see the advantage of overriding what could be the user's choice in order.
Certainly for primary keys in MySQL or Postgres, the order of a compound
key could be chosen so that it could aid in joins. While this might not
matter now for dat, in the future perhaps it could, and this behavior could
be problematic.
Your choice -- just glad to see this on the roadmap!
—
Reply to this email directly or view it on GitHub
https://github.com/maxogden/dat/issues/372#issuecomment-127812790.

--
Karissa McKelvey
http://karissa.github.io/ http://karissamck.com
Send me an encrypted message:
https://keybase.io/karissa https://keybase.io/karissa/key.asc

Awesome, thanks.  I'll take this feature for a spin with some civic datasets next week.


Remove INFN-PADOVA-STACK

Summary
Site INFN-PADOVA-STACK no longer available.

Related issue :

Sure, thanks!
xref: https://github.com/tdviet/fedcloudclient/pull/195

https://github.com/tdviet/fedcloudclient/pull/195 has been merged and tests are passing locally.
Is there a way to retrigger the tests?


[ALLUXIO-3306] delay fuse open if file is not completed

https://alluxio.atlassian.net/browse/ALLUXIO-3306

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/285/
Test FAILed.

@calvinjia Changed the constant name, PTAL, thanks!

@calvinjia Added the unit test for delayed open, change file completed status when opening a file. PTAL, thanks

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/293/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/297/
Test PASSed.


Create a map to show possible path of shipping trucks

Describe the solution you'd like
Build something like google maps where we can add object, pins, paths, ....
Describe alternatives you've considered
Use the component out of bootstrap
Additional context
Need to be branded for swisspost.

If it's only about the map styles used in this project https://places.post.ch/, we could simply add a docs page in storybook and provide the json config there as a download link, so other projects can use google maps with the same styles in there own project.

Resolution: Provide map styles in JSON format and create a page under patterns with a small demo.

Check in pattern discussion if https://places.post.ch/standortsuche is the way to go design wise.

Resolution: Re-style from scratch and use accessible colors (https://mapstyle.withgoogle.com/).


want to blink led on arduino

hello sir, i want to glow led on arduino board with pyserial when threshold value reaches. can u please help me with it?

Hi, haven't worked with it, afraid cannot help you with this


ng-repeat directive doesn't work

Thanks for a great module!
Unfortunately, I can't get a simple ng-repeat directive to work inside the <inspector-content> tag.
For example, this
<inspector-gadget>
    <inspector-content>
        [[ i ]]
    </inspector-content>
</inspector-gadget>

results in no content output.
Any thoughts?  Thanks!

Belated thanks @softwaredoug!  I definitely would not have figured that one out.


feat: add an attribute for Popover, control whether prevent overflow

IMPORTANT
Currently, this project is closed to any external contributions. Any pull request made against this project from external sources will likely be closed. If you would like to make changes to this project, please fork this project.
Guide
This "Help" section can be deleted before submitting this pull request.
Update the name of this pull request to reflect the following shape:
{type}/{scope?}/{message}

type - A conventional commit type REQUIRED
scope - The kabab-case scope of the changes in this request
message - A short, kebab-case statement describing the changes REQUIRED

Provide a general summary of the scope of the changes in this pull request.

Readme
Getting Started Guide
Contributing Guide

Description
The full description of the changes made in this request.
When a dropdown list Item need a popover component, it will limit by parent element's width. So I add an attribute for Popover  component - isPreventOverflow to overflow parent element.
Links
https://jira-eng-gpk2.cisco.com/jira/browse/SPARK-487032
Links to relevent resources.

Snapshot tests are failing
Have you linked this locally with Cantina and tested the changes that need to be made? Is this working the way you need it to?
Can you add a storybook to show how this prop changes the behaviour of the component?

Solve my problem without momentum component changed. I don't need to add any change for this project.