Not able to find Meter - with mbus-serial-scan/mbus-serial-scan-secondary

Hello,
i'm having some trouble with getting libmbus to work.
Host for Libmbus : Ubuntu 16.04
Level Converter : HD67020-B2-10 from ADFWEB
Meter: GWF Water Meter
Issue looks like : https://github.com/rscada/libmbus/issues/24
The Solution marked there is changing the behavior due to the echo from the level converter:
I've tried all three options in the "mbus-protocol-auc.c" file at line 1523:
handle->purge_first_frame = MBUS_FRAME_PURGE_S2M;
The behavior doesn't change at all.
With the free software from Relay (MBSHEET) the meter can be found and read without any problems.
Maybe it's just a handling problem on my side.... any help is appreciated !
See also attached console output:
mbus-serial-scan -d -b 2400 /dev/ttyUSB0 Scanning primary addresses: 0 [2016-08-14 15:42:57] SEND (005): 10 40 00 40 16 [2016-08-14 15:42:58] RECV (005): 10 40 00 40 16 1 [2016-08-14 15:42:58] SEND (005): 10 40 01 41 16 [2016-08-14 15:42:58] RECV (005): 10 40 01 41 16 2 [2016-08-14 15:42:58] SEND (005): 10 40 02 42 16 [2016-08-14 15:42:58] RECV (005): 10 40 02 42 16 3 [2016-08-14 15:42:58] SEND (005): 10 40 03 43 16 [2016-08-14 15:42:58] RECV (005): 10 40 03 43 16 4 [2016-08-14 15:42:58] SEND (005): 10 40 04 44 16 [2016-08-14 15:42:58] RECV (005): 10 40 04 44 16 5 [2016-08-14 15:42:58] SEND (005): 10 40 05 45 16 [2016-08-14 15:42:58] RECV (005): 10 40 05 45 16
Thank you for your help !

Does your Ubuntu run in a virtual machine or is it native one?

The Ubuntu is running inside a VM.
The MBSHEET on the Windows Machine is also running inside a VM on the same ESX Host.
With Option Search Address:
`Search baud rate 2400!
Slave primary search started!
TX-buffer:
10 40 FD 3D 16
RX-buffer:
TX-buffer:
10 40 FF 3F 16
RX-buffer:
TX-buffer:
10 40 01 41 16
RX-buffer:
E5
TX-buffer:
10 7B 01 7C 16
RX-buffer:
68 1B 1B 68 08 01 72 58 63 77 15 E6 1E 3C 07 10 00 00 00 0C 78 58 63 77 15 0C 13 04 11 00 00 18 16
*** Slave found at primary address 1`
With Option Search ID:
`Search baud rate 2400!
Slave ID search started!
TX-buffer:
10 40 FD 3D 16
RX-buffer:
TX-buffer:
10 40 FF 3F 16
RX-buffer:
0FFFFFFF
1FFFFFFF
*** Slave found at ID (secondary adress) 15776358
2FFFFFFF
Slave ID search finished!
`

Did i do the change to Frame purge at the correct location ?
Can i somehow verify that ?
Just want to make sure that i didn't make a fault.

@derbodo I don't think it's your fault. But there could be many reason for this behavior.
For instance VM plus USB to RS232 adapter changes runtime behavior. So it's possible that the responses are accidentially dropped.
Are you able to verify that the water meter response to libmbus over serial line?

Ok just checked it with an old Notebook that still has a serial port.
It works fine.
Switched from Serial Port to USB2Serial adapter.
Just works fine.
After that i just reinstalled the RPI2 an connected the USB2Serial adapter.
Works like a charm.
So the Problem is the ESXi host.
Not a problem related to libmbus - but has someone already successfully testet this within a linux VM ?
If possible i do not want to operate another PI beside the ESXi Server...
The RS485 Modbus Stick works just fine within the ESXi Ubuntu VM.....

Solution on my side is actually:

Use the USB Serial Adapter on a Raspberry
or
When using it within an ESX VM - do NOT use the Serial Adapter with a USB 3 Port - I'Ve now connected it to a USB 2 Hub and it works....

I'm closing this issue....


Contributed pixelshuffle op

Thanks to @justinchuby for a significant amount of help on this :)

Looking forward for this merge, thanks for the work.


fix typo, publically -> publicly

found via codespell -S deps

:partying_face: :partying_face: :partying_face: :partying_face: :partying_face:


Wrong AES-CCM tag calculation

Description

Type: Bug
Priority: Minor

The tag calculated with mbedtls_ccm_encrypt_and_tag() does not match the expected tag for the NIST #25 test vector. Hence the call to mbedtls_ccm_auth_decrypt() fails when using the tag given in the test vector.

Bug
OS
Linux
mbed TLS build:
Version: 2.16.5-1
OS version: Arch Linux (up-to-date)
Expected behavior
Name       : AES-128 CCM NIST #25
Key        : F9FDCA4AC64FE7F014DE0F43039C7571
Iv         : 5A8AA485C316E9
Cipher     : 6BE31860CA271EF448DE8F8D8B39346DAF4B81D7E92D65B3
Plaintext  : A265480CA88D5F536DB0DC6ABC40FAF0D05BE7A966977768
Tag        : 38F125FA

Crypto++   : Encrypted and verified data
Cipher     : 6BE31860CA271EF448DE8F8D8B39346DAF4B81D7E92D65B3
Tag        : 38F125FA

Crypto++   : Decrypted and verified data
Plaintext  : A265480CA88D5F536DB0DC6ABC40FAF0D05BE7A966977768

Actual behavior
 CCM-AES #1: passed
 CCM-AES #2: passed
 CCM-AES #3: passed

Name       : AES-128 CCM NIST #25
Key        : F9FDCA4AC64FE7F014DE0F43039C7571
Iv         : 5A8AA485C316E9
Cipher     : 6BE31860CA271EF448DE8F8D8B39346DAF4B81D7E92D65B3
Plaintext  : A265480CA88D5F536DB0DC6ABC40FAF0D05BE7A966977768
Tag        : 38F125FA

MbedTLS    : Encryption successful
Ciphertext : 6BE31860CA271EF448DE8F8D8B39346DAF4B81D7E92D65B3
Tag        : FC24CF6A

MbedTLS    : Authenticated decryption failed.
Plaintext  :<PHONE_NUMBER>00000000000000000000000000000000000000

Steps to reproduce
Test code mbedtls.c with MbedTLS and cryptopp.cpp with Crypto++ for comparison.
Compile:
gcc -g -o mbedtls mbedtls.c -lmbedcrypto
g++ -g -o cryptopp cryptopp.cpp -lcryptopp

This exact test vector is in the Mbed TLS test suite and it passes. So what's the bug supposed to be?

Hi, I found the problem with the tag calculation. Its actually just a mistake in my test code mbedtls.c.
In the call of mbedtls_ccm_encrypt_and_tag() I was using testVectorCCM.datasize. I should have used testVectorCCM.authsize instead.
I close the issue.

No worries, glad you found the problem!


Write manual testing scripts for Qualifying Tests

We need to perform manual testing of qualifying test functionality, both for the Candidate in Apply and for Admin users.

[ ] Write manual test scripts
[ ] Circulate with testers (including ourselves) and ensure we collect and process the results
[ ] Schedule manual testing ahead of any upcoming dry run and live tests

See apply#561 for test scripts covering the Candidate experience. Have therefore reduced the estimate of this story from 5 to 3


Tooltip should close when moving cursor to another tooltip, even when inside grace area

Bug report
Current Behavior

When two tooltip triggers are next to each other, and the content is larger than usual, when hovering over first tooltip and later hovering over second tooltip, the 1st tooltip is visible and 2nd tooltip doesn't appear.
This is because of Grace Area code, that detects horizontal movement and doesn't close the first tooltip. This is generally fine, but since the action is to hover onto second tooltip, I'd expect the grace area code to not have effect.
Expected behavior
When hovering over close tooltip trigger, it should always cause the 1st tooltip to disappear and cause 2nd tooltip to show.
Reproducible example
https://codesandbox.io/p/sandbox/serene-kepler-5f3cfv?file=%2FApp.jsx%3A34%2C22
Suggested solution
I haven't looked closely, but I'd guess to check if target is another instance of tooltip trigger, then remove grace area.
Additional context

Your environment

Software
Name(s)
Version

Radix Package(s)
tooltip
latest

React
n/a
latest

Browser
Chrome, Safari
latest

Assistive tech

Node
n/a

npm/yarn

Operating System
MacOS

+1, would love to get an update on this.

I am not familiar with the codebase but I found that disableHoverableContent to true (default is false) on the TooltipProvider solved this.
I took your previous codesandbox as an example: https://codesandbox.io/p/devbox/pedantic-frog-swk7zy?file=%2FApp.jsx.
This is what the documentation mentions as a FYI:

Prevents Tooltip.Content from remaining open when hovering. Disabling this has accessibility consequences.

but I'd guess to check if target is another instance of tooltip trigger, then remove grace area.

This would negate completely the point of the grace area algorithm though.
I believe this is a really niche edge case as which is not easily solvable and would be an issue with any library implementing this strategy.
The solution provided @bqm works because there is no need for a grace area if hovering the content from the trigger isn't needed. However please note that this is an accessibility requirement you would be disabling.

How can we ignore or alter the size of this grace area? I have run into this same issue where I am attempting to render multiple tooltip triggers next to eachother to provide inline citations.
I also discovered that disableHoverableContent fixes the issue, but then the content inside the tooltip can't be interacted with. It appears to me that there is some 10px width to the left and right of the tooltip where the user's mouse being in this padded area still keeps the tooltip open.

@benoitgrelard do we have any plans to solve this issue. As it occurs when we have tooltip triggers near to each other.


Update Dockerfile.gpu to use Tensorflow 0.10.0

Tensorflow 0.10.0rc0 was compiled against CuDNN v4. Previous Dockerfile wasn't working.

Can one of the admins verify this patch?

CLA signed 👍

This is done as a part of #4344 sorry for the delay on that.
merging that PR now.


Possible false positives on C-0059 - CVE-2021-25742-nginx-ingress-snippet-annotation-vulnerability

Running kubescape scan control c-0059 using the :latest docker image within an EKS 1.21 cluster.
This reports that all 4 of my ingress controllers have failed, yet all 4 have the following in their configmap:
apiVersion: v1
kind: ConfigMap
metadata:
  labels:
    app.kubernetes.io/component: controller
    app.kubernetes.io/name: ingress-nginx
    app.kubernetes.io/version: 1.0.4
  name: nginx-controller
data:
...
  allow-snippet-annotations: “false”
...

I actually fixed this by changing my configmap data to:
 allow-snippet-annotations: false
Which is contrary to the ingress-nginx instructions, but actually mitigates the issue. Quoted false fails to get recognised by the controller.
tl;dr This wasn't a kubescape issue, it was an issue with how I was applying the CVE mitigation.


Fix pthread cmdline issue

Summary

fs/procfs: Avoid the duplicated 0x prefix in pthread cmdline
pthread: Change the default name from  to <0xyyyyyyyy>
Before change:

   63    20 100 RR       pthread --- Waiting  Signal    00000000<PHONE_NUMBER>44  12.1%    0.0% <pthread> 0x0x341de68c

After change:
   63    20 100 RR       pthread --- Waiting  Signal    00000000<PHONE_NUMBER>44  12.1%    0.0% <0x34187c00> 0x341de68c

Note, you can still change pthread name through pthread_setname_np, this patch just modify the default behaviour.
Impact
The content from: /proc/xxx/cmdline
Testing

Will you consider a "pt-" prefix?

Is it enough to identify pthread by <>? task/kthread doesn't quote his name by <> normally.

My feeling is that before it said what it was now it does not.  The prefix of pt at least is a clue. Not really more obvious than <> but pthread-0xyyyyy would tell the whole story.

Ok, it show pt-0xyyyyyyyy now.


marimo: init at 0.3.1

Description of changes
Closing out my packaging request in #279873, by adding marimo from pypi packaging. Marimo is "A reactive notebook for Python — run reproducible experiments, execute as a script, deploy as an app, and version with git."
Github here: https://github.com/marimo-team/marimo
Things done

Added myself as a maintainer (but happy to cede authority to anyone)
Added akshayka as maintainer- for reviewer reference they are the marimo lead
Added marimo package
Made sure it works!

