Fixed NIO Bug

Fixed java.nio.channels.CancelledKeyException

Here is the stacktrace:
ERROR 29/02/2012 09:27:43 [ClientRequestExecutorFactory$ClientRequestSelectorManager]/SelectorManager
java.nio.channels.CancelledKeyException
at sun.nio.ch.SelectionKeyImpl.ensureValid(SelectionKeyImpl.java:55)
at sun.nio.ch.SelectionKeyImpl.readyOps(SelectionKeyImpl.java:69)
at java.nio.channels.SelectionKey.isReadable(SelectionKey.java:271)
at voldemort.utils.SelectorManager.run(SelectorManager.java:191)
at java.util.concurrent.ThreadPoolExecutor$Worker.runTask(ThreadPoolExecutor.java:886)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:908)
at java.lang.Thread.run(Thread.java:662)
Since I applied the patch in my production code I haven't seen this anymore.

Hmmm. interesting. I think the connection breaks after the last read/write, before the next select(). Cool.


Join string instead of putting list in URL

Proposed Changes

(Describe the changes and rationale behind them)

Related Issues

(Github link to related issues or pull requests)

Codecov Report

Merging #60 (38c0ff9) into main (0449843) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main       #60   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            6         6
  Lines          325       362   +37
  Branches        42        49    +7
=========================================
+ Hits           325       362   +37

Impacted Files
Coverage Δ

src/youtubeaio/youtube.py
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


Add new compiler flags for VCS compatibility

@mrutt92, this too.

@drichmond cool - use bigblade as your base from now on. I'll rebase this for you.


"ssl_reject_handshake on" disables TLSv1.3

Have you pulled and found the error with jc21/nginx-proxy-manager:latest docker image?

Yes

Are you sure you're not using someone else's docker image?

Yes

Have you searched for similar issues (both open and closed)?

Yes

Describe the bug
If you use any version newer than v2.9.19
TLSv1.3 breaks and is no longer working.
See the change here
See issue ticket https://trac.nginx.org/nginx/ticket/2071
Nginx Proxy Manager Version
v2.9.20+
To Reproduce
Steps to reproduce the behavior:

Install any version v2.9.20+
add a proxy and check the TLS version
maximum is TLSv1.2

Expected behavior
TLSv1.3 should work
Found by @piuswalter and myself.

+1

+1


UI Glitch on soon expiring or already expired certificate screen (Symbol is squeezed together)

Avoid duplicates

[X] Bug is not mentioned in the FAQ
[X] Bug is specific for iOS only, for general issues / questions that apply to iOS and Android please raise them in the documentation repository
[X] Bug is not already reported in another issue

Technical details

Device name: iPhone 6s
iOS version: 15.5
App version: 2.24.1

Describe the bug
There is an UI Glitch / issue on the soon expiring / already expired EU DCC screen. The symbol is squeezed together. Here is a screenshot of the bug under iOS & for comparison one from Android where the symbol is displayed correctly:

iOS (wrong)
Android (right)

Steps to reproduce the issue

Open the CWA
Click on "Certificates"
Click on the person from whom you have a soon expiring or an already expired certificate in the app
Scroll to the very bottom
Click on the certificate that is soon expiring or is already expired
See the symbol which is squeezed together

Expected behaviour
The symbol should not be squeezed together.

Internal Tracking ID: EXPOSUREAPP-13640

Thanks for your report @Ein-Tim. We have created an internal ticket for it and will raise this topic internally.
Internal Tracking ID: EXPOSUREAPP-13640
Unfortunately, I was personally not able to reproduce this on both an iPhone 11 Pro Max (iOS 16 Beta) and an iPhone 8 (iOS 15.5) using the latest Corona-Warn-App versions. Is there any other step that is essential to reproduce this issue?
Corona-Warn-App Open Source Team

@larswmh Maybe this bug is specific to the iPhone 6(s) display aspect ration. I can reliably reproduce this bug as well as on my real iPhone 6s & on an emulator iPhone 6s.
I will see if I can reproduce this on other iPhones using the emulator.

PR https://github.com/corona-warn-app/cwa-app-ios/pull/4690

@Ein-Tim Thanks for mentioning the PR link. The bug has fixed now.

Fixed.


Can this library be used to create a new folder?

I've searched the docs, and looked through the lib itself and don't see any methods for creating a new folder. The dropbox api has an create_folder endpoint, but I couldn't find anything like that in here.
Also, I've tried simply creating new files in a new path, hoping that it would create any necessary folders if they didn't already exist. Doing this:
dropbox.writeFile path, drawing, (error, stat) ->
      if error
        console.log error
      else
        console.log stat

returns a success stat object that looks like:
Stat {
  _json:
   { revision: 92,
     bytes: 4446,
     thumb_exists: true,
     rev: '5c3c5ddbb1',
     modified: 'Wed, 28 Oct 2015 00:28:30 +0000',
     mime_type: 'image/png',
     path: '/2015-44/459d8fc4-2f73-4ba2-be06-eda828ff8990.png',
     is_dir: false,
     size: '4.3 KB',
     root: 'app_folder',
     client_mtime: 'Wed, 28 Oct 2015 00:28:30 +0000',
     icon: 'page_white_picture' },
  path: '/2015-44/459d8fc4-2f73-4ba2-be06-eda828ff8990.png',
  name: '459d8fc4-2f73-4ba2-be06-eda828ff8990.png',
  isFolder: false,
  isFile: true,
  isRemoved: false,
  typeIcon: 'page_white_picture',
  modifiedAt: Tue Oct 27 2015 17:28:30 GMT-0700 (PDT),
  clientModifiedAt: Tue Oct 27 2015 17:28:30 GMT-0700 (PDT),
  inAppFolder: true,
  size: 4446,
  humanSize: '4.3 KB',
  hasThumbnail: true,
  versionTag: '5c3c5ddbb1',
  contentHash: null,
  mimeType: 'image/png' }

but the file and folder do not exist in my dropbox folder?
Any help is appreciated - thanks!

I think mkdir is what you're looking for. (I searched the code for "create_folder" to find this.)
But yes, there's no need to create the folder if you're about to write files into it. The folder will be implicitly created. It looks like the file is being written from your example.
Why do you say the file and folder don't exist? Are you sure you're looking in the right account and the right location? (E.g. if you have an "app folder" permission app, the folder you created will be in ~/Dropbox/Apps/<your app name>/2015-44.)

hmm you're right, the files were being created but my dropbox app crashed and things weren't being synced
slaps forehead
thx :)


FOL module

what is FOL module?
how can I use this module and DoTA dataset to dectect traffic anomaly in my video?

Please read our paper for how to use FOL to detect traffic anomaly, thanks!

@MoonBlvd, Is the source code available for the paper "DoTA: Unsupervised Detection of Traffic Anomaly in Driving Videos" you mentioned above?


Project count is wrong when unloading and loading more projects than previously loaded

I'll preface this by saying that I love this extension, nice job!
To replicate:

Use latest Visual Studio 2022
Have one or more projects loaded

Filter projects to unload previous projects and load more projects than were previously loaded

Notice how the project count has decreased

Additionally, there are some issues when using the extension with Solution Filter files (.slnf) whereby doing the aforementioned loading/unloading will sometimes hide projects after loading them.

Thanks for reporting.
The project count issue is a duplicate of #1. If you do it enough, the project count eventually becomes negative! Unfortunately it appears to be out of my control and the Visual Studio team  have decided not to fix it . 😞

Additionally, there are some issues when using the extension with Solution Filter files (.slnf) whereby doing the aforementioned loading/unloading will sometimes hide projects after loading them.

Would you be able to log a new issue with some more information about this? Ideally with some reproduction steps.
Thanks.

Apologies. I should've checked the closed issues before reporting that first one!


feat: Custom Themes

This PR introduces the ability for Mode Watcher to track custom themes, which are persisted to local storage alongside the mode and are applied to the root element via the data-theme attribute.
You can set a default theme like so:
<ModeWatcher defaultTheme="dracula" />

and the rendered html will look like this:

You can programatically set a theme using the setTheme function:

Select a theme
<button on:click={() => setTheme('retro')}>Retro</button>
<button on:click={() => setTheme('dracula')}>Dracula</button>
<button on:click={() => setTheme('usa')}>Red, White, and Blue</button>

Closes: #66

I would appreciate a review here, @ollema 😃 Once this is merged, planning to add the ability for users to define custom local storage keys, and then we'll release 1.0 before Svelte 5 so that 2.0 is full Svelte 5 support.


Is it supposed to run on Mac?

$ cloudstrap-azure environment
/Library/Ruby/Gems/2.0.0/gems/cloudstrap-azure-0.7.1.pre/command/internal/cloudstrap-azure.environment:85:in `<main>': undefined method `to_h' for #<Array:0x007fcce22287b8> (NoMethodError)

$ ruby -v
ruby 2.0.0p648 (2015-12-16 revision 53162) [universal.x86_64-darwin16]

Yep, sure is. The majority of the development happens on a Mac!
That said, Ruby 2.0.0 reached End of Life on 2016-02-24. If you're on macOS 10.12, 2.0.x is the included version. If you're on macOS 10.13, you should have 2.3.x, which is still supported.
I'd recommend using a stable release via Homebrew (or your package manager of choice):
brew install ruby

Hope that helps!

Thank you, @colstrom, it helps!
It would be honest to note Ruby version in Prerequisites.

Thanks, that's a good idea.


[Bug] [ui] The move task button is missing in the task definition.

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
The move task button is missing in the task definition. After upgrading to 3.1.7, I noticed that the move task button is gone.
Version: 3.1.7
What you expected to happen
Add move task button.
How to reproduce
The move task button is missing in the task definition.

Anything else
No response
Version
3.1.x
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

this button remove by pr #13441

I understand. But how can I move tasks now? Is there an API for it?

I understand. But how can I move tasks now? Is there an API for it?

Hi, @luker629 , task moving api is now deprecated (ref https://github.com/apache/dolphinscheduler/pull/8190#discussion_r791559980).
BTW, if you think feature of reusing task definition is really helpful, you can leave some comments in https://github.com/apache/dolphinscheduler/issues/13513


Auto option to change resolution

How to add Auto option to change resolution, to automatically display video based on bandwidth.

Sadly there's no, easy cross-browser way to do this currently.

Is any other way to implement this ?

Dash.js, hls.js and Shaka player probably supports it, but we don't support switching quality for those libraries yet (I'm working on it).

I managed to implement the switching quality for hls.js (happy to share my code) but I had to hack the compiled source code of plyr (dist).
What I would like to do is to create another class to extend Plyr. But I don't know how to do that without recompiling all the src code.

Please help! Change resolution video quality not keep current playing position

@francoiscop HI Francoiscop, I would love to know how you manage to implement quality swtich for hls.js

@francoiscop I am also very interested to know.

In my case, I have DASH integration (via Shaka because Dash example is broken even on codepen), I'd like to use .mp4 sources, or even better, have .mpd and .mp4 listed inside video  tags with no extra js trickery

You can check how to display multi-resolution mpeg dash source on the following link https://codepen.io/adis0308/pen/bGpQmwr


[sw] Clang-Tidy produces false positives

