fix(button): restore disabled state set from outside when loading sta…

…te changes to default
Signed-off-by: EndzeitBegins<EMAIL_ADDRESS>PR Checklist
Please check if your PR fulfills the following requirements:

[X] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)
[ ] If applicable, have a visual design approval

PR Type
What kind of change does this PR introduce?

[X] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] clarity.design website / infrastructure changes
[ ] Other... Please describe:

What is the current behavior?

Issue Number: #129
What is the new behavior?
When changing to the loadingState to default the disabled value set from outside is restored, whether is was set before or while the loadingState was non-default.
Does this PR introduce a breaking change?

[ ] Yes
[X] No

Other information
My solution includes replacing the field disabled in CdsBaseButton with a corresponding getter and setter in order to override its behavior in CdsButton. This in turn adds a private field _disabled to CdsBaseButton. To avoid collisions I renamed _disabled to _disabled_from_outside in CdsButton.
According to npm run public-api:check this results in changes to the "public" API, even though only private fields were added / renamed and the field disabled replaced with getter / setter.

Thanks for taking a stab at this, I'm reviewing and looking through it now. I don't love the idea of completely disregarding the value attempting to be set in the setter.
We already had the _disabled hidden variable that was tracking the manually set disabled status. And  uses it to set the disabled value when the loading state changes back to default. I'm trying to first understand why this isn't doing the job in Angular. Is it because Angular is bulk updating the web component properties at the same time (settings loadingState and disabled at the same time?) or in a different order than expected (setting loadingState before disabled)? That would help me better understand your fix.
Likewise, I'm not sure I follow your unit tests updates and what they're attempting to do that's different than what they did before.

This is causing some of the react snapshot/unit tests to fail and I'm investigating why.

Thank you @ashleyryan for investing time looking into this issue.
I'm not quite certain whether there is something I can help you with at this moment.
Let me know if I can be of any help.
Should I change my pull-request in any way? Or do you simply take over the issue?

I've found a slightly different solution that also passes the added tests. But still fails the aforementioned snapshot tests, as lit seems to add the attribute disabled="true" instead of former disabled="" to the DOM. However I don't know why that is the case or whether that is an actual problem or not.
However it no longer needs the conditional logic in the setter as you've lined out.
The idea is to basically use the getter to determine the disabled state both on the value stored in the parent and the loadingState:
  get disabled(): boolean {
    return super.disabled || !this.isDefaultLoadingState(this.loadingState);
  }

  set disabled(value: boolean) {
    super.disabled = value;
  }

Coupled with notifying lit about the potential update whenever the loadingState changes, this seems to have the same effect, without requiring an additional private property or explicit setting of the disabled state.
  update(props: PropertyValues<this>) {
    if (props.has('loadingState')) {
      // change in loadingState might also affect disabled state
      this.requestUpdate('disabled', super.disabled);

      if (this.isDefaultLoadingState(this.loadingState)) {
        this.style.removeProperty('width');
      } else {
        this.style.width = getElementWidth(this);
      }
    }

    super.update(props);
  }

With this approach, the override of firstUpdated can be dropped altogether.

I can either take the PR over or you can try and make some of the changes you or I have suggested.
 We also need to update the commit message to include a link to the original issue. So update it to something like

fix(button): restore disabled state when loading state finishes
fixes #130

I've spent most of the day digging into why lit is doing that with boolean attributes and I've been talking to them. I've narrowed it down to something not working correctly when using a getter or setter vs a simple property. I think it's fine and I can update the react tests if necessary. But let's try the updated solution for now.
I'm also happy to take it over, if you'd like me to.

I ended up taking it over @EndzeitBegins . I couldn't get your suggested approach to work - even though the tests were passing I was finding that in storybook when I was manipulating the component with
document.querySelector('cds-button').loadingState = 'loading';
document.querySelector('cds-button').loadingState = 'default';

That it wasn't losing the disabled status for some reason.

Thanks for looking into this and finding a solution that works @ashleyryan with the remaining use cases.
I hope my initial test cases were helpful in aiding the way. Looking forward to the merge!


Nanopolish Eventalign

Hi,
I would like to know if the command nanopolish eventalign changes the file size/content of the sequencing_summary file we specify in the command. If so, why? Since we are not writing the output in the sequencing_summary file.
The command used by me was:
nanopolish eventalign --reads  --bam  --genome  --scale-events --signal-index --summary   --threads 50 >
After I ran this command, the size of the sequencing_summary file also changed. I need some clarity on this.
Thanks,
Sukriti

Hi, I also overwrote my summary file. I think you can check this reply:
https://github.com/jts/nanopolish/issues/940#issuecomment-927891281


Fix simulation results zip download

The results are being generated and the api endpoint is working. The front end still displays N/A.


Reduce the reauthenticate timeout to 333ms

It should not take all that long to load everything into redis

Ready to merge.

I've noticed you bringing the sleep times of tests down a few times, I just wanted to make sure that you know you can run tests in isolation -- I do this when I don't want to wait for the full suite:
./node_modules/.bin/mocha ./test/commands/foo-command.js
^ might be helpful .

@bcoe I use .only a lot. But I also run the whole suite quite often (each branch / pr * each rebase). That adds up quite a bit :)
I can't see the connection of lowering the reauthenticate timeout to the test timeouts though?


Add new devices in the snapshot example

The current snapshot example doesn't contain all the devices required by appstore connect e.g. iPhone 6.5" display.
https://github.com/fastlane/fastlane/blob/1d0155730dd3a788ce5db097dc8a89bcd120e3a3/snapshot/example/fastlane/Fastfile#L5

iPhone Xs Max should get you what you need.

iPhone Xs Max, lowercase s here.

I added iPhone Xs Max to my snapfile, but it take screenshots with 1241px width, instead of 1242px. Any solution for this error?

@gersonmdesouza my 9 screenshots of iPhone Xs Max also got 1241px width as well, only one is 1242px. Seems a bug, I just resize those correctly manually, so they can be used w/ frameit.


improved vox laugh

About the PR
Improved vox laugh, since the "original replacement" was a bit quiet and sounded muffled. This is still way less harsh than the 1.0 laugh
kinda meant to be a part of #31679 but was not tacked onto the PR in time
Media
live vs replacement replacement
https://github.com/user-attachments/assets/73a737bd-2a95-46ee-8b08-80701d5a9e00
Requirements

[X] I have read and I am following the Pull Request Guidelines. I understand that not doing so may get my pr closed at maintainer’s discretion
[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

me when    me when you take 3 days to update your pr and dont draft it in the meanwhile so we just end up merging it cause we had no idea

me when me when you take 3 days to update your pr and dont draft it in the meanwhile so we just end up merging it cause we had no idea

I lost track of time I have been juggling a bunch of PRs recently

also you should have a cl since publish has already came with your previous pr

I thought this is a rather minor tweak that wouldn't need one


[$250] Unable to scroll to the old history quickly

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: 1.4.68-0
Reproducible in staging?: needs reproduction
Reproducible in production?: needs reproduction
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @johnmlee101
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1714491953222949
Action Performed:

Open a report with a numerous chat history
Scroll to the older history quickly

Expected Result:
Should be able to scroll to older history
Actual Result:
The cursor resets back to the bottom
Workaround:
unknown
Platforms:

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

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

Screenshots/Videos
Add any screenshot/video evidence
https://github.com/Expensify/App/assets/38435837/aa4984ad-b7f5-46ec-8c96-4176e957aeee

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

Upwork Job URL: https://www.upwork.com/jobs/~017e9d87b15177f6cd
Upwork Job ID:<PHONE_NUMBER>666443264
Last Price Increase: 2024-05-13

Issue OwnerCurrent Issue Owner: @abdulrahuman5196

This has been labelled "Needs Reproduction". Follow the steps here: https://stackoverflowteams.com/c/expensify/questions/16989

I also can't reproduce, have asked John to try on the latest version here.

Checked in with @johnmlee101 to confirm if this is still an issue.

Checked in with @johnmlee101 to confirm if this is still an issue.

Still continues to be an issue for John.

Not overdue, not crucial enough to increase just yet.

Still not overdue

I am Michał from Callstack - expert contributor group. I’d like to work on this job.

Can you check if the issue still exists? It seems to be working fine. Maybe some more steps are needed to recreate this?
https://github.com/Expensify/App/assets/23487612/15e8cd16-d292-42be-89d4-4112eb0ccdb9

@MrMuzyk thank you!

Are there maybe any public chats I could join that have bigger history so I can try reproducing it again?

I don't know if we have one, rather it might be useful to reproduce the same circumstances by spamming images/messages with multiple accounts in a test chat. And then if you make it public could be a good stress test for others as well

@MrMuzyk could you please follow these steps to request a high-traffic account? This should give you access to a Big Chat which has thousands of messages.

@kadiealexander Hey, when I find some free time I will definitely do that :) Last week I've started helping with one of the waves and I'm fully occupied with feature development 😅

Not overdue.

Requested a retest here https://expensify.slack.com/archives/C9YU7BX5M/p1718081137015559

Hey @johnmlee101 I think it would be better if you could troubleshoot this in #newdot-quality, since no one is able to reliably replicate the issue. Please feel free to reopen once we have some logs or indicators for what is causing the issue. Thanks!


0-bootstrap: backend.tf does not require a projects backend update - just a bucket id update

in step 10 of https://github.com/GoogleCloudPlatform/pbmm-on-gcp-onboarding/blob/main/0-bootstrap/README.md#deploying-with-cloud-build
the 2nd placeholder in backend.tf does not have the following key - hence step 2 of the following has no effect.

-    validate_git
+    #validate_git

     if [[ ! "$ERRORS" == *"gcloud"* ]]; then
         echo "Validating local gcloud configuration..."
(END)


Add automatic migration for ng-button class .btn-check on the input element

This needs to implement an css-classes-add functionality first, to be able to add a automatic migration for it.
We'll try this later...

Done within #687


Determine and document MSRV

1.60 is enough, but for tests, which use rand version 0.8.5, later rust versions are needed.
(In the future, random arbitrary integers will be supported, but feature gated.)
Maybe we'll need to lower the rand version used.

BREAKING: core::error::Error implementations for ParseError and BaseError
are now gated by the std feature. no_std is the default.
CI tests both the latest and MSRV (and checks with clippy as well)


python311Packages.podman: 4.9.0 -> 5.0.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python311Packages.podman is: Python bindings for Podman's RESTful API
meta.homepage for python311Packages.podman is: https://github.com/containers/podman-py
meta.changelog for python311Packages.podman is: https://github.com/containers/podman-py/releases/tag/v5.0.0
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 5.0.0 with grep in /nix/store/qpjdisn8fgaakxnbsrf71yrxhww4qvab-python3.11-podman-5.0.0
found 5.0.0 in filename of file in /nix/store/qpjdisn8fgaakxnbsrf71yrxhww4qvab-python3.11-podman-5.0.0

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

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath
python311Packages.podman
python312Packages.podman

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/qpjdisn8fgaakxnbsrf71yrxhww4qvab-python3.11-podman-5.0.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

