Feature/stations merging adding station interaction and development

Description
Added station interaction with the player inventory and a way to get items i.e. the apple tree. As it randomly spawns an item. Additionally developed the use of a station to include components which enable both the cooking and chopping of items. Also added the servery station which allows for items to be submitted to the customer.
Fixes / Closes # (issue)
Tickets numbers: #194, #167, #122, #163
Type of change

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

How Has This Been Tested?
This has been tested through in game testing and ensuring that the creation, transferring, and serving of items was implemented. It was also tested through JUnit testing to ensure that all of the station interaction and other areas work correctly. This has ensured that it is functional.
Checklist:

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

Summary

Added station-player interaction
-Refactored existing item updating code to align with the new station architecture
-Added UnitTesting for StationComponents

Worked with Hayden on this and approving it


refactor(api): add create_file_runner factory

Overview
This PR is 3 of 4 working towards #7816.

~chore(api,robot-server): update testing dev deps #7855~ Merged
~refactor(robot-server): add experimental protocol router #7856~ Merged
refactor(api): add create_file_runner factory #7857
refactor(robot-server): start wiring /sessions into runner and engine #7858

This PR prepares for session wireup to the JsonFileRunner by adding a factory function to set up a JsonFileRunner.
~Blocked by #7856, will require sync after merge~
Changelog

Addded create_file_runner factory to create a file runner instance given:

file_type: Python or JSON (Python and others left unimplemented)
file_path: path on the filesystem to the file
engine: ProtocolEngine instance

Review requests

[ ] Code and tests look good
[ ] Wire-up makes sense in followup PR #7858

Risk assessment
Low, not hooked to production code

Changes addressed during PR review


QnAMakerDialog should support passed-in text instead of only Activty.Text

Use this query to search for the most popular feature requests.
Is your feature request related to a problem? Please describe.
When the incoming activity into a bot does not include the text to be sent to a QnAMakerDialog, there is not a good way to pass this in as a parameter. QnAMakerDialog ONLY works with Message activities where the question is the Text property.
Describe the solution you'd like
I would like the text to send to QnAMakerDialog to start it to be passed in as part of the options, instead of only looking.
Also, the need for the Activity to be a Message type could also then be removed (i.e. if I wanted to call QnAMakerDialog after an OAuthPrompt, there would be an event activity in the TurnContext.
Describe alternatives you've considered
Currently i have to change the type of Activity and set the Text property of the Activity used by the TurnContext.
[enhancement]

Triaged by development team. This will, likely, need a rewrite. Reference: https://github.com/microsoft/botframework-sdk/issues/5851


Create out/assembly dir on build so that intermediate .ts files can be generated

fixed #122

closing since merged


Added ability to force restart a VM

Signed-off-by: Vatsal Parekh<EMAIL_ADDRESS>What this PR does / why we need it:
This PR brings an additional option --force to virtctl restart and also the endpoint which will delete the VMI when this flag is passed, eventually doing a forced restart by rescheduling the VMI pod.
We explicitly do not pass any signal to the VMI, as we understand that this option will be used for VMs that are stuck in some way and need a forced restart
Release note:

Added ability to force restart a VM (virtctl + endpoint for it)

/hold
Could you describe what exactly you try to solve?

/hold
Could you describe what exactly you try to solve?

@rmohr Updated the description

/retest

I would suggest adding an url parameter to the existing restart endpoint. something like force and a grace-period parameter (the same parameter like it is used on pods when you do deletes there). And in fact as implementation it should then directly do a --force delete on the pod where the VMI is in.

I know that I suggested the url parameter. However, I research the details in k8s a liile more. If I do a kubectl delete pods mypod --force --grace-period  -v9, I see that the following happens:
I1021 14:59:01.848401   24795 request.go:942] Request Body: {"gracePeriodSeconds":0,"propagationPolicy":"Background"}
I1021 14:59:01.848535   24795 round_trippers.go:419] curl -k -v -XDELETE  -H "Accept: application/json" -H "Content-Type: application/json" -H "User-Agent: .kubectl/v1.13.3 (linux/amd64) kubernetes/721bfa7" 'https://<IP_ADDRESS>:32768/api/v1/namespaces/default/pods/mypod'

Can we align with that?
So basically, --force would just be a confirmation in virtctl to allow passing --grace-period 0.
In general, I am not sure if our code-base is ready to dynamically alter the grace-period. So a force-reset with 0 can be done and will have the desired effect. However we don't have a way right now to tell the application inside virt-launcher about the new deadline. We would have to adjust that also. I think we can process with --grace-period 0, however I think we may want  to prevent setting anything else than 0 when it is specified for now.

I would suggest adding an url parameter to the existing restart endpoint. something like force and a grace-period parameter (the same parameter like it is used on pods when you do deletes there). And in fact as implementation it should then directly do a --force delete on the pod where the VMI is in.

I know that I suggested the url parameter. However, I research the details in k8s a liile more. If I do a kubectl delete pods mypod --force --grace-period -v9, I see that the following happens:
I1021 14:59:01.848401   24795 request.go:942] Request Body: {"gracePeriodSeconds":0,"propagationPolicy":"Background"}
I1021 14:59:01.848535   24795 round_trippers.go:419] curl -k -v -XDELETE  -H "Accept: application/json" -H "Content-Type: application/json" -H "User-Agent: .kubectl/v1.13.3 (linux/amd64) kubernetes/721bfa7" 'https://<IP_ADDRESS>:32768/api/v1/namespaces/default/pods/mypod'

Can we align with that?
So basically, --force would just be a confirmation in virtctl to allow passing --grace-period 0.
In general, I am not sure if our code-base is ready to dynamically alter the grace-period. So a force-reset with 0 can be done and will have the desired effect. However we don't have a way right now to tell the application inside virt-launcher about the new deadline. We would have to adjust that also. I think we can process with --grace-period 0, however I think we may want to prevent setting anything else than 0 when it is specified for now.

@rmohr
I feel --force without passing --grace-period 0 is better UX and we should keep that, user can anyway pass other number if they want.
About setting grace-period other then 0, it seems tricky, but my understanding is we expect this to be used in the scenarios where the VM is not in a state for a clean restart, and we want to force-restart by deleting a pod, or with setting shorter grace-period on the pod for deletion.
@fabiand your views here?

I feel --force without passing --grace-period 0 is better UX and we should keep that, user can anyway pass other number if they want.

I would prefer to stick with the kubectl way. --force there basically allows to pass a grace-period of 0.
Here the help from  kubectl delete:
      --force=false: Only used when grace-period=0. If true, immediately remove resources from API and bypass graceful
deletion. Note that immediate deletion of some resources may result in inconsistency or data loss and requires
confirmation.
      --grace-period=-1: Period of time in seconds given to the resource to terminate gracefully. Ignored if negative.
Set to 1 for immediate shutdown. Can only be set to 0 when --force is true (force deletion).

I would say that in our case it applies even more than with pure k8s that giving no grace-period can potentially corrupt the workload.

About setting grace-period other then 0, it seems tricky, but my understanding is we expect this to be used in the scenarios where the VM is not in a state for a clean restart, and we want to force-restart by deleting a pod, or with setting shorter grace-period on the pod for deletion.

I would say that right now, you don't need anything else than 0, because you want to implement an immediate restart. Still, for the future it will make sense to allow modifying the pre-defined grace-period on stop, restart and delete. It is for instance perfectly valid that one configures a grace-period of 10 hours. However later constraints require to do a reboot within a grace-period of one hour.

/retest

/retest

Ok, so I thought the whole process through again. I think we need to take the corner-cases a little bit more into account. I think that the best flow, which we can come up right now would be the following:

If the usual restart checks fail (like no VMI, the VM run-strategy is wrong, ...) return with an appropriate error.
If the VMI exists, just delete it as usual
Finally, if the grace period is 0, then in addition also delete the Pod, if it exists.

/retest

/retest

/lgtm
/approve
@vatsalparekh can you please create a PR in the user-guide which describes the new feature?

/hold cancel

/lgtm
/approve
@vatsalparekh can you please create a PR in the user-guide which describes the new feature?

@rmohr sure


Provide an interface for specifying preferred output style

The desired output style can be based on many factors, including indentation, quote preference (single or double), trailing commas in arrays/objects, etc.
There should be an interface for specifying this. Perhaps as argument(s) to dump[s]

I'll also tack on that this should provide an interface for things like identifiers and when to display an integer as a hexadecimal number, or floats with leading/trailing decimal points.


Seperate File loading Per Difficulty

