Unable to query multiple label values using PromQL and variables

When I create a variable that allows multiple selections and attempt to include it in a PromQL query such as foo{label~="$values"} it becomes:
foo{label~="{bar,baz}"}
This is not valid PromQL and the query returns empty.
The correct way would be to use foo{label~="bar|baz"} but I have no idea how to instruct the plugin to construct this type of multivalue representation.

Nvmd wrong plugin


Abnormal memory usage

Phenomenon
We found that ng-monitoring sometimes uses unexpected amounts of memory, up to tens of GiB (on a cluster with a medium number of nodes). It may even cause OOM.
Troubleshooting
By observing the buffer/cache usage of memory, it is found that a large amount of memory usage comes from badger. It is worth noting that whenever there is high memory usage, it is usually accompanied by abnormal disk space usage of the
docdb directory. The docdb directory is maintained by badger. In particular, whenever the problem occurs, there will be a large number of .sst files (hundreds to thousands) in the docdb directory. It seems that badger's LSM-tree compaction did not work as expected. Combined with the buffer/cache usage, we infer that the abnormal memory usage is caused by abnormal disk files.
Purpose

Fix the problem of large number of .sst files in badger and the abnormal memory usage caused by it through upgrading or other means.
Try using another storage engine, such as SQLite. (#116)

/assign


Update to newer MOAB for CI

I noticed that CI is using MOAB 5.3.1.  Is there a reason for not updating to something newer?

I believe it has been added since we encountered an issue with MOAB, which has recently been resolved in DAGMC. I think we can move it to the latest version. I think, @pshriwise will be agreed with me.

No reason in particular. I was probably in a hurry to get testing setup for external contributions and selected that version on-the-fly. I'm not too worried about it as this all interacts through PyMOAB and in turn through MOAB's core interface -- both of which are pretty stable at this point. That said, no reason not to update it if someone has the bandwidth.

I have a request for you, @pshriwise. I recently created a PR on the MOAB repo for wheel support, as I noticed you were one of the core developers of PyMOAB from your commits there. I would be very grateful if you could review the PR. Once it's released, I hope we can add the new MOAB to our CI.

No reason in particular. I was probably in a hurry to get testing setup for external contributions and selected that version on-the-fly. I'm not too worried about it as this all interacts through PyMOAB and in turn through MOAB's core interface -- both of which are pretty stable at this point. That said, no reason not to update it if someone has the bandwidth.

I can probably give it a shot, but this may also precipitate an HDF5 update since CI currently relies on the Ubuntu std HDF5 that is kind of old...


[Question] How to handle possible duplicate "notification" event for notification with content-available=1

First of all, this is not an issue but a question on how to use the awesome "phonegap-plugin-push" plugin to meet my requirements. My question is specifically for iOS platform, as I haven't tried the plugin with Android just yet.
For notification with content-available=1, the notification event handler will potentially get called at two times. First is when the push notification is received while the app is in the background. And then when the user taps on the push notification to open the app, the event handler gets called again. The data argument provided to the event handler is the same in both occurrences. My question is how can my code distinguish one case from the other to avoid duplicate processing. The only way that I can think of is relying on Cordova events - "pause" and "resume" - to detach/attach handlers to the notification event. I think that is doable but I am looking forward to see if there are other more straightforward solutions. Something supported by the "phonegap-plugin-push" plugin would be nice. Thanks.

@tamakisquare in the push payload you send to your app include a unique identifier. Then in local storage you can indicate that the payload has already been processed. That way you handle it in the background and can safely ignore it when the user taps on the notification.
Make sense?

Sounds good! Thanks for your great work and help, @macdonst . Cheers.

@tamakisquare kk, I'm going to re-open this though as a doc issue as more than one person has asked so I should add it to the docs.

@macdonst , since you mentioned about documentation, here are some notes of mine on how the plugin behaves in different scenarios. Since it's written from a user's perspective, I thought they or part of them might be of value to the docs. Let me know if there's anything that I can help with this regard. Cheers.
Cases Observed with iOS using phonegap-plugin-push

When a notification is received, regardless whether or not the user has granted your app the permission for Notifications

With your app in the foreground

"notification" event handler is called with data.additionalData.foreground=true and data.additionalData.coldStart=false

With your app is in background AND content-available set to 1

"notification" event handler is called with data.additionalData.foreground=false and data.additionalData.coldStart=false

User's permission for Notifications affects notifications from your app only in the following ways:

Whether to display the notifications in Notification Center
Whether to display the notifications on Lock Screen
Whether to alert upon receiving the notifications
Whether to show app icon badge

When there's some notification for your app in Notification Centre

Tap on your app icon (regardless whether or not the app is in the background)

no "notification" event

Tap on the push notification

With your app in the background

"notification" event with data.additionalData.foreground=false and data.additionalData.coldStart=false

With your app closed (ie. Not even in the background)

"notification" event with data.additionalData.foreground=false and data.additionalData.coldStart=true

Notification with content-available=1 will potentially trigger the notification event twice

With assigning a unique identifier to each notification, use localStorage to record down the notification that has been processed (1st time) and this will be used to avoid repeating the same processing on the same notification (2nd time).

@macdonst I am getting the same issue which is very random however I did not understand where I would use localstorage and do the check in the background.

@jasonatt - Here you go. Hope it makes things clearer for you.
if (data.additionalData.foreground) {
  /**
   * This block is reached when a push notification is received when the app is in foreground.
   * Depending on the design of your app, you might want to save the incoming data to the localstorage here.
   * That's not really what your question is about, but I thought mentioning it would give a better picture.
   */
}
else {
  if (data.additionalData.coldstart) {
    /**
     * This block is reached when a push notification is tapped on *AND*
     * the app is closed (ie. Not even in background).
     */
  }
  else {
    /**
     * The following block is reached in the following two scenarios:
     * 1. A push notification is tapped on *AND* the app is in background.
     * 2. A push notification with "content-available=1" is received while the app is in background. (iOS only)
     *
     * Therefore, for iOS devices, both of the above two scenarios can occur for an individual push notification.
     *
     * For Android, it's always the first scenario as the other one is not applicable.
     */

    // Here we check for the existence of the incoming data in the localstorage and, with that, we determine
    // which scenario of the two possible is happening.
    var pastPushSavedID = window.localStorage.getItem("pastPushSavedID");

    if (pastPushSavedID) {
      // `pastPushSavedID` is found in localStorage and thus this is scenario 1 mentioned above.
    }
    else {
      // `pastPushSavedID` is `null` and thus this is scenario 2 mentioned above.
    }
  }
}

@tamakisquare thank you for the detailed reply mush obliged.
Just to confirm that goes in the  push.on notification correct?

@jasonatt Yes, the whole thing should go in the handling function for push.on('notification'). I am not sure what you mean by "notification areas". Do you mean the notification center at the os-level for unread push notifications? Or are you referring to the notification area, assuming there is one, within the app you are developing?

@tamakisquare yes os level. If I am in the foreground with your code I can detect if the push was already received by checking the id and I can take action from there however this is not working in the background as I still get the notification with sound and alert (os level)

@jasonatt If I understand you correctly, what you are saying is that you want your app to be able to interfere with how device handles push notification. As far as I know, that's not possible because it's beyond the boundary of what an app can do. Push notification behaviour can only be controlled by its sender, via APNs or FCM or any other push notification services, and the device local settings.
As for the "notification" event being fired twice when your app is in the foreground, I have never got that issue with iOS 9 nor Android 4.4.3.

@tamakisquare
Thanks for the update.
Yes I know it was not possible was just checking if this got changed :)
To be clear the issue is not from the plugin as after some debugging I noticed that the server was actually sending them double :/
Thanks again!
Thanks


How Integrate with sailsjs

I'm not familiar with sailsjs, but if you figure it out, please, let us know so we can add it to the docs :)


Fix permission denied in download function

Fixes issue #18 caused by trying to move the temporary file that is still open

It's not working for me i had the same issue everytime i try to download a file


Invalid "File Not Found" Error

The code below runs without issue, but the IDE gives me this error:
── FILE NOT FOUND in Lib/Lib/A.roc ─────────────────────────────────────────────

I am looking for this file, but it's not there:

    /Users/francois/SoftwareProjects/Roc/RocBug/Lib/Lib/A.roc

Is the file supposed to be there? Maybe there is a typo in the file
name?

Directory structure
RocBug/
├── Lib/
│   ├── A.roc
│   └── B.roc
└── main.roc

main.roc
# main.roc
app [main] {
    pf: platform "https://github.com/roc-lang/basic-cli/releases/download/0.10.0/vNe6s9hWzoTZtFmNkvEICPErI9ptji_ySjicO6CkucY.tar.br",
}

import pf.Stdout
import Lib.B

main =

    Stdout.line Lib.B.second

A.roc
module [first]

first = "Hello"

B.roc
module [second]

import Lib.A

second = Str.joinWith [Lib.A.first, "World"] ", "

Thanks for taking the time and I apologize for the missing details.  I'm on an Apple M1 running macOS 14.5 and VSCode 1.89.1 with unofficial Roc extension .  I just upgraded to
roc nightly pre-release, built from commit 33075285b72 on Wed May 22 09:12:12 UTC 2024
and I'm still having the problem.

I think this is a known issue, and I think this is something @agu-z mentioned that he would start on after module params.
We need some way to be able to traverse back up the folder structure until we find a main.roc (platform/package) so we can know what the import structure is.

Resolved as of roc nightly pre-release, built from commit 4d4776784ad on Tue Jun 11 09:12:11 UTC 2024.  Thanks again.


build: add version stamp to release output

Adds a version stamp to the package.json in the release output. The version stamp consists of the current HEAD commit sha, and the branch name.

Closes #12407

@jelbourn Addressed your feedback. We could probably need a spawnSync utility function with cwd set to the project dir.. but not sure how much benefit we'll have from it. I like being explicit here.


Store transaction log snapshots in Arrow format

Background
We initially stored transaction log snapshots in Parquet format:

https://github.com/gchq/sleeper/issues/2270
https://github.com/gchq/sleeper/issues/2122

Description
We'd like to consider whether we should store snapshots in Arrow format instead, if that would improve the speed of saving/loading a snapshot.
Analysis
https://arrow.apache.org/docs/format/Columnar.html#ipc-streaming-format

This is duplicated and replaced by:

https://github.com/gchq/sleeper/issues/2574


Close method is unsafe.