I am also cross-posting this PR into their discord channel for upstream feedback.

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

@dmadisetti, thanks for creating this PR. I'm happy to be added as another maintainer. I am not familiar with Nix (though I have heard great things!) so it would be best if you stayed on as a co-maintainer if possible.

Oh, I see now that the source given by fetchPypi includes all the frontend files, so no need to manually generate it with nodejs.

By the way, is there any reason someone might want to use the package as a python package (so actually importing stuff from it) or is it just used as a framework for running the code?
If it's just for running, I'd say that you should just use buildPythonApplication directly, and move the package to pkgs/by-name

Added akshayka as maintainer- for reviewer reference they are the marimo lead.
Yes! You can use marimo as a standalone package for building execution graphs.
Will fix maintainer typos and rebase- one second

Bit of clarity, it's not just the app- you can use marimo for managine scripts or just import its UI tools in jupyter

Oh woops, suggestions 4 days ago? Sorry I don't know why I didn't see them.
Good and reasonable suggestions, all implemented

Oh woops, suggestions 4 days ago? Sorry I don't know why I didn't see them.

I think I just forgot to press the "Finish review" button until today.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/1529

Thanks! Applied


Backtrack Not Working

Hey everyone, backtracking is not working at all for me. Here's a video of it Anybody have any suggestions to help? It's been like this for a while, no matter how many times i recompile the cheat.

Try recent pull request.

Try recent pull request.

As shown in the video, I have actually already tried that. I appreciate the input though.

try the lastest change, idk why it doesnt work for you

Osiris' backtrack can be inconsistent sometimes, from my experience at least.
Also, I am pretty sure Osiris doesn't check for valid ticks, which doesn't help. Correct me if I am wrong.

Osiris' backtrack can be inconsistent sometimes, from my experience at least.
Also, I am pretty sure Osiris doesn't check for valid ticks, which doesn't help. Correct me if I am wrong.

it does check if the ticks are valid also removing getlerp from cmd tickcount makes it more consistent

Bruh, dont aim at last model of the player, try to hit the middle.

Bruh, dont aim at last model of the player, try to hit the middle.
I've tried hitting every single part of the backtrack model I can.


unknown config id: vliteu

What version of V2Ray are you using?

5.0.6
What problems have you encountered?
v2ray exits with error when trying to use the vlite protocol.

Please attach your configuration here

Server configuration:
{
  "log": {
    "loglevel": "debug"
  },
  "inbounds": [
    {
      "port": 19232,
      "protocol": "vliteu",
      "settings": {
        "password": "pw"
      }
    }
  ],
  "outbounds": [
    {
      "protocol": "freedom"
    }
  ]
}

Please attach error logs here

Server error log:
$ v2ray run -c test.json
V2Ray 5.0.6 (V2Fly, a community-driven edition of V2Ray.) Custom (go1.18.1 linux/amd64)
A unified platform for anti-censorship.
Failed to start: main/commands: failed to load config: [test.json] > infra/conf/v4: failed to load inbound detour config. > infra/conf/cfgcommon/loader: unknown config id: vliteu

If V2Ray cannot start up, please attach output from --test command
$ v2ray test -c test.json
V2Ray 5.0.6 (V2Fly, a community-driven edition of V2Ray.) Custom (go1.18.1 linux/amd64)
A unified platform for anti-censorship.
Test failed: main/commands: failed to load config: [test.json] > infra/conf/v4: failed to load inbound detour config. > infra/conf/cfgcommon/loader: unknown config id: vliteu

@xiaokangwang the v2ray binary is taken from the releases section for linux amd64.

Set format argument to jsonv5.
v2ray run --config /path/to/config --format jsonv5

@AkinoKaede that fixed it, thanks. Shouldn't systemd service for v5 releases execute v2ray with this option by default?

Closing the issue as it's resolved.

how o fix

Closing the issue as it's resolved.

how to fix

[Unit]
Description=V2Ray Service
Documentation=https://www.v2fly.org/
After=network.target nss-lookup.target
[Service]
User=nobody
CapabilityBoundingSet=CAP_NET_ADMIN CAP_NET_BIND_SERVICE
AmbientCapabilities=CAP_NET_ADMIN CAP_NET_BIND_SERVICE
NoNewPrivileges=true
ExecStart=/usr/local/bin/v2ray run -config /usr/local/etc/v2ray/config.json  --format jsonv5
Restart=on-failure
RestartPreventExitStatus=23
[Install]
WantedBy=multi-user.target
it's correct

Test failed: main/commands: failed to load config: [test.json] > infra/conf/v4: failed to load inbound detour config. > infra/conf/cfgcommon/loader: unknown config id: vliteu  not fixed


column: deprecate 'key_name' in favor of 'name'

Using 'key_name' felt like we were leaking an API abstraction into the resource needlessly.
This PR also removes a surprising behavior in the column resource where an existing column would silently be used on create instead of erroring.

Codecov Report

Merging #242 (ff30e1a) into main (5d0ef8c) will increase coverage by 0.06%.
The diff coverage is 76.31%.

@@            Coverage Diff             @@
##             main     #242      +/-   ##
==========================================
+ Coverage   80.52%   80.58%   +0.06%
==========================================
  Files          42       42
  Lines        3477     3493      +16
==========================================
+ Hits         2800     2815      +15
  Misses        562      562
- Partials      115      116       +1

Impacted Files
Coverage Δ

honeycombio/resource_column.go
80.00% <76.31%> (+1.70%)
:arrow_up:

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


Empty nuxt project has high LCP and FCP value in lighthouse

Discussed in https://github.com/nuxt/nuxt/discussions/28460

Originally posted by siporoz August  8, 2024
Hi,
I am faced with the problem that my project has high LCP and FCP values in lighthouse, even if there is a minimum number of elements on the page.
As an experiment, I decided to create a new empty project through the command:
npx nuxi@latest init my-awsome-project
Next, I built the project and launched it.
And here are the values I got:

Even if change <NuxtWelcome /> e.g. <h1>, the values remain high for an empty page.

Environment
Operating System: mac os 14.4.1
Node Version: v22.1.0
Nuxt Version: 3.12.4

The issues you're seeing are:

You need a <title>.  This isn't something Nuxt can provide for you.
You need a meta description. This isn't something Nuxt can provide for you.
You need a lang attribute for your <html> tag. We considered defaulting to English but felt this wouldn't be appropriate - it's a choice for you.
You need a robots.txt - I've added this for new starters https://github.com/nuxt/starter/commit/6a0abda7507f523ca009aeed549c2324a7a2e29a

Note that running Lighthouse tests against a local build isn't recommended as it's highly dependent on your local server and doesn't include optimisations you'll get when deploying your site (like compressing assets, etc.).


Add CAMO_HASH option to allow customizing the hash used.

Can you write a test for another hash as an example?

I will try.
I don't actually code ruby, but I could just copy-pasta the code.
Unless there is a way to make a paramatized test?

Actually not sure how to do this in a good way at all, given that the ruby tests require the cofeescript server to be already running.

If you want to test behaviour yourself, set hash to md5 and do so accordingly in the tests as well.

Is there really a benefit to having a feature to support any hashing algorithm? i.e. What's the use-case where sha1 isn't good enough?

@rmm5t another website I was using (XenForo) uses md5.
Although if you wanted you could just change the other software.

@md-5 What does forum software have anything to do with the interface to an SSL image proxy?
:-1:

@rmm5t different pre-existing softwares may use different hashes. In this case it would be beneficial to allow the selection of a different hash to enable compatibility with such software.

different pre-existing softwares may use different hashes. In this case it would be beneficial to allow the selection of a different hash to enable compatibility with such software.

@md-5 That argument doesn't really hold water with me. I expected your desired use-case to be that you wanted to use a more secure hashing algorithm than sha1 (like sha256 or sha512), not a less secure algorithm (md5). I'd argue that more secure hashing isn't necessary for a service like camo, but I was curious about possibly a different use case.
You mentioned you were using XenForo (PHP forum software). Camo has it's own URL format and digesting path structure that other software isn't necessarily going to automatically handle for SSL image proxying. No matter what you choose, there's going to be some custom integration, and it's better to stick with Camo's default, IMO.
Since you're using PHP, md5 vs sha1 digests are trivial:
md5("foobar")
// vs
sha1("foobar")

http://php.net/manual/en/function.md5.php
http://php.net/manual/en/function.sha1.php
The desire for customizing the hashing algorithm still sounds unwarranted to me.

Looking at this I'm seeing two things that make me think this PR is unnecessary.

Introducing this new behavior makes it difficult to test w/o restarting the server process w/ different hash environmental variables.
SHA1 is pretty common and available on most modern languages.

I'm unaware of other apps having custom support for camo itself so if you're extending functionality to support camo you can probably use sha1.


default-networking: restore default settings when switching back from…

… lowpower

This profile is defined as: Restores default network settings. Inverse profile of the test profile.
It's not clear exactly what the plan is here. There are lots of default settings defined in https://github.com/ipfs/go-ipfs-config/blob/4fd3da76524ebfbd5726261792960c75900efe67/init.go that aren't modified in this PR.
We can potentially add new profiles, but we need to make it clear what we are targeting. For example, is it just reversing lowpower mode or resetting everything but datastores? Would the user expect it to reset their pinning services keys and CORS configurations as well, etc.
I'd prefer not to add a ton of different profiles if we can avoid it, but if we are going to add more we should be precise.

Closing for now.  Feel free to reopen if have additional information.


Make images not selectable and not draggable

Description
Make images on the website not selectable or draggable. All images on the website can be dragged and selected which is against convention.

Implementation

[x] I would be interested in implementing this feature.

@TrustedMercury I disagree with your conventions on this topic.
I believe allowing images to be dragged is acceptable. I frequently use drag-and-drop to save images from the browser to my device.

The same goes for being able to select images as you can then copy and paste into a rich text editor to get not only the text but also the images on the page. I'm not sure this is an issue that we need to solve but I'll still keep it open in case someone aligns with your observations.

I concur with @dhruvkb, we do not need to fix this. We hope to see other contributions from you, @TrustedMercury! :)


Extract histogram builder from CPU Hist.

Fixes incorrect tests where distributed methods are not actually tested.
Extract histogram builder to be a reusable module.

Perf

Dataset
Train time
Accuracy

Before
HIGGS
191.195842235989
0.7342790909090909

covtype
74.0480548529886
0.9398466476769103

After
HIGGS
184.16188054496888
0.7342790909090909

covtype
68.06914013501955
0.9398466476769103

@ShvetsKS

@trivialfis sorry for long response.
LGTM
I didn't observe any degradations also
Thanks for great refactoring


[SPARK-39963][SQL] Simplify SimplifyCasts.isWiderCast

What changes were proposed in this pull request?
SimplifyCasts is a optimize rule used to removes Cast that are unnecessary because the input is already the correct type.
Currently, the implementation of SimplifyCasts seems a little redundant. Cast.canUpCast can completely cover the function of isWiderCast.
On the other hand, SimplifyCastsSuite does't cover the case when from is non-numeric and to is numeric. This PR add a new test case for this case.
Why are the changes needed?
Simplify the implementation of SimplifyCasts.
Does this PR introduce any user-facing change?
'No'.
Just update the inner implementation.
How was this patch tested?
N/A

ping @gengliangwang @MaxGekk @dongjoon-hyun cc @cloud-fan @viirya

Thank you for pinging me, @beliefer . Could you rebase to the master branch once more? SparkR failure is fixed.

Thank you for pinging me, @beliefer . Could you rebase to the master branch once more? SparkR failure is fixed.

Thank you for the reminder.

thanks, merging to master!

thanks, merging to master!

@cloud-fan @dongjoon-hyun @viirya @gengliangwang Thank you !
@xkrogen Thank you for you comments.


Repo.preload looses prefix when nested preloads are requested.

Precheck

Do not use the issues tracker for help or support requests (try Elixir Forum, Stack Overflow, IRC or mailing lists, etc).
For proposing a new feature, please start a discussion on elixir-ecto.
For bugs, do a quick search and make sure the bug has not yet been reported.
Finally, be nice and have fun!

Environment

