How do you send a notification to a specific installation template (.NET)?

I am registering installations from my .NET backend server code with multiple templates each. For example:
var installation = new Installation
{
    InstallationId = id,
    PushChannel = token,
    Templates = new Dictionary<string, InstallationTemplate>(),
    Tags = new List<string> { "userId:123456" },
    Platform = NotificationPlatform.Gcm
};

installation.Templates.Add("template1", new InstallationTemplate { Body = "{\"data\":{\"message\":\"$(message)\"}}"});
installation.Templates.Add("template2", new InstallationTemplate { Body = "{\"data\":{\"message2\":\"$(message)\"}}"});

await _client.CreateOrUpdateInstallationAsync(installation);

How do I target a specific template when sending a notification? All I see in the SDK is the following:
await _client.SendTemplateNotificationAsync(
    new Dictionary<string, string>
    {
        { "message",  "Hello world." }
    }, "userId:123456");

The SendTemplateNotificationAsync method does not have any parameters that let me specify which template I am targeting (for example, template2).
Which template will be used? Am I misunderstanding something here?

This is probably too late for you but I was trying to figure out this exact same issue and hopefully this will help someone else. From this page, it says: "Send the notification as a template notification. All template registrations that contain "messageParam" and the proper tags will receive the notifications. This includes APNS, GCM, WNS, and MPNS template registrations.". So if you have two templates that contain: $(messageParam) they will both get the message and who knows which one will actually win.
But when I viewed the list of Device Registrations in my Notification Hub, I noticed that even though I didn't send through any tags, that the template name was listed in the tag column. So I changed my code from this:
var result = await hub.SendTemplateNotificationAsync(templateParams);
to this:
var result = await hub.SendTemplateNotificationAsync(templateParams, templateName);
And I got the desired result by specifying the template name as a Tag.

I'm having the same iussue, but with the template name as tag it works properly.
It seems reasonable because i can send a notification to a specific tagList (or tag expression) specified into Intallation and to template name: so, if a device is subscribed to a specific topic (tag) with the specified template (template name) the notification is sent to a correct device with his correct template.

@satowine1 do you mean you set up one installation per template?
Or do you mean just by virtue of having named your templates, you can automatically use those names in tag expressions?

@johnkwaters i can use the template name as tag into tag list and it works properly :)
Like this:


Failed to activate the one-dark-ui theme

Having a different theme activated (Isotope UI), attempt to switch to One Dark UI.
Observe Errors popping up, without changing the theme:

Note that this keeps coming up, even if I reload or completely restart the editor.
However, I can change to Atom Dark or Light without a problem.
Atom Version: 0.199.0
System: Microsoft Windows 8.1 Pro
Thrown From: one-dark-ui package, v0.8.1
Stack Trace
Failed to activate the one-dark-ui theme
At Recursive variable definition for @button-background-color in C:\Users\free\AppData\Local\atom\app-0.199.0\resources\app.asar\node_modules\one-dark-ui\index.less:28:4

LessError: Recursive variable definition for @button-background-color
  at C:\Users\free\AppData\Local\atom\app-0.199.0\resources\app.asar\node_modules\one-dark-ui\index.less:28:4

Commands

Config
{
  "core": {
    "disabledPackages": [
      "minimap",
      "enhanced-package-list",
      "themed-settings"
    ],
    "themes": [
      "one-dark-ui",
      "kuali-dark-syntax"
    ]
  }
}

Installed Packages
# User
coffee-navigator, v0.0.16
file-icons, v1.5.5
git-log, v0.3.0
kuali-dark-syntax, v1.3.0
symbols-tree-view, v0.9.2

# Dev
No dev packages

I just noticed that I also get it for One Light UI, but I'm holding off opening an issue on that repo until I know more about this one.

Seems like kuali-dark-syntax conflicts with One light/dark. Other syntax themes are fine.

Ok, I think the problem is that Kuali Dark has a variable called @bg https://github.com/murphyrandle/atom-kuali-dark-syntax/blob/master/styles/colors.less#L9 and then One dark/light also have the same @bg variable: https://github.com/atom/one-dark-ui/blob/master/styles/buttons.less#L38
Does Less have scoped variables to just a ruleset?

So the scopes of UI and syntax themes can pollute each other?

Yeah, if they import each others variables and use the same variable names there is a change they conflict. Usually it should just override each other, but maybe in this case it was problematic because the @bg variable was used inside a mixin that got called from different places.

Interesting.
Indeed if I switch the syntax theme to something else first, One UI works ok again.


fix(config): support MCO Home MH-S412 parameters properly

Checklist
It was not merged recently nor has a pending PR
Device
Mco Home Technology Co. LTD MH-S412
Manufacturer ID
0x015f
Product Type
0x4121
Product ID
0x5102
Is the device listed on the Z-Wave Alliance website?
No
Do you have a manual?
Yes
http://mcohome.com/index.php?c=Front/DownDetail&a=downloadFujian&name=MH-S412&path=L2NvbWRhdGEvOTUzMDEvcHJvZHVjdGZ1amlhbi8yMDIxMDIwMzE3Mjc0NjYwMWE2YzkyYjIxMTQucGRm

Just to be sure - do you have this exact device and can confirm the parameters?
Because there also seems to be a 700 series (?) version with product ID 0x7102, but the manual on the website is not clear for which generation it is.

Absolutely! I've confirmed this particular device works correctly with the proposed parameters in this PR. I have the device physically and tested the changes locally.
As a side note, I previously used this device with Jeedom and noticed a regression in behavior after switching to ZWaveJS. This prompted me to investigate and submit this PR.


Add transaction content-type to history network

Initial stab at updating history spec to reflect thoughts from issue #73

outdated.


harmonization for n_layer==1

sequential layers are [dropout(0.1), linear, batchnorm, relu]. Before, we concatenated [latent_features, one_hot_batch_index] as input to the sequential layer decoder, therefore submitting one_hot_batch_index to the dropout as well, and giving poor batch mixing entropy results. We customize the forward function to circumvent that problem.

also simplifying the modules surrounding FCLayers. The additional feature is the ability to easily add edges from categorical variables in the graphical model, by specifying n_cat in the construct for each variable as a list (n_cat_list), and providing them as a sequence in the forward function.

Latent space vis. as supporting evidence ! (BE ~=0.6)

Looks like I merge+delete the branch a little fast so that the second check didn't pass (cannot find the branch related to pull request), but https://travis-ci.org/YosefLab/scVI shows the merge tests on master passed successfully.
All is OK in theory

This order is common: Linear, BatchNorm, ReLU, Dropout.
Also, I usually wouldn't use any dropout or BatchNorm on the final layers of the encoder and the decoder.


[BUG] Rebuild() throws System.Threading.SynchronizationLockException

Version
Which LiteDB version/OS/.NET framework version are you using. (REQUIRED)
Latest version from master, commit 6c0886ec063f8bd272a91dd87973863e4668550d
Describe the bug
A clear and concise description of what the bug is.
Calling Rebuild() on a LiteDatabase instance results in a SynchronizationLockException.
Code to Reproduce
Write a small snippet to isolate your bug and could be possible to our team test. (REQUIRED)
var db = new LiteDatabase(new ConnectionString("Lite.db")
{
    Connection = ConnectionType.Direct
});

try
{
    db.Rebuild();
    Console.WriteLine("Rebuild succeeded.");
}
catch (Exception e)
{
    Console.WriteLine(e);
    throw;
}

Expected behavior
A clear and concise description of what you expected to happen.
Should rebuild the datafile without throwing an exception.
Screenshots/Stacktrace
If applicable, add screenshots/stacktrace
System.Threading.SynchronizationLockException: The write lock is being released without being held.
   at System.Threading.ReaderWriterLockSlim.ExitWriteLock()
   at LiteDB.Engine.LockService.ExitExclusive() in /Users/jens/Documents/Projects/LiteDB/LiteDB/Engine/Services/LockService.cs:line 140
   at LiteDB.Engine.LiteEngine.Rebuild(RebuildOptions options) in /Users/jens/Documents/Projects/LiteDB/LiteDB/Engine/Engine/Rebuild.cs:line 87
   at LiteDB.LiteDatabase.Rebuild(RebuildOptions options) in /Users/jens/Documents/Projects/LiteDB/LiteDB/Client/Database/LiteDatabase.cs:line 273
   at LiteDB.Sandbox.Program.Main(String[] args) in /Users/jens/Documents/Projects/LiteDB/LiteDB.Sandbox/Program.cs:line 23

Additional context
Add any other context about the problem here.
As far as my testing goes, the bug was introduced in commit 3bda0f068de9295ff033628976c82a524f2995fd as my code snippet was still working when I was testing with commit 8c956c541a9414ada9700f996475ea8a1847e216

@JensSchadron Could you test with the current master? The transaction model was entirely rewritten.

I'was following the progression of PR #1560 "live". But was going to close this ticket when it got merged into master.
That being the case, I'll close this ticket as it works just fine now.


RGA layer creates only logfile

I am trying to create a dump with VK_LAYER_RGA_pipeline_extraction on Windows 10 with a Vulkan <IP_ADDRESS> runtime. Application is Vulkan 1.0 compatible.
The application is utilizing VkSpecializationInfo (with a several dozen parameters, typical uber shader) and push constants, but aside from that nothing unusual.
I have all the documented environment variables set: ENABLE_RGA_PIPELINE_EXTRACTION_LAYER=1;RGA_LAYER_LOG_ENABLE=1RGA_LAYER_OUTPUT_PATH=C:\RGADump
VK_LAYER_RGA_pipeline_extraction is the only active layer, and I can confirm that it actually intercepts all the relevant API calls, for several pipelines, render passes etc.
The VK_EXT_debug_marker extension is not available (and hence also unused).
Yet this is the only output it ever creates:

***Convertion started ***
version 1
Handling samplers.
Handling descriptor set layout.

The log file stops there and no warnings or anything whatsoever during application shutdown.

Hi Ext3h,
Thanks for reporting this issue.
Will you be able to:

Set the following environment variable: RGA_LAYER_DUMP_FOSSILIZED=1 and rerun the app with the layer enabled. This should generate a fossilize.json file in the output directory.
Share fossilize.json (note: that file will contain compressed blobs of your shader binaries).

fossilize.zip
Ok, got a fossilize dump.

I have the same problem

any updates? I found out that Vulkan examples by Sascha Willems work fine, but only when API version set to 1.0
My app however doesn't work event with 1.0 API

Found a solution! - I had some sync issues in my app, discovered by Khronos sync validation layer. One issue was in texture staging barrier, another in render pass, missing dependency. Visually all was good. After fixing the sync issues pointed out by sync layer, the RGP layer works!

Thanks for sharing your use case and solution, @pmathia0.


[BUG] - When navigating with keys on autocomplete component with sections it does not highlight the currently selected item

