Nissan Leaf component stops updating

Home Assistant release with the issue:

0.94.4
Last working Home Assistant release (if known):
Operating environment (Hass.io/Docker/Windows/etc.):

Ubuntu 18.04.2 LTS Hassio
Component/platform:

https://www.home-assistant.io/components/nissan_leaf/
Description of problem:
Updates have worked fine from the car after a restart of Home Assistant, but then after a while it stops updating. Looking at the entities they say update in progress = true. Looking at all the entities for the component right now, they all says updated 4 days ago and update in progress = true. Restarts have fixed the issue before, but now I've tried to restart Home Assistant multiple times without getting updates. Calling the nissan_update service updates the next update time on the entities, but nothing happens after that, still (update in progress = true) .Tried both the Nissan EV app and Leaf manager app, they both update fine.
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):

Traceback (if applicable):

Additional information:

I just restarted my Home Assistant twice yesterday, and once more this morning, but the entities are not updating. Restarts have fixed the issue earlier. I can only see update in progress = true. I also tried to call the update service, and I can see that the update in progress updates on the entities, but nothing happens after that. Tried both the Nissan EV app and Leaf Manager app, and those update just fine.

I can find this in the log.
2019-06-25 19:56:32 WARNING (SyncWorker_5) [homeassistant.components.nissan_leaf] WARNING: This may poll your Leaf too often, and drain the 12V battery.  If you drain your cars 12V battery it WILL NOT START as the drive train battery won't connect. Don't set the intervals too low. Traceback (most recent call last):   File "/usr/src/homeassistant/homeassistant/components/nissan_leaf/__init__.py", line 226, in async_update_data     await self.async_refresh_data(now)   File "/usr/src/homeassistant/homeassistant/components/nissan_leaf/__init__.py", line 276, in async_refresh_data     server_response = await self.async_get_battery()   File "/usr/src/homeassistant/homeassistant/components/nissan_leaf/__init__.py", line 352, in async_get_battery     self.leaf.get_latest_battery_status)   File "/usr/local/lib/python3.7/concurrent/futures/thread.py", line 57, in run     result = self.fn(*self.args, **self.kwargs)   File "/usr/local/lib/python3.7/site-packages/pycarwings2/pycarwings2.py", line 396, in get_latest_battery_status     return CarwingsLatestBatteryStatusResponse(response)   File "/usr/local/lib/python3.7/site-packages/pycarwings2/responses.py", line 596, in __init__     bs = recs["BatteryStatus"] TypeError: list indices must be integers or slices, not str

I just sent an update from the Nissan EV app, got updates from the car, then restarted Home Assistant. Now the values have updated in Home Assistant.
Im pretty sure the issue hasn't been fixed, as it will freeze up in Home Assistant again after some hours +-day or two.

Thanks fo raising this. I'm facing the same issue.

I got this fixed for me by adding the device_tracker entry for my car. I had automatically track new devices turned off because I kept getting new entries for guests on my wifi network, but this also blocked the addition of my Leaf to the device_trackers. From the moment I set my car to tracked the integration hasn't stopped working. So it apparently couldn't handle not having the device_tracker in place at some point.
Now it's also stopped working but that's because Nissan apparently changed their API, at least for European users...

That is awesome @Qonstrukt! I have also turned off track new devices, and will try to set my car to tracked and see if it resolves the problem. Can't really check if its working before the API issue is fixed though. Will close this issue when/if tested and working. Thanks again.

On a second note @Qonstrukt . I have a pre 2015 Nissan Leaf without tracking. So nissan_connect is set to false as the documentation states. This won't generate a device tracker for my car. Any suggestion?

Ah, then no, I have no clue. I have a 2018 Leaf 2.0.

I have just updated the library that the nissan leaf uses, pycarwings to version 2.9 to fix the login issue.  It has changed the API again, so I'm updating home assistant now to match.  Apologies for the delay.


Questions on run time of Hsg, normalization, and reason for adaptation

Hi @thackl
From the paper, the run time of proovread on Hsg dataset is 2288d and 1009 for proovread def and proovread norm respectively. It confused me that why did you conduct the experiment for so long time. How do you calculate the run time?
Since the runtime of proovread norm on Hsg is about half of proovread def. I suppose the normalization of Hsg SRs is to sample the SRs in order to reduce the total data size with even coverage. Is my understanding correct?
As for Hst dataset, since the throughput of proovread norm is much smaller than proovread adapt, while the run time is much longer. Why adaptation causes the result change? Why use adaptation version of proovread for trancriptome dataset?
Best
Weilong

