.extensions.action.enableAllWorkspace ignores code.workspace and Executes workbench.extensions.action.enableAll, suggestions only half work half the time, unwanted suggestions does nothing at all

Issue Type: Bug
Multiple Extension and Workspace related Implementation inconsistencies/bugs

**.extensions.action.enableAllWorkspace insteadeExecutes workbench.extensions.action.enableAll, ** not workspace specific extensions.
*.code-workspace : `"extensions": { "recommended" : [ ] } does not reliably execute.  Reliably Enable, Or even reliably suggest.  I'd say it suggests half about a quarter of the time.
3-  "Unwanted Recommendations" has no discernible effects.  It does not disable extensions upon loading a code-workspace.  There is a CLI command to enable an extension but not disable making scripting harder.
4-Having a *.code-workspace as a child of the root it describes - e.g. "path" : ".." causes unpredictable behavior, which keeps it from staying in the .vscode folder with its friends.

I use several languages.  I have elaborate configs for each of them, that are constantly changing- as I am sure many of us do.  I have a Java config, a python config, a PlatformIO config, a markdown config, and then a slew of others I dabble in.
I don’t want all the extensions running all at once.  As I have discovered more and more useful extensions, my vs code would slow to a crawl and become unreliable.
So now I make heavy use of workspaces.  I change the settings for each by dropping a file in each directory as needed.  A new java project gets my java config dropped in the root, that has several hundred lines for customization and settings.
I use the *.code-workspace file to set my configuration.  I have fewer and fewer global configs as I find exceptions to where I'd want each thing.
Ideally though, I would have some way of replicating a workspace that I have gotten just perfect - after hours of neurotic tweaking- in a new workspace.  Managing which extensions stay enabled and which are to be disabled is still very poor at this late stage of dev.
You can't enable extensions from the command line you can only disable.  So we can’t manage extensions our self in any fashion.  We can’t do it in the work-space settings.  And cant do it via a script.  I have to manually SEARCH and FIND each extension and enable it and disable others each time I switch tasks.
I have trouble managing the extensions which are enabled for each given task/workspace.  When I work on a python file/ws I want all my python settings/extensions.  Likewise Java.  These settings may (obviously ) conflict and extensions interfere and react strangely.
So- the obvious solution is *.code-workspace- you’d think right.  Except it doesn’t work like its supposed to.
I carefully set up the recommendations section in that file.  But VS Code could care less.  It only sometimes listens to my suggestions (if I put it in the file, its more like an "!important" or required than a suggestion...  It might enable half.  And leave extensions enabled from the last file I worked on.  Even if I set up unwanted extensions.  That does essentially nothing.  I have Suggestions enabled in settings. I WANT them to take effect, and install.  And if I list unwanted extensions, they shouldn’t just not recommend, they should NOT RUN.
HERE is the actual bug / implementation issue
So them I try from the Command Processor >Disable all extensions  (workbench.action.reloadWindowWithExtensionsDisabled)
Then, execute Reload Workspace Extensions  (.extensions.action.enableAllWorkspace)
Expectation
All the extensions listed in ws.code-workspace would be enabled. So I have started with a clean slate, then enabled the Workspace ones.  I should have ONLY the workspace extensions running now.
Actuality
However, this enables ALL extensions.  Not the ones given in the workspace config.
I expect that to occur with the command (.extensions.action.enableALL), not …enableWorkspace
I expect that command to execute what is in *.code-workspace.  while I might expect the command  workbench.extensions.action.enableAll to enable everything.
SO there are two things here in error.
Here is a third bug/ implementation issue.  Having a *.code-workspace as a child of the root it describes causes odd – non-specific behaviors.  It is convenient to put all my settings in root/.vscode (with tasks, launch, snippets) and set the path to parent – e.g.
”path” : “..”
Or
”path” : “.\..”  or ”path” : “../.” or whatever else you want to do to represent the relative parent (even ./.././.././rootname/
Discussion
You guys seem undecided about implementing workspaces in general.  Its not very easy.  Its actually quite complicated.  Its why I have read the documentation all the way through on the VSCode website all the way through several times.
BY Default Code installs all extensions in the (what I will call) Global User Directory %user%/.vscode on Windows, ~\.vscode on Linux, and we all know there aren’t any real programmers on Mac (too busy editing videos and photos all day – or pretending too, while mailing Tim Cook their retirement assets)  noticed today just a minute ago, there’s a little publicized command on the CLI code command for Extensions Management   --extensions-dir <dir> (from code -h) that I think is new (?). maybe not.

I suppose perhaps a possible intent is to make a local extensions directory for each project or class of project.  I have an idea for a work around, I guess I will have separate user dirs. and extension dirs. for each language. – this could be why there is a .vscode  settings dir in the local project, and a .vscode config dir on the Home dir of Win and Linux.  But this isn't very clear.  I have read all the documentation (ALL of it,, several times) and there’s nary a mention.   So, is the intent that we run code like an IDE, with plugins, or are we running it like a NPM / maven project, where we have local extension sources for each project.  Its a pain to go and redownload them all again.
Further- unwanted suggestions is worthless as implemented.  Taking the name literally, it does what it says it does maybe- I never checked- to suppress suggestions.  But it does not disable an enabled extension shifting between projects.  These should be regarded as contraindicated extensions- incompatible with the workspace.  It shouldn’t just suppress a recommendation- I ignore that toast box most the time anyway.  Its only useful if it does some action.  I don’t know – maybe you guys are passing around multi root of Node/NPM/Electron/.js/.css builds  VS Code (Insiders) all day, and this setting made sense.
Finally, I like everyone else, my global user config eventually becomes littered with the detritus of the settings from scores of extensions that I installed but didn't live up to their authors promise.
Things like this become rather common:

There must be a better way to manage these extensions.  Its critical I believe--  VS Code is so popular because of its extensibility.  But it becomes unmanageable as more and more good extensions come out, and VS Code continues its move towards a one IDE for all solution.
We need portable profiles that we can engage and disengage and most importantly encapsulate.

And a proposed, unifying solution
Making a few- small, not breaking changes would solve most of my and most other peoples issues with configuration.  Briefly, it involves unifying the settings files (Keybindings, Tasks, Launch, Snippets, Settings, and WS.code-workspace) and allowing referencing in a tree like fashion.  This is much more clear, much more extensible, backwards compatible and much more capable.
Briefly I propose, everything remains as is.  With one exception, in each of the parsers for User settings, Folder Settings and Workspace Settings an additional recursive subroutine \method\function is added.  Making an option something like this:
//#myworkspace.code-workspace
{
    "include": [
        {
            "profile.name": "markdown-light",
            "path.to.file": "markdownsettings.code-settings"
        },
        {
            "profile.name": "java-general",
            "path.to.file": " javalanguageprofile.code-settings"
        }
    ],
    "settings": {
        "debug.allowBreakpointsEverywhere": false,
        "debug.enableAllHovers": true,
        "debug.inlineValues": true,
        "debug.openDebug": "openOnSessionStart"
        //...
        //<perhaps many more>
        //...
    },
    "extensions": {
        "recommendations": [
            //[Editor Helpers]
            "albert.TabOut",
            "weakish.complete-statement",
            "CoenraadS.bracket-pair-colorizer-2",
            "aaron-bond.better-comments",
            "oderwat.indent-rainbow",
            "predrag-nikolic.php-class-helper",
            "maptz.camelcasenavigation",
            "kevinkyang.auto-comment-blocks",
            //[Other]
            "eriklynd.json-tools",
            "Tyriar.shell-launcher",
            "helixquar.asciidecorator",
            "rexebin.classlens",
            "marlon407.code-groovy",
            "ArgyleWolfLabs.vscode-groovy",
            //[git]
            "ankalus.git-tools",
            "codezombiech.gitignore",
            "donjayamanne.git-extension-pack",
            "donjayamanne.githistory",
            //[Markdown]
            "jebbs.markdown-extension-pack"
        ],
        "unwantedRecommendations": [
            //I used to carefully fill this out but I realized theres no point, it doesnt do anything.
        ]
    }
}

An Included File
Then we include the included 'profile' file, which is code-worrkspace settings and all the other general settings types (or any of the settings types in one - tasks, launcher, snippets, keybindings, settings, color-customizations, extension specific settings.
//Markdown-light
//this is my snap in configuration for markdown editors when used within other projects - usually for documentation. etc.
{
    "extensions": {
        "enable": [
            //<list of enabled exts>
            "markdown.ext.vs.code"
        ],
        "prohibit": [
            "linter.markdown2"
        ],
        //then presumably, because SOMEONE uses those recommendations, allow recommendations here-
        "recommended": {
            []
        },
        "unwanted recommendations": {
            []
        },
    },
    "settings": {

Aside
Here are settings   they are treated the same as if they were in the file which included them.
This way you can have a config for a single extension.  Or a whole workflow.
It is suggested thus that the settings here be related to the task, for instance this calls in markdown settings into my java workspace, so it would have the markdown settings I use.- in addition to the settings for the above mentioned extensions.
And- Importantly if you're paying attention- this makes a recursive tree.  Each of the @includes is a node with a name.  We can snap-in or purge that tree, and if it has subtrees, they can be purged as well.  So for instance we may have-

Global
|
|---Java - a java Config which itself has 2 includes
|       |
|       |-EditorCustomizations
|       |-SpringBoot - settings for a spring boot build and the related extensions it depends on
|       |       |
|       |       |-- Project A settings
|       |       |-- Project B settings
|       |       |         |
|       |       |         |-RevA-Overwrites some of the earlier configuration with new settings
|       |       |
\       \
\       \       \

Feature within a Feature Request
WHILE I AM AT IT.    But you didnt go all the way to the logical conclusion- which is why there are so many config related issues.  Just make it so that the following members are all considered top level members-

.
...1-Settings
......Editor Config
.........Landuage Specific Settomgs
...Workspace Config
......-(themes)
.........-Workspace Configuration/ Overrides
.........-TextMate, tokenization Configs/overides
... 2-Extensions
......- Include
......- Exclude
......-Recommend
......-Dont Recommend(?)
... 3-Keybindings
... 4-Snippets
... 5-Launch Config
...6- Tasks
... ?- Others I forgot, or future extensiability.

Then Includes __is like a meta top level- since its above all the prior, equivalent,  and beneath- as shown in examples above or below. This is no different then what happens now, as settings can be defined at the Global, Workspace and Folder Level.  Perhaps we restrict Paths / Launch/ and Tasks to at least 1 level deep- technically only Defaults exist at the highest level.  Anything can be rewritten by the layer(s) below.
In addition to Settings, Extensions, and Paths, which may be defined where ever they fit in the users arrangement of setting configuration.  Just parse the tree.  If you hit a paths block, add the paths.  Users can snap in and out paths with a command- since each @include is essentially a switch._
Continue the same doc as above
//continuing
 },

    "tasks": {
        // <exactly the same as the current tasks file just nested- parse until you hit the closing bracket, and pass the entire chunk to existing parser.  Its JSON so its fast and easy to do.>
    },
    "launch": {
        "version": "0.2.0",
        "configurations": [
            {
                "type": "java",
                "name": "CodeLens (Launch) - App",
                "request": "launch",
                "mainClass": "main.App",
                "projectName": "p3hencoding",
                // "debugServer": 4711
            },
        ],
    },
    "snippets": {
        "ConvertLink": {
            "scope": "markdown",
            "prefix": ".linktitle",
            "body": [
                "[$1](${TM_SELECTED_TEXT})",
                "",
                "$0"
            ],
            "description": "Take a site and make it into a link"
        },
        "SelectToLink": {
            "scope": "markdown",
            "prefix": ".selectedTextToHyperlink",
            "body": [
                "[${1:${TM_SELECTED_TEXT}}](${CLIPBOARD})",
                "",
                "$0"
            ],
            "description": "Take a site and make it into a link"
        },
        {
            //etc
        }
    },
    "tasks": {
        "version": "2.0.0",
        "presentation": {
            "echo": true,
            "reveal": "always",
            "clear": false
        },
        "type": "shell",
        "echoCommand": true,
        "tasks": [
            {
                "label": "initialize",
                "dependsOrder": "sequence",
                "group": "build",
                "command": "mvn",
                "args": [
                    "initialize"
                ],
            },
            {
                "label": "VALIDATE",
                "dependsOrder": "sequence",
                "group": "build",
                "command": "mvn",
                "args": [
                    "validate"
                ],
            },
        ]
    },

Say, I have lots of keybinding customizations.  Maybe just to make the file prettier I want to break it out.  Maybe I want to break it out bc its language specific and I call it in the language specific file Ive got here (Markdown)
    "keybindings": {
        {
            "include": [
                {
                    "profile.name": "markdown-keybindings",
                    "path.to.file": "mkdnkeys.code-settings"
                },
            ],
        },

but I still define a few here anyway as well- for expediency.  The two are equivalent.
        {
            "key": "ctrl+` ctrl+`",
            "command": "shellLauncher.launch"
        },
        {
            "key": "ctrl+k ctrl+k",
            "command": "workbench.action.openDefaultKeybindingsFile"
        },
    }\

Finally, perhaps we want this recursive- the "include" aspect- where include appears as a meta top level domain.
    "include": {
        [
            {
                "profile.name": "spring-boot",
                "path.to.file": "spring-boot-settings.code-settings"
            },
            {
                "profile.name": "maven",
                "path.to.file": "maven-setup.code-settings"
            },
        ]
    },
    "settings": {
        "git.enabled": true,
        "window.zoomLevel": -1,
        "python.terminal.executeInFileDir": true,
        "better-comments.highlightPlainText": true,
        "editor.acceptSuggestionOnEnter": "smart",
        "[markdown]": {
            "editor.showFoldingControls": "always",
            "editor.foldingStrategy": "auto",
            "editor.fontLigatures": true,
            "editor.accessibilitySupport": "off",
            "editor.cursorBlinking": "expand",
            "editor.cursorSmoothCaretAnimation": true,
            "editor.copyWithSyntaxHighlighting": false,
            "editor.defaultFormatter": "vscode.markdown-language-features",
            // "editor.defaultFormatter": "jebbs.markdown-extended",
            "editor.fastScrollSensitivity": 7,
            "editor.formatOnPaste": true,
        },
        "markdown.preview.openMarkdownLinks": "inEditor",
        "include": [
            {
                "profile.name": "workbench customlook2",
                "path.to.file": "work"
            },
            {
                "token.color.name": "mylintersettings.json",
                "path.to.token.color.defs": "editor.tokenColorCustomizations",
                "other.settings" {
                    "workbench.iconTheme": "material-icon-theme",
                    "markdown-preview-enhanced.codeBlockTheme": "atom-dark.css",
                    "markdown-preview-enhanced.mermaidTheme": "dark",
                    "markdown-preview-enhanced.previewTheme": "atom-material.css",

it can appear as a field within any of the other levels (just like key bindings).  Since I have a name, I can turn it on and off.  Turning off a parent - in this case the parent we just filled out above Markdown-Light would turn this off too, as its a child node.
We can also declare a few settings within  the include block for expediency.
                    "markdown-preview-enhanced.revealjsTheme": "night.css",
                    "materialTheme.accent": "Blue",
                    "workbench.colorTheme": "Hopscotch (TextMate Theme)",
                    "editor.fontFamily": "Fira-Code, 'Fira Code Retina', monospace",
                }
            ]
        ]
    }
}

I hope this all makes sense.  I think it will drammatically improve the capabilities of VS Code with minimum effort.  I am open to other thoughts, and would love to see this get the attention of some of the Dev team.

 Please provide simple steps to repro and what is expected and what is the outcome


Winter boots require departmental sprites

Description

As the person who originally made the winter boot sprites, it was originally going to have a version for every department so it can match with the coat variant, but i never got around to doing it. So, every department that has a winter coat should also have winter boots able to be dispensed.

Hey, is this post still relevant? I just made icons for some boots.

Still relevant!

I have problems with launching the game and testing, can you help?
I get the following error

Im not great at debugging, you should probably go to the discord for this

How do I request a review of my PR?

on the discord


error:import tensorflow_hub

tensorflow=2.4.1
tensorflow_hub=0.11.0
error:
AttributeError: module 'tensorflow._api.v2.compat.v2.internal' has no attribute 'monitoring'

Please try upgrading TensorFlow to 2.8 as proposed by https://github.com/tensorflow/tensorflow/issues/54286#issuecomment-1031205739.


check the availableInRun() of the next PageRun in current allocation_ whether satisfied with the preferredSize in AllocationPool::newRun

@Dream-hu It is hard to read a screenshot. Would you provide a text-based explanation of the problem and proposed solution? Would you like to send a PR?
CC: @oerling


Not working after updating to 2024.3 release

Region and Brand of car
US 2024 Kia Niro EV
Describe the bug
None of the entities have been updated since upgrading to the 2024.3 release of Home Assistant.
Debug logs if an error occurred
Logger: custom_components.ha_kia_hyundai.vehicle
Source: helpers/update_coordinator.py:318
integration: Kia/Hyundai ([documentation](https://github.com/dahlb/ha_kia_hyundai), [issues](https://github.com/dahlb/ha_kia_hyundai/issues))
First occurred: March 6, 2024 at 5:57:30 PM (3918 occurrences)
Last logged: 11:42:09 AM
Unexpected error fetching Vehicle 095852 data: Attribute hass is None for <entity unknown.unknown=unknown>

Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 318, in _async_refresh
    self.data = await self._async_update_data()
                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 274, in _async_update_data
    return await self.update_method()
           ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/config/custom_components/ha_kia_hyundai/vehicle.py", line 126, in async_update_data
    self.calls_today_for_update.mark_used()
  File "/config/custom_components/ha_kia_hyundai/sensor.py", line 153, in mark_used
    self.async_write_ha_state()
  File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 969, in async_write_ha_state
    raise RuntimeError(f"Attribute hass is None for {self}")
RuntimeError: Attribute hass is None for <entity unknown.unknown=unknown>

To Reproduce
Steps to reproduce the behavior:

Upgrade to 2024.3

Expected behavior
Integration continues to work

Thanks for writing this up, I can reproduce and should be able to fix it tomorrow night

fixed in version 1.8.11 thanks again for writing this up, it was a small typo that has existed for a long time in this integration, 2024.3 removed some code that had surprisingly managed to mask this error until now.


Populate Gateway Status address for ClusterIP type Envoy Service.

support #1212

Codecov Report

Merging #1219 (fefaa3c) into main (3b883fa) will decrease coverage by 0.02%.
The diff coverage is 0.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main    #1219      +/-   ##
==========================================
- Coverage   63.26%   63.25%   -0.02%
==========================================
  Files          79       79
  Lines        9831     9839       +8
==========================================
+ Hits         6220     6224       +4
- Misses       3192     3197       +5
+ Partials      419      418       -1

Impacted Files
Coverage Δ

internal/status/gateway.go
0.00% <0.00%> (ø)

... and 1 file with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Some questions about the chart

I'd like to know if there is a way to zoom after i click on a part of the chart. I mean... as you see here i have a chart-line. I'd like to click on the red circle (day n°4) and it has to zoom in and show me 3 days before day n°4 and 3 days after. It has to zoom on click. The same thing must work on the other days.
Is there a way to do that? thanks :)

The other thing i've noticed is that if you reload the page with the canvas display none, the graph won't show up even if you toggle dislplay none from the style. It shows only the legend. So i must reload the page to see the graph and there's no way to hide it on reload page. I must show it only when i click on a certain button.
Can you please give me some information about these things? Thanks again. :D

Hi there. I am unsure on the first part regarding zooming, maybe updating the dataset on click could do that?
For the chart breaking when hidden there is an open issue for this that is yet to be solved: #9
For creating click events there is another issue with some examples: #10
Let me know if any of this is useful.


Cancel Order Models.py Update

As a developer
I need to update the orders data tables
So that I can register an order cancelation
Details and Assumptions

When an order is canceled, the action needs to be registered in the data table

Acceptance Criteria
Given an order_ID
When I cancel an order
Then the models.py data tables will be updated accordingly

Hey team! Please add your planning poker estimate with Zenhub @bartconsedine @EmiliePourchet @LotteLa @marianaruizallende @nilesh1401 @Victoria-Obasa


[storage][azblob]Content validation test for DownloadFile/DownloadBuffer functions

[x] The purpose of this PR is explained in this or a referenced issue.
[x] The PR does not update generated files.

These files are managed by the codegen framework at Azure/autorest.go.

[x] Tests are included and/or updated for code changes.
[x] Updates to CHANGELOG.md are included.
[x] MIT license headers are included in each file.

This PR will add a test to verify DownloadFile and DownloadBuffer methods. A blob with randomly generated data is uploaded as source, and then downloaded to a file and a buffer. The content length and md5 of the data is verified. This will be performed on blobs of sizes (0, 6MiB, 10MiB, and 14MiB).

/azp run go - azblob


Raspotify (<IP_ADDRESS>) shows "Unavailable to control" in Spotify device list

Due Diligence

[X] I have done my due diligence

What can we help you with?
I have a Raspberry Pi Zero W (v1). I've installed Raspberry Pi OS Lite and used the README instructions to install Raspotify <IP_ADDRESS>. Everything looks great except it's grayed-out and shows "Unavailable to control" in my Spotify devices list.

If I run librespot it will also show up in the devices list and also as "Unavailable to control".
After some Googling I found #385 but I believe this is different (and that fix was before <IP_ADDRESS>). I've also scoured the Wiki for any troubleshooting tips and haven't found any. What else should I try?

The README clearly states:

Support for ARMv6 (Pi v1 and Pi Zero v1.x) has been dropped.
<IP_ADDRESS> was the last version to be built with ARMv6 support.

You can install and run that version on an ARMv6 device, but you will never get updates and doing so is completely unsupported.

There's not really much use debugging something that will never get fixed even if you figure out what the issue is. There will never be another ARMv6 compatible Raspotify build.
I just noted that release so PiZv1 owner know which is the most up to date version still built with PiZv1 support.
If it works for you great, if not, sorry. That version is completely unsupported.

Just wanted to update that my issue was identical to #253 with the problem being that somehow Spotify Connect doesn't work for podcasts and I had a podcast that I was recently listening to and was using for testing


Label contributor PRs with a Contributor label

@clayburn please provide some context to the description on what this PR wants to achieve. Also, please provide some context on the individual files what exactly they do. This will make the PR much easier. Thanks.


🚀 Feature Request: Allow local Service Bindings to proxy to a wrangler dev --remote session

Describe the solution
When I try to acess a rpc Service Binding in my next-on-pages application:
Error: Cannot access 'ImageSearch' as we couldn't find a 'wrangler dev' session for service 'vectorize-worker' to proxy to.
As the error says, I should run wrangler dev to test locally and it works when running wrangler pages dev (it do not work with next dev but that is another story.
However, my "vectorize-worker" do need to access both Vectorize, and production R2 buckets.
For Vectorize, this means it needs wrangler dev --remote to work, as it says:
Vectorize bindings are not currently supported in local mode. Please use --remote if you are working with them.
And so does R2 non-preview buckets.
Then we can go back to my initial error:
Error: Cannot access 'ImageSearch' as we couldn't find a 'wrangler dev' session for service 'vectorize-worker' to proxy to.
The only way out of my situtation would be if it could proxy to a "wrangler dev --remote" session too.
I hope this makes sense!

Is this fixed yet? or does vectorize only work in prod still

Closing as duplicate of https://github.com/cloudflare/workers-sdk/issues/1182 (tracked in the Mixed Mode section)


Multiple Python.rc files for each language ?

If I want visual studio code to read specific pylint.rc files for specific languages; must I put that in the python environment file ?

Not sure what you mean by specific pylint.rc files for specific languages
Please check the documentation on pylint to learn where the config files (pylintrc) can be placed, see here https://docs.pylint.org/en/1.6.0/run.html#command-line-options


Atlantis Plan through curl

Hello,
How would we do an atlantis plan through curl?
Also, we need to extract the atlantis plan command from the gitlab http request? Could you kindly tell us what field in the body we need to look for - that would be great.

hi @ginski47 , sorry but that is outside of the scope of what this module provides


Updated command for installing theme

The previous command did not install the theme correctly. It gave me this error:
$ ya pack -a tkapias/nightfly.yazi                                                                                [22:30:17]

  Upgrading package `nightfly.yazi.yazi`

Cloning into '/my/yazi/packages/directory'...
remote: Repository not found.
fatal: repository 'https://github.com/tkapias/nightfly.yazi.yazi.git/' not found
Error: `git` command failed: exit status: 128

Here are the logs with this change:
$ ya pack -a tkapias/nightfly                                                                                [22:30:55]

  Upgrading package `nightfly.yazi`

Cloning into '/my/yazi/packages/directory'...
remote: Enumerating objects: 10, done.
remote: Counting objects: 100% (10/10), done.
remote: Compressing objects: 100% (9/9), done.
remote: Total 10 (delta 1), reused 7 (delta 0), pack-reused 0 (from 0)
Receiving objects: 100% (10/10), 675.35 KiB | 4.93 MiB/s, done.
Resolving deltas: 100% (1/1), done.

  Deploying package `nightfly.yazi`

Done!

I think this was demonstrated to fix this issue in this video, actually 🙂
https://youtu.be/iKb3cHDD9hw?t=527

Also based on new updates. Please add on "dark" in theme.toml, to toggle the theme for dark mode as well.
This way light and dark mode can also used!
@Zohair-coder pls add that to the PR too! Thanks


Permission Issue (namespaces is forbidden)

Thanks for maintaining this project by the way! It was exactly what I was looking for.
My Setup:
k8s version: 1.12.3
OS: Ubuntu 16.04 LTS
My problem:
my job fails with the following error.
Error from server (Forbidden): namespaces is forbidden: User "system:serviceaccount:kube-system:kube-backup" cannot list resource "namespaces" in API group "" at the cluster scope

Possible Solution:
I'm working on testing a ClusterRole and ClusterRoleBinding being added to the yaml will fix the issue. would you like me to draft up a PR?

I missed that there was a RBAC.yaml file that had those permissions.


SSKDF controls

There is a TODO in the SSKDF code questioning the need for all the various settings. Since this is part of the public API, do we need to do something for beta 1?
Reference
https://github.com/openssl/openssl/blob/a48309cb5c58be8fa310608124925d4729664cf8/providers/implementations/kdfs/sskdf.c#L374-L378

My advice would be to remove the TODO.
This code follows the standard. See https://nvlpubs.nist.gov/nistpubs/SpecialPublications/NIST.SP.800-56Cr2.pdf
i.e it supports:
H(x) = hash(x)
H(x) = HMAC-hash(salt, x)
H(x) = KMAC#(salt, x, H_outputBits, “KDF”)
AND the standard spefically talks about the salt length..

This  Recommendation  places  no  restriction  on  the  length  of  a  chosen  salt  other  than  the  requirement that its byte length be greater than zero but no greater than the length of a single input block to  the  hash  function, hash, used  to  implement  HMAC-hash.  That  freedom  of  choice  is  somewhat  illusory,  however,  since  the  HMAC  algorithm  will  convert  an  input  salt  value  (as  needed)  into  a  string  of  the  indicated  hash-dependent  length.A  shorter salt  (used  by  H  as  an HMAC key) will be padded by appending an all-zero bit string to obtain a string of the indicatedlength (the length of a single input block for the hash function, hash); a longer salt will be hashed to produce a shorter string (of bit length H_outputBits), which will then be padded (by appending an  all-zero  bit  string)  to  obtain  a  string  of  the  indicated  length (see [FIPS  198]  for additional information).


Fatal erro on winmactu

I start it and got bellow:

what can i do?
my system is winmactu20

This snippet worked for me in linwin18.04:

Hope this could help.


Hardcore stumping - add stump texture toggle

It'd be nice to have a config option to remove the special stump texture.
BWM seems very texture pack friendly, since it avoids using a lot of custom textures. The stumps are very out of place when using texture packs without custom mod support.

Like all blocks stumps have a blockstate model which you can override in a resourcepack with whatever texture you want. There is no reason to add a config for something which can be done with resourcepacks already.


Add font kerning support for arc_of

Feature request.
Currently arc_of to print text considers the global font size and works well for monospaced fonts, but non-monospaced fonts can look strange around skinny letters.
I don't know much about fonts but kerning is used to position characters for a more aesthetic look in desktop packages. I don't know how that could be achieved with the SCAD language so perhaps a user-defined modification table can be applied for making adjustments to specific letters by calling arc_of multiple times to make up the segments of the text.
I think it may be possible to add a check in a module that calls the "select" function to check for the character codes and set the $idx accordingly. No example code I'm afraid since I'm new to SCAD but it is something I'll be looking into.

OpenSCAD does not yet provide userspace access to font info like kerning, though there is a fontmetrics feature in beta testing. Without the fontmetrics feature, the only way to handle kerning would be a list of character widths for each font. This, while possible, is annoying to implement, and would become obsolete as soon as the fontmetrics feature is merged into mainline.
Having said all that, there is already a way to have text follow a path, which will take fontmetrics into account once it becomes available:
include <BOSL2/std.scad>
path = path3d(arc(100, r=25, angle=[245, 370]));
color("red")stroke(path, width=.3);
path_text(path, "Example text", font="Courier", size=5, lettersize = 5/1.2);

Just to be clear, currently with the stable OpenSCAD that doesn't have textmetrics, there is NO INFORMATION about the font size available to path_text.  So no, it doesn't "consider" the font size.  There's no way to do that.  You have to tell it what letter spacing you want with the lettersize arg and that's what you get.  It's totally decoupled from the actual size of the font.  You can set lettersize to an array to individually position all the letters if you want to mess with that, so arguably kerning is available.
Now arguably with textmetrics turned on you don't have kerning, so maybe lettersize should still be used in that case to perform relative adjustment of the size if you want to tweak the layout from the font metric size?

Better to add a new kern parameter to make adjustments when textmetrics are used as opposed to repurposing lettersize, which seems sort of confusing?

Actually it looks like the problem the original poster had was using arc_of() instead of path_text().

Added a kern parameter to path_text.

Many thanks for looking into this @adrianVmariano . I woke up to quite a few messages this morning. I used the arc_of() because that is the example given in the text section (https://github.com/revarbat/BOSL2/wiki/shapes2d.scad#module-text); example 6. I've not come across path_text() yet because the library is absolutely massive and I only discovered it recently. It is absolutely amazing; a real game changer, but quite difficult to find specific features. I'll have a look at the path_text() presently. Again, many thanks.

I was wondering if there was an example where arc_of was used for
text, but I didn't see it.  I have deleted that obsolete example.


Fix auto-merge PR to run after labels are added

Since the introduction of the Set  Validate PR Labels workflow, the auto-merge workflow step often fails to merge in cases when it should because it runs before the labels are added, and the PR is in a "blocked" state. It's not a huge problem, but maintainers then have to perform a manual step to merge the PR. This is the whole reason that the auto-merge functionality exists, so we should make sure it works.
We should determine a clean way to ensure that the auto merge workflow re-runs when labels are added, or waits to run until the PR review is submitted as well as labels being added.

I investigated this a bit, but the issue lies in the fact we only want to run the auto-merge step in the case that the auto-approve workflow succeeds (current behaviour). If we modify the auto-merge workflow triggers to include labeling, it will always run even when the auto-approve workflow hasn't run which is a waste of resources. Github does not have a nice way of ANDing together triggers.

@fg-j is currently testing a solution in occam, jam, and packit. Upon seeing expected behaviour consistently in those places, we can consider adding this to our other repositories and closing this out.

Here's an example of the auto-merge in action on packit: https://github.com/paketo-buildpacks/packit/pull/326. The PR opened, was approved, auto-squash enabled. I intervened to add a semver label, then the branch automatically merged once all checks went green.

I'd like to see this rolled out to our other repos. @paketo-buildpacks/tooling-maintainers can we get a 👍 or 👎 on that?


Updated tox.ini to run unit tests on py39

Updated tox.ini to run unit tests on py39

Thanks for reviewing, Italo. I'll go ahead and merge this since the tests are passing locally, and then fix the issue on scrutinizer in an upcoming PR


Flannel systemd unit needs an [Install] section

I have automation set up to configure flanneld on machine installation via Ignition. However, Ignition is unable to enable flanneld, because the systemd unit file provided in the coreos image doesn't have an [Install] section. Per the semantics of systemd units, this section cannot be provided in a drop-in unit, which leaves Ignition no way of setting flannel to start automatically on system boot.
Please consider adding an [Install] section to /usr/lib64/systemd/system/flanneld.service that targets multi-user.target (like e.g. locksmithd does).

Apologies, this is not the right project to bug, since the systemd unit is not shipped in this repository. I'll refile in the right place.


performance: fix grafana dashboard layouts

use same scale
interruptions on some charts

fixed dashboard can be found here


Old version ionic in documentation

a link button on the download section goes to old version of ionic 1.2.1.

Made a PR to fix this. https://github.com/driftyco/ionic-site/pull/355


Support for Symfony console

Hi, I have created a super simple custom command to support Sympfony console by using ddev console ..., this makes super easy to execute Symfony console commands without ssh into the container.
I believe that other people might find it handy so I wanted to see if it's worth submitting it here as well.
Command file path:
commands/web/console
Command script:
#!/bin/bash

## Description: Run Symphony console
## Usage: console <command>
## Example: "ddev console about" or "ddev console debug:config"
bin/console $@

If approved, I've already forked this repo and can submit a PR.

Of course, and thank you. You don't have to open an issue first, just do the PR.
Please make sure to follow the existing examples, put your name and a link at the bottom, and add the new item to the top-level README.

Of course, and thank you. You don't have to open an issue first, just do the PR.
Please make sure to follow the existing examples, put your name and a link at the bottom, and add the new item to the top-level README.

Thanks, I wasn't sure if I need to open an issue here because this branch does not have a Contributing guide, so I followed the ddev which starts with creating an issue... Anyway, the PR was submitted.

Thanks, I wasn't sure if I need to open an issue here because this branch does not have a Contributing guide, so I followed the ddev which starts with creating an issue... Anyway, the PR was submitted.


Support for TV Shows aswell

It would be good to have support for TV shows aswell.
Example, if i have Looney Tunes, i might also like baby looney tunes. Or if i have Mickey Mouse club house, i might also like mikey mouse or ducktales.

That's not an option until I find a service that adds collections to TV Shows for me to search. When I've looked in the past there wasn't any. If you know of one, please post it as I'd like to have this feature for myself as well.

That's not an option until I find a service that adds collections to TV Shows for me to search. When I've looked in the past there wasn't any. If you know of one, please post it as I'd like to have this feature for myself as well.

That's not an option until I find a service that adds collections to TV Shows for me to search. When I've looked in the past there wasn't any. If you know of one, please post it as I'd like to have this feature for myself as well.

What service/site do you use currently for Gaps for movies?

That's not an option until I find a service that adds collections to TV Shows for me to search. When I've looked in the past there wasn't any. If you know of one, please post it as I'd like to have this feature for myself as well.

What service/site do you use currently for Gaps for movies?

TMDB

TMDB

TMDB

Hi,
So i am assuming Gaps gets the movie name and then passes it to TMDB to get recommendations and then filters those that are already on the drive and recommends the one those are not.

TMDB

Hi,
So i am assuming Gaps gets the movie name and then passes it to TMDB to get recommendations and then filters those that are already on the drive and recommends the one those are not.

Basically, yup.

Basically, yup.

Can you not use thetvdb to search to TV.

I haven't seen an API that has TV collections. If there is one, I could add.


Frequent Attempting to call unbound fn: errors : Windows

Hi,
I keep getting these errors very frequently on Windows in Run in REPL option (no problem during Build) . They sometimes go away on rebuilding, but other times stay. I can run the absolutely same program on lein, so it is bugfree as such.

Please let me know if you notice this issue in the new version of Nightcode; it may no longer be an issue now that the rewrite has occurred.


Add the flutter detach command

#111
done :)

Only thing I'm unsure about is the command_requests for the dap runner. not really sure what detach would be for it so I just put detach = "detach". Would there be a place for me to verify this?
The line im talking about

@joshpetit thanks for working on this, generally looks fine to me. Regarding the command_requests @clement-buchart actually added the functionality and tbh I don't know how he figured out which names matched which for the dap runner. Will see if he can enlighten us, otherwise I'll merge this as a best effort we can try and improve on if it doesn't work

Hello,
Looking at https://github.com/Dart-Code/Dart-Code/blob/f58ca38e451d81f4d0e20bbb6b8ebb06a66264a4/src/debug/run_daemon_base.ts#L104 there is a detach method, but it's only ever called if the dap configuration was started in attach mode, and we always start a "launch" configuration.
So for it to work we would need either :

Add some codepath in Dart-Code to be able to call this method explicitly via a request and contribute this upstream.
Refactor the dap-runner to use an attach configuration. This would imply actually starting the flutter process ourselves and then attaching to it.

First solution is probably simpler to implement and "better", but dependent on another repo.
I personally will not have a lot of time to contribute it is the foreseeable future.
May I suggest simply not adding anything to the command_requests of the dap runner ?
This way it should fall back to this codepath which will indicate it's not implemented yet for this runner :
elseif not quiet then
    utils.notify("Command " .. cmd .. " is not yet implemented for DAP runner")
end

(thus current implementation n this PR will do nothing silently)
Cheers.

I agree with omitting the functionality for the dap runner as both alternative options are quite involved. Thanks for the input @clement-buchart 👍🏾

@akinsho Just removed it 👍

Thanks for the PR @joshpetit just merged it 👍🏿


Added missing semi-colon

@sguitardude : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Verified the update.
@sguitardude Thanks for submitting this fix.


Notebooks: Fix starting and separators in a row

I don't like this. But the canRun() method only takes into account CellContext, not the other items in the items array. If anyone has any other ideas please let me know 😄
Fixes #7785.
Code cells:

Text cells:

This is what vscode menu filling code is for shrug

@anthonydresser can you expand on this with an example? Sorry I'm just not super familiar there

The entire menuing system that vscode has built, utilizing contributions is meant to handle these cases.

@anthonydresser can you give Chris an example code fix for this? E.g. for internal actions (not contributed actions), what code change would be needed to do things right? Convert things into menu groupings for example and then use a layout function based on that? Helping Chris (and myself) learn best practice would ensure future scenarios don't have the same issue.


Add an article about MemberwiseInit

Step by step tutorial and a repository with complete source codes and tests is attached.

thanks for contributing!


Add pss to roles that have turnout maintenance mode

Add the Primary Source Sets app to those Rails apps that are connected to PostgreSQL and that need to be taken offline during PostgreSQL maintenance.

:rocket:


Executable

Close https://github.com/gazay/ids_please/issues/11
Add executable to project
It works with grab, parse and recognize methods
See ids_please help for instructions
TODO: need to document it in README (Also I can try to document it)

A lot of code in binary is not a good sign. Maybe better to create a new class like IdsPlease::CLI or Console or Cmd (or whatever) and put all logic there? And from binary only require it and run something like IdsPlease::Cli.new(ARGV).run. It allows to test that logic in isolation easy way.
And I don't like solution with using many puts instead of multiline string syntax.
Also, there is possibility that gem can be used from source, without loading its classes in LOAD_PATH. In that case just require 'ids_please' will throw an error. You can look how I've solved it here: https://github.com/gazay/talks/blob/master/bin/talking
And the best thing for CLI tool creation with options parsing, automatic help message generation, etc. is thor (https://github.com/erikhuda/thor)

Ok. Will do it
I thought about Thor, but also I thought, that you don't like this idea, because ids_please hasn't got any dependencies and obviously not all people will use CLI version. So add a single dependency for only CLI version - bad idea

And why you don't like using many puts instead of multiline string syntax?

I agree that thor is not needed here as not so many logic and options in CLI.
Many puts with many strings 1) looks not clean 2) many excess objects (strings) 3) many excess IO operations instead of one

Ok. I haven't thought about it.
Btw I almost done

Updated. What about now?

You're right about multi-line strings: they're prettier than a lot of puts

Thank you! Yes, that's better. I'll refactor it a little later as there are some logic that not related to CLI but to grabbers and parsers itself.

And we should said about CLI version in README


Non-functional D and . Keys

I got my M60 fully assembled but I'm having issues with a couple of the keys. Everything seems to work fine right now except for the D and . keys. I've checked and as far as I can tell the switch is seated correctly. I'm not sure if I'm looking at a hardware issue with those two keys or something else. Any ideas?

I'd swap the keyswitch for another and see what happens. Are the keys not working at all?
There is also a software issue with the default firmware and the d key, but I don't think it affects the . key:
https://github.com/makerdiary/python-keyboard/issues/11

How did you test these keys?
By default, D, ; and B are used as tap-keys. Tapping these keys outputs normal keys. Holding down these keys activates additional functions.

Neither tapping, nor holding produced any input from these keys. Holding and pressing the additional function keys that should work for D did not produce any results either.

It is likely to be a hardware issue.
To double-check, We can test these keys with https://keyboard.makerdiary.com/#/test

You said N and  . didn't work on twitter. Now N and  . don't work. Is it related?

This was a hardware issue on my end. This can be closed. Sorry for the false issue.

This was a hardware issue on my end. Closing this. Sorry for false issue.


KeyError: 'none'

Hi,
When I perform the signature extraction on a matrix, the process was interrupted by "KeyError: 'none'".
See below for detailed error reporting information and the matrix file is attached.
Can you help me to handle this issue?
Thanks for all the hard work and a great tool!
COAD.SBS18.all.txt

Time taken to collect 500 iterations for 25 signatures is 1072.57 seconds
Optimization time is 2.986114025115967 seconds
The reconstruction error is 0.0219, average process stability is 0.17 and
the minimum process stability is -0.41 for 25 signatures
Traceback (most recent call last):
File "/lustre/wangzh/TR/test.py", line 16, in
main_function()
File "/lustre/wangzh/TR/test.py", line 13, in main_function
sig.sigProfilerExtractor("matrix", output, path, reference_genome= Genome, opportunity_genome=Genome, cpu = -1)
File "/lustre/wangzh/tools/miniconda3/lib/python3.7/site-packages/SigProfilerExtractor/sigpro.py", line 1031, in sigProfilerExtractor
final_signatures = sub.signature_decomposition(processAvg, m, layer_directory2, genome_build=genome_build, add_penalty=0.05, remove_penalty=0.01, mutation_context=mutation_context, make_decomposition_plots=make_decomposition_plots, originalProcessAvg=originalProcessAvg)
File "/lustre/wangzh/tools/miniconda3/lib/python3.7/site-packages/SigProfilerExtractor/subroutines.py", line 1349, in signature_decomposition
merger.write(directory+"/"+contexts[mtype_par]+"_Decomposition_Plots.pdf")
KeyError: 'none'

@ntbeileh I am just checking,  were you able to figure out what is causing the issue?
Thanks,
Mishu

Hi @WZo0o could you please set the parameter "make_decomposition_plots" =True and run the analysis again? I was able to solve the issue in that way. Here is code I used:
public_test.zip

Hi @mishugeb ,
I tried to run the analysis with your code and found the same issue. Here is  error reporting information:
"""
Time taken to collect 10 iterations for 3 signatures is 8.23 seconds
Optimization time is 3.02370023727417 seconds
The reconstruction error is 0.1803, average process stability is 0.9 and
the minimum process stability is 0.8 for 3 signatures
Traceback (most recent call last):
File "/lustre/wangzh/TR/test/public_test/sigpro.py", line 28, in
run()
File "/lustre/wangzh/TR/test/public_test/sigpro.py", line 22, in run
nnls_add_penalty=nnls_add_penalty, nnls_remove_penalty=nnls_remove_penalty, initial_remove_penalty=initial_remove_penalty, make_decomposition_plots=False, get_all_signature_matrices=False, cpu=-1)
File "/lustre/wangzh/tools/miniconda3/lib/python3.7/site-packages/SigProfilerExtractor/sigpro.py", line 1031, in sigProfilerExtractor
final_signatures = sub.signature_decomposition(processAvg, m, layer_directory2, genome_build=genome_build, add_penalty=0.05, remove_penalty=0.01, mutation_context=mutation_context, make_decomposition_plots=make_decomposition_plots, originalProcessAvg=originalProcessAvg)
File "/lustre/wangzh/tools/miniconda3/lib/python3.7/site-packages/SigProfilerExtractor/subroutines.py", line 1349, in signature_decomposition
merger.write(directory+"/"+contexts[mtype_par]+"_Decomposition_Plots.pdf")
KeyError: 'none'
"""
In addition, I noted that code in your script "make_decomposition_plots=False".  So, I seted the parameter  "make_decomposition_plots=True" and run it again. The error is still existed and  here is  error reporting information:
"""
Time taken to collect 10 iterations for 3 signatures is 6.35 seconds
Optimization time is 3.0817880630493164 seconds
The reconstruction error is 0.1805, average process stability is 0.86 and
the minimum process stability is 0.72 for 3 signatures
Traceback (most recent call last):
File "/lustre/wangzh/TR/test/public_test/sigpro.py", line 28, in
run()
File "/lustre/wangzh/TR/test/public_test/sigpro.py", line 22, in run
nnls_add_penalty=nnls_add_penalty, nnls_remove_penalty=nnls_remove_penalty, initial_remove_penalty=initial_remove_penalty, make_decomposition_plots=True, get_all_signature_matrices=False, cpu=-1)
File "/lustre/wangzh/tools/miniconda3/lib/python3.7/site-packages/SigProfilerExtractor/sigpro.py", line 1031, in sigProfilerExtractor
final_signatures = sub.signature_decomposition(processAvg, m, layer_directory2, genome_build=genome_build, add_penalty=0.05, remove_penalty=0.01, mutation_context=mutation_context, make_decomposition_plots=make_decomposition_plots, originalProcessAvg=originalProcessAvg)
File "/lustre/wangzh/tools/miniconda3/lib/python3.7/site-packages/SigProfilerExtractor/subroutines.py", line 1349, in signature_decomposition
merger.write(directory+"/"+contexts[mtype_par]+"_Decomposition_Plots.pdf")
"""
Thanks for all the hard work and a great tool!

Hi @mishugeb ,
I tried to run the analysis with your code and found the same issue. Here is  error reporting information:
"""
Time taken to collect 10 iterations for 3 signatures is 8.23 seconds
Optimization time is 3.02370023727417 seconds
The reconstruction error is 0.1803, average process stability is 0.9 and
the minimum process stability is 0.8 for 3 signatures
Traceback (most recent call last):
File "/lustre/wangzh/TR/test/public_test/sigpro.py", line 28, in
run()
File "/lustre/wangzh/TR/test/public_test/sigpro.py", line 22, in run
nnls_add_penalty=nnls_add_penalty, nnls_remove_penalty=nnls_remove_penalty, initial_remove_penalty=initial_remove_penalty, make_decomposition_plots=False, get_all_signature_matrices=False, cpu=-1)
File "/lustre/wangzh/tools/miniconda3/lib/python3.7/site-packages/SigProfilerExtractor/sigpro.py", line 1031, in sigProfilerExtractor
final_signatures = sub.signature_decomposition(processAvg, m, layer_directory2, genome_build=genome_build, add_penalty=0.05, remove_penalty=0.01, mutation_context=mutation_context, make_decomposition_plots=make_decomposition_plots, originalProcessAvg=originalProcessAvg)
File "/lustre/wangzh/tools/miniconda3/lib/python3.7/site-packages/SigProfilerExtractor/subroutines.py", line 1349, in signature_decomposition
merger.write(directory+"/"+contexts[mtype_par]+"_Decomposition_Plots.pdf")
KeyError: 'none'
"""
In addition, I noted that code in your script "make_decomposition_plots=False".  So, I seted the parameter  "make_decomposition_plots=True" and run it again. The error is still existed and  here is  error reporting information:
"""
Time taken to collect 10 iterations for 3 signatures is 6.35 seconds
Optimization time is 3.0817880630493164 seconds
The reconstruction error is 0.1805, average process stability is 0.86 and
the minimum process stability is 0.72 for 3 signatures
Traceback (most recent call last):
File "/lustre/wangzh/TR/test/public_test/sigpro.py", line 28, in
run()
File "/lustre/wangzh/TR/test/public_test/sigpro.py", line 22, in run
nnls_add_penalty=nnls_add_penalty, nnls_remove_penalty=nnls_remove_penalty, initial_remove_penalty=initial_remove_penalty, make_decomposition_plots=True, get_all_signature_matrices=False, cpu=-1)
File "/lustre/wangzh/tools/miniconda3/lib/python3.7/site-packages/SigProfilerExtractor/sigpro.py", line 1031, in sigProfilerExtractor
final_signatures = sub.signature_decomposition(processAvg, m, layer_directory2, genome_build=genome_build, add_penalty=0.05, remove_penalty=0.01, mutation_context=mutation_context, make_decomposition_plots=make_decomposition_plots, originalProcessAvg=originalProcessAvg)
File "/lustre/wangzh/tools/miniconda3/lib/python3.7/site-packages/SigProfilerExtractor/subroutines.py", line 1349, in signature_decomposition
merger.write(directory+"/"+contexts[mtype_par]+"_Decomposition_Plots.pdf")
"""
Thanks for all the hard work and a great tool!

Hi @WZo0o could you please install the SigProfilerExtractor package from GitHub and let me if that is working?
Thanks,
Mishu

Hi @mishugeb
Do you mean that I uninstall SigProfilerExtractor that was installed  through the following command "pip install SigProfilerExtractor" and re-install the SigProfilerExtractor package from GitHub?

Yes, that's correct.
Sent via the Samsung Galaxy S7 edge, an AT&T 4G LTE smartphone
Get Outlook for Androidhttps://aka.ms/ghei36

From: WZo0o @.>
Sent: Tuesday, May 25, 2021 1:28:20 AM
To: AlexandrovLab/SigProfilerExtractor @.>
Cc: Islam, S @.>; Mention @.>
Subject: Re: [AlexandrovLab/SigProfilerExtractor] KeyError: 'none' (#76)
Hi @mishugebhttps://urldefense.com/v3/__https://github.com/mishugeb__;!!LLK065n_VXAQ!2C-aloMMdMb3PKQRvM54Q52S3Se9NWNF3fNzpF7QIaqKX36CyW3aOF88hakL_GHvrA$
Do you mean that I uninstall SigProfilerExtractor that was installed through the following command "pip install SigProfilerExtractor" and re-install the SigProfilerExtractor package from GitHub?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://urldefense.com/v3/__https://github.com/AlexandrovLab/SigProfilerExtractor/issues/76*issuecomment-847666187__;Iw!!LLK065n_VXAQ!2C-aloMMdMb3PKQRvM54Q52S3Se9NWNF3fNzpF7QIaqKX36CyW3aOF88hanuXN9oCA$, or unsubscribehttps://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/AFOQ2HLXA6SEDXHVSIW2K3TTPNNSJANCNFSM44MCKOSA__;!!LLK065n_VXAQ!2C-aloMMdMb3PKQRvM54Q52S3Se9NWNF3fNzpF7QIaqKX36CyW3aOF88hak1g7Fn2w$.

Hi @mishugeb , after I install the SigProfilerExtractor package from GitHub,  I can run the analysis successfully.
Thanks for all the hard work and a great tool!


Feature/#48: Android integration tests

closes #48

@gsocher What else should be added to the travis-ci pipe?

We have currently the following issue:
Our app needs the new java.time-API, and therefore a minimum android-API Level of 26 is a hard requirement. On the other side travis-CI cannot execute the integration tests with this API Level. The execution Level with API-Level 22 works as expected.

This pull request will be closed, because the CI will be implemented with Github Actions. This means the pull request #62 itself manages all needs.
The reason for this is the following: We currently need a minimum API-Level of 26 in order to use the java.time-API and travis couldn't handle android integration tests with that version as describe in the thread https://travis-ci.community/t/add-kvm-support/1406/12


chore(types): fixed types to use selenium-webdriver

Currently when installed e.g. with Aurelia skeleton project (in my case) there are several errors thrown when building due to protractor.
ERROR in [at-loader] node_modules\protractor\built\browser.d.ts:260:37
    Cannot find namespace 'webdriver'.

ERROR in [at-loader] node_modules\protractor\built\browser.d.ts:267:55
    Cannot find namespace 'webdriver'.

ERROR in [at-loader] node_modules\protractor\built\browser.d.ts:267:78
    Cannot find namespace 'webdriver'.

ERROR in [at-loader] node_modules\protractor\built\browser.d.ts:358:31
    Cannot find namespace 'webdriver'.

ERROR in [at-loader] node_modules\protractor\built\browser.d.ts:369:26
    Cannot find namespace 'webdriver'.

ERROR in [at-loader] node_modules\protractor\built\browser.d.ts:425:36
    Cannot find namespace 'webdriver'.

ERROR in [at-loader] node_modules\protractor\built\ptor.d.ts:13:29
    Cannot find namespace 'webdriver'.

This should fix just that.

I believe the latest commit should be style(browser): code formatting to respect angular commit convention. But other than that it looks good. It would be great to have it merged soon, it is breaking the build on our side as well. Thanks @stephenlautier for this !

Closing this PR since this has been resolved with #3848. The JSDoc changes does not fix the transpile errors. The errors were due to a DefinitelyTyped @types/selenium-webdriver release that changed the typings. We will have a hot fix later today. Thanks for the PR.

@cnishina i know the JSDocs doesnt fix the errors, that was extra since they were not pointing correctly. the fixes are for the actual params/return types. its good if its fixed else were👍


Is it possible to change the toolbox icons?

One-line summary [问题简述]
Is it possible to change the toolbox icons? Would be nice to do so it can fit in better with an icon set already in use within the app.
Version & Environment [版本及环境]

ECharts version [ECharts 版本]: 4.1
Browser version [浏览器类型和版本]: Chrome Dev
OS Version [操作系统类型和版本]: OSX

Expected behaviour [期望结果]
ECharts option [ECharts配置项]

option = {
toolbox: {
            show: true,
            right: 120,
            orient: 'horizontal',
            itemSize: '13',
            itemGrap: '10',
            feature: {
                restore: {
                    show: true,
                    iconStyle: {
                        borderColor: '#4b0082'
                    },
                    emphasis: {
                        iconStyle: {
                            borderColor: '#4b0082'
                        },
                    }
                },
                saveAsImage: {
                    show: true,
                    iconStyle: {
                        borderColor: '#d2691e'
                    },
                    emphasis: {
                        iconStyle: {
                            borderColor: '#d2691e'
                        },
                    }
                }
            }
        }
}

Other comments [其他信息]

It's possible using icon property (eg. https://echarts.apache.org/en/option.html#toolbox.feature.saveAsImage.icon)
NOTE: in your code you have a typo itemGrap should be itemGap


Cannot Able to Deploy tokens on Private Node Running on Geth

Hi
I was trying to deploy token on my private geth node by modifying your your this section of code,
goerli:{web3Provider:"HTTP://<IP_ADDRESS>:8545",etherscanLink:"https://goerli.etherscan.io",id:15230,name:"Local Test Network"}
but I was able to genrate transaction hash & contract address but after that, I'm getting this error
The contract code couldn't be stored, please check your gas limit.
My genesis.json is this:
{
    "config": {
        "chainId": 15230,
        "homesteadBlock": 0,
        "eip150Block": 0,
    "eip150Hash": "0x0000000000000000000000000000000000000000000000000000000000000000",
    "eip155Block": 0,
    "eip158Block": 0,
    "byzantiumBlock": 0,
    "constantinopleBlock": 0,
    "petersburgBlock": 0,
    "istanbulBlock": 0
    },
    "difficulty": "0x400",
    "gasLimit": "0x2100000",
    "alloc": {
        "BFC348d948154f9a77332B0081135527c79d8dab":
         { "balance": "0x1337000000000000000000" }
    }
}

My Geth version is
VERSION:
1.10.16-stable-20356e57
Can you please help me with this..!?

This repository is for documentation.
I do not provide support for token generator as many people forked and use it to scam.


Add affinity, tolerations and priorityClassName

Right now, cvmfs-csi does not have functionality for affinity, tolerations and priorityClassName to daemonSets
https://github.com/cernops/cvmfs-csi/blob/master/deployments/helm/cvmfs-csi/templates/csi-cvmfsplugin.yaml
Adds  affinity, tolerations and priorityClassName functionality.
Reference:  https://github.com/cernops/cvmfs-csi/pull/31#issuecomment-806039776

Superseded by #39. Closing.


Encrypted messages are supported for a Shared Mailbox. What are the limitations?

In this article https://docs.microsoft.com/en-us/microsoft-365/compliance/ome-faq, it is mentioned that Yes! Encrypted messages are supported for a Shared Mailbox. Some limitations apply. So what are the limitations? Those limitations should be mentioned in this article.
Also, I am not able to read encrypted message from a shared mailbox via Outlook desktop client. So is this supported now?

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

ID: d4c75d7b-eec1-720b-bc4d-91d3dfe72889
Version Independent ID: 0382d4b5-3fe2-f353-ce73-5d437d80b7bf
Content: Office 365 Message Encryption FAQ
Content Source: microsoft-365/compliance/ome-faq.md
Service: o365-seccomp
GitHub Login: @KCCross
Microsoft Alias: krowley

@prashant1380  - Thank you for submitting feedback.
I will get this issue over to the Microsoft 365 writing team for investigation.
Thank you for reporting and making the docs better. Much appreciated.
I made a note to request the team to update this when the work is complete.

Equally interested in an answer to; "I am not able to read encrypted message from a shared mailbox via Outlook desktop client. So is this supported now?". Please can this be clarified. Please can we indicate the positioning of this within the Microsoft 365 RoadMap.

The list is coming from Ignite. Soon as I have clarification on it, I'll publish.

@prashant1380  - Thank you for your message. Can you please share the error screenshot you are getting while open email on outlook desktop client?
Can you please confirm if that is accessible via OWA/Webmail?
Thanks.

Yes, encrypted email opens in OWA and in Outlook mobile app. But in Outlook desktop client, first it says below when viewed in reading pane.

Then i double click the message to open it, but i get this message

Then I click OK and click on Read Message link available in email which tries to open the message in OWA but this gives an error like below

So tried all possible ways in desktop client but email is not readable. If I directly open OWA, I am able to read the message.

@prashant1380 as @KCCross is the primary author of the document and she will update the details regarding this soon.
In the meantime I would suggest you to you a support case with Microsoft to help you with this.
Thanks.
@mypil FYA Please.

I am having the same problem.  Any updates on the "limitations" or more important, a fix?

We have been battling with OME and shared mailboxes for weeks, as far as we can see nothing has technically changed whatsoever. I suspect Microsoft have decided to change their wording so it didn't sound as negative, but instead has probably made it worse. I suspect the limitations will be what we all know a.k.a you must open the content in OWA.
We've had to setup a transport rule which relies on us knowing the external recipient team mailbox address and asking our users to trigger encryption using the confidential flag, the result being OMEv2 is removed and the message is encrypted with OMEv1. It means a user can email an external team shared inbox and that inbox receives an OMEv1 message which gives them the OTP option instead. A huge pain to support but it works.

@prashant1380 @nsonew - You may try the suggested fix provided by @AHawky or the best way forward is if you can open a service ticket in your tenant so this can get resolved ASAP. Based on outcome, let me know if it is something that can be called out in the docs.
If you don't mind please keep us posted here on the resolution and I really want to get this information you are discovering back into the docs.

Please advise how this is closed. This constraint is prohibiting us from adopting OME and sadly, it seems there have been no changes to the functionality, simply a change in wording as previously where 'shared mailboxes are not supported' is deemed too negative. I personally would prefer the truth over a politicians answer. To summarise - Shared Mailboxes are NOT supported unless using OWA (which is not feasible).

Update: I am updating the content with a more complete picture of supported scenarios. Yes, Outlook on the web is supported, but it's not the only method for consuming encrypted email or email attachments sent to shared mailboxes. Experiences that are blocked within supported scenarios indicate things that we need to fix. Please do continue to escalate through support and our product team will work diligently to fix them. You can expect the content to be updated much later today.

Content updated to describe supported scenarios.

Still not fixed. Still not working for Outlook on Desktop (As it clearly says it should). Is this just being ignored?? Please move this back to open. Thanks


build

build fail

:tada: This issue has been resolved in version 2.0.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Running NLog in ASP.net shared hosting environment

NLog does not currently run in ASP.NET partial trust. The issue has been reported multiple times on the forum:
http://n2.nabble.com/Running-NLog-in-ASP-net-shared-hosting-environment-td1685842.html#a1685842

Security Exception
Description: The application attempted to perform an operation not allowed by the security policy. To grant this application the required permission please contact your system administrator or change the application's trust level in the configuration file.

Exception Details: System.Security.SecurityException: That assembly does not allow partially trusted callers.

Source Error:

An unhandled exception was generated during the execution of the current web request. Information regarding the origin and location of the exception can be identified using the exception stack trace below.

Stack Trace:

[SecurityException: That assembly does not allow partially trusted callers.]
 MyAssembly.MyClass..cctor() +0
Comments:
I've been able to get NLog to run in Partial Trust environments without a problem.
A couple of things I've had to do.

Add the "AllowPartiallyTrustedCallers" attribite to the assembly.
Strong Name sign the assembley
Made a change to bypass the "WindowsCreateFile" in BaseFileAppender - This causes File logging to fail

Of course under partial trust, you can only log to a file or directory under the current application.
Happy to send you my code if you want.

Could you please send code or could you please let know how exactly you've made changes?

@kshkrao3 this issue has been closed for more than 3 years. Is your question in the right issue?

@304NotModified Sorry to say but yes. I have hosted my application in a shared hosting wherein I cannot override full trust configuration and hence application is in partial trust mode. But I am getting the said exception because of above reasons. So, I would be glad if I receive any help on this.. :)

A new issue would be the best. Please provider the requested info and exception details/traces. Thanks!

Should I be opening new issue for this? Because I think this has been resolved as per @Xharze's workaround and I was asking for detailed help on same as he has mentioned at the end Happy to send you my code if you want. :)

Yes please. I don't know of the same fixes would work in NLog 4. A lot has been changed.

I must say then, I have to copy the same exception details as above in a new issue.. :) Please give me a Go or No Go.. :)

New one please. :)


Bug fixes

Fixes #9
Fixes wrong Constant.UPLOADED value causing the browser to close before longer videos are done uploading (YouTube changed their upload progress bar to say "Uploading ... " instead of "% uploaded")

why is this not getting merged?

why is this not getting merged?


Check night forecast is ok

OCF humans currently have not visibility of early morning forecast, lets plot these and check they are ok
BONUS: would be good to add a script that plot these for any time

Until we get to 05:30 in the morning, the forecasts dont look good.

Thanks for doing this analysis!
For now, maybe we just shouldn't produce any forecasts until, say, the Sun is 5 degrees above the horizon? For now, I'd rather publish nothing than publish something that's wrong :slightly_smiling_face:

gets tricky as if one GSP forecast is not made, then a national on is no made either.

For 2. Could we set the forecast for the gsp to 0, so then it can still run national but that gsp doesn't count?

For 2. Could we set the forecast for the gsp to 0, so then it can still run national but that gsp doesn't count?

This would potentially make the national forecast 50% of what it should be (which isn far off what we have now).
I think it would have to be, if any GSP elevation now < Y, then dont save the forecast?

Would it help if we used a single elevation threshold for the entire country? (e.g. the "centroid" of GB)? That way, we'd know that we'd either have all the GSPs, or we'd have none of the GSPs. Which maybe helps ensure that we're always computing the national forecast with the full complement of GSPs?

From discussion

Use 2. use limit of 5 sun elevation (5 to be safe, can adjust later)
Centroid will be oxford (PV weighted uk center)
Also set status to warning, when no PV forecast is made. This then show up in UI (when ready)


Deleting duplicate video references

Hi @odvoskin, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;

@odvoskin : Thanks for your contribution to the Azure documentation! The author, @azdocprs, has been notified to review your proposed change.

@SyntaxC4 - Cory, can you review this pull request in the public repo?

#sign-off

Only the author of the article can use the sign off comment to add the ready-to-merge label.


Fix publications links & button labels

English-language publications got a few fixes:

Renamed the source for all existing werf blog entities.
Removed external links from publications' descriptions.
Fixed date for the latest article; fixed service names for a few old articles.

I'm not sure why the content validation failed. My only guess is: that happened due to removing the habr_btn_title definition from the English configuration?

❤


typo

What I am changing
Typo
How I did it
Edited via Github

@wildintellect thank you so much for your :eyes:, so I will merge it.


Fix blocking read in Common.run

@psss This fixes my 'xdg-open' not returning issue. I'm not really happy to report green on timeout but I couldn't think of anything better.

Sorry, haven't noticed this is filed against the report-html branch.

This looks really complicated :) I love it.
Some questions come to my mind though:

Why would xdg-open block? I regularly run it everywhere and it did never block AFAIR.

Leaving the stdout could maybe cause this issue. Will this help?  xdg-open file:///my/path/index.html >/dev/null
If not, what would happen if it was run like this? nohup xdg-open file:///my/path/index.html &

What about https://docs.python.org/3/library/webbrowser.html ?

@pvalena Thank you for webbrowser idea. I have created #455 with that. This PR will be about blocking Common.run

Recap of issue this is trying to fix:
On my fedora-33 when I had no firefox running following  didn't finish (=continue on next line) until I manually closed newly opened window:
tmt.utils.Common().run('xdg-open /var/tmp/tmt/run-018/DEMO/report/default/index.html', timeout=2)
Note there is timeout=2 which successfully killed the process, however execution was blocked either on select.select(descriptors, [], []) in process.poll() loop OR on for line in process.stdout.readlines()

The change looks good to me!

Rebased and merged. Thanks much for investigating this, @lukaszachy!


Algorithm impala cannot use GPU[ALTER]

machin/frame/algorithms/impala.py, line 363, 373
vs[idx] = (value[idx] + delta_v[idx] +                                self.discount * c[idx] *                                (vs[idx + 1] - value[idx + 1]))
This should be corrected to the following code
vs[idx] = (value[idx].to('cpu') + delta_v[idx] +                                self.discount * c[idx] *                                (vs[idx + 1] - value[idx + 1].to('cpu')))
Do the same for line 373

Fixing the problem, currently I am restructuring my test suite to support GPU testing.

OK, problem should have been fixed, now merge to release branch and push as a new version:
https://github.com/iffiX/machin/commit/5fca74e733962c28e3190b60f8b79fbbe18a0aaf


Add draw_spaces option

Determines what kind of whitespaces should be drawn. Use comma-separated
list of: space, tab, newline, nbsp, leading, text, trailing or all.

With Boost 1.60 this will generate warnings (from Spirit), but this will be fixed (I hope) in 1.61 (almost released). Is it ok to use Spirit in this project at all?
Should I rename the option from "draw_spaces" to "show_spaces" as in "show_line_numbers"?
Why don't you use raw string literals in files.h?

Thank you, I'll have a look at this in a day or two. Default value should be empty I think (no whitespace shown).

You might be able to disable the warnings by adding system in CMakeLists.txt like this:
include_directories(SYSTEM ${global_includes})
show_whitespaces? If newlines are shown, this is not a space but whitespace I guess
Actually, I have not tried raw string literals, but it seems like a good idea to use it (if g++4.8 supports it)

The currently oldest system we support now is Ubuntu 14, but in a few months we'll stop supporting Ubuntu 14. Then, the oldest supported system will be Debian stable. If this works on these two old systems I'm happy.

I've set it to empty by default and fixed the warning. Let's continue when you have time.

Great, and also, before I forget, we have to up the JUCI_VERSION to ""1.1.3-2" so that people get the new preference item.


Prevent readRules from lowercasing the css selectors

Fixed the readRules to prevent accidental lowercasing of the css rules that ends up breaking on IE10/11,

fixes #60


HOTFIX: Reverted timeouts to larger values

LGTM. cc @ijuma

LGTM

Until we have a better solution anyway (I hope it's soon!)

Merged to trunk and 0.10.0.


[bug] Downloads with same chapter name but different scanlators both show download when only 1 did

fixed in <IP_ADDRESS>


Added ability to provide headers for Qlik request

So our QlikSense proxy requires header authentication. I've simply allowed developers to set a headers object in the config file that allows them to specify any headers that out to be sent with the request for the health check.

Added in version 9.10.0


Migrated to the federated architecture

:sparkles: What kind of change does this PR introduce? (Bug fix, feature, docs update...)
I migrated the plugin to the new federated architecture
:arrow_heading_down: What is the current behavior?
The plugin is in the old architecture
:new: What is the new behavior (if this is a feature change)?
The plugin is split in separate projects for android/ios and is thus in the federated architecture
:boom: Does this PR introduce a breaking change?
No
:bug: Recommendations for testing
Run the example project
:memo: Links to relevant issues/docs
:thinking: Checklist before submitting

[x] I made sure all projects build.
[x] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy.
[x] I updated CHANGELOG.md to add a description of the change.
[x] I followed the style guide lines (code style guide).
[x] I updated the relevant documentation.
[x] I rebased onto current master.

Hi @paulppn,
Thanks for this PR looks very good, a few points:

The .gitignore files in the geocoding_android and geocoding_ios look like the default (minimum) configuration. It would be good to update these with the default Flutter .gitignore versions.
The geocoding_android/example and geocoding_ios/example projects contain a test folder both with an empty test file. Since we'll not be maintaining tests for the example projects these folders can be removed.
Can we separate the changes to the app-facing package (geocoding) into a separate PR? This way we can update the and merge it when the new geocoding_android and geocoding_ios packages have been published.


Fetch is not defined, when running migrator

When running npx directus-migrator -s development -t staging I get the following error, this is the first time I'm using this library, followed the docs.
This is my config
const config = {
    "environments": [
        {
            "name": "development",
            "endpoint": "http://localhost:3000",
            "accessToken": "XXXXXXX"
        },
        {
            "name": "staging",
            "endpoint": "https://XXXXXXXX.cleaver.rocks",
            "accessToken": "XXXXXXX"
        }
    ]
}
 module.exports = config%

This is the error
2023-06-18<EMAIL_ADDRESS>info: Migrating Schema Started
/Users/miguelstevens/.npm/_npx/2f9ed6e235e43e28/node_modules/directus-migrator/dist/utils/CRUD.js:48
        const response = yield fetch(url, {
                         ^

ReferenceError: fetch is not defined
    at /Users/miguelstevens/.npm/_npx/2f9ed6e235e43e28/node_modules/directus-migrator/dist/utils/CRUD.js:48:26
    at Generator.next (<anonymous>)
    at /Users/miguelstevens/.npm/_npx/2f9ed6e235e43e28/node_modules/directus-migrator/dist/utils/CRUD.js:8:71
    at new Promise (<anonymous>)
    at __awaiter (/Users/miguelstevens/.npm/_npx/2f9ed6e235e43e28/node_modules/directus-migrator/dist/utils/CRUD.js:4:12)
    at CRUD (/Users/miguelstevens/.npm/_npx/2f9ed6e235e43e28/node_modules/directus-migrator/dist/utils/CRUD.js:45:12)
    at /Users/miguelstevens/.npm/_npx/2f9ed6e235e43e28/node_modules/directus-migrator/dist/migrators/schema-migration.js:69:51
    at Generator.next (<anonymous>)
    at /Users/miguelstevens/.npm/_npx/2f9ed6e235e43e28/node_modules/directus-migrator/dist/migrators/schema-migration.js:31:71
    at new Promise (<anonymous>)

I can't seem to reproduce this error

I have the same issue, It seem it's required node v18 that come with global fetch.


HIGH: Thread messages are missing from public room

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number:
Reproducible in staging?: needs reproduciton
Reproducible in production?: needs reproduciton
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers): @quinthar  https://staging.new.expensify.com/r/868417741568295 and https://staging.new.expensify.com/r/6776919265133947
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @quinthar
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1710204955795119
Action Performed:

Open a public room in which there are already a bunch of threads

Expected Result:
User should be able to view all the thread messages
Actual Result:
Thread messages are disappaeared
Workaround:
unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01fe380d1a88aa6cc2
Upwork Job ID:<PHONE_NUMBER>137488896
Last Price Increase: 2024-03-12

Seems like David's OpenReport call is not even getting the childReportID property and other thread related properties for the report actions which should be added here https://github.com/Expensify/Auth/blob/bfd557749a4f372ad331f74defb7df026fe1037d/auth/lib/Report.cpp#L6984-L7031

My plate is quite full right now focusing on performance and Critical wave collect tasks so sending this to the pool if anyone has spare cycles to dig into it. Given the reproduction steps will be quite tough locally it might take some time.

Still haven't picked up

@quinthar do you want to do an open call for volunteers for this bug?

Can look into this

Thanks @jasperhuangg !

@jasperhuangg is working on this now.

I'm going OOO for the rest of this week, so I'm looking to get started on this next week

@jasperhuangg I know you just got back today but can you add an update either here or in Slack so we can keep the issue up to date?

is this still happening for anyone?  This is working fine for me.  I'm going to just close this; reopen if it's happening to you.

Tried to reproduce it as well with the same room and I'm able to load all the threads. Feel free to reopen if we see it crop up again and provide the reportID of the parent report and the thread, thanks!


Attempt to decode JSON with unexpected mimetype: text/html

https://pastebin.com/kwhuKzbJ
this is my keyboard
https://pastebin.com/mfCEcLte
this is traceback
I am trying to pass this keyboard to user but i get this error.
Also I tried to pass content-type = None in the file aiovk/drivers.py on line 52 to the json method, but this did not solve the problem.

Sorry, I found an issue that is similar to mine in # 49 issue. There are more than 5191 symbols in url.


Use remote state and add ACTION variable to manual Terraform commands

Notes
Sorry, no story ID. I'm running out of things to do and flying off piste 😈
What
manually_upload_certs: Use Terraform remote state
This makes it consistent with the set-up-pingdom.sh script which was
modified in b01f941 to use Terraform's remote state functionality. This is
simpler and safer than downloading and uploading the statefile with aws-cli.
When testing this with terraform plan instead of terraform apply against
staging and production (it's not possible to use this in dev or CI) the
following error was produced on the very first run:
# check password store and if varables are accesible
Initialized blank state with remote state enabled!
Remote state configured and pulled.
Remote state cache has no remote info
make: *** [manually_upload_certs] Error 1

After which a remote hash is written to the remote statefile and
subsequent runs work as expected.
Add ACTION variable to manual Terraform commands
So that we can run plan instead of apply to safely review changes. I
made a change similar to this to my local working copy when reviewing a
credentials PR
because I didn't want to make any unexpected changes to the live config.
I thought about setting a safe default action of plan, but decided against
it because people might assume that it had run apply and changes wouldn't
have been applied. It would have also been possible to re-use the ACTION
variable within the scripts but I thought it was better to explicitly parse
a CLI argument.
I've updated the documentation accordingly, listing both commands that you
are likely to use.
How to review
I'll describe using the least privileged environment that each target can be used in.
Pingdom

Export AWS keys and DEPLOY_ENV=master for CI.
Run the command with ACTION and confirm that it reports a useful error:

make ci pingdom

Run plan and confirm that it reports no changes or errors:

make ci pingdom ACTION=plan

Run apply and confirm that it reports no changes or errors:

make ci pingdom ACTION=apply

Certs

Export AWS keys and DEPLOY_ENV=staging for Staging.
Export CERT_PASSWORD_STORE_DIR to match where you have the credentials-high repo.
Run the command with ACTION and confirm that it reports a useful error:

make staging manually_upload_certs

Run plan and confirm that it reports no changes or errors:

make staging manually_upload_certs ACTION=plan

Run apply and confirm that it reports no changes or errors:

make staging manually_upload_certs ACTION=apply

Who can review
Probably needs a Civil Servant with SC because it the targets depend on the government-paas/credentials-high repo.

Was reviewing the Pingdom part of this one and encountered an issue. Using Terraform version 0.6.12 and running DEPLOY_ENV=master make ci pingdom ACTION=plan I get the following error:
Error loading local state: State version 3 not supported, please update.

Source code trace:
https://github.com/hashicorp/terraform/blob/v0.6.12/terraform/state.go#L1188
https://github.com/hashicorp/terraform/blob/v0.6.12/command/remote_config.go#L63
The state file from the CI bucket seems to indicate version 1, not version 3. The code also indicates Terraform v0.6.12 uses state version 1.
Would be good to know which version of Terraform the state file was produced by and whether anybody else has this issue.

Version 3 would seem to track to the (0.7)[https://github.com/hashicorp/terraform/commit/706ccb7dfedb9807fd65643a6e4eefd6a8b4f8f0#diff-1eac40478f2df1bf18b772c48cd7ce9aR22] series of terraform.

@richardc it does, but I don't even have that installed anymore. Wonder if something has been cached somewhere.

@HenryTK So I think that must be because of https://github.com/alphagov/paas-cf/pull/448/files#diff-a1cb0f2dcf908bac1ed895c8413bcdd0R7 which looks erroneous.
That statefile is version 4 on ci.

@HenryTK IGNORE. That is the serial and my eyes are malfunctioning. I can't find any tfstate with a  version other than 1.
Are you sure you don't have a local tfstate file that was confusing things during testing? The code looks correct.

@HenryTK DOUBLE IGNORE. Github hadn't refreshed and I hadn't seen your comment.
I give up.

I seem to remember @bazbremner saying something about them nuking local tfstate at the start of every run to prevent issues - is that still the case?

@Jonty - the script now does rm -rf .terraform to remove the local copy of the remote state at the end, but if the script errors it does not get cleaned up. We probably want to consider removing it at the start, because if anyone has any errors that result in the state file being left behind it could cause problems the next time they run it (particularly if somebody else has modified the Pingdom checks since then).
It seems like strange behaviour of terraform remote config to use the local cached copy in this way, especially when one of the main advantages of a remote config is the ability to share it across a team. I wouldn't consider it an issue, but perhaps a feature request to add -disable-local-state may be necessary. From this comment they seem to trust the local copy more for some reason.

@HenryTK Yes, it's very odd that it functions this way, I've heard others discussing it.
I've added a statement to remove the terraform state before running to stop this happening in the future.

This needs someone with access to credentials-high to run it and check it over now, it all looks fine now henry's problem has been resolved.

I've added a statement to remove the terraform state before running to stop this happening in the future.

Should we add this to the pingdom script as well as the certs script?

Tested the certs upload on staging, and it all works fine. Other than the 2 small comments above, this looks good to me.

@Jonty @HenryTK @alext I added a commit to handle the local state differently. Does this look sensible?

Apart from the review I requested above, I think we've dealt with everything we needed to.
I have one remaining question to check I understand Dan's original PR comment:
He talks about the least privileged environment each target can be run in; these being CI for pingdom and Staging for the cert upload. Is that simply because we don't run pingdom checks on dev environments, and we use self-signed certs in dev and CI environments?


Add an option to filter auto-discovered routes

Fixes #396

ok to test

ok to test

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/camel-quarkus-pr/392/

Need to be rebases after https://github.com/apache/camel-quarkus/pull/358 get merged

Superseded by https://github.com/apache/camel-quarkus/pull/402


Enable HW Flow control for NCP by default

In PR https://github.com/openthread/openthread/pull/2401 hardware flow control was enabled for UART because it is recommended by Spinel specification. But pyspinel and borderrouter don't configure HW flow control by default, neither other platforms has flow control enabled by default. So, NCP setup doesn't work out of the box and required additional configuration for EFR32.
The proposal is to disable HW flow control for EFR32 for now. To follow Spinel specification recommendation it should be enabled for all platforms once pyspinel and borderrouter set HW flow control by default. Or any other complex solution. Also flow control should not be enabled for CLI application by default.

Thanks for clarification. The drawback of such solution is creating another configuration file for UART only. There is one place where all configuration done, it is PROJECT_CORE_CONFIG_FILE which overrides default core config and obviously UART configuration should be done there. My proposal was to add possibility to override some settings by specific application. This solution is more generic. It would allow to have different openthread settings for different applications (not only different UART settings but different number of buffers etc.). But I will think about other solution which doesn't change core files.

Update: currently all platform specific code is compiled to a platform lib and then linked with ncp or cli application. It means that to implement above solution with uart header config two different platform libs are required. Which requires to make changes in build system. Basically it will not be the easy and beauty solution.

@al1img what about adding an API to examples/platforms/platform.h that allows for dynamically configuring the UART?

Technically this is possible: for example to add following function PlatformSetUartConfig(int speed, UartFlowControl flow) which stores UART config in some variables and this config will be applied on otPlatUartEnable call. But IMHO it will look like some kind of workaround.

@al1img, I agree that the approach is not ideal.  At the same time, we really want to minimize the interface between OpenThread and the platform layer - as a result, we want to limit the otPlat* APIs to represent only the functionality that OpenThread actually uses.  Open to other suggestions if you have them.

I'm not sure that this is an EFR32 issue per se. EFR32 enables flow control by default (except on the BRD4166 which has it disabled due to a feature of this demo board)

Closing stale issue.


Add working examples of NFT and Jettons for toncli

Most work done by @Trinketer22
But we want to add examples of NFT and Jettons with tests to toncli, so anyone can deploy those contracts via toncli

Done!


Non-linear wizards

I really like the idea of declarative wizard and how easy it is to setup and add animations. It's a very useful abstraction that otherwise takes a lot of efforts to implement.
For more complex use-cases, can I implement a non-linear, hierarchical flow with this library? I mean when the choice of the next step depends on previous steps?
Basically, what I want is a sort of a local react-router, where the routes can be nested, but without actual routing and URLs, just switch the components. I imagine something like that:
<Wizard>
  <StepStart />
  <StepA exclusive>
    <StepA1 />
    <StepA2 />
    <StepShared />
    <StepA3 />
  </StepA>
  <StepB exclusive>
    <StepB1 />
    <StepB2>
      <StepB2one />
      <StepB2two />
    </StepB2>
    <StepShared />
    <StepBFinish />
  </StepB>
</Wizard>

In this scenario, girst you are greeted with StepStart, then depending on a choice made there, you are taken to either StepA1 or StepB1, and then continue recursively along the hierarchy of components.
Think of it as a sequence of paths or URLs:
/start

# if user chose to go to the step A:
/start/step-a                        # not reachable, just a container
/start/step-a/step-a1
/start/step-a/step-a2
/start/step-a/step-shared
/start/step-a/step-a3
# finish here, because steps A and B are marked as exclusive

# if user chose to go to the step B:
/start/step-b                        # not reachable, just a container
/start/step-b/step-b1
/start/step-b/step-b2                # not reachable, just a container
/start/step-b/step-b2/step-b2-one
/start/step-b/step-b2/step-b2-two
/start/step-b/step-b2/step-shared
/start/step-b/step-b2/step-b-finish

Note that parent steps are not real steps, but just containers for their child steps. For that we probably need to introduce another component in the library. Alternatively, the Wizard component might also be adjusted to act as a nested sub-wizard probably.
There's need of some sort of a flag (exclusive ?), breaking sequentiality of steps: saying that I don't want to continue with StepB1 after StepA3. But in some cases you might want to continue. Or perhaps there could be an explicit Break component. This is the same as avoiding fall-through in a switch/case construct.
Navigation to a concrete step with goToStep might become challenging. One could use explicit IDs for each step and path/URL syntax to identify the step, e.g. goToStep('/start/step-b/step-b2')

Thanks for your feature request, but out of scope for this library. Feel free to fork!