AndroidNotification::VibrateTimingsMillis not operating as documented

[REQUIRED] Step 2: Describe your environment

Operating System version: Windows 10 Enterprise
Firebase SDK version: 2.3.0
Firebase Product: Firebase Cloud Messaging
.NET version: 6.0
OS: Windows 0 Enterprise

[REQUIRED] Step 3: Describe the problem
The AndroidNotification::VibrateTimingMillis property is documented as follows:

Gets or sets a list of vibration timings in milliseconds in the array to use. The first value in the
array indicates the duration to wait before turning the vibrator on. The next value
indicates the duration to keep the vibrator on. Subsequent values alternate between
duration to turn the vibrator off and to turn the vibrator on.

Today I wanted to send a notification that would vibrate for 500 milliseconds, so I set this property as follows:
AndroidNotification notification = new
{
    DefaultVibrateTimings = false,
    VibrateTimingsMillis = new long[] { 0, 500 }
};

On the device though, this resulted in a very long vibration, that lasted for several seconds.
After experimenting, sending a value of { 0, 1 } results in a vibration lasting 1 second - I've reviewed the relevant code for this (in VibrateTimingsString) and tested it on its own, and the above code results in the value 0s, 0.500000000s being sent to the API, which appears correct.
I'm not sure if this is a fault on the device I was testing against as it is running a very dated version of Android unfortunately (we're stuck with it due to customer requirements and it's stuck on Android 7.1.1).
PS: I also noticed a few possible micro-optimisations that could be made to some of this new code, such as pre-sizing the list for the timingsStringList, I'm not sure if it's worth opening a PR for these changes.

Hi @euantorano thank you for reporting this issue. You are right, it appears that the SDK is mapping the millisecond durations to seconds strings correctly. Are you by any chance able to test on a different device?
Also, (a speculation from my side), sometimes if DefaultSound is set to true and if the device is on vibrate it vibrates instead of playing a sound in addition to vibrating based on the VibrateTimingsMillis config you set (I highly doubt this is the case though).
@chong-shao Do you know if we have resources to test the BE API on this?
@euantorano : regarding the optimisations, this feature is also a community contribution #203 so please feel free suggest any changes by opening PRs. :) We appreciate it!

I’ve got a couple of devices to test with on varying API levels, I’ll give them a go tomorrow morning or early next week.
We have DefaultSound disabled by the way, as we don’t want any sound with the notification (we only want it to vibrate).

On 30 Mar 2022, at 20:24, Lahiru Maramba @.***> wrote:

Hi @euantorano thank you for reporting this issue. You are right, it appears that the SDK is mapping the millisecond durations to seconds strings correctly. Are you by any chance able to test on a different device?
Also, (a speculation from my side), sometimes if DefaultSound is set to true and if the device is on vibrate it vibrates instead of playing a sound in addition to vibrating based on the VibrateTimingsMillis config you set (I highly doubt this is the case though).
@chong-shao Do you know if we have resources to test the BE API on this?
@euantorano : regarding the optimisations, this feature is also a community contribution #203 so please feel free suggest any changes by opening PRs. :) We appreciate it!
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.


Chem guidebook

About the PR

Requires https://github.com/space-wizards/RobustToolbox/pull/4111
Revival of #14060
Creates some new guidebook tags for displaying reagents:

GuideReagentEmbed: embeds a control displaying a reagent's name, description, recipe, and effects.
GuideReagentGroup: embeds a list of multiple GuideReagentEmbeds for a given category.

These are combined to essentially duplicate the chemistry wiki page in the guidebook. (The guidebook page is also linked on the Chem Dispenser via GuideHelpComponent)
Media

blurb

general entries

recipe + effects

the long boy

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog

:cl: moony, EmoGarbage404

add: The chemistry wiki page is now available in the guidebook. Learn all about the different chemicals today.

Ready for review

rest in piss wiki

chem has officially been upgraded from a wiki job to an F1 job

num0*****


Replace log.Fatal by return errors in deploy.go

What this PR does / why we need it: Return errors instead of doing log.Fatal inside functions
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
If applicable:

[ ] documentation
[ ] unit tests
[ ] tested backward compatibility (ie. deploy with previous version, upgrade with this branch)

Release note:

/approve
/lgtm

/approve
/lgtm


https://æsthetic.dev/ longer working

The README, and possibly other documents, contain links to https://æsthetic.dev/ which no longer resolves in DNS, with or without the www. prefix.
This is not a browser-specific issue as I've checked both Firefox and Chrome.
I've also checked its decoded version, xn--sthetic-lxa.dev, and experienced the same issue.

Hey @dse
Yeah, I am aware of this one. Given that the site is now using .js.org subdomain the documentation can be found via:
https://aesthetic.js.org
The æ was hard for folks to use, so instead aesthetic.js.org is the preferred.


3dsmax: Use custom modifiers to hold instance members

Changelog Description
Moving logic to handle members of publishing instance from children/parent relationship on Container to tracking via custom attribute on modifier. This eliminates limitations where you couldn't have one node multiple times under one Container and because it stores those relationships as weak references, they are easily transferable even when original nodes are renamed.
Additional info
Because of this change, all publishing plugins were affected. Now there is a collector plugin getting all nodes referenced by instance node and storing them on instance.data["members"] for easy access. This is also changing type case to align it more with 3dsmax python API and few style changes.
Testing notes:
Publishing of all types should work as before. In particular the tyFlow publishing needs testing because there was some refactoring involved.
[cbID:OP-5754]

For some reason Hound is heavily fighting with the walrus operator := so until I can figure that out, I've used # noqa there.

Removing instance errors out. Using the original removing instance code may resolve this as removing container node in modifier meaning unparenting the added objects from the container

I have some comment on the point cloud code, it will probably resolve the extractor errors

A question: should the creators also be refratored?

Speaking of first tests I did for Model all works ok (publish of the asset and its later Loading via Asset Loader) but I have a question if desirable when loaded to the scene there is one more level hierarchy compared to when loaded old published asset (without the new implementation of modifiers) in my case Alembic Cache. See Outliner hierarchy for Teapot asset

That is interesting and it wasn't intended. Additional levels shouldn't be there. 👀

Another issue I have faced is when trying to add items to existing Container/modifier already present in the scene.
Quite common scenario when user wants/needs to move items between Containers or just add new ones.
State before adding (check column list on the right side)

And when trying Add Selection it replaces all listed items with the new one

I've updated the publishing instance script so there are now two buttons - Add to Container and Delete from Container. To test it, the instance must be re-created.

When using Add to Container or  Delete from Container when no selection made (basically not selecting anything and hitting Cancel

Error shows up and user cant open add / delete dialog box afterwards.

I adjusted some of the codes in the loaders function for custom modifiers, including the bug fixing in the up-versioning and error with loading the max file format. There is a related issue #4675 done before, but we can make the adjustment altogether in the same ticket.
During the testing I also find out that I can publish when creating the first instance with "use_selection" turns on. But when I create another instance again, the publish progress bar doesn't move on and I can't really publish.
https://github.com/ynput/OpenPype/assets/64118225/32295588-5b6d-4feb-8c53-d2b71ca6fd7e

can confirm, seems like some problem with Publishe @iLLiCiTiT?

Will be fixed in https://github.com/ynput/OpenPype/pull/4915 . You can change imports of InstancePlugin and ContextPlugin to fix it.
instead of
from pyblish.api import InstancePlugin

class MyPlugin(InstancePlugin):

use
import pyblish.api

class MyPlugin(pyblish.api.InstancePlugin):

So I quess its good to go merged??

There is still issue with removing items from the instance, where they seem to be removed successfuly, but when you re-select the container, the removed items are still there.

There is still issue with removing items from the instance, where they seem to be removed successfuly, but when you re-select the container, the removed items are still there.

I noticed that too, it can be because the custom attributes doesn't get removed. I updated the removal of the custom attributes while removing the instance in the updated PR.

Removing instance is fixed and supposes that it can remove any Openpype attributes.
https://github.com/ynput/OpenPype/assets/64118225/1f20a920-fb78-4b28-abc2-8c172fc42805
Also I update extractor to select the object of node references in the instance reference before exporting, rather than select the object parented to the containers

Deleting still not work. I've fixed the error when you clicked on Cancel in both Delete and Add dialog, but the deleted item is still hanging there.

Steps to reproduce:

In 3dsmax, create two object (torus and teapot).
Go OpenPype > Publish, select the Teapot and Model instance and hit create.
Close the publisher.
Select your newly created instance (modelMain), go to the modifier list and select the Container.
In OP Parameters click on Add to Container and select your second object (Torus001 for example)
After confirming, that object should correctly appear in the list, so far so good.
Now click on Delete from Container, select your second object (Torus001) and confirm.
It should disappear from the list.
But if you click anywhere (select another object) and then re-select the instance, you'll see the list with two object.

@antirotor this is because the bitarray for all_handle and list_node.items hasn't updated when pressing the delete button. The updated PR has fixed this issue mentioned above.

Now it deletes everything :) but we are on the good track now!

@antirotor i have updated the PR for removing instance through the OP Parameter, it should be working now. Also I edited the string for the new collector to ease the confusion from collect_render.py

It works! I've removed two debug prints, and when approved by @LiborBatek , we can finally merge it.


Fail to compile binary for Android

Is there simple way to compile a native binary for inference on Android? Now I can successfully build the benchmark and run it on my Android device. But it does not contain any actions about ops. I want to do something similar to the benchmark_model_test but it fails to build. I think it now lacks a android version of //tensorflow/cc APIs. How can I tackle it?

Have you read this? https://github.com/tensorflow/tensorflow/tree/master/tensorflow/contrib/android

Thanks @flawless2011 I've read that. I want to use the apis under //tensorflow/cc/..., but it seems that the lib you provide doesn't refer to these apis? Code in https://github.com/tensorflow/tensorflow/blob/master/tensorflow/tools/benchmark/benchmark_model_test.cc uses some ops apis instead. For most TF apis, we already have //tensorflow/core:android_tensorflow_lib, but in my case I want to compile more but failed.

@flawless2011 Did I make my question clear? I hope you can have comments or help refer to someone who should be familiar with this. Really appreciate that.

Sorry I'm not getting this. What do you want to do? Compile directly against the C++ library on android?
/CC: @asimshankar

@drpngx Basically, I want to build a native binary on Android, which simply creates a graph with some ops and execute it with some random input. The benchmark is a good example which can run on Android, but it just reads the graph without constructing its own graph. From what I've found via the benchmark_model_test and label_image examples, I need to include apis under //tensorflow//cc to use ::tensorflow::ops::XXX (such as ResizeBilinear).
So my question is how to compile the //tensorflow/cc for Android? I didn't find any target inside //tensorflow/cc.BUILD, and failed to build it by myself.

@PKUEcho : So you want to compile essentially an arbitrary binary that uses the C++ API for graph construction for Android?
This seems more like a bazel question :), but the good news is that that the only difference between compiling for Android or for your host machine should be flags to bazel.
I suppose you could do one of the following:

Modify benchmark.cc with the code you want, and add //tensorflow/cc:cc_ops to the deps section in the BUILD file. And build the binary as per the instructions you pointed to. Those instructions show how to build any bazel target for Android.