Describe your problem here.
this can be considered a question, asking for help, AND a request for the base engine.
Is it possible to play different audio and load seperate events on Different difficulties?
i am making a mod of my own with a new "Legend" difficulty and it's supposed to have remixed or extended versions of the original songs on legend difficulty.
i am hoping there is a way it can load the files by adding -"difficulty" at the end of the file name. and if there isnt a file with the current difficulty at the end of the searched file, it would search for the file without a difficulty attached. for example,
lets list the files here.
Inst
Inst-hard
Inst-legend
Voices
Voices-hard
Voices-legend
Events
Events-hard
Events-legend
if i was on easy or normal it would use files 1, 4, and 7
but it i was on hard it would use files 2, 5, and 8
and if legend then it would use files, 3, 6, and 9
Is there a way to make this happen and if not can something like this be made or added to the base engine?
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
I did not modify anything from the base game

Is it possible to play different audio

you could use playMusic

load seperate events on Different difficulties

yes, events can be saved into the chart itself

can something like this be added to the base engine?

no (#3730)

I want this.

Create new "song" for each difficulty and in carting menu put the "song" difficulty name there


Timing problems - better to use writeValueWithResponse

First of all, thank you for the example code of how to print on a Phomemo Bluetooth printer. I managed to use it on the wider, and higher resolution model, the Phomemo M04AS.
One thing you or other users might find useful, is that after a lot of searching for the cause of a nasty error of randomly stopping printing, at last I found the problem:
in the printer.js module I changed  all    characteristic.writeValueWithoutResponse methods  to  characteristic.writeValueWithResponse,  and that solved the unreliable data transfer problem.
I guess, the writeValueWithoutResponse is better suited for streaming devices where an intermittent data loss is OK. With a printer it is not the case.

Thanks! I haven't run into any issues but it may be dependent on hardware/OS or due to the higher resolution of your printer. I've updated to using writeValueWithResponse.


Add message on home screen about requiring invite codes

Can we add a message on the pre-login home screen that "RecNet is under early development, and signing up requires an invite code"?
Thanks

Looks great. Let's say (keeping the emoji too):
"RecNet is under early development. Signing up requires an invite code."
Thanks!


UI namespace filter state is sticky between reports and workflows view

Summary

Enter a namespace into the namespace filter on reports page
Clear it out with the "x" button

Expected: the namespace is cleared out.
Actual: the namespace sticks around and persists across reports view and workflows view
https://user-images.githubusercontent.com/71290498/107568794-1c1b5680-6bad-11eb-8ccf-bcaef2e4cbf2.mp4
Diagnostics
What Kubernetes provider are you using? GKE
What version of Argo Workflows are you running? 2.12.8

Message from the maintainers:
Impacted by this bug? Give it a 👍. We prioritise the issues with the most 👍.

Related: clicking the workflows link in the side nav restores the namespace filter
https://user-images.githubusercontent.com/71290498/107571568-b0d38380-6bb0-11eb-892b-ec45559d1b08.mov

can you try v3.0.0-rc1?

I am also impacted by this. Will try latest v3.0.0-rc3 to see if it fixes the issue.

Also affected by this. And v3.0.0-rc1 is not supported by helm install yet

I think it is the case that the code for no workspace set (i.e. show all clusters) gets confused with undefined workspace. Would anyone like to investigate this?

Would anyone like to submit a PR to fix this?

Did some tests on 2.12.5, 3.0.0, 3.1.1 and master,
This is fixed since version 3.0


Add spinner to opening folder in Files view

There is sometimes a slight delay, potentially if the server is down it could also sit and wait for / fail.
Would be good to show a spinner to indicate to the user that we're doing someting

Added this in latest commit


Feature/cce 681

@saadmhybris  Remember the one reviewing should also merge the changes if everything is fine.

@joseegm thanks for the reminder. it's been a while so need to get back in the groove ;)

@saadmhybris  No big deal :)


Fix StyleCI config

Allow StyleCI to skip over the large test file, mainly full of data.

This change is

Coverage remained the same at 79.538% when pulling 34bcffd0a42473890569cb8418d1c7f1a5a871f8 on GrahamCampbell:patch-1 into 8814546b5ced07fdd0c8e8a1a3886fb3e177ae92 on voku:master.

Thanks for the fix.


Bug

Verify latest release

[X] I verified that the issue exists in the latest chatgpt release

Verify webapp is working

[X] I verify that the ChatGPT webapp is working properly for this account.

Environment details
New Account and API KEY
Describe the Bug
ChatGPTError: ChatGPT error 429: {
"error": {
"message": "You exceeded your current quota, please check your plan and billing details. For more information on this error, read the docs: https://platform.openai.com/docs/guides/error-codes/api-errors.",
"type": "insufficient_quota",
"param": null,
"code": "insufficient_quota"
}
}

The OpenAI API has rate limits. This error is telling you that you've exceeded your quota and need to request more from the OpenAI dashboard.

Just explained the issue: We need to charge 5$ to be able to consume now.
So just go to Billing section and register any credit card to charge.
And it works again.
Bye!


Reduce the number of ITs in CDS groups

The two CDS IT groups cds-task-schema-publish-disabled & cds-coordinator-metadata-query-disabled
runs a lot of ITs serially, leading to increased CI runtime.
Reducing the number of ITs running in the two groups to,
ITAppendBatchIndexTest (append-ingestion)
ITIndexerTest (batch-index)
ITOverwriteBatchIndexTest (batch-index)
ITCompactionSparseColumnIndexTest (compaction)
ITCompactionTaskTest (compaction)
ITKafkaIndexingServiceDataFormatTest (kafka-data-format)
Both these groups will be removed once CDS is enabled by default.

Unrelated UT failure


Addon still shows 1.10.1 although 1.12.0 should be available

Hi,
I'm running Hassio with HA 107.4.
The addon shows me that I'm on version 1.10.1 and doesn't offer any update. Other addons don't seem to have this problem. Is there something wrong with my installation or with the addon itself? Version 1.12 seems to be out for a week already.

