Updated build definition link in readme

Summary of the Pull Request
Updated the broken link for build definition in README.md

@enricogior, this seems to be the correct link, unsure why it was pointing to the other one.  any background info?

@udit3333 wait for enrico to comment

I'm not sure what happened. Also last week I paused a second pipeline that was also active and not sure when and who created it.
Anyway the fix looks good.


Scroll to top doesn't work within the app after integration Hoko SDK.

Hello,
We have faced an issue with "scroll to top" functionality after integration Hoko SDK. After some debugging we found that issue related with deferred deeplinking. SFSafariViewController is never releasing.
Could you please check this out?
Thanks in advance,
Best regards,
Alexey

Hi @AlexeySergienko,
Thank you for reporting this issue. Can you help us out here and check if the bugfix at #30 fixes it?
Remove our pod, and link the source code present at this branch instead:

Edit your Podfile and remove the pod 'Hoko' line
Run pod update to update your local pod repository
Open your Xcode project and link the SDK source code (it should be a drag-drop thing)

Run your app with this version and check if the issue is not longer present. Also, test if your smart links and deferred smart links (those that drive the user through the app store) continue to work.
To test deferred smart links, uninstall your app and then just deploy/install your app through Xcode, once you reach the App Store.
Let us know how it goes.

Hi @hugocore,
Thank you for your quick answer. We've checked the version from rollback-fingerprinting branch. For us everything works fine.
Best regards,
Alexey

Hi again,
We've release v2.5.0 which includes this fix.
Please update your local pod.
Best regards,
Hugo


How to define custom collection with shared property member

Hello,
How to define custom collection with shared property member? Please kindly see attached, property "Data" cannot be serialization/deserialization.
Thanks a lot
B/r
Alex Luo
using MemoryPack;
using System.Collections;
namespace MemoryPackCollectionsTest
{
[MemoryPackable(GenerateType.Collection)]
public partial class MyDictionary : Dictionary<string, MyDictionary> where T : INodeData
{
#region Properties
    [MemoryPackOrder(0)]
    public T? Data { get; set; }

    #endregion

    #region Constructor

    [MemoryPackConstructor]
    public MyDictionary()
    { }

    public MyDictionary(T data)
    {
        Data = data;
    }

    #endregion

}

}

Sorry, I don't know why “T” be deleted while copy & paste.


AUR package in working order

Fixes #258

Thanks, I haven't tested it but I'll take your word for it!


Add number of photos as second column in benchmarks data

Number of photos is available if you reference the datasets list but it would be helpful to see in the raw benchmarks data as well.  It is redundant info, but also useful to see in place.

New benchmarks data parser now provides a clustered "by dataset" view, which includes number of photos.  I believe the needs that produced this issue are addressed in that update.  Closing...


unsupported Swift 3.0

I cannot build it with Carthage because Xcode 10.2 doesn't support Swift 3.X. Could you upgrade it to minimum Swift 4.0 at least.
SWIFT_VERSION = 3.0;

Duplicated #43

@jshier indicated there would be a new release this week that includes the upgrade to Swift 4.0. In the meantime you can point at the master branch.
You can see his comments on #44.

2.4.0 has been released to address this issue. Thanks for your patience! Cheers! :beers:


Generated sites does not show properly in TLS only environments

Having generated a site with version 1.1.1 yields a broken layout if shown in a browser with httpseverywhere, since the links to

JQuery
Bootstrap
html5shim
are http only.

Either they should be changed to https or a configuration option could be added that allows locally downloaded versions to avoid the mix of http and https.
The following links provoke the problem:
<link href="http://netdna.bootstrapcdn.com/bootswatch/2.3.2/cosmo/bootstrap.min.css" rel="stylesheet" />
<link href="http://netdna.bootstrapcdn.com/twitter-bootstrap/2.3.1/css/bootstrap-responsive.min.css" rel="stylesheet" />
<script src="http://html5shim.googlecode.com/svn/trunk/html5.js"></script>
<script src="http://ajax.googleapis.com/ajax/libs/jquery/1.9.1/jquery.min.js"></script>
<script src="http://netdna.bootstrapcdn.com/twitter-bootstrap/2.3.2/js/bootstrap.min.js"></script>

Changing them to https works fine:

https://netdna.bootstrapcdn.com/bootswatch/2.3.2/cosmo/bootstrap.min.css
https://netdna.bootstrapcdn.com/twitter-bootstrap/2.3.1/css/bootstrap-responsive.min.css
https://ajax.googleapis.com/ajax/libs/jquery/1.9.1/jquery.min.js
https://netdna.bootstrapcdn.com/twitter-bootstrap/2.3.2/js/bootstrap.min.js
except for
http://html5shim.googlecode.com/svn/trunk/html5.js, which should be referenced statically.

Which way would you prefer?

configuration option to enable SSL and providing html5.js with a relative link?
configuration option to have all external references locally and use relative links?

The simplest option would be to change all above links to https and provide a local copy of htlm5shim - shall I start a pull request?
Thanks

@ottlinger,
Have you looked at the local resources option, documented at http://andriusvelykis.github.io/reflow-maven-skin/skin/resources.html?

Bindul

+1


BotAddedToConversation in Slack

I install new bot for the team and select channel #general. Bot does not receive BotAddedToConversation event.

The "Add To Slack" button authorizes a bot on a Slack team but does not actually add the bot to any Slack channels.  You still have to use the "/invite @botname" command to add the bot to a channel at which time your bot should receive the BotAddedToConversation event.

There seems to be a problem with the slack implementation. Since at least five days we do not receive the BotAddedToConversation event when we add our bot to a new channel.

@craigjensen No. When you add bot for team, you need select channel. The BotAddedToConversation event not fired.

There are two issues here.
First, while the UX does ask for a channel during registration, the bot is NOT added to the channel so the BotAdded event is not fired.  A bug has been opened to address this.
Second, when you /invite a bot to a channel, that's when the event should get fired.  We'll look into this and file a bug if it's not firing as expected.

@craigjensen We tested it Yesterday and it is working again. Thanks for fixing this!


Create hyperlinks to maintainers github

Need to change out text for maintainers previous and current to hyperlinks to github about pages

will be available in v2.2.3


Button Not Working

I recently enabled the HID only mode but when I plugged in the USB while holding down the button inside, nothing happened. Before this I also notice that holding down the button while inserting the USB seemed to do nothing, the USB would still execute the on insertion payload.

It might be that some solder joint got damaged. But maybe this is
just because the button is not held down long enough. Try to do this:
hold the button, plug the device in, wait for 5 seconds, release. Check
if it helped or not.

Just tried that, didn't seem to do anything, it must be a physical issue with the PocketAdmin.

Well, then it is probably pin 12 on the microcontroller that got a damaged solder joint.

It is not really hard to fix, though I am not sure if you have a soldering iron around.
Good news is that this button is not very necessary for 1.3 revision devices. You can
get out of HID-only mode by tapping capslock 20 times or more (while device is inserted)
And if you want to prevent payload from being run you can use ALLOW_EXIT command.
That is, you start every payload with a sequence of WAITFOR_INIT and then ALLOW_EXIT
commands. This way you can prevent payload execution by spamming capslock while
you insert the device.

Great, the capslock trick worked.

Though, if you are able to reflow that solder joint then you could restore MSD-button
operation as well. If you don't have that, maybe just press down on that pin with something
sharp, like tweezers or needle and it could restore the connection again


Fixs issue #800

Add new command add new command: obc-peer network, to show all
existing network connections for the target peer node.
Signed-off-by: David Geng<EMAIL_ADDRESS>

@jeffgarratt @binhn , please help to review it.

@genggjh Thanks! I'll merge this. My one concern is whether a peer wishes to expose its connections to all clients of the peer. ACL will soon be added for certain functions such as deploy and I imagine this command will also need access control.
I've opened issue #935 to discuss.


Problem to Setup the Configuration

Hallo,
i am using a Rapsberry Pi and try to make a magic mirror. I am a noob in programming. I am trying to Setup the Configuration, but I stuck here. Dont know the path to database, source folder and so on. Can someone help me?

Run genConfig.sh, including the following as arguments:

bash genConfig.sh config.json {device serial number}
/{{path to database}}
/{{path to source folder}}/avs-device-sdk
{{path to build}}/Integration/AlexaClientSDKConfig.json
Thanks and best regards
Mat

Hi @ond0pe, can you please try to follow these instructions instead: https://github.com/alexa/avs-device-sdk/wiki/Raspberry-Pi-Quick-Start-Guide-with-Script
Thanks,

Hi @ond0pe,
Have you had any success using the Raspberry Pi Quick Start Guide?

Hi, I have a similar issue. I have a raspberryPi 3 with a reSpeaker hat 4-Mic.
I got to the step of running the setup:
sudo bash setup.sh config.json -s

... and it gets stuck at 97%

@MM-DeeDee your issue seems similar to the one found here https://github.com/alexa/avs-device-sdk/issues/1191#issuecomment-457371346
@ond0pe are you still having trouble with this issue?

@ond0pe closing due to inactivity. Please feel free to open a new issue or reopen this one if you are still having trouble.


Side menu navbar button not working on targets below 11.0

Just stuck in the touched state and no side menu presentation occurs.

FIxed in 5d87481.


Project details are not updated when removing Team member from project

Is your feature request related to a problem? Please describe.
When we try to remove the team member associated with Project from Project detail page, error occurs on updating team mebers.
Refer screenshot

Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Editing  the project details

Issue fixed in https://github.com/ColoredCow/portal/pull/1390


add nyc

It doesn't seem to use OAuth. It should be good to go if you change this column to No, but an api key is mandatory


howto set background color properly?

I like this colorscheme but seems like it use black background color, I want to change it, how can I do that?

You can play with the contrast parameter, this changes the background color. In my case, I use
Monotone 44 97 68 0 0 0 0.85
where the contrast setting is 0.85.

As mentioned you can change the background color with the contrast parameter. A negative contrast parameter will render a light background if that's what you prefer.


obfuscate or compile flutter packages to prevent reverse engineering

Currently obfuscate only for release build. But packages are in source code.
For non-open source projects, non-public classes/methods names and local variable names in packages needs to be obfuscated.
Java code can be obfuscated using proguard, and javascript can be minified.
Flutter package dart code also needs to be obfuscated or minified or compiled, and make it difficult to be reverse engineered.

You can obfuscate code using proGuard in flutter also. Link.

@pranavo72bex The issue is mentioned for dart codes obfuscation rather than platform specific obfuscations.

@javaone199
A non-source-based package option is something that would need to be filed at https://github.com/dart-lang/sdk and/or https://github.com/dart-lang/pub. Flutter doesn't have a separate package system from Dart.
Also see this relevant comment for your further reference.
Closing from here.


Yield locks

