Replace Travis with Circle as PR CI

I've moved tests but will need @humblearner help on getting coverage reports moved too.
We would also need to experiment moving around jobs between parallel containers to ensure utilization\acceptable timing.

Current coverage is 75.30% (diff: 100%)

Merging #3117 into master will decrease coverage by 2.02%

@@             master      #3117   diff @@
==========================================
  Files           430        430
  Lines         22153      22144     -9
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
- Hits          17130      16675   -455
- Misses         5023       5469   +446
  Partials          0          0

Powered by Codecov. Last update 49294dc...a64786b

As far as I can tell, we're at parity between circle and travis. @humblearner please 👀 to make sure we reporting coverage properly. @lakshmi-kannan you've seem to volunteer for review 😛

Re better utilization and faster tests: Yeah, we should try to run those tasks in parallel and see how it goes:

lint
unit tests
integration tests
pack tests

That's basically what we did on Travis. In theory, as long as dependencies are cached, running those tasks in parallel should be quite faster, but yeah still good to try it out and actually measure it.

We already running them in parallel. Kami, please read the code before review 😛

Just want to point out that the purpose of this PR is not to make CI faster or achieve great utilization (although both are desirable), but to switch from Travis CI to Circle CI for PR checks, keep feature parity between the two and, hopefully, make it so it would take the same amount of time.
The last travis test took us 18 minutes of actual time and about 32 minutes of total time including waiting for containers to free. The test for this PR took 8 minutes total. Clearly we're not running travis anymore. So the only thing I need your help in determining is whether we achieved feature parity?
We can work on further speeding up build process and getting better utilization in the other PR.

I don't think there is a significant change in speed between Travis and Circle for things like Unit/Integration tests.
18 mins in last Travis check is an exception from the rule.
Circle can be slow too from time to time too: ex 21 mins here https://circleci.com/gh/StackStorm/st2/2609#build-timing
And "Total time" in Travis is how they summ up time from all parallel executions, makes almost no sense.
Here is the usual 10.5 mins Travis build: https://travis-ci.org/StackStorm/st2/builds/183518380
So the diff in Circle vs Travis here I believe is mostly about usability. Example like that:

We already running them in parallel. Kami, please read the code before review

is a sign that it's hard to understand how stuff works in parallel in Circle (it's known issue).

18 mins in last Travis check is an exception from the rule.

Units, being the longest ones, take about 10+ minutes to execute. The result still qualifies as "not slower than Travis".

BTW, just remembered. There are CircleCI metrics with all bells & whistles.
Yeah, timing stability for st2 looks not that bad for simple unit & integration tests. Only 4 regressions for last 2 months:
https://circleci.com/build-insights/gh/StackStorm/st2/master

Much worse for st2-packages, due to increased complexity and more upstream interactions:
https://circleci.com/build-insights/gh/StackStorm/st2-packages/master

Apart of that, remove first comment lines of circle.yml to avoid confusion:
# Setup in CircleCI account the following ENV variables:
# IS_PRODUCTION (default: 0)
# IS_ENTERPRISE (default: 0)
# PACKAGECLOUD_ORGANIZATION (default: stackstorm)
# PACKAGECLOUD_TOKEN
# DOCKER_USER
# DOCKER_EMAIL
# DOCKER_PASSWORD

everything that is not needed.

@enykeev Can you also verify if https://github.com/StackStorm/st2/blob/c3655c5d82210d6a184a70eb2bc4c44e4ad10e93/circle.yml#L10-L12 notify webhook to st2 is actually used and remove it if there is no need?
I guess it's from some old plans, but not sure.

Awesome, besides the minor comments from others, I think we should forward here. You have my +1.

@armab even if we don't use this hook now (and I'm reasonably sure we do), we should start using it in a nearest future. So it's in. Minor things are fixed.

I'm OK with that, since everything was cleaned :+1:


License suite tests

I'd like to set up some tests that verify that askalono doesn't regress on its ability to identify licenses. Basically, a directory of license files named/sorted according to their actual license. askalono runs on them, identifies them, and verifies it got the answer right.
Possible implementation

A directory of license files with the following structure:

test-licenses/
  Apache-2.0/
    sample-project-name/
        LICENSE
    other-project/
        COPYING
  BSD-3-Clause
    and-another-one/
      LICENSE.txt
  ...

An integration test/suite that crawls the above structure, analyzes each license file, and ensures it reasonably matches (say, 0.8 confidence) the license name from the directory structure.

Another layout for this is perfectly acceptable; this could also be set up with some metadata files describing what a file should be identified as, with what confidence, etc. That may be overkill for the time being.

@Phrohdoh I clarified the description a bit. If you're still interested in this, I'd recommend playing around with Store as a starting point: have it load a cache file (or load from the SPDX directory; see cli/build.rs for an example of that) and get it to identify a license. examples/basic.rs has some of that as well. The documentation sucks (sorry) but the types involved should be relatively easy to figure out.
I opened up a Gitter room at https://gitter.im/amzn/askalono if you want to ping me for help; you may need to @ me as I'm not entirely sure I've configured notifications correctly 🙃

Great, thanks!
I will take a stab at this over the upcoming weekend.

@Phrohdoh no rush, but let me know if you'd like any extra help here. If you want, feel free to PR the code you showed on Gitter and we can iterate from there.

I've set up the remainder of this infrastructure:
https://github.com/amzn/askalono/blob/master/tests/real_world.rs
This builds on @Phrohdoh's initial work and crawls a directory, parsing out expected license names and thresholds to test licenses in a flexible manner. While I definitely want to add more licenses to this test dataset, I think we can consider this issue resolved. :)


Do you have any plan to support FlexboxLayoutManager?

Do you have any plan to support FlexboxLayoutManager?
At the moment, It's always crash.
04-18 10:01:58.802 2288-2288/? E/AndroidRuntime: FATAL EXCEPTION: main     Process: com.examples.myapplication, PID: 2288     java.lang.ClassCastException: com.google.android.flexbox.FlexboxLayoutManager cannot be cast to android.support.v7.widget.LinearLayoutManager         at eu.davidea.flexibleadapter.common.FlexibleLayoutManager.findLastVisibleItemPosition(FlexibleLayoutManager.java:188)         at eu.davidea.flexibleadapter.AnimatorAdapter.animateView(AnimatorAdapter.java:320)         at eu.davidea.flexibleadapter.FlexibleAdapter.onBindViewHolder(FlexibleAdapter.java:1782)
I'd like to use Flexbox to implement some special ui, but it's not work.
Thanks in advance!

It's already supported: any third class are supported even for the ones created in the future.
Just need to implement the interface. Check the closed issues.


FEATURE: Approve and merge in one step

Add a flag or combination to approve and merge in the same command.
Feels kind of dirty to force merge in a repository where approvals are required.

Hi @Yelinz, thanks for your suggestion. I will try and find some time today or this week to see how to go about it.


Move dev-related files and instructions to dev/ directory; add committer's instructions

Adds committer-instructions.md file for Druid committers. For now, it has a PR action item checklist, i. e. a list of actions that a committer should do with their own PRs or a PR from a non-committer that they encounter.
Open question: what about PRs from new contributors? Should they sign something like Apache ICLA, and therefore a committer should suggest the new contributor do this?
In the future, committer-instructions.md may be updated with more general "Code of Conduct"  for committers, an updated version of "General committer guidelines" from this page: http://druid.io/community/

Open question: what about PRs from new contributors? Should they sign something like Apache ICLA, and therefore a committer should suggest the new contributor to do this?

For ICLA, only the committers need that<EMAIL_ADDRESS>

@gianm @asdf2014 I've extended the scope of this PR. I've moved some files from root to a new dev/ directory. I've also added more committer's instructions:

PR merge action item checklist
Creating a new tag on Github

@gianm @asdf2014 do you have more comments?

@leventov looks nice! two small suggestions:

There is no hyperlink to committer-instructions.md, I think CONTRIBUTING.md should link to it.
I think the asset files should be organized into sub-folders (perhaps TeamCity_images, IDE_images), especially since there is no README.md in the folder so it's harder to sort things out.

@yurmix good suggestions, thanks. I've moved images to a folder. I didn't mention committer-instructions.md in CONTRIBUTING.md because committers don't read CONTRIBUTING.md, but I referenced the dev/ directory in general. I think it should be the place for all development-related prose, like checklists, code style description, principles for choosing logging severity, etc.

@gianm do you have more comments?

@gianm do you want to leave more comments?


Fix test_regressions tests failing in GitHub Actions

I had to mark the test_regressions tests with XFAIL when running the checks in CI because they are currently failing when being run in GitHub Actions. See here and here.
This is probably due to small differences in output between environments (local vs. remote macOS, Windows, and Linux).
The test, for reference:
https://github.com/tpvasconcelos/ridgeplot/blob/34498e54b2381ed97ac09f182df5a40f8e4a4943/tests/e2e/test_examples.py#L30-L37
Regardless of the solution, one nice feature to implement here is the ability to upload the plot artefacts that resulted in the failed tests and expose these artefacts to the relevant PR. This discussion might be useful for this:

https://github.com/actions/upload-artifact/issues/50
other references on this topic:
https://github.com/orgs/community/discussions/51403
https://github.com/djthornton1212/djthornton1212/blob/main/.github/workflows/pr-comment-artifact-url.yml
https://stackoverflow.com/questions/60789862/url-of-the-last-artifact-of-a-github-action-build

See https://github.com/tpvasconcelos/ridgeplot/blob/34498e54b2381ed97ac09f182df5a40f8e4a4943/tests/e2e/test_examples.py#L30-L37

For some reason, the tests/e2e/test_examples.py::test_examples_width_height_set[lincoln_weather-load_lincoln_weather] hangs forever and never finishes.
Some examples here: https://github.com/tpvasconcelos/ridgeplot/actions/runs/7481625128/job/20363565240
This ran succeeded but it took over 14minutes to run the e2e test suite: https://github.com/tpvasconcelos/ridgeplot/actions/runs/7481625128/job/20363565240
For this reason, I have also changed the xfail mark to a skipif (https://github.com/tpvasconcelos/ridgeplot/pull/163/commits/8faa9df224c1b35c95385e5144936736d016fb25)


[Bug]: Page.emulateTimezone doesn't work with Firefox and BiDi

Minimal, reproducible example
import puppeteer from "puppeteer";

(async () => {
  const browser = await puppeteer.launch({
    browser: "firefox",
    headless: false,
    protocol: "webDriverBiDi",
    extraPrefsFirefox: {
      "remote.log.level": "Trace",
    },
    //dumpio: true,
  });

  const page = await browser.newPage();
  await page.goto(`data:text/html,

  `);
  await page.bringToFront();
  await page.waitForSelector("p");

  await page.emulateTimezone("Pacific/Honolulu");

  await page.evaluate(() => {
    const time = document.getElementById("time");
    time.textContent = new Date().toLocaleString();
  });

  await page.close();
  await browser.close();
})();

Background
I wrote a test for pdf.js related to a timezone issue.
Expectation
Just work.
Reality
It throws:
throw new UnsupportedOperation('CDP support is required for this feature. The current browser does not support CDP.');

Puppeteer configuration file (if used)
No response
Puppeteer version
23.9.0
Node version
20.18.0
Package manager
npm
Package manager version
10.8.2
Operating system
Windows

Cc @whimboo.

Yes, it is not supported and not part of the standard yet: https://github.com/w3c/webdriver-bidi/issues/749

Related documentation: https://pptr.dev/webdriver-bidi#puppeteer-features-not-supported-over-webdriver-bidi


Reformat with ktlint, some code refactoring

I will add another PR later that will refactor all the code to bring it up to a good standard.

I'm on vacation right now, will take a look at it when I get home


[ALIASES] Fix column renaming for normal users

STR:

Normal user (no aliases feature flag) uploads dataset
Create map with dataset
Get into table view in created map
Rename column

Expected behaviours:
Column gets renamed
Actual behaviour
No popup for rename appears.

Pr off source please

Hey Josh, be sure to leave a status update here before leaving for the day :)