First I uninstalled the addon and was surprised that it doesn't show up in the addon-store at all. I saw that the repository wasn't configured anymore. It seems that when you backup HA and restore it, that the repository URLs were not taken over (that's the only explanation I have). After re-adding it, I was able to see the newest version! :)


Adding changes to reduce false positives seen by customers

The changes made in the script are as follows:

For  Cve26858 we only show the error if the path in the error message “Download failed and temporary file  needs to be removed” is invalid (does not start with C:\Program Files\Microsoft\Exchange Server\V15\ClientAccess\OAB)
For set-virtual directory we throw an error only if the url is invalid
For the get-suspicious files we only show those if one of the 4 vulnerabilities (Cve26855 Or  Cve26857 or  Cve26858 or  Cve27065 ) is found.
This was done because even customers who did not have any vulnerabilities were alerted due to the presence of some zip files that they had created in their system.
These false positives lead to a lot of confusion and the uneasiness for the customers.
We also show an additional error message regarding web shells if we find logs of successful reset-virtualdirectory hits (having a bad anchor mailbox object) in the httpsproxy folder.

Please run .build\CodeFormatter.ps1 -Save and then correct the PSScriptAnalyzer issues manually:
RuleName                              Severity    ScriptName          Line Message
--------                              --------    ----------          ---- -------
PSAvoidTrailingWhitespace             Information Test-ProxyLogon.ps1 172  Line has trailing whitespace
PSAvoidTrailingWhitespace             Information Test-ProxyLogon.ps1 194  Line has trailing whitespace
PSAvoidTrailingWhitespace             Information Test-ProxyLogon.ps1 210  Line has trailing whitespace
PSAvoidTrailingWhitespace             Information Test-ProxyLogon.ps1 219  Line has trailing whitespace
PSAvoidTrailingWhitespace             Information Test-ProxyLogon.ps1 228  Line has trailing whitespace
PSAvoidTrailingWhitespace             Information Test-ProxyLogon.ps1 229  Line has trailing whitespace
PSPossibleIncorrectComparisonWithNull Warning     Test-ProxyLogon.ps1 186  $null should be on the left side of
                                                                           equality comparisons.
PSPossibleIncorrectComparisonWithNull Warning     Test-ProxyLogon.ps1 186  $null should be on the left side of
                                                                           equality comparisons.
PSUseDeclaredVarsMoreThanAssignments  Warning     Test-ProxyLogon.ps1 225  The variable 'FoundMaliciousResetVDir' is
                                                                           assigned but never used.


[Chore] Update Workflows to use self-hosted runner

Why
The update to the workflow for the iOS template repository must incorporate the newly set up GitHub self-hosted runner on a Mac Mini from MacStadium. The document can be found HERE.
The update will include the following changes:

Using the Self-Hosted Runner for workflows
Update Build, Sync Code Signing, and Distribute Steps

Who Benefits?

Development Team: Faster build times and improved performance lead to increased productivity and reduced waiting times.
Project Managers and Stakeholders: Increased reliability and stability facilitate smoother project management and timely delivery.
Clients and End Users: The updated workflow delivers more robust and bug-free applications, improving the user experience.
Company: Utilizing the self-hosted runner lowers costs and maximizes credits, reducing dependency on GitHub-hosted runners and providing scalability and flexibility for resource allocation.

@ducbm051291 For iOS templates, currently, when generating the project, we need to choose: github, bitrise, codemagic. I would prefer to add one more option (e.g. github-self-hosted), so the team can easily switch between different platforms.
cc @blyscuit

@suho Good idea. This issue can be for creating the new workflow files and the setup script 👍

Ok, got it


Opening a dimension with expression as title will break the FP

See movie:

Console error:
app.971ab3a2d32bcc3164dc.js:2 Error: Minified React error #31; visit https://reactjs.org/docs/error-decoder.html?invariant=31&args[]=object%20with%20keys%20%7BqStringExpression%7D for the full message or use the non-minified dev environment for full errors and additional helpful warnings.     at Ka (3577.d4eb0d58b075ebc43242.js:2:56251)     at e (3577.d4eb0d58b075ebc43242.js:2:61250)     at wo (3577.d4eb0d58b075ebc43242.js:2:72875)     at xi (3577.d4eb0d58b075ebc43242.js:2:121705)     at bs (3577.d4eb0d58b075ebc43242.js:2:108876)     at vs (3577.d4eb0d58b075ebc43242.js:2:108804)     at gs (3577.d4eb0d58b075ebc43242.js:2:108667)     at is (3577.d4eb0d58b075ebc43242.js:2:105808)     at Hl (3577.d4eb0d58b075ebc43242.js:2:46753)     at ts (3577.d4eb0d58b075ebc43242.js:2:102904)

This should be fixed by https://github.com/qlik-trial/sense-client/pull/23364.


Scan for games

I know a lot of people have requested this, so I'm creating an issue for this. We need a way to be able to scan a folder/pull in a list of games to be added automatically. However, this is no easy task. There is no centralized list of exes/hashes that I can use to check against. If I allow scanning a folder, people need to be able to pick the right exe, then have it do the necessary checks before adding each game.

You could add some logic to it and make it so VnManager look up the name of the folder or file (game.rar etc) and atribute the the game with the best match (assuming the user has valid game names on their folders/files, in my case I usually even add the release date and developer to the folder name I store my game at, so I guess that that information would also help with the identifying of the game by name).
Also would be nice to add games manually without the need of pointing out to a .EXE file and in additional make it possible to import/export user lists from VNDB.

for a start I'd suggest allowing games to be added without having them installed...
you'd either be supplied a link or vndb v# code, or a name...
in the case of the name you'll search vndb using it and then expand the form downwards with the results in a list view of small cards on the left and some game info to the side of it and then the user either chooses a game or changes his search and has to click a button to redo the search to remove any unnecessary calls to the api
you can then use the current library of added games in the program to scan folder names for matches with (partial) matches with the different titles of the games then search the contents of that match for an exe file and assume that's the correct one... or search for every exe in the scanning folder and then look in their paths for matches with the names...
but just as you said I'd suggest at least implementing a scan feature that scans for exe files then represents the user with 2 lists, one for the games and one for the scanned exes and the user matches some exes with some games and confirms
regardless of the approach I'm not sure about this but maybe have 3 categories and a summary of the scan that simply shows which games got matched with which paths and a list of games that didn't get matched (as text) which can be quickly read to check mismatches instead of testing each game or checking it's settings. as for the categories, one for games with no path and the another for games with path and the last one for path clashes aka. pointing to the same path (which is a child of the path category if you allow that). this will allow importing a user list from vndb (assuming that's implemented) and then scanning for the games which then get separated into uninstalled/installed/clashing so that the user can check/correct the scanned games easily... and that'll also allow having "uninstalled" games in one place for manually pointing to the exe file and having all the "installed" games in one place to look for a game to play next
regarding that summary... maybe you can omit path clashes from it since it's useless info because of the category...


lowercase zone info

Do like Knot, lowercase it all, only the question section matters in this case.
https://lists.nic.cz/pipermail/knot-dns-users/2016-April/000825.html
if done it already simplifies some code.

Done in #124 for middleware/file for middleware/etcd I have no control on what is put in etcd.


New design system

https://www.figma.com/file/zt7aTQ5wuXycIwxq5oAmF9/Wallet--Refresh?node-id=2930%3A0

[ ] Design Kit

[x] colors, fonts, Buttons (#3691)
[ ] Avatars
[ ] Inputs fields
[ ] Top bar elements
[ ] Banners and Toast
[ ] Messaging
[ ] Dialogs
[ ] Custom Keyboard
[ ] Lists (General, Transactions / People, Settings, Receipt Layout)

[ ] Account Key (#3660)
[ ] Gold (#3659)
[ ] Home

[ ] Home Screen
[ ] Notifications
[ ] Transactions drilldowns
[ ] Scanning
[ ] Send/Request flow
[ ] Secure Send flow

[ ] Add and Withdraw
[ ] Settings
[ ] Help
[ ] Invite
[ ] Onboarding flow
[ ] Navigation (react-navigation v5, drawer menu)

This is a good start, we should just be aware that there will be things in progress and that we might complete part of these

I've detailed some bullets and added who is working on it right now notices.

Are we ready to close this ? @jeanregisser


Exam: Roman Venediktov

MatrixIO was partially generated by robots
Used:

Mutability
Dsl

How good were robots at generating the code?)
Ok

Actually, just one request to o1 per function. (Plus wrapping in maybe by hand)


gradLoss from nn.criterion

When I was running the code, I found that the sign of "gradLoss" from nn.criterion (main.lua, line 100) needs to be negated.
Otherwise, the output of criterion:backward(xHat, x) (MSE for example) is giving "xHat-x" instead of "x-xHat".

It's my fault. Because I used the Rmsprop from original Atari to update the parameters, it is expecting "x-xHat", but the actual gradient should be "xHat-x". Thanks a lot for the response.


Print out the name of the user when enabling/disabling integrations

Previous:
pagerduty :] | disabled an integration in this channel: https://…

Now:
pagerduty :] | foo disabled an integration in this channel: https://…

Handy! I didn't realize these were missing. Thanks @tfheen


Missing (any python version causes venv not be created

[tools]
python = [
   "3.12",
   "3.13-dev",
   "3.10",
]
[env]
_.python.venv = { path = "venv", create = true }

With this config, if venv does not exist and any of the Python versions specified in the config are missing, the resulting error and behavior when cding to the dir is (using 3.10 missing as the example)
mise no venv found at: .../venv

To create a virtualenv manually, run:
python -m venv .../venv
mise missing<EMAIL_ADDRESS>
In combination with #1897, this seems a bit odd given that the missing version here would not even be the version with which the venv would be created.
Further, when not cd'ing to the dir but just hitting enter in it (bash integration), the error message is the same except the last mise missing: ... line is not output. This got me quite confused because it was the scenario where I hit the issue in, and I did not know that one of the python versions was missing.
Maybe either output the "mise missing: ..." also when just hitting the enter and not cd'ing, and/or not bother with missing python versions regarding venv creation in case the venv would not be created or activated with the missing version?
mise version 2024.4.4 linux-x64 (37b44e2 2024-04-11)

Agree, this also blocks a normal ls command so it is a weird behavior.

The "no venv" warning

this also blocks a normal ls command

It doesn't block anything, it just print the warning before every shell prompt inside the directory, which I think is a really good thing.

Further, when not cd'ing to the dir but just hitting enter in it (bash integration), the error message is the same except the last mise missing: ... line is not output. This got me quite confused because it was the scenario where I hit the issue in, and I did not know that one of the python versions was missing.

I think this is a great point and should be addressed.
Today the warning is fine when create = false or create is not set. But when create = true is set, it should have better instructions on what is wrong and what should be done to resolve it. My proposal is:
mise WARN  no venv found at: .../venv
mise will automatically create the venv once all requested python versions are installed.
To install the missing python tools and create the venv, please run:
mise install

I don't think we should change the behaviour of the existing mise missing<EMAIL_ADDRESS>warning because thats part of the status.missing_tools setting.
Only block venv creation when missing the required python version.

In combination with https://github.com/jdx/mise/issues/1897, this seems a bit odd given that the missing version here would not even be the version with which the venv would be created.

I don't think its a big deal that mise checks all python versions. Perhaps mise could check if the required version (first or env._.python.venv.python) is missing, but that might be a little tricky because it currently iterates over the list of missing tools and checks if any are python.
if ts
    .list_missing_versions()
    .iter()
    .any(|tv| tv.ba().tool_name == "python")
{
    // ...
}

So it would need a bit of rework. So I would say this is a nice to have enhancement, but not a bug.
Next steps

I'll create a PR for new warning when create is true
I'll create a new issue to only check the required python version

PR for new warning: https://github.com/jdx/mise/pull/3573

I think we can close this ticket now. Warning is merged and the enhancement is split into a new ticket.


chores(doc): update self-hosted deploying

Hey there @Skn0tt, based on our earlier conversation yesterday, I am sending this PR to see if we can improve a little bit the documentation for people who would like to deploy their own server instance.
Feel free to change anything, I'd rather you or somebody else double-check everything to see if I missed something (especially the Docker section, since I haven't used it) or if you'd rather rephrase things in a different way.
Cheers,

