Training LLaMa?

In theory the LLaMa 30b & 65b should be much more capable than the GPT-NeoX 20b.
Does OpenChatKit support LLaMa? If not, is it on the roadmap?
I appreciate that togethercomputer might not be able to release pretrained LLaMa weights due to the licence, but it'd be great if researches can at least play with it.

same question

same request here

OpenChatKit is a properly free, open-source model under a permissive licence.
Meta's LLaMa is none of those things, and anything built on top of it won't be either.
I don't see what kind of interaction between those 2 models would be helpful.


Add a method to load an audio file

The library should be able to load an audio file to be played.
I see the following methods:

void load(String) - load the audio file from a string representing the URI to the file
void load(URI) - load the audio file from an URI pointing to the file
void load(AudioFile) - load the audio file directly from an audio file

The two firsts methods would call the later one to avoid redundancy.
These methods should throw an exception if the audio file can't be loaded.

I was wondering if the load and unload methods were needed if we changed the play method to accept string, URI or AudioFile parameters. What would be the benefits to have these two methods?

IMO, they are needed because we don't want for sure to play a file directly after it has been loaded.
Having a load method allows loading a file and let the choice of playing it to the user.
Concerning the unload method, I still have my doubts about its purpose. It will be clearer in the future.

Do we really need to offer multiple parameters? I think one specifying the file's URI is enough. Documentation about the Uri class is available here.
Here are my thoughts:

The String parameter is useless since it is a URI itself.
The AudioFile parameter is also useless as one of its members is the URI of the audio file. So when calling the load method, the developer simply has to pass audiofile.uri as the parameter.

What do you think about offering only a load(URI) method?

As the commit above states, I started to work on this method.

The method signature is now : Future load(String uri).
It now throws a StereoFileNotPlayableException exception if the file pointed by the uri parameter is not playable.

The platform-specific code is implemented only for iOS at the moment. I will begin working on the Android part as soon as I can.


Coverage percentages don't match in the UI and in the coverage.py output

Type: Bug

Behaviour
Coverage percentages don't match in the UI and in the coverage.py output. For both "missing lines" and "branch" coverage. See attached screenshot.
Steps to reproduce:
???

Diagnostic data

Output for Python in the Output panel (View→Output, change the drop-down the upper-right of the Output panel to Python)

---

Extension version: 2024.17.2024100401
VS Code version: Code 1.94.0 (Universal) (d78a74bcdfad14d5d3b1b782f87255d802b57511, 2024-10-02T13:08:12.626Z)
OS version: Darwin arm64 22.1.0
Modes:

Python version (& distribution if applicable, e.g. Anaconda): 3.12.5
Type of virtual environment used (e.g. conda, venv, virtualenv, etc.): Venv
Value of the python.languageServer setting: Default

User Settings

languageServer: "Pylance"

testing
• pytestArgs: "<placeholder>"
• pytestEnabled: true

Installed Extensions

Extension Name
Extension Id
Version

Even Better TOML
tamasfe.even-better-toml
0.19.2

JavaScript Debugger
ms-vscode.js-debug
1.94.0

JavaScript Debugger Companion Extension
ms-vscode.js-debug-companion
1.1.3

Jupyter
ms-toolsai.jupyter
2024.9.1

Jupyter Cell Tags
ms-toolsai.vscode-jupyter-cell-tags
0.1.9

Jupyter Keymap
ms-toolsai.jupyter-keymap
1.1.2

Jupyter Notebook Renderers
ms-toolsai.jupyter-renderers
1.0.19

Jupyter Slide Show
ms-toolsai.vscode-jupyter-slideshow
0.1.6

Live Preview
ms-vscode.live-server
0.4.15

Markdown All in One
yzhang.markdown-all-in-one
3.6.2

Markdown Emoji
bierner.markdown-emoji
0.3.0

Markdown Preview Mermaid Support
bierner.markdown-mermaid
1.25.0

Pylance
ms-python.vscode-pylance
2024.8.2

Python
ms-python.python
2024.17.2024100401

Python Debugger
ms-python.debugpy
2024.10.0

Ruff
charliermarsh.ruff
2024.50.0

Sort lines
Tyriar.sort-lines
1.11.0

SQLTools
mtxr.sqltools
0.28.3

SQLTools PostgreSQL/Cockroach Driver
mtxr.sqltools-driver-pg
0.5.4

Table Visualizer for JavaScript Profiles
ms-vscode.vscode-js-profile-table
1.0.10

System Info

Item
Value

CPUs
Apple M1 Pro (10 x 2400)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: enabled_ondirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_graphite: disabled_offvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabledwebgpu: enabledwebnn: disabled_off

Load (avg)
2, 2, 2

Memory (System)
16.00GB (0.49GB free)

Process Argv
--crash-reporter-id b18eb1dd-5763-4eb9-9747-ba339864022c

Screen Reader
no

VM
0%

A/B Experiments
vsliv368:30146709
vspor879:30202332
vspor708:30202333
vspor363:30204092
vscod805cf:30301675
binariesv615:30325510
vsaa593:30376534
py29gd2263:31024239
c4g48928:30535728
azure-dev_surveyone:30548225
vscrpc:30673769
2i9eh265:30646982
962ge761:30959799
pythongtdpath:30769146
welcomedialogc:30910334
pythonnoceb:30805159
asynctok:30898717
pythonmypyd1:30879173
2e7ec940:31000449
pythontbext0:30879054
accentitlementst:30995554
dsvsc016:30899300
dsvsc017:30899301
dsvsc018:30899302
cppperfnew:31000557
dsvsc020:30976470
pythonait:31006305
dsvsc021:30996838
0ee40948:31013168
a69g1124:31058053
dvdeprecation:31068756
dwnewjupytercf:31046870
newcmakeconfigv2:31071590
impr_priority:31102340
nativerepl2:31139839
refactort:31108082
pythonrstrctxt:31112756
flighttreat:31134774
wkspc-onlycs-t:31132770
nativeloc2:31134642
wkspc-ranged-t:31151552
cf971741:31144450
defaultse:31146405
iacca2:31150323
cc771715:31146322

A few examples:

Hello,

Ah, ok, thanks, these definitely align more.
account.py is fully covered, therefore is excluded from coverage report. This file is good. This was accidentally highlighted, sorry for confusion
Seems like aligning well keeping 1. in mind.
It also seems like coverage.py does some percentage rounding while Python extension is more precise.
I think I can take another look once branch coverage is fully supported, otherwise this seems like a guesswork.
I can close this issue for now if you want.

Yes- the rounding does exist and I noticed this as well but decided surfacing more precision wasn't bad since it is ultimately a fraction we are evaluating that comes from line number counts. And yes we are looking at branch coverage but currently branch coverage numbers are not exposed by the coverage.py API and therefore I am working with the maintainer there to surface it so we can safely support branch coverage on our end too. Closing as I think this resolves all the issues you surfaced, thanks!


Update meta.yaml

Checklist

[x] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[x] 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.

Updated to 2022.6.30, fixed issues updating the datastore

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


Adding Task Slot Count Metrics to Druid Overlord

Motivation
The primary motivation of this work is to provide more visibility into the task slot utilization over time. Monitoring utilization can help cluster administrators determine when to add/remove middlemanager/indexer nodes. With native ingestion, this has become even more important.
Proposed changes
I propose to add a new TaskSlotCountStatsMonitor in Overlord similar to TaskCountStatsMonitor class. It will expose the following metrics:

taskSlot/total/count: Total number of task slots per emission period
taskSlot/idle/count: Total number of task slots available for adding tasks
taskSlot/used/count: Total number of task slots being currently used
taskSlot/lazy/count: Total number of task slots that have been marked lazy
taskSlot/blacklisted/count: Total task slots of peons that have been blacklisted

Proposed Design
I am planning to add the following apis in TaskRunner
  /**
   * APIs useful for emitting statistics for @TaskSlotCountStatsMonitor
  */
  long getTotalTaskSlotCount();

  long getIdleTaskSlotCount();

  long getUsedTaskSlotCount();

  long getLazyTaskSlotCount();

  long getBlacklistedTaskSlotCount();

The implementation for TaskSlotCountStatsMonitor will be similar to TaskCountStatsMonitor. It will use the TaskSlotCountStatsProvider interface which will be implemented by TaskMaster. TaskMaster will use taskRunner to emit the required metrics
Operational impact
We are adding some new metrics for better cluster monitoring.
Test plan
Other than unit tests, we are also planning to test this in our local Druid clusters. We will be using the statsd-based emitter framework to collect all the new emitted metrics for visualization.

This is a fantastic idea @mghosh4 !

Hi @mghosh4, thanks for the nice proposal!

taskSlot/total/count: Total number of task slots per emission period
taskSlot/idle/count: Total number of task slots available for adding tasks
taskSlot/used/count: Total number of task slots being currently used

These 3 metrics will be definitely useful! To be honest, I can't believe we don't have these yet.

taskSlot/lazy/count: Total number of task slots that have been marked lazy
taskSlot/blacklisted/count: Total task slots of peons that have been blacklisted

