cupsd.conf MaxSubscriptions* directives not documented

It seems the following cupsd.conf directives are not documented
MaxSubscriptions
MaxSubscriptionsPerJob
MaxSubscriptionsPerPrinter
MaxSubscriptionsPerUser

because in current CUPS GitHub master sources they appear only in those files
scheduler/subscriptions.c
scheduler/subscriptions.h
scheduler/conf.c
test/4.4-subscription-ops.test
test/run-stp-tests.sh

Is there a reason why they are not documented?
E.g. are they perhaps not meant to be set by the user?
FYI background information:
We (i.e. SUSE) got the following customer issue report (excerpts):
While troubleshooting a cupsd problem (print jobs did not get
forwarded to Windows Print Server / lpstat ran into timeouts etc.)
we found in /var/log/cups/error_log the following lines :

D ... cupsdAddSubscription: Reached MaxSubscriptions 100 (count=100)

D ... Create-Printer-Subscriptions client-error-too-many-subscriptions: There are too many subscriptions.

E ... Returning IPP client-error-too-many-subscriptions for Create-Printer-Subscriptions (/) from x.x.x.x

There is a MaxSubscriptions directive,
which is unfortunately not explained in cupsd.conf(5),
but increasing the same plus MaxClients resolved the problem

These should be documented, and it is possible they never got added to the man page (previously we focused on the HTML documentation). But the directives have been there since CUPS 1.2...

