any reason blocking non-BigQuery (or TEXT) result handler in config_manager.py

In config_manager.py line 467 (copied below), any reason non-BigQuery (or TEXT) result handler is blocked?
        else:
            raise ValueError(f"Unknown ResultHandler Class: {result_type}")

Even though we can still use --format <type> for json, csv from the command line w/o using a configuration file, it does block these text result output when using config yaml file if the config file has:
result_handler:
  type: json

Changing the above lines like below unblocked it as I tested it:
        else:
            #raise ValueError(f"Unknown ResultHandler Class: {result_type}")
            return TextResultHandler(result_type)

Hi @nj1973! Since you just worked on the result handler code for #1240, could you please take a look at this issue?  It might be something we can incorporate in the same PR


Runtime out of memory error

C:\repos\onnxruntime\onnxruntime\core\providers\cuda\cuda_call.cc:97 onnxruntime:cudacall c:\repose\onnxruntime\onnxruntime\core\providers\cuda\cuda_call.cc:91 onnxruntime::cudacall CUDS failure 2: out of memory ; GPU=9; hostname=<machinename> ; expr=cudamalloc((void**)&p, size);

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

ID: 3b745a96-f55d-5924-0f29-614186393b6d
Version Independent ID: 6e583bdd-485a-98f6-599b-503d2d4f5480
Content: Quickstart - Set up Azure Kinect body tracking
Content Source: articles/kinect-dk/body-sdk-setup.md
Product: kinect-dk
GitHub Login: @qm13
Microsoft Alias: quentinm

Thanks for the question. We are currently investigating and will update you shortly.

@Shri-Humrudha were you able to overcome this issue?

@Shri-Humrudha Can you confirm that your PC meets the Body Tracking requirements
https://docs.microsoft.com/en-us/azure/Kinect-dk/system-requirements#body-tracking-host-pc-hardware-requirements

Would you please open an issue in the Azure Kinect repository? https://github.com/microsoft/Azure-Kinect-Sensor-SDK/issues this way the engineering team can address this and also track the issue.

@Shri-Humrudha Closing this issue since this is not a doc issue and I haven't heard back from you.
If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.

@Alberto-Vega-MSFT I verified my system requirements. I followed this tutorial. As suggested I will open an issue with Azure Kinect repository.

@Shri-Humrudha Thank you for that. Sharing the link here for people coming across the same issue: https://github.com/microsoft/Azure-Kinect-Sensor-SDK/issues/576


New Rule: requireDefaultParameters (ES6)

airbnb/javascript#7.7

Use default parameter syntax rather than mutating function arguments.

// really bad
function handleThings(opts) {
  // No! We shouldn't mutate function arguments.
  // Double bad: if opts is falsy it'll be set to an object which may
  // be what you want but it can introduce subtle bugs.
  opts = opts || {};
  // ...
}

// still bad
function handleThings(opts) {
  if (opts === void 0) {
    opts = {};
  }
  // ...
}

// good
function handleThings(opts = {}) {
  // ...
}

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/15587668-new-rule-requiredefaultparameters-es6?utm_campaign=plugin&utm_content=tracker%2F281640&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F281640&utm_medium=issues&utm_source=github).

We released 3.0 recently! We also wrote a post about how 3.0 will be our last major release (since we are deprecating the project) and that we are merging with ESLint!
Thus we won't be working on any new features moving forward which includes new rules/options. We will continue to fix bugs and work on CST/better autofixing for current rules in the upcoming months both to help JSCS users and to gain experience that we can hopefully use in ESLint. But otherwise we both teams will be working on getting ESLint feature parity with JSCS and just a nice transition for our users.
I'm putting an orphaned label to note this issue isn't planned to be worked on, but might be a good contribution as a custom rule (not in JSCS core itself).
I would encourage anyone who wants the rule to try writing it themselves: I would either write a custom rule or submit an issue to ESLint and if accepted a PR. Otherwise a custom rule that won't be in core repo itself.
JSCS rules are also very similar to ESLint rules: here are some resources to get started:

Project-specific Lint Rules with ESLint
Creating Custom Rules for ESLint
Maintaining JavaScript Code Quality with ESLint


Change how access properties in get_scope

PR closes #1878
What is being addressed
Change how properties are referenced in get_scope.

/test-force-approve

/test-force-approve


ultrawide Fix for the 1.5 update

Based on https://github.com/techiew/EldenRingMods/commit/6ec583094b73cad83d0a9104cdf37e5adb792715
Tested locally and it works.

Can't get this patch to work on my machine, black bars appear as before. 1.04 was fine.  Will update when I can do some debugging.

Not working for me neither


Add .idea config

The Problem/Issue/Bug
Fixes #22
How this PR Solves The Problem
Adds initial .idea
Manual Testing Instructions
Run a session, open PhpStorm. Run the drush status runner.
Related Issue Link(s)
Release/Deployment notes

We should add an entry to git-exclude.template for .idea so general users do not see VCS changes on .idea in case something modifies on launch


v 2.1.1 fixes missing implementation of devicePixelRatio

tested in example app.

Fixed by https://github.com/optimist-dev/scrolls_to_top/pull/12


feat: add views for completion events

Description
This PR creates MV an a table for completion events
Testing instructions

In a tutor dev|local environment install edx-completion from the master branch.
Import the demo course tutor dev to importdemocourse
Run tutor config save
Apply the migrations: tutor dev do alembic -c "upgrade 0021"
Complete some components of the course
Verify the events are stored in the MV

Thanks for the pull request, @Ian2012!  Please note that it may take us up to several weeks or months to complete a review and merge your PR.
Feel free to add as much of the following information to the ticket as you can:

supporting documentation
Open edX discussion forum threads
timeline information ("this must be merged by XX date", and why that is)
partner information ("this is a course on edx.org")
any other information that can help Product understand the context for the PR

All technical communication about the code itself will be done via the GitHub pull request interface. As a reminder, our process documentation is here.
Please let us know once your PR is ready for our review and all tests are green.

@Ian2012 🎉 Your pull request was merged!  Please take a moment  to answer a two question survey so we can improve your experience in the future.


Density Function Fixes and Improvements

a few fixes and improvements for the density functions. I'll explain each change separately in comments.

Changes all look good! Thanks a lot for spotting all these issues!


rewrite middleware not identifying multiple slashes

From @parmarnishant on February 10, 2017 6:16
Problem : Redirect to url with single slash if url has multiple slashes.
http://localhost/segment1///segment2//segment3
The above url when read removes multiple slashes and shows only one slash.
I tried following methods to read url.
request.Path
Microsoft.AspNetCore.Http.Extensions.UriHelper.GetEncodedUri (request)
Microsoft.AspNetCore.Http.Extensions.UriHelper.GetDisplayUri(request)
Thanks.
Copied from original issue: aspnet/Mvc#5779

@mikaelm12 does some lower-level component do some canonicalization before we even get to it?
cc @blowdart in case there are some security concerns here about who should canonicalize what.

There ought to be a raw URL that rewrite works on, but generally this should be normalised in the Request classes that everything else uses.

I think it's IIS that removes these slashes. Then it wouldn't even show up in the original url (as received by Kestrel). @mikaelm12 to verify.

@Tratcher yeah that's what I was thinking.

@mikaelm12 could you take a look?

IIS is removing the slashes. So it's not that the middleware isn't identifying multiple slashes, the url it gets doesn't have them. You shouldn't see this if you don't run behind IIS.


Feat: Logo is not properly visible

As I am going through the website, I found that the logo of KrishiConnect is not properly visible as it is very small in size.
We can make it a bit bigger in size and a little more vibrant so that it can impact the users more.
I would like to work on this issue. Kindly allow me by assigning it under gssoc'24 label.

Fixed in #242


Add security to Turnstile

Add to Turnstile a demonstration of how to use the CPS overlay.

See example in TurnstileSecurity.sadl. Picture below shows how this example uses the core ontology and our Cps.sadl overlay.


feat: add popover

Description
Zeplin: https://zpl.io/g8Z8zK9, https://zpl.io/Z0GvyZw
Fixes https://github.com/appsmithorg/appsmith/issues/25567
Slack threads: https://theappsmith.slack.com/archives/C0293DVQACW/p1689113798773239, https://theappsmith.slack.com/archives/C0293DVQACW/p1690382932770629
Type of change

New feature (non-breaking change which adds functionality)
This change requires a documentation update

How Has This Been Tested?

Manual on storybook
Manual on main repo
Jest
Cypress

Test Plan

Add Testsmith test cases links that relate to this PR

Issues raised during DP testing

Link issues raised during DP testing for better visiblity and tracking (copy link from comments dropped on this PR)

Checklist:
Dev activity

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

QA activity:

[ ] Test plan has been approved by relevant developers
[ ] Test plan has been peer reviewed by QA
[ ] Cypress test cases have been added and approved by either SDET or manual QA
[ ] Organized project review call with relevant stakeholders after Round 1/2 of QA
[ ] Added Test Plan Approved label after reveiwing all Cypress test

/generate-alpha

/generate-alpha

/generate-alpha

Size selector in storybook is a text field. Please update this to a select.

QA done for Popover for the Font color, Icon color, Spacing Px, Icon Px, for both Small and Medium sizes, Controls of Header. Working as expected CC: @tanvibhakta

/generate-alpha

Design and storybook headers look different

@albinAppsmith

Design and storybook headers look different
We don't typically add the font weights etc when developing components because Text hasn't been developed, but nonetheless I have added it as a comment here already

@albinAppsmith

Design and storybook headers look different

We don't typically add the font weights etc when developing components because Text hasn't been developed, but nonetheless I have added it as a comment here already

No but this is not right. When designers use popover component and give it to developers for developing, they will get confused seeing this difference in UI. Please update the UI and then do QA on the same.

/generate-alpha

/generate-alpha

/generate-alpha

alpha: 2.1.19-alpha.7


Can't remove an image once added to specializations

