Failed to start lsp-server in 0.14.5

Precheck

Do a quick search and make sure the issue has not yet been reported
Do not use the issues tracker for guidance, questions or support (try Elixir Forum, Stack Overflow, Slack, etc. instead)
If using a client other than VS Code, please try VSCode's "ElixirLS: Elixir support and debugger" extension. Does it reproduce your failure?

Environment

Elixir & Erlang versions (elixir --version):
elixir: 1.14.4-otp-25
erlang: OTP-25.3
Elixir Language Server version:
Operating system: Linux
Editor or IDE name (e.g. Emacs/VSCode): vscode
Editor Plugin/LSP Client name and version: 0.14.5

Current behavior
Upgrade ls to 0.14.5
[Error - 22:51:35] Client ElixirLS - lib-redis: connection to server is erroring. Shutting down server.
[Error - 22:51:35] Stopping server failed
Error: Client is not running and can't be stopped. It's current state is: startFailed
    at LanguageClient.shutdown (/home/tt67wq/.vscode/extensions/jakebecker.elixir-ls-0.14.5/node_modules/vscode-languageclient/lib/common/client.js:914:19)
    at LanguageClient.stop (/home/tt67wq/.vscode/extensions/jakebecker.elixir-ls-0.14.5/node_modules/vscode-languageclient/lib/common/client.js:885:21)
    at LanguageClient.stop (/home/tt67wq/.vscode/extensions/jakebecker.elixir-ls-0.14.5/node_modules/vscode-languageclient/lib/node/main.js:150:22)
    at LanguageClient.handleConnectionError (/home/tt67wq/.vscode/extensions/jakebecker.elixir-ls-0.14.5/node_modules/vscode-languageclient/lib/common/client.js:1146:18)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)

Expected behavior

Please provide a project that reproduces it or the output of running language_server.sh in your project


Avoid RequestLogAvailabilityException while writing access logs

Modifications:

Fix a bug introduced by #1157

A better fix would be use RequestLog.addListener() so that the Consumer<RequestLog> is invoked after all RequestLogAvailability is fulfilled:
reqCtx.log().addListener(accessLogWriter::accept, RequestLogAvailability.COMPLETE);

.. instead of modifying the logger implementation.
There are two classes that call Consumer.accept(RequestLog):

HttpResponseSubscriber
HttpServerHandler

Closing in favor of #1159


11/8/2022 Publish

Learn Build status updates of commit 598cd1a:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docs-ref-services/latest/active-directory.md
:warning:Warning
View (azure-node-latest)
Details

docs-ref-services/latest/analysis-services.md
:warning:Warning
View (azure-node-latest)
Details

docs-ref-services/latest/api-for-fhir.md
:warning:Warning
View (azure-node-latest)
Details

docs-ref-services/latest/api-management.md
:warning:Warning
View (azure-node-latest)
Details

docs-ref-services/latest/app-configuration.md
:warning:Warning
View (azure-node-latest)
Details

docs-ref-services/latest/appcontainers.md
:warning:Warning
View (azure-node-latest)
Details

docs-ref-services/latest/application-insights.md
:warning:Warning
View (azure-node-latest)
Details

docs-ref-services/latest/app-platform.md
:warning:Warning
View (azure-node-latest)
Details

docs-ref-services/latest/appservice.md
:warning:Warning
View (azure-node-latest)
Details

docs-ref-services/latest/app-service.md
:warning:Warning
View (azure-node-latest)
Details

docs-ref-services/latest/authorization.md
:warning:Warning
View (azure-node-latest)
Details

docs-ref-services/latest/azureadexternalidentities.md
:warning:Warning
View (azure-node-latest)
Details

docs-ref-services/latest/azure-confidential-ledger.md
:warning:Warning
View (azure-node-latest)
Details

docs-ref-services/latest/azure-stack.md
:warning:Warning
View (azure-node-latest)
Details

docs-ref-services/latest/batch-ai.md
:warning:Warning
View (azure-node-latest)
Details

docs-ref-services/latest/bot-service.md
:warning:Warning
View (azure-node-latest)
Details

docs-ref-services/latest/changeanalysis.md
:warning:Warning
View (azure-node-latest)
Details

docs-ref-services/latest/changes.md
:warning:Warning
View (azure-node-latest)
Details

docs-ref-services/latest/cognitive-service-for-language.md
:warning:Warning
View (azure-node-latest)
Details

docs-ref-services/latest/cognitive-services.md
:warning:Warning
View (azure-node-latest)
Details

docs-ref-services/latest/commerce.md
:warning:Warning
View (azure-node-latest)
Details

docs-ref-services/latest/commitment-plans.md
:warning:Warning
View (azure-node-latest)
Details

docs-ref-services/latest/communication.md
:warning:Warning
View (azure-node-latest)
Details

docs-ref-services/latest/communication-common-readme.md
:warning:Warning
View (azure-node-latest)
Details

docs-ref-services/latest/compute.md
:warning:Warning
View (azure-node-latest)
Details

This comment lists only the first 25 files in the pull request.

docs-ref-services/latest/active-directory.md

Line 5, Column 13: [Warning: ms-service-subservice-invalid - See documentation] Invalid value for 'ms.service': 'activedirectory'.
Line 17, Column 211: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://docs.microsoft.com/azure/active-directory/develop/reference-v2-libraries' will be broken in isolated environments. Replace with a relative link.
Line 17, Column 343: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://docs.microsoft.com/graph/overview' will be broken in isolated environments. Replace with a relative link.
Line 20, Column 38: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://docs.microsoft.com/azure/active-directory/develop/sample-v2-code' will be broken in isolated environments. Replace with a relative link.
Line 21, Column 26: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://docs.microsoft.com/azure/active-directory/develop/msal-compare-msal-js-and-adal-js' will be broken in isolated environments. Replace with a relative link.
Line 21, Column 143: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://docs.microsoft.com/graph/migrate-azure-ad-graph-overview' will be broken in isolated environments. Replace with a relative link.
Line 67, Column 68: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://docs.microsoft.com/samples/browse/?products=azure&terms=node.js' will be broken in isolated environments. Replace with a relative link.

docs-ref-services/latest/analysis-services.md

Line 5, Column 13: [Warning: ms-service-subservice-invalid - See documentation] Invalid value for 'ms.service': 'analysisservices'.
Line 31, Column 70: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://docs.microsoft.com/samples/browse/?languages=javascript' will be broken in isolated environments. Replace with a relative link.

docs-ref-services/latest/api-for-fhir.md

Line 5, Column 13: [Warning: ms-service-subservice-invalid - See documentation] Invalid value for 'ms.service': 'apiforfhir'.

docs-ref-services/latest/api-management.md

Line 5, Column 13: [Warning: ms-service-subservice-invalid - See documentation] Invalid value for 'ms.service': 'apimanagement'.

docs-ref-services/latest/app-configuration.md

Line 5, Column 13: [Warning: ms-service-subservice-invalid - See documentation] Invalid value for 'ms.service': 'appconfiguration'.

docs-ref-services/latest/appcontainers.md

Line 5, Column 13: [Warning: ms-service-subservice-invalid - See documentation] Invalid value for 'ms.service': 'appcontainers'.

docs-ref-services/latest/application-insights.md

Line 5, Column 13: [Warning: ms-service-subservice-invalid - See documentation] Invalid value for 'ms.service': 'applicationinsights'.

docs-ref-services/latest/app-platform.md

Line 5, Column 13: [Warning: ms-service-subservice-invalid - See documentation] Invalid value for 'ms.service': 'appplatform'.

docs-ref-services/latest/appservice.md

Line 5, Column 13: [Warning: ms-service-subservice-invalid - See documentation] Invalid value for 'ms.service': 'appservice'.

docs-ref-services/latest/app-service.md

Line 5, Column 13: [Warning: ms-service-subservice-invalid - See documentation] Invalid value for 'ms.service': 'appservice'.
Line 37, Column 68: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://docs.microsoft.com/samples/browse/?languages=javascript' will be broken in isolated environments. Replace with a relative link.

docs-ref-services/latest/authorization.md

Line 5, Column 13: [Warning: ms-service-subservice-invalid - See documentation] Invalid value for 'ms.service': 'authorization'.
Line 34, Column 70: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://docs.microsoft.com/samples/browse/?languages=javascript' will be broken in isolated environments. Replace with a relative link.

docs-ref-services/latest/azureadexternalidentities.md

Line 5, Column 13: [Warning: ms-service-subservice-invalid - See documentation] Invalid value for 'ms.service': 'azureadexternalidentities'.

docs-ref-services/latest/azure-confidential-ledger.md

Line 5, Column 13: [Warning: ms-service-subservice-invalid - See documentation] Invalid value for 'ms.service': 'azureconfidentialledger'.

docs-ref-services/latest/azure-stack.md

Line 5, Column 13: [Warning: ms-service-subservice-invalid - See documentation] Invalid value for 'ms.service': 'azurestack'.

docs-ref-services/latest/batch-ai.md

Line 5, Column 13: [Warning: ms-service-subservice-invalid - See documentation] Invalid value for 'ms.service': 'batchai'.

docs-ref-services/latest/bot-service.md

Line 5, Column 13: [Warning: ms-service-subservice-invalid - See documentation] Invalid value for 'ms.service': 'botservice'.

This comment lists only the first 25 errors (including error/warning/suggestion) in the pull request.
For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Learn Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel


Handle the zoom and Pan for graph modal

Add legends if necessary to control zoom and pan.

Completed and merged


Authorization Filters

ASP.NET MVC -> Security -> Authorization Filters

Any status update on when this will be written?

Not yet - it's still on the backlog.

Hi Dan. Thanks for the update.

I hope this stays:
[ClaimsPrincipalPermission(SecurityAction.Demand, Operation = "OperationA", Resource = "DeletePage")]
or improves.
At the moment I am unable to find anything that parallels it. I have been searching all of 30 minutes though.

@blowdart

Long gone. Use imperative resource based checks. It was never part of asp.bet in the first place, it was code access security based which has been removed.

I hope this stays:

What @blowdart is referring to is something similar added to ASP.NET 5, but much improved. Check out the updated docs for more info (policy-based and resource-based authrization).

https://docs.asp.net/en/latest/security/authorization/resourcebased.html

Merging with #119

This issue is linked to here https://docs.microsoft.com/en-us/aspnet/core/security/authorization/authorization-filters
It seems like it's been closed incorrectly as  #119 does have a little bit about authorization filters but it doesn't cover it properly and the dedicated page has been left as a stub.
It seems either this issue should be re-opened or the stub should be removed.

@rtpHarry I think we just need to remove the stub. Most of the authorization related content is actually covered under the Authorization section, which is linked to from the Filters doc: https://docs.microsoft.com/en-us/aspnet/core/security/authorization/introduction.
Are there specific concerns related to authorization filters that you would like to see added to the docs?

I don't know about them, I'm just learning. I just wanted to point it out but I don't have a solution :)

@rtpHarry That's cool. Let us know if you have any specific questions!


Wasm update for parside wasm: add missing Deref's

This update affects the only wasm part of cesride:

Add Deref to each wrapper class to allow parside code to access the wrapped class. This is needed to create/re-wrap original rust classes in groups.

This is close. In the future it would be great if the demos you made in the other PR were actually exercised in CI, I have a pather PR up and this PR (#140) makes me realize that during the Bexter refactor I'd break wasm. But, tests are passing, giving me false confidence.

I agree. Someday I'll add basic tests for this.


[Cognitive Services - Anomaly Detector] API Review add version stable/2022-08-23

This meeting was created by Chunlei Wang.
Notes/Discussion

Document the ranges much better

POST /multivariate/models/{modelId}/entire/detect
These actions do not follow the convention for actions.
See: Performing an action
API Version

The use of version does not conform with the Azure guidelines, .../v1.1/

Recommendation

Update the API to conform to the Azure Guidelines.
Issue another preview
Work to migrate any customers

References
Design Doc & Change Description
Meeting Chat/Recording

See: #16984

Changes approved in #16984. Closing.


Website Bugs

On clicking Home, Features, About Us, or Services tab, the URL becomes this

Would it not technically be Routes instead of Subdomains here?

I think it would be helpful for the end user to know the ratelimits?

Actually yes you are right , I have to change my whole codebase and i will remove the front end with readthedocs file/website . This project is becoming very confusing and complex of extravagance.

Uhhh, sorry to bother you again, however there's been no action on this yet :no_mouth:


Generate percentiles from a neighbourhood for a single percentile

As an IMPROVER developer I want to be able to generate percentiles from a neighbourhood for a single percentile so that I can, for example, generate a wind gust diagnostic successfully, where I want to focus on processing a single percentile.
Related issues: #408
Currently, it is the case that when using the generate percentiles from a neighbourhood code (GeneratePercentilesFromACircularNeighbourhood) with a single percentile value there is a shape mismatch due to the promotion/demotion of what effectively becomes a scalar coordinate within pad_and_unpad_cube function. This needs to be fixed, so that the wind_gust_diagnostic processing chain can complete properly.
Acceptance criteria:

Fix the known bug.
Add unit tests and/or acceptance tests, so that this bug would be caught in future testing.

Completed.


Compass to inquiry pr

Changes have been added for Examination feature. In add library, filters list of questions based on evaluation mode 'server' or 'client'

@pavankumar0408 Please change the package version to "version": "7.0.0-beta.0" here - https://github.com/Sunbird-inQuiry/sunbird-resource-library/blob/release-7.0.0/projects/resource-library/package.json#L3
Note - Player web component version should also be updated in the package.json file. For reference: https://github.com/Sunbird-inQuiry/sunbird-resource-library/commit/0330a01ba0fed4bdbe198e9057e4b68200de4880

updated

@rajnishdargan , package has been updated


Table data and queries have been updated

What is added to the DATABASE

[ ] Add a column species of type string to

animals table.

[ ] update the animals table by setting the species column to unspecified. Verify that change was made. Then roll back the change and verify that the species columns went back to the state before the transaction.

[ ] Update the animals table by setting the species column to digimon for all animals that have a name ending in mon.

[ ] Update the animals table by setting the species column to pokemon for all animals that don't have species already set.

[ ] Verify that changes were made.

[ ] Commit the transaction.

[ ] Inside a transaction delete all records in the animals table, then roll back the transaction.

[ ] Inside a transaction:

Delete all animals born after Jan 1st, 2022.
Create a savepoint for the transaction.
Update all animals' weight to be their weight multiplied by -1.
Rollback to the savepoint
Update all animals' weights that are negative to be their weight multiplied by -1.
Commit transaction

[ ] How many animals are there?

[ ] How many animals have never tried to escape?

[ ] What is the average weight of animals?

[ ] Who escapes the most, neutered or not neutered animals

[ ] What is the minimum and maximum weight of each type of animal

[ ] What is the average number of escape attempts per animal type of those born between 1990 and 2000

Hello @MMhaidari👋,
This is an additional review as a result of the second opinion request.
@zieeco requested changes can be right only on some specific cases. However, based on your cases and the project requirements, it is acceptable to insert an integer number into a decimal column in PostgreSQL. PostgreSQL is designed to handle this scenario intelligently and will appropriately process the integer input without issues.
As this was the only reason that your project was not approved it will be approved as soon as possible.
Please ask for another review in your Dashboard and the next Code Reviewer will mark it as approve right away :)
Keep rocking 🚀!

#Note:

[ ] please read the above second opinion about the project


Using shared_mutex instead of mutex in CatalogSet

Description
With 50 concurrent short queries, mutex becomes a performance bottleneck.
Contributor agreement

[x] I have read and agree to the Contributor Agreement.

@ted-wq-x can you also fix the compilation warnings and clang-tidy checks?


Theme Toggle

[ ] Clean all CSS files from posts
[x] Add support for Dark/Light mode
[x] Automatic toggle based on system pref.

Known Bugs

[ ] Sub-menu in nav-bar is transparent (Looks bad only in light theme)


Belive in community

Hi!!
I would like so much help you!! Please share here the code with the community. Have some issues to fix and we have the time to fix for you! your First Macro Deck work so fine need a little fix. Please consider this!
Thanks!!

Macro Deck 2 is in development and many things are fixed there. Also all integrations are now plugins which will be open-source.


requirements: Upgrade 'Django'

Upgrade Django 2.2.18 -> 2.2.21 with fixes to some known vulnerabilities

CVE-2021-3281
CVE-2021-28658
CVE-2021-31542

Codecov Report

Merging #223 (dd19d8e) into develop (fd39c62) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #223   +/-   ##
========================================
  Coverage    81.02%   81.02%
========================================
  Files           32       32
  Lines         2525     2525
  Branches       375      375
========================================
  Hits          2046     2046
  Misses         306      306
  Partials       173      173

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 fd39c62...dd19d8e. Read the comment docs.


use python tools / test.py error?