NextUI Version
2.2.5
Describe the bug
When navigating with keys on autocomplete component with sections it does not highlight the currently selected item.
Works fine without sections.
The issue can be reproduced on the docs.
Your Example Website or App
https://nextui.org/docs/components/autocomplete#with-sections
Steps to Reproduce the Bug or Issue
Try to navigate autocomplete with sections on the docs.
Expected behavior
I expected the currently selected item to be highlighted like on autocomplete without sections.
Screenshots or Videos
No response
Operating System Version
Windows
Browser
Edge

No longer reproducible (tested in 2.3.5).


 Is this on your roadmap/would you welcome adding this to the project?

I may consider adding it if more people demand this feature, for now you can build your own <Head> component using the head manager API. https://github.com/vueuse/head/blob/3260b3e34b18c664600b1b58747f061b3460ae70/src/index.ts#L79

I may consider adding it if more people demand this feature, for now you can build your own <Head> component using the head manager API. https://github.com/vueuse/head/blob/3260b3e34b18c664600b1b58747f061b3460ae70/src/index.ts#L79


Pin dependency so that package works again

Fixes #59 until someone can migrate to new API.

If it's fixed, why don't you merge it now

Sure!

@all-contributors please add @arcanemachine for code


How can DINOv2 takes image with different shape as input when inference?

If I have an image with a shape of 240*320, what should I do to generate the mask with same shape before and after passing the image to the pertrain DINOv2? Thanks!

Your image must be a multiple of the patch size on each dimension. In the case of DINO, this means it must be an even multiple of 14 -- try resizing the image to 238x322

Thanks for your advice!

Your image must be a multiple of the patch size on each dimension. In the case of DINO, this means it must be an even multiple of 14 -- try resizing the image to 238x322

Yes, exactly and see the last sentences of the "Model Details" section: https://github.com/facebookresearch/dinov2/blob/main/MODEL_CARD.md#model-details for reference. Closing as resolved.


Fix readme typo

Thanks for contribution, assistant typos can be fixed in https://github.com/awesome-assistants/awesome-assistants/blob/main/assistants.yml#L2711
can you fit it there?

fixed with #7


Differential stability values

Hi,
is it possible to retrieve the differential stability values for each gene that are calculated during the parcellation process?
Cheers!

Hi @gkleinman84 ! Welcome to abagen :wave:
Unfortunately the differential stability estimates aren't directly accessible from the abagen.get_expression_data() workflow; however, you can obtain a variation of them via the following:
>>> import abagen
>>> atlas = abagen.fetch_desikan_killiany()
>>> expression = abagen.get_expression_data(atlas['image'], atlas['info'], return_donors=True)
>>> stability = abagen.correct.keep_stable_genes(list(expression.values()), return_stability=True)[1]
>>> expression = abagen.samples_.aggregate_samples(expression)

Here, stability will be an array where each entry indicates the differential stability of the corresponding gene in the columns of the expression matrix. This is slightly different from the differential stability estimates calculated internally during abagen.get_expression_data(), as here the stability estimates are calculated as correlations between parcels in atlas rather than between AHBA-defined brain structures from their ontology; however, the values tend to be very similar!
Hope this helps, but let me know if you have any questions.

Hi @rmarkello,
Thank you very much for your answer! This already helps a lot. When you say "the values tend to be very similar", can you estimate whether they are less or more conservative or generally just differ slightly without a direction?
Is the last part of the code (expression = abagen.samples_.aggregate_samples(expression)) necessary for the DS values or is that unrelated?

