not render fill-pattern ???

plugin not render fill-pattern from sprite ???
I show on web, it ok.
If use vector_map_tiles plugin, it show black color

fill-pattern is not implemented
feel free to submit a PR against https://github.com/greensopinion/dart-vector-tile-renderer


Question related to controls

Hi, i have run the code but i want to use the controls in my application. How can i achieve it? i tried to add Mechanism.AvaloniaUI and Mechanism.AvaloniaUI.Sample solution in my project and copy pasted the MainWindow.xaml from Mechanism.AvaloniaUI.Sample into my project but the output was not clean
. Thanks in advance.

Have you added the Mechanism styles to your App.xaml/App.axaml?

Have you added the Mechanism styles to your App.xaml/App.axaml?

It works. thanks for the quick response.

It works. thanks for the quick response.

It works. thanks for the quick response.

Not a problem, glad to hear it's working now.

It works. thanks for the quick response.

Not a problem, glad to hear it's working now.


docs: update libraries-bom

@chingor13

Tets failure is likely a flake:
[ERROR] Failures:
[ERROR]   ITFirewallTest.tearDown:101->BaseTest.waitForOperation:118 Operation failed: Error{errors=[Errors{code=RESOURCE_IN_USE_BY_ANOTHER_RESOURCE, location=null, message=The network resource 'projects/gcloud-devel/global/networks/test-firewall-network-d3c40e2d' is already being used by 'projects/gcloud-devel/global/firewalls/test-firewall-network-d3c40e2d-wj4bksyacgri4ifo5hsu5q6u-3'}]}
[INFO]


Fix notebook examples

Looks good.  Can you revert my recent changes to landsat here as well?

Looks good. Ready to merge?

Yes, ready to merge.


Refactor to avoid channels and goroutines

In terms of features, the addition of AfterFunc is the main benefit here. Otherwise it addresses a number of bugs, mainly around out-of-order or duplicate delivery of events, as exposed by the various tests added by these changes.
These changes merge the sleeper and the fakeTimer task. Now a fakeTimer object is used to represent any future event on the timeline of a given fake clock. When the time advances to that point, a callback on the fakeTimer gets called. This makes sending to a channel only one of many possible responses on expiry. Not using channels, but instead an internal callback that gets called with the clock's lock held, allows using the timer far more flexible. It is now also the basis of ticker and of the After method. This change in general will do more things while holding the clock's lock, leading to more consistent behavior and less risk of data races.
Basing After on Timer instead of the other way round also allows clearing the future events when timers get stopped or reset. This in turn helps BlockUntil actually track active blockers, ignoring obsolete ones that used to exist in the past.
This fixes https://github.com/jonboulle/clockwork/issues/42.

Oh!   I see this has AfterFunc too.  Can this get merged?   Then #45 can be closed w/o merging.

@sagikazarmark could you tag a new release with these changes?
the old code (v0.3.0) leavings zombie go routines if .Stop() wasn't called on a timer

Unfortunately this change seems to introduce some regressions. Waiting for those to be fixed.


Can't find a way to parameterize AzureRM Subscription in Azure CLI tasks to make them reusuable between subscriptions

When using the Azure CLI task in Release Management you have to specify the AzureRM Subscription as part of the task arguments.  This is a selectable dropdown list that contains the list of Azure subscriptions that have been connected to VSTS.
I can't find a way of making this into a variable or parameter so that the task can be re-used across multiple subscriptions.  Am I missing something or is this not an option?  If it isn't an option, could it be made one?
Many thanks in advance.

@samattridge That is not an option as of now. Adding @RoopeshNair to take a call on this

@samattridge , currently Azure connection type, Azure subscription fields are of complex types which cannot be parameterized. In other words, you cant use a variable like $(subscription) in the release definitions. It has to be set from the drop-down.

Thanks @N-Usha.  So this means we need to duplicate any tasks that use the Azure CLI Preview type for each subscription we want to use them in?  Seems rather counter-intuitive.  I'd obviously prefer not to do this so I'll have to find a work-around.

@samattridge: Have you found a workaround for this? We are stuck in the same situation...

We are in a situation where we have multiple developers - with multiple connected MSDN Azure subscriptions - registered as services in TFS. If I wanted to deploy a new Web App (and related assets) to my MSDN, I need to create a duplicate Release Management environment, and switch all of the ARM deployment tasks to my subscription. Everyone will have to do this, then our UAT/Production environments will point to our enterprise subscription.
If any tasks change, this is now a manual effort to keep them all in sync. If we could use a variable to pass in the service/subscription ID, that would make this a lot more painless.

We are sorry for the trouble you are having Mitchell. We have now added the feature enhancement to our backlog and would soon be addressing it soon (exact timelines are not available though for now). Will keep you posted once its available. Thanks!

Any update on the timeline for this feature?
We use taskgroups to ensure its the exact same way we run some of the release tasks through out the different environments.
Obviously, we dont have production and development in same Azure subscription - so this is definitely a must for us.
Thanks.

@mqlkmd, In your taskgroup specify a $(subscriptionNameVar) or something similar for the subscription field. This will expose the subscription field as an input parameter for the task group. Now in each of your environment, you will be able to specify a different subscription endpoint.

@RoopeshNair , are you sure this is working? I've tried to set the variable both as the subscription id and as the service enpoint name (like its listed if you use the normal picklist), but to no avail. The error is something regarding endpoint url not found.
I think the subscription picklist selection is also setting some hidden fields like endpoint url and auth settings and thats why its not working with a variable.
@N-Usha do you have any updates on this issue?
Thanks.

@mqlkmd, where are you setting the variable? in task group UI or in the build/release definition? You need to specify $(foobar) in the endpoint field in the task group UI. If it isn't working for you, drop a note on https://developercommunity.visualstudio.com/spaces/21/index.html

@RoopeshNair , I have tried both from the release def. and from the task group where I put the vsts variable in the "Azure Subscription" field of the task. The value Im providing for the variable is the endpoint id, which is the "id" field when looking at the REST api:
       {             "data": { "deleted": "for simplicity"             },             "id": "5574c7d9-bdf4-6cb4-8e3b-9f2di4ne4e87",             "name": "My Service Endpoint Name",             "type": "azurerm",             "url": "https://management.azure.com/",             "createdBy": { "deleted": "for simplicity"             },             "authorization": {                 "scheme": "ServicePrincipal"             },             "isReady": true,             "operationStatus": {                 "state": "Ready",                 "statusMessage": ""             }         },
Anyways, I will post the issue on the link you sent.
Thanks for your help.

@RoopeshNair , btw I have only tried this in the "Azure Powershell" task. Does it matter whether its the "Azure Powershell", "Azure CLI" or "Azure App Service Deploy" task?

No it shouldn't matter. btw, I requested to post on dev community so that we can share screenshots :)

@RoopeshNair , thanks. I've posted the question on VS community site and share the link here if anybody else are interested: https://developercommunity.visualstudio.com/content/problem/149337/use-a-vsts-variable-to-set-subsriptionservice-endp.html

For those interested, for me this actually works if you are using the "Azure App Service Deploy" task in your task group.
If you dont use that and only have "Azure Powershell" tasks, there is a workaround.
I posted my findings and the workaround on the issue here: https://developercommunity.visualstudio.com/content/problem/149337/use-a-vsts-variable-to-set-subsriptionservice-endp.html


[ua|ru] added translations starts_with and date_equals, uk.json

Added starts_with and date_equals translations to both languages (ru|ua).
Added "verify email address" to uk.json, double checked with what it actually used for in Laravel.
Removed items from todo.