If a task enounters a lock that another task is currently mutating, the entire thread sleeps and cannot perform other tasks. This can deadlock a thread if a task is waiting on a task in another thread that is waiting on a task later in the queue on the original thread. If there are other threads available, as long as the other thread doesn't also encounter this issue, it only decreases performance as the other threads can still steal tasks and eventually get the work done. However, if this type of situation happens across all available threads, the entire thing deadlocks.
What I recommend as the best solution to this problem is introducing custom "lock" types that simply call yield_now or something if locked. When the thread resumes the task, if it is still locked, it keeps calling yield_now, and otherwise proceeds with the task as usual. This way, even if the situation mentioned above happens, it doesn't really matter because the threads will still be able to get the work done.

I'm going to close this as a duplicate of #592.
However, just yielding is not sufficient. If the task that is holding the lock is earlier on the same thread stack as the task that is now trying to obtain the lock, then no amount of yielding can resolve that. It's essentially an attempt at recursive locking, even thought that may be unintentionally induced by rayon's work-stealing.
There's a proposal for "blocking install" that I still need to get around to. This would give a way to better isolate work across threadpools, without work-stealing while you wait, so for example you could safely hold a lock in the caller.


Add constrain option

add a geometric constraint option

This is related to AMSJob. We need the option to add geometry constraints (see documentation).


how to use vue-apollo in vue-typescript program

Hi! our program are using Vue with typescript, while using vue-property-decorator, I want to know how to use vue-apollo in our program

Have you checked the docs? https://akryum.github.io/vue-apollo/

thanks a lot @philmueller . when I use vue-apollo with vue-property-decorator, I found the support of vue-apollo is not good enough


fix(storagebox): fix clearing storage

🙈 good catch, thanks for this. We should get some tests added for StorageBox overall at some point 😅 but this is ok to merge since the fix is clear


vyper: 0.3.10 -> 0.4.0

Description of changes
Fixes CVE-2024-22419, CVE-2024-24559, CVE-2024-24560, CVE-2024-24561, CVE-2024-24563 and CVE-2024-24567.
Release notes:
https://github.com/vyperlang/vyper/releases/tag/v0.4.0

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review run on x86_64-linux 1

4 packages built:

vyper (python311Packages.vyper)
vyper.dist (python311Packages.vyper.dist)
python312Packages.vyper
python312Packages.vyper.dist

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

Result of nixpkgs-review pr 323252 run on aarch64-linux 1

4 packages built:

vyper (python311Packages.vyper)
vyper.dist (python311Packages.vyper.dist)
python312Packages.vyper
python312Packages.vyper.dist


No way to detect authentication before using http client

Scenario
I would imagine the use case is pretty common. You  wrap your entire application in a @preload that dispatches a redux action to fetch the logged-in user. The credentials for such a call come from configuration. Something like:
authentication: {
  accessToken ({ getCookie }) {
    return getCookie('token');
  },
}

The magic here being that getCookie fetches client-side cookies and sets them as the Authorization header for every request (even server-side preloads :blush:) made by the http client.
Problem
Obviously, the first time a user visits the site, there will be no token cookie.  This would cause the preload request to raise an exception (401 or something of that ilk).  My current hack is to catch the exception in the redux action. For example;
export const getUser = redux.action(
  'GET_USER',
  () => async (http) => {
    try {
      return await http.get('/api/user/');
    } catch(e) {
      return null;
    }
  },
  'user'
);

This works, but generates unnecessary calls to /api/user on every initial page load - calls we shouldn't even make because there is no token cookie.  Unfortunately, there currently is no way to check if the above accessToken found anything.
Possible Solutions
I would love to hear if one of these solutions strikes your fancy more than the other:

Make the magic of getCookie available as a parameter in @preload. You could choose not to dispatch getUser if no token is present.
Expose the authorization headers on the http client itself making it possible for the redux action to no-op if it knows no header is present.

Hi.
That's a good issue description, very informative.
I had something like this before (in 2016) but then I decided that React is too slow for server-side rendering (something like PHP is much faster I guess) so I dropped the feature.
Added getCookie parameter to @preload() in the latest version<EMAIL_ADDRESS>

Thanks so much for the quick update!


Redesigned "Make an Appointment" Section

Description
The section has been revamped for improved usability and visual appeal, aligning with current design standards and user expectations.

Updated HTML structure to enhance accessibility and semantic clarity.
Implemented modern CSS styling for improved aesthetics and responsiveness.
Enhanced user interaction with updated form elements and button design.
Ensured compatibility across different devices and screen sizes through responsive design practices.

Related Issues

Closes #609

Type of PR

[X] Bug fix
[ ] Feature enhancement
[ ] Documentation update
[ ] Other (specify): _______________

Screenshots / videos (if applicable)
Before:

After:
https://github.com/Anishkagupta04/RAPIDOC-HEALTHCARE-WEBSITE-/assets/146121869/7c23954d-7479-4ab8-850b-855667905782
Checklist

[X] I have gone through the contributing guide
[X] I have updated my branch and synced it with project main branch before making this PR
[X] I have performed a self-review of my code
[X] I have tested the changes thoroughly before submitting this pull request.
[X] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[X] I have commented my code, particularly in hard-to-understand areas.

@Anishkagupta04, ma'am please review the pr

@AsmitaMishra24 The form blocks are too long. Can it be minimized and keep them in the center?

@AsmitaMishra24 The form blocks are too long. Can it be minimized and keep them in the center?

I will make the changes by tomorrow.

@varshith257, i have made the changes

@varshith257, Please review the pull request. I have made the changes as suggested.


Improve rendering behavior in JS-free environments

Bug/issue #, if applicable: Part of #765
Summary
This PR adds some @media (scripting: none) media queries to some parts of the app to better improve the flow when pre-rendering to a static site, then loading with JS disabled.
Note that the scripting: none media query is not widely supported, so it might be wise to swap this with another approach for detecting the presence of JS. The first thing that comes to mind is adding a class like no-js to the <html> tag, then removing it once JS runs. I’m interested to hear any thoughts on this!
Dependencies
No formal dependencies.
Testing
Unfortunately the prerendering script is not yet available, but you can simulate it:

