Vim commands are no longer defined

Since the last refactor, I am no longer able to call the vim commands defined by rust-tools.nvim (RustRunnables, RustSetInlayHints, etc). I've tried with both my personal config and the one shown in the README. Mine looks like this:
require('rust-tools').setup {
  tools = {
    autoSetHints = false,
    hover_with_actions = false,
    runnables = { use_telescope = false },
  },
}

Am I doing something wrong or am I now supposed to define my own commands for the lua functions?

Since the last refactor, I am no longer able to call the vim commands defined by rust-tools.nvim (RustRunnables, RustSetInlayHints, etc). I've tried with both my personal config and the one shown in the README. Mine looks like this:
require('rust-tools').setup {
  tools = {
    autoSetHints = false,
    hover_with_actions = false,
    runnables = { use_telescope = false },
  },
}

Am I doing something wrong or am I now supposed to define my own commands for the lua functions?

The commands seem to work fine for me. FYI the commands are only registered when the LSP is registered, so make sure rust-analyzer is running before trying the commands

I think i found the culprit. The commands appear only when require('rust-tools').setup(opts) is done after the on_attach setup for lspconfig.

Are you by any chance setting up lspconfig for rust analyser yourself? This plugin does that automatically so that might be causing some mismatch.

Nope. I am using the common config suggested in the lspconfig wiki:
local nvim_lsp = require('lspconfig')
local servers = { "clangd", "rust_analyzer", "pyright", "texlab", "hls", "gopls" }
for _, lsp in ipairs(servers) do
  nvim_lsp[lsp].setup { on_attach = on_attach }
end

I think i found the culprit. The commands appear only when require('rust-tools').setup(opts) is done after the on_attach setup for lspconfig. The new problem is that if I do this, the lspconfig rust-analyzer setup seems to break (I no longer get the lsp mappings set in the on_attach function) sob

BTW you can still override rust-analyzer's on attach by passing it into the setup function, see the readme for instructions. You DONT have to setup rust-analyzer yourself, this plugin will do it for you.
I'll close the issue but feel free to open it again if you have any issues.

It all works well now. Thanks for the help!

I'm hitting this too, but I don't really understand the solution.
I have my mapings defined on on_attach, and when I require("rust-tools") it makes all my mappings disappear. This is my LSP config: https://github.com/lovesegfault/nix-config/blob/neovim-0.5/users/bemeurer/core/neovim/lua/lovesegfault/lsp.lua
How can I enable rust-tools without it touching the rest of my config?


Add median pruner

Contributor Agreements
Please read the contributor agreements and if you agree, please click the checkbox below.

[x] I agree to the contributor agreements.

Motivation

This PR aims to add the pruner category in addition to sampler and visualization

Description of the changes

Create thepackage/pruners directory
Add MedianPruner as an example

Screenshots
List

Tag

Details

@y0z I changed the PR status to review-ready. Could you take a look, please?


script to parse concept ancestry for non standard vocabularies

Script will parse through non-standard vocabularies and creates a hierarchy
As of now script should work for

ICD10 and ICD10fi
ICD9CM and ICD9fi
ICD8fi
NCSPfi
VNRfi

replace by #7


Mode indicator isn't updating

Switching from oh my zsh vi-mode. I have a theme I've created based on keymaps. It seems that this plugin is somehow changing the keymap or preventing my theme from updating on keymap changes.
Here is the theme I'm using, pointing to the section where keymap is updated. It hooks into the precmd func.
https://github.com/berkeleytrue/dotfiles/blob/master/.config/zsh/ghanima.zsh-theme#L99

Hi @BerkeleyTrue
Thanks for your reporting this issue, and now this issue has been addressed and settled down.
Thanks and regards

Hi @BerkeleyTrue
Thanks for your reporting this issue, and now this issue has been addressed and settled down.
Thanks and regards

Works!

Works!


SSL Certificates

It is my sincere pleasure to thank you for creating this DNS-proxy application.
When I tried to pull from an HTTPS repository, such as http://archive.ubuntu.com/ubuntu, I received these errors.

Certificate verification failed: The certificate is NOT trusted. The name in the certificate does not match the expected.  Could not handshake: Error in the certificate verification. [IP: MY_PUBLIC_IP 443]
Could not handshake: The TLS connection was non-properly terminated. [IP: MY_PUBLIC_IP 443]
Does this app accept SSL certificates? If so, could you please tell me where they should be entered?

docker run -it --rm -p 80:80 -p 443:443 -p 53:53/udp -v "$(pwd):/tmp/" ghcr.io/mosajjal/sniproxy --domainListPath https://raw.githubusercontent.com/mosajjal/sniproxy/master/domains.csv --bindDnsOverTls --publicIP MY_PUBLIC_IP

Hi there. glad you're enjoying it :)
does this error happen to most domains or only certain URLs? it looks like Ubuntu's servers are doing something that I might've not anticipated and that's why we're seeing the error. Also, <IP_ADDRESS> seems to be unreachable from your server.

This IP address is one of the Ubuntu repositories and can be reached manually (apt update) in this dns-proxy machine.
The majority of domains with HTTP already work, but HTTPS doesn't.
Also, I tried using byosh, everything was fine, but IPv6 didn't work
Does SNI-proxy support IPv6?

I'll test Ubuntu update and try to see if I can replicate the issue. and yes sniproxy should be able to handle ipv6 just fine. let me know if there's any issues there.

hey @x004A try 0.8.x see if that solves your problem. if not, please paste the logs here. I added more logs so it'll be easier to pinpoint the issue


Update Install Script to Print Warning if Used in a Client Environment

The npm install command should warn the user if the admin SDK is installed in a client environment.

what's the proper use-case for firebase-admin? I can't seem to use the Admin sdk on a react web app.

Admin SDK cannot be used in client environments. That includes pretty much all web browser-based runtimes. Admin SDK can only be used in developer-owned, privileged environments.


flask

Type: Feature Request
When I try to do from flask import Flask I get an error saying moduleNot_Found; no module named Flask
VS Code version: Code 1.85.1 (0ee08df0cf4527e40edc9aa28f4b5bd38bbff2b2, 2023-12-13T09:47:11.635Z)
OS version: Linux x64 5.15.108-18910-gab0e1cb584e1
Modes:

/extPython

It looks like this is caused by the Python extension. Please file the issue to the Python extension repository. Make sure to check their issue reporting template and provide them relevant information such as the extension version you're using. See also our issue reporting guidelines for more information.
Happy Coding!

/extPython

But where I run it? I tried all kind of comands but nothing la working. I did it on the terminal of Chromebook Linux.


feat: Add telemetry with the help of segment.io

Close #241
Some commands return so fast. The main goroutine returns too fast to send the async telemetry event.
Signed-off-by: Ce Gao<EMAIL_ADDRESS>

/assign @kemingy @VoVAllen

LGTM. Is it possible to record the envd build time?

I was thinking about it. It is hard because the event is sent async. Thus when the build is finished, the main goroutine exits and we don't have the chance to send the request to the server.


What OS do we need to have on our device?

Windows IoT Core
Linux

I believe both of them works. However, it would be better to go with Windows IoT core because it's a Microsoft challenge.
But there are some technical things we should consider:

Windows IoT Core supports a smaller set of sensor and devices compared to Linux. The List of all supported devices is here.
Choosing Windows IoT lets us use .NET with more ease and less trouble. Although .NET core runs on Linux, I guess there are some difficulties around connecting Camera and Microphone using .NET in Linux. I found some open source libraries but there is no official support. +,+

I agree that Windows IoT is better since it is a Microsoft challenge.
However, should we not take more factors into account? For example, Linux is inherently more scalable than Windows IoT, and so it will also support a larger number of cheaper devices, which makes it more accessible to more people. Also, since we need to take the business aspect into account - we can also make the case that the cost of production is also lowered by using the Linux OS - increasing profits.
Nevertheless, you do know more about C# and .NET framework. I agree that it intertwines better with Windows IoT. To summarize, I guess it's two issues I would be concerned with:

Cost efficiency ---> in terms of using Linux (cheaper to scale to more devices ?)
Accessibility to 3rd party developers ---> Windows IoT is obviously less prevalent than Linux, and so open-sourcing to 3rd party developers will not be as effective if we do it on IoT.

Let me know what you think.


Adding quickstart into Pinot admin command

With type options: for batch/stream/hybrid
So we can run quickstart in docker.

Codecov Report

Merging #5080 into master will increase coverage by 0.14%.
The diff coverage is 0%.

@@             Coverage Diff              @@
##             master    #5080      +/-   ##
============================================
+ Coverage     58.46%   58.61%   +0.14%
  Complexity       12       12
============================================
  Files          1208     1209       +1
  Lines         64512    64531      +19
  Branches       9482     9482
============================================
+ Hits          37720    37824     +104
+ Misses        24038    23952      -86
- Partials       2754     2755       +1

Impacted Files
Coverage Δ
Complexity Δ

...g/apache/pinot/tools/admin/PinotAdministrator.java
0% <ø> (ø)
0 <0> (ø)
:arrow_down:

...e/pinot/tools/admin/command/QuickStartCommand.java
0% <0%> (ø)
0 <0> (?)

.../impl/dictionary/LongOffHeapMutableDictionary.java
61.29% <0%> (-9.68%)
0% <0%> (ø)

.../realtime/impl/ThreadSafeMutableRoaringBitmap.java
92.85% <0%> (-7.15%)
0% <0%> (ø)

...e/operator/dociditerators/BitmapDocIdIterator.java
60.71% <0%> (-7.15%)
0% <0%> (ø)

...e/operator/dociditerators/SortedDocIdIterator.java
58.33% <0%> (-5.56%)
0% <0%> (ø)

...lix/core/realtime/PinotRealtimeSegmentManager.java
77.94% <0%> (-3.08%)
0% <0%> (ø)

...org/apache/pinot/client/DynamicBrokerSelector.java
69.69% <0%> (-3.04%)
0% <0%> (ø)

.../main/java/org/apache/pinot/client/Connection.java
47.61% <0%> (-2.39%)
0% <0%> (ø)

...rg/apache/pinot/core/operator/CombineOperator.java
66.29% <0%> (-2.25%)
0% <0%> (ø)

... and 27 more

Continue to review full report at Codecov.

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


⚠ Bofrost.es has degraded performance

In 9942c76, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1744 ms

Resolved: Bofrost.es performance has improved in dada145 after 6 minutes.


Problem install

brainmaestro/composer-git-hooks v2.6.0 requires php ^5.6 || >=7.0 <7.3 -> your PHP version (7.3.5) does not satisfy that requirement.

brainmaestro/composer-git-hooks v2.6.0 requires php ^5.6 || >=7.0 <7.3 -> your PHP version (7.3.5) does not satisfy that requirement.
brainmaestro/composer-git-hooks v2.6.0 requires php ^5.6 || >=7.0 <7.3 -> your PHP version (7.3.5) does not satisfy that requirement.
Installation request for brainmaestro/composer-git-hooks (locked at v2.6.0) -> satisfiable by brainmaestro/composer-git-hooks[v2.6.0].

Can you upgrade php version to install success. Many thanks!

Did you report this to the wrong repository?
This talks about brainmaestro/composer-git-hooks and not laravel-ide-helper.
The version constraint of this package is "php": ">=7"

@barryvdh seems this was reported in the wrong repo, issue can be closed :}


MRDL->MRTK: Multi-step pointer sources

This is a first pass at adding multi-step pointer support to the FocusManager.
The main change is in IPointingSource, where we go from this:
public interface IPointingSource
{
    ...
    Ray Ray { get; }
    ...
}

