[Checkstyle] Checkstyle should check for * imports and refuse them, according to OpenSearch style guidelines

Is your feature request related to a problem?
The repo currently allows for * imports, which goes against opensearch's style guidelines:
https://github.com/opensearch-project/OpenSearch/blob/main/gradle/formatting.gradle#L89-L94
This should be changed to match the upstream.
What solution would you like?
A clear and concise description of what you want to happen.
What alternatives have you considered?
A clear and concise description of any alternative solutions or features you've considered.
Do you have any additional context?
Add any other context or screenshots about the feature request here.

I noticed that it stopped checking the license header for new files.

I fixed it


(OUI component update) Update behavior of OuiComboBox with single selection

OuiComboBox component with singleSelection prop applied appears to be editable as the cursor stays present after selection is made.
Although the selection isn't editable, a user could press backspace to delete the selection from the box. The element seems to use that behavior regardless of how its singleSelection and isClearable settings are configured. It doesn't appear that we have a means of removing the cursor in the component.
Describe the solution you'd like
Hide the cursor when selection is made.
Describe alternatives you've considered
@AWSHurneyt researched the ways to remove the cursor using the current implementation of the component.
Additional context
Originating from this issue: https://github.com/opensearch-project/alerting-dashboards-plugin/issues/257

@xeniatup  can you give some insight into the "why" of why you would like to remove this cursor.
In the case of a single selection, the cursor remains in the input box and the user can still utilize the search bar to quickly replace the value or to clear the value using back space (as per your line "...a user could press backspace to delete the selection from the box.").
Removing the cursor has deep implications in usability and accessibility for those who interact with the interface primarily via keyboard. It would also impact the interaction across all components utilizing the ComboBox primitive.
isClearable controls if the X appears to quickly clear the selection, not if the element is immutable or not.

I believe the cursor is intended behavior as that's what allows for searching the drop down if you want to change your selection

@kgcreative Are you good with us closing this with "not planned"?`

I'm ok closing as "won't fix"


Implement flag sqrlonly

I'm not sure if Wordpress is going to allow the deletion of the basic method of having an account. That's going to be very interesting. Once you have an email in the system, there's always the chance of a password reset to be performed.

Hi @brianoflondon
I have a hard time keeping these features apart.
But sqrlonly is similar to disabled but with only password login so that's doable.
But hardlock disabling external changes are harder, maybe I will set the flag and inform the user on the profile page that it is not possible at the moment.
Best regards
Daniel


Login User Flow

This pr I created the user flow of a user who is not new to growpod and is logging in. It will sign the user in using google authentication and redirect the user to "My Gardens". Also included in this pr are the functions needed to post user data to the backend and put that data into a map. Those functions are not used in this pr and will be implemented in the next pr where I create the flow for a new user who is signing up and creating an account.
Things to improve on in another pr:

Better styling for the actual login page

@kwell755: it is worth fixing the merge conflicts sooner rather than later. Ping me if you'd like help resolving them.

@stephypy: don't forget to leave a review!


No matching distribution for tf

Trying to install with pip

Mh. Given that TF has changed lot since then (given the integration of Keras), it may be that they don't have the version on PyPI anymore. But I'm sure there is a way to manually install a TF version that satisfies the requirement.

Okay, will try to install it manually.

Hey @marcus-voss
I am still not able to install 1.8, tried installing from whl but it is not working

I assume you have a Linux with 64 bit? Just to make sure.
I'm happy to support you getting it to run, but have no specific insights into how get older TF version. Unless @AlexElvers happens to know something, I can try myself on the weekend.

I assume you have a Linux with 64 bit?  Yes
Sure,Let me know if there is any work-around.
Thanks!

One possibility, but I guess that depends on your skills with TF, would be to migrate it to TF 2.0. See for instance:

https://www.tensorflow.org/guide/migrate
https://www.tensorflow.org/guide/upgrade
I once did this for a script, and it can be as simple as changing some imports, but may also need some manual adjustments.

Hey @dmoonat!
I would guess that you are using a Python version that is not compatible with the wheel. The wheel is compiled for Python 3.6 (the filename contains 'cp36'). Unfortunately, more recent versions of Python cannot be used for TensorFlow 1.8.
In Python 3.6, installing the TensorFlow wheel from https://pypi.org/project/tensorflow/1.8.0/#files (tensorflow-1.8.0-cp36-cp36m-manylinux1_x86_64.whl) works on my computer.

One more thing, how we can run it for custom dataset?changes needed?

Hey!
Using this data loader config, you can load data from data/pecanstreet/boulder/H/boulder_H_load_agg_10.csv. The data should look as follows:
$ head -n3 data/pecanstreet/boulder/H/boulder_H_load_agg_10.csv
,10_1,10_2
2015-01-01 00:00:00-07:00,5.657,9.287
2015-01-01 01:00:00-07:00,8.052,8.347

The corresponding weather data is loaded from data/pecanstreet/boulder/boulder_weather.csv:
$ head -n3 data/pecanstreet/boulder/boulder_weather.csv
,temperature,apparent_temperature,dew_point,humidity,visibility,pressure,wind_speed,cloud_cover,wind_bearing,precip_intensity,precip_probability
2015-01-01 00:00:00-07:00,-12.86,-17.07,-1.67,0.61,10.00,1029.59,4.04,0.05,245.00,0.00,0.00
2015-01-01 01:00:00-07:00,-13.44,-18.17,-2.50,0.62,10.00,1028.76,4.54,0.02,242.00,0.00,0.00

If you are using a different format, you can add a custom data loader class. The return value of read_data should be a DataFrame with timestamp as index, one column named 'load' containing load data and weather (or other) data in the other columns.
Here is an example for an estimator config that also might help you:
type: CNN1Estimator
time_horizon: !ref 'hor-24h-H'
quantile_levels: !ref 'q-7'
history_shape: [7, 24]
conv_layers_params:
- filter_shape: [5, 5]
  num_filters: 26
  pooling_shape: [1, 2]
fc_layers_params:
- {activation: relu, num_outputs: 100}
- {activation: relu, num_outputs: 82}
- {}
historical_inputs: []
horizon_inputs: [temperature]  # this will use the temperature column of the DataFrame

Hey, Thanks for sharing this info!
But i guess the data folder,files are missing from repo

Yes, we can't upload the data there. You will have to get a license from Pecan street. Unfortunately, they made the data unavailable for free even for academics. Now, you need an expensive license, or you can only get a smaller subset than the one we used.

Okay,No Worries!
@AlexElvers @marcus-voss Thanks for the continuous support!
Great approach to probabilistic forecasting!
Kudos to the team!
PS:[Closing this issue,will connect if have any other questions]


File Not Found IOException

For some reason the FFMPEG binary is not located on the /data/data/my.app.package/files
So whenever I try to use th FFMPEG I get
java.io.IOException: Error running exec(). Command: [/data/data/my.app.package/files/ffmpeg (...) ] Working Directory: null Environment: null
It used to work ok, but I had some major changes on other parts of the apps combined with a new SSD and now it's broken. Any idea?

Ok, nice, you have to execute the "isSupported". I changed from other forks and didn't include that part on the code. Working now, thanks for you work!


Managing packages with Lerna?

Background: OpenCensus Node uses Lerna to organize codebase into multi-package repositories. Lerna give us the ability to build libraries and apps in a single repo without forcing us to publish to npm or other registries. This results in faster code-test-debug cycles by sharing components locally.
https://hackernoon.com/the-highs-and-lows-of-using-lerna-to-manage-your-javascript-projects-ff5c5cd82a99 -> mentioned some highs and lows of using lerna.
@draffensperger shared below article with me (offline).
https://gist.github.com/nolanlawson/457cdb309c9ec5b39f0d420266a9faa4

I'm definitely open to Lerna, but in my limited experience with it for OpenCensus Web, it often felt like onre more dependency to have to manage (upgrade for security fixes, etc.) and that a lot of its functionality might be able to be replaced using simple scripts or tricks as explained in the gist above.

When working with OpenCensus I had the problem that the build on master was broken for some time because one component had a problem. To me, this whole experience felt very much like a monolithic approach with a lot of moving parts involved.
Here, I usually manage each dependency as a separate module. I simply add the local relative path as dev dependency and run tsc -w in them. This gives a very seamless experience as every change in every package will be propagated instantly.
That all said, I am sure that OpenCensus had good reasons to use lerna and a lot of expertise around it and I am open to learning to like it.

In my experience using it in a few large projects, it's really a must-have for monorepos. Using scripts is possible, but it becomes very cumbersome very quickly, and in the end you end up pretty much rewriting Lerna but as a less feature complete, more difficult to use, and with more bugs implementation since it's not an external project that can be contributed to easily.
I'd say this is a general rule for using scripts in general. For example why use npm or yarn at all? They are just downloading files and resolving dependencies using semver which can be done with scripts.
To be honest, I would even go as far as to say not using Lerna might end up preventing a lot of users from contributing due to how complex the project will become. This is especially true for instrumentation repos where many different library versions have to be tested at the same time.
Let's focus on our domain (telemetry) and not on managing monorepos.

Most of the downsides mentioned in the articles are also mitigated by combining Lerna with TypeScript and Yarn.

I'm fine with use of Lerna, just wanted to make sure including it is worth the extra weight of an extra dependency, sounds like in this case it is worth it!

All agreed to use Lerna in today's meeting.

It has its shortcomings (extra dependency) but it’s worth using it.
OpenCensus uses it and worked well.
It doesn't have the overhead on end users.


notify / notify job is failing after skipping publish / mirror job

As titled, here is the screenshot:

and the link to the log:
https://github.com/facebookexperimental/free-threading-benchmarking/actions/runs/10819936093/job/30027186768
Should we remove this line ?
Or we can just remove notify / notify job if we decide to skip publish / mirror job.
cc. @mdboom

Sorry, this needs docs. What this does is it puts a comment on an issue in your benchmarking repository (which then sends a notification through the Github notification system, which may result in an e-mail if the user has configured it that way). It's failing because it's trying to add a comment to issue #0, which doesn't exist.
If you:

create a new issue in your repo
Add it's issue number to [notify]/notification_issue = N in bench_runner.toml

Does that work? If so, I think all we need to do is add these instructions to the README.md

Oh my concern is that I saw it's trying to do something in f"(https://github.com/{github_repo}-public/tree/main/results/{dirname})" but this repo doesn't exist.

Oh my concern is that I saw it's trying to do something in f"(https://github.com/{github_repo}-public/tree/main/results/{dirname})" but this repo doesn't exist. We don't have a public repo.

Oh sorry! Never mind! It's just the notification body instead of really doing something in the public repo. I will try to create a new issue and see if that works. Thanks

Makes sense -- it probably also should be configurable where those links point to, since, as you say, you don't have a public repo so those links won't go anywhere useful.  Maybe we change the code so it doesn't add the -public if the publish configuration is turned off...


Schema validation fails when using expression

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest version
[X] Search the existing issues.

Steps to reproduce
As part of https://github.com/PowerShell/DSC/pull/291, discovered that if you have a schema that limits property values (like OSInfo resource), then you can't use an expression that would resolve to a valid value since schema validation happens before expression execution.
Expected behavior
Expressions should be executed before schema is validated

Actual behavior
Schema is validated before expression execution

Error details
No response
Environment data
NA

Version
dsc 3.0.0-alpha.5
Visuals
No response

I would certainly prefer a way to perform static validation before we run any code, but dynamically modifying resource schema seems like a recipe for disaster.  For now, my intent is to perform validation after expression invocation before calling the resource, but that means resources may run but validation can fail later leaving in an undesired state.


Saving/exporting/importing/tabs/bookmarks?

Some features that would be super useful:

entire chart save in query string for easy bookmarking?
allow exporting chart data for savings or later importing (json/csv/whatever)?
Save chart data somewhere like localstorage and add tabs at the top to work on multiple charts at once?

entire chart save in query string for easy bookmarking?

This is the solution I've had my eye on for a while. Just haven't had time to implement. The others are cool ideas as well.
Thanks for the feedback!

@dionsaputra Going to keep this Issue open because there is another feature I want to remember to add 👍

@rob-gordon thankyou for merged my previous PR. I was realized that it does not necessary to save complex json from cytoscape to generate the graph back. The syntax in text-editor is sufficient to do it, I have open another PR (#61) to export & import using the text in editor

@VictorioBerra wanted to mention you can now store separate charts at separate url's. See this for an example.
I'm leaving this issue open because I plan on implementing the query string idea soon as well, although more for read-only sharing purposes.

@rob-gordon  thanks! How do I save my URL?

@rob-gordon  thanks! How do I save my URL?

I might add a small navigation to move between charts in the future but for now you'll need to bookmark it / remember it.

Ohh, I meant when you hit the page, does it generate a moniker style url? https://www.npmjs.com/package/moniker
In the twitter gif, it wasn't clear how the URL is created.

Ah I see. Sorry it's not clear. You choose the url! (Just by typing it in)

@VictorioBerra Added a small Share button as well which opens a link to a read-only version of the chart, stored in the url, like you mentioned.
I'm going to go ahead and close this issue. Thank you for all the suggestions!


Fixed typo in readme

Fixed a small typo in the Setting Input Properties Manually section of the readme.

Thanks!


feat: ✨ add average salary problem

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@7b2329f). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             master        #2   +/-   ##
==========================================
  Coverage          ?   100.00%
==========================================
  Files             ?         2
  Lines             ?        73
  Branches          ?        12
==========================================
  Hits              ?        73
  Misses            ?         0
  Partials          ?         0

Continue to review full report at Codecov.

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


use safe_instance for plots fixes #1247

uses safe_instance to detect pandas in plotting functions

this no longer applies since plots take Explanation object


Support of Arrays in followupEvent

#38
Added support for arrays in AIEvent + extended tests

I signed it!
On 11 September 2017 at 20:13, googlebot<EMAIL_ADDRESS>wrote:

Thanks for your pull request. It looks like this may be your first
contribution to a Google open source project. Before we can look at your
pull request, you'll need to sign a Contributor License Agreement (CLA).
📝 Please visit https://cla.developers.google.com/
https://cla.developers.google.com/ to sign.
Once you've signed, please reply here (e.g. I signed it!) and we'll
verify. Thanks.

If you've already signed a CLA, it's possible we don't have your
GitHub username or you're using a different email address. Check your
existing CLA data https://cla.developers.google.com/clas and verify
that your email is set on your git commits
https://help.github.com/articles/setting-your-email-in-git/.
If your company signed a CLA, they designated a Point of Contact who
decides which employees are authorized to participate. You may need to
contact the Point of Contact for your company and ask to be added to the
group of authorized contributors. If you don't know who your Point of
Contact is, direct the project maintainer to go/cla#troubleshoot.
In order to pass this check, please resolve this problem and have
the pull request author add another comment and the bot will run again.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/api-ai/apiai-java-client/pull/39#issuecomment-328630007,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEPULuCGlxhQWMMK3FQHFYFFYsTv2TXRks5shYZtgaJpZM4PToyd
.

I signed it!


Accept an imported svg in Logo

👀 Overview
As well as a url string, I wanted to be able to pass an imported SVG to display in the Logo component. This lets me do just that.
Also adding a max-height to the AvatarGroup dropdown, since if theres a load of Avatars it can easily go off screen.
✅ Checklist

[ ] Tests written
[ ] Browser tested
[ ] Added to documentation
[ ] Added to storybook

 Definitely think we need a but more of a brand presence on the shareable link UI


dotnet-trx reporter: "Error: No test report files were found"

I've started using test-reporter with the following code (source):
name: .NET

on:
  push:
    branches: [ main ]
  pull_request:
    branches: [ main ]

jobs:
  build:

    runs-on: windows-latest

    steps:
    - uses: actions/checkout@v3
    - name: Setup .NET
      uses: actions/setup-dotnet@v2
      with:
        dotnet-version: |
          6.0.x
          5.0.x
    - name: Restore dependencies
      run: dotnet restore
    - name: Build
      run: dotnet build --no-restore
    - name: Unit tests
      run: dotnet test --verbosity normal --no-build --filter "FullyQualifiedName~Tests.Unit" --logger "trx;LogFileName=test-results.trx"
    - name: Unit tests report
      uses: dorny/test-reporter@v1
      if: success() || failure()    # run this step even if previous step failed
      with:
        name: Unit tests
        path: test-results.trx
        reporter: dotnet-trx

But, I'm receiving the following error on the "Unit tests report" step (see attempt):
Check runs will be created with SHA=da3a5b4d795ed79cb4ed16b4c8377837fbd1e77e
Listing all files tracked by git
Found 171 files tracked by GitHub
Using test report parser 'dotnet-trx'
Creating test report Unit tests
  Warning: No file matches path test-results.trx
Error: No test report files were found

How can I fix that?

@dariusz-wozniak you need to specify path as wildcard
...
    - name: Unit tests report
      uses: dorny/test-reporter@v1
      if: success() || failure()    # run this step even if previous step failed
      with:
        name: Unit tests
        path: '**/test-results.trx'
        reporter: dotnet-trx
...

I've just created a PR to your repository with fixed workflow

Thank you for the help, @sturman.
However, the merged PR fails:
Run dorny/test-reporter@v1
Check runs will be created with SHA=4ae3c23951059174c71107fc7030c7c6d6b55277
Listing all files tracked by git
Found 171 files tracked by GitHub
Using test report parser 'dotnet-trx'
Creating test report Unit tests
  Processing test results from tests/TestsAreFun.Tests.Acceptance/TestResults/test-results.trx
  Processing test results from tests/TestsAreFun.Tests.Approval/TestResults/test-results.trx
  Processing test results from tests/TestsAreFun.Tests.Integration/TestResults/test-results.trx
  Processing test results from tests/TestsAreFun.Tests.Load/TestResults/test-results.trx
  Processing test results from tests/TestsAreFun.Tests.Specification.LoFu/TestResults/test-results.trx
  Processing test results from tests/TestsAreFun.Tests.Specification.SpecFlow/TestResults/test-results.trx
  Processing test results from tests/TestsAreFun.Tests.Unit/TestResults/test-results.trx
  Processing test results from tests/TestsAreFun.Tests.Unit.Multitargeted/TestResults/test-results.trx
  Creating check run Unit tests
Error: Resource not accessible by integration

URL: https://github.com/dariusz-wozniak/TestsAreFun/runs/6852167174?check_suite_focus=true
Do you have any idea why it happens?

@dariusz-wozniak try to add these permissions
permissions:
  id-token: write
  contents: read
  checks: write

Works like a charm, thank you @sturman! 🎉


add delayed response

Codecov Report
Patch coverage: 25.00% and project coverage change: -3.60% :warning:

Comparison is base (c696f49) 91.33% compared to head (d744e59) 87.73%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##           master      #47      +/-   ##
==========================================
- Coverage   91.33%   87.73%   -3.60%
==========================================
  Files          88       90       +2
  Lines        1962     2047      +85
  Branches      227      245      +18
==========================================
+ Hits         1792     1796       +4
- Misses        157      237      +80
- Partials       13       14       +1

Files Changed
Coverage Δ

jj/responses/_delayed_response.py
0.00% <0.00%> (ø)

jj/responses/_response.py
97.26% <83.33%> (-1.58%)
:arrow_down:

jj/responses/_utils.py
100.00% <100.00%> (ø)

... and 1 file with indirect coverage changes

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


Align WebSocket.Frame.encode/2 spec with WebSocket.encode/2.

The Mint.WebSocket.shorthand_frame() is missing in the Mint.WebSocket.Frame.encode/2 spec.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>4f07b7ccb8cd7ac0c85bfe38fe143a-PR-29

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.3%) to 89.459%

Totals

Change from base Build d17898b3b37c980e33e9f428b07698312a8cbc85:
0.3%

Covered Lines:
314

Relevant Lines:
351

💛  - Coveralls


Tests that should abort?

Is it possible with nextest to have tests that should abort?
I'd like to have a better test suite for https://github.com/RalfJung/cargo-careful, but unfortunately the expected behavior I need to test for is that the entire process gets aborted. I figured with nextest's per-test process this would at least be possible, but when I tried adding #[should_panic] to one of these aborting tests, nextest still considers that a failure:
    Starting 4 tests across 3 binaries (1 skipped)
        PASS [   0.003s]        test-cargo-careful extern_dep
        PASS [   0.003s] test-cargo-careful::tests c_str
        PASS [   0.004s] test-cargo-careful::tests cfg_flag
     SIGABRT [   0.195s] test-cargo-careful::tests uninit_bool_array

--- STDOUT:              test-cargo-careful::tests uninit_bool_array ---

running 1 test

--- STDERR:              test-cargo-careful::tests uninit_bool_array ---
thread 'uninit_bool_array' panicked at 'attempted to leave type `[bool; 1]` uninitialized, which is invalid', /home/r/.rustup/toolchains/nightly-x86_64-unknown-linux-gnu/lib/rustlib/src/rust/library/core/src/panicking.rs:123:5
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
thread caused non-unwinding panic. aborting.

   Canceling due to test failure: 0 tests still running

That makes sense, but is there some way to say that SIGABRT is exactly what I expected? I didn't find anything in the docs, but maybe I missed it. If not, please consider this a feature request. :)

Nextest doesn't currently support this, but it should definitely be possible to add it! I think the main blocker for this is the UI design more than the implementation.
Some thoughts:

Configuration for this would be a per-test override, where you specify something like expect-status. The default would be expect-status = { exit-code = 0 }, but you can also specify a unix-signal or a windows-error-code. (Unix and Windows have different semantics here -- on Unix, it's either a signal or an exit code; while on Windows, the NTSTATUS exception code is communicated as the exit code. Need to find a way to avoid confusion here.)
For Unix signals we should accept the name (should be easy to map name -> number), for Windows we probably have to accept the error code as a number because there's no easy way to map a string like "EXCEPTION_ACCESS_VIOLATION" or "STATUS_ACCESS_VIOLATION" to a number AFAIK.
If a test produces an unexpected result, we need to figure out a way to display what's happening.

I'd like to get to this at some point but it might take a couple of months to do.


Update Yavrio in ConnectivityAppDefinitions.Codeunit.al

Added US to Yavrio Open Banking
Fixes AB#543398

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 22667.


Identify all Repositories in the RKE2 Sphere using Drone

https://github.com/rancher/ecm-distro-tools/blob/master/docs/rke2-dependencies.md


Add opnsense

This is my first ever pull request on Github! There was a bit to learn, but I wanted to contribute to the community. If there are any questions or if my readme's are off, please let me know. I read the contributing.md doc at the end, but I think I got everything covered :-)

Thanks!


cant import

Collecting CapBypassWrapped
Using cached capbypasswrapped-1.1-py3-none-any.whl (4.5 kB)
Requirement already satisfied: requests in c:\python\lib\site-packages (from CapBypassWrapped) (2.27.1)
Collecting logging (from CapBypassWrapped)
Using cached logging-<IP_ADDRESS>.tar.gz (96 kB)
Preparing metadata (setup.py) ... error
error: subprocess-exited-with-error
× python setup.py egg_info did not run successfully.
│ exit code: 1
╰─> [19 lines of output]
Traceback (most recent call last):
File "", line 2, in
File "", line 14, in
File "C:\Python\lib\site-packages\setuptools_init_.py", line 16, in
import setuptools.version
File "C:\Python\lib\site-packages\setuptools\version.py", line 1, in
import pkg_resources
File "C:\Python\lib\site-packages\pkg_resources_init_.py", line 83, in
import('pkg_resources.extern.packaging.specifiers')
File "C:\Python\lib\site-packages\pkg_resources_vendor\packaging\specifiers.py", line 24, in
from .utils import canonicalize_version
File "C:\Python\lib\site-packages\pkg_resources_vendor\packaging\utils.py", line 8, in
from .tags import Tag, parse_tag
File "C:\Python\lib\site-packages\pkg_resources_vendor\packaging\tags.py", line 5, in
import logging
File "C:\Users\ps390\AppData\Local\Temp\pip-install-wkx9dw44\logging_0320bda7d24b49afab6d8ce180770027\logging_init_.py", line 618
raise NotImplementedError, 'emit must be implemented '
^
SyntaxError: invalid syntax
[end of output]
note: This error originates from a subprocess, and is likely not a problem with pip.
error: metadata-generation-failed
× Encountered error while generating package metadata.
╰─> See above for output.

Fixed issue with importing logging as a third party package was accidently being installed with pip via setup.py - c2bc945684532de18eb56925802bfc5da12d0f14


Import module based on path relative to project root (tsconfig.json).

Counting ../../../ is painful, it would be great if TypeScript compiler could convert something like ~/modules/my-module for us based on project root and file path.

:+1:

this should be handled in #2338, by setting a Path mapping for "~*" : "........\Root*"

@mhegazy and it's possible? Because this issue #2338 is closed, and I did not find any solution there.

#2338 gives you a way to tell the compiler where the files are, the compiler is not going to rewrite ~/ for you to be ../.././../. see https://github.com/Microsoft/TypeScript/issues/15479 for more details.


add terraform.tfvars explicitly

Reasons for making this change:
terraform.tfvars is a file which is used by convention to populate variables in a Terraform build.  As terraform.tfvars does not point to any specific environment, but is global this is a common place to put persistent secrets that should never be committed.  I agree with the logic of removing the global *.tfvars file for the multi-environment scenario, but this makes sense to still ignore as it's auto-loaded by Terraform at a global level, and as such might be a good place to store AWS/Azure credentials without manually setting the environment variables which could have its own set of repercussions.
Links to documentation supporting these rule changes:
https://www.terraform.io/intro/getting-started/variables.html

Does this make sense @jkodroff @apparentlymart?

This terraform.tfvars file is honestly just auto-loaded supported for backward compatibility since the new *.auto.tfvars convention allows for multiple auto-loaded variables files that allow separating different sorts of variables and giving the files meaningful names. This allows mixing both auto-loaded variables files that are in version control with ones that aren't.
We've generally recommended putting credentials outside of the configuration directory altogether (whether that be in environment variables or in some system-level file like ~/.aws/credentials) since that avoids the need for any special .gitignore rules to prevent those credentials getting committed.
With all of that said, whether this file is ignored or not depends on local policy, and ignoring it with the goal of putting sensitive values in it would be contrary to usual best-practices, so if any change is to be made here I'd prefer to have a comment that draws attention to the possibility of *.auto.tfvars files, similar to the one already present, and leave it to the user to decide which files make sense to ignore. For many users, no files are ignored because these variable values are considered to be part of the configuration.

That's fair, but I don't believe the same mechanism of the ~/.aws/credentials file exists for Azure that I've found.  It also starts to break when you have Azure Storage Accounts with primary and secondary access keys.   They can be retrieved from the state after creation, but writing them in plain text to an environment variable in your ~/.bashrc isn't secure nor scalable (consultant on multiple projects), neither is setting it using export in the event that you close your terminal.  Either of the two tfvars routes are the way to go IMO, because they can be explicitly ignored thereby making things like creating an Azure remote state backend in Terraform to create and save your state backend dynamically is made much easier, and the security is mitigated by the inability for the dev to check that in unless they're really trying.
That said, it's not the end of the world to have to add this but it does mean that I have to change my workflow to include ignoring those files.

We support passing credentials to providers in their configuration blocks as a last resort for weird scenarios, but we don't recommend it as the standard way to use Terraform. For Azure in particular, the provider team's recommendation is to use the Azure CLI's own session management for local work and to create a service principal when running Terraform in automation.
I understand that this is not convenient when you regularly work with many distinct sets of credentials (for different customers) and you want to scope these to each project. In such cases I've usually seen people solve that by helpers to set environment variables as they switch between directories in their shells, or other similar strategies.
The general recommendation of using the standard (i.e. not-Terraform-specific) mechanisms to pass credentials is partially motivated by ensuring that you can set up credentials once and use them across both Terraform and other tools, and so switching environment variables as you move between projects (whether manually or via helper scripts) will configure all of the software that needs those credentials at once.
Using terraform.tfvars to set some variables is certainly another possible way to get there with some different tradeoffs, but I'd consider that a local policy decision on your part rather than a general Terraform best-practice that I'd expect everyone to follow, and thus not something I'd expect to see in a standard .gitignore template.


Error with react-fontawesome

Hello!  Unsure if this goes in miniflare or react-fontawesome, but since react-fontawesome works with version 2.0.0 of miniflare and fails with version 2.1.0, I am starting here.
Essentially, I have a remix project where I try adding a font awesome component to it.  Immediately, it fails on load with this error:
TypeError: Cannot convert undefined or null to object
    at Function.getPrototypeOf (<anonymous>)
    at ordinaryHasInstance (/Users/bduffey2/projects/test-remix/node_modules/@miniflare/runner-vm/src/instanceof.ts:99:22)
    at instanceOf (/Users/bduffey2/projects/test-remix/node_modules/@miniflare/runner-vm/src/instanceof.ts:114:5)
    at Function.value (/Users/bduffey2/projects/test-remix/node_modules/@miniflare/runner-vm/src/instanceof.ts:139:14)
    at checkPropTypes (/Users/bduffey2/projects/test-remix/node_modules/react/cjs/react.development.js:1929:13)
    at validatePropTypes (/Users/bduffey2/projects/test-remix/node_modules/react/cjs/react.development.js:2136:7)
    at Object.createElementWithValidation (/Users/bduffey2/projects/test-remix/node_modules/react/cjs/react.development.js:2240:5)
    at Index (/Users/bduffey2/projects/test-remix/build/route-module:/Users/bduffey2/projects/test-remix/app/routes/index.tsx:7:11)
    at processChild (/Users/bduffey2/projects/test-remix/node_modules/react-dom/cjs/react-dom-server.browser.development.js:3352:14)
    at resolve2 (/Users/bduffey2/projects/test-remix/node_modules/react-dom/cjs/react-dom-server.browser.development.js:3269:5)

If I downgrade to miniflare 2.0.0, the error goes away and the icon shows as expected.  To reproduce, you can create a new remix project with npx create-remix@latest.  Then, upgrade miniflare to 2.1.0 and add the font awesome deps to your package.json with npm i --save @fortawesome/fontawesome-svg-core @fortawesome/free-regular-svg-icons @fortawesome/react-fontawesome.  Finally, load any icon on the index route (such as <FontAwesomeIcon icon={faCheckCircle} />) and open the app in a browser.  Let me know if I can provide more information.  Thanks!

Same here with 2.1.0
[mf:err] GET /: TypeError: Cannot convert undefined or null to object
The error goes away if I downgrade to 2.0.0


Make background image pan

Hello,
At the moment if I add a background to my react flow background like so:
<Background
          color='#fff'
          gap={16}
          style = { { backgroundImage: 'url(myImage.png)'}}
/>

The background stays fixed while i pan the canvas. Is it possible to make the background image move with the panning of the canvas?
Example image bellow:
Before panning:

After panning canvas to the right:

What exactly are you trying to do here? What is your use case? There might be another way rather than adding an image to the background. The background it self doesn't move but we calculate the positions of the dots. That's why your image doesn't move here.

I want to allow my users to upload an image that sits on the canvas below the grid dots and graph nodes so that each graph nodes can be visualized in relation to the background image. For example a car as a background image and the graph nodes to represent the different components of the car by being positioned where those components would be in the image (engine, wheel, exhaust etc). Thus I need my users to be able to pan and zoom and the image and graph nodes and grid dots to stay in the same position relative to each other. Is this possible?

You could add an inactive node (draggable:false,selectable:false,connectable:false,maybe some extra css with pointer-events:none) that only has an image as content without handles.

thanks for the suggestion, is it possible to make that node sit behind the grid dots?

No this is not possible. All nodes would be behind the background if you change the z-indices.


Bugs on the Transfer screen when change account, change token

Describe the bug
Bugs on the Transfer screen when change account, change token
To reproduce the behavior
Version test: v1.0.2 (145)
Steps:

Go to the Transfer screen
Select account, token, receive account
Input invalid Amount to show error message
Perform:

Change to valid token with Amount
Change to valid account with Amount

Actual:

In case change token: Some token verify Amount is incorrect (ex: change from DOT to NEER, NUUM)
In case change account: Still show error message although valid data

https://user-images.githubusercontent.com/130966515/238196939-fcf00550-1960-403c-a3e8-545dde200f14.mp4
Expect: Validate correctly

Version test: v1.0.2(149)
Devices test: IP 11
Validation passed.


Use Bach.java 1.0.0

Prior to this commit, the build script depended on "current master" version of Bach.java -- which is now using Java 10 syntax. This commit ensures that a Java 9 compatible version of Bach.java is
loaded from Github.

You tested this, right? So I'll just go ahead and merge. :smiley:

Rémi tested that, without knowing ... https://github.com/forax/beautiful_logger/blob/master/build.jsh#L7


Go test case for the SRT receiver component

Hi,
We are working on using the gosrt component for receiving SRT packets.We would like to write test cases for the receiver component.
Can you please guide us in writing unit test cases ? An example unit test for the sample example will be really helpful.

Isn't this enough?
https://github.com/openfresh/gosrt/blob/master/examples/livetransmit/main.go
Could you tell me more detail about the unit test cases?


data-* props

It'd be nice to support data-* props; probably just by passing any "extra" props found to the root elements for each component.
In case anyone is interested or wants let me know this is crazy (I don't know), specifically I'm using a data attribute to avoid having to allocate a new function for every element of a list for an onClick handler, or having to fudge around with memoisation. I suspect this is a lighter-weight approach though I haven't done any measurements to show which is best. Then again, this needs DOM access, which JS folklore has lead me to believe is slow...
The code looks like
...
onClickItem(ev) {
  foo(ev.target.dataset.thing);
}
render() {
  return this.props.something.map(a => <ListItem key={a.key} onClick={this.onClickItem} data-thing={a.thing} caption={a.caption} />)
}

Support for passing unknown props might be worth considering, there are both up and downsides with that. For your specific use case though I would recommend allocating a new function every time though. I have very rarely and only in extreme special cases had that give any noticeable performance effect. The simpler code wins here and the time is probably better spent optimizing something else :smiley:


Site Search is not working.

https://www.boonex.com/forums/topic/UNA-search-bugged.htm

Please visit https://manofteal.com and try to use the site search.


chore: bdd-nh-1586610371 to 0.0.1

chore: Promote bdd-nh-1586610371 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described 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


Updating maintenance logs does not update maintenanceType field

What were you trying to achieve?
How to replicate the issue
Hit the update log endpont PUT http://<IP_ADDRESS>:31002/api/v1/devices/ts/activity/update?id=604c88fb3fc26c002bea5d7d&tenant=airqo
sample data
{
    "description": "new description",
    "maintenanceType": "custom new"
}

What are the expected results?
updated maintenance log (including the maintenanceType field but it's not updated)
What are the received results?

I guess this was resolved accordingly, please see attached screenshot.


Spec difficult to understand surrounding mount namespace

From @eparis on December 1, 2014 20:26
https://github.com/coreos/rocket/blob/master/app-container/SPEC.md#container-runtime-manifest
States that all apps in a container share the same mount namespace. But then goes on to talk about how volumes are mounted into specific apps.
Apps sharing the mount namespace seems really hard to get right, so I'm assuming that different apps in the same container have different mount namespaces?
Copied from original issue: coreos/rocket#127

@eparis @lgomez do either of you have any outstanding questions here?

Closing due to inactivity, please file another issue if you have any questions about this


Enhancement to DatabaseBackup.sql

Added functionality to create multiple backup files based upon the size of the database and a given size in MB.
New Parameter @DatabaseSizeSplit
Addresses feature request #227

Please let me know how I can improve upon the process.  I'm still a Github n00b.

@DatabaseSizeSplit variable name could be made more clear by renaming it to @DatabaseSizeSplitInMB

@chadwestes I too am looking for this change, looks like this PR has been out there for a while. Any chance of getting it updated and re-submitted?
My suggestion is to rename @DatabaseSizeSplit to @BackupStripeSizeInMB

Seems to be implemented now (@MinBackupSizeForMultipleFiles / @MaxFileSize parameters)


Fix an edge case around calculating resourcetype from resourceid

Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

API change check
API changes are not detected in this pull request.

/check-enforcer reset

/azp run net - core - ci


align the social media icon to center of footer

Description
align the social media icon
Mocks

Definition of Done

[ ] All of the required items are completed.
[ ] Approval by 1 mentor.

assign it to me.


Release the GIL

I might be missing something, but does llama-cpp-python release the Python GIL at the moment?
If it doesn't, would releasing the GIL allow multi-threaded execution of GGUF files?

?????????????
It's llama.cpp that handle the multithreading, and if it's running on GPU, multithreading is useless

These are language bindings for llama.cpp, the actual gguf processing is not done in python. The GIL is released when the underlying llama cpp execution is happening. Use n_threads when instantiating a model to set parallelism of llama.cpp.

@simonw yes llama-cpp-python uses ctypes.CFUNCTYPE to biind to the llama.cpp functions and as stated in the docs there.

The returned function prototype creates functions that use the standard C calling convention. The function will release the GIL during the call.


Chore: Adds Swagger @Tag annotations to API controllers

What this PR changes/adds
This PR adds the @Tag annotation to the REST API Controllers to enable better rendering in HTML
Why it does that
To improve readability.
Further notes
Checklist

[ ] added appropriate tests?
[ ] performed checkstyle check locally?
[ ] added/updated copyright headers?
[ ] documented public classes/methods?
[ ] added/updated relevant documentation?
[ ] added relevant details to the changelog? (skip with label no-changelog)
[x] formatted title correctly? (take a look at the CONTRIBUTING and styleguide for details)

Codecov Report

Merging #1110 (fd228d5) into main (ad9b80a) will increase coverage by 0.03%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##               main    #1110      +/-   ##
============================================
+ Coverage     62.57%   62.61%   +0.03%
- Complexity     2955     2969      +14
============================================
  Files           692      694       +2
  Lines         15362    15416      +54
  Branches       1041     1044       +3
============================================
+ Hits           9613     9652      +39
- Misses         5320     5334      +14
- Partials        429      430       +1

Impacted Files
Coverage Δ

...e/selector/api/DataplaneSelectorApiController.java
100.00% <ø> (ø)

...main/contract/negotiation/ContractNegotiation.java
0.00% <0.00%> (ø)

...nnector/api/auth/BasicAuthenticationExtension.java
30.00% <0.00%> (ø)

...connector/api/auth/BasicAuthenticationService.java
97.05% <0.00%> (ø)

Continue to review full report at Codecov.

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


Colab can't start web

/content/Retrieval-based-Voice-Conversion-WebUI
Use Language: en_US
2023-06-16 16:13:15.539160: I tensorflow/core/util/port.cc:110] oneDNN custom operations are on. You may see slightly different numerical results due to floating-point round-off errors from different computation orders. To turn them off, set the environment variable TF_ENABLE_ONEDNN_OPTS=0.
2023-06-16 16:13:15.592641: I tensorflow/core/platform/cpu_feature_guard.cc:182] This TensorFlow binary is optimized to use available CPU instructions in performance-critical operations.
To enable the following instructions: AVX2 AVX512F AVX512_VNNI FMA, in other operations, rebuild TensorFlow with the appropriate compiler flags.
2023-06-16 16:13:16.580038: W tensorflow/compiler/tf2tensorrt/utils/py_utils.cc:38] TF-TRT Warning: Could not find TensorRT
2023-06-16 16:13:18 | INFO | fairseq.tasks.text_to_speech | Please install tensorboardX: pip install tensorboardX
Traceback (most recent call last):
File "/content/Retrieval-based-Voice-Conversion-WebUI/infer-web.py", line 87, in
class ToolButton(gr.Button, gr.components.FormComponent):
AttributeError: module 'gradio.components' has no attribute 'FormComponent'. Did you mean: 'IOComponent'?

Same issue

Same issue

Same here

open infer-web.py and change
class ToolButton(gr.Button, gr.components.FormComponent):
to
class ToolButton(gr.Button, gr.components.IOComponent):

I talked about it on Discord,
You can change
!pip3 install faiss-gpu fairseq gradio ffmpeg ffmpeg-python praat-parselmouth pyworld numpy==1.23.5 numba==0.56.4 librosa==0.9.2
to
!pip3 install faiss-gpu fairseq gradio==3.35.1 ffmpeg ffmpeg-python praat-parselmouth pyworld numpy==1.23.5 numba==0.56.4 librosa==0.9.2

open infer-web.py and change
class ToolButton(gr.Button, gr.components.FormComponent):
to
class ToolButton(gr.Button, gr.components.IOComponent):

That right.
or use  gr.components.base.FormComponent
This issue will fix on #543

thx

I talked about it on Discord, You can change !pip3 install faiss-gpu fairseq gradio ffmpeg ffmpeg-python praat-parselmouth pyworld numpy==1.23.5 numba==0.56.4 librosa==0.9.2 to !pip3 install faiss-gpu fairseq gradio==3.35.1 ffmpeg ffmpeg-python praat-parselmouth pyworld numpy==1.23.5 numba==0.56.4 librosa==0.9.2

open infer-web.py and change
class ToolButton(gr.Button, gr.components.FormComponent):
to
class ToolButton(gr.Button, gr.components.IOComponent):

That right. or use gr.components.base.FormComponent
This issue will fix on #543

excuse me, change after or before run it?  I still can't find a fix..

Can confirm, downgrading to 3.4 version fixed the issue for me and web-URL launched with no issue.  thanks!

excuse me, change after or before run it? I still can't find a fix..

change before run!

I wish that this could work for me, but I am still getting the following error message. Any help will be greatly appreciated:


It seems Harbor CR does not update

Reproduce steps:
install harbor core components with make sample, harbor "sample" is deployed
install full harbor with make sample-harbor-full, no more components are deployed.
expectations: the optional components like notary, chartmuseum and trivy should be deployed.

there's still an error when install harbor-full

notary and trivy components are using github token which is not correctly injected.


Adding Disney Plus service

Also remember to update the read e-file with the new service.

Please run the linter and fix the errors (npm run fix).

src/streaming-services/disneyplus/DisneyplusParser.ts:62:36 - error TS2339: Property 'style' does not exist on type 'Element'.
It does exist though.
As said earlier, new to typescript and not sure how to handle this one.
It seems that subitems from querySelectorAll don't handle the "style" option, though querySelector does have it (and returns the same object)
Any idea how to fix / ignore this?

Please run the linter and fix the errors (npm run fix).

src/streaming-services/disneyplus/DisneyplusParser.ts:62:36 - error TS2339: Property 'style' does not exist on type 'Element'.
It does exist though.
As said earlier, new to typescript and not sure how to handle this one.
It seems that subitems from querySelectorAll don't handle the "style" option, though querySelector does have it (and returns the same object)
Any idea how to fix / ignore this?

querySelectorAll returns NodeListOf<Element>, and style is only present in HTMLElement, so all you'd have to do is cast the type: document.querySelectorAll(...) as NodeListOf<HTMLElement>


This fixes issues with accidental reloads, mostly on mobile clients.

Looks good to me.

:shipit: :rocket:


#284 changeable pagination icons added

problem fixed by adding props to b-table and b-pagination
default values didn't changed
#284
tnx

@moeenzarrinfam I think it's a good idea. Please resolve conflicting files.

@jtommy resolved

@moeenzarrinfam I'm looking your commit and the problem is about 'both' prop to b-icon component. This prop is used to internal component to manage different pack names with the same icon name.
Your commit allow to use all 'mdi' icons only and the current equivalent icon (see Icon component source)

@jtommy oops!... I didn't know that
last commit contains my idea to solve the problem .
please check it.
tnx

@moeenzarrinfam I think that it's only a workaround because now there are 'mdi' and 'fa' icon pack but in the future?
You should enable/disable both prop if the current icon is equals to default or not, or another solution but it have to general.

@moeenzarrinfam We thought about this PR and at the end most of the components (panel, table, datepicker, ..) using icons don't allow to change these. So we prefer follow this way, at the moment.
Anyway let me thank you for the support and the spent time :thumbsup:


LICENSE?

Can I please know under what LICENSE is the code distributed? I really prefer it is either MIT or CC0 but it would be better if it is mentioned

Hey there, it's under MIT, I just added the LICENSE file. Enjoy!

Wow! Thanks


Open existing Spec.out file in "TLA+ model checking" panel

Please add support to open an existing Spec.out file from a previous TLC run in the "TLA+ model checking" panel.
Background: Toying with VSCode Online where two users collaborate on a single spec. One runs TLC the other one wants to look at the error trace in the the shared Spec.out file.  Collaboration works at the spec level but the  "TLA+ model checking" panel is unfortunately not a first-class entity.

It's already implemented. You can open any .out file and run the "Visualize TLC output" command. It works if the -tool option was used when running in TLC.
I believe, it relates to this discussion, and the current implementation should do the job.

I'm closing this one, since the feature is already supported. Feel free to reopen the issue if you meant something else, or file a bug if there're problems with visualising of .out files.


Temporary revert of map refactor

I think it might have caused a performance and/or freeze regression when mapping over large inputs that require blob uploads. Reverting the relevant patch(es) until I can properly debug this.
Side note: it turns out the client doesn't have any unit tests for e2e function calls with large inputs, and adding a trivial one freezes the test runner for me regardless of this patch... Need more time to investigate that and add tests

@prbot approve

👍 thanks for catching it!


update dependencies and deprecated functions

Hi, when I write my project depending on both flurry and crossbeam-epoch, I meet this error.
error[E0308]: mismatched types
   --> src/store/flurry_store.rs:348:26
    |
348 |         map.insert(1, 1, &g);
    |                          ^^ expected struct `flurry::epoch::Guard`, found struct `crossbeam_epoch::Guard`
    |
    = note: expected reference `&flurry::epoch::Guard`
               found reference `&crossbeam_epoch::Guard`
    = note: perhaps two different versions of crate `crossbeam_epoch` are being used?

I found some dependencies in this crate that are out-of-date, so I update the dependencies and deprecated functions.

This change is

Hi, @jonhoo. I have no idea why the tarpaulin test failed, could you give a review? Thanks!

Hi, jonhoo. Thanks for your suggestions, all suggestions have been applied now.


Rewrite all urls in a playlist

I again, I need to use threadfin via vpn on another server in another lan, when I connect with the vpn ip to my server I get the m3u and xmltv, but when I stream it tries to connect at the other threadfin as it is in the same lan, I need a way to rewrite the ips in the second lan to match the vpn ip of the first lan, I tried to modify the urls manually to test but threadfin overwrites
Could you add a field to do this?

You need to set the domain option I guess. That will overwrite the IPs within the m3u and xml file.

Yes it does, but only in egress, not in ingress

Maybe you need a proxy?

Maybe that will help you

I tried to use it but it doesn't change, probably I don't know how to use it

It should forward every stream to a specific endpoint. I‘ll have to check how it works, before I can say anything specific

These are the comments to the code that has been checked into the original project:
https://github.com/Threadfin/Threadfin/pull/325
So this is something with the buffer.
To be honest I don‘t think that Threadfin should touch the urls in the original playlist as this could lead to malformed URL’s. Even the force https option is a bad option in Threadfin and I just left it since it has been used eventually by few users. Therefore I will not implement this feature. Sorry for the inconvenience.


How do I enable CSR signing

What kind of request is this (question/bug/enhancement/feature request):
I created a custom (RKE)k8s cluster through rancher UI. How do I enable CSR signing
Steps to reproduce (least amount of steps as possible):
How do I enable CSR signing
Result:
Other details that may be helpful:
Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI):
Installation option (single install/HA):

Cluster information

Cluster type (Hosted/Infrastructure Provider/Custom/Imported):
Machine type (cloud/VM/metal) and specifications (CPU/memory):
Kubernetes version (use kubectl version):

(paste the output here)

Docker version (use docker version):

(paste the output here)

did you find a solution for this, I'm having the same issue. I can create the CSR but seems the kube controller manager is not configured to use a CA wondering how to get this working with rancher built clusters.

Edit Cluster.
Under Cluster Options select Edit as YAML
and change kube-controller: {}
TO
kube-controller:
extra_args:
cluster-signing-cert-file: /etc/kubernetes/ssl/kube-ca.pem
cluster-signing-key-file: /etc/kubernetes/ssl/kube-ca-key.pem


FR: Zipped release for manual install

Is Your Feature Request Related to a Problem? Please Describe.
In order to install the plugin manually, you have to download the three files individually, then create a folder and rename it with the plugin id and put these files in it.
Describe the Solution You'd Like
I know these files are needed for automatic installation to work, but it would be much easier for the user to be able to download a zipped file also from the releases tab in the following configuration:
plugin-id-{v}.zip
    plugin-id\
        main.js
        manifest.json
        styles.css

This way we can just extract the zip file in the plugin folder. Thank you.

Hey @1ku1 , could you explain the reason for why you would install the plugin manually this way as opposed to automatic installation or BRAT installation?
Also, could you point to a plugin that currently does this? I want to make sure this will not cause problems with Obsidian.

Hey @1ku1 , could you explain the reason for why you would install the plugin manually this way as opposed to automatic installation or BRAT installation?

Firewall policy.

Also, could you point to a plugin that currently does this? I want to make sure this will not cause problems with Obsidian.

https://github.com/javalent/settings-search
https://github.com/AlexBieg/obsidian-better-command-palette
Thanks

Thanks for those examples @1ku1 . I am looking at adding that workflow for the next release. I will see how well it works with a test run and hopefully then it can be in place for the next release.

Test looks to have passed for me. The changes are now on master and should start running with the next release. Please let us know if this has some kind of issue.

Thanks


merge resolver and janitor init to one API

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@2175564). Click here to learn what that means.
The diff coverage is 0%.

@@            Coverage Diff            @@
##             master     #425   +/-   ##
=========================================
  Coverage          ?   43.29%
=========================================
  Files             ?       22
  Lines             ?    11067
  Branches          ?        0
=========================================
  Hits              ?     4791
  Misses            ?     5064
  Partials          ?     1212

Continue to review full report at Codecov.

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

LGTM


What is this for?

Hosting our images

OK.

make something.

@redmangospros  How?

Css Theme,etc

OK. I just made my first SVG image though! idk.svg

... :D Make themes

OK.


[ML] Fixing bugs related with workspace resource operations.

Description

Fixing the issue where Diagonse API response can't be print into json string which is against what we have in our doc.
Remove constrains on Project type workspace creation to allow user set PNA flag and ManagedNetworkSettings to avoid policy check fail.
Fix the issue in get_keys call where the older version of rest_client is used and result in schema missmatch.

If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[x] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

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


E/Filament: Panic

in filament::MaterialParser *filament::createParser(filament::backend::Backend, const void *, size_t):69
reason: Material version mismatch. Expected 17 but received 20.

E/Filament:

Do you use this repository directly or a version from MavenCentral? This error is related to materials which are compiled for a different Version of Filament then used in Sceneform.


Remove charconv include

No functions like to_chars and from_chars are used in code.

Can one of the admins verify this patch?

for #2018

@jenkins-droid test this please

Yes, I accept the DCO

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

@jenkins-droid test this please

Jenkins Linux amd64 Build #9685 [push] Remove charconv include ... started at 12:43

Jenkins Linux s390x Build #9704 [push] Remove charconv include ... started at 13:43

Jenkins Linux ppc64le Build #8750 [push] Remove charconv include ... started at 13:47

Jenkins Linux amd64 Build #9685 [push] Remove charconv include ... passed after 1 hr 45 min

Jenkins Linux ppc64le Build #8750 [push] Remove charconv include ... aborted after 2 hr 12 min

Jenkins Linux s390x Build #9704 [push] Remove charconv include ... aborted after 2 hr 13 min


Use rlibc

cc @NikVolf

On the second thought, how about making it feature-gated?

What do you mean exactly?

I'd like to add feature so that if I compile with pwasm-std/ext_memory (whatever name), the older variant with ext_memcpy/ext_memmov/ext_memset externs is compiled
I think we need it pretty soon

I think we need it pretty soon

Could you elaborate it a little more?

Optimization of contracts run via interpreter? I think using this kind of operations will be very slow with purely wasm instruction set


Fix degree to radian in latitude and latitude_optimal in orientation module

The current implementation does not convert degrees to radian in the make_latitude and make_latitude_optimal function. This fixes it.
Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist

[x] I tested my contribution locally and it seems to work fine.
[x] I locally ran pytest inside the repository and no unexpected problems came up.
[x] I have added a note to release notes doc/release_notes.rst.

merged into #217

Upps. Good catch!


Minor: Improve code readability

This change includes some basic formatting fixes (spaces around operators, etc.), which wasn't consistent in the original version.
I also added a space that was missing in an error message.

+1

Merged. Thanks!


temp revert script for bash

similar to #74

Not sure who has context on either revert PR to approve, is there more info?


Log statements don't respect logging option from TypeORM

This bug was added in this commit:
https://github.com/odavid/typeorm-transactional-cls-hooked/commit/ee15911fa9945363812d7f5f5428789439e921cd#
console.log(`Transactional@initializeTransactionalContext\`)
console.log(`Transactional@patchRepositoryManager repositoryType: ${repositoryType?.constructor?.name}`)
These console.log statements don't respect the logging option from TypeORM. This results in log statements even when you have logging set to false.

Thank you for fixing this so quickly!


Re-enable VB.NET tests once xbuild is fixed

From @slide on September 26, 2016 21:10
xbuild (14.0/mono <IP_ADDRESS>) currently can't be used to build VB.NET applications.
The following tests need to be re-enabled when the PR for fixing xbuild is in a release version of mono
interop/net/type/test_defaultmember.py
interop/net/property/test_indexervb.py
interop/net/derivation/test_property_override.py (test_vbindexer)
Copied from original issue: IronLanguages/main#1438

msbuild is now being used, however the Roslyn VB.NET compiler is not packaged by mono yet.

See https://github.com/IronLanguages/dlr/pull/107

See https://github.com/IronLanguages/ironpython2/pull/345

Fixed via #345


Fonts : VictorMono & FiraCode incorrect & overlap italic style.

Windows Terminal version
1.11.3471.0
Windows build number
10.0.19044.1415
Other Software
Terminal itself , Vi, Vim & Neovim.
VictorMono : Terminal display incorrect italic VictorMono fonts  in and outside text editor.

Font size : 14 (all)
Font Weight : normal or medium

FiraCode/FuraCode : same as above.

Font size : 14 (all)
Font Weight : maybe all.

Actually it already happened before the latest release .
Steps to reproduce
VictorMono

FiraCode/FuraCode

Expected Behavior
No response
Actual Behavior
As you expect ...

This is essentially the same bug as #9381. If the italic characters don't fit within a cell, they're likely to get truncated on rendering boundaries where the color or style needs to change.

Thanks! /dup #9381


Support theme localization and ParamArray in createMuiTheme

https://material-ui.com/guides/localization/
https://material-ui.com/customization/theming/#createmuitheme-options-args-theme
There might be some feasible quick-and-dirty changes to get this in now, but unless anyone pushes me, I'll probably wait until a new theme DSL is implemented, see #2.

Released in 0.13.1.


Merges Master to Live. See commits for details.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/advanced-threat-protection/Get-AdvancedThreatProtectionDocumentDetail.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/advanced-threat-protection/Get-AdvancedThreatProtectionDocumentReport.md
:white_check_mark:Succeeded
View (exchange-ps)

sharepoint/sharepoint-ps/sharepoint-online/Get-SPOSiteDesignRun.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-online/Get-SPOSiteDesignRunStatus.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-server/Add-DatabaseToAvailabilityGroup.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-server/Add-FASTSearchResource.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-server/Add-FASTSearchSpelltuning.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-server/Add-SPAppDeniedEndpoint.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-server/Add-SPClaimTypeMapping.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-server/Add-SPDiagnosticsPerformanceCounter.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-server/Add-SPDistributedCacheServiceInstance.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-server/Add-SPEduClassMember.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-server/Add-SPEduUser.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-server/Add-SPInfoPathUserAgent.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-server/Add-SPPluggableSecurityTrimmer.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-server/Add-SPProfileLeader.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-server/Add-SPProfileSyncConnection.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-server/Add-SPRoutingMachineInfo.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-server/Add-SPRoutingMachinePool.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-server/Add-SPRoutingRule.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-server/Add-SPScaleOutDatabase.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-server/Add-SPSecureStoreSystemAccount.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-server/Add-SPServerScaleOutDatabase.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-server/Add-SPServiceApplicationProxyGroupMember.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-server/Add-SPShellAdmin.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-server/Add-SPSiteSubscriptionFeaturePackMember.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-server/Add-SPSiteSubscriptionProfileConfig.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-server/Add-SPSolution.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-server/Add-SPThrottlingRule.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-server/Add-SPUserLicenseMapping.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-server/Add-SPUserSolution.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-server/Backup-SPConfigurationDatabase.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-server/Backup-SPEnterpriseSearchServiceApplicationIndex.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-server/Backup-SPFarm.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-server/Backup-SPSite.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-server/Clear-FASTSearchContentCollection.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-server/Clear-SPAppDeniedEndpointList.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-server/Clear-SPBusinessDataCatalogEntityNotificationWeb.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-server/Clear-SPContentDatabaseSyncData.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-server/Clear-SPDistributedCacheItem.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-server/Clear-SPLogLevel.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-server/Clear-SPMetadataWebServicePartitionData.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-server/Clear-SPPerformancePointServiceApplicationTrustedLocation.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-server/Clear-SPScaleOutDatabaseDeletedDataSubRange.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-server/Clear-SPScaleOutDatabaseLog.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-server/Clear-SPScaleOutDatabaseTenantData.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-server/Clear-SPSecureStoreCredentialMapping.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-server/Clear-SPSecureStoreDefaultProvider.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-server/Clear-SPServerScaleOutDatabaseDeletedDataSubRange.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-server/Clear-SPServerScaleOutDatabaseLog.md
:white_check_mark:Succeeded
View (sharepoint-ps)

This comment lists only the first 50 files in the pull request.
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Flaky LUCI scheduling of presubmit jobs can make a PR green

On flutter/cocoon there are several times where jobs are not scheduled for a PR. In my experience, this seems to be about ~10% of the time. This can make the PR green and allowed to be submitted, when it has had 0 tests run against it.
It would be good to investigate the scope of this flakiness and see if other repos are impacted by it. Additionally, seeing if it's flakiness in Cocoon scheduling the job, or LUCI scheduling the job.

For example, https://github.com/flutter/cocoon/pull/707 never got a job scheduled so it shows as passing. CLA is signed and there's no WIP. No tests were run against the PR.

/cc @godofredoc


Add strikethrough when hovering over added files (data processing)

Add a strikethrough and "clickable" cursor when hovering over an element to show it can be removed upon clicking.

Might want to make a tag for issues relating to processing. Do you have the permissions to do that? I'll do it if not.

I don't have the permissions for that.

Made a tag and added it to all the open issues regarding the processing system.


[11.x] Add an emitAs method to Events

I'm contributing this as an idea to allow object events to be a bit more flexible.
By using an emitAs method, this can override the default usage of the class as the event name, and allow the data that's been passed to the object determine the event name that should be emitted.
For example, if you had a CommunicationSentEvent and you pass it aCommunication model containing a type attribute, using the emitAs method you can override the event name to be communication.{type} rather than the class name.
The new emitAs method is purely optional, and if a method is not present on the object, then it will fall back to the class name as default.
If this PR is merged into the 11.x branch, I'll create documentation supporting this new method

I would probably just use Event::dispatch('communication.'.$type', ...)

@taylorotwell - In my opinion, I think this is a missing feature of Laravel.
The problem with stringed events is that can be tightly coupled. If I created a stringed event with an array payload, I would need to keep referring back to the listener to find out what I need to send in the payload, what I called the event, etc... and there is very minimal IDE type-hinting.
use Illuminate\Support\Facades\Event;

Event::dispatch("communication.{$type}", [
    "communication" => $communucation
]);

By merging the two, it can allow code to be Loosely Coupled. It brings the elegance of using an object to package the data up with type hinting, while allowing the flexibility the event to be dispatched as a string. I’d also argue allows code to look and be more elegant and easier to follow:
use Illuminate\Support\Facades\Event;

class CommunicationEvent {
    public __construct(public Communication $communication) {}

	public function emitAs(): string
	{
		return "communication.{$this->type}";
	}
}

Event::dispatch(new CommunicationEvent($communication));
// or event(new CommunicationEvent($communication));

This also mirrors the broadcastAs behaviour when broadcasting events to the frontend allowing you to customise how events should be emitted via Laravel Echo, so why not add the same behaviour to the event system for Laravel.


[Mosco] Option for checking whether there is a valid provisioning profile

When a project is switched to iOS as a target, coordinate with Mosco to check whether there is a matching provisioning profile, and that it is valid (e.g. not expired)

Codex 1.6 beta has an option for warning when a certificate has expired or is about to expire - Requires Mosco 1.4 Beta 2, which is here.


Weapon selection text misaligned on 1440p

When using 2560x1440 as my resolution, weapon selection text has inconsistent height. The effect seems to vary by font size and boldness but is always noticeable. Switching to 1920x1080 on the same setup makes the issue go away.
Does not appear to be an issue on any other text.
1440p, Verdana

1440p, Arial

1440p, Cogan Light

1080p, Verdana

1440p, Scheme editor is unaffected

Fixed in 9580da0.


Allow generators into Model's fit, evaluate and predict

Summary
I've ported the interesting sections of the code implemented in tensorflow/keras/engine/training.py#L749
and some necessary utils. Generators and Sequences can now be passed to Model#fit, Model#evaluate and Model#predict.
The parameters max_queue_size, workers, use_multiprocessing were added to these methods (they now have the exact same signature as the ones in tf.keras.
Related Issues
#11772
PR Overview

[y] This PR requires new unit tests [y/n] (make sure tests are included)
[n] This PR requires to update the documentation [y/n] (make sure the docs are up-to-date)
[y] This PR is backwards compatible [y/n]
[n] This PR changes the current API [y/n] (all API changes need to be approved by fchollet)

@fchollet everything's fixed now.


Merge work into PCL proper

https://discourse.ros.org/t/10x-acceleration-for-point-cloud-processing-with-cuda-pcl-on-jetson/18800/2?u=smac

@SteveMacenski
see this https://github.com/NVIDIA-AI-IOT/cuda-pcl/issues/26  for intergration with PCL
Thanks.


Support for randcase

Hi,
System Verilog has a rand case property, that specifies the probability of any case statement to be selected. We tried some workaround but haven't found the solution. So, can we have support for the same in pyvsc or some other way to do it?
Thanks & Regards,
Ishita

Hi @ishitapvips,
This is an interesting question, given that Python doesn't have a builtin case statement. I could see a couple of approaches. Perhaps you can provide feedback on which would fit best in your application.
Fundamentally, rand-case is the combination of weighted selection and a case statement. We could model this in Python with a weighted selection from a collection of values that represent the case options.
Let's say we're replacing a rand-case statement like the following:
randcase
    3 : x = 1;
    1 : x = 2;
    4 : x = 3;
endcase

This approach would look something like:
branch = vsc.select({ 1 : 3, 2 : 1, 3 : 4})
if branch == 1:
  x = 1
elif branch == 2:
  x = 2
elif branch == 3:
  x = 3

In other words, we specify a map with the key being the branch and the value being the branch weight. select returns the selected branch, and we use a regular if/else statement.
Another approach would be to use lambda expressions. Lambda expressions are pretty limited in Python, so you would likely need to put the actual implementation of the case branches in separate functions. That might look like this:
vsc.randcase([
  [3, lambda : do_branch1()],
  [1, lambda : do_branch2()],
  [4, lambda : do_branch3()]
  ])

So, a couple approaches to doing this in Python with PyVSC. Any thoughts on which approach would work best for your application?
Thanks and Best Regards,
Matthew

Hi Matthew,
As per your convenience, any of the approaches will work for us. But lambda function would be straight forward so can you implement that one?
Thanks & Regards,
Ishita

Hi Ishita,
I've added support for a 'randselect' method that performs a weighted selection between a list of lambda functions. Here's an example:
https://github.com/fvutils/pyvsc/blob/487620ad2ccca6c2327731f20c2665a823971656/ve/unit/test_select.py#L22-L36
I'll leave this issue open until you're able to confirm that it works for you.

Best Regards,
Matthew

Hi Matthew,
Randselect is working fine for me right now.
If any issue is found later will comment here.
Thanks & Regards,
Ishita Shah


How to display total world time in hours

I am currently using the playerstats plugin, papi plugin, and TAB plugin to display the playtime in tab. However I would like to know if it is possible to show the playtime in just the hours instead of Days + Hours + Minutes + Seconds since it takes up a lot of space (like shown in the screenshot).
I installed the expansion and used %Playerstats_ me, only:number, Total_World_Time% in the TAB config to display the playtime in the tablist.
I tried changing all of the following lines to hours, then reloading the statistic config reloading tab, however that did not work.
biggest-time-unit: hours
smallest-time-unit: hours
biggest-time-unit-for-hover-text: hours
smallest-time-unit-for-hover-text: hours

Sorry for the late reaction, totally forgot about this. Changing "min_time_unit:" from second to hour in the PAPI config worked. Thank you for helping me :)


fix non-passing tests for logging on pytest >= 3.3.0

This fixes issue #2550, where some tests fail when using newer pytest (after version 3.2.5).
The reason for this issue is that pytest added a plugin that instantiates a handler for the logging module, while the tests were expecting no handlers.
The PR solves the issue by temporarily disabling the logging module and removing any handlers for all tests on the file

Since it's also possible to disable the plug-in in other ways, it would be better to only restore it if it was actually enabled to begin with.

You can ignore the failures on 2.6 and 3.3.

@davidism are we going to remove the support for 2.6 and 3.3?

Yes

Yes. First step is removing the tests from CI, I just haven't gotten around to it for the other Pallets projects yet.

@davidism done, now it only adds it back if it was set to begin with

Thanks for figuring this out. I did a bit of refactoring: use the pytestconfig fixture instead of accessing pytest.config, and unregister returns the plugin.


Toastify alerts 🍞

Use react-toastify to replace all alerts in the code

Issue completed 🎉
with commits:

Using toastify instead of alert 🍞 #12
Handling one more exception with toastify 🍞 #12


Make dcp32 the default catalog for HCA

The approvals can be found in this Slack thread.
This ticket blocks https://github.com/DataBiosphere/azul/issues/5642

The Azul team is removing the accidentally included dataset.  Please see  https://github.com/DataBiosphere/azul/issues/5642

@bvizzier-ucsc informed me the bad dataset has been removed. We will go ahead and deploy this, hopefully this evening.

Complete


[BEAM-5999] Reconcile timer proto representation.

This adds a coder field to the timer spec, and moves the Java executor to use this spec alone when constructing executable graphs (and Python does). It should unblock Python timers working on Flink once #6981 also goes in.
See also the thread at<EMAIL_ADDRESS>
Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

It will help us expedite review of your Pull Request if you tag someone (e.g. @username) to look at it.
 We were already mutating it by cloning the output collection, but this is more invasive. I still do like the idea of timers being pushed to the SDK, but didn't fully flesh out all the options. (On the write side setting them as one sets state is easy.) I think it's more important to get the runner representation fixed, the fn-execution one is ephemeral and easier to evolve.
Fair point that timers are implicitly reserving their names, rather than explicitly doing so, which might be more motivation for the above change.

Run Java PreCommit

Run JavaPortabilityApi PreCommit

@robertwb, can you fix up https://s.apache.org/beam-portability-timers based upon the new representation. Want to keep the docs up to date.

I think there is a problem with this PR. It doesn't produce unique names anymore for the timer collection ids. For example, timer foo:
stageBundleFactory
.getProcessBundleDescriptor()
.getProcessBundleDescriptor()
.getPcollectionsMap()
.get("ParDo(Anonymous)2/ParMultiDo(Anonymous).foo.out:0")
.getUniqueName()

Should produce: "ParDo(Anonymous)2/ParMultiDo(Anonymous).foo"
Actual: "ParDo(Anonymous)2/ParMultiDo(Anonymous).output"
I couldn't find a way to resolve the timer input collection id from the timer output collection id. So I had to revert the commit for #6981.

Looking into this...
On Wed, Nov 14, 2018 at 11:30 AM Maximilian Michels <
<EMAIL_ADDRESS>wrote:

I think there is a problem with this PR. It doesn't produce unique names
anymore for the timer collection ids. For example, timer foo:
stageBundleFactory
.getProcessBundleDescriptor()
.getProcessBundleDescriptor()
.getPcollectionsMap()
.get("ParDo(Anonymous)2/ParMultiDo(Anonymous).foo.out:0")
.getUniqueName()
Should produce: "ParDo(Anonymous)2/ParMultiDo(Anonymous).foo"
Actual: "ParDo(Anonymous)2/ParMultiDo(Anonymous).output"
I couldn't find a way to resolve the timer input collection id from the
timer output collection id. So I had to revert the commit for #6981
https://github.com/apache/beam/pull/6981.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/apache/beam/pull/6986#issuecomment-438614604, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAdqgb_toAcgKxj-QhhQT26SxewKQMgbks5uu_CugaJpZM4YVAjc
.

https://github.com/apache/beam/pull/7036 should now generate unique names. Looks like the original expansion had the same issue, which is why I didn't see it. What are these unique names being used for?

What are these unique names being used for?

We use them to resolve the timer input collection id from the timer output collection id. Like in the above example.


feat: oidc proxy, handle authRequest, callback

Adds OIDC functionality by implementing a proxy for relevant oidc endpoints and executes a callback at the end of an auth flow.
Closes #30
Definition of Ready

[ ] I am happy with the code
[ ] Short description of the feature/issue is added in the pr description
[ ] PR is linked to the corresponding user story
[ ] Acceptance criteria are met
[ ] All open todos and follow ups are defined in a new ticket and justified
[ ] Deviations from the acceptance criteria and design are agreed with the PO and documented.
[ ] Jest unit tests ensure that components produce expected outputs on different inputs.
[ ] Cypress integration tests ensure that login app pages work as expected. The ZITADEL API is mocked.
[ ] No debug or dead code
[ ] My code has no repetitions

Hi @peintnermax for bring this topic forward! I currently myself, also using nextjs, am trying to build our own login UI, following the docs and this PR.
For the proxied OIDC requests I wonder, what should happen to the set-cookie headers of the responses. I see that the domain of these is still set to the Zitadel API instance (on a different host), which would mean, that they are never sent by the client when going to the login UI, as the client never directly requests from the Zitadel API, but through the login UI' backend/proxy.

Of course, the browser won't even set the cookie, as the hosts don't match in the first place.
So, are these cookie relevant and if so, I suspect the host/domain needs to be adjusted as well, before returning the response to the client?
I also started a thread on discord: https://discord.com/channels/927474939156643850/1156239247322402876

Hi @peintnermax, thnks a lot for bring this topic forward! I currently myself, also using nextjs, am trying to build our own login UI, following the docs and this PR.
For the proxied OIDC requests I wonder, what should happen to the set-cookie headers of the underlying Zitadel API instance responses. I see that the domain of these cookies is still set to the Zitadel API instance (which is on a different host), which would mean, that they are never sent by the client when going to the login UI, as the client never directly requests from the Zitadel API, but through the login UI' backend/proxy.
 Of course, the browser won't even set the cookie, as the hosts don't match in the first place.
So, are these cookies relevant, and if so I suspect the host/domain needs to be adjusted as well before returning the response to the client?
I also started a thread on discord: https://discord.com/channels/927474939156643850/1156239247322402876

Hi @sPaCeMoNk3yIam ,
so the cookie you mention comes from the old login UI. Its has no use in the new resource APIs 👍


[Bug] Broken OAuth Workflow

Summary
Upon introduction of #125, it appears that a logout and a relogin causes HTTP 401 issues - presumably from an invalid token which has been regenerated. While we don't anticipate users to consistently logout and re-login, this is a wider issue which should be mitigated with an upgraded OAuth workflow.
@martindstone has suggested using a native React component to handle OAuth login.

@martindstone has updated our codebase with a pure React component which handles the PD OAuth workflow (#130)
Closing ticket.


Mapped types incorrect intellisense

VSCode Version: 1.9.1
OS Version: OS X El Capitan 10.11.6

Steps to Reproduce:

Clone this repo: https://github.com/goodmind/vscode-intellisense-bug
Open files test/ok_intellisense.ts and test/fail_intellisense.ts and try to write some properties

FAIL
OK

This issue was moved to Microsoft/TypeScript#14344


Clean up foodcritic errors.

Resolves #190

Remove deprecated use_inline_resources syntax

Use build_essential resource instead of cookbook

[ ] All tests pass.

[x] New functionality includes testing.

[x] New functionality has been documented in the README if applicable

You want to keep the depends on build-essential cookbook and still use the resource. A long time ago we added a resource to the build-essential cookbook, but not many people used it. In Chef 14 we shipped that within chef-client. The first step in updating things is to move people over to the resource, but still depend on the cookbook.

@tas50 gotcha, thanks for the info.

Trying to figure out why unit tests are still failing...

Closing this down now, thanks very much for taking the time on this PR!
@PowerSchill's done some great work that's now in master.


0.8.2 kafka 1489 - ZEN-2245: Kafka percent disk free patch

Probably we don't really want to ever merge this, since we'd rather keep the base vendor branch 0.8.2 pristine. But we can deploy this branch directly after your review.

Whoops, I merged this by mistake


Allow args into grad_f for ULD

In the ULD drift term you provide the grad f, which is then called https://github.com/patrick-kidger/diffrax/blob/main/diffrax/_term.py#L966. It would be nice to be able to have that function also accept args as well beyond the differentiated input (e.g. grad_f(x, args) where args is just auxiliary information like is used in other areas). I can also curry the args in for my case, but it seems like in general most things allow args as input in diffrax, so I wanted to inquire about making this consistent.

@andyElking WDYT? I have vague memories of us discussing this but I may be misremembering.

Hello, I'm new to the project but I'd like to contribute. Can you assign me this issue? Thanks

Hi, sorry for the late reply @patrick-kidger. @lockwo I think I can make a PR that fixes this shortly, the needed fix is very minor.

But @ricor07 I think this is a very reasonable introductory contribution, so feel free to give it a try. Otherwise I am happy to do it over the next few days.

@andyElking i'll implement it in the next 7 days. Thank you. Can i get the issue assigned?

I think @patrick-kidger has to assign it.