Compile the shared library for Android using something like:

bazel build -c opt \
  --crosstool_top=//external:android/crosstool \
  --cpu=armeabi-v7a \
  --host_crosstool_top=@bazel_tools//tools/cpp:toolchain \
  tensorflow:libtensorflow_cc.so

This would be a shared library that you can link against in any native code you build for Android.
Hope that helps.

@asimshankar thanks for your reply! I'm missing some details here. So, basically you provide two solutions here, right?

Inside the deps as following, I think I need to insert //tensorflow/cc:cc_ops after //tensorflow:android rather than //conditions:default? Because I think that's the true deps when I build it for Android.
cc_library(   name = "benchmark_model_lib",   testonly = 1,   srcs = [       "benchmark_model.cc",   ],   hdrs = [       "benchmark_model.h",   ],   copts = tf_copts(),   visibility = ["//visibility:public"],   deps = select({       "//tensorflow:android": [           "//tensorflow/core:android_tensorflow_lib",           "//tensorflow/core:android_tensorflow_test_lib",       ],       "//conditions:default": [           "//tensorflow/core:core_cpu",           "//tensorflow/core:lib",           "//tensorflow/core:framework",           "//tensorflow/core:framework_internal",           "//tensorflow/core:framework_lite",           "//tensorflow/core:protos_all_cc",           "//tensorflow/core:tensorflow",           "//tensorflow/core:test",       ],   }), )

After that, I tried use the following command to build it but failed.  I think there's something about the compilers. How do you think about?

@PKUEcho : What errors were you getting when you tried to build benchmark_model?
(Based on updates in #2412 - I'd avoid that second suggestion altogether for now, though the errors you're seeing are different, but it seems using libtensorflow_cc.so isn't convenient yet)
That said, CCing @andrewharp in case he has ideas, but marking this as "community support" since building native binaries using the C++ API for Android isn't something that we are currently actively working on or supporting.

Right, unifying the android_* libs with the other cc_library TF targets isn't really something on our radar.
However, if you take a look at https://github.com/tensorflow/tensorflow/pull/9563, they manage to build cc_ops into an Android binary, so that might be a possible route to investigate.

@asimshankar @andrewharp Thanks for your help. After reading the #9563, I found that I missed the flags --cxxopt="-std=c++11" --cxxopt="-DTENSORFLOW_DISABLE_META" which makes my previous compilation fail. Though I'm not sure why in other cases such as benchmark_mode I don't need these flags?
Also, I found that some ops are not supported in mobile version currently. How can I check which ops are supported and which are not? This question might be out of the scope of this thread so you may close this.

There are kernels that definitely work, and kernels that may work.
The kernels that may work are here.
The ones that definitely work are core and extended.
Closing for now.

@PKUEcho some ops use C++11 features, but the default cross-toolchain in NDK (gcc-4.9) uses C++98. That's why you need --cxxopt="-std=c++11"

@freedomtan Thanks for your reply, that makes sense :)

Hi, I'm trying to build the standard Android C API (not the JNI API) so I can build a standalone binary in Rust on Android, what would be the recommended way? I saw the contrib/android builds the JNI lib, should I simply do a cross-compile bazel build of //tensorflow:libtensorflow.so ?

@maelp The target you're looking for is //tensorflow/core:android_tensorflow_lib. See //tensorflow/contrib/android/README.md for the set of build flags to use.

Hi @andrewharp thanks for your answer, my question is not so much about Rust integration as it is about getting the C or C++ library cross-compiled on Android
From your comment I assume that I need to add a build target with both android_tensorflow_lib and c_api to build those? There is not yet an existing cross-compile build target which compiles the C .so for Android without the rest of the JNI bindings?

Correct: the c_api target depends on android_tensorflow_lib_lite, which is runtime-only (no ops), so you'll need a target that adds them in. My suggestion would be to take //tensorflow/contrib/android:libtensorflow_inference.so as a starting point and modify it to fit your needs (remove the jni files, change the whitelisted methods so they don't get stripped out, etc).


Add Conda env library path to RStudio configuration

Currently RStudio packages installed in the environment do not show up in RStudio. The current workaround is to set the LD_LIBRARY_PATH environment variable in the start file. This change applies a similar fix during the build, adding the proper library path to rserver.conf.
Relevant discussion can be found in #1153 and here:
https://discourse.jupyter.org/t/glibcxx-3-4-26-not-found-from-rstudio/7778/5

Thanks @TimStewartJ and apologies for the late review!

No worries! Thank you!


Source Manifest uses template instead of OCS_IMAGE for backward compatibility

Before migrating to new project structure and tools, CSV Merger was using templates added by source-manifests to inject OCS_IMAGE value. When we moved to using Kustomize, we set the images directly instead of first templating and replacing.
This broke the build pipeline as it didn't safeguard itself against uninitialized OCS_IMAGE var.
This PR reverts back to using templates instead of Kustomize to unblock the build pipeline.
Alternatively, build pipeline can set OCS_IMAGE env var a little early and all should be well (without this PR)

Build pipeline is fixed. Closing this PR as we decided to use Kustomize instead of Go Templates for simplicity.

Build pipeline is fixed. Closing this PR as we decided to use Kustomize instead of Go Templates for simplicity.


Improved the select example.

Introduction
This PR is an effect of my initial confusion with the intended way of handling Callbacks, and aims to help prevent that from happening to others in future.
Story time!
I wanted to add custom keybindings to a SelectView, so I looked through the examples and found the select example which looked like exactly what I needed. After replicating the OnEventView wrapper part of the code, the keybindings worked as far as navigating the SelectView went, but I noticed all the callbacks set with SelectView.on_select() were ignored with the custom keybindings but worked with the default arrow ones.
I started reading into the documentation, and noticed the SelectView.select_up() method returned a Callback, and the documentation said it should be ran on the Cursive. I couldn't figure out how to get a reference to Cursive inside the closure in OnEventView.on_event_inner() to run it like the documentation example has shown (something along the lines of callback(&mut siv) ), so I tried using OnEventView.on_event(), then getting a referece to the SelectView by name, and using Cursive.cb_sink() to send the Callbacks but apparently they weren't Sync so that didn't work out.
Not until having found #433 and this comment in particular did I realize that I should look at the documentation of EventResult, which, in turn, made me realize that I can just pass the Callback inside the EventResult.
Summary
I thought it would be a good idea to create this small PR to hopefully help other Cursive beginners be less confused :)
It effectively doesn't change anything in this example as far as functionality goes, but it helps bring attention to this important way of handling Callbacks.

Hi, and thanks for the PR!
Indeed, it's probably best practice to always forward the event result here, even if the example itself does not make use of it.


Paragraph framed incorrectly

Challenge http://www.freecodecamp.com/challenges/waypoint-target-html-elements-with-selectors-using-jquery has an issue. User Agent is: Mozilla/5.0 (Windows NT 10.0, WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/45.0.2454.85 Safari/537.36. Please describe how to reproduce this issue, and include links to screenshots if possible.
Hello there, in the following paragraph, the function is not understandable. I figured it out but it took me about 10 minutes and I think it could have been avoided if the sentence was framed better.
"For example, let's make all of your button elements bounce. Just add this code inside your document ready function: $("button").addClass("animated bounce")."
The "document ready function: " is all written in purple aswell so I thought it was part of what I had to put into the code.

@amarsuchak thank you for pointing this out!

Please review the Guidelines for Contributing, thank you!.

Closing because issue has been resolved by PR.


[BUG]: pattern*2d functions do not repeat holes

Describe the bug
Consider the following KCL for a series of circles with holes:
let x = startSketchOn("XY")
  |> circle([0, 0], 5, %)
  |> hole(circle([0,0], 2, %), %)
  |> patternLinear2d({
    axis: [0, 1],
    distance: 10,
    repetitions: 3.5,
  }, %)

Instead of producing five circles with holes in them, the first circle correctly gets a hole, and the repeated patterns do not.
Steps to Reproduce

Paste the above KCL

Expected Behavior
All five circles would have holes.
Screenshots and Recordings

Desktop OS
Windows 10
Browser
No response
Version
0.24.4
Additional Context
No response

This is almost certainly simply repeating the hole at [0, 0] on the XY plane five times -- but startSketchOn only works with planes and ExtrudeGroup faces, so at the present there doesn't seem to be a way to define a hole with relative position directly.

we acttually have a test for something like this, see the difference of the code here: https://github.com/KittyCAD/modeling-app/blob/main/src/wasm-lib/tests/executor/main.rs#L936
const circles = startSketchOn('XY')
    |> circle([5, 5], 1, %)
    |> patternLinear2d({axis: [1,1], repetitions: 12, distance: 3}, %)

const rectangle = startSketchOn('XY')
  |> startProfileAt([0, 0], %)
  |> line([0, 50], %)
  |> line([50, 0], %)
  |> line([0, -50], %)
  |> close(%)
  |> hole(circles, %)
  |> extrude(10, %)

I agree the above that you mentioned should work, but as a work around doing two patterns should work for now

cc @gserena01 for understanding if engine bug or not, happy to help!

confirmed! It's an engine issue, and it only appears with 2d holes, not 3d holes. Expect a fix this week!


Add new var concurrentInSystem to local mac environment

Add new var concurrentInSystem to local mac environment.
The new var was left out in one of the recent commits and it breaks local deployment on mac.
See https://github.com/openwhisk/openwhisk/pull/1191/files#r80839287 for original PR and comment.

There is still something missing. I get an error that the default limit is not set although I can see it in the props file.

Merged as https://github.com/openwhisk/openwhisk/commit/dee761c07449c9636c0b0999c1b352f1e4dfd87e. You need to restart consul to pick up the changes.

Argh. Sorry I even discarded this change manually not knowing why it was still there :/


Feature/persist

Feature to persist the cache between sessions by giving a persistCache object to the Fetch component. Works with window.localStorage or react-natives AsyncStorage.
I have used await async in source. Not sure if that breaks compatability with the targeted browsers.

Pull Request Test Coverage Report for Build 455

3 of 11 (27.27%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-5.4%) to 93.056%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/fetch.js
3
11
27.27%

Totals

Change from base Build 453:
-5.4%

Covered Lines:
106

Relevant Lines:
114

💛  - Coveralls

Thanks for the PR, @CruelMoney ! I can see you've put a lot of thought into this. Unfortunately, I won't be able to review this for awhile. I'm going to be super busy over the next few weeks!

@jamesplease no stress

I know it's been awhile, but I just had a chance to review this. I like the idea here, and I think it is important that this library allow developers to do something like this. However, I'd rather not add something this opinionated into the library.
One of the benefits of the lib, I think, is how it isn't too prescriptive about what technologies you use it with. I wonder if it'd be possible to make it more generic. i.e.; adding more props that act as hooks where developers can then plug in their own cache layers.
Another thought I have is that this may make sense to be even lower-level than React Request. Perhaps it belongs in the http lib that this library uses under-the-hood, fetch-dedupe...
I'm not sure what would be best at the moment, but I do think what is most appropriate looks slightly different from this PR. I'm going to close this out, but I'd love to continue this discussion in an issue if you'd like to open one.


