Question regarding thread safety

Hi,
Thank you for this library.
I just wanted to ask if there are any guarantees regarding thread safety. More specifically, if I compile sqlite3 with thread safety enabled can I use the same sqlpp::sqlite3::connection object for requests from multiple threads? I suspect that the answer is no, but want to make sure anyway.

Hi,
I am not really an expert for sqlite3. The sqlpp::sqlite::connection wraps an sqlite3 connection. You will get the same thread guarantees. I assume that means that you should not share a connection between threads, but I am not sure.
Best,
Roland

@rbock thanks for quick reply.
SQLite has a so-called serialized mode that allows using "SQLite by multiple threads with no restriction". At a glance I found a couple of things in sqlpp (like this one) that could probably cause race conditions.
Sharing db connections between threads might not be a great idea, but I was just wondering if it's possible with this library.

Good point about the transaction state. I'll would need to think about that for a bit (which will take a while because of other constraints). My tendency would be to say you are right: Sharing is dangerous without explicit locks.

Thank you for answering my questions.

I'll would need to think about that for a bit (which will take a while because of other constraints)

Should I close the issue then or leave it as reminder?
I'll leave it so you can close it yourself if you want to.

Hi, what about MySQL/MariaDB ? I have Server in Qt which is highly multi-threaded. Each thread in requires its own connection to database. is this same in sqlpp11 ?

@SmitTheTux
That would be the safe option, yes.

Finally added a few word of documentation:
https://github.com/rbock/sqlpp11/blob/main/docs/Threads.md


fix: access public runtime config value directly from the top level.

[nuxt] [runtimeConfig] You are trying to access a public runtime config value ('form") directly from the top level. This currently works (for backward compatibility with Nut 2) but this compatibility layer will be removed in v3.5. Instead, you can update config ['form']' to config.public ['form']*

fixed in 8b986f


Inspector GUI very slow when showing lots of SharedInstances

The TryEnsureScript call inside fiSharedInstancePropertyEditor calls fiSharedInstanceUtility.GetSerializableType, which I'm finding is extremely slow, mostly because of the Assembly.GetTypesWithoutException call which allocates a lot of memory, and iterating through the many, many types which costs a lot of CPU time.
Is there some way to cache the results of e.g. fiRuntimeReflectionUtility.AllSimpleTypesDerivingFrom? I imagine the cache would need to be regenerated on any assembly reload but I'm not sure how to go about doing that. Any ideas?

Yep, caching this should be pretty easy. Rename AllSimpleTypesDerivingFrom to AllSimpleTypesDerivingFromInternal.
Then the cache is just
static Dictionary<Type, IEnumerable<Type>> cache = new Dictionary<Type, IEnumerable<Type>>();

public IEnumerable<Type> AllSimpleTypesDerivingFrom(Type t) {
    IEnumerable<Type> result;
    if (!cache.TryGetValue(t, out result)) {
        result = AllSimpleTypesDerivingFromInternal(t);
        cache[t] = result;
    }
    return result;
}

I'll try to get to this in a few days, but I'd be happy to accept a PR :)

Thanks - I've done something similar to this locally and it's a great speed improvement, but I still don't know what happens with reloading assemblies - if you think it's ok though, that's good enough for me :) I'll cobble a PR together if I get time (and will say I'm doing so here so you don't waste time doing it yourself!).

I've done this and tweaked it a bit; I'll submit a PR later


Using regular expressions in JSON result testing

Assume I test a JSON in the following way ...
And the JSON should be equal to:
"""
      {
          "success": true,
          "products": [
              {
                  "id":"1",
                  "name":"Milk"
              },
              {
                  "id":"2",
                  "name":"Bread"
              }
          ]
      }
      """

I'd like to test it using regular expressions like bellow
And the JSON should be equal to:
"""
      {
          "success": true,
          "products": [
              {
                  "id":"[\d]+",
                  "name":"[\w]+"
              },
              {
                  "id":"[\d]+",
                  "name":"[\w]+"
              }
          ]
      }
      """

Is it possible?

@bystro no, as you can see in the implementation:
https://github.com/rezzza/rest-api-behat-extension/blob/5ca62bf4fb68d6e7ef1ffd7acbd8600b3671baf8/src/Json/JsonContext.php#L177-L193
But... you can use one of the other expressions based steps:
https://github.com/rezzza/rest-api-behat-extension/blob/5ca62bf4fb68d6e7ef1ffd7acbd8600b3671baf8/src/Json/JsonContext.php#L195-L230
Those expressions are based on the powerful http://jmespath.org/
Also, for what you want to do if you only want to validate a json, json schema is your friend, you can write a json schema and validate the payload against it.
You can even use https://jsonschema.net/#/editor for example to quickly interpolate a payload into a first json schema draft that you will refine.

Closing this issue unless I didn't answered correctly to you question 👼


No details about from where external auth buttons appear