Thank you


Error found on _check_if_valid_collateral_for_market

I was trying to use IncreaseOrder and got stuck at the OrderArgumentParser
The error was:
if collateral_address == market['long_token_address']
I took a quick look and found the solution

Line 290 was missing the market_key
I'm still not able to use the IncreaseOrder for other errors, but I'll let you know if I get it fixed.

Hey @diegodargos1 I'm facing the same error as well:
Traceback (most recent call last):
  File "/home/shubham/Desktop/exponential/compiler/core/deploy/template/test.py", line 143, in <module>
    open_position(config=setup_gmx())
  File "/home/shubham/Desktop/exponential/compiler/core/deploy/template/test.py", line 120, in open_position
    ).process_parameters_dictionary(
  File "/home/shubham/Desktop/rsi-algo/rsi-algo/lib/python3.10/site-packages/gmx_python_sdk/scripts/v2/order/order_argument_parser.py", line 77, in process_parameters_dictionary
    self.missing_base_key_methods[missing_key]()
  File "/home/shubham/Desktop/rsi-algo/rsi-algo/lib/python3.10/site-packages/gmx_python_sdk/scripts/v2/order/order_argument_parser.py", line 220, in _handle_missing_collateral_address
    if self._check_if_valid_collateral_for_market(collateral_address) and not self.is_swap:
  File "/home/shubham/Desktop/rsi-algo/rsi-algo/lib/python3.10/site-packages/gmx_python_sdk/scripts/v2/order/order_argument_parser.py", line 293, in _check_if_valid_collateral_for_market
    if collateral_address == market['long_token_address'] or \
KeyError: 'long_token_address'

I found that I see the same error on any OrderArgumentParser config:
    parameters = {
        "chain": 'arbitrum',

        # the market you want to trade on
        # "index_token_symbol": "BTC",
        "index_token_symbol": "SOL",

        # token to use as collateral. Start token swaps into collateral token
        # if different
        # "collateral_token_symbol": "BTC",
        "collateral_token_symbol": "SOL",

        # the token to start with - WETH not supported yet
        # "start_token_symbol": "USDC",
        "start_token_symbol": "USDC",

        # True for long, False for short
        "is_long": True,

        # Position size in in USD
        "size_delta_usd": 5,

        # if leverage is passed, will calculate number of tokens in
        # start_token_symbol amount
        "leverage": 2,

        # as a decimal ie 0.003 == 0.3%
        "slippage_percent": 0.003
    }

except for the (BTC, BTC, USDC) pair as commented above. Is this an SDK bug? How do I go about this? cc: @snipermonke01

This has been fixed with this commit and will be included in the next release of the pip package. Currently available on the staging branch.
https://github.com/snipermonke01/gmx_python_sdk/commit/85ca4961187a0a72ab6015808f5845760149ac9c


gtk4: Add an unsafe-assume-initialized feature

This is useful for building a cdylib/staticlib with C ABI bindings, where gtk_init() will be called from outside Rust.
See also https://github.com/gtk-rs/gtk3-rs/pull/762 for GTK3.

I think this makes sense

Could you document the new feature as part of the gtk4 crate's readme https://github.com/gtk-rs/gtk4-rs/blob/master/gtk4/README.md#features-1?

Added it to the features table in the README.md.

Thanks!


[IRGen] Use EnumImplStrategy to generate getEnumTag function for layo…

…ut strings
rdar://110794898
The implementation in TypeLayout seems to have a bug causing wrong tags to be returned on 32 bit systems.

@swift-ci smoke test


Canceling since a higher priority waiting request for 'e2e-nodejs-push-main-disttag-slsa3' exists

https://github.com/slsa-framework/example-package/actions/runs/5498370673
This causes the status badge for this test to show that it's failing.

We can probably use the paths filter to only trigger on updates to certain files that correspond with the right test.
https://docs.github.com/en/actions/using-workflows/events-that-trigger-workflows#running-your-workflow-only-when-a-push-affects-specific-files

Related discussions

https://github.com/orgs/community/discussions/41518
https://github.com/orgs/community/discussions/5435


[Supernova profile] light-curve visualisation

Show in the light-curve detections and points taken into account by the classifier (do not pass quality cuts) for clarity. Not urgent but nice as visualisation.

Now implemented.


Prerender localize routes

🚀 Feature request
Prerender localize routes

[x ] builders

I have a use case that my routes are localize, like:
en/about
pt/sobre
es/acerca
Describe the solution you'd like
Could be a possibility to have in the file that we specify for the prerender builder with the /en or /pt
This will allow the builder to generate only the one that is specific for this language
Describe alternatives you've considered
The alternative is have one builder per language, but this case, I have 24 languages, it will be a huge config file because I need one for browser and one for server builds...

How are localized routes being registered in the Angular app? IE: what is the logic determines if route about or sobre is registered in the router?

Thanks for reporting this feature. However, you didn't provide sufficient information for us to understand how in your case routes are being localized.


Don't install md5sha1sum

PR Summary:
Same as https://github.com/mosdef-hub/mbuild/pull/660

https://dev.azure.com/mosdef/mosdef/_build/results?buildId=492&view=logs&j=98ace938-b225-5d59-3f1f-66f906980a26&t=cefec5f8-d842-5498-9136-6a27eb08fa5f&l=199
Since we removed the examples, some tests will fail if they import mbuild.examples
This looks like test_apply_residues and test_independent_residues_molecules


Keystroke Visualizer - Feature

Description of the new feature / enhancement
Tool to display/visualize your keystrokes.

Video reference
I am aware there are already apps that can do this for Windows, like Carnac and Keystroke Visualizer, however they look outdated and out of place in Windows 10 and specially Windows 11.
Scenario when this would be used?
When screen recording, presenting code, collaborating with others, streaming.
I would see this feature be super usefull for developers that stream for teaching purposes, which popularity has increased these past 2 years.
Supporting information
A perfect example would be these MacOS apps:

KeystrokePro
Keystroker

/dup #981


Similar stories

Recommend similar stories using machine learning to create onward journeys

Link to feature: https://wellcomecollection.productboard.com/feature-board/planning/features/18356113

Closing as no longer valid.


implement map sectors

allow the usage of map sectors to allow the user to understand a map more quickly. this tasks is composed of:

[x] allow capability of API mock or other way to mock development data (because the API is not ready yet; see https://github.com/mangelio/app/issues/78)
[x] create concept of room types & according colours
[x] display map sectors to the user and allow interaction

Labels would be nice, we already read that out

Label added in da4dec4.

It would also be nice (as discussed) to group the issues in the issue list by the sector they're in, which shouldn't be hard at all.

Concept posted in slack; leaving issue open because of issue grouping though.

implemented at https://github.com/mangelio/app/blob/d20c17ede258e77bad61af928c7a26a0ac1c2fe4/src/Service/FileSystemSyncService.php#L555-L599


Remove premature post tagging from ingester

Right now, the ingester is responsible for scanning hash tags and the media status of a post and then assigning it a "tag" from a pre-selected list. This means when we add new hash tags for new feeds, we end up having to wait for new data to come in rather than being able to use old data. It feels like the classification is essentially too premature.
What I'd suggest instead is we start extracting more "general" data about a post and deprecate the tags field. I'd like to see the following new columns:

hashtags - I think there's still a big performance benefit in us storing hashtags as an array column rather than doing a text search during feed generation. But rather than only supporting a fixed list, we should extract all hash tags within the post and normalize them (e.g lowercase them).
media_attached - A simple boolean value indicating if the post has any media.
raw - The JSON representation of the raw resource we received from the firehose. We probably won't query on this as part of standard feed generation, but we can use it for migrations in future which add new extracted fields.

I think with these new columns, we'll still have good performance but we'll be more able to design new feeds on the fly without needing to modify the ingester and wait for posts matching that description to be made.

Don't worry about backfill - we'll just run it for a good few days with the new system before switching over generation.


Expand the invite group when for a new invite

Whe a invite is received in the "Massages" screen, a new goup "INVITES" appears but it is not expanded

It has been with the latest commit.
The client defines it as a preference i.e. each group visibility is defined by a dedicated preference


Update README.md

@bbowman,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

@PatrickFarley

https://github.com/bbowman/project-rome/tree/bbowman-readme shows the new version. Looks like you updated a lot recently and I'd need to update for MSGraph. Thoughts on the table layout instead of just the links?

Thanks, I will get back to you on this shortly

@PatrickFarley updated.

It looks good, but I do have some concerns, let's discuss at Monday's meeting

What do you think about making the "API Docs" column just "Documentation", and then linking to the general platform folders? If we link to the Android/ folder instead of Android/reference%20documentation/, then users will notice the how-to guides as well as reference docs. Same for the Windows/ folder. And these folders all have brief READMEs that will display by default, which will help users find what they're looking for.

seems reasonable. My main concern was keeping everything up to date. By landing people at the folder instead of directly on the content (especially for Windows SDK or MSGraph where we don't directly host the documentation), we introduce another thing to keep in sync / update and most of the content on the README is going to be a duplicate of what the actual page would show/explain. I do like that we'd have the how-to guides have a bit more visibility though. @cmknox thoughts?

I've resolved the conflicts locally and pushed. closing this PR


Error in how polygons are drawn

There is an error in the way polygons are draw. In short, the path defining the polygon is somehow not really being closed, and so linejoins are not rendered correctly. Here's an example showing smooth line joins being rendered jaggedly.
http://share-elm.com/sprout/54e64591e4b09711f39c2b0d

Fixed by using cap <- Padded in the above example
http://share-elm.com/sprout/54e90d3be4b09711f39c2c88

I wouldn't call that "fixed", it's just a different thing since the corners are sharp rather than smooth.

Fair. Seems like it isn't treating the lower right corner as a join, and is basically just drawing it as a path. Not quite sure where to fix this in Graphics.Collage.

Is the fix "you can't draw closed polygons with lines"?
Currently the internal representation of Forms does not have any special cases, but it'd be good to have for circles and rectangles probably. I'd like to do a big set of improvements like this that includes supporting arcs, bezier curves, etc. but I think it'll be a while until then unfortunately :/

Sorry this did not get attention til now! The Graphics.* modules now live in evancz/elm-graphics so I am trying to get stuff migrated over there.
I don't really know what action to take on this one though, so unless there is a proposed fix, I'm not sure how valuable it is to keep this open.


CAMEL-8817: Expose additional geolocation search api.

https://issues.apache.org/jira/browse/CAMEL-8817
The current camel-twitter component only supports geography searches using the streaming API which only returns results that have latitude and longitude values.
The standard search also supports geolocation queries and can return results that do not have latitude and longitude metadata but are geocoded by twitter and determined to be within a geolocation.
It would be ideal if this API was also exposed.
Patch and pull request to be provided shortly.

Thanks for the PR the code has been merged. Do you mind closing this?


edit users and campaign improvements

Motivation and context
This PR adds ability to edit users from Admin role and extends the campaign description to 40K chars.
Some small text styling and UX fixes too.

increased campaign description to 40K symbols due to html expansions
improved campaign creation by opening create beneficiary/company in new tabs
added: users grid and edit form for user changes

wrong branch - closing


Bug reported: Tensorflow GPU version(Cuda) may cause thread blocking when calling scripts across languages.  C#.

Actually I'm not sure whether it's a bug 'cause I've solved my problem, but i'll still trying to make a report.

Basically, I wrote a py script to generate a TFLite model.
Trying to run the test code, and worked good.

import threading
from tensorflow import keras
import tensorflow as tf
import numpy as np
import sys
import time
import asyncio

def load_interpreter(model_dir:str):
    interpreter = tf.lite.Interpreter(model_path=model_dir)
    interpreter.allocate_tensors()
    return interpreter
def classify_single_image(img_dir:str, interpreter:tf.lite.Interpreter):
    start_time = time.time()
    img = keras.preprocessing.image.load_img(img_dir, target_size=(160, 160))
    img_array = keras.preprocessing.image.img_to_array(img)
    img_array = np.expand_dims(img_array, 0)
    input_details = interpreter.get_input_details()
    output_details = interpreter.get_output_details()
    interpreter.set_tensor(input_details[0]['index'], img_array)
    interpreter.invoke()
    output_data = np.squeeze(interpreter.get_tensor(output_details[0]['index']))
    probs = tf.nn.softmax(output_data)
    probs = np.round(probs * 100, 2)
    elapsed_time = time.time()
    result = "{}+{:.2f}".format(probs, (elapsed_time - start_time) * 1000)
    print(result, flush=True)

interpreter = load_interpreter("trash_model_lite.tflite")
threading.Thread(classify_single_image("1.jpg", interpreter)).start()
threading.Thread(classify_single_image("2.jpg", interpreter)).start()
threading.Thread(classify_single_image("3.jpg", interpreter)).start()
threading.Thread(classify_single_image("4.jpg", interpreter)).start()
threading.Thread(classify_single_image("5.jpg", interpreter)).start()
# 很显然，如果继续这样做, interpreter 不是线程安全的. 需要加锁.

i'm tring to call it in my C# project just by openning python and let it to run the script
Problem is that C# cannot get any result, and i'll give the code in the end 'cause this is a tf bug report.
i'm trying to see where's the problem, just by print("n", flush=True) after every slice of code.
The result shows that the tensorflow module will "BLOCK"(maybe not, but will stop here) the program such as:

    print("0", flush=True) # will see 0 in the python console and C# console
    img_array = tf.expand_dims(img_array, 0)
    print("1", flush=True) # will see 1 printed in the python console but not in C# console

trying to avoid tensorflow will exam the hypothesis:

    print("0", flush=True) # 0 in python and C#
    img_array = np.expand_dims(img_array, 0)
    print("1", flush=True) # 1 in python and C#
......
    probs = tf.nn.softmax(output_data) # still wrong

I suddenly understand that the tensorflow module might be made a mistake, so i'm tring to install cpu version and get the correct answer by only work with cpu.

import os
os.environ["CUDA_VISIBLE_DEVICES"]="-1"
import tensorflow as tf
......

After delete the print code, i've got the almost same answer in both C# and python, and which is correct after deleting the empty line.

[  0.   0.   0. 100.]+960.62
[9.997e+01 0.000e+00 0.000e+00 2.000e-02]+31.92
[  0. 100.   0.   0.]+35.90
[ 0.35  0.2  41.75 57.7 ]+32.91
[  0.   0. 100.   0.]+33.91

[  0.   0.   0. 100.]+382.50
[9.997e+01 0.000e+00 0.000e+00 2.000e-02]+31.99
[  0. 100.   0.   0.]+31.91
[ 0.35  0.2  41.75 57.7 ]+31.96
[  0.   0. 100.   0.]+32.90

If i delete the code which forbide the cuda, the C# program will still wrong. So the right thing is to just work on cpu, and i think this maybe a bug( or config problem maybe)

THESE ARE MY SYSTEM CONFIG:
OS: Windows 10 from Microsoft
Python: 3.8, download in python.org
Tensorflow: GPU==2.4.1, CPU==2.4.1( installed after the bug became) with tflite, download with pip
Keras: download with TF
IDE or Text Editor: VSCode, Visual Studio
Cuda: 11.0
Cuddn: the right version with cuda 11.0
GPU: Laptop gpu -- MX250
Bazel: No bazel.
Config with C#:
.Net: .Net 5 or .Net Core 3.1
Project: dotnet new Console
Nuget packages: None
C# code :
using System;
using System.Diagnostics;

namespace ConsoleApp1
{
    class Program
    {
        static void Main(string[] args)
        {
            string[] args_array = new string[2];
            string pyDir = "test.py";
            args_array[0] = "";
            args_array[1] = "";
            RunPythonScript(pyDir,"", args_array);
        }
        public static void RunPythonScript(string sArgName, string args = "", params string[] teps)
        {
            Process p = new Process();
            p.StartInfo.FileName = @"C:/Program Files/Python38/python.exe";
            string sArguments = sArgName;
            foreach (string sigstr in teps)
            {
                sArguments += " " + sigstr;
            }
            sArguments += " " + args;
            p.StartInfo.Arguments = sArguments;
            p.StartInfo.UseShellExecute = false;
            p.StartInfo.RedirectStandardOutput = true;
            p.StartInfo.RedirectStandardInput = true;
            p.StartInfo.RedirectStandardError = true;
            p.StartInfo.CreateNoWindow = true;
            p.OutputDataReceived += new DataReceivedEventHandler(p_OutputDataReceived);
            p.Start();
            p.BeginOutputReadLine();
            p.WaitForExit();
            Console.ReadLine();
        }
        static void p_OutputDataReceived(object sender, DataReceivedEventArgs e)
        {
            if (!string.IsNullOrEmpty(e.Data))
            {
                AppendText(e.Data + Environment.NewLine);
            }
        }
        public delegate void AppendTextCallback(string text);
        public static void AppendText(string text)
        {
            Console.WriteLine(text);
        }
    }
}

And i hope you could solve your problem like this. That's why i made this report. Have a good time! :)

Hi,
Thank you for opening this issue. Since this issue has been open for a long time, the code/debug information for this issue may not be relevant with the current state of the code base.
The Tensorflow team is constantly improving the framework by fixing bugs and adding new features. We suggest you try the latest TensorFlow version with the latest compatible hardware configuration which could potentially resolve the issue. If you are still facing the issue, please create a new GitHub issue with your latest findings, with all the debugging information which could help us investigate.
Please follow the release notes to stay up to date with the latest developments which are happening in the Tensorflow space.


KAFKA-4831: Extract WindowedSerde to public APIs

Now that we have augmented WindowSerde with non-arg parameters, extract it out as part of the public APIs so that users who want to I/O windowed streams can use it.

@huxihx @mjsax @guozhangwang
Unfortunately, I cannot assign reviewers for some reason, so that's why I am requesting a review manually.
Thank you.

@mjsax You also mentioned that we may want to add a windowed serde in Serdes.java, right?
Also that part may need some careful thought as the WindowedSerde still need an internal Serde and how to specify that in the Serdes factory is tricky.

Just double checked the JIRA. Yes, having WindowedSerde would be idea. As it should be type agnostic, user would need to inherit from it I guess to have a generics free UserWindowSerde -- thus WindowSerde should also be abstract. We can add a usage pattern example in the JavaDocs. Does this make sense (please correct me, did not think about it in detail...)

@guozhangwang @mjsax ok, I got it! I thought WindowedSerde is just an abbreviation for the concept.
Will refactor this.

@bbejeck @huxihx @vitaly-pushkar I have pushed to this PR with new commits. Ping for reviews.
I will update the above description of this PR soon.

Overall looks good, left some comments, mostly nits.

@guozhangwang Are you working on a KIP already? This public API change requires one.

@guozhangwang Are you working on a KIP already? This public API change requires one.

Yes, that's the plan. While doing that I'd also like to have your first opinions on the public APIs summarized in 1) in description above.

The proposal from above sounds good to me.


#88 #89 #90 set up the basic workflow classes with demo

Includes a refactor of the code originally laid out in #2 (now closed) adds a demo of the basic app behaviour at /apply/demo/.

I don't know much about this "workflows" feature, there doesn't seem to be any information on codebase about it. But the code generally looks fine to me (except for few issues I have with the demo_workflow view, but I assume that's only temporary).


Support Vault as a backend for st2kv

Vault is an easy sell as a verified secure KV store in high security environments. It would be nice to be able to use a Vault deployment as a backend for st2kv.

I agree a plugin is a good way to cover this and allow for more then just hashi vault, though I think it would be simpler to leave it to the plugin to organise its own token be that from a file like st2kv or from an instance profile/Azure MSI or any other trust system. I think this will allow for an easier implementation.

We've written a custom vault function within our deployment that makes secret retrieval from Vault a breeze.  IIRC, @xorso brought it up a while back (maybe in the Slack channel?) but because it requires upgrading Oslo it was turned down.

How easy is it to add to the jinja filters? So at the moment you have {{ st2kv.* }}  is it difficult to add jinja filters to the engine for say {{ hashivault.* }} or {{ mycustomsecretengine.* }} ?

I'm not sure I understand, but here's an example of a task:
get_device_id_from_fstab:
    action: core.remote
    input:
      cmd: grep -iw <% ctx().mount %> /etc/fstab | awk '{print $1}'
      hosts: <% ctx().cdb_host %>
      username: oracle
      password: "{{ vault_get('data/dbsa/operations/test/oracle', 'st2.dba_refresh', mount='kv') }}"
      cwd: /tmp/
      timeout: 180

Where the first param is the secret path (in this case it's a kv2 secret), the second parameter is the st2.kv entry that holds the approle, and the last parameter is the secret mount.

This is good, but doesn't address the case of using a thirdparty pack which uses st2kv and transparently mapping that to vault.

+1 with @cognifloyd.  Vault or any external secret provider would require a new API.

I was looking at this over the weekend, would anyone object if we left st2kv as it was? Then created a new class called st2secret and then an abstract class you can write backends to? Doing this would mean st2kv users wouldn't have a breaking change, the community could then make a decision to move st2kv into the new framework if there is a desire.

Is there any reason we can't make st2kv a shim to different backends?
How do we ensure that third-party packs using st2kv get appropriately
redirected to vault to get their secrets?
On Sun, Feb 20, 2022, 4:41 PM chris3081 @.***> wrote:

I was looking at this over the weekend, would anyone object if we left
st2kv as it was? Then created a new class called st2secret and then an
abstract class you can write backends to? Doing this would mean st2kv users
wouldn't have a breaking change, the community could then make a decision
to move st2kv into the new framework if there is a desire.
—
Reply to this email directly, view it on GitHub
https://github.com/StackStorm/st2/issues/5401#issuecomment-1046336811,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABPWAWSAD622K2FM6IIFPVDU4FUZDANCNFSM5GMA7HZA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

The main concern I have is looking at the references to the st2kv code there seems to be a lot of dependencies on it. The st2 Key Vault also has a wider scope of use cases it can host unencrypted keys as well. Whilst I'm not opposed personally and in the past have suggested exactly what you have I do think there is some difference between the 2.
For example st2kv uses mongo to hold the keys and pulls the symmetric key from a file on the disk. In the secrets plugin IAM would likely be used to remove the need for persistent token storage and secrets would ideally be pulled at the last moment and only be in ram for a short time, in either case this would be down to the plugin to manage.

My thoughts on this is we repeat the design pattern used in st2auth, but yes that's essentially what I would like to do too.

I'd like to see Vault as st2kv backend as well but I can also see tha the feature parity doesn't align.
I see another option here - do not use Vault as a backend for the kv storage but rather as a backend for encrypted values. Vault has something called "transit engine" which essentially works as "encryption-as-a-service" which could be leveraged for encrypted vaules while keeping the unencrypted vaules untouched.
The downside of this is that one would still need to manage the content of the st2kv store independently though.


Specify default Vault backend node

Without passing -node, Vault will panic:
panic: runtime error: index out of range

goroutine 1 [running]:
panic(0xb9ece0, 0xc820010040)
	/usr/local/go/src/runtime/panic.go:464 +0x3e6
github.com/kelseyhightower/confd/backends.New(0x0, 0x0, 0x0, 0x0, 0x7fff1a2aee58, 0x5, 0x0, 0x0, 0x0, 0x0, ...)
	/Users/kelseyhightower/go/src/github.com/kelseyhightower/confd/backends/client.go:61 +0xbdf
main.main()
	/Users/kelseyhightower/go/src/github.com/kelseyhightower/confd/confd.go:27 +0x1f4

LGTM

@bacongobbler Thanks. Any idea when there's going to be a new release?
It's been a while since the last one (47 commits):
$ git describe --tags
v0.12.0-alpha3-47-gc6622ed

@mlafeldt I have no idea when the next release is coming out. It depends on when kelsey's available to cut one. I'm just a maintainer :)

I see. So maybe @kelseyhightower knows? :)


Avoid spurious recompilations when unrelated constructor with default argument changes

Issue
Constructor sometimes has name <init>$default$ instead of <init>. Hence two unrelated constructors have the same name.
#288 ensured that two unrelated <init> constructor have different names by prepending constructor name with class name, however #288 did not handle <init>$default$.
Fix
Prepend <init>$default$ with the class name.
c.c. https://github.com/sbt/zinc/pull/288, https://github.com/sbt/zinc/issues/578#issuecomment-417599988

I am deliberating on whether the PR is actually a good change to have.
This particular name <init>$default$ is rare, while the extra logic introduced by this PR adds overhead to all Def handling. It also makes API phase slightly more confusing.

I shall request for review and let reviewer determine if this is actually good to have.


auro-modal: remove enforced header

Is your feature request related to a problem? Please describe.
When importing content from other resources, there is a conflict to the header of the dialog. In these cases, the header is actually in the content and the pre-formatted dialog header is not needed.
Describe the solution you'd like
If there is no header content in the slot, this area will collapse. Much in the same way that the footer does.
A dialog in the Auro site with the current support:

With the suggested update

This update is requesting input from design prior to making updates.

How will the dialog label be set in this case?
Currently we set aria-labelledby to the id of the h1 w/ the slot (irrelevant attributes removed). If this is empty or removed, the dialog no longer has a label.
 <dialog aria-labelledby="dialog-header">

      <slot name="header">Default header ...</slot>

Options I see:

Programmatically determine (get the first heading element in the slotted content)
Have the user pass aria-label or aria-labelledby as a prop. Log a warning/error if heading not provided and label not provided.

Have the user pass aria-label or aria-labelledby as a prop. Log a warning/error if heading not provided and label not provided.

The issue there assumes that either the user has control over the content or there is an ID on the header node. In the use case I illustrated I have neither.

Going with this, we probably need a more robust management of the dialog header. Meaning, that if this content node is not addressed, there should be an error.
In addition to this, we can still populate this node, then the request moves from ignoring the content node to hiding the node from sight.

Yeah, we definitely need something for the label. So I like the idea of requiring header to be populated for a11y, while exposing the option to visually hide it.
Even if the consumer doesn't control the content, I would assume that they have some idea of what content is in the modal and can provide a decent enough label for it.


[QUESTION] How to set OMP_NUM_THREADS=12 for Matlab in Ubuntu

Have you read the documentation?

[x] Yes
[ ] No - then this issue will be closed.

Post your theoretical questions / usage questions here.
Dear author,
Thank you for your code.
I know how to set OMP_NUM_THREADS=12 in a C++ code, and it surely will avoid the PMC crash problem. Now I want to use teaser++ in matlab with the matlab bindings. is there a way to set OMP_NUM_THREADS=12 for matlab? I have tried using "set OMP_NUM_THREADS=12" in a Ubuntu command shell before opening matlab, but it didn't work.

Try the latest develop branch. I fixed the issue with thread count issue so you don't have to set the environment variable. Make sure to clean all the CMake build folders, reload CMake project and rebuild.

Thank you! I am not sure if I build it right or not, but when the number of points gets over 10000, Matlab would  crash. I will use the C++ code, it works good for me. Thanks again.

Yes due to the implementation the memory consumption will be large with high number of points/correspondences.


Github Action: reference master branch

Task/Issue URL: https://app.asana.com/0/414730916066338/1204943033123149/f
Description
Steps to test this PR
Feature 1

[ ]
[ ]

UI changes

Before
After

!(Upload before screenshot)
(Upload after screenshot)

Current dependencies on/for this PR:

develop

PR #3294   👈

This comment was auto-generated by Graphite.


Zero mAP and no detections on custom dataset

Dear Haotian Liu,
currently I'm trying the yolact_edge trained on custom coco-like dataset for one class ('person'). Contrary to coco the image resolution is 512.
When I run evaluation with TensorRT conversion I get the following error during the protonet conversion:
[02/23 15:26:23 yolact.eval]: Converting protonet to TensorRT...
Traceback (most recent call last):
  File "eval.py", line 1241, in <module>
    convert_to_tensorrt(net, cfg, args, transform=BaseTransform())
  File "/home/oidpsv/yolact_edge/utils/tensorrt.py", line 156, in convert_to_tensorrt
    net.to_tensorrt_protonet(cfg.torch2trt_protonet_int8, calibration_dataset=calibration_protonet_dataset, batch_size=args.trt_batch_size)
  File "/home/oidpsv/yolact_edge/yolact.py", line 1565, in to_tensorrt_protonet
    self.trt_load_if("proto_net", trt_fn, [x], int8_mode, batch_size=batch_size)
  File "/home/oidpsv/yolact_edge/yolact.py", line 1534, in trt_load_if
    module = trt_fn(module, trt_fn_params)
  File "/opt/conda/lib/python3.6/site-packages/torch2trt-0.1.0-py3.6-linux-x86_64.egg/torch2trt/torch2trt.py", line 555, in torch2trto
    engine = builder.build_cuda_engine(network)
  File "/opt/conda/lib/python3.6/site-packages/torch2trt-0.1.0-py3.6-linux-x86_64.egg/torch2trt/calibration.py", line 51, in get_batch
    buffer[i].copy_(tensor)
RuntimeError: The size of tensor a (69) must match the size of tensor b (64) at non-singleton dimension 2

which can be fixed by changing line 1563 in yolact.py from x = torch.ones((1, 256, 69, 69)).cuda()
to x = torch.ones((1, 256, 64, 64)).cuda().
The problem is that in this case further evaluation with TensorRT conversion gives zero mAP and processing of images provides empty result (no masks or boxes). Could you please help me? Many thanks.

What is the mAP if without TensorRT? Also did you set the classes to ('person') or ('person',), the first one is incorrect.

It is in the range of 70-80 without TensorRT. Also, the classes are set as ('person', ) label map is = { 1: 1,}

If you try evaluate with FP16 TensorRT, will the AP improve?

Please try the latest code base in haotian-dev branch.

Hi @oidpsv ，I also encountered this problem. Have you found a solution now？

@XYunaaa Please see the newest code base and use the safe mode as explained here.

@haotian-liu sorry to bother you.
I have encountered the similar issue.
But in my situation,I replaced the backbone with rexnetv1 (https://github.com/clovaai/rexnet/blob/master/rexnetv1.py).
It can evaluate normally when I disable tensorrt.
But when I want to convert the model with tensorrt command, it showed the error.

And I changed the line 1512:
x=torch.ones((1,256,69,69)).cuda() -> x=torch.ones((1,256,112,112)).cuda()

The error will not showed , and I add the following code in trt modules.
elif cfg.backbone.name == "MobileNetV2":
            x = [
                torch.randn(1, 32, 69, 69).cuda(),
                torch.randn(1, 64, 35, 35).cuda(),
                torch.randn(1, 160, 18, 18).cuda(),
                ]
        elif cfg.backbone.name == "MobileNetV3":
            x = [
                torch.randn(1, 24, 69, 69).cuda(),
                torch.randn(1, 40, 35, 35).cuda(),
                torch.randn(1, 40, 18, 18).cuda(),
                ]
        elif cfg.backbone.name == "Rexnet":
            x = [
                torch.randn(1, 16, 69,69).cuda(),
                torch.randn(1, 27, 35, 35).cuda(),
                torch.randn(1, 50, 18, 18).cuda(),
                ]

    if cfg.backbone.name == "ResNet50" or cfg.backbone.name == "ResNet101":
            x = [
                torch.randn(1, 256, 69, 69).cuda(),
                torch.randn(1, 256, 35, 35).cuda(),
                torch.randn(1, 256, 18, 18).cuda(),
                ]
        elif cfg.backbone.name == "MobileNetV2":
            x = [
                torch.randn(1, 256, 69, 69).cuda(),
                torch.randn(1, 256, 35, 35).cuda(),
                torch.randn(1, 256, 18, 18).cuda(),
                ]
        elif cfg.backbone.name == "MobileNetV3":
            x = [
                torch.randn(1, 256, 69, 69).cuda(),
                torch.randn(1, 256, 35, 35).cuda(),
                torch.randn(1, 256, 18, 18).cuda(),
                ]
        elif cfg.backbone.name == "Rexnet":
            x = [
                torch.randn(1, 256, 69,69).cuda(),
                torch.randn(1, 256, 35, 35).cuda(),
                torch.randn(1, 256, 18, 18).cuda(),
                ]

        if cfg.backbone.name == "ResNet50" or cfg.backbone.name == "ResNet101":
            x = torch.randn(1, 512, 69, 69).cuda()
        elif cfg.backbone.name == "MobileNetV2":
            x = torch.randn(1, 32, 69, 69).cuda()
        elif cfg.backbone.name == "MobileNetV3":
            x = torch.randn(1, 24, 69, 69).cuda()
        elif cfg.backbone.name == "Rexnet":
            x = torch.randn(1, 16,69,69).cuda()

The setting  format about Mobilenetv3 and Rexnet refer the code.
The result model with MobilenetV3 backbone  looks great ,and it didn't show any error with tensorrt .
So I think maybe the problem is caused by my setting about torch.randn
Could you tell me how to set the torch.randn in correct way ?
How to define the parameter in torch module ?
Thanks for your reading~
Have a nice day.

@ntut108318099 You should put in torch.randn(n,c,h,w) where it is the expected shape of the tensor for all the images when passing through the module.

@haotian-liu  Thanks for your reply.
Sorry,
Could you tell me where can I expect shape of the tensor for all the images when passing through the module ?
Which code can I used to saw the result ?
Thanks for your help.

@ntut108318099
In each module, e.g. FPN_phase 1, it will have some inputs def forward(self, x1, x2, x3), we will put x = [ torch.randn(*x1.shape), torch.randn(*x2.shape), torch.randn(*x3.shape) ] there. Replace them with actual numbers.

@haotian-liu
Depending on your  advice, I convert the trt module successfully.
But it showed the another error , it's about   shape '[1 ,25575, 81]' is invalid for input of size 1559088.

Have you ever met this error before ?

@haotian-liu Hi,Sorry to bother you.
I have fixed the problem .
The problem was caused by "PredictionModuleTRTWrapper" . I didn't notice that have some parameter also need to set  .
The result looks great !!
Thanks for your supply.
Have a nice day .

@ntut108318099 Glad you have the problem solved!


Possible to change parameters without a full re-render?

I am using this library to create a dynamic background, and am trying to get it so the color of the particles shifts every few seconds. Currently it seems changing the parameters on the <Particles /> component causes a full re-render and causes the particles to appear to "jump".
I've created a sandbox demo that shows that here
Would it be reasonable to have certain parameters (like color) not cause a full re-render?
I've poked through the code a bit, but was curious if you had an opinion @Wufe.

Unfortunately it is not possible to change parameters without a full re-render.
A while ago I was thinking about implementing a more complex way of defining particles to be able to add, remove and change particles at runtime without causing a full reload.
This idea did require a lot of work, and I made an experimental branch and a little website showing this ( https://www.timeon.space/ ).
However this experimental branch is not usable: it was a draft.
If taken seriously, this idea would require tackling more challenges, like defining the entity "particle system" which should be capable of orchestrating single particles.
Trying to change the behavior of the library for certain parameters only, seems to me like exception-driven development.
Let me know what you think about it.

@Wufe Do you have particle config for this effect (https://www.timeon.space/)?

@rainstormza the website timeon.space is not powered by the current version of react-particles-js.
It is an experimental branch you can find it here but it is not usable, it was just a prototype.


error when testing a picture that has a low resolution?

I use a 294x78 PNG to test grounded_sam2_hf_model_demo.py, but I get following errors, any solution?
  File "D:\project\Grounded-SAM-2\sam2\sam2_image_predictor.py", line 417, in _predict
    low_res_masks, iou_predictions, _, _ = self.model.sam_mask_decoder(
  File "D:\program\anaconda3\envs\llama_adapter\lib\site-packages\torch\nn\modules\module.py", line 1553, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)
  File "D:\program\anaconda3\envs\llama_adapter\lib\site-packages\torch\nn\modules\module.py", line 1562, in _call_impl
    return forward_call(*args, **kwargs)
  File "D:\project\Grounded-SAM-2\sam2\modeling\sam\mask_decoder.py", line 136, in forward
    masks, iou_pred, mask_tokens_out, object_score_logits = self.predict_masks(
  File "D:\project\Grounded-SAM-2\sam2\modeling\sam\mask_decoder.py", line 203, in predict_masks
    assert image_embeddings.shape[0] == tokens.shape[0]
AssertionError

Would you like to provide the demo image you use for us to test this issue, which may be more convenient for solving this problem. It seems like SAM 2 did not support image with very low resolution.

I find the problem. The input_boxes shape is [0,4] after the  processor.post_process_grounded_object_detection， so I add if(xx) to avoid this situation. Thanks.

I find the problem. The input_boxes shape is [0,4] after the processor.post_process_grounded_object_detection， so I add if(xx) to avoid this situation. Thanks.

It seems like grounding_dino detect nothing in this image, I think it's better to add some warnings here for debugging, we will try  to update our code in the future release

Seems like the problem has been solved, I will close this issue first, feel free to reopen it if needed.

Hi, @baiyuting
so I add if(xx) to avoid this situation
In which code and where you add this logic?

Hi, @baiyuting so I add if(xx) to avoid this situation
In which code and where you add this logic?

I think it means to set a detection to verify the input_boxes.shape, like if it's [0, 4]. It shows detect nothing.

Hi,
I am using the grounded_sam2_local_demo.py. For this code, I do not see the boxes. shape. The assertion error occurs when the tokens[0]==0. That means the grounding dino is probably not predicting anything. If it does not predict anything, how can we pass the error?


Bug: Downvote mana/indicator

Downvote always showing 100% while actual value is low.

@feruzm If I downvote 2 different posts with 2 different values and try to downvote third one, third one  has 0 default value. What should it be? the latest downvoted value?

@mghazanfar this is about indicator not voting value. Click on user menu after login or click on i icon on profile page, you will see information about downvote percentage.


Mercenary rogue forces, charges, cooldowns and special units

Summary of a discussion from the discord:
Rogue Forces
Talk point
There is some concern that this item is too powerful, to the point that some players choose to disable it.
Notes
Making it a progressive upgrade that adds charges, with the Rogue Forces we know being the last step should work. One must be careful with giving too many charges. Otherwise, it wouldn't make a difference in practice between having the Rogue Forces and a lot charges.
Cooldowns
Talk point
There's been suggestions for an item that may reduce the cooldowns to call mercenaries. However, there is no agreement on how much is a reasonable amount.
Notes
A flat amount benefits shorter cooldowns while percentual amounts benefit longer cooldowns.
Special Units
Talk point
Some desire the possibility of having mercenaries that are good "out of the box" and be more than just a "unit with better stats". The amount of each type of special units should be limited to one per map (like a mothership).
Notes
Some want to move existing mercenaries (Jackson and Jotun) to this category and give them some changes. Note that a unit type (for example a Battle Cruiser) could have both a Special (ie. Pride of Augustgrad) and Regular (ie. Sovereign Battlecruise) rmercenary variants.

Discussion goes through these topics around here:

I'm still not a fan of Rogue Forces, it is one of the upgrades I always exclude.  I think that one upgrade completely destroys the balance of the mercenaries and significantly limits their design space

--Alice

The thing is that Rogue Forces essentially adds 1-2 charges for the most of the missions

The thing is that Rogue Forces essentially adds 1-2 charges for the most of the missions

On live, yes, and I agree that's a nice place for it. If you get more than that, it's because the mission is going long enough that you could probably use the extra power and still have fun.
With the other upgrades added, it adds a lot more. Fast Delivery essentially adds 1 to all Rogue Forces cooldown, bringing us to 2~3 charges depending on money and the unit. Rapid Reinforcement at 50% doubles the effectiveness of Rogue Forces, which is way too much. Changing Rapid Reinforcement to -60s makes it far more reasonable, essentially adding ~1 charge effective value to Rogue Forces and allowing you to scale up faster without Rogue Forces. I think Fast Delivery is reasonable as it is now as you tend to be money-limited at the start of a mission anyways, though I haven't played enough yet for that opinion to be final.
I would like to make Rogue Forces not apply to "ultimate" mercenaries like Jackson's Revenge, Wise Old Torrasque, or any future BC mercs we add. That means having multiple of them is more valuable, we can have upgrades for them, and can keep their hiring cost relatively low without Rogue Forces breaking the balance again.


Move ValueList to functions/lib/aggregates for Spark reuse

The array_agg function in Spark and Presto has some inconsistent behavior
when handling null values. We need to re-implement Spark's array_agg in Velox.
To be able to reuse ValueList, it needs to be moved to the
functions/lib/aggregates directory first.
The copyValueListToArrayWriter method has also been moved to the
ValueList, making it reusable.

The array_agg function in Spark and Presto has some inconsistent behavior when handling null values. We need to re-implement Spark's array_agg in Velox.

What is the difference in behavior between Presto and Spark?

What is the difference in behavior between Presto and Spark?

The semantics are generally consistent, but there are inconsistencies between Presto and Spark in the handling of null values. Spark always ignores null values in the input, whereas Presto preserves them. Moreover, Presto returns null when all inputs are null, while Spark returns an empty list or set.
I plan to rewrite Spark's array_agg based on the simple aggregate function interface.

Hi @mbasmanova, there is a draft PR https://github.com/facebookincubator/velox/pull/9231 for Spark's array_agg, you could get more details from it. I plan to split the move of ValueList and the implementation of Spark's array_agg into two PRs, although they could be combined into a single PR.

@liujiayi771 Thank you for clarifying. Presto's implementation depends on presto.array_agg.ignore_nulls config flag. Does Spark implementation match Presto's when presto.array_agg.ignore_nulls = true?

@liujiayi771

When all inputs are null, the output of Spark is an empty array, while Presto returns null.

Got it. Thank you for clarifying.
CI is red. Please, take a look.

@liujiayi771 It might be #9200

@Yuhta @mbasmanova Did we encounter any issues with the merge? Do I need to rebase with the latest code?

@Yuhta @mbasmanova Did you encounter any issues with the merge? Do I need to rebase with the latest code?

There were a few smaller internal fixes required since the header got moved, but I'm merging it today.


Reorder the assets

Example:

assets/images/team/smaranjit_ghose.png
assets/images/about/open_source.svg

i would like to work on this issue.


Legacy JDBC instantiation

Community Edition 7.0.4 has a check-box "Use legacy JDBC instantiation" on the Create New Driver dialog.  I have a driver that can't be loaded when this is selected.  What exactly does this do to load a driver?

Hi @gordonthorpe ,
in the recent version this checkbox is unchecked by default. It is needed for custom drivers only.


Is it normal for the script to be here?

Is it normal for the script to be here? To after use the Terminal commands, it is safe to be deleted?
Note: I'm on M3 with Sequoia 15.2 I just found this and testing it. This issue still persists on Sequoia

Issue solved.


[REQUEST] Explosion upon death

When the player reaches 0 health, their ship will explode and a game over screen will be shown.
Describe the solution you'd like
Upon the health reaching zero, the camera zooms in on the players ship as final alarms blare. An explosion consumes the player's ship. The enemies stop shooting. On the place where the player's ship once was, after the explosion has dissipated, the text "Game Over" should appear as a fade in. Once that appears, the options "Main Menu" and "Exit" are shown as buttons, allowing the player to go back to the main menu or quit the game respectively.
Additional context
The explosion should be large and dramatic. Ideally, it would almost appear in slow motion, being drawn out over 3-4 seconds with lots of sparks, fire, everything just shooting out from this. It will be the focus on the screen, consuming not only the entire player ship, but the majority of the screen as it zooms in. As mentioned in the discord, see FTL, Battlefleet Gothic 2.

Task Estimation (time): 6h (guess)
Does this issue address a risk? Not exactly but it'll be a shitty game if we don't have something like this in it.
Risk: While I have used Unity's particle sim system before, it was only in a very basic way, so it might take me a while to get a good-looking explosion.
Risk Resolution: If Unity's system is too difficult, I might be able to use Blender.  I'll also ask people if they have any exp doing related things.
Risk: Timing of camera zoom/explosion/UI stuff might get dodgy, the concept is simple but the execution might get tricky.
Risk Resolution: As ever, if I get stuck/confused/etc, I'll ask for help.


observer kits don't work

maps like boombox or cannon duel have a obs kit that gives night vision, it works for  players that have joined, but not for observers

Cardinal version please, and isn't this a duplicate of #467 ?
Also, could you show us the XML? If you are using the included Cannon Duel XML, note that the kit "obs+" does not exist.

does the kit name matter? it should be given to observers if it's stated to, in ocn  observers do get the effect

i checked the xml again, cannon duel has the spawns like this:

so it gives the players the kit obs+, and that kit is on: house-of-core.xml

in HoC xml:

night vision

and yes, i do have HoC xml, and it's on "../house-of-core.xml"

I fixed your XML links :)

last line of my coment: -> and yes, i do have HoC xml, and it's on "../house-of-core.xml"


PCB request

Hello,
i was searching for other PCB-producers and the agent suggested to update the PCB design: the vias on the attached file should be moved away from SMD pads to guarantee assembly quality.
Can i send it in production as-is or does the gerberfile needs to be changed?
Thank you very much
MS

This was my first time doing any PCB layout work, so I am open to any and all suggestions.
I will move those vias and post updated gerbers. However, JLCPCB did produce me a batch of trackers based on the exact gerbers I put in the PCB folder, and they seem to work okay. I have not been able to flight test them yet though. (I did successfully flight test an earlier rev of my PCB design, that was slightly less congested).

Thank you very much, i will post the name of the other PCB-producer if the tests are ok


add SSR support for the dev server

Based off of work done in an internal project with @lukeed and in https://github.com/Rich-Harris/snowpack-svelte-ssr with @Rich-Harris
Changes

Add an experimental API for the dev server (exposed as import {unstable__startServer} from 'snowpack')
Add SSR support for the dev server via this new API.
Add SSR support for plugins, and adds support to the svelte plugin specifically.

Testing

Not exposed to the user, but tested manually in the projects mentioned above.

@rich-harris This file is honestly the best take on importing ESM from the dev server directly into Node.js that I've seen or can think of, at least until those ESM loader hooks are finalized: https://github.com/Rich-Harris/snowpack-svelte-ssr/blob/master/tasks/SnowpackLoader.js
It would be out of scope of all users of SSR to implement this themselves. How do you feel about us pulling this into Snowpack directly or shipping this as a separate utility helper package to help people integrate SSR with Snowpack?

I was secretly hoping you'd suggest something like that! The main reason I didn't bring it up yet was that the result of this approach isn't totally spec compliant (cycles, live bindings) and figured you might prefer something more robust if it's to be part of Snowpack. (Also, I realised that it caches too aggressively - modules aren't invalidated when their dependencies change. Need to fix that.)
But more broadly, yeah, I think it would be super useful to have this functionality as part of the API


Created iplookup.c in sockets folder.

Worked on with Stephen Monaghan at @smonaghan. Using getaddrinfo, it is possible to find the primary IP address for a given hostname (assuming the hostname is entered correctly in command line). I'm not entirely too sure where to add a makefile, as sockets folder already contains a makefile for client-tcp/udp and server-tcp/udp. We're assuming as error checking should be implemented, so that will be added by tomorrow or Wednesday.

let's make it a bit more advanced--the program should be able to either return an IP from a hostname, or return a hostname from a given IP.  Since you get back the linked list, you might as well have it print out all of the information in the list, not just the first.

I've updated iplookup.c to handle both types of inputs, those being IPv4 addresses and hostnames. The choices of inputs are handled by flags where -h flag dictates a hostname input and a -i flag dictates an IPv4 address input. I've also edited it so that searching for IPv4 address from hostname lists all found addresses.

Some formatting issues, missing comments, and descriptions of input/output examples were added.

overall the code is pretty good.  You still have some formatting issues (tabs instead of spaces) and you should get used to using a better command line argument processing function like getopt.  I'm accepting this for now and we'll fix those small issues later.


🐛 BUG & 🌟 Feature : No Colored Logs in wrangler2 dev and console.dir(object, { depth: null }) shows redacted output

What version of Wrangler are you using?
0.0.19
What operating system are you using?
Linux
Describe the Bug
The logs in wrangler2 dev are so creepy and the behaviour is weird sometimes also. Things like:
console.dir(object, { depth: null });

Doesn’t output full object. Also colors are missing.

Needs Feature Request Tag also

Closing as a duplicate of #188


Raise AttributeError in lightning_getattr and lightning_setattr when attribute not found

What does this PR do?
Fixes #5932.
Description of the changes

Make lightning_getattr and lightning_setattr raise AttributeError but not ValueError.
Add death tests for AttributeError.
Rename lightning_get_* to _lightning_get_* because they are supposed be used within the file.

Before submitting

[x] Was this discussed/approved via a GitHub issue? (not for typos and docs)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together?
[x] Did you make sure to update the documentation with your changes? (if necessary)
[x] Did you write any new necessary tests? (not for typos and docs)
[x] Did you verify new and existing tests pass locally with your changes?
[x] Did you update the CHANGELOG? (not for typos, docs, test updates, or internal minor changes/refactorings)

PR review
Anyone in the community is free to review the PR once the tests have passed.
Before you start reviewing make sure you have read Review guidelines. In short, see the following bullet-list:

[x] Is this pull request ready for review? (if not, please submit in draft mode)
[x] Check that all items from Before submitting are resolved
[x] Make sure the title is self-explanatory and the description concisely explains the PR
[ ] Add labels and milestones (and optionally projects) to the PR so it can be classified

Did you have fun?
Make sure you had fun coding 🙃

@akihironitta mind check the comments from @rohitgr7

@Borda @rohitgr7 The match string is added to the tests now, so would you mind having a look at the changes again?


KAFKA-2998: log warnings when client is disconnected from bootstrap brokers

Is it relatively simple to add a test for the new behavior? I think this change might also solve KAFKA-3468.

Filed https://issues.apache.org/jira/browse/KAFKA-3508 for unit test failure.

LGTM. Merged to trunk.


Add bazel-xcode-demo-swift-driver demo project

I don't know if you'd like to include Bazel demo project, but if you do here's the Bazel/Xcode/Swift demo project one I recently shared.

Good idea, thanks!


Use SpringResourceAccessor instead of ClassLoaderResourceAccessor

When running a JAR produced by the Spring Boot Gradle Plugin. The JAR that is produced will not run correctly due to the autoconfiguration using the ClassLoaderResourceAccessor. In the Spring Boot JDBC version of the autoconfiguration SpringResourceAccessor is used, which is aware of the structure of the Spring Boot produced JAR so is able to find the liquibase changelogs.

Hello @surecloud-slloyd
Sorry, was very busy with ongoing project
I will check it ASAP

Okay I will merge it, but as far all maven central nexus required configurations aren't present on my current laptop, I will publish it later

@surecloud-slloyd
new version 2.1.9 with your changes has been released.
Soon it should be available here: https://repo1.maven.org/maven2/io/github/daggerok/liquibase-r2dbc-spring-boot-starter/2.1.9/


Add $$ $$ and $  $ highlight?

It seems that the $ for math mode aren't highlighted. And it would be great help(I think?) to highlight them.

They are, your colorscheme just needs to set @text.math.
(And $$ is not valid LaTeX but plain TeX and therefore not recommended.)

Note that this repository just provides the parser and not the actual highlighting queries.
The rules to parse math mode can be found here:
https://github.com/latex-lsp/tree-sitter-latex/blob/6b7ea839307670e6bda011f888717d3a882ecc09/grammar.js#L445-L453

Yes, this is an nvim-treesitter issue (or, rather, a colorscheme issue, since we do have queries for those nodes).

I can't even get syntax highlighting to work inside the dollar signs at all :(. Is it currently broken or am I doing something wrong?