Android 10: Image Picker showing Gray images in the image picking activity and when a image is selected app crashes.

Expected Behavior
Show images in the image picker
Actual Behavior
shows gray thumbnails in the image picker activity
Steps to Reproduce the Problem

Android 10 device
Open image picker
Select a image

Specifications

Image Picker Version: 2.2.0
Android OS: 10
Phone: One Plus 7 Pro

i was able to fix this problem by adding this in my manifest. but i guess this is temporary solution.
<manifest ... >     <!-- This attribute is "false" by default on apps targeting Android Q. -->     <application android:requestLegacyExternalStorage="true" ... >      ...     </application> </manifest>

OMG Thank you!

i was able to fix this problem by adding this in my manifest. but i guess this is temporary solution.
<manifest ... > <!-- This attribute is "false" by default on apps targeting Android Q. --> <application android:requestLegacyExternalStorage="true" ... > ... </application> </manifest>

android:requestLegacyExternalStorage="true" won't be allowed when targeting Android 11. Would be nice to have an update that handles images on Android 10 and up sooner than later.
https://developer.android.com/preview/privacy/storage

just recently ran into this bug. using the workaround for now, but +1 for an update that handles newer versions of Android

Thanks, @onirban27, But I think we need clarity reason from google. Because recently I updated the app to play store, My user have been announcement this is bug for me.

Hey guys, this should be fixed on version 2.3.0
Stay safe!

Its not fixed,its throw an exception  file not found(EACCESS DENIED),because scoped storage is not working,if i enable legacy storage true in manifest.its working fine


Update pubspec.yaml

Hi, what is the purpose of this change? https://dart.dev/tools/pub/pubspec#publish_to

This package needs to be published, don't know why this is a PR? Closing.


chore: remove redundant setState

Description
A setState() was used inside of the didUpdateWidget() method which is technically unnecessary as the flutter framework calls the build() method immediately after didUpdateWidget() is invoked.
Type of Change

[ ] ✨ New feature (non-breaking change which adds functionality)
[ ] 🛠 Bug fix (non-breaking change which fixes an issue)
[ ] ❌ Breaking change (fix or feature that would cause existing functionality to change)
[ ] 🧹 Code refactor
[ ] ✅ Build configuration change
[ ] 📝 Documentation
[x] 🗑 Chore

thanks for your contribution! 💯 💙