Elixir version (elixir -v): 1.10.4
Database and version (PostgreSQL 9.4, MongoDB 3.2, etc.): Postgres 12
Ecto version (mix deps): 3.5.1
Database adapter and version (mix deps): Ecto SQL 3.5
Operating system: Xubuntu 20.04

Current behavior
We are working on an app that uses multi-tenancy.
At some point, deep in our function call tree, we're trying to preload stuff. The weirdest thing happens:

This works: c |> Repo.preload(:agent)
This works:  c |> Repo.preload(:notifier)
This does not work:  c |> Repo.preload(:agent, :notifier) - Ecto tries to find the notifier association back in the public prefix...
But then this works: c |> Repo.preload([:agent:, :notifier], prefix: "whatever")
And also this works: c |> Repo.preload(:agent) |> Repo.preload(:notifier)

Expected behavior
Ecto should stay on the prefix for the structure passed on the first argument of preload.
Notes
Our Repo.default_options looks in process storage to find which prefix to work on. We are wondering whether Repo.preload is trying to either bypass default_options past one preload, or spawn other processes (which won't inherit storage from current process, and therefore won't have the prefix set when calling Repo.default_options). Which one is it? :)
Also something worth mentioning, when the bug occurs all bindings from my IEX shell are lost.

Can you please fill in the issues template?

Done @josevalim

Well that was fast. Thanks @josevalim !


hash share passwords

Passwords should never be stored in plaintext. The manager stored them base64 encoded which is very easily decoded to plaintext. Now the passwords are hashed using the bcrypt algorithm with a default cost of 11. The cost can be configured.

@C0rby good to merge?

Yes, from my PoV it's ready.


📊 Tourism: CO2 emissions due to aviation

leaving open for now as no new data seems to have been added yet

closing as the most recent data doesn't seem to have been updated by the OECD yet


COMPRESS-560: Do not return false if the entry is a tar sparse entry

Also some unit test cleanup to use assumeTrue/assumeFalse with the OS check. Now tests will be shown as skipped instead of success when they are not run.

Coverage increased (+0.004%) to 87.273% when pulling 30dfb6394bdbbc494b002dc2662d7ecc06e9d342 on theobisproject:COMPRESS-560 into 0bff34c1b409b10f4066ced3e146ed8ae0ad0fe9 on apache:master.


API is down

Not really a code issue, but not really sure where best to ask.  All API requests are resulting in 404 responses from the new CfA site.  Not sure if this is a temporary issue, or if support for this service is stopping permanently.

Hey @verythorough I just submitted a Pull Request to get your project page working again.
We're having some drama on our side of things with the CfAmerica website routing. All the Brigade stuff, the project search, the issue finder, is all not linked to anymore. The services are still running though, so the PR links to them directly.
Sorry if our drama took up any of your volunteer time.


Document class contextType as the primary consuming mechanism

Supersedes #1265

@sebmarkbage I think the document should also mention that when using the new contextType, they create conflict with libraries (like react-redux < 5.1) that hoist the non-react static properties, making the library unusable.


use test_data macro instead of include_str

 Test manipuliert working dir

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

7 of 7 (100.0%)  changed or added relevant lines in 2 files are covered.
31 unchanged lines in 3 files lost coverage.
Overall coverage decreased (-0.09%) to 85.532%

Files with Coverage Reduction
New Missed Lines
%

services/src/datasets/external/gfbio.rs
2
91.01%

services/src/datasets/in_memory.rs
8
77.62%

services/src/datasets/add_from_directory.rs
21
19.05%

Totals

Change from base Build<PHONE_NUMBER>:
-0.09%

Covered Lines:
26165

Relevant Lines:
30591

💛  - Coveralls


Update delete-calendar.md

Updated the document to reflect connection to exchange online v2

@JohanFreelancer9 Please copy review

Issue#https://github.com/MicrosoftDocs/microsoft-365-docs/issues/3249

@kwekuako Please merge this PR. Thanks


Agents Copy/paste is not working

Agents tree trying to copy and agent then paste
To Reproduce
Steps to reproduce the behavior:

Go to Configuration/Agents
Select agent
Right click copy
Paste

Expected behavior
Copy of the agent

working on it

fixed.


do not ignore errors during downloading of tenant index parts

Instead of swallowing errors it catches them but this state is not recoverable other than through detach/attach sequence
Misses tests, I wonder should we have a Mock impl of remote storage trait that serves like a proxy to another implementation but randomly (or semi randomly) throws errors. WDYT @SomeoneToIgnore?

Yeah, I agree with your point, I can change poisoned variant so it still contains the data without throwing it away. It can collect both, failed and successful downloads, so the later part would be to actually reschedule failed tasks


Issuing "Reformat Code" command with KDoc active throws "Attempt to modify PSI for non-committed Document!" exception

I use this plugin along with the KDoc formatter.  I filed an issue over there, and am now filing an issue here, at the request of that plugin's author.
Indeed, if I disable this ktlint plugin and reformat code, I get no exceptions.  It is only with both plugins active that I do.  It appears that this plugin might be leaving uncommitted changes to documents.  See this comment for specific exception details, including screen shot & exception stack trace.

Thanks for the heads up @matthewadams @tnorbye 😁 will take a look at this combination and see why my plugin is misbehaving!

Any progress on this?  I'm getting an error on almost every reformat command right now.

I have investigated this and do believe that the fix is to be made in the KDoc formatter plugin. See this comment with my findings and reproducer.


CONTRIBUTING.md missing at that place

I'm NOT sure it's best to point to this specific file (and not e.g. copy it to the place that was pointed to). Also problematic if forked (possible to have a relative path?) See here: https://github.com/dashpay/dash/pull/840#issuecomment-222540862
[skip ci]

Thanks!
However there is v<IP_ADDRESS> tag is wrong and I guess was created by an accident. Should be v0.12.1.x. Please update url there.
PS. Also please note that we don't accept PRs to master. I changed base to v0.12.0.x for this one but just be aware next time. ;)

@UdjinM6 I think you asked for the next commit (and not to v0.13.0.x). I'm not sure there will be a next time.. I just happened to stumble on this repo, I mostly to PRs to Julia (and then master, didn't think you should/could in some cases do otherwise [through GitHub]).

Thanks!
yep, notes about v0.13 was for @schinzelh only, sorry for confusion :)

Yeah, you can drop the <IP_ADDRESS> tag - not sure on the 0.13.0.x branch though - i guess it got deprecated by 0.12.1/0.12.2
ping @evan82


Update component registration logic

Alaska Airlines Pull Request
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Resolves: # (issue, if applicable)
Summary:
Please summarize the scope of the changes you have submitted, what the intent of the work is and anything that describes the before/after state of the project.

Update custom component registration by moving functionality to auro-library in order to fix conflicts with two different versions of the same components loading on the same page

Type of change:
Please delete options that are not relevant.

[ ] New capability
[x] Revision of an existing capability
[ ] Infrastructure change (automation, etc.)
[ ] Other (please elaborate)

Checklist:

[x] My update follows the CONTRIBUTING guidelines of this project
[x] I have performed a self-review of my own update

By submitting this Pull Request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Pull Requests will be evaluated by their quality of update and whether it is consistent with the goals and values of this project. Any submission is to be considered a conversation between the submitter and the maintainers of this project and may require changes to your submission.
Thank you for your submission!
-- Auro Design System Team

Summary by Sourcery
Merge changes from the beta branch into master, refactoring the custom component registration to utilize AuroLibrary, updating documentation and tests accordingly, and adding a new ESLint configuration for improved code quality.
Enhancements:

Refactor component registration to use the AuroLibrary for custom element registration, improving maintainability and reducing code duplication.

Documentation:

Update documentation to reflect changes in component registration, including instructions for installing AuroLibrary and using the new registration method.

Tests:

Modify tests to accommodate the new component registration method using AuroLibrary.

Chores:

Add a new ESLint configuration file to standardize code quality checks across the project.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: jason-capsule42:x: semantic-release-botYou have signed the CLA already but the status is still pending? Let us recheck it.

This work is now being handled in this PR:

https://github.com/AlaskaAirlines/auro-icon/pull/137


Kaizen Yoshida School request add school to JetBrains

Kaizen Yoshida School request add school to JetBrains
Thank you

@bedarren Please provide us with:

the school official website URL
the school street address, including city and country
an URL of a page on the official website where the school offers an IT-related long-term (>1 year) course
a proof, which shows that the school recognizes the domain you are submitting as an official email domain for the students.

You can provide us with any of the following as the proof:

A copy of instruction (a hard-copy or electronic one) given to you by the school that tells you to use an email address in this email domain
An URL of a page at the official school website that says that students use this email domain
A reference to a recognized journal where an article was published by someone from the school, with explicit affiliation to the school, and e-mail address from the email domain
A WHOIS data that link the email domain to the school somehow
A copy of official letter from the school representative saying that the school provides e-mail addresses in this domain to the students
Anything else that can prove this fact in similar way, i.e. can be verified independently.

@bedarren You might miss my previous comment. Please react.

@bedarren I am sorry, this request remains unanswered for a while. I have to close it without merging. Please reopen and comment if you have an update regarding the aforementioned issue. Thank you for your understanding!


[DataFrame] Add a LoadCsv overload or other method that takes CSV content directly

Getting from a CSV string to a DataFrame requires some ceremony involving creating a stream first. While this might not be the ideal approach from a performance perspective for larger volumes of data, it's very useful for prototyping with sample subsets of data. An API like this will also play nicely with the .NET Interactive #!value magic command.
DataFrame.LoadCsvFromString(@"col1,col2,col3
1,2,3
4,5,6
");

I think this is a good idea. I'll put a PR up for this soon and we can include this in the next preview


Memory issues during gen_avg for high-res fMRI data

What happened?
For a while now I've been getting memory issues like the following, despite using a machine with 128GB RAM and allowing fMRIprep 24GB. I realize I am using pretty high resolution data and long scans (and fMRIprep is using float64), but I feel like it should still be able to allocate more than 13GB.

What command did you use?
running the following in python using os.system:

docker run --rm -v /data2/Brian/random/license.txt:/opt/freesurfer/license.txt -v /data/2104/raw_data:/data/2104/raw_data -v /data/2104/derivatives:/data/2104/derivatives -v /data/2104/ignore/temp_21040017:/data/2104/ignore/temp_21040017 nipreps/fmriprep:latest /data/2104/raw_data /data/2104/derivatives participant --participant-label 21040017 -w /data/2104/ignore/temp_21040017 --output-spaces anat MNI152NLin6Asym:res-2 --fs-license-file /opt/freesurfer/license.txt --mem 24000 --output-layout legacy')

What version of fMRIPrep are you running?
23.0.1
How are you running fMRIPrep?
Docker
Is your data BIDS valid?
Yes
Are you reusing any previously computed results?
FreeSurfer, Anatomical derivatives, Work directory
Please copy and paste any relevant log output.
No response
Additional information / screenshots
I am using previously computed results in the sense that I've tried running it multiple times after the crashes. Would be grateful for any help. Thank you!

A few things:

Have you checked that your Docker containers have access to all the RAM you need? https://docs.docker.com/config/containers/resource_constraints/#memory
Assuming you do have access to 128GB of RAM, the 13GB allocation is probably not the problem in itself so much as that it is happening while other nodes are also making large allocations. We may have some badly tagged nodes that are using much more RAM than they claim. Watching the system with htop can be enlightening.
I think we need to rewrite this interface. There's absolutely no reason that we can't take an average voxel-by-voxel without loading the whole file into RAM.

Thanks for the suggestions. fMRIprep continued crashing when I supplied --mem 32GB of RAM, however I've run it twice now with 64GB of RAM and it hasn't crashed. Given that the docs say about 8GB of RAM/subject is necessary, well, that definitely wasn't true in my case.


consistent-chaining false positives

Describe the bug
The bug pertains to the new consistent-chaining rule. As shown in the documentation, the following should be correct
const foo2 = []
  .map(x => x + 'bar')
  .filter(Boolean)

However, the following code returns an error
const x = {foo: 1, bar: 2 }
const result = Object.keys(x)
  .reduce((acc, key) => ({ ...acc, ...x[key] }), {});

It looks like the rule explicitly expects the first property in the chain to be a variable. If its a function, it breaks.

Reproduction
https://github.com/MrVauxs/antfu-repro
System Info
System:
    OS: Windows 11 10.0.22631
    CPU: (12) x64 AMD Ryzen 5 3600 6-Core Processor
    Memory: 12.42 GB / 31.93 GB
  Binaries:
    Node: 22.6.0 - C:\Program Files\nodejs\node.EXE
    npm: 10.8.3 - C:\Program Files\nodejs\npm.CMD
    bun: 1.1.12 - ~\.bun\bin\bun.EXE
  Browsers:
    Edge: Chromium (127.0.2651.74)
    Internet Explorer: 11.0.22621.3527

Used Package Manager
npm
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guide.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A, please open a GitHub Discussion instead.
[X] The provided reproduction is a minimal reproducible of the bug.

Contributions

[ ] I am willing to submit a PR to fix this issue
[ ] I am willing to submit a PR with failing tests (actually just go ahead and do it, thanks!)

Also, properties on this should also not be wrapped
const result = this.userList
  .filter(user => user.name.startWith('T'))
  .map(user => user.phoneNumber)

https://github.com/antfu/eslint-plugin-antfu/commit/10e9920e51d691635016921dc72700e2624726c0 should fix it. thanks


fixes #14 live time reset

Hi Ash,
This will reset the on-screen timer to zero if  "ResetTimeWithNoCheckpoint" is true.
Steve

Great spot, thank you! :)