Or:
nix build github:r-ryantm/nixpkgs/60435b7315f42847dee09c711679743b55771147#python311Packages.podman

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/qpjdisn8fgaakxnbsrf71yrxhww4qvab-python3.11-podman-5.0.0
ls -la /nix/store/qpjdisn8fgaakxnbsrf71yrxhww4qvab-python3.11-podman-5.0.0/bin

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

4 packages built:

python311Packages.podman
python311Packages.podman.dist
python312Packages.podman
python312Packages.podman.dist

Maintainer pings
cc @fabaff for testing.

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

4 packages built:

python311Packages.podman
python311Packages.podman.dist
python312Packages.podman
python312Packages.podman.dist


Html Help Text property

I added this property to the form fields to allow viewing html elements

#64


Error thrown when Recording started

Chrome Version 68.0.3440.106, Puppeteer Recorder 0.2.0
Steps:

run: npm run dev
load unpacked extension in Chrome
Launch the Plugin and click "Record"
Notice the following error - https://screencast.com/t/C5fXkTMJrjz
after interacting with a page, I notice there is no script saved off.

@jakefromqa Yeah, sorry that was a botched release. Still getting the hang of how storage works in extension. Checkout release 0.2.1 that was released right after. It should already be live on the Web Store. 0.2.0 is DOA.


Issue building on MSVC 19.11

Hey guys, upon attempting to build using Visual Studio 2017, this is the error that I'm getting:

C:\Projects\eth\cpp-ethereum\build (master)
λ cmake ..
-- Building for: Visual Studio 15 2017
-- The C compiler identification is MSVC 19.11.25507.1
-- The CXX compiler identification is MSVC 19.11.25507.1
-- Check for working C compiler: C:/Program Files (x86)/Microsoft Visual Studio/2017/Community/VC/Tools/MSVC/14.11.25503/bin/HostX86/x86/cl.exe
-- Check for working C compiler: C:/Program Files (x86)/Microsoft Visual Studio/2017/Community/VC/Tools/MSVC/14.11.25503/bin/HostX86/x86/cl.exe -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working CXX compiler: C:/Program Files (x86)/Microsoft Visual Studio/2017/Community/VC/Tools/MSVC/14.11.25503/bin/HostX86/x86/cl.exe
-- Check for working CXX compiler: C:/Program Files (x86)/Microsoft Visual Studio/2017/Community/VC/Tools/MSVC/14.11.25503/bin/HostX86/x86/cl.exe -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- [hunter] Calculating Config-SHA1
-- [hunter] Calculating Toolchain-SHA1
-- [hunter] HUNTER_ROOT: C:/Users/r0ot/.hunter
-- [hunter] [ Hunter-ID: 435b09a | Config-ID: 61c532d | Toolchain-ID: 8743852 ]

[hunter ** INTERNAL **] Unexpected MSVC_VERSION: '1911'
[hunter ** INTERNAL **] [Directory:C:/Projects/eth/cpp-ethereum]

CMake Error at C:/Users//.hunter/_Base/Download/Hunter/0.18.57/435b09a/Unpacked/cmake/modules/hunter_wiki.cmake:12 (message):
Call Stack (most recent call first):
C:/Users//.hunter/_Base/Download/Hunter/0.18.57/435b09a/Unpacked/cmake/modules/hunter_internal_error.cmake:13 (hunter_wiki)
C:/Users//.hunter/_Base/Download/Hunter/0.18.57/435b09a/Unpacked/cmake/modules/hunter_setup_msvc.cmake:68 (hunter_internal_error)
C:/Users//.hunter/_Base/Download/Hunter/0.18.57/435b09a/Unpacked/cmake/modules/hunter_finalize.cmake:114 (hunter_setup_msvc)
C:/Users//.hunter/_Base/Download/Hunter/0.18.57/435b09a/Unpacked/cmake/modules/hunter_add_package.cmake:23 (hunter_finalize)
CMakeLists.txt:86 (hunter_add_package)

-- Configuring incomplete, errors occurred!
See also "C:/Projects/eth/cpp-ethereum/build/CMakeFiles/CMakeOutput.log".

Any ideas on how to fix this? Much appreciated!

Same issue, anyone?