Update
After looking at the code I feel that we should be able to adjust the logic in the template to handle the needed change.  The file table_header_view.jst.ejs is where the changes need to be made to fix the column name change bug.  The <input class="col_name_edit" type="text" value="<%- col_name %>" /> is in the if (hasAliasesFeatureFlag) check and this is preventing a non data user from changing the column name.

see PR #155
Let me know if you find any errors in testing. Thanks

I have completed the changes that you requested! Thanks.  Let me know if any further changes are needed.


Upgrade kevinrob/guzzle-cache-middleware

Could we update the following dependencies:

kevinrob/guzzle-cache-middleware 4.0.0? This resolves some return type issues in previous versions. They did drop PHP 7.2 but to be fair, you guys should too.
phpstan to 1.4.x
Support for PHP 8.1

It shouldn't be too painful to upgrade. I ran a quick test on my machine and apart from some deprecations and two errors during tests, it all worked. I have no clue how to solve the failed tests myself.
@bytestream I can PR a failing setup if you like?

Sorry I missed this @ju5t

They did drop PHP 7.2

They only dropped versions prior to 7.2. PHP 7.2.0 is the required minimum https://github.com/Kevinrob/guzzle-cache-middleware/blob/22d79bb234861469e0c1d6b9541eb6d8e97ecd72/composer.json#L16

I can PR a failing setup if you like?

Sure, if you could send a PR but maintain PHP 7.2 support then we can help with the failing tests

I've opened the PR.

Awesome, thanks for the quick release!


Use crates.io keywords and categories

Closes #651 .

r? @alexcrichton
(rust_highfive has picked a reviewer for you, use r? to override)

:sunny: Test successful - status-appveyor, status-travis
Approved by: gnzlbg
Pushing 530f31f6c17624f20f3897ae930eaea34a574296 to master...


[GO-AGE-VIEWER] Linked

Make links among them

Reviewed with Nimra in meeting, @aked21 please merge


Failed to configure Azure AD authendication

Hi all,
I've followed the instruction for configuring Azure AD authentication however I keep getting following error from microsoft login page:
AADSTS50011: The reply url specified in the request does not match the reply urls configured for the application
When I created the app registration I already filled in reply URL which exactly matches our rancher login page: https://rancher.xxxx.com, and I triple checked it after, configuration seems to be correct. I've tried both https://rancher.xxxx.com and https://rancher.xxxx.com/, nothing worked

Useful
Info

Versions
Rancher v2.0.5 UI: v2.0.60

Access
local admin

Route
global-admin.security.authentication.azuread

We are also facing the same issue in our installation. Any advise would be recommended.
regards,
Fabian

I have this too.

Same issue here

same issue with rancher v2.0.4
any updates on that issue?

Can you try using https://rancher.xxxx.com/verify-auth-azure as Reply URL in the App Registration and let me know if that works? It took around 2 minutes after saving the Reply URL before it was actually applied.

I can confirm that this works!
I think it would be great if you could update the documentation at: https://rancher.com/docs/rancher/v2.x/en/admin-settings/authentication/azure-ad/

Will update docs and fix the UI showing the proper Reply URL when configuring Azure AD

@westlywright Can we update the message to display rancherUrl (server/path) and also mention that it can take up to 5 min for the url to propagate through Azure?

Version - 2.0 master 8/27
Verified fixed


Multiple Pairings being sent

I recently added IceBreaker to a team and it was running fine on a daily basis for the test group.  I change the run frequency and added 300 "live " users.  The first run was today and the bot sent two pairings to every Team member.
I can see anywhere obvious to check for multiple accounts or multiple runs.  Could you advise please?

Hi @kf-paulbrown
Thank you for raising the query, you can check the Azure Cosmos DB 'TeamsInstalled' table to see the number of channels where bot is installed. You can also check the Azure app insights on the metrics on the number of channels installed and the number of messages sent etc.

The Cosmos DB shows that the app has been installed in 3 channels (although two of these have been deleted).  I'm having trouble finding the data in the App insights.

Hi @kf-paulbrown,
Go to Azure App Insights, look for Event type named 'Custom Event' and look for 'ProcessedPairUps', you should see details like users notified count, installed Teams count etc.
Refer below screenshot for reference,

Hi @kf-paulbrown
Please let us know if you've any other queries, if not we'll proceed with closing this issue.

Hi @kf-paulbrown,
Please wait for the next notification scheduled day and check if you're still getting duplicates, if the users are part of more than 1 channel, they might be notified more than once as well.

Hi @kf-paulbrown, please let us know if you've any updates on the response provided, if not we'll proceed with closing out the issue.

Thank you, this issue is resolved.
Kind Regards
Paul Brown

Paul BrownBSc. (Hons) (he/him)
Associate, Office 365 Architect
Office 365 | SharePoint | Intranets | Yammer | Teams

D:  +44 20 8106 1393
knightfrank.com
S:  +44 20 7629 8171
55 Baker StreetLondon,W1U 8AN,United Kingdom
PA: Paul Brown
This email (including any attachments) is confidential. If it is not addressed to you, please do not read, disclose, copy or forward it on, but notify the sender immediately and delete it. Any copying, disclosure, distribution or other use of this email (and any attachments) by anyone other than the intended recipient is prohibited and may be unlawful.
This e-mail is sent on behalf of Knight Frank LLP, a limited liability partnership registered in England (registered number OC305934). Our registered office is at 55 Baker Street, London W1U 8AN where you may look at a list of members' names (this can also be found at http://www.knightfrank.co.uk/about-us). If we use the term ‘Partner’ when referring to one of our representatives, that person will either be a Member or an employee of Knight Frank LLP. Knight Frank LLP is authorised and regulated by the Financial Conduct Authority for insurance and mediation activities only. Each entity or practice in the Knight Frank global network is a distinct and separate legal entity. No Knight Frank entity acts as agent for, or has any authority to represent, bind or obligate in any way, any other Knight Frank entity.
We have tried to ensure this email does not contain any viruses, but please check this before opening any attachments, as we cannot accept any responsibility for damage caused by a virus. Further information concerning our website, information technology and our monitoring, processing and use of personal data can be found at http://www.knightfrank.com/legals. Firms such as Knight Frank LLP and their clients are increasingly being targeted by fraudsters, often requesting funds to be transferred to a different bank account or seeking to obtain confidential information. If you receive a suspicious or unexpected email from us, or purporting to have been sent on our behalf, please do not reply to the email, click on any links, open any attachments, or comply with any instructions contained within it without first speaking (in person or by telephone) with your Knight Frank LLP contact to verify the email. Knight Frank LLP cannot take responsibility for any losses arising from your transfer of funds or disclosure of confidential information.
Except where specifically stated otherwise, any information about price or value contained in this email is provided purely as guidance. It does not constitute a formal valuation and should not be relied upon for any purpose. Unless specifically stated otherwise, any negotiations, intention to treat, offers, acceptances or consideration contained in this email are not intended to create legal obligations or relations and are all subject to contract.
 [[KFLLP]]