Describe the bug
So I saw that you could add an image to specializations. So I added a picture then realized I would rather just have the default image only I could see no way to remove it. I could swap it out for another but no way to take it back to default.
To Reproduce
Add an image to a specialization then try to return it back to default image.
Expected behavior
An option to revert back to it's default image.
Desktop (please complete the following information):
Foundry Version: v12
Star Wars FFG: 1.902
Hosted on Forge: No

You can set it back to the default of icons/svg/item-bag.svg the same way you changed it.


internal: Remove SemanticScope::speculative_resolve_as_mac

Fixes https://github.com/rust-analyzer/rust-analyzer/issues/11132

Hm this is not quite right actually

Hm this is not quite right actually


[Mention The Feature]: Improvement of the  CSS styling of the scholarships section

Is your feature request related to a problem? Please describe.
I would like to work on the Improvement of the  CSS styling of the scholarships section, as we cant view it properly

Describe the solution you'd like.
No response
Describe alternatives you've considered.
No response
Additional context.
No response
Show us the magic with screenshots
No response
Checklist

[X] I have checked the Existing Issues
[X] I have read the Contributing Guidelines
[X] I want to work on this issue. (optional)

@aditya-bhaumik please assign me under this issue !!!


Email configuration

i create a account in MAILGUN and test it in my server using swaks command and it works !
But i input the information in application.yml file and it does not work. could anybody help me ??
This command works:
./swaks --auth
--server smtp.mailgun.org:587
--au<EMAIL_ADDRESS>--ap ?????????????????????
--to<EMAIL_ADDRESS>--h-Subject: "Hello"
--body "Testing some Mailgun awesomness!"
Configuration on application.yml:
SMTP_PORT: 587
SMTP_DOMAIN: sandboxfdbda7d5b99841dfb4c7000b94cae911.mailgun.org
SMTP_ADDRESS: smtp.mailgun.org
SMTP_USERNAME<EMAIL_ADDRESS>SMTP_PASSWORD: xxxxxxxxxxxxxx

All is working Well now. I deploy production environment and review application.yml and it works correctly.

Did you make any changes to the settings for it to work?


Custom easing was not properly called

The SimpleSlider.defaultEase was calles no matter what was set on options.ease.

Cool! You are welcome :-) long live OSS.

Thank you very much for providing this, loved that you also added a test for it!
Already merged and made a v0.5.1 release with it!
—
Reply to this email directly or view it on GitHub.


Avoid use-after-free errors in the Python API

When calling in the C API from Python we are allocating Tiling objects
from Python. The lifetime of the corresponding memory is linked to the
lifetime of the corresponding Python object. We were using temporaries
for the Python object, which goes out of scope and deallocate the memory
as soon as they are passed to the C API function. Instead, we now use
explicit variables which live until after the C API call has returned.

bors r+


improve batch scanning

Signed-off-by: hirokuni-kitahara<EMAIL_ADDRESS>
improve batch scanning

close this and have submitted another PR


Channel header with members & topic

Adds some more info to the AppBar from a channels.info call:

There are a few issues with this right now:

In store.fetchInitialChannelList, we call setDirect on an individual channel with the result from users.counts. The problem is if you try to listen to that Updatable, you'll only ever get the small users.counts result. So in its current state we have to invalidate the current result to make it use its factory method.
Shuffling properties from one VM to another will cause issues if you try to use that property in the destination view-model. when throws on properties that don't yet exist. We're working around it by passing the view-model that has the info (in this case, the ChannelList) into the one that doesn't (the ChannelHeader).

Going to rework this with Paul's Store changes and fix the first issue.


Three column front-end most recent readings

Similar to the implementation of front-end that Nic showed us, we need three blocks that display attributes of a sensor such as: most recent reading, ts of reading, health of sensor.

In progress - waiting for implementation of bits-settings

To show current readings we need to add bits-settings. I have an experimental implementation that is currently breaking and is pushed to ui-testing branch.


test: Check whether alertmanager mesh network initializes correctly

Spin up three alertmanagers via prometheus operator and check with the
/status API endpoint if they discover each other correctly.
❗ ❗ ❗  Depends on alertmanager PR 644

Great! There is actually a container image built from master for the Alertmanager with the master tag.
https://quay.io/repository/prometheus/alertmanager?tag=latest&tab=tags
Since the PR is merged and it was already built you can just set the test to use the master version, for the time being.

@brancz Updated PR to use alertmanager:master. Let's see what the CI says. Runs green locally.
Kept changes in separate commit so we can revert it as soon as meshStatus information is present in  alertmanager api v1.

@brancz Could you re-review?

Good stuff! :+1: lgtm


[4.0.1] lastAbsoluteLocation in request closure

I'm implementing now version 4.0.1 and replacing the old one.
Now there is a field in the locationRequest named "lastAbsoluteLocation". I want to access this last location, when my locationRequest failed (for example because of a timeout). The problem is, I can't access it in the request closure, only over the locationRequest itself.
Is it possible to add this lastLocation to the ErrorReason
LocationManager.shared.locateFromGPS(.oneShot,
       accuracy: .house,
       timeout: Timeout.Mode.delayed(30.0)
) { (result: LocationRequest.Data) in
             switch result  {
                  case .success(let location):
                       print("GET Location Success: \(location)")
                       // doing something with location
                   case .failure(let error):
                       print("GET Location Error: \(error)")
                       // I want to accèss lastAbsoluteLocation here
              }
 }