Description
See clang_tidy_results.txt, which is the output of  ./bazelisk.sh test //quality:clang_tidy_check at a76371a6c309727f0451a3e1cad5011410ef3dfb.
Here's a cherry-picked error about the use of an uninitialized value:
/home/dan_zerorisc_com/.cache/bazel/_bazel_dan/e8b9a1cec6d6126ab719828a176ad974/sandbox/linux-sandbox/1269/execroot/lowrisc_opentitan/sw/device/silicon_creator/rom/rom.c:234:9: error: 1st function call argument is an uninitialized value [clang-analyzer-core.CallAndMessage,-warnings-as-errors]
        sigverify_spx_key_id_get(&ext_spx_key->key), lc_state, &spx_key));
        ^
/home/dan_zerorisc_com/.cache/bazel/_bazel_dan/e8b9a1cec6d6126ab719828a176ad974/sandbox/linux-sandbox/1269/execroot/lowrisc_opentitan/sw/device/silicon_creator/rom/rom.c:532:3: note: Loop condition is false.  Exiting loop
  CFI_FUNC_COUNTER_INIT(rom_counters, kCfiRomMain);
  ^
./sw/device/silicon_creator/lib/cfi.h:118:3: note: expanded from macro 'CFI_FUNC_COUNTER_INIT'
  do {                                            \
  ^
/home/dan_zerorisc_com/.cache/bazel/_bazel_dan/e8b9a1cec6d6126ab719828a176ad974/sandbox/linux-sandbox/1269/execroot/lowrisc_opentitan/sw/device/silicon_creator/rom/rom.c:534:3: note: Loop condition is false.  Exiting loop

[I'm eliding a few hundred lines of static analyzer reasoning...]

The gist of it is that Clang-Tidy flagged rom.c:234 because it thinks &ext_spx_key->key could be undefined.
 const sigverify_spx_key_t *spx_key;
  const sigverify_spx_signature_t *spx_signature;
  uint32_t sigverify_spx_en = sigverify_spx_verify_enabled(lc_state);
  if (launder32(sigverify_spx_en) != kSigverifySpxDisabledOtp) {
    const manifest_ext_spx_key_t *ext_spx_key;
    HARDENED_RETURN_IF_ERROR(manifest_ext_get_spx_key(manifest, &ext_spx_key));
    HARDENED_RETURN_IF_ERROR(sigverify_spx_key_get(
        sigverify_spx_key_id_get(&ext_spx_key->key), lc_state, &spx_key));

I think the issue was that it couldn't see through HARDENED_RETURN_IF_ERROR because it's implemented with launder32().

@luismarques I wrote up a more minimal input that reproduces the uninitialized value warning. Do you think there's a feasible way to improve Clang-Analyzer to "see through" the inline assembly?
For convenience, here's the repro case on Compiler Explorer with a clang-tidy tab on the right: https://godbolt.org/z/Kbd3P853c
And for future proofing, here's the code. I'll have to play with this a little more tomorrow.
// This file is a minimal test case for reproducing the false-positive
// Clang-Analyzer results documented in issue #19412.
//
// https://github.com/lowRISC/opentitan/issues/19412.

typedef int err_t;
const err_t kError = 0xff;
const err_t kOk = 0;

typedef struct foo {
  int data;
} foo_t;

foo_t global_foo = {.data = 42};
int global_decision = 0;

// This function is based on OpenTitan's `launder32()`.
int launder(int val) {
  asm volatile("" : "+r"(val));
  return val;
}

// This macro aborts the calling function when `expr_` is not `kOk`.
#define HARDENED_RETURN_IF_ERROR(expr_) \
  do {                                  \
    err_t err_ = expr_;                 \
    if (launder(err_) != kOk) {         \
      return err_;                      \
    }                                   \
  } while (0)

#define SOFTENED_RETURN_IF_ERROR(expr_) \
  do {                                  \
    err_t err_ = expr_;                 \
    if (err_ != kOk) {                  \
      return err_;                      \
    }                                   \
  } while (0)

err_t process(int val);

err_t maybe_initialize_foo(const struct foo **foo) {
  if (global_decision) {
    *foo = &global_foo;
    return kOk;
  }
  return kError;
}

// This function reproduces the false positive from Clang-Analyzer documented in
// issue #19412: https://github.com/lowRISC/opentitan/issues/19412.
//
// [...]sw/device/lib/base/minimal.c:58:36: error: Access to field 'data' results in a dereference of an undefined pointer value (loaded from variable 'foo') [clang-analyzer-core.NullDereference,-warnings-as-errors]
//   HARDENED_RETURN_IF_ERROR(process(foo->data));
//                                    ^
err_t repro(void) {
  const struct foo *foo;
  HARDENED_RETURN_IF_ERROR(maybe_initialize_foo(&foo));
  // If control reaches this point, `foo` is initialized.
  HARDENED_RETURN_IF_ERROR(process(foo->data));
  return kOk;
}

// This function does not reproduce the false positive.
err_t no_repro(void) {
  const struct foo *foo;
  SOFTENED_RETURN_IF_ERROR(maybe_initialize_foo(&foo));
  // If control reaches this point, `foo` is initialized.
  SOFTENED_RETURN_IF_ERROR(process(foo->data));
  return kOk;
}


Version History Path Mismatch

Title: Mismatch of version_history page for v1.17.1 from Home page of Envoyproxy wrt github path
Description:

Home page of envoyproxy "https://www.envoyproxy.io/docs" there is no direct link for v1.17.1. it is through github ref "https://github.com/envoyproxy/envoy/releases/tag/v1.17.1".
Is there any change in doc root/ structure ?

[optional Relevant Links:]

Any extra documentation required to understand the issue.

@ankatare We can see v1.17.1 link on https://www.envoyproxy.io/docs. Would you mind describing this problem for more detail?

This was just fixed by @moderation

@mattklein123 thanks for clarifying it


Allow tests for httpd-ex and RHEL-9

Signed-off-by: Petr "Stone" Hracek<EMAIL_ADDRESS>

[test]

[test]


Add a --clippy option

Right now, there are two main interfaces for running fixes:

`cargo fix [--tests|--benches|--lib]
cargo clippy --fix

It's weird that they are so different - I would expect this to be a cargo fix --clippy option. Would it be possible for rustfix to add support so we can unify the interface?
cc @rust-lang/clippy

Aren't those flags completely handled by cargo?
I seem to remember that we had this discussion before in the cargo repo. I think it was in the context of making the clippy subcommand a first class citizen of cargo instead of having it as an external tool that adds this subcommand (which was rejected back then, because Clippy doesn't have to be installed). Having Clippy as a first class citizen would have had the advantage that cargo could also handle things like cargo fix --clippy. But since Clippy has to register itself as a subcommand, we also had to implement the --fix flag on our own.
This is the PR were this discussion took place: https://github.com/rust-lang/cargo/pull/7382 (warning: it's really long!)

Thanks for the link! Yes, that would be a cargo change. I commented on the PR :)

Closing as this isn't related to the rustfix library. I commented on the linked PR for the reasoning.


[Documentation]: Install Migraphx for radeon gpus  and <ERROR: Could not find a version that satisfies the requirement onnxruntime-rocm>

Description of errors
Hello, theses two lines:
echo 'export PATH=$PATH:/opt/rocm-6.2.3/bin' >> ~/.bashrc
source ~/.bashrc

needs to be added in the documentation here https://rocm.docs.amd.com/projects/radeon/en/latest/docs/install/native_linux/install-migraphx.html after the command "sudo apt install migraphx" or else the command "migraphx-driver perf --test" will not be found.
another important error is this one: "ERROR: Could not find a version that satisfies the requirement onnxruntime-rocm" after typing the command
pip3 install onnxruntime-rocm -f https://repo.radeon.com/rocm/manylinux/rocm-rel-6.0/
because the right command is this one:
pip3 install onnxruntime-rocm -f https://repo.radeon.com/rocm/manylinux/rocm-rel-6.2.3/

You can modify the second error in this webpage: https://rocm.docs.amd.com/projects/radeon/en/latest/docs/install/native_linux/install-migraphx.html#install-migraphx-for-onnx-runtime
Attach any links, screenshots, or additional evidence you think will be helpful.
No response

Hi @WareZTv, thanks for pointing this out. This will be fixed shortly.

no problem, it helps everyone including me so i'm happy to share the issues here

Both issues have been addressed over at Install MIGraphX for ONNX Runtime. It was decided to reference the absolute path for migraphx-driver for simplicity as it's a simple check to verify the installation. Could you please confirm that the instructions work on your end and close out the ticket?

@harkgill-amd  Works perfectly fine now. Something related to the subject but is less important is this. Under the "Verify MIGraphX installation for ONNX Runtime" section in the same webpage: https://rocm.docs.amd.com/projects/radeon/en/latest/docs/install/native_linux/install-migraphx.html#verify-migraphx-installation-for-onnx-runtime, if i copy paste

first this line:

python3

then these two lines:
import onnxruntime as ort
ort.get_available_providers()

then the result is as expected:
>>> import onnxruntime as ort
>>> ort.get_available_providers()
['MIGraphXExecutionProvider', 'ROCMExecutionProvider', 'CPUExecutionProvider']

But if i copy paste the 3 lines at the same time only
python3
is taken into account so i have to copy paste the two other lines again which can be destabilizing for those not comfortable with the python console
so maybe this command will be more appropriate for the tutorial:
python3 -c "import onnxruntime as ort; print(ort.get_available_providers())"

or you can separate the first line from the two others lines, as you prefer

Anyway thanks a lot for being responsive. Not many companies care about their customer like you do

python3 -c "import onnxruntime as ort; print(ort.get_available_providers())"

The docs have been updated to use this command. Thanks!

your welcome 👍


Random crash on hashtables?

10,000,000 iota [ dup ] H{ } map>assoc

erg@jobim ~/factor $ [master*] ./Factor.app/Contents/MacOS/factor
fatal_error: Memory protection fault during gc: 0x10b04
Abort trap: 6

Confirmed on Mac OS X 64-bit.

IN: scratchpad : foo ( -- ) 10,000,000 iota [ dup ] H{ } map>assoc drop ; foo
assertion "blocki != all_blocks.begin()" failed: file "vm/code_heap.cpp", line 121

Program received signal SIGABRT, Aborted.
0x00007fff80b3bda6 in __kill ()
(gdb) where
#0  0x00007fff80b3bda6 in __kill ()
#1  0x00007fff80bacf89 in abort ()
#2  0x00000001000019d0 in factor::abort () at vm/os-unix.cpp:555
#3  0x0000000100016516 in factor::code_heap::code_block_for_address (this=0x100217a10, address=4295186487) at vm/code_heap.cpp:121
#4  0x000000010001cb61 in factor::factor_vm::iterate_callstack<factor::call_frame_code_block_visitor<factor::code_compaction_fixup>, factor::code_compaction_fixup> (this=0x100807800, ctx=0x11291f190, iterator=@0x100400c70, fixup=@0x100400d58) at callstack.hpp:58
#5  0x000000010001cc37 in factor::factor_vm::iterate_active_callstacks<factor::call_frame_code_block_visitor<factor::code_compaction_fixup>, factor::code_compaction_fixup> (this=0x100807800, iter=@0x100400c70, fixup=@0x100400d58) at vm.hpp:188
#6  0x000000010001cc8a in factor::code_block_visitor<factor::code_compaction_fixup>::visit_context_code_blocks (this=0x100400d50) at code_block_visitor.hpp:114
#7  0x000000010001a19d in factor::factor_vm::collect_compact_code_impl (this=0x100807800, trace_contexts_p=true) at vm/compaction.cpp:324
#8  0x000000010001a2ca in factor::factor_vm::collect_growing_heap (this=0x100807800, requested_size=0, trace_contexts_p=true) at vm/compaction.cpp:360
#9  0x000000010002d110 in factor::factor_vm::collect_full (this=0x100807800, trace_contexts_p=true) at vm/full_collector.cpp:117
#10 0x0000000100030946 in factor::factor_vm::gc (this=0x100807800, op=factor::collect_nursery_op, requested_size=0, trace_contexts_p=true) at vm/gc.cpp:181
#11 0x0000000100030c19 in factor::factor_vm::primitive_minor_gc (this=0x100807800) at vm/gc.cpp:276
#12 0x0000000100040385 in primitive_minor_gc (parent=0x100807800) at vm/primitives.cpp:11

@jckarter Running the test with a large enough tenured space upfront such that it doesn't have to grow fixes the crash.

Bootstrapping and running (both commands) with -pic=0 seems to fix it.

Falls under #460


Updated anaconda removal from PATH

Older version does not account for multiple anaconda paths to different environments. Ex: anaconda3/envs/env_name/<bin,include,...> being there in addition to the default environment

thanks!


Issue generating model definitions

I'm seeing some odd behavior when using the plugin (3.1.4) to generate my swagger file.
In one controller, I have two methods.  Method A returns List.  Method B returns List.
Both methods show up in the swagger file.  The definition for Object A shows up but the definition for Object B does not.  Both objects are part of the same package.  Neither Object has the @ApiModel annotation on them (adding the annotation does not fix the issue).  I have no idea why the plugin is having issues with the Object B model class and not with Object A.  Any pointers on how to debug this would be appreciated.

Finally found the issue and a fix.  In the above, when Object B was not showing up in the swagger definitions, it was because it only ever appeared in a List.  Whereas Object A was sometimes in a List and sometimes a single instance of it was returned.
Since Object B only appeared in generics, it seems like type erasure was preventing the plugin from defining the object.
For each undefined object, I added the following Swagger Core annotation to the service method: @ApiResponses(value = {@ApiResponse(code = 200, message = "Success", responseContainer = "List", response = ObjectB.class
This solved my project.


fix(crit/stats): add const for dump and restore

Closes #92
PTAL @adrianreber @rst0git

Codecov Report

Merging #93 (be1c695) into master (c54aaf7) will not change coverage.
The diff coverage is n/a.

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


Update PIP please

Tool information

Tool name: PIP
Update
Desired version: 21.1
Approximate size:

Area for Triage:

Question, Bug, or Feature?:

Starting this week, I start seeing this error message on our pipelines:
##[error]WARNING: You are using pip version 21.0.1; however, version 21.1 is available.
Log here.
Virtual environments affected

[ ] Ubuntu 16.04
[ ] Ubuntu 18.04
[ ] Ubuntu 20.04
[ ] macOS 10.15
[ ] macOS 11.0
[ ] Windows Server 2016 R2
[ ] Windows Server 2019

Can this tool be installed during the build?

Tool installation time in runtime

Are you willing to submit a PR?

Hi @josesimoes!
The new image with pip 21.1 will be deployed today. However, this warning shouldn't fail the pipeline, it seems that you have a non-default task configuration. If it's necessary to have the task modified you can add the following command as a very first line of the step to make sure pip is updated:
python -m pip install --upgrade pip

Hey @miketimofeev !
Thank you very much for the prompt reply and, of course, for letting me know that the update will roll out today.
As for your suggestions on how to workaround this, I've added them to the yaml. Thanks for the review. 😃 👍🏻

@josesimoes the windows images have been updated.
I'm going to close the issue. Please feel free to contact us if you have any concerns.
Thank you!


Provide python-like os.path.expanduser and os.path.expandvars

I naively use ~/tmp_output or similar a lot across commands.
But when I try with Dart commands, I end up creating a '~' directory in the working directory -- hilariously followed by a quite naive 'rm -rf ~'. A really great morning.
It'd be good to make it trivial to use standard path notation in the shell.
Python's implementations look like great inspiration
https://docs.python.org/2/library/os.path.html#os.path.expanduser
https://docs.python.org/2/library/os.path.html#os.path.expandvars
This is related to https://code.google.com/p/dart/issues/detail?id=2930 for VM command line options
Perhaps we could hack this together in pkg/path -- but I feel it belongs in dart:io. Platform, perhaps?

I'm calling this a P3 because it is possible for Dart developers to implement this themselves or for it to be implemented as a separate package.

Ran into this again today.  It feels like something from an official Dart library should provide some way to expand ~ in a path since on Mac/Linux that's a common shorthand.  But maybe it's not as common as I think?


Prepare 2.0.0 release (Part 1)

Implemented ChatMessageContent to encapsulate the representation of content parts in ChatMessage, ChatCompletion, and StreamingChatCompletionUpdate.
Changed the representation of function arguments to BinaryData in ChatToolCall, StreamingChatToolCallUpdate, ChatFunctionCall, and StreamingChatFunctionCallUpdate.
Renamed OpenAIClientOptions's ApplicationId to UserAgentApplicationId (commit_hash)
Renamed StreamingChatToolCallUpdate's Id to ToolCallId
Renamed StreamingChatCompletionUpdate's Id to CompletionId
Replaced Auto and None in the deprecated ChatFunctionChoice with CreateAutoChoice() and CreateNoneChoice()
Replaced the deprecated ChatFunctionChoice(ChatFunction) constructor with CreateNamedChoice(string functionName)
Renamed FileClient to OpenAIFileClient and the corresponding GetFileClient() method in OpenAIClient to GetOpenAIFileClient().
Renamed ModelClient to OpenAIModelClient and the corresponding GetModelClient() method in OpenAIClient to GetOpenAIModelClient().

ChatMessageContent triggers a conflict with our ChatMessageContent type, this change causes an very big impact on all of our customers.


YF Migration

In the event that a user has locked LP tokens for a V1 pool, and that pool has a migration path to a matching V3 pool, we supply an option for the user to easily migrate their liquidity over, and re-lock the resulting LP tokens and pair change back into the YF contract.

Rebuilt here: https://github.com/SundaeSwap-finance/sundae-sdk/pull/46


⚠ Bofrost.at has degraded performance

In 62468ad, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1569 ms

Resolved: Bofrost.at performance has improved in 8602e39.


panic at attempt to multiply with overflow when doing math on Decimal128 columns

Describe the bug
A clear and concise description of what the bug is.
When changing the TPC-H tests to use Decimals, getting panics with this error:
thread 'sql::subqueries::tpch_q17_correlated' panicked at 'attempt to multiply with overflow', datafusion/physical-expr/src/expressions/binary/kernels_arrow.rs:359:61

To Reproduce
Steps to reproduce the behavior:
Expected behavior
A clear and concise description of what you expected to happen.
Additional context
Add any other context about the problem here.

When fixed, pls un-ignore test tpch_q17_correlated() in core/tests/sql/subqueries.rs


[Web] Arabic text rendering issue on a web platform

I am experiencing an issue with the rendering of Arabic text on a Web platform. The text appears to be rendered incorrectly or inconsistently. However, it works as expected on the Windows platform.
Steps to Reproduce

Run the below application on the web platform.
Check the rendering of the Arabic text on web platform.

Expected results:
Note that the output of the same application on the Windows platform is as follows:

Actual results:
Here is the output of the application on the web platform.

Code sample
import 'package:flutter/material.dart';

void main() {
  runApp(MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(primarySwatch: Colors.blue),
      home: const DemoSample()));
}

class DemoSample extends StatefulWidget {
  const DemoSample({super.key});

  @override
  State<DemoSample> createState() => DemoSampleState();
}

class DemoSampleState extends State<DemoSample> {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
        appBar: AppBar(title: const Text(' Flutter Sample')),
        body: const Center(
          child: SizedBox(
            height: 16,
            child: Text('إظهار الصفوف حيث',
                style: TextStyle(fontWeight: FontWeight.bold, fontSize: 14)),
          ),
        ));
  }
}

Logs
[!] Flutter (Channel stable, 3.7.7, on Microsoft Windows [Version 10.0.19044.2486], locale en-US)
    ! Warning: `dart` on your path resolves to C:\Dart\dart-sdk\bin\dart.exe, which is not inside your current Flutter SDK
      checkout at D:\Flutter\flutter. Consider adding D:\Flutter\flutter\bin to the front of your path.
[√] Windows Version (Installed version of Windows is version 10 or higher)
[√] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
[√] Chrome - develop for the web
[√] Visual Studio - develop for Windows (Visual Studio Professional 2022 17.4.3)
[√] Android Studio (version 2021.3)
[√] Connected device (4 available)
[√] HTTP Host Availability

Hello @Tamilarasan-Paranthaman. Removing or increasing the height of the SizedBox fixes this issue. You have too little height for the text to render properly. This is working as expected. Since this is not an issue, I will close this issue. Thank you

Okay I see the differences. This might be a regression of https://github.com/flutter/flutter/issues/64792
cc: @mdebbar


Email address already registered

I keep getting the message email is already registered even though the email entered has never been invited before.

Could you share the server log when running it? Thanks 👍

2017-05-03T18:18:51.017182+00:00 app[web.1]: Processing by InviteController#submit as HTML
2017-05-03T18:18:51.017244+00:00 app[web.1]:   Parameters: {"utf8"=>"✓", "authenticity_token"=>"v7+xxxxxxx==", "first_name"=>"Jennie", "last_name"=>"Slow"<EMAIL_ADDRESS>"commit"=>"Get Beta Access"}
2017-05-03T18:18:51.027714+00:00 app[web.1]: Creating a new tester<EMAIL_ADDRESS>- Jennie Slow
2017-05-03T18:18:52.384600+00:00 app[web.1]: Tester<EMAIL_ADDRESS>exists in iTunesConnect but not on application
2017-05-03T18:18:52.384857+00:00 app[web.1]: Addding tester to application in group(s): ["External Testers"]
2017-05-03T18:18:54.492505+00:00 heroku[router]: at=info method=POST path="/submit" host=arrivalreq.herokuapp.com request_id=09353b85-4d5e-47be-bb45-3c0fd0c7f118 fwd="<IP_ADDRESS>" dyno=web.1 connect=1ms service=3486ms status=200 bytes=2135 protocol=http
2017-05-03T18:18:54.475498+00:00 app[web.1]: Done
2017-05-03T18:18:54.482584+00:00 app[web.1]:   Rendered invite/index.html.erb within layouts/application (2.7ms)

the user gets properly added. it's just that even though its successful, I get the email already exists everytime.

I'm having a difficult time repro'ing this. One way I am able to repro is by doing the following:

Add tester to app (default external tester)
Remove tester from app
Add tester (get message)

This is expected now, though. Since the new TestFlight changes, testers get added to a pool of testers when you add them to your app. When you remove the tester from the app, they still remain in the pool of testers un "Users and Roles"
So even if they don't show up in the App's external testers, they are still associated with your account.
One really good way to test this is to create a new tester using a gmail account like I did:
my email is taquitos@gmail, but I created a new tester of taquitos+josh1@gmail (you probably know, anything past the + is ignored by gmail and gets forwarded to taquitos@gmail) and everything worked as expected:
Started POST "/submit" for ::1 at 2017-05-04 12:29:00 -0700
Processing by InviteController#submit as HTML
  Parameters: {"utf8"=>"✓", "authenticity_token"=>"RbkWk0lrlxAyMrR4zCFGctc3XMNobxUDNmYSPoqS3GLxWdan2Pz3YouNye3ukBLUJD3NxYboEI4jIHbr8uNsOQ==", "first_name"=>"Josh", "last_name"=>"Test2"<EMAIL_ADDRESS>"commit"=>"Get Beta Access"}
Creating a new tester<EMAIL_ADDRESS>- Josh Test2
Successfully created tester<EMAIL_ADDRESS>Addding tester to application in group(s): ["External Testers"]
Done

From my recent experience doing tests: If you remove a user from the APP section of iTunesConnect, the user is not actually removed! You can still see her/him in the USERS & ROLES/External Testers of iTunesConnect.
The only way to completely remove testers from iTunesConnect (in order to be able to add them later) is to do it from the USERS & ROLES and not from the APP section!
Now, what does this entail to as far as Fastlane Boarding/Pilot ... .

I've been adding completely new users and still get the duplicate message.
I think it has to do with how the controller code and how it's handling the
@message response.
On Sun, May 7, 2017 at 10:18 AM Arshia Cont<EMAIL_ADDRESS>wrote:

From my recent experience doing tests: If you remove a user from the APP
section of iTunesConnect, the user is not actually removed! You can still
see her/him in the USERS & ROLES/External Testers of iTunesConnect.
The only way to completely remove testers from iTunesConnect (in order to
be able to add them later) is to do it from the USERS & ROLES and not
from the APP section!
Now, what does this entail to as far as Fastlane Boarding/Pilot ... .
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/fastlane/boarding/issues/146#issuecomment-299689895,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AByJPjcdEeg6cAYVcEXwFyqfHLVOjOfMks5r3X5IgaJpZM4NPpjU
.

I'm getting this too. It seems rather obvious that the code in boarding_service.rb treats this as a message worth relaying the user, even though it's not a big deal (and, as observed above, expected).
I think the message should only be "Email address is already registered" if the tester exists and was already added to external testers
It's perfectly plausible that the user (or admin) removed him/herself from the external testers group., or that the user is signing up for a beta on a different app under the same developer account.

Also getting the above - seems to be saying the user is already registered


Data contributors want to demonstrate the impact of their research and submissions

Data contributors want to privately receive metrics for downloads or explorations of their datasets to share for reporting on grants.
With appropriate permissions during submission, it would also be possible to annotate datasets with public metrics. There are portals such as openneuro which include public counts for downloads, views, and watching:

Data contributors may want embedded "smart" citations for their datasets to demonstrate the influence of their publication and research. Data consumers may want to use "smart" citations to find datasets with high reputation or impact.

scite.ai
plumX

This is probably correct (although I haven't heard any cellxgene contributors bring it up yet). But there's always a huge grain of salt when looking at download & click counts for datasets because they can mean many different things. For example, downloads can be artificially increased by datasets being used as examples in popular tutorials or they have happen to be the one dataset that I always use when I need a sample dataset.

I like the proposal and think this will be particularly important for groups funded by NIH who want to demonstrate the utility of their resource. It may also help us understand how use in cellxgene compares to other resources. One question that I have is whether the metrics will be made visible on the site or just collected and shared with authors privately. I can see pros/cons to both and the value may be worth clarifying with contributors and users.

Raised again by @jonahcool on #single-cell-analytics.


First rev of Pinpoint the items/docs sample

#446

Hey @kasperbolarsen, thank you 👏 again for another cool sample. This should be live shortly 😊


Ensure that metric hooks are added idempotently.

This also removed the deprecated Playground#define method as it contributes to the confusion.
This should close #232.

[ ] Needs to ensure that it still can add multiple hooks in the case of two simultaneous experiments using the same metric.

when is this supposed to close?

I'm not sure I understand the question?
On Thursday, January 15, 2015, David Guthu<EMAIL_ADDRESS>wrote:

when is this supposed to close?
—
Reply to this email directly or view it on GitHub
https://github.com/assaf/vanity/pull/234#issuecomment-70167880.

@phillbaker sorry I meant when is this going to merge into master, I'm noticing the same behavior.

@davidguthu have to fix the last comment in https://github.com/assaf/vanity/issues/232#issuecomment-57104758, pulled that into a checkbox in the PR description. I actually have some time tonight, I'll see if I can knock that out.

Why did dealing with multiple metrics get included with this fix?

@davidguthu the comment was about re-using the same metric from different experiments.
Regardless, went with a simplified approach that removes the redundant call to Experiment#save.


Any plans for netbird?

I am moving away from tailscale to netbird, as they use the real kernel mode wireguard for better performance and easier self-hosting,
any plans to redo all of your tailscale work for netbird? they also offer arm64 binaries on github :)
https://github.com/netbirdio/netbird/releases

Hi @0xFEEDC0DE64, I'm afraid I currently don't have any plans to do so - primarily because I dogfood this code and test all changes myself before releasing it (and have no personal plans to move away from Tailscale for the time being). That said, I suspect you'll find that most of the logic in this repository is relatively easy to adapt to the Netbird use case and I'd be happy to help you bootstrap a repository doing the same if you're happy to test and maintain it.
On the topic of kernel mode wireguard, I believe Tailscale does offer that on many of their platforms, however I ran into issues with it on the UDM which led to us using userspace networking here.


Crash when clicking on read_text_file when no file is selected

Environment:
Windows 7
Python 3.9 (backport)
Steps to reproduce:

Open a new file
Add a read_text_file node
Click on the file preview

The application crashes to the desktop.
Error message:
Traceback (most recent call last):
  File "C:\Python39\lib\runpy.py", line 197, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "C:\Python39\lib\runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "d:\nodezator\nodezator\__main__.py", line 95, in <module>
    parse_args_and_execute_main()
  File "d:\nodezator\nodezator\__main__.py", line 89, in parse_args_and_execute_
main
    main(filepath)
  File "d:\nodezator\nodezator\__main__.py", line 56, in main
    run_app(filepath)
  File "d:\nodezator\nodezator\mainloop.py", line 209, in run_app
    raise err
  File "d:\nodezator\nodezator\mainloop.py", line 98, in run_app
    loop_holder.handle_input()
  File "d:\nodezator\nodezator\ourstdlibs\collections\general.py", line 20, in _
_call__
    item()
  File "d:\nodezator\nodezator\winman\states\loadedfile.py", line 106, in loaded
_file_event_handling
    self.loaded_file_on_mouse_release(event)
  File "d:\nodezator\nodezator\winman\states\loadedfile.py", line 506, in loaded
_file_on_mouse_release
    obj.on_mouse_release(event)
  File "d:\nodezator\nodezator\graphman\callablenode\vizop\main.py", line 87, in
 on_mouse_action
    method(event)
  File "d:\nodezator\nodezator\widget\pathpreview\base.py", line 362, in on_mous
e_release
    self.preview_paths()
  File "d:\nodezator\nodezator\widget\pathpreview\text.py", line 140, in preview
_paths
    text = path.read_text(encoding="utf-8")
  File "C:\Python39\lib\pathlib.py", line 1266, in read_text
    with self.open(mode='r', encoding=encoding, errors=errors) as f:
  File "C:\Python39\lib\pathlib.py", line 1252, in open
    return io.open(self, mode, buffering, encoding, errors, newline,
  File "C:\Python39\lib\pathlib.py", line 1120, in _opener
    return self._accessor.open(self, flags, mode)
PermissionError: [Errno 13] Permission denied: '.'

Some form of error handling would be nice. Ideally an application should never crash to the desktop.

It maybe a windows problem?
Under Linux it works

Mr. Banbury,
Thank you for your detailed bug report, it is much appreciated.
Just wanted to let you know I'm aware of your issue and will solve it asap.
ETA for a solution, including its upload to PyPI, is a couple of days. It may be much earlier, though, since I suspect this problem will only take a few minutes to solve.
Peace

Also, I forgot to thank you, Mr. @dh1ao, for taking the time to verify it for me on linux. This donation of time is very selfless on your part. Thank you a lot!

Mr. Banbury,
Just fixed the bug. The new 1.3.11 version with the bugfix was already pushed to github and uploaded to PyPI. To access this latest version with the bugfix just upgrade your installation with the command:
pip install --upgrade nodezator
Could you please, confirm that the problem was fixed in your machine so that I can close this issue?
When following the steps you described to reproduce the error, instead of the app crashing on you, a dialogue should appear communicating the error, just like in the image below:

Mr. Banbury,
Since the issue is already solved (the bug is fixed) and we waited for your confirmation for 10 calendar days already, I'm closing the issue now.
Don't worry about not having replied. We just ask for confirmation to make sure the changes really took effect on the user's end.
If users don't reply, I just assume they were too busy at the time or had their reasons.
Peace.


[BEAM-2926] Add initial support to be able to read singleton and iterable side inputs to Java SDK harness.

Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Make sure there is a JIRA issue filed for the change (usually before you start working on it).  Trivial changes like typos do not require a JIRA issue.  Your pull request should address just this issue, without pulling in other changes.
[x] Each commit in the pull request should have a meaningful subject line and body.
[x] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Run mvn clean verify to make sure basic checks pass. A more thorough check will be performed on your pull request automatically.
[x] If this contribution is large, please file an Apache Individual Contributor License Agreement.

R: @kennknowles

retest this please

Coverage increased (+0.06%) to 69.604% when pulling c49c564daa2a32d7930b819175e77d414e617419 on lukecwik:fn_api2 into 2090ee324753604b4adaaaefa835ee09ee362510 on apache:master.

retest this please

rebased against master to resolve merge conflicts

Coverage increased (+1.0%) to 69.584% when pulling d1c0ffde59d0b17126deecb840cbeec36719d7a6 on lukecwik:fn_api2 into 92615ca4c4e09bc63c37a933102035040c23aac9 on apache:master.

Run Java PreCommit


Add generics to ContainerInterface (new proposal)

Inspired by #44
I've tried @nicolas-grekas' proposal but I do not have autocompletion without adding "@param" annotation.

Now works well with PHPStan https://phpstan.org/r/03c35ee8-10f5-490c-a263-37855651e35d
And Psalm https://psalm.dev/r/8b5c98aa39

I'm a big fan of generics and I think a container that implements this functionality should feel free to apply this to their container implementation. I think applying them here however doesn't quite make sense and goes against the spirit of this PSR.
For one PSR-11 says:

An entry identifier is an opaque string, so callers SHOULD NOT assume that the structure of the string carries any semantic meaning.

Which to me heavily implies that a class string key should not automatically result in an instance of that class. By typehinting ($id is class-string<T> ? T : mixed) we are requiring the return type to be an instance of a class if the key looks like a class string and preventing valid usage like ->get('\Exception') // 'Exception is the base class for all user exceptions'.  ($id is class-string<T> ? T|mixed : mixed) could work instead, but I still think that goes against the quoted text by implying there's some potential structure to keys.
What's more, PSR-11 recommends:

Users SHOULD NOT pass a container into an object so that the object can retrieve its own dependencies. This means the container is used as a Service Locator which is a pattern that is generally discouraged.

And I'd argue having generics on the base get method implies the opposite.


In-line error messages

Description of feature / user story
Currently, the wallet displays pop-up error messages. We propose showing in-line error messages to the users to help them easily find the field containing the error.
Acceptance Criteria
Given when the user is required to fill a field or choose from a list of options in the app
And the criteria for the UI component (field, radio button, checkbox etc) hasn't been satisfied
Then the user sees error messages above the component
AND
When the user fulfills the criteria
Then the error message automatically goes away
And the user is allowed to save changes or move forward to the next step in the app
Wireframes or relevant image assets / links

Attached image is just for reference purposes.
The text will need to be customizable to enable the developers to use this on components wherever applicable.

@MalikJumani-OPS  Has your issue already been escalated?

@laalauren1 not yet

@MalikJumani-OPS can you give more details what app are you trying to set up or trying to logging in

@laalauren1 this is a feature request.

@ okay what app is that ?


go-ethereum: 1.10.21 -> 1.10.22

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/go-ethereum/versions.
meta.description for go-ethereum is: "Official golang implementation of the Ethereum protocol"
meta.homepage for go-ethereum is: "https://geth.ethereum.org/"
meta.changelog for go-ethereum is: ""
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
Warning: a test defined in passthru.tests did not pass
found 1.10.22 in filename of file in /nix/store/b6sj04skl06airb28zgh7fxd6q3bgbcf-go-ethereum-1.10.22

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath
bee-clef
go-ethereum

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/b6sj04skl06airb28zgh7fxd6q3bgbcf-go-ethereum-1.10.22 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A go-ethereum https://github.com/r-ryantm/nixpkgs/archive/5198016b1cc4f5f72a8223bc4c5736864f496e51.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/b6sj04skl06airb28zgh7fxd6q3bgbcf-go-ethereum-1.10.22
ls -la /nix/store/b6sj04skl06airb28zgh7fxd6q3bgbcf-go-ethereum-1.10.22/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

2 packages built:

bee-clef
go-ethereum

Maintainer pings
cc @adisbladis @lionello @RaghavSood for testing.

Do NOT merge this. This release contains a bug which corrupts already synced state, see the release notes:

WARNING: This release seems to contain a regression that can corrupt the local state. Please hold on with updating while we investigate the issue!

https://github.com/ethereum/go-ethereum/releases/tag/v1.10.22
Instead we should upgrade straight to 1.10.23 which fixes that bug.

And actually, it would be good to get 1.10.23 release in quickly because The Merge is coming up:
https://blog.ethereum.org/2022/08/24/mainnet-merge-announcement/
People should upgrade to 1.10.23 before 6th of September.

@jakubgs I'm trying to package 1.10.23, please chime in on Discourse if you've got any tips.


[Telegram] Getting error: 'timed out after 15s' when sending to Telegram channel?

App ID: 62516114-227c-436c-8e55-e7dcf50e7474
SDK Platform: .NET
SDK Version: 3.11.0
Active Channels: skype, telegram
Deployment Environment: aws

Issue Description
Message 'POST to nowinfinity_assistant timed out after 15s' in reply in Telegram channel.
https://ibb.co/kN3osb
In Skype all works well
Reproduction Steps

add bot to contacts
send 'abn for Abbott'

Expected Behavior
no messages like 'POST to nowinfinity_assistant timed out after 15s' in reply to user activity
Actual Results
https://ibb.co/kN3osb

+1
Same error on receiving attachment in telegram
POST to berend_botje timed out after 15s


Add Java support to entryset_proto. [phab:D1909]

Created by fromberger at 2017-08-28 20:24:36:

schroederc accepted this revision at 2017-08-28 20:26:10

fromberger closed via commit 3dc325e901623e75a6376b82ae5d025186aa279b at 2017-08-28 20:29:10


"View all Spaces" returns random subsets of a user's spaces

I first started seeing this issue on July 3 - when a user views all spaces, random subsets of that user's spaces are seen:

This is a f8-ui issue, not planner. But possible caused by no pagination support in the UI and random returned order from server.

This issue was moved to fabric8-ui/fabric8-ui#1626

Deleting spaces does not affect this bug - random subsets of the remaining spaces are seen.


Check that terms and rules are connected

All terms are defined in a rule (probably) so they should be connected somehow
e.g. Negative levels + the rules for negative levels

Currently a Term has a related Rule that spawned this term


ValueError when using --resume_from [previous run]

When I am using --resume_from to branch off a former run of the ESMValTool it processes the diagnostic, but throws a ValueError shortly before the run is finished.
E.g.:
ValueError: '/storage/fran_fr/esmvaltool_output/1d_monitor_20230425_090733/preproc/hus_profile/hus/EMAC_EMAC_E5RF-2000-03_ECHAM5-selvar-p-mm_Amon_hus_2000-2009.nc' is not in the subpath of '/storage/fran_fr/esmvaltool_output/1d_monitor_20230425_091522' OR one path is relative and the other is absolute.
I might be wrong, but it seems to me it happens when the recipe output is written (some index.html file). Everything else works just fine (plots are generated), however the error at the end puzzles me a bit. Any suggestion how that can be solved?
main_log_debug.txt

Hi @FranziskaWinterstein 👋 I noticed this (closed) ESMValCore issue discusses this exception: https://github.com/ESMValGroup/ESMValCore/issues/1154. I wonder whether it would be worth re-opening the ESMValCore issue based on your use case?


why does no one love me

:(

Bruh

sending you a hug 2 make u feel better, meanwhile i wait for an answer.


bug canceling dataset which has LET/D variables

Reported by ansley b manke on 18 Apr 2013 22:14 UTC
This goes back a long way. Supposedly, LET/D variables are deleted when the dataset they are attached to is cancelled. But grids are still around even though the variable is not defined.
The symptom is that after the dataset is canceled, the axes or vestiges of axis info is still around when trying to do a CANCEL AXIS/ALL. It needs an explicit CANCEL VAR/all to be able to remove the axes from the canceled dataset.
> ferret
        NOAA/PMEL TMAP
        FERRET v6.85
        Linux 2.6.18-348.3.1.el5PAE 32-bit - 04/10/13
        18-Apr-13 15:02

yes? use coads_climatology
yes? let/d=1 temp = (airt+sst)/2
yes? load/l=3 temp

yes? cancel dat 1

yes? can axis/all
 *** NOTE: Not deleted: COADSX
 *** NOTE: Axis is in use by grid GSQ1
 *** NOTE: Not deleted: COADSY
 *** NOTE: Axis is in use by grid GSQ1
 *** NOTE: Not deleted: TIME
 *** NOTE: Axis is in use by grid GSQ1

yes? show var
 Created by DEFINE VARIABLE:

yes? can var/all
yes? can axis/all
yes? sh axis/all
 name       axis              # pts   start                end
 PSXT      LONGITUDE          160mr   130.5E               70.5W
...
 ABSTRACT                 9999999 r   1                    9999999
 EZ                         20480 r   1                    20480

V5.41 (still available on stout), we can do a "cancel axis all" after the "cancel data 1" with no errors, but a SHOW AXIS still has some of the axis information set.
> ferret_v541
        NOAA/PMEL TMAP
        FERRET v5.41
        Linux 2.4.3-12smp - 08/30/02
        18-Apr-13 14:59

yes? use coads_climatology
yes? let/d=1 temp = (airt+sst)/2
yes? load/l=3 temp
yes? can dat 1

yes? can axis/all

yes? sh axis/all
 name       axis              # pts   start                end
 PSXT      LONGITUDE          160 r   130.5E               70.5W
...
ABSTRACT                 9999999 r   1                     9999999
 EZ                         20480 r   1                    20480
 %%        LONGITUDE          180mr   21E                  19E(379)
 %%        LATITUDE            90 r   89S                  89N
 %%        TIME                12mr   16-JAN      06:00    16-DEC      01:20

yes? show var
 Created by DEFINE VARIABLE:
yes?
yes? can var/all
yes? can axis/all
yes? sh axis/all
 name       axis              # pts   start                end
 PSXT      LONGITUDE          160 r   130.5E               70.5W
...
 ABSTRACT                 9999999 r   1                     9999999
 EZ                         20480 r

Migrated-From: http://dunkel.pmel.noaa.gov/trac/ferret/ticket/2063

Modified by ansley.b.manke on 18 Apr 2013 22:15 UTC

Comment by ansley.b.manke on 10 Jul 2013 23:36 UTC
This appears to be fixed with the checkins for ticket 2046, the DEFINE DATA ensemble command. That work did include work on CANCEL DATA; if member datasets are canceled then the ensemble is canceled, and so forth.

Modified by ansley.b.manke on 10 Jul 2013 23:36 UTC


[ENHANCEMENT] Rename nparameters to type_parameter_length

I think nparameters is fine, I was trying to avoid the naming convention n* but it follows the naming convention of ndims so I think it is ok and the alternatives are very long and a bit confusing.


--feature-powerset crashes cargo-hack and my terminal

I'd like to adopt cargo-hack for testing a feature matrix of the libp2p facade crate, see https://github.com/libp2p/rust-libp2p.
Unfortunately, I can't seem to get it to work. I've tried to following locally:
cargo hack --package libp2p check --feature-powerset

This hangs for a few seconds and then crashes my terminal (Alacritty). The libp2p crate does have a few features (~40). Is that a problem?
If I resort to --each-feature, it starts compiling the crate. However, I'd like to use --at-least-one-of and that seems to be only compatible with --feature-powerset.
Let me know how I can provide more debug information or whether you can reproduce that locally.

Thanks for the report. The problem is OOM due to too many feature combinations.
In your case, this can be worked around by --depth option with a value smaller than 8 or 9.
Btw, with --depth 8, there are 51738694 feature combinations.
I think this is a reasonable limitation since it is not realistic to actually check such a large number of combinations (although should report error instead of crashing), but improving the implementation of the powerset calculation may reduce memory usage.

Ah I see :sweat_smile:
I guess what I want is --depth 1 but being able to use --at-least-one-of!


Added import hint "singleconvcol" for QuickHull convex decomposition when importing 3d scenes

This fixes #29971
Adds "singleconvcol" and "singleconvcolonly" to the options of import hints, making the importer use the QuickHull convex decomposition method instead of VHACD. I'm not sure if it's the best terminology, but I think it's nice to have the option at least.

Some feedback from discussion on IRC:
14:45 <@Akien> reduz: OK with #40622 ?
14:47 <reduz> Akien: I don't think so, quickhull is not a great idea because if the geometry is complex, it will create a very complex convex object that will be unusable in practice
14:47 <reduz> Akien: would be interesting to know the use case behind such a feature also, likely better to open a proposal instead
14:49 <reduz> Akien: to be honest, we should  at some point discuss with blender how to better expose physics stuff via the Blender API and via extensions to gltf or escn so we stop using the current hackish way
14:50 <reduz> Akien: also, it is possible to make an import script that does this, so there is not much of a need to add this as a feature, given how rare the use case probably is
14:50 <@Akien> reduz: well check the referenced issue
14:50 <@Akien> The problem is that for simple geometries, quick hull works well and the other option we use now is bogus
14:51 <@Akien> So adding this import mode is just a workaround, the better fix would be to fix VHACD I guess
14:51 <reduz> Akien: i have seen this, but you can already export simple meshes with hints afaik
14:52 <PouleyKetchoupp> reduz: Akien: Just want to mention the QuickHull thing is the first PR by this contributor, so it would be nice to give them advice or direction if it´s not the right fix :)
14:54 <reduz> PouleyKetchoupp: I don't think its a problem of advise as much as something that would require discussing what the best solution would actually be
14:57 <reduz> PouleyKetchoupp: the PR itself is ok, the problem is whether this is wanted or not, it seems very corner caseish to me and the problem in the original issue has better solutions (so its most likely a documentation problem?), on the other hand the PR is small and does not hurt much, the problem is that users may misunderstand it if we add it and create unusable geometry for physics easily
14:58 <reduz> we had this functionality before and it was literally unusable because users would throw complex geometry to quickhull, which would result in extremely complex convex objects that killed performance

So in short, the PR's implementation is fine, but it's not decided yet whether it's a good thing to re-expose a QuickHull import option as it can be misused for complex meshes and lead to issues.

Understandable! I learnt a lot from this so I'm happy anyway!

It's unfortunate that this doesn't seem to be merged. For my level the standard decomposition creates a huge amount of unnecessary collision shape chunks where meshes are already convex (and not concave-convex) by design which leads to problems with my character controller (many small gaps between these highpoly collision chunks).
As this suffix is optional and does not replace the default, misuse (it shouldn't even be called that) is left to the intention.
And as the nature of open source is, some intentions of fixing other libraries or just creating a whole new export option should generally be considered to be in the 'maybe' or 'later' realm, so fixes that don't break functionality but add more freedom (and even allow fixing certain misbehaviour), like this one, should be given more consideration than dismissal.
Please rethink merging this.
This:

vs this:

Since we now expose this in the Mesh menu at the top of the 3D viewport, we should also provide this as an import hint. Otherwise, the exposed feature set is inconsistent depending on your workflow.

For anyone in need of this feature, as it's not merged for now, here's a workaround (creates single convex static collision children);
This script has to be assigned in the import dialog as 'Custom Script':
tool
extends EditorScenePostImport

func post_import(scene):
	iterate(scene)
	return scene

func iterate(node: Node):
	if node != null:
		if "-singleconvcol" in node.name and node is MeshInstance:
			node.name = node.name.replace("-singleconvcol", "") #optional
			node.create_convex_collision()
		for child in node.get_children():
			iterate(child)


Repo install instruction

This PR adds a new instruction called repoInstall. It behaves similar to repoConfig in that it only executes during the lockfile generation phase, but beside the condition the output is the same as install.
This allows for the installation of dependencies that are required to add a repository (e.g. software-properties-common), or packages that themselves install a repository (e.g. epel-release).
It also sets DEBIAN_FRONTEND=noninteractive by default for Ubuntu and Debian images to prevent package installations from asking for input.

Closed in favour of https://github.com/elastic/gradle-plugins/pull/28


feat: Add 'raw_token' reader option

To give access to the low level token stream
Ion:

Add 'raw_tokens' option to makeTextReader(), allowing the raw text parser
to be used as a lexer
Export the raw parser's get_ion_type() function to map the raw token type
to the corrent Ion type.
IonParserTextRaw:
Fix bug: get_ion_type incorrectly returned "STRING" for single quoted symbols
For raw mode, add "_read_token" as a variant of "_read_value" which returns
individual tokens.
Add accessors for start/end.
Add locator() and error_line() functions which wraps the corresponding
IonSpan implementations.  Leverage them in error reporting.
Rationalize start/end handling -- start always points to the first
character of the raw token, end always points one character after the
last character of the raw token.
IonSpan:
fix spelling of CARRIAGE_RETURN
add locator() function to map a position in the span to line/column information
add error_line() which returns the line containing a given token followed
by a line which points to the token
IonText:
Add is_single_operator_char() function so that e.g., "a::-1" tokenizes
as 'a', '::', '-1' instead of 'a', '::-', '1'
IonTextReader:
add raw_parser() accessor
tests/unit:
add tokenizer test
tests/unit/iontests:
Replace backslashes in paths with forward slashes so these tests work on Windows
Add a way to run only specific files for debugging
Since makeSpan creates a binary span if you give it a Buffer, convert text
files to strings so makeSpan will create a text span.
tests/unit/tokens:
new unit test for raw mode token handling

merged manually


fix resource already exists for shadow apis

Signed-off-by: Di Xu<EMAIL_ADDRESS>
What type of PR is this?
kind/bug
What this PR does / why we need it:
Failed to create a shadow resource with dryRun when real resource existed in the apiserver with the same name.
Which issue(s) this PR fixes:

Fixes #315
Special notes for your reviewer:

Codecov Report

Merging #316 (5b276a9) into main (e00bc04) will increase coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #316      +/-   ##
==========================================
+ Coverage   11.09%   11.11%   +0.01%
==========================================
  Files          47       47
  Lines        4976     4986      +10
==========================================
+ Hits          552      554       +2
- Misses       4386     4394       +8
  Partials       38       38

Impacted Files
Coverage Δ

...et/clusternet/pkg/registry/shadow/template/rest.go
2.70% <0.00%> (-0.05%)
:arrow_down:

...clusternet/pkg/registry/shadow/template/trimmer.go
64.70% <0.00%> (+2.20%)
:arrow_up:

Continue to review full report at Codecov.

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


Issue #2927 Fix user avatars

The getNonAPIResourceWithAuth Function is questionable.
But the User-Portrait is called like: http://localhost:8080/Plone/portal_memberdata/portraits/jm without '/++api++'.
Maybe the getAPIResourceWithAuth Function could be updated to conditionally not use the '/++api++' part.
Or an new Api resource for User-Portraits is needed.

@jmevissen thanks for opening the PR! From my point of view, if the portrait URL is not accessible using the ++api++ traversal, it should be, and it's a backend issue. In short, all resources used by the frontend should be accessible using the api traversal, and if they do not, they should be fixed.
@davisagli @jensens any idea why it might not be traversable using the api traversal?

@sneridagh thanks for your comment.
The User Portraits are called like: http://localhost:8080/Plone/portal_memberdata/portraits/jm
Where "portal_memberdata" is a MemberDataTool and "portraits" is a BTreeFolder2 that is an attribute of the MemberDataTool.
The ++api++ traversal does not work in this case and throws a KeyError quite early,
when trying to call http://localhost:8080/Plone/++api++/portal_memberdata/portraits
(https://github.com/plone/plone.rest/blob/master/src/plone/rest/traverse.py#L25)

@sneridagh
I think that options maybe:

enable the ++api++ traversal to access attributes of Tools
add a new rest-api service for user portraits

@davisagli @jensens what do you think about this one?
My take is that we should make the ++api++ traversal work, but I don't like the idea to expose things inside a tool through that...

@sneridagh I need to think about it some more. I agree in principle that user portraits should be made available by the API. But on the backend member info is looked up via the pluggable auth service so we can't assume it is always stored in portal_memberdata; for example with membrane and Dexterity-based members it could come from a NamedBlobImage field on a content item.

The whole portrait handling in the backend is a PITA and I would like to see plone.api and plone.restapi to abstract this away from the existing tools and to not depend Volto to traverse to portal_memberdata behavior at all .
I.e. if it comes to pas.plugins.ldap there is still an ugly patch active https://github.com/collective/pas.plugins.ldap/blob/master/src/pas/plugins/ldap/monkey.py to get portraits into our old (IIRC GRUF based, or even earlier CMF based) approach. This never got lifted up to PAS. And IIRC other plugins do the same (so you can not use them together).
That said, I have no good idea why traversal fails.

This gets superseded by: https://github.com/plone/volto/pull/3487


ProviderFactory: provide a way to get Gradle properties by a prefix

Expected Behavior
ProviderFactory provides a way to get all Gradle properties by prefix (has a method Provider<Map<String, String>> gradlePropertiesPrefixedBy(String variableNamePrefix);).
Current Behavior
There's no way to get a map of Gradle properties with ProviderFactory, neither a full one, nor filtered by the prefix.
Context

At the moment ProviderFactory provides no way to get a list of Gradle properties by a prefix, while there are systemPropertiesPrefixedBy and environmentVariablesPrefixedBy.
Because of that ProviderFactory can't really be considered as an alternative to using project.properties when adopting configuration cache.
I.e. with Project I can do something like project.properties.filterKeys { it.startsWith(prefix) }, but I can't do the same with ProviderFactory.

Thank you for your contribution @monosoul!
#21877 was merged
This will be in Gradle 8.0


addons: Update from Prometheus v1.8.2 to v2.0.0

Migration Guide: https://prometheus.io/docs/prometheus/2.0/migration/
Use the promtool (its in the tarball) to convert v1 rules to v2

Hm, alertmanager doesn't trigger as it did in Prometheus v1.8.2. Prometheus seems to have some difficulty reaching alertmanager.
level=error ts=2017-11-08T10:08:52.910573492Z caller=notifier.go:444 component=notifier alertmanager=http://alertmanager:9093/api/v1/alerts count=1 msg="Error sending alert" err="context deadline exceeded"

kubectl exec -it prometheus-7444d9dccc-rgm2d -n monitoring /bin/sh
wget alertmanager:9093
Connecting to alertmanager:9093 (<IP_ADDRESS>:9093)
wget: can't connect to remote host (<IP_ADDRESS>): Connection timed out

<IP_ADDRESS> is the correct service IP so this isn't a DNS issue. Nothing interesting in alertmanager logs, unclear if it receives the request.

This turned out to be completely a problem with my alertmanager manifests. I haven't yet added alertmanager to Typhoon's addons so its not really a concern here.
Prometheus 2.0 is looking grand!

Rebased and merged as f570af9418522dc5ce7db28762ff69257af0e493.

@dghubble is alertmanager still missing? Noticing PrometheusErrorSendingAlerts and the pods trying to query alertmanager

Its not "missing" per-se. If you deploy it on your own, it'll be used. If not it won't.
I personally do deploy alertmanager and recommend it. But haven't opted to maintain the manifests here.


Prompt Compression

Summary
Will be nice to see some prompt compression to reduce the amount of tokens used looking better performance.
Motivation
Cost saving and looking for better performance as far the compression keeps the essentials with loosing information or context.
Technical Design
More here: https://llmlingua.com/llmlingua2.html
So I can see this as a configurable feature that compress the user prompt.
Alternatives to Consider
Additional context
an example:
User
can you create a scratch and win game in a webpage?

After compression
create scratch win game webpage?

We're doing compression in the MonologueAgent already. This is mostly up to each Agent to implement.
I think we'll want to put a "best practices" doc together for Agent design at some point--will keep this one in mind!

@rbren I saw summarization Agent, wich is kind of different, maybe I am wrong, Happy to help to build that!

Similar idea, but I suppose a bit different 😄
Would love to see a new EfficientAgent that minimizes tokens!


How I made my calculator app using function components

In coding my calculator app using react, I followed the processes outlined below.

I first set up a create component using these commands; npx create-react-app my-app, cd math-magic and npm start, with linters set along side.
I created a component directory where I also created a calcultor.js file as a parent component.
I created 2 children components(Button and Input) where I made the UX design for the app, I then imported them to the calculator.js file that I finally used in the app.js.
I downloaded 2 js(calculate and operate) files that I used to implement my logic. I started with the useState which I imported, to initiate the state of the app.
I created an onClickhandler which I called in the buttons to implement the functions.

Thank you
Sent from Yahoo Mail on Android
On Thu, Feb 23, 2023 at 4:29 PM, Daniel @.***> wrote:
Hi @Divinity-dev ,
Your project is complete! There is nothing else to say other than... it's time to merge it
Congratulations! 🎉
Highlights

Nicely Done ✔

Optional suggestions
Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers and Happy coding! 👏 👏 👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me @danielufeli in your question so I can receive the notification.
_As described in the Code reviews limits policy you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form._ate_comments_counter`
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>


conda-forge package

Hello, I've added antropy to conda-forge; please let me know if you'd like to be added as a co-maintainer for the respective feedstock. It could also make sense to amend the installation instructions, WDYT?

Hi @hoechenberger,
Yay, thank you! And thanks for everything that you're doing for the Python open-science community :) Please do add me as a co-maintainer.
Agree for the installation instructions, so conda install antropy should work?

Hey @raphaelvallat,
thanks!! 😍 I can only say the same about your efforts 😄
I've added you as a feedstock maintainer.
The recommended way to install the package from conda-forge is documented here:
https://github.com/conda-forge/antropy-feedstock#installing-antropy

Installing antropy from the conda-forge channel can be achieved by adding conda-forge to your channels with:
conda config --add channels conda-forge
conda config --set channel_priority strict

Once the conda-forge channel has been enabled, antropy can be installed with conda:
conda install antropy

or with mamba:
mamba install antropy

Thanks — added in https://github.com/raphaelvallat/antropy/pull/19


[G-drive] A folder is duplicated

Problem
Not sure yet what's going on. I've noticed the following over the past few days:
A folder ("2021") duplicated on G-Drive. There's no sign of conflict on both DB and GD world so far.

I saw several folders under "Periodic" were duplicated, mostly older than 2011 where less number of files are, so I went ahead visually checked which folder looks more legit. Took probably 15 mins for total to delete the dups.
Related or not, on mac1 2021 folder, which is the folder in question in this ticket, keeps disappearing locally. Dropbox selective sync shows it there but G-Drive selective sync doesn't list 2020.
When it once happened, I manually copied it from another host. Then disappeared again (here now).


Upgrade to Python 3.10 o3 3.11

We would like to upgrade our code base to Python 3.10 or 3.11.
Some packages that we would like to use, like PyRit, require Python >= 3.10.
Promptflow still recommends using 3.9.
Describe the solution you'd like
Promptflow should fully support Python 3.10 or higher

Hi @FriederikeBrezing , we publicly support python 3.10 and 3.11. Check here for current supported versions:

Hi, I had seen that but it also says that python 3.9 is recommended.
What is the reason for that? Will it affect the performance if other versions of python are used?

This is a comment written before we supported Python 3.9. I will delete it to avoid confusion.

The Docker file in the dev container is still based on the python:3.9 image. Can it be upgraded as well?


Implement byte unit for api_request_size config

Codecov Report

Merging #193 into master will decrease coverage by 0.07%.
The diff coverage is 88.46%.

@@             Coverage Diff              @@
##             master     #193      +/-   ##
============================================
- Coverage     72.74%   72.66%   -0.08%
- Complexity      992     1003      +11
============================================
  Files           111      113       +2
  Lines          3856     3878      +22
  Branches        358      359       +1
============================================
+ Hits           2805     2818      +13
- Misses          889      898       +9
  Partials        162      162

Impacted Files
Coverage Δ
Complexity Δ

...a/co/elastic/apm/report/ReporterConfiguration.java
100% <100%> (ø)
12 <1> (ø)
:arrow_down:

...pm/configuration/converter/ByteValueConverter.java
75% <75%> (ø)
3 <3> (?)

...elastic/apm/configuration/converter/ByteValue.java
88.88% <88.88%> (ø)
9 <9> (?)

.../co/elastic/apm/report/ApmServerHealthChecker.java
63.63% <0%> (-18.19%)
3% <0%> (ø)

...stic/apm/report/IntakeV1ReportingEventHandler.java
75.55% <0%> (-2.23%)
12% <0%> (-1%)

...in/java/co/elastic/apm/report/ReporterFactory.java
64.7% <0%> (-1.97%)
6% <0%> (ø)

Continue to review full report at Codecov.

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


Fetch all

This should resolve issue #38. I can make any changes if you see any potential issues.

Reminder to self: add test cases

I'm a little concerned with accidentally making too many requests. So far, octokat makes only 1 request for each call and calling something like octo.users.fetchAll() will kill the rate limit. Thoughts?

Might also be worth increasing the default per_page parameter to the maximum allowed (100, I think) when using fetchAll.

+1 @philschatz any chance of merging & releasing this simple but useful change?

+1 As a GHE user this would be great since the usual rate limits don't apply.

Ah, I didn't know about the lack of limits for GHE!
This PR seems to assume a Promise implementation exists so I am reluctant to merge it in.
But I did start refactoring the code a bit to make it easier to enable/disable features (like camelCase in the JSON and injecting hypermedia) and adding/removing verbs (like fetchAll) in #81

Though it also assumes a Promise implementation, you could consider using my promise-recurse module, e.g.:
function fetchAll (startPromise) {
  return promiseRecurse(startPromise, function (result) {
    return result.nextPage ? result.nextPage() : null
  }).then(function (results) {
    return _.flatten(results)
  })
}

Using that with Octokat here: https://github.com/adjohnson916/github-mirror


JSpecify: changes for using addMetadata and dropMetadata for jdk21 using MethodHandle

tests for Jdk21 were not able to run since cloneMetadata was removed so these changes incorporate using addMetadata and dropMetadata methods of Jdk21 using MethodHandle.

No, if tests are failing then you can just leave the changes.
On Nov 30, 2023 at 20:32:49, Abhijit Kulkarni @.***>
wrote:

@.**** commented on this pull request.
In nullaway/src/main/java/com/uber/nullaway/NullAway.java
https://github.com/uber/NullAway/pull/869#discussion_r1411589287:

@@ -476,7 +476,7 @@ public Description matchAssignment(AssignmentTree tree, VisitorState state) {
doUnboxingCheck(state, tree.getExpression());
}
// generics check

if (lhsType != null && lhsType.getTypeArguments().length() > 0) {

if (lhsType != null && lhsType.getTypeArguments().length() > 0 && config.isJSpecifyMode()) {

I tried removing the changes and saw some tests failing, I haven't looked
into it more deeply yet, Should I do that?
—
Reply to this email directly, view it on GitHub
https://github.com/uber/NullAway/pull/869#discussion_r1411589287, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AABPEUOULW66HFFOKE2R5U3YHFMXDAVCNFSM6AAAAAA72L4ZR2VHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTONJYHE4DKMJYGQ
.
You are receiving this because you commented.Message ID:
@.***>


Assign hardware port to docker-compose file

@ipa-nhg
To connect hardware with docker container, we also need to assign hardware port to docker container, e.g.
docker run --rm -it --device=/dev/input/js1 --device=/dev/jackal  rachelwu/agriculture_demo_sprint2:agriculture_demo_sprint2
Some of ports are also assigned in launch file, e.g.
 <arg name="joy_dev" default="$(optenv JACKAL_JOY_DEVICE /dev/input/ps4)" />
I propose to generate a docker-compose file also for rossystem(without ComponentStack). So we can assign a port like:
devices:
  - "/dev/input/js1:/dev/input/js1"

But there is an issue: how can I get this information from model level?

From my point of view, this information should be part of a new "deployment" model but from of the "rossystem" model. The Rossystem model is independent of the machine where the software will be deployed.
This new deployment model should also contain information about the machine, the OS, the distro... @ipa-rwu what do you think?

Yes, I like this idea.


Website: aliexpress.com

Website you would like added. Include multiple URLs if needed

https://aliexpress.com/item/1005003170671440.html?blablablal
https://aliexpress.com/item/1005003170671440.html

Bad parameters. Leave blank if you don't know.
Additional context
Add any other context or screenshots about the feature request here.
It also have regional versions like aliexpress.*/

Both the links you provided seem to be fine. What parameters are you having trouble with? There's roughly 20 rules for aliexpress already

@DrKain https://aliexpress.ru/item/1005003170671440.html?gatewayAdapt=glo2rus&spm=a2g2w.home.15002.76.5a50501dwThjGw&_evo_buckets=165609,165598,188873,224373,176818,194275&sku_id=12000024476377327&gps-id=pcJustForYou&scm=1007.13562.265877.0&scm_id=1007.13562.265877.0&scm-url=1007.13562.265877.0&pvid=ed2b99b7-359e-4c84-8a13-761e16c1d360&_t=gps-id:pcJustForYou,scm-url:1007.13562.265877.0,pvid:ed2b99b7-359e-4c84-8a13-761e16c1d360,tpp_buckets:21387%230%23250274%230#/

Ah, I was not aware aliexpress used anything other than .com, Thank you for reporting. I'll update in a moment

@DrKain Thank you! Great module


Model Zoo ".caffemodel" use with digits

I download the "train_val.prototxt" and the "*.caffemodel" file from the Model Zoo site.
I copy and paste the "train_val.prototxt" content to the "custom Network" area adding a "#" to the "source" and "backend" lines.
I specify the path to the folder were I have the "*.caffemodel" file but I obtain the error:
"Pretrained model
File does not exist"
What I am doing wrong? I didn't found proper support for this use, can you help me?
Thanks in advance.

I specify the path to the folder were I have the "*.caffemodel" file

I assume you mean you specified the path to the file, and not just the file's parent directory?

Yes, I specify the path to the file and I'm not understanding what I am doing wrong.
Can you confirm me that I only need to add "#" to the "source" and "backend" lines of  "train_val.prototxt" and copy it to the Custom Network field and them specify the path to the caffemodel file?

That path looks suspicious to me - you're sure you didn't mean to include your username?
/home/<username>/squeezenet_v1.0.caffemodel

I solve the problem downloading again the file from the caffe model zoo  and choosing another directory. I am not sure of what was the solution, but now it works.
Now I have the "ERROR: must specify a loss layer" with a network trained for imagenet.  Do you now why?

Click on the little blue question mark next to "Custom Network". That should tell you everything you need to know - let me know if it doesn't and we'll update the documentation!

I'm trying to use this pre-trained model  with the ".caffemodel" and "deploy.prototxt" file available to initialize my network weights to be able to find a better classifier for my dataset.
I didn't understand how I add the "SoftmaxWithLoss" layer for this example. Can you help me?

The model description claims that it's an object detection network, but it only has a 1x100 shape output? Not sure what's going on there. Are you sure you want to repurpose this network for image classification?
If so, you probably want to remove the Sigmoid layer and replace it with 3 layers that activate on different phases, as with all the standard networks:
https://github.com/NVIDIA/DIGITS/blob/v3.3.0/digits/standard-networks/caffe/lenet.prototxt#L156-L178
https://github.com/NVIDIA/DIGITS/blob/v3.3.0/digits/standard-networks/caffe/alexnet.prototxt#L365-L387
https://github.com/NVIDIA/DIGITS/blob/v3.3.0/digits/standard-networks/caffe/googlenet.prototxt#L2474-L2508

"Are you sure you want to repurpose this network for image classification?"
I just want to learn how to use another previously trained networks (pre-trained weights) to use them with my dataset and increase the quality of my classifier. That was the reason I went to the caffe Model Zoo website because I think that the application was more direct in Digits. But I verified that most of them didn't work directly without some previous adaptations.
I didn't test your recommendation from your last answer yet ("3 layers ") but I will make it as soon as I can.

For example with the SqueezeNet it works in a simple way to, just adding it to the DIGITS.

The main difference here is the three layers described in the Figure that are not present in some of the models available in the model Zoo website.

Those layers are typical for classification networks. For solving other problems, you'll probably want to use the "Generic"/"Other" path - that will work for networks which don't have a Softmax layer output.


Change case object to case class to make avro4s derivation work

Hot fix for  a bug  in avro4s derivation of Feedback.
The reason for it is that avro4s seems incapable of deriving for sealed traits whose subclasses are both case classes and case objects, and that's the case for EmailMetadata.
I've changed Opened to a case class as a workaround, however @filosganga suggests to have another look at the schema for Feedback before completing this, and possibly use manual instances to make the schema look the way we want it, since this topic will be for public consumption.

Ready for merge I think. Will it be published on merge as well?

Nope it does not unfortunately

right, could you release then when you have time please?


Publish driver status even if robot is locked

Background

Publish driver status even if robot is locked

What's new

✅ [New feature description. Only a single new major feature is allowed per PR.]
✅❌ New feature is accompanied by new test: [name and description of new test].

Related work

https://github.com/DexaiRobotics/dig/pull/1875 needs this PR

TODOs / Nice-To-Haves

❌ [Add new unit test for new feature.]
❌ [Add new system test for new feature.]

Tests

❌ Tested on simulated robot: [Describe the tests/commands used.]
✅❌ Tested on physical robot: [Describe the tests/commands used.]

Quality

✅ New code is written in pure C++17 to the best of my knowledge.
✅ New code follows established C++17 best practices (C++ Core Guidelines).
✅❌ New code passed clang-format and cpplint at least, if not all of our static code analysers.
✅ I have included Doxygen-style documentation in the new C++ code.

Note to reviewers
Please verify that all sections are accurately filled out and that all 6 numbered entries above are present and ticked/checked off, with their requirements met, if applicable.

/test pr

/hold

/hold cancel


Scroll down effect for core team page

Description
Core team page is too simple want to add scroll down effect, assign me this issue under gssoc'24
Screenshots
core team page:
https://github.com/SauravMukherjee44/CodeIN-Community-Website/assets/169196836/c0b99e72-2ec7-4784-8493-3499c6413009
scroll down effect I want to add:
https://github.com/SauravMukherjee44/CodeIN-Community-Website/assets/169196836/783eaadd-a333-4c2b-a6a2-def044d056e6

Hello @paulkrishanu4, I'd like to take ownership of this particular issue. Could you please assign it to me? Thank you!

@paulkrishanu4  You can work on this issue!
Thanks for choosing our community project. 🙌🏻
✅ For any queries, Please reach out to me here: https://www.linkedin.com/in/sauravmukherjee44/


Feature: add action to open quickfixlist

Add action which also opens the quickfix list after sending items to it.

The proposed addition doesn't scale well. There are already 4 different actions for behavior that could be parameterized (send_to_qflist, add_to_qflist, send_to_loclist, add_to_loclist). If we add another boolean open then logically we would need to double this number to add a _and_open variant to each. Arguably I shouldn't have even split it as much as it currently is, but I definitely don't want to split it out further. The preferred way of doing this would be to define your own keymap that leverages the built-in action. Something like:
require("oil").setup({
  keymaps = {
    ["<C-q>"] = {
      callback = function()
        require("oil.actions").send_to_qflist.callback()
        vim.cmd.copen()
      end,
      desc = "Send items to quickfix and open the quickfix"
    }
  }
})

That makes sense. Thank you for the explanation and keymap example!


Adds recommendation to use IonElement

Issue #, if available:
None
Description of changes:
Recommends using IonElement instead of IonValue.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Should we also put this note in the JavaDoc overview for IonValue?

Yes, i can do that.


$id is not defined

Just started using this plugin, and it seems to be working really well. Was just wondering about this console error I saw, any idea why that's showing up?

I think this might have had something to do with not having any content yet. I can't reproduce it now that there are entries in the Craft installation.

Glad you got it working!


'Special circumstances' should be checkboxes, not a dropdown

Patients can have more than one 'Special Circumstance' at a time. For MVP, you could eliminate the 'Other' plus text field for now and add it later.

I'll revert this to a plain text field for the time being.
In the long term, this probably involves setting up an associated special circumstance model with the options, and then doing a has_many :through. Or at least that's how you do it in postgres, might be a little different in Mongo.
Backlogging for the time being.

The UX team would like to know if we can prioritize this so that the data is clean.  @mebates would especially like this!

Acknowledged; I'll see if I can spin something up tomorrow.

We should have this in place by showtime, marking deploy.

Taking a stab at this.

Fixed by @ashlynnpai in #591


installed but not working..

When I go on chrome://inspect
in node I see this error:
ERR! devtools-backend:Proxy Internal request failed http://<IP_ADDRESS>:9222/json/version Error: connect EMFILE <IP_ADDRESS>:9222 - Local (undefined:undefined)
(same for localhost)
and if a manually go to http://<IP_ADDRESS>:9222/json/version
I see:
EMFILE <IP_ADDRESS>:9222 - Local (undefined:undefined)

and also
ERR! devtools-backend:Proxy Internal request failed http://<IP_ADDRESS>:9222/json/version Error: socket hang up
ERR! devtools-backend:Proxy Internal request failed http://<IP_ADDRESS>:9222/json/version Error: connect EMFILE <IP_ADDRESS>:9222 - Local (undefined:undefined)

Can you describe more what your setup is?

Ubuntu 17.10 I just did what's written in the readme... but it does not work and I don't know why.
I badly need this to remote debug a smart tv application..

$ node -v
v8.11.3
$ uname -a
Linux zibrixnb 4.13.0-45-generic #50-Ubuntu SMP Wed May 30 08:23:18 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux
$ cat /etc/lsb-release
DISTRIB_ID=Ubuntu
DISTRIB_RELEASE=17.10
DISTRIB_CODENAME=artful
DISTRIB_DESCRIPTION="Ubuntu 17.10"

if you need anything else to debug.. just ask

Can you provide any logs from the proxy? Also check out my talk on this project.

I explain... I wish to do what I usually do with "weinre" but using chrome devtools.
With weinre, I just add a

When I go on chrome://inspect
in node I see this error:
ERR! devtools-backend:Proxy Internal request failed http://<IP_ADDRESS>:9222/json/version Error: connect EMFILE <IP_ADDRESS>:9222 - Local (undefined:undefined)
(same for localhost)
and if a manually go to http://<IP_ADDRESS>:9222/json/version
I see:
EMFILE <IP_ADDRESS>:9222 - Local (undefined:undefined)

Have you solved the problem? Can you share the solution?

For future users (including myself): The issue is that the server leaks sockets, EMFILE is caused by too many files being opened. The solution is to add a route that just does req.status(404).end() after registering the other paths in lib/index.js. This resolves the issue and makes the server work.


Put angular-ui-notification on CDN

Is there any particular reason why this library isn't on a CDN?
I've never done this but I am willing to take care of it if you would let me. If so, any help would be apprecited.

I haven't shared any of the libraries on CDNs
If you want you may.


Private channel site management is scoped to PowerShell

Under the section Private channel SharePoint sites
https://docs.microsoft.com/en-us/MicrosoftTeams/private-channels#private-channel-sharepoint-sites
We should add a note stating - that, currently Private channel site management is scoped to PowerShell

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

ID: ce34b12b-aa50-608b-bbfa-a1fb4bb79f8d
Version Independent ID: 9ddeb6f4-f30a-1e72-fc94-ae66def91f40
Content: Private channels in Microsoft Teams - Microsoft Teams
Content Source: Teams/private-channels.md
Service: msteams
GitHub Login: @MikePlumleyMSFT
Microsoft Alias: mikeplum

Thank you for submitting feedback and contributing to the docs. We are currently investigating this.

Thanks for your feedback @JorgeALeitao


strange error (error 500) when inserting pid with a leading or trailing space

This error is reported on https://wiki.eduuni.fi/pages/viewpage.action?pageId=450081765 by Serafeim Chatzopoulos (OpenAIRE),
e.g., https://api.pidmr.devel.argo.grnet.gr/v1/metaresolvers/resolve?pid=10.1038/scientificamerican0501-34 &pidMode=metadata&redirect=true

The solution would be to trim the pid before sending the request to pidmr backend as currently we have no means of triming in the backend.

@asajedi We fixed this issue and deployed it on https://api.pidmr.argo.grnet.gr.


Fixed 'or' error for BitSets of longer length

Code to reproduce below. To correct I think you could either make a temp copy of the shorter and increase it's length (slow), permanently increase the length of the shorter (little faster), request the # of flags length in the constructor (faster, but prone to user error & changes in requirements), or perform a length check & flip (as shown in the PR)
a = new BitSet();
b = new BitSet();
a.set(1);
b.set(50);
a.or(b).toString(2)
"10"
b.or(a).toString(2)
"100000000000000000000000000000000000000000000000010"

Thanks for that!

I changed your pull request and fixed it for the other methods as well. You made a really good point here. Thanks again for that!
BTW I just upgraded to 3.0.0, which also incorporates a change on the API that every function call returns a new instance to make it an immutable class. This was a flaw in the previous design.
Robert


List item count incorrect

See #456
In my comment in 456 on GitHawk the numbers go 1. 1. On the website they go 1. 2. (As they should)
🤔
? 1. test
2. test
3. test
edit// looks like it's when a list starts without the number being right at the beginning of the line

Bug Report Dump (Auto-generated)

Version 1.11.0 (1942)
Device: iPhone 6s Plus (iOS 11.0)
TestFlight: true

I'm gonna close this b/c tbh that formatting shouldn't work. The 1. is part of the paragraph. I don't think that's MD spec lol.

Cool all good, was just confusing switching between desktop and mobile and seeing different!

Yeah, it's a little weird - if you have a list that starts with 2., markdown converts it to 1. anyway. But as @rnystrom - that's markdown for you. GitHub probably wasn't happy with it and changed that.

test
test