From: Porkai @.>
Sent: Thursday, April 7, 2022 12:38:00 AM
To: OfficeDev/microsoft-teams-apps-icebreaker @.>
Cc: Paul Brown @.>; Mention @.>
Subject: [EXTERNAL] Re: [OfficeDev/microsoft-teams-apps-icebreaker] Multiple Pairings being sent (Issue #229)
Hi @kf-paulbrownhttps://github.com/kf-paulbrown, please let us know if you've any updates on the response provided, if not we'll proceed with closing out the issue.
—
Reply to this email directly, view it on GitHubhttps://github.com/OfficeDev/microsoft-teams-apps-icebreaker/issues/229#issuecomment-1090927700, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AC62A667XMKPVWULFNKB2YTVDYN5RANCNFSM5PBDC7XA.
You are receiving this because you were mentioned.Message ID: @.***>


Read this policy

It is great to see that users now does not have to know who to reach out to for admin consents!
We are developing apps that integrate to Graph API and have an onboarding functionality where users need to fill out contact information to admins since we require admin-consent and there has not been any way of getting info about who is the admin.
Now since there is a setting for this and also API's for this, it would be good if we could read if the adminConsentRequestPolicy setting is enabled or not without admin-consent. In that way we could change the onboarding flow depending on this setting. But since it needs Policy.Read.All (which requires admin-consent), we cannot use it anyway?
// Jarle

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

ID: f6e93be2-bad4-bae7-91bc-d4122dad86fd
Version Independent ID: caf48884-743b-8f83-58a0-a92c3940fbba
Content: Configure the admin consent workflow - Microsoft Entra
Content Source: articles/active-directory/manage-apps/configure-admin-consent-workflow.md
Service: active-directory
Sub-service: app-mgmt
GitHub Login: @omondiatieno
Microsoft Alias: jomondi

@jarleskogheim
Thanks for your feedback! We will investigate and update as appropriate.

@jarleskogheim
Hello! It's great to hear that you find the admin consent workflow helpful. Regarding your question, unfortunately, you are correct that the Policy.Read.All permission is required to read the adminConsentRequestPolicy setting, and this permission requires admin consent. Therefore, you cannot read this setting without admin consent. However, you can still change your onboarding flow to accommodate both scenarios, where the admin consent workflow is enabled or disabled. You can prompt the user to provide contact information for the admin regardless of whether the workflow is enabled or not. If the workflow is enabled, the admin will receive a notification to review the request, and if it's disabled, the user can still provide the contact information for the admin to grant consent. I hope this helps! Let me know if you have any other questions.


Difference in behaviour of persistent icon between wlr/workspaces and hyprland/workspaces

In wlr/workspaces, the "persistent" icon is only set if the workspace is not empty: https://github.com/Alexays/Waybar/blob/b7a527b122831235fb36f661d415f5f3ed378d92/src/modules/wlr/workspace_manager.cpp#L498-L503
But in hyprland/workspaces, the "persistent" icon is set if the workspace is a persistent one: https://github.com/Alexays/Waybar/blob/b7a527b122831235fb36f661d415f5f3ed378d92/src/modules/hyprland/workspaces.cpp#L390-L395
The behavior in wlr/workspaces allowed me to have this rule:
"format-icons": {
    "active": "",
    "default": "",
    "persistent": "",
},

where the active workspace gets the active icon, inactive workspaces that contain windows get the default icon (since they don't count as persistent, as they have a window in them), and inactive workspaces that don't contain windows get the persistent icon.
When I switched to using hyprland/workspaces, all inactive workspaces now use the persistent icon regardless of whether they have a window in them.
I like to have all my workspaces visible, and also an indication that a workspace contains a window... and I do think this way of doing it is pretty hacky, but I can't think of any other way I'm supposed to do this. It also seems this behaviour is intended considering on hyprland/workspaces, persistent workspaces do have the .persistent class taken away when they contain a window: https://github.com/Alexays/Waybar/blob/b7a527b122831235fb36f661d415f5f3ed378d92/src/modules/hyprland/workspaces.cpp#L325
so I guess this means I could technically work around it with some weird .persistent::before CSS rule, but it would be nice to use the format-icon config object to format the icons.

What if I added a way to add "empty" icons? If a persistent workspace is empty, it would get the "empty" icon, and if it has isn't, it would get the "persistent" icon.
I should then probably add an "empty" css class, and change the "persistent" class to match the "persisten" icon.
What do you think?

Yes, that's would be perfectly fine. In fact if it's documented it would be way better as then people don't have to hunt around for a way to do this.


Add stop_looping to sound_player

Coverage decreased (-0.04%) to 89.127% when pulling 5149471bf2a5d570fe729831b03bd7b996a5b3cd on audio-dev into 21af1961cd0afccef772b33f653724bf210b42f8 on dev.


Failed to load, Error 500

Hi, today i install your program. All ok in the setup, but at search for anything i get nothing.
In the console show:

Failed to load resource: http://<IP_ADDRESS>:8080/api.php?search=&page=0&order=posted&action=galleries the server responded with a status of 500 (Internal Server Error)

The error is from the server but i don't get it, is from e-hentai server ? or local server?
Info:
-Used "Vagrant Setup" (currently trying "Windows Setup"...)
-Windows 8.1 (Admi)
-Virtualbox & Vagrant (last version)
-Default value at "accessKey" (if change, nothing happend)
Thanks.

Hey there @ZeroDrako
I'll start investigating this right away.
Just to get some more info, can I get a screenshot of your browser when you go to localhost:8080/?
I'll also be redoing some things relating to the the vagrant setup, as I've just realised I forgot to impliment a way to actually get the server to download the archives you send to it.

I've take 3. Hope this help.

Thanks.

Okay,
I've done quite a bit of updating, I've adding actually archiving from the Vagrant setup now.
I'd like to check one thing for your specific case however, did you install the Virtual Box Guest Extensions?
You can find it here.
As well as install the Vagrant plugin for Virtual Box Guest Extensions, by running vagrant plugin install vagrant-vbguest?
If you didn't do both of those things, run vagrant destroy and then after executing that vagrant up once more to purge the box.
I'll be commiting my new update soon, so perhaps hold off until then.

Yes, installed VirtualBox +  Guest Extensions...
Well, i re install from zero. Still have a conexion problem, but now is a problem over https.
I think the problem is that local server is on HTTP and the web is no HTTPS (exhentai changed 3 or 4 days ago to https)...
I've trying load exhentai in http but the problem persist. Change the "header" to allow the error is gone but nothing happened.

PD: yes, i've misinterpreted, i thought i could also search on the local server to download directly from it :/

I'm still going to keep looking into this, on my end everything works from scratch, but I'll take another look.
Perhaps it has something to do with Chromium. I'll keep investigating.

I've fixed the issue.
 The issue will be fixed from then on (or so I believe)
Sorry for all the hastle.


feat: add forceExternalList option

The Rolllup equivalent plugin has both include and exclude functions --- confusingly named, but functional:

purpose
rollup-plugin-node-externals
esbuild-node-externals

exclude from externalization (include in bundle)
exclude option
allowList option

force treating as external (exclude from bundle)
include option
???

I'm proposing forceExternalList to fill the missing space in this plugin, patch included.
This is helpful for externalizing dependencies of dependencies --- packages I need externalized, but that aren't listed directly in my package.json file.

Thanks for the pr, will create a new release with it


Update scala3-library, ... to 3.2.1

Updates

org.scala-lang:scala3-library
org.scala-lang:scala3-library_sjs1

from 3.1.3 to 3.2.1.
GitHub Release Notes - Version Diff - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-lang" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "@monthly" },
  dependency = { groupId = "org.scala-lang" }
}]

labels: library-update, early-semver-minor, semver-spec-minor, commit-count:n:2

Superseded by #442.


Add domain operations in Graph rbac

This checklist is used to make sure that common issues in a pull request are addressed. This will expedite the process of getting your pull request merged and avoid extra work on your part to fix issues discovered during the review process.
PR information

[x] The title of the PR is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For information on cleaning up the commits in your pull request, see this page.
[x] Except for special cases involving multiple contributors, the PR is started from a fork of the main repository, not a branch.
[ ] If applicable, the PR references the bug/issue that it fixes.
[x] Swagger files are correctly named (e.g. the api-version in the path should match the api-version in the spec).

Quality of Swagger

[x] I have read the contribution guidelines.
[x] My spec meets the review criteria:

[x] The spec conforms to the Swagger 2.0 specification.
[x] The spec follows the guidelines described in the Swagger checklist document.
[x] Validation tools were run on swagger spec(s) and have all been fixed in this PR.

Hi There,
I am the AutoRest Linter Azure bot. I am here to help. My task is to analyze the situation from the AutoRest linter perspective. Please review the below analysis result:
File: arm-graphrbac/1.6/swagger/graphrbac.json
Before the PR: Warning(s): 33  Error(s): 25
After the PR: Warning(s): 35  Error(s): 25
Know more about AutoRest Linter Guidelines.
Send feedback and make AutoRest Linter Azure Bot smarter day by day!
Thanks for your co-operation.

Hello @jianghaolu, Feel free to loop in the service team owners to make sure they are aware and to get their signoff on PR as well :)

Hi @shuagarw , please advise if this change is acceptable to you. Thank you!

ATTENTION REQUIRED.
We have changed the default branch in the azure-rest-api-specs repository to use the branch labeled current.
This PR is opened against the master branch.
You have two options:

Keep this PR open, and complete the review process as normal, and once this PR is merged against the master branch, we (@olydis & @fearthecowboy)  will create a PR against the current branch and make any updates required to work in the newly reorganized default branch.

Close this PR and create a new one. That would mean pulling in the changes from the new default branch current moving any files and switching to the configuration-based layout.  (You are more than welcome to choose option 2, but I don't expect anyone to do that)

Either way, the work you are currently doing against the previous default branch master will be the last PR against that branch that we will accept and all new PRs will go against the new default branch  current.
If you have any questions or concerns, ping @fearthecowboy or @olydis (or go ask on the #swagger channel in slack) ASAP and we can help you thru the process.

Breaking changes eliminated.

 https://github.com/Azure/azure-sdk-for-ruby/pull/847

No modification for AutorestCI/azure-sdk-for-python

 https://github.com/Azure/azure-sdk-for-node/pull/2221


Hyperparameter multilabel

modified code to include f1-score metric for multilabel hypeparameter optimization step.

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

In the file decavision/model_training/tfrecords_image_classifier.py:

Line 502:9: E722 do not use bare 'except'


Critical CSS not being included in production builds

I have a custom css file with the path assets/css/critical/01_giscus.css. When running Hugo as a server, it is included in the HTML, but when running hugo by itself for production builds it is not included.
I know this because when I run hugo server --renderToDisk I can view the generated files in public and see that the CSS is there. I also set relativeURLs = true in Hugo.toml with no effect. Not sure if I'm doing something wrong or if it's a bug.

It probably has to do with the PostCSS plugins which are executed in production:
https://github.com/schnerring/hugo-theme-gruvbox/blob/main/postcss.config.js#L24-L34
Specifically, PurgeCSS removes unused CSS. Have a look at the PurgeCSS Safelisting docs. Let me know how it goes!
I should probably mention this in the docs. 🤔

Thank you. Putting /* purgecss ignore */ above the rule fixed the issue

I've also ran into this issue without adding any additional CSS. There's stock critical css missing in my production builds and the only way I can get the page to render nicely is to remove this.
I'm not sure if I'm missing something in the config. The css purging seems to be overly aggressive for production builds.

Do you have a sample repo containing the issue? @davegallant

Do you have a sample repo containing the issue? @davegallant I can't reproduce the issue.

Yeah: https://github.com/davegallant/davegallant.github.io
Oddly, I also had to remove the integrity checks for the css and js for it to load correctly. I'm wondering now if it has something to do with the fact it's hosted on Cloudflare Pages.

Do you have a sample repo containing the issue? @davegallant I can't reproduce the issue.

Yeah: https://github.com/davegallant/davegallant.github.io

And what CSS are you missing?
I checked out the commit 3f549f757687ffe090907e1624cee4342bd18bd9, but I can't see any CSS files in assets/css/ folder.

What CSS are you missing exactly?
I checked out the commit 3f549f757687ffe090907e1624cee4342bd18bd9, but I can't see any CSS files in assets/css/ folder.

Ah, that was before I pulled the theme in-tree. It's here:
https://github.com/davegallant/hugo-theme-gruvbox/tree/62aac1ea9f6d42dd735dc76d9d59771230ca6054/assets

That's a fork of this theme, and a different repo alltogether.
Please provide clear steps to reproduce the issue, or provide a sample repo that I can simply hugo server -e production.

That's a fork of this theme, and a different repo alltogether.
Please provide clear steps to reproduce the issue, or provide a sample repo that I can simply hugo server -e production.
Also specify exactly what CSS you're missing in production.

Yes, there were previous commits before that were pointing to the non-forked repo in the go.mod.
I should have documented this better at the time, but was able to repro again:
https://github.com/davegallant/hugo-theme-gruvbox-161/tree/main

Okay, I reproduced the issue. You can achieve the same result by running:
cd example.com
hugo server -e production

Well, the defaults inside the config of the theme are there for a reason. Why did you remove the following?
build:
  noJSConfigInAssets: true
  writeStats: true

Try adding that to your config.yaml.

I'll add a comment to the config explaining that PurgeCSS requires writeStats = true. I'll also have to upgrade that setting, because it will be deprecated in future versions in favor of buildStats.
Also a question regarding user experience. Did the confusion maybe come from having the config split into multiple files under config/_default/? Would it make things easier for for time users if I consolidated these files?

See https://github.com/schnerring/hugo-theme-gruvbox/commit/d3df58b1529a85702e68e588fefd640b6d7c5adf
Your config must include the following:
[build]
  [build.buildStats]
    # With each build, Hugo writes a hugo_stats.json file that contains arrays
    # of the class attributes, id attributes, and tags of each HTML element
    # within the published site. For production builds, this file is required
    # by PurgeCSS to prune unused CSS.
    enable = true

To debug PurgeCSS with hugo server -e production, also add the following Hugo mount:
[[module.mounts]]
  # Add hugo_stats.json to Hugo's server watcher
  # See https://gohugo.io/getting-started/configuration/#configure-cache-busters
  source = "hugo_stats.json"
  target = "assets/watching/hugo_stats.json"


Warning on creating node object

When I define a node object I will get 4 warnings about the clause not matching. If I remove node from in front of object then the error goes away.
Current versions:
absinthe              1.5.0-rc.3
absinthe_plug         1.5.0-rc.2
absinthe_relay        1.5.0-rc.0
elixir -v
Erlang/OTP 22 [erts-10.5.1] [source] [64-bit] [smp:16:16] [ds:16:16:10] [async-threads:1] [hipe] [dtrace]

Elixir 1.10.2 (compiled with Erlang/OTP 22)

defmodule ZB.JournalEntryLine.Types.Queries do
  use Absinthe.Schema.Notation
  use Absinthe.Relay.Schema.Notation, :modern

  node object(:journal_entry_line) do
    field(:id, :id)
  end
end

warning: this clause cannot match because a previous clause at line 1 always matches
  lib/zb/banks/bank_member/types/queries.ex:1

warning: this clause cannot match because a previous clause at line 1 always matches
  lib/zb/banks/bank_member/types/queries.ex:1

warning: this clause cannot match because a previous clause at line 1 always matches
  lib/zb/banks/bank_member/types/queries.ex:1

warning: this clause cannot match because a previous clause at line 1 always matches
  lib/zb/banks/bank_member/types/queries.ex:1

Realized after I created this issue that I had set the id field, which you shouldn't do on node objects. A better warning might be nice.

Seconding this notion.


Cursor in wrong position after inserting Tabnine completion items

Describe the bug
The cursor does not move to the end of some completion item after it is inserted into the buffer. The bug is reproducible with completion items that have closing brackets ) or parenthesis }. This happens mostly with Tabnine sources. I attached a video below showing the issue.
Sometimes the cursor will momentarily move to the end of the insertion after completion, but will jump back to its previous position before insertion.
According to this cmp-tabnine issue, this bug was introduced around 8c08bdf7a4e0f218b233765913ff0669358f4a6e. Reverting to that commit fixes this issue for me.
Minimal config based on this
if has('vim_starting')
  set encoding=utf-8
