scheduler: Use heap-sort instead of quickselect

The HA scheduling algorithm introduced code that built a slice of all
nodes meeting the constraints, then selected the k best nodes from that
slice.
Building the slice can be expensive if there are a lot of nodes.
To improve performance, we can build a max-heap of nodes and pop all of
the elements to perform a heap sort. We never need to have more than k
items in the heap at any time, since the heap enables a trivial
optimization to check if a node is "better" than the root of the heap.
If it isn't, we can skip that node, and if it is, we can replace the
root of the heap with it.
This approach needs less memory allocation with k << len(nodes).
It also removes the dependency on quickselect.
Thanks to @LK4D4 for the idea.

Benchmark results:

benchmark                                           old ns/op       new ns/op       delta
BenchmarkScheduler1kNodes1kTasks-8                  99535349        98353639        -1.19%
BenchmarkScheduler1kNodes10kTasks-8                 546633387       548325249       +0.31%
BenchmarkScheduler1kNodes100kTasks-8<PHONE_NUMBER><PHONE_NUMBER>      +1.65%
BenchmarkScheduler100kNodes1kTasks-8<PHONE_NUMBER>      311341903       -83.68%
BenchmarkScheduler100kNodes100kTasks-8<PHONE_NUMBER><PHONE_NUMBER>      +1.66%
BenchmarkScheduler100kNodes1MTasks-8<PHONE_NUMBER>2<PHONE_NUMBER>4     -6.23%
BenchmarkSchedulerConstraints1kNodes1kTasks-8       165552129       153443398       -7.31%
BenchmarkSchedulerConstraints1kNodes10kTasks-8      729888274       722834862       -0.97%
BenchmarkSchedulerConstraints1kNodes100kTasks-8<PHONE_NUMBER><PHONE_NUMBER>      +2.97%
BenchmarkSchedulerConstraints5kNodes100kTasks-8<PHONE_NUMBER><PHONE_NUMBER>      -1.31%

benchmark                                           old allocs     new allocs     delta
BenchmarkScheduler1kNodes1kTasks-8                  243095         244065         +0.40%
BenchmarkScheduler1kNodes10kTasks-8                 2760071        2760575        +0.02%
BenchmarkScheduler1kNodes100kTasks-8                31112694       31106768       -0.02%
BenchmarkScheduler100kNodes1kTasks-8                348289         349075         +0.23%
BenchmarkScheduler100kNodes100kTasks-8              31005741       31105897       +0.32%
BenchmarkScheduler100kNodes1MTasks-8                345009629      345098851      +0.03%
BenchmarkSchedulerConstraints1kNodes1kTasks-8       255069         257582         +0.99%
BenchmarkSchedulerConstraints1kNodes10kTasks-8      2879668        2880483        +0.03%
BenchmarkSchedulerConstraints1kNodes100kTasks-8     32308673       32314324       +0.02%
BenchmarkSchedulerConstraints5kNodes100kTasks-8     32164883       32168084       +0.01%

benchmark                                           old bytes       new bytes       delta
BenchmarkScheduler1kNodes1kTasks-8                  17889218        17944530        +0.31%
BenchmarkScheduler1kNodes10kTasks-8                 198313008       198338596       +0.01%
BenchmarkScheduler1kNodes100kTasks-8<PHONE_NUMBER><PHONE_NUMBER>      -0.04%
BenchmarkScheduler100kNodes1kTasks-8                69665912        43189427        -38.00%
BenchmarkScheduler100kNodes100kTasks-8<PHONE_NUMBER><PHONE_NUMBER>      +0.24%
BenchmarkScheduler100kNodes1MTasks-8<PHONE_NUMBER>0<PHONE_NUMBER>4     -0.00%
BenchmarkSchedulerConstraints1kNodes1kTasks-8       18517088        18688487        +0.93%
BenchmarkSchedulerConstraints1kNodes10kTasks-8      204626968       204619544       -0.00%
BenchmarkSchedulerConstraints1kNodes100kTasks-8<PHONE_NUMBER><PHONE_NUMBER>      +0.04%
BenchmarkSchedulerConstraints5kNodes100kTasks-8<PHONE_NUMBER><PHONE_NUMBER>      -0.03%

As expected, this is a big improvement if there are many more nodes than tasks being scheduled. Otherwise, the difference is minor.

Current coverage is 54.11% (diff: 92.00%)

Merging #1550 into master will increase coverage by 0.11%

@@             master      #1550   diff @@
==========================================
  Files            82         82
  Lines         13528      13539    +11
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           7305       7327    +22
+ Misses         5233       5228     -5
+ Partials        990        984     -6

Powered by Codecov. Last update a2e755d...c6d3ee5

LGTM


cli fails with go 1.5.x

go get -d github.com/cloudfoundry-incubator/lattice/ltc

package github.com/docker/docker/nat: cannot find package "github.com/docker/docker/nat" in any of:
	/usr/local/Cellar/go/1.5.1/libexec/src/github.com/docker/docker/nat (from $GOROOT)
	/Users/<username>/go/src/github.com/docker/docker/nat (from $GOPATH)

I tried cleaning my pkg and trying to godep restore.. still no luck. Does the CLI work with go 1.5.x?

The ltc CLI should work with Go 1.5.1 (at least) - Check our TravisCI out.  However, the Lattice cluster is only supported against go1.4, there's a fair number of changes that need to be made to support go1.5 there.
Our official development environment is described at: https://github.com/cloudfoundry-incubator/lattice/tree/master/development
Another way to make it work without godep restore is to tweak your GOPATH, similar to how we do in https://github.com/cloudfoundry-incubator/lattice/blob/master/.travis.yml
I haven't specifically tested the godep restore flow on go1.5, though.  See if any of these alternatives meet your needs, and if not, I can see if there's something up with godep restore + go1.5

We're now managing our ltc dependencies using submodules from lattice-release.  If you use direnv to set up the environment variables, ltc should just work after you:
$ git clone https://github.com/cloudfoundry-incubator/lattice-release --recursive
$ cd lattice-release
$ git submodule update --init --recursive
$ go install github.com/cloudfoundry-incubator/ltc

Let us know if that works for you


Add filters to let you filter by request method or status code

Feature Request
Filters should be added to the monitor page to filter by request method (POST, GET, PUT etc.) and by the response status code, e.g. to only show errors.

@alexbrazier
https://github.com/alexbrazier/react-native-network-logger/pull/39


Improve BatchedTensor(Seq)

Codecov Report
Patch coverage: 100.00% and no project coverage change.

Comparison is base (55d34bb) 100.00% compared to head (ac6b0a7) 100.00%.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #417   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           19        19
  Lines         1700      1695    -5
  Branches       176       172    -4
=========================================
- Hits          1700      1695    -5

Files Changed
Coverage Δ

src/redcat/tensor.py
100.00% <100.00%> (ø)

src/redcat/tensorseq.py
100.00% <100.00%> (ø)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


chromium: 123.0.6312.122 -> 124.0.6367.60, fix M124+ build by using the vendored libwebp

Description of changes
https://chromereleases.googleblog.com/2024/04/stable-channel-update-for-desktop_16.html
This update includes 23 security fixes.
CVEs:
CVE-2024-3832 CVE-2024-3833 CVE-2024-3914 CVE-2024-3834 CVE-2024-3837
CVE-2024-3838 CVE-2024-3839 CVE-2024-3840 CVE-2024-3841 CVE-2024-3843
CVE-2024-3844 CVE-2024-3845 CVE-2024-3846 CVE-2024-3847
Building with our libwebp runs into the following error:
Running phase: configurePhase
ERROR Unresolved dependencies.
//third_party/libavif:libavif_enc(//build/toolchain/linux/unbundle:default)
  needs //third_party/libwebp:libwebp_sharpyuv(//build/toolchain/linux/unbundle:default)

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 304768 run on x86_64-linux 1

2 packages marked as broken and skipped:

archivebox
archivebox.dist

20 packages built:

aws-azure-login
chromium
chromium.sandbox
fast-cli
google-chrome
mermaid-cli
mermaid-filter
netflix
percollate
playwright-test
puppeteer-cli
python311Packages.pytest-playwright
python311Packages.pytest-playwright.dist
python312Packages.pytest-playwright
python312Packages.pytest-playwright.dist
sharedown
single-file-cli
sitespeed-io
vhs
wayback

Manual testing on wayland (wlroots, sway) failed.
Looks like ozone is having some issues.
--ozone-platform-hint=x11 works, while --ozone-platform-hint=wayland spawns an empty, transparent window, with no window decoration.
The VM test on the other hand passes, because it's x11 only:
# git describe --abbrev=40
24.05-pre-61968-g3b11cc57660b6db07b13a91c9338f33035072252

# nix-build -A stable nixos/tests/chromium.nix
/nix/store/mb6axl86yvq4v18viqw63ymw7r59hbb5-vm-test-run-chromium-stable

Seems to be a known bug for M124: https://issues.chromium.org/issues/329678163
Will try to backport https://chromium-review.googlesource.com/c/chromium/src/+/5375669 (https://github.com/chromium/chromium/commit/c7f4c58f896a651eba80ad805ebdb49d19ebdbd4) manually.

Yay --ozone-platform-hint=auto and --ozone-platform-hint=wayland work again :)
Result of nixpkgs-review pr 304768 run on x86_64-linux 1

2 packages marked as broken and skipped:

archivebox
archivebox.dist

20 packages built:

aws-azure-login
chromium
chromium.sandbox
fast-cli
google-chrome
mermaid-cli
mermaid-filter
netflix
percollate
playwright-test
puppeteer-cli
python311Packages.pytest-playwright
python311Packages.pytest-playwright.dist
python312Packages.pytest-playwright
python312Packages.pytest-playwright.dist
sharedown
single-file-cli
sitespeed-io
vhs
wayback

# git describe --abbrev=40
24.05-pre-62320-gd9ec9a35dfcc45be442a038fde944221e843ab28

# nix-build -A stable nixos/tests/chromium.nix
/nix/store/aj4pmh37x0ncmdbaqnl5ps8fkj0gz486-vm-test-run-chromium-stable


There are typos