Tried to find where the google button came from. This details weren't specified in documentation. Then I found the source code of project that already have this button. And only then I got where the external authentication buttons coming from. https://github.com/AnkitSharma-007/ASPCore.GoogleAuth
@{
                var loginProviders = (await SignInManager.GetExternalAuthenticationSchemesAsync()).ToList();
                if (loginProviders.Count == 0)
                {

                            There are no external authentication services configured. See this article
                            for details on setting up this ASP.NET application to support logging in via external services.

                }
                else
                {
                    <form asp-action="ExternalLogin" asp-route-returnurl="@ViewData["ReturnUrl"]" method="post" class="form-horizontal">

                                @foreach (var provider in loginProviders)
                                {
                                    <button type="submit" class="btn btn-default" name="provider"<EMAIL_ADDRESS>title="Log in using your @provider.DisplayName<EMAIL_ADDRESS>                                }

                    </form>
                }
            }

Please, add this detail to documentation.
Thanks

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

ID: 6d19b295-f935-3755-4ca4-9d0717b21e75
Version Independent ID: 747a179e-90b7-de07-e270-82414963048a
Content: Google external login setup in ASP.NET Core
Content Source: aspnetcore/security/authentication/social/google-logins.md
Product: aspnet-core
Technology: aspnetcore-security
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

I just dealt with this myself, it looks like they moved the entirety of the account login code into its own package/library. You can get your project to scaffold it by executing the following -
dotnet tool install --global dotnet-aspnet-codegenerator
dotnet aspnet-codegenerator identity -dc WebApplication1.Data.ApplicationDbContext
and you change the namespace location to where your DbContext is.
I believe one or both of these nuget packages is required as well. Microsoft.VisualStudio.Web.CodeGeneration.Utils and Microsoft.VisualStudio.Web.CodeGeneration.Design
You can read more about it here - https://devblogs.microsoft.com/aspnet/aspnetcore-2-1-identity-ui/. Not sure if this is exactly what you were asking but I stumbled a bit with it myself.


100% coverage

This is the only line of code that is not covered yet: https://github.com/wearefractal/glob-stream/blob/1f26389f0408ed33201c17f4658b882242fde881/index.js#L105
The return true doesn't make much sense to me, should we throw an error instead? Then we can just add a test to cover that.

Or perhaps, return false if you'd prefer to fail silently. Though throwing would make more sense, IMO.

Or perhaps, return false if you'd prefer to fail silently. Though throwing would make more sense, IMO.

Should throw on unknown glob type or invalid glob on construction of the stream


Release 1.11 scope

[ ] make dependencies_v2 the default for Cassandra #1344
[x] make gRPC reporter agent-collector default https://github.com/jaegertracing/jaeger/issues/1218
[x] ES support TLS with credentials https://github.com/jaegertracing/jaeger/issues/1327

If the gRPC works well I would like to cut the release the next week. Let's say on Monday/Tuesday.

@tiffon what is the status of Jaeger UI? At the moment we depend on 1.0.0, I will update to 1.0.1 or do you want to release a new version with the latest fixes?

@pavolloffay We'll cut a UI release for 1.11. Will report back here, when it's done. :+1:

@pavolloffay We've released Jaeger UI @ v1.1.0.
LMK if you need anything else.

thanks @tiffon. I have updated it in the main repo.

@yurishkuro what should be documented regarding the v2 schema?

I think v2 now comes by default, and 1.10 changelog already provides migration steps


Run CI on GitHub Actions instead of CircleCI

Description
This completely removes the need for CircleCI and also swaps coveralls for Codecov, which has better integration with GitHub checks in PRs (once the Codecov app is installed).

@jgerigmeyer looks like Codecov can't display the checks here until it has coverage data for main to compare against. I think we can merge this one and check the Codecov checks run on other PRs

@jgerigmeyer looks like Codecov can't display the checks here until it has coverage data for main to compare against. I think we can merge this one and verify that the Codecov checks run on other PRs

@jerivas That's fine. Thinking through this more, I'm questioning whether Codecov is offering something we really need for these projects -- it might be just one more tool to maintain once we hand-off. What's the advantage of using that vs the existing nyc/coverage steps we have in package.json?

To me the main advantage is the annotations you get on the file diff on GitHub itself and the fact that you can browse the report easily. But yeah, other than that we can just run a separate step that checks the coverage numbers on CI

@jgerigmeyer hmm, for some reason my local installation is broken and cannot find the values defined in .env after a rebuild. Can you try it on your end? Basically anything that requires django is broken for me: yarn django:serve, yarn test:py, etc.

cannot find the values defined in .env after a rebuild

I think this was caused by 80e0f984ba6e2cb6fad8620154f6aca14980c35f. In oddbooks we don't need it because django-environ reads the file directly from the volume whenever Django is initialized. In Metecho the env helper function expects the vars to be actually present in the environment. I'll update Metecho to use django-environ to get rid of our own implementation


Swap pip for uv

Is there an existing issue for this?

[X] I have searched existing issues, it hasn't been reported yet

Use case description
uv was just released (from the makers of ruff), and is substantially faster than pip, pip-tools, and virtualenv. This seems desirable to speed up serverless builds.
Proposed solution (optional)

Announcing uv: an extremely fast Python package installer and resolver, written in Rust.
uv is designed as a drop-in alternative to pip, pip-tools, and virtualenv.
With a warm cache, uv installs are near-instant. Here, it's > 75x faster than pip and pip-tools.

— Source
The “drop-in” claim seems to suggest you might be able to just change the binary i.e. re-alias pip to uv pip

Hey @lmmx - how do you imagine support for uv on the plugin level? As the plugin uses pip that is installed locally, I think "just" aliasing on your own should be enough, but maybe I'm missing some specific use cases. Please let me know what do you think 🙇

uv doesn't (yet) support the pip install -t flag with which you can specify a target directory.
This flag is used by this project to install the requirements into a cache directory or if you've disabled the cache to have the requirements under .serverless/requirements.
Zer0x00 ➜ /project $ uv pip install -t /project/.serverless/requirements -r /project/.serverless/requirements/requirements.txt

error: unexpected argument '-t' found

  tip: to pass '-t' as a value, use '-- -t'

Usage: uv pip install [OPTIONS] <PACKAGE|--requirement <REQUIREMENT>|--editable <EDITABLE>>

For more information, try '--help'.

Thanks for sharing @Zer0x00 🙇 Unless there will be a very high demand for uv-specific support, I think the plugin shouldn't include anything specific for uv if the general plan for it is to be a drop-in replacement for pip with full compatibility.
If anyone is interested in having support for uv sooner, please vote on the issue with 👍 and/or write a comment below 🙇

This is something I really would like to see supported. I have recently transitioned from using pip to uv pip with some quite amazing speed ups across the board, and it would be great to have support for uv (or perhaps/eventually switch to uv entirely).
I have had success patching serverless-python-requirements manually with the following patch (be warned though, use at your own risk):
diff --git a/node_modules/serverless-python-requirements/lib/pip.js b/node_modules/serverless-python-requirements/lib/pip.js
index 40140d3..bfc75ab 100644
--- a/node_modules/serverless-python-requirements/lib/pip.js
+++ b/node_modules/serverless-python-requirements/lib/pip.js
@@ -147,7 +147,7 @@ async function installRequirements(targetFolder, pluginInstance, funcOptions) {

   try {
     const dockerCmd = [];
-    const pipCmd = [options.pythonBin, '-m', 'pip', 'install'];
+    const pipCmd = ['uv', 'pip', 'install', '--verbose', '--system'];

     if (
       Array.isArray(options.pipCmdExtraArgs) &&
@@ -159,7 +159,10 @@ async function installRequirements(targetFolder, pluginInstance, funcOptions) {
       });
     }

-    const pipCmds = [pipCmd];
+    const pipCmds = [
+      [options.pythonBin, '-m', 'pip', 'install', 'uv'],
+      pipCmd
+    ];
     const postCmds = [];
     // Check if we're using the legacy --cache-dir command...
     if (options.pipCmdExtraArgs.indexOf('--cache-dir') > -1) {
@@ -196,9 +199,9 @@ async function installRequirements(targetFolder, pluginInstance, funcOptions) {
     if (!options.dockerizePip) {
       // Push our local OS-specific paths for requirements and target directory
       pipCmd.push(
-        '-t',
+        '--target',
         dockerPathForWin(targetFolder),
-        '-r',
+        '--requirement',
         dockerPathForWin(targetRequirementsTxt)
       );
       // If we want a download cache...
@@ -226,7 +229,7 @@ async function installRequirements(targetFolder, pluginInstance, funcOptions) {
     // If we are dockerizing pip
     if (options.dockerizePip) {
       // Push docker-specific paths for requirements and target directory
-      pipCmd.push('-t', '/var/task/', '-r', '/var/task/requirements.txt');
+      pipCmd.push('--target', '/var/task/', '--requirement', '/var/task/requirements.txt');

       // Build docker image if required
       let dockerImage;

Just a quick note

shouldn't include anything specific for uv if the general plan for it is to be a drop-in replacement for pip with full compatibility.

This was the idea for the initial release of uv, but now it supports lots of new functionality beyond pip. I'm not sure if that changes anything here :)

uv added support for --target back in April :slightly_smiling_face: :tada:

https://github.com/astral-sh/uv/issues/1517

It is also now able to replace Pipenv:

https://github.com/astral-sh/uv/issues/1465

I would imagine that this could be put in as a switch the same way there is usePoetry for the poetry package manager


FAQs Page for .net

https://www.figma.com/file/buzGz2v44DfkUQf112KlcO/Landing-Page-Complete-Design?node-id=1776%3A0
Content to be pulled in from the current FAQ page and repurposed in the new design

Messaged you with a couple of small visual tweaks (dividers, rounded corners, hover states). Rest looks good

Looks good. closing


Event is not shown, if game over.

If an event causes the game to be lost, the event itself is not shown. Therefore, it is not apparent to the user, why the game is over.

Fixed by aa2b47dbd7f02ab9f027b88cd54ae91d719f1380.


Feat client side render member

Fixes #151

@iropal sounds like you are facing #164 try cleaning your local storage


why i first run train.py then run yolo.py .  having an issue

I first run train.py then model.save  '.h' files.
But when I test it using yolo.py and change the load h5 file path.
But it errors.... why

I dont know if I run train.py and save h5 file .. how to test the weights

Use your trained weights and your own class file in yolo.py.

actually I just re-run the voc experiments... so the class file I did not change.. I just change the weights path.. But it error runs : shape not correct or not yolo_head ... lots of strange hints!
Did the model in train.py is same to the model in yolo.py??

Oh, the training model adds another layer. So don't run model.save, try model.save_weights.

so if i run the train.py and save weights. but I don't have model
So i should create a model in the yolo.py ... then load the weights.
But I don't know which model is correct?
could I just load yolo.h5 .. then I can get a model and weights, the I re-put my weights in the model by name==True??

so if i run the train.py and save weights. but I don't have model
So i should create a model in the yolo.py ... then load the weights.
But I don't know which model is correct?
could I just load yolo.h5 .. then I can get a model and weights, the I re-put my weights in the model by name==True??

so if i run the train.py and save weights. but I don't have model
So i should create a model in the yolo.py ... then load the weights.
But I don't know which model is correct?
could I just load yolo.h5 .. then I can get a model and weights, the I re-put my weights in the model by name==True??

Just use weights.h5,  yolo.py can handle this after 0f3567c.

if i just use weights...  error information::
File "yolo.py", line 58, in generate
self.yolo_model = load_model(model_path, compile=False)
File "/usr/local/lib/python3.6/dist-packages/keras/engine/saving.py", line 259, in load_model
raise ValueError('No model found in config file.')
ValueError: No model found in config file.
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/framework/ops.py", line 1567, in _create_c_op
c_op = c_api.TF_FinishOperation(op_desc)
tensorflow.python.framework.errors_impl.InvalidArgumentError: Dimension 0 in both shapes must be equal, but are 1 and 75. Shapes are [1,1,1024,255] and [75,1024,1,1]. for 'Assign_360' (op: 'Assign') with input shapes: [1,1,1024,255], [75,1024,1,1].
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "yolo.py", line 217, in
detect_img(YOLO())
File "yolo.py", line 33, in init
self.boxes, self.scores, self.classes = self.generate()
File "yolo.py", line 62, in generate
self.yolo_model.load_weights(self.model_path) # make sure model, anchors and classes match
File "/usr/local/lib/python3.6/dist-packages/keras/engine/network.py", line 1180, in load_weights
f, self.layers, reshape=reshape)
File "/usr/local/lib/python3.6/dist-packages/keras/engine/saving.py", line 929, in load_weights_from_hdf5_group
K.batch_set_value(weight_value_tuples)
File "/usr/local/lib/python3.6/dist-packages/keras/backend/tensorflow_backend.py", line 2430, in batch_set_value
assign_op = x.assign(assign_placeholder)
File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/ops/variables.py", line 615, in assign
return state_ops.assign(self._variable, value, use_locking=use_locking)
File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/ops/state_ops.py", line 283, in assign
validate_shape=validate_shape)
File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/ops/gen_state_ops.py", line 60, in assign
use_locking=use_locking, name=name)
File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/framework/op_def_library.py", line 787, in _apply_op_helper
op_def=op_def)
File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/framework/ops.py", line 3392, in create_op
op_def=op_def)
File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/framework/ops.py", line 1734, in init
control_input_ops)
File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/framework/ops.py", line 1570, in _create_c_op
raise ValueError(str(e))
ValueError: Dimension 0 in both shapes must be equal, but are 1 and 75. Shapes are [1,1,1024,255] and [75,1024,1,1]. for 'Assign_360' (op: 'Assign') with input shapes: [1,1,1024,255], [75,1024,1,1].
root@ce92d39b48eb:~/workspace/3dbc/yolov3/keras-yolo3#
conv2d_30 (Conv2D)              (None, None, None, 2 131072
i am confused because i just use the weights trained by train.py but the shape is wrong

Maybe you forget change the classpath
self.classes_path = 'model_data/voc_classes.txt'

make sure you classes file like voc_calsses.txt without "\n" or  blank lines

you can try changing save_weights_only=False in your training file and than load the entire model to the yolo.py when testing the model.

@luvWY 你好，请问你的问题解决了吗？
我现在也是报错in load_model raise ValueError('No model found in config file.') ValueError: No model found in config file.

@luvWY 你好，请问你的问题解决了吗？
我现在也是报错in load_model raise ValueError('No model found in config file.') ValueError: No model found in config file.

@qqwweee please help me to solve Error:list index out of range error while using train.py

@luvWY 你好，请问你的问题解决了吗？
我现在也是报错in load_model raise ValueError('No model found in config file.') ValueError: No model found in config file.

Did you solve this issue?i am getting the same error

@luvWY 你好，请问你的问题解决了吗？
我现在也是报错in load_model raise ValueError('No model found in config file.') ValueError: No model found in config file.

您好，请问您解决这个问题了吗


Grammatical Error

Under "Publish and copy over the app" selection:
Review this sentence: "If the app is run locally"

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

ID: dcdd2c35-bf2b-2673-d150-303a8aaa54d2
Version Independent ID: 815fe034-c9fc-aee3-a8b5-f2e860b30302
Content: Host ASP.NET Core on Linux with Nginx
Content Source: aspnetcore/host-and-deploy/linux-nginx.md
Product: aspnet-core
Technology: aspnetcore-hostdeploy
GitHub Login: @guardrex
Microsoft Alias: riande

It's correct.


Edit Actions for This Policy Event crash with  Couldn't find MiqPolicy without an ID [miq_policy/x_button]

Clicked Control
Choose an event (say, Host Maintenance Exit Request").
Click Configuration - > Edit Actions for This Policy Event
get error screen

this is with <IP_ADDRESS>.20151201120956_653c0d4

@gmcculloug Please investigate

@itamarh if this issue is no longer relevant can you please close.

no idea if still relevant. closing.


Socket reconnection on retries of JoinGroup, SyncGroup, LeaveGroup

Today, if JoinGroup sees a socket disconnection, it will retry again without reconnecting the connection which results in subsequent retries all to fail.
This change will ensure that we attempt to reconnect the connection if see a disconnected connection while selecting a unique RequestHandler to use for this request.
Alternative approaches that can be considered:

Reconnect automatically in RequestHandler.request before calling self.shared.requests.put
Reconnect automatically in BrokerConnection.request if required

Codecov Report

Merging #939 into master will decrease coverage by 7.95%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master     #939      +/-   ##
==========================================
- Coverage   83.32%   75.36%   -7.96%
==========================================
  Files          36       36
  Lines        3807     3811       +4
  Branches      562      563       +1
==========================================
- Hits         3172     2872     -300
- Misses        489      782     +293
- Partials      146      157      +11

Impacted Files
Coverage Δ

pykafka/managedbalancedconsumer.py
45.03% <0%> (-39.08%)
:arrow_down:

pykafka/broker.py
65.53% <0%> (-28.11%)
:arrow_down:

pykafka/rdkafka/helpers.py
20% <0%> (-73.34%)
:arrow_down:

pykafka/utils/compat.py
35.13% <0%> (-50.01%)
:arrow_down:

pykafka/connection.py
64.35% <0%> (-20.8%)
:arrow_down:

pykafka/handlers.py
73.52% <0%> (-18.39%)
:arrow_down:

pykafka/topic.py
71.13% <0%> (-10.31%)
:arrow_down:

pykafka/simpleconsumer.py
75.22% <0%> (-9.83%)
:arrow_down:

... and 11 more

Continue to review full report at Codecov.

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

Thanks for the contribution. I think your first alternate suggestion of putting this functionality in RequestHandler might be a better way to maintain encapsulation.


ListBox value cannot be nullable anymore

Listbox previously was able to handle be bound to a nullable property however with the new update it throws an exception.
<Listbox Context="paymenType" @bind-Value="_createPayment.SelectedPaymentType">
Microsoft.AspNetCore.Components.WebAssembly.Rendering.WebAssemblyRenderer[100]
Unhandled exception rendering component: Object reference not set to an instance of an object.
System.NullReferenceException: Object reference not set to an instance of an object.
at Ignis.Components.HeadlessUI.ListboxOption1[[Shared.PaymentType, Shared, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null]].<.ctor>b__24_0()    at Ignis.Components.AttributeCollection.GetEnumerator()+MoveNext()    at Microsoft.AspNetCore.Components.Rendering.RenderTreeBuilder.AddMultipleAttributes(Int32 sequence, IEnumerable1 attributes)
at Ignis.Components.HeadlessUI.ListboxOption`1[[Shared.PaymentType, Shared, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null]].BuildRenderTree(RenderTreeBuilder builder)
at Ignis.Components.IgnisComponentBase.<.ctor>b__10_0(RenderTreeBuilder builder)
at Microsoft.AspNetCore.Components.Rendering.ComponentState.RenderIntoBatch(RenderBatchBuilder batchBuilder, RenderFragment renderFragment, Exception& renderFragmentException)
works fine on 0.13.1

Thank you for creating the issue, I will have a look as soon as possible!

Just to give you a little but more clarification, SelectedPaymentType would be a nullable Enum property. Now it works on the Listbox but in the ListboxOption it will throw an error. This worked fine prior and still works if i make the property not nullable and set a default value.
                    <Listbox Context="interval" @bind-Value="_createPayment.SelectedPaymentType">

                            <ListboxButton class="relative w-full ring-1 ring-inset ring-gray-300 rounded-lg bg-white py-1 pl-3 pr-10 text-left focus:outline-none focus-visible:border-emerald-600 focus-visible:ring-2 focus-visible:ring-white focus-visible:ring-opacity-75 focus-visible:ring-offset-2 focus-visible:ring-offset-emerald-600 sm:text-sm">

                                    @if (_createPayment.SelectedPaymentType is null)
                                    {
                                        Select a payment type
                                    }
                                    else
                                    {

                                            <CreditCardIcon class="h-5 w-5 text-emerald-600 mr-2" />
                                            <p<EMAIL_ADDRESS>

                                    }

                                    <ChevronUpDownIcon class="h-5 w-5 text-gray-400" />

                            </ListboxButton>

Hi, so I am struggling to reproduce this. Can you please extend the above example with the ListboxOption code?

I will close this issue for now since I cannot reproduce it with the given details. Feel free to comment with more details if the issue still persists.

Sorry i was very busy. I will produce you some code over the weekend that showcases the issue.


Play Routes plugin: Tool chain should live in external plugin

Corresponding tool chain migrated to external plugin (this could potentially involve using the worker API instead)
Simplify implementation as needed
Custom task type in external plugin does not call off to Gradle core anymore

Implementation:

https://github.com/gradle/play/tree/master/src/main/java/com/lightbend/play/tools/routes


Add release bundles V2 indexing configuration

Is your feature request related to a problem? Please describe.
Release bundles (V2) indexing configuration cannot be adjusted using this provider.
Describe the solution you'd like
Add support for https://jfrog.com/help/r/xray-rest-apis/add-release-bundles-v2-indexing-configuration
Additional context
Builds (https://registry.terraform.io/providers/jfrog/xray/latest/docs/resources/binary_manager_builds) and repositories (https://registry.terraform.io/providers/jfrog/xray/latest/docs/resources/binary_manager_repos) are already supported but release bundles are missing:

@Sam13 Thanks for the suggestion. I'll add this to our plan.


sprintfix: 创建订阅报错 "can not get ip in obj type host info"

Codecov Report

Merging #1288 (04e8023) into V2.2.32-rc (0368384) will increase coverage by 0.01%.
The diff coverage is n/a.

@@              Coverage Diff               @@
##           V2.2.32-rc    #1288      +/-   ##
==============================================
+ Coverage       73.32%   73.33%   +0.01%
==============================================
  Files             360      360
  Lines           24379    24379
==============================================
+ Hits            17875    17879       +4
+ Misses           6504     6500       -4

Impacted Files
Coverage Δ

apps/backend/subscription/steps/plugin.py
54.97% <ø> (+0.62%)
:arrow_up:

apps/node_man/handlers/job.py
85.19% <0.00%> (+0.30%)
:arrow_up:

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


OEP5 Enhancements

I am proposing, based off the conversation from #46, that we eliminate the takeOwnership method from OEP5, as well as introduce a new method tokensOf.
I am also including an optional method, clearApproved. This is to further mitigate any issues that can arise from the approve function, like the one reported in https://github.com/ONT-Avocados/python-template/issues/20.
Based off input from @tanZiWen I have also included the methods allowance and approvalForAll.

@Honglei-Cong @tanZiWen @laizy

@Honglei-Cong @tanZiWen @laizy @javajoker any other comments/questions/concerns for this?

it need to be solved the conflict?

@tanZiWen what conflict?

@laizy, due to our conversation in this PR and the one on the depth issue here I have decided to modify this proposal to make tokensOf optional. The DynamicList object I defined in https://github.com/TowerBuilders/Ontology-Advanced-Storage-Objects will allow over 7 million tokenIds to be stored.

@tanZiWen @Honglei-Cong @laizy we have a contract using this on the testnet at https://explorer.ont.io/contract/other/a47222204212ef759df954f1e1b156528098153d/10/1/testnet
We'll be launching one on the mainnet soon

LGTM

@tanZiWen @Honglei-Cong @laizy there is now a contract running on the mainnet using this: https://explorer.ont.io/contract/other/6585d17294994a0b16fb06b74bed8ae4051ea7c5/10/1

Anything else to fix before merging this in?

I've renamed tokenMetadata to tokenURI to keep it consistent with ERC721 and NEP11.

I've also renamed allowance to getApproved to make it consistent with ERC721 - and because allowance did not describe what the method was actually doing.

@tanZiWen @Honglei-Cong @laizy would it make sense to introduce an additional method properties(tokenId) that returns a serialized NVM object with a given NFT's properties? Similar to tokenURI, but these would be immutable properties, so they wouldn't be stored off-chain.

This PR looks good for me.
But there's interface changes, create a new OEP?

@Honglei-Cong I don't think there's really anyone using the current interface - and only optional methods were changed.

@javajoker any other comments on this?

@tanZiWen @Honglei-Cong

I removed tokenURI, as it can be accomplished with just properties.


Indicator: Saving target periods throwing error

Current behavior
Create target period is throwing 'Invalid form' error

To Reproduce
Steps to reproduce the behavior:

Go to Indicator list
Click on an Indicator and go to 'Targets' tab
Fill in the information in the form
Add Target frequency and periods with targets
Save the form
See error 'Invalid form'

Expected behavior
The target periods can be saved without any erro

Is this issue still open ?


CodeActionsProvider no longer receives the full selection range

VSCode Version: Code - Insiders 1.9.0-insider (3af0e9d6b79145011a7be3970c1d4b83d69b0443, 2017-01-27T10:46:43.625Z)
OS Version: Windows_NT ia32 10.0.14986

The behavior of this code changed from 1.8.0 to the current Insider's build: the range received by provideCodeActions is no longer the full selection.
'use strict';
import * as vscode from 'vscode';

export function activate(context: vscode.ExtensionContext) {
    vscode.languages.registerCodeActionsProvider('haxe', {
        provideCodeActions(doc, range, message) {
            return [{
                title: doc.getText(range),
                command: ""
            }];
        }
    });
}

In the current Insider's build:

In VSCode 1.8.0:

I stumbled over this when trying to implement an "Extract variable" feature for the Haxe extension, and noticed different behavior in 1.8.0 and the Insider's build.
In case this is an intentional change and not a bug: In theory, this could be worked around by just checking the selection of the active editor. However, it's not so simple if you want to implement this in a language server, since there doesn't seem to be any way to check what the current selection is (?).

Yeah, there was a change that we should fallback to the current selection when there is not squiggle. We added asking for code actions for the word under cursor and forget to check if the current selection exceeds that word.


goreleaser failing with [FATAL] when it shoud be disabled and there is no GO code in project

Is there an existing issue for this?

[X] I have searched the existing issues

Are you using the latest Super-linter version available?

[X] I am using the latest Super-linter version.

[X] I can reproduce the issue running Super-linter using complete version identifier (example: vX.Y.Z), and not just with a partial one (example: vX)

[X] I am using the super-linter/super-linter action or container image, and not the deprecated github/super-linter action or container image.

Are you resonably sure that it's a Super-linter issue, and not an issue related to a tool that Super-linter runs?

[X] I think that this is a Super-linter issue.

 Upgrade from super-linter@v4 to super-linter@v6 on a python project

### Anything else?

no

Hi! It's not Go releaser. The error is about the default branch missing:
2024-07-18 23:46:53 [FATAL]   Neither master, nor origin/master exist in /github/workspace

Well the first issue then is there is no indication which linter produced this error if it's not goreleaser, I logged it as a goreleaser issue as that was the last linter that identified itself
The second issue is the master branch exists, so I don't understand why it thinks it's missing, this error makes no sense at all, this error provides no link or clue to further information or what's required to resolve it.

When I google the error I don't get any further explanation.
I also referred to the upgrade guide, which also had no mention of this error or how to deal with it.

I resolved it for now by rolling back to v5 (which is still newer than v4 that I was on).
Until I know how to resolve this issue I can't update to v6.
Sorry this should not be closed as it's not resolved, please re-open

Hi! Please open a new issue by completing all the fields, including your full Super-linter configuration, otherwise it's hard to understand what is going on.


Fixed #24245 -- Added introspection for database defaults.

Needed for tests for migrations handling of database defaults.

Also looks like adding default introspection support for sqlite is straightforward.

Claude, I added the database feature and introspection for sqlite as you suggested.


Fix bug where execFileSync returned Uint8Array instead of String

I'm using cmder (/ConEmu) on Windows 10, and child_process.execFileSync is returning an Uint8Array instead of a String. Resulting in a fixed console width of 80 columns; the fallback width, as the array doesn't have a trim method.
Fixed this by forcing the response to become a string, before calling the trim method.

Can you report this on https://github.com/nodejs/node and add a comment with a link to the issue?

Is it really an issue of node? Or just of the used shell?

We request utf8 from a Node.js API and don’t get what we requested, so it’s an issue with Node.js.

Closing for now. Happy to reopen when there's a Node.js issue created so it can be fixed upstream at some point.


LLVM command line options integrations (issue #3327)

Implementations of RFC 64.
Now developers can use --llvm-args with a comma-separated string to pass LLVM-specific arguments to debug build of ponyc.

@mshockwave we recently made a change to the CircleCI setup for the ponyc repo and it appears that at least some forked repos do not get PRs run now. Yours being one of them. I'm merging this now. After that, can you, if possible, try deleting your fork and recreating a new one? That might fix the issue.
Also, can you add release notes for this to https://github.com/ponylang/ponyc/issues/3314

we recently made a change to the CircleCI setup for the ponyc repo and it appears that at least some forked repos do not get PRs run now. Yours being one of them. Once this is merged, can you, if possible, try deleting your fork and then fork again?
Sure, I'll delete my fork after this PR is merged

Also, can you add release notes for this to #3314
Will do

Sorry for the delay on merging this!


Please provide the "item-height" attribute.

ionic.bundle.js:25510 Error: collection-repeat tried to compute the height of repeated elements "item in allContacts | searchContacts: searchCon", but was unable to. Please provide the "item-height" attribute.
Can anyone tell me what is the issue here?

This issue was selected to be spring-cleaned by a script. More information can be found here.
If anyone thinks this was a mistake and this issue should be reopened, please leave a message below explaining why. Before doing so, please consider reading the CONTRIBUTING file.


FR: setup option to use default nord palette

Hi @rmehri01, I've been rocking this theme for quite a while and it is lovely, so thanks for your hard work!
I was wondering if it would be possible to add an option to override highlight groups to use the default nord color palette without having to do it manually. I think it would be great to just have a quick flag to switch between your implementation and the default one without having to add or remove a lot of override options.

Hi, thanks for the kind words, I'm glad you like it! I'm not sure how I feel about having a default Nord palette because its not really the purpose of this project and has been done by https://github.com/shaunsingh/nord.nvim. I think it also might not be too much work to use custom_colors to define your own palette if you really want to, what are your thoughts?

The only reason for having it all in this plugin was because this allows for much more customisation options, but since I opened this issue I played around with custom_colors to achieve that and it wasn’t a hard work at all.
I’m closing the issue, thank you again!


check if graph G is empty or contains no nodes

test output:
python test_osmnx.py

Configured osmnx
test debug
test info
test warning
test error

Coverage decreased (-0.5%) to 94.156% when pulling 0f284ae78ccbf732f5550f96d0deebe287dab115 on Wario7:enhancement-get_nearest_neighbor into a28f367ed36a912415be223c5fa9289eae107894 on gboeing:master.

resolves #97
@Wario7 looks good, thanks!


[React] Add thumbnail mode feature

It allows to enable/disable the video subscription for remote feeds.

Replaced by #351.


integrate FileService for app-sync.js

Related to #420
~Depends on #433, and on #425, and needs them to be merged first.~
Ready for review!

Thanks for this! I'm looking to have it fully reviewed this week, sorry it's taken a little while.
One of the tests is failing, please could you have a look?

Thanks for this! I'm looking to have it fully reviewed this week, sorry it's taken a little while. One of the tests is failing, please could you have a look?

Hey, thanks!
The test is flaky. Manual testing revealed that it's because we're not handling the database state correctly across the app. When all tests run in parallel across all the services, something goes wrong in the accountDb, and makes the test fail because the database couldn't be updated. Not only this test is affected, but other tests in app-sync.test.js as well. If I only run the app-sync.test.js tests, it never fails. It's only when they are run in parallel with the whole test suite. I haven't been able to pinpoint the exact problem.
The current usage (one user per server) may make this a non-issue in production, but I think on a multi user system, with lots of calls, the database will be an issue.

Looking at this now
Just out of interest...
@coderabbitai review

@coderabbitai review


Goflow fix failed to parse ip

Description of Changes
A user has the following error:
{
  "level": "error",
  "ts": "2021-06-23T15:13:23.145-0500",
  "msg": "Failed to parse netflow message%!(EXTRA zapcore.Field={error 26 0  error converting DstAddr to string: cannot convert byte slice to ip address, expected length of 4 or 16 got 0})",
  "operator_id": "$.dec6c632-c843-467f-a262-75fddb2f7a8e.goflow_input",
  "operator_type": "goflow_input"
}

When an ip address field is length zero, we get this error when instead we should be ignoring the field. I was able to replicate this error by adding the test cases in this PR. The issue is resolved by skipping ip address parsing if the value is empty.

Log correct key that is failing to convert to an IP
If ip address field is empty, skip parsing it instead of returning an error
Add test cases for empty ip address fields
Add test case for invalid ip address length (malformed address)

Please check that the PR fulfills these requirements

[x] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)
[x] Add a changelog entry (for non-trivial bug fixes / features)
[ ] CI passes

Log Files
Logs / Second
CPU Avg (%)
CPU Avg Δ (%)
Memory Avg (MB)
Memory Avg Δ (MB)

1
1000
1.4827853
+0.034441948
127.25377
-1.6217651

1
5000
5.034576
+0.32750034
139.26993
+5.1365814

1
10000
9.138108
-0.82742786
143.97481
-2.7619934

1
50000
46.2082
-2.0851402
176.06493
+3.6578674

1
100000
103.89996
+5.6227493
233.84294
-3.1095123

10
100
1.9482563
+0.17234683
131.39049
-2.4598694

10
500
5.896599
+0.41374874
139.6805
-0.43089294

10
1000
11.500098
-0.70541763
146.74165
-2.9137878

10
5000
50.481827
-5.9502563
177.28705
-0.18843079

10
10000
108.28174
-10.375099
228.3816
-5.027603

Log Files
Logs / Second
CPU Avg (%)
CPU Avg Δ (%)
Memory Avg (MB)
Memory Avg Δ (MB)

1
1000
1.4310942
+0.12073064
127.726295
-0.7308731

1
5000
4.7069917
-0.43104744
137.61409
+0.029769897

1
10000
10.672364
+1.344574
145.40652
-1.2831268

1
50000
47.449066
+1.3091011
177.3684
+3.1423645

1
100000
89.052216
-0.9149475
227.33742
+0.35560608

10
100
1.8793962
+0.17245066
133.83728
+0.30818176

10
500
5.569044
-0.06899023
141.56883
-0.8162689

10
1000
11.534627
+0.77590847
146.86166
-1.0040436

10
5000
53.6561
+2.799549
176.32678
+1.8263855

10
10000
99.07312
+1.7139511
220.7003
+3.3544006


replace deprecated -target with -release jdk version

I get this warning
compileGatlingScala
[Warn] : -target is deprecated: Use -release instead to compile against the correct platform API.

I hope this fixes it.
See
https://docs.scala-lang.org/overviews/compiler-options/index.html
https://docs.gradle.org/current/userguide/upgrading_version_7.html#scala_compilation_target
https://docs.gradle.org/8.1.1/userguide/scala_plugin.html#sec:scala_target

Hi,
Instead, could you please remove this option? Java 8 is the default anyway and your change might break users enforcing an older Scala version.
Also, could you please sign our CLA?

Also, could you please sign our CLA?

@MahatmaFatalError up

@MahatmaFatalError gentle ping

@MahatmaFatalError gentle ping

I signed ✍

Perfect, thanks!

Just released <IP_ADDRESS>


Extend with WM_SETTEXT monitoring

Extend the project to also monitor changes to the window application name shown in the titlebar.
Doc:

https://learn.microsoft.com/en-us/windows/win32/api/winuser/nf-winuser-setwindowtextw

WIP branch: https://github.com/forderud/IconMonitor/tree/win-title


Add guide for migrating in OpenStack

Summary
Document how to migrate from MitreID to Keycloak for OpenStack providers

Related issue :

@vardizzo-lab please review this PR.


Gap in Functionality

There appears to be a tiny gap between your QBWC and quickbooks_api gem.
Nether seems to provide the quickbooks xml wrapper  IE
<QBXML><QBXMLMsgsRq onError="stopOnError">  .....LOTS OF QBXML........  </QBXMLMsgsRq></QBXML>

It's not exactly a difficult workaround but I just wanted to see if this should be implemented in the defaultservant.rb in QBWC place else.
Below is the code I am currently using.  But I don't feel like it is ideal. (maybe i just missed something)
  def wrap_in_version(xml_rq)
    if QBWC.quickbooks_type == :qbpos
      %Q( <?qbposxml version="#{QBWC.quickbooks_min_version}"?> <QBPOSXML><QBPOSXMLMsgsRq onError="stopOnError"> ) + xml_rq + %q( </QBPOSXMLMsgsRq></QBPOSXML> )
    else
      %Q( <?qbxml version="#{QBWC.quickbooks_min_version}"?> <QBXML><QBXMLMsgsRq onError="stopOnError"> ) + xml_rq + %q( </QBXMLMsgsRq></QBXML> )
    end
  end

I was not using using the Ruby to QBXML functions. Instead was Calling a wrapper class and calling to_qbxml on that.


Incorrect routing of anchors when root is set

When a non-empty root is set during router start, routing of clicked anchors fails.
This happens because the anchor pathname is used, which includes the root, which is passed to navigate, which passes that to getFragment, which fails to remove the root from the fragment.

Just ran into the same problem, thanks @nrbrook, it would be a valuable fix in ampersand-history!

I got same bug today, @nrbrook  thanks for temp solution


.editorconfig: Do not trim trailing whitespace in Rust files

We would normally want this, but since whitespace can be important for some tests it doesn't make any sense here.

Maybe a better solution would be to override .editorconfig for those particular tests?

I mean, storing a second .editorconfig file in the tests directory

hmm... might not be a bad idea. would you be okay with an testData/.editorconfig file?

I think testData is of a too broad scope. We do want to strip whitespace for almost all .rs. I guess it should be testData/org/rust/lang/typing/testdata/.editorconfig

hmm I just realized that the test also works without the whitespace... closing this for now


Don't copy the partition Id when using copy constructor

Fixes https://github.com/Azure/azure-sdk-for-net/issues/39886

API change check
API changes are not detected in this pull request.


token-configuration

I'm trying to use tokens library on kubi...
Somehow it fails on first try with error:
no token
and somehow ok on further requests.
I've added
.getFilesystemSecretsRefresherConfiguration.validateTokensOnStartup()
to a builder and now I'm receiving an error:
The following token-configurations couldn't be found : [default-token]

@skokorin If you still have trouble using the library or you still experience this issue please ping me via company chat, to discuss in more detail and maybe see some logs.


Return value of tryNewDot NVP in authenticate response

Problem
We need to return the tryNewDot NVP value to HybridApp users so that we can route users to the correct app experience. This is required for the TTI app start improvement issue.

The SignInUser command should return an additional field with nvp_tryNewDot, allowing us to determine based on the value of the dismissed key whether to stay on NewDot or transition to OldDot.

Solution
Return the value of tryNewDot dismissed in the authenticate command.

SignInUser
SignUpUser

Notes from my dev env testing:
NewDot user
284 -<EMAIL_ADDRESS>
OldDot user
251 -<EMAIL_ADDRESS>
empty NVP user
<EMAIL_ADDRESS>

Data being returned to Web. I might need to work on the Web-E changes next.

Spent all day on chores, reviews, App deploy No progress.

Auth PR: https://github.com/Expensify/Auth/pull/13016

Auth PR merged, awaiting deployment. I have some Web changes to make in addition.

Web-E PR in review: https://github.com/Expensify/Web-Expensify/pull/44422

PRs merged, awaiting deployment

Auth - done
Web-E on staging, should hit prod today

Just deployed to prod


I have problems with pull request

I need help. I downloaded literally everything, both files, no errors in them, but my request was still denied. Maybe I'm doing something wrong, which is why my request is rejected 15 seconds after creation.

My pull request number is #649

files werent in correct directory


docs: simplify various sections

use lists for things that are lists
use sublists instead of the non-breaking-space indented paragraphs
move the default configuration up, since that's the thing most people will regularly come back to
fix the lspconfig snippet, which was assuming the existence of some variables that aren't there.
put the luasnip section into a <details> section. It being the only opened section down there, even though other sections are more relevant makes little sense.
open the "Community Sources" <details> section. There aren't enough there yet to warrant folding them away, and it increases their visibility. Also add a heading to them for navigability, linkability and visibility.

Thanks, I'll take just the changes to the "Special Thanks" and "How it works" sections and moving the default configuration up. I'll rework the community sources section in the future. Luasnip is used commonly enough that I prefer it left open.
Can you explain what's wrong with the lspconfig snippet? I've been using that snippet without issue in my config so curious where it doesn't work

I'll rework the community sources section in the future. Luasnip is used commonly enough that I prefer it left open.

done

Can you explain what's wrong with the lspconfig snippet? I've been using that snippet without issue in my config so curious where it doesn't work

Sure:
{
  'neovim/nvim-lspconfig',
  dependencies = { 'saghen/blink.cmp' },
  config = function(_, opts)
    local lspconfig = require('lspconfig')
    for server, config in pairs(opts.servers or {}) do
      config.capabilities = require('blink.cmp').get_lsp_capabilities(config.capabilities)
      lspconfig[server].setup(config)
    end
  end
}

I assume it works in your config, cause your config you have more stuff set:

the lazy.nvim opts spec. If the user does not have opts set, the opts you pass in config is empty, meaning this part won't work.
Even if the user has configured the opts spec, this snippet assumes that they have set a capabilities key for each server, something that many people do not do (since they normally do not mess with a server's capabilities). This will result in an error.

The change I made is more straightforward, and shows examples how you can in principle add the capabilities without using lazy.nvim and just with lspconfig. Also, it works standalone, without assuming that the user has set up opts.server.capabilities (where all three parts are not necessarily a given.)

config.capabilites shouldn't be required here since blink will ignore the argument if it's nil. I believe using opts is common enough that we should provide an example for that. How about something like
{
  'neovim/nvim-lspconfig',
  dependencies = { 'saghen/blink.cmp' },

  -- example using `opts` for defining servers
  opts = {
    lua_ls = {}
  },
  config = function(_, opts)
    local lspconfig = require('lspconfig')
    for server, config in pairs(opts.servers or {}) do
      -- passing config.capabilities to blink.cmp merges with the capibilities in your
      -- `opts[server].capabilities, if you've defined it
      config.capabilities = require('blink.cmp').get_lsp_capabilities(config.capabilities)
      lspconfig[server].setup(config)
    end
  end

 -- example calling setup directly for each LSP
  config = function()
    local capabilities = require('blink.cmp').get_lsp_capabilities()
    local lspconfig = require('lspconfig')

    lspconfig['lua-ls'].setup({ capabilities = capibilities })
  end
}

Shouldn't it be require('blink.cmp').get_lsp_capabilities(nil, true)? If I understand this line correctly, it does not add the default capabilities without a truthy second argument:
https://github.com/saghen/blink.cmp/blob/81069e3955a8b4d5063287a2a6a47a844e9ca2cb/lua/blink/cmp/sources/lib/init.lua#L257

lspconfig ends up merging with the default capabilities internally so that should be okay: https://github.com/neovim/nvim-lspconfig/blob/dafd61d6533bd90ecf6e2a3a972298fdddc74d82/lua/lspconfig/configs.lua#L95

Looks great, tysm!


daemon - Make Uptime Save to File After Exit

Write uptime to log.txt on exit in addition to printing the uptime on screen.

This should save time and date as well so we can see how the bot become more / less stable as time goes on.

I added what should work (haven't committed yet), haven't gotten a natural crash to test it yet.

Closed in 02c2bc6121


Add fast-jwt as an option for JWT libraries

Thank you for contributing to the Node.js Toolbox catalog! 🙏
Note: Node.js Toolbox is specifically for backend libraries. If a library works both in the browser AND in Node.js feel free to add it. Libraries that work ONLY in the browser are out of scope for now.
Before submitting your pull request, please make sure:

[X] To use the exact package name as it appears on NPM (and npm install <package-name>)
[X] The packages are listed in alphabetical order
[X] The package doesn't already exist in a different category
[X] Description doesn't have a dot (.) at the end
[X] To format with Prettier before committing (npm run prettier)

While the library has fewer downloads than node-jose, it is used by fastify for example.
More information at https://github.com/nearform/fast-jwt

@d3x7r0 Thank you for your contribution! https://nodejstoolbox.com/categories/json-web-token


ImportError: dlopen failed: "/data/data/com.github.andremiras.qrscan/files/app/_python_bundle/site-packages/PI...

https://sentry.io/organizations/andre-5t/issues/1225879526/?referrer=github_plugin
ImportError: dlopen failed: "/data/data/com.github.andremiras.qrscan/files/app/_python_bundle/site-packages/PIL/_imaging.so" is 64-bit instead of 32-bit
(1 additional frame(s) were not displayed)
...
  File "/home/andre/workspace/QrScan/.buildozer/android/platform/build/build/python-installs/qrscan/kivy_garden/zbarcam/__init__.py", line 16, in <module>
  File "/home/andre/workspace/QrScan/.buildozer/android/platform/build/build/python-installs/qrscan/kivy_garden/zbarcam/zbarcam.py", line 12, in <module>
  File "/home/andre/workspace/QrScan/.buildozer/android/platform/build/build/python-installs/qrscan/kivy_garden/zbarcam/utils.py", line 2, in <module>
  File "home/andre/workspace/QrScan/.buildozer/android/platform/build/build/python-installs/qrscan/PIL/ImageOps.py", line 20, in <module>
  File "home/andre/workspace/QrScan/.buildozer/android/platform/build/build/python-installs/qrscan/PIL/Image.py", line 64, in <module>

ImportError: dlopen failed: "/data/data/com.github.andremiras.qrscan/files/app/_python_bundle/site-packages/PIL/_imaging.so" is 64-bit instead of 32-bit
(1 additional frame(s) were not displayed)
...
  File "/home/andre/workspace/QrScan/.buildozer/android/platform/build/build/python-installs/qrscan/kivy_garden/zbarcam/__init__.py", line 16, in <module>
  File "/home/andre/workspace/QrScan/.buildozer/android/platform/build/build/python-installs/qrscan/kivy_garden/zbarcam/zbarcam.py", line 12, in <module>
  File "/home/andre/workspace/QrScan/.buildozer/android/platform/build/build/python-installs/qrscan/kivy_garden/zbarcam/utils.py", line 2, in <module>
  File "home/andre/workspace/QrScan/.buildozer/android/platform/build/build/python-installs/qrscan/PIL/ImageOps.py", line 20, in <module>
  File "home/andre/workspace/QrScan/.buildozer/android/platform/build/build/python-installs/qrscan/PIL/Image.py", line 64, in <module>

Sentry issue: QRSCAN-Y

Fixed in last release d169e79bcbee7b6049e93ee556a24bce11857d98 also available on the Play store


Bold text color

This is probably a dumb question but I haven't figured it out: When i have txt in bold the text appears in a red font. How can i change that?
And how can i change it so that it adjusts to it's 'surroundings'? (for example, bolded text in a green callout would appear green)

This is probably a dumb question but I haven't figured it out: When i have txt in bold the text appears in a red font. How can i change that? And how can i change it so that it adjusts to it's 'surroundings'? (for example, bolded text in a green callout would appear green)

... and relatedly, how can i make titles in callouts appear less thick?

i figured this out :)

+1. how did u fix it?

+1. how did u fix it?

the bold text in red? i went to the theme.css file and looked for the section on italics and bold and just erased the color part


Add container filtering for fields/attributes with reference type

I've created 2 collections of custom objects, meaning multiple objects in 2 containers, brands and merchants and I want to be able to reference them from a product.
I've added 2 attributes, brand and merchant to a new product type, using a key-value-document type and I can add references to them.
My problem now is that I can put the id of any kind of custom object for brand, I can add a merchant there by mistake.
It would be nice if the attribute definition could also have a filter for this case, to be able to indicate that only custom objects from a certain container is allowed.
This would also allow a new feature in Merchant Center, where the product edit form could show a drop-down with the keys of the custom objects defined in that container for these kind of attributes, because adding a UUID by hand is really not user friendly.

Your request is an enhancement of the API and MC itself. Please refer to https://support.commercetools.com and open an issue there.


Center grid class half

Looks better (In my opinion)
The half class is used on the landing page, about page and tools page.

Do you have a screenshot of before/after? I'm not exactly sure of what to look for.

Do you have a screenshot of before/after? I'm not exactly sure of what to look for.

I have made the grid layout visible. For reference.
before:

after:

Ah thank you @RVA2869! I agree!


ChefDK to 0.10.0

Thank you for the contribution. It was merged directly as 811b14cbcd6d7641a5cee00c793ef01f6f3e8314 to keep commit history cleaner. Your contribution is still credited to you.


Update README.md / Add browsh to examples

Add browsh to the list of examples.

This is very very cool.


Simplify test run in github action

Codecov Report

Merging #1921 (e22bd84) into master (ce5cd62) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1921   +/-   ##
=======================================
  Coverage   62.84%   62.84%
=======================================
  Files          92       92
  Lines        4697     4697
  Branches      321      321
=======================================
  Hits         2952     2952
  Misses       1709     1709
  Partials       36       36

Continue to review full report at Codecov.

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


Fix icons

The Jekyll rendering of GitHub does not support the colon-notation for emojis:

This PR fixes it.
Note that on Windows OS, one can use Win+. to select emjos to type. Not sure for Linux though...

Alternative is to build the jekyll page in a GitHub action and use the GitHub-flavored emoji plugin for Jekyll


Drop interpolates adjust param and add new mask method

Still need to decide whether mask should take multiple positional arguments or a list. Currently it takes multiple positional arguments.

@gir-bot lgtm


Added Bedrock Cohere Embeddings Provider + explicit apache arrow as d…

Description

I added a Bedrock cohere embeddings provider.
I had to add apache arrow as dependency because there were build issues without that package as vectorDB depends on it. (I pasted the error log in the issue).

Checklist

[x] The base branch of this PR is dev, rather than main
[x] The relevant docs, if any, have been updated or created

Screenshots

There's no visual changes.

Testing

I have tested this change in debug mode as mentioned in the documentation, it was able to use bedrock cohere embedding model and fetch the results.
I ran the npm run package , copied the extension to another machine and tried it. It ran successfully.

Fixes Apache-arrow error:
detail: undefined, id: '', location: { column: 31, file: '../../core/node_modules/vectordb/dist/remote/index.js', length: 14, line: 19, lineText: 'const apache_arrow_1 = require("apache-arrow");', namespace: '', suggestion: '' }, notes: [ { location: null, text: 'You can mark the path "apache-arrow" as external to exclude it from the bundle, which will remove this error. You can also surround this "require" call with a try/catch block to handle this failure at run-time instead of bundle-time.' } ], pluginName: '', text: 'Could not resolve "apache-arrow"' }, { detail: undefined, id: '', location: { column: 31, file: '../../core/node_modules/vectordb/dist/sanitize.js', length: 14, line: 24, lineText: 'const apache_arrow_1 = require("apache-arrow");', namespace: '', suggestion: '' }, notes: [ { location: null, text: 'You can mark the path "apache-arrow" as external to exclude it from the bundle, which will remove this error. You can also surround this "require" call with a try/catch block to handle this failure at run-time instead of bundle-time.' } ], pluginName: '', text: 'Could not resolve "apache-arrow"' } ], warnings: [] }

Fix
This PR  fix #2475

I will squash and merge.

@Anantha-Kandrapu thanks for the PR! It looks like a lot of the logic here might overlap with the original Bedrock provider—do you think that we could maintain just a single bedrock embeddings provider that can also support cohere?

Also, I would prefer if the adding of the dependency were in a separate PR, as I'll probably want to review that a bit more closely and it isn't strictly necessary for this update

@Anantha-Kandrapu thanks for the PR! It looks like a lot of the logic here might overlap with the original Bedrock provider—do you think that we could maintain just a single bedrock embeddings provider that can also support cohere?

I have also added a interface so that multiple other embedding providers from bedrock can be added in future.

Hey @chezsmithy , Thanks for the approval earlier.
My fork was out of sync. There's one more approval I need.
Thanks.

Thanks for the great work here @Anantha-Kandrapu !


PMMatrix >> #asString should be distinct from PMVector >> #asString

Currently, PMMatrix just prints its contained vectors separated by newlines. This can be misleading:
a := PMMatrix rows: #((1 2)).
b := #(1 2) asPMVector.
a asString = b asString. "true"

However, a 1x2 matrix and a 2-vector are not currently polymorphic. Probably, each should have its own print signature. Maybe
a "[<1 2>]" or "<<1 2>>" or "[1 2; ...]"
b "<1 2>"