Due to the the structure of the Result type in swift we cannot pass the instance of the request as error; we could pass it along with one of the available error enums but this mean we need to have different sets of the errors for each type of request which is not best solution.
Since 4.0.2 LocationRequest produces a new error when timeout is reached without a valid location received; this is called requiredLocationNotFound and it includes the timeout interval and the last absolute location found (request's lastAbsoluteLocation property value).
So your code will be something like this:
LocationManager.shared.locateFromGPS(.oneShot,
       accuracy: .house,
       timeout: Timeout.Mode.delayed(30.0)
) { (result: LocationRequest.Data) in
             switch result  {
                  case .success(let location):
                       print("GET Location Success: \(location)")
                       // doing something with location
                   case .failure(let error):
                      switch error: {
                         case . requiredLocationNotFound(let interval, let lastLocation):
                            // do something
                         default:
                            break
                       }
                       print("GET Location Error: \(error)")
                       // I want to accèss lastAbsoluteLocation here
              }
 }

Thanks for your report.


Unit Tests

Very much a work in progress.
Status:

[ ] Models/Api/Server.cs (in progress)
[ ] Models/Api/ServerList.cs (in progress)
[ ] Models/ConfigFile/Config.cs (in progress)
[ ] Models/ConfigFile/HubClientConfig.cs (in progress)
[ ] Models/ConfigFile/Preferences.cs (in progress)
[ ] Models/Download.cs (in progress)
[x] Models/Installation.cs
[ ] Services/AuthService.cs (in progress)
[ ] Services/DownloadService.cs
[ ] Services/InstallationService.cs
[ ] Services/ServerService.cs
[ ] Services/StateService.cs
[x] ValueConverters/HumanizerValueConverter.cs

Going to close this for now, will continue on it after the 931 release. Will probably split up the tests into separate PRs to reduce the code review burden. Plan is to start with the services (skip AuthService as it is going to be removed) since those are fairly self contained.


Allow defining round robin with teams instead of components

We have a team mapping in Bugzilla, we should reuse that instead of defining lists of components.

We could allow both options, either a list of components or a team name. This way teams that have different people working on different things can still define two separate round robin calendars, and teams working on similar things can have a simpler definition file.
The fallback can be the team manager, if a fallback is not defined in the round_robin.json file.
This way a simple round_robin definition file would just be:
{
    "team": "TEAM NAME",
    "default": {
      "calendar": "URL TO CALENDAR"
    }
}


data_publication field

While working on 608, I noticed that the data_publication field generated automatically for this study in G0 says Yes, although the registration says No. Could you verify the field/what might have happened, please?

@gufranpathan fyi

Fixed in commit 7aeda00.


[SPIKE] As an EVCM, I want to make sure cumulative index scores for US territories are shown on the map

Description
Spike to ensure the map and indicator data can be visualized for the following five US territories that are part of the US Census

Puerto Rico
American Samoa
Commonwealth of the Northern Mariana Islands
Guam
U.S. Virgin Islands

Solution
TBD
Describe alternatives you've considered
TBD
Links to user research or other resources
Tasks

[ ] Ensure map tiles can be shown for the five territories listed
[ ] Ensure datasets can be visualized for Puerto Rico
[ ] Ensure datasets can be visualized for American Samoa
[ ] Ensure datasets can be visualized for Commonwealth of the Northern Mariana Islands
[ ] Ensure datasets can be visualized for Guam
[ ] Ensure datasets can be visualized for U.S. Virgin Islands
[ ] Understand what our policy or decision rule could be if only a subset of the datasets we use to create cumulative indices for the state CBGs is available

Fleshed out github issue for this feature based on the output of discovery


Select + Union + Limit + Order

Hello,
I should notice the following.
$select1 = $sql->select('table')->limit(5)
$select2 = $sql->select('table')->limit(15)
$select1->combine($select2).

That code gives me incorrect Sql query (15 with quotes):
( SELECT `table`.* FROM `table` LIMIT 5 ) UNION ( SELECT `table`.* FROM `table` LIMIT '15' )

Thank you!

Shouldn't we add an exclusion when the value is a limit ?
We could do this by passing an additional parameter indicating the clause to Zend\Db\Adapter\Platform\Mysql::QuoteValue. But I think it is poor design. Another idea ?
Unless, I misunderstood the normal use of Zend\Db component or the problem ...

I don't think that the MySQL adapter should be quoting values for the Select::limit method at all. The query SELECT * FROM table LIMIT '10'; will always fail with a syntax error, at least on my environment (MySQL 5.6.21).
However, the limit method always seems to quote the value for me:
$select = $this->tableGateway->getSql()->select()->limit(10);
echo $select->getSqlString($this->tableGateway->getAdapter()->getPlatform());
//=> SELECT * FROM `table` LIMIT '10'

This issue has been closed as part of the bug migration program as outlined here - http://framework.zend.com/blog/2016-04-11-issue-closures.html


TAJO-1825: Remove zero length fragments when file length is zero.

+1 thank you for the nice work!


[WIP][GSB] Clean up ArchetypeResolutionKind et al

WIP patch to clean up our handling of ArchetypeResolutionKind, which is on the road to eliminating unresolved potential archetypes. Still failing one test (test/Constraints/same_types.swift) because we're not building some nested types that we do, in fact, need to build

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please smoke test

Trivial (independent) repro for the failure on Linux:
func foo(c: LazyFilterCollection<[Int]>) {
  for x in c { }
}

because some types aren't matching when we deserialize.

@swift-ci please smoke test and merge

@swift-ci please smoke test and merge

All of this got merged already


Fix for #68: Updated coloring and added visual cues in rule descriptions

Fix for #68

As we have discussed it is not as easy using CSS here. So the PR is fine to be merged as it resolves the contrast issue.
For now ready to be merged and go into the release...
As a nice-to-have we can later ask in SonarQube forum for help how CSS could be used or try some own solution e.g. via generation by processing the HTML with maven during the build...


Disconnect device properly if controller manager is shutdown via Ctrl + C

Currently shutting down controller manager via Ctrl+C won't call on_deactivate() method. The fd device will still operate with force enabled after killing the program. Although it's okay to manually deactivate force using the button the device, it's convenient to have the device automatically closed.
The workaround is adapted from https://github.com/UniversalRobots/Universal_Robots_ROS2_Driver/blob/e5ba31cd38c34076f95b25a047daaa16f7c3466c/ur_robot_driver/src/hardware_interface.cpp#L58C1-L63C2
Ideally this should be handled in ros2_control but there's still no update on their side, see https://github.com/ros-controls/ros2_control/issues/472.

Thanks, that's a handy workaround :)


[Feature] Vim support

Currently only supports Neovim.  Not good :(

Only supporting Neovim.  For now at least.


Add double quotes to the -script script filename

Reported by karl smith on 2 Jan 2013 23:12 UTC
From Andrew:
There's also a minor bug in how the argument of "-script" is passed into Ferret -- it ought to be inserted inside of double-quotes, otherwise the "go" command will complain due to its misinterpreting directory slashes as qualifiers:

$FER_DIR/bin/pyferret -unmapped -script /home/atw/mytest/ferret/pyferret/ps_multi.jnl

 	NOAA/PMEL TMAP
 	FERRET v6.83 (PyFerret 1.0.0)
 	Linux 2.6.32-279.5.2.el6.x86_64 - 10/19/12
 	28-Dec-12 19:58

 **ERROR: unknown command qualifier: home
go /home/atw/mytest/ferret/pyferret/ps_multi.jnl
Command file, command group, or REPEAT execution aborted

Migrated-From: http://dunkel.pmel.noaa.gov/trac/ferret/ticket/2020

Comment by karl.smith on 2 Jan 2013 23:25 UTC
Double quotes added around the script filename in the Python source that makes the go call.  No quotes are added around any additional arguments.
If the ferret script being run expects a filename argument, and the user wants to give a path in that argument, the user will need to add two pairs of escaped double quotes around that argument on the shell command line - exactly the same as with ferret.


Feature request: run tailscale logout automatically at the end of a job

It would be nice to run tailscale logout after the job, since even ephemeral machines stick around in the admin UI for a day or two.  Unfortunately composite actions don't support "post-job steps," but JavaScript actions do: https://docs.github.com/en/actions/creating-actions/metadata-syntax-for-github-actions#post.

Though ephemeral nodes initially were deleted 48 hours after becoming inactive, a few weeks ago we reduced it to one hour before deletion. Is that sufficient for this?

That's probably fine.  I made this issue after seeing https://twitter.com/apenwarr/status/1394794106417733634, before I realized that tailscale logout doesn't actually hide the machine from the panel anyway.


Ped Delay Summary Report

Issue Addressed
This PR closes #970 .
Description
We implement ReportPedDelaySummary, and make several changes across the data-handling parts of the MOVE web application to accommodate the new counts2.studies schema.
Tests
Updated tests to match changes, npm run ci:jest-coverage to ensure all tests still pass.  Tested report manually in frontend.  Downloaded PDF version to ensure that it fits within the page.

LGTM


[ISSUE #6236] change ThreadLocalIndex.threadLocalIndex.Integer to AtomicInteger

…micInteger
Make sure set the target branch to develop
What is the purpose of the change

fix #[ISSUE #6236] change ThreadLocalIndex.threadLocalIndex.Integer to AtomicInteger
Brief changelog
XX
Verifying this change
XXXX
Follow this checklist to help us incorporate your contribution quickly and easily. Notice, it would be helpful if you could finish the following 5 checklist(the last one is not necessary)before request the community to review your PR.

[x] Make sure there is a Github issue filed for the change (usually before you start working on it). Trivial changes like typos do not require a Github issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[x] Format the pull request title like [ISSUE #123] Fix UnknownException when host config not exist. Each commit in the pull request should have a meaningful subject line and body.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Write necessary unit-test(over 80% coverage) to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[x] Run mvn -B clean apache-rat:check findbugs:findbugs checkstyle:checkstyle to make sure basic checks pass. Run mvn clean install -DskipITs to make sure unit-test pass. Run mvn clean test-compile failsafe:integration-test  to make sure integration-test pass.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Hi @yglovewms  I have submit pr to fix issue #6236 and give the benchmark report.

close because duplicated with https://github.com/apache/rocketmq/issues/6236


dart error

Analyzer Feedback from IntelliJ
Version information

IDEA AI-173.4907809
2.0.0-edge.0d5cf900b021bf5c9fa593ffa12b15bcd1cc5fe0
AI-173.4907809, JRE 1.8.0_152-release-1024-b02x64 JetBrains s.r.o, OS Windows 10(amd64) v10.0 , screens 1366x768

Exception
Please append the contents of:
file://C:\Users\bshaik\AppData\Local\Temp\1\report.txt
Dart analysis server, SDK version 2.0.0-edge.0d5cf900b021bf5c9fa593ffa12b15bcd1cc5fe0, server version 1.18.7, FATAL error: Failed to handle request: {id: 3, method: analysis.setAnalysisRoots, params: {included: [C:\Basha\Personal\WeServe\Flutter\flutter_baseapp-master], excluded: [C:\Basha\Personal\WeServe\Flutter\flutter_baseapp-master\.idea, C:\Basha\Personal\WeServe\Flutter\flutter_baseapp-master\.dart_tool, C:\Basha\Personal\WeServe\Flutter\flutter_baseapp-master\.pub, C:\Basha\Personal\WeServe\Flutter\flutter_baseapp-master\build]}, clientRequestTime:<PHONE_NUMBER>412}
FileSystemException: Exists failed, path = 'C:\Basha\Personal\WeServe\Flutter\flutter_baseapp-master\build\image_picker\intermediates\classes\debug' (OS Error: Access is denied.
, errno = 5)
#0      _Directory.existsSync (dart:io/directory_impl.dart:73)
#1      _WindowsDirectoryWatcher._eventsBasedOnFileSystem (package:watcher/src/directory_watcher/windows.dart:323)
#2      _WindowsDirectoryWatcher._onBatch.<anonymous closure> (package:watcher/src/directory_watcher/windows.dart:166)
#3      _HashVMBase

dupe of https://github.com/dart-lang/sdk/issues/33280


illegal state in lorawan process stack crash - v4.5.0

stack v4.5.0, esp32 port (internal at the moment)
I'm seeing this crash 306s after startup:
INFO: smtc_modem_run_engineI (306875) smtc_modem_hal: disable_modem_irq
I (306875) smtc_modem_hal: enable_modem_irq
E (306875) smtc_modem_hal: crash_string lr1mac_core_process:203 FAILSAFE EVENT OCCUR (lr1mac_state:0x1)
E (306875) smtc_modem_hal: :556 Modem panic: lr1mac_core_process:203 FAILSAFE EVENT OCCUR (lr1mac_state:0x1)
E (306875) smtc_modem_hal: crash_string lr1mac_core_process:458 Illegal state in lorawan process
E (306875) smtc_modem_hal: :556 Modem panic: lr1mac_core_process:458 Illegal state in lorawan process

 Start a new join sequence now on stack 0
I (306875) LoRaWAN: Modem event callback
W (306875) LoRaWAN: Event received: JOINFAIL
I (306875) LoRaWAN: attempting to join again
INFO: smtc_modem_join_network

I don't believe it is related to stack overflow as I have stack checking enabled in esp-idf.
Thoughts on how to debug? I'm getting more familiar with the stack every day but still early in understanding it fully.
Also, I see radio IRQ during the porting tests (which pass), but for whatever reason not during my relatively simply use case of joining. Timer irqs are occurring and look to be working as expected.

Hi,
In a first step, could you please check and confirm you are using the same HAL for porting tests & your application ?
Thank you very much,
Best regards,


A "Before" middleware can stop the execution chain

Pull Request Test Coverage Report for Build 253

9 of 11 (81.82%)  changed or added relevant lines in 2 files are covered.
3 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-1.1%) to 94.966%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

router.go
7
9
77.78%

Files with Coverage Reduction
New Missed Lines
%

router.go
3
94.35%

Totals

Change from base Build 252:
-1.1%

Covered Lines:
415

Relevant Lines:
437

💛  - Coveralls


Pipfile.lock hashes back to determinitic order

The intent has been for hashes to be in a stable order to minimize diff
noise. Commit 1fa3e2a567da779b0c081ac28893aca3353c0674 caused a
regression here by converting to a set and back.
Fixes #1529
Fixes #1664

Thanks for the pr! One minor clarification and then I think we're good

Few changes:

Just use sorted to convert to list
Remove unnecessary sorted calls that were added earlier that no longer have an effect since we sort at the last second
Typofix: determinitic  -> deterministic in title

LGTM, waiting for CI to finish.

Grr, CI passed, but the report got stuck. Need admin to merge.

Waiting on CI, then merging


[scan] Add support for disabling "KeyboardContinuousPathEnabled"

Checklist

[x] I've run bundle exec rspec from the root directory to see all new and existing tests pass
[x] I've followed the fastlane code style and run bundle exec rubocop -a to ensure the code style is valid
[x] I've read the Contribution Guidelines
[x] I've updated the documentation if necessary.

Motivation and Context

With the release of iOS 13, newly recreated/reset simulators all have "Slide to type" enabled by default.
This new functionality now causes an issue when executing XCUITest via scan which interact with textfields. As they will see the following screen when interacting with the UITextField for the first time.

This will then cause the XCUITest to fail, due to the inability to interact with the keyboard.
Description

This change adds a new config option for scan,SCAN_DISABLE_SLIDE_TO_TYPE this is disabled by default, to revert back the keyboard behavior of a clean simulator.
Resolves: https://github.com/fastlane/fastlane/issues/15742
Testing Steps

Testing in the following project:
https://github.com/AlexTrott/Fastlane-iOS13-TextField-test
Test:
  desc "Test: Fastlane sets the default param, so the user has to do nothing"
  lane :test_default_param do
    scan(
      reset_simulator: true,
      disable_slide_to_type: true
    )
  end

  desc "Test: Fastlane responds when the user inputs the same selection as the default param"
  lane :test_setting_param do
    scan(
      reset_simulator: true,
      disable_slide_to_type: true
    )
  end

  desc "Test: Fastlane responds when the user sets the property to false"
  lane :test_which_will_fail do
    # This test will fail, as the 'Slide to Type' prompt is shown
    scan(
      reset_simulator: true,
      disable_slide_to_type: false
    )
  end

@googlebot I signed it!

@googlebot I consent.


Missing sample response

Validate Account Holder request has not corresponding sample response.
https://mtn-momo-api-documentation.firebaseapp.com/use-cases/#validate-account-holder
I'm wondering what the response could be if,

The account holder is registered and active
The account holder is not registered and, or is not active

The other documentation seems to be missing out this the 2nd response.

The other documentation is here: https://momodeveloper.mtn.com/api-documentation Go through it and let me know if it answers your query.

I managed to get the sample response after try and error.
But as you can see; the sample response is missing, hope that bit can be updated in the documentation.

https://momodeveloper.mtn.com/docs/services/collection/operations/get-v1_0-accountholder-accountholderidtype-accountholderid-active

Soon hopefully. Thank you for catching it.


Navigate to protest when clicking a marker on the map

When clicking on a marker on the map user will now be directed to the protest page

Had to use of title instead of <Popup> because it crashes the app with a nasty bug

Hi, thanks for that.
I think that a better UX will be using #64, and from there have an option to view the protest page.
What do you think?

whatever you decide @guytepper . let's wait for #64 to resolve and see

Ended up using #64


Line number migration

The idea of this PR is to make as a proof of concept for the decoupling of the editors components (line number, scrollbar... ) and make the editor easier to extend through the creation of new panels
I had move the linenumber methods to the LineNumberArea class, there's missing the creation of the Panel class (base class for editor components) and add some management to editors Class to automatize the  painting of panels.

there's missing the creation of the Panel class (base class for editor components) and add some management to editors Class to automatize the painting of panels.

Do you plan to add all of this in this PR?

I mean, I'd prefer if you only focus on moving the LineNumberArea widget out of CodeEditor :-)

I think we can make the Panel be the Last PR, before code folding? (or with code folding, since it will better adapt to that concept)

I was planing to add that changes too, but as @goanpeca says It will be better to let that for the last PR, after moving the others panels (scrollflagarea, edgeline, and gotoline).

:+:

@rlaverde, I agree with @goanpeca: please one panel by PR. If PRs are atomic, we can review and merge them quite quickly :-)

I don't understand what's happening with Travis. I'll fix it when I come back to Colombia :-)

@rlaverde, could you rebase or merge this with master? It seems you started this work before 3.0 was released, and that's making Travis to fail :-)

@rlaverde is this working fine and ready to be reviewed? or work is still missing?

@goanpeca yes, I think it's ready (because Panel class will be other PR)

@ccordoba12 or @Nodd or @jitseniesen could you review this one?

Will do tomorrow :-)

@goanpeca In BaseEditMixin there's get_linenumberarea_width() function, https://github.com/spyder-ide/spyder/blob/master/spyder/widgets/mixins.py#L46 that is reimplemented in CodeEditor.
It's ok to delete that function? and what should I do with that function call in https://github.com/spyder-ide/spyder/blob/master/spyder/widgets/mixins.py#L108? (change it for linenumberarea.get_width() will break others child classes because  It'll assume that they have a linenumberarea attribute)