Hi @lopermo! This PR is very welcome, and it already looks very good :) I'll take a closer look next week.

@allcontributors add @lopermo for docs

Coverage decreased (-3.6%) to 66.742% when pulling f3c61833d3ec6616404bcfd996c19788f923d7cc on lopermo:main into 517d614436bbbaa0eb0e2df28223bf0dfaa5ad24 on quirrel-dev:main.


Support for Proxmox VE 7.0

have you seen this one?
https://github.com/tteck/Proxmox/blob/main/misc/post-pve-install.sh#L100-L106


Setup with granian and python3.13

Hi @gi0baro ,
I have an error with python 3.13
2.990   × Preparing metadata (pyproject.toml) did not run successfully.
2.990   │ exit code: 1
2.990   ╰─> [13 lines of output]
2.990       error: failed to run `rustc` to learn about target-specific information
2.990
2.990       Caused by:
2.990         process didn't exit successfully: `/root/.rustup/toolchains/nightly-aarch64-unknown-linux-gnu/bin/rustc - --crate-name ___ --print=file-names -C --crate-type bin --crate-type rlib --crate-type dylib --crate-type cdylib --crate-type staticlib --crate-type proc-macro --print=sysroot --print=split-debuginfo --print=crate-name --print=cfg` (exit status: 1)
2.990         --- stderr
2.990         error: unknown codegen option: `--crate-type`
2.990
2.990       💥 maturin failed
2.990         Caused by: Cargo metadata failed. Does your crate compile with `cargo build`?
2.990         Caused by: `cargo metadata` exited with an error:
2.990       Error running maturin: Command '['maturin', 'pep517', 'write-dist-info', '--metadata-directory', '/tmp/pip-modern-metadata-9kfw5xiz', '--interpreter', '/usr/local/bin/python3.13']' returned non-zero exit status 1.
2.990       Checking for Rust toolchain....
2.990       Running `maturin pep517 write-dist-info --metadata-directory /tmp/pip-modern-metadata-9kfw5xiz --interpreter /usr/local/bin/python3.13`
2.990       [end of output]
2.990
2.990   note: This error originates from a subprocess, and is likely not a problem with pip.
3.193 error: metadata-generation-failed
3.193
3.193 × Encountered error while generating package metadata.
3.193 ╰─> See above for output.
3.193
3.193 note: This is an issue with the package mentioned above, not pip.
3.193 hint: See above for details.

I use the following dockerfile
 FROM python:3.13-slim

WORKDIR /usr/src/app

  ARG DEBIAN_FRONTEND=noninteractive
  RUN apt-get -y update

    RUN apt-get -y install libffi-dev
    RUN apt-get -y install libssl-dev
    RUN apt-get -y install curl

  COPY server.py server.py
  COPY pyproject.toml pyproject.toml

  RUN curl https://sh.rustup.rs > init.sh
  RUN sh init.sh -y
  RUN find $HOME/.cargo/bin -type f -exec install {} /usr/local/bin \;
  RUN rustup default nightly

  ENV RUSTFLAGS=-C target-feature=-crt-static

RUN pip install .

CMD emmett serve --host <IP_ADDRESS> --port 3000 --workers $(nproc)

The codebase is https://github.com/the-benchmarker/web-frameworks/blob/master/python/emmett/server.py
this is working fine with 3.12
any idea ?
Regards,

🤔 I don't get why is trying to compile anything whereas all the dependencies already have wheels for 3.13..
Which dependency is failing? Granian or emmett-core?

Probably due to the arch, I'm on mac m1 (arm based)

Arm64 wheels are published both for Macos and Linux though..

any idea on how to debug this ?

@waghanza tried locally, seems an issue with pendulum. I suggest to open up an issue in the relevant repository https://github.com/sdispater/pendulum
You can see it yourself trying to install emmett55 which has the same dependencies but pendulum.

Seems that pendulum does not support 3.13 yet, https://github.com/sdispater/pendulum/issues/844


loading libcrypto in unsafe way

My MacBook Pro (Intel 2017) gives me this error
.rbenv/versions/3.0.2/bin/ruby is loading libcrypto in an unsafe way
zsh: abort      rails db:create
I have tried with rvm, asdf and system ruby, and changed openssl version from 3 to 1.1, but not able to resolve this error.

found solution - just update eth gem


Innerloop: Add RadioButtons control dependency to NavigationView

Description
As discussed in https://github.com/microsoft/microsoft-ui-xaml/issues/3733, this PR adds a RadioButtons control dependency to the NavigationView's innerloop build process.
Motivation and Context
Closes https://github.com/microsoft/microsoft-ui-xaml/issues/3733
How Has This Been Tested?
Tested manually by building the NavigationView in the innerloop solution. RadioButtons control is recognized correctly in the test UI.

Is there a reason you checked .sln changes in?

@chingucoding These updates are automatically made by VS when using a "clean" innerloop (so no additional feature areas selected to be built). When switching local branches yesterday and updating my local master branch to this repository's master branch, I had tp first commit these local innerloop sln changes (as otherwise VS complained that I cannot merge because I have local chnges not yet committed). Trying to undo the local sln changes does not work as these changes happen automatically by VS each time I open the innerloop sln. As such, I first had to commit these changes to my local master branch in order to be able to bring it up-to-date with the repo.
As such, you are seeing these changes show up here as part of this PR since it is based on my local master branch, yet they are not required for this PR at all.

Why not instead revert the changes to the innerloop.sln file before merging? It feels like VS will always make changes to the innerloop, commiting them doesn't make much sense. Ignoring innerloop.sln changes is also outlined in the docs righ above this section.

I think the underlying issue here is that some PRs commit required changes to the sln file (for example a new project was added to the solution) yet at the time the innerloop solution was not re-opened with a clean innerloop file - causing these additional lines to be included as well. For this reason, when I am making such changes (like adding a missing API test project) I always reload the innerloop solution with a clean InnerLoopAreas file and only then commit the changes to the sln file. This seems to work as all unrequired changes are automatically removed by VS and only the actually relevant lines are included in the sln file.
These changes you are seeing here will show up the next time someone modifies the innerloop sln file (for example by adding a project) so it's not like they will only be part of this PR and if I revert them, they will never show up in the future again. However, if I do revert them, I will have to deal with uncommitted innerloop sln file changes again, if I'm not mistaken? And those are an unnecessary pain to work with as outlined above. So...seeing as these changes are not harmful at all (PipsPager, PagerControl,... are all present and correct in the innerloop solution), yet I can now again merge and switch branches just fine, I would welcome it if these automatic changes could be included in this PR here.
GeneralIy speaking, I think we just have to take greater care when reviewing PRs which modify the innerloop sln file that the sln file only contains changes which are still present when using a clean InnerLoopAreas file. If we do that, I am cautiously optimistic changes of this nature will not show up and cause inconveniences any longer.

The last time I worked with the innerloop, when switching feature areas, it would start modifying the file regardless if I commited the innerloop.sln changes or not. So either way you would need to deal with changes to that file since you won't open the innerloop without any feature areas selected. But that might have changed now, not sure if that's still present. Also, assuming the most recent changes to the innerloop file were made as part of adding a new control, the innerloop file almost certainly was opened by visual studio as the add project script opens the innerloop file.
I think we can check in those changes, but next time, it would be better to also mention them in the PR and not "sneak" them in.

/azp run

@Felix-Dev If you have verified the interloop solution that you currently have works, then it sounds fine to check that in. I agree  that dealing with VS automatically changing these sln files is a pain. You can run those commands in the future if you want to avoid your PRs including this unrelated file.

The last time I worked with the innerloop, when switching feature areas, it would start modifying the file regardless if I commited the innerloop.sln changes or not. So either way you would need to deal with changes to that file since you won't open the innerloop without any feature areas selected. But that might have changed now, not sure if that's still present. Also, assuming the most recent changes to the innerloop file were made as part of adding a new control, the innerloop file almost certainly was opened by visual studio as the add project script opens the innerloop file.