Hit this as well, a temporary fix is to edit the file that is complaining (C:\.hunter\_Base\Download\Hunter\0.18.57\435b09a\Unpacked\cmake\modules\hunter_setup_msvc.cmake ) with the fixed code found in the main hunter repo (https://github.com/ruslo/hunter/commit/b8c81ae2b2ed049e5c5deb49fa5d55bea61655c0):
-string(COMPARE EQUAL "${MSVC_VERSION}" "1910" _vs_15_2017)
+string(REGEX MATCH "^191[01]$" _vs_15_2017 "${MSVC_VERSION}")


Refactor Plate component

Notes after checking #903:

use hard shadow instead of the border
remove negative margins (won't be needed to hide it after finishing #872)
I think position: fixed should be defined by the parent component, also consider using sticky instead
at the end overflow-x: hidden shouldn't be needed
remove extra div

cc: @imollov

@davidyuk any specific reason to use shadow instead of border?

resolved in https://github.com/aeternity/superhero-wallet/pull/872 and https://github.com/aeternity/superhero-wallet/pull/952


Add support for --strict flag

@mrbrowning @mohitsoni We use env vars everywhere else for program configuration.  Please stay consistent, and don't use --args


Add code 53

Hey @tiffanyfay! An episode with the same title and description was added today in #1481, is that intended? Just wanted to double check before merging. Thanks!

Closing as a duplicate of #1481 as per our convo


ERROR: unable to download video data: <urlopen error EOF occurred in violation of protocol (_ssl.c:1129)

when i was download pronhub videos

Do not ignore the template.


Incorrect description for /serializer:Auto flag

Incorrect description for /serializer:Auto flag
Updated the description for the /serializer:Auto option to reflect what svcutil.exe returns as the help text description for this option. Value was incorrectly set to the same description as the /serializable option.

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

@danroth27  please review

Thanks for spotting this and submitting the fix @fuzzy-oink we appreciate it.
We've merged this, and the changes will show up on our live site on our next publish cycle, usually within a day or so.


soc: xtensa: intel_adsp: ace: multiprocessing: add missing init.h

File was using SYS_INIT without including init.h.

fixed by https://github.com/zephyrproject-rtos/zephyr/pull/62075, sorry lost count


fix rhel7 jdk 11 build; enable tests for centos and jdk 11

Signed-off-by: rcernich<EMAIL_ADDRESS>Thanks for submitting your Pull Request!
Please make sure your PR meets the following requirements:

[ ] Pull Request title is properly formatted: [CLOUD-XYA] Subject
[ ] Pull Request contains link to the JIRA issue
[ ] Pull Request contains description of the issue
[ ] Pull Request does not include fixes for issues other than the main ticket
[ ] Attached commits represent units of work and are properly formatted
[ ] You have read and agreed to the Developer Certificate of Origin (DCO) (see CONTRIBUTING.md)
[ ] Every commit contains Signed-off-by: Your Name<EMAIL_ADDRESS>- use git commit -s

Following the instructions in the readme for rhel7 jdk11 did not include jdk11.  I discovered this while trying to simplify building jdk11 by using a single overrides file.  (I think it's more intuitive to build with cekit --overrides-file=rhel7-jdk11-overrides.yaml build, as opposed to layering it atop the rhel7 jdk8 overrides.)


Audit --> RuboCop migration

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[x] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew tests with your changes locally?

Migrate some of the audit rules to RuboCop (closes #7392).
Each rule is implemented in its own commit which might make reviewing easier.
Migrated Rules

separate make calls
env :std has been deprecated in homebrew/core
env :userpaths has been deprecated
don't use "#{foo + "bar"}"
require openjdk or :java dependency to use JAVA_HOME
use bin instead of prefix + "bin"
use pkgshare instead of share/"foo"
use :optional or :recommended  instead of build.with? or build.without? in dependencies (not in homebrew/core)
style now warns when using deprecated build.include?
don't need require "formula"

I think this is finally ready! 🎉 @MikeMcQuaid there have been a few minor changes since your last review so I'll leave this open for a little longer if you want to take a look.

I'm going to go ahead and merge this once CI is green. I've added TODOs and will follow up on those in a future PR.
Thanks for looking at this!


Fix JsonFileMappingsSource.remove() for subdirectories

The JsonFileMappingsSource.loadingMappingsInfo() method looks for JSON files in the mappings directory recursively, which is useful since it allows organizing stubs in a directory hierarchy. However, the remove() and removeAll() methods will not remove stubs in a subdirectory, because the path components are stripped out of the filename.
This was introduced in 7f93b5eb996722c1919a6e1e627d550f7929ac39, which changed the getFileName() to strip out all path components. I updated that method to only do so for non-file URLs.

I think I figured out a better fix: just change AbstractFileSource#writableFileFor() to handle file paths. In addition to fixing the deleteFile() method, it will also allow writing new stub mappings to subdirectories when calling writeBinaryFile() or writeTextFile().
Since ClassPathFileSource does not extend AbstractFileSource, it won't use this new logic.


Team page

I have now set it to automatically scroll down when we change the team. This however is not a good solution. Ideally, the component should update there itself. I think instead of using URLs, we can have a react component (to take advantage of reactive states) to achieve this. But let's keep it for sometime later.

@harshkhandeparkar  can you create an issue for this so that it doesnt get left out, Atleast we'll have soemthing working right now

You mean like the other 3? @rajivharlalka  and I thought to keep the alums section always present in the page. WHat do you think will be better?

You mean like the other 3? @rajivharlalka and I thought to keep the alums section always present in the page. WHat do you think will be better?

Since there are many alums, that section will be too long imo.

And there goes the flat heirarchy. Anyways....


Loaded mesh shifted

Light gray voxel is loaded from ".vox" file with Transform::from_translation(Vec3::ZERO). But in fact it located at (1,1,1)

This stems from the meshing which requires the voxel data to be padded by 1 on each direction, resulting in the observed offset. This just needs a processing step post-voxel meshing to fix the offset issue.

Since this is a known specificity of the internal implementation, then we can close the problem. I thought that this might be a bug in the implementation

I'll leave this open as a remainder to fix this.

If I understand correctly, it's dependent issue: https://github.com/bonsairobo/block-mesh-rs/issues/6 ?

yes altough that can be fixed by applying a post-processing step on the generated vertices to remove the offset.

@kgv Just pushed c152b3cf5923b9fae50e0c8d97abde9cf8ea5d39 on master which should fix the aforementionned issue.


Adjust proxy configuration and resources

@WadeBarnes changes look good, but it appears that the commit is failing the DCO check and the branch is out-of-date. COuld you please fix and resubmit for review?

Done


[Quantum] [DRAFT - Do not merge] Convert provider name to lowercase

When provider names were capitalized in the as quantum workspace create command, the providers did not appear in the portal.  Converting all provider names to lowercase before executing this command.
OK, it’s not that simple:
The lowercase conversion works great for "Quantinuum/test1", but I tested it with "Microsoft/Basic" and no providers showed in the portal! ☹

This checklist is used to make sure that common guidelines for a pull request are followed.

OK, it’s not that simple:
The lowercase conversion works great for "Quantinuum/test1", but I tested it with "Microsoft/Basic" and no providers showed in the portal! ☹

Quantum

Wrong design -- need to look up correct case in offerings list instead.


Love it! Thank you.

Hello, another Ryan! 😄👋
First of all, I have to say I absolutely love this idea & project. Thank you a million for putting it together, it's a genuinely novel & lovely addition to the Spring-o-sphere, and one I'm certain will have users... as I'm already in need of it! I really want to toss my current embedding code and use this everywhere instead.

Hello to you as well, other Ryan! 😄 Thank you so much! I appreciate the kind words.
I did see what the issue originally was and I got really worried, hah, but I'm guessing you might have found the same info — when innerHTML injects


Use escape of Qutation Mark for use with ngerman babel package

If you use a vowel after a " while using the ngerman babel package, the vowel will not be escaped correctly:

If the " is replaced by a \dq{} it works again, but only if the ngerman babel package is active. Probably would be best to provide an additional configuration parameter, but for now I only added it as an extra feature :shrug:.

Latex Sample
\documentclass{article}

\usepackage{xcolor}
\usepackage{listings}
\usepackage[ngerman]{babel}

\begin{document}

\input{test.rs.tex}

\end{document}

Good catch! How about I make an optional flag for the cli --german (or -g) that will enable the extra replacement? It seems to fit better than feature flag.

Sure, I don't mind.
The only disadvantage is that you would always have to include this flag in the cli.
And maybe -g is a bit too short for such a specific feature.

I just published new release 1.1.0 where this issue is solved by the CLI flag --german.


Charts duplicate labels

Hey,
I found something about this bug.
It's not only a duplicate of the label, it's a stacking of the bars and the labels.
Each time the NewWeekStepsView appears, the bars are stacked.

Observe the daily goal  line of 10,000 steps.
What else I noticed: The file NewWeekStepsView.swift has two times a reference in the project navigator.

Removing the reference and adding the file again did not fix the problem.
I am a Swift and SwiftUI beginner and have not found a solution. 😔
I hope this information can help someone fix this error.
I would be pleased to receive a message with the solution when it has been found.
~Ingo

@in60dev thanks for sharing this! Definitely will help solving this error!

I can take a look at this one.

@kritiagarwal13 thank you! Just assigned you


why the raw image is rotated

Im using the following code to get the bayer image.
But I dont get why it is rotated?
Any suggestion
raw = rawpy.imread(path)
rgb = raw.raw_image

What do you mean by rotated? 90°? If you hold your camera to take a portrait oriented picture then it stores the rotation just as metadata, it doesn't rotate the RAW image. Is that what you mean?

so I have a .nef image, which is read it via rawpy.imread.
I can just convert it to png via raw.postprocessing, when I do that it is fine. However, when I look at the output of raw.raw_image it is rotated by 90 degree. why is that so? anyway to fix it?

What you consider as "rotated" is actually the original. What you get when you postprocess it is when you apply the orientation that the camera had when you shot the picture so that it appears correctly. You know, older cameras didn't have those orientation sensors and you would also see what would correspond to the raw image.
Without knowing what your goal is, there isn't really anything to fix since that is by design. I'm closing this issue since rawpy library is just a wrapper around https://www.libraw.org/ which does all the heavy lifting. I recommend to ask your question (if it's still unclear) on https://photo.stackexchange.com/


Handle '-local-name' option in the CE View

Background: https://jira.opensciencegrid.org/browse/SOFTWARE-2477
Also some spec file updates. I tested the option handling and verified that the CE View page now works upon service restart.

@edquist the Travis-CI checks will fail because of the version dependencies in the latest CE :(

LGTM.


Training to create a new predictor

I don't know if it is the right place to post, apologies if it isn't
I am trying to train to create a new predictor just to output landmarks of eyes. I have followed everything you have written as how to train . I get an error as
bhargav$ python3 train.py all_train
Traceback (most recent call last):
File "train.py", line 78, in
dlib.train_shape_predictor(training_xml_path, "predictor.dat", options)
RuntimeError: There is a fatal error on line 22 so parsing will now halt

this is the xml file i am using to train
please let me know where it went wrong or what i should do to solve this?

Start out by looking at line 22, like the error says.  Also, no one can tell what that line is because you posted an image that's cropped.  Moreover, don't post images of text.

Sorry  for the mistake sir.
I am attaching the correct picture now .

The error is because I am only giving the landmarks of 12 points corresponding to just the eyes in the training xml file .
so my question is do I need to give all the 68 landmarks in the training xml file?
If yes, how do I do create a model that will just output the 12 landmarks of both the eyes and not the whole face?
Thanks in advance sir!

Don't post images of text.
Anyway, the error has nothing to do with how many landmarks there are.  It's telling you your XML is malformed.  You have part tags that aren't closed.

sincere apologies for posting image.
It was a silly mistake , Thank you so much!

@ProudCule
if you don't mind , I want to ask you about the training you did?is it worked fine??
because now I'm dealing with the same issue,after training i got bad prediction
XML fine looks like

left:prediction
right:groud truth


Secret exports from cfn-env are not redacted

Elastic CI Stack for AWS: v5.17.0
We deploy all our infrastructure via AWS CDK and store our secrets in AWS Secrets Manager.
In our Buildkite bootstrap script, we alter the /var/lib/buildkite-agent/cfn-env to fetch and export our secrets from secrets manager, thus avoiding the need to create extra files in S3 to set our secret environment variables.
I have noticed that the output from cfn-env being run is not redacted according to the redaction rules and was hoping that this could be fixed.
Here's a screenshot of such output (manually redacted):

Blue values are redacted for the purpose of posting this publicly but the red value is also what I would expect Buildkite to have redacted.

Hi @ChrisLane, thanks for using the buildkite-agent and bringing this to our attention.
The redaction of values only works after the environment variables that contain those values have been exported for the first time. Furthermore, the redaction requires the hook that is exporting the variable to have completed before it knows the values to redact.
I assume you're using one of the functions in the cfn-env script, set_unless_present or set_always to create the secrets as environment variables. Unfortunately, those functions print out the values of their arguments BEFORE using them to create environment variables. Here's the body of the simpler of the two:
1	function set_always() {
2	    local target=$1
3	    local value=$2
4
5	    echo "export ${target}=\"${value}\""
6	    declare -gx "${target}=${value}"
7	}

The text you see in your screenshot is from the echo command in line 5 (or its equivalent for set_unless_present). If you passed in a secret value as the second argument, the buildkite-agent does not know whether it should redact the secret value until it is turned into an environment variable. This is because the redaction is based on the name of the environment variable associated to the secret value, and that association is not made until at least line 6. However, as I mentioned above, the design of the redaction also has to wait until the hook has completely finished execution.
While we could change the cfn-env script to not print these values, we don't want to do so without considering any impact on other customers that expect these to appear in the job log. So I think the simplest workaround in your case is to not use the function set_always or set_unless_present, and just use export or declare -gx to turn your secrets into environment variables instead.
However, it should be noted that the cfn-env script is sourced for more than just jobs. There are scripts on the elastic stack that are not related to running jobs that also sources the cfn-env. So if you only need these secrets for running jobs, I don't think this is the best place to put them. Instead, I recommend creating a plugin or modifying the global environment hook instead.

Thanks for the detailed response.
I will make some silent versions of the functions in that file and hopefully migrate to modifying the global hook.
On May 4, 2023, 00:52, at 00:52, Narthana Epa @.***> wrote:

Hi @ChrisLane, thanks for using the buildkite-agent and bringing this
to our attention.
The redaction of values only works after the environment variables that
contain those values have been exported for the first time.
Furthermore, the redaction requires the
hook that is exporting the
variable to have completed before it knows the values to redact.
I assume you're using one of the functions in the cfn-env script,
set_unless_present or set_always to create the secrets as
environment variables. Unfortunately, those functions print out the
values of their arguments BEFORE using them to create environment
variables. Here's the body of the simpler of the two:
1	function set_always() {
2	    local target=$1
3	    local value=$2
4
5	    echo "export ${target}=\"${value}\""
6	    declare -gx "${target}=${value}"
7	}

The text you see in your screenshot is from the echo command in line
5 (or its equivalent for set_unless_present). If you passed in a
secret value as the second argument, the buildkite-agent does not know
whether it should redact the secret value until it is turned into an
environment variable. This is because the redaction is based on the
name of the environment variable associated to the secret value, and
that association is not made until at least line 6. However, as I
mentioned above, the design of the redaction also has to wait until the
hook has completely finished execution.
While we could change the cfn-env script to not print these values,
we don't want to do so without considering any impact on other
customers that expect these to appear in the job log. So I think the
simplest workaround in your case is to not use the function
set_always or set_unless_present, and just use export or declare -gx to turn your secrets into environment variables instead.
However, it should be noted that the cfn-env script is sourced for
more than just jobs. There are
scripts
on the elastic stack that are not related to running jobs that also
sources the cfn-env. So if you only need these secrets for running
jobs, I don't think this is the best place to put them. Instead, I
recommend creating a plugin or
modifying the global environment
hook
instead.
--
Reply to this email directly or view it on GitHub:
https://github.com/buildkite/agent/issues/2061#issuecomment-1533891080
You are receiving this because you were mentioned.
Message ID: @.***>

Cool, I think this plugin could be useful for you too: https://github.com/seek-oss/aws-sm-buildkite-plugin


generated metadata is invalid

after https://github.com/italia/spid-testenv2/commit/8e27c964cf8fc13a851d4c866020411405177b77 the metadata generated by spid-php-lib is deemed invalid by the testenv2
the error reported in the console is:
spid-testenv2_1  | ERROR:__main__:ValidationDetail(
  value=None,
  line=None,
  column=None,
  domain_name=None,
  type_name=None,
  message='required key not provided',
  path='xpath
    {urn:oasis:names:tc:SAML:2.0:metadata}EntityDescriptor/
    {urn:oasis:names:tc:SAML:2.0:metadata}SPSSODescriptor/
    {urn:oasis:names:tc:SAML:2.0:metadata}AttributeConsumingService/0/
    {urn:oasis:names:tc:SAML:2.0:metadata}RequestedAttribute/0 -
    attribute: NameFormat')

I have prepared a PR, but I have also raised the issue with testenv2 ... let's wait a bit

fixed by testenv2, no action required


Should be able to create paths if they don't exist

This task should be able to create all the paths along it's way. Considering the fact that there is no separate task to create paths.

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

ID: 81f02378-b85b-d3a4-d3c8-f07d935367da
Version Independent ID: fab57f76-35d2-4024-6f7f-1740d7b24743
Content: Copy Files task - Azure Pipelines
Content Source: docs/pipelines/tasks/utility/copy-files.md
Product: devops
Technology: devops-cicd-tasks
GitHub Login: @vtbassmatt
Microsoft Alias: macoope

@okezieokpara  Thank you for the suggestion.
On product-related issues we recommend using Report a problem or Suggest a feature in the Azure DevOps community to solicit feedback.
You can also try starting an issue here.
Otherwise, to request changes to the documentation please include a reference below.

Closing this issue now.


bluetooth: controller: Handle non-priority events correctly

In certain cases the response to a command can come in the form of a
non-priority event. This is the case of LE Create Connection Cancel,
which generates a Command Complete and then an LE (Enh) Connection
Complete. Take this case (and other future ones) into account by calling
the correct Host recv function.
Fixes #3446
Signed-off-by: Carles Cufi<EMAIL_ADDRESS>

@jonpry can you try this out?

@carlescufi works great. amazing even.

@jonpry thanks for testing it out and helping us debug.


KeyError: 'Images'

Got the following error after starting painting:
File "C:\Users\ikben\AppData\Local\Programs\Python\Python310\lib\threading.py", line 1016, in _bootstrap_inner     self.run()   File "C:\Users\ikben\AppData\Local\Programs\Python\Python310\lib\threading.py", line 953, in run     self._target(*self._args, **self._kwargs)   File "C:\StableDiffusion\stable-diffusion-webui\extensions\SdPaint-main\Scripts\SdPaint.py", line 92, in send_request     return_img = r['images'][0] KeyError: 'images'
I enabled API mode and allowed controlnet.

Make sure the "--api" is in the .bat.
Install controlnet.

http://<IP_ADDRESS>:7860/
Extensions
Available
CLICK on "Load from:"
Search for "controlnet" and Click "Install"
Restart the UI and launch the SdPaint
**If you want to change prompts, styles, models, whatever - edit the "payload.json"

I'm getting the same error as @ikbentheo

Just updated Auto1111 and Controlnet via git pull
--api is saved in webui-user.bat
"allow other scripts" checked - applied those settings and restarted UI
twirled open controlnet interface in Auto1111 webui, and checked "Enable" box
run SdPaint Start.bat
Canvas launches, but first stroke gives same error as above. Only difference is in the line# of the SdPaint.py:

Desktop\SdPaint PyScript\SdPaint-main\Scripts\SdPaint.py", line 113, in send_request     return_img = r['images'][0] KeyError: 'images'
What am I missing?
Thanks

I had this issue, and turns out I needed to update from Python 3.9 to 3.10.11!
Sorry ikbentheo, since it doesn't seem to apply to you since you already seem to be on 3.10

The API is receiving an empty model name. Can you try to delete your config.json and controlnet.json files, start the webui + API, and only when the webui is started start SdPaint. It should fetch your model names from http://<IP_ADDRESS>:7860/controlnet/model_list and write those in the newly created config.json (check the controlnet_models entry in the file).

Or it could simply has something do to with the ControlNet side. I see that some of your .safetensors files have no .yaml pendant. Try to copy some of the yaml files with the corresponding filename.

Oh and I just saw in your first picture that /controlnet/txt2img is called, you have an old version of SdPaint. Please do a git pull to get the latest one.

hey! when you told me to check the .json files, i realized that i didn't have them and then i saw that i actually cloned only half the code, several files were missing. i downloaded the repo again and now it works! :-) thanks for this awesome program!!


APIGOV-21089 - check the status of the jobs on an interval

I thought we were trying to update/replace go calls to jobs? Or it's not relevant here 🙈

This job needs those go routines running to process any subscriptions the job finds while polling.  We could make the go routines here jobs themselves, I would think.


[QUERY] src vs sdk folders

Query/Question
Vast majority of the code is found under src as a convention.
Some code is under sdk. What's the reason for that?

Wes should be able to answer this one.
CC: @shahabhijeet

Thanks for the query @seanfelman but this is a temporary inconsistency as we are in the process of transitioning the a new convention with the source under the sdk folder, see issue https://github.com/Azure/azure-sdk-for-net/issues/5213.


auth with GitHub using nuxt auth module

AUTH SUCKS IN NUXT 😭😭😭😭

I would like to pick this one @lukeocodes Could you provide more info on this. I can not see login/signup in this page

@git-srinivas I would like to be able to log in to GitHub from the Nuxt application, hopefully without the need for a backend. If you need a backend, please create a Netlify function to act as the backend.

No, you should use GitHub OAuth2 workflow

okay.

@lukeocodes  shall we use the existing netlify function for authentication?

@lukeocodes  shall we use the existing netlify function for authentication?

@git-srinivas I would like to be able to log in to GitHub from the Nuxt application, hopefully without the need for a backend. If you need a backend, please create a Netlify function to act as the backend.

I could see we are already authenticated with github personal access token and able to get repositories information from github.  You mean to provide login functionality to the website visiting user?

Hi and yes. I should have updated the issue when you first showed interest, sorry for the lack of info.
There are a few ways to connect to GitHub APIs.
One is using a static personal access token (mine), which is stored as an environment variable and deployed to the Netlify Function (AWS Lambda function) that connects to GitHub.
Another way is to have a user of the site "Login with GitHub", they then get an Acess Token which can be stored in the memory of the frontend application (not in cookies or local storage), then sent through to the Netlify Function whenever they make a request. Not only for this purpose, but it also will provide the users GitHub profile information, to enrich the experience on the site (i.e. they could track their entire Hacktoberfest progress).
There are packages like nuxt-auth that will make some of this work easier, but from my own experimentation, those packages offered incomplete solutions. I believe I was closest with nuxt-auth, which appeared to be using an auth code grant, but I should have created a new Netlify Function to act as the backend to exchange the authorization code for an access token.
GitHub has some guides on building an OAuth application.

No worries. I will go through the documentation and thanks for the info 👍


Trusted profile with resource group only

https://github.com/terraform-ibm-modules/terraform-ibm-trusted-profile/blob/81d7c6a571a3d59f35a483a74e3cd56cf88dbad3/main.tf#L22
This validation logic does not account for valid use case for resource group only.
trusted_profile_policies with:
{
    roles = ["Viewer"]
    resources = [{
      resource = module.resource_group.resource_group_id
      resource_type           = "resource-group"
    }]
  }

Fixed in v1.0.5

:tada: This issue has been resolved in version 1.0.5 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


-g && --genre option help

bash@Shadows:~$ sudo spotify-ripper spotify:track:1dNIEtp7AY3oDAKCGg2XkH -g artist
Spotify Ripper - v2.9.1
  Encoding output:      MP3, CBR 320 kbps
  Spotify bitrate:      320 kbps
  Unicode support:      ASCII only
  Output directory:     /mnt/c/Users/xxxxx/Music
  Settings directory:   /home/xxxxx/.spotify-ripper
  Format String:        {album}/{track_num}. {track_name}.{ext}
  Overwrite files:      No
Logging in...
Logged in as xxxxx
Loading track...
Ripping spotify:track:1dNIEtp7AY3oDAKCGg2XkH
/mnt/c/Users/xxxxx/Music/Something Just Like This/1. Something Just Like This.mp3
Track Download Size: 9.46 MB
Progress: [========================================] 04:08 / 04:08        (~00m 00s remaining)
Rip complete
WARNING: libsndfile may ignore -r and perform fseek's on the input.
Compile without libsndfile if this is a problem.
1. Something Just Like This.mp3 [ 9.47 MB ]
Attempting to retrieve genres from Spotify's Web API
https://api.spotify.com/v1/artists/69GGBxA162lTqCwzJG5jLp
URL returned non-200 HTTP code: 401
-------------------------------------------------------------------------------
Setting artist: The Chainsmokers
Setting album: Something Just Like This
Setting title: Something Just Like This
Setting track info: (1, 0)
Setting disc info: (1, 1)
Setting release year: 2017
Adding cover image
Time: 04:08     MPEG1, Layer III        [ 320 kb/s @ 44100 Hz - Joint Stereo ]
-------------------------------------------------------------------------------
ID3 v2.3: 7 frames
Writing ID3 version v2.3
-------------------------------------------------------------------------------
Logging out...

Why does it returns this error code?
URL returned non-200 HTTP code: 401
I don't know how this API works but does this need some token to work?
{
  "error" : {
    "status" : 401,
    "message" : "No token provided"
  }
}

Edit: Perhaps it really does [?] (I don't know)
Error Code 401

Unauthorized - The request requires user authentication or, if the request included authorization credentials, authorization has been refused for those credentials.

Hi there! You are right. This feature now requires authorization. For that, you need a client id and a client secret that you can get from Spotify's developer portal. Maybe other forks of the spotify-ripper already implemented it. If not, I could try to take a look at it myself even though it looks like this repo is not really supported anymore...

It's been so long, so now how do we make it work?


Implements new component scafollding with CLI

Closing as we should use #66 as the only PR here. Feel free to test things in a branch, but no need to create new PRs for things that are tests


Fix correctness for DIRECT_SWITCH in InCodeGenerator

Instead of returning null immediately when we found NULL argument in a switch we must remember
that one of the arguments was null, and return it only if not matched.
Was:
'what?' in (cast('what?' AS VARCHAR(10)), cast('foo' AS VARCHAR(10)), null, 'mellow', 'end')" => NULL
Should be:
'what?' in (cast('what?' AS VARCHAR(10)), cast('foo' AS VARCHAR(10)), null, 'mellow', 'end')" => TRUE

Merged


use smb poster / backdrop

I have all my artwork stored locally (smb) and would rather like to use those. Is this possible? I took a quick look at the source but am not sure how the caching works (in addition the locally stored artwork is in a higher res and would probably need to be down scaled before caching...)

It already works with custom cover art and backdrop images via SMB. You just have to enable it in the settings and make sure that your files follow the supported naming convention:
https://github.com/MizzleDK/Mizuu/wiki/Custom-movie-data
It only works for movies, though.

It already works with custom cover art and backdrop images via SMB. You just have to enable it in the settings and make sure that your files follow the supported naming convention:
https://github.com/MizzleDK/Mizuu/wiki/Custom-movie-data
It only works for movies, though.

It doesn't seem to be working for me.
In SmbMovie.java:
addNfoFile(MizLib.removeExtension(file.getCanonicalPath()), file.getInputStream());

While in MovieIdentification.java
            String nfo1 = ms.getFilepath().replaceAll("part[1-9]|cd[1-9]", "").trim();
            String nfo2 = MizLib.convertToGenericNfo(ms.getFilepath());

            if (mNfoFiles.containsKey(nfo1)) {
                new NfoMovie(ms.getFilepath(), mNfoFiles.get(nfo1), mContext, mCallback, mCount);
            } else if (mNfoFiles.containsKey(nfo2)) {
                new NfoMovie(ms.getFilepath(), mNfoFiles.get(nfo2), mContext, mCallback, mCount);
            } else {
                match = false;
            }

So I don't think the .nfo files will ever match (unless I'm missing something).

It doesn't seem to be working for me.
In SmbMovie.java:
addNfoFile(MizLib.removeExtension(file.getCanonicalPath()), file.getInputStream());

While in MovieIdentification.java
            String nfo1 = ms.getFilepath().replaceAll("part[1-9]|cd[1-9]", "").trim();
            String nfo2 = MizLib.convertToGenericNfo(ms.getFilepath());

            if (mNfoFiles.containsKey(nfo1)) {
                new NfoMovie(ms.getFilepath(), mNfoFiles.get(nfo1), mContext, mCallback, mCount);
            } else if (mNfoFiles.containsKey(nfo2)) {
                new NfoMovie(ms.getFilepath(), mNfoFiles.get(nfo2), mContext, mCallback, mCount);
            } else {
                match = false;
            }

So I don't think the .nfo files will ever match (unless I'm missing something).

The NFO file code is unrelated to the code that handles custom cover art :-)
On Jan 1, 2015 3:34 PM, "srirams"<EMAIL_ADDRESS>wrote:

It doesn't seem to be working for me.
In SmbMovie.java:
addNfoFile(MizLib.removeExtension(file.getCanonicalPath()), file.getInputStream());
While in MovieIdentification.java
        String nfo1 = ms.getFilepath().replaceAll("part[1-9]|cd[1-9]", "").trim();
        String nfo2 = MizLib.convertToGenericNfo(ms.getFilepath());

        if (mNfoFiles.containsKey(nfo1)) {
            new NfoMovie(ms.getFilepath(), mNfoFiles.get(nfo1), mContext, mCallback, mCount);
        } else if (mNfoFiles.containsKey(nfo2)) {
            new NfoMovie(ms.getFilepath(), mNfoFiles.get(nfo2), mContext, mCallback, mCount);
        } else {
            match = false;
        }

So I don't think the .nfo files will ever match (unless I'm missing
something).
—
Reply to this email directly or view it on GitHub
https://github.com/MizzleDK/Mizuu/issues/498#issuecomment-68487927.

The NFO file code is unrelated to the code that handles custom cover art :-)
On Jan 1, 2015 3:34 PM, "srirams"<EMAIL_ADDRESS>wrote:

It doesn't seem to be working for me.
In SmbMovie.java:
addNfoFile(MizLib.removeExtension(file.getCanonicalPath()), file.getInputStream());
While in MovieIdentification.java
        String nfo1 = ms.getFilepath().replaceAll("part[1-9]|cd[1-9]", "").trim();
        String nfo2 = MizLib.convertToGenericNfo(ms.getFilepath());

        if (mNfoFiles.containsKey(nfo1)) {
            new NfoMovie(ms.getFilepath(), mNfoFiles.get(nfo1), mContext, mCallback, mCount);
        } else if (mNfoFiles.containsKey(nfo2)) {
            new NfoMovie(ms.getFilepath(), mNfoFiles.get(nfo2), mContext, mCallback, mCount);
        } else {
            match = false;
        }

So I don't think the .nfo files will ever match (unless I'm missing
something).
—
Reply to this email directly or view it on GitHub
https://github.com/MizzleDK/Mizuu/issues/498#issuecomment-68487927.

I can't seem to get that to work either. I have a folder.jpg in the movie folder in the correct size and ratio, but it's not used.
I can't seem to find the code path for getting the image either.

I can't seem to get that to work either. I have a folder.jpg in the movie folder in the correct size and ratio, but it's not used.
I can't seem to find the code path for getting the image either.

The code is available here:
https://github.com/MizzleDK/Mizuu/blob/master/app/src/main/java/com/miz/functions/OkHttpDownloader.java
There might be a caching issue, though. If it's already loaded the local picture once, it might just ignore the one on your SMB share despite the setting for custom movie data being enabled. It's something I'll fix in a future release, but I'm awaiting an update for the image loading library that Mizuu uses.

The code is available here:
https://github.com/MizzleDK/Mizuu/blob/master/app/src/main/java/com/miz/functions/OkHttpDownloader.java
There might be a caching issue, though. If it's already loaded the local picture once, it might just ignore the one on your SMB share despite the setting for custom movie data being enabled. It's something I'll fix in a future release, but I'm awaiting an update for the image loading library that Mizuu uses.

found the problem:
s = MizLib.getCustomCoverArt(filename, MizLib.getAuthFromFilepath(MizLib.TYPE_MOVIE, imageUri), MizLib.COVER);

should be:
s = MizLib.getCustomCoverArt(filename, MizLib.getAuthFromFilepath(MizLib.TYPE_MOVIE, filename), MizLib.COVER);

found the problem:
s = MizLib.getCustomCoverArt(filename, MizLib.getAuthFromFilepath(MizLib.TYPE_MOVIE, imageUri), MizLib.COVER);

should be:
s = MizLib.getCustomCoverArt(filename, MizLib.getAuthFromFilepath(MizLib.TYPE_MOVIE, filename), MizLib.COVER);

Oh yeah, good eye! I'll test it when I can and fix it for the next minor release. Thanks for letting me know :-)

Oh yeah, good eye! I'll test it when I can and fix it for the next minor release. Thanks for letting me know :-)


some zsh themes did not work in tmux

This issue appear on CentOS, and didn't on Mac OS X.
I use default config for oh-my-zsh, tmux, powerline-status, with some themes, such as"agnoster", "terminalparty". I just add "source /usr/lib/python2.7/site-packages/powerline/bindings/zsh/powerline.zsh" to the end of .zshrc.
when use tmux, zsh with some themes, such as"agnoster", "terminalparty", and powerline-status together, the command line's prompt didn't work. If I just use zsh and powerline-status, it works OK, or just use tmux and zsh, it also works OK.
when use zsh theme "pygmalion", all is OK. I guess there is something different in themes. or I loss some additional config.
I have search around in google, it has been long time, finally I given up.
I very like oh-my-zsh tmux and powerline-status. please help check it!

Can you elaborate on "didn't work"? What's not working? Are some characters displayed incorrectly? Can you provide some example screenshots?

Firstly, it only exists in centos.
Normally, use zsh with theme "agnoster" and enable powerline, it work OK. see flow

but, when in tmux, the prompt seems not work correctlly.

ps: some theme like terminalparty appears the same issue in tmux(same prompt)

Hmmm. Looks kind of like it's running in only 8-color mode inside tmux there. Or some of the color-setting terminal escape sequences aren't getting propagated through tmux to your actual terminal, or some escape sequences are being issued wrong. The agnoster theme uses at least 16 colors, I think.
What's the value of $TERM normally, and when you're running inside tmux? And the result of echoti colors in both cases?

@apjanke Hi, I use Xshell to connect my vms. I have check the value of $TERM, default $TERM is screen-256colors, while using xshell, it changed. I think it is compatibility problem.
Thank you for you help!

Cool. I'll give Xshell a try too; I hadn't heard of it before. Thanks for the pointer.


synapse release

https://github.com/Azure/sdk-release-request/issues/3671

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-arm-synapse


Example Currently Broken due to Power BI bug

See https://community.powerbi.com/t5/Desktop/PowerBI-changes-data-table-name-to-RealTimeData-and-creates/td-p/152268 - basically, unless the table name is "RealTimeData" in the streaming analytics output, this doesn't work today.
Also, I was unaware that PowerBi didn't like nested data structures and just imported them as Text...  Might be worth adding a note to clarify that you must use a flat data structure if you are using a real device.

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

ID: 97ce2614-0e74-0f67-bf3f-8a477bd5f329
Version Independent ID: e29a2f4e-63a3-2f28-35bf-6d990f9f3b84
Content: Real-time data visualization of sensor data from Azure IoT Hub – Power BI
Content Source: articles/iot-hub/iot-hub-live-data-visualization-in-power-bi.md
Service: iot-hub
GitHub Login: @rangv
Microsoft Alias: rangv

@dominicbetts Hi Dominic, I think this issue is for one of your articles.

@rangv - Hi Ranga - this issue is for one of your articles. @sergaz-msft - can you reassign this please?

in this step
https://docs.microsoft.com/en-us/azure/iot-hub/iot-hub-live-data-visualization-in-power-bi#add-an-output-to-the-stream-analytics-job
after authorization, the save button is still greyed out so I can't save the output.

however, I tried to use the different type of output, like Blob storage or Table storage, it worked fine.

Dataset is not showing in the PowerBi dataset list.
Dataset was created in the the step which creates Output for Streaming Analytics. This has been checked for login credentials and for the path to My workspace.
Any Suggestion please!

Same here. Streaming dataset not showing in PowerBI

@delrid1, @xsqian, @ahmedhafeez, @RDSimmons01, @Busch -- Thanks for reporting these issues. We've updated the topic and the product has undergone some upgrades as well. The tutorial should work as-advertised now. Please have a look and let us know if we can close this issue.
@delrid1 -- thanks for your suggestion about adding a note wrt flat data structures. We may not add this at this time, but will monitor the topic and make a note in case of future revisions.

#reassign:jimacoms3

@deldrid1, @xsqian, @ahmedhafeez, @RDSimmons01, @Busch  Thanks again for reporting these issues. Since we haven't heard back from you, we are going to close this issue. Please feel free to re-open it if you feel the problem you reported hasn't been adequately addressed.

#please-close

Same Error. every thing is okay on IoT hub and job side but  Streaming dataset is not showing in PowerBI

Hi @Ardutec, I'm not having any trouble running this. I'm using a dataset named  ContosoSampleHubTelemetry and a data table named TelemetryData.  Everything seems to work fine.

I think we need to figure out what is different in our scenarios. Can you tell me:

Are you using Power BI online or the desktop app? (I'm using online)
What device  you are using to send data? (I would suggest trying the Raspberry Pi Online Simulator as a sanity check).
Have you verified that data is actually being arriving at your hub? You can use the az iot hub monitor-events CLI command for this.
When you say everything is OK on IoT hub and job side -- do you mean that the job starts and gets to the running state with no errors? Or?
When you say that the Streaming dataset is not showing in PowerBI. Do you mean that data is not showing up or that you cannot select the dataset to configure a line chart?

I'm not really sure that these are going to make a difference. I'm just trying to narrow down the differences between our scenarios.
Thanks,
Jimaco

What subscription model you are using? I am using free tier subscription, I think that may be difference.

I tried both desktop app and Power BI online.
I am using IIOT gateway but that is sending data on IoT hub.
NO I am not able to see in CLI monitoring.
Yes
Nothing change on Power BI, I couldn't find any data set or any other thing from Iot hub side.

I'm using a standard (S1) tier hub. The Free tier could be an issue if you have exceeded the 8k/day message limit: https://azure.microsoft.com/en-us/pricing/details/iot-hub/ ; otherwise, I don't think that is the problem.
Your answer to #3 points to a potential issue. How are you verifying that you are actually getting messages from your gateway to your IoT hub? You should be able to see these with the az iot hub monitor-events command.  Specifically, in the tutorial you create a consumer group so you should be able to do the following:
az iot hub monitor-events --hub-name {yourHubname} --cg {yourConsumerGroup}

Note: You may have to add the -g parameter to specify your hub's resource group. You can also use other parameters to format output and/or filter for a device.
Another potential issue is if you have custom routes created on your IoT hub -- for example to route every message received to storage. If that is the case, the custom route will suck the data before it ever hits the default endpoint, so you need to add a compensatory route to make sure the data also gets sent to the default endpoint.
Can you also try using the Raspberry Pi online simulator that I pointed you to in #2 above? This might help narrow down the problem.

I only use S1 tier hubs. You can’t use more than one endpoint on a free hub, so you can’t do the routing tutorial with a free hub, as noted by the big-ass note that’s in the article. So that’s fine. You might add a note as to why you don’t recommend they use a free hub. These hubs are expensive, and people will try the free hub unless you tell them not to.
I don’t have any idea what is this “gateway” that you’re talking about??
For testing the routing, I need specific content in the messages, so there’s an app that sends the messages and randomizes the content so I get what I need. It’s the one for iot-hub in the c# samples iot samples repo. The messages are routed to different destinations, and I can check the destinations to see if the whole thing is working. I don’t have any reason to use anything else, and anything else won’t result in the messages being routed correctly, so it will just confuse the customer. This is ok. It’s a routing tutorial, the messages are supposed to be routed, and this is how Ashita wanted them to be.
As this came through in a github comment e-mail, I don’t know what problem you’re talking about here. I thought the main q was about PowerBI.
Robin
From: Jimaco Brannian<EMAIL_ADDRESS>Sent: Monday, July 15, 2019 4:53 PM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Robin Shahan<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] Example Currently Broken due to Power BI bug (#10205)
I'm using a standard (S1) tier hub. The Free tier could be an issue if you have exceeded the 8k/day message limit: https://azure.microsoft.com/en-us/pricing/details/iot-hub/https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fazure.microsoft.com%2Fen-us%2Fpricing%2Fdetails%2Fiot-hub%2F&data=02|01|Robin.Shahan%40microsoft.com|cf2ca6f3ccdb45bd268108d7097f9416|72f988bf86f141af91ab2d7cd011db47|1|0|636988315856046678&sdata=DRezNoH1kiyl6NHKJL9TYSnClkRQZLID%2BG3zCnq%2BaSw%3D&reserved=0 ; otherwise, I don't think that is the problem.
Your answer to #3https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fpull%2F3&data=02|01|Robin.Shahan%40microsoft.com|cf2ca6f3ccdb45bd268108d7097f9416|72f988bf86f141af91ab2d7cd011db47|1|0|636988315856056673&sdata=skrvrzzpfTJyCXPPVPqlHa%2BcT9tF6lhVGMYG%2Ff%2FDNsM%3D&reserved=0 points to a potential issue. How are you verifying that you are actually getting messages from your gateway to your IoT hub? You should be able to see these with the az iot hub monitor-events command. Specifically, in the tutorial you create a consumer group so you should be able to do the following:
az iot hub monitor-events --hub-name {yourHubname} --cg {yourConsumerGroup}
Note: You may have to add the -g parameter to specify your hub's resource group. You can also use other parameters to format output and/or filter for a device.
Another potential issue is if you have custom routes created on your IoT hub -- for example to route every message received to storage. If that is the case, the custom route will suck the data before it ever hits the default endpoint, so you need to add a compensatory route to make sure the data also gets sent to the default endpoint.
Can you also try using the Raspberry Pi online simulator that I pointed you to in #2https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fpull%2F2&data=02|01|Robin.Shahan%40microsoft.com|cf2ca6f3ccdb45bd268108d7097f9416|72f988bf86f141af91ab2d7cd011db47|1|0|636988315856066669&sdata=nz5WsaZfMSdeI1VSQecaykQiYEgSytIG0F2QPl%2BtdsM%3D&reserved=0 above? This might help narrow down the problem.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F10205%3Femail_source%3Dnotifications%26email_token%3DADK7Q2HJLG6XFAKIXMH3M33P7UEV7A5CNFSM4FE7MCD2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZ7JBFI%23issuecomment-511611029&data=02|01|Robin.Shahan%40microsoft.com|cf2ca6f3ccdb45bd268108d7097f9416|72f988bf86f141af91ab2d7cd011db47|1|0|636988315856066669&sdata=DYFqUgekGEdFFgEZzOOlW0On3hXwmOUox%2BJHuvd%2B3%2BQ%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FADK7Q2APUMKN3MVZUL7MCJLP7UEV7ANCNFSM4FE7MCDQ&data=02|01|Robin.Shahan%40microsoft.com|cf2ca6f3ccdb45bd268108d7097f9416|72f988bf86f141af91ab2d7cd011db47|1|0|636988315856076661&sdata=eBy2slsCnHoTLBb45HDDxucwBEdqJzdgH8zs4Uam1aY%3D&reserved=0.

HI @Ardutec, let me know if any of the above helped.

@Ardutec please feel free to open a new issue and reference this one if you are still blocked. Thanks!


[Mobile] Custom icons from assets and change its theme

Is there any way to add customIcons with icon from assets and change its theme? while QuillCustomIcon existing only accept IconData and the other property is onTap only

We recently have added a lot of options to customize the widgets
This issue has been fixed, if you still facing this issue or any others, feel free to contact us on GitHub


Migrate timetable

You need to change the channel and flutter_version in/github/workflows files so the Code Review step doesn't fail.


use bibtex in maxwell.py

Convert references in mne/preprocessing/maxwell.py to use footcite / footbibliography.

I'll take this issue :)


changed repo structure to include setupy.py and simplify example folders

You can install topotherm thorugh:
python setup.py install
Now you can without being in the main folder. Therefore the folder structure of the example has been simplified. This also allows a publication to pypi further down the line and separate the preprocessing and optimization further in notebooks or similar. I tested the single time step.
If everything works as expected, we should rewrite the readme and publish version 1.0.0.

PS: example/results has not been included due to the gitignore, we should think about including it or not. I would prefer not to reduce the size of the package.


Convert country status

i.e. Great Power, Unrecognized, Decentralized

After thinking on it more, decentralized is the only one I want to carry forward into HoI4 for now. Great Power will naturally fall out but shouldn't get bonuses, and the recognition of who was a great power changed in this time period or shortly after. Likewise unrecognized powers.


panic running security-secretstore-setup

🐞 Bug Report
Running security-secretstore-setup with res/configuration.toml missing causes a panic due to undefined LoggingClient.
See:
./security-secretstore-setup --init=true
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x20 pc=0x8a5ae4]

goroutine 17 [running]:
github.com/edgexfoundry/edgex-go/internal/pkg/config.LoadFromFile(0x0, 0x0, 0x0, 0x0, 0x8e1480, 0xc0000b3d40, 0x5d8a5505, 0xc0223daddb)
        /home/ijohnson/git/edgex/edgex-go/internal/pkg/config/loader.go:52 +0x214
github.com/edgexfoundry/edgex-go/internal/security/secretstore.initializeConfiguration(0xbf5ab321623dad00, 0x0, 0x0, 0x0, 0x0, 0xe47d20, 0xe47d01, 0x1)
        /home/ijohnson/git/edgex/edgex-go/internal/security/secretstore/init.go:68 +0x78
github.com/edgexfoundry/edgex-go/internal/security/secretstore.Retry(0xc000140100, 0x0, 0x0, 0x0, 0x0, 0x7530, 0xc00013c050, 0xc0001401e0)
        /home/ijohnson/git/edgex/edgex-go/internal/security/secretstore/init.go:37 +0x131
created by github.com/edgexfoundry/edgex-go/internal/pkg/startup.Bootstrap
        /home/ijohnson/git/edgex/edgex-go/internal/pkg/startup/bootstrap.go:37 +0xe4

This is because the LoggingClient in LoadFromFile is nil because we haven't been able to load the configuration file at this point in time yet.
Affected Services
The issue is located in: security-secretstore-setup
Is this a regression?
Maybe...
This was from master @ abc9af1de59ab5c4130efe3914aaf2c6688b4e14

I wonder if this could be deferred until after https://github.com/edgexfoundry/edgex-go/issues/1806 (I suspect https://github.com/edgexfoundry/edgex-go/issues/1806 will address the underlying issue).

Sure, sounds fine to me to wait for that (I assume that is queued up soonish)

(I assume that is queued up soonish)

I expect it to be so.

Does this need to be fixed in Fuji, and is it really the operators fault for running without a configuration.toml?  (When is running without a configuration toml supported?)

It's a bad user experience to have a panic show up like this. It's probably not critical for Fuji, but still it would be nice to fix

This issue is still happening with security-proxy-setup:
Oct 31 14:57:14 systemd[1]: Starting Service for snap application edgexfoundry.security-proxy-setup...
Oct 31 14:57:14 edgexfoundry.security-proxy-setup[73554]: panic: runtime error: invalid memory address or nil pointer dereference
Oct 31 14:57:14 edgexfoundry.security-proxy-setup[73554]: [signal SIGSEGV: segmentation violation code=0x1 addr=0x20 pc=0x8a6574]
Oct 31 14:57:14 edgexfoundry.security-proxy-setup[73554]: goroutine 6 [running]:
Oct 31 14:57:14 edgexfoundry.security-proxy-setup[73554]: github.com/edgexfoundry/edgex-go/internal/pkg/config.LoadFromFile(0x7ffd63cf2c29, 0x39, 0x0, 0x0, 0x8f3380, 0xc0001103c0, 0x5dbb3c9a, 0xc025f01a47)
Oct 31 14:57:14 edgexfoundry.security-proxy-setup[73554]:         /root/parts/edgex-go/src/internal/pkg/config/loader.go:52 +0x214
Oct 31 14:57:14 edgexfoundry.security-proxy-setup[73554]: github.com/edgexfoundry/edgex-go/internal/security/proxy.initializeConfiguration(0xbf66ed06a5f01a00, 0x7ffd63cf2c29, 0x39, 0x0, 0x0, 0xe6ae40, 0xe6ae01, 0x1)
Oct 31 14:57:14 edgexfoundry.security-proxy-setup[73554]:         /root/parts/edgex-go/src/internal/security/proxy/init.go:69 +0x78
Oct 31 14:57:14 edgexfoundry.security-proxy-setup[73554]: github.com/edgexfoundry/edgex-go/internal/security/proxy.Retry(0xc0000a6400, 0x7ffd63cf2c29, 0x39, 0x0, 0x0, 0x7530, 0xc000026490, 0xc0000a6480)
Oct 31 14:57:14 edgexfoundry.security-proxy-setup[73554]:         /root/parts/edgex-go/src/internal/security/proxy/init.go:38 +0x131
Oct 31 14:57:14 edgexfoundry.security-proxy-setup[73554]: created by github.com/edgexfoundry/edgex-go/internal/pkg/startup.Bootstrap
Oct 31 14:57:14 edgexfoundry.security-proxy-setup[73554]:         /root/parts/edgex-go/src/internal/pkg/startup/bootstrap.go:37 +0xe4
Oct 31 14:57:14 systemd[1]: snap.edgexfoundry.security-proxy-setup.service: Main process exited, code=exited, status=2/INVALIDARGUMENT
Oct 31 14:57:14 systemd[1]: snap.edgexfoundry.security-proxy-setup.service: Failed with result 'exit-code'.

Anyway we can port the fix for this issue to security-proxy-setup for Fuji still?

Just so it is more obvious on where to look.  For security-secretstore-setup it was fixed in this commit:  https://github.com/edgexfoundry/edgex-go/commit/72ebc617e5544205d3e97b6458656f707fb7934c


exports Table interface

Exports the Table interface so that developers can import it and type their table objects properly before calling the tables function.

This change is already in https://github.com/natancabral/pdfkit-table/pull/55


This commit makes mithril stop working on android 2.2

Hello !
I'm testing mithril an discovered that it's not working on android 2.2/2.3 and probably others because of this commit:
https://github.com/lhorie/mithril.js/commit/dfb37fec9cd25ca5f18282cbe97120dc69dad43a
That commit uses a reserved word "catch" that's rejected by android 2.2/2.3 and probably others.
Changing that property name to any non reserved keyword seems to make mithril work again on android 2.2/2.3 .
P.S.: With the minified mithril it works because the minifier replace the "catch" by a short name.
Cheers !

I think this can be fixed by just making it prop["catch"] = // ...
I don't have an Android 2.2/2.3 to test it, though.
Using reserved words as property names is a ES5 feature and unfortunately, unpolyfillable. Maybe this changes the claim (Vue section) about everything used by Mithril can be polyfilled.

Changed to the bracket syntax


⚠ Bofrost.at has degraded performance

In 2e8f1e9, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1336 ms

Resolved: Bofrost.at performance has improved in 29af34e.


Brand color in #RRGGBB

Following the the how to get listed and I dont understand point 8: brand color
https://github.com/irisli/stellarterm/tree/master/directory#8-brand-color-in-rrggbb
How is this to be added in the stellar.toml file?
Other than that I am good to go, do I paste my details here?
regards
Matt

I made my requested reddit post https://www.reddit.com/r/Stellar/comments/895edj/xim_export_import_token_stellar_adoption/ and I believe my toml file is in order. https://ximcoin.com/.well-known/stellar.toml how do I proceed with getting listed?


add sort_by macro method

{{ ["abc", "a", "ab"].sort_by { |string| string.size } }}

@bcardiff done :) Sorry I forgot about that. I was writing a kind of model interface similar to json mapping and was building an initializer that would take arguments from the mapping named tuple. There i needed to sort the arguments by nilable so that args that are mandatory are leftmost.