What is the following question?
Traceback (most recent call last):
File "tools/test2.py", line 3, in
from mmdet.models import build_detector
File "/usr/local/lib/python3.5/dist-packages/mmdet-0.5.2+3f5df4f-py3.5.egg/mmdet/models/init.py", line 1, in
from .detectors import (BaseDetector, TwoStageDetector, RPN, FastRCNN,
File "/usr/local/lib/python3.5/dist-packages/mmdet-0.5.2+3f5df4f-py3.5.egg/mmdet/models/detectors/init.py", line 1, in
from .base import BaseDetector
File "/usr/local/lib/python3.5/dist-packages/mmdet-0.5.2+3f5df4f-py3.5.egg/mmdet/models/detectors/base.py", line 8, in
from mmdet.core import tensor2imgs, get_classes
File "/usr/local/lib/python3.5/dist-packages/mmdet-0.5.2+3f5df4f-py3.5.egg/mmdet/core/init.py", line 6, in
from .post_processing import *  # noqa: F401, F403
File "/usr/local/lib/python3.5/dist-packages/mmdet-0.5.2+3f5df4f-py3.5.egg/mmdet/core/post_processing/init.py", line 1, in
from .bbox_nms import multiclass_nms
File "/usr/local/lib/python3.5/dist-packages/mmdet-0.5.2+3f5df4f-py3.5.egg/mmdet/core/post_processing/bbox_nms.py", line 3, in
from mmdet.ops import nms
File "/usr/local/lib/python3.5/dist-packages/mmdet-0.5.2+3f5df4f-py3.5.egg/mmdet/ops/init.py", line 1, in
from .nms import nms, soft_nms
File "/usr/local/lib/python3.5/dist-packages/mmdet-0.5.2+3f5df4f-py3.5.egg/mmdet/ops/nms/init.py", line 1, in
from .nms_wrapper import nms, soft_nms
File "/usr/local/lib/python3.5/dist-packages/mmdet-0.5.2+3f5df4f-py3.5.egg/mmdet/ops/nms/nms_wrapper.py", line 4, in
from .gpu_nms import gpu_nms
ImportError: /usr/local/lib/python3.5/dist-packages/mmdet-0.5.2+3f5df4f-py3.5.egg/mmdet/ops/nms/gpu_nms.so: undefined symbol: _Py_ZeroStruct

May be related to #48

Thanks. It has been settled.

@yghcats How do you solve it?


[suggestion] Support for Android

would it be out of the realm of possibility to make an android app version of better discord for android I'm sure there are better ways to do it but maybe use chromium (open source version of chrome) to built a better discord app Discord is a web based app on linux windows and mac os so building it out of open source chromium may be one way to achieve

Android support? that's effort.
No one's willing to do it.
You don't have to go through such effort to bundle Chrome, it's not like you can't modify the native Android app.
Use Substratum, this is assuming that people only needed the theming aspect of BetterDiscord.

Yes it's much out of the realm. Android is an entirely different environment and nothing that currently exists in the BD world would work there. There are many other factors that go into not doing this as well but that is a big one.


error when downloading a large Crunchyroll episode

When I try to download a long episode of Demon Slayer it only ends up downloading the subtitles.
https://www.crunchyroll.com/pt-pt/watch/GX9UQE82J/the-hashira-unite

hello bro can you tell me how can i get cdm keys

hello bro can you tell me how can i get cdm keys

I saw the tutorial in an article Crunchyroll Downloader which was probably removed because it was illegal in .
original link: https://github.com/OpenSTDL/CrunchyDL/discussions/23
ex:

but here you have it, I used the one from the first link:
https://forum.videohelp.com/threads/408031-Dumping-Your-own-L3-CDM-with-Android-Studio
https://github.com/wvdumper/dumper

hello bro can you tell me how can i get cdm keys

I can't, as you can see, I tried to explain it to you but the post was removed.

When I try to download a long episode of Demon Slayer it only ends up downloading the subtitles.
https://www.crunchyroll.com/pt-pt/watch/GX9UQE82J/the-hashira-unite

i have no problem downloading the episode with the same dub/sub settings - so the problem most likely is on your end
make sure your settings are correct and maybe turn on log mode to see if there are any errors

When I try to download a long episode of Demon Slayer it only ends up downloading the subtitles.
https://www.crunchyroll.com/pt-pt/watch/GX9UQE82J/the-hashira-unite

i have no problem downloading the episode with the same dub/sub settings - so the problem most likely is on your end make sure your settings are correct and maybe turn on log mode to see if there are any errors

https://github.com/user-attachments/assets/a5224195-d790-48ce-b357-da37df1f4f0d
Even with Log Mode it continues to give problems and I don't know what it could be, I'll try to install a clean installation and see if it could be some configuration...

did you change anything in the muxing settings?

did you change anything in the muxing settings?

yes only, the first 4 options.

most likely a problem with the mp4 option

most likely a problem with the mp4 option

I can post the logfile later if necessary to analyze what it could be.

logfile.txt:
Log mode enabled.
[ffmpeg] Merging failed with exit code -12

https://github.com/user-attachments/assets/6f8d4325-128c-4819-9e32-f1af1e944bde

if you disable chapters it should work

if you disable chapters it should work

I will test thanks.


bug with brackets

Dim Json As Object
Set Json = JsonConverter.ParseJson("{""a"":123,""b"":[{""Name"":""abc""},{""Name"":""cde""}]}")
Debug.Print (Json("a"))
Debug.Print (Json("b"))

I'm not sure why the first print statement works whereas the second one doesn't

Your "b" data is contained as two elements in an array (the square brackets) so you have to address the elements of that array to return results.
These statements worked for me:
Debug.Print (Json("b")(1)("Name"))  'gives abcDebug.Print (Json("b")(2)("Name"))  'gives cde
Hope it helps!
On Thursday, December 9, 2021, 04:22:46 PM EST, Raunak Thomas<EMAIL_ADDRESS>wrote:

Dim Json As Object
Set Json = JsonConverter.ParseJson("{""a"":123,""b"":[{""Name"":""abc""},{""Name"":""cde""}]}")
Debug.Print (Json("a"))
Debug.Print (Json("b"))
I'm not sure why the first print statement works whereas the second one doesn't
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.

Glad it helped, we've all suffered through it at one time or another!
Another user helped me through it over the Christmas holidays a few years ago....amazing.
On Monday, October 10, 2022 at 03:20:31 PM EDT, Chris Cocuzzo<EMAIL_ADDRESS>wrote:

Your "b" data is contained as two elements in an array (the square brackets) so you have to address the elements of that array to return results. These statements worked for me: Debug.Print (Json("b")(1)("Name")) 'gives abcDebug.Print (Json("b")(2)("Name")) 'gives cde Hope it helps! On Thursday, December 9, 2021, 04:22:46 PM EST, Raunak Thomas @.***> wrote: Dim Json As Object Set Json = JsonConverter.ParseJson("{""a"":123,""b"":[{""Name"":""abc""},{""Name"":""cde""}]}") Debug.Print (Json("a")) Debug.Print (Json("b")) I'm not sure why the first print statement works whereas the second one doesn't — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub, or unsubscribe. Triage notifications on the go with GitHub Mobile for iOS or Android.
Tremendously helpful - thank you. Was suffering through this today and your answer solved my problem!
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: @.***>


PhantomJS has crashed.

<EMAIL_ADDRESS>commented:

> Which version of PhantomJS are you using?
1.6.0
> What steps will reproduce the problem?

Run "phantomjs test.js" in your terminal. Find test.js file in attach.

> What is the expected output? What do you see instead?
Expected output is html page.
"Segmentation fault (core dumped)" I see instead of it.
> Which operating system are you using?
Ubuntu Desktop 12.04
> Did you use binary PhantomJS or did you compile it from source?
Binary
> Please provide any additional information below.
See the crash dump file in attach.

Disclaimer:
This issue was migrated on 2013-03-15 from the project's former issue tracker on Google Code, Issue #675.
:star2:   2 people had starred this issue at the time of migration.

Supposedly this was fixed with commit 40a14b72b132468f4bbdfc6ff2dca30407cf866e.


Add option override if yarn / npm is used

I have a strange bug with electron-builder. If it is started with npm or npx, it just freezes at some point. However, when started with yarn, it works fine.
So I'd like to tell the action to definitely use yarn, even if the package-lock.json file exists. Is that possible?
My current workaround is to rename the package-lock.json file to something else before running this action, but that's not very practicable.

You need to pick between Yarn and NPM, don't use both. It can cause some annoying issues.

OK, I understand that it is quite a weird feature that I'm requesting. So never mind, for me it works that way.


Add more benchmarks to documentation

After https://github.com/47deg/helios/pull/50 was merged we will have the current Helios benchmarking.
We also want to add the Benchmarks comparing it with the other libraries (executeBenchmarks task will give us that) and upload it as part of the CI on the same way master_benchmark does.

Closed by https://github.com/47deg/helios/pull/62


SplashScreen Prefs for delay quirks

Currently this project won't run quite right off the bat for iOS (and potentially other platforms other than android) due to the config.xml file's current preferences for Splash Screen. It will hang on the splash screen indefinitely.
When these preferences are specified:
  <preference name="AutoHideSplashScreen" value="false" />
  <preference name="SplashScreenDelay" value="10000" />

There's a quirk in the Splash Screen plugin that requires you to also add some code like the following after deviceready to delay the splash screen for a certain number of seconds (in this case 1):
  setTimeout(function() {
       navigator.splashscreen.hide();
   }, 1000);

This could be added into the starter project in the app.js startApp() function.
Or you could move those preferences into the android platform in the config.xml file specifically:
    <!-- ANDROID -->
<platform name="android">
          <preference name="SplashScreenDelay" value="10000" />
          <preference name="AutoHideSplashScreen" value="false" />
           ...
</platform>

I plan to create a PR to help fix up the config.xml file and whatever else based on the other things I opened this morning but wanted to document these issues at least for now in case others run across them. :)

  setTimeout(function() {
       navigator.splashscreen.hide();
   }, 1000);

Thanks for the report @hollyschinsky!  I think the above solution is fine,  as it is the probably the most platform consistent?
Ideally the splash screen would only be open for as long as necessary by the device,  does the above facilitate that?

Yes @dcousens, I think overall it's a better approach to programmatically handle it using the splash screen plugin for better control.
If you're looking for the least possible amount of display time, you could first try to hide it immediately upon deviceready with just the navigator.splashscreen.hide() line itself and no timeout function.
It's really dependent on how much time any given app needs to load as far as including it in some kind of timeout or not. You'll know if it's not enough time if you see something display between the splash screen and the main app's view, like a plain white screen (or the default webview background color). If you do then you'll need to configure an appropriate timeout.
Hopefully this makes sense :)

@hollyschinsky what would be the settings to enforce "forever displayed" on both devices?
I feel like that would be the best default configuration if we are implying users should use .hide().

You could just not use the plugin or timeout at all and let the default native behavior occur.

I think in a similar way to how the KeystoneJS generator works we should add comments explaining how to handle splash screens properly, but let the the default behaviour occur.
Thoughts @dcousens, @hollyschinsky, @JedWatson?

@jossmac I think that would work well,  though,  I'm not familiar with what would classify as the default behaviour?

