[MINOR][DOCS] Update supported versions for Hive Metastore

What changes were proposed in this pull request?
Since SPARK-18112 and SPARK-13446, Apache Spark starts to support reading Hive metastore 2.0 ~ 2.1.1. This updates the docs.
How was this patch tested?
N/A

Hi, @gatorsmile .
If possible, can we update the docs consistently with the code, too?

Test build #75717 has started for PR 17612 at commit d6792c2.

Test build #75717 has finished for PR 17612 at commit d6792c2.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75717/
Test PASSed.

Thanks! Merging to master

Oh, thank you, @gatorsmile !


Update with master

This PR aligns beta with latest contents from master

:tada: This PR is included in version 7.7.0-beta.1 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Update Docs related to configuring access control policy for service provider

Is your suggestion related to a missing or misleading document? Please describe.
In the [1], It describes how to add a xacml policy to a service provider, to authorise users based on the role. As the 'authenticate' is used as the action in the xacml policy, this is not working for the 'password grant' though it works fine for the other grant types.
Describe the improvement
In the docs it should be mentioned that this policy is only working for grant types other than password grant.
[1] https://is.docs.wso2.com/en/5.10.0/learn/configuring-access-control-policy-for-a-service-provider/

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


chore: allow change of install directory windows

Description of change
Creating this pull request in case we feel that this would be beneficial. Specifically when we create alpha or beta builds, windows will install it over the users main firefly application. Switching from the NSIS one click installer to the multi step will allow the user to select the location where they want to install.
Type of change
Choose a type of change, and delete any options that are not relevant.

Chore (refactoring, build scripts or anything else that isn't user-facing)

How the change has been tested
Built branch using github actions and test install on windows machine.
Change checklist
Add an x to the boxes that are relevant to your changes, and delete any items that are not.

[ ] I have followed the contribution guidelines for this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have checked that my latest changes pass CI tests
[ ] New and existing unit tests pass locally with my changes

Specifically when we create alpha or beta builds, windows will install it over the users main firefly application.

This may stem from the fact that electron-builder may be using the package name desktop instead of the app name Firefly/Firefly Beta on Windows. I noticed some people reporting that some files were located in C:/Users/<username>/AppData/Local/Programs/desktop/ on Windows. I might investigate that issue later but this is a good fix for now


Swaps positions of the "confirm" and "cancel" buttons in the dialogue window for better usability.

Hi! Love your plugin. But there is some thing I changed for myself locally and thought maybe you would be interested to see it.
When I use the "Super Export", the dialogue window opens up, and its more handful if the "Confirm" button is right below the percentage input field.

For example, here how the buttons in the default Aseprite Export tool are positioned:

Also! The "On the following window, set the percentage at 100%." is quite annoying to click on every time i export images. Could you please add a "never show again" button on it or just delete it (and possibly add this as a description in the dialogue window)

Hey there! I am so sorry I didn't see this! I will be happy to review this PR shortly. Thank you for your hard work!

This PR looks good to me! I'll go ahead and merge. Then I'll work on the "never ask me again" feature. Thanks for the request!


SLF4J Logging Warnings

When I trying running a simple http4s client app with Typelevel Toolkit, I get warnings about SL4FJ at startup:
SLF4J: Failed to load class "org.slf4j.impl.StaticLoggerBinder".
SLF4J: Defaulting to no-operation (NOP) logger implementation
SLF4J: See http://www.slf4j.org/codes.html#StaticLoggerBinder for further details.

I can get rid of them with //> using dep ch.qos.logback:logback-classic:1.5.3
Maybe the toolkit should include a default logging implementation out of the box? I have no interest in remembering to configure logging when I'm trying to do a quick script.

I think this issue sits on the edge between being an http4s issue and being a toolkit issue.
That said, IMHO there's no simple solution: adding logback is absolutely not the right thing to do, as it won't work on scala native and scala.js.
On the other hand adding something like log4cats no-op will mute every possible log stament and code that is copy pasted from bigger codebases will work but not log any statement.
Forcing users to decide and pick their own logging framework is probably still the best thing to do.
Despite the fact I appreciate the "all-in-one" nature of the toolkit, I think this is one of the cases where keeping it general has more value than keep it EVEN MORE practical.

adding logback is absolutely not the right thing to do, as it won't work on scala native and scala.js.

I wouldn't go so far as to say that. We could add it for JVM only.

I think this issue sits on the edge between being an http4s issue

This is literally an http4s issue :)

https://github.com/http4s/http4s/issues/6721

I wouldn't go so far as to say that. We could add it for JVM only.

Yay, it was probably too bold, sorry

Not having SLF4J in the toolkit dependency tree would be even better. Is http4s 0.23.x stuck with it? If I'm reading the issue right, it's already removed from 1.0?

Forcing users to decide and pick their own logging framework is probably still the best thing to do.

But I don't want to be forced to pick a logging framework for the web scrapper I'm quickly typing up and going to use 3 times! IMO, the value of having a toolkit is greatest for people writing fairly small scripts that aren't going to see extensive use. Having to puzzle out a cryptic warning from a non-Typelevel dependency that is somehow the first thing my http4s script prints out is not great ergonomics for that use case.

Not having SLF4J in the toolkit dependency tree would be even better

I agree.

Is http4s 0.23.x stuck with it? If I'm reading the issue right, it's already removed from 1.0?

Yes, stuck in 0.23, removed in 1.0.

Having to puzzle out a cryptic warning from a non-Typelevel dependency that is somehow the first thing my http4s script prints out is not great ergonomics for that use case.

😢 yeah this sucks

That is really unfortunate. So there is no way out until 1.0. Adding an slf4j logger to the toolkit would kind of work, until the user does want to pick a logger, because slf4j also complains about having more then one.


Additional ESLint rules

I had a bunch of ideas yesterday so I just built them all. Just get to this the you can :) @GoodForOneFare

Quibbles over wording, but the code is solid 👍


Update QAEvaluator to indicate ground truth is required

Since QAEvaluator is a composite that includes SimilarityEvaluator and F1ScoreEvaluator, both of which require ground_truth as an input, we should specify that ground_truth is required for the composite evaluator as well.

@lgayhardt
Can you review the proposed changes?
Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Thanks so much!
#sign-off


banglerun v0.06 - HDOP from GPS, added README

This PR introduces a change in the BangleRun app, which now uses the GPS event instead of the GPS-raw event to read GPS data. The change is made possible by the introduction of the HDOP (Horizontal Dilution Of Precision) value in the GPS event since Espruino 2v07. The value is needed by the Kalman filter to work properly, and to provide accurate tracking of position and distance.
This PR also adds a README file, containing the instructions to transpile the project's source files (written in Typescript) into code compatible with the BangleJS.

That looks great - thanks!

That looks great - thanks!

Ouch... it actually doesn't look that great, now the data is read but not used at all! I'm applying the fix in another PR

Ouch... it actually doesn't look that great, now the data is read but not used at all! I'm applying the fix in another PR


Ruby Group Capstone - Kanban Board

Hello,
Find the Kanban board Project Link for your review.
The tasks are assigned among the team members as follows

@cherelemma - Team member #1
@Hisoka37 - Team member #2
@danutibe07 - Team member #3

Hi @cherelemma, @danutibe07 and @Hisoka37 ,
Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉

Highlights

Well Prepared Kanbanbord ✔
Distributed in a fair way ✔
Descriptive Issue ✔

Optional suggestions
Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.

Hello @binhussen thank you for taking the time, review and approve our capstone project kanban board.


Update reduction count to 4K

Please keep both old and new value and at least major version when it was changed. Old value still is "valuable" 😀

@kvakvs Thank you for the review, how about introducing some format that can be used in everywhere in the document for similar situations? For example, Erlang doc has info for APIs since when it was introduced or modified like OTP R13B04.
Also, I wish we could introduce some variables to be referenced from multiple places in the document so that we don't have to modify multiple times.

Yes, let us continue the versioning question in your previous issue #77 and just add a note here.
Thanks for the contribution.

@happi @kvakvs Added link to the source and note. On my local, it looks like:

and

I also tried differently:

and

Please let me know which is better, or if there is any suggestion. I'll make a change accordingly.

I like the one with the “version info” title.

@happi Updated.

Thanks.
Remind me to do a new release soon. I should take the time and do some other fixes also.

Great, thank you too!


Documentation for RAPIDS-accelerated Hive UDFs

Adding documentation for how RAPIDS-accelerated UDFs work, tips for how to write one, and pointers to the examples.
Closes #1354

Marking this as a draft since the documentation links to assets that are provided by #1472 which is still pending.

Marking this as a draft since the documentation links to assets that are provided by #1472 which is still pending.

#1472 has merged, so this is almost ready to go.  It will probably need to me upmerged after @sameerz doc changes for the 0.3 release are done.

#1472 has merged, so this is almost ready to go.  It will probably need to me upmerged after @sameerz doc changes for the 0.3 release are done.

build

build


Release V1.5

CH-UI V1.5 Release Notes
I'am excited to announce the release of CH-UI v1.5, a major update that brings significant improvements and new features to enhance your experience. This release includes a full TypeScript refactor, a redesigned metrics page, a new documentation website, custom table management, SQL editor IntelliSense, an intuitive data explorer, and a fresh new design.
What's New
1. Full TypeScript Refactor

The entire application has been refactored using TypeScript, improving code quality, maintainability, and developer experience.

2. Enhanced Metrics Page

Completely redesigned metrics dashboard
New scopes: Overview, Queries, Storage, and more
Improved data visualization for better insights

3. New Documentation Website

Visit the new docs at ch-ui.caioricciuti.com!

4. Custom Table Management

Tables are now created and handled internally, removing dependency on 3rd party packages
Improved performance.

5. SQL Editor intellisense

Enhanced SQL editing experience with intellisense suggestions and syntax highlighting

6. Intuitive Data Explorer

Redesigned interface for easier navigation and data manipulation and exploration

7. Fresh New Design

Modern, clean UI overhaul for improved usability and aesthetics.

Get Started

GitHub Repository: https://github.com/caioricciuti/ch-ui
Documentation: ch-ui.caioricciuti.com
Blog: ch-ui.caioricciuti.com/blog

🚀🚀🚀🚀


lab extension should help avoid various obvious mistakes while adding tests

[x] Doesn't make sense to add tests to a non-code cell. (Maybe covered enough by "test class generation failure": ValueError: Cell 3 is not a code cell, but metadata contains test code!. Currently only in the logs, but that's a general issue - #154) #182
[x] What happens when changing cell type (e.g. from source to markdown - presumably shouldn't just permanently wipe out the cell's test, since someone might switch the type by mistake...)? #181
[x] What happens when clicking to a new cell without saving the tests (they are just lost?) #180
[x] Indicate if %cell is missing? (Covered by requiring deliberate skip, and by warning attached to any test with a skipped cell)
[x] When you click through cells while looking at the "CELLTESTS" box, you can't tell if you've actually written a test without expanding 'advanced tools' and looking at the 'cell metadata'. #183
[x] There's no distinction between the "CELLTESTS" box and the "CELLTESTS - RULES" box in terms of tests vs. lint. #184

