MAINT: 1.3.0 prep / safe backport

Hopefully pytest doesn't have a fit without backporting the warnings stuff. Beyond that, the changes are summarized below. I went through the backport labels and only 1 seemed appropriate & not already present for the 1.3.x branch at this point close to 1.3.0 "final" release.
If something is missing / looks wrong let me know--way cheaper to fix now vs. after release.

set version to 1.3.0 unreleased
(remove the rc2)

update release notes with a backward
incompatible change entry for scipy.optimize
as requested in gh-10160; remove note
about 1.3.0 not being released yet (now imminent
pending release commit) Fixes #10160

backport #10182

LGTM. I'll wait with merging to give @mdhaber or @Kai-Striega the chance to check the linprog note.

The notes look good to me.
I'm not sure if it's worth adding to the release notes, but, it may be worth mentioning that interior-point does not necessarily end on a vertex while discussing some of the pros/cons.

If we're adding more detail, perhaps add a sentence to make clear that the change is a major improvement overall. Otherwise the reader may ask why this is done in the first place if it comes with so many disclaimers.

Yes, I don't think it's worth adding, as it relates to the existing
interior-point solver. But I thought it would be worth mentioning in the
discussion in case anyone thinks differently.
On Thu, 16 May 2019 at 18:00, Ralf Gommers<EMAIL_ADDRESS>wrote:

If we're adding more detail, perhaps add a sentence to make clear that the
change is a major improvement overall. Otherwise the reader may ask why
this is done in the first place if it comes with so many disclaimers.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/scipy/scipy/pull/10185?email_source=notifications&email_token=AHFOJNNMAN52TJRTVJ6DTD3PVUWC3A5CNFSM4HNILIW2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODVRKEFY#issuecomment-493003287,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHFOJNKU42WINMEGYRBKIP3PVUWC3ANCNFSM4HNILIWQ
.

Ok, sounds like I can get away without expanding the release note even more, so I think I'll do that :)


Error caused by exports field in package.json