@rlaverde since the mixin was made to have a common set of funcs for the editor and consoles then I guess we need to leave it there for backwards compatibility. So you need to leave tha method there, and in the case of the editor we need to overload it to do
def get_linenumberarea_width():
    return linenumberarea.get_width()

So no, dont delete those methods from mixin, we just override it on the code editor to get the correct behavior.

I left some minor comments, but this looks good to me :-)
Thanks @rlaverde!

Thanks a lot @rlaverde! Merging now :-)

WOOT :-)


Sandbox: run script/downloadReleases.js for 4.2.2

Sure, thanks ( these are automatically applied on deploys, so I don't normally do it unless it's a side-effect of doing something else but I appreciate the thought )


fix(columns): create source columns with the same name used by features

If a GPKG has a gpkg_data_columns table, the columns returned by getInfoForTable may have differing name and displayName fields. We were using the name field, but GeoJSON returned by iterateGeoJSONFeatures uses the displayName field for feature properties. This was breaking things like feature info, labels, etc that map source columns to feature properties.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Change doc-builder workflow container

doc-builder is failing with tensorflow dependency issues.
let's see if changing the worflow container fixes the issue

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

docbuild is working now


fix: excluded IPublishedElement from GetTypeForAllowedTypes

Excluded IPublishedElement from GetTypeForAllowedTypes to let the default behaviour use IPublishedContent, see more https://github.com/callumbwhyte/super-value-converters/issues/12
smartypants 😃

Code looks great! Almost like I wrote it...
This is now released as part of v2.2.1 (for Umbraco 8) and v4.0.1 (for Umbraco 9+)


[Storage] Support new api version for blob Service properties

Description

Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change.
[Component Name 2] az command b: Add some customer-facing feature.

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

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

Storage


Add support for raft repl dev replace member.

When replacing a member, add the new member, sync raft log for replace and finally remove the old member. Once we add new member, baseline or incremental resync will start. Remove the old member will cause nuraft mesg to exit the group and we periodically gc the destroyed group. Made the repl dev base test common so that both tests files can use. Tests by default create repl group with num_replica's. Dynamic tests create additional spare replica's which can be added to the test dynamically by calling replace member.

we may need add_member and remove_member, if we do expansion/shrink. Our use case is only replace and we want to update in the homeobject PG metablk in single write atomically. The member out will have its PG metablk with destroyed flag enabled. GC will use this delete all blobs for that PG.

The member out will have its PG metablk with destroyed flag enabled. GC will use this delete all blobs for that PG.

do we need gc for the member out? I think if we remove it from the group , we can just re-fromat it with homestore, then it will be  clean , no?

The member out will have its PG metablk with destroyed flag enabled. GC will use this delete all blobs for that PG.

do we need gc for the member out? I think if we remove it from the group , we can just re-fromat it with homestore, then it will be clean , no?

One SM could have multiple PG's on same disk. We could move out some PG for space issues or load balance.

the code generally looks good to me.  pls resolve the conflict

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 65.90909% with 15 lines in your changes missing coverage. Please review.

Project coverage is 67.33%. Comparing base (1a0cef8) to head (e98bb96).
Report is 64 commits behind head on master.

Files with missing lines
Patch %
Lines

src/lib/replication/repl_dev/raft_repl_dev.cpp
62.16%
13 Missing and 1 partial :warning:

src/lib/replication/repl_dev/common.cpp
0.00%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff             @@
##           master     #546       +/-   ##
===========================================
+ Coverage   56.51%   67.33%   +10.81%
===========================================
  Files         108      109        +1
  Lines       10300    10439      +139
  Branches     1402     1404        +2
===========================================
+ Hits         5821     7029     +1208
+ Misses       3894     2735     -1159
- Partials      585      675       +90

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


Show only direct connections

Hi,
Sorry for posting here stuff about the HA integration but I thought it'd be faster to reach you :)
Would it be feasible to patch the HA integration to configure a given connection to only show direct connections? This lib and the API seem to support it.
It does not seem complicated: maybe an extra user_input in config_flow which would enable the direct parameter in the OpendataTransport object. I don't have much experience touching the HA codebase but I can try.
What do you think?

Maybe I spoke too fast: https://github.com/OpendataCH/Transport/issues/216

So yeah, according to the reply in the ticket above the direct parameter (and many more) are not supported anymore.
There's still the chance to keep the parameter in the library and do the filtering at this stage to kind of simulate the missing functionality in the API. Would you be willing to review a patch to do so?

Ping @miaucl

Hey guys, as you might know, it takes some time to integrate into HA and as it was a old integration, the focus lies first on clean up before new features are considered. But yes, this is already on my radar 👍

Great, if I can help somehow here I am. I'm specially interested in being able to filter out non-direct connections, preferably at HA level. I've seen a patch of yours has been recently merged in HA to have separate entities per departure. If each departure_x had the transfers attribute it'd be easy to filter in templates, for instance.


Can't run now-cli on PC

I installed now on my windows 10 pc and after an update, it always say

OS: Windows 10
Node Version: v6.9.5

now is bundled from now-desktop for windows

Is your Windows version 32 bit or 64?

https://www.microsoft.com/en-us/download/details.aspx?id=11533
Also please run fciv.exe -both now.exe against now executable that causes that error message.

No answer for 15 days

I'm also getting the message This app can't run on your PC
Versions:
OS: Windows 10 pro 64 bit
Now: 1.2.3
Node: v6.9.5

Ran this: certutil -hashfile now.exe
SHA1 hash of file now.exe:
fb ff 64 3c 3e 46 70 a8 c7 d2 3d e5 c3 85 d9 5c df dd 29 6e
CertUtil: -hashfile command completed successfully.