Can we just create asString for PMMatrix and print something like this:
a := PMMatrix rows: #((1 2)).
a asString.
>>> a PMMatrix(
a PMVector(1 2)
)


Added Styling to Logout button And Reckeck before login out

Added Styling to Logout button And Reckeck before login out

Hey @surajvast1 you should use dark theme for the confirm logout modal as it matches more with the application.

Ohkkk I added Dark Mode Its Looks FIne Now I suppose

Can you merge this pr i
And if there anything else I am happy to contribute ☺

Hey, @surajvast1 the Reckeck before login out is too good but the styling to Logout button and username doesn't look that good so can you change those two back.

Ohk sure I'll change the logout button but I didn't understood the username what about it ?

Thanks I'll change them

Ohk  i changed It ,I  just added a small zoom in on button hover I hope it doen't creates problem I liked It better ,If any changes I am happy to contribute.

Ohk i changed It ,I just added a small zoom in on button hover I hope it doen't creates problem I liked It better ,If any changes I am happy to contribute.

Hey @surajvast1 it would be nice if you could use only tailwind css for the styling.

Hey @surajvast1 it would be nice if you could use only tailwind css for the styling.

Leave it's looking good. I am merging it.


Remove "externals"

Overview
Remove all the plumbing that enabled _external/ request handling,
leaving only the functions necessary for list and show.
closes https://github.com/apache/couchdb/issues/2166
Testing recommendations
N/A
Related Issues or Pull Requests
N/A
Checklist

