Milestone work flow

Hi team,
As we discussed before, I have changed our target question to "Does a person’s frequency of recycling influence her/his opinion and attitudes towards the importance of sustainability?"
Please read through my draft proposal and thumb up if you agree.
Here are some things you need to do:

[x] Elaborate "Plan for analyzing the survey results" section and explain why we plan to include confounding variables. @K3ra-y

[x] Write an introduction paragraph about why we choose the topic. @heathervant

[x] Write "Aspects of the UBC Office of Research Ethics document on Using Online Surveys that are relevant to our survey" section. @mchiriboga

Also, please proofread the draft. Thank you for your support.

Items addressed on Milestone1 release


Link Playbooks, Opportunities, and KPIs into content flow

Either as ToC links or inline.

Done via #18


App not running after cloning

After running git clone
an error comes after npm i

@Vidhu007 Use Yarn

It's working now! I usednpm install --save --legacy-peer-deps


I can't make the cost count

Hi.
I can't make the cost be accounted. Can you help me?
fvtt 8.6 + google chrome + swade 19.4
drag ruler + enhanced-terrain-layer
https://user-images.githubusercontent.com/4999783/121898435-bfb27a00-ccf9-11eb-86b9-1f77e2db6a2a.mp4

You need to install "Terrain Ruler" module, it makes Foundry's rulers respect difficult terrain that was put down using the Enhanced Terrain Layer module.

Luisphigor is correct.  Enhanced Terrain Layer just stores information.  Terrain Ruler is what allows a ruler to use the information.  Drag Rule is just an amazing ruler.


Version 1.5.0 release PR - "Able adenine"

New and Improved Features:

Since this release, umi-transfer features internal multi-threaded output compression. As a result, umi-transfer 1.5 now runs approximately 25 times faster than version 1.0 when using internal compression and about twice as fast compared to using an external compression tool via a buffered pipe.
The new CLI arguments -t, --threads <NUM_THREADS> and -l, --compression_level <COMPRESSION_LEVEL> have been introduced accordingly. The compression level defaults to 3. Higher numbers result in marginally smaller files but take significantly longer to compress. For the number of threads, we recommend 9 or 11, if a sufficient number of logical cores are available.
The Docker base image has been updated from Debian Bullseye (Version 11) to Debian Bookworm (Version 12).
Improved integration tests, including CLI prompts.
Updates to the internal Rust libraries and dependencies.

Discontinued Previous Features:

None

This PR is merged despite failing CI tests, because the one failing test pertains to the code test coverage: No patches must have a lower coverage than the overall coverage of the project.
But for that particular file, testing the remaining three lines of code would require mocking hardware detection failures. These tests are very hard to implement and unhelpful with regard to umi-transfer.


Lot of bots qualified

There are lot of bots qualified for spacedrop. Keybase support please delete bot accounts

There are also a lot of real people who qualify at first but can't participate in the airdrop. No one seems to be doing anything about it.


[BEAM-12548] Add unit tests for passert.go

Adds basic unit tests for True(), False(), and Empty() as they were previously untested. Diff is indirectly tested in equals_test.go but could use its own tests somewhere down the line.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] 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.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

R: @lostluck


refactor(ibmsm): Support v2 API and KV secret types

Description
IBM Cloud SM API v1 is going out of support in a few months and clients must migrate to v2. This implements the migration for AVP. It also adds support for KV secret types along with logging to indicate when "new" types, that are added by IBM Cloud SM that AVP might not support, are found and skipped.
Notable changes in the v2 API/SDK:

Secret type isn't a necessary parameter to API calls (ID uniquely identifies a secret regardless of group/type)
The IBM Cloud SM SDK no longer returns the generic SecretResource struct SecretData key that allowed AVP to access the metadata and payload for (almost) any secret type. Instead structs specific to each type are returned (example: https://github.com/IBM/secrets-manager-go-sdk/blob/v2.0.0/secretsmanagerv2/secrets_manager_v2.go#L7211). While they all implement a common interface, it has 0 useful methods. This means AVP must handle each possible type explicitly. For this reason, the SDK return values are wrapped in new types IBMSecretData, IBMVersionedSecretData, IBMSecretMetadata; each type implements a generic GetMetadata or GetSecret method, so that the type-by-type code can be in 1 place and not each function
Due to the above, only select keys (whatever was previously included in the SecretData in the v1 API) for each secret type are available for replacing a placeholder. This isn't configurable outside the codebase

Checklist
Please make sure that your PR fulfills the following requirements:

[x] Reviewed the guidelines for contributing to this repository
[x] The commit message follows the Conventional Commits Guidelines.
[x] Tests for the changes have been updated
[x] Are you adding dependencies? If so, please run go mod tidy -compat=1.17 to ensure only the minimum is pulled in.
[x] Docs have been added / updated
[ ] Optional. My organization is added to USERS.md.

Type of Change

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] New tests
[ ] Build/CI related changes
[ ] Documentation content changes
[ ] Other (please describe)

Other information

Codecov Report

Merging #513 (f0d0df7) into main (717f1c0) will decrease coverage by 4.08%.
The diff coverage is 47.45%.

@@            Coverage Diff             @@
##             main     #513      +/-   ##
==========================================
- Coverage   75.38%   71.30%   -4.08%
==========================================
  Files          25       25
  Lines        1694     1903     +209
==========================================
+ Hits         1277     1357      +80
- Misses        323      450     +127
- Partials       94       96       +2

Impacted Files
Coverage Δ

pkg/backends/ibmsecretsmanager.go
60.69% <47.24%> (-18.82%)
:arrow_down:

pkg/config/config.go
84.14% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


I can’t see the home page route.

I got this error after running the server in the home page route.
An unhandled exception occurred while processing the request.
HttpRequestException: Response status code does not indicate success: 500 (Internal Server Error).
System.Net.Http.HttpResponseMessage.EnsureSuccessStatusCode()

Stack Query Cookies Headers Routing
HttpRequestException: Response status code does not indicate success: 500 (Internal Server Error).
System.Net.Http.HttpResponseMessage.EnsureSuccessStatusCode()
System.Net.Http.Json.HttpClientJsonExtensions.<FromJsonAsyncCore>g__Core|12_0<TValue, TJsonOptions>(HttpClient client, Task<HttpResponseMessage> responseTask, bool usingResponseHeadersRead, CancellationTokenSource linkedCTS, Func<Stream, TJsonOptions, CancellationToken, ValueTask<TValue>> deserializeMethod, TJsonOptions jsonOptions, CancellationToken cancellationToken)
FluentCMS.Web.UI.Pages.Home.OnParametersSetAsync() in Home.razor.cs
+
        var siteResult = http.GetFromJsonAsync<ApiResult<SiteResponse>>($"Site/GetByUrl?url={AppState.Host}").GetAwaiter().GetResult();

https://localhost:7164 is for HTTPS mode. and most of the times we are running project as https.
I think we don't need this change.
Alternatively you can enable both urls in default data


Fixed: index.php inside the examples folder throwing error while accessing '/error' route.

Hey, I gotta solution 😃 for issue #24 .
Since we provided three arguments, PHP assumes that they correspond to the first three parameters in the constructor's signature. It doesn’t automatically skip to the fifth parameter, which is why the error occurs.
PHP determines which arguments correspond to which parameters based on their order. As we are using PHP 8.3 or later for Dumbo PHP, we can utilize named arguments to pass values directly to specific parameters, regardless of their order. Named arguments have support for PHP 8.0 or Later versions.
Thank you.

We can modify our HTTPException constructor to accept a ResponseInterface object as the third parameter, like this:
public function __construct(
        int $statusCode,
        string $message = "",
        ResponseInterface $customResponse = null,
        string $errorCode = "",
        array $errorDetails = []
)

To avoid using named arguments, we must maintain the correct order of parameters when dealing with multiple default values. If $errorCode and $errorDetails are less important, we can modify the constructor like the above code. What are your thoughts on this?

Sounds good to me! 😍

@notrab Do I need to commit any changes like the above constructor or do you want to merge this existing PR?

@prince-noman let's discuss it in another PR/issue perhaps? Happy to merge this so it unblocks the current stuff, but we should probably rethink the exception handler and see if we really need the shortcut error/details. I kinda of like less magic in those situations, and allow the user to construct the response signature.

@notrab Definitely, we can talk about it later. We can continuously make things better. ✌


The footer is floating above [STYLE]

Describe
A clear and concise description of what would you like to improve.

Screenshots
If applicable, add screenshots to help explain your problem
CheckList

[ ] I'm a JWOC'24 Contributor
[ ] I've checked the existing issues, this issue is not a duplicate

