Unit test rollup

wip

All commits above SQUASH ME will be squashed

12 commits! there were ~40 originally

Waiting on CI

On my system, it seems that I receive several network up events:
Not exiting when it is the case seems to solve it. See: https://github.com/ATmobica/connectedhomeip/blob/development/src/test_driver/openiotsdk/unit-tests/main/main.cpp#L51.

On my system, it seems that I receive several network up events:
Not exiting when it is the case seems to solve it. See: https://github.com/ATmobica/connectedhomeip/blob/development/src/test_driver/openiotsdk/unit-tests/main/main.cpp#L51.

Thanks, same happened in CI.


TGALoader.writeTGA does not seem to work(Linux)

When I run PtahRendererDemoOffline, I see the following output but the rendered files are nowhere to be found:
./.build/debug/PtahRendererDemoOffline
Starting Ptah Renderer...
Done!

I'm running swift-3.1 on Arch Linux:
swift -v
Swift version 3.1 (swift-3.1-RELEASE)
Target: x86_64-unknown-linux-gnu

The TGA writer was silently failing because the destination directory wasn't created in the repo. I have fixed both issues, please tell me if this works for you.

Thanks for the prompt response! It works for me now. By the way, is there any plan to support the online demo for Linux?

Well, your question has given me the motivation to implement it! 😃
I have added the PtahRendererDemoOpenGL module, that relies on an OpenGL 3 backend to display in real-time the output of the Ptah renderer, and handle simple user interactions.
The setup is a bit more involved than for the other modules: you need GLFW3 installed in its default location ; the other dependencies (Swift wrappers for OpenGL and GLFW3) will be downloaded by the Swift Package Manager.
I have been able to test it under Ubuntu 16.04, don't hesitate to report any bug you encounter.

Worked for me, although I had to make a copy of the GLFW shared object file to get it compiling. Thanks, I will close the issue now.


ML 3.11 : Implement K-Means without using any ML Library like scikit-learn or more (D)

Description
📌  Issues for Week 3
Welcome to 'ML' Team, good to see you here
:arrow_forward: This issue will helps readers in acquiring all the knowledge that one needs to know about ----
Implement K-means without using any ML Library like scikit learn or, more.
:red_circle: To get assigned to this issue, add your Batch Numbers mentioned in the spreadsheet of "Machine Learning", the approach one would follow and choice you prefer (Documentation, Audio, Video). You can go with all three or any number of options you're interested to work on.
✔ Domain : Machine Learning
:red_circle::yellow_circle: Points to Note :

The issues will be assigned on a first come first serve basis, 1 Issue == 1 PR.

"Issue Title" and "PR Title should be the same. Include issue number along with it.

Changes should be made inside the Machine Learning/Machine_Learning/Unsupervised_Machine_Learning Branch.

Follow Contributing Guidelines & Code of Conduct before start Contributing.

This issue is only for 'GWOC' contributors of 'Machine Learning' domain.

:white_check_mark: To be Mentioned while taking the issue :

Full name
Batch Number
GitHub Profile Link
Which type of Contribution you want to make :

[ ] Documentation (Coding with proper visualization)

Happy Contributing 🚀
All the best. Enjoy your open source journey ahead. 😎
Domain
Machine Learning
Type of Contribution
Documentation
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of this project.

I would like to work on this issue

/assign

Sorry @Ruchika-20  you've been unassigned due to violating the rules of not to use /assign

Name: Hrugved Kolhe
Batch: 3
GitHub Profile
[x]  Documentation
I would like to work on this issue

/assign

Aditya Kumar
batch: 04
Documentation

Tanvi Deshmukh
batch-5
I would like to contribute to this issue.. plz assign it to me.

I would like to work on this issue.
Name : Rahul Singhal
Batch : 1
Github Profile link: https://github.com/rahulsinghal1904
Approach : Intro about K-means and its required implementation without scikit-learn
Type : Documentation

@hrugved06 Issue assigned to you.

Sorry @Ruchika-20 you've been unassigned due to violating the rules of not to use /assign

I've used the label /assign sir....kindly have a look

Abuzar Bagewadi
Batch: 2
Github Profile: https://github.com/abuzarbagewadi
Documentation
I would like to work on this issue

@abhisheks008,  I have created a PR, please do check it :-D


Support ServiceAccount Tenant owner with SA name