@leo Please open this issue ....I too have got the same error...I downloaded Now Desktop 2.3.5 and I'm getting the following error on my Win 10 Pro 64 bit
Node version : 7.4.0
λ certutil -hashfile now.exe
SHA1 hash of file now.exe:
d17c6509c92822121abcd37b799e675e0b32d330
Is there any way we can gets logs of now?

Have you installed the binary through Now Desktop or did you just download Now Desktop expecting it to replace the binary automatically? 😊

Here's the thing @leo . As @hutch120 mentioned when we click on the Now Desktop , we get that error...no interface is even presented to us to download ....we just get  that Win10 error message.

Oh. So Now Desktop doesn't work? Not Now CLI.

Actually both....even typing cli in the terminal gives the same issue

Also running into this issue.

@Esslamb I had to redo npm global install of now for the problem to go away. YMMV.

@indcoder I just tried to remove it, it gave me some trouble so I removed it manually and re installed but still the same. The prompt just launches and then it prints access is denied.  Any ideas ?

@Esslamb  which version are you running? I have got 7.3.1 but this issue cannot be resolved till we know what the logs are saying @leo Where does now-cli log to default when initializing?

I tried installing the CLI alone without the desktop app but I still run into the same smart screen issue.

Same issue, can't run now from CLI

I'm running Win7 32-bit

Can someone confirm if this is still broken or fixed?

I will DL the latest version and try tonight @leo I uninstalled the CLI my windows machine due to the problems and just used my mac so hopefully, it's fixed.

@leo This seems fixed for me mate no more smart screen warnings. Thanks!

Going to optimistically close then, then :) Thanks for the confirmation @Esslamb!

https://user-images.githubusercontent.com/15871340/31042210-2de75320-a5c0-11e7-83bb-048524348e5b.png
Same issue, can't run now from CLI.
I'm running Win7 32-bit.


Come up with better name

I have decided I don't like the name. taking suggestions here. Only requirement is must be valid python package name

gh_scribe

gha_scribe

pencil_pusher


Default Array of String includes the square bracket on 2.1.1

On version 2.1.1, The Default value of :strings includes the [] bracket as a string. This did not happened on 2.0
opts = Trollop.options do
  opt :suites, 'List of Suites', :type => :strings, :default => ["AwSuites"]
  opt :debug, 'debug level', :default => "ERROR"
end

On 2.0
c:\>ruby duku.rb --suites  --debug ERROR
Error: option '--suites' needs a parameter.
Try --help for help.

c:\>ruby duku.rb  --debug ERROR
opts[:suites]: ["AwSuites"]
opts[:suites_given]:

c:\>ruby duku.rb --suites a b c --debug ERROR
opts[:suites]: ["a", "b", "c"]
opts[:suites_given]: true

But on 2.1.1
c:\>ruby duku.rb --suites  --debug ERROR
opts[:suites]: ["[\"AwSuites\"]"]        <<<<<<===============   Bracket included, instead of Error
opts[:suites_given]: true

c:\>ruby duku.rb  --debug ERROR
opts[:suites]: ["AwSuites"]
opts[:suites_given]:

c:\>ruby duku.rb --suites a b c --debug ERROR
opts[:suites]: ["a", "b", "c"]
opts[:suites_given]: true

EDIT: kbrock added syntax highlighting

@student any thoughts on this one?

Yikes.  I will try to look over this tonight.

git checkout 42b691285e29f3c408a47a72a2669840f59ad4f1
ruby duku.rb  --suites  --debug ERROR
{:suites=>["["AwSuites"]"], :debug=>"ERROR", :help=>false, :suites_given=>true, :debug_given=>true}
git checkout 1b132c1e749eaccf95d6f916aa2924136156045c
ruby duku.rb  --suites  --debug ERROR
Error: option '--suites' needs a parameter.
I have to take off.  I hope to have a writeup tonight.

The changes from #11 assumed that the parameter type is singular, not an array.  If the default is an array of integers, for instance, there will be a failure saying that the option needs an integer (what it has at that point is an array).  Since strings are hit with :to_s, the array gets the treatment.
I don't see any approach to doing this that I like.  Probably the least disruptive thing is to view the situation as exceptional:
      vals["#{sym}_given".intern] = true # mark argument as specified on the commandline

      opts = @specs[sym]
      if params.empty? && opts[:type] != :flag
        if opts.has_key?(:default)
          vals[sym] = opts[:default]
          next
        else
          raise CommandlineError, "option '#{arg}' needs a parameter"
        end
      end

That is, set the #{sym}_given up top, then shortcut out if we have a non-flag with no parameter but a default given.

@student thanks. I'll add this use case to the test suite and see if that code fixes it.
Still trying to figure out the organization to this test suite.

@student I had thought this was a change you introduced. Just realized it was not one of your PRs. Thanks for the thoughts and help.
@hansuiloe Would you be ok with the results if the array was properly populated?
@Fryguy I'm thinking about rolling #11 back. Any thoughts?

Glad to be of help. :)

@kbrock  Thanks for addressing this. Although you can't really differentiate between 2 and 4 example below (2.1.2), I guess this is acceptable for now.
c:\>ruby duku.rb --debug ERROR
opts[:suites]: ["AwSuites"]
opts[:suites_given]:

c:\>ruby duku.rb --suites  --debug ERROR
opts[:suites]: ["AwSuites"]
opts[:suites_given]: true

c:\>ruby duku.rb --suites a, b c --debug ERROR
opts[:suites]: ["a,", "b", "c"]
opts[:suites_given]: true

c:\>ruby ruby.rb --suites AwSuites --debug ERROR
opts[:suites]: ["AwSuites"]
opts[:suites_given]: true

