Remove external routing overrides - DO NOT MERGE YET

This is a work in progress, as it would be good to make the necessary changes in affected applications prior to merging this.

I think most of the application changes to remove this configuration have now been written, but most haven't been merged yet [1].
1: https://github.com/pulls?utf8=✓&q=is%3Apr+org%3Aalphagov+author%3Acbaines+"external+URL"

Right, 20ish Pull Requests down, just 2 more to go I think before this can be merged:

https://github.com/alphagov/manuals-publisher/pull/1281
https://github.com/alphagov/whitehall/pull/3760

I think this is ready to go now, I'm planning to merge this after I've had some lunch.


Reading null from xx_RESOURCES.DATA causes NullPointerException

Describe the bug
Certain databases used for development or testing may contain a null data column in the xx_RESOURCES tables. Trying to read a resource in this state will cause a NullPointerException to be thrown.
This causes issues with reindex because it prevents the reindex from continuing past the broken resource record.
This is extremely unlikely to occur in user databases. Although the data column is nullable, it is always set to a value by the persistence layer. The data column can only be null if it has been updated outside of the IBM FHIR Server, or as part of some feature development activity.
To Reproduce
Attempt to run reindex on a database which contains null values for the xx_RESOURCES data column.
Expected behavior
For reindex, the record should be skipped, because there's no resource payload to process.
For other read operations, generate an appropriate operational outcome.

Hit while working on experiment "off-loading of resource payload to external storage"

Here's the updated behavior:

Read/vRead:  Resource xxxxxxxxxxxx not found.'
Reindex: No search parameters are extracted.
Search: IllegalStateException: Returned resources must have an id.  (currently does this if resource.id == null, so just expanded it to also cover resource == null).

I added a 2nd commit with the following update:
Instead of failing the whole search, return the bundle and include an OperationOutcome with "A resource with no data was found." or "A resource with no id was found".  So the whole search doesn't just fail if there's one nulled-out resource in the DB.
I can backout that commit if just failing the whole search is better.

Ran local tests with search and $reindex. now seems to be working as expected. Closing issue.


ng update conflict with pc user folder name

🐞 Bug report
Command (mark with an x)

[ ] new
[ ] build
[ ] serve
[ ] test
[ ] e2e
[ ] generate
[ ] add
[x] update
[ ] lint
[ ] xi18n
[ ] run
[ ] config
[ ] help
[ ] version
[ ] doc

Description
unfortunately my pc user name is: s&a and because npm and node have problem with this user name i moved my npm and npm-cache folder to c:\npmGlobal and install node on my D drive and save path to my system variable and edit npmrpc file but when i try to use command ng update i got this error:
PS D:\Projects\Angular\MofidBot\MofidBot\ClientApp> ng update
Your global Angular CLI version (9.1.7) is greater than your local
version (8.3.14). The local Angular CLI version is used.

To disable this warning use "ng config -g cli.warnings.versionMismatch false".
The installed Angular CLI version is older than the latest published version.
Installing a temporary version to perform the update.
Installing packages for tooling via npm.
Error: EPERM: operation not permitted, mkdir 'C:\Users\s'
The system cannot find the path specified.
An unhandled exception occurred: Package install failed, see above.
See "C:\Users\s&a\AppData\Local\Temp\ng-lOKcYV\angular-errors.log" for further details.