[Fix #9245] Fix Lint/AmbiguousRegexpLiteral when given a match_with_lvasgn node

Cop::Util#add_parentheses expected every node to have arguments, but match_with_lvasgn does not. This will not correct spaces properly (see the tests), but it will be picked up by Lint/ParenthesesAsGroupedExpression.
Fixes #9245.

Before submitting the PR make sure the following are checked:

[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Ran bundle exec rake default. It executes all tests and runs RuboCop on its own code.
[x] Added an entry (file) to the changelog folder named {change_type}_{change_description}.md if the new code introduces user-observable changes. See changelog entry format for details.

@koic updated! Thanks for the suggestions.

Thanks!


[Feature Request]: Version file

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Potential Enhancement
Let's say 1.19.3 comes out tomorrow. How would I add this to the installer? There needs to be a really easy way to manage these versions.
How it could be implemented
I would prefer a single file of name-link mapping, e.g.
{
   "1.16.5": "https://example.com/1.16.5/pack.toml",
   "1.17.1": "https://example.com/1.17.1/pack.toml",
   "1.18.2": "https://example.com/1.18.2/pack.toml",
   "1.19.3": "https://example.com/1.19.3/pack.toml"
}

The installer would simply ping that file on every launch to get the latest info, no installer updates are needed.
Anything else?
No response

Okay, where should the mapping file be located?

envs maybe? Just somewhere in this repo, not included with the download.

currently, the installer takes the versions from the packwiz folder on the main repo using the github api so it's all automated: https://github.com/Fabulously-Optimized/vanilla-installer/blob/5df7663611371e1fa1ee3457448c6d4c248e4755/vanilla-installer/main.py#L319-L340
unless i'm misunderstanding something

Hmm... On one hand, this automation is cool, on the other hand I don't want more people using 1.19 than there are now, aka I would still prefer to have a subset of these folders that are displayed in the installer.

Those are good points and it would let us not need to use github's api anymore.

I don't want more people using 1.19 than there are now

Sorry, I don't understand what you mean.

To sum it up, we (being FO) don't support or recommend 1.19(.0) even though it doesn't have chat reporting since we have NCR - so we shouldn't encourage users to install the 1.19(.0) version in VI.


Failed to start application on hoot.glitch.me

Hi, I wanted to checkout your project and visited hoot.glitch.me but unfortunately this page responds with an error:

failed to start application on hoot.glitch.me
This is most likely because your project has a code error.
Check your project logs, fix the error and try again.

Weird, looks like the application didn't like the Node 14 requirement in the package.json. I bumped it down to 12 and it appears to be up again. We also don't really recommend the Glitch link anymore, so I updated it to hoot.at!


Ability to setup and teardown applications

Is your feature request related to a problem? Please describe.
We'd like to be able to run application level tests without having to go through appium as an intermediary. This means that you load the app with every test case, and then interact with it directly via javascript, instead of at a distance of three processes and two HTTP hops like you would with Appium and WebDriver.
However, the the call to bootstrapModule() that is invoked in the boilerplate for main.ts never even appears to return.
Because of this, there is no way to get your hands on the application in order to call destroy() on it when it is time to tear down.
Describe the solution you'd like
A way to deterministically setup and teardown the angular application repeatedly like we do in this angular test suite for the web
https://github.com/thefrontside/vacation-tracker/blob/angular/bigtest/helpers/setup-app.ts
Additional context
On the web, when we ditched controlling our UI tests from a separate process, and moved them into the application JavaScript environment, life became beautiful again.
We'd love to be able to repeat this for native.
See also https://github.com/NativeScript/NativeScript/issues/7057

@cowboyd,
You’re totally right that bootstrapModule() never returns a value.
Let me first explain what is the reason for this behavior. Actually the problem is in the native UIApplicationMain function we’re using when initializing the native iOS application. As you can see here this function never returns, even though an integer return type is specified.
However, NativeScript iOS runtime relies on this function when bootstrapping the native iOS application. As this function blocks the execution and never returns, bootstrapModule() never returns as well.
We’ve discussed the problem with our team and we’d like to suggest you the following two approaches:

After receiving the moduleRef here, we can emit an event similar to this one.  This  way we can attach on this event before bootstrapping the application:

const nativescriptPlatform = platformNativeScriptDynamic();
nativescriptPlatform.on(“applicationInitialized”, (application) => {
	console.log(“Application: ”, application);
});
nativescriptPlatform.bootstrapModule(AppModule);

Or something similar can be done using callback:
platformNativeScriptDynamic((application) => {
	console.log(“Application: ”, application);
}).bootstrapModule(AppModule);

Or we can even use observable, if we’d like to be more angular friendly.

NativeScript supports bootstrapping of angular application inside an existing page. In other words we can create for example gridLayout and tells NativeScript to bootstrap the angular inside this gridLayout. We've prepared a simple project here demonstrating this behavior. As you can see, we're bootstrapping the angular application when the main-page is loaded. In other words we've already executed the blocking application.run function here. When onLoaded event of main-page is received, we can bootstrap the angular application but this function is not blocking anymore. As a result we'll receive a promise and can get the reference to the application object.

Let us know, if any of these approaches work for you.

Thank you for the consideration.
I think the option (2) is the approach that works because our ultimate goal is to setup and teardown the Angular application thousands of times during the course of the test suite run, not merely grab a reference to it.
It sounds like what we'll need to do is make a custom entry point that loads the test suite, which can then setup and teardown the angular application in the manner that you've described.
I'm wondering though, why not just adjust the generated entry point for angular applications to run the UIApplication and have the platformNativeScriptDynamic assume that it is in a running inside of a functioning UIApplication?
That way, there's no asymmetry between running as the main app, or running in an existing page, and the platform dynamic can have an API that matches other Angular platforms.
Also, am I missing something, because when I was investigating it before, it looked as though the bootstrapModule() promise was  null https://github.com/nativescript/nativescript-angular/blob/master/nativescript-angular/platform-common.ts#L147
In any case, I'm very encouraged that if we can setup and teardown the app within an empty shell page, that we can accomplish what we're setting out to do.

 Here are the answers to your questions:

I'm wondering though, why not just adjust the generated entry point for angular applications to run the UIApplication and have the platformNativeScriptDynamic assume that it is in a running inside of a functioning UIApplication?

When using 2. there will be a flash of white screen as the "app-wrapper" (test entry point in this case) is started and only after that the actual angular application is bootstrapped inside it. This is probably OK for tests, but is bad UX for production app.

Also, am I missing something, because when I was investigating it before, it looked as though the bootstrapModule() promise was null https://github.com/nativescript/nativescript-angular/blob/master/nativescript-angular/platform-common.ts#L147

The main difference here is that when creating the platform we pass in { bootInExistingPage: true}.  This creates a platform that does not execute the additional bootstrap logic related to NativeScript (here is the relevant location in the code). It also expects the APP_ROOT_VIEW token to be provided so that the NativeScript renderer knows where to render the application.


Bump Electron to 14

Closes #6

I think it's ready to push but test it first

Also fixes #5 i think

The best part is that thanks to your work we can even use the latest supported Electron version (19.0.8). I have set up a local branch and I'm testing to see how it works.

The best part is that thanks to your work we can even use the latest supported Electron version (19.0.8). I have set up a local branch and I'm testing to see how it works.

Leave it to 14 for now we'll have to see the breaking changes of all the versions to make sure nothing else breaks (it could break stuff on specific senarios, systems, OSes, etc)

I have to say I agree with you. Maybe on a separate branch or something. At least we know it is viable. I'm going to daily drive it though to see if there are bugs or anything Electron-related.


iOS/iPhone 15.2.1 reports navigator.platform as MacIntel

Expected behaviour
plyr correctly determines if it's running on iPhone/iOS 15.2.1
Actual behaviour
plyr doesn't realize it's using the iPhone/iOS
Steps to reproduce
call browser.isIPhone on an iPhone running iOS 15.2.1
Environment

Browser: Safari
Version: 15.2.1
Operating System:15.2.1
Version:15.2.1

Console errors (if any)
Link to where the bug is happening
browser.js

As a note, hls.js detects an iPhone because it's missing the MediaSource extensions. iPad/iOS and MacOS both support MediaSource, making it a convenient way of determining the platform.

Just noting that the iPhone only identifies itself as MacOS if the user has requested the desktop version of the website and browser.isIos still works in this mode (although it can't differentiate from iPads).


Peer to peer code review

Hi @albertkantwi
Good job so far!
Highlights!
Good code organization and design ✔
Good implementation on your codes ✔
Working with comments perfectly ✔
Hello ✋, you have done well reaching this point of the milestone 🤝 . But there are some issues that can be implemented in your project.
Status: Required Changes ♻⛔
In your index.html file. You did great work, but I suggest you start your heading with h1 followed h2 for your other headings.
https://github.com/albertkantwi/to_do_list/commit/b01844d4404254f393168904693cdd5d0f405ca1#r122596528.
In your README.md file
You made a great effort in implementing it to look professional.
Kindly, include live demo link section so that user can go through your project.
Optional suggestions
It is better to add hover element to your button element, so that user could know that this part is clickable.
Every comment with the [OPTIONAL] prefix is not crucial enough to stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread and tag me @the-faizmohammad if something is not 100% clear.

@the-faizmohammad good comments


[ci] Use default feed to install the .NET SDK

Attempts to install .NET 6.0.403 have been failing on various jobs:
Error: Unable to download https://dotnetcli.blob.core.windows.net/public/Sdk/6.0.403/dotnet-dev-win-x<IP_ADDRESS>3.zip. Returned HTTP status code: 404
Could not find ".NET Core SDK" with version = 6.0.403

Fix this by preferring the default feed found in the dotnet-install.ps1 script.

@mattleibow it looks like this fixes most of the failing jobs, but can not be auto-merged due to another issue with expired Apple certs.


Update Readme to ensure people update to latest setup tools

python -m pip install setuptools --upgrade

Has this been added?


Non-portable terminal control sequences in installer

The oh-my-zsh installer (install.sh as run through curl <installer_url> | sh) uses some terminal control sequences to change the color of the text it outputs. Something about these is not portable to some systems. On some systems, instead of changing the text color, it outputs some non-human readable "garbage" characters. These look like the original escape sequences as printed in the install.sh source.
This varies with some combination of the OS, terminal emulator, and what flavor of shell /bin/sh is on that system. Here's a table of what I've seen.

I think this is also supported by the fact that the original backslashes appear in the output. Not just for the terminal control escape sequences, but for the "\n" LF characters later on. So maybe the quote and backslash string syntax used in the installer is not portable. Or the behavior of the echo command is different between the shells.

Here are screenshots of the runs I did for figuring out that table. If anybody else has test cases under other combinations of system shells and terminals that would be enlightening, feel free to post and I'll add them to the chart. If you can, include the output of echo $TERM, uname -a, and (if your sh supports it) sh --version.
(Note the screenshots that say "FreeBSD" are actually PC-BSD, which is built on FreeBSD.)

One more data point: if I explicitly run install.sh under ksh instead of sh on PC-BSD, I get OK color changes, instead of the garbage when run under sh. This seems consistent with it being an issue of quotes/backslash interpretation under different shell flavors.
I've added this to the table in the issue.

Oh, maybe it's that using echo this way is not portable, because its interpreting of \-escapes is system-dependent.
The POSIX echo doc says:

If the first operand is -n, or if any of the operands contain a backslash ( '' ) character,
the results are implementation-defined.

And, on my OS X 10.9 box, man ksh says:
       echo [ arg ... ]
              When the first arg does not begin with a -, and  none  of  the
              arguments  contain a \, then echo prints each of its arguments
              separated by a space and terminated by a new-line.  Otherwise,
              the  behavior  of echo is system dependent and print or printf
              described below should be used.  See  echo(1)  for  usage  and
              description.

The man bash page for bash 3 says that its echo command interprets those escape sequences, but only if the -e option is given (but implies that might be the default on some systems).
So, maybe this should be switched to use printf instead of echo?

A couple references on the echo vs printf portability problem:
http://unix.stackexchange.com/questions/65803/why-is-printf-better-than-echo
http://www.in-ulm.de/~mascheck/various/echo+printf/

I put in a PR that replaces the echo calls with portable printf calls. Using these fixes the terminal control characters on FreeBSD and Cygwin, and I expect it will work across other systems too.
Screenshots showing the fix being tested:


added code to perform hab pkg export function

Added a couple functions for hab pkg export function (setFormat, getFormat)
Added case for export
Added exportCommand function

please make sure I got the array instantiation and checking for format string correct (line 170-175)

Please let me know if I missed anything!
resolves #1

I use vim with plugins for whatever language I'm writing, a java plugin in this case. If you point out where the formatting got weird I'll get a new commit on this pull request

Yeah javac would totally give you errors, you'll need to use maven, E.g.
$ mvn clean install

ref: https://maven.apache.org/what-is-maven.html

release 0.3.1 is available on update centre. :)


Add your details to membership.md

Go ahead and add your details to membership.md
Add the s.no , your name within a tags that hold the link to your github profile and your batch.
Open for all!

what do you mean by batch?

 For example, if you're an undergrad pursuing a major in let's say Computer Science and you started this year, mention 2018-2022

@rahulkumaran oh cool thanks

@rahulkumaran yo I tried to push a branch up to this but it says I don't have permission, am I missing something?

You cannot directly push anything to the repo as you're not the owner or a collaborator. You can send a PR and we'll merge it. @juniormice


No option to add ML module

Problem encountered on https://dotnet.microsoft.com/learn/ml-dotnet/get-started-tutorial/create
Operating System: Windows 8.1
Visual Studio 2019 v16.5.2
I installed the ML.NET Model Builder extension, restarted VS, created a new project, tried to add it to the project, but was not seeing a 'Machine Learning' option under Docker Support in the context menu.  I also noticed it isn't appearing in my Manage Extension -> Installed window.  I tried reinstalling the .vsix file but it said it was already installed.

@dtdono0 Sorry you've run into this problem and thanks for reporting! It sounds like you've done everything right, installed and restarted.
Just to double check, the extension isn't disabled, correct?

@dtdono0 Friendly ping, is this still broken for you? Thanks!

@beccamc Yes, still broken.
The ML.NET extension does not appear in the Manage Extensions window.  If I run Microsoft.ML.ModelBuilder.vsix from my Downloads folder, it says the extension is already installed

@dtdono0 Do you have more than one instance of Visual Studio installed? I've seen similar problems on my machines when I've had more than one VS. You can usually see the specific version(s) in the VS Installer.

@dtdono0 Friendly ping. Would love to see a screenshot of all installed version(s). The Visual Studio Installer should show all this information in one place.

@dtdono0 Do you have more than one instance of Visual Studio installed? I've seen similar problems on my machines when I've had more than one VS. You can usually see the specific version(s) in the VS Installer.

@beccamc Sorry for the delay, I didn't see your last ping.  I only have VS 2019 Enterprise installed:

@dtdono0 I am not sure what is going on, I'm going to pull in our extension installation team. Would you mind updating your VS? I think this is going to be the first thing they ask to do.

@dtdono0 Did you update your VS? Is this still happening?

Closing this due to inactivity - ping me or reopen if you need additional help!


autoflake seems to exit 0 even if a diff is produced

% /path/to/python3 -m autoflake ~/autoflake_test --remove-all-unused-imports --remove-unused-variables -r --exclude whatever/*
--- original//Users/brandon/autoflake_test/test.py
+++ fixed//Users/brandon/autoflake_test/test.py
@@ -1,3 +1,3 @@
 def main():
-       thing = "blah"
+       pass
 main()
--- original//Users/brandon/autoflake_test/test2.py
+++ fixed//Users/brandon/autoflake_test/test2.py
@@ -1,3 +1,3 @@
 def main():
-       thing = "blah"
+       pass
 main()
% echo $?
0

This makes it harder to run in CI because it's not exiting zero. What's the intended behavior here?

-m autoflake ~/autoflake_test --remove-all-unused-imports --remove-unused-variables -r --exclude whatever/*

If -c is used, the diff is suppressed. So this would make CI fail but would make it harder to know why. :/
Would you consider a bug that -c causes the diff to get suppressed?

+1. Ideally -c should print the diff to stdout. An alternative is a flag which toggles diff view or the condensed view that -c defaults to now.

@fsouza is there a requirement to keep backwards compatibility? Otherwise I'd propose that --check spits out the diff as well. If we need backwards compatibility maybe another keyword should be introduced which makes --check print diffs?
I'm able to fix it either way but I'd appreciate your input first.

@ericstengard can we do that via a new flag? This way we can keep backwards compatibility.

@ericstengard can we do that via a new flag? This way we can keep backwards compatibility.

Yes I'd say that it's fine. I'm thinking something like --check-verbose or --check-diff since --check by default makes it less verbose. I'm leaning more towards --check-diff.


Support AWS clusters.

Read the config file from local and connect to the cluster.

prerequisite - config file must be present in the local already.


Cannot use super.to_s in enum

Apparently, the Enum class doesn't currently allow using the original to_s method, e.g. when overriding:
enum MyEnum
        Foo
        Bar

        # the following code fails
        def to_s
                super.to_s.downcase
        end
end

This fails with:
Error in /opt/crystal/src/enum.cr:27: expanding macro

  macro def to_s : String
            ^~~~

in /opt/crystal/src/enum.cr:28: unknown macro instance var: '@enum_flags'

    {% if @enum_flags %}
          ^~~~~~~~~~~

As far as I can understand, the user-defined class cannot access the @enum_flags used in the macro of the parent's to_s.
Am I doing something wrong when overriding or is this a bug?

This works though: http://carc.in/#/r/d84

Or not really..

Played around a bit, and any variation of this code: http://carc.in/#/r/d90 - causes compiler errors and compile-time segfaults.

I just ran the supplied code and it seems to work.
https://carc.in/#/r/to4
Are you still seeing this issue?

Actually calling to_s then still fails though: https://carc.in/#/r/to5

so in more reading about overloading the to_s it seems like you should overload to_s(io) that seems to work in a weird way but no compiler bugs.
https://carc.in/#/r/tru
It seems like @type.constants is not populated when expanding the macro.  I have found a work around but I don't know why that is not available when the macros are being expanded.

Same issue still if you actually call it: https://carc.in/#/r/try

The problem is that Enum#to_s is actually:
struct Enum
  macro def to_s : String
    # ...
  end
end

A regular enum implicitly extends Enum... calling super on a to_s method ends up instantiating that method in Enum, and it doesn't have @enum_flags, nor it has enum members, because it's the base class.
In short, it makes no sense to invoke super there.
Solution:
enum MyEnum
  Foo
  Bar

  def to_s
    {% for member in @type.constants %}
      return {{member.stringify.downcase}} if self == {{member}}
    {% end %}
    value.to_s
  end
end

p MyEnum::Foo.to_s
p MyEnum::Bar.to_s

I'll close this. If someone comes up with an idea on how to reuse super here, please reopen.


Add jsonResponse

This helper is help to write easier json response

Hi @taylorotwell
Why this pull request closed?

because it makes no sense to shadow response()->json() as just a jsonResponse() function :)


Field with leading zeros being treated as a number rather than a string

I am using a String field to store Identification Numbers. When exporting the data to CSV, all leading zeros are removed. Is my String being treated as a Number during the conversion?
I cannot wrap the values in quotes as there are many other dependencies on these values in the system.
Can you suggest the best approach to this one?

I have the same issue as well. Please help with an example.


Autocomplete 'string' to 'String' after input dot or comma

VSCode Version: 1.40.2
OS Version: Windows_NT x64 10.0.18363

Steps to Reproduce:

Create a js file and try to add type for a variable(cause I use flow)
auto changed 'string' to 'String' after I input a comma or dot

I'm not sure this is the correct behavior, but in my view,  if I don't select any code snippet in the list,  VScode shouldn't change my code instead of selecting the first one because I entered a comma or dot

Does this issue occur when all extensions are disabled?: Yes

We don't support types in JS files natively. Please file this against the flow extension

@mjbvz But this bug is not caused by flow extension, I think the deep reason is why it autocomplete the option I didn't choose?

We don't support writing : types in JS files. If you write types, we cannot say that IntelliSense will do the right thing or will work at all . Please use the flow extension if you are writing flow code


Invoke-DbaDbGrow

Type of Change

[ ] Bug fix (non-breaking change, fixes #)
[x] New feature (non-breaking change, adds functionality)
[ ] Breaking change (effects multiple commands or functionality)
[ ] Ran manual Pester test and has passed (`.\tests\manual.pester.ps1)
[ ] Adding code coverage to existing functionality
[ ] Pester test is included
[ ] Nunit test is included
[ ] Documentation
[ ] Build system

Purpose
New functionality to help with proactive file growths
Approach
Since it is my first contribution to the dbatools I'd like to start (very) early with the review process and welcome comments, and suggestions how to shape this function. I think this PR will help to do so.
This function is part of the changes discussed here https://github.com/sqlcollaborative/dbatools/issues/3359
Generally the idea is to follow example of Invoke-DbaDatabaseShrink while implementing the growth functionality. At the same time I will try to keep an eye on the changes coming in as 1.0 and try to shape things in that direction.
Commands to test
Nothing yet
Screenshots
Not yet

I would recommend not having a PR sit around for getting each iteration of your code reviewed. It is best to create a branch under your fork and then have folks check it and/or discuss the plan in the issue, it keeps everything together.


correct cmdstanpy / cmdstanr functions

changes per issue #16

@jgabry - changed descriptions - is this better?


Add SniffResponseTesting to expose SniffResponse.AddressRe for unit t…

Added SniffResponseTesting just to expose SniffResponse.AddressRe so we don't have to adjust the visibility of SniffResponse.
Also looked at adding InternalsVisibleTo for the Test project to Elasticsearch.Net, but the strong name / signing prevented this.
fixes #2350
CLA signing is in progress

@awelburn thanks for submitting this, looks good other than my small suggestion.
Would you mind signing our CLA so that we are allowed to pull your bits in?
https://www.elastic.co/contributor-agreement

@awelburn thanks for the updated commits! I'm eager to pull this in but I can't find you on our CLA list.
Did you sign our https://www.elastic.co/contributor-agreement ?

@Mpdreamz It's with our CTO for signing, will update when there's progress.

@Mpdreamz we signed the contract a few minutes ago

Awesome, I can see it in our database. We'll pull this in before 5.0 GA hits the streets (this week or early next week).

Merged to 5.x and master.


Workspace - In workspace list,1 is displayed with space in android but not in mweb

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: 9.0.10
Reproducible in staging?: Y
Reproducible in production?: Y
Logs: https://stackoverflow.com/c/expensify/questions/4856
Issue reported by: Applause-Internal team
Action Performed:

Launch app in both mweb and android with same account using one after plus in email accounts (eg<EMAIL_ADDRESS>)
Tap profile -- workspaces -- workspace
Under Workspace name, note the number one format in the email id displayed

Expected Result:
In workspace list, number 1 should be displayed with same spacing in android & mweb.
Actual Result:
In workspace list, number 1 is displayed with space in android but not in mweb. Inconsistency in behaviour.
Workaround:
Unknown
Platforms:

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

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

Screenshots/Videos

https://github.com/user-attachments/assets/04464dcf-cfad-4469-a1f6-d689e8bbb930

View all open jobs on GitHub

We think this issue might be related to the  #vip-vsb

Proposal
Please re-state the problem that we are trying to solve in this issue.

In the workspace list, a workspace name containing "1" is displayed with extra space on Android but not on the mobile web (mWeb), indicating an inconsistency in behavior.

What is the root cause of that problem?

We are applying the labelStrong style to the workspace name as shown below:
To apply bold styling, we need to manage different styles according to the platform, which is already handled here

https://github.com/Expensify/App/blob/8e4dc0f032387c00e7ae4a51aff4792bdede5bfa/src/pages/workspace/WorkspacesListRow.tsx#L220-L225
https://github.com/Expensify/App/blob/8e4dc0f032387c00e7ae4a51aff4792bdede5bfa/src/styles/index.ts#L1370-L1375
What changes do you think we should make in order to solve the problem?

We need to update the style object as shown below to ensure consistency across the app and all platforms:

fontFamily: FontUtils.fontFamily.platform.EXP_NEUE_BOLD,
fontWeight: FontUtils.fontWeight.bold,

What alternative solutions did you explore? (Optional)
Result after changes

This seems like extreme polish and not critical given the roadmap priorities, closing.


Script Keylogger execute and store on sd card ?

Hello,
I would have liked to know if it was possible to store and execute the ps1 script of the keylogger on the sd card.
To be able to avoid downloading it from the internet.
Is it possible with this type of "Bad USB" in arduino?
Thank you

I don't think it's possible you will need to find a way to mount the sd card in your OS. I didn't find code for mounting and sd card in an arduino. If it existe please let me know.


play_html5.php and site_html5.php not working with .htaccess in place

Hi all
I know this dates back to a short period of time (between late version 2 and early version 2.1 I think) where we originally had play_html5.php and site_html5.php so isn't a widespread issue but I had a report that existing links accessed from a vle or blog etc were no longer working after an upgrade to version 3 and it turns out the links were quite old and all play_html5.php?template_id= etc.
I've been investigating and testing with one of my own installations and it seems that although we have a header redirect in play_html5.php which deals with this normally and redirects to play.php it doesn't work with Apache set to true and .htaccess in place to shorten the URL's. It looks like the existing .htaccess rules do try to work when following a link to play_html5.php?template_id= etc but the url in the address bar doesn't actually change and gives and error like:
Warning: file_get_contents(website_code/error_top): failed to open stream: No such file or directory in /var/www/xerte/play.php on line 144 Sorry this resource does not exist
Does this need a change to the .htaccess or is there another solution?
Ron

How widespread a problem is it do you think? Does it need fixing, or is it sufficiently low volume to patch the installs exhibiting the problem?

It's likely to be low volume but one of those where it might not crop up at the same time for everyone depending on use of _html5 links embedded in vle's etc.I couldn't work out what and how to fix it but it does seem like it ought to work with the right rules in .htaccess. e.g. the problem at the moment is that it works ok (redirects) without apache set to true but doesn't with apache set to true and an .htaccess in place.

no longer used


Scene activation - Italian

Hi -
can you confirm that activating scenes is working with Italian now? If so, what's the correct phrase?
I've got some users complaining that they don't work -
Thanks :)

I am unsure whether scenes are working in Italian specifically. Saying something like "Activate party mode" should cause the scene named "Party Mode" to activate.

I can confirm that scenes are working in english and are NOT working in italian. You'd better to convert  action.devices.types.SCENE into action.devices.types.LIGHT to use them in italian.

Right now scenes are not supported in Italian.

Thanks @Fleker  👍


All words are marked as unused

Seems like this command is the culprit:
https://github.com/Zamiell/cspell-check-unused-words/blob/a7652eaa46dd52a995f6fbbccef38bf977eb68d2/src/checkUnusedWords.ts#L116
Does this work for you? How? When I try to invoke cspell without any paths or globs it just prints help.

yeah it works for me because I use this in my cspell config: https://github.com/Zamiell/cspell-check-unused-words/blob/main/cspell.config.jsonc#L5

if you add that to your config does it work?

Ah! I assumed it was something like that, and looked for this option in CSpell's docs really hard, and it just wasn't there 😄 Yep, adding this helps, and also something I wanted to have regardless, thanks!

i guess its a bug in the checker though since it should handle that case

fixed by this commit: https://github.com/complete-ts/cspell-check-unused-words/commit/b30640fb3e4b7a8e8501a99adef9a3d3dfd4b6f7


Shield duration incorrect

Describe the bug
Shield is set to last 6 seconds, but per the spell description text it should last until the start of the caster's next turn. I have modified this in a custom copy.
Steps to fix the behavior:

Open Shield Active Effect.
Click Duration tab.
Clear Effect Duration (Seconds) field.
Click Special Duration dropdown list.
Select "Turn Start: Expires at the start of the character's next turn (in combat)."

Thanks!

I'm not sure I understand. The default duration is 6 seconds because that is the length of a turn. If you apply Shield outside of combat, it lasts 6 seconds and then expires (assuming times-up and dependencies). If you apply Shield during combat, it expires when it reaches that player's next turn. Again, times-up handles that logic of 6 seconds == until next turn.

I should have been more clear. When I use the default Shield Convenient Effect instead of my custom one, the AC bonus and the TokenMagic FX do not get removed until the end of the caster's turn, when they should be removed at the start of the caster's turn. Do I have something messed up?

I'm assuming something else is messed up. When I run it with the default one, it correctly expires at the start of the turn.

OK. Apologies for the trouble. I will play around with it more and see if I can isolate where mine is going is wrong. Thanks!

OK. Apologies for the trouble. I will play around with it more and see if I can isolate where mine is going is wrong. Thanks!

Do you have both the "About Time" and "Times Up" modules installed? That might be the issue.


[Feature] Would it be possible to implement an RBF NN in MLX

Could the MLX team consider implementing a native Radial Basis Function (RBF) layer in the MLX framework? An RBF layer would greatly enhance MLX's capabilities in function approximation and interpolation, particularly for applications requiring smooth, localized transformations of inputs. This addition would be beneficial for users seeking to integrate complex, non-linear mappings into their MLX-based machine learning models. This would be particularly useful in scientific computing for scenarios where the loss needs to be  based on data generated with external libraries that cannot be  used directly because autograd would not be able to compute the gradients.

Are you looking for something like the following or something more complicated?
import mlx.core as mx
import mlx.nn as nn

class RBFLayer(nn.Module):
    def __init__(self, input_dims, hidden_dims, output_dims, bias=True):
        super().__init__()

        # The initialization of the centers and betas depends on the input
        # distribution, but the following should be kind of fine for a standard
        # normal. Also the betas are going to be learnable this way, maybe it
        # is better to freeze them or make sure they are positive by
        # parameterizing them as logbetas.
        self.centers = mx.random.normal((hidden_dims, input_dims))
        self.betas = mx.full((hidden_dims,), 1/input_dims)
        self.projection = nn.Linear(hidden_dims, output_dims, bias=bias)

    def __call__(self, x):
        # Normalize the shape a bit
        shape = x.shape
        x = x.reshape(-1, shape[-1])

        # Pairwise distance
        x2 = 0.5 * x.square().sum(-1)
        c2 = 0.5 * self.centers.square().sum(-1)
        xc = x @ self.centers.T
        dist = x2[:, None] + c2[None, :] - xc

        # Compute the rbf
        # Here we could use mx.softmax(-dist, -1) for normalized rbf
        # or have fixed β_i etc.
        rbf = mx.exp(-(self.betas * dist))

        # Finally gather the outputs
        return self.projection(rbf).reshape(*shape[:-1], -1)

Hi Angele,
Thank you, this is a great. I will have to experiment with it and fine tune it a bit but it gives me a good starting point and I will reach out for your wisdom if I run into trouble. If I manage to get it working with a couple of examples I am trying right now, I will share it back including any adjustments.

I am closing the issue but feel free to reopen if something comes up.


feat: template

#__
Checklist

[ ] I didn't over-scope my PR
[ ] My PR title matches the commit convention
[ ] I did not include breaking changes
[ ] I included unit tests that cover my changes
[ ] I added/updated the documentation about my changes
[ ] I made my own code-review before requesting one

Description of the changes

Technical highlight/advice

You should add informations to the README.md just like in the Rust template.

I have this error when I try to use the template: [200]: POST /function/test - Uncaught Error: Failed opening required 'vendor/autoload.php' (include_path='.:/usr/local/lib/php') in /app/index.php:7

Merge this and edit the README to say the project is deprecated and archived to to use the FrankenPHP template


(Build) Fail on dev branch.

This issue is created automatically by CI.
Linux Build State: success
Windows Build State: failure
Commit: aaf8f8172502063a950c3478cea0431fcb8c1a5c
Commit Branch: dev
Commit Author: Monika0000

Auto-closing as build in this branch is successful now. Commit: 128424f0d930822465b9441ca21ee0a91260f918


Class 'App\Http\Controllers\Alert' not found

Class 'App\Http\Controllers\Alert' not found

Sorry forgot add 'use Alert;'

No problem! :)

not working in alert in redirect()->back();
because working in return view();
why?


Please add .gitattributes file with config line ending styles

https://git-scm.com/docs/gitattributes

Sorry can you clarify what you want me to set? Can you give me an example please?

read more https://help.github.com/articles/dealing-with-line-endings/
Since this is a completely windows project - need enforces CRLF line endings for all files.
Maybe samething like that:
* text eol=crlf

*.pdf binary
*.exe binary

Ah right! Good idea, thanks, I'll do that soon!


code . crashes if opened directly or from lxterminal

VSCode Version: 1.40-1470329130_amd64.deb
OS Version: Lubuntu / ubuntu trusty 14.04

Steps to Reproduce:

open vscode from gui

Or:

open lxterminal

type code .
/usr/bin/code: line 36: 26163 Aborted                 (core dumped) ATOM_SHELL_INTERNAL_RUN_AS_NODE=1 "$ELECTRON" "$CLI" "$@"

It only seems to open from xterm / uxterm via code .

I've seen this bug before on Linux.
Can you post the output of env in lxterminal? Or even better only the difference between its output in lxterminal vs xterm.

These are in xterm and missing from lxterm:
XTERM_SHELL=/bin/bash
WINDOWID=54525988
XTERM_LOCALE=en_US.UTF-8
XTERM_VERSION=XTerm(322)

Try setting each of these (using e. g. export XTERM_SHELL=/bin/bash) and see if that fixes the crash.

Still no luck.  Is there a way to make it emit a more specific exception message?

@benlowry first off please update VS Code so you're on the latest version.
After that it looks like it's failing on the following line: https://github.com/Microsoft/vscode/blob/5b3afc132ef5d50670c1c43a2b9650da532f20e8/resources/linux/bin/code.sh#L36
So you can try tweaking that command. Maybe that shell has issues with setting the environment variable at the beginning of a line?


Feat/perf test

Summary
Adds a performance testing tool. Three "drivers" are implemented
depending on use cases:

local driver: useful to compare between git commits, branches and releases, will run
Kong, postgres and upstream Nginx directly on the host machine. Uncommited changes
will be stashed and recovered after test run.
docker driver: useful to compare between docker image releases, will run all components
in docker containers. This driver doesn't support generating flamegraphs.
terraform driver: useful to get meaningful RPS number on given hardware, currently
it supports the "equinix-metal" provier only. It's possible to extend to other hosting
providers.

The testing tools can reuse the spec.helpers functions to setup Kong entities. It will then
runs wrk to measure latency and RPS with given configuration.
In this PR, two "purposes" has been added, spec/04-perf/01-rps is used for comparing
request per seconds, and spec/04-perf/02-framegraph is used for generating flamegraph.
To select a driver, pass it in the PERF_TEST_DRIVER env var:
PERF_TEST_DRIVER=terraform bin/busted -o gtest spec/04-perf/01-rps/01-baseline_spec.lua -t single_route

Full changelog

Add a performance testing tool.

/perf aaa

/perf aaa


boards: usb: Add USB support for nRF5340dk

This PR adds USB support for nRF5340dk platform.
Did some manual testing of the samples and performed USB3CV tests.
Some will fail due to other bugs in the tree: #27025 (hid-cdc/cdc_acm_composite affected).
Mass storage sample could use QSPI to access the external, on board, flash.
I don't have high need to do so but if anyone insists I am okay with adding those.
@pabigot do You think this should be there as well?

@nordic-krch I took some of Your commits You shared with me offline thus I added Your sign off, please feel free to comment.

@cvinayak could you make sure the clock control changes do not negatively affect Bluetooth?

Mass storage sample could use QSPI to access the external, on board, flash.
I don't have high need to do so but if anyone insists I am okay with adding those.
@pabigot do You think this should be there as well?

According to @anangl we need an updated nrfx for this, since the frequencies are different from the ones in nRF52840

@cvinayak could you make sure the clock control changes do not negatively affect Bluetooth?

I get runtime error on nrf52dk_nrf52832 (BT_HCI_ERR_HW_FAILURE). Waiting for LFCLOCK to be running or to be started here: https://github.com/emob-nordic/zephyr/blob/usb_nrf53_support/subsys/bluetooth/controller/ll_sw/ull_adv.c#L817
*** Booting Zephyr OS build zephyr-v2.3.0-1362-gd6aa0aff0d14  ***
Bluetooth initialized
[00:00:00.007,843] <inf> fs_nvs: 6 Sectors of 4096 bytes
[00:00:00.007,843] <inf> fs_nvs: alloc wra: 0, fc0
[00:00:00.007,843] <inf> fs_nvs: data wra: 0, 4c
[00:00:00.009,307] <inf> bt_hci_core: HW Platform: Nordic Semiconductor (0x0002)
[00:00:00.009,307] <inf> bt_hci_core: HW Variant: nRF52x (0x0002)
[00:00:00.009,338] <inf> bt_hci_core: Firmware: Standard Bluetooth controller (0x00) V3Build 99
[00:00:00.009,613] <inf> bt_hci_core: No ID address. App must call settings_load()
[00:00:00.012,207] <inf> bt_hci_core: Identity: c1:ab:66:13:11:2e (random)
[00:00:00.012,207] <inf> bt_hci_core: HCI: version 5.2 (0x0b) revision 0x0000, manufacturer 0x05f1
[00:00:00.012,207] <inf> bt_hci_core: LMP: version 5.2 (0x0b) subver 0xffff
Advertising failed to start (err -5)
[00:00:00.517,211] <wrn> bt_hci_core: opcode 0x200a status 0x03
[00:00:00.517,242] <err> bt_hci_core: Failed to start advertiser

I get error building for nrf51dk_nrf51422.
[jack@win-vm-archlinux peripheral]$ cmake -GNinja -DBOARD=nrf51dk_nrf51422 -DCONFIG_BT_CTLR_DEBUG_PINS=y ../../samples/bluetooth/peripheral
Including boilerplate (Zephyr base): /home/jack/workspace/west_zephyr/zephyr/cmake/app/boilerplate.cmake
-- Application: /home/jack/workspace/west_zephyr/zephyr/samples/bluetooth/peripheral
-- Zephyr version: 2.3.99 (/home/jack/workspace/west_zephyr/zephyr)
-- Found Python3: /usr/bin/python3.8 (found suitable exact version "3.8.2") found components: Interpreter
-- Board: nrf51dk_nrf51422
-- Found west: /home/jack/.local/bin/west (found suitable version "0.7.2", minimum required is "0.7.1")
-- Found dtc: /usr/bin/dtc (found suitable version "1.6.0", minimum required is "1.4.6")
-- Found toolchain: gnuarmemb (/opt/gcc-arm-none-eabi-9-2019-q4-major)
-- Found BOARD.dts: /home/jack/workspace/west_zephyr/zephyr/boards/arm/nrf51dk_nrf51422/nrf51dk_nrf51422.dts
-- Generated zephyr.dts: /home/jack/workspace/west_zephyr/zephyr/build/peripheral/zephyr/zephyr.dts
-- Generated devicetree_unfixed.h: /home/jack/workspace/west_zephyr/zephyr/build/peripheral/zephyr/include/generated/devicetree_unfixed.h

warning: NRFX_CLOCK (defined at modules/Kconfig.nordic:76) has direct dependencies HAS_HW_NRF_CLOCK && HAS_NRFX with value n, but is currently being y-selected by the following symbols:
 - CLOCK_CONTROL_NRF (defined at drivers/clock_control/Kconfig.nrf:13), with value y, direct dependencies SOC_COMPATIBLE_NRF && !CLOCK_CONTROL_NRF_FORCE_ALT && CLOCK_CONTROL (value: y), and select condition SOC_COMPATIBLE_NRF && !CLOCK_CONTROL_NRF_FORCE_ALT && CLOCK_CONTROL (value: y)

warning: NRFX_POWER (defined at modules/Kconfig.nordic:152) has direct dependencies HAS_HW_NRF_POWER && HAS_NRFX with value n, but is currently being y-selected by the following symbols:
 - CLOCK_CONTROL_NRF (defined at drivers/clock_control/Kconfig.nrf:13), with value y, direct dependencies SOC_COMPATIBLE_NRF && !CLOCK_CONTROL_NRF_FORCE_ALT && CLOCK_CONTROL (value: y), and select condition SOC_COMPATIBLE_NRF && !CLOCK_CONTROL_NRF_FORCE_ALT && CLOCK_CONTROL (value: y)
Parsing /home/jack/workspace/west_zephyr/zephyr/Kconfig
Loaded configuration '/home/jack/workspace/west_zephyr/zephyr/boards/arm/nrf51dk_nrf51422/nrf51dk_nrf51422_defconfig'
Merged configuration '/home/jack/workspace/west_zephyr/zephyr/samples/bluetooth/peripheral/prj.conf'
Merged configuration '/home/jack/workspace/west_zephyr/zephyr/build/peripheral/zephyr/misc/generated/extra_kconfig_options.conf'

error: Aborting due to Kconfig warnings

CMake Error at /home/jack/workspace/west_zephyr/zephyr/cmake/kconfig.cmake:216 (message):
  command failed with return code: 1
Call Stack (most recent call first):
  /home/jack/workspace/west_zephyr/zephyr/cmake/app/boilerplate.cmake:511 (include)
  /home/jack/workspace/west_zephyr/zephyr/share/zephyr-package/cmake/ZephyrConfig.cmake:24 (include)
  /home/jack/workspace/west_zephyr/zephyr/share/zephyr-package/cmake/ZephyrConfig.cmake:35 (include_boilerplate)
  CMakeLists.txt:4 (find_package)

-- Configuring incomplete, errors occurred!

you are missing using nrfx_clock in couple of places (bluetooth issues reported by @cvinayak are caused by that). You can apply this: https://github.com/nordic-krch/zephyr/commit/ed56126a592b1238f948152c2859ef5ba1994115

since LF clk configuration will be in nrfx_config.h definition of #define CLOCK_CONTROL_NRF_K32SRC in nrf_clock_control.h should be removed. It would be used only in lfclk_spinwait(CLOCK_CONTROL_NRF_K32SRC) which can be replaced with something like lfclk_spinwait(nrf_clock_lf_src_get(NRF_CLOCK))

By this PR I stuck my nose into other people's affairs (clock_control). I wanted to speed up nRF53 USB support but failed significantly. Agreed offline with @nordic-krch to close this one and introduce the support step by step. @nordic-krch will post clock_control changes on which I will base USB.


update mobile layout reports page

small updates for mobile layout reports page (making the grid stackable and reorder on mobile)

Will check it out tomorrow. Thank you for the help!

updated the save report form as well

Thanks! Did you close this PR by accident perhaps?

yes sorry!

Thank you for these updates. I'll add some work to it and merge.


How can i recalculate layout after some item width was changed?

I noticed that layout recalculates on window resize. How we can trigger this?

You can call isotopeLayout on ViewModel to trigger an update after element size change. The read.me document gives more details on how to do it.


[table] adding new column definitions after initial rendering fails when hasResize is true

Describe the bug
Adding new column definitions after initial rendering fails when hasResize is true. This issue was identified in the discussion
https://github.com/carbon-design-system/carbon-addons-iot-react/issues/1358
To Reproduce
Steps to reproduce the behavior:

Manually add a new column after the initial rendering. I don't think we currently have a story for this
but story code below can be used.
Click on 'Add column'
Notice how the new column header is missing

Expected behavior
The new column header should be showing and resizing should work. It should also work with adding two new or more new column at a time
Environment/versions:

OS: [e.g. MacOS, Windows]
Browser: [e.g. chrome, safari]
carbon-addons-iot-react version: [e.g. v2.60.0]

Additional context
Story example code:
  .add(
    'with resize, onColumnResize callback and no initial column width',
    () => {
      return React.createElement(() => {
        const modifiedColumns = [
          ...tableColumns,
          {
            id: 'newField',
            name: 'Brand new field',
          },
        ];

        const modifiedOrdering = [
          ...defaultOrdering,
          {
            columnId: 'newField',
            isHidden: false,
          },
        ];

        const [myColumns, setMyColumns] = useState(tableColumns);
        const [myOrdering, setMyOrdering] = useState(defaultOrdering);

        const modify = () => {
          setMyColumns(modifiedColumns);
          setMyOrdering(modifiedOrdering);
        };

        const onColumnResize = cols => setMyColumns(cols);
        return (
          <>
            <button onClick={modify}>Add column</button>
            <Table
              options={{
                hasResize: true,
                wrapCellText: select('wrapCellText', selectTextWrapping, 'always'),
              }}
              columns={myColumns}
              view={{
                filters: [],
                table: {
                  ordering: myOrdering,
                },
              }}
              data={tableData}
              actions={{ ...actions, table: { ...actions.table, onColumnResize } }}
            />
          </>
        );
      });
    }
  )

Specific timeline issues / requests
Do you want this work within a specific time period? Is it related to an
upcoming release?
NB: The core contributors will try to work with your timeline, but it's not
guaranteed. The earlier you make a request in advance of a desired delivery
date, the better!

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add package priority

Ideally we would mark things such as qt5 so that they get built first in each run, and if not possible then skipped.

qt5 is now split up and this hasn't been such an issue lately, so closing for now.


[Shortcut Guide] Show All Application Shortcuts

A hot key that can show all the applications shortcuts, so it makes it easier to remember them. E.g. you press that and it shows this window.

It should be able to find all the application's shortcuts which is in focus and list them.
ergonis KeyCue  is a product on a Mac that does this

Again I shouldn’t have to say it but... it should be searchable. And useable with mouse.

neat idea, so many questions ... some stuff comes from menu bars, context menus, things that are contextual for what you're doing.
I'm just thinking about how office's ribbon menu changes based on context.
I think we'd be able to scan apps but i doubt we'd be perfect.  Would love to nerd out with Ergonis to see how they accomplish this on MacOS

Let’s get it to show static shortcuts first then think about anything contextual.
Cheers
Chandan
On Dec 7, 2019, 00:36 +0000, Clint Rutkas<EMAIL_ADDRESS>wrote:
neat idea, so many questions ... some stuff comes from menu bars, context menus, things that are contextual for what you're doing.
I'm just thinking about how office's ribbon menu changes based on context.
I think we'd be able to scan apps but i doubt we'd be perfect. Would love to nerd out with Ergonis to see how they accomplish this on MacOS
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/PowerToys/issues/129?email_source=notifications&email_token=ACZPVBL5SEIVUJXZ5TG2NT3QXLV2HA5CNFSM4HQOGVJ2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEGFYWGY#issuecomment-562793243, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ACZPVBMG5P6WV7EUKPKPBBLQXLV2HANCNFSM4HQOGVJQ.

Let’s get it to show static shortcuts first then think about anything contextual.

neat idea, so many questions ... some stuff comes from menu bars, context menus, things that are contextual for what you're doing.
I'm just thinking about how office's ribbon menu changes based on context.
I think we'd be able to scan apps but i doubt we'd be perfect. Would love to nerd out with Ergonis to see how they accomplish this on MacOS

It would be just awesome if this turned into anything like KeyCue, which is just a life saver on a macOS computer that I've always wished were available on a Windows one!!!
By the way, KeyCue also allows you to customize the list, e.g. you can hide Ctrl+S shortcut as you'll never need a reminder that it's "Save" so there is not need for it to clutter your list or you can manually add shortcuts for applications that are not recognized by KeyCue or if you've remapped them via a different app (hopefully you'd be able to automatically import the remappings from PowerToys Keyboard Manager, but meanwhile for Autohotkey users this would also be a nice feature)
Thanks for the desire to nerd out with the developers of such a great app :)!

From @tarik02 and issue #4439

Proposed technical implementation details
I'm not an expert in WinAPI, but searching through internet gives some ideas about possible implementation details. I've discovered the following links:
https://docs.microsoft.com/en-us/windows/win32/api/winuser/nf-winuser-getforegroundwindow
https://docs.microsoft.com/en-us/windows/win32/api/winuser/nf-winuser-getmenu
https://stackoverflow.com/a/18590728/7519762
https://stackoverflow.com/questions/2862883/is-it-possible-to-enumerate-menu-items-of-other-programs-on-windows
So, we can capture current foreground window when opening PowerToys Run window, get its menu and use it in search results.

Nice to see all other issues closed, referring to this one... but the issue itself remains unchanged for quite a while and only seems to get more and more complicated.
Idea to dumb-down the issue for a minimum viable fix? (E.g, just include the powertoys shortcuts in current view and add extra functionality in a later stage)


Feature/behavior

Action Decider that processes recognized spoken commands and publishes to topics accordingly:
flowchart TD
    A(fa:fa-microphone Voice Recognition) --> |voice_recognition_node/recognized_command| B{fa:fa-brain Behavior}
    E(fa:fa-eye Vision) --> |vision_node/recognized_sentence| B
    B --> |behavior_node/read_sentence| G(MIDI Output)
    B --> |behavior_node/play_sentence| C(fa:fa-robot Motion Planning)
    B --> |behavior_node/play_sentence| F(Audio Feedback)
    B --> |behavior_node/response| D(fa:fa-comment Speech Output)

voice_recognition_node/recognized_command : String
-> The spoken command recognized by the voice recognition module
vision_node/recognized_sentence : String
-> The note sequence recognized by the vision module
behavior_node/read_sentence : String
-> The (latest) note sequence recognized by the vision module after the "read" command was issued
behavior_node/play_sentence : String
-> The (latest) read note sequence after the "play" command was issued. If the "stop" command was issued, publishes empty string
behavior_node/response : String
-> Response text for synthesized speech

changed base branch due to node refactoring in https://github.com/UHHRobotics22-23/marimbabot/pull/72


List all options in sqitch-rework.pod

So command line help matches longer docs in terms
of at least listing all the switches.

Changes Unknown when pulling f0c07b3128568e265279d50eebd2dbd260150ad0 on jhannah:master into ** on theory:master**.


[feature]: add support for worker timeout and failOnTimeout

Feature request
Add support for custom worker timeout and job failOnTimeout property.

#2


Visual Mappings don't update on first try

Via @AlexanderPico:

Very odd behavior. In order to get a new visual mapping to stick, I have to run the PUT method twice. See wrapper function:
https://github.com/cytoscape/cytoscape-automation/blob/master/for-scripters/R/r2cytoscape/R/updateStyleMapping.R#L37
When you run this PUT once, you get no effect or some "odd" effect (like clearing a style mapping), then the second application fixes it. I think you can continue to get this effect with every even application fixing the effect of odd applications. WEIRD.

Behavior is confirmed and definitely weird. To replicate, load galFiltered.cys, switch to the Metallic style, the start making a POST requests to /v1/styles/Metallic/mappings with JSON like the following:
[{

    "mappingType": "passthrough",
    "mappingColumn": "COMMON",
    "mappingColumnType": "String",
    "visualProperty": "NODE_LABEL"

}]

Change the mappingColumn field between calls; sometimes the mapping is recognized, other times it is not applied, or disappears altogether.

Addressed in 7a8a369


Error connecting

After I pasted in my token from Trello, I got this error:
connected as aharris88
TypeError: Cannot read property 'toString' of undefined
    at module.exports (/usr/local/lib/node_modules/trelew/node_modules/pad/lib/index.js:22:14)
    at /usr/local/lib/node_modules/trelew/helpers.js:101:71
    at Array.map (native)
    at CommandInstance.listBoards (/usr/local/lib/node_modules/trelew/helpers.js:97:56)
    at /usr/local/lib/node_modules/trelew/index.js:410:24
    at process._tickCallback (node.js:382:9)

You don't have a board with a blank name, do you?

No, I don't.


Added spiral matrix pattern

This is my first PR, Hope you accept it !! Thanks for this opportunity :)


Update windows-whatsnew.md

@denisebmsft please review and approve

Learn Build status updates of commit a219ca9:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-365/security/defender-endpoint/windows-whatsnew.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Learn Build status updates of commit 0373683:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-365/security/defender-endpoint/windows-whatsnew.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Learn Build status updates of commit 0358589:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-365/security/defender-endpoint/windows-whatsnew.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel


Macro: Update attachment points selection dialog for unknown chems

Background
User should be able to establish bonds with monomers with unknown structure
AP - attachment point
Requirements

Modal window for AP selection should be displayed anytime when user establishes bond with unknown monomer.
This window should allow to select appropriate attachment point, but no structure should be displayed for unknown monomer (see mockups).

UX
TBD

Merged with https://github.com/epam/ketcher/issues/4431


Crash when using with Google Play Services 6.5

The issue is caused by the deprecation of the LocationClient in favour of the GoogleApiClient.
https://developer.android.com/google/play-services/index.html
Updated the code to use the new client should fix any issues.

Updates available on develop but untested. Release to come.

This should now be fixed in the 2.2.5 release


Question: new theming approach

Hello folks!
I am reaching out to ask you if there would be a possibility to see a spoiler of how the new theming approach will be?
The reason is that in my team we are starting to build a component library based on prime vue.
We are literally just starting, and in the phase of understanding how the sass-theme structure works, and configuring the tokens etc.
I just watched this video: https://youtu.be/iZp_l0Yl3Zw?t=829
Where you explains high level that the new theming will use only css variables, and be backwards compatible.
As we are just starting, we could perhaps start with the "new approach" or have a temporal thing that allow us to migrate to the new way sooner.  And that is the reason why I am asking if there was some doc, diagram, or a bit more insights about how the new theming will work?
Could be a nice opportunity to get early feedback from early adopters :)
Thank you for your work on this!
Daniela

Sorry for the late reply, we missed the tickets here as team is focused on the core issue tracker. The new styled mode is similar to unstyled mode, the POC is done and the implementation starts in a week or two. We'll announce it soon with more details.


Unable to import shape file bigger than 100Mb

The admin4example user has a quota of 100Mb and a max import size of 150Mb.


⚠ Bofrost.nl has degraded performance

In 4901f28, Bofrost.nl (https://www.bofrost.nl) experienced degraded performance:

HTTP code: 200
Response time: 1173 ms

Resolved: Bofrost.nl performance has improved in 2daee60.


updated StoryRouter to use recommended data router

Attempt to fix issues related to new data routers: https://github.com/JesusTheHun/storybook-addon-react-router-v6/issues/19

@JesusTheHun Can you merge this in? 🙏

@afn No. This PR breaks many things and does not port the current features to the new router. I'm working on a proper implementation.

Ah okay. Meanwhile if anyone wants to use the forked package:
yarn add -D storybook-addon-react-router-v6@cygnuseducation/storybook-addon-react-router-v6

(I had to add a prepare script to allow the package to be installed from git — see 0130cd4455d0c2c714f1b20e0b31022c2fb9254f)

I'm closing this as my implementation is now complete.


DOC: Docstring examples for app.translate, app.align, and app.dist

Added docstring examples for the apps used in the wrangling paper.
app.translate:

select_translatable
translate_seqs

app.align:

progressive_align
ic_score
cogent3_score
sp_score

app.dist:

jaccard_dist
approx_pdist
approx_jc69

#1636

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
29762

Relevant Lines:
32443

💛  - Coveralls


[WIP] [refactor] Move logger.finalize to Trainer's teardown call

What does this PR do?

This is currently only being called at the end of the fit loop. However, this would be useful in validate/test/ predict too.
As a result, can we move this to the Trainer's teardown to centralize this?
pros:

less loop-specific handling (we might forget to do this)

cons:

?

Not yet ready for review: checking CI
Before submitting

[ ] Was this discussed/approved via a GitHub issue? (not for typos and docs)
[ ] Did you read the contributor guideline, Pull Request section?
[ ] Did you make sure your PR does only one thing, instead of bundling different changes together?
[ ] Did you make sure to update the documentation with your changes? (if necessary)
[ ] Did you write any new necessary tests? (not for typos and docs)
[ ] Did you verify new and existing tests pass locally with your changes?
[ ] Did you update the CHANGELOG? (not for typos, docs, test updates, or internal minor changes/refactorings)

PR review
Anyone in the community is free to review the PR once the tests have passed.
Before you start reviewing make sure you have read Review guidelines. In short, see the following bullet-list:

[ ] Is this pull request ready for review? (if not, please submit in draft mode)
[ ] Check that all items from Before submitting are resolved
[ ] Make sure the title is self-explanatory and the description concisely explains the PR
[ ] Add labels and milestones (and optionally projects) to the PR so it can be classified

Did you have fun?
Make sure you had fun coding 🙃

Hey! I like the idea and for single processes this works great.
However the concern for ddp spawn is the following:
on_run_end(), where finalize() was originally, happens inside the spawned process where the logger is active.
Now you moved it the call to after dispatch which means that this finalize() will happen in the master process where no training/logging ever happened. I need to find a better place where to move this.

@awaelchli I see, so then the question is:

Do we repeat this across loops, similar to how profiler.describe is handled today? However, as seen here, we can forget this in loop implementations, and with fit and evaluation loops, we need to add a check for whether we're in trainer.fit vs trainer.test/trainer.validate
Or do we special case the spawn plugins and handle finalization/teardown in post_dispatch?

Neither sounds too great :/

@awaelchli n00b question: would calling trainer._call_teardown_hook here work? https://github.com/PyTorchLightning/pytorch-lightning/blob/1afc1ca7ef8d069895f4065fb4000b24771585f4/pytorch_lightning/plugins/training_type/ddp_spawn.py#L205-L208

This was taken care of in https://github.com/PyTorchLightning/pytorch-lightning/pull/8685


Output files open path error (%5C instead of backslash issue)

Description

Reproduce

Go create some job
Download output file (Notebook/ HTML)
Found Unhandled error

Expected behavior
No Unhandled error popup
Context

Operating System and version: Window10
Browser and version: Chrome Version 118.0.5993.71 (Official Build) (64-bit)
Jupyter Server version: Jupyter lab version 4.0.8

Browser Output

Paste the output from your browser Javascript console here, if applicable.
![error](https://github.com/jupyter-server/jupyter-scheduler/assets/13539556/52fb9603-8f0b-49c7-bacb-e590b0d3fe64)

Hi @snn2spade. Thank you for opening this issue and reporting this problem. I apologize for the delay in response. I can reliably reproduce this bug on Windows 11 (when not using Windows Subsystem for Linux).

same here.


updated packages

Changes
So I updated A LOT of packages. I originally tried to do them one by one starting with Gatsby, but updating Gatsby required me to update a lot of other packages as well.
I guess this is the full list:
@material-ui/core               4.11.0  4.12.3  4.12.3 dependencies    https://material-ui.com/

@material-ui/icons              4.9.1   4.11.2  4.11.2 dependencies    https://github.com/mui-org/material-ui/tree/master/packages/material-ui-icons
@svgr/webpack                   5.3.0   5.5.0   5.5.0  dependencies    https://react-svgr.com

classnames                      2.2.6   2.3.1   2.3.1  dependencies    https://github.com/JedWatson/classnames#readme

dayjs                           1.10.4  1.10.7  1.10.7 dependencies    https://day.js.org

eslint                          6.8.0   6.8.0   8.1.0  devDependencies https://eslint.org

eslint-config-standard          14.1.1  14.1.1  16.0.3 devDependencies https://github.com/standard/eslint-config-standard

eslint-plugin-import            2.20.2  2.25.2  2.25.2 devDependencies https://github.com/import-js/eslint-plugin-import

eslint-plugin-promise           4.2.1   4.3.1   5.1.1  devDependencies https://github.com/xjamundx/eslint-plugin-promise

eslint-plugin-react             7.19.0  7.26.1  7.26.1 devDependencies https://github.com/yannickcr/eslint-plugin-react

eslint-plugin-react-hooks       3.0.0   3.0.0   4.2.0  devDependencies https://reactjs.org/

eslint-plugin-standard          4.0.1   4.1.0   5.0.0  devDependencies https://github.com/standard/eslint-plugin-standard#readme
gatsby-image                    2.3.1   2.11.0  3.11.0 dependencies    https://github.com/gatsbyjs/gatsby/tree/master/packages/gatsby-image#readme
gatsby-plugin-manifest          2.3.3   2.12.1  4.0.0  dependencies    https://github.com/gatsbyjs/gatsby/tree/master/packages/gatsby-plugin-manifest#readme
gatsby-plugin-material-ui       2.1.10  2.1.10  4.0.3  dependencies    https://github.com/hupe1980/gatsby-plugin-material-ui#readme
gatsby-plugin-react-helmet      3.2.1   3.10.0  5.0.0  dependencies    https://github.com/gatsbyjs/gatsby/tree/master/packages/gatsby-plugin-react-helmet#readme
gatsby-plugin-sharp             2.5.3   2.14.4  4.0.1  dependencies    https://github.com/gatsbyjs/gatsby/tree/master/packages/gatsby-plugin-sharp#readme
gatsby-remark-autolink-headers  2.2.1   2.11.0  5.0.0  dependencies    https://github.com/gatsbyjs/gatsby/tree/master/packages/gatsby-remark-autolink-headers#readme
gatsby-remark-copy-linked-files 2.2.1   2.10.0  5.0.0  dependencies    https://github.com/gatsbyjs/gatsby/tree/master/packages/gatsby-remark-copy-linked-files#readme
gatsby-remark-images            3.2.1   3.11.1  6.0.0  dependencies    https://github.com/gatsbyjs/gatsby/tree/master/packages/gatsby-remark-images#readme
gatsby-remark-prismjs           3.4.1   3.13.0  6.0.0  dependencies    https://github.com/gatsbyjs/gatsby/tree/master/packages/gatsby-remark-prismjs#readme
gatsby-source-filesystem        2.2.2   2.11.1  4.0.0  dependencies    https://github.com/gatsbyjs/gatsby/tree/master/packages/gatsby-source-filesystem#readme
gatsby-transformer-remark       2.7.1   2.16.1  5.0.0  dependencies    https://github.com/gatsbyjs/gatsby/tree/master/packages/gatsby-transformer-remark#readme
gatsby-transformer-sharp        2.4.3   2.12.1  4.0.0  dependencies    https://github.com/gatsbyjs/gatsby/tree/master/packages/gatsby-transformer-sharp#readme
gatsby-transformer-yaml         2.4.10  2.11.0  4.0.0  dependencies    https://github.com/gatsbyjs/gatsby/tree/master/packages/gatsby-transformer-yaml#readme
gh-pages                        2.2.0   2.2.0   3.2.3  devDependencies https://github.com/tschaub/gh-pages

husky                           4.2.3   4.3.8   7.0.4  devDependencies https://typicode.github.io/husky

prettier                        2.0.2   2.4.1   2.4.1  devDependencies https://prettier.io

react                           16.13.1 16.14.0 17.0.2 dependencies    https://reactjs.org/

react-dom                       16.13.1 16.14.0 17.0.2 dependencies    https://reactjs.org/

react-helmet                    5.2.1   5.2.1   6.1.0  dependencies    https://github.com/nfl/react-helmet#readme

react-slick                     0.27.10 0.27.14 0.28.1 dependencies    https://react-slick.neostack.com

sharp                           0.25.2  0.25.4  0.29.2 dependencies    https://github.com/lovell/sharp

svgo                            1.3.2   1.3.2   2.7.0  dependencies    https://github.com/svg/svgo

svgo-loader                     2.2.1   2.2.2   3.0.0  dependencies    https://github.com/pozadi/svgo-loader

The upgrades of some of these packages have also required me to manually install some new packages:

@emotion/react
path-browserify -> this also had to be added to webpack as a fallback due to polyfill no longer being supported??

In particular, husky, webpack, and eslint updates gave me a lot of headaches - and I'm not sure if I have resolved all of the issues.
Currently, I am experiencing some really bad latency issues whenever I click a button to change pages, and I wanted to make sure if this will be an issue when the website is built. So more investigation needs to be done on this front too.
This was my first time dealing with a lot of these libraries, so any help is appreciated. (Help me senpai)
Oh, and I also fixed some bugs that I did not manage to catch in #262
Type of change
Please delete options that are not relevant.

[ ] Content Update (non-breaking change which updates the content of the website)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Related Issues
#203

Ok yeah something is very wrong with this. Will need to investigate.

Ok, although this current version doesn't seem break anything in the website, the build logs (from netlify deploy-preview) still has some warnings that I am not sure if we should address.
 warning " ><EMAIL_ADDRESS>has incorrect peer dependency "eslint@^3 \|\| ^4 \|\| ^5 \|\| ^6 \|\| ^7".
--
  | warning " ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>\|\| ^4.0.0 \|\| ^5.0.0 \|\| ^6.0.0 \|\| ^7.0.0".
  | warning "gatsby > @typescript-eslint/eslint-plugin ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>\|\| >= 3.2.0-dev \|\| >= 3.3.0-dev \|\| >= 3.4.0-dev \|\| >= 3.5.0-dev \|\| >= 3.6.0-dev \|\| >= 3.6.0-beta \|\| >= 3.7.0-dev \|\| >= 3.7.0-beta".
  | warning "gatsby > eslint-plugin-graphql > graphql-config > @endemolshinegroup/cosmiconfig-typescript-loader ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>  | warning " ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>  | warning " ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>  | warning "gatsby-plugin-material-ui ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>  | warning "gatsby-plugin-material-ui ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>  | warning " ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>  | warning " ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>  | warning " ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>

I get this error when I try to build it locally after running yarn install.
We should also probably upgrade the material-ui dependencies since the it seems our versions conflict.
Also take a look at this blog post about upgrading gatsby from v3 to v4. Looks like a lot of breaking changes were made.

After running a codemod for the new gatsby-image api, got these messages. Will be addressing them. Am also going to paste the messages here to document it somewhere.
It appears you're passing a variable to your image component. We haven't changed it, but we have updated it to use the new GatsbyImage component. Please check src\components\PhotoPage\Cropper.js manually.
It appears you're passing a variable to your image component. We haven't changed it, but we have updated it to use the new GatsbyImage component. Please check src\components\PhotoPage\Cropper.js manually.
It appears you're passing src or srcSet directly. This API is no longer exposed, please update src\components\SvgImg.js manually.
It appears you are referencing the image component in some way. We've updated the reference, but you will want to verify src\components\EventCard\EventCard.js manually.
maxWidth is no longer supported for fluid (now fullWidth) images. We've updated the query in src\components\EventCard\EventCard.js to use a constrained image instead.
maxHeight is no longer supported for fluid (now fullWidth) images. It's been removed from your query in src\components\EventCard\EventCard.js.
It appears you're passing src or srcSet directly. This API is no longer exposed, please update src\components\HomePage\Carousel.js manually.

The SvgImg component is now no longer being used. (it can no longer be used because of API changes). The new way I tried to render svg's is using the StaticImage component, which doesn't render the svg's at the best quality. The homepage banner got  a decrease in image quality while all the other images got an increase in image quality.


Make show return "toList [1, 2, 3]" instead of "(Cons 1 (Cons 2 (Cons 3 Nil)))"

I think it would be beneficial to make show return toList [1, 2, 3] instead of (Cons 1 (Cons 2 (Cons 3 Nil))), as that's much more readable.

I agree.

:+1:


flang-activation v19.1.3

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


[Launcher] Something is putting focus on the listview items.

Something is putting focus on the listview items.  Should be easy to fix, but we need to identify a clear repro to say it's fixed.

@ryanbodrug-microsoft I don't know the details of the issue you mentioned here. Please add some more background info.

@ryanbodrug-microsoft is this still repro? We need repro steps/guidance

I dont' have repro steps.  Somehow the results list gets put into focus.  A boarder appears around the listviewitem, and you can no longer type into the textbox.   Udit reported seeing this in the bug bash as well.

This still exists for the mix UWP/WPF mode.  We should wait until we do the full WPF flip

I found a repro.  Right-Click and the press spacebar.

We just released 0.18.1 which will have this fix included in it.


Nothing gets sent, no errors

I have API keys, SPF and DKIM set up, have SparkPost 1.0.7 package from NuGet and I'm using this code to send mail:
            var transmission = new Transmission();
            transmission.Content.From.Email =<EMAIL_ADDRESS>            transmission.Content.From.Name = "xyzInquiry";
            transmission.Content.Subject = "xyzwebsite inquiry form";
            transmission.Content.Text = textContent;
            transmission.Content.Html = htmlContent;
            transmission.Content.ReplyTo = inquiry.Email;

            var recipient = new Recipient
            {
                Address = new Address { Email = CloudConfigurationManager.GetSetting("ContactFormMailbox") }
            };
            transmission.Recipients.Add(recipient);
            var sparkPost = new Client(CloudConfigurationManager.GetSetting("SparkPostApiKey"));
            var result = sparkPost.Transmissions.Send(transmission);

When I send mail, I receive no error yet nothing gets sent. Doing a breakpoint for result variable suggests the following:

Id = 27, Status = RanToCompletion

However I get this when I intentionally throw an exception with the result variable, underneath the send line:

Id = 31, Status = WaitingForActivation, Method = "{null}", Result = "{Not yet computed}" System.Threading.Tasks.Task<SparkPost.SendTransmissionResponse>

Hello @wastedige
Can you add the line at the bottom of this, and see if that helps?
var result = sparkPost.Transmissions.Send(transmission);
result.Wait();

I wonder if the Wait() method call is enough?  I just played with it a little bit in a console app, and it only seemed to send when I put that call in at the bottom.

Is this async code? If it is you probably need an async method modifier. Also if you are in a MVC or Web API controller you may need to inherit from AsyncController.

This is async code, it's using the standard HttpClient under the hood.  No fanciness, it's just making http calls.
I don't think you need the async modifier if you call Wait().

I have the same problem. Just installed the Nuget package and created a test method, but nothing seems to be sent, but no errors are received.  Here's my code:
`
public static void SendViaSparkPost()
{
var transmission = new Transmission();
transmission.Content.From.Email =<EMAIL_ADDRESS>transmission.Content.Subject = "Oh hey!";
transmission.Content.Text = "Testing SparkPost - the world's most awesomest email service!";
transmission.Content.Html = "test content";
        var recipient = new Recipient
        {
            Address = new Address { Email =<EMAIL_ADDRESS>}
        };
        transmission.Recipients.Add(recipient);

        var client = new Client("SparkPostAPIKey".AppSetting());

        try
        {
            client.Transmissions.Send(transmission);

        }
        catch (Exception ex)
        {
            Console.Write(ex.Message);
        }

}`
If you've got any ideas - I'd love to hear them!

@nickc73 Did you try the suggestion above? To either tack on .Wait() or check for async?
The library returns a Task, not the final result. The code still has to be executed, this is the way the .Net Async works.

Ah, I had tried it - and it still didn't work - until I double checked my reports on SparkPost and spotted a mis-typed email address!  It does work! Thanks for helping.

Thank you @nickc73 , glad it works!
@wastedige I've tested the sample code from the readme in Linqpad, a brand-new console app, and... in a working application.  It all seems to be working fine, so I'm going to close this ticket.  But if you are still having problems, please feel free to post the information here and I'll do what I can to help you.
I'm pretty sure it's just a matter of either calling Wait() or making your method async.  I was able to reproduce your issue... I think.... by not doing those things, and I was able to fix them with Wait().

@darrencauthon Thanks for following up!
When I add result.wait() under the 'send' line it just never makes it past that line. Then I tried making the method async and adding await at the beginning of send line, in which case the code goes through.
It's probably a good idea to mention this at the Readme file example :)

@wastedige You are correct, I will do so. :smile:

@wastedige Updated the readme.  Thank you for bringing this up so we can help other people using the library!
:clap: :clap: :clap:

@wastedige Version 1.3.0 has just been released with sync support.
Details here: https://github.com/SparkPost/csharp-sparkpost#special-note-about-async


move example files to examples folder

I think it would be good to make an examples folder for the examples to let the user have a rather clean notebook to directly start simulating

Can you also update https://github.com/proppy/rad-lab/blob/tuning/modules/silicon_design/scripts/build/cloudbuild.yaml#L39 so that it only copy stuff in examples?

Why are the examples at all in the bucket? Shouldn't they just be in the notebook? Also, would there be a way to also update them in the notebooks when the image is updated?

Why are the examples at all in the bucket?

@HelgeGehring it seems that /home/jupyter get mounted from the VM in the jupyter container at startup so anything that we would but in /home/jupyter during the image building process would get effectively shadowed by the mount.
Copying them from GCS during the startup script seemed like a good workaround, but maybe provisining a dataDisk would be a proper way to achieve this (assuming it would get mounted instead at /home/jupyter)

Okay, sounds good, then copying the examples (and deleting the folder before) seems like a nice solution!

I've additionally added folders for electronics and photonics (and added there an mmi-tuning-example)

FYI, renamed the eletric dir to digital: https://github.com/proppy/rad-lab/commit/8c40f9d59f3ed4c0c1039140e44adf901acabfcd


How to delete file attribute from model?

I have uploaded a hero_image to the blog model successfully.
How do I delete this attribute in the CMS? It seems this is not supported yet.
My config/cms.php:
    'forms' => [
        'blog' => [
            ....
            'content' => [
                ....
                'hero_image' => [
                    'type' => 'file',
                    'alt_name' => 'Hero Image',
                ],
            ],

Just comment out that field
//'hero_image' => [
//    'type' => 'file',
//    'alt_name' => 'Hero Image',
//],

We use the FormMaker kit for the whole CMS so it makes it easy to add and remove components from the base modules.

@mlantz I'm sorry I wasn't clear.
I mean how to delete a file attribute value from a model in het CMS? As a user I can only upload a file. Not delete it. I would need some sort of delete button.

lol wow, I'm a moron, totally missed that, give me a day or so to add that

It would also be nice if the 'delete'-feature would be available to all fields of type 'file', based on the cms/config.php file.

Hey @yapsr I forgot to issue a release with those fixes, I'll release that today.
I like the idea of having the ability to delete any file input types, add that to my list, not sure when I'll be able to add that, but as always PRs are welcome


COFFFileReader::read_lib() fails on poorly generated yet correct .lib files

By correct, I mean that they can be used without error.  The PE spec doesn't seem to cover this case.
Synopsis
The COFFFileReader::read_lib() function currently reads the number of [archive] members from the second linker data member.  It then uses that number to loop through the remaining archive members and verify their headers.
However, it is apparently possible for some of those member to have offsets which point to 0.  When this happens, vcpkg will continue reading the std::fstream past EOF, and will end up calling exiting when COFFFileReader::verify_equal_strings() fails.
Example
When installing  pdcurses:x86-windows, vcpkg fails with error:
Incorrect string (LIB HEADER_END) found. Expected: `\n but found

This is because the second archive member claims there are 479 members, and there are in fact 479 member offsets.  This is the end of dumpbin's member offset table:
      1D1    133F6
      1D2    14224
      1D3    142F0
      1D4        0
      1D5        0
      1D6        0
      1D7        0
      1D8        0
      1D9        0
      1DA        0
      1DB        0
      1DC        0
      1DD        0
      1DE        0
      1DF        0

This makes it so that the second linker member data section is technically correct, but there are actually only 467 remaining archive members.
Reproduced on two similar 64-bit Windows 10 Anniversity Update machines, each with VS2015 update 3.
Possible Fixes

Use the second linker member's offset table to iterate over the archive members (difficult because it would require moving backwards in the file stream)
Use the second linker member's offset table to count the number of archive members without an offset of 0.
Check for ios::eofbit when reading the file stream.
Patch link.exe (if this is undesirable behavior)

I fixed this in PR #288 where I check if there is actually a header. It is a quickfix but your possible fixes could do the trick.
I will try your suggestions and PR if I come up with a better fix.

Figures there's a fix while I'm debugging.  I merged the latest master into my fork and this does fix my issue.
Your fix is pretty close to checking for the EOF, however I would suggest checking the eof bit after reading inside of read(), something like:
if (fs.eof() && ret.data[0] == '\0')
    return ret;

This would prevent members with incorrect data sizes from passing (they shouldn't).
And then in read_lib() after read():
if (fs.eof())
    break;

This way it doesn't try to read multiple empty headers.

Use the second linker member's offset table to iterate over the archive members (difficult because it would require moving backwards in the file stream)

I am not sure what you mean by moving backwards.
We could read the offset table and keep it in memory, sort it (although it should be sorted already), then remove the zeros. Then, we can use it to do the checks.

I am not sure what you mean by moving backwards.
We could read the offset table and keep it in memory, sort it (although it should be sorted already), then remove the zeros. Then, we can use it to do the checks.

For some reason I didn't think of sorting the table, but in my example the the offsets were only mostly sorted, the following was common:
       B1    10338
       B2    10402
       B3    1039C

But yeah, reading the table, sorting it, and using that is likely the most reliable and long-term solution.

Current implementation:
-Reads the offset table and stores them, ignoring those that point to 0
-Sorts them just in case. It wouldn't be wrong anyway, it would just move back and forth in the file stream.
-Use these offsets to go through the archive members and do the appropriate checks.
Also did a bit of refactoring in the file to improve clarity.


issue with operators

Hello,
I am struggling with plus and minus operators
"x + 1" is giving  tex "x+1"
But "x +- 1" is giving tex "x+-1"
and "x -- 1" is giving tex "x--1"
is there a way to remove the extra operator after using toLatex ?
Thank you in advance

x +- 1 will be parsed as x + (-1), I guess the formatter could be improved by adding the explicit parenthesis


web-based robozonky

are there any plans for a web-based robozonky?
it would be a great options for those using tablets or who only have company issued computers.
thanks :)

I don't have any plans of doing it myself. But I won't be stopping anyone else from doing so. :-)
That said, I think that - for this use case - it'd be almost easier to build a brand new robot. Reuse some of the modules, like strategies and notifications, and do the rest all over again.

I am planning to implement something like that in Zvonky, thus I am assigning the issue to myself.


Fluid Mixer fills both input slots

When using the fluid mixer to make Flux Infused Oil when inputting destablized redstone it fills both slots at the same time so I am unable to add the oil, just wondering am I missing something?

that's a bug in the gui, are you using the latest version of thermal extra?

i'm on latest and it works, please update thermal extra.

if you're playing a modpack, ask the modpack dev to update the mod in the pack.

This is on the latest version, I don't think you understand what I am saying, surely the 2 fluid input tanks should be classed as seperate tanks, unless you are requiring both fluids to be inputted at the same rate?

I am having the same issue. In my situation I have crude oil input from the top of the FM and destabilized redstone input from the bottom. If neither input tank is empty and I am able to input both fluids, there is no issue and each input tank gets its desired input.
The issue occurs when either input source stops and the other continues. When one of the input tanks in the FM empties, it will then get filled by whatever input source is still sending fluid, causing  the tanks to both be filled by one fluid type, and breaking automation.
Ideally both tanks in the FMs gui should be able to be assigned to different input colors in the I/o configuration gui. Just my 2 cents at least.


OnValidate logs an error if NetworkIdentity was added after other NetworkBehaviors

Describe the bug
NetworkBehaviors throw error message Mirror.NetworkTransformReliable on Spine requires a NetworkIdentity. Please add a NetworkIdentity component to Spine or it's parents. even though the game object has NetworkIdentity on it. Event though the error message is shown by OnValidate, everything still works. It's just the error message.
[IMPORTANT] How can we reproduce the issue, step by step:
Please tell us how to reproduce your issue, STEP BY STEP, with one of our built in examples.
-> Set the script execution order to run NetworkIdentity after NetworkTransformReliable
-> In any script attempt to create a Game Object on runtime
-> Add component NetworkTransformReliable first
-> Then after add component NetworkIdentity
var networkTransform = splitRootObj.AddComponent<NetworkTransformReliable>();
var networkIdentity = splitRootObj.AddComponent<NetworkIdentity>();

-> Run the code
-> NetworkTransformReliable triggers OnValidate and before the NetworkIdentity's Awake method which results in NetworkTransformReliable not having netIdentity and the error message to be shown.
On the other hand, if I move AddComponent<NetworkIdentity>() to be called first, I end up with a null reference thrown on every frame as the NetworkIdentity's Awake is triggering before NetworkTransformReliable is registered in NetworkBehaviours.
var networkIdentity = splitRootObj.AddComponent<NetworkIdentity>();
var networkTransform = splitRootObj.AddComponent<NetworkTransformReliable>();

Any bug that can be reproduced, can be fixed.
If we can't reproduce it, we can't fix.
Expected behavior
No errors to be shown.
Desktop (please complete the following information):

OS: MacOS
Build target: Editor
Unity version: 2022.3.25f1
Mirror branch: main

Creating networked objects at runtime isn't supported, and we don't test for that or try to make that work, especially not with changing script execution order.

I don't understand what that means. What are the custom spawn handlers for then?

Object pooling, adding art from addressables at runtime, stuff like that.  To spawn anything, client has to already have the prefab with the asset id that the server will send.
Bug report is not for discussion - join Discord for that.