@hansuiloe Yes. That is actually per design. If you have a --process to process records, defaulting to all, but optionally pass a parameter to specify a different value. This could be done with 2 options, but then you have to do special processing when you receive a value option but not the flag processing.
I'm mixed on whether I should have merged this feature request.
I had a bug in my code (didn't have a default), but I wonder how hard it would have been to track down if I did have a default.
db_script accepts parameter --user that defaults to ENV["PGUSER"].
wrapper_script calls the db_script via:
user_name = ""
system("db_script --user #{user_name}")

Yes, it is a bug in wrapper_script, but if --user has a default, then it won't show up until later. Again, curious how hard this feature would make tracking this down.
The fix for me was to ensure I had a value for user_name in wrapper_script.
Are you good for now?


The allowed QuestionFormats when adding a new question to a customisa…

Fixes # 1771
Changes proposed in this PR:

Ensure that "Multi select box" question format is no longer visible when customising questions.

looks good @johnpinto1


Bluetooth: controller: use sys_get_le24 to fetch 24-bit data

Is your enhancement proposal related to a problem? Please describe.
Refer to https://github.com/zephyrproject-rtos/zephyr/pull/27357#discussion_r477324409
and
https://github.com/zephyrproject-rtos/zephyr/pull/27357#discussion_r477319846
Describe the solution you'd like

Use sys_get_le24() to pass crc_init value
Define CRC polynomial and use the define instead of magic numbers.

Describe alternatives you've considered
NA
Additional context
NA

Hi, I want to give it a try, can you assign this to me?

Hi, please go ahead and send a pull request.


Wrong color name - Gray, not Grey

Yeah
Line |
 437 |  … rite-Header -Message "Backup Frequency" -ForegroundColor Grey -Separa …
     |                                                             ~~~~
     | Cannot validate argument on parameter 'ForegroundColor'. The argument "Grey" does not belong to the set
     | "Black,Blue,Cyan,DarkBlue,DarkCyan,DarkGray,DarkGreen,DarkMagenta,DarkRed,DarkYellow,Gray,Green,Magenta,Red,White,Yellow" specified by the ValidateSet attribute.
     | Supply an argument that is in the set and then try the command again.
Do you want to change backup frequency from each Sunday, 0:00 AM UTC to else? [Y/n]: n
Write-Header: /home/nacia/animeManga-autoBackup/Modules/Environment-Generator.ps1:447
Line |
 447 |  … ader -Message "Script Update Frequency" -ForegroundColor Grey -Separa …
     |                                                             ~~~~
     | Cannot validate argument on parameter 'ForegroundColor'. The argument "Grey" does not belong to the set
     | "Black,Blue,Cyan,DarkBlue,DarkCyan,DarkGray,DarkGreen,DarkMagenta,DarkRed,DarkYellow,Gray,Green,Magenta,Red,White,Yellow" specified by the ValidateSet attribute.
     | Supply an argument that is in the set and then try the command again.
Do you want to change script update frequency from each day, 0:00 AM UTC to else? [Y/n]: n
Write-Header: /home/nacia/animeManga-autoBackup/Modules/Environment-Generator.ps1:457
Line |
 457 |  … der -Message "Wayback Machine Snapshot" -ForegroundColor Grey -Separa …
     |                                                             ~~~~
     | Cannot validate argument on parameter 'ForegroundColor'. The argument "Grey" does not belong to the set
     | "Black,Blue,Cyan,DarkBlue,DarkCyan,DarkGray,DarkGreen,DarkMagenta,DarkRed,DarkYellow,Gray,Green,Magenta,Red,White,Yellow" specified by the ValidateSet attribute.
     | Supply an argument that is in the set and then try the command again.

Oh damn, i really missed that huh XD
then again, the repo is in stale period rn since i'm focusing in rewriting as CLI program in Python
Fix will be committed in few mins, alongside issue #3 '-')/


PAGER and KUBE_PAGER environment variable support

What would you like to be added:
When a user has a PAGER or KUBE_PAGER environment variable configured, kubectl should pipe it's output through that pager if the stdout is a tty.
Why is this needed:
Respecting the user's configured PAGER environment variable would allow for better user experience, just like respecting EDITOR and KUBE_EDITOR does today when running kubectl edit.
Scrolling through json or yaml output with the configured pager (eg. less or vimpager) integrates well into the linux ecosystem of pagers and would align kubectl with other tools that respect the PAGER variable.

I can work on this issue once it's triaged.

Just wondering, what to do if pager executable exists in $PATH but none of the environment variables (PAGER/KUBE_PAGER) are set? Should kubectl use pager or just print the text to stdout? What do other tools do in similar circumstances?

What is pager in your example? Are you referring to an actual binary called pager?

What is pager in your example? Are you referring to an actual binary called pager? I'm not aware of any convention where pager means anything.

$ which pager
/usr/bin/pager
$ ls -l $(which pager)
lrwxrwxrwx 1 root root 23 Feb 20  2022 /usr/bin/pager -> /etc/alternatives/pager
$ ls -l /etc/alternatives/pager
lrwxrwxrwx 1 root root 13 Feb 20  2022 /etc/alternatives/pager -> /usr/bin/less

If it's OK to add distro-specific logic into kubectl then go ahead.
But IMHO adding distro-specific behaviour belongs into distro-package patches.
Since /etc/alternatives/pager does not exist in Fedora, various RHEL flavours (Rocky, Alma, AmazonLinux, OracleLinux, etc.) I think it does not make much sense to be preferential about debian-based linux.

If it's OK to add distro-specific logic into kubectl then go ahead. But IMHO adding distro-specific behaviour belongs into distro-package patches.
The /etc/alternatives/pager is definitely not standard, it does not exist in Fedora, or various RHEL flavours (Rocky, Alma, Amazon, Oracle, etc.). I think it does not make much sense to be preferential about debian-based linux.
I'd say just don't use a pager if the env var is not defined.

Totally agree.

Can you please review this PR so I know I am going to the right direction or I'm completely wrong? 👀
kubernetes/kubernetes#120946

This should be a shell function or something similar imo, e.g.:
kubectl () {
        command kubectl "$1" | pager
}

Add the function to your RC file or something. Reason for this is that this would be something extra for kubectl to be responsible for that is very distro specific, and it can be solved using shell scripts rather than adding functionality (and more code) to kubectl that will then need to be supported going forward.

@mpuckett159
I agree that keeping it out of kubectl would make the work of kubectl maintainers easier, but I don't agree (yet) that it's really out of scope. I have two reasons:

It needs some context that's already known by kubectl, but not by a blind shell function. For example, pager should not be used for some commands like exec, or it's behavior might be a little different when --watch flag is passed to some commands like get. Also, the pager should not be used when stdout is not a tty (e.g. kubectl get pods | grep pattern)
This one isn't really a logical reason but makes me doubt. I'm just wondering why the developers of git log or man decided to handle pager themselves instead of forcing the users to define a shell function or something for paging the output.

@mpuckett159
A simple shell script you provided cannot shim the pager functionality. It's at least 100 lines or more, because one would have to partially reimplement the kubernetes command parsing in bash to determine what commands NOT to use a pager for (eg.: edit, port-forward, attach, debug, exec, and I haven't even talked about subcommands). Also having to be mindful of the output being a tty or not (eg. when kubectl output is piped to another command you most definitely don't want a pager gobbling up everything in-between).
Also PAGER is fully distro-agnostic, it's an environment variable.maybe stdin/stdout piping works differently in windows, but definitely works the same across linux and various different BSD flavours. For example I'm on MacOS and I 100% do want PAGER, as it's a distro (and OS) agnostic solution.

The example script was not meant to be a functional solution merely an example. You can make the script aware enough to run only on a subset of commands.
You'll have to ask the git and man maintainers why they decided to include native support for this.
Distro support is not the right words, I should say rather support for a very specific tool that is not maintained or controlled by the Kubernetes project or sig-cli. This tool is outside our scope and I think makes this out of scope for the project because it's possible to write your own integration that doesn't require access to the guts of kubectl. There are already issues that I've been added to even this month for external tools kubectl integrates with natively that need extra support, and these tools are fundamental to the commands they support, so I am saying this now with that in mind to reduce the surface area of potential issues arising from adding support for this.
Also it was said the shell script would require "at least 100 lines" to implement because you would have to reimplement kubectl's command parsing. Unfortunately it is not any easier in the internals of kubectl. There will likely need to be many lines over several files added to support this feature. You would still need to implement much of the logic mentioned in kubectl directly.

I'm almost convinced. So, I closed the PR.

I agree that we shouldn't add any logic in kubectl depending on external binaries. This can be managed via plugin or aliases, etc. rather than an implementation in kubectl. I'm closing this issue, feel free to reopen it if you think I'm missing something. Thanks.
/close

< I agree that we shouldn't add any logic in kubectl depending on external binaries
Just as a closing note, you already do depend on external binary when choosing the user's own EDITOR when doing kubectl edit.

I'd love to be able to see this implemented with a shell function, but I really can't see how it can reliably distinguish kubectl $A_BUNCH_OF_PARAMS get $OTHER_PARAMS from kubectl $A_BUNCH_OF_PARAMS edit $OTHER_PARAMS to avoid triggering the pager in the latter case.
Arguably, even kubectl get --watch should not trigger the pager, so I agree it is likely complex to deal with even from inside the kubectl codebase, but it is definitely harder to deal with it from outside.
Another big drawback of the piping approach is that foo | pager by default reports the exit status of pager, not the foo one. It can be worked around, but it is yet another hurdle and there are enough of them that I do not think it is reasonable to implement this as a shell function.
For reference, here are a few Go modules implementing the "use a pager if any of them can be found in the $PATH and stdout is actually a terminal":

https://pkg.go.dev/go.chromium.org/luci/common/system/pager
https://pkg.go.dev/github.com/gerow/pager
https://pkg.go.dev/github.com/jackdoe/go-pager

Kubecolor now has pager support. https://github.com/kubecolor/kubecolor


Remove Back Side of DVD Cover for Poster Image

Expected Behavior
Poster image only contain the front part of DVD Cover.
Current Behavior
Some title have correct behavior (for example DVDMS-693), but most have full front and back cover.
Or is there an option / settings for this feature that I'm not aware of?
Your Environment

version 2.5.10
Windows 11 and PowerShell 7.2.5:

After reviewing some of closed issue, I found a similiar problem which can be resolved by reinstalling pillow. So I did that and the issue is fixed.


chore(main): release 1.0.0

:robot: I have created a release beep boop
1.0.0 (2024-11-26)
Features

implement Python ecosystem support (71c4e71)
initial Commit (1180f7b)

Bug Fixes

adds .nvmrc file (301ab81)
adds bin entry for the init script (9c64572)
adds repository info to package.json (881c2e1)
enables pyproject.toml as a project marker file (6471ca2)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v1.0.0
:sunflower:


can not pull image from harbor

I used jib command to build the image harbor.test.ximalaya.com/test/community-vote-service:20181206-103704
mvn clean install -DskipTests -Dmaven.javadoc.skip=true -Ptest -f=community-vote-service/pom.xml && mvn com.google.cloud.tools:jib-maven-plugin:0.10.1:build -Djib.from.image=harbor.test.ximalaya.com/common/runit-jdk8-tomcat8 -Djib.from.auth.username=barge -Djib.from.auth.password=Barge.Xmly.2018 -Djib.to.image=harbor.test.ximalaya.com/test/community-vote-service:20181206-103704 -Djib.to.auth.username=barge -Djib.to.auth.password=Barge.Xmly.2018 -f=community-vote-service/pom.xml -Djib.container.appRoot=/usr/local/tomcat/webapps/community-vote-service

but can not pull image with docker command
docker pull harbor.test.ximalaya.com/test/community-vote-service:20181206-103704
...
Unexpected EOF

and then , I deleted the repository(harbor.xxx/test/community-vote-service) in harbor ui. build image with jib successfully, but now
docker pull harbor.test.ximalaya.com/test/community-vote2:20181206-111607
Error response from daemon: manifest for harbor.test.ximalaya.com/test/community-vote-service:20181206-111607 not found

our harbor version is 1.4.0,  I guess that garbage collection cause the lose of some layer.

when I used -Djib.useOnlyProjectCache=true, the Unexpected EOF solved in another maven project. is there more detail about -Djib.useOnlyProjectCache=true?

useOnlyProjectCache=true forces Jib to not use the per-user cache that caches base images and layers.  The base images and layers are instead maintained within the per-project cache (target/jib-cache).  I'm puzzled as to how it could have an effect here.
Reading through your description, it would seem to me that Harbor should be failing during the Jib's image push.  I see that Harbor 1.6.2 is the latest release, and 1.7 is on the horizon.  Is it possible to confirm with the latest?

Regarding this error
docker pull harbor.test.ximalaya.com/test/community-vote2:20181206-111607
Error response from daemon: manifest for harbor.test.ximalaya.com/test/community-vote-service:20181206-111607 not found

when I try to pull a non-existing-repo in my existing Docker Hub account, I get the exact same error, so I think you should double-check that the image is actually available from the registry and you specified the correct tag name. (BTW, I also remember Harbor was a bit slow to show uploaded images in the web UI.)
$ docker pull francium25/non-existing-repo
Using default tag: latest
Error response from daemon: manifest for francium25/non-existing-repo:latest not found

--
Regarding -Djib.useOnlyProjectCache=true, first, as @briandealwis said, try the latest Harbor. If it still gives you Unexpected EOF, I'd like to know if you can consistently reproduce that

mvn clean install ... ...:jib-maven-plugin:0.10.1:build ... always ends up Unexpected EOF.double
mvn clean install ... ...:jib-maven-plugin:0.10.1:build -DuseOnlyProjectCache=true ... always works?

(BTW, always double-check if you're specifying the correct tag on the command line.)
If you can consistently reproduce these, I wonder if clearing the user-level image cache will resolve this. On Linux, the cache directory is $HOME/.cache/google-cloud-tools-java/jib. Try renaming that directory and rebuilding:
mv $HOME/.cache/google-cloud-tools-java/jib \
   $HOME/.cache/google-cloud-tools-java/jib-cache-backup-for-later-diagnosis

@briandealwis @chanseokoh I am sorry for that useOnlyProjectCache=true is a coincidence.
we implement the "online gc" in harbor
// The two most recent versions are kept and delete others
do
    curl --insecure -X DELETE --header 'Accept: text/plain' "https://<IP_ADDRESS>/api/repositories/$line/tags/$deltags"
done
// start gc
docker run  --name gc --rm --volumes-from registry vmware/registry-photon:v2.6.2-v1.4.0 garbage-collect  /etc/registry/config.yml

the "online gc" cause that we can get deleted layer information (whose http status is 200) with registry api, but the real size of the deleted layer is 0.
I guess that the jib may not verify the size of deleted layer, so that jib think that the layer is exist and  do not push the layer again, so that Unexpected EOF  happen

Oh that's interesting!  We do a HEADrequest against each layer and skip if the registry reports that it's available:
Dec 07, 2018 11:30:14 AM com.google.api.client.http.HttpResponse <init>
CONFIG: -------------- RESPONSE --------------
HTTP/1.1 200 OK
Accept-Ranges: bytes
Cache-Control: max-age=31536000
Content-Length: 7768687
Content-Type: application/octet-stream
Docker-Content-Digest: sha256:d455ccc1cea9edcc50a50e0ee33a9f90c1f45486bacf8cb59e13203d7ee3be66
Docker-Distribution-Api-Version: registry/2.0
Etag: "sha256:d455ccc1cea9edcc50a50e0ee33a9f90c1f45486bacf8cb59e13203d7ee3be66"
X-Content-Type-Options: nosniff
Date: Fri, 07 Dec 2018 16:30:14 GMT

Could you please dump the full response headers for these deleted layers?

@briandealwis
request is
HEAD /v2/test/xxm-subscribe/blobs/sha256:f14ad73052287ffa32abf3613f9f758b8d8e1a1783c667db21c5e3a1b9afafcb HTTP/1.1
Host: harbor.test.xx.com
User-Agent: Go-http-client/1.1
Authorization: xxx

response is
HTTP/1.1 200 OK
Server: nginx
Date: Sun, 09 Dec 2018 10:13:45 GMT
Content-Type: application/octet-stream
Content-Length: 11984
Accept-Ranges: bytes
Cache-Control: max-age=31536000
Docker-Content-Digest: sha256:f14ad73052287ffa32abf3613f9f758b8d8e1a1783c667db21c5e3a1b9afafcb
Docker-Distribution-Api-Version: registry/2.0
Etag: "sha256:f14ad73052287ffa32abf3613f9f758b8d8e1a1783c667db21c5e3a1b9afafcb"
Set-Cookie: beegosessionID=15fbe5e6088b2ca776c6ad18793e796f; Path=/; HttpOnly
Connection: keep-alive

Yeah, if that is really the layer causing the problem, it genuinely looks like the fault of Harbor not returning correct info. If you believe it is the layer that got deleted from Harbor, you should file a bug against Harbor with the relevant info.

@briandealwis @chanseokoh ok, I think it may be the bug of harbor（1.4.0）


Quarter for Qt5 and Qt4

Original pull request by Christian Bühler (Bitbucket: cbuehler, GitHub: cbuehler).

With this pull request, Quarter can be built using Qt5. If it does not exist, Qt4 is used.

Original comment by Roy Walmsley (Bitbucket: walroy, GitHub: walroy).

Any comments on this?  It is not something I am familiar with. If I get some positive feedback I will merge it.
Roy

Original comment by Edward Smith-Rowland (Bitbucket: [Ed Smith-Rowland](https://bitbucket.org/Ed Smith-Rowland), ).

Without having built this yet I think this needs to go in.  Right now I'm trying to get SoQt working with Qt5.
I'll try building it tonight.

Original comment by Bastiaan Veelo (Bitbucket: veelo, GitHub: veelo).

This is the only spot I have questions about. This code only had effect when HAVE_SPACENAV_LIB is defined, and then only relevant (I presume) when using a SpaceNavigator the Quarter way (which IIRC is only implemented for Linux). Why is this disabled for Qt5? Did it cause problems, or does the SpaceNavigator work without it? If it caused problems, what can we do to not introduce a regression for the SpaceNavigator support under Linux when using Qt5?

Original comment by Roy Walmsley (Bitbucket: walroy, GitHub: walroy).

I have now merged this request, as it has several approvals, although I an unable to test it myself.
Roy


contextIsolation false or true? electronEnableSteamOverlay has to be called?

Hi,
I was going through the documentation and noted that to utilize steamworks.js, the recommended setup is:
const mainWindow = new BrowserWindow({
    // ...
    webPreferences: {
        // ...
        contextIsolation: false,
        nodeIntegration: true
    }
});

Followed by:
require('steamworks.js').electronEnableSteamOverlay()
Interestingly, when I set contextIsolation to false, things don't seem to work. But, when it's set to true, everything functions smoothly even without invoking electronEnableSteamOverlay.
I'd like to confirm that I'm not overlooking any details.
Thank you in advance for any insights.
P.S.: I'm using Windows 10, and my build targets the same platform.

I think I've encountered the same thing.  When contextIsolation is set to false, I get an infinite loading animation, even when not 'packaged'  (just running npm start without making an actual packaged .exe).  If I set to true, things load instantly.
Also Windows 10, 22H2

However my game is published live on Steam, achievement works, cloud saves works, all works perfectly, so contextIsolation: true is okay.

I'd be interested in how you got the achievements working with contextIsolation: true!  That's what I'm trying to do (in an HTML/JS game that I'm packaging with an Electron wrapper) but while it's true that I can't make any API calls between the Electron/Node side (where steamworks.js lives) and the renderer (my browser window that actually has the game).

Here it is what I do and that works;
In preload.js
contextBridge.exposeInMainWorld("electronAPI", {
    gotAchievement: (code) => ipcRenderer.send("got-achievement", code),
    ...
    ...

In main.js
ipcMain.on("got-achievement", (event, code) => {
    const steamworks = require("steamworks.js");
    const client = steamworks.init();
    client.achievement.activate(code);
});

And in my renderer.js
...
window.electronAPI.gotAchievement(type);
...

I Hope this helps!

This is the way, keeping contextIsolation: true and nodeIntegration: false (default settings) is recommended for security reasons. Keeps nodejs api unavailable from renderer process.


Don’t run verify_integrity if the Serialized DAG hasn’t changed

dag_run.verify_integrity is slow, and we don't want to call it every time, just when the dag structure changes (which we can know now thanks to DAG Serialization)

^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code change, Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in UPDATING.md.

There is an existing test in test_scheduler_job, currently marked xfail in this branch called test_dag_file_processor_add_new_task that we either need to "subsume" or delete.
(If we can keep the existing test but just update it to point at SchedulerJob/new code instead it makes it clearer it's the same test)

fixed in https://github.com/astronomer/airflow/pull/919/commits/f4cb918ab784178ba0069a2814c87da3055e41b9


Bump autorest to 3.0.6335

Broken due to sample issue being fixed in https://github.com/Azure/autorest.csharp/pull/1001
Will bump and rebuild after.

Broken due to sample issue being fixed in https://github.com/Azure/autorest.csharp/pull/1001
Will bump and rebuild after.


Jupyterlab Extension

WIP

[ ] We need to fix the use of private ipywidgets APIs (widget abbreviations)

@ellisonbg can you comment on what the InteractMixins were doing? This does not work with more recent versions of ipywidgets and I am not sure what it is supposed to do.

@jasongrout ping.
This version works with the ipywidgets 6 and jupyterlab 0.7.

@jasongrout self merging.


Adding a Upgrade module guide

This PR adds an "upgrade module guide". It's meant to be read in conjunction with the regular upgrade your project guide.
I also fixed some lies in the regular upgrade guide.
Parent issue

https://github.com/silverstripe/silverstripe-upgrader/issues/85

FYI there's some content change to the initial upgrading guide which are not apparent because the file got moved around.
To view those specific changes look at a138e26e64835aa375be91280bcbeee763dd2a33

4653b6916805628de3bcb7201a272af62319f9b5 adds a blurb about self-updated the upgrader related to this issue https://github.com/silverstripe/silverstripe-upgrader/issues/130
Maybe review this bit individually.

I've addressed all of @ScopeyNZ's feedback. I've also added an extra blurb to the index to tell people to start with the project upgrade guide.


Make water in animated_skin example more bluish

The color of the water seems strange to me.
Personally I think that it looks better if it is more blue.
Before:

This PR:

:+1:

Can one of the admins verify this patch?

test me

:eyes:


Chrome is not launching when i upgrade to the latest taiko version

I have upgraded my taiko version from 1.0.3 to 1.2.7 . also updated before scenario as
beforeScenario(async () => {
if (process.env.host === "local") {
    await openBrowser({ headless: false, ignoreCertificateErrors: true, args: ['--no-sandbox', '--start-fullscreen', '--disable-web-security', '--disable-RendererCodeIntegrity'] })
    await setConfig({navigationTimeout : 3000000})
    await deleteCookies();
} else if (process.env.host === "docker") {
    await openBrowser({ headless_chrome: true, ignoreCertificateErrors: true, args: ['--disable-setuid-sandbox','--no-sandbox','--start-fullscreen', '--disable-web-security'] })
    await deleteCookies();
}

});
even then i see this issue as:
Error Message: Error: Chromium revision is not downloaded. Provide TAIKO_BROWSER_PATH or Install taiko again to download bundled chromium.
Stacktrace:
Error: Chromium revision is not downloaded. Provide TAIKO_BROWSER_PATH or Install taiko again to download bundled chromium.
at BrowserFetcher.getExecutablePath (node_modules\taiko\lib\browserFetcher.js:237:13)
at launchBrowser (node_modules\taiko\lib\browserLauncher.js:324:44)
at module.exports.openBrowser (node_modules\taiko\lib\taiko.js:95:65)
at module.exports.(anonymous function) (node_modules\taiko\lib\taiko.js:2622:36)
at Object.beforeScenario (step_implementations\ui\browser.js:11:15)
Any suggestions on this?

@sriv  This issue is fixed. Thanks


Update README.md (syntax highlighting)

Code blocks with syntax highlighting

@TohidEq Thank you for the contribution.
Peace & Love from Israel ✌🏼💜


apache service creation doesn't fully use the node['apache2']['service_name'] attribute

The creation of the apache2 service still has some hard coded service names in there.  This causes the recipe to fail if the httpd service is named differently.  For example, the software collections repo for redhat installs apache 2.4 on centos-6.6 as 'httpd24-httpd'.

Merged #368


don't register SubscriptFunction for Set data types

implementation was not supporting that and it resulted
in a stuck query used with a Set

@mfussenegger pls review

lgtm


Numpy - Initialize Array Problem – challenge structure

Describe your problem and how to reproduce it:
In the numpy module, the Numpy - Initialize Array Problem challenge has a particular issue.
The challenge is introduced by the following line:

What is another way to produce the following array?:

For accessibility, I though it would be best to add the original solution from the video. Something similar to

As instructed in the video, the code:
output = np.ones((5, 5))
z = np.zeros((3, 3))
z[1, 1] = 9

output[1:-1, 1:-1] = z

Produces the following array
[[1. 1. 1. 1. 1.]
[1. 0. 0. 0. 1.]
[1. 0. 9. 0. 1.]
[1. 0. 0. 0. 1.]
[1. 1. 1. 1. 1.]]

What is another way to produce the same data structure?

However, the solution included in the challenge is the one provided in the video.
I'd be more than willing to make a pull request describing an alternative solution 👍
Add a Link to the page with the problem:
https://www.freecodecamp.dev/learn/data-analysis-with-python/numpy/initialize-array-problem

Hey,
So, you are suggesting a new matrix for the question, so that it is not a copy-pasta of the video?

More than a new matrix another way to build the proposed one. Something like:
output = np.zeros((5, 5))

output[0,:] = 1
output[-1,:] = 1
output[:,0] = 1
output[:, -1] = 1

output[2, 2] = 9

With the wrong options being slightly different. For instance mistaking the position of the 9 (playing on the topic of zero-based indexing)
output[3, 3] = 9

Or again forgetting to provide the shape of the original array as a tuple
output = np.zeros(5, 5)

I haven't gone through the the videos, is that something you would expect users to know at this point @borntofrappe?
After a short glance, I think I would go about it by asking something like this:
What code would produce the following matrix:
[[ 0. 0. 0. 0. 0. 0. 0.
   0. 1. 1. 1. 1. 1. 0.
   0. 1. 1. 1. 1. 1. 0.
   0. 1. 1. 5. 1. 1. 0.
   0. 1. 1. 1. 1. 1. 0.
   0. 1. 1. 1. 1. 1. 0.
   0. 0. 0. 0. 0. 0. 0. ]]

And have the potential solutions look more like the code in the video.

@moT01 Are you ok with my opening this up to first-timers only, with the expectation that they change the question to use:
[[ 0. 0. 0. 0. 0. 0. 0. ]
 [ 0. 1. 1. 1. 1. 1. 0. ]
 [ 0. 1. 1. 1. 1. 1. 0. ]
 [ 0. 1. 1. 5. 1. 1. 0. ]
 [ 0. 1. 1. 1. 1. 1. 0. ]
 [ 0. 1. 1. 1. 1. 1. 0. ]
 [ 0. 0. 0. 0. 0. 0. 0. ]]

And the answers to include:
output = np.zeros((7,7))

o = np.ones((5,5))
o[2,2] = 5

output[1:-1,1:-1] = o

Plus, I would leave the current answer as an option, just to catch-out 🤷♂ , but the other options should all be 7x7 matrices.

Sorry for the delayed response here @Sky020. I am fine with that if you are.

Hey, I created my first PR. @Sky020 could you please review it.

Closed in #39773


DateRangePicker makes application freeze

DateRangePicker makes application freeze if you click on inmediatly after it's render in the page

I have this same issue.
I first noticed it on a personal app so had assumed it was (possibly) a local CSS conflict. But it also happens on the native docs webapp. It's both DatePicker and DateRangePicker, and is somewhat sporadic.
I can reproduce it with 30-40% confidence by:

going to https://www.dash-mantine-components.com/components/datepicker
clicking any DatePicker quite quickly after load
Sometimes, no popover shows and eventually my browser tab times out.

Video of the behaviour (happened in 3rd refresh):
https://user-images.githubusercontent.com/32452466/236429279-16e3a3ae-a2f6-48a6-ac1a-e4fa89caca9f.mov
I've been unable to track down the cause, but in a local app the locking happens 100% of the time when the DatePicker is used within a modal.
It's a behaviour specific to 0.12 and it didn't happen in 0.11.x

Thanks @andresballenf for creating this issue and @gofford for the analysis. I'm on it.

I'm able to reproduce this issue on the docs too but not even once in isolation.

Hi @snehilvj thanks for investigating this. Is there any solution? Should we wait until Mantine fix it?

Hello, we are currently facing the same issue in our app, any updates on this?

Hi not yet. The only way for now is to use a 0.11 release.

Can confirm that this affects HoverCard (thank goodness I found this thread, I was pulling my hair out). Am mitigating the issue by setting the position to "bottom" and ensuring that there is always enough of a margin at the bottom of the page.
Also, It has been 3 months, any update?

This has been fixed in the latest 0.13 alphas.


Cleanup buildah code

Sort options so they are in alphabet order
Remove extra lines of code for options parsing that really do not accomplish anything.
Remove variables when they are not necesary, IE Don't create a variable to hold an
option that is only used once, use the option instead.

bot, retest please

bot, retest this please

Looks like the test is suffering from some kind of version wonkiness.
not ok 45 buildah version up to date in .spec file
(in test file ./version.bats, line 15)
`test "$bversion" = "$rversion"' failed

Lets merge https://github.com/projectatomic/buildah/pull/269 to fix the version issue, which I caused.  :^(

@TomSweeneyRedHat @nalind Now passes Tests PTAL
@umohnani8 PTAL

One possible small change, and a fence sitting question.  Otherwise LGTM.

@TomSweeneyRedHat Fixed.

@mheon PTAL

As someone who hasn't used buildah much, I have some issues with some of the argument usage strings, which don't really explain much to me about how they're used. Code looks like a massive improvement, though.

@mheon @TomSweeneyRedHat PTAL,
Lets keep signaturepolicy confersation separate from this patch, I will open a new PR as soon as this gets merged.

LGTM. Thanks for cleaning up the usage text @rhatdan - it's a lot more clear now than it was previously

I do like when you tpyo typo :^)

https://answers.yahoo.com/question/index;_ylt=A0LEViR6UMpZMc8Ax1UnnIlQ;_ylu=X3oDMTBydWNmY2MwBGNvbG8DYmYxBHBvcwM0BHZ0aWQDBHNlYwNzcg--?qid=20081121134801AAkf5YF&p=supercedes
they are both right
"Supercede has occurred as a spelling variant of supersede since the 17th century, and it is common in current published writing. It continues, however, to be widely regarded as an error."
" Supercede comes from the same Latin root as intercede, cessation etc
SuperSede is a new dumb Americanism
Who prints the dictionaries these days? Check an English dictionary!!"

http://www.dictionary.com/browse/supercede?s=t

@TomSweeneyRedHat Also changed to supersedes, since I had to repush.

LGTM, one small casing issue that can be fixed later if you want

oops didn't see your latest comment.   LGTM, ship the bad boy

@rh-atomic-bot r-@TomSweeneyRedHat 90630a5

@rh-atomic-bot -r@TomSweeneyRedHat 90630a5

@rh-atomic-bot r@TomSweeneyRedHat 90630a5

I'll bite, what's the last comment supposed to do?  The changes called out in the  90630a5  link LGTM fwiw.  If you need me to push other buttons, please let me know.

Heh, yeah it's tricky. The syntax here is: @rh-atomic-bot r=<username_without_leading_@> [optional SHA]. Note that the [optional SHA] has to be just that, not a link to a GitHub commit. See https://homu-projectatomic-ci-infra.e8ca.engint.openshiftapps.com/ for full documentation.


Arabic translation

I have translated the strings to Arabic
sphinx_rtd_theme_Ar.txt

@geohadab thanks for this, we manage the transaltions using transifex, please check https://sphinx-rtd-theme.readthedocs.io/en/stable/contributing.html#translations

@geohadab thanks for this, we manage the transaltions using transifex, please check https://sphinx-rtd-theme.readthedocs.io/en/stable/contributing.html#translations

I already requested to arabic translation in the website ... But i could not join the team and the language is not added.

I already requested to arabic translation in the website ... But i could not join the team and the language is not added.

@geohadab sorry about that, please subscribe to https://github.com/readthedocs/sphinx_rtd_theme/issues/1013

@geohadab sorry about that, please subscribe to https://github.com/readthedocs/sphinx_rtd_theme/issues/1013


Course number and tool name in headers

If possible, it would be great to include the course number (for example: SMPL101 (Spring 2017) or NURS 897 02 (S15), plus the tool name, in a header at the top of each tool's HTML index page.
For instance: in a course named NURS 897 02 (S15), the very top of the page would say:
NURS 897 02 (S15): Forums
for the Forums and Topics data;
NURS 897 02 (S15): Assignments
for the Assignments data.
The rationale: many of our professors will likely print the HTML pages, and without an identification of the course number and tool name they might get confused about which course/tool the data is for.

Done. Needs backport.

Note that in m5 it still has the toolId in the header eg NURS 897 02 (S15) (SPRING 2015): sakai.iframe.site but I have raised #108 to update this.

#108 done so this is ready to go.

@steveswinsburg will this be in M6 for me to test?

This is in m5 with the exception of the toolname. It still shows the tool id as above. I am almost finished the customised photo roster export and will rebuild m6 then.

OK, Thanks, Steve.

This looks good!


Add Uri to string conversion

This pull request fixes #6
I'm not sure if I'm doing it right. should I have done it differently?  :sweat_smile:

Hi, thanks for the pull request. You could probably just add Uri to StringConverter (there is a collection of type names there). It's gonna be simpler.

Great! I've updated the code with your suggestion.
Should the test be kept as a separate test or should I integrate it with the "ConvertBasicTypes" test?

It's fine either way. Thanks again! I'm gonna merge 😄