Load a doc page up in a browser with JS available
Wait for the page to fully load
Save the HTML out to the appropriate location in the .doccarchive (making sure to name it index.html
Serve the doccarchive as a static site, then load that page with JS disabled.

Checklist
Make sure you check off the following items. If they cannot be completed, provide a reason.

[ ] Added tests (I’m not sure there’s a good way to do automated tests for this)
[x] Ran npm test, and it succeeded
[x] Updated documentation if necessary (no changes needed)

@swift-ci test


Bump golang.org/x/net to 0.33.0

Describe the feature
Bump golang.org/x/net to 0.33.0

[!TIP]
Vote this issue reacting with :+1: or :-1:

Resolved by https://github.com/artemiscloud/activemq-artemis-operator/commit/3b618b3478ba17287e35d9ad5f52174d6ee844ab


Can't set titleColor and subtitleColor for a PopupMenuItem

Environment Information

Platform:

iOS

Package version(s): (fill this out, include which package manager you're using)
Xcode and OS versions: Xcode version: 14.3

Please provide a reproduction of the bug:
Create an instance of PopupMenuItem, try to change the color of title or subtitle by changing the titleColor or subtitleColor properties.
In the PopupMenuItem class also provided by FluentUI, I don't see these two fields being used, they're just exposed but not used I think (titleColor & subtitleColor).
Actual behavior:
The color doesn't get changed for title or subtitle
Expected behavior:
The color should get changed for title or subtitle
Priorities and help requested:
Are you willing to submit a PR to fix? (Yes, No)
Requested priority: (Blocking, High, Normal, Low)
Blocking
Products/applications affected: (if applicable)
Stream iOS App

I couldn't reproduce the bug as both titleColor and subtitleColor are working properly. Which version of FluentUI are you using?

I am using 0.8.5


Added Carcass Crawler 3 classes

Added Beast Master, Dragonborn, Mutoid, Mycelian, and Tiefling from Carcass Crawler 3

@mvanella merged, thanks for the contribution!


feat: Integration of pyo3-stub-gen

This PR introduces pyo3-stub-gen into the project to automate the generation of Python interface type hint files (.pyi) from Rust code.
However, during the integration process, several compatibility issues were encountered.
1. Our project currently uses pyo3 0.20.3, (primarily due to the dependency on pyo3-asyncio..?) which is incompatible with pyo3 0.21.0.
This version mismatch leads to compilation failures when trying to run the stub generator, specifically when executing cargo run --bin stub_gen as per the documentation.
2. Errors from Example Code:
The provided example code results in errors when following the documented steps to generate stubs.

Since pyo3-stub-gen requires pyo3 version 0.21 or higher, I think this issue is related to pyo3's version mismatch.
Ref: https://github.com/lablup/raftify/issues/90


v0.14.0 incompatible with nock v8.0.0

I recently updated from axios v0.12.0 to v0.14.0 and my unit tests containing nock js began failing. Rolling axios back to latest previous version v0.13.1 fixed the incompatibility issue.
The error I was getting from axios in my unit tests containing nock calls looked like this:
TypeError: Cannot read property 'slice' of null
  at dispatchHttpRequest (node_modules/axios/lib/adapters/http.js:84:37)
  at new Promise (node_modules/core-js/modules/es6.promise.js:191:7)
  at httpAdapter (node_modules/axios/lib/adapters/http.js:19:10)
  at run (node_modules/core-js/modules/es6.promise.js:87:22)
  at node_modules/core-js/modules/es6.promise.js:100:28
  at flush (node_modules/core-js/modules/_microtask.js:18:9)
  at _combinedTickCallback (internal/process/next_tick.js:67:7)
  at process._tickCallback (internal/process/next_tick.js:98:9)

Not sure this is related to nock (I use nock and it is working). Can you add code example?

I have an example with nock @ axiosV0.15.2 - everything is working fine. I think that this issue is not related to nock / was already fixed. We might consider closing it..

I think you're right. Please, feel free to close this issue. Thanks for the response.


chore(main): release 1.4.6

:robot: I have created a release beep boop
1.4.6 (2023-04-16)
Styling

Format code with prettier (fbbce9f)

Code Refactoring

：添加聊天界面样式 (0aab2af)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/School-of-Website-Engineering/future-front-end/releases/tag/v1.4.6 :sunflower:


Fix typo error

This was causing the package from running because no .mjs file was being generated.

Thanks


refactor(Storage): Retries HMAC key cleanup.

And moves the cleanup to individual Dispose methods so that tests themselves are not delayed.
Note: HMAC keys need to be deleted inmediately because only 5 are allowed per service account.
Fixes #1272

It's okay, but given that we've got the same code in multiple tests, I wonder whether a common base class might be worth considering.

Yep, makes sense, will push that shortly.

@jskeet Added a second commit with a common class in case you want to take another look. Else, I'll wait on green before merging. Thanks!

Taking a quick look now.

Failures are unrelated and being tracked in:

#1340


Do we have plans for data packing?

Thank you for creating this excellent project. The design is elegant and the implementation is very efficient.
I'm wondering if there are plans to support sequence packing?
OpenRLHF has demonstrated significant speedup after supporting sequence packing. Given VERL's already impressive performance, I believe adding packing support could bring even greater efficiency gains.

Hi @YixinSong-e, do you mean data packing in the FSDP backend?
We already implemented data packing (i.e. remove padding?) in the Megatron-LM backend.
For the FSDP backend, we do have some monkey-patch to implement it upon transformers but it's not general for all models in transformers. So we haven't released them yet.
And we also noticed that only some important models (e.g., llama, qwen...) support data packing naturally in transformers.
We're thinking about some elegant way to support data packing for all models in our fsdp workers. Do you have any suggestions or references?

Hey @PeterSH6!
+1 for such an awesome project. My favourite in the new wave of open-source RLHF frameworks.

For the FSDP backend, we do have some monkey-patch to implement it upon transformers but it's not general for all models in transformers. So we haven't released them yet.

I'm curious what kind of monkey patch you mean here? I'd be interested in contributing if you'd like some help, I'm working on something similar in another repo.


TraceID - TypeError: Cannot read property 'FromString' of undefined

I'm trying to generate consistent Trace ID's across lambda invocations (in and event streaming platform).
However, I'm getting and error when calling static FromString method.
import * as AWSXray from 'aws-xray-sdk-core';

const foo = 'foo'
const bar = 'bar'

AWSXray.TraceID.FromString(`${foo}${bar}`).toString()

/**
 * trace-id-test.ts:6
 * AWSXray.TraceID.FromString(`${foo}${bar}`).toString()
 *                 ^
 * TypeError: Cannot read property 'FromString' of undefined
 */

and
import { TraceID } from 'aws-xray-sdk-core';

const foo = 'foo'
const bar = 'bar'

TraceID.FromString(`${foo}${bar}`).toString()

/**
 * trace-id-test.ts:16
 * TraceID.FromString(`${foo}${bar}`).toString()
 *         ^
 * TypeError: Cannot read property 'FromString' of undefined
 */

As best I can tell, I'm following the implementation in the tests: https://github.com/aws/aws-xray-sdk-node/blob/master/packages/core/test-d/index.test-d.ts
Many thanks for any pointers and help!
node -v v14.19.3
aws-xray-sdk-core ^3.4.0

Looks like this is an import issue. From what I can tell at first glance there may be a typo in the second code snippet that may be causing this: the ID part of TraceID should've been capitalized.
By directly importing the TraceID class, using AWSXray.TraceID is no longer necessary. Maybe try this?
import { TraceID } from 'aws-xray-sdk-core';

TraceID.FromString(`${timestring}${iemi}`).toString()

Sorry, that was terrible reporting from me with a copy paste error in the import.
I've corrected the initial report.


*: replace failure with anyhow and thiserror

What problem does this PR solve?
Problem Summary: part of #8698
What is changed and how it works?
What's Changed: replace failure with anyhow and thiserror in some crates
Related changes
Check List
Tests

No code

Side effects

No

Release note
No

/cc @BusyJay @nrc

/cc @overvenus

/merge

/merge

/merge


Request: Drag&Drop enhancements | command line option

First of all: great player - almost exactly what I was looking for.
Some things I suggest for making it even better:

would be nice to provide a certain area within the main window where you can drag&drop a folder and/or file(s) onto to replace the current playlist with. Sometimes I just want to listen to files from a folder, so I would like to drop that folder onto Aural and have that folder (and only that folder) played back instantly. Another workflow could be holding down the option key while dropping the folder or file.
integrate into Finder's "Open With" context menu for folders. So in Finder, right-clicking on a folder, then select "Open With" should provide Aural as an option. Action within Aural should also be replacing the current playlist with the files from the folder as well instead of appending it.
same stuff, but via command line: provide the option to hand a folder/file(s) to play in Aural via the command line. Something like /Applications/Aural.app/Contents/MacOS/Aural <path/to/folder>. This should include a check if Aural is already running (so don't open another Aural instance) and replace the current playlist with. With the command line option it would also be possible to create custom actions via Automator/Shortcuts ...

Thanks again for that great player!

Hi @zpete , thank you so much for the great feedback !
This is all really useful, I think. About your point number 2, I tried researching a way to implement it but was not able to find a way ... I already thought about it.
The command-line option is also something I thought of but didn't want to invest the effort as I was not sure if someone would find it useful ... but now you have mentioned that it would be useful, so I will look into implementing it.
BTW, please note that I am spending limited time on this project at the moment due to some personal stuff but I will try to grant your wishes when I get back to it !
Cheers !
Kartik

Thanks @kartik-venugopal for your kind reply.
I was able to get point number 2 kind of working myself.
What I did was editing Aurals' info.plist and add this snippet to the CFBundleDocumentTypes array:
                <dict>
			<key>CFBundleTypeName</key>
			<string>folder</string>
			<key>CFBundleTypeRole</key>
			<string>Viewer</string>
			<key>LSItemContentTypes</key>
			<array>
				<string>public.folder</string>
			</array>
		</dict>

After restarting lsregister and Finder via
/System/Library/Frameworks/CoreServices.framework/Versions/A/Frameworks/LaunchServices.framework/Versions/A/Support/lsregister -kill -r -domain user -domain local -domain system && killall Finder Aural is now included within "Open With" when I right-click a folder. The files from that folder are added to the current playlist and playback is started.
Thanks!
pete

That's good to know ! I added that XML snippet just like you did but did not restart lsregister or Finder ... will try that out !

Hi @zpete ,
I have pretty much finished working on your suggested points 1 and 2. Watch out for the upcoming release v3.13.0.

Number 1 works as you intended (you can drag/drop onto most of the player window's area), and in addition, I have added a preference (that can be adjusted under Aural > Preferences > Playlist) that affects the behavior when dragging / dropping files or folders from Finder into Aural - both the main window and the playlist window. This way, users can decide the behavior they want instead of being stuck with one.
The above mentioned setting offers 3 modes:

Append: This is the default (and currently the only) behavior ... files/folders are appended to the playlist.
Replace: This is the behavior you wanted.
Hybrid: This is the alternative "workflow" you suggested. When ⌥ is pressed, replace mode will be activated, with append mode activated otherwise.

I also tried to implement your point 2 with the XML snippet and your terminal command ... unfortunately, it didn't work. Furthermore, some of my apps stopped working, which required a system restart (Big Sur). Even if it did work, since it involves a manual user action to activate it, it would not be a feasible solution. So, it will require more research. If you figure out a way to make it work without any user actions (programmatic actions are okay) and without any potential problems, let me know !
I also added a preference similar to the one for drag/drop. This one is for the "Open With" from Finder. Again, you can decide the behavior, the only difference being that hybrid mode is not available (the option key cannot be pressed when you do Open With).

Finally, regarding point 3, I tried to open Aural.app as it is, passing in a folder URL. It worked. The only thing is - currently, you cannot dynamically set the playlist behavior (append or replace) at open time via the CLI. It would rely on the preference setting mentioned above.
./Aural.app/Contents/MacOS/Aural /Users/kven/Music/Dld

Of course, it keeps the Terminal prompt waiting while Aural is running, so it is not very smart. If you run it with &, then you may end up with multiple instances of the process. This could be improved in a future version. I will look into it and see the best way to do this.

For the new preference setting, see the below image:

Released in v3.13.0.
Feel free to try it out and close the issue if you are satisfied. Let me know if you run into issues.
Cheers !

Thank you sooo much @kartik-venugopal . Really appreciate your work! The Drag&Drop-Implementation is immensely useful!
I'll try to find out more about point 2 ...

Thanks for letting me know, Pete !
BTW, I wanted to let you know that I've been working hard on getting a command line interface developed for Aural. I had to research ways to perform inter-process communication so that my CLI app can talk to the main Aural app. I found a pretty simple way to do it (CFMessagePort, if you're wondering).
So now, it will be able to do a LOT more than you requested:

Play a set of URLs (files, folders, M3U playlists) ... either append or replace.
Set volume, mute, or unmute
Affect sequencing: Repeat / Shuffle
Change tracks and/or stop playback
Effects
... and more !

Coming soon, hopefully !

Hi, I am shelving the command line features for now. Closing this issue since the main requested feature has been released.


Living Whites Plug not controlable with v3

Like in our other conversation about Osram Smart Plugs one of my users is using an old Philips Living Whites Plug.
With node-hue-api v2 he could control it. With v3 the error is: Error: Unknown type, Dimmable plug-in unit
If you need more information, please let me know.

{"state":{"on":true,"bri":254,"alert":"select","mode":"homeautomation","reachable":true},"swupdate":{"state":"notupdatable","lastinstall":null},"type":"Dimmable plug-in unit","name":"Weihnachten Terrassenfenster","modelid":"LWL001","manufacturername":"Philips","productname":"LivingWhites Plug","capabilities":{"certified":true,"control":{},"streaming":{"renderer":false,"proxy":false}},"config":{"archetype":"plug","function":"functional","direction":"omnidirectional"},"uniqueid":"00:17:88:01:00:2f:80:b1-0c","swversion":"<IP_ADDRESS>91"}

Thanks, will sort that tomorrow for you.
This was one of the reasons I did not originally go with matching of types to instance objects... starting to regret the decision to match to types as there is no definitive list of types that are supported.

3.3.2 is in the npm registry that should resolve this


[Search Collection] Add custom searches via GUI

Extension name
Search Collection
Feature Description
Currently, it's possible to manually edit SearchCollection.psm1 to write in additional search engines or amend existing ones (see example of changing YouTube to add "gameplay" to the search). Would be nice to have that option more easily accessible, e.g. see screenshot from Augmented Steam.
Screenshots

Should be done after a complete rewrite of the extension in C# since it currently is made in Powershell

I've rewritten the extension from scratch in C# and implemented this in https://github.com/darklinkpower/PlayniteExtensionsCollection/commit/8194cebba2295720d721da93896bd000660837a6


Confirm moving a node

I'm trying to use
"check_callback": async function (operation, node, node_parent, node_position, more)
{
    if(operation !== 'move_node')
    {
        return false;
    }

    if(more && more.core && !confirm('Are you sure ?'))
    {
        return false;
    }

    return true;
},

But even if I decline the confirmation dialog, the node still gets moved, what is the proper way to do this ?

The function can not be async. Let me know if this fixes the issue.

Without async it indeed works properly. However I'd like to use sweetalert2 which returns a promise (hence the async), is there a workaround for it to work with async ?


more project / workspace operations on dashboard

I think it'd be great if there were a few more functions available to the user when he is confronted with their dashboard.
some of those functionalities could be:

open in a new window
edit the project name from the dashboard
remove from the dashboard
go to workspace settings for that workspace
go to github repo of that repository
reorder tiles by dragging them

These are some neat ideas!

open in a new window

Already works by CTRL +Click. Maybe should have mentioned that in the readme :)

go to workspace settings for that workspace
go to github repo of that repository

These are interesting features! I need to have a look on the logic behind that, though. Firstly, if the VSCode API supports opening the workspace settings of another workspace. Secondly, how the extension should check for the Github URL. And as I think about it: Some people (including me and coworkers) use this extension mostly/also for work, where the projects are hardly pushed to GitHub. This may be an optional option for the project, however I would like to have the user select only the minimum amount of information when creating a project.

edit the project name from the dashboard
remove from the dashboard
reorder tiles by dragging them

I initially planned to make a designated editing mode for exactly these functions. I decided against it as I did not want to include too many custom UI elements into VSCode (which is also discouraged by the docs). But if it's a desired feature for many users, we could surely implement this! So +1 this if you would like such a feature!!! ;)

Reordering, editing and deleting projects has been implemented in version 1.3, released today. :)


Unable to debug using sourcelink

When I try to use LanguageServer, I am getting -32602 (InvalidParams) error. So, I would like to Debug using Source Link. When I step through to Power Fx code, I get the error below.

Please update the Source Link information for the assemblies published on Nuget. If I ignore the warning and use the LanguageServer file on my local machine, the debug is all messed up and I am unable to step through properly.

I have a similar problem. SourceLink doesn't seem to point to the public sources.

Thanks for pointing this out. The public Nuget packages were published from our private staging repo, we'll publish a new one from this repo and get that fixed up soon.

0.2.2-preview published to nuget.org. Should have up-to-date source link information.  (https://www.nuget.org/packages/Microsoft.PowerFx.Core/0.2.2-preview)


Build libtensorflow_cc.dll x86 error on Window

System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Window 10
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device: NA
TensorFlow installed from (source or binary): Source
TensorFlow version: 2.6.0
Python version: Python 3.9.7
Installed using virtualenv? pip? conda?: NA
Bazel version (if compiling from source): 3.7.2
GCC/Compiler version (if compiling from source): MSVC 2019
CUDA/cuDNN version:  11.2
GPU model and memory: NVIDIA GeForce GTX 1650

As title, I got an error when try to create libtensorflow_cc.lib and libtensorflow_cc.dll which can be used with MSVC2019_32bit compiler.
I used below command:
bazel build --cpu=x86 --config=opt --config=cuda --define=no_tensorflow_py_deps=true --copt=-nvcc_options=disable-warnings --local_ram_resources=2048 tensorflow:tensorflow_cc
But I got below error:
 The following configs were expanded more than once: [cuda]. For repeatable flags, repeats are counted twice and may lead to unexpected behavior. INFO: Options provided by the client:   Inherited 'common' options: --isatty=1 --terminal_columns=225 INFO: Reading rc options for 'build' from c:\users\1111\desktop\work\tensorflow\.bazelrc:   Inherited 'common' options: --experimental_repo_remote_exec INFO: Options provided by the client:   'build' options: --python_path=C:/Users/1111/AppData/Local/Programs/Python/Python39/python.exe INFO: Reading rc options for 'build' from c:\users\1111\desktop\work\tensorflow\.bazelrc:   'build' options: --define framework_shared_object=true --java_toolchain=@tf_toolchains//toolchains/java:tf_java_toolchain --host_java_toolchain=@tf_toolchains//toolchains/java:tf_java_toolchain --define=use_fast_cpp_protos=true --define=allow_oversize_protos=true --spawn_strategy=standalone -c opt --announce_rc --define=grpc_no_ares=true --noincompatible_remove_legacy_whole_archive --enable_platform_specific_config --define=with_xla_support=true --config=short_logs --config=v2 --define=no_aws_support=true --define=no_hdfs_support=true INFO: Reading rc options for 'build' from c:\users\1111\desktop\work\tensorflow\.tf_configure.bazelrc:   'build' options: --action_env PYTHON_BIN_PATH=C:/Users/1111/AppData/Local/Programs/Python/Python39/python.exe --action_env PYTHON_LIB_PATH=C:/Users/1111/AppData/Local/Programs/Python/Python39/lib/site-packages --python_path=C:/Users/1111/AppData/Local/Programs/Python/Python39/python.exe --action_env TF_CUDA_VERSION=11.2 --action_env TF_CUDNN_VERSION=8 --action_env CUDA_TOOLKIT_PATH=C:/Program Files/NVIDIA GPU Computing Toolkit/CUDA/v11.2 --action_env TF_CUDA_COMPUTE_CAPABILITIES=3.5,7.0 --config=cuda --copt=/d2ReducedOptimizeHugeFunctions --host_copt=/d2ReducedOptimizeHugeFunctions --define=override_eigen_strong_inline=true INFO: Found applicable config definition build:short_logs in file c:\users\1111\desktop\work\tensorflow\.bazelrc: --output_filter=DONT_MATCH_ANYTHING INFO: Found applicable config definition build:v2 in file c:\users\1111\desktop\work\tensorflow\.bazelrc: --define=tf_api_version=2 --action_env=TF2_BEHAVIOR=1 INFO: Found applicable config definition build:cuda in file c:\users\1111\desktop\work\tensorflow\.bazelrc: --repo_env TF_NEED_CUDA=1 --crosstool_top=@local_config_cuda//crosstool:toolchain --@local_config_cuda//:enable_cuda INFO: Found applicable config definition build:opt in file c:\users\1111\desktop\work\tensorflow\.tf_configure.bazelrc: --copt=/arch:AVX --host_copt=/arch:AVX INFO: Found applicable config definition build:cuda in file c:\users\1111\desktop\work\tensorflow\.bazelrc: --repo_env TF_NEED_CUDA=1 --crosstool_top=@local_config_cuda//crosstool:toolchain --@local_config_cuda//:enable_cuda INFO: Found applicable config definition build:windows in file c:\users\1111\desktop\work\tensorflow\.bazelrc: --copt=/W0 --copt=/D_USE_MATH_DEFINES --host_copt=/D_USE_MATH_DEFINES --cxxopt=/std:c++14 --host_cxxopt=/std:c++14 --config=monolithic --copt=-DWIN32_LEAN_AND_MEAN --host_copt=-DWIN32_LEAN_AND_MEAN --copt=-DNOGDI --host_copt=-DNOGDI --copt=/experimental:preprocessor --host_copt=/experimental:preprocessor --linkopt=/DEBUG --host_linkopt=/DEBUG --linkopt=/OPT:REF --host_linkopt=/OPT:REF --linkopt=/OPT:ICF --host_linkopt=/OPT:ICF --verbose_failures --distinct_host_configuration=false INFO: Found applicable config definition build:monolithic in file c:\users\1111\desktop\work\tensorflow\.bazelrc: --define framework_shared_object=false DEBUG: C:/users/1111/_bazel_1111/ejl7ywpf/external/tf_runtime/third_party/cuda/dependencies.bzl:51:10: The following command will download NVIDIA proprietary software. By using the software you agree to comply with the terms of the license agreement that accompanies the software. If you do not agree to the terms of the license agreement, do not use the software. ERROR: C:/users/1111/_bazel_1111/ejl7ywpf/external/local_config_cuda/crosstool/BUILD:24:19: in cc_toolchain_suite rule @local_config_cuda//crosstool:toolchain: cc_toolchain_suite '@local_config_cuda//crosstool:toolchain' does not contain a toolchain for cpu 'x86' ERROR: Analysis of target '//tensorflow:tensorflow_cc' failed; build aborted: Analysis of target '@local_config_cuda//crosstool:toolchain' failed INFO: Elapsed time: 1.352s INFO: 0 processes. FAILED: Build did NOT complete successfully (35 packages loaded, 175 targets configured)
Is there any way to fix this?
If yes, please share with me.
Thanks

@tilakrayal
Could you please help me check this issue?
Thanks

@VNTamao
This may be due to conflicts in dependencies.Always it would be better to create an virtual environment and install tensorflow .
You might be facing this issue because of the following reasons
You are running 32-bit Python or 32-bit OS
You have not installed the Microsoft Visual C++ Redistributable package
Your CPU does not support AVX instructions.
Please take a look at the system requirements and check if you have the correct dependencies installed.
Thanks!

@Saduf2019
Can we create a library 32 bit to using with Desktop applition?
I'm developing a Desktop application, use Qt IDE and MSVC 2019 32bit.
I tried with MSVC 2019 64bit and it's ok. But when I tried change to MSVC 2019 32bit, I cannot create tensorflow_cc library.
Does tensorflow support to create library using for MSVC 2019 32bit?
Thanks

No, as explained in most of the issues, there is no support for 32 bits. Please use 64 bit its tested and works as expected. You may move this issue to closed status as the issue reported is addressed.


[5.5][Collections] Auth tokens need to be mutable (#3443)

https://github.com/apple/swift-package-manager/pull/3443 for 5.5

@swift-ci please test

Rebased to resolve conflicts. @swift-ci please test


Incorrect info

Hi,
In below URL, under "Support Matrix for Control Plane Hosting Options" section, the last row and last column value is "Y". i.e "Customer-hosted Runtimes" is supported for "Anypoint platform PCE".
This has contradiction with:
Same page, under "Control Plane and Runtime Plane Support Matrix" section, second row last column is mentioned as "N". i.e "Standalone runtimes" is not supported for "Anypoint platform PCE".
Please clarify why these two sections are not in sync.
URL:
https://docs.mulesoft.com/general/intro-platform-hosting#support-matrix-for-control-plane-hosting-options
Thank you
Sujatha

Hello @sujatha-mule It looks like this issue was resolved, so I am closing this issue. Thank you for pointing out the contradiction.


How to handle duplicate keys?

Sorry for the silly question, I'm new to this entire C# scene, and I'm using this parser for a project.
If two keys with the same name exist in the same object, they're both used in that object but have the same value. In the VDF formatted file that I'm using, though, they both have different values.
In other words, is it possible to access these values like a numeric array to avoid this issue? Or is there anything else in this parser that lets me do this?

No, your question is definitely a good one. I'll include this in the FAQ at some point.
In general, you cannot access values of duplicate keys in the same way you would a dictionary, which VObject inherits from. So when you access a property by its key name, you always get the first corresponding value.
But there is something you can do. VObject maintains a copy of every child VProperty, which you can access by calling Children(). Assume you have VDF data like:
"root"
{
    "key1" "value1"
    "key2" "value2"
    "key1" "value3"
}

To access the last value of "key1", just do:
VProperty root = VdfConvert.Deserialize(vdf);
Console.WriteLine(root.Value["key1"]); // Prints 'value1'
Console.WriteLine(root.Value["key2"]); // Prints 'value2'
Console.WriteLine(root.Value.Children().Last(x => x.Key == "key1").Value); // Prints 'value3'

As an aside, could you tell me where you found the erroneous VDF data you have? Did you find it in the Steam installation directory?

To add to my previous response, if you need an array of values, just do:
VToken[] vals = root.Value.Children().Where(x => x.Key == "key1").ToArray();

I'm closing this issue due to inactivity. If you have further questions, feel free to reopen.

If anyone else stumbles across this issue and just wants to treat duplicates differently:
var json = VdfConvert.Deserialize(text).ToJson(new VdfJsonConversionSettings() { ObjectDuplicateKeyHandling = DuplicateKeyHandling.Ignore, ValueDuplicateKeyHandling = DuplicateKeyHandling.Ignore });

@Bluscream Note that your solution only applies when converting a VObject to a JObject. I believe the author was asking about accessing properties with duplicate keys directly within a VObject.

There is one misconception about VDF: it's not a dictionary! The best unambiguous way to represent parsed VDF data is to use a list.


Issues with CRTP Evaluate on GPUs

I am experiencing an issue in a downstream app wherein an Evaluate call (see https://github.com/lanl/singularity-eos/blob/dcd7a6efd5acbfa53903f08f02aca7abfa5c3f5f/singularity-eos/eos/eos_base.hpp#L135-L139) leads to data corruption on GPUs (volta with nvhpc).  I can seemingly only encounter the issue when working with Spiner EOS tables.
Printing data at addresses from explicit ptrs inside and outside the Evaluate call show differing values.
@jonahm-LANL asked me to make an issue --- we will work on isolating the problem to see if this is a downstream app or singularity issue.  If the latter, I am sure a reproducer is in order...

Which pointers? The functor? The class pointer? Or data pointers specific to spiner?

I think @pdmullen is refering to the pointers to data accessed. The call pattern that fails looks something like this:
// on device
Real *some_device_pointer = &my_device_view(0);
auto my_functor = [=](auto eos) {
  Kokkos::parallel_for(inner_range, [=](int it) {
    eos.SomeCall(pointer[i]);
  });
};
eos.Evaluate(my_functor);

I think @pdmullen is refering to the pointers to data accessed. The call pattern that fails looks something like this:
// on device
Real *some_device_pointer = &my_device_view(0);
auto my_functor = [=](auto eos) {
  Kokkos::parallel_for(inner_range, [=](int it) {
    eos.SomeCall(pointer[i]);
  });
};
eos.Evaluate(my_functor);

I assume that should be a KOKKOS_LAMBDA rather than a regular lambda?

Nope because that whole call is inside a kokkos kernel already.

Nope because that whole call is inside a kokkos kernel already.

Evaluate isn't a PORTABLE_FUNCTION. Does it need to be if it's being called on device?

That's why Evaluate is marked constexpr that's supposed to allow us to evaluate it on device. But it's possible something is going wrong there.

ahhhhhh gotcha


ORC Test App

This PR introduces the ORC test app, which lets us set up small examples that reproduce ODRVs that we can then enforce ORC to detect and report on. There are small examples that come with this PR, though I hope this battery to expand over time as the list of ODRVs that ORC can test expands. If the ODRVs found do not match up with the expectations, the tool will yell at you.
This also replaces all the other sample apps in the ORC project file, except for one. I kept one around to demonstrate/test the integration of ORC with Xcode's ld/libtool chain. Otherwise, all new ORC tests should be conducted with this tool.

@leethomason I've got a solid set of tests and I think things are in pretty good shape. Let me know if you have any issues with the PR. Thanks!

example_vtable is still in the project list - did you mean to remove that?

(You may have - that's an example and not a test?)

I assume most of the main.cpp vs. orc.cpp is moving stuff around

Right - the old main.cpp got split into two files, with orc.cpp getting the engine routines, and main.cpp keeping the work related to reading settings, forwarding to the linker, etc.

example_vtable is still in the project list - did you mean to remove that?

I meant to keep it, specifically to enforce/test the Xcode integration aspect of ORC.


Make sure you strip the whitespace between tags

Examples are formatted, real Recipts are not. If you try to verify XML signature with whitespace preserved, it can fail validation, depending on the language and library used.

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

ID: 28d2904b-93c3-1b6d-307d-8ee17a8210e7
Version Independent ID: 90c6af9f-fc0b-febc-f031-d57f5c34d862
Content: Use receipts to verify product purchases - UWP app developer
Content Source: windows-apps-src/monetize/use-receipts-to-verify-product-purchases.md
Service: unspecified
GitHub Login: @mcleanbyron
Microsoft Alias: mcleans

Thank you for your feedback. I have added notes before each of the receipt examples in this article to clarify that real receipts do not have whitespace between the XML elements. These notes should be published to the live article in a day or two.


Occasional authentication failure

Description
Jenkins tests sometimes fails with output in oned.log
[AuM][I]: Command execution failed (exit code: 255): /var/lib/one/remotes/auth/server_cipher/authenticate
[AuM][I]: bad decrypt
[AuM][D]: _authenticate: bad decrypt
[Lis][I]: Stopping
[AuM][E]: Auth Error: bad decrypt

To Reproduce
Almost every run of full jenkins tests contains this error on random microenv and random test case
Expected behavior
No random errors on jenkins
Details

Affected Component: oned authentication
Hypervisor: --
Version: 6.0.3 and master, probably also earlier versions

Additional context
Error is triggered here probably caused by cipher.final
Could padding solve the issue?

Progress Status

[ ] Branch created
[ ] Code committed to development branch
[ ] Testing - QA
[ ] Documentation
[ ] Release notes - resolved issues, compatibility, known issues
[ ] Code committed to upstream release/hotfix branches
[ ] Documentation committed to upstream release/hotfix branches

After some testing this usually happens on very low resource environments, and it is related to a failure accessing the Database. Marking this as worksforme


custom0 opcode not defined in riscv-gnu-toolchain

How is the custom0 opcode defined for RV32 toolchain?
I am trying to work with your IRQ examples in the firmware directory.
When I compile with the riscv32im toolchain I am getting unrecognized opcode custom0 errors:
start.S: Assembler messages:
start.S:22: Error: unrecognized opcode `custom0 2,x1,0,1'
start.S:23: Error: unrecognized opcode `custom0 3,x2,0,1'
start.S:28: Error: unrecognized opcode `custom0 x2,0,0,0'
start.S:31: Error: unrecognized opcode `custom0 x2,2,0,0'
start.S:34: Error: unrecognized opcode `custom0 x2,3,0,0'
start.S:77: Error: unrecognized opcode `custom0 a1,1,0,0'
start.S:88: Error: unrecognized opcode `custom0 0,x2,0,1'
start.S:91: Error: unrecognized opcode `custom0 1,x2,0,1'
start.S:94: Error: unrecognized opcode `custom0 2,x2,0,1'
start.S:126: Error: unrecognized opcode `custom0 x1,1,0,0'
start.S:127: Error: unrecognized opcode `custom0 x2,2,0,0'
start.S:129: Error: unrecognized opcode `custom0 0,0,0,2'

Does the custom0 opcode need to be patched into the binutils riscv-opc.c file?

Support for customN has been removed in riscv-gnu-toolchain. See https://github.com/riscv/riscv-gnu-toolchain/issues/190 for details.
The PicoRV32 riscv-gnu-toolchain-update branch contains some preliminary work to work around the changes in recent versions in riscv-gnu-toolchain. Until this work is finished, use version 7e48594 of riscv-gnu-toolchain, as described in the PicoRV32 README file.


First Good Issue Template

Name:
Issue Template

About:
This is a template for issue submission. Please modify it according to the  specific content.

Title:
''XXXX"

Labels:
First Good Issue

Assignees:
''XXXX"

Issue Type
Please select the type of issue you are submitting:

1.
Bug Report

2.
Feature Request

3.
Documentation Feedback

4.
Security Vulnerability Report

5.
Problem Discussion

6.
Collaboration Proposal

7.
Version Release

8.
Contribution Guidelines

9.
User Support

10.
Other (Customize by you)

Issue Description
Provide a detailed description of the issue, feature request, or discussion topic, or provide reproduction steps,etc.
Expected Behavior
Describe the behavior you expected (if it's a bug report) or the expected implementation of the feature (if it's a feature request).
Current State
Describe the actual behavior observed or the current state.
Screenshots/Logs/Example Code (optional)
If possible, please provide relevant screenshots, logs, example code, etc.
Environment Information（optional）

Operating System:
Browser/Application Version:
Other Relevant Information:

Other Context/Notes
Provide any additional context information or notes about the issue here.
Idea
Please write down your thoughts or suggestions on this issue here.
Contact
Email or others.

First-good-issue-template.md

What's the issue? What's to be done?


Update validation_trace.go

Compilation error fixed
type TraceValidationFinishFunc = TraceQueryFinishFunc
change to:
type TraceValidationFinishFunc TraceQueryFinishFunc

what's golang version you use?

What problem does this solve?

Is this related to #231 ?

@pavelnikolov definitely

What version of Go are you using?

Go version 1.9, I was getting the compilation error.

I'm closing this PR due to inactivity and I believe that it is no longer relevant after the tracing refactoring.


Support fetching openAPI schema from cluster or read from disk

This PR allows users to use one of three different sources for the openAPI schema. It can fetch the schema from the cluster using the current context, it can read it from a local file, or it can use the schema compiled into kpt.
The default is to fetch it from the cluster, but automatically fall back to using the complied-in schema if that doesn't work.
It also adds a new command under live, fetch-openapi, which just downloads the openAPI schema from the cluster given in the current context and outputs it to stdout.
Still need tests, but wanted to get feedback on whether we think this is the right approach.
@pwittrock @phanimarupaka @monopole

Why is it implemented here and not in cli-utils.

/lgtm


Fix adapt for server with both curl and busybox version wget

NOTE: In my trial last week, it was reported that a parameter (I forgotten ...) in the wget command was not supported by the busybox version wget, and I just temporarily renamed the wget on that server ...

Isn't there a better way around this?
In my current setup, curl performs significantly worse than wget (sometimes by a long shot) and here you just moved the priority to prefer curl. This would be detrimental for some of us.
You could maybe try distinguishing if you are running GNU Wget or BusyBox Wget by getting the --version output and checking for "BusyBox".


BSCStudio interactions

Can we provide screenshots to show how to setup / interact with this development tool?

To setup:

To deploy a contract:

To add the Metamask account to Binance Studio:

Lower left corner, click on the yellow key.

Write a name for the account to want to import from Metamask

Go to Metamask

Click on Account Details

Click on Export Private Key

Type your Metamask private password

Copy your password

Paste the password at Binance Studio

Interact with your contract:

Query how many tokens are available:

Query how many tokens you, the creator of the contract, have:

Proceed to transfer 1 token to 1 account to also added from Metamask

Check the balance of that account. Should be 1 token.

Check the balance of your account. Should be 99 tokens.


Contribution: Make comment optional when updating segment & feature flag targeting

Description
The Audit log is a running tally of changes made to feature flags and segments in a given environment. You can access the audit log from the side menu. To learn more, read the documentation
Whenever a change is made to a feature flag or a segment, the user is required to review the changes, edit a comment then save it.
Currently, the comment is a mandatory field. This is very unconvenient as sometimes the user may have no comments on it. The present task is to make the comment field optional.
Scope
Tech stack: Angular 14.0
Service： UI
Component： core components -> change-view
Files to modify: modules\front-end\src\app\core\components\change-review\
Screenshot

Id like to contribute to this issue please assign it to me


Add/Update repositories CRUD to use ArgoCD secret and new API endpoint

TODO:

remove original Helm repositories dialogs and utilities
update repositories form to correctly configure credentials for Private repositories

The labels were changed to Repositories instead of HELM Repositories
I think UX should also approve this
The wizard currently only supports HELM, so if the repositories tab is meant to support also git repos the wizard should as well

/lgtm


tor-browser-bundle-bin: 8.0.4 -> 8.0.5 (Backport)

Motivation for this change
https://blog.torproject.org/new-release-tor-browser-805 says "This release features important security updates to Firefox."
Please add the security tag if you can.
Things done

[x] 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"
[x] 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)
[ ] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Thanks a lot!


Fix misleading error messages

This fixes the case where the file, line number, and error message reported are incorrect, showing the error message for a previous call to error() that was caught with a pcall().
describe('Test assert after pcall', function()
  it('should report line number of assert', function()
    local ok, msg = pcall(function() error('error message') end)
    assert(false, 'should report this line number')
  end)
end)

In the above example, the error message reported would be from the pcall(), instead of the error message from the assert(), where the actual test failure occurred.
The solution is to not replace error in the test environment. This can lead to more inconsistent error messages that may be difficult to trace.  Go back to rewriting the error message to insert the file and line
information for complex error objects.

Typos aside, this looks really good. Will test and roll a new rc tonight.

I especially like moving nil handling into rewrite message, a much more robust approach.

Just did a rebase to fix the typo.  Good catch.

I was able to test this early. Good to go!


Project groups localization

Contribution Type: Impovement
Primary Area: Project groups, Lang
Description of change:

Added localization for Project Groups (+ ru localization)
Added fallback to english for missing localization strings

:+1:

Rebase onto master done.


Extend grisp_gpio module

Quoted from @peerst (thank you so much !) :

From the Erlang level driver standpoint I would add a function grisp_gpio:set_mode/1  which takes atom values:

disabled
single
continuous

In single mode calling the function that gets you the range does:

enable sonar
wait for serial message,
disable sonar
return value

possible implementation ?
set_mode(single) ->
     configure(uart_2_txd, output_1, [default]),
     Delay = 10, % 10ms just to make sure
     timer:sleep(50 + Delay),
     Range = pmod_maxsonar:get(),
     configure(uart_2_txd, output_0, [default]).

(PR will to be opened if any success)

That wouldn't work.  pmod_maxsonar:get() just returns the last stored value and doesn't wait for a new one.  In general all this mode switching needs to be done in the server callbacks.
Also switching mode doesn't read anything it should switch mode and then the get/0 should return a value.

Thanks ! :slightly_smiling_face:
So more like this for example ? :
handle(single, State) ->
     configure(uart_2_txd, output_1, [default]),
     get_the_range(),
     {noreply, State#state{ mode = single }}.

handle({got_a_range, Val}, State) ->
     configure(uart_2_txd, output_0, [default]),
     process_the_value(Val),
     {noreply, State#state{mode = single_done}}.

handle({range, Val}, State) ->
     {noreply, State#state{ measured = Val }}.

get_the_range() ->
     case  pmod_maxsonar:get() of
             Val ->
                   erlang:send({got_a_range,Val}, ?SERVER);
             undefined ->
                   % retry
    end.

process_the_value(Val) ->
     erlang:send({range, (Val * 2.54)}, ?SERVER).

Thanks !
So more like this for example ? :
handle(single, State) ->
     configure(uart_2_txd, output_1, [default]),
     get_the_range(),
     {noreply, State#state{ mode = single }}.

handle({got_a_range, Val}, State) ->
     configure(uart_2_txd, output_0, [default]),
     process_the_value(Val),
     {noreply, State#state{mode = single_done}}.

handle({range, Val}, State) ->
     {noreply, State#state{ measured = Val }}.

get_the_range() ->
     case  pmod_maxsonar:get() of
             Val ->
                   erlang:send({got_a_range,Val}, ?SERVER);
             undefined ->
                   % retry
    end.

process_the_value(Val) ->
     erlang:send({range, (Val * 2.54)}, ?SERVER).

My apologies for the horrendous pseudocode above, I barely managed to write it on my phone somehow but I hope the idea is more or less understandable 😅


add ender viewer which can check the EnderStorage fluid in a GUI

See https://github.com/GTNewHorizons/GT-New-Horizons-Modpack/issues/7860

@DSLM any news here?

@DSLM any news on this ?

Already added


Add line count

I was wondering if you wanted to add line count as well. I did on my project and thought it was worth asking if it's a feature you would want. If so I can send a PR for it, it's pretty small to be honest. Let me know!

Yeah, that would be great, thanks!


custom age makefile fails with g++

the Makefile for the custom version of age_align has -O3 option added, on my debian systems at least this pretty well breaks age_align.  it hangs most of the time, even at -O it hangs on the test.sh script 90%+ of the time.
tried compiling with gcc 4.6.3 and 4.9.2, nearly the same results.
removing the -O option has it working

Hi Joel,
Thanks for your interest in MetaSV. Do you get the same problems when
compiling from the main AGE repository (https://github.com/abyzovlab/AGE)?
Can you email me the logs or sample test input which indicate the
failure/breaking of age_align?
On Mon, Jun 15, 2015 at 3:00 PM, Joel Martin<EMAIL_ADDRESS>wrote:

the Makefile for the custom version of age_align has -O3 option added, on
my debian systems at least this pretty well breaks age_align. it hangs most
of the time, even at -O it hangs on the test.sh script 90%+ of the time.
tried compiling with gcc 4.6.3 and 4.9.2, nearly the same results.
removing the -O option has it working
—
Reply to this email directly or view it on GitHub
https://github.com/bioinform/metasv/issues/44.

--
http://www.bina.com
*Marghoob Mohiyuddin, PhD  *//  Sr. Bioinformatics Scientist, Bina
Technologies, part of Roche Sequencing
E<EMAIL_ADDRESS>Learn More  //  video http://www.bina.com/product.html  //  news
http://www.bina.com/press-coverage.html  //  posters
http://f1000.com/posters/search?query=bina+technologies  //  science
http://www.bina.com/science.html  //  platform
http://www.bina.com/platform.html
*Test Drive Bina  //  Experience NextGen analysis for yourself *
http://www.bina.com/test-drive.html

I have the problem with either age repository when -O3 is added ( or -O or -O2 )
make clean && make
./test.sh
occasionally it will make it through one or 2 tests, mostly it hangs.
make clean && make
rm -f obj/AGEaligner.o obj/age_align.o
Compiling with parallel support.
g++ -fopenmp -DOMP -DAGE_VERSION="v0.4" -DAGE_TIME -O3 -c AGEaligner.cpp -o obj/AGEaligner.o
g++ -fopenmp -DOMP -DAGE_VERSION="v0.4" -DAGE_TIME -O3 -c age_align.cpp -o obj/age_align.o
g++ -fopenmp -DOMP -DAGE_VERSION="v0.4" -DAGE_TIME -O3 -o age_align obj/AGEaligner.o obj/age_align.o
j_martin@genepool12:~/gscratch_jm/brachyinsanity/metasv/AGE$ ./test.sh

Test #1
For alignment with large deletion and micro insersion. Mismatches
and small deletions are present in each flanking sequence.
Run as: ./age_align test1_*.fa
[hangs here]

It possibly indicates an error in AGE but I haven't worked through all the -Wall reports.

I did a bit of testing with the compiler flags. It seems that compiling without OpenMP supports works fine with -O3 option. Could you try make OMP=no and run the tests again?

Yep that works.
wall was giving
AGEaligner.cpp:911:0: warning: ignoring #pragma omp shared [-Wunknown-pragmas]
AGEaligner.cpp:1027:0: warning: ignoring #pragma omp shared [-Wunknown-pragmas]

The OpenMP pragmas are fine. However, there may be a concurrency issue which pops up when the -O flags are used-this will require more investigation. However, for the time being, you have the option of either disabling OpenMP or not using -O flags.

Since we don't require the customized AGE aligner anymore, can we close this issue here and advise reopening on AGE?


How can we smooth the 3D object ?

Is it possible to do some pre-process as opencv to the 3D object?
Such as smooth ?

It's not impossible but that's quite complicated to do and far beyond the
scope of this project.
On Nov 8, 2017 07:58, "ybdesire"<EMAIL_ADDRESS>wrote:

Is it possible to do some pre-process as opencv to the 3D object?
Such as smooth ?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/WoLpH/numpy-stl/issues/62, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAQg67hML4KhRmC2Zk-QGHs10Xxy2243ks5s0VEQgaJpZM4QV7I9
.

Thanks @WoLpH for the update. Do you have any idea to do this ?

It depends on the type of object but scipy has some methods available:
https://stackoverflow.com/questions/32753449/what-to-do-if-i-want-3d-spline-smooth-interpolation-of-random-unstructured-data
On Nov 8, 2017 08:11, "ybdesire"<EMAIL_ADDRESS>wrote:

Thanks @WoLpH https://github.com/wolph for the update. Do you have any
idea to do this ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/WoLpH/numpy-stl/issues/62#issuecomment-342730094, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAQg65sFPsXn59Iy2xQP7QdeFdMx0LXtks5s0VQPgaJpZM4QV7I9
.

Thank you very much @WoLpH .   : )


chore: get dependency revision from npm registry

Add dependency revision in lib/THIRD_PARTY_NOTICES from npm registry. Helps addressing https://crbug.com/347863537
Tested manually by running THIRD_PARTY_NOTICES_REVISION=true npm run build.

Addressed in chromium code base: https://crrev.com/c/5644823


Naming inconsistencies inherrited from Java 8 stream

From @venkats:

I truly do not like how the (Java 8) Stream API has been inconsistent in naming. For example, limit, but takeWhile, and skip but dropWhile. I encourage the API used in the Builders to be more consistent. For example, it may be better to either use limit and limitWhile or take and takeWhile.

I don't have a particularly strong opinion here - it seems like a choice between two evils, inconsistency with naming already used in the JDK, or internal inconsistency in its own API. The latter is possibly worse, so it may be a good idea to consider this.

I don't have a strong opinion either as soon as the differences (mapping) are documented in the spec. Do you want me to proceed?

Keep them consistent is a good thing.


ERROR: Cannot install -r requirements.txt (line 2) and numpy==1.19.5 because these package versions have conflicting dependencies

pip install -r requirements.txt

Collecting piexif==1.1.3
  Downloading piexif-1.1.3-py2.py3-none-any.whl (20 kB)
INFO: pip is looking at multiple versions of <Python from Requires-Python> to determine which version is compatible with other requirements. This could take a while.
INFO: pip is looking at multiple versions of numpy to determine which version is compatible with other requirements. This could take a while.
ERROR: Cannot install -r requirements.txt (line 2) and numpy==1.19.5 because these package versions have conflicting dependencies.

The conflict is caused by:
    The user requested numpy==1.19.5
    opencv-python <IP_ADDRESS> depends on numpy>=1.14.5; python_version >= "3.7"
    opencv-python <IP_ADDRESS> depends on numpy>=1.17.3; python_version >= "3.8"
    opencv-python <IP_ADDRESS> depends on numpy>=1.21.2; python_version >= "3.10"

To fix this you could try to:
1. loosen the range of package versions you've specified
2. remove package versions to allow pip attempt to solve the dependency conflict

@eiery Yes, with numpy==1.21.2 works

I might be wrong, but shouldn't the requirements.txt file be updated since 1.21.2 is expected, instead of 1.19.5?
The same issue happens with Python 3.8, so this does not related to "old" python versions, right?

Yes you update requirements.txt to use the newer numpy version. From my experience the older version worked with Python 3.8 but not with 3.10 (this is clearly shown in the OP's error message).
The conflict is caused by:
    The user requested numpy==1.19.5
    opencv-python <IP_ADDRESS> depends on numpy>=1.14.5; python_version >= "3.7"
    opencv-python <IP_ADDRESS> depends on numpy>=1.17.3; python_version >= "3.8"
    opencv-python <IP_ADDRESS> depends on numpy>=1.21.2; python_version >= "3.10"

In my PR https://github.com/bes-dev/stable_diffusion.openvino/pull/125 I propose an update to requirements.txt to get it to install.


Bulk actions/pagination issues

Old

[x] Single [...] action menu on every row
[x] Background color for selected rows
[x] Clear selectedness of rows that become invisible due to searching
[x] Sorting not very sort-ey
[x] "Are you sure you want to delete thing1 thing2 ... thingN" is kinda hard to read will not go well with 50 long names... Put a newline between each name, and if length > 10 then say "Are you sure you want to delete thing1 and 42 others"
[x] Move search into middle of floating row (between actions and paging).
[x] Z-index:

[x] Toggling "Show system" or removing container can result in invalid pages:

[x] Clicking on sort breaks selection, bulk actions
[x] Headers don't line up

[x] Call it just actions, use input-addons to make search look nicer:

[x] Clicking on row action icon more than once toggles the radio/selection
[x] tableCount preference ~doesn't seem to have any effect~ is read from session instead of prefs, and not recomputed when changed
[x] When a single item is selected, the additional actions are shown as the action name instead of human name:

[x] Clicking anywhere in the TD that contains the checkbox should act like the checkbox (preserve selection) not the row

Additional issues:

[ ] If I run a stop action on container, then select it again without refreshing page, it still thinks it's running and the Actions menu does not update.

[ ] Unselecting containers using checkbox, does not change row color back to white (it stays yellow)

[ ] Can still click on Actions and run that action even thou it's dimmed.

[ ] Search is case sensitive

[ ] After doing a search, the Show System checkbox, Actions drop-down and the yellow select no longer function properly.  Have to refresh page to get them working again.

Carryover:
Round 2

[ ] i18n: Headers & paging
[ ] (lacsso) Border on buttons (1px) does not match border on inputs (2px), see above.  @lvuch
[ ] page, perPage, content, currentPage, totalPage bindings deprecation warnings
[ ] If I command + Delete on multiple containers, it opens a new window instead of deleting the containers without bringing up delete modal.

Version - 1.3.0
Verified all issues are fixed


executed and other counters are not incremented after inspecting example

Thanks. This will be replaced with the functionality described in: #1516


expand set of characters allowed in a capture group name

here is the simple example from doc.rs,  just modify the group name to chinese
https://play.rust-lang.org/?version=stable&mode=debug&edition=2018&gist=3775118597eecd8c8e4a98475917b9d1
throw this error:
error: invalid capture group character

As the docs say, only [_0-9a-zA-Z] characters are allowed as capture group names:

(?P<name>exp)  named (also numbered) capture group (allowed chars: [_0-9a-zA-Z])

However, this is something that can, and probably should be expanded. But this isn't a bug and the error you're getting is correct.

sorry for that.
thanks ,if you can impl this

Why did you close this? I marked it as an enhancement and updated the title to reflect as such.

Before implementation, this requires a careful specification.

ok,thanks

is there any updates on this?

If there were, they would be recorded here.

cjk support mark , in case someone needs it

The set of characters will be expanded soon to be defined in terms of Unicode. Specifically, here is what the new rule will be:
Capture group names must be any sequence of alpha-numeric Unicode codepoints,
in addition to `.`, `_`, `[` and `]`. Names must start with either an `_` or
an alphabetic codepoint. Alphabetic codepoints correspond to the `Alphabetic`
Unicode property, while numeric codepoints correspond to the union of the
`Decimal_Number`, `Letter_Number` and `Other_Number` general categories.

@aohan237 See above. The new rule will include CJK support.

Thanks


Can't arm area through MQTT

This could very well be user error.  I have two areas set up (house & garage).  When sending the command to the command topic, I get an error that area was not defined.  Just for giggles I tried to define an area in JSON (but my alarm app/panel doesn't support this anyway) and got this error:
2021-01-21 08:06:17 DEBUG (MainThread) [custom_components.alarmo.mqtt] DISARM
2021-01-21 08:06:17 DEBUG (MainThread) [custom_components.alarmo.mqtt] Received command disarm
2021-01-21 08:06:17 ERROR (MainThread) [homeassistant.util.logging] Exception in async_message_received when handling msg on 'alarm/set': '{
"command": "DISARM",
"area": "house"
}'
Traceback (most recent call last):
File "/config/custom_components/alarmo/mqtt.py", line 175, in async_message_received
await entity.async_alarm_disarm(code, skip_code)
AttributeError: 'filter' object has no attribute 'async_alarm_disarm'
Would it be possible to define the MQTT commands and states in the areas?  Or perhaps just automatically add a listener similar to how you add the state publishing?

Hello,
I must say that the mqtt implementation is still in the works.
It should be possible to arm a dedicated area using the method you use, but apparently it doesn’t work out of the box yet.
I will try to test this in the coming days.

Would it be possible to define the MQTT commands and states in the areas? Or perhaps just automatically add a listener similar to how you add the state publishing?

These are the kind of questions I have been asking myself as well.
I recommend users to enable the “master alarm” and use that for arming/disarming.
But ofcourse the user should have the freedom to interface with areas directly.
A problem is that the MQTT configuration panel has a lot of settings (due to customizable payloads), I don’t think it’s wise (or needed) to have this setup for each area.
Would you prefer to see dedicated state+command topics per area, or would the addition of a JSON “area” parameter be sufficient?
Note that I’m planning to expand mqtt functionality, for example by reporting diagnostically information (like wrong password or open sensors list), but also adding a command out + state in topic, such that a remote alarm panel can be kept in sync.
I’m afraid these additions together with the area customizations, will result in a complicated procedure.
How would you like to see this?

Hello,
I must say that the mqtt implementation is still in the works.
It should be possible to arm a dedicated area using the method you use, but apparently it doesn’t work out of the box yet.
I will try to test this in the coming days.

Would it be possible to define the MQTT commands and states in the areas? Or perhaps just automatically add a listener similar to how you add the state publishing?

These are the kind of questions I have been asking myself as well.
I recommend users to enable the “master alarm” and use that for arming/disarming.
But ofcourse the user should have the freedom to interface with areas directly.
A problem is that the MQTT configuration panel has a lot of settings (due to customizable payloads), I don’t think it’s wise (or needed) to have this setup for each area.
Would you prefer to see dedicated state+command topics per area, or would the addition of a JSON “area” parameter be sufficient?
Note that I’m planning to expand mqtt functionality, for example by reporting diagnostically information (like wrong password or open sensors list), but also adding a command out + state in topic, such that a remote alarm panel can be kept in sync.
I’m afraid these additions together with the area customizations, will result in a complicated procedure.
How would you like to see this?

I think leaving the payload customization in the global settings is fine.  It is likely that someone using MQTT to command the alarm system is likely to use the same solution in every zone.  Perhaps just moving the command and state topics to the zone is the solution?  Or something like this in global:  Alarm State: /alarm/{zone}/state where zone is substituted from the list of zones when creating listeners, etc.  JSON is cool, but not every app support sending JSON payloads.
I myself use the wonderful MQTT Alarm Panel for Android app from ThanksMister.

I think leaving the payload customization in the global settings is fine.  It is likely that someone using MQTT to command the alarm system is likely to use the same solution in every zone.  Perhaps just moving the command and state topics to the zone is the solution?  Or something like this in global:  Alarm State: /alarm/{zone}/state where zone is substituted from the list of zones when creating listeners, etc.  JSON is cool, but not every app support sending JSON payloads.
I myself use the wonderful MQTT Alarm Panel for Android app from ThanksMister.

Also willing to help out if you want, but I would want to know the direction you wanted to head in before creating a PR.

Also willing to help out if you want, but I would want to know the direction you wanted to head in before creating a PR.

I updated the documentation on MQTT quite a bit, hoping it makes it more clear how things work (also in combination with multiple areas).
I squashed out a nasty bug that caused your error when trying to arm an area, it should work better after updating to v1.5.0.
As said before, the MQTT implementation deserves more attention, hopefully I can take this up soon.

I updated the documentation on MQTT quite a bit, hoping it makes it more clear how things work (also in combination with multiple areas).
I squashed out a nasty bug that caused your error when trying to arm an area, it should work better after updating to v1.5.0.
As said before, the MQTT implementation deserves more attention, hopefully I can take this up soon.

On a fresh install, with just one area (the default Alarmo area) and mqtt set to default topics, I've got this when sending a payload with DISARM or another command to /alarmo/command topic. Note: fresh install with this last updates above.
Logger: homeassistant.util.logging
Source: util/logging.py:108
First occurred: 2:53:15 PM (1 occurrences)
Last logged: 2:53:15 PM
Exception in async_message_received when handling msg on 'alarmo/command': 'DISARM' Traceback (most recent call last): File "/config/custom_components/alarmo/mqtt.py", line 167, in async_message_received entity = self.hass.data[const.DOMAIN]["areas"][0] KeyError: 0
Did the following on line 167:
entity = list(self.hass.data[const.DOMAIN]["areas"].values())
but then, got this:
Exception in async_message_received when handling msg on 'alarmo/command': 'DISARM' Traceback (most recent call last): File "/config/custom_components/alarmo/mqtt.py", line 175, in async_message_received await entity.async_alarm_disarm(code, skip_code) AttributeError: 'list' object has no attribute 'async_alarm_disarm'

On a fresh install, with just one area (the default Alarmo area) and mqtt set to default topics, I've got this when sending a payload with DISARM or another command to /alarmo/command topic. Note: fresh install with this last updates above.
Logger: homeassistant.util.logging
Source: util/logging.py:108
First occurred: 2:53:15 PM (1 occurrences)
Last logged: 2:53:15 PM
Exception in async_message_received when handling msg on 'alarmo/command': 'DISARM' Traceback (most recent call last): File "/config/custom_components/alarmo/mqtt.py", line 167, in async_message_received entity = self.hass.data[const.DOMAIN]["areas"][0] KeyError: 0
Did the following on line 167:
entity = list(self.hass.data[const.DOMAIN]["areas"].values())
but then, got this:
Exception in async_message_received when handling msg on 'alarmo/command': 'DISARM' Traceback (most recent call last): File "/config/custom_components/alarmo/mqtt.py", line 175, in async_message_received await entity.async_alarm_disarm(code, skip_code) AttributeError: 'list' object has no attribute 'async_alarm_disarm'

Changing line 167 to
entity = list(self.hass.data[const.DOMAIN]["areas"].values())[0]
Now it's working !

Changing line 167 to
entity = list(self.hass.data[const.DOMAIN]["areas"].values())[0]
Now it's working !

Looks like you’re right!
If you want the honors, you could make a PR for it. Else I will make sure it’s covered for the next release.

Looks like you’re right!
If you want the honors, you could make a PR for it. Else I will make sure it’s covered for the next release.

Sorry, whats a PR ?
No no no.. No need to honors kkk.. Just tried yesterday to migrate from bwalarm to alarmo and got this error. Today I've searched here and found the new release.
Thank you for your attention !

Sorry, whats a PR ?
No no no.. No need to honors kkk.. Just tried yesterday to migrate from bwalarm to alarmo and got this error. Today I've searched here and found the new release.
Thank you for your attention !

@mariomazon  I updated this bug in the code, thanks for reporting.

@mariomazon  I updated this bug in the code, thanks for reporting.

v1.5.1 includes your fix @mariomazon, thanks again.
This issue can be closed.

v1.5.1 includes your fix @mariomazon, thanks again.
This issue can be closed.


Support per-file _CoqProject

cf https://github.com/coq/coq/pull/19826

@rtetley CI was not very green, but at least on mster it should work


Lua Syntax Highlighting

Lua is a popular language, it should have syntax highlighting support in the RTD theme!

There is Lua support - see e.g. http://splash.readthedocs.org/en/latest/scripting-tutorial.html.
Use
.. code-block:: lua

    function foo()
    end

The code generated by markdown via mkdocs does not appear to have Lua syntax highlighting as far as I can tell.
It generates
<code class="lua hljs">
function foo()
end
</code>

Output for this code block with the theme appears to be: http://i.imgur.com/eI2qoxT.png

This is likely more of a mkdocs issue, feel free to file a bug report there:
https://github.com/mkdocs/mkdocs/issues


Unstable fps when using NVIDIA GPU

https://0x0.st/otqa.mp4 (please use mpv)
UPD: In mpv in full screen mode the fps is smooth only when I move the mouse (what?).

The video playback is quite smooth in my browser. Hence I do not think this is really an issue in wf-recorder.


Event infrastructure

Thomas wrote in the agreement: Event Infrastructure to register and report events about resources. What is the state about that?

There is something like the prometheus alert manager:
https://prometheus.io/docs/alerting/latest/alertmanager/
I need more details what this event infrastructure should be doing.

Since it is the comment of Thomas, I would propose we ask him. He is currently on vacation, maybe after that.


Added check for source being the same as destination

Closes #754

Codecov Report

Merging #780 (6f1dde1) into master (9b414c4) will decrease coverage by 0.61%.
The diff coverage is 50.00%.

@@             Coverage Diff             @@
##            master     #780      +/-   ##
===========================================
- Coverage   100.00%   99.38%   -0.62%
===========================================
  Files           12       12
  Lines          161      163       +2
  Branches        19       20       +1
===========================================
+ Hits           161      162       +1
- Misses           0        1       +1

Impacted Files
Coverage Δ

src/backend/move.ts
94.44% <50.00%> (-5.56%)
:arrow_down:


Separate out BigFloat FFT

This would be useful for other packages, e.g., ToeplitzMatrices.jl

Maybe it should be moved to FastTransforms.jl?@MikaelSlevinsky

It seems like the relevant Julia issue is https://github.com/JuliaLang/julia/issues/5155 and closed pull request is https://github.com/JuliaLang/julia/pull/6193. It seems like eventually there will be something like an FFTW.jl (sub)module part of a StdLib (set of) package(s).
I don't know exactly what the issue is. Is it that the BigFloat FFT in ApproxFun is useful in ToeplitzMatrices, but it's non-sensical to make ToeplitzMatrices dependent on ApproxFun just for a BigFloat FFT?
It would be nice to have a pure-Julia FFT supporting BigFloat with FFTW1-like code generation available @stevengj.

I just rewrote the BigFloat FFT so that ToeplitzMatrices.jl can work without requiring it, by making the plans <: Base.DCT.Plan, so it's not really a dependancy issue anymore.
It's more to make the functionality available to others without requiring ApproxFun.  FastTransforms.jl seems like a more reasonable home for it.

Maybe SlowTransforms.jl for the BigFloat case?  :wink:


SOF-699 TSR now handles a SetConcept timelineObject for VizMSE

Dependent on changes in v-connection

Codecov Report

Merging #22 (8bae2ef) into mirror/release37 (249032d) will increase coverage by 0.19%.
The diff coverage is 100.00%.

:exclamation: Current head 8bae2ef differs from pull request most recent head 634e0a4. Consider uploading reports for the commit 634e0a4 to get more accurate results

@@                  Coverage Diff                  @@
##           mirror/release37       #22      +/-   ##
=====================================================
+ Coverage             99.80%   100.00%   +0.19%
=====================================================
  Files                    18        18
  Lines                   508       510       +2
  Branches                  1        49      +48
=====================================================
+ Hits                    507       510       +3
+ Partials                  1         0       -1

Impacted Files
Coverage Δ

...kages/timeline-state-resolver-types/src/sisyfos.ts
100.00% <100.00%> (ø)

...ckages/timeline-state-resolver-types/src/vizMSE.ts
100.00% <100.00%> (ø)

...ackages/timeline-state-resolver-types/src/index.ts
100.00% <0.00%> (+2.27%)
:arrow_up:

Continue to review full report at Codecov.

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


PartialEq for errors

I would like to add PartialEq to errors in Anybuf. Comparing with matches!() works, but it is a bit more verbose. And even more verbose to compare BufanyError::UnsupportedWireType(u8)

Released as part of https://github.com/webmaster128/anybuf/releases/tag/v0.5.1

Please note that RepeatedStringError is currently not public such that you cannot compare against that one. Will fix soon.


Validate heat vector prior to propagation

Should check for p-value of 0 before converting to heat via "negative_log" option. Ideally, there would be a flag to set ceiling of p-value/heat(negative_log).

In tables.py:convert_to_heat() it looks like a kwargs arguments were added to let caller set floor and ceiling values "negative_log" option. These arguments are floor and ceiling respectively.


"2020-09-28: Improve README"

Change Grant to 2 statements, explicitly CREATE USER so it works even if implicit user creation is off, and to avoid creating a user with no password.  Match username delphix_os

CREATE USER delphix_os@<staging_host> IDENTIFIED BY 'delphix123'
GRANT SELECT, RELOAD, REPLICATION CLIENT,REPLICATION SLAVE,SHOW VIEW, EVENT, TRIGGER on . to 'delphix_os'@'<staging_host>';
Add flush privileges;

Add Eliminate GRANT ALL, use least need privs

Specify to create delphix_os as a typical Delphix target user.  Reference the docs.  Specify to refresh the target and insure connectivity to delphix_os.

Specify to add plugin to Delphix (Reference docs), discover environment, add database, link dsource.

Updated documentation
https://delphix.github.io/mysqllinux/


fix(deps): update dependency @actions/core to v1.10.0

This PR contains the following updates:

Package
Change
Age
Adoption
Passing
Confidence

@actions/core (source)
1.9.1 -> 1.10.0

Release Notes

actions/toolkit
v1.10.0

saveState and setOutput now use environment files if available #1178
getMultilineInput now correctly trims whitespace by default #1185

Configuration
📅 Schedule: Branch creation - "after 6am and before 6pm every weekday" (UTC), Automerge - "after 8am and before 5pm every weekday" (UTC).
🚦 Automerge: Enabled.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

⚠ Artifact update problem
Renovate failed to update an artifact related to this branch. You probably do not want to merge this PR as-is.
♻ Renovate will retry this branch, including artifacts, only when one of the following happens:

any of the package files in this branch needs updating, or
the branch becomes conflicted, or
you click the rebase/retry checkbox if found above, or
you rename this PR's title to start with "rebase!" to trigger it manually

The artifact failure details are included below:
File name: actions/pnpm-lock.yaml
 ERR_PNPM_UNSUPPORTED_ENGINE  Unsupported environment (bad pnpm and/or Node.js version)

Your pnpm version is incompatible with "/tmp/renovate/repos/github/pleo-io/pleo-spa-cicd/actions".

Expected version: 8.x
Got: 7.30.4

This is happening because the package's manifest has an engines.pnpm field specified.
To fix this issue, install the required pnpm version globally.

To install the latest version of pnpm, run "pnpm i -g pnpm".
To check your pnpm version, run "pnpm -v".

Your Node version is incompatible with "/tmp/renovate/repos/github/pleo-io/pleo-spa-cicd/actions".

Expected version: 16.x
Got: v18.15.0

This is happening because the package's manifest has an engines.node field specified.
To fix this issue, install the required Node version.