(Tip - place an "x" between the square brackets to check mark)

@ritika728 Allow me to fix it : )

@MADHURYAHAIT go ahead


add ch552 support

learn contains several ch552 examples (https://github.com/adafruit/Adafruit_Learning_System_Guides/tree/main/QT_Py_CH552_Examples) which need to build with ch55xduino.
Add support for this in ci-arduino so we can test-build these and release the appropriate artifacts.

also missing: several of the recent trinkeys, see https://github.com/adafruit/Adafruit_Learning_System_Guides/pull/2837


Fix #45 Robust handling of inconsistent TabularInput keys

The issue was caused by inconsistent fieldnames. To fix this issue, I first update the fieldnames to original + new fieldnames, and then read all previous data from csv file, finally write back with updated fieldname with blank data in new fieldnames. rlworkgroup#45

The changes has been made according to the comments, also I believe that the writing along with reading should works because I firstly read the original file's data and stored locally in original_data, then I started to actually overwriting the corresponding rows while do the actual read each row from original_data.


Add ChatTurnManager 0.0.1

A script to simplify Turn Order Management, and move it into chat.

@oliviakristiansen Whoops! I just pushed a bunch of new changes right after you merged it. I guess I’m making a version 0.0.2. :-D

@erikogan if you get that PR up in the next few minutes, I'll merge it in so you have those changes this week :)


Add ruff

Motivation: ruff is 🚤 , and it resolves the pre-commit issue of PR #111

Removing myself from reviewers since I'm not a Ruff expert. Suggestions: Pavel, Oliver, Moritz


The Visual F# team is growing

Please welcome David Stephens (@NumberByColors / @NumberByColors)), who started today as a dedicated PM for Visual F#.  We are thrilled to have him join us, and very happy to see the team here in Redmond growing.

Hey everyone! I’m David. I’m super excited to work with you all on F#. It’s been one of my favorite programming languages ever since I was introduced to it (and functional programming) in college. Looking forward to meeting more of you, soon!

You'll start seeing David pop up around here soon.

Thanks, @latkin!

Wow. Very cool.
On Aug 4, 2015 1:17 AM, "David Stephens"<EMAIL_ADDRESS>wrote:

Thanks, @latkin https://github.com/latkin!
—
Reply to this email directly or view it on GitHub
https://github.com/Microsoft/visualfsharp/issues/567#issuecomment-127429447
.

Great!

Welcome @NumberByColors!