The runtime is the theoretical value for single core runtime. Of course, we did not wait 5 years, but we distributed and parallelized the jobs on a cluster, but used single core runtime as a comparable runtime parameter.
Regarding normalization, this is not a sampling with even coverage - proovread uses sampling internally and samples in each run, but normalization is rather the opposite. Based on kmer counts, you identify repetitive reads and reduce the number of these reads, assuming they carry redundant information (http://ivory.idyll.org/blog/what-is-diginorm.html).
About Hst, you must have misread the table. We did not run proovread norm on the Hst set, only default and adapted, and adapted increased throughput from 48% to 70%. This is because the adapted run handled low coverage transcripts much better, which in genomic sets are not expected and are often lost with default settings.
Nevertheless, you should not read to much into the statistics provided in the paper.  We did these computations at the end of 2013/early 2014. Since then, a lot has changed in proovread, in particular, we changed the mapper from SHRiMP2 to BWA MEM. The current version is much faster and also shows some different behaviour with respect to chunk sizes, repeats, etc...
Cheers
Thomas

Thank you for the detailed answer!


Revert "enhancement filter empty partition before export table to remote storage"

Reverts apache/doris#35102

run buildall


C# support in VS??

There is no C# support in VS 2019 by default. It does not come with C# or anything .NET preinstalled. I have no idea what packages to install to create a C# project in VS. This should be explained in any VS-C# tutorial... but sadly nobody does, people expect a noob programmer who's never used VS, to be able to understand the infinite complexity of VS dependencies. I tried installing VS packages before, with no luck. It's far from trivial. Moreover, it would be better if VS itself had better support for one of the most popular programming languages out there. Basically, if I want to code in C#, the only way I can do it is by installing Unity. Which is a game engine and it's not designed for general purpose C# programming.. so on.

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

ID: f783f152-b26c-0ea4-8017-3929c7465368
Version Independent ID: 8777c084-6578-8a7f-89db-80b3f9d547a7
Content: Tutorial: Create a simple C# console app - Visual Studio
Content Source: docs/get-started/csharp/tutorial-console.md
Product: visual-studio-windows
Technology: vs-ide-general
GitHub Login: @TerryGLee
Microsoft Alias: tglee

@v-i-m The tutorial includes an instruction in the Create a project section that asks you to install the .NET Core cross-platform development workload. Installing a workload by way of the Visual Studio Installer is the easist way to get the programming language support that you are looking for. You can find more information about how to install workloads, here: Install Visual Studio. You can also find out more about the workloads themselves, here: https://visualstudio.microsoft.com/vs/visual-studio-workloads/
However, if you are looking for a different experience other than what the product provides, please use our “Suggest a feature” option in the Visual Studio Developer Community to tell us more about it. We’d be happy to have this on record, where others can view it and upvote it, too. For more information about this works in the dev community, see our Suggest a feature page.

Thank you for replying.
The Install VS feature is really well hidden. Basically.. yo have to
remember the magic words: Install Visual Studio. To access the elusive IVS.
Which opens up a scary UI that tries to convince you to install
mind-bendingly-gigantic packages, and you have no idea which one. Maybe
all? Yeah, sure, if my C drive was 100 times larger. I have failed to
install prerequisites for Unreal, in the past, despite following their
instructions to the letter. For C#.. I only installed .NET, which made no
sense, and I found out about it because of YouTube, not because of VS
explaining or, better, automatically picking the correct packages to
install.
My point being, your interface is hard to understand and not intuitive at
all, doesn't try to help the user. You hide the Install VS button, which I
think should be one of the Main buttons, always visible in the VS startup.
Like an.. Add-Remove features button. Maybe it's all designed for the
hardcore programmer [who loves problem solving more so than the average
human?..], and to weed out the softies.
Sorry for replying here, I don't really usually use VS, and I'm not a *real
*programmer. : D
Cheers!
VM
On Mon, Oct 7, 2019 at 7:35 PM Terry G. Lee<EMAIL_ADDRESS>wrote:

@v-i-m https://github.com/v-i-m The tutorial includes an instruction in
the Create a project
https://docs.microsoft.com/en-us/visualstudio/get-started/csharp/tutorial-console?view=vs-2019#create-a-project
section that asks you to install the .NET Core cross-platform
development workload. Installing a workload by way of the Visual Studio
Installer is the easist way to get the programming language support that
you are looking for. You can find more information about how to install
workloads, here: Install Visual Studio
https://docs.microsoft.com/en-us/visualstudio/install/install-visual-studio?view=vs-2019.
You can also find out more about the workloads themselves, here:
https://visualstudio.microsoft.com/vs/visual-studio-workloads/
However, if you are looking for a different experience other than what the
product provides, please use our “Suggest a feature” option in the Visual
Studio Developer Community https://developercommunity.visualstudio.com/
to tell us more about it. We’d be happy to have this on record, where
others can view it and upvote it, too. For more information about this
works in the dev community, see our Suggest a feature
https://docs.microsoft.com/visualstudio/ide/suggest-a-feature page.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/visualstudio-docs/issues/4059?email_source=notifications&email_token=AMOTSTVUSSSNXO6IMKCKPA3QNNXMHA5CNFSM4I54EWZKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEARFTXA#issuecomment-539122140,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMOTSTXUBKI7TZCZXTSKYOTQNNXMHANCNFSM4I54EWZA
.

--
Virgil Mihailescu
www.virgil.weebly.com

@v-i-m You bring up some good points. However, I think they fall into the realm of product feedback rather than documentation feedback. If you are looking for functionality that isn’t currently provided by the product, please use our “Suggest a feature” option in the Visual Studio Developer Community. We’d be happy to have this on record, where others can view it and upvote it, too. For more information about this works in the dev community, see our Suggest a feature page.


cordova: Error: spawn node_modules/cordova/bin/cordova [param] ENOENT

Is there any permission setting I should add?
It keeps throwing an error each time I try to run a cordova command such as $gulp --cordova "platform ls".
[13:52:01] Using gulpfile ~/m-ionic/gulpfile.js
[13:52:01] Starting 'default'...
[13:52:01] Starting 'cordova'...
[13:52:01] Finished 'default' after 22 ms
events.js:154
      throw er; // Unhandled 'error' event
      ^

Error: spawn node_modules/cordova/bin/cordova platform ls ENOENT
    at exports._errnoException (util.js:856:11)
    at Process.ChildProcess._handle.onexit (internal/child_process.js:178:32)
    at onErrorNT (internal/child_process.js:344:16)
    at nextTickCallbackWith2Args (node.js:475:9)
    at process._tickCallback (node.js:389:17)

Node: v5.6.0
Cordova: v6.5.0
Gulp:  v3.9.1

I found a temporary solution in case somebody is having the same issue.
Rolled back "gulp-shell" to version 0.5.2 as the 0.6.1 is causing the issue when passing any parameter to shell commands.
package.json

"gulp-shell": "^0.5.1",

Hi @marcoramires. Another way to fix this might be to upgrade to at least node 6.x. Someone had a similar issue: https://github.com/mwaylabs/generator-m-ionic/issues/499
Can you confirm upgrading to a newer node version fixes this?

Hi @gruppjo. Thanks for the reply.
Yeah it works fine once upgraded to node v6.0.0. Interesting enough npm will install the version 0.5.2 instead of the 0.6.1 ("gulp-shell": "^0.5.1").


build: add workflow concurrency control

:tada: This PR is included in version 1.0.29 :tada:
The release is available on:

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


Could you please share your visualization code?

I didn't see how to visualize in the readme, so I assumed there was no code for visualization

Hello! At the moment, we do not have available code for visualizing results, such as point cloud semantic label predictions or retrieved neighbors. However, I plan to refine several scripts and add them to the repository soon to facilitate this feature.

Hello! At the moment, we do not have available code for visualizing results, such as point cloud semantic label predictions or retrieved neighbors. However, I plan to refine several scripts and add them to the repository soon to facilitate this feature.

thanku very much!

Hello! At the moment, we do not have available code for visualizing results, such as point cloud semantic label predictions or retrieved neighbors. However, I plan to refine several scripts and add them to the repository soon to facilitate this feature.

I'm also wondering if these codes can be used with my own plant data set, which contains three labels


--no-offsite-links doesn't appear to work anymore

The --no-offsite-links option doesn't appear to work anymore.  Offsite links are still grabbed.

The command you entered in IRC used an em dash (—) instead of (--).  It's probably possible to make the command parser recognize both.

Ohhhh. My IRC client probably made it "pretty" and that's how it happened.
So yeah, can we change all the various double-dash settings to also accept em-dashes?

Not yet sure.  Option parsing is implemented by the bot via trollop, and I haven't looked very much at its internals.  Maybe tweaking https://github.com/ManageIQ/trollop/blob/master/lib/trollop.rb#L38 is all that'd be required.
It's also possible that we don't need to use trollop and could substitute a custom parser.


Not published on npm

What version of Node are you using?
12.15.0
What did you do?
If possible, provide a recipe for reproducing the error.
Tried to npm install --save @opentelemetry/plugin-express and npm returned E404 not found

Hi,
We just merged the pull request. It will be released at the end of the week.
Thanks for your comprehension.

any update on timing of the release?

@cchatfield we have been focusing on the beta launch and haven't released in a little while. @mayurkale22 WDYT about releasing another minor version before beta?

@cchatfield Is this something you blocked on? I would be happy to cut minor version, but feel like we should wait for https://github.com/open-telemetry/opentelemetry-js/pull/837 and https://github.com/open-telemetry/opentelemetry-js/pull/839. @dyladan WDYT?

If it's not blocking I think we should focus on beta. I have a couple more rename PRs coming too (scope => context, formatter => propagator)

Not critical. When is beta?

The current target date is March 16th, 2020

Published in 0.5.0, let us know if you encounter any issues!


[FEATURE][AZURE] Add the ability to deploy a zip file instead of a directory

We currently run our deploys from azure based on a zip file. It would be really nice if we could use sfpowerscripts (DeploySourceToOrgTask) to manage our deployments but its just missing this one feature.
sfdx force:mdapi:deploy supports a -f option for a zip file.
DeploySourceToOrgImpl

      //directory
      command += ` -d ${this.mdapiDir}`;

I have made the change locally but am struggling to build it on a windows box.

@JoelBrenstrum  Can you shed up some more light on how the zip file is being generated and whats the idea behind it. Are you zipping it during a build stage?

Yup. We're making the zip a build artifact and picking it up further in the release pipeline.
https://developer.salesforce.com/docs/atlas.en-us.api_meta.meta/api_meta/file_based_zip_file.htm
Zip contents

package.xml

I can probably unzip into a temp location and attempt to deploy the contents.
If I could deploy the zip directly that would be ideal.

Got it, we are updating our create source package task for similar scenario (currently it just publishes the metadata, and you have to use checkout and deploy).  This will be combined (in addition) to a new Task called install Source package (which will automatically pull down the artifact) and install it.
Cheers,
Azlam
From: Joel Brenstrum<EMAIL_ADDRESS>Sent: Thursday, 20 August 2020 9:28 AM
To: Accenture/sfpowerscripts<EMAIL_ADDRESS>Cc: Abdulsalam, Azlam<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: [External] Re: [Accenture/sfpowerscripts] [FEATURE][AZURE] Add the ability to deploy a zip file instead of a directory (#128)
This message is from an EXTERNAL SENDER - be CAUTIOUS, particularly with links and attachments.

Yup. We're making the zip a build artifact and picking it up further in the release pipeline.
https://developer.salesforce.com/docs/atlas.en-us.api_meta.meta/api_meta/file_based_zip_file.htmhttps://urldefense.proofpoint.com/v2/url?u=https-3A__developer.salesforce.com_docs_atlas.en-2Dus.api-5Fmeta.meta_api-5Fmeta_file-5Fbased-5Fzip-5Ffile.htm&d=DwMCaQ&c=eIGjsITfXP_y-DLLX0uEHXJvU8nOHrUK8IrwNKOtkVU&r=1gQkLqWJ0FVakrZpcYuRfbSfKnJcxUN9BD1bMOCTMSA&m=eAg3WM-mSfz5oK1gxp2FAieZSgC_tPo9ZFkvR9HZEWA&s=fwav_tWMVUAa_RBn_O9ACh-LQ_fUk9deiP45WGJZuoI&e=
Zip contents
[image]https://urldefense.proofpoint.com/v2/url?u=https-3A__user-2Dimages.githubusercontent.com_4349208_90699521-2D9abac980-2De2d7-2D11ea-2D9041-2D726bb22f41fd.png&d=DwMCaQ&c=eIGjsITfXP_y-DLLX0uEHXJvU8nOHrUK8IrwNKOtkVU&r=1gQkLqWJ0FVakrZpcYuRfbSfKnJcxUN9BD1bMOCTMSA&m=eAg3WM-mSfz5oK1gxp2FAieZSgC_tPo9ZFkvR9HZEWA&s=6H8L4pFBK7lgvaoELC-nQxpQG_kqHJsj8Ce8Id_d88g&e=
package.xml
[image]https://urldefense.proofpoint.com/v2/url?u=https-3A__user-2Dimages.githubusercontent.com_4349208_90699537-2Da3130480-2De2d7-2D11ea-2D9af8-2Dbc65be0966ac.png&d=DwMCaQ&c=eIGjsITfXP_y-DLLX0uEHXJvU8nOHrUK8IrwNKOtkVU&r=1gQkLqWJ0FVakrZpcYuRfbSfKnJcxUN9BD1bMOCTMSA&m=eAg3WM-mSfz5oK1gxp2FAieZSgC_tPo9ZFkvR9HZEWA&s=P_gCv1SyfJqVLvKPxgUZcVfDqvIAYuaSAXPTNWvkz64&e=
I can probably unzip into a temp location and attempt to deploy the contents.
If I could deploy the zip directly that would be ideal.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_Accenture_sfpowerscripts_issues_128-23issuecomment-2D676808802&d=DwMCaQ&c=eIGjsITfXP_y-DLLX0uEHXJvU8nOHrUK8IrwNKOtkVU&r=1gQkLqWJ0FVakrZpcYuRfbSfKnJcxUN9BD1bMOCTMSA&m=eAg3WM-mSfz5oK1gxp2FAieZSgC_tPo9ZFkvR9HZEWA&s=Xt7B5kNdnaMrm0OlkvgAlnIE5KpiF1iHvHVDyCVQ70g&e=, or unsubscribehttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AKN5RJGKKDZNTERF32SDHGTSBRNYNANCNFSM4QEB5DNQ&d=DwMCaQ&c=eIGjsITfXP_y-DLLX0uEHXJvU8nOHrUK8IrwNKOtkVU&r=1gQkLqWJ0FVakrZpcYuRfbSfKnJcxUN9BD1bMOCTMSA&m=eAg3WM-mSfz5oK1gxp2FAieZSgC_tPo9ZFkvR9HZEWA&s=lszxeQ-DNAB93_-__o4fMrK23n8QDOLCbz2qY2JAvC4&e=.

This message is for the designated recipient only and may contain privileged, proprietary, or otherwise confidential information. If you have received it in error, please notify the sender immediately and delete the original. Any other use of the e-mail by you is prohibited. Where allowed by local law, electronic communications with Accenture and its affiliates, including e-mail and instant messaging (including content), may be scanned by our systems for the purposes of information security and assessment of internal compliance with Accenture policy. Your privacy is important to us. Accenture uses your personal data only in compliance with data protection laws. For further information on how Accenture processes your personal data, please see our privacy statement at https://www.accenture.com/us-en/privacy-policy.

www.accenture.com

Closing this issue, as it will be part of the updated Create Source Package/Delta Package Task. Hope it alleviates your issue


nixos/pantalaimon: add pantalaimon service

Motivation for this change
Adding a module for https://github.com/matrix-org/pantalaimon
Note that this module is a prerequisite for getting Mjolnir running with encryption. See: https://github.com/matrix-org/mjolnir#bot-configuration

It is recommended to use Pantalaimon so your management room can be encrypted. This also applies if you are looking to moderate an encrypted room.

This change also adds me (@sumnerevans) to the Matrix maintainer team.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[x] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Added a release notes entry if the change is major or breaking
[x] Fits CONTRIBUTING.md.

Oh, wow. Didn't see the other PR. I'll close this and make comments over there.


https://orbitdb.org redirects to https://njal.la/

did you guys forget to renew the domain?
https://duckduckgo.com/?q=orbit+db&t=brave&ia=images

It still resolved to the website for me?

Double checked the settings, everything seems fine

now i tried and it resolves to https://github.com/orbitdb


Fix causal UI strings according to classification/regression tasks

Description
Made help text in causal chart have separate blurbs for classification and regression.
Before:-

After:-

Checklist

[x] I have added screenshots above for all UI changes.
[ ] Documentation was updated if it was needed.
[ ] New tests were added or changes were manually verified.

Codecov Report

Merging #1419 (4572767) into main (d0047f1) will decrease coverage by 10.33%.
The diff coverage is n/a.

:exclamation: Current head 4572767 differs from pull request most recent head c413751. Consider uploading reports for the commit c413751 to get more accurate results

@@             Coverage Diff             @@
##             main    #1419       +/-   ##
===========================================
- Coverage   85.91%   75.58%   -10.34%
===========================================
  Files         100       10       -90
  Lines        4801      258     -4543
===========================================
- Hits         4125      195     -3930
+ Misses        676       63      -613

Flag
Coverage Δ

unittests
75.58% <ø> (-10.34%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...ponsibleai/responsibleai/_tools/shared/versions.py

...dgets/raiwidgets/error_analysis_dashboard_input.py

raiwidgets/raiwidgets/interfaces.py

...tools/error_analysis/dashboard_schemas/__init__.py

...onsibleai/_tools/shared/attribute_serialization.py

responsibleai/responsibleai/__version__.py

responsibleai/responsibleai/_internal/__init__.py

raiutils/raiutils/webservice/webservice.py

responsibleai/responsibleai/managers/__init__.py

raiutils/raiutils/common/__init__.py

... and 80 more

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 d0047f1...c413751. Read the comment docs.


Missing key features like "View Counter"

Closing for https://github.com/lbryio/lbry-desktop/issues/1951


[W5.6a][W13-2]Arjo Chakravarty

Extracted out a contacts class. Hopefully did this correctly

@arjo129
Some comments added. Please take note and close the pull request.

Thank you!


[plugins/home] Add Featured Docs Card

Hey, I just made a Pull Request!

Contributing a Home page card we use on our team called Featured Docs.
This is a simple card that expects a filter to pull whichever entity/s you want to display.
Additional optional props are added to allow for more customization.

:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[x] Added or updated documentation
[x] Tests for new functionality and regression tests for bug fixes
[x] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

Bad rebase, working on fixing this

@awanlin, I am running into issues resolving these conflicting files.
Is the appropriate way to update this branch to run: git rebase origin/master or git merge master?
I also followed instructions here regarding yarn.lock conflicts, but pulled in a bunch of unintended changes also.
Any advice would be helpful 🙏 Thanks in advance!

Hi @nikolarAutodesk, I'm running out of steam for the day, I'll try and look at this tomorrow though 👍

Thank you, @tudi2d we are going to add to the Adopters list separately, likely through the form. We need another Autodesk member to approve that on our side 👍

Looks like we have all the approvals? Can someone do the merge? 🎉

Looks like @backstage-service is blocking us as a codeowner to merge this in 😆 @backstage/maintainers?


[API Chat] Issue when calling multiple connected resources from the API Chat

Description
I have an API with following resources

I issued following query
List all the station_id where city is London and get the platforms for each station_id
The expected outcome is first call /stations and get all the station_id from the payload and then call /stations/{station_id}
But API chat only executes the first part and stops responding (message saying it is trying to fetch rest). See below

Definition and the backend can be found in https://drive.google.com/drive/folders/1NajcZY7E2KOW6sSN7Sp-wzA-E0r64aZe

For single entry returned from GET /stations, it worked without an issue
ex: List all platforms in station 'Waterloo'

Addtionally, Noticed in the latest pack, there is a dummy request happening. See below

These issues were noticed because resource parameters were getting dropped from the API definition when feeding to the LLM. Thereby, the LLM was unaware that such path parameters were needed for API invocation, hence, the hallucinations.
A workaround for this issue is mentioned below.
Instead of having parameters under the path item object like shown below:
  /stations/{station_id}:
    parameters:
      - in: path
        name: station_id
        required: true
        schema:
          type: string
        description: ID of the station to retrieve
    get:
      summary: Get station by ID
      responses:
        '200':
          description: OK
          content:
            application/json:
              schema:
                $ref: '#/components/schemas/Station'
    put:
      summary: Update station information
      requestBody:
        required: true
        content:
          application/json:
            schema:
              $ref: '#/components/schemas/Station'
      responses:
        '200':
          description: OK
          content:
            application/json:
              schema:
                $ref: '#/components/schemas/Station'

Specify parameters within the operation object (i.e. within get, put, post, etc.) like so:
  /stations/{station_id}:
    get:
      summary: Get station by ID
      parameters:
        - in: path
          name: station_id
          required: true
          schema:
            type: string
          description: ID of the station to retrieve
      responses:
        '200':
          description: OK
          content:
            application/json:
              schema:
                $ref: '#/components/schemas/Station'
    put:
      summary: Update station information
      parameters:
        - in: path
          name: station_id
          required: true
          schema:
            type: string
          description: ID of the station to retrieve
      requestBody:
        required: true
        content:
          application/json:
            schema:
              $ref: '#/components/schemas/Station'
      responses:
        '200':
          description: OK
          content:
            application/json:
              schema:
                $ref: '#/components/schemas/Station'

Since the provided API definition is valid, we will have to fix this from our codebase.


Authorization related enhancements

If we do not use "Overll All Read" permission authorization then a error page is shown in jenkins
If we use "Over All Read" permission SAML security is not enforced untill user tires to log in. This means general read features as availalbe to all public users.
To over come #1 we can use "Overl All Read" permission to "authenticated" users.

There is not support for authenitcated authority in this plugin so if we use this in Matirx Based security; it doesnt work as user does not have authenticated authority.

I have done the changes related to this so that after successful login using SAML; user would get the "Authenticated" authority assinged which can be used in Authorization strategies.
I will send the pull request please review and merge the code so that it can be released.

Hi, thank you for filing this issue. Apologies for closing this issue, but the right place to track is on https://issues.jenkins-ci.org with component "saml-plugin". We are removing the issue tracker from this repo to avoid further confusion. Please refile on the JIRA


0.1.0 should be removed when installing/updating old add-on.

You can see that after installing 0.2.0, the old 0.1.0 add-on is still active and should be removed:

The Oculus viewport distortion would not work until I removed the 0.1.0 add-on.

Ouch, good find on the conflict.


Gradle build failure

See https://github.com/slsa-framework/example-package/actions/runs/5905135220/job/16018679278
Welcome to Gradle 8.1.1!

Here are the highlights of this release:
 - Stable configuration cache
 - Experimental Kotlin DSL assignment syntax
 - Building with Java 20

For more details see https://docs.gradle.org/8.1.1/release-notes.html

Starting a Gradle Daemon (subsequent builds will be faster)
> Task :compileJava
> Task :processResources NO-SOURCE
> Task :classes
> Task :jar
/home/runner/work/example-package/example-package/e2e/gradle/workflow_dispatch/src/main/java/hello/Greeter.java:3: warning: no comment
public class Greeter {
       ^

/home/runner/work/example-package/example-package/e2e/gradle/workflow_dispatch/src/main/java/hello/Greeter.java:4: warning: no comment
> Task :javadoc
  public String sayHello() {
                ^
/home/runner/work/example-package/example-package/e2e/gradle/workflow_dispatch/src/main/java/hello/HelloWorld.java:3: warning: no comment
public class HelloWorld {
       ^
/home/runner/work/example-package/example-package/e2e/gradle/workflow_dispatch/src/main/java/hello/HelloWorld.java:4: warning: no comment
  public static void main(String[] args) {
                     ^
4 warnings

> Task :javadocJar
> Task :sourcesJar
> Task :assemble
> Task :check
> Task :build

BUILD SUCCESSFUL in 57s
5 actionable tasks: 5 executed
Run cd "${PROJECT_ROOT}" && "${GITHUB_WORKSPACE}"/../__TOOL_ACTION_DIR__/collect_release_artifacts.sh
/home/runner/work/example-package/example-package/../__TOOL_ACTION_DIR__/collect_release_artifacts.sh: line 20: ./gradlew: No such file or directory

@AdamKorcz

Argh, I made a mistake in https://github.com/slsa-framework/slsa-github-generator/pull/2645. https://github.com/slsa-framework/slsa-github-generator/pull/2646 should fix this.


ENOENT error for valid proto files

I've been using protobufjs and other tools for other languages, and everything is working great so far.
I wanted to try swapping protobufjs for pbf, but pbf has an issue with the proto files.
At the top of one of my files, I have
// test.proto

syntax = "proto3";

package foo.models.v1;

import "foo/models/v1/Common.proto";

but then when I run ./node_modules/.bin/pbf ./foo/models/v1/test.proto > example.js I get an error:
❯ ./node_modules/.bin/pbf ./foo/models/v1/test.proto > example.js
internal/fs/utils.js:220
    throw err;
    ^

Error: ENOENT: no such file or directory, open '/Users/trusktr/src/test-data-models/foo/models/v1/foo/models/v1/Common.proto'
    at Object.openSync (fs.js:440:3)
    at Object.readFileSync (fs.js:342:35)
    at readSync (/Users/trusktr/src/test-data-models/node_modules/resolve-protobuf-schema/index.js:14:23)
    at /Users/trusktr/src/test-data-models/node_modules/resolve-protobuf-schema/index.js:18:22
    at Array.forEach (<anonymous>)
    at Function.readSync [as sync] (/Users/trusktr/src/test-data-models/node_modules/resolve-protobuf-schema/index.js:17:11)
    at Object.<anonymous> (/Users/trusktr/src/test-data-models/node_modules/pbf/bin/pbf:13:32)
    at Module._compile (internal/modules/cjs/loader.js:956:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:973:10)
    at Module.load (internal/modules/cjs/loader.js:812:32) {
  errno: -2,
  syscall: 'open',
  code: 'ENOENT',
  path: '/Users/trusktr/src/test-data-models/foo/models/v1/foo/models/v1/Common.proto'
}

As you can see, it is repeating part of the path twice (foo/models/v1/foo/models/v1).
As far as I can tell, our proto files are all valid, as they've worked with all of our tools for all our languages so far, just not with pbf due to the above error.
Any ideas?

Seems like the issue may be in resolve-protobuf-schema, considering that pbf uses that. Opened the issue there: https://github.com/mafintosh/resolve-protobuf-schema/issues/10

Hi @trusktr,
I release the vector tiles(.pbf) by using Geoserver, but I have no idea how to generate a .proto file. Where does your proto file come from?
Oliver


New-PSSession fails with Could not load file or assembly 'System.Security.Cryptography'

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.
[X] Refer to the FAQ.
[X] Refer to Differences between Windows PowerShell 5.1 and PowerShell.

Steps to reproduce

Install PowerShell v7.3.0 on a remote computer.
Execute the following PowerShell script on the local machine:

New-PSSession -ComputerName <remote-computer> -Credential (Get-Credential) | Enter-PSSession;
The issue occurs when the remote computer has PowerShell v7.3.0 installed. It works fine when PowerShell v7.2.7 is installed on the remote computer.
Expected behavior
A remote PowerShell session is created and opened.

Actual behavior
The following error occurs:

New-PSSession: [<remote-computer>] Connecting to remote server <remote-computer> failed with the following error message : Could not load file or assembly 'System.Security.Cryptography, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a'. The system cannot find the file specified. For more information, see the about_Remote_Troubleshooting Help topic.

Error details
Exception             :
    Type        : System.Management.Automation.Remoting.PSRemotingTransportException
    ErrorCode   : -2141974624
    ErrorRecord :
        Exception             :
            Type    : System.Management.Automation.ParentContainsErrorRecordException
            Message : Connecting to remote server <remote-computer> failed with the following error message : Could not load file or assembly
'System.Security.Cryptography, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a'. The system cannot find the file specified. For more
information, see the about_Remote_Troubleshooting Help topic.
            HResult : -2146233087
        CategoryInfo          : ResourceUnavailable: (:) [], ParentContainsErrorRecordException
        FullyQualifiedErrorId : System.Management.Automation.Remoting.PSRemotingDataStructureException
    Message     : Connecting to remote server remote-computer failed with the following error message : Could not load file or assembly
'System.Security.Cryptography, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a'. The system cannot find the file specified. For more
information, see the about_Remote_Troubleshooting Help topic.
    HResult     : -2146233087
TargetObject          : System.Management.Automation.RemoteRunspace
CategoryInfo          : OpenError: (System.Management.A…tion.RemoteRunspace:RemoteRunspace) [New-PSSession], PSRemotingTransportException
FullyQualifiedErrorId : IncorrectProtocolVersion,PSSessionOpenFailed
ErrorDetails          : [remote-computer] Connecting to remote server remote-computer failed with the following error message : Could not load file or assembly
'System.Security.Cryptography, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a'. The system cannot find the file specified. For more
information, see the about_Remote_Troubleshooting Help topic.
InvocationInfo        :
    MyCommand        : New-PSSession
    ScriptLineNumber : 1
    OffsetInLine     : 1
    HistoryId        : 66
    Line             : New-PSSession -ComputerName <remote-computer> -Credential (Get-Credential) | Enter-PSSession;
    PositionMessage  : At line:1 char:1
                       + New-PSSession -ComputerName <remote-computer> -Credential (Get-Credential) | En …
                       + ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    InvocationName   : New-PSSession
    CommandOrigin    : Internal
ScriptStackTrace      : at <ScriptBlock>, <No file>: line 1
PipelineIterationInfo :

Environment data
Local machine:

Name                           Value
----                           -----
PSVersion                      7.3.0
PSEdition                      Core
GitCommitId                    7.3.0
OS                             Microsoft Windows 10.0.19044
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Remote machine:

Name                           Value
----                           -----
PSVersion                      7.3.0
PSEdition                      Core
GitCommitId                    7.3.0
OS                             Microsoft Windows 10.0.17763
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Visuals
No response

This is a duplicate of: https://github.com/PowerShell/PowerShell/issues/18587, and is fixed with: https://github.com/PowerShell/PowerShell-Native/pull/83


[hw] Make reggen generate a *_REG_OFFSET macro.

This change makes it possible for DIFs to access the register offset directly, rather than having to do tricky things with macros (example at #1111). This will make writing DIFs a little bit easier.
The *_REG_OFFSET name was chosen since both _OFFSET and _BASE_OFFSET are in use; I can change it to something else if desired.

If you have no plan to use (ID) to access the macro but to use OFFSET only, can you remove the previous offset macros and revise the DIF too?

You probably should link it against #1083 - I have opened an issue some time ago.

If you have no plan to use (ID) to access the macro but to use OFFSET only, can you remove the previous offset macros and revise the DIF too?

That is why #1111 was done the way it was done. I think It would have been more productive to have this discussion on the original pull request, even if it might not be exactly right. I would argue that it's the whole point of code reviews.

@eunchan @silvestrst I think the changes are the same, only that you've added a compatibility layer. I really don't want to bother with that, though, since those older DIFs are going to go away, anyway. No point in making extra work for ourselves.

so i guess the idea here is the same? the headers only have relative offsets, and within the DIF itself we will determine the absolute address by something passed in?
At first glance, I still kind like @silvestrst suggestion of a base address in his PR, but it's probably a minor detail.

@tjaychen this is what a DIF winds up looking like: https://github.com/lowRISC/opentitan/pull/1135
Every function in a DIF library takes a base_addr parameter that we offset off of.

Is there a consensus about the preferred way #1090 or approach in this change?

@silvestrst I don't care either way, as long as someone submits something. You can feel free to do yours, since you mailed it out first.

@silvestrst I don't care either way, as long as someone submits something. You can feel free to do yours, since you mailed it out first.

I just didn't want #1090 to get lost, as I felt like it could be a reasonable alternative to the method in this PR. Really I would like to go with the approach that everyone agrees to be best. As a temporary solution, before the DIFs come to light - we could just fake up the offset to be 0 as we are doing currently, and when the DIFs are good to be merged. Then we could go with #1090 and that hopefully would avoid annoying compatibility definitions, or if it is deemed that method in this PR is more applicable - merge this PR.
I just think that with introduction of registers.{c,h} there is no need to have #define DEVICE_STATUS(id) (DEVICE##id##_BASE_ADDR + 0x10), and instead force consumers to use registers.{c,h}?

@mcy I was thinking, and I think this change for now is better than #1090 , as it avoids adding compatibility defines.
I think this is a good first step change to merge, for the following reasons:

keeping the existing format, so we don't need to mess around with compatibility defines in consumers.
Allows DIFs to use the new register definitions format.

After all DIFs have been implemented we then can fire a follow up change that removes the generation of redundant #define DEVICE_STATUS(id) (DEVICE##id##_BASE_ADDR + 0x10) definitions.
I have completely missed this the first time around, but this change effectively achieves what #1090 does with less effort.

I'm thinking about introducing a new output type to reggen altogether, that, rather than emitting a pile of macros, emits structs of some kind. E.g.,
static const struct {
  // ...
  uint32_t rxf_ptr_offset;
  uint32_t rxf_ptr_read_mask;
  // ...
} kSpiRegisters = { ... };

This is mostly in response to a suggestion @gkelly made. We can land this change as is and discuss this more... invasive version down the line, though.

I'm thinking about introducing a new output type to reggen altogether, that, rather than emitting a pile of macros, emits structs of some kind. E.g.,
static const struct {
  // ...
  uint32_t rxf_ptr_offset;
  uint32_t rxf_ptr_read_mask;
  // ...
} kSpiRegisters = { ... };

The main upshot is, of course, better namespacing.
This is mostly in response to a suggestion @gkelly made. We can land this change as is and discuss this more... invasive version down the line, though.

I completely missed this discussion, could you please point me to it?

just approving this in its original form.  As @eunchan and others have pointed out, since the (id) based scheme will no longer be used, should we remove that part of the code altogether? Or there's probably a larger clean-up PR later?

Larger future cleanup, unfortuately. I think it will happen gradually with the creation of our new glorious difs.

sgtm! thanks!
On Thu, Dec 19, 2019 at 11:27 AM Miguel Young<EMAIL_ADDRESS>wrote:

Larger future cleanup, unfortuately. I think it will happen gradually with
the creation of our new glorious difs.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/lowRISC/opentitan/pull/1118?email_source=notifications&email_token=AAH2RSTUFFCQ3ZJDBJB52CLQZPDJPA5CNFSM4JUKDSV2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEHKVI5Y#issuecomment-567628919,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAH2RSXLGSJMBSIHDIUHRQDQZPDJPANCNFSM4JUKDSVQ
.

@gkelly There might be an open review comment from you on this PR. Please merge if you feel it's resolved.


Rango fails to display hints when page uses some new colors from CSS Color Module Level 4

Up until recently, retrieving the color or the background color of an element using getComputedStyle always returned a string starting with rgb or rgba. With new color spaces added to CSS Color Module Level 4 that is no longer the case. Here are some examples of color strings return from getComputedStyle:

oklch(0.7 0.1 248)
lch(70 0.1 248)
lab(70 10 -40)
oklab(0.7 10 -40)

Here is an example of a page that uses these new colors where the hints aren't displayed: daisyUI — Tailwind CSS Components ( version 4 update is here )
The color library that I am currently using can't handle any of those colors. The colord library seems to be more complete. Although it still can't handle oklch or oklab there seems to be ongoing work to add them.
The solution for this issue would be two steps:

[x] Handle errors when passing colors. Default to hints with dark text over white background.
[ ] Switch to colord and rewrite the color logic to be able to handle these new color spaces.

This library seems interesting too: https://colorjs.io/


List method broken

causing the build to fail: https://travis-ci.org/facundoolano/app-store-scraper/builds/335353628
the itunes rss feed api has changed, causing the method to break. url structure changed and seems like the category field has been removed as well.

fixed


QRCodes in lowercase?

Hi!
URL QRCodes generated are in lowercase, what am i doing wrong? :-)

Sorry, it seems my mistake


OutOfMemory exception for the JobManager

Is there a way to tweak memory usage for the JobManager?  I've deployed the Flink operator on a GKE cluster and I was able to successfully launch the WordCount job.  However, when I launch a custom job, it almost immediately fails due to java.lang.OutOfMemoryError: Java heap space.
        at org.apache.flink.client.program.OptimizerPlanEnvironment.generateException(OptimizerPlanEnvironment.java:150)
        at org.apache.flink.client.program.OptimizerPlanEnvironment.getPipeline(OptimizerPlanEnvironment.java:90)
        at org.apache.flink.client.program.PackagedProgramUtils.getPipelineFromProgram(PackagedProgramUtils.java:108)
        at org.apache.flink.client.program.PackagedProgramUtils.createJobGraph(PackagedProgramUtils.java:58)
        at org.apache.flink.runtime.webmonitor.handlers.utils.JarHandlerUtils$JarHandlerContext.toJobGraph(JarHandlerUtils.java:128)
        at org.apache.flink.runtime.webmonitor.handlers.JarRunHandler.lambda$getJobGraphAsync$6(JarRunHandler.java:138)
        at java.util.concurrent.CompletableFuture$AsyncSupply.run(CompletableFuture.java:1604)
        at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511)
        at java.util.concurrent.FutureTask.run(FutureTask.java:266)
        at java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.access$201(ScheduledThreadPoolExecutor.java:180)
        at java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(ScheduledThreadPoolExecutor.java:293)
        at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
        at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
        at java.lang.Thread.run(Thread.java:748)
Caused by: java.lang.OutOfMemoryError: Java heap space

I'm deploying a custom container, which runs Tensorflow models.  I already tried setting the limits of the jobManager to "8Gi", but even then it still failed.  I had a look at the CRD for the JobManager, but there doesn't seem to set a parameter where you can tweak memory usage.  I also had a look at the parameters to run the job, but I can't find anything either.

When the JobManager is started, I can see this in the logs:
2021-11-30 08:47:07,063 INFO  org.apache.flink.runtime.entrypoint.ClusterEntrypoint         -  Starting StandaloneSessionClusterEntrypoint (Version: 1.10.3, Rev:2b8b670, Date:20.01.2021 @ 11:45:46 CST)
2021-11-30 08:47:07,063 INFO  org.apache.flink.runtime.entrypoint.ClusterEntrypoint         -  OS current user: flink
2021-11-30 08:47:07,063 INFO  org.apache.flink.runtime.entrypoint.ClusterEntrypoint         -  Current Hadoop/Kerberos user: <no hadoop dependency found>
2021-11-30 08:47:07,063 INFO  org.apache.flink.runtime.entrypoint.ClusterEntrypoint         -  JVM: OpenJDK 64-Bit Server VM - Oracle Corporation - 1.8/25.292-b10
2021-11-30 08:47:07,063 INFO  org.apache.flink.runtime.entrypoint.ClusterEntrypoint         -  Maximum heap size: 1820 MiBytes
2021-11-30 08:47:07,063 INFO  org.apache.flink.runtime.entrypoint.ClusterEntrypoint         -  JAVA_HOME: /usr/local/openjdk-8

That maximum heap size seems really small.

Hi @debakkerb!
Can you provide an example of your FlinkCluster resource? I suspect you are not setting the flinkVersion property.

Hey @regadas
Apologies, I forgot to update the ticket.  I didn't fully realise that you could set the heap size in the flinkProperties-section in the YAML config:
flinkProperties:
  taskmanager.numberOfTaskSlots: "1"
  jobmanager.memory.heap.size: "4Gi"

Now I get a different error though, which is causing a lot of confusion on our end:
2021-12-03 10:54:50,752 INFO  org.apache.flink.api.java.ExecutionEnvironment                - The job has 0 registered types and 0 default Kryo serializers
2021-12-03 10:54:50,788 ERROR org.apache.flink.runtime.webmonitor.handlers.JarRunHandler    - Unhandled exception.
org.apache.flink.optimizer.CompilerException: Cannot currently handle nodes with more than 64 outputs.
        at org.apache.flink.optimizer.dag.OptimizerNode.addOutgoingConnection(OptimizerNode.java:348)
        at org.apache.flink.optimizer.dag.SingleInputNode.setInput(SingleInputNode.java:202)
        at org.apache.flink.optimizer.traversals.GraphCreatingVisitor.postVisit(GraphCreatingVisitor.java:272)
        at org.apache.flink.optimizer.traversals.GraphCreatingVisitor.postVisit(GraphCreatingVisitor.java:82)

I haven't spent a lot of time investigating, but the guys who wrote our code state that they are not using that many outputs.

Hi @debakkerb I think we can close this issue as it seems unrelated now. If if you feel it's still relevant please reopen.


OneAPI & Visual Studio

Hi,
I'd like to compile mumps using oneAPI compiler under Visual Studio. To do so, i configure cmake -G "Visual Studio 17 2022" -A x64 -Dopenmp=true -Dparallel=false -B build using the oneAPI command line (in order to dect the fortran compiler). When it comes to the mumps visual studio solution, the project mumps_common failed to compile with the oneAPI compiler (a lot of fortran files are ignored also).
Does anybody already succed ?
Another related question : is there a way to set the oneAPI toolchain compilation in cmake ?
Best regards.
PS : I also tried to configure using Ninja mode but i get the same error when compiling.

with Intel oneAPI on Windows, using
cmake -B build -Dopenmp=true -Dparallel=off -G Ninja
cmake --build build

I get configure text including
-- Building for: Ninja
-- The C compiler identification is IntelLLVM 2022.0.0 with MSVC-like command-line
-- The Fortran compiler identification is IntelLLVM 2022.0.0 with MSVC-like command-line

the "build" directory contains:
dmumps.lib
mumps_common.lib
pord.lib
smumps.lib

Using Visual Studio generator doesn't work for me with MUMPS, I never use the Visual Studio generator. I use Ninja instead.

I don't use a toolchain file for CMake with oneAPI, I just use the oneAPI command prompt for Intel 64 for Visual Studio {2019,2022}

Hi @scivision
I followed your steps but the compilation failed, here the log file
Thanks for helping.
PS : I used the Intel oneAPI command promp
F:\Work\mumps>cmake -B build -Dopenmp=true -Dparallel=off -G Ninja
-- The C compiler identification is IntelLLVM 2022.0.0 with MSVC-like command-line
-- The Fortran compiler identification is IntelLLVM 2022.0.0 with MSVC-like command-line
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working C compiler: C:/Program Files (x86)/Intel/oneAPI/compiler/latest/windows/bin/icx.exe - skipped
-- Detecting C compile features
-- Detecting C compile features - done
-- Detecting Fortran compiler ABI info
-- Detecting Fortran compiler ABI info - done
-- Check for working Fortran compiler: C:/Program Files (x86)/Intel/oneAPI/compiler/latest/windows/bin/ifx.exe - skipped
-- checking that C and Fortran compilers can link
-- checking that C and Fortran compilers can link - OK
-- Populating mumps
-- Configuring done
-- Generating done
-- Build files have been written to: F:/Work/mumps/build/_deps/mumps-subbuild
[1/8] Creating directories for 'mumps-populate'
[1/8] Performing download step (download, verify and extract) for 'mumps-populate'
-- Downloading...
   dst='F:/Work/mumps/build/_deps/mumps-subbuild/mumps-populate-prefix/src/MUMPS_5.4.1.tar.gz'
   timeout='none'
   inactivity timeout='15 seconds'
-- Using src='http://mumps.enseeiht.fr/MUMPS_5.4.1.tar.gz'
-- [download 0% complete]
-- ...
-- [download 100% complete]
-- verifying file...
       file='F:/Work/mumps/build/_deps/mumps-subbuild/mumps-populate-prefix/src/MUMPS_5.4.1.tar.gz'
-- Downloading... done
-- extracting...
     src='F:/Work/mumps/build/_deps/mumps-subbuild/mumps-populate-prefix/src/MUMPS_5.4.1.tar.gz'
     dst='F:/Work/mumps/build/_deps/mumps-src'
-- extracting... [tar xfz]
-- extracting... [analysis]
-- extracting... [rename]
-- extracting... [clean up]
-- extracting... done
[3/8] No patch step for 'mumps-populate'
[4/8] No configure step for 'mumps-populate'
[5/8] No build step for 'mumps-populate'
[6/8] No install step for 'mumps-populate'
[7/8] No test step for 'mumps-populate'
[8/8] Completed 'mumps-populate'
-- Looking for pthread.h
-- Looking for pthread.h - not found
-- Found Threads: TRUE
-- Performing Test LAPACK_s_links
-- Performing Test LAPACK_s_links - Success
-- Performing Test LAPACK_d_links
-- Performing Test LAPACK_d_links - Success
-- Found LAPACK: C:/Program Files (x86)/Intel/oneAPI/mkl/latest/lib/intel64/mkl_intel_lp64.lib;C:/Program Files (x86)/Intel/oneAPI/mkl/latest/lib/intel64/mkl_sequential.lib;C:/Program Files (x86)/Intel/oneAPI/mkl/latest/lib/intel64/mkl_core.lib  found components: MKL
-- The following features have been enabled:

 * Openmp, OpenMP API https://www.openmp.org/

-- The following features have been disabled:

 * Parallel, parallel MUMPS (using MPI and Scalapack)
 * 64-bit-integer, use 64-bit integers in C and Fortran
 * Scotch, Scotch graph partitioning https://www.labri.fr/perso/pelegrin/scotch/
 * BuildScalapack, auto-build Scalapack
 * BuildLapack, auto-build Lapack

-- Configuring done
-- Generating done
-- Build files have been written to: F:/Work/mumps/build

F:\Work\mumps>cmake --build build
[22/504] Generating ../mumps_io.c
FAILED: _deps/mumps-build/mumps_io.c F:/Work/mumps/build/_deps/mumps-build/mumps_io.c
cmd.exe /C "cd /D F:\Work\mumps\build\_deps\mumps-build\src && "C:\Program Files\CMake\bin\cmake.exe" -Din_file:FILEPATH=F:/Work/mumps/build/_deps/mumps-src/src/mumps_io.c -Dpatch_file:FILEPATH=F:/Work/mumps/cmake/mumps_io.c.patch -Dout_file:FILEPATH=F:/Work/mumps/build/_deps/mumps-build/mumps_io.c -P F:/Work/mumps/cmake/PatchFile.cmake"
-- Found Git: C:/Program Files/Git/cmd/git.exe (found version "2.33.0.windows.2")
'C:/Program Files/Git/usr/bin/patch.exe' 'F:/Work/mumps/build/_deps/mumps-src/src/mumps_io.c' '--input=F:/Work/mumps/cmake/mumps_io.c.patch' '--output=F:/Work/mumps/build/_deps/mumps-build/mumps_io.c' '--ignore-whitespace'
patching file F:/Work/mumps/build/_deps/mumps-build/mumps_io.c (read from F:/Work/mumps/build/_deps/mumps-src/src/mumps_io.c)
Hunk #1 FAILED at 13 (different line endings).
1 out of 1 hunk FAILED -- saving rejects to file F:/Work/mumps/build/_deps/mumps-build/mumps_io.c.rej
CMake Error at F:/Work/mumps/cmake/PatchFile.cmake:30 (message):
  Failed to apply patch F:/Work/mumps/cmake/mumps_io.c.patch to
  F:/Work/mumps/build/_deps/mumps-src/src/mumps_io.c with C:/Program
  Files/Git/usr/bin/patch.exe

[23/504] Generating ../include/mumps_io_patched.h
FAILED: _deps/mumps-build/include/mumps_io_patched.h F:/Work/mumps/build/_deps/mumps-build/include/mumps_io_patched.h
cmd.exe /C "cd /D F:\Work\mumps\build\_deps\mumps-build\src && "C:\Program Files\CMake\bin\cmake.exe" -Din_file:FILEPATH=F:/Work/mumps/build/_deps/mumps-src/src/mumps_io.h -Dpatch_file:FILEPATH=F:/Work/mumps/cmake/mumps_io.h.patch -Dout_file:FILEPATH=F:/Work/mumps/build/_deps/mumps-build/include/mumps_io_patched.h -P F:/Work/mumps/cmake/PatchFile.cmake"
-- Found Git: C:/Program Files/Git/cmd/git.exe (found version "2.33.0.windows.2")
'C:/Program Files/Git/usr/bin/patch.exe' 'F:/Work/mumps/build/_deps/mumps-src/src/mumps_io.h' '--input=F:/Work/mumps/cmake/mumps_io.h.patch' '--output=F:/Work/mumps/build/_deps/mumps-build/include/mumps_io_patched.h' '--ignore-whitespace'
patching file F:/Work/mumps/build/_deps/mumps-build/include/mumps_io_patched.h (read from F:/Work/mumps/build/_deps/mumps-src/src/mumps_io.h)
Hunk #1 FAILED at 33 (different line endings).
1 out of 1 hunk FAILED -- saving rejects to file F:/Work/mumps/build/_deps/mumps-build/include/mumps_io_patched.h.rej
CMake Error at F:/Work/mumps/cmake/PatchFile.cmake:30 (message):
  Failed to apply patch F:/Work/mumps/cmake/mumps_io.h.patch to
  F:/Work/mumps/build/_deps/mumps-src/src/mumps_io.h with C:/Program
  Files/Git/usr/bin/patch.exe

ninja: build stopped: subcommand failed.

try
cmake -B build -Dopenmp=true -Dparallel=off -G Ninja -DMUMPS_UPSTREAM_VERSION=5.3.5

cmake --build build

I have just emailed the MUMPS developers again asking them when they'll release MUMPS 5.4.2 that has a fix for MUMPS 5.4.0, 5.4.1. For now, using MUMPS 5.3.5 should work in the case where patching 5.4.x fails as you have experienced.


please publish a new version !

Carthage and cocoaPod's using the latest version is 0.10.1,  it is  not support to swift 3.0 . please publish a new version !
thanks !

it's in progress: #524


abnormal perchloroethylene metabolism

Proposed parent: abnormal xenobiotic pharmacokinetics     (MP:0008875)
New term: abnormal perchloroethylene metabolism
Draft def: altered ability to metabolize perchloroethylene (tetrachloroethylene), a chlorinated olefin solvent with a variety of industrial applications, a ubiquitous contaminant of ground water, soil, ambient and urban air, and is one of the most common pollutants present in many hazardous waste sites.
Thanks!
Molly

MP:0020367
I used tetrachloroethene as the preferred name as this matches CHEBI

new ID MP:0020372


Making Category does not save any sub actions

mc 1.16.5, can be contributing to the issue in #120

looks like my config file is shoving the subcategories at the top of the file rather than the bottom, which might be the issue

Should be fixed in the Minecraft 1.19.4 version of MineMenu :)


Destroy Method

First up, thanks loads for making this library!
A wondering if there's a way to de-initialize the lib somehow after calling configureMonacoTailwindcss()?
I see the MonacoTailwindcss type for the created instance has setTailwindConfig and generateStylesFromContent methods, but no destroy or similar.
Is there any way to achieve this, for example on unmount or to toggle the Intellisense on/off conditionally?

Actually, just seen there is a .dispose() method returned from configureMonacoTailwindcss()
This is what I am after 👍


config: fix cln listconfig output

CLN PeerSwap listconfig currently outputs a liquid.disabled field, which doesn't match how liquidswaps is set in the peerswap.conf file. The output will print as the opposite of how liquidswaps is set. This should fix it to properly match how the config option is set.

Should we go ahead and remove the legacyOptions in this PR? What about RegisterOptions()?

I went ahead and just renamed the rest. We can open another PR to remove that stuff in the future.

Yes, I agree.
I will create it separately.


Edit and Delete rows have errors

i cannot delete a row.. it shows deleted successfully but when i click again on the table name it shows the deleted row.. when i edited a column value whole table got updated to the new row(redundant rows). help me with this.

This bug only happen when you insert/edit wrong values type (ex: insert a string in a sql number typed column).


Fix selection and sorting restoration from query string

This PR fixes two issues with refreshing from a querystring:

Previously, selecting on the 2D plots would not update the query string. Now it does.
Previously, an empty sort key was not parsed correctly from the query string. Now it is.

Hmm. I tried to rebase onto master for this PR but not it says we have merge conflicts. I tried to resolve the conflicts on github but it doesn't seem to be sticking. If you want to merge, then just keep all three lines around the >>>>> and <<<<< in save_settings.js.

@richford Alright, I just worked through a long rebase on master where the only conflicts were in tract-details.js... now selection sticks on refresh, but the subject lines are not colored by group after sort and refresh. It looks like after merging your latest changes my PR can be merged with master sans rebase?

Cool. Thanks for wading through the rebase. The group coloring for the subject lines still works for me. I can't reproduce that error.

I didn't push after working out the rebase locally because it broke group coloring. Sorry for the confusion!


Multiline value with comment

Hello,
I just noticed that this multine value:
pizza=yes \
    no \
    maybe \ ; comment for this line only
    i dont know

will result in this after being parsed:
yes no maybe \

It will ignore the last line i dont know because of the comment in the previous line. Also the \ at the end of the line will be part of the parsed value.
Is this this behaviour expected when adding comments to multine values?

Is this this behaviour expected when adding comments to multine values?

Yes, I think it is expected. The backslash \ in a multiline comment is escaping the line feed LF character.
pizza=yes \
    no \
    maybe \ ; comment for this line only
    i dont know

In the third line of your example, the backslash is not followed by linefeed , but by ; which is indicating the start of a comment.
In this respect Iniparser's behavior is in line with the  Line Continuation  in C. The two examples below will give you errors in C:
	const char test[] = "this is a test " \
						 "about multiline " \// comment
						 "values";

#define test this is a test \
	about multiline \//comment
	values


brew cask reinstal: Error: No such file or directory - brew

General troubleshooting steps

[x] I have retried my command with --force and the issue is still present.
[x] I have checked the instructions for reporting bugs (or making requests) before opening the issue.

[x] None of the templates was appropriate for my issue, or I’m not sure.

[x] I ran brew update-reset && brew update and retried my command.
[x] I ran brew doctor, fixed as many issues as possible and retried my command.
[x] I understand that if I ignore these instructions, my issue may be closed without review.

Description of issue
failure with brew cask reinstall
Error: No such file or directory - brew
i've tested with two casks.
it works coconutbattery but fails with virtualbox
might just be some issue in the virtualbox cask, not sure.
i've tried at least the following things:

--force
brew doctor
brew update-reset && brew update
cd $(brew --repo); git fetch; git reset --hard origin/master; brew update

thanks for looking
Output of your command with --verbose --debug
$ brew cask reinstall virtualbox --verbose --debug
==> Hbc::Installer#reinstall
==> Hbc::Installer#install
==> Printing caveats
==> Hbc::Installer#fetch
==> Satisfying dependencies
==> Downloading
==> Downloading http://download.virtualbox.org/virtualbox/5.2.6/VirtualBox-5.2.6-120293-OSX.dmg
Already downloaded: /Users/ftc2/Library/Caches/Homebrew/Cask/virtualbox--5.2.6-120293.dmg
==> Downloaded to -> /Users/ftc2/Library/Caches/Homebrew/Cask/virtualbox--5.2.6-120293.dmg
==> Verifying download
==> Determining which verifications to run for Cask virtualbox
==> Checking for verification class Hbc::Verify::Checksum
==> 1 verifications defined
Hbc::Verify::Checksum
==> Running verification of class Hbc::Verify::Checksum
==> Verifying checksum for Cask virtualbox
==> SHA256 checksums match
==> Uninstalling Cask virtualbox
==> Un-installing artifacts
==> 4 artifact/s defined
#<SortedSet:0x00000102056768>
==> Un-installing artifact of class Hbc::Artifact::PreflightBlock
==> Executing: ["brew", "cask", "uninstall", "virtualbox-extension-pack"]
Error: No such file or directory - brew
Follow the instructions here:
  https://github.com/caskroom/homebrew-cask#reporting-bugs
/usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/2.3.3/lib/ruby/2.3.0/open3.rb:199:in `spawn'
/usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/2.3.3/lib/ruby/2.3.0/open3.rb:199:in `popen_run'
/usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/2.3.3/lib/ruby/2.3.0/open3.rb:95:in `popen3'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/system_command.rb:88:in `each_output_line'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/system_command.rb:25:in `run!'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/system_command.rb:14:in `run'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/system_command.rb:18:in `run!'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/dsl/base.rb:14:in `system_command'
/usr/local/Caskroom/virtualbox/.metadata/5.2.0-118431/20171105181403.163/Casks/virtualbox.rb:15:in `block (2 levels) in load'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/artifact/abstract_flight_block.rb:38:in `instance_eval'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/artifact/abstract_flight_block.rb:38:in `abstract_phase'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/artifact/abstract_flight_block.rb:26:in `uninstall_phase'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/installer.rb:403:in `block in uninstall_artifacts'
/usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/2.3.3/lib/ruby/2.3.0/set.rb:674:in `each'
/usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/2.3.3/lib/ruby/2.3.0/set.rb:674:in `each'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/installer.rb:400:in `uninstall_artifacts'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/installer.rb:353:in `uninstall'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/installer.rb:115:in `uninstall_existing_cask'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/installer.rb:76:in `install'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/installer.rb:104:in `reinstall'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli/reinstall.rb:10:in `block in run'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli/reinstall.rb:5:in `each'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli/reinstall.rb:5:in `run'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli/abstract_command.rb:35:in `run'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli.rb:98:in `run_command'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli.rb:168:in `run'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli.rb:132:in `run'
/usr/local/Homebrew/Library/Homebrew/cmd/cask.rb:8:in `cask'
/usr/local/Homebrew/Library/Homebrew/brew.rb:100:in `<main>'
Error: Kernel.exit
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli.rb:179:in `exit'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli.rb:179:in `rescue in run'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli.rb:156:in `run'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli.rb:132:in `run'
/usr/local/Homebrew/Library/Homebrew/cmd/cask.rb:8:in `cask'
/usr/local/Homebrew/Library/Homebrew/brew.rb:100:in `<main>'

Output of brew cask doctor
$ brew cask doctor
==> Homebrew-Cask Version
Homebrew-Cask 1.5.4-88-g650d6db
caskroom/homebrew-cask (git revision 58514; last commit 2018-02-24)
==> macOS
10.12.6
==> SIP
Enabled
==> Java
N/A
==> Homebrew-Cask Install Location
<NONE>
==> Homebrew-Cask Staging Location
/usr/local/Caskroom
==> Homebrew-Cask Cached Downloads
~/Library/Caches/Homebrew/Cask (16 files, 613.0MB)
==> Homebrew-Cask Taps:
/usr/local/Homebrew/Library/Taps/caskroom/homebrew-cask (3921 casks)
/usr/local/Homebrew/Library/Taps/caskroom/homebrew-drivers (144 casks)
/usr/local/Homebrew/Library/Taps/caskroom/homebrew-eid (10 casks)
/usr/local/Homebrew/Library/Taps/caskroom/homebrew-fonts (1159 casks)
/usr/local/Homebrew/Library/Taps/caskroom/homebrew-versions (172 casks)
==> Contents of $LOAD_PATH
/usr/local/Homebrew/Library/Homebrew/cask/lib
/usr/local/Homebrew/Library/Homebrew
/usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/2.3.3/lib/ruby/gems/2.3.0/gems/did_you_mean-1.0.0/lib
/usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/2.3.3/lib/ruby/site_ruby/2.3.0
/usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/2.3.3/lib/ruby/site_ruby/2.3.0/x86_64-darwin9.0
/usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/2.3.3/lib/ruby/site_ruby/2.3.0/universal-darwin9.0
/usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/2.3.3/lib/ruby/site_ruby
/usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/2.3.3/lib/ruby/vendor_ruby/2.3.0
/usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/2.3.3/lib/ruby/vendor_ruby/2.3.0/x86_64-darwin9.0
/usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/2.3.3/lib/ruby/vendor_ruby/2.3.0/universal-darwin9.0
/usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/2.3.3/lib/ruby/vendor_ruby
/usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/2.3.3/lib/ruby/2.3.0
/usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/2.3.3/lib/ruby/2.3.0/x86_64-darwin9.0
/usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/2.3.3/lib/ruby/2.3.0/universal-darwin9.0
==> Environment Variables
LC_ALL="en_US.UTF-8"
PATH="/usr/bin:/bin:/usr/sbin:/sbin:/usr/local/Homebrew/Library/Homebrew/shims/scm"
SHELL="/usr/local/bin/zsh"

brew cask install --force virtualbox

thanks
please close issue (unless reinstall is expected to work here)

This was an uninstall issue with the old version of the Cask, it won't occur again.


[storage] toolbox: storage_download_to_pvc: tasks/main: correctly detect the pod completion

…

test passed in https://github.com/openshift-psap/topsail/pull/542, merging


Library using "api.form.io" insted of "my-project.form.io"

I really hate posting things that are so vague like this, but I honestly have no idea what happened. Everything was working great and then all of a sudden all my calls are getting executed with "api.form.io" instead of "my-project-url.form.io". So everything is getting a 401. I doubled checked I have set Formio.setProjectUrl to my project url (which makes sense, I am literally executing the exact same code that was working early. Am I missing something or is it possible I accidentally did something where the library is no longer using my url?
Edit: so my code is definitely hitting this line: https://github.com/formio/react-formio/blob/5bdf014aeebea749b1330b0a8afdec9202e92cd5/src/modules/forms/actions.js#L59
But when I open up chrome debugger, that line is being displayed as _Formio2.default.getProjectUrl() which returns "api.form.io". I'm not sure why all of a sudden the global variable is changing to that.

I was able to figure this out. I didn't realize we had "formiojs" installed as a standalone module and we were setting the projectUrl for it and not the dependent Formio object react-formio uses. So we essentially had two Formio global objects. Changing import { Formio } from 'formiojs'; to import { Formio } from 'react-formio'; fixed the issue


Mention imagemagick dependency on Pypi

People who google "heic to jpg python" will find the Pypi that is connected to this repo. This repo mentions installing imagemagick as a dependency, but the pypi page doesn't mention it and probably should.

It does say

Essentially, this is just a wrapper around imagemagick's convert cli.

I can make it even more explicit though.

I think you should, not everyone knows what imagemagick is.

I think this should be sufficiently clear now.


Create .npmignore

Tests and lint configuration need to stay in npm package. We may consider filtering .travis.yml but it doesn't make much difference (and just for that another file in a repo?), so I'm closing it.

Why would tests and lint ever need to stay in the npm package?

It's been discussed here:
https://github.com/medikoo/es5-ext/issues/11
https://github.com/npm/npm/issues/5264

re: npm/npm#5264
Considering the npm team uses package.json#files for all their repositories I think you should probably follow suit.


Ensure that only visible paired lights are used

This is a longstanding issue in both the Mobile and GL Compatibility renderer.
Meshes pair with all lights that touch them, and then at draw time, we send all paired lights indices to the shader (even if that light isn't visible). The problem is that non-visible lights aren't uploaded to the GPU and don't have an index. So we end up using a bogus index
Fixes: https://github.com/godotengine/godot/issues/81143
Fixes: https://github.com/godotengine/godot/issues/83340
Fixes: https://github.com/godotengine/godot/issues/79463

Thanks!


Android app crash

Hi, i would like to run your age predictor on android. I am able to compile and run app but when i put this line
deserialize("/sdcard/dnn_age_predictor_v1.dat") >> age_net;
app is crushing. File is available and deserializing is ok bcs. when i comment ">> age_net" its not crushing.
I also use all your implementation to apredictor_t type. Its look like
const unsigned long number_of_age_classes = 81;
// The resnet basic block.
template<
int num_filters,
template class BN,  // some kind of batch normalization or affine layer
int stride,
typename SUBNET

using basicblock = BN<con<num_filters, 3, 3, 1, 1, relu<BN<con<num_filters, 3, 3, stride, stride, SUBNET>>>>>;
// A residual making use of the skip layer mechanism.
template<
template<int, template class, int, typename> class BLOCK,  // a basic block defined before
int num_filters,
template class BN,  // some kind of batch normalization or affine layer
typename SUBNET

// adds the block to the result of tag1 (the subnet)
using residual = add_prev1<BLOCK<num_filters, BN, 1, tag1>>;

// A residual that does subsampling (we need to subsample the output of the subnet, too).
template<
template<int, template class, int, typename> class BLOCK,  // a basic block defined before
int num_filters,
template class BN,
typename SUBNET

using residual_down = add_prev2<avg_pool<2, 2, 2, 2, skip1<tag2<BLOCK<num_filters, BN, 2, tag1>>>>>;
// Residual block with optional downsampling and batch normalization.
template<
template<template<int, template class, int, typename> class, int, templateclass, typename> class RESIDUAL,
template<int, template class, int, typename> class BLOCK,
int num_filters,
template class BN,
typename SUBNET

using residual_block = relu<RESIDUAL<BLOCK, num_filters, BN, SUBNET>>;
template<int num_filters, typename SUBNET>
using aresbasicblock_down = residual_block<residual_down, basicblock, num_filters, affine, SUBNET>;
// Some useful definitions to design the affine versions for inference.
template using aresbasicblock256 = residual_block<residual, basicblock, 256, affine, SUBNET>;
template using aresbasicblock128 = residual_block<residual, basicblock, 128, affine, SUBNET>;
template using aresbasicblock64  = residual_block<residual, basicblock, 64, affine, SUBNET>;
// Common input for standard resnets.
template
using aresnet_input = max_pool<3, 3, 2, 2, relu<affine<con<64, 7, 7, 2, 2, INPUT>>>>;
// Resnet-10 architecture for estimating.
template
using aresnet10_level1 = aresbasicblock256<aresbasicblock_down<256, SUBNET>>;
template
using aresnet10_level2 = aresbasicblock128<aresbasicblock_down<128, SUBNET>>;
template
using aresnet10_level3 = aresbasicblock64;
// The resnet 10 backbone.
template
using aresnet10_backbone = avg_pool_everything<
aresnet10_level1<
aresnet10_level2<
aresnet10_level3<
aresnet_input>>>>;
using apredictor_t = loss_multiclass_log<fc<number_of_age_classes, aresnet10_backbone<input_rgb_image>>>;
apredictor_t age_net;
Don't have any idea. Has you?

same problem for gender classifier


Ignore generated code in GitHub diffs

It appears that we can mark certain paths to be hidden by default in diffs and to not count against the repository's language statistics. See https://docs.github.com/en/repositories/working-with-files/managing-files/customizing-how-changed-files-appear-on-github

That's very cool. I didn't know about this.
Some points to consider:

This makes it easier for unreviewed malicious code to sneak in via an innocuous pull request; we still have to review the contents of these directories before merging if they appear in a PR, and
GitHub's UI shows these suppressed files in a format that is very similar to their automatic suppression of large diffs that still need to be reviewed, so you still have to scroll past them in the UI and read the message for each one to determine if they need to be reviewed individually.
Since this only affects how generated files are displayed in GitHub, it's still nice to have a separate commit in the git history for when one is reviewing the logs on the command-line or in other git tools.

GitHub's UI for this could be a little better. I'd rather have toggle to automatically hide/show all generated files, similar to GitHub's hide/show whitespace changes toggle. Also, I'd really like the generated code to be in a separate commit for the command-line history when we make big changes to the thrift files.
My conclusion: I think using this is probably better than not using it, but it doesn't help with everything we care about.

re point 1 above - I wonder if we can reduce some of the risk by removing all of the files in this directory and regenerating them during the release process.

That's already done. The release process activates the thrift profile.

This is definitely a nice feature but as stated, number 1 is my main concern too. Would be easy to sneak in something or even just screw up the generated code by mistake and might get missed.
Also, when I view #3081 now it doesn't seem like there is an easy way to still view the diff for the generated code with a toggle option so it seems like it's completely hidden and no way to easily view it if you wanted which is kind of annoying.

it doesn't seem like there is an easy way to still view the diff for the generated code with a toggle option

You have to click on each "Load diff" link for each separate file. You can't do it for all at once.

I don't understand how # 1 is a concern if the code in that directory is deleted and then regenerated at release.

Well one issue could be is that there would be a long delay from when the issue was introduced vs catching it (if we don't do releases for a long time). So we could be potentially developing/testing against something that is not what will be generated later. And (in the unlikely change) there was something introduced that was malicious it would be loaded on any developer machines still.
I wonder if there is some way to auto detect changes to the directory when unintentional (ie we didn't run thrift). Not sure if there is some way to mark a directory as read only in Git or use some kind of git post commit hook or add something in maven to detect changes, etc. Maybe we could do something with Jenkins/CI to detect changes and report so we catch it.

sure, but we could just change it from pre-release to when integration tests are done.

The main point for items 1 and 2 is that it puts responsibility on the committer who does the merge to ensure they've done their due diligence before merging. One way they can ensure the code is good is to just check out the pull request, delete the generated directory, and build with mvn package -Pthrift -DskipTests to verify that the contents of the generated directory are the same using their local build as what is in the PR. I normally do this anyway, but it saves me steps when we merge a PR that omits those changes, and I just commit what I generate locally.
For what it's worth, it is extremely rare for a non-committer to touch thrift... as that generally requires expertise and a developer environment that most new contributors don't have yet. And, for committers, who who we already trust with write access to the repo, this isn't really an issue.
Item 2 is just suboptimal GitHub tooling.
Item 3 is my biggest concern, because looking at changes in the history is very inconvenient when the thrift generated code is squashed into the non-generated code in the same commit, and the generated code diff drowns out the important diffs. In my mind, it's just as easy to just omit the generated code from the PR to begin with, and apply those changes later... but then you don't get the automated checks if it doesn't compile (unless we configure GitHub Actions to build thrift).
One way this could be done is to have a PR that omits the generated code, then a separate PR that depends on it that includes it. We can review the one that omits it, and rely on the automated checks from the PR that includes it. But then there's a risk that they could be different.
Another option is to just stop checking in generated code to the repo, and just make that part of the build. However, that also is troublesome, because thrift is a pain to build (we can't just run a code generator as a maven plugin), and because the code needs to be generated prior to loading the project into an IDE.
None of the available options are perfect. We just have to find the least bad one.

I feel like the simple answer here is to just remove the Thrift profile and run Thrift during the compile phase. It only takes a couple of seconds and is dwarfed by the time it takes to run tests. Removing the contents of the directory that contain the Thrift generated code and regenerating the code should alleviate the concern about unreviewed malicious code being introduced into that directory. Using the .gitattributes feature I found will hide the thrift generated java files during the PR, and I think we can safely ignore them during a review.
It may be the case that not everyone has Thrift installed, and I agree that it's not the easiest thing to install. I'm thinking that for this issue we create a Docker build environment that can be used locally and by the automated build systems. The Docker image would have Java, Thrift, Maven, etc installed and would build the code that is checked out on the local filesystem (see this for an idea of what I'm thinking). This would not preclude someone from building Accumulo in the way that they want.

If you did exclude it from the build then you wouldn't even need the .gitattributes at all as you would just ignore the generated code in git entirely and not commit it so it wouldn't be part of the commit/repository.
While normally I would say that's the way to go (just generate each time and not commit) I don't know that I like the idea of someone having to set up Thrift to build or Docker or whatever. Adding that extra dependency is a bit annoying vs just being able to checkout and build using Maven. But this may be the best option as long as it's pretty quick to generate the Thrift code, we can just have instructions for setting up Thrift and for also using Docker.
Something else I thought of as an alternative is that this has been making me wonder if if it makes sense to just pull all of the Thrift stuff into it's own separate project and make it a separate dependency that the main project depends on. Not saying we necessarily want to do that but it's another option. I guess it mostly depends on how often we change the protocol, if there's a lot of changes then it would get annoying to have to keep updating another project and releasing it so may or may not be the right answer.
One other benefit of moving it to a separate project is that there has been some initial discussion about having a plugable protocol at some point. Right now we require Thrift and it's tightly coupled but we don't have to in the future so if we decided to refactor it to have an API and allow people to use a different RPC protcol (something like grpc and protofbuf for example) having it separated out makes sense too.

One of the first tickets I worked on when the project went open source was https://issues.apache.org/jira/browse/ACCUMULO-756. That never got merged, but I think it can/should be revisited. It would make it easier to put the thrift stuff in its own project. However, that's a much bigger task than what's being discussed here, and even that doesn't solve the problem we're discussing much, because the changes that prompted this discussion, as well as many Thrift changes involving generated code changes, are changes where we need to change the RPC layer to support some feature on top, so both layers are changing simultaneously.
I share the concerns about raising the barrier for new folks. Requiring docker instead of thrift isn't much better. I'm not entirely opposed to dropping the generated code from the source tree and just always activate the thrift profile, so it is generated each build, but I'd lean towards just keeping the status quo as my first choice. My second choice would be to do the .gitattributes thing and just deal with the generated code being squashed into the same commits as the changed code that generated it, as proposed here.

It may be the case that not everyone has Thrift installed, and I agree that it's not the easiest thing to install. I'm thinking that for this issue we create a Docker build environment that can be used locally and by the automated build systems. The Docker image would have Java, Thrift, Maven, etc installed and would build the code that is checked out on the local filesystem (see this for an idea of what I'm thinking). This would not preclude someone from building Accumulo in the way that they want.

Re: barrier to entry... What @dlmarion suggests could also be used to set up codespaces. Codespaces are free for open source development, and as far as I can tell, are enabled on the Apache GitHub org. You can work entirely in a browser if you want (though at least Visual Studio Code and JetBrains IDEs are supported too for remote development in the container), so there is no requirement to install software if you just want to play around a little bit.

Draft PR up that contains the beginning of what I was thinking.


nodePackages.vue-cli: drop

The Vue CLI is now deprecated(per upstream, and create-vue(not package in nix) should be used to scaffold Vue projects. The last commit to the upstream repo for this tool was 2 years ago, so it's no longer maintained and should be safe to drop.
Vue 2 projects are still supported by the new tooling, so even if users have to migrate, they do not have to upgrade their framework versions.
Description of changes

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/4244

Not familiar if other steps are needed, maybe @NixOS/node can comment?


Add cursor model

Description
This PR will close #1614 by adding a model for the cursor. Right now it only does position of the cursor in world coordinates. It's also not bidirectionally synced, so viewer.cursor.position = ... won't move the cursor, but is actually settable. I guess I should probably not make it settable or do the syncing, but curious what others think.
We might want to remove the cursor from the layer model, but that can come later.
Type of change

[x] Refactor (non-breaking change which fixes an issue)

But I am confused about how to get access to the cursor in data coordinates? Or maybe that's an antipattern, since the cursor is part of the viewer, not a layer. Perhaps updating some of the examples would be useful!

Indeed, I can work on this. Ultimately I think it will be provided by a method, and the cursor position won't actually live on the layer itself - i.e. layers shouldn't know about how they are sliced or where the cursor is etc.
I'll work on this a bit more

Ok @jni I've update all I want to update. Here's the API as it is now
viewer.cursor.position #position of cursor in world coordinates
layer.position #position of cursor in world coordinates, identical to viewer.cursor.position, gets updated whenever that changes
layer.coordinates #position of cursor in data coordinates, computed from layer.position based on transforms

I think we can continue to refine this during the 0.4 series of release. I imagine that we will drop layer.position and layer.coordinates as explicit properties ultimately under the logic of #1353 and that layers shouldn't know about where the cursor is, but that can come for another day.
Want to give this a final look before merge?

Ok, I've added comments will merge once tests pass!!


Ways to decode pbf to geojson keeping the xyz file structure (either directly in 1 geojson or multiple geojson in xyz folders)

Hello,
Is there any way to decode the pbf to geojson, while keeping the Slippy map file structure ? I would like to either to have one geojson with an extra line with the xy information, or to quickly transform all the pbf files that are split in multiple folders (in slippy map format) to geojsons ?
I tried the command tippecanoe-decodewith option -cbut it's not what I want.
Thank you

If you have a directory of .pbf files named tiles, you could make a parallel directory geojsons of corresponding JSON with something like
find tiles -name '*.pbf' -print |
while read i
do
  out="$(echo "$i" | sed 's/^tiles/geojsons/').json"
  mkdir -p "$(dirname "$out")"
  tippecanoe-decode "$i" $(echo "$i" | awk -F/ '{print $2, $3, $4}') > "$out"
done

Thank you, I will try it.
Is there any way to read the pbf files in python ?

You can use https://github.com/mapbox/vector-tile-base to decode vector tiles from python.


Clarify install steps for macOS

macOS is using zsh as the default shell since macOS Catalina (the previous version of macOS Big Sur). I think now it's time to update our documentation.

Force-pushed after merge to fix a typo in line 31.


[Bug]: High CPU utilization

zot version
v2.0.0
Describe the bug
zot is consuming one CPU core minimum.  You can see this behavior with the example/config-minimal.json
The problem appears to be in the scheduler, there is a spin loop at line 261 in pkg/scheduler.go that appears to never hit the throttle condition, and instead loops infinitely against scheduler.getTask looking for tasks that don't (and won't) exist.
To test this theory, I added a 1ms sleep in the loop I mentioned above, and it reduced the CPU utilization down to approx 20 millicpu
diff --git a/pkg/scheduler/scheduler.go b/pkg/scheduler/scheduler.go
index 174559d..e21528b 100644
--- a/pkg/scheduler/scheduler.go
+++ b/pkg/scheduler/scheduler.go
@@ -282,6 +282,7 @@ func (scheduler *Scheduler) RunScheduler() {
 					// push tasks into worker pool until workerChan is full.
 					scheduler.workerChan <- task
 				}
+				time.Sleep(1 * time.Millisecond)
 			}
 		}
 	}()

To reproduce

make OS=darwin ARCH=arm64 binary
./bin/zot-darwin-arm64 serve examples/config-minimal.json

Expected behavior
The expected behavior would be near-zero CPU utilization when there are no images and no work being done.
Screenshots

This is from a 30s cpu profile, showing the main utilization coming from the scheduler and scheduler.getTask

Additional context
Here is the log output, when running using the minimal config.
{"level":"info","params":{"distSpecVersion":"1.1.0-dev","GoVersion":"go1.20.9","Commit":"v2.0.0-1-g59f41ac-dirty","ReleaseTag":"v2.0.0","BinaryType":"-imagetrust-lint-metrics-mgmt-profile-scrub-search-sync-ui-userprefs","Storage":{"RootDirectory":"/tmp/zot","Dedupe":true,"RemoteCache":false,"GC":true,"Commit":false,"GCDelay":3600000000000,"GCInterval":3600000000000,"Retention":{"DryRun":false,"Delay":0,"Policies":null},"StorageDriver":null,"CacheDriver":null,"SubPaths":null},"HTTP":{"Address":"<IP_ADDRESS>","ExternalURL":"","Port":"8080","AllowOrigin":"","TLS":null,"Auth":{"FailDelay":0,"HTPasswd":{"Path":""},"LDAP":null,"Bearer":null,"OpenID":null,"APIKey":false},"AccessControl":null,"Realm":"","Ratelimit":null},"Log":{"Level":"debug","Output":"","Audit":""},"Extensions":null,"scheduler":null},"goroutine":1,"caller":"zotregistry.io/zot/pkg/api/controller.go:221","time":"2023-12-27T16:49:40.024082-06:00","message":"configuration settings"}
{"level":"info","cpus":8,"max. open files":61440,"goroutine":1,"caller":"zotregistry.io/zot/pkg/api/controller.go:90","time":"2023-12-27T16:49:40.024119-06:00","message":"runtime params"}
{"level":"warn","goroutine":1,"caller":"zotregistry.io/zot/pkg/debug/swagger/swagger_disabled.go:20","time":"2023-12-27T16:49:40.043161-06:00","message":"skipping enabling swagger because given zot binary doesn't include this feature, please build a binary that does so"}
{"level":"warn","goroutine":1,"caller":"zotregistry.io/zot/pkg/debug/gqlplayground/gqlplayground_disabled.go:17","time":"2023-12-27T16:49:40.043198-06:00","message":"skipping enabling graphql playground extension because given zot binary doesn't include this feature, please build a binary that does so"}
{"level":"info","goroutine":1,"caller":"zotregistry.io/zot/pkg/extensions/extension_metrics.go:31","time":"2023-12-27T16:49:40.043265-06:00","message":"setting up metrics routes"}
{"level":"info","goroutine":1,"caller":"zotregistry.io/zot/pkg/extensions/extension_search.go:79","time":"2023-12-27T16:49:40.043292-06:00","message":"skip enabling the search route as the config prerequisites are not met"}
{"level":"info","goroutine":1,"caller":"zotregistry.io/zot/pkg/extensions/extension_image_trust.go:30","time":"2023-12-27T16:49:40.04333-06:00","message":"skip enabling the image trust routes as the config prerequisites are not met"}
{"level":"info","goroutine":1,"caller":"zotregistry.io/zot/pkg/extensions/extension_mgmt.go:89","time":"2023-12-27T16:49:40.043344-06:00","message":"skip enabling the mgmt route as the config prerequisites are not met"}
{"level":"info","goroutine":1,"caller":"zotregistry.io/zot/pkg/extensions/extension_userprefs.go:33","time":"2023-12-27T16:49:40.043369-06:00","message":"skip enabling the user preferences route as the config prerequisites are not met"}
{"level":"info","goroutine":1,"caller":"zotregistry.io/zot/pkg/extensions/extension_ui.go:64","time":"2023-12-27T16:49:40.043405-06:00","message":"skip enabling the ui route as the config prerequisites are not met"}
{"level":"info","component":"dedupe","goroutine":23,"caller":"zotregistry.io/zot/pkg/storage/common/common.go:991","time":"2023-12-27T16:49:40.043706-06:00","message":"no repositories found in storage, finished."}

It looks like the bug was introduced in https://github.com/project-zot/zot/pull/2085, when the <-throttle was moved out of the for loop, and placed into a conditional which we don't appear to hit.
https://github.com/project-zot/zot/pull/2085/files#diff-fb3d6b36d3cd727e1dd2353fb86c0aee0bfaae1c900e0f005ab913620021d551L272

can confirm this also for amd64

@patrick-esentire @danielsand yes, we are aware of this issue. Thanks for reporting it. We will get to this as soon as folks are back from holidays.

Note that depending on configuration, zot has many background tasks - CVE scanning, scrub, sync, stats, etc which can keep it quite busy.


Configure import organizer

Hello!
It would be nice to have a way to configure the import organizer. Especially useful when paired with codeActionsOnSave.
As a concrete example, when creating a new file, I start with the imports and if I save the file before I use everything, they will get deleted, having to reimport or rely on extensions that auto import. Pressing ⌘S often has turned into a habit and I presume that's the case for a lot of people.
I currently don't see any other configurations that might be useful except "ignore unused imports", because I think it does a great job, but other suggestions might be raised in the future.
Thank you!

Hey, @mjbvz,
Thanks for the fast reply!
I don't quite understand why it was marked as an extension candidate. It would imply a new "feature", but the issue is with 2 existing features offered by VSC which kind of conflict with one another:

codeActionsOnSave
organizeImports

When used together, organizeImports become unusable, as the imports get deleted. How could an extension fix this? The only solution is to not use the two features together 😞