Describe the feature
The proposal is to support in the ownerSpec, ServiceAccount-kind owners with name as Service Account name, besides the related  user's name (i.e. system:serviceaccount:<namespace>:<service account name>).
What would the new user story look like?
As a cluster/platform admin I would like to specify ServiceAccount tenant owners with the ServiceAccount name instead of the related user's name.
Proposed behavior
The tenant controller would keep the subject of the role binding either from the user name (name and namespace splitted by ':'):
func (r *Manager) ownerClusterRoleBindings(owner capsulev1beta1.OwnerSpec, clusterRole string) capsulev1beta1.AdditionalRoleBindingsSpec {
    ...
    if owner.Kind == "ServiceAccount" {
		splitName := strings.Split(owner.Name, ":")

		subject = rbacv1.Subject{
			Kind:      owner.Kind.String(),
			Name:      splitName[len(splitName)-1],
			Namespace: splitName[len(splitName)-2],
		}
                ...

or the subject.Name directly from the owner.Name and subject.Namespace from the Tenant's Namespace, e.g.:
func (r *Manager) ownerClusterRoleBindings(tenant *capsulev1beta1.Tenant, owner capsulev1beta1.OwnerSpec, clusterRole string) capsulev1beta1.AdditionalRoleBindingsSpec {
    ...
    if owner.Kind == "ServiceAccount" {
		...
		subject = rbacv1.Subject{
			Kind:      owner.Kind.String(),
			Name:      owner.Name,
			Namespace: tenant.Namespace,
		}
                ...

The tenant controller would keep the subject of the role binding either from the user name (name and namespace splitted by ':') or the subject.Name directly from the owner.Name and subject.Namespace from the Tenant's Namespace, e.g.:

This looks to me an egg/chicken problem: if the Tenant is created by the Cluster administrator, and no Namespaces are there, how can a ServiceAccount create Namespaces?
Although the verbosity for ServiceAccout name is not pleasant, I'd say it's pretty Kubernet-ish, just thinking of kubectl --as system:serviceaccount:default:my-sa create deployment ...
WDTY?


Space After Dot

CoffeeScript allows this:
a.    x
#or
a.

x

but it's almost certainly bad style when you can do things like this:
[1,2,3].
    filter((x) -> x % 2 is 0).
    map((x) -> x**2)

# or even worse
[1,2,3].

forEach (x) -> # Not obviously a method call of the previous list just from seeing the function
    console.log(x)

over this:
[1,2,3]
    .filter((x) -> x % 2 is 0)
    .map((x) -> x**2)

[1,2,3]

.forEach (x) -> # Easy to tell it belongs to something previous because of the dot
    console.log(x)

My proposed addition to add a rule to prevent spaces after a property access (spaces before the property access are unchanged)

You're welcome to build and publish a 3rd party rule for this. I don't use CoffeeScript any more, so this is unlikely to get implemented here. If you do implement it make sure to leave a comment here and take a look at https://github.com/clutchski/coffeelint/blob/master/3rd_party_rules.md
I highly recommend replacing CoffeeScript with Babel and CoffeeLint with ESLint. They are better tools that are better supported, and CoffeeScript doesn't appear to have a future as far as I can tell.

this is probably not implementable without hacking it like the indent rule. coffeescript just slurps up the first method in a chained call to the same line as the object callee(?)


.NET type DevExtreme.AspNet.Data.Types.AnonType cannot be mapped to a BsonValue.

Hi,
I'm trying to use total summary in DataGrid using mongodb and it's throwing this error ".NET type DevExtreme.AspNet.Data.Types.AnonType cannot be mapped to a BsonValue.". I didnt find how to overcome this error.

Hello
Currently, there are known limitations for IQueryable<BsonDocument> sources. Please refer to issues #421 and #422 for more information.
In general, these 2 settings may help resolve AnonType errors:
loadOptions.RemoteGrouping = false;
loadOptions.RemoteSelect = false;

For better compatibility, I suggest that you use mapped classes instead of BSON documents.

Reproduced.
Internal notes:

Use AllowEmpty = false for Mongo at
https://github.com/DevExpress/DevExtreme.AspNet.Data/blob/4cfd796b39d861798d22ab102c9b49c4edcbf8aa/net/DevExtreme.AspNet.Data/DataSourceLoadContext.cs#L47
GroupSummary without Group and without TotalSummary should not perform any aggregation.
Notify T900956 when fixed

Workaround:
BsonTypeMapper.RegisterCustomTypeMapper(
    Type.GetType("DevExtreme.AspNet.Data.Types.AnonType, DevExtreme.AspNet.Data"),
    new DevExtremeAspNetDataAnonTypeMapper()
);

class DevExtremeAspNetDataAnonTypeMapper : ICustomBsonTypeMapper {
    public bool TryMapToBsonValue(object value, out BsonValue bsonValue) {
        bsonValue = new BsonDocument();
        return true;
    }
}

Thanks, that works.


python311Packages.google-cloud-monitoring: 2.23.1 -> 2.24.0

Changelog: https://github.com/googleapis/google-cloud-python/tree/google-cloud-monitoring-v2.24.0/packages/google-cloud-monitoring

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 365856

x86_64-linux

:white_check_mark: 4 packages built:

python311Packages.google-cloud-monitoring
python311Packages.google-cloud-monitoring.dist
python312Packages.google-cloud-monitoring
python312Packages.google-cloud-monitoring.dist


[Feature request] Add transparent navbar for Android 10 navigation gestures

Why is this not fixed after all this time


Cannot run postprocessing because the postprocessing file is missing

There is a warning at the end of the run, similar to https://github.com/NeuroAI-HD/HD-BM/issues/1
WARNING! Cannot run postprocessing because the postprocessing file is missing.
Make sure to run consolidate_folds in the output folder of the model first!
The folder you need to run this in is /root/NeuroAI-HD/MS_Lesions/without_t1

More details below:
%cd /content
!git clone https://github.com/NeuroAI-HD/HD-MS-Lesions.git
%cd /content/HD-MS-Lesions
%pip install .

%cd ~/NeuroAI-HD/MS_Lesions
!wget https://zenodo.org/record/4915908/files/ms_lesions_weights.zip
!unzip -qq ms_lesions_weights.zip

!ms_lesions_noT1ce_predict \
 -i /content/mri_data \
 -id case \
 -o output

Just a warning. cf. https://github.com/NeuroAI-HD/HD-BM/issues/1#issuecomment-1446261035


NTR dermal chromatophore

fixes #2307

@aleixpuigb @dosumis Thank you for reviewing. I get this warning, and I do not understand what it means and what I should do

@paulacalipho, no worries. It's just a difference between Operational Systems and how they interpret the line ending in the file.

@aleixpuigb I understood I did not need to do the logical definition, right? Is there anything else to improve?

@aleixpuigb thanks, It is indicated merging is blocked. Could you do it for me?


System-tests: webconsole: new test for testSortConnectionsByContainerID

create test 'testSortConnectionsByClientID' by following document: https://github.com/EnMasseProject/enmasse/blob/master/documentation/design_docs/systemtests/selenium-tests/scenarios.adoc
if tests steps are not on place then must be defined

In connection tab there is no type for sort by ClientID -> closed

test is blocked by  #634


[ci:component:github.com/gardener/apiserver-proxy:v0.10.0->v0.11.0]

How to categorize this PR?

/area TODO
/kind TODO
What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Release note:

/close in favor of https://github.com/gardener/gardener/pull/7422

/close
in favor of https://github.com/gardener/gardener/pull/7422


Plugin Error: "YiiStorm" is incompatible (marked as broken).

Hi,
i get an error from the Version of PHPStorm 2020.1.
Plugin Error: Plugin "YiiStorm" is incompatible (marked as broken).
Will come to fix in near future?
Beste regards

+1

+1

+1

+1

+1

+1


fixed the region problem for : osdctl account cli -i  -o env

assign @fahlmant

@lnguyen1401 Can you update the description to explain what you're changing and why?


Add a reverse proxy option to MockWebServer

I'd like to include in my tests (which use Robolectric and MockWebServer) an analysis of the staging web server.
For unit tests, I could of course query the back-end server from the tests directly, but for integration-level tests, I would like to have my app generate the request, send it as usual (except I'll have mocked the web server), and for the tests to analyze the server response to the test.
I've built a crude Dispatcher that does this, but a first-party solution would be nice, in particular it would allow the RecordedRequest to also contain the server's response…

I think it’s unlikely we’ll build this. It’s a lot of work to do well! If you’d like to promote your dispatcher as a standalone gist or project, please do.

Here's a gist that implements a basic reverse proxy. Because the MockWebServer makes available only a subset of the Request data, not all data is sent to the upstream server.


Crash Reporter command doesn't support relative directories

Testing #98021
When utilizing the --crash-reporter-directory=<path> command line option, passing in a relative path causes code not to launch.

I require the path to be absolute, but now you should see a message on the console indicating this.


Getting participants in a call using the API

Hi
This is related to a post I made on Stackoverflow, but it didn't gain much traction and this seems like a better place for it.
When I try to request the list of participants in a call (as detailed on https://docs.microsoft.com/en-us/graph/api/call-list-participants?view=graph-rest-beta) the response suggests there are none. For example:
I place a call to the bot, and the calling endpoint receives a notification of an incoming call with id 471f0300-401f-4c4a-9967-3cee9a052519. The bot answers the call with a POST on:
https://graph.microsoft.com/beta/app/calls/471f0300-401f-4c4a-9967-3cee9a052519/answer
The bot subsequently receives a message on the calling endpoint that the call has been established. I can query the graph about this call by making a GET:
https://graph.microsoft.com/beta/app/calls/471f0300-401f-4c4a-9967-3cee9a052519
I get a response with code 200 and the details of the call in progress, so the call is clearly valid and accessible. However, if I attempt to get the list of participants with this GET:
https://graph.microsoft.com/beta/app/calls/471f0300-401f-4c4a-9967-3cee9a052519/participants
I get a response with code 200, but the body contains the following:
{"@odata.context":"https://graph.microsoft.com/beta/$metadata#app/calls('471f0300-401f-4c4a-9967-3cee9a052519')/participants","value":[]}
Shouldn't this contain the list of participants in the call? Is this implemented, and if so, is there an example of this working?
Thanks.
--
Edit:
Further to this, I am not getting a notification of the roster being updated when the bot invites participants. Assuming the bot invites someone to join the call with the above call Id, I issue a POST to:
https://graph.microsoft.com/beta/app/calls/471f0300-401f-4c4a-9967-3cee9a052519/participants/invite
The response contains the Id of a comms operation, and when the invitee accepts, the calling endpoint receives a message that the comms operation completed. I do not however get an update to the participants resource as detailed in this piece of documentation. I can't see that I'm doing anything wrong here. Any ideas?

Another issue is that I can't get the participant resource when I use the myParticipantId field which is returned when I GET the call. Surely I should at least be able to get this?
Example assuming the callId is 471f0300-401f-4c4a-9967-3cee9a052519:
I perform a GET on:
 https://graph.microsoft.com/beta/app/calls/471f0300-401f-4c4a-9967-3cee9a052519
I get back some JSON which includes the following:
"myParticipantId": "64e965cf-3672-4a58-accd-6fdd162be212"
I assumed I would be able to get the participant resource by doing the following:
https://graph.microsoft.com/beta/app/calls/471f0300-401f-4c4a-9967-3cee9a052519/participants/64e965cf-3672-4a58-accd-6fdd162be212
But instead I get a 404 with "Participant not found" as the message.

The inability to get participants is becoming a real problem for me, I'd like to try out audio routing groups but once again it seems I need to provide participant Ids. Does anyone else have this working?

Running the incident bot sample I see that a GET on calls/{id}/participants does actually work and I get a list of participants. That's when I follow the instructions in README.md and schedule a meeting, join it myself, then post on https://{botUrl}/joinCall with the meeting ID so the bot can join.
Can we only get participants from a pre-existing meeting the bot has called into? Or is there a way for the bot to get the participants in a call it has created?

I see this note on the documentation:
"Important: When a call is escalated from peer-to-peer to multiparty, not all multiparty features are available. Specifically, the bot will not receive roster updates."
So we don't get roster update notifications for a call, but does this mean that we cannot access the roster at all? And if so, why? Tracking participants seems like basic functionality.

@VinodRavichandran can you discuss the above subtleties?  It looks like gaps with escalation to me.  Any plan to address these?

@zhengni-msft  Can you please evaluate this. Is it just documentation update or fixes?

Hi, I am now getting roster updates on escalated calls via the calling endpoint, as well as being able to request the list of participants on escalated calls doing beta/app/calls/{id}/participants. This is what we were after, thanks.

@rugt0r
We're running into the same problem as you described in the first post of this thread:
Unable to get a list of participants of an incoming call. Also querying myParticipantId fails like you mentioned.
Since this issue is now closed, did you find a way to mitigate this?

Hi @zhengni-msft , @VinodRavichandran , @zhengni-msft,
Is the support of communications/calls/{id}/audioRoutingGroups/{id}  still planned for the  Graph API ?
this is a vry important feature for call centers Scenarios..
Any change to get this GA this year ?
Many thank !

Any news on this bug?

i tried all and i got
"code": "BadRequest",
"message": "Resource not found for the segment 'app'.",

i tried all and i got
"code": "BadRequest",
"message": "Resource not found for the segment 'app'.",

Anyone figured this out? I tried using the beta version as suggested but I still get value: [].


CASMTRIAGE-3336 use chronyc waitsync for skew test

no longer needed as this was merged and is a higher version https://github.com/Cray-HPE/csm/commit/471bc65fa2b53b6501432c3e53cfea10e90bb548


feat: EmitConfigOptions - add remove_comments

For the opt-in ability to remove comments from the transpiled output.

Nobody has asked for this. Let's just close this for now.


adding vijay kalmath as maintainer

Checklist

[ ] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] 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)
[ ] 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.

I merged the bot PR because it has the appropriate skips.


Documentation or wiki

Maybe a silly request, but can you add some documentation in the readme or add a wiki?
It'll help people like me whose coding skill is intermediate to get started a little bit easier.

Thank you for your reply. I'll try to understand the rawdraw.c then :)
BTW, an irrelevant statement, I am a big fan of your youtube channel and my interest towards embedded systems is highly inspired by your works. Thanks again for your reply. :)

Thank you!
Also, if you do end up making inroads here, I would REALLY appreciate help documenting this beast.


Font license issue

Excerpt from the Segoe MDL2 icons article https://msdn.microsoft.com/windows/uwp/style/segoe-ui-symbol-font
How do I get this font?
To obtain Segoe MDL2 Assets, you must install Windows 10.
And as stated in microsoft typography faq https://www.microsoft.com/typography/faq/faq11.htm
What can I do with the fonts supplied with Microsoft products?
The fonts are governed by the same restrictions as the products they are supplied with. You are not allowed to copy, redistribute or reverse engineer the font files. For full details see the license agreement supplied with the product.
That is: Segoe MDL2 font is not suited for MIT-licensed project or any other project at all because it can't be shipped with it
I'm looking forward to your response about this issue because i really like this project, but these issues can get users into some troubles

With WinJS, they had to fallback to a font package that was similar to but still different from MDL2.
https://github.com/winjs/winjs/tree/master/src/fonts
Some of those icons are kind of ugly though so an alternative to this alternative would be nice...

Found a decent alternative:
https://github.com/icons8/windows-10-icons
This one is free and has very limited set of icons. You can get more with paid version, but still - it is better than being sued

@Saurer @limefrogyank thanks for attention, on the issue of font license agreement, I hope I can redesign a set of icon font, but it will take time.

@myxvisual Is it possible to modify orionLibrary's icon and makes it fit the need of this Lib? It may reduce much work.
https://orioniconlibrary.com/

Intel's Clear Sans seems good.
https://www.fontsquirrel.com/fonts/clear-sans

Any update on this licensing issue? Unless changed, I believe everything developed using this package will be a subject of a copyright infringement due to inclusion of this folder which I guess is always included?

I found this post which pretty much covers the case when people do npm build i.e. embedding of source code files and resources. In this case Segoe MDL2.
So pretty much either there is no useful answer or use https://github.com/Microsoft/fonts and move away from Segoe.


rw_ddl_progress show 99% several seconds after create, but takes a few minutes to reach 100%

Is this a known issue?
Don't think it is super urgent,
just record this issue, and see if we can improve it later with more statistics at hand, or tone down the expectation a bit

Which version do you use? Recently I have refined it. https://github.com/risingwavelabs/risingwave/pull/10881

v1.0.0
let me try a newer version later


Aegir impossible to install (Drush was not able to start (bootstrap) the Drupal database.)

Hello Dominic, Helmo,
To learn Docker, I've tested both of your repositories and try to run a working Aegir.
To train myself a bit, I've structured the repository files.
You can find my work here: https://github.com/Polzme/docker-aegir-mariadb
Right now, everything is working except the Aegir installation. It fails at a certain point with this error.
You can review the whole log from ./run.sh in the container here: http://pastebin.com/RrV6GfDW
To reproduce the error,
Start: ./Docker_build.sh
Then: ./Docker_run.sh
Then: ./run.sh
Do you think the problem is coming from the installation script or Aegir ?
Thanks.

I think there is a issue with the permissions. I think aegir is not installed correctly, but i am not sure why the installation script fails. Sorry, i did not try to install this version of aegir yet....

It fails with aegir 6.x too, same error.

About Helmo, not yet asked.

I think this can be your problem: http://community.aegirproject.org/installing/manual#DNS_configuration

I tried, no luck :(
I'm trying to find where it fails, it's not easy to debug this thing.

It's been a while since I've worked with this. But from what I remember I had trouble with sudo.
Aegir uses sudo to reload apache, but Docker does not allow sudo.


Fix documentation and signature issues

This commit fixes some issues with the documentation and text_signature
attribute for methods. The documentation of digraph_union needed a bit
of cleanup and more details to explain how it works. The other issue
fixed here is the text_signature attribute for methods were missing the
'self' parameter. While the self is implicit in the typical call,
without it in the text_signature attribute the sphinx builds assume the
first parameter in the signature is self, and removes it from the
documentation. This commit fixes this by adding self to all the method
signature attributes.

Pull Request Test Coverage Report for Build 390304908

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

Totals

Change from base Build 389209601:
0.0%

Covered Lines:
2955

Relevant Lines:
3120

💛  - Coveralls


TLS support for the HTTP listener

To avoid having to run an additional webserver on the routinator host to get transport layer security (TLS) via a reverse proxy like nginx, it would be great if routinator itself had support for HTTPS like for example node_exporter.

We will add TLS to RTR in the next (major) release, anyway, so adding it to HTTP should be easy enough.
As a disclaimer: there won’t be ACME support nor will there be rate limiting and such, so at least public-facing Routinators should probably still be kept behind a reverse proxy.

We will add TLS to RTR in the next (major) release, anyway, so adding it to HTTP should be easy enough.
As a disclaimer: there won’t be ACME support nor will there be rate limiting and such, so at least public-facing Routinators should probably still be kept behind a reverse proxy.


[Awaiting C+ Payment] [HOLD for payment 2023-08-01] [$1000] Dev: Email address is not displayed on task preview when user assigns task to themselves in DM

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Open a tab where user A is signed-in
Open an existing chat with user B
Click on 'Actions' icon
Click on 'Assign task' link
Enter title & click on 'Next' button
Click on 'Assignee' & assign the task to yourself
Click on 'Confirm task' button
Switch to the tab where account B is signed in
Open the chat with user A

Expected Result:
Task report item should display user A's email address for both user A, and user B
Actual Result:
User A's email address is not displayed on task report item for user B
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number:  DEV
Reproducible in staging?: Dev only
Reproducible in production?: dev only
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/43996225/b14f2638-c17b-4012-a74a-81bb21c1cb12
Expensify/Expensify Issue URL:
Issue reported by: @natnael-guchima
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1688671191419889
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01cae9815a84594b7f
Upwork Job ID:<PHONE_NUMBER>028834304
Last Price Increase: 2023-07-17

:wave: Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open StagingDeployCash deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

Identify the pull request that introduced this issue and revert it.
Find someone who can quickly fix the issue.
Fix the issue yourself.

I can't help repro this on dev, sorry. @thienlnam might be aware of any recent task related PRs that might have caused this.

Definitely not a deploy blocker if it's only on dev

Are you able to repro though @thienlnam? Would be good to keep it moving if so!

I have checked this issue on my computer and it runs well.

Contributor details
Your Expensify account email<EMAIL_ADDRESS>Upwork Profile Link: https://www.upwork.com/freelancers/~012e4006e26ea0c254

I have checked this issue. But expensify is working properly on my browser. I can't see any issue.
It shows email address correctly.

Cool cool, going to close since we're getting multiple unable to reproduce comments - if someone can reproduce this reliably on main, please comment the steps

Hello @Oleksandrr97 @b4s36t4, can one of you share a video reproducing the issue? I am still seeing the issue on staging.
Screencast from 2023-07-12 10-50-20.webm

Sorry, I have checked again and I can see that issue.
Yesterday, I have checked it three times and I couldn't see that issue. But I can see today.

Let's reopen. @thienlnam did you attempt to reproduce? The steps in the OP I think are pretty clear to be able to.

Sorry, I have checked again and I can see that issue.
Yesterday, I have checked it three times and I couldn't see that issue. But I can see today.

No problem 👍

I can reproduce this as well. The steps are clear and I can reproduce it every time.

Proposal
Please re-state the problem that we are trying to solve in this issue.
When we assign a task to ourself, and share it with someone else, he doesn't see our email before the name of the task
What is the root cause of that problem?
In TaskPreview here: https://github.com/Expensify/App/blob/76e53bb3a49f00b4dd5cfeb3bc3b4ed219428eb7/src/components/ReportActionItem/TaskPreview.js#L69
We retrieve the account Id of the assigne from the taskReport prop.
But as explained in this comment:
https://github.com/Expensify/App/blob/76e53bb3a49f00b4dd5cfeb3bc3b4ed219428eb7/src/components/ReportActionItem/TaskPreview.js#L62-L64
The taskReport prop can be empty and in this case the information are present in the report action
What changes do you think we should make in order to solve the problem?
We can retrieve the assigne acount id in the report action when the task report is empty.
I believe the account id of the assignee is present in props.action.actorAccountID
So instead of this:
https://github.com/Expensify/App/blob/76e53bb3a49f00b4dd5cfeb3bc3b4ed219428eb7/src/components/ReportActionItem/TaskPreview.js#L69
We can put this:
const taskAssigneeAccountID = Task.getTaskAssigneeAccountID(props.taskReport) ||  props.action.actorAccountID;
What alternative solutions did you explore? (Optional)
Some code improvments are possible but I want to keep the proposal short and easy

@mananjadhav what do you think of this proposal?

Thanks for the bump, I missed this notification. I'll check this in a while.

I think @ShogunFire's proposal should work.
🎀 👀 🎀 C+ reviewed.

@mananjadhav  PR ready for review

I've added a comment to the offending PR.
I don't think any further action is needed on this one. I also don't think we need a regression test for this one.
@trjExpensify This would be ready for payout on 08/01, and eligible for timeline bonus.

Yup, agreed this one is covered under the task report regression tests now added!
Confirming payments as follows:
$1,500 - @ShogunFire for the fix & #urgency bonus (offer sent)
$1,500 - @mananjadhav for the C+ review & #urgency bonus (Manan to request)
$250 - @Natnael-Guchima for the bug report.
Sent on this job.

Accepted the offer. Thanks, Tom.

Hello @trjExpensify , thanks I accepted the offer but it was 1000$ instead of 1500$

Yeah, that will usually happen. I then modify upon ending the contract to incl. the bonus :)
Settled up with @ShogunFire and @Natnael-Guchima! Over to you Manan. I'll close this issue once actioned.

@trjExpensify I've raised a request on NewDot. I'll track the request there. You can close this one out.

I believe we leave them open until confirmation on our side is provided on the GH, then we close. :) CC: @JmillsExpensify

Reviewed details for @mananjadhav. These details are accurate based on summary from Business Reviewer and are now approved for payment in NewDot.

Perfect, thanks. Closing!


Middleware bearer for multiple client apps

Hi everybody,
I am intending to use the bearer middleware for multiple client apps (for instance I have a web app & a mobile app).
So, there is a way allowing me to define mulitple CLIEND_ID / IssuerUrl ?
Thank you very much,

I don't believe that the existing bearer middleware implementation supports that yet though it sounds like an useful feature enhancement to have.


Support Even/Odd and 90+ Version Schemes

Lastversion is supposed to return the latest stable versions, and often it returns an unstable version. For example:
> lastversion https://gitlab.gnome.org/GNOME/gtk
4.9.2

However the latest stable version is 4.8.3.
Many open-source projects, especially in the Linux ecosystem, use alternatives to semver or calver to indicate that a release is an unstable version:

Even/Odd versioning with 1.2.0 being stable and 1.3.0 being unstable
90+ versioning with 1.2.10 being stable and 1.2.90 being unstable

Although there is probably no way for lastversion to know that a project is using one of those schemes, maybe we could add an additional option to explicitly declare that a project is using them?

In the https://gitlab.gnome.org/GNOME/gtk there's really no designation of stability on tags that I see, whatsoever.
My interpretation still is that 4.9.2 somewhat stable but conservatively speaking and from their point, it's "preview".
They don't seem to define what's stable by any means other than their website.
We could use Wikipedia as a second source of truth.
Can you please provide other such repository examples?

FYI, this does work with the correct result:
lastversion https://en.wikipedia.org/wiki/GTK
4.8.3

Very interesting.
I wasn't aware of the .90+ thing. For this, I'd probably just have it added without an extra switch: to treat any x.x.90+ as unstable (since there are, I assume, no real projects with that many stable releases).
For the odd/even, add an --even switch?. Let me know your thoughts.

That sounds great to me! 👍

Added in lastversion 2.4.9

@dvershinin Thanks, that's a great update!
I did notice for one of the examples above:
> lastversion --even https://gitlab.gnome.org/GNOME/gobject-introspection
1.72.1

It should have return 1.74.0, which is the newest stable version. However, 1.72.1 was release just after 1.74.0.

@danyeaw please check lastversion v2.4.10. Addressed that issue.

Thanks @dvershinin!


Add support for Ollama LLM

Add support for the Ollama LLM (https://ollama.ai/)

Set it up like any other LLM provider
Add support for complete() and embed() methods: https://github.com/jmorganca/ollama/blob/main/docs/api.md#generate-a-completion

Working on this one.


prompt to reboot breaks automated runs

when I run this in an automated manner (i.e. from ansible) and don't specify the "auto reboot" option, it prompts me to reboot at the end, which times out/breaks since there's no user to give input.
Could it be made to not prompt? Perhaps it makes sense to add this as a feature to the new (and amazing) -q flag?

Yeap makes sense :) I'll sort this tomorrow.

Or today lol

Hey @ThisGuyCodes - this should be fixed in the master branch now, can you check and report back? :)

Beautiful! Actually broke me for a moment because I was using expect. Thanks again!


OMotM: Ban Unburden and implement Item Clause

http://www.smogon.com/forums/threads/full-potential-om-of-the-month.3596777/page-6#post-7264445

@TheImmortal


Shimmer Guthaben weg

Version
neuestes heute 18.10.2023
Description
Mein shimmer Guthaben ist nach der Aktualisierung Firefly weg - wo ist es ?
<EMAIL_ADDRESS>Expected behaviour
shimmer weg
Actual behaviour
shimmer weg
Can the issue reliably be reproduced?
Yes
Steps to reproduce the issue
shimmer weg
Operating System
Mac
Diagnostics
shimmer weg

Developer Profile
No
Profile Type
Software (Stronghold)
Staking
None
Crash Status
None of the above
Error Messages
kein
Error Logs
Guthaben shimmer weg

Duplicate declaration

[X] I have searched the issues tracker this issue and there is none

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Quick information about SMR: what the old firefly version displayed was just a representation of what you owned, but they were not the real shimmers. You must download the firefly shimmer app to control those funds, you can read more about it here https://blog.shimmer.network/how-to-claim-smr-firefly/
Will close the issue because the new firefly app is not intended to display old staking rewards, but the shimmer tokens should be managed from the shimmer app.
Please let us know if you have any other issues


NEI Nitrogen Oxides mapped to Nitrogen dioxide

I was surprised to find this mapping - is it intentional? As in, is there some reason that entities reporting NOx to NEI are actually reporting NO2?

I believe it was an assumption that NOx is primarily comprised of NO2 and the interest in making sure it is characterized, where factors for NOx are missing from some categories where factors for NO2 are present @hottleta @ashleyedelen


delete all messages from all channels older than certain date

I am trying to delete all messages from all channels/groups for all users before a certain date. How would I modify this command to achieve that?
slack-cleaner --token  --message --channel general --user "*" --before 20180424
Thank you.

@agilangovan  you must add '--perform' to the end.

But is it possible to target all channels?

And it is not possible to remove "RSS Feed messages". It simply cannot find them.

And it is not possible to remove "RSS Feed messages". It simply cannot find them.

need to add --bot

Do we have this option to delete from all channels?


libc: re-export libc properly

With rust 1.7, the following warning was being emitted by the compiler:
warning: `pub extern crate` does not work as expected and should
not be used. Likely to become an error. Prefer `extern crate` and
`pub use`.

Based on https://github.com/rust-lang/rust/issues/26775 it appears that
this change is the current best approach for properly exporting the
libc dependency so it may be used outside of nix.
Signed-off-by: Paul Osborne<EMAIL_ADDRESS>

Relates to https://github.com/nix-rust/nix/pull/284.
r? @fiveop

That is confusing. We just added the pub to extern crate because of an error in nightly, with a message stating that we cannot pub use a crate, that is private. The CI for this PR fails for exactly this reason in beta and nightly, now.
We need to figure out how to properly reexport a library in bea/nightly.

Agreed.

Ok, I believe what is present now should result in the correct behavior prior to the recent changes coming out of https://github.com/rust-lang/rust/issues/26775 and should also have the same behavior for beta/nightly.

Looks good here, assuming it works. I'll let @fiveop take another look though.

:umbrella: The latest upstream changes (presumably #294) made this pull request unmergeable. Please resolve the merge conflicts.

@homu r+

:pushpin: Commit f590b35 has been approved by fiveop

:hourglass: Testing commit f590b35 with merge 29aaccb...

:sunny: Test successful - status


Move to libadwaita by default

Let's switch to stock GNOME now that we have accent colors. I guess we should keep the yaru theme on the image for those that want it.
We need to figure out what accent color we want to set ootb. Note that we should merge this one close to release, it's best to make the change all at once when the OS updates.

We need to figure out what accent color we want to set ootb.

Easy! Blue for Bluefin, and purple for Aurora because it's a prominent color in the branding.

resoved by: https://github.com/ublue-os/bluefin/pull/1813 ?


Feature request: allow specifying files to exclude in ArchiveFiles task

Feature Request
Currently, ArchiveFiles adds every file in a given directory to the created zip file. It would be nice to be able to specify particular files/folders to not publish even if present, e.g. ".git". This can be worked around by adding a script that moves unwanted files outside the directory before running the ArchiveFiles task, but it would be nicer to be able to do this as part of the task itself.

This issue is stale because it has been open for a year with no activity. Remove the stale label or comment on the issue otherwise this will be closed in 5 days

a built-in option to ignore file and folder names that usually are not included in a release (.git folder, dev-stage-only stuff from the language, intermediary files from compilation, etc.) would also be useful in keeping down the size of final files and the time it takes to compress them.

This is still relevant.
The symmetry of having the Contents option like the sibling CopyFiles and DeleteFiles tasks would be good.

+1

Any workaround to this one? Or do I have to remove everything we don't want before creating the archive?

Instead of ArchiveFiles task found I could use Publish Pipeline Artifacts with a .artifactignore file.
Which let me deploy it in a release build.
          - task: PublishPipelineArtifact@1
            inputs:
              targetPath: $(rootFolder)
              artifact: WebApp
              publishLocation: 'pipeline'

Did the trick.

Instead of ArchiveFiles task found I could use Publish Pipeline Artifacts with a .artifactignore file.
Which let me deploy it in a release build.
          - task: PublishPipelineArtifact@1
            inputs:
              targetPath: $(rootFolder)
              artifact: WebApp
              publishLocation: 'pipeline'

Did the trick.

@LiamKarlMitchell is the Artifact compressed or zipped?

Not aware.
https://docs.microsoft.com/en-us/azure/devops/pipelines/tasks/utility/publish-pipeline-artifact?view=azure-devops

ok, it won't be a substitute for most.

+1 it would be great to get this feature implemented!
In the meantime, I used the CopyFiles task to create a folder with the items I needed, then used the ArchiveFiles task to create a zip of that folder.

This seems like such an important and obvious option to include I can't believe it wasn't in the original scope of requirements.
There are always going to be some files/folder we don't want to include, eg *.sqlproj or other development scripts not intended for others.
In my particular case, the zip being created is used by a DevOps Release pipeline which has a stage to copy to a fileshare for manual deployment, possibly by a customer! Sure we can export zip to temp folder, delete unwanted stuff, re-zip and delete temp folder before copying, but it would be so much easier to simply exclude files/folders in the first place.
It is now half way through 2021, 3.5 years since this was raised; is this ever likely to see the light of day?
Can we have an update please?

Plain and simple, being able to exclude specific files/folders when using ArchiveFiles is a much needed option. In the meantime, will be using an additional CopyFiles task to "stage" what I actually want to archive.

Anybody home? This option is really necessary. Or just excluding 'hidden' files (started with dot) will work too.

+1

ok, it won't be a substitute for most.

Yes, I have a similar issue where I want to create an artifact out of a subset of the files in a certain folder, and I want the artifact to be a zip file. Using an .artifactignore file and PublishPipelineArtifact is perfect to do the filtering, but in order to get it as a zip file I'm having to do a dance involving DownloadPipelineArtifact on the first artifact, ArchiveFiles on that, and then PublishPipelineArtifact again on the .zip. This results in two redundant artifacts, since I can't (easily) remove one of them, nor can I replace the original (unzipped) artifact.
I know this directly address the OP's issue, but having an option to zip on PublishPipelineArtifact would simplify this use case tremendously.
Or maybe I'm missing a simpler way that already exists?

+1
Excluding files from an archive is what most people do, it would be really beneficial if it can be added to ArchiveFiles tasks.

+1

In some cases, removing the unnecessary files with the DeleteFiles@1 task might serve as a workaround.

still needed

+1

+1
Adding a folder strictly to include/exclude specific files is unnecessary overhead.

+1

+1

+1

Any news about this feature? It's would be very useful

+1

+1

+1

If you're posting +1, make sure to also react with 👍 to the first post -- I understand that's a metric that might actually get used for prioritisation, whereas the comments just stave off stalebot.

Go with the workarounds.... this is 3 years old... I don't see this getting through.

+1

+1

+1 seems like a basic requirement and not that much work

+1

I used this task before archiving files. Hope this helps for time being.

task: Bash@3
inputs:
targetType: 'inline'
script: |
echo 'rsync'
rsync -av --exclude='.git' --exclude='app/Config/app.config . code

Been strugling but then realized I could just bash and keep calm :D
- bash: zip -r ${{ variables.ARTIFACT_NAME }} . -x .git/\*
      displayName: 'Create deployment artifact'

Hi @anilsakr,
Your suggestion worked for me. Thanks

1

We still want it please, just there isn't really to discuss hence the lack of activity.

Bump! Reacted to the post and commented to avoid this becoming stale. Much needed.

I assume you should use DeleteFiles@1  https://learn.microsoft.com/en-us/azure/devops/pipelines/tasks/reference/delete-files-v1?view=azure-pipelines
We have a 4GB .git dir so we have to do this too.


update save marks changes for performance main

@dileep-gadiraju  some test cases is failing for this pr .


Docker container /gtfs2ldes-js:20230306t2329: crashed with 2023-04-21 16:23:55 SyntaxError: Unexpected token { in JSON at position 1232

Hello @julianrojas87 ,
I am using your component  /gtfs2ldes-js:20230306t2329 to test the VSDS solution.
The full set up could be seen here: https://github.com/Informatievlaanderen/VSDS-LDES-E2E-testing/blob/main/tests/011.server-geo-fragment-large-ldes/docker-compose.yml
 gtfs2ldes-js:
    container_name: ${USECASE_NAME:-gtfs-geo-fragment}_gtfs2ldes-js
    image: ghcr.io/julianrojas87/gtfs2ldes-js:${GTFS2LDES_TAG:-20230306t2329}
    environment:
      - GTFS_SOURCE=${GTFS_SOURCE:-https://www.rtd-denver.com/files/gtfs/bustang-co-us.zip}
      - GTFSRT_SOURCE=${GTFSRT_SOURCE:-https://www.rtd-denver.com/files/bustang/gtfs-rt/Bustang_TripUpdate.pb}
      - BASE_IRI=${GTFS_BASE_IRI:-http://www.rtd-denver.com/}
      - NODE_OPTIONS=${NODE_OPTIONS:-}
      - TARGET_URL=http://ldio-workflow:8080/gtfs-pipeline
      - GTFS_CRON=${GTFS_CRON:-0 0 3 1 * *}
      - GTFSRT_CRON=${GTFSRT_CRON:-*/30 * * * * *}
      - RUN_ON_LAUNCH=${RUN_ON_LAUNCH:-true}
      - THROTTLE_RATE=${THROTTLE_RATE:-100}
      - AUTH_HEADER=${AUTH_HEADER:-}
      - AUTH_HEADER_VALUE=${AUTH_HEADER_VALUE:-}
    networks:
      - ldes
    profiles:
      - delay-started

After a certain period, there is following crash showed inside the docker log:
2023-04-21 16:23:55 SyntaxError: Unexpected token { in JSON at position 1232
The full log is here:
gtfs2ldes_js_20230306t2329.txt
Thanks
XD

I wasn't able to reproduce this issue. I would guess that it happened due to some corruption in the source data.
I will close it now but please re-open it if you encounter again, ideally with a source data sample of the moment it happened.


bug of incompatibility with the library time.h

I wanted to point out that the library RTClib.h not fully compatible with the library Time.h.
The incompatibility is caused by the function dayOfWeek () because also the other library has the same function. Personally, I solved the problem by renaming the function dayOfWeek () with dayOfWeekRtc ().
Hoping that the bug report can be of help, Best regards and good in 2015.

pardon the delay - did a sweep of this library and fixed this!

Excellent  :-)


Modify dialog prompt component

Type of change

Enhancement / new feature

Description
Modify dialog prompt component to handle confirm label button and color and add icon in the header

@anukritijha , No it will not affect other designs.


no issue, just wanted to show off my animation

https://user-images.githubusercontent.com/79553716/178054711-6e0890fc-4ab6-4e95-949d-0b51af17b9b1.mp4


Readiness probe is failing because of bad SSL

Chart version:
8.5.1
Kubernetes version:
v1.25.5
Kubernetes provider:
K8s (on prem / bare metal)
Helm Version:
"v3.9.4"
values.yaml
---
antiAffinity: "soft"

# Shrink default JVM heap.
esJavaOpts: "-Xmx128m -Xms128m"

resources:
  requests:
    cpu: "1000m"
    memory: "6Gi"
  limits:
    cpu: "2000m"
    memory: "6Gi"

volumeClaimTemplate:
  accessModes: ["ReadWriteOnce"]
  storageClassName: cstor-disk-pool
  resources:
    requests:
      storage: 20Gi

protocol: https

service:
  enabled: true
  labels: {}
  labelsHeadless: {}
  type: ClusterIP
  # Consider that all endpoints are considered "ready" even if the Pods themselves are not
  # https://kubernetes.io/docs/reference/kubernetes-api/service-resources/service-v1/#ServiceSpec
  publishNotReadyAddresses: false
  nodePort: ""
  annotations: {}
  httpPortName: http
  transportPortName: transport
  loadBalancerIP: ""
  loadBalancerSourceRanges: []
  externalTrafficPolicy: ""

updateStrategy: RollingUpdate

podSecurityContext:
  fsGroup: 1000
  runAsUser: 1000

securityContext:
  capabilities:
    drop:
      - ALL
  # readOnlyRootFilesystem: true
  runAsNonRoot: true
  runAsUser: 1000

esConfig:
  elasticsearch.yml: |
    xpack.security.enabled: true
    xpack.security.transport.ssl.enabled: true
    xpack.security.transport.ssl.verification_mode: certificate

Provide logs and/or server output (if relevant):
Events:
  Type     Reason     Age                     From     Message
  ----     ------     ----                    ----     -------
  Warning  Unhealthy  2m13s (x19360 over 3d)  kubelet  Readiness probe failed: Waiting for elasticsearch cluster to become ready (request params: "wait_for_status=green&timeout=1s" )
Cluster is not yet ready (request params: "wait_for_status=green&timeout=1s" )

Inside the pod (exec):
It's failing with https because the certificate is invalid:
[1] 144779
elasticsearch@elasticsearch-master-0:~$ curl: (60) SSL certificate problem: unable to get local issuer certificate
More details here: https://curl.haxx.se/docs/sslcerts.html

curl failed to verify the legitimacy of the server and therefore could not
establish a secure connection to it. To learn more about this situation and
how to fix it, please visit the web page mentioned above.

On http it's not working as well:
elasticsearch@elasticsearch-master-0:~$ curl http://localhost:9200/_cluster/health?wait_for_status=green&timeout=1s
[1] 144849
elasticsearch@elasticsearch-master-0:~$ curl: (52) Empty reply from server

I have created a pull request to address this: https://github.com/elastic/helm-charts/pull/1774


FileNotFoundError: Local file G:/En3d/200000.zip doesn't exist

Getting this error when running app.py:
Traceback (most recent call last):
File "app.py", line 467, in
app = WebApp(debug_mode=False)
File "app.py", line 141, in init
MsDataset.load('3DHuman_synthetic_dataset', namespace='damo', subset_name='200000').config_kwargs[
File "G:\miniconda3\envs\en3d\lib\site-packages\modelscope\msdatasets\ms_dataset.py", line 284, in load
dataset_inst = remote_dataloader_manager.load_dataset(
File "G:\miniconda3\envs\en3d\lib\site-packages\modelscope\msdatasets\data_loader\data_loader_manager.py", line 132, in load_dataset
oss_downloader.process()
File "G:\miniconda3\envs\en3d\lib\site-packages\modelscope\msdatasets\data_loader\data_loader.py", line 83, in process
self._prepare_and_download()
File "G:\miniconda3\envs\en3d\lib\site-packages\modelscope\msdatasets\data_loader\data_loader.py", line 147, in _prepare_and_download
self.dataset = self.data_files_manager.fetch_data_files(
File "G:\miniconda3\envs\en3d\lib\site-packages\modelscope\msdatasets\data_files\data_files_manager.py", line 112, in fetch_data_files
builder.download_and_prepare(
File "G:\miniconda3\envs\en3d\lib\site-packages\modelscope\msdatasets\download\dataset_builder.py", line 293, in download_and_prepare
self._download_and_prepare(dl_manager=dl_manager)
File "G:\miniconda3\envs\en3d\lib\site-packages\modelscope\msdatasets\download\dataset_builder.py", line 296, in _download_and_prepare
self.split_path_dict = dl_manager.download_and_extract(
File "G:\miniconda3\envs\en3d\lib\site-packages\datasets\download\download_manager.py", line 434, in download_and_extract
return self.extract(self.download(url_or_urls))
File "G:\miniconda3\envs\en3d\lib\site-packages\datasets\download\download_manager.py", line 257, in download
downloaded_path_or_paths = map_nested(
File "G:\miniconda3\envs\en3d\lib\site-packages\datasets\utils\py_utils.py", line 511, in map_nested
mapped = [
File "G:\miniconda3\envs\en3d\lib\site-packages\datasets\utils\py_utils.py", line 512, in
_single_map_nested((function, obj, batched, batch_size, types, None, True, None))
File "G:\miniconda3\envs\en3d\lib\site-packages\datasets\utils\py_utils.py", line 380, in _single_map_nested
return [mapped_item for batch in iter_batched(data_struct, batch_size) for mapped_item in function(batch)]
File "G:\miniconda3\envs\en3d\lib\site-packages\datasets\utils\py_utils.py", line 380, in
return [mapped_item for batch in iter_batched(data_struct, batch_size) for mapped_item in function(batch)]
File "G:\miniconda3\envs\en3d\lib\site-packages\datasets\download\download_manager.py", line 313, in _download_batched
return [
File "G:\miniconda3\envs\en3d\lib\site-packages\datasets\download\download_manager.py", line 314, in
self._download_single(url_or_filename, download_config=download_config)
File "G:\miniconda3\envs\en3d\lib\site-packages\datasets\download\download_manager.py", line 323, in _download_single
out = cached_path(url_or_filename, download_config=download_config)
File "G:\miniconda3\envs\en3d\lib\site-packages\datasets\utils\file_utils.py", line 222, in cached_path
raise FileNotFoundError(f"Local file {url_or_filename} doesn't exist")
FileNotFoundError: Local file G:/En3d/200000.zip doesn't exist

those files are hosted on chinese provider - you can sign up without china number
I pushed a single zip files to this branch 000182.zip
https://github.com/johndpope/En3D/


:bug: Applied AutoValidate and DisableAutoFillHints appropriately

Fixes Issue 138.
Applied autovalidateMode to all TextFormField as per the value of widget.autovalidateMode.
Fixed passing value of autofillHints to all TextFormField based on the value of widget.disableCardNumberAutoFillHints.

please update changelog.md

please rebase this pr.


Blur amount is not respected in latest master

Hi there,
I'm testing the latest bits from the master branch and it looks like blurAmount property is not respected on iOS. It looks like the issue is from this change.
@eliperkins Do you have an idea what might be wrong?

latest commits are not released because I'm waiting for https://github.com/react-native-community/react-native-blur/pull/174 which solves the issue you're mentioning.

Hi @ligaz - in the meantime, you could point to my branch which works on both iOS and Android. You can add this to your package.json:
"react-native-blur": "ndbroadbent/react-native-blur#fix-ios",


(BKR-715) Use puppet_agent_version for MSI from dev repos

If a puppet_agent_version is specified, use it in
install_puppet_agent_dev_repo_on to fetch the MSI, so we get an MSI
unique to that SHA. This provides a work-around for BKR-712, and
normalizes behavior with install_puppet_agent_on.

This breaks when we set puppet_agent_version to opt[:version].

Refer to this link for build results (access rights to CI server needed):
http://jenkins-beaker.delivery.puppetlabs.net//job/qe_beaker_btc-intn/2233/
Test FAILed.

Updated to fix specs and ignore if puppet_agent_version is a SHA.

Refer to this link for build results (access rights to CI server needed):
http://jenkins-beaker.delivery.puppetlabs.net//job/qe_beaker_btc-intn/2234/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
http://jenkins-beaker.delivery.puppetlabs.net//job/qe_beaker_btc-intn/2235/
Test PASSed.


implement systemd-resolved metrics collector

collect systemd-resolved metrics through dbus
add new option to enable collector
fix #37

here are some test results with systemd 237
$ systemd --version
systemd 237
+PAM +AUDIT +SELINUX +IMA +APPARMOR +SMACK +SYSVINIT +UTMP +LIBCRYPTSETUP +GCRYPT +GNUTLS +ACL +XZ +LZ4 +SECCOMP +BLKID +ELFUTILS +KMOD -IDN2 +IDN -PCRE2 default-hierarchy=hybrid

$ uname -a
Linux vietnam 5.4.0-1029-aws #30~18.04.1-Ubuntu SMP Tue Oct 20 11:09:25 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

$ go version
go version go1.21.5 linux/amd64

run exporter
$ go run main.go --systemd.collector.enable-resolved

check metrics
$ curl -s http://localhost:9558/metrics |egrep -i systemd_resolved
# HELP systemd_resolved_cache_hits_total Resolved Total Cache Hits
# TYPE systemd_resolved_cache_hits_total counter
systemd_resolved_cache_hits_total 1.019122e+06
# HELP systemd_resolved_cache_misses_total Resolved Total Cache Misses
# TYPE systemd_resolved_cache_misses_total counter
systemd_resolved_cache_misses_total 896694
# HELP systemd_resolved_current_cache_size Resolved Current Cache Size
# TYPE systemd_resolved_current_cache_size gauge
systemd_resolved_current_cache_size 4
# HELP systemd_resolved_current_transactions Resolved Current Transactions
# TYPE systemd_resolved_current_transactions gauge
systemd_resolved_current_transactions 0
# HELP systemd_resolved_dnssec_bogus_total Resolved Total number of DNSSEC Verdicts Boguss
# TYPE systemd_resolved_dnssec_bogus_total counter
systemd_resolved_dnssec_bogus_total 0
# HELP systemd_resolved_dnssec_indeterminate_total Resolved Total number of DNSSEC Verdicts Indeterminat
# TYPE systemd_resolved_dnssec_indeterminate_total counter
systemd_resolved_dnssec_indeterminate_total 0
# HELP systemd_resolved_dnssec_insecure_total Resolved Total number of DNSSEC Verdicts Insecure
# TYPE systemd_resolved_dnssec_insecure_total counter
systemd_resolved_dnssec_insecure_total 1
# HELP systemd_resolved_dnssec_secure_total Resolved Total number of DNSSEC Verdicts Secure
# TYPE systemd_resolved_dnssec_secure_total counter
systemd_resolved_dnssec_secure_total 0
# HELP systemd_resolved_transactions_total Resolved Total Transactions
# TYPE systemd_resolved_transactions_total counter

$ systemd-resolve --statistics
DNSSEC supported by current servers: no

Transactions
Current Transactions: 0
  Total Transactions: 1871553

Cache
  Current Cache Size: 5
          Cache Hits: 1019123
        Cache Misses: 896695

DNSSEC Verdicts
              Secure: 0
            Insecure: 1
               Bogus: 0
       Indeterminate: 0

I believe this PR is ready for review. Could someone please check it out?


Sharing config for many nodes

I'm wondering if there is an easy way to configure a node to use a specified shared config folder from the hub?  We're looking into managing and auto scaling a large grid and it would be great to be able to specify that an auto created node would use say "config1" or "config2", rather than relying on the machine name folder exist on the hub.
Thanks...

@rnavarette  So config1 might be for Chrome, config2 might be for Firefox, config3 might be for IE9, config4 might be for IE10, etc ?
So in this case you only had 4 configs , instead of the number of nodes.
I just wanted to make sure I understood your request clearly.

Yep, thats the idea.  Then, I might be able to spin up say 5 nodes of each.

I like the idea.

I've got more :)
Seriously though, is this the place for these discussions or is there a forum or mailing list?  I think this project makes the node configuration WAY easier than SG out of the box.  However, I think there is quite a bit more than could be done yet to make it even better.
I'd love to commit where I can as we put together our solution, but a good first step would be having a solid group vet some of the ideas and see where they best fit.

There is no mailing list or forum specific to Selenium-Grid-Extras. Though you can usually find me in #selenium on irc.freenode.net .


New branding

May I say, I'm glad this lives on. Such a shame somebody bought the project and has just left it to wilt. With that in mind, it'd be great to get this rebranded and going again. For that you need a domain. I'd like to suggest: pleasedelete.me If you like it, buy it quick before someone holds it to ransom over you.

We appreciate the sentiment but I believe answer will be similar to the one on this issue.

The answer there is to leave discovery to a google search. If that is to be the case, it at least needs rebranding, otherwise people will always end up back at the original.

Also there's so many links / social accounts on the site that end up back at the original. Do you need a site maintainer? I'm happy to go through and update it but I would insist on a rebrand.


Extension is using an API proposal 'lmTools' that is not compatible with the current version of VS Code

Copilot Chat Extension Version: v0.18.1
VS Code Version: 1.93.0-insider
OS Version: 14.5 (23F79)
Logs:
GitHub Copilot Extension Version: v1.220.0
Other system information:

Version: 1.93.0-insider
Commit: ff8504ae52f8e0d0e9a33e6cb326efc21e42db3e
Date: 2024-08-06T05:03:52.944Z (6 hrs ago)
Electron: 30.1.2
ElectronBuildId: 9870757
Chromium: 124.0.6367.243
Node.js: 20.14.0
V8: <IP_ADDRESS>-electron.0
OS: Darwin arm64 23.5.0

Steps to Reproduce:

Just updated VS code insiders to the version above (1.93.0-insider) and started getting the error (screenshot below). The extension is not loaded and copilot chat view no longer appears in the activity bar.
I have tried restarting the extension, reloading the window, restarting the application as well as switching to the pre-release version of Github Copilot (v1.220.1032 (pre-release)) but none of that worked.

Screenshots:

Extension page

Startup error

Please switch to pre-release of GitHub Copilot Chat. With insiders you must use the pre-release version of copilot chat

I can probably get the message to mention this.

Updating the VS Code Insiders to the current build fixes the issue! Guess the issue can be closed now.

Please switch to pre-release of GitHub Copilot Chat. With insiders you must use the pre-release version of copilot chat

how to do this?
Thanks

I started seeing this message after updating VS Studio Insiders to the latest version this morning. I am using GitHub CoPilot Chat v0.19.2024080501 (pre-release). So I suppose updating VS Studio Insiders is not an option, given I have the latest version and this causes the problem?

I started seeing this message after updating VS Studio Insiders to the latest version this morning. I am using GitHub CoPilot Chat v0.19.2024080501 (pre-release). So I suppose updating VS Studio Insiders is not an option, given I have the latest version and this causes the problem?

I uninstalled and re-installed using the specific version option. I picked one version older and it works again. Just wait to update again after a new version comes up

I started seeing this today as well in VS Code Insiders. I tried uninstalling both GitHub Copilot and GitHub Copilot Chat, then successfully installed GitHub Copilot v1.223.1064 (pre-release) but failed to install GitHub Copilot Chat v0.20.2024082301 (pre-release) with error Can't install 'Github Copilot Chat' extension. Extension is using an API proposal 'lmTools' that is not compatible with the current version of VS Code.
My VS Code version is:
Version: 1.93.0-insider
Commit: c2aa3c4625ed4b46448fa546ac5f2e93b0757718
Date: 2024-08-22T05:03:56.076Z
Electron: 30.3.1
ElectronBuildId: 9960165
Chromium: 124.0.6367.243
Node.js: 20.15.1
V8: <IP_ADDRESS>-electron.0
OS: Linux x64 6.10.6-061006-generic

I'm getting the same error.

I'm seeing this as well with VSCode Insiders build.

@sandy081 It sounds like when the vscode build doesn't support the api version of the latest extension, maybe it's not finding the compatible pre-release version in the marketplace? Is that possible?

@sandy081 I installed c2aa3c4625ed4b46448fa546ac5f2e93b0757718, one build before Friday's build which had an API version increase, and I can repro the issue.

I am on the following version of VS Code Insiders:
Version: 1.93.0-insider (Universal)
Commit: c2aa3c4625ed4b46448fa546ac5f2e93b0757718
Date: 2024-08-22T05:03:56.076Z
Electron: 30.3.1
ElectronBuildId: 9960165
Chromium: 124.0.6367.243
Node.js: 20.15.1
V8: <IP_ADDRESS>-electron.0
OS: Darwin arm64 23.6.0

I have the GitHub Copilot extension installed: v1.223.1065 (pre-release)
But when I try to install GitHub Copilot Chat v0.20.2024082301 (pre-release) I get the following error message:
'Can't install 'GitHub Copilot Chat' extension. Extension is using an API proposal 'lmTools' that is not compatible with the current version of VS Code.'

@josephdaw  You can update vscode or go back an extension version to get unstuck.

I updated VS Code Insider to 1.93.0-insider and the GitHub Copilot extension to v1.223.1066 (pre-release). There is now error message any more, but the Copilot chat has disappeared.

maybe it's not finding the compatible pre-release version in the marketplace? Is that possible?

That's true. I am not going back in versions to find the one that is api compatible. I do not remember the reason, it could be an oversight.
Fixing it.

Please switch to pre-release of GitHub Copilot Chat. With insiders you must use the pre-release version of copilot chat

how to do this? Thanks

I haven't even really initiated Insiders, but hat this too. After a few checks I saw a button to "restart extension" at the gitGub Copilot. After klicking that it was fine.

Also an issue for me once I updated to this months release:

Currently facing the same issue
github copilot chat: v0.20.3
VSCode: 1.94.0 (Universal)


in_app_purchase plugin - Auto-Renewable Subscription never expires in Sandbox

I've setup an Auto-Renewable Subscription, and was successfully able to purchase it using my sandbox user account (on an actual device).
The problem is that unlike what is described here (Testing Subscription: https://medium.com/swift-india/auto-renewable-subscriptions-for-ios-3a4068f11acd), the subscription doesn't expire. I've waited for more than 2 hours, terminated the app, uninstalled and re-installed it - nothing helped...

Hi,
I'm interested to know how do you set up your auto renewal subsciprition. I try also to set up this, with buy non consumable, but with or without the sandbox account, i have no error or message and nothing append when i pass the purchase method.
To the second question, with sandbox account, the management of the subscription is different from a classic account ? If not maybe in settings you can manage this.

I've followed the following tutorial and just setup a "normal" subscription (AppStore):
https://fireship.io/lessons/flutter-inapp-purchases
At first, I got exceptions inside the plugin itself, when I called the queryPastPurchases method, see here: https://stackoverflow.com/questions/57133835/flutter-getting-an-exception-when-calling-querypastpurchases-in-app-purchase-pl
It doesn't happen any more, not sure why...
Please verify that:

You test this on an actual device (and not an emulator) - since some of the methods won't work there
That you are running with the latest version of Flutter, pods etc.

Using the plugin I've observed that:

Buying a subscription works just fine (I can see 1 item in the purchaseResponse.pastPurchases)
As documented (for sandbox users), after 5 minuets it seems like there's an automated renewal (I can see 1 item in the purchaseResponse.pastPurchases)
After 5 more minuets I can see another renewal and so on, until there's a total of 5 subscriptions
But then - I can see 1 subscription, where I expect to see 0 (no active subscriptions). See here: https://stackoverflow.com/questions/57164366/flutter-how-to-use-in-app-purchase-plugin-to-implement-subsciptions

Not sure if that's a bug on something I'm doing wrong.
Used the sample code (from the plugin repository) on a clean Flutter project.
If you manage to make it work - please share your workflow

Thanks you,
I miss the emulator clause, in iphone it's work nice. Not the best to debug, but better than nothing.
I follow the code and the guideline from apple, nothing special.

This is also happening for me, @zaidmain, did you solve it? I think we are missing docs here. Thanks.

@dannycortesv
Since then I've learned that subscription is quite a complicated issue: you need to validate a receipt against Apple/Google servers (and you can't do that locally), which is quite a pain if you're not using the native libraries (google for "validate subscription receipt).
After doing some tests, I've ended up using revenuecat (read this: https://www.revenuecat.com/2018/05/22/ios-subscriptions-are-hard).
Quite happy with them :)

@dannycortesv
Since then I've learned that subscription is quite a complicated issue: you need to validate a receipt against Apple/Google servers (and you can't do that locally), which is quite a pain if you're not using the native libraries (google for "validate subscription receipt).
After doing some tests, I've ended up using revenuecat (read this: https://www.revenuecat.com/2018/05/22/ios-subscriptions-are-hard).
Quite happy with them :)

Thanks for your help @zaidman i'm considering using revenuecat

Any update on this? I'm still experiencing this issue. I purchased the last auto-renewable subscription in Sandbox almost 24 hours ago and pastPurchases is still returning this. It was my understanding that sandbox subscription expire in about 30 mins, is that not that case?
Tested on : iPad (running iOS 13.5.1)
in_app_purchase version: 0.3.4

After some struggle with this mechanism, I've implemented RevenuCat solution.
Working fine for me: https://www.revenuecat.com

Any update on this? I'm still experiencing this issue. I purchased the last auto-renewable subscription in Sandbox almost 24 hours ago and pastPurchases is still returning this purchase. It was my understanding that sandbox subscription expire in about 30 mins, is that not the case?
Tested on : iPad (running iOS 13.5.1)
in_app_purchase version: 0.3.4
[✓] Flutter (Channel stable, v1.17.1, on Mac OS X 10.15.2 19C57, locale en-US)
    • Flutter version 1.17.1 at /Users/user/Applications/flutter
    • Framework revision f7a6a7906b (6 weeks ago), 2020-05-12 18:39:00 -0700
    • Engine revision 6bc433c6b6
    • Dart version 2.8.2

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
    • Android SDK at /Users/user/Library/Android/sdk
    • Platform android-29, build-tools 29.0.2
    • ANDROID_HOME = /Users/user/Library/Android/sdk
    • ANDROID_SDK_ROOT = /Users/user/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b4-5784211)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 11.5)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.5, Build version 11E608c
    • CocoaPods version 1.8.4

[✓] Android Studio (version 3.6)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 45.1.1
    • Dart plugin version 192.8052
    • Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b4-5784211)

[!] VS Code (version 1.45.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    ✗ Flutter extension not installed; install from
      https://marketplace.visualstudio.com/items?itemName=Dart-Code.flutter

[✓] Connected device (1 available)

any solution for this? how to detect canceled/not renewed subscriptions?


Removing 'parkingpass.bcparks.ca' as a vanity url

Ticket:
NOBUG
Description:
Due to the wildcard cert causing problems with *bcparks.ca domains, DUP's certificates were recently upgraded to be more specific. DUP CloudFront no longer contains a wildcard certificate for *bcparks.ca, rather, it has reserve.bcparks.ca for DUP public and reserve-admin.bcparks.ca for DUP admin. The added complexity of doing this in Go Daddy vs Route53 has forced us to remove the other vanity url for DUP public, parkingpass.bcparks.ca.
Terraform had this removed url hardcoded in its build files and would bomb when the ACM Cert ARN that it pulled didnt support parkingpass.bcparks.ca. This ticket is to remove that hardcode. Another ticket will have to be created to investigate the feasibility of reintroducing this vanity url if necessary.

@JLWade FYI

@cameronpettit I dont think we need the parkingpass vanity url, just reserve.bcparks.ca/daypass


Single portable .exe

Is your feature request related to a problem? Please describe.
The folder with the executable is cluttered.
Describe the solution you'd like
Have the build output a single .exe or equivalent that can be run standalone.
Describe alternatives you've considered
Have tried warp-packaging, and CoreRT. Neither suited our needs.
System Configuration (please complete the following information):

OS: All
CLI or GUI: CLI

Alternatively, a directory structure where all of the supporting files were stored in a separate directory, like:
attack_surface_analyzer.exe
readme.txt
resources\
   resources\foo.dll
   resources\bar.dll
   …

I think this would be fine too.

As a stopgap we could make a .bat/.sh file that just runs "resources\AttackSurfaceAnalyzer.exe", and put everything into the resources directory.

That would certainly work for me.

I saw that this is going to be a native feature of .NET Core 3, at least on Windows platforms.
I did experimentally create the running scripts, checked in in the Cli directory, but haven't fully tested that everything works with them (for example, I suspect the Compare command may not because it needs to load Output.cshtml from the filesystem).

Unfortunately the .NET Core 3.0 approach is basically the warp packer approach, which is not desirable for our use case.  Further minor experiments with other ILMerge type approaches haven't been successful either.

The best workaround for now is to first install .NET Core and then run
dotnet tool install Microsoft.CST.AttackSurfaceAnalyzer.CLI
See nuget.org

This should be possible in .NET 5.
https://devblogs.microsoft.com/dotnet/introducing-net-5/

Depends on https://github.com/dotnet/runtime/issues/36590 in .NET 5.
dotnet publish -r win-x64 /p:PublishSingleFile=true /p:IncludeNativeLibrariesInSingleFile=true
dotnet publish -r linux-x64 /p:PublishSingleFile=true

Depends on dotnet/runtime#36590 in .NET 5.
dotnet publish -r win-x64 /p:PublishSingleFile=true /p:IncludeNativeLibrariesInSingleFile=true
dotnet publish -r linux-x64 /p:PublishSingleFile=true

Note that native binaries are still extracted to the Temp folder if bundled into a single binary. Otherwise they are placed in the same directory

@fabi that is the implementation in .net core 3.1 which is why we are not using it currently.
In the issue linked this is addressed in .NET 5
Can run managed components of the app directly from bundle, without need for extraction to disk.

@Fabi that is the implementation in .net core 3.1 which is why we are not using it currently.
In the issue linked this is addressed in .NET 5
Can run managed components of the app directly from bundle, without need for extraction to disk.

Thats why I write native binaries :)
Managed are directly executed, native are always extracted.

I understand now the distinction you are drawing. It sounds like it may still need to extract native binaries. Thanks for the extra info!


add a CI comment

You can see that CI started building all of mathlib, since it did not find Mathlib.init.


Detect when users are trying to access the 1.x REPL, and return an explicit error

A common question we see from users migrating from 1.x is "how do I find the REPL?". We should detect when users are trying to access the REPL and return a clear error, ideally guiding the user towards the 2.x UI for interactive queries.

But maybe also add a link in the error message to docs on how to build the repl. There are some use cases for it (like wanting to write csv form a local file with flux).

See issues above to add the REPL's back to the CLI - maybe we get those done soon enough this is unecessary?

We now have a REPL again using influx v1 repl, merged in #386.
Should running influx hint to the user something like NOTE: If you're looking for the 1.x REPL, try "influx v1 repl" or something like that?


Add Reader.read_row for decoding into caller-provided buffer.

Add Reader.read_row for decoding into caller-provided buffer.
Reader.read_row is an alternative to Reader.next_row.

Reader.next_row is convenient when the caller doesn't want to manage
their own buffer, and doesn't mind having extended borrows on Reader
to accommodate Row data.
OTOH the new Reader.read_row avoids an extra copy in some scanarios
which may lead to better runtime performance - see the benchmark
results below.

The PR adds a row-by-row/128x128-4k-idat benchmark (initially based on
Reader.next_row which requires an extra copy).  Using the new
Reader.read_row results in the following performance gains in the
benchmark:
time:   [-16.414% -16.196% -15.969%] (p = 0.00 < 0.05)
time:   [-15.570% -15.218% -14.856%] (p = 0.00 < 0.05)
time:   [-16.101% -15.864% -15.629%] (p = 0.00 < 0.05)

This PR replaces https://github.com/image-rs/image-png/pull/421.  (This PR can be seen as a non-breaking-change alternative of the other, earlier PR.)

There doesn't seem to be any reason not to merge this. @anforowicz could you resolve the conflicts? I'll push the merge button as soon as that's done.

@anforowicz could you resolve the conflicts?

Ack.  I'll do that later today.  My apologies for missing this message earlier.

There doesn't seem to be any reason not to merge this.

Let me share some of the lingering doubts I have regarding this PR:

Cons of this PR are small, but non-zero:

New public API (and public APIs are "forever";  or at least changing the APIs requires the friction of a new major version release and/or imposing some pain on crate users)
A bit of an unclear story wrt keeping 1) keeping next_row / next_interlaced_row and also read_row from this PR, vs 2) deprecating next_row / next_interlaced_row and only exposing read_row in some future major version release

Pros of this PR are unclear:

Helps to avoid an extra copy to widen the scanline in SkPngRustCodec::expandDecodedInterlacedRow, but interlaced images are relatively rare + SkPngRustCodec has other inefficiencies around processing them.
This PR definitely helps if the output of png doesn't need any further transformations before being usable by the crate client.  But... I now think that maybe this is not the case for Skia/Chromium, which requires alpha-premultiplied output.  Depending on png output:

Indexed: can't save the extra copy / palette expansion hop (this hop is done at the Skia/Chromium layer - see https://crbug.com/356882657)
Rgb, Gray or GrayAlpha: can't save the extra copy (need to transform into RGBA)
Rgba (~54% of images from top 500 websites):

Even with this PR there would still be a need for an extra transformation loop for alpha-premul (and sometimes for rgba=>bgra, or for gamma/iccp).  (I have my brief notes about post-decoding transforms in a doc here.)
Maybe this PR can save a memcpy and then apply alpha-premul and/or rgba=>bgra transformation in-place.  But this is tricky, because SkSwizzler doesn't support in-place transformations, so we'd have to switch to skcms_Transform in some cases.

I think it's okay to merge this. Having read_row doesn't hurt, because even if we wanted to remove it, we can always just make a copy into the buffer.


AWS RDS error - No database instance with id: found. Please specify a valid db instance.

I am trying to connect to my RDS db with cloud AWS. Please see attached source files.
I used @EnableRdsInstance to specify the dbInstanceIdentifier and password as specified in document. I created an EC2 instance to run this application as fully executable jar, while RDS instance was launched separately. Please note I did not use CloudFormation template.
I also updated RDS security group to allow traffic from all IPs on 5432 port (PostgrSQL). I have attached demo code which produces this issue (RDSDemo.zip.txt) and the output file.
If this is not an issue it would be good to add a sample for AWS RDS.
Thanks,
output.txt
RDSDemo.zip.txt

This was a configuration issue. Functionality is working fine as expected.


Error with make_inverse_operator

Hi! We are trying to run source analysis.  @kingjr
We are running MNE version '0.16.dev0'
So far this is what we have done:
1 We prepare the source space:
src = setup_source_space(subject=freesurfer_subject, subjects_dir=freesurfer_dir,
                                 spacing='oct6', surface='white',
                                 add_dist=True, n_jobs=-1, verbose=None)
bem_dir = os.path.join(freesurfer_dir, freesurfer_subject, 'bem')
src_fname = os.path.join(bem_dir, freesurfer_subject + '-oct-6-src.fif')
write_source_spaces(src_fname, src, overwrite=True)

2 We run the forward model
fwd = forward_pipeline(raw_fname, subjects_dir=freesurfer_dir, subject=freesurfer_subject,
                       overwrite=overwrite)

3 We compute noise covariance
covs = compute_covariance(epochs, tmin=None, tmax=0,
                          method=['shrunk', 'empirical'],
                         return_estimators=True)
covs[0].save(cov_fname)
cov = covs[0]

4 And then, when we try to calculate the inverse model:
inv = make_inverse_operator(epochs.info, fwd, cov, loose=0.2, depth=0.8)
We get this error:

TypeError: 'NoneType' object has no attribute 'getitem'

Here you have a caption with the full report:

Does anybody know what might be wrong here? Many thanks in advance

forward_pipeline is not an MNE function so it's tough to help you debug. But based on the error it looks like forward[...] is causing the problem, which means the fwd you have is probably None

Closing due to non-response / likely bug in custom code, but if you can isolate it to MNE calls let us know


Export to google street view

Hi is it possible to export it to google street view view ?

No, sorry.


TPKeyboardAvoidingScrollView not working at all at version 1.2.2 via cocoapods

reverting to 1.2.1 solves the problem.  I really don't know enough about the project to provide more details other than I tested this with a simple UIView containing a UIScrollView (set to TPKeyboardAvoidingScrollView) and a single UITextField at the bottom of the screen.
Did not set the delegate, so it's running off of the notifications.
Update: setting the delegate didn't help either.

I am experimenting same issue, TPKeyboardAvoidingScrollView is not working at all after moving to iOS 8. I cannot even see the keyboard to appear. Revert back not working at all.

I'm unable to reproduce this on the sample app; can you put together something that demonstrates the issue?

Hello.
There are a couple of issue I guess with high priority.
First one, related to this issue, I guess not already solved yet but important:
https://github.com/michaeltyson/TPKeyboardAvoiding/issues/75
when you perform an installation with cocoa pods simply run the controller, you get:
[TPKeyboardAvoidingScrollView TPKeyboardAvoiding_updateContentInset]: unrecognized selector sent to instance
Simply add UIScrollView+TPKeyboardAvoidingAdditions.m to my main project as resource to be compiled it disappears, even if it was already added to pods related project. I think it was not good at all and this happens just with new iOS 8 SDK (Xcode 6). Maybe a issue related to cocoa pods? I am using latest version, 0.35.0.
Second one, more important: using iOS 8 SDK as a base but running the controller under iOS 7 Simulator you cannot see even the keyboard to appear. Everything goes right if iOS 8 simulator is used.
To see this issue, to answer you question, I just run your sample project, once compiled with iOS 8 SDK, into iOS 7 Simulator. If you pick "View", "Table" or "CollectionView" is the same, keyboard won't even appear. Just try.

Related to the second issue I told you before, I opened another issue you can take care of:
TPKeyboardAvoiding not working at all when using IOS 8.1 SDK and IOS 7.1 Simulator #154
Thank you.

It works fine for iOS 8.1 iPhone 4s Simulator, but the same project for iOS 8.1 iPhone 6 Simulator it dosen't. Keyboard doesn't pop up.

I am not experimenting the same issue, because with iOS 8 and iOS 8 Simulator everything is fine, but with iOS 7 Simulator keyboard does not pop up. I do not know if this is an issue related to Apple Simulator or it is an issue of this controller. Michael can you tell us something about it, please. I will try on real device but I would happy to know something from you since you can actually understand if it is an issue.
Thank you very much.

Hello every one. I think I have found the actual problem. I am speaking about iOS 7 Simulator issue (keyboard does not pop up). Read this, they hall me solve this issue. Days wasted anyway...Hope this helps.
http://stackoverflow.com/questions/25842168/cant-find-keyplane-that-supports-type-4-for-keyboard-iphone-portrait-numberpad/25954053#25954053
http://stackoverflow.com/questions/24871532/xcode-ios-8-keyboard-types-not-supported

@foffux thank you, it helped.

Glad to hear that. Xcode 6 sucks....

Glad you guys figured it out =) Good to know.


Feat: ES module support (Node)

Issue: https://github.com/open-runtimes/open-runtimes/issues/63
If you have index.ts:

It will build to index.js:

OPR for Node now supports this, and executes properly:

If you keep the sanity check, you may want to check if userFunction.default is also a function, and I would maybe look into changing the error message to something along the lines of "The exported code must export a function."

@JakeAi  I think that's a good suggestion.
@Meldiron Please incorporate this check as well.


Open previous session

With the introduction of chat history you can now resume an old session, are there any plans to add support for this?

When you enable logging, the log has context strings written to it.  You can load these contexts using the context command.

@Tecuya is this also a feature for the API, or is it just the shell program? I didn't see any options or arguments that the ChatGPT class accepts that would allow for logging / context restoring, but maybe I am missing something there.

This is more of a feature of the shell. It should be easy to integrate in the python part. I don't have currently time to do it, however any PR would be welcome :)

I wrote this method to load the context in my application using the chatgpt-wrapper. It is pretty generic so you should be able to use it easily.
def load_context(self, context_id):
    try:
        (conversation_id, parent_message_id) = context_id.split(":")
        assert conversation_id == "None" or len(conversation_id) == 36
        assert len(parent_message_id) == 36
    except Exception as e:
        print(e)
        raise
    if conversation_id == "None":
        conversation_id = None
    self.chatGPTBot.parent_message_id = parent_message_id
    self.chatGPTBot.conversation_id = conversation_id

I wrote this method to load the context in my application using the chatgpt-wrapper. It is pretty generic so you should be able to use it easily.
def load_context(self, context_id):
    try:
        (conversation_id, parent_message_id) = context_id.split(":")
        assert conversation_id == "None" or len(conversation_id) == 36
        assert len(parent_message_id) == 36
    except Exception as e:
        print(e)
        raise
    if conversation_id == "None":
        conversation_id = None
    self.chatGPTBot.parent_message_id = parent_message_id
    self.chatGPTBot.conversation_id = conversation_id

How can I get the uuid of a certain message (i.e. parent_message_id) by pressing F12 to inspect the website in a browser?

You can look under the Network tab after pressing F12. iirc the payload for the conversation request will have it

It works well. Thanks a lot!
Is it possible to get the parent_message_id of previous messages? It seems that this method only works on newly generated messages.

/history and /switch commands implemented long ago in the shell.


[T3A5][T11-A2]Joshua Ching

Ready for review

I messed up my commits. 2nd and 3rd commit should be one together. Sorry about that!
Ready for review


Support for App Protocol

Seems like currently its impossible to use app protocol for the main file. https://github.com/nwjs/nw.js/wiki/App-protocol Setting the main html file to: "app://index.html" Wont let me export. Maybe just a checkbox which adds the app://?
Thanks for your great project!

Hey Ben,
The problem with doing that is that it creates an issue with exporting the executable. When specifying the app protocol, nwjs expects that your application will lay in the current directory (instead of the temporary directory that nwjs creates for the app). This results in a blank "NWJS" page. I don't think there is a way to do this (currently) without breaking how I currently have exporting set up.
What is the reason that you need the app protocol?


Do not merge

Just a test for travis and scritinizer. :angel:
/me is too lazy to enable them for my repo. :stuck_out_tongue:

That was broken anyway.


Added two links.

SimulaQron (quantum internet simulator) and QFlow (experimental tune-up software).

Great projects! Thanks for the contribution @csferrie!


The video show an abnormality after the character turn, is that expected?

First of all, I would like to thank the author for his work, but after 38 seconds of the video, it is abnormal and can not be played, is that expected?
https://github.com/user-attachments/assets/a3bd22ca-a3c3-48d0-8702-f21793e59143

Hi @bailehang, yes we observed this kind of behavior too! These are autoregressive generations, so the accumulating error can cause the model to be out of distribution. It was actually quite surprising to us how long the generation manages to remain stable.
Also, you can press Enter to reset the generation to a fresh starting point, so it should be "playable" again.
We think of our repo as a way to share our research with the community. We did not design it as a fully-featured, playable, user-facing product.

Thank you for your reply and sharing.


Adding support for objectRestSpread within props

We hit at an issue properly parsing props that looked like
<div foo={{ params: {...router.params}}} />

because the SpreadProperty doesn't have a key/value pair. Opting to ignore spread properties, but properly recurse over the key/values we can parse.

Coverage increased (+0.02%) to 98.844% when pulling a5b9461b3f705497e1a6c5acf70ef4dc7291d59f on jnwng:jw/fix-spread-property into e4c031233d6bf220bcf15b4c0d80367b2c340e0e on evcohen:master.

Coverage increased (+0.02%) to 98.844% when pulling a5b9461b3f705497e1a6c5acf70ef4dc7291d59f on jnwng:jw/fix-spread-property into e4c031233d6bf220bcf15b4c0d80367b2c340e0e on evcohen:master.

Coverage increased (+0.02%) to 98.844% when pulling 26d15eaa644bd6a884d1dfa3b0c747d611d45cd4 on jnwng:jw/fix-spread-property into e4c031233d6bf220bcf15b4c0d80367b2c340e0e on evcohen:master.

Coverage increased (+0.02%) to 98.844% when pulling 2eefda376f4a75d1f99c81f6c36b43aa7cc3e860 on jnwng:jw/fix-spread-property into e4c031233d6bf220bcf15b4c0d80367b2c340e0e on evcohen:master.

@evcohen @ljharb thanks for the quick review! as an FYI, this was an issue because eslint-plugin-jsx-a11y is 1) stuck in npm with a dependency on<EMAIL_ADDRESS>(not sure what happened there, the code in master is fine and 2) the objectRestSpread errors coming from that.
when this gets merged, would it be possible to bump eslint-plugin-jsx-a11y too? here's the pull request for that one
thank you so much!

@ljharb When can this be merged? It fixes #69.

@ljharb Thanks for getting this merged! What's the timeline for a release?

That's up to @evcohen; i don't have publish rights nor am I familiar with the release procedure for this project.

@evcohen When can we get a new release with this PR?

Can work on that this weekend! thanks for your patience 😄

Thanks!


Write a TypeScript program to convert string to snakecase

Description
Write a TypeScript program to convert string to snakecase
Snakecase is a style of writing in which all letters are lowercase and spaces are replaced with underscores.
Input  : "hello world"
Output : "hello_world"

How to contribute

Save the solution in program/ConvertStringToSnakecase.ts file
Add ConvertStringToSnakecase.ts file in convert-string-to-snakecase folder

!assign


feat(orchestrator): add Workflow Run List

FLPATH-693
A component listing running workflows is added.
TODO:

[x] rebase once one of following lands:

https://github.com/janus-idp/backstage-plugins/pull/956 (preferred, Janus backstage-plugins)
or https://github.com/caponetto/backstage-plugins/pull/24 (PR-local)

[ ] clarify backend API and adapt

@jkilzi could you please take a look at this PR when you have some time?


github and snyk warn of security issues in dependency 'marked' 0.3.6

There are apparent security issues in marked==0.3.6, 0.3.7 .. solution is to upgrade to 0.3.9 or better.  0.3.12 appears to be latest.  Adding esdoc as a dev dependency to my project causes me to have to use npm-shrinkwrap.json to force marked to update, which is a bit less than desireable.

also having this issue. marked is not on 0.3.19! so its been a while :)

You need to update it again due to this.


🐛 [BUG] - Slack link is inactive

Description
When clicking the Slack link on the website, it reads This link is no longer active.
Reproduction steps

Go to https://opentofu.org/slack

Screenshots
No response
Logs
No response
Browsers
No response
OS
No response

Fixed in #210


getPendingFriends

Hey,
We have already got getFriends() function to get Users with Friendship ACCEPTED status and other methods like getMutualFriends(), getFriendsOfFriends() . How about to introduce getPendingFriends to get Users (not friendships model like we do in getPendingFriendships()) ?
I want to show a list of pendings friends with some user info that I need get from the User Model and a button to accept the friendship request with a clean and easy method? What do you think? Or maybe I missed something?

Hi @nikolaynesov,
It sounds good to me. What do you expect getPendingFriends to return?
The pending friends that the user has sent a friend request to or the pending friends despite if the user sent or received the friend request?

@hootlex I am expecting to get Users that sent requests to Current User but their requests are still not ACCEPTED.  Right now in my controller I temporary do it like this:
` /**
* Get authorized User pending friends
*
* @return \Illuminate\Http\JsonResponse
*/
public function ajaxGetPendingFriends() {
    $friendships    = Auth::user()->getPendingFriendships();
    $senders        = $friendships->pluck('sender_id')->all();
    $query          = User::where('id', '!=', Auth::user()->id)->whereIn('id', $senders);
    $pendingFriends = $query->paginate($this->friendsPerPage);

    return response()->json($pendingFriends);

}`

But it's a bit wrong and not efficient because getPendingFriendships() returns all friendships and not only where current user is recipient so then I need to use WHERE for user query to exclude current user... looks like we need another querybuilder method?
BTW how is it going with #16 ? is it going to solve my issue and make the package more flexible?

Another thing is what if I want to make and auto-complete ajax search by pending friends, declined friends, accepted friends or friends in some group? ))) It means we need a more flexible way to build queries. Should it look something like:
$user->friends()->accepted()->group('close_friends')
$user->friends()->pending()->search('John S')

A more flexible API is definitely needed. Unfortunately, @stephane-monnot and I, are too busy at the moment. I will probably start working on #16 next month.
Regarding your current situation, I suggest to create two temporary methods to use until we go to V2:

getPendingIncomingFriends
getPendingOutgoingFriends

BTW, I think that V2 deserves a decent documentation. I'd be glad if you guys could help me when the time comes. 😄

@hootlex Also another question related:
Why did you decide to use separate query to get friendships, then merge recipient and sender ids and then run another query to get actual users in getFriendsQueryBuilder() instead of creating a single INNER JOIN query? Are separate queries faster then INNER JOIN ... ON ..sender_id... OR ...recipient_id... in this case? I did not run any tests so maybe you did that.

@nikolaynesov are you going to implement these methods before the v2?

@hootlex not sure yet, but will let you know by Friday.


Add jQuery prompt in v8+

Continue of https://github.com/h5bp/create-html5-boilerplate/pull/12

Punt!


Various axom::Array changes

Summary

Adds Array::front(), Array::back() methods, and Array::resize(size, value) overload to more closely conform with std::vector API
Adds ArrayView::empty() method to match std::span
Makes Array move constructors noexcept

@kanye-quest can you add some tests for these?