Refactor speed and velocity

Description
Refactors usage of speed to velocity where vector variables were labeled as speed and  speedLen to speed. Also keeps original pre/su-fixes of such variables.
Testing

Search for speed occurences in codebase, they should refer to scalar variables only
Search for speedLen and velocityLen, there should be none
Search for velocity, should refer only to Vector2 objects
Start the game, play it for a while, nothing should change from the play perspective

Outstanding Work
Nothing right now.
Notes
Commits are made per package refactoring, so some continuity could be accomplished during refactoring, but it is intended to be squashed before merging.
Review required @eviltak
Fixes #290

Hooray Jenkins reported success with all tests good!

Hooray Jenkins reported success with all tests good!

Hooray Jenkins reported success with all tests good!


[Core] Add SQLiteBuildDBTest.cpp to CMakeLists.txt

This test wasn't being run.

Good catch, looks like we've only been running this from the Xcode project.

@swift-ci please smoke test

👏


One Checkpoint Model Missing

Hey there,
I believe that the checkpoint model NeuroLS/checkpoints/jssp_30x15_uniform/SELECT_EPSILON-SELECT_LS+-ALL  /ep74_model_rew=0.009175.ckpt is empty.
Is that so and if so, would you mind uploading it?

Hi there,
Thank you for raising the issue.
I checked my local files but I fear that I do not have the original versions of the checkpoints there since I do not have access to my old work computer. Let me check, if there still exist some old backups which include this checkpoint.

Hi again,
so I was able to recover and re-upload the missing checkpoint. Sorry for the inconvenience.
If there is anything else missing or not working, feel free to just reopen the issue.


Adding Eligibility check write up

Coverage remained the same at 95.232% when pulling f417e5d5eb94d10043607ca2119232305b283d01 on reshma-eligibility-writeup into 1970924f31d3a19939d09c49394e74ab1d028f74 on main.

This is on purpose. The person reading this guide is generally not using the API, and is trying to set up the bot, subscription or actually test.

👍


Fix: Make the facebook indicator actually build correctly

Description
Fix
I forgot to include a basic setup.py which caused Jenkins to fail.
Changelog

Adds setup.py

@krivard Ah, oops. I had rebased my branch to fixup a couple of test commits and I think I went a bit too far back in the history and included QX's unintentionally. Appreciate the 👀
I think I have fixed it now, and have also accepted your changes 👍


[bug]: CommandItem presents an error in the NextJS when used

Describe the bug
I'm creating a combobox for Brazilian states, and when using the CommandItem component, an error is presented in the system
Affected component/components
Combobox, CommandItem
How to reproduce
1- I create the combobox component with the popover and command, inside a FormField, and apply my options with an array.map()
                    <Popover>
                      <PopoverTrigger asChild>
                        <FormControl>
                          <Button
                            variant={'outline'}
                            className="items-start justify-start"
                          >
                            {'Selecione a UF'}
                          </Button>
                        </FormControl>
                      </PopoverTrigger>
                      <PopoverContent className="w-full max-h-80 overflow-scroll">
                        <Command>
                          <CommandInput placeholder="Selecione a UF" />
                          <CommandGroup>
                            {brazilianStates.map((uf) => (
                              <CommandItem
                                key={uf.abbreviation}
                                value={`${uf.abbreviation} - ${uf.name}`}
                                onSelect={() =>
                                  form.setValue('state', uf.abbreviation)
                                }
                              >
                                {uf.abbreviation} - {uf.name}
                                <CheckIcon
                                  className={cn(
                                    'ml-auto h-4 w-4',
                                    uf.abbreviation === field.value
                                      ? 'opacity-100'
                                      : 'opacity-0',
                                  )}
                                />
                              </CommandItem>
                            ))}
                          </CommandGroup>
                        </Command>
                      </PopoverContent>
                    </Popover>

2- when using this code, I try to use the combobox and the following error is presented

3- but when I replace the CommandItem with a div, the code works perfectly

Codesandbox/StackBlitz link
No response
Logs
No response
System Info
I'm use in Arc Browser, Google Chrome and Brave.
As the development framework, I am using NextJS 14.2.4.

Before submitting

[X] I've made research efforts and searched the documentation
[X] I've searched for existing issues

You have to wrap the CommandGroup inside a CommandList to make it work. Your code should be like this:

               {'Selecione a UF'}

               {brazilianStates.map((uf) => (

                     form.setValue('state', uf.abbreviation)
                   }
                 >
                   {uf.abbreviation} - {uf.name}

               ))}

You need to modify the component previously installed by shadcn, because when you add de commandList, probably can disbaled any of your commandItems, you can wrapped of course the commandGroup by the commandList to fixed the iteration problem, then if you have problems selecting your items, try this:
const CommandItem = React.forwardRef<
React.ElementRef,
React.ComponentPropsWithoutRef

(({ className, disabled, ...props }, ref) => (
<CommandPrimitive.Item
ref={ref}
disabled={disabled}
className={cn(
"relative flex cursor-pointer select-none items-center rounded-sm px-2 py-1.5 text-sm outline-none aria-selected:bg-accent aria-selected:text-accent-foreground",
disabled ? "opacity-50" : '',
className
)}
{...props}
/>
))

Correct and the documentation needs to be updated as it still only shows how to do it with previous versions of the radix library (used for those Command components): https://github.com/pacocoursey/cmdk/releases/tag/v1.0.0

while wrap the CommandGroup inside a CommandList we lost click or onselect behavour
Could anyone please provide potential solutions for maintaining the click and on-select behaviors while encapsulating the CommandGroup within a CommandList?

There's a bug in the generated Command component (downloaded from shadcn).
<CommandPrimitive.Item
ref={ref}
className={cn(
"relative flex cursor-default select-none items-center rounded-sm px-2
py-1.5 text-sm outline-none aria-selected:bg-accent
aria-selected:text-accent-foreground data-[disabled]:pointer-events-none
data-[disabled]:opacity-50",
className
)}
{...props}
/>
))

(({ className, ...props }, ref) => {
return (
The CSS is checking for data-[disabled]
but the attribute is being set as disabled="false" which is still
considered truthy in CSS. I changed it with a check for disabled='true', as
per below: (You need to do it for pointer-events and opacity classes.)

<CommandPrimitive.Item
ref={ref}
className={cn(
"relative flex cursor-default select-none items-center rounded-sm px-2
py-1.5 text-sm outline-none aria-selected:bg-accent
aria-selected:text-accent-foreground
data-[disabled='true']:pointer-events-none
data-[disabled='true']:opacity-50 ",
className
)}
{...props}
/>
);
});
Le mar. 30 juill. 2024, à 06 h 17, prajanyasoft-prabin <
@.***> a écrit :

while wrap the CommandGroup inside a CommandList we lost click or onselect
behavour
Could anyone please provide potential solutions for maintaining the click
and on-select behaviors while encapsulating the CommandGroup within a
CommandList?
—
Reply to this email directly, view it on GitHub
https://github.com/shadcn-ui/ui/issues/4377#issuecomment-2257997760, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AJE2LESY4OOMEWVJOC2WMXLZO5R5HAVCNFSM6AAAAABLIHDGSGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENJXHE4TONZWGA
.
You are receiving this because you commented.Message ID:
@.***>


question from beginner

Hello, I followed the instructions to deploy metagpt on my mac, while my first time testing it appears to not found specific module.
File "/metagpt/startup.py", line 5, in
import fire
ModuleNotFoundError: No module named 'fire'
Can anyone help me with this, im not familiar with coding

Follow README Traditional Installation section. Install all the dependencies first.

Thanks for your soon reply, i reinstalled metagpt and i caught error like following :
Download error on https://pypi.org/simple/typing-inspect/: [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local issuer certificate (_ssl.c:1002) -- Some packages may not be found!
Couldn't find index page for 'typing-inspect' (maybe misspelled?)
Scanning index of all packages (this may take a while)
Reading https://pypi.org/simple/
Download error on https://pypi.org/simple/: [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local issuer certificate (_ssl.c:1002) -- Some packages may not be found!
No local packages or working download links found for typing-inspect==0.8.0
error: Could not find suitable distribution for Requirement.parse('typing-inspect==0.8.0')

I just wrote this up incase it'll help you get started on your project.
https://github.com/geekan/MetaGPT/issues/166


Support custom startup parameters for extensions

Proposal
Allow extensions to register and consume their own startup configuration options so that e.g. WireMock standalone can be started with extra parameters like --my-extension-parameter=123
e.g. we could add an extra method to the Extension interface like:
List<OptionDefinition> getCustomStartupOptions();

And one to Options for retrieval:
Optional<OptionValue> getOptionValue(String key);

This should build on top of #2578.
References
No response

@dirkbolte as we discussed last week

Can/should we scope them to avoid conflicts between different extensions/wiremock core? Thinking about adapting something like spring's externalized configuration: --extensions.<extensionName>.<parameter>=<value> . Same scheme could also be stored in a properties/yaml/json file.

I've wondered about that, then wondered if it's overkill. Not sure how likely it is that collisions will occur, but maybe I'm lacking imagination...

My hope would be that this would not bloat the whole topic when it's added now - while it would be a breaking change when added later. If there would be at any time two extensions with conflicts, there's practically no way for a user to mitigate it and neither for the extension developers to adapt it with another breaking change.
I can imagine one use case where we might run into conflicts relatively fast: when having extensions for subsystems like Store. I can assume that the parameters for stores might be very similar (timings, URLs, credentials).
I would hope that users are very familiar with this notation, so this should hopefully not cause any usage issues.

OK, agree playing it safe might be a good idea. How about just --ext.<extensionName>.<parameter>=<value> so it's not quite as many keystrokes?


Coding Standards and Style

Let's use this issue to discuss about and decide on a set of coding style, standards, and other guidelines.
A new project offers us an opportunity to adopt more modern coding style than what is used by ROOT/Geant4.
I suggest we look into existing widely used standards (including ROOT/Geant4), and pick something based on them to follow.
References:

Geant4 Coding Guidelines
GNU Coding Standards
Google C++ Style Guide
Indentation Style (Wikipedia)
ISO C++ Coding Standards
Linux Kernel Coding Style
LLVM Coding Standards
ROOT Coding Conventions
WebKit Code Style Guidelines

I think the style question is already done, we have a .clang-format (see #10 and #12). Is there anything in particular about the semantic aspect (like naming of attributes) that you'd like to discuss?

Yes, clang-format only tells you how to format your code, and I agree that having a tool do it and just accept its output as the right way to format is best. However that does not cover how to naming, and many other things from the links above that are a matter of style.

There are further options in clang-tidy. For naming you have e.g.: https://clang.llvm.org/extra/clang-tidy/checks/readability-identifier-naming.html

Thanks, @bernhardmgruber. Looks like a lot of work to configure, but should do the job well once that's done.


Advantage of elasticache client

I am able to connect to AWS elasticache using the XMemcached client, which is logical since AWS is Memcached-protocol-compatible.
What advantage would I get out of using the ElastiCache client? How should I decide which one to use?

Probably the main feature of ElastiCache client is dedicated auto discovery memchanizm: https://docs.aws.amazon.com/AmazonElastiCache/latest/mem-ug/AutoDiscovery.html


Inject JS before screenshot

Hello,
Is it possible to inject JS in page ?
The idea is to remove cookie popup before screenshot. :)