Dealing with this first in nbcelltests, but after that, the lab extension should be improved.

nbcelltests itself now helps with these things underneath, but have opened issues for the (important!) work of making the frontend a bit less hostile.


enable pipeline to process data with < 3 time intervals

Fixed the issue up until esteemer. Pictoralist is not capable of generating images if there is a gap in time interval at the last 3 records.


fix: set default value for nodeAlign

[中文版模板 / Chinese template]
🤔 This is a ...

[ ] New feature
[x] Bug fix
[ ] TypeScript definition update
[ ] Bundle size optimization
[ ] Performance optimization
[ ] Enhancement feature
[ ] Refactoring
[ ] Update dependency
[ ] Code style optimization
[ ] Test Case
[ ] Branch merge
[ ] Release
[ ] Site / documentation update
[ ] Demo update
[ ] Workflow
[ ] Other (about what?)

🔗 Related issue link
close #343

🔗 Related PR link

🐞 Bugserver case id

64c8fd92053faa683476512f
💡 Background and solution

📝 Changelog

Language
Changelog

🇺🇸 English

🇨🇳 Chinese

☑ Self-Check before Merge
⚠ Please check all items below before requesting a reviewing. ⚠

[ ] Doc is updated/provided or not needed
[ ] Demo is updated/provided or not needed
[ ] TypeScript definition is updated/provided or not needed
[ ] Changelog is provided or not needed

🚀 Summary
copilot:summary
🔍 Walkthrough
copilot:walkthrough

加上 close #343

加上关闭#343


New type: subscription

Suggestion
I would suggest to add subscription type.
Value
When using Azure Event Grid, I can subscribe to Microsoft Graph API events.
To create a Partner Topic, I need to create a Graph API subscription. the notificationUrl property specifies Partner Topic name, resource group name in which the partner topic is created, region (location) and Azure subscription.
To be able to create a subscription and de facto Partner Topic via Bicep file, will simplify the configuration of Azure Event Grid. Once Partner Topic is created, I can continue with creating event subscriptions for Partner Topic.
Microsoft Graph API REST reference
https://learn.microsoft.com/en-us/graph/api/resources/subscription?view=graph-rest-1.0
https://learn.microsoft.com/en-us/azure/event-grid/subscribe-to-graph-api-events?tabs=http

Thanks for this submission.  We'll use to help prioritize this work with the Graph subscription API owners.
@MartinM85 Can you help us understand the value of this operation via Bicep a little more please...
I can see why, with Bicep, there would be one place where you could configure both the Graph events subscription and Azure Event Grid. Since Graph events subscriptions need regular "renewing", would you be looking to redeploy the Bicep file each time the Graph event subscription needs renewing (just with a different expirationDateTime)?

Hi @dkershaw10 When the subscription is created with a notificationUrl which defines Event Grid Partner Topic details (Azure subscription, resource group, topic name) then the Event Grid Partner Topic will raise the event microsoft.graph.subscriptionReauthorizationRequired before the subscription is expired.
It's expected that Event Grid Event Subscription (event handler) is created for microsoft.graph.subscriptionReauthorizationRequired event and handler (e.g. Azure Function) will send request to the Graph API to renew the subscription (just with a different expirationDateTime).

Sorry I'm coming back to this late @MartinM85 - so this request is just to support Bicep for the initial subscription creation?
I'm asking because typically folks redeploy Bicep template files on a regular basis - the subscription resource declaration in the Bicep file should represent what is deployed in infrastructure.  However, for this resource, expirationDateTime is a required property.  When redeploying, you would need to be careful to update this property, or you would either get an error or invalidate the subscription.
Are you sure this is still appropriate for your needs via Bicep? If so, I will put the ask in with the resource owner team.

@dkershaw10 said:

typically folks redeploy Bicep template files on a regular basis

agreed 👆...
but not with this 👇...

the subscription resource declaration in the Bicep file should represent what is deployed in infrastructure

Bicep (and ARM) doesn't represent what is deployed in infrastructure, rather it expresses what the desired state should be in infrastructure. IOW, it represents the desired state config (DSC).
For instance, a Bicep file that represents an Azure Function app should be - the DSC. The first time I run it, I expect it will create the missing resource. If I modify the Bicep file to add/remove environment variable settings, I expect Azure deployment to update the existing resource that's defined... and Azure deployment does exactly that.
So for a subscription, I expect I should be able to define it and it's created. If during my deployment, I include a GUID (for an existing subscription) and a new expiry date, the execution of that template will update the existing subscription with the new expiry.

So for a subscription, I expect I should be able to define it and it's created. If during my deployment, I include a GUID (for an existing subscription) and a new expiry date, the execution of that template will update the existing subscription with the new expiry.
I agree with this behavior.

Sorry - yes - desired state configuration.
My point was more that you would need to update the expiryDateTime each time the Bicep file is run (unless you have a variable that can calculate this based on the current date + some offset), otherwise the service is likely to error, because of an invalid expiry time.


Whitespace in page URL paths is now changed to dashes

Previously, Gollum::Page#url_path would change whitespace in the
page name portion of the path to dashes. However, whitespace
in the path itself was retained. This caused problems in URLs.
This was originally created as a bug in GitLab at https://gitlab.com/gitlab-org/gitlab-ce/issues/26831, for reference.
I hope I went about adding an example to the lotr.git repo correctly. Admittedly, I'm not a LOTR fan 😱  (sorry) so I had to ask a coworker to help me come up with a new item. He suggested 'Dol Guldur' and 'Sauron' - I hope this fits with the theme.
Thanks for your review.

Thanks for the PR! The test failure is because of a general problem with testing jruby on travis we're experiencing, so I will merge regardless. The only question I have is whether there is a scenario in which this might break existing wikis. If so, I'd like to merge into a different branch than master. Or was it simply impossible to have whitespace in the path until now?
And yes, your coworker did well. :)

The only question I have is whether there is a scenario in which this might break existing wikis.

Great question. I tested before and after in gollum proper and both work. gollum itself seems to honor both Dol%20Guldur and Dol-Guldur in the URL, though I'm not sure how that works. I tested the patch in GitLab and it works after the patch.
I don't think it will break anything, but I can't be 100% sure.

@dometto What do you think?

Since the tests pass, I'm willing to merge it into master, unless @bartkamphorst objects.

Actually, I prefer to merge this into the 5.x branch, just to be on the safe side. @dblessing Would you mind opening this PR against that branch?

Hmm, but now in the 5.x branch we're working on removing all whitespace->dash conversion. Shouldn't we just consider this a bugfix for versions < 5 and not merge it into 5.x?

I'm not in favor of doing so because we don't know whether it will break existing behavior.

In that case perhaps we should just close this? Sorry for the trouble @dblessing: the story is we want to get rid of the whitespace conversion because it leads to endless bugs like the one you caught, and serves no real purpose anymore (it is an artifact of using grit).


Problem: Hard to see the cursor in cell-editing

Hi, i cannot see input cursor while editing. Theme's problem?
I'm using v1.4.13 under Windows 11.

Sorry, fixed style in 1.0.1


googleapis_auth: add note about Flutter app usage

@kevmoo I'm not sure to understand this note. We've currently experimenting with this plugin and we're able to use it in a Flutter desktop App.
Are there reasons to not use it?

For flutter desktop it...might work. If possible, use the referenced package. If not, you can try this package. It just might be a bit more tricky!


Screen Ruler: mesaurement unvisible

Microsoft PowerToys version
0.64.1
Installation method
Other (please specify in "Steps to Reproduce")
Running as admin
None
Area(s) with issue?
Screen ruler
Steps to reproduce
Select one of the spacing utilities

✔ Expected Behavior
Visible mesaurement
❌ Actual Behavior
In some cases the mesaurement is not visible. When left clicking (normal mesaurement end) the clipboard contains the correct measured data.
Other Software
No response

@4amVim Hey, are you still interested in helping with this? If so, are you able to reproduce the issue reliably?

I am still interested. I tried and couldn't reproduce this issue.

It'd be great to have at least a semi-reliable way of reproducing this.

as we can't repro, i'm going to close this


confirm file is a file and not directory before attempting to read.

Issue
When including remote css ie.
http://fonts.googleapis.com/css?family=Open+Sans:400italic,600italic,400,600
Module would attempt to map and read /public/css as file.
This in turn would cause an error to be thrown particularly:
Errno::EISDIR: Is a directory - public/css (if public/css already existed)

Resolution
Resolved by confirming that File is a file and not a directory before reading file.
Only tested in Rails 3.2.8, ruby 1.9.3p551

@ss44 Thanks. Since File.file? also checks whether the file exists, you could remove the File.exist? check on that line. Can you do that so I can merge this?
Also, you might want to tell premailer-rails to ignore that <link> by adding the attribute data-premailer="ignore".

@fphilipe I've removed the redundant File.exists?, good catch.
Would adding that attribute not prevent premailer from doing it's magic? Think there maybe other edge cases where someone attempts to load an external css template file from a remote server with a similar name schema that may also trigger a similar error.

Would adding that attribute not prevent premailer from doing it's magic?

Yes, for that <link>.

Think there maybe other edge cases where someone attempts to load an external css template file from a remote server with a similar name schema that may also trigger a similar error.

Not sure I follow.

Would you mind combining the two commits and adding a changelog entry?

@fphilipe combine :)

Not sure I follow.

Please disregard my comments, I believe I just mis-interpreted your previous comments.

Thanks! Merged in 43594e1


Fixed #34455 -- language_from_path returning None in LocaleMiddleware

Fixed: https://code.djangoproject.com/ticket/34455
When using LocaleMiddleware, the value of language_from_path was coming out to be None initially, and because of that, the redirect was not working.

Please create patch against the main branch.

Thanks Mariusz, on it, I am new to open source so it's taking a little bit
of time.
On Wed, Apr 5, 2023 at 11:17 AM Mariusz Felisiak @.***>
wrote:

Please create patch against the main branch.
—
Reply to this email directly, view it on GitHub
https://github.com/django/django/pull/16727#issuecomment-1496951775, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AMB7FTLJAPKRHHYHVCWGO63W7UBQLANCNFSM6AAAAAAWTSD4HE
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

Can you please check this, should i send the pull request?
https://github.com/sinsniwal/django/tree/ticket_34455
On Wed, Apr 5, 2023 at 11:26 AM Mohit Singh @.***>
wrote:

Thanks Mariusz, on it, I am new to open source so it's taking a little bit
of time.
On Wed, Apr 5, 2023 at 11:17 AM Mariusz Felisiak @.***>
wrote:

Please create patch against the main branch.
—
Reply to this email directly, view it on GitHub
https://github.com/django/django/pull/16727#issuecomment-1496951775,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMB7FTLJAPKRHHYHVCWGO63W7UBQLANCNFSM6AAAAAAWTSD4HE
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

Looks good as a draft patch (release notes 4.2.1.txt and a regression test is missing). However, I don't believe it is the correct solution, see comment.

Also, get_language_from_request() is intended to return a language from requests, not from settings.