endif
scriptencoding utf-8

if &compatible
  set nocompatible
endif

let s:plug_dir = expand('/tmp/plugged/vim-plug')
let s:plug_path = s:plug_dir . '/plug.vim'
if !filereadable(s:plug_path)
  execute printf('!curl -fLo %s/autoload/plug.vim --create-dirs https://raw.githubusercontent.com/junegunn/vim-plug/master/plug.vim', s:plug_dir)
end

execute 'set runtimepath+=' . s:plug_dir
call plug#begin(s:plug_dir)
Plug 'hrsh7th/nvim-cmp'
Plug 'tzachar/cmp-tabnine', { 'do': './install.sh' }
Plug 'hrsh7th/cmp-buffer'
Plug 'hrsh7th/cmp-nvim-lsp'
Plug 'hrsh7th/vim-vsnip'
Plug 'neovim/nvim-lspconfig'
call plug#end()
PlugInstall | quit

" Setup global configuration. More on configuration below.
lua << EOF
local cmp = require "cmp"
cmp.setup {
  experimental = { native_menu = false, ghost_text = true },
  completion = { completeopt = "menu,menuone,noinsert" },
  confirmation = { default_behavior = cmp.ConfirmBehavior.Replace },
  snippet = {
    expand = function(args)
    vim.fn["vsnip#anonymous"](args.body)
  end,
  },

mapping = {
  ['<CR>'] = cmp.mapping.confirm({ select = true })
  },

sources = {
  { name = "tabnine" },
  { name = "nvim_lsp" },
  { name = "buffer" },
  },
}
EOF

lua << EOF
local capabilities = require('cmp_nvim_lsp').update_capabilities(vim.lsp.protocol.make_client_capabilities())

require'lspconfig'.cssls.setup {
  capabilities = capabilities,
}
EOF

To Reproduce
This video shows how I reproduce the issue.
https://user-images.githubusercontent.com/273399/141614968-42d527df-ea99-4755-9252-8ed6bd803056.mp4
Expected behavior
I expect the cursor to jump to the end of a completion insertion.
Additional context
More context is available where I originally reported the issue as a cmp-tabnine bug here.

Perhaps cmp-tabnine is returning the invalid items.
Basically, the "completed state" and the "returned item" must be synchronized to the same buffer state.
cmp-tabnine saves the "latest  callback" as a variable at https://github.com/tzachar/cmp-tabnine/blob/main/lua/cmp_tabnine/source.lua#L158 and call it later asynchronously.
If there are consecutive completion requests, it is possible to call the callback with the result of the previous request to tabnine.
google translated

@hrsh7th
It is not a problem with the async behavior of cmp-tabnine.
I successfully bisected the issue to e1f880b7e0ae3cae251fb6c7901bc856fe546e0b which is the first commit where the issue appears.

@tzachar Thank you bisected. Could you test with this?
Hm... I should write tests for core.confirm but it uses feedkeys so hard to create tests...

mmmm, not working:
E5108: Error executing lua ...re/nvim/site/pack/packer/start/nvim-cmp/lua/cmp/core.lua:420: Expected 2 arguments stack traceback:                                                                                [C]: in function 'nvim_win_set_cursor'                                                                                ...re/nvim/site/pack/packer/start/nvim-cmp/lua/cmp/core.lua:420: in function <...re/nvim/site/pack/packer/start/nvim-cmp/lua/cmp/core.lua:377>                                                                                .../site/pack/packer/start/nvim-cmp/lua/cmp/utils/async.lua:44: in function 'next'                                               ...re/nvim/site/pack/packer/start/nvim-cmp/lua/cmp/core.lua:374: in function <...re/nvim/site/pack/packer/start/nvim-cmp/lua/cmp/core.lua:344>                                                                                .../site/pack/packer/start/nvim-cmp/lua/cmp/utils/async.lua:44: in function 'next'                                               ...re/nvim/site/pack/packer/start/nvim-cmp/lua/cmp/core.lua:340: in function <...re/nvim/site/pack/packer/start/nvim-cmp/lua/cmp/core.lua:321>                                                                                .../site/pack/packer/start/nvim-cmp/lua/cmp/utils/async.lua:44: in function <.../site/pack/packer/start/nvim-cmp/lua/cmp/utils/async.lua:42>                                                                                ...te/pack/packer/start/nvim-cmp/lua/cmp/utils/feedkeys.lua:51: in function <...te/pack/packer/start/nvim-cmp/lua/cmp/utils/feedkeys.lua:49>

10x :)
This solves the issue


Set SENTRY_ENVIRONMENT variable when building front-end

Depending on the CircleCI context, the value is
staging/demo/production. See also
https://docs.sentry.io/platforms/javascript/guides/react/configuration/options/#environment

 2.0 Add Sentry to new FE


[Browser Bookmarks] open in different profiles

Extension
https://www.raycast.com/raycast/browser-bookmarks
Description
It's great that I can search bookmarks from different profiles (I use chrome), but I would like the option to then open that bookmark in that profile, rather than the last selected window being used. This is similar to the "Open Bookmark's Browser" option, but for the browser profile.
Who will benefit from this feature?
Anyone that keeps a work and personal profile in their browser and wants to search for a bookmark to open in one or the other would benefit.
Anything else?
No response

Thanks for the feature request. Would you like to work on it? I can provide some guidance on how to contribute to this extension so that you can open a pull request. You can contact me here or in the Slack community if you need further help.
Note that opening up a PR and having it merged will earn you some credits as we want to reward contributions. You’ll then soon be able to exchange them in our swag store.


Remove libcurl-dev dependency

This dependency is not required for tools/sgx in Gramine after the core Gramine's commit "[tools/sgx,curl,Docs] Link all dependencies of tools/sgx statically".
Relates to https://github.com/gramineproject/gramine/pull/773.

This change is

https://github.com/gramineproject/gramine/pull/773 was already merged; a new Gramine release v1.4 was also released. Should we move this PR forward?


BUG: read_excel with squeeze=True

closes #12157

as usual top notch!
ping when green

thanks! - green

thanks!


trestle author ssp-generate does not seem to do the correct thing with aggregates parameters

Describe the bug
For aggregates parameters (such as si-7_prm_1 from https://raw.githubusercontent.com/usnistgov/oscal-content/main/nist.gov/SP800-53/rev5/json/NIST_SP-800-53_rev5_catalog.json) ssp-generate is showing a warning about the parameter has no values and was referenced by prose and the markdown ends up with the label field inserted, rather than the values aggregated from si-07_odp.01, si-07_odp.02, and si-07_odp.03.
To Reproduce
Steps to reproduce the behavior:

Import the following profile: profile.json
Run trestle author ssp-generate -p imported-profile -o markdown
Note the warnings about values
Open the markdown/si-7.md file
See that the content from the various parameters are set in the assessment objective section, but not in the control statement

Expected behavior
I'd expect there to be no warning about missing values, since the individual aggregated parameters are present, and the control statement output to reference the set parameters instead of the generic catalog label.
Environment

OS: macOS / Docker
Python version: python-3.11.9-slim docker image
Installed packages: trestle 3.2.0

Hi @rahearn . Just wanted to reach out and let you know that I will be looking at this issue and give feedback about it soon. Thanks for reaching out and will contact again soon.

@rahearn the PR has been linked to the issue, if you want to take a look at it. Thanks


correctly resolve the variable of jpeg library prefix

The resulting string is later exported (as part of GDIPLUS_LIBS) to the
generated pkg-config file and this file shouldn't contain
-L${libjpeg_prefix}/lib -ljpeg as libjpeg_prefix is unknown in this context...

Hello! I'm the build bot for the Mono project. I need approval from a Mono team member to build this pull request. A team member should reply with "approve" to approve a build of this pull request, "whitelist" to whitelist this and all future pull requests from this contributor, or "build" to explicitly request a build, even if one has already been done. Contributors can ignore this message.

approve

did that mean "merge"? Because it's still not merged :)

Hello! I'm the build bot for the Mono project.
I need approval from a Mono team member to build this pull request. A team member should reply with "approve" to approve a build of this pull request, "whitelist" to whitelist this and all future pull requests from this contributor, or "build" to explicitly request a build, even if one has already been done.
Contributors can ignore this message.

Hello! I'm the build bot for the Mono project.
I need approval from a Mono team member to build this pull request. A team member should reply with "approve" to approve a build of this pull request, "whitelist" to whitelist this and all future pull requests from this contributor, or "build" to explicitly request a build, even if one has already been done.
Contributors can ignore this message.

This looks definitely fine to me. Merging.


add support for python 3.5

As suggested in Issue #8 I added support for python 3.5. Please review and merge if it is fine
Best Carsten

Hi, is this work in progress? Would like to merge a pull-request which passes tests. ;-)