[master 10e4edb35] Add missing MaxSubscriptionsXxx docos (Issue #177)

@michaelrsweet
it seems https://github.com/OpenPrinting/cups/commit/10e4edb359ecc20782c99fe6d0a4b5dbbc88ac09
does not add the missing directives but replaces existing directives.
For example it seems MaxJobs got replaced by MaxSubscriptions
because in a current git clone from today I have
# find . -type f  | xargs grep -il 'MaxSubscriptions'
./scheduler/subscriptions.c
./scheduler/subscriptions.h
./scheduler/conf.c
./man/cupsd.conf.5
./doc/help/man-cupsd.conf.html
./test/4.4-subscription-ops.test
./test/run-stp-tests.sh

# find . -type f  | xargs grep -il 'MaxJobs'
./scheduler/job.h
./scheduler/conf.c
./scheduler/job.c
./scheduler/ipp.c
./cgi-bin/admin.c

while in a git clone from some days before I have
find . -type f  | xargs grep -il 'MaxSubscriptions'
./scheduler/subscriptions.c
./scheduler/subscriptions.h
./scheduler/conf.c
./test/4.4-subscription-ops.test
./test/run-stp-tests.sh

# find . -type f  | xargs grep -il 'MaxJobs'
./scheduler/job.h
./scheduler/conf.c
./scheduler/job.c
./scheduler/ipp.c
./cgi-bin/admin.c
./man/cupsd.conf.5
./doc/help/man-cupsd.conf.html

Ugh, I must not have checked "in selection for the find/replace I did when copying MaxJobsXxx to MaxSubscriptions. Fixed:
[master eda69a24c] Fix errant find/replace that messed up the jobs documentation (Issue #177)


Create Segmentation SDK structure for source and unit tests.

Please wait for Ryan/Paul review before merging.

If you like, the CI integration can wait until the PR to merge to master.
To add to CI for md-floc-master add the branch name to the bottom of .travis.yml.

The diff is unmanageable now. One solution might be to merge master into md-floc-master.
If you haven't yet, review https://github.com/firebase/firebase-ios-sdk/blob/master/AddNewPod.md

The diff is unmanageable now. One solution might be to merge master into md-floc-master.
If you haven't yet, review https://github.com/firebase/firebase-ios-sdk/blob/master/AddNewPod.md

I did merge into master. Will do a rebase next time.


"index out of range" error in Go BWT Forward transform

Kanzi --compress --input=bwt2.bin --entropy=none --transform=bwt --force
Kanzi 1.1 (C) 2017,  Frederic Langlet
Encoding ...
panic: runtime error: index out of range
goroutine 5 [running]:
kanzi/transform.(*BWT).Forward(0xc42000e900, 0xc4201f2000, 0xcccc, 0xcccc, 0xc420200003, 0xcccd, 0xcccd, 0x3, 0xc420042c80, 0x40cd6d, ...)
/home/user/go/src/kanzi/transform/BWT.go:146 +0x249
kanzi/function.(*BWTBlockCodec).Forward(0xc42000c0a8, 0xc4201f2000, 0xcccc, 0xcccc, 0xc420200000, 0xccd0, 0xccd0, 0x100, 0x0, 0x101000000000000, ...)
/home/user/go/src/kanzi/function/BWTBlockCodec.go:73 +0x110
kanzi/function.(*ByteTransformSequence).Forward(0xc42000ad20, 0xc4201f2000, 0xcccc, 0xcccc, 0xc420200000, 0xccd0, 0xccd0, 0x0, 0x0, 0x0, ...)
/home/user/go/src/kanzi/function/ByteTransformSequence.go:85 +0x1eb
kanzi/io.(*EncodingTask).encode(0xc4200d82a0)
/home/user/go/src/kanzi/io/CompressedStream.go:466 +0xb97
created by kanzi/io.(*CompressedOutputStream).processBlock
/home/user/go/src/kanzi/io/CompressedStream.go:391 +0x39d
bwt2.bin.gz

Interestingly this issue happens in the Java implementation as well but not the C++ one.

The root cause is 2 variables initialized in the wrong order in trPartialCopy() in the port of DivSuftSort.

This one was hard to locate. Hopefully the last one. Fixed in commit 79b3c36.


[BUG] Plasmo refuses to work with relative imports in package.json

What happened?
Starting since v47, it seems like Plasmo will refuse to work if the react dependency is declared as a relative import in the package.json file.
Version
Latest
What OS are you seeing the problem on?
Linux
What browsers are you seeing the problem on?
No response
Relevant log output
><EMAIL_ADDRESS>dev
> plasmo dev

🟣 Plasmo v0.50.1
🟠 The browser extension development framework.
🔵 INFO   | Starting the extension development server...
🔵 INFO   | Loaded env from /home/kuro/Celtiberian/hiring/new-chrome-extension/.env.local
🔴 ERROR  | Invalid Version: file:../resume-ranker-frontend/mysite/app/react/node_modules/react
          | TypeError: Invalid Version: file:../resume-ranker-frontend/mysite/app/react/node_modules/react
    at new SemVer (/home/kuro/Celtiberian/hiring/new-chrome-extension/node_modules/plasmo/node_modules/semver/classes/semver.js:38:13)
    at Object.major (/home/kuro/Celtiberian/hiring/new-chrome-extension/node_modules/plasmo/node_modules/semver/functions/major.js:2:29)
    at Vo (file:///home/kuro/Celtiberian/hiring/new-chrome-extension/node_modules/plasmo/dist/index.js:59:9215)
    at Lt.#m (file:///home/kuro/Celtiberian/hiring/new-chrome-extension/node_modules/plasmo/dist/index.js:59:11266)
    at Lt.updatePackageData (file:///home/kuro/Celtiberian/hiring/new-chrome-extension/node_modules/plasmo/dist/index.js:59:11227)
    at async At (file:///home/kuro/Celtiberian/hiring/new-chrome-extension/node_modules/plasmo/dist/index.js:59:16519)
    at async ol (file:///home/kuro/Celtiberian/hiring/new-chrome-extension/node_modules/plasmo/dist/index.js:59:17593)
    at async Dl (file:///home/kuro/Celtiberian/hiring/new-chrome-extension/node_modules/plasmo/dist/index.js:59:22354)
🔴 EXIT   | 👋 Good bye and have a great day!

(OPTIONAL) Contact Details
<EMAIL_ADDRESS>Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I checked the current issues for duplicate problems.

This was done in https://github.com/PlasmoHQ/plasmo/releases/tag/v0.47.1 to support other UI library scaffolding. Is there a way to check the version of the react package coming from relative path like in your project? At the moment, plasmo assume semver so it parse and throw error. We can def read the package.json at the location of the import path, and try to get the version number there?
I'm travelling atm, but will get a PR up and fix to canary in a bit!

@Kurolox can you try canary and see if the issue get resolved?

@Kurolox can you try canary and see if the issue get resolved?

I've just tried it, unfortunately I'm facing a different error now.
🟡 STEP 0 | Running command: dev
🟣 Plasmo v0.51.0-alpha.2
🟠 The browser extension development framework.
🔵 INFO   | Starting the extension development server...
🟡 STEP 1 | Making sure .plasmo exists
🟡 STEP 2 | Make sure generated assets directory exists
🟡 STEP 3 | icon512.png found, create resized icons in gen-assets
🟡 STEP 4 | Creating Extension Manifest...
🔵 INFO   | Loaded env from /home/kuro/Celtiberian/hiring/new-chrome-extension/.env.local
🔴 ERROR  | EISDIR: illegal operation on a directory, read
          | Error: EISDIR: illegal operation on a directory, read
🔴 EXIT   | 👋 Good bye and have a great day!

@Kurolox can you try again (reinstall canary), it should be alpha.3

Now it works properly! But I have an issue, which might be unrelated.
Now everything seems to work properly, however under the built files the manifest file is called chrome-mv3.plasmo.manifest.json, rather than manifest.json, so Chrome isn't able to load the unpacked extension until I manually rename the manifest file.
I don't really think that it's related to this current issue, but I'd love your insight. I can open a different issue if so.

Argh! I'm on it rn

@Kurolox try latest canary! Just found the issue with how parcel bundle naming mechanism work.

Latest canary works flawlessly! Thanks for all the effort in fixing this


Agency endpoints created

Codecov Report
Attention: Patch coverage is 7.09220% with 131 lines in your changes missing coverage. Please review.

Project coverage is 30.17%. Comparing base (d5d0d9e) to head (30ee8f8).

Files
Patch %
Lines

...pliance-api/src/compliance_api/resources/agency.py
0.00%
69 Missing :warning:

...mpliance-api/src/compliance_api/services/agency.py
0.00%
37 Missing :warning:

...ompliance-api/src/compliance_api/schemas/agency.py
0.00%
16 Missing :warning:

...ance-api/src/compliance_api/services/staff_user.py
0.00%
5 Missing :warning:

...iance-api/src/compliance_api/resources/__init__.py
0.00%
2 Missing :warning:

...pliance-api/src/compliance_api/schemas/__init__.py
0.00%
1 Missing :warning:

...liance-api/src/compliance_api/services/__init__.py
0.00%
1 Missing :warning:

:exclamation:  There is a different number of reports uploaded between BASE (d5d0d9e) and HEAD (30ee8f8). Click for more details.
HEAD has 1 upload less than BASE

Flag
BASE (d5d0d9e)
HEAD (30ee8f8)

compliance-web
1
0

Additional details and impacted files
@@             Coverage Diff              @@
##           develop      #11       +/-   ##
============================================
- Coverage    42.81%   30.17%   -12.65%
============================================
  Files           38       29        -9
  Lines          334      822      +488
  Branches        61        0       -61
============================================
+ Hits           143      248      +105
- Misses         178      574      +396
+ Partials        13        0       -13

Flag
Coverage Δ

compliance-api
30.17% <7.09%> (?)

compliance-web
?

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Char horizontal tabulation (Unicode: 9) is rendering on iOS but not on Android

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce
In project tnc is fetched from server and display in app screen. The tnc is text (multi line text) has many horizontal tabulation characters (its unicode is 9) used for spaces. In iOS it is properly rendered as space but in Android its rendering as empty boxes when used in Text/RichText component with custom font 'Kanit'
Some more info:

If fontFamily: 'Kanit' removed from app ThemeData then it works, but that font is required for our project
In our project for Flutter v2.5.2 it working properly for both platforms, but for Flutter v3.0.2 only working in iOS. In given sample app tried for both mentioned version, not working in Android for both versions.
As per this issue thread #111377 tried in v3.3.2 but still showing boxes in Android.

Expected results
In Android it should render same as iOS
Actual results
In Android rendered as boxes
Code sample
Code sample
Added reproducible sample code: https://github.com/amsiddh/char_tabulation

Screenshots or Video

Screenshots / Video demonstration
[[Upload media here]]
Android

iOS

Flutter Doctor output
Doctor output
[✓] Flutter (Channel stable, 3.0.2, on macOS 11.6.5 20G527 darwin-x64, locale en-IN)
    • Flutter version 3.0.2 at /Users/sidbingi/Documents/DevTools/Flutter/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision cd41fdd495 (11 months ago), 2022-06-08 09:52:13 -0700
    • Engine revision f15f824b57
    • Dart version 2.17.3
    • DevTools version 2.12.2

[✓] Android toolchain - develop for Android devices (Android SDK version 32.1.0-rc1)
    • Android SDK at /Users/sidbingi/Library/Android/sdk
    • Platform android-33, build-tools 32.1.0-rc1
    • Java binary at: /Users/sidbingi/Library/Java/JavaVirtualMachines/corretto-1.8.0_352/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment Corretto-8.352.08.1 (build 1.8.0_352-b08)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[!] Android Studio (version 2022.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    ✗ Unable to find bundled Java version.
    • Try updating or re-installing Android Studio.

[✓] VS Code (version 1.76.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.62.0

[✓] Connected device (4 available)
    • sdk gphone64 x86 64 (mobile) • emulator-5554                        • android-x64    • Android 13 (API 33) (emulator)
    • iPhone 13 (mobile)           • CE7F4CA9-CE7C-46EE-83C0-D8DD54C7F53C • ios            •
      com.apple.CoreSimulator.SimRuntime.iOS-15-2 (simulator)
    • macOS (desktop)              • macos                                • darwin-x64     • macOS 11.6.5 20G527 darwin-x64
    • Chrome (web)                 • chrome                               • web-javascript • Google Chrome 112.0.5615.137

[✓] HTTP Host Availability
    • All required HTTP hosts are available

If fontFamily: 'Kanit' removed from app ThemeData then it works, but that font is required for our project
In our project for Flutter v2.5.2 it working properly for both platforms, but for Flutter v3.0.2 only working in iOS. In given sample app tried for both mentioned version, not working in Android for both versions.
As per this issue thread #111377 tried in v3.3.2 but still showing boxes in Android.

@amsiddh
I think the problem is that your flutter version is ancient!
Doing the following

create a new app from the latest stable
copy the asset folder  from your project and add it to pubspec.yaml

pubspec.yaml
name: issue_125300
description: A new Flutter project.
publish_to: 'none'

version: 1.0.0+1

environment:
  sdk: '>=2.19.6 <3.0.0'
dependencies:
  flutter:
    sdk: flutter

dev_dependencies:
  flutter_test:
    sdk: flutter
  flutter_lints: ^2.0.1

flutter:
  uses-material-design: true
  fonts:
    - family: Kanit
      fonts:
        - asset: assets/fonts/Kanit-Regular.ttf
        - asset: assets/fonts/Kanit-Medium.ttf

copy your main.dart (cleaning a bit the legacy part)

main.dart
import 'package:flutter/material.dart';

void main() => runApp(const MyApp());

class MyApp extends StatelessWidget {
  const MyApp({super.key});

  @override
  Widget build(context) => MaterialApp(
        title: 'Flutter Demo',
        theme: ThemeData(
          primarySwatch: Colors.blue,
          fontFamily: 'Kanit',
        ),
        home: const MyHomePage(title: 'Flutter Demo Home Page'),
      );
}

class MyHomePage extends StatelessWidget {
  const MyHomePage({required this.title, super.key});
  final String title;
  @override
  Widget build(context) => Scaffold(
        appBar: AppBar(title: Text(title)),
        body: Center(
          child: Text(
            '	1.		คำนิยาม',
            style: Theme.of(context).textTheme.headlineMedium,
          ),
        ),
      );
}

running the app as flutter run --release outputs the result below

screenshot

doctor
[!] Flutter (Channel stable, 3.7.12, on macOS 13.3.1 22E261 darwin-arm64, locale en-EE)
    • Flutter version 3.7.12 on channel stable at /Users/yakforward/fvm/versions/stable
    ! Warning: `dart` on your path resolves to /opt/homebrew/Cellar/dart/2.19.6/libexec/bin/dart, which is not inside your current Flutter SDK checkout at /Users/yakforward/fvm/versions/stable. Consider adding
      /Users/yakforward/fvm/versions/stable/bin to the front of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 4d9e56e694 (5 days ago), 2023-04-17 21:47:46 -0400
    • Engine revision 1a65d409c7
    • Dart version 2.19.6
    • DevTools version 2.20.1
    • If those were intentional, you can disregard the above warnings; however it is recommended to use "git" directly to perform update checks and upgrades.

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.2)
    • Android SDK at /Users/yakforward/Library/Android/sdk
    • Platform android-33, build-tools 33.0.2
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.2)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14C18
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2022.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)

[✓] VS Code (version 1.77.3)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.62.0

[✓] Connected device (3 available)
    • sdk gphone64 arm64 (mobile) • emulator-5554 • android-arm64  • Android 13 (API 33) (emulator)
    • macOS (desktop)             • macos         • darwin-arm64   • macOS 13.3.1 22E261 darwin-arm64
    • Chrome (web)                • chrome        • web-javascript • Google Chrome 112.0.5615.137

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.

Sorry for replying late, as I was on vacation.
@iapicca you were right. I followed your steps and it worked properly.
Thanks for the support @iapicca @darshankawar

Closing as resolved based on above comment.


Feature/deploy release specification for scratch org creation

One test is failing which is not related to any of our file changes.Can we discuss ? @davidmreed

Hi @davidmreed Please review our PR and suggest any changes if required.


[ui] Support GKE regional or zonal clusters

What kind of request is this (question/bug/enhancement/feature request):
Rancher should be able to create zonal and regional GKE clusters.
Steps to reproduce (least amount of steps as possible):

Create a GKE cluster
Choose region or zone (exclusive or)

Result:

Other details that may be helpful:
From GKE:
A regional cluster provides a single static endpoint for the entire cluster, enabling you to access the cluster's control plane regardless of any outage or downtime in an individual zone. https://cloud.google.com/kubernetes-engine/docs/concepts/regional-clusters?hl=en_US
Location type is permanent
GKE regions: https://cloud.google.com/compute/docs/regions-zones/
Address issue: https://github.com/rancher/rancher/issues/17099

This may require another authed handler to GET gke regions https://github.com/rancher/rancher/blob/4f3943c624f01b96106095c8a19d8ebcc1bb771a/server/server.go#L118

Version: master-head (v2.3) (8/16/19)
Summary:
This change adds a new option when launching a GKE cluster in the Rancher UI.
It allows us to pick if we want to configure our GKE cluster in a zonal manner or regional manner.
Previously, we could only configure via zones. If we wanted to cover a whole region, you would basically pick your zone, and ALL additional zones.
NOTE: for all my tests I only chose 1 small worker node. This is because the more zones you pick the more worker nodes are created and makes it easier to count. That is to say, your specification of the count of worker nodes is per zone. When you pick a region you get your one worker in each zone for the region.
The change adds a "Location Type" option where the user can choose either Zonal or Regional. The dropdown (and additional zones when using zonal type) were moved to the right in the form.

The screenshot above shows the default values.
Test zones scenario:
I ensured that when I chose the default zonal type, and switch zones in the zone dropdown, the Additional Zones section still behaves as expected (it only shows the remaining zones for the region). This is still working correctly just as it was before the change. I tested with various zones, and added additional zones to ensure this works correctly both in the rancher HTTP/API and in the Google Platform web console. I spot checked various zones and additional zones. I ensured that I had my one worker in each zone chosen.
Test regions scenario:
Last, I tried the new option to pick regions ("Regional"). When you pick this the form changes to just show a dropdown now of the various regions.

The screenshot above shows the default value when switching to "Regional"
I again ensured that the rancher HTTP/API appear correct as well as the Google Platform web console. All the various options worked as expected. I spot checked various regions. I ensured that for each region I get the expected count of workers (1 worker per zone in my case). So for example, when I pick the us-west-2 region, I get 3 worker nodes (since I chose 1 worker node, you get 1 per zone).
API/HTTP:
When you send a request to use a region, "region" is sent in the googleKubernetesEngineConfig.
region: "us-west2" (default when chose Regional in UI)
location: is not sent at all (this is expected).
zone: is null (this is expected)
When you send a request to use a zone or zones (pick additional zones in the UI), "zone" is sent as well as "locations" when additional zones are chosen. (This is expected).
region: is null (this is expected)
(here is an example where I picked us-east-4-a and in the UI chose us-east-4-c as my additional zone)
locations: ["us-east4-c", "us-east4-a"]
  0: "us-east4-c"
  1: "us-east4-a"

zone: "us-east4-a"

Conclusion:
During my various spot checks with different regions and zones, I wasn't able to identify any issues. HTTP/API/UI all look good, and my GKE cluster config was confirmed by checking the Google Cloud Platform web UI (i.e. ensure in proper zone(s)/region). In each scenario I got the expected count of worker nodes per my spec for zone(s)/region.


Selecting account for orders_v2

I was able to specify the account in orders_v2() by adding:
self.headers["schwab-client-account"] = account_id
The bigger issue I'm having is that the response doesn't seem to populate the RemainingQty (it's always zero in my attempts), so I had to figure out a much more convoluted and probably unreliable way to figure out how my orders executed.

I ran into a similar issue where the account_info API only returns the first account's information. Not sure if these originate from the same root cause, though.
In my case, it turns out that the new login code that logs in directly to the trade ticket was the root cause. For some reason, the Schwab homepage returns only the selected account's info for "https://client.schwab.com/api/PositionV2/PositionsDataV2" requests if you log in directly to that page. The weird thing is that it works fine if you go to the summary page and then come back to the trade ticket page. I'm trying to circumvent it by taking advantage of the behavior, but wonder if there is a better solution?
I'll file a separate issue if you think this is not the same issue.

I think that's a separate issue since the current implementation of get_account_info() uses the "older" api at client.schwab.com, while orders_v2 uses the "new" api at ausgateway.schwab.com.
FWIW, I wrote my own method to retrieve account info from a "v2" api endpoint (ausgateway.schwab.com). It seemed to require some extra handling of the bearer token, and I don't have time to figure out child options with it. For anyone interested in carrying that forward, here's what I do:
    def update_token(self):
        r = self.session.get("https://client.schwab.com/api/auth/authorize/scope/api")
        token = json.loads(r.text)['token']
        self.headers['authorization'] = f"Bearer {token}"

    def get_account_info(self):
        account_info = dict()
        self.update_token()
        url = (
            "https://ausgateway.schwab.com/api/is.Holdings/V1/Holdings/Holdings?="
            "&includeCostBasis=true&includeRatings=true&includeUnderlyingOption=true")
        r = self.session.get(url, headers=self.headers)
        response = json.loads(r.text)
        for account in response['accounts']:
            positions = list()
            for security_group in account["groupedPositions"]:
                if security_group["groupName"] == "Cash":
                    continue
                for position in security_group["positions"]:
                    positions.append(
                        Position(
                            position["symbolDetail"]["symbol"],
                            position["symbolDetail"]["description"],
                            int(position["quantity"]),
                            float(position["costDetail"]["costBasisDetail"]["costBasis"]),
                            float(position["priceDetail"]["marketValue"])
                        )._as_dict()
                    )
            account_info[int(account["accountId"])] = Account(
                account["accountId"],
                positions,
                account["totals"]["marketValue"],
                account["totals"]["cashInvestments"],
                account["totals"]["accountValue"],
                account["totals"].get("costBasis", 0),
            )._as_dict()

        return account_info

Hi @WinesProof, I've adopted the code above as a function in v0.3.3 and added an optional account_id parameter to orders_v2().
Strongly appreciate you paving the way here!

I would also like to add that I am also seeing only 1 account being returned with the get_account_info() function. When running on 0.2.3 it returns all of my accounts, but when I run it on 0.3.0 and up it only returns 1 account.

I would also like to add that I am also seeing only 1 account being returned with the get_account_info() function. When running on 0.2.3 it returns all of my accounts, but when I run it on 0.3.0 and up it only returns 1 account.

Upgrading to the latest version and then switching to get_account_info_v2 worked for me.

Perfect, that worked for me!

I see the same issue at 0.3.5 (from 0.2.3) of coming back only 1 account using get_account_info() but must use get_account_info_v2() to see all my accounts. @itsjafer I was wondering does that make sense to copy the current login() to login_v2() for all the v2 type login and then copy the legacy login() from 0.2.3 to latest in the authentication.py for best backward compatibility?
I have tested it on my side and can do the add but it will create extra work for who has modified based this thread or using the latest library. So would like to know people opinions.

Indeed, in hindsight, I should've created an entirely new login function to avoid backwards compatibility issues. I'm considering a legacy parameter taken by login that avoids the navigation for now, so that those using the v2 functions aren't inconvenienced since that seems to be the path forward with future functionality

I ran into a similar issue where the account_info API only returns the first account's information. Not sure if these originate from the same root cause, though.
In my case, it turns out that the new login code that logs in directly to the trade ticket was the root cause. For some reason, the Schwab homepage returns only the selected account's info for "https://client.schwab.com/api/PositionV2/PositionsDataV2" requests if you log in directly to that page. The weird thing is that it works fine if you go to the summary page and then come back to the trade ticket page. I'm trying to circumvent it by taking advantage of the behavior, but wonder if there is a better solution?
I'll file a separate issue if you think this is not the same issue.

Hi,
I also get info for the selected account, not all accounts.
What do you do to go to the summary page and then come back to the trade ticket page?
I tried add self.session.get(urls.account_summary()) before the position request. But still returns one account info.

I ran into a similar issue where the account_info API only returns the first account's information. Not sure if these originate from the same root cause, though.
In my case, it turns out that the new login code that logs in directly to the trade ticket was the root cause. For some reason, the Schwab homepage returns only the selected account's info for "https://client.schwab.com/api/PositionV2/PositionsDataV2" requests if you log in directly to that page. The weird thing is that it works fine if you go to the summary page and then come back to the trade ticket page. I'm trying to circumvent it by taking advantage of the behavior, but wonder if there is a better solution?
I'll file a separate issue if you think this is not the same issue.

Hi, I also get info for the selected account, not all accounts.
What do you do to go to the summary page and then come back to the trade ticket page? I tried add self.session.get(urls.account_summary()) before the position request. But still returns one account info.

Hi @QingqiLei could you try v0.3.6? I've merged in https://github.com/itsjafer/schwab-api/pull/32 which might solve the problem for v1 methods

I ran into a similar issue where the account_info API only returns the first account's information. Not sure if these originate from the same root cause, though.
In my case, it turns out that the new login code that logs in directly to the trade ticket was the root cause. For some reason, the Schwab homepage returns only the selected account's info for "https://client.schwab.com/api/PositionV2/PositionsDataV2" requests if you log in directly to that page. The weird thing is that it works fine if you go to the summary page and then come back to the trade ticket page. I'm trying to circumvent it by taking advantage of the behavior, but wonder if there is a better solution?
I'll file a separate issue if you think this is not the same issue.

Hi, I also get info for the selected account, not all accounts.
What do you do to go to the summary page and then come back to the trade ticket page? I tried add self.session.get(urls.account_summary()) before the position request. But still returns one account info.

Hi @QingqiLei could you try v0.3.6? I've merged in https://github.com/itsjafer/schwab-api/pull/32 which might solve the problem for v1 methods

Closing this, because choosing an account for orders_v2 appears to be working now.


Daprd Env Variables from Secrets

In what area(s)?

/area runtime

/area test-and-release

Describe the proposal

Issue:
Today the only Env variables that can be injected into the sidecar pod, are the plain key value (see here).
Some applications or libraries require the environment variables to include secret values, such as Auth tokens, headers, etc.
This is a suggestion to solve this issue and discuss potential implementation and possible problems with it.
Solution
Similar to the dapr.io/env annotation, add dapr.io/env-from-secret annotation.
The values of this annotation are formatted like so:
Single key value secret: <ENV_VAR_NAME>=<SECRET_NAME>
Multi key value secret: <ENV_VAR_NAME>=<SECRET_NAME>:<SECRET_KEY>
Since the k8s annotations are limited in size, supporting multi key-value secrets might be considered as optional.
This annotation then will be injected via dapr-injector into the daprd container manifest like so.
...
spec:
  containers:
  - name: daprd
    image: docker.io/daprio/daprd:1.13.5
    env:
    - name: <ENV_VAR_NAME>
      valueFrom:
        secretKeyRef:
          name: <SECRET_NAME>
          key: <<SECRET_KEY>

Then Env variable can be consumed by Dapr sidecar, component or configuration.

Due to the restriction of the secretKeyRef, name field is required (read more here, in section "env.valueFrom.secretKeyRef.name"), this makes the single key value secret requires us to have both name and key.
Example:
Annotation: dapr.io/env-from-secret: "HEADERS=auth-headers-secret"
Will be injected as:
    env:
    - name: HEADERS
      valueFrom:
        secretKeyRef:
          name: auth-headers-secret
          key: auth-headers-secret

This will require the Secret to have both a name and a key as "auth-headers-secret"
Example secret (for demo purposes only, don't leave secrets in plain text)
apiVersion: v1
kind: Secret
metadata:
  name: auth-headers-secret
type: Opaque
stringData:
  auth-headers-secret: "AUTH=mykey"

Single key value secret: <ENV_VAR_NAME>=<SECRET_NAME>
Is it possible to have single key secrets that don't require you to specify a key name, even though it's the only name in the secret?


Adding the lv_chart_example4 from lvgl examples to factory demo tile doesn't scroll properly.

The basic idea is to add the examples and demos into the build and call the 4th chart example on a new fourth tile, but the problem also happens if you just add the chart to the first tile (tv1 object). By "add the chart", I mean I'm duplicating chart4 from the examples but modifying the creation to be on one of the tiles, although the problem exists with just the unmodified lv_example_chart_4() call.
The symptom is the chart scrolls to the right after the user releases the touch. You cannot scroll the chart left and release because the chart scrolls right again after release.

I have tried adding the chart to a tile in the lv_demos.ino file in this repo, but the bug doesn't show and the chart can be scrolled successfully.

 It's disabled in lv_demos but not in factory.
https://github.com/Xinyuan-LilyGO/T-Display-S3/blob/main/example/factory/factory.ino#L3
Disabling it stopped the bug, but now I want to know why, as there is only one conditional bit of code doing touch.read() so I don't get how it could be double firing or firing a second time late.

Although the graph does reliably scroll now, there have been side effects. Disabling the interrupt has caused other problems like the tooltips for chart points not showing, and the Touch Test label on tile3 doesn't reset back to Zero fingers.

I guess there is a problem with the interrupt, that is, every time the last interrupt comes, I will get the coordinates of (0, 0) to read. Causes the icon to always move upwards.

If the user experience is still not good, then the best function of interrupt is the function of waking up during sleep

The effect of not using interrupts is no problem in my test. Regarding the battery test, I feel that the overall voltage is too high. It is possible that the current sampled by GPIO is too small to cause inaccuracy.

A similar issue on lvgl forum, points out the clean solution of not registering as pressed if number of fingers is zero.
https://forum.lvgl.io/t/slider-arc-roller-values-changes-to-zero-or-wrong-values-after-release/10587/2
I've patched mine with a renamed variable: https://github.com/Xinyuan-LilyGO/T-Display-S3/commit/2e5eb82e35853b495cbc55df37345e202ed5d1f8#diff-6eb686fe7cc6799a846fc7bcc08bf4d4566a82d71f003de9858308622b75d259L65-R84

If you still have problems, please restart


feat: add method getNewLogger to public API

This method will return a new logger configured with a provided name. This can be used by generated service classes to create a new logger specific to its module context.
Signed-off-by: Dustin Popp<EMAIL_ADDRESS>
Checklist

[x] npm test passes (tip: npm run lint-fix can correct most style issues)
[x] tests are included
[x] documentation is changed or added

:tada: This PR is included in version 3.2.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Added flattening examples

This is for adding flattening examples for Either, Pair for #203

Coverage remained the same at 100.0% when pulling 67d0459f258f0a9fe5d1d71f0e270101a52b8340 on docs/flattening into caf04ed62328efc68f9b8880064acc169cad7e7e on master.


Error while building js-base64

Uncaught SyntaxError: Invalid regular expression: /[Ã€-ÃŸ][Â€-Â¿]|[Ã -Ã¯][Â€-Â¿]{2}|[Ã°-Ã·][Â€-Â¿]{3}/: Range out of order in character class
Chrome : 74.0.3729
"js-base64": "^2.5.1",
"@vue/cli-plugin-babel": "^3.5.5",
"@vue/cli-plugin-eslint": "^3.4.0",
"@vue/cli-plugin-unit-jest": "^3.8.0",
"@vue/cli-service": "^3.6.0",
"@vue/eslint-config-airbnb": "^4.0.0",
"@vue/test-utils": "1.0.0-beta.29",
"babel-core": "7.0.0-bridge.0",
"babel-eslint": "^10.0.1",
"babel-jest": "^23.6.0",
"eslint": "^5.16.0",
"eslint-loader": "^2.1.2",
"eslint-plugin-vue": "^5.2.2",
"moment-locales-webpack-plugin": "^1.0.7",
"node-sass": "^4.12.0",
"prettier": "^1.17.0",
"sass": "^1.18.0",
"sass-loader": "^7.1.0",
"vue-template-compiler": "^2.6.10",
"webpack": "^4.32.0"
Build code :     "staging": "vue-cli-service build --mode staging --dest dist --target app",
It looks like during minification process the regex gets weird encoding ... so that the browser won't understand it ...
[Ã€-ÃŸ][Â€-Â¿]|[Ã -Ã¯][Â€-Â¿]{2}|[Ã°-Ã·][Â€-Â¿]{3}
instead of  :
var re_btou = new RegExp([
'[\xC0-\xDF][\x80-\xBF]',
'[\xE0-\xEF][\x80-\xBF]{2}',
'[\xF0-\xF7][\x80-\xBF]{3}'
].join('|'), 'g');

It works when I build on Windows
but when I build on centos .. it won't

Fixed by forcing the file encoding on nginx to utf-8

I ought to have closed this.  I am doing so now…


Migrate from bing maps web services

#sign-off

@stevemunk - Could you resolve the issues I've noted in the diff view?
Because you're making extensive substantive changes in your PRs, it would be a good idea to stage all of your PRs in the private repo to take advantage of automated checks rather than relying solely on human review.
#hold-off


Chrome/96.0.4664.45 on Android 11 (Pixel 4a)

I was getting weird behaviour:

canvas too wide for the screen
strokes appearing 1 or 2 cm to the right of where i drew

until I removed the demo resizeCanvas() code.  Then Android seemed to work fine.
If you have a question, please ask it on StackOverflow and use signature-pad tag instead of creating an issue.
Do you want to request a feature or report a bug?
What is the current behavior?
If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem via https://jsfiddle.net or similar (template: https://jsfiddle.net/szimek/h80phxyr/).
What is the expected behavior?
Which versions of SignaturePad, and which browser / device are affected by this issue? Did this work in previous versions of SignaturePad?
Signature Pad v3.0.0-beta.4
Mozilla/5.0 (Linux; Android 11; Pixel 4a) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/96.0.4664.45 Mobile Safari/537.36

It sounds like you were able to fix the issue by removing the resizeCanvas() code. If you have any other issues make sure you fill out the template completely so we can more easily help you.


shifted submenu

Hi,
how can I solve this, for information, little11 is the only tweak I've installed.
PS:

in case it is not understandable, I can't answer with a Yes or No because the menu is out of place.

in the general settings the only item that is enabled is iPhone X Status Bar

sorry duplicate #23


DateTime.ToString doesn't conform to Custom Date and Time Format Strings

DateTime.ToString(format, provider) does not conform to Custom Date and Time Format Strings.
Steps To Reproduce
http://deck.net/e7851899331080a5240e80f698554ecb/top
public class Program
{
    public static void Main()
    {
        var date = new DateTime(2017, 2, 6, 10, 42, 52, 1);

        Console.WriteLine(date.ToString("yyyy-MM-dd HH:mm:ss.FFFFFFF", CultureInfo.InvariantCulture));
    }
}

Expected Result
2017-02-06 10:42:52.001
See https://dotnetfiddle.net/JmJhwW
Actual Result
2017-02-06 10:42:52.FFFFFFF
I know that JS does not hold such precision but replacing with ms should be sufficient I think.

Date object doesn't manage accuracy beyond milliseconds
So, Bridge can support f{1,3}|F{1,3} modifiers only (f ff fff F FF FFF)
We will include support of such modifiers

.Net truncates the result if the precision cannot be met, it would be great if Bridge supported FFFFFFF but always returned truncated result (so current code doesn't generate completely invalid output).


Add persistant configuration + Themes

This is how the default configuration file looks now
discovery_addr = "<IP_ADDRESS>:5877"
tcp_server_port = 0
user_name = "mrcool"
terminal_bell = true

[theme]
message_colors = ["Blue", "Yellow", "Cyan", "Magenta"]
my_user_color = "Green"
date_color = "DarkGray"
system_info_color = ["Cyan", "LightCyan"]
system_warning_color = ["Yellow", "LightYellow"]
system_error_color = ["Red", "LightRed"]
chat_panel_color = "White"
progress_bar_color = "LightGreen"
command_color = "LightYellow"
input_panel_color = "White"

The next thing should be making termchat accept predefined themes, instead of the need of changing the code, or changing each color in the config file.
$Config/termchat would look like
               |
               |____ config
               |_____themes
                          |________default
                          |________light
                          |________other_custom_themes ..

with config having a theme entry like theme = dark/light/custom_name which will make theme search for it it in the aforementioned path.
I prefer to leave that to another pr.

Don't worry about the delay, I see you're busy with message-io hopefully that works out well!

Hi @sigmaSd, I saw that the send_file integration test, is broken after these changes, could you fix it whenever you can? Thanks!


Allow https

I think it would be a better practice to send requests over https by default or at least make it configurable.
To me it seems like you can simply change "http" to "https" in AmazonSign::Sign(IDictionary<string, string> request).

done 7a5100769836b62da0783df69c7ab6b8a3fe6321 change all requests over https


Option to link TK and TF workspace names

Feature description 💡
Currently there is no link between TerraKube workspace name and the Terraform workspace name, i.e. all TF workspace names are the standard default in each TK workspace unless explicitly set with the TF_WORKSPACE variable.
Can there be an option to link them? Or perhaps a global variable which could be set at the organization level.

Can you explain a little bit more maybe with some example/screenshot because I am not sure if I understood correctly

It's about the Terraform workspace concept. All TF projects have a default workspace that everything TF builds goes into. The following command creates the workspace dev and switches to it:  terraform workspace new dev. You could then terraform apply again and build the same infrastructure.
With Terrakube the only way to change a TF workspace is using the TF_WORKSPACE environment variable, like the screenshot below:

Otherwise, the workspace is default
What I would like is something I could set at the Global Variable level, like a special environment variable that would return whatever the TerraKube workspace name is. Something like this:

Do you know if using TF_WORKSPACE environment variable is supported in TFC?
Because workspace is a different concept there as explain here

I don't see too much of a difference I believe with TFC HashiCorp hard coded workspaces the way I describe above.
The workspace name in TFC is automatically the TF_WORKSPACE value, you can't change it.
With Terraform CLI you have more options; each directory can have multiple workspaces.

I believe Terrakube should execute this command before each job, to set the workspace name:
terraform workspace new 'Terrakube_Workspace_name'

@SolomonHD temporarily you can try adding it to your plan and apply template:
Plan example:
flow:
  - type: "terraformPlan"
    name: "Plan"
    step: 100
    commands:
      - runtime: "BASH"
        priority: 200
        before: true
        script: |
               export RESPONSE=$(curl --silent --show-error -X GET "${terrakubeApi}//api/v1/organization/${organizationId}/workspace/${workspaceId}" \
                 -H "Authorization: Bearer ${terrakubeToken}" \
                 -H "Content-Type: application/vnd.api+json")
              export WORKSPACE_NAME=$(echo $RESPONSE | jq -r '.data.attributes.name')
               terraform workspace new $WORKSPACE_NAME

Looks like this option doesn't works as based on #1008 the environment variables aren't shared between executions

Interesting...so you'd have to setup custom Terraform apply and delete flows as well?
Each one querying for the workspace name before running the terraform command?

A good reason to implement this is to be able to use the workspace name within the code by using terraform.workspace. While in TFC we were able to use that as a value, in Terrakube we can't use it because the value is default instead of the actual workspace name.
Example
For the workspace called org-terrakube-staging-euw1, we were able to extract/infer some valuable info from there, which now it's impossible out of the box
locals {
  # Infer vars from workspace name
  workspace_name_parts = flatten(regexall("([^-]+)-([^-]+)-([^-]+)-([^-]+)", terraform.workspace))
  inferred_org         = local.workspace_name_parts[0]
  inferred_project     = local.workspace_name_parts[1]
  inferred_environment = local.workspace_name_parts[2]
  inferred_region_code = local.workspace_name_parts[3]
}

A good reason to implement this is to be able to use the workspace name within the code by using terraform.workspace. While in TFC we were able to use that as a value, in Terrakube we can't use it because the value is default instead of the actual workspace name.
Example
For the workspace called org-terrakube-staging-euw1, we were able to extract/infer some valuable info from there, which now it's impossible out of the box
locals {
  # Infer vars from workspace name
  workspace_name_parts = flatten(regexall("([^-]+)-([^-]+)-([^-]+)-([^-]+)", terraform.workspace))
  inferred_org         = local.workspace_name_parts[0]
  inferred_project     = local.workspace_name_parts[1]
  inferred_environment = local.workspace_name_parts[2]
  inferred_region_code = local.workspace_name_parts[3]
}

Hello @juan-vg  when you setup a CLI driven workflow in TFC and you want to the workspace new command you will get the following error, not sure how you are using that in TFC maybe you can add more context
λ terraform workspace new develop
workspaces not supported

Hello @juan-vg when you setup a CLI driven workflow in TFC and you want to the workspace new command you will get the following error, not sure how you are using that in TFC maybe you can add more context
λ terraform workspace new develop
workspaces not supported

@alfespa17 We create the workspaces directly in TFC via TF (we have our own workspace/project/... bootstraper from GitHub to TFC, so our TFC org is driven by GitOps).
BTW, I've tried the TF_WORKSPACE environment variable, and it works as a charm. Now terraform.workspace has the proper value.


Convert to UWP Class Library?

Not sure how much we're really benefiting from being a Windows Runtime Component. Plus it means we can't use Task as a return type in our signatures which makes some of the Providers weird to use with having to wrap returns in the AsyncInfo.Run pattern.
It'd be worth thinking about making this jump in the future and cleaning this up a bit?

This would also let us use abstract base classes in the library for the CssStyles...

Being a pure UWP class library means that monaco-editor-uwp won't be able to use in UWP which is written in C++/WinRT.
My opinion is that monaco type bindings can be convert to a class library, and still keep the control itself a Windows Runtime Component.

@hez2010 are you using it from C++ currently? I'm not sure what the audience for that usage is.
Also, even if it's a class library you can include it from another C# component utilized by C++ project, so there's a workaround to access it still, though a bit more indirectly.  There's just a lot of restrictions which come with being a pure Windows Runtime Component.

I'm not sure what will happen exactly if we convert it to a UWP class library, but seems that it worth a try.

Just noticed this remark: https://docs.microsoft.com/en-us/uwp/api/windows.ui.xaml.controls.webview.addweballowedobject#remarks

The object passed into AddWebAllowedObject must be imported from a Windows Runtime component that is separate from the app assembly.

Not sure if this will be the case still for WebView2?

🦙 Cross-ref https://github.com/microsoft/microsoft-ui-xaml/issues/2571


Fix constness when taking an array of string pointers

i.e. change char *const * to const char *const *
The problem was apparent from incorrect constness in the bindgen generated bindings: https://gist.github.com/teohhanhui/647227ea483dabdcd8317c6ea228665e
e.g. *const *mut c_char instead of the expected *const *const c_char, which matches https://github.com/containers/libkrun/blob/5e376689d58c1c56a22d3935722e66fe09d28685/src/libkrun/src/lib.rs

clippy failures not related...

Thanks for the PR! Could you please rebase on top of #182 once it gets merged?


Simple question

Hello)
I have a problem.
I need a request that goes to sub.example.com:8080 to hit localhost:3000.
How should i config client for that?
Seems like ClientConfig.LocalAddr:":3000" does not help.
It does work locally. but i try to run it on appengine and always get no client session established error.
Thanks a lot)

Hi @yarikbratashchuk
We haven't tested tunnel on appengine. In some cases appengine requires some additional work/configuration, you might be hitting that edge.

Hi @cihangir
Thanks for the answer.


Fix generic api messages

This is a quick fix to correct the generic api messages.
No need to do the full pr if you want to just add it in yourself.

Closing in favour of #422


Blank terminal on Mac

VSCode Version: 1.44
OS Version: Mojave 10.14.5

Steps to Reproduce:
Not sure how to reproduce, was working on a rails project and the terminal started typing behind the console and then I tried clearing it and the terminal just came up as a blank slate and no matter if I restarted VScode, opened a new file it would still be blank. I tried looking up solutions but have not found any that works for macs.

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

Please try enabling trace logging to give us some insight into what's happening when the terminal launches:
https://github.com/microsoft/vscode/wiki/Terminal-Issues#enabling-trace-logging


Add gist ID from command line to force sync on VSC install

I'm creating bash scripts to allow me setup dev configuration by running few commands.
Currently I'm installing VSC from brew, running
code --install-extension Shan.code-settings-sync
And then I want to pass GIST id and force sync on VSC start. However looks like it's not possible now, as I could understood (https://github.com/shanalikhan/code-settings-sync/issues/465#issuecomment-349903749). Guys, maybe you have any ideas how I could implement it?

Duplicate of #403

If you want to perform upload / download on code start
Checkout autoUpload and autoDownload configurations. Download can be performed on startup. Upload can be performed on settings changed.

@shanalikhan what do yo mean by autoUpload ? there are no comments related such sections on issue that you've provided ¯_(ツ)_/¯

Check the features on Readme.
https://github.com/shanalikhan/code-settings-sync#features
If you want to do manually see the Global Settings

I'm making a Linux docker file with online version of VSCode called code-server. If you change code-server in script below to code, it will work for normal VSCode too.
RUN code-server --install-extension Shan.code-settings-sync

RUN echo "{}" >> /root/.local/share/code-server/User/settings.json

RUN tmp=$(mktemp) && \
  jq -M '. +{ \
  "sync.syncExtensions": true,\
  "sync.autoDownload": true,\
  "sync.gist":"a9cdcbed2cf202f440a8a5f0626a38ac",\
  "telemetry.enableTelemetry": false }' /root/.local/share/code-server/User/settings.json > "$tmp" && mv "$tmp" /root/.local/share/code-server/User/settings.json

RUN echo "{}" >> /root/.local/share/code-server/User/syncLocalSettings.json

RUN tmp=$(mktemp) && \
  jq -M '. +{ "downloadPublicGist": true }' /root/.local/share/code-server/User/syncLocalSettings.json > "$tmp" && mv "$tmp" /root/.local/share/code-server/User/syncLocalSettings.json

Basically creating VSCode settings.json and adding gist ID to it using jq and then creating syncLocalSettings.json config with "downloadPublicGist": true.
When VSCode (code-server) is opened first time, settings download gets successfully started.
If you don't want auto-download to start, change "sync.autoDownload": true to false
Now I only need to find a way to initialize VSCode first run bootstrap through command line /bash


#288 Fixed ScrollToTop Btn

Related Issue #288

Fixed the positioning of the ScrollToTop icon.
Closes: #288
Description of Changes

Checklist:

[x] My code adheres to the established style guidelines of this project.
[x] I have conducted a self-review of my code.
[x] I have included comments in areas that may be difficult to understand.
[x] I have made corresponding updates to the project documentation.
[x] My changes have not introduced any new warnings.

Screenshots

Original
Updated

Please provide any necessary screenshots to illustrate the changes made.

Resolve Conflicts, tag on discord with pr link.

Hey @AKD-01 I have resolved the conflicts in the PR


Create falco-0-29-0.md

Add falco-0-29-0.md Chinese document, easy for readers to read.

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind user-interface

/kind content

/kind translation

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area blog

/area documentation

/area videos

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: gogolovefish
Once this PR has been reviewed and has the lgtm label, please assign leogr for approval by writing /assign @leogr in a comment. For more information see:The Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

e129675 Create falco-0-29-0.md

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Hey @gogolovefish, thank you for your submission.
Could I ask you to sign the commit too according to the contributing guide?

/kind content
/area blog

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

e129675 Create falco-0-29-0.md
f204058 Update falco-0-29-0.md

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

e129675 Create falco-0-29-0.md
f204058 Update falco-0-29-0.md

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

e129675 Create falco-0-29-0.md
f204058 Update falco-0-29-0.md

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@vjjmiras @mstemm hello ,i have modify the error in falco-0-29-0.md Chinese document ,please _1 ,thanks


input.input_update has trouble with single options

BUG Description
When only a single option is passed to input.input_update on change, the user is unable to select it. See GIF below.

Expand GIF

The issue is also present in the input.input_update demo, where the first new item on the list cannot be selected. One has to first select another option and then switch back to the first option. See GIF below.

Expand GIF

Code for the first example:
type2amount = {
        "accident or illness": ["One single accident or illness"), ("Two separate accidents or illnesses"],
        "military or civil service": ["one single period (option can only be changed for accident or illness)")],
        "pregnancy": ["one single period (option can only be changed for accident or illness)")],
    }

    incapacity_types = list(type2amount.keys())

    case = input.input_group("Your Case", [
        input.radio("Which type of incapacity would you like to evaluate?"),
            options=incapacity_types,
            value=incapacity_types[0],
            name="incapacity_type",
            inline=True,
            onchange=lambda c: input.input_update("period_amount", options=type2amount[c])),
        input.select("During how many periods were you incapacitated?"),
            options=type2amount[incapacity_types[0]],
            name="period_amount",
            inline=True),
    ])

Environment Information

OS and Version:  Windows 11
Browser and Version:  Firefox  100.0, also tested on Brave 11.38.111
Python Version: 3.10.4
PyWebIO Version: 1.6.0

It's a bug, I will fix it on this weekend.
Here is a workaround before it's fixed, also set the value in input_update():
country2city = {
    'China': ['Beijing', 'Shanghai', 'Hong Kong'],
    'USA': ['New York'],
}
countries = list(country2city.keys())
location = input_group("Select a location", [
    select('Country', options=countries, name='country',
           onchange=lambda c: input_update('city', options=country2city[c], value=country2city[c][0])),
    select('City', options=country2city[countries[0]], name='city'),
])
put_text(location)

The bug is fixed in new release v1.6.1

The bug is fixed in new release v1.6.1

Wow, that was quick - thanks!


[Tracker] Base Config Compatibility

(copied from roadmap.md)
Ensure that the base configuration format is viable for various platforms
Systems:

Solaris
Windows
Linux

We'll need some folks to contribute the technical requirements for these platforms, as Linux is what is currently getting our attention.

Currently comparing the Spec structs from:

https://github.com/docker/containerd/blob/master/vendor/src/github.com/opencontainers/specs/specs-go/config.go
https://github.com/docker/docker/blob/master/libcontainerd/windowsoci/oci_windows.go

Have been in-touch with Jesse from Oracle and expecting more info soon.

Looks like this needs to be addressed before 1.0, I'm adding it to 1.0 milestone.

@RobDolinMS as you're heading this effort up, can you make a call on whether or not the final round of cleanup in PR #673 is enough to close the issue out? If not, please let me know what else I can do to wrap this up.

Since #673 was merged I'm going to close this as resolved.  If this is not the case please open actionable issues that can be addressed.


note about tags support

This is a bit misleading - while true for lib-rails and lib-rack, lib-metrics still supports both tags and source-based submission.
@chancefeick - The docs for this gem need to be updated, they still all refer to submission with sources (or nothing) except for the section specifically about tags. We should flip this so the default examples are tag-based but there is information about how to submit with sources if needed in a specific section.
This header-notice should note that the gem now submits with tags by default (and it should do that if it doesn't already), but should link down to the source subsection for people who are still on sources.


Using OAuth or Personal Access Tokens rathen than username/password

Hi,
Thanks for this nice app. I was wondering if it would be possible to use OAuth authentication or Personal Access Tokens to access my account rather than the username/password combo ?
Thanks again for your work.

Thank you so much @GMouron :)

Interesting, do you think that would be something you would like to do and we can provide the option in the sign in screen?

Hey, thanks for proposing. Unfortunately, react native is not really my area of expertise :-/

I think there was a little confusion, as we are using oAuth, not login/password authentication.
The username/password combo are requested in the GitHub oAuth screen, not on our side.
Closing this for now, feel free to re-open if the confusion is on my side :)


Community forum

Community Forums: Establish community forums or discussion boards where users can interact, share music recommendations, engage in conversations, and connect with fellow music enthusiasts.
I would like to work on this issue
@varun-singhh

Do ho have any idea how to begin with it, if you can provide me proof of concept you can start working on it.
Bttw assigning you the ticket it's a good idea tho.

I will create the responsive forum with the help of HTML, CSS, Javascript or Django either case the forum will show the title posts ,comment, last reply  etc , search box and if needed-a navbar.

But that will require a separate deployment for backend. Am I correct?

Only if I use Django


fix: (un)pad encrypted values in private store

resolves https://github.com/uport-project/3box-js/issues/38

@christianlundkvist made change for your comment, take a look again

@zachferland _pad and _unpad don't need to be instance methods. Do you think it would make more sense to just add them as separate functions?

yeah, but on the other hand just had find a place for them for now and this was the only place they are used. Wasn't concerned with placing them here for now since these are just private methods, if needed elsewhere they could be moved to a utils/crypto or something at that time. @oed any other place you would prefer them?

@zachferland I'd just make them standalone functions that are outside the class like so:
function pad (data) {}

But yeah, I don't think it matters that much. Feel free to merge!


[1376] Change vacancy status on course options to reflect study mode changes

Context
When a provider changes a course mode of study, we need to close any irrelevant course_options. In other words, if a course goes from being "full time and part time" to just "part time", then the full time course option should be closed (have no vacancies).
Changes proposed in this pull request

Additional logic in the TeacherTrainingPublicApi:: SyncSites job to update course options to match the current mode of study.

Guidance to review
Will this effect anything else?
Link to Trello card
Card for this task
Course option update epic with more information

 Bug: TTAPI allow closing study_mode for course

About the PR description question:

Will this effect anything else?

It will affect when the candidate selects the course or already have a choice option selected (in draft or submitted).
I recommend to check on candidate interface what happen on each scenario.
If we don't have any feature tests for this I'd recommend to add one.
We do have all add course selection and submission scenarios tested on
spec/system/candidate_interface/continuous_applications/course_selection/
spec/system/candidate_interface/continuous_applications/submitting/

About the PR description question:

Will this effect anything else?

It will affect when the candidate selects the course or already have a choice option selected (in draft or submitted).
I recommend to check on candidate interface what happen on each scenario. If we don't have any feature tests for this I'd recommend to add one. We do have all add course selection and submission scenarios tested on spec/system/candidate_interface/continuous_applications/course_selection/ spec/system/candidate_interface/continuous_applications/submitting/

I'm not sure what the expectations should be for the user tests / what the expected user journey is. Eg.
GIVEN I have selected a part-time options for a course
AND the course changes to full-time only.
WHEN I view my application
THEN I see...?
I'm not sure what

Code looks good 🚀.
I have 3 considerations:

Can we have a review app (I think automatically will point to Publish QA but we need to double check) and simulates changing the course study mode on publish and triggering the sync)
No problem.
Can we have some query in place after this code is merged so then we can see how many courses were set the vacancies when the study mode change happened? (I don't think we use audits for courses, it worth discussing how to proper investigate if there is any problem).
Where would this query live? I'm afraid I'm not clear what you are asking.
One thing that always bothered me on TTPAPI integration is that the logging is almost non existent. Should we start to add more logging? (only for this change)
I think it would be fine to add more logging, but I'm not clear what are our minimum expectations for logging and if it is aligned the update logging card currently in ready-for-dev. It's good to have clear requirements otherwise we are just introducing scope creep to this ticket.

About the PR description question:

Will this effect anything else?

It will affect when the candidate selects the course or already have a choice option selected (in draft or submitted).
I recommend to check on candidate interface what happen on each scenario. If we don't have any feature tests for this I'd recommend to add one. We do have all add course selection and submission scenarios tested on spec/system/candidate_interface/continuous_applications/course_selection/ spec/system/candidate_interface/continuous_applications/submitting/

I'm not sure what the expectations should be for the user tests / what the expected user journey is. Eg. GIVEN I have selected a course with part-time and full-time options for a course AND the course changes to full-time only. WHEN I view my application THEN I see...?
There are no UI changes noted on the ticket, and I can't find anything relevant in the epic.

I assume for submitted nothing changes and the unsubmitted (draft) they can't submit after the course has no vacancies.  Can we maybe align this with PM to see if I got it right?

Code looks good 🚀.
I have 3 considerations:

Can we have a review app (I think automatically will point to Publish QA but we need to double check) and simulates changing the course study mode on publish and triggering the sync)

No problem.

Can we have some query in place after this code is merged so then we can see how many courses were set the vacancies when the study mode change happened? (I don't think we use audits for courses, it worth discussing how to proper investigate if there is any problem).

Where would this query live? I'm afraid I'm not clear what you are asking.
Sorry I was not clear. I mean the moment this in production, let's say there is a problem and then we need to investigate how many courses were changed part time or full time to just part time or just full time. It would be good to know easily and quickly about this. We do have audits on course model, only in Publish, so I assume we can use that? I think the query can live in a documentation about troubleshooting the application? In case there is none we can start to create one?

One thing that always bothered me on TTPAPI integration is that the logging is almost non existent. Should we start to add more logging? (only for this change)

I think it would be fine to add more logging, but I'm not clear what are our minimum expectations for logging and if it is aligned the update logging card currently in ready-for-dev. It's good to have clear requirements otherwise we are just introducing scope creep to this ticket.
I understand, not my goal to scope creep this ticket, my goal is to have things in place if something goes wrong. I think adding a logger just to say the course id x, code y, provider code z were set to no vacancies and that's it? I think this won't affect the logging ticket.

Closing this PR
We need design and user journey decisions before proceeding, mostly around the candidate experience -- how do we handle Candidates who have selected options which are no longer available? How are they notified? What do they see on the application form.


run example video upload

https://github.com/EverNorif/chord_simulation/assets/79133788/e014906e-cae7-491a-93eb-5d04f643eec3

https://github.com/EverNorif/chord_simulation/assets/79133788/3e3d0049-69a1-472d-99b8-401dfa9dc89a


fix(create-vite): strict checks on vite.config.ts

Description
After creating a project using create-vite with React and TypeScript, and adding plugin:@typescript-eslint/strict-type-checked according to README.md, rule @typescript-eslint/no-unnecessary-condition throws an error on vite.config.ts when running lint.
/path/to/my-project/vite.config.ts
  0:1  error  This rule requires the `strictNullChecks` compiler option to be turned on to function correctly  @typescript-eslint/no-unnecessary-condition

Here is a small project to reproduce this error: https://github.com/nekketsuuu/vite-config-js-strict/commit/380208d43e5348293297e454d5d94c41d9d56fee.
That error is because TS strict mode is not enabled at tsconfig.node.json. So, this patch enables the same linting config at tsconfig.node.json as that of https://github.com/vitejs/vite/blob/cd6bf29ca4b677ff0d6e2bf36416a65b1a6ffa21/packages/create-vite/template-react-ts/tsconfig.json.
Also see: 4ffaeee2033531788bc6d4d2541b8b5e44352591 https://github.com/vitejs/vite/pull/12604
Follow-up: 2ad78aa205563f87b1607d0789608c13695cd9da https://github.com/vitejs/vite/pull/13749
Additional context
Nothing.

What is the purpose of this pull request?

[x] Bug fix
[ ] New Feature
[ ] Documentation update
[ ] Other

Before submitting the PR, please make sure you do the following

[x] Read the Contributing Guidelines.
[x] Read the Pull Request Guidelines and follow the PR Title Convention.
[x] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[x] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[ ] Ideally, include relevant tests that fail without this PR but pass with it.

Hi!
I think this good to improve this small TS config but it will be better to make it more consistent with the rest

The structure should follow the main one and only diverge for what makes sense for node
The change should be made too all TS templates

Hi. Thanks for your advise! Unfortunately, I cannot afford the time to improve all other TS templates. So I'm closing this pr, but feel free to change these configs later.


adds hasEnded() method to playbacks, container, and player

Tested in Chrome with Html5Video, FlasHLS and HLS playbacks.

A dependency has updated recently and broken karma. All the tests passed on my machine, I npm update'd, and now I get the same error as on travis. Also a build that previously passed on travis now fails.

I'm in favor of another property (like we did for ready) or we should change everything to methods :P what do you think @clappr/core ?
get ended() {  return this._ended }
//or
isEnded() { return this._ended }

I vote in favor of property because the states (ready, ended, paused ...) sound more like a property than a method/function.

I tend to prefer methods because if I see a property I tend to think it's something that can be set by something else/I can set it to something, but I guess that's just what I'm used to from other languages.
If it's in the api documentation I guess it should be clear to the developer. Also the video element itself uses properties.

just curious about the use case of this method, why not listen to PLAYBACK_ENDED?

Listening to the event would be fine but it's not enough alone because ended can go from true to false if the user seeks somewhere else/plays again.
I think this might be why the html5 video element has the ended property as well as the event.

@clappr/core are we going to use attribute or method to cover this ? hasEnded (or ended)

@leandromoreira I think a dependency must have been updated recently because I just reran the build on another PR that failed for me before and now that's working too, and this one's working now as well.
I didn't realise before that the get variable properties can't be replaced by the developer, the new values just get ignored.
Therefore I think just ended makes sense, as that matches the video tag api.

Cool @tjenkinson are you going to rebase or do another PR for ended instead of hasEnded()

I think I'll create a new one as it doesn't really fit this title anymore.

On 9 Jan 2016, at 21:09, Leandro Moreira<EMAIL_ADDRESS>wrote:
Cool @tjenkinson are you going to rebase or do another PR for ended instead of hasEnded()
—
Reply to this email directly or view it on GitHub.

I'll close, thanks once more :)


Regional recovery hangs on hook requiring ready pod/container with not-yet-recovered volume

Problem
Hooks are executed in the cluster data recovery phase.  VR is created later in protection phase.  VR is needed to make pod/container ready.  If hook is to be executed in a container that attaches a volume requiring a primary VR be present, recovery deadlocks.
$ kubectl get recipe -nasdf asdf
spec:
  appType: ""
  groups:
  - excludedResourceTypes:
    - deploy
    - po
    - pv
    - rs
    - volumereplications
    - vrg
    name: everything-but-deploy-po-pv-rs-vr-vrg
    type: resource
  - includedResourceTypes:
    - deployments
    - pods
    labelSelector:
      matchExpressions:
      - key: pod-template-hash
        operator: DoesNotExist
    name: deployments-and-naked-pods
    type: resource
  hooks:
  - labelSelector:
      matchLabels:
        appname: busybox
    name: busybox
    onError: fail
    ops:
    - command:
      - date
      container: busybox
      name: date
    type: exec
  recoverWorkflow:
    failOn: any-error
    sequence:
    - group: everything-but-deploy-po-pv-rs-vr-vrg
    - group: deployments-and-naked-pods
    - hook: busybox/date

$ kubectl describe po -nasdf -lappname=busybox
Events:
  Type     Reason       Age                    From     Message
  ----     ------       ----                   ----     -------
  Warning  FailedMount  24m (x9 over 108m)     kubelet  Unable to attach or mount volumes: unmounted volumes=[mypvc], unattached volumes=[default-token-rl9f9 mypvc]: timed out waiting for the condition
  Warning  FailedMount  20m (x34 over 115m)    kubelet  Unable to attach or mount volumes: unmounted volumes=[mypvc], unattached volumes=[mypvc default-token-rl9f9]: timed out waiting for the condition
  Warning  FailedMount  5m33s (x55 over 115m)  kubelet  MountVolume.MountDevice failed for volume "pvc-b2d5afb4-9dc2-4299-9f5d-94b36a60aabd" : rpc error: code = Internal desc = fail to check rbd image status: (cannot map image replicapool/csi-vol-acd02406-21f4-4f07-8616-9a1cdea57aa3 it is not primary)

After creating VR manually as primary, recovery resumes
$ kubectl describe po -nasdf -lappname=busybox
Events:
  Type    Reason     Age   From               Message
  ----    ------     ----  ----               -------
  Normal  Scheduled  38s   default-scheduler  Successfully assigned asdf/busybox-75666c475f-lwckq to cluster1
  Normal  Pulled     15s   kubelet            Container image "docker.io/library/busybox:stable" already present on machine
  Normal  Created    14s   kubelet            Created container busybox
  Normal  Started    14s   kubelet            Started container busybox

Approaches
Approach 1 - Create VR in recovery phase when corresponding PV and PVC are recovered
Approach 2 - Recover Kube objects after VR creation
Also, wait for DataReady condition.
Defer setting of ClusterDataReady until Recipe's recovery workflow completes.  Need internal ClusterDataReady sub-conditions for PV/PVC and Recipe.

Thanks @ShyamsundarR.  I added an approach 2.

Labeled medium priority because issue causes recovery to hang.


add fireDetection

添加火灾和烟雾检测demo

 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.


Add UsedImplicitly to visualizers and entity systems

Because Rider is annoying sometimes.

How about you add [MeansImplicitUse] to the base classes instead so we don't have to add the attribute to each inheritor?

How about you add [MeansImplicitUse] to the base classes instead so we don't have to add the attribute to each inheritor?

I tried and it didn't work for me, only when used with annotations.

Weird, I tried doing it before posting the comment and it worked. Then again I'm on Rider EAP...

Weird, I tried doing it before posting the comment and it worked. Then again I'm on Rider EAP...

Now it doesn't even show anything as unused even without any attributes.

Rider moment


New version: MPI v0.9.0

Registering package: MPI
Repository: https://github.com/JuliaParallel/MPI.jl
Created by: @lcw
Git reference: register/MPI/v0.9.0
Version: v0.9.0
Commit: 0140244acd21d92d52f71ab05ce437f879695340
Reviewed by: @lcw
Reference: https://github.com/JuliaParallel/MPI.jl/commit/0140244acd21d92d52f71ab05ce437f879695340#commitcomment-33569820

Can you add
[compat]
julia = "1"

to the project file and retrigger the bot?

bump @lcw


device discovery is kinda slow at startup

When running 'flutter daemon', It takes 4 seconds after sending "device.enable" before seeing the first "device.added" event.
[{"method": "device.enable", "id": 0}]
[{"id":0}]
[{"event":"device.added","params":{"id":"514C4FCC-6CC2-4620-8101-A06E87A86C26","name":"iPhone SE","platform":"ios","emulator":true}}]

Looking at PollingDeviceDiscovery, I see:
  static const Duration _pollingDuration = const Duration(seconds: 4);

It seems we wait 4 seconds before doing anything.

A second (minor) issue is that the caller doesn't know when it got the initial device list, except based on event timing.
I wonder if we could fix both of those by polling immediately, before returning the response to device.enable? That is, do this:
[{"method": "device.enable", "id": 0}]
[{"event":"device.added","params":{"id":"514C4FCC-6CC2-4620-8101-A06E87A86C26","name":"iPhone SE","platform":"ios","emulator":true}}]
[{"id":0}]


chore(main): release 0.8.0

:robot: I have created a release beep boop
0.8.0 (2024-05-02)
Features

Chat Item (#37) (c2dd630)
Filter Selection (#36) (53af9e7)
Select input (#35) (07ddba1)
ZetaScreenHeaderBar (#39) (e07fc48)

Bug Fixes

Avatar update (#30) (3f51508)
Update banners (#41) (0a881b5)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/ZebraDevs/zeta_flutter/releases/tag/v0.8.0 :sunflower:


MI - TRANSPORTS (VFS Servlet)

Complete [1].
[1] https://github.com/wso2/docs-ei/blob/master/en/micro-integrator/docs/references/transports/vfs-transport.md

Moved the contents to 'Concepts' [1] and the 'Transport Parameters' [2] pages.
[1] https://github.com/wso2/docs-ei/blob/master/en/micro-integrator/docs/concepts/concepts.md
[2] https://github.com/wso2/docs-ei/blob/master/en/micro-integrator/docs/references/transports/transport-catalog.md


Define DataAccess Layer.

I think we cloud share "DataAcces" module between controllers(Clients and Policies) since the functionality is identical and the only thing that changes is the URL to where the request is made.

We are going to receive in firs arguments called factory functions of cache and auth.
import Cache = require('...')
import Oauth2 = require('...')

cache = Cache()
auth= Oauth2()

DataAccess ({ cache, auth})

We are shearing DataAccess module between Users and Policies controllers.
`import Cache = require('...')
import Oauth2 = require('...')
import Users= require('...')
import Policies= require('...')
cache = Cache()
auth= Oauth2()
const users = Users({
dataAccess: DataAccess ({ cache, auth})
})
const policies = Policies({
dataAccess: DataAccess ({ cache, auth})
})
`


[STYLE CHANGE] Change the style of the filter modal

Description:
The modal that appears when the filter button is clicked is too big in size but the contents occupy very less space which does not look visually good from a perspective of a user.
Proposed Styling:
I would like to improve the styling of the current modal similar to the website's design.
Screenshots:
The current view looks like this:

@Avinash905
go for it


default profile also mute my speakers

Hello,
Somehow when using my Jabra phs200, the default profile mutes not only it's microphone but also the speaker.
I've tested with the default profile set to mute "default", or the "jabra mic..." or "all microphones" and it still heppens.
The strange thing is that if I create a "Profile 2" with the same settings it works fine and only the mic gets muted and not the speaker.
Looking at the settings.json, both default and profile2 entries are exactly identical except profile name...
No idea what is special about the Default profile that fails. That did not happen with another PC and another soundard... But works fine with my other all-in-one usb jabra headser for example.
Not sure how to give some more logs or details, I'm not sure how I can help further...

Could you give me a copy of the log when the default profile is set?
you can open the log by clicking the Help option in the tray menu while holding Shift and then click Copy Log

This is the log and config.json :
#2 51750: [Main] Running as user ***, A_IsAdmin = 0
#3 51750: [UI] Creating 'config' window
#4 52062: [UI] Creating 'about' window
#5 52296: [Main] Initilizing MicMute
#6 52296: [Config] Importing config.json
#7 52453: [Tray] Initilizing tray menu
#8 52484: [Main] Switching to profile 'Default'
#9 52484: [OSD] Creating OSD window
#10 52546: [MicrophoneController] Enabled: [ Interphone avec annulation d'écho (Jabra SPEAK 510 USB) ]
#11 52578: [Config] exporting the config object
#12 52578: [Main] MicMute startup took 844ms
#13 62640: [Updater] Fetching tag_name from https://api.github.com/repos/SaifAqqad/AHK_MicMute/releases/latest
#14 95671: [MicrophoneController] Disabled: [ Interphone avec annulation d'écho (Jabra SPEAK 510 USB) ]
#15 95671: [Main] Switching to profile 'Profile 2'
#16 95671: [OSD] Creating OSD window
#17 95671: [MicrophoneController] Enabled: capture
#18 108968: [MicrophoneController] Disabled: capture
#19 108968: [Main] Switching to profile 'Default'
#20 108968: [OSD] Creating OSD window
#21 108968: [MicrophoneController] Enabled: [ Interphone avec annulation d'écho (Jabra SPEAK 510 USB) ]

	"AllowUpdateChecker": 1,
	"DefaultProfile": "Default",
	"ForceMicrophoneState": 0,
	"MuteOnStartup": 0,
	"PreferTheme": -1,
	"Profiles": [
		{
			"afkTimeout": 0,
			"ExcludeFullscreen": 0,
			"LinkedApp": "",
			"Microphone": [
				{
					"MuteHotkey": "AppsKey",
					"Name": "All Microphones",
					"PushToTalk": 0,
					"UnmuteHotkey": "AppsKey"
				}
			],
			"OnscreenFeedback": 1,
			"OnscreenOverlay": 0,
			"OSDPos": {
				"x": 1922,
				"y": 1100
			},
			"OverlayPos": {
				"x": -1,
				"y": -1
			},
			"OverlayShow": "2",
			"OverlayUseCustomIcons": 0,
			"ProfileName": "Default",
			"PTTDelay": 100,
			"SoundFeedback": 0,
			"SoundFeedbackDevice": "Default",
			"SoundFeedbackUseCustomSounds": 0,
			"UpdateWithSystem": 1
		},
		{
			"afkTimeout": 0,
			"ExcludeFullscreen": 0,
			"LinkedApp": "",
			"Microphone": [
				{
					"MuteHotkey": "AppsKey",
					"Name": "Default",
					"PushToTalk": 0,
					"UnmuteHotkey": "AppsKey"
				}
			],
			"OnscreenFeedback": 1,
			"OnscreenOverlay": 0,
			"OSDPos": {
				"x": 1926,
				"y": 1098
			},
			"OverlayPos": {
				"x": -1,
				"y": -1
			},
			"OverlayShow": "2",
			"OverlayUseCustomIcons": 0,
			"ProfileName": "Profile 2",
			"PTTDelay": 100,
			"SoundFeedback": 0,
			"SoundFeedbackDevice": "Default",
			"SoundFeedbackUseCustomSounds": 0,
			"UpdateWithSystem": 1
		}
	],
	"SwitchProfileOSD": 1
}```

I didn't realize I could change the default profile in there so it just fixed my issue by changing the default profile to `Profile 2`.
Well, not fixed the issue, I still see the speaker being muted even in the little icon left of the clock when I press my toggle and I'm using the Default profile, but now MicMute starts with my own profile on start that doesn't have the issue ;) So urgency for this drops way way down ;)

I think the problem is that the device's driver is using the same exact name for both the microphone and the speaker, usually, devices like this differentiate between the recording and the playback devices by assigning them different names (like the corsair device in the pics below)

does the problem persist If you change the microphone's name?

You nailed it, I renamed both :

And the problem went away even in the default profile !
Wasn't sure I was allowed to rename it since it's a corporate PC and I'm not admin on it... But I did and it fixed it !  I'll test later if that resists reboots and if other models of the same speaker kindly use my new name of if I have to rename them for this to work.
Nevertheless I'm already fully avoiding the issue now that my default profile is Profile2 ;)
Unless you see something that could help you mute only input sources and not outputs and it makes sense for you to change it, we can safely close this even as wontfix and there are at least 2 workarounds for other users in this thread ;)
Also, let me take the opportunity to highlight one of the things I love about this tool : I can use it as a simple user without admin rights on my corporate machine, so avoiding priviledge escalation requirements in install/update/autostart/run is really a great feature for me ;) Thanks a lot for your time and effort !

That's great, I'm still gonna fix this issue though. It should not touch output devices at all.


Senadina Repair/Delta/Cache Compatibility Fix by @neon-nyan

JHFBABFALIFBA:AB:UJAIBF:UQ:UBF:QUHQ:UFQFUIQ
yes, this took a time
sorry.
-team
lord, help us

I won't do any CodeQA further than anything in main CollapseLauncher project, for EncTool's CodeQA session will come at a later date due to time (and sanity! <-- definitely not null) constraint


Extract conda files into submodule

Need to be used also by other packages for testing.
Or should these packages simply wget it from the main package?

Actually the latter is much easier.


History data of temperature and humidity

Is it possible to get history data of temperature and humidity from devices like H5103? I mean that it is possible to view old data like from hue sensors in Home+ or EVE app?

Only missing in Home+. Should be problem with app


chore(ci): move papyrus_execution dep to root toml

This change is

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#280  👈
#279
#278
#277
#276
#275
#274
#273
#272
#271
#270
#269
#268
#267
#266
#265
#264
#263
#262
#261
#260
#259
#258
#257
#256
#255
main-v0.13.2

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @dorimedini-starkware and the rest of your teammates on  Graphite


Issue with fortios_firewall_addrgrp

I have the commit 6a7973c89ad2824532bd4b19601e4f0f21624761 (current head) of ansible_fgt_modules in library of my ansible repo, and I’ve got this playbook:
---
- hosts: localhost
  gather_facts: false
  tasks:
  - fortios_firewall_addrgrp:
      host: "<IP_ADDRESS>"
      username: breakglass
      password: mypassword
      vdom: root
      https: True
      firewall_addrgrp:
        state: present
        name: all_the_things
        member:
        - source_group_a

Assuming “source_host_a” exists as an address object, I believe this should create a group “source_group_a” with one member - “source_host_a”.
Instead, I get:
PLAY [localhost] *******************************************************************************************************************************************************************************

TASK [fortios_firewall_addrgrp] ****************************************************************************************************************************************************************
fatal: [localhost]: FAILED! => {"changed": false, "meta": {"build": 200, "error": -1, "http_method": "POST", "http_status": 500, "name": "addrgrp", "path": "firewall", "revision": "<IP_ADDRESS>50766792421668275.1551864364", "serial": "FGVM08TMREDACTED", "status": "error", "vdom": "root", "version": "v6.0.3"}, "msg": "Error in repo"}

PLAY RECAP *************************************************************************************************************************************************************************************
localhost                  : ok=0    changed=0    unreachable=0    failed=1

I have previously created the address object using the api and the same credentials… so I know that works.
Is there anything I can check on the firewall to see why it’s returning a “500”? “diagnose debug enable / diagnose debug cli 8” isn’t showing me anything for this item.

Hi @JonTheNiceGuy ,
this case is similar to what is described in https://github.com/fortinet-solutions-cse/ansible_fgt_modules/issues/7.
I'm showing a working example:
- hosts: localhost
  vars:
   host: "<IP_ADDRESS>"
   username: "admin"
   password: ""
   vdom: "root"
  tasks:
  - name: Configure IPv4 address groups.
    fortios_firewall_addrgrp:
      host:  "{{ host }}"
      username: "{{ username }}"
      password: "{{ password }}"
      vdom:  "{{ vdom }}"
      https: "False"
      firewall_addrgrp:
        state: "present"
        allow-routing: "enable"
        color: "4"
        comment: "Comment."
        member:
         - name: "all"
         - name: "google-play"
        name: "source_group_a"
        visibility: "enable"

The difference is in the way 'member' is used. We have to add 'name' before the address. This example is creating an address group named "source_group_a" with addresses "all" and "google-play" ('all' already covers everything but it is just there to show multiple entries are possible).
Note that this module is already approved to be in Ansible 2.8 and you can see the official documentation with a better look:
https://docs.ansible.com/ansible/devel/modules/fortios_firewall_addrgrp_module.html#fortios-firewall-addrgrp-module
Another hint: It is useful to run some debug commands to check why FortiGate is complaining about ansible/http api operations. Just type in the CLI:
diagnose debug application httpsd -1
diagnose debug enable
And the traces will give you some useful hints.
Hope that helps


API  Management exercise failure

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az functionapp deployment source config-zip
Errors:
The command failed with an unexpected error. Here is the traceback:
Expecting value: line 1 column 1 (char 0)
Traceback (most recent call last):
  File "/opt/az/lib/python3.6/site-packages/knack/cli.py", line 231, in invoke
    cmd_result = self.invocation.execute(args)
  File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/__init__.py", line 657, in execute
    raise ex
  File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/__init__.py", line 720, in _run_jobs_serially
    results.append(self._run_job(expanded_arg, cmd_copy))
  File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/__init__.py", line 691, in _run_job
    result = cmd_copy(params)
  File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/__init__.py", line 328, in __call__
    return self.handler(*args, **kwargs)
  File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/command_operation.py", line 121, in handler
    return op(**command_args)
  File "/opt/az/lib/python3.6/site-packages/azure/cli/command_modules/appservice/custom.py", line 394, in enable_zip_deploy_functionapp
    remove_remote_build_app_settings(cmd, resource_group_name, name, slot)
  File "/opt/az/lib/python3.6/site-packages/azure/cli/command_modules/appservice/custom.py", line 526, in remove_remote_build_app_settings
    should_contain=app_settings_should_contain)
  File "/opt/az/lib/python3.6/site-packages/azure/cli/command_modules/appservice/custom.py", line 985, in validate_app_settings_in_scm
    scm_settings = _get_app_settings_from_scm(cmd, resource_group_name, name, slot)
  File "/opt/az/lib/python3.6/site-packages/azure/cli/command_modules/appservice/utils.py", line 74, in call
    raise exception
  File "/opt/az/lib/python3.6/site-packages/azure/cli/command_modules/appservice/utils.py", line 70, in call
    return func(*args, **kwargs)
  File "/opt/az/lib/python3.6/site-packages/azure/cli/command_modules/appservice/custom.py", line 1016, in _get_app_settings_from_scm
    return response.json() or {}
  File "/opt/az/lib/python3.6/site-packages/requests/models.py", line 900, in json
    return complexjson.loads(self.text, **kwargs)
  File "/opt/az/lib/python3.6/json/__init__.py", line 354, in loads
    return _default_decoder.decode(s)
  File "/opt/az/lib/python3.6/json/decoder.py", line 339, in decode
    obj, end = self.raw_decode(s, idx=_w(s, 0).end())
  File "/opt/az/lib/python3.6/json/decoder.py", line 357, in raw_decode
    raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az functionapp deployment source config-zip -g {} -n {} --src {}

Expected Behavior
Environment Summary
Linux-5.4.0-1051-azure-x86_64-with-debian-10.2 (Cloud Shell)
Python 3.6.10
Installer: DEB

azure-cli 2.25.0 *

Extensions:
ai-examples 0.2.5
ssh 0.1.5

Additional Context

route to service team

Thanks @yuva2g I'll take a look

Dupe of #18160


'mat-drawer-container' is not a known element not works even though i installed all necessary @Angular/material library

Is this a regression?

[X] Yes, this behavior used to work in the previous version

The previous version in which this bug was not present was
No response
Description
Error: src/app/app.component.html:1:1 - error NG8001: 'mat-drawer-container' is not a known element:

If 'mat-drawer-container' is an Angular component, then verify that it is part of this module.
If 'mat-drawer-container' is a Web Component then add 'CUSTOM_ELEMENTS_SCHEMA' to the<EMAIL_ADDRESS>of this component to suppress this message.

Reproduction
Error: src/app/app.component.html:1:1 - error NG8001: 'mat-drawer-container' is not a known element:

If 'mat-drawer-container' is an Angular component, then verify that it is part of this module.
If 'mat-drawer-container' is a Web Component then add 'CUSTOM_ELEMENTS_SCHEMA' to the<EMAIL_ADDRESS>of this component to suppress this message.

Expected Behavior
Error: src/app/app.component.html:1:1 - error NG8001: 'mat-drawer-container' is not a known element:

If 'mat-drawer-container' is an Angular component, then verify that it is part of this module.
If 'mat-drawer-container' is a Web Component then add 'CUSTOM_ELEMENTS_SCHEMA' to the<EMAIL_ADDRESS>of this component to suppress this message.

Actual Behavior
Error: src/app/app.component.html:1:1 - error NG8001: 'mat-drawer-container' is not a known element:

If 'mat-drawer-container' is an Angular component, then verify that it is part of this module.
If 'mat-drawer-container' is a Web Component then add 'CUSTOM_ELEMENTS_SCHEMA' to the<EMAIL_ADDRESS>of this component to suppress this message.

Environment

Angular:
CDK/Material:
Browser(s):
Operating System (e.g. Windows, macOS, Ubuntu):

HI @ksabada, may you please complete the Environment part of your message?

It looks like the MatSidenavModule isn't imported in the correct place.


Cypress Testing

Cypress is a testing framework I can finally work with 👍
I'm writing tests now. This will take a while but I might merge this before having much coverage and do this gradually over the next releases.
While not a breaking change, this is the last feature missing for me to put leaflet.pm to version 1.0.
So, this will close #210.

awesome, I'm also very interested in Cypress but got stuck since our apps need certificate on and this is one thing that Cypress doesn't have solved :( looking forward how will it go!

it's beautiful...


Added handling of swipe cancellation via delegate

Added proper handling to swipe cancellation and a corresponding
onCancel callback. Closes #39.

Awesome--thanks, @stkhapugin!! :clap: