About streampetr evaluation

Hi, how can we reproduce the evaluation on StreamPETR-S (single frame), using only key frames generated? Please shed some light!

Sorry for the late reply, the StreamPETR-s is obtained by closing the temporal propagation, which you may refer to the original StreamPETR paper for more information. I'm sorry that I haven't organized the single-frame-related code yet.

Thanks! Does setting queue_length=1 make sense if I only make this change to your 'window' training script?

Hello, theoretically it might not be sufficient. Maybe you should check the original implementation in StreamPETR for more specific details.


Small typos

#155

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

No reply for 8 days. I merge this PR without CHANGELOG update.


I can't access web_server?

when it enters ap mode it delivers the ip class 192.168.4.x and I try to access it through the port http://<IP_ADDRESS>:80 and I have also used nmap to test the ports but I don't think so.
The question: is this a bug or the web server system hasn't been implemented yet or I don't know how to configure it?
Thank you in advance for your attention and for sharing the project, congratulations on your project.

Are you running ESPHome or your custom firmware?

I did the procedure as it is in the document, I didn't even do a customization.


making error and errorHander lazy vals to avoid init NPE's

Fixes #378 .  This will solve any silly initialization errors that can arise when extending this class and using vals for/in handlers or error handlers.  I did some benchmarking and there was no difference in performance whatsoever.

opened this against the wrong branch, closing


Make microplan to understand states

currently we generate state.json file, when microplan publish is executed.
Whenever we run microplan publish, we need to persist a state inside the init file also. ( may be have a issue url or id, in every plan object ) This will help us match the plan object to a result object in state.json and further enables us to do advanced operations like editing an issue created via microplan.
TODO:

User should be able to add new issues and republish with the same file. Right now, every time I want to use microplan, I need to create different files and switch between them. This kind of offends microplan's core idea.
Failed plans in previous publish should be published in this publish.

I think, this is little too much for people from Hactoberfest, because there are lof unknown parameters that may arise during developing this issue. So, I am removing the Hacktoberfest label.
We need to think more on this and until that, let's not touch this.


glossary: apply  tags to 'workspace' term et. al.