Describe the bug
Using the latest version of the package (v2.0.4) along with esbuild and Next.js v13 this error pops up in the console when trying to import any icon from the library.
Package path . is not exported from package /my_pc_project_path/node_modules/@phosphor-icons/react (see exports field in<EMAIL_ADDRESS>
Looking up the path reported I can see this entry
// other stuff...
"exports": {
  ".": {
    "import": "./dist/index.es.js"
  }
},
// more stuff...

To Reproduce
It is quite the hassle, it invlolves setting up a monorepo with Turborepo and creating different apps and packages. If necessary I can provide a link to my repository.
Expected behavior
I expect the app to work without throwing any errors both in developement and production.
Desktop:

OS: Fedora Linux 36 (Workstation Edition)
Browser: Brave
Version: Version 1.48.164 Chromium: 110.0.5481.100 (Official Build) (64-bit)

Transferring issue to @phosphor-icons/react, since it pertains to this library only.

Glad to hear you got it resolved. It sounds like the issue is actually that we don't provide a CJS export anymore 😟

Out of curiosity, why is that? I mean, why don't you provide a CJS export anymore? Is it just a decision to move towards an ESM future or is there a technical issue that prevents you from being able to provide both?

Any solution?(

The solution rn is to convert your package type from commonjs to module in your package.json.
Once you do that you can then add
"module": "nodenext",

in your tsconfig file. This should fix the issue.
If for some reason you can't convert to a module type package you can remove the exports field from the library's package.json present in the node_modules directory, but that's just a temporary fix.

I use js. Instead of ts config, i added in jsconfig
This solution didn't work for me(

From VS Code's jsconfig.json reference page

Tip: jsconfig.json is a descendant of tsconfig.json, which is a configuration file for TypeScript. jsconfig.json is tsconfig.json with "allowJs" attribute set to true.

So you should still be able to declare this in there
"module": "nodenext",

Does anyone know what the solution for this is? I am also getting this issue when using Phosphor Icons with Sanity

Nope, it goes either in your tsconfig.json or jsconfig.json file.
This is necessary if you're trying to use an ESM-only package, and @phosphor-icons/react became ESM-only after v2.

As I mentioned above that only works if you have set your package type to "module" instead of "commonjs" in your package.json file.
But this isn't possible for every type of application/package. Sanity requires you to use common js and switching is quite the hassle. I've tried doing that in the past with various degrees of success and you have to make compromises whichever way you chose to go.

I've just published v2.0.7, which includes a UMD export (compatible with CJS build tooling), and does not specify "type": "module" in package.json. This should alleviate all of the build issues mentioned here. As a side-effect of the build, we won't support any qualified imports in that environment:
import AddressBook from "@phosphor-icons/react/AddressBook"; // CAN'T DO THIS IN CJS ENVIRONMENT
import { AddressBook } from "@phosphor-icons/react"; // CAN DO THIS IN EITHER ENVIRONMENT

I also recommend using the fully qualified path if you're doing modular imports (@phosphor-icons/react/dist/icons/Foo)

@rektdeckard I just testing the latest update of @phosphor-icons/react with Sanity (latest version 3.8.2) and its working great! No issues are logged when deploying graphql or with importing icons into react templates!

Any other issues here, or safe to close?

Everything is all good for me. Seems like the original issue is resolved as well @milovangudelj?

I haven't had the chance to try it out yet but judging from what I've read so far it seems resolved. I'll close the issue for now.
Feel free to reopen it if you find more errors related to this.


(feat) TELOS integration

Before submitting this PR, please make sure:

[x] Your code builds clean without any errors or warnings
[x] You are using approved title ("feat/", "fix/", "docs/", "refactor/")

A description of the changes proposed in the pull request:

Added integration with TELOS.
Added a supported tokens list for TELOS.
Added support for TELOS in OpenOcean.
Related Hummingbot PR: https://github.com/hummingbot/hummingbot/pull/7119

Tests performed by the developer:

Tested by running locally and making requests using curl.
Tested with Hummingbot
Added tests for TELOS routes in OpenOcean.

Tips for QA testing:

Test performed:

Created telos wallet, added deposit
connected openocean telos evm wallet successfully
review gateway balance is correct
added WTLOS,USDC to connector tokens
review tokens added and show correct balance
approve token (USDC) through the client - failed, have to approved manually
amm_arb created successfully using openocean telos evm
started and review status showed correct prices
after some time got error price quiery request failed, same on curl

retested this morning:

wallet/add: ok
amm/price: ok
amm/trade: ok

https://www.teloscan.io/tx/0x6b55a4b7f081e36f520a0d38693e244fa016097aa5fe19472230c95696e5da01
https://www.teloscan.io/tx/0xa04145f51ab8ae805e2c28e901f2cb4ad41825c9c7c08102221e763250db2113


experiment with reporting ingress updates

I wanted to investigate the bugs I heard about ingress not working after an undeploy/deploy and look at reporting the ingress IPs after deploying with ingress.
Running this code showed me that after deploying it can take a while to actually assign an IP. K8s docs say it can take "up to a minute or two". So I am thinking we want to wait for it to take and maybe show some progress when running in interactive mode. We can put it in here like I have but I was thinking it seems like it's time to add a StatusStep to the end of DeployCommand.cs that has a set of things we check on and report after deployment has happened.
What do you think?

Fix the formatting!

But it's a meme at this point, if I don't submit formatting errors y'all will think something is wrong.

@glennc I assume you won't have time to finish this PR, want one of us to take it over?

@glennc I assume you won't have time to finish this PR, want one of us to take it over?

@glennc I assume you won't have time to finish this PR, want one of us to take it over?

@glennc I assume you won't have time to finish this PR, want one of us to take it over?


Missing content from generated HAR

I'm using the code from the example to generate a HAR for a specific page. The generated HAR is  ~40 KB. When I use browsermob-proxy in python for the same page the generated HAR is ~1.5 MB. At least 1 html file is missing from the generated HAR from puppeteer-har.
Why is this? Is there a maybe an option to avoid it?
Even if use a wait function like in https://github.com/Everettss/puppeteer-har/issues/5, I'm getting the same result.

Hey @hubitor, did you try removing the asynchronous parts from your script or get it to work in the end?

Hey @kendfss : It's been a long a time since then. I don't remember anymore, but I think it didn't work since I would have reported it here.

await this.har.start({ path: `${flow.title}.har`, saveResponse: true, captureMimeTypes: ['application/font-sfnt','application/octet-stream', 'image/svg+xml', 'application/json','application/json; charset=UTF-8'] });

With this I get all answers


Updated AW API Group to codeflare.dev

Issue link
#272
What changes have been made
API groups for AW changed from mcad.ibm.com to codeflare.dev
Verification steps
Verify no remaining files with mcad.ibm.com, and all have switched to codeflare.dev
Checks

[x] I've made sure the tests are passing.
Testing Strategy

[x] Unit tests
[ ] Manual tests
[ ] Testing is not required for this change

/approve


Post subscribers to list - unknown parameter ConsentToTrack

Hi there,
I have tried to post new subscribes to a list using the node.js wrapper and kept getting unknown parameter ConsentToTrack. This would happen every time I pass a value to the key ConsentToTrack. If I leave the value as an empty string, the POST call succeeds but that's of no use for me as I need to track users. Anyhow, I've contacted Campaign Monitor and this was their reply:

In the last few months we updated our API to v 3.2, and as such, all of our own wrappers were updated as well: https://www.campaignmonitor.com/api/wrappers/ However, as you can see there, we don't actually manage the Node wrapper solution, so it's most likely not been updated to v3.2, which would mean it doesn't support the newer Consent to Track field. You'd either need to use one of our own supported wrappers, or manage that consent to track field in app or via list imports.

This is the code I was using:
`const createsend = require('createsend-node');
const auth = { apiKey: 'api-key'};
const api = new createsend(auth);
const userName = user_first_name;
const userEmail = user_email;
const listId = 'list-id'
const details = {
EmailAddress: userEmail,
Name: userName,
"ConsentToTrack": "Unchanged"
};
api.subscribers.addSubscriber(listId, details, (err, res) => {
if (err) console.log(err);
console.log(res);
})`
Not sure it's an issue but worth flagging it here. Your feedback is much appreciated.
Alin
P.s. I got it working with the Python wrapper but I'd very much prefer node.js :D

Hi Alin, I know its an old post but i got it to work with node.js by upgrading the version in lib/createsend.js on line 30 to version: 'v3.2',

Thank you, Tolkien. Sounds great and seems to do the trick. I'll close the issue. Cheers!

Sorry about posting to old ticket, but this is the only result in Google and maybe this helps someone.
So, I had the same problem, but with PHP.
Solution, if you can't update wrapper or you have custom code:
The root cause is that their old API stopped supporting this parameter.
Search your code for CM API URL and replace old URL (eg https://api.createsend.com/api/v3.1/) with new one: https://api.createsend.com/api/v3.2/

And I too just ran into this same error on Drupal 7. Turns out the library was old. Grabbed the latest from https://github.com/campaignmonitor/createsend-php and now I can see new entries on the lists. Thanks to @raulir for the tip.


11/02/2018 05:53 : Create Pull Request for branch prTesting_2018110205 -> master

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details
Dependent Files

split/updateTestCon.md
:white_check_mark:Succeeded

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


drf_yasg errors when you attempt to start label_studio for all versions> 1.4.1 and  < 1.6.0

Attempting to run label-studio with the version information available in the dockerfile and requirements.txt causes the error below
=> Database and media directory: /home/gitpod/.local/share/label-studio => Static URL is set to: /static/ Traceback (most recent call last):   File "/home/gitpod/.pyenv/versions/3.8.16/bin/label-studio", line 8, in <module>     sys.exit(main())   File "/home/gitpod/.pyenv/versions/3.8.16/lib/python3.8/site-packages/label_studio/server.py", line 282, in main     _setup_env()   File "/home/gitpod/.pyenv/versions/3.8.16/lib/python3.8/site-packages/label_studio/server.py", line 40, in _setup_env     application = get_wsgi_application()   File "/home/gitpod/.pyenv/versions/3.8.16/lib/python3.8/site-packages/django/core/wsgi.py", line 12, in get_wsgi_application     django.setup(set_prefix=False)   File "/home/gitpod/.pyenv/versions/3.8.16/lib/python3.8/site-packages/django/__init__.py", line 19, in setup     configure_logging(settings.LOGGING_CONFIG, settings.LOGGING)   File "/home/gitpod/.pyenv/versions/3.8.16/lib/python3.8/site-packages/django/conf/__init__.py", line 82, in __getattr__     self._setup(name)   File "/home/gitpod/.pyenv/versions/3.8.16/lib/python3.8/site-packages/django/conf/__init__.py", line 69, in _setup     self._wrapped = Settings(settings_module)   File "/home/gitpod/.pyenv/versions/3.8.16/lib/python3.8/site-packages/django/conf/__init__.py", line 170, in __init__     mod = importlib.import_module(self.SETTINGS_MODULE)   File "/home/gitpod/.pyenv/versions/3.8.16/lib/python3.8/importlib/__init__.py", line 127, in import_module     return _bootstrap._gcd_import(name[level:], package, level)   File "<frozen importlib._bootstrap>", line 1014, in _gcd_import   File "<frozen importlib._bootstrap>", line 991, in _find_and_load   File "<frozen importlib._bootstrap>", line 975, in _find_and_load_unlocked   File "<frozen importlib._bootstrap>", line 671, in _load_unlocked   File "<frozen importlib._bootstrap_external>", line 843, in exec_module   File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed   File "/home/gitpod/.pyenv/versions/3.8.16/lib/python3.8/site-packages/label_studio/core/settings/label_studio.py", line 43, in <module>     from label_studio.core.utils.common import collect_versions   File "/home/gitpod/.pyenv/versions/3.8.16/lib/python3.8/site-packages/label_studio/core/utils/common.py", line 36, in <module> Starting new HTTPS connection (1): o227124.ingest.sentry.io:443     from drf_yasg.inspectors import CoreAPICompatInspector, NotHandled   File "/home/gitpod/.pyenv/versions/3.8.16/lib/python3.8/site-packages/drf_yasg/inspectors/__init__.py", line 5, in <module>     from .field import (   File "/home/gitpod/.pyenv/versions/3.8.16/lib/python3.8/site-packages/drf_yasg/inspectors/field.py", line 406, in <module>     (serializers.NullBooleanField, (openapi.TYPE_BOOLEAN, None)), AttributeError: module 'rest_framework.serializers' has no attribute 'NullBooleanField' Sentry is attempting to send 2 pending error messages Waiting up to 2 seconds Press Ctrl-C to quit [https://o227124.ingest.sentry.io:443](https://o227124.ingest.sentry.io/) "POST /api/5820521/store/ HTTP/1.1" 200 41

See fix here: https://github.com/mhw32/data-centric-deep-learning/pull/8


Refactor cert contents

This is currently on top of my existing PR to move mounting the certs, keys and cacerts to an "install-certs" service.
We also:

modify the service spec attributes to parametrize the contents of the cert
the tlsCertsEnabled attribute is now no longer needed and has been removed.
modified the doc to more accurately describe what is going on.

/test dataplane-operator-build-deploy-kuttl

@slagle Lets keep this simple and just merge this one.  Thanks!

recheck

/lgtm

recheck

/test dataplane-operator-build-deploy-kuttl

/lgtm

/test dataplane-operator-build-deploy-kuttl

/test dataplane-operator-build-deploy-kuttl

/lgtm

/test dataplane-operator-build-deploy-kuttl


fix(Accordion): 🐛 Fix hover state on Accordion.Header when used on touch devies

Button hover state are by default "active" when used on touch devices. This fix is to make it so that the hover styling is not stuck on touch devices after tapping to open an Accordion

Tested on Device Mode (mobile) in chrome, and the new functionality works as expected.

This will give you a false positive, you need to test on an actual mobile device or emulator 😅


WIP: nm: Persistent controller and parent connection to disk

When a NM connection's parent or controller is memory only connection,
next OS reboot will cause this interface not activated due to missing
parent/controller. To fix it, we persistent its controllers and parents.
Integration test cases included.
Ref: https://issues.redhat.com/browse/OCPBUGS-15132

Do not merge. This is not final code yet.

Closing as this approach will not fix original issue. We will fix the use case in #2571


Unable to process data

Hello!
I'm processing my 16S data but it's not working. I've added a set of metadata to my previous study
ID: 10942
Basically, I get the attached file when I try to process the data and then I click on the dflt_name-FASTQ and then process, and then I do
Split libraries —> golly_12; pared offset 33 —> pick_closed ref OTUs —> default_serial —> RUN
And the work is submitted,
But I never get any data to “add to analysis”
Thank you for the help!

You can select the data from the "View studies" option then add to
analysis.  I see data there so I am not sure why you don't.  After that you
select "create new analysis" .
On Sat, Jun 17, 2017 at 6:43 AM, rimanasra<EMAIL_ADDRESS>wrote:

I'm processing my 16S data but it's not working.
Basically, I get the attached file when I try to process the data and then
I click on the dflt_name-FASTQ and then process, and then I do
Split libraries —> golly_12; pared offset 33 —> pick_closed ref OTUs —>
default_serial —> RUN
And the work is submitted,
But I never get any data to “add to analysis”
[image: screen shot 2017-06-16 at 6 39 24 pm]
https://user-images.githubusercontent.com/28489284/27253378-68e4fe0e-5341-11e7-9d16-e0d211e46458.png
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/biocore/qiita/issues/2148, or mute the thread
https://github.com/notifications/unsubscribe-auth/AB69gN4D9jKW2LySpd54bOJWY0GkE5Irks5sE9gMgaJpZM4N9OpO
.

--
Gail Ackermann
Knight Lab
UCSD
<EMAIL_ADDRESS><EMAIL_ADDRESS>

Dear Gail,
The data present is the older set of data (the one that is not updated). So
basically, I had done analysis before and it worked, and now when I add the
metadata that is new, I re-run it but the data available to analyse is not
up to date.
Any ideas why?
Thank you,
Rima
On Sun, Jun 18, 2017 at 2:09 AM, Gail Ackermann<EMAIL_ADDRESS>wrote:

You can select the data from the "View studies" option then add to
analysis. I see data there so I am not sure why you don't. After that you
select "create new analysis" .
On Sat, Jun 17, 2017 at 6:43 AM, rimanasra<EMAIL_ADDRESS>wrote:

I'm processing my 16S data but it's not working.
Basically, I get the attached file when I try to process the data and
then
I click on the dflt_name-FASTQ and then process, and then I do
Split libraries —> golly_12; pared offset 33 —> pick_closed ref OTUs —>
default_serial —> RUN
And the work is submitted,
But I never get any data to “add to analysis”
[image: screen shot 2017-06-16 at 6 39 24 pm]
<https://user-images.githubusercontent.com/28489284/27253378-68e4fe0e-
5341-11e7-9d16-e0d211e46458.png>
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/biocore/qiita/issues/2148, or mute the thread
<https://github.com/notifications/unsubscribe-auth/
AB69gN4D9jKW2LySpd54bOJWY0GkE5Irks5sE9gMgaJpZM4N9OpO>
.

--
Gail Ackermann
Knight Lab
UCSD
<EMAIL_ADDRESS><EMAIL_ADDRESS>—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/biocore/qiita/issues/2148#issuecomment-309258997, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AbK2RORQ2vOiuFKcISx8-pHwBa85V6pOks5sFL8sgaJpZM4N9OpO
.

Looks like you tried to add age and age_units to the prep and got an error,
therefore only the old data (with no age) is what is being accessed.  Try
add to the Sample info (replacing the existing fields
Gail
On Mon, Jun 19, 2017 at 12:13 PM, rimanasra<EMAIL_ADDRESS>wrote:

Dear Gail,
The data present is the older set of data (the one that is not updated). So
basically, I had done analysis before and it worked, and now when I add the
metadata that is new, I re-run it but the data available to analyse is not
up to date.
Any ideas why?
Thank you,
Rima
On Sun, Jun 18, 2017 at 2:09 AM, Gail Ackermann<EMAIL_ADDRESS>wrote:

You can select the data from the "View studies" option then add to
analysis. I see data there so I am not sure why you don't. After that you
select "create new analysis" .
On Sat, Jun 17, 2017 at 6:43 AM, rimanasra<EMAIL_ADDRESS>wrote:

I'm processing my 16S data but it's not working.
Basically, I get the attached file when I try to process the data and
then
I click on the dflt_name-FASTQ and then process, and then I do
Split libraries —> golly_12; pared offset 33 —> pick_closed ref OTUs —>
default_serial —> RUN
And the work is submitted,
But I never get any data to “add to analysis”
[image: screen shot 2017-06-16 at 6 39 24 pm]
<https://user-images.githubusercontent.com/28489284/27253378-68e4fe0e-
5341-11e7-9d16-e0d211e46458.png>
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/biocore/qiita/issues/2148, or mute the thread
<https://github.com/notifications/unsubscribe-auth/
AB69gN4D9jKW2LySpd54bOJWY0GkE5Irks5sE9gMgaJpZM4N9OpO>
.

--
Gail Ackermann
Knight Lab
UCSD
<EMAIL_ADDRESS><EMAIL_ADDRESS>—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/biocore/qiita/issues/2148#issuecomment-309258997,
or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AbK2RORQ2vOiuFKcISx8-
pHwBa85V6pOks5sFL8sgaJpZM4N9OpO>
.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/biocore/qiita/issues/2148#issuecomment-309538348, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AB69gFk6PrWVBx0jammAoex4kRDyeY_qks5sFshCgaJpZM4N9OpO
.

--
Gail Ackermann
Knight Lab
UCSD
<EMAIL_ADDRESS><EMAIL_ADDRESS>

Thanks Gail, I will try and let you know.
How do I replace the existing fields?
Thank,
Rima
On Mon, Jun 19, 2017 at 11:22 PM, Gail Ackermann<EMAIL_ADDRESS>wrote:

Looks like you tried to add age and age_units to the prep and got an error,
therefore only the old data (with no age) is what is being accessed. Try
add to the Sample info (replacing the existing fields
Gail
On Mon, Jun 19, 2017 at 12:13 PM, rimanasra<EMAIL_ADDRESS>wrote:

Dear Gail,
The data present is the older set of data (the one that is not updated).
So
basically, I had done analysis before and it worked, and now when I add
the
metadata that is new, I re-run it but the data available to analyse is
not
up to date.
Any ideas why?
Thank you,
Rima
On Sun, Jun 18, 2017 at 2:09 AM, Gail Ackermann <
<EMAIL_ADDRESS>wrote:

You can select the data from the "View studies" option then add to
analysis. I see data there so I am not sure why you don't. After that
you
select "create new analysis" .
On Sat, Jun 17, 2017 at 6:43 AM, rimanasra<EMAIL_ADDRESS>wrote:

I'm processing my 16S data but it's not working.
Basically, I get the attached file when I try to process the data and
then
I click on the dflt_name-FASTQ and then process, and then I do
Split libraries —> golly_12; pared offset 33 —> pick_closed ref OTUs
—>
default_serial —> RUN
And the work is submitted,
But I never get any data to “add to analysis”
[image: screen shot 2017-06-16 at 6 39 24 pm]
<https://user-images.githubusercontent.com/
28489284/27253378-68e4fe0e-
5341-11e7-9d16-e0d211e46458.png>
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/biocore/qiita/issues/2148, or mute the thread
<https://github.com/notifications/unsubscribe-auth/
AB69gN4D9jKW2LySpd54bOJWY0GkE5Irks5sE9gMgaJpZM4N9OpO>
.

--
Gail Ackermann
Knight Lab
UCSD
<EMAIL_ADDRESS><EMAIL_ADDRESS>—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/biocore/qiita/issues/2148#issuecomment-309258997,
or mute
the thread
<https://github.com/notifications/unsubscribe-
auth/AbK2RORQ2vOiuFKcISx8-
pHwBa85V6pOks5sFL8sgaJpZM4N9OpO>
.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/biocore/qiita/issues/2148#issuecomment-309538348,
or mute
the thread
<https://github.com/notifications/unsubscribe-auth/
AB69gFk6PrWVBx0jammAoex4kRDyeY_qks5sFshCgaJpZM4N9OpO>

.

--
Gail Ackermann
Knight Lab
UCSD
<EMAIL_ADDRESS><EMAIL_ADDRESS>—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/biocore/qiita/issues/2148#issuecomment-309634015, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AbK2RLCtPm0iXykfdL4S-H1x7hFcFHyXks5sFzsKgaJpZM4N9OpO
.

closing as this kind of questions should go<EMAIL_ADDRESS>


Add utility classes for form input field erros

When using client-side JS validation, an html element, typically a div or span is inserted to the DOM to provide a visual cue to the user that the current field being entered is not a valid format. We need to add a helper class like 'nypl-required-field' to add styling to such fields.
CC: @ricardoom

solved in af3451d793af1c46388fe96e574d7e0025417378 and documented here


jupyterlab-lsp 3.4.1

Checklist

[x] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin please rerender

@krassowski sorry it took me a while to get around to this, fabulous work getting this out so quickly, and the broader contribution to the ecosystem docs, etc. i realize now we could add the source build test here, for giggles... though if we get all the way to this point, it's awfully far down the line on any given sunday...

Thanks for adding the extra check!


Add support for build versions

e.g. 0.0.0+4

I think this is complete, no?

I think this is complete, no?

Ya


weatherPhrase is not an int

php cron.php
INSERT INTO stat(coolLowerSetpLimit, coolNextPeriod, coolSetpoint, coolUpperSetptLimit, deviceID, dispTemperature, displayedUnits, heatLowerSetptLimit, heatNextPeriod, heatSetpoint, heatUpperSetptLimit, isInVacationHoldMode, schedCoolSp, schedHeatSp, scheduleCapable, statusCool, statusHeat, systemSwitchPosition, weatherHumidity, weatherPhrase, weatherTemperature) VALUES (50.0000, 0, 76.0000, 99.0000, secured, 81.0000, 'F', 40.0000, 0, 62.0000, 90.0000, false, 76.0000, 62.0000, true, 0, 0, 2, '64', 'Mostly clear', '72')
Errormessage: Incorrect integer value: 'Mostly clear' for column 'weatherPhrase' at row 1
note: I purposely changed the unit #

Future installs will have this fixed. To fix existing installs, update the following columns which got switched-
weatherHumidity int(11) NOT NULL
weatherPhrase varchar(32) COLLATE latin1_german2_ci NOT NULL

Thank you


Hide intra-line timestamps

The LRC enhanced format supports additional timestamps within a line.
I'm not convinced its useful to support rendering those (just because its rather niche) but we could probably fairly easily parse them out and not render the timestamps.
Requested on reddit @ https://www.reddit.com/r/foobar2000/comments/mm70rb/i_made_an_opensource_alternative_to/j0h3dgd/?context=3

Also add support for https://en.wikipedia.org/wiki/LRC_(file_format)#A2_extension:_word_time_tag


Amended instructions for Check for Palindrome challenge to reference casing

Pre-Submission Checklist

[x] Your pull request targets the staging branch of FreeCodeCamp.
[x] Branch starts with either fix/, feature/, or translate/ (e.g. fix/signin-issue)
[x] You have only one commit (if not, squash them into one commit).
[x] All new and existing tests pass the command npm run test-challenges. Use git commit --amend to amend any fixes.

Type of Change

[x] Small bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds new functionality)
[ ] Breaking change (fix or feature that would change existing functionality)
[ ] Add new translation (feature adding new translations)

Checklist:

[x] Tested changes locally.
[x] Closes currently open issue (replace XXXX with an issue no): Closes #10094

Description

Amended instructions for Check for Palindrome challenge to reference casing.

@weejimmy thanks a lot for the fix.
:sparkles: Congratulations on your first contribution to Free Code Camp :sparkles:


Treebank Tokenizer buggy

The NLTK TreebankWordTokenizer appears to have at least two bugs in it:
(a) It doesn't properly handle sentences with quotes at the end.
 Input:
``If we don't have any further suggestions as to how we should present our case, we should be able to finish this.''

Output:
['`', '`', 'If', 'we', 'do', "n't", 'have', 'any', 'further', 'suggestions', 'as', 'to', 'how', 'we', 'should', 'present', 'our', 'case', ',', 'we', 'should', 'be', 'able', 'to', 'finish', "this.''"]

(b) According to the official Penn Treebank Tokenization page at http://www.cis.upenn.edu/~treebank/tokenization.html, double quotes (") should be changed to doubled single forward- and backward- quotes (`` and ''). However, nltk.word_tokenize() (TreebankTokenizer) does not follow this rule.
Input:
He went to the "Concert for Hope" yesterday.

Output:
['He',  'went',  'to',  'the',  '"',  'Concert',  'for',  'Hope',  '"',  'yesterday', '.']

One of my colleagues, Michael Heilman, took the official Penn Treebank sed script (http://www.cis.upenn.edu/~treebank/tokenizer.sed) and ported it to NLTK using the Tokenizer API. It can be found here: https://gist.github.com/1506443
This script should replicate the sed script and, therefore, should not have either of these issues.

In this version, CONTRACTIONS2 requires tokens to be space separated, when \b should be used instead (as in the existing implementation).  Thus, it breaks in cases like this:
"This cannot cannot be right!" --> "This can not cannot be right !"
"This	cannot	work." --> "This	cannot	work ."

Resolved in
https://github.com/nltk/nltk/commit/e660a6827d44e242b4114abba7fc30fef6b0125c

Thanks steven!

Nitin

On Feb 3, 2012, at 12:48 AM, Steven<EMAIL_ADDRESS>wrote:

Resolved in
https://github.com/nltk/nltk/commit/e660a6827d44e242b4114abba7fc30fef6b0125c

Reply to this email directly or view it on GitHub:
https://github.com/nltk/nltk/issues/6#issuecomment-3792313

In case of numbers that are written in the financial format like 2,700 the current tokenizer split the number to three tokens. That is actually does not comply with the treebank corpus. Moreover, it does not seem reasonable as it changes the meaning of the text.
Here is the output of my personal benchmark for tokenizers.
https://gist.github.com/1748164


Allows to select no date at all

Here a quick PR for allowing to have no selected date on the component.
This is useful when we want to reset the range to none and to have the UI update accordingly.
I used it a while ago in a project. Should be tested again, as I was not able to make run the demo and no time to create automated tests.

covered in the new version.


Can not paste html content to custom block

Draft js can help me create custom block content with all custom class and style.
I was very happy to export contentState to html string to store in Database. But the problem is when I try to paste this html content to state, "convertFromHTML" auto remove all my class and replace some tags in my content.
For instance:
I has this html content:

googleABC, XYZ

How can I prevent Draft remove all my class name and remove my "a tag"?

How did it work out


Add SABnzbd extension

Description
SABnzbd is a binary newsreader.
There's a few simple commands at the moment but covers a few of the core things you'd want to do.
Type of change

New extension

Screencast

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I checked that extension's icon has size of 512x512

Since I never tried it before it takes a lot of time setting it up and getting it to work and with every step something new happens and I'm not sure what I'm doing 😄

@pernielsentikaer: I've added a screencast. Please excuse the poor quality. 🤷🏼♂

No idea what's happening there, but UX seems fine 😅

Small improvement suggestion for the future – expand a little bit in the description on what SABnzbd is so that users who have no clue what SABnzbd is could get some idea what it's for.

Thank you for the review and acceptance!

No idea what's happening there, but UX seems fine 😅

Hopefully someone in the community can try it to verify it for them 😆 I'm happily using it day-to-day now though.

Small improvement suggestion for the future – expand a little bit in the description on what SABnzbd is so that users who have no clue what SABnzbd is could get some idea what it's for.

Oh yes, good idea!


python38Packages.deezer-py: 1.2.3 -> 1.2.4

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python38Packages.deezer-py is: "A wrapper for all Deezer's APIs"
meta.homepage for python38Packages.deezer-py is: "https://gitlab.com/RemixDev/deezer-py"
meta.changelog for python38Packages.deezer-py is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 1.2.4 with grep in /nix/store/2s66ym2i69dkyb5v2jjp4yy3aj0piha9-python3.8-deezer-py-1.2.4

found 1.2.4 in filename of file in /nix/store/2s66ym2i69dkyb5v2jjp4yy3aj0piha9-python3.8-deezer-py-1.2.4

directory tree listing: https://gist.github.com/5c54d3ba4110f57b720680df70e9c979

du listing: https://gist.github.com/4fc4ceab6c853f2827744dcbca7c67a5

Rebuild report (if merged into master) (click to expand)

16 total rebuild path(s)

4 package rebuild(s)

4 x86_64-linux rebuild(s)
4 i686-linux rebuild(s)
4 x86_64-darwin rebuild(s)
4 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
python38Packages.deemix
python38Packages.deezer-py
python39Packages.deemix
python39Packages.deezer-py

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/2s66ym2i69dkyb5v2jjp4yy3aj0piha9-python3.8-deezer-py-1.2.4 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python38Packages.deezer-py https://github.com/r-ryantm/nixpkgs/archive/628f61fe1a5830787ae359aa61be877858c36100.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/2s66ym2i69dkyb5v2jjp4yy3aj0piha9-python3.8-deezer-py-1.2.4
ls -la /nix/store/2s66ym2i69dkyb5v2jjp4yy3aj0piha9-python3.8-deezer-py-1.2.4/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

4 packages built:

python38Packages.deemix
python38Packages.deezer-py
python39Packages.deemix
python39Packages.deezer-py

Maintainer pings
cc @natto1784 for testing.

Result of nixpkgs-review pr 139501 at 628f61fe run on aarch64-linux 1

4 packages built successfully:

python38Packages.deemix
python38Packages.deezer-py
python39Packages.deemix
python39Packages.deezer-py


Extra logging for offloaders

Some extra logging for offloader loading and initialization which
would have been useful when debugging #2697.

rerun integration tests
rerun cpp tests

run integration tests

run integration tests

rerun integration tests
rerun cpp tests


Ensure ParameterUtil.splitParameterString correctly parses consecutive unescaped delimiter

This PR fixes #514 and makes modifications to ParameterUtil.splitParameterString such that a QuantityParam value of '5.5||mg' the 'mg' is treated as the 'unit' and not as the 'system'.

Coverage increased (+0.007%) to 81.488% when pulling a6a1416888514fef0e8d1a5ed3d94c589cf24cd0 on CarthageKing:feature-parameterutil-fix into af32c4b7e9c27b0a48a2fb42047b8d0d2007b720 on jamesagnew:master.


[ob] Make separate dlma handler

It's too much big file the ob_dl_handler.
Need to make it separate ob_dl_handler and ob_dlma_handler.

Done! :)


Allow users to disable rounding

cleancss --rounding-precision -1 should disable rounding altogether.
The binary test I added is currently failing and I have no idea why. I'm sure one of you will spot the mistake right away.
I guess this could also be merged into the 2.2 branch.

Part of this PR undid #348, maybe it should be brought back?


Skip post-clean if aborted before the image build stage in pre-merge [skip ci]

Signed-off-by: Peixin Li<EMAIL_ADDRESS>Current setup doesn't hurt anything but could cause misunderstanding, so make it more clearly.
Tested in my forked repo.
TAG is null or empty,

Comparing to TAG is not null and not empty,

build


Version problem

I encountered a version conflict problem. When will the new version appear?

hi, You can use version 6.4.0, elasticsearch upgrade frequently this year. No need use newest version.


Fix parsing GTIDs from mysqldump, fixes #550

Hello, @dobegor.
Unfortunately, this repo is abandoned. I made PR about this situation, but could not merge it :)
I'll be glad to see your PR in my fork, which I continue to support and develop.

Accepted in https://github.com/atercattus/go-mysql/pull/1


Bad redraw on refill array of items

Use case: Put some items to dragular, than I get some changes back, what mean assign different array of items to dragular. For a second is showing item from both arrays. What am I doing wrong? I tried delete previous version of array, but without changes. Maybe bug in dragular?
In GIF are 2 items at first time, than I put one away by assign previous version of array. For a second there are 3 items - 2 from previous version and 1 from actual version.

Once you provide array of items to dragular you cannot replace it, because dragular will lost reference. You need to edit same array you provided on initialisation. So if you want to clean the array, use myArray.length = 0 and then if you wish to fill it with new data, you can push into array new values. This is going to be change in future as many user ask for it. Let me know if your problem was solved, thanks ;)

Seems to be no more actual..


Updating  primitives names for 'nmoscap' in lvs gds files

Fixes https://github.com/google/globalfoundries-pdk-libs-gf180mcu_fd_pr/issues/1

Renaming nmos_cap devices with its new name in the gds files used in LVS testing under the following directory:

globalfoundries-pdk-libs-gf180mcu_fd_pr/rules/klayout/lvs/testing/testcases/

@mithro Changes is completed including cap_mim as you agreed with @atorkmabrains


do not select on resourceType of a dataset

Describe the improvement
Currently, only datasets with a DOI and resourceType = "dataset" are accepted in the first step of an "add dataset" workflow. Instead, we do not want to apply any filter on resourceType and accept all resources that have a DOI that can be resolved at DataCite. As such, data curators can assess the submitted resources and decide whether the submission is appropriate to biblio. In a later phase, after learning about which resources are typically submitted, we can revisit and decide whether automated filtering based on resourceType can be useful.

Wouldn't it make the import more complex to accept all types? For every type
the mappings could be different.

No it's a small change, mappings stay the same, just remove the filter on resourcetype

https://github.ugent.be/Universiteitsbibliotheek/librecat/pull/625


upping version number to something crazy high to superseded upstream

Assigned to @po1 since he's dealing with debs building.


Remove unneeded fields on historical work page

Link to Ticket
Description of Changes Made
A follow-on tidy up from investigating the issues with crashing pages on the production server. The base cause was very large images - but this removes some unnecessary image fields on historical work pages. I checked a few historical case studies and most seemed to use both the homepage image field and the feed image field - so I've updated the listing_image property to use the feed image, as that is more intuitive.
How to Test
Check that the work listing page still displays images
Edit a historical work page and you will no longer see the 'screenshots' or 'homepage image' fields.
Screenshots

Expand to see more

MR Checklist

[x] Add a description of your pull request and instructions for the reviewer to verify your work.
[x] If your pull request is for a specific ticket, link to it in the description.
[x] Stay on point and keep it small so the merge request can be easily reviewed.
[x] Tests and linting passes.

Unit tests

[ ] Added
[x] Not required

Documentation

[ ] Updated build docs
[ ] Updated editor guidelines (https://docs.google.com/document/d/1PAWccdQ4tfaZsrEWmpDhvP3GH5RRmBOARFVp4b-kje8/edit?usp=sharing)
[x] Not required

Browser testing

[ ] I have tested in the following browsers and environments (edit the list as required)

Latest version of Chrome on mac
Latest version of Firefox on mac
Latest version of Safari on mac
Safari on last two versions of iOS
Chrome on last two versions of Android

[x] Not required

Data protection

[x] Not relevant
[ ] This adds new sources of PII and documents it and modifies Birdbath processors accordingly

Accessibility

[ ] Automated WCAG 2.1 tests pass
[ ] Manual WCAG 2.1 tests completed
[ ] I have tested in a screen reader
[ ] I have tested in high-contrast mode
[ ] Any animations removed for prefers-reduced-motion
[x] Not required

Sustainability

[ ] Images are optimised and lazy-loading used where appropriate
[ ] SVGs have been optimised
[ ] Perfomance and transfer of data considered
[ ] If JavaScript is needed alternatives have been considered
[x] Not required

Pattern library

[ ] The pattern library component for this template displays correctly, and does not break parent templates
[ ] The styleguide is updated if relevant
[x] Changes are not relevant the pattern library

Re-created from https://github.com/torchbox/torchbox.com/pull/137

Thanks @helenb 👍
Perhaps we need to also remove references to homepage_image in this template?
https://github.com/torchbox/torchbox.com/blob/daf8775c77b3229b2a9e9febd5d647dcd029754d/tbx/project_styleguide/templates/patterns/molecules/related-item/related-item.html#L5-L6

Actually that template, and the one that includes it, are no longer needed in the build, so I have deleted them.


Update Documenter to 1.3

This enables the automatic writing of an objects.inv inventory file (and also gives the documentation a slightly nicer UI).
I'd like to be able to use DocumenterInterLinks to link from my project to the documentation of CompomentArrays, and this update would make that easier. See also https://github.com/SciML/RecursiveArrayTools.jl/pull/364, although here you were still using a pinned pre-1.0 version of Documenter. (Actually, so old that even DocumenterInventoryWritingBackport doesn't support it anymore.) I tried building locally, and everything seems to work fine with Documenter 1.3.
I didn't touch the CI still using Julia 1.6, although using a more recent version of Julia would probably be a good idea.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 73.18%. Comparing base (44a9d4c) to head (d76d832).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #253      +/-   ##
==========================================
- Coverage   74.09%   73.18%   -0.91%
==========================================
  Files          25       25
  Lines         772      772
==========================================
- Hits          572      565       -7
- Misses        200      207       +7

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

Looks good to me, thanks!


http://localhost/ is not valid url for UrlValidator

Not sure it is by design or bug if bug i can create a PR.
What steps will reproduce the problem?
$val = new UrlValidator;
$result = $val->validate('http://localhost/');

What is the expected result?
true
What do you get instead?
false

Both ::1 and <IP_ADDRESS> must also be kept in mind.

I would say this is a feature rather than a bug :) localhost will point to different host for every user of the system so if you allow adding such URLs it could introduce flaws with leaking information from internal network. That is true for all non-fqdn domain names. Why do you want to allow this?

@amincheloh you can adjust the $pattern property of the validator to make it work in your case.
In general I think the validator could be based on parse_url() function instead of a regex and allow further options like forcing https urls only for example. Or a switch to enable/disable fqdn names. I suggest to rewrite it for 2.1.0.

I ran into the same issue during functional testing with codeception where one of the params for a model was the referal Url and codeception runs everything on http://localhost.

What steps will reproduce the problem?
$urlValid = new \yii\validators\UrlValidator();
var_dump($urlValid->validate("http://www.google.com/uncheck/orderPay/ejbPaySucess.do\\"));
exit;

What is the expected result?
bool(true)
What do you get instead?
bool(false)
Additional info

Q
A

Yii version
<IP_ADDRESS>

PHP version
7.0.1

Operating system
windows

@phpyii what's the purpose of the backslash at the end of the URL?

@cebe  Today I found the URL stored in the database. I added URL validation


[Inputs.json]: Talkback and Keyboard focus are not in sync in DatePicker under Due Date.

https://microsoft.visualstudio.com/OS/_workitems/edit/38004752

Hi @MohitTyagi13,
We utilize the Android's DatePickerDialog to create this control, so we are unable to modify the code.
I have investigated updating to Android 13 and updating androidx.appcompat:appcompat; however, there does not seem to be accessibility fixes that impact will impact our Input.Date control.
Please let me know if there is anything else I can do here. Thanks!

Varified on Adaptive Card, Teams app and Android Native Calendar the issue seems to be similar through out. However, with Talk Back keyboard navigation [Alt + arrow keys] user can navigaate on to the Calendar dates.
Hence the severity can be reduced as keyboard users including Braille Display users aren't impacted.

Issue is still repro.
Reactivating since this bug depend on Android.

@jwoo-msft,
Hope you are doing great.
Could you please provide us the codepen of this bug?

Hi @jwoo-msft , @JeanRoca
Hope you are doing great.
Could you please provide us the codepen of this bug?


Add randomizers for all message types

Should be in a separate package
Should return sensible values

See https://github.com/TheThingsNetwork/ttn/issues/658

If we want to be compatible with gogoproto stuff, the randomizers should be of form NewPopulatedXXX. And take a rand as parameter, see https://github.com/TheThingsNetwork/ttn/pull/628 for reference

Work started in https://github.com/TheThingsNetwork/api/tree/feature/2-go-populate


Add django-waffle

django-waffle allows us to use feature flags to turn specific bits of the application on or off on a very granular basis, ideal for user testing and staged rollout.
Heads up: We should probably approve and merge the ADR first.

Codecov Report

Merging #630 (3f33d82) into main (5d42c18) will not change coverage.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@           Coverage Diff           @@
##             main     #630   +/-   ##
=======================================
  Coverage   70.14%   70.14%
=======================================
  Files          22       22
  Lines         767      767
  Branches       35       35
=======================================
  Hits          538      538
  Misses        221      221
  Partials        8        8

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


Kubeapps inception: link to Kubeapps project shown on Kubeapps chart page

Perhaps we need an exception for this particular chart? It could be quite confusing?
cc @earnubs @arapulido @andresmgot

Nice to have, but I don't think it should be a priority

well, I don't see it that confusing since you can see that piece of info for every chart. Also the link is to an external page so it's not a link to itself (which could be more confusing).
In any case, it's true that it's something nice to have.


update OpenSSH key management

Update the documentation regarding OpenSSH key management, and include instruction how to use ssh-keygen.exe.
closes https://github.com/MicrosoftDocs/windowsserverdocs/issues/2747
closes https://github.com/MicrosoftDocs/windowsserverdocs/issues/3400
closes https://github.com/MicrosoftDocs/windowsserverdocs/issues/3456
closes https://github.com/MicrosoftDocs/windowsserverdocs/issues/3806
closes https://github.com/MicrosoftDocs/windowsserverdocs/issues/3970
closes https://github.com/MicrosoftDocs/windowsserverdocs/issues/4345
closes https://github.com/MicrosoftDocs/windowsserverdocs/issues/4518
closes https://github.com/MicrosoftDocs/windowsserverdocs/issues/4598
closes https://github.com/MicrosoftDocs/windowsserverdocs/issues/4622
closes https://github.com/MicrosoftDocs/windowsserverdocs/issues/4634
closes https://github.com/MicrosoftDocs/windowsserverdocs/issues/4690
closes https://github.com/MicrosoftDocs/windowsserverdocs/issues/4702
closes https://github.com/MicrosoftDocs/windowsserverdocs/issues/4743
closes https://github.com/MicrosoftDocs/windowsserverdocs/issues/4770
closes https://github.com/MicrosoftDocs/windowsserverdocs/issues/4787
Signed-off-by: Thomas Sjögren<EMAIL_ADDRESS>

https://github.com/MicrosoftDocs/windowsserverdocs/pull/4788/files?diff=split&w=1&short_path=8ff87b2#diff-8ff87b2d6abe3fd889d32fbf4ee13d7e
@illfated

#sign-off

(Also looking forward to getting all those similar and related tickets closed.)

@therealkenc : Forgive me for not understanding the issue the way you see it, but are you saying that the proposed change is not useful in this context? If so, do you have a suggestion for what the proposed change should look like to make it useful?

Thank you for the the comment @therealkenc, but is there any incorrect information in the PR? Feel free to point them out so I can fix them.
If you read https://docs.microsoft.com/en-us/windows-server/administration/openssh/openssh_overview in order, you'll come across https://docs.microsoft.com/en-us/windows-server/administration/openssh/openssh_install_firstuse#initial-use-of-ssh which describes how to connect with a username and password, and you'll finish on "OpenSSH key management"
which states: "Most authentication in Windows environments is done with a username-password pair. This works well for systems that share a common domain. When working across domains, such as between on-premise and cloud-hosted systems, it becomes more difficult." and then proceeds describing User key generation.

The documentation (as modified with the PR) has:
# Appropriately ACL the authorized_keys file on your server
ssh --% user1<EMAIL_ADDRESS>powershell -c $ConfirmPreference = 'None'; Repair-AuthorizedKeyPermission C:\Users\user1\.ssh\authorized_keys

This is despite the PR removing lines including:
- * The OpenSSHUtils PowerShell module has been created to set the key ACLs properly, and should be installed on the server

Repair-AuthorizedKeyPermission will cause an error, because OpenSSHUtils is no longer a thing.

In any case, the Deploying the public key section does not work for users in the Administrator group, because %userprofile%\.ssh\authorized_keys is not read.
I followed some non-authoritative instructions found here. Which are:
$acl = Get-Acl C:\ProgramData\ssh\administrators_authorized_keys
$acl.SetAccessRuleProtection($true, $false)
$administratorsRule = New-Object system.security.accesscontrol.filesystemaccessrule("Administrators","FullControl","Allow")
$systemRule = New-Object system.security.accesscontrol.filesystemaccessrule("SYSTEM","FullControl","Allow")
$acl.SetAccessRule($administratorsRule)
$acl.SetAccessRule($systemRule)
$acl | Set-Acl

Whether those instructions are appropriate or not I couldn't tell you. They did work here.
The instructions as presented before or after this pull request do not.

Thanks for catching this @therealkenc, I've removed the OpenSSHUtils reference in the document.
Regarding users in the Administrator group, I can´t really say if that's the "official" way of doing it, since I don't actually work with Windows systems.
Probably need a verification.

Appreciated, thanks. But the proposed changes will result in hundreds of thousands of developers who have Admin accounts and are trying to ssh from their VM to windows to waste at least an hour (and perhaps more) of their lives (which they will never get back) finding out how to get their remote authorized_keys working.
To be clear, that's the common (not exceptional) case. Most every developer using Docker or WSL or Hyper-V to ssh into Windows is going to give themselves Admin privs.
Since I am here, the statement at the top of the page doesn't make sense:

Most authentication in Windows environments is done with a username-password pair.

I don't know how that got in there, but there is no basis for the premise that Windows users, for reasons unexplained, prefer to enter a password every time they type ssh versus Linux users or MacOS users.

Most authentication in Windows environments is done with a username-password pair. in this context you're right, but I assume (I'm no the original author obviously), that it meant to cover standard Windows logins as well.
Is there a way to switch User -> Administrator (e.g sudo)  on the Windows command line? Perhaps that should be the recommended way, if possible.

Most authentication in Windows environments is done with a username-password pair. in this context you're right, but I assume (I'm no the original author obviously), that it meant to cover standard Windows logins as well.
Is there a way to switch User -> Administrator (e.g sudo) on the Windows command line? Perhaps that should be the recommended way, if possible.

Myself, I have only used the GUI method of "Run as administrator", but there are other suggestions found elsewhere:

https://superuser.com/questions/42537/is-there-any-sudo-command-for-windows

runas /noprofile /user:Administrator cmd (to start a command shell as a administrator)

@therealkenc : Sorry about the confusion. The documentation makes it look like RunAs is outdated and/or deprecated: https://docs.microsoft.com/en-us/previous-versions/windows/it-pro/windows-server-2012-r2-and-2012/cc771525(v=ws.11) -- the command is available in Windows 10, though:
Microsoft Windows [Version 10.0.19041.388]
(c) 2020 Microsoft Corporation. All rights reserved.

C:\Users\illfated>runas
RUNAS USAGE:

RUNAS [ [/noprofile | /profile] [/env] [/savecred | /netonly] ]
        /user:<UserName> program

RUNAS [ [/noprofile | /profile] [/env] [/savecred] ]
        /smartcard [/user:<UserName>] program

RUNAS /trustlevel:<TrustLevel> program

   /noprofile        specifies that the user's profile should not be loaded.
                     This causes the application to load more quickly, but
                     can cause some applications to malfunction.
   /profile          specifies that the user's profile should be loaded.
                     This is the default.
   /env              to use current environment instead of user's.
   /netonly          use if the credentials specified are for remote
                     access only.
   /savecred         to use credentials previously saved by the user.
   /smartcard        use if the credentials are to be supplied from a
                     smartcard.
   /user             <UserName> should be in form USER@DOMAIN or DOMAIN\USER
   /showtrustlevels  displays the trust levels that can be used as arguments
                     to /trustlevel.
   /trustlevel       <Level> should be one of levels enumerated
                     in /showtrustlevels.
   program         command line for EXE.  See below for examples

Examples:
> runas /noprofile /user:mymachine\administrator cmd
> runas /profile /env /user:mydomain\admin "mmc %windir%\system32\dsa.msc"
> runas /env<EMAIL_ADDRESS>"notepad \"my file.txt\""

NOTE:  Enter user's password only when prompted.
NOTE:  /profile is not compatible with /netonly.
NOTE:  /savecred is not compatible with /smartcard.

C:\Users\illfated>_

The confusion stems from the irrelevance of runas, not its usage.

@konstruktoid : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@konstruktoid : I hope you have noticed the file conflict (just mentioning this in case you haven't yet). I also hope this PR will get some attention regarding a possible merge soon too.

@konstruktoid : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@konstruktoid : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@konstruktoid : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@konstruktoid : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off

@illfated: I'm sorry - only the author of this article, @maertendmsft, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

If it's not too late, it might be a good idea to add a copy of the instructions about setting the ssh-agent service startup type to Manual, into the section under User Key Generation.

Hi @Hamled, doesn't https://github.com/MicrosoftDocs/windowsserverdocs/pull/4788/files#diff-e9b13ca0f3a6dcda51255be3272c0d6b2e11096a4bcc961a9b1d8c6b638d5568R74 suffice?

I guess it just depends on the expected use case. For my part I was trying to make use of OpenSSH on a machine that only acts as a client to connect to remote Git servers via SSH so I had skipped down to the user key generation section.
If I hadn't encountered a previous version of this document and read the instructions about setting the startup type, I think I would have been left completely confused why the provided instructions in that section didn't work.

Ah, good point.

@konstruktoid : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@Hamled, have a look at https://github.com/MicrosoftDocs/windowsserverdocs/blob/8b1b619f7051dd563bd0d72a0bebb0de7e985591/WindowsServerDocs/administration/OpenSSH/OpenSSH_KeyManagement.md#user-key-generation

So, does this mean that we are drawing closer to a conclusion? If so, that would be good news for final review & merge. Any remaining issues at the moment?

Any remaining issues at the moment?

Yes; the documentation as presented does not work for any user in the local Administrator group, because %USERPROFILE%/.ssh will not be read. Non-authoritative citation here. Noting that a plurality of developers using VMs with ssh (in both directions) have admin privs.

A user with admin privs will make it to the concluding sentence "After this, the user can connect to the sshd host from any client that has the private key" to find their keys do not work.

Ok, fair enough. Thank you for the feedback.
@konstruktoid : Do you have any thoughts on how to address that issue?

Sorry I don't, that´s too much Windows for me.

Fair enough. We should aim for a conclusion to this PR anyway.
@therealkenc :: Is it fair to say that you are still opposed to the current PR and that you would like to make your own PR for the necessary changes, if you know what should be added to make it work for administrators like you?

if you know what should be added to make it work for administrators like you?

That question was answered in September: "Whether those instructions are appropriate or not I couldn't tell you."

if you know what should be added to make it work for administrators like you?

That question was answered in September: "Whether those instructions are appropriate or not I couldn't tell you."

Sorry. I should have read up. I quite tired when I wrote my question.
That leaves us in a very sticky situation. I guess this was all for nothing, then.

I reported https://github.com/MicrosoftDocs/windowsserverdocs/issues/5068 -- and I can confirm that some kind of permission fix is need and the state of the documentation before and after this PR is not functioning. The permissions of the authorized_keys files are broken after adding a key.

I can confirm that the options listed in https://www.concurrency.com/blog/may-2019/key-based-authentication-for-openssh-on-windows and  by @therealkenc worked for me

FWIW: I spent 8 hours to debug this

Thank you very much for the effort and for your feedback. That is both welcome and very worth while to us.

Updated list of related issue tickets:

#2747 A better explanation can be found here (Open)
#3400 Can't install the "OpenSSHUtils" (Closed)
#3456 Misleading title/summary causing search confusion (Open)
#3806 OpenSSHUtils is deprecated (Open)
#3970 OpenSSH installation (Closed)
#4345 OpenSSHUtils Not Needed (Open)
#4518 Unable to configure OpenSSH for public key auth following docs (Open)
#4598 Instructions provided ends with an error (Open)
#4622 Out-of-date - OpenSSHUtils no longer available. (Open)
#4634 More documentation required (Open)
#4690 OpenSSHUtils install reference no longer valid (Open)
#4702 OpenSSHUtils got unlisted from the PowerShell gallery (Open)
#4743 Nope. First example errors out. (Open)
#4770 Error when following instructions to install OpenSSHUtils (Open)
#4787 OpenSSHUtils module does not exist (Open)
#5000 Clarification needed (Open)
#5138 These instructions do not work on Windows 10 20h2 (Open)

Updated list of related issue tickets:
* #2747 **A better explanation can be found here** (Open)

* #3400 **Can't install the "OpenSSHUtils"** (Closed)

* #3456 **Misleading title/summary causing search confusion** (Open)

* #3806 **OpenSSHUtils is deprecated** (Open)

* #3970 **OpenSSH installation** (Closed)

* #4345 **OpenSSHUtils Not Needed** (Open)

* #4518 **Unable to configure OpenSSH for public key auth following docs** (Open)

* #4598 **Instructions provided ends with an error** (Open)

* #4622 **Out-of-date - OpenSSHUtils no longer available.** (Open)

* #4634 **More documentation required** (Open)

* #4690 **OpenSSHUtils install reference no longer valid** (Open)

* #4702 **OpenSSHUtils got unlisted from the PowerShell gallery** (Open)

* #4743 **Nope. First example errors out.** (Open)

* #4770 **Error when following instructions to install OpenSSHUtils** (Open)

* #4787 **OpenSSHUtils module does not exist** (Open)

* #5000 **Clarification needed** (Open)

* #5138 **These instructions do not work on Windows 10 20h2** (Open)

Will those issues be closed as well if this PR is merged?

If you update your list of "Closes #xxxx" to include all of them, they should normally be closed, yes.
Otherwise, they can be closed manually by commenting #please-close (non-encapsulated, of course), since the bots have been scripted to react to that command string.

Yeah, I should rephrase that. Are all those issues supposed to get closed by this PR?

Yeah, I should rephrase that. Are all those issues supposed to get closed by this PR?

Oh. I see. Maybe, maybe not. I guess I was a bit hasty to draw the conclusion that the new ones are directly related. The tickets are from the same MS Docs page.

Just to throw this in here, on 202H, the only thing I need to run is icacls.exe "$($env:USERPROFILE)/.ssh/authorized_keys" /inheritance:r /grant "$($env:USERNAME):F" /grant "NT AUTHORITY\SYSTEM:F" in an elevated Powershell to fix my authorized_keys. This seems to be functionally identical to the proposed change in documentation for this step.

The instruction suggest a scp to the authorized_keys file... which will blow away any previous entries. It is common for this file to contain multiple keys, hence the pluralized name.

The instruction suggest a scp to the authorized_keys file... which will blow away any previous entries. It is common for this file to contain multiple keys, hence the pluralized name.

But the keys in the authorized_keys file should only be yours and yours alone, @gbraad, and the management and responsibility is up to the user.
Otherwise a server configuration management system should be used.

That is not I refer to. The action describes a scp, while normally we add a key. This is also part of the responsibility of something like ssh-copy-id.
I have multiple keys, each representing a different host or group of users. Key management is not only about users, but also traceability and being able to terminate the use of a particular key

Just out of curiosity.
When and why use multiple keys, each representing a different host or group of users, for the same account?
And if management of users accounts, access policies,authorized_keys etc is getting overly complicated (or just a daily repetition) perhaps is should be (semi-)automated.

That is not what I refer to. The action describes a scp, while normally we add a key. This is also part of the responsibility of something like ssh-copy-id, a helper script on most distros to 'add' to the authorized keys.
This is why overwriting the content with a copy will leave only one user remaining. It lacks explanation what the users should do.

Ah, @gbraad, thant is true. The issue is I believe is that ssh-copy-id isn't present on Windows 10.

The following list of issue tickets (includes both open and closed tickets) targets the same page as the one targeted by this PR:

OpenSSHUtils is deprecated -- #3806 opened on Jan 28, 2020 by FizzBuzz791
PubKey Auth not working [solution] -- #3828 opened on Jan 31, 2020 by ackvf
Outdated instructions? -- #4023 by RussKeldorph was closed on Sep 2, 2020
OpenSSHUtils Not Needed -- #4345 opened on May 14, 2020 by gynnantonix
Unable to configure OpenSSH for public key auth following docs -- #4518 opened on Jun 20, 2020 by jaraco
Instructions provided ends with an error -- #4598 opened on Jun 28, 2020 by jyavenard
Out-of-date - OpenSSHUtils no longer available. -- #4622 opened on Jul 8, 2020 by Jammrock
More documentation required -- #4634 opened on Jul 11, 2020 by garylawuk
OpenSSHUtils install reference no longer valid -- #4690 opened on Aug 1, 2020 by wrjohnson
OpenSSHUtils got unlisted from the PowerShell gallery -- #4702 opened on Aug 5, 2020 by jagraveldinger
Does not list possible ssh commands -- #4726 opened on Aug 16, 2020 by morgzz
ssh-agent service not installed upon installation of OpenSSH Server or Client -- #4729 by ThatNerdyPikachu was closed on Oct 10, 2020
Nope. First example errors out. -- #4743 opened on Aug 22, 2020 by boomalator
Error when following instructions to install OpenSSHUtils -- #4770 opened on Aug 31, 2020 by c0f
OpenSSHUtils module does not exist -- #4787 opened on Sep 4, 2020 by oliversalzburg
Does not explain how to ensure keys get reloaded to agent on boot -- #4858 opened on Sep 30, 2020 by g24swint
Private keys from the ssh-agent can be accessed -- #4871 opened on Oct 2, 2020 by stefanfritsch
Install-Module OpenSSHUtils failure -- #4893 opened on Oct 8, 2020 by LeeJenkins
Does ssh-agent on windows have persistent storage of ssh keys? -- #4946 opened on Oct 27, 2020 by zaaj
OpenSSHTools doesn't exist anymore on powershell gallery -- #4948 opened on Oct 28, 2020 by JustinGrote
Clarification needed -- #5000 opened on Nov 10, 2020 by conyg
OpenSSHUtils has been delisted -- #5068 by esc was closed on Dec 1, 2020
Update suggestion of OpenSSHUtils -- #5080 by aamir0 was closed on Dec 4, 2020
These instructions do not work on Windows 10 20h2 -- #5138 opened on Dec 19, 2020 by mihai-preda
Out of date - Repair-AuthorizedKeyPermission -- #5395 opened 4 days ago by nekdima

Some of these may be not specific to this PR content, but I thought it would be useful to see how much attention this page has received since January 28, 2020 (more than a year ago).

Thanks for this PR and to everyone for the conversation throughout. I have reworked this document to remove or replace parts that are no longer relevant or correct based on this PR and several other issues raised for this page.
The changes have been done through a different PR, so I'm going to close this one. Please file new issues if needed. The changes should be visible in around 6-12 hours.
#please-close


Web app is not responsive now

🐞 Problem
Web app doesn't render well on small screens
🎯 Goal
It should look good on smaller screens too.
💡 Possible solutions
Make it responsive

📋  Steps to solve the problem

Comment below about what you've started working on.
Add, commit, push your changes
Submit a pull request and add this in comments - Addresses #<put issue number here>
Ask for a review in comments section of pull request
Celebrate your contribution to this project 🎉

I can work on this issue

Hi @Roshanjossey I want to work on making the app responsive.

@jimjja, @hereisnaman I'm sorry, we already have a pull request addressing this issue. #593  Can you help me out reviewing that pull request?

Hi I would like to help out with this feature, I can help review #593 👍

Closing this as issue is resolved in #593


Top/Latest for hashtags

https://github.com/bluesky-social/social-app/assets/810438/1a96fe3b-1fe1-4881-9494-1bf5cbf164d0

sighh i thought i fixed that... looking

Tabs sit above the title on web? Other than that, looks good

i think you may have had an outdated version of this PR, i'm pretty sure this is fixed in the last commit


[ports.cmake] Remove unnecessary variables for better debug experience

This removes around 50 extra variables which are defined in portfile.cmake. I don't think any of those are necessary.
(It also removes accidental usage of these variables)

I cancelled the build because:

The build that ran before looks like it contains no related issues
I need to mess with the macos machine that build happened to take

Is it OK to merge this? (Not doing so because it's still in draft form)

Sure, but it disposes of a PR :)


Remove unnecessary img tag for the foreignkey search input.

This PR fixes https://github.com/django-extensions/django-extensions/issues/1221 (and similar https://github.com/django-extensions/django-extensions/issues/229, https://github.com/django-extensions/django-extensions/issues/786).
As mentioned in the issue, Django's css already add a background image for the .related-lookup class, so instead of adding the missing asset admin/img/selector-search.gif we can just remove the img tag.
Before this PR:

After this PR:

Thanks !

Coverage decreased (-0.2%) to 53.317% when pulling 432fb9e3dc62d85d8dc66e62fa8fc9b0c77dedd3 on minervaproject:master into da5673c43a6e0bc7ebed753c0b66be230f9ae09b on django-extensions:master.


build: improve app bundle size and speed up client builds

Description
Introduces multiple concepts to address bundle size as well as build performance.
Dynamic imports + React Suspense
This enables webpack to create smaller chunks which reduces the code needed to be sent over the wire for first load. This also improves build speed.
Smarter imports
Eslint rules help us import packages from ES6 modules (like lodash and office ui). Webpack can tree shake these imports resulting in smaller bundles.
Improved typescript support in client
Uses ts-loader instead of babel to parse ts files in the client. It is faster than babel, but did expose some issues (@jsx pragma required for any file using css prop -- eslint && fixable).
Updates to webpack dependencies
This is suggested by the webpack docs to get the best performance. Some packages jumped major versions. I did not see any issues with the build result though.
Task Item
closes #1289

Fantastic ! @lei9444 can you help check this?


fix: handle the case when securitytoken fetcher returns null

Continuation of #32 for a couple more procedures.
Export/import procedures from src/procedures/index.

:tada: This PR is included in version 2.0.1-beta.10 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add per-subzone total plants to search API

To allow clients to query the number of plants in each subzone across sites
in a single organization, add a search field to get the total plants of all
species in a subzone. The per-species totals continue to be available in the
populations sublist.

Current dependencies on/for this PR:

main

PR #1293   👈

This comment was auto-generated by Graphite.

@sgrimm started a stack merge that includes this pull request via Graphite.

@sgrimm merged this pull request with Graphite.


[BUG] {{ dl color picker }} - {{ UI issues }}

The component is broken in open annotation.

Storybook:

@rotemshaham seems to have a duplicated issue? > https://github.com/dataloop-ai/components/issues/368
could I delete?


require gateway resource ID to be a gateway resource ID

on the Microsoft.Kubernetes/connectedClusters
ARM (Control Plane) API Specification Update Pull Request

[!TIP]
Overwhelmed by all this guidance? See the Getting help section at the bottom of this PR description.

PR review workflow diagram
Please understand this diagram before proceeding. It explains how to get your PR approved & merged.

Purpose of this PR
What's the purpose of this PR? Check the specific option that applies. This is mandatory!

[ ] New resource provider.
[ ] New API version for an existing resource provider. (If API spec is not defined in TypeSpec, the PR should have been created in adherence to OpenAPI specs PR creation guidance).
[x] Update existing version for a new feature. (This is applicable only when you are revising a private preview API version.)
[ ] Update existing version to fix OpenAPI spec quality issues in S360.
[ ] Convert existing OpenAPI spec to TypeSpec spec (do not combine this with implementing changes for a new API version).
[ ] Other, please clarify:

edit this with your clarification

Due diligence checklist
To merge this PR, you must go through the following checklist and confirm you understood
and followed the instructions by checking all the boxes:

[x] I confirm this PR is modifying Azure Resource Manager (ARM) related specifications, and not data plane related specifications.
[x] I have reviewed following Resource Provider guidelines, including
ARM resource provider contract and
REST guidelines (estimated time: 4 hours).
I understand this is required before I can proceed to the diagram Step 2, "ARM API changes review", for this PR.

Additional information

 Viewing API changes
For convenient view of the API changes made by this PR, refer to the URLs provided in the table
in the Generated ApiView comment added to this PR. You can use ApiView to show API versions diff.

Suppressing failures
If one or multiple validation error/warning suppression(s) is detected in your PR, please follow the
suppressions guide to get approval.

Getting help

First, please carefully read through this PR description, from top to bottom. Please fill out the Purpose of this PR and Due diligence checklist.
If you don't have permissions to remove or add labels to the PR, request write access per aka.ms/azsdk/access#request-access-to-rest-api-or-sdk-repositories
To understand what you must do next to merge this PR, see the Next Steps to Merge comment. It will appear within few minutes of submitting this PR and will continue to be up-to-date with current PR state.
For guidance on fixing this PR CI check failures, see the hyperlinks provided in given failure
and https://aka.ms/ci-fix.
For help with ARM review (PR workflow diagram Step 2), see https://aka.ms/azsdk/pr-arm-review.
If the PR CI checks appear to be stuck in queued state, please add a comment with contents /azp run.
This should result in a new comment denoting a PR validation pipeline has started and the checks should be updated after few minutes.
If the help provided by the previous points is not enough, post to https://aka.ms/azsdk/support/specreview-channel and link to this PR.

I don't currently have the necessary permission to add the label - request pending - but am expecting per aka.ms/brch to add BreakingChange-Approved-BugFix on the grounds that

this is updating the definition to accurately describe the service behaviour
this API version is currently only available via feature flags, set only in internal subscriptions so we definitely won't break any public usage

/azp run


Sort package list and remove duplicates

/retest

appstudio bootstrap fail
/test jvm-build-service-in-repo-with-pruning-e2e

now all passing / merging


Add missing host parameter to exception call

send_error and recv_error create a Mongo.Error without
host and this fails with the following error message:
no function clause matching in Mongo.Error.exception/1
Mongo.Error.exception([tag: :tcp, action: "recv", reason: :timeout])

Thank you!


Disabled navigation arrows

Is there any way to show default (not external) arrows in "disabled" state? Arrows are being removed from DOM when slider reach start/end.
Maybe better solution is to add "disabled" attribute to the arrows and leave ability to hide them just with CSS.

May be you are right🤓but I think it right behaviour for not looped carousel... in loop mode arrows visible
How you see implemented disabled flag?


Dashboard Token Listing: WARU

Discord Handle of Submitter: xbi0_0
Token Symbol: WARU
Network: Polygon
Token Address: 0xFA3872970c1952741C518B33f1DcE815f55Dc00A
Network Explorer Link: https://discord.com/channels/962352288557563955/962476019627995176/971484191759929364
Is a Super Token? false
Address of Underlying Token:
SVG Logo Link: https://waruprotocol.one/assets/logo.94e2b55c.png
Coingecko or Hardcoded Price?: Coingecko
Coingecko ID: https://www.geckoterminal.com/polygon_pos/pools/0x5250525a8fd922ec87b42c42b0ea71fd46565cb0
Harcoded Price (if Coingecko ID is unavailable):
Token HEX Color: #6cebae
Info on Listing: streaming to TBA ( nft smart wallet addresses
WARUx upgrade
https://polygonscan.com/tx/0x1487566d603a328541a786c098d60980325e3ea22bdc9d9bc3b390e278060f16
https://polygonscan.com/token/0x5034f401716a6ceb26c8232fb5640075508085af
https://console.superfluid.finance/matic/supertokens/0x5034f401716a6ceb26c8232fb5640075508085af?tab=streams
Please close only after token has been added to resolver!

listing done, but isn't picked up by the subgraph. Currently can't tell why (transfer event was also triggered).

is now listed in console


thread RateLimiter through lightweight datasources

released as 150.1.0-6e2cf24

released as 150.1.0-7ffe1e7

@djspiewak this is ready for review, but I don't want to merge it until https://github.com/slamdata/quasar-datasource-url/pull/397 is approved; at that point I'll merge the breaking change through the whole stack.


[BUG] Links in index.html are broken

What bug did you encounter?
When opening index.html in a browser, the CSS isn't working and the logo is broken.
What did you expect to happen?
Webpage should be stylized by using ./public/css/styles.css and look identical to This link on the repo.
Screenshots
Currently opening index.html looks like this:

It should look like this:

Additional information
To fix we just need to re-link styles.css and the logo image.
I would be willing to fix if you could assign it to me :smiley:
Thank you,

Oh nice catch @TeaBizzy:clap:
Feel free to open a PR on this, @MizouziE will go through it as soon as possible.

Please refer to #137
I'll close this for the time being.


Make MermaidFlowGenerator more resilient to class loading errors

I might have a test for this too but it requires changing the scanner to also allow scanning classes on the test classpath and I need to verify that there are no negative side-effects on other tests.

@nielsm5 I added some tests in a separate commit. If you do not like those changes, it's best I have to adding a unit test for this case. We can revert that commit but keep the actual fixes. :-)


This does not work on Android

Ive tried implementing it various ways and nothing is being blurred. I get a error saying I cannot wrap children in it when I try to use it to wrap the content. So I tried  to close it with  />'"

Got it to work by binding imageLoaded() {
this.setState({viewRef: findNodeHandle(this.refs.backgroundImage)})
} to the ref


New "Bicycle Dynamics for Microscopic Traffic Simulation" project page.

I finally wrote the project page for my PhD project.
I also adapted Anna's and my topic names on the member page to align better with what we are doing.
Finally, I also added a few things to .gitignore to keep the repo clean from any files of my local build. Pls remove if this is unwanted.

One issue, you've written directly to your source branch so I can't clone it without writing over my source branch. You should always create a new branch (branched from the latest source branch) then add your changes.

This looks great! If you fix that formatting issue we can merge.

One issue, you've written directly to your source branch so I can't clone it without writing over my source branch. You should always create a new branch (branched from the latest source branch) then add your changes.

Ah, I missed that. Should I open a new PR from another branch?

Close in favor of another PR from a different branch.


Project status and roadmap

Is there a rough plan for the project development?

+1

+1

1.0.0 was released so there is progress.


Implement user registration functionality in the Sudoku application

Description:
To enhance the user experience and allow for personalized features, we need to implement user registration functionality in our Sudoku application. This will enable users to create accounts, save their progress, and access additional features.
Tasks:

User Model:

Create a user model if not already present, using Django's built-in User model or extending it if additional fields are needed (e.g., user preferences).

Registration Form:

Create a registration form that includes fields for necessary user information (e.g., username, email, password).
Implement form validation to ensure data integrity (e.g., unique username, strong password).

Views and URLs:

Create a view to handle user registration requests and render the registration form.
Set up the necessary URL patterns to direct users to the registration page.

Templates:

Design a user-friendly registration template that matches the application’s overall design.
Include success and error messages to guide users through the registration process.

Email Verification (Optional):

Implement an email verification process to confirm user accounts after registration, enhancing security.

Testing:

Write tests to ensure that the registration process works correctly, including validation checks.

Acceptance Criteria:

Users can register successfully with a unique username and valid email address.
Form validation works as expected, providing feedback on errors.
The registration process is secure and user-friendly.
Optional email verification can be implemented to enhance security.

Won't do.


feat(hmr): improve circular import updates

Description
fix https://github.com/vitejs/vite/issues/10208
fix https://github.com/vitejs/vite/issues/7893
fix https://github.com/vitejs/vite/issues/3033
fix https://github.com/vitejs/vite/issues/10118 (not quite an exact fix but I think close enough, the issue has a lot of circular imports and fast refresh warnings that's causing things to update slowly)
When propagating HMR updates, we only propagate until we reach HMR-accepted modules, or the root (which we do a full reload.
A HMR-accepted module doesn't need a full reload as it can update itself. However, if the accepted module is within an import loop, updating itself can break the execution order of the import loop. This PR detects this and forces a full reload instead.

Any other import loops without a HMR-accepted module are fine. For example,
App.vue -> a.js -> b.js -> c.js -> a.js

If we update b.js and App.vue accepts HMR, App.vue will refetch a.js?t=123 and since a.js -> b.js -> c.js -> a.js is its own self-contained loop, the execution order and state should be fine.
If b.js (or any within the loop) accepts HMR, we can't guarantee the execution order anymore, e.g. it will refetch b.js?t=123, causing b.js -> c.js -> a.js (import order matters)
Additional context
Tests ported from https://github.com/vitejs/vite/issues/1477. The commit that fixes that issue introduces the full reload, but another the issue was that the leaf of circular imports wasn't properly invalidated. It is now properly invalidated, but I don't think we always need a full reload.

What is the purpose of this pull request?

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

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

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

Marking this back as draft. Actually it looks like currentChain when propagating the updates will never contain HMR boundaries (except partial accepts), so pretty much this PR will never reload now.
I think we still need to reload and that might be the real fix for https://github.com/vitejs/vite/issues/3033

/ecosystem-ci run

Amazing work on HMR all around for Vite 5! It looks good to me, but there is a new fail in plugin react that may be related: https://github.com/vitejs/vite-ecosystem-ci/actions/runs/6783049879/job/18436527990. It would be good if @ArnaudBarre gets some time to review this one, and then maybe we could ping Evan to speed up merging it as part of the major.

I ran the plugin-react's test locally and it's working fine. Could be a fluke 🤔

For the code, great, very clear thanks!
For the feature, I will just say that I don't think we're helping people in the long term to add a patch for circular imports.
This is too easy to have code working one day and not the other because you change one export or the chain start from another point because of another link. And as Evan W. pointed out once, the spec can be ambiguous for runtime on the order execution for complex graph.
People using circular imports are also adding really strong requirement on bundlers, and they will miss perf improvement on that side because new bundlers will not always have the edge cases right (ex. esbuild).

Oops, I forgot about this comment and merged this one. 😅

It would be good if @ArnaudBarre gets some time to review this one, and then maybe we could ping Evan to speed up merging it as part of the major.

@patak-dev Do you think we should wait for Evan? or was it because of the plugin-react's (flaky?) fail?

@ArnaudBarre I think we should still add more nudges in the docs for people to avoid circular imports (maybe in the performance or troubleshooting guide?). The PR doesn't feel like a patch though complexity-wise, so I think it was good merging it.
@sapphi-red we are good 👍🏼

Yeah I think the PR is leaning towards the real fix for the problem. The previous reload and circular detection was more of a patch that made HMR not ideal for those case. I agree though that circular imports is not great in general, and hopefully the explicit reloads compared to the silent fails before will bring more attention to the issue.

The latest Vite 5 beta should already include this

@bluwy I just tried 5.0 out an a large codebase with a lot of circular imports. Unfortunately we can end up in an infinite loop when changing certain files – vite becomes irresponsive and the only solution is to kill vite. I'll try to reproduce this on a smaller example that I can share.
But maybe this can already now guide us on the right track to fix this, by adding a log line for one of our files src/vite/index.tsx:
function isNodeWithinCircularImports(node, nodeChain, currentChain = [node], recursion=0) {
  if (node.url === '/src/vite/index.tsx') {
     console.log("isNodeWithinCircularImports 0", node.url, {nodeChainLength: nodeChain.length, currentChainLength: currentChain.length, recursion});
  }

Then I basically see the same file being analyzed with varying currentChain length but a stable nodeChain length. I also instrumented the recursion level but that doesn't explode – we just seem to be cycling around in the same nodes for some reason.
Let me know how I can help to fix this.

This is expected that the nodeChain is stable for a given node and then all the rest of the graph is explored. There is probably an issue with barrel exports where a long part of the graph is explored repetitively.
I think adding something similar to #12658 for caching could help.
Can you try to update your patch with this:
function isNodeWithinCircularImports(node, nodeChain, currentChain = [node], traversedModules = new Set()) {
  if (traversedModules.has(node)) return false
  traversedModules.add(node)
  // [...]
  isNodeWithinCircularImports(importer, nodeChain, currentChain.concat(importer), traversedModules)

@ArnaudBarre that works as expected, thanks!
PR up to fix this: https://github.com/vitejs/vite/pull/15034


generated package path failed

After installing with the command, the automatically generated package path failed to convert. Can you fix it？
I know it can be fixed by modifying the path, but it's not user-friendly for beginners because installing according to the steps in the documentation is not easy. Can you adjust this path so that it can be installed directly through commands without needing to be adjusted? Thank you

go mod tidy,

用这个命令是安装失败的

You need to init project and run go mod tidy.

Pls check your GOMODULE status, turn it on or auto and then execute go mod tidy, you can execute go env to show your GOMODULE status, such as:
$ go env
GO111MODULE='on'
GOARCH='arm64'
GOBIN=''
....


Microservices are having security issues reported on artifactory.io

Need to figure out why we have an F score.  https://artifacthub.io/packages/search?ts_query_web=ortelius&sort=relevance&page=1

Done for the following microservices:
ortelius-ms-compitem-crud
ortelius-ms-dep-pkg-cud
ortelius-ms-dep-pkg-r
ortelius-ms-textfile-crud
ortelius-ms-validate-user


Add sharing callback

The callback should ideally provide information as to whether the user actually shared, or whether they cancelled.

Any news about this feature?
I saw the PR here but it seems it won't be merged.

Hi there Valentin. The reason I haven't merged #14 is because we can't add the same feature on Android. We try to offer the exact same feature set on iOS and Android at the very least, and this feature will break that. I guess a feature like this is worth it. I'll merge #14 today or tomorrow.

Hey @ValentinChareyre, sorry I got to this super late. Check out #35 (and consider leaving a review, especially with your thoughts on the share callback dilemma).

@olokobayusuf : Thanks a lot for the work you and @sroske did on this feature. I believe the "parameter-less" callback is better than nothing and is a great improvement for your very nice plugin! I hope Android will fix the issue with their API on this problem.

Might be good to add this to SaveToCameraRoll also:
If you do:
NatShare.SaveToCameraRoll(path);
System.IO.File.Delete(path);

The file will never be saved to the Camera Roll.

@mikkoh this is pretty fair, but providing a callback for saving to the camera roll seems a bit overkill since the only use case I've seen so far is just deleting the file from app storage. Instead, it'll be more natural for the client to provide a copy argument to SaveToCameraRoll. Let's move this conversation to #38 .


compiling with node-sass leads bad fonts file url contaning quotes

Prerequisites

[X] I have searched for duplicate or closed issues
[X] I have validated any HTML to avoid common problems
[X] I have read the contributing guidelines

Describe the issue
Including the node_modules/bootstrap-icons/font/bootstrap-icons.scss and compiling with node-sass leads to quotes in the url("...") and the url is bad: /css/fonts/%22bootstrap-icons%22.woff2?24e3eb84d0bcaf83d77f904c78ac1f47
When I change this line:
$bootstrap-icons-font: "bootstrap-icons" !default;

to
$bootstrap-icons-font: bootstrap-icons !default;

The url is correct and can be loaded by the browser: /css/fonts/bootstrap-icons.woff2?24e3eb84d0bcaf83d77f904c78ac1f47
Reduced test cases
Install packages:
npm i bootstrap-icons
npm install node-sass

scss/main.scss file, include it:
@import "node_modules/bootstrap-icons/font/bootstrap-icons.scss";

Compile:
node-sass --output-style compressed -r scss/ -o css/

Include file in <link href="css/main.css"> and the font url is bad and cant be loaded by the browser:
/css/fonts/%22bootstrap-icons%22.woff2?24e3eb84d0bcaf83d77f904c78ac1f47
What operating system(s) are you seeing the problem on?
Windows
What browser(s) are you seeing the problem on?
Chrome
What version of Bootstrap are you using?
v1.10.3

This repo is for Bootstrap, not Bootstrap Icons.


⚠ SceneTime has degraded performance

In 063aaeb, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 1389 ms

Resolved: SceneTime performance has improved in fc548fa after 6 minutes.


bug: input looses autocapitalize after setFocus()

in ionic4 project which should default capitalizes the first letter when text area is focused and keyboard opens (The shift-arrow on the iOS keyboard is solid).
@ViewChild('focusInput', { static: false }) myInput;

    ionViewWillEnter() {
        setTimeout(() => {
          this.myInput.setFocus();
          this.nativeKeyboard.show();
        }, 1300);
      }

this code set focus in text area and opens the keyboard but the shift-key is no longer solid and the first letter is not automatically uppercased. How do I force the first letter to be automatically uppercased?

it is all working properly in ionic 3.

https://github.com/rpragesh/rpragesh
does this enough? Its ionic 4 project

@rpragesh I need a full app that compiles in order to test this. I recommend trying to reproduce the issue on a blank Ionic starter application.

@liamdebeasi  here is the repo [https://github.com/rpragesh/Keyboard-Focus-Demo](keyboard demo app) can you please try?
I have same issue with newly created app as well.

Thanks for the issue. I am going to close this as this is not a bug in Ionic Framework.
autocapitalize defaults to "off" on ion-textarea, so you will need to enable it in order to get the default capitalization to work. For more information, please see the docs: https://ionicframework.com/docs/api/textarea#properties

autocapitalize='sentences'   I used this instead of autocapitalize='true'. It was a mistake :(
Thank you for clarification.


increase test coverage to 78%

Mostly this will involve forcing more errors.

nomialmaps is at ~85% everywhere except for gpkit.tools


[ignore] Update dev-tr branch - merge main

Describe your changes
Related issue number or link (ex: resolves #issue-number)
Checklist before opening this PR (put x in the checkboxes)

[ ] This PR does not contain plagiarism

don’t copy other people’s work unless you are quoting and contributing it to them.

[ ] I have signed off on all commits

signing off (ex: git commit -s) is to affirm that commits comply DCO. If you are working locally, you could add an alias to your gitconfig by running git config --global alias.ci "commit -s".

/close
There are conflicts! Going to create a new PR manually.