./src/content/blog/2023/03/22/react-labs-what-we-have-been-working-on-march-2023.md:77:187: `visiblity` -> `visibility`
./src/content/reference/react-dom/components/form.md:276:70: `Optimisitc` -> `Optimistic`
./src/content/reference/react/cache.md:312:117: `accessibile` -> `accessible`
./src/content/reference/react/experimental_taintUniqueValue.md:134:53: `explictly` -> `explicitly`
./src/content/learn/passing-data-deeply-with-context.md:988:103: `repells` -> `repels`
./src/content/learn/passing-data-deeply-with-context.md:1127:103: `repells` -> `repels`
./src/components/Layout/getRouteMeta.tsx:61:6: `Travesal` -> `Traversal`
./src/components/Layout/getRouteMeta.tsx:67:14: `Travesal` -> `Traversal`
./src/components/Layout/getRouteMeta.tsx:82:8: `Travesal` -> `Traversal`

Please consider adding typos to your workflow.

Configuration: .typos.toml
[files]
extend-exclude = [
    ".git/",
    "patches/",
]
ignore-hidden = false

[default]
extend-ignore-re = [
    '"hel"',
]

[default.extend-words]
# False positives
"seeked" = "seeked"

i want to work on this. I would fix issue


Dockerfile running for frontend and Secrets for Terraform

dockerfile for frontend builds and runs
Added GitHub Actions Workflow for terraform configurations using secrets as environment variable and changed credentials so that it's not hardcoded

why did you remove the terraform credentials setting block?  - name: Set up Terraform credentials run: | echo "GOOGLE_CREDENTIALS=${{ secrets.DEFAULT_SA_KEY }}" >> $GITHUB_ENV

whoops didn't mean to, i forgot to pull before pushing last merge so I had merge conflicts, I fixed it and added hold for approval into the yml


LinkFix: biztalk-docs (2022-05)

This PR is the result of running a Link Repair script on the included content. This script is primarily being run to clean up links to function correctly in Air Gapped Clouds (AGC). Here's a breakdown of what the script that generated the changes in this PR fixes:

docs.microsoft.com Fully Qualified Domain Name (FQDN) Links to either Relative (if in same Repo/DocSet/ContentSet) and Site Relative if not

azure.microsoft.com/documentation/articles that reflect to Docs to either Relative (if in same Repo/DocSet/ContentSet) and Site Relative if not

All known flavors of MSDN/TechNet domains (we know of 8 or 9 now) that reflect to Docs to either Relative (if in same Repo/DocSet/ContentSet) and Site Relative if not

This also includes cleanup of the appended query string for MSDN (redirectedfrom=MSDN).

Fixes articles that are redirected in .openpublishing.redirection.json and updates the link to the current final location (this has a lot of customer usability fixes).

Please note the Contributor Guide: Links has been updated with the following link-type preference order:

By order of preference, links hosted on Docs should be Relative if they are in the same repository and docset. If they are in a different docset, even if in the same repository, they should be Site Relative. Links to content hosted on Docs shouldn't use a complete URL, otherwise known as Fully Qualified Domain Names (FQDN). Using a complete URL from Docs to other content on Docs will cause that link to be non-functional in air-gap environments.

Docs Build status updates of commit 6d719c2:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

biztalk/core/tools-and-utilities-to-use-for-troubleshooting.md
:white_check_mark:Succeeded
View

biztalk/install-and-config-guides/troubleshoot-known-issues-biztalk-install-setup.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
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.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Great Plugin

I was waiting for someone to start with oAuth2 using passport for octobercms.
Passport is best way to oAuth2.
Im using it on my many production instances using laravel.
making it working with OCMS is little tricky as its user auth is different than laravel's.
I hope you might have checked https://github.com/LukeTowers/oc-passport-plugin
If i get time I'll definitely try to contribute.
Thanks!

Firstly, thanks a lot for the review. I’m glad that you like it.
I’ve already tried Luke’s plugin and it’s currently incomplete nor does not suit my need, which is why I made this plugin. Besides, Luke has planned it a particular way whereas I have planned that my plugin integrates with my RESTful plugin as well which would boost up the usage and would also make my users happy as it will solve REST API auth in a concise way. I had planned this plugin a long time ago but haven’t been able to work on it due to work contracts and was finally able to build this only now. Hopefully, this month I’ll get to finish the Auth plugin as well and make this plugin compatible with Auth and also link it up to RESTful so that OctoberCMS becomes a goto platform for building API-based backends that work with mobile apps.


Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
It seems the relative path are not working as expected in VS code
Version
1.0.2 (Stable)
Relevant log output
No response
Anything else?
No response

There are two parts in that issue.
Giving a file path
Pycharm runs the script from the script itself while VScode does it from the opened directory by default. So using the function os.get_cwd() gives 2 different results:

Pycharm : user_path_to_reho\REHO_tools\REHO\scripts\examples

VScode : user_path_to_reho\REHO_tools\REHO\

So when later on the function os.path.realpath(path_given) is used, we obtain:

Pycharm : user_path_to_reho\REHO_tools\REHO\scripts\template\data\buildings.csv

VScode : user_path_to_reho\REHO_tools\template\data\buildings.csv

This can be corrected by changing the settings of VScode python.terminal.executeInFileDir. It can be done by:

Going into File > Preferences > Settings > Python > Terminal: Execute In File Dir
Setting the "python.terminal.executeInFileDir": true in the settings.json file.

:warning: This should be properly added to the documentation

Importing the module
Now, with the current directory set to the script, we are not able to import reho as it does not belong to the path.
The reason why it still works in Pycharm is that it automatically adds to the sys.path the opened directory and so the same should be done but manually in VScode.
The workaround is to add in the settings.json the path to the environment:

Windows : Ctrl + Shift + P > open settings > click on Preferences: Open User Settings (JSON) > write:

"terminal.integrated.env.windows": {"PYTHONPATH": "${workspaceFolder}"}

Mac : F1 > open settings > click on Preferences: Open User Settings (JSON) > write:

"terminal.integrated.env.osx": {"PYTHONPATH": "${workspaceFolder}"}

If you have not done it before, add in the settings the line:
"python.terminal.executeInFileDir": true

The final file should look like this:
{
    "python.defaultInterpreterPath": "user_path_to_python\\python3.10.exe",
    "[python]": {
        "editor.formatOnType": true
    },
    "git.openRepositoryInParentFolders": "always",
    "hediet.vscode-drawio.resizeImages": null,
    "hediet.vscode-drawio.codeLinkActivated": false,
    "hediet.vscode-drawio.theme": "atlas",
    "svg.preview.mode": "svg",
    "terminal.integrated.env.windows": {"PYTHONPATH": "${workspaceFolder}"},
    "python.terminal.executeInFileDir": true,
}

Hi.  I also have this same issue when working in the terminal.  I have placed my .env file in a folder, and now wish to run the test run.py script which I downloaded from the repository using
    wget https://github.com/IPESE/REHO/raw/main/scripts/template/run.py

My folder looks like this:
my_folder
    - .env
    - run.py

Expected behaviour would be for the following to work:
$ cd my_folder
$ python run.py

But I also get the error involving .env.

Does the problem still remain after this commit https://github.com/IPESE/REHO/commit/4a63b03aec4f85ae097582e4bd1ab2c3b5d03e0c ?

@DorsanL Yes, this problem remains. I thought it was due to the Cloud license in #34, but I believe it is this issue rearing its head again.
If I install REHO (e.g. pip install REHO), create a .env file in my local working folder and then run the test reho-test-import , I still get the error Exception: AMPL_PATH is not defined. Please include a .env file at the project root (e.g., AMPL_PATH='C:/AMPL').  Except now, I do not have a project root.  I just have a working folder which could be anywhere on my file system.  It seems that REHO is making assumptions about where the .env file should go, but this is incompatible with a pip install (which places the package in an arbitrary location on my computer).

This should no longer be a problem after the changes here: https://github.com/IPESE/REHO/commit/b0ba53c1118e49b7a52d66655314b7280491da82
To sum up:

an .env file is no longer required with the cloud-based Community Edition license
to use a standard local license, one must specify the AMPL_PATH environment variable in a .env file that can be located either at the package root directory (i.e., in the reho/ directory) or at the current working directory (i.e., where the script is run)


Cart checkout

Fixes #49
Proposed Changes

(see #49)

Related

tests will come with a separate PR

Ok! :-) Feel free to merge.


Question: UIRayCastCamera Disappearing on click of create new in profiles

Overview
This is a question basically -  Is it an expected behavior that the UIRaycastCamera will be removed and the GazeProvider script component also be removed from Main camera when we try to create new Profiles? (Not Copy and Edit)
Is the developer expected to manually create the UIRaycastCam if needed and to assign the GazeProvider script to Main camera manually?
Unity version 2018.3.3 - 5f1
Mixed Reality Toolkit Release Version
mrtk_development

@krishnan-unni, thank you for the bug report. we will investigate and advise as soon as possible,

UI Raycast camera now created at runtime, not editor. Closing bug


need toggle_labels() function

Problem
Hi I see this plugin has create_labels() and remove_labels(), can you also add an toggle_labels() implement? so I can use one key binding for this plugin, thx
Expected behavior
can use toggle_labels() to toggle the window label's display.

That makes sense, thanks for the suggestion! I just added this feature. Do you mind testing it?


Documentation of Mongo DB

User story

As a developer
I need to document the analytics and log files needed in the process of working.

Acceptance criteria

Documentation of required info from mongoDB

Definition of done

Code review from peers has been completed and passed

Code is fully implemented and commented
Adheres to coding guidelines
Commits are not squashed
Code is checked into the repository

Unit tests for backend have been written and are passing
Manual performance tests on the browser are passing
No critical bugs are open

scraped


Update getting-user-input.md

I just found no space before =.

 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.


string -> stringBuffer on strcpy_s... page