I think there 2 metrics will fit better to middleManagers/indexers rather than task slots, since middleManager/indexers can be marked as lazy or blacklisted, not individual task slot. How about worker/lazy/count and worker/blacklisted/count (or indexingWorker/lazy/count and indexingWorker/blacklisted/count)?

taskSlot/lazy/count: Total number of task slots that have been marked lazy
taskSlot/blacklisted/count: Total task slots of peons that have been blacklisted

I think there 2 metrics will fit better to middleManagers/indexers rather than task slots, since middleManager/indexers can be marked as lazy or blacklisted, not individual task slot. How about worker/lazy/count and worker/blacklisted/count (or indexingWorker/lazy/count and indexingWorker/blacklisted/count)?

On a second thought, these 2 metrics seem useful as well since it is easier to compare with other metrics such as total or idle. But I would suggest fixing the descriptions since task slots are not marked as lazy or blacklisted, but middleManagers and indexers are. Maybe they can be "total number of task slots of lazy/blacklisted middleManagers and indexers".


add non-interactive switch, closes #19

add non-interactive switch, closes #19
bump version to 0.2.3
update date and change to ISO

Thanks for the PR and fixing the date.
The switch is a bit ugly since it only works when it's the first and exactly -a. Could you please try to build an option selection that also shows the help info regarding the available switches when using -h|--help?
You may check the script I wrote regarding PMG mgmt logic for reference:
https://github.com/DerDanilo/proxmox-stuff/blob/337cf36d5baae247ba18dbbea4558b21a7843f39/Ansible/pmg_domain_mgmt/templates/pmg_domain_mgmt.sh.j2#L150
Thanks!

Yes, this is a quick and dirty fix. I don't plan to put any more time in it, so I'll go ahead and close this PR.


Can't have CtrpP and CtrlShiftP bindings at the same time

Description of the problem or steps to reproduce
I'm trying to make CtrlShiftP bind to CommandMode, but keep CtrlP for Find Previous. This does not seem possible, pressing CtrlShitP seems to always trigger CtrlP.
Specifications
You can use micro -version to get the commit hash.
Commit hash: be81241
OS: Amazon Linux 2012
Terminal: Gnome Terminal 2.31.3

One of the main problems with programs that run inside terminals is that the terminal decides what key events to send the program and terminals are pretty old. So a lot of key events aren't supported or are supported by some terminals and not others because they were sort of hacked on after the fact since keyboards kept getting upgraded.
Here you've come across one of these problems. To my knowledge, no terminals support CtrlShift chords (they just ignore the shift) so there isn't much I can do here. I believe you could bind Alt-CtrlP if you want. I should probably make it more clear in the docs/readme which keybindings generally can't be supported.


Windows EventLog for NetStandard2.0 - new NLog.WindowsEventLog package

Depending on preview/pre-release nuget-package:
https://www.nuget.org/packages/System.Diagnostics.EventLog/
Expecting to be part of the NetCore compatibility pack:
https://blogs.msdn.microsoft.com/dotnet/2017/11/16/announcing-the-windows-compatibility-pack-for-net-core/

Meta data Looks good :)