Remember before we started using the plugin/timeout in our apps there was that white flash between the splash screen and the loaded app?

Right, we certainly don't want that.   So what I'm thinking is we find the equivalent configuration for an infinite timeout,  such that it never goes away unless you ask it to.
IMHO, that makes the path easy.

Wouldn't that be confusing for someone unfamiliar with cordova, touchstone, et al.? Imagine this experience:

clone app
cordova prepare
splash screen for dayz


avoid to add one header key twice

when a http request to a standby node, it will proxy to leader node, some header key will be added twice, such as 'Access-Control-Allow-Origin', that would cause trouble when we do cross domain request. detail refer to http://stackoverflow.com/q/37594403

Codecov Report

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

@@           Coverage Diff           @@
##           master     #600   +/-   ##
=======================================
  Coverage   19.74%   19.74%
=======================================
  Files          29       29
  Lines        1732     1732
=======================================
  Hits          342      342
  Misses       1346     1346
  Partials       44       44

👍


$ionicPopup problem in beta 13

there seems to be a new bug introduced in beta 13:
inside a popup show I have a popup alert and the second one can;t be closed anymore because the taping happens under it
if i use e.preventDefault instead of the caps comment then it closes but the other popup (the show) appears again
 $ionicPopup.show({
                    template: template,
                    title: resources.SUCCESS,
                    scope: $scope,
                    buttons: [
                      {
                          text: resources.REMOVE_UPLOAD,
                          type: 'button-assertive',
                          onTap: function (e) {
                              ServerCall.removeUpload(uploaded.id).then(function (res) {
                                  // I CANT CLOSE THIS POPUP BECAUSE IT SCROLLS THE MODAL CONTENT UNDER
                                  $ionicPopup.alert({
                                      title: resources.SUCCESS,
                                      template: resources.REMOVED_WITH_SUCCESS,
                                  });
                              }, function (res) {
                                  $ionicPopup.alert({
                                      title: 'Code: ' + res.error,
                                      template: res.error_message,
                                      okType: 'button-assertive'
                                  });
                              });
                          }
                      },
                      {
                          text: 'OK',
                          type: 'button-positive',
                          onTap: function (e) {
                              $scope.closeFoodAddModal();
                          }
                      },
                    ]
                });

this solves the problem for me, but is ugly:
*** note the * var parent = * and * parent.close()*
var parent = $ionicPopup.show({
                    template: template,
                    title: resources.SUCCESS,
                    scope: $scope,
                    buttons: [
                      {
                          text: resources.REMOVE_UPLOAD,
                          type: 'button-assertive',
                          onTap: function (e) {
                              e.preventDefault();
                              ServerCall.removeUpload(uploaded.id).then(function (res) {
                                  $ionicPopup.alert({
                                      title: resources.SUCCESS,
                                      template: resources.REMOVED_WITH_SUCCESS,
                                  }).then(function () {
                                      parent.close();
                                  })
                              }, function (res) {
                                  $ionicPopup.alert({
                                      title: 'Code: ' + res.error,
                                      template: res.error_message,
                                      okType: 'button-assertive'
                                  });
                              });
                          }
                      },
                      {
                          text: 'OK',
                          type: 'button-positive',
                          onTap: function (e) {
                              $scope.closeFoodAddModal();
                          }
                      },
                    ]
                });

Greetings!
My sensors indicate a reply was requested, however, since we have not received one yet, this will be closed. If you are still experiencing this issue, please feel free to reopen it and include any information that may have been requested.
Thank you!


New path name for GoogleVR camera

Cardboard has changed to GoogleVR now that they have added additional support for the upcoming Daydream

Good catch and thank you! It completely slipped my mind that I mentioned the old directory and prefab here and that would be a pretty confusing mistake.

Glad to help! Nice work on the code, it's helping me with my senior project. I'm using your Unity package for a virtual building tour program. I was gonna do the same thing, but yours does everything I want so far.

Thanks, I love hearing from people who get use out of it. I'd also be interested in seeing your finished product if you're willing to share. Best of luck!


[Weird Bug] [kbfsfuse] Disrupts video playback even when not on a keybase folder

I have the same video. One in a folder on my HDD, one in a folder on keybase share.
opening both of them in VLC produces errors which I have sent in the message of the log.
I sent logs right after viewing the video for a good 2-3 minutes and observing the behavior.
watch from direct HDD = 2bf5d9ffffc8fcb82330291c
This was stopping every 10-20 seconds for about 5 seconds per stop, then an error would occur in VLC terminal. (sent in comment)
watch from kbfsfuse dir = b6b5e24038dc42017cd4331c
This was stopping much more often. Every 3-4 seconds... The errors were changing every time, talking about jitter as well.
killing kbfsfuse process and then watching the same video from the HDD is 100 smooth. No stops. No errors.
I am running on a GPU of Nvidia GTX 1060 using proprietary drivers 396.51 on Linux Mint 18.3 Cinnamon.
Note: None of the other Linux PCs I tried this on had any problems. kbfsfuse wasn't causing problems with video playback of unrelated files.
Also, I recently uploaded 95 GB of photos and movies to a kbfs share... so that might have caused problems as it's a weird edge case-y sounding situation.
Let me know if you need more info.

@junderw KBFS is getting input/output errors from the disk when trying to read from our data cache.  I realize the video plays fine from the HDD when kbfsfuse is not running, but are you sure your HDD is ok?
If you're confident that it's not a problem with your disk, maybe it's just some corruption in our on-disk cache files?  (Normally I wouldn't expect an I/O error in that case, but I'm not 100% sure.)  If so, you can try stopping Keybase, fully removing $HOME/.local/share/keybase/kbfs_block_cache, and restarting it.

I will check when I get home next.

fully removing $HOME/.local/share/keybase/kbfs_block_cache, and restarting it.

Fixed! Now I can watch my babies eating cakes and being weird without lag! :-D

Seems like the cache was broken or something.... hmmm....

I sent another log (I lost the code, can I list the IDs of all previous log sends?). It happened again.
I wrote a detailed comment on the log so it shouldn't be hard.
It seems clear that the cause is probably a mixture of the following:

using rsync -av
uploading a large video file

Though I don't think that rsync alone is causing it, since I synced the folder 4 times over a week and no problems occurred. But those were all just pictures and the rsync command finished very quickly.
However yesterday there was one 530 MB mp4 file (4K video) mixed in that, for some reason, took a very long time to upload. my cpu and internet monitor was showing cpu spikes of kbfsfuse, then upload @ 3.5 MB/s for abour 30-40 seconds, then it stops, repeat...
But it took much much longer than 20 minutes... my computer is pretty beefy, so I highly doubt kbfsfuse is having trouble encrypting.
Then after kbfsfuse finished, and cpu stopped rising, and internet stopped uploading for a while, I started checking the movie, and the same issue popped up. (including non-fuse files getting choppy)
Deleting the cache fixed the problem again.
It looks like I can reproduce the problem consistently on my PC... but uploading the same files using rsync with the repro steps I found on my desktop on a different computer causes no problems....................... ??????
perhaps I should send logs directly after finishing uploading the problem files...?

00b6b4db5e36ed667a41fc1c
Sent another log. This time it was caused just by watching a movie. But now it only causes stutter for kbfs and not for direct-disk movies...

And this time deleting cache doesn't fix the problem... :-(

I don't see anything obviously wrong in the logs you sent (at first glance), but each only cover a couple minutes or less since there's so much activity.
Did you ever run a hard drive diagnostics against the drive serving your home directory?  Based on what I saw last time, I wouldn't be surprised if it was having issues.


[dashboard] How to handle storage?

When creating a new TfJob, specifying a volume might me needed depending on what storage is used, i.e for rook, gluster, azure files etc.
How can we provide a solution in the front end that would allow mounting all of those?

Wouldn't giving the user options to specify the volume be sufficient?
Can you assign a priority to this issue?

@jlewi Yes but each kind of volume might have different parameters (and so a specific component in the UI).
For example:

rook:

- name: storage
  flexVolume:
    driver: rook.io/rook
    fsType: ceph
    options:
      fsName: somefs
      clusterName: somecluster

gluster:

- name: storage
  glusterfs:
    endpoints: someendpoint
    path: somepath

Azure files:

- name: storage
  azureFile:
    secretName: secret
    shareName: share
    readOnly: false

So I am wondering if there is a smarter solution than creating one component for each kind of volume?

/close
Closing it since we do not have dashboard now.


Parameter #1 ...$selectors of method PHPat\Test\Builder\SubjectStep::classes() expects PHPat\Selector\SelectorInterface, PHPat\Selector\Modifier\AndModifier given

Bug Description
I'm not sure if this is a phpat bug or a phpstan bug. I've written the following test, but I'm getting an error, but IMHO this is not a valid error since AndModifier implements SelectorInterface
 Line   test/PHPat/Process.php
 ------ ---------------------------------------------------------------------------------------------------------------------------------------------------------------------
   7     Parameter #1 ...$selectors of method PHPat\Test\Builder\SubjectStep::classes() expects PHPat\Selector\SelectorInterface, PHPat\Selector\Modifier\AndModifier given.

final class Process
{
    public function testImplementsInterface(): Rule
    {
        return PHPat::rule()
            ->classes(
                Selector::AND(
                    Selector::inNamespace('Process'),
                    Selector::NOT(
                        Selector::isInterface()
                    )
                )
            )
            ->shouldImplement()
            ->classes(
                Selector::classname(ProcessInterface::class)
            )
            ->because('Each process should implement ProcessInterface');
    }
}