[ ] Code is written and works correctly
[ ] Changes are covered by tests
[ ] Any new configurable parameters are documented in rel/overlay/etc/default.ini
[ ] A PR for documentation changes has been made in https://github.com/apache/couchdb-documentation

I think once _show and _list are removed, we can remove those modules but I didn't think now was the right time to move these (ancient) functions or attempt to consolidate them (there's a ton of overlap but perhaps subtle differences). Will merge if the Travis fairy grants my wish.


Help popup font size: title ok but content too big

@octref I'm deliberately "reopening" issue https://github.com/microsoft/vscode/issues/80724 here because I've new input that seems relevant to our disagreement there, but sadly the other issue was locked.
The new input is related to https://github.com/microsoft/vscode/issues/88916 and https://github.com/microsoft/vscode-python/issues/9667. It seems relevant to me because it's pointing to the fact that the size of the content of the help popup (but not its title) is coupled to the size of a font of a very different nature (a monospace font belonging to a different family) which a fortiori supports:

yet you can see that everything in the GUI (without taking into account the editor and terminal fonts, which are independently controlled) is smaller than the contents of the help popup

If things are how this evidence suggest, I really don't get the rationale behind the decision to fetch the size of a non monospaced font, which is the same one used for the rest of the GUI, from the size of a font of another family, knowing how unreliable this could be. Wouldn't it be better to use some scale of the base font size since this font is of the same family and the user has already accepted it for the rest of the GUI? Or to measure the actual bounding box of some reference text rendered using one font and make the other one occupy the same area?
 Currently we don't allow you to customize that font yet. You can track #519.