When this page is viewed on the microsoft documentation page (https://docs.microsoft.com/en-us/cpp/c-runtime-library/reference/strcpy-s-wcscpy-s-mbscpy-s?view=msvc-160), the variable "string" is highlighted in code as if it were a type, and an array of type "wchar_t string" were being allocated. This is confusing - the "string" variable should be highlighted as if it were a variable.
I've renamed the "string" variable to "stringBuffer" to ensure it highlights correctly without changing the meaning of the code. However, I'm not familiar with this repository or the coding conventions used.

@cartwrightluke : Thanks for your contribution! The author(s) have been notified to review your proposed change.


translation of railways

Hi,
In transifex we have 3 'railway' texts :
railway=railway_crossing
railway=level_crossing
railway=crossing
They are all translate in french.
But in ID, the translations are not visible, we have english words.
Best regards

This is a duplicate of #207.

@Mahabarata
Are translations updated like reported in #207 ?

No, nothing is working :

the railway-railway crossing is missing (for tags railway=railway_crossing) : we have still the english version

the railroad-road crossing hasn't the present translation we find in transifex (i.e. "Passage à niveau (route)") but an old one (i.e. "Passage à niveau")

same thing for the railroad path crossing (present translation in transifex = "Passage à niveau (piéton)", present translation in ID = "Passage à niveau" (an old one in transifex).

For me it looks like you have imported in ID an old file from transifex so all the new translations are missing (me and another contributor have created about one thousand of missing translations in french for 2 months and nothing is in ID).
To be more specifix, after my searsh by date in transifex, it looks like all the translations made in transifex since August 2021 are missing.
Is it possible to delete the "closed issue" ?
Best regards

Another thing :
I think this issue is not the same than the issue #207.

Here I'm talking of translations in transifex.
in the other issue, they talk about translations in translatewiki.net : I changed a translation 3 weeks ago in translatewiki and yes, the translation is visible in ID.

Not the translation from transifex.
Best regards

Hi,
a small test :
here what we have in transifex for Portuguese :

Here what we have in ID :

transifex translations in Portuguese are, like for french, not in ID, for at least 3 months (between August and October, very few translations were made and no translation for objects like node, way, area. But it works for a Portuguese translation made July 19).
I also tried for Russian : translations after August 31 are not in ID.
My first supposition seems the good one : translations from translatewiki.net are ok, not from transifex.
This issue is clearly not closed, please change that.
Best regards

I assume the reasons is, that the id-tagging-schema was not released since "29 Jul 2021". And likely none of the translations there where updated here due to the missing release.
See https://github.com/openstreetmap/id-tagging-schema/releases
But Martin will know the internals better.
Martin wrote, somewhere, that a new release is planned for the next few weeks.

For easy access testing links with locale=pt

dev version https://ideditor.netlify.app/?lang=en#background=Bing&disable_features=boundaries&id=n-1&locale=pt&map=20.00/52.47409/13.44561
released version https://ideditor-release.netlify.app/#background=Bing&disable_features=boundaries&id=n-1&locale=pt&map=20.00/52.47409/13.44561

Yes, I agree with that :
but it is not what I'm asking in my last comments : tyrasd closed my issue and I'm not agree.
What I ask seems very simple : please open back my issue.
Or tell me you can't if nobody can do that.
Best regards

It is normal to close issues on tracker once it was fixed in development version, even if released version still have the bug.

I try both links of tordans, the release and the dev version, with an area with "leisure=indoor_play".
Both are identical : "indoor play center", not "centro de diversoes interior"
This issue is not closed for the release AND not closed for the dev version.
Please open it back.

@Mahabarata you're right, some recent translations, like the one you mentioned above (leisure=indoor_play) that was translated last October, were not updated in iD. I still don't know how this is implemented, but I can guarantee that translations that were stalled for almost a year (like place=city, which was changed a loooong ago) were updated on December 11th. That's why I told you on the other topic that they were updated, but as you confirmed, not all of them were updated. Don't ask me why...

Maybe the end is coming, yay (see #363)


Decrease margins of price scale for candlesticks

Hi there,
I have a question regarding the horizontal price line that gets automatically created when creating a line (e.g. for an indicator).
Is it possible to remove or disable this feature? I looked through the documentation and also in the js.py file but can't figure out how to do this.
Also how do I scale the y-axis so that it shows the lowest candle in the viewport at the bottom and the highest at the top and ignores all indicators? Is this somehow possible? Currently I have to manually adjust the y-axis to achieve this.
Thanks!

Hi,
The first part of your issue was raised in #16; I am adding an option to disable this line in the next update.
For the second part of your issue, could you drop a screenshot here of the output your getting on the chart, and the output that you would like to happen automatically? That way I can understand exactly what you mean.
Thanks!

Is it possible to remove or disable this feature? I looked through the documentation and also in the js.py file but can't figure out how to do this.
Can add 2 line like this if you need a workaround while waiting for the new version

Hi and thanks for your quick reply. Thank for adding the option in the next version can't wait!
As for the second point. Maybe it is what emma-uw mentioned in #16 with the fitContent(). So when I create a chart this is what my default screen looks like: Everything is centered but the candles are small due to the large y-axis spread.

This is what I want. So essentially I want the bottom of the y-axis to be the lowest wick of the viewport and the top to be the highest wick of the viewport (I circled the two candles white) and also +- an offset would be ideal. I managed to achieve this with other packages but always struggled with ignoring the indicators in the viewport (I circled them red). In other packages the y-axis would always fit to the highest and lowest value of the viewport, so this also includes the indicators and not only the candlestick series. Hope this helps to understand what I would want ideally. Basically a way to fit the y-axis to the high/low of the candlestick series in the viewport.

Is it possible to remove or disable this feature? I looked through the documentation and also in the js.py file but can't figure out how to do this. Can add 2 lines like this if you need a workaround while waiting for the new version

This worked perfectly, thank you very much! Gotta try more things now. :)

Just to update; the first part of your issue has been implemented in the latest version (1.0.11).
I'll take a look into adding price scale margins!
Louis

Just to update; the first part of your issue has been implemented in the latest version (1.0.11).
I'll take a look into adding price scale margins!
Louis

It works perfectly, thank you very much. And the lines common method is also really useful. I am currently storing all the lines in a dictionary for a specific ticker but thanks to the new update I can adjust my code.
Btw. are you also thinking about implementing the series compare? I will give it a try later and see if it works.

Just to update; the first part of your issue has been implemented in the latest version (1.0.11).
I'll take a look into adding price scale margins!
Louis

It works perfectly, thank you very much. And the lines common method is also really useful. I am currently storing all the lines in a dictionary for a specific ticker but thanks to the new update I can adjust my code.
Btw. are you also thinking about implementing the series compare? I will give it a try later and see if it works.

Glad I could help!
Technically the series compare example is already possible, by just using create_line and not setting any candle data.

I will give it a try and will let you know.
If you open TradingView and right click on the y-axis, it says 'scale price chart only'. So basically this is the second request. I think this makes it easier to understand.
Btw. Can you add a feature to disable the x-axis on subcharts? I did it like this:
def create_subchart(self, volume_enabled: bool = True, position: Literal['left', 'right', 'top', 'bottom'] = 'left',
                    width: float = 0.5, height: float = 0.5, sync: Union[bool, str] = False,
                    topbar: bool = False, searchbox: bool = False, timescale_enabled: bool = False):
    subchart = SubChart(self, volume_enabled, position, width, height, sync, topbar, searchbox,timescale_enabled)
    self._charts[subchart.id] = subchart
    return subchart

And then self.run_script(f'''
{self.id}.chart.applyOptions({{
timeScale: {{
visible: {_js_bool(timescale_enabled)},
timeVisible: {_js_bool(timescale_enabled)}
}}
}})''')
within the init of SubChart(LWC). I disabled it by default for me now since I don't need the x-axis for subcharts.

Hi,
This should already be possible with subchart.time_scale(visible=False)

The latest version of the library now has scale_margin_top and scale_margin_bottom parameters within the price_scale method, so you can directly change the margins from there.
As for the 'scale price chart only' option you mentioned in TradingView, I don't believe it has been implemented in Lightweight Charts.

Hey @TagHaendler,
Just to update, The 'scale price chart only' option you mentioned will be implemented in the next update.


Changed 'Prerequisite For' in course's additional information section…

… to 'Prerequisite for'
Summary
I changed the "For" in the course's additional information section to a lowercase "for"

Test Plan

Verified that the lowercase "for" shows up instead of the uppercase one

Issues
Closes #269

I’m going to upgrade my npm version and then I'll re-review this.

Also I just wanna point out, the package-lock.json stuff is totally not your fault!
I didn't realize that I was using an older version of npm. Thanks for bringing that to my attention - it should help other fellowship students who open PRs after you.

@LawrenceDavidLu can you "Allow edits from maintainers"?
I'd like to resolve the merge conflicts in the package-lock.

https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/working-with-forks/allowing-changes-to-a-pull-request-branch-created-from-a-fork#enabling-repository-maintainer-permissions-on-existing-pull-requests

@ChaseC99 Okay! "Allow edits from maintainers" is now checked.


HAProxy needs a metric type facelift.

As of the new HAProxy stats we need to emit metrics in their correct
types.

+1
But it might be a good idea to wait for a +1 from someone who knows more about these metrics.

Is there corresponding ticket in JIRA @tsheasha ?

+1 this looks good as far as I know


Auto build Docker images on new releases

As the title says.
Currently the build & push of docker images is manual, this would add a daily check and push newer docker image if there is a newer upstream release.

Why is this closed?

I created my own build at https://hub.docker.com/r/r2dh/v2fly-core.
The source can be found at https://github.com/raidenii/v2fly-docker.

I did the same already lol.
I did a quick search and it seems https://docs.github.com/en/actions/writing-workflows/choosing-when-your-workflow-runs/events-that-trigger-workflows#repository_dispatch can be used for auto-triggering instead of periodical checking. But it can only be done by v2fly admin I think.


Better management for Xdebug

Current state
We have XDEBUG_ENABLE boolean in docker-compose.yml. Once it is enabled:

all cli PHP processes can be debugged
some frontend requests can be debugged

according to Xdebug rules, there should be XDEBUG_SESSION cookie or XDEBUG_SESSION_START query parameter in the request
usually developers use the Xdebug Helper browser extension to be able to quickly toggle the debugger with a hotkey

Current issue
While it's easy to turn on/off the Xdebug for frontend requests, all cli PHP processes go through the debugger. And it's not so fast to edit docker-compose.yml and restart the containers. As a result, Drush commands execute much slower than usual.
Proposed solution

Introduce XDEBUG_ENABLE_CLI and XDEBUG_ENABLE_FRONTEND (_FPM?)
Deprecate XDEBUG_ENABLE (or just leave it as it is?)

I think it should not be hard for me to prepare a PR if the idea sounds interesting.

Ha! I was completely wrong. There is no need to change the current setup:

XDEBUG_ENABLE sits under x-environment.&default-environment just to make it available from all containers at once
it is possible to add XDEBUG_ENABLE per service
if XDEBUG_ENABLE is set per service, it is possible to use docker-compose.override.yml like this:

version: '2.3'

services:
  php:
    environment:
      XDEBUG_ENABLE: "true"
  cli:
    environment:
      XDEBUG_ENABLE: "true"

(just checked this, works like a charm)
@Schnitzel what is the best place to put this information?
Maybe I can just update this comment? https://github.com/amazeeio/drupal-example/blob/f9ea2b521d264249f1373172c403bec881f76dd6/docker-compose.yml#L20
Or better to put it to lagoon.readthedocs.io?

So the recipe is:

Exclude docker-compose.override.yml from Git
Create docker-compose.override.yml locally. Example:

version: '2.3'

services:
  php:
    environment:
      XDEBUG_ENABLE: 'true' # the value does not matter
  cli:
    environment:
      XDEBUG_ENABLE: null # means the environment variable will not exist


How to execute shell commands inside handler?

I tried doing this:
import { command, optional, positional, run, string } from "cmd-ts"
import { execa } from "execa"

const cmd = command({
  name: "gitty",
  description: "automate git",
  version: "0.0.1",
  args: {
    query: positional({ type: optional(string), displayName: "query" }),
  },
  handler: async (args) => {
      const { stdout } = await execa("echo", ["unicorns"])
      console.log(stdout)

  },
})

run(cmd, process.argv.slice(2))

But then if you try run it, you won't get echo unicorns as output.
If you run below code outside the handler:
      const { stdout } = await execa("echo", ["unicorns"])
      console.log(stdout)

It works as expected. Not sure why that happens or how to fix.

repo where I am using cmd-ts: https://github.com/nikitavoloboev/gitty
code I want to make work is here

What’s the console output then?

It's empty.

Very suspicious. I hope I will have some time to take a look. You are more than welcome to fork and add a test.

I think i might have a similar problem using  bun shell instead of "execa"
running
const pwd= await $`pwd`.text()
console.log(pwd)

inside a "handler" will simply terminate the script and not output anything
A workaround is using a custom script.
function execShellCommand(cmd:string):Promise<string> {
  const exec = require('child_process').exec;
  return new Promise((resolve, reject) => {
   exec(cmd, (error:unknown, stdout:string, stderr:string) => {
    if (error) {
     console.warn(error);
    }
    resolve(stdout? stdout : stderr);
   });
  });
 }

const pwd=await execShellCommand(`pwd`)
console.log(pwd)


Using NodeSession with iisnode

I am interested in using NodeSession with following configuration:
iisnode version is 0.2.21,
node version is v10.7.0,
express version 4.16.3
My code is as follows:
// express
var express = require('express');
var app = express();

// path
var path = require('path');

// session management
var NodeSession = require('node-session');
session = new NodeSession({secret: 'mysecretkeygoeshere'});
session.startSession(req, res, callback);

I am not sure where I should put the line 'session.startSession(req, res, callback)', it throws error showing "'req' is undefined".

Please refer to this example https://github.com/quorrajs/NodeSession/blob/master/examples/express.js to use this library with express js.
Hope it helps.


cd! needs to fail fast if the directory doesn't exist

Otherwise it's easy to typo and end up in a bad path without realizing it, only to have things blow up later

Fixed https://github.com/lihaoyi/Ammonite/pull/138


Getting "Application not running/connected" message at AppCode v3.3

Hi guys,
I can get Injection working at Xcode 7, with a Swift project and I'm excited to try the plugin at AppCode v3.3, too. So far I couldn't get this working and here's what I'm doing:

I created a dummy main.m file and when I tell Injection plugin to patch the project, this file gets modified as expected. Xcode injection works fine.
When I switch to AppCode, using the menu commands under Run, I revert the patching changes and then re-patch the project for AppCode. Here's what my main.m looks like after:

#ifdef DEBUG
#define INJECTION_PORT 31444 // AppCode
static char _inMainFilePath[] = __FILE__;
static const char *_inIPAddresses[] = {"<IP_ADDRESS>", 0};

#define INJECTION_ENABLED
#import "/Users/aslisabanci/Library/Application Support/Developer/Shared/Xcode/Plug-ins/InjectionPlugin.xcplugin/Contents/Resources//BundleInjection.h"
#endif

I choose Debug, the project starts in the simulator, I make my changes in the code and select Inject Source from the Run menu, to inject the new code. Then I get an error saying "Application not running/connected". Can you tell me how I can fix this?
Btw, thanks a lot for sharing this plugin with the community and keep up the good work! :]

Hi, you seem to be doing all thie right things and you should be getting a message such as the following in the console as the app tries to connect:
2016-01-03 17:23:17.430 zapp[75117:4436375] +[BundleInjection load] <IP_ADDRESS> (see project's main.(m|mm)
2016-01-03 17:23:17.436 zapp[75117:4436421] Injection attempting connection to: <IP_ADDRESS>:31444
2016-01-03 17:23:17.439 zapp[75117:4436421] Connected to "Injection" plugin, ready to load x86_64 code.
is DEBUG #defined in your project build?

Hi @johnno1962, thanks for the reply.
Yes, under Apple LLVM 7.0 - Preprocessing settings, under Preprocessor Macros -> Debug, I see DEBUG = 1 and I'm running the project in Debug scheme so this definition must be in action. Also, I guess Xcode requires this DEBUG definition too right? If so, this shouldn't be the problem because I can run the Injection plugin with Xcode.

Are you getting the message I mentioned in the console though? If so it should be connecting, if not the #ifdef DEBUG is most likely the problem. Try removing it.

I wasn't getting that message but now, after removing the #ifdef DEBUG line and adding main.m file to the project's files (I didn't need to add main.m to the project with Xcode btw, it was enough that it existed on the project's folder), I could get past the connection issue and now I can see the console log:
+[BundleInjection load] <IP_ADDRESS> (see project's main.(m|mm)
Injection attempting connection to: <IP_ADDRESS>:31444
Connected to "Injection" plugin, ready to load x86_64 code.

Now for the next step I have another problem though. When I change a piece of code and try to inject this, I get an error saying that:
Injection: Use of uninitialized value $logDir in -d at /Users/aslisabanci/Library/Application Support/Developer/Shared/Xcode/Plug-ins/InjectionPlugin.xcplugin/Contents/Resources/injectSource.pl line 30.
Injection:
Injection: *** Could not locate compile command for /Users/aslisabanci/Desktop/X/X/Views/Intro/IntroViewController.swift
Injection: If you have switched xcode versions, please cmd-shift-k to clean then rebuild the project so there is a complete build history logged and try again.
Injection: iOSInjectionProject/x86_64/learnt_commands.gz ***
Injection:
Injection:  at /Users/aslisabanci/Library/Application Support/Developer/Shared/Xcode/Plug-ins/InjectionPlugin.xcplugin/Contents/Resources/common.pm line 55.
Injection: 	main::error('Could not locate compile command for /Users/aslisabanci/Deskt...') called at /Users/aslisabanci/Library/Application Support/Developer/Shared/Xcode/Plug-ins/InjectionPlugin.xcplugin/Contents/Resources/injectSource.pl line 236

Btw, I installed the plugin by downloading the source code and building the InjectionPlugin.xcodeproj myself.
My DerivedData build location is Unique.
I can compile my generated InjectionBundle.xcodeproj just fine.
Here is what my BundleContents.m file looks like:
/*
    Generated for Injection of class implementations
*/

#define INJECTION_NOIMPL
#define INJECTION_BUNDLE InjectionBundle3

#define INJECTION_ENABLED
#import "/tmp/injectionforxcode/BundleInjection.h"

#undef _instatic
#define _instatic extern

#undef _inglobal
#define _inglobal extern

#undef _inval
#define _inval( _val... ) /* = _val */

#import "BundleContents.h"

extern
#if __cplusplus
"C" {
#endif
    int injectionHook(void);
#if __cplusplus
};
#endif

@interface InjectionBundle3 : NSObject
@end
@implementation InjectionBundle3

+ (void)load {
    Class bundleInjection = NSClassFromString(@"BundleInjection");
    [bundleInjection autoLoadedNotify:0 hook:(void *)injectionHook];
}

@end

int injectionHook() {
    NSLog( @"injectionHook():" );
    [InjectionBundle3 load];
    return YES;
}

I couldn't trace what I'm missing to get AppCode find this "compile command". And many thanks for your support so far!

Not sure what the problem is now. it’s effectively grepping for the filename in iOSInjectionProject/x86_64/learnt_commands.gz which is a modified log of an xcodebuild on your project. Try removing the iOSInjectionProject and injecting again. If this log doesn’t contain the swift file you are trying to inject it doesn’t know how to compile the source.

Ok, I removed IOSInjectionProject, re-injected some code and here's the console log:
Injection: Use of uninitialized value $logDir in -d at /Users/aslisabanci/Library/Application Support/Developer/Shared/Xcode/Plug-ins/InjectionPlugin.xcplugin/Contents/Resources/injectSource.pl line 30.
Injection: Copying iOSBundleTemplate into project.
Injection: Learning compilations for files in project: xcodebuild  -configuration Debug -arch x86_64 -sdk iphonesimulator
Injection: Use of uninitialized value $file in substitution (s///) at /Users/aslisabanci/Library/Application Support/Developer/Shared/Xcode/Plug-ins/InjectionPlugin.xcplugin/Contents/Resources/injectSource.pl line 169, <GEN3> line 439.
Injection: Use of uninitialized value $file in substitution (s///) at /Users/aslisabanci/Library/Application Support/Developer/Shared/Xcode/Plug-ins/InjectionPlugin.xcplugin/Contents/Resources/injectSource.pl line 169, <GEN3> line 446.
Injection: Use of uninitialized value $file in substitution (s///) at /Users/aslisabanci/Library/Application Support/Developer/Shared/Xcode/Plug-ins/InjectionPlugin.xcplugin/Contents/Resources/injectSource.pl line 169, <GEN3> line 453.
Injection: Use of uninitialized value $file in substitution (s///) at /Users/aslisabanci/Library/Application Support/Developer/Shared/Xcode/Plug-ins/InjectionPlugin.xcplugin/Contents/Resources/injectSource.pl line 169, <GEN3> line 460.
Injection:
Injection: *** Could not locate compile command for /Users/aslisabanci/Desktop/X/X/Views/Intro/IntroViewController.swift
Injection: If you have switched xcode versions, please cmd-shift-k to clean then rebuild the project so there is a complete build history logged and try again.
Injection: iOSInjectionProject/x86_64/learnt_commands.gz ***
Injection:
Injection:  at /Users/aslisabanci/Library/Application Support/Developer/Shared/Xcode/Plug-ins/InjectionPlugin.xcplugin/Contents/Resources/common.pm line 55.
Injection: 	main::error('Could not locate compile command for /Users/aslisabanci/Deskt...') called at /Users/aslisabanci/Library/Application Support/Developer/Shared/Xcode/Plug-ins/InjectionPlugin.xcplugin/Contents/Resources/injectSource.pl line 236

I checked learnt_commands file, and /Users/aslisabanci/Desktop/X/X/Views/Intro/IntroViewController.swift is contained in that, so I couldn't get something from this. Do you have any ideas about what else I can check?

The line involved is this from injectSource.pl
            while ( my $line = <LOG> ) {
                if ( index( $line, $filename ) != -1 && index( $line, " $arch" ) != -1 &&
                    $line =~ m!@{[$xcodeApp||""]}/Contents/Developer/Toolchains/XcodeDefault\.xctoolchain.+?@{[
                            $isSwift ? " -primary-file ": " -c "
                        ]}("$selectedFile"|\Q$escaped\E)! ) {
                    $learnt .= ($learnt?';;':'').$line;
                    last FOUND;
                }
            }

We need to work out why that is not matching from iOSInjectionProject/x86_64/learnt_commands.gz. You could send me this file to injectionforxcode at johnholdsworth.com and I could atke a look

I emailed the file to you @johnno1962, just checking to make sure you got it and if you like any other input from my side I'd happily help.

Hi I got your email thanks. Did you not get my replies? I was asking if you could send me the output of:
xcodebuild clean; xcodebuild  -configuration Debug -arch x86_64 -sdk iphonesimulator >/tmp/a.txt
as the file you sent is missing commands to build certain classes.

Thanks for the info on this problem. I’ve taken a different tack for using injection for AppCode now that xcodebuild has changed with Xcode 7.2. It now creates a symbolic link to the logs directory for AppCode to process them in the same way as Xcode. Do you want to try the new version? You’ll need to inject a particular project with Xcode before you can inject with AppCode though it should be more reliable.

My pleasure :) Yes of course, I'd be happy to try the new version and let you know how it goes.

Hey @johnno1962, wanted to let you know that the new version is working fine with AppCode and I can inject new code. Thanks for your efforts! :beers:


Update README.md

Certain typo changes.

Hope its better now... i wish i was productive here, though quite meagerly.

Definitely! Every bit counts. Thanks a lot for your contribution :smiley:

indeed ... Thanks a lot for giving me the opportunity 😸 :


Core-selector didn't display selected item properly after some operations on "selected" array.

I set "valueattr" to some attribute and there are six elements (a, b, c, d, e, f) in the core-selector.
At beginning, selected is [ 'e', 'c' ] and core-selector displayed properly.
But after following operations, c element is not selected.
1 remove e,c
2 add a, b, c, d, e, f
I traced the code and found polymer will generate two splices for core-selector:
1 [ index:0, addedCount:4 ]
2 [ index:5 addedCount:1,  removed: ['c'] ]
and core-selector will set c from true to true and to false.
Maybe we need to check whether there are any repeated elements in the splices before directly set element to true or false.

Closing this issue due to age and the release of version 1 of Polymer - please feel free to re-open if this is incorrect.


Update scalafix-core to 0.9.25

Updates ch.epfl.scala:scalafix-core from 0.9.24 to 0.9.25.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "ch.epfl.scala", artifactId = "scalafix-core" } ]

labels: library-update, semver-patch

[error] java.lang.NoSuchMethodError: scala.tools.nsc.Global.reporter()Lscala/tools/nsc/reporters/Reporter;
[error] 	at scoverage.ScoverageInstrumentationComponent$$anon$1.run(plugin.scala:119)
[error] 	at scala.tools.nsc.Global$Run.compileUnitsInternal(Global.scala:1511)

https://github.com/scoverage/sbt-scoverage/issues/321
https://github.com/scoverage/sbt-scoverage/issues/319


HARP-7373: Remove unnecessary groupState parameter from TextElementSt…

…ate methods.
Signed-off-by: Andres Mandado<EMAIL_ADDRESS>Thank you for contributing to harp.gl!
Before requesting a pull request, please remember to check the following documents:

contribution guidelines
coding style

If you are adding new functionality we would highly appreciate if you can describe what is the capability you are adding and even better if you can add some examples. Please also remember to add tests for it.
CI Check
Our bots will check whether your PR can be directly integrated into the mainline. We have some internal integration tests running on the background, our bots will inform you of the next steps and someone from our team will take a look and help if needed!
And please do not forget to sign-off your commit! You can read more about DCO here. But, in short, you just need to use git commit -s or append --signoff when you are committing to the repo.
Happy contributing!

Codecov Report

Merging #1118 into master will not change coverage.
The diff coverage is 0%.

@@           Coverage Diff           @@
##           master    #1118   +/-   ##
=======================================
  Coverage   46.33%   46.33%
=======================================
  Files         254      254
  Lines       22480    22480
  Branches     5022     5022
=======================================
  Hits        10417    10417
  Misses      12063    12063

Impacted Files
Coverage Δ

...here/harp-mapview/lib/text/TextElementsRenderer.ts
9.84% <0%> (ø)
:arrow_up:

...ere/harp-mapview/lib/text/TextElementGroupState.ts
8.33% <0%> (ø)
:arrow_up:

@here/harp-mapview/lib/text/TextElementState.ts
4.87% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


add Monadless library

the build seems to fail with an unrelated error

The https-url doesn't have a valid certificate. Did you mean to use plain http?

@ochrons fixed

Can you also squash the commits into one ;)

@ochrons done! :)

@ochrons Thanks for merging the PR! I think I've missed something, it's returning this error:
ScalaFiddle.scala:-5: ERROR: Library io.monadless %%% monadless-core % 0.0.11 is not allowed

https://scalafiddle.io/sf/7YCMKer/1


Fixing Count Method

Issue Description
In previous versions, when a built query was chained with a count method, any previous references to the same query would be modified as well. This is a fix for that issue.
For example:
> query = Lockstep::Invoice.where(invoice_type_code: 'AR Invoice') # => #<Lockstep::Query:0x00000001381073c0 @criteria={:conditions=>[{:invoice_type_code=>"AR Invoice"}]}, @klass=Lockstep::Invoice>
> count_query = query.count # => 4
> p query # => #<Lockstep::Query:0x00000001381073c0 @criteria={:conditions=>[{:invoice_type_code=>"AR Invoice"}], :count=>true}, @klass=Lockstep::Invoice>

The original query object got updated which is a side-effect and shouldn't have updated.
What's Changed
The addition of count is done using the with_clone helper causing the original query to not be referenced while being executed.
The above example repeated after this change:
> query = Lockstep::Invoice.where(invoice_type_code: 'AR Invoice') # => #<Lockstep::Query:0x00000001381073c0 @criteria={:conditions=>[{:invoice_type_code=>"AR Invoice"}]}, @klass=Lockstep::Invoice>
> count_query = query.count # => 4
> p query # => #<Lockstep::Query:0x00000001381073c0 @criteria={:conditions=>[{:invoice_type_code=>"AR Invoice"}]}, @klass=Lockstep::Invoice>

Local Sanity
Before:

After:

Changes look good.


Update authentication cache to work for clusters

Fixes #5532
This update will allow the meta clients to update their authentication caches (password changes or dropped users) when a new metadata snapshot is received.
Some things I removed:

Duplicated test code (dropping privileges twice)
"pluggable" hashing function type that always used bcrypt.CompareHashAndPassword() for validation
unused authCache on the store that was never accessed

I also added a test file that will actually reduce the bcrypt work cost when the package is being tested (the comment that was there lied about already doing it)

:+1: Looks like needs a rebase though.

LGTM :+1:


Name potential processing folder according to plugin_package (fix #43)

If processing provider and algorithm is included rename their folder after the plugin_package option, for example "myqgisplugin_processing". This is to avoid confusion and errors with the built-in PyQGIS 'processing' package.

The provider.py still tries to import processing_algorithm.py from a directory called processing. Should also be updated to myqgisplugin_processing.

The remove_processing_files() in post_gen_project.py also is removing an incorrect directory and should be updated

@nmaarnio I've prepared a fix for the post_gen_project.py but I'm not sure what you mean with the processing_algorithm.py.
from .processing_algorithm import ProcessingAlgorithm

I think the "processing" folder/package is not being referred to at all in the provider, since this is an relative import (the processing_algorithm.py being in the same folder). I haven't modified the name of the algorithm module.

I noticed this while I was checking PR #41 and was in the corresponding branch. Now I switched to main and it seems the issue in the provider.py is not present there. I think there might have been some merging mistake in modernize-template since the processing directory's name is updated there, but provider.py.j2 exists still there and does not match the new directory name.

Ah okay, I see that now. I'm going to add a suggestion to #41 then.


Avoid using AddComponentParameter in design time

Fixes AB#1869483.
I think using AddComponentParameter over AddAttribute has no benefit in design time (the motivation for introduction of the method were runtime errors, see https://github.com/dotnet/razor/issues/8258) so avoiding it unconditionally in design time should be the safest fix.

has no benefit in design time

Looking at the difference in codegen, and the complete lack of source mapping or validation of user code, I'm not sure either has a benefit in design time, but probably best not to rock the boat too much 😁

@333fred @dotnet/razor-compiler for a second review, thanks


Fix schedule on leadership summit page

Currently the community track reads
12:00 - 1:00 am: Registration
1:15 - 2:00 am: Keynote Address
2:05 - 2:55: Interactive Session
should read
12:00pm - 1:00pm: Registration
1:15pm - 2:00pm: Keynote Address
2:05pm - 2:55pm: Interactive Session

I would like to fix this.


Fix error of the description of 'SensorAdapterFinishCapture' in sensor-adapter-funciton.md

The description of 'SensorAdapterFinishCapture' isn't correct, which is duplicated with 'SensorAdapterGetIndicatorStatus', should be corrected.

@gangdong : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Thanks for your contribution, I've merged your changes and they should be live soon.


Search for Nx Generate (UI) menu

Currently, I have a very long list of generators (schematics), it would be easier if we can search from the list

I think something similar to the angular generator popup would be more appropriate

Yes we can implemented like the angular generator.
But if you are using keyboards, you can have similar experience.
cmd+n > Nx Generate Ui > start typing, the lis will be filtered

or
you can use cmd+shif+a > Nx Generaate Ui >  start typing, the lis will be filtered

Let me know if this satisfy your needs.

Yes we can implemented like the angular generator.
But if you are using keyboards, you can have similar experience.
cmd+n > Nx Generate Ui > start typing, the lis will be filtered

or
you can use cmd+shif+a > Nx Generaate Ui >  start typing, the lis will be filtered

Let me know if this satisfy your needs.

I think that's fine, thank you 👍

I think that's fine, thank you 👍

Thank you for commenting other issues.

Thank you for commenting other issues.


[Feature Request] make it for forge

i want it for forge server pls can u make it
thanks

I need too

asking a fabric dev to port to forge is like asking a blacksmith to use a stone hammer instead of steel because "all the other tribes use stone hammers" "i cant use steel hammers" "the other blacksmiths say stone hammers are better"

please close this issue, it's pointless

Close this issue and use Server Tab Info, which is for forge

Close this issue and use Server Tab Info, which is for forge

Ok. LGTM


Cuda OOM (1k+ frame video) on A100

Hi @hitachinsk. Thanks for the great work.
Is there any recommendations you or anyone else can give on how to solve a Cuda OOM issue using FGT? I tried experimenting with allocation size as well as cleared my cache but nothing works. I see the batch size is already low.
I'm on an A100 which should be plenty.
This line of code:
filled_frames = FGT_model(masked_frames, selected_flows, selected_masks)

Also if I have over a thousand frames, what is the recomended sequence length if my objects are moving slowly across those frames?

You can reduce the number of the sampled frames in each training iteration. But high resolution video inpainting is always a tough problem, maybe it's better to train on a lower resolution and enlarge the resolution gradually.

Thanks @hitachinsk . I have a followup question:
What is your recommendation for fps for test videos? I am unable to find info on train/val/test info on the fps of the videos.

We adopt 25fps videos.


[Feature Request] Force Email Verify

it will be  cool if you can make Force Email Verify
when client sign up even i enabled " Send email confirm " at my whmcs Settings
Client still can place order and get The Product With fake email
even worse there clients use a fake email on my site and when pay with paypal use another one . and i will not able to give paypal a proof that was same pesron
it will be nice if you can make Hook that not let user Place orders only When He Verify His Account First!
Thanks For all this great work!

Yeah this would be an amazing hook. It is weird whmcs doesn't already do this and this would save alot of those hassles.


Fails if TELEGRAM_STAT is not set

For me, using docker, the container fails if the TELEGRAM_STAT variable isn't set.
Maybe because the healthcheck uses it but it's not available?

Can you send us the logs please?

Hi @MarcoBuster, which logs?
When the container starts, the status is starting for a long time, then it says unhealthy. I can't connect to the API and simply get a "404 page not found".
The container logs only have a single line with the command for starting the API server, nothing more.

@noplanman set TELEGRAM_VERBOSITY=4 then the logs will appear. Without them we can't understand the problem.

@MarcoBuster Isn't the health check like a curl request to the stats port?
It simply says "curl -f http://localhost:8082/ || exit 1"
In the Dockerfile
Of cause that'll fail if there's no :8082 port serving the stats.
The locks would be the wrong place to look here anyway, and I think it's not needed; that's some simple logic error.
As we now have the ping command, we should use that one instead of the stats endpoint.

@luckydonald *logs
(Sorry am on mobile)

@luckydonald the ping command requires a valid token

@luckydonald Wait, we can't use a ping command as that is bound to a bot so we would need to add a bot token?

@luckydonald So to fix it properly, we should either so if the ping port without a token (unauthorized) as well.
Maybe for now, if the stats are not explicitly enabled they could be instead be enabled but bound to just the local network interface needed to get the curl request done?

@luckydonald Theoretically ping doesn't require a token, we could move the ping command out of there, but it is more confusing with the URL layout then.

...or we could just disable the health check if you don't have TELEGRAM_STAT set

@MarcoBuster is that even possible?

@giuseppeM99 I don't know, we should investigate

@giuseppeM99 I don't know, we should investigate

@MarcoBuster i'll fix it asap

@MarcoBuster i'll fix it asap

Okey, just to give my two cents, maybe we should actually look into creating a real /healtcheck endpoint.
So if it does a ping or not, would be a implementation detail.
Maybe that could even be served at the root directly, doing a few non-critical stats there.

Okey, just to give my two cents, maybe we should actually look into creating a real /healtcheck endpoint.
So if it does a ping or not, would be a implementation detail.
Maybe that could even be served at the root directly, doing a few non-critical stats there.


⚠ Spryr has degraded performance

In 9887ad4, Spryr (https://spryr.com) experienced degraded performance:

HTTP code: 200
Response time: 7117 ms

Resolved: Spryr performance has improved in 122ff50 after 15 minutes.


Scripts simple delay

Extended simple.sh (atop @safchain 's bridge related commit) so enable:
./simple.sh start                               # invocation using default subnets
./simple.sh start --delay                       # injection of netem delays
./simple.sh ping                                # ping: vm1-->vm2 and vm2-->vm1

I notice the following failing - but was not sure why:
12:26:31.626 Gremlin: g.Context(1520598366543).V().Has("Type", "bridge", "Name", "br-test").Out("Name", "intf1")
12:26:32.667 Gremlin: g.Context(1520598366543).V().Has("Type", "bridge", "Name", "br-test").Out("Name", "intf1")
12:26:33.716 Gremlin: g.Context(1520598366543).V().Has("Type", "bridge", "Name", "br-test").Out("Name", "intf1")
12:26:34.756 Gremlin: g.Context(1520598366543).V().Has("Type", "bridge", "Name", "br-test").Out("Name", "intf1")
12:26:35.797 Gremlin: g.Context(1520598366543).V().Has("Type", "bridge", "Name", "br-test").Out("Name", "intf1")
12:26:36.845 Gremlin: g.Context(1520598366543).V().Has("Type", "bridge", "Name", "br-test").Out("Name", "intf1")
12:26:37.248 2018-03-09T13:26:37.183+0100	DEBUG	flow/table.go:231 (*Table).update	dpdk2: flow table update:<PHONE_NUMBER>183,<PHONE_NUMBER>183
12:26:37.282 2018-03-09T13:26:37.218+0100	DEBUG	flow/table.go:231 (*Table).update	dpdk2: flow table update:<PHONE_NUMBER>217,<PHONE_NUMBER>217
12:26:37.892 Gremlin: G.Context(1520598366543)
12:26:38.431 Gremlin: G.Context(1520598366543).V().Flows().Sort()
12:26:38.581 --- FAIL: TestBridge (31.99s)

simple.sh is used by some functional tests, please check if the changes don't break them

I made a point of not breaking the existing API, will check.
On Fri, Mar 9, 2018, 19:23 Sylvain Afchain<EMAIL_ADDRESS>wrote:

simple.sh is used by some functional tests, please check if the changes
don't break them
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/skydive-project/skydive/pull/846#issuecomment-371881300,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACogI_WeIo5aBOazsvS7oYcatoC-aczSks5tcrqFgaJpZM4SkLn_
.

I see no correlation between changes to scripts.sh and the failure:
12:26:31.626 Gremlin: g.Context(1520598366543).V().Has("Type", "bridge", "Name", "br-test").Out("Name", "intf1")
12:26:32.667 Gremlin: g.Context(1520598366543).V().Has("Type", "bridge", "Name", "br-test").Out("Name", "intf1")
12:26:33.716 Gremlin: g.Context(1520598366543).V().Has("Type", "bridge", "Name", "br-test").Out("Name", "intf1")
12:26:34.756 Gremlin: g.Context(1520598366543).V().Has("Type", "bridge", "Name", "br-test").Out("Name", "intf1")
12:26:35.797 Gremlin: g.Context(1520598366543).V().Has("Type", "bridge", "Name", "br-test").Out("Name", "intf1")
12:26:36.845 Gremlin: g.Context(1520598366543).V().Has("Type", "bridge", "Name", "br-test").Out("Name", "intf1")
12:26:37.248 2018-03-09T13:26:37.183+0100	DEBUG	flow/table.go:231 (*Table).update	dpdk2: flow table update:<PHONE_NUMBER>183,<PHONE_NUMBER>183
12:26:37.282 2018-03-09T13:26:37.218+0100	DEBUG	flow/table.go:231 (*Table).update	dpdk2: flow table update:<PHONE_NUMBER>217,<PHONE_NUMBER>217
12:26:37.892 Gremlin: G.Context(1520598366543)
12:26:38.431 Gremlin: G.Context(1520598366543).V().Flows().Sort()
12:26:38.581 --- FAIL: TestBridge (31.99s)

Moreoever - I see no direct relation between simple.sh and the tests scripts undr scripts/ci

rerun tests

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@8846ea4). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #846   +/-   ##
=========================================
  Coverage          ?   33.81%
=========================================
  Files             ?       65
  Lines             ?     8993
  Branches          ?        0
=========================================
  Hits              ?     3041
  Misses            ?     5629
  Partials          ?      323

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 8846ea4...b8e177d. Read the comment docs.

@safchain - any additional comments?

I would make the bridge stuff optional

@safchain - ok bridge is optional now using -b or --bridge

@lebauce - can you rerun tests?

rerun tests

rerun tests

rerun tests

rerun tests

rerun tests

rerun tests

rerun selenium-tests

@lebauce @safchain - I notice we are still only running one test - should I open a new PR?

@hunchback I don't think it's necessary to open a new PR..
We have CI issues right now, no job are scheduled

please rebase this PR

rerun tests

rerun tests


Need clarification on dependencies (OL and Cesium)

From the documentation it is not clear in how far OL3-Cesium depends on certain OpenLayers and/or Cesium versions.
The release archive ships with certain OL and Cesium versions, but is this really how OL3-Cesium is meant to be used in larger applications? Can it be combined with other versions of OL and Cesium, or does it depend on some internal details of these specific shipped versions? Are these shipped versions original ones, or do they contain some additional patches?
It would be nice to have some documentation (e.g. NPM style package.json) which states exactly which versions of OL and Cesium should be used in combination with OL3-Cesium v1.0.0.

Recently, Ol3 and Cesium are both releasing a stable version every month. Ol3-Cesium is going to adopt the same release frequency and be tested using the latest stable Ol3 and Cesium versions.
I am going to release a v1.1 of ol3-cesium this week, based on the stables OL 3.1.1 and Cesium 1.5 releases and add a note on github.
Of course, other versions may work, but it is your responsibility to test. There are currently no dependencies on internal details.
Does it answer your concerns?

Thanks for the clarification!


[Request] Update for Foundry v10

An update for Foundry v10 would be amazing.

Done in version 1.2.5.


Extra move reported

This position:
position startpos moves a2a3 d7d5 c2c4 d5c4 d1a4
which leaves black to play gives d8d2 as a valid move. However, black is in check from the queen on a4, and d8d2 doesn't evade it. After a bit of digging the problem is the else here in Board::Init():
	Checks = 0;
	if (BitBoards[KW] & UnsafeForWhite)
		Checks |= CHECK_WHITE;
	else if (BitBoards[KB] & UnsafeForBlack)
		Checks |= CHECK_BLACK;

Although black's move is illegal, it's one that puts the white king immediately in check, so the first condition above is met but not the second. And just like that, black isn't in check as far as the game is concerned :)
Removing the else from the above code fixes the perft tests (found at startpos, depth 6 initially) and stops a few oddities.

issue fixed in commit 887b50


Idea: python exporters and importers should be able to consume protobufs generated by go sysl

GO SYSL's grammar is not the same as that of python sysl.
To make existing python utilities work with new go sysl, one idea can be to change the python sysl loader module to invoke the go sysl binary's pb command, load the protobufs and return to python utilities.
Swagger exporter could benefit from this approach.

Closing this as it seems no longer relevant to the current Go implementation of Sysl


little to no usage documentation

Hi Nick :)
I've been looking at this for some time and trying to decipher your examples.

installation: do you just rename "bin" to "pystream" and copy it to the "site-packages" directory?
or is it like Richie Hindle's goto module where it goes in your src's directory?

initialization: what is the proper import statement to call the initialization of PyStream's module's?
(referring to the main program, your examples cover the shader half)

Shaders: how do you properly send a python shader to GL?

from the examples, PyStream just parses a shader...
where's the programming that actually uses the shaders?
I really love this idea, very much! ;)
and from what I've read, it's extremely fast and efficient. :D
but you know, what good is something if you can't figure out how to use it...

Hi,
Also came here to ask for examples !
S


Recommended value for member-ordering rule is pointless

The recommended_ruleset.js includes a "member-ordering": true rule which in that form doesn't actually do anything.
See the documentation.

Actually, over 10 of the rules give me linting errors because they expect arrays when the value in the recommended is "true" or "false"

@tommck I think that's due to the schema not allowing either an array or a boolean at the same time. It's perfectly valid to disable an option with just a false even if enabling it would require an array of parameters.

Do any of you have a preferred recommended default value?
@tommck which other rules have invalid default parameters? I am not linting the file, so whether or not you get linting errors depends I guess on what your linting tool and linting rules are.

If anyone wants to make a proposed default value then just edit line 143 of this file: https://github.com/Microsoft/tslint-microsoft-contrib/blob/master/additional_rule_metadata.json
I'll accept the pull request that automatically is sent when you click save :)

I set the default to [true, { "order": "fields-first" }]
Feel free to change this!


[New game]: Quantum Maze

🎮 Game Request
Game logic and basic description
Quantum Maze" that involves navigating through a maze with quantum mechanics principles, including superposition and entanglement.
Point down the features
Game points
Quantum Walls: Walls that can exist in multiple states simultaneously. Players must collapse these walls by observing them to determine if they are passable.
Entangled Paths: Certain paths are entangled, meaning if you choose one path, the corresponding entangled path changes as well.
Superposition Gates: Gates that require the player to be in two places at once, achieved by splitting the player's state.
Select program in which you are contributing
GSSoC24
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @Aditi22Bansal , You can't have another issue before completing the previous one 😀
you already have the following 1 open issues 👀 ! :
#3977

Hey @Aditi22Bansal ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇

Hey @Aditi22Bansal !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹


Associate block with metadata and fix transaction encoding

Updates the BlockPayload trait.

Add a Metadata type and a constructor to build the payload and metadata.
Add transaction encoding and decoding functions.

Updates the BlockHeader constructor to take metadata as an input.
Implements the new BlockPayload functions for the VIDBlockPayload struct.
Fixes transaction flattening to include the length of each transaction.

Closes https://github.com/EspressoSystems/HotShot/issues/2024.

All the task implementations (for consensus, transactions, etc.) are still using VIDBlockPayload which used to be the only payload struct. I should have replaced all them in the production code with the app-specified payload type.
Will update this. Thanks for the comments! @jbearer @ggutoski

I think it looks good, except see this thread about possibly removing the commit() method of BlockPayload and having HotShot keep track of the commit on its own. WDYT about that @shenkeyao @ggutoski

@jbearer As mentioned in that thread, I think it makes sense to remove the Commitable bound. I've made an issue and will do that after merging this PR.


UI code editor

A TextInput with support for syntax highlighting.
It is currently behind a "code_editor" feature flag. We might decide to have it enabled by default (as part of "shards") or only when building an app such as claymore.
Uses the Syntech crate and the current implementation is mainly based on the egui_demo_lib from the egui repository for the layouting part.

Part of fragcolor-xyz/management#31

@guusw god point. I have a hardcoded .desired_rows(10) which was copied from the egui sample. I will try play with it (eventually expose it) to see how it impacts the layout.
We might need to have the code editor inside a scroll area. Either the user would need to do it manually (using (UI.ScrollArea) or it could also be built-in the code editor shard.


Add note about cropping support to assets.mdx

What kind of changes does this PR include?

New or updated content

Description

The experimental assets api doesn't yet support cropping images, which might confuse users when moving from the @astro/image integration.
This PR adds an aside to the assets page in the properties section

I am not sure whether to put the aside in the properties section or in the migration section.

Thanks so much @eyrewiut for this contribution! I might not get to it today, but this will absolutely be on my list to get started on next week, and I do see your question about which section to place this in, so I'll give that some consideration too!  :raised_hands:


Validate billable email when updating account

Update AccountsController#update to validate billable email
Add email_validator to Gemfile
Rename AccountsController spec file
Don't use strong_parameters in AccountsController because it doesn't not return a JSON-encoded response with nicely formatted error messages, per this issue: https://github.com/rails/strong_parameters/issues/157

LGTM :+1:


[MetaSchedule] Change anchor trace apply to block-mapping based approach

The old anchor trace apply assumes that the prefix of the "new" outputs matches with the "old" outputs, which is not the case for some tasks in complex models. Also the old approach use block name to match corresponding blocks among the old module in database and the new module to apply trace on.  In some cases, the block names may differ even though the block body are identical. In this PR, I maintain a block mapping between the old IRmodule and the new IRmodule, so that we can find the matching parts in both IRmodule with no assumptions of the relationship of their locations in IRModule.
cc: @tqchen @junrushao @masahi

I have to admit I don't fully understand this...Would you like to elaborate a bit? Thanks!

I'll can take a look tomorrow. Please fix the CI issues in the mean time.

The CI seems weird. I cannot reproduce the error locally. The error mainly results from the string difference of v_ax2 + v_ax1 * T.int64(16) vs v_ax1 * T.int64(16) + v_ax2. I don't know why it will generate different results on different machine.

To elaborate this PR, this PR solves 2 cases that the old trace apply doesn't handle:

struturally equal blocks with different block names are not recognized as matched blocks
trace apply assumes that the prefix of the "new" outputs matches with the "old" outputs for get_child_blocks inst. But it's not true for some cases. Maybe the prefix of the old outputs match the new outputs or maybe the mapping is not for an interval of blocks.


To add LCP_ECHOINTERVAL/LCP_MAXECHOFAILS configuration items (IDFGH-3098)

Hi @projectgus
Base on the discussion on this thread:
https://www.esp32.com/viewtopic.php?f=13&t=13756
Please add below configuration items to PPP settings via menuconfig
LCP_ECHOINTERVAL
LCP_MAXECHOFAILS
Thanks,
Axel

@AxelLin
Thanks for posting this feature request, we will look into it, but this may take some time, since we'd like to test at least on general level
Meantime, might I suggest enabling this options simply on project level?
The below code in your project CMake file should apply the define to the lwip component
idf_component_get_property(lwip lwip COMPONENT_LIB)

target_compile_definitions(${lwip} PRIVATE "-DLCP_ECHOINTERVAL=XXX")

Meantime, might I suggest enabling this options simply on project level?
The below code in your project CMake file should apply the define to the lwip component
idf_component_get_property(lwip lwip COMPONENT_LIB)

target_compile_definitions(${lwip} PRIVATE "-DLCP_ECHOINTERVAL=XXX")

Hi @david-cermak
Above change works.
Thanks.

@AxelLin Glad that the workaround is functioning, but we can leave this request open until we actually have this feature in ESP-IDF.

This workaround does not work for me:

Makefile:

Is this functionality already available in esp-idf 4.3/release ?


fix(plugin-webpack): fix incorrect PRELOAD_WEBPACK_ENTRY.

BrowserWindow preload parameter should be an absolute path. Remove prepending 'file://' from prod
value of PRELOAD_WEBPACK_ENTRY.
Properly escape backslashes in the dev value of PRELOAD_WEBPACK_ENTRY to prevent failure on Windows.

[ x] I have read the contribution documentation for this project.
[x ] I agree to follow the code of conduct that this project follows, as appropriate.
[x ] The changes are appropriately documented (if applicable).
[ x] The changes have sufficient test coverage (if applicable).
[- ] The testsuite passes successfully on my local machine (if applicable).
These tests are failing, but are unrelated:

1) electron-forge API (with installer=yarn)
       after init
         after package
           make
             throws an error when the specified maker doesn't support the current platform:
     AssertionError: expected promise to be rejected with an error matching /the maker declared that it cannot run/ but got 'Could not find module with name: /Users/tom/code/electron-forge/packages/api/core/test/fixture/maker-unsupported'

  2) electron-forge API (with installer=yarn)
       after init
         after package
           make
             throws an error when the specified maker doesn't implement isSupportedOnCurrentPlatform():
     AssertionError: expected promise to be rejected with an error matching /incompatible with this version/ but got 'Could not find module with name: /Users/tom/code/electron-forge/packages/api/core/test/fixture/maker-incompatible'

  3) rebuilder
       no config
         "before all" hook:
     Error: Exited with status 1

  4) rebuilder
       with config
         "before all" hook:
     Error: Exited with status 1