no it's not possible currently, if you want to implement it you can do that close to here:
https://github.com/elestio/ws-screenshot/blob/a15ce4f4cb725a0e5532b19adbed50169ffd85bc/API/shared.js#L214


CI: Optimize for faster results and lower resource use

Three commits:

Split specs running on ubuntu and windows.
We'll have some duplication, but it allows for some optimization.
Reduce some test matrixes
Reduce the test matrix for ascii_spec from 13 combinations to just four, and for windows specs from six to three.
Based on discussion in https://github.com/rubocop/rubocop/pull/12967#discussion_r1626082395 and https://github.com/rubocop/rubocop/pull/12965#issuecomment-2147419092.
Only run windows specs after ubuntu specs pass.
We don't require windows specs to pass, and windows specs are very slow. By only running them after ubuntu specs pass, we get the most important results first - and in case of failing specs, we'll save 30 minutes of CI use.

It looks like the few specs in spec/rubocop/cli* and spec/rubocop/server/ make up about 2/3 of the total runtime of our specs. If we could either 1) make them faster or 2) run them less often, our CI would become quite a bit faster.

We could use the :isolated_environment tag to separate fast and slow specs:
$ bundle exec rspec --tag=isolated_environment
Finished in 1 minute 14.25 seconds (files took 3.12 seconds to load)
974 examples, 0 failures

$ bundle exec rspec --tag=~isolated_environment
Finished in 22.79 seconds (files took 3.21 seconds to load)
21427 examples, 0 failures, 1 pending

We could use the :isolated_environment tag to separate fast and slow specs:

Have these two runs be part of the matrix for windows, already getting some gains from splitting it up. Since they run as the last step there are likely a few free runners available: actions has a concurrency of 20 https://docs.github.com/en/actions/learn-github-actions/usage-limits-billing-and-administration#usage-limits

Windows tests (to my knowledge) didn't run on CircleCI: they were moved from AppVeyor to GitHub Actions a few years ago

Great work!

Windows tests (to my knowledge) didn't run on CircleCI: they were moved from AppVeyor to GitHub Actions a few years ago

Yep, that's how I remember it as well. For a very long time we didn't have a Windows build at all, btw.


Doesn't update over multiple tmux panes

Description
When I switch the tmux pane, the presence status is still stuck on the previous one
Neovim version
NVIM v0.9.5
Build type: Release
LuaJIT 2.1.1702233742

OS information
Arch linux lts ver 6.6.12-1
Steps to reproduce

Setup nvim instance within a tmux session with multiple panes
Setup another instance in another pane
Switch panes

Logs
Loaded 32 plugin files!
Client 1 quit with exit code 150 and signal 0
Client 4 quit with exit code 150 and signal 0
NVIM v0.9.5
Build type: Release
LuaJIT 2.1.1702233742

This could also possibly be resolved by adding a command, something like :NeocordOverride, which makes the current neovim instance take control of discord presence management. Though this would force users to either call that manually every time they switch panes or make a script that does it for them

I was thinking about making whatever client is focused, do that override.


Can't run grammY on Cloudflare Workers / wrangler locally

How to reproduce

Create a blank workers project with Wrangler
Add grammy
Run wrangler dev --local

✘ [ERROR] Could not resolve "http"

    node_modules/grammy/out/platform.node.js:5:23:
      5 │ const http_1 = require("http");
        ╵                        ~~~~~~

  The package "http" wasn't found on the file system but is built into node.
  Add "node_compat = true" to your wrangler.toml file to enable Node compatibility.

✘ [ERROR] Could not resolve "https"

    node_modules/grammy/out/platform.node.js:6:24:
      6 │ const https_1 = require("https");
        ╵                         ~~~~~~~

  The package "https" wasn't found on the file system but is built into node.
  Add "node_compat = true" to your wrangler.toml file to enable Node compatibility.

✘ [ERROR] Could not resolve "stream"

    node_modules/grammy/out/platform.node.js:7:25:
      7 │ const stream_1 = require("stream");
        ╵                          ~~~~~~~~

  The package "stream" wasn't found on the file system but is built into node.
  Add "node_compat = true" to your wrangler.toml file to enable Node compatibility.

✘ [ERROR] Could not resolve "fs"

    node_modules/grammy/out/types.node.js:18:21:
      18 │ const fs_1 = require("fs");
         ╵                      ~~~~

  The package "fs" wasn't found on the file system but is built into node.
  Add "node_compat = true" to your wrangler.toml file to enable Node compatibility.

✘ [ERROR] Could not resolve "path"

    node_modules/grammy/out/types.node.js:20:23:
      20 │ const path_1 = require("path");
         ╵                        ~~~~~~

  The package "path" wasn't found on the file system but is built into node.
  Add "node_compat = true" to your wrangler.toml file to enable Node compatibility.

✘ [ERROR] Build failed with 5 errors:

  node_modules/grammy/out/platform.node.js:5:23: ERROR: Could not resolve "http"
  node_modules/grammy/out/platform.node.js:6:24: ERROR: Could not resolve "https"
  node_modules/grammy/out/platform.node.js:7:25: ERROR: Could not resolve "stream"
  node_modules/grammy/out/types.node.js:18:21: ERROR: Could not resolve "fs"
  node_modules/grammy/out/types.node.js:20:23: ERROR: Could not resolve "path"

If I add node_compat = true, I get another error
[mf:err] TypeError: globalThis.XMLHttpRequest is not a constructor

/node_modules/rollup-plugin-node-polyfills/polyfills/http-lib/capability.js:20
    xhr = new global.XMLHttpRequest()
          ^
TypeError: globalThis.XMLHttpRequest is not a constructor

You need to import from grammy/web. A guide for this is in the works. You can look at a preview of it at https://github.com/grammyjs/website/pull/587#issuecomment-1407387869

Hey @KnorpelSenf, thank you for helping out. Do you think it is possible to use grammyjs with a router library? The most popular on Workers is this one.
https://github.com/kwhitley/itty-router
I was trying to call webhookCallback(bot, "cloudflare") with little success. Sorry if my question is too noob.

Yes, there is a way to do this. I'm not familiar with that project. Can you share the code that you used when you tried to integrate the two libraries?

This is the original way your documentation suggests using Grammy, it works on Workers just fine
https://github.com/dandaka/workers-grammy/blob/main/src/index.ts
This is a version with a router that does not work
https://github.com/dandaka/workers-grammy/blob/main/src/index-router.ts
I assume there should be a way to link 2 different event listeners, but could not find a way yet

Also when I call
webhookCallback(bot, "cloudflare");
I get the following TS error in my IDE:
No overload matches this call.
  Overload 1 of 2, '(bot: Bot<Context, Api<RawApi>>, adapter?: "callback" | "express" | "koa" | "fastify" | "std/http" | "oak" | "hono" | "http" | "https" | "aws-lambda" | "azure" | "next-js" | "sveltekit" | FrameworkAdapter | undefined, onTimeout?: "throw" | ... 2 more ... | undefined, timeoutMilliseconds?: number | undefined, secretToken?: string | undefined): (...args: any[]) => any', gave the following error.
    Argument of type '"cloudflare"' is not assignable to parameter of type '"callback" | "express" | "koa" | "fastify" | "std/http" | "oak" | "hono" | "http" | "https" | "aws-lambda" | "azure" | "next-js" | "sveltekit" | FrameworkAdapter | undefined'.
  Overload 2 of 2, '(bot: Bot<Context, Api<RawApi>>, adapter?: "callback" | "express" | "koa" | "fastify" | "std/http" | "oak" | "hono" | "http" | "https" | "aws-lambda" | "azure" | "next-js" | "sveltekit" | FrameworkAdapter | undefined, webhookOptions?: WebhookOptions | undefined): (...args: any[]) => any', gave the following error.
    Argument of type '"cloudflare"' is not assignable to parameter of type '"callback" | "express" | "koa" | "fastify" | "std/http" | "oak" | "hono" | "http" | "https" | "aws-lambda" | "azure" | "next-js" | "sveltekit" | FrameworkAdapter | undefined'.

The code runs fine on Workers thou.

Using the latest version is always a good idea.
I'm not sure why the type error happens, I'll ask the person who contributed it about that.
Thanks for sharing the code snippets. It looks to me like the router expects handlers in the form of (req: Request) => Promise<Response>. Coincidentally, this is also the function signature that event handlers in worker modules expect. Can you try using the "cloudflare-mod" adapter and report back if that does the job? It isn't documented at https://grammy.dev/guide/deployment-types.html#web-framework-adapters yet but I'll include your experience when I get to edit the page. (Or do you feel like adding it to the docs if you can make it work?)

I have managed to launch Grammy with honojs/hono router, works great.


[Bug]: Activation hot key doesn't allow ctrl-space any more

Before submitting your bug report, please confirm you have completed the following steps

[X] Searched the issue is not already listed in the Issue List (I've searched it)
[X] Searched in the Quicksilver Users' Group for questions relating to tech support.
[X] Downloaded the latest version of Quicksilver for your system and confirmed the issue still persists.

Bug description
I used to use ctrl-spacebar as my hotkey to call up Quicksilver.  After downloading the latest version this doesn't work any more.
Run setup, type ctrl-spacebar and what appears in the window is ^1
Close
Type  ctrl-space, no Quicksilver any more.  Type ctrl-1 nothing happens either... (not that that is what I want).
Help - it's my muscle memory to use that combination...  (NB Still works on my laptop - haven't downloaded the latest version there).
Steps to reproduce

Try to set hot keys for activation to Ctrl-space
Exit setup / preferences
Type Ctrl-space
Nothing happens

Expected behavior
Press ctrl-space and Quicksilver opens...
MacOS Version
macOS 10.15
Quicksilver Version
2.0.0 (4027)
Relevant Plugins
n/a
Crash Logs or Spindump
n/a
Screenshots
can't remember how to take screenshots without quicksilver - used just to launch it and type screenshot ;-(
But anyhow, not much to show
Additional info
No response