doesn't seem to be pertinent at the light of my comments above: the size is indeed not taken to be the size of the non-configurable workbench font and I don't see why you will revise that decision once #519 was hopefully implemented, since you already decided to relate the doc font size to another setting (probably to make it consistent with the popup title, I don't know, but, at any rate, that's not happening in this case).
Platform:

VSCode Version: 1.41.1
OS Version: Ubuntu 19.04
Screen: 3000x2000 hidpi 2x scaling factor.

Steps to Reproduce:

Set Ubuntu Font as the editor font (but other fonts will do it, for example Inconsolata).
At the default size of 14px this font is a bit small, so set it to a more reasonable size, say 16px or 17 px.
3.Now type some code, say import argparse, and open the doc popup window for argparse.

I tried it, setting one to 14px and the other to 17px. If a user chooses 17px font, maybe he/she cannot read smaller text comfortably. Also then the widget is not consistent.


error spawn zip ENOENT

any idea about this error?
$ uname -a
Linux cdm6_58 4.15.0-47-generic #50-Ubuntu SMP Wed Mar 13 10:44:52 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

$ sudo apt install npm
$ sudo npm install -g share-cli

$ share myfile.txt
✖ Preparing file failed
  Server
  Tunnel
  Copy
  Download

spawn zip ENOENT

Thanks for raising this issue!
Yes, zip has to be in your PATH so share-cli can call it:
https://github.com/marionebl/share-cli/blob/26e9239782aa8fa1b87015f4104e08198c83f8b4/lib/get-file.js#L39-L44
This should either be replaced by a node implementation or be part of the docs.

this solves the issue, thanks! :)

I encountered this issue as well (on windows) and it seems that the zip command doesn't exist. I was able to find a binary for zip and unzip for windows and using it works however it probably isn't the best solution.

I encountered this issue as well (on windows) and it seems that the zip command doesn't exist. I was able to find a binary for zip and unzip for windows and using it works however it probably isn't the best solution.


Remove needless #ifndef OPENSSL_NO_SOCK for X509_{CRL_}load_http

It turned out that #ifndef OPENSSL_NO_SOCK is no more needed for X509_{CRL_}load_http().

Merged - thanks @mattcaswell for the immediate approval.


White boxes on install packwiz gui

https://uploadi.ng/fazzi/jNScKfQo
as you can see here, there are white boxes and I'm unable to see the options. I'm running Arch Linux. Any suggestions?

Hmm, that's strange... try updating/reinstalling Java? Make sure you're not using the headless JDK/JRE.


[bug]: Building env in conda (latest miniconda) on colab and getting this error

Is there an existing issue for this?

[X] I have searched the existing issues

OS
Linux
GPU
cuda
VRAM
16
What happened?
When building environment in miniconda on colab from environment-lin-cuda.yml i get this error.
Running command git clone --filter=blob:none --quiet https://github.com/invoke-ai/PyPatchMatch /tmp/pip-install-bjpfxgtd/pypatchmatch_cdb9d4b3f65d48de8c9a3e58222ebcb6
ERROR: file:///content/drive/MyDrive/InvokeAI/environments-and-requirements (from -r /content/drive/MyDrive/InvokeAI/environments-and-requirements/condaenv.v7l8ukul.requirements.txt (line 32)) does not appear to be a Python project: neither 'setup.py' nor 'pyproject.toml' found.failed
CondaEnvException: Pip failed
Screenshots
No response
Additional context
No response
Contact Details
No response

This PR is relevant (to fix PyPatchMatch):
https://github.com/invoke-ai/PyPatchMatch/pull/7
You could temporarily use the PR branch to work around the issue.


Request to become a Collaborator

Hello @neetcode-gh, I've been contributing to this project for a bit, approved merges dating back 1.5 years. I would like to have collaborator access so I can also help review PRs.
My main focus would be C# but I would also be able to help with Java and C++.

Could I also request collaborator access @neetcode-gh ? Will just focus on C++.
I'm on the leetcode grind right now and already submitted a couple missing C++ solutions.
My specialty is C++ and see some C++ PR's I could help with, too.


Update connector to 0.10.2-SNAPSHOT in branch r0.10

Problem description
Update the connector version to 0.10.2-SNAPSHOT post 0.10.1 release.
Problem location
gradle.properties
Suggestions for an improvement
Should cherry pick to all 0.10 branches

Closed via #614


Find a good place for the testing configuration

https://github.com/alibaba/GraphScope/blob/101ab6f3f19dfcaceb8f1ea5bbcd97420815668c/interactive_engine/src/assembly/conf/vineyard.json#L3

duplicated to #539


FullName in CFF table is not changed

I recently used your script and it works fatastic, however I realised, that the FullName in CFF table is not changed.
I had no problems whatsoever so far with renamed fonts, yet I’m still wondering what this entry in the CFF table does.
So far all apps I tested the fonts with don't seem to care about this entry. Is it not important? Do I not need to change it?
For the sake of consistency I changed it manually with ttx, but I was wondering if there is a better way/it is necessary.

Good question.  I came across this in the OpenType specification for nameID 6:

In a CFF OpenType font, there is no requirement that this name be the same as the font name in the CFF’s Name INDEX. Thus, the same CFF may be shared among multiple font components in a Font Collection.

A good check might be to see what names show up in Adobe products when the names differ.
Did the FullName value match the previous name table nameID 6 value of the font that you edited?

Did the FullName value match the previous name table nameID 6 value of the font that you edited?

Yes, that is how I noticed (I changed everything but the entry in the CFF table was still the previous name)
So far I had no problems using the fonts. Everything was displayed correctly on Mac and Win :)

93f719f42dc15a1e22c4190842668fb515465b02 should close this issue.  Mind having a look and let me know if this gets you to the CFF name updates that you needed to make manually?