Ah, was a typo ;) sorry!


How does p4vfs work with P4 Client correctly

When I Installed p4vfs service, and rehydrating some files, their size becomes 0, but it seems that just a refresh operation from the p4 client will trigger IO to read the complete file, causing these files to be re-downloaded. How can I correctly cooperate with the P4 client to avoid unexpected dewatering?

Hi alexjingsi,
By "P4 Client" I'm assuming that you mean the P4V application. In that case it may depend on the file extension and if P4V is possibly reading it from disk to preview it. For example, within the tab "Files -> Preview".
It's often surprising to find which process may try to hydrate the file first. It's no uncommon to see Windows Defender service (MsSense.exe) read the file in search for possible malware. You can look at the P4VFS.Service logs ("%PUBLIC%\Public Logs\P4VFS\P4VFS.Service") to inspect the process ID which hydrated the file. You can also use the p4vfs monitor application in the task tray p4vfs monitor show to tail the service log to show the same info.
P4VFS service does have an ExcludedProcessNames configuration variable, which could possibly be used to prevent p4v.exe from hydrating the file. I generally wouldn't recommend this though ... we typically only use this feature for temporary unit testing.

p4vfs set ExcludedProcessNames=p4v.exe

Regards
-Jess


Aldrnari 2.1 - Bow Rapid Combo not fully working

Issue:
As is, only the basic 'rapid shot' function is working when using a bow and pressing M2, trying all the other shot types with directional key requirements (such as multi-shot with forward+M2) only does another rapid shot.
Suggested partial fix:
dTry's Key Utils is listed as a requirement for the mod and after installing it, I managed to get everything but the triple shot and the multi shot working properly - both of those make the player character T-pose instead with no arrows fired. As the Bow Rapid Combo description mentions, these two are the only 'combo' moves, so there is something wrong with the input when you chain the same shot twice.
In the Bugs tab someone else was having the same issue and they claimed to have fixed it by installing MCO, no idea if that helps since animation issues are beyond me to troubleshoot.

This isnt the place to make bug reports.

Why? Every other project collaborator/organizer/developer uses this for bug reports. This is a bug reporting section. I'm not using discord. Act like a proper developer.

This isn't a developper project; i'm not submitting code, I'm taking notes elsewhere on bug reports; I answer ASAP on Discord when issues arise.
I dont care what your beliefs are, I've been doing lists for longer than I was a developper; I've always functionned that way.
It also allows me to ban assholes that dont follow rules.
If you'd use Discord, "lo and behold", you'd know why it doesnt work.
This isn't a bug, you're just unable to read.


feat: not install localtunnel by default (#1225)

fix #1225
There is a security vulnerability related to the axios dependency of localtunnel, but there isn't a newer version of localtunnel. This PR turns localtunnel into an optional peer dependency.

I think instead, we could switch to https://unjs.io/packages/untun


CSS animation added

Title and Issue number

Add turning pages animation
Close #4703

Checklist:

[x] I have mentioned the issue number in my Pull Request.

@MohitGupta121  @TusharKesarwani  please review this pr under gssoc'23


Add new color palette apply tool and a new "Current frame" color palette

This adds a new tool to the transformsection, that allowsyou to apply the currently selected palette to all the colors of the frame or the colors of all frames..
The pr also adds a new dynamic palette, called "Frame colors". This palette automatically updates itself to show you all the colors of the selected frame and their index numbers. Those index numbers are what the new tool uses for reference

For anyone who wants to try all of the new features I am adding in one branch, check out
https://github.com/blurymind/piskel/tree/piskel-plus

To be clear, this new tool is a Transform type operation for palette swapping. The way it works is

You have a dynamic reference palette (current frame palette), which gets updated every time you add a new color to your frame. You can then duplicate your reference palette and change any of the colours in it. Select that new palette (should be selected when you create it), then using this tool, the colours in it will be compared to the reference palette. A colour replace operation is applied to each of the reference colours, using their indexes - the same operation that the tool @juliandescottes showed uses - but automatically on each colour of the frame.
Howding a modifier key will apply the operation on all frames as swapout all the colours with the ones  you have in your new palette - in a single click


[FR] Improve exception handling for rate limiting

Is your feature request related to a problem? Please describe.
When using the SDK to create email action links, such as for password reset or email verifications, Firebase is strongly rate limiting for good reason. However, it is currently impossible to (programmatically) detect whether the reason for failure was indeed rate limiting, or something else, since in both cases a generic InvalidArgumentError exception is raised.
Describe the solution you'd like
A way to distinguish rate limiting from other client or server side errors, so that my API is able to return 429 or 503 responses with a Retry-After header instead of a generic 500 response in those cases.
Describe alternatives you've considered
In order to prevent my API from returning generic 500 I'm considering to implement my own rate limiting in front now. However, that is less elegant since I would need to heuristically match Firebase's rate limiting algorithm in order to prevent emitting 500's anyway.
Additional context
Currently the RESET_PASSWORD_EXCEED_LIMIT error code from Firebase is swallowed and converted into a INVALID_ARGUMENT code, because this type of exception is not special-cased:
InvalidArgumentError('Error while calling Auth service (RESET_PASSWORD_EXCEED_LIMIT).')

Note that the same holds for building email verification links:
InvalidArgumentError('Error while calling Auth service (TOO_MANY_ATTEMPTS_TRY_LATER).')

In both cases the original code is not introspectable in the exception, except by parsing the error string.

@lahirumaramba , Is anyone actively working on this issue ? If not then please assign me, thanks

@lahirumaramba @pragatimodi FWIW I linked a PR a while ago that I believe solves this problem. Can someone have a look at it?


Installing from setup.py fails

Currently when installing from a setup.py of another project gives me the error:
Adding nervaluate 0.1.7 to easy-install.pth file

Installed /Users/campea/datalabs/modelling/grants/grant-inference/build/virtualenv/lib/python3.7/site-packages/nervaluate-0.1.7-py3.7.egg
Searching for en-core-web-sm@ https://github.com/explosion/spacy-models/releases/download/en_core_web_sm-2.2.0/en_core_web_sm-2.2.0.tar.gz#egg=en_core_web_sm
Reading https://pypi.org/simple/en-core-web-sm/
No local packages or working download links found for en-core-web-sm@ https://github.com/explosion/spacy-models/releases/download/en_core_web_sm-2.2.0/en_core_web_sm-2.2.0.tar.gz#egg=en_core_web_sm
error: Could not find suitable distribution for Requirement.parse('en-core-web-sm@ https://github.com/explosion/spacy-models/releases/download/en_core_web_sm-2.2.0/en_core_web_sm-2.2.0.tar.gz#egg=en_core_web_sm')

I suspect it is not processing dependencies correctly. A minimal setup.py to re-create the problem:
from setuptools import setup, find_packages

setup(
    name='test',
    author='Antonio Campello',
<EMAIL_ADDRESS>    description='Testing dependency links',
    packages=find_packages(),
    include_package_data=True,
    version='0.0.1',
    install_requires=[
        "wellcomeml"
    ],
    dependency_links=[
        "https://github.com/wellcometrust/WellcomeML.git#egg=wellcomeml[deep-learning]"
    ]

)

Fixed by latest move to PyPI


[ENH]: Add comparison operations to the Interface

Indexing and selecting data with common logical operation is a common and widely used feature. It should be added to the CommonDataStructureInterface, allowing expression suchs as:
obj[:, "group"]== val
obj>val

Completed as per commit 06ae93ebcbf37fe5d88ca7abef66c8178df41a22


[3.2] Document what can be used as a type hint

Documents what can be used as a type hint in two resource loader methods. My other PR for the master branch can't be cherry picked for 3.2 since the methods of ResourceLoader have changed.

Thanks!


Improve validation

Client only allows importing tifs
Import accepts S3 URI instead of bucket/key
Change validation so most fields are optional

Connects #253

On Firefox, I am able to get to a state where it won't let me go forward but gives me no error messages. If I fill out the form and use only one of the dates it will show no error and not let me move on.

The issue I reported about Firefox looks to be generic. The same thing happens to me on Chrome.

Looks good, would be great to get this merged in today.

Addressed feedback in latest commit. Left in error message that appears if only one capture date was entered since it doesn't hurt.

+1

@azavea-bot rebase

@azavea-bot rebase

That would be cool but I don't think it can do that.

@azavea-bot rebuild


generated xfreerdp.1 man page is badly formatted

FreeRDP 2.1.0
Archlinux with

xmlto 0.0.28
docbook-xsl
docbook-xml 4.5
libxslt 1.1.34
libxml2 2.9.10

Here's the bug report for archlinux. ATM the archlinux package carries the suggested changes for FindDocBookXSL.cmake.
Only xfreerdp.1 is affected, all other man pages are fine (tested: freerdp-shadow-cli.1 winpr-hash.1 winpr-makecert.1 wlfreerdp.1 wlog.7).
man_xfreerdp_errors.txt
man_xfreerdp_output.txt

@tinywrkb I´ve just compiled current FreeRDP master fd5bdee8b3b1e985e201e9cd0c7426456ee028a1on ubuntu 20.04 and debian 10, the man page is just like as it should be. This looks like an error on the arch end

I can't reproduce on Debian unstable and from the Debian docbook-xml dokbook-xsl packaging patches nothing stands out.
As it's not possible for me to reproduce on a distro with similar versions of packages I'm closing this issue.


Image Cropping not available

How can i apply cropping on selected image like we can do on UIImagePickerController. Thanks

@waqasarshad right now it's not supported in the library. Feel free to fork the project and add it and create a PR if you like. I am tied up with some other stuffs.


Added sample for MI

@chandan-anjani,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


Fixing problems of slow performance with Ruby server in local environments

Problems were mainly a ruby environment 5x slower.

Frontend tests were OK :+1: (details)


1.16.5 Silent's Gems Gravity V & VI do not work

I had a player of my modpack, Create on a Potato PC report:

There's a problem with Gravity enchantment on the pickaxes. I have two with Gravity V and VI but they don't mine underwater at all, as if I have mining fatigue! And the same thing is while flying

Minecraft: 1.16.5
Forge: 36.1.0
Apotheosis: 1.16.3-4.4.1
I tried to replicate this player's setup but was unable to spawn in a Gravity V or VI book. For now, I have limited the level of Gravity to the default level 3

That would be on SG to fix, I cannot make higher levels of another mod's enchantment work.


/trace output is less helpful than it could be due to buffering

Just a minor nitpick. When running Boogie with /trace, it produces output such as:
Verifying CheckWellformed$$_.foo ...
  [0.716 s, 2 proof obligations]  verified

Verifying Impl$$_.foo ...
  [187.421 s, 225 proof obligations]  verified

However, the line for "Verifying" only appears in the console window at the same time that the second line with the result information is printed, so you're never sure what Boogie is currently attempting to verify. I assume this could be fixed by adding an fflush()-equivalent between the two print statements.
This issue also applies to Dafny as Microsoft/dafny#10

Per comments on the Dafny issue: the output makes sense for parallel verification, but only when /vcsCores is > 1.


output window

Issue Type: Bug
The output window keeps popping up randomly and fills alomsot my display screen.

I have the same behavior on my machine. Even when I close the Output window, and am on Terminal. the Output window opens up and starts showing results.

@loopy3025 same question to you then. This is very likely an extension you have installed.

@Tyriar  Ya, I'll do that. I just need to wait for a build to finish first. I'd also like to be able to report which extensions I'm running. The instructions for creating a new issue do not include that step. How would I go about doing that?

Looks like the ones we have in common are the php ones.

@Tyriar : actually it happens when i jux launch vscode . the output window randomly starts running and even after i close it, it reopens after awhile

Which output channel is opening will probably give you the answer.


[Feature] Schedule RayJob

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Description
Is to add an extra functionality to RayJob to make it able to be running on a schedule (like everyday, every week, etc). This is similar to what Spark operator has, called ScheduledSparkApplication
Use case
Running a training job that trains a model every 2 weeks. Right now we have a to use Argo to do this which complicates the toolset.
Related issues
No response
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

The discussions in https://github.com/ray-project/kuberay/issues/1206 might be relevant.

Are you using Argo Workflow to create resource templates to arrange Ray Jobs?

@shuangkun No, we use Argo workflow to schedule running jobs on a standing Ray cluster


using the ess_mute_fet pin

Thanks for an interesting project. Sounds good in my opinion.
I changed the project a bit and used WM8804 (it will work by default too, you just need to switch to 1 TOSLINK). The device works like a USB Sound Card.
The question is, how is the ESS_MUTE_FET pin connected to the MUTE pin of the ESS9023 DAC used?

Thanks a lot for the feedback, I'm happy you found this useful! :)
I originally intended the ESS_MUTE_FET pin to be able to mute the DAC if there was any popping when switching inputs or when powering on. Since that didn't happen it's unused and you can leave it disconnected and skip the FET.
You might want to keep R4 and C11 (100kR to VCC / 2.2uF to GND) on the mute pin as they act as a soft-start for any power-on pops. The FET was there as the DAC runs off of 3.6V as opposed to the 3.3V of the Pico.

Hello.
When connecting via USB, extraneous sound is heard. Maybe use the ESS_MUTE_FET pin, turn it on at the beginning of the RPI-PICO periphery initialization, and turn it off after the end of the transition processes?
Also interesting pin TPA6130 - SD(shutdown), it can also be used in the project...
I hope you don't mind that I changed the code a bit. Since I use WM8804, there is an idea (probably somewhat wrong) to use a relay for switching: external TOSLINK or SPDIF RPI-PICO. Use a free RPI-PICO pin to switch.
Here is a rough diagram:

And make the menu look like this:

Hey!
You can definitely use the mute pin if you want to get rid of power-on pops with software instead of the RC network, you'll just need to have a pulldown to make sure it doesn't pop while the pico is still booting and set it high a few hundred milliseconds later. For reference the 2.2u/100k network on the schematics gives you ~220ms of delay.
On the TPA6130 you can send a shutdown over I2C (Bit0 on register 1) and save yourself a pin if you want to shut it down :)
You shouldn't get any pops out of the headphone output as the chip starts up with both channels muted though (and I don't hear any on my boards).
I did notice you're tapping off your line out after the DC coupling caps, but you don't have to do this as the ESS9023 has a line-level opamp driver. They're there for the TPA6130 when in single-ended mode (section <IP_ADDRESS> in the datasheet) and should ideally be close to it.
I don't see why you wouldn't be able to use a relay to switch SPDIF, the bandwidth isn't that high and it should work just fine!
Make sure to get a 3v3 compatible TOSLINK receiver (i.e. TORX147 instead of TORX179) if you're using optical inputs.
Please feel free to modify the code as you need, looks great with two inputs :)
Are you planning on building this inside a box?