Sorry for the inconvenience!
This is a known issue -- please update to the latest version (https://github.com/quicksilver/Quicksilver/releases); if you're still having trouble let us know, and I'll be happy to reopen.
See also:

https://github.com/quicksilver/Quicksilver/issues/2662
https://github.com/quicksilver/Quicksilver/issues/2682
https://github.com/quicksilver/Quicksilver/issues/2687
https://github.com/quicksilver/Quicksilver/issues/2692
https://github.com/quicksilver/Quicksilver/issues/2699
https://github.com/quicksilver/Quicksilver/issues/2700
https://github.com/quicksilver/Quicksilver/issues/2712

Apologies, I had forgotten that when I tried to update from 2.0 it hung.
Deleted quicksilver, installed from your web page and now all works perfectly.  Thank you for your quick response.


/speechbubble command error

Command that caused the error
/speechbubble
Image that caused the error
n/a
Steps to reproduce

Use /speechbubble command
Result

Self-hosted instance?
No
Error file
Message: Error: A boolean was expected
    at file:///home/ubuntu/esmBot/utils/image-runner.js:50:39
    at processTicksAndRejections (node:internal/process/task_queues:96:5)

Stack Trace: Error: A boolean was expected
    at file:///home/ubuntu/esmBot/utils/image-runner.js:50:39
    at processTicksAndRejections (node:internal/process/task_queues:96:5)

Additional context
No response

Judging by the error given, it looks like this error is from a self-hosted instance. Check if the natives were built properly.


PARADISEC export

The current csv export needs a bit of work to get into PARADISEC. Would it be possible to export into the same format as the PARADISEC minimal metadata import spreadsheet?

Hi John, just checking in if we need to get some more funds to support this effort? Thanks.

No, I'm sorry I've slowed down, but not for lack of funding. Where I
thought COVID 19 would slow down the world of Bloom, it actually doubled
our work as everyone is now more interested in enabling remote learning.
This has left me too worn out to get as much done on weekends. We're not
entering a period of calm, however.
jh
On Wed, Sep 9, 2020 at 1:57 PM nthieberger<EMAIL_ADDRESS>wrote:

Hi John, just checking in if we need to get some more funds to support
this effort? Thanks.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/onset/lameta/issues/12#issuecomment-689786438, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAACCADOGDQ7DXOIQ3GM5QDSE7M2TANCNFSM4MMJ4R5A
.

OK, I hope it settles down soon, or that you get an apprentice.
Nick
On Thu, 10 Sep 2020 at 06:07, John Hatton<EMAIL_ADDRESS>wrote:

UoM notice: External email. Be cautious of links, attachments, or
impersonation attempts *

No, I'm sorry I've slowed down, but not for lack of funding. Where I
thought COVID 19 would slow down the world of Bloom, it actually doubled
our work as everyone is now more interested in enabling remote learning.
This has left me too worn out to get as much done on weekends. We're not
entering a period of calm, however.
jh
On Wed, Sep 9, 2020 at 1:57 PM nthieberger<EMAIL_ADDRESS>wrote:

Hi John, just checking in if we need to get some more funds to support
this effort? Thanks.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/onset/lameta/issues/12#issuecomment-689786438, or
unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AAACCADOGDQ7DXOIQ3GM5QDSE7M2TANCNFSM4MMJ4R5A
.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/onset/lameta/issues/12#issuecomment-689791349, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEVJIDCU7PDKZZPTQKGLILSE7N6VANCNFSM4MMJ4R5A
.

This feature has shipped, and I'm looking for feedback.


Clicking Outside Alert Box Performs Action Instead of Retaining Previous State

Current behavior
The action gets performed successfully, and the toggle changes its state.
Expected behavior
The action should not be performed. The toggle should retain its previous state.
Steps to reproduce the issue
Navigate to the notifications section on settings page.
Click on any toggle to enable or disable a notification.
An alert box appears, prompting the user to confirm or cancel the action.
Click outside the alert box.
Environment details
App version : v2.24.0
App environment : Prod
Additional information
https://jam.dev/c/da62b021-9cde-42cd-883c-e453fec6cb66

Issue verified and working fine in release v2.24.1. Hence, closing it.
https://jam.dev/c/b609b33c-4094-473e-b43f-cea1fea2dd76


Add init to test_utils/scripts

Fixes missing __init__.py. Confused as to why the tests didn't raise the same error as transformers.
@sgugger

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


Move topo-descriptor under the MeteoSwiss account on pypi

Hey @dnerini and @matschaer,
I'd like to suggest to move the existing pypi releases of topo-descriptor under the pypi account MeteoSwiss.
To do so requires @dnerini to login onto pypi, and under the package page add "MeteoSwiss" as owner.
To ease the CI/CD, I could suggest the addition of this action to your Github repo.
What do you think ? If this works out smoothly, I'd try to do the same for all other public MCH codes on pypi.

Hey @dnerini, it worked ! "MeteoSwiss" is now an owner of topo-descriptors on pypi. If that is ok for you, I'll set your role as "maintainer" on there. You'll still be able to push new releases using your credentials, but won't be able to delete anything (only "MeteoSwiss" would be able to do that). What do you think ?
For the Github action, I just remembered that you had something similar already setup. The main difference would be to use the organization-wide secrets PYPI_API_TOKEN and TESTPYPI_API_TOKEN to do the authentication.

For the Github action, I just remembered that you had something similar already setup. The main difference would be to use the organization-wide secrets PYPI_API_TOKEN and TESTPYPI_API_TOKEN to do the authentication.

Done in 371b604.


Client Freezes When Exiting a Remote Server Connection

Brief Description:
I mentioned this on the Slack channel, but see the bug still persisting in RC4 . When trying to exit the Desktop Client, the application hangs if you are connected to a remote server. I believe it is ineffectively trying to shut down the remote server instead of the local one.
Operating System (OS and version): Windows 10 Home edition.
OpenBazaar version (shown on About OpenBazaar page in menu): RC4
Hardware: Intel i7 based laptop.
Reproducible (Always / Almost Always / Sometimes / Rarely / Couldn't Reproduce):
Always
Steps to reproduce:

Open the desktop client.
Connect to a remote server.

Observed Behavior:
This is the screen showing my local server and my remote server:

This is the screen that is displayed when OpenBazaar freezes:

This is happening to me as well on Linux. Running version 2.3.0 and that is when I first began having the problem. I've let it sit overnight and it never closes.


Landing screen on certain screen sizes has ability to scroll and see "Press Enter To Start" flashing under.

The launch Screen has the ability to scroll to see the press enter to start. You are also able to press enter while still on the launch screen.

My screen size is 1920x1080.
This error disappears at the screen size of 1600x900.

I have problems trying to replicate this issue. May I know what OS and Browser you are using? Trying to replicate this issue by using Google and Safari Responsive Design Mode doesn't appear to work.

@Dinervc Hi! Yes I'm using Google chrome on Windows 11.

Hi! @Oia20 Hope all Well, i am See Your Project code. Landing screen is scroll Right and Down because Of You Give Margin in Img and start Div. Fix margin and olso fix width of Some Div. ( Give width in "%" and Height in "vh"). try it if not Solve i Will Help :)


Reset a point light that has a body takes a lot of time

Version

Phaser Version: 3.55.2

Operating system: Win10

Browser:

Description

First create a static group, and add a lot of PointLight, and then calling 'body.reset', the time consumed is increasing as the number of calls increases. I don't know if this is the correct usage!
Example Test Code

create(){
      const group = this.physics.add.staticGroup();

      for (let i = 0; i < 20; i++) {
         for (let j = 0; j < 20; j++) {
            const p = new PointLight(this, i * 40 + 100, j * 40 + 100, 0xff0000, 20, Math.random())
               .addToDisplayList();
            group.add(p);
         }
      }

    const pl = new PointLight(this, 20, 20, 0xff0000, 20, 1)
         .addToDisplayList();
    group.add(pl);

    // Test
    setInterval(() => {
       console.time('body.reset');
       // this.pl.body.enable = false;
       // this.pl.body.enable = true;
       pl.body.reset(30, 30);
       console.timeEnd('body.reset');
    }, 1000);
}

Additional Information

Long story, but the ultimate problem is that StaticBody#reset() calls PointLight#getTopLeft(), which spoils the StaticBody#position values.
I think the GetBounds component needs to be removed from PointLight, since the methods will fail.
You can probably work around this by resetting the body manually — remove from static tree, modify position, insert in static tree. But you will have to figure out the offset between the body position (top-left) and light position (center).

Yes, I also found that the old body has not been removed from the static tree. I temporarily replaced it with other methods, thank you for your suggestion @samme


[GSoD 2021] Added Swagger UI to Express Route by default

Fixes: #4
Related: wechaty/wechaty.js.org#1012

@huan @proudofsimin @Rohitesh-Kumar-Jain Please take a look.

I think only @huan could review this PR.

Thanks for pinging me.
Will review it later, thanks to all @iamrajiv @rohi
@lijia

It's a great feature and I'm working on it this week.


Opentelemetry metrics are not supported in AppInsights

Please provide us with the following information:

This issue is a: (mark with an x)

[ ] bug report -> please search issues before submitting
[x ] documentation issue or request
[ ] regression (a behavior that used to work and stopped in a new release)

based on documentation metrics should be supported https://learn.microsoft.com/en-us/azure/container-apps/opentelemetry-agents?tabs=arm

But this metrics are actually not supported when I tried to enable it.

@SophCarp

You probably just forgot a backslash after ...traces true.
That is why zsh screams later "command not found"

Docs have been updated, thank you for reporting the issue.
https://learn.microsoft.com/en-us/azure/container-apps/opentelemetry-agents?tabs=arm


What is the reason to have non-generic request interfaces?

Its make them similar for notification interfaces.
So, using of non-generic requests can confuse.
If you want response you must use requests, otherwise just notification.

In the case of a request without a response, in case you don't have one. Notifications target multiple handlers, also.

Why you want to know what notification will handle once?

Not quite sure I understand you - both Send and Publish block for handlers to respond.
Sometimes Send doesn't have a response, but since C# doesn't let you use a Void type as a type parameter, I had to create a helper interface.
Send = 1 -> 1
Publish = 1 -> *
Send = Request/Response. Sometimes response is null
Publish = Broadcast. Can't have a response.


[debug] Add data-debug

This can be useful when debugging sequences.

It should just do a console.log of the value of data-debug, for starters.


Check for successful response on scan download

webbreaker webinspect scan
add in logic to check the response after the scan is 'complete' for a 200 error. Give user appropriate feedback.

remediated with 167 PR.


How to Debug Multiple Dapr Projects Managed by Aspire

Hello,
I am managing several WebAPI projects with Aspire. These WebAPI projects use Dapr for microservice interactions. When I create an Aspire project and add these WebAPI projects with Aspire support, including the aspire.Dapr dependency, I encounter errors when trying to run them.
I would like to know how to migrate existing Dapr-based WebAPI projects to be managed by Aspire.

Can you provide more detail about the errors you are experiencing. It would be helpful if you could create a small repo in a GitHub repository and link it here so we can quickly reproduce what you are seeing.

Here is my GitHub demo: https://github.com/xiaofang-code/dapr_aspireDemo.
My goal is to be able to set breakpoints and step into the code while making requests through Aspire, to facilitate development and debugging. This demo currently throws errors, possibly due to issues in the JSON.
Thank you for your help.

Can you provide more detail about the errors you are experiencing. It would be helpful if you could create a small repo in a GitHub repository and link it here so we can quickly reproduce what you are seeing.您能否提供有关您遇到的错误的更多详细信息。如果您可以在 GitHub 存储库中创建一个小型存储库并将其链接到此处，这将很有帮助，以便我们可以快速重现您所看到的内容。

What errors do you encounter? Can you show them here?

What errors do you encounter? Can you show them here?您遇到什么错误？你能在这里展示它们吗？

Here is the error I get when running the demo:
System.Text.Json.JsonException: “The JSON value could not be converted to System.Nullable1[System.Boolean]. Path: $.profiles.CheckoutService.dotnetRunMessages | LineNumber: 5 | BytePositionInLine: 33.”
`
If possible, please pull the demo repository, run it, and check for more detailed information.
https://github.com/xiaofang-code/dapr_aspireDemo
Thank you for your help.

Where does that exception come from? Can you show the entire call stack?

Where does that exception come from? Can you show the entire call stack?这个异常从何而来？你能显示整个调用堆栈吗？
This is the error message I received after running the program:
System.Text.Json.JsonException   HResult=0x80131500   Message=The JSON value could not be converted to System.Nullable1[System.Boolean]. Path: $.profiles.CheckoutService.dotnetRunMessages | LineNumber: 5 | BytePositionInLine: 33.

[调用堆栈已截断]
内部异常 1:
InvalidOperationException: Cannot get the value of a token type 'String' as a boolean.
`
My ultimate goal is to take over the existing Dapr project through Aspire and debug the code locally. Does the official team have any demos or resources that they can provide for reference?

Your launch profile is broken. This is what it’s complaint about您的启动配置文件已损坏。这就是投诉的内容
https://github.com/xiaofang-code/dapr_aspireDemo/blob/0a85acc1a9f508fc4753696aa9827da5c86be9eb/Services/order-processor/Properties/launchSettings.json#L6
That remove the quotes around “true”删除“true”周围的引号

Thank you! I have resolved the issue and can now use it successfully.

That's weird: I just had the same issue, also for the dotnetRunMessages. Must have been generated this way by an early version of Visual Studio or Rider.

I had this issue too.  I found that it was the launchsettings.json in my service projects rather than the AppHost project that had the "true" that I needed to update to true.


TRIGGER with RAISE keyword issue while running build runner

First off, thank you for this really helpful package.
After completing my .moor file, I tried running the builder and encountered the following issue:
Could not parse this expression. Note: This is a reserved keyword, you can escape it in double ticks
RAISE (ABORT, "Attempting to add a parent category that does not exist")
^^^^^

For a better context, here is my the whole trigger definition
CREATE TRIGGER categories_bi BEFORE INSERT ON categories
BEGIN
    SELECT
        CASE WHEN new.is_top_level < 1 THEN
	    CASE WHEN NOT EXISTS (SELECT id FROM categories WHERE id = new.parent_id) THEN
		RAISE (ABORT, "Attempting to add a parent category that does not exist")
	    END
        END;
END;

Tried wrapping the RAISE and inner CASE statement with parenthesis but the runner still throws the issue.
I might be missing something, as SQL is not my most strongest suite. Could you provide some clue on how to go about this? Many thanks!

I think RAISE is not yet supported by the sql parser used in the generator. It shouldn't take much to implement it, I'll take a look at that now.

This should be fixed now. If you want to try it out before the next update, you can temporarily add this to your pubspec:
dependency_overrides:
  moor_generator:
    git:
      url: https://github.com/simolus3/moor.git
      ref: 17f43bc9fc192510f3bc62893c219e92d6c0332a
      path: moor_generator
  sqlparser:
    git:
      url: https://github.com/simolus3/moor.git
      ref: 17f43bc9fc192510f3bc62893c219e92d6c0332a
      path: sqlparser

Thank you for your quick response! I will try it and report back here.


Error building app when asset fingerprint prepend is set

Hello, I am having an issue where if my app has an asset fingerprint prepend set, it gets this error while building:
$ ember build
Could not start watchman
Visit https://ember-cli.com/user-guide/#watchman for more info.
cleaning up...
Build failed.
Build Error (AssetRewrite) in assets/vendor.js

Cannot read property '2' of null

Stack Trace and Error Report: /var/folders/sq/n70gy7gj2yl1lhdpbdsk5cf40000gn/T/error.dump.7ca510575e7a9fc80ba3a7c583c28aa2.log

The contents of that "Stack Trace and Error Report" file
=================================================================================

ENV Summary:

  TIME: Tue May 29 2018 15:04:00 GMT-0400 (EDT)
  TITLE: ember
  ARGV:
  - /usr/local/Cellar/node/10.2.1/bin/node
  - /usr/local/bin/ember
  - build
  EXEC_PATH: /usr/local/Cellar/node/10.2.1/bin/node
  TMPDIR: /var/folders/sq/n70gy7gj2yl1lhdpbdsk5cf40000gn/T
  SHELL: /usr/local/bin/zsh
  PATH:
  - /Users/markoleson/.rvm/gems/ruby-2.5.0/bin
  - /Users/markoleson/.rvm/gems/ruby-2.5.0@global/bin
  - /Users/markoleson/.rvm/rubies/ruby-2.5.0/bin
  - /Users/markoleson/.avn/bin
  - /Users/markoleson/.cargo/bin
  - /usr/local/heroku/bin
  - /usr/local/sbin
  - /usr/local/bin
  - /usr/bin
  - /bin
  - /usr/sbin
  - /sbin
  - /Users/markoleson/bin
  - /Users/markoleson/.antigen/bundles/zsh-users/zsh-syntax-highlighting
  - /Users/markoleson/.rvm/bin
  PLATFORM: darwin x64
  FREEMEM: 166084608
  TOTALMEM:<PHONE_NUMBER>4
  UPTIME: 1473985
  LOADAVG: 2.81103515625,2.53076171875,2.49755859375
  CPUS:
  - Intel(R) Core(TM) i7-4870HQ CPU @ 2.50GHz - 2500
  - Intel(R) Core(TM) i7-4870HQ CPU @ 2.50GHz - 2500
  - Intel(R) Core(TM) i7-4870HQ CPU @ 2.50GHz - 2500
  - Intel(R) Core(TM) i7-4870HQ CPU @ 2.50GHz - 2500
  - Intel(R) Core(TM) i7-4870HQ CPU @ 2.50GHz - 2500
  - Intel(R) Core(TM) i7-4870HQ CPU @ 2.50GHz - 2500
  - Intel(R) Core(TM) i7-4870HQ CPU @ 2.50GHz - 2500
  - Intel(R) Core(TM) i7-4870HQ CPU @ 2.50GHz - 2500
  ENDIANNESS: LE
  VERSIONS:
  - ares: 1.14.0
  - cldr: 33.0
  - http_parser: 2.8.0
  - icu: 61.1
  - modules: 64
  - napi: 3
  - nghttp2: 1.29.0
  - node: 10.2.1
  - openssl: 1.1.0h
  - tz: 2018c
  - unicode: 10.0
  - uv: 1.20.3
  - v8: 6.6.346.32-node.8
  - zlib: 1.2.11

ERROR Summary:

  - broccoliBuilderErrorStack: TypeError: Cannot read property '2' of null
    at AssetRewrite.rewriteAssetPath (/Users/markoleson/Developer/ember-cli-showdown-asset-prepend-issue/node_modules/broccoli-asset-rewrite/index.js:127:91)
    at AssetRewrite.processString (/Users/markoleson/Developer/ember-cli-showdown-asset-prepend-issue/node_modules/broccoli-asset-rewrite/index.js:167:24)
    at AssetRewrite.processFile (/Users/markoleson/Developer/ember-cli-showdown-asset-prepend-issue/node_modules/broccoli-filter/index.js:165:31)
    at asyncProcessFile (/Users/markoleson/Developer/ember-cli-showdown-asset-prepend-issue/node_modules/broccoli-filter/index.js:122:21)
    at tryCatch (/Users/markoleson/Developer/ember-cli-showdown-asset-prepend-issue/node_modules/rsvp/dist/rsvp.js:525:12)
    at invokeCallback (/Users/markoleson/Developer/ember-cli-showdown-asset-prepend-issue/node_modules/rsvp/dist/rsvp.js:538:13)
    at /Users/markoleson/Developer/ember-cli-showdown-asset-prepend-issue/node_modules/rsvp/dist/rsvp.js:606:14
    at flush (/Users/markoleson/Developer/ember-cli-showdown-asset-prepend-issue/node_modules/rsvp/dist/rsvp.js:2415:5)
    at process._tickCallback (internal/process/next_tick.js:61:11)
  - codeFrame: Cannot read property '2' of null
  - errorMessage: Build Canceled: Broccoli Builder ran into an error with `AssetRewrite` plugin. 💥
Cannot read property '2' of null
  - errorType: Build Error
  - location:
    - column: [undefined]
    - file: assets/vendor.js
    - line: [undefined]
    - treeDir: /Users/markoleson/Developer/ember-cli-showdown-asset-prepend-issue/tmp/asset_rewrite-input_base_path-aQLdKWk2.tmp
  - message: Build Canceled: Broccoli Builder ran into an error with `AssetRewrite` plugin. 💥
Cannot read property '2' of null
  - name: Error
  - nodeAnnotation: AssetRewrite
  - nodeName: AssetRewrite
  - originalErrorMessage: Cannot read property '2' of null
  - stack: TypeError: Cannot read property '2' of null
    at AssetRewrite.rewriteAssetPath (/Users/markoleson/Developer/ember-cli-showdown-asset-prepend-issue/node_modules/broccoli-asset-rewrite/index.js:127:91)
    at AssetRewrite.processString (/Users/markoleson/Developer/ember-cli-showdown-asset-prepend-issue/node_modules/broccoli-asset-rewrite/index.js:167:24)
    at AssetRewrite.processFile (/Users/markoleson/Developer/ember-cli-showdown-asset-prepend-issue/node_modules/broccoli-filter/index.js:165:31)
    at asyncProcessFile (/Users/markoleson/Developer/ember-cli-showdown-asset-prepend-issue/node_modules/broccoli-filter/index.js:122:21)
    at tryCatch (/Users/markoleson/Developer/ember-cli-showdown-asset-prepend-issue/node_modules/rsvp/dist/rsvp.js:525:12)
    at invokeCallback (/Users/markoleson/Developer/ember-cli-showdown-asset-prepend-issue/node_modules/rsvp/dist/rsvp.js:538:13)
    at /Users/markoleson/Developer/ember-cli-showdown-asset-prepend-issue/node_modules/rsvp/dist/rsvp.js:606:14
    at flush (/Users/markoleson/Developer/ember-cli-showdown-asset-prepend-issue/node_modules/rsvp/dist/rsvp.js:2415:5)
    at process._tickCallback (internal/process/next_tick.js:61:11)

=================================================================================

The line it is talking about causing the error is here.
The value of replaceString looks to be:
assets/showdown.js-e62a355f75b7091f050b33ee62504bdb.map

}