Sure, I'll try

A unit test, please.

@malept, a test added

Sorry, rebased and squashed, because I used my company email address by mistake in the original commit

@MarshallOfSound, any chance to merge this soon, please, as this bug is blocking the use of preload scripts on Windows?


Autoplay not working

Autoplay doesn't work, meaning that after the last song in the queue has finished, it doesn't play any suggested songs, and doesn't give any errors.
I worked out that one of the problems was from when it defines the soup object in the voice class, as I hadn't pip installed lxml.
However, It is now seeming to get stuck at:
recommended_urls = []
                        for li in soup.find_all('li', class_='related-list-item'):
                            a = li.find('a')

I think this is due to soup.find_all('li', class_=related-list-item'): returning None

@nose-gnome @Lenart12 did you find any solution for the autoplay.

Didn't know this bot was in use still, I will take a look

Thank if you find any solution please let me know😊😊

So i worked on the autoplay command i got it working but i want to make a few changes but i am confused how to do so😅.
song_url = self.current.source.url
                        # Get the page
                        html = urllib.request.urlopen(song_url)
                        video_ids = re.findall(r"watch\?v=(\S{11})", html.read().decode())
                        ol = []
                        length = len(video_ids)
                        for i in range(length):
                            song = f'https://www.youtube.com/watch?v={video_ids[i]}'
                            if song == song_url:
                                pass
                            else:
                                ol.append(song)
                                print(song)

                        print(ol)
                        # Parse all the recommended videos out of the response and store them in a list
                        ctx = self._ctx
                        # Chose the next song so that it wasnt played recently
                        next_song = ol[0]

so i want to make it so that the song with most views is chosen and not just random but i am confused how to do so could you please help me?

Ok so i got the autoplay working as i wanted but now after playing one song it autoplays another song then stops after that not playing anymore songs even if i use -play command it only queues so my guess is that the bit gets stuck somewhere in that loop so is there anyway i can make it continuosly play songs automatically.
    async def audio_player_task(self):
        while True:
            self.next.clear()
            self.now = None

            if self.loop == False:
                # If autoplay is turned on wait 3 seconds for a new song.
                # If no song is found find a new one,
                # else if autoplay is turned off try to get the
                # next song within 3 minutes.
                # If no song will be added to the queue in time,
                # the player will disconnect due to performance
                # reasons.
                if self.autoplay and self.current:
                    try:
                        async with timeout(3):
                            self.current = await self.songs.get()
                    except asyncio.TimeoutError:
                        # Spoof user agent to show whole page.
                        song_url = self.current.source.url
                        # Get the page
                        html = urllib.request.urlopen(song_url)
                        video_ids = re.findall(r"watch\?v=(\S{11})", html.read().decode())
                        ol = []
                        ov = []
                        length = len(video_ids)
                        for i in range(length):
                            song = f'https://www.youtube.com/watch?v={video_ids[i]}'
                            if song == song_url:
                                pass
                            else:
                                if song not in ol:
                                    ol.append(song)
                        ol = ol[:10]
                        for j in ol:
                            vid = pafy.new(j)
                            val = vid.viewcount
                            ov.append(int(val))
                        max_value = max(ov)
                        max_index = ov.index(max_value)

                        # Parse all the recommended videos out of the response and store them in a list
                        ctx = self._ctx
                        # Chose the next song so that it wasnt played recently
                        next_song = ol[max_index]

                        async with ctx.typing():
                            try:
                                source = await ytdl.YTDLSource.create_source(ctx, next_song, loop=self.bot.loop)
                            except ytdl.YTDLError as e:
                                await ctx.send('An error occurred while processing this request: {}'.format(str(e)))
                                self.bot.loop.create_task(self.stop())
                                self.exists = False
                                return
                            else:
                                song = Song(source)
                                self.current = song
                                self.voice.play(self.current.source, after=self.play_next_song())
                                await ctx.send('Autoplaying {}'.format(str(source)))

                else:
                    try:
                        async with timeout(180):  # 3 minutes
                            self.current = await self.songs.get()
                    except asyncio.TimeoutError:
                        self.bot.loop.create_task(self.stop())
                        self.exists = False
                        return

                await asyncio.sleep(1)
                await self.current.source.channel.send("Playing next song...")
                self.song_history.insert(0, self.current)
                self.current.source.volume = self._volume
                self.voice.play(self.current.source, after=self.play_next_song)
                await self.current.source.channel.send(embed=self.current.create_embed())

            # If the song is looped
            elif self.loop == True:
                self.song_history.insert(0, self.current)
                self.now = discord.FFmpegPCMAudio(self.current.source.stream_url, **ytdl.YTDLSource.FFMPEG_OPTIONS)
                self.voice.play(self.now, after=self.play_next_song)

            await self.next.wait()

here is my code


Site files are not being installed.

Site files are not being installed. Help me please :/
https://i.hizliresim.com/vbMGam.png
https://i.hizliresim.com/1LzbpY.png
https://i.hizliresim.com/Ly9Mva.png

You have to point your webserver to the public directory.

how will I do?

Hey bro?


[FEATURE_REQUEST] Externally Hosted Config Files and API keys

Is your feature request related to a problem? If so, please describe.
I currently host my conf files externally on my self hosted Gitea and would like to be able to add Widgets to my pages. The issue is that I don't want to have my API keys exposed in plane text.
Describe the solution you'd like
I'm still pretty new to hosting services and coding so I'm not entirely sure how to implement this. possibly having a secrets file hosted on the web server itself and being able to refer to them in the externally hosted conf files
Priority
Medium (Would be very useful)
Is this something you would be keen to implement
No

Linking up to #799


Changed the way tooltips work, for better usability

I felt like there where a couple of issues with the way the Tooltip functionality was implemented:

The tooltip is drawn immediately after the control and, in a tight panel of controls, this space can be later occupied by another control, thus hiding the tooltip
Every time the tooltps string were added with GuiSetTooltip(my_tooltip_string) before a control they had to be reset with another call to GuiSetTooltip(NULL) with NULL as argument.

This change aims at fixing both.
Point 1:
Now during a frame the guiTooltipDrawPtr and guiTooltipDrawRec are modified in case a tooltip should activate and at the end of the frame (or whenever the user wants) the tooltip can be drawn by calling GuiTooltip(), now taking no arguments and being exposed to the users.
Point 2:
The guiTooltipPtr is reset at the end of the controls that allow it. This eliminates te verbosity of calling GuiSetTooltip(NULL) after every control.
I tried to be consistent with the naming and the comments, let me know if anything else should change. This will be a breaking change for tooltips, there are no examples using them so maybe we could add one as well. Let me know. I will definitely use my implementation in my projects.

@AndreaBoroni I'm afraid this redesign is a breaking change and it would break all the codebases using previous implementation, including all my tools. I'm afraid I'm not merging it.


fix(speakers): fixed some spelling mistakes

What does this PR do?
How should it be tested manually?

Tire un PR para no molestar da pipol yo'


v0.7.1 development

For maintainers: auto detect evals
Little fix to the GUI of /net ip

For maintainers: auto detect evals
Little fix to the GUI of /net ip


unknown extension 'vscode.git-base' dependency

I'm getting:

Cannot activate the 'GitHub Repositories' extension because it depends on unknown extension 'vscode.git-base'

when running Visual Studio Code for Web in my codespace. The codespace runs Debian Bullseye and is pretty generic with only these added:
"features": {
	"git": "os-provided",
	"github-cli": "latest",
	"azure-cli": "latest",
	"python": "os-provided"
}

at the same time I'm unable to find the missing Git Base extension

This is a duplicate of https://github.com/microsoft/vscode-remote-repositories-github/issues/223 and will be fixed in next week's release.


Added new theme

Fishfinger is a theme inspired in Colombian flag colors.

Awesome job Jinme! Love the theme!


Test key for Enrollment 100 no longer working...

The test key that is published for the 100 enrollment which is for testing purposes no longer seems to function.

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

ID: 3e71361e-a9bb-89ee-27ee-7e38a60914a0
Version Independent ID: df5a9b26-4eb1-a396-1c84-9f3ec6232753
Content: Connect to Azure Consumption Insights data in Power BI Desktop - Power BI
Content Source: powerbi-docs/connect-data/desktop-connect-azure-consumption-insights.md
Service: powerbi
Sub-service: powerbi-desktop
GitHub Login: @davidiseminger
Microsoft Alias: davidi

Hi, @ryanferguson -- thanks for asking! Looking at the article, it appears the "100" is an example, not a number for testing purposes. The text after the screen shot is:
"You'll need to replace the enrollmentNumber value with your enrollment number. You can get your number from the Azure Enterprise Portal. The numberOfMonth parameter is how many months of data you want going back, from the current dat. Use zero (0) for the current month."
Adding the author in case he has other comments, and perhaps to clarify in the article.
Thanks!


Support for megav2 and megav3 with MEGACMD

New mega.nz schemes megav2:// and megav3:// can be supported using the megacmd utility. I've released a working image at flowgunso/volumerize-mega.
IMHO the two most viables options to provide the megacmd utility:

Rely on danielquinn/megacmd-alpine to provide the compiled binairies and libraries, then install the dependencies. Which is what I've done. But it adds a dependency to the latter.
Compile the megacmd utility. But it will add significant build time to the CI.

What do you think ?
In the meantime, I've added a Mega tutorial.

I would prefer the compile version.

MegaCMD is compiled in a separate stage to reduce the image footprint. Based on @danielquinn's MEGAcmd for Alpine Linux.

Build error message is curl: (22) The requested URL returned error: 503. I've tried the three relevant curls and they're fine now.


[1.10.2] Induction Matrix doesn't report input to OpenComputer

I'm using OpenComputer to have a graphical monitoring of my induction matrix.
Induction Matrix shows in the computer component, all methods return correct values (getOutput(), getEnergy(), getMaxEnergy(), ...) except getInput() wich return always 0.
The only input is a solar panel ( Environmental Tech), with enhanced energy cable (EnderIO).
The Energy goes down when it is the night (obviously solar panel), and goes up in the day.
So everything is good, except this getInput()
Mekanism 9.2.1
Forge <IP_ADDRESS>5
OpenComputer <IP_ADDRESS>
MC 1.10.2

This is related to another issue that I'm currently in the process of finding a fix for.