Please stay safe! I hope peace returns soon, nobody deserves to ever endure what's happening right now.
We all stand with Ukraine 🇺🇦

https://defence-line.org/2022/03/world-war-iii-is-knocking-at-your-door/

Yes

IMG_20220729_230434.jpg

Hello.
Tell me, I can't understand in which file the pin of the SPDIF OUT is assigned?
When I compile an example from the sine_wave pico_playground, with a SPDIF output, the pin for the SPDIF OUT is changed in the file ../pico/sine_wave/pico-extras/src/rp2_common/pico_audio_spdif/include/pico/audio_spdif.h

Sorry for the slow response, the SPDIF pin is assigned here: https://github.com/alexstanoev/FoxDAC/blob/master/firmware/foxdac/CMakeLists.txt#L27
Putting the front panel on a vertical board is a good idea, looks great!

Thank you very much. I just started learning сmake, so some things are not clear to me.
Here are a few more photos, TPA6130 and toslink input has not yet been implemented


Update this sample to use WAM.

Update the sample to use WAM. Nothing else should be used on UWP
Update the README.md

https://github.com/Azure-Samples/active-directory-dotnet-native-uwp-v2/pull/47


Robot doesn't show

Hi,
I did have this sort of working using 0.6.3 but it was flakey - so I thought I would try the Beta (Now 0.7.0-11) and my vacuum has disappeared.  I am no home bridge or Linux wizard though so any help would be appreciated!

Hi, what Robot do u use? Does it maybe show in another room? Can you provide homebridge logs from the startup?

I'm just setting up homebridge-neato for the first time, and I'm not seeing any devices show up in the home app. Not sure if it's an error on my part or not. I've followed the 'simple' configuration on the main page, this is what my startup log looks like:
`
[10/12/2019, 7:09:27 PM] Loaded config.json with 0 accessories and 2 platforms.
[10/12/2019, 7:09:27 PM] ---
[10/12/2019, 7:09:27 PM] Loaded plugin: homebridge-neato
[10/12/2019, 7:09:27 PM] Registering platform 'homebridge-neato.NeatoVacuumRobot'
[10/12/2019, 7:09:27 PM] ---
[10/12/2019, 7:09:27 PM] Loaded plugin: homebridge-tcc
[10/12/2019, 7:09:28 PM] Registering platform 'homebridge-tcc.tcc'
[10/12/2019, 7:09:28 PM] ---
[10/12/2019, 7:09:28 PM] Loading 2 platforms...
[10/12/2019, 7:09:28 PM] [Thermostat] Initializing tcc platform...
[10/12/2019, 7:09:28 PM] [Thermostat] Logging into tcc...
[10/12/2019, 7:09:28 PM] [NeatoVacuumRobot] Initializing NeatoVacuumRobot platform...
[10/12/2019, 7:09:28 PM] [NeatoVacuumRobot] Refresh is set to: auto
[10/12/2019, 7:09:29 PM] [Thermostat] Logged into tcc! [ { deviceID: '5192195',
name: 'Thermostat',
usePermanentHolds: false } ]
[10/12/2019, 7:09:30 PM] [Thermostat] Adding TCC Device Thermostat 5192195
[10/12/2019, 7:09:30 PM] [Thermostat] Initializing platform accessory 'Thermostat'...
[10/12/2019, 7:09:30 PM] [Thermostat] getServices Thermostat
Setup Payload:
X-HM://0023ISYWY4BJA
Scan this code with your HomeKit app on your iOS device to pair with Homebridge:
`

Thanks for commenting. What robot do you use?

I've got the Botvac D3 Connected. As far as I understand it's very similar to the D4 and D5.

@maydaydarkhorse do you have the D3 as well?

@chrisaiello Do you have a floor plan saved in the map where you can set nogo lines?

I’ve only had it for a couple days, so I haven’t set up the saved floor plan yet.

Ok then I know the problem and will fix it asap.

Hi guys, I have a D5 and no floor plan saved either.  I will see if adding a floorpan fixes anything.

Should be fixed in v0.7.1-beta.1
You can install the beta version with npm install -g homebridge-neato@beta

Looks good so far, thanks for the quick fix! I’ll let you know if I experience any problems.

Great, thanks for the feedback!

Now my Robot shows up but I can not control it at all - Homekit says no response.
I can not find any home bridge log files anywhere.... But I get this error in home bridge after trying to activate the robot.
Cannot update robot. Check if robot is online. TypeError: Cannot read property 'width' of null
(node:938) UnhandledPromiseRejectionWarning: TypeError: Cannot read property 'width' of null
at NeatoVacuumRobotAccessory.clean (/usr/lib/node_modules/homebridge-neato/accessories/neatoVacuumRobot.js:354:43)
at platform.updateRobot (/usr/lib/node_modules/homebridge-neato/accessories/neatoVacuumRobot.js:255:12)
at robot.device.getState (/usr/lib/node_modules/homebridge-neato/index.js:234:5)
at Robot. (/usr/lib/node_modules/homebridge-neato/node_modules/node-botvac/lib/robot.js:39:17)
at /usr/lib/node_modules/homebridge-neato/node_modules/node-botvac/lib/robot.js:506:17
at /usr/lib/node_modules/homebridge-neato/node_modules/node-botvac/lib/api.js:34:45
at process._tickCallback (internal/process/next_tick.js:68:7)
(node:938) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 1)
(node:938) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

What version of the plugin do you use, 0.7.0 or 0.7.1-beta.1?

Did you try the spot cleaning or normal cleaning?

HI!  I am on 0.7.1-beta.1 ofter upgrading from 0.6.3 to 0.7.0-Beta.10 (Which I then had to google some sort of home bridge json instructions because the install was failing)
I have tried a spot clean and the robot will come off the dock and return to it. But it does nothing when I trigger a "House Clean"

This is what I had to do to get the -10 beta to install...
npm init
This utility will walk you through creating a package.json file.
It only covers the most common items, and tries to guess sensible defaults.
See npm help json for definitive documentation on these fields
and exactly what they do.
Use npm install <pkg> afterwards to install a package and
save it as a dependency in the package.json file.
Press ^C at any time to quit.
package name: (pi)
version: (1.0.0)
description:
git repository:
keywords:
author:
license: (ISC)
About to write to /home/pi/package.json:
{
"name": "pi",
"version": "1.0.0",
"main": "index.js",
"dependencies": {
"node-botvac": "^0.4.0"
},
"devDependencies": {},
"scripts": {
"test": "echo "Error: no test specified" && exit 1"
},
"author": "",
"license": "ISC",
"description": ""
}
Is this OK? (yes) yes
pi@raspberrypi:~ $

@maydaydarkhorse That looks a bit strange. Usally you can uninstall and install packages (including betas) all day long.
However your bug should be fixed in v0.7.1-beta.2

G'day Again! That seems to have fixed it!
What was it? Was it because I don't have a map saved or something else?

