az vm list-skus filtering out VM sizes that are available regionally when they are restricted in all zones

Related command
az vm list-skus
Describe the bug
The current logic for "vm list-skus" is filtering out VM sizes that are available regionally when they are restricted in all zones. Relevant code line is this https://github.com/Azure/azure-cli/blob/b4759064b7e38203ef9d67a581105206a17f513b/src/azure-cli/azure/cli/command_modules/vm/custom.py#L1285
This does not appear to be correct behavior, since a VM size maybe restricted for all 3 zones "zonally" but still be available for a subscription when deployed "regionally".

When restrictionInfo.type=”Zone” it means it is a zonal restriction for the subscription: i.e., not available to deploy this VM size in the subscription in the zones specified in the restrictionInfo properties (and could be all 3 zones)
When restrictionInfo.type=”Location” it means it is a regional restriction for the subscription: i.e., not available to deploy this VM size in the subscription in the region even when not specifying a zone

To Reproduce
Execute "az vm list-skus" on an Azure subscription that has some VM sizes that are restricted zonally in all 3 zones but are not restricted regionally. Review the output to see that the VMs sizes are not listed.
As expected, all VM sizes are returned when using the --all parameter, but the available sizes should also be returned without using --all if these sizes are not restricted regionally as well as zonally.
Expected behavior
VM sizes that are available for a subscription, either regionally or zonally, should be returned by the "az vm list-skus" command even without including the --all parameter.

@zhoxing-ms for awareness

Hello @arsenvlad, we will fix this issue in #23457, could you please help to confirm if it meets your expectations?
https://github.com/Azure/azure-cli/pull/23457/files

Thanks. I left a comment on the PR #23457


Remove test dependency from package:flutter

The mojo_sdk has cleaned up its dependencies, so we no longer need to depend on
package:test from package:flutter.

@Hixie

lgtm


Add image source label to dockerfiles

To get changelogs shown with Renovate a docker container has to add the source label described in the OCI Image Format Specification.
For reference: https://github.com/renovatebot/renovate/blob/main/lib/modules/datasource/docker/readme.md

Hello @wuast94
Sorry, we are not interested in Renovate so far
I close this PR, but thanks for the suggestion 😊 We will re-open this PR if we change our mind

@curquiza its not to add renovate here, its so that repos that run renovate like mine can show the release notes. i run renovate on my repo to make pull requests on new versions.

My baaaad, I just understood, I went too fast. Reopening and merging

bors merge


Teach MetadataReader's demangling procedure to lookup external references to Symbolic Reference contexts

As-is, the code assumes that we are able to read out the SymbolicReference context directly, which isn't true in the case when we are using the ObjectMemoryReader, for example. Then, we may have to ask the reader to resolve the context pointer to see if it points at an external symbol.
Resolves rdar://89966652

@swift-ci please test

@swift-ci please test

@swift-ci please test Linux platform

@swift-ci please test Linux platform

@swift-ci please test

@swift-ci please test macOS platform

@swift-ci please test


feat: add support for root_markers in before_run hook

Draft fix to close #272. Unsure if this type of implementation would be supported. Would like some feedback before cleaning up the PR.
Introduces a 'test#root_markers' variable that could be set to provide a directory marker.
Make sure these boxes are checked before submitting your pull request:

[ ] Add fixtures and spec when implementing or updating a test runner
[ ] Update the README accordingly
[ ] Update the Vim documentation in doc/test.txt

@hay-kot sorry I didn't give you any feedback, last year was a busy one. Is there anything I can help with?


Reorganize Getting Started and concepts Sections in Docs

What this PR does / why we need it:
This PR updates the Litmus Chaos documentation to streamline the user experience and align with the recommended changes from the feature request. It involves reorganizing the Installation/Getting Started section in the table of contents and removing the 'Resources' section. The content from resources.md has now been moved to concepts/chaos-center.md for better organization. Additionally, the 'Getting Started' link on the Doc landing page is maintained as-is, while the 'Installation' page content is moved up to create a standalone 'Getting Started' section in the TOC
For points 1 and 2, see the related PR in the litmus-website-2 repository: https://github.com/litmuschaos/litmus-website-2/pull/121
Which issue this PR fixes : fixes #297
Special notes for your reviewer:
Checklist:

[X] Fixes #297
[X] Signed the commit for DCO to be passed

@andoriyaprashant ,
We already have a chaos infrastructure section, so why not integrate this part into that section?
Rest looks good to me 👍

Done @namkyu1999


Remove twine from production dependencies

Twine is not needed in production deployments, so was deleted from requirements.txt (-e . will install everything in setup.py, which does not contain twine). A requirements-development.txt was added, containing twine and other libraries needed on the workflow to publish to PyPI (so the workflow uses this new file instead of hard-coding every requirement). The python-jose library was also removed from the README, since it was not being used.
My editor automatically removed unneeded trailing spaces also.

Note: since I don't have access to this package on PyPI, I was unable to properly test the packaging process and the pipeline itself.

Hi, @turicas .
Sorry about my delay, but I'll check your PR soon.
Best regards!

@turicas your PR has been merged successfully.
Thanks for contributing it!


examples: ensure the server port matches the client port

Unless I misunderstood the intent, the server port doesn't seem to match the client port defined below.
https://github.com/rsocket/rsocket-js/blob/7b1fe16e50d41935480e0b3e1beb381e8b07c41f/packages/rsocket-examples/src/ClientServerRequestChannelResumeExample.ts#L100
The port mismatch causes the example to fail with the following error.
(node:20091) UnhandledPromiseRejectionWarning: Error: connect ECONNREFUSED <IP_ADDRESS>:8001
    at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1144:16)
(node:20091) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag `--unhandled-rejections=strict` (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 1)
(node:20091) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

Changing the server port to 8001 fixes the example.

That is intentionally. This example suppose to use a tool called socat

Ah, this makes sense, I missed it. 👍


Add Webpack Support

Medium-Editor is webpack compatible and published to NPM.
Would be amazing if this plugin was too
Right now we're forced to use https://github.com/joar/medium-editor-insert-plugin which is fairly outdated

You're absolutely right. Back in a day when I started this plugin, there was a clear division - bower was used for browsers and npm for nodejs. Since then things definitely changed and we need to be on npm and compatible with webpack and amd...
I'll wrap the plugin into this: https://github.com/umdjs/umd/blob/master/templates/jqueryPlugin.js

v2.3.0 was just released and we're finally on npm: https://www.npmjs.com/package/medium-editor-insert-plugin


CocoaLumberjack setup

Hello,
I'm trying to setup KZLinkedConsole with CocoaLumberjack 2.2 using your formatter at KZBootstrap/Logging in an Obj-c project.
Here is my code :
 [DDLog addLogger:[DDTTYLogger sharedInstance] withLevel:globalLogLevel];   [DDLog addLogger:[DDASLLogger sharedInstance] withLevel:globalLogLevel];   [DDTTYLogger sharedInstance].logFormatter = [[KZBootstrapLogFormatter alloc] init];   [[DDTTYLogger sharedInstance] setColorsEnabled:YES];
I had some compilation errors in KZBootstrapLogFormatter.h L87 formatLogMessage:
So I changed
 NSString *formattedMsg = [NSString stringWithFormat:@"%04ld-%02ld-%02ld %02ld:%02ld:%02ld:%03d [%s] %s:%d (%s): %@",                                                       (long)components.year,                                                       (long)components.month,                                                       (long)components.day,                                                       (long)components.hour,                                                       (long)components.minute,                                                       (long)components.second,                                                       milliseconds,                                                       CRLLogFlagToCString(logMessage->logFlag),                                                       CRLPointerToLastPathComponent(logMessage->file),                                                       logMessage->lineNumber,                                                       logMessage->function ?: "",                                                       logMessage->logMsg];
to
 NSString *formattedMsg =       [NSString stringWithFormat:@"%04ld-%02ld-%02ld %02ld:%02ld:%02ld:%03d [%s] %s:%d (%s): %@", (long)components.year,                                  (long)components.month, (long)components.day, (long)components.hour, (long)components.minute,                                  (long)components.second, milliseconds, CRLLogFlagToCString(logMessage->_file), logMessage->_file,                                  logMessage->_line, logMessage->_function ?: @"", logMessage->_message];
but all what I get in console is :
2016-04-13 10:48:48:359 [DBG] »B:376 (»B): my_log_message
any suggestions ?
Thank you !

well you can't mix C strings with NSString *, they are completely different beasts, and you do that for sure given that you use %s while using @"", check your types and match it with formatting expressions and it should be ok


Maps forced update & More leaderboards & Status check

Add a command and api for BN to force updating a map in order to solve some maps stucking in NotSubmitted or UpdateAvailable status.
Add plays and playtime as the sort of leaderboard
Change the update logic when maps update available (old ways ignored the condition that a qualified map changed to ranked map without md5 changes)

also in future, please don’t pr from a master branch and also split this into a PR for each change

hey this looks good! no need to close it - we can keep working on it


Data validation

Would be good to have a staging bucket and copy files from there to the current production bucket after the data has been validated by a data validation utility, for example like this one or similar.
The set of validation checks would address the existing data irregularities issues and cover the space where future issues could develop.

Thanks for the suggestion! This is something we are actively looking into. A couple of notes:

Would be good to have a staging bucket and copy files from there to the current production bucket after the data has been validated by a data validation utility

We already have a staging bucket and copy the files over to the production bucket. The issue with doing validation at this level is that it's all-or-nothing, and we would not want to block updating the entire output because a handful of inputs look off. We also wouldn't want to "fix" data at this stage since it may mask issues with underlying data sources.

for example like this one or similar

That looks very interesting! The claims around performance seem great too. We would be open to implementing something similar, but I would suggest doing it at the individual data source level rather than the final output. If this is something you are interested in, we would definitely welcome your contributions. My only request is having a high-level discussion about design first, before submitting a big PR :-)

The set of validation checks would address the existing data irregularities issues and cover the space where future issues could develop.

We are aware of some of the data irregularities, but please don't feel discouraged from opening an issue anytime you see something that looks off!

Thanks for the feedback!

The issue with doing validation at this level is that it's all-or-nothing, and we would not want to block updating the entire output because a handful of inputs look off.

I’d assume it doesn’t has to be all-or-nothing and that is why the data validation utility I referred to has configurable thresholds. The idea is stop issues like issue #59 where the final output was truncated from containing data for ~200 countries to one country’s data. Unless something major like this happens, the utility will happily return zero exit code indicating success and produce (as a side benefit) error files with rejected epidemiology and index rows for review. The rejections can be treated as warnings.
For instance, right now the side benefit includes flagging the index LY_SR as invalid because it has the country_name field set to SR. The set of checks mentioned in the README is meant to detect regressions related to existing (and fixed) data irregularity issues along with attempting to proactively foresee and cover future problems.

We would be open to implementing something similar, but I would suggest doing it at the individual data source level rather than the final output.