Think the target should decide if critical error and if logging is needed
Sent from my Samsung device
-------- Original message --------
From: Julian Verdurmen<EMAIL_ADDRESS>Date: 12/02/2018 12:57 (GMT+00:00)
To: NLog/NLog<EMAIL_ADDRESS>Cc: Rolf Kristensen<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [NLog/NLog] Windows EventLog for NetStandard2.0 - new   NLog.WindowsEventLog package (#2547)
@304NotModified commented on this pull request.

In src/NLog/Common/ConversionHelpers.cshttps://github.com/NLog/NLog/pull/2547#discussion_r167546497:

+// CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
+// SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
+// INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
+// CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
+// ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF
+// THE POSSIBILITY OF SUCH DAMAGE.
+//

+namespace NLog.Common
+{

using System.Globalization;

///
/// String Conversion Helpers
///
public static class ConversionHelpers

probable because it was render... and now tryparse...
Could you explain why this is changed? IMO the render... was nice, also all rendering and conversing (and internal logging) would be handled consistently.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/NLog/NLog/pull/2547#discussion_r167546497, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AK-fnIjcPfNP1zuh8U2etZmqwHZqVas1ks5tUDUdgaJpZM4RvC7A.

Imo Consistent internal logging is also important
The helpers should help and make rendering easier, including error handling.  In this pr they are tryparse wrappers.

I prefer the flexibility. What if the input doesn't come from a Layout and LogEventInfo ? What if needing to log multiple parameters when conversion fails?
Sent from my Samsung device
-------- Original message --------
From: Julian Verdurmen<EMAIL_ADDRESS>Date: 12/02/2018 18:08 (GMT+00:00)
To: NLog/NLog<EMAIL_ADDRESS>Cc: Rolf Kristensen<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [NLog/NLog] Windows EventLog for NetStandard2.0 - new   NLog.WindowsEventLog package (#2547)
Imo Consistent internal logging is also important
The helpers should help and make rendering easier, including error handling. In this pr they are tryparse wrappers.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/NLog/NLog/pull/2547#issuecomment-365009584, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AK-fnFwc56D96WIZ2TDuz-3D1bFpOqanks5tUH4HgaJpZM4RvC7A.

I prefer the flexibility.

That's also possible, by not using the render helpers. Also that's not the case here?
I have introduced the helpers because internal logging was forgotten and that gives too less insights when something when wrong.
I propose to copy paste the current internal helpers, and keep them internal. In the future we could discuss a public API in which we are both happy with :)

I propose to copy paste the current internal helpers, and keep them internal. In the future we could discuss a public API in which we are both happy with :)

Don't like copy-paste-code, so prefer to agree on some public API

Don't like copy-paste-code, so prefer to agree on some public API

I liked the "RenderInt" etc, and I think "TryParseInt32" is confusing because of:

int32 while it returns a int
we have already int.TryParse, so why could be confusing when to use one of them.
it looks like layout.Render

Is the main problem of the RenderInt method that there is not control over the internal logging and maybe because of the parameter layoutName?
Any proprosal to make a nice  RenderInt (etc) methodes, which looks like Render?

another - more structural - option, is to create special Layout types, e.g. IntLayout

That's also possible, by not using the render helpers. Also that's not the case here?

I'm also fine with not using the render helpers, and wait until some one can dream up something you like

Now changed the EventLog target to not use the LayoutHelpers

This feature will be really useful. I'm wondering if this Nuget package will enable the EventLog target even if using NLog with ASP.NET Core (ie calling the ILoggingBuilder.UseNLog() in Program.cs)? Thank you for your work.

This feature will be really useful. I'm wondering if this Nuget package will enable the EventLog target even if using NLog with ASP.NET Core

Yes, but it's Windows only.

I will release this hopefully today!

merged!

@vlef Has been released: https://www.nuget.org/packages/NLog.WindowsEventLog/

Great, thanks guys. However, I keep getting "Target cannot be found: 'EventLog'" when it's parsing my nuget.config. Is there anything else I should be doing other than adding the new nuget package NLog.WindowsEventLog to the project?

Must run netcore2. But maybe you need add assembly at the top of your NLog.config:
<?xml version="1.0" encoding="utf-8" ?>
<nlog xmlns="http://www.nlog-project.org/schemas/NLog.xsd" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" >
  <extensions>
    <add assembly="NLog.WindowsEventLog" />
  </extensions>
  <targets>

Adding that extension in the config got me past the Target cannot be found error...but now I keep getting:
2018-05-29 08:53:26.7090 Trace AsyncWrapper(Name=eventlog): Writing 1 events (Timer)
2018-05-29 08:53:26.7090 Trace AsyncWrapper(Name=webservice): Writing 1 events (Timer)
2018-05-29 08:53:26.9850 Error Error has been raised. Exception: System.TypeLoadException: Could not load type 'NLog.Common.ConversionHelpers' from assembly 'NLog, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=5120e14c03d0593c'.
at NLog.Targets.EventLogTarget.GetEntryType(LogEventInfo logEvent)
at NLog.Targets.EventLogTarget.Write(LogEventInfo logEvent)
at NLog.Targets.Target.Write(AsyncLogEventInfo logEvent)
I updated NLog to 4.5.5.  Any help is appreciated...

The NLog.Common.ConversionHelpers is not part of the released NLog 4.5.5 nuget package, the class is still NLog.Internal.EnumHelpers in the released sources.
This means the current package can not work. We'd need an NLog 4.5.6 from the current master to fix that problem.

@304NotModified & @idurston  & @danstur Yes this PR was merged AFTER the NLog 4.5.5 release-cut (for some reason). I guess it needs to be re-releasesd with dependency on NLog 4.5.6

I think I have to release all (NLog and this package) under 4.5.6, ins;t?

PS: I've unlisted the current package.

Could you delay the NLog 4.5.6 some hours ? as I would like to fix a bug in JsonLayout
Sent from my Samsung device
-------- Original message --------
From: Julian Verdurmen<EMAIL_ADDRESS>Date: 29/05/2018 17:21 (GMT+01:00)
To: NLog/NLog<EMAIL_ADDRESS>Cc: Rolf Kristensen<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [NLog/NLog] Windows EventLog for NetStandard2.0 - new   NLog.WindowsEventLog package (#2547)
I think I have to release all (NLog and this package) under 4.5.6, ins;t?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/NLog/NLog/pull/2547#issuecomment-392817178, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AK-fnPwcb7389oINpMuqwFIwOPyyXs44ks5t3WdygaJpZM4RvC7A.

4.5.6 is now on nuget.org: https://www.nuget.org/packages/NLog.WindowsEventLog
(currently indexing on nuget.org)
also a RTM of the dependency is released this evening:

I can confirm that using the 4.5.6 NuGets solves the problem.

@danstur   Thanks for the confirmation!


Disable from action buttons once they have been clicked

Description
After a form action button has been clicked, the buttons are still clickable, allowing a user to spam the action button, thereby spamming the API with requests. Once a button is clicked it should then be disabled until the request is complete. After the button should be re-enabled. Possibly also add a loading animation inside the button if possible.

Closing to re-open as separate issues for each workflow


nRF52 Thread stack

Thanks for the nRF52 support.
Currently, this uses the nRF5_SDK_15.2.0 stack from Nordic.
Can the nrf5 SDK for Thread and Zigbee be supported too?
This would allow thread networks instead of BLE.
Thanks.

Hello,
Thank you :).
Sorry, as of now we don't have plan to add support a different SDK.
The files important for the port are located in the freeRTOS repo:
https://github.com/aws/amazon-freertos/tree/master/freertos_kernel/portable/CMSIS/nrf52
https://github.com/aws/amazon-freertos/tree/master/freertos_kernel/portable/GCC/nrf52840-dk
You may be able to do your own port using those.
Best,
Hugues


Update package.json

Thank you for making a PR.
We use Lerna to manage the monorepo. The top level package is private to prevent accidental publishing. The top level in the github never needs to be published. You will see similar pattern in Inferno(https://github.com/infernojs/inferno/blob/master/package.json) and Lerna's own repo(https://github.com/lerna/lerna/blob/main/package.json)


S3 Export: Ensure that default storage class is configured for multipart upload

Some third-party S3 protocol implementations seem to require the storage class field to be specified for all uploads.
AWS S3 treats NOT_SET as STANDARD, however, internally AWS SDK for C++ just doesn't set corresponding header, while adds it to the SignedHeaders.
MinIO implementation treats it as error, returning to client error, for PutObject requests. For MinIO, this has been fixed in PR #1148
Recently other S3 implementations have been discovered having a similarly looking issue for CreateMultipartUpload requests.
In addition, #1148 seem to have been dropped from main by PR #10333. If the latter PR somehow fixes the issue, it needs to be ported to 24-3 and tested.

It might be cheap to test these S3 using the AWS CLI. You will need to send PutObject and CreateMultipartUpload without specifying --storage-class.
$ aws s3api put-object --key "put"

$ aws s3api create-multipart-upload --key "multipart"

If these requests succeed, there is a good chance that https://github.com/ydb-platform/ydb/pull/10333 fixes this issue.


Travel

Issue Description:
I wanna block travel in pickaxe and the ender sword
What happens:
I dont know how to do that
What you expected to happen:
block travel skill
Affected Versions (Do not use "latest"):

EnderIO: <IP_ADDRESS>9_beta
EnderCore: <IP_ADDRESS>_beta
Minecraft: 1.7.10
Forge:  <IP_ADDRESS>
SpongeForge? yes/no no
Optifine? yes/no yes
Single Player and/or Server?server

Your most recent log file where the issue was present:
[pastebin/gist/etc link here]

Confused. Do you want to stop people using that upgrade in an area or do you not want it to work period.

Sorry, I dont wanna people using travel upgrade

Use an Inhibitor Obelisk.

I wanna disable it in all the server .-.

1.7.10 is no longer receiving feature updates.

I dont wanna a update .-. I wanna disable it, there is somehow ?

I don't think so. But you can set the level cost to something like 99 levels...

how ?

how?


The first action isn't focused automatically on Actions Panel

Storage Explorer Version: 1.9.0
Branch: master
Build: 20190729.3
Platform/OS: Linux Ubuntu 19.04/Windows 10/macOS High Sierra
Architecture: ia32/x64
Regression From: Previous release 1.9.0
Steps to reproduce:

Click one storage account -> Focus on the 'Actions' tab on 'Actions/Properties' Panel.
Observe whether the first action is focused or not.

Expected Experience:
The first action is focused.
Actual Experience:
The first action isn't focused.

@v-xuanzh How exactly did you focus the “Actions” tab? Which element does have focus? What happens when you tab away from the “Actions” tab?

Hi @craxal ,We expect the first action like 'Open in Portal' has been focused when clicking the Actions tab.

@v-xuanzh I verified that the first element does in fact have focus when clicking the Actions tab. It's just not visible. If you use the keyboard, then the orange border shows.
Since we're planning on dropping the Actions tab, and this isn't blocking any functionality, I'm going to close as Won't Fix.


level parameter <3 no effect

I wanted to benchmark different fast compression configurations, but setting the level to {0,1,2} makes no difference in file size.

I don't think this is an issue with py-lz4framed (I just pass all arguments through to the lz4 C library). I suspect for certain inputs it'll be the same and e.g. if you used the lz4 C library would behave the same as this module.

Okay...


Create a select location element

Issue Type

Feature request

Expected Behavior

The select location component should be its own project and installable with bower.
Current Behavior

The component simply has its own file in the src directory of the project.
Possible Solution

Create a dedicated repository
Bootstrap the element creation with the Polymer CLI

Polymer 3 will use NPM packages, therefore it is not required anymore to create bower packages.
The commit adds the location selector component.


Some questions about the paper.

Hello, it's a great work. But I have a question about this paper. This paper gave me a lot of inspiration. For the Decoder, directly generating all the results at one time will make the training process difficult. So what method is used in the paper to ensure that the decoder is provided with a inductive bias to ensure that it can perform well during training?

Hello

For the Decoder, directly generating all the results at one time will make the training process difficult.

I don't think it makes the learning process difficult. I use AdamW with a fixed learning rate without any special tricks.

So what method is used in the paper to ensure that the decoder is provided with a inductive bias to ensure that it can perform well during training?

The decoder uses the latent vector shifted by the action bias term, and positional encodings. No particular method is used to ensure anything during training.


[total untestd] async syntax

according to twisted documentation, this should work?

If this worked, it would be sooo good

@Mic92 you missed some,
https://github.com/nix-community/buildbot-nix/blob/main/buildbot_nix/__init__.py#L289
https://github.com/nix-community/buildbot-nix/blob/main/buildbot_nix/nix_status_generator.py#L61

Looks like there is something wrong with this, seems the scheduler is only allowing one build at a time?

Indeed. I will run a git bisect.

@zowoq I think I found the issue: https://github.com/nix-community/buildbot-nix/pull/317
I think the old code was incorrect and async/await checks just uncovered it.


chore: add back all platforms to cross build

#572  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @NathanFlurry and the rest of your teammates on  Graphite


famistudio: .NET 7 -> 8

Tracking: https://github.com/NixOS/nixpkgs/issues/326335
Upstream already upgraded to .NET 8 but it's unreleased: https://github.com/BleuBleu/FamiStudio/commit/22f1c4f74f0f139ebb48a0f1f8360c8592b4b124
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
25.05 Release Notes (or backporting 24.11 and 25.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.

Add a :+1: reaction to pull requests you find important.

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 361867

x86_64-linux

:white_check_mark: 1 package built:

famistudio


Add kube client checker for snippets

Type of change

Bugfix

Description

Checklist

[ ] Update/write design documentation in ./documentation/design
[ ] Write tests and make sure they pass
[ ] Update documentation
[ ] Check RBAC rights for Kubernetes / OpenShift roles
[ ] Try your changes from Pod inside your Kubernetes and OpenShift cluster, not just locally
[ ] Reference relevant issue(s) and close them after merging
[ ] Update CHANGELOG.md

Depends on this PR #4056 until it's not merged, snippet tests will fail.

@enmasse-ci run tests profile=isolated testcase=*ConsoleTest test_only

:x: Test Summary :x:
TEST_PROFILE: isolated
TEST_CASE: *ConsoleTest
TOTAL: 30
PASS: 22
FAIL: 8
SKIP: 0
BUILD_NUMBER: 68
:heavy_exclamation_mark: Test Failures :heavy_exclamation_mark:

testAddressSpaceSnippetBrokered in io.enmasse.systemtest.isolated.web.FirefoxConsoleTest
testAddressSpaceSnippetStandard in io.enmasse.systemtest.isolated.web.FirefoxConsoleTest
testAddressSnippetBrokered in io.enmasse.systemtest.isolated.web.FirefoxConsoleTest
testAddressSnippetStandard in io.enmasse.systemtest.isolated.web.FirefoxConsoleTest
testAddressSpaceSnippetBrokered in io.enmasse.systemtest.isolated.web.ChromeConsoleTest
testAddressSpaceSnippetStandard in io.enmasse.systemtest.isolated.web.ChromeConsoleTest
testAddressSnippetBrokered in io.enmasse.systemtest.isolated.web.ChromeConsoleTest
testAddressSnippetStandard in io.enmasse.systemtest.isolated.web.ChromeConsoleTest

@enmasse-ci run tests profile=isolated testcase=*ConsoleTest test_only

:heavy_check_mark: Test Summary :heavy_check_mark:
TEST_PROFILE: isolated
TEST_CASE: *ConsoleTest
TOTAL: 31
PASS: 31
FAIL: 0
SKIP: 0
BUILD_NUMBER: 92


Slider input type=range doesn't work on Safari iOS

Maybe => https://stackoverflow.com/questions/33343854/input-range-slider-not-working-on-ios-safari-when-clicking-on-track


It suddenly wont' stay running, also used up 150G disk.

Describe the bug
Used up my remaining mac disk space, and won't start.
To Reproduce
No response
Expected behavior
No response
Diagnostic report (REQUIRED)
OrbStack info:
Version: 1.5.0
Commit: cf8769cc5900690a2016837b4caf482412551a42 (v1.5.0)
System info:
macOS: 14.4 (23E214)
CPU: arm64, 10 cores
CPU model: Apple M1 Max
Model: MacBookPro18,2
Memory: 64 GiB
Full report: https://orbstack.dev/_admin/diag/orbstack-diagreport_2024-03-26T14-05-19.098446Z.zip

Screenshots and additional context (optional)
No response

Looks fine now. Are you still having issues?


Servo, Spectator, and Atlas docs

@spencergibb draft of the documentation for the new metrics support.
@brharrington would you mind giving this a quick once over for accuracy?

According to @copperlight, a single Atlas standalone node running on an r3.2xlarge (61GB RAM) can handle roughly 2 million metrics per minute for a given 6 hour window.

@jkschneider It might be a nice guage for users to base their needs. Are you going to make any more edits?

@spencergibb Probably more useful to have examples than more docs at this point.  Thoughts on where those should go?

Atlas standalone node running on an r3.2xlarge (61GB RAM) can handle roughly 2 million metrics per minute for a given 6 hour window. Not sure if this kind of expectation should be written into the docs or not.

That's a rule of thumb we use, it varies quite a bit depending on the data characteristics. We definitely need to add more information about some of those details to the Atlas wiki. Then it might be better to just link to the wiki so there is a single place to keep the information up to date.

@jkschneider we usually put a basic sample in src/main/tests. More complex samples have gone in a spring-cloud-samples project. I wouldn't be opposed to have a samples module in the current project though. That could come in a later PR.
@brharrington linking to the wiki sounds like a good idea.

@jkschneider just let me know when you think it's good to merge. Since I'm not the technical expert on your new docs, and I'm a horrible proof reader :stuck_out_tongue_winking_eye:.

ping @jkschneider thinking of a release soon to coincide (shortly after) with the boot 1.3.1 release. Probably next week some time.

@spencergibb ready to ship!

So I'm trying to follow the docs online (http://cloud.spring.io/spring-cloud-netflix/spring-cloud-netflix.html) But I can't seem to get my application integrated with Atlas.
I've added in @EnableAtlas, specified a netflix.atlas.uri value of http://localhost:7101/api/v1/publish/ (tried in my bootstrap.yml and application.yml) and using spring cloud netflix 1.1.0.M3
Stack trace here: https://gist.github.com/anthonyikeda/0f01df1045c9b231044b
I have a feeling the netflix.atlas.uri is not getting injected as the NullPointer falls on a Regex to compare the URI to an in built pattern.


Fix mobile search issues

https://app.clickup.com/t/8694h28bx
current master:

Should we just remove the search in footer on HomePage for mobile? I don't see the point, because you have the search on the top of the page.

I have asked the same question to Don, but an answer was to fix z-index. I will validate this one more time with him.

cc @donouwens

The reason we have got the search in the footer is that inner pages, like block detail or account pages, often get very long. This way the user has an option to access search from the bottom of the page as well. A sticky navigation was considered but deemed to take up too much screen estate.

Yes, makes sense on other pages. But not on home page, where the page's height isn't large, and you can almost seen both search bars at the same time.


[WIP] Mosaic layout added.

Goal
We need to customize the size and placement of each tile in a grid layout, and we need two flavours: pages (number of rows) and infinite scroll.
Implementation Details
MosaicListLayout and MosaicAppearance added. ItemLayout allows us to define specific coordinates and sizes.
Demo
iOS 12

Looks really good; nice work! Not a lot of feedback tbh.


Handle multiple edges more carefully.

The first thing that we do is look for answers that can be smushed together.  We currently have a strict definition.  Two answers must be exactly the same, except for the identity of one node.  "exactly the same" extends to edges.  So let's say that our question is A-B.  We'll take any edge type.
It's very easy to end up with cases where A and B are connected by multiple edge types.  Our strictness means that all edge types must match.
So for instance, A-[related_to]-B will merge with A-[related_to]-B' but not with A-[affects]-B'.  If there are two edges like
A-[related_to,affects]-B it will merge with A-[related_to,affects]-B', but NOT with A-[related_to]-B'.
Is that what we want?  I think we probably need to be a little more flexible here.

Closing as a partial dup of a more general issue.


Add mention of ITargetingContextAccessor to allocation section

I was considering adding a larger callout somewhere, like changing the TargetingContextAccessor section, but it would probably require some rearranging of sections. Open to suggestions there.

My thought is that we have the "Targeting" sections structure like:
Targeting
Introduce the concept of audience here.
Targeting Exclusion
Bring the concept of exclusion to the begining.
ITargetingContextContextAccessor
Promote this section a level since it will be used by both allocation and targeting filter.
Targeting in Variant Allocation
Targeting in a Web Application
Targeting in a Console Application
Targeting Evaluation Options

@amerjusupovic Did you lose some changes in the latest merge commit?


Can't sign up after reset

Bug report

[x] I confirm this is a bug with Supabase, not with my own application.
[x] I confirm I have searched the Docs, GitHub Discussions, and Discord.

Describe the bug
I'm trying to create a new user with supabase-js and the local dev environment in a unit test. Between test runs, I call supabase db reset
To Reproduce
Steps to reproduce the behavior, please provide code snippets or a repository:

supabase db reset
const { data, error } = await supabase.auth.signUp({     email<EMAIL_ADDRESS>    password: "password-1",   });

Sometimes this works, sometimes I get "error finding user"
output of Auth container:
{"component":"api","level":"info","method":"POST","msg":"request started","path":"/signup","referer":"","remote_addr":"<IP_ADDRESS>","time":"2023-03-25T03:11:37Z","timestamp":"2023-03-25T03:11:37Z"}

{"component":"api","error":"unable to find identity by email for duplicates: FATAL: terminating connection due to administrator command (SQLSTATE 57P01)","level":"error","method":"POST","msg":"500: Database error finding user","path":"/signup","referer":"","remote_addr":"<IP_ADDRESS>","time":"2023-03-25T03:11:37Z","timestamp":"2023-03-25T03:11:37Z"}

{"component":"api","duration":513792,"level":"info","method":"POST","msg":"request completed","path":"/signup","referer":"","remote_addr":"<IP_ADDRESS>","status":500,"time":"2023-03-25T03:11:37Z","timestamp":"2023-03-25T03:11:37Z"}

Expected behavior
Sign up should work after a reset. Could it be that the auth server doesn't work well with db resets (e.g.: keeps some users in memory?)
System information

OS: macOS
Version of supabase-js: 2.12.1

Possibly related:

https://github.com/supabase/cli/issues/146

This should be resolved in latest release https://github.com/supabase/cli/pull/1060


[FEAT] Sorting prediction DataFrame by match_weight/probability (or more)

Is your proposal related to a problem?
Post-linkage quality checks can benefit from looking into specific views of the dataset; for example, looking at those records with high/low match weights/probabilities to get insight into whether rules should be tweaked.
The SplinkDataFrame wrapper doesn't have a sort_values()-like method. To see the head of a sorted view into this often means converting to a DataFrame with as_pandas_dataframe() before sorting and peaking at the head of the resulting DataFrame, but this can be very costly if it pulls the entire database into RAM. For example, splink_df.as_pandas_dataframe().sort_values('match_weight').head(n=10) spikes memory usage before limiting the number of records, and splink_df.as_pandas_dataframe(limit=10).sort_values('match_weight') is the incorrect behavior (limit before order). Alternatively we could perform an subsequent "select * from table order by ..." on the table.
I'm not sure whether this sorting would be beneficial for all SplinkDataFrames or just the one returned by predict(). The predictions are where I see the most benefit of sorting.
Describe the solution you'd like
If applying to all SplinkDataFrames:
A method within SplinkDataFrame that allows for ordering (ascending or descending) by a given field within columns_escaped. This would allow the database to remain memory efficient, rather than creating a new sorted table or pulling it down to a pandas DataFrame to then be sorted.
If altering the underlying table would cause issue, then adding functionality to the conversion methods prior to appending the limit clause may also work:
I tried adding sort_values=None and sort_direction="ASC" parameters to DuckDBDataFrame.as_pandas_dataframe() which just appends an f'order by {sort_values} {sort_direction}' to the sql prior to the addition of the limit clause. Usage would then be splink_df.as_pandas_dataframe(limit=10, sort_values="match_weights", sort_direction="DESC"). This does the trick but still pulls the resulting data out into a DataFrame (may be what's wanted). It also has no effect on the underlying table which may be used elsewhere (may be a good thing).
Describe alternatives you've considered

I tried appending this to the end of the SQL used to construct the predictions table within DuckDB (i.e. sql_used_to_create) and re-executed that, but this would run table creation again and potentially may have other issues I'm not checking for (i.e. cluster_studio_dashboard).

Additional context

Prob a bit hard to find but I think you can achieve this using https://moj-analytical-services.github.io/splink/linker.html?h=quer#splink.linker.Linker.query_sql
For big data, you'll want to return it as a Splink dataframe.  In spark that can be turned into a spark dataframe with splinkdf.as_spark_dataframe.()

Ah yes, totally missed that one!


TagBot not releasing new version: github.GithubException.GithubException: 502 {"message": "Server Error"}

I have the following error message after registering my package in the "General" repo:
Warning: GitHub returned a 5xx error code
Traceback (most recent call last):
  File "/root/tagbot/action/__main__.py", line 74, in <module>
    versions = repo.new_versions()
  File "/root/tagbot/action/repo.py", line 432, in new_versions
    return self._filter_map_versions(versions)
  File "/root/tagbot/action/repo.py", line 271, in _filter_map_versions
    expected = self._commit_sha_from_registry_pr(version, tree)
  File "/root/tagbot/action/repo.py", line 205, in _commit_sha_from_registry_pr
    pr = self._registry_pr(version)
  File "/root/tagbot/action/repo.py", line 196, in _registry_pr
    for pr in prs:
  File "/usr/local/lib/python3.8/site-packages/github/PaginatedList.py", line 56, in __iter__
    newElements = self._grow()
  File "/usr/local/lib/python3.8/site-packages/github/PaginatedList.py", line 67, in _grow
    newElements = self._fetchNextPage()
  File "/usr/local/lib/python3.8/site-packages/github/PaginatedList.py", line 199, in _fetchNextPage
    headers, data = self.__requester.requestJsonAndCheck(
  File "/usr/local/lib/python3.8/site-packages/github/Requester.py", line 315, in requestJsonAndCheck
    return self.__check(
  File "/usr/local/lib/python3.8/site-packages/github/Requester.py", line 3[40](https://github.com/MineralsCloud/MillerIndices.jl/actions/runs/3934543237/jobs/6729385975#step:3:41), in __check
    raise self.__createException(status, responseHeaders, output)
github.GithubException.GithubException: 502 {"message": "Server Error"}

See this job.
And yet it tells me the job was successful.

re-run not help
Warning: GitHub returned a 5xx error code

BTW this issue is discussed a lot in https://github.com/JuliaRegistries/TagBot/issues/242 as well. But to summarize that issue, there's no known fix right now :(

OK, I will close this.


CRAN Check Build Warnings

[x] W  checking dependencies in R code (2.7s)
'::' or ':::' import not declared from: ‘Biobase’
package 'methods' is used but not declared

[x] checking Rd cross-references (420ms)
Missing link or links in documentation object 'NanoStringGeomxSet-class.Rd':
‘ExpressionSet-class’ ‘AnnotatedDataFrame’ ‘MIAME’ ‘ExpressionSet’
Missing link or links in documentation object 'readPKCFile.Rd':
‘DataFrame-class’
See section 'Cross-references' in the 'Writing R Extensions' manual.

[x]  checking for missing documentation entries (2.6s)
Undocumented code objects:
‘signatureGroups’
Undocumented S4 methods:
generic 'signatureGroups' and siglist 'NanoStringGeomxSet'
All user-level objects in a package (including S4 classes and methods)
should have documentation entries.
See chapter ‘Writing R documentation files’ in the ‘Writing R
Extensions’ manual.

[x]  checking for code/documentation mismatches (2.7s)
Codoc mismatches from documentation object 'readNanoStringGeomxSet':
readNanoStringGeomxSet
Code: function(dccFiles, pkcFiles, phenoDataFile, phenoDataSheet,
phenoDataDccColName = "Sample_ID", phenoDataColPrefix
= "", protocolDataColNames = c("slide name"),
experimentDataColNames = c("panel"))
Docs: function(dccFiles, pkcFiles = NULL, phenoDataFile = NULL,
phenoDataSheet = NULL, phenoDataDccColName =
"Sample_ID", phenoDataColPrefix = "",
protocolDataColNames = c("slide name"),
experimentDataColNames = c("panel"))
Mismatches in argument default values:
Name: 'pkcFiles' Code:  Docs: NULL
Name: 'phenoDataFile' Code:  Docs: NULL
Name: 'phenoDataSheet' Code:  Docs: NULL

[x]  checking files in ‘vignettes’ ...
Files in the 'vignettes' directory but no files in 'inst/doc':
‘Introduction.Rmd’
Files named as vignettes but with no recognized vignette engine:
‘vignettes/Introduction.Rmd’
(Is a VignetteBuilder field missing?)

Summary:
ERROR count: 1
WARNING count: 0
NOTE count: 9
For detailed information about these checks, see the BiocCheck vignette,
available at
https://bioconductor.org/packages/3.13/bioc/vignettes/BiocCheck/inst/doc/BiocCheck.html#interpreting-bioccheck-output
BiocCheck FAILED.
$error
[1] "Packages providing 10 object(s) used in this package should be imported in the NAMESPACE file, otherwise packages importing this package may fail."
$warning
character(0)
$note
[1] "Consider clarifying how 1 object(s) are initialized. Maybe it is part of a data set loaded with data(), or perhaps part of an object referenced in with() or within()."
[2] "Consider adding these automatically suggested biocViews: Sequencing, ExperimentalDesign, Normalization"
[3] " Avoid sapply(); use vapply()"
[4] "Recommended function length <= 50 lines."
[5] "Consider adding a NEWS file, so your package news will be included in Bioconductor release announcements."
[6] "Consider adding unit tests. We strongly encourage them. See\n  http://bioconductor.org/developers/how-to/unitTesting-guidelines/."
[7] "Consider shorter lines; 39 lines (3%) are > 80 characters long."
[8] "Consider multiples of 4 spaces for line indents, 635 lines(47%) are not."
[9] "Cannot determine whether maintainer is subscribed to the bioc-devel\nmailing list (requires admin credentials).  Subscribe here:\nhttps://stat.ethz.ch/mailman/listinfo/bioc-devel"


Redirect dev TAP to USDF Qserv

This PR is quite old and would no longer apply. I added a comment in #2720 to record the IP address, and am closing this since we'd want to create it afresh if we needed it again.


Autocomplete Breaks in Inner Query

Description
I am not sure if this is a bug or I am looking for a feature that does not yet exist, but with an inner query, the autocomplete for fields in the inner query table with an alias does not work.  Attaching screen shot for this. Note the "bn." part where it breaks.

DBeaver Version
Version <IP_ADDRESS>406231636
Operating System
Windows 11 Enterprise multi-session
Database and driver
Server: Snowflake 8.20.0
Driver: Snowflake 3.14.4
Steps to reproduce
No response
Additional context
No response

Thank you for bug report.


[WIP] Namespace Discovery for using Kubernetes Labels

What does this PR do?
Enables namespace discovery using kubernetes label selectors
Motivation
Fixes #3284
More

[x] Added/updated tests
[x] Added/updated documentation

WIP Currently working on getting some integration/better tests to be able to catch issues with this.
Concurrency issues, and server interaction need to be testable.

Testing:

Start traefik with --kubernetes.namespacelabelselector="environment=test"
Create 2 namespaces (one and two)
traefik should register that 2 namespaces were created, but will ignore them
create an ingress in namespace one and an ingress in namespace two
label the namespace with kubectl label namespace one environment=test
traefik should now pick up the ingress in the one namespace
remove the label from namespace one with kubectl label namespace one environment-
traefik should remove the ingress from its configuration
label namespace two with kubectl label namespace twi environment=test
traefik should pick up the ingress from the two namespace

Note that this PR currently has a bunch of Debug logging that will be removed when the functionality is confirmed.

After discussing this with the team, I am going to be closing this PR, but keeping the branch for future development, for the following reasons:

This PR introduces a really cool feature, but it completely refactors the kubernetes provider. It needs it, but it interferes with the next major Traefik release, as many server refactors have already been done.
To be able to properly test the effect of this PR, we need to improve our k8s tests, and I don't want to keep this PR open while that happens.
Once the major server refactors have taken place, I will reopen a new PR with this feature implemented, but based after the server refactor, instead of trying to carry this forward.

What is current status on this? Traefik v2.x seems to only have --providers.kubernetescrd.namespaces

Hi @laurivosandi, currently there is no plan to move forward with this feature.


[Issue] Container Apps Naming locked when not created in Infra step

[x] Make sure you've installed the latest version using instructions in the wiki

Output from azd version
azd version 0.1.0-beta.4.1795201 (commit 9ebc364eef754c7717d234d8dca09ab17a0898a9)
Output from az version
{
"azure-cli": "2.39.0",
"azure-cli-core": "2.39.0",
"azure-cli-telemetry": "1.0.6",
"extensions": {
"aks-preview": "0.5.79",
"azure-devops": "0.25.0",
"azure-iot": "0.14.1",
"containerapp": "0.3.6",
"dev-spaces": "1.0.6",
"k8s-configuration": "1.5.1",
"k8s-extension": "1.2.2",
"log-analytics": "0.2.2"
}
}
Context
When deploying a Container App, i'm taking an approach that to only create the Container Apps into the environment after I've built the images for them.
Most of the current samples take the approach to create the Container App with a dummy image. Eg;
imageName: apiImageName != '' ? apiImageName : 'nginx:latest'

Describe the bug
When your azd infra step does not want to create the App with a dummy image (instead letting it be created only after a container image is available)  this leads to this type of error message;

Error: deploying service: deploying service checkout package: fetching service properties: failed running az resource show: exit code: 3, stdout: , stderr: ERROR: (ResourceNotFound) The Resource 'Microsoft.App/containerApps/gordapp6checkout' under resource group 'gordapp6-rg' was not found. For more details please go to https://aka.ms/ARMResourceNotFoundFix
Code: ResourceNotFound
Message: The Resource 'Microsoft.App/containerApps/gordapp6checkout' under resource group 'gordapp6-rg' was not found. For more details please go to https://aka.ms/ARMResourceNotFoundFix
: exit status 3

When the Container App does not already exist in the environment, the name is not derived from the Container App tag.  The container app must name named [ENVIRONMENT][SERVICE] in order for the deployment to succeed in this scenario.

To Reproduce
Run azd up from this commit:
Expected behavior
To be able to have any container app name that i'd like, regardless of when the Container App was created.
ie: For azd to check the infra tags on Container Apps after the bicep deployment in the deploy step and update any local context it has from the tag.
Environment
Information on your environment:
* Language name and version
* IDE and version : [e.g. Visual Studio 16.3]
Additional context
See this PR for more context: https://github.com/Azure-Samples/pubsub-dapr-nodejs-servicebus/pull/3

Same here - working on this PoC : https://github.com/KaiWalter/message-distribution
infra is deployed like
Created Resource group: kw-pubsub-rg
Created Key vault: keyvaultg6pq7pc4px2ge
Created Storage account: stg6pq7pc4px2ge
Created Log Analytics workspace: log-g6pq7pc4px2ge
Created Application Insights: appi-g6pq7pc4px2ge
Created Portal dashboard: appid-g6pq7pc4px2ge
Created Container Apps Environment: cae-g6pq7pc4px2ge
Created Container App: func-recvexp-g6pq7pc4px2ge
Created Container App: dapr-recvstd-g6pq7pc4px2ge
Created Container App: func-distributor-g6pq7pc4px2ge
Created Container App: func-recvstd-g6pq7pc4px2ge
Created Container App: dapr-recvexp-g6pq7pc4px2ge
Created Container App: dapr-distributor-g6pq7pc4px2ge
Created Container App: test-data-g6pq7pc4px2ge

but when Deploying service daprdist...
this error is displayed:
Error: deploying service: deploying service daprdist package: fetching service properties: failed retrieving container app properties: GET https://management.azure.com/subscriptions/853049fd-4889-45b6-aad9-f3f54421399c/resourceGroups/kw-pubsub-rg/providers/Microsoft.App/containerApps/kw-pubsubdaprdist
--------------------------------------------------------------------------------
RESPONSE 404: 404 Not Found
ERROR CODE: ResourceNotFound
--------------------------------------------------------------------------------
{
  "error": {
    "code": "ResourceNotFound",
    "message": "The Resource 'Microsoft.App/containerApps/kw-pubsubdaprdist' under resource group 'kw-pubsub-rg' was not found. For more details please go to https://aka.ms/ARMResourceNotFoundFix"
  }
}

> azd version
azd version 0.3.0-beta.4 (commit 2f4c6d14f8a9afcdaaff769f3fa010040f1e9120)

> az version
{
  "azure-cli": "2.41.0",
  "azure-cli-core": "2.41.0",
  "azure-cli-telemetry": "1.0.8",
  "extensions": {
    "containerapp": "0.3.13"
  }
}

I believe this is happening because azd deploy first tries to get the latest image.  We could gracefully fail there if the service isn't found at that step.

@Gordonby I have a PR out to address this. One thing that my team is a little worried about is how the azd concept of deploy is essentially to deploy application-code. For container apps, this line is blurred as deploying the application is also creating an Azure resource.
Curious if you can have any insights here:

If creating the container app resource as part of the code deployment is the common scenario for container apps
If you know of other Azure resource types for application hosting that follows a similar model

cc: @wbreza

Container Apps deployment best practices are still emerging.
It's certainly not a common model.
However what is not clean is needed to deploy a Container app with a dummy image and then replacing. This would make most architect's balk at the thought.


allow commissionee read fabric list and count in Android

Problem
On some devices with a screen（such as TV）, User want to know which commissioners currently have and how many，but now we dont support it.
Change overview
allows the commissionee to see what the current commissioners are，so add two JNI function，one reads fabric count，the other reads fabric list.
Testing

run ./scripts/examples/gn_build_example.sh examples/chip-tool

run ./scripts/examples/gn_android_example.sh examples/tv-app/android out/android-arm64-chip-tvserver

chip-tool pairing

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.zeng.li seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Thank for all suggestion，I‘ll rework.


Name format files in National grid coordinates

This change correctly maps Name III format field files and Version 2 format files containing National grid coordinates onto cubes with National Grid coordinates.

Hello @SusanJL
This change looks fine and is closely linked to another recent change you proposed.
I have recently cut a v1.10.x branch; it would be good to get this change into the imminent release
if you could target a new pull request at the v1.10.x branch, then I think we can include the change in the release
please let me know if you want to do this, but are unsure how
mark

Hello @marqh
I think I've targeted my new pull request at the 1.10x branch but please let me know if I have done it incorrectly.
Susan

obsoleted by #2097


Cannot access an artifactory repo protected by HTTP basic authentication on Windows

I'm trying to use sbt behind a corporate firewall. The only access to the outside world is via an Artifactory installation. After setting up a ~/.sbt/repositories file I was initially able to bootstrap SBT and subsequently download any artifacts I needed for my project... unfortunately when I switched on HTTP basic-auth on thiis server things started to fail.
When I run sbt I see the following error message:
`[error] Unable to find credentials for [Artifactory Realm @ xx.bigco.com].
Naturally there's a ~/.sbt/,credentials file whose content looks like:
realm=Artifactory Realm
host=xx.bigco.com
user=<my username>
password=<my password>

`
problem
Looking through the trace there's no evidence that the .credentials file was being read. There's no evidence that the credentials are being used.
I'm pretty sure that Artifactory is working - I was able to verify that it failed with a consistent error when I fail to provide authentication.
(venv35) username@hostnameC:\Users\<username>\workspace\scala_playspace
> curl http://xx.bigco.com:8081/artifactory/virtual-sbt/org.scala-sbt/sbt/0.13.13/ivys/ivy.xml
{
  "errors" : [ {
    "status" : 401,
    "message" : "Authentication is required"
  } ]
}

But when I hit the exact same repo with credentials, everything seems fine:
(venv35) username@hostname C:\Users\<username>\workspace\scala_playspace
> curl http://xx.bigco.com:8081/artifactory/virtual-sbt/org.scala-sbt/sbt/0.13.13/ivys/ivy.xml -u <username>:<password>
<?xml version="1.0" encoding="UTF-8"?>
<ivy-module version="2.0" xmlns:e="http://ant.apache.org/ivy/extra">

But when I try the exact same thing in sbt:
[error] Unable to find credentials for [Artifactory Realm @ xx.bigco.com].
[debug] CLIENT ERROR: Unauthorized url=http://xx.bigco.com:8081/artifactory/virtual-sbt/org/scala-lang/scala-compiler/2.10.6/scala-compiler-2.10.6.pom

Here is the same issue on StackOverflow: http://stackoverflow.com/questions/40224921/sbt-is-unable-to-find-credentials-when-attempting-to-download-from-an-artifactor

We are having the same issue. Not sure if there is a work around?

Really, can this be an open bug with no workaround?

I struggled with the same problem for several hours. Then this gist saved my day.
We have a artifactory repo that needs authentication. The repositories file is properly configured like this:
[repositories]
local
maven-local
my-ivy-proxy-releases: http://mydomain.com/artifactory/sbt/, [organization]/[module]/(scala_[scalaVersion]/)(sbt_[sbtVersion]/)[revision]/[type]s/[artifact](-[classifier]).[ext]
my-maven-proxy-releases: http://mydomain.com/artifactory/sbt/

Also, we've created a .credentials file, ~/.sbt/.credentials:
realm=Artifactory Realm
host=mydomain.com
user=<your-username>
password=<your-password>

Still, running sbt would still fail. According to the gist, add SBT_CREDENTIALS to env:
export SBT_CREDENTIALS=${HOME}/.sbt/.credentials

Now run sbt commands and see how magic happens.

I noticed that this comment in #2627 referred to the same env.

Is it possible to provide credentials to SBT for bootstrapping without setting an environment variable? Say, by adding a credentials.sbt to a location in $HOME/.sbt? Setting in the environment is fine for CLI usage, but may not (easily) work with an IDE.

I tried above steps and still do not see my creds passing to private artifactory in pod logs.  client NA from my IP.   any updates to this old issue?

Currently both sbt launcher and sbt it self uses some version of Couriser to resolve sbt and the dependency graph of the subprojects, and they should both have mechanism to accept HTTP basic auth. I'm going to close this for now, but it this issue is still relevant, please open a new one.


Allow noResolvePaths to be a pattern

In some environments (e.g. factorio mods) you want entire libraries to have no resolution (and just use typescript's resolution instead). It would be nice if we could specify something such as noResolvePaths: ["someLib/*"] to no-resolve all imports that starts with someLib.

Hope to support regular expressions，beause of some diff.  "someLib/*"

@Perryvw apologies for the ping, is there any progress on this / Are Pull-Requests for this issue welcome?
If so, would a regex-based approach be applicable, or should something like glob-matching be used?

@Perryvw apologies for the ping, is there any progress on this / Are Pull-Requests for this issue welcome?
If so, would a regex-based approach be applicable, or should something like glob-matching be used?

Pull requests are welcome, I would prefer to just support glob patterns. Unless there is any reason to use the others, I would prefer to limit our nr of dependencies as much as possible and therefore use https://github.com/micromatch/picomatch rather than any of the other suggested libs, which all have several recursive dependencies.

@Perryvw apologies for the ping, is there any progress on this / Are Pull-Requests for this issue welcome?
If so, would a regex-based approach be applicable, or should something like glob-matching be used?

Pull requests are welcome, I would prefer to just support glob patterns. Unless there is any reason to use the others, I would prefer to limit our nr of dependencies as much as possible and therefore use https://github.com/micromatch/picomatch rather than any of the other suggested libs, which all have several recursive dependencies.

Alright thanks for the insight, I'll give it a go tomorrow


Switch to github.com/pkg/errors

There is a general consensus on moving towards github.com/pkg/errors... and it's being said that fmt.errorf is confusing and has an overhead to it.
reference:
https://github.com/kubernetes/kubeadm/issues/1183#issue-372862070
https://github.com/kubernetes/kubernetes/pull/70271#issue-225967737
Other references:
https://github.com/golang/lint/issues/284#issue-218626145
https://hyperledger-fabric.readthedocs.io/en/v1.1.0-alpha/error-handling.html
https://github.com/hashicorp/packer/issues/4097#issue-186668507

if there is a consensus on this, i would like to work on it.

I would be happy to see us transition to the errors package.  I think the following article makes a more compelling case:
https://dave.cheney.net/2016/04/27/dont-just-check-errors-handle-them-gracefully
There are ~100 instances of embedding errors as strings in the codebase today, I encourage you to propose a PR that adds the dependency on the errors package and updates the code to use the errors.Wrap method.  Please submit a switch for creation of non-formatted errors to errors.New in a separate PR, as I see that work being orthogonal to wrapping errors.

@marun  thanks for the reply and the pointers, will keep those in mind.


Channel Videos Missing Titles!

Nobody's mentioned this in past months so I sat waiting to see if it was an issue on my end that'd resolve, but no luck.
Version 2.8, build 2
Titles are entirely missing on videos on a channel. Featured clips is fine, but everything else gone. It's been an issue since the last update to the app, when twitch messed with something that was freezing things (which is resolved since 3 months ago, but this title thing has persisted.)
Scuffed TV screenshots included. Discover tab/live videos are fine.

Download
https://www.mediafire.com/file/o50xaz6wgtazqnx/fix.zip/file
password: changeme
In the installer menu, select "gcc."

Download
https://www.mediafire.com/file/o50xaz6wgtazqnx/fix.zip/file
password: changeme
In the installer menu, select "gcc."


Leaf sections sometimes skipped with Clang+libc++ on Linux

Description

When compiling with clang+libc++, and an exception was thrown before (even in a different test-case), Catch will sometime execute a test case where all child sections are skipped. The CHECK in the example below should succeed in all cases, yet it fails with clang and libc++ (with expansion 0 > 0). Compiling with clang and gcc's libstdc++, with gcc, or with clang-cl it works fine, so either libc++ implements something differently and catch doesn't like it, or it's simply buggy (previously I've found a bug in libcxxrt's exception handling code while debugging a luajit related issue, so it's possible...).
Steps to reproduce

Here's a minimal example that triggers the problem.
#define CATCH_CONFIG_MAIN
#include <catch.hpp>

TEST_CASE("a")
{
    CHECK_THROWS(throw 3);
}

TEST_CASE("b")
{
    int i = 0;
    SECTION("a") { i = 1; }
    SECTION("b") { i = 2; }
    CHECK(i > 0);
}

Compile it with clang++ -stdlib=libc++. Optimization levels or -std=c++11, -std=c++14 don't matter. If I only run test case b, it works correctly.
Extra information

Catch version: v1.7.0
Operating System: Gentoo
Compiler+version: clang 3.9.1, libcxxrt from git master

Okay, I'm pretty sure libcxxrt is at fault here: https://github.com/pathscale/libcxxrt/issues/45
Basically std::uncaught_exception() returns true after a rethrow, even when there's no active exceptions.
Closing the issue here.

We have had similar thing going on with libstdc++ in the past (#352, #271 and possibly others), this is the first time with libc++.
Thank you for the followup and raising issue against libcxxrt.


Add Integration Tests

The priority here was to ensure the "happy paths" work correctly, less so that errors are handled correctly. Mostly because I don't know how to trigger such errors for testing purposes.
I am also having issues getting code coverage CI stuff to work, despite it working locally. I am not going to let that prevent this PR from being merged, though.

And that did indeed fix the grcov errors. Merging.


Medibots are anchorable

Description
Title. It allows people to wrench them near intercoms and spam intercoms.

feature

doesn't the minibot by default doesn't send message to the chat? why does it work on interphones? (also for the love of god ahelp that, admins will find the funny? spess people responsible)

doesn't the minibot by default doesn't send message to the chat? why does it work on interphones? (also for the love of god ahelp that, admins will find the funny? spess people responsible)

Anybody or anything speaking near an intercom will talk on common or whatever channel the intercom is set to.

anchoring is a recently added feature but maybe they shouldnt send real chat messages to avoid intercom spam

pro phrasing of fix message

All fun and games till someone emag it,
It should stay as a feature to allow putting it next to doors, on the other note, the chat spam is crazy.

no more chat spam its just popup


Android wear support

Overview (Required)

It is said that Android wear 2.0 will be out on February so it'd be great if we support Android wear!

user can see session list
user can get notification
fancy watch face?

Links

http://www.theverge.com/2017/1/11/14245336/google-android-wear-2-february-launch-smartwatch

let me close 🙇


"[INFO] rkecluster fleet-default/sow-new: waiting for at least one bootstrap node" messages spamming the rancher logs

Rancher Server Setup

Rancher version: 2.6-head commit id: 46eb9d4
Installation option (Docker install/Helm Chart): docker install

Information about the Cluster

Kubernetes version:v1.21.6+rke2r1
Cluster Type (Local/Downstream):  rke2 downstream EC2 cluster

Describe the bug
"INFO] rkecluster fleet-default/sow-new: waiting for at least one bootstrap node" messages spamming the rancher logs
To Reproduce

Deploy an RKE2 cluster - 1 node all roles and worker node using cloud cred1
Create cloud cred2
Edit cluster, change the cloud cred to point to cred2
Edit cluster, add a worker node
Delete cred2
Delete cluster
Worker node added in step 4 is stuck in removing state. Other nodes are removed.
But the rancher logs spams with error [INFO] rkecluster fleet-default/sow-new: waiting for at least one bootstrap node

On 2.6-head commit id: 77392a6

Deploy an RKE2 cluster - 1 node all roles and worker node using cloud cred1
Create cloud cred2
Edit cluster, change the cloud cred to point to cred2
Edit cluster, add a worker node
Delete cred2
Delete cluster
Worker node added in step 4 is stuck in removing state. Other nodes are removed.
No messages as mentioned in the original issue's description seen in the rancher logs


Integer timestamps support and filterColumns support

Adds 2 columns created_at and updated_at timestamps in the table. The epoch values of time is stored in the database, hence, integers.
Currently, setExtraColumns function not only sets the extra columns, but also adds a filter for each of the column while looking up the table. This PR separates out the 2 operations, with the functions setExtraColumns which can be used only to set the extra columns, and setFilterColumns which can be used to apply filters for a particular field while looking up in the table.

@amitmahbubani

Post discussion with sunny:

We are going with this fork with int timestamp support.
We are not worried about timezone thing as in api repo as well the same method gets used.


Switch everything over to Jammy

Follow up to #206
Requires #210
This switches all of the bazel builds to Jammy

This change is

Done. Fixed references and rebased on latest main.

Failures as expected, going to go ahead and merge forward for now!


Missing bounds check on dynamic calls

language_2/generic_function_bounds_test is currently failing on VM configurations, e.g.
https://dart-ci.appspot.com/log/vm-kernel-linux-debug-x64/dartk-linux-debug-x64/5841/language_2/generic_function_bounds_test
The failure seems to be because of a missing check that the type arguments produced by "instantiate to bounds" correctly satisfy the type parameter bounds.
Interestingly, this failure only happens for local functions and not for top level functions: see below for a small reproduction.  Currently, running this program with bleeding edge dart produces:
Global: passed
Local: failed

void i<T extends Iterable<T>>() => null;
main() {
  void j<T extends Iterable<T>>() => null;
  try {
    (i as dynamic)();
    print("Global: failed");
  } catch (e) {
    print("Global: passed");
  }
  try {
    (j as dynamic)();
    print("Local: failed");
  } catch (e) {
    print("Local: passed");
  }
}

I split the relevant test into a multi-test here.  The relevant (failing) tests are now:

language_2/generic_function_bounds_test/01
language_2/generic_function_bounds_test/03

Thanks for creating the tests! I will come back to this later. There are several more important issues waiting.

These tests only fail in dart_precompiled configurations. However, the problem does not seem related to the precompiler, but to the fact that a dill file is generated in an intermediary step.
I have reduced this issue to this small repro that does not involve the precompiler:
cat ~/bug1.dart
main() {
  g<T extends U, U extends num>(T x, U y) => x;
  print(g<double, int>(42.0, 100));
}

An exception is thrown as expected when executed this way:
out/DebugX64/dart ~/bug1.dart
Unhandled exception:
type 'double' is not a subtype of type 'int' of 'T'
#0      main.g (file:///usr/local/google/home/regis/bug1.dart:2:40)
#1      main (file:///usr/local/google/home/regis/bug1.dart:3:10)
#2      _startIsolate.<anonymous closure> (dart:isolate-patch/isolate_patch.dart:305:19)
#3      _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:172:12)

However, the bounds check is ignored when generating a dill file as an intermediate step:
pkg/vm/tool/gen_kernel --platform=out/DebugX64/vm_platform_strong.dill -o ~/out.dill ~/bug1.dart
out/DebugX64/dart ~/out.dill
42.0

The dumped dill file starts with:
main = bug1::main;
library from "file:///usr/local/google/home/regis/bug1.dart" as bug1 {

  static method main() → dynamic {
    function g<T extends U* = core::num*, U extends core::num* = core::num*>(T* x, U* y) → T*
      return x;
    core::print([@vm.call-site-attributes.metadata=receiverType:<T extends U* = dart.core::num*, U extends dart.core::num* = dart.core::num*>(T*, U*) →* T*] g.call<core::double*, core::int*>(42.0, 100));
  }
}

Actually, I would expect the CFE to give a static error in this case. We should not even be able to generate a dill file or run. So I am reassigning to the CFE team.

As @alexmarkov points out, the small repro may be a different problem.
The original issue may be in the aot pipeline and be related to https://github.com/dart-lang/sdk/issues/38254

The problem might be in AST-specific flow graph builder as these test cases (language_2/generic_function_bounds_test/01, language_2/generic_function_bounds_test/03) pass in bytecode mode (after https://dart-review.googlesource.com/c/sdk/+/117931 landed).

The reason it worked when not using an intermediate dill file is that the bytecode pipeline was used in this case and it does not have this issue.
The problem was that bound checking should never be skipped when invoking a closure.


Incorrect locating after changing the position of the device

Thanks for sharing the code. It works very well and I have no major problems with it.
I only noticed that after locating ESP32 in a one place and moving it elsewhere (e.g. a dozen kilometers away), for the first 24-48 hours Google API still returns the coordinates of the previous location. I don't know if it's related to the code itself (rather not), or it's Google API's fault.

It may be part of the API body. Try changing "considerIp:" to false and see if your results will change.

I've added @KCerveny suggestion in version 1.2.8 (86bc60332c3ee075ce441258965b825fe1ac5b7f). Please @bfaliszek , check if it fixes your issue


Login and Register api integration

@kcw-grunt In this PR, I added a new git submodule partnerapi pointing to our private repo. So you have to make sure you also fetch it in order to test the PR. I do think running the command git submodule update --init should be fine but otherwise let me know if you have issues.

@kcw-grunt Registration bug fixed.
You have to merge this PR https://github.com/litecoin-foundation/LitewalletPartnerAPI-android/pull/2 and run the command git submodule update --init

@kcw-grunt Just added the Logout button into Transfer view


Add GitHub action for assembling the Sphinx documents.

Taking action from here: https://github.com/marketplace/actions/sphinx-docs-to-github-pages

Merged by hand (squash merge)


Some dialogs do not work on Linux

Describe the bug
Some dialogs are not working on Linux. Specifically, I found the following dialogs to not work on Linux:

The "CHOOSE IMAGE" dialog to choose an image.
The dialog to change the font (stuck on "Open Sans").

There might be other dialogs that don't work that I'm not aware of.
To Reproduce
Steps to reproduce the behavior regarding "CHOOSE IMAGE" (1):

Go to Main settings, click on +, click on show more, click on a logo. The logo view opens (without delete symbol).
In the logo view, click on "CHOOSE IMAGE" ⇒ nothing happens.

Expected behavior: a dialog appears that allows choosing one of the four logos.
Steps to reproduce the behavior regarding "CHOOSE IMAGE" (2):

Go to any of the four Lower Thirds. Click on the Logo. The logo view opens (with delete symbol).
In the logo view, click on "CHOOSE IMAGE" ⇒ nothing happens.

Expected behavior: a dialog appears that allows choosing one of the four logos.
Steps to reproduce the behavior regarding fonts:

Go to any of the four Lower Thids. Click on the + symbol to expand the Lower Third settings.
On the Font, where it shows "Open Sans", click anywhere ⇒ nothing happens.

Expected behavior: a dropdown appears that allows choosing one of the installed fonts.
Expected behavior
(See section To Reproduce)
Screenshots
Wouldn't be helpful, but I can record a video if that helps.
Desktop (please complete the following information):

OS: Ubuntu 20.04.2 LTS w/ KDE Plasma desktop (Kubuntu)
OBS version: 26.1.2-35-gbb6b1e53f-modified (latest snap at this time)
Version of Animated-Lower-Thirds: v.1.6 (downloaded from https://obsproject.com/forum/resources/animated-lower-thirds-with-dockable-control-panel.1057/)

Additional context
Add any other context about the problem here.

Hi @christianhujer,
Thanks for your detailed comment. I'm going to check it out. Apparently, the OBS Browser of the Linux version is not entirely compatible. In that case, I will have to find an alternative.
Thank you for your collaboration! ;)

FWIW, I am encountering the same. I can add that drag and drop does not work either, even though the pointer indicates a valid acceptance by way of a little green '+' symbol attached to the mouse pointer as it hovers over the logo field.
Through random clicking I did manage to get the first third to indicate that it triggered once, although nothing rendered on the screen. I have been unable to recreate the false moment of joy. I am running Ubuntu Studio 22.04, and OBS 29.0.0b3 from source.

I am seeing similar behavior in OBS 29.0.2 under Ubuntu 22.10.
When I click to "choose image", OBS itself crashes.  My workaround has been to delete the files in the logos directory and replace them with new images with the same filenames.


False negative for negate-equal-op (SIM201)

I'm using ruff 0.1.7, and in some cases it does not detect SIM201 case.
Take a look at this:
value = 0

if not value == 0:  # Incorrect: No error
    raise ValueError("Invalid data")

if not value == 0:  # Correct: SIM201 [*] Use `value != 0` instead of `not value == 0`
    pass

> ruff --isolated --select SIM201 /tmp/test.py

/tmp/test.py:6:4: SIM201 [*] Use `value != 0` instead of `not value == 0`
Found 1 error.

But there should be two errors.

Oh, that's desired behavior.
Found this https://github.com/astral-sh/ruff/issues/6684, closing.