I'm guessing it is not supposed to include that linebreak and }, but I really don't know.
I have a simple reproduction here: https://github.com/fusion2004/ember-cli-showdown-asset-prepend-issue

This should now be resolved in 4.4.3.  Thanks!


Keep getting error: Type does not conform to protocol 'TargetType'

I'm using Moya 8.0.0-beta.6 with Swift 3.
I keep getting an error saying Type 'foo' does not conform to protocol 'TargetType'. It says that some things are missing, but I have them all. Last time, I had this problem, fixing it required changing the order of the variables. What is going on

Can you post the code for your TargetType?

extension GitHub: TargetType {
    var parameters: [String: Any]? {
        switch self {
        case .getUserInfo, .getOrganizationInfo, .getUserRepos, .getOrganizationRepos, .getRepoInfo:
            return nil
        case .searchUsers(let searchQuery), .searchRepos(let searchQuery), .searchCode(let searchQuery), .searchIssues(let searchQuery):
            return ["?q=": searchQuery]
        }
    }

    var sampleData: Data {
        switch self {
        case .getRepoInfo:
            return convertFileToString(forResource: "GetUserInfoExample", ofType: "json")
        case .getOrganizationInfo:
            return convertFileToString(forResource: "GetOrganizationInfoExample", ofType: "json")
        case .getUserRepos, .getOrganizationRepos:
            return convertFileToString(forResource: "GetOrgUserReposExample", ofType: "json")
        case .searchUsers:
            return convertFileToString(forResource: "SearchUsersExample", ofType: "json")
        case .searchCode:
            return convertFileToString(forResource: "SearchCodeExample", ofType: "json")
        case .searchRepos:
            return convertFileToString(forResource: "SearchReposExample", ofType: "json")
        case .searchRepos:
            return convertFileToString(forResource: "SearchIssuesExample", ofType: "json")
        case .getRepoInfo:
            return convertFileToString(forResource: "GetRepoInfoExample", ofType: "json")
        }

        var method: Moya.Method {
            switch self {
            case .getUserInfo, .getOrganizationInfo, .getUserRepos, .getOrganizationRepos, .getRepoInfo, .searchUsers, .searchRepos, .searchCode, .searchIssues:
                return .get
            }
        }

        var task: Task {
            switch self {
            case .getUserInfo, .getOrganizationInfo, .getUserRepos, .getOrganizationRepos, .getRepoInfo, .searchUsers, .searchRepos, .searchCode, .searchIssues:
                return .request
            }
        }

        var baseURL: URL { return URL(string: "https://api.github.com")! }
        var path: String {
            switch self {
            case .getUserInfo(let username):
                return "/users/\(username)"

            case .getUserRepos(let username):
                return "/users/\(username)/repos"

            case .getOrganizationInfo(let orgName):
                return "/orgs/\(orgName)"

            case .getOrganizationRepos(let orgName):
                return "/orgs/\(orgName)/repos"

            case .searchUsers:
                return "/search/users"

            case .searchRepos:
                return "/search/repositories"

            case .searchCode:
                return "/search/code"

            case .searchIssues:
                return "/search/issues"

            case .getRepoInfo(let owner, let repoName):
                return "/repos/\(owner)/\(repoName)"
            }
        }
    }
}

