feat: prevent usage of enums without a default variant in storage

I think enums without a default variant should throw an error at compile time if they try to derive starknet::Store - otherwise, attempting a storage read on an uninitialized slot will cause a runtime panic.
See description in https://github.com/cairo-book/cairo-book/issues/990

But why is that an issue specifically? it is the decision of the contract writer to know they only are reading valid values.
adding a diagnostic now may even be more confusing - even possibly marking any of the existing keys as #[default] causing a storage layout change.
(not saying no - just want to make sure the cure is better than the disease)

it is the decision of the contract writer to know they only are reading valid values.

I consider it an issue because I don't think reading a value from storage should default to a panic if that value is not set. By ensuring a default variant exists, you are leveraging the compiler so that:

You catch eventual errors at compile time by forcing a variant to act as the default one - if required, the contract creator can just add a None variant that fulfills this role.
You force the definition of a type associated to an empty storage
You can "clear" an enum in storage by resetting it to the default variant
Contract creator might not now about this default panic scenario (as reported in the issue) - I actually did not even know about this.


[WFRP4e System] Blinded effect does not end properly if it was in effect prior to combat.

If a character has blinded condition before combat is initiated, it wont automatically end that condition during combat rounds. Even if another blinded condition is added say from 1 to 2, or removed from 2 to 1 blinded conditions, it will not track the conditions. You have to completely remove all conditions and add them back if one was present before hand. From: Ajon1foundry: 11.315wfrp4e: 7.0.3wfrp4e-archives1: 4.0.2wfrp4e-archives2: 4.0.2wfrp4e-core: 4.0.2wfrp4e-owb1: 4.0.2wfrp4e-ua1: 4.0.1wfrp4e-up-in-arms: 4.0.1wfrp4e-wom: 4.0.1wfrp4e-zoo: 4.0.1Active Modules: wfrp4-wfrp4e-npc-generator, wfrp4e-actor-sheet-print, wfrp4e-gm-toolkit

Not planned


Keep wallet files upon uninstall of Station

Context
Wallet files (.yaml) are currently stored in the same directory as other stations Files/folders. We experienced a few issues because of this approach on install and uninstall side.
Wallet .yaml files allow users to import existing accounts, with the current limitation of not being backward compatible.
For uninstall:

Given than user doesn't back up the wallet account (via exporting .yaml file and/or saving private keys), once he uninstalls Station on all OS (to be confirmed), we delete wallet files. This will lead to permanent loss of funds.
Some related issues: massalabs/station#1073

For install/update:

Recognized wallet files following the supported format, should be recognized in the wallet UI.

How to
Technical details
Give the technical insights so anyone in the team can tackle the tasks - Dev
QA testing
Does this task require some QA tests ?
If yes, explain how to validate it

I propose to store the wallet files and assets.json in:

~/.massa-station-wallet for Ubuntu and MacOS
%APPDATA%/massa-station-wallet for Windows

working as expected when followed Q/A steps

Done on MacOS M1

QA on windows: it works


Split config between warn and purge rules

warn and purge should be mutally exclusive rules.
While were updating the agent, add flags to report and purge and make
them optional.
Signed-off-by: Joe Grund<EMAIL_ADDRESS>

Codecov Report

Merging #1009 into master will decrease coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1009      +/-   ##
==========================================
- Coverage   95.42%   95.39%   -0.04%
==========================================
  Files           2        2
  Lines         153      152       -1
==========================================
- Hits          146      145       -1
  Misses          7        7

Impacted Files
Coverage Δ

iml-rabbit/src/lib.rs
95.3% <0%> (-0.04%)
:arrow_down:

Continue to review full report at Codecov.

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


Parametrize building the MIE deployment package

Break up the build process into smaller actions, especially for parts that are delivered separately.
Some possible build options:

Build deployment package for a certain app: Instant Translate, Media Insights
Build API docs
Build web app

done in v2.0.4


"No Fill" Option

A resolution to https://github.com/excalidraw/excalidraw/issues/79 might be have an option to just have no fill. Right now its Solid, Hatchure, and Cross-Hatch. Might be worthwhile to have None.
Edit: Another note, there is no transparency option for fills either.

Yes!
We should either have a no-fill option or a transparent background color. Either would work.

I added the ability to select a transparent color


Resizing window on desktop sometimes triggers onClick handlers of Composables

Reproducable with experimental ImageViewer project. Resize the window in the gallery view while having the cursor at the height of the preview image, and sometimes, it'll trigger the transition to the Memory View.
https://user-images.githubusercontent.com/2178959/223841903-55c74b1c-4c7c-4829-b187-53e2256aee72.mov

Reproduced with a simple button example:
import androidx.compose.material.MaterialTheme
import androidx.compose.desktop.ui.tooling.preview.Preview
import androidx.compose.foundation.background
import androidx.compose.foundation.layout.Row
import androidx.compose.foundation.layout.fillMaxSize
import androidx.compose.foundation.layout.size
import androidx.compose.foundation.lazy.LazyColumn
import androidx.compose.foundation.lazy.items
import androidx.compose.material.Button
import androidx.compose.material.Text
import androidx.compose.runtime.Composable
import androidx.compose.runtime.mutableStateOf
import androidx.compose.runtime.*
import androidx.compose.ui.Modifier
import androidx.compose.ui.graphics.Color
import androidx.compose.ui.window.Window
import androidx.compose.ui.window.application

@Composable
@Preview
fun App() {
    val colors = listOf(Color.Red, Color.Green, Color.Blue)
    val numberOfColors = colors.size
    var index by remember { mutableStateOf(0) }
    Row(Modifier.background(colors[index % numberOfColors]).fillMaxSize()) {
        Button(onClick = { index++ }) {
            Text("ClickMe")
        }
    }
}

fun main() = application {
    Window(onCloseRequest = ::exitApplication) {
        MaterialTheme {
            App()
        }
    }
}

To reproduce:

Start resizing outside of a button (press mouse/touchpad down).
Move cursor inside of the button.
Move cursor outside of the button & stop resizing. Click on the button is often registered (but not always).

Kotlin 1.8.0
Compose 1.3.0
JDK 18

This sounds like an unintended side-effect of the synthetic events + somehow receiving/processing the mouse event before the measure/layout cycle.

On Linux Fedora 38 Gnome Desktop, I have a similar problem, but slightly different. After resizing the window, I release the mouse button and immediately move the mouse pointer into the window area, at which point the Click event is triggered.

Ok, I did some digging and what happens is that during the resizing AWT sends us MOUSE_ENTER and MOUSE_EXIT events (no press or release events though) with extModifiers=Button1. However PointerInputScope.detectTapAndPress doesn't look at the event type - it only cares about whether any button is pressed or released. So when it receives an event which says a button is pressed, and then another one which says it's released, it detects a tap.
@igordmn Where do you think it's best to fix/workaround this? I'm thinking we should completely filter out these MOUSE_ENTER and MOUSE_EXIT events because that's the root cause of the problem. AWT should not be sending us mouse events while resizing the window.

we should completely filter out these MOUSE_ENTER and MOUSE_EXIT

I agree, this will be a good solution if we manage to do that (ideally, AWT should consume hover on resize in the first place)
But how we can know that we should filter the received event? Maybe we should filter enter's with pressed buttons?

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


Issues related to shadow dom vs. shady dom

Hi,
I ran into a couple of issues when applying paper-tree, I think they are related to shadow dom vs. shady dom, but I'm not sure. That might explain why they work in you're demo but not in my application.
My application is based on the Polymer Starer Kit. I'm working on Chrome v55.
I went ahead and forked your repository and applied some fixes. In all just 4 lines of code. The Fork/branch can be found here:
https://github.com/neuralquest/paper-tree/tree/lazy-load
Issues:
I ran in to a problem related to event retargeting. The e.target in paper-tree _selectNode was pointing to the paper-tree node, as opposed to the selected paper-tree-node. I was able to fix this using  var normalizedEvent = Polymer.dom(event). See:
https://www.polymer-project.org/1.0/docs/devguide/events#retargeting
I also had an issue with node-row highlighting. What I found was that the class selector under :host, needs to be placed in parentheses, like so: :host(.selected). There are examples of this in Rob Dodson's cheat sheet:
http://robdodson.me/shadow-dom-css-cheat-sheet/
We may have to add ::content to the statement as wel. I didn't bother with this since its working for me without.
https://www.polymer-project.org/1.0/docs/devguide/styling#styling-distributed-children-content
New Features:
I propose firing a toggle-children event from toggleChildren so that we can use the tree for lazy loading.
I tried to add a demo that illustrates this, but failed. I cant figure out how to add javascript, inside a demo-snippet template. Is this at all possible? Perhaps you can point me in the right direction. It's under demo/index.html.
See Tree lazy loading in action: https://neuralquest-395e9.firebaseapp.com/ (work in progress)
I added  src$="[[data.src]]" to the iron-icon so that users can optionally provide a url/svg/base64 string. The docs say that scr takes precedence over icon, but this is not true. To work around this I also changed the parm in _computeIcon to data and added if(data.src) return '';. The app above uses base64 strings read from the database.
I also added white-space: nowrap; to .node-container to prevent wrapping of the tree node.
I also propose adding an animated loading icon. By setting loading:true you are in effect saying "I don't know yet if there are any children". See _computeClass and class selector .node-preicon.loading:before. The app above shows this in action.
Hope this helps.
I learned a lot from studying your code.
Chris

Thx for posting.
About your issues:

Normalized event sometimes does not work as expected depending on which element catch the event. That is why I directly pass information into the event detail (in the last version).

Indeed the CSS syntax was not shadow compliant. This is now fixed.

::content selector is not needed here since there is no light dom.

Would you try the release v1.0.2 and tell me if it fixes all the things.
Then we could talk about your feature requests.

I can confirm that v1.0.2 works form.
I also added a fourth enhancement request for an animated 'loading' icon.

Issues resolved.
New features discussed in pull request


Demonstration of how system activation fails if the template path is not "external"

This demonstrates one way of reproducing
https://github.com/Mic92/sops-nix/issues/659. There may be others: I think you could achieve the same thing with a regular secret rather than a template.
Here's how this fails:
$ nix build .#checks.x86_64-linux.template-path-not-external -L
...
vm-test-run-sops-template-path-not-external> machine: must succeed: /run/current-system/bin/switch-to-configuration test
vm-test-run-sops-template-path-not-external> machine # [   11.028130] nixos[949]: switching to system configuration /nix/store/0jbh5rys5x1br66s4n777jq8ny2bq9nb-nixos-system-machine-test
vm-test-run-sops-template-path-not-external> machine # [   11.029653] systemd[1]: Stopped target Local File Systems.
vm-test-run-sops-template-path-not-external> machine # [   11.031619] systemd[1]: Stopped target Remote File Systems.
vm-test-run-sops-template-path-not-external> machine # activating the configuration...
vm-test-run-sops-template-path-not-external> machine # /nix/store/8788mz0w701ff12zpsgnbm7nzsfy290k-sops-install-secrets-0.0.1/bin/sops-install-secrets: failed to prepare new secrets directory: cannot access /run/secrets: readlink /run/secrets: invalid argument
vm-test-run-sops-template-path-not-external> machine # Failed to run activate script
...