Woooo! Welcome, @NumberByColors (even if I am jealous to learn that they're now teaching F# in college).

Welcome @NumberByColors!


restyle main page

Restyled the main page. css pages were conflicting. we no longer have a welcome page at the moment. will add welcome when I figure out how to style components differently.

Resolved conflict, will have to delete "city" from App and update "city" in Header


Description of literal in people etc

From the AGM 2021 it was said to adjust this description to better align with NAR.
Can we do this in 2.0 before we publish or does it have to be in a 2.1?

Johan to check with Philippe and Dave.

https://github.com/iptc/newsinjson/pull/119


MINOR: Pydantic fixes for redshift & kafka

Describe your changes:
Fixes

I worked on ... because ...

Type of change:

[ ] Bug fix
[ ] Improvement
[ ] New feature
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation

Checklist:

[x] I have read the CONTRIBUTING document.
[ ] My PR title is Fixes <issue-number>: <short explanation>
[ ] I have commented on my code, particularly in hard-to-understand areas.
[ ] For JSON Schema changes: I updated the migration scripts or explained why it is not needed.

@sushi30 I've included tests and the airflow tests are actually not failing this is a know issue which is yet to be fixed, if you check the logs all tests actually passed


Minor Updates & Bug Fixes

citation error in Chris' article
fix #589
fix $569
minor edits in Abel's latest article

@abelhancock can you take a look?
I updated your cover image so its a little cleaner on the site — also removed the "thanks" bit at the end since its not really relevant to the audience at large...maybe you can share this on Loop or Slack and add the thanks bit there?

Thanks @plhnk! It definitely needed some help. Reads a lot better now.


Controller not working until I unplug/replug usb

Describe the bug
I'm using a ps3 controller with ViGEm/Shibari/Fireshock. The first week I had no issues, but now every time i reboot my pc my controller does not work until i unplug the usb and plug it back in. I have checked that the shibari dom server is running (it is) and have even tried reinstalling all the drivers.
To Reproduce
Installed ViGEm, Shibari dom server with automatic service startup enabled in Windows, and Fireshock.
Reboot pc with usb cable plugged in. Controller is not working until I unplug the usb and plug it back in.
Expected behavior
The controller should automatically work after reboot.
System details (please complete the following information):

OS: Windows 10 19041.746
Feeder software: ViGEm, Shibari, Fireshock
Driver Version: ViGEm 1.17.333, Shibari 1.6.180, Fireshock <IP_ADDRESS>

Edit: I did some more testing, and it's actually only on manually turning off pc, and turning it back on that it doesn't work. Using the reboot option causes the controller to work fine. Could this be caused by windows itself, e.g. maybe fast boot which only activates on shutting down, not reboots? Just a guess to why it might happen though.
Edit 2: Well I tried disabling fast boot and then turning off/on my pc and now the controller is working as intended. This leads me to believe there might be a problem with one of the drivers not loading properly in fast boot.

See #18


test-branch/execute-job/comment-from-wrong-user-on-pr/a0239391-61c5-4f74-a73a-d305ffe93079

PR for testing d674f7f7391efbc3e8e237a11b24352d8c2047cf

/canonical/self-hosted-runners/run-workflows d674f7f7391efbc3e8e237a11b24352d8c2047cf


[dv/pwrmgr] Check wakeup interrupts

Enhance wakeup interrupt covergroup.
Signed-off-by: Guillermo Maturana<EMAIL_ADDRESS>

The CI failure will be fixed once Tim commits an RTL fix.


Bug: Searching for a user/repository name throws error and crashes page

It looks like in certain cases we need to check if data is defined or not.

Hi @nickytonline,
I would like to work on the issue. I have actually resolved it on my local environment. The issue occurs when the API inside searchOpenSaucedAPI  promise returns an empty response, and userData.data is undefined. To solve this, I added a null check and initialized userData.data  as an empty array if it is undefined. Should I proceed for PR?
Before:
screen-capture (25).webm
After:
screen-capture (24).webm

Hi @SURAJ-SHARMA27. Please read the contributing guidelines to get assigned to this issue.

.take

@nickytonline  , I have submitted the PR. Could you please review it?


[WIP] Upgrade DB handling

Remove deprecated SQLAlchemy API
Use exception in place of boolean for error handling
Use with statement instead of .close() and .commit()
Start implementing MyPy recommendations (WIP)

After exploring db.py, I was wondering if it wouldn't be better to split this into entities files like this:

db/

users.py
projects.py
models.py
...

Each entity coming with its CRUD methods

You think it would simplify the project, to create decidated submodules ? I am not sure to see the benefits, but if you think it is better, start doing it for the generate part and I will diffuse it on the rest of the project


Rename tests to conform with PSR4

Description
I am including tests/system/ to composer's psr4 autoload-dev key so that new subfolders and classes will be read. However, during composer update there were test classes marked by Composer as not conforming to PSR4 standard which will be skipped in autoload. These were changed in this PR.
Checklist:

[x] Securely signed commits
[x] Conforms to style guide

@paulbalandan thank you.


Use dedicated reasons for ManagedSeedSet events

How to categorize this PR?
/area usability ops-productivity
/kind enhancement
What this PR does / why we need it:
Adds dedicated reasons for ManagedSeedSet events, e.g. "CreatingShoot" instead of simply "Reconciling", do avoid combining different events due to their reason being the same in kubectl describe view.
Which issue(s) this PR fixes:
Fixes #3908
Special notes for your reviewer:
Release note:
NONE

/cc @timebertt


AttributeError: module 'osrm' has no attribute 'Client'

I have installed osrm-py in my jupyter successfully. Yet, when I run the following test code it throws "AttributeError: module 'osrm' has no attribute 'Client'
import osrm
client = osrm.Client(host='http://localhost:5000')
response = client.route(
coordinates=[[-74.0056, 40.6197], [-74.0034, 40.6333]],
overview=osrm.overview.full)
print(response)
My environment:
Operating System: CentOS 7, 64bit
Python 3.5.6 :: Anaconda 4.1.1 (64-bit)

I am getting the exact same error. My environment is Python 3.8.5 and Ubuntu 20.04.1 LTS. Is there any idea on this?

AttributeError: module 'osrm' has no attribute 'table'
i have installed osrm-py and im getting this error. Do we have any alternative for osrm.table

Could you please elaborate more? I successfully use osrm.table. First, you have to run the osrm server.

Could you please elaborate more? I successfully use osrm.table. First, you have to run the osrm server.

osrm_result = osrm.table(locations,
output='raw',
annotations='distance'), i need to run this particular block of code
so it gives me this error AttributeError: module 'osrm' has no attribute 'table'

I see. To my understanding though, it does not seem to work.
Maybe you could try to go with this [1]. I am able to run the exact command you put above and get the results.
In this, the "table" is absolutely there.
dir(osrm)
Out[20]:
['AccessIsochrone',
'DefaultRequestConfig',
'Point',
'RequestConfig',
'builtins',
'cached',
'doc',
'file',
'loader',
'name',
'package',
'path',
'spec',
'version',
'_chain',
'base64',
'core',
'extra',
'match',
'namedtuple',
'nearest',
'simple_route',
'sys',
'table',
'trip']
If I run your command, I get:
locations = [[25.7175216,35.190589],[25.8753275,40.845692]]
In [22]: dist_od = osrm.table(locations, output='pandas', annotations='distance')
In [23]: dist_od
Out[23]:
(           0          1
0        0.0  1195886.8
1  1197131.4        0.0,
[[25.7182, 35.191183], [25.875501, 40.845421]],
None)
So maybe try this [1] instead.
[1] https://github.com/ustroetz/python-osrm

But I'm not able to install osrm=0.11.3. I tried gdal and Fiona packages
which was early errors. Then I was able to run pip install osrm command on
terminal and it is getting installed but when I check the same on python
interpreter it says package to be installed and it gives me error when
trying to install it.
Any suggestions is much appreciated
On Tue, 28 Dec, 2021, 6:08 pm Babis Babalis, @.***>
wrote:

I see. To my understanding though, it does not seem to work.
Maybe you could try to go with this [1]. I am able to run the exact
command you put above and get the results.
In this, the "table" is absolutely there.
dir(osrm)
Out[20]:
['AccessIsochrone',
'DefaultRequestConfig',
'Point',
'RequestConfig',
'builtins',
'cached',
'doc',
'file',
'loader',
'name',
'package',
'path',
'spec',
'version',
'_chain',
'base64',
'core',
'extra',
'match',
'namedtuple',
'nearest',
'simple_route',
'sys',
'table',
'trip']
If I run your command, I get:
locations = [[25.7175216,35.190589],[25.8753275,40.845692]]
In [22]: dist_od = osrm.table(locations, output='pandas',
annotations='distance')
In [23]: dist_od
Out[23]:
( 0 1
0 0.0 1195886.8
1 1197131.4 0.0,
[[25.7182, 35.191183], [25.875501, 40.845421]],
None)
So maybe try this [1] instead.
[1] https://github.com/ustroetz/python-osrm
—
Reply to this email directly, view it on GitHub
https://github.com/gojuno/osrm-py/issues/7#issuecomment-1002081005, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AW752OZCH6ZOMFYN6VHBLEDUTGVVLANCNFSM4LF4BEWQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you commented.Message ID:
@.***>

But I'm not able to install osrm=0.11.3.

@Abhi8162 Maybe you're mixing up the packages osrm-py and python-osrm (which is comprehensible).
Here, we're in the repository for osrm-py, but your problem seems related to python-osrm. Try pip install osrm-py to install osrm-py, instead of pip install osrm. It works like a charm for me.


TTM Squeeze PRO indicator #313

Added a new indicator called squeeze_pro.
Added unit tests.

Hello @zlpatel,
This looks great! And you even provided tests! 😃
I will try and roll this in soon!
Thank you for the contribution!
KJ

Hello @zlpatel,
It is up on the development branch if you want to check it out before I push to main.
$ pip install -U git+https://github.com/twopirllc/pandas-ta

Thanks,
KJ

@twopirllc I am unable to access squeeze_pro, when I download code from development branch.
Here's what I see when I install it:
Collecting git+https://github.com/twopirllc/pandas-ta
  Cloning https://github.com/twopirllc/pandas-ta to /tmp/pip-req-build-c_f_0r9l
  Running command git clone -q https://github.com/twopirllc/pandas-ta /tmp/pip-req-build-c_f_0r9l
Requirement already satisfied, skipping upgrade: pandas in /usr/local/lib/python3.7/dist-packages (from pandas-ta==0.2.93b0) (1.1.5)
Requirement already satisfied, skipping upgrade: numpy>=1.15.4 in /usr/local/lib/python3.7/dist-packages (from pandas->pandas-ta==0.2.93b0) (1.21.0)
Requirement already satisfied, skipping upgrade: python-dateutil>=2.7.3 in /usr/local/lib/python3.7/dist-packages (from pandas->pandas-ta==0.2.93b0) (2.8.1)
Requirement already satisfied, skipping upgrade: pytz>=2017.2 in /usr/local/lib/python3.7/dist-packages (from pandas->pandas-ta==0.2.93b0) (2018.9)
Requirement already satisfied, skipping upgrade: six>=1.5 in /usr/local/lib/python3.7/dist-packages (from python-dateutil>=2.7.3->pandas->pandas-ta==0.2.93b0) (1.15.0)
Building wheels for collected packages: pandas-ta
  Building wheel for pandas-ta (setup.py) ... done
  Created wheel for pandas-ta: filename=pandas_ta-0.2.93b0-cp37-none-any.whl size=207877 sha256=5d4a4e64936bc3ba1e0afc89d72bc5efe84936563582418818c15bc67a260f81
  Stored in directory: /tmp/pip-ephem-wheel-cache-9r5v8m6l/wheels/64/67/96/15e918c3b53b4a323b5bd037c7f08be5ef6908141c50f07c76
Successfully built pandas-ta
Installing collected packages: pandas-ta
  Found existing installation: pandas-ta 0.2.93b0
    Uninstalling pandas-ta-0.2.93b0:
      Successfully uninstalled pandas-ta-0.2.93b0
Successfully installed pandas-ta-0.2.93b0

why does it still reference 0.2.93b0 instead of 0.2.97b0 ?
Here's the error I get:
module 'pandas_ta' has no attribute 'squeeze_pro'

@twopirllc I figured it out. I needed to use below command to install package from development branch:
!pip install -U git+https://github.com/twopirllc/pandas-ta.git@development

@zlpatel,
Doh! Sorry I sent the wrong link. Glad you found it.

@twopirllc I have tested it out in my code. It looks good.


Add util for making Markdown links

Links in Markdown should not be constructed with [${title}](${url}), because if title contains a closing square bracket it will mess up the formatting at best and cause API errors at worst.
We should make a function in Util to construct Markdown links, and make all plugins using Markdown use that function.

It seems that Markdown parsing in Telegram is somewhat broken, as \[ and \] are not interpreted correctly. This means that user-generated titles should only go in HTML links (for which a sanitizer would also be needed).


More query tests (1 failing)

One test is failing, think it might have to do with RPC not liking multiple lines.
Might be related to #20

Yea I ran the test in debug, I agree its probably related to the newline issue in #20


Delete/remove client method (flask client)

I can only speak for the Flask client as that is what I'm using. It would be great if the oauth.register(...) method had an oauth.remove(client_name).
Is your feature request related to a problem? Please describe.
I wanted to write a test for something and it involved registering an oauth2 (flask) client for testing purposes. After the tests complete, I want a clean way to remove that testing client from the oauth object, but I cannot see a way to do this.
Without it, running the test poses the following issues. (1) If I create the client in my test with a fixed name (e.g. 'Test Client'), it will return the first client from the oauth registry for all subsequent tests, instead of creating a new client each time. But if I create the client with a unique name for each test (e.g. 'Test Client 1', 'Test Cleint 2', ...), the oauth registry will grow for as many clients are registered until the server restarts. Obviously the latter is the expected behaviour, but a .remove_client method seems missing.
Describe the solution you'd like
Just like with oauth.register(...), maybe something like oauth.remove_client(client_name) would be nice. For example, in the https://github.com/lepture/authlib/blob/master/authlib/integrations/base_client/registry.py, we could add a new method:
def remove(self, client_name):
    if client_name not in self._registry:
        raise KeyError(f"Client {client_name} not found in registry.")

    if client_name not in self._clients:
        raise KeyError(f"Client {client_name} not found in registry.")

    del self._registry[client_name]
    del self._clients[client_name]

But I am not sure if this is sufficient or if it overlooks other issues.
Describe alternatives you've considered
As far as I can tell, we might be able to do this by simply editing the internal oauth._clients and oauth._registry dictionaries. For example del oauth._clients[my_test_client_name]; del oauth._registry[my_test_client_name] but I don't know if this fully accomplishes the goal. It definitely doesn't feel nice.

I think it is a good feature, feel free to send a pull request. The method should be added in every clients.

Great, I'll take a look.

The method should be added in every clients

Could you please explain a bit more what you mean by this? The BaseOAuth class looks like it is inherited by the clients, so I assumed it would be enough to just add the method there.


initial upgrade to Hyrax 2.3.3

also remove redundancies from gemspec
Connects to #22.

LGTM, merging.


Do not merge, just a test: Re-adding the colors to automated functional tests

What this PR does / why we need it:
Just checking how bad the raw logs look when colors are enabled.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:

NONE

@jean-edouard hey, maybe you want to use sth like this, using colors only when supported:
https://github.com/openshift-kni/performance-addon-operators/blob/master/hack/run-functests.sh#L9


inject ssh keys for bigip VE

I did not find the related staff to inject ssh keys into bigip VE using DO.
Is there any plan for this? if not, any particular considerations? thanks.

AT-327 opened for tracking.

Resolved in 1.5.0


Correction ?

when I read this :
The Buffer Cache Hit Ratio counter is specific to an application. However, a rate of 90 percent or higher is desirable. Add more memory until the value is consistently greater than 90 percent.
I think the third sentence is in opposite to the second : 90 percent or higher is desirable and add more memory until the value is consistently greater than 90 percent
Thank for your answer
[Enter feedback here]

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

ID: 56ae21ee-e345-0f62-9e51-e1495228c9ed
Version Independent ID: 7ae44dfe-8aba-5945-e758-a3c5c7fc991a
Content: Monitor Memory Usage - SQL Server
Content Source: docs/relational-databases/performance-monitor/monitor-memory-usage.md
Product: sql
Technology: performance
GitHub Login: @julieMSFT
Microsoft Alias: jrasnick

#assign:julieMSFT

@yvesgui41 Hello, thank you for contacting Microsoft. Buffer Cache Hit Ratio indicates the percentage of pages found in the buffer cache without having to read from disk. Because reading from the cache is much less expensive than reading from disk, you want this ratio to be high. Generally, you can increase the buffer cache hit ratio by increasing the amount of memory available to SQL Server for storing pages in memory. Please refer to this article.  I hope this answers your question. I will go ahead and close this issue. The @notification mechanism continues to work even after an item is closed, in case further communication is needed. We hope to hear from you again.

#please-close


Raise NotImplementedError for other Attractors other than Earth in Heliosynchronous orbit creation

Fixes #868
The classmethod Orbit.heliosynchronous uses the raan_from_ltan which works only Earth at the moment. Therefore it raises a NotImplementedError.

Perfect! Thanks @iamabhishek0 :raised_hands: notice though that this doesn't fix the issue, so I will reopen it if it gets closed automatically.
Merging!


relevant errors

controllers should be less ambiguous when failing
return the errors not a general message
i see this in holds and on rentals

should always do this

closing this issue, will open a different one if something specific arises


Report uncaught exceptions occurring on the Teletype package

This will allow us to get a sense of what and how many exceptions are thrown in the Teletype package even if it is not yet bundled in Atom Core.
To test this, I created a fake exception on the Teletype package and verified it was correctly reported to BugSnag. I think we should hotfix this to stable and beta so that we can start gathering information without having to wait for a full release cycle.
/cc: @atom/maintainers

Given that we're only a few weeks away from the next stable release, how would you feel about hotfixing only to beta? I feel like that offers a good balance of risk management (i.e., zero risk of breaking things in the stable release 😅) and expediency (i.e., we're only waiting just a little bit longer before this change comes to the stable release in early December).