Related to (but doesn't fully fix) #461

Up to this point the PR does exactly what the title says.
Further commits will contain other misc. changes I didn't have time to finish in the previous "regular updates" PR.


Compile-AppInBCContainer : Exception calling "DownloadFile" with "2" argument(s): "The remote server returned an  error: (401) Unauthorized."

Describe the issue
I'm running a pipeline and getting errors on Compile-AppInBCContainer Saying it is not authorized.
Scripts used to create container and cause the issue
Compile-AppInNavContainer -appProjectFolder C:\Agent\_work\4\s\CBC\SWL-Base -credential $Appcredential -containerName BCDEV-ci
Using Symbols Folder: C:\Agent\_work\4\s\CBC\SWL-Base\.alpackages

Full output of scripts
NavContainerHelper is version <IP_ADDRESS>
NavContainerHelper is running as administrator
Host is Windows Server 2019
Docker Client Version is 19.03.8
Docker Server Version is 19.03.8
...

Screenshots

Sorry, but it is impossible to give any kind of help on this issue.
It doesn't say which container is used or how it is created.
As you can imagine, 1000s of pipelines are running every day using this exact command, which indicates that the problem probably isn't in the exact line but in how the container was created, the credentials, the authentication mechanism or other things.

Sorry here is the script I am using to create the container and the raw log from the Create build Container Script.  The PowerShell scripts are in a different repository using a template and checking it out into the build repository to run.  Let me know if you need any more info.
New-BCContainer @Parameters                     -doNotCheckHealth
-updateHosts                     -useBestContainerOS
-containerName BCDEV-ci                     -imageName "mcr.microsoft.com/businesscentral/sandbox:us"
-alwaysPull:$alwaysPull                     -auth "UserPassword"
-Credential $credential                     -additionalParameters @("--network=tlan","--ip <IP_ADDRESS>","--volume ""C:\Agent:C:\Agent""")
-includeTestToolkit                     -includeTestLibrariesOnly
-doNotUseRuntimePackages `
-enableTaskScheduler:$false
2020-06-27T21:37:44.9160102Z ##[section]Starting: Create Build Container
2020-06-27T21:37:44.9275326Z ==============================================================================
2020-06-27T21:37:44.9275689Z Task         : PowerShell
2020-06-27T21:37:44.9275978Z Description  : Run a PowerShell script on Linux, macOS, or Windows
2020-06-27T21:37:44.9276411Z Version      : 2.170.1
2020-06-27T21:37:44.9276881Z Author       : Microsoft Corporation
2020-06-27T21:37:44.9277440Z Help         : https://docs.microsoft.com/azure/devops/pipelines/tasks/utility/powershell
2020-06-27T21:37:44.9277975Z ==============================================================================
2020-06-27T21:37:45.7590041Z Generating script.
2020-06-27T21:37:45.7645041Z Formatted command: . 'C:\Agent_work\4\s\DevOpsScripts\Create-Container.ps1'
2020-06-27T21:37:45.7961036Z ========================== Starting Command Output ===========================
2020-06-27T21:37:45.8168570Z ##[command]"C:\Windows\System32\WindowsPowerShell\v1.0\powershell.exe" -NoLogo -NoProfile -NonInteractive -ExecutionPolicy Unrestricted -Command ". 'C:\Agent_work_temp\1719e7f3-7d6f-4726-ad4d-d2464324bc9b.ps1'"
2020-06-27T21:37:46.1022121Z Create BCDEV-ci from mcr.microsoft.com/businesscentral/sandbox:us
2020-06-27T21:38:22.3137718Z NavContainerHelper is version <IP_ADDRESS>
2020-06-27T21:38:22.3151433Z NavContainerHelper is running as administrator
2020-06-27T21:38:22.3209524Z Host is Microsoft Windows Server 2019 Standard - ltsc2019
2020-06-27T21:38:22.5701908Z Docker Client Version is 19.03.5
2020-06-27T21:38:22.6975224Z Docker Server Version is 19.03.5
2020-06-27T21:38:23.3257699Z Removing C:\ProgramData\NavContainerHelper\Extensions\BCDEV-ci
2020-06-27T21:38:41.4188382Z Using image mcr.microsoft.com/businesscentral/sandbox:us-ltsc2019
2020-06-27T21:38:41.4238411Z Disabling Health Check (always report healthy)
2020-06-27T21:38:41.6322247Z Creating Container BCDEV-ci
2020-06-27T21:38:41.7857635Z Version: 16.2.13509.14155-us
2020-06-27T21:38:41.7878083Z Style: sandbox
2020-06-27T21:38:41.9422909Z Platform: 16.0.13440.14115
2020-06-27T21:38:42.0941284Z Generic Tag: <IP_ADDRESS>
2020-06-27T21:38:42.2389150Z Container OS Version: 10.0.17763.973 (ltsc2019)
2020-06-27T21:38:42.2401539Z Host OS Version: 10.0.17763.107 (ltsc2019)
2020-06-27T21:38:42.2445329Z A better Generic Container OS exists for your host (mcr.microsoft.com/dynamicsnav:10.0.17763.437-generic)
2020-06-27T21:38:42.3986575Z Using generic image mcr.microsoft.com/dynamicsnav:10.0.17763.437-generic
2020-06-27T21:38:42.8215279Z Generic Container OS Version: 10.0.17763.437 (ltsc2019)
2020-06-27T21:38:42.9568634Z Generic Tag of better generic: <IP_ADDRESS>
2020-06-27T21:38:42.9581140Z WARNING: The container operating system matches the host operating system, but the revision is different.
2020-06-27T21:38:42.9596773Z If you encounter issues, you might want to specify -isolation hyperv
2020-06-27T21:38:42.9638662Z Using locale en-US
2020-06-27T21:38:42.9656843Z Using process isolation
2020-06-27T21:38:42.9679029Z Disabling the standard eventlog dump to container log every 2 seconds (use -dumpEventLog to enable)
2020-06-27T21:38:42.9815943Z Using license file C:\License\SWNAVBC.flf
2020-06-27T21:38:43.0674730Z Additional Parameters:
2020-06-27T21:38:43.0773875Z --network=tlan
2020-06-27T21:38:43.0786525Z --ip <IP_ADDRESS>
2020-06-27T21:38:43.0795087Z --volume "C:\Agent:C:\Agent"
2020-06-27T21:38:43.0804285Z --env customNavSettings=EnableTaskScheduler=False
2020-06-27T21:38:43.0814183Z Files in C:\ProgramData\NavContainerHelper\Extensions\BCDEV-ci\my:
2020-06-27T21:38:43.0951116Z - AdditionalOutput.ps1
2020-06-27T21:38:43.0963594Z - license.flf
2020-06-27T21:38:43.0972055Z - MainLoop.ps1
2020-06-27T21:38:43.0983152Z - SetupVariables.ps1
2020-06-27T21:38:43.0995623Z - updatehosts.ps1
2020-06-27T21:38:43.1010648Z Creating container BCDEV-ci from image mcr.microsoft.com/dynamicsnav:10.0.17763.437-generic
2020-06-27T21:38:43.3348455Z 693c40e1223410849351757eedefa52b36368122f76bbd313cbf77b82efa3353
2020-06-27T21:38:44.8189132Z Waiting for container BCDEV-ci to be ready
2020-06-27T21:38:48.2789701Z Installing Business Central
2020-06-27T21:38:57.6825426Z Installing Url Rewrite
2020-06-27T21:38:59.9759722Z Installing OpenXML
2020-06-27T21:39:14.2003862Z Installing DotNetCore
2020-06-27T21:39:16.5373670Z Starting Local SQL Server
2020-06-27T21:39:17.6860551Z Starting Internet Information Server
2020-06-27T21:39:20.0017428Z Copying Service Tier Files
2020-06-27T21:39:34.1784984Z Copying Web Client Files
2020-06-27T21:39:35.3258879Z Copying Client Files
2020-06-27T21:39:36.4723529Z Copying ModernDev Files
2020-06-27T21:39:36.4737458Z Copying PowerShell Scripts
2020-06-27T21:39:36.4751901Z Copying ConfigurationPackages
2020-06-27T21:39:36.4764184Z Copying Test Assemblies
2020-06-27T21:39:37.6296816Z Copying Extensions
2020-06-27T21:39:41.1006362Z Copying Applications
2020-06-27T21:39:42.2729159Z Copying Applications.*
2020-06-27T21:40:02.5896774Z Copying ReportBuilder
2020-06-27T21:40:40.3700309Z Changing Database Server Collation to Latin1_General_100_CS_AS
2020-06-27T21:40:46.3200968Z Copying Cronus database
2020-06-27T21:40:46.3213024Z Modifying Business Central Service Tier Config File for Docker
2020-06-27T21:40:46.3223435Z Creating Business Central Service Tier
2020-06-27T21:40:46.3233312Z Installing SIP crypto provider: 'C:\Windows\System32\NavSip.dll'
2020-06-27T21:40:46.3242628Z Installation took 118 seconds
2020-06-27T21:40:46.3363014Z Installation complete
2020-06-27T21:40:46.3380131Z Initializing...
2020-06-27T21:40:46.3395433Z Setting host.containerhelper.internal to <IP_ADDRESS> in container hosts file (copy from host hosts file)
2020-06-27T21:40:46.3406144Z Starting Container
2020-06-27T21:40:46.3416535Z Hostname is BCDEV-ci
2020-06-27T21:40:47.5258122Z PublicDnsName is BCDEV-ci
2020-06-27T21:40:47.5268822Z Using NavUserPassword Authentication
2020-06-27T21:40:48.6955482Z Creating Self Signed Certificate
2020-06-27T21:40:48.6973992Z Self Signed Certificate Thumbprint BDD2B176C3FB80E288CFF2537FC5B2BC1EFC838B
2020-06-27T21:40:48.6989843Z Modifying Service Tier Config File with Instance Specific Settings
2020-06-27T21:40:48.7005689Z Modifying Service Tier Config File with settings from environment variable
2020-06-27T21:40:48.7022630Z Setting EnableTaskScheduler to False
2020-06-27T21:43:47.9543653Z Starting Service Tier
2020-06-27T21:43:47.9556971Z Registering event sources
2020-06-27T21:44:18.9674449Z Creating DotNetCore Web Server Instance
2020-06-27T21:44:18.9689912Z Enabling Financials User Experience
2020-06-27T21:44:18.9701261Z Using license file 'c:\run\my\license.flf'
2020-06-27T21:44:20.1308163Z Import License
2020-06-27T21:44:20.1320183Z Creating http download site
2020-06-27T21:44:20.1335459Z Setting SA Password and enabling SA
2020-06-27T21:44:21.2932341Z Creating admin as SQL User and add to sysadmin
2020-06-27T21:44:27.2575289Z Creating SUPER user
2020-06-27T21:44:27.2584436Z Container IP Address: <IP_ADDRESS>
2020-06-27T21:44:27.2594112Z Container Hostname  : BCDEV-ci
2020-06-27T21:44:27.2602584Z Container Dns Name  : BCDEV-ci
2020-06-27T21:44:27.2611502Z Web Client          : http://BCDEV-ci/BC/
2020-06-27T21:44:27.2623311Z Dev. Server         : http://BCDEV-ci
2020-06-27T21:44:27.2635845Z Dev. ServerInstance : BC
2020-06-27T21:44:27.2646954Z Setting BCDEV-ci to <IP_ADDRESS> in host hosts file
2020-06-27T21:44:27.2650196Z
2020-06-27T21:44:27.2660000Z Files:
2020-06-27T21:44:27.2670039Z http://BCDEV-ci:8080/ALLanguage.vsix
2020-06-27T21:44:27.2673278Z
2020-06-27T21:44:27.2682130Z Initialization took 221 seconds
2020-06-27T21:44:27.2691632Z Ready for connections!
2020-06-27T21:44:27.2702875Z Reading CustomSettings.config from BCDEV-ci
2020-06-27T21:45:38.3983137Z Copy from container BCDEV-ci (C:\Applications.us\Microsoft_Any_16.2.13509.14155.app) to C:\ProgramData\NavContainerHelper\Extensions\BCDEV-ci_Microsoft_Any_16.2.13509.14155.app
2020-06-27T21:45:39.5696593Z Publishing C:\ProgramData\NavContainerHelper\Extensions\BCDEV-ci_Microsoft_Any_16.2.13509.14155.app
2020-06-27T21:45:43.3767585Z Synchronizing Any on tenant default
2020-06-27T21:45:46.8453334Z Installing Any on tenant default
2020-06-27T21:45:52.8752627Z App successfully published
2020-06-27T21:45:54.5957459Z Copy from container BCDEV-ci (C:\Applications.us\Microsoft_Library Assert_16.2.13509.14155.app) to C:\ProgramData\NavContainerHelper\Extensions\BCDEV-ci_Microsoft_Library Assert_16.2.13509.14155.app
2020-06-27T21:45:55.6479323Z Publishing C:\ProgramData\NavContainerHelper\Extensions\BCDEV-ci_Microsoft_Library Assert_16.2.13509.14155.app
2020-06-27T21:45:56.1099694Z Synchronizing Library Assert on tenant default
2020-06-27T21:45:57.5344173Z Installing Library Assert on tenant default
2020-06-27T21:45:57.6549338Z App successfully published
2020-06-27T21:45:59.3831268Z Copy from container BCDEV-ci (C:\Applications\TestFramework\TestRunner\Microsoft_Test Runner.app) to C:\ProgramData\NavContainerHelper\Extensions\BCDEV-ci_Microsoft_Test Runner.app
2020-06-27T21:46:00.4639711Z Publishing C:\ProgramData\NavContainerHelper\Extensions\BCDEV-ci_Microsoft_Test Runner.app
2020-06-27T21:46:01.6606791Z Synchronizing Test Runner on tenant default
2020-06-27T21:46:02.0679377Z Installing Test Runner on tenant default
2020-06-27T21:46:02.4870336Z App successfully published
2020-06-27T21:46:04.1967031Z Copy from container BCDEV-ci (C:\Applications.us\Microsoft_System Application Test Library_16.2.13509.14155.app) to C:\ProgramData\NavContainerHelper\Extensions\BCDEV-ci_Microsoft_System Application Test Library_16.2.13509.14155.app
2020-06-27T21:46:05.2811014Z Publishing C:\ProgramData\NavContainerHelper\Extensions\BCDEV-ci_Microsoft_System Application Test Library_16.2.13509.14155.app
2020-06-27T21:46:06.5216914Z Synchronizing System Application Test Library on tenant default
2020-06-27T21:46:07.8990945Z Installing System Application Test Library on tenant default
2020-06-27T21:46:08.2355461Z App successfully published
2020-06-27T21:46:09.9758442Z Copy from container BCDEV-ci (C:\Applications.us\Microsoft_Tests-TestLibraries_16.2.13509.14155.app) to C:\ProgramData\NavContainerHelper\Extensions\BCDEV-ci_Microsoft_Tests-TestLibraries_16.2.13509.14155.app
2020-06-27T21:46:11.0794325Z Publishing C:\ProgramData\NavContainerHelper\Extensions\BCDEV-ci_Microsoft_Tests-TestLibraries_16.2.13509.14155.app
2020-06-27T21:46:33.9310317Z Synchronizing Tests-TestLibraries on tenant default
2020-06-27T21:46:36.2965848Z Installing Tests-TestLibraries on tenant default
2020-06-27T21:46:40.1796532Z App successfully published
2020-06-27T21:46:40.1807454Z TestToolkit successfully imported
2020-06-27T21:46:40.1832690Z Container BCDEV-ci successfully created
2020-06-27T21:46:41.1792631Z Backing up CRONUS to C:\ProgramData\NavContainerHelper\onprem-16.2.13509.14155-us-bakFolders\BCDEV-ci\database.bak
2020-06-27T21:46:45.6760868Z ##[section]Finishing: Create Build Container

A few things to try out

the compile step uses $appCredential - are you 100% sure that this is the same as the $credential you use when creating the container?
if that is ok, I would try to remove the special things - the transparent networking - try to remove that and see if that changes anything.
It seems like your host hasn't been updated - you are running a very old version of 1809, for which there is no matching generic image - this is likely not the issue, but I would update my host

This is working now.  I updated Windows to the latest updates and then download and then had to restart the container, but now it's working.  This can be closed.


Correct test cases for Naming/VariableNumber

Follow https://github.com/rubocop-hq/rubocop/pull/8727#discussion_r489470828
This PR corrects two test cases for Naming/VariableNumber.
Thanks @dvandersluis!

Before submitting the PR make sure the following are checked:

[x] Wrote good commit messages.
[ ] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[ ] Added an entry (file) to the changelog folder named {change_type}_{change_description}.md if the new code introduces user-observable changes. See changelog entry format for details.
[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Run bundle exec rake default. It executes all tests and RuboCop for itself, and generates the documentation.

Oh good! I forgot about this, thanks @koic!

I overlooked it. Thank you very much!


When i push to new screen, tabbar disappears.

Tabbar is only shown in first screen, i dont hide tabbar, but when push tabbar stay on old screen, does not cross to new screen. Anyone face that problem?

In SwiftUI, you determine this behavior by putting the NavigationStack inside or outside of the TabView. This is the same behavior for using native NavigationStack or RouterView...
// Each tab has a NavigationStack and does not hide tabbar
TabView {
     RouterView { router in
          Text("Screen1")
     }
}

// The entire tabbar has one NavigationStack that does hide tabbar
RouterView { router in
     TabView {
          Text("Screen1")
     }
}

Thanks for quick response , i will check this

I think after IOS 18, this problem appeared. I tested your code, it does not work. But this case worked before IOS 18, did you test your code in any project

Problem is on showScreen(.push) it does not work correctly

Can you provide sample code example that does not work?

there are a lot of codes.  I found problem, problem is when i put
RouterView { router in
TabsView(viewModel: TabsViewModel())
}
problem occur.
But when i use
TabsView(viewModel: TabsViewModel())

it works.
I dont understand, why?

After 16 hours of testing all cases. i found how to solve.
RouterView(addNavigationView: false) { router in
}
this solved problem.

That's explained in the ReadMe if you want some background...
That means you probably still have NavigationStack in your code. If you delete your NavigationStack and use only RouterView, you won't need to set addNavigationView: false.

But i only use  RouterView.

Check your whole codebase for NavigationStack? If you link the repo I can look.

i search for NavigationStack, but nothing is found. Repo is private and belong to company. But i never use NavigationStack , i only use your library. I think problem is nested RouterView . Becuase in TabView i have 4 view with RouterView. And when i put TabsView(viewModel: TabsViewModel()) inside RouterView. Problem appeared. I think problem is nested RouterView

You should only have 1 RouterView per view hierarchy, there would be no need to add a 2nd. If you did, then I guess you can set addNavigationView: false but it shouldn't be necessary to add.

Without addNavigationView: false, adding a RouterView is the same as adding a NavigationStack

But i have add button in tab, and i show AddNewPost view, from TabsView(viewModel: TabsViewModel())'s router. And I have  other four tabs in tabview.  when is set this addNavigationView to false , add dont work. when dont set , tabbar cant disappear.  i start to search new solution for this. Do you have an idea for that?

Sorry, I can't tell what the problem is without seeing the code. I've built many MVVM apps using this so there must be something unique to your setup. I would start by only having 1 RouterView per view hierarchy.

I use fullscreencover for add button in tabview. Thanks for help)

@SwiftfulThinking Hi again, when i user showModal it did not overlay tabbar, the modal start after tabbar , example: fullScreenCover overlay tabbar. How to achieve this in showModal?


Interface setting "Draw background" appears twice in settings

On the Interface Settings page, the setting "Draw background" appears twice -- once after "Show current date and time" and once at the very end of the list. This is in MaterialSkin 4.0.4 on Chrome Version 120.0.6099.71 (Official Build) (64-bit) on Linux.

That's because there's multiple options. These control background drawing on separate areas.

Ah, now I see the section headers "Now Playing" and "Queue". I didn't notice them before -- they don't stand out. Maybe make section headers a larger size or a different color? Just a thought.


PR for CI test purposes, DO NOT MERGE

DO NOT MERGE

Hi @mlesnic, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;

PR is added to the queue for testing as 6. task. (62167dd)

Test (Fail) 82861722 build is completed (62167dd)
See https://github.com/ThaliTester/TestResults/tree/8286172262167dd_PR_for_CI_test_purposes__DO_NOT_MERGE_mlesnic/ for the logs

PR is added to the queue for testing as 4. task. (c8396da)

Test (Fail) 82861722 build is completed (c8396da)
See https://github.com/ThaliTester/TestResults/tree/82861722c8396da_PR_for_CI_test_purposes__DO_NOT_MERGE_mlesnic/ for the logs


rake requires 1.9.3 starting with v11.0.0

This should make travis green again on ruby 1.9.2 and below

This has passed Travis now. Please merge so new branches can get accurate Travis runs.
Thanks!

@jeremy or @bf4 can you please merge this or something like it to bring master back to green so that future PRs can be properly evaluated for errors.

Yes.
On Fri, Apr 1, 2016 at 07:25 Kevin Glowacz<EMAIL_ADDRESS>wrote:

@jeremy https://github.com/jeremy or @bf4 https://github.com/bf4 can
you please merge this or something like it to bring master back to green so
that future PRs can be properly evaluated for errors.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/mikel/mail/pull/979#issuecomment-204416032

Backported to 2-6-stable


Add filter logic to SearchRequestAdapter

Change Summary
Updated logic so filters can be used in addition to facet filters.  My original code with AND and OR based filters works just the way it did with Algolia.
PR Checklist

[x] I have read and signed the Contributor License Agreement.

Added filter logic and additionalSearchParameters to SearchRequestAdapter and Configuration


Dependency System

Currently it is fairly difficult and hacky to allow scripts to depend on other scripts. I propose a dependency system to allow scripts access to their dependent scripts app domains as well as loading scripts based on a hierarchical dependency structure.

@JohnnyCrazy @crosire is there anyway I can contact you guys outside of here?

We have a channel on slack, https://scripthookvdotnet.slack.com/

Thank you sir!

@zorg93 Can somebody invite me<EMAIL_ADDRESS>

@Dakota628 think you'll have to wait for @JohnnyCrazy to invite you

Done


There may be an incorrect reaction to user actions in the interface

Godot version
4.0 beta10
System information
Fedora Linux 37 (W.E.) GNOME 43.2
Issue description
CollisionObject2D click the Mask property and uncheck the first checkbox. This ensures that mobs will not collide with each other. This is recommended in the documentation for action

After it did not occur to edit the CollisionObject2D mask correctly, it is not visible
Steps to reproduce
The documentation and documentation actions are described above
Minimal reproduction project
"N / A"

This is visibility layer, not physics layer

Closing per @Zireael07's comment.


Arrow Head Size

If its not too much work, i think a size option for the arrow head would be useful.
If you annotate with arrows with pen size 2 in a zoomed in view, often the arrow head lines are too big in proportion to the length.
If you want to get really fancy, an additional "auto" option would be amazing where the arrow head scales with the line length.
Also for zoomed in view, it would be helpful to have pen sizes below 2.

Please try v1.0.9 that allows pixel size of 1 and arrow heads automatically calculated from the arrow length:
http://www.sourceforge.net/projects/mrv2/files/beta

The arrow head feature is awesome. Thanks for that!
But i had no luck with the pen size. The width of any drawing tool stays the same as setting 2.
So changing it to 1 (and even 0) produces the same width as 2.

And i think i've found a bug that happens with saving and loading the session with these new annotation changes.
SAving is without error, but when i load the session again, the log says:
ERROR: [mrv2s] Invalid OCIO Ics ''.

And i think i've found a bug that happens with saving and loading the session with these new annotation changes.
SAving is without error, but when i load the session again, the log says:
ERROR: [mrv2s] Invalid OCIO Ics ''.
And no annotations are loded.

It is a bug with the new Input Color Space flag kept on each image.  I'll have a fix later.

Both the 1 pixel cursor and the broken Session loading should be fixed in the latest beta in sourceforge.  Give it a try and report back.

Broken session loading is fixed. 👍
Still no luck with Pen Size 1.

Another bug with the rotate function is the placement of text. After you created the text and clicked on another tool (i have not yet discovered a different way of confirming the text input), the text will be set in a different place than you entered it.
I've also not discovered how to move already placed annotations.

Still no luck with Pen Size 1.

You may need to tell me OS, what resolution, what tool and if using soft or hard pencil, as it works for me.  Here's a zoom view with an arrow of one pixel.

After you created the text and clicked on another tool (i have not yet discovered a different way of confirming the text input).

You can click on another tool or click on the green X next in the corner of box input.  You can hit Esc or delete all text and hit the now red X in the cordner.

the text will be set in a different place than you entered it.

That's a feature, not a bug.  You can, at a later time, reposition the text by using the RMB on the text tool.  A requester will show allowing you to select which text you want to edit and if you click ok, you will be back at the editing mode of the text to change its contents.

I've also not discovered how to move already placed annotations.

That currently cannot be done.  It's not like we have Photoshop with layers!  Albeit the data is stored, so I may add that feature at a later time (the UI aspect of it is tricky -- like show the annotation with a bounding box and then use some hotkey to go to the next or previous annotation.  The problem is with Erased annotation, where you would not see anything).

After you created the text and clicked on another tool (i have not yet discovered a different way of confirming the text input).

I am compiling a version now where text input can be confirmed by pressing SHIFT + Enter.  It should be in sourceforge beta in an hour if you are on Windows, in a couple of hours if you are on Linux or macOS.

Now this looks way better. Finer on Pen Size 1 and i even can enter Pen Size 0 to go a bit finer still. Im super happy. Thanks 👍
(btw im on Win10, 1440p)

the text will be set in a different place than you entered it.

That's a feature, not a bug. You can, at a later time, reposition the text by using the RMB on the text tool.

I meant that in the rotated view the text gets teleported to a different position after you confirmed it. Here is a recording:

(this is with the newest version)

New betas fixing the text problem:
.exe:
https://mega.nz/file/vW5ViQJK#jGkzUAmjyoFB__Skt6ymtgM9EGJq1T1K3AW_wztFA9E
.zip:
https://mega.nz/file/vPRSFZqA#I3yv0I7ZGn4KkBSw-0ZHCPeDrBDPyA39qOfQFwWxR2s

All fixed and addressed in v1.1.0.


write_verilog: dump zero width sigspecs correctly

See commit 4ff44d85a5 for details.

Are you sure that's correct? from the very chapter you refered to:
"A replication operation may have a replication constant with a value of zero. This is useful in parameterized code. A replication with a zero replication constant is considered to have a size of zero and is ignored. Such a replication shall appear only within a concatenation in which at least one of the operands of the concatenation has a positive size."
It seems we're not in a concatenation?

You're absolutely right: this isn't a legal transformation. Thanks for catching this.
The problem here is that the current code is blatantly wrong, too.

There are essentially two cases here:

Empty sigchunk in a sigspec. That can be just skipped, and AFAICT already gets skipped (or at least I can't construct a pathological input in textual IR).
Empty sigspec.

Thoughts?

I came to the same conclusion: verilog really doesn't like 0-bit assignments, so you have to skip the assignment entirely, and there's probably no way around it.

Right. I mostly mean the specific approach to skipping them. Ideally without modifying half of write_verilog.

Could you do a prepass which eliminates the zero entries before running the write_verilog pass?

Yes, but it's not ideal because AFAIK there's no Yosys pass which does just that, and e.g. for nMigen it's important for the Verilog output to stay close to the RTLIL it emits. (nMigen is not affected by this issue directly because it strips out assignments to zero width sigspecs on its own, effectively implementing the pass you suggest.)


Added username/password support for nodetool - cassandra-graphite.rb - Cassandra monitoring

We can now run cassandra-graphite.rb with -u/-p options for those using
authentication on the JMX.This is best practice considering the new recent
vulnerability affecting Cassandra:
CVE-2015-0225 ( http://www.securityfocus.com/archive/1/535154 )

@lcrisci
Thanks for this but I have a personal problem with taking usernames and passwords as commandline args.
What are your thoughts other options such as reading them in from a file or another way.  As I said this is a personal preference so i will merge this in without issue but want to get thoughts on it as well.

Hi Matty,
Good point and this feature is actually supported since Cassandra 2.1:
https://issues.apache.org/jira/browse/CASSANDRA-6660
I will amend my patch and add an option to also be able to read from the password file.
The old username/password mechanism should still be in the patch since old nodetool softwares are only able to get username/password from the cli.
As long as people restrict the role/permissions of the nodetool user, it should still be safe to use.

@lcrisci
I am ok with leaving it there as well.  I am not about to start fixing human issues with code.  If they want to use the commandline for passwords and such so be it but if a tool supports other methods they should be incorporated for those that use them.

@lcrisci
Could also pull against sensu-plugins-cassandra
Thanks


Command disappear switching from default value to empty

Lapce Version
v0.2.1
System information
Arch Linux
Describe the bug
Removing the default keybind for a Command makes the Command disappear from the Keybindings list
Additional information
I wanted to remove the default keybind for a Command, so I clicked on the keybind combination, the window to re-assign it appeared and I clicked on Save leaving the field empty. After that the Command disappeared from the list.
Looking in the keymaps.toml file the following entry appeared:
[[keymaps]]
command = "-insert_cursor_end_of_line"
mode = "i"
key = "Alt+Shift+i"

If I delete this entry from keymaps.toml the commands appeared again in the Command list with its default value, in this case Alt+Shift+i.

With Command I mean for example insert_cursor_end_of_line.
I'm referring it as Command because of the name of the column in the table in Keybindings side menu in Settings.

This happened for me also.
Steps to reproduce:

go to settings (CMD + ,)
in the Keybindings menu and click any command
click save (without typing anything)

actual behavior:
command disappears from the list and cant find again. But it is accessible from command plate.
expected:
I should be able to see the the command if I search again.
Lapce version: 0.2.5
system: macOS Ventura 13.2 (M1 pro)


deleteItem(...).where(...) does not seem to work

Investigate why a where clause added on to a deleteItem does not seem to work. It may be worth adding a test for this in the LiveSpec IT test

Do you have a specific example of this that is failing? I added a test to the LiveSpec but it's passing currently without any code changes.

I have added a test to my branch here https://github.com/zio/zio-dynamodb/blob/ro_for_pe_type_param_2/dynamodb/src/it/scala/zio/dynamodb/LiveSpec.scala#L157-L173 to reproduce the behaviour.
        testM("delete item with where clause") {
          withDefaultTable { tableName =>
            for {
              _     <- deleteItem(tableName, pk(aviItem))
                         .where(
                           ConditionExpression.Equals(
                             ConditionExpression.Operand.ProjectionExpressionOperand($(number)),
                             ConditionExpression.Operand.ValueOperand(
                               AttributeValue(42)
                             ) // this should fail delete as number is 1
                           )
                         )
                         .execute
              after <- getItem(tableName, pk(aviItem)).execute
            } yield assert(after)(isNone)
          }
        },

After doing some investigation I believe this is due to the DeleteItem request being turned into a BatchWriteItem, which does not have condition expressions. We may need to update the auto-batching to check to see if there is a request that contains a condition expression.

Many thanks for investigating this. I will try and have a look at BatchWriteItem later on in the week

that would be great


[not urgent] enable treesitter parsing by default

I've been thinking about how to start enabling tree-sitter parsing by default, and I think this might be a good way to go about it - the treesitter option is set to true by default if the user has an http parser installed, and documentation encourages the user to try setting this option to false if they encounter an issue.
No rush to merge this, but curious about thoughts on the approach

I want to know @Grueslayer 's take on that.
I'm glad that we don't rely on ts at all for parsing, so we can implement features completely independent on our own, but maybe I'm just an old grumpy man :see_no_evil:

As long as @gorillamoe does not take care of a correct and up-to-date treesitter parser matching the Kulala syntax, I think parsing by TS should be opt-in only.
Let's have a look at this running (stripped down but looks identically with all request variables used) example:

The URLs are not correct parsed by treesitter: Kulala will (like vscode-rest-client ans others) urlencode query parameters so we can use spaces, ampersands in it's native way. TS http parser at least has a problem with the single quotes here. Maybe in that case it's cosmetic only.

To be honest, when Kulala started in v1, we had full reliance on TS and that's when so many people complained about stuff not working as expected, because TS was error'ing even for basic stuff.
That days might be gone and stuff might be fixed, but the biggest downside I see is that when we want to add additional stuff that e.g. jetbrains supports, but isn't part of the http grammar, we can't just implement and ship it, because we have to wait for our PR to get approved. And worst thing would be that the PR gets rejected, which would be totally fair, because http grammar is part of the rest.nvim project, so they can decide what goes in.

my only real opinion (as a very new user and contributor to this project) - and the reason why I felt like utilizing treesitter here would be a worthwhile effort - is that rolling your own parsing logic with only the tools available in vimscript/lua seems unpleasant and tree-sitter makes adding/changing parser logic a lot more straightfoward, especially with a lot of the groundwork already done.
in my experience contributing to the http parser, @boltlessengineer seems very amenable to aligning with the jetbrains spec and has been quick to get my changes merged so I don't really think there would be too much friction in the future if more changes were needed to add important features to this project.

Okay, let's try this then, but first we need to make sure gql in http in  treesitter. I use it on a daily basis and this needs to work for me reliable.
When working on https://github.com/rest-nvim/rest.nvim/pull/452, I noticed that at least one of my queries failed in rest.nvim. I haven't checked if it's due to treesitter or just related to rest.nvim.
Queries are from work stuff, so I can't exactly share it as it is, but probably supply a repro example.
I think it's due to variable replacement.
I also noticed rest and kulala are quite different 🤔 which is a good thing on its own, but I struggled to get my login query to work.
Maybe I did something wrong, but named requests with response.body.$.token wasn't working for me.
Isn't this (yet) supported by rest @NTBBloodbath @boltlessengineer?
I'm referring to this here: https://kulala.mwco.app/docs/usage/request-variables

Maybe @Grueslayer can also did some quick tests on his stuff with treesitter enabled.

named requests with response.body.$.token wasn't working for me.

rest.nvim won’t support them because thats not a thing from intellij’s spec. Maybe it will be an opt-in feature in future.
Also, you won’t get answer by asking to ntb. He is completely out of rest.nvim development now.

rest.nvim won’t support them because thats not a thing from intellij’s spec. Maybe it will be an opt-in feature in future.

Ah, thanks for the clarification. I thought that it came from intellij, but maybe it's just a thing form the vscode-rest-client by @Huachao :thinking:


Improved Template

Re-Improved our template

Added analysis_options.yaml for server/
Added pubspec.yaml to template to mananage template deps
Tightened analysis_options.yaml in template folder

@passsy is this approach better than adding the deps to dev_dependencies?
And any more feedback from you?

pubspec.yml still contains the dev_dependencies 🤔

thanks, forgot to push


UI login with wrong credentials shows misleading error message

entering a wrong username and/or password simply results in "Middleware Api Error:"
Should rather say "login failed" ideally in the default language.

fixed


Task panics hang

I tried to add a panic handler similar to the following:
                    let send = send.clone();
                    scope.spawn(move |_| {
                        let panicked = std::panic::catch_unwind(move || {
                            let result = work.exec(work_context);
                            if let Err(e) = send.send((id.clone(), result)) {
                                error!("Unable to write {:?} to completion channel: {}", id, e);
                            }
                        }).is_err();

                        if panicked {
                            let result = Err(match &id {
                                AnyWorkId::Fe(..) => AnyWorkError::Fe(FeError::Panic),
                                AnyWorkId::Be(..) => AnyWorkError::Be(BeError::Panic),
                            });
                            if let Err(e) = send.send((id.clone(), result)) {
                                error!("Unable to write {:?} to completion channel: {}", id, e);
                            }
                        }
                    })

And got a 180 line error that would make a C++ compiler proud:
error[E0277]: the type `(dyn Work<fontir::orchestration::Context, WorkError> + Send + 'static)` may not be safely transferred across an unwind boundary
   --> fontc/src/main.rs:407:65
    |
407 |                           let panicked = std::panic::catch_unwind(move || {
    |  ________________________________________------------------------_^
    | |                                        |
    | |                                        required by a bound introduced by this call
408 | |                             let result = work.exec(work_context);
409 | |                             if let Err(e) = send.send((id.clone(), result)) {
410 | |                                 error!("Unable to write {:?} to completion channel: {}", id, e);
411 | |                             }
412 | |                         }).is_err();
    | |_________________________^ `(dyn Work<fontir::orchestration::Context, WorkError> + Send + 'static)` may not be safely transferred across an unwind boundary
    |
    = help: the trait `UnwindSafe` is not implemented for `(dyn Work<fontir::orchestration::Context, WorkError> + Send + 'static)`
    = note: required for `Unique<(dyn Work<fontir::orchestration::Context, WorkError> + Send + 'static)>` to implement `UnwindSafe`
    = note: required because it appears within the type `Box<(dyn Work<fontir::orchestration::Context, WorkError> + Send + 'static)>`
    = note: required because it appears within the type `AnyWork`
note: required because it's used within this closure
   --> fontc/src/main.rs:407:65
    |
407 |                         let panicked = std::panic::catch_unwind(move || {
    |                                                                 ^^^^^^^
note: required by a bound in `catch_unwind`
   --> ~/.rustup/toolchains/stable-x86_64-unknown-linux-gnu/lib/rustlib/src/rust/library/std/src/panic.rs:136:40
    |
136 | pub fn catch_unwind<F: FnOnce() -> R + UnwindSafe, R>(f: F) -> Result<R> {
    |                                        ^^^^^^^^^^ required by this bound in `catch_unwind`

error[E0277]: the type `(dyn Work<fontbe::orchestration::Context, fontbe::error::Error> + Send + 'static)` may not be safely transferred across an unwind boundary
   --> fontc/src/main.rs:407:65
    |
407 |                           let panicked = std::panic::catch_unwind(move || {
    |  ________________________________________------------------------_^
    | |                                        |
    | |                                        required by a bound introduced by this call
408 | |                             let result = work.exec(work_context);
409 | |                             if let Err(e) = send.send((id.clone(), result)) {
410 | |                                 error!("Unable to write {:?} to completion channel: {}", id, e);
411 | |                             }
412 | |                         }).is_err();
    | |_________________________^ `(dyn Work<fontbe::orchestration::Context, fontbe::error::Error> + Send + 'static)` may not be safely transferred across an unwind boundary
    |
    = help: the trait `UnwindSafe` is not implemented for `(dyn Work<fontbe::orchestration::Context, fontbe::error::Error> + Send + 'static)`
    = note: required for `Unique<(dyn Work<fontbe::orchestration::Context, fontbe::error::Error> + Send + 'static)>` to implement `UnwindSafe`
    = note: required because it appears within the type `Box<(dyn Work<fontbe::orchestration::Context, fontbe::error::Error> + Send + 'static)>`
    = note: required because it appears within the type `AnyWork`
note: required because it's used within this closure
   --> fontc/src/main.rs:407:65
    |
407 |                         let panicked = std::panic::catch_unwind(move || {
    |                                                                 ^^^^^^^
note: required by a bound in `catch_unwind`
   --> ~/.rustup/toolchains/stable-x86_64-unknown-linux-gnu/lib/rustlib/src/rust/library/std/src/panic.rs:136:40
    |
136 | pub fn catch_unwind<F: FnOnce() -> R + UnwindSafe, R>(f: F) -> Result<R> {
    |                                        ^^^^^^^^^^ required by this bound in `catch_unwind`

error[E0277]: the type `UnsafeCell<std::option::Option<Arc<fontir::ir::StaticMetadata>>>` may contain interior mutability and a reference may not be safely transferrable across a catch_unwind boundary
   --> fontc/src/main.rs:407:65
    |
407 |                           let panicked = std::panic::catch_unwind(move || {
    |  ________________________________________------------------------_^
    | |                                        |
    | |                                        required by a bound introduced by this call
408 | |                             let result = work.exec(work_context);
409 | |                             if let Err(e) = send.send((id.clone(), result)) {
410 | |                                 error!("Unable to write {:?} to completion channel: {}", id, e);
411 | |                             }
412 | |                         }).is_err();
    | |_________________________^ `UnsafeCell<std::option::Option<Arc<fontir::ir::StaticMetadata>>>` may contain interior mutability and a reference may not be safely transferrable across a catch_unwind boundary
    |
    = help: within `lock_api::rwlock::RwLock<parking_lot::raw_rwlock::RawRwLock, std::option::Option<Arc<fontir::ir::StaticMetadata>>>`, the trait `RefUnwindSafe` is not implemented for `UnsafeCell<std::option::Option<Arc<fontir::ir::StaticMetadata>>>`
    = note: required because it appears within the type `lock_api::rwlock::RwLock<parking_lot::raw_rwlock::RawRwLock, std::option::Option<Arc<fontir::ir::StaticMetadata>>>`
    = note: required for `Arc<lock_api::rwlock::RwLock<parking_lot::raw_rwlock::RawRwLock, std::option::Option<Arc<fontir::ir::StaticMetadata>>>>` to implement `UnwindSafe`
    = note: required because it appears within the type `fontir::orchestration::Context`
    = note: required because it appears within the type `AnyContext`
note: required because it's used within this closure
   --> fontc/src/main.rs:407:65
    |
407 |                         let panicked = std::panic::catch_unwind(move || {
    |                                                                 ^^^^^^^
note: required by a bound in `catch_unwind`
   --> ~/.rustup/toolchains/stable-x86_64-unknown-linux-gnu/lib/rustlib/src/rust/library/std/src/panic.rs:136:40
    |
136 | pub fn catch_unwind<F: FnOnce() -> R + UnwindSafe, R>(f: F) -> Result<R> {
    |                                        ^^^^^^^^^^ required by this bound in `catch_unwind`

error[E0277]: the type `UnsafeCell<HashMap<GlyphName, Arc<fontir::ir::Glyph>>>` may contain interior mutability and a reference may not be safely transferrable across a catch_unwind boundary
   --> fontc/src/main.rs:407:65
    |
407 |                           let panicked = std::panic::catch_unwind(move || {
    |  ________________________________________------------------------_^
    | |                                        |
    | |                                        required by a bound introduced by this call
408 | |                             let result = work.exec(work_context);
409 | |                             if let Err(e) = send.send((id.clone(), result)) {
410 | |                                 error!("Unable to write {:?} to completion channel: {}", id, e);
411 | |                             }
412 | |                         }).is_err();
    | |_________________________^ `UnsafeCell<HashMap<GlyphName, Arc<fontir::ir::Glyph>>>` may contain interior mutability and a reference may not be safely transferrable across a catch_unwind boundary
    |
    = help: within `lock_api::rwlock::RwLock<parking_lot::raw_rwlock::RawRwLock, HashMap<GlyphName, Arc<fontir::ir::Glyph>>>`, the trait `RefUnwindSafe` is not implemented for `UnsafeCell<HashMap<GlyphName, Arc<fontir::ir::Glyph>>>`
    = note: required because it appears within the type `lock_api::rwlock::RwLock<parking_lot::raw_rwlock::RawRwLock, HashMap<GlyphName, Arc<fontir::ir::Glyph>>>`
    = note: required for `Arc<lock_api::rwlock::RwLock<parking_lot::raw_rwlock::RawRwLock, HashMap<GlyphName, Arc<fontir::ir::Glyph>>>>` to implement `UnwindSafe`
    = note: required because it appears within the type `fontir::orchestration::Context`
    = note: required because it appears within the type `AnyContext`
note: required because it's used within this closure
   --> fontc/src/main.rs:407:65
    |
407 |                         let panicked = std::panic::catch_unwind(move || {
    |                                                                 ^^^^^^^
note: required by a bound in `catch_unwind`
   --> ~/.rustup/toolchains/stable-x86_64-unknown-linux-gnu/lib/rustlib/src/rust/library/std/src/panic.rs:136:40
    |
136 | pub fn catch_unwind<F: FnOnce() -> R + UnwindSafe, R>(f: F) -> Result<R> {
    |                                        ^^^^^^^^^^ required by this bound in `catch_unwind`

error[E0277]: the type `UnsafeCell<std::option::Option<Arc<fontir::ir::Features>>>` may contain interior mutability and a reference may not be safely transferrable across a catch_unwind boundary
   --> fontc/src/main.rs:407:65
    |
407 |                           let panicked = std::panic::catch_unwind(move || {
    |  ________________________________________------------------------_^
    | |                                        |
    | |                                        required by a bound introduced by this call
408 | |                             let result = work.exec(work_context);
409 | |                             if let Err(e) = send.send((id.clone(), result)) {
410 | |                                 error!("Unable to write {:?} to completion channel: {}", id, e);
411 | |                             }
412 | |                         }).is_err();
    | |_________________________^ `UnsafeCell<std::option::Option<Arc<fontir::ir::Features>>>` may contain interior mutability and a reference may not be safely transferrable across a catch_unwind boundary
    |
    = help: within `lock_api::rwlock::RwLock<parking_lot::raw_rwlock::RawRwLock, std::option::Option<Arc<fontir::ir::Features>>>`, the trait `RefUnwindSafe` is not implemented for `UnsafeCell<std::option::Option<Arc<fontir::ir::Features>>>`
    = note: required because it appears within the type `lock_api::rwlock::RwLock<parking_lot::raw_rwlock::RawRwLock, std::option::Option<Arc<fontir::ir::Features>>>`
    = note: required for `Arc<lock_api::rwlock::RwLock<parking_lot::raw_rwlock::RawRwLock, std::option::Option<Arc<fontir::ir::Features>>>>` to implement `UnwindSafe`
    = note: required because it appears within the type `fontir::orchestration::Context`
    = note: required because it appears within the type `AnyContext`
note: required because it's used within this closure
   --> fontc/src/main.rs:407:65
    |
407 |                         let panicked = std::panic::catch_unwind(move || {
    |                                                                 ^^^^^^^
note: required by a bound in `catch_unwind`
   --> ~/.rustup/toolchains/stable-x86_64-unknown-linux-gnu/lib/rustlib/src/rust/library/std/src/panic.rs:136:40
    |
136 | pub fn catch_unwind<F: FnOnce() -> R + UnwindSafe, R>(f: F) -> Result<R> {
    |                                        ^^^^^^^^^^ required by this bound in `catch_unwind`

error[E0277]: the type `UnsafeCell<std::option::Option<Arc<Vec<u8>>>>` may contain interior mutability and a reference may not be safely transferrable across a catch_unwind boundary
   --> fontc/src/main.rs:407:65
    |
407 |                           let panicked = std::panic::catch_unwind(move || {
    |  ________________________________________------------------------_^
    | |                                        |
    | |                                        required by a bound introduced by this call
408 | |                             let result = work.exec(work_context);
409 | |                             if let Err(e) = send.send((id.clone(), result)) {
410 | |                                 error!("Unable to write {:?} to completion channel: {}", id, e);
411 | |                             }
412 | |                         }).is_err();
    | |_________________________^ `UnsafeCell<std::option::Option<Arc<Vec<u8>>>>` may contain interior mutability and a reference may not be safely transferrable across a catch_unwind boundary
    |
    = help: within `lock_api::rwlock::RwLock<parking_lot::raw_rwlock::RawRwLock, std::option::Option<Arc<Vec<u8>>>>`, the trait `RefUnwindSafe` is not implemented for `UnsafeCell<std::option::Option<Arc<Vec<u8>>>>`
    = note: required because it appears within the type `lock_api::rwlock::RwLock<parking_lot::raw_rwlock::RawRwLock, std::option::Option<Arc<Vec<u8>>>>`
    = note: required for `Arc<lock_api::rwlock::RwLock<parking_lot::raw_rwlock::RawRwLock, std::option::Option<Arc<Vec<u8>>>>>` to implement `UnwindSafe`
    = note: required because it appears within the type `fontbe::orchestration::Context`
    = note: required because it appears within the type `AnyContext`
note: required because it's used within this closure
   --> fontc/src/main.rs:407:65
    |
407 |                         let panicked = std::panic::catch_unwind(move || {
    |                                                                 ^^^^^^^
note: required by a bound in `catch_unwind`
   --> ~/.rustup/toolchains/stable-x86_64-unknown-linux-gnu/lib/rustlib/src/rust/library/std/src/panic.rs:136:40
    |
136 | pub fn catch_unwind<F: FnOnce() -> R + UnwindSafe, R>(f: F) -> Result<R> {
    |                                        ^^^^^^^^^^ required by this bound in `catch_unwind`

For more information about this error, try `rustc --explain E0277`.
error: could not compile `fontc` due to 6 previous errors

todo!() in async work is unfun because of this

Fixed by #371


[FLINK-14560] [runtime] The value of taskmanager.memory.size in flink-conf.yaml is set to zero will cause taskmanager not to work

What is the purpose of the change
In order to fix the issue FLINK-14560
Brief change log
ConfigurationParserUtils.java : I modify the logical in getManagedMemorySize() to obtain a correct result when TaskManagerOptions.LEGACY_MANAGED_MEMORY_SIZE is set to zero.
TaskManagerServices.java : I modify the condition in getManagedMemoryFromHeapAndManaged() to calculate the taskmanager.memory.size according to fraction  even when taskmanager.memory.size is set to zero .
TaskManagerHeapSizeCalculationJavaBashTest.java : Add a scenario with taskmanager.memory.size equal to 0
Verifying this change
(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
TaskManagerHeapSizeCalculationJavaBashTest.compareHeapSizeShellScriptWithJava()
NettyShuffleEnvironmentConfigurationTest.calculateHeapSizeMB()
NetworkBufferCalculationTest.calculateNetworkBufFromHeapSize()
YarnConfigurationITCase.testFlinkContainerMemory()
This change added tests and can be verified as follows:
By setting taskmanager.memory.size : 0
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): ( no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (yes)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

Thanks for opening this PR @faaronzheng. It took me a while to understand the underlying problem. Instead of defining 0 to be the "take the fraction" value, I would suggest to remove the default value from LEGACY_MANAGED_MEMORY_SIZE. If the user should set this value to 0, then it should fail as it represents an invalid system configuration.

LEGACY_MANAGED_MEMORY_SIZE is already removed the default value, I think it is okay to remove the default value from MANAGED_MEMORY_SIZE in release-1.9 and branch

@faaronzheng I think LEGACY_MANAGED_MEMORY_SIZE still has a default value: https://github.com/apache/flink/blob/master/flink-core/src/main/java/org/apache/flink/configuration/TaskManagerOptions.java#L195.
I would propose the following: Let's remove the default value from LEGACY_MANAGED_MEMORY_SIZE in master and from MANAGED_MEMORY_SIZE in release-1.9. Then let's update the code to simply check whether (LEGACY_)MANAGED_MEMORY_SIZE has been set or not instead of comparing the default values with each other. Could you update this PR accordingly and open another one against release-1.9?

@faaronzheng I think LEGACY_MANAGED_MEMORY_SIZE still has a default value: https://github.com/apache/flink/blob/master/flink-core/src/main/java/org/apache/flink/configuration/TaskManagerOptions.java#L195.
I would propose the following: Let's remove the default value from LEGACY_MANAGED_MEMORY_SIZE in master and from MANAGED_MEMORY_SIZE in release-1.9. Then let's update the code to simply check whether (LEGACY_)MANAGED_MEMORY_SIZE has been set or not instead of comparing the default values with each other. Could you update this PR accordingly and open another one against release-1.9?

Surely

Let me know once you've updated the PR @faaronzheng.

@flinkbot run travis

@flinkbot run travis


Add rustfmt and clippy to travis build

Fixes #10

It's the wayland C libraries. The wayland sockets must be placed in $XDG_RUNTIME_DIR.
Le 11 mars 2017 15:16:22 GMT+01:00, Victor Brekenfeld<EMAIL_ADDRESS>a écrit :

Drakulix commented on this pull request.

@@ -24,17 +24,17 @@ branches:
- master

before_script:

|

 pip install 'travis-cargo<0.2' --user &&

 export PATH=$HOME/.local/bin:$PATH

|

 mkdir $(pwd)/socket &&

 export XDG_RUNTIME_DIR="$(pwd)/socket"

which rustfmt || cargo install rustfmt

pip install 'travis-cargo<0.2' --user

export PATH=$HOME/.local/bin:$HOME/.cargo/bin:$PATH

mkdir $(pwd)/socket

export XDG_RUNTIME_DIR="$(pwd)/socket"

What program actually needs that?
--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/vberger/smithay/pull/11#pullrequestreview-26423327


Fixing Tags to accomodate #113 and #143

The automated tests fail without the div tag being closed.

Fixes #113 and #143
The idea of adding navbar was suggested in the linkfree repository but the Js elements weren't closed accordingly to pass the tests.

I closed the PR since I've registered the same under review.


Double root on export as glb a gltf object

Discusion here:
https://forum.babylonjs.com/t/double-root-when-saving-out-glb-from-sandbox/2608

I initially looked at this change: https://github.com/BabylonJS/Babylon.js/pull/7344/files
Reverted via this PR
This change added some extra checking of nodes to be exported and inverted between coordinate systems, and appears to work properly for Left Handed coordinate systems.  Unfortunately, we detect whether or not a node is used as a coordnate system transformation via isNodeConvertingToLeftHanded() this particular line seems somewhat suspect:
https://github.com/BabylonJS/Babylon.js/pull/7344/files#diff-27fc9e54be12c9c3784cf3eb48540e70R1315
In that it works when we use a left handed coordinate system, however, round tripped right handed scenes would still have an encapsulating root node, but would have an Identity transform matrix:

I'll add some additional logic to use the proper transform matrix depending on the scene handedness.

In addition, #7377 also indicates some issue with the encapsulating node used to represent punctual lights in glTF, the logic is similar, we need to prevent duplication of a light's node as well:


Update to prost 0.12, tonic 0.10.

This is useful for two reasons:

It looks like prost 0.12 has a Name trait: https://docs.rs/prost/latest/prost/trait.Name.html so we could skip updating all the TYPE_URLs, switch to prost 0.12, and delete all the impls instead of updating them.

Originally posted by @hdevalence in https://github.com/penumbra-zone/penumbra/issues/3077#issuecomment-1730866734
and also, because Tonic 0.10.0 has an add_routes feature that will let us composably build up RPC services defined in each component: https://docs.rs/tonic/latest/tonic/transport/server/struct.Server.html#method.add_routes
This will require pulling all our dependencies over.

https://github.com/cosmos/ibc-proto-rs/issues/145

Resolved via https://github.com/penumbra-zone/penumbra/pull/3192. Notably we've not yet implemented:

and also, because Tonic 0.10.0 has an add_routes feature that will let us composably build up RPC services defined in each component: https://docs.rs/tonic/latest/tonic/transport/server/struct.Server.html#method.add_routes

but we're now unblocked from doing so.


Text directly under header rendered wrong

When I type for example
[discrete]
== Header

Author: {author} +
Version: {version}

Author and version are rendered in another fontsize (smaller than the header but bigger than the normal text) than expected, I think they might be seen as a sub header? Or am I doing something wrong?

It seems to work when I remove the [discrete] thing. However, I don't want the header in the TOC

I think I fixed the issue by putting the text in a text block with --

It should not be necessary to put the content in a block and the text that follows a discrete heading should not be affected by that heading. Can you please provide a complete example? Are you using a custom theme?

Thanks for the reply. Yes, I am using a custom theme:
extends: default
page:
  layout: portrait
  margin: [0.75in, 1in, 0.75in, 1in]
  size: Book
base:
  font-color: #333333
  font-family: Noto Serif
  # font-size: 12
  # line-height-length: 15
  # line-height: $base-line-height-length / $base-font-size
vertical-spacing: $base-line-height-length
header:
  font_color: 999999
  height: 0.5in
  recto:
    columns: "<40% =20% >40%"
    right:
      content: '_{organization}_'
    center:
      content: image:image.png[pdfwidth=0.50in]
  verso:
    columns: $header_recto_columns
    left:
      content: $header_recto_right_content
    center:
      content: $header_recto_center_content
heading:
  font-color: #262626
  font-size: 17
  font-style: bold
  line-height: 1.2
  margin-bottom: $vertical-spacing
link:
  font-color: #333333
  # font-style: bold
outline-list:
  indent: $base-font-size * 1.5
  text-align: left
footer:
  height: 0.45in
  recto:
    right:
      content: '{section-or-chapter-title} | {page-number}'
  verso:
    left:
      content: '{page-number} | {chapter-title}'

AsciiDoc code:
[discrete]
== Some title
[discrete]
=== Title1
[.text-left]
Lorem ipsum dolor sit amet, consectetur adipiscing elit.
[discrete]
=== Title2
[.text-left]
Lorem ipsum dolor sit amet, consectetur adipiscing elit.

<<<

So what's the problem? I don't see anything abnormal about the document that is generated.

Although not required in this case, you should always put a blank line around a section title or discrete heading.

Maybe it is normal behavior, but I don't understand why the content under title1 is generated bigger than the content under title2 (the font size seems bigger). See report.pdf for clarification.

Aha! Now the truth is revealed. (This is why providing a complete example is so important).
What you're observing is the lead paragraph. The lead paragraph, which is the first paragraph of the preamble (the blocks before the first section) is displayed in a larger font by default. This styling is controlled using the lead category in the theme.


dnCNV button does not work

The app crash when pressing the show dnCNV button. In the Rstudio console, the error shows:
'Error in h: error in evaluating the argument 'x' in selecting a method for function 'as.data.frame': could not find function "subtract"'
@clifflun Can you specify the package where the 'subtract' function coming from?

subtract is from the latest version of GenomicRanges
R also has to be above 4.2 for this to work properly. Ideally you download the R 4.2.2 version
in base R CLI (not RStudio):
install.packages("installr")
library(installr)
updateR()

Got it. The R version on my end is 4.1.2.


Having a few dozen sub-scenes slows down the editor tremendously.

Godot version:

1b8f56c
OS/device including version:

Win10 64-bit
Issue description:

When having a lot of scenes nested inside of scene, the editor becomes incredibly laggy.  The editing of these scenes have become very tedious, and nearing impossible.
Runs fine when launched.
@reduz I had encountered something like this in 2.1 a year ago (terrible load times), and you had mentioned it was resolved in 3.0.  Any insight into this?  A bit concerned since we're going to be having stages that will likely be several times more complex than what I've included here.  I imagine this will drag everything to a halt at some point in the future.
Steps to reproduce:
Makes a few dozen scenes with a few dozen nodes, then combine into one scene.
Minimal reproduction project:

Note:  Some things may be a little wonky, as I cut everything down from our main project to make it a manageable upload.
slow scenes.zip

Cannot reproduce on Ubuntu, with amdgpu on R5.
If I double the number of scenes, it skips some frames while zooming but is still usable (just crashes when mass unlocking nodes).

@eon-s My PC is a bit on the older side of things, but it does run what I have smoothly outside of the Editor.  The individual scenes run fine, but together it becomes trouble.
I also did get some crashes when doing some other operations on all the TSCNs.  Something regarding cowdata, but will have to investigate further on how to reproduce it.

Windows 10 x64
I can somewhat reproduce the issue. Reproduction project does have slowdowns (I wouldn't call them incredibly laggy, but still). However, after moving a bit around and trying to open instanced scene from scene tree, I get a crash with this backtrace:
[0] CanvasItemEditor::_draw_hover (x:\inne\godot\source\godot\editor\plugins\canvas_item_editor_plugin.cpp:3020)
[1] CanvasItemEditor::_draw_viewport (x:\inne\godot\source\godot\editor\plugins\canvas_item_editor_plugin.cpp:3214)
[2] MethodBind0<CanvasItemEditor>::call (x:\inne\godot\source\godot\core\method_bind.gen.inc:139)
[3] Object::call (x:\inne\godot\source\godot\core\object.cpp:943)
[4] Object::emit_signal (x:\inne\godot\source\godot\core\object.cpp:1229)
[5] Object::emit_signal (x:\inne\godot\source\godot\core\object.cpp:1286)
[6] CanvasItem::_update_callback (x:\inne\godot\source\godot\scene\2d\canvas_item.cpp:455)
[7] MethodBind0<CanvasItem>::call (x:\inne\godot\source\godot\core\method_bind.gen.inc:139)
[8] Object::call (x:\inne\godot\source\godot\core\object.cpp:943)
[9] MessageQueue::_call_function (x:\inne\godot\source\godot\core\message_queue.cpp:256)
[10] MessageQueue::flush (x:\inne\godot\source\godot\core\message_queue.cpp:305)
[11] SceneTree::iteration (x:\inne\godot\source\godot\scene\main\scene_tree.cpp:477)
[12] Main::iteration (x:\inne\godot\source\godot\main\main.cpp:1861)
[13] OS_Windows::run (x:\inne\godot\source\godot\platform\windows\os_windows.cpp:2804)
[14] widechar_main (x:\inne\godot\source\godot\platform\windows\godot_windows.cpp:151)
[15] _main (x:\inne\godot\source\godot\platform\windows\godot_windows.cpp:173)
[16] main (x:\inne\godot\source\godot\platform\windows\godot_windows.cpp:185)
[17] __scrt_common_main_seh (f:\dd\vctools\crt\vcstartup\src\startup\exe_common.inl:283)
[18] BaseThreadInitThunk

Seems to happen randomly after zooming in a lot and repeatedly opening a scene. Tested on newest master.

I get the same backtrace as @KoBeWi. When editing the AnimationPlayer of the counter instance scene, this error is spammed in the console:
ERROR: AnimationPlayer::_animation_update_transforms: Failed setting key at time 0 in Animation 'angles' at Node '/root/EditorNode/@@5/@@6/@@14/@@16/@@20/@@24/@@25/@@26/@@42/@@43/@@52/@@53/@@5079/@@4942/@@4943/@@4944/@@4945/@@4946/counter/aim', Track 'root/tilt_anchor/body_anchor/aim_anchor_B/recoil_anchor:influence'. Check if property exists or the type of key is right for the property
   At: scene\animation\animation_player.cpp:
870

This issue is about slowdown, open another with the crash report and animation player error @KoBeWi
@girng

Alright, well in regards to slowdown.. I'm on a GTX 950, W10, and i7 2600. Editor runs smooth for me. Can't reproduce :D

Besides the fact @groud may need to optimize viewport drawing a bit, the main issue is the lack of accumulation of motion events. Godot should most likely implement this in order to avoid this kind of situation.

Works fine for Mac + AMD Radeon Pro 455

ok, runs smooth as silk now.

@reduz Yes, massive improvement in a1e73dc, it is functional again.  X)  👍 👍 🎆 Thanks for the quick response.
Seems others are getting errors in the animations, so was expecting that to be the culprit.  So maybe there is some smoke from this that things still lurk in the AnimationPlayer?  I will keep an eye in that area.


Elasticsearch Client Plugin Contribution

Title: Elasticsearch Client Plugin Contribution
Link: https://github.com/yin-bp/pinpoint
Target: BBoss Elasticsearch Java API
Supported Version: All Elasticsearch version,BBoss version must be up to <IP_ADDRESS>,pinpoint 1.6.x,1.7.x,1.8.x
Description: This agent plugin is worked with elasticsearch rest client BBoss,
BBoss is a best Java Highlevel Rest client for ElasticSearch.
ServiceTypes: [1971, "Elasticsearch"],[1972, "ElasticsearchEvent"],[9977, "ElasticsearchExecutor"]
Annotations: [971, "es.args"],[972, "es.url"],[973, "es.dsl"],[974, "es.action"],[975, "es.responseHandle"]
Configurations: List of configuration keys and description the plugin adds.
profiler.elasticsearch.enabled=true  //enable or disable this plugin, default true.
profiler.elasticsearch.recordResult=false  //record or disable record method return value ,default false.
profiler.elasticsearch.recordArgs=true  //record or disable record method args,default true.

Codecov Report

Merging #4392 into master will decrease coverage by 0.21%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #4392      +/-   ##
==========================================
- Coverage   40.85%   40.63%   -0.22%
==========================================
  Files        2449     2460      +11
  Lines       75435    75838     +403
  Branches    10225    10280      +55
==========================================
  Hits        30820    30820
- Misses      41783    42188     +405
+ Partials     2832     2830       -2

Impacted Files
Coverage Δ

...n/elasticsearch/ElasticsearchProviderDetector.java
0% <0%> (ø)

...interceptor/ElasticsearchOperationInterceptor.java
0% <0%> (ø)

...tor/ElasticsearchExecutorOperationInterceptor.java
0% <0%> (ø)

...asticsearch/ElasticsearchInterceptorClassInfo.java
0% <0%> (ø)

...rceptor/ElasticsearchBaseOperationInterceptor.java
0% <0%> (ø)

...n/elasticsearch/ElasticsearchMetadataProvider.java
0% <0%> (ø)

...lugin/elasticsearch/ElasticsearchPluginConfig.java
0% <0%> (ø)

.../plugin/elasticsearch/ElasticsearchMethodInfo.java
0% <0%> (ø)

...n/elasticsearch/RestSeachExecutorMethodFilter.java
0% <0%> (ø)

...elasticsearch/ElasticsearchCustomMethodFilter.java
0% <0%> (ø)

... and 19 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 eebdf4f...5a743c0. Read the comment docs.


Fix lockfile to restore Heroku deploys

Same idea as #3904.
Last known working was 316749bd69b684274b2ffd952f9c0872849d82af so I reverted to that and then ran npm install.
I'm on<EMAIL_ADDRESS>

Messages

:book:
:sparkles: Thanks for your contribution to Shields, @paulmelnikow!

  Generated by :no_entry_sign: dangerJS against becff7cb7cfe973f88c026598be6025f99362bd1

Big nope!


Add awesome vue graphql list

@praveenweb I have added all the files. Please review and suggest changes.

Closing in favor of https://github.com/hasura/awesome-vue-graphql/pull/2


cupertino_icons-0.1.1 shows as error in Intellij 2017.3.2

Doesnt seem to affect ability to build, but shows up as problem in intellij :
Library Dart Packages has broken classes path:   /home/david/.pub-cache/hosted/pub.dartlang.org/cupertino_icons-0.1.1/lib [Fix]
It looks to be because it has no lib directory.

@devoncarew, doesn't seem like an analyzer warning when I try it on a sample project. Is this an IntelliJ-plugin specific feature? Is there an approach we want to take with resource only packages?

It is just an IntelliJ issue - we expect all pub packages to have a lib/ folder.
Perhaps not a hard constraint for pub packages, but an easy fix here is just to add a lib/ folder to the cupertino_icons package (w/ perhaps a small file in there so that you can commit the folder to git).

/cc @jwren, @alexander-doroshko

This path is taken from .packages file.
I don't think it's a problem though.
I'm sure this shouldn't be fixed on the IDE end: IntelliJ must respect all paths listed in .packages file.

Ah, right about .packages. Not a IntelliJ issue. Will check with the dart team for the appropriate approach with resource packages.

https://github.com/dart-lang/sdk/issues/32662

Hi , i also got this problem in android studio. and i solved it by download the new version Flutter package, unzip it ,and replace the old one . and then it's work.


trip-history not working

Since a few days I cannot retrieve the trip-history using the methods vehicle.requestDailyStatistics().
The response body indicates an error:
$ Invoking NissanConnect/Kamereon API: https://alliance-platform-caradapter-prod.apps.eu2.kamereon.io/car-adapter/v1/cars/SJNFxxxx/trip-history?start=2022-04-26&end=2022-04-26&type=0
$ Params: null
$ Headers: {Authorization: Bearer yyyy}
$ Result: {errors: [{status: 502, code: gateway.processing, title: VNEXT, detail: an internal error occured while processing request : 404 NOT_FOUND}]}

It seems to be a problem on the Nissan side, but maybe one can suggest some parameter changes on the API request?
Any ideas on this?

I cannot reproduce this with the account I use for testing. Could be a temporary problem.
Den 26. april 2022 08.20.37 CEST, Franz-Josef Kaiser @.***> skrev:

Since a few days I cannot retrieve the trip-history using the methods vehicle.requestDailyStatistics().
The response body indicates an error:
$ Invoking NissanConnect/Kamereon API: https://alliance-platform-caradapter-prod.apps.eu2.kamereon.io/car-adapter/v1/cars/SJNFxxxx/trip-history?start=2022-04-26&end=2022-04-26&type=0
$ Params: null
$ Headers: {Authorization: Bearer yyyy}
$ Result: {errors: [{status: 502, code: gateway.processing, title: VNEXT, detail: an internal error occured while processing request : 404 NOT_FOUND}]}

It seems to be a problem on the Nissan side, but maybe one can suggest some parameter changes on the API request?
Any ideas on this?
--
Reply to this email directly or view it on GitHub:
https://github.com/Tobiaswk/dartnissanconnect/issues/6
You are receiving this because you are subscribed to this thread.
Message ID: @.***>

Thanks for checking. Yesterday I could retrieve some data, too. But only few times. I have to dig deeper into the issue, maybe I hit some access limits or somthing like this.
I'm closing the issue for now.

Let me add some additional info:

I've made some tests and found that I cannot get the daily stats for the current day.
Requesting monthly stats works, as well as requesting daily stats for previous days.
Requesting daily stats for multiple past days also works, if the end date is at most today (but the result only contains data up to yesterday)
So only querying the stats for the current day fails

Interestingly, this affects also the My Leaf app as well as the original Nissan Connect app. Both apps only show trip details for past days, but not for the current day.
And yes, the problem is with the Kameroen API, not with dartnissanconnect.

Just did some more research: The issue seems to be, that the Kamereon API returns a 502 response when there is no data available for the requested day.
The debug output looks like:
$ Invoking NissanConnect/Kamereon API: https://alliance-platform-caradapter-prod.apps.eu2.kamereon.io/car-adapter/v1/cars/xxxxxxxxx/trip-history?start=2022-04-27&end=2022-04-27&type=0
$ Params: null
$ Headers: {Authorization: Bearer yyyyyy}
$ Result: {errors: [{status: 502, code: gateway.processing, title: VNEXT, detail: an internal error occured while processing request : 404 NOT_FOUND}]}
Unhandled exception:
NoSuchMethodError: The method '[]' was called on null.
Receiver: null
Tried calling: []("attributes")
#0      Object.noSuchMethod (dart:core-patch/object_patch.dart:38:5)
#1      NissanConnectVehicle.requestDailyStatistics
<asynchronous suspension>

Exited (255)

My suggestion would be to add some kind of error handling, especially to requestDailyStatistics. However, since I'm not a native Dart programmer, I currently have no idea how to do this.

Very interesting! This must be vehicle specific. I cannot reproduce it with the vehicle I use for testing.
Maybe it could be your vehicle that has problems with up-to-date NissanConnect data for some strange reason.
Like you state yourself this seems like an vehicle specific and/or NissanConnect issue, not dartnissanconnect issue.
I'm not sure I can really help you much sadly. I'd be interested in knowing that cause though; as other could be affected be the same issue.
Den 27. april 2022 20.01.19 CEST, Franz-Josef Kaiser @.***> skrev:

Let me add some additional info:

I've made some tests and found that I cannot get the daily stats for the current day.
Requesting monthly stats works, as well as requesting daily stats for previous days.
Requesting daily stats for multiple past days also works, if the end date is at most today (but the result only contains data up to yesterday)
So only querying the stats for the current day fails

Interestingly, this affects also the My Leaf app as well as the original Nissan Connect app. Both apps only show trip details for past days, but not for the current day.
And yes, the problem is with the Kameroen API, not with dartnissanconnect.
--
Reply to this email directly or view it on GitHub:
https://github.com/Tobiaswk/dartnissanconnect/issues/6#issuecomment-1111317350
You are receiving this because you commented.
Message ID: @.***>


Docs on STREAMLIT_BROWSER_GATHER_USAGE_STATS env var

Link to doc page in question (if any):
I read somewhere (and being completely honest, I can no longer find the link) that setting the env var STREAMLIT_BROWSER_GATHER_USAGE_STATS=false will disable Streamlit browser usage stats.
I think a related config item is: browser.gatherUsageStats: https://docs.streamlit.io/library/advanced-features/configuration#browser
Can you add this env var (and other similarly related env vars) to the docs?
Name of the Streamlit feature whose docs need improvement:
Configuration --> Telemetry and Configuration --> Browser
What you think the docs should say:
Mention that one can use env vars, instead of .streamlit/config.toml

I've added the explanation about converting config options from TOML format to environment variable format in a PR. Thanks.


Update SE and MP guides to adopt new review feedback

Romain and Joe supplied some good feedback about the first guide. This task incorporates that feedback into the first guide.
This is important to do now so future guides can use the SE RESTful web app one as a pattern.
Here is my list of the topics we covered:

Change guides to use the Helidon archetype.
Because #1 generates a lot of Java code, change the guide from largely adding code to largely explaining the code that is generated. Some code (e.g., metrics and health) will still be added.
Link to or include pre-req information from the separate, common file.
a. Maybe a simple approach is for the common AsciiDoc attribute assignment file to assign attributes containing pre-req versions so the guide .adoc files can refer to those.
b. Another approach would be for the existing pre-reqs info in the Getting Started file to be tagged so individual guides can include only the pre-reqs they need.
Add default attribute settings in the guide so GitHub rendering works correctly. (Will evolve over time as the common AsciiDoc attribute assignment file appears.)
Change ReSTful to RESTful. (done as part of #215)
Migrate the guide files from src/main/docs/index.adoc to top-level README.adoc. Part of this item is to make sure the README.md renders properly, esp. regarding includes.
Break long command lines explicitly to avoid horizontal scrolling of code blocks on small screens. (done in #215)
See about inlining the startServer() code block.
Change output code blocks top reformatted (so suppress the built-in copy feature on our site). (done in #215)
Specify the exact git clone command in the last section for cloning the repo.
Fix list problem after the HTTP method table. (done in #168)
See Joe’s example from Getting Started to narrow tables where possible.
See if there is a way to strip indentation from included code snippets.

@romain-grecourt @barchetta FYI some bad news...It turns out that GitHub does not support the AsciiDoc include directive.
https://github.com/github/markup/issues/1095
The issue makes a reference to perhaps being available "next year" (2019).

@tjquinno  That's fine with me :D - I actually would prefer linking the pre-req ; it does not make the guide self contained but makes it lighter.
It goes against the rule of "links are bad", but we could be smart and have a little section at the top like "before you begin" where we list the things that must be done before following a guide.

I'm not so concerned with the pre-requisites. The current guides pull in a great deal of code from the source .java files, pom.xml', resource files, etc. and use callouts to discuss why bits of code are there. In a branch I moved the AsciiDoc to README.adoc at the top level and it's full of ugliness like this:

(see the link:xxx parts)
We can still move the .adoc files to README.adoc at that top level but GitHub will not render it nicely -- not unless we change how the included content in the guides will be handled on GitHub and I don't know what we can do to improve the situation, other than waiting for GitHub to fix this.

Yikes, I missed that. One other way would be to not use the include directives at all but instead duplicate the content.
The duplication sucks, it means we'd loose the advantages of using the include directives.
We could workaround that with custom maven plugin and match the snippets in the adoc files with their corresponding sources. I.e add a couple of annotation in the adoc files so that we know what files and what snippet to match with, then we can add some check in the pipeline to verify these.
This sounds like a potential good idea, we could also be smart and fix the indentation issues we get with the include directives.

Yes, I was wondering what our options might be to do something like that.
About the indentation with includes... I think there are further options on the include directive I have not tried. Anything clever we do with a custom maven plug-in should honor normal AsciiDoc as much as possible so, if and when AsciiDoc-on-GitHub improves we can take advantage without having to rework our .adoc source.


ValueError when using QAT-PACT

ValueError occurs when I run quantization-aware-training using PACT.
The error message is "ValueError: saturation_min must be smaller than saturation_max" and it seems like the error occurs when the clipping value of ReLU is below zero because the clipping value cannot be negative theoretically.
setting:

architecture: preact_resnet20_cifar
cifar10
yaml: distiller/examples/quantization/quant_aware_train/preact_resnet20_cifar_pact.yaml (act_clip_decay: 0.01 added, file attached)

================= Error Log ============================
File "compress_classifier.py", line 212, in
main()
File "compress_classifier.py", line 78, in main
app.run_training_loop()
File "/home/jelim/f_github/test_distiller/distiller/distiller/apputils/image_classifier.py", line 192, in run_training_loop
top1, top5, loss = self.train_validate_with_scheduling(epoch)
File "/home/jelim/f_github/test_distiller/distiller/distiller/apputils/image_classifier.py", line 130, in train_validate_with_scheduling
top1, top5, loss = self.train_one_epoch(epoch, verbose)
File "/home/jelim/f_github/test_distiller/distiller/distiller/apputils/image_classifier.py", line 116, in train_one_epoch
loggers=[self.tflogger, self.pylogger], args=self.args)
File "/home/jelim/f_github/test_distiller/distiller/distiller/apputils/image_classifier.py", line 577, in train
output = model(inputs)
File "/Data2/home/jelim/.local/lib/python3.6/site-packages/torch/nn/modules/module.py", line 541, in call
result = self.forward(*input, **kwargs)
File "/Data2/home/jelim/.local/lib/python3.6/site-packages/torch/nn/parallel/data_parallel.py", line 150, in forward
return self.module(*inputs[0], **kwargs[0])
File "/Data2/home/jelim/.local/lib/python3.6/site-packages/torch/nn/modules/module.py", line 541, in call
result = self.forward(*input, **kwargs)
File "/home/jelim/f_github/test_distiller/distiller/distiller/models/cifar10/preresnet_cifar.py", line 153, in forward
x = self.layer1(x)
File "/Data2/home/jelim/.local/lib/python3.6/site-packages/torch/nn/modules/module.py", line 541, in call
result = self.forward(*input, **kwargs)
File "/Data2/home/jelim/.local/lib/python3.6/site-packages/torch/nn/modules/container.py", line 92, in forward
input = module(input)
File "/Data2/home/jelim/.local/lib/python3.6/site-packages/torch/nn/modules/module.py", line 541, in call
result = self.forward(*input, **kwargs)
File "/home/jelim/f_github/test_distiller/distiller/distiller/models/cifar10/preresnet_cifar.py", line 76, in forward
out = self.relu(out)
File "/Data2/home/jelim/.local/lib/python3.6/site-packages/torch/nn/modules/module.py", line 541, in call
result = self.forward(*input, **kwargs)
File "/home/jelim/f_github/test_distiller/distiller/distiller/quantization/clipped_linear.py", line 65, in forward
scale, zero_point = asymmetric_linear_quantization_params(self.num_bits, 0, self.clip_val, signed=False)
File "/home/jelim/f_github/test_distiller/distiller/distiller/quantization/q_utils.py", line 91, in asymmetric_linear_quantization_params
raise ValueError('saturation_min must be smaller than saturation_max')
ValueError: saturation_min must be smaller than saturation_max
====================================================
error_saturation_min_PACT.log
pact_yaml.zip

Hi, did you find a solution to this issue?


Spec Export not working

The spec export function is not working. the UI shows a completion but the folder is not created.
Here's the log error
Nov 21 12:15:16 Sketch[50330]: Exception: TypeError: color.hexValue is not a function. (In 'color.hexValue()', 'color.hexValue' is undefined)
Nov 21 12:15:16 Sketch[50330]: Info: {
	    column = 79;
	    line = 284;
	    sourceURL = "/Users/xxx/Library/Application Support/com.bohemiancoding.sketch3/Plugins/Sketch Measure.sketchplugin/Contents/Sketch/mark.sketchscript";
	}
Nov 21 12:15:16 Spec Export (Sketch Plugin)[50330]: TypeError: color.hexValue is not a function. (In 'color.hexValue()', 'color.hexValue' is undefined)
	line: 284
	sourceURL: /Users/xxx/Library/Application Support/com.bohemiancoding.sketch3/Plugins/Sketch Measure.sketchplugin/Contents/Sketch/mark.sketchscript
	column: 79

I actually get an error message. I'm running Version 41.1 (35376) of Sketch.

v2.3.43 now, update the plugin and restart sketch.app to try it newest.

Works perfect !!! Thanks a lot!

@utom I updated but I still get the same error, just later in the process.

@iamtimm if you restarted sketch.app and still get the same error, please sent me the document file, let me check~

@florianpnn I'm curious: Where can one see the log output?

Working for me 👍
@marvinhagemeister it's coming from the console.app


Remove user specific settings

Hi @JasonBeck2 would mind removing the VS Code settings that were checked in? I've created a .gitignore so you can still have it for your local machine.

Good catch @julie-ng!!


next-assembly-info-files

This PR adds a new attribute to the DotnetAssemblyInfo provider called files, which is all of the files in the package. This is ground-work for building a rules_fable.
Fable will attempt to "crack" a Nuget package by starting from a referenced DLL and walking up the directory tree till a nuspec file is found. Then it will try to collect any fs, fsi and fsproj files in that sub-tree. To support Fable without major modifications, it is easiest to give all package files as inputs to the Bazel action.
Details here: https://github.com/fable-compiler/Fable/blob/main/src/Fable.Cli/ProjectCracker.fs

https://github.com/bazel-contrib/rules_dotnet/pull/455


[7.x] Restore default connection after Artisan migrate/seed call

This will be useful when having an application that handle migrations
for more than one database connection.
The side affect of current implementation is that until the request
(web/console) is terminated the database will used the latest migration
connection as default.
Solved #28253
Signed-off-by: Mior Muhammad Zaki<EMAIL_ADDRESS>

I understand the use case but something about this particular design maybe feels a bit weird. For example say I was to just instantiate a migrator directly... call setConnection... then call runPending... now my connection is set to null again?

Is there something preventing us from managing this totally at the comment level? It feels weird for the Migrator to be aware of this and to be managing this state in this way.

Another alternative is to convert singleton to bind so it resolve a new instance every time we need migrator and migration repository IoC.
But not sure how we can solve the Seeder command where it change ConnectionResolver defaults.

The changes to seed and status commands seem fine to me. I'm mainly curious if we could just take a similar approach on the migrations. It feels weird for the migrator to be maintaining that state.

It feels weird for the migrator to be maintaining that state.

I agree. It feels like this is actually a code smell to tell us that we shouldn't need to mutate the state in this way.

Instead of setting the connection, we should pass the actual connection object around, somehow...

I made pretty big changes to the Migrator on this and got rid of all previous connection state. Added a new usingConnection(name, fn) method that we can use to run a block of code with a different connection. Then tweaked migrate and rollback commands to use this.

👍


Celery Job cannot handle empty CVE when sending finding to Jira

Bug description
The DefectDojo Docker Celery worker cannot handle an empty CVE when bulk sending issues to JIRA.
Steps to reproduce
Steps to reproduce the behavior:

Setup a DefectDojo docker with a Jira connection
Create some issues with an Emptu CVE (Nessus import by default does not fill the CVE field)
Select some issues and press Bulk Edit
Click on Send To Jira
See the Error in the console log for the docker.
Edit a single issue in DefectDojo with an Empty CVE
Select the Push To Jira at the bottom of the page
See that it gets send to Jira.

Expected behavior
Would expect the bulk edit to not care about an empty CVE or other field and just send an empty string over the API. Would also expect DefectDojo to give a clear error when fields are required.
Temporary workaround
Either, send all issues to Jira one by one
Or, fill out all the fields of all issues and then bulk send.
Deployment method (select with an X)

[ ] Kubernetes
[X] Docker
[ ] setup.bash / legacy-setup.bash

Environment information

Operating System: DefectDojo Docker on MacOS
DefectDojo version: v. 1.5.4

Screenshots (optional)

hello, @FrankSh0rty it is fixed in the dev branch. Wait until next release or use dev version.

Hi @dr3dd589, thanks for the reply. That is good news. Do you know if this branch also includes other bug fixes concerning the JIRA integration, like that findings are not synced when importing test results or that those findings do not attach to the epic that DefectDojo creates?
Thank you!


Update sample code to latest Compose version and add version badge

The sample code in the Readme was out of date. I also added a badge that displays the latest GH release (lmk if you think a Maven badge would be better, I didn't find one that works for the Google repo 🤔).

Thanks!

Thanks @jossiwolf!


Assertion Failed when open monopoly (Altstore + Sideloady edition)

Describe the bug
After Install monoply (cracked) and open it this error appear
Assertion failed: (LiveContainerUIHandle), file main.m, line 506.
Call stack: (
0   CoreFoundation                      0x000000019157d098 4250D45C-51A9-3AC5-8C3D-69DE936FAD46 + 540824
1   libobjc.A.dylib                     0x000000018e87f2e4 objc_exception_throw + 88
2   CoreFoundation                      0x0000000191678648 4250D45C-51A9-3AC5-8C3D-69DE936FAD46 + 1570376
3   LiveContainer                       0x00000001041575e8 __assert_rtn + 96
4   LiveContainer                       0x000000010415113c LiveContainerMain + 2524
5   dyld                                0x00000001b6d20d34 5DB83988-2EE6-3756-BD07-B8D67B1133A5 + 212276
)
Instructions to reproduce
1- Install Sidestore from altserver (mac)
2- Install Live Container from sidestore
3- Setup JIT less
4- Add Monopoly iPA
5- Run and bugs appear
What version of LiveContainer are you using?
Version 3.0.0-release (main/e5cf20b)
Other info
Monopoly not cracked working but cracked not working
Older version from same game same site works

You need JIT, unless there is another fix that dev might implement.

I don't know where you got the ipa, but if you found it in ipaomtk or iosgods then it's a bad ipa. It does not work even with TrollStore. The old version does work.

The issue of a bad IPA file can be resolved by creating a Signature for the app through Esign and enable the four options from More settings as in the screenshot.
Credit: @n9ipa from Telegram


[rmodels] Remove redundant call for disabling vertex color attribute

This PR removes the call to rlDisableVertexAttribute for SHADER_LOC_VERTEX_COLOR in DrawMesh and DrawMeshInstanced, because it was being called in cases where this location is not set in the shader. This resulted in an invalid value (-1) being passed to rlDisableVertexAttribute which made an OpenGL error GL_INVALID_VALUE.
Furthermore the same function is already being called  with correct checks (checking if location attribute is set and vboId[3]==0) few lines above (rmodels.c:1450 and rmodels.c:1671 in the original source).
This solves issue https://github.com/raysan5/raylib/issues/3841

@zyperpl thanks for the review!


Links at top of screen

It seems like within the past day 4 links have appeared at the top of the screen, linking to the Privacy Policy, Cookie Disclaimer, Terms & Conditions, and Acceptable Use Policy. Clicking on them brings to to what you would expect, but there is no way out of them other than restarting the app.

(Sorry if I misformatted the issue or left information out, I'm not used to using GitHub and I'm on mobile.)

I've got this fixed for the desktop version at least in an upcoming release, but haven't looked at the mobile version yet.


merge package from arlac77/npm-package-template-esm-only

package.json

chore(package<EMAIL_ADDRESS>chore(scripts): cover@#overwrite c8 --temp-directory build/tmp ava && c8 report -r lcov -o build/coverage --temp-directory build/tmp
chore(scripts): posttest@markdown-doctest

Codecov Report

Merging #550 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #550   +/-   ##
=======================================
  Coverage   75.94%   75.94%
=======================================
  Files           2        2
  Lines         474      474
=======================================
  Hits          360      360
  Misses        114      114

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 47a2d63...0c349ca. Read the comment docs.

Codecov Report

Merging #550 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #550   +/-   ##
=======================================
  Coverage   75.94%   75.94%
=======================================
  Files           2        2
  Lines         474      474
=======================================
  Hits          360      360
  Misses        114      114

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 47a2d63...0c349ca. Read the comment docs.

Coverage remained the same at 76.225% when pulling 0c349ca12618048453767277196a06a86befb5a1 on npm-template-sync/1 into 47a2d63b24a4347ca011f6e2ce68848ded77df73 on master.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Removes yellow boxes, adds pull-to-refresh controls, pre-renders sibling tabs for Home+Favs

Related to DISCO-978. Here's what's up:

I removed the yellow boxes from our dev experience, with links back to why we're ignoring those warnings.
I added pull-to-refresh controls to the favs tab components (Works, Artists, Shows, etc). The control doesn't work on empty data sets, so there's still room for improvement, but having a pull-to-refresh on the fav artworks page is an important step forward.
~I specified the Home and Favs tabs to pre-render sibling components on load. This means that, when the app launches on the home tab (artists sub-tab), the WFY and Auctions sub-tabs will load in the background, making the app feel more responsive.~

That last one is the most contentious – I'm hoping to get a 👍 from @orta to make sure we're not doing it for a reason. Otherwise, it seems like a low-effort, high-impact change.
The components are really similar – I've opened a ticket to return to them to refactor.

Yep, looks good 👍