That is correct, which is why I suggested in the future that such PRs reopen the innerloop sln file with a clean InnerLoopAreas file so that such temporary changes to the sln file will not be committed and as a result cause problems with local branch management.
Observe the following: Adding the PipsPager control feature area to the InnerLoopsArea file causes the following lines to be added to the innerloop sln file:
dev\PipsPager\PipsPager.vcxitems*{ad0c90b0-4845-4d4b-88f1-86f653f8171b}*SharedItemsImports = 4
dev\Repeater\Repeater.vcxitems*{ad0c90b0-4845-4d4b-88f1-86f653f8171b}*SharedItemsImports = 4

dev\PipsPager\APITests\PipsPager_APITests.projitems*{dedc1e4f-cfa5-4443-83eb-e79d425df7e7}*SharedItemsImports = 4
dev\PipsPager\TestUI\PipsPager_TestUI.projitems*{dedc1e4f-cfa5-4443-83eb-e79d425df7e7}*SharedItemsImports = 4

dev\PipsPager\APITests\PipsPager_APITests.projitems*{fbc396f5-26dd-4ca3-981e-c7bc9fea4546}*SharedItemsImports = 4
dev\PipsPager\TestUI\PipsPager_TestUI.projitems*{fbc396f5-26dd-4ca3-981e-c7bc9fea4546}*SharedItemsImports = 4

Now compare that with the lines removed as part of loading the innerloop solution with a clean InnerLoopAreas file:
dev\PipsPager\PipsPager.vcxitems*{ad0c90b0-4845-4d4b-88f1-86f653f8171b}*SharedItemsImports = 4
dev\Repeater\Repeater.vcxitems*{ad0c90b0-4845-4d4b-88f1-86f653f8171b}*SharedItemsImports = 4

dev\PipsPager\APITests\PipsPager_APITests.projitems*{dedc1e4f-cfa5-4443-83eb-e79d425df7e7}*SharedItemsImports = 4
dev\PipsPager\TestUI\PipsPager_TestUI.projitems*{dedc1e4f-cfa5-4443-83eb-e79d425df7e7}*SharedItemsImports = 4

dev\PipsPager\APITests\PipsPager_APITests.projitems*{fbc396f5-26dd-4ca3-981e-c7bc9fea4546}*SharedItemsImports = 4
dev\PipsPager\TestUI\PipsPager_TestUI.projitems*{fbc396f5-26dd-4ca3-981e-c7bc9fea4546}*SharedItemsImports = 4

What do we notice? They match, which is why I believe PRs are accidentally adding these lines at a time they have not re-opened the innerloop solution with a clean InnerLoopAreas file. If we make sure that future PRs modifying the innerloop sln file only contain innerloop sln changes which are still present after reloading the innerloop solution with a clean InnerLoopAreas file then I believe we won't see these future troubles occuring again.
I think it makes sense to call this out in the developer guide. Sometimes, changes to the innerloop sln file are unavoidable, such as when adding a new control like the PipsPager, but we should make sure that only the necessary lines are included in the updated sln file. By re-opening the innerloop solution with a clean InnerLoopAreas file before committing the changes made to the innerloop sln file, we make sure that all temporarily added lines are not included any longer. This additional step is a simple one yet, at least for me, is much appreciated as it will make working on WInUI in the innerloop solution quite a bit simpler.
@chingucoding @ranjeshj FYI.

An even better solution would be to know why exactly the innerloop solution keeps changing. If we knew why that happens, it would of course be best to eliminate the reason. In the meantime though, I think it would be best to just point this out in the documentation to make our lives a bit easier here.

An even better solution would be to know why exactly the innerloop solution keeps changing. If we knew why that happens, it would of course be best to eliminate the reason.

@chingucoding I believe the changes keep happening because we do not yet re-load the innerloop solution with a clean InnerLoopsArea file before we make the final innerloop .sln file commit. The lines highlighted directly above (the removed dev\PipsPager... lines in the .sln file) are not included by default in the innerloop .sln file to provide a working innerloop experience. They only show up when we include the PipsPager control (feature area) in the innerloop build process.
So...based on all these observations, it is my understanding that when the PipsPager control (or other controls) were added to the solution, the control was, at some point, added to the InnerLoopAreas file to be included in the innerloop build process. Was that done manually or automatically by VS? I am not sure on that since I have yet to work with the script which automatically add new projects to the solution. But I am convinced that, as outlined above, we can easily remove these temporary additions to the innerloop sln file by re-loading the innerloop solution with a clean InnerLoopAreas file.
I am not sure how feasible something like a "innerloop cleanup" script would be which would have to be be run before committing changes which contain necessary innerloop sln changes (such as project additions), but one more manual step we can do is to add a note to the InnerLoopAreas file stating roughly "if required changes were made to the innerloop sln file (such as adding a project), then please re-open the innerloop solution without any feature area selected and only then commit changes made to the innerloop sln file."
If we could somehow automate this process, that would be even better sinc there's always a chance we might miss out on removing these lines if we have to manually do the innerloop cleanup/review innerloop .sln changes. And once such temporary lines managed to slip through, we will VS removing them automatically again in local builds, which could cause some (minor) inconveniences again.

There are multiple things here so let me split it up:

Why were they included in the first place?

I think the issue was that (for my part with PagerControl) the innerloop builds contained the PagerControl and such added the files. So this was the reason why those at least entered the innerloop.

Does the add control script care?
I would say that the "add control script" does the same as opening the solution yourself. Of course the thing here is that if you add a new control and immediately commit the changes, theoretically it should add the "temporary" entries as those feature areas did not exist before calling the script and as such weren't included.

Can we automagically clean innerloop files?
In theory we could have a script call git reset on those two files so that developers don't have to bother with that. Given that you need git anyway to be able to contribute, this isn't an unreasonable thing to request to exist on developers machines.

Stop temporary entries
That is what I hope we would do soon, those temporary entries are not great to work with. Unfortunately, I have no idea why those exist in the first place or if we could something in the build process to circumvent that. Maybe someone from the team @llongley or @kmahone knows more what we could do.

There are multiple things here so let me split it up:

Why were they included in the first place?

I think the issue was that (for my part with PagerControl) the innerloop builds contained the PagerControl and such added the files. So this was the reason why those at least entered the innerloop.

Does the add control script care?
I would say that the "add control script" does the same as opening the solution yourself. Of course the thing here is that if you add a new control and immediately commit the changes, theoretically it should add the "temporary" entries as those feature areas did not exist before calling the script and as such weren't included.
Can we automagically clean innerloop files?
In theory we could have a script call git reset on those two files so that developers don't have to bother with that. Given that you need git anyway to be able to contribute, this isn't an unreasonable thing to request to exist on developers machines.
Stop temporary entries
That is what I hope we would do soon, those temporary entries are not great to work with. Unfortunately, I have no idea why those exist in the first place or if we could something in the build process to circumvent that. Maybe someone from the team @llongley or @kmahone knows more what we could do.

I don't fully understand why VS keeps updating this file sometimes. Even when there are no changes, I notice that VS likes to move things around unfortunately. What if we just add this file to git ignore ?

Wouldn't that result in the file not being there when someone clones the repository?

Also, sometimes we do have to make valid changes to the innerloop sln files (such as when adding new projects). Wouldn't adding this file to gitignore mean these necessary changes then wouldn't show up on devices of other contributors (like missing projects)? Developers could locally add these missing projects whenever they need to work on them but that sounds like additional unnecessary work we would burden contributors with 🤔


Stride.Launcher - Could not find a part of the path

Release Type: GitHub
Version: master a265571
Platform(s): Windows
Describe the bug
When I launch Stride.Launcher from the code I get this error:
System.IO.DirectoryNotFoundException: 'Could not find a part of the path 'C:\Users\Vacla.nuget\packages\stride.gamestudio\<IP_ADDRESS>9-beta\tools'.'
I can ignore it and the Luancher is working normally.
To Reproduce
Steps to reproduce the behavior:

Clone master branch
Open Stride.Launcher.sln in VS 2022
Rebuild
Run Stride.Launcher

The same issue happened also when I used VS 2019 a few days ago..
Expected behavior
No error.
Screenshots

The location doesn't contain tools

This error message doesn't show up with master-4.0 branch.

I found the exception still happens in the master branch but is swallowed.
Per the fix for #1139, the executables were moved to the tools directory on September 1, 2021 but the current 4.1 package, <IP_ADDRESS>9-beta, was uploaded on May 10, 2021 so does not have a tools directory.
I'd suggest a minimal change would be to make the code @VaclavElias shows in the first post more robust to path changes by checking for the existence of each target directory before trying to delve into it.
I'm happy to make that change if this makes sense.

would be great if you could make a PR and link this issue in it. thanks for looking into it.

Thank you @jrinker03. I wasn't able to fix it.


IOS 3D Copy on Image and Paste into TextInput does not fire onPaste Callback

When copying and pasting a local file, the component works fine, but when doing a 3D copy and pasting, IOS Seems to only copy the url to the file which is not detected by the component since its not in the format of the PastedImage
Would be nice to add support for url's beginning with file:/// for IOS when pasted into the text input.
With axios call we can get a blob from the url but currently the onPaste callback is not fired with such paste.
This is a feature request

Hi @Rutvik17 thanks for the report, wiuld you be willing to try and submit a PR for this?

Hi @Rutvik17 thanks for the report, wiuld you be willing to try and submit a PR for this?

I am not sufficient with Swift but i could take a look.

I am not sufficient with Swift so i found a work around on changeText callback on TextInput
if (value.includes('file:///')) {
            const strings = value.split(' ');
            for (let i = 0; i < strings.length; i++) {
                if (strings[i].startsWith('file:///')) {
                    try {
                        const axiosResponse = await axios.get(strings[i], {responseType: 'blob'});
                        // Handle Image Blob
                        strings.splice(i, 1);
                    } catch (e) {
                        console.log(e.message);
                    }
              }
         }
    value = strings.join(' ');
}

But would be nice if your team can add the support in native.

will leave it open and see if someone picks it up ;)


[Refactor]: Refactor the evaluation directory

What problem or use case are you trying to solve?
Right now in the evaluation directory, the directory structure is very flat, and it is hard to tell which subdirectories are utilities related to implementing benchmarks or doing basic tests for openhands (utils, integration_tests, regression, static), and which are actual benchmarks from the ML literature (everything else).
To make this more clear, we can move all benchmarks to live under the evaluation/benchmarks/ directory. In addition, all other files that have to do with evaluation (including documentation, github workflows, etc.) will need to be checked and changed to maintain consistency.
While we do this, we can also add some of the benchmarks that are missing from the evaluation/README.md documentation.

Sorry, to clarify, this was me accidentally logged in as the openhands-agent account...


Added multiplayer support for Example Laser

Uhm what is the issue? I'm pretty sure I fixed the laser for MP here

@CXUtk, I reviewed your changes and can confirm the code works in multiplayer, however, the drawing code and the projectile itself are botched. I revise you to review your code and make changes to that it works properly again.

Code tested, works in multiplayer correctly. Merging.

@CXUtk In the off chance you're still subscribed to this thread, I'm just looking to get in touch for your approval on our license change, Can you drop on Discord or have a look at #705
Thanks


Fix the 'Binary Interface and Notification Format' url in notification doc

The Binary Interface and Notification Format url is invalid, replaced with new available url.

Coverage remained the same at 61.865% when pulling c115af6e5ac34dc07909cacd8880c635b8a833fe on williambelle:notification-doc-fix into 0bc5e746c93db792bd1b2a7c87ee79bab4de1a43 on argon:master.


LCD on bionic-minimal-pine64-bspkernel-0.7.28-136.img

Setting up an LCD in /boot/uEnv.txt doesn't seem to affect anything. HDMI screen works, official MIPI DSI LCD screen doesn't. I've checked it with Armbian KDE image -- works fine.
Doing disp_mode=disabled makes the screen actually lightup but it seems that it goes into kernel panic and instantly reboots.
Can anybody help with it?

Ok, proper install is
disp_screen0=lcd disp_screen0=screen0


flutter run --profile on iOS doesn't have profiling turned on in observatory?

I just ran flutter -d i run --profile and then I opened Observatory to look at a CPU profile. I see this:

"""
Profiling is disabled
Perhaps the profile flag has been disabled for this VM.
"""
and
Modified Flags

// Run optimizing compilation in background
background_compilation  =  false

// Ignore unrecognized flags.
ignore_unrecognized_flags  =  true

// Time between profiler samples in microseconds. Minimum 50.
profile_period  =  1000

// Enable the profiler.
profiler  =  false

flutter/engine@3a4e0a0c80c71b7cf122b3c36d59828ab222184c

Thanks @rmacnak-google !

Curious, how long until this takes effect?
I see https://github.com/flutter/engine/pull/2804 landed yesterday, but I just tried it, and still no profiling.

The change was reverted because some flows were broken. It seems XCode attaches lldb by default to simulators. The likely fix is have profiling disabled by default and have flutter run --profile explicitly enable it.

Confirmed, --profile isn't enabling CPU profiling on iOS hardware.

This seems like a serious limitation of our iOS workflow. We definitely want to be able to profile both --debug (default) and --profile apps, like we can on Android.

We'd have to verify if this was still true.  @xster or @cbracken might cross this in their iOS explorations.

Fixed by flutter/engine@327d122de9eaf3739996610a3630408bdcc52103 and flutter/flutter@4efb4282549e4b1d5379a87370d5891a1a4d746e.


Updated PNG file. eReduced number of bars and lines in collection icon, ...

...plus removed SS 1.0 graphics

Restarted the build.

Thanks! Silly Travis...

It's getting farther this time...


[BUG/FEATURE] OneDrive Support

Hello Collaborators,
Describe the bug
When using OneDrive as the WebDav server things get weird. In short: The authentication fails because of Microsoft.
Software

OS: Linux (Docker)
Golang: 1.19.4
Version: 60ec5ad560124272abec0b904f79c913c3cb347c

To Reproduce

Create a client using OneDrive credentials
Try to use any (authenticated) request (e.g. ReadDir())
Response is empty / unexpected (e.g. no files)

Expected
At least an authentication error should be returned. Even better would be support for MS's need to be proprietary.
Additional context
Microsoft (as expected) does not follow industry standards and instead of returning 401 on unauthenticated requests they return 302 to redirect you to the login page. The login page then return a 200 status so the webdav client thinks it is a valid result for the original request. On its own this would not be too bad as you could catch the redirect and handle it like a 401. But since Microsoft use their own authentication method (MS-PASS with Passport1.4) you also need a new authenticator.

Related information:

Rclone's WebDav implementation of OneDrive: https://github.com/rclone/rclone/blob/v1.61.1/backend/webdav/webdav.go#L559
Rclone's WebDav document about OneDrive: https://rclone.org/webdav/#sharepoint-online

Current workaround:

set cookie with interceptor: https://github.com/alist-org/alist/blob/v3.9.2/drivers/webdav/util.go#L18-L28
and refresh cookies every 12 hours: https://github.com/alist-org/alist/blob/v3.9.2/drivers/webdav/driver.go#L33-L39

Thanks for all the references @bai-jie!
Unfortunately, this does not work for OneDrive but only SharePoint. But it still helped a lot with getting me started.
In the end, I mostly followed MS's example for Passport1.4 though.

if I´d not get it wrong. we could check for a redirect  or compare the request and response with the path in Veriify of nullAuth then do the partner server auth to obtain the credentials. we could move the code into the Authorize block and make a kind of preflight request. everything should happen in the nullAuth which is going to be replaced by a real authenticator.


Adds action icons for download import and delete.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
606

Relevant Lines:
786

💛  - Coveralls


javac should show errors and warnings of current buffer only

I'm quite happy with both javac and pmd as linters so far. My workflow got much faster in comparison to when I was using Eclipse. However, there is one little thing that annoys me about javac as linter: It shows errors and warnings from the entire project instead of the current file/buffer only.
When working on a class calling :lopen I am only interested in errors and warnings related to the file I am currently editing. From my point of view it makes no sense to show errors and warnings of the entire project. If I want to lint the entire project, I could still do that by running the appropriate command in another terminal.
I'd call myself a beginner in Java. So I looked around for possible arguments I could pass to javac that makes it behave as described above. While I could not find any way to change the behavior of javac, I stumbled upon the following lines
https://github.com/w0rp/ale/blob/e88243687a2da8cd02962bc78874ec0e60aecaf2/ale_linters/java/javac.vim#L90-L98
I'm not very experienced in VimL either, but wouldn't it be possible to "grep" for the lines including the buffer's file name somewhere there? What do you think about this suggestion? Or am I missing some flag to make the linter behave the way I think of?

That's by design. Your code can fail to work due to errors present in other files you import. You shouldn't see the errors indicated incorrectly in the file you're looking at, but you should see them in the location list.

Hm. While I understand your argument that if something's wrong in another file the project I work on won't compile, this is the very same for other projects written in another language using a different linter. E.g., in an Angular project where I write in TypeScript, the project won't build if there is something wrong in another file, however the location-list is only populated with errors or warnings tslint finds in the current file.
As I see it, the current behavior does not comply to the purpose of the location-list; from Vim's help (:h location-list):

A location list is a window-local quickfix list.

I also find it very impractical. My current workflow is to :ll on some line in a file with errors and I would jump to the first error of the current file. When using javac as linter though, I might end up somewhere else in the project.
Thus, I don't see any reason why the location-list should be populated with errors or warnings from other files.

I figured to map both <Plug>(ale_next_wrap) and <Plug>(ale_previous_wrap) to quickly jump between problems within the current open buffer. This at least helps me to circumvent the -- from my point of view -- odd behavior when calling :ll and ending somewhere else in the Java project.


Quick Claw and Custap Berry should override Stall

Rounding the priority also simplifies the code for Full Incense and Lagging Tail.

But what about two Custaps activating and one user has Stall?

As I recall, @Marty-D says they ignore Stall, so you can outspeed another Custap/Quick Claw user if you're naturally faster, although this is Sableye we're talking about, so ...


Applying TypeDecorator to db.Column

I am not sure whether this is a feature request or simply a problem arising from my limited understanding of how to access sqlalchemy's underlying functionality.
I would like to apply a TypeDecorator to a table column, to pre-process data that is assigned. The column is declared in the 'usual way':
class User(db.Model):
    name = db.Column(db.String)

It would be nice if that was possible through additional arguments to 'db.Column'. But maybe there is a different obvious solution...

db.Column() is literally the same as SQLAlchemy's Column(). You can even use them interchangeably.
AFAIK using type decorators always happens outside declaring a model since it basically defines a new type - and then you use that new type in your model.


[DOCFIX] HBase docs structure improvement

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/Alluxio-Pull-Request-Builder/461/
Test PASSed.

@apc999 Change to links to new format and remove the two fs.alluxio.impl and fs.AbstractFileSystem.alluxio.impl. They are not necessary in my local mode testing.
PTAL, thanks!

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/464/
Test FAILed.

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/Alluxio-Pull-Request-Builder/470/
Test PASSed.

@apc999 There was still an outstanding issue
Currently the pre-requisites imply that HBase doesn't work with Alluxio when Alluxio is deployed on Mesos, Kubernetes, Yarn, or Docker. I don't think this limitation is correct, so we should remove it

@aaudiber All the current compute framework documents contain that line:
An Alluxio cluster has been set up and is running according to either local mode or cluster mode.

Do you mean that this line implies that Compute frameworks don't work with Alluxio when Alluxio is deployed on Mesos, Kubernetes, Yarn, or Docker?

Right, the linked docs are two out of the many different deploy docs


Not Working On Ubuntu Server 20.04 for Raspberry pi 4B

RPi Model - 4B
OS - Ubuntu Server 20.04 x64

@ProgrammingPlanet I am having the same issue. What problems did you run into? Were you able to find a solution?


[5.5] Fix dueEvents docblock

Fix dueEvents docblock

Related to https://github.com/laravel/framework/pull/20096

Why are you making the other change? Just change the doc block.

Done https://github.com/laravel/framework/pull/20104/commits/874a2baebdfc9d03e3e92b489b6416ce7be51c2a
It's just that we usually don't do much foreach on a collection.


remove prop-types and warning

change also the View with the TouchableOpacity to be expand info when you click on component
place also  component wrap the {...}
Closes #10
Closes #11

change also the View with the TouchableOpacity to be expand info when you click on component
place also component wrap the {...}
Closes #10
Closes #11

This is not resolving the issue, you have removed all prop types validation


Change Log contract to LoggerInterface

Hello,
the Illuminate\Contracts\Logging\Log does not exist in 5.6,
we need to use the Psr\Log\LoggerInterface instead.
thanks.

Closing in favor of earlier PR #1624 ,Thanks!


Scale image

Hi
I need to resize an image using ffmpeg. please tell me if there is a wrapper for the following ffmpeg command?
ffmpeg -i input.jpg -vf scale=320:240 output_320x240.png
regards

The list of implemented "native" filters can be found in the ffmpeg/_filters.py source file.  (The answer is currently "no").
However, you can explicitly implement any native ffmpeg filter by passing the correct arguments to the .filter() directive.
Something like:
    freeze = (
        ffmpeg
        .input(infile)
        .filter('scale', width='320', height='240')
        .output('outfile.mp4')
    )

All that said, why are you using a video-focused tool to resize a single image?  It can be done, but it's an odd use case.  (If you really just want to resize images, look up ImageMagick...)

You are totally right. There is no other purpose but reduce dependencies and do all pre processings using ffmpeg.
Lots of thanks.


Multiple video player in LazyColumn

It would be a good feature to have multiple exo player in LazyColumn such that when one player (nearest from center of screen) is playing, others should be paused. I tried but didn't achieved 100% accuracy. I can share the code if required.

You are not alone with this problem. Many people are in this problem. A related issue is tracked here: https://issuetracker.google.com/issues/190920985
I didn't try it yet. Will will check it soon and let's see if any workaround I found will share.
Please share your code.

In release v4 I added an ExoPlayer example. Please check it. And give feedback. :)

seems great. i'll see if i can integrate it with photo + video feed.


ERROR in NgProgressModule is not an NgModule

Hi, cool module but I have a problem.
An error has started to appear after update angular-cli from .21 to 1.0.0-beta.24

"dependencies": {
"@angular/common": "2.4.1",
"@angular/compiler": "2.4.1",
"@angular/core": "2.4.1",
"@angular/forms": "2.4.1",
"@angular/http": "2.4.1",
"@angular/material": "^2.0.0-alpha.11-3",
"@angular/platform-browser": "2.4.1",
"@angular/platform-browser-dynamic": "2.4.1",
"@angular/router": "^3.3.1",
"angular-cli": "1.0.0-beta.24",
"angular2-cookie": "^1.2.6",
"core-js": "^2.4.1",
"file-saver": "^1.3.3",
"flexboxgrid": "^6.3.1",
"font-awesome": "^4.7.0",
"hammerjs": "^2.0.8",
"lodash": "^4.17.2",
"muicss": "^0.9.7",
"ng2-pagination": "^1.0.1",
"ng2-progressbar": "^1.0.3",
"ng2-translate": "^5.0.0",
"object-fit-images": "^2.5.9",
"rxjs": "^5.0.1",
"ts-helpers": "^1.1.2",
"zone.js": "0.7.4"
},
"devDependencies": {
"@angular/compiler-cli": "2.4.1",
"@types/jasmine": "^2.5.38",
"@types/lodash": "^4.14.44",
"codelyzer": "^2.0.0-beta.3",
"jasmine-core": "2.5.2",
"jasmine-spec-reporter": "2.7.0",
"karma": "1.3.0",
"karma-chrome-launcher": "^2.0.0",
"karma-cli": "^1.0.1",
"karma-jasmine": "^1.1.0",
"karma-remap-istanbul": "^0.4.0",
"protractor": "4.0.14",
"ts-node": "1.7.2",
"tslint": "^4.1.1",
"typescript": "2.0.10"
}
}

Hi @MateuszG, What is the error?

The package needs to be recompiled, by the time I push the new updates you can use the source typescript in your project

Now I have:

ERROR in ./~/ng2-progressbar/dist/index.d.ts
Module build failed: Error: Debug Failure. False expression: Output generation failed
at Object.assert (/media/mateusz/MAGAZYN/PROJECTS/ihar/frontend/node_modules/typescript/lib/typescript.js:2437:23)
at Object.transpileModule (/media/mateusz/MAGAZYN/PROJECTS/ihar/frontend/node_modules/typescript/lib/typescript.js:54677:18)
at TypeScriptFileRefactor.transpile (/media/mateusz/MAGAZYN/PROJECTS/ihar/frontend/node_modules/@ngtools/webpack/src/refactor.js:165:25)
at /media/mateusz/MAGAZYN/PROJECTS/ihar/frontend/node_modules/@ngtools/webpack/src/loader.js:151:37
@ ./src/app/app.module.ts 20:0-83
@ ./src/main.ts
@ multi main

when use import

import { NgProgressModule } from '../../node_modules/ng2-progressbar/dist/index.d';

The typescript files doesn't exist on npm, copy the src from here into your project e.g. 'temp-folder' and import the module from there like
import {NgProgressModule} from 'temp-folder'

@MateuszG The package has been updated to 1.1.1, problem should be fixed, if you get any issue feel free to reopen


Sprite Lab Pointer Blocks Shadowing Functionality

These changes will be used in an upcoming PR in the cdo repo to make the shadowing behavior work for the clicked/subject/object sprites

Codecov Report

Merging #218 into master will increase coverage by 0.02%.
The diff coverage is 21.42%.

@@            Coverage Diff             @@
##           master     #218      +/-   ##
==========================================
+ Coverage   59.68%   59.71%   +0.02%
==========================================
  Files          95       95
  Lines       11588    11594       +6
==========================================
+ Hits         6916     6923       +7
+ Misses       4672     4671       -1

Impacted Files
Coverage Δ

core/ui/fields/field_rectangular_dropdown.js
75.75% <0.00%> (-1.34%)
:arrow_down:

core/ui/block.js
74.04% <35.29%> (+0.04%)
:arrow_up:

core/ui/block_space/block_space.js
64.85% <0.00%> (+0.21%)
:arrow_up:

core/connections/connection.js
69.60% <0.00%> (+0.22%)
:arrow_up:

core/ui/block_svg/block_svg.js
84.21% <0.00%> (+0.45%)
:arrow_up:

Continue to review full report at Codecov.

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

Offline conversation: There are currently a couple of scenarios that might seem a little odd from a UI perspective, but will be addressed with some follow-up work

Future work: Clear the shadow block when it is removed from its root stack.


Doc'd return values of as_sql() for Func and query expressions.

Mostly to make it clear that params may be a list or tuple.

@adamchainz Thanks :+1:


Add location_id to front end location shapes

Checklist

[x] Appropriate branch selected (all PRs must first be merged to dev before they can be merged to master)
[x] Any modified or new methods or classes have helpful JSDoc and code is thoroughly commented
[x] The description lists all applicable issues this PR seeks to resolve
[x] The description lists any configuration setting(s) that differ from the default settings
[x] All tests and CI builds passing

Description
Regression where we wouldn't include location_id when editing and saving a polygon.

Looks great thanks! I think keeping it in the front end is fine -- better to pass it than to have to fish it out, no?

I agree. Easier to finagle on FE.


CloudFormation create_stack Fails with Long Stack Name and S3 Bucket in Template

The following unit test fails:
`
@mock_cloudformation
def test_boto3_create_stack_s3_long_name():
cf_conn = boto3.client("cloudformation", region_name="us-east-1")
stack_name = "MyLongStackName01234567890123456789012345678901234567890123456789012345678901234567890123456789012345678901234567890123456789012"

template = '{"Resources":{"HelloBucket":{"Type":"AWS::S3::Bucket"}}}'

cf_conn.create_stack(StackName=stack_name, TemplateBody=template)

cf_conn.get_template(StackName=stack_name)["TemplateBody"].should.equal(
    json.loads(template, object_pairs_hook=OrderedDict)
)

`
Moto tries to use a bucket name that's longer than the 63 char max.  CloudFormation truncates the StackName portion of the bucket name such that it doesn't exceed the limit.

This was fixed as of moto >= 1.3.15.dev1010


Problem with probing

In my G-Code file I have several M66P0L4Q5.000 commands and it works as expected. But after applying height map this command disappear everywhere. Even my custom G-Code M101 is remain but M66P0L4Q5.000 is disappear. After applying height map editing button is inactive and it's not possible to edit file and add these commands manually.
Another problem when probing with "Set Z = 0 at X0Y0:" option and negative X Y in "Area to probe". Z is not become 0 at X0Y0 in .map file. For example: Area to probe X -150, Y -150, W 300, H 300,  Grid 50mm.  Z0 will be at X-150 Y-150 but not at X0 Y0.
Or may be I am not understand how this option should work.
It's also would be nice to add possibility to edit and save G-Code file after applying height map.
My custom code visible but doesn't work after applying height map either. M101L1Q60 should send command via UART but it doesn't. Before applying height map everything works fine.

New edge versions with M66 bugfix uploaded - 2.0.45p5.

Another problem when probing with "Set Z = 0 at X0Y0:" option and negative X Y in "Area to probe". Z is not become 0 at X0Y0 in .map file.

Which version are you running?

It's also would be nice to add possibility to edit and save G-Code file after applying height map.

Save then edit works.

My custom code visible but doesn't work after applying height map either. M101L1Q60 should send command via UART but it doesn't.

I'll have to look into how to better handle arbitrary M-commands when transforming the gcode, e.g. the L word is removed in the above example.

the needle touches and scratches the surface in many places.

What kind of motion? Rapids (G0) or regular (G1, G2 or G3)?
Are the map files above generated from scanning the same surface? And by following the same pattern?

Motion is G1, normal.
Patterns are exactly same. 300x300mm, X-150 to 150, Y -150 to 150, grid size 50mm. Surface is same.
Only difference measuring direction. ioSender starts from X-150 Y-150  and than moves by Y axis (Y-100, Y-50, Y 0, etc.) and Candle starts also from X-150 Y-150 but than moves by X axis   (X-100, X-50, X 0, etc.)

Here is the hegthmap from Candle, offset to 0 for the origin.
0;-0,082;-0,136;-0,162;-0,137;-0,087;-0,037
-0,037;-0,109;-0,16;-0,187;-0,182;-0,129;-0,051
-0,013;-0,087;-0,141;-0,173;-0,172;-0,116;-0,047
-0,03;-0,102;-0,154;-0,176;-0,179;-0,13;-0,057
-0,071;-0,144;-0,195;-0,222;-0,224;-0,169;-0,106
-0,128;-0,199;-0,252;-0,276;-0,276;-0,222;-0,15
-0,133;-0,242;-0,302;-0,333;-0,336;-0,279;-0,215

As you can se there is quite a bit of difference between them suggesting to me that the probed surface might be bending when probed.  This might be due the probing speed beeing different. Or a bug somehwere causing the heighmaps to differ - it looks like ioSender is not probing the full width of the surface.
The differences are largest on the right hand side, is this where the scraping occurs?

Yes, left and right side are different height after applying a height map. I've changed a probe and made more tests. Circle 300mm. On the photo you can see difference.

Another strange behavior. I put probe to Z 1mm and start probing. And after probing is done probe returns to approx the same position but ioSender shows Z -4.xxx mm which is wrong. Not big problem because anyway I have to set new zero for Z axis with tool. But I think it should not be like this.

Is Candle probing each point twice with different feed rates like ioSender does?
Any backlash in the Z-axis?

No, Candle probing only once, but speed I use is same - 10 mm/min.  And Candle shows immediately result of each point and visualization during probing ))
I think there is very slight backslash on Z axis. But it's not so big and result I still get acceptable for my need.

I would also like to thank you for this amazing project. Helped me a lot...

BTW. Now I'm using ESP32 TinyBee board with grblHAL and would like to ask you does any ESP32 board support I2C keypad? I've tried but no luck. I2C EEPROM is working but keypad is not. Now using keypad with UART. Not sure is it right place to ask...

ioSender shows Z -4.xxx mm which is wrong.

Do you have Set Z = 0 at X0Y0 checked when probing?

No, Candle probing only once

Ok, then you may set the ioSender Latch distance to 0 to skip the slow step. If you do so are the maps more equal?

does any ESP32 board support I2C keypad?

This board has the pin definition for the keypad strobe.

Ok, then you may set the ioSender Latch distance to 0 to skip the slow step. If you do so are the maps more equal?

Yes. Now much better!!! Thank you!!!  I also hope one day ioSender will stop braking my custom G-Code )))

[This board]  has the pin definition for the keypad strobe.

Ok. I will try.  Is there definition example to use UART 3 in ESP32 boards? If I want to use MPG on UART 2 and UART keypad (input) and Debug (output) on UART 3. Is it possible? Couldn't get it to work.

I also hope one day ioSender will stop braking my custom G-Code )))

This is a bit complicated since a block (line) may contain several gcodes and some parameter words are single meaning. If the gcode is unknown (which parameter letters are asscociated with it) it becomes impossible to parse unambiguously.
I could assume that an unknown gcode "owns" all the parameter letters and just keep them (and the values) in a string and pass them unchanged through the transform routines and I may do this later.

Is there definition example to use UART 3 in ESP32 boards?

Not yet? You have to use instance number 2 to claim it - but it cannot be claimend by two "clients" unless you modify the code by commenting out or removing this line.

I could assume that an unknown gcode ...

Is it necessary to apply height map to M-Codes? Why not parse only G-code and leave M-codes as they are?

Not yet? You have to use instance number 2 ...

Thank you so much!!! With your help everything works now. I2C, Serial2, Serial3...
In the file "uart_serial.h" line 132 there is missing "," after "}" but error come out only if  "SERIAL3_ENABLE" is defined.
I guess no one else accept me needed Serial3 )))
Keypad plugin also contain display plugin but there is no description. Is there any info or examples? Will it work with ESP32 boards?
I tried to compile with this plugin enabled but got errors:
"msg_type_t too large for I2C display interface"
"machine_state_t too large for I2C display interface"
"coord_system_id_t too large for I2C display interface"

Why not parse only G-code and leave M-codes as they are?

I'll have to leave unknown M- and G-codes alone, and that can be tricky as explained above.
BTW gcode transforms other than the heightmap transform also uses the parsed/tokenized gcode, and I am not going to write a separate parser just for the heightmap.

In the file "uart_serial.h" line 132 there is missing ","

I'll fix this in the next commit.

Keypad plugin also contain display plugin but there is no description. Is there any info or examples?

It is for the Jog2K Jog Controller. Or for MPGs or displays that uses the grblHAL message parser.

Will it work with ESP32 boards?

No - at least not yet. The ESP compiler do not understand the assert statements and currrently the I2C driver code does not support the required API functions. So a bit more code, and testing, is required to make it work.