It is an issue with your braces. You put most of the properties inside of the sampleData computed property.


test: Swap k8s vsn between cilium/ubuntu{,-next} CI jobs

See commit messages.

test-me-please

Coverage increased (+0.005%) to 45.882% when pulling 42a802aac25ed1c5311be5c079d3457095230f19 on pr/brb/ci-swap-k8s-vsn into b8a3accbaffbfe54a016ce23b1c78a153d8a03f6 on master.

test-me-please

Applying istio-cilium.yaml timed out:
time="2020-04-06T19:39:52Z" level=error msg="Error executing command 'kubectl apply --force=false -f /home/jenkins/workspace/Cilium-PR-Ginkgo-Tests-Validated/k8s-1.18-gopath/src/github.com/cilium/cilium/test/k8sT/manifests/istio-cilium.yaml'" error="signal: killed"
cmd: "kubectl apply --force=false -f /home/jenkins/workspace/Cilium-PR-Ginkgo-Tests-Validated/k8s-1.18-gopath/src/github.com/cilium/cilium/test/k8sT/manifests/istio-cilium.yaml" exitCode: -1 duration: 30.002656564s stdout:

Re-running to see whether it's a flake.

test-me-please

test-me-please

This PR conflicts with https://github.com/cilium/cilium/pull/10880 has one needs to be rebase with the other. In this case will be documentation update.

@brb please rebase against master and update the docs added by https://github.com/cilium/cilium/pull/10880

test-me-please

test-me-please

Suite-k8s-1.11.K8sConformance Portmap Chaining Check connectivity-check compliance with portmap chaining failed:
/home/jenkins/workspace/Cilium-PR-Ginkgo-Tests-Validated/k8s-1.11-gopath/src/github.com/cilium/cilium/test/ginkgo-ext/scopes.go:430
connectivity-check pods are not ready after timeout
Expected
    <*errors.errorString | 0xc0002784e0>: {
        s: "Timeout reached: timed out waiting for pods with filter  to be ready",
    }
to be nil
/usr/local/go/src/reflect/value.go:460

Rerunning to see whether my PR does not introduced the failure.

test-me-please

Rerunning to see whether my PR does not introduced the failure.

@brb You did not introduce this. That test is broken on K8S 1.11 apparently. It was previously skipped on K8s 1.11 because it requires kube-proxy. See #11118, which enables kube-proxy on K8s 1.11 net-next, for example.

test-me-please

restart-ginkgo

@aanm It's going to fail again with the same test. That test is broken on K8s 1.11.

I haven't run it locally yet, but my gut feeling is that some api definitions might not be valid on k8s 1.11.

@brb is this PR still relevant now that we have an additional test job for kernel 4.19?

The k8s Service Topology feature is currently being reworked, so we can close this PR.


[files.mini]: Alternate modifid style.

Contributing guidelines

[X] I have read CONTRIBUTING.md
[X] I have read CODE_OF_CONDUCT.md

Module(s)
files.mini
Description
Hey!,
Love files.mini but i have a small style issue where a personal plugin and a theme function i have is conflicting with the border highlights constantly being set, is it possible to disable that feature or add an alternate like text color difference or an alternate way to visual a changes.
Thank you!
NOTE: and after some though just disabling might be the simplest thing it would be very hard to represent a deleted file.

Hi! Thanks for the suggestion!
Unfortunately, I don't think adding this kind of customization aligns well with 'mini.files' design.
Closing as not planned.

On this note, what kind of issues do you have because of this? Maybe I can you with this?
'mini.files' changes border highlighting via 'winhighlight' window option. It doesn't modify existing highlight groups, so I can't think of a way it might interfere with theming.

yeah that was the issue and it's fine, fine, it doesn't overwrite the whole winhighlight just highlights for floats :)


[Docs] Main contributing docs missing link to mesheryctl contributing.md

Current State:  In the github repo under the contributing mark down file there is no link to to the mesheryctl contributing markdown file.
Desired State:  A small section about contrtibuting to mesheryctl and a link to the mesheryctl contributing file

Contributor Resources

Meshery documentation site
Meshery documentation source
Instructions for contributing to documentation

Good one, @Maxgit3 👍

@Maxgit3, would you like to address this issue?

@leecalcote sure I can work on this

Thank you, @Maxgit3

@Maxgit3 Any progress here? 👀

@adithyaakrishna can I take up this issue, if yes, you can assign me

@Chadha93 I guess, @Maxgit3 is working on this, let's wait for 2 days. If it's not done, then you can go ahead :)

@adithyaakrishna sorry for the late reply.  I was going to work on it after figuring out the other issue.  if need be, you can assign it to another.

@Chadha93 You can go ahead with this 🚀

@adithyaakrishna I have made the required changes you may kindly check and provide feedback.


Update golden files, fix doc, and restore golden files check

The golden files check should not have been disabled.  Re-enabling.

Fixed by #373


examples/windowsize: A window size becomes unstable after restoring from maximized on Linux

Press R to set the window resizable
Press M to maximize the window
Press E to restore the window

Interestingly, the SetSize callback is called twice when maximizing/restoring the window. The first call has the new width and the old height, and the second call has both new values. Hmmmmm


Add retry & delay while creating dirs on worker node

While creating dirs on worker node, ci_local_storage role fails randomly. Re-running the same task fixes the issue.
As a temprory measure, we are adding retries to the task to fix the issue.
As a pull request owner and reviewers, I checked that:

[x] Appropriate testing is done and actually running

/retest-required


When the language is Chinese, there is a problem with the File Input and Theme controller display in the menu bar.

When the language is Chinese, there is a problem with the File Input and Theme controller display in the menu bar.

What is the problem? and why change them to lowercase?
Previously, the Chinese translation of the left navigation bar was English plus Chinese. File Input had this copywriting before, which would cause the copywriting of the left navigation bar to be only Chinese, not English plus Chinese. So refer to the Text input to modify it.

Is it required to include both Chinese and English names?
We had this issue with Japanese translation as well, the problem is, contributors translated all the strings to native language but most names were uninformative because component names are technical design/development terms. So then another contributor appended english strings to fix the issue but that doesn't look good in my opinion.
As a Chinese speaker, do you think all/some names for component names on sidebar is good enough or do they need Chinese translation?

For new developers, Chinese translation is also needed

But, is the English string needed as well? For all component names? I'm looking for a way to use one of them and I think if a translated component name is informative enough, it's safe to translate it. If the translation makes it confusing, we can only use English name. But using them both at the same time, next to each other, defeats the whole purpose of translation.

It should be a Chinese habit to use English strings. You can take a look at antd's component library (https://ant.design/components/overview-cn/). Of course, it is also possible not to use English strings. After all, people from other countries may also use daisyui. As a Chinese, it may take some time for me to adapt.

So if possible, in a new PR, please use either English or Chinese for component names.
Using both doesn't seem to be a good solution and of course having lower case and caption case keys for the same string is not a good idea.


Question: mongo $inc in arango?

What is the best way to $inc a counter in arango?
Want to build a auto-inc counter. In mongo, we can do
db.receipt.findAndModify({
    query: { name: "CompanyA" },
    update: { $inc: { receiptNum: 1 } },
    new: true,
})

Thanks

You can achieve this with an AQL query like the following:
FOR doc IN receipt
  FILTER doc.name == "CompanyA"
  // LIMIT 1 // optionally limit to a single record
  UPDATE doc WITH { receiptNum: doc.receiptNum + 1 } IN receipt
  RETURN NEW // optionally return updated records

AQL queries can be run using https://github.com/arangodb/arangojs#databasequery

Thanks for your quick reply.
Is it atomic?
2016年9月21日 上午1:22 於 "Simran"<EMAIL_ADDRESS>寫道：

You can achieve this with an AQL query like the following:
FOR doc IN receipt
FILTER doc.name == "CompanyA"
// LIMIT 1 // optionally limit to a single record
UPDATE doc WITH { receiptNum: doc.receiptNum + 1 } IN receipt
RETURN NEW // optionally return updated records
AQL queries can be run using https://github.com/arangodb/
arangojs#databasequery
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/arangodb/arangojs/issues/311#issuecomment-248370320,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAOd5BNXeN1GibWwdR44NpC_i0DTCrz3ks5qsBZYgaJpZM4KBuEw
.

AQL queries are fully ACID, except for multi-collection queries if the collections are sharded in a cluster.

Wow, thanks
2016年9月21日 上午7:51 於 "Simran"<EMAIL_ADDRESS>寫道：