🌍 Your Environment
     _                      _                 ____ _     ___
    / \   _ __   __ _ _   _| | __ _ _ __     / ___| |   |_ _|
   / △ \ | '_ \ / _` | | | | |/ _` | '__|   | |   | |    | |
  / ___ \| | | | (_| | |_| | | (_| | |      | |___| |___ | |
 /_/   \_\_| |_|\__, |\__,_|_|\__,_|_|       \____|_____|___|
                |___/

Angular CLI: 8.3.14
Node: 14.3.0
OS: win32 x64
Angular: 8.2.12
... animations, common, compiler, compiler-cli, core, forms
... language-service, platform-browser, platform-browser-dynamic
... platform-server, router

Package                                    Version
--------------------------------------------------------------------
@angular-devkit/architect                  0.803.24
@angular-devkit/build-angular              0.803.24
@angular-devkit/build-optimizer            0.803.24
@angular-devkit/build-webpack              0.803.24
@angular-devkit/core                       8.3.24
@angular-devkit/schematics                 8.3.14
@angular/cli                               8.3.14
@ngtools/webpack                           8.3.24
@nguniversal/module-map-ngfactory-loader   8.1.1
@schematics/angular                        8.3.14
@schematics/update                         0.803.14
rxjs                                       6.5.4
typescript                                 3.5.3
webpack                                    4.39.2

Hi @alan-agius4, could you point me in the right direction where/ what the problem is. I can have a look at this issue.

Likely it's caused here https://github.com/angular/angular-cli/blob/1873f2d28f8f508c1f360e07b03b60ff20c20d25/packages/angular/cli/tasks/install-package.ts#L146-L148 by unescaped paths.

Just an update, I've not have found the time to research this.

Just an update, I've not have found the time to research this.

is this problem happens with just update or happens with other commands?

update only

I think this node/issue#7367 should be helpful to fix this issue.


Please invite me to the community

Name
Lenny Bayer
Additional Context
I hope to join to this org

Invitation sent to join the GitHub Organization. Welcome to Infraform 🎉Please check your mail to accept the invitation sent by GitHub! 🤖About usInfraform is an open source solution for serverless platforms, it is designed for developing solutions and enhancing creators to create their infrastructure 💻 Great to have you in our community ✨ Looking forward to your valuable contributions! 🤓 You can also check our community ✨ Github ✨ LinkedIn 💜 You can also share our community with other folks who want to learn about development, DevOps, tech, and contribute together and with us 👉💜👈


Provide type conversion between protobuf well known types and rust types

All of types in wrappers.rs can be mapped with Rust types without any loss. Thus we can implement From between them.
For example:
impl From<String> for StringValue { ... }
// And back
impl From<StringValue> for <String> { ... }

And also, it seems that this change will not break backward compatibility.

Sounds good. Patch welcome!

Ok, I will send it later.


chore(deps): update module github.com/terraform-ibm-modules/ibmcloud-terratest-wrapper to v1.2.17

This PR contains the following updates:

Package
Type
Update
Change

github.com/terraform-ibm-modules/ibmcloud-terratest-wrapper
require
patch
v1.2.16 -> v1.2.17

Release Notes

terraform-ibm-modules/ibmcloud-terratest-wrapper
v1.2.17
Compare Source
Bug Fixes

deps: update gomod (#232) (5dc2397)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, click this checkbox.

This PR has been generated by Renovate Bot.

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


DropperSelectedRow never called

i followed the instructions to the word and it didn't work
then i tried some other variations and it didn't work as well
no need for code because i'm doing what you wrote
the framework looks great! i really hope to use it
thanks :)

Hey! I created a new project and followed my instructions again, but I wasn't able to replicate the issue you are having.
Here is my View controller:

class ViewController: UIViewController {
    let dropper = Dropper(width: 75, height: 200)
    @IBOutlet weak var dropButton: UIButton!

    override func viewDidLoad() {
        super.viewDidLoad()
    }

    override func didReceiveMemoryWarning() {
        super.didReceiveMemoryWarning()
        // Dispose of any resources that can be recreated.
    }

    @IBAction func buttonSelected() {
        if dropper.status == .hidden {
            dropper.items = ["Item 1", "Item 2", "Item 3", "Item 4"] // Item displayed
            dropper.theme = Dropper.Themes.white
            dropper.delegate = self
            dropper.cornerRadius = 3
            dropper.showWithAnimation(0.15, options: Dropper.Alignment.center, button: dropButton)
        } else {
            dropper.hideWithAnimation(0.1)
        }
    }
}

extension ViewController: DropperDelegate {
    func DropperSelectedRow(_ path: IndexPath, contents: String) {
        print(path)
        print(contents)
    }
}


Don't upgrade pip

Appears to be a bug in pip 10.0.0
Not fixed in pip 10.0.1

Coverage remained the same at 92.439% when pulling 8a2b92c0d7926006bea3d8cd7c243b33950490f5 on bellert:fix-appveyor-pip into 710e5c88a9c8946211343fa48b09fdfd75b4df19 on dwavesystems:master.


Lookup::fillJavaMethodInfo may cause crash on JDK 8/11/17

Describe the bug
Hi, Andrei
We recently encountered a crash. After analysis, I think this problem may occur on OpenJDK 8, 11, and 17. It will not occur on OpenJDK 18 and above because JDK-8268364 avoids this problem at the JDK level.

JDK 8

#
# A fatal error has been detected by the Java Runtime Environment:
#
#  SIGSEGV (0xb) at pc=0x00007fa5e1ef9108, pid=12810, tid=0x00007fa5234f4700
#
# JRE version: OpenJDK Runtime Environment (8.0_412-b08) (build 1.8.0_412-b08)
# Java VM: OpenJDK 64-Bit Server VM (25.412-b08 mixed mode linux-amd64 compressed oops)
# Problematic frame:

V  [libjvm.so+0x766108]  oopDesc::is_a(Klass*) const+0x48
V  [libjvm.so+0x7635ec]  jvmti_GetClassSignature+0x15c
C  [libasyncProfiler.so+0x35389]  Recording::finishChunk()+0x20c9
C  [libasyncProfiler.so+0x3599c]  FlightRecorder::stop()+0x9c
C  [libasyncProfiler.so+0x39559]  Profiler::stop(bool)+0x159
C  [libasyncProfiler.so+0x3b4af]  Profiler::runInternal(Arguments&, Writer&) [clone .constprop.772]+0x32f
C  [libasyncProfiler.so+0x3b6d1]  Profiler::run(Arguments&)+0x181
C  [libasyncProfiler.so+0x3d019]  Agent_OnAttach+0x209
V  [libjvm.so+0x7c12d9]  JvmtiExport::load_agent_library(AttachOperation*, outputStream*)+0x329
V  [libjvm.so+0x3328df]  attach_listener_thread_entry(JavaThread*, Thread*)+0x15f
V  [libjvm.so+0xb11d32]  JavaThread::thread_main_inner()+0x212
V  [libjvm.so+0x942a62]  java_start(Thread*)+0xf2
C  [libpthread.so.0+0x7ea5]  start_thread+0xc5

JDK 11

#
# A fatal error has been detected by the Java Runtime Environment:
#
#  SIGSEGV (0xb) at pc=0x00007f73c96aa658, pid=31632, tid=31752
#
# JRE version: OpenJDK Runtime Environment (Red_Hat-<IP_ADDRESS>.9-2.el7_9) (11.0.23+9) (build 11.0.23+9-LTS)
# Java VM: OpenJDK 64-Bit Server VM (Red_Hat-<IP_ADDRESS>.9-2.el7_9) (11.0.23+9-LTS, mixed mode, sharing, tiered, compressed oops, concurrent mark sweep gc, linux-amd64)
# Problematic frame:
V  [libjvm.so+0x9be658]  oopDesc::is_a(Klass*) const+0x48
V  [libjvm.so+0xa04153]  jvmti_GetClassSignature+0x173
C  [libasyncProfiler2540885687583512417.so+0x72b6]  _jvmtiEnv::GetClassSignature(_jclass*, char**, char**)+0x38
C  [libasyncProfiler2540885687583512417.so+0x131f3]  Lookup::fillJavaMethodInfo(MethodInfo*, _jmethodID*, bool, int)+0x7b
C  [libasyncProfiler2540885687583512417.so+0x135cb]  Lookup::resolveMethod(ASGCT_CallFrame&, int)+0x11b
C  [libasyncProfiler2540885687583512417.so+0x16afb]  Recording::writeStackTraces(Buffer*, Lookup*)+0x147
C  [libasyncProfiler2540885687583512417.so+0x16381]  Recording::writeCpool(Buffer*)+0x19d
C  [libasyncProfiler2540885687583512417.so+0x141d3]  Recording::finishChunk(bool)+0xdf

Expected vs. actual behavior

Expected: not crash
Actual: It will definitely crash in certain scenarios as described bellow

Reproduction Steps
I can't provide a simple reproducer, but I can describe the reproduction steps clearly (in my environment, this problem can be reproduced very easily)

async-profiler/build/bin/asprof -e cpu -f profile.jfr -d 30
Use a custom ClassLoader C and load a class A
Invoke the Method M of Class A
async-profiler records CPU sample，Method M is on the stack, and the jmethodId of M is recorded.
ClassLoader C and class A become dead objects
GC occurs, and ClassLoader C is unloaded (including Class A). On OpenJDK 8, 11, and 17, Method::clear_jmethod_ids is not called synchronously when unloading, but will be called at the next SafepointSynchronize::begin, resulting in a delay.
async-profiler begin to dump JFR, before next SafepointSynchronize::begin
async-profiler calls jvmti->GetMethodDeclaringClass(method, &method_class), which returns normally. This is because the Method* in jmethodId is not set to NULL, Method* and InstanceKlass* have not been purged, and the memory address is valid. Method* points to InstanceKlass*, but the java_mirror oop (that is, method_class) in InstanceKlass has been GCed.
Async-profiler calls jvmti->GetClassSignature(method_class, &class_name, NULL), which may cause a crash. This is because the java_mirror memory corresponding to method_class has been released by GC, and the Klass Word in the object header is invalid.

Additional Information/Context
The info from core dump:
#8  0x00007f654c046ee3 in Klass::is_subtype_of (this=0x6f56df778, k=0x100002000) at /root/openjdk/jdk8u-dev-fork/hotspot/src/share/vm/oops/klass.hpp:438
#9  0x00007f654c04849d in oopDesc::is_a (this=0xaad81430, k=0x100002000) at /root/openjdk/jdk8u-dev-fork/hotspot/src/share/vm/oops/oop.inline.hpp:178
#10 0x00007f654c4d4627 in jvmti_GetClassSignature (env=0x7f6484706758, klass=0x7f64545fb910, signature_ptr=0x7f64bc402e10, generic_ptr=0x0) at ../generated/jvmtifiles/jvmtiEnter.cpp:3242
#11 0x00007f64bd6f2336 in _jvmtiEnv::GetClassSignature (this=0x7f6484706758, klass=0x7f64545fb910, signature_ptr=0x7f64bc402e10, generic_ptr=0x0) at /usr/lib/jvm/java-11-openjdk-<IP_ADDRESS>.9-2.el7_9.x86_64//include/jvmti.h:2242
#12 0x00007f64bd6fe29c in Lookup::fillJavaMethodInfo (this=0x7f64bc402fa0, mi=0x7f64545fb648, method=0x7f646c003120, first_time=true, i=1) at src/flightRecorder.cpp:198

(gdb) x/xg 0x7f646c003120 [fillJavaMethodInfo(....method=0x7f646c003120)]
0x7f646c003120:	0x00007f64bdb54858 [*(Method**)jmethodId is not NULL]

(gdb) x/xg 0x7f64545fb910 （The klass returned from GetMethodDeclaringClass）
0x7f64545fb910:	0x00000000aad81430 (java_mirror oop)

(gdb) x/32xw 0xaad81430 (the memory of java_mirror oop is freed)
0xaad81430:	0xdeadbeef	0xdeadbeef	0xdeadbeef	0xdeadbeef
0xaad81440:	0xdeadbeef	0xdeadbeef	0xdeadbeef	0xdeadbeef
0xaad81450:	0xdeadbeef	0xdeadbeef	0xdeadbeef	0xdeadbeef
0xaad81460:	0xdeadbeef	0xdeadbeef	0xdeadbeef	0xdeadbeef
0xaad81470:	0xdeadbeef	0xdeadbeef	0xdeadbeef	0xdeadbeef
0xaad81480:	0xdeadbeef	0xdeadbeef	0xdeadbeef	0xdeadbeef
0xaad81490:	0xdeadbeef	0xdeadbeef	0xdeadbeef	0xdeadbeef

(gdb) p/x 0xdeadbeefl << 3 (from invalid Klass word, get invalid Klass*)
$2 = 0x6f56df778

(gdb) x/xg 0x6f56df778 （Klass* is not accessible）
0x6f56df778:	Cannot access memory at address 0x6f56df778

For comparison, let's look at the normal situation.
#0  Klass::is_subtype_of (this=0x100002000, k=0x100002000) at /root/openjdk/jdk8u-dev-fork/hotspot/src/share/vm/oops/klass.hpp:437
#1  0x00007ffff617049d in oopDesc::is_a (this=0xaae013e0, k=0x100002000) at /root/openjdk/jdk8u-dev-fork/hotspot/src/share/vm/oops/oop.inline.hpp:178
#2  0x00007ffff65fc627 in jvmti_GetClassSignature (env=0x7fff382095c8, klass=0x7ffefc001100, signature_ptr=0x7fff1259de10, generic_ptr=0x0) at ../generated/jvmtifiles/jvmtiEnter.cpp:3242
#3  0x00007fff1388d2b6 in _jvmtiEnv::GetClassSignature (this=0x7fff382095c8, klass=0x7ffefc001100, signature_ptr=0x7fff1259de10, generic_ptr=0x0) at /usr/lib/jvm/java-11-openjdk-<IP_ADDRESS>.9-2.el7_9.x86_64//include/jvmti.h:2242
#4  0x00007fff138991f3 in Lookup::fillJavaMethodInfo (this=0x7fff1259dfa0, mi=0x7ffefc5bef28, method=0x7fff39378e58, first_time=true, i=0) at src/flightRecorder.cpp:198

(gdb) x/xg 0x7fff39378e58 [fillJavaMethodInfo(....method=0x7fff39378e58)]
0x7fff39378e58:	0x00007fffa5ec9670 [*(Method**)jmethodId is not NULL]

(gdb) x/xg 0x7ffefc001100 （The klass returned from GetMethodDeclaringClass）
0x7ffefc001100:	0x00000000aae013e0 (java_mirror oop)

(gdb) x/32xw 0xaae013e0 (valid java_mirror oop)
0xaae013e0:	0xceb1a511	0x0000006a	0x20000400	0x00000000
0xaae013f0:	0x00000000	0xab575050	0x00000000	0xab074f10
0xaae01400:	0xaabae230	0x00000000	0x00000000	0x00000000
0xaae01410:	0x00000000	0x00000000	0x00000000	0x00000000
0xaae01420:	0x00000000	0x00000000	0x0013dcb8	0x00000001
0xaae01430:	0x00000000	0x00000000	0x00000000	0x00000016
0xaae01440:	0x00000006	0x00000000	0xaaba5b18	0xaaad5508
0xaae01450:	0xaab56ca8	0xab075868	0xab04e728	0xaaad0ab8
(gdb) p/x 0x20000400l << 3 (valid Klass*，accessible)
$1 = 0x100002000

There is a check for invalid jmethodId (NULL Method*) in async-profiler,
https://github.com/async-profiler/async-profiler/blob/b8025a24499a690ccd4f5ce909d348d82768bb35/src/flightRecorder.cpp#L175
But it is obviously impossible to avoid the problem described above, because the address in jmethodId is not NULL.
This seems to be a bad design in JDK, which causes a crash when calling JVMTI.
I feel that I can do something in async-profiler workaround it.

One way is to force a Safepoint to be triggered before dump, it needs to rely on WB_ForceSafepoint, but this is a tool for testing scenarios.
Before the next Safepoint, Method*, InstanceKlass* and other more structs are still accessible, so Another way is to avoid using java_mirror and directly read the Class Name from InstanceKlass through offset (similar to the approach used by https://github.com/open-telemetry/opentelemetry-ebpf-profiler/blob/3cede2b41bfc9b1bd272ced4cf528e49b635bcfb/interpreter/hotspot/instance.go#L270).

I don't know if you have a better idea.  I would be grateful if you could help me fix this.
Async-profiler version
latest
Environment details

Linux (any version)
JDK 8/11/17
All CPU architectures
Its not related to container environment

Thank you for the detailed report. If your analysis is correct, this is certainly a JVM bug.
We'll try to create a reproducer and if successful, submit a fix in OpenJDK.

Here are some of the hs_err files：
OpenJDK 11 hs_err_pid31632.log
OpenJDK 8u412 hs_err_pid12810.log
OpenJDK8 slowdebug hs_err_pid15470.log

I can provide a reproducer to save your time. Wait for me for a moment.

I have successfully reproduced it on both OpenJDK 8 and 11 today (see the files below, they are also included in the attachment).
On different machines, you may need to adjust the profiling duration (the only parameter of reproduce-11.sh and reproduce-8.sh) so that the dump occurs after the Class is unloaded and before the next Safepoint.
You need to try more times because there are many variables involved, which is essentially a race condition.
If you see the "Done" and no crash occurred, then need to retry.
If a crash occurs, there will be clear logs on the screen, refer to the examples below.
issue-974-reproducer.tar.gz
-rw-r--r-- 1 root root 161181 Aug 22 11:16 hs_err_pid9934.log
-rw-r--r-- 1 root root 159660 Aug 22 11:42 hs_err_pid18519.log
-rw-r--r-- 1 root root 154176 Aug 22 11:51 hs_err_pid21948.log
-rw-r--r-- 1 root root 157954 Aug 22 11:59 hs_err_pid24809.log
-rw-r--r-- 1 root root  72309 Aug 22 16:48 hs_err_pid31831.log
-rw-r--r-- 1 root root 152441 Aug 22 16:56 hs_err_pid3089.log

OpenJDK 11
bash reproduce-11.sh 20

[2024-08-22T16:48:01.219+0800] GC(2) Concurrent Reset
[2024-08-22T16:48:01.242+0800] GC(2) Concurrent Reset 23.495ms
[2024-08-22T16:48:03.466+0800] GC(5) Concurrent Reset
[2024-08-22T16:48:03.494+0800] GC(5) Concurrent Reset 27.993ms
[2024-08-22T16:48:10.579+0800] unloading class com.example.demo.TempClass 0x00000008403e2040
[2024-08-22T16:48:10.592+0800] GC(15) Concurrent Reset
[2024-08-22T16:48:10.595+0800] GC(15) Concurrent Reset 2.295ms
[2024-08-22T16:48:17.621+0800] unloading class com.example.demo.TempClass 0x0000000840424840
[2024-08-22T16:48:17.621+0800] unloading class com.example.demo.TempClass 0x0000000840425040
[2024-08-22T16:48:17.621+0800] unloading class com.example.demo.TempClass 0x0000000840425840
[2024-08-22T16:48:17.621+0800] unloading class com.example.demo.TempClass 0x0000000840426040
[2024-08-22T16:48:17.621+0800] unloading class com.example.demo.TempClass 0x0000000840426840
[2024-08-22T16:48:17.621+0800] unloading class com.example.demo.TempClass 0x0000000840427040
[2024-08-22T16:48:17.634+0800] GC(16) Concurrent Reset
[2024-08-22T16:48:17.637+0800] GC(16) Concurrent Reset 2.193ms
Done
#  SIGSEGV (0xb) at pc=0x00007f55e89c0658, pid=31831, tid=31924
# V  [libjvm.so+0x9be658]  oopDesc::is_a(Klass*) const+0x48

OpenJDK 8
bash reproduce-8.sh 25
1.107: [CMS-concurrent-reset: 0.034/0.034 secs] [Times: user=0.11 sys=0.04, real=0.03 secs]
8.557: [CMS-concurrent-reset: 0.003/0.003 secs] [Times: user=0.03 sys=0.00, real=0.00 secs]
13.759: [GC (CMS Final Remark) [YG occupancy: 106902 K (111808 K)]13.759: [Rescan (parallel) , 0.0008202 secs]13.760: [weak refs processing, 0.0000369 secs]13.760: [class unloading[Unloading class com.example.demo.TempClass 0x0000000100466028]
13.777: [CMS-concurrent-reset: 0.002/0.002 secs] [Times: user=0.02 sys=0.00, real=0.00 secs]
20.901: [GC (CMS Final Remark) [YG occupancy: 108890 K (111808 K)]20.901: [Rescan (parallel) , 0.0008640 secs]20.902: [weak refs processing, 0.0000437 secs]20.902: [class unloading[Unloading class com.example.demo.TempClass 0x0000000100485028]
[Unloading class com.example.demo.TempClass 0x0000000100484828]
20.917: [CMS-concurrent-reset: 0.002/0.002 secs] [Times: user=0.00 sys=0.00, real=0.00 secs]
27.989: [GC (CMS Final Remark) [YG occupancy: 108890 K (111808 K)]27.989: [Rescan (parallel) , 0.0008756 secs]27.990: [weak refs processing, 0.0000449 secs]27.990: [class unloading[Unloading class com.example.demo.TempClass 0x0000000100484028]
[Unloading class com.example.demo.TempClass 0x0000000100483828]
[Unloading class com.example.demo.TempClass 0x0000000100483028]
[Unloading class com.example.demo.TempClass 0x0000000100482828]
28.005: [CMS-concurrent-reset: 0.002/0.002 secs] [Times: user=0.00 sys=0.00, real=0.00 secs]
Done
#  SIGSEGV (0xb) at pc=0x00007f6b96faa108, pid=24809, tid=0x00007f6b32cc5700
# V  [libjvm.so+0x766108]  oopDesc::is_a(Klass*) const+0x48

is this would happen with --cstack vm?

is this would happen with --cstack vm?

Yes. --cstack vm and AsyncGetCallTrace all get jmethodIds for the methods on the stack. Later on dump time, the jmethodIds are translated to Class and Method names, the crash occurs at this phase.

java have so many problem of profiling that I don't dare to open it in prod.

@krk managed to create a test that reliably reproduces the issue in <1 sec. The test only uses pure JVM TI, no AsyncGetCallTrace or other profiling code. The bad news is that the test crashes all JVM versions, including those having the patch for JDK-8268364. However, it still makes sense to backport the patch, since it helps to avoid crashes with a modified test that calls JNI DeleteLocalRef as soon as jclass is no longer needed.
So, a backport of JDK-8268364 together with DeleteLocalRef change in async-profiler will likely prevent crashes.
In the meantime, I'm investigating other possible workarounds. Forcing a safepoint does not solve the issue, but VMStructs approach looks promising.

Is my analysis correct ?
Why forcing a safepoint does not solve the issue ?

Why forcing a safepoint does not solve the issue ?

What prevents from class unloading after a safepoint?

You are right, other class unloading can happen after the manually triggered safepoint, and we can not prevent this.

The test only uses pure JVM TI, no AsyncGetCallTrace or other profiling code.

does the reproducer created by @krk follow my reproduction steps, or only used contrived jclass (with invalid pointer inside) to call jvmti GetClassSignature ?
If used my steps, I don't know why this issue happens on JDK 18 and above.


Expose membership through node api

@aaronlehmann @diogomonica
Docker changeset: https://github.com/docker/docker/commit/b40d97676428cc2dc9fe730374363796dacc45b5
Signed-off-by: Tonis Tiigi<EMAIL_ADDRESS>

[Current coverage][cc-pull] is 53.61%

Merging [#1004][cc-pull] into [master][cc-base-branch] will decrease coverage by 0.12%

@@             master      #1004   diff @@
==========================================
  Files            75         75
  Lines         11570      11583    +13
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
- Hits           6218       6210     -8
- Misses         4471       4489    +18
- Partials        881        884     +3

[][cc-pull]

Powered by Codecov. Last updated by [be7bd8d...16c988a][cc-compare]
[cc-base-branch]: https://codecov.io/gh/docker/swarmkit/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/docker/swarmkit/compare/be7bd8dec14b9876a6c7380f659a62ea6e86d406...16c988acde19cbebfcf7319fd29e77e7884e186d
[cc-pull]: https://codecov.io/gh/docker/swarmkit/pull/1004?src=pr

LGTM

LGTM
Only thing I'm not sure about is the pattern of calling CertificateRequested, waiting for that channel to close, then calling NodeMembership. It might be cleaner if there was a function that returned a channel that will eventually receive the node membership update. But this channel couldn't be shared between different callers, so there would be resource management issues, and it's probably not worth the complexity. I don't really have a better suggestion.

I think this code in general needs a refactor but, @aaronlehmann isn't technically this the same patter that we use for Ready()?

@aaronlehmann Having a function is also useful for this: https://github.com/docker/docker/commit/b40d97676428cc2dc9fe730374363796dacc45b5#diff-9cb7240401da846ac7c2252759538e9aR66 . If we returned something from the channel we also need to include NodeID there and we already have a method for that.


support for none response type

OpenID spec (https://openid.net/specs/oauth-v2-multiple-response-types-1_0.html#:~:text=None) allows Authorization servers to support none response_type. It should be added in this rubygem too.

it's not openid spec, but oauth one.


Visualize the pseudo-labels

Hello! Thanks for your great work, it is really nice.
I was wondering, is it possible to visualize the job done by the Student to assign pseudo-labels to the images while running the training? My goal is to see the Student improves directly on the images from the dataset! (by example, see the bounding boxes around the objects and notice that the evaluation is better and better when the iterations increase)
Thank you for your help!

My suggestion is to add the visualize_training(a function under GeneralizedRCNN class) to the output of student's prediction output in TwoStagePseudoLabGeneralizedRCNN (rcnn.py).
Here is the reference:
https://github.com/facebookresearch/unbiased-teacher/blob/6977c6f77c812fae4064dc1b3865658c2ed247b1/ubteacher/modeling/meta_arch/rcnn.py#L7
visualize_training function in class GeneralizedRCNN is defined in this link:
https://github.com/facebookresearch/detectron2/blob/9eb0027d795bb9a38098bb05e2ceb273bfc9cf41/detectron2/modeling/meta_arch/rcnn.py#L87
Thanks

Thanks for the quick answer! I tried your idea and it works fine while I am in the "supervised" branch, since the batch contains the ground truth information for each images. But when I am trying to use the visualize_training function in the "unsup_data_weak" branch, it doesn't work since the images in the batch have no annotation..!
Do you have an idea how to get over this problem?

Thanks for the tip! It works fine with some slight modifications :)

Can you explain how you make these modifications ? Is this should only be visualized in the training phase ?
How can i visualize using a trained model? Thanks! @ogamache @ycliu93


[changelog] ClassCastException when changelog extraProperties "categorizeScopes" is set and a commit is not conventional

I also noticed that we may have the wrong regex for conventional commit type: https://github.com/jreleaser/jreleaser/blob/36b04bf3818c6233288b29626601f323ed0ae461/sdks/jreleaser-git-java-sdk/src/main/java/org/jreleaser/sdk/git/ChangelogGenerator.java#L700-L700
I was trying to use i18n as type, but it's not allowed.
Task List

[x] Steps to reproduce provided
[x] Stacktrace (if present) provided
[ ] Example that reproduces the problem (link to git repository is ideal)
[x] Full description of the issue provided (see below)

Steps to Reproduce

have a commit that is not conventional in the git history
set changelog.extraProperties.categorizeScopes=true
run jreleaserChangelog

Expected Behaviour
Changelog should be generated
Actual Behaviour
JReleaser crashes with
Environment Information ClassCastException

Operating System: macOS
JReleaser Version: 1.5.0
JDK Version: 11

Please paste stacktraces from [out|build|target]/jreleaser/trace.log.
trace.log

🎉 This issue has been resolved in v1.5.1 (Release Notes)


At which step is the dlc_message_handler. send_message processing executed? How can the event event in the contract be triggered?

Offered contract: 7c8482d4c1ea57a50d7e7bfbf1ce4df4410ca1dc58831ed1b880bd36a20c816c
Closed contract: 7f7ae9cb72a09892f73f47f07077860139f4c7f869c0762ec4738fa79d70a532
Outcomes: [["0", "0", "0", "0", "0", "0", "1", "0", "0", "0", "1", "1", "0", "0", "1", "0", "1", "0", "0", "0"]]
PnL: -100000000 sats
I would like to ask, should the contract status change to confirmation status after it is blocked? At which step is the dlc_message_handler. send_message processing executed? How can the event event in the contract be triggered? Can anyone help me answer this? Thank you.

The LDK-sample project handles messages, but there is very little explanation for this project. Can you help improve the usage examples?

At which step is the dlc_message_handler. send_message processing executed?

dlc_message_handler.send_message is called in multiple places in cli.rs. It pushes back messages in a queue which is then emptied by the get_and_clear_pending_msg method of message_handler.rs, which is called by the process_events method of peer_handler.rs in LDK.

The LDK-sample project handles messages, but there is very little explanation for this project. Can you help improve the usage examples?

LDK is indeed used to send and receive messages. It's out of the scope of this project to provide documentation for it though (but there is already a lot of documentation for LDK, I think you can check it and get a better idea of how things work).

I am organizing the offer contract, The execution flowchart of acceptbuffer and listcontracts was used to help me understand DLC, but I encountered an issue and saw dlc_message_handler. send_message (pubkey, offer);
Peer_manager. process_events(); I don't know how to run these two pieces of code specifically?
DLC Contract Process：
https://www.processon.com/view/link/665027cf61f8c171fd1cc881
message handller
https://www.processon.com/view/link/66502817e57102599635aba7?cid=64cc9c49a433c931b2f90c10
I thought it would be executed immediately, but after adding the log, it was not discovered to be executed. Do I understand where the problem lies? For example, after the accept DLC process ->send_message, does it execute the method in on_accept_message?

For example, after the accept DLC process ->send_message, does it execute the method in on_accept_message?

It should indeed call on_accept_message (or on_accept_channel if you are using channels). Otherwise I'm sorry but I don't really understand what is your issue. Maybe you can try having a look at the integration tests in dlc_manager/tests, that should help you see how things are expected to be processed.

Sorry, there may be some issues with my description,I added a log to the on_accept_message method, but it was not executed. My contract status has not been changed to confirmation status.Do I need to use any other services besides the Rust dlc project to start?


[Issue-3326] Fix missing data with earning reward info

Related issue(s)

#3326

Is your feature request related to a problem(s)? Please describe.

Fix missing data with earning reward info

Describe the solution you'd like

Fix missing data with earning reward info

Describe alternatives you've considered

No

Additional context

No

🚀 Deployed on https://pr-3338--sw-web-runner.netlify.app


Add Logitech G900 SVG for gnome theme

Related issues: #300, #213.
@kudlatyamroth or @kaymio please test this.

Thanks for the SVG.
Couple of issues:

you are missing:

diff --git a/meson.build b/meson.build
index c4f104b..4bb8755 100644
--- a/meson.build
+++ b/meson.build
@@ -280,6 +280,7 @@ gnome_svg_files = [
        'data/gnome/logitech-g500s.svg',
        'data/gnome/logitech-g502.svg',
        'data/gnome/logitech-g700.svg',
+       'data/gnome/logitech-g900.svg',
        'data/gnome/roccat-kone-xtd.svg',
 ]
 install_data(gnome_svg_files,

all the texts that are supposed to be shown on right are aligned from the left (so they are overlapping with the leader instead of moving out from it)

And some more information:

you need to use style="text-align:start... instead of style="text-align:end...
the LED are inverted, and you can not really configure the LED that show the DPI/profile (right under the resolution up/down buttons). So I'd remove the second LED and keep only the logo one as LED-0

I knew I forgot something. Is it good now?

I missed your message about the LEDs. I changed that too, now.


[YUNIKORN-2504] Support canonical labels for queue/applicationId in scheduler

What is this PR for?
Support canonical Queue/ApplicationId labels in Pod, allows it coexist with the existing metadata.

yunikorn.apache.org/app-id (New, Canonical Label)
yunikorn.apache.org/queue (New, Canonical Label)

Check metadata consistency before move task state from 'New' to 'Pending',
Run the pod metadata check in task.sanityCheckBeforeScheduling()

If sanity check is failed due to inconsistent metadata, move the task from 'New' to 'Rejected' state. And fail the pod with reasons. (Please check below screenshots.)

What type of PR is it?

[x] - Feature

Todos
Admission Controller should fail the pod request too if the metadata is inconsistent. Will create another Jira once this PR got approved.
What is the Jira issue?
https://issues.apache.org/jira/browse/YUNIKORN-2504
How should this be tested?
Run e2e test:

(basic_scheduling) Verify_Pod_With_Conflicting_AppId
(basic_scheduling) Verify_Pod_With_Conflicting_QueueName
(recovery_and_restart) Verify_Pod_Restart_After_Add_Conflict_Metadata

Run below simple sleep pods:

apiVersion: v1
kind: Pod
metadata:
  labels:
    app: sleep
    yunikorn.apache.org/app-id: "application-sleep-0001"
    yunikorn.apache.org/queue: "root.sandbox"
  annotations:
    yunikorn.apache.org/queue: "root.sandbox-another"
  name: pod-with-inconsistent-queue
spec:
  schedulerName: yunikorn
  restartPolicy: Never
  containers:
    - name: sleep-6000s
      image: "alpine:latest"
      command: ["sleep", "6000"]
      resources:
        requests:
          cpu: "100m"
          memory: "500M"

---
apiVersion: v1
kind: Pod
metadata:
  labels:
    app: sleep
    yunikorn.apache.org/app-id: "application-sleep-0002"
  annotations:
    yunikorn.apache.org/app-id: "application-sleep-0002-another"
  name: pod-with-inconsistent-app-id
spec:
  schedulerName: yunikorn
  restartPolicy: Never
  containers:
    - name: sleep-6000s
      image: "alpine:latest"
      command: ["sleep", "6000"]
      resources:
        requests:
          cpu: "100m"
          memory: "500M"

---
apiVersion: v1
kind: Pod
metadata:
  labels:
    app: sleep
    yunikorn.apache.org/app-id: "application-sleep-0003"
    yunikorn.apache.org/queue: "root.sandbox"
  name: pod-with-correct-labels
spec:
  schedulerName: yunikorn
  restartPolicy: Never
  containers:
    - name: sleep-6000s
      image: "alpine:latest"
      command: ["sleep", "6000"]
      resources:
        requests:
          cpu: "100m"
          memory: "500M"
---

Screenshots (if appropriate)
Below is the screenshot without admission controller:

Questions:
NA

Mark this PR as draft after the discussion of the community sync-up.
To be changed:

Ordering change to (Cannonical Label -> Annotation -> Queue(Existing))
Version 1.6.0 only prints out warnings; task rejection should occur in version 1.7.0.

Close this draft PR and another PR for formal review was created.
https://github.com/apache/yunikorn-k8shim/pull/871


Fetch error due to CORS

Followed your example and got the following error, using both a test version (on localhost:3000) and on a website (with https://). Might you know how to work around it?
Fetch API cannot load http://www.morphemeris.com/ephemeris.php?date=1974-02-17T23%3A30Z&lat=37.4381927&lon=-79.18932.
Redirect from 'http://www.morphemeris.com/ephemeris.php?date=1974-02-17T23%3A30Z&lat=37.4381927&lon=-79.18932'
to 'https://morphemeris.com/ephemeris.php?date=1974-02-17T23%3A30Z&lat=37.4381927&lon=-79.18932'
has been blocked by CORS policy:
No 'Access-Control-Allow-Origin' header is present on the requested resource.
Origin 'http://localhost:3000' is therefore not allowed access.
If an opaque response serves your needs, set the request's mode to 'no-cors' to fetch the resource with CORS disabled.

I ported morphemeris.com/ephemeris.php to a private domain, manually changed the astrologyjs.min.js file inside node_modules to fetch from this new file, and set the php header of this new file to allow for CORS, Access-Control-Allow-Origin:
<?php
	header( 'content-type: application/json' );
	header( 'Access-Control-Allow-Origin: *' );
	ini_set( 'display_errors', 0 );

With that error fixed, I now ran into the following error:
(index):1 Uncaught SyntaxError: Unexpected end of JSON input
    at Object.parse (<anonymous>)
    at exports.IncomingMessage.<anonymous> (astrologyjs.min.js:1)
    at exports.IncomingMessage.EventEmitter.emit (events.js:96)
    at endReadableNT (_stream_readable.js:920)
    at afterTickTwo (index.js:27)
    at Item.run (browser.js:153)
    at drainQueue (browser.js:123)


Unicode patch breaks readLine() of JConsole input

Attempting something like this:
Reader consoleIn = console.getIn();
BufferedReader bufConsoleIn = new BufferedReader((InputStreamReader)consoleIn);
...
String line = bufConsoleIn.readLine();
Will block.
The "unicode patch" applied to acceptLine() in JConsole.java is incompatible with readLine().  This flaw makes JConsole much less useful for applications outside the built-in interpreter.
Reported by: *anonymous
Original Ticket: beanshell/bugs/331

console is undefined
Closed: cannot reproduce


lapce not responsive and rendering issues on MacOs Mojave

Lapce Version
Not sure as 'About Lapce' opens 'off-screen' and is not responsive. I downloaded lapce for MacOs today from the website.
System information
MacOs Mojave 10.14.6 with Intel UHD Graphics 630 1536 MB Graphics
Describe the bug
After Lapce opens with rendering issues (parts of the window just grey). The editor is not responsive apart from scrolling in the code window (typing does not work). Lapce also seems to have a wrong resolution (very low res).
Additional information
Other information that can be used to further reproduce or isolate the problem.
This commonly includes:
added a screenshot showing how it opens.

This might be related to #1083 due to the fact this also happens on Intel iGPU


Feature/merge many paths

Thank you for PR. But, I am really not sure that it needs to parse multiple directories. Generally, putting a project root directory will cover most cases in general.

@mattdamon108 I agree that putting a project root directory will cover most cases. Merging schema that splitted between is a real production case.

@3timeslazy Can you explain little more in detail about the case of 2 paths in a real production case?

@mattdamon108 in our company, we have large GraphQL API. API is a union of several services, each with it's own methods. And there is no possibility to put them into single directory.

@3timeslazy Alright! Why not to confirm PR for additional feature? ;)

@3timeslazy Appreciate your valuable PRs which enriches a feature of this module.

@mattdamon108 this is all for now. Next feature - tests.

@3timeslazy Good to know that. I think I need to wrap up and release.


Cheater mad

thats crazy, look what i found

Being a small app nobody cares about licenses and code rights, not nice but what can you do about it...

Being a small app nobody cares about licenses and code rights, not nice but what can you do about it...

yeah, atleast they gave credit


Senec entities can‘t be choosen in Energy Dashboard

I installed everything according to the instructions. I can now also create a new map in the dashboard and select the Senec entities. However, I have the problem that in the Energy Dashboard the Senec entities cannot be selected because they are not shown.

I assume this is related to the change by SENEC -> please see additional documentation here #11 - if this is the solution, please be so kind and close this issue - TIA

Additional Info:
https://www.home-assistant.io/docs/energy/electricity-grid/#troubleshooting
If you are unable to select your energy sensor in the grid consumption drop-down, make sure that its value is being recorded in the Recorder settings.


Non-admin feature tracker

Tracking issue for non-admin work issues and general discussion.
Mocks: https://docs.google.com/presentation/d/1JU_Lw1A-gUO4gBRofZzi74aONcknYRHbqBl_xHxikYQ

[ ] Determine multi-tenant vs. single-tenant non-admin design
[ ] UI needs to be made aware if in admin vs. non-admin mode
[ ] Implement tokens screen
[ ] Implement regenerate OAuth token
[ ] Implement welcome screen
[ ] Implement expired token modal
[ ] Implement non-admin MigPlan general step
[ ] Implement token edit
[ ] Add MigToken support to k8s client
[ ] Implement add token for Cluster
[ ] Implement non-admin cluster status
[ ] Automatically add MigToken with host logged-in user OAuth token
[ ] Implement host MigToken OAuth update on login
[ ] List of clusters should be sourced from discovery service
[ ] Implement empty state view for clusters w/ non-admin

https://github.com/konveyor/mig-ui/issues/883 needs to be closed ASAP as that decision will dictate a lot of the implementation details.


Filters

This plugin already has filters? I think it doesn't if you can add filters please.

They are a work in progress.


Deduplication

By project
By custodian
By custodians in order

Re-deduplication, should work with adding data to the project

easy with db


Add specs for Operations::Account validations

Would you consider adding the shoulda-matchers gem  to the project? It would make it easier to write specs for validations.

@elias19r could you rebase please we want to merge this in 2-3


Merge Ground Overlay Support (PR-1586)

Replaces #1586, as after rebase to the current master.

@isener mind checking my rebased PR? (BTW thank you very much)

Good work mate, I don't have time to check fully. Will do it asap on the fly

@rborn frankly speaking, no. I just checked that it was building ok. Did not check the features.

I'll merge and wait for the community feedback.

React.PropTypes is deprecated. Every where else in this package we're using the separate package.


Update the TensorBoard notebooks to set the right TensorBoard Profiler version

REQUIRED: Add a summary of your PR here, typically including why the change is needed and what was changed. Include any design alternatives for discussion purposes.

Set TensorBoard profiler version to ensure it has the right version installed.

REQUIRED: Fill out the below checklists or remove if irrelevant

If you are opening a PR for Official Notebooks under the notebooks/official folder, follow this mandatory checklist:

[ ] Use the notebook template as a starting point.
[ ] Follow the style and grammar rules outlined in the above notebook template.
[ ] Verify the notebook runs successfully in Colab since the automated tests cannot guarantee this even when it passes.
[ ] Passes all the required automated checks. You can locally test for formatting and linting with these instructions.
[ ] You have consulted with a tech writer to see if tech writer review is necessary. If so, the notebook has been reviewed by a tech writer, and they have approved it.
[ ] This notebook has been added to the CODEOWNERS file under the Official Notebooks section, pointing to the author or the author's team.
[ ] The Jupyter notebook cleans up any artifacts it has created (datasets, ML models, endpoints, etc) so as not to eat up unnecessary resources.

If you are opening a PR for Community Notebooks under the notebooks/community folder:

[ ] This notebook has been added to the CODEOWNERS file under the Community Notebooks section, pointing to the author or the author's team.
[ ] Passes all the required formatting and linting checks. You can locally test with these instructions.

If you are opening a PR for Community Content under the community-content folder:

[ ] Make sure your main Content Directory Name is descriptive, informative, and includes some of the key products and attributes of your content, so that it is differentiable from other content
[ ] The main content directory has been added to the CODEOWNERS file under the Community Content section, pointing to the author or the author's team.
[ ] Passes all the required formatting and linting checks. You can locally test with these instructions.

/gcbrun

/gcbrun

The change has been submitted in another PR: https://github.com/GoogleCloudPlatform/vertex-ai-samples/pull/1369


Replace Cardano.API.Typed import with Cardano.Api

Rename Cardano.API to Cardano.Api
Replace Cardano.API.Typed with Cardano.Api
Additionally import Cardano.Api.Byron or Cardano.Api.Shelley as necessary

bors merge


add ability to add hooks to common webelement actions, e.g. to "slow" them for "demo reasons"

It will be great to have "demo mode" for test. Just make a pause after action to allow see the system behavior - useful thing when make a demo for customer etc.

Here is an example of how this feature is implemented in Selenide (at least its 2.7 version):
        WebDriverRunner.addListener(
                new AbstractWebDriverEventListener() {

                    public void beforeClickOn(WebElement element, WebDriver driver) {

                        debugLog("CLICKING: " + element);
                        sleep(100);
                    }

                    public void beforeFindBy(By by, WebElement element, WebDriver driver) {
                        sleep(100);
                    }

                });

There should be also an ability to provide one hook for all actions.
Use cases:

slow down everything "for demo reasons"
add waitForAngular waiting before all actions


Feature Request: Search nodes by tag

Is it hard to implement additional field in the node structure and API GET /api/nodes/{tag}?
It allows ability to manage collections of nodes by some tags.

Could you share which usecase this would serve?

@brocaar, I see few usecases:
1st usecase.
As loraserver user I would like manage node collections by node types. One type can be served by different applications.
2nd usecase.
As network provider I would like manage node collections for clients. Each client can have nodes on different applications.
Next usecase.
As client of network provider I would like manage node collections on my own reason. Each collection can be served by different applications.
Right now to get info about few nodes we have to call API GET /api/nodes/{devEUI} few times. With tag we could get the collection of nodes in one API call. I prefer use API and not direct queries to DB.
The main goal is implementing the business logic separately from loraserver.

@brocaar, a few words more.
The tagging feature seems for me as general solution that allows manage collections based on the different properties. The placement, the ownership, some trait can be described with tags and used for not only better UX but statistics and analytics.

For the time being, this won't be implemented in LoRa App Server. However the good news is that I'm moving more and more LoRaWAN logic into LoRa Server (currently in the process of refactoring the downlink queue, this will be moved to LoRa Server). With that, it will be fairly easy to implement your own device management dashboard on top of LoRa Server instead of LoRa App Server. Note that in the last release, all the device settings already moved to LoRa Server. See also: https://forum.loraserver.io/t/release-lora-server-0-22-0-lora-app-server-0-14-x-read-carefully/350


chore(sdk): fix client bug

In the past, if an endpoint is mounted as "/", client cannot invoke it other than explicitly using web endpoint get or put. This makes "/" mounted on __call__.

@vthinkxie we might need to change the frontend code so this gets fixed as well. Right now frontend shows a dot with no name, which is actually not legal python code.

 @vthinkxie we might need to change the frontend code so this gets fixed as well. Right now frontend shows a dot with no name, which is actually not legal python code.

got it, track https://github.com/leptonai/lepton/issues/5711


Update linear progress with new sizes and thickness

Follow up from #4116 and #3978, add new sizes to linear progress, and use new thickness tokens

Lungo Goal: @bhoppers2008 to unblock as part of border discussions

Moved to Macchiato sprint. No capacity in Lungo

Macc Goal: unblock by EOS, how to implement in design

Take off assignee and sprint, leave in Q4, see https://github.com/jpmorganchase/salt-ds/issues/3978#issuecomment-2470103303


Application Checklist Remains 'Failed' After Clearinghouse Service Restoration and Successful Retrigger

Summary:
Despite the successful retriggering of the Clearinghouse process by the CX Admin following a service downtime, the application checklist fails to reflect the correct status and remains in a 'Failed' state. Additionally, the SD Factory process step was not initiated as expected.
Steps to Reproduce:

Begin the registration process on the platform.
Encounter a failure/error at the 'Clearinghouse' step due to service unavailability.
Wait for the Clearinghouse service to become available again.
CX Admin retriggered the Clearinghouse process.
Observe that the process worker steps '7' and '9' change status to '2' (DONE).
Check the application checklist and the SD Factory process creation.

Expected Result:

The application checklist updates to reflect the successful completion of the Clearinghouse step.
The SD Factory process step is created and initiated following the successful completion of the prior steps.

Actual Result:

The application checklist still shows a 'Failed' status despite the successful retrigger and execution of the Clearinghouse step.
The SD Factory process step has not been created.

Additional Information:

The issue seems to be with the status update mechanism which did not trigger as expected after the Clearinghouse step completed successfully.
There is a discrepancy between the process worker steps' statuses and the application checklist status.

Closing, cant reproduce


PHOENIX-5960 : Creating view on non-existent table should throw TNFE

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 33s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s
The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ master Compile Tests _

+1 :green_heart:
mvninstall
12m 16s
master passed

+1 :green_heart:
compile
0m 58s
master passed

+1 :green_heart:
checkstyle
0m 33s
master passed

+1 :green_heart:
javadoc
0m 45s
master passed

+0 :ok:
spotbugs
2m 57s
phoenix-core in master has 966 extant spotbugs warnings.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
7m 35s
the patch passed

+1 :green_heart:
compile
0m 53s
the patch passed

+1 :green_heart:
javac
0m 53s
the patch passed

+1 :green_heart:
checkstyle
0m 35s
phoenix-core: The patch generated 0 new + 159 unchanged - 1 fixed = 159 total (was 160)

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
javadoc
0m 42s
the patch passed

+1 :green_heart:
spotbugs
3m  8s
the patch passed

_ Other Tests _

-1 :x:
unit
107m 11s
phoenix-core in the patch failed.

+1 :green_heart:
asflicense
0m 29s
The patch does not generate ASF License warnings.

141m 10s

Reason
Tests

Failed junit tests
phoenix.end2end.ViewMetadataIT

phoenix.end2end.TableSnapshotReadsMapReduceIT

Subsystem
Report/Notes

Docker
ClientAPI=1.40 ServerAPI=1.40 base: https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-987/1/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/phoenix/pull/987

Optional Tests
dupname asflicense javac javadoc unit spotbugs hbaseanti checkstyle compile

uname
Linux 0a46bf80e71b 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev/phoenix-personality.sh

git revision
master / cfbd997

Default Java
Private Build-1.8.0_242-8u242-b08-0ubuntu3~16.04-b08

unit
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-987/1/artifact/yetus-general-check/output/patch-unit-phoenix-core.txt

Test Results
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-987/1/testReport/

Max. process+thread count
7047 (vs. ulimit of 30000)

modules
C: phoenix-core U: phoenix-core

Console output
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-987/1/console

versions
git=2.7.4 maven=3.3.9 spotbugs=4.1.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 44s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s
The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
7m 35s
the patch passed

+1 :green_heart:
compile
0m 52s
the patch passed

+1 :green_heart:
javac
0m 52s
the patch passed

-1 :x:
checkstyle
0m 35s
phoenix-core: The patch generated 2 new + 155 unchanged - 5 fixed = 157 total (was 160)

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
javadoc
0m 44s
the patch passed

+1 :green_heart:
spotbugs
3m  3s
the patch passed

_ Other Tests _

-1 :x:
unit
106m 56s
phoenix-core in the patch failed.

+1 :green_heart:
asflicense
0m 30s
The patch does not generate ASF License warnings.

146m 53s

Reason
Tests

Failed junit tests
phoenix.end2end.ViewMetadataIT

Subsystem
Report/Notes

Docker
ClientAPI=1.40 ServerAPI=1.40 base: https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-987/3/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/phoenix/pull/987

Optional Tests
dupname asflicense javac javadoc unit spotbugs hbaseanti checkstyle compile

uname
Linux 656f28297fd8 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev/phoenix-personality.sh

git revision
master / d1fc3f5

Default Java
Private Build-1.8.0_242-8u242-b08-0ubuntu3~16.04-b08

checkstyle
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-987/3/artifact/yetus-general-check/output/diff-checkstyle-phoenix-core.txt

unit
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-987/3/artifact/yetus-general-check/output/patch-unit-phoenix-core.txt

Test Results
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-987/3/testReport/

Max. process+thread count
7037 (vs. ulimit of 30000)

modules
C: phoenix-core U: phoenix-core

Console output
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-987/3/console

versions
git=2.7.4 maven=3.3.9 spotbugs=4.1.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

Thanks for the reviews @swaroopak @yanxinyi

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 18s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s
The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ master Compile Tests _

+1 :green_heart:
mvninstall
13m 55s
master passed

+1 :green_heart:
compile
1m  3s
master passed

+1 :green_heart:
checkstyle
0m 33s
master passed

+1 :green_heart:
javadoc
0m 47s
master passed

+0 :ok:
spotbugs
3m 11s
phoenix-core in master has 966 extant spotbugs warnings.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
9m 14s
the patch passed

+1 :green_heart:
compile
0m 57s
the patch passed

+1 :green_heart:
javac
0m 57s
the patch passed

+1 :green_heart:
checkstyle
0m 33s
phoenix-core: The patch generated 0 new + 156 unchanged - 4 fixed = 156 total (was 160)

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
javadoc
0m 46s
the patch passed

+1 :green_heart:
spotbugs
3m 22s
the patch passed

_ Other Tests _

-1 :x:
unit
210m 49s
phoenix-core in the patch failed.

+1 :green_heart:
asflicense
1m  5s
The patch does not generate ASF License warnings.

251m 13s

Subsystem
Report/Notes

Docker
ClientAPI=1.40 ServerAPI=1.40 base: https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-987/4/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/phoenix/pull/987

Optional Tests
dupname asflicense javac javadoc unit spotbugs hbaseanti checkstyle compile

uname
Linux 8e883417f7b0 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev/phoenix-personality.sh

git revision
master / d1fc3f5

Default Java
Private Build-1.8.0_242-8u242-b08-0ubuntu3~16.04-b08

unit
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-987/4/artifact/yetus-general-check/output/patch-unit-phoenix-core.txt

Test Results
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-987/4/testReport/

Max. process+thread count
6113 (vs. ulimit of 30000)

modules
C: phoenix-core U: phoenix-core

Console output
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-987/4/console

versions
git=2.7.4 maven=3.3.9 spotbugs=4.1.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m  5s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s
The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
7m 40s
the patch passed

+1 :green_heart:
compile
0m 53s
the patch passed

+1 :green_heart:
javac
0m 53s
the patch passed

+1 :green_heart:
checkstyle
0m 35s
phoenix-core: The patch generated 0 new + 156 unchanged - 4 fixed = 156 total (was 160)

+1 :green_heart:
whitespace
0m  1s
The patch has no whitespace issues.

+1 :green_heart:
javadoc
0m 46s
the patch passed

+1 :green_heart:
spotbugs
3m  5s
the patch passed

_ Other Tests _

-1 :x:
unit
181m 32s
phoenix-core in the patch failed.

+1 :green_heart:
asflicense
0m 27s
The patch does not generate ASF License warnings.

216m 14s

Reason
Tests

Failed junit tests
phoenix.end2end.UpsertSelectIT

Subsystem
Report/Notes

Docker
ClientAPI=1.40 ServerAPI=1.40 base: https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-987/5/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/phoenix/pull/987

Optional Tests
dupname asflicense javac javadoc unit spotbugs hbaseanti checkstyle compile

uname
Linux eab3d6927f6d 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev/phoenix-personality.sh

git revision
master / 457a67c

Default Java
Private Build-1.8.0_242-8u242-b08-0ubuntu3~16.04-b08

unit
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-987/5/artifact/yetus-general-check/output/patch-unit-phoenix-core.txt

Test Results
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-987/5/testReport/

Max. process+thread count
6764 (vs. ulimit of 30000)

modules
C: phoenix-core U: phoenix-core

Console output
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-987/5/console

versions
git=2.7.4 maven=3.3.9 spotbugs=4.1.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.


Change over time bug

@vijaykiran can you describe the problem?
Is it the same as #1150 ?

duplicate #1150


register addons

I noticed that the addons weren't working properly, because they actually were never registered.
I noticed because the hierarchy separator was not actually working, it looks like this:

with this fix.
references #48

Thanks @joscha!  I guess I had previously added just enough to reproduce the setOptions error, and hadn't got all the way to making sure they were working in the UI.  :)


Accept Function<Input, Output> for Try.onSuccess or Try.onFailure

Right now we have to do this (due to onSuccess and onFailure only accepting Consumers):
Try<Boolean> resultTry = Try.success(true);

if (resultTry.isSuccess) return this.someTransformation(resultTry.get());
return someOtherTransormation(resultTry.getCause());

It would be nice if we could do this instead:
return Try.success(true)
    .onSuccess(this::someTransformation)
    .onFailure(cause -> someOtherValue(cause));

What is the reason that onSuccess and onFailure only accept Consumers?

Please use map / mapTry instead of onSuccess.

Of course, the answer is so obvious and I did not think my question it through. Thank you!

Thank you for asking!


How use in a React Component?

Is there an example of how to use on react?

The same question

If you are using webpack: http://stackoverflow.com/questions/35099440/wow-js-with-webpack-and-react
If not, thats no secret.

I'm getting errors and the answer from @renatorib doesn't seem to be my issue.
I do import WOW from 'wowjs';
componentDidMount() {
  const wow = new WOW();
  wow.init();
}

but I get TypeError: _wowjs2.default is not a constructor
If I do import { WOW } from 'wowjs';
I get
MutationObserver is not supported by your browser.
WOW.js cannot detect dom mutations, please call .sync() after loading new content.

If I do
componentDidMount() {
  const wow = new WOW();
  wow.sync();
}

Notice the wow.sync()
I get TypeError: Cannot read property 'nodeType' of undefined
Stuck there :(

@johhansantana
Your code worked for me, I just had to include the script into my html file

if (typeof window !== 'undefined') {
let WOW = require('wowjs');
}
componentDidMount() {
if (typeof window !== 'undefined') {
const wow = new WOW();
wow.init();
}
}

Supposedly you're supposed to double the WOW and write new WOW.WOW().init() but it still doesn't work. I can see in the rendered html that the animated class is indeed being added, but the animation doesn't run.

@adammoisa you still need help to get it working?

I have the same question.

@johhansantana  yeah. I have the same question .my code works when  I use the following.
import WOW from 'wowjs/dist/wow.js';
componentDidMount() {
const wow = new WOW();
wow.init();
}
But.I get WOW.js cannot detect dom mutations, please call .sync() after loading new content.
How do I slove the question?

tried to browserify dist/wow.js and
  componentDidMount() {
    const WOW = require('../WOW/dist/wow.js')
    this.wow = new WOW.WOW()
    this.wow.init()
  }

but it logs
e.WOW is not a constructor

The same question

Ok so what I did to get it working is to include the .js plugin in the  of my index.html:
<script src="//cdnjs.cloudflare.com/ajax/libs/wow/1.1.2/wow.min.js" type="text/javascript"></script>
And in my component:
componentDidMount() {         new WOW().init() }

Doing
import {WOW} from 'wowjs/dist/wow';

new WOW().init();

Worked for me! ;-)

This should work out of the box with Webpack:
window.WOW = require('wowjs').WOW;
new WOW({
  mobile: false
}).init();
```

Why takes people from making their stuff compliant?

Hi!
In React.js I initiate this plugin with this code:
import WOW from 'wowjs';

class App extends React.Component {
  componentDidMount() {
    new WOW.WOW().init();
  }
...

But I cannot get rid of these warnings:
'MutationObserver is not supported by your browser.'
'WOW.js cannot detect dom mutations, please call .sync() after loading new content.'

They are showed in all browsers. I'm not adding something dynamically
How could it be fixed?

@01Kuzma
try this:
import WOW from 'wowjs';

componentDidMount() {
    new WOW.WOW({
        live: false
    }).init();
}

worked for me...

I have resolved it

@vitorarantes, nice! Works!

try this:
import { WOW } from "wowjs/dist/wow";
new WOW({
live: false
}).init();

This works for me:
import WOW from 'wowjs'

  componentDidMount() {
    if (typeof window !== 'undefined') {
      const wow = new WOW.WOW({
        live: false,
      })
      wow.init()
    }
  }

import WOW from "wow.js/dist/wow.js";

componentDidMount() {
    new WOW().init();
  }

How can I init for functional components
 componentDidMount() {
        new WOW.WOW({
            live: false
        }).init();
    }

who can i use functional component

Hello，我是林昕，我已收到来信~


Results Interpretation

Hi,
When I run the CROWN verified error with let's say \epsilon = 0.3, what's the exact meaning of the terms in the output like Loss, CE loss etc.? I'd like to estimate or desume a lower bound with just CROWN on some architectures+data, is it possible with this code?
p.s. the architecture has not been trained with CROWN-IBP, it is just naturally trained (or at least with adversarial robustness).
Written in other words, given an epsilon radius, I'd like to know if that n-ball is safe using just CROWN.. is it possible to do that with your code and with a naturally trained architecture that I've built by myself?
Thank you,
Best.

Sorry for the late reply. Yes, you can use the code to evaluate your model, no mater how they were trained.
When you run the code example for computing CROWN verified error, Loss and CE loss are not useful. They are used to monitor the training process. The metrics that do make sense include Err (clean error) and Rob Err (verified error). You should read the numbers in parentheses (they are the mean over the epoch, rather than a batch).
It is easy the dump CROWN bounds on some architecture + data, on any networks, not necessarily trained using CROWN-IBP. There is some commented code in train.py which shows how to call the bound API to obtain CROWN bounds: https://github.com/huanzhang12/CROWN-IBP/blob/master/train.py#L165
These comments print out lower and upper bounds for all examples in a batch. You can check if the lower bound is less than 0 to determine if an example is guaranteed to be safe or not, just like what I did for computing the verified error here.
In order to make the code read your model and data, you can follow instructions on how to train your own model, except for the last step where you run eval.py instead of train.py. Don't forget to add necessary command line arguments like "eval_params:method_params:bound_type=crown-full" to enable the full CROWN bounds (see instructions here).
Let me know if there is anything unclear and if you have any further questions.

No worries, thank you for the detailed reply, I'll try it in the next days.

Sorry for the late reply. Yes, you can use the code to evaluate your model, no mater how they were trained.
When you run the code example for computing CROWN verified error, Loss and CE loss are not useful. They are used to monitor the training process. The metrics that do make sense include Err (clean error) and Rob Err (verified error). You should read the numbers in parentheses (they are the mean over the epoch, rather than a batch).
It is easy the dump CROWN bounds on some architecture + data, on any networks, not necessarily trained using CROWN-IBP. There is some commented code in train.py which shows how to call the bound API to obtain CROWN bounds: https://github.com/huanzhang12/CROWN-IBP/blob/master/train.py#L165
These comments print out lower and upper bounds for all examples in a batch. You can check if the lower bound is less than 0 to determine if an example is guaranteed to be safe or not, just like what I did for computing the verified error here.
In order to make the code read your model and data, you can follow instructions on how to train your own model, except for the last step where you run eval.py instead of train.py. Don't forget to add necessary command line arguments like "eval_params:method_params:bound_type=crown-full" to enable the full CROWN bounds (see instructions here).
Let me know if there is anything unclear or if you have any further questions.

I am just wondering why you only need to check lowerbound<0 to guarantee safety? I thought we needed to check whether the perturbation is within the boundary - But I might just be confused about the definition of verification

How could one print out model parameters for each layer?

I don't remember if models allowed are from keras implementations, but in that case it's enough
[print(l.weights) for l in model.layers]

I also wonder why You can check if the lower bound is less than 0 to determine if an example is guaranteed to be safe?
Don't we need to check whether the margin is negative? Something along the lines lb[true_class] - ub[all_except_true_class].max() > 0?


Longest Palindromic Subsequence

💥 Proposal
Given a string s, find the longest palindromic subsequence's length in s.
A subsequence is a sequence that can be derived from another sequence by deleting some or no elements without changing the order of the remaining elements.
Have you read the Contributing Guidelines on Pull Requests ?
Yes

Hi @ankitapuri
I would like to solve it in Python. Please assign it to me.

Hi @ankitapuri
Please assign it to me.

Hi @ankitapuri
Any update?


TypeErrors: infoHashes is null

A few releases ago, probably since v8.3.0, two TypeErrors have appeared (my bad for not reporting this before):
var index = peer.socket.infoHashes.indexOf(this.infoHash)
                                      ^

TypeError: Cannot read property 'indexOf' of null
    at Swarm._onAnnounceStopped (/fakepath/tracker/node_modules/bittorrent-tracker/lib/server/swarm.js:106:39)
    at LRU.<anonymous> (/fakepath/tracker/node_modules/bittorrent-tracker/lib/server/swarm.js:31:10)
    at emitOne (events.js:96:13)
    at LRU.emit (events.js:188:7)
    at LRU._checkAge (/fakepath/tracker/node_modules/lru/index.js:104:10)
    at LRU.peek (/fakepath/tracker/node_modules/lru/index.js:64:13)
    at Swarm._getPeers (/fakepath/tracker/node_modules/bittorrent-tracker/lib/server/swarm.js:149:27)
    at Swarm.announce (/fakepath/tracker/node_modules/bittorrent-tracker/lib/server/swarm.js:65:17)
    at announce (/fakepath/tracker/node_modules/bittorrent-tracker/server.js:684:11)
    at /fakepath/tracker/node_modules/bittorrent-tracker/server.js:654:7

And this one:
if (socket.infoHashes.indexOf(params.info_hash) === -1) {
                           ^

TypeError: Cannot read property 'indexOf' of null
    at /fakepath/tracker/node_modules/bittorrent-tracker/server.js:513:28
    at /fakepath/tracker/node_modules/bittorrent-tracker/server.js:716:7
    at Swarm.announce (/fakepath/tracker/node_modules/bittorrent-tracker/lib/server/swarm.js:62:3)
    at announce (/fakepath/tracker/node_modules/bittorrent-tracker/server.js:684:11)
    at /fakepath/tracker/node_modules/bittorrent-tracker/server.js:654:7
    at /fakepath/tracker/node_modules/bittorrent-tracker/server.js:365:5
    at _combinedTickCallback (internal/process/next_tick.js:67:7)
    at process._tickCallback (internal/process/next_tick.js:98:9)

Seems like it is trying to access the infoHashes of a peer already evicted.
I would like to fix them myself, but, unfortunately, I don't have too much time these days.
@feross @DiegoRBaquero @yciabaud

@alxhotel Do you have latest version?

Yep.
v8.5.0 of bittorrent-tracker.
v6.9.5 of NodeJS.

@feross https://github.com/feross/bittorrent-tracker/pull/198#discussion_r99274685
This is why closing the socket must be done in nextTick 😛
When requesting the peer from the swarm, we evict and close its socket but we use it later on.
How I did it in uwt: https://github.com/DiegoRBaquero/uWebTorrentTracker/blob/master/lib/swarm.js#L89
Gonna PR.

@DiegoRBaquero I'm not sure that adding a nextTick is enough to completely solve this issue. Also, it's a bit hacky.
I think we should just ensure that we don't try to use socket.infoHashes after the socket is destroyed. PR here: https://github.com/feross/bittorrent-tracker/pull/208

Released as 8.5.1


Sessions not updating upon empty units

When a user has no units, the new units are broken, not updating correctly when a user adds a new session

should be fixed in the next update


feat: Localized font-family & upgrade nuxt

Change font-family with locales
Upgrade NuxtJS version to 2.14 to improve generate performance
Reference:

https://stackoverflow.com/questions/25619949/changing-font-family-based-on-locale
https://nuxtjs.org/blog/nuxt-static-improvements

This could fix oruyanke/okayu-button#3


Deleted Relationships may still appear selected in Lookup window in submission form

Describe the bug
When using Configurable Entities, if you add a related Entity and then delete the relationship, the deleted relationship still appears to exist in the Lookup window
To Reproduce
Steps to reproduce the behavior:

First, tested using the demo Entities data & Collections configured to match that data: https://github.com/DSpace-Labs/AIP-Files/releases/tag/demo-entities-data  This same environment exists on https://api7.dspace.org
Create a new Submission in the "People" collection. This uses the Person form
In the "Publication" field, select an existing Publication to link to this Person.  The Publication should appear back on the Submission form.  Close the Lookup window.
On the Submission form, click the delete button next to that Publication.  If you have Chrome DevTools open, you should see the DELETE request returns a 204.  The Publication disappears from the submission form.
Click on the Lookup button again (opening the Lookup window). In that window the Publication will still appear as selected. (This is incorrect, as the Person should no longer be related to any Publication)

Expected behavior
After deleting the relationship, the Lookup window should no longer show any selected value.
Related work
Discovered when testing #1096, but it was not caused by that PR.

@artlowel : Assigning to you to review for possible inclusion in 7.0. I gave this low priority for now, as the feature works, but looks misleading to a user.


add idiopathic logical axioms

@sabrinatoro
The next step to add idiopathic logical definitions is:

In the spreadsheet, could you please add the parent classes for each term that should have an equivalence axiom in column C, see an example in row 20. (Rows 3-19)
Let Nico know when you are done, and he can automatically add all of these equivalence axioms and subclassOf axioms to the respective terms.
Thank you!

@matentzn
Could you please automatically add the equivalence axioms and subclassOf axioms?
The list of terms is here: https://docs.google.com/spreadsheets/d/13eIa2Ihe6ChwOjH0s-8hp0K9IGMtveIqpFTtVvuvWK4/edit#gid=577155275
(see sheet called "idiopathic_template")
Let me know if you have questions.
Thank you!


[$1000] Attachment modal shortcut bugs on native

Coming from this comment, we'll address:
Issue 3 - Attachment modal
iOS: Add attachment modal does not navigate based on arrow keys
iOS | Android: Left & Right arrow keys does not work (seems to be duplicate of the issue above)
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01da468373ed9db4e0
Upwork Job ID:<PHONE_NUMBER>037057536
Last Price Increase: 2023-05-02

Job added to Upwork: https://www.upwork.com/jobs/~01da468373ed9db4e0

Triggered auto assignment to @twisterdotcom (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

Bug0 Triage Checklist (Main S/O)

[ ] This "bug" occurs on a supported platform (ensure Platforms in OP are ✅)
[ ] This bug is not a duplicate report (check E/App issues and #expensify-bugs)

If it is, comment with a link to the original report, close the issue and add any novel details to the original issue instead

[ ] This bug is reproducible using the reproduction steps in the OP. S/O

If the reproduction steps are clear and you're unable to reproduce the bug, check with the reporter and QA first, then close the issue.
If the reproduction steps aren't clear and you determine the correct steps, please update the OP.

[ ] This issue is filled out as thoroughly and clearly as possible

Pay special attention to the title, results, platforms where the bug occurs, and if the bug happens on staging/production.

[ ] I have reviewed and subscribed to the linked Slack conversation to ensure Slack/Github stay in sync

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

@azimgd please comment on this issue so I can assign you

Triggered auto assignment to Contributor-plus team member for initial proposal review - @rushatgabhane (External)

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

📣 @rushatgabhane You have been assigned to this job by @luacmartins!
Please apply to this job in Upwork and leave a comment on the Github issue letting us know when we can expect a PR to be ready for review 🧑💻
Keep in mind: Code of Conduct | Contributing 📖

Ah, I love the ones where I'm just a money man. Keep me posted!

📣 @azimgd You have been assigned to this job by @twisterdotcom!
Please apply to this job in Upwork and leave a comment on the Github issue letting us know when we can expect a PR to be ready for review 🧑💻
Keep in mind: Code of Conduct | Contributing 📖

I was able to reproduce "Add attachment modal does not navigate based on arrow keys".
@rushatgabhane Could you connect a hardware keyboard and try to reproduce both, on production Android please.
@luacmartins What should "Left & Right arrow keys" actually do?

@azimgd left & right should cycle through the attachment list, same as pressing the visual arrows on the screen.

vice versa, the question is "why somebody would use left/right keys to move up/down". I don't see a point and that was submitted as a bug. I'm in favor of just removing this from the scope.

I'm confused about where we're at with this now. We agreed here this needs to be done: https://expensify.slack.com/archives/C01GTK53T8Q/p1683208083045299.
Are we just waiting on the PR to be ready?

Latest update is from 4 hours ago in the PR: https://github.com/Expensify/App/pull/18407#issuecomment-1550147983

Deployed to staging 10 hours ago: https://github.com/Expensify/App/pull/18407#issuecomment-1562134209

@rushatgabhane which Upwork job? I'm only seeing this one which is closed: https://www.upwork.com/ab/applicants/1653543158120652800/job-details

@twisterdotcom what's the correct upwork url?

The automated one was this one, but it looks like that was closed for some reason: https://www.upwork.com/jobs/~01da468373ed9db4e0.
Ideally, I'd have found the one @rushatgabhane applied to, but whatever, I've just sent you both an offer to a new job referencing this issue.
Am I right in thinking there's no bonus on this one? We haven't posted a comment like this automatically: https://github.com/Expensify/App/issues/18300#issuecomment-1561748636

Paid out.


Refactor prepards

This is a partial view of the larger name resolution PR. For review, not intended to merge.
In summary, this refactor caches saves a buffer of the original query string, with indices for bind variable replacement. Post-prepare is: 1) generate a query string with bind variables, 2) run analysis and execute.
Related vitess PR: https://github.com/dolthub/vitess/pull/262


An example of working with arrays in the docs?

The readme for this project is quite good and I've been been very productive with it so far.
However, I'm having problems getting jql to print only the values. The closest I can get is screened below.
My question is, how does one start by selecting all the objects in an array then filtering out by keys, then displaying the children?
I haven't been able to figure it out from looking at the docs.
[
{
"txid": "15a10e97de1dca95dfeb3d8d06dac87dbb494eab8acc00abca0f96c140b97a73",
"vout": 0,
"address": "tb1qd2cvqk30f8kw3hj53wlj053dd9amnhhefg02gx",
"label": "",
"scriptPubKey": "00146ab0c05a2f49ece8de548bbf27d22d697bb9def9",
"amount": 0.001,
"confirmations": 61,
"spendable": true,
"solvable": true,
"desc": "wpkh([b9a5ac96/0'/0'/2']03d94b78001ec7bdfffe41cca947e77219342272bb62946af779311358f835e57f)#3e78xplt",
"safe": true
},
{
"txid": "393e1b08b7d2afd21f41f23025f5d79a06a7d12bc95a598de8b3c2f0340293a0",
"vout": 1,
"address": "tb1qd2cvqk30f8kw3hj53wlj053dd9amnhhefg02gx",
"label": "",
"scriptPubKey": "00146ab0c05a2f49ece8de548bbf27d22d697bb9def9",
"amount": 0.001,
"confirmations": 61,
"spendable": true,
"solvable": true,
"desc": "wpkh([b9a5ac96/0'/0'/2']03d94b78001ec7bdfffe41cca947e77219342272bb62946af779311358f835e57f)#3e78xplt",
"safe": true
},
{
"txid": "faa64c516c90302c624c7f7ffc26366376ac12813b1cc6e4eb8a192e3e0f08e4",
"vout": 0,
"address": "tb1qtlefx6f36vyx4l4g2wyvas4ghyuynn46xptyya",
"label": "",
"scriptPubKey": "00145ff2936931d3086afea85388cec2a8b93849ceba",
"amount": 0.0022,
"confirmations": 61,
"spendable": true,
"solvable": true,
"desc": "wpkh([b9a5ac96/0'/0'/1']03a430c19febf6d59786fe600ff185e2e8c4efa81212ead8f5a741a9427090e766)#9lt0k576",
"safe": true
}
]
Final Goal;
$ bt listunspent | jql -r 'something something something'
15a10e97de1dca95dfeb3d8d06dac87dbb494eab8acc00abca0f96c140b97a73
0
0.001
393e1b08b7d2afd21f41f23025f5d79a06a7d12bc95a598de8b3c2f0340293a0
1
0.001
faa64c516c90302c624c7f7ffc26366376ac12813b1cc6e4eb8a192e3e0f08e4
0
0.0022

Hi @keblek.
What you're trying to achieve is currently not possible. The raw -r flag was recently introduced and only works with String primitives (it mimics what jq also does). There's no way for jq to know how to cast an Object like:
{
  "txid": "15a10e97de1dca95dfeb3d8d06dac87dbb494eab8acc00abca0f96c140b97a73",
  "vout": 0,
  "amount": 0.001
}

to
15a10e97de1dca95dfeb3d8d06dac87dbb494eab8acc00abca0f96c140b97a73
0
0.001

because why should it iterate over an object and return three lines of String? jql is meant to take JSON in and output JSON back (the -r flag is already quite out of scope IMHO).
So in your example jql '.|{"txid","vout","amount"}' is enough to get:
[
  {
    "txid": "15a10e97de1dca95dfeb3d8d06dac87dbb494eab8acc00abca0f96c140b97a73",
    "vout": 0,
    "amount": 0.001
  },
  {
    "txid": "393e1b08b7d2afd21f41f23025f5d79a06a7d12bc95a598de8b3c2f0340293a0",
    "vout": 1,
    "amount": 0.001
  },
  {
    "txid": "faa64c516c90302c624c7f7ffc26366376ac12813b1cc6e4eb8a192e3e0f08e4",
    "vout": 0,
    "amount": 0.0022
  }
]

You can get an array of `amount` with `jql -r '.|{"txid","vout","amount"}|"amount"'` e.g.

Thank you very much for replying. I have found a different way to get to where I wanted and have realised that my original way of doing things would have resulted in an overly complicated jql expression (even if it were supported - which its not). The last tip is helpful so this can be closed.

If jql is only meant to input/output json, perhaps you can give a recommendation on how best to strip out the json brackets and keys to get the raw data.
grep or sed?

@keblek Did you reopen intentionally?

@keblek oh sorry, we both sent our messages at the same time.
What are you doing now - as a command ?

Sorry for the confusion, I just had a small after thought. I don't mean to monopolise your time.
What I am trying to do is too complicated and is better done by doing multiple commands rather than one shell command, otherwise I am having to write a crazy regex expression. Again sorry about the confusion.

@keblek no worries! Feel free to open an issue if something doesn't work as expected.


Benchmark baseline for stable-diffusion-2 ?

Hello,
Is there a benchmark of stable-diffusion-2 based on GPU type?
I am getting slowness on text2img, generating a 768x768 image, my Tesla T4 GPU processing speed is around 2.5it/s (as 100% utilization) and takes 13~14 seconds to be completed. I wonder is a good result for this kind of GPU, or do I have to upgrade to a higher tier for a faster process?
Please advise,

I have a RTX3090 and get 9-10it/s when generating from sd-2.1 768x768

Tesla T4 is a low power GPU. I think it is reasonable to get the speed you are getting.


True Colors

Now stores color along with UIBezierPath for drawing in multiple colors.  The paths NSArray now contains tuples of UIBezierPath and UIColor like so:
[
    [ path, color],
    ...
]

Modified the example code to include a very simple color toggle.
closes #3
note - the Pods directory needs to be removed from the repo but I'm going to submit another pull request for that with mods to .gitignore to fix this.

Not sure if this library is being maintained, but I'm no longer using my fork in any projects so won't be updating this pull request.   Going to close it.


episode 01: mention code review after step 12

The discussion happened at that point in this workshop and was really useful.  Add a heading for it there (check if it's somewhere else).

I think this has already been done. I will consolidate more but at least the discussion box is there already.


Question: Telecarla works with the latest Carla patch 0.9.12

Hello ,can telecarla works with the carla 0.9.12?

In the current version <IP_ADDRESS> is supported. So far I did not publish/tag versions for 0.9.11 or 0.9.12 as the carla ros bridge does not provide those tags.
I already opened a PR in the ros bridge to support 0.9.12 at https://github.com/carla-simulator/ros-bridge/pull/563 and wait for this to be merged.
But I will provide some changes to support 0.9.12. The changes mainly cover an updates in the sensor json structure for the ros bridge.

Firstly ty for replying too soon.Secondly i make my study for university.We
want to make a carla environment as a server and programming a car to run
inside the carla world,then we want to give the opportunity to an another
client to enter in our world and control a car or pedestrian remote from
his computer via the internet and interact with our car....i read your
paper and i think telecarla is that we are looking for....do you believe
that telecarla is what we need?
Ty in advance!
Στις Τετ, 18 Αυγ 2021, 6:05 μ.μ. ο χρήστης Markus Hofbauer <
@.***> έγραψε:

In the current version <IP_ADDRESS> is supported. So far I did not publish/tag
versions for 0.9.11 or 0.9.12 as the carla ros bridge
https://github.com/carla-simulator/ros-bridge does not provide those
tags.
I already opened a PR in the ros bridge to support 0.9.12 at
carla-simulator/ros-bridge#563
https://github.com/carla-simulator/ros-bridge/pull/563 and wait for
this to be merged.
But I will provide some changes to support 0.9.12. The changes mainly
cover an updates in the sensor json structure for the ros bridge.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/hofbi/telecarla/issues/20#issuecomment-901192176, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AT5UDBWJKRDNFCFE6MDWF2LT5PD25ANCNFSM5CMAZLAA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email
.

Yes, this is the basic usecase telecarla is designed for.
With #21 I provide support for 0.9.12. I will try to merge it today once all checks pass.

thank you a lot

The latest version now works with carla 0.9.12. But you need to use the carla-ros-bridge with the changes from https://github.com/carla-simulator/ros-bridge/pull/563


[WIP] comparison policies

WIP, not yet for merging

superseded by https://github.com/MediaArea/MediaConch_SourceCode/pull/420


[NEW-FEATURE] Implement string localization and interpolation

closes #98

@PTKu Can you please look at this?


Edge detection seems a bit weak.

When in auto mode, the edge detection of pages seems to desperately try to find "letter" sized paper regardless of the actual page size.  Also, it seems to not detect pages well in general.  Perhaps I'm simply missing a configuration option.  🤔

Hey again @humblecoder
Yes, you are talking about the iOS version of the plugin I assume. I noticed this too, edge detection is perfect only when the contrast between the document to be scanned and the surrounding area is really high. That is an inherent defect in the library  we used. It would seem we would have to either find a new open source one with better edge detection or write an entirely new library.
Cheers,
Chris
Neutrinos

does android support edge detection?

@crapthings It does, but it isn't as great as the WeScan ios one. You need a decent amount of contrast between the document to be scanned and the background. For eg a white paper on a black background will perfectly be captured. But white on a cream background has lower accuracy of edge detection.
Cheers,
Chris
Neutrinos

there's a decent android scanner
https://github.com/ctodobom/OpenNoteScanner
will this work better than the old one?

It is an entire app. This might be useful, it could work with some changes to the code base. Although I do not enjoy the fact that the opencv app has to be installed separately. I will have to find a way around that as well. Thanks for sharing this info. Will look into upgrading the android plugin as well, but not in the immediate future.
Also WeScan has a discussion going on about an android version of their scanner. Was hoping to get that in here because they seem to be creating some crazy good software. But no work has been done on it yet.
Cheers,
Chris
Neutrinos

Also to the original poster @humblecoder the 3.x.x version of this plugin uses a new library with great edge detection. Although I must warn you that, it is in its infancy and it uses swift 4 so please bear with me if you run into problems. Will be here to help if you run into problems
Cheers,
Chris
Neutrinos

@ChrisTomAlx awesome!

Does this mean this library uses WeScan for iOS now ?

Hey @MentalGear

Does this mean this library uses WeScan for iOS now ?

Yup. Plugin version 3.x.x uses the latest WeScan cocoapod along with swift version 4.0 as of today.

PS: Thanks for your amazing work this far !

It is an honor to be part of the open source community. Hope you have fun with the plugin!
Cheers,
Chris
Neutrinos

Hey Christy,
Thanks fir the quick reply! Great to have wescan functionality.
As a head’s up: Just checked the readme file. Could it be that it is a bit outdated, as it still describes the irl as ios library.

On 11 Apr 2019, at 07:45, Christy Thomas Alex<EMAIL_ADDRESS>wrote:
Hey @MentalGear
Does this mean this library uses WeScan for iOS now ?
Yup. Plugin version 3.x.x uses the latest WeScan cocoapod along with swift version 4.0 as of today.
PS: Thanks for your amazing work this far !
It is an honor to be part of the open source community. Hope you have fun with the plugin!
Cheers,
Chris
Neutrinos
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Currently the document has been updated only on the github branch ver/3.x.x. Although the 3.x.x version of the plugin is stable, since cordova doesn't work as well as it should with swift, I was waiting to test this on cordova 9.0.0 before pushing this as the stable release of the plugin.
You can refer the ver/3.x.x documentation  on github for now. Will be updating the docs soon. Thanks for bringing this up though :)
Cheers,
Chris
Neutrinos


Parse JSON into org.embulk.spi.json.JsonValue, not org.msgpack.value.Value

As commented in embulk-util-json, we are going to release embulk-parser-json:0.11.1 just with dropping use of org.msgpack.value.Value, but without one-path parsing.

https://github.com/embulk/embulk-util-json/pull/33#issuecomment-1727651903

This pull request includes :

Replacing use of the old JsonParser to new JsonValueParser, with replacing use of org.msgpack.value.Value to org.embulk.spi.json.JsonValue
Testing it with org.embulk.spi.json.JsonValue, without org.msgpack.value.Value
Removing unnecessary dependencies

Each step is tested as an independent commit on CI.
embulk-parser-json:0.11.1 would be released with this, and also Embulk v0.11.1.

Thanks!


Aviation provider: Airport names and more ICAO airport codes.

Added "airport_name" field.
Added more ICAO airport codes.

It would be good to have source info for these PRs.

i wonder whether we should add this notice to PULL_REQUEST_TEMPLATE

@kingthorin -  I took the list from http://www.flugzeuginfo.net/table_airportcodes_country-location_en.php . If you like I could add  that info this PR or the next one.

@snuyanzin thats a great idea, I’ll set something up.

Codecov Report

Merging #774 (4f7d2cd) into main (5fc6904) will decrease coverage by 0.04%.
The diff coverage is 100.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@             Coverage Diff              @@
##               main     #774      +/-   ##
============================================
- Coverage     92.82%   92.79%   -0.04%
+ Complexity     2633     2632       -1
============================================
  Files           284      284
  Lines          5202     5203       +1
  Branches        537      537
============================================
- Hits           4829     4828       -1
  Misses          245      245
- Partials        128      130       +2

Impacted Files
Coverage Δ

...in/java/net/datafaker/providers/base/Aviation.java
100.00% <100.00%> (ø)

... and 2 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

That's a lot of data! I don't mind it, but just curious : are you building something related to airlines or so?

@bodiam - nothing real actually, I am just extending an educational application related to aviation (a flight ticket booking app). But the further it goes the more features I would like to add there.


Added FileRequiredProperties for copying classpath files to container.

what is the justification for this change?

Needed a library file to be present in the container. So, added a Class to read from bootstrap property file the name of the files and locations of them to put into the container.

is this class used by postgre or something?

No. We use a function which is .so file for indexing in postgres. So, these kinds of external files need to be present at the container while migrating database using flyway.

not sure I got the point of this "feature", but anyway change requires unit-test

Ok. Will add unit test for the feature.

@wahid-anwar-m4 could you please also update readme.md with newly added properties and short explanation?
@vasilievip @tdanylchuk guys, i suppose that this feature is rather generic. should we consider placing that in testcontainers-common module?

Thanks @Aloren for the reviews.Will try to address the issues.

@wahid-anwar-m4 do you plan to implement changes or I can close this PR?

Hi Ivan,
I cannot use this account to complete this PR implementation. However, I think this will be helpful for someone needs additional files from system or classpath  on their test containers. I can use my personal account to complete this if  you think this is useful. Just let me know.
Best regards
Wahid

I think it would be nice to have, you may open separate PR, just remember about comments.

OK. I will create a separate PR from my personal account.


Allow alias page to be templated

This is a bug to track work I'm doing to allow the redirect page created for aliases to have templated information in it.  This will function similarly to the 404 page template, i.e. one template for the whole site.  The template should be given the metadata from the page it is aliasing, so you can customize the redirect per-aliased page.

Note that the template will not always have a Page context (it is used for other redirects as well).

Being worked on in #2576


Overlap of Title and Close link in Forum Comment Dialog

The "Add Comment" title on the overlay dialog for adding a comment on a forum topic is overlapping the "X" to close the overlay, making it impossible to tap. See screenshots.
Perhaps we should use native Cancel/OK controls for this rather than a custom item in the dialog?

rewrite is planned. It is currently impossible to support decent anilist forum functions.
I have already asked permission for the design and Josh granted it.

Related to: https://atarashii.freshdesk.com/helpdesk/tickets/16

Closing because of the new design rewrite. Atarashii 2.2.1.


select blockchain in config

fixes address prefixes being non-configurable

tested reg test mode and works fine LGTM