Yes, the plugin had two consecutive bugs on robots without a map. It got pretty hard to integerate so many features with so many combinations of different robots and user setups. I’m glad it works now :) .

Released in v0.7.1


python3Packages.ruff-lsp: init at 0.0.24

Description of changes
Init https://github.com/charliermarsh/ruff-lsp package, there's another pr but it doesn't work for me and no activity for two months https://github.com/NixOS/nixpkgs/pull/207250
I'll be happy with any of the PRs merged.
cc @cpcloud @fabaff

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[x] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

There is also #216841.

ruff-lsp can be updated to 0.0.19 before merging.

@fabaff could you review this please?

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

4 packages failed to build:

python310Packages.ruff-lsp
python310Packages.ruff-lsp.dist
python311Packages.ruff-lsp
python311Packages.ruff-lsp.dist

Tests are failing:
       > +                  {'code': 'F821',
       > +                   'data': {'fix': None, 'noqa_row': 3},
       > +                   'message': 'Undefined name `x`',
       > +                   'range': {'end': {'character': 7, 'line': 2},
       > +                             'start': {'character': 6, 'line': 2}},
       > +                   'severity': 1,
       > +                   'source': 'Ruff'}],
       > - {'diagnostics': [], 'uri': 'file:///build/tmp7_ngkmwz.py'}
       > ? -------------------
       >
       > +  'uri': 'file:///build/tmp7_ngkmwz.py'}
       >
       > ----------------------------------------------------------------------
       > Ran 2 tests in 0.828s
       >
       > FAILED (failures=2)
       > /nix/store/5s1yg5l36wzgy1dj0vv1ibarc4g7vrdr-stdenv-linux/setup: line 1594: pop_var_context: head of shell_variables not a function context
       For full logs, run 'nix log /nix/store/sf96dvqm6k82hz8qvfpq8cdhvamsr76s-python3.10-ruff-lsp-0.0.24.drv'.
error: builder for '/nix/store/2y40r03xspznxfqr8pd968i6i3mvn1zf-python3.11-ruff-lsp-0.0.24.drv' failed with exit code 1;
+                   'range': {'end': {'character': 10, 'line': 0},
+                             'start': {'character': 7, 'line': 0}},
+                   'severity': 2,
+                   'source': 'Ruff',
+                   'tags': [1]},
+                  {'code': 'F821',
+                   'data': {'fix': None, 'noqa_row': 3},
+                   'message': 'Undefined name `x`',
+                   'range': {'end': {'character': 7, 'line': 2},
+                             'start': {'character': 6, 'line': 2}},
+                   'severity': 1,
+                   'source': 'Ruff'}],
- {'diagnostics': [], 'uri': 'file:///build/tmp7mmxyo8r.py'}
? -------------------

+  'uri': 'file:///build/tmp7mmxyo8r.py'}

======================================================================
FAIL: test_no_initialization_options (tests.test_server.TestServer.test_no_initialization_options)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/build/ruff_lsp-0.0.24/tests/test_server.py", line 181, in test_no_initialization_options
    self.assertEqual(actual, expected)