AQL queries are fully ACID, except for multi-collection queries if the
collections are sharded in a cluster.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/arangodb/arangojs/issues/311#issuecomment-248470361,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAOd5M8Zzb0aUnYnmok2QbOMSlblMpfJks5qsHFxgaJpZM4KBuEw
.


[Docusaurus v3.4] build error with scaffolded create-docusaurus

Have you read the Contributing Guidelines on issues?

[X] I have read the Contributing Guidelines on issues.

Prerequisites

[X] I'm using the latest version of Docusaurus.
[X] I have tried the npm run clear or yarn clear command.
[X] I have tried rm -rf node_modules yarn.lock package-lock.json and re-installing packages.
[X] I have tried creating a repro with https://new.docusaurus.io.
[X] I have read the console error message carefully (if applicable).

Description
Installed the vanilla repo with npx create-docusaurus@latest deepmemory-documentation classic --typescript
Tried npm run build and it fails :
The server build bar finishes.
Then the client bar stops at:
● Client █████████████████████████ sealing (92%) asset processing
 TerserPlugin

first this repeated many times
[WARNING] {"moduleIdentifier":"doc-test\\node_modules\\babel-loader\\lib\\index.js??ruleSet[1].rules[5].use[0]!doc-test\\node_modules\\@docusaurus\\utils-common\\lib\\applyTrailingSlash.js","moduleName":"./node_modules/@docusaurus/utils-common/lib/applyTrailingSlash.js","message":"There are multiple modules with names that only differ in casing.\nThis can lead to unexpected behavior when compiling on a filesystem with other case-semantic.\nUse equal casing. Compare these module identifiers:\n*<EMAIL_ADDRESS>  Used by 1 module(s), i. e.\n<EMAIL_ADDRESS><EMAIL_ADDRESS>  Used by 1 module(s), i. e.\n<EMAIL_ADDRESS>
then:
client (webpack 5.93.0) compiled with 12 warnings
<w> [webpack.cache.PackFileCacheStrategy/webpack.FileSystemInfo] Resolving '../../url-loader/dist/cjs' in doc-test\node_modules\loader-runner\lib for build dependencies doesn't lead to expected result 'doc-test\node_modules\url-loader\dist\cjs.js', but to 'doc-test\node_modules\url-loader\dist\cjs.js' instead. Resolving dependencies are ignored for this path.
<w>  at resolve commonjs file ../../url-loader/dist/cjs (expected doc-test\node_modules\url-loader\dist\cjs.js)
<w>  at file dependencies doc-test\node_modules\loader-runner\lib\loadLoader.js
<w>  at file doc-test\node_modules\loader-runner\lib\loadLoader.js
<w>  at file dependencies doc-test\node_modules\loader-runner\lib\LoaderRunner.js
<w>  at file doc-test\node_modules\loader-runner\lib\LoaderRunner.js
<w>  at file dependencies doc-test\node_modules\webpack\lib\NormalModule.js
<w>  at file doc-test\node_modules\webpack\lib\NormalModule.js
<w>  at file dependencies doc-test\node_modules\webpack\lib\LoaderTargetPlugin.js
<w>  at file doc-test\node_modules\webpack\lib\LoaderTargetPlugin.js
<w>  at file dependencies doc-test\node_modules\webpack\lib\index.js
<w>  at file doc-test\node_modules\webpack\lib\index.js
<w>  at file dependencies<EMAIL_ADDRESS><w>  at file<EMAIL_ADDRESS><w>  at file dependencies<EMAIL_ADDRESS><w>  at file<EMAIL_ADDRESS><w>  at resolve commonjs<EMAIL_ADDRESS><w> [webpack.cache.PackFileCacheStrategy/webpack.FileSystemInfo] Resolving '../../@docusaurus/plugin-content-blog/lib/markdownLoader' in doc-test\node_modules\loader-runner\lib for build dependencies doesn't lead to expected result<EMAIL_ADDRESS>but to<EMAIL_ADDRESS>instead. Resolving dependencies are ignored for this path.
<w>  at resolve commonjs file ../../@docusaurus/plugin-content-blog/lib/markdownLoader (expected<EMAIL_ADDRESS><w>

Then this:
[ERROR] Error: Unable to build website for locale en.
    at tryToBuildLocale (doc-test\node_modules\@docusaurus\core\lib\commands\build.js:54:19)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async doc-test\node_modules\@docusaurus\core\lib\commands\build.js:65:9
    at async mapAsyncSequential (doc-test\node_modules\@docusaurus\utils\lib\jsUtils.js:20:24)
    at async Command.build (doc-test\node_modules\@docusaurus\core\lib\commands\build.js:63:5) {
  [cause]: Error: Docusaurus static site generation failed for 25 paths:
  - "/404.html"
  - "/blog"
  - "/blog/archive"
  - "/blog/first-blog-post"
  - "/blog/long-blog-post"
  - "/blog/mdx-blog-post"
  - "/blog/tags"
  - "/blog/tags/docusaurus"
  - "/blog/tags/facebook"
  - "/blog/tags/hello"
  - "/blog/tags/hola"
  - "/blog/welcome"
  - "/markdown-page"
  - "/docs/category/tutorial---basics"
  - "/docs/category/tutorial---extras"
  - "/docs/intro"
  - "/docs/tutorial-basics/congratulations"
  - "/docs/tutorial-basics/create-a-blog-post"
  - "/docs/tutorial-basics/create-a-document"
  - "/docs/tutorial-basics/create-a-page"
  - "/docs/tutorial-basics/deploy-your-site"
  - "/docs/tutorial-basics/markdown-features"
  - "/docs/tutorial-extras/manage-docs-versions"
  - "/docs/tutorial-extras/translate-your-site"
  - "/"
      at generateStaticFiles (doc-test\node_modules\@docusaurus\core\lib\ssg.js:85:15)
      at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
      ... 5 lines matching cause stack trace ...
      at async Command.build (doc-test\node_modules\@docusaurus\core\lib\commands\build.js:63:5) {
    [cause]: AggregateError
        at generateStaticFiles (doc-test\node_modules\@docusaurus\core\lib\ssg.js:86:20)
        at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
        at async executeSSG (doc-test\node_modules\@docusaurus\core\lib\commands\build.js:175:23)
        at async buildLocale (doc-test\node_modules\@docusaurus\core\lib\commands\build.js:135:31)
        at async tryToBuildLocale (doc-test\node_modules\@docusaurus\core\lib\commands\build.js:47:13)
        at async doc-test\node_modules\@docusaurus\core\lib\commands\build.js:65:9
        at async mapAsyncSequential (doc-test\node_modules\@docusaurus\utils\lib\jsUtils.js:20:24)
        at async Command.build (doc-test\node_modules\@docusaurus\core\lib\commands\build.js:63:5) {      [errors]: [

I've tried the prerequisite steps, it changes nothing and I get the sames errors.
Since this is an unaltered install, I really don't know what the problem is.
Reproducible demo
No response
Steps to reproduce

npx create-docusaurus@latest deepmemory-documentation classic --typescript
npm run build

Expected behavior
The build to succeed and be able to start learning how to use the lib.
Actual behavior
Error mentioned above occurs.
Your environment

Docusaurus version used:

"@docusaurus/core": "3.4.0",
"@docusaurus/preset-classic": "3.4.0",

Environment name and version (e.g. Chrome 89, Node.js 16.4): nodejs v20.10.0
Operating system and version (e.g. Ubuntu 20.04.2 LTS): windows 10

Self-service

[ ] I'd be willing to fix this bug myself.

Do you have a reproduction? We build scaffolded templates in our CI, and people make new websites everyday, so it must be something specific to your environment.

Hi @Josh-Cena I've tried creating one on https://docusaurus.new/ but since I've modified nothing, I'm not able to reproduce.
I've compared the package.json, docusaurus.config.ts,  babel.config.js in my project with the one on sandbox and they are similar.
Only difference is the node version.

It could also be specific to Windows. Unfortunately I don't think we have access to a Windows system and our Windows CI is working, so there's no way for us to investigate.

Thanks @Josh-Cena  for your quick reply.
I don't think I'll be able to solve it on my own (too junior), I'll close the issue.


default g2p to ovos-g2p-plugin-heuristic-arpa

the plugin is part of ovos-classifiers and usually present https://github.com/OpenVoiceOS/ovos-classifiers/pull/16

with https://github.com/OpenVoiceOS/ovos-plugin-manager/pull/169 this is now safe


how many gpus and which type of gpu are needed for pretraining, down-stream task finetuning?

specifically, gpu type,  gpu number and gpu training time for pretraining, detection training and segmentation training

Pretraining is done with 8 A6000 with 48G GPU memory
Detection is done with 8 A100 with 80G GPU memory which tooks around 1 day and 5 hours
Segmentation is done with 8 V100 with 32G GPU memory

Pretraining is done with 8 A6000 with 48G GPU memory Detection is done with 8 A100 with 80G GPU memory which tooks around 1 day and 5 hours Segmentation is done with 8 V100 with 32G GPU memory

thanks

Please check FastConvMAE for a fast implementation of pretraining and also benchmarking  different MIM paradigm.

Please check FastConvMAE for a fast implementation of pretraining and also benchmarking different MIM paradigm.

pls provide a link？  Google failed to find it

https://github.com/Alpha-VL/FastConvMAE. Code (one or two days) and paper (one week) will be released soon.


Updating docker file used by Azure Pipelines fails

@muradsater ❤
https://github.com/cognitedata/function-action-oidc/actions/runs/4495578740/jobs/7909342602

Solved by #56


Allow sentry-trace header in CORS restrictions

The way Sentry works in the browser is that it injects itself into the XMLHttpRequest function and adds a sentry-trace header there. This ends up causing calls to the Flagsmith api to fail. I think the proper way to do it is just allow the sentry flags to pass through
Optionally just allow ALL headers. I am not sure why there are header restrictions in the first place.

@tm1000 can you make sure that your PRs have been correctly formatted using black? This is why the unit tests are failing. I will have to look into why the Full E2E tests are failing.


The stage of Archive Analyzer takes a lot of time

Hello,
I have a question for you,why does the archive analyzer take so long?The File size is only 40M.
[The Log]
[INFO] Analysis Started
[INFO] Finished Archive Analyzer (1529 seconds)
[INFO] Finished File Name Analyzer (0 seconds)
[INFO] Finished Jar Analyzer (0 seconds)

Depending on what is inside the zip - I could see this happening. For instance, if you had the JavaScript analyzer enabled and there was a ton of JS inside the zip. Another one may be if the zip file itself contained several zip files. However, I agree that the archive analyzer could be improved.


proj: create project labels

https://github.com/DevOps-Boot/fastapi-k8s/issues/labels

I believe that's enough for a start :sweat_smile:
I tried to create relevant labels along different tagging "axes" like project management, issues state and nature, architectural and functional components... These should allow for nice and helpful filtering and grouping capabilities.
Please review and close if that's ok for y'all


Day 1 build system hotfixes

Summary

Various build system fixes from meeting 1.

Seems good to me. If we're going native linux/wsl don't we need the Pipfile back?

we could but for development personally I don't think python virtual environments add a lot of value
the situation where you need a different version of package for project A vs. B is pretty rare
also pipenv is very slow