To this:
public interface IPointingSource
{
    ...
    RayStep[] Rays { get; }
    ...
}

Where RayStep is a struct with an origin and terminus.
This enables IPointingSources with curving trajectories, similar to what we see in WMR (parabolic navigation pointers, bendy transform pointers, etc).

Because this is a breaking change I'm creating a separate pull request to discuss it. So far I can't see a way to do this without breaking previous IPointingSource implementations, but it would be nice if I'm wrong. Would appreciate thoughts / input / suggestions.
-L

I'll need to look into this in depth quite a bit more.

@Railboy please open a PR for this on my gui raycasting branch.

@StephenHodgson When are we planning to merge the Pointer uGUI raycasting branch? The 214 tutorial depends on multi-step pointers, so we may have to merge this ahead of that branch to get it out on time.

When are we planning to merge the Pointer uGUI raycasting branch?

I'd do it today if I could.  I want more eyes on it.

It sounds like the uGUI merge date is open-ended. If that's the case, what I'll have to do is steer clear of your incoming changes as much as possible, so we can merge this ahead of the uGUI branch and make the tutorial deadline. I'm going through your uGUI branch now to see how I can stay out of your way.

It sounds like the uGUI merge date is open-ended.

I disagree.  We need to merge it asap

Also, we need to update the InputManagerTests to ensure that everything is passing.
@Zod- Could you also review this please?

I hear you. By open-ended I just mean that it sounds like we can't guarantee a merge date, which means I have to plan for the possibility that it won't happen in time to get 214 out the door. If things start looking that way we'll have to shift things around. In the meantime I'll download the uGUI branch and see if I can help with testing.

After going through #1256 I don't see anything that conflicts with these changes in a major way. I leave the critical functions like RaycastUnityUI mostly alone, and I don't go anywhere near your canvas-related stuff. If you merge #1256 by tomorrow I could rebase this by Thursday.

@Railboy don't forget that this needs to pass the InputManagerTests.  If you need help updating the tests don't be afraid to reach out to either me or @Zod-
Run the test via Window/TestRunner/RunAll

I'm going to give running the tests a try, will let you know if I need help.

All tests succeeded except for InputManagerTests->FocusChangeFullIntegration. Should this be a causing a failure, given that it's a warning instead of an error?
FocusChangeFullIntegration (0.032s)
---
Expected: 1
  But was:  0
---
at HoloToolkit.Unity.Tests.InputManagerTests.FocusChangeFullIntegration () [0x00033] in C:\Users\v-lasimk\Documents\GitHub\Railboy\MixedRealityToolkit-Unity\Assets\HoloToolkit-UnitTests\Editor\Input\InputManagerTests.cs:221
---
No UIRaycastCamera assigned! Falling back to the RaycastCamera.
It's highly recommended to use the RaycastCamera found on the EventSystem of this InputManager.

The UIRaycastCamera is falling back correctly, so it shouldn't be causing any test failures.
It passed all tests in the last PR https://github.com/Microsoft/MixedRealityToolkit-Unity/pull/1256#issuecomment-343034910

Got it, I see the problem now. Will push a fix shortly.

All tests are passing now.

Unless there are other ideas or requests ,I'm going to pull the trigger on this.

I haven't had a chance to pull this and build.
Can you confirm that a build for HoloLens and IHMDs works?

Also please remove the duplicated commits from this PR

I'm going to close this PR and submit a completely fresh one.


Getting error while fetching tag field in pre-save signals

I am using django-taggit but unable to get tags when I print(instance.tags) in pre-save signal method. And getting this error.
resource objects need to have a primary key value before you can access their tags

