chore(release): v0.5.1

Version bump in package.json for release v0.5.1

:tada: This PR is included in version 0.5.2 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Multi-kernel tests (plus a few others) don't work with driver date 20240819

The Windows E2E PR #689 is blocked because in order to get Windows to work I had to use a particular commit of XRT (required by XRT-MCDM). At that commit, the current linux driver (date 20240819) fails some tests while the older driver (such as the one on the sharkbox phoenix nuc) passes.
Possibly there's some connection to https://github.com/nod-ai/iree-amd-aie/blob/main/runtime/src/iree-amd-aie/driver/xrt/native_executable.cc#L270, which I had to ifndef _WIN32 away on Windows (which @jtuyls added when battling numerics issues) but it's not clear since it's not only the multi-kernel tests that fail.

Could you call out specific tests... They might not all get resolved in one go.

Could you call out specific tests... They might not all get resolved in one go.

  for name in [
      "two_matmul_switching",
      "matmul_f32_8_8_4",
      "matmul_f32_8_4_8",
  ]:
      aie_vs_llvm_cpu(config, test_files_dir / f"{name}.mlir")

  aie_vs_llvm_cpu(
      config,
      test_files_dir / "three_matmuls.mlir",
      function_name="three_$mm$",
  )

  # Test(s) of the form matmul(A,B) where A:MxK, B:KxN
  test_name = output_dir / "test_from_template.mlir"
  template_name = matmul_template_dir / "matmul_MxK_KxN.mlir"
  generate_matmul_test(test_name, template_name, 32, 32, 64, "bf16", "f32")
  aie_vs_llvm_cpu(config, test_name)

  # Test(s) of the form matmul(A,B) + C where A:MxK, B:KxN, C:N
  test_name = output_dir / "test_from_template_bias_N.mlir"
  template_name = matmul_template_dir / "matmul_bias_MxK_KxN_N.mlir"
  generate_matmul_test(
      test_name, template_name, 1024, 1024, 512, "bf16", "f32"
  )
  if config.vitis_dir:
      aie_vs_llvm_cpu(
          config, test_name, tile_pipeline="pack-peel", use_ukernel=True
      )
  aie_vs_llvm_cpu(
      config, test_name, tile_pipeline="pack-peel", use_ukernel=False
  )

Note, I believe I did do the whole stratification thing (some on, some off) and they indeed all do fail independtly of each other. But I'll do that again to be sure.

Btw, locally I have debug turned on for my driver (linux) and I see this for every test:
28320403: PID(68358): Allocated KMQ BO (userptr=0x7f3966c9d000, size=32768, flags=0x20000000, type=1, drm_bo=8)
28354699: PID(68358): Freeing KMQ BO, type=AMDXDNA_BO_SHMEM, drm_bo=8, size=32768
28357514: PID(68358): Unmap BO, aligned 0x7f3966c9d000 parent (nil)
29546943: PID(68358): Freeing KMQ BO, type=AMDXDNA_BO_SHMEM, drm_bo=4, size=8192
29553385: PID(68358): Unmap BO, aligned 0x7f3966cb1000 parent (nil)
29612528: PID(68358): Freeing KMQ BO, type=AMDXDNA_BO_SHMEM, drm_bo=5, size=16384
29617438: PID(68358): Unmap BO, aligned 0x7f3966cad000 parent (nil)
29662174: PID(68358): Freeing KMQ BO, type=AMDXDNA_BO_SHMEM, drm_bo=6, size=32768
29664648: PID(68358): Unmap BO, aligned 0x7f3966ca5000 parent (nil)
29709454: PID(68358): Destroying KMQ pcidev
29712640: PID(68358): Device node fd leaked!! fd=3 <------------------- ???

On the new driver, for two_matmul_switching test:
EXEC @matmul_small
134363871: PID(65459): Allocated KMQ BO (userptr=0x7ff8204c7000, size=128, flags=0x20000000, type=1, drm_bo=8)
134405692: PID(65459): Allocated KMQ BO (userptr=0x7ff8204c6000, size=384, flags=0x20000000, type=1, drm_bo=9)
134479183: PID(65459): Allocated KMQ BO (userptr=0x7ff8204c5000, size=4096, flags=0x80000000, type=4, drm_bo=10)
134490635: PID(65459): Added arg BO 3 to cmd BO 10
134494753: PID(65459): Added arg BO 6 to cmd BO 10
134497568: PID(65459): Added arg BO 7 to cmd BO 10
134499712: PID(65459): Added arg BO 9 to cmd BO 10
134525592: PID(65459): Submitted command (1)
134529940: PID(65459): Waiting for cmd (1)...
134921743: PID(65459): Added arg BO 5 to cmd BO 10
134924488: PID(65459): Added arg BO 9 to cmd BO 10
134926372: PID(65459): Added arg BO 6 to cmd BO 10
134928245: PID(65459): Added arg BO 9 to cmd BO 10
terminate called after throwing an instance of 'xrt_core::system_error'
  what():  DRM_IOCTL_AMDXDNA_EXEC_CMD IOCTL failed (err=114): Operation already in progress

  std::cerr << "before start\n";
  run.start();
  std::cerr << "after start\n";
  run.wait();
  std::cerr << "after wait\n";
  for (xrt::bo& bo : bos) bo.sync(XCL_BO_SYNC_BO_FROM_DEVICE);

prints
139451362: PID(67134): Waiting for cmd (1)...
139820560: PID(67134): Added arg BO 5 to cmd BO 10
139824186: PID(67134): Added arg BO 9 to cmd BO 10
139827483: PID(67134): Added arg BO 6 to cmd BO 10
139830078: PID(67134): Added arg BO 9 to cmd BO 10
before start
after start
after wait
before start
terminate called after throwing an instance of 'xrt_core::system_error'
  what():  DRM_IOCTL_AMDXDNA_EXEC_CMD IOCTL failed (err=114): Operation already in progress

So seems iree_hal_xrt_direct_command_buffer_dispatch is getting called twice when XRT already has both dispatches or commands buffers or whatever and doesn't except it to get called twice? Maybe @nirvedhmeshram has some insight.

static bool runOnce;