Server Crash calling meBridge.getAvailableFluidStorage()

Describe
When calling meBridge.getAvailableFluidStorage() the server crashes.
Happens both on a server and in singleplayer
The included log is from singleplayer where I reproduced the issue.
I have requested the multiplayer server log, but due to time zones don't have a response yet.
Steps to reproduce
Setup a trivial ME network: I used a ME drive with any storage cell inside (kind does not matter). A power source is not required. Attach a meBridge to the drive, and a computer to the bridge.
On the computer open a lua prompt and enter:
peripheral.find('meBridge').getAvailableFluidStorage()
The server will crash.
Multiplayer?
Yes
Version
1.19.2-0.7.28r
Minecraft, Forge and maybe other related mods versions
Minecraft: 1.19.2, Forge: 43.2.14
Screenshots or Videos
No response
Crashlog/log
https://pastebin.com/bMxGR0Tw (0.7.28r)
https://pastebin.com/uZu5CZ9R (0.7.27r)

This bug should be fixed in the newest AP Version (1.19.3-0.7.28r)

Would it be possible to fix it also for minecraft 1.19.2 ?
I looked into updating to 1.19.3 but many mods in the pack don't have a version that supports it and break.

Why did you select 1.19.3 if you play on 1.19.2?
The newest 1.19.2 should also have this fix, but I can quick check that for you