models.py
class resource(models.Model):

    title=models.CharField(max_length=100)
    size=models.CharField( max_length=20, default="")
    desc=models.TextField(default="")
    file=models.FileField(default="", blank=True)
    url= models.URLField(max_length=200, blank=True)
    varient=models.CharField(max_length=100, default="")
    Brand = models.ForeignKey(brand,on_delete=models.CASCADE, default="")
    Model = models.ForeignKey(model,on_delete=models.CASCADE, default="")
    Categories = models.ForeignKey(category,on_delete=models.CASCADE, default="")
    update_at=models.DateField(auto_now=True)
    slug=models.SlugField(default="", unique=True, blank=True)
    Tags = TaggableManager(blank=True)`

pre-save function
def tag_set(sender, instance, **kwargs):

   print(instance.Tags)

pre_save.connect(tag_set, sender=resource)

@naveedur tags are stored as a many-to-many relation, so instance.Tags actually fetches the data from the database. In order to save tags in the database, you want to first save the resource (why? because the tags are saved in a table that have an ID for the "tag object" that hold the name, and an ID for the resource itself)
Since you're in the pre-save method, your object is not saved... and nothing is in the database!
So kinda dependent on what you are doing, but you can't really access the tags in the pre-save method in a way that you would like. The tags are not stored on the resource itself, but in a separate table. I understand that this is a bit conceptually confusing though.

@naveedur tags are stored as a many-to-many relation, so instance.Tags actually fetches the data from the database. In order to save tags in the database, you want to first save the resource (why? because the tags are saved in a table that have an ID for the "tag object" that hold the name, and an ID for the resource itself)
Since you're in the pre-save method, your object is not saved... and nothing is in the database!
So kinda dependent on what you are doing, but you can't really access the tags in the pre-save method in a way that you would like. The tags are not stored on the resource itself, but in a separate table. I understand that this is a bit conceptually confusing though.

@rtpg, thanks for the reply. I understand what you are saying. But is there any method to do this.


Get the api token from an env var

This allows passing to hubtraf-check the name of the env var holding the Hub api token.
We could also assume its name (eg JUPYTERHUB_API_TOKEN) and read it directly.
@yuvipanda, WDYT? Does this need to be customizable?

Thanks for the PR, @GeorgianaElena!


Linux ARM (arm7l, arm8l) sdks in downloadable zip files (2.0 and 1.24) builds missing bin/dart2js and supporting snapshots and infrastucture.

Thank you for taking the time to file an issue!
In order to route, prioritize, and act on this, please include:

Dart SDK Version (dart --version)
Whether you are using Windows, MacOSX, or Linux (if applicable)
Whether you are using Chrome, Safari, Firefox, Edge (if applicable)

Missing some or all of the above might make the issue take longer or be
impossible to act on.

Is it really an issue? For general questions consider starting with Stack
Overflow:
https://stackoverflow.com/questions/tagged/dart
Also consider our Gitter channel for light-weight/quick discussions:
https://gitter.im/dart-lang/sdk

These are present the in the current version of the SDK


Add Herbiboar to the boss killcount leaderboards

Even though Herbiboar is technically not a boss, the killcount is still tallied in the chatbox. I think it would be cool to be able to see everyone elses killcount on Herbiboar on the leaderboard search. Seems like it wouldn't really hurt to add it to the list since Tempoross is added to the list as well.

We only list bosses which are listed on the official Old School RuneScape hiscores. Presently, you can share your herbiboar kill count using !kc herbiboar command with the Chat Commands plugin.


Store RevertReason in storage

I assume we don't need a migration here since RevertReason will unmarshal to ""?

Yeap, you are correct.


add download links for visualizations

simple download link is hooked up. I still need to do a little bit of bug fixing in d3plus, but it's there!
also, need the icon from @housegeoff 😄


Suggest adding a row in the table for Azure Container Registry

Suggest adding a row in the table for Azure Container Registry:
Azure Container Registry | *.Azurecr.io
Thanks for all your hard work.

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

ID: 790ff17b-0306-897d-f0df-59dd92441426
Version Independent ID: 6bfac7c1-8909-73f9-4212-26cc7ffaaa81
Content: Reference list of Azure domains
Content Source: articles/security/fundamentals/azure-domains.md
Service: security
Sub-service: security-fundamentals
GitHub Login: @memildin
Microsoft Alias: memildin

@cqueern Thank you for pointing that out. We will investigate and update the thread.

@memildin Could you please help on the same ?

Thanks for taking the time to help improve this page, @cqueern
I've added the ACR domain to the table and the changes will go live within 24hrs.
#sign-off

Thank you Team for your hard work.


Mouse crosshairs turns Focus Assist on.

Microsoft PowerToys version
0.55.1
Running as admin

[ ] Yes

Area(s) with issue?
Mouse Utilities
Steps to reproduce
Enable Mouse Crosshairs
✔ Expected Behavior
Focus assist doesn't turn on
❌ Actual Behavior
Focus assist turns on
Other Software
No response

Does this happen the moment you flip the toggle switch (in settings), or when you press the hotkey? What hotkey do you have?

Does this happen the moment you flip the toggle switch (in settings), or when you press the hotkey? What hotkey do you have?

I have it too, with the standard [Ctrl] + [Alt] + [P] hotkey. I tested the shortcut itself without having the crosshairs enabled and it's not a default hotkey for Focus Assist already. (The Focus Assist didn't turn on when pressing [Ctrl] + [Alt] + [P], only when the crosshairs are enabled in the settings and then you press the hotkey.)

Fun fact, if you disable Focus Assist manually from the Action Center, and close the Action Center, it re-enables itself. (As long as the crosshairs are turned on)

Same cause as #14122

Same cause as #14122

Thanks, I solved it by disabling the setting given in #14122

I have this issue and the solution above does not solve it for me. This bug is still unresolved. Anytime I have Mouse Crosshairs enabled and turned on, all incoming notifications function as if Focus Assist is enabled (Windows 10). No mail desktop alerts or other action center notifications appear or make sounds so long as Mouse Crosshairs is on. However, Focus Assist appears to still be turned off (even though it is off) and I have checked and re-checked ALL the settings in Focus Assist (including the settings given in . Please advise of any solutions you may have. Thank you.


Update Batch command module

#3715 #5142 #6751 #6914 #6916
Update adding tasks through --json-file to support AddTaskCollectionParameter syntax
Update documentation of accepted --json-file formats
Add --max-tasks-per-node-option to az batch pool create
Update show behavior of az batch account to show currently logged in account if no options are specified.
Separate test case needing advanced permissions to allow for remaining tests to be executed

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] The PR has modified HISTORY.rst describing any customer-facing, functional changes. Note that this does not include changes only to help content. (see Modifying change log).

[x] I adhere to the Command Guidelines.

@bgklein Fix the CI error: https://travis-ci.org/Azure/azure-cli/jobs/420310936

Somehow it looks like my environment got an old version of the mgmt sdk.  Regenerated test env and will re-record tests

@troydai @xingwu1 have addressed the major concerns and passing CI.
With regards to nested try-excepts I will differ to judgement of what cli team wants in their codebase.
The main feasible alternate implementation I can think of involves returning from the try block and swallowing parsing errors. We still have the same amount of try blocks using this method but then they are sequential rather than embedded.

Let's leave this behind for now. I do like to keep the ball rolling. I will take another look at the changes.

@troydai made requested changes.  Error in CI though. Tried to fix, but don't see how my changes  would've broken.  Looks to be there was a pypi release to 2.0.45 but latest in the github repo hasn't been updated. Could this be the cause?

Please rebase on the latest dev, then the CI problem will be resolved.

The CI will probably fail again because of an external dependency change. The following PR will resolve it: https://github.com/Azure/azure-cli/pull/7167

Ok ... seems you're lucky. I will merge this as soon as PR #7167 is merged and dev turns green.


Added Windows Support

Changes

Added windows support to the GitHub Action, as outlined in #129
Added new test case in src/model/results-parser.ts to account for issue outlined in comment, https://github.com/game-ci/unity-test-runner/issues/129#issuecomment-1133424203
Fixed null pointer exception that can happen in src/model/results-parser.ts

Checklist

[x] Read the contribution guide and accept the code of conduct
[x] Readme (not needed)
[x] Tests (updated)
[x] Confirmed to work on a Windows environment (tested with a Unity Professional License and on a self hosted runner)

@webbertakken Thank you for taking the time to review my PR! I went ahead and made the edits to my PR to reflect the changes done in game-ci/unity-builder for windows support. It should be much easier to read now.
There's a slight difference in using a class vs const in the docker.ts (since the game-ci/unity-builder was using a class) for this PR, but upon testing it and linting it, it still works as intended.
Let me know if there's anything I should clarify on. 😃

Excellent! Thank you so much for taking the time to process that feedback.
LGTM from my side.


update front page

Pull Request Test Coverage Report for Build 182

11 of 11 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.09%) to 54.48%

Totals

Change from base Build 179:
0.09%

Covered Lines:
4188

Relevant Lines:
7125

💛  - Coveralls


support array append

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@9325843). Click here to learn what that means.
The diff coverage is n/a.

:exclamation: Current head da86aaa differs from pull request most recent head dd14fdf. Consider uploading reports for the commit dd14fdf to get more accurate results

@@            Coverage Diff            @@
##             master       #6   +/-   ##
=========================================
  Coverage          ?   50.00%
=========================================
  Files             ?        2
  Lines             ?       62
  Branches          ?        1
=========================================
  Hits              ?       31
  Misses            ?       31
  Partials          ?        0

Flag
Coverage Δ

csharp
50.00% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


fix(rust): Allow more than 2 hops when paring show secure channel rou…

Current Behavior
After creating secure channel with a forwarder node, the secure-channel list command returns invalid route error message.
Proposed Changes
When parsing the route returned in the ShowSecureChannelResponse, allow the route contains more than two parts.
Fix #3989
Checks

[X] All commits in this Pull Request are signed and Verified by Github.
[X] All commits in this Pull Request follow the Ockam commit message convention.
[X] I accept the Ockam Community Code of Conduct.
[X] I have accepted the Ockam Contributor License Agreement by adding my Git/Github details in a row at the end of the CONTRIBUTORS.csv file in a separate pull request to the build-trust/ockam-contributors repository.

Great, thanks @0x00A5 for the PR. The changes look good but I'd ask you to include the following command near the end of this test, right after creating the secure-channel and tcp-inlet: https://github.com/build-trust/ockam/blob/b7a15c2d9ee0c85c16b4c76addd0c3adf846e558/implementations/rust/ockam/ockam_command/tests/bats/unit.bats#L347
ockam secure-channel list --at green
You can assert the output with assert_output --partial "whatever" to check the From and To lines.

When asserting the output, the test fails due to the curl output from the previous step. I am not very familiar with bats, and by glancing through its documentation I didn't find a way to neglect the curl output. Do you have any suggestion?

You have to use the run bats function to capture the output, so you want to write the list command as run "$OCKAM" secure-channel list --at green. Then the follow-up assert calls will be checked against that command output.


Change type of chargemaster adapter to energy

I think this fits better than vehicle ??

RE-CHECK!

RE-CHECK!


Imaging Selection Features Not Working in Jupyter

Describe the bug
Check boxes (turning channels on/off) do not work in Jupyter for imaging.
To Reproduce
Open the imaging notebook off of my branch and then try to turn off and on channels
Expected behavior
Channels should turn off and on
Environment:

Release or git hash: 1.1.2
Browser: Firefox
Browser version: 84.0.1

Relatedly, the dropdowns are also buggy.

Relatedly, the dropdowns are also buggy.

Looks like a CSS bug, the actual input element is too short for some reason

Looks like a CSS bug, the actual input element is too short for some reason

The Jupyter Lab height: auto is taking precedence over the Material UI height: 100%. I wish the MUI styles would scope themselves under a parent class to give them higher specificity. I have had issues with the MUI style specificity in the docs site too.

The Jupyter Lab height: auto is taking precedence over the Material UI height: 100%. I wish the MUI styles would scope themselves under a parent class to give them higher specificity. I have had issues with the MUI style specificity in the docs site too.

Looks like a CSS bug, the actual input element is too short for some reason

This is what I just discovered as well....wish I had seen this issue first.  I will look into solving this.

I believe this issue is isolated to jupyter lab, not the actual notebooks themselves.  There is a different issue with the style in the notebooks where everything is tiny.  Not sure if it's related:

Closing, I believe this was fixed in https://github.com/vitessce/vitessce/pull/1375


Problematic handling of startup warning messages

Recently ran into something very similar to #225 with the latest branch. Though it has been fixed, I think there is a tiny little problem in the regexp,
https://github.com/whonore/Coqtail/blob/f88e1b425d96b331a5375d6e35c741abdedc858d/python/xmlInterface.py#L53
There should be no whitespace after Warning:. Removing that fixed my problem locally.

Although I don't see any problem with removing the space from the regex, I haven't seen a case yet where keeping it breaks anything either. Do you have an easy way I can try to reproduce your problem?

Invitation sent.
Not sure how to trigger other warnings so just sent my original repo... Many thanks if you are okay with it!

Thanks, it looks like the problem is when the warning message is long enough (e.g., because of a long file path that doesn't exist) it wraps to the next line so there's a newline instead of a space after Warning:. Your suggestion of just removing the space from the regex should be all that's needed.


Express checkout for logged in users

What I'm trying to achieve

Ignore Add to cart -> Go to cart -> Checkout steps
Allow user to Checkout a single product immediately

Describe a proposed solution
https://github.com/mirumee/saleor/blob/9502467c0e745eb8afdbfa373d634814d133e864/saleor/graphql/checkout/mutations.py#L290-L300
According to the code above, a logged-in user is limited to have single Checkout while anonymous user can have multiple Checkout.

Remove the limitation
Add a new field is_temporary(for example) to Checkout
Add a new parameter is_temporary to CheckoutCreate mutation. If is_temporary is true, create a new Checkout with is_temporary set to true.
Query.me.checkout returns the latest Checkout with is_temporary set to false.

@gsoec Hey! Due to the holiday season, our support is limited. We'll get back to you as soon as possible. Thanks for understanding.

@gsoec I've sent your suggestion to our Saleor team. We appreciate your support!


Multiple joins

Currently sq only supports joining two tables, it should be possible join an arbitrary number of tables

Released in v0.40.0.
See docs.


feat(@schematics/angular): add enableI18nLegacyMessageIdFormat defa…

…ult to new projects
The ViewEngine message extraction generates a variety of legacy formats
for extracted message ids. These formats have a number of issues related
to whitespace handling and reliance upon information inside the original
HTML of a template. The new message format is more resilient to things
like whitespace changes, and can be generated directly from calls to
$localize, which allows messages in application code to have the same
id as identical messages in templates.
As a first step in migrating projects away from the legacy id format
for i18n messages, this commit updates newly generated projects to turn
off the legacy ids.
In the future the default will be flipped and this can be removed.
Eventually the legacy message id support will be removed altogether,
probably in sync with removal of ViewEngine.

Related docs fix here https://github.com/angular/angular/pull/39453


Soft line break button missing in Draftail panel

Issue Summary
Comparing two Wagtail instances – <v3.0 and v4.2 – the newer Draftail editor UI is missing the button for "soft line break", the one with the return arrow icon. See screenshots.
Before (with line break button):

After (without line break button:

I tried to add this button via the feature list, but both "linebreak", "line-break" and "br" didn’t work as I was hoping. The docs and release notes are not mentioning this as well, as far as I can tell after a quick search. How can I get this button in the most recent Wagtail versions?

Hi @andre-fuchs, thank you for the report. This is likely an unintended regression from previous releases – will investigate further.

Hi @andre-fuchs, I’m unable to reproduce this so far. We did change this toolbar quite a bit in the next release but it looks to me like it’s still not possible for the line break feature to be missing (it’s hard-coded, see #4445).
Here is a screenshot of the default toolbar options in a brand new site with no customisations:

You might have more customisations on your site that explain this?

Thanks @thibaudcolas for looking into it. I cannot find any customization to this line break feature. Here is the list of features for this specific TextBlock that lacks the line break button in the UI.
class BaseTextBlock(StructBlock):
    style = ChoiceBlock(TEXT_STYLES, icon='cog', required=False)
    text = RichTextBlock(
        label="Text",
        icon="pilcrow",
        template="blocks/text_block.html",
        features=['h2', 'h3', 'bold', 'italic', 'link', 'ol', 'ul', 'artist', 'small', ],
    )

    class Meta:
        label="Text"
        icon = "pilcrow"
        template = "blocks/text_block.html"
        group = '(1)  Text'

It contains two custom buttons, named 'artist' and 'small'. The one is an inline feature the other a block feature. If I remove these the line break button is still missing.

We’ve added it back in v4.2, along with all other toolbar options except for undo/redo buttons, so the toolbar options are the same for both floating and pinned toolbars.

This solves it. Thanks!


Animation: Update Page Autoadvance calculation

Right now behavior is:

if animation or media present, use max duration of either for page duration
if neither is present, use auto advance duration for page duration

We want to update to:

use the max of auto advance duration, media duration and animation duration
this only applies if the story is set to auto-advance, if set to manual advance then ignore all this. :)

Looks good! Can we fit in this fix for the release on Wednesday? @carlos-kelly @littlemilkstudio @choumx

Looks good! Can we fit in this fix for the release on Wednesday?

yea absolutely! I'll put up a PR for this rn

Verified in QA


Removed java.util.Objects

Replaced the java.util.Objects usage since it has been added on API level 19 (this library should works with API >= 16).
I just tested it on 3 emulators and it seems to work correctly :)

P.S.: Thanks for this lib guys!

Hey, great stuff! Thank you, will do a (super late, sorry about that!) merge.


Add more dependencies

@alvaradobonilla
This adds the dependencies of Polymer so that they are directly included in the acwater package on conda-forge. Actually, these dependencies should be included in a conda-forge package "polymer", however, as long as this does not exist, I think it makes sense to include them here. This simplifies the installation procedure on the EnPT side.

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

@conda-forge-admin, please rerender


现在只能下载试看部分了。看来改变了加密策略

Describe the bug
Only the trial part can be downloaded now. It seems that the encryption strategy has been changed
python udemy-dl.py https://www.udemy.com/course/deeplearning/ -k 1.txt -o "C:/"
Python Environment (please complete the following information):

OS Platform: [Windows 10]
Python Version [e.g. 3.9]

Additional context
Add any other context about the problem here.

more related or same to #627 as the cause is same so closing the issue as dup


Error During Restore: IO Error

Hi, Im having difficulties restoring sms to my phone.
OnePlus One, CyanogenOS, 6.0.1, no root.
I have no problem backing SMS up. Please assist.
I ran logcat on PC and get the output below.
Let me know if there are any other information I can share to help fix this.
 Origin
al location:Location[fused 3.203899,101.735894 acc=30 et=+25m29s408ms {Bundle[{w
ifiScan=[B@771eeb3, noGPSLocation=Location[fused 3.203899,101.735894 acc=30 et=+
23m15s670ms {Bundle[mParcelledData.dataSize=188]}], locationType=3}]}] because S
MD didn't trigger.
09-06 21:40:25.642  3296  8796 I GCoreUlr: Successfully inserted 1 locations
09-06 21:40:29.412  1715  4734 D NetlinkSocketObserver: NeighborEvent{elapsedMs=
1533253, <IP_ADDRESS>, [E48D8C4DADBC], RTM_NEWNEIGH, NUD_PROBE}
09-06 21:40:31.534 16426 16760 E k9      : IOException for com.zegoggles.smssync
.mail.BackupStoreConfig@5646414:SMS/AsyncTask #4/conn83497510
09-06 21:40:31.534 16426 16760 E k9      : java.net.SocketTimeoutException: Read
 timed out
09-06 21:40:31.534 16426 16760 E k9      :      at com.android.org.conscrypt.Nat
iveCrypto.SSL_read(Native Method)
09-06 21:40:31.534 16426 16760 E k9      :      at com.android.org.conscrypt.Ope
nSSLSocketImpl$SSLInputStream.read(OpenSSLSocketImpl.java:705)
09-06 21:40:31.534 16426 16760 E k9      :      at java.io.InputStream.read(Inpu
tStream.java:162)
09-06 21:40:31.534 16426 16760 E k9      :      at java.io.BufferedInputStream.f
illbuf(BufferedInputStream.java:149)
09-06 21:40:31.534 16426 16760 E k9      :      at java.io.BufferedInputStream.r
ead(BufferedInputStream.java:234)
09-06 21:40:31.534 16426 16760 E k9      :      at com.fsck.k9.mail.filter.Peeka
bleInputStream.peek(PeekableInputStream.java:34)
09-06 21:40:31.534 16426 16760 E k9      :      at com.fsck.k9.mail.store.imap.I
mapResponseParser.readResponse(ImapResponseParser.java:41)
09-06 21:40:31.534 16426 16760 E k9      :      at com.fsck.k9.mail.store.imap.I
mapResponseParser.readResponse(ImapResponseParser.java:32)
09-06 21:40:31.534 16426 16760 E k9      :      at com.fsck.k9.mail.store.imap.I
mapResponseParser.readStatusResponse(ImapResponseParser.java:73)
09-06 21:40:31.534 16426 16760 E k9      :      at com.fsck.k9.mail.store.imap.I
mapConnection.executeSimpleCommand(ImapConnection.java:307)
09-06 21:40:31.534 16426 16760 E k9      :      at com.fsck.k9.mail.store.imap.I
mapConnection.executeSimpleCommand(ImapConnection.java:288)
09-06 21:40:31.534 16426 16760 E k9      :      at com.fsck.k9.mail.store.imap.I
mapStore$ImapFolder.executeSimpleCommand(ImapStore.java:724)
09-06 21:40:31.534 16426 16760 E k9      :      at com.zegoggles.smssync.mail.Ba
ckupImapStore$BackupFolder.access$200(BackupImapStore.java:140)
09-06 21:40:31.534 16426 16760 E k9      :      at com.zegoggles.smssync.mail.Ba
ckupImapStore$BackupFolder$1.search(BackupImapStore.java:164)
09-06 21:40:31.534 16426 16760 E k9      :      at com.fsck.k9.mail.store.imap.I
mapStore$ImapFolder.search(ImapStore.java:1259)
09-06 21:40:31.534 16426 16760 E k9      :      at com.zegoggles.smssync.mail.Ba
ckupImapStore$BackupFolder.getMessages(BackupImapStore.java:168)
09-06 21:40:31.534 16426 16760 E k9      :      at com.zegoggles.smssync.service
.RestoreTask.restore(RestoreTask.java:105)
09-06 21:40:31.534 16426 16760 E k9      :      at com.zegoggles.smssync.service
.RestoreTask.doInBackground(RestoreTask.java:85)
09-06 21:40:31.534 16426 16760 E k9      :      at com.zegoggles.smssync.service
.RestoreTask.doInBackground(RestoreTask.java:47)
09-06 21:40:31.534 16426 16760 E k9      :      at android.os.AsyncTask$2.call(A
syncTask.java:295)
09-06 21:40:31.534 16426 16760 E k9      :      at java.util.concurrent.FutureTa
sk.run(FutureTask.java:237)
09-06 21:40:31.534 16426 16760 E k9      :      at java.util.concurrent.ThreadPo
olExecutor.runWorker(ThreadPoolExecutor.java:1113)
09-06 21:40:31.534 16426 16760 E k9      :      at java.util.concurrent.ThreadPo
olExecutor$Worker.run(ThreadPoolExecutor.java:588)
09-06 21:40:31.534 16426 16760 E k9      :      at java.lang.Thread.run(Thread.j
ava:818)
09-06 21:40:31.542 16426 16760 E SMSBackup+: error
09-06 21:40:31.542 16426 16760 E SMSBackup+: com.fsck.k9.mail.MessagingException
: IO Error
09-06 21:40:31.542 16426 16760 E SMSBackup+:    at com.fsck.k9.mail.store.imap.I
mapStore$ImapFolder.ioExceptionHandler(ImapStore.java:2087)
09-06 21:40:31.542 16426 16760 E SMSBackup+:    at com.fsck.k9.mail.store.imap.I
mapStore$ImapFolder.search(ImapStore.java:1288)
09-06 21:40:31.542 16426 16760 E SMSBackup+:    at com.zegoggles.smssync.mail.Ba
ckupImapStore$BackupFolder.getMessages(BackupImapStore.java:168)
09-06 21:40:31.542 16426 16760 E SMSBackup+:    at com.zegoggles.smssync.service
.RestoreTask.restore(RestoreTask.java:105)
09-06 21:40:31.542 16426 16760 E SMSBackup+:    at com.zegoggles.smssync.service
.RestoreTask.doInBackground(RestoreTask.java:85)
09-06 21:40:31.542 16426 16760 E SMSBackup+:    at com.zegoggles.smssync.service
.RestoreTask.doInBackground(RestoreTask.java:47)
09-06 21:40:31.542 16426 16760 E SMSBackup+:    at android.os.AsyncTask$2.call(A
syncTask.java:295)
09-06 21:40:31.542 16426 16760 E SMSBackup+:    at java.util.concurrent.FutureTa
sk.run(FutureTask.java:237)
09-06 21:40:31.542 16426 16760 E SMSBackup+:    at java.util.concurrent.ThreadPo
olExecutor.runWorker(ThreadPoolExecutor.java:1113)
09-06 21:40:31.542 16426 16760 E SMSBackup+:    at java.util.concurrent.ThreadPo
olExecutor$Worker.run(ThreadPoolExecutor.java:588)
09-06 21:40:31.542 16426 16760 E SMSBackup+:    at java.lang.Thread.run(Thread.j
ava:818)
09-06 21:40:31.542 16426 16760 E SMSBackup+: Caused by: java.net.SocketTimeoutEx
ception: Read timed out
09-06 21:40:31.542 16426 16760 E SMSBackup+:    at com.android.org.conscrypt.Nat
iveCrypto.SSL_read(Native Method)
09-06 21:40:31.542 16426 16760 E SMSBackup+:    at com.android.org.conscrypt.Ope
nSSLSocketImpl$SSLInputStream.read(OpenSSLSocketImpl.java:705)
09-06 21:40:31.542 16426 16760 E SMSBackup+:    at java.io.InputStream.read(Inpu
tStream.java:162)
09-06 21:40:31.542 16426 16760 E SMSBackup+:    at java.io.BufferedInputStream.f
illbuf(BufferedInputStream.java:149)
09-06 21:40:31.542 16426 16760 E SMSBackup+:    at java.io.BufferedInputStream.r
ead(BufferedInputStream.java:234)
09-06 21:40:31.542 16426 16760 E SMSBackup+:    at com.fsck.k9.mail.filter.Peeka
bleInputStream.peek(PeekableInputStream.java:34)
09-06 21:40:31.542 16426 16760 E SMSBackup+:    at com.fsck.k9.mail.store.imap.I
mapResponseParser.readResponse(ImapResponseParser.java:41)
09-06 21:40:31.542 16426 16760 E SMSBackup+:    at com.fsck.k9.mail.store.imap.I
mapResponseParser.readResponse(ImapResponseParser.java:32)
09-06 21:40:31.542 16426 16760 E SMSBackup+:    at com.fsck.k9.mail.store.imap.I
mapResponseParser.readStatusResponse(ImapResponseParser.java:73)
09-06 21:40:31.542 16426 16760 E SMSBackup+:    at com.fsck.k9.mail.store.imap.I
mapConnection.executeSimpleCommand(ImapConnection.java:307)
09-06 21:40:31.542 16426 16760 E SMSBackup+:    at com.fsck.k9.mail.store.imap.I
mapConnection.executeSimpleCommand(ImapConnection.java:288)
09-06 21:40:31.542 16426 16760 E SMSBackup+:    at com.fsck.k9.mail.store.imap.I
mapStore$ImapFolder.executeSimpleCommand(ImapStore.java:724)
09-06 21:40:31.542 16426 16760 E SMSBackup+:    at com.zegoggles.smssync.mail.Ba
ckupImapStore$BackupFolder.access$200(BackupImapStore.java:140)
09-06 21:40:31.542 16426 16760 E SMSBackup+:    at com.zegoggles.smssync.mail.Ba
ckupImapStore$BackupFolder$1.search(BackupImapStore.java:164)
09-06 21:40:31.542 16426 16760 E SMSBackup+:    at com.fsck.k9.mail.store.imap.I
mapStore$ImapFolder.search(ImapStore.java:1259)
09-06 21:40:31.542 16426 16760 E SMSBackup+:    ... 9 more
09-06 21:40:31.546  1715  4049 D WifiService: releaseWifiLockLocked: WifiLock{SM
SBackup+ type=3 binder=android.os.BinderProxy@1fda5bb}
09-06 21:40:31.561 16426 16426 D SMSBackup+: finished (RestoreStateChanged{state
=ERROR, currentRestoredCount=0, itemsToRestore=0, actualRestoredCount=0, duplica
teCount=0}/0)
09-06 21:40:31.570 16426 16426 D SMSBackup+: restoring SMS provider com.google.a
ndroid.apps.messaging
09-06 21:40:31.571  1715  3819 I ActivityManager: START u0 {act=android.provider
.Telephony.ACTION_CHANGE_DEFAULT cmp=com.android.settings/.SmsDefaultDialog (has
 extras)} from uid 10200 on display 0
09-06 21:40:31.588 16426 16426 D SMSBackup+: stopping service, stateRestoreState
Changed{state=ERROR, currentRestoredCount=0, itemsToRestore=0, actualRestoredCou
nt=0, duplicateCount=0}
09-06 21:40:31.658  2801  2801 W Binder_3: type=1400 audit(0.0:1156): avc: denie
d { ioctl } for path="socket:[143072]" dev="sockfs" ino=143072 ioctlcmd=7704 sco
ntext=u:r:system_server:s0 tcontext=u:r:system_server:s0 tclass=unix_stream_sock
et permissive=0
09-06 21:40:31.658  2801  2801 W Binder_3: type=1400 audit(0.0:1157): avc: denie
d { ioctl } for path="socket:[143072]" dev="sockfs" ino=143072 ioctlcmd=7704 sco
ntext=u:r:system_server:s0 tcontext=u:r:system_server:s0 tclass=unix_stream_sock
et permissive=0
09-06 21:40:31.705  1715  1733 I ActivityManager: Displayed com.android.settings
/.SmsDefaultDialog: +99ms
09-06 21:40:36.818 11415 11447 I PlayCommon: [810] com.google.android.play.a.l.e
(787): Preparing logs for uploading
09-06 21:40:36.818 11415 11447 I PlayCommon: [810] com.google.android.play.a.l.e
(789): No file ready to send
09-06 21:40:37.252 11415 11496 I PlayCommon: [823] com.google.android.play.a.l.e
(787): Preparing logs for uploading
09-06 21:40:37.252 11415 11496 I PlayCommon: [823] com.google.android.play.a.l.e
(789): No file ready to send
09-06 21:40:37.320 11415 11493 I PlayCommon: [820] com.google.android.play.a.l.e
(787): Preparing logs for uploading
09-06 21:40:37.320 11415 11493 I PlayCommon: [820] com.google.android.play.a.l.e
(789): No file ready to send
09-06 21:40:37.330 11415 11495 I PlayCommon: [822] com.google.android.play.a.l.e
(787): Preparing logs for uploading
09-06 21:40:37.330 11415 11494 I PlayCommon: [821] com.google.android.play.a.l.e
(787): Preparing logs for uploading
09-06 21:40:37.330 11415 11495 I PlayCommon: [822] com.google.android.play.a.l.e
(789): No file ready to send
09-06 21:40:37.330 11415 11494 I PlayCommon: [821] com.google.android.play.a.l.e
(789): No file ready to send

I see several discussions directed to 675, but no fix. Please advise.

Agreed. It appears all support is down for this app.

It's a timeout error, from the server. How many items are you trying to restore? You can also try the beta version which has some improvements.

I have just updated to the beta version of sms backup+ and no change, I am also having the same issues as described here on a Samsung Galaxy S5  with android version 4.4.2 I'm curious if some Google security settings might be causing these issues?

I would also like to note that when I turn the log on and try to restore nothing is kept in the log

There's another beta now (1.5.11-beta7) which you can try

I am having the same problem with the restore io error message. I have a little over 6000 messages i would like to restore and I see no resolution anywhere on this thread. My messages are also being backed up in the incorrect order of date received. Please help.

@nikkib1181 what version? Anyhow, see https://github.com/jberkel/sms-backup-plus#faq-restore-many-messages - restoring tons of messages is not really supported.
I leave old messages in my email - restoring them to my phone seems to be more trouble than it's worth.
@jberkel I think we should stick to closing issues with restoring thousands of messages and strongly recommend the advice at https://github.com/jberkel/sms-backup-plus#faq-restore-partial

Thank you so much for your response. I'm using version 1.5.11. What if I restore messages from just one number? Am I able to do that?
And what about my time stamp and message order being incorrect? Is there a solution for that?
Many thanks!

You could try this https://github.com/jberkel/sms-backup-plus#faq-restore-partial:

If you have a lot of messages backed up (let's say over 5000) restoring can be very slow, especially if you're only interested in the most recent messages.
A workaround is to use the Gmail web interface (or an IMAP email client) to move the bulk of the messages to another label in Gmail (e.g. SMSARCHIVED), and only keep a few hundred or so messages in the SMS label.

As far as the timestamps issue, can you open another issue? looks like a duplicate of https://github.com/jberkel/sms-backup-plus/issues/94 - we have an link about it in the Table of Contents but it's broken https://github.com/jberkel/sms-backup-plus/commit/b35efe23aefb4943b8c96c72dde3d3344b44b9fd#diff-04c6e90faac2675aa89e2176d2eec7d8R346

@joshuatly can you use the partial restoration method advised above?
There's also a new beta version if you would like to try that.
The heart of the matter would appear to be this log line:
09-06 21:40:31.534 16426 16760 E k9      : java.net.SocketTimeoutException: Read timed out

which is almost certainly caused by the large number of messages involved.

no response, closing issue


Drum Kit Website added

No c++/c flies are there in the project;
i have star the repo
Started following you ;
there s a snapshot of the website for your reference  in the folder as well and im attaching here also;

You really started following him?

Nevermind

Hey I followed the rules but still it's showing not accepted ?
On Mon, 3 Oct, 2022, 10:52 pm Nehal, @.***> wrote:

Merged #196 https://github.com/shubham7668/hacktober/pull/196 into main.
—
Reply to this email directly, view it on GitHub
https://github.com/shubham7668/hacktober/pull/196#event-7509363777, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ATQZMK5K7YQ2BOMXH6J27ALWBMI67ANCNFSM6AAAAAAQ3ZAZ4Y
.
You are receiving this because you authored the thread.Message ID:
@.***>

Hey there , my submission is getting counted as not accepted on hactober site !!

Hey there my pr is not getting count for hacktober , what's the problem?


fix #425 change version to 0.2.2-SNAPSHOT

Ⅰ. Describe what this PR did
change version to 0.2.2-SNAPSHOT
Ⅱ. Does this pull request fix one issue?
fix #425 change version to 0.2.2-SNAPSHOT
Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Codecov Report

Merging #426 into develop will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             develop     #426   +/-   ##
==========================================
  Coverage      29.68%   29.68%
  Complexity       626      626
==========================================
  Files            176      176
  Lines           6962     6962
  Branches         848      848
==========================================
  Hits            2067     2067
  Misses          4677     4677
  Partials         218      218

Impacted Files
Coverage Δ
Complexity Δ

...java/com/alibaba/fescar/core/protocol/Version.java
0% <ø> (ø)
0 <0> (ø)
:arrow_down:

Continue to review full report at Codecov.

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


added an initializer for SectionHeaderTitleFormItem that accepts the title

I have merged your changes to the 'develop' branch and made a few minor changes, see here.


Warning if endpoint is removed which never had policy attached

WARN[0120] Unable to close policy map /sys/fs/bpf/tc/globals/cilium_policy_47130: bad file descriptor

Likely caused by closing a file descriptor with value 0

False negative


Provide exhaustive mapping rules for Airflow StatsD Integration

An Airflow user might choose to use dots in their DAG name. The Pattern will then not match and produce very expensive custom metrics in datadog.
With this change, all DAG names are accepted
What does this PR do?
Update Airflow Statsd integration documentation
Motivation
The original mapping rules cost us a lot of money
Additional Notes

Review checklist (to be filled by reviewers)

[x] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[x] PR title must be written as a CHANGELOG entry (see why)
[x] Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
[x] PR must have changelog/ and integration/ labels attached
[x] If the PR doesn't need to be tested during QA, please add a qa/skip-qa label.

Thanks for the PR, I've created DOCS-5498 to review.

@chillinger Thanks for the contribution! Would you mind giving me a few example of DAGS that this PR is targeted towards? Can be something random. Thanks in advance!

Hi @steveny91
the original mapping works for metrics like
airflow.ti.start.dag_name.task_name
Airflow users are free to use every symbol in their DAG names, unfortunately including the "."
This can lead to dag names like
some.dag
resulting in the following metrics:
airflow.ti.start.some.dag.task_name
This will not be covered by the original mappning and as a result will still be forwarded to Datadog and create a new custom metric for every dag.name - task_name combination resulting in cost.
Here are some further examples, which will be covered by the proposed mapping:
airflow.ti.start.base_dag.DE.task_name
airflow.ti.start.base_dag.EN.task_name
Or
airflow.ti.start.base_dag.1.task_name
airflow.ti.start.base_dag.2.task_name
Or even
airflow.ti.start.base_dag.DE.BER.1.task_name
airflow.ti.start.base_dag.DE.CGN.1.task_name
I hope those examples illustrate the motivation for the MR


lang: add constraint that allows seeds constraint to check PDA that were derived from a different program than the currently executing one

the seeds constraint can be used to verify that a given account is a PDA created with the given seeds. However, it assumes that the PDA was derived using the program id of the executing program. This is an unnecessary restriction and users should have the choice to verify PDAs that were derived from a different program.
Solution:
Add new constraint that allows users to specify the program seed

I dont think I can assign issues to ppl outside the org so I hereby assign this issue to @cqfd.

the parser should not allow this constraint

if init/init_if_needed is also used

closed by #1197


The effect of llvc is not as good as rvc

Hi, sorry for the inconvenience, i need desesperatly the checkpoints of a pretrained model this implementation because i want to do a finetuning. I need help on this please!

@pauortegariera You can find our checkpoints here: https://huggingface.co/KoeAI/llvc_models/tree/main/models/checkpoints

And to answer the original query: LLVC is optimized for low-latency inference on CPU. It operates under fundamentally different constraints than RVC, which is designed for medium-latency inference on GPU.

And to answer the original query: LLVC is optimized for low-latency inference on CPU. It operates under fundamentally different constraints than RVC, which is designed for medium-latency inference on GPU.

But the effect in your paper is relatively good. I am very confused.

And to answer the original query: LLVC is optimized for low-latency inference on CPU. It operates under fundamentally different constraints than RVC, which is designed for medium-latency inference on GPU.

But the effect in your paper is relatively good. I am very confused.

In the paper, we picked a relatively "easy" target voice (clear audio, level in tone and volume) and made sure to use an RVC checkpoint that produces high-quality converted output. The input that you choose for our model may also differ from the input that we used as examples for our paper. You can get a sense for which voices work well as target voices by listening to the CPU samples provided here (note that the linked audio is at 22.5kHz and our retrained model is at 16kHz). You can also test download the checkpoint and run it on your own voice samples, or (if you have Windows) download the Koe desktop app and try the CPU voices there. Choice of mic and how close you are to it have a large impact on conversion quality.

And to answer the original query: LLVC is optimized for low-latency inference on CPU. It operates under fundamentally different constraints than RVC, which is designed for medium-latency inference on GPU.

But the effect in your paper is relatively good. I am very confused.

In the paper, we picked a relatively "easy" target voice (clear audio, level in tone and volume) and made sure to use an RVC checkpoint that produces high-quality converted output. The input that you choose for our model may also differ from the input that we used as examples for our paper. You can get a sense for which voices work well as target voices by listening to the CPU samples provided here (note that the linked audio is at 22.5kHz and our retrained model is at 16kHz). You can also test download the checkpoint and run it on your own voice samples, or (if you have Windows) download the Koe desktop app and try the CPU voices there. Choice of mic and how close you are to it have a large impact on conversion quality.

Thank‘s for your reply，I will try it. I have another question, how to make the student model support multi speakers，through label embedding of waveformer?


2 Motor vehicle

I have a 2 motor vehicle (Tesla kids power car).
Can I just connect both motors to the same motor driver or would I need 2 motor drivers?
Thanks for this nice project!!

Yes, one driver is fine. Mine has actually 2 motors in parallel.
They are connected under the car with a fuse each (original harness), and only 2 wires (red/black) are reaching the electronic
Thanks!

Alright! Thanks for the reply....I'll do some ordering and keep this past updated on my progress later :-)

I successfully powered 2 motors with separate drivers wired in parallel. One driver would overload and turn off when climbing a small incline. Puting each motor on its own driver fixed the issue and is working great currently. I am running a 20v 8ah hercules battery from harbor freight.


chore: use new constructor for ScriptOrigin when >= 17.x

Refs https://chromium-review.googlesource.com/c/v8/v8/+/3395880.
Chromium did an advance deprecation removal of the ScriptOrigin ctor without an Isolate.
This allows Electron to remove a patch

Why was the check originally for Node 18? I am AFK for about a week and cannot look into it until later.
On November 1, 2023 9:26:18 PM GMT+01:00, Shelley Vohr @.***> wrote:

Refs https://chromium-review.googlesource.com/c/v8/v8/+/3395880.
Chromium did an advance deprecation removal of the ScriptOrigin ctor without an Isolate.
This allows Electron to remove a patch
You can view, comment on, or merge this pull request online at:
https://github.com/nodejs/nan/pull/960
-- Commit Summary --

chore: use new constructor for ScriptOrigin when >= 17.x

-- File Changes --
M test/cpp/news.cpp (4)
-- Patch Links --
https://github.com/nodejs/nan/pull/960.patch
https://github.com/nodejs/nan/pull/960.diff

@kkoopa that's when the new constructor was initially slated to be removed, but they moved up the date after all uses in Chromium got removed it seems.

@kkoopa I'd still love a look at this when you can!

They've now removed this ctor entirely:  https://chromium-review.googlesource.com/c/v8/v8/+/5539888
So we'll likely need to pend this until Node.js contains the version of V8 that removes it as well.


Some questions about the voxelization and convolution ops

Hi, @zhijian-liu , I am considering using torchsparse to build my own model. Here I have some questions about the voxelization and convolution operations.
I notice that the sparse_quantize can be used to remove the duplicates. What does it mean by removing the duplicates? Duplicated points in the voxel and only one point for a single voxel?
The second question is about the dimensions of the input and output of the voxelization and convolution operations. Do the output tensors of the mentioned operations have fixed size once the input is given? This question is mainly about the deployment of the model since TensorRT requires fixed dimensions of inputs and outputs.
By the way, have you considered the deployment of the models? It would be great if the feature of the deployment can be added into the project.

As for the first point, you're correct that sparse_quantize removes duplicates, where duplicates means multiple points contained in the same voxel. sparse_quantize voxelizes the point cloud based on the grid and selects one point from each voxel using np.unique.

You may be interested in this thread regarding model deployment. https://github.com/mit-han-lab/torchsparse/issues/71#issuecomment-897639382


does opencv engine support jpeg2000 format?

I see that there is a thumbor plugin for jpeg2000 (https://github.com/thumbor/thumbor-plugins/wiki/JPEG2000). However it seems like it will only work with PIL, and hence pil engine. Would it be possible to support jp2 format with opencv engine as well?

Yes! Straight from the docs.
Note OpenCV offers support for the image formats Windows bitmap (bmp), portable image formats (pbm, pgm, ppm) and Sun raster (sr, ras). With help of plugins (you need to specify to use them if you build yourself the library, nevertheless in the packages we ship present by default) you may also load image formats like JPEG (jpeg, jpg, jpe), JPEG 2000 (jp2 - codenamed in the CMake as Jasper), TIFF files (tiff, tif) and portable network graphics (png). Furthermore, OpenEXR is also a possibility.

I mean would it be possible to support jpeg2000 via format filter like localhost:8000/unsafe/filters:format(jp2)/someimage.png. Currently it returns a jpeg image.


pkg(go-1.16.2): Added go 1.16.2

Added Go to piston v3.

^^vv testing CI (this package is faster than ruby to build)

This fixes issue 116: https://github.com/engineer-man/piston/issues/116


Conda environment cannot initialized in VSCode at certain folder

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version: 1.86.1
OS Version: MacOS 14.2.1 (23C71)

Steps to Reproduce:

I cannot reproduce this since it happens without any clue
Open one folder, then the terminal will show this:

# >>>>>>>>>>>>>>>>>>>>>> ERROR REPORT <<<<<<<<<<<<<<<<<<<<<<

    Traceback (most recent call last):
      File "/opt/homebrew/Caskroom/miniforge/base/lib/python3.9/site-packages/conda/exception_handler.py", line 17, in __call__
        return func(*args, **kwargs)
      File "/opt/homebrew/Caskroom/miniforge/base/lib/python3.9/site-packages/conda/cli/main.py", line 110, in main_sourced
        print(activator.execute(), end="")
      File "/opt/homebrew/Caskroom/miniforge/base/lib/python3.9/site-packages/conda/activate.py", line 203, in execute
        return getattr(self, self.command)()
      File "/opt/homebrew/Caskroom/miniforge/base/lib/python3.9/site-packages/conda/activate.py", line 168, in activate
        builder_result = self.build_activate(self.env_name_or_prefix)
      File "/opt/homebrew/Caskroom/miniforge/base/lib/python3.9/site-packages/conda/activate.py", line 347, in build_activate
        return self._build_activate_stack(env_name_or_prefix, False)
      File "/opt/homebrew/Caskroom/miniforge/base/lib/python3.9/site-packages/conda/activate.py", line 437, in _build_activate_stack
        deactivate_scripts = self._get_deactivate_scripts(old_conda_prefix)
      File "/opt/homebrew/Caskroom/miniforge/base/lib/python3.9/site-packages/conda/activate.py", line 769, in _get_deactivate_scripts
        for entry in os.scandir(join(prefix, "etc", "conda", "deactivate.d"))
      File "/opt/homebrew/Caskroom/miniforge/base/lib/python3.9/posixpath.py", line 76, in join
        a = os.fspath(a)
    TypeError: expected str, bytes or os.PathLike object, not NoneType

`$ /opt/homebrew/Caskroom/miniforge/base/bin/conda shell.posix activate base`

  environment variables:
                 CIO_TEST=<not set>
                CONDA_EXE=/opt/homebrew/Caskroom/miniforge/base/bin/conda
           CONDA_PREFIX_2=/opt/homebrew/Caskroom/miniforge/base/envs/ML
           CONDA_PREFIX_3=/opt/homebrew/Caskroom/miniforge/base
         CONDA_PYTHON_EXE=/opt/homebrew/Caskroom/miniforge/base/bin/python
               CONDA_ROOT=/opt/homebrew/Caskroom/miniforge/base
              CONDA_SHLVL=4
           CURL_CA_BUNDLE=<not set>
                 INFOPATH=/opt/homebrew/share/info:
               LD_PRELOAD=<not set>
                  MANPATH=/opt/homebrew/share/man:/usr/share/man:/usr/local/share/man:/opt/X11/s
                          hare/man:/Library/TeX/Distributions/.DefaultTeX/Contents/Man:/opt/home
                          brew/share/man::
                     PATH=/opt/homebrew/bin:/opt/homebrew/sbin:/usr/local/bin:/System/Cryptexes/
                          App/usr/bin:/usr/bin:/bin:/usr/sbin:/sbin:/var/run/com.apple.security.
                          cryptexd/codex.system/bootstrap/usr/local/bin:/var/run/com.apple.secur
                          ity.cryptexd/codex.system/bootstrap/usr/bin:/var/run/com.apple.securit
                          y.cryptexd/codex.system/bootstrap/usr/appleinternal/bin:/opt/X11/bin:/
                          Library/Apple/usr/bin:/Library/TeX/texbin
       REQUESTS_CA_BUNDLE=<not set>
            SSL_CERT_FILE=<not set>

     active environment : None
            shell level : 4
       user config file : /Users/yanbozhang/.condarc
 populated config files : /opt/homebrew/Caskroom/miniforge/base/.condarc
          conda version : 23.11.0
    conda-build version : not installed
         python version : 3.9.18.final.0
                 solver : libmamba (default)
       virtual packages : __archspec=1=m1
                          __conda=23.11.0=0
                          __osx=14.2.1=0
                          __unix=0=0
       base environment : /opt/homebrew/Caskroom/miniforge/base  (writable)
      conda av data dir : /opt/homebrew/Caskroom/miniforge/base/etc/conda
  conda av metadata url : None
           channel URLs : https://conda.anaconda.org/conda-forge/osx-arm64
                          https://conda.anaconda.org/conda-forge/noarch
          package cache : /opt/homebrew/Caskroom/miniforge/base/pkgs
                          /Users/yanbozhang/.conda/pkgs
       envs directories : /opt/homebrew/Caskroom/miniforge/base/envs
                          /Users/yanbozhang/.conda/envs
               platform : osx-arm64
             user-agent : conda/23.11.0 requests/2.31.0 CPython/3.9.18 Darwin/23.2.0 OSX/14.2.1 solver/libmamba conda-libmamba-solver/23.12.0 libmambapy/1.5.6
                UID:GID : 501:20
             netrc file : /Users/yanbozhang/.netrc
           offline mode : False

An unexpected error has occurred. Conda has prepared the above report.
If you suspect this error is being caused by a malfunctioning plugin,
consider using the --no-plugins option to turn off plugins.

Example: conda --no-plugins install <package>

Alternatively, you can set the CONDA_NO_PLUGINS environment variable on
the command line to run the command without plugins enabled.

Example: CONDA_NO_PLUGINS=true conda install <package>

^C
Timeout reached. No report sent.

Notes

This bug only happens at certain folder;
If I make a copy of that folder, the bug disappeared;
Only happen in VSCode terminal;
Only happened in ML env, which is my machine learning environment;
Open a python file, then click bottom right to change the environment, sometimes, this will resolve this issue;
I tried to delete the cache folders, not helping;

Me too! Have u solved this problem?

Me too! Have u solved this problem?

@chenyifanthu  See this issue in conda repo: https://github.com/conda/conda/issues/13585#issuecomment-1987090935
Not an elegant solution, but it works.

As I stated here conda/conda#13585 (comment), I think it's a VSCode problem after investigating the issue and trying some fixes.


Variable quantum state assignment

Hey again 👋🏻
Just wanted to ask, how Silq manages to handle Variables. As far as I know, quantum states cannot be stored inside a variable since the measurement will trigger a mirror-state.
Best regards
Timo Sarkar
Zug, Switzerland

Hi Timo,
An assignment
x=f(x,y)

is treated like:
x'=x;
x:=f(x',y);
forget(x');

forget(x') is only valid if the type checker can show that it is possible to synthesize an expression g(x,y) such that forget(x'=g(x,y)) would work. To run the code on a quantum computer, this code synthesis has to be performed during compilation. (This is relatively straightforward for the cases that Silq currently allows.)
Best,
Timon

Allright thanks !!


Fix bug with indentation in multiplex.js

Some modes may return CodeMirror.Pass from their indent methods, which in the current code can get fed into the start state of another mode and mess things up. Fixed to check for this case.

Thanks. Modes and addons are still written in ES5 and not compiled. Could you change your lets to vars?

Done

Thanks! Merged.


Syntax sugar for most common routing/selecting logic

We have some ideas for how to make nevalang programs more readable with #719 #721 and #722 but that's not everything that programming language usually helps with through interface of so called "operators". These are left:
Routers:

If
Cond
Switch

Selectors:

Ternary
Match
Select

We don't talk here about for look (we have ForEach HoC but #711 and it's not that powerful on context of scope capturing (we don't have closures))
Existing syntax (including deferred and chained connections as well as fan-in and fan-out) makes dataflow clear. Don't we damage dataflow readability by adding control-flow-ish (visually of course) statements? Is it still clear that instructions are not executed "from top to bottom" or the dataflow itself (where data comes from, how are connections formed)?

Close in favor of #802


Update README before changing repo name to GATK

Added links to current GATK website + repos to stave off possible confusion as mentioned in #945

Back to @vdauwera for some patches

Can't seem to do git clone https://github.com/broadinstitute/hellbender/
This is my console output:
wmd16-c9e:codespace vdauwera$ git clone http://github.com/broadinstitute/hellbender/
Cloning into 'hellbender'...
remote: Counting objects: 22221, done.
remote: Compressing objects: 100% (142/142), done.
remote: Total 22221 (delta 47), reused 4 (delta 4), pack-reused 22046
Receiving objects: 100% (22221/22221), 36.63 MiB | 3.58 MiB/s, done.
Resolving deltas: 100% (9903/9903), done.
Checking connectivity... done.
Downloading src/test/resources/large/CEUTrio.HiSeq.WGS.b37.NA12878.20.21.bam (76.16 MB)
Username for 'http://github.com': vdauwera
Password for<EMAIL_ADDRESS>
Error accessing media: src/test/resources/large/CEUTrio.HiSeq.WGS.b37.NA12878.20.21.bam (6b1304800e60c0ac0358df137bdad48b7857a36465b04fef3fbbb09380f04746)

Errors logged to /Users/vdauwera/codespace/hellbender/.git/lfs/objects/logs/20151005T220016.510795175.log.
Use `git lfs logs last` to view the log.
Downloading src/test/resources/large/CEUTrio.HiSeq.WGS.b37.NA12878.20.21.bam.bai (11.25 KB)
Username for 'http://github.com':

Looks like I'm failing to download large test files. Do I need to be on VPN for this to work? Or is it expected and I should ignore it?

Well, looks like I got the code at least, if not the test data. Commits squashed and ready for re-review, @droazen ;)


Why can't JET infer that m is not nothing

julia> VERSION
v"1.10.0-DEV.395"

julia> using JET, UUIDs

julia> function f()
           if (m = match(r"", ""); m !== nothing)
                   return UUID(m[1])
           end
       end
f (generic function with 1 method)

julia> JET.@report_call f()
═════ 1 possible error found ═════
┌ @ REPL[3]:3 UUID(m[1])
│┌ @ uuid.jl:9 convert(fieldtype(Base.UUID, 1), value)
││ no matching method found `convert(::Type{UInt128}, ::Nothing)`: convert(fieldtype(Base.UUID, 1)::Type{UInt128}, value::Nothing)
│└─────────────

m is inferred as m::Base.RegexMatch. m[1] is inferred as Union{Nothing,SubString{...}}.
Use the annotate_types=true option to make sure you understand intermediate types of code.


Performance issues at 300 MBit/s

Hi,
I have two lines:
docsis -> 200 / 25 Mbit/s
vdsl -> 100 / 40 Mbit/s
The aggregation site has a Gigabit NIC and can reach full speeds to (and from) both lines.
My server configuration looks like this:
[general]
statuscommand = "/etc/mlvpn/mlvpn0_updown.sh"
tuntap = "tun"
mode = "server"
interface_name = "mlvpn0"
timeout = 30
password = ""
reorder_buffer_size = 128
loss_tolerence = 50
cleartext_data = 1

[filters]

[docsis]
bindhost = "<IP_ADDRESS>"
bindport = 5080
bandwidth_upload = 26214400

[vdsl]
bindhost = "<IP_ADDRESS>"
bindport = 5081
bandwidth_upload = 13107200

and the client configuration is this:
[general]
statuscommand = "/etc/mlvpn/mlvpn0_updown.sh"
tuntap = "tun"
mode = "client"
interface_name = "mlvpn0"
timeout = 30
password = ""
reorder_buffer_size = 128
loss_tolerence = 50
cleartext_data = 1

[filters]

[vdsl]
bindhost = "<IP_ADDRESS>"
remotehost = "<IP_ADDRESS>"
remoteport = 5081
bandwidth_upload = 5242880

[docsis]
bindhost = "<IP_ADDRESS>"
remotehost = "<IP_ADDRESS>"
remoteport = 5080
bandwidth_upload = 3276800

Now, the problem is, that when downloading:
VDSL 100 gets 100 MBit/s traffic,
DOCSIS 200 gets only 100 MBit/s traffic, too.
In total 200 Mbit/s only, not 300.
Same story with the upload speed, ~40 MBit/s total max.
I can't surpass the maximum speed of a single link.
I have tried to change many of the parameters.
Setting bandwidth_upload on the server and on the client side has had no effect (that I could observe).
Maybe setting some more extreme values here could lead to a change?
I also manipulated the reorder_buffer_size parameter, I tried using 0, 64, 128 and 256.
Both sides have powerful CPUs and are not limiting in either CPU time or interrupts.
What could I try to get this to work properly?
Thanks,
Tobias

I effectively have the same situation over a pair of adsl lines (seeing significantly less bandwidth than I expect). I'm currently wondering if this has anything to do with DUP'd ACK's (see https://github.com/zehome/MLVPN/issues/100). But - any other ideas welcome
Cheers
Mark.

Hello Mark,
interestingly, there is talk about using different network cards in the other issue.
My setup only uses 1 GbE Ethernet card and multiple VLANs to get to the uplinks.
That could probably be an issue. Also maybe offloading or a similar "smart" feature of the NIC could be an issue. I will try to make 2 seperate (untagged) connections to each VLAN after work today.
So long,
Tobias

Yeah - the more we can rule out, the better :-)

Wich mlvpn version do you use?

Master + a couple of minor mods of my own

I should have tagged a name in my question, it was for @Manawyrm initially :)

I'm using mlvpn version 2.1.2.
And yes, I seem to have dup ACKs, Out-of-Order, Segment not captured and all of that, too.
https://screenshot.tbspace.de/hmzaevqjiwk.png


LITE-24299: updated the multipart calls to do not use external library

Awesome! Thank you for your improvements, great job!


Readme update suggestion: drop @json call

The @json blade directive isn't intended to be used this way. It works fine (sort of) with 3 arguments, but as soon as you add more, it breaks.
https://github.com/laravel/framework/issues/31149

Thanks for the tip @chris-ware 😄  I've updated the documentation.


make codefixes and refactorings aware of coding style

codefixes and refactorings emit the source code in TypeScript (just like developers), coding standards and styles are common across dev teams, currently the codefix emitter can't know about these styles and standards, so inevitably it emits some bad looking code
not sure what you can do about it

I understand this was the most interesting special case of this problem: https://github.com/Microsoft/TypeScript/pull/17750

And code fixes and refactorings do, to some extent, have configurable formatting: https://github.com/Microsoft/TypeScript/blob/a3d113bed3ac127084c8795da43161413c072510/src/services/types.ts#L522.  Obviously, there are many more knobs that could be added.

Refactorings are supposed to read the formatting settings from your editor and then format the emitted code accordingly. If you run into a case where this doesn't happen (basically, if running Format Document after a refactor causes new edits in the refactored code to appear) please log a specific bug with repro steps.


FIX | Adding support for sharedInstances to managed SNI

fixes #1231
Manged SNI did not have sharedInstances support for lcoaldb.

in SNI Proxy for localdb the assumption is that when we split the connection string by \\ it always returns an array with length of 2 however when the connection string is set for a shared instance we get an array with a length of 3 since it is in a format of (localdb)\.\shared Instance name

Based on MS Docs all shared instances connection strings should be in a format of
(localdb)\.\ "shared instance's name"
a shared instance could be created as below:
Make sure you have installed SqlLocalDB

Open Developer Command Prompt
Type: SqlLocalDB create "" (you can choose any name)
Type: SqlLocalDB share " your instance name " "provide a name for shared instance"
SqlLocalDB start "name your instance"
Continue either on SSMS or CLI with SqlCmd: for SSMS use (localdb)\your instancename
Create LOGIN [Domain\User]; (run this if the login is not created by default for the user otherwise it fails)
GRANT LOGIN TO [Domain\User]

in the same format one can unshare and delete a shared instance.
One can log into shared instances using ssms with the format mentioned above. Also, Visual studio provides the connection string when you go to View > SQL Server Object Explorer and select local. (It only shows localdb and shared isntances if you have already created one and it is running).

Regex is much faster in 5 but this is a really trivial case and what you really want is just to split the string and then count the number of resulting strings. Regex isn't needed or much easier here.

Ok. seems fair. It was an over kill. I've changed it.

Is (localdb)\LocalDbAppName a valid name? It looks like it should be from your test input but if I replicate the function GetLocalDBInstance it returns false, perhaps you wanted to negate that if (string.IsNullOrEmpty(..)) check?

Is case important in the output instanceName? because if it isn't I think I've got a version of the function which only allocates the output string. If i have to lower case it then I'll need to allocate a stringbuilder.

Have a look, see what you think: https://gist.github.com/Wraith2/b2dbcf6af7d21c48620dc00eb0879364

Have a look, see what you think: https://gist.github.com/Wraith2/b2dbcf6af7d21c48620dc00eb0879364

I like the idea, but System.Range is not supported in NetCoreApp 2.1. Although the support is dropped for that version, but it will take some time for us to get there.
I was thinking about this, what is preventing us from getting the very last index of the array?
We can get instanceName = input[^1] or instanceName = tokensByBackSlash[tokensByBackSlash.Length-1]

You can simply replace the range expressions with Slice, they're all start offset only so the single arg version works.
We can get the last index if the split works but then what happens if a user passes us a string with 4 slashes in? Invalid yes but will it behave the same? More importantly I was trying to avoid the allocation that the split causes, it's an array and 3 strings all of them on the heap and we know that 2 of the strings are going to get thrown away instantly.
The split version is fine if you want to stick with that. It's not a hot path so it's not going to cause anyone a problem. I just like to see if there are efficient ways to implement things like this, as I did with the azure endpoint parsing.


ngrok testing?

Hi, this rewardenv is great, but is there any way to test the local environment using ngrok? when I run the ngrok command  ./ngrok http https://mywebsite.test and then I try to open the url provided by ngrok, then I am always getting '404 page not found'. Thanks so much

Hi @joseborngroup !
Well I never used this stuff before.
Could you please be a bit more specific about it?
What's the purpose of ngrok?
What environment type did you use?
How should I implement it?
Also any contributions are more than welcome here so feel free to create a pull request.

Hi, thanks for replying, ngrok is basically to share your localhost website with someone else through a public url that ngrok provides https://ngrok.com/docs , you can basically run a command ngrok http 80 and ngrok will give you a public url ie. http://92832de0.ngrok.io  forwarding to your locahost. I am using a Magento 2 Reward Environment. Also I have noticed, it is not possible to access the reward environment by just calling the host and the port on browser, I am trying to call https://<IP_ADDRESS>:80 but this doesn't display the https://mywebsite.test website. So I also wonder what is the host and the port that rewardenv uses?

Well that's by a good purpose.
The main functionality of Reward is to be able to run multiple environments simultaneously.
In front of your Magento environment Reward runs a Traefik container which proxies the requests (based on the Host header) to the different Magento environments.
Anyway. I was able to run ngrok like this:
# get current environment name
ENV=$(cat .env | grep 'REWARD_ENV_NAME' | cut -d '=' -f2)

# get current docker network name
NETWORK=$(docker network ls --filter=label=dev.reward.environment.name=${ENV} --format "{{.Name}}")

# get current nginx container
NGINX=$(docker ps --filter name=${ENV}_nginx --format '{{.Names}}')

# run ngrok container in the same network and link it to the nginx container
docker run --rm -it --link $NGINX --net $NETWORK wernight/ngrok ngrok http $NGINX:80

I had to change the magento base url to reflect to the ngrok url.

For me this cmd also working
./ngrok http -host-header=rewrite your-url.test:443
OR
./ngrok http -host-header=rewrite your-url.test:80
Just need to download ngrok (https://ngrok.com/) and need to add ngrok authtoken for port 443
So now your url is accessible from outside just without css/images...
For magento 2 it is a temp solution but you can try this:
Open:
vendor/magento/framework/App/Http.php
Add these 3 lines before "return $this->_response;"
$body = $this->_response->getBody();
$body = str_replace('https://your-url.test/', 'https://your-ngrok-url.ngrok.io/', $body);
$this->_response->setBody($body);


wiretrustee: init at 0.3.3

Motivation for this change
Add wiretrustee to nixpkgs.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Closing as it is implemented in #186669 after a project name change, which I totally missed.


Add info about which signing keys will be used for published artifacts.

Add info about which signing keys will be used for published artifacts.
For security purposes, it would be great if you were able to publish details (in the project docs) about gpg public keys that are "valid" for use when verifying signing artifacts uploaded to maven central.
This allows for "out of band" verification of the expected signing key.
Some examples of other libs publishing their signing keys:
https://square.github.io/okhttp/security/security/#verifying-artifacts
https://github.com/eclipse/jetty.project/blob/jetty-10.0.x/KEYS.txt
https://downloads.apache.org/commons/KEYS
https://downloads.apache.org/logging/KEYS

The signing key ID is:
E1188CE08349E884A5C4B8458A583BB0ED7ADE95
and it can be found e.g. on the https://keyserver.ubuntu.com/ keyserver or I suppose also from https://keys.openpgp.org/
I shall put this information somewhere.

This information is now in: http://www.apfloat.org/apfloat_java/maven.html


[1.10, 9.2.1] Logistical Sorter default route non-functional

Placing the sorter on a chest with misc items coming in, a cobble filter set as the only filter, there's absolutely no way to get the default route to work. Even when using the filter in-line and/or having the default as the same colour as the filter, it remains jammed and/or will only ever allow the filtered item through. Round-robin, auto-eject and filter settings all make no difference.

Should be fixed in latest release


nbdev breaks subcaptions on figures

See the example here, Quarto can do something like the following:
#| label: fig-charts
#| fig-cap: Charts
#| fig-subcap:
#|   - "First"
#|   - "Second"
#| layout-ncol: 2

import matplotlib.pyplot as plt
plt.plot([1,23,2,4])
plt.show()

plt.plot([8,65,23,90])
plt.show()

And render carts with captions:

Currently this will break nbdev with the following trace:
Traceback (most recent call last):
  File "/home/zach/.local/lib/python3.8/site-packages/nbdev/quarto.py", line 278, in _f
    try: serve_drv.main(res)
  File "/home/zach/.local/lib/python3.8/site-packages/nbdev/serve_drv.py", line 22, in main
    if src.suffix=='.ipynb': exec_nb(src, dst, x)
  File "/home/zach/.local/lib/python3.8/site-packages/nbdev/serve_drv.py", line 16, in exec_nb
    cb()(nb)
  File "/home/zach/.local/lib/python3.8/site-packages/nbdev/processors.py", line 243, in __call__
    def __call__(self, nb): return self.nb_proc(nb).process()
  File "/home/zach/.local/lib/python3.8/site-packages/nbdev/process.py", line 126, in process
    for proc in self.procs: self._proc(proc)
  File "/home/zach/.local/lib/python3.8/site-packages/nbdev/process.py", line 119, in _proc
    for cell in self.nb.cells: self._process_cell(proc, cell)
  File "/home/zach/.local/lib/python3.8/site-packages/nbdev/process.py", line 104, in _process_cell
    if f in cell.directives_: self._process_comment(proc, cell, f)
  File "/home/zach/.local/lib/python3.8/site-packages/nbdev/process.py", line 115, in _process_comment
    return proc(cell, *args)
TypeError: __call__() takes 2 positional arguments but 3 were given

cc @seeM

Oof, looks like Quarto directives support a more extended version of YAML these days. Our parser is super simple and doesn't understand "multi-line" directives like this. An immediate workaround would be to use single line syntax, for example I tested this locally and it works:
#| label: fig-charts
#| fig-cap: Charts
#| fig-subcap: ["First", "Second"]
#| layout-ncol: 2

import matplotlib.pyplot as plt
plt.plot([1,23,2,4])
plt.show()

plt.plot([8,65,23,90])
plt.show()

Note: #| fig-subcap: ["First", "Second"] isn't actually parsed correctly by nbdev, but it doesn't need to be. It just needs to not lead to an error so it can pass-through to Quarto.
Perhaps a decent short-term fix would be for nbdev to not raise on directive "syntax errors", but instead print a warning and move on, since nbdev doesn't need to understand Quarto's full syntax.

@jph00 @hamelsmu, curious to hear your thoughts too.

It might be annoying for Quarot-compliant users to have to see a warning every time they preview their blog.
Do you think there might be a way to parse the directives as YAML?  We already have pyyaml as a dependency.  Since #|export isn't valid yaml, we would have to pre-process that kind of thing first to turn things into key: value  (with a space after the colon IIRC YAML is strict about the space).  for example we might have to turn #|export into #|export: true behind the scenes.
The upshot of that approach is that it would give users a shortcut to all quarto directives as well; for example people could just write #| echo and it will get turned into #| echo: true by nbdev and passed along to quarto. This way, the behavior is consistent, and we can declare that we have enabled a "shortcut" for all cell directives even quarto ones.
WDYT about this?

Message ID: @.***>Yeah I was wondering about that too. I think #|export actually is valid YAML though - it's a YAML string. The issue AFAICT is that Quarto expects everything to be a yaml dictionary.

@jph00 just out of curiosity, when I put the following into any YAML validator, like this one it errors:
foo
bar: baz

Similarly, in python this errors:
In [5]: import yaml

In [6]: yml = "foo\nbar: baz"

In [7]: yaml.safe_load(yml)

ScannerError: mapping values are not allowed here
  in "<unicode string>", line 2, column 4:
    bar: baz
       ^

But this does not
In [8]: yml = "foo: hello\nbar: baz"

In [9]: yaml.safe_load(yml)
Out[9]: {'foo': 'hello', 'bar': 'baz'}

I think the gap is the multi-line yaml stuff.  so
#|export
#|echo: true

Would not be valid yaml if taken together, but line-by-line separately it would be.  🤔

I think the gap is the multi-line yaml stuff. so
#|export #|echo: true
Would not be valid yaml if taken together, but line-by-line separately it would be. 🤔

Yes exactly!


Failed Atom install

When I try to install I get this error :
Installing<EMAIL_ADDRESS>failed.Show output…

@Fufl0  facing same error message as  #815 ?

You need Git installed on your system to install script through APM. (Atom Package Manager.) I stumbled upon this myself on a fresh installation of my OS.

This is new problem:
error message is below:
npm ERR! Command failed: Cloning into bare repository 'C:\Users\karottc.atom
apm_git-remotes\git-github-com-monsterkodi-node-simple-plist-git-6a71a541'...
npm ERR! fatal: unable to connect to github.com:
npm ERR! github.com[0: <IP_ADDRESS>]: errno=No error
npm ERR!
npm ERR!
npm ERR!
npm ERR! If you need help, you may report this error at:
npm ERR! https://github.com/npm/npm/issues
I hava already installed git and set PATH variable.

I've got the same problem as @karottc, although I'm on a Mac.  git is installed and in my path.  Would appreciate any suggestions.  Other git interactions with github outside of the atom package manager work fine.
npm ERR! Command failed: Cloning into bare repository '/Users/eadasiak/.atom/.apm/_git-remotes/git-github-com-monsterkodi-node-simple-plist-git-6a71a541'...
npm ERR! fatal: unable to connect to github.com:
npm ERR! github.com[0: <IP_ADDRESS>]: errno=Operation timed out
npm ERR!
npm ERR!
npm ERR!
npm ERR! If you need help, you may report this error at:
npm ERR!     <https://github.com/npm/npm/issues>

npm ERR! Please include the following file with any support request:
npm ERR!     /private/var/folders/v_/73_dbw0s42ddhkn8ylk52f5h0000gn/T/apm-install-dir-11631-75591-wxteno/npm-debug.log

@karottc and @eadasiak - According to what I could find on Google. That error could possibly be firewall issues. Apparently the git protocol uses port 443 for the main repository and port 9418 for submodules.
You could try to use https instead or open up the ports. It's up to you. There are two ways to force git to use a specific protocol:
1.) Change your ~.gitconfig file manually. Open up your ~.gitconfig file and paste in the following. The snippet should be self explanatory.
[url "https://"]
    insteadOf = git://

2.) Or, you could simply type the following in a Terminal window. (Git Bash for Windows users.)
git config --global url."https://".insteadOf git://

I hope this helps you both.

've got the same problem as @karottc and @eadasiak, on Windows.
But trying install on Linux, successfully installed.
$ git --version
git version 2.8.1

$ git config --global -l
http.proxy=<my proxy addr>
core.autocrlf=false
core.safecrlf=true
user.name=Koutarou Tanaka
<EMAIL_ADDRESS>url.https://.insteadof=git://

$ apm install script
Installing script to C:\Users\fkfk\.atom\packages failed

npm WARN addRemoteGit Error: Command failed:
npm WARN addRemoteGit     at ChildProcess.exithandler (child_process.js:658:15)
npm WARN addRemoteGit     at ChildProcess.emit (events.js:98:17)
npm WARN addRemoteGit     at maybeClose (child_process.js:766:16)
npm WARN addRemoteGit     at Process.ChildProcess._handle.onexit (child_process.js:833:5)
npm WARN addRemoteGit  git://github.com/monsterkodi/node-simple-plist.git resetting remote C:\Users\fkfk\.atom\.apm\_git-remotes\git-github-com-monsterkodi-node-simple-plist-git-6a71a541 because of error: { [Error: Command failed: ] killed: false, code: 1, signal: null }
npm ERR! git clone --template=C:\Users\fkfk\.atom\.apm\_git-remotes\_templates --mirror git://github.com/monsterkodi/node-simple-plist.git C:\Users\fkfk\.atom\.apm\_git-remotes\git-github-com-monsterkodi-node-simple-plist-git-6a71a541: Cloning into bare repository 'C:\Users\fkfk\.atom\.apm\_git-remotes\git-github-com-monsterkodi-node-simple-plist-git-6a71a541'...
npm ERR! git clone --template=C:\Users\fkfk\.atom\.apm\_git-remotes\_templates --mirror git://github.com/monsterkodi/node-simple-plist.git C:\Users\fkfk\.atom\.apm\_git-remotes\git-github-com-monsterkodi-node-simple-plist-git-6a71a541: fatal: Unable to look up github.com (port 9418) (Temporary failure in name resolution)
npm ERR! Windows_NT 6.1.7601
npm ERR! argv "D:\\opt\\Atom\\resources\\app\\apm\\bin\\node.exe" "D:\\opt\\Atom\\resources\\app\\apm\\node_modules\\npm\\bin\\npm-cli.js" "--globalconfig" "C:\\Users\\fkfk\\.atom\\.apm\\.apmrc" "--userconfig" "C:\\Users\\fkfk\\.atom\\.apmrc" "install" "D:\\opt\\msys32\\tmp\\d-116311-2396-15dz5pn\\package.tgz" "--target=0.34.5" "--arch=ia32"
npm ERR! node v0.10.40
npm ERR! npm  v2.13.3
npm ERR! code 128

npm ERR! Command failed: Cloning into bare repository 'C:\Users\fkfk\.atom\.apm\_git-remotes\git-github-com-monsterkodi-node-simple-plist-git-6a71a541'...
npm ERR! fatal: Unable to look up github.com (port 9418) (Temporary failure in name resolution)
npm ERR!
npm ERR!
npm ERR! If you need help, you may report this error at:
npm ERR!     <https://github.com/npm/npm/issues>

@dhhdev - thanks for the advice!  There was indeed a firewall blocking outbound traffic on non-standard ports that was getting in the way.  I've since been able to get around the problem and re-install the package.  Thanks again!

https://github.com/rgbkrk/atom-script/issues/881

@fkfk It sounds like you're having path issues. Make sure that Git is properly installed. The installer for Windows from https://git-scm.com/ has the option to add this correctly to your Windows PATH variable. The error you get, is most likely because port 9418 is not open. (Possibly in your router.) This is not an error as such, since most routers doesn't just have ports open out of the box left, right and center.
I would try and make sure the PATH variable is properly set and revert back to use git:// instead of https:// as you have set in Git config. I hope that helps.
@eadasiak I'm glad I could help you! 👍


Update Swift grammar from https://github.com/textmate/swift.tmbundle/blob/master/Syntaxes/Swift.tmLanguage

The grammar at https://github.com/textmate/swift.tmbundle/blob/master/Syntaxes/Swift.tmLanguage has been recently updated, please pull the changes back into VS Code. Thanks!

@alexr00 are you still the right person for this?

Done, thanks for the ping! 57f776891738fbc70388baf541afcbba1c1bdc8c