static iree_status_t iree_hal_xrt_direct_command_buffer_dispatch(...) {
  ...
  if (runOnce)
    return iree_ok_status();

  run.start();
  run.wait();

gives
EXEC @matmul_small
22151753: PID(69421): Allocated KMQ BO (userptr=0x7fb06cfe8000, size=128, flags=0x20000000, type=1, drm_bo=8)
22190698: PID(69421): Allocated KMQ BO (userptr=0x7fb06cfe7000, size=384, flags=0x20000000, type=1, drm_bo=9)
22263948: PID(69421): Allocated KMQ BO (userptr=0x7fb06cfe6000, size=4096, flags=0x80000000, type=4, drm_bo=10)
22272795: PID(69421): Added arg BO 3 to cmd BO 10
22274789: PID(69421): Added arg BO 6 to cmd BO 10
22275730: PID(69421): Added arg BO 7 to cmd BO 10
22276542: PID(69421): Added arg BO 9 to cmd BO 10
22308413: PID(69421): Submitted command (1)
22312300: PID(69421): Waiting for cmd (1)...
22735479: PID(69421): Added arg BO 5 to cmd BO 10
22736411: PID(69421): Added arg BO 9 to cmd BO 10
22736962: PID(69421): Added arg BO 6 to cmd BO 10
22737594: PID(69421): Added arg BO 9 to cmd BO 10
22738686: PID(69421): Added arg BO 3 to cmd BO 10
22739307: PID(69421): Added arg BO 9 to cmd BO 10
22739868: PID(69421): Added arg BO 7 to cmd BO 10
22740449: PID(69421): Added arg BO 9 to cmd BO 10
22741311: PID(69421): Added arg BO 5 to cmd BO 10
22741872: PID(69421): Added arg BO 9 to cmd BO 10
22742423: PID(69421): Added arg BO 6 to cmd BO 10
22742974: PID(69421): Added arg BO 9 to cmd BO 10
22743966: PID(69421): Added arg BO 3 to cmd BO 10
22744547: PID(69421): Added arg BO 9 to cmd BO 10
22745088: PID(69421): Added arg BO 7 to cmd BO 10
22745629: PID(69421): Added arg BO 9 to cmd BO 10
22746450: PID(69421): Added arg BO 5 to cmd BO 10
22746961: PID(69421): Added arg BO 9 to cmd BO 10
22747502: PID(69421): Added arg BO 6 to cmd BO 10
22748074: PID(69421): Added arg BO 9 to cmd BO 10
22748885: PID(69421): Added arg BO 3 to cmd BO 10
22749396: PID(69421): Added arg BO 9 to cmd BO 10
22749917: PID(69421): Added arg BO 7 to cmd BO 10
22750448: PID(69421): Added arg BO 9 to cmd BO 10
22751280: PID(69421): Added arg BO 5 to cmd BO 10
22751811: PID(69421): Added arg BO 9 to cmd BO 10
22752332: PID(69421): Added arg BO 6 to cmd BO 10
22752883: PID(69421): Added arg BO 8 to cmd BO 10
22759014: PID(69421): Freeing KMQ BO, type=AMDXDNA_BO_SHMEM, drm_bo=9, size=384
22759606: PID(69421): Unmap BO, aligned 0x7fb06cfe7000 parent (nil)
22794923: PID(69421): Freeing KMQ BO, type=AMDXDNA_BO_SHMEM, drm_bo=6, size=128
22795354: PID(69421): Unmap BO, aligned 0x7fb06d023000 parent (nil)
22815743: PID(69421): Freeing KMQ BO, type=AMDXDNA_BO_SHMEM, drm_bo=7, size=128
22816094: PID(69421): Unmap BO, aligned 0x7fb06cfe9000 parent (nil)
22875086: PID(69421): Allocated KMQ BO (userptr=0x7fb06d023000, size=128, flags=0x20000000, type=1, drm_bo=6)
22882891: PID(69421): Freeing KMQ BO, type=AMDXDNA_BO_SHMEM, drm_bo=8, size=128
22883292: PID(69421): Unmap BO, aligned 0x7fb06cfe8000 parent (nil)
23083305: PID(69421): Freeing KMQ BO, type=AMDXDNA_BO_SHMEM, drm_bo=6, size=128
23084066: PID(69421): Unmap BO, aligned 0x7fb06d023000 parent (nil)
23119464: PID(69421): Destroying KMQ pcidev
23120456: PID(69421): Device node fd leaked!! fd=3

Time spent in running the model: 43.0 [ms]
Running 'compare' with rtol=1e-06 and atol=1e-06
Values are not all close. Here is a summary of the differences:
Number of positions where values are different is 32 out of 32
Maximum difference:<PHONE_NUMBER>776.0
Discrepancies:
At index: 0 0
AIE value: 0.0
CPU value:<PHONE_NUMBER>328.0
At index: 0 1
AIE value: 0.0
CPU value:<PHONE_NUMBER>776.0
At index: 0 2
AIE value: 0.0
CPU value:<PHONE_NUMBER>616.0
At index: 0 3
AIE value: 0.0
CPU value:<PHONE_NUMBER>472.0
At index: 1 0
AIE value: 0.0
CPU value:<PHONE_NUMBER>528.0
At index: 1 1
AIE value: 0.0
CPU value:<PHONE_NUMBER>472.0
At index: 1 2
AIE value: 0.0
CPU value:<PHONE_NUMBER>216.0
At index: 1 3
AIE value: 0.0
CPU value:<PHONE_NUMBER>240.0
At index: 2 0
AIE value: 0.0
CPU value:<PHONE_NUMBER>656.0
At index: 2 1
AIE value: 0.0
CPU value:<PHONE_NUMBER>400.0
And 22 more discrepancies...

So no crash but not numerically correct.

I guess their implementation of multiple kernels has changed (I vaguely remember Sonal discussing this during our on-site).

So seems iree_hal_xrt_direct_command_buffer_dispatch is getting called twice when XRT already has both dispatches or commands buffers or whatever and doesn't except it to get called twice? Maybe @nirvedhmeshram has some insight.

For the multi-kernel test this is expected for the way it is implemented, if you have N calls in you test to M dispatches, you will see N iree_hal_xrt_direct_command_buffer_dispatch


Passing arguments to onclick

Not sure if this is doable in 0.3.0, but I want to pass arguments to my onclick method like so:
button onclick=@MyMethod(MyParam)
Doing so results in a compile error of "CS 1503 cannot convert from 'void' to 'string'"

try <button onclick=@(() => Mymethod(MyParam)) />

Thanks, @Suchiman!

Thanks guys!  Side note: So happy you guys are responding so quickly to help requests, but is there a better spot for these?  I feel guilty opening issues when they aren't really issues :P

@alexanderjwoods https://gitter.im/aspnet/Blazor is a quite active place 😉

@alexanderjwoods You can also post questions on StackOverflow. That way they can be referenced over time

@Suchiman Awesome!  I will post over there unless it is a real bug!
@danroth27 I refuse to post on SO because of their attitude toward beginners.  I have had more than one intern tell me they don't like SO because of the condescending nature toward "n00b" questions, and I quite agree.  It turns people off to programming at worst, and at best it makes it harder for people to become proficient!

Fair enough. I think you'll find the folks on the Blazor Gitter to be very helpful.

when doing it within svg got following error:
Uncaught Error: Microsoft.AspNetCore.Blazor.Browser.Interop.JavaScriptException: Cannot set attribute on non-element child
Error: Cannot set attribute on non-element child
at BrowserRenderer.applyEdits (BrowserRenderer.ts:76)
at BrowserRenderer.updateComponent (BrowserRenderer.ts:33)
at renderBatch (Renderer.ts:45)
at Array. (mono.js:1)
at _emscripten_asm_const_iiiiii (mono.js:1)
at wasm-function[2679]:25
at wasm-function[3541]:44
at wasm-function[3548]:408
at wasm-function[3550]:224
at wasm-function[3766]:189
at Microsoft.AspNetCore.Blazor.Browser.Interop.RegisteredFunction.InvokeUnmarshalled[T0,T1,T2,TRes] (System.String identifier, T0 arg0, T1 arg1, T2 arg2) <0x1c21a40 + 0x00054> in <92ed9c6772a34798bccecc99cc26cbcd>:0
at Microsoft.AspNetCore.Blazor.Browser.Interop.RegisteredFunction.InvokeUnmarshalled[T0,T1,TRes] (System.String identifier, T0 arg0, T1 arg1) <0x1c21958 + 0x00022> in <92ed9c6772a34798bccecc99cc26cbcd>:0

This doesn't work now (Blazor Preview 9 Rc1 everything else):
try <button onclick=@(() => Mymethod(MyParam)) />
I get a message Cannot convert lambda expression to type 'object' because it is not a delegate type.


Refactored ListView into RecycleView

Refactored ListView into a RecyclerView to get shared element transitions working again for > Oreo devices.
Implemented custom RecyclerView to support using an empty view and a
auto fitted grid layout.
Reenabled shared element transitions for Oreo
NOTE: As I am not able to test the PVR section I have not refactored the PVR activities and fragments.

Just to let you know that it will take some time reviewing this. Not only it's a big PR, but given that we just made a beta release, we want to keep the codebase as is, just in case we need to fix something before going into prod.

Hey @poisdeux i can merge this as is, the only comment i have is the inclusion of that seemingly unnecessary file, but it can be removed later.
Do you want to remove it or can i merge this?

Which file is unnecessary?

Sorry I forgot to submit the review...
It's the divider.xml (I'm on mobile so I'm not able to link to it)

Hold the merge!
I just tested this once more and now it keeps failing in the onCreateViewHolder method in the new recyclerview adapter for Addons...

Ok. I'm not going to be able to test/merge this for the next week
(vacations) so if you feel it is good to go feel free to merge it yourself.
I didn't see any showstopper
On Tue, 24 Jul 2018, 21:34 Martijn Brekhof<EMAIL_ADDRESS>wrote:

Hold the merge!
I just tested this once more and now it keeps failing in the
onCreateViewHolder method in the new recyclerview adapter for Addons...
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/xbmc/Kore/pull/550#issuecomment-407542935, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AFwrQM4EdyRSVns-yMe5TwQaCK8Q3rAiks5uJ4TlgaJpZM4UVtWz
.


Duplicate logging on el7

On a base installation of CentOS 7 using this module and the following parameters, double logging can be seen under /var/log/messages.
  class { 'rsyslog::client':
    log_local => true,
    log_auth_local => true,
    log_remote => false,
  }

Example:
$ logger test

yields
Oct 21 16:44:03 mad-west-vcs006 centos: test
Oct 21 16:44:03 mad-west-vcs006 centos: test

This seems to be something to do with the journal integration added in el7.
One of the configuration items in the stock config (removed under puppet) fixes this behaviour
# Turn off message reception via local log socket;
# local messages are retrieved through imjournal now.
$OmitLocalLogging on

Should this be made part of the stock config for el7?

Any news on this?


is_genesis_trigger fails to verify deposit merkle branch

Issue
Deposits are processed against empty state with undefined eth1_data while genesis_eth1_data should be in place.

Closing in favour of #1202 which addresses the issue.


More flexible gdp adapter

Fixes #94 and #235.

CI on Ubuntu and macOS has been failing since yesterday due to segmentation fault in codecov (tests otherwise pass). Windows is OK. I haven't found anything about this from a cursory search. It could be transient or not.
@selipot unless you really want codecov to succeed, I think we can safely merge.

What happened to all the tests?

@philippemiron see https://github.com/Cloud-Drift/clouddrift/pull/237#issuecomment-1693354094. Codecov is crashing in CI. It started as yesterday as far as I know.

@philippemiron see #237 (comment). Codecov is crashing in CI. It started as yesterday as far as I know.

I see.. and it's passing for Windows. 😕

For once something works in Windows :)

I feel a Microsoft bias in here.. haha

I'd say just keep it in this PR for simplicity.

I'd like to know if it resolves the issue of not being able to generate the parquet file via awkward

Hmm, works for me. What about it doesn't work for you? Is there an error message?

I cannot create a parquet file: it requires to use to_awkward from the ragged array instance and this fails because clouddrift.raggedarray.to_akward expects "count" instead of "rowsize"

@selipot A-ha, I see this now. OK, I think your commit worked, because I can write a parquet file via awkward.

I'd like to know if it resolves the issue of not being able to generate the parquet file via awkward

Hmm, works for me. What about it doesn't work for you? Is there an error message?

Wrote earlier: For some reason I can't get my environment to recognize my latest commit and test it.
So I am not that skilled at VS code!

The tests fail because of rowsize


Link preview text on hover are inconsistent

Our link preview text on hover are inconsistent across various views, we should work to make all of them consistent.
Terminal

Markdown

Webviews

Related: #78125

Related https://github.com/microsoft/vscode/issues/77964

https://github.com/microsoft/vscode/pull/83175 was merged which fixes the terminal hover position, created https://github.com/microsoft/vscode/issues/83293 so we can verify it in endgame.

This looks like it got fixed:


strcmp with nullptr

Hi @Guldoman,
I like the album art check. But I do get the motivation why you used bitfields (does not seem necessary).
However, maybe we should avoid passing NULL to strcmp here
https://github.com/Guldoman/mpv_inhibit_gnome/blob/a8b4d2c9be24baa7438e65778de512191406bb11/src/main.c#L45
https://github.com/Guldoman/mpv_inhibit_gnome/blob/a8b4d2c9be24baa7438e65778de512191406bb11/src/main.c#L149
See here:
https://stackoverflow.com/questions/21865041/in-c-what-exactly-happens-when-you-pass-a-null-pointer-to-strcmp
Additional confusion:
Why do we even need  {NULL                            , MPV_FORMAT_NONE  }?
Do we really need to check value = data > 0 why not leave it at value = data != 0?
bool value = *(int64_t *)prop->data > 0;

However, maybe we should avoid passing NULL to strcmp here

strcmp won't receive NULL from us because the for exits when it finds a NULL name:
https://github.com/Guldoman/mpv_inhibit_gnome/blob/a8b4d2c9be24baa7438e65778de512191406bb11/src/main.c#L147

Additional confusion: Why do we even need {NULL , MPV_FORMAT_NONE }?

I used it to find the end of the array without having to keep track of its length.

Do we really need to check value = data > 0 why not leave it at value = data != 0?
bool value = *(int64_t *)prop->data > 0;

I don't know if mpv can send us a negative track, but I doubt it would be a valid one, so > 0 covers that case.

Ah okay, but wouldn't the union struct with
typedef struct __attribute__((packed)) {
	bool pause;
	bool idle_active;
	bool stop_screensaver;
	bool window_minimized;
	bool mute;
	bool albumart;
	bool vid;
	bool aid;
} PropsFlags;

and for(unsigned i = 0; i < sizeof(PropsFlags); i++) be less confusing?

To be honest I find it more confusing, mostly because it needs the packed attribute to even work.
Also consider that bool is only guaranteed to be enough to contain true/false, but it can be bigger than 1 byte, which would make sizeof not work as you'd expect it to.

All right :+1:


Admgecko

I'm happy with the contents changes and hierarchy changes. @einsteinagogo can you review the content additions as per your discussion?

Yes, I think the only addition I would make would be front end (management tools).


⚠ Cioos Pacific CKAN Sitemap has degraded performance