I added support for FullName, FamilyName, and FontName fields in the CFF table.


update_service doesn't recognize changed certificates

Bug Report
Current Behavior
Hi!
We are facing an issue with the update_service method for loadbalancers. We are trying to add/delete certificates within the http-Object of a service. After adding or deleting a certificate inside the http-Object and calling the update_service method, the changes aren't made.
Input Code
Example:
created_cert = hcloud_client.certificates.create(
    name=f"{cert_name}_{datetime.strftime(datetime.now(), '%Y%m%d')}",
    certificate=cert["fullchain"],
    private_key=decrypted_privkey.decode('utf-8'),
)

hcloud_load_balancers = hcloud_client.load_balancers.get_all()

for load_balancer in hcloud_load_balancers:
    for service in load_balancer.services:
      service.http.certificates.append(created_cert)
      update_service_action  = load_balancer.update_service(service)
      update_service_action.wait_until_finished()

We already debugged the code, and especially looked into the service.http.certificates-object/list, before calling update_service, to observe, if the certificate is inside the service-object. Everything seems fine. The update_service_action.status also contains success after calling it.
Expected behavior/code
We would think, the above code should add a new certificate to the specified loadbalancers service.

Python Version: 3.8
Hcloud-Python Version: 1.12.0

Hey @dithmer,
i tried it with your code above and for me it works perfectly, can you try it again?


Adding empty array test case for drop and take methods

By this pull request, I am trying to add empty array test for both drop and take methods in
ArrayExtenstions. By adding this, it is expected to have 100% test coverage for ArrayExtensions.swift file alone.
Checklist

[x] I checked the Contributing Guidelines before creating this request.
[x] I have only one commit in this pull request.
[ ] New extensions support iOS 8 or later.
[ ] New extensions are written in Swift 3.
[ ] I have added tests for new extensions, and they passed.
[x] Pull request was created to master head branch.
[ ] All extensions have a clear comments explaining their functionality, all parameters and return type in English.
[x] All comments headers have the word SwifterSwift: before description.

@codecov[bot]  test again

@tapanprakasht I don't have time to review this right now. Sometimes CodeCov has a failing check because of a bad UIImageView test I wrote lol. I will have to fix this shortly because it's starting to annoy me. If there is a problem I will let you know.

@SD10 I was thinking how ArrayExtensions test case affected UIImageViewTest. Because of that, I tried to retrigger the covdecov. Anyway, you have cleared it. But I have a doubt regarding drop  extensions. Is there any method which has the same name in ArraySlice class? For me, Xcode auto-suggestion gives me that drop function for an empty array.

@tapanprakasht None of the Array methods should affect ArraySlice.
It looks like drop(while:) has been added to the standard library though. I will have to do more investigation in the morning 😴

It looks like we don't need to create a drop extension for the array. See my below code from playground I created. drop function is already there.

Thank you guys, great job!


Added tenderly utils package

Added a new tenderly utils package that can be used to set a testing env required for integration tests

@robercano to create a fork from a block number, I'll try to add it, thanks for heads-up


Is it possible to use OCI hooks? How?

How it is possible to use OCI hooks such as oci-systemd-hook?
/etc/containers/oci/hooks.d appears to be a canonical location for hooks, at least according to oci-hooks(5)
What is rkt's compliance with spec 1.0.1 hooks?

See also #3953. Probably using hooks is irrelevant if there is a way to boot contained under rkt...


Uncaught TypeError: Reflect.getMetadata is not a function

I was writing unit test cases to test a simple service but while running test case, I am getting following error:
Uncaught TypeError: Reflect.getMetadata is not a function
Please suggest something to fix it.

i faced same error

I solved this issue by adding import 'reflect-metadata' on the top of my test specifications code file.

that works for me too.

Worked for me too, but can anyone explain why this is needed? I ran into this issue trying to incorporate preboot with Angular Universal, so I'm not sure why the problem occurred in the first place.


Migrating from ReactJS to NextJS and having this error:  404 Error()

Before opening, please confirm:

[X] I have checked to see if my question is addressed in the FAQ.
[X] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting questions.
[X] I have removed any sensitive information from my code snippets and submission.

App Id
d1plugrqiuk4q4
AWS Region
us-east-1
Amplify Hosting feature
Deployments, Frontend builds, Git providers
Question
I had a project on github and it was deployed to AWS Amplify built in Reactjs, Then I decided to migrate from Reactjs to Nextjs. I made new project and after making all the changes, I pushed and replaced all the changes in that repository, it deployed on AWS successfully, but it doesn't showing anything on the given domain link by Amplify and getting this error: Failed to load resource: the server responded with a status of 404 ().
Is it a good approach to migrat ReactJS project to NextJS and deploy in the same instance in AWS Amplify?

Hi @ghufran2508 👋 , thanks for raising this!
It seems that your production branch is resolving as expected (both the .amplifyapp.com subdomain and custom domain). The outlined behavior is typically caused due to an incorrect baseDirectory field within the build specification file (amplify.yml). Thus, I'd recommend reviewing the same.


TypeError: Cannot read property 'existsAt' of undefined

I'm trying to use this with the vue-cli 3.0 build.
vue.config.js
const BundleAnalyzerPlugin = require('webpack-bundle-analyzer').BundleAnalyzerPlugin;
const CompressionPlugin = require('compression-webpack-plugin');
const BrotliPlugin = require('brotli-webpack-plugin');
const { BugsnagSourceMapUploaderPlugin } = require('webpack-bugsnag-plugins');

module.exports = {
  configureWebpack: (config) => {
    if (process.env.NODE_ENV === 'production') {
      return {
        plugins: [
          new BugsnagSourceMapUploaderPlugin({
            apiKey: '<api key>',
            // appVersion: '1.0.0',
          }),
          new CompressionPlugin(),
          new BrotliPlugin(),
          new BundleAnalyzerPlugin({ analyzerMode: 'static' }),
        ],
      };
    }
    return {
      plugins: [
        new BundleAnalyzerPlugin({ openAnalyzer: false }),
      ],
    };
  },
  chainWebpack: (config) => {
    if (process.env.NODE_ENV === 'production') {
      return config.plugins.delete('fork-ts-checker');
    }
    return config;
  },
};

yarn run v1.9.4
$ vue-cli-service build

⠋  Building for production.../<vue project>/node_modules/webpack-bugsnag-plugins/source-map-uploader-plugin.js:54
            map: compilation.assets[map].existsAt,
                                         ^

TypeError: Cannot read property 'existsAt' of undefined
    at maps.map.map (/<vue project>/node_modules/webpack-bugsnag-plugins/source-map-uploader-plugin.js:54:42)
    at Array.map (<anonymous>)
    at chunkToSourceMapDescriptors (/<vue project>/node_modules/webpack-bugsnag-plugins/source-map-uploader-plugin.js:43:21)
    at Array.map (<anonymous>)
    at plugin (/<vue project>/node_modules/webpack-bugsnag-plugins/source-map-uploader-plugin.js:67:39)
    at _err0 (eval at create (/<vue project>/node_modules/tapable/lib/HookCodeFactory.js:32:10), <anonymous>:12:1)
    at callback (/<vue project>/node_modules/copy-webpack-plugin/dist/index.js:126:17)
    at afterEmit (/<vue project>/node_modules/copy-webpack-plugin/dist/index.js:220:13)
    at AsyncSeriesHook.eval [as callAsync] (eval at create (/<vue project>/node_modules/tapable/lib/HookCodeFactory.js:32:10), <anonymous>:7:1)
    at AsyncSeriesHook.lazyCompileHook (/<vue project>/node_modules/tapable/lib/Hook.js:154:20)
    at asyncLib.forEach.err (/<vue project>/node_modules/@vue/cli-service/node_modules/webpack/lib/Compiler.js:355:27)
    at done (/<vue project>/node_modules/neo-async/async.js:2854:11)
    at /<vue project>/node_modules/neo-async/async.js:2805:7
    at /<vue project>/node_modules/graceful-fs/graceful-fs.js:43:10
    at FSReqWrap.oncomplete (fs.js:141:20)
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Could be https://github.com/Izhaki/nodemon-webpack-plugin/commit/8475d3f7c3b9aecedaa2e3af61c77105d94b32c0

Hey thanks for reporting the issue. I'll get this looked into. We're working on a lot of higher priority stuff right now so it may be a little while. Cheers!

So I've narrowed down what happens. Vue by default generates sourcemaps for only js, skipping css. This plugin is trying to upload both. Maybe it should skip anything not ending in ".js"?

Same issue here. I'm using Nuxt.js
 ERROR  Cannot read property 'existsAt' of undefined                                                                                09:20:48

  at maps.map.map (node_modules/webpack-bugsnag-plugins/source-map-uploader-plugin.js:53:48)
  at Array.map (<anonymous>)
  at chunkToSourceMapDescriptors (node_modules/webpack-bugsnag-plugins/source-map-uploader-plugin.js:43:21)
  at Array.map (<anonymous>)
  at plugin (node_modules/webpack-bugsnag-plugins/source-map-uploader-plugin.js:67:39)
  at _err0 (eval at create (node_modules/tapable/lib/HookCodeFactory.js:32:10), <anonymous>:19:1)
  at reportBuild.then.catch (node_modules/webpack-bugsnag-plugins/build-reporter-plugin.js:20:11)
  at process._tickCallback (internal/process/next_tick.js:68:7)

Hey thanks for reporting the issue. I'll get this looked into. We're working on a lot of higher priority stuff right now so it may be a little while. Cheers!

Eagerly awaiting this fix. Seems this plugin is unusable for Vue apps in the meantime.

I had this happen when my entry file was incorrect, double check it!


Serialization refactoring

Issue #, if available:
https://bitquill.atlassian.net/browse/AN-1122
Description of changes:
performance enhancements for Golang Driver
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Codecov Report

Merging #103 (429e9f3) into gremlin-go-ms6 (b3d378b) will decrease coverage by 3.68%.
The diff coverage is 68.46%.

@@                Coverage Diff                 @@
##           gremlin-go-ms6     #103      +/-   ##
==================================================
- Coverage           67.20%   63.51%   -3.69%
==================================================
  Files                  23       23
  Lines                3494     3281     -213
==================================================
- Hits                 2348     2084     -264
- Misses                960     1077     +117
+ Partials              186      120      -66

Impacted Files
Coverage Δ

gremlin-go/driver/performance/performanceSuite.go
0.00% <ø> (ø)

gremlin-go/driver/graphBinary.go
48.87% <64.48%> (-20.53%)
:arrow_down:

gremlin-go/driver/serializer.go
68.02% <80.00%> (+14.32%)
:arrow_up:

Continue to review full report at Codecov.

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

Overall LGTM, going to hold merging for a day or so because this will likely have 3.6-dev and master conflicts as they have additional types.

moving to 3.5-dev


VS 17.5.3 Error List tab shows nonsense

This issue has been moved from a ticket on Developer Community.

[severity:It bothers me. A fix would be nice]
The attached screenshot shows the error list. The 1st entry is "Propane is not defined". You can see in the edit window there is no "Propane" on the first line. The 2nd entry says "TANK" is not defined. That, also is not on the 1st line of the edit window.