I’d recommend considering both types of checks as complimentary to each other.
When it comes to epidemiology data, most use cases will need both epidemiology and index files because case counts from the former are less meaningful without the ability to attribute those to a particular region described in the latter. Therefore it would make sense to consider both files as a single unit of testing/deployment. Checks on one file (including ones performed at the individual data source level) can be invalidated, as far as end user is concerned, by last minute changes affecting the other file. Performing testing on both files immediately prior to the deployment into production would give everyone more confidence that this won’t happen.

Good point, let me clarify. When I say "all or nothing" I mean that we would not want to selectively filter outputs at the final stage, either we copy them or we don't. That said, it's perfectly valid to run some validation on those outputs, so we can spot issues and fix them at the data source -- the LY_SR key is a perfect example of that.
So the validation can be run with different thresholds, some can be considered an error which would prevent copying altogether (like #59), some can be considered a warning which would result in an issue being open for us to investigate.

If you'd like to go ahead with integrating the utility, then its repository can be cloned/copied and the existing functionality can be used as is, I assume there is no need in PR. Please let me know if this assumption is incorrect or you’d like to change the utility and make it more suitable for testing of the unit comprised of the index and epidemiology files.


Issues in cart modal with large number of assets

Describe the bug
When deleting asset in cart modal (which has more than 100 assets) using inline asset removal button then the cart view does not get refreshed as there is Error 414: "Request-URI Too Long" since there is a GET request for fetching cart partial. Please provide a solution for that.
Environment

AEM Version: 6.5.2
Asset Share Commons Version: 1.8.0
Author, Publish or both: Publish

To Reproduce
Steps to reproduce the behavior:

Go to Asset share home page.
Add more than 100 assets to cart (exact threshold unknown; might happen for 80, 90+ assets as well).
Click on Remove inline with the asset.
Remove disappears but asset remains in cart and the count badge gets reduced.

Expected behavior
Cart should behave the same way when it has a small number of assets.

@davidjgonzalez Can I please have an update on this? as this needs to be fixed ASAP. An ETA for the fix would be helpful.
Thanks,
Karthik

@a-karthik I think this function: https://github.com/Adobe-Marketing-Cloud/asset-share-commons/blob/master/ui.apps/src/main/content/jcr_root/apps/asset-share-commons/components/modals/cart/clientlibs/site/js/cart.js#L59 Needs to be updated to POST request in order to avoid passing each of the Asset paths as query parameters.
It might* be as simple as updating the following but I haven't tested...
      function updateCartModal() {
-            $.get(CART_URL, serialize()).then(function (htmlResponse) {
+            $.post(CART_URL, serialize()).then(function (htmlResponse) {
                ns.Elements.update(htmlResponse, WHEN_CART_UPDATED);
            });
        }

If you can test and file a Pull Request it will help expedite getting this bug fixed!

@godanny86 Thank you for the inputs Daniel. I have tried this but the problem with this is that the modal gets refreshed i.e. loaded from top again. It does not stay at the removed asset position which happens with GET.

@godanny86 Thank you for the inputs. It is working as expected. I will do the same.

@godanny86 Will the fix be released in upcoming version 1.8.1? An ETA for the release would be helpful.

hi @a-karthik , we will look to include this in 1.8.2 and target a release of January 9th/10th...


Lorem.image link to placehold.it doesn't work

Expected behavior and actual behavior
Expecting
In a .haml file I want to put a placeholder with lorem.image with the following:
= image_tag lorem.image('128x128')
I'm expecting an image sized 128x128.
Actual
I'm getting nothing.
The HTML is correctly generated
<img src="http://placehold.it/128x128">
However the image can't be loaded.
The browser can't access the page "http://placehold.it/128x128"
Additional information

Ruby version: 2.7.2
Middleman version: 4.4.2
OS version: macOS 10.13.6

Looks like that service is no longer available. Would accept suggestions for alternatives. https://www.placecage.com/ probably isn't for everyone

Maybe https://placehold.co? They accept custom sizes, colors, fonts, ...

unstale!
@tdreyno as we already commented out a couple of times, the v5 is a bit uncertain at this point, but for those kind of fixes, in which branch should they be fixed (v4 or v5)? Probably more people have the same doubt about contributing in MM.

@markets v4 is probably the quickest way to get a PR released

@tdreyno we can close this one, the service seems to be available again: https://placehold.it/128x128


New package: PyPlotX v0.0.1

Registering package: PyPlotX
Repository: https://github.com/aenarete/PyPlotX.jl
Created by: @ufechner7
Version: v0.0.1
Commit: 1d6668c5a07285d3d280418da81c5c3da9252f1f
Reviewed by: @ufechner7
Reference: https://github.com/aenarete/PyPlotX.jl/issues/3#issuecomment-2002511149
Description: Plot functions for controller design

Shouldn’t this be contributed to PyPlot?

Closing in favor of https://github.com/JuliaRegistries/General/pull/103049


Rework cost estimation gas prices

TODO:

[ ] Rework mumbai gas cost estimation
[ ] Add simulator estimations to request command

We should debug and fix it properly, not use an arbitrary approximation.


Iterators seems to be not designed to run on parallel

I think that I found that iterators are not meant to execute yield function concurrently. Doesn't mean that inside your yield function you execute goroutines. But yield function needs to be executed sequentially. See PR #2

Hey! You are correct. Since version 1.23 the parallel iterator will no longer work and will panic.
Thanks for finding that, I've added a note to the readme!


*: update all dependencies of golang and rust to newest version

Update dependencies:
golang/:

Use libp2p from go-libp2p monorepo, v0.23.4
Update go-multiaddr v0.3.3 -> v0.8.0

rust/:

Update libp2p v0.45.0 -> v0.50.0
Update async-std v1.6.2 -> 1.12.0
Update bytes v1.0.1 -> 1.3.0

thread 'main' panicked at 'Could not find protoc installation and this build crate cannot proceed without

Can you add the protoc installation step to CI in this pull request @elenaf9?

Thanks @elenaf9 for the help here.


Difference Spectra Stored in RadClass

The difference between the count-rate spectrum at all timestamps and an estimated background could be stored as a result of RadClass.RadClass for use as features in future ML model training. I currently do this in post-processing analysis for rejected hypotheses but these "difference spectra" would have to be computed for all timestamps (not just "events") if they are to be used as features. I currently estimating the background by taking the spectra with the minimum gross count-rate in the past 10 minutes:
np.argmin(np.sum(spectra[(evt_idx-span):evt_idx][1:], axis=1))

It would be worth having this directly in RadClass and storing it as another dataset of output spectra (assuming there are not computational/performance constraints on doing so).

Resolved by #37, closing.


feat: add reference for jest-extended in typings.spec.d.ts

Add /// <reference types="jest-extended" /> due to the fact that some editors (ie. WebStorm) do not recognizes the custom jest-extended matchers properly.

PR Type

[x] Code style update (formatting, local variables)
What Is the Current Behavior?
Currently, some editors do not recognize jest-extended matchers.
For example:
TS2551: Property 'toBeFalse' does not exist on type 'JestMatchers<boolean>'. Did you mean 'toBeFalsy'?

What Is the New Behavior?
Editor properly recognize jest-extended matchers without any hassle.
Does this PR Introduce a Breaking Change?

[ ] Yes
[X] No

@kepek Thank you for your contribution !


Class renames for advanced-networking vcloud-director

See https://github.com/brooklyncentral/advanced-networking/pull/116

Looks good

Thinking about this more, given that brooklyncentral/advanced-networking is not part of Apache Brooklyn, maybe these renames should not be included in core brooklyn. It could be the responsibility of downstream projects to do this (if they are bundling advanced-networking).
Longer term, if we support multiple properties files that are aggregated, then advanced-networking can supply its own renames file.


[models] Add keypoint detection models

Currently we perform keypoint detection by:

using a robust image classifier
use its class activation to extrapolate the keypoint

We should explore alternative ways with light backbones to perform actual keypoint detection and train it properly.

Is this issue still relevant ?

Hey @Aym98 :wave:
Yes, so far we've defaulted to object detection, but this is still not ideal. Add comparatively, keypoint detection is a much more adequate task to model. We could train much lighter models for this :ok_hand:


CI: update black to fix click compat issue

See https://github.com/psf/black/issues/2964

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 81.125%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
2063

Relevant Lines:
2543

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

2 of 2 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+2.9%) to 83.991%

Totals

Change from base Build<PHONE_NUMBER>:
2.9%

Covered Lines:
2193

Relevant Lines:
2611

💛  - Coveralls


Create more Requests for Waldur API

As of this moment we only have 2, which means we can only get 2 kinds of data from the Waldur API

After next meeting with the client, we will expand this issue by creating child-issues and we will also track progress here.

Associated functional requirement: 3

All child issues are listed here: https://github.com/WaldurChatbot/Waldur-Chatbot/issues?q=is%3Aissue+is%3Aopen+label%3Arequest

Moved to 4th iteration as some requests should be implemented by then,

Most of them are done.. just 1 (create VPC) is totally unimplemented, issues are in place, no need for this issue.


UI too small / UI scaling?

When trying the Game on my 14 inch Macbook the UI is way too small.
It'd be nice to scale the UI somehow.
Imagine this on a 14 inch laptop:

Just setting Window>Mode to canvas_items in project settings already helped

thanks, I'll check what would be the best way to address that


Add hand tracking box

Add a box selection tool that allows a user to select were the hands should be tracked.

A new branch feature/tracking-box has been created


🐛 BUG: Cannot back up D1 DB with virtual tables

Which Cloudflare product(s) does this pertain to?
D1
What version(s) of the tool(s) are you using?
3.65.1
What version of Node are you using?
v20.14.0
What operating system and version are you using?
Linux Mint 21.3 (Kernel 5.15.0-113-generic)
Describe the Bug
Observed behavior
Wrangler exits with an error message when backing up a database that contains a virtual table.
Expected behavior
Virtual tables should be exported just like any other table, or ignored and the export should continue on tables which can be exported (with a warning). Under no circumstances should it be impossible to perform an export.
Steps to reproduce

echo "CREATE VIRTUAL TABLE location_names_fts USING fts5(name, location_id UNINDEXED, tokenize='trigram');" > pending.sql
npx wrangler d1 execute <database name> --remote --file=./pending.sql
npx wrangler d1 export <database name> --remote --output=db_backup-22-07-2024.sql
See error message: ✘ [ERROR] D1 Export error: cannot export databases with Virtual Tables (fts5)

Please provide a link to a minimal reproduction
Minimal reproduction provided in steps to reproduce
Please provide any relevant error logs
 ⛅ wrangler 3.65.1
-------------------

🌀 Executing on remote database <REDACTED> (<REDACTED>):

✘ [ERROR] D1 Export error: cannot export databases with Virtual Tables (fts5)

If you think this is a bug then please create an issue at https://github.com/cloudflare/workers-sdk/issues/new/choose
✔ Would you like to report this error to Cloudflare? … yes

Attached is workaround for anyone else having the same problem. The attached script will download the database one table at a time, ignoring the virtual table you specify and then concatenate everything into one SQL file.
backup_db.js.zip

User @geelen wrote the following in issue #2471, which has since been closed but seems very relevant to this bug.
Quote:
We only support one kind of virtual table, FTS5, for full text search. But, when porting SQLite's .dump command we found that it would generate SQL requiring writable_schema, a pragma we don't support.
This input SQL:
CREATE TABLE documents (id INTEGER PRIMARY KEY, title TEXT NOT NULL, content TEXT NOT NULL);
CREATE VIRTUAL TABLE documents_fts USING FtS5(id, title, content, tokenize = porter);

Generates the following SQL using SQLite's native .dump command:
PRAGMA writable_schema=ON;
INSERT INTO sqlite_schema(type,name,tbl_name,rootpage,sql)VALUES('table','documents_fts','documents_fts',0,'CREATE VIRTUAL TABLE documents_fts USING FtS5(id, title, content, tokenize = porter)');
CREATE TABLE IF NOT EXISTS 'documents_fts_data'(id INTEGER PRIMARY KEY, block BLOB);
INSERT INTO documents_fts_data VALUES(1,X'');
INSERT INTO documents_fts_data VALUES(10,X'00000000000000');
CREATE TABLE IF NOT EXISTS 'documents_fts_idx'(segid, term, pgno, PRIMARY KEY(segid, term)) WITHOUT ROWID;
CREATE TABLE IF NOT EXISTS 'documents_fts_content'(id INTEGER PRIMARY KEY, c0, c1, c2);
CREATE TABLE IF NOT EXISTS 'documents_fts_docsize'(id INTEGER PRIMARY KEY, sz BLOB);
CREATE TABLE IF NOT EXISTS 'documents_fts_config'(k PRIMARY KEY, v) WITHOUT ROWID;
INSERT INTO documents_fts_config VALUES('version',4);
PRAGMA writable_schema=OFF;

We chose to disallow D1 exports containing virtual tables until we could find a resolution, as we didn't want to allow writable_schema (it requires SQLITE_DBCONFIG_DEFENSIVE to be disabled). Our best option was to skip the create table statements for the generated tables (documents_fts_data, documents_fts_idx, documents_fts_content documents_fts_docsize and documents_fts_config) and instead of the INSERT INTO sqlite_schema call, instead recreate the CREATE VIRTUAL TABLE that created it. The result won't necessarily be identical, but should be functionally equivalent for normal use. But maybe we should explore alternatives.


How to flatten nested array of key/value pairs

I can't get figure out how I'd flatten the nested array.
Is it possible using the module instead of the cli?

json2csv v4.5.1
Node 12.6
CMD: json2csv -i test.json -o result.csv -E -u "LABOR_COSTS_SEGMENT" -F
Dataset:

{
	"LABOR_COSTS_SEGMENT": [
		{
			"LABOR_COST_RECORD": [
				{ "key": "value pairs" },
				{ "each": "in their own object" }
			]
		},
		{
			"LABOR_COST_RECORD": [
				{ "key": "value pairs" },
				{ "each": "in their own object" }
			]
		}
	]
}

Or even just:
 [
	{
		"LABOR_COST_RECORD": [
			{ "key": "value pairs" },
			{ "each": "in their own object" }
		]
	},
	{
		"LABOR_COST_RECORD": [
			{ "key": "value pairs" },
			{ "each": "in their own object" }
		]
	}
]

OUTPUT:
"=""LABOR_COSTS_SEGMENT.LABOR_COST_RECORD"""
"=""[{""key"":""value pairs""},{""each"":""in their own object""}]"""
"=""[{""key"":""value pairs""},{""each"":""in their own object""}]"""

Hi @josh-hemphill ,
So what's the result that you would want to see??

A table of the deep values like:

key
each

"value pairs"
"in their own object"

"value pairs"
"in their own object"

Thanks!
Array flattening is currently not supported so I think that you need to preprocess your data.
const data = {
	"LABOR_COSTS_SEGMENT": [
		{
			"LABOR_COST_RECORD": [
				{ "key": "value pairs" },
				{ "each": "in their own object" }
			]
		},
		{
			"LABOR_COST_RECORD": [
				{ "key": "value pairs" },
				{ "each": "in their own object" }
			]
		}
	]
};

const preprocessedData = data.LABOR_COSTS_SEGMENT
  .map(item => item.LABOR_COST_RECORD)
  .map(([{ key }, { each }]) => ({ key, each }));

And convert the preprocessed data to CSV.

Closing this because of no response. Feel free to reopen if there is anything else left to tackle.


semaphore with dynamic key+number

Summary
Currently mutex allows a dynamic key (example incoming tablename from an argo event). But only allows max 1 wf to consume it.
Currently semaphore requires a static key name (pre-defined in configmap) but allows n number of wfs to consume it.
This new proposal is about having the flexibility from both worlds. ie a semaphore that allows dynamic key (not pre-defined in configmap) and ability to specify n wfs to consume it.
I imagine on a sensor it could look like:
  synchronization:
    semaphore:
      name: "{{=...parse_tbl from sensor message}}"
      maxInParallel: 3

Use Cases
To dynamically rate limit the platform, avoid new tables being resource hogs

Hmm I think this proposal has a bit of an issue -- maxInParallel is defined on one Workflow in the example, but it can actually be defined on any Workflow.
The ConfigMap for semaphores acts as a single source of truth between all Workflows that use the same semaphore.
I suppose this could be rewritten where only the number is taken from a ConfigMap, but that is perhaps a bit awkward/unintuitive. Might be good to think of a more ergonomic API for this

maxInParallel in this case could even be specific to the workflow for my usecase. don't mind having the wfprefix name appended to the semaphore name, the key thing being that a new key name (ie table value could appear in sensor expression) but me not having to statically define that name beforehand

maxInParallel in this case could even be specific to the workflow for my usecase

Hmm that is a slightly different use-case. That straddles in between the current parallelism feature and semaphores.
But yea I understand the use-case and do think it's not altogether that unique and therefore broadly useful.
The ergonomics of the API and how that compares to existing APIs would be the main challenge now.
If there are multiple Workflows, then the ConfigMap I still think makes the most sense and would be most familiar.
Perhaps valueFromKey while key can be overwritten. I don't really like any of the options I proposed so far, so if you have any ideas on that, please do list them!

Dynamic keys were also suggested in https://github.com/argoproj/argo-workflows/pull/7302#issuecomment-982917593


Some icons are not being displayed properly

box-icons: boxBxsDroplet and boxBxsHot
Fluent UI: fuiFlagPride16Filled, fuiFlagPride20Filled, fuiFlagPride24Filled, fuiFlagPride28Filled and fuiFlagPride32Filled
Hero icons (solid): hisFolderAdd, hisFolderDownload and hisFolderRemove
Iconoir icons: ico1stMedal, icoAddKeyframeAlt, icoAddKeyframes, icoAddMediaImage, icoAirplaneOff, icoAlignBottomBox, icoAlignLeftBox, icoAlignRightBox, icoAlignTopBox, icoAntennaOff, icoArrowArchery, icoAttachment, icoBbq, icoBicycle, icoBluetoothRounded, icoBluetooth, icoBorderBottom, icoBorderInner, icoBorderLeft, icoBorderOut, icoBorderRight, icoBorderTl, icoBorderTop, icoBoxIso, icoBubbleStar, and many more (black dot in top-left)
Pixelart: pixGift and pixSliders2
Tenny Icons: many, similar to Iconoir (black dot in top-left)

Fixed in v1.2.2


Add global variable to be able to change download url

example:
set -gx $FIN_MIRROR ...

#22

This looks great @bjmin, I want to merge this right away.
Do you know how tj/n or creationix/nvm handle mirrors? Do they also use a variable in this way or another mechanism?


Multiple Orgs

Hi, does this plugin work with multiple organisations within Grafana, or is only one org supported?

It should work with multiple organisations. The plugin shouldn't have any limitations with multi org setup.
Do you have any issues with multi org setup? If so, please report them here so we can try to fix them.

I haven't tried it as of yet. Do I just add another block in the reporter.yml file with another org, like so:
apps:
  - type: mahendrapaipuri-dashboardreporter-app
    org_id: 32
    org_name: Test
    disabled: false
    org_id: 33
    org_name: Test 2

Also I have another question about the logo. It is base64 encoded, but shows up like this on my generated report. Is there anything I'm doing wrong?

Provisioning should be rather like below:
apps:
  - type: mahendrapaipuri-dashboardreporter-app
    org_id: 1
    org_name: Main Org.
    disabled: false
    jsonData:
      appUrl: http://localhost:3000

  - type: mahendrapaipuri-dashboardreporter-app
    org_id: 2
    org_name: Main Org.
    disabled: false
     jsonData:
      appUrl: http://localhost:3000

About the logo, the encoding should be of a PNG file. If you still have issues, you can enable Persist Data Files in plugin config in Grafana UI which will save all the templated HTML files in $GRAFANA_PATH_DATA/reports/debug folder and there you can inspect the footer.html if logo has been correctly embedded or not.

I enabled the Persist Data Files, but no footer.html is created only report.html. I am encoding a PNG file and putting its path in the report.yml file like /etc/grafana/provisioning/plugins/logo.

Yeah, you are right. There wont be any footer.html file. The templated footer will be passed to chromium directly without saving to filesystem.
You will to put the actual encoded string in the report.yml file. For example it would be somthing like
apps:
  - type: mahendrapaipuri-dashboardreporter-app
    org_id: 1
    org_name: Main Org.
    disabled: false
    jsonData:
      appUrl: http://localhost:3000
      logo: iVBORw0KGgoAAAANSUhEUgAAABcAAAAXCAYAAADgKtSgAAAAAXNSR0IArs4c6QAAAAlwSFlzAAASdAAAEnQB3mYfeAAAABl0RVh0U29mdHdhcmUATWljcm9zb2Z0IE9mZmljZX/tNXEAAAVbSURBVEhLbVVdbFRFFP7mzty9+9N2W4NIqQq2pRSt+OS/IMS/8EAMEtTEYOKLIT7x4oMPZLNBY2LiC09qYngwJhoIiRhDCKIhNIRISEAKpaWlLVBopZXdbXe3vT8zfnO3W37sZCd378zcc77znW/OUfl8HkuN3LjX6KBlo9TO6zDYZIzp4rk0hJgVBgNCiD+1ML+Ho/pkfsNEZSkb6sHFXN+KrGrU25WT2AblrTESWYR+E6IoCWjAOCnjyG7juithzFb1hH/pi5EVh/zy1cP5nnTpXnv3Gd871NyhmtQnJpF6Rxi9GsKhsYiz/ongH7oTyNBwBkK2QibWRjK1Xsk1PV9eLnzzWXd5tH560XhuaFmXTGV2C+V9yJAzCOaBMLDGOK3R+uB75HOP0667CQg30QEnsyuUyssNufvynYURezo2Tn4fUl7qY9Kwk5xmTLWEKAiBgKhdCekqLi/4iX3wpe4vtCDoKNWU5fcfKS9TzvUlv8r3TJRi48rPvm089b4QToOpFInVIEmDTWkPpTBCoO2KuA//3UgWvBAQ0tmscdydiXR0gfs/q9z4uqxyZnbQcBv8CjTRphISm3oexebulei9MoHjf1/DrB9Bcf0u/w9KgXT5VaZJPq5d573c+Eu/KieceYW8dcTJiwIYGm9s8LB5XRs+eLETmYTCX/03MVusIpTJGnpjIwEccuU4dX74tPQkGwHprVPR6HNKOnqLEWiGJr82eE+i6Ic42ncdPinpHbyFAt9phcjCBeA0zZ+2iaBzR/JpvdFpbEeIZv5/U0Gb1xCFdEcNE5c9I4lmjhGMTs2iWAmQTiWwYW0r0jR0404ZK5vTSCqJC+N3MDhZ5OcaUtK5dWaVxNwJIzYpou7kbo1Mu0l0LQ1JbH1mFTbS4LG+G6gEAbY9244NXa2oMppljSkYrXH5ZgGfHz6LU4zOxh07iJFHSVrrsjEp3r672Yk0EkS+5pEmvNCxHFdvl5CictpaMniqrQXnx6bQe+kGXmayt6x/DGfHbuMS1wplSpIRxsMYG0aGtEQkVFjktcwQfUS35fkQFUZR4TPkgqYc7fu+387h4Kkr+PTd57H7rafR/nATskkXBSZ8UatCaBJRVixCQ0aqVYwnVUvq0sPmYZroxqdmUCpVcX16FhEdSoK59/6SCcv5nNDBIGkRxyHVDnpKIaL3GnhQRovTSs4a95hE11MsJxIeZ3ymLsX6DZaWGjNrnPAPFWnniGPwBj0ujyFQTvFV5/CpAjusCgPmwtIVI11Aa/ftzbXOYxnap0Ue+QU40TGlVWOvE8wMI+F1sYiwlviYCTRODNzEXBjGN3SSNJwYuIXR6TLGChUY1pt+KuXgmWGcG5vm+VoNguL3mimM/P7QaT+j8m39xb0jyw8Y464XiXSb9HyUmLiDp4dw9OxI/H+WCH88ORCraHqOlTKZwOmhSQxen0aZhu0Z6ZEOlxL1q9dkgJ/y7acqceEKE8VflE49SfHuEplsQ1guYqpYwZSlhYgEpThZLhMVQ6cyFLmfqTLCAtfIu0NnTibLfV0UOvzB9/QRazc2nm+b/zc3Nv+dDHRGqOROJ9Pc4LjsXPfW87RN1AK3RpMBJsJlc1Je7anDEmvL/nC+/D3r+cyi8djBqokr7ERfwzMV46a2C0ethscPbSeyThZqT+zA5oZ3r3Y9mNQoHBZh5YAqF7/ds1Qnsg72dBaGc32te1XmzkXqjD002UUFZDlZeyhVNlB7tymROa6VaJVog4tSzx/yVfHwnu75GHF9/K9B53v6i9zcz0Z9wMlW2f0pUyFetbXCXmkIU+UFGRB+wO4vFrr/P0t2//8A3e1remyXHJwAAAAASUVORK5CYII=

You dont need to include the header data:image/png;base64 as it will be included automatically by the plugin.

Thanks the logo appears now. The issue was that I put the path to the base64 encoded file in report.yml instead of just the encoded string. Is there any way I can make the logo any bigger?

Is there any way I can make the logo any bigger?

Unfortunately, no. As placing panel PNGs in the report is always tricky with CSS magic, the logo size is hard coded to the size of footer to avoid distortions.

Hi, how do I customise the text in the report. What file do I make this change in?

Hey, there is no way to customize the report content as of now. There are few features in pipeline that can allow the users to be able to do this.
If I may know your use case, what exactly you would like to change? This helps us to better draft the feature.

I am looking to add like a cover page to the report with my logo, name, title of the report etc. Also maybe customise the text at the top of every page so that it doesn't show and only the images are on the pages.

We will definitely allow the users to customize the header and footer of the report so that you can remove the title from each page. Regarding the cover page, we will look into it if we can support it. If not you can generate the cover page with third party tools and merge it with generated report.

The issue arises when dealing with multiple organizations. For example, if you have the following organizations:
1.	Main.org
2.	Second.org

The issue stems from the service account token. Initially, Grafana sets up the service account for the default organization, which in this case is Main.org. You can verify this by checking the service accounts on the UI. However, when you add another organization, that doesn’t happen automatically.
Now, if you switch to Second.org and attempt to generate a report, you’ll encounter the error: “permission denied”. You can resolve this by manually adding a service account under the second organization and then adding the token in the plugin settings. So, the issue is more of a persistent one.
By checking the repository, I can see that the developer is aware of this problem. Here’s how you can generate a report using Postman by adding the correct Bearer token. This is why they added the following code:
case conf.Token != "": 	ctxLogger.Debug("using user configured token") 	credential = client.Credential{ 		HeaderName:  backend.OAuthIdentityTokenHeaderName, 		HeaderValue: "Bearer " + conf.Token, 	}
You can find this code under pkg -> plugin -> resources.
To understand the issue, you need to grasp how the plugin’s anatomy works. When switching between organizations, the plugin’s backend doesn’t reload. This means the token needs to be correctly managed for each organization. You cannot fetch a dashboard from Second.org while using the token from Main.org. So, adding another service account under Second.org is a permanent solution, but when you reload the container, the token is lost.
Now, someone might suggest adding another token within the plugin’s context. No, the problem arises because when fetching via the HTTP API http://<IP_ADDRESS>:3000/api/dashboards/uid/<dashboard_uid>, it doesn’t return the organization to which the dashboard belongs. You already need the correct token to even retrieve that information via the API.
To provision the organization which doesn't work. The response from @mahendrapaipuri :
`apps:

type: mahendrapaipuri-dashboardreporter-app
org_id: 1
org_name: Main Org.
disabled: false
jsonData:
appUrl: http://localhost:3000

type: mahendrapaipuri-dashboardreporter-app
org_id: 2
org_name: Second Org.
disabled: false
jsonData:
appUrl: http://localhost:3000 `

The only potential solution I can think of is to add the organization in the URL, which could allow us to retrieve the corresponding token. However, this would only be feasible if Grafana generates a separate service account token for each organization.

Hey @Davidkramer1999
Thanks for this detailed report. I confirm all your findings and Grafana developers have acknowledged this issue as well. Please check the issue in Grafana repo. Until someone fixes that upstream, we can use the following workaround:
Whenever an organisation is created in Grafana, create a service account token "manually" using Admin as scope. Then we can use this token in provisioned config for the plugin as follows:
# Assuming that externalServiceAccounts feature flag is enabled. Grafana
# will create a token for Main Org. automatically. However, it wont for
# for any additional Orgs. So, we need to create a token with `Admin`
# scope manually and provision it with the plugin.
apps:
  - type: mahendrapaipuri-dashboardreporter-app
    org_id: 1
    org_name: Main Org.
    disabled: false
    jsonData:
      appUrl: http://localhost:3000

  - type: mahendrapaipuri-dashboardreporter-app
    org_id: 2
    org_name: Main Org.
    disabled: false
    secureJsonData:
      saToken: <myscrettoken>
    jsonData:
      appUrl: http://localhost:3000

This will ensure that the token survives Grfana reloads/restarts. The inconvenience is that every time we create a new Org., we need to create a token manually and re-provision the plugin. But this has to be done only once.
Please let me know if it works for you.

Thanks for the reply!
The main challenge is that I cannot recreate the organizations because we already have so many users on Grafana. So, I’ll likely have to manually create the service token for the new organization.
Here’s my current approach:
1.	Manually create a service account token for the second organization (Second.org).
2.	Store the second token on the backend.
3.	Include the orgId in the request.
4.	On the backend, use logic to switch between tokens: if the orgId is for Main.org, use the Main.org token; otherwise, use the Second.org token.

On the backend, use logic to switch between tokens: if the orgId is for Main.org, use the Main.org token; otherwise, use the Second.org token.

Does it mean the plugin is always using the "manually" configured token from Main.Org when you are in Second.Org?

I’ve found a different solution that involves saving multiple tokens in the backend context. When making a request, you can include the active organization ID in the URL parameters, and then use the corresponding token.
Here’s how I’ve implemented it:
1.	In App.go: (This code is adapted from your previous version.)

`var secrets Secrets

if settings.DecryptedSecureJSONData != nil {
if saToken, ok := settings.DecryptedSecureJSONData["saToken"]; ok && saToken != "" {
    object := map[string]string{orgId: saToken}

    // Assign this map to secrets.tokens
    secrets.tokens = object

    secrets.token = saToken // Also set the general 'token' field as a readily available default token
}

}`
2.	When you need the token:
`grafanaConfig := backend.GrafanaConfigFromContext(req.Context())

if token, exists := a.secrets.tokens[orgId]; exists && token != "" {
// Token exists for the given organization ID, use it
ctxLogger.Info(fmt.Sprintf("Using specific token for organization: %s", orgId))
a.secrets.token = token
} else {
// No token for the given organization ID, attempt to get a new token
saToken, err := grafanaConfig.PluginAppClientSecret()
if err != nil {
ctxLogger.Warn("Failed to get plugin app secret", "err", err)
// If no new token and no existing general token, handle absence
if a.secrets.token == "" {
ctxLogger.Warn("No active token available")
a.secrets.token = saToken
}
}
}`

Hey @Davidkramer1999 Thanks a lot for the feedback and sorry for the delay. Was quite occupied these days.
Seems like there is a little bug while fetching correct token during permissions check. An attempt to fix it is in #147
The plugin works with multiple Orgs as long as you disabled externalServiceAccounts feature flag completely and provision the plugin with a service account for each Org. For example, a basic config for the plugin app in multiple Orgs look like this in here.
I have tested it locally and it seems to work. Could you please test it on your deployment?
Cheers

@Davidkramer1999 Seems like Grafana explicitly mentioned that externalServiceAccounts are supported only for Single Org deployments in the latest v11.3.0 docs. So, I guess it is better to avoid externalServiceAccounts if you are on Multi Org setup.


Load accelerators from the resource file

Support loading of accelerators from the resource file, rather than manual creation using CreateAcceleratorTable()

This function already exists.

Hi @rodrigocfd,

This function already exists.

True, but it returns HACCEL, not DestroyAcceleratorTableGuard. Currently my code looks like this:
    let accel_table = HINSTANCE::NULL.LoadAccelerators(w::IdStr::Id(123)).unwrap();
    let wnd = gui::WindowMain::new(gui::WindowMainOpts {
        accel_table: Some(unsafe { DestroyAcceleratorTableGuard::new(accel_table) }),
        ..Default::default()
    });

Is there a way of doing this without unsafe code?

I didn't implement it because, according to the documentation:

Accelerator tables loaded from resources are freed automatically when the application terminates.

Do you have an use case where this won't apply?

It doesn't matter if DestroyAcceleratorTable() gets called on my accelerator table or not. Like you say, it'll get freed when the app terminates.
I just need to be able to pass my accelerator table, for which I only have an HACCEL, to WindowMainOpts. WindowMainOpts only accepts DestroyAcceleratorTableGuard, not HACCEL. So I need to convert my HACCEL to a DestroyAcceleratorTableGuard. Unless there's some other way?

WindowMainOpts only accepts DestroyAcceleratorTableGuard, not HACCEL

Yes, that's a valid use case I didn't notice.
But instead of having the same function implemented in two different places, make another PR changing the return value of LoadAccelerators, so it always return a DestroyAcceleratorTable. The I'll merge your PR.


Semantic tokens for formatting directives in strings

The Go textmate grammar applies a special coloring to formatting directives (%s, %d, etc.) in strings. For example:

We were wondering how we can achieve the same behavior through semantic tokens implemented through LSP. Logically, it makes sense to apply one of the SemanticTokenModifiers to the relevant parts of the string, but that did not appear to produce a difference in colorization, and it's not clear which modifier best fits this specific use case.
/cc @pjweinb

The Go textmate grammar applies a special coloring to formatting directives (%s, %d, etc.) in strings. For example:

We were wondering how we can achieve the same behavior through semantic tokens implemented through LSP. Logically, it makes sense to apply one of the SemanticTokenModifiers to the relevant parts of the string, but that did not appear to produce a difference in colorization, and it's not clear which modifier best fits this specific use case.

I would expect this to work. Have you tried making it a completely different token type like "variable" or "function" to double check that the offsets in the payload is correct?

Which client(s) have you tested this on and verified that it is not working?

I would expect this to work. Have you tried making it a completely different token type like "variable" or "function" to double check that the offsets in the payload is correct?

Yes--and different token types do work.

Which client(s) have you tested this on and verified that it is not working?

This was originally tested with the latest VS Code language client, but the same behavior also reproduces with the 7.1.0-next.4 version. I confirmed that VS Code saw the modifier here:

@stamblerre Maybe try making changes to your theme?

Adding that customization does seem to work. I am using the default VS Code dark theme with VS Code Insiders. Is there any way to get these modifications to show up by default? This is preventing us from switching completely to the LSP's semantic tokens in the Go extension.

Adding that customization does seem to work.

That is good to hear.

I am using the default VS Code dark theme with VS Code Insiders. Is there any way to get these modifications to show up by default? This is preventing us from switching completely to the LSP's semantic tokens in the Go extension.

I suggest opening an issue in the microsoft/vscode-languageserver-node/ repository or with Visual Studio Code (since you are using the default theme). This seems like a theming issue and/or VS Code doing things differently for TextMate grammars vs LSP semantic tokens.

IMO this has nothing to do with the LSP client.
@aeschli can you comment on why this might not work for semantic tokens but does work for syntax tokens.

Yes, this would have to be filed against VSCode. You want VScode to add a TextMate scope mapping for string.modification.
string.modification is not (yet) an established way to represent placeholders in strings, so I'm not sure we will just add that in VScode. But extensions can also such TextMate scope mappings: https://code.visualstudio.com/api/language-extensions/semantic-highlight-guide#custom-textmate-scope-mappings (second example)


Trimming output?

Hi!
Thanks for creating Foil. It solves a big issue I had with Plates: lack of multi-folder support and so creating skinnable templates with Plates was not possible.
One thing with Foil that keeps biting me is this line which is:
    private function doRender($path, array $data = [], $class = null)
    {
        // ...
        $output = trim($template->render($data));
        // ...
        return $output;
    }

Why trim()? Can this not be left up to the developer as desired? E.g. me! Or made an option? One of things we use foil for (as well as HTML templates) is building plain text configuration files and whitespace is important here.
Thanks, Barry.

Hi @barryo the trim was added to make comparing templates easier in unit tests, but I see more people using Foil as code builder, which was not what I had in mind, but fine :), so it makes sense.
Spaces are not a big issue in PHP so I can remove that trim without backward compatibility issue.
Don't really have time right know, happy to merge a PR to dev if you have time to send it.

Done with 1169c54, thanks @barryo

Hey @gmazzap - any plans to merge this into master / release?
We're shipping IXP Manager with a foil-dev requirement for a while now.
Thanks a million,
Barry

I will merge very soon.

Thanks for that @gmazzap !


Turn printer on with Tasmota plugin

I use the Tasmota plugin to turn on my printer when I want to start a print.
I would very much like if thie PrintScheduler plugin would turn the printer on using Tasmota, instead of me turning on the printer manually. Maybe there is an option for that in the "System Command Before Each Print", but I can't find any documentation on that.

I found some information here: https://github.com/jneilliii/OctoPrint-PrintScheduler/wiki/Helpful-System-Commands
Didn't look good!


⚠ Bofrost.at has degraded performance

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

HTTP code: 200
Response time: 2048 ms

Resolved: Bofrost.at performance has improved in 3bddab4 after 43 minutes.


Fixed contributing.md bug

@keshavgbpecdelhi please merge my PR

Thanks for contributing @aakashgarg-19. I was waiting for this 😄 wrt #78


Folder uploading

Is there a way to upload entire folders?
I had to implement a recursive function to make this work and i was wondering if the API has a way to do this.

Yes, see https://github.com/ferristseng/rust-ipfs-api/commit/0cf5be505470b2bcaf837d93239283be1f9a4a1e
-------- Original Message --------
On Oct 30, 2019, 20:54, Eduardo Flores wrote:

Is there a way to upload entire folders?
I had to implement a recursive function to make this work and i was wondering if the API has a way to do this.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Thank you! This is perfect!


RSDDs: Rail Surface Defect Datasets link does not work

RSDDs: Rail Surface Defect Datasets link does not work. Can anyone send me a working link for this code?

Download Link：pan.baidu.com/share/init?surl=svsnqL0r1kasVDNjppkEwg (password：nanr)
https://github.com/Charmve/Surface-Defect-Detection/blob/master/README.md#L305

icn.bjtu.edu.cn/Visint/resources/RSDDs.aspx this link was not working. Also since I am outside china, baidu app does not let me register or download from that link


Fix social media links in footer

There are links in the footer to social media channels.

Remove everything other than Github and Twitter
Link Twitter to https://twitter.com/wottsecurity
Linkup Github to https://github.com/WoTTsecurity

This complete in PR - https://github.com/WoTTsecurity/wott-io/pull/25

closes #15


Manual check that tutorial labels are correct, before soft launch

We should do a manual (visual) inspection that tutorials are labeled properly and take note of any potential missing labels before the soft launch.
I recommend fixing any wrong labels or cross-references before the soft launch.
I recommend that missing labels be addressed after the soft launch.

I discussed this with @jonathansick during our meeting today and it was noted that in order to update the content labels we will need to update the tutorials themselves. (Astropy Librarian pulls the labels from the keyword terms.) So this will likely need to happen after the soft launch. I updated the header.
Also needed: Look at description (from first paragraph of the summary section) and sometimes it doesn't work well. The summary can get cut off, formatted weirdly, or can have issue for tutorials that don't follow the standard structure.
I will use this issue to take notes on these needed changes.

Label issues

There is a label under Tasks called "fits" -- this should be in all caps (FITS) because it is an acronym. Otherwise, people might click on lowercase "fits" when they are looking for tutorials on model fitting.
Same issue for "latex" -> "LaTeX" although it is less likely to be a source of confusion
Similar issue for "wcs" -> "WCS", is there a reason all the labels are lowercase?
Astronomical Coordinates 1 - "gaia" keyword term is missing from side search bar
Using scipy.integrate - Add "scipy" to the keywords (or use Astropy Librarian to draw this directly from the import commands)
Read in catalog information from a text file and plot some parameters - consider adding a keyword called "projections" because it shows how to plot coordinate information on a mollweide projection with matplotlib. I think there is another tutorial that does this?
Read in catalog information from a text file and plot some parameters - Some tutorials like this one use from astropy.io import ascii. How would Astropy Librarian read that as a keyword? Should it be listed as an "io" or "tables" subpackage? I note that the UVES tutorial lists "astropy.io" as a keyword term, but there is no "io" subpackage in the search list. So perhaps "Read in catalog information" just needs the "table" keyword.
Similar question for FITS files, how should we handle package labeling for from astropy.io import fits ?
Working with FITS-cubes - "data cubes", "spectral cubes", "reproject" and "contour plots" are all missing from the search terms at left
Searching major star catalogs - has the keyword term "SIMBAD", not represented in the search box. Do we want to keep it?
Modeling 2 - has the keyword "Vizier", not represented in the search box. Do we want to keep it?
Perhaps we could make the keyword "catalogs" to cover both? Where should it go (science domains)?
Note that the "observational astronomy" keyword can apply to the majority of the tutorials. Should we toss it? Otherwise, expand its use.
Analyzing interstellar reddening - uses astropy.visualization, add "visualization" to the keywords and list it under Astropy Packages

Description text issues

Synthetic images from simulated data - description text is cut off
Using scipy.integrate - "units" is listed twice under keywords
Analyzing interstellar reddening - remove "dust extinction" keyword because "extinction" is also listed (and represented in the side bar)

Description text: suggested modifications

Modeling 2 - the description text is sparse.
Working with FITS cubes - the description text is sparse.
Make a plot with both redshift and universe age axes using astropy.cosmology - description text is ample but cuts off with a colon. Is more information needed?
Viewing and manipulating FITS images - learning goals switches from bulleted text to a numbered list midway through

Here's the sheet I was using to keep track of the taxonomy. It's in the Keyword Table tab. Feel free to edit directly! https://docs.google.com/spreadsheets/d/1JmOF82Zmrs397wLwAmmL1rgDSsvP6Z4fLMr0hV5ZIb4/edit#gid=953884002


PR: Remove extra blank lines when running statements with input() (IPython console)

Description of Changes

The main change in this PR is removing extra blank lines when running statements with input() in the IPython console.
Add subrepo for Qtconsole because this introduces a breaking change in one of the methods overrode by us in RichJupyterWidget.
Update our requirement on Qtpy because the next Qtconsole will require 2.0.1+
Update contributing and release instructions to account for this new subrepo.
Depends on jupyter/qtconsole#501

Issue(s) Resolved

Fixes #16504
Affirmation
By submitting this Pull Request or typing my (user)name below,
I affirm the Developer Certificate of Origin
with respect to all commits and content included in this PR,
and understand I am releasing the same under Spyder's MIT (Expat) license.

I certify the above statement is true and correct: @ccordoba12

Yes, let me do that so you can merge this one.


SubmitExtrinsic error:  kind: Unreachable

When I encountered this error when submitting my transaction, I am not clear what the specific error is and which parameters are not in compliance with the requirements? Where can I know? Thank you!
let tx = '0x210281ff88ccfe1c52268e4f5dfb6228e71b2cff7a24446fad25fcd2bc3481fad2c63ccf00225c93ef4919723b8b5b368461734f4e10b88842fb5a1c718f91b756d1784c24fd6967e009e80bf2138172f61be4a9d969b290be276a52b96c0d45584337c104000600ff47c9440935f46211a67b04514420dec0bbc79d7751b3757fb0e0cc164b6d2b1e1c';
let ret = api.rpc.author.submitExtrinsic(tx);

 RPC-CORE: submitExtrinsic(extrinsic: Extrinsic): Hash:: 1002: Verification Error: Execution(Wasmi(Trap(Trap { kind: Unreachable }))): RuntimeApi("Execution(Wasmi(Trap(Trap { kind: Unreachable })))

This transaction I created when assembling polkadot offline transaction body
     //1 create tx
        let tx = registry.createType('Extrinsic', decorated.tx.balances.transfer(to, 7));
        let data = {
            address: from,
            blockHash: genesisHash,
            blockNumber: '',
            era: '',
            genesisHash: genesisHash,
            method: tx.method.toHex(),
            nonce: nonce,
            runtimeVersion: runtimeVersion,
            signedExtensions:signedExtensions,
          //  specVersion: runtimeVersion.specVersion,
          //  tip: '',
          //  transactionVersion: runtimeVersion.transactionVersion.toString(),
            version: version
        };
        //2.create signerPayload
        let signerPayload = registry.createType('SignerPayload', data);

        //3.create extrinsicPayload
        let extrinsicPayload = registry.createType('ExtrinsicPayload', signerPayload.toPayload(), {version: signerPayload.version});

        //4.sign
        const {signature} = extrinsicPayload.sign(pair, {withType: true});

        //5.add sig
        tx.addSignature(from, signature, extrinsicPayload);
        console.log('tx', tx.toHex());

And I got the tx.toHex() and submitExtrinsic, I encountered some problems when assembling the transaction body, but no one answered me in polkadot-js/api, thank you very much for your willingness to listen to my questions and reply to me


headers not getting proper width if first column header is empty [work around included for webkit]

I have a table that I just added a new first column to. The column is a time scale for the rest of the table so it doesn't have any content in the first header entry.
This causes the table headers to be the wrong size - default size? And it screws up the rendering of the whole table.
I didn't debug the code to figure out why but I did think about a work around. I put &nbsp as the content of the first column header so that it now has some content. This worked around the problem fine.
You might want to put this as a note in the docs.
I just tried this in Edge and Firefox. It didn't work there. In both those browsers the floating headers appear but they have narrow widths and they get inserted into the DOM in such a way that they affect the rendering.
Another workaround I just thought of was to put in a letter that has the same colour as the background. Although I think you will see it covering up the contents of the first column as it scrolls.

it turns out that this might have been related to the problem with setting up sticky headers on a dynamic table. When I added the extra column, my check to see if the table had been setup properly was no longer valid. I was checking to see if there was at least one header column in the table, if there wasn't, then I would wait and check again. The problem was that in adding the time scale, there was always one non-dynamic column.
I changed the test and the sticky headers started working again. I also put a width element on the empty header so that may have contributed to it helping as well.


Improve the help of the "transaction view" arguments

Internal/External
Internal   if an IOHK staff member.
Area
Other   Any other topic (Delegation, Ranking, ...).
Summary
When executing cardano-cli transaction view, it is not clear that:

--tx-body-file expects a tx body (raw\unsigned transaction)
--tx-file expects a signed transaction
the JSON keyword from the description of the arguments could be misleading too as the command expects a tx file (and not a json file)

Steps to reproduce
Steps to reproduce the behavior:
# cardano-cli transaction view
Usage: cardano-cli transaction view (--tx-body-file FILE | --tx-file FILE)
  Print a transaction

Available options:
  --tx-body-file FILE      Input filepath of the JSON TxBody.
  --tx-file FILE           Input filepath of the JSON Tx.
  -h,--help                Show this help text

Expected behavior
A clear and concise description of what you expected to happen.
System info (please complete the following information):

OS: [e.g. Ubuntu]
Version [e.g. 18.04]
Node version

# cardano-cli --version
cardano-cli 1.25.0 - linux-x86_64 - ghc-8.10
git rev 58c1e06f3618b546627916bd24649d5cecf63ff8

# cardano-node --version
cardano-node 1.25.0 - linux-x86_64 - ghc-8.10
git rev 58c1e06f3618b546627916bd24649d5cecf63ff8

The wording of the error messages could also be improved:
# cardano-cli transaction view --tx-file tx-mask.body
Command failed: transaction view  Error: tx-mask.body: TextEnvelope type error:  Expected one of: TxSignedByron, TxSignedShelley, Tx AllegraEra, Tx MaryEra Actual: TxBodyMary

I think the issue is that I provide a raw tx file instead of a signed one but the error is quite misleading.
Suggestion --> Expected a TxSigned; Actual: TxBodyMary

If the undocumented convention of --tx-body-file for a raw/unsigned transaction and --tx-file for a signed transaction is not kept, the error message is also not helpful in determining the real problem:
$ cardano-cli transaction view --tx-body-file withdraw.signed

Command failed: transaction view  Error: withdraw.signed: TextEnvelope type error:  Expected one of: TxUnsignedByron, TxUnsignedShelley, TxBodyAllegra, TxBodyMary Actual: TxSignedShelley

Closing this. If this is still relevant please reopen.


[Diaper Calculator] Different sizes of containers on the old design Calculator page

Environment: Windows 11 Home Version 22H2 Build	22621.2506
Chrome Версія 118.0.5993.118 (Official build) (64-bit)
Reproducible: always
Build found:
Preconditions

https://zero-waste-staging.onrender.com is opened in browser
User is logged in as admin
Admin tab is opened

Steps to reproduce

Go to Site settings page
Uncheck Enable new calculator design in Site features container
Click on Save button
Go back to ZeroWaste home page with clicking on logo at the left upper coner
Ckick on Let's go button

Actual result
Old design calculator page is opened with different sizes of containers on it

Expected result
One size containers  of old design calculator are displayed on Diaper Calculator page
User story and test case links
Labels to be added
"Bug", Priority ("pri: medium"), Severity ("severity: low"), Type ("UI").

Fixed


Power Platform - Azure API Management connector premium vs standard

When I export an Azure API Management API to an environment it is seen as standard within a D4T environment, but it's premium in a 'normal' Power Platform environment. Why is there a difference between those two types of environments, because you also have to pay for the API Management resources, why also handle it as a premium connector.
Also nothing is mentioned regarding premium vs standaard within this documentation/article.

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

ID: 83f8200f-95b3-42b0-9e95-f803e13d3952
Version Independent ID: f0fc9512-4abe-73de-bcf4-24fad32eea88
Content: Export APIs from Azure API Management to Microsoft Power Platform
Content Source: articles/api-management/export-api-power-platform.md
Service: api-management
GitHub Login: @dlepow
Microsoft Alias: danlep

@rbarten Thanks for your feedback. We will review and update as appropriate.

@rbarten - Thanks for opening this issue and apologies for a delayed response. The details of licensing options for Power Platform and the treatment of connectors are beyond the scope of this article but are introduced here. We'll add a link to this article which I hope will help.
#please-close


Linq: Use a list of conditions

Introduces IConditionItem that represents a CQL relation in Linq. This patch separates Linq expression parsing and CQL generation, to allow more complex conditions in the future.
Fixes CSHARP-498 by introducing conversions when generating parameters.
Its a large changeset but helps to make CqlExpressionVisitor more readable where possible (expression tree evaluation is usually nested, so there's that limitation to consider).

I've added a small fix to invert the condition of "yoda" expressions and a small fix for the test.


Update webpack configuration

Changes

Update webpack to 5.3.1
Update webpack-stream to 6.1.0
Update webpack config to exclude node_modules

How to test this PR

Pull branch and ./frontend.sh. Site should have no JS errors.

Curious that npm-packages-offline-cache/webpack-4.44.2.tgz is added here. Also curious that many of the Node package tarballs that get added don't have corresponding deletions.

Hmm I'll try ye olde cache flush and re-install everything.

Well, no changed packages from flushing everything and re-installing, so that's good. Looks like the webpack 4.x is a dependency of webpack-stream, so that's why that is getting installed. Hopefully they update webpack-stream soon.


Possible to use on Bamboo Cloud

Is there any way to utilize this plugin on Bamboo Cloud? If not what options are there for deploying to and downloading from Artifactory in Bamboo?

Hi @snowe2010,
Some of the Bamboo Artifactory Plugin functionality is supported in Bamboo Cloud, through the use of the newly released JFrog CLI or for Maven and Gradle builds you can use the Maven Artifactory Plugin or Gradle Artifactory Plugin,


An internal error in Motoko compiler

$ .vessel/.bin/0.9.1/moc -r --package base .vessel/base/moc-0.9.0/src --package base-0.7.3 .vessel/base-0.7.3/master/src --package btree .vessel/btree/v0.3.1/src --package encoding .vessel/encoding/main/src --package matchers .vessel/matchers/3dac8a071b69e4e651b25a7d9683fe831eb7cffd/src --package sha .vessel/sha/master/src --package stable-buffer .vessel/stable-buffer/v0.2.0/src --package stable-rbtree .vessel/stable-rbtree/v0.6.1/src -wasi-system-api test/*Test.mo
prim:276.4-276.24: execution error, Value.prim: cyclesBalance

The error happens in the directory stress-test/ of this commit on make command.

Moc -r runs the interpreter which doesn't support the primitives needed to deal with cycles that are only available on the IC.
Ideally, moc should error more gracefully with a static error instead.


AMBARI-25433. Ambari should add login and password to urls populated from VDF

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/5589/
Test FAILed.
Test FAILured.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/5590/
Test FAILed.
Test FAILured.


feat: experimentally add more language support

Added more language, including:

French
German
Japanese
Russian

It's translated by ChatGPT, of course!
May be useful, may be fun, who knows?

@Chanzhaoyu what do you think 😄

One disadvantage is that I don't understand these languages,  Machine translation seems unreliable.

OpenAI translate quality is somehow acceptable IMO, and friends come over the world can help review, the most important thing might be that: we don't really need a perfect translation or code to make it available? 😄

It will also add to the trouble of the contributing staff, let me think about it. 🤔

Temporarily closed, if needed, I will contact you again, thank you~


Native Support for Spot Termination

Tell us about your request

Native Support for Spot Termination

Tell us about the problem you're trying to solve. What are you trying to do, and why is it hard?
What outcome are you trying to achieve, ultimately, and why is it hard/impossible to do right now? What is the impact of not having this problem solved? The more details you can provide, the better we'll be able to understand and solve the problem.
Are you currently working around this issue?
How are you currently solving this problem?
Using aws-node-termination-handler up to this point (https://github.com/awslabs/karpenter/issues/105)
Additional context

This will be an aws specific spot termination controller.

Attachments
If you think you might have additional information that you'd like to include via an attachment, please do - we'll take a look. (Remember to remove any personally-identifiable information.)

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

If we're using something like CapacityOptimizedAllocationStrategy, we might be able to simply rely on fleet to not give us an instance in the same pool. In this sense, we can make karpenter unaware of the details and just let ec2 do the decision making.

I think it will be easier for us to talk about this in a call, but that's not how the CO strategy works. You can maybe use CO prioritized and set that pool at a low priority, but otherwise there's no reason for it to avoid that pool. If all the pools you provide to the CO strategy are constrained, it will give you the least constrained pool, which may be the pool the instance that received the rebalance recommendation was in.

are there any plans to support this?

Yes, we are working on this functionality currently

Looking forward to seeing this feature implemented. It will save tons of IPs for our cluster.

please update karpentar documentation that users need to use NTH(with Queue Processor) for aws api termination for spot and on-demnad instances

Is "spot termination" for this issue restricted to spot instances?  Does this issue's scope include termination of non-spot instances (for underlying hardware maintenance by AWS, for example)?

Is "spot termination" for this issue restricted to spot instances? Does this issue's scope include termination of non-spot instances (for underlying hardware maintenance by AWS, for example)?

Yes, it does include support for AWS Health events as well, similar to the aws-node-termination-handler.


fix: stepper: Handle decimal scrollLeft value for scroll buttons

SUMMARY:
ScrollLeft property can have decimal values for systems using display scaling (ref). Handle decimal values when deciding to disable scroll buttons.
GITHUB ISSUE (Open Source Contributors)
JIRA TASK (Eightfold Employees Only):
https://eightfoldai.atlassian.net/browse/ENG-43824
CHANGE TYPE:

[x] Bugfix Pull Request
[ ] Feature Pull Request

TEST COVERAGE:

[ ] Tests for this change already exist
[ ] I have added unittests for this change

TEST PLAN:
Before
https://user-images.githubusercontent.com/77000634/220738620-ed678fff-aaa5-45bb-9ff0-0d719a26c503.mov
After
https://user-images.githubusercontent.com/77000634/220738699-22f1fe25-318b-4551-be4b-f092e4069084.mov

Would be extra if you add a CSS modifier to give the small variant paddles some top margin. If not now, I can do this in a future change.

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@76304f3). Click here to learn what that means.
Patch has no changes to coverable lines.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #546   +/-   ##
=======================================
  Coverage        ?   84.23%
=======================================
  Files           ?      790
  Lines           ?    16781
  Branches        ?     5886
=======================================
  Hits            ?    14135
  Misses          ?     2580
  Partials        ?       66

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


AdError 1101: The provided content information: currentTime is invalid.

Expected behaviour
Actual behaviour
Environment

Browser: Chrome & Firefox
Version:
Operating System: Windows 7
Version:

Steps to reproduce

Does this happen on the plyr demo https://plyr.io/? Didn't get this myself but there is a similar error logged recently https://sentry.io/plyr-js/demo/issues/550854898/?referrer=slack (which only Sam can see).

Don't take it personal. The ads called me that as well. It's a minified error so it means pretty much nothing. If you expand it it will say "The ad response was not understood and cannot be parsed" or "Unable to request ads from server due to network error" or something.

This should now be resolved in v3.4.0 but please report back if you're still seeing it.


[FLINK-32603] Avoid consuming twice when two pipelines have the same table source

What is the purpose of the change
This PR avoids consuming twice when two pipelines of datastream have the same table source.
Brief change log

Adds the WithTraceInfo interface to make the source transformation able to record the table information.
During translating the table source scan nodes, finds the source transformation via the trace info from the previously translated ones first. If exists, reuse it.

Verifying this change
This change added tests and can be verified as follows:

Added test that validates that only one source transformation is generated even if it is derived to multiple datastreams.

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (not applicable)

@lindong28 Could you help review this PR?


MH-12576 start task failing

This patch changes the behavior of the submit button in the Actions -> Start Task dialog: Instead of waiting for the response of the TasksEndpoint before closing the dialog (which could freeze the ui), this is handed over to a new window and the dialog is closed. The new window can be closed at any time if the user does not want to wait for the backend response. If he does want to wait, then upon response the window displays for each event if the workflow could be started successfully and the error message if it couldn't.
As this patch provides a partial workaround for the ui-freezing-issue, it conflicts with pullrequest #27 which provides another workaround. For easier comparison and discussion this pullrequest was created prematurely, it should not be merged since affected test are not updated yet.
EDIT: This PR builds upon the changes introduced in Christian's PR at Bitbucket which can be found here: https://bitbucket.org/opencast-community/opencast/pull-requests/1845/mh-12576-start-tasks-will-fail-if-start-of/diff

@staubesv noticed your comment on #27, should this go on hold as well since it's solving a similar problem?  Or should we decline #27 in favour of this one?

Talked to @staubesv, this is on hold for the same reason that #27 is on hold!

From the recent discussion with @staubesv, I understand that pull requests #40 and pull request #27 (both from SWITCH; both basically blocking each other) shall be reviewed, possibly combined and re-submitted in the near future Because of that, is there any reason to not decline this for now?

I was not aware that that was the plan.  I'm fine with declining for now for when it's ready to go.

@KatrinIhler I'm closing this for now. We can do a new PR once we continue to work on this.


Fix enable sparksql aggregate compile config in CmakeList

Unify with Presto aggregate, and if we build without VELOX_BUILD_TESTING, we will not find SparkSql aggregate functions

@mbasmanova Can you help review? Thanks!

Can you help merge it? Thanks! @mbasmanova


Boolean actuation works in test bed for boost and pump

This code change happened as George and I set up the test heating system in the garage. The system now has a working boost element and circulator pump, and the tank is full of water that circulates through the one radiator. We successfully heated up the water so that the temperature change was noticeable when touching the tank or pumps (from cold to lukewarm) and also ran the pump.
Changes happened a bit on the fly. Things that I noticed:

We can change the gpio pin of the component dataclass object. Doing this should generate an MQTT message that gets logged by the authority for this part of the component configuration data.
Some of the component dataclass properties belong to the GNodeRegistry (properties related to max rated power, for example) and some do not. Have not yet figured out where the non GNodeRegistry authority should be. Perhaps local on the SCADA?
the boolean actuator drivers can typically detect if their relay is open or closed. This can/should be the first of a series of sensing tests in the control logic. If we implement this, it will be likely that we will want an actor for the boolean actuator that is constantly sensing this. In this case, modularity suggests that turn_on and turn_off will become MQTT messages from the primary scada actor to the boolean actuator actor, instead of function calls. This is stubbed out in for example line 68 of the primary_scada code: if the boolean actuator ShNode has an actor, then turn_on will generate that dispatch MQTT message (not implemented yet).
Fixed a mis-categorization in the input_data (the primary_scada listed as a boolean actuator)
Reworked the structure of the primary_scada so that turn_on() and turn_off call an ShNode, not a driver or a component. The primary_scada main logic should reference all parts of the system as ShNodes.

The plumbing industry calls the heat distribution pumps "Circulator Pumps" so I changed our name to match this.


Steam Logout

Describe the bug
Every time Steam is closed and an attempt is made to switch to the game environment, it logs out of the account.
What did you expect to happen?
I expect the mode to start
big picture mode
and will not log out of the account when the client is closed
Output of rpm-ostree status
State: idle
Deployments:
  ostree-image-signed:docker://ghcr.io/ublue-os/bazzite-deck-gnome:stable
                   Digest: sha256:1b7420b6b73e170a80da4c40192169114fa1c10c77023744d9ed700d2836a781
                  Version: v3.6-40.20240809.0 (2024-08-09T21:13:36Z)
                     Diff: 6 added
          LayeredPackages: sunshine

● ostree-image-signed:docker://ghcr.io/ublue-os/bazzite-deck-gnome:stable
                   Digest: sha256:1b7420b6b73e170a80da4c40192169114fa1c10c77023744d9ed700d2836a781
                  Version: v3.6-40.20240809.0 (2024-08-09T21:13:36Z)
    InterruptedLiveCommit: c87c11769b4b782e0fd16de5e50f4cc33e399db6c2360e0a68ffebf144e4fa75
                 Unlocked: transient

  ostree-image-signed:docker://ghcr.io/ublue-os/bazzite-deck-gnome:stable
                   Digest: sha256:1b7420b6b73e170a80da4c40192169114fa1c10c77023744d9ed700d2836a781
                  Version: v3.6-40.20240809.0 (2024-08-09T21:13:36Z)

Hardware
Kernel: 6.9.12-203.fsync.fc40.x86_64
Architecture: x86_64
Desktop: GNOME 46.4
Distro: Bazzite 40.20240809.0 (Silverblue), Fedora 40
Motherboard: ASUS PRIME Z390-A, UEFI: American Megatrends v: 1802 (12/01/2020)
CPU: Intel Core i7-9700K
Graphics: AMD Radeon RX 6600 (ASRock), driver: amdgpu
Extra information or context
https://github.com/user-attachments/assets/49f0ce48-f9ee-4827-8d94-b67251d9c7dc

This happens when steam crashes, find out why it crashes
or try reset steam with ujust fix-reset-steam and see if that helps. (this will force you to re-log one more time)


operator-sdk: 1.27.0 -> 1.28.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/operator-framework/operator-sdk/releases.
meta.description for operator-sdk is: SDK for building Kubernetes applications. Provides high level APIs, useful abstractions, and project scaffolding
meta.homepage for operator-sdk is: https://github.com/operator-framework/operator-sdk
meta.changelog for operator-sdk is: https://github.com/operator-framework/operator-sdk/releases/tag/v1.28.0
Updates performed

Golang update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.28.0 with grep in /nix/store/a785dc4qbis1gd9sgizrag60rgfpr7w3-operator-sdk-1.28.0
found 1.28.0 in filename of file in /nix/store/a785dc4qbis1gd9sgizrag60rgfpr7w3-operator-sdk-1.28.0

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
operator-sdk

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/a785dc4qbis1gd9sgizrag60rgfpr7w3-operator-sdk-1.28.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A operator-sdk https://github.com/r-ryantm/nixpkgs/archive/002aa7c51ed536ece6f54c2a884d8cdfe5a79a8d.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/a785dc4qbis1gd9sgizrag60rgfpr7w3-operator-sdk-1.28.0
ls -la /nix/store/a785dc4qbis1gd9sgizrag60rgfpr7w3-operator-sdk-1.28.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

operator-sdk

Maintainer pings
cc @arnarg for testing.

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

1 package built:

operator-sdk


Block edge 4.5.14

4.5.14 needs to be disabled:
2 clusters currently failing (18%),  2 gone (18%),and  7 successful (64%), out of 11 who attempted the update over 7d
Currently failing classification:
* Monitoring issues: 1
* Untriaged: 1

4.5.14 can be enabled again:
1 clusters currently failing (10%),  3 gone (30%),and  5 successful (50%), out of 10 who attempted the update over 7d
Currently failing classification:
* Monitoring issues: 1


Enable HSTS in Frontend pod and Support for AL_HSTS_MAX_AGE environment variable

Support for AL_HSTS_MAX_AGE environment variable in ui pod.
Add serve.json when enabled to set the HSTS header in the frontend pod.
See PR https://github.com/CybercentreCanada/assemblyline-ui/pull/818 for further details.

The UI API related to this was approved, please set default values in the helm chart.

Closing in favour of: https://github.com/CybercentreCanada/assemblyline-helm-chart/pull/109


Merge gen-payloads into one

Depends on https://github.com/openshift/doozer/pull/144

@jupierce PR needs rebase.

Merging since https://github.com/openshift/doozer/pull/144 merged.


feat(blogpost): what is nft staking

Thank you for your contribution to the KodaDot - One Stop Shop for Polkadot NFTs.
👇 __ Let's make a quick check before the contribution.
PR Type

[ ] Bugfix
[x] Feature
[ ] Refactoring

Context

[x] Closes #6889
[ ] Requires deployment <snek/rubick/worker>

Before submitting pull request, please make sure:

[x] My contribution builds clean without any errors or warnings
[x] I've merged recent default branch -- main and I've no conflicts
[x] I've tried to respect high code quality standards
[x] I've didn't break any original functionality

Optional

[ ] I've tested it at </ksm/collection>
[ ] I've tested PR on mobile
[ ] I've written unit tests 🧪
[ ] I've found edge cases

Did your issue had any of the "$" label on it?

[x] Fill up your DOT address: Payout

Community participation

[x] Are you at KodaDot Discord?

Screenshot 📸

[ ] My fix has changed something on UI; a screenshot is best to understand changes for others.

Copilot Summary

🤖 Generated by Copilot at bbdfc47
Added a new blog post content/blog/exploring-the-world-of-nft-staking-an-in-depth-guide.md that explains NFT staking, its benefits and challenges, and some examples of platforms that support it. The goal is to educate and engage the blog readers about this new and innovative concept in the NFT space.

🤖 Generated by Copilot at bbdfc47

NFT staking post
Exploring a new concept
Autumn of learning

pay 30 usd

😍 Perfect, I’ve sent the payout
💵 $30 @ 4.42 USD/DOT ~ 6.787 $DOT
🧗 13rv1SWoLg9Gb3tmvHPZxb7JbVy51BtMziX7k9WQGSJ7Kp3A
🔗 0x341cab96bd3079536f11b96c73b74b04fb74ec410971665dd8d155b269c4e43d
🪅 Let’s grab another issue and get rewarded!
🪄 github.com/kodadot/nft-gallery/issues


No access to sign-up email anymore. How to proceed?

Hi. I'm coming here from the https://pypi.org/help/ page. I don't have access to the email address I used to signup on PyPi anymore. How do I properly reset my password and change the email address?

@di Thanks very much for the quick support. Works fine now.


Add 'behavior of infected organism' to Apollo-SV with definition, elucidation, UAL, and term editors.

'behaviorOfInfected' was removed from the XSD. No action taken.


Blank white authentication (chrome) window pops-up

When activating the openfortivpn using Linux Mint 22 on a 2011(?) Intel Macbook pro, the SSO authentication window opens with the electron install of openfortivpn-webview, and the fields are present because if I hover over the window the cursor changes. I can "invisibly" enter my email address, press , enter my password, and I can see the terminal window accepting the inputs, but since the GUI window for the webview stays solid white, I cannot see the multifactor authentication number that pops up. The authenticator app on my phone activates so I know all the "code" is working, it's just the GUI window staying solid white that is stopping it from working correctly.

This is just a guess, but try to launch the application with --disable-gpu and see if anything changes. It has helped other users in the past.
Instead of disabling the GPU altogether you could use more specific --disable-gpu- options (I do not have a list, they are Chromium's flags), but finding what works is probably not worth the effort given that the browser just needs to run for few seconds.

Thanks for the tip. I tried --disable-gpu and some more specific variants like --disable-accelerated-2d-canvas but none of them prevented the blank/white window.

I'm sorry, but I don't think I can help you here. Electron does the heavy lifting, this application just adds some listeners here and there to retrieve the session cookie.
You could either try to build the application using a different version of Electron (if you have some Electron based application that works you could try to figure out the version it uses) or you could try the Qt variant.
Building the Qt variant is quite easy, just clone the repo and follow this: https://github.com/gm-vm/openfortivpn-webview/tree/main/openfortivpn-webview-qt#how-to-build
Unfortunately each distro packages stuff differently, so I don't a list of dependencies, but in general you need the development files of QtWebEngine (the package is generally something like qt6-qtwebengine-dev or qt6-qtwebengine-devel or some other combination), g++ and make. Hopefully these will pull all the other dependencies needed to build the application. qmake/cmake will otherwise tell you what you what is missing.

I'm closing this since I don't think there is anything I can do, but feel free to add more comments here.


SqlServer drivers need to be downloaded every time i restart dbeaver

I am using dbeaver to connect and work with SQL Server and I have a connection set to use the MS SQL driver. After I setup, the connection things work fine. When I close the app and reopen and I try to open the connection I am prompted to download the drivers again. When I redownload them things work for that session. Close and reopen and I need to download them again. I have tried several things like Force check box. I tried using the jtds driver but had the same issue, needed to download again when i opened the connection in a new session, I prefer the MS drivers.
I am running Windows Server 2016 as admin with SQL Server 2016.
thanks for your time and support.
John

Can't reproduce. Do you know - does this happen only with SQL Server driver or with any driver?
DBeaver downloads and saves driver files in ${USER_HOME}.dbeaver-drivers\ folder. Binary is saved in drivers\mssql\auth\x64\ subfolder, Maven jars are in maven/maven-central/com.microsoft.sqlserver subfolder. Are these folders/files present on your machine?
BTW did you try another driver (jTDS)?

First off i was able to fix it by deleting everything in the

C:\Users\Administrator.dbeaver4.metadata
and starting the app up to reinitialize that folder. I needed to download the drivers once and now it appears to be fixed.

While snooping around I noticed some files in the .plugins directory appeared older than I would expect. One of the files mentioned version 4 but I am currently running 5. I upgraded a while back.
I've attached debug log before I cleaned out the metadata directory.
dbeaver-debug.log
Thanks
John

Yep, these errors seems to be a result of incorrect version upgrade (e.g. if you use zip archive then previous version has to be deleted completely before extracting new one in the same location).
There were a lot of Access Denied errors, DBeaver wasn't able to save configuration properly.
Thanks for letting know.


Add mewzherder to Flux Community Maintainers

Membership from: https://github.com/fluxcd/community/issues/341

There are some issues, I'm thinking somehow branches got mixed up - did you mean to make updates to SUPPORT.md in this PR? I can help resolve this, and it says I've got permission to edit the branch. Let's talk off thread and figure out out.
Will touch base at our next check-in to determine if there's meant to be a separate PR, or if these changes got included by mistake. We should only include the relevant changes in this PR – update to the MAINTAINERS file, and COMMUNITY.md - we should also be sure to remove your entry in the PROJECT-MEMBERS.md since otherwise you'll probably appear in both lists, and I don't think we want that.
🎉  welcome to the team (though it seems a bit strange to say this as if you haven't been notionally on this team even longer than myself!)

It looks like that commit was from a stale branch that had already been merged.
It dropped out, so there's no DCO issue, so I can approve this and merge it now (except the discussion is in #341 and I think we are still waiting on a few maintainer approvals there)
It's an LGTM from me, as soon as the appropriate approvals are done in the discussion 👍

@kingdonb thx! who / what steps are next to merge?

I reviewed PROCESS.md near this section:

Applying for Flux Maintainership
Process:

First it says we need all the active maintainers in the same repo (community) to vote, then a code maintainer (not me) must approve the PR - I'm summarizing the text as I don't have it in front of me, nor able to link to subheadings because the GitHub Android app is ... not helping
I will check if all maintainers have voted and ping whomever we are missing then we can move on to the next step.

I think we resolved all the issues in this PR, @stefanprodan can you confirm


NPM has version 2.0.13, not latest

Can you push the latest to NPM ?
I am getting bundle issues, using Webpack 3
✖ Webpack compiler encountered errors.
2.171ef0b57ad1fb93a522.js from UglifyJs
Invalid assignment [./~/goorl/index.js:5,0][2.171ef0b57ad1fb93a522.js:31486,37]
✖ Compiler encountered errors. Error: Webpack compiler encountered errors

I believe this is due to package dependencies ..

Added interface for promises and pushed version 2.1.0 to GitHub and npm


Fix sensor tick

Description
The sensors were not ticking according to the sensor_tick attribute.
A smoke test was added to check this scenario too.
Where has this been tested?

Platform(s): Ubuntu 18.04
Python version(s): 3
Unreal Engine version(s): 4.24

This change is

Does this PR solve the issue for ray-cast sensors? I see in the PR files that only camera sensors were changed

Hello ! Did you already fix the problem?
I need to reduce the frequency of a collision sensor that doesn't have the sensor_tick attribute.


k3s cluster should use GPU without mentioning runtimeClassName: "nvidia" in pod specs

Hi,
I have installed a k3s cluster and gpu operator using nvidia official docs.
Everything seems to work fine and pods are able to use GPUs but I have to define runtimeClassName: "nvidia" in pod specs.
Do we have any other way of doing it?
Can I somehow make the k3s pods to use GPUs without defining runtimeClassName: "nvidia" in pod specs?

This is expected behavior at this time, although we are looking at ways to make using GPUs easier

This is expected behavior at this time, although we are looking at ways to make using GPUs easier

I am using kubeflow manifest to deploy kubeflow on K3s, how can I add runtimeClassName in pods creation for it?
I have been looking for admisssion webhooks for it, but still I am on R&D.
I would appreciate your kind guidance on this.

Duplicate of https://github.com/k3s-io/k3s/issues/8700 - see the --default-runtime flag from https://github.com/k3s-io/k3s/pull/8936

Duplicate of #8700 - see the --default-runtime flag from #8936

Can I somehow change default runtime on single node k3s cluster?