Oh sorry looks like I selected the wrong version. I really thought I double checked it too 🥴.
I'll look through the issue again now to make sure everything is correct.

I just retested with 1.19.2-0.7.28r and get crash, the stack trace also appears the same at a quick glance. I updated the main post to reflect the version numbers of this test.

So the issue is that if you don't have ae2things installed, the function will lead to a crash
Could you try this version? Unzip the file before using it
AdvancedPeripherals-1.19.2-0.7.28r.zip

It works 🎉
That was fast, thanks!

Fixed in 1.19.2-0.7.29r


Can't get it to work

Hey, I'm using Windows 10 1709 and VS Code 1.17.2.
I renamed wslgit.exe into git.exe and added it to the PATH variable.
VSCode settings
   "git.enabled": true,     "git.path": "C:\\Users\\<username>\\AppData\\Local\\Git\\git.exe",
I can use git with PowerShell, CMD and of course Bash for Ubuntu, but not in VS Code.

Are you using an old WSL Beta environment (Bash on Windows) or did you install the Linux Distribution from the Store (which is the new way since the Fall Creators Update)?
If C:\Users\<username>\AppData\Local\Git\git.exe --version works in PowerShell or in CMD, then you probably have an old style environment, because wslgit currently does not support the new WSL Linux environments.
What exactly does not work in VS Code? Any errors? Does it detect the repository if you open a version controlled directory?
Could you post the output of the Git console in VS Code?