Hi @gkleinman84 ! Sorry for the delay in getting back to this.
Yes, you're absolutely right: that last line is unrelated to the DS calculations, It's just for aggregating the donor-level expression values into a single group-average expression matrix (so you don't have to re-run the abagen.get_expression_data() workflow again). If you don't need the expression data then you can safely ignore that!

As for your question of DS value similarity: I can't say with any certainty whether one estimate is more or less conversative, but you could dig into the code to try it out yourself!
The relevant function that calculates the DS from the sample level information is abagen.probes_._diff_stability(). You could step through the main abagen.get_expression_data() workflow by hand and then modify that function to return the DS values directly, if you wanted to compare them!


Handle Worker and Algorithm failure

[ ] If the Job has failed - who will handle this?
[ ] If the Worker container has failed - what to do?
[ ] If the Algorithm container has failed - what to do?

#204 #203


Support for H7015

I got these cheap, but did not realize it does not have WiFi, I can make a WLED controller, but this would be a lot easier, for now.

How to contact them? If they can give us some info on how to extract data from the devices, we can do it ourselves.

Their Telegram contact info is on the repo somewhere.
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: baudneo @.>
Sent: Saturday, October 5, 2024 8:57:46 AM
To: Beshelmek/govee_ble_lights @.>
Cc: Bobby C. Young @.>; Manual @.>
Subject: Re: [Beshelmek/govee_ble_lights] Support for H7015 (Issue #43)
How to contact them? If they can give us some info on how to extract data from the devices, we can do it ourselves.
—
Reply to this email directly, view it on GitHubhttps://github.com/Beshelmek/govee_ble_lights/issues/43#issuecomment-2395067467, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AMMFBWWR46JQSRO7IZ5A35LZZ7V5VAVCNFSM6AAAAABKVOPHSSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOJVGA3DONBWG4.
You are receiving this because you are subscribed to this thread.Message ID: @.***>


[FLINK-23417][tests] Harden MiniClusterITCase.testHandleBatchJobsWhenNotEnoughSlot

This commit hardens the MiniClusterITCase.testHandleBatchJobsWhenNotEnoughSlot by configuring
a higher ResourceManagerOptions.STANDALONE_CLUSTER_STARTUP_PERIOD_TIME than JobManagerOptions.
SLOT_REQUEST_TIMEOUT. This ensures that the slot request times out instead of being failed by
the SlotManager.

One further comment: Seeing as the real failure reason is not visible in the test output, maybe it would be a good idea to add the actual exception (including the cause chain) to the log output in case the test fails?
This may apply to (many many...) other tests as well though :-D

In order to reproduce the problem you have to delay the registration of the TaskExecutor a bit. You could do this by adding Thread.sleep(1000L) to the TaskExecutor.onStart() method. That way you can reproduce the race condition between the TaskExecutor registration and the arrival of the slot request that would fail after the start up period is over. I hope this makes the problem a bit clearer.

One further comment: Seeing as the real failure reason is not visible in the test output, maybe it would be a good idea to add the actual exception (including the cause chain) to the log output in case the test fails?
This may apply to (many many...) other tests as well though :-D

Irrelevant now - thanks for explaining to me where to find the interesting logs @tillrohrmann


remove unneed stack & scripts

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

agree


Authenticating during repo cloning process is not succeeding.

Dev Home version
No response
Windows build number
No response
Other software
No response
Steps to reproduce the bug
Description
When trying to sign into a GitHub account through the repo cloning window, I am able to authorize PG Internal to my account, but the experience in Dev Home doesn't seem to recognize my account. Clicking Connect does not result in an error, but the next page where you are supposed to choose a repo is completely unpopulated. The pop up window screen moves past the connect screen before I give the extension access.
Expected result
No response
Actual result
No response
Included System Information
No response
Included Extensions Information
No response

Duplicate of #1099


[JS]:HTTP Request Types

Description
A Documentation on HTTP Request Types.
Topics to be included in this documentation:

What are HTTP Request Methods?
How Do HTTP Requests Work?
Different types of HTTP Request Methods

Domain
JavaScript_Domain
Type of Contribution
Documentation
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of this project.

/assign


Add Entry in wsl.conf for manual resolv.conf settings

Problem
I have to use my own Nameserver so get Internet Access which i have to add in /etc/resolv.conf.
This only works if i add the following in /etc/wsl.conf
# This file was automatically generated by WSL. To stop automatic generation of this file, add the following entry to /etc/wsl.conf:
# [network]
# generateResolvConf = false

Solution
Add an option to include
[network]
generateResolvConf = false

in the /etc/wsl.conf file

I believe this is already available via https://github.com/nix-community/NixOS-WSL/blob/3651abbfb84b3f40f09f63a515f61aa8d49608f9/modules/wsl-distro.nix#L76

I believe this is already available via
https://github.com/nix-community/NixOS-WSL/blob/3651abbfb84b3f40f09f63a515f61aa8d49608f9/modules/wsl-distro.nix#L76

Thank you. In the last Release Version this is not included.
I assume i have to follow "Building your own system tarball" to get the last Version?

Update worked as described. Thank you!


Update patch versions

Updates hotfix/2.1.4 to manage package versions with Directory.Build.props as on dev.

Bumps dependency patch versions:

Package
From
To

Microsoft.EntityFrameworkCore*
2.1.3
2.1.4

Microsoft.NET.Test.Sdk
15.7.2
15.9.0

Npgsql
4.0.3
4.0.4

Npgsql.*
1.0.*
4.0.4

SourceLink.Create.CommandLine
2.8.1
2.8.3

xunit*
2.3.1
2.4.1

One more important point, is that I think this release should be called 2.1.4 rather than 2.1.3, as it depends on EF Core 2.1.4. Even if this would leave a "hole" in our versioning, I think there's value in aligning versions with EF Core as much as possible. Let me know what you think.

That sounds great. I just created a branch for hotfix/2.1.4. I'll address these comments here, then re-open this PR against the new branch.

Updated, but note that this now depends on the availability of version 4.0.4 of Npgsql (and plugins).
So checks are expected to fail until those are released.

Updated, but note that this now depends on the availability of version 4.0.4 of Npgsql (and plugins).
So checks are expected to fail until those are released.

4.0.4 just hit NuGet, rebuilding on AppVeyor and Travis.

Test failures in EFCore.PG.Plugins.FunctionalTests.

I'm trying to update this for Npgsql.NetTopologySuite <IP_ADDRESS>, but that triggers test failures while seeding NetTopologySuiteContext in EFCore.PG.Plugins.FunctionalTests.
I'm starting to debug these now, but there's no direct comparison with dev (where the tests pass), due to the spatial support refactoring in 2.2.
Does anyone have a sense of what's happening here?
/cc @roji @YohDeadfall?
System.IndexOutOfRangeException: Index was outside the bounds of the array.
   at NetTopologySuite.Geometries.GeometryCollection.GetGeometryN(Int32 n) in C:\TeamCity\BuildAgent\work\9f3b4b5334e77626\NetTopologySuite\Geometries\GeometryCollection.cs:line 175
   at Npgsql.NetTopologySuite.NpgsqlPostGisWriter.CheckOrdinates(IGeometry geometry) in C:\projects\npgsql\src\Npgsql.NetTopologySuite\NpgsqlPostGisWriter.cs:line 512
   at Npgsql.NetTopologySuite.NpgsqlPostGisWriter.Write(IGeometry geometry, Ordinates ordinates, ByteOrder byteOrder, BinaryWriter writer) in C:\projects\npgsql\src\Npgsql.NetTopologySuite\NpgsqlPostGisWriter.cs:line 103
   at Npgsql.NetTopologySuite.NpgsqlPostGisWriter.Write(IGeometry geometry, Ordinates ordinates, Stream stream) in C:\projects\npgsql\src\Npgsql.NetTopologySuite\NpgsqlPostGisWriter.cs:line 80
   at Npgsql.NetTopologySuite.NpgsqlPostGisWriter.Write(IGeometry geometry, Stream stream) in C:\projects\npgsql\src\Npgsql.NetTopologySuite\NpgsqlPostGisWriter.cs:line 50
   at Npgsql.NetTopologySuite.NetTopologySuiteHandler.ValidateAndGetLengthCore(IGeometry value) in C:\projects\npgsql\src\Npgsql.NetTopologySuite\NetTopologySuiteHandler.cs:line 186
   at Npgsql.NetTopologySuite.NetTopologySuiteHandler.ValidateAndGetLength(IGeometry value, NpgsqlLengthCache& lengthCache, NpgsqlParameter parameter) in C:\projects\npgsql\src\Npgsql.NetTopologySuite\NetTopologySuiteHandler.cs:line 131
   at Npgsql.NetTopologySuite.NetTopologySuiteHandler.Npgsql.TypeHandling.INpgsqlTypeHandler<NetTopologySuite.Geometries.GeometryCollection>.ValidateAndGetLength(GeometryCollection value, NpgsqlLengthCache& lengthCache, NpgsqlParameter parameter) in C:\projects\npgsql\src\Npgsql.NetTopologySuite\NetTopologySuiteHandler.cs:line 176
   at lambda_method(Closure , NpgsqlTypeHandler , Object , NpgsqlLengthCache& , NpgsqlParameter )
   at Npgsql.TypeHandling.NpgsqlTypeHandler`1.ValidateObjectAndGetLength(Object value, NpgsqlLengthCache& lengthCache, NpgsqlParameter parameter) in C:\projects\npgsql\src\Npgsql\TypeHandling\NpgsqlTypeHandler`.cs:line 198
   at Npgsql.NpgsqlParameter.ValidateAndGetLength() in C:\projects\npgsql\src\Npgsql\NpgsqlParameter.cs:line 551
   at Npgsql.NpgsqlCommand.ValidateParameters() in C:\projects\npgsql\src\Npgsql\NpgsqlCommand.cs:line 800
   at Npgsql.NpgsqlCommand.ExecuteDbDataReader(CommandBehavior behavior, Boolean async, CancellationToken cancellationToken) in C:\projects\npgsql\src\Npgsql\NpgsqlCommand.cs:line 1142
   at System.Threading.Tasks.ValueTask`1.get_Result()
   at Npgsql.NpgsqlCommand.ExecuteDbDataReader(CommandBehavior behavior) in C:\projects\npgsql\src\Npgsql\NpgsqlCommand.cs:line 1130
   at System.Data.Common.DbCommand.ExecuteReader()
   at Microsoft.EntityFrameworkCore.Storage.Internal.RelationalCommand.Execute(IRelationalConnection connection, DbCommandMethod executeMethod, IReadOnlyDictionary`2 parameterValues)

Looks like this line triggers the seeding errors (though others still exist):
https://github.com/npgsql/Npgsql.EntityFrameworkCore.PostgreSQL/blob/cbbd243018f377083032e0111150950e06628498/test/EFCore.PG.Plugins.FunctionalTests/NetTopologySuiteTest.cs#L463
Here's the spatial data seeded by EF Core. It doesn't look like they seed any GeometryCollection data.

Just to keep things moving, I've opened the following PRs with patches for NpgsqlPostGisWriter and PostGisWriter:

npgsql/npgsql#2258
NetTopologySuite/NetTopologySuite.IO.PostGis#7


Missing tag for 0.2.4

The tag for the 0.2.4 release seems to be missing on GitHub and thus is not available in yarn. Can you please add/push it?

Hi! I'm sorry about this awfully late reply. Multi.js has unfortunately been deprioritized by me lately but I'm hoping to get back on it.
Is this still an issue? What kind of tag do you need added?
Thank you for your patience!

I just realised you probably meant that 0.2.4 wasn't available on npm/yarn! I just published it and it should be available. Thanks for pointing it out!


try and validate config inside every potentially breaking concat/file…

… resource

Pull Request (PR) description

This Pull Request (PR) fixes the following issues

Ok, the CI/CD tells me clearly that trying to validate each component config part individually isn't going to work, so we're going to have to try wrapping the service notify up in an exec to validate the config and avoid restarting rsyslog (and stopping logging) when the config is not valid.


Reduce DRY code in askme script

Many parts in the askme scripts are repeatable like the read calls and the validation of the user inut. We can follow the same approach the mount nnn plugin is taking, wrapping read/validate in an abstract universal function.

Fixed by #59 PR.


Add configuration files

Add configuration files
I'm closing https://github.com/aws/sagemaker-container-support/pull/23 since both have the setup.py file changes.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

can you fix the build error?

@laurenyu @nadiaya , Yuan will work in the name. Using the old name in this PR.


Reading from stdin without - using default value leads to hanging prompt

The Reading from Stdin without special characters tip using default values with clap is great when input is provided from stdin (echo "Hello, world!" | cargo run) though when nothing is provided - cargo run - the program basically freezes.

My fear is that this might not be something you can even fix or detect! If so, sorry for this dumb question xD

In this case when the program is reading from stdin (since it wasn't piped in), you'll need to press ctrl+d to send EOF.
This is a good point that can cause confusion though, I'll see if it's possible to detect and log some instruction for that

I agree piping and a program seeming to hang can be confusing, but since it's standard shell command behavior it seems nice to not print instructions by default. Maybe instructions could be an opt in option if added? Or if they're default, I'd appreciate an opt out option. 🙂

since it's standard shell command behavior it seems nice to not print instructions by default
@havenwood Agreed!

@uncenter I believe the best way to address this is to add notes to the CLI help strings and/or docs?
E.g. something along these lines
struct Args {
    ...
    /// Read from a value or stdin. If no value or stdin is provided, will block on stdin until EOF is received (ctrl+d)
   input: MaybeStdin<String>,
   ...
}

Fair enough! Thanks for looking into it.


Segmentation fault when eval =.. on REPL

Eval following =.. expression on REPL will cause segmentation fault.
Output:
% tpl
Trealla v0.1alpha, Jun  9 2017
?- T =.. L.
 L: L
Segmentation fault (core dumped)

Backtrace:
Core was generated by `tpl'.
Program terminated with signal SIGSEGV, Segmentation fault.
#0  0x000000000043e4bb in sprint2_compound (depth=1, dstbuf=0x7ffeca1cd8e0, bufsize=0x7ffeca1cd8e8, _dst=0x7ffeca1cd7f8,
    pl=0x17fc010, q=0x1820630, n=0x1821210, listing=1) at src/print.c:205
205             if (is_atom(nf)) {
(gdb) bt
#0  0x000000000043e4bb in sprint2_compound (depth=1, dstbuf=0x7ffeca1cd8e0, bufsize=0x7ffeca1cd8e8, _dst=0x7ffeca1cd7f8,
    pl=0x17fc010, q=0x1820630, n=0x1821210, listing=1) at src/print.c:205
#1  0x000000000043fbef in sprint2_term (depth=1, dstbuf=0x7ffeca1cd8e0, bufsize=0x7ffeca1cd8e8, _dst=0x7ffeca1cd8d8,
    pl=0x17fc010, q=0x1820630, _n=0x1821010, listing=1) at src/print.c:381
#2  0x0000000000440f08 in term_sprint (dstbuf_=0x7ffeca1cd910 "L", bufsize=65536, pl=0x17fc010, q=0x1820630, n=0x1821010,
    listing=1) at src/print.c:545
#3  0x0000000000408b1e in query_dump (self=0x1820630) at src/trealla.c:1092
#4  0x0000000000405432 in main (ac=1, av=0x7ffeca1de568) at tpl.c:443
(gdb)

Fix confirmed, thanks.


Elementary support for jaxpr with einsums.

Jaxpr translate the 'ij,jk->ik' to a dot_general and transpose, hence some supplement on transpose.

Looks good


fix grid neighbor diagonal issues

Unrelated to this PR, but could you make neighbourDirections a val instead of a def. As far as I can tell, making it a method is unnecessary


What type of update are you requesting?
A new locale, architecture, or installer type of an existing version
Current Package Identifier
3CX.Softphone
Package Version
18.13.959
Please describe the changes you would like to see
Support for Machine scope
This installer itself supports deploying the 3CX Softphone in the machine scope, however, the 3CX.Softphone.installer.yaml is limited to user scope.
Can this package be extended to the also include the machine scope to better suite SOE deployments?

Could you please explain how *.msi is provided on their servers? I can only navigate to Microsoft Store from PackageUrl.


Signal/SignalIterator has an infinite recursion problem with eq

Steps to Reproduce
deepdiff.DeepDiff(my_signal)

This seems to occur due to the nature of the deepdiff algortihm together with the fact that SignalIterator.__next__() returns itself an iterator-object again.


Issue w/ read_emails()

SYSTEM:  Command read_emails returned: Error: decode() argument 'encoding' must be str, not None
Auto-GPT is throwing this error when it attempts to read_emails() and fetch any of my recent mails for processing or replies.
I believe the Imap/smtp is configured properly as I was able to send an email out with auto-gpt, but I could be wrong.
Also attempted some work-arounds in the python code, but was unsuccessful after a few different attempts.
Using 2FA w/ app password setup.

Thanks for the bug report. Can you let me know, what parameters Auto-GPT is using when calling the read_email command? A screenshot also works.

@BotezanTechnologic in addition I extended the imap search parameter such that more complex imap searches can be handled. This might resolve the issue.

Also testing this. Send email works pretty well but I'm also getting the permissions errors when trying to read emails. Also getting a permission error when trying to attach a file:

Oh, here are a couple of shots from the read_emails command:

Thanks for looking into this Riensen! Appreciate you already pushing a possible fix.
When testing this I was just using the same test setup as in the docs.
Will test with the updated imap params later today and let you know what happens.
If it throws another error, I will provide screenshots.

Confirmed fixed & working on my end ✅
Thank you @riensen !

Oh, here are a couple of shots from the read_emails command:

Auto-GPT seems to try to pass additional parameters to the read emails function. There is nothing that can be done to fix this within the plug-in. Its a problem within Auto-GPT (GPT not following directions).

@BotezanTechnologic Thanks for confirming!

@BotezanTechnologic can you tell me what you did to fix? I'm using the latest imap config and still having the same error

@runquik just updated with the new changes that got pushed. I don't think you and I were experiencing the same error.


[Renderer] echoes-13.bsp crashes as soon as it loads (Sven Co-op port)

Title pretty much.
Using AMD RX570 8gb as my GPU
Link to the ported Sven Co-op version:
http://scmapdb.wikidot.com/map:half-life:echoes

Can't repro
Could you please try latest amd graphic driver?

Tried both.
Still crashes without fail. I'm attaching a video that shows exactly what happens (sorry for the quality, had to compress it to keep it under the 10MB limit)
https://user-images.githubusercontent.com/129798156/233725650-3b0a257c-1ea2-43e5-85c4-7d17c157c273.mp4
If I turn off the renderer by adding a ; to it in the plugins.lst file, then it loads fine, but I'd rather have it because my framerate is awful without it, specially on echoes.

Doesn't seem to crash anymore on recent MetahookSV versions, if anyone else is having this issue, just update to the lastest version.

Seems to be fixed. Just comment here or reopen this issue if anyone encounter the crash again.


Changed from RawSqlStatement to UpdateStatement

name: Pull Request
about: Create a report to help us improve
title: 'Statement to updating MD5SUM column value changed from RawSqlStatement to UpdateStatement'
labels: Status:Discovery
assignees: ''

Environment
Liquibase Version: v4.2.2
Liquibase Integration & Version: <Pick one: CLI, maven, gradle, spring boot, servlet, etc.>
Liquibase Extension(s) & Version: -
Database Vendor & Version: -
Operating System Type & Version: -
Pull Request Type

[x] Bug fix (non-breaking change which fixes an issue.)
[ ] Enhancement/New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Description
Creating an update query using UpdateStatement.
Steps To Reproduce
List the steps to reproduce the behavior.

Please be precise and ensure private information is redacted
Include things like

Files used - sql scripts, changelog file(s), property file(s), config files, POM Files
Exact commands used - CLI, maven, gradle, spring boot, servlet, etc.

Actual Behavior
A clear and concise description of what happens in the software before this pull request.

Include console output if relevant
Include log files if available.

Expected/Desired Behavior
A clear and concise description of what happens in the software after this pull request.
Screenshots (if appropriate)
If applicable, add screenshots to help explain your problem.
Additional Context
Add any other context about the problem here.
Fast Track PR Acceptance Checklist:

[x] Build is successful and all new and existing tests pass
[ ] Added Unit Test(s)
[ ] Added Integration Test(s)
[ ] Documentation Updated

Need Help?
Come chat with us on our discord channel
┆Issue is synchronized with this Jira Story by Unito

Hi @biryukovpavel Thanks for submitting this PR.
Here’s what happens next:
A member of the Liquibase team will take a look at your contribution and may suggest:

Changes based on guidelines for PR submission
Additional tests (unit and/or integration) and/or Test Harness Tests
Provide other feedback, like requesting changes.

The PR will be prioritized according to our internal development and testing capacity.
We’ll let you know when it’s ready to move to the next step or if any changes are needed.

Hi @biryukovpavel Thanks for submitting this PR.
Here’s what happens next:
A member of the Liquibase team will take a look at your contribution and may suggest:

Changes based on guidelines for PR submission
Additional tests (unit and/or integration) and/or Test Harness Tests
Provide other feedback, like requesting changes.

The PR will be prioritized according to our internal development and testing capacity.
We’ll let you know when it’s ready to move to the next step or if any changes are needed.

Hi @biryukovpavel Thanks for submitting this PR.
What problem are you trying to solve that had you updating from a raw sql statement to an update statement?
Here’s what happens next:
A member of the Liquibase team will take a look at your contribution and may suggest:

Changes based on guidelines for PR submission
Additional tests (unit and/or integration) and/or Test Harness Tests
Provide other feedback, like requesting changes.

The PR will be prioritized according to our internal development and testing capacity.
We’ll let you know when it’s ready to move to the next step or if any changes are needed.

Hi @molivasdat!
In some DBMS, the update statement is syntactically different from the standard SQL update statement.
In the case of change statement from raw to update it will be possible to implement UpdateGenerator for the specific DBMS (as an extension) and the problems will not occur.
In the current implemantation, this is difficult to do and the code will be confusing.

Hi @biryukovpavel Thanks for submitting this PR.
What problem are you trying to solve that had you updating from a raw sql statement to an update statement?
Here’s what happens next:
A member of the Liquibase team will take a look at your contribution and may suggest:

Changes based on guidelines for PR submission
Additional tests (unit and/or integration) and/or Test Harness Tests
Provide other feedback, like requesting changes.

The PR will be prioritized according to our internal development and testing capacity.
We’ll let you know when it’s ready to move to the next step or if any changes are needed.

Hi @molivasdat!
In some DBMS, the update statement is syntactically different from the standard SQL update statement.
In the case of change statement from raw to update it will be possible to implement UpdateGenerator for the specific DBMS (as an extension) and the problems will not occur.
In the current implemantation, this is difficult to do and the code will be confusing.

Thanks for the the clarification @biryukovpavel We will add this to the list of PRs that are processing.

Thanks for the the clarification @biryukovpavel We will add this to the list of PRs that are processing.

Thanks for the the clarification @biryukovpavel We will add this to the list of PRs that are processing.

Hi @molivasdat!
Please tell me which version of the Liquibase release do you plan to include this PR?

Hi @molivasdat!
Can I help speed up the release of this fix?

HI @biryukovpavel , we are currently in the process of re reviewing all the issues and PRs and are starting the process to make updates to these repos to get temporary builds out that you can use as is and also re prioritize which issues and PRs get updated and when.  At this time, I don't have anything for you to do.


[Snyk] Security upgrade nokogiri from 1.10.1 to 1.15.7

Snyk has created this PR to fix 1 vulnerabilities in the rubygems dependencies of this project.
Snyk changed the following file(s):

examples/ruby/parse/Gemfile
examples/ruby/parse/Gemfile.lock

Vulnerabilities that will be fixed with an upgrade:

Issue
Score

Cross-site Scripting (XSS) SNYK-RUBY-NOKOGIRI-8453714
  86

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
Max score is 1000. Note that the real score may have changed since the PR was raised.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
📜 Customise PR templates
🛠 Adjust project settings
📚 Read about Snyk's upgrade logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Cross-site Scripting (XSS)

:tada: Snyk checks have passed. No issues have been found so far.
:white_check_mark: security/snyk check is complete. No issues have been found. (View Details)
:white_check_mark: license/snyk check is complete. No issues have been found. (View Details)


Fancy zones: my custom layouts disappears from time to time

Microsoft PowerToys version
0.37.2
Running as admin

[ ] Yes

Area(s) with issue?
FancyZones, FancyZones Editor
Steps to reproduce
I have 3 preset custom layouts, but sometimes they just disappear (maybe once every month) and I have to recreate them manually
PowerToysReport_2021-05-21-11-26-19.zip
✔ Expected Behavior
the custom layouts preserve
❌ Actual Behavior
I have 3 preset custom layouts, but sometimes they just disappear (maybe once every month) and I have to recreate them manually
PowerToysReport_2021-05-21-11-26-19.zip
Other Software
No response

Hi, @peacefulzephyr can you please verify if the issue is still present on the latest PowerToys version?

I think so.
From: Seraphima @.>
Sent: Thursday, July 22, 2021 06:41
To: @.>
Cc: @.>; @.>
Subject: Re: [microsoft/PowerToys] Fancy zones: my custom layouts disappears from time to time (#11408)
Hi, @peacefulzephyrhttps://github.com/peacefulzephyr can you please verify if the issue is still present on the latest PowerToys version?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/PowerToys/issues/11408#issuecomment-884880488, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AUDLILJEJZWAFJXCQLSX7ITTZAGWDANCNFSM45JLXS6A.


BSO: Return the correct ID's from mystery boxes

If a clue scroll or casket is rolled, it will get the correct ID for it based on the ClueTier that it should belong to.
If a duplicate item is rolled, it will try to find the non-duplicate version and return that, if not it will re-roll

This is no longer an issue because the bot only gives tradeable items now - and therefore no clues.


Log tail

to #22
decided to add it back for now. Logging may take some extra time to make it properly (and to agree with you the concept ;)))
So:

in-mem log is now in a separate module + UT
with some extra utils
it checks ALB_SUPPRESS_FRONPAGE_LOG env to remove it from frontpage
logs wrong requests too (for security overview)
docs updated
some minor cleanups with tests, sorry couldn't stand from

offtopic: btz why you prefer " to ' ? ' is easier to type

prefer consistency not super much the one over the other, double quotes is part of normal written language, feel free to change for this repo
cf push tested ok, merging

:+1:
quotes. ok ... well, if you don't mind maybe I'll change it later


feat(tools): an audit for some changes

Checklist:

[X] I have read freeCodeCamp's contribution guidelines.
[X] My pull request has a descriptive title (not a vague title like Update index.md)
[X] My pull request targets the master branch of freeCodeCamp.
[x] All the files I changed are in the same world language, for example: only English changes, or only Chinese changes, etc.

About #40483

@stevending1st Thank you for your contribution.

@stevending1st Thank you for your contribution.


Uprev pydantic-core to 0.24.1

Waiting for build
Selected Reviewer: @kludex

please review

I'm quite confused by the recursion error test failure we're seeing. @dmontagu if you an 👀 this a bit that would be appreciated


Update goreleaser/nfpm to v2.37.0

Update Dockerfile ENV values
Update README tools version table

refs GH-1529

From CI output:

go<EMAIL_ADDRESS><EMAIL_ADDRESS>requires go >= 1.22 (running go 1.21.10; GOTOOLCHAIN=local)

Will need to downgrade nfpm to v2.36.1 for the oldstable series until Go 1.23 is released in a few months.

Superseded by https://github.com/atc0005/go-ci/pull/1533.


copy group invite link subtext in dark mode is still dark

@technoplato When logging these let's make sure to include a screenshot or link to the message
Also for specific devices it seems like there is an issue that fr some reason the theme is not being set, it's been kind of a consistent problem, so there are logs added for this type of use case


Add vallabhtiwari

Related issue

What changes does this PR do?
Adds vallabhtiwari to the hacktoberfest-practice repo
Screenshots (If any)
Questions

Thank you very much @schmelto . This was my first pr merged .

Sorry, that was a not on purpose. I was just exploring different options.


Allow pubsub cloudevent override with special metadata

Signed-off-by: Bernd Verst<EMAIL_ADDRESS>Description
Allows overriding cloud event property via metadata / headers in pubsub publishing. The cloudevent- prefix must be used for the metadata key. For example cloudevent-tracestate would manually override the tracestate value.
Fixes #5515.

/ok-to-test

PR looks good without this latest E2E test change. Let's see if this more sophisticated test also works...

/ok-to-test

As discussed on discord, my recommendation would be to keep an allowlist of options that can be overridden, excluding things like topic and data. If users were to ask for that in the future we can add those, but otherwise it seems risky.

As discussed on discord, my recommendation would be to keep an allowlist of options that can be overridden, excluding things like topic and data. If users were to ask for that in the future we can add those, but otherwise it seems risky.

There is no real risk here.
A user must explicitly specify metadata with hand "cloudevent-topic" or "cloudevent-data" to override those parts of the envelope.
Keep in mind that this would occur on the same request on which the user is literally specifying the data and the topic.
Sure it doesn't make a ton of sense, but there is no risk.
The request was to be able to override any of the cloudevent properties with metadata. That's what this PR does with the exception of time.

Reviewers - this PR works but I'm still struggling to figure out the E2E tests. There is a little bit more work required.
Maybe someone with knowledge of the PubSub E2E tests can see what I'm doing wrong there.

@ItalyPaleAle per your request Topic, Pubsub name, Data and Data content type cannot be overriden.

I restructured the code such that everything can now be tested in unit tests. I no longer need to modify E2E tests, so I removed those changes.

Removed the E2E test comments as the E2E test changes no longer apply.

Using "-" as separator is not intuitive. We should use "." instead. That is most common when dealing with nested properties in JSON or tree-like structures.

Using "-" as separator is not intuitive. We should use "." instead. That is most common when dealing with nested properties in JSON or tree-like structures.

On the contrary, I don't find a period intuitive. Don't think of this as indexing into the event or JSON - that's not how this works at all.
Besides, in the original issue nobody suggested a period to be used for the prefix. The issue has been open for discussion for months and the PR has been sitting there for a week.
I personally don't feel this needs changing. The period would also give it the feeling you could alter every property (or add new ones) when you cannot and should not actually.

Using "-" as separator is not intuitive. We should use "." instead. That is most common when dealing with nested properties in JSON or tree-like structures.

On the contrary, I don't find a period intuitive. Don't think of this as indexing into the event or JSON - that's not how this works at all.
Besides, in the original issue nobody suggested a period to be used for the prefix. The issue has been open for discussion for months and the PR has been sitting there for a week.
I personally don't feel this needs changing. The period would also give it the feeling you could alter every property (or add new ones) when you cannot and should not actually.

The original issue starts with ".", using "-" was your idea.


updating test classes with main methods

removed main methods in test classes and replaced with @Test; removed System.out statements and added slf4j dependency (doesn't interfere with other logging facilities and works as expected); minor beautification

we don't want to add additional logging library, is good remove the System.out.print... but should be replaced with
OLogManager.instance().warn() //or any other lels

I'm waiting for input in https://github.com/orientechnologies/orientdb/issues/3706 (it's hard to see through the logging facilities). Maybe you can explain the situation there or even better in javadoc, HACKING, README or something similar (including links to existing docs) and fix #3706. Then I can open a new pull request reflecting your comment for this one. Thanks for taking a look.


Replace license with MIT.

Remove proprietary JuliaPro license.

@bmharsha The JuliaInXL plugin needs to be rebuilt and re-uploaded.

Sure, I'll rebuild and share it

I've released a new version of JuliaInXL with MIT license (The Micrsoft addin was built using this PR), once https://github.com/JuliaRegistries/General/pull/70699 is merged, this version should be available through General registry as well.


label_replace() replacement argument is wrongly rejected as unknown variable

See label_replace() documentation. The replacement argument can use capturing groups like $1, $2 that get wrongly interpreted as dashboard variables by the backend validation, which makes it fail.
{
    "message": "bad request: variable \"1\" is used in the variable \"extlabels_prometheus\" but not defined"
}

after discussing with Antoine, a way to mitigate the issue is to forbid the possibility to name a variable with just a number.


Key Binding

Building Gadgets Version
BuildingGadgets2-1.0.6
Describe the bug
Wanting to put my build copy and put it in mirror mode, I watched them control and learned that with the key "range" we could do it but this key does nothing even by changing it
To Reproduce

Copy a construction
enter paste mode
3 . used the "range" key

Expected behavior
See my construction in mirror mode of what I copy
Screenshots

Ah sorry, Range is only used for the building and exchanging gadget.  It has no functionality for the copy/paste gadget. Sorry!


create bucket_usage_log table and add file_counts, last_usage_update_…

according to the discussion about bucket usage update problem:

create bucket_usage_log table
alter existing buckets table and add two columns file_counts & last_usage_update_time.

Please help to review, thanks a lot.

Dear maintainers,
Please help to review this PR which creates the usage_log table and  add column usage_update into buckets table. I have modified all the undering go source files which uses the buckets table. thanks.


hls: add llvm-hs-combinators to hiedb

get hls to work with nix develop

Seems to be working (tried nix develop and then a cabal build). Thanks!


Auto minscale - image size

How can I make the minScale equals to the image size, so that when we zoom out to the maximum we see the full image.
thanks for the awesome project!

@laazaz
you can just add styles to the image itself to be 100% width of the component, this way it will be responsive
for now developing the way to adjust those scales in the code itself will take some


dart2js: Allow read-only access to initializing formals by default

This is the dart2js specific issue for #27891; please refer to that issue for additional information.

This CL will enable the feature by default.

Landed in https://github.com/dart-lang/sdk/commit/9bf3b95cc5f4677269830054fb5c12b64f1e7f21.


Make use of nested configuration Breaking Change

Summary:

Config follows the same structure as the configuration.yaml file of z2m
Removed set_config.py
Removed all python3 dependencies from docker container
Image size down to about 180MB
Made edge build locally
Needed to ad run.py for zigbe2mqtt-edge because stable and edge are very
different currently

Note:

I'm not sure about any kind of upgrade path. Hassio would update the
default values for options only upon manually deleting the data docker volume
that was created up on initial addon installation. Thus upon updating, a
custom configuration.yaml file will be overwritten by an invalid one.
I know from the AppDaemon
guys, when they had this problem, they updated to a complete new addon name,
appending a number. E.g. they would make something like:
zigbe2mqtt2-edge
so custom configurations of configuration.yaml like network_key wont get
overwritten.

The config now provides the same default values as they are documented at
https://www.zigbee2mqtt.io/configuration/configuration.html I don't know if
this is a good idea because it might trigger people to belife they have to
change for e.g. client_id.

I dind't test everything, I mostly made sure it runs at all. In particular
the device config, pand_id and network_key I'm not sure about.
pand_id and network_key expect hey values afak from z2m documentation but
I couldn't find out if this works with ints as well.

Oh I just realized this might royal y break all devices. So plz don't merge this yet.


List of predefined pipeline variables is missing from published site

I went looking for the list of predefined pipeline variables today and realized that the list of variables is missing on the published site: https://learn.microsoft.com/en-us/azure/devops/pipelines/build/variables?view=azure-devops&tabs=yaml

I'm not sure if this was caused by a recent change, but it makes using the variables much harder. Fortunately, it looks like the variables are accessible through the raw markdown files here: https://github.com/MicrosoftDocs/azure-devops-docs/tree/main/docs/pipelines/build/includes (in case anyone else stumbles upon this issue before the published content is fixed).

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

ID: 11d3469c-83c2-c2f6-41d0-08f82c330bbc
Version Independent ID: a373c48d-6395-3002-d2d8-bbc1fea2e48e
Content: Predefined variables - Azure Pipelines
Content Source: docs/pipelines/build/variables.md
Service: azure-devops-pipelines
Sub-service: azure-devops-pipelines-build
GitHub Login: @juliakm
Microsoft Alias: jukullam

@sgerace -- Sal, thank you for your feedback.  This is strange, because I am seeing all of the tables on my screen, both in the published article and in the GitHub source.  Can you provide further details on your view?

Hmm, strange indeed. I just checked the site again and everything looks good now. I can't say why the problem arose in the first place, but it seems to be resolved now.


Document the existence of DBT_PACKAGE_HUB_URL

Contributions

[X] I have read the contribution docs, and understand what's expected of me.

Link to the page on docs.getdbt.com requiring updates
Maybe the packages page? Maybe a page on secret env vars? I'm opening an issue instead of a PR because I don't really know where it should go
What part(s) of the page would you like to see updated?
It's possible to use a custom package hub by specifying the DBT_PACKAGE_HUB_URL environment variable.
As far as I can see, there's no documentation for this, so I had to ask chatgpt to remember the name of the variable 😅
Additional information
No response

closing issue as addressed in https://github.com/dbt-labs/docs.getdbt.com/pull/5122/files


[PubSub] PubSubInboundChannelAdapter stop pulling messages from PubSub

I have SpringBoot App deployed in GKE pod, that pulls messages from pubsub:
 PubSubInboundChannelAdapter adapter = new PubSubInboundChannelAdapter(pubSubTemplate, subscription);
 logger.info("Created subscription to " + subscription);
 adapter.setOutputChannel(inputChannel);
 adapter.setAckMode(AckMode.MANUAL);

It works fine, but then it stopped pulled messages, if I restart the pod it works fine.
Nothing in the logs.
I have many pods that subscribed to different subscriptions and this issue happens for all of them.
can be once in a month, but consistently happens, and require us to restart the pods all the time (production cluster)
spring boot: 2.3.0.RELEASE
spring-cloud-gcp-starter-pubsub  1.2.5.RELEASE

@brachipa When the listener stops, is there an unusually long amount of time that passed from when the last message was sent to the topic?
Do you override spring.cloud.gcp.pubsub.keepAliveIntervalMinutes property to any custom value?
Do you override any of the Spring Cloud GCP pub/sub subscription beans -- subscriberTransportChannelProvider, SubscriberFactory, PubSubSubscriberTemplate?

@brachipa Do you have other pods that are getting the messages at the same time? Perhaps this node is just being starved?
Since Spring  Cloud GCP relies on the Pub/Sub client library, it's possible that there is an issue there.
cc/ @chingor13

Yes, I also forget to mention that I have 2 subscriptions in the same pod, one is getting message in the same time and one is totally disconnected.

I would try the Synchronous Pull method, which is less prone to starvation, and you have more control over.


Feature request: Dictionary Definitions on PowerToys Run (alt+space)

Description of the new feature / enhancement
Please include a plugin for dictionary definitions so we can get results for word definitions when we type the word into the bar.
Scenario when this would be used?
When looking up the meaning of words that you don't know while you're reading something online
Supporting information
No response

/dup #3309


rules_distroless_dependencies are out of date

PR https://github.com/GoogleContainerTools/rules_distroless/pull/12 updates MODULE.bazel but not the dependencies.

@thesayyn ?

@prestonvanloon do you mind sending a PR for this?

Sure, i'll send a PR today. Thanks


Fix get version for maven-release.yml action

Running now on my fork to test

Looks like it worked as expected. https://github.com/cfkoehler/emissary/actions/runs/7906005205/job/21579961380
From the create branch step:
Run git switch -c release/8.0.0-Ctest
Switched to a new branch 'release/8.0.0-Ctest'


ArcGis JS API 4.11, 4.12 Incorrect work GraphicsView2D (from esri/views/2d/layers/grapics/GraphicsView2D) hitTest method

ArcGis JS API 4.11, 4.12
GraphicsView2D (from esri/views/2d/layers/grapics/GraphicsView2D) hitTest method return only one feature(graphic) with zero index.
Better to be return all graphics.
Is it issue or correct work?

In API documentation hitTest method get top graphic from layer


Adjust colors for theme on calltips and update language from console and editor on calltip call

On spyder/widgets/mixins.py
# TODO: Select language and pass it to call

    _PARAMETER_HIGHLIGHT_COLOR = '#DAA520'
    _DEFAULT_TITLE_COLOR = '#2D62FF'
    _CHAR_HIGHLIGHT_COLOR = 'red'
    _DEFAULT_TEXT_COLOR = '#999999'

@dalthviz we need to be able to ask the kernel what language it is running (right now python, but it could be anything)
spyder/plugins/ipythonconsole/widgets/help.py
# TODO: Pass the language from the Console to the calltip

@dalthviz, the language name is obtained after making a kernel_info request to the kernel. The incoming info is handled here:
https://github.com/jupyter/qtconsole/blob/77238a8195eb1f7ac085579d6ad38cd258123aee/qtconsole/jupyter_widget.py#L288
So please do a PR in qtconsole to set language_name as an attribute of JupyterWidget in _handle_kernel_info_reply. Also initialize it to None in the __init__ method of that class. That's a minor change to avoid overriding _handle_kernel_info_reply in our side.
After that's in place, we could easily get language_name through the shellwidget of each console.

For the Editor, we need to add, there's already self.language in CodeEditor that we need to pass to tooltips and calltips to set its language correctly.


Allows any flag to expect optional arguments

The motivation for this change is to allow any flag or flag type (not only boolean) to support optional arguments as long as they are declared to do so. Example:
git diff --stat[=<width>]

The logic around flags with empty arguments (e.g. --debug) were detached from the boolean type and moved to a separate attribute called Optional. This allows to mark any flag type as supporting optional arguments.
The difference in comparison to not providing the flag at all is that the Changed attribute is set, so it allows to control the difference of not providing the flag or providing but without an argument.
A possible use case would be:
flag.IntVar(&flagvar, "list", 10, "list the number of items")
flag.MarkOptional("list")

So when using the flag users could either provide the number of items to display in the list, or not provide any argument to use the default int value:
deploy           // perform a deployment
deploy --list    // will list 10 deployments (default value)
deploy --list=20 // will list 20 deployments

The behavior for default types were not changed (the only flags that allow empty arguments by default are boolean).

@eparis @spf13 FYI

Given
flag.IntVar(&flagvar, "list", 10, "list the number of items")
flag.MarkOptional("list")

What's the different between
./deploy --list

and
./deploy

Sort of unrelated:
The bash autocompletion stuff in spf13/cobra has special case code around booleans.  It considers them to not be a "two_word_flags" and it doesn't autocomplete to include an =.  How do you think I should handle an int flag? Should ./deploy --lis[tab][tab] show --list=, as it does now, or should it show --list  and suggest people type another flag?

@eparis Updated the description with an answer and use case to your first question.

@eparis I'd say in autocomplete it should not show the = in any flag explicitly set as Optional (same behavior as booleans today), and show = in everything else (same as everything that's not boolean today).

In fact I think the help should be aware of it too: when a flag is set as Optional, the help could be displayed with [], e.g.:
--list[=size]

So I've been doing some thinking. Never a good idea for anyone... Tell me what you think... (And mind you, I have no idea how to make it work sanely)
What if we changed the Value interface somehow, so it had both a default and a no-option default. So in your code above somehow
flag.IntVar(&flagvar, "list", 10, "list the number of items")

becomes
flag.IntVar(&flagvar, "list", 0, "list the number of items")
SetNoOptionDefault("list", 10)

Bools all turn into SetNoOptionDefault($name, True)
What it would mean for the consumer of pflag is that they can just use the list value.
deploy           // flagvar == 0 so just perform a deployment
deploy --list    // flagvar == 10 so list 10 deployments (no option default value)
deploy --list=20 // flagvar==20 so list 20 deployments

But maybe the code on our end isn't worth it just to keep the users of pflag from having to look at changed...

actually, it is probably as easy as doing it as a string in the Flag structure and calling Set() if the flag was called without an argument....
So probably not as hard to code as I first thought....

@eparis I like how it looks, but the main reason I didn't choose to internalize the no-option-default value initially (instead delegating to the user the control of storing and managing that value and just giving a way to check if it was set, through Changed) is that you would have to handle all possible flag types, e.g.: SetNoOptionDefaultInt, SetNoOptionDefaultString, SetNoOptionDefaultBool and so on.
I'm ok with it if you think it's reasonable, but in a first glance it looked a little too much. If we are adding something to every possible type, then we could just add new *Var methods to specifically handle flags with optional arguments, explicitly marking them as optional and providing the no-option-default value at the same time, e.g.:
flag.OptionalIntVar(&flagvar, "list", 0, 10, "list the number of items")

So I drank a lot and did some hacking on this over the weekend.  I completely rewrote parseLongArg and parseShortArg.  Not because they were wrong, just because I thought I could do better. In some ways I did better, in some ways worse.  But that's not the point here....  The point is the default-ing method isn't too hard.
https://github.com/eparis/pflag/commit/b783f345682172fdc8f45b99a8294d5b14635ef9
Before this change, given a variable --list with shorthand -l any of the following would be valid

deploy --list=20
deploy --list 20
deploy -l=20
deploy -l 20
deploy -l20

Now if we define list as optional we lose 2 and 4. And I guess we lose 5.  I guess that makes sense and people shouldn't be toooo shocked about it....

Ok, so you are storing NoOptDefVal as a string and relying on the type conversion to convert it. I think it overall is cleaner and I like it better than having multiple SetNoOptionDefault for multiple types. My only concern would be having two different approaches for similar things facing end users: regular defaults are specified in their original types (true, 10, "string", etc); and argument-present-defaults always as strings ("true", "10", "string", etc). Let me think about it.
That said, I think loosing any of the short|long forms is an issue.

#20 is my alternative.  Thoughts?

@eparis I do like how it's exposed to users. But I think loosing any form of short or long flags is an issue. If you can assure we still support all forms then I'm good.

If a user sets NoOptDefVal:
Works:
--list=10
--list
-l
-l=10

Does not work
--list 10
-l 10
-l10

So trying to follow along... I think I understand why this feature is valuable and the different approaches..
It seems though with either approach the experience will be different for the user depending on if the feature is enabled or not. This seems weird to me that a call that works today will no longer work (or could possibly not work in the future) because a developer decided to make a flag optional.
As I understand it
With optional params (works):
--list=10
--list
-l
-l=10

Without optional params (works):
--list=10
-l=10
--list 10
-l 10
-l10

I haven't given it enough thought, but I would hope a solution exists that could cover every case that works today.

@spf13 correct.  Both approaches have the same limitation.
When a flag has been set as taking an 'optional' argument, the code has no idea if user intended the next word to be the argument to the flag, to be the next flag, or to be some interspersed word we should ignore.
You are right. If developers turn an existing flag into a 'arg optional' flag users may see breakage.

although admittedly we already this rather odd usage around bools.
Works:
--bool=true
--bool
-b=true (fixed today)
-b

Fails:
--bool true
-b true
-btrue

So the bool case makes a lot of sense to me because it's only used to change the current behavior to the opposite behavior. With the case of bool it's a bit surprising that it takes values at all, but I understand that it would want to do that to enable people to be explicit which is typically a good practice when scripting things as it would remain consistent even if the default changes.
Is there a practical use case where you intend to use this feature? I'd like to understand it a bit better. I see why it's nice, but I don't think I fully understand it's value.
We've also strived to be posix compliant so far. I don't claim to be an expert on the subtleties of the POSIX specification, but I believe this feature is in disagreement with guideline 7 (at the bottom).
http://pubs.opengroup.org/onlinepubs/9699919799//basedefs/V1_chap12.html

@eparis @spf13 Wait, that's not correct. This PR (https://github.com/spf13/pflag/pull/18) as originally proposed does not break the current behavior or make any difference either the optional arg feature is enabled or not. All these forms work correctly when having an optional arg (testing right now with my real use case):
deploy -L         deploy --list
deploy -L 2       deploy --list 2
deploy -L=2       deploy --list=2
deploy -L2

I agree that the solution must cover every case that works today, the short and long forms with a space between the flag and argument are important and must work.

@fabianofranz so what does deploy -L --some-other-flag do?

@fabianofranz or deploy -lb where b is some boolean shorthand?

just to be thorough, GNU amends the POSIX standards a bit
http://www.gnu.org/software/libc/manual/html_node/Argument-Syntax.html
I believe it doesn't address optional argument values in it's specification.

@eparis Fair enough,  -L --some-other-flag fails.
This has basically NEVER worked, and no one noticed!
I'm just stunned...

Try it yourself:
package main

import (
	"fmt"
	"os"

	"github.com/spf13/pflag"
)

var _ = fmt.Fprintf
var o = os.Stdout

func main() {
	var flagvar int
	pflag.IntVar(&flagvar, "flagname", 1234, "help message for flagname")
	pflag.Parse()

	fmt.Fprintf(o, "val=%v\n", flagvar)
}

$ ./test --flagname 1235
flag needs an argument: --flagname
Usage of ./test:
      --flagname=1234: help message for flagname

my fix is below.  But since --flag name hasn't worked since like 2012, maybe we don't have a problem ehre....
diff --git a/flag.go b/flag.go
index 9f16733..93222c8 100644
--- a/flag.go
+++ b/flag.go
@@ -530,11 +530,15 @@ func (f *FlagSet) parseLongArg(s string, args []string) (a []string, err error)
        }
        var value string
        if len(split) == 1 {
-               if bv, ok := flag.Value.(boolFlag); !ok || !bv.IsBoolFlag() {
+               if bv, ok := flag.Value.(boolFlag); ok && bv.IsBoolFlag() {
+                       value = "true"
+               } else if len(a) > 0 {
+                       value = a[0]
+                       a = a[1:]
+               } else {
                        err = f.failf("flag needs an argument: %s", s)
                        return
                }
-               value = "true"
        } else {
                value = split[1]
        }

I decided to take a look at what ls does (and how it represents things in the man page).  Basically the ls command in linux does what is being described here. Sometimes you can use a space and sometimes you can't...
It makes me feel like we should go with the optional arg stuff.
$ ls --color
$ ls --color=never
$ ls --color never
ls: cannot access never: No such file or directory

       --color[=WHEN]
              colorize the output; WHEN can be 'never', 'auto', or 'always' (the default); more info below

$ ls -T 7
$ ls --tabsize=7
$ ls --tabsize 7
$ ls --tabsize
ls: option '--tabsize' requires an argument
Try 'ls --help' for more information.

       -T, --tabsize=COLS
              assume tab stops at each COLS instead of 8

@thockin since you pointed out that --flag val doesn't work.

I still have my PR version of this open #20


Corrected text to be "of space will be freed."

To address issue #31

Closed and reopened, trying to figure out how to link in issue #31 but could not.  Probably a maintainer option.

Makes sense, thanks!


Allow otel trace ids to be output in logs

Reading the variables section of the docs, we can access datadog_trace_id and datadog_span_id which are integers. It would be great to be able to instead output the OpenTelemetry version of trace and span ids based on a hex string instead.
This would be similar to how traceparent works just with the ability to write those variables to logs.
Thanks!

Hi @phillipuniverse
I understand you would like to output the trace and span IDs in the OpenTelemetry format (hex strings) instead of the current Datadog format (decimal representation). Can you confirm?

@dmehala yes that is right.


CWE-1045 Parent Class with a Virtual Destructor and a Child Class without a Virtual Destructor

Usage Name
Child class missing virtual destructor
Reference
https://cwe.mitre.org/data/definitions/1045
Roles

the ParentClass
the ParentClassVirtualDestructor
the ChildClass

Detection Patterns

8.2.35 ASCQM Implement Virtual Destructor for Classes Derived from Class with Virtual Destructor

data regarding instances of this anti-pattern in a repository can now be gathered by running:
/static/driver/main.py --init --ast_pass='cwe-1045'


Allow user to ignore server errors.

This PR resolves #1206 by allowing users to ignore server errors by error message until the server is restarted.
The error dialog is not displayed, but the error is still shown on the output channel.

Hi @dbaeumer while working on this PR I noticed that the status bar color is not changing on error or warning status, it's on purpose? Maybe with the option to silent server errors we should highlight the status bar to show to the user that something is wrong and the file is not being parsed, what do you think?
Another option that I thought about adding with this (or another) PR is some commands to allow the user to silent or display all server errors for the session. Could this be a good option when the error message changes on repeated errors?
Maybe this could be helpful to alleviate other issues like #1238

@edupsousa sorry for the delay :-)
We should definitely update the color. It did in the past, but the code got somehow lost :-). It is here: https://github.com/microsoft/vscode-eslint/blob/7568b4ce8413e890e5f7d99b18845ccd00fd3d1a/client/src/extension.ts#L834
The background is never updated.
I would actually not add any additional commands. The reasons are:

I think not showing the same message over and over is a good default
clicking on the ESLint status takes you to the output channel.

@edupsousa sorry for the delay :-)
We should definitely update the color. It did in the past, but the code got somehow lost :-). It is here:
https://github.com/microsoft/vscode-eslint/blob/7568b4ce8413e890e5f7d99b18845ccd00fd3d1a/client/src/extension.ts#L834
The background is never updated.
I would actually not add any additional commands. The reasons are:

I think not showing the same message over and over is a good default
clicking on the ESLint status takes you to the output channel.

No problem with the delay. The year is ending but the work just increases. :-)
I was playing around another day and already did the background color update, but the branch where I did it is polluted with other things that I was playing. I will do some cleanup and tests before submitting it in a new PR.

@edupsousa I looked at the code again and did you do any chances to update the background color? It is not in the PR.

@edupsousa I looked at the code again and did you do any chances to update the background color? It is not in the PR.

@dbaeumer, I was planning to put it on a separate PR, sorry for the confusion. But, to keep it simple I just added the status bar color code to this same PR.

@edupsousa thanks. Will have a look


Remove or tone done the promo on the Theme fisher themes

https://themes.gohugo.io/bigspring-hugo-startup-theme/
/cc @themefisher -- we have talked about this before; one simple link to "my premium themes" or something is fine, but the massive ad sections I see on the theme above (which I suspect is also on the other themes) is pushing the limits, and I don't like it.
If you want this kind of ads, put them in  your own store.
I will follow up on this Monday.
/cc @digitalcraftsman

Hello @bep
I am putting down the premium theme items from the GitHub readme. We released almost 16 themes under MIT license those were previously CC 3.0 . And if we could not make something from premium ones it's hard for us to contribute more in HUGO . Hope you consider this and bring a good solution.May be a premium theme section in hugo theme webiste.
Thanks


Overeager stale bot?

I am finding myself to reopen most of the autoclosed issues here. Maybe we want to have some more time elapse between marking as stale and closing?
I am also wondering if we should include bug in the exempt labels?
@andersy005

@jbusecke, the default number of days before an issue is marked stale is 60. We can definitely increase to a more reasonable value. What would you like the new default number of days to be? The list of available options is here: https://github.com/actions/stale#all-options

I am also wondering if we should include bug in the exempt labels?

👍🏽

Was it already 30 days since that darn thing warned us about stale issues? I think that is actually fine, I guess I just have to go through the rest of these.


Vague "Could not fetch new updates" error if webhook is set

I had 2019/07/24 22:49:53 telebot: could not fetch new updates errors when starting a bot. It doesn't tell me that another webhook is set neither it provides method RemoveWebhook

Same issue when using with Taskfile with flag --watch (e.g. restarting on change)

I don't know how relevant this problem is now. If it is, please re-open and I will check it.

seems relevant to me

@stek29 Oh, you're right.. There is no implemented deleteWebhook. Gonna fix it in 2.1.

I also noticed that there is no implemented getWebhookInfo method to check if bot using a webhook or not. We still have some work on here. Gonna keep it for this issue.

Implemented in https://github.com/tucnak/telebot/commit/1157f8b301813dfdc61495f9a12867baf0dd62ad. WebhookInfo fields are available in the Webhook type.


Parallel conversions with office

From the README.MD the issue came from the unoconv library.
A workaround seems possible from https://github.com/unoconv/unoconv/issues/225#issuecomment-514236053
Is it possible to integrate this fix or keep an eye on a possible PR from it?

Hello @L3o-pold
I'm working on this for the v6 actually 😄:
return []string{
	// see https://ask.libreoffice.org/en/question/42975/how-can-i-run-multiple-instances-of-sofficebin-at-a-time/.
	fmt.Sprintf("-env:UserInstallation=file:///tmp/%d", p.port),
	"--headless",
	"--invisible",
	"--nocrashreport",
	"--nodefault",
	"--nofirststartwizard",
	"--nologo",
	"--norestore",
	fmt.Sprintf("--accept=socket,host=%s,port=%d,tcpNoDelay=1;urp;StarOffice.ComponentContext", p.host, p.port),
}

Hi @gulien
Oh ok, I saw your changelog but I was not sure that'll fix this issue.
Is that already available on the v6 branch?

It was not planned but I've encountered a memory issue with LibreOffice during load testing which forces me to rethink the process management almost entirely.
In others words, the goal is now to allow multiple instances of Google Chrome and Soffice to run in parallele, each one being able to do one conversion at a time. When a conversion is done, it checks the process memory usage and restart it if its higher than a configurable value. Having multiple instances helps mitigating those restart impact.

Ok great, I'll keep an eye on the 6.0 release.


1.19 unstable version

TNT's can be rendered in half sometimes when inside a portal (TNT Entity)
Arrows or projectiles such as snow can go through the obsidian from the portal
Water doesn't break portals (Vanilla portals break if water interacts with them)
Throwable potions break on impact of portal instead of going through the portal.
Sculk sensors don't detect any sound when you go inside the portal. (If sensor is not in same dimension)
TNT's or explosives do not deal damage through portals.
Mobs and entities have glitchy rendering when going through the portal
Hostile mobs forget who they were attacking when attacker goes through a portal
Damage animations for a mob or player stop when going through a portal
Zombies and husks do not attack villagers when they are not in the same dimension
Iron golems do not attack the hostile mobs when they are not in the same dimension
Villagers are not scared of zombies when they are not in the same dimension
Zombies do not go after villagers if they are not in the same dimension.
Leads act very strangely when you change dimensions.
Cobweb and Powdered snow is ignored when inside a portal.
All the bugs i know.

As I tested, the nether portal can be break by water bucket by using it on the obsidian frame. Are you using other mods?

Going to split this issue into separate issues.

https://github.com/qouteall/ImmersivePortalsMod/issues/931

What do you mean by "Damage animations for a mob or player stop when going through a portal"?


luci device tracker broken since last two commits

Home Assistant release with the issue: 0.89.2

Last working Home Assistant release (if known):
Two commits before that. i.e. https://github.com/home-assistant/home-assistant/commit/1aba4699b91c310eca597ed29c092a4ea3f60448#diff-c3450b6c46060bb83e8cb5bbd6850c97
Operating environment (Hass.io/Docker/Windows/etc.):

VirtualEnv on OSMC
Component/platform:

luci device_tracker
Description of problem:
The device tracker does not report devices from my Openwrt 15.05 router anymore. It was working two commits before. But some extensive work has been done on that by @fbradyirl in #21236 and #21321. I suspect the new library used here does not support older versions as was the case in the working version.
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
device_tracker:
  - platform: luci
    host: !secret luci_host
    username: !secret luci_username
    password: !secret luci_password

Traceback (if applicable):
error setting up platform luci

Traceback (most recent call last):
  File "/srv/homeassistant.release/lib/python3.5/site-packages/homeassistant/components/device_tracker/__init__.py", line 177, in async_setup_platform
    platform.get_scanner, hass, {DOMAIN: p_config})
  File "/usr/lib/python3.5/asyncio/futures.py", line 380, in __iter__
    yield self  # This tells Task to wait for completion.
  File "/usr/lib/python3.5/asyncio/tasks.py", line 304, in _wakeup
    future.result()
  File "/usr/lib/python3.5/asyncio/futures.py", line 293, in result
    raise self._exception
  File "/usr/lib/python3.5/concurrent/futures/thread.py", line 55, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/srv/homeassistant.release/lib/python3.5/site-packages/homeassistant/components/device_tracker/luci.py", line 32, in get_scanner
    scanner = LuciDeviceScanner(config[DOMAIN])
  File "/srv/homeassistant.release/lib/python3.5/site-packages/homeassistant/components/device_tracker/luci.py", line 47, in __init__
    config[CONF_SSL])
  File "/srv/homeassistant.release/lib/python3.5/site-packages/openwrt_luci_rpc/__init__.py", line 28, in __init__
    self.router = OpenWrtLuciRPC(host_url, username, password, is_https)
  File "/srv/homeassistant.release/lib/python3.5/site-packages/openwrt_luci_rpc/openwrt_luci_rpc.py", line 48, in __init__
    = self._determine_if_legacy_version()
  File "/srv/homeassistant.release/lib/python3.5/site-packages/openwrt_luci_rpc/openwrt_luci_rpc.py", line 72, in _determine_if_legacy_version
    self._call_json_rpc(*rpc_ip_call)
  File "/srv/homeassistant.release/lib/python3.5/site-packages/openwrt_luci_rpc/openwrt_luci_rpc.py", line 184, in _call_json_rpc
    raise InvalidLuciTokenError("Luci responded "
openwrt_luci_rpc.exceptions.InvalidLuciTokenError: Luci responded with a 403 Invalid token

Additional information:
See also here: https://community.home-assistant.io/t/device-tracker-luci-luci-responded-with-a-403-invalid-token/101733
Upgrading OpenWRT is no immediate option due to some complicated setup.

One issue I have here is that I have no hardware which supports such an old version of OpenWrt. If anyone can send me an old router to fix this then let me know. Or else expose port 80 for me in the WAN so I can do brief testing (on an unused installation of course).
@robbiet480 can you assign this to me?

@fbradyirl Accept the invite found in your email or at https://github.com/home-assistant and report back once done and then I can assign you, sure 😄

In the short term you could simply place the old luci.py in your custom_components directory and it will work again for you.

@robbiet480 done. Thank you.

In the short term you could simply place the old luci.py in your custom_components directory and it will work again for you.

Thanks! That solves the problem in the short term. And it might allow me to upgrade OpenWRT in the long run. Or to find another older router to set up for you to test.

Hello, can you please describe exactly which version of luci.py should I put in the custom folder to make it work again?

This one:
https://github.com/home-assistant/home-assistant/blob/1aba4699b91c310eca597ed29c092a4ea3f60448/homeassistant/components/device_tracker/luci.py

thanks, but i can't make it work. I'm adding the file to my custom_folder/device_tracker/ (in my case /home/homeassistant/.homeassistant/custom_components/device_tracker/luci.py) and I'm still having the "403 Invalid token" error. What I'm doing wrong?

There are quite some LUCI 403 errors in the OpenWRT and HA forums. One of those might be your salvation. ;-)
The one that helped me (with my veeery old installation) is this one: https://github.com/home-assistant/home-assistant/issues/753
Other than that, I have no more ideas (or lack of experience with these errors)...

thanks, but i can't make it work. I'm adding the file to my custom_folder/device_tracker/ (in my case /home/homeassistant/.homeassistant/custom_components/device_tracker/luci.py) and I'm still having the "403 Invalid token" error. What I'm doing wrong?

Try instead putting it in /home/homeassistant/.homeassistant/custom_components/luci/device_tracker.py

I think this workaround is enough to close this issue since the OpenWRT version (15.05) is not supported. It is 9 years old.

I'd prefer, if you could have a look at the test installation, I am preparing. It's just taking some time to set up. Maybe, it's a quick and simple solution. If you don't mind?

I give up. Both my spare devices won't be able to set up this easily. I'll go the hard way and update my routers eventually. Might actually be the safer choice. ;-)
As a last remark: both luci integrations should be able to run in parallel, if the custom_component is renamed to something like luci_15.
Thanks again for you help, @fbradyirl.


[Enhancement] Revise Python SQLAlchemy dialect and add testing

Why I'm doing:
The python SQLAlchemy dialect was a very basic override to the MySQL dialcet provided by SQLAlchemy. There were a bunch of things not implemented and it did not work very well.
What I'm doing:
Improving the dialect to be able to reflect tables and data types more succesfully. Implement the testing system provided by SQLAlchemy to test the dialect behavior.
Fixes https://github.com/StarRocks/starrocks/issues/39179
What type of PR is this:

[x] BugFix
[ ] Feature
[x] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[x] Tool

Does this PR entail a change in behavior?

[ ] Yes, this PR will result in a change in behavior.
[x] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[ ] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr needs user documentation (for new or modified features or behaviors)
[x] I have added documentation for my new feature or new function (updated Readme)
[ ] This is a backport pr

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch

[ ] 3.2
[ ] 3.1
[ ] 3.0
[ ] 2.5

Hey @rad-pat I'd like to send you some swag (as long as it can get to the UK).  Can you send me your email to<EMAIL_ADDRESS>


If you have fps drops, please post your information here

If you are experiencing fps drops that don't resolve when you lower your graphics settings while running this tool, please post:

Your windows version
Your game resolution
The %cpu that is being used by TekkenBot FrameDataOverlay in the task manager and the %cpu being used by Tekken7.exe

It's a combination of Windows 7, VSync OFF, and transparency.
Windows 7 users can still disable VSync by using a non-Aero theme aka turning Desktop composition OFF (Win8 and Win10 can't do it anymore afaik).
Overlaying transparent stuff on windowed games with VSync OFF causes frame rate slowdowns. This can be caused by other programs too e.g. Skype notifications.
Users can work around the problem by enabling Aero, which isn't optimal for fighting games because of the added display lag.
You as the developer can probably solve it in two ways:
easy = get rid of anything transparent (coding wise, make sure you set the transparency to OFF and not just to 0)
hard = fancy DirectX injection of frame data display à la Frame Trapped (would also make the tools work in true Full Screen and not just Windowed/Borderless)
I've just made a simple program that lets you toggle the transparency of an always-on-top dummy window to quickly spot the frame rate drops in a Win7+transparent window overlay+VSync OFF scenario for testing purposes as a proof: http://www.mediafire.com/file/6p7gagzji2mv9vl/TransToggle.zip

Wow, that is the best issue report I've ever seen.
I'm reluctant to mess with the dll, especially since I haven't had to so far and there's multiple other people going down that path.
Tkinter without transparency is pretty ugly but I can add a config option to the next release to disable transparency for users having frame rate issues.

Just wanted to confirm that setting transparency = 0 in frame_data_overlay.config fixed the issue.


Update joy function

The previous joy_manager.py file was configured for an Xbox controller, which caused it to perform incorrect functions. It has been updated to work with the joystick used during testing.

then you can load different config files for different joysticks

  ...
  <rosparam command="load" file="$(find auv_teleop)/config/psp4.yaml"/>
  ...

Yes, that makes sense; it’s a much better solution. I’ll be working on it. :)

Btw hardware test of this PR could be as simple as observing the cmd_vel output with the joystick. There is no need to test this with vehicle in pool.

Should I remove the README.md file in the package. Is it neccessary ? @senceryazici


PhoneGap + bbb

Hi, I'm somewhat lost trying to integrate with PhoneGap. My app looks ok when loaded in browser and grabs templates from JST when running with bbb server.
I followed https://github.com/tbranyen/backbone-boilerplate/wiki/Filesystem and opened the app from file:/// --it fails from localhost-- but it tries fetching template rather than using JST.
I suppose that pointing index.html to those files in dist/release might work, but I hear I should not modify it.
Thanks

Closing stagnant issue.


Height of histogram bars is incorrect for TCGA data

still a problem

Entries 476,13,93
If specify minimum height then the entries 13 and 93 makes hard to differentiate.

There should not be a minimum size. A histogram should just show the unmodified counts encoded in the length of the bar.
Btw: Why is the label not below the bar as it was already the case?

Ok. I have not updated from the master so it is not shown below.

If one of the bars gets too small to be picked, you can still make the transparent pick area bigger than the visible bar.

Since the PR https://github.com/Caleydo/mothertable/pull/167 is merged I will closed this issue.


feat(core): remove quick add button from PLP

What/Why?

Remove 'Quick add' button from PLP for products with options. Instead it will just link to the product.
No longer need api/product path, since we no longer use product sheet component.
Removed unused components.
Moved product form component to inside /product, since it's no longer shared.

Testing
Locally it works, link links to product.

Had to update tests @deini @chanceaclark


feat(api): update checkpoint when cancel subscription on efo mode

If canceled in the middle of time interval, not update check point.
I want to add test case. but it is hard to add test case for efo mode. because with localstack kinesis, subscription not work correctly.
If you care about test case in this case, tell me a advice

I think it seems more like bug fix or refactoring rather than feature addition.


PR: Flexbox

thanks for the guest post. Unfortunately, this does not meet the quality standards maintained by darshanime/til for our readers.


Line numbers collide with vertical line

It seems the right-shift of text in the editor doesn't move over far enough for longer listings.  When it reaches > 99 lines, the higher-numbered lines' line number indicators overlap the vertical separator line between the editor and the project/file list.
The effect is most visible at lines in the 300s.  It's uncomfortably close to the fading shadow effect.
Here's a screenshot:

Just a friendly reminder that we need a screenshot for this issue.  Once that's in, I'll reassign.

@urbantumbleweed, done... see first comment.

Thanks, reassigning to @drquinn


Metadata for v2/nodes/{}/files

When getting the list of files, return more than just the Waterbutler-provided information; return the OSF-stored metadata for those files as well.

Apparently the OSF Stores no metadata, and I should be sending along all of the waterbutler created metadata.


Decide on a Hazelcast cluster format

There are members and clients.  The arch is p2p with the distinction that clients do not have to bear the load of storing and replicating the data. Right now every service instance is a member, but it makes sense to have small number of Hazelcast members with lots of RAM and have lots of service instances with much less RAM.

The Hazelcast code now is in the https://github.com/EricssonResearch/scott-eu/commit/39c5c1fc6d4768851f1ba8cbe174a9136ee3e90b commit, but unfortunately I experimented too much and this is a mega commit with all the things from the last week 🙈


Change default region to us-west-2

 Issue by sorenbs
Thursday Jun 22, 2017 at 09:43 GMT
Originally opened as https://github.com/graphcool/api-bugs/issues/135

Change default region to us-west-2

This is handled by CLI and Console. The console already uses ping time to decide the region.


Public CI?

Is there a public CI/build?

we will add it soon

look at Travis CI or Circle CI or Azure Pipelines

we add travis ci to build and run ltp test case.

fixed #41 　#42


Change default output format of arangoexport from json to jsonl

@Simran-B please follow up on this PR according to https://arangodb.atlassian.net/browse/DOC-218

@ansoboleva Do you refer to "like JSON, CSV or XML"? It's a non-exhaustive list. Should XGMML also be mentioned?

@Simran-B yes, this bit. The list doesn't have to be exhaustive, but it should mention the default format.

JSONL is less known than JSON, so we should mention both, but the only format not mentioned would then be XGMML. BTW. the proper name is JSON Lines, not JSONL. Which one should we use?


Update in-product-notices.rst

I think the comma will be placed before "and"
Fixed a minor grammatical change

Summary

Ticket Link

Hello @Sayanta66,
Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.

Thanks for your PR! These changes aren't something we're looking to implement at the moment so I'm closing this PR. :)


downloadJPLSnapshot dosnt exist

it is just not there as well as ADDOONS section. what is the otehr way to go on?

Yea still dosnt work even after changin the nxt.properties file

How to fix it?

Have you followed this:
https://bitbucket.org/Jelurida/nxt-clone-starter/src/master/

yes

It does not take effect

What I did was skip step 3 from the guide, and download the NXT client directly from the official site.
https://bitbucket.org/Jelurida/nxt/downloads/
OPTIONAL: Download the latest database, and replace the nxt_db folder, so you don't need to resync the blockchain. https://www.jelurida.com/NXT-nxt_db.zip
Enable the JPLSnapshot add-on in conf/nxt.properties by setting:
nxt.addOns=nxt.addons.JPLSnapshot
Compile and run.
You should see it in http://localhost:7876/test?requestTag=ADDONS

ok

will try when avalable

Yes, you can drag the files into the nxt folder if you want.


⚠ Bofrost.at has degraded performance

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

HTTP code: 200
Response time: 1185 ms

Resolved: Bofrost.at performance has improved in 93969c1.


Switch packs to 775

770 is restrictive and causes local scripts to fail. Switching to 775 should unblock those features. We can work out how to properly support this so that packs can be more secure.

Feel free to add your comments for the preferred behavior. For now I just want to unblock packages.


♻refactor (config.json) Use new names for outdated topics

Closes #197
Note: A suitable replacement for unicode topic is still missing

@devkabiir Feel free to drop the use of unicode in this PR, I think it is too specific and not general enough of a topic, which is why they don't have a replacement for it.