Original Comments
Feedback Bot on 5/3/2023, 07:47 PM:
(private comment, text removed)

Original Solutions
(no solutions)

Hi there, thanks for reporting this issue. It has been fixed in https://github.com/microsoft/PTVS/issues/7068. VS has a long release cycle, which means that once we have identified and fixed an issue, it may take some time before the fix is incorporated into a new version of Visual Studio that is publicly available. Could you please try it again in the latest preview version of VS and let us know if the issue still persists?


Adding NASDAQ and BSE scraping support to Finance Module

Proposed Method:  adding NASDAQ and BSE classes.
Description
I would like to create a nasdaq.py and bse.py and add NASDAQ and BSE classes to the respective files which would handle requests from nasdaq and bse websites. Then the stock_price.py would import these and StockPrice class can support scraping from these indexes as well.
Example Usage
The usage would remain same as NASDAQ and BSE classes are being imported by StockPrice
Please assign this issue to me under GSSoC'23

Great idea @brohithkr | Go  ahead.


feat(ci): add Docker Scout vulnerabilities scanning

Motivation
We must avoid publishing new releases without being fully aware of any new vulnerabilities that might be introduced into the image. This ensures we have visibility and can take the necessary actions, such as updating our READMEs, fixing the vulnerabilities, or implementing any other required measures.
Specifications & References

https://docs.docker.com/scout/explore/analysis/

Solution

Implement Docker Scout CLI with their GitHub Action: https://github.com/docker/scout-action

Tests
Trigger a change in our Dockerfile to validate the behavior of the action and the results
Follow-up Work
Implementing policies for Docker Scout that are compliant with our supply chain requirements https://docs.docker.com/scout/policy/
PR Author's Checklist

[x] The PR name will make sense to users.
[x] The PR provides a CHANGELOG summary.
[x] The solution is tested.
[x] The documentation is up to date.
[x] The PR has a priority label.

PR Reviewer's Checklist

[ ] The PR Author's checklist is complete.
[ ] The PR resolves the issue.

Need a workaround for: https://github.com/docker/scout-action/issues/16


[ Re #7 ] Add a flag extra-instances for an instance Data.Set (enabled by default)

As title.

Hi, thank you very much. Will look into this ASAP (I hope this week).
Best,
Moritz

Thank you.


Fleet State

** Brought to you by Hack Week **
This was a feature in the 1.0 days where an operator
could find insight into the current state of all
their deployed servers.
After the inception of 2.0 we lost this insight as the 1.0
builders were gone and Nomad didn't give us the data.
Using the limited functionality of Nomad we can scrape the
current state of the fleet one host at a time. This takes
a while, but it seems like the safest option. In a scenario
where there are hundreds of nomad clients, I don't know how
long this will take to run, but it seemed safer than forking
hundreds of kubectl commands simultaneously.
The VM service gives us some insight via an exposed endpoint
where curl is already installed. Using this data we can display,
in a human readable fashion, how many machines of each type are
currently deployed.
Runner support is coming soon after the endpoint is published.
Sample output:
ID: ip-172-16-4-163
CPU: 43 / 20000 MHz
RAM: 1.2 / 31 GB
----
ID: ip-172-16-4-135
CPU: 217 / 20000 MHz
RAM: 1.5 / 31 GB
----
ID: ip-172-16-4-6
CPU: 7083 / 20000 MHz
RAM: 1.4 / 31 GB
----
Total CPU: 7343 / 60000 MHz
Total RAM: 4.1 / 93 GB
{
  "VM #": 4,
  "Task #": 4,
  "Name": "docker-default",
  "Remote Docker": true,
  "Type": "l1.medium",
  "Boot Avg": 63.364245
}
{
  "VM #": 4,
  "Task #": 4,
  "Name": "default",
  "Remote Docker": false,
  "Type": "l1.medium",
  "Boot Avg": 38.403722
}

I have realized this script is actually not accounting for the reserved RAM & CPU, just the resources actively in use. I'll have to talk to Fish to determine the intended functionality, and if we need it in JSON.

Going to close this to clean up PRs. Please re-open if still needed.


feat: add jmespath module

closes #142

Nice!
What goes into choosing between these two?

https://github.com/jmespath-community/go-jmespath
https://github.com/jmespath/go-jmespath

I just want to confirm the usage of the "community" one was intentional, since it seems to be the lesser used one.

The community one seems to be more actively maintained, see also https://github.com/jmespath/go-jmespath/issues/29


adding search and pagination to new Relic component

Hey, I just made a Pull Request!
I have added Search and Pagination to the New Relic's Application Performance Monitoring table, which makes it easy to search for applications rather than finding them over the long list of applications.It's a small fix.

:heavy_check_mark: Checklist

[ ] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[x] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

Before

After

Hey @shvmbisht you'll need to add a patch changeset for this package. more-info

Nevermind I went ahead and did it for you :)

@benjdlambert Thanks ✌🏼


Simplify change address generation parameter in balanceTx

[x] Replace GenChange s => ArgGenChange s -> ... with ChangeAddressGen s -> ...

Comments

Not sure if a State monad is preferable, but can be added later and this still a step towards it

Issue Number
ADP-2268 - Believe I will need to use a dummy change address generation function for fee estimation

bors r+

Actually, I suspect we probably even want this:
data ChangeGeneration s = ChangeGeneration
    { genChangeAddress :: State
    , changeState0 :: s
    , maxLengthAddress :: Address
    }

changeAddressesFromCyclingList :: [Address] -> ChangeGeneration ()

walletChangeAddressGeneration :: (GenChange s, BoundedAddressLength k) => s -> Proxy k ->  ChangeGeneration s

To replace the tree related balanceTx arguments:
    :: BoundedAddressLength k
    => ChangeAddressGen s
    -> s
    ->

with just one ChangeGeneration s.

bors r+


apply plugin: 'io.fabric' cause getResourceAsStream null

Hi, there:
I create a empty flutter project using Android Studio 3.4, build.gradle for android is:
buildscript {
    repositories {
        google()
        jcenter()
        maven { url 'https://maven.fabric.io/public' }
    }

    dependencies {
        classpath 'com.android.tools.build:gradle:3.4.0'
        classpath 'io.fabric.tools:gradle:1.+'
    }
}

allprojects {
    repositories {
        google()
        jcenter()
    }
}

rootProject.buildDir = '../build'
subprojects {
    project.buildDir = "${rootProject.buildDir}/${project.name}"
}
subprojects {
    project.evaluationDependsOn(':app')
}

task clean(type: Delete) {
    delete rootProject.buildDir
}

build.gradle at app like this:

def localProperties = new Properties()
def localPropertiesFile = rootProject.file('local.properties')
if (localPropertiesFile.exists()) {
    localPropertiesFile.withReader('UTF-8') { reader ->
        localProperties.load(reader)
    }
}

def flutterRoot = localProperties.getProperty('flutter.sdk')
if (flutterRoot == null) {
    throw new GradleException("Flutter SDK not found. Define location with flutter.sdk in the local.properties file.")
}

def flutterVersionCode = localProperties.getProperty('flutter.versionCode')
if (flutterVersionCode == null) {
    flutterVersionCode = '1'
}

def flutterVersionName = localProperties.getProperty('flutter.versionName')
if (flutterVersionName == null) {
    flutterVersionName = '1.0'
}

apply plugin: 'com.android.application'
apply from: "$flutterRoot/packages/flutter_tools/gradle/flutter.gradle"

android {
    compileSdkVersion 28

    lintOptions {
        disable 'InvalidPackage'
    }

    defaultConfig {
        // TODO: Specify your own unique Application ID (https://developer.android.com/studio/build/application-id.html).
        applicationId "xxxxx.flutter_crashlytics_test"
        minSdkVersion 16
        targetSdkVersion 28
        versionCode flutterVersionCode.toInteger()
        versionName flutterVersionName
        testInstrumentationRunner "android.support.test.runner.AndroidJUnitRunner"
    }

    buildTypes {
        release {
            // TODO: Add your own signing config for the release build.
            // Signing with the debug keys for now, so `flutter run --release` works.
            signingConfig signingConfigs.debug
        }
    }
}

flutter {
    source '../..'
}

dependencies {
    testImplementation 'junit:junit:4.12'
    androidTestImplementation 'com.android.support.test:runner:1.0.2'
    androidTestImplementation 'com.android.support.test.espresso:espresso-core:3.0.2'
}
apply plugin: 'io.fabric'

then at MainActivity.java like this:

public class MainActivity extends FlutterActivity {
  @Override
  protected void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    GeneratedPluginRegistrant.registerWith(this);

    ClassLoader classLoader= this.getClass().getClassLoader();
    Log.d("MainActivity",classLoader.toString());
    InputStream is=classLoader.getResourceAsStream("assets/data.bin");
    int i=0;
  }
}

then inputstream is will be null, but if I comment this line
#apply plugin: 'io.fabric'
the inputstream is ok.
please give a hand as I struggle for many days to figure out this problem and my project don't have too much time leave.
Thanks
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, v1.2.1, on Mac OS X 10.14.4 18E226, locale en-CA)
[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
[✓] iOS toolchain - develop for iOS devices (Xcode 10.2.1)
[✓] Android Studio (version 3.4)
[✓] Connected device (1 available)

@dragon2934 was it resolved?

yes, you can close it
On Wed, Oct 16, 2019 at 9:12 AM Stanislav Bondarenko <
<EMAIL_ADDRESS>wrote:

@dragon2934 https://github.com/dragon2934 was it resolved?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/flutter/flutter/issues/31906?email_source=notifications&email_token=AE2CZ5L55PNAE2KEGFO6PSTQO4HMPA5CNFSM4HJTZYOKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBMNJPQ#issuecomment-542692542,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AE2CZ5KH3F6L2AVWBNFQDBTQO4HMPANCNFSM4HJTZYOA
.


Documentation needs updating

Need to update document "In-place upgrades do not support migrations from BIOS to UEFI " This can be done now using MBR2GPT.
https://docs.microsoft.com/en-us/windows/deployment/mbr-to-gpt

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

ID: a7ce50b9-cb28-a5ba-58f9-010e69f4fde5
Version Independent ID: a161fed8-151a-1eca-8663-93e9365877d2
Content: Upgrade Readiness requirements (Windows 10)
Content Source: windows/deployment/upgrade/upgrade-readiness-requirements.md
Service: unspecified
GitHub Login: @jaimeo
Microsoft Alias: greglin

@greg-lindsay I think this is in your area.

@officedocsbot assign @mypil

@astzum I have looked into this issue and the content in question refers specifically to in-place upgrades from Windows 7 and Windows 8.1 to Windows 10. The documentation on MBR2GPT states  that "Windows 7, 8, or 8.1 are not officially supported. The recommended method to convert these disks is to upgrade the operating system to Windows 10 first, then perform the MBR to GPT conversion." I hope you will agree that In this specific context the content need not be changed.

@astzum - Thank you for submitting this feedback.
From our understanding, the issue has been resolved. If you feel it hasn't been resolved, please re-open this issue.
Thank you for your contribution to make the docs better! Much appreciated!

@officedocsbot close