I think it's the old Beta, because it's called "Bash on Ubuntu on Windows". I'll try it with the Ubuntu Distribution from the Microsoft store.
VS Code just doesn't recognize that the project has git.

Ok. However, it won't work with the new version from the store, as they use different executables to call into the WSL environment. I haven't implemented support for them, currently.
Can you post the output of the git console from VS Code? It should contain the git commands VS Code tries to run to detect the git repository, etc.
You can then run the same commands in a CMD shell to check their outputs.

Cool.

I just released a new version that should hopefully fix this. Please try the latest release here:
https://github.com/andy-5/wslgit/releases/tag/v0.3.0

I'll try it this evening and give you some feedback.

JFYI: Your fix is working.

Great, thanks for reporting back.


Improve chainspec

Make fields easier to understand and remove any than can be derived from others.

Won't do.


Project dead?

No commits since 10 months, no updates. Is the project dead?

Hi,
I proposed some help as a maintainer and John accepted it. So I'll do my best to prepare a bugfix release soon.
Olivier

4.2.10 is out :)
I'll prepare a release of mediaelement-plugins asap

5 years later, there are many issues & PRs for both repositories (core, plugins). Is mediaelement still a good choice and will be maintained?


Update AbstractStepFragment.kt

Was unable to fix talkback work here, but colon look much less ugly than a semicolon.