In 822c482, Cioos Pacific CKAN Sitemap (https://catalogue.cioospacific.ca/sitemap/sitemap-1.xml) experienced degraded performance:

HTTP code: 200
Response time: 20016 ms

Resolved: Cioos Pacific CKAN Sitemap performance has improved in e111fd4 after 5 minutes.


Touch events do not get passed to the state machines

Description
I'm trying several rive animations from the community site and although RiveAnimationView obviously consumes touch events, they seem to never arrive to the actual scene: it continues playing "idle" animations. At the same time clicking scene on the community web site does play animations.
Provide a Repro

Insert this scene into any RiveAnimationView
Try clicking on the view
Observe that nothing happens

Source .riv/.rev file
Download riv file from here:
https://rive.app/community/3783-7910-point-and-click/
Device & Versions (please complete the following information)

Device: Samsung S10fe
Android SDK API Level 30
Rive-android version: 4.2.2

Thanks @dimsuz ! We'll take a look to see what's going on

This should be resolved in the latest version. Please reopen and provide sample code if that is not the case.


Add api for Memberships

Coverage increased (+0.05%) to 85.46% when pulling d6144c008fa0267393c3b02a5b2e8f023e32e6f0 on membership_api into 0c2b0a2717b3801a478a7b4c73fc7f21c3819310 on master.


Update dependencies

Looks good to me, but the PR has been open for quite a while now, should it be updated again? @jasperat


[minifb] Bump to 2023-01-30

Bump to 2023-01-30
Fix install
Add license
Modernization

Tested usage
cmake_minimum_required(VERSION 3.2)
project(test_minifb)

set(CMAKE_CXX_STANDARD 14)

find_package(minifb CONFIG REQUIRED)

add_executable(test_minifb main.cpp)

target_link_libraries(test_minifb PRIVATE minifb::minifb)

#include <iostream>
#include <MiniFB_cpp.h>
#include <cmath>

int main() {
    const int windowWidth = 1280;
    const int windowHeight = 960;
    auto *window = mfb_open_ex("minifb", windowWidth, windowHeight, 0);

    float scaleX, scaleY;
    mfb_get_monitor_scale(window, &scaleX, &scaleY);

    const int width = (int) round(float(windowWidth) * scaleX);
    const int height = (int) round(float(windowHeight) * scaleY);

    auto *image = new uint32_t[width * height];

    do {
        for (int i = 0; i < width * height; ++i) {
            image[i] = rand();
        }
        int state = mfb_update_ex(window, image, width, height);

        if (state < 0) {
            window = nullptr;
            break;
        }
    } while (mfb_wait_sync(window));

    delete[] image;

    return 0;
}

Also open a pr on upstream https://github.com/emoon/minifb/pull/102.
Please wait before merging this pr.
If upstream willing to take the change, then we can remove fix-install.patch file.

I manually checked that the SHA in here is not affected by https://github.com/microsoft/vcpkg/issues/29288

@MonicaLiu0311 @dg0yt Since upstream has accepted https://github.com/emoon/minifb/pull/102, I've update this pr, it's ok to be merged now.


Matmul example fails.

I copy sample codes from tutorial[https://triton-lang.org/main/getting-started/tutorials/03-matrix-multiplication.html] section Final Results and Unit Test. Exception shows :
Traceback (most recent call last):
File "matmul.py", line 135, in
triton_output = matmul(a, b)
File "matmul.py", line 122, in matmul
matmul_kernel[grid](
File "/python3.8/site-packages/triton/runtime/autotuner.py", line 81, in run
self.cache[key] = builtins.min(timings, key=timings.get)
TypeError: '<' not supported between instances of 'tuple' and 'float'

I downloaded source code, and it raises the exception too.

In addition, layer-norm example code crushes:
Argument rematerialization not implemented
UNREACHABLE executed at /project/lib/Dialect/TritonGPU/Transforms/TritonGPUConversion.cpp:45!
[1]    475998 abort (core dumped)  python 05-layer-norm.py
Are the docs out-of-updated?

Trying to debug by device_print but got an exception of no such function:

...
AttributeError: module 'triton.language' has no attribute 'device_print'.

I'm running samples with python3.8 and triton 2.0.0

(test) ➜  tutorials python
Python 3.8.16 (default, Mar  2 2023, 03:21:46)
[GCC 11.2.0] :: Anaconda, Inc. on linux
Type "help", "copyright", "credits" or "license" for more information.
>>>
(test) ➜  tutorials pip list | grep triton
triton                        2.0.0

Problem solved. Tutorials from https://triton-lang.org/main/getting-started/tutorials/03-matrix-multiplication.html use codes of main branch and there are a lot updates since release 2.0.0.
Use sample code from tag v2.0.0. It works fine.


On opening a new project, seeing "leaveAccessor is not a function" error

Repro:

New Project
See orange error banner with "leaveAccessor is not a function" error

Only showed up once for me :(
System:
Chromebook
Chrome OS 77

Huh, that's being thrown from the simulator...

I can reproduce the error. Simply use the betaversion. But you have to use blocks!

Thanks @MKleinSB, that screenshot is helpful! It looks like you have the old simulator, which should have been upgraded automatically.
I think this is probably an issue with mismatched simulator versions and shouldn't be an issue going forward. If you still aren't seeing the new simulator (screenshot below), try refreshing a few times.

If that doesn't work, open up the developer tools in the browser and see if you get any errors in the console.

Thanks, but makecode beta still refuses to upgrade the new simulator. But only in Firefox 72.0.2 (on Win10).
On Chrome the new simulator appears in beta, too and it works fine!

Hmmmm.... Clearing the cache might fix it. Be careful, though, back up any projects before you try that. We store projects in indexedDB so if that gets cleared they will be lost forever.

Yes, clearing the cache fixed it! Thanks.


bad udp cksum when using vxlan

When using Ubuntu 20 with kernel 5.8 (after apt upgrade), we see again the issue https://github.com/rancher/rke2/issues/1541. The workaround is disabling the checksum offload in the calico.vxlan interface.
The issue appears when from a node we try to access a service which is implemented by a pod in another node, e.g. coredns service. In that case, the traffic must traverse de vxlan tunnel and in the receiving node, we see in tcpdump:
06:29:06:4b:55:9a > 06:47:af:1e:bf:ca, ethertype IPv4 (0x0800), length 143: (tos 0x0, ttl 64, id 16380, offset 0, flags [none], proto UDP (17), length 129)
    <IP_ADDRESS>.7103 > <IP_ADDRESS>.4789: [bad udp cksum 0xbd67 -> 0x4e7f!] VXLAN, flags [I] (0x08), vni 4096

Expected Behavior

When deploying calico as cni plugin in Ubuntu 20 with kernel 5.8 (after apt upgrade), I expect to access successfully all kubernetes services from any node (e.g. dns). When doing tcpdump at the receiver node, I expect to see:
06:29:06:4b:55:9a > 06:47:af:1e:bf:ca, ethertype IPv4 (0x0800), length 143: (tos 0x0, ttl 64, id 21070, offset 0, flags [none], proto UDP (17), length 129)
    <IP_ADDRESS>.50078 > <IP_ADDRESS>.4789: [udp sum ok] VXLAN, flags [I] (0x08), vni 4096

Current Behavior

When deploying calico as cni plugin in Ubuntu 20 with kernel 5.8 (after apt upgrade), I can only access services (e.g. dns) from the node, if the pod implementing the service is in that node, i.e. as soon as the traffic must take the vxlan tunnel, things don't work. When doing tcpdump in the receiver node, I see:
06:29:06:4b:55:9a > 06:47:af:1e:bf:ca, ethertype IPv4 (0x0800), length 143: (tos 0x0, ttl 64, id 16380, offset 0, flags [none], proto UDP (17), length 129)
    <IP_ADDRESS>.7103 > <IP_ADDRESS>.4789: [bad udp cksum 0xbd67 -> 0x4e7f!] VXLAN, flags [I] (0x08), vni 4096

Possible Solution

sudo ethtool -K vxlan.calico tx-checksum-ip-generic off
or
featureDetectOverride: "ChecksumOffloadBroken=true"
But this have a performance impact
Steps to Reproduce (for bugs)

1.Deploy Kubernetes on Ubuntu 20 with kernel 5.8 in 2 or more nodes
2. Run dig @<IP_ADDRESS> www.google.com in all nodes. It will only work in one (assuming <IP_ADDRESS> is the clusterIP of the dns service)
3.
4.
Context

Your Environment

Calico version 3.19
Orchestrator version (e.g. kubernetes, mesos, rkt): kubernetes
Operating System and version: Ubuntu 20 kernel 5.8
Link to your project (optional): https://github.com/rancher/rke2/issues/1541

@fasaxc recently put in a fix to automatically disable checksum offload based on the kernel version, but it sounds like perhaps there are some kernels for which that fix isn't working properly?
I think the best solution I'm aware of at the moment is to explicitly disable the offload as you suggested in your post.


etl-framework available ?

https://massstreetuniversity.com/etl-framework/

Is this a dead link? Where is it? I was tired of having a cryptic URL so I gave it it's own URL. You can find it at: http://www.datawarehouseetlframework.com/

@kiquenet Before I closed this, I wanted to make sure you saw my comment. I figured you'd get a notification but tagged just in case.

URL OK : http://www.datawarehouseetlframework.com/


helm: Allow configuration of Envoy --base-id for Envoy DaemonSet via Helm values

This PR allows manual configuration of Envoy --base-id via Helm for the Cilium Envoy DaemonSet.
Currently, Envoy --base-id is hard coded to use 0 for the Cilium Envoy DaemonSet which can lead to Envoy startup issues if another Envoy based solution on the same node uses the same --base-id.
Having the --base-id user configurable allows to solve such issues easily.
This fixes e.g. kubermatic/kubermatic#12922

Thank you! Please also make sure to squash your commits into one

Can you merge 2 commits into one ?


Solve PublicKey ambiguity

Two distinct structs share the same name: bitcoin_hashes::PublicKey and bitcoin::util::key::PublicKey. The first one is re-exported by the bitcoin crate, creating confusion and necessity to use as for importing when both of them are used in the same mod.
I propose to rename one of the structs to avoid the confusion. I have already done it in my local branch, renaming bitcoin::util::key::PublicKey into BitcoinPubkey, so I can PR this version if you will accept this idea.

I think you're talking about secp256k1::PublicKey, right?
We're aware of the confusion. I was personally against the new bitcoin::PublicKey and bitcoin::PrivateKey types because they don't really add any value over the original secp256k1 types.
However, now that they exist, I think most of us are aware of both types and when to use which. Basically you shouldn't ever have to use the secp256k1 types anymore.

Yes, sure, I meant secp256k1::PublicKey. Yes, I'm aware about the difference, however PublicKey/EC cryprography can be used outside of the scope of Bitcoin transactions, where scep256k1 will be usable again — we have this situation in the RGB project (off-chain client-validated data), that's why I had actually made this proposal.

secp256k1::PublicKey does not have any notion of compressedness. bitcoin::PublicKey does. There is no way to equate these types.
I do not want to rename bitcoin::PublicKey to BitcoinPublicKey. You save two characters in exchange for making it less clear what crate the type came from.

What was the reason for adding this notation of compressedness through bitcoin::PublicKey instead changing secp256k1::PublicKey?

I do not want to rename bitcoin::PublicKey to BitcoinPublicKey. You save two characters in exchange for making it less clear what crate the type came from.

Just to notice, if you have to use both version (which is the case in some of my code, you end up not with bitcoin:PublicKey and  secp256k1::PublicKey but rather
use bitcoin::secp256k1::PublicKey;
use bitcoin::util::key::PublicKey as BitcoinPubkey;

or
use bitcoin::secp256k1::PublicKey as SecpPubkey;
use bitcoin::PublicKey;

i.e. there is no simple way of having bitcoin:PublicKey and secp256k1::PublicKey without  importing the whole secp256k1::*; so one is forced to stick with  as-renaming for at least one of them during the import.

i.e. there is no simple way of having bitcoin:PublicKey and secp256k1::PublicKey without importing the whole secp256k1::*; so one is forced to stick with as-renaming for at least one of them during the import.

The names are "standard Rust", at least according to RFC356.
Given that RFC, I've found it helpful to import the next high-level namespace to disambiguate instead of using as:
use bitcoin::secp256k1;

and then in your code do:
let bitcoin_key = bitcoin::PublicKey::new(...)
let secp_key = secp256k1::PublicKey::new(...)

Personally, I try to avoid renaming imports with as. If your IDE supports searching for types and you use as a lot, it forces you to always check the imports first before you can do a type-search (otherwise you get 0 results and wonder where the type is coming from).
Another approach (which we ended up going for), is to simply define your own type PublickKey type that wraps whichever one is more suited for your case and provide conversion methods.
Still, I'd be curious for why there was a need to actually introduce a new type @apoelstra ?
Did you just not want to make a breaking change to secp256k1::PublicKey or is there another reason why the feature of compressedness does not fit into secp256k1::PublicKey?

I do agree that the compressedness flag shouldn't be in secp256k1::PublicKey, though. Au contraire, I actually think it shouldn't be in bitcoin::PublicKey either :P
It's not a property of the key itself. A public key is a point on the curve. So a pubkey type should have the information to represent that point on the curve. The compressedness flag only indicates which format is used to serialize the key into bytes. So it's a property of the serialization rather than the key itself.
However, in Bitcoin, the mode of serialization is relevant for determining which address is generated. So in Bitcoin, the compressedness flag is also a property of how the key is (intended to be) used. I believe that's why it's included in the bitcoin::PublicKey type. While still not semantically correct to have it there, it avoids some annoyances and prevents some easy mistakes.
Like a WIF "Bitcoin private key" also contains a flag to indicate that addresses for that private key should be generated with a compressed or uncompressed public key.
Andrew and probably others, value the ability to make the following series of conversions:
WIF string -> PrivateKey -> PublicKey -> Address

If PrivateKey and PublicKey don't keep track of this compressedness flag, it would have to be brought along when calculating an Address.

About a year ago I did a PR that removed the bitcoin::*Key types and just used the secp256k1::*Key types. I don't recall the exact implementation, but basically it'd have a util::wif::WIF type that is strictly used for WIF serialization and would just be a tuple of secp256k1::SecretKey and the compressed flag. WIF is not a very common format anymore anyway since BIP39 seeds became the standard. Also uncompressed public keys are very much discouraged and only still used in legacy systems. That aside, the PR was closed and I got over it (clearly) :sweat_smile:

Also, this will all change once again with Taproot and Schnorr pubkey serialization. In general, I find concept of Bitcoin addresses highly legacy... May be it worth to take it all together and leave for upper layers on top of rust-bitcoin (such as wallets)? So we can not bother with anything that is related for the intentions of using public key in some way at the user level.

Well for users PublicKey==Bitcoin Address.
An Address is somewhat a template specifying a scriptPubKey.
anyhow. secp doesn't care about scriptPubKey and/or Bitcoin Address.
Bitcoin does. at least for users, I do think we should provide Bech32/Base58/WIF formats.
I wouldn't care if you rename bitcoin::PublicKey to Address. but other than naming I see no reason for anything to change


option parsing: continue building ip tuple list for lengths > 1

Without this patch, we only get the first tuple from the list.

Great finding !
I will only merge once you add yourself to the copyright 🎉

Merged, I've added you to dhcp_wire.ml and test.ml as well on the copyrights.
I'm keeping the line by line format.
I had written this message before, just can't find it. Maybe I commented out on a totally unrelated project :D


Programmer doesn't seem to be forwarded to Arduino.cmake

When running the upload target, I get the following message:
avrdude: no programmer has been specified on the command line or the config file
         Specify a programmer using the -c option and try again

I guess PROGRAMMER_ID isn't set properly by the plugin.
Possibly related to #6.

@TheAssassin, I think there is still an issue of configuration for project failing to initialize selections for new boards. I will double check that it has not regressed.

Thanks, let me know if you need more information.

I think I saw and fixed this bug but did not add it to the pending fixes.
The only thing I have in the version notes:

Fix: templates

#ifdef to #ifndef
remove User_Setup.h
add README.md to arduino library template
add missing ; after lib class declaration

I will keep the bug open until I can confirm or deny it.

fix for get ARDUINO_SDK_PATH from ENV on Windows:
ArduinoToolchainFiles.java, after line 52 insert code:
                if (System.getProperty("os.name").toLowerCase().contains("nux"))
                {
                    String arduinoSdkPathEnv = System.getenv("ARDUINO_SDK_PATH");
                    String unixPathsString =
                            "set(ARDUINO_SDK_PATH \"" + arduinoSdkPathEnv + "\")" + "\n" +
                                    "message(\"ARDUINO_SDK_PATH ${ARDUINO_SDK_PATH}\")";

                    VirtualFile unixPaths = platformDirectory.createChildData(this, "UnixPaths.cmake");
                    OutputStream unixPathsOutputStream = unixPaths.getOutputStream(this);
                    InputStream unixPathsInputStream = IOUtils.toInputStream(unixPathsString);

                    try {
                        IOUtils.copy(unixPathsInputStream, unixPathsOutputStream);
                    } finally {
                        closeStreams(unixPathsOutputStream, unixPathsInputStream);
                    }
                }

after this user can define ARDUINO_SDK_PATH in ~/.bashrc, for example:
echo "export ARDUINO_SDK_PATH=/home/robert/bin/arduino-1.8.11" >> ~/.bashrc
but CLIon must be started using terminal for proper loading env in jvm, for example:
~/bin/clion-2019.3.3/bin/clion.sh
I this solution on Ubuntu 16, and Ubuntu 18.


Can it play from a specific time?

i want to stream youtube from a specific time
so i add youtube url ?t=30 (likes 'https://www.youtube.com/watch?v=sqgxcCjD04s&t=149s')
but this plays the song from the beginning
const stream = ytdlexec(startUrl, {     o: '-',     q: '',     f: 'bestaudio[ext=webm+acodec=opus+asr=48000]/bestaudio',     r: '100K',   }, { stdio: ['ignore', 'pipe', 'ignore'] })
this is my code

I don't think so


Using a own task for cleanup of worker threads.

Made a attempt to fix #258 and #342
Using an own task for cleanup of the two worker tasks in client, the usage of an own task for cleanup is due to an deadlock when the two worker threads start so wait for each other ...

This was a bad attempt to fix this deadlock


[WIP] Better fix for different CoreOS os family facts

Signed-off-by: Bogdan Dobrelya<EMAIL_ADDRESS>

ci check this

Passed pipeline after resolving the preemptive instances issue. Merging!


Panic on slice bounds

I have an executable in the release/ folder, the release seems to get created and a tag created too, but no files are actually uploaded against the release.
panic: runtime error: slice bounds out of range

goroutine 1 [running]:
main.MimeType(0xc20800a030, 0x1, 0x1)
	/home/ubuntu/gh-release/gh-release.go:65 +0x35d
github.com/progrium/go-basher.(*Context).HandleFuncs(0xc20805e000, 0xc20800a000, 0x4, 0x4, 0x0)
	/home/ubuntu/.go_workspace/src/github.com/progrium/go-basher/basher.go:174 +0x2e5
github.com/progrium/go-basher.Application(0xc20803a420, 0xc20805af68, 0x1, 0x1, 0x193478, 0x1)
	/home/ubuntu/.go_workspace/src/github.com/progrium/go-basher/basher.go:56 +0x30e
main.main()
	/home/ubuntu/gh-release/gh-release.go:106 +0x21c

goroutine 2 [runnable]:
runtime.forcegchelper()
	/usr/local/go/src/runtime/proc.go:90
runtime.goexit()
	/usr/local/go/src/runtime/asm_amd64.s:2232 +0x1

goroutine 3 [runnable]:
runtime.bgsweep()
	/usr/local/go/src/runtime/mgc0.go:82
runtime.goexit()
	/usr/local/go/src/runtime/asm_amd64.s:2232 +0x1

goroutine 4 [runnable]:
runtime.runfinq()
	/usr/local/go/src/runtime/malloc.go:712
runtime.goexit()
	/usr/local/go/src/runtime/asm_amd64.s:2232 +0x1

Yeah I ran into this today as well. I'm assuming the API changed a bit.


Render view on top of splash screen?

Before, when I was using @dbonner1987's code, I was able to render a View on top of my splash screen (in practice, the splash screen had a blue background + text and the View had a loading gif). However, because the background was blue, it would show through when I did opacity transitions.
NSArray *allPngImageNames = [[NSBundle mainBundle] pathsForResourcesOfType:@"png" inDirectory:nil];
  for (NSString *imgName in allPngImageNames){
    if ([imgName containsString:@"LaunchImage"]){
      UIImage *img = [UIImage imageNamed:imgName];

      if (img.scale == [UIScreen mainScreen].scale && CGSizeEqualToSize(img.size, [UIScreen mainScreen].bounds.size)) {
        rootView.backgroundColor = [UIColor colorWithPatternImage:img];
      }
  }
}

Now, using your node module, I no longer see the View on top of my splash screen. Any way to fix this?
Thanks!

@nezaid, tagging you just in case you know the answer and will respond quickly :).

As far as i am conserned you can hide splash screen any time you want to and then show something else.

But I wanted to keep the splash screen and then add a gif on top of it. The splash screen has the title of the app and the gif is a loading indicator.

Maybe this is because @dbonner1987 used a background color but @mehcode uses a loading view...

But I wanted to keep the splash screen and then add a gif on top of it. The splash screen has the title of the app and the gif is a loading indicator.

I understand what you want. We could probably do something here with a native method exposed in JavaScript. Try adding a second UIImageView in the Show method and composing them to get what you want and then separate it out into a second method we can expose. I don't mind merging that in.

I'm closing this but I'd still accept a PR to do something.

Sorry to bring this dead issue back - but I would like to be sure no one is working and that this is still a valid idea for a PR before I start hacking something.
I am thinking not just a spinner, but a <Text /> (to show the version number - that are interesting in some business apps to be sure users are up to date). Thoughts?


Add $MIGUEL: Miguel Piedrafita

[x] I understand that token listing is not required to use the Polygon Wallet with a token.

Please provide the following information for your token.
Token Name: $MIGUEL
Token Symbol: Miguel Piedrafita
Decimals: 18
Polygon Address: 0x9EeAb5dACAD364017c400bFBf717543A0e3C35CB
Ethereum Address: 0x9EeAb5dACAD364017c400bFBf717543A0e3C35CB
Logo URI: https://miguelpiedrafita.com/images/_site/logo.png
Project Name: Miguel Piedrafita
Project Summary: Social token for Miguel Piedrafita
Project Contact<EMAIL_ADDRESS>Project Website: https://miguelpiedrafita.com
Mainnet/Testnet: Mainnet
Additonal Notes:

Token mapped to child address 0x4B9D66B1d0343EB2f16ebc427270718b7E010f78.


Question

can i convert interface{} or []interface{} to []model

No since they are of different types. it would help to know more about what you are doing?

This package converts map[string]interface{} to struct.
I am trying to convert direct interface{} to struct.
That interface{} is nothing but map[string]interface{} only.

So these type assertions have to be accurate. Trying to work with a raw map[string]interface{} that has JSON is tedious. If you can define a Go struct that is better. This is what this package is trying to provide support for.
Give a variable of type interface{} named a and a struct type named type Example struct
The type assertion syntax of
e, ok := a.(Example)
would allow you to get a copy of the Example value inside of a and assigned to variable e with ok being true. If ok is false, then there was not a value of type Example inside of a


PDF content is empty in IE11

Attach (recommended) or Link to PDF file here:
Configuration:

Web browser and its version: IE 11
Operating system and its version: win 7
PDF.js version:  master, 2.0843, 1.10
Is a browser extension:

Steps to reproduce the problem:

Start the service and enter the URL(http://localhost:8888/web/viewer.html) by IE11

What is the expected behavior? (add screenshot)

Start the service and enter the URL(http://localhost:8888/web/viewer.html) by IE11

Please refer to the README, in particular https://github.com/mozilla/pdf.js#getting-the-code which explains why this doesn't work:

[...]
and then you can open:

http://localhost:8888/web/viewer.html

Please keep in mind that this requires an ES6 compatible browser; refer to Building PDF.js for usage with older browsers.

The built viewer, see e.g. the demo available at https://mozilla.github.io/pdf.js/web/viewer.html, does work in IE11. However IE11 cannot be used for development, as is also mentioned in https://github.com/mozilla/pdf.js/wiki/Contributing#prerequisites.


Text paragraph text index <=> position bug

Conversion from text index to position and the other way around does not work thus leading to weird cursor issues.

fixed and added examples that also involve a real (moz2d or cairo) backend


Checking the status of a node takes too much time

Description
Checking the status of a node takes too much time. i takes around a minute to check the status of a node.
Get Mastodon (33).webm

The issue is still valid. as seen below it took almost 2 mins for the first node to check whether it's online or not. and for the second node it also took more than a minute to check
Get Mastodon (39).webm

fixed in the grid client, but the instance needs to be updated

@Mahmoud-Emad Please update the issue after updating the instance

@AhmedHanafy725
The instance should be updated now

@AhmedHanafy725 The instance should be updated now

Tested a couple of different nodes after the instance was updated, but it still take around 1 min.
Get Mastodon (42).webm
Get Mastodon (43).webm

@mohamedamer453 Should be fixed https://github.com/threefoldtech/www-mastodon/pull/137

Verified on https://getmastodon.threefold.io/
By checking the same 2 nodes the time was cut by more than half and now it only takes around 20 seconds to check the status of the node.


More settings options

Allows making checkboxes to configure Danser inside Settings frame.

Need to delete all danser related files such as logs and settings before merging.


exafmm cannot be included in multiple compilation units

Including the library in multiple cpp files fails with a linker error, because of duplicate symbols:
$ g++ a.cpp b.cpp
/tmp/ccKkGMZN.o:b.cpp:(.bss+0x0): multiple definition of `exafmm::P'
/tmp/cctPFgHT.o:a.cpp:(.bss+0x0): first defined here
/tmp/ccKkGMZN.o:b.cpp:(.bss+0x4): multiple definition of `exafmm::NTERM'
/tmp/cctPFgHT.o:a.cpp:(.bss+0x4): first defined here
/tmp/ccKkGMZN.o:b.cpp:(.bss+0x8): multiple definition of `exafmm::NCRIT'
/tmp/cctPFgHT.o:a.cpp:(.bss+0x8): first defined here
/tmp/ccKkGMZN.o:b.cpp:(.bss+0x10): multiple definition of `exafmm::THETA'
/tmp/cctPFgHT.o:a.cpp:(.bss+0x10): first defined here
collect2: error: ld returned 1 exit status

Minimal example:
a.cpp
#include "b.h"
int main() {
        return 0;
}

b.cpp
#include "b.h"
void dosomething() {
        return;
};

b.h
#ifndef _B_H_
#define _B_H_
#include "exafmm.h"

void dosomething();
#endif

My actual problem is, that I am using exafmm in a class, which has exafmm::Bodies member variables. This means the header file for the class needs to include exafmm, which defines the global variables in the compilation unit using the class and the source file of the class contains them as well.

There are many ways to solve this problem, and I am wondering which one to choose. I could stop using global variables, and use more class variables. Or I could have macro switch for the header and source files, which will automatically add extern in front of the global variables when exafmm.h is called from the headers. Either way, it doesn't feel like the optimal solution. Let me think about this. In the meantime you can use the above mentioned hack.

At the moment I split exafmm.h into exafmm_types.h, exafmm_body.h and exafmm_cell.h.
This helps for my class declaration, but won't help on projects which need to run it in different source files.


Speed issues [enhancement]

First of all, I like the style of standard and I use it in all projects I started lately.
Buuuut, it always takes several seconds to test a single file in my test directory even if it only requires small files. My test file is about 150 LOC, the source is in this case even smaller.
Maybe this is an issue not with standard it self but with one of it's dependencies ... this is really interrupting my workflow.
<EMAIL_ADDRESS>Intel Core i5, 4 Gigs RAM, Ubuntu 14.04

After investigating, there are two main places where time is spent:

~500ms. Locating the eslint and jscs binaries with the npm run which-eslint and npm run which-jscs commands. We locate the binaries in parallel. See https://github.com/feross/standard/pull/30 for why we need to do this.

~400ms running the linters. We run the linters in parallel.

We could speed things up a lot by using the eslint and jscs JS APIs directly, instead of relying on the command line programs. This would eliminate step 1 entirely, and eliminate a child_process.exec call in step 2.
PR welcome – I don't have time for this right at the moment.

I'll dig into that when I have some free time, maybe I can do something ... :+1:

eh, first try at using eslint directly... WIP
https://github.com/Flet/standard/commit/6bb49bac0ca0b70a47a154b7cddfaf408c324a59

@Flet Good start. :+1: I say we hold off on merging this until we've eliminated jscs (#45), so we don't waste effort converting that too.

@feross
You're the boss, but as far as I looked in to that I think that actually these two can work side by side quiet harmonously.
@Flet
Are you going to finish that? Or should I begin to work on it too?

@reg4in yeah probably... but maybe it makes sense to create an eslint-only branch in this repo and continue work collaboratively?
Once the eslint rules from #45 are in, I'd guess we would want remove jscs and gain the speed benefits of using eslint directly. The branch could focus on that end state.
Thoughts, @feross? Or should we redirect our efforts to creating the missing eslint rules instead? :)

I'd totally be in :+1:

@Flet Either approach is fine. I just added you as a collaborator on the repo.

Thanks for the first attempt code, @Flet – it was very helpful.
I just pushed a branch (speed), which improves performance of standard by 2-4x! This branch does both items discussed in my first comment:

remove the need for npm run which-eslint.
remove the child_process.exec calls and use the node api of eslint directly.

I also remove jscs, which we're close to doing anyway (https://github.com/feross/standard/issues/45).
Linting feross/standard:
$ time standard
standard  2.21s user 0.36s system 161% cpu 1.596 total
$ time ./bin/cmd.js
./bin/cmd.js  0.61s user 0.06s system 105% cpu 0.630 total

Linting npm/npm:
$ time standard
standard  4.63s user 0.44s system 175% cpu 2.895 total
$ time ./bin/cmd.js
./bin/cmd.js  2.18s user 0.16s system 114% cpu 2.052 total

Sweet! I was just thinking of returning to this with the eslint rule movement :) Amazing what a few dollars does! :moneybag:
Taking a gander at the branch now.

 Also should be easier to use standard programmatically now that it takes a callback.

I take full blame for the sync stuff :sob:

Hah, no worries. I'm still doing a bit more cleanup to make the api nicer for programmatic use :)

Okay! New, cleaned up node.js apis are available on the speed branch. Really happy with how they turned out!
Deleted around 200 lines total, increased speed 2-4x, and it's way clearer how everything works now!

nice one!

instead of using npm run which-eslint you could use the npm-run lib. the slow part is not the which command, but spawning the npm executable itself.

Ah, handy! Wish I knew about npm-run before.
In any case, we've removed npm run and child_process.exec in favor of require('eslint') which requires less code and fewer hacks. :) This is currently on the speed branch and will become the next version of standard once https://github.com/eslint/eslint/issues/1619 is fixed.

This is released as 3.3.0.

nice! good job!


Bugfix/3183/portal media align header and close button

Targets #3183

Description

I have done the changes to the close button which is now on the same level as Add New Post heading.
After that, I remove the top blank card and the outer card.
Before changes:

Here is the screenshot after the changes:

Estimated Time

Align the close button  to the same level as Add New Post heading.        30 min
Remove the top blank card  and the outer card.        1-2 Hour
For testing and code review .     2-3 hours

Actual Time

Align the close button  to the same level as Add New Post heading.      10 min
Remove the top blank card  and the outer card.      30min
For testing and code review.      1hour

Checklist:

I have performed a self-review of my own code.

This is not needed now


Not showing all completions; popup docs window off in the wilderness

Make sure you have done the following

[X] I have updated to the latest version of blink.cmp
[X] I have read the README

Bug Description
Running 0.7.2 released just moments ago.
Edit a Lua file. Start typing and only one suggestion is shown. Type first letter of that suggestion and the rest show.
Using  to popup docs. First doc shows in right spot. Second doc in spaced way out there. When cursor through the docs again they show up properly.
Video should be self explanatory:
https://github.com/user-attachments/assets/83dd6015-60ad-4797-9e63-ca6691257c73
Relevant configuration
No response
neovim version
NVIM v0.10.2 Build type: Release LuaJIT 2.1.1731601260
blink.cmp version: branch, tag, or commit
0.7.2

I have the same issue. With 0.7.2, the menu window size is sometimes all over the place. This occurs when typing local in a lua file, with the default blink.cmp settings:


More log at console output

Please add more detailed output on console
For example I want to know when it says fetching playlist what is actully happening

Well, it sends a POST http request to the spotfy servers, and the response time depends on how long the servers take to respond. This has to looped multiple times as the servers return 10 songs at a time by default. So, adding more detail isn't really going to tell you anymore about how the 'fetching' is progressing. Thats the reason that it isn't implemented.
If you're interested, there is a major speed up coming to 'fetching'. We'll see if its sensible to add more detail after that.
That bing said, if you really want that functionality, fork this project, implement it. We'll put up a link to your fork on our homepage for others who'd be interested.


dummy change to test github actions

Description of the change

Related issue number

How to verify the PR

Was the PR tested

[ ] I have added >=1 unit test(s) for every new method I have added.
[ ] I have ensured all unit tests pass

wanted to test to ensure that caching isn't occuring across github action runs. verified that test failed as expected due to picking up trl==0.12.2 which previous tests picked up tel==0.11.4


AZ-379: Copy DataNetwork from old to new network

I'm closing this PR, since @timorl get an idea how to do it better :disappointed:


Specify proxy function for http transport

The client does not use DefaultTransport or DefaultClient and hence, HTTP proxies cannot be set.
This small modification allows to set proxy from the environment through http_proxy and no_proxy.

Thanks! 🍦


Region fields and custom Graphics

Some of the graphics are coming up with errors that they will be deprecated. As well some fields are empty such as the Region field, how do i populate the region field?
[Enter feedback here]

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

ID: 9e657b86-072e-1dff-0e96-1ec0efa8e4d7
Version Independent ID: 81d2ec45-8c85-220c-760a-443971fd27f9
Content: View Microsoft Teams utilization in Power BI using CQD data - Microsoft Teams
Content Source: Teams/CQD-teams-utilization-report.md
Service: msteams
GitHub Login: @LolaJacobsen
Microsoft Alias: lolaj

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

Hi @TheResearcher-Hub, thank you for your feedback.
Could you kindly provide us with more detail?

@TheResearcher-Hub We appreciate any feedback that improves the content in the Microsoft docs but we haven't had any response from you so we are going to close this issue.
Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thank you.


Cannot save configurable settings for this mod (custom tooltips, etc.)

Title

Did you press Save?

How are you editing them?

I press the Controllable icon and go into settings. I am on 16.2 because Optifine can't be used on versions that support 16.3.

https://cdn.discordapp.com/attachments/368376229679726593/1001975692830982234/2022-07-27_16-11-39.mp4


Avoid a race in retry where we might do a split when we don't need to

This fixes #1066 but depends on https://github.com/NVIDIA/spark-rapids/pull/8306 to be merged first so it does not break retry in the plugin when a split is needed.
The old code had a race where if all of the threads were put into BUFN, but didn't finish releasing memory, then one of the threads would switch over and throw a SplitAndRetryOOM exception. Ideally what we want is to have all of the threads finish rolling back and then we retry the highest priority thread and only then do we throw a SplitAndRetryOOM.
I didn't do that exactly here. Instead what I did is to only throw a SplitAndRetryOOM if all of the tasks are TASK_BUFN or are blocked on shuffle. This has the effect of delaying the SplitAndRetryOOM until all tasks are done rolling back. This shifts the API that would throw this exception to blockThreadUntilReady after a RetryOOM is already thrown.
Then to let the highest priority BUFN thread wake up early I have it restart iff all of the threads are blocked and memory is freed by some thread that is not this thread. In the best case all of the memory is freed before the BUFN thread needs it. But in the worst case it would retry again as it waits for more memory to be freed.
I ran some tests to see if this actually fixed the issue and using the Monte Carlo simulator I think I was able to reproduce the problem.  Without this patch when I run the following I will see a number of splits, typically between 1 and 5.  But if I run with the patch the number of splits is 0. Also in many cases I saw the performance end up being slightly better (because it was able to avoid the splits that can be costly).
java -cp ./target/spark-rapids-jni-23.06.0-SNAPSHOT-tests.jar:./target/spark-rapids-jni-23.06.0-SNAPSHOT-cuda11.jar:./target/dependency/\* com.nvidia.spark.rapids.jni.RmmSparkMonteCarlo --seed=11511967777172538 --taskMaxMiB=950

build

Just merged in the patch that this depends on so now this is ready for review.


Displaying stops

In the same way that selecting a route on the map will scroll to that route in the list, users should be able to click on a stop on the map which will make visible a corresponding stop widget in the list.
Long list of stops?
Having a long list of stops is one option, and clicking on that stop could scroll to the corresponding widget. However, the number of stops can be very large and even if we hide most of the widget behind expanders as described in #8 there is still likely to be performance problems. It also makes more sense to initially show the stop_time from which the stop can be naviagted to, since the inverse is not really possible. There doesn't seem to be any other sensible way to group/subset stops other than by route, or maybe alphbetically by first letter of name?
There doesn't seem to be much need for having all stops in one list, and makes more sense to only display individual stops
The only exception is selecting a new stop for a stop time.
It seems like it would be nicer to be able to add a stop at the route or shape level, and then trips/stop_times select from the routes/shapes subset of stops when updating stops, but the data structure doesn't really support this which would likely make it hard and convoluted to implement.
Displaying corresponding routes for a particular stop
Is it possible to clearly display all the corresponding routes/trips/stop_times for a particular stop? Possibly not in the list, only on the map? But even on the map it will not be clear since different routes for the same stop will of course overlap and not be easily differentiable - but only for that section of the route? Maybe displaying a list of all route names that use that stop in the stop widget would be best?
Will need to make it clear that changes to a stop will affect other routes also referencing that stop. Maybe want editing functionality that will automatically create a new copy of the original stop, allowing a stop to be easily updated for a specific route.

No comment from me other than to say: thanks for thinking of this and overall sounds good to me!

This issue was mostly for jotting down thinking. I've now broken it into a number of smaller self contained issues.


Bad render for certain rowspan colspan table

I suspect this is due to having both a rowspan and a colspan for the cell.

Fixed in 0.9.9.


Support for Shell32's SHFileOperation() function

Do you have plans implementing SHFileOperation() from Shell32 https://learn.microsoft.com/en-us/windows/win32/api/shellapi/nf-shellapi-shfileoperationa.
It is useful for sending files to the recycle bin.
Below is the sample usage in Delphi
function SendToRecycleBin(const FileName: TFileName): Boolean;
const
  FOF_COMPLETELYSILENT = FOF_SILENT or FOF_NOCONFIRMATION or FOF_NOERRORUI or FOF_NOCONFIRMMKDIR;
type
  TSHDeleteOption  = (
    Silent,
    AllowUndo,
    FilesOnly
    );
  TSHDeleteOptions = set of TSHDeleteOption;

  function DeleteOptionsToCardinal(Options: TSHDeleteOptions): Cardinal;
  begin
    Result := 0;
    if TSHDeleteOption.Silent in Options then
      Result := Result or FOF_COMPLETELYSILENT;
    if TSHDeleteOption.AllowUndo in Options then
      Result := Result or FOF_ALLOWUNDO;
    if TSHDeleteOption.FilesOnly in Options then
      Result := Result or FOF_FILESONLY;
  end;

begin
  var FileOp := Default(TSHFileOpStruct);
  with FileOp do
  begin
    Wnd := 0;
    wFunc := FO_DELETE;
    var Source := FileName + #0#0;
    pFrom := PWideChar(Source);
    fFlags := DeleteOptionsToCardinal([TSHDeleteOption.AllowUndo]);
  end;
  Result := SHFileOperation(FileOp) = 0;
end;

This should now be supported in win32 v4.0.1. I've added an example in the examples folder (recycle_bin.dart) which demonstrates using it and a couple of other relevant Win32 API calls. Hope this helps!


Unexpected token f in JSON at position 1

Hi.
Seems like I'm getting a very similar issue as in https://github.com/anvilco/spectaql/issues/83, but it should've been resolved on 16 Sep 2021
I'm getting the following error:
➜  nestjs git:(staging) ✗ npx spectaql -d spectaql-config.yml
undefined:1
{first: 10}
 ^

SyntaxError: Unexpected token f in JSON at position 1

Schema: schema.txt
Config:
introspection:
  schemaFile: ./schema.gql

info:
  title: GraphQL API Reference
  description: Welcome to the party!
  termsOfService: https://www.example.com/terms
  contact:
    name: API Support
    url: http://www.example.com/support
    email<EMAIL_ADDRESS>  license:
    name: Apache 2.0
    url: https://www.apache.org/licenses/LICENSE-2.0.html

Anything I'm doing wrong? Or anything I'm missing?
Full stack trace:
➜  nestjs git:(staging) ✗ npx spectaql -d spectaql-config.yml
undefined:1
{first: 10}
 ^

SyntaxError: Unexpected token f in JSON at position 1
    at JSON.parse (<anonymous>:null:null)
    at Object.resolveDefaultValue (/path/to/spectaql/node_modules/graphql-2-json-schema/dist/lib/reducer.js:64:21)
    at introspectionFieldReducer (/path/to/spectaql/node_modules/graphql-2-json-schema/dist/lib/reducer.js:38:46)
    at arrayReduce (/path/to/spectaql/node_modules/lodash/lodash.js:697:21)
    at Function.reduce (/path/to/spectaql/node_modules/lodash/lodash.js:9749:14)
    at introspectionFieldReducer (/path/to/spectaql/node_modules/graphql-2-json-schema/dist/lib/reducer.js:26:46)
    at arrayReduce (/path/to/spectaql/node_modules/lodash/lodash.js:697:21)
    at Function.reduce (/path/to/spectaql/node_modules/lodash/lodash.js:9749:14)
    at /path/to/spectaql/node_modules/graphql-2-json-schema/dist/lib/reducer.js:77:34
    at arrayReduce (/path/to/spectaql/node_modules/lodash/lodash.js:697:21)
    at Function.reduce (/path/to/spectaql/node_modules/lodash/lodash.js:9749:14)
    at Object.fromIntrospectionQuery (/path/to/spectaql/node_modules/graphql-2-json-schema/dist/lib/fromIntrospectionQuery.js:30:30)
    at jsonSchemaFromIntrospectionResponse (/path/to/spectaql/app/spectaql/graphql-loaders.js:90:20)
    at module.exports (/path/to/spectaql/app/spectaql/index.js:110:22)
    at loadData (/path/to/spectaql/app/index.js:172:95)
    at module.exports (/path/to/spectaql/app/index.js:176:80)
    at Object.<anonymous> (/path/to/spectaql/bin/spectaql.js:74:1)
    at Module._compile (node:internal/modules/cjs/loader:1101:14)
    at Object.Module._extensions..js (node:internal/modules/cjs/loader:1153:10)
    at Module.load (node:internal/modules/cjs/loader:981:32)
    at Function.Module._load (node:internal/modules/cjs/loader:822:12)
    at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:81:12)
    at node:internal/main/run_main_module:17:47

Similar to #83 this looks to be an issue with graphql-2-json-schema. Could you open an issue over there, please, and I'll have a look ASAP?

I downgraded graphql-2-json-schema to 0.7.0 and 0.8.0 manually and issue still persists

In my case, It actually works with a previous version of the schema.gql file. So there must be something that Apollo is actually adding to the output gql file that is not compatible with graphql-2-json-schema. All my tools currently mark the .gql file as valid. I'll keep digging and will update back.

In my case, It actually works with a previous version of the schema.gql file. So there must be something that Apollo is actually adding to the output gql file that is not compatible with graphql-2-json-schema. All my tools currently mark the .gql file as valid. I'll keep digging and will update back.

Can you provide the 2 files - 1 that works and 1 that does not?
I'm pretty sure that graphql-to-json-schema simply is not expecting / supporting what it's encountering because it's a bug there. This type of thing has come up before.

In my case, It actually works with a previous version of the schema.gql file. So there must be something that Apollo is actually adding to the output gql file that is not compatible with graphql-2-json-schema. All my tools currently mark the .gql file as valid. I'll keep digging and will update back.

I'm unable to show the schema as it's proprietary. I will continue to dig into it and provide relevant details as they become available.

The problem I'm having arises from having default values for complex object types (non scalar). For example, the following syntax is a valid graphql syntax.
input SectionAttributesInput {
  render_on_page: Boolean = true
}

input NewSectionInput {
  archived: Boolean
  attributes: SectionAttributesInput = {render_on_page: true}
  ...
}

however, graphql-2-json-schema.fromIntrospectionQuery,  is unable to parse it with an error in the likes of SyntaxError: Unexpected token r in JSON at position 1

I opened an issue over at graphql-to-json-schema https://github.com/charlypoly/graphql-to-json-schema/issues/67)(https://github.com/charlypoly/graphql-to-json-schema/issues/67

Working on fix: https://github.com/charlypoly/graphql-to-json-schema/pull/69

@jcuna fixed and published in 0.14.0...thanks for your patience!

cc @NGuldager

@jcuna fixed and published in 0.14.0...thanks for your patience!

Thank you for tackling this. Appreciate your time.


Add 'switch' to *newlineBeforeBlockStatements

https://github.com/jscs-dev/node-jscs/issues/2045

So?..

Sorry for the delay! Thanks!

@markelog waiting for my nice work, bro https://github.com/jscs-dev/node-jscs/issues/2045#issuecomment-168816726 :)

@markelog still waiting for my bro...

Sorry, was preoccupied lately, check this out - https://twitter.com/jscs_dev/status/694073841568583682

@markelog great, 10x :)


Promote DEPRECATE

Perl6::World::DEPRECATED is in 'user' code on github.   This offers Raku users more facility than  is DEPRECATED by allowing the deprecation-version, elimination-version, and the alternative.
There is no apparent reason that deprecation should be constrained to OO code.  And in OO, the direct use of  DEPRECATED  in Callables  can look convenient and clear.
Please promote DEPRECATED from an implementation detail to a roasted item.

Sorry, why is this issue in this repo? Wouldn't either problem-solving or the Rakudo repo itself be a better venue for it?

In my previous,  DEPRECATE should be DEPRECATED in every instance. And profiling shows I found the wrong  method, the work is done by Rakudo::Deprecations::DEPRECATED.  Edits made.
Thanks, @JJ.
I am asking that DEPRECATED's use beyond is DEPRECATED be specified as part of Raku.  That usage being like this:
#!/usr/bin/env raku
sub d(--> Str) {
    DEPRECATED(  'Use-me', 'Since', 'Gone', :what('sub d') );
    'd';
}
d();

In that case, I thing problem-solving is the right venue; it's where new features are proposed and discussed. I can move the issue there if you want.

@JJ, Thank for the education.  I'll close here, clean up and move it.


generate_data from data.py causes warning with default flags because of parameter deprecation that never happened

I am new to this project but haven't found a related issue.
data.py provides the function generate_data:
https://github.com/yzhao062/pyod/blob/bb3a14ea4df149e3773fa34116dfc62e1c8d5c89/pyod/utils/data.py#L112-L114
The parameter behavior is set to old by default which causes a warning to be displayed, notifying about this behavior getting deprecated in version 0.9.0.
The 'fix' would be to call the function with behavior='new' as it would not warn you about anything, however the documentation states, that the parameter will be removed altogether in 0.9.0. I guess that never happened since we are now on 1.0.1?
https://github.com/yzhao062/pyod/blob/bb3a14ea4df149e3773fa34116dfc62e1c8d5c89/pyod/utils/data.py#L142-L155
Can we just move on and either remove that parameter now or un-deprecate it, if it is that important? We would need to go through all examples and tests and change the order of the expected return values, right? I would happily do that so that I can rely on this python package.
I also wondered why the tests would not fail when there are warnings?

That is right. I will remove the warning in the next release.


Expose uid and gid - as user and group names to Agent

Following up, We've found that in Agent/NodeJS, there's no stock interface to NSS (i.e. getpwent()) so in order to expose user and group names in a UI for scoped (and unscoped) processes, we end up having to exec id -n -u UID. If the overhead of adding the string names to the process-start messages is acceptable, this could help.

Let's add the resolved string of uid/gid in both statsd metric and start event

Issue address in #663.
QA Instructions:
Scenario 1:
Goal: Verify that we can observe group name dnd user name in proc.start metrics
docker run --rm -it cribl/scope:<VERSION_ID>
scope run -- top
scope metrics

proc.start       1          Count    process      ..     args: top,gid: 0,`groupname`: root,host: ....,proc: top,uid: 0,`username`: root

Scenario 2:
Goal: Verify that we can observe group name and user name in payload start message
(Terminal 1) Run TCP server
docker run --rm -it cribl/scope:<VERSION_ID>
apt update
apt install netcat
nc -l 9111

(Terminal 2) Connect to scope container and run example scope command
docker exec -it <name_of_cribl/scope_container> /bin/bash
scope run -p -c tcp://<IP_ADDRESS>:9111 -- ls

Check in TCP Server in payload start message that group name and user name are present
{"format":"ndjson","info":{"process":{"libscopever":"....","pid":353,"ppid":343,"gid":0,"groupname":"root","uid":0,"username":"root","hostname":"a5133d09aed5","procname":"ls","cmd":"ls","id":...
```

Verified on version 1.0.0-tc2.
Part 1:

Part 2:


Update setup->kubernetes->multicluster to reflect latest helm chart changes

Several changes have merged to the istio-remote helm charts and the multicluster procedure needs updates to be compatible.  ie. remote cluster auto-sidecar injection support

I'm working on this

/assign @tiswanso


pkg/cover: ensure that all PCs returned by kcov have matching callbacks

In the case some modules' addresses are off, certain kernel addresses returned by kcov may not have corresponding coverage callbacks in the .ko files. Keep an additional map in the backend to verify those addresses and report an error if that is the case.
Also adjust text expectations in pkg/cover/report_test.go, as inexact coverage will result in an error now.
Because GCC < 14 may tail-call coverage callbacks, the described check is not performed for targets that explicitly list "gcc" as the compiler. See https://github.com/google/syzkaller/issues/4447 for more details.

Before sending a pull request, please review Contribution Guidelines:
https://github.com/google/syzkaller/blob/master/docs/contributing.md

I'm factoring out the anonymous function from dwarf.go to reduce its cyclomatic complexity.

@dvyukov @a-nogikh please take another look


Simplify the various different "offset" types.

As the persistence system grows more complex and more subsystems make references to events within the store it becomes increasingly nonsensical to have separate Offset types in each package.
This idea was originally intended to distinguish between events that were definitely persisted in the store (eventstore.Offset) and events that may belong to some other application, having been consumed via a stream (eventstream.Offset).
I now have a requirement to reference offsets of persisted events in the aggregatestore package. I don't particularly like the idea of referring to eventstore.Offset from aggregatestore, nor do I like the idea of defining yet another aggregatestore.Offset type. The conversions between all the types defeat the safety they were originally intended to provide.
I think we need to return to only having two types. For persisted offsets, I think it makes sense to move eventstore.Offset to persistence.EventOffset, and share it across all persistence subsystems.
I'm not so sure what to do with eventstream.Offset, however. This is the type that should be used from the offsetstore ideally. However, my gut feeling is that there's something incorrect if we're making the persistence layer depend on this eventstream. At time of writing in master no persistence subsystems depend on anything outside the persistence layer except the envelopespec package.
Also, because eventstream has a stream implementation that's based on the eventstore, it would create a package dependency chain that looks like persistence/offsetstore -> eventstream -> persistence/eventstore, sort of "leaving" the persistence layer and "returning" to it, which also makes me uncomfortable.
/cc @danilvpetrov this is relevant to your offsetset store PR #148.

I've going to go ahead and move eventstore.Offset to persistence.EventOffset now. We can discuss the other one whenever you have a chance.

Oh my, I'm a fool.. I can't move that type to persistence, that's an immediate cyclic dependency >_<.

This is a graph of package dependencies as they are in master (excluding cmd)

As part of addressing this cyclic dependency issue, I'm going to address #75 now, and separate the parts of eventstream that use the persistence layer into subpackages.

Ok, after a few refactorings this is the new graph.

There are more packages over all, due to #176, but I think it's getting clearer that there is a separate between the "application coordination" stuff on the left vs the "persistence" stuff towards the right.
The persistence stuff itself has a bit of a cartesian-product thing going on. It might be worth collapsing the "subsystem" packages back into persistence directly.

Another option of course is to just return to using uint64 directly. I don't think this is any less safe than what we're doing right now, having to convert between types for each package.


Lost subscription items in firebase

Bug report

Extension name: firestore-stripe-subscriptions

Describe the bug
If a subscription with two items is created, the items structure is lost in firebase. Only the list of prices is saved. It means in particular that the 2nd item quantity is lost.
See https://github.com/stripe/stripe-firebase-extensions/blob/a33c2ad0f79d14c2102d6adbbeccba04b76ddb44/firestore-stripe-subscriptions/functions/src/index.ts#L360 with the hard-coded index 0.
it means also that item metadata are lost as well.
To Reproduce
Create a subscription with two items programmatically or directly in the stripe dashboard.
Expected behavior
I would expect to keep the same data model as stripe to ease understanding and avoid loosing some information (only replace data types: for instance timestamp and document ref in order to leverage firebase capacities).

Quantity used to be a top-level param on the subscription, only has been removed in the most recent API version. However, I decided to keep it around for backwards compatibility.
I'll add the subscription items with the next release.

great thanks !


Fix typos on README

fixing minor typos

Coverage remained the same at 70.253% when pulling f1711b80333aa72c979d1e539a7e66452f47a969 on everson:patch-1 into 0916fe9d7b154383b7f0cc224ee93750fb4581ae on glenjamin:master.


Fix for latest Rust

This fixes everything except a series of ICEs when compiling
all example. Small victories.

Examples are ICEing, nothing that can be done right now so merging.


Some BaseChainDB methods may leave the database in an inconsistent state.

For instance, persist_header_to_db() stores the header in the DB, calculates the total difficulty for that block and then stores that in the DB as well, but it can crash when calculating the difficulty and in that case we'd end up with the header stored but without its total difficulty.
There are a few ways to address that:

Use a database that supports ACID transactions
Batch DB updates and apply them atomically
Deal with it manually, with a bare except that rolls back updates

Number 1) is problematic because LevelDB does not support transactions, 3) is error prone and tedious, so I think 2) is our best option, but it means we need to add support for that to MemoryDB as well.

What about a wrapper that works for any KV store that I'll refer to as AtomicDB until a better name surfaces.  It would present the following API
with AtomicDB(db) as atomic_db:
    atomic_db[b'a'] = b'arst'  # doesn't write the key to the database but rather to a local cache.
    assert atomic_db[b'a'] == b'arst'  # intelligently returns the value from the cache
    atomic_db[b'b'] = b'tsra'
# upon exiting the context, it writes all of the keys from the cache into the underlying database.

In the case of an exception within the context block, no change would occur to the underlying database.  We could also implement an alternate API something like the following.
atomic_db = AtomicDB(db)
transaction = atomic_db.transaction()
transaction[b'a'] = b'arst'
transaction[b'b'] = b'tsra'

transaction.commit()  # writes all the changes.

Seems simple enough and I think it fits loosely into option 2.

That is a nice API indeed. We'd still need the underlying KV stores to support batch updates, though, or else in our batch.commit() implementation we'd need to implement a rollback mechanism ourselves, in case anything fails while we're performing any of the updates

I specified this in a separate ticket: https://github.com/pipermerriam/py-evm/issues/160


Unable to decode JSON response from Blockchain

I have installed Blockchain local machine and got the appropriate api key too. While using the code below, I am getting errors:
Code:
// Include the autoload.php from its vendor directory
require 'vendor/autoload.php';

// Create the base Blockchain class instance
$Blockchain = new \Blockchain\Blockchain('my_api_key');

// Needed before calling $Blockchain->Wallet or $Blockchain->Create
$Blockchain->setServiceUrl('http://localhost:3030/');

$wallet = $Blockchain->Create->create('1234', $email=null, $label=null);
print_r($wallet);

Errors:

Fatal error: Uncaught Blockchain\Exception\Error: Unable to decode JSON response from Blockchain: in /var/www/html/vendor/blockchain/blockchain/src/Blockchain.php:157 Stack trace: #0 /var/www/html/vendor/blockchain/blockchain/src/Blockchain.php(113): Blockchain\Blockchain->_call() #1 /var/www/html/vendor/blockchain/blockchain/src/Create/Create.php(39): Blockchain\Blockchain->post('api/v2/create_w...', Array) #2 /var/www/html/vendor/blockchain/blockchain/src/Create/Create.php(14): Blockchain\Create\Create->doCreate('1234', NULL, NULL, NULL) #3 /var/www/html/index.php(11): Blockchain\Create\Create->create('1234', NULL, NULL) #4 {main} thrown in /var/www/html/vendor/blockchain/blockchain/src/Blockchain.php on line 157

May you help me to solve this issue please?

Yes i have the same problem, i running with server nginx and i check log and response with error 504 Time out
2019/10/24 16:14:56 [error] 28407#28407: *46 upstream timed out (110: Connection timed out) while reading response header from upstream, client: <IP_ADDRESS>, server: dutibits.com, request: "GET /favicon.ico HTTP/1.1", upstream: "fastcgi://unix:/var/run/php/php7.3-fpm.sock", host: "dutibits.com", referrer: "http://dutibits.com/"

am Also experiencing this now
``atal error:  Uncaught Blockchain\Exception\Error: Unable to decode JSON response from Blockchain:

404 Not Found
`atal error:  Uncaught Blockchain\Exception\Error: Unable to decode JSON response from Blockchain:

404 Not Found
`
when trying to create wallet


⚠ Fortnite has degraded performance

In 807191c, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1346 ms

Resolved: Fortnite performance has improved in 9432a0c.


Improving our contributions document

See #90 - this PR is all about making our contribution document easier to understand!

Thanks for all of your comments. Combining this feedback with much more of my lived experience working on the 0.2 docs, I've completely redone this document.
Let me know if any issues persist here.


(maint) Update $ to jQuery

Updates any usage of $ to using jQuery. This was done to avoid
conflicts with additional libraries that may be used. Ultimately, any
mentions of "jQuery" will be replaced with native javascript instead,
so this may be a temporary fix.
See:
https://api.jquery.com/jquery.noconflict/

@gep13 thanks for catching these! I have force pushed an update to fix the issues you have brought up, and have tested that they are working.

@designxsteph thanks for getting this updated!


Update drush for Backdrop to 1.0.0, fixes #1693

The Problem/Issue/Bug:
Fixes https://github.com/drud/ddev/issues/1693
How this PR Solves The Problem:
Updates Backdrop drush version in Dockerfile to 1.0.0
Related Issue Link(s):
https://github.com/backdrop-contrib/drush/releases/tag/1.0.0

@rfay how do I go about updating the pkg/version/version.go file like you did in https://github.com/drud/ddev/pull/1687? ...is that var WebTag arbitrary? ...in other words, can I simply change it to something like 20190706_klonos_backdrop?

Since the change is to the container, the tests need to run against a built container. I can push to drud/ddev-webserver, but you can't. However, you could push to your own docker account (and put that in version.go), or I can push to the official place.  The push is like this: make push VERSION=<branchname> - you're using pretty useless branch names (patch-2) but just use a good one with a date and purpose, or whatever. Or I'll push one and update version.go when you are ready. Congrats on the good drush/backdrop work.

I updated and pushed it.  We'll let this sit for a while in chase there's any more churn/improvement going on with the drush backdrop stuff. Wonderful progress. I'll make sure to ask before pulling it, hopefully we can get it into a pre-release in a week or so.

...you're using pretty useless branch names (patch-2)

That's because I have made the PR using the GitHub web UI (lame 😅). It picks branch names for you.

Congrats on the good drush/backdrop work. ...Wonderful progress.

Thanks ...the whole gang have been buzy bees :)

Yeah, sorry I was kind of harsh on that, I do hate how github does that. Should get used to it. But it makes pushing a matching image for a branch really hard.

@klonos Do you think that the upstream project is "done" at this point, good time to pull this, or should we wait a while yet?


update footer component fallback legal text to reflect registered office move

This has been implemented to check the current date and return the new registered office from moving day onwards.

rebased to fix Changelog clash - think it'd be good to get this into the next release so we can roll it out unless you've got any objections @marianayovcheva ?

🚀

Just had a thought that we probably need to set reminders and let people in the design-system channel know that on 15.03 if their product has VRTs those will likely fail (the VRTs don't always block the build) and will need updating.


Add ^^ as a logical boolean XOR expression

This issue was originally filed by<EMAIL_ADDRESS>
The binary expressions include &, | and ^ but the logical boolean expressions only have && and ||. Adding ^^ for XOR will be consistent (and it will make my code shorter).

&& and || are short-circuit because 'and' and 'or' 'stick' at false and true respectively.
^^ has no such property - both operands must always be evaluated.
This makes ^^ just a synonym for !=
Removed Type-Defect label.
Added Type-Enhancement, Area-Language, Triaged labels.

oh! Thank you!!! that's what I need!!!! Trying to make sure both of my operands are not the same

For the record, since Dart 2.1, the bool class has had non-short-circuit operators &, | and ^.


Timestamp formats expect 6 digits for microseconds when 3 digits seems to be the standard

Got this error on a cp:
Error while reading data, error message:

Could not parse '2021-06-30 08:33:37.000 -0700' as TIMESTAMP for field created_at__utc_ (position 2) starting at location 661  with message

'Could not parse '2021-06-30 08:33:37.000 -0700' as a timestamp. Required format is YYYY-MM-DD HH:MM[:SS[.SSSSSS]] or YYYY/MM/DD HH:MM[:SS[.SSSSSS]]'

Based on moment.js docs it does seem YYYY-MM-DDTHH:mm:ss.SSS is a standard format for timestamps. Can it be supported?

@blakew What are your source and destination databases (just the types like "postgres:" or "bigquery:", not the actual URLs)? This looks like a driver-specific problem.

The most likely problem is you field is named created_at__utc_, but it includes a -0700 timezone offset in '2021-06-30 08:33:37.000 -0700', which is definitely not UTC! dbcrossbar supports two different column types, timestamp_without_time_zone and timestamp_with_time_zone. My guess is that you're getting this error not because of how many digits of milliseconds you're passing (.SSS versus .SSSSSS), but because your data is in timestamp_with_time_zone format and you're trying to store it in a timestamp_without_time_zone column. The underlying destination database (whatever that is) is rejecting it because of this mismatch.

source = snowflake
destination = BigQuery

Yes, this is an issue with a mismatch between the destination schema and the CSV data. BigQuery is not required to accept timezones for a column of type timestamp_without_time_zone (or TIMESTAMP in BigQuery's terminology).


No 'Access-Control-Allow-Origin' does not let site render properly

Chrome Version : 55.0.2883.87 m
Os : Windows 7

Clipart icons are not rendered correclty
It looks like some placeholder text (or hover effect texts) are shown.

Console shows following error.
Access to Font at 'http://fonts.gstatic.com/s/materialicons/v19/2fcrYFNaTjcS6g4U3t-Y5ZjZjT5FdEJ140U2DJYC3mY.woff2' from origin 'http://romannurik.github.io' has been blocked by CORS policy: No 'Access-Control-Allow-Origin' header is present on the requested resource. Origin 'http://romannurik.github.io' is therefore not allowed access.

Is this still happening? I can't reproduce this, and curl'ing that URL doesn't hit me with any CORS restrictions. My guess is this was a transient server issue. Please comment if this is still occurring.

still happening to me...

Do you have a proxy server set up? (Asking per this FAQ):
https://developers.google.com/fonts/docs/troubleshooting#missing_cross-origin_resource_sharing_cors_response_header

At workplace I'm behind a firewall, but on the previous version I didn't have this issue. On mobile is ok BTW.


Sort inputs and remove whitespaces after exclamation

Fixes #345.
Opened as a replacement PR for https://github.com/actions/cache/pull/473 to check failing tests and fix those.

make a commit

👍


Neomake displays no marker signs

When running neomake on a python file, no signs are placed next to lines with errors.
Expected behavior
Expected markers to appear next to lines with problems.  I was rather hoping I could get neomake to work as syntastic does when flake8 is found in PATH.
Steps to reproduce
[2017-02-12T01:40:34-0700 @2.685, debug] Running makers: flake8
[2017-02-12T01:40:34-0700 @2.687, verb ] [1] Starting async job: ['flake8', '--format=default', '/Users/david/projects/passport-django/apps/ppsessions/views/api.py']
[2017-02-12T01:40:34-0700 @2.688, debug] [1.0] Vim job: {'status': 'run', 'stoponexit': 'term', 'exitval': 0, 'exit_cb': 0, 'channel': channel 0 open, 'process': 59292}
[2017-02-12T01:40:34-0700 @2.689, debug] [1.0] Vim channel: {'status': 'open', 'id': 0, 'in_io': 'pipe', 'err_mode': 'RAW', 'in_status': 'open', 'out_io': 'pipe', 'err_io': 'pipe', 'err_status': 'open', 'out_timeout': 2000, 'out_status': 'open', 'out_mode': 'RAW', 'in_mode': 'RAW', 'in_timeout': 2000, 'err_timeout': 2000}
[2017-02-12T01:40:34-0700 @3.135, debug] [1.0] stdout: flake8: ['/Users/david/projects/passport-django/apps/ppsessions/views/api.py:7:80: E501 line too long (83 > 79 characters)', '/Users/david/projects/passport-django/apps/ppsessions/views/api.py:23:80: E501 line too long (102 > 79 characters)', '/Users/david/projects/passport-django/apps/ppsessions/views/api.py:26:80: E501 line too long (81 > 79 characters)', '/Users/david/projects/passport-django/apps/ppsessions/views/api.py:28:80: E501 line too long (98 > 79 characters)', '/Users/david/projects/passport-django/apps/ppsessions/views/api.py:34:1: E305 expected 2 blank lines after class or function definition, found 1', '/Users/david/projects/passport-django/apps/ppsessions/views/api.py:42:1: E305 expected 2 blank lines after class or function definition, found 1', '/Users/david/projects/passport-django/apps/ppsessions/views/api.py:54:80: E501 line too long (82 > 79 characters)', '/Users/david/projects/passport-django/apps/ppsessions/views/api.py:56:1: E305 expected 2 blank lines after class or function definition, found 1', '/Users/david/projects/passport-django/apps/ppsessions/views/api.py:59:80: E501 line too long (92 > 79 characters)', '/Users/david/projects/passport-django/apps/ppsessions/views/api.py:64:1: E305 expected 2 blank lines after class or function definition, found 1', '/Users/david/projects/passport-django/apps/ppsessions/views/api.py:64:80: E501 line too long (80 > 79 characters)', '']
[2017-02-12T01:40:34-0700 @3.156, debug] [1.0] exit: flake8: 1
[2017-02-12T01:40:34-0700 @3.156, debug] [1.0] flake8: processing 11 lines of output.
[2017-02-12T01:40:34-0700 @3.161, debug] [1.0] flake8: completed with exit code 1.
[2017-02-12T01:40:34-0700 @3.161, debug] [1.0] Cleaning jobinfo
[2017-02-12T01:40:34-0700 @3.162, debug] File-level errors cleaned in buffer 1
[2017-02-12T01:40:34-0700 @3.162, debug] Skipping User autocmd NeomakeJobFinished: no hooks.
[2017-02-12T01:40:34-0700 @3.162, debug] Skipping User autocmd NeomakeFinished: no hooks.

Output of the ":verb NeomakeInfo" command
#### Neomake debug information
Async support: 1
Current filetype: python
##### Enabled makers
For the current filetype (with :Neomake):
 - flake8
   postprocess: function('neomake#makers#ft#python#Flake8EntryProcess')
   errorformat: '%E%f:%l: could not compile,%-Z%p^,%A%f:%l:%c: %t%n %m,%A%f:%l: %t%n %m,%-G%.%#'
   args: ['--format=default']
   get_argv: function('264', {'get_argv': function('264')})
   exe: 'flake8'
NOTE: you can define g:neomake_python_enabled_makers to configure it (or b:neomake_python_enabled_makers).
For the project (with :Neomake!): None.
NOTE: you can define g:neomake_enabled_makers to configure it.
Default maker settings:
 - remove_invalid_entries: 0
 - buffer_output: 1
##### Settings
'''
g:neomake_place_signs = 1
g:neomake_logfile = '/Users/david/neomake.log'
g:neomake_python_enabled_makers = ['flake8']
shell: /bin/bash
shellcmdflag: -c
Windows: 0
'''
Press ENTER or type command to continue

Nevermind.  It was another plugin I have that was interfering with neomake.


Update pulsar-client, ... to 3.2.3

About this PR
📦 Updates

org.apache.pulsar:pulsar-client
org.apache.pulsar:pulsar-functions-api

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

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.apache.pulsar" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.apache.pulsar" }
}]

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

Superseded by #199.


fix subscriptions messages in protocol.c

subscription mechanism was not working for me and this change fixes it.
So I share it ;-)
Basically, in protocol.c line 507, the subscription code is erased.
So, I made the registration of the subscription at PRE_WRITE time instead of POST_WRITE time.

Can you try with the updated protocol if the subscriptions problem is still there?
If so, we can adapt this PR to match the latest branch.

I wanted to test it but the latest version of the firmware does not compile with this new version of the protocol. This is because the firmware (cf. src/main.c:911) accesses a var named enable_immediate that has been moved into a struct by this new PR. I did not investigate more to do the required changes.

You are right, wanted to merge all repos at once but I underestimated the effort needed :)
Master branch is merged now, please note that the robustness of the serial protocol is way better now, but it is not compatible anymore.
I'll update the Arduino Library soon.

I messed up with the pull request branch on my own fork and it closes the pull request...
We always underestimate the effort, otherwise we would not start ;-)
I tried this new version of the protocol and subscriptions does not work for me.
I also compiled the firmware with this new protocol and tried the ASCII protocol and it is weird. For example the Immediate Speed "Is" mode continuously increase speed even if I release "w".