Hi. Did you figure out what was happening?
I just run your code and it works properly. Which PHPStan/PHPat version are you using?

Unable to reproduce it, please let me know if you can provide a way to do it


Detach nw utilities from _

Contour relies on lodash being on the global namespace which it then alters to add utility functions.  Requiring lodash in other app modules would reset lodash breaking future instances of Contour.  Found that putting utilitities in own namespace nw resolves this issue.
Future We should still consider:

not relying on libraries in global space.
registering forio-contour with npm

Thanks for this Dan! Will take a look and merge later next week.

Thanks for this. FYI, some projects are dependent on utilities in _.nw.

What projects?  If you really want to extend _ with nw why not extend the library and make _ an internal dependency?  Otherwise it seems like it might be better to put nw on its own.

Hi @breadbaker - appreciate the PR.
I'm a little curious as to why we need to set nw as a global variable on the window here - can you explain your thought process, and could this be accomplished without setting an additional global variable?

I agree with @parikhshiv's https://github.com/forio/contour/pull/237#issuecomment-268656463 about polluting the global namespace. Ideally we'd reduce access to nw at the same time as detaching it from _.
A couple ideas:

Attach it to Contour as Contour.nw.
Use binding context, ie. make it available on the prototype. This would keep nw totally private.

Note that some visualizations use _.nw, so those would need to be changed. If I remember correctly, at least one sim uses _.nw, and I don't remember which.

Contour.NW is a great solution!
On Thu, Dec 22, 2016 at 5:12 AM David Rosen<EMAIL_ADDRESS>wrote:

I agree with @parikhshiv https://github.com/parikhshiv's #237 (comment)
https://github.com/forio/contour/pull/237#issuecomment-268656463 about
polluting the global namespace. Ideally we'd reduce access to nw at the
same time as detaching it from _.
A couple ideas:

Attach it to Contour as Contour.nw.

Use binding context, ie. make it available on the prototype. This
would keep nw totally private.

Note that some visualizations use _.nw, so those would need to be
changed. If I remember correctly, at least one sim uses _.nw, and I don't
remember which.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/forio/contour/pull/237#issuecomment-268700276, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AArbWJl45EkaUS5wZZiN8_HC8UtniefKks5rKdx1gaJpZM4JS6sX
.

Contour is not defined until well after all of the _.nw helpers are defined.


Docs

Addresses #3

Current coverage is 98.61% (diff: 100%)

No coverage report found for master at 8f59796.

Powered by Codecov. Last update 8f59796...c9bf1a2


DataSet._get_valuemap(): force list of ints as return for delimited sets.

Otherwise .reindex() fails and data is NaN.

Resolved by #533.


Crash with Adobe Acrobat DC (x64)

Hello,
I really love your tool, the only problem I encountered is that when the tool is active, Adobe reader won't work anymore and crash everytime on opening.
I don't see what link there could be between those two application.
Any possibility it could be fix, or maybe a workaround ?
Thanks !

Please try to change config.ini noerror=true

Same result.
Also, I tested it on Windows 11 and on a fresh Win 10 install.

This should be a problem caused by the file dialog feature.
The temporary solution is to disable this feature, at the cost of not being able to display the background image in the file dialog in third-party applications.
Delete the registry entry:
HKEY_LOCAL_MACHINE\SOFTWARE\Classes\Drive\shellex\FolderExtensions\{ED15A97D-FE3E-4CDE-98FF-BC46B02896B0}
(This registry entry is the registry entry for this tool. Deleting it will do no harm to your computer)

That was it indeed.
Actually, I just wanted to be able to add a background in the file explorer, so I'm fine with that.
It work perfectly now, thanks!


Use discord.py from source & request message_content intent

PR changes discord.py back to the source repository and adds the message content intent.
This is only a PR because I don't know if the message content intent is ticked on in the dev portal.

Should we sit around until dpy is properly released? The only reason it's pulled directly from git at the moment is because your fork was just rolling-release.


Values can be ignored by using _ in place of a variable or const declaration.

In chapter 1, assignment, https://ziglearn.org/chapter-1/#assignment, there is the following:

Values can be ignored by using _ in place of a variable or const declaration.

 _ = 10;
Can someone give me a more practical example. I don't see how this can be used. I tried multiple things but I only get errors.

pub fn doStuff() u32 {
    // do stuff with side effects like printing
    return 10;
}

pub fn main() !void {
    doStuff(); // error: expression value is ignored
    _ = doStuff(); // no error
}

If doStuff would be a pure function (no side effects, only depends on the inputs) then calling it without using its result would be meaningless.
If doStuff has side effects (writes a file, prints to terminal, changes global variables), then calling it without using its result can be a valid use case.

Ok thanks. I think it would make sense if the example on the ziglearn page would be a bit more clear about this.


Added offEvent to the exports for manual call

Thank you. How about this API? Does it meet your needs?
import { useWebApp } from "vue-tg"

const { onEvent } = useWebApp()

const { unsubscribe } = onEvent(
  "mainButtonClicked",
  () => {
    /* main button handler */
  },
  {
    manual: true, // 1. Add option to disable auto-unsubscribe
  },
)

// 2. Manually unsubscribe later
unsubscribe()

Thank you. How about this API? Does it meet your needs?
import { useWebApp } from "vue-tg"

const { onEvent } = useWebApp()

const { unsubscribe } = onEvent(
  "mainButtonClicked",
  () => {
    /* main button handler */
  },
  {
    manual: true, // 1. Add option to disable auto-unsubscribe
  },
)

// 2. Manually unsubscribe later
unsubscribe()

I think so


Fix RYB to RGB conversion

The previous technique resulted in the darkest colour possible being brown.
I tried fixing the RYB interpolation cube constants as there was a mistake (I found the paper in which they were presented), but the browinish color remained.
So I used a different techique for the conversion which works wonderfully.
If there's any objections on the code I'll handle them.
Before:

After:

This fixes #5.

Nice! That fixes #5

Is this going to get merged?

So I'm quite attached to the current RYB colour model which looks and feels very natural and is easy to create pleasing colour combinations with. It's from Gossett and Chen's 'Paint Inspired Color Mixing and Compositing for Visualization' (http://web.siat.ac.cn/~baoquan/papers/InfoVis_Paint.pdf) - what was the mistake that you found in the implementation?
However, I do appreciate the need for a colour model that includes black and the other colours that aren't attainable in this RYB model. So I've added the ability to toggle to a standard straight RGB model (dubbed 'Digital'). I prefer having this straight-up choice between the two models as opposed to a halfway colour model that I feel dilutes the respective advantages of either model. I have tried previously to extend the RYB model to include black by making the dark brown corner of the colour space cube black, but colours become duller when mixed and black paint doesn't behave quite as you'd expect (it ends up having a brownish tinge).
Thanks anyway for taking the time to think about this and contribute this pull request!

Thanks, I understand why having both options would be preferable.
The mistake in the implementation is in the 110 vertex:
    return trilinearInterpolate(ryb,
        vec3(1.0, 1.0, 1.0),
        vec3(1.0, 0.0, 0.0),
        vec3(0.163, 0.373, 0.6),
        vec3(1.0, 1.0, 0.0),
        vec3(1.0, 0.5, 0.0),
        vec3(0.0, 0.66, 0.2),
        vec3(0.5, 0.0, 0.5),  <-------
        vec3(0.2, 0.094, 0.0));
}

According to the paper, it should be vec3(0.5, 0.5, 0.0) instead. :)

Ah, I believe that's actually a typo in the paper. The purple corner in figure 3 should definitely be (0.5, 0.0, 0.5).


chore: split MetricSpace.basic

This reduces the main file from 3340 to 2220 lines.
The remaining file is somewhat entangled, so splitting is less obvious. Help is welcome, though a follow-up PR is probably better :-)
I've kept copyright and authors as they were originally.

Open questions.

should Cauchy.lean be merged with CauSeqFilter?
split the results on Additive, Multiplicative and Order duals into metric and pseudometric parts? (I also like how they're grouped currently.)

@urkud FWIW, this will conflict with #7903 - though these are rather small.

Thanks a lot for doing this! I agree that some points are open to discussion, but all in all the choices you've made are pretty coherent, so let's get this is now, and revisit later if needed.
bors r+


Providing PDF Vector Assets

Hey there 🤘
what about PDF assets? This would save a lot of space in an iOS Project when providing 250 PDF's instead of 750 PNG's ;-)
Cheers
Stefan

Please se this for the reason we don't use svg:s for apple platforms: https://bjango.com/articles/idontusepdfs/


MSID: 99990 Version: 2 DOI: 10.1101/2<IP_ADDRESS>1083

MSID: 99990
Version: 2
Preprint DOI: https://doi.org/10.1101/2<IP_ADDRESS>1083
Step 1. Awaiting reviews
Editorial to post reviews via hypothesis
Useful links:

DocMap: https://data-hub-api.elifesciences.org/enhanced-preprints/docmaps/v2/by-publisher/elife/get-by-manuscript-id?manuscript_id=99990
New model tracking: https://docs.google.com/spreadsheets/d/1_fHaoOy7hjyocptKtVJRijeNpUY4hBS7Ck_aVmx6ZJk/
Reviews on sciety: https://sciety.org/articles/activity/10.1101/2<IP_ADDRESS>1083