Yeah that sounds good. 👍


Dual battery bugs

Removing one of my thinkpad's two batteries makes waybar lose it's sanity.
It usually shows a random amount above 200%. After a couple of minutes it shows either 0%, 5% whilst flashing white/red or copies the second batteries amount. Clicking the battery module to show remaining time and back to percentage again resets the amount to a random value above 200%. Although I'd really love to have my batteries charge past 100% :]
Also for some reason it breaks the wifi module the first time I remove one of the batteries, making it shows disconnected for about a minute, even though I'm clearly still connected to a hotspot...

@SairesArt this is possibly fixed in current master. Could you please test and see?


Bias-Tee in ini file?

Hi, it would be great if there was a setting in the .ini file to enable the bias-tee automatically.
I use an active amplifier near the SDR, so auto-start isn't very useful unless it also powers the bias-tee at the same time.
Also thanks for making this software, it seems to work great and is a huge improvement over using VAC and SDR# instances or similar!

Closing this one


First time not work After ajax request

Hi there!
I'm having issue, when i click first time, it send request with Ajax, then i wanna copy the ajax response.First time it don't copy i need to click again. Here is my code snippet.
<script src="/zero_clip/ZeroClipboard.js"></script>
$(document).ready(function() {
  var client = new ZeroClipboard( $("#copyLink") );
  $("#copyLink").click(function () {
    $.ajax({
      url: "/controller/method/",
      method: "GET"
    }).success(function (res,status) {
      client.setData('text/plain', res.url);
    });
  });
});

It works fine this this way
<script src="/zero_clip/ZeroClipboard.js"></script>
$(document).ready(function() {
  var client = new ZeroClipboard( $("#copyLink") );
  $("#copyLink").click(function () {
    client.setData('text/plain', 'Some text');
  });
});

Thanks

From docs/instructions.md#synchronicity-required-during-copy:

Synchronicity Required During copy

If a handler of copy event intends to modify the pending data for clipboard injection, it MUST operate synchronously in order to maintain the temporarily elevated permissions granted by the user's click event. The most common "gotcha" for this restriction is if someone wants to make an asynchronous XMLHttpRequest in response to the copy event to get the data to inject — this will not work. You must make it a synchronous XMLHttpRequest instead, or do the work in advance before the copy event is fired.


fix badges / clean up README

Hi,
Nice having the icons clickable :+1:
But I would like to keep the references to both the documentation and the API documentation since parts of the README is included into the book
(see book/en/src/preface.md)
From the book context it makes sense to have both the icons and the headers.
So please modify the PR to only fix nice and clickable icons and I'm happy to merge :smiley:

bors retry


fix: Added stateRef for getting updated value in setState

Interesting commit. I'll need to do some more thorough testing to ensure the ref doesn't conflict with deeper mutation/state update issues.
Can you give me a short, explicit description of what the ref offers?

See my comment on issue #5 for information about why this PR was closed.


Shadow remote win 10 session?

Is it possible to shadow a session on a remote computer? Currently, mstsc says it cannot connect to the computer if I use this line:
mstsc -v:[ip] /shadow:2 /control
but it works correctly if I rdp to another account on the remote machine and then shadow locally on the rdp session using:
mstsc /shadow:2 /control

Closed as solved. If not - reopen it.


[HiveMQ] CODEOWNERS

I see the following error from GH, seems we have to make some additional changes to utilize this feature?

I see the following error from GH, seems we have to make some additional changes to utilize this feature?

Yes, in order to be listed within CODEOWNERS file users need to have write access to the repository. I think we can hold the access and close the PR for now. WDYT @kiview ?

I will close this until we find a proper way to give access.


Building Main branch Godot from source for Windows(constant errors)

Godot version:
Master branch
OS/device including version:
Win 10 home
Issue description:
Error occur image:

Steps to reproduce:
Download Python 3.8.2 install it with all requirements, install VS with required c++ packages, install SCons 3.1.1(stable), install pywin32 for multi threading, clone master branch with Git Bash, try to build Godot with  SCons and then it fails while building.
Minimal reproduction project:

Did you install the Windows SDK?

Did you install the Windows SDK?

Try a different compiler for now.

In a future version of MSVC,  doesn't transitively include.

https://github.com/BinomialLLC/basis_universal/commit/8cd007442103174a12733301a7508a7e9c1a441f is most probably the reason.
But there hasn't been a new release version upstream with this patch so I am not sure if should get the latest commit or wait for a new release.

But there hasn't been a new release version upstream with this patch so I am not sure if should get the latest commit or wait for a new release.

We're already tracking an upstream commit and not a stable release, so we can update to latest master.

In a future version of MSVC,  doesn't transitively include.

BinomialLLC/basis_universal@8cd0074 is most probably the reason.
But there hasn't been a new release version upstream with this patch so I am not sure if should get the latest commit or wait for a new release.

Adding  library in basisu_enc.h will solve problem. So as I understand all older versions of Godot will not compatible with VS 2019(then it is better use older version like VS 2017?)  with out adding this library, new version of Godot will adapt to a changes made by VS 2019?
Thanks on help @akien-mga && @Anutrix