The only fix I can think of is to check if the chosen path is inside of /run/secrets and reject it if it is (of course allowing for the default value for these paths, which actually is inside of /run/secrets).

On the other hand this could also happen with any other invalid symlink as a target.

On the other hand this could also happen with any other invalid symlink as a target.

Sorry, what does "invalid symlink as a target" mean?
The issue I'm demonstrating here is showing how /run/secrets can end up being a directory rather than a symlink.
I suppose there are more subtle scenarios involving symlinks to /run/secrets that would not be handled by the fix. I'm not sure if that's what you're referring to.


onClusterPress does not returns right number of markers

onClusterPress always returns markers numbers = 10 if number is great than 10.
If the markers number is less than 10 then it returns the right value.
So the issue only comes when number of markers is create than 10 in the cluster.

@harishchopra86 You will get the right number of markers in the next version. I've opened a pull request with new feature that includes this, and this might get merged by end of this week. :)

Sounds good.
Thanks @chetanparmar95

Fixed in version 3.1.2 🎉


Add-DbaRegServer - Add CSV piping support

awww nooooooo i thought i'd implemented this before. It's supported in Import D:


Support for standardjs

There's no plugin for tslint like standardjs. Can tslint be extended via plugins similar to styles like eslint?

This is old, but yes it does have custom rules which can be used to extend tslint. Please see the docs.


error 404 request entity not found

where do i find my agentuser-id
The reply is something like:
HTTP/2 404
vary: X-Origin
vary: Referer
vary: Origin,Accept-Encoding
content-type: application/json; charset=UTF-8
date: Tue, 12 Jun 2018 06:19:49 GMT
server: ESF
cache-control: private
x-xss-protection: 1; mode=block
x-frame-options: SAMEORIGIN
x-content-type-options: nosniff
alt-svc: quic=":443"; ma=2592000; v="43,42,41,39,35"
accept-ranges: none
{
"error": {
"code": 404,
"message": "Requested entity was not found.",
"status": "NOT_FOUND"
}
}

and the login page does not open properly, there is something wrong with it, I even enabled the javascript content in chrome.

The agentUserId is 1234 for the default user in this sample.
Can you take a look at what errors are in the chrome developer tools when you open the login page?

yeah, its working fine right now. thank you.
Can you please tell me how the flow of data happens, with the command prompt commands and the google assistant and the frontend, i'm not quite understanding how it all gets interconnected.

thank you for the help, I appreciate your patience :)