Integration tests for atomic-server

The server currently has minimal coverage for some helper methods, but what it really needs is some integration coverage. What I'd like to do is simply call the Main process and fire a bunch of requests, see if they're handled properly.
Some reading material for doing this:

https://cloudmaker.dev/actix-integration-tests/

Some time ago I did add some end to end tests. Could still be better, but it tests basic things like authentication and search.


Custom metric mix train and validation data

System information

Linux ubuntu, CPU
installed packages using anaconda with python 3.6
Tensorflow version v1.12.0-0-ga6d8ffae09 1.12.0
Keras version  2.2.4

Describe the current behavior
When using a custom metric, the metric is computed on training data and validation data together.
Describe the expected behavior
The train metric should be on train data only and validation metric (*_val) should be on validation data only.

In order to expedite the trouble-shooting process, please provide a code snippet to reproduce the issue reported here. Thanks!

Automatically closing due to lack of recent activity. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!


WARNING: An illegal reflective access operation has occurred (Gradle, Java 9, Windows 10)

input command gradle -version (Windows 10)
Expected Behavior

Gradle 4.2.1
Build time:   2017-10-02 15:36:21 UTC
Revision:     a88ebd6be7840c2e59ae4782eb0f27fbe3405ddf
Groovy:       2.4.12
Ant:          Apache Ant(TM) version 1.9.6 compiled on June 29 2015
JVM:          9 (Oracle Corporation 9+181)
OS:           Windows 10 10.0 amd64
C:\Windows\system32>java -version
java version "9"
Java(TM) SE Runtime Environment (build 9+181)
Java HotSpot(TM) 64-Bit Server VM (build 9+181, mixed mode)
Current Behavior

WARNING: An illegal reflective access operation has occurred
WARNING: Illegal reflective access by org.codehaus.groovy.reflection.CachedClass (file:/C:/gradle/lib/groovy-all-2.4.12.jar) to method java.lang.Object.finalize()
WARNING: Please consider reporting this to the maintainers of org.codehaus.groovy.reflection.CachedClass
WARNING: Use --illegal-access=warn to enable warnings of further illegal reflective access operations
WARNING: All illegal access operations will be denied in a future release

Gradle 4.2.1
Build time:   2017-10-02 15:36:21 UTC
Revision:     a88ebd6be7840c2e59ae4782eb0f27fbe3405ddf
Groovy:       2.4.12
Ant:          Apache Ant(TM) version 1.9.6 compiled on June 29 2015
JVM:          9 (Oracle Corporation 9+181)
OS:           Windows 10 10.0 amd64
C:\Windows\system32>java -version
java version "9"
Java(TM) SE Runtime Environment (build 9+181)
Java HotSpot(TM) 64-Bit Server VM (build 9+181, mixed mode)
Context

trying to build projects with gradle using java 9 wondering is this will be a problem
Steps to Reproduce (for bugs)

no current project
Your Environment

Build scan URL:

Please see #2995 for more details and solution.

The linked ticket contains a workaround, but not really a solution. The real issue is a problem in the JDK. There is actually no illegal access, but the JDK method trySetAccessible is misbehaving. This will hopefully be fixed in a future JDK version.

https://issues.apache.org/jira/browse/GROOVY-8339 is the Groovy Ticket which is tracking the issue

So, as much as we could pass this off to the JDK for being silly, I think it's totally within our power to simply output a version number without seeing an error. We don't need Reflection in order to spit out a version...right?

Not sure what you mean. Why would we invest in a fix that only works for gradle -version when every other task or command will still print the warning?

Odd, I don't get it on other commands:
[2018-03-09T15:20:22-0700]: mitchell.ludwig@Mitchells-MacBook-Pro:~/dev/playgradle
$ gradle -version
WARNING: An illegal reflective access operation has occurred
WARNING: Illegal reflective access by org.codehaus.groovy.reflection.CachedClass (file:/usr/local/Cellar/gradle/4.6/libexec/lib/groovy-all-2.4.12.jar) to method java.lang.Object.finalize()
WARNING: Please consider reporting this to the maintainers of org.codehaus.groovy.reflection.CachedClass
WARNING: Use --illegal-access=warn to enable warnings of further illegal reflective access operations
WARNING: All illegal access operations will be denied in a future release

------------------------------------------------------------
Gradle 4.6
------------------------------------------------------------

Build time:   2018-02-28 13:36:36 UTC
Revision:     8fa6ce7945b640e6168488e4417f9bb96e4ab46c

Groovy:       2.4.12
Ant:          Apache Ant(TM) version 1.9.9 compiled on February 2 2017
JVM:          9.0.4 (Oracle Corporation 9.0.4+11)
OS:           Mac OS X 10.13.3 x86_64

[2018-03-09T15:20:30-0700]: mitchell.ludwig@Mitchells-MacBook-Pro:~/dev/playgradle
$ gradle wrapper
Starting a Gradle Daemon (subsequent builds will be faster)

BUILD SUCCESSFUL in 5s
1 actionable task: 1 up-to-date
[2018-03-09T15:20:41-0700]: mitchell.ludwig@Mitchells-MacBook-Pro:~/dev/playgradle
$ cat build.gradle
plugins {
    id "base"
}

version '1.0-SNAPSHOT'

apply plugin: 'java'

sourceCompatibility = 1.8

repositories {
    mavenCentral()
}

dependencies {
    testCompile group: 'junit', name: 'junit', version: '4.12'
}

task copy(type: Copy, group: "Custom", description: "Copies sources to the dest directory") {
    from "src"
    into "dest"
}

task zip(type: Zip, group: "Archive", description: "Archives sources in a zip file") {
    from "src"
}
[2018-03-09T15:20:49-0700]: mitchell.ludwig@Mitchells-MacBook-Pro:~/dev/playgradle
$ gradle copy

BUILD SUCCESSFUL in 0s
1 actionable task: 1 up-to-date
[2018-03-09T15:20:59-0700]: mitchell.ludwig@Mitchells-MacBook-Pro:~/dev/playgradle
$

That is surprising indeed. Last time I checked this always happened when the Groovy Runtime was loaded (i.e. on every build). Thanks for checking!

I got the same issue under MacOS
bash-3.2$ gradle -v
WARNING: An illegal reflective access operation has occurred
WARNING: Illegal reflective access by org.codehaus.groovy.reflection.CachedClass (file:/usr/local/Cellar/gradle/4.6/libexec/lib/groovy-all-2.4.12.jar) to method java.lang.Object.finalize()
WARNING: Please consider reporting this to the maintainers of org.codehaus.groovy.reflection.CachedClass
WARNING: Use --illegal-access=warn to enable warnings of further illegal reflective access operations
WARNING: All illegal access operations will be denied in a future release

------------------------------------------------------------
Gradle 4.6
------------------------------------------------------------

Build time:   2018-02-28 13:36:36 UTC
Revision:     8fa6ce7945b640e6168488e4417f9bb96e4ab46c

Groovy:       2.4.12
Ant:          Apache Ant(TM) version 1.9.9 compiled on February 2 2017
JVM:          9 (Oracle Corporation 9+181)
OS:           Mac OS X 10.12.6 x86_64

bash-3.2$ java -version
java version "9"
Java(TM) SE Runtime Environment (build 9+181)
Java HotSpot(TM) 64-Bit Server VM (build 9+181, mixed mode)

92:~ macbookairmd760$ cd
192:~ macbookairmd760$ gradle -v
WARNING: An illegal reflective access operation has occurred
WARNING: Illegal reflective access by org.codehaus.groovy.reflection.CachedClass (file:/usr/local/Cellar/gradle/4.6/libexec/lib/groovy-all-2.4.12.jar) to method java.lang.Object.finalize()
WARNING: Please consider reporting this to the maintainers of org.codehaus.groovy.reflection.CachedClass
WARNING: Use --illegal-access=warn to enable warnings of further illegal reflective access operations
WARNING: All illegal access operations will be denied in a future release

Gradle 4.6
Build time:   2018-02-28 13:36:36 UTC
Revision:     8fa6ce7945b640e6168488e4417f9bb96e4ab46c
Groovy:       2.4.12
Ant:          Apache Ant(TM) version 1.9.9 compiled on February 2 2017
JVM:          9.0.4 (Oracle Corporation 9.0.4+11)
OS:           Mac OS X 10.13.3 x86_64
192:~ macbookairmd760$

Maybe something is specific to OSX? I'm on High Sierra. Maybe it's high? :P

Uninstalling java-9 and installing java-8 solved my issue

Same issue on Windows 10 with IntelliJ
WARNING: An illegal reflective access operation has occurred
WARNING: Illegal reflective access by org.codehaus.groovy.reflection.CachedClass (file:/C:/Users/Lane/.gradle/wrapper/dists/gradle-4.6-bin/4jp4stjndanmxuerzfseyb6wo/gradle-4.6/lib/groovy-all-2.4.12.jar) to method java.lang.Object.finalize()
WARNING: Please consider reporting this to the maintainers of org.codehaus.groovy.reflection.CachedClass
WARNING: Use --illegal-access=warn to enable warnings of further illegal reflective access operations
WARNING: All illegal access operations will be denied in a future release

Gradle 4.6
Build time:   2018-02-28 13:36:36 UTC
Revision:     8fa6ce7945b640e6168488e4417f9bb96e4ab46c
Groovy:       2.4.12
Ant:          Apache Ant(TM) version 1.9.9 compiled on February 2 2017
JVM:          9.0.4 (Oracle Corporation 9.0.4+11)
OS:           Windows 10 10.0 amd64