So as I understand all older versions of Godot will not compatible with VS 2019(then it is better use older version like VS 2017?) with out adding this library, new version of Godot will adapt to a changes made by VS 2019?

Basis universal is a new dependency for Godot 4.0, the 3.2 branch and earlier don't include it and won't have this build issue.


pull request for e2e test at 02/01/2017 13:45:12

:white_check_mark: Validation status: passed
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.


Want to add BMI CALCULATER Extensions

this extension help to calculate body mass index
pls check pr mam

Can i work on this issue under gssoc 2022

hi @ridsuteri
can i work on this issue
kalash singhal
gssoc contributor

hi sir i am sudip basak ..i am work this issue

@sarthakk24 @kalash2205 @sudip004 the issues are assigned on first come first served basis, hence I am assigning the issue to @adi2002vicky . If he fails to submit a Pr It will be assigned to the person after him in the queue :)

@ridsuteri  please check Pr mam/sir i have done this work..

@adi2002vicky We will start reviewing and merging PRs in 1-2 days after proper orientation of mentors, Don't worry about that. Keep contributing :)

okk thankyou mam

closing this issue since the PR is merged


Save some context when using functions

With functions it's easy to blow past all context limits. Atm i delete messages one by one once i feel I have too many.

Add "Delete Messages After This"
should be able to choose to not send function output to gpt when sending next message eg: [x] never send

Add "Delete Messages After This"

Created https://github.com/tarasglek/chatcraft.org/pull/211 to fix this part.


remove windows 2004 sac, bump versions, switch to SLES from Ubuntu for dapper

remove Windows Server 2004 SAC from drone build steps and manifest template
fix a few aliases in package.ps1 to use full cmdlet
bump to dapper 0.5.7
bump to go1.16.9 in Dockerfile.dapper and remove rancher/trash
switch dapper dockerfile from using ubuntu to sles
after switching to sles, use docker installed from zypper instead of the antiquated docker-cli builds

Commits squashed


Is it dead?

The last commit was nearly a year ago. I really like the Rhai syntax and the way to use it from Rust, but I don't want to use a dead project...

Sadly, it's definitely in a "deep sleep" mode.  I have some ideas around how to make it easier to use and support more function types, but I unfortunately haven't had time to come back to this project and implement them.

Thank you for the quick reply! That's very sad to hear. I will definitely keep Rhai in my watch list, as I don't have time for yet another project at the moment. I really hope to see it "wake up" again in the future :)


use ES2015 object property shorthand

mdn

I'm a simple man. I see opportunity to create concise code, I do so. :smile:
ES2015+ fanatic

New commit takes care of #11 as well


SMART-141 Add formatting for WinRM log stream

Signed-off-by: Mykola Mandra<EMAIL_ADDRESS>

looks really good.  one comments which needs considering (use of document v $element), one more i think worth doing (tooltip)
don't have access to windows so not tested but assuming you've tested i'm happy to merge and correct later if there are glitches


Bug 1088

Changes
closes #1088

Checklist

[x] I followed contributing guidelines

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Niklas Wolf seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


sony: sepolicy: Do not audit cameraserver timerslack_ns

avc: denied { write } for pid=3982 comm="Binder:867_E" name="timerslack_ns"
dev="proc" ino=36459 scontext=u:r:system_server:s0
tcontext=u:r:cameraserver:s0 tclass=file permissive=0
Signed-off-by: Humberto Borba<EMAIL_ADDRESS>Change-Id: I05a62de452b1ef93eaeaa099bb8c8f4678814c32

What is the justification on fully silencing this denial ?
Thanks!

logspam

I meant the reason we don't want it allowed in the first place, should be clear in a dontaudit otherwise someone looking through it can't know why this was fully silenced.

well so we have a big problem [hidden] :)
https://github.com/sonyxperiadev/device-sony-sepolicy/blob/master/system_server.te#L10-L15

I'm missing a description on how to reproduce this. What actions did you perform to trigger these denials? If I can reproduce, I can review this more efficiently.


Discuss tuning parameters in generative music

Linked to #2, discuss how to review the generative process and tune parameters to achieve the balance between unpredictable and predictable.

Closed with commit 4299b26ed687c3aaae6a81dbb27cfbe3b71d5f93


[Spark] Support enable Delta Uniform on tables with DV supported but not present

Which Delta project/connector is this regarding?

[x] Spark
[ ] Standalone
[ ] Flink
[ ] Kernel
[ ] Other (fill in here)

Description
Support enable Delta Uniform on tables with DV enabled but not present.
How was this patch tested?
Unit test
Does this PR introduce any user-facing changes?

No

This is great!
It would be good backport this to the 3.0 branch as well.

This needs a unit test!

@tdas I just rename this as a refactor only. The actual behavior change will come in a later PR.


ui: Make logo fit inside the card

The wordmark logo was not properly resizing inside of the card it resides in. In my case, I was only seeing part of it.


CombinedPlot horizontal scales should be linked

if you zoom in one, the other should zoom also.
this is a regression from classic beaker.

and for resize

dragging


When i try to start this error appears Traceback (most recent call last):   File "/content/gdrive/MyDrive/sd/stable-diffusion-webui/webui.py", line 15, in      from modules import paths, timer, import_hook, errors   File "/content/gdrive/MyDrive/sd/stable-diffusion-webui/modules/paths.py", line 4, in      from modules.paths_internal import models_path, script_path, data_path, extensions_dir, extensions_builtin_dir ModuleNotFoundError: No module named 'modules.paths_internal'

Traceback (most recent call last):
File "/content/gdrive/MyDrive/sd/stable-diffusion-webui/webui.py", line 15, in
from modules import paths, timer, import_hook, errors
File "/content/gdrive/MyDrive/sd/stable-diffusion-webui/modules/paths.py", line 4, in
from modules.paths_internal import models_path, script_path, data_path, extensions_dir, extensions_builtin_dir
ModuleNotFoundError: No module named 'modules.paths_internal'

that was fixed, try renaming/removing your sd folder

Same error. I'm getting this error on this latest Colab version.

if that specific error persists, do a manual git pull to the webui folder and see if it updates

@TheLastBen Hi, new here. Can you explain step-by-step how to do that in the fast_stable_diffusion_AUTOMATIC1111.ipynb colab notebook? Including where to input the code on the page? Thank you!


Improve coverage by adding quality tests

Improve coverage by adding quality tests.
To run coverage, prefix

COVERAGE='YES' rake

or

RAILSGOAT_MAINTAINER='yes' rake

then open this file in the browser coverage/index.html.
Then pick some uncovered lines to write tests for and see the coverage go up.

Pruning


better-crowdin-prs

What

Better crowdin PRs for more autonomy

Codecov Report

Merging #3889 (711cf48) into develop (fe68ea5) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop    #3889   +/-   ##
========================================
  Coverage    10.59%   10.59%
========================================
  Files          270      270
  Lines        13365    13365
========================================
  Hits          1416     1416
  Misses       11949    11949

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Drawing tools fix for TV charts

Increasing chart height to 500 in settings makes the drawing tools bar show up on left side. Maybe set this by default?

I used to have it at 500px, but then I got feedback that it made sense to default the size to the original chart's dimensions. Perhaps the better way to do this is to offer this as part of the settings vs defaults.. thoughts?

Could, yes. You mean make a checkbox "Show drawing tools"? I like this idea.


[enhancement] Tachyone Entangler

Hi there, thanks for these great modules!

About the Chaos rotation knob, would it be an idea to have either an attenuator for it, ora slector knob next to it with 3 different settings: max,med,min.
Where max is how it is now, medium 50% of max spread out over the rotation knob resolution, 30% of max spread out over the rotation knob resolution.
Cause the 100% it has now is very wild, the more interesting subtle effects are in the very first start positions of the rotation knob, after that it quickly turns into noise.

yeah, i was thinking about that too. i'll figure out some way to make it more precise on the smaller end for the next update. thanks for the feedback!


[Bug] Double where clause in get_where_subquery

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
It seems get_where_subquery add the where clause twice:
Compiled query looks like this:
WITH validation AS (
    SELECT        "Date Time" AS not_null_field
    FROM (select * from (select * from "dbname"."schemaname"."tablename" where a = 'something') dbt_subquery where a = 'something') dbt_subquery -- https://docs.getdbt.com/reference/resource-configs/where
),
validation_errors as (
    SELECT *
    FROM validation WHERE not_null_field IS NULL
)
SELECT * FROM validation_errors