I just have another query, after all the setup:
Suppose I have a table lamp and i turn it on using google assistant. for a particular interval of time I do not perform any task(turn on/off), I let it be in the same state, but what is happening is after a particular time interval the smart-home itself keeps updating the status of the device even though it has not changed. ( I am actually using your code to map it to my device, for example I turn my device on when I say turn on the lights. So everytime the api sends the status(even though the device state is not changed) my device also gets a post request to turn on. So this is stacking up many post requests making my device to crash.
if you have any solution to this please do help.

That's probably an issue with regards to the sample in of itself, where the browser and backend server send messages to update the status.

Instead of using the ngrok url , i'm now using my own authentication, after logging-in onto my oauth url it redirects to google home app, here it says "couldn't update setting check your connection", I checked online about this, some people say that if you have not logged into "google chrome" with the same account as "google home" app then this issue might occur, I tried that too, but it doesn't seem to make any difference.
Do you have any idea about this?

Ok thank you for everything.
sorry to bother but I just have another query
I started a custom project in action.google,  and then I created two intents switchon and switchoff.
in the fullfillment( firebase cloud functions) on matching a intent i'm calling the respective function.
**function switchon(agent){
agent.add('Ok');
req.post('URL for my post request',
{ json: { "Name":"FanOn",
"ResponseTimeout": "00:01:08",
"ConnectionTimeout":"00:01:05",
"JsonPayload": "{}",
"Message" : "{"Name" : "TurnFanOn",  "Parameters" : {}}"
}
},
function (error, response, body) {
if (!error && response.statusCode == 200) {
console.log(body);
}
}
);
agent.add("all ok");
}**
But when I test it, the assistant responds with "OK" and "ALL OK" but the post request in between doesn't seem to be called. I checked if the post request is working(it works absolutely fine).
Do you have any idea about this?

First it looks like you're using the Dialogflow fulfillment library, which is different than using the smart home fulfillment.
Second, it's because the post request is asynchronous. You will need to return something like a Promise to tell the fulfillment that it is not done until the post request is done.


Handle deprecation of naive datetime functions like utcnow()

Fixes #163
The approach taken follows the advice in https://github.com/webrecorder/warcio/issues/163#issuecomment-2313410986 of returning naive datetimes by default but allowing callers to specify a timezone such as datetime.timezone.utc for aware datetimes. This should prevent comparison errors between naive and aware datetimes for now while giving us a clear path forward if and when naive datetimes are fully deprecated in a future Python version.

Oops, missed a few calls, putting back to draft for now

Doesn't the WARC standard require utc? From looking at my crawler-that-uses-warcio I appear to have forced everything to UTC already, but then again my side-gig is astronomy.

Doesn't the WARC standard require utc? From looking at my crawler-that-uses-warcio I appear to have forced everything to UTC already, but then again my side-gig is astronomy.

Hi Greg, the question here is less about UTC vs other timezones and more about Python's naive vs aware datetime times. Miguel Grinberg has a pretty good blog post about how exactly this issue came about: https://blog.miguelgrinberg.com/post/it-s-time-for-a-change-datetime-utcnow-is-now-deprecated
With this PR, by default naive datetimes representing UTC will continue to be used, or for a UTC aware timezone a user can pass in datetime.timezone.utc as an argument to the methods. I suppose we could simply make the flag aware or similar to ensure that UTC is always being used if a timezone is specified.

@ikreymer @wumpus Since Greg had a good point that the WARC spec always expects UTC, I changed the tzinfo argument on the time utility functions to an aware bool defaulting to False. If set to true, the returned datetime will be timezone-aware with tzinfo=datetime.timezone.utc


Error on executing command 'flutter build windows'

Hi,
I am trying to create a release build of a flutter project with flutter_nsd package
When I build this project for web (flutter build web) seems to be working.
On building for Windows (flutter build windows) I get an error:
used version of package:
flutter_nsd: ^1.4.0
flutter --version
Flutter 3.13.1 • channel stable • https://github.com/flutter/flutter.git
Framework • revision e1e47221e8 (28 hours ago) • 2023-08-22 21:43:18 -0700
Engine • revision b20183e040
Tools • Dart 3.1.0 • DevTools 2.25.0
flutter build windows
C:\temp\cnc-client\windows\flutter\ephemeral.plugin_symlinks\flutter_nsd\windows\flutter_nsd_plugin.cpp(104,92): error C2220: the following warning is treated as an error [C:\temp\cnc-client\build\windows\plugins\flutter_nsd\flutter_nsd_plugin.vcxproj]
C:\temp\cnc-client\windows\flutter\ephemeral.plugin_symlinks\flutter_nsd\windows\flutter_nsd_plugin.cpp(104,92): warning C4834: discarding return value of function with 'nodiscard' attribute [C:\temp\cnc-client\build\windows\plugins\flutter_nsd\flutter_nsd_plugin.vcxproj]
C:\temp\cnc-client\windows\flutter\ephemeral.plugin_symlinks\flutter_nsd\windows\flutter_nsd_plugin.cpp(104,120): warning C4834: discarding return value of function with 'nodiscard' attribute [C:\temp\cnc-client\build\windows\plugins\flutter_nsd\flutter_nsd_plugin.vcxproj]
C:\temp\cnc-client\windows\flutter\ephemeral.plugin_symlinks\flutter_nsd\windows\flutter_nsd_plugin.cpp(164,47): warning C4834: discarding return value of function with 'nodiscard' attribute [C:\temp\cnc-client\build\windows\plugins\flutter_nsd\flutter_nsd_plugin.vcxproj]
C:\temp\cnc-client\windows\flutter\ephemeral.plugin_symlinks\flutter_nsd\windows\flutter_nsd_plugin.cpp(164,66): warning C4834: discarding return value of function with 'nodiscard' attribute [C:\temp\cnc-client\build\windows\plugins\flutter_nsd\flutter_nsd_plugin.vcxproj]
Building Windows application...                                    67,5s
Build process failed.
Workaround:
Edit Windows/CMakelists.txt
Original:
target_compile_options(${TARGET} PRIVATE /W4 /WX /wd"4100")
to replace with:
target_compile_options(${TARGET} PRIVATE /W4 /wd"4100")

Hi,
I can reproduce this issue when I try to compile the example app...
FYI: To build I had to change cupertino_icons to 1.0.5 this should be no problem..
Output:
Changed 23 dependencies!
Resolving dependencies in .\example...
The current Dart SDK version is 3.1.0.
Because cupertino_icons 0.1.3 doesn't support null safety and no versions of cupertino_icons match >0.1.3 <0.2.0, cupertino_icons ^0.1.3 is forbidden.
So, because flutter_nsd_example depends on cupertino_icons ^0.1.3, version solving failed.
The lower bound of "sdk: '>=2.0.0-dev.28.0 <3.0.0'" must be 2.12.0 or higher to enable null safety.
For details, see https://dart.dev/null-safety

fake_async 1.3.1 (was 1.3.0)
flutter_lints 1.0.4 (2.0.2 available)
flutter_nsd 1.4.0 from path .. (was 1.3.3 from path ..)
lints 1.0.1 (2.1.1 available)
matcher 0.12.16 (was 0.12.11)
material_color_utilities 0.5.0 (was 0.1.4) (0.8.0 available)
meta 1.9.1 (was 1.7.0)
path 1.8.3 (was 1.8.1)
source_span 1.10.0 (was 1.8.2)
stack_trace 1.11.0 (was 1.10.0) (1.11.1 available)
stream_channel 2.1.1 (was 2.1.0) (2.1.2 available)
string_scanner 1.2.0 (was 1.1.0)
term_glyph 1.2.1 (was 1.2.0)
test_api 0.6.0 (was 0.4.9) (0.6.1 available)
vector_math 2.1.4 (was 2.1.2)

web 0.1.4-beta
These packages are no longer being depended on:

charcode 1.3.1
Changed 21 dependencies!

windows/runner/Runner.rc does not use Flutter version information, updating.
C:\temp\flutter_nsd-master\example\windows\flutter\ephemeral.plugin_symlinks\flutter_nsd\windows\flutter_nsd_plugin.cpp(104,92): error C2220: the following warning is treated as an error [C:\temp\flutter_nsd-master\example\build\windows\plugins\flutter_nsd\flutter_nsd_plugin.vcxproj]
C:\temp\flutter_nsd-master\example\windows\flutter\ephemeral.plugin_symlinks\flutter_nsd\windows\flutter_nsd_plugin.cpp(104,92): warning C4834: discarding return value of function with 'nodiscard' attribute [C:\temp\flutter_nsd-master\example\build\windows\plugins\flutter_nsd\flutter_nsd_plugin.vcxproj]
C:\temp\flutter_nsd-master\example\windows\flutter\ephemeral.plugin_symlinks\flutter_nsd\windows\flutter_nsd_plugin.cpp(104,120): warning C4834: discarding return value of function with 'nodiscard' attribute [C:\temp\flutter_nsd-master\example\build\windows\plugins\flutter_nsd\flutter_nsd_plugin.vcxproj]
C:\temp\flutter_nsd-master\example\windows\flutter\ephemeral.plugin_symlinks\flutter_nsd\windows\flutter_nsd_plugin.cpp(164,47): warning C4834: discarding return value of function with 'nodiscard' attribute [C:\temp\flutter_nsd-master\example\build\windows\plugins\flutter_nsd\flutter_nsd_plugin.vcxproj]
C:\temp\flutter_nsd-master\example\windows\flutter\ephemeral.plugin_symlinks\flutter_nsd\windows\flutter_nsd_plugin.cpp(164,66): warning C4834: discarding return value of function with 'nodiscard' attribute [C:\temp\flutter_nsd-master\example\build\windows\plugins\flutter_nsd\flutter_nsd_plugin.vcxproj]
Building Windows application...                                    32,1s
Build process failed.
PS C:\temp\flutter_nsd-master\example>


Accept single path index for incremental merkle tree proofs

In the sparse merkle tree circuit we accept the index of the leaf element and determine whether parent elements are left or right by converting the index to binary.
In the incremental merkle tree we instead accept a list of 1's and 0's indicating this. We should be able to replace this list with a single input argument that is the leaf element index. This will remove 1 constraint per depth (e.g. tree of depth 32 = 32 constraints removed).
Related #103

@vimwitch should we keep this issue open or close?

@vimwitch should we keep this issue open or close?

Definitely keep it

Closing, accepting the path index reduces the number of constraints.


Don't shadow err variable because we can recover

When checkout fails because cached repository doesn't have requested commit, we do pull, but still return the error because err is shadowed in the if block


Scripts for mon-dev environment

Type of PR

[ ] Documentation changes
[X] Code changes
[ ] Test changes
[ ] CI-CD changes
[ ] GitHub Template changes

Purpose of PR
Initial merge of terraform scripts to provision CAF environment for monitoring/dev
Validation

[ ] Unit tests updated and ran successfully
[ ] Update documentation or issue referenced above

Issues Closed or Referenced

Closes #640
Closes #653

@jkeane What is the long term plan for this content as PnP makes changes to the deployment? Is it all going to get deleted when a new version of the dev cluster gets provisioned?
Also wondering if this is something where we could check in only the tfvars files for the infrastructure deployment and keep all of the other files in the caf terraform

@jomalsan , for now I wanted to keep the scripts separate until we have parity with the PnP team.  Once we get there, then we can prune out the tf scripts and use a common set of scripts.  Only the tfvars should remain in an environment specific child dir.


Icon-Based Navbar Enhancement

Related Issue
We propose to enhance the website's navigation bar (navbar) by replacing text-based menu items with visually appealing icons. This change aims to improve user experience, making the navbar more effective, engaging, and modern.
Fixes:  #995
Description
Changes Made:
Navigation Items Update:
Replaced text-based navigation links with corresponding icons for sections such as Home, Benefits, Genre, Contact, Rate Us, and Login/Signup.
Ensured that the "More" dropdown menu item now uses an icon instead of text, while retaining the dropdown functionality.
Hover Effect:
Implemented a hover effect that enlarges the icons when the user hovers over them.
Styling Adjustments:

Used flexbox to stack icons and their corresponding text vertically within the navigation items.
Added inline styles to adjust the font size for specific dropdown menu items where necessary.

fixed #995
Type of PR

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

Before: Navigation links displayed as text.

After: Navigation links displayed as icons, which enlarge on hover.

Checklist:

[✅] I have made this change from my own.
[✅] I have taken help from some online resources.
[✅] My code follows the style guidelines of this project.
[✅] I have performed a self-review of my own code.
[x] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.
[✅] I have tested the changes thoroughly before submitting this pull request.
[✅] I have provided relevant issue numbers and screenshots after making the changes.

the icon doesnt look similar one is filled one is just empty ones make it just unfilled it will go good also
I regret to inform you that the project code has undergone some major changes. Consequently, you will need to pull the new code and make the necessary adjustments to the codebase. After making these changes, please create a new pull request.
I apologize for any inconvenience this may cause and appreciate your understanding of the situation. Thank you for your cooperation.

I can't understand..please explain @huamanraj
What do u want me to do

I can't understand..please explain @huamanraj
What do u want me to do

here is two different type of styles
one is filled and one is unfilled with black
use any one filled or unfilled for all icon!

okay okay.. will change and update u soon

okay okay.. will change and update u soon

great 👍


Timeout in minutes( docs fix )

It looks, that we can specify timeout for poltergeist not only in seconds, but also in munutes, like this:
Capybara.register_driver 'poltergeist' do |app|
options = {
  timeout: 5.minutes
}
Capybara::Poltergeist::Driver.new(app, options)

May be we can notice about this in readme?

Numeric#minutes is a method provided by ActiveSupport and basically just multiples the number by 60.  No need to document this in poltergeist.


IGNITE-12723: getMXBean for tests introduced.

@nizhikov ,

Should we slightly rename GridAbstractTest#metricRegistry()?

Do we need to keep CacheMetricsManageTest#mxBean() ClusterBaselineNodesMetricsSelfTest#mxBean(), GridCacheLongRunningTransactionDiagnosticsTest#txMXBean(), SetTxTimeoutOnPartitionMapExchangeTest#txMXBean(), TcpCommunicationStatisticsTest#mbean()? Why not using GridAbstractTest#getMxBean() directly?

@Vladsz83 Thanks for the review

Should we slightly rename GridAbstractTest#metricRegistry()?

Rename of the existing methods is not the goal of this ticket.
What name is more convenient?

Do we need to keep

Yes. These methods used multiple times, so I think it OK to keep them.

@nizhikov ,

What name is more convenient?

I was confused with 'metric registry' and dynamic mbeans, dynamic mbean creation. The method is about dynamic mbean to show some numbers, but not exactly for a metric registry . Better name might be ‘metricMXBean’

Better name might be ‘metricMXBean’

Got it. Thank you.


Multiple messages in Azure Storage Queue which contains duplicate message content, cannot find out who inserted duplicate message

Library name and version
Azure.Storage.Queues 12.14.0
Query/Question
Hello
I am facing strange behaviors in the Azure storage queue with results with a couple of messages containing duplicate content.
Application is moving items from the source system to the target system. The current architecture is as follows:

Windows service in VM is retrieving data from the source system and sends them to the service bus in batches. Using .NET 6
Azure function is processing a batch of items from the Service bus, each item is stored in Azure Table Storage and each item is separately sent to Azure Storage Queue.
Another Windows service in VM is reading data from the queue and ingressing items to the source system. When ingress is done for a specific item, the corresponding queue message is deleted from the queue

Every message in the Azure storage queue contains a single item with a unique id.  I noticed that some messages in the Azure storage queue contain duplicate item content. So I  have 2 messages in the queue, one message with Queue Message Id 1 and this message contains Item with id 1, and a second message with Queue Message Id 2, which contains the same item with Item id 1.
The Azure function is ingesting data to the storage queue with the following call
await QueueClient.SendMessageAsync(item.ToJson(), timeToLive: TimeSpan.FromSeconds(-1));
Windows service is reading messages from the queue with the following code
await QueueClient.ReceiveMessagesAsync(maxMessages: 32, visibilityTimeout: TimeSpan.FromHours(1));
Windows service deleting messages from the queue with this code
await this.QueueClient.DeleteMessageAsync(MessageId, PopReceipt);
During testing, I noticed this behavior:

Windows service receives in one batch 2 different queue messages with the same Item

Retrieve item from queue [migration-py6xs2lersgbjwluevzm] {"ItemId":"50613a9a-e081-f57c-dbd6-f854d947ffe1", "QueueMessageId":"04e02661-a977-403b-859f-f3d6da632a95"}
Retrieve item from queue [migration-py6xs2lersgbjwluevzm] {"ItemId":"50613a9a-e081-f57c-dbd6-f854d947ffe1", "QueueMessageId":"46499ca0-e6ac-42e7-9cea-56c9a10f20da"}

From the function logs in App Insight, I can see the message was sent to the storage queue only once, so this call should NOT create duplicates.

I enabled diagnostic settings for storage queues and logs are sent to Log analytics.

The result of the following Kusto query

StorageQueueLogs
| where  ObjectKey  has '04e02661-a977-403b-859f-f3d6da632a95' or ObjectKey has '46499ca0-e6ac-42e7-9cea-56c9a10f20da'
| project  TimeGenerated, OperationName, StatusCode, DurationMs, ServerLatencyMs, ObjectKey

return this

As you can see, in the output I can see only one PutMessage operation for a message with id 46499ca0-e6ac-42e7-9cea-56c9a10f20da. The second message I received from the queue with id 04e02661-a977-403b-859f-f3d6da632a95
doesn't have a corresponding PutMessage operation, so I have no idea how it can be inserted into the queue. Also from the output, you can see both messages were deleted from the queue, so I assume both of them really exists in the queue.
Also, the duration of the PutMessage operation took 2014 ms, while the rest of the put operations took less than 10 ms.

During more digging in Log analytics I noticed a couple of failed operations in the queue

StorageQueueLogs
| where TimeGenerated > ago(12h) and StatusText !contains "Success"
| summarize count() by OperationName
| top 10 by count_ desc

from which most of them are related to OperationTimedOut
StorageQueueLogs
| where TimeGenerated > ago(12h) and StatusText !contains "Success"
| summarize count() by StatusText
| top 10 by count_ desc

When I started looking more deeply at what is happening with the problematic queue, I noticed 2 PutMessage operations with StatusText OperationTimedOut. What is very suspicious, is that for mentioned queue I have exactly 2 duplicate items.
StorageQueueLogs
| where Uri has 'migration-py6xs2lersgbjwluevzm'
| where StatusCode == 500
| project  TimeGenerated, OperationName, StatusCode,StatusText, DurationMs, ServerLatencyMs, ObjectKey
| order by TimeGenerated asc

According to the times, the PUT operation for one of the duplicate items was executed at 2023-06-14T10:02:32.4264083Z and both OperationTimedOut PutMessage operations were executed around  2023-06-14T10:02:29.5077103Z, so this is before the PutMessage operation for message 46499ca0-e6ac-42e7-9cea-56c9a10f20da were executed
So I would like some help with answers to the following questions.

It is possible, that call for insert message to queue from Azure Function ended up in timeout at 2023-06-14T10:02:29.5077103Z, the retry logic in Azure.Storage.Queues SDK try the insert operation again and the retry was successful
at 2023-06-14T10:02:32.4264083Z and the message was inserted with message-id 46499ca0-e6ac-42e7-9cea-56c9a10f20da with corresponding PUT logs, but it is possible, that the first failed insert operation was still running in the background
and was finished successfully eventually with the creation of a second message containing duplicate content with message-id 04e02661-a977-403b-859f-f3d6da632a95, but without corresponding PutMessage operation log in log analytics?
If the previous assumption is not correct do you have any idea what can cause messages with duplicate contents in Azure Storage queue and how it is possible to insert a message to the Storage queue without the corresponding PutMessage log in Log analytics?
Or do you know some other things I can check to better understand what is happening inside the queue and how can I find what in the world inserted message with id 04e02661-a977-403b-859f-f3d6da632a95

To have better insight, I was able to insert 966015 messages to multiple queues and I have only 6 messages with content duplicates, so this can probably happen only during some bigger load to the queues.
According to this Kusto query, I can see I have plenty of queue operations with the longest latency around 3 minutes before the incident.
let window = 5m;
let eventTime = todatetime('2023-06-14T10:02:32.4264083Z');
StorageQueueLogs
| where TimeGenerated between ((eventTime - window) .. (eventTime + window))
| top 30 by DurationMs desc
| project TimeGenerated, OperationName, DurationMs, ServerLatencyMs, ClientLatencyMs = DurationMs - ServerLatencyMs

Any help with this issue is really appreciated, thank you very much.
Environment
Windows services using .NET6

Hi
After some more digging into this issue and after adding more logs, I have more info to share.
First of all, the original problem originate when we were using the following QueueClientOptions setup
var queueClientOptions = new QueueClientOptions();
queueClientOptions.MessageEncoding = QueueMessageEncoding.None;
queueClientOptions.Retry.Mode = RetryMode.Exponential;
queueClientOptions.Retry.MaxRetries = 5;

return new QueueServiceClient(config.StorageAccount, queueClientOptions);

I changed the setup for QueueClientOptions as following
var queueClientOptions = new QueueClientOptions();
queueClientOptions.MessageEncoding = QueueMessageEncoding.None;
//queueClientOptions.Retry.Mode = RetryMode.Exponential;
queueClientOptions.Retry.MaxRetries = 0;
queueClientOptions.Retry.Delay = TimeSpan.FromMilliseconds(0);

return new QueueServiceClient(config.StorageAccount, queueClientOptions);

As you can see, in the first example, I was trying to retry any failed operation, while in the second example retry should be turned off.
I ran more tests and noticed the following behavior.

For one call to insert an operation into Storage Queue I received the following exception

Failed to insert an item with ItemId [70be710c-1c3c-3018-eba5-caedc0b88921] to migration queue [migration-o2qtylpkmseg2kfhnjqr]  NOTE: ItemId is our internal identification.
And here is the response from Azure
Operation could not be completed within the specified time.
RequestId:a68f3c93-7003-0071-2192-a0952d000000
Time:2023-06-16T20:41:27.5213174Z
Status: 500 (Operation could not be completed within the specified time.)
ErrorCode: OperationTimedOut

Content:
<?xml version="1.0" encoding="utf-8"?><Error>OperationTimedOut<Message>Operation could not be completed within the specified time.
RequestId:a68f3c93-7003-0071-2192-a0952d000000
Time:2023-06-16T20:41:27.5213174Z</Message></Error>

Headers:
Server: Windows-Azure-Queue/1.0 Microsoft-HTTPAPI/2.0
x-ms-request-id: a68f3c93-7003-0071-2192-a0952d000000
x-ms-version: 2018-11-09
x-ms-error-code: OperationTimedOut
Date: Fri, 16 Jun 2023 20:41:26 GMT
Connection: close
Content-Length: 245
Content-Type: application/xml

I can see also a log in StorageQueueLogs for mentioned correlation

So I would assume, if the PutMessage operation failed with 500 OperationTimedOut, I would expect, that message is not created in the queue, but that is not the case here.

When I was receiving data from queue migration-o2qtylpkmseg2kfhnjqr for processing, I noticed, that we receive from queue one message, which contains our own item with id 70be710c-1c3c-3018-eba5-caedc0b88921, which according to the first point was NOT inserted into the queue

Retrieve item from queue [migration-o2qtylpkmseg2kfhnjqr]| {"ItemId":"70be710c-1c3c-3018-eba5-caedc0b88921", "MessageId":"5533d7b8-1d30-4bb1-beee-3589d051a895"}

When I check StorageQueueLogs for mentioned Queue MessageId, again, I can not find any PutMessage operation, only DeleteMessage, which we call when the processing of a message from the queue is done. So as I cannot find any PutMessage operation for queue message id 5533d7b8-1d30-4bb1-beee-3589d051a895, I believe the message was created in Queue in a call with correlation id a68f3c93-7003-0071-2192-a0952d000000 nevertheless, this call receive 500 error

So now I do not understand what is happening,

It is normal behavior, that if we received 500 OperationTimedOut from the queue, that message is created in the queue anyways?
Why the OperationTimedOut is received after 3 Ms?

This behavior explains, why I had duplicate messages in the queue in my first question, the original failed request caused an automatic retry, but in the end, both original and retried operations inserted messages into the queue.
Thanks for any help

Update/Sample app:
Hello/Good morning,
we can provide you with an update. We are able to simulate this issue from the simple console application. We would like to know:

if something is wrong with the code,
if duplicates are expected/possible on OperationTimeOut exception
is it some other issue related to SDK or Azure?

Here is the link to the sample application which you can try or take a look at: https://github.com/pgorbar/QueueDuplicatesDemo/blob/main/QueueDuplicatesDemo/Investigation.cs
In our sample run (20 queues filled in parallel each with 50k items). With every run, we end up with some duplicates (1 - 3) in some of the queues.
Thanks for any suggestions or answers.

@JerryBian we created a support ticket in Azure and received the following answer
I followed up internally and got the below update:

This is by design.  In these error conditions, a transaction may have been committed durably, but a failure and/or timeout could occur afterward.  With queues, this is a little challenging.  The guidance link says that if you get a 500: "Please query the server state before retrying the operation.".  But there isn't a great way to do this with queues (as far as we can tell) without peeking or popping messages.  And depending on the design, a Consumer may have already popped it before the Producer can Peek to see if it is there or not.  Given that this situation is unavoidable (a 500 may happen, and it may or may not have been committed on the backend), the customer will probably benefit from looking at the differences between "exactly once" and "at least once" designs and how to handle various cases.  There's a stack overflow thread that may be useful:  https://stackoverflow.com/questions/44204973/difference-between-exactly-once-and-at-least-once-guarantees

https://learn.microsoft.com/en-us/rest/api/storageservices/common-rest-api-error-codes

OperationTimedOut	Internal Server Error (500)	The operation could not be completed within the permitted time. **The operation may or may not have succeeded on the server side.** Please query the server state before retrying the operation.

we hoped that we can receive more info from this GitHub issue, but unfortunately, it is without any response. So we decided to abandon Azure storage queues for our solution and use a different approach,

@JerryBian we created a support ticket in Azure and received the following answer
I followed up internally and got the below update:

This is by design.  In these error conditions, a transaction may have been committed durably, but a failure and/or timeout could occur afterward.  With queues, this is a little challenging.  The guidance link says that if you get a 500: "Please query the server state before retrying the operation.".  But there isn't a great way to do this with queues (as far as we can tell) without peeking or popping messages.  And depending on the design, a Consumer may have already popped it before the Producer can Peek to see if it is there or not.  Given that this situation is unavoidable (a 500 may happen, and it may or may not have been committed on the backend), the customer will probably benefit from looking at the differences between "exactly once" and "at least once" designs and how to handle various cases.  There's a stack overflow thread that may be useful:  https://stackoverflow.com/questions/44204973/difference-between-exactly-once-and-at-least-once-guarantees

https://learn.microsoft.com/en-us/rest/api/storageservices/common-rest-api-error-codes

OperationTimedOut	Internal Server Error (500)	The operation could not be completed within the permitted time. **The operation may or may not have succeeded on the server side.** Please query the server state before retrying the operation.

we hoped that we can receive more info from this GitHub issue, but unfortunately, it is without any response. So we decided to abandon Azure storage queues for our solution and use a different approach,

Appreciate it.
Yeah, we have same feelings.. Azure storage queue is not our best choice, we have other issues as well. Maybe Service Bus queue is an candidate.

Anyone tried to downgrade to a previous version ? We have not seen these on 12.11.0 and below, but can see these issues almost every day causing severe impact to our customers.
In comparing above, we do not get any 500 error in the logs, only 201 on PUT but see in the Azure diagnostics that two PUTs exist for one SDK call.

We've opened a case as this can be simulated using the REST API as well. As the SDK wraps the API, Azure must solve this in the API.

Closing as this is not an SDK specific issue and it's a REST API issue


Issue 13263

Obsoletions for tRNA methylation involved in response to nitrogen starvation and regulation term.  New terms for tRNA methylation, regulation and tRNA C5-cytosine methylation, regulation.
@ukemi  - for some reason the Reviewers widget is getting hung up on 'Loading suggestions...' so I can't flag this for review but if you could take a look to see if there's anything that needs fixing, that'd be much appreciated.  Thx.

@vanaukenk  This all looks good to me, (disclaimer that I am notorious for missing typos) but as I was reading the diff I wondered if we remembered to include adding replaced_by and consider tags to obsolete terms in the SOPs. If not, we should.

Thanks @ukemi   I've added info about replaced_by and consider to the SOP Google doc.


Review Charm for Listing

PR for reviewing charm code as defined in: https://discourse.charmhub.io/t/reviewing-charms/11698.

It looks good! I left some notes that are more related to style.
I am missing the lib for airbyte-server relation.

Thanks for the review! I've addressed most of the comments in #11


chore: make credential dependencies platform-specific

What does this PR try to resolve?
Starting from #11993, we made cargo-credential-macos-keychain and cargo-credential-wincred able to build on all platforms. However, some of their dependencies are not. This PR turns them into platform specific dependencies to circumvent the situation.
How should we test and review this PR?
Run the following commands on all platforms Cargo supports.
cargo check --workspace --exclude cargo-credential-gnome-secret

@bors r+

:pushpin: Commit ad6c4144cc8eedd2fa74803855cc12bea465f811 has been approved by epage
It is now in the queue for this repository.

:hourglass: Testing commit ad6c4144cc8eedd2fa74803855cc12bea465f811 with merge 9fd25101703fe889ce374d5730d139275ec12b3a...

:sunny: Test successful - checks-actions
Approved by: epage
Pushing 9fd25101703fe889ce374d5730d139275ec12b3a to master...


MData2 ingest needs to support number format and co-ordinate reference variants

MData2 packets have two-byte message ID's, where the last byte is used to describe the number format (#7) and coordinate reference (#5).
The ability to generate a correct MID exists for output as part of #9, but adding unit tests for parsing the resulting data fails as the MData2 parser assumes a table which maps to 0xabc0.

[x] MData2 user-facing event types (XsensEventDataType_t) needs to support fixed-point and double typed data, and the array variations.
[x] User-facing event union XsensEventData_t needs to support fixed-point and double types
[x] xsens_mdata2_decode_field() needs to work out the data-type and coordinate frame from the last nibble of the inbound ID
[x] Find the type in the table without the last nibble applied
[x] Payload-into-union handling supports additional type possibilities
[ ] Refactor event handling structure to support co-ordinate reference metadata (enum for ENU, NED, NWU etc)
[ ] Existing test cases pass
[x] Generate test packets with varied data types
[x] Test case for each type (single, fixed-points, double) rotation matrix outputs
[x] Write new test cases which include conversion of fixed-point data back to floating point

This issue is blocked until #11 is resolved.


Update use of importlib.metadata.entry_points

As of Python 3.10, the functionality of the importlib.metadata.entry_points function has changed. Instead of returning a dict in Python 3.9 (see docs) it now returns an EntryPoints object (see docs), on which one should use select instead of get, and probably more.
Use of dict methods on the returned object is deprecated as of Python 3.10.

This deprecation is now taking effect in Python 3.12, meaning this package cannot work in Python 3.12 as it is now.


Internal Server Error (500)

I followed the setup instructions but keep getting an "Internal Server Error (500)" when testing the service hook:
Failed
Sent at: Wednesday, 30. October 2019 15:35:09
Message
Bug #5 (Some great new idea!) updated by Jamal Hartnett.
Error Message
Internal Server Error (500)

Any ideas what I'm doing wrong?

Have you tried to take a body of the message and post that to the service while in debug mode?
You can use the swagger endpoint to do it manually.

Hi,
Same here. Your repository needs a httpClient and I think it is not well regsitered in the service provider.
regards

Ok, I figured out the issue. The way you register the cliuent is wrong. Please just register the client factory :  services.AddHttpClient();
Then in RulesRepo, inject the factory and use it to create the client, someting like
`public class RulesRepo : IRulesRepo, IDisposable
{
private IOptions _appSettings;
private IHelper _helper;
private readonly IHttpClientFactory _clientFactory;
	public RulesRepo(IOptions<AppSettings> appSettings, IHelper helper, IHttpClientFactory clientFactory)
    {
        _appSettings = appSettings;
        _helper = helper;
		_clientFactory = clientFactory;
    }

    public async Task<RulesModel> ListRules(string wit)
    {
        string src = _appSettings.Value.SourceForRules;
		var client = _clientFactory.CreateClient();

		var request = new HttpRequestMessage(HttpMethod.Get, src + "/rules." + wit.ToLower() + ".json");

		var response = await client.SendAsync(request);
		if (response.IsSuccessStatusCode)
		{
			var json = await response.Content.ReadAsStringAsync();
			RulesModel rules = JsonConvert.DeserializeObject<RulesModel>(json);
			return rules;

		}
		else
		{
			return new RulesModel();
		}

    }`

You will have to handle bad requests...

Just remove the line services.AddTransient<IRulesRepo, RulesRepo>(); from Startup.cs

I removed the line but I'm still getting the Internal server error as response:
Status Code: 500 Reason Phrase: Internal Server Error HTTP Version: 1.1 Headers: {   Date: Thu, 13 Feb 2020 10:48:06 GMT   Set-Cookie: ARRAffinity=f36c8130531c157fc790e7052450319f91d9a1fed78535477b558504530e1fd5;Path=/;HttpOnly;Domain=autostatetransitions20200213102712.azurewebsites.net   Server: Microsoft-IIS/10.0   X-Powered-By: ASP.NET   Content-Length: 0 }

Please enable stdout logging in your web.config to see the exact error. This is very general, can't help

Same issue for me, don't think this code has been tested properly

@TAK-WTW @Gizmo0815 what happens when you step through the code in debug mode?

@danhellem The implementation of this solution was too cumbersome for me. I created an Azure Logic App that traggers on updates of tasks, gets the parent work item and sets its state to active.
Here's the logic app, if someone wants to do the same:

@danhellem The implementation of this solution was too cumbersome for me. I created an Azure Logic App that triggers on updates of tasks, gets the parent work item and sets its state to active.
Here's the logic app, if someone wants to do the same:

Excellent suggestion @Gizmo0815 ! I will also definitely look into this instead of trying to deploy a full web app into the Azure Infrastructure unless some complicated processing was needed.

Wondering how much do you incur in monthly charge on average if when you are using the logic app?

That depends on the usage of your DevOps. For our region an execution costs 0,000025 US$. We have a team of 30 developers using Azure DevOps. On an average day we have ~100 updates that trigger the logic app. Assuming 20 working days per month the execution costs are somewhere around 5 cts/mo.
To get an impression for your organization I would set up a webhook that fires on updates of tasks and puts a message into an Azure queue for one day. At the end of the day you can just count the messages in the queue and do the math for your organization.

Thank you @Gizmo0815 for the nice suggestions. I am less familiar with the Azure Logic Apps and hence please pardon my  questions. Is there any other monthly integration fees that needs to be paid with the logic apps integration with the Azure DevOps ?

@bbarman4u I checked the costs on the azure website and found out that my last post was partially incorrect. Each action used within the logic app costs 0,000025 US$ (not one execution of the whole app). For details on pricing and connectors check out https://azure.microsoft.com/en-us/pricing/details/logic-apps/.

@bbarman4u I checked the costs on the azure website and found out that my last post was partially incorrect. Each action used within the logic app costs 0,000025 US$ (not one execution of the whole app). For details on pricing and connectors check out https://azure.microsoft.com/en-us/pricing/details/logic-apps/.

@Gizmo0815 Thanks again for this suggestion.
I tried out by creating a logic app and integrating into the Azure DevOps. The flow of the logic app is like this and works fine for my current needs. Also I checked that the billing cost is quite minimal on a daily basis. For the complex needs I guess I will have to try out this extension.


BUG Installation does not finish (2004 Win 10 Pro)

Describe the bug and expected behavior
When installing commando-vm to a new Windows 10 Pro VM the installation process does not finish, but stop at some point (after one or two restarts, see picture below)
To Reproduce
Steps to reproduce the behavior:

Setup base 2004 Windows 10 Pro VM (Virtualbox, 100GB, offline install)
Download install.ps1
Disable tamper protection
Go to download folder and execute Unblock-File .\install.ps1 and Set-ExecutionPolicy Unrestricted -f
Start the install script .\install.ps1
Confirm to try and install updates automatically
Start installation

Screenshots
First screenshot shows the start of the installation script, second the confirmation screen and the third shows where the  installation stops.

 If you need more information I am happy to help.

I forgot to mention:
I tried to restart the installation with the -nochecks flag like mentioned in the install notes (screenshot 2). But then after a few seconds it stops with the following error:

Funny, I was coming on here to post the exact same issue. Seems to be failing at installing proxycap.fireeye.

try a ps command
cinst -y proxycap.fireeye
restart

Executing that command (after the installation "aborted") and restarting seems to fix it. The installation is resuming, I will update this comment when it (hopefully) finishes. Thanks!

fixed with june update #262


windows没法装deepspeed吧 ，有人验证过没？

Looking in indexes: https://pypi.tuna.tsinghua.edu.cn/simple
Collecting deepspeed==0.8.3
Using cached https://pypi.tuna.tsinghua.edu.cn/packages/0f/c0/9b57e9ec56f6f405726a384b109f8da1267e41feea081850c2fce1735712/deepspeed-0.8.3.tar.gz (765 kB)
Preparing metadata (setup.py) ... error
error: subprocess-exited-with-error
× python setup.py egg_info did not run successfully.
│ exit code: 1
╰─> [17 lines of output]
No CUDA runtime is found, using CUDA_HOME='C:\Program Files\NVIDIA GPU Computing Toolkit\CUDA\v11.7'
test.c
LINK : fatal error LNK1181: 无法打开输入文件“aio.lib”
Traceback (most recent call last):
File "", line 2, in
File "", line 34, in
File "C:\Users\1\AppData\Local\Temp\pip-install-yqlqsg3j\deepspeed_43959dd1e4284318a6a8bc41546a4984\setup.py", line 156, in
abort(f"Unable to pre-compile {op_name}")
File "C:\Users\1\AppData\Local\Temp\pip-install-yqlqsg3j\deepspeed_43959dd1e4284318a6a8bc41546a4984\setup.py", line 48, in abort
assert False, msg
AssertionError: Unable to pre-compile async_io
[WARNING] Torch did not find cuda available, if cross-compiling or running with cpu only you can ignore this message. Adding compute capability for Pascal, Volta, and Turing (compute capabilities 6.0, 6.1, 6.2)
DS_BUILD_OPS=1
[WARNING]  async_io requires the dev libaio .so object and headers but these were not found.
[WARNING]  If libaio is already installed (perhaps from source), try setting the CFLAGS and LDFLAGS environment variables to where it can be found.
[WARNING]  One can disable async_io with DS_BUILD_AIO=0
[ERROR]  Unable to pre-compile async_io
[end of output]
note: This error originates from a subprocess, and is likely not a problem with pip.
error: metadata-generation-failed
× Encountered error while generating package metadata.
╰─> See above for output.
note: This is an issue with the package mentioned above, not pip.
hint: See above for details.

Thanks for the attention on LMFlow. Our support for Windows/Mac OSX is still testing and not fully prepared yet.
As a first step, we recommend you use Linux servers, Linux virtual machines, Docker or Google Colab to run experiments, which provides free T4 GPUs and V100/A100 if you have Google Pro/Pro+ account.
我们对Windows以及Mac OSX的支持仍在开发测试中，建议大家先使用Linux服务器、Linux虚拟机、Docker或者Google Colab来进行实验，Google Colab上有免费的T4 (16GB显存）GPU，以及收费的V100（16GB显存）和A100（40GB显存）GPU（需要Google Pro/Pro+用户）🙏

Thank you, please ask this recommended configuration and the minimum configuration of what graphics cards are required

Thanks for the attention on LMFlow. Our support for Windows/Mac OSX is still testing and not fully prepared yet.
As a first step, we recommend you use Linux servers, Linux virtual machines, Docker or Google Colab to run experiments, which provides free T4 GPUs and V100/A100 if you have Google Pro/Pro+ account.
我们对Windows以及Mac OSX的支持仍在开发测试中，建议大家先使用Linux服务器、Linux虚拟机、Docker或者Google Colab来进行实验，Google Colab上有免费的T4 (16GB显存）GPU，以及收费的V100（16GB显存）和A100（40GB显存）GPU（需要Google Pro/Pro+用户）🙏

Thank you, please ask this recommended configuration and the minimum configuration of what graphics cards are required

Thanks for the attention on LMFlow. Our support for Windows/Mac OSX is still testing and not fully prepared yet.
As a first step, we recommend you use Linux servers, Linux virtual machines, Docker or Google Colab to run experiments, which provides free T4 GPUs and V100/A100 if you have Google Pro/Pro+ account.
我们对Windows以及Mac OSX的支持仍在开发测试中，建议大家先使用Linux服务器、Linux虚拟机、Docker或者Google Colab来进行实验，Google Colab上有免费的T4 (16GB显存）GPU，以及收费的V100（16GB显存）和A100（40GB显存）GPU（需要Google Pro/Pro+用户）🙏

Thanks! The configuration may vary if you use different models. According to our experience, finetuning a 7b models can be done in a RTX 3090 (24GB) GPU card. For larger models, you may need quantization or larger GPU memory. Hope that answers your question. Thanks 😄

This issue has been marked as stale because it has not had recent activity. If you think this still needs to be addressed please feel free to reopen this issue. Thanks!

No CUDA runtime is found, using CUDA_HOME='C:\Program Files\NVIDIA GPU Computing Toolkit\CUDA\v11.7'
在window10 安装 python 环境中安装
conda install pytorch torchvision torchaudio pytorch-cuda=11.7 -c pytorch -c nvidia
详情见https://pytorch.org/get-started/locally/


[CSStep] Conjunction: Stop after first element ambiguous or fixed element

Stop a conjunction after the first invalid element in diagnostic mode.
No need to run the inference for other elements once the problem has been
found. All ambiguities and solutions with fixes are propagated back to the
outer context to be diagnosed.

I have also discovered a problem in findBestSolutions which prevents it from filtering out some of the solutions based on score if a single best couldn't be identified, going to fix that separately.

@swift-ci please test Windows platform


Add support for optionally startable services

Problem:
I have a relatively large process-compose service graphs, some with services that are seldomly wanted to be running.
Still whenever they are needed, it would be nice to have the dependency mechanism/retries of process-compose.
Proposal:
Allow process-compose services to be in "Not Started" state by some config parameter, so the TUI user can start/stop them manually.

Hi @thenonameguy
Have you tried to use the disabled flag?
some_process:
  command: "..."
  disabled: true

It allows you to skip it in execution, but start it manually from TUI if needed.

Good catch @thenonameguy :)
Documented


Merge TopicMessages by topic in sendBatch

Fixes #622
I'm not entirely satisfied with the testing here. Basically what I'm asserting is that the expected number of messages are produced, but I'm not asserting message ordering.
Unfortunately I can't really validate that without consuming the topic, which means that the producer test depends on the consumer. Maybe that's okay? On the other hand, the logic is pretty trivial, but still.

Implementation looks fine to me. Test wise, I think it's probably worth consuming the messages we've produced in batch to verify their order has been preserved.

I should learn to trust my gut, because of course we missed something and the implementation didn't actually work. I was using merged.find to find the TopicMessages for each topic, and was relying on mutating that to add the messages, but that didn't actually work:
const topicBatch = merged.find(({ topic: batchTopic }) => batchTopic === topic)

if (!topicBatch) {
  merged.push({ topic, messages })
} else {
  // this doesn't actually mutate `merged`
  topicBatch.messages.concat(messages)
}

return merged

Noticed when I updated the tests to actually consume the produced messages.


Add button to center map

Adds a new button to recenter the map to the default position and zoom level.

Great job creating this feature simply and efficiently! It works great.

Excellent work, really clean fix that doesn't overcomplicate the task by utilizing built-in mapbox functionality

Great job! I spent an hour looking for this functionality in the mapbox documentation, glad you found it!


getQueueLength() ?

Hi and many many thanks for the great work you are sharing with the community.
Do you envision any function returning the current fill grade of the command queue ?
In particular, I do only want to add a command if the queue is either empty or has just one single more task do.  I currently see no easy way of checking this.  The function isQueueEmpty() is not generic enough.
Thank you, Jhr1972

Sure. The latest commit add these functions, which should fulfill your needs:
  // functions to get the fill level of the queue
  //
  // To retrieve the forward planning time in the queue ticksInQueue()
  // can be used. It sums up all ticks () of the not yet processed commands.
  // For pauses, the summed up value is entry.ticks.
  // For steps, the summed up value is entry.steps*entry.ticks
  uint32_t ticksInQueue();

  // This function can be used to check, if the commands in the queue
  // will last for <min_ticks> ticks. This is again without the
  // currently processed commands.
  bool hasTicksInQueue(uint32_t min_ticks);

  // This function allows to check the number of commands in the queue.
  // This is including the currently processed command.
  uint8_t commandsInQueue();

The first two only needed to be exposed. The last one is new.

funny commandsInQueue() has too an existing implementation. So I used that instead:
  // This function allows to check the number of commands in the queue.
  // This is including the currently processed command.
  uint8_t queueEntries();

WOW. That was quick! Will pull and try tomorrow. Thank you

Seems there are some issues with queueEntries(). Pls compare the following snipplets:
         Serial.print("qes premove: ");
         qes = stepper->queueEntries();
         Serial.print (qes);
         stepper->move(-moveIntervall);
         stepper->move(-moveIntervall);
         stepper->move(-moveIntervall);
         delay(20);
         Serial.print(" qes postmove: ");
         qes = stepper->queueEntries();
         Serial.println (qes);

This will result in the following printout:
qes premove: 0 qes postmove: 10
However, I would have expected
qes premove: 0 qes postmove: 3
If I remove the delay(20) then I get
qes premove: 0 qes postmove: 0
I would have though that every moveTo() statement is one record in the queue that piles up. Is this a bug in the function or to I misunderstand the concept of queues in this context ?
Thank you!

The stepper->move() are not blocking. So this sequence will actually boil down to stepper->move(-3*moveIntervall)
One moveTo are either:

pause (up to 4ms)
steps (intervall up to 4ms), number of steps 1-255

Any step rate lower than 250 Hz will require pauses between steps.
In addition a move() will implement acceleration/deceleration, which results in several queue entries.
How many queue-entries have been created after a complete move, you cannot see, because the max number of reported commands will be 31.
I would expect after delay(20) you will have qes premove: 0 qes postmove: 0 and not immediately after the move().


update to 5.3.0 and py310

Checklist

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

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

@conda-forge-admin, please rerender

@conda-forge-admin, please rerender

@conda-forge-admin, please rerender

@thewtex I can't make it work for win. Maybe you know what needs to be done.

@conda-forge-admin, please rerender

@conda-forge-admin, please rerender

@thewtex If you think it's ok, please merge in order to roll the ball for itkwidgets as well.

hello. please merge this one. Thanks

@conda-forge-admin, please rerender

@conda-forge-admin, please rerender

@conda-forge-admin, please rerender

@conda-forge-admin, please rerender


fix: was using table name instead of model name for "references" property

References model name was using table name directly instead of
first converting with getModelNameFor().

I did not change the tests as I'm still not able to run them

Thank you for your contribution. I merged it and made a minor change in a test.


[🐛 Bug]: npx @cloudflare/next-on-pages fails on windows

next-on-pages environment related information
PS C:\Users\tdesmond\Projects\cloudflare\temp\my-next-app> npx @cloudflare/next-on-pages --info
EXIT WHEN NOT EXPECTED
SHELLAC COMMAND FAILED!
Executing: npm --version in C:\Users\tdesmond\Projects\cloudflare\temp\my-next-app
STDOUT:
STDERR:
/bin/bash: line 1: cd: C:\Users\tdesmond\Projects\cloudflare\temp\my-next-app: No such file or directory
node:internal/process/promises:389
new UnhandledPromiseRejection(reason);
^
UnhandledPromiseRejection: 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(). The promise rejected with the reason "#".
at throwUnhandledRejectionsMode (node:internal/process/promises:389:7)
at processPromiseRejections (node:internal/process/promises:470:17)
at process.processTicksAndRejections (node:internal/process/task_queues:96:32) {
code: 'ERR_UNHANDLED_REJECTION'
}
Node.js v20.16.0
Description
npx @cloudflare/next-on-pages fails on my Windows machine. At this point it feels like something is configured wrong on my machine but I have not come across documentation pointing me to what I should set up.
I have tried using the Windows Subsystem for Linux like the error for npm run page:build prompts me to do but there is no change in error when running in a Ubuntu terminal on Windows.
Reproduction
I created a brand new Next.js application with npm create cloudflare@latest my-next-app -- --framework=next.
I attempt to run npm run pages:build I get the following error.
/mnt/c/Users/tdesmond/Projects/cloudflare/temp/my-next-app$ npm run pages:build

><EMAIL_ADDRESS>pages:build
> npx @cloudflare/next-on-pages

⚡ @cloudflare/next-on-pages CLI v.1.13.2

⚡ Warning: It seems like you're on a Windows system, the Vercel CLI (run by @cloudflare/next-on-pages
⚡ to build your application) seems not to work reliably on Windows so if you experience issues during
⚡ the build process please try switching to a different operating system or running
⚡ @cloudflare/next-on-pages under the Windows Subsystem for Linux

EXIT WHEN NOT EXPECTED

SHELLAC COMMAND FAILED!
Executing: npm --version in C:\Users\tdesmond\Projects\cloudflare\temp\my-next-app

STDOUT:

STDERR:

/bin/bash: line 1: cd: C:\Users\tdesmond\Projects\cloudflare\temp\my-next-app: No such file or directory

⚡ Unexpected error: {"retCode":1,"cmd":"npm --version","stdout":"","stderr":"/bin/bash: line 1: cd: C:\\Users\\tdesmond\\Projects\\cloudflare\\temp\\my-next-app: No such file or directory
⚡ "}

Pages Deployment Method
None
Pages Deployment ID
No response
Additional Information
No response
Would you like to help?

[X] Would you like to help fixing this bug?

do you see this when just building the same app without next-on-pages?

npm run build passes. I deployed the app to Pages as well and it builds on Pages.
So definitely looking like a local setup issue. But I do not know what is missing locally.

I was able to get this to run by fudging a few lines in index.js to play nicely with WSL. I'm sure there's a better way but this seems to work for now...
Update Command to format cwd to WSL format (/mnt/ at start, flipping slashes and lowercasing drive letter, removing ':' )
var Command = class {   shell;   cmd;   cwd;   interactive;   exec;   runningState;   pipe_logs;   exit_expected;   retCode;   promiseResolve;   promiseReject;   promise;   timer;   stdout;   stderr;   constructor({     cwd,     shell,     cmd,     interactive,     pipe_logs = false,     exit_expected = false   }) {     this.shell = shell;     this.cmd = cmd;     this.cwd = '/mnt/' + cwd.split('\:')[0].toLowerCase() + cwd.split(':')[1].replace(/\\/g, '/');     this.interactive = interactive;     this.exit_expected = exit_expected;     this.exec = cd "${this.cwd}" &&
${this.cmd};echo END_OF_COMMAND[$?]_
;
Add shell: true to options of vercel build child process spawn
async function getVercelBuildChildProcess(pm, additionalArgs = []) {   console.log("trying vercel")   const spawnCmd = await pm.getRunExecStruct("vercel", {     args: ["build", ...additionalArgs],     download: "prefer-if-needed"   });   console.log(spawnCmd);   if (!spawnCmd) {     throw new Error("Error: Failed to generate vercel build command");   }   return (0, import_child_process2.spawn)(spawnCmd.cmd, spawnCmd.cmdArgs, {shell: true}); }

Same problem on Windows:

same issue on WSL Ubuntu


Improve Area codes recognition for Nigeria

Fixes https://github.com/floere/phony/issues/450
The area codes parsing code is changed from a 3, 4 split to a 3, 3 one, with more strict rules on the codes.
Doc links are also updated as the previous one was broken.

Coverage decreased (-4.6%) to 93.225% when pulling 0c02c39bf18100b514f37cf7e24ad2ee5d1582d9 on skateinmars:chore/nigeria-area-codes into 495452cd3673e76d9bc5d29b5d7e7a8dda7eacc5 on floere:master.

@skateinmars Thanks for your excellent contribution! 🙇🏻♂ Released in 2.18.10 🚀


models from a package whose name conflicts with other packages

UPD:
But if i have smth like this in imports
"ordering/internal/gateways/http/presenters/api/products/v1/create"
_ "ordering/internal/gateways/http/views/api/products/v1/create" // here is JsonApiAttributesV1
It failes with message ParseComment error in file ordering\application\internal\gateways\http\controllers\api\products\v1\create.go :cannot find type definition: create.JsonApiAttributesV1
Originally posted by @doublebigguy in https://github.com/swaggo/swag/issues/1353#issuecomment-1296534328

Enhancements:

search imports sequencely,  till find the type.
consider every import mode ,such as alias，_,  .
a model imported from an external by mode .

View details in the testdata


Update step-by-step guide for windows

Update step-by-step guide for windows to give an alternative to make style.

Hi @thomwolf, for simplification purposes, I think you could remove the "pip install ..." steps from this commit, 'cause these deps (black, isort, flake8) are already installed on pip install -e ".[dev]" on the Start by preparing your environment


Please require jquery in your component

Please put
var $ = require('jquery')
in your FullCalendar.vue.
I don't need jquery but have another lib which defines $
And I think if your component depends on jquery, it should require it by it self.

I'm not sure if the $ definition is enough for fullcalendar to work as it is a jQuery plugin. See this codepen, the console ouput gives the following
Uncaught ReferenceError: jQuery is not defined

What do you have that defines $?

fullcalender imports jquery too. So we don't have to care about that.

But window.jQuery needs to be defined for jQuery to load plugins I think.
So even if I am importing jQuery into the component.
import $ from 'jQuery'

We are still going to see
Uncaught ReferenceError: jQuery is not defined

Maybe I'm wrong?

Oh sorry, I see what you mean.
Ok, I'll commit something into a new branch and if you could test with your setup I will merge it in when we are ready.

Thank you.
Now it works with my setup.

Perfect, I will merge it in.
Thanks for your contribution


Context menu callback function on custom panel receive an unexpected object in VSCode 1.44

Hi, team
In VSCode 1.44, context menu call back function would receive a object like below:
{
isTrusted:"true"
}
Our context menu settings is like below:
"menus":{
   "view/title":[
      {
         "command":"azure-iot-toolkit.updateDistributedTracingSetting",
         "when":"view == iotHubDevices",
         "group":"azure-iot-toolkit-4@1"
      }
   ]
}

In previous version, it just received null
Is this by design or just a bug? It break our extension because we use this parameter to determine where it triggered.

In VSCode 1.44, context menu call back function would receive a object like below:

How about repro steps?

Is this by design or just a bug? It break our extension because we use this parameter to determine where it triggered.

Generally, your extension must be fit for gracefully handling any kind of parameter, using the parameter to determine the trigger location is flawed.

I cannot repro it with our sample here - https://github.com/microsoft/vscode-extension-samples/tree/master/tree-view-sample
Please provide us the repro.

I stumbled across this issue as well with the Test Explorer extension.
Here's how to reproduce with the tree-view-sample:

Remove this line in package.json (it doesn't happen when you set "group": "navigation")
Change this line to vscode.commands.registerCommand('jsonOutline.refresh', offset => jsonOutlineProvider.refresh(offset)); - the expectation would be that offset will be undefined
Set a breakpoint here
Start the extension in the debugger, open the "JSON OUTLINE" view and select "Refresh" from its main menu

Voila:

your extension must be fit for gracefully handling any kind of parameter

<rant>
Sorry, but that is just close to impossible - what if I get a parameter that looks very similar to what I expect but is subtly different? The amount of validation that extension authors would have to do every single time they interact with VS Code is just crazy.
A public API should be well-defined and not throw random stuff at anyone interfacing with it and telling them "you figure it out yourself, good luck with that".
</rant>

A public API should be well-defined and not throw random stuff at anyone interfacing with it and telling them "you figure it out yourself, good luck with that".

export function registerCommand(command: string, callback: (...args: any[]) => any, thisArg?: any): Disposable; - I read a clear ...args: any[] which is well defined and it means, it can be anything. Obviously we don't call this with random stuff but via the executeCommand-API any extension can invoke this command, any user can bind this to a keybinding etc etc

Sorry, but that is just close to impossible - what if I get a parameter that looks very similar to what I expect but is subtly different?

Really? You are expecting objects from your domain, right? How about an instance-of check and if that fails do nothing or, if possible, have a default.

via the executeCommand-API any extension can invoke this command, any user can bind this to a keybinding etc etc

I don't expect the commands in question to be called "from outside", i.e. they should only by called from my own code or by VS Code's tree implementation. Of course I can't enforce this, but the fact that they don't show up anywhere in the docs and are excluded from the command palette should be a strong hint that these are considered internal and shouldn't be relied on in any way by others.

I read a clear ...args: any[] which is well defined and it means, it can be anything

Of course command arguments can be anything in general, but I would still expect to be able to rely on how VS Code's tree implementation calls commands. Otherwise extensions will break unnecessarily (even if they do so gracefully, the functionality would still become inaccessible).

How about an instance-of check

... this is exactly what I ended up doing - I only realized after writing my rant that instanceof provides the perfect solution in this case (because I expect objects from my domain).

thank you @hbenl for your steps to repro the issue.
Hi, @jrieken , my question is what's the purpose of this parameter { isTrusted: true}? Only triggered from the context menu would contain this parameter, however, if I triggered from command palette, it would be null, if it is by design, please add some document about this parameter.

Hi, @jrieken , my question is what's the purpose of this parameter { isTrusted: true}

There is none and shouldn't be there -> ping @sandy081

I really can't see this {isTrusted: true} with the given steps

@jrieken Do you think if extensions can inject these parameters some how when commands are executed?

No, there is no way to inject this. The {isTrusted: true} smells like markdown command links...

It seems we need assistance from users that how this is getting added

I really can't see this {isTrusted: true} with the given steps

That's because you're using a Mac. I just checked and I can't reproduce it on a Mac either, but on Linux and Windows I can.

@alexr00 Can you please try to repro and see where the unexpected args are coming from?

I do see the arg:

The action bar onClick passes a MouseEvent as context here:
https://github.com/microsoft/vscode/blob/fd85f8d631c8f6edded8cac46e6533f7903bb44b/src/vs/base/browser/ui/actionbar/actionbar.ts#L192-L192
That contains the isTrusted and it get's passed all the way through to the command.

@bpasero any insights ?

Is this maybe an issue with native contextmenu vs custom context menu? Does that make any difference?
To be clear: when running a extension command via context menu we are now passing {isTrusted: true} and before we did not? I only see this string being used with markdown?

Yes, we were not passing this before.
@alexr00 Can you please help in checking with native & custom context menu?

I'm hitting this in the Kubernetes extension.  A command on the title bar of a custom view that was previously being invoked with no argument is now being invoked with this { isTrusted: true } value and it has broken our users.  https://github.com/Azure/vscode-kubernetes-tools/issues/741  I can provide repro details if it would be helpful.

When I set "window.titleBarStyle": "native" I do not get {isTrusted: true}

@sbatten This seems to be an issue with custom menus and we should make sure we fix this for the next release.

Thanks @alexr00 for checking this.

@bpasero @jrieken I don't see how this could have been introduced in 1.44, so please review my fix. Based on the comment here from long ago, I have removed the passing of the event used to create the action context altogether from both native and custom menus. This means only actions that provide a context will get one.

I was about to comment and then saw https://github.com/microsoft/vscode/issues/55921#issuecomment-412757167 which is exactly what I still think. Passing the event as context into actions is bogus imho, so I like this change. It is hard though to find any code that relies on the event being passed in...


oclif rejects empty string given for required argument

Do you want to request a feature or report a bug?
Report a bug
What is the current behavior?
oclif rejects an empty string argument when a string argument is requried
Example code:
import {Command, flags} from '@oclif/command'

class Foo extends Command {
  static description = 'describe the command here'

  static args = [{name: 'file', required: true}]

  async run() {
    const {args, flags} = this.parse(Foo)
  }
}

export = Foo

Example output:
$ ./bin/run
 ›   Error: Missing 1 required arg:
 ›   file
 ›   See more help with --help
$ ./bin/run ''
 ›   Error: Missing 1 required arg:
 ›   file
 ›   See more help with --help
$ ./bin/run 'foo'
$

The first error is expected (we omitted the argument). The second is not (we supplied an empty string as argument). In the 3rd case, the command succeeds as expected.
What is the expected behavior?
An empty string argument should be accepted.
I'm using oclif/command 1.8.0 on Ubuntu 20.04.

Many thanks for fixing this issue! I used argparse for the project I was working on 3 years ago, as oclif had this problem, and yargs had another similar edge case which their devs marked WONTFIX (single - passed as positional arg disappeared), so it's good to know that in future I can just use oclif again!


per metric dimensions

This PR makes it possible to provide dimensions per metric. Typically by doing something like this;
final DimensionedName dimensionedName = DimensionedName.withName("test")
        .withDimension("key1", "val1")
        .withDimension("key2", "val2")
        .withDimension("key3", "val3")
        .build();

metricRegistry.counter(dimensionedName.encode()).inc();

Sadly AWS doesn't support aggregating custom metrics on dimensions (yet?), so the usefulness is more or less limited to improved search and filtering in the CloudWatch console for now...
Related to: https://github.com/azagniotov/codahale-aggregated-metrics-cloudwatch-reporter/issues/19

Hi @jebl01, I released a new version


Script line gives error Specify a unique value.

$AllUsers | foreach {Get-MailboxJunkEmailConfiguration -Identity $_.Identity} | etc
Identity          Enabled BlockedSendersAndDomains

...
Multiple mailbox users match identity "xxxxxxxx". Specify a unique value.
This script line is better, because now it can handle every mailbox. Also foreach with Capitals.
$AllUsers | ForEach {Get-MailboxJunkEmailConfiguration -Identity $_.Guid.ToString()}

Hi @parlevjo2 , thank you for your feedback.
I couldn't find the article you are talking about. Could you gently let me know that it is? Thank you!

It was https://docs.microsoft.com/en-us/powershell/module/exchange/antispam-antimalware/get-mailboxjunkemailconfiguration?view=exchange-ps. It is already fixed.
Fix is: foreach {Get-MailboxJunkEmailConfiguration -Identity $_.UserPrincipalName}

Thank you!


continue button does not work

This repository is only for reporting bugs or issues. If you need support, please use
other channels:

Write an email -<EMAIL_ADDRESS>

Create support ticket on https://bagisto.uvdesk.com

Visit forums to get support from our community (https://forums.bagisto.com)

Alternatively, you may use Facebook (https://www.facebook.com/groups/bagisto/).
We promise that more channels are coming soon!!!

@shazej
please describe the issue in detail or you may also show a screenshot of the bug.

I seem to have the same issue.
Console says:
script.js:68 Uncaught TypeError: Cannot read property 'style' of null
    at HTMLButtonElement.myFunction (script.js:68)

You can take a look here: https://machneyehuda.com/installer/

@jadeye
Bagisto is successfully installed at provided URL https://machneyehuda.com/ so closing this issue,

bagisto cannot be installed on digitalocean managed database as there is no
primary keys set on the database
On Sun, Aug 30, 2020 at 12:16 PM Vaishali Agarwal<EMAIL_ADDRESS>wrote:

@shazej https://github.com/shazej
please describe the issue in detail or you may also show a screenshot of
the bug.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/bagisto/bagisto/issues/3848#issuecomment-683396680,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEUX26B2KNC2QLUL3L46YFDSDIKG7ANCNFSM4QPH2Z4Q
.

--

Sincerely,
shaz saleem ej
<EMAIL_ADDRESS>/
+97455170700
+97455170700
Al numan street, Doha-Qatar,P.B.No:5483
https://htmlsig.com/signatures/0001C5R4CY
[image: Facebook]  https://htmlsig.com/t/000001CD2WV9 [image: Google +]
https://htmlsig.com/t/000001C9KZHZ [image: LinkedIn]
https://htmlsig.com/t/000001CA5QWW
This e-mail message may contain confidential or legally privileged
information and is intended only for the use of the intended recipient(s).
Any unauthorized disclosure, dissemination, distribution, copying or the
taking of any action in reliance on the information herein is prohibited.
E-mails are not secure and cannot be guaranteed to be error free as they
can be intercepted, amended, or contain viruses. Anyone who communicates
with me by e-mail is deemed to have accepted these risks. I am not
responsible for errors or omissions in this message and denies any
responsibility for any damage arising from the use of e-mail. Any opinion
and other statement contained in this message and any attachment are solely
those of the author.

now this is the issue.
Illuminate \ Contracts \ Container \ BindingResolutionException
Target class [encrypter] does not exist.
Previous exceptions

Class encrypter does not exist (-1)

[image: image.png]
On Sun, Aug 30, 2020 at 12:16 PM Vaishali Agarwal<EMAIL_ADDRESS>wrote:

@shazej https://github.com/shazej
please describe the issue in detail or you may also show a screenshot of
the bug.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/bagisto/bagisto/issues/3848#issuecomment-683396680,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEUX26B2KNC2QLUL3L46YFDSDIKG7ANCNFSM4QPH2Z4Q
.

--

Sincerely,
shaz saleem ej
<EMAIL_ADDRESS>/
+97455170700
+97455170700
Al numan street, Doha-Qatar,P.B.No:5483
https://htmlsig.com/signatures/0001C5R4CY
[image: Facebook]  https://htmlsig.com/t/000001CD2WV9 [image: Google +]
https://htmlsig.com/t/000001C9KZHZ [image: LinkedIn]
https://htmlsig.com/t/000001CA5QWW
This e-mail message may contain confidential or legally privileged
information and is intended only for the use of the intended recipient(s).
Any unauthorized disclosure, dissemination, distribution, copying or the
taking of any action in reliance on the information herein is prohibited.
E-mails are not secure and cannot be guaranteed to be error free as they
can be intercepted, amended, or contain viruses. Anyone who communicates
with me by e-mail is deemed to have accepted these risks. I am not
responsible for errors or omissions in this message and denies any
responsibility for any damage arising from the use of e-mail. Any opinion
and other statement contained in this message and any attachment are solely
those of the author.


several errors upon creating the CloudFormation stack

followed all instructions, receive this for CF events....at first I did not have unique S3 buckets I don't think. after I got past that, I renamed the new stack to DetectiveControls2 because it said the stack already existed even though it did not exist.
Later, I get this error - in 2 regions I try it in:
Unresolved resource dependencies [CloudTrailDestinationBucket] in the Outputs block of the template

Unresolved resource dependencies [CloudTrailDestinationBucket] in the Outputs block of the template

@benjipotter is this issue possibly for https://github.com/awslabs/aws-well-architected-labs/blob/master/Security/200_Automated_Deployment_of_Detective_Controls/Lab_Guide.md?

did I post this in the wrong section? yes, it's the 200 lab, Detective Controls that I had several issues deploying as instructed. Any logs I can provide?

I was able to reproduce the error that @SebouhAra reported.
If you select 'No', for all of the services (CloudTrail, Config, GuardDuty), the template will fail because the output dependency is not met, generating the error from OP.
I was able to get around this by enabling only the CloudTrail service (I already had Config and GD set up) and specifying buckets to send the CloudTrail and Config audit to.
One way to fix this would be to not make CloudTrail setup optional. Arguably, if you don't need CloudTrail, you don't need this lab. Any other thoughts?

I have fixed by making CloudTrail non-optional, and it will simply create an additional trail (and bucket, log group) if you have configured already. You can check it out now: https://www.wellarchitectedlabs.com/security/200_labs/200_automated_deployment_of_detective_controls/


support azurerm backend storage for tfstate files

A number of the terraform stacks I work with are configured with azurerm backend values, which doesn't currently appear to be supported, based on documentation and examples.
I'd like to request support for azurerm-backed terraform be built in; there's already support for s3 at this point, so this seems like a logical step, and equivalent functionality would be a boon for those of us working in azure storage accounts & containers for tfstate management.
Thanks.

i would like this not to expire, if possible.  if there is a resource issue, i can maybe pitch in


RxMoya watchOS yields Carthage warnings

Just stumbled upon this Carthage output for RxMoya watchOS. I do not have time to investigate right now, but we might want to fix them before they become errors.
*** Building scheme "RxMoya watchOS" in Moya.xcodeproj
ld: warning: linking against dylib not safe for use in application extensions: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/lib/swift/iphoneos/libswiftXCTest.dylib
ld: warning: Auto-Linking supplied '/Applications/Xcode.app/Contents/Developer/Platforms/iPhoneOS.platform/Developer/Library/Frameworks/XCTest.framework/XCTest', '/Applications/Xcode.app/Contents/Developer/Platforms/iPhoneOS.platform/Developer/Library/Frameworks/XCTest.framework/XCTest' does not contain bitcode. You must rebuild it with bitcode enabled (Xcode setting ENABLE_BITCODE), obtain an updated library from the vendor, or disable bitcode for this target.
ld: warning: linking against dylib not safe for use in application extensions: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/lib/swift/iphoneos/libswiftXCTest.dylib
ld: warning: Auto-Linking supplied '/Applications/Xcode.app/Contents/Developer/Platforms/iPhoneOS.platform/Developer/Library/Frameworks/XCTest.framework/XCTest', '/Applications/Xcode.app/Contents/Developer/Platforms/iPhoneOS.platform/Developer/Library/Frameworks/XCTest.framework/XCTest' does not contain bitcode. You must rebuild it with bitcode enabled (Xcode setting ENABLE_BITCODE), obtain an updated library from the vendor, or disable bitcode for this target.
ld: warning: linking against dylib not safe for use in application extensions: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/lib/swift/iphonesimulator/libswiftXCTest.dylib
ld: warning: linking against dylib not safe for use in application extensions: /Applications/Xcode.app/Contents/Developer/Platforms/iPhoneSimulator.platform/Developer/Library/Frameworks/XCTest.framework/XCTest
ld: warning: linking against dylib not safe for use in application extensions: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/lib/swift/iphonesimulator/libswiftXCTest.dylib
ld: warning: linking against dylib not safe for use in application extensions: /Applications/Xcode.app/Contents/Developer/Platforms/iPhoneSimulator.platform/Developer/Library/Frameworks/XCTest.framework/XCTest

Sorry for the late reply, where are you seeing these errors? Is it in the Demo app? The warnings seem to say that XCTest (the unit testing framework) is being linked against for an app target, which I suppose could be caused by our Demo app (I can't remember how Carthage is configured). I've looked through our app right now to see if we're linking against XCTest and I don't see it. Is this bug still happening?


fontforge

font forge will be compiled?

It compiles and starts but does not work quite right last I checked; it got "stalled" after not a lot of input. However that was before some of the more recent rounds of mouse event fixes, so maybe it would do better these days, I didn't really spend much time on it.

It seems recent FontForge has a GDK backend, that should probably be used instead of the X11 one, as it will function more reliably, I'd expect.


How to use 'Try in PWD' button to launch multi node swarm cluster on PWD

I have added Try in PWD  button in my github repo to launch a stack of services. However, I have more than 10 services in the stack which can't fit into one node.
How do I launch 5 node cluster using the PWD url ?
http://play-with-docker.com/?stack=https://raw.githubusercontent.com/achautha/PiggyMetrics/master/docker-compose-3.yml&stack_name=piggymetrics

This is not supported for the moment. The button only launches just one instance.


Poor reconstruction usinf DAE pre-trained weights

I am trying to load the pretrained weights from the tutorial https://github.com/Project-MONAI/tutorials/blob/main/self_supervised_pretraining/vit_unetr_ssl/ssl_finetune.ipynb that were trained using the DAE method https://github.com/Project-MONAI/research-contributions/tree/main/DAE/Pretrain_full_contrast. The weights used are the "ssl_pretrained_weights.pth" available at https://github.com/Project-MONAI/MONAI-extra-test-data/releases.
When I run the following code to see how well I can reconstruct one sample for the HNSCC dataset:
`
   swinUnetr = SwinUNETR(
        img_size=(96, 96, 96),
        in_channels=1,
        out_channels=1,
        feature_size=48,
        drop_rate=0.0,
        attn_drop_rate=0.0,
        dropout_path_rate=0.0,
        use_checkpoint=True,
    )

    pretrained_path = (
        "/home/ol18/Downloads/ssl_pretrained_weights.pth"
    )
    model_dict = torch.load(pretrained_path)["model"]
    pretrained_weights_keys = model_dict.keys()
    store_dict = swinUnetr.state_dict()
    net_keys = store_dict.keys()

    print(len(net_keys), len(pretrained_weights_keys))

    count = 0
    del model_dict["encoder.mask_token"]
    del model_dict["encoder.norm.weight"]
    del model_dict["encoder.norm.bias"]

    for key, value in model_dict.items():

        if key[:8] == "encoder.":
            if key[8:19] == "patch_embed":
                new_key = "swinViT." + key[8:]
            else:
                new_key = "swinViT." + key[8:18] + key[20:]
            store_dict[new_key] = value
            count += 1
        elif key in net_keys:
            store_dict[key] = value
            count += 1
        else:
            print(key)

    print(count)
    swinUnetr.load_state_dict(store_dict)

    for key in batch.keys():
        # get inputs
        inputs = batch[key]["image"]
        B, C, H, W, Z = inputs.shape
        noise = (0.1**0.5) * torch.randn(B, C, H, W, Z).to(
            inputs.device
        )
        img_noisy = inputs + noise
        img_lowres = F.interpolate(
            img_noisy, size=(int(96 / 4), int(96 / 4), int(96 / 4))
        )
        img_resam = F.interpolate(img_lowres, size=(96, 96, 96))

        swinUnetr.to(inputs.device)
        x_rec = swinUnetr(img_resam)

        self._log_image_tensorboard(
            inputs[0],
            img_resam[0],
            x_rec[0],
            f"test_images_0_{key}",
        )

`
I've got the following outputs for the original image, noisy and noisy + low resolution:
original image as input

noisy image as input

noisy + low resolution image as input

From my understanding these reconstructions are fairly poor for the amount of pretraining that is done. Am I missing something in here? Can someone give me a direction?
Many thanks
@tangy5

I also had similar results like yours. Were you able to figure it out?


show image for particular date

i have this array
 var arr = ["2017-02-03","2017-02-04","2017-02-05","2017-02-06","2017-02-07"]
and i use shouldSelect func like :
` func calendar(_ calendar: FSCalendar, shouldSelect date: Date, at monthPosition: FSCalendarMonthPosition)   -> Bool {
    for d in arr{
      let thisDateMustNotBeSelected = formatter.date(from: d)
        if thisDateMustNotBeSelected == date {
            return false
        }

    }

    return monthPosition == .current
}

`
because i dint want select this item (mean items in arr)
now
how can draw image for this date ?!

i want to load Selected Dates Array with color when calendar get Load in view controller like today's date is with color