If Close method is called more than one time then the second call could close an invalid or in use file descriptor.
I think this method must check fd.closed flag:
func (fd *FD) Close() error {
	// Take both locks, to exclude read and write operations from
	// accessing a closed sysfd.
	if err := fd.Lock(); err != nil {
		debugf("FD %03d: CL: Closed", fd.id)
 		return err
 	}
 	defer fd.Unlock()
+	if fd.closed {
+		return ErrClosed
+	}
 	fd.closed = true
        ...

Thanks.

No it is not (unsafe).
The suggested check is not required, since it is already preformed inside fd.Lock().

Ops! You are right. ;)


Feature request: Visualize JSON on selection

Feature
Thanks for a great extension.
When working with JSON embedded in other file types (like say Javascript), it would be nice to be able to select a JSON object and visualize with JSON Crack via right-click context menu or command pallette.
Similarly, when navigating a really complex JSON structure, it would be nice to visualize just the selected content.
Alternative solutions or implementations
Copy and pasting the JSON into a new file doesn't really work because the JSON Crack view is not available if the file is unsaved.
Other context
No response

I want to work on this, can you assign this to me


Add copypaste.html to web_accessible_resources

The copypaste.html not being in the web_accessible_resources array for
the chrome extension manifest was causing redirects to it through the
Google proxy to fail to load.  This caused clicking on the link from in
a hangout message to just endlessly load.
Fixes #1065
Tested by clicking on a copy+paste link from hangouts

:+1:


Add aspectRatio in mbr

In feret we have aspectRatio.
The same property should exist in mbr (mbr.height / mbr.width).

Duplicate https://github.com/image-js/image-js-typescript/issues/251


update app swarm for amp-agent appcelerator/amp#15

appcelerator/amp#15
add amp-agent
remove amp-docker-agent and amp-service-log
add kafka-manager and kibana in amp 'min' to make kafka and log tests easier

tested locally before merge

Not sure we really need kafkamanager in our set of min services, but otherwise LGTM.


Ensure release automation for security-partners works in .NET 8.0/VMR

The recent change to introduce CI needs to be updates to work in the VMR.  There may be other changes needed with release automation.

I saw the triage recording but I don't quite understand this task. The linked pipeline seems to only build the tarball and I thought we have agreed that we don't need that as the VMR is validated in different places.
Or maybe I misunderstand what "release automation" means? Is this issue about mirroring the VMR too?

I have also seen this one - https://github.com/dotnet/source-build/pull/3187 - should have this one been linked instead?

Or is this about making sure the whole release checklist is applicable for the new VMR?

@premun - I link the wrong PR in the original description.  I updated the description.  I mean to link to the mirror pipeline.

I think I have the code ready but to run it, I'd need to wait for the branch snap and some release/8.0 build
https://github.com/dotnet/source-build/compare/prvysoky/net-8.0?expand=1

I opened a PR to bring 8.0 up to par with 6.0/7.0
Next steps
We could make the pipeline multi-stage and compose it of the various pipelines that need to be invoked separately. The stages could be divided by approval gates. So something like:

Get info stage - Current pre-release pipeline
Approval gate - Confirmation that smoke test prereqs are in place
Mirror stage - Current mirroring pipeline
Approval gate - Wait for .NET 8.0 release
Release stage - Current release pipeline

We could then have 1 pipeline run for each release of each version (so a 6.0 run, 7.0 run, 8.0 run).
Only thing that is unclear is whether we have all the information in the beginning of the process. I found only one possible problem in a step that I don't fully understand: https://github.com/dotnet/source-build/blob/f93822b50a82dda6cc00bf545280c0a3f6f877a4/.github/ISSUE_TEMPLATE/releases/release_checklist.md?plain=1#L47
If I can get clarity on when this tag is created, we could answer this. The reason is that we cannot input further information once we kick off the pipeline. So it depends whether this tag is known in the beginning.

Okay, closing this one as we got the preview out the door. Fallouts and follow-ups will be handled in different issues


Minor whitespace fixes to ci config

#279  👈
main

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

Join @maurelian and the rest of your teammates on  Graphite


Add external repositories for some third_party/go packages [phab:D848]

Created by schroederc at 2016-04-21 00:22:26:
A future commit will delete the newly unused sources in
third_party/go/src.

zarko accepted this revision at 2016-04-21 00:42:09

schroederc closed via commit 7200fd8bfa9f1c7b64f8e73b4d36685c8d0ad396 at 2016-04-21 01:14:23


Does not register all notes when looping

When trying to loop for x in a NBS file, it stops about halfway through. First song I tried is 218 notes, and the script ended at 93. Second one was 774, ended at 441
Example code:
import pynbs
currentnote = 0
song = pynbs.read("example.nbs")
for x in song:
  print(song.notes[currentnote])
  print(currentnote)
  currentnote += 1

this was a me issue, apologies.


Feat:adding a new page to display all contributors✨

I want to add a page to showcase all the contributors of this website. @DeadmanAbir can you assign me this issue,

I want to add a page to showcase all the contributors of this website. @DeadmanAbir can you assign me this issue,

By adding a page do you mean adding a new page on the website or editing the README file and showing the contributors there?


[DRAFT]- Elizabeth/cord poc

SUMMARY

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

Codecov Report
Patch coverage: 65.94% and project coverage change: -10.39% :warning:

Comparison is base (3630d68) 68.89% compared to head (7ce6542) 58.50%.
Report is 77 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff             @@
##           master     #605       +/-   ##
===========================================
- Coverage   68.89%   58.50%   -10.39%
===========================================
  Files        1906     1909        +3
  Lines       74628    74762      +134
  Branches     8208     8206        -2
===========================================
- Hits        51412    43742     -7670
- Misses      21086    28891     +7805
+ Partials     2130     2129        -1

Flag
Coverage Δ

hive
53.82% <65.94%> (+0.04%)
:arrow_up:

mysql
?

postgres
?

presto
53.72% <65.94%> (+0.04%)
:arrow_up:

python
61.29% <65.94%> (-21.59%)
:arrow_down:

sqlite
?

unit
55.12% <65.94%> (+0.04%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Changed
Coverage Δ

...dashboard/components/SliceHeaderControls/index.tsx
71.57% <ø> (+1.17%)
:arrow_up:

...rontend/src/dashboard/containers/DashboardPage.tsx
8.69% <ø> (ø)

superset/message_threads/commands/create.py
44.00% <44.00%> (ø)

superset/dashboards/api.py
50.61% <47.05%> (-42.00%)
:arrow_down:

superset/daos/thread.py
70.73% <70.73%> (ø)

superset/message_threads/models.py
72.22% <72.22%> (ø)

superset/message_threads/api.py
80.00% <80.00%> (ø)

superset/initialization/__init__.py
91.15% <100.00%> (+0.05%)
:arrow_up:

... and 293 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Fix DPP/DFP cases when join keys are rewritten when there are multiple integral columns [databricks]

Fixes #7707.
This PR handles the case when join keys are rewritten in Apache Spark when there are multiple integral columns that can fit inside a Long/BigInt column. It uses the same method HashJoin.extractKeyExprAt() that Spark uses in those cases but uses the logic of just checking that the build keys can be rewritten (since we don't have the LongHashedRelation that Apache Spark has). Spark will always rewrite the build keys if it can to use a hash join (this behavior is actually not controlled behind a configuration flag), so the need for this PR. This behavior is present in all versions of Spark from 3.1.1 (including Databricks), so no need for shim updates.

Scalastyle check fails
error file=/home/runner/work/spark-rapids/spark-rapids/sql-plugin/src/main/scala/org/apache/spark/sql/rapids/execution/GpuHashJoin.scala message=Use Javadoc style indentation for multiline comments line=230 column=0

build


Error with code generation

Error generating code. Check the Developer Console AND the backend logs for details.
Connecting to backend @  ws://<IP_ADDRESS>:7001/generate-code
generateCode.ts:26 WebSocket connection to 'ws://<IP_ADDRESS>:7001/generate-code' failed:

Closing due to inactivity


Python

[Enter feedback here]
Not seeing any reasonable documentation for promoting python packages from azure dev/or github.

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

ID: 210908e8-c062-c323-8463-fcace04ec775
Version Independent ID: c5d9920b-15d1-ba28-b6e0-121977f4164f
Content: Migrate your packages to Azure Artifacts - Azure Artifacts
Content Source: docs/artifacts/tutorials/migrate-packages.md
Product: devops
Technology: devops-artifacts
GitHub Login: @ramiMSFT
Microsoft Alias: rabououn

@swei7917 Thank you for your contribution. Please see Promoting your packages to a view. You can also checkout these other Python resources:

Get started with Python packages in Azure Artifacts

Publish Python packages in Azure Pipelines


Configuration not propagated to compiler defines

I'm trying to disable the std::format support when compiling with Clang 17 in C++23 mode (i.e. std::format is definitely supported). I have been trying multiple things for hours now, and can't find any way to make this work.
Here's (part of) my conanfile.py:
  def configure(self):
        self.options["mp-units"].std_format = False

   def requirements(self):
        self.requires("mp-units/2.2.0@mpusz/testing")

And part of the CMakeLists.txt file:
find_package(mp-units REQUIRED)

target_link_libraries(${PROJECT_NAME}
  PUBLIC
    mp-units::mp-units
)

No matter what I try, MP_UNITS_API_STD_FORMAT doesn't get defined when compiling. This then causes MP_UNITS_STD_FMT to be set to std, instead of fmt which is what I want. I tested this by adding some extra preprocessor checks/prints to src/core/include/mp-units/compat_macros.h
I tried setting MP_UNITS_API_STD_FORMAT via my project's CMakeLists.txt file to "FALSE", or OFF, but neither does the trick. I'm very confused, because it seems the set_feature_flag() function in src/core/CMakeLists.txt should define MP_UNITS_API_STD_FORMAT. And as far as I understand, I shouldn't need to do this anyway, because mp-units' conanfile.py sets these variables in its generate() function.
Although I'm a bit confused that the manual states that MP_UNITS_API_STD_FORMAT should be set to "AUTO"/"ON"/"OFF", but then the set_feature_flag() checks that it's one of "TRUE" or "FALSE". But again, this shouldn't matter when using conan, because it will set it to either "FALSE" or "TRUE".
I hope I'm just making a silly mistake, because I have no clue what I'm doing wrong and what else to try.

Hi @AnthonyVH, I am not sure if the syntax self.options["mp-units"] is still valid in Conan 2.0? Maybe you should do:
self.dependencies["mp-units"].options.std_format = False

?
Please also note that as described here, setting std_format to False switches to fmtlib usage. Text formatting is still there. If your intent is to get rid of all the text formatting because, for example, you work with bare metal hardware, then freestanding option should be used.
BTW, Clang 17 does not provide __cpp_lib_format as it is not fully supported by this compiler.

the manual states that MP_UNITS_API_STD_FORMAT should be set to "AUTO"/"ON"/"OFF", but then the set_feature_flag() checks that it's one of "TRUE" or "FALSE"

You are right. This is a bug on my side. I have to think about how to fix it.

I'm not trying to get rid of text formatting. We're using fmt in our project, so that's what I want mp-units to use as well.

self.dependencies["mp-units"].options.std_format = False

The conan manual states that self.dependencies is not accessible from configure().
The thing is that the following "kind of" seems to work, because the default ("gsl-lite") causes compile errors with the project I'm trying to use mp-units with. I say "kind of", because I can't find any -DMP_UNITS_API_CONTACTS=3 anywhere in the compile_commands.json after running the build (nor do I see it in the terminal).
self.options["mp-units"].contracts = "ms-gsl"

I also found this which seems to indicate I'm using the right way to configure the option: https://github.com/conan-io/conan/pull/13467. I should note that I find the whole conan documentation pretty terrible though. A lot of things don't seem to be explained very well if at all.

Hmmm, strange. compile_commands.json contains all of the compilation flags for me.

I do not think that you should set the options of dependencies in the Conan configure() function. At this stage, the dependencies graph is not yet established correctly. According to Conan docs, you should set the options either from the profile or command line or by setting the default_options https://docs.conan.io/2/reference/conanfile/attributes.html#default-options, but the last one has some issues.

I'll dig more into this later this week.
I'm wondering about something that seem related though. All the examples depend on mp-units::mp-units, which sets all the required defines in src/core/CMakeLists.txt. However, all those defines are defined once again for each example executable (see example/CMakeLists.txt). AFAIK those should be set due to the mp-units::mp-units dependency. Why is this redefinition required?

The definitions in the example directory apply only to example_utils, which are not dependent on mp-units::core. However, thanks for pointing this out. Initially, it was just one innocent find_package(gsl-lite), but it grew recently. Now it is a lot of copy-paste code. I will try to refactor this to one CMake target and link with it in both places.

I've tried a bunch more things, but can't get this to work.
Below is a minimal example that reproduces the issue for me. The std_format option is not passed to mp-units (or if it turns out it is, nothing is done with it). Passing it via the command line (-o 'mp-units/*:std_format=False) doesn't work either. The compiler spews out a bunch of errors about v1 not being formattable and MP_UNITS_API_STD_FORMAT is not defined at the point mp-units/compat_macros.h is included.
Is there something obvious missing here?
-- conanfile.py
from conan import ConanFile
from conan.tools.cmake import CMake, CMakeDeps, CMakeToolchain, cmake_layout
from conan.tools.env import VirtualBuildEnv

required_conan_version = ">=1.59.0"

class HelloConan(ConanFile):
    name = "hello"
    settings = "os", "compiler", "build_type", "arch"
    build_policy = "missing"

    options = { "build_folder": ["ANY"] }
    default_options = { "build_folder": "build" }

    def layout(self):
        cmake_layout(self, build_folder=self.options.build_folder.value)

    def configure(self):
        self.options["mp-units"].std_format = False

    def requirements(self):
        self.requires("fmt/10.2.1")
        self.requires("mp-units/2.2.0@mpusz/testing")

    def generate(self):
        env = VirtualBuildEnv(self)
        env.generate()

        tc = CMakeToolchain(self, generator="Ninja")
        tc.generate()

        deps = CMakeDeps(self)
        deps.generate()

    def build(self):
        cmake = CMake(self)
        cmake.configure()
        cmake.build()

# CMakeLists.txt
cmake_minimum_required(VERSION 3.12)

project(conan-mp-units LANGUAGES CXX VERSION 1.0.0)

set(CMAKE_CXX_STANDARD 23)
set(CMAKE_CXX_STANDARD_REQUIRED TRUE)
set(CMAKE_CXX_EXTENSIONS FALSE)

set(CMAKE_EXPORT_COMPILE_COMMANDS ON)

find_package(fmt REQUIRED)
find_package(mp-units REQUIRED)

add_executable(${PROJECT_NAME} main.cpp)
target_link_libraries(${PROJECT_NAME} PUBLIC fmt::fmt mp-units::mp-units)

// main.cpp
#include <mp-units/format.h>
#include <mp-units/systems/si.h>
#include <fmt/format.h>
#include <iostream>

int main() {
  using namespace mp_units;
  using namespace mp_units::si::unit_symbols;
  constexpr quantity v1 = 110 * km / h;
  std::cout << fmt::format("{}", v1) << '\n';
}

You are right. There is a bug in package_info() Conan method. I am really not the biggest function of this Conan functionality :-(
I should fix it soon.

I hope it fixes your problem. If not, please feel encouraged to reopen.

There are some bugs in the CI. I will fix it in a few minutes.


Wip/6.0 Fix ToOne @Fetch(FetchMode.SELECT)

Querying the parent a select was executed to select the parent but not a second one for the association annotated with @Fetch(FetchMode.SELECT), the issue was that a EntityFetchDelayedInitializerwas created for the association instead of aEntityInitializerSelectFetch because the fetchTimingwas equal toFetchTiming.DELAYED`

@sebersole I agree with you about exploring a bit more the "subtlety" as I realised that it is not completely clear to me how the fetch mode should influence the fetch timing.

Added more tests and fixed the code

Thanks @sebersole and @NathanQingyangXu , applied upstream


Clickable Coordinates

Currently when you locate a biome through terra, the coordinates are white / non clickable.
Normal /locate coordinates are both highlighted green and clickable, actioning as /tp @s ~ ~ ~.
Would be a nice touch to add :)
EDIT: probably worth mentioning I'm on Fabric.

I know this sounds like it should be pushed to the back of the list but this would be super useful for newcomers and old users of this mod

Not exactly the same but you can always add -t to the end of the locate to automatically teleport.

We no longer have commands which would require clickable coordinates (we use Vanilla's /locate and /locatebiome now), and if an addon wishes to implement one they can use Adventure with cloud.


prevent webpack config from loading from node module cache

The require-new module prevents the webpack config from being cached. There could be a performance hit if a user's webpack config is complex but that risk is low. Perhaps there should be a warning in the package settings with the enabling checkbox? Or maybe the user should have the responsibility to be aware of the implications of enabling support for this? I feel like there should be more error handling and notification. Right now errors are caught and allowed to silently fail. I'll look into adding a toast notification in the catch blocks later. I'm open to any thoughts on this.

I removed the redundant check and early return.

Hey, just cleaning up some old PR's. Is this a viable solution to the caching problem?

Closing due to inactivity.


Add DELETE query support

Overview
This commit adds query language support for deleting series with a DELETE query.
Fixes #1647
/cc @jwilder
Required for all non-trivial PRs

[x] Rebased/mergable
[x] Tests pass
[x] CHANGELOG.md updated
[x] Sign CLA (if not already signed)

Required only if applicable

[x] InfluxQL Spec updated

+1

Needs a CHANGELOG update.

@jwilder I forgot to push the CHANGELOG update. Fixed and pushed up.

👍

OMGYES!
@benbjohnson any initial thoughts on how performant this is? Is it something a user could run during normal operations or would it be advisable only against a quiescent database?
What's the difference between DELETE SERIES cpu vs. DROP SERIES cpu, neither with WHERE conditions. In the first case the series is still in the index and in the second it's not?

@beckettsean I'm not sure. @jwilder did the tsm1 implementation. He could probably speak on the performance characteristics.

@beckettsean I'm still testing performance.  Should work like drop series.
The difference between DELETE SERIES and DROP SERIES is that DELETE SERIES does not remove the series from the index so SHOW SERIES will still list it even if you deleted all data.
This behavior is a little odd after playing with this because, if you restart the server, the series are gone.  @pauldix is there reason why DELETE SERIES should not remove the series from the index if the series data is fully removed?

Thanks for adding this.


Improve testing

Improve testing. Resolves #475.

[x] Restructure directories on LCRC (in /lcrc/group/e3sm/public_html/e3sm_diags_test_data/) and their corresponding references in E3SM Diags.
[x] Update documentation on how to update expected images for the tests besides complete_run.py.
[x] Refactor test directory structure: move integration tests into their own directory
[x] Refactor test directory structure: remove the need for a system subdirectory?

Not doing:

Refactor test directory structure: move unit tests (tests/acme_diags) to top level tests directory

After merging:

[ ] Delete system_test, unit_test_data and unit_test_images in /lcrc/group/e3sm/public_html/e3sm_diags_test_data/

@tomvothecoder This is ready for review. In particular, let me know if anything should be renamed or if there should be any directory structure changes. I've run all unit tests and integration tests successfully.
On the LCRC side: /lcrc/group/e3sm/public_html/e3sm_diags_test_data/ currently has subdirectories system_test, unit_test_data, and unit_test_images, which can be deleted after this pull request is merged. With this pull request, there is the subdirectory integration which contains integration_test_data and expected. The latter is set up much like /lcrc/group/e3sm/public_html/e3sm_diags_test_data/unit_test_complete_run/expected/.
On the E3SM Diags repo side: tests/system has been removed. Now, we have tests/acme_diags for unit tests (as before), testscomplete_run.py for the complete_run test (as before), and tests/integration. Any tests that requires downloaded data is located in this last directory.
Documentation has also been updated to reflect how to update expected images. This closely parallels how it is done for the complete_run test.


Broken voiceforge voices

Can you please fix the voiceforge voices quicker?

i'm not fixing them at all
just use the android app

Oh, sorry for bothering you.

From: octanuary @.>
Sent: Sunday, February 12, 2023 6:31 PM
To: Wrapper-Offline/Wrapper-Offline @.>
Cc: BencorpProductions @.>; Author @.>
Subject: Re: [Wrapper-Offline/Wrapper-Offline] Broken voiceforge voices (Issue #136)
i'm not fixing them at all
just use the android app
—
Reply to this email directly, view it on GitHubhttps://github.com/Wrapper-Offline/Wrapper-Offline/issues/136#issuecomment-1427240457, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A52ZDDMIBOA3HB6EPAFAUXDWXGMI5ANCNFSM6AAAAAAUZW3UAU.
You are receiving this because you authored the thread.Message ID: @.***>


Peer Review

Great work! I think the research questions are really interesting and the use of mapping to create multiple models and plots was great. The main comments I made in the script have to do with more granularly narrating why each step is being taken, there were a few places where I had to stop and think about what was being done and why. I also think it would be good to write a bit about some of the plots you create at the very end -- those are some really interesting findings and probably could benefit from some unpacking.

@datalorax

Nice review, Ian!


Not working in iphone11 and iphone xs max

Can't work properly on iphone11 and iphone xs max web pages。Nothing happens after shaking the iPhone, how can I do it ?

See https://developer.apple.com/documentation/safari_release_notes/safari_13_release_notes#3314664
And https://dev.to/li/how-to-requestpermission-for-devicemotion-and-deviceorientation-events-in-ios-13-46g2


Add automated tests for executing tasks excluding Export, Upload, and Custom Task

More execution tests for #10:

Expanded Copy Files
Clear Directory Files
Pack ZIP
Sub-Routine
Custom command in Bash
Custom command in Windows

I made a few refactoring changes to the project outside of tests:

I changed referencing Data.current_routine to rely on it not being reset across execution, and then clearing it only in Execution._exit_tree(). This allows Execution to run without it being the highest-level scene (for sub-routines), since GutRunner is the highest-level scene during tests. It also makes things a little cleaner.
I emitted a finished signal in Execution.finish(). This helps to check against stalling in tests and to make tests faster and more reliable.

Notes on tests:

The expanded Copy Files test found that copying a directory with Recursive=False does it recursively anyway, at least for me.
I didn't actually run the Windows custom command test.
Custom command tests rely on the working directory being the Project Builder project being tested. They won't work correctly otherwise. I think I can make this assumption, but if not then I don't think it would be worth keeping them around in this form.

I ran the tests. 2 seem to be failing (test_copy_files and test_windows_custom_task), also in one test there was an error due to finish_mutext being null.
I think the last one was in test_windows_custom_task. I'm getting Could not create child process: copy Tests/Projects/TestProject1/File1.txt Tests/Projects/TestProject1/WindowsFile.txt error. There are 2 problems here:

CustomTask can't execute tasks that aren't specified by full path (on Windows at least); I think I ran into this myself recently. Needs more testing, might be engine bug.
CustomTask does not get (pre)validated properly and fails at execution.

I changed referencing Data.current_routine to rely on it not being reset across execution, and then clearing it only in Execution._exit_tree().

Yeah the previous "solution" was a really bad hack. I was thinking about reworking it too 😅 The idea behind that was to be 1000% sure that the current routine is reset properly.

The expanded Copy Files test found that copying a directory with Recursive=False does it recursively anyway, at least for me.

https://github.com/KoBeWi/Godot-Project-Builds/blob/b87bc62a657c53be20ba9283a8c0b98ee623b496/Tasks/CopyFiles.tscn#L47-L51
This should be checking for recursive.button_pressed 🤦♂
Feel free to fix that.

I removed the custom command tests. They look like a maintainance hassle and I'm not completely sure what I'm doing anyway.
I improved PackZIP tests to test globbing and some other things. It was weird to me that the filters work only per file and don't allow something like SubDir/*, but I'm guessing this is intentional? If not, I should change the Exclude test some more.

It was weird to me that the filters work only per file and don't allow something like SubDir/*, but I'm guessing this is intentional?

Well it's more or less intentional, I'm only matching the filename, not whole path. I think it's consistent with how Godot uses export filters.
Can always be changed if that's wanted of course.

After pulling latest changes, the tests are failing for me

It looks like some operations are slower in Godot v4.4-dev4. I think that's what caused the tests to fail.
I increased the waiting time for execution to finish from 5 to 20 seconds to be on the safe side. The longest test for me in 4.4-dev4 was 7.5 seconds.
It once happened to me that I had to wait the full 20 seconds even though execution apparently finished. That's annoying but I think it catches a bug that Execution.finish() is not always called when execution is done, so I guess it's fine to not hide that bug. I couldn't reproduce it.

That's annoying but I think it catches a bug that Execution.finish() is not always called when execution is done, so I guess it's fine to not hide that bug.

This might be engine bug. I noticed the Project Builder hangs when it's supposed to finish a task.
The tests run fine in 4.3 (although they failed at my first attempt, idk what happened).

Thanks!

Actually the hang issue was caused by wrong use of threading. I previously made it work with a hack, but it stopped working. e2afa20c0e140c0e813f8b51102f06487c19856e makes it handled properly.
The one failed attempt test I mentioned was because I stopped the test mid-execution and it didn't do proper cleanup. Everything works fine.


Add RSSI to device data model

@brahma-dev build is failing please check

LGTM, great contribution, thanks!


AnimationPlayer timeline scrollbar not functioning properly.

I'm using Windows 64 version of v2.1.1
In the AnimationPlayer, whenever I drag on the timeline's slider, it will only jump from end to end.  It also does some strange jumps when just clicking on the empty areas of the bar.  Is there some sort of option that I mistakenly set, or is this a bug?
Here is what it looks like when I drag from side to side.

Looks weird -- is it also present in 2.1(.0)?

No it works fine in 2.1.0

I confirmed it happens with current master head.

Might be related to the PR that changed step usage for sliders?

@akien-mga probably yeah...


Correctly construct filter query string for api

Updating the api in
https://github.com/prometheus/alertmanager/commit/bc373f562fcd4a765d59c6b44e8dfca4bcd2fe4c
resulted in the UI sending query strings for the
filter that no longer match what the backend
expects. This brings that in to line.
fixes https://github.com/prometheus/alertmanager/pull/1868
Question:
The filter query string that is visible to users does not match the filter string format expected by the server now that https://github.com/prometheus/alertmanager/commit/bc373f562fcd4a765d59c6b44e8dfca4bcd2fe4c has been merged. I've opted to keep the user-facing format the same in the event of users having bookmarked certain URLs with filters in them. I would like to do this for now and we can discuss what we want to do about that in the future, but this is up for discussion. While it's good to keep the user facing URL the same, in my opinion, it does lead to a more or less duplicated function for generating the query string. Maybe that's the lesser of two evils.
This also solves https://github.com/prometheus/alertmanager/issues/1731

LGTM after testing though I've got no clue whether the code is ok or not...

going to go ahead and merge.
@mxinden have you had a chance to look at generating the elm client from our openapi spec? I remember at one time there were issues with it hardcoding the URL or something

@mxinden have you had a chance to look at generating the elm client from our openapi spec? I remember at one time there were issues with it hardcoding the URL or something

@stuartnelson3 I tried back when we moved the UI to API v2. I remember not being able to configure the backend without changing openapi.yaml. As of today, the UI only uses the generated model, which in my eyes is the most important part.


⚠ Bofrost.de has degraded performance

In d47ce1e, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1875 ms

Resolved: Bofrost.de performance has improved in 4983124 after 21 minutes.


Sync of GitHub repositories within 2024.11.18-24

The aim is to synchronize GitHub repositories in the given period.

Repositories are synchronised. Goal is achieved.


Use more modern ImGui input API

The main reasoning behind this is that the old API doesn't work well with the mouse position when the window isn't in full screen, causing desyncs between the actual mouse position and the position ImGui thinks where the mouse is.

Ok re: our Discord discussion, I didn't realise you had solved the problem by actually rewriting the input system I was originally complaining about. 🤦
I'll take a look at this and maybe update imgui in the process (or in another PR).  Note that an imgui update will be a breaking change for the like 2 mods that use it.

Code looks fine, will look at doing a release this weekend.  I'll probably need to coordinate with Ja so that MappingUtils works immediately.


CI: Move android_build workflow to Engflow's remote execution

Signed-off-by: Luis Fernando Pino Duque<EMAIL_ADDRESS>Description:
CI: Move android_build workflow to Engflow's remote execution and cut build times by 5x.
With remote execution, a couple of genrules need to be marked as local since they need access to local artifacts.
Before (~1h30min)

After (~18min):

Risk Level: Low
Testing: See android_build
Docs Changes: N/A
Release Notes: N/A

/cc @ulfjack @goaway


💚 Fix Unit Test

We had the error TypeError: Manager.need_new_runner() missing 1 required positional argument: 'manager', on closer inspection we realized that need_new_runner() needed 2 arguments, the first one referring to self and the second one referring to manager.
To solve this error, we have to call the method need_new_runner() with r - the instance of manager - and r itself.


Aws mitre t1119

Background
Adding ODIN MITRE detection for  T1119 Automated Collection.   I took a look at actions that could be used to exfiltrate data collections from: object level, bucket level, IAM level events as well as any command that could execute a script.  I set the dedup to 5 action with 1 hr.

closing since it is duplicate of #642


Unhandled exception in script

I force-quit the program while running Buzz once, and this happens when I double-click to run Buzz again. I uninstalled and reinstalled Buzzx and still have this problem. How do I deal with this?

And my computer has Windows 10 version 1607 and OS version 14393.5850.

Duplicate of #406


Gradient norm is zero for training Qwen2.5-0.5B-Instruct in unsloth=="2024.11.6"

Hi,
I encountered an issue after updating to unsloth=="2024.11.6". When training the Qwen2.5-0.5B-Instruct model without PEFT, I observed that the model's gradient norm is 0, resulting in no weight updates.
I noticed a discrepancy in the number of trainable parameters:

unsloth=="2024.11.6": 357,898,112 parameters
unsloth=="2024.10.7": 494,032,768 parameters (works correctly)

This difference in trainable parameters might be related to the training issue.

Oh wait without PEFT? Hmm would it be possible for you to use with torch.autograd.set_detect_anomaly(True): trainer.train()


python311Packages.reolink-aio: 0.8.2 -> 0.8.4

Diff: https://github.com/starkillerOG/reolink_aio/compare/refs/tags/0.8.2...0.8.4
Changelog: https://github.com/starkillerOG/reolink_aio/releases/tag/0.8.4
Description of changes

Things done

Built on platform(s)

[ ] 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
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 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

[ ] Fits CONTRIBUTING.md.

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

Result of nixpkgs-review pr 275254 run on x86_64-linux 1

4 packages built:

python310Packages.reolink-aio
python310Packages.reolink-aio.dist
python311Packages.reolink-aio
python311Packages.reolink-aio.dist


Infraestructure Symbols not working

Hello JMSML community
First and foremost, this is an amazing library and I will like to thank you all for your hard work. :)
I found some issues with symbols that were not available, but there were one less that 0,5%.
Unfortunately, I started working with emergencies and I have encounter many symbols that are not available, mostly all the emergency - infrastructure,
EFF-A----------
EFF-DB---------
etc. or the emergency - operations - installations.
EFO-DC---------
EFO-DDC--------
for example.
Perhaps I'm doing something wrong.
Could you gladly help me or explain how can I add them to the library if I have the SVG file pieces?
Thanks and best regards.

Not sure what you are using to test, but if it is the demo app in this repo, then probably just a bug - they look like valid (2525C) symbols. I get same (bad) results.

Hello csmoore,
then how can I proceed?
I do not know if I will be able to debug the code.

@Dash83UPV - I probably would not be able to debug either given the complexity of the datamodel/code - I believe this repo was primarily designed/tested with 2525D - but there are other tested repos/libraries you could use that do fully support 2525C.

Chris:
I haven't been following this discussion.  What's the problem?
Bill McGrane
Chair, SSMC
DISA  BDE4
Standards Management Branch
Comm:<PHONE_NUMBER>
DSN:<PHONE_NUMBER>
-----Original Message-----
From: Chris Moore<EMAIL_ADDRESS>Sent: Friday, February 9, 2018 7:26 AM
To: Esri/joint-military-symbology-xml<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Subject: [Non-DoD Source] Re: [Esri/joint-military-symbology-xml] Infraestructure Symbols not working (#483)
All active links contained in this email were disabled. Please verify the identity of the sender, and confirm the authenticity of all links contained within the message prior to copying and pasting the address to a Web browser.

@Dash83UPV < Caution-https://github.com/dash83upv >  - I probably would not be able to debug either given the complexity of the datamodel/code - I believe this repo was primarily designed/tested with 2525D - but there are other tested repos/libraries you could use that do fully support 2525C.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub < Caution-https://github.com/Esri/joint-military-symbology-xml/issues/483#issuecomment-364420246 > , or mute the thread < Caution-https://github.com/notifications/unsubscribe-auth/AI5Tbe5iLc4pUV1SZzjNDy8eVBfganu-ks5tTDlFgaJpZM4R-rw6 > . Caution-https://github.com/notifications/beacon/AI5TbeXSG4K_oJMhgHREt07vds37q4fHks5tTDlFgaJpZM4R-rw6.gif

@wmcgrane - its just a bug in the test app (screenshot above) / API converting 2525D SIDCs/symbols to 2525C. We used the mapping table provided so we know the data generally works for this - though we did have to make numerous manual changes to complete this mapping.

@Dash83UPV this is a 2525D repository, with some legacy mapping back to 2525C, but it's incomplete (as you found). Look for this code, and see if that's what you want: 20112200.
That said, any updates you want to provide to this repository are welcome, the XML is here: https://github.com/Esri/joint-military-symbology-xml/blob/dev/instance/Land_Installation.xml
As @csmoore said, depending on what you're trying to do, there are other repositories out there. So, you want to use 2525C emergency symbols in what platform? ArcGIS?

Joe:
A concern is the comment that Esri has found some incomplete mappings.  Do you know which ones and how many?  We'd like to be able to update and fix our spreadsheet.
Bill McGrane
Chair, SSMC
DISA  BDE4
Standards Management Branch
Comm:<PHONE_NUMBER>
DSN:<PHONE_NUMBER>
-----Original Message-----
From: Joe Bayles<EMAIL_ADDRESS>Sent: Friday, February 9, 2018 11:58 AM
To: Esri/joint-military-symbology-xml<EMAIL_ADDRESS>Cc: Mcgrane, William M (Bill) CIV DISA BD (US<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: [Non-DoD Source] Re: [Esri/joint-military-symbology-xml] Infraestructure Symbols not working (#483)
All active links contained in this email were disabled. Please verify the identity of the sender, and confirm the authenticity of all links contained within the message prior to copying and pasting the address to a Web browser.

@Dash83UPV < Caution-https://github.com/dash83upv >  this is a 2525D repository, with some legacy mapping back to 2525C, but it's incomplete (as you found). Look for this code, and see if that's what you want: 20112200.
That said, any updates you want to provide to this repository are welcome, the XML is here: Caution-https://github.com/Esri/joint-military-symbology-xml/blob/dev/instance/Land_Installation.xml < Caution-https://github.com/Esri/joint-military-symbology-xml/blob/dev/instance/Land_Installation.xml >
As @csmoore < Caution-https://github.com/csmoore >  said, depending on what you're trying to do, there are other repositories out there. So, you want to use 2525C emergency symbols in what platform? ArcGIS?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub < Caution-https://github.com/Esri/joint-military-symbology-xml/issues/483#issuecomment-364492704 > , or mute the thread < Caution-https://github.com/notifications/unsubscribe-auth/AI5TbUXFrjX13mwkuan6Z3lXXkV0F5Gzks5tTHk0gaJpZM4R-rw6 > . Caution-https://github.com/notifications/beacon/AI5TbQN6wWnI518JKYw5roLG2gUqXWsSks5tTHk0gaJpZM4R-rw6.gif


Fixed "Time until fully vested" display

Summary
Minor fix, but I can assume that after converting all references of TIME to RUG when forking wonderland, you had accidentally replaced Time to Rug. Obviously on the contract the reference for lastRug doesn't exist.
Suggestions
How would you like PR's to be made? Perhaps making a template would be good.

Re-wrote the PR for the dev branch

@GhostWalker562 make sure the branch is dev to staging. This is master to staging...that would be pushing the branches back. If you need some help you can also ping me in discord BabyGravvy

Looks good tho nevertheless, happy to progress this one through


crash when using still mode

tested device is samsung SM-G9500, method set to still and this exception will be thrown.
android.view.ViewRootImpl$CalledFromWrongThreadException: Only the original thread that created a view hierarchy can touch its views.                                                                        at android.view.ViewRootImpl.checkThread(ViewRootImpl.java:7913)                                                                        at android.view.ViewRootImpl.invalidateChildInParent(ViewRootImpl.java:1373)                                                                        at android.view.ViewGroup.invalidateChild(ViewGroup.java:5449)                                                                        at android.view.View.invalidateInternal(View.java:14825)                                                                        at android.view.View.invalidate(View.java:14789)                                                                        at android.view.View.setFlags(View.java:12684)                                                                        at android.view.View.setVisibility(View.java:8673)                                                                        at com.kingyun.hawkeye.camera.CaptureActivity$setupCamera$1.onPictureTaken(CaptureActivity.kt:60)                                                                        at com.flurgle.camerakit.CameraView$CameraListenerMiddleWare.onPictureTaken(CameraView.java:397)                                                                        at com.flurgle.camerakit.Camera1$3$1.onStillProcessed(Camera1.java:227)                                                                        at com.flurgle.camerakit.ProcessStillTask.run(ProcessStillTask.java:47)                                                                        at java.lang.Thread.run(Thread.java:762)

I have the same issue

@xxxifan do you had any alternative, could you plz let me know

onPictureTaken() has been called in async thread, so if you want to do something related to view you need run in main thread


[add] docker: build caching

Motivation for changes:

add caching for faster builds

Detailed changes:

added qemu builder, github caching, metadata provider for tags and labels

@trim21 let me know if this breaks anything for you. I hope I got all the tags correct.

this won't affect downstream users

Seems good to me? Let's check after first release.

develop image was published correctly. next version should be working too then when the other pr's are merged.

looks like it failed on the release build. also caught a caching typo on the dev build. I cleaned it up and will submit another PR.


Missing title field in JSON schema for plans

The title field is missing in plan schemas fields. It's good to have it, especially if someones (like me) is using some libraries for displaying forms from JSON schema. I've used a few solutions for React and Angular and all of them displays title as label. For example, https://github.com/mozilla-services/react-jsonschema-form fallbacks to object key name, so it results in a rather ugly form:

I'm working right now on a pull request that introduces titles for all fields for plans.
Hope you'll like it 🙂 Cheers!

@pkosiec thanks for the issue (and the PR).
I'm open to discussing this change, but still feel a bit reluctant.
My rationale here is that while you may perceive fallback field titles that match parameter keys names (e.g. resourceGroup or sslEnforcement in your example above) as ugly, I actually see them as ideal because they are the true parameter names.
That said...
Day to day, what I (selfishly) care about most is the experience for people working without a GUI. Obviously, this change doesn't really impact those people at all. But... there are some existing GUIs that work with our broker (and others). For instance, OpenShift and KubeApps. Whether it is for better or worse, I'm going to guess that adding title to input parameter fields in the schemas will affect how forms are displayed in those existing products. I'm not keen to change behavior under the feet of existing users.
At the end of the day, as I said, I'm open to discussing this change, but I don't want to rush blindly into merging it.
I'm going to delegate some decision on this to @seanmck.

Hi @krancour, thanks for your answer!
Generally, in my point of view, GUI should be an abstraction for an user. If he uses GUI, then he shouldn't be bothered with parameter names etc. All labels should be clear, concise and in readable form. But I understand your concerns. We have different opinion on this topic and that's absolutely fine.
I just would like to have a choice, how to display those titles. This change is an improvement, because it gives an user ability to display label for a field in a different form.
Adding title field shouldn't break UIs of existing users - if they really mean to use object key names, then it will be fine. It's just an additional field which may be used - but users are not forced into displaying it. In JS library which I am using, it fallbacks to field key name - but it depends on the used solution. Some libraries can simply just don't display titles, when they are not provided. So this change can even improve users' UIs.

@pkosiec suppose the existing GUIs are using the same library as you. (React and Angular are both really popular.) Wouldn't it then be likely that if title is added to each param to make it display differently for you that those other GUIs will also use title by default and no longer fall back on the key as they do today?
Disclaimer: I am not a JS / front-end guy at all. So if the question seems naive, realize that I haven't personally used React or Angular.

I know that one React solution does the fallback. One solution for Angular hasn't displayed a label at all, if the title hasn't been provided. I don't know about others. However, I think this is issue of a developer, that are not explicitly pointing that they want to use key names as field labels.
I've just checked OpenShift console UI and I haven't found any familiar library used for parsing JSON schema, so they are probably doing that on their own (some kind of custom solutions).
They are also not displaying object key names as field labels:

Hi @krancour, any news regarding the issue? Did you discuss it with @seanmck? Thanks!

@krancour Thank you! I've just rebased my PR.


Wiki changes

On the linux build there are some modifications that has be done to the build instructions.
For Ubuntu 16.10 the libboost library is referenced by another name than the one mentioned in wiki
cp shared/libUtyMapLib.so ../../unity/Assets/Plugins/libUtyMap.Shared.so
should be
cp shared/libUtyMapLib.Shared.so ../../unity/Assets/Plugins/libUtyMap.Shared.so
Copy the plugin folder should be from base (not mentioned) to demo directory (wiki is copying to non-existing directory)
Can you enable wiki permissions to all so we can contribute to this too?
Thanks!
Ganesh

Hi Ganesh! Thanks for reporting this. I forgot to update linux wiki after I renamed library. Will fix it.
Regarding contribution, there is ability to checkout wiki pages. I believe that it is possible to create pull request then which I can review before it is applied. I don't want just to enable write access to everyone as it will be difficult to track changes.

Close as answered


Change 'underscore-min.map' file name to 'underscore.min.js.map'

'min.js' is standard extension for map files, some tools do not work with differ extensions.

underscore-min.js.map sounds fine to me.

Sorry, '.min.js' is for minified script, '.min.js.map' is for map.

Then, I don't think so.

some tools do not work with differ extensions

What tools exactly?

Bundle Transformer do not understands -min.js extension (https://bundletransformer.codeplex.com/)
I have a lot of third-party scripts in my project, all of them except Underscore have .min.js extension. I think this is tacit convention.

It's not. In file.min, .min is not a valid file extension.
Open a ticket on Bundle Transformer, instead of here.


Switch to Rust edition 2021

Updated with more clippy lints and various CI build fixes.

The security issues cannot be resolved by this PR.


nixos/ibus: detail workaround for issue #11558

Motivation for this change
Translation of https://github.com/NixOS/nixpkgs/issues/11558#issuecomment-170702789 into the manual.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Fits CONTRIBUTING.md.

cc @Profpatsch

I’d add a note that this should only happen with programs installed via nix-env.

Or in nix-shell or when channels are mixed or cachix is used or when applications from a system profile interferes with a user system profile... I think we can keep it as it is.


Auth - authz

As far as I can see there is not any auth or authz in tarmac.
i generally use NATS for rerouting events into tarmac . Nats just wants a jwt to control Auth  and Authz.
But even without NATS , Tarnac needs to assert who can do what.
https://github.com/pocketbase/pocketbase looks like a nice solution to this.
.Or at least to use as one way to add auth / authz. It’s probable that  others might want a different solution and that’s why jwt is loose coupled.
the cool thing about pocketbaae is that it’s real time and simple.
have a look

Auth N/Z is def something I plan to add. Will look into the best way to implement this.

With the latest release, there is now Mutual TLS as an Auth option (thanks to this issue). BTW would love to hear more about how you are using Tarmac (i.e., is it for fun or actual work, is it a POC or in production, etc.)

Thanks for the addition of m-told.
I am experimenting with using Tarmac to allow scientists to build data processing and charting. It’s for biomimicry and open science platform so that peer review done by both other scientists and the public is reproducible.
The problem it solves is one of transparency. Currently scientists write a paper with detailed explanations of their experiments and thesis, but without the data and logic and charts that they used to come to their conclusions,
I want to help this situation by making it easy to run other scientists code and data. Most scientists work with data in excel and do their charts also with excel.
The reason for auth and authz is so that a scientist can give access or share their code and data with others.

Btw jwt would be perfect for me because I use NATS as the routing system for events and rpc.
nats has a full identity and authz system based on jwt. So I can great as users account in nats or a service account and then use tarmac jwt to authenticate into NATS system.
nats is golang based.
https://docs.nats.io/running-a-nats-service/configuration/securing_nats/auth_intro/jwt

@gedw99 sounds really cool. I will be adding jwt for sure.

Consider NATS for the other layers maybe:
I noticed you have a KV store using cassandra  at https://github.com/madflojo/tarmac/blob/main/pkg/callbacks/kvstore/kvstore.go
Here is a KV store with different drivers, and one of them is NATS: https://github.com/HeavyHorst/easykv
And for durable TLS Certs: https://github.com/HeavyHorst/certmagic-nats
NATS also has an Object store.
The potent thing here is that the NATS security controls access to ALL of this. Doing authz is one thing but authouriation is hard as it needs to be enforced throughout the stack. NATS does it for you and is highly extensible.
https://github.com/bots-garden/capsule/blob/main/capsulemodule/hostfunctions/nats.go is similar to Capsule and exposes NATS directly as  WASM Host function.
Many people also ship Logging and Metrics over NATS, using non durable topic.  SO then even this gets the NATS security overlay.
Anyways, don't want to be pushy pushy..  I use NATS for many different projects as its highly reusable.


Installation fails with fatal PHP-Error with PHP 7.2.2

This is the message in the PHP log:
[16-Feb-2018 23:04:27] WARNING: [pool {hidden}.net] child 22258 said into stderr: "NOTICE: PHP message: PHP Fatal error:  Declaration of BxDolModuleQuery::getInstance() must be compatible with BxDolDb::getInstance($aDbConf = false, &$s Error = NULL) in /var/www/vhosts/{hidden}.net/httpdocs/inc/classes/BxDolModuleQuery.php on line 10"

Thanks @AlexTr for taking this up. I have made a rough and ready fix to get our UNA running - at line 31 of BxDolModuleQuery.php I have simply copied in the previous function definition from BxDolDb:
public static function getInstance($aDbConf = false, &$sError = null)
I'm a bit rusty on PHP so don't know if this might cause problems, but at least UNA now runs - and I must say, the guided tour through the UNA Studio really impressed me. Thanks for this great software!

Hm after successful installation and all running on PHP 7.0 I tried switching back to PHP 7.2. Sadly it does not work, the same problem as noted above, and my amateur hack does not fix it. Sorry )-:

One more question: In PHP 7.0 I have apc compatibility and apcu modules. In PHP 7.2 I only have apcu, not the compatibility module - is it required?

BTW the requirements state short_open_tag = On, but I had it off, worked anyway. Is it really needed or are those outdated requirements?

@timreeves it maybe some problems with 3rd-party apps when this option is off.

https://github.com/unaio/una/issues/1141

One more question: In PHP 7.0 I have apc compatibility and apcu modules. In PHP 7.2 I only have apcu, not the compatibility module - is it required?

@timreeves I've missed your question somehow, you can use different PHP opcode caches, it just your preference.


Add snippet to login telegram via pyrogram

Sorry that I didn't get to you earlier than this.
This snippet would literally just print the message "New login from IP x.x.x.x" you receive when successfully logging in with a new Client. So that isn't really "useful" for anything in my opinion.


patch to fix issue #126 with no alias function

Following discussion in https://github.com/felixge/node-mysql/pull/127, issued a patch which is exactly the same as raszi's, but with no aliased function.
when calling the escape function for the first time, it's called 'binded' to the client instance. that way, the "this" works ok.

I've tested it passes.

let's hope felixge merges it now.

Great job guys, I'll definitely be using this a lot!

I've updated my pull request and it works as expected.

Any chance this will be merged soon?  I'm still unable to use arrays as params.


[HUDI-1218]Introduce BulkInsertSortMode as Independent class

Tips

Thank you very much for contributing to Apache Hudi.
Please review https://hudi.apache.org/contributing.html before opening a pull request.

What is the purpose of the pull request
Introduce BulkInsertSortMode as Independent class
Brief change log
(for example:)

Modify AnnotationLocation checkstyle rule in checkstyle.xml

Verify this pull request
(Please pick either of the following options)
This pull request is a trivial rework / code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end.
Added HoodieClientWriteTest to verify the change.
Manually verified the change by running a job locally.

Committer checklist

[ ] Has a corresponding JIRA in PR title & commit

[ ] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

hi,@yanghua，can u take a look when freeeeeeeee ?


[AVM Module Issue]: Output ID returns the entire resource instead of resource ID

Check for previous/existing GitHub issues

[X] I have checked for previous/existing GitHub issues

Issue Type?
Bug
(Optional) Module Version
0.1.1
(Optional) Correlation Id
No response
Description
Use following code to reproduce
terraform {
  required_version = ">= 1.3.0"
  required_providers {
    azurerm = {
      source  = "hashicorp/azurerm"
      version = ">= 3.7.0, < 4.0.0"
    }
    random = {
      source  = "hashicorp/random"
      version = ">= 3.5.0, < 4.0.0"
    }
  }
}

provider "azurerm" {
  features {}
}

# This picks a random region from the list of regions.
resource "random_integer" "region_index" {
  min = 0
  max = length(local.azure_regions) - 1
}

# This ensures we have unique CAF compliant names for our resources.
module "naming" {
  source  = "Azure/naming/azurerm"
  version = "0.3.0"
}

# This is required for resource modules
resource "azurerm_resource_group" "rg" {
  name     = module.naming.resource_group.name_unique
  location = local.azure_regions[random_integer.region_index.result]
}

resource "azurerm_virtual_network" "vnet" {
  name                = module.naming.virtual_network.name
  resource_group_name = azurerm_resource_group.rg.name
  location            = azurerm_resource_group.rg.location
  address_space       = ["<IP_ADDRESS>/16"]
}
resource "azurerm_subnet" "subnet" {
  name                 = "AzureFirewallSubnet"
  resource_group_name  = azurerm_resource_group.rg.name
  virtual_network_name = azurerm_virtual_network.vnet.name
  address_prefixes     = ["<IP_ADDRESS>/26"]
}

resource "azurerm_public_ip" "public_ip" {
  name                = module.naming.public_ip.name
  resource_group_name = azurerm_resource_group.rg.name
  location            = azurerm_resource_group.rg.location
  allocation_method   = "Static"
  sku                 = "Standard"
  zones               = ["1", "2", "3"]
}

module "firewall_policy" {
  source              = "Azure/avm-res-network-firewallpolicy/azurerm"
  resource_group_name = azurerm_resource_group.rg.name
  location            = azurerm_resource_group.rg.location
  name                = "fwp-test"
}

# This is the module call
module "firewall" {
  source             = "Azure/avm-res-network-firewall/azurerm"
  name                = module.naming.firewall.name
  location            = azurerm_resource_group.rg.location
  resource_group_name = azurerm_resource_group.rg.name
  firewall_sku_tier   = "Standard"
  firewall_sku_name   = "AZFW_VNet"
  firewall_zones      = ["1", "2", "3"]
  firewall_ip_configuration = [
    {
      name                 = "ipconfig1"
      subnet_id            = azurerm_subnet.subnet.id
      public_ip_address_id = azurerm_public_ip.public_ip.id
    }
  ]
  firewall_policy_id = module.firewall_policy.id
}

Error received;
The given value is not suitable for module.vwan_with_vhub.var.firewalls declared at modules/avm-ptn-irtualwan/variables.tf:83,1-21: element "xxx": attribute "firewall_policy_id": string required.

Suspected issue is in outputs.tf of module;
output "id" {
  description = "The ID of the Firewall Policy."
  value       = azurerm_firewall_policy.this
}

This returns the entire resource, which is already being done by the "resource" output.
Suggest to change this code to;
output "id" {
  description = "The ID of the Firewall Policy."
  value       = azurerm_firewall_policy.this.id
}

thank you for opening the issue @philipstreet will look into this and add an update into the next version.

thank you for opening the issue @philipstreet will look into this and add an update into the next version.

I was going to submit a PR to fix but didn't realise that wasn't an option until reading the contribution page.

@philipstreet moving forward use resource output.  will update the documentation in the next PR

module.firewall_policy.resource.id

new release has been pushed. Outputs have been updated.


Node Js hangs up frequently

Hi ,
I am using Node Js with Express. Our Node js basically pull data from source on request and sends it to the requester. It sends the data in HTTP POST request.I have a scheduler in the requester which generate thousands of the requests for data. I observe the server is getting hanged frequently when the scheduler is running.
1.Is there any limitations of how many requests can be spawned to Node js .?
2. Is there any condition when the Node js event loop can be blocked?
Please guide me.
Thanks in advance,
Regards,
Rajesh

Unless you have an issue with digestAuthRequest.js specifically, this is not the place to post. Try stack overflow. Thanks.


Add warning for overlapping hours

That is correct. I had not thought through that I will rework and examine. Make it more robust.


docs: fix typographical errors

This pull request is intended to fix the typographical errors that are present in the nuxt-pages-plus documentation.

OMG, thank you so much!


Add non-english game title support

First at all, I don't have any experience in web development/typescript/React. I did my best searching on Stackoverflow and google then modifying the source code based on the original hltb-for-deck, so please feel free to reject or modify my changes.
I found the non-english game title will become a empty string after applying normalize function, and it leads the result always points to "https://howlongtobeat.com/game/95647". So instead I using steam api "http://store.steampowered.com/api/appdetails?appids={APP_ID}" to get json information and parsing the english game title, and then pass it to useHtlb.
Tested languages: Traditional Chinese, Korean, Japanese, Spanish and Russian. All of them work as expected.
Tested steam item: 'Hogwarts Legacy' 990080 , 'Paradise Killer' 1160220, 'Detention' 555220, 'Black Desert' 1258320
During the testing, found steam id 1258320 didn't get correct hltb record, which was due to the item is no longer available on steam and there's a '(retire)' after the game title. Modify the normalize function in src/utils.ts to remove '(retire)' string.
Hogwarts Legacy

Paradise Killer

I don't think this will work for non-Steam games.

I don't think this will work for non-Steam games.

I can maybe try to incorporate this code to try this explicitly for steam games. I appreciate the contribution. I will credit you if/when I make the change

@alecchen
Hi
As far as I'm aware this is the current active repo
https://github.com/SDH-Stewardship/hltb-for-deck


fix(ci): upgrade action github/codeql-action/upload-sarif@v3

Merging as it does not affect tests.


german letter ü in username

Hi,
I ran into a problem with my username while entering the file path into node.
My username contains the german letter ü and i dont know how to enter that into node.
I tried ue because thats how its usually abreviated but it couldnt find the files then.

Hi! Good question. I'm not sure how to enter that either - I created a user on my computer with this character and trying to copy+paste didn't work.
After entering the path normally the program will just store it in a text file. You can create this file yourself and it will stop asking for the path. Create a file called song-directory.txt (in the same folder as the start.bat file) and paste the path into it. Example file:
song-directory.txt


Constant name "target" doesn't conform to UPPER_CASE naming style

https://github.com/lbonanomi/scripts/blob/d4369094fb277f5c08521500b3186b83b4c530b9/natural.py#L14

Issues for naming style are now ignored.


cleanup(rules): adjust priority for Run shell untrusted

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind feature

/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area rules

/area registry

/area build

/area documentation

Proposed rule maturity level

Uncomment one (or more) /area <> lines (only for PRs that add or modify rules):

/area maturity-stable

/area maturity-incubating

/area maturity-sandbox

/area maturity-deprecated

What this PR does / why we need it:
@darryk10 adjusting priority for "Run shell untrusted". In addition, will propose an update to the https://falco.org/docs/rules/style-guide/ stating no upstream rule shall be of priority debug, so that no adopter needs to run Falco with priority set to debug unless they choose to based on their customizations ...
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: incertum
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [incertum]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: darryk10, incertum
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [incertum]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: 707d2ba4856b2223e5bc52e62dbfa50735b703c8

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, darryk10, incertum
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,incertum]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


vfox backend returns error on installing plugin

Describe the bug
Backend for plugin changed to vfox without notice..
[DEBUG] install_versions<EMAIL_ADDRESS>[DEBUG] [vfox:kubectl] available_async
[DEBUG] Getting metadata for kubectl
[DEBUG] starting new connection: https://raw.githubusercontent.com/
[DEBUG] [vfox:kubectl] pre_install
[DEBUG] Getting metadata for kubectl
[DEBUG] starting new connection: https://raw.githubusercontent.com/
Error:
   0: error converting Lua table to PreInstall (no version returned from vfox plugin)

Location:
   src/backend/vfox.rs:66

Version:
   2024.10.4 linux-arm64 (6c21c0a 2024-10-14)

Backtrace omitted. Run with RUST_BACKTRACE=1 environment variable to display it.
Run with RUST_BACKTRACE=full to include source snippets.

To Reproduce
Latest version, install package
Expected behavior
Install to not change backends and installed :)
*mise doctor output
version: 2024.10.4 linux-arm64 (6c21c0a 2024-10-14)
activated: yes
shims_on_path: no

build_info:
  Target: aarch64-unknown-linux-musl
  Features: DEFAULT, NATIVE_TLS, OPENSSL
  Built: Mon, 14 Oct 2024 04:04:10 +0000
  Rust Version: rustc 1.81.0 (eeb90cda1 2024-09-04)
  Profile: release

shell:
  /usr/bin/fish
  fish, version 3.7.0

dirs:
  data: ~/.local/share/mise
  config: ~/.config/mise
  cache: ~/.cache/mise
  state: ~/.local/state/mise
  shims: ~/.local/share/mise/shims

config_files:
  ~/.config/mise/config.toml

backends:
  cargo
  core
  go
  npm
  pipx
  spm
  ubi
  vfox

plugins:
  ansible-core  https://github.com/amrox/asdf-pyapp#41f5275
  argocd        https://github.com/beardix/asdf-argocd.git#429bbf9
  awscli        https://github.com/MetricMike/asdf-awscli.git#c26bbb4
  bat           https://github.com/s3than/asdf-bat#4464f27
  chezmoi       https://github.com/joke/asdf-chezmoi.git#53f30be
  clusterctl    https://github.com/pfnet-research/asdf-clusterctl.git#169bd58
  coredns       https://github.com/s3than/asdf-coredns.git#218ef9c
  dust          https://github.com/looztra/asdf-dust.git#11bdb8a
  eza           https://github.com/lwiechec/asdf-eza.git#11f578d
  gcloud        https://github.com/jthegedus/asdf-gcloud.git#00cdf06
  helm          https://github.com/Antiarchitect/asdf-helm.git#091d00f
  jb            https://github.com/beardix/asdf-jb.git#ea246ac
  jq            https://github.com/mise-plugins/asdf-jq.git#6d86d19
  just          https://github.com/olofvndrhr/asdf-just.git#3f907ee
  kops          https://github.com/s3than/asdf-kops#15de991
  mimirtool     https://github.com/asdf-community/asdf-mimirtool.git#f15ee85
  packer        https://github.com/asdf-community/asdf-hashicorp.git#22eb1c4
  pipx          https://github.com/yozachar/asdf-pipx.git#31db618
  rust          https://github.com/code-lever/asdf-rust.git#95acf4f
  starship      https://github.com/gr1m0h/asdf-starship.git#9f7504b
  terraform     https://github.com/asdf-community/asdf-hashicorp.git#22eb1c4
  velero        https://github.com/looztra/asdf-velero.git#1d964b6
  yq            https://github.com/sudermanjr/asdf-yq.git#772992f

toolset:
<EMAIL_ADDRESS><EMAIL_ADDRESS>(missing)
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>
env_vars:
  MISE_SHELL=fish

settings:
  activate_aggressive = false
  all_compile = false
  always_keep_download = false
  always_keep_install = false
  asdf_compat = false
  cache_prune_age = "30d"
  ci = false
  color = true
  debug = false
  disable_backends = []
  disable_default_registry = false
  disable_hints = []
  disable_tools = []
  experimental = true
  fetch_remote_versions_cache = "1h"
  fetch_remote_versions_timeout = "10s"
  go_default_packages_file = "~/.default-go-packages"
  go_download_mirror = "https://dl.google.com/go"
  go_repo = "https://github.com/golang/go"
  go_set_gopath = false
  go_set_goroot = true
  go_skip_checksum = false
  http_timeout = "30s"
  jobs = 4
  legacy_version_file = true
  legacy_version_file_disable_tools = []
  libgit2 = true
  log_level = "info"
  not_found_auto_install = false
  paranoid = false
  pin = false
  plugin_autoupdate_last_check_duration = "7d"
  python_default_packages_file = "~/.default-python-packages"
  quiet = false
  raw = false
  trace = false
  trusted_config_paths = [
      "~/Projects",
      "~/projects",
  ]
  use_versions_host = true
  verbose = false
  yes = false

  [cargo]
  binstall = true

  [node]

  [pipx]
  uvx = false

  [python]
  default_packages_file = "~/.default-python-packages"
  pyenv_repo = "https://github.com/pyenv/pyenv.git"
  venv_auto_create = false
  venv_stdlib = false

  [ruby]
  default_packages_file = "~/.default-gems"
  ruby_build_repo = "https://github.com/rbenv/ruby-build.git"
  ruby_install = false
  ruby_install_repo = "https://github.com/postmodern/ruby-install.git"

  [status]
  missing_tools = "if_other_versions_installed"
  show_env = false
  show_tools = false
No warnings found
No problems found

Additional context
Add any other context about the problem here. Consider running mise with --debug or --trace for extra debug info.

@jdx That was amazingly quick.. Thankyou for your hardwork on this project


Davidson for MPO with complex entries: inner product lambda has small complex part

I'm doing DMRG on a 3-state Potts model. (Code and error message below). The Hamiltonian (hence the effective nsite Hamiltonian) has complex entries, so in davidson() the inner product
  lambda::Float64 = dot(V[1],AV[1])

(iterativesolvers.jl:90) has a small complex part. Two issues here, one straightforward to fix and one not:

We're trying to assign a Complex{Float64} to a Float64. This is easily fixable: write a function that drops the imaginary part if it is below some threshold (say 1e-10), and errors if it is above that threshold.
If I do that, I see that the thing runs fine for a little while, but then the imaginary part grows beyond the threshold. (I think this is over the course of one call to davidson(), i.e. it's an instability towards imaginary parts in the Davidson algorithm as opposed to a matter of increasing bond dimension hence MPS matrix size, but I actually haven't checked that.)

What's the right way to deal with this?
Code:

################################################################
# Potts siteset

function pottsSites(N :: Int; q :: Int = 3)
  return [Index(q, "Site,Potts,n=$n") for n = 1:N]
end

const PottsSite = makeTagType("Potts")

# 1-index my Potts states
# so diagonal elements of Z are
#    e^{2πi/q}
#    e^{2πi*2/q}
#    e^{2πi*3/q}
#    e^{2πi*(q-1)/q}
#    e^{2πi*q/q} = 1
# this seems like the least bad thing
function state(::PottsSite,
               st::AbstractString)
  return parse(Int64, st)
end

function op(::PottsSite,
            s :: Index,
            opname :: AbstractString)::ITensor
  sP = prime(s)
  q = dim(s)

  Op = ITensor(Complex{Float64},dag(s), s')

  if opname == "Z"
    for j in 1:q
      Op[j,j] = exp(2*π*im*j/q)
    end
  elseif opname == "ZH"
    for j in 1:q
      Op[j,j] = exp(-2*π*im*j/q)
    end
  elseif opname == "X"
    for j in 1:q
      Op[j % q + 1,j] = 1
    end
  elseif opname == "XH"
    for j in 1:q
      Op[j,j % q + 1] = 1
    end
  elseif opname == "X+XH"
    for j in 1:q
      Op[j,j % q + 1] = 1
      Op[j % q + 1,j] = 1
    end
  else
    throw(ArgumentError("Operator name '$opname' not recognized for PottsSite"))
  end
  return Op
end

################################################################
# DMRG

#-----------
# setup
#-----------

θ = π/8

N = 100
sites = pottsSites(N)
psi0 = randomMPS(sites)

ampo = AutoMPO()
for j = 1:N-1
    add!(ampo, -sin(θ), "Z", j,"ZH",j+1)
    add!(ampo, -sin(θ), "ZH",j,"Z", j+1)
end

for j = 1:N
    add!(ampo, -cos(θ), "X+XH", j)
end

H = toMPO(ampo, sites);

#-----------
# DMRG
#-----------

sweeps = Sweeps(15)
maxdim!(sweeps, 10,20,100,100,200)
cutoff!(sweeps, 1E-10)

#energy, psi = dmrg(H,psi0,sweeps,observer=observer)
energy, psi = dmrg(H,psi0,sweeps)

Error:
InexactError: Float64(3.7091034021132 + 5.051406153050126e-18im)

Stacktrace:
 [1] Type at ./complex.jl:37 [inlined]
 [2] convert(::Type{Float64}, ::Complex{Float64}) at ./number.jl:7
 [3] #davidson#123(::Base.Iterators.Pairs{Union{},Union{},Tuple{},NamedTuple{(),Tuple{}}}, ::Function, ::ProjMPO, ::ITensor{3}) at /home/christopher/work/src/ITensors.jl/src/iterativesolvers.jl:90
 [4] davidson(::ProjMPO, ::ITensor{3}) at /home/christopher/work/src/ITensors.jl/src/iterativesolvers.jl:59
 [5] macro expansion at /home/christopher/work/src/ITensors.jl/src/mps/dmrg.jl:34 [inlined]
 [6] macro expansion at ./util.jl:213 [inlined]
 [7] #dmrg#159(::Base.Iterators.Pairs{Union{},Union{},Tuple{},NamedTuple{(),Tuple{}}}, ::Function, ::MPO, ::MPS, ::Sweeps) at /home/christopher/work/src/ITensors.jl/src/mps/dmrg.jl:21
 [8] dmrg(::MPO, ::MPS, ::Sweeps) at /home/christopher/work/src/ITensors.jl/src/mps/dmrg.jl:9
 [9] top-level scope at In[9]:5

Thanks for pointing this out. Indeed, it looks like we never tested davidson with complex numbers. I am fixing it now!


Persist bot's and user's state and ability to create conversations

Beautiful library python-telegram-bot has some killer features I'd love to see in this .NET package:

bot_data: a dictionary, that can be used to store some data assosiated with the bot itself. It is available in every update request.
user_data: a dictionary that is user specific. When we get updates from the user, we can always take a look at user's data and change smth there or get additional info from there
conversations: this is so-so-so cool! You can define a conversation which is just a set of triggers and handlers. One can lead to another and it is very easy to create various flows of interaction with the bot. Here is an example: https://docs.python-telegram-bot.org/en/v20.0a6/examples.persistentconversationbot.html

Hi,
I've create a small library that achieves exactly this. It is not public, because it is not separated from my business logic, but I could think about releasing it to the public if needed.
It uses the Dipendency Injection pattern and it is really easy to use :)

https://github.com/TgBotFramework
It gives you all of the above, except for conversations, but you can easily implement them. Plus you can store them in DB with some amount of EF code to support it )
Any way data that you need to store is unique to your bot, so its hard to find solution that fits everyone

A simple Dictionary<string,object> will fit everyone 🙃

Dictionary<long, Dictionary<string,object>> 🙃

For now we are making the choice that Telegram.Bot is the basis library for building bots and more high-level bot frameworks.
It is up to these framework to keep context & state for following users conversation.
See issue #1072 for a list of such frameworks available.


An attempt at fixing tests on master

Lets see if this pleases CI.

@yaahc unfortunately it appears different rust versions produce different results :(

yea, it looks like we might need to add some custom post processing to strip out whitespace and *'s or something if we want to support block doc comments consistently across versions. I'm now kicking myself because I never setup a regular CI job to test new nightly versions so I completely missed this change going into stable :(.

I pushed a dirty hack to work around the *s but now nightly is failing because it appears to be generating a different error message... :(

Not the most elegant workaround but hopefully good enough...

Not the most elegant workaround but hopefully good enough...

yea, I'm sorry about that and thank you again for helping fix this for me :heart:

No problem! Thank you for maintaining this crate :)


refactor(scalable): ♻ extract isEnharmonicWith extension method

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

14 of 14 (100.0%)  changed or added relevant lines in 5 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1071

Relevant Lines:
1071

💛  - Coveralls


Azure Table Storage not available with Premium Performance

It is not described that Table Storage is only available on Standard Performance. The "Create Storage Account" blade makes it very hard to create a storage account for a table storage, since it's a trial and error which options lead to the expected result.

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

ID: 06288d35-3c83-c29d-95d8-a56823d910a3
Version Independent ID: 002fb5f5-5e4a-a95a-4564-2c399bcb5e06
Content: Azure Storage account options | Microsoft Docs
Content Source: articles/storage/common/storage-account-options.md
Service: storage
GitHub Login: @jirwin
Microsoft Alias: jirwin

@mguntli
Thanks for the feedback! We are currently investigating and will update you shortly.

@Adam-Smith-MSFT - can you assign this one to me? Thanks.


tests: add a dummy value for 'dev' release

Functional tests are broken when testing against 'dev' release (ceph).
Adding a dummy value here will fix them.
Typical error:
>       if request.node.get_marker("from_luminous") and ceph_release_num[ceph_stable_release] < ceph_release_num['luminous']:
E       KeyError: 'dev'

Signed-off-by: Guillaume Abrioux<EMAIL_ADDRESS>

jenkins test dev-ansible2.4-centos7_cluster

I was working on this in #2693 already. I don't think we want to have the tests think that anything 'dev' is a bogus release. I think it's causing trouble in the playbook run as well according to the CI tests here. I also made a patch to ceph-build to set the release to nautilus when dev tests are run. https://github.com/ceph/ceph-build/pull/1040
I think in the context of PR testing we can always assume dev tests are going to be run against the latest ceph master. This would need to be updated each time there is a version bump in ceph master, which really isn't ideal, but I can't think of a quick and robust approach here. We gotta get our tests running against ceph master again.


Cannot create a virtual document with URI: bpasero://mySuperDocument

Code:
vscode.workspace.openTextDocument(vscode.Uri.parse('bpasero://mySuperDocument')).then((doc) => {
    return vscode.window.showTextDocument(doc);
});

Ends up on the console:
error:"Uri must have scheme and path. One or both are missing in: bpasero://mysuperdocument"

@bpasero The uri has only scheme and authority. It's some sort of arbitrary restriction I can remove, you can get further with bpasero:///mysuperdocumentor bpasero://mysuperauthority/mysuperdocument

@jrieken ah right I forgot the third slash, somehow I thought I would only need that when using file URIs, but it applies to any URI. I am not sure if we should remove this restriction because it has meaning in the file scheme, but maybe we ignore it for other schemes.


Uncaught TypeError: Cannot read property 'bind' of undefined

Thanks for sharing inferno-starter.  It appears to be exactly what I am looking for.
However, I get this loading any page.  Here is the stack trace.  I can't see any issues that it creates, but it does not feel good.
Uncaught TypeError: Cannot read property 'bind' of undefined
at createReactCompositeComponent (http://localhost:2002/build/bundle.js:3389:42)
at updateReactComponent (http://localhost:2002/build/bundle.js:3306:23)
at createReactCompositeComponent (http://localhost:2002/build/bundle.js:3393:29)
at updateReactComponent (http://localhost:2002/build/bundle.js:3306:23)
at createReactCompositeComponent (http://localhost:2002/build/bundle.js:3393:29)
at updateReactComponent (http://localhost:2002/build/bundle.js:3306:23)
at http://localhost:2002/build/bundle.js:3321:110
at Array.map (native)
at normalizeChildren (http://localhost:2002/build/bundle.js:3321:80)
at createReactDOMComponent (http://localhost:2002/build/bundle.js:3350:39)

I forgot to say where this error occurs.  It appears to be when inferno tries to pass through forceUpdate().

I just figured out that this has something to do with devtools

@scottburch Thanks for reporting this, I updated the code.


pypi entry out of date

The latest version of mongoctl on pypi is 0.3.7. The latest stable version of mongoctl is 0.7.5.
Please update asap, as it would make patching much easier!
https://pypi.python.org/pypi/mongoctl

Updated pypi to 0.7.8.
Thanks!


fix(crypto/sha256): crash on sha256 init

crypto hash function consists of init()->update()->final()
during init(), struct for functor was initialized
but not with internal ptr and may results in crash.
This commit fix it.

TEST_CASE("[hash] sha256", "[crypto]")
can check output of this commit


Unabble to execute on Mac M1

I got it working using https://support.apple.com/en-gb/guide/mac-help/mh40616/mac.