Expected Behavior
I would expect the compiled query to look like this
WITH validation AS (
    SELECT        "Date Time" AS not_null_field
    FROM (select * from "dbname"."schemaname"."tablename" where a = 'something') dbt_subquery -- https://docs.getdbt.com/reference/resource-configs/where
),
validation_errors as (
    SELECT *
    FROM validation WHERE not_null_field IS NULL
)
SELECT * FROM validation_errors

Steps To Reproduce
dbt==0.21.0
I created this test:
-- test_quoted_not_null.sql
/*
    This test was needed because the standard not_null test
    doesn't handle column names with special characters in it

    So create a new version where the column name is quoted
*/
{% test quoted_not_null(model, column_name) %}
WITH validation AS (
    SELECT        "{{ column_name }}" AS not_null_field
    FROM {{ get_where_subquery(model) }} -- https://docs.getdbt.com/reference/resource-configs/where
),
validation_errors as (
    SELECT *
    FROM validation WHERE not_null_field IS NULL
)
SELECT * FROM validation_errors
{% endtest %}

And use the test on a source
-- sources.yml
version: 2

sources:
  - name: source_name
    schema: source_name
    tables:
      - name: table_name
        columns:
          - name: "Date Time"
            description: "Date of data in the report"
            tests:
              - quoted_not_null:
                  config:
                    where: "a = 'something'"

Then run the tests: dbt test
And check out the compiled sql
Relevant log output
test still passes, so not much relevant info here

Environment
- OS: Docker Dev Container, based on mcr.microsoft.com/vscode/devcontainers/python:0-3.8
- Python: 3.8
- dbt: 0.21

What database are you using dbt with?
redshift
Additional Context
No response

@timsteinkuhler Thanks for the issue! There are two things at play here:

You don't need to manually specify {{ get_where_subquery(model) }} in your custom generic test definition. dbt will do it for you, by replacing {{ model }} with {{ get_where_subquery(ref(...)) }} or {{ get_where_subquery(source(...)) }} accordingly. That's how the where config is supported on all generic tests. So, try changing your test definition to:

{% test quoted_not_null(model, column_name) %}
WITH validation AS (
    SELECT        "{{ column_name }}" AS not_null_field
    FROM {{ model }}
),
validation_errors as (
    SELECT *
    FROM validation WHERE not_null_field IS NULL
)
SELECT * FROM validation_errors
{% endtest %}