Maybe. But now it resembles clock. I will think about other options, e.g. ˋ´•|* etc


Should be a share path

Followup for
https://github.com/MicrosoftDocs/win32/pull/338
@Karl-Bridge-Microsoft

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

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

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

@Karl-Bridge-Microsoft you ruined it again :( \\symbols is a share path, it should start with two backslashes.

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

#sign-off


[feature] add client demo for sending request to easyrec processor

add client demo for sending request to easyrec processor
refactor processor test code

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


method attachFile shown in wiki does not appear to exist

Thank you for your excellent package modelizer/laravel-selenium.
I am trying to test an uploaded file with:
            ->see(sprintf('Add an Image to %d', $this->Lodge->lodge_number))
            ->attachFile('image',  __DIR__.'/lodge81.jpg')
            ->type('My Image Description for '.$this->Lodge->lodge_name, 'description')
            ->press('Add Image!')

The error I get is:
You cannot call a command with multiple method arguments.
The traceback points to the line containing ->attachFile
I am running modelizer/selenium 1.2.2.
I looked through my entire vendor directory, and cannot find attachFile.
What do I need to do?

Thanks for raising this issue but as far as I know we don't have attachFile method. Please feel free to submit a PR.
Thanks. :)

Shaikh, I agree that you don't have it in the code, but you do have it in the wiki for the project, along with the other methods.
If we don't have this method, what is the correct way to test uploading a file through the browser?
PS I don't know what a PR is.

Okay, I will look into this issue. It will be added in 2.0 version of this package.


Load slot-level detail in second pass for Rite Aid scraper

The Rite Aid web UI now makes two requests for each zip code — the first one returns a list of stores, in mostly the same format as before. The difference is that the array of slots in it is now empty and it has a firstAvailableSlot property. You then have to make a second call to the same endpoint, but with a list of store numbers in the querystring, to get slots filled in.
The first two changes make the code able to handle both the new summary/non-slot data and to optionally make extra requests for the slot-level data. (It gets the slot data by default, but it’s a function argument so we can easily turn it off if needed.)
The next change adds fairly strict validation of the returned data with AJV. My thinking here is to avoid a subtle change like the one that precipitated this — we’d get notifications that firstAvailableSlot was added, and would know to take a look at what’s going on. (That said, this one would still be subtle to understand the important of — availableSlots is still there, and still an array, it’s just empty. But I think this is the best we can do for proactive notification.)
Fixes #528.

This is a work-in-progress. I need to clean up the AJV validation code (the schema is good, but it needs a light-weight wrapper to make setup and throwing exceptions nicer).

OK, should be ready to go.

Yeah, in hindsight, we should definitely have been doing something like this more.


Latest update broke something