For trouble shooting (e.g. no Docmaps available):

DocMap issue addressing: https://miro.com/app/board/uXjVNCwK6EI=/
Explore DataHub DocMaps API: https://lookerstudio.google.com/reporting/4c2f0368-babb-4beb-b5b3-497e7e7b0f08/page/ejphD
Unmatched submissions and preprints: https://lookerstudio.google.com/u/0/reporting/9f86204f-3bf7-477c-9b18-5c5ef141bf69/page/p_gxi57ha93c
Unmatched manuscripts spreadsheet: https://docs.google.com/spreadsheets/d/15QcK8w-ssB7109RQEDtFpJPZ0J5HTGxoHa_2TtpMBbg/edit#gid=1336081641

Step 2. Preview reviewed preprint
Production QC content ahead of publication
Instructions:

QC preview: https://prod--epp.elifesciences.org/previews/99990v2
Update ticket with any problems (add blocked label)
When QC OK, add QC OK label to ticket and add publication date and time to https://docs.google.com/spreadsheets/d/1amAlKvdLcaDp5W8Z8g77NmkwbMF5n_u89ArSqPMO8jg
Move card to next column
(At end of the day post link in #enhanced-preprint and ask for PDF to be generated)

Useful links:

Preprint DOI :  https://doi.org/10.1101/2<IP_ADDRESS>1083
Confirm reviews returned by EPP: https://prod--epp.elifesciences.org/api/reviewed-preprints/99990/v2/reviews
To update the MECA path in the docmap: https://docs.google.com/spreadsheets/d/1mctCQuNFBjSn97Lihy7_vBO6z7-N-oqyLv4clyi6zHg

Step 3: Awaiting search reindex
This step adds the reviewed preprint to the homepage: https://elifesciences.org
The search reindex is triggered once an hour. We need the reviewed preprint to be indexed as the search application serves the journal homepage.
Useful links:

Jenkins pipeline to reindex search can be triggered sooner or monitored here: https://alfred.elifesciences.org/job/process/job/process-reindex-reviewed-preprints/

Step 4: Published! PDF requested
Waiting for PDF to be generated
Useful links:

PDF tracking: https://docs.google.com/spreadsheets/d/106_XeDjmuBae7gexOTNzg60lapeqjl2aRn9DzupGyS8/

Step 5: Introduce PDF to data folder and git repo
Upload PDF to relevent folder in git repo https://github.com/elifesciences/enhanced-preprints-data/
Step 6: Done!

[ ] Kettle is on!

Hi @JGilbert-eLife the authors aren't happy with the eLife Assessment on this; please could we hold the revised RP while we discuss with the editors?

Sure thing!


Users/midenn/symbols compress

This pull request adds support for the /compress option on the symstore.exe command that the Index & Publish Symbols task executes when users are publishing symbols to a fileshare. This was in result to a customer request and it was pretty easy to add support for it.

@arunkm and @TingluoHuang could you both take a quick look?

@arunkm updated the PR - did a squash on the commits too. In the end making the argument a switch doesn't add much value because we need to use Get-VstsInput anyway to acquire the value which means the argument gets specified either way (we end up with a : instead of a space)

Ready for another review @arunkm


Let students restart the course and take the quizzes as many times as they like.

What can we improve?
I ask that the Restart Course be enabled at all times. I am an adult learner and as a software professional, I am in charge of my learning. I know what learning style works for me. I paid for the course. I can access the videos and the text as many times as I like, but apparently Leanpub or the course creators decided that I can only take each quiz twice and that's it. If I "complete the course", I can no longer go back and test myself on the material. Why? It doesn't hurt anyone. It costs way more to stream the videos and load the text every time I access the course. Let me also take the quizzes as many times as I like. Eliminate this "No. of Attempts" concept. Also, what the heck is "completing" a course about  Spring and AWS. It takes months and years of revisiting this stuff over and over again to actually "learn it".

Version
Revision 1.1 (2023-04-21)
Suggested Changes
I ask that the Restart Course be enabled at all times. Remove any limitation on the number of times I take a quiz, or complete the course.

Hi @shantdashjian,
I wasn't aware of such a restriction. I agree that this should not be. I have reached out to Leanpub support because I can't find a setting to change this restriction. Will keep you posted here.

@shantdashjian @thombergs I've now changed the number of quiz attempts to unlimited on each individual request and created this pull request for making that setting global as well: https://github.com/stratospheric-dev/stratospheric-course/pull/11
This change should therefore be available with the next version of the course.

@shantdashjian just to be clear: we already published a new version and it should work already. The above PR is just an improvement. Please let us know if it works.

I see no change. It may be because you would also need to enable restarting the course/making that unlimited as well. I also don't see a new version.

@shantdashjian Unfortunately, there currently is an error with Leanpub's publishing system that prevents us from setting an infinite number of quiz attempts. Therefore, for the time being, we've set the quiz attempts to 100, which should be plenty enough. I hope this helps with your issue (and an independent, self-directed learning approach in general).
The new course release should be available in a bit.


update JsonResponse: support set json_dumps_params

I see, thanks for the details!
And if you have not done it yet, would you mind adding your name to the list of contributors?
https://github.com/cloudera/hue/wiki/Ccla#online-form
Thanks!


Example in document does not work

Tutorial (https://giftless.datopian.com/en/latest/quickstart.html)
does not work as raise following error occured.

Expected result (in documents)

INFO <IP_ADDRESS> - - "POST /my-organization/test-repo/objects/batch HTTP/1.1" 200 -
INFO <IP_ADDRESS> - - "PUT /my-organization/test-repo/objects/storage/30e14955ebf1352266dc2ff8067e68104607e750abb9d3b36582b8af909fcb58 HTTP/1.1" 200 -
INFO <IP_ADDRESS> - - "POST /my-organization/test-repo/objects/storage/verify HTTP/1.1" 200 -

Actual result

2022-11-24 08:08:40,040 werkzeug        INFO  * Running on http://<IP_ADDRESS>:12789 (Press CTRL+C to quit)
2022-11-24 08:09:09,525 werkzeug        INFO <IP_ADDRESS> - - [24/Nov/2022 08:09:09] "POST /my-organization/test-repo/locks/verify HTTP/1.1" 404 -
2022-11-24 08:09:09,533 werkzeug        ERROR <IP_ADDRESS> - - [24/Nov/2022 08:09:09] code 400, message Bad request syntax ('{"ref":{"name":"refs/heads/master"}}')
2022-11-24 08:09:09,533 werkzeug        INFO <IP_ADDRESS> - - [24/Nov/2022 08:09:09] "None /my-organization/test-repo/locks/verify HTTP/0.9" HTTPStatus.BAD_REQUEST -

So, How can I solve this issue?
I think /my-organization/test-repo/locks/verify  path does not exist.

I also experienced this issue.

FIXED. I believe this got fixed in https://github.com/datopian/giftless/pull/115 so closing ... Feel free to re-open if this reappears.


[BEAM-8780]Improve docker container release script

Improve docker container script: reuse verified rc images instead of recreating images.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

R: @markflyhigh
Cc: @kyle-winkelman do we need to do the same thing for Flink images?
Cc: @Ardagan

LGTM overall.
There's also a way to do this without pulling image. See https://issues.apache.org/jira/browse/BEAM-8780
Would be great if we can utilize that one.

Thanks for reviewing it.
I investigate both options mentioned at BEAM-8780. The option with updating manifest is not recommended to use in production. (https://docs.docker.com/engine/reference/commandline/manifest/). The second option with docker build creates new images, so that is not an option here.
After all, I think pull, tag and push process is the most stable and easy to understand for now.

The PR is ready to be merged to master.


SAscorer looks for fpscores in hard-coded path

Hi,
I tried to run the example in the readme to process CrossDock molecules, but the sascorer.py file tries to look up fpscores in a hard-coded path, /srv/home/mahdi.ghorbani/FragDiff/fpscores.pkl.gz. Is this file available somewhere, or can I generate it myself? Thanks,
Amanda

Hi Amanda,
Thanks for reaching out. I've added the fpscores.pkl.gz file in the latest push to the repository. You can find it here. Please let me know if you encounter any issues or have further questions.


make TextField() behave more as expected

Describe the bug
there is some basic universal expected behaviour of textfields. this should also be the case in compose desktop.
currently it is not so.
Affected platforms

Desktop

Versions

Kotlin version*: 1.9.22
Compose Multiplatform version*: 1.5.12
OS version(s)* (required for Desktop and iOS issues): Linux
OS architecture (x86 or arm64): x86
JDK (for desktop issues): 17

To Reproduce
create a TextField() then:
keys on the numblock of the keyboard (when numlock is not active) do nothing
numblock enter does nothing
middle mouse button click does nothing
middle mouse button marking of some text doesnt not put it into the copy buffer
pressing shift+return does nothing (instead alt+return inserts a newline, which is less common. people expect shift return to do that)
pressing shift+numblock_return does nothing
Expected behavior
keys on the numblock of the keyboard (when numlock is not active) do as expected: cursor up/down/left/right/pos1/end/pgup/pgdown/del
numblock enter sends return
middle mouse button click does insert contents from the copy buffer
middle mouse button marking of some text does put it into the copy buffer
pressing shift+return inserts a newline
pressing shift+numblock_return inserts a newline
and last, if alt-return inserts a newline then at least alt-numblock_return should do the same

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.

why are you closing tickets and referring to another platform?


Management network container optional

Issues Resolved by this Pull Request
Fixes #710
#714
#715
#716
#717
#718
#719
#736
#737
#742
Description of the Solution
Added changes for making the management network container optional .
Suggested Reviewers
@sujit-jadhav @j0hnL @lwilson

These changes are for devel branch.


Test: use Bypass for remote context test

Thank you for fixing my problem in #7.
That URL will likely not stay alive, so I wanted to make sure tests won't fail if it goes offline.

Oh, I thought it was the official context of the litepub spec.
Perfect. Thanks for contribution.


How to use Ludusavi with Rclone Mount

What's your question?
I see the documentation says If you use Rclone's mounting functionality, then you can configure Ludusavi to use the mount folder.
How? I'm trying to do this and I don't understand.

Do I just need to make my backup folder a folder on the mount?

Do I just need to make my backup folder a folder on the mount?

Yeah, you can just change the path on the backup screen. I can edit that section to make it clearer.

Got it. Thank you, this worked 🙂


Add SetAutoFormatHeaders to CustomTable

Hey team this library is very useful!
This PR adds an option to prevent the automatic formatting of table headers.

@Avihuc
Thank you for the PR.
Your PR may break compatibility, but it's not a big issue. I will merge it.


Add support for matrix jobs

This PR adds support for matrix jobs, including fail-fast and max-parallel.

 Parallelise deployments to different environments


Feature Request: Support generic validation

Hi,
I just employed a hack to use a validation library other than class-validator. Since validateModels: true will import class-validator dynamically, I just fed it a fake package of the same name. It simply exports a validate function act as a stand-in replacement for class-validator.
The validation interface is quite trivial:
interface IEntityValidator {
  (entity: IEntity, validateOptions?: Record<string, unknown>): Array<any>
}

So changing initialize to accept
interface IFireormInitOptions  {
  validateModels?: boolean
  validate?: IEntityValidator
}

would allow for trivially wrapping every validation library under the sun in an IEntityValidator and returning an array containing arbitrary validation errors, whether that's simple strings or elaborate objects.
It would still support the current behavior of passing in validateModels and default to using class-validator. There is an easy deprecation path for phasing out the class-validator-only method, if desired. Eventually, if one chooses to use class-validator it's merely
import { getFirestore } from 'my-firestore-setup';
import { validate } from 'class-validator';
import { initialize } from 'fireorm';

initialize(getFirestore(), { validate })

which in my estimation is acceptable boilerplate for being able to use arbitrary validation methods.
I'd be happy to submit a PR in case this proposal is to your liking.

Thanks for your help @juni0r! How coupled fireorm is with class-validator is one of the reasons I don't like the integration. Thanks for your help, your insights have been helpful!


refactor(header): :lipstick: fix width overflow

Changed the tailwind class 'w-screen' to 'w-full' to ensure it doesn't overlap with the scrollbar.

Thanks for contributing!


Uncaught TypeError: Cannot read property 'id' of undefined

[Enter steps to reproduce:]

...
...

Atom: 1.45.0 x64
Electron: 4.2.7
OS: Mac OS X 10.14.6
Thrown From: Atom Core
Stack Trace
Uncaught TypeError: Cannot read property 'id' of undefined
At /Applications/Atom.app/Contents/Resources/app/static/<embedded>:11

TypeError: Cannot read property 'id' of undefined
    at extraRenderedScreenLines.forEach (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:11:76051)
    at Map.forEach (<anonymous>)
    at TextEditorComponent.renderLineTiles (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:11:75983)
    at TextEditorComponent.renderContent (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:11:74509)
    at TextEditorComponent.renderScrollContainer (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:11:73969)
    at TextEditorComponent.render (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:11:72748)
    at Object.updateSync (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:14:132897)
    at TextEditorComponent.updateSyncBeforeMeasuringContent (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:11:70606)
    at TextEditorComponent.updateSync (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:11:67993)
    at TextEditorComponent.didShow (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:11:91682)
    at TextEditorComponent.didFocus (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:11:91873)
    at HTMLElement.addEventListener.event (/app.asar/src/text-editor-element.js:37:27)
    at HTMLElement.activeItemChanged (/app.asar/src/pane-element.js:164:16)
    at Function.simpleDispatch (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:11:1214120)
    at Emitter.emit (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:11:1215561)
    at Pane.setActiveItem (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:11:455685)
    at Pane.activateItem (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:11:457933)
    at TabBarView.onClick (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:11:1165207)

Commands
     -2:37.7.0 markdown-preview:toggle (input.hidden-input)
     -2:29.7.0 core:copy (div.markdown-preview.has-selection)
  2x -0:49.6.0 markdown-preview:toggle (div.markdown-preview.has-selection)

Non-Core Packages
ask-stack 2.2.0
atom-beautify 0.33.4
atom-ide-ui 0.13.0
atom-language-r 1.4.8
file-icons 2.1.40
highlight-selected 0.17.0
ide-r 0.1.8
pigments 0.40.2
todo 0.6.0

Thanks for reaching out!
We require the template to be filled out on all new issues and pull requests. We do this so that we can be certain we have all the information we need to address your submission efficiently. This allows the maintainers to spend more time fixing bugs, implementing enhancements, and reviewing and merging pull requests.
Thanks for understanding and meeting us half way 😀

I've just had this happen to me as well. Atom's error message says that the bug has already been reported, and links to this page.
At the time, I had a markdown file open. I:

Opened markdown-preview-enhanced
Closed markdown-preview-enhanced
Switched focus to a different window (a zoom meeting)
Re-sized the zoom window to be larger, which:
Re-sized the atom window to be smaller

I assume that this triggered the scroll event in the stack trace

Two identical error messages appear in the upper-right corner of the window

Attempting to click into the atom window caused another error message to appear. I have the contents of the second message below. The contents of the third, from clicking into atom, were identical, but the "commands" section was empty. Unfortunately I do not have the contents of the first message.
This is the first time I have seen this error after many years of use (and many months with this package set). I understand that this may not be worth investigating due to its rarity, but I would like to document this trace here anyways in the hope it might help in the future.
Atom: 1.55.0 x64
Electron: 6.1.12
OS: "Arch Linux"
Thrown From: Atom Core
Stack Trace
Uncaught TypeError: Cannot read property 'id' of undefined
At /usr/lib/atom/src/text-editor-component.js:698

TypeError: Cannot read property 'id' of undefined
    at /usr/lib/atom/src/text-editor-component.js:698:42
    at Map.forEach (<anonymous>)
    at TextEditorComponent.renderLineTiles (/usr/lib/atom/src/text-editor-component.js:694:37)
    at TextEditorComponent.renderContent (/usr/lib/atom/src/text-editor-component.js:622:12)
    at TextEditorComponent.renderScrollContainer (/usr/lib/atom/src/text-editor-component.js:596:12)
    at TextEditorComponent.render (/usr/lib/atom/src/text-editor-component.js:541:14)
    at Object.updateSync (/usr/lib/atom/node_modules/etch/lib/component-helpers.js:119:34)
    at TextEditorComponent.updateSyncBeforeMeasuringContent (/usr/lib/atom/src/text-editor-component.js:409:10)
    at TextEditorComponent.updateSync (/usr/lib/atom/src/text-editor-component.js:279:10)
    at TextEditorComponent.didMouseWheel (/usr/lib/atom/src/text-editor-component.js:1735:12)

Commands
     -8:54.6.0 core:backspace (input.hidden-input)
     -8:54 editor:move-to-end-of-screen-line (input.hidden-input)
  8x -8:51.9.0 core:backspace (input.hidden-input)
  2x -8:13.6.0 editor:newline (input.hidden-input)
  2x -8:11.1.0 core:backspace (input.hidden-input)

Non-Core Packages
atom-clock 0.1.18
atom-ide-ui 0.13.0
autosave-onchange 1.5.1
dracula-syntax 3.0.2
firacode 0.3.0
ide-rust 0.22.0-beta.3
language-babel 2.85.0
language-docker 1.1.8
language-dot 1.7.0
language-ini 1.23.0
language-latex 1.2.0
language-nginx 0.8.0
livestyle-atom 0.2.10
markdown-preview-enhanced 0.18.9
pdf-view 0.72.0
teletype 0.13.4
wordcount 3.2.0
zotero-citations 5.2.0

Thanks for reaching out!
We require the template to be filled out on all new issues and pull requests. We do this so that we can be certain we have all the information we need to address your submission efficiently. This allows the maintainers to spend more time fixing bugs, implementing enhancements, and reviewing and merging pull requests.
Thanks for understanding and meeting us half way 😀

😱 I am mortified that I opened this issue and didn't follow up! I was new to GitHub then -- thanks for being so kind @darangi. I haven't used Atom for Markdown editing since April 2020 and don't remember what my experience was, so unfortunately I can't amend the original issue to see if it aligns with the experience @Navigatron is having.

I am still hoping to learn in any case. :)


[BugFix] reset the logicalProperty in low cardinality rewrite process

What type of PR is this：

[x] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：

Fixes #
Problem Summary(Required) ：

In low cardinality rewrite process, we need update the logicalProperty of an operator when it's children had been encoded. The limit operator forgot to do it while other operator has the process.
@Override
        public OptExpression visitPhysicalLimit(OptExpression optExpression, DecodeContext context) {
            visitProjectionBefore(optExpression, context);
            OptExpression childExpr = optExpression.inputAt(0);
            context.hasEncoded = false;

            OptExpression newChildExpr = childExpr.getOp().accept(this, childExpr, context);
            optExpression.setChild(0, newChildExpr);
            return visitProjectionAfter(optExpression, context);
        }

The isExecuteInOneTablet  field in LogicalProperty is used to clear partitionExprs of OlapScanNode (the bucket keys to pass to BE) when it's true and we alse need to keep this info when update LogicalProperty.
Checklist:

[x] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto backported to target branch

[ ] 2.5
[ ] 2.4
[ ] 2.3
[ ] 2.2

run starrocks_fe_unittest

[FE PR Coverage Check]
:heart_eyes: pass : 8 / 8 (100.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/sql/optimizer/rule/tree/AddDecodeNodeForDictStringRule.java
8
8
100.00%
[]

run starrocks_admit_test

run starrocks_admit_test

[FE PR Coverage Check]
:heart_eyes: pass : 8 / 8 (100.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/sql/optimizer/rule/tree/AddDecodeNodeForDictStringRule.java
8
8
100.00%
[]

run starrocks_admit_test

run starrocks_admit_test


Improve the extension system

Currently, the extension has a few flaws:

Unloading an extension does not automatically remove its callbacks. Event callback ownership could help on this point.
Should this include an API change or not? It might be possible to use the extension's classloader to determine if a callback is owned by a given extension when unloading that extension.
If we are willing to trade some memory for performance, Minestom could even cache the callbacks when they are added to immediately remove the extensions callbacks on unloading, without iterating through all active callbacks.
If Mixin are not used by an extension, and the Mixin library does not load, this should not crash the server.
Extensions should be able to mixin into some Minestom classes before they are loaded (InstanceContainer for instance). This might require to load extensions earlier than the MinecraftServer.init() call inside the Bootstrap class
Because extension loading is currently toggleable, this option may require a command line option or a System property to be added to Minestom to turn extension loading.
Investigate issues with "new" JRE versions when J9 modules are present in the classpath, as security has been reinforced.

Callback to unload automatically:

[ ] Global event handlers
[ ] SchedulerManager tasks
[ ] // TODO, add others here

Additional features

[ ] Unload extensions properly when the server stops


demos: Fix disappearing objects in smoketest

Fixes LunarG/VulkanSamples#150
When an object reaches the end of its "curve" path
it receives a new curve.  To ease the transition from
the old curve to the new curve, the origin of the new
curve is the ending point of the old curve.  However,
this can result in the starting point of the new curve
being out of the view.  This results in objects slowly
leaving the view.
Add code to move the origin of the new curve back inside
the original view bounds only if the end of the old curve
is out of the view bounds.  This does not add any additional
jarring because any object getting a new origin is already
outside the view anyway.
Change-Id: I50e88be453f31b9c37878f63c97d4c7c8154e718

Force-pushed a fix to address build failure on Windows.


Move calico-policy-controller into separate role

By default Calico CNI does not create any network access policies or profiles if 'policy' is enabled in CNI config. And without any policies/profiles network access to/from PODs is blocked.
K8s related policies are created by calico-policy-controller in such case. So we need to start it as soon as possible, before any real workloads.
This patch also fixes kube-api port in calico-policy-controller yaml template.
Closes #1132

ci check this


Added gitignore that ignores asset files for text example when compiled with VS

The VS build generates some assets for the text example, that were not gitignored. Added a new gitignore file for the VS examples.

Ah, different name so https://github.com/pimoroni/32blit-beta/blob/7ec17189544e75f7e17a135d38d13988c7bc2524/vs/.gitignore#L359 didn't catch it...

Ah, different name so https://github.com/pimoroni/32blit-beta/blob/7ec17189544e75f7e17a135d38d13988c7bc2524/vs/.gitignore#L359 didn't catch it...

Should probably just stick this in 32blit-beta/vs/.gitignore, although this project using "font_asset" against the grain of others is probably going to quietly gnaw away at me :laughing:

Should probably just stick this in 32blit-beta/vs/.gitignore, although this project using "font_asset" against the grain of others is probably going to quietly gnaw away at me :laughing:

OCD is a wonderful feature to have!! 🤣

OCD is a wonderful feature to have!! 🤣

Will... uh... stick that on the TODO :laughing:
Thanks!

Will... uh... stick that on the TODO :laughing:
Thanks!


Introduce Natural language API

Docs: https://cloud.google.com/natural-language/
@jdpedrie @bshaffer @tmatsuo Please let me know what you think of this direction :smile:

I'm on the fence regarding your use of the name Document. In the service, a Document refers to a request, whereas in your wrapper a Document is a representation of a response. Unless there's a strong reason for how this is setup, I'd prefer closer adherence to those concepts in the way they are used by the service.
I think Annotation, as used in Vision is a reasonable alternative for representing a response.

Would DocumentResponse be more acceptable?

@jdpedrie I renamed Document to Annotation, could you please take a look when you have a moment and make sure everything looks okay? :)

Looks good other than noted.


Python API Fix

Use _ for system packages so sealing doesn't break the API.

Yes, "I'm ok with my commits being contributed to this project". See https://github.com/tensorflow/tensorflow/pull/15890.

@aselle  Looks like you just need to fix the conflict and call this done.  I checked and I am fairly sure CLA is all covered and we have a note from the secondary author.

This is now fixed in 1.8.0. So I think we can close this.


Printing debug info about browser taking ~75 sec or more on BrowserStack, need a way to turn it off

DriverFactoryHelper.printDebugInfoForUser is taking ~75 sec or more on BrowserStack which is huge impact on automation suites in terms of time and this is obstacle for us to scale our automation.
DriverFactoryHelper.printDebugInfoForUser is being called from WebDriverFatory:
https://github.com/paypal/SeLion/blob/develop/client/src/main/java/com/paypal/selion/internal/platform/grid/browsercapabilities/WebDriverFactory.java#L87
Here is simple calculation on how time we are going to save (just for an example):
No. of tests * 75 sec * no. of browsers (i.e., 500 tests * 75 sec = 37500 sec ~ 10 hrs per browser )
I think having a configurable option to turn it off (default = ON) would be useful for users who uses BrowserStack.

Note: @SudhakaraPeram asked to also contribute the fix here. 😊

@SudhakaraPeram closing this issue since your changes are merged. Hopefully you are seeing an improvement in the latest 1.1.0-SNAPSHOT :)

@mach6 Yes, we are seeing an improvement in total execution time. Thanks!


Rename "apis" dir to exclude it by default

Without this, things like gopls will try to load all the generated files and die. This lets the IDE tooling be more useful, but to have e.g. controller-tools scan every file the ./... path must be used from the _apis directory or it will be ignored.

Need to restore some files before review

Need to restore some files before review

Codecov Report

Merging #372 (f2c2156) into master (0ab0045) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #372   +/-   ##
=======================================
  Coverage   62.48%   62.48%
=======================================
  Files         123      123
  Lines        8296     8296
=======================================
  Hits         5184     5184
  Misses       2637     2637
  Partials      475      475

Impacted Files
Coverage Δ

...oft.resources/v20200601/resourcegroup_arm_types.go
100.00% <ø> (ø)

...crosoft.resources/v20200601/resourcegroup_types.go
43.47% <ø> (ø)

hack/generated/main.go
10.63% <ø> (ø)

hack/generated/pkg/testcommon/kube_test_context.go
78.16% <ø> (ø)

hack/generated/pkg/testcommon/test_context.go
54.87% <ø> (ø)

.../generator/pkg/astmodel/local_package_reference.go
100.00% <ø> (ø)

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 0ab0045...f2c2156. Read the comment docs.

Codecov Report

Merging #372 (f2c2156) into master (0ab0045) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #372   +/-   ##
=======================================
  Coverage   62.48%   62.48%
=======================================
  Files         123      123
  Lines        8296     8296
=======================================
  Hits         5184     5184
  Misses       2637     2637
  Partials      475      475

Impacted Files
Coverage Δ

...oft.resources/v20200601/resourcegroup_arm_types.go
100.00% <ø> (ø)

...crosoft.resources/v20200601/resourcegroup_types.go
43.47% <ø> (ø)

hack/generated/main.go
10.63% <ø> (ø)

hack/generated/pkg/testcommon/kube_test_context.go
78.16% <ø> (ø)

hack/generated/pkg/testcommon/test_context.go
54.87% <ø> (ø)

.../generator/pkg/astmodel/local_package_reference.go
100.00% <ø> (ø)

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 0ab0045...f2c2156. Read the comment docs.


feat: Allow endpoints in sub folders

Allows endpoints to be placed in sub folders.
Close #675
Pre-launch Checklist

[x] I read the Contribute page and followed the process outlined there for submitting PRs.
[x] This update contains only one single feature or bug fix and nothing else. (If you are submitting multiple fixes, please make multiple PRs.)
[x] I read and followed the Dart Style Guide and formatted the code with dart format.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///), and made sure that the documentation follows the same style as other Serverpod documentation. I checked spelling and grammar.
[ ] I added new tests to check the change I am making.
[ ] All existing and new tests are passing.
[x] Any breaking changes are documented below.

Breaking changes
none

I still have to add tests.
@vlidholt Would it be enough to add an endpoint in a sub folder? I think having no compiler errors after generation should be enough.
Open for debate: https://github.com/serverpod/serverpod/issues/675#issuecomment-1452353370