As far as the reason for needing a custom generic test (This test was needed because the standard not_null test doesn't handle column names with special characters in it), you can set the column-level quote property (docs), and then the standard not_null test should work just fine:

version: 2
models:
  - name: model_a
    columns:
      - name: 123abc
        quote: true
        tests:
          - not_null

select
      count(*) as failures,
      count(*) != 0 as should_warn,
      count(*) != 0 as should_error
    from (
select *
from "jerco"."dbt_jcohen"."model_a"
where "123abc" is null
    ) dbt_internal_test

Either way, you should be able to avoid this issue.

Thanks for taking the time to explain, this helps me understand dbt even better! Great work by the way!


Turn api in rtc engine

Done so far:

pass media packets directly between TURN server and RTC Engine via Erlang messages, skipping libNICE

TODO:

add support to Firefox
remove dependency to libNICE


Add LD_LIBRARY_PATH as an env varible for the use of vsphere plugin

Hi folks,
We are working on the vSphere plugin for velero. To enable the data path of backup and restore using VDDK API, VDDK libraries need to be dynamically linked in the run time. Hence, we plan to copy required libraries to the directory /plugins in velero pod container and propose to add the env variable, LD_LIBRARY_PATH, to /plugins. Feel free to suggest any better alternatives.
Best,
Lintong

This would be a good thing to revisit along with the other open q's on install/plugin/BSL/etc. workflow.

Agreed. We should probably compile a list of things we want to address, even if we don't do that now.

Thanks Steve for the approval.
Best,
Lintong
From: Steve Kriss<EMAIL_ADDRESS>Reply-To: heptio/velero<EMAIL_ADDRESS>Date: Monday, September 23, 2019 at 8:35 AM
To: heptio/velero<EMAIL_ADDRESS>Cc: Lintong Jiang<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [heptio/velero] Add LD_LIBRARY_PATH as an env varible for the use of vsphere plugin (#1893)
@skriss approved this pull request.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fheptio%2Fvelero%2Fpull%2F1893%3Femail_source%3Dnotifications%26email_token%3DANHQY2DZL3YT6327OIRLDRLQLDO5FA5CNFSM4IYRDAA2YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCFSZIOY%23pullrequestreview-291869755&data=02|01|lintongj%40vmware.com|c6e7a1e57f9f4dad8a1e08d7403bb47e|b39138ca3cee4b4aa4d6cd83d9dd62f0|0|0|637048497488294499&sdata=3mGan6dtbjabwIFFdQr5NGUKorad8YvZWDdO6LEjoeY%3D&reserved=0, or mute the threadhttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FANHQY2AXZ643SCC35HTMDLTQLDO5FANCNFSM4IYRDAAQ&data=02|01|lintongj%40vmware.com|c6e7a1e57f9f4dad8a1e08d7403bb47e|b39138ca3cee4b4aa4d6cd83d9dd62f0|0|0|637048497488294499&sdata=ruv3Ew9AlNSrnESzEdpmj8i9KrNM3Th%2BR2ZlfIfWtOo%3D&reserved=0.

Should we add some documentation to https://velero.io/docs/v1.1.0/plugins/ to mention that LD_LIBRARY_PATH is set to /plugins in the Velero container, so can be used to copy any libraries a plugin might need?

If @skriss is ok with revisiting this later then this is ok. It makes sense to me to try to avoid adding something that won't be used in all cases.
I also think @prydonius's suggestion is good: if you can add to our docs and also a changelog file (https://github.com/heptio/velero/tree/master/changelogs/unreleased) it would be great.

I agree with all the sentiments above; I'll approve when the documentation and changelog are updated.
LD_LIBRARY_PATH can be fairly useful for any plugins that require C extensions, but I also think this should be added to our list of things to address for a plugin design iteration.

Agreed with Nolan that LD_LIBRARY_PATH could be useful any plugins that requires c extensions.
Any guidance about where to update the documentation and changelog?

site/docs/master/plugins.md is probably a good place to start for docs.
https://github.com/heptio/velero/blob/master/CONTRIBUTING.md#changelog has details on the changelog.

Thanks for the approvals.


[YUNIKORN-2963] E2E Test for Foreign Pod Tracking

What is this PR for?
This pull request introduces several end-to-end tests for verifying the tracking of foreign pods in the YuniKorn scheduler. The tests cover the following scenarios:

Verify foreign pod tracking fails for untracked or incorrectly mapped pods
Verify multiple foreign pods
Verify foreign pods on different nodes

What type of PR is it?

[ ] - Bug Fix
[x] - Improvement
[ ] - Feature
[ ] - Documentation
[ ] - Hot Fix
[ ] - Refactoring

Todos

[ ] - Task

What is the Jira issue?
https://issues.apache.org/jira/browse/YUNIKORN-2963
How should this be tested?
Screenshots (if appropriate)
Questions:

[ ] - The licenses files need update.
[ ] - There is breaking changes for older versions.
[ ] - It needs documentation.

Hi @pbacsko , Could you please take a look ?

@rrajesh-cloudera I'm sorry to say but we already have e2e tests. Please see tests.
I'll take a look but I don't think we need more tests.

@rrajesh-cloudera I'm sorry to say but we already have e2e tests. Please see tests. Looks like there was a communication issue :/
I'll take a look but I don't think we need more tests.
EDIT: oh, I can see you added the new tests to the same file. Anyway, I'm looking at them.

The First one is positive test case which you added , the 2nd is a negative test case for the first in which we are simulating an error where a pod from kube-system is either untracked or incorrectly mapped.
The Third case  Focuses on ensuring multiple pods from kube-system are all being tracked and 4th case Verifies that foreign pods are accurately tracked across different nodes.
Each case targets different potential issues, ensuring comprehensive coverage of the system’s functionality and robustness

The First one is positive test case which you added , the 2nd is a negative test case for the first in which we are simulating an error where a pod from kube-system is either untracked or incorrectly mapped.
The Third case  Focuses on ensuring multiple pods from kube-system are all being tracked and 4th case Verifies that foreign pods are accurately tracked across different nodes.
Each case targets different potential issues, ensuring comprehensive coverage of the system’s functionality and robustness

This is just simply not the case.


No throughput argument on az cosmosdb database create

This page says there is a throughput argument on "az cosmosdb database create" but there isn't. How do we use the cli to create a shared throughput database?

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

ID: da15197a-eca2-19d6-72e3-be853337a759
Version Independent ID: eb84d467-86df-121e-3b04-66b499342f7f
Content: az cosmosdb database
Content Source: src/command_modules/azure-cli-cosmosdb/azure/cli/command_modules/cosmosdb/_help.py
Service: cosmos-db
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

I guess I was using an old cli.


avoid name conflict of another npm package called html-purifier

fixing the wrong bin path and avoid name conflict of another npm package called html-purifier

:+1:  cc @adon-at-work @yukinying for second review

CLA is valid!


Extension is not available in Firefox Add-On Store

Firefox also supports v3 manifests (as of Firefox 109), so it should also be possible to ship it there as well …

Work is being done in the https://github.com/bramus/mastodon-profile-redirect/tree/firefox branch.
A version was submitted and is “Awaiting Review”

After some trial and error I got it working: https://github.com/bramus/mastodon-profile-redirect/commit/2a2e17a322e7c1bd479d90c4ea6d402870d2c3f4
A build has been submitted, awaiting approval …

Extension has been approved and is now published: https://addons.mozilla.org/en-US/firefox/addon/mastodon-profile-redirect/
Note that it is only compatible with Firefox 109 and up due to the usage of Manifest v3. To make it work in Firefox prior to 107, follow these instructions


Release minor fixes

Have done:

#65
#53

List of oncomings:

[x] Serialization problem #48

Hello @Roman223! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file bamt/networks/base.py:

Line 330:17: W503 line break before binary operator
Line 336:17: W503 line break before binary operator
Line 468:17: W503 line break before binary operator
Line 469:17: W503 line break before binary operator
Line 470:17: W503 line break before binary operator
Line 543:17: W503 line break before binary operator
Line 544:17: W503 line break before binary operator
Line 545:17: W503 line break before binary operator

In the file bamt/networks/big_brave_bn.py:

Line 32:21: W503 line break before binary operator


Update benchmarks

Hello! When comparing to scoop-search, please use sfsu with --mode both otherwise it is not representative. sfsu does not read files without it, and that is the main (unavoidable) bottleneck of search. With the mode both the functionality is comparable. In my benchmarks scoop-search is consistently faster than sfsu with --mode both (about 30%).


Adding an "overwrite" option when copying files

This feature request is related to files/copy end point. Currently there is no option to overwrite a file if it already exists, when copying file from somewhere to another.
Why is this feature valuable to you? Does it solve a problem you're having?
It is a common occurrence that when copying a file to somewhere, the file might already exist in the target path. For example we have an automation script which backs up some files every night by copying them from one folder to the another folder, but the folder we are copying to every night, stays the same. So the files already exist in the target folder, but we want them to be overwritten every night.
Describe the solution you'd like
Just like in the files/upload end point, a "mode" parameter which would allow us to overwrite the target file (if exists) would make things very convenient.
Describe alternatives you've considered
We are currently first checking if the file exists, if it exists we delete the file, and then we copy it. So we need to send 3 different requests instead of 1. Also one of the requests is relatively dangerous one (delete), which might cause other issues in the future, like accidentally deleting a file/folder.

Thanks for writing this up! I can't make any guarantees, but I'll pass this request along to the team.


Revert "chore(deps): monaco-editor"

This reverts commit 5f8520f0d44273871dde9739e9967e65a81eec29.
I did some digging on this issue and found that this commit is where it breaks for me. Not sure if this is the best fix - seems to work. Thought I would just make a PR so it can be tested.
Here's the changelog for the updates on this package in case anyone wants to take a look.
Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the main branch of freeCodeCamp.
[x] All the files I changed are in the same world language, for example: only English changes, or only Chinese changes, etc.

Closes #42102

Did this work, Tom?  There are another couple of version bumps on the horizion: https://github.com/freeCodeCamp/freeCodeCamp/pull/42122 and https://github.com/freeCodeCamp/freeCodeCamp/pull/42119 which may make things worse/better.


Prepare release

Versions
@safe-global/api-kit: v2.5.4
@safe-global/protocol-kit: v5.0.4
@safe-global/relay-kit: v3.2.4
@safe-global/sdk-starter-kit: 1.0.4

Main changes

#1039

Pull Request Test Coverage Report for Build<PHONE_NUMBER>5
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 76.251%

Totals

Change from base Build<PHONE_NUMBER>0:
0.0%

Covered Lines:
784

Relevant Lines:
961

💛  - Coveralls


Feature Request - API - Graphite.Host

Low priority, wanted to get this on the radar if at all possible...
v1/settings/graphite.host is super-useful for us gathering statistics out of our Rancher instances. However after using it for a while in a few environments, i feel a few updates to how it works could make it even more useful.

Enable setting to make use of the rancher-dns. Useful when pointing it to a managed rancher node.
Enable the value to be updated without requiring a server restart to take effect.
Allow it to be updated through the API with a curl command. This works when browsing the UI but when i try run it in by copying the curl command out i get errors.

Useful
Info

Versions
Rancher v0.59.0 Cattle: v0.148.0 UI: v0.90.0

Access
github admin

Route
null

It's a server setting so using internal container DNS doesn't really make any sense (which environment?); you can use the external-dns (Route53, etc) to create a public entry that the setting could then use.
Updating without restart seems reasonable...
The UI is the API.  Anything that's possible can be done with curl or any other http client.  Show us the request you're making and response you're getting back.  There's probably something simple wrong.

Good point, will have to come up with something else for non-cloud setups.

The request i'm taking out of the curl command it gives me, for example:
curl -u "${CATTLE_ACCESS_KEY}:${CATTLE_SECRET_KEY}" \
-X PUT \
-H 'Accept: application/json' \
-H 'Content-Type: application/json' \
-d '{"activeValue":"", "id":"1as13", "name":"graphite.host", "source":"Database", "value":"example"}' \
'http://<IP_ADDRESS>:8080/v1/activesettings/1as13'

I'm trying to run the above from a container with the following labels set, without joy sadly.
      io.rancher.container.create_agent: true
      io.rancher.container.agent.role: environment

I'm going to close this, after some thought.... i'm not sure the graphite endpoint is the best way to go. It might be easier to expose the metrics to URL, something that could be scraped. Would avoid the setup issues and the errors in the logs when graphite isn't there. Closing.


Release/r86

Hey @chuwy - this is the branch for the next release but it's in a bit of a mess (mostly self-inflicted by me):

Somehow I can't find this commit in this branch, #e678d299fc7402a1a77d31bd5d94d8597c816653
Somehow I can't find this commit in this branch, #c3bf6f1441bbf99d372eba7984917dc5748589e9
This issue #2983 should also be reverting this commit, #22c60598b6d28ffb2fadaab1eef4760d3557101f

I suspect I did some kind of force pulling or somesuch - apologies. Hope you have a clean local copy.
Could you please get this prepped for release as a priority Monday? You will also need to cherry-pick over all of the data modeling tickets, which are moving up into this release:

https://github.com/snowplow/snowplow/issues/2956
https://github.com/snowplow/snowplow/issues/2954
https://github.com/snowplow/snowplow/issues/2929
https://github.com/snowplow/snowplow/issues/2925
https://github.com/snowplow/snowplow/issues/2836
https://github.com/snowplow/snowplow/issues/2626

Also do please add the CHANGELOG in a final "Prepared for release" commit.
And please delete branches as/when they don't contain anything useful...
Thanks! Please let me know when this release branch is ready to go...

@alexanderdean it seems you pushed branch feature/dedupe which was removed github. Actual ready-for-release branch is release/dedupe.

I cherry-picked data-modeling commits. All except #2929, because there's no commit yet.
/cc @bogaert

@alexanderdean I need to force-push this branch now with ready release/dedupe. Did you change/add anything except #2963 and #2985?

Thanks for sorting all this out @chuwy - my commits are #2963, #2985 and #2986.

Ok, cool. Force-pushing!

Hey @chuwy - can you liaise with @dmpacheco to get his avatar on #9c53ec3?

@alexanderdean done!

Travis failed because OpenWeatherMap didn't respond at time. Hope this is just short outage.

Rebased with missed data-modelling commits.
/cc @bogaert

Hey @alexanderdean! Can I rebase it one more time? I need to delete event-recovery-ci.bash as part of #2983.

Sure - go for it!


feat(docker): Create Docker Compose file for local setup WITH Postgres & LogTo [DEV-2682]

[x] Create Docker Compose file for local setup WITH Postgres & LogTo.

[x] Create credential service in docker compose file.
[x] Create LogTo service in docker compose file.
[x] Create Postgres DB service in docker compose file.
[x] Create documentation about how to local up credential-service with docker compose.

Task linked: DEV-2682 Create Docker Compose file for local setup WITH Postgres & LogTo

Task linked: DEV-2681 Credential Service: Create Docker Compose file for local setup WITHOUT using Postgres/LogTo

@abdulla-ashurov would be good to add inline env variables in docker compose. As we have separate docker comopose files, it will make it clearer for a user to configure it.
We can create a separate folder for the docker compose without logto. e.g. custodianmode & private.

@abdulla-ashurov would be good to add inline env variables in docker compose. As we have separate docker comopose files, it will make it clearer for a user to configure it. E.g. pointing the external_db_connection_url by default to the one in the docker network. So that user doesn't have to enter everything
We can create a separate folder for the docker compose without logto. e.g. custodianmode & private.

Done! ✅


[IMPROVEMENT] Check NFS versions in /etc/nfsmount.conf instead

Is your improvement request related to a feature? Please describe (👍 if you like this request)
https://longhorn.io/docs/1.5.4/deploy/install/#installing-nfsv4-client
only checks that the kernel is able to use nfs4, NOT that it is actually in use!
The documentation on that page should be updated to check (at laest) /etc/nfsmount.conf instead of just checking the kernel.

Describe the solution you'd like

[ ] Update the official document
[ ] Update the longhorn-manager node condition check  @mantissahz
[ ] Update the longhorn cli cc @mantissahz

Describe alternatives you've considered

Additional context

PR

longhorn/cli#129

We also need to ensure this works on a Talos Linux cluster. cc @longhorn/qa

PR for node controller: longhorn/longhorn-manager/#3335

PR for node controller: https://github.com/longhorn/longhorn-manager/pull/3335

@COLDTURNIP could you update the PR and necessary information like reproduce steps on the Pre Ready-For-Testing Checklist?

Test Steps

Step 1. a Longhorn ready cluster, ensure the following changes are applied:

latest char (longhorn/longhorn#9910)
the longhornctl is updated (longhorn/cli#129)
the longhorn-manager is updated (longhorn/longhorn-manager#3335)
Ensure longhornctl check preflight all green.

Step 2. enter the shell of each longhorn-manager daemonset containers. The host's /etc should be mounted at container's /host/etc
Step 3. backup host's /host/etc/nfsmount.conf if present
Step 4. create/modify host's /host/etc/nfsmount.conf, change the Defaultvers to specific version (see below)
Step 5. check the nodes using longhornctl check preflight. If the Defaultvers is not acceptable, a warning item will be attached: NFS4 is supported, but default protocol version is not 4, 4.1, or 4.2. Please refer to the NFS mount configuration manual page for more information: man 5 nfsmount.conf
Step 6. wait for Longhorn node update, and check Longhorn node for given host (kubectl describe -n longhorn-system lhn tested-host-name). For NFSClientInstalled item In the Status.Conditions list, if the Defaultvers is not acceptable, it will indicate that NFS is misconfigured:

Status: false
Message: NFS clients default protocol version is 4.3, which is not supported
Reason: NFSClientIsMisconfigured

Step 7. change Defaultvers to another version, and back to Step 5.
Step 8. After test, restore the /host/etc/nfsmount.conf which is backup at Step 3.

A minimal /host/etc/nfsmount.conf sample:
[ NFSMount_Global_Options ]
Defaultvers=4

The acceptable Defaultvers:

Defaultvers=4 -> acceptable
Defaultvers=4.0 -> acceptable
Defaultvers=4.1 -> acceptable
Defaultvers=4.2 -> acceptable
Defaultvers=4.3 -> NOT acceptable
Defaultvers=5 -> NOT acceptable
Defaultvers=3 -> NOT acceptable
any other Defaultvers are NOT acceptable
/host/etc/nfsmount.conf absent -> acceptable
/host/etc/nfsmount.conf present without Defaultvers definded -> acceptable

/host/etc/nfsmount.conf present and Defaultvers is commented (e.g. # Defaultvers=4.3) -> acceptable

@COLDTURNIP Please move it to Ready For Testing if all PRs are ready and merged. Thank you.

Verified on v1.8.0-dev-20241215 20241212

longhorn v1.8.0-dev-20241215 https://github.com/longhorn/longhorn/commit/7cd5936abde0f177d9ae85492ec60aab0f115181
cli v1.8.0-dev-20241215 https://github.com/longhorn/cli/commit/0f3067177c4081e914a3e067611829a392f95d87

The test steps
https://github.com/longhorn/longhorn/issues/9830#issuecomment-2530704625
Result Passed

[x] lobghorn-cli can successfully detect the host's /etc/nfsmount.conf.
[x] longhorn-manager can verify the NFS versions in /etc/nfsmount.conf.


++[OFFICIAL/TV]* — England v All Blacks LiVe 𝐒𝐭𝐫𝐞𝐚𝐦𝐬 FrEe BrOaDcAsT oN TV Guide & 𝐒𝐭𝐫𝐞𝐚𝐦𝐢𝐧𝐠 01 November 2024 #1150

The All Blacks and England are set to face off at Twickenham Stadium on Saturday, November 2, 2024. This match is part of the Autumn Internationals and will also decide the winner of the Hillary Shield. The All Blacks are the current holders of the Shield, having won the last two Test series against England in New Zealand in July.
⏯⏯ CLICK HERE ⏯⏯==►► England Vs New Zealand All Blacks
⏯⏯ CLICK HERE ⏯⏯==►► England Vs New Zealand All Blacks
It’s going to be an exciting match! Are you planning to watch it?
England vs All Blacks Live Stream — Click Here
All Blacks vs England: A Legendary Showdown
The rugby world is set for a monumental clash as the All Blacks take on England at Twickenham on November 2nd. This match is more than just a game; it’s a battle steeped in history and rivalry.
England’s Home Advantage
England, under the guidance of head coach Steve Borthwick, is looking to leverage their home advantage. The squad has been meticulously preparing, focusing on strategic plays and strengthening their defense. Key players like Owen Farrell and Maro Itoje are expected to lead the charge, bringing both experience and tenacity to the field.
The All Blacks’ Might
The All Blacks, renowned for their relentless and aggressive style, come into this match with a string of impressive victories. Their recent performances have solidified their reputation as one of the most formidable teams in rugby. Players like Beauden Barrett and Ardie Savea will be crucial in maintaining their momentum and executing the tactical brilliance that the All Blacks are known for.
Key Matchups to Watch
Fly-Half Faceoff: The duel between England’s Marcus Smith and New Zealand’s Beauden Barrett promises to be electric. Both players are playmakers and their decisions will be pivotal.
Forward Clash: The showdown between England’s Maro Itoje and New Zealand’s Sam Whitelock will be a test of strength and strategy in the lineouts and scrums.
Backline Battle: With England’s Henry Arundell and New Zealand’s Rieko Ioane on the wings, expect some high-paced, thrilling runs and potential game-changing tries.
Historic Context
The rivalry between these two giants of rugby dates back to 1905. Each encounter is a blend of strategic depth, physicality, and sheer willpower. Matches between England and the All Blacks are often seen as a measure of the highest standards in world rugby.
Fans and Atmosphere
The atmosphere at Twickenham is set to be electric. Rugby enthusiasts from all over the globe will be tuning in, adding to the pressure and excitement. The roar of the crowd, the strategic plays, and the raw emotion of the game are what make encounters like these truly unforgettable.
As the date approaches, fans are eagerly anticipating what promises to be a thrilling contest. Will England defend their home turf, or will the All Blacks continue their dominance?


Implement DateTime fields support

Good. Anything else to expect?) I just fixed the problems with the nugget :)

:x: Build DaData.ApiClient 1.1.37 failed (commit https://github.com/Xambey/DaData.ApiClient/commit/8cb76a1c98 by @amphasis)

:x: Build DaData.ApiClient 1.1.37 failed (commit https://github.com/Xambey/DaData.ApiClient/commit/8cb76a1c98 by @amphasis)


Windows compatibility

These changes should allow the script to run on windows. The only caveat is that I had to hack the logger so that it sends all logs to stdout by default on windows. Apparently you can't have multiple file handles opened to a single file on that platform.

Coverage decreased (-2.004%) to 83.893% when pulling 6bf4bb4e0175f4ef244c1c6576d9229d89b1ffe1 on windows_compatibility into 65a7d6640c1cb933153e87eda4d4b60535688b77 on kyrylo/update_appveyor.

Coverage decreased (-1.9%) to 84.041% when pulling cdc91038be32d1c21d0b840d6be149e89a21be04 on windows_compatibility into 65a7d6640c1cb933153e87eda4d4b60535688b77 on kyrylo/update_appveyor.