But the thing is for the first time if someone enters a URL like admin,
there is no language to get from there, than in those cases we can redirect
too, that's what i got , like once i open the admin using en/admin, after
that i was able to access admin directly without using en, by implementing
this code instead of None, everything seems to work.
Even i will try to look more deeper.
On Wed, Apr 5, 2023 at 11:47 AM Mariusz Felisiak @.***>
wrote:

Also, get_language_from_request() is intended to return a language from
requests, not from settings.
—
Reply to this email directly, view it on GitHub
https://github.com/django/django/pull/16727#issuecomment-1496974135, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AMB7FTJPA4RHJPQQNAAE4ITW7UE6NANCNFSM6AAAAAAWTSD4HE
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

Yes, Sir, You are correct, This issue is not even persistent, and now I am
unable to simulate the issue, the issue can be on edge cases, I was able to
simulate like an hour ago and now its working fine without any changes
On Wed, Apr 5, 2023 at 11:49 AM Mohit Singh @.***>
wrote:

But the thing is for the first time if someone enters a URL like admin,
there is no language to get from there, than in those cases we can redirect
too, that's what i got , like once i open the admin using en/admin, after
that i was able to access admin directly without using en, by implementing
this code instead of None, everything seems to work.
Even i will try to look more deeper.
On Wed, Apr 5, 2023 at 11:47 AM Mariusz Felisiak @.***>
wrote:

Also, get_language_from_request() is intended to return a language from
requests, not from settings.
—
Reply to this email directly, view it on GitHub
https://github.com/django/django/pull/16727#issuecomment-1496974135,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMB7FTJPA4RHJPQQNAAE4ITW7UE6NANCNFSM6AAAAAAWTSD4HE
.
You are receiving this because you modified the open/close state.Message
ID: @.***>


Sprint 6: Release 3 Summary

As a client, I want to be able to track the progress of the project.

[ ]  Write a summary of the third release for the client.
[ ] Mockups for QR Scanner.

[Story Points: 3]
[Risk: Low]
[Priority: High]

@ggebran @DanielPri I've placed release summary in the Google Drive folder for ease of access.


NEP-3 Ammendment