Gradle 4.7
/Library/Java/JavaVirtualMachines/jdk-10.jdk/Contents/Home/bin/java -agentlib:jdwp=transport=dt_socket,address=<IP_ADDRESS>:51218,suspend=y,server=n --illegal-access=permit -XX:TieredStopAtLevel=1 -noverify -Dspring.output.ansi.enabled=always -Dcom.sun.management.jmxremote -Dcom.sun.management.jmxremote.port=51217 -Dcom.sun.management.jmxremote.authenticate=false -Dcom.sun.management.jmxremote.ssl=false -Djava.rmi.server.hostname=<IP_ADDRESS> -Dspring.liveBeansView.mbeanDomain -Dspring.application.admin.enabled=true -javaagent:/Users/donhuvy/Library/Caches/IntelliJIdea2018.1/groovyHotSwap/gragent.jar -javaagent:/Users/donhuvy/Library/Caches/IntelliJIdea2018.1/captureAgent/debugger-agent.jar=file:/private/var/folders/lj/tgdhz0mx5t54vglcc8982vbc0000gn/T/capture.props -Dfile.encoding=UTF-8 -classpath "/Users/donhuvy/Documents/source_code/github.com/donhuvy/accounting/out/production/classes:/Users/donhuvy/Documents/source_code/github.com/donhuvy/accounting/out/production/resources:/Users/donhuvy/.m2/repository/org/springframework/boot/spring-boot-starter-actuator/2.0.1.RELEASE/spring-boot-starter-actuator-2.0.1.RELEASE.jar:/Users/donhuvy/.m2/repository/javax/annotation/javax.annotation-api/1.3.2/javax.annotation-api-1.3.2.jar:/Users/donhuvy/.m2/repository/org/apache/tomcat/embed/tomcat-embed-core/8.5.29/tomcat-embed-core-8.5.29.jar:/Users/donhuvy/.m2/repository/org/apache/tomcat/embed/tomcat-embed-el/8.5.29/tomcat-embed-el-8.5.29.jar:/Users/donhuvy/.m2/repository/org/apache/tomcat/embed/tomcat-embed-websocket/8.5.29/tomcat-embed-websocket-8.5.29.jar:/Users/donhuvy/.m2/repository/org/springframework/boot/spring-boot-starter-tomcat/2.0.1.RELEASE/spring-boot-starter-tomcat-2.0.1.RELEASE.jar:/Users/donhuvy/.m2/repository/org/springframework/boot/spring-boot-starter-data-jpa/2.0.1.RELEASE/spring-boot-starter-data-jpa-2.0.1.RELEASE.jar:/Users/donhuvy/.gradle/caches/modules-2/files-2.1/org.springframework.boot/spring-boot-starter-security/2.0.1.RELEASE/d573123d27bc3c7333fbc352eced092737c7303f/spring-boot-starter-security-2.0.1.RELEASE.jar:/Users/donhuvy/.m2/repository/org/springframework/boot/spring-boot-starter-web/2.0.1.RELEASE/spring-boot-starter-web-2.0.1.RELEASE.jar:/Users/donhuvy/.gradle/caches/modules-2/files-2.1/org.springframework.boot/spring-boot-starter-thymeleaf/2.0.0.RELEASE/3f77dc6b3bfa8e9cc9e0eaa9abc32c5b00b5046a/spring-boot-starter-thymeleaf-2.0.0.RELEASE.jar:/Users/donhuvy/.m2/repository/com/oracle/ojdbc7/<IP_ADDRESS>/ojdbc7-<IP_ADDRESS>.jar:/Users/donhuvy/.m2/repository/javax/xml/bind/jaxb-api/2.3.0/jaxb-api-2.3.0.jar:/Users/donhuvy/.gradle/caches/modules-2/files-2.1/com.sun.xml.bind/jaxb-core/2.3.0/d044c784e41d026778693fb44a8026c1fd9a7506/jaxb-core-2.3.0.jar:/Users/donhuvy/.gradle/caches/modules-2/files-2.1/com.sun.xml.bind/jaxb-impl/2.3.0/3a3c6a62719d967175b76b63925f1fb495f11437/jaxb-impl-2.3.0.jar:/Users/donhuvy/.m2/repository/org/thymeleaf/extras/thymeleaf-extras-java8time/3.0.1.RELEASE/thymeleaf-extras-java8time-3.0.1.RELEASE.jar:/Users/donhuvy/.gradle/caches/modules-2/files-2.1/org.thymeleaf.extras/thymeleaf-extras-conditionalcomments/2.1.2.RELEASE/d188a0c94614956d79644b71bdd05098fa4d6a3/thymeleaf-extras-conditionalcomments-2.1.2.RELEASE.jar:/Users/donhuvy/.m2/repository/org/thymeleaf/extras/thymeleaf-extras-springsecurity4/3.0.2.RELEASE/thymeleaf-extras-springsecurity4-3.0.2.RELEASE.jar:/Users/donhuvy/.m2/repository/org/springframework/boot/spring-boot-starter-aop/2.0.1.RELEASE/spring-boot-starter-aop-2.0.1.RELEASE.jar:/Users/donhuvy/.m2/repository/org/springframework/boot/spring-boot-starter-jdbc/2.0.1.RELEASE/spring-boot-starter-jdbc-2.0.1.RELEASE.jar:/Users/donhuvy/.m2/repository/org/springframework/boot/spring-boot-starter-json/2.0.1.RELEASE/spring-boot-starter-json-2.0.1.RELEASE.jar:/Users/donhuvy/.m2/repository/org/springframework/boot/spring-boot-devtools/2.0.1.RELEASE/spring-boot-devtools-2.0.1.RELEASE.jar:/Users/donhuvy/.m2/repository/org/springframework/boot/spring-boot-starter/2.0.1.RELEASE/spring-boot-starter-2.0.1.RELEASE.jar:/Users/donhuvy/.m2/repository/org/springframework/boot/spring-boot-actuator-autoconfigure/2.0.1.RELEASE/spring-boot-actuator-autoconfigure-2.0.1.RELEASE.jar:/Users/donhuvy/.m2/repository/io/micrometer/micrometer-core/1.0.3/micrometer-core-1.0.3.jar:/Users/donhuvy/.m2/repository/org/hibernate/hibernate-core/5.2.16.Final/hibernate-core-5.2.16.Final.jar:/Users/donhuvy/.m2/repository/javax/transaction/javax.transaction-api/1.2/javax.transaction-api-1.2.jar:/Users/donhuvy/.m2/repository/org/springframework/data/spring-data-jpa/2.0.6.RELEASE/spring-data-jpa-2.0.6.RELEASE.jar:/Users/donhuvy/.m2/repository/org/springframework/spring-aspects/5.0.5.RELEASE/spring-aspects-5.0.5.RELEASE.jar:/Users/donhuvy/.gradle/caches/modules-2/files-2.1/org.springframework.security/spring-security-config/5.0.4.RELEASE/355fc8c3d1c61fe85915082587946ad346250d85/spring-security-config-5.0.4.RELEASE.jar:/Users/donhuvy/.gradle/caches/modules-2/files-2.1/org.springframework.security/spring-security-web/5.0.4.RELEASE/bd2592c928d043f70742fd8ae409f751a63132dd/spring-security-web-5.0.4.RELEASE.jar:/Users/donhuvy/.m2/repository/org/springframework/spring-webmvc/5.0.5.RELEASE/spring-webmvc-5.0.5.RELEASE.jar:/Users/donhuvy/.m2/repository/org/springframework/boot/spring-boot-autoconfigure/2.0.1.RELEASE/spring-boot-autoconfigure-2.0.1.RELEASE.jar:/Users/donhuvy/.m2/repository/org/springframework/boot/spring-boot-actuator/2.0.1.RELEASE/spring-boot-actuator-2.0.1.RELEASE.jar:/Users/donhuvy/.m2/repository/org/springframework/boot/spring-boot/2.0.1.RELEASE/spring-boot-2.0.1.RELEASE.jar:/Users/donhuvy/.gradle/caches/modules-2/files-2.1/org.springframework.security/spring-security-core/5.0.4.RELEASE/f2924cd62fa8b14546b2b3c31dcd9e55abf9a5e/spring-security-core-5.0.4.RELEASE.jar:/Users/donhuvy/.m2/repository/org/springframework/spring-context/5.0.5.RELEASE/spring-context-5.0.5.RELEASE.jar:/Users/donhuvy/.m2/repository/org/springframework/spring-aop/5.0.5.RELEASE/spring-aop-5.0.5.RELEASE.jar:/Users/donhuvy/.m2/repository/org/hibernate/validator/hibernate-validator/6.0.9.Final/hibernate-validator-6.0.9.Final.jar:/Users/donhuvy/.m2/repository/org/springframework/spring-web/5.0.5.RELEASE/spring-web-5.0.5.RELEASE.jar:/Users/donhuvy/.m2/repository/org/thymeleaf/thymeleaf-spring5/3.0.9.RELEASE/thymeleaf-spring5-3.0.9.RELEASE.jar:/Users/donhuvy/.m2/repository/org/thymeleaf/thymeleaf/3.0.9.RELEASE/thymeleaf-3.0.9.RELEASE.jar:/Users/donhuvy/.m2/repository/com/zaxxer/HikariCP/2.7.8/HikariCP-2.7.8.jar:/Users/donhuvy/.m2/repository/org/springframework/data/spring-data-commons/2.0.6.RELEASE/spring-data-commons-2.0.6.RELEASE.jar:/Users/donhuvy/.m2/repository/org/springframework/boot/spring-boot-starter-logging/2.0.1.RELEASE/spring-boot-starter-logging-2.0.1.RELEASE.jar:/Users/donhuvy/.m2/repository/ch/qos/logback/logback-classic/1.2.3/logback-classic-1.2.3.jar:/Users/donhuvy/.m2/repository/org/apache/logging/log4j/log4j-to-slf4j/2.10.0/log4j-to-slf4j-2.10.0.jar:/Users/donhuvy/.m2/repository/org/slf4j/jul-to-slf4j/1.7.25/jul-to-slf4j-1.7.25.jar:/Users/donhuvy/.m2/repository/org/slf4j/slf4j-api/1.7.25/slf4j-api-1.7.25.jar:/Users/donhuvy/.m2/repository/org/attoparser/attoparser/2.0.4.RELEASE/attoparser-2.0.4.RELEASE.jar:/Users/donhuvy/.m2/repository/org/springframework/spring-orm/5.0.5.RELEASE/spring-orm-5.0.5.RELEASE.jar:/Users/donhuvy/.m2/repository/org/springframework/spring-jdbc/5.0.5.RELEASE/spring-jdbc-5.0.5.RELEASE.jar:/Users/donhuvy/.m2/repository/org/springframework/spring-tx/5.0.5.RELEASE/spring-tx-5.0.5.RELEASE.jar:/Users/donhuvy/.m2/repository/org/springframework/spring-beans/5.0.5.RELEASE/spring-beans-5.0.5.RELEASE.jar:/Users/donhuvy/.m2/repository/org/springframework/spring-expression/5.0.5.RELEASE/spring-expression-5.0.5.RELEASE.jar:/Users/donhuvy/.m2/repository/org/springframework/spring-core/5.0.5.RELEASE/spring-core-5.0.5.RELEASE.jar:/Users/donhuvy/.m2/repository/org/yaml/snakeyaml/1.19/snakeyaml-1.19.jar:/Users/donhuvy/.m2/repository/com/fasterxml/jackson/datatype/jackson-datatype-jsr310/2.9.5/jackson-datatype-jsr310-2.9.5.jar:/Users/donhuvy/.m2/repository/com/fasterxml/jackson/datatype/jackson-datatype-jdk8/2.9.5/jackson-datatype-jdk8-2.9.5.jar:/Users/donhuvy/.m2/repository/com/fasterxml/jackson/module/jackson-module-parameter-names/2.9.5/jackson-module-parameter-names-2.9.5.jar:/Users/donhuvy/.m2/repository/com/fasterxml/jackson/core/jackson-databind/2.9.5/jackson-databind-2.9.5.jar:/Users/donhuvy/.m2/repository/org/hdrhistogram/HdrHistogram/2.1.10/HdrHistogram-2.1.10.jar:/Users/donhuvy/.m2/repository/org/latencyutils/LatencyUtils/2.0.3/LatencyUtils-2.0.3.jar:/Users/donhuvy/.m2/repository/org/aspectj/aspectjweaver/1.8.13/aspectjweaver-1.8.13.jar:/Users/donhuvy/.m2/repository/org/hibernate/common/hibernate-commons-annotations/5.0.1.Final/hibernate-commons-annotations-5.0.1.Final.jar:/Users/donhuvy/.m2/repository/org/jboss/logging/jboss-logging/3.3.2.Final/jboss-logging-3.3.2.Final.jar:/Users/donhuvy/.m2/repository/org/hibernate/javax/persistence/hibernate-jpa-2.1-api/1.0.0.Final/hibernate-jpa-2.1-api-1.0.0.Final.jar:/Users/donhuvy/.m2/repository/ognl/ognl/3.1.12/ognl-3.1.12.jar:/Users/donhuvy/.m2/repository/org/javassist/javassist/3.22.0-GA/javassist-3.22.0-GA.jar:/Users/donhuvy/.m2/repository/antlr/antlr/2.7.7/antlr-2.7.7.jar:/Users/donhuvy/.m2/repository/org/jboss/jandex/2.0.3.Final/jandex-2.0.3.Final.jar:/Users/donhuvy/.m2/repository/com/fasterxml/classmate/1.3.4/classmate-1.3.4.jar:/Users/donhuvy/.m2/repository/dom4j/dom4j/1.6.1/dom4j-1.6.1.jar:/Users/donhuvy/.m2/repository/javax/validation/validation-api/2.0.1.Final/validation-api-2.0.1.Final.jar:/Users/donhuvy/.m2/repository/org/unbescape/unbescape/1.1.5.RELEASE/unbescape-1.1.5.RELEASE.jar:/Users/donhuvy/.m2/repository/org/springframework/spring-jcl/5.0.5.RELEASE/spring-jcl-5.0.5.RELEASE.jar:/Users/donhuvy/.m2/repository/com/fasterxml/jackson/core/jackson-annotations/2.9.0/jackson-annotations-2.9.0.jar:/Users/donhuvy/.m2/repository/com/fasterxml/jackson/core/jackson-core/2.9.5/jackson-core-2.9.5.jar:/Users/donhuvy/.m2/repository/ch/qos/logback/logback-core/1.2.3/logback-core-1.2.3.jar:/Users/donhuvy/.m2/repository/org/apache/logging/log4j/log4j-api/2.10.0/log4j-api-2.10.0.jar:/Applications/IntelliJ IDEA.app/Contents/lib/idea_rt.jar" com.donhuvy.AccountingApplication
Connected to the target VM, address: '<IP_ADDRESS>:51218', transport: 'socket'
The Class-Path manifest attribute in /Users/donhuvy/.gradle/caches/modules-2/files-2.1/com.sun.xml.bind/jaxb-core/2.3.0/d044c784e41d026778693fb44a8026c1fd9a7506/jaxb-core-2.3.0.jar referenced one or more files that do not exist: file:/Users/donhuvy/.gradle/caches/modules-2/files-2.1/com.sun.xml.bind/jaxb-core/2.3.0/d044c784e41d026778693fb44a8026c1fd9a7506/jaxb-api.jar
The Class-Path manifest attribute in /Users/donhuvy/.gradle/caches/modules-2/files-2.1/com.sun.xml.bind/jaxb-impl/2.3.0/3a3c6a62719d967175b76b63925f1fb495f11437/jaxb-impl-2.3.0.jar referenced one or more files that do not exist: file:/Users/donhuvy/.gradle/caches/modules-2/files-2.1/com.sun.xml.bind/jaxb-impl/2.3.0/3a3c6a62719d967175b76b63925f1fb495f11437/jaxb-core.jar
22:50:24.905 [main] DEBUG org.springframework.boot.devtools.settings.DevToolsSettings - Included patterns for restart : []
22:50:24.908 [main] DEBUG org.springframework.boot.devtools.settings.DevToolsSettings - Excluded patterns for restart : [/spring-boot-starter-[\w-]+/, /spring-boot/target/classes/, /spring-boot-starter/target/classes/, /spring-boot-devtools/target/classes/, /spring-boot-actuator/target/classes/, /spring-boot-autoconfigure/target/classes/]
22:50:24.908 [main] DEBUG org.springframework.boot.devtools.restart.ChangeableUrls - Matching URLs for reloading : [file:/Users/donhuvy/Documents/source_code/github.com/donhuvy/accounting/out/production/classes/, file:/Users/donhuvy/Documents/source_code/github.com/donhuvy/accounting/out/production/resources/]

  .   ____          _            __ _ _
 /\\ / ___'_ __ _ _(_)_ __  __ _ \ \ \ \
( ( )\___ | '_ | '_| | '_ \/ _` | \ \ \ \
 \\/  ___)| |_)| | | | | || (_| |  ) ) ) )
  '  |____| .__|_| |_|_| |_\__, | / / / /
 =========|_|==============|___/=/_/_/_/
 :: Spring Boot ::        (v2.0.1.RELEASE)

WARNING: An illegal reflective access operation has occurred
WARNING: Illegal reflective access by org.springframework.cglib.core.ReflectUtils$1 (file:/Users/donhuvy/.m2/repository/org/springframework/spring-core/5.0.5.RELEASE/spring-core-5.0.5.RELEASE.jar) to method java.lang.ClassLoader.defineClass(java.lang.String,byte[],int,int,java.security.ProtectionDomain)
WARNING: Please consider reporting this to the maintainers of org.springframework.cglib.core.ReflectUtils$1
WARNING: Use --illegal-access=warn to enable warnings of further illegal reflective access operations
WARNING: All illegal access operations will be denied in a future release

Happens with me running Java 10 on Ubuntu 18:04 Lts

It happens with Maven too, not a Gradle issue per se...

Happens with me running Gradle 4.9 on Java 10 SDK with Windows 10 Pro

In case of org.codehaus.groovy.reflection.CachedClass it's a Groovy issue and we'll have to update our Gradle version as soon as there's one that does not produce this warning anymore.
@volare2016 org.apache.poi.util.SAXHelper does not sound like it's called from Gradle core. Where does read.xlsx() come from, a plugin?

As explained several times, this is a Groovy issue and can't be fixed by Gradle.

@oehme / @marcphilipp  it probably makes sense to lock the issue (believe you can do that in GitHub)

when I use the netty, the situation also happend ,I am trying to change the jdk

Get same issue on Ubuntu 16.04LTS
Use gradle -v get warning msg:
WARNING: An illegal reflective access operation has occurred
WARNING: Illegal reflective access by org.codehaus.groovy.reflection.CachedClass (file:/usr/local/gradle/gradle-4.10/lib/groovy-all-2.4.15.jar) to method java.lang.Object.finalize()
WARNING: Please consider reporting this to the maintainers of org.codehaus.groovy.reflection.CachedClass
WARNING: Use --illegal-access=warn to enable warnings of further illegal reflective access operations
WARNING: All illegal access operations will be denied in a future release

------------------------------------------------------------
Gradle 4.10
------------------------------------------------------------

Build time:   2018-08-27 18:35:06 UTC
Revision:     ee3751ed9f2034effc1f0072c2b2ee74b5dce67d

Kotlin DSL:   1.0-rc-3
Kotlin:       1.2.60
Groovy:       2.4.15
Ant:          Apache Ant(TM) version 1.9.11 compiled on March 23 2018
JVM:          10.0.1 ("Oracle Corporation" 10.0.1+10)
OS:           Linux 4.15.0-33-generic amd64

and JDK version is:
java 10.0.1 2018-04-17
Java(TM) SE Runtime Environment 18.3 (build 10.0.1+10)
Java HotSpot(TM) 64-Bit Server VM 18.3 (build 10.0.1+10, mixed mode)

Happens with me when I use Mybatis Plus. I solved this problem by update  Mybatis Plus Version to 3.2.0.

Just recceived a similar error with gradle 7.0.2, Apache Spark and Java 11


[Feature] iter support running from a map

conc is very wonderful to writing concurrent code, thanks for your project!
Currently, there is only iter.Map() run from a slice. But sometimes we need to concurrently run from a map.
Could you consider provide a function like:
func RunFromMap[K comparable, V any, R any](input map[K]V, f func(*T, *V) R) []R

(iter.Map running from slice has occuppied the best name. The naming maybe a little hard to decide😭)

Spoiler: I'd like the next version of the conc/iter package to support the new stdlib iter package. In this case, you could create a Seq2 from any map and conc could consume that.
Still pretty early in the design process (especially since iterators haven't stabilized yet), but stay tuned 🙂

It sounds cool, thanks!


SourceCode Pro with incorrect SHA256

I am not sure why the hash would have changed in Adobe's archive...
zsh/3 474 [255] # nix-channel --list
nixos https://nixos.org/channels/nixos-15.09
nixpkgs https://nixos.org/channels/nixpkgs-unstable

zsh/3 476 [130] # nix-channel --update
downloading Nix expressions from ‘https://nixos.org/releases/nixos/15.09/nixos-15.09.911.53286a8//nixexprs.tar.xz’...
downloading ‘https://nixos.org/releases/nixos/15.09/nixos-15.09.911.53286a8//nixexprs.tar.xz’... [6400/6760 KiB, 335.7 KiB/s]
downloading Nix expressions from ‘https://nixos.org/releases/nixpkgs/nixpkgs-16.03pre74770.2b7b3aa//nixexprs.tar.xz’...
downloading ‘https://nixos.org/releases/nixpkgs/nixpkgs-16.03pre74770.2b7b3aa//nixexprs.tar.xz’... [5056/6678 KiB, 1140.2 KiB/s]
unpacking channels...

trying https://github.com/adobe-fonts/source-code-pro/archive/2.010R-ro/1.030R-it.tar.gz
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100   148    0   148    0     0    796      0 --:--:-- --:--:-- --:--:--   875
100 7502k  100 7502k    0     0  3416k      0  0:00:02  0:00:02 --:--:-- 5919k
output path ‘/nix/store/zajz82ssvqvjkkw526rq2vq4z82rkkbq-1.030R-it.tar.gz’ has sha256 hash ‘1y44p2i7hd1klq81xbh796y7n4rzjvn37jrqw0nz31k59v8a1r9y’ when ‘12wijgxrdzqxpw2q420nsq9aj454vhg3rq6n81jbqvgzxhxjpf7w’ was expected
building path(s) ‘/nix/store/dcz57s155y1hpkfjinmkl2dnnp0hrvfm-Xresources-Xft’
cannot build derivation ‘/nix/store/j0cv6jhxjij0sk3s56pbscy1b0mgsjp9-source-code-pro-2.010.drv’: 1 dependencies couldn't be built

Because someone used fetchurl to get a dynamically generated tarball from GitHub. There's fetchFromGitHub just for that reason...

I will fix this some time this week.

This stung me, so I ended up fixing it in the above PR.

Awesome, got me yesterday, I just didn't have time...
Thanks.


Add Homebrew to suggested installation methods

This can now be installed using Homebrew as of Homebrew/homebrew-core#193855.

Thanks for handling the homebrew packaging!


Update index.php

https://github.com/andrieslouw/imagesweserv/blob/67d1c4362c44b103ae5d5223727b3b49342c49a1/index.php#L508
$q = (empty($_GET['q']) OR !ctype_digit($_GET['q']) OR $_GET['q'] > 95 OR $_GET['q'] < 0) ? '85' : $_GET['q'];

Only value up to 95 making any changes.

Thanks for spotting this issue! This is indeed some odd behaviour that was from a time that the GD-library had issues with high JPEG-quality numbers. I'll need to test if this behaviour is still necessary, or that qualities up to 100 are acceptable nowadays. So I will fix line 508 in that case, to match the documented behaviour.

Fixed in https://github.com/andrieslouw/imagesweserv/commit/ee6c570a33ee4ebd596cd3bcbceffc679f6c08f0


.npy Extension Issue

I am trying to train the model using default parameters. All of the files I downloaded from "eeg" on the readme end in .pth even though there are lines of code that indicate that a .npy file is expected. What can I do to fix this issue? Is there something I am missing? Additionally, the datasets/mne_data folder does not exist for me. Where was I supposed to get this from? I am trying to use this framework for a school project so any insight would be greatly appreciated.

You can directly download this data from the MOABB project   https://github.com/NeuroTechX/moabb   and use their respective processing code for operations like filtering. Finally, save them as npy files.

Could you be more specific as to which files from the MOABB repo to save as .npy in the datasets/mne_data folder?

@bbaaii was there a specific set of MOABB datasets you used for pre-training? Or does it not matter because it's just auto encoder?


Analysis script organization

We should agree on an organization strategy for the analysis pipeline.
Option 1: Do everything in a jupyter notebook

By "everything" I mean, download the dataset from openneuro, segment the spinal cord,  identify pre-existing manual segmentations, co-register segmentation onto various contrasts, compute metrics, plot figures.
This was first approach we took and we found that computation times (for downlaoding data, segmenting, co-registering) were generally very long.

Option 2: Separate data analysis from figure generation

Create a script to download the dataset from openneuro, segment the spinal cord, identify pre-existing manual segmentations, co-register segmentation onto various contrasts, compute metrics. Save metrics to a file, save file to openneuro repo or github (?).
Create a jupyter notebook to download metrics file and generate figures/plots of interest.
Motivation: with this option, people who are interested to playing around with interactive figures and changing the way the data is represented do not have to go through the time-consuming steps listed above.

Once https://github.com/spinal-cord-7t/coil-qc-code/pull/9 is merged and the data are uploaded on OpenNeuro, I will write a prototype notebook script that does "most" of the processing tasks. Then we can decide if having all the processing done on a notebook (option 1) is too cumbersome/slow.


Change GCLOUD_TESTS_KEY to "./key.json" in Travis

Should change GCLOUD_TESTS_KEY environment variable to "./key.json" in Travis.

We really don't need this after #505.


Predict test data locally

See https://github.com/Azure/MachineLearningNotebooks/issues/656

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

ID: 89cb4f9c-34e0-b4df-f911-e083c420cee8
Version Independent ID: 5be8b3bb-fb17-175a-7ca0-d69cef414799
Content: Image classification tutorial: Deploy models - Azure Machine Learning
Content Source: articles/machine-learning/service/tutorial-deploy-models-with-aml.md
Service: machine-learning
Sub-service: core
GitHub Login: @sdgilley
Microsoft Alias: sgilley

@raybellwaves
Thanks for the feedback! We are currently investigating and will update you shortly.

@raybellwaves
Thanks for reaching out. Azure Notebook repo is the right place for this issue. Please keep eyes on it. I will close this since it's a duplicate of that one: https://github.com/Azure/MachineLearningNotebooks/issues/656
We will now proceed to close this thread. If there are further questions regarding this matter, please respond here and @YutongTie-MSFT and we will gladly continue the discussion.


[Others]:  a list of references appears in the "Others" page

Specify the model/test case you evaluated

[X] Model
[ ] Testcase

Name the model/test case ID, you would like to evaluate
0000
What is the impact to the model or testcase?

[ ] minor
[X] moderate
[ ] major

Briefly describe your opinion about the model/testcase
A list of references appears (some are "dummy" references I guess).
To be removed

The issue is fixed and merged to main


Fix typo in 20.1 release notes

This change is

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/0aeeb6ee2127e2b33866255ca8afe2fc95a78d76/
Edited pages:

releases/v20.1.0.md


MD scraper broken

Sever returns:
{"status":"Failed","error":{"message":"Failed while trying to get item information","code":400,"request":"https://www.arcgis.com/sharing/rest/content/items/ca77764e722c442986ef6514da88411c?f=json","response":"Item does not exist or is inaccessible.","timestamp":"2020-03-16T23:11:04.078Z"},"generating":{}}

@SteveWallace can you look at this?

Something went wrong with a merge, that's LA not MD. The MD code from my PR last night is gone

@SteveWallace oh no! I may have messed it up.

no biggie, I'm about to delete and refork to fix a bunch of issues with the submodules. After that I'll submit a PR with the proper LA  state code and the MD code

#93


Code has a compiler error but not being reported

The handler.SendJSON function does not exist, yet no compiler error is reported.
package main

import (
    "log"
    "net/http"
)

func main() {
    Routes()

    log.Println("listener : Started : Listening on :4000")
    http.ListenAndServe(":4000", nil)
}

// Routes sets the routes for the web service.
func Routes() {
    http.HandleFunc("/sendjson", handler.SendJSON)
}

Can you please include a zip of the project and/or the output with -v. Could this be #46 ?

DEBUG project root "/Users/bill/Spaces/Go/Projects/src/github.com/goinaction/code"
DEBUG matchPackages: [/Users/bill/Spaces/Go/Projects/src/github.com/goinaction/code/src] ../chapter9/listing04/...
DEBUG args: []
DEBUG removing work directory: /var/folders/8q/d2pfdk_x4qd4__l6gypvzsw40000gn/T/gb705588546
DEBUG build duration: 54.501µs map[]

[ ~/Spaces/Go/Projects/src/github.com/goinaction/code/chapter9/listing04/ ] #

Ah, interesting. there is that silly src folder hanging out there that I abandoned. Let me move the code.

Sorry, it is working when the code is moved out
$ gb build
test
FATAL command "build" failed: [/Users/bill/go/pkg/tool/darwin_amd64/6g -p test -pack -o /var/folders/8q/d2pfdk_x4qd4__l6gypvzsw40000gn/T/gb913145059/test/main.a -I /var/folders/8q/d2pfdk_x4qd4__l6gypvzsw40000gn/T/gb913145059 -I /Users/bill/Spaces/Go/Projects/pkg/darwin/amd64 -complete listing04.go]: exit status 1
listing04.go:19: undefined: handle

Thanks for confirming. I'll have to mark this one up to an unfortunate outcome of project root auto detection.


refactor: replace namedSlotChangeHandlerAspect with NamedSlotStateController functionality

With namedSlotChangeHandlerAspect we had a functionality that checked for the existence of slotted elements and we rendered slots depending on that state. This is at odds with the server side rendering functionality from Lit. Additionally, the performance gain assumed from less HTML elements is minimal in comparison to the additional DOM operations needed.
This PR replaces the namedSlotChangeHandlerAspect with the NamedSlotStateController. This is a fundamental change, which instead of rendering slots depending on usage, always renders them, but adds a data-slot-names attributes with a list of slot names (and unnamed for the unnamed slot) space-separated.
This allows writing the necessary rules in purely CSS, which is from rendering perspective more performant.

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

:exclamation: No coverage uploaded for pull request base (main@87d0e68). Click here to learn what that means.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2290   +/-   ##
=======================================
  Coverage        ?   95.95%
=======================================
  Files           ?      334
  Lines           ?    38742
  Branches        ?     2215
=======================================
  Hits            ?    37174
  Misses          ?     1526
  Partials        ?       42

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

TODO:
Check if everything is fine after the Teaser #2211 has been merged

TODO:

Test the SSR of the dialog component


Rename ephemeralKeypair to operationalKeypair

Problem

Wherever ephemeralKeypair was used in a method name,
it was actually the operational key pair. A keypair may
be ephemeral at the caller, but it is not within
the fabric table.
This terminology caused semantic misunderstanding of the
usage of the operational keys in chip-tool

Issue #13501
Change overview

This PR simply renames the necessary fields and methods, with
no functional changes. The semantics intended are now respected.

Testing

Testing done: unit and cert tests still pass

/rebase

Fast tracking a straight rename.

/rebase


xarchiver: 0.5.4 -> <IP_ADDRESS>

Motivation for this change
Update xarchiver to the last version
Things done

[X] Tested using sandboxing
(nix.useChroot on NixOS,
or option build-use-chroot in nix.conf
on non-NixOS)
Built on platform(s)

[X] NixOS
[ ] OS X
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[X] Fits CONTRIBUTING.md.

:+1: This is the first PR on the new Travis infrastructure and everything seems to be working nicely.

:100: tested locally, compiles, is <IP_ADDRESS> and created a tar.lzma successfully.

This is amazing, thanks for your awesome work @matthewbauer.
Thanks a lot @grahamc for your feedback.


💂♀ SPIKE: Sentry TLS version

"Hi Ministry of Justice,
As you may have seen, Sentry is planning to retire support for TLS 1.0 and 1.1.
Our records show that your organization has some projects using TLS 1.0.
We encourage you to upgrade some of your systems to support TLS 1.2+. We know that in some situations, it may not be possible to switch to newer TLS versions, so alternatively, we ask that you update your Sentry SDK configuration to allow TLS 1.0 usage.
If you are using the domain oXXX.ingest.sentry.io in your Sentry SDK configuration, then you can update it to oXXX.insecure.sentry.io to continue using TLS 1.0.
If you have any questions, please feel free to reply directly to this email.
The Sentry Team"

Slack discussion here

Spike completed: our systems support TLS 1.2+ so no further action is required.


add redisson

Pull Request Test Coverage Report for Build 110

131 of 4370 (3.0%)  changed or added relevant lines in 45 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.02%) to 3.055%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

opentracing-redis-redisson/src/main/java/io/opentracing/contrib/redis/redisson/TracingRReadWriteLock.java
0
6
0.0%

opentracing-redis-redisson/src/main/java/io/opentracing/contrib/redis/redisson/TracingRedissonHelper.java
20
26
76.92%

opentracing-redis-redisson/src/main/java/io/opentracing/contrib/redis/redisson/TracingRCollectionMapReduce.java
0
7
0.0%

opentracing-redis-redisson/src/main/java/io/opentracing/contrib/redis/redisson/TracingRMapReduce.java
0
7
0.0%

opentracing-redis-redisson/src/main/java/io/opentracing/contrib/redis/redisson/TracingRPriorityBlockingDeque.java
0
8
0.0%

opentracing-redis-redisson/src/main/java/io/opentracing/contrib/redis/redisson/TracingRPriorityBlockingQueue.java
0
8
0.0%

opentracing-redis-redisson/src/main/java/io/opentracing/contrib/redis/redisson/TracingRListMultimap.java
8
18
44.44%

opentracing-redis-redisson/src/main/java/io/opentracing/contrib/redis/redisson/TracingRSetMultimap.java
8
20
40.0%

opentracing-redis-redisson/src/main/java/io/opentracing/contrib/redis/redisson/CompletableRFuture.java
3
18
16.67%

opentracing-redis-redisson/src/main/java/io/opentracing/contrib/redis/redisson/TracingRBoundedBlockingQueue.java
0
15
0.0%

Totals

Change from base Build 108:
-0.02%

Covered Lines:
663

Relevant Lines:
21702

💛  - Coveralls


Exception deriving fixes

See https://github.com/prolic/fpp/pull/85#discussion_r230636647 regarding the is_a usage please

@gquemener merged, but...
is_a is deprecated by PHP, so there is no reason at all to use a deprecated function. Maybe you are looking for is_subclass_of?

Just read your other comment, sorry, my bad :)

No worry,
Undeprecated functions are quite unusual to be fair!


markPoints set zlevel

Version
5.1.2
Steps to reproduce
markPoint: { symbol: 'emptyCircle', symbolSize: 10, position: 'top', label: { show: false }, data: [{ type: 'max' }, { type: 'min' }], },
What is expected?
The points are displayed on the polyline.
What is actually happening?
The point is obscured by a polyline

https://user-images.githubusercontent.com/43264755/128654272-ccdfd575-b5b3-4d4e-817b-b03f2c403935.png

https://codesandbox.io/s/immutable-cookies-qf4sg?file=/src/index.js
have a look, when I set the  zlevel of the line, the point was under the line.
I want  the point always above the line.

How about this one ?

Still no news with this? I'm also facing this problem

see also #19108


Multithreaded dynamic compilation type resolver

Prereqs:

[x] https://github.com/ppy/osu-framework/pull/4390

Intentionally blocked here?

Wanted to go through and make sure there wasn't any thread races, since I already solved a few. I think it should be fine.


Creating and then immediately deleting context and Aeron object crashes the JVM

This code:
import uk.co.real_logic.aeron.Aeron;
import uk.co.real_logic.aeron.Aeron.Context;

public class CreateDeleteTest
{
    public static void main(final String[] args)
    {
        final Context ctx = new Aeron.Context();
        final Aeron aeron = Aeron.connect(ctx);

        aeron.close();
        ctx.close();
        System.out.println("Done!");
    }
}

Crashes the JVM for me, on my Mac, in different places; the hs_err report for one example crash includes:
# Problematic frame:
# Done!
V  [libjvm.dylib+0x54a3e8]  Unsafe_GetLongVolatile+0x3e

and the interesting thread in the jstack output for the corresponding core file is:
Thread 21763: (state = IN_VM)
 - sun.misc.Unsafe.getLongVolatile(java.lang.Object, long) @bci=0 (Interpreted frame)
 - uk.co.real_logic.agrona.concurrent.UnsafeBuffer.getLongVolatile(int) @bci=21, line=347 (Interpreted frame)
 - uk.co.real_logic.agrona.concurrent.broadcast.BroadcastReceiver.receiveNext() @bci=12, line=149 (Interpreted frame)
 - uk.co.real_logic.agrona.concurrent.broadcast.CopyBroadcastReceiver.receive(uk.co.real_logic.agrona.concurrent.MessageHandler) @bci=14, line=62 (Interpreted frame)
 - uk.co.real_logic.aeron.DriverListenerAdapter.receiveMessages(long) @bci=10, line=56 (Interpreted frame)
 - uk.co.real_logic.aeron.ClientConductor.doWork() @bci=18, line=104 (Interpreted frame)
 - uk.co.real_logic.agrona.concurrent.AgentRunner.run() @bci=42, line=83 (Interpreted frame)
 - java.lang.Thread.run() @bci=11, line=745 (Interpreted frame)

It doesn't appear to really matter if a Media Driver is running or not; it crashes either way.  The original version of the test launched an embedded media driver and tried to close it as well, but that's not necessary to reproduce the crash.

This looks like a race in creating the conductor thread shutting it down again. I'll need to update Agrona to protect against this.

Fixed by https://github.com/real-logic/Agrona/commit/a3affdac9b45ed4e8125770cf7556272adbaedf8


'animate({update: false})' is invalid.

4.1.1:
windows 10:
**https://codepen.io/wyhleonard/pen/rNMWExJ**:

Have a try with G2 5.0.


rmqt::ErrorCallback is not invoked

Describe the bug
I'm trying to utilize the Error Callback functionality to handle the cases when connection to RabbitMQ is broken:
void onError(const bsl::string& errorText, int errorCode)
{
    bsl::cout << "Error: " << errorText << "\tErrorCode: " << errorCode << "\n";
}

rmqt::ErrorCallback errorCb = &onError;
rmqa::RabbitContextOptions rabbitOptions;
rabbitOptions.setErrorCallback(errorCb);
rmqa::RabbitContext rabbit(rabbitOptions);

However, the onError() function is never called.
To Reproduce

Stop the RabbitMQ broker
Run the librmq_multithread_producer.m.cpp example
Note that the onError() function is never called

Expected behavior
The onError() function should be called when the connection to RabbitMQ is broken

I've figured that out. To make this work, you need to set the ConnectionErrorThreshold property as well:
rabbitOptions.setConnectionErrorThreshold(10s);
I think this should be reflected in the documentation.


Separate language-sqf.json into multiple files

http://spacetelescope.github.io/understanding-json-schema/structuring.html

Its easy, look at #21.

https://github.com/acemod/language-arma-atom/pull/21#issuecomment-135575114

Done.


In v1.12.6 and later .GetAttribute("value") fails to return the correct value if it contains "{"

After upgrading from v1.11.7 -> v1.12.6, .GetAttribute("value") fails to return the value if it contains "{" (my use case here is that I'm displaying json 'log' data in a text area)
Simplified (not real json, no binding, etc):
<textarea value="@("{ \"abc\" }")"></textarea>

The test:
Assert.Equal(
    expected: "{ \"abc\" }",
    actual: cut.Find("textarea").GetAttribute("value")
);

In v1.12.6 and later the test fails with value =  "{\n  "
In all versions, printing the output in the immediate:  cut.Find("textarea") outputs:
OuterHtml: "<textarea id=\"myText\" value=\"{ \" abc\"=\"\" }\"=\"\"></textarea>"

Hey @brettwinters,
that is expected behavior. Prior to 1.12.6 we did encode attribute-values which was not correct. Blazor does not do so, so we shouldn't do either.
For your specific use-case you should do something like that:
<textarea>@myField</textarea>

<textarea> does not have a value attribute. I guess your browser is just defensive in that situation. So in this case: Two stones one bird ;)

Hi @brettwinters,
Regarding your first question: bUnit as well as the underlying Blazor renderer does not really care about valid HTML or not. Of course, you are allowed to use the value attribute in any shape or form. The change we did in 1.12 was to be aligned to the Blazor renderer itself. We have our representation of what the Blazor compiler does to create HTML from your razor markup. We aligned that behavior.  I am wondering why in your case the actual result contains \n. I have to reproduce that locally. I am suspecting it has something to do with the expression syntax @().
To your second question. Well it is a bit awkward to see that they use textarea like that and I will open a ticket on the aspnet GitHub tracker. And it makes sense that you see the same result, as the underlying InputTextArea get transformed to your initial example. GetAttribute is from AngleSharp that we use under the hood. Basically, we transform the renderer HTML into a C# object representation via AngleSharp. To give you a more detailed explanation I have to make some local experiements.

Yeah in the browser. But it gets weirder from there as even between the tags there is no content:
<textarea _bl_4b9cdf47-b92a-4204-bbb3-473daec5430f=""></textarea>

But I guess that is a different topic.
I try to reproduce your initial @() statement. I guess for now let's assume that the value attribute is fine (even if it isn't)

Yeah, I checked the rendered output from an InputTextArea and noticed the same too.
document.getElementById("textarea")

Shows nothing between the brackets
document.getElementById("txtProposalDescription").getAttribute('value')

outputs null
document.getElementById("txtProposalDescription").value

outputs the correct value

Okay I could reproduce this and I am currently figuring out what we can do here. In the end, the whole transformation is done by AngleSharp.  And AngleSharp sees your example as 5 key-value pairs instead of 1. I'll investigate a bit later


fix(ui): constrain wallet name to 30 chars max

Description:

maxLength and minLength input validators
constrain wallet name to 30 chars max

Motivation and Context:
Fix #1529
How Has This Been Tested?
Manually
Types of changes:
Enhancement
Checklist:

[x] My code follows the code style of this project.
[x] I have reviewed and updated the documentation accordingly.
[ x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes where needed.
[ ] All new and existing tests passed.
[x] My commits have been squashed into a concise set of changes.

Coverage increased (+0.02%) to 20.199% when pulling 58d123db92453cf435debbad19a4e45a16bcc8e5 on mrfelton:fix/1529-wallet-name-maxlength into 3bf48855fc950f74e2f9a631f6193ca8dc797ea0 on LN-Zap:next.

Coverage increased (+0.02%) to 20.199% when pulling 58d123db92453cf435debbad19a4e45a16bcc8e5 on mrfelton:fix/1529-wallet-name-maxlength into 3bf48855fc950f74e2f9a631f6193ca8dc797ea0 on LN-Zap:next.


Unity: Never ignore asset meta data

Reasons for making this change:
If I were to use the Unity.gitignore (for example as part of a project) and the VisualStudio.gitignore (for example as my global gitignore) at the same time, I could run into problems, because the VS one ignores *.meta and Unity relies heavily on these.
This change just makes sure that the *.meta files in the Assets directory are always added to the version control.
I can see that the necessity of this is debatable, but due to the fact that Visual Studio is the de facto default for Unity under Windows, I think this fix is reasonable.
Links to documentation supporting these rule changes:
https://docs.unity3d.com/Manual/ExternalVersionControlSystemSupport.html

@AdamKane if you believe this should be reverted, please open a new PR and provide context for the change


fix: np-opto correction

Closes #59
And pre-calculates correct NP-opto locations

@alejoe91 I merged in a github action that checks that the version number is being incremented. It's not perfect, but we have to manually change the version number in transfer/init.py on every pull request. (The current version in main is 0.0.4, so it needs to be bumped to 0.0.5)

Ok, is it necessary to bump up at every PR? Then is the package automatically deployed on pypi?

@alejoe91 I merged in a github action that checks that the version number is being incremented. It's not perfect, but we have to manually change the version number in transfer/init.py on every pull request. (The current version in main is 0.0.4, so it needs to be bumped to 0.0.5)

done

Ok, is it necessary to bump up at every PR? Then is the package automatically deployed on pypi?

I'm gonna add the publishing to pypi step next week.


Impossible to highlights two portions of text starting at the same offset

I have an annotation system and a search engine using the annotation module to highlight contents.
So I want to be able to highlight some portion of text twice.
My problem is that if I highlight two CFI with the same start offset, i got an error.
For example :
Annotated text : /4/2/8,/1:4,/1:30
Search result   : /4/2/8,/1:4,/1:17
On the second call i got the following error :
"Selection start and end must be text nodes" annotations_module.js (619)
The annotation module should ignore the cfi marker node.

It sounds like the CFI "blacklist" is not taken into account when making a selection, if so, definitely a bug.

Adding the following code here resolve my problem :
        while (startNode.nextSibling && startNode.classList && (startNode.classList.contains("cfi-marker") || startNode.classList.contains("mo-cfi-highlight"))) {
            startNode = startNode.nextSibling;
        }

        while (endNode.previousSibling && endNode.classList && (endNode.classList.contains("cfi-marker") || endNode.classList.contains("mo-cfi-highlight"))) {
            endNode = endNode.previousSibling;
        }

@JCCR would it make sense to migrate this "selection-to-cfi" code from annotation_module to the CFI library, as a common utility? (whatever helps harmonizing the blacklisting API)

Follow-up to my above comment: https://github.com/readium/readium-shared-js/issues/201


[Reference] vsock NFS support

NFS via vsock is under developing, and is promising for the future FS sharing of hyper, because:

NFS has much better performance/feature set than 9p
vsock doesn't need to add a network interface to guest, and then doesn't setup a network connection between host and guest, i.e. better isolation than open a network connection to every guest.
and it may also substitute the virt-serial for communication

Some blocker of adopting vsock NFS

the tool set is under developing:
the xen doesn't support vsock yet, i mean xenbus based transport, once it is supported xenbus, we can use it to provide FS sharing in PV mode.

Some references:

The initial patch set: https://lwn.net/Articles/647516/
Slides: http://events.linuxfoundation.org/sites/events/files/slides/stefanha-kvm-forum-2015.pdf
qemu wiki: http://qemu-project.org/Features/VirtioVsock

Related repos:

kernel, vsock-nfs: https://github.com/stefanha/linux/tree/vsock-nfs
user space tools: https://github.com/stefanha/nfs-utils
qemu vsock support: https://github.com/stefanha/qemu/tree/vsock

update the xen vsock support status
design in 2013:

mailing list: http://lists.xenproject.org/archives/html/xen-devel/2013-06/msg01123.html
pdf: http://xenbits.xen.org/people/dvrabel/inter-domain-comms-C.pdf

update: manila project will adopt vsock-nfs to introduce cephfs to guest

A update slides: http://vmsplice.net/~stefan/stefanha-connectathon-2016.pdf

update: nfs-ganesha support for vsock:
https://github.com/nfs-ganesha/nfs-ganesha/commit/c2d6ca82f61c4014daca96df63b251a67a3b0488
It is queued for nfs-genasha v2.4 that is still under development right now.


Adds error when attempting to refund a charge that has already been refunded / refunding more than the charge amount

Reverts stripe-ruby-mock/stripe-ruby-mock#888
Readds changes to throw error when attempting to refund a charge for more than has been charged already or more than the original charge amount, as per Stripe API https://stripe.com/docs/api/refunds/create

@jakebills1 can you fix tests for this PR?


Refactoring and Improving Readability

Since and condition is there, verify is not True will be always false.

Hey @tusharmakkar08,
Thanks for sending this pull request. Could you explain what bug you were seeing that made you change this? What instigated this change? It's not clear that this exhibits the same behaviour it did before, especially since tests have failed pretty consistently across all versions of Python.

@sigmavirus24 : In line 174 if url.lower().startswith('https') and verify  is there and then if not verify is there. Since and condition is there, if not verify won't be True.

@tusharmakkar08 your commit no longer affects that region and simply turns an easy to read if/else statement into a ternary which is not "pythonic" nor is it easier to read.
I'll ask again: What bug are you seeing that prompted this (if there was any bug)?

There was no bug per se.

Fewer lines does not lead to an increase in readability. In this instance, the ternary conditional is particularly unclear when combined with tuple unpacking assignment as well as creating an intermediate tuple in many cases.
In this instance, I suggest that the only clear readability improvement would come from using tuple unpacking in the first branch of the conditional. I don't think any change beyond that is warranted.

We're not interested in random refactorings that change logic for no apparent reason. Sorry @tusharmakkar08

@sigmavirus24 @Lukasa : I have done couple of more changes and few of them are anti-patterns as suggested by http://docs.quantifiedcode.com/python-code-patterns/ . You can review the changes here and let me know if I should open the pull request again.
Thanks.

@tusharmakkar08 while your contributions are appreciated, I want to give you feedback — all of the changes present in this pull request actually reduce readability, significantly.

@kennethreitz: Since requests is being mentioned over here among the best python codes, I believe these kind of antipatterns shouldn't exist over here.

@tusharmakkar08 please keep in mind that your continuous arguments are emailing roughly 841 people. Since it doesn't seem as if you will respectfully let this conversation end and it serves to benefit none of those 841 people (because it has little value to the project as a whole and is likely not why they are subscribed) I am ending the conversation forcefully by locking it.

Antipatterns are an antipattern. :)