The latest updates seems to have broken something.
Now every page is getting detecting, using a regex like /.*test.*/ for Dangerous keywords

Just facepalmed so hard, introduced another bug while fixing another bug. Epic fail. Fixing ASAP!

This should be fixed in the latest version (2.16) now. Please  re-open if it is not.

Thanks, working properly now.


kernel: add abstractions and a memory implementation

This PR is based on https://github.com/engula/engula/pull/153. Check #143 for example usage of Kernel.
Close #149 #146
I realized that maybe we can generalize a local kernel and provide a memory kernel and file kernel as type alias. For example:
pub mod local {
    pub struct Kernel<S: Stream, B: Bucket> { ... }
}

pub mod mem {
    pub use engula_journal::mem::Stream;
    pub use engula_storage::mem::Bucket;
    pub type Kernel = super::local::Kernel<Stream, Bucket>;
}

pub mod file {
    pub use engula_journal::file::Stream;
    pub use engula_storage::file::Bucket;
    pub type Kernel = super::local::Kernel<Stream, Bucket>;
}

Maybe we can try it in next PR. I have already added the idea in this PR :)

@zojw @tisonkun This PR is ready. And I think other tasks for v0.2 depend on the Kernel added here. So please help to review this when possible so that we can move forward :)

@huachaohuang thanks for your reminder! I'm reviewing this now.

OK, let's merge this and call it a day :)


PLAT-84902: Create LabeledSlider

Issue Resolved / Feature Added
I created LabeledSlider for Settings App and Volume App.
The LabeledSlider allows decrementIcon and incrementIcon props for icons and decrementText and incrementText props for texts. In addition, the slider could have icon or text elements as children prop instead of those props so app developers to customize styles for the icons or the texts.
Links
PLAT-84902
Comments
GUI problem
When dragging the knob of LabeledSlider to the rightmost or the leftmost, then the knob positions over the icons or the texts. This is the GUI problem. I'm still discussing it with the GUI developer. I'll fix it in another seperate issue.
Additional work
I removed IncrementSlider/utils.js which might not be used anywhere.
I fixed the lint for agate/ProgressBar.
Enact-DCO-1.0-Signed-off-by: YB Sung<EMAIL_ADDRESS>

No more available.


feat: read default exports from a plugin if it exists

In order to allow plugins to export values other than the monkeypatch itself, this PR makes the plugin loader accept the default exported value if it exists.
Also, this adds a top-level export defaultConfig, which exposes the default configuration options.

Closed in favor of #826.


How to pass two value rule as object?

how can I pass this as an object? It's especially the true value at the after rule that I'm confused about.
v-validate ="'required|date_format:YYYY-MM-DD|after:startDate,true'"
Tried an array but got an error. How can I pass this as an object?

<input name="field" type="text" v-validate="{ required: true, date_format: 'YYYY-MM-DD', after: ['startDate', true] }">

Note that there was a bug regarding array params #752 which should be fixed in the next release.

ha, that was exactly my problem - thanks!


Angular.js + AlaSQL on Safari

Thank you for the great work on AlaSQL, it has made my life easy for exporting angular scope variables to xls.
I am utilizing the technique found here : https://github.com/agershun/alasql/wiki/Angular.js,  but I have run into a bit of snag. My client is using the export feature on Safari but it throws an error. Same error occurs on the example fiddle (http://jsfiddle.net/agershun/00nfeq12/)
I have added two scripts (https://github.com/SheetJS/js-xls/blob/master/shim.js & https://github.com/eligrey/Blob.js/) that removes the error, however on exporting , Safari now opens in a new window with a data:text/plain encoding
Any help would be greatly appreciated.

ITs one of the oldest unresolved issues: https://github.com/agershun/alasql/issues/27
In short: safari limits you from downloading generated files thats not plain text. If you can use .csv that will solve the problem...


Checking out C# shared project doesn't check out projitems file

Can't see any obvious way to detect that this file is related to the .shproj. Probably needs handling in the same way as the .filters file?
PR to follow hopefully.

Predictably, this is proving harder than expected, as the projitems file isn't actually part of the project. It appears in any dependent projects (???!) - but it'd be nice for this to work in all cases, even if you're working with the shared project independently.
I will probably add in a check for a matching projitems file if the project's file name matches *.shproj. Or maybe just get p4 edit to check a projitems file out anyway, and Perforce can deal with it.


Rendezvous and demonstration on Spsc_queue API

This is a small proposal to avoid spinlocks in this repository by introducing some mean of synchronizing domains. I think it would help users if the APIs provided facilities for this common pattern, while working well with fibers and custom schedulers (so not just system locks in place of spinlocks).
I did some experiment on the Spsc_queue benchmark where a spinlock is used for the consumers/producers when the queue is empty/full. Currently this is absurdly slow if the two domains happen to be scheduled on the same CPU core: it takes up to 3s to push/pop a thousand elements, rather than milliseconds on two cores! (.. this is why the CI benchmarks don't terminate in any reasonnable time atm, you can try it with taskset --cpu-list 1 ./_build/default/bench/main.exe but I recommend lowering the items counts first)
Adding a couple of external semaphores to this benchmark solves the single core worst-case for spinlocks, but at the price of being 4x slower in general. By integrating the locking logic into the datastructure, we can achieve nearly no observable regression in performances while also providing an API that don't encourage users to spinlock. It's debatable if the resulting datastructure is still "lockfree", but at least the locking is opt-in :-°
This PR builds on a previous experiment in domainslib where the goal was to synchronize fibers with an Mpmc_queue, by using a "rendezvous" lock between matching pop and push operations. Essentially we want some mean for a pushing domain to unlock a stuck poping domain (which happens when the queue is empty, but the user has signified that it really wants to wait). Dually if the queue was full, we want the next pop to unstuck a waiting pushing domain. By exploiting the custom scheduler, locking is cheap since the domain can keep working on other tasks.
... Now this is just a quick demo to gather feedback on the proposed Rendezvous abstraction, there's still a bunch of spinlocks to remove from this repo if there's interest. Some questions:

This PR example only uses a unit Rendezvous.t, so perhaps the rendezvous polymorphism is not required and could be added on the side on a case-by-case basis? I remember that it was nice for the Mpmc_queue in domainslib, and the type is a bit easier to read than with unit everywhere (?)
I tried a rendezvous type (('a -> unit) -> unit) -> 'a since @polytypic used something similar in kcas, but it was less convenient to handle the edge-case "I thought I needed to lock but don't anymore"... The callback pair is less fancy but easier to understand, maybe (?)
Should the Rendezvous type definition be public? I did so with the hope that other datastructure libraries could adopt the pattern and custom schedulers would be able to provide implementations, without requiring an explicit dependency on lockfree just for this type, but perhaps that won't be the case... We could expose standard synchronizations methods like mutexes instead then.

A quick note: I currently strongly feel this sort of thing should be put into a minimal library of its own.  IOW, I would not put it into lockfree (nor kcas for that matter).  The ability to suspend and later resume seems to be useful/needed for a lot of low level facilities and would likely be implemented by any and all libraries providing schedulers of some sort that want to co-operate.

perhaps the rendezvous polymorphism is not required and could be added on the side on a case-by-case basis?

It is difficult to say, but generally speaking, there are likely to be a lot of cases where the ability to pass value is not needed (only the suspend/resume is needed), and it usually requires a bit more work (an allocation or two) to make it possible to pass a value.

I currently strongly feel this sort of thing should be put into a minimal library of its own.

Yeah I don't have an opinion on this.. It really depends on whether custom schedulers and datastructure libraries need a lot more than an alias for the type unit -> ('a -> unit) * (unit -> 'a) (well, datastructures will need some default implementations of this type when testing, but schedulers will not)

it usually requires a bit more work (an allocation or two) to make it possible to pass a value.

Indeed, that's why I added some specialized _unit to show that it shouldn't be too much of an issue (but either way, I'm not seeing a performance difference in my tests, as we only allocate extra stuff when we have to block)

To quote your example from kcas on continuations vs products for the rendezvous type:
scheduler (fun resume ->
    if not (Atomic.compare_and_set self `Init (`Waiting resume))
    then resume ());

vs
let resume, wait = scheduler () in
if Atomic.compare_and_set self `Init (`Waiting resume)
then wait ()

A small issue I had with the continuation is that you need to call resume () in the happy case when you don't need to lock (so it's slightly awkward for the scheduler which now need to check if blocking is necessary or not after the callback completes)

I currently strongly feel this sort of thing should be put into a minimal library of its own.

Yeah I don't have an opinion on this..

I'm thinking this more in terms of having the capability available without explicit parameterization.  One approach would be to use the DLS as I suggested previously.  This way one could just use libraries like kcas with blocking — no matter how many different schedulers are being used in the program things would just work (as long as the schedulers just install the mechanism to DLS when running their event loops).  Passing things through parameters or functorizing everything easily becomes a mess — althrough it definitely has the advantage of not using ambient state.  Also, if every library would have slightly different definitions (one would use rendezvous, another would use the CPS suspend, third would use a Suspend effect), then it would become much less practical to e.g. use DLS.

A small issue I had with the continuation is that you need to call resume () in the happy case when you don't need to lock (so it's slightly awkward for the scheduler which now need to check if blocking is necessary or not after the callback completes)

Hmm... Yes, this could be an advantage of breaking the suspend/resume protocol into two like this.  So, IIUC, basically, in an effects based implementation it would be the wait implementation that actually performs the effect to capture the stack.  By making wait a simple first-order function, things potentially become simpler.

@art-w you scared me, but the benchmarks are run properly and the graphs are present! Link here. The problem is that these benchmarks have the name "Lockfree" and hence aren't shown when just clicking on the PR, you also need to click on Lockfree here.

However the graphs aren't particularly useful as there is nothing to compare to, but they're present!

Also mentioning here that the domain-local-await or DLA library is now available as an opam package and provides a blocking mechanism inspired by the "rendezvous" primitive described in this PR.  I plan to soon release a version of kcas using DLA and there are PRs to add DLA support to Eio and Domainslib.  As described in the DLA library README it may be replaced later by an official blocking mechanism, but I hope that we can use it now to experiment providing blocking abstractions that from the POV of casual users  just work ™ with OCaml 5 today.

@art-w : as we are not moving to blocking data structures for now, can we close this PR ?

Yes of course! The ideas have been integrated in DLA and Picos now, this PR is obsolete :)


Redundant fields section when no extra fields in link drawer

Link to reproduction
No response
Describe the Bug
The highlighted title Fields is redundant in link drawer of text editor:

To Reproduce
Open any richText editor that enables link.
Payload Version
2.11.2
Adapters and Plugins
richtext-slate

I made a check on the extraFields' length and fixed it locally:
https://github.com/kpkonghk01/payload/commit/28a00fac7f37a20973ee83154bda026af10e6217