AssertionError: {'uri[46 chars]s': []} != {'uri[46 chars]s': [{'range': {'start': {'line': 0, 'characte[533 chars]f'}]}
+ {'diagnostics': [{'code': 'F401',
+                   'data': {'fix': {'edits': [{'content': '',
+                                               'end_location': {'column': 0,
+                                                                'row': 2},
+                                               'location': {'column': 0,
+                                                            'row': 1}}],
+                                    'message': 'Remove unused import: `sys`'},
+                            'noqa_row': 1},
+                   'message': '`sys` imported but unused',
+                   'range': {'end': {'character': 10, 'line': 0},
+                             'start': {'character': 7, 'line': 0}},
+                   'severity': 2,
+                   'source': 'Ruff',
+                   'tags': [1]},
+                  {'code': 'F821',
+                   'data': {'fix': None, 'noqa_row': 3},
+                   'message': 'Undefined name `x`',
+                   'range': {'end': {'character': 7, 'line': 2},
+                             'start': {'character': 6, 'line': 2}},
+                   'severity': 1,
+                   'source': 'Ruff'}],
- {'diagnostics': [], 'uri': 'file:///build/tmppxpuymvg.py'}
? -------------------

+  'uri': 'file:///build/tmppxpuymvg.py'}

----------------------------------------------------------------------
Ran 2 tests in 0.722s

FAILED (failures=2)

@ofborg build python3Packages.ruff-lsp python3Packages.ruff-lsp.passthru.tests

Result of nixpkgs-review pr 214720 run on aarch64-darwin 1

4 packages built:

python310Packages.ruff-lsp
python310Packages.ruff-lsp.dist
python311Packages.ruff-lsp
python311Packages.ruff-lsp.dist

Will investigate linux build when I have time

I tested locally and I don't understand why the test is failing.
The ruff binary is clearly present.
Running
ruff --version
python -m ruff_lsp --version

in the preCheck phase works fine.
However, when running the LSP from the test_server.py file to analyze
import sys

print(x)

no diagnostic is returned, so the test fails.
It is even weirder if the same test succeeds on your mac.
Note: Running the test manually from the repo works fine.
git clone https://github.com/charliermarsh/ruff-lsp.git
cd ruff-lsp
python -m venv venv
source venv/bin/activate
just install
just test

nix-build -A python3Packages.ruff-lsp --option sandbox false does succeed on my aarch64-linux machine.

nix-build -A python3Packages.ruff-lsp --option sandbox false does succeed on my aarch64-linux machine. So, disable the tests for now?

The same command is failing on my x86_64-linux machine.
However, the ofborg build seems to fail for both x86_64-linux and aarch64-linux.
Do you know why @wegank ?
I don't mind disabling the tests, but it might be worth it to test it in real life condition then.

I thought it was a sandbox issue, since sandbox is usually enabled on Linux and disabled on macOS. If this is the case, I guess there's nothing we can do in the Nix expressions to work around the failure.

I thought it was due to sandbox, since sandbox is usually enabled on Linux and disabled on macOS. Which implies, for example, that network access is only disabled under Linux.
If this is the case, I guess there's nothing we can do in the Nix expressions to work around the failure.

Does it work for you without disabling sandbox ?

Does it work for you without disabling sandbox ?

Nope.

Nope.

I tested on two x86_64-linux systems (one is NixOS, the other one is Ubuntu 22.04) and the tests fail on both systems with and without sandbox...

Well @kalekseev, we might then want to add
doCheck = stdenv.isDarwin; then. Or maybe disable them completely.

Well @kalekseev, we might then want to add
doCheck = stdenv.isDarwin; then. Or maybe disable them completely.

I'll review and update today or tomorrow, thanks for your help!

Pushed doCheck=isDarwin.
On my aarch64-linux tests fails in build but passes in nix-shell so I guess it some env problem, not a broken build.
[nix-shell:~/nixpkgs/ruff_lsp-0.0.24]$ unittestCheckPhase
Executing unittestCheckPhase
..
----------------------------------------------------------------------
Ran 2 tests in 0.516s

OK
Finished executing unittestCheckPhase


Always open README with utf-8 encoding

This is another regression in 0.14.0
Fixes: https://github.com/crsmithdev/arrow/issues/600

Codecov Report

Merging #601 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master   #601   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           7      7
  Lines        1501   1501
  Branches      225    225
=====================================
  Hits         1501   1501

Continue to review full report at Codecov.

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

@andrewmwilson  Looks good. Any difference between with codecs.open and with open("README.rst", "r", encoding="utf-8") as f:?

@jadchaar I don't think the encoding parameter exists in Python 2.
There is also io.open in Python 2.6+.

You are very right! Also, mind adding encoding='utf-8' to the reading of init.py?
Then we can get this merged in.

Done!

Thanks for the contribution @andrewmwilson. Apologies for these regressions; hopefully this will be the last one :).


Does not work with Register-PSRepository (PowerShell 7.2.3, Ubuntu 22.04.1)

Ultimately I want this to work on an Azure DevOps agent running Ubuntu, but I'm testing on Ubuntu 22.04.1 LTS running under WSL2 on Windows 11 Pro.
I have configured the Microsoft package repositories and pinned all Microsoft packages to install from the Microsoft repos as described here
I have installed the .NET 6 SDK and PowerShell:
sudo apt install dotnet-sdk-6.0 powershell

I have installed the credential provider in the recommended way:
wget -qO- https://aka.ms/install-artifacts-credprovider.sh | bash

In PowerShell, I have configured the provider as described for running unattended in an Azure DevOps runner:
$env:VSS_NUGET_EXTERNAL_FEED_ENDPOINTS = '{"endpointCredentials": [{"endpoint":"https://pkgs.dev.azure.com/<<our org name>>/MAIN/_packaging/<<our feed name>>/nuget/v2", "username":"AzurePipelines", "password":"<<Azure Artifacts PAT>>"}]}'

I've configured Register-PSRepository using splatting:
PS > $options
Name                           Value
----                           -----
SourceLocation                 https://pkgs.dev.azure.com/autobooksrepo/MAIN/_packaging/AbPowerShell/nuget/v2
InstallationPolicy             Trusted
name                           AbPowerShell
PublishLocation                https://pkgs.dev.azure.com/autobooksrepo/MAIN/_packaging/AbPowerShell/nuget/v2
Verbose                        True

When I run Register-PSRepository @options, I get the following.  Note that this works correctly on Windows in PowerShell 7.3:
PS > register-PSRepository @options
Could not execute because the specified command or file was not found.
Possible reasons for this include:
  * You misspelled a built-in dotnet command.
  * You intended to execute a .NET program, but dotnet--V does not exist.
  * You intended to run a global tool, but a dotnet-prefixed executable with this name could not be found on the PATH.
Could not execute because the specified command or file was not found.
Possible reasons for this include:
  * You misspelled a built-in dotnet command.
  * You intended to execute a .NET program, but dotnet--V does not exist.
  * You intended to run a global tool, but a dotnet-prefixed executable with this name could not be found on the PATH.
VERBOSE: PowerShellGet Calling 'CallCredProvider' on https://pkgs.dev.azure.com/autobooksrepo/MAIN/_packaging/AbPowerShell/nuget/v2
Could not execute because the specified command or file was not found.
Possible reasons for this include:
  * You misspelled a built-in dotnet command.
  * You intended to execute a .NET program, but dotnet--V does not exist.
  * You intended to run a global tool, but a dotnet-prefixed executable with this name could not be found on the PATH.
Could not execute because the specified command or file was not found.
Possible reasons for this include:
  * You misspelled a built-in dotnet command.
  * You intended to execute a .NET program, but dotnet--V does not exist.
  * You intended to run a global tool, but a dotnet-prefixed executable with this name could not be found on the PATH.
 Are you missing 'Credential' parameter in the cmdlet?
VERBOSE: Suppressed Warning Unable to resolve package source 'https://pkgs.dev.azure.com/autobooksrepo/MAIN/_packaging/AbPowerShell/nuget/v2'.
VERBOSE: Repository details, Name = 'PSGallery', Location = 'https://www.powershellgallery.com/api/v2'; IsTrusted = 'False'; IsRegistered = 'True'.
VERBOSE: Performing the operation "Register Module Repository." on target "Module Repository 'AbPowerShell' (https://pkgs.dev.azure.com/autobooksrepo/MAIN/_packaging/AbPowerShell/nuget/v2) in provider 'PowerShellGet'.".
VERBOSE: User did not specify the PackageManagement provider name, trying with the provider name 'NuGet'.
VERBOSE: Successfully registered the repository 'AbPowerShell' with source location 'https://pkgs.dev.azure.com/autobooksrepo/MAIN/_packaging/AbPowerShell/nuget/v2'.
VERBOSE: Repository details, Name = 'AbPowerShell', Location = 'https://pkgs.dev.azure.com/autobooksrepo/MAIN/_packaging/AbPowerShell/nuget/v2'; IsTrusted = 'True'; IsRegistered = 'True'.
Could not execute because the specified command or file was not found.
Possible reasons for this include:
  * You misspelled a built-in dotnet command.
  * You intended to execute a .NET program, but dotnet--V does not exist.
  * You intended to run a global tool, but a dotnet-prefixed executable with this name could not be found on the PATH.
PS /home/cdoherty> Could not execute because the specified command or file was not found.
Possible reasons for this include:
  * You misspelled a built-in dotnet command.
  * You intended to execute a .NET program, but dotnet--V does not exist.
  * You intended to run a global tool, but a dotnet-prefixed executable with this name could not be found on the PATH.
  * ```

PowerShellGet 2.2.5 requires NuGet v2 feeds; PowerShellGet 3.0 does not yet work with Azure Artifacts v3 feeds.

Hi is there anyone looking into this case?
This case is not about the environment of Azure DevOps side, but it's totally about the local environment behavior with powershell of this Azure Artifact Credential Provider tool. Why would you redirect it to Developer Community!!??

The bot closes all issues and punts them to the "Developer Community".  I re-posted the bug there and the support droid told me to post it here.
It's pretty obvious this project has been abandoned. It's worth checking out other artifact repository services.


🐞 Unable to find text in file.

Description
Unable to find text within an open file.
To Reproduce

Open a file with CodeEdit. It will show the file in its own window.
Hit cmd+f, or open the find in the menubar, and everything is grayed out.

Expected Behavior
I expected to be able to find text within a file. There was no way to find text in many file windows.
Version Information
CodeEdit: Version 0.3.2-alpha (43)
macOS: 15.2
Xcode: 16.2
Additional Context

Screenshots

Duplicate of https://github.com/CodeEditApp/CodeEditTextView/issues/1. Closing.


Not working with Create React App

import Particles from 'react-particles-js';
Inside my render function, I'm returning just the Particles component.
When I run the React app, nothing shows up. No errors in console either.
Any ideas what could be happening?

Nevermind. Fixed.

would you mind telling us how do you fix that


test/test_http.py: stop HTTP server (thread) properly

test/test_http.py: stop HTTP server (thread) properly
Description
Hello,
The HttpLayerTestCase test class creates a HTTPServer instance and runs it in a separate thread.
After running all the test-cases, we attempt to close the server and the thread by killing the thread. Unfortunately, the code that does that is unreachable because the tear-down member function is called tearDownCls intead of tearDownClass [1].
Moreover, there is no threading.Thread.kill function. This was undetected because the code was unreachable.
The proper way to clean things up is to:

Stop the HTTP server using HTTPServer.shutdown()
Join the thread using threading.Thread.join()

Thanks!
[1] https://docs.python.org/3/library/unittest.html#unittest.TestCase.tearDownClass
Status

done, ready for review

Checklist (tick everything that applies)

[x] Code linting (required, can be done after the PR checks)
[ ] Documentation
[ ] Tests changes are in the tests only
[ ] ICLA (required for bigger changes)

Thanks for the contribution and a good catch.
The change looks good to me, but funny enough, I get this warning with your change, but I don't get it without it:
sys:1: ResourceWarning: unclosed <socket.socket fd=3, family=AddressFamily.AF_INET, type=SocketKind.SOCK_STREAM, proto=0, laddr=('<IP_ADDRESS>', 57898)>

Will do some more digging in. It may actually be related to outgoing client connection.

Added server.socket.close() and merged it into trunk, thanks!


Add ReSharper support to the Razor view engine

Previously, ReSharper was unable to resolve views when using Orchard. Someone from the R# team recommended a fix to allow the resolution of views.
Here's the R# recommendation, for reference: http://spermophilus2.rssing.com/browser.php?indx=3176698&item=47964

Look good to me.
Can you file a bug to link this PR to, so that we can track it has been fixed?

Sure thing. Here's the issue:  #6815


Modification of footer

Feedback:

Please do not change the color of the navigation bar from black to gray. Gray does not go with white at all!

^^ The toggle button is poorly messed up. Please be cautious of the changes you do

The color of the social media icons in the home page should not be changed

Reduce the size of the contact form. It is bloating out a lot! 😅

Remove that white line between the contact form and the left side of the footer

The send button is never the size of the input fields

Feedback:

Please do not change the color of the navigation bar from black to gray. Gray does not go with white at all!

^^ The toggle button is poorly messed up. Please be cautious of the changes you do

The color of the social media icons in the home page should not be changed

Reduce the size of the contact form. It is bloating out a lot! 😅

Remove that white line between the contact form and the left side of the footer

The send button is never the size of the input fields

To resolve the above conflicts you need to pull upstream master.  use git version control for that,
you can also follow the steps mentioned in the readme file.

To resolve the above conflicts you need to pull upstream master.  use git version control for that,
you can also follow the steps mentioned in the readme file.


Bug: [Reanimated] onStoryStart

Hi folks, im having a bug when I pass onStoryStart to props and try to go back to previous story, I was using just to print some ids, but even with empty callback it keeps crashing. To next stories it works as expected, but when u hit to previous this crash happens:

Hello, try 1.2.2 and let us know, thanks for the announcement.


Wrong parameter order in OssGadgetLib.cs?

I am reading the source code of oss-find-source. Although I am not familiar with C#, after checking the API reference of Check.NotNull, I think this line of code may have wrong order of nameof(projectManagerFactory) and projectManagerFactory?
https://github.com/microsoft/OSSGadget/blob/ebd2e46ccf880491910dec33530dfe067df9e0cb/src/Shared/OssGadgetLib.cs#L35

I suspect this is already closed because you discovered this on your own. Just for documentation purposes, the reason this isn't an issue is because this call to Check.NotNull is implemented in OssGadget itself with a different parameter pattern than the azure one linked above: https://github.com/microsoft/OSSGadget/blob/4c84c9100eb0684ab7a7484d1755bc86a70ad008/src/Shared/Helpers/Check.cs#L46


as a user, I don't known that ACME HTTP01 solvers cannot be used for wildcard certs

I wanted to obtain a wildcard Let's Encrypt certificate and setup the issuer with http01: { ingress: { class: "contour" }.
When I inspected the order, all I could see was that it's trying to use DNS01, and the error message was:
Failed to determine a valid solver configuration for the set of domains on the Order: no configured challenge solvers can be used for this challenge

It would be nicer if the error message contained more specific reason, i.e. that wildcard certs can only be issued only on the basis of DNS01 challenges, or whatever most readable formulation would be. I also couldn't see anything in the logs, and didn't know where to look, but I did get a very quick response from @meyskens on Slack.

This is a Let's Encrypt limitation. If you use another ACME server you might not run into such issues, as the ACME protocol itself does not prevent it.

We could add a documentation entry in here or make the error more useful like adding ACME server responed with challanges for type DNS-01 which would be nicer.
/area acme
/kind feature
/priority important-longterm

Yes, I think it is just the error message that needs to be more explicit. I
am not questioning the nature of the problem. Adding this to the docs would
be insufficient IMO.
On Mon, 17 Aug 2020, 8:08 am jetstack-bot<EMAIL_ADDRESS>wrote:

@meyskens https://github.com/meyskens: The label(s) kind/ cannot be
applied, because the repository doesn't have them
In response to this
https://github.com/jetstack/cert-manager/issues/3181#issuecomment-674702905
:
We could add a documentation entry in here or make the error more useful
like adding ACME server responed with challanges for type DNS-01 which
would be nicer.
/area acme
/kind feature
/priority important-longterm
Instructions for interacting with me using PR comments are available here
https://git.k8s.io/community/contributors/guide/pull-requests.md. If
you have questions or suggestions related to my behavior, please file an
issue against the kubernetes/test-infra
https://github.com/kubernetes/test-infra/issues/new?title=Prow issue:
repository.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/jetstack/cert-manager/issues/3181#issuecomment-674702917,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAB5MS4XGR7PGPFPRVYMFUTSBDJOJANCNFSM4P7PXJMA
.


DBL_MAX not defined!

https://github.com/vim/vim/blob/master/src/ex_cmds.c#L560
ex_cmds.c:                  nrs[lnum - eap->line1].st_u.value_flt = -DBL_MAX;
DBL_MAX is not defined in the entire project!

Please check issues before creating new ones #571 #570, closing as duplicate


Template should write the first commit message as "feat: initial commit"

Currently when using the template, it produces the first commit message in the created repo as "Initial commit". My proposal is to change it to "feat: initial commit", so as to remain consistent with the conventional commits specification.

I actually don't know how to do this. I asked for help on StackOverflow, and I also contacted GitHub via their support channel. I will follow up here once I find out the solution - if there is one!

I error to
⧗   input: Hello World!
✖   subject may not be empty [subject-empty]
✖   type may not be empty [type-empty]
✖   found 2 problems, 0 warnings
ⓘ   Get help: https://github.com/conventional-changelog/commitlint/#what-is-commitlint
husky > commit-msg hook failed (add --no-verify to bypass)

@zinderud this template enforces conventional commits. You can read the full specification here.
You cannot write "Hello World" as a commit message - it will have to be "feat: hello world".

I have an idea where by using a GitHub action that runs just once when the template is used for a new repo.
We could replace some values like repo links and maybe replace the package.json author and other details like the badges in the readme.

Any thoughts?

Ah, this would very interesting. Great idea @Janther. We could take inspiration from the femplate Foundry template on how do to the initial renaming.


Is azure's open source project dotnetty still under maintenance?

Hello,azure community
Please forgive me for submitting a issue here, I can't think of a better solution for dotnetty Author to answer our issues. Dotnetty is not maintained, no one has merged the request, reply to the issue, based on the dotnetty ecosystem has been established, Do not want dotnetty to die like this, my open source project surging is also based on dotnetty, wait for your reply, please do not give up after open source, so Will make us cool heart

@nayato @mikelchai  Can either of you take a look at this issue and redirect it appropriately?
Not marking this as a Cliwnt/Mgmt issue as it is neither.

We're sorry but this repo is only tracking Azure SDK work.