I believe NeoContract ABI (https://github.com/neo-project/proposals/blob/master/nep-3.mediawiki) may have useful ammendments:

Officially support "Map" types, as proposed by Ricardo some time ago
Support "Any" type for return and parameters. This was unofficially supported on C# compiler, and further changed to ByteArray, but it would be really better to allow returning and receiving any type of stack item (BigInteger, ByteArray, Map, etc) officially in some dapp functions
Few wording changes, such as explaining Signature format as a 64-byte signature regarding Neo supported ECDSA standard (secp256r1), or even keeping this open to future possibilities on signature formats (with even different sizes if needed to)

Good suggestion.

https://github.com/neo-project/neo/issues/504
Ricardo, please see this. @lock9

Another important feature is a Witness type on NEP-3. I'll explain the details.
It will work as the Address type (same as a 20-length byte[]), however with the requirement that this is a verified address (by means of the witness system). This strongly simplifies logic for users, and make code safer.
Using this, someone could write this code on NEP-5 (general language):
transfer(Witness from, Address to, Integer value)
This means that parameter from is already validated by witness system (no explicit Runtime.CheckWitness will be needed on code).
Devpacks could adapt to it, for example, on C# we could provide a type conversion from Address type (or byte[]) to a Witness object. This function basically calls Runtime.CheckWitness over the parameter, verifies if it's correct, and return a Witness (same as a byte[], but validated from code perspective).
On code, users could do (C# style):

object Main(string op, object[] args) {
...
    if (op == "transfer") {
        Witness from = ((byte[])args[0]).ToWitness(); // fails if it's not a verified witness
        Address to = ((byte[])args[1]).ToAddress(); // fails if size is not 20
        BigInteger value = ((BigInteger)args[2]).AsBigInteger();
        return Transfer(from, to, value); // we could do in a single line too
    }
...
}

If this was standard since the beggining, we could have also avoided some validation problems on smart contracts.
What do you think @lightszero @erikzhang @shargon ?

See: https://github.com/neo-project/neo/issues/906

Another important addition here would be the Witness type.

@igormcoelho  what if you divide this into 2 issues? it will be easier to implement, review and test.
Regarding the initial proposition, I think we should support JSON. Someone mentioned some json stuff these days, I just don't remember where. It was in the persistence layer I think 🤔
No Json = No Happy developers

Ok, so we have many possibilities for ammendments here hahaha
JSON is interop-layer element... but since NEP-3 covers all Neo, I don't see any problem in that.
Thinking from this side, we would need too for enumerators (NFT), perhaps contracts, and all available (and useful) interop types. Let's think on how to proceed, without dumping all at once :joy:
For Witness, I think it's absolutely necessary, in order to simplify wallet operations.

@igormcoelho  can you make a list?
We can have other issues to add other features, I'm afraid we get stuck if we don't reduce the scope of this amendment, maybe "nep-3 amendment" is too broad? 😅
The smaller you make this issue, the best chances we have it is implemented in a short time frame.


Pulling fork

https://github.com/Recycleye/labelbox-python/commits/develop/labelbox/data/serialization/coco/instance_dataset.py
@Nicholas-Kastanos and @f-silverio  can  you check your commits from above are right in the instance dataset file then we are ok to merge I think

Only conflict (I think) was requirements.txt


Master is failing

Master is failing in hash-de-prueba-45.

More info at: https://travis-ci.org/delapuente/qiskit-core/builds/job-de-prueba


onProxyClosed not always received in app multiplexing scenarios

When a transport error occurs (such as bt disconnect), an onProxyClosed notification should be sent to each multiplexed app.  Sometimes a concurrent modification exception occurs in the InternalMsgDispatcher class that prevents the onTransportError method from being called.  As a result, the onProxyClosed method is never called and the app never receives the notification.  The listenerlist Vector in SdlConnection needs to be changed to a list that is thread safe.  The logic in the onTransportError for loop also need to be revised.

Is this for #151 feature/multiplexing?

Hey Joey,
The problem affects the internal multiplexing functionality on the master and develop branches (that is already implemented), its not specific to the feature changes in #151.

Ok, just making sure. Thanks!

Merged.


Add serviceaccount match under feature flag.

Description
Add support for service account matches and also add a feature flag. SA match will be under feature flag.
This is related to PR https://github.com/projectcalico/libcalico-go/pull/693
Todos

[ ] Tests
[ ] Documentation
[ ] Release note

Release Note

None required

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


No Ports shown on Linux dotnet core 2.0

I'm using dotnet core 2.0 (preview 3 at the time of writing) on

x64 Debian Stretch Linux
armhf "selfmade" buildroot 2017-3 linux

I followed your instructions on compiling the serialunix library (I don't say libnserial to avoid confusion with the very common libserial library) and installed it on both target systems. Finally, setting the $LD_LIBRARY_PATH accordingly allows the following simple code to run without a DllNotFoundException.
            string[] portnames = SerialPortStream.GetPortNames();
            Console.WriteLine("GetPortNames returned {0} items", portnames.Length);
            foreach (string name in portnames)
            {
                System.Console.WriteLine(" > port[{0}]",  name);
            }

However, on both machines the result is empty:
nzain:~/git/demoapp$ dotnet run
GetPortNames returned 0 items

Any ideas? How can I debug this problem?

Running the same code on a Win 10:
GetPortNames returned 2 items
 > port[COM3]
 > port[COM1]

This is not the same computer (at work now) and it is dotnet core 2.0.0-preview1-002111-00 (and of course a different native serial implementation).
I just found your wiki entry on "providing debug infos" - sorry for being late on this. I'll provide every info this evening from my private PC.

Thanks for the report. Windows and Linux have two completely different implementations. Running on your Linux box uses the nserial library and is coded in the "portlinux.c" file. As described in the header there:
// We look in the path '/sys/class/tty/*' for a list of all TTY devices. Each
// TTY will be considered a real device if it contains a softlink
// 'device/driver' which is a symlink to the directory for the driver for the
// TTY. That then excludes a lot of the PTY (Pseudo TTYs).
//
// Then for each device, we look at the file 'dev' to get the major and minor
// device node. Once we have a list of all the device nodes, we then do a
// recursive search through '/dev' looking for any entries with those device
// nodes. If the device is of type 'platform:serial8250' which is readable
// by '/sys/class/tty/*/device/modalias', we will also open the device and
// check that it is a real serial port or not.

Probably the simplest explanation why you get zero devices is you do not have access (on a standard Linux, users are not in the dialup group). That is checked using the "access" system call.
// Check user has permissions before adding
        if (access(path, R_OK | W_OK)) {
          nslog(handle, NSLOG_WARNING, "getports: file not accessible: %s (errno=%d)", path, errno);
          continue;
        }

If you want to further debug, you can install google test. You can modify the build script to provide debugging of the native library itself (it's already there, but commented out)
CFLAGS="-O0 -g -Wall" cmake -DCMAKE_BUILD_TYPE=Debug -DNSLOG_ENABLED=ON .. && make

Then the library will write to /tmp/nserial.log.

You can also install the google test framework and build, which you can test the cases on your PC. For most test cases, you'll need two serial ports with a NULL modem cable.

Thanks for the detailed description! Your assumption was correct, insufficient permissions (didn't know that only root can access those). Running the app as root indeed shows my connected ftdi on /dev/ttyUSB0:
$ dotnet ./demoapp.dll
GetPortNames returned 2 items
 > port[/dev/ttyAMA0]
 > port[/dev/ttyUSB0]

The next days will show, if this library works on my arm cortex-A7.

You should add your user to the correct group rather than running as root. Cheers!

Agreed, but since I'm kind of experimenting for an embedded linux based on buildroot, there won't be any users :) In particular, there is no extra user for serial ports, they are owned by root:root.
# ls -l /dev/ttyUSB0
crw-------    1 root     root      188,   0 Jan  1 00:00 /dev/ttyUSB0


Guvnor 2309, 2310, 2315

Contains 3 commits
View/Layout changes only
GUVNOR-2309: Comments section in overview uses inefficient layout
GUVNOR-2310: When commenting on an unlocked asset, the first (few) char(s) of the comment are removed
GUVNOR-2315: Improved fix so the users current asset type collapse state stays in place in case the project explorer gets refreshed (i.e. new file gets added or lock state changes)

@manstis as discussed, can you take a look, please?

+1 to merge (we could arguably have a test for the collapses change; but IDK if GwtMockito will play well with the View implementation)

@manstis that's true, but for your original fix as well ;). If you want to try and write a test for it we can hold back the fix for GUVNOR-2315. I just added it here as I shouldn't have merged the original one without trying it out first and now want to fix it ;). It's somewhat on the edge for requiring a test...but we can try.

Happy for you to merge "as is"

@manstis tbh I am not sure if collapsed by default is better than expanded thinking about it. Wdyt?

See the JIRA; I think collapsed by default is better.. especially
considering projects larger than our silly examples.

+1 to merge.

4f5ee268ea68f8bb6643b5bb9917c09a627fb844 62880b27307e07dabb20c682e16fdd33d885f15a 4f5ee268ea68f8bb6643b5bb9917c09a627fb844


Using of Signal Strength should improve location accuracy

You can add Signal Strength (getCellSignalStrength) in the weight to improve location calculation. As stronger signal as closer to the tower.

This code could be just copied from WIGLE, but from my experience it's very inaccurate, sometimes it located me a few kilometres away from actual location.

Do you have a phone that supports either the getNeighboringCellInfo() or getAllCellInfo() calls in TelephonyManager? The one phone I have to work on this returns null and/or empty depending on which call I try. The result is that I can only test with the one tower that is returned by getCellLocation() so I can't adequately test any type of averaging.
If you have a phone that supports that and are willing to help, I can code up an signal strength weighted average location estimate for you to test.

How can I check this? I've found in my phone's EngineeringMode menu on NetworkInfo_Info page information about Neighboring Cells that consists of: neigh_cell_count (and then for each cell) cellid, arfcn, psc, qQualmin, qRxlevmin, srxlev, squal, rscp & ec_no. Will this be helpful?

Sounds like it. I'll work up the changes needed to use received signal strength in the weighted averaging. It may take a few days for me to get too it. By the way, what version of Android are you running?

4.2.1, but I can engage a friend of mine in testing as well. He use 5.0 or 5.1

I'm interested in this too. Could I give any help? I'm running on 5.1.1.

+1

I checked this and it's harder than you think. There are dozens of variables that add noise to estimating location, let's list a few of the most important:

devices incorrectly implement and provide signal strength
BTS has latitude, longitude and... altitude
I took some measures and even non-moving device gets signal strength varying by 5-14dB very n-seconds
weather conditions don't help
different signal strength is used for 2G, 3G and 4G networks, we would have to relay on correct implementation for each of them for all devices, tested it on 6 devices, 4 different Samsung models and 2 LGs, each had different problem

if you know how to overcome those problems... write scientific paper about it.

start with using neighbouring cellinfo, and if you have more than 1 celltower just use the geometric middle between all tower locations.
that would be a prgmatic thing to start with?
i guess more accurate than just 1 tower location

4 devices I tested don't provide at least one of:

signal for neighbouring cells
CID
LAC
technology
MCC/MNC

Do you want to test my app that I was working on?

The backend already attempts to use as many towers as it can get information for using the TelephonyManager.getAllCellInfo() and TelephonyManager.getNeighboringCellInfo() method calls. And it does a geometric mean (inversely weighted on reported tower coverage range) on the towers it can get information for to estimate position.
The issue is that many phones return null on the TelephonyManager.getAllCellInfo() and TelephonyManager.getNeighboringCellInfo() method calls. On those phones the only information you have is for the one tower you are connected to. No matter how you calculate it, your position will be the tower location with your uncertainty the reported tower range.
And, as of a year or so ago, Samsung phones are in the group that only report the one tower you phone is connected to. So for a major portion of the user base the current code does as well as is possible.


add https server functionality

for regular nodeos we have the https-server-address parameter.  Can you please add in https functionality for the wasm ql server wql-listen

There are 2 common camps:

All servers should support https
It's a maintenance nightmare keeping up with mitigations for the never-ending stream of new https attacks. Leave it to the experts who maintain reverse proxies.

I'm in the 2nd camp. nginx is a good choice.

Makes sense.  I will use the second approach. Thank you


update dependecy intl 0.18.0

Hi,
Could you please update to intl 0.18.0 dependecy?
thanks

Hello @robertoltrocha !
Thank you so much for adding issue!
I just published it with intl 0.18.0.
Please use v2.1.9.


[SPARK-48475][PYTHON] Optimize _get_jvm_function in PySpark.

What changes were proposed in this pull request?
It is a performance optimization for PySpark. For the context, sc._jvm is a JVMView object in Py4J. It has an overloaded __getattr__ implementation (source). Accessing .functions internally sends a command to the Py4J server. The server then searches through a list of imports to find a package that contains the functions class (source), and will eventually find org.apache.spark.sql.functions. The failed reflection attempts are much more expensive than the last successful reflection. Instead, we can directly use this fully qualified class name and prevent all failed reflection attempts.
Why are the changes needed?
It improves the performance in PySpark when building large DataFrame.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Existing tests.
The following code can verify the performance improvement:
import pyspark.sql.functions as F
from datetime import datetime

for i in range(5):
  T = datetime.now()
  df = spark.range(0, 10).agg(F.array([F.sum(F.col("id")) for i in range(0, 500)]))
  print(datetime.now() - T)

On local PySpark shell, the time consumption before/after the optimization is about 1s/0.5s.
Was this patch authored or co-authored using generative AI tooling?
No.

@cloud-fan @HyukjinKwon Could you help review? Thanks!

Nice finding!

It improves the performance in PySpark when building large DataFrame.

I’m curious why “building large DataFrame”? I thought the improvement would be maximized by calling the built-in functions more frequently

@xinrong-meng This is correct. I mean it is common to have a lot of built-in function calls when building a large DataFrame (as the one in the PR description).

Merged to master.


Can't open any files with serveFile

Whenever I call Pistache::Http::serveFile, the server throws a runtime error. It doesn't work with relative paths or full paths. For example, I'm trying to load /mnt/c/Users/Aidan/Code/UYP/Server/cmake-build-debug/site/favicon.ico, which loads perfectly fine in a filestream, but not Pistache. The call looks like this: Pistache::Http::serveFile(writer, file.c_str());. I'm sure this is a problem on my end, but what am I doing wrong?

@dennisjenkins75 It doesn't show up.

Please be more specific.  "What" does not show up?  I'm guessing that you do not have the "strace" command installed on your Linux distro.  Find the correct package (you'll need to search) and install it.
https://en.wikipedia.org/wiki/Strace
$ which strace
/usr/bin/strace
On my Gentoo Linux dev box, strace is in the package "dev-util/strace".  No idea about other Linux distros.

I ran strace, the file never showed up in it.
Also, switched Pistache to a submodule so I could step through the code, it looks like this is the problem:

Can't tell why it's happening though, it's hard to follow this async stuff.

Ah, sorry, i forgot.  "strace" does not automatically follow threads.  Try with "strace -ff -o /tmp/output.log my_binary" (or whatever).  Read the man page :)

Odd.  I don't know.  Personally, I am not using "serveFile", so I have no direct experience with it.  Can you use gdb to determine if the value of "fileName" is being changed before FileBuffer::init() is called, and/or if the RAM that it points to is getting over-written?
Its late here, so I'm going to bed soon.
Can you reproduce this problem in the pistache unit-test framework?  If so, please create a PR so that I have something to pull down to my machine and test with.

@dennisjenkins75 Late here too, but I'll try to do that unit test.

@dennisjenkins75 I couldn't figure out the unit test framework, but I made this: https://github.com/Aidoboy/PistacheServeTest

Run your original program (before you hacked it up w/ strdup()) under valgrind.

@dennisjenkins75 https://gist.github.com/Aidoboy/da7035e4672d2ca5f2a26bb89948a73a

@dennisjenkins75 @Aidoboy I have fixed this problem in PR #349

I guess this is resolved then! Thank you guys so much!


Migration to Weblate

this is a follow-up of: https://pagure.io/gssntlmssp/issue/15
I migrated your project, you have a first PR here: https://github.com/gssapi/gss-ntlmssp/pull/38
you can keep it open as long as you wish, Weblate will reuse it and squash commits per author to limit the number of commits
Weblate will update all po files each time you change the pot file.
Weblate will update your LINGUAS file to keep your like easier.
please connect to https://translate.fedoraproject.org/ so I can make you admin
kind advice:

please rewrite your readme in Pagure so we don't loose time (yes, you wrote you migrated to github, but I missed it, other will loose time too)
if you choose not to forward issues to github, please close all Pagure issues, at least we would be informed of the transfer
as a translator, have no idea what your project does, having example on where it is used would help me

please rewrite your readme in Pagure so we don't loose time (yes, you wrote you migrated to github, but I missed it, other will loose time too)

Can you give suggestions here?  It says "This project has been moved to github" in big letters at the very top, so it's not clear to me what you're for.

@Jibec I changed the README and removed all content but the notice that the project moved, hope that works better.
I also temporarily re-enabled the issue tracker and closed all the remaining issues with a message that the project moved.
HTH

I also logged on weblate, is that sufficient to "connect" ?

@Jibec also note that we changed the license, is that something that I can change in weblate once you make me admin ?

@Jibec, one more question, weblate keeps rebasing the PR while is open, is there any signal that will tell me when the translator thinks it is "done" ?

I also logged on weblate, is that sufficient to "connect" ?

yes, you are now admin

@Jibec also note that we changed the license, is that something that I can change in weblate once you make me admin ?

as long as it still it opensource, yes, no problem

@Jibec, one more question, weblate keeps rebasing the PR while is open, is there any signal that will tell me when the translator thinks it is "done" ?

No, why?
When you notice a translation is complete, you are welcome to release.
When you want to release for whatever reason, you merge what is ready.
The best is either to inform translators about an incoming release or make frequent release. But that's easy to say.

@Jibec sorry but I do not see a way to change the translation license ...

Found it, nvm


fix: Return the external ALB IP correctly

What type of PR is this? (check all applicable)
Specify one of the following as the first element in the title:

build: Changes that affect the build system or external dependencies (example scopes: gulp, broccoli, npm)
ci: Changes to our CI configuration files and scripts (examples: CircleCi, SauceLabs)
docs: Documentation only changes
feat: A new feature
fix: A bug fix
perf: A code change that improves performance
refactor: A code change that neither fixes a bug nor adds a feature
test: Adding missing tests or correcting existing tests

The NOTABLE CHANGES that will create a new entry in CHANGELOG.md and
create a new tag are these types:

feat
fix

Description

This PR is included in version 1.0.2 :tada:


How to link to 'SDL2main.lib' in Visual Studio 2019

Hi,
I installed sdl2 by using vcpkg, and I created a new project in Visual Studio 2019.
I added the statement:
#pragma comment(lib, "SDL2main.lib")
However, when building, there was still Error	LNK1104	cannot open file 'SDL2main.lib'
It seems that SDL2main.lib was moved to manual-link folder, so how to link to 'SDL2main.lib' in Visual Studio 2019?
Reference: https://github.com/Microsoft/vcpkg/commit/3d8ba3e775c13816f4c298ba16b8094e2affc4dd

Try manual-link\SDL2main.lib ?


download-product should output assign stemcell artifact when checking if already uploaded

@jghiloni brought this to our attention and was going to investigate. Any update @jghiloni ?

@jghiloni brought this to our attention and was going to investigate. Any update @jghiloni ?

Not yet, I haven't had time. If you don't have time to investigate, I can.

Not yet, I haven't had time. If you don't have time to investigate, I can.


Appstudio update test-component-pac-izeh

Pipelines as Code configuration proposal
To start the PipelineRun, add a new comment with content /ok-to-test
For more detailed information about running a PipelineRun, please refer to Pipelines as Code documentation Running the PipelineRun
To customize the proposed PipelineRuns after merge, please refer to Build Pipeline customization

Pipelines as Code CI/test-component-pac-izeh-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
7 seconds
init

✅ Succeeded
18 seconds
clone-repository

✅ Succeeded
1 minute
build-container

✅ Succeeded
17 seconds
deprecated-base-image-check

✅ Succeeded
17 seconds
inspect-image

✅ Succeeded
44 seconds
clamav-scan

✅ Succeeded
10 seconds
clair-scan

✅ Succeeded
11 seconds
sbom-json-check

✅ Succeeded
17 seconds
label-check

✅ Succeeded
10 seconds
show-summary

✅ Succeeded
8 seconds
show-sbom


Use file:// urls for path and directory dependencies

fixes https://github.com/python-poetry/poetry/issues/5273

I wonder if this will break some "poetry-only" use cases where someone has a couple of projects relative to each other and everything works fine when the entire folder structure is moved.
Nevertheless, I think this fix is necessary because we are creating invalid PEP 508 names at the moment.

@radoering Could you please make a new release.
Really keen on using this change.

We are working on a new release (and it's linked here); as this will have to be coordinated with a Poetry release (and you, as a user of the poetry frontend won't benefit until it's shipped by Poetry), you can follow progress there.


Feature/qingke v4 delay

To close https://github.com/ch32-rs/ch32-hal/issues/27 I have looked at the QingKeV4 Microprocessor Manual and the register names are the same for the QingKeV2 and QingKeV4 except than on QingKeV2 the Systick timer is 32-bit and on the QingKeV4 it is a 64-bit timer.
But given the implementation, we reset the timer to 0 for each delay and delay_ns() takes a u32 so I had the same implementation for both processors, hence the use of the same file.
I have also included a basic example without embassy. I have renamed the blinky using embassy embassy_blinky to use the embassy_ prefix as done in esp-hal examples. Let me know if you are ok with this. I can rename the other examples for more consistency.
Note that according to the issue https://github.com/ch32-rs/ch32-hal/issues/6 I had to make these changes to run the examples:
diff --git a/Cargo.toml b/Cargo.toml
index b7750e4..24752da 100644
--- a/Cargo.toml
+++ b/Cargo.toml
@@ -37,10 +37,10 @@ defmt = { version = "0.3.5", optional = true }
 embassy-sync = { version = "0.6.0", optional = true }
 embassy-futures = { version = "0.1.1", optional = true }
 embassy-time-driver = { version = "0.1.0", features = [
-    "tick-hz-1_000_000",
+    "tick-hz-10_000",
 ], optional = true }
 embassy-time = { version = "0.3.0", features = [
-    "tick-hz-1_000_000",
+    "tick-hz-10_000",
 ], optional = true }
 embassy-usb-driver = "0.1.0"

diff --git a/examples/ch32v307/Cargo.toml b/examples/ch32v307/Cargo.toml
index eda3ed0..c1008f6 100644
--- a/examples/ch32v307/Cargo.toml
+++ b/examples/ch32v307/Cargo.toml
@@ -4,7 +4,7 @@ version = "0.1.0"
 edition = "2021"

 [dependencies]
-ch32-hal = { path = "../../", features = ["ch32v307vct6", "embassy", "rt"] }
+ch32-hal = { path = "../../", features = ["ch32v305rbt6", "embassy", "rt"] }
 embassy-executor = { version = "0.5.0", features = [
     "nightly",
     "integrated-timers",

CI build is fixed by rebase.


Allow certificates and keys to be read from memory

An SSL server currently can only set the certificate and key from files.
It should also be able to read them from memory.

Change example_https_server.cpp to use certificate and private key from memory.


ESMPy for re-gridding?

For re-gridding, ESMPy might be an alternative to css going forward, since I can only use css with a UV-CDAT install (see #7). This AMS talk wasn't particularly useful, but their website looks ok. The problem is that the installation is from source code (i.e. it's not on binstar)

The documentation for ESMPy is here and they now have a page on anaconda.org. Apparently the install works on linux and the moment but not mac.

Another option would be to take the code for cssgrid from here at the uvcdat repo and use it to write my own Python package.

It turns out that because I'm only using the values near the equator (i.e. where a degree of latitude and longitude is the same distance), I don't need to perform the regridding on a sphere. Instead, I can just use scipy.interpolate.griddata.
For future reference, pyresample might also be an option for this type of regridding.


Nullable attributes for MinBy/MaxBy

It's a little unclear from the documentation what should be returned by MinBy and MaxBy when given an enumerable of null values. The regular LINQ Min and Max explicitly state that they return null in this case, which is very useful because we can use it with DefaultIfEmpty.
Should the documentation be more explicit? And if null is a valid return value that should probably be annotated with [return: MaybeNull].

Hi @sid-6581,
Interesting question... To be honest, I never really thought about this.
I did a quick test in LinqPad
static void Main()
{
	var foos = new[]
	{
		new Foo("hello"),
		new Foo(null),
		new Foo("world"),
	};

	foos.MinBy(f => f.Name).Name.Dump();
	foos.Select(f => f.Name).Min().Dump();
}

class Foo
{
	public Foo(string name)
	{
		Name = name;
	}

	public string Name { get; }
}

This dumps null, then "hello". So the behavior of MinBy isn't consistent with Linq's Min... I guess this is a bug. MinBy should ignore null values, unless there are only null values.
I'm not sure about the [return: MaybeNull] attribute, though... If T is not nullable, the sequence shouldn't contain nulls, and the method would never return null. And if T is nullable, then the attribute is redundant, since we already know that it could be null.
I'll clarify the documentation, though.
Thanks for reporting this!

I'm not sure about the [return: MaybeNull] attribute, though... If T is not nullable, the sequence shouldn't contain nulls, and the method would never return null. And if T is nullable, then the attribute is redundant, since we already know that it could be null.

Actually, I see that Linq's Min has the attribute. Not sure why it's necessary, but I guess they put it there for a reason, so I'll do the same.

Wow. I just realized the behavior of Linq's Min is different for value types and reference types...
https://docs.microsoft.com/en-us/dotnet/api/system.linq.enumerable.min?view=netcore-3.1#System_Linq_Enumerable_Min__1_System_Collections_Generic_IEnumerable___0__

If TSource is a reference type and the source sequence is empty or contains only values that are null, this function returns null.

My implementation always throws if the sequence is empty. I guess I have a few things to review...

Thanks for looking into it so quickly! As for T not being nullable, it will by default allow nullable unless you restrict it with the notnull constraint.

As for T not being nullable, it will by default allow nullable unless you restrict it with the notnull constraint.

Yes, but when you actually call the method with a given T, you know whether T is nullable or not (and the compiler knows it too). In a nullable context, Min<string> or Min<int> could never return null, but Min<string?> could.
Anyway, I'll add the attribute to remove any ambiguities. I should probably add it to some other methods as well... I'll take my guidance from Linq's code, I guess.

Now that I am digging into it a little deeper, it seems like LINQ is missing some annotations as well. DefaultIfEmpty isn't annotated, so it won't transform a T into a T?. So var s = (new string[] { }).DefaultIfEmpty() looks to be string, not string?.   I'll report that and see what they say about that.

@sid-6581 I think I fixed it. I'll make a pass on missing nullability attributes before I make a new release

OK, I don't think there are other [return: MaybeNull] attributes missing. The only candidates were the *OrDefault methods, but the default value is passed explicitly, so the compiler would warn if it were null for a non-nullable TSource anyway.

Technically, this is a breaking change, since it changes the behavior, and could produce warnings that weren't there before.
I should probably release this as a major version. I'll release a beta, could you test it and check that it works as you expect?

Yes, I can definitely do that! Thanks for getting that done so quickly!

Here it is:
https://github.com/thomaslevesque/Linq.Extras/releases/tag/4.0.0-beta.1
https://www.nuget.org/packages/Linq.Extras/4.0.0-beta.1

Sorry it took a while to get to it, I was going down the rabbit hole of cleaning a lot of other nullable stuff in my solution. I tried the beta today and that took care of it. Thanks!

No worries, there's no rush!
Thanks for testing, I'll publish 4.0.0 as stable

It looks like I spoke too soon. In ExtremeBy you are checking if the default for TKey is null, but if you use DefaultIfEmpty in front of MinBy or MaxBy what will happen is you'll get a collection with one element, where TSource is null.
If I remove the DefaultIfEmpty I would also expect a null return, but for the same reason as above it triggers an EmptySequenceException.

It looks like I spoke too soon. In ExtremeBy you are checking if the default for TKey is null, but if you use DefaultIfEmpty in front of MinBy or MaxBy what will happen is you'll get a collection with one element, where TSource is null.

You mean something like that?
items.DefaultIfEmpty().MinBy(item => item.Value)

I made a few tests, and the results are consistent with Linq's Min(Func<TSource, TResult>).
If the type of the items is a reference type, it would throw a NullReferenceException when accessing .Value, but I'd say it's the caller's fault for not taking that into account...
If it's a value type and Value is a reference type, it would return null.
If it's a value type and Value is also a value type, it would return the default value for the type of Value.

If I remove the DefaultIfEmpty I would also expect a null return, but for the same reason as above it triggers an EmptySequenceException.

Again, it's the same behavior as Linq

Here's a Gist that shows that it's consistent with Linq:
https://gist.github.com/thomaslevesque/8e0cc655710e8632a59e530348878f78

You're right, it does appear to be consistent. I totally forgot what I wrote above, that DefaultIfEmpty() can't correctly be annotated for nullability with the current annotations. It didn't convert my TSource to TSource?, so I didn't get a nullability warning on my direct item property access. If I do item?.Value it works as expected.
If I do the clunky workaround of writing DefaultIfEmpty((TSource?)null).MinBy(item => item.Value) I get the compiler warning I was expecting when referencing item.Value improperly, but that's quite ugly and hard to remember. It's just one of those limitations we have to work around currently, I suppose.
Apologies for that!

No problem. Actually I had misunderstood your comment, I thought you were talking about the behavior, not the nullability annotations


TRG 1.02 appropriate install instructions either INSTALL.md or in README.md

https://github.com/eclipse-tractusx/vas-country-risk/blob/2d4df2e235be7112450c097c44aaa63dc083007f/INSTALL.md?plain=1#L13-L16

same folder problem like in https://github.com/eclipse-tractusx/vas-country-risk/issues/89  in Install.md

https://github.com/eclipse-tractusx/vas-country-risk/blob/2d4df2e235be7112450c097c44aaa63dc083007f/INSTALL.md?plain=1#L20C1-L21C29

suggestion:
Do the following command to install the application:

npm install @emotion/react  @emotion/styled<EMAIL_ADDRESS><EMAIL_ADDRESS>react-simple-maps cx-portal-shared-components react-scripts

s

was updated


Geolocator's distance does not return value or return always the same one

I use geolocator package for device coordinates tracking. I need to send current device coordinates to server each specified interval if distance is more than specified value. I noticed that if app is in background then getDistanceBetween returns almost the same value (I think this is because the current position is not changed), if device is off then it looks like even geoPositionStream stops generate events. So it looks like geo tracking is working only if app is in foreground but I need to track coordinates in both foreground and background modes.
Could you please to explain the current behaviour? What I had did wrong and how I can achieve what I need?

Hi @bambinoua,
At the moment the geolocator indeed only support fetching the location in the foreground. Issue #53 discusses background support but hasn't been implemented yet.
Since this is a duplicate of issue #53 I will close this issue and keep tracking background support there.


Swift and Xcode 7 beta 3 compatibility fix

fixing:
Async.swift:104:18: Redundant conformance of 'Async<Value>' to protocol 'MutableAsyncType'

and:
Async.swift:106:18: Redundant conformance of 'Async<Value>' to protocol 'CustomStringConvertible'

Async.swift:106:43: Redundant conformance of 'Async<Value>' to protocol 'CustomDebugStringConvertible'

Thanks for the PR!
These errors puzzeld me and also only appeared for the iOS target, not the Mac one.
I'd like to keep the fact that Async (and Future) are mutable. Users of BF should use a Promise. In your fix, this is not the case. I'm hoping there's a different solution. (On the swiftier-2.0 branch, where I made a first attempt at implementing the same kind of basic async object, I did not run into this error)
Do you agree about keeping mutability 'internal'?

I re-added the Async.swift file to the iOS target and that seems to remove the warnings for me https://github.com/Thomvis/BrightFutures/commit/e8f6d7b20ecd42a128f8ae6bc0a0b4d3b5551853.

nice, thanks, will close this PR now


Reckon with cancelling animations

It introduces too many bugs to make it useful, maybe.

Well, now we need it, so gotta figure something else out.

I think I've smoothed out all the edges with this.


fix(regexp-assemble): Fix types

Fixing types for regex assembler.

Started from scratch with a new virtualenv and was able to reproduce original problems. This PR seems to fix them and make it work again.

@theseion We need tests for these utils. And they need to be executed in the pipeline so regexp-assembly keeps working.


Consider if any cops should be renamed

Now that the next major release draws closer, I think it’s time to consider whether all of our cops are properly named.
Previous to their v1.0.0, RuboCop did a massive cop rename (see rubocop/rubocop#6983, rubocop/rubocop#7077, rubocop/rubocop#7468) and I wonder if we ever truly went over our cop names. Just one example:

RSpec/DuplicatedMetadata
RSpec/EmptyMetadata
RSpec/MetadataStyle
RSpec/SortMetadata

These four cops all check metadata, but three are named FooMetadata and one is MetadataFoo. Should one or three (or zero) be renamed?

cc @rubocop/rubocop-rspec

Gotcha, you are correct that renaming might cause more pain than joy among our users. But perhaps we should copy that section from development.adoc into our repository and think a little more about cop names before adding them? 😅

That makes sense. I also commit to extracting the guideline part in hope to merge it


In which case will this occur?

I add a module writed by myself in OpenNMT, and it reported this, maybe there is something wrong with l:storage()?
/usr/local/bin/luajit: /usr/local/share/lua/5.1/onmt/utils/MemoryOptimizer.lua:31: Torch object, table, thread, cdata or function expected
stack traceback:
[C]: in function 'pointer'
/usr/local/share/lua/5.1/onmt/utils/MemoryOptimizer.lua:31: in function 'tensorIncluded'
/usr/local/share/lua/5.1/onmt/utils/MemoryOptimizer.lua:34: in function 'tensorIncluded'
/usr/local/share/lua/5.1/onmt/utils/MemoryOptimizer.lua:46: in function 'canShare'
/usr/local/share/lua/5.1/onmt/utils/MemoryOptimizer.lua:183: in function 'callback'
/usr/local/share/lua/5.1/nn/Module.lua:344: in function 'apply'
/usr/local/share/lua/5.1/nn/Module.lua:348: in function 'apply'
/usr/local/share/lua/5.1/nn/Module.lua:348: in function 'apply'
/usr/local/share/lua/5.1/nn/Module.lua:348: in function 'apply'
/usr/local/share/lua/5.1/nn/Module.lua:348: in function 'apply'
/usr/local/share/lua/5.1/onmt/utils/MemoryOptimizer.lua:172: in function 'optimize'
/usr/local/share/lua/5.1/onmt/utils/Memory.lua:36: in function 'optimize'
/usr/local/share/lua/5.1/onmt/train/Trainer.lua:79: in function 'closure'
/usr/local/share/lua/5.1/onmt/utils/Parallel.lua:79: in function 'launch'
/usr/local/share/lua/5.1/onmt/train/Trainer.lua:61: in function 'train'
cover_train.lua:118: in function 'main'
cover_train.lua:123: in main chunk
[C]: in function 'dofile'
/usr/local/lib/luarocks/rocks/trepl/scm-1/bin/th:150: in main chunk
[C]: at 0x004057a0

I still do not know why, but I fixed this by replacing if torch.isTensor(l) then with if torch.isTensor(l) and l:storage() then like if torch.isTensor(t) and t:storage() then.


Does not work on Android with USB-Serial adapters

Does not work on Android with FTDI or Prolific USB serial adapters. Other programs like "GCodePrintr" and "Serial USB Terminal" works without any problems with the same adapters.

Hello @jnorre ,I had the requirement for Serial USB Terminal and testing device available for the same.So i was not able to test on all types of adapter.You can modify the code with according to your requirement,Any PR in this regard will be welcomed.I have used this repository "https://github.com/Andjdd/usbserialport" from github for building this up. I hope this repository can help you too.


Use instance type from variable

The servers use a hard coded instance type, of t2.micro. Added the variable server_instance_type and set the variable default to t2.micro.

Thank you for your submission! We require that all contributors sign our Contributor License Agreement ("CLA") before we can accept the contribution. Read and sign the agreement
Learn more about why HashiCorp requires a CLA and what the CLA includes
Have you signed the CLA already but the status is still pending? Recheck it.

Thanks for the PR. Please note that, as documented in the README, the code at the root of this repo is example code. The code you want to use for real use cases is in the modules sub-folder, and that code already allows instance type and all other relevant params to be configured.

@brikis98 thanks, good point. I forgot these are made to be copied, not technically "used". We are good to close this out then.


Enable native libraries on macOS

Fixes #3595 by using -fvisibility=hidden and the visibility attribute supported by both gcc and clang rather than the previous gcc-only mechanism for symbol hiding. Also brings over cfg changes from #3594 which enable native-lib functionality on all unixes.
Thanks for taking a look, feedback very welcome!
cc @RalfJung

Thanks for the quick review, I made these changes.

Hrm, I expected that to fix the labels. Maybe:
@rustbot ready

Great, thanks :-)
@bors r+

:pushpin: Commit f310010b194ca1a640a222df6f71d891e0997b8b has been approved by RalfJung
It is now in the queue for this repository.

:hourglass: Testing commit f310010b194ca1a640a222df6f71d891e0997b8b with merge e128132e7c7b0545f60f26220df60dca84f4b024...

:sunny: Test successful - checks-actions
Approved by: RalfJung
Pushing e128132e7c7b0545f60f26220df60dca84f4b024 to master...


Train with my own dataset

Thanks for your great work,
I want to train with my own dataset, how can I do it ?

Change the dataset related informations ot the configure file please.


Support raw SQL Alchemy

Based on pull request from dtheodor for issue #71. With the conflict fixed.
Happy to fix any other issues that come up to get this in a release.

For anyone that wants this sort of functionality, a separate python module exists called fdt-sqlalchemy. It has a bug where there is an errant print, which I've removed and documented a workaround for here.

Thanks for the work and the information....
is there anything that block this from being merge or something ?
it could be really cool to have that :)

I know it's been a long time, but do you still have any interest in this?
If so, it will need to catch it up with master and retest. If not, we'll probably close it out.
Is it safe to assume that this supersedes #75?

Hi. Yes I have interest.
Hopefully will get to it on a weekend soon.
Please ping me if I forget.

Sounds good, thanks @illume , no rush. Thanks!


Is NodeOs  intended to be Self Hosting?

I am wondering with the advent of web assembler there is the need to have Emscripten / Binaryen supported by clang / LLVM tool chains (I am looking a hosting something like web assembly explorer)
https://mbebenita.github.io/WasmExplorer/
given that to configure a new kernel and compile it, and creating a new bootable image is not that great
I previously mentioned the tiny C compiler and tccboot which actually compiled a linux kernel as it booted!
I post here to avoid doing as much duplication of effort, I have worked in the past with very simple co-operative kernel such as the one on the ESP-Vroom32 which seems to support much of Node.js
Thanks in Advance
Laust

Ideally, NodeOS would get to be self-hosted in the way of beng able to have build tools and compile NodeOS images from inside NodeOS itself, but there's still pending work towards have a C compiler running inside NodeOS, ideally one written or transpiled to Javascript like LLVM.js or a binary one, but didn't have advances on this area yet. This is also important due to compiled Node.js modules, and maybe wasm could help here, but I've not found anything regarding to this yet, any help in this area is welcome.
Anyway, I don't fully understand what's your purposse nor the relationship with wasm nor tinycc, could you be able to explain?

timy c is the smallest compiler that I know of that is able to compile a linux kernel and very fast indeed
https://bellard.org/tcc/ its dated but an interesting data point :-)
I have not had the time to try this out in wasm, but I like the idea of generating c in javascript have it compiled and run as a promise within a javascript context without referring to any external resources.
one you have this many more things are possible :-) ;-)
Cheers
Laust


...at TrafficManager.UI.UIBase.Finalize ()

Getting this at end of my log file, apparently something to do with recently updated Esc handler.
Colossal Handler caught:
  at (wrapper managed-to-native) UnityEngine.Component:GetComponentFastPath (System.Type,intptr)
  at UnityEngine.Component.GetComponent[CustomKeyHandler] () [0x00000] in <filename unknown>:0
  at TrafficManager.UI.UIBase.Finalize () [0x00000] in <filename unknown>:0

Could you upload that output_log for research?

Ah, sorry, was doing #91
LMK if you need more infos for this #92


Reorder List in cpp

Aim
Add a program that reorders the LinkedList as the order given below
For example: Given 1->2->3->4->5 its reorder is 1->5->2->4->3.
It is the GFG problem
Problem link

What features will your script have
It has the basic LinkedList template and reorders function which returns the head of the new LinkedList.
Time complexity: O(N)& Space complexity O(1)

Programming language

[ ] C
[x] C++
[ ] Java
[ ] Python

I want to do this in java


Add OMF Ingress Samples

@derekendres For some reason I can't seem to add you as a reviewer, but here are the samples for OMF Ingress with our Public libraries. I've done some work to better mimic what the OCS Portal does in terms of connections, and added a mock "device" to send OMF messages to hopefully give a better idea of a use case


Adding custom configmap to Grafana deployment

What did you do?
I'm trying to add a configmap to the grafana deployment, without modifying the original /vendor/grafana/grafana.libsonnet file.
However, due to inexperience with jsonnet/ksonnet, I seem to be stuck. By trying to add a new volume, I'm actually overwriting the existing volumes attached to the deployment. I'm looking for a way to add to rather than overwrite the existing volumes, as well as add a mount to the volume mounts of the grafana container that is already defined in grafana.libsonnet.
Did you expect to see some different?
Ideally, I would be able to mount the custom config map in the grafana container without modifying grafana.libsonnet itself.

Manifests:
I added this block to my own custom.jsonnet file

  grafana+:: {
      local deployment = k.apps.v1.deployment;
      local volume = deployment.mixin.spec.template.spec.volumesType;
      local customVolume = volume.withName(volumeName) + volume.mixin.configMap.withName(configMapName);
      deployment.mixin.spec.template.spec.withVolumes(customVolume),
      // TODO add volume mount to container
  },

Relevant parts from grafana.libsonnet
  // [...]
  grafana+: {
    // [...]
    deployment:
      local deployment = k.apps.v1.deployment;
      local container = k.apps.v1.deployment.mixin.spec.template.spec.containersType;
      local volume = k.apps.v1.deployment.mixin.spec.template.spec.volumesType;
      // [...]

      local configVolumeName = 'grafana-config';
      local configSecretName = 'grafana-config';
      local configVolume = volume.withName(configVolumeName) + volume.mixin.secret.withSecretName(configSecretName);
      local configVolumeMount = containerVolumeMount.new(configVolumeName, '/etc/grafana');

      local storageVolumeName = 'grafana-storage';
      local storageVolume = volume.fromEmptyDir(storageVolumeName);
      local storageVolumeMount = containerVolumeMount.new(storageVolumeName, '/var/lib/grafana');

      local datasourcesVolumeName = 'grafana-datasources';
      local datasourcesSecretName = 'grafana-datasources';
      local datasourcesVolume = volume.withName(datasourcesVolumeName) + volume.mixin.secret.withSecretName(datasourcesSecretName);         local datasourcesVolumeMount = containerVolumeMount.new(datasourcesVolumeName, '/etc/grafana/provisioning/datasources');
      local dashboardsVolumeName = 'grafana-dashboards';
      local dashboardsConfigMapName = 'grafana-dashboards';
      local dashboardsVolume = volume.withName(dashboardsVolumeName) + volume.mixin.configMap.withName(dashboardsConfigMapName);            local dashboardsVolumeMount = containerVolumeMount.new(dashboardsVolumeName, '/etc/grafana/provisioning/dashboards');

      local volumeMounts =
        [
          storageVolumeMount,
          datasourcesVolumeMount,
          dashboardsVolumeMount,
        ] +
        [
          local dashboardName = std.strReplace(name, '.json', '');
          containerVolumeMount.new('grafana-dashboard-' + dashboardName, '/grafana-dashboard-definitions/0/' + dashboardName)
          for name in std.objectFields($._config.grafana.dashboards)
        ] +
        [
          local dashboardName = std.strReplace(name, '.json', '');
          containerVolumeMount.new('grafana-dashboard-' + dashboardName, '/grafana-dashboard-definitions/0/' + dashboardName)
          for name in std.objectFields($._config.grafana.rawDashboards)
        ] +

        if std.length($._config.grafana.config) > 0 then [configVolumeMount] else [];

      local volumes =
        [
          storageVolume,
          datasourcesVolume,
          dashboardsVolume,
        ] +
        [
          local dashboardName = 'grafana-dashboard-' + std.strReplace(name, '.json', '');
          volume.withName(dashboardName) +
          volume.mixin.configMap.withName(dashboardName)
          for name in std.objectFields($._config.grafana.dashboards)
        ] +
        [
          local dashboardName = 'grafana-dashboard-' + std.strReplace(name, '.json', '');
          volume.withName(dashboardName) +
          volume.mixin.configMap.withName(dashboardName)
          for name in std.objectFields($._config.grafana.rawDashboards)
        ] +
        if std.length($._config.grafana.config) > 0 then [configVolume] else [];

      local c =
        container.new('grafana', $._config.imageRepos.grafana + ':' + $._config.versions.grafana) +
        (if std.length($._config.grafana.plugins) == 0 then {} else container.withEnv([env.new('GF_INSTALL_PLUGINS', std.join(',', $._config.grafana.plugins))])) +
        container.withVolumeMounts(volumeMounts) +
        container.withPorts(containerPort.newNamed(targetPort, portName)) +
        container.mixin.readinessProbe.httpGet.withPath('/api/health') +
        container.mixin.readinessProbe.httpGet.withPort(portName) +
        container.mixin.resources.withRequests($._config.grafana.container.requests) +
        container.mixin.resources.withLimits($._config.grafana.container.limits);

      deployment.new('grafana', 1, c, podLabels) +
      deployment.mixin.metadata.withNamespace($._config.namespace) +
      deployment.mixin.metadata.withLabels(podLabels) +
      deployment.mixin.spec.selector.withMatchLabels(podLabels) +
      deployment.mixin.spec.template.spec.withNodeSelector({ 'beta.kubernetes.io/os': 'linux' }) +
      deployment.mixin.spec.template.spec.withVolumes(volumes) +
      deployment.mixin.spec.template.spec.securityContext.withRunAsNonRoot(true) +
      deployment.mixin.spec.template.spec.securityContext.withRunAsUser(65534) +
      deployment.mixin.spec.template.spec.withServiceAccountName('grafana'),
  },

Anything else we need to know?:
I'm not trying to add a new dashboard, the configmap contains a file needed for a cluster-specific problem.

I'm not trying to add a new dashboard, the configmap contains a file needed for a cluster-specific problem.

Could you elaborate what it is that you are trying to solve?

Usually, add a new dashboard works by adding a new object to the global grafanaDashboards object. As we loop over that object to generate the configmaps and volumes those object are added as dashboards automatically.
Basically:
{
  grafanaDashboards+:: {
    'mydashboard.json':
    ...
  }
}

I'm adding an nsswitch.conf file containing hosts: files dns so that host aliases are resolved in the correct order. I'm not fully clear about the details, but basically, some go/alpine based containers ignore the hosts file as set by Kubernetes when using hostAlias specifications in the deployment .yaml, unless the nsswitch.conf file is added.
See for example https://github.com/golang/go/issues/22846
I have to use host aliases to make sure the authentication setup for this cluster works correctly. Manually updating the grafana deployment yaml works for now, but obviously adding a few lines to my .libsonnet file is vastly preferred.

@metalmatze thanks for your reply, but this is specifically NOT about dashboards, but a separate configMap, see my previous reply.
I already have some custom dashboards up and running, indeed through grafanaDashboards, which works like a charm!

I think this is something that is pretty specific to your environment so I would suggest you to do a patch specific to your problem. A jsonnet patch for this would
{
   grafana+:: {
    deployment+:
      {
        spec+: {
          template+: {
            spec+: {
              volumes+: [ * volume definition * ],
              containers:
                std.map(
                  function(c)
                    if c.name == 'grafana' then
                      c {
                        volumeMounts+: [
                          * volume mount definition *,
                        ],
                      }
                    else
                      c,
                  super.containers,
                ),
            },
          },
        },
      },
  },
}

Thank you, that did the trick!


Laravel 5.x support?

Any chance you will change the required code changes to support Laravel 5?  I'd super love to use this service provider in conjunction with bitbucket.  Nice work so far!!!

yeah, how about 5.x support?


Error on dayahead-optim POST call

Hi
I'm getting an error when triggering the REST call for the dayahead-optim.
This is what I get in the logs:
[2022-06-19 10:45:17,391] INFO in command_line: Setting up needed data
[2022-06-19 10:45:17,399] INFO in forecast: Retrieving weather forecast data using method = scrapper
[2022-06-19 10:45:19,945] INFO in forecast: Retrieving data from hass for load forecast using method = naive
[2022-06-19 10:45:19,947] INFO in retrieve_hass: Retrieve hass get data method initiated...
[2022-06-19 10:45:47,744] INFO in web_server:  >> Performing dayahead optimization...
[2022-06-19 10:45:47,745] INFO in command_line: Performing day-ahead forecast optimization
[2022-06-19 10:45:47,755] INFO in optimization: Perform optimization for the day-ahead
[2022-06-19 10:45:47,894] ERROR in optimization: It was not possible to find a valid solver for Pulp package
[2022-06-19 10:45:47,895] INFO in optimization: Status: Not Solved
[2022-06-19 10:45:47,895] WARNING in optimization: Cost function cannot be evaluated, probably None
[2022-06-19 10:45:47,904] ERROR in app: Exception on /action/dayahead-optim [POST]
Traceback (most recent call last):
  File "/usr/local/lib/python3.9/dist-packages/flask/app.py", line 2077, in wsgi_app
    response = self.full_dispatch_request()
  File "/usr/local/lib/python3.9/dist-packages/flask/app.py", line 1525, in full_dispatch_request
    rv = self.handle_user_exception(e)
  File "/usr/local/lib/python3.9/dist-packages/flask/app.py", line 1523, in full_dispatch_request
    rv = self.dispatch_request()
  File "/usr/local/lib/python3.9/dist-packages/flask/app.py", line 1509, in dispatch_request
    return self.ensure_sync(self.view_functions[rule.endpoint])(**req.view_args)
  File "/usr/local/lib/python3.9/dist-packages/emhass/web_server.py", line 151, in action_call
    opt_res = dayahead_forecast_optim(input_data_dict, app.logger)
  File "/usr/local/lib/python3.9/dist-packages/emhass/command_line.py", line 195, in dayahead_forecast_optim
    opt_res_dayahead = input_data_dict['opt'].perform_dayahead_forecast_optim(
  File "/usr/local/lib/python3.9/dist-packages/emhass/optimization.py", line 493, in perform_dayahead_forecast_optim
    self.opt_res = self.perform_optimization(df_input_data, P_PV.values.ravel(),
  File "/usr/local/lib/python3.9/dist-packages/emhass/optimization.py", line 384, in perform_optimization
    opt_tp["P_grid"] = [P_grid_pos[i].varValue + P_grid_neg[i].varValue for i in set_I]
  File "/usr/local/lib/python3.9/dist-packages/emhass/optimization.py", line 384, in <listcomp>
    opt_tp["P_grid"] = [P_grid_pos[i].varValue + P_grid_neg[i].varValue for i in set_I]
TypeError: unsupported operand type(s) for +: 'NoneType' and 'NoneType'

This is my config.yaml configuration:
rest_command:
  dayahead_optim:
    url: http://<IP_ADDRESS>:5000/action/dayahead-optim
    method: POST
    content_type: 'application/json'
    payload: '{}'

  publish_data:
    url: http://<IP_ADDRESS>:5000/action/publish-data
    method: POST
    content_type: 'application/json'
    payload: '{}'

Hi, the system is not finding a valid solver.
Try changing the solver in the configuration to COIN_CMD and pass the path /usr/bin/cbc.
As a second option you could try to use the alternative GPLK solver, you will need to use GLPK_CMD as the solver and maybe manually install the GLPK solver with:
sudo apt-get install glpk-utils

You will need to to this inside the container. To go inside the container do:
docker exec -it <container-name> /bin/bash

The container name can be found using:
docker ps -a

Hi, closing this for now as this is a know and proven solution for this. Again feel free to reopen if you need assistance.


Don't see mention notification icon

I am on Version 1.1.0 (259) of the Mac App.
This also does not work on my Version 1.1.0 (339) of the iOS App.
When I get a mention in a tribe, I don't get the separate "@" icon in the left sidebar. For example:

@soup20 found the issue and working on the fix. I will release a version with the fix today or tomorrow at most


Wrong query building when fetching with withRelated

I came across this issue running queries with option withRelated and polymorphic props. The queries executed against the db end up having their "deleted_at" checks mixed up and break.
In my scenario there are 3 entities: girls, boys, parents. parents have a child prop that can either be a girl or a boy.
const Boy = bookshelf.Model.extend({
  tableName: 'boys',
  softDelete: true
})
const Girl = bookshelf.Model.extend({
  tableName: 'girls',
  softDelete: true
})
const Parent = bookshelf.Model.extend({
  tableName: 'parents',
  child () {
    return this.morphTo('child', Boy, Girl)
  },
  softDelete: true
})

Then, have to create a boy, a parent having that boy, a girl and a parent having that girl:
const waterfall = require('promise-waterfall')

function createBoy() {
  return Boy.forge().save().then(function (child) {
    return child.id
  })
}
function createGirl() {
  return Girl.forge().save().then(function (child) {
    return child.id
  })
}
function createParent() {
  return Parent.forge().save().then(function (parent) {
    return parent.id
  })
}
function setChild(parentId, childId, type) {
  return Parent.forge({
    id: parentId
  }).save({
    child_id: childId,
    child_type: type
  })
}

const tasks = waterfall([
  function () {
    return createBoy()
  },
  function (childId) {
    return createParent().then(function (parentId) {
      return setChild(parentId, childId, 'boys')
    })
  },
  function () {
    return createGirl()
  },
  function (parentBoyId) {
    return createParent().then(function (parentId) {
      return setChild(parentId, parentBoyId, 'girls')
    })
  }
])

Finally, fetch all parents with their children:
tasks.then(function () {
  return Parent.fetchAll({
    withRelated: ['child'],
    debug: true
  })
})

The output is:
{ method: 'select',
  options: {},
  timeout: false,
  bindings: [],
  sql: 'select "parents".* from "parents" where "parents"."deleted_at" is null and "parents"."deleted_at" is null' }
{ method: 'select',
  options: {},
  timeout: false,
  bindings: [ '1' ],
  sql: 'select "boys".* from "boys" where "id" in (?)' }
{ method: 'select',
  options: {},
  timeout: false,
  bindings: [ '1' ],
  sql: 'select "girls".* from "girls" where "id" in (?) and "boys"."deleted_at" is null and "girls"."deleted_at" is null' }
Err: error: select "girls".* from "girls" where "id" in ($1) and "boys"."deleted_at" is null and "girls"."deleted_at" is null - missing FROM-clause entry for table «boys»

As can be seen in the logs, the and "boys"."deleted_at" is null in the last query (girls) is wrong and breaking. In addition, that condition is missing from the second query (boys).
Tested using:

<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>
Will really appreciate any help with the issue. Thanks!!!
PS: You can see the whole test file at test-bookshelf-paranoia.js

Hello @gabmontes and thanks for the reporting. This seems to be related with how bookshelf handles morph relations. I'll do a better research during the week.

Any updates on this?
Thanks!

Unfortunately we haven't had a chance to look into it yet - we'll do our best to do so in the next week or so. PR's welcome!

@gabmontes @kamronbatman sorry for the delay was a tricky one to debug. Morph relations should be working with v0.8.0, let me know if everything is ok :-)


Version conda builds using git

We'd like to build conda packages without having to specify the version number manually. The version of the conda package should always correspond to the version according to git.
Use git to version the conda package, so that:

The most recent tag on the branch being built is used as the version number, and
The number of commits since that tag on this branch is used as the build number.

(2) means that we need to stop rebasing non-local branches. Instead, use git merge.
Note that the most recent tag on master is UNSTABLE which isn't a version number:
$ git describe --abbrev=0 --tag
UNSTABLE

Once this is merged in, we should tag the merge commit with a new version (v0.8.1?) so that the resulting package has a name that makes sense.

@netj looks like dockerhub access is flaky:
$ DOCKER_IMAGE_TO_PUSH=$DOCKER_IMAGE_TEST_PREFIX; if [[ $TRAVIS_TEST_RESULT = 0 ]]; then DOCKER_IMAGE_TO_PUSH+=PASS; else DOCKER_IMAGE_TO_PUSH+=FAIL; fi;
after_script.2
0.01s$ if [[ $TRAVIS_PULL_REQUEST = false ]]; then tag=travis.$TRAVIS_BRANCH; else tag=travis-pr$TRAVIS_PULL_REQUEST; fi; tag=$(echo -n "$tag" | tr -c 'A-Za-z0-9_.-' _);
$ docker login -e "$encrypted_DOCKER_EMAIL" -u "$encrypted_DOCKER_USERNAME" -p "$encrypted_DOCKER_PASSWORD";
Flag --email has been deprecated, will be removed in 1.13.
Login with your Docker ID to push and pull images from Docker Hub. If you don't have a Docker ID, head over to https://hub.docker.com to create one.
Username:
No output has been received in the last 10m0s, this potentially indicates a stalled build or something wrong with the build itself.

@alldefector Looks like the credentials are not decrypted for non-HazyResearch github users.  Will fix travis.yml to skip the pushing instead of hanging


PowerToys 0.87.1-x64 does not install

Microsoft PowerToys version
0.87.1
Installation method
GitHub
Running as admin
Yes
Area(s) with issue?
Installer
Steps to reproduce
Attempted to install PowerToys as Admin and received a popup message that it could not be installed as the previous version must be uninstalled first. I tried to uninstall PowerToys with Control Panel - Programs and Features and got the same popup. I then used Iobit Uninstaller to remove it and all residual files, then rebooted the PC. The same error window came up when attempting to install PowerToys 0.87.1 and 0.87
✔ Expected Behavior
Expected PowerToys to update as usual.
❌ Actual Behavior
Unable to update or install PowerToys
Other Software
No response

Here is the install log file
[8264:8EE8][2024-12-20T10:14:54]i001: Burn v<IP_ADDRESS>22, Windows v10.0 (Build 26100: Service Pack 0), path: C:\Users\dave\AppData\Local\Temp{1EF8CAC9-898A-435C-9A85-C0061C5DDAFE}.cr\PowerToysUserSetup-0.87.1-x64.exe
[8264:8EE8][2024-12-20T10:14:54]i000: Initializing string variable 'InstallFolder' to value '[LocalAppDataFolder]PowerToys'
[8264:8EE8][2024-12-20T10:14:54]i000: Initializing string variable 'MsiLogFolder' to value '[LocalAppDataFolder]\Microsoft\PowerToys'
[8264:8EE8][2024-12-20T10:14:54]i000: Initializing version variable 'MinimumVersion' to value '<IP_ADDRESS>'
[8264:8EE8][2024-12-20T10:14:54]i000: Initializing version variable 'TargetPowerToysVersion' to value '0.87.1'
[8264:8EE8][2024-12-20T10:14:54]i000: Initializing version variable 'DetectedPowerToysVersion' to value '<IP_ADDRESS>'
[8264:8EE8][2024-12-20T10:14:54]i000: Initializing version variable 'DetectedPowerToysUserVersion' to value '<IP_ADDRESS>'
[8264:8EE8][2024-12-20T10:14:54]i000: Initializing version variable 'DetectedWindowsBuildNumber' to value '0'
[8264:8EE8][2024-12-20T10:14:54]i009: Command Line: '-burn.clean.room=C:\Zip\PowerToysUserSetup-0.87.1-x64.exe -burn.filehandle.attached=796 -burn.filehandle.self=624'
[8264:8EE8][2024-12-20T10:14:54]i000: Setting string variable 'WixBundleOriginalSource' to value 'C:\Zip\PowerToysUserSetup-0.87.1-x64.exe'
[8264:8EE8][2024-12-20T10:14:54]i000: Setting string variable 'WixBundleOriginalSourceFolder' to value 'C:\Zip'
[8264:8EE8][2024-12-20T10:14:54]i000: Setting string variable 'WixBundleLog' to value 'C:\Users\dave\AppData\Local\Temp\powertoys-bootstrapper-msi-0.87.1_20241220101454.log'
[8264:8EE8][2024-12-20T10:14:54]i000: Setting string variable 'WixBundleName' to value 'PowerToys (Preview) x64'
[8264:8EE8][2024-12-20T10:14:54]i000: Setting string variable 'WixBundleManufacturer' to value 'Microsoft Corporation'
[8264:8700][2024-12-20T10:14:54]i000: Setting numeric variable 'WixStdBALanguageId' to value 1033
[8264:8700][2024-12-20T10:14:54]i000: Setting version variable 'WixBundleFileVersion' to value '<IP_ADDRESS>'
[8264:8EE8][2024-12-20T10:14:54]i100: Detect begin, 3 packages
[8264:8EE8][2024-12-20T10:14:54]i000: Setting version variable 'DetectedPowerToysUserVersion' to value '<IP_ADDRESS>'
[8264:8EE8][2024-12-20T10:14:54]i000: Setting version variable 'DetectedPowerToysVersion' to value '<IP_ADDRESS>'
[8264:8EE8][2024-12-20T10:14:54]i000: Setting string variable 'DetectedWindowsBuildNumber' to value '26100'
[8264:8EE8][2024-12-20T10:14:54]i000: Registry key not found. Key = 'Software\Microsoft\EdgeUpdate\Clients{F3017226-FE2A-4295-8BDF-00C3A9A7E4C5}'
[8264:8EE8][2024-12-20T10:14:54]i000: Setting numeric variable 'HasWebView2PerUser' to value 0
[8264:8EE8][2024-12-20T10:14:54]i000: Setting numeric variable 'HasWebView2PerMachine' to value 1
[8264:8EE8][2024-12-20T10:14:54]i052: Condition 'HasWebView2PerMachine OR HasWebView2PerUser' evaluates to true.
[8264:8EE8][2024-12-20T10:14:54]i103: Detected related package: {17E94930-2027-47F4-BE39-422251714851}, scope: PerUser, version: <IP_ADDRESS>, language: 0 operation: MajorUpgrade
[8264:8EE8][2024-12-20T10:14:54]i103: Detected related package: {17E94930-2027-47F4-BE39-422251714851}, scope: PerUser, version: <IP_ADDRESS>, language: 0 operation: MajorUpgrade
[8264:8EE8][2024-12-20T10:14:54]i101: Detected package: TerminatePowerToys, state: Absent, cached: None
[8264:8EE8][2024-12-20T10:14:54]i101: Detected package: WebView2, state: Present, cached: None
[8264:8EE8][2024-12-20T10:14:54]i101: Detected package: PowerToysUserSetup_0.87.1_x64.msi, state: Absent, cached: None
[8264:8EE8][2024-12-20T10:14:54]i052: Condition 'MinimumVersion >= DetectedPowerToysVersion' evaluates to false.
[8264:8EE8][2024-12-20T10:14:54]e000: PowerToys is already installed on this system for all users. We recommend first uninstalling that version before installing this one.
[8264:8EE8][2024-12-20T10:14:54]e000: Error 0x81f40001: Bundle condition evaluated to false: MinimumVersion >= DetectedPowerToysVersion
[8264:8EE8][2024-12-20T10:14:54]i199: Detect complete, result: 0x0

Looks like there might be an machine-scope installation. Can you please try installing 0.87.1 machine-scope? That might override whatever is in there that didn't clean up first. Does this help? /needinfo

What is a machine-scope installation?