Adds font metrics to support font commands.

This is part 1 of 3.  Rendering, screenshots, MathML, and unit tests will
follow in susbequent pull requests.

How shall we deal with this commit here in relation to #303? On the one hand, this commit here is the older one.
On the other hand, reviewing this here might be a lot easier if #303 got merged and this one here rebased onto that one. And there are other benefits as well. With @1f8610cebe2c99223db4082a834fb814afd2b697 you could suggest make PYTHON=/usr/local/Cellar/python/2.7.9/bin/python in the README, without danger of a changed shebang getting committed accidentially. You could also drop the install instructions for fontforge.
Perhaps we can simply disregard these issues about python version and fontforge installation here, merge things as they stand knowing that it's a temporary solution, then rebase #303 onto this and add a commit to that which cleans those things up. If this request here is about ready to be merged, then that solution would be preferable, since I'm really waiting for this font stuff to hit master so I can gain some traction with my subsequent restructuring plans.

I'll rebase this on to https://github.com/Khan/KaTeX/pull/303.

@xymostech this is ready for review.

(Sorry for taking so long!) This all looks good now! Thanks for the docstring about rsfs, that's very strange!

Hopefully it's not just my install of TeX.


Point eclipse-pass.org/ to use the JHU AWS Pass Deployment

IP Address: <IP_ADDRESS>

@aforward Please let me know if there are any additional pieces of information needed.

@cshannon2 here are the three URLs we have

URL
Notes
Status

eclipse-pass.org
Github Pages HTML / CSS
running-locally (DNS not configured)

demo.eclipse-pass.org
GitHub Self-Hosted Runner / Docker Compose
running

nightly.eclipse-pass.org
GitHub Self-Hosted Runner / Docker Compose
running-locally (DNS not configured)

(documented https://github.com/eclipse-pass/main/blob/main/docs/infra/eclipseops.md)
So I don't think we should change the eclipse-pass.org record, let me know if we are replacing the current ones (and if so, then the current GH action runners will break unless updated), or if we want new URLS (e.g.  aws-demo.eclipse-pass.org and aws-nightly.eclipse-pass.org, or some other prefix like jhu-)

Here are the three options to choose from.

Adding AWS URLs (leaving EF Azure's untouched and adding new AWS / JHU urls)
Changing URLs (so moving EF Azure's URL to something like ef-demo/ef-nightly, or based on the architecture so azure-demo/azure-nightly)
Replaced with AWS URLs (so we would nuke the Azure EF servers in addition to changing the DNS records

We're shooting for option 3 - replacing with AWS and shutting down the EF servers. As you note, the runners will need to be updated. Note that we'll also be retiring the "nightly" subdomain and using "dev" instead, though the dev environment doesn't exist just yet.

Ticket has been created (only I have access, but linking here so I can easily find it)
https://gitlab.eclipse.org/eclipsefdn/infrazilla/-/issues/1143
We would like the following changes

Name
Type
Value
Notes

dev.eclipse-pass.org
A
<IP_ADDRESS>
New record

demo.eclipse-pass.org
A
<IP_ADDRESS>
Update to existing record

Both will be routed to AWS infrastructure (so yes the IPs are the same).  Once the new infrastructure is up and running I will separately request that our Azure "EC2" instances be terminated (but I'll hold off until it is confirmed that the AWS is setup and working).

13:08 ~/sin/projects/arrowmight (f/one-server-3-apps)$ ping demo.eclipse-pass.org
PING demo.eclipse-pass.org (<IP_ADDRESS>): 56 data bytes
Request timeout for icmp_seq 0
^C
--- demo.eclipse-pass.org ping statistics ---
2 packets transmitted, 0 packets received, 100.0% packet loss
14:09 ~/sin/projects/arrowmight (f/one-server-3-apps)$ ping dev.eclipse-pass.org
PING dev.eclipse-pass.org (<IP_ADDRESS>): 56 data bytes

@cshannon2 the IPs are set!!!

@aforward thanks for the update. We were able to confirm that the demo url correctly points to our AWS infrastructure. Closing this ticket.


misc add-on

convenient copying of example's runtime into projects working directory
added new examples to the examples list
changed runtime shader gles output to essl to match lastest bgfx
added spirv support for upcoming vulkan support. Please note that vulkan is still not working and still work in progress.
prefer GLVND over LEGACY OpenGL if available. Please read https://cmake.org/cmake/help/v3.10/module/FindOpenGL.html
for more details.

woops. don't merge this yet. it doesn't look like the shaders of 37-gpudrivenrendering compile anymore. Most likely an issue on bgfx side.
cd /home/mchiasson/git/build-bgfx.cmake-Debug && /home/mchiasson/git/build-bgfx.cmake-Debug/shaderc -f /home/mchiasson/git/bgfx.cmake/bgfx/examples/37-gpudrivenrendering/fs_gdr_instanced_indirect_rendering.sc -i /home/mchiasson/git/bgfx.cmake/bgfx/src -o /home/mchiasson/git/bgfx.cmake/bgfx/examples/runtime/shaders/gles/fs_gdr_instanced_indirect_rendering.bin --platform android --type fragment && /home/mchiasson/git/build-bgfx.cmake-Debug/shaderc -f /home/mchiasson/git/bgfx.cmake/bgfx/examples/37-gpudrivenrendering/fs_gdr_instanced_indirect_rendering.sc -i /home/mchiasson/git/bgfx.cmake/bgfx/src -o /home/mchiasson/git/bgfx.cmake/bgfx/examples/runtime/shaders/glsl/fs_gdr_instanced_indirect_rendering.bin --platform linux --type fragment --profile 120
Code:
---
    354: void main()
    355: {
    356: vec4 color = u_color[uint(v_materialID)];
    357: if (color.w < 1.0f)
    358: {
    359: if ( (int(gl_FragCoord.x) % 2) == (int(gl_FragCoord.y) % 2) )
    360: {
    361: discard;
    362: }
    363: }

>>> 364: gl_FragColor = vec4(color.xyz, 1.0);
>>>   1: ^

    365: }
---
Error: (364,1): error: `gl_FragColor' undeclared
(364,1): error: value of type vec4 cannot be assigned to variable of type error

Failed to build shader.

For now, I disabled 37-gpudrivenrendering since it can't generate properly for essl (however, it generates fine in glsl and spirv). I'll come back later to uncomment this once shaderc is fixed on bgfx, and that your submodules are updated.

Thanks for these changes. I'll look over them more thoroughly after the new year.

@JoshuaBrookover sorry, it looks like it's breaking on vs2017, creating a cyclic dependency for some reason. I'll close this PR and create a bunch of smaller ones instead. Hopefully this will help me pin point the issue.


Update DO ssh key documentation

This PR updates the DO example documentation for the SSH key configuration. Currently the code requires that the DO SSH key name be the same as the cluster name but this is not documented.
The following error results when a SSH key is not known by DO that matches the name of the cluster.
Error during apply of atomic reconciler, attempting clawback: strconv.Atoi: parsing "": invalid syntax

I believe this is a bug and I'll try to see can I fix it around.

@clintkitson Thank you very much for reporting the issue. I created a PR to fix this behavior, #308. This should do the job.


Chatters List API deprecation

It looks like the V1 of the Chatters List API is being deprecated on December 12, 2018.   It appears they are removing a couple endpoints and changing the Chat User model to remove some fields.
Deprecation reference: https://dev.mixer.com/content/blog/chatlistdeprecations
New API reference: https://dev.mixer.com/reference/chat/chatchatterslist

Because Mixer isn't planning on doing a clean cut over for their versions, we're gonna need to straddle the version fence for a while. Current plan is to do a nested class structure & re-purpose existing methods when possible.
So for example, we'll have a private ChatsV2Service class that will live inside the ChatsService class. This class will implement the V2 methods & the regular ChatsService class will simply act as a pass-through for any V2 methods. This will allow us to gradually migrate new methods over & attempt to minimize as much change for users of the API as possible.
Currently we're blocked on finishing this as the API doesn't appear to be working; all my calls are coming back with 404's. So once we have that resolved, we'll commit the change & roll out an update.


Durations

Use this issue to discuss the Durations pattern in the ONS Design System

Link to the durations component on the patterns page shouldn't be there as we don't document the component in the components area, this is all covered in the pattern.

When durations are configured to only render one field, and that field has an error, the field wraps the error panel rather than the field being within the error panel.


Set CFG_RELEASE_CHANNEL and CFG_RELEASE env vars when building rustfmt

After updating rustc-ap-* crates to 659.0.0, it is now required to set both CFG_RELEASE_CHANNEL and CFG_RELEASE environment variables.
Unfortunately, cargo does not seem to provide a way to pass environment variables down to dependencies; we need to define it before running cargo.
We need to write a build script or add a build tool like cargo-make, which sets both environment variables to nightly when they are undefined.
We need this check before cargo check, cargo build, cargo test and cargo publish.

I'm not opposed to leveraging additional scripts or tooling, but I wonder how well those would play with editor/IDE extensions and the impact that may have on contributors (for example I believe Rust Analyzer uses cargo check extensively and we'd have to advise users on how to configure things when working on rustfmt).
I wonder if we could leverage the existing cargo settings file we have (though not immediately sure which config key(s) it would be)
https://doc.rust-lang.org/cargo/reference/config.html#configuration-format

I can't get master to build even with the env variables set (mac, rustc 1.45.0-nightly (a74d1862d 2020-05-14) and cargo 1.45.0-nightly (cb06cb269 2020-05-08)):
export CFG_RELEASE_CHANNEL=nightly
export CFG_RELEASE=nightly
cargo build
# fails to build rustc_span-659

Am I missing something?

@ayazhafiz You are probably not using the latest nightly compiler. It is recommended to use the latest nightly when working on rustfmt.

You're right. Thanks.


Story: [CCLS 1943] [CCLS 1937] [CCLS 2302] Upload & Edit Notification Attachments

Base will be updated to main when https://github.com/ministryofjustice/laa-ccms-caab/pull/252 is merged.


Stuck at installing soroban-cli v0.6.0 (Ubuntu - Windows WSL2)

What version are you using?
v0.6.0

What did you do?
Installing soroban-cli using this command on Ubuntu-Windows WSL2

What did you expect to see?
Installation Completed
What did you see instead?

Stuck at here,wasm-opt-sys(build) for more than 1 hour,i didnt experienced this when installing v0.4.0 version

Yeah that crate uses c++. We should make a feature flag for it so that you could install without it.

Yeah that crate uses c++. We should make a feature flag for it so that you could install without it.

Do you have any quick solution for it? so i could install it

Som options:

Install it via windows and not wsl

Clone the repo and comment out the optimize command.

Look up the wasm-opt project and see if there is anything about building on WSL.

Here is the repo
https://github.com/brson/wasm-opt-rs
So it could be a cmake or other C++ issue.

This issue is old. The current ways to install Stellar CLI is documented in README.


[AutoRelease] t2-networkfunction-2022-11-18-74549(can only be merged by SDK owner)

https://github.com/Azure/sdk-release-request/issues/3427
Live test success
https://dev.azure.com/azure-sdk/internal/_build?definitionId=5019

issue link:https://github.com/Azure/sdk-release-request/issues/3427

/check-enforcer override


The waste heat for ICT is 3x higher than the Berenschot paper

The Berenschot paper (https://www.berenschot.nl/actueel/2018/november/410-kton-co2-besparing/ ) states a waste heat of 6 PJ and 100% use of waste heat in the ETM of ICT is 18 PJ. The difference is due to the fact that in the ETM ICT is more than datacenters
https://github.com/quintel/documentation/blob/master/general/residual_heat_industry.md

Good point Mart!

The difference is due to the fact that in the ETM ICT is more than datacenters

You're right.
Currently residual heat potential is calculated based on the electricity demand of the ICT sector and the share of energy that can be recovered as residual heat from datacenters (77% of electricity demand of datacenters). However, we should indeed take into account the share of datacenters in the ICT sector!
There are two ways to fix this in my opinion:

We could split the ICT sector in 'datacenters' and the 'rest' in the graph.
We could use the calculated residual heat potential that Berenschot calculated in 2018: 6.3 PJ as a basis. This is a technical and economical potential; they took into account the proximity to heat demand and the gas price. We could use this to calculated the potential residual heat share from ICT in the graph.

Note: there's a big difference in the electricity demand for central ICT in NL2015 (from RVO) which is 23.5 PJ and the electricity demand of Informatie en communicatie (SBI J) on Klimaatmonitor which is 12.4 PJ in 2015.

Note: there's a big difference in the electricity demand for central ICT in NL2015 (from RVO) which is 23.5 PJ and the electricity demand of Informatie en communicatie (SBI J) on Klimaatmonitor which is 12.4 PJ in 2015.

It looks like the data used for the 2015 dataset is old, RVO now also reports a demand close to 12.4 PJ:

@michieldenhaan and I just looked into this issue and concluded that it's more a data issue than a potential residual heat share issue. The percentage we are using now (77%) is correct for electricity demand of datacenters.
What should happen is:

[ ] Update electricity demand ICT for NL2015 23.5 PJ -> 13.0 PJ. This will have huge scenario impacts, so should be carefully done!
[ ] Electricity demand ICT should be split into electricity demand for data centers and 'other'. This share should be calculated based on a good data source (that we currently do not have) and the share should be dataset-dependent. Somehow the electricity demand of datacenters is not correctly reported: many datacenters have been built, but the electricity demand of ICT is not increasing the last couple of years (see Klimaatmonitor and RVO data in previous posts)
[ ] A disclaimer should be added to the data center residual heat slider that user should use the slider carefully. We have to do this before this deploy: I will add it to the text changes on BC.

I will remove the deploy milestone from this issue and add the NL2017 update milestone


I get the error "HttpError: Not Found"

Hi,
first of all thanks for building this project. We've been using it successfully for a while on a project.
On two other projects, I do get this error:
(node:1478) UnhandledPromiseRejectionWarning: HttpError: Not Found
    at /home/runner/work/_actions/daneden/enable-automerge-action/v1/dist/index.js:2845:26
    at processTicksAndRejections (internal/process/task_queues.js:93:5)
    at async main (/home/runner/work/_actions/daneden/enable-automerge-action/v1/dist/index.js:4591:31)
(node:1478) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 1)
(node:1478) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

The configuration is exactly the same as for the other project where it does work. The configuration looks like this:
name: Enable automerge on dependabot PRs

on:
  pull_request_target:

jobs:
  automerge:
    name: 🚀  Enable automerge on dependabot PRs
    runs-on: ubuntu-latest
    steps:
      - name: 🚀  Enable automerge on dependabot PRs
        uses: daneden/enable-automerge-action@v1
        with:
          github-token: ${{ secrets.DEPENDABOT_AUTO_MERGE_PERSONAL_ACCESS_TOKEN }}

The Action itself says that it successfully completed, but the above error is outputted in the CLI output.
Any idea why this would work on some projects and not on others?
Best Regards
Wouter

Okay, I figured out that the user I created the personal access token with has no access to the repository.


chore: [Snyk] Upgrade pnpm from 8.11.0 to 8.12.0

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade pnpm from 8.11.0 to 8.12.0.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 1 version ahead of your current version.
The recommended version was released 22 days ago, on 2023-12-08.

Release notes

Package name: pnpm

8.12.0 - 2023-12-08Minor Changes

Add support for basic authorization header #7371.

Patch Changes

Fix a bug where pnpm incorrectly passes a flag to a run handler as a fallback command #7244.
When dedupe-direct-deps is set to true, commands of dependencies should be deduplicated #7359.

Our Gold Sponsors

Our Silver Sponsors

8.11.0 - 2023-11-24 Read more

    from pnpm GitHub release notes

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (38bcbb6) 77.83% compared to head (ca83a60) 77.83%.
 Consider uploading reports for the commit 02244e1 to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2000   +/-   ##
=======================================
  Coverage   77.83%   77.83%
=======================================
  Files          40       40
  Lines        3086     3086
  Branches      625      625
=======================================
  Hits         2402     2402
  Misses        479      479
  Partials      205      205

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


Warning about i18n locale files

Versions

Windows 10
Angular CLI: 6.0.0-rc.1
Node: 8.11.1
OS: win32 x64
Angular: 6.0.0-rc.2
... animations, common, compiler, compiler-cli, core, forms
... http, platform-browser, platform-browser-dynamic, router

@angular/cdk: 6.0.0-rc.1
@angular/cli: 6.0.0-rc.1
@angular/flex-layout: 5.0.0-beta.14
@angular/material-moment-adapter: 6.0.0-rc.1
@angular/material: 6.0.0-rc.1
@angular-devkit/architect: 0.0.10
@angular-devkit/build-angular: 0.0.10
@angular-devkit/build-optimizer: 0.4.9
@angular-devkit/core: 0.4.9
@angular-devkit/schematics: 0.5.1
@ngtools/json-schema: 1.1.0
@ngtools/webpack: 6.0.0-beta.9
@schematics/angular: 0.5.1
@schematics/update: 0.5.1
typescript: 2.7.2
webpack: 4.1.0

Repro steps

Step 1
Setup internationalization:

import { registerLocaleData } from '@angular/common';

import localePTBR from '@angular/common/locales/pt';
registerLocaleData(localePTBR);

Observed behavior
Emits a warning:
WARNING in<EMAIL_ADDRESS>10:24-31 Critical dependency: require function is used in a way in which dependencies cannot be statically extracted

Desired behavior

Not emit this warning
Mention any other details that might be useful (optional)
I'm not really sure if this warning affects the app. It happens in dev mode. I couldn't manage to build a prod compilation yet (my first try after the upgrade was unsuccessful)

I have the same warning in RC.2
And one more issue:
For PROD mode If locale set in angular.json as "i18nLocale": "en-US" (not via registerLocaleData function), then an error occurs in @angular/common module in registerLocaleData function: "Cannot read property 'toLowerCase' of undefined", because localeId is undefined.

This is a webpack warning which is triggered when webpack can't statically analyse uses of e.g. require.ensure which can be caused by passing variables into require() or require.ensure() or dynamically constructing the URLs/module paths you pass into the functions.
Probably by dynamically loading things based on a given locale?
See https://github.com/webpack/webpack/issues/2675#issuecomment-269417932

This was an issue with Angular itself in https://github.com/angular/angular/issues/23217, and has since been fixed there.

it should be fixed in rc4 once it's released


win32: native cygwin/msys2 tty support, and other minor improvements

All the commits in this PR are independent of the others, and I can split some of them to a new PR if you prefer. But they're all quite minor so I thought I'd start with a single PR.

Huh, enforced formatting. I'll handle it.

I've fixed the minor formatting issues, but now I don't see any travis indication (here at the PR page) of either success or failure. Not sure what's going on... but the formatting is hopefully fine now.

The last commit/force-push only updates the commit-message of win32: detect and use normal ansi on msys2/cygwin tty without touching any code.

Have you checked #1099?

Have you checked #1099?

I did just now. It seems to do pretty much exactly the same thing[*] as my win32: detect and use normal ansi on msys2/cygwin tty commit, but in much more lines of code and files.
Their files also technically support XP while jumping through some hoops (which also require manual configuration of defines), but my code only "supports" XP by allowing to disable this detection on XP. Do note that XP support was dropped in msys2 and cygwin in 2016.
[*] There are very minor procedural differences I noticed:

When checking if we need to search stdin, my code aborts earlier if stdin is a tty, while their code leaves fstat and the other tests first, but it should behave identically as far as I can tell.
Their pipe name matching/detection is a bit more elaborate and in quite more LOC. I wouldn't say one is more robust or better, as it's heuristics either way. I preferred a simpler matching, which is also more similar to what git-for-windows uses.

In general, it's pretty much the same as #1099, but in about 1/10 LOC of that PR.

Also, I think #1099 has a potential buffer overflow here

Ah, you are right. I have updated my code. This is also used in Vim and some other software. So I sent patches for them.

Last force push fixes a type at the commit build: workaround when fopencookie exists but zlib or lzma do not. Thanks @k-takata for noticing it.


Added awesome meetup

Added 'Awesome Meetup' list: https://github.com/SerCeMan/awesome-meetup

It seems like that list is abandoned


overhaul local dev / package scripts / documentation

Pre-Flight checklist

[ ] Testing

[ ] Changes have been manually tested

What are the relevant tickets?
Closes https://github.com/mitodl/ocw-hugo-themes/issues/87
What's this PR do?
Over the course of working on https://github.com/mitodl/ocw-studio/issues/568, we've altered the way the theme sources and renders metadata and content to match the way ocw-studio publishes sites.  This PR makes some further adjustments to the theme to make it easier to build courses locally that have been published by ocw-studio.  The use of Hugo modules has also been removed and replaced with the new strategy of running hugo from the content directory, pointing to the relevant config from ocw-hugo-projects with --config and pointing to ocw-hugo-themes with --themesDir.  When running sites locally, concurrently (https://github.com/open-cli-tools/concurrently) is now being used to manage killing off both the Hugo and Webpack processes when you hit ctrl + c to stop the local dev server.
How should this be manually tested?

Read through the new readme thoroughly and make sure it makes sense: https://github.com/mitodl/ocw-hugo-themes/tree/cg/overhaul-local-dev
Following the instructions in the new readme:

Spin up ocw-www locally
Spin up a course site locally

Verify that concurrently is working properly, which should automatically close both the Webpack dev server and the Hugo server when you press ctrl + c in your terminal.  After closing the dev servers, you should be able to run ps aux | grep hugo and not see any lingering processes.

Any background context you want to provide?
References to ocw-to-hugo have been removed as we are nearing production readiness with ocw-studio.  Since ocw-to-hugo converted courses are now being imported into ocw-studio and published in the same way the rest of the ocw-studio courses are, spinning up courses directly from ocw-to-hugo is mostly irrelevant.  Spinning up courses directly generated by ocw-to-hugo can still be done, but will need to be done manually.

@alicewriteswrongs I added some functionality that will by default clone the necessary repos you need for spinning up www or course.  Read the readme changes and try it out and let me know what you think.


Can't click on any simple-bar element? (/enter settings)

When I try clicking on the elements on the bar nothing happens. I already enabled the ubersicht preference to make the enable interaction. i can't access the settings. Is there another way to add the weather widget to simple-bar?

I already tried COMMAND and , but it only opens the finder preferences.

Fixed by opening settings.js and using that


DO NOT REVIEW: Stable kafka branch

This change is

test-me-please

Last run succeeded, running again

test-me-please

test-me-please

Hit another issue #4357 Trying again

test-me-please

test-me-please

test-me-please

test-me-please

Not sure why CI is not being triggered. trying again

test-me-please

test-me-please

Hit kafka issue:
https://jenkins.cilium.io/job/Cilium-PR-Ginkgo-Tests-Validated/4072/testReport/junit/k8s-1/7/K8sValidatedKafkaPolicyTest_Kafka_Policy_Tests_KafkaPolicies/

Yup, the fix is not going to work unfortunately :(

@tgraf : I still feel this change is a good to have, even if it does not fix the issue. Should I put it out for review. This way we will not depend on latest kafka docker image and stick to a more stable version, avoiding any failures if something changes in the latest kafka image.

I still feel this change is a good to have, even if it does not fix the issue. Should I put it out for review. This way we will not depend on latest kafka docker image and stick to a more stable version, avoiding any failures if something changes in the latest kafka image.

Agreed

test-me-please

test-docs-please


Is location possible?

Is it possible to integrate the location of the car

duplicated https://github.com/leeyuentuen/polestar_api/discussions/114
is more discussion then issue


Make default notifications translatable

This uses the JSON translation-strings feature to make it easier
to localize notifications and the password reset mail.

I also added a note to the docs.
https://github.com/laravel/docs/pull/4239

AFAIK Lumen does not use password reminded and thus it doesn't use this class. If a lumen user wants to use this class for password notifications they need to pull in notifications at least.

I suggest using toMailUsing() method to register your own way of sending this notification.

@taylorotwell do you have a hint on how to actually use that method? It seems I can't figure it out. ;(


Build Error

Describe the bug
Getting Build Error in BlazorHero.CleanArchitecture.Application when building solution.
To Reproduce
Steps to reproduce the behavior:
Download the ZIP File and run update-data PM command.
Expected behavior
To Build
Screenshots

Additional context
Version 3.4

Hi @djjohnson-net,
I had the same issue while building, the path naming issue is strictly on Windows OS users, to Solve this you could rename the complaining files, I renamed them to something I can relate with.

Rename the file and you should be able to build successfully.


feat(Tree): virtualized tree prototype

closing in favor of https://github.com/stardust-ui/react/pull/1677


Java bucket followup PR to #2209

Java bucket followup PR to https://github.com/lukesampson/scoop/pull/2209

@r15ch13 We forgot to remove oraclejdk.ps1. Should I create a PR?


3.0.0-m2 - integer instead of floating point is returned

Hi,
demodb, query:
SELECT
count() as OrdersNumber,
sum(Amount) AS TotalRevenuesFromOrders,
min(Amount) as MinAmount,
(sum(Amount)/count()) as AverageAmount,
max(Amount) as MaxAmount
FROM Orders
AverageAmount should be floating point (an integer result is returned instead)
Thanks

Hi @santo-it
Fixed on develop branch
Closing, thanks!
Luigi

Thanks Luigi!
I've added a test as part of the demodb integration tests.


Incorrect validation accuracy

Currently working on a fashion dataset where I am predicting the clothing categories (e.g shoes, jeans, jackets etc.) My training/validation split is 2500/500, equally distributed over all the classes.
Accuracy after 20 epochs:
Train accuracy:
Final: train_loss: 0.1132 - train_acc: 0.9623
My validation accuracy is extremely low and doesn't change throughout training:
Final: val_loss: 12.7359 - val_acc: 0.0492
Validation accuracy is roughly 1/n where n is my number of classes.
When I manually evaluate my validation set, my results confirm the train accuracy to be mostly correct.
So my question is, has anyone experience with the validation accuracy being calculated incorrectly? Or does someone have an idea what may cause this.

Issue was using two different ways of preprocessing for my imageGenerators.


GC: itemAttachment Support

Item attachments hold items that can be any Outlookable object provided by M365. The specific supported list for Outlook Mail can be found here.
Can be closed when itemAttachments of the the following types are supported

[x] https://github.com/alcionai/corso/issues/2355 Events
[ ] https://github.com/alcionai/corso/issues/2372 Mail
[ ] https://github.com/alcionai/corso/issues/2426 Contacts
[ ] ContactGroup
[ ] PostItem
[ ] Task

Until this ticket is closed, the solution provided by #2359 will not allow for non-supported objects to be restored.  All  itemAttachment objects are stored correctly at this time.

[ ] https://github.com/alcionai/corso/issues/2428
Reconnect after https://github.com/microsoft/kiota-serialization-json-go/issues/61 is patched


fix: Add missing initialization for transaction version

Added explicit default ctors/assign for copy/move to builder
Changed  builder sign function to return void and pass transaction in by reference to avoid copies.
Updated transaction tests to check for version.

What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Refactor
[ ] Performance
[ ] Tests
[ ] Build
[ ] Documentation
[ ] Code style update
[ ] Continuous Integration
[ ] Other, please describe:

Does this PR introduce a breaking change?

[ ] Yes
[ ] No

Does this PR release a new version?

[ ] Yes
[ ] No

If yes, please describe the impact and migration path for existing applications:
The PR fulfills these requirements:

[ ] It's submitted to the develop branch, not the master branch
[ ] All tests are passing
[ ] New/updated tests are included

If adding a new feature, the PR's description includes:

[ ] A convincing reason for adding this feature (to avoid wasting your time, it's best to open a suggestion issue first and wait for approval before working on it)

Thanks for submitting this pull request! A maintainer will review this in the next few days and explicitly select labels so you know what's going on.
If no reviewer appears after a week, a reminder will be sent out.

Tests pass on Windows and Ubuntu for Windows.  I am able to get it to fail on my "real" Ubuntu machine.  Very weird.  I will investigate.

Codecov Report

Merging #124 into develop will decrease coverage by 0.1%.
The diff coverage is 82.35%.

@@             Coverage Diff             @@
##           develop     #124      +/-   ##
===========================================
- Coverage    91.52%   91.41%   -0.11%
===========================================
  Files           30       30
  Lines          861      874      +13
===========================================
+ Hits           788      799      +11
- Misses          73       75       +2

Impacted Files
Coverage Δ

src/include/cpp-crypto/transactions/transaction.h
66.66% <ø> (ø)
:arrow_up:

src/transactions/transaction.cpp
89.41% <100%> (+0.47%)
:arrow_up:

src/transactions/builder.cpp
76.81% <81.25%> (+0.44%)
: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 03288a3...24c98e5. Read the comment docs.

A contributor has approved this PR. A maintainer will merge this PR shortly. If it shouldn't be merged yet, please leave a comment saying so and we'll wait.
Thank you for your contribution!

Your pull request has been merged and marked as tier 4. It will earn you $20 USD.


Remove remaining online/offline statuses from data-only hotspots.

"online/offline" circle appears on hotspot list in wallet.
e.g. Bent Heather Condor here: https://explorer.helium.com/accounts/13CYRHKKoFUNQ441sfsAnePvJNmwJVxxz8v3r6gQRMEeXoq1F1c/hotspots
"online/offline" circle appears on hotspot list in hex.
e.g. "Bent Heather Condor` here:
https://explorer.helium.com/hotspots/hex/8829b6c133fffff

This might be tricker than it seems. Technically a data only hotspot can use the miner (instead of gateway-rs). This means it will gossip sync status and have a valid online/offline status for Explorer to display.

Bent Heather Condor doesn't have the miner installed, just gateway-rs.  I can certainly install Docker and the miner, if you need it, for any testing / troubleshooting purposes.   Thanks!

Bent Heather Condor doesn't have the miner installed, just gateway-rs. I can certainly install Docker and the miner, if you need it, for any testing / troubleshooting purposes. Thanks!

FYI, there are already 9 dataonly hotspots that are currently gossiping status:

address
name

14WVFUpWQWf1pRRLsdji7mdwFTKqhw4P9FSXYV9KnunzoFpoFqz
able-pickle-goldfish

112QWL5jauuzKcQUnN7CmW5kom2WFNP7JUxtmFxDnWqXPN7DBZWZ
agreeable-lace-sealion

14JjrGkSVgkK76oBSVomomvLg7uG16W8mTD2fqLatTvbcbnpZWH
fun-cherry-leopard

1126FyZCCFDgLWEdxJbaYxQDYjnh3AF4yCcaYrPX2iBYEoH3cz7j
orbiting-violet-perch

112oTtA4B9bULBWM2TnH84C8Y5csoQNcmmPg2DigGvJ24j5Nd6zE
old-walnut-anteater

13h38nFNqM6a2XzaR99k8CVG6S6vhiLR11zd5UEnZJLGmHhoN9b
bumpy-indigo-duck

14T6gfkT2nsVxKQ7jFH4Bj1Q5VRsBgEM36q1JcBTK8pHhG4XrVX
brisk-neon-guppy

14ahqv7JRKM9WB1Pa8b8o6kMKJZ2eEreXDJ9aV9VqsQfQTGbNVZ
cheesy-indigo-seahorse

14hKfXzq22QiaY2BPEd3WSxXteSDHAc27j1YJ5aoWqJh6As3S6a
cheesy-vermilion-anteater

I should clarify, this gateway_key has been recycled several times over, and in the past has had the docker container for the miner on it.  But, this SD Card was freshly formatted and RaspPi OS re-installed, then only gateway-rs installed, about a week ago.   Currently SSH'ed:
pi@raspberrypi:~ $ docker ps -a
-bash: docker: command not found

You mentioned "currently gossiping status"...if by current you mean literally within the past ~12 hours, then it is happening via gateway-rs.        :-)
If those gossips occurred further in the past, perhaps ~2 weeks ago, then that could have been via a miner container that was running at that time.
Thanks!


customizable scroll bar -need a way to specify css class for scroll or override the existing one

Feature Request
Need a way to incorporate custom scroll in webchat. The regular one looks dull and pulls away from the design of the app.

I think we've already decided this as a no-op: https://github.com/microsoft/BotFramework-WebChat/issues/2215. I'll leave this issue open for now to see if team members want to chime in with a different opinion.
Could you elaborate on why you're unable to make these css changes in the root of the app?

Hi @corinagum ,i have added the custom scroll class in the root app and in the component where webchat is embeded. It is not reflected in the chat window.

Embeded versions of Web Chat are not style-able, save a few options like username. Please try using a CDN or npm package version of Web Chat if you want to make stylistic changes.
To see what kind of customizations you can make, please take a look at Customizing Web Chat


Replaced levelOfAssurance parameter with confidenceLevel

+1

There are still open questions around whether we can continue to use default values (it might break GG users) and the config should be validated like before to produce errors at startup time for illegal CL values and not at runtime.
Do not merge yet!

LGTM now.


example code for Drop module + claim conditions using merkle

We have a repo for this now, will publish a new version when merkle root handling is in the sdk.


Restrict ability to apply extra credit items to an extra credit category

When creating a GB item, if an extra credit category is selected for the item, prevent the "Extra Credit" checkbox from being selectable for the item, thereby restricting both the item and the category from being extra credit at the same time.

Done. Ready for testing. Requires SAK-29883

Looks good!


Support foreign key checks

improved validate
connects https://github.com/frictionlessdata/datapackage-py/issues/150

@amercader
Please take a look. It's a straight port of https://github.com/frictionlessdata/tableschema-js/pull/88 but may be it makes sense to review this one first because there is no noise.

@amercader
Yes. On tableschema level we expect lookup table in references argument. And on datapackage level we create this lookup table - https://github.com/frictionlessdata/datapackage-py/pull/153.
Sorry I've added more comments to JavaScript PR (https://github.com/frictionlessdata/tableschema-js/pull/88) but forget to clarify where to find it.

So as a user I actually will be using it like this - https://github.com/frictionlessdata/datapackage-py/pull/153/files#diff-ed1e95b24fbf90fddd0edfbc46764fadR895 (on datapackage level)

@roll that makes much more sense, thanks


59 setup raspberry pi

SLU scripts worked fine from the RaspberryPi. The connection was set with SSH from my Mac. A tutorial on how to set it up is also provided and available on Wiki. Please review, and comment on this PR.

can you please move the readme.md into the slu-scripts and name it raspberrry-pi-configuration.md.

Readme renamed and moved in the slu-scripts directory


Deprecate Scrollbar isAlwaysShown -> thumbVisibility

Part one of several scrollbar clean up PRs and prepare for desktop scrollbars.
Original PR was #96935, but it was becoming too large
Tracking the progress overall in https://github.com/flutter/flutter/issues/96936
This PR handles the addition of thumbVisibility and deprecates isAlwaysShown.
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

The analyzer is unhappy :)

The analyzer is unhappy :)

Bah sorry, fixed now. I think.


Removed trailing namespace comments. #353

please verify...thanks!!

 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.

Hey there @DeepakReddyVelagala, while I have approved the PR, I'll need you to download the git patch file generated from our CI, and apply it to your changes. You can download the file CI has generated from here. Thank you for your contribution! :D

I applied the patch to the forked repo, is there something else I need to do...

It doesn't appear to have been pushed to your remote. Once it is on your forked repository (on the branch this PR targets) then CI should just run and pass, and then we can merge it for you. :)

can you please direct me on how to do it

Well, assuming you have already applied the patch, you should just need to commit and then push.

got it...thanks

Hey, really sorry about this, but I've just realised these commetns are coming from clang-format, not from my editor. I've made a mistake, here.
I need you to modify our .clang-format file at the root of the repository, and add the following line:
FixNamespaceComments: 'false'
I then need to send you a new git patch file once you've pushed that up. Really sorry about this. I made a mistake in diagnosing the original issue.

I'll need you to re-apply your changes to the namespace removals across all the files again after this.

@RubyNova, won't CI push a new git patch to handle this?

Ohh! do I need to fork your original repo again or should I do it with the repo I already have

No, the setting only cares about adding them and fixing invalid ones @capnkenny, so the changes will need to be re-applied, which is why I feel bad. :(
@DeepakReddyVelagala no, you can use this existing branch and PR. Thank you for your patience, sorry for the inconvenience.

Alright, moment of truth... Here is the new patch file
Can you double check it is changing what we expect before committing and pushing please? Thanks for your help.

Sure.
On Fri, 1 Oct 2021, 19:08 Matt Jones, @.***> wrote:

Alright, moment of truth... Here is the new patch file
https://github.com/novelrt/NovelRT/suites/3931179431/artifacts/98426066
Can you double check it is changing what we expect before committing and
pushing please? Thanks for your help.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/novelrt/NovelRT/pull/366#issuecomment-932237376, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AT4JI2ANSDGNUEOZVEYV6ELUEW2XBANCNFSM5FEMGM5Q
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

I removed few comments after adding FixNamespaceComments: 'false' to clang format file and applied patch and the comments which I deleted did not reappear again does this mean I can proceed with rest of the work...

Sounds like it worked! Yes you can continue, if it breaks again I will let you know, but fingers crossed it is smooth sailing from here.

Yes, seems you do, here it is, its looking promising from what I've browsed of the git patch file this time.

clang-format passed. I'm enabling auto-merge. Once CI is done it'll enter our main branch. Thank you again for your contribution! It is very much appreciated. 🎉

thanks for your guidance and support


Introduce Ems#class_for_ems

This is a factory method equivalent to ems.class::Vm and Vm.class_for_ems(ems)
It felt unnatural to call the Model.class_for_ems(ems) version and we wanted more flexibility than calling ems.class::Model

One more minor thing from me but overall looks good

@kbrock A conflict occurred during the backport of this pull request to morphy.
If this pull request is based on another pull request that has not been marked for backport, add the appropriate labels to the other pull request. Otherwise, please create a new pull request direct to the morphy branch in order to resolve this.
Conflict details:
diff --cc app/models/ext_management_system.rb
index 46a2c2036f,aa52e20723..0000000000
--- a/app/models/ext_management_system.rb
+++ b/app/models/ext_management_system.rb
@@@ -1017,6 -1028,75 +1017,78 @@@ class ExtManagementSystem < Application
      BlacklistedEvent.where(:provider_model => name, :ems_id => nil)
    end

++<<<<<<< HEAD
++=======
+   ###################################
+   # Refresh Worker
+   ###################################
+
+   def self.refresh_worker_class
+     nil
+   end
+   delegate :refresh_worker_class, :to => :class
+
+   def refresh_worker
+     return if refresh_worker_class.nil?
+
+     refresh_worker_class.find_by_ems(self).first
+   end
+
+   def start_refresh_worker
+     return if refresh_worker_class.nil?
+
+     refresh_worker_class.start_worker_for_ems(self)
+   end
+
+   def stop_refresh_worker
+     return if refresh_worker_class.nil?
+
+     _log.info("EMS [#{name}] id [#{id}]: Stopping Refresh Worker.")
+     refresh_worker_class.stop_worker_for_ems(self)
+   end
+
+   def stop_refresh_worker_queue
+     MiqQueue.put_unless_exists(
+       :class_name  => self.class.name,
+       :method_name => "stop_refresh_worker",
+       :instance_id => id,
+       :priority    => MiqQueue::HIGH_PRIORITY,
+       :zone        => my_zone,
+       :role        => "event"
+     )
+   end
+
+   def stop_refresh_worker_queue_on_change
+     if refresh_worker_class && !self.new_record? && default_endpoint.changed.include_any?("hostname", "ipaddress")
+       _log.info("EMS: [#{name}], Hostname or IP address has changed, stopping Refresh Worker.  It will be restarted by the WorkerMonitor.")
+       stop_refresh_worker_queue
+     end
+   end
+
+   def stop_refresh_worker_queue_on_credential_change
+     if refresh_worker_class && !self.new_record? && self.credentials_changed?
+       _log.info("EMS: [#{name}], Credentials have changed, stopping Refresh Worker.  It will be restarted by the WorkerMonitor.")
+       stop_refresh_worker_queue
+     end
+   end
+
+   # Takes a child class (e.g.: "NetworkRouter") and returns the EMS specific version of it
+   #
+   # @param [String|Symbol] class_name name of the child class
+   # @param [Boolean] inherit - return the base class if the child does not have a specific one (default: false - return nil if no child)
+   # @returns The EMS specific version of the class requested
+   def class_for_ems(class_name, inherit = false)
+     self.class.class_for_ems(class_name, inherit)
+   end
+
+   def self.class_for_ems(class_name, inherit = false)
+     class_name = class_name.name if class_name.kind_of?(Class)
+     const_get(class_name, inherit)
+   rescue NameError
+     nil
+   end
+
++>>>>>>> 603b40a9d0 (Merge pull request #21496 from kbrock/class_for_ems)
    # @return [Boolean] true if a datastore exists for this type of ems
    def self.datastore?
      IsoDatastore.where(:ems_id => all).exists?

https://github.com/ManageIQ/manageiq/pull/22237

Backported to morphy in commit a4a47a598dde9e64eaaf3a268e312c920d5a6c5d.
commit a4a47a598dde9e64eaaf3a268e312c920d5a6c5d
Author: Jason Frey<EMAIL_ADDRESS>Date:   Thu Oct 14 11:12:37 2021 -0400

    Merge pull request #21496 from kbrock/class_for_ems

    Introduce Ems#class_for_ems

    (cherry picked from commit 603b40a9d03188c4009fa80269ffdad3e255bb0d)

Skipping backport to najdorf, because it is already in the branch.


Greyed out blocked classes

This is to make the blocked classes greyed out in the replay to not give the impression that the website is broken during replay.

Hi @IMFIL
I believe we already have this style in https://github.com/rrweb-io/rrweb/blob/master/src/replay/styles/inject-style.ts#L2

Hey @Yuyz0112 ,
When are those styles applied ? I have been using replayer and all the blocked classes are transparent.

@IMFIL It was applied when init the replayer.


Issue 143 (proposed)

Fix issue #143
Adapted format_query_response() to also handle tags returned.
I'd like some advices on the format I use to distinguish same serie with different tags.
Basically : if there are some tag(s) returned by a query then store them in the result by providing a unique entry for each such unique serie + tag(s) value(s) with name = (row['name'], tuple(tags.values()))
Now after thinking about it twice I think in fact I should also include the tags.keys() somewhere.. also more than probably in the name attribute of each result dict with name = (row['name'], tuple(tags.items()))   (items simply instead of values so) ; so that the caller can know which tag value correspond to which tag name..  otherwise if the groupby is done on multiple tags so then the caller can't know for sure the order of the tags as returned in the answer.
wdyt ?

I think that the series identifier should be a tuple with a name and a dict.
for example: ("cpu", {"host_name": "example.com"})

I think that the series identifier should be a tuple with a name and a dict.
for example: ("cpu", {"host_name": "example.com"})

yep, that's what I also get in my second thought presented here above.
Problem is that the name attribute of the main dict result (from the format_query_response function) is a key (in the dict result so), so we can't use a dict as one of its component (not hashable).. so my tags.items() good second choice I think..

so we can't use a dict as one of its component (not hashable)

You are right! :(

so my tags.items() good second choice I think..

Absolutely.

will travis make it succeed on the first pass ?

will travis make it succeed on the first pass ?
What?

What?

I mean  : "is travis going to succeed the tests without any temporary error" :p

Looks like it failed for a valid reason. The order of tags is problematic. Looks like we will have to sort them.

fixed in the test.
I'm not in favor of sorting things "undesirably" in the library itself. If one wants the sorted serie names from the library it should re-use the same "trick".. after its query() call.
So I could eventually put the function sorted_series_tags in the library itself. (with a better name eventually ? if you have one..)  or not at all.
what do you think ?

We are not sorting the series but just the tags,  so I don't see why this is underisable.

Hmm, makes sense..

Seems all good..

Seems good to me too!


What kind of beacons does it support to scan, as IBeacons and Eddystone type of beacons are not getting detected

Hi @Keyurkariya , Eddystone beacons are supported without any restrictions by the vendor. Related scanning code can be found here: https://github.com/AgileVisionCompany/proximity-sdk-android/blob/master/indoor/navigator/src/main/java/io/agilevision/proximity/indoor/BeaconsSearcher.kt
Can you please share device you are using and Eddystone packet types you tried.

Hi @Keyurkariya , Eddystone beacons are supported without any restrictions by the vendor. Related scanning code can be found here: https://github.com/AgileVisionCompany/proximity-sdk-android/blob/master/indoor/navigator/src/main/java/io/agilevision/proximity/indoor/BeaconsSearcher.kt
Can you please share device you are using and Eddystone packet types you tried.


This project still alive?

I'm used this project to get information about my internet running in my home, I'm running a more updated version and I really appreciate If I can contribute with this project to be used for me and another users.
But I saw a opened PR from two years ago and not be merged and due that I'm asking if this project still alive?

Hi @bergpb - I blame getting too many notifications from github and this getting lost in that noise.  The old PR is merged now.  I've been using https://github.com/MiguelNdeCarvalho/speedtest-exporter in my personal cluster for a while now and think I forgot about this project.


don't fail when termcolor is not installed

Maybe we should just write the ansi escape codes by hand?

Yea, I'm fine with that

https://github.com/geohot/tinygrad/pull/434/commits/8ee8d01ca17132c069db89eceab01f5ed5a9222f
Replaced library with one line

What a hacky one-line, I love it!


Apply YAML notifications are hidden behind the modal's backdrop

What steps did you take and what happened:
When submitting new resources via 'Apply YAML', there is a success and/or error notifications that are obscured by the modal's backdrop making them difficult to notice and very difficult to read.
What did you expect to happen:
The notifications should be highly visible to the user.
Anything else you would like to add:

Environment:

Octant version (use octant version): HEAD

Since the modal doesn't go away after the resource is applied and messages relevant to the user's action in the modal, one approach is to display modal instead.


new command for running 'git log -S' against all repos in the team

@glasseyes42 I know this is not exactly what you are looking for, but might still be quite useful in a number of instances. Maybe we should have a team that has ALL the repos in it?  ;-)
not totally convinced that 'history' is the right name for it, but I thought it was a start in the right direction.

Coverage decreased (-0.2%) to 20.912% when pulling 3cac2aabe4a4f28cb7a95bc7d627c3565ea14e5e on history-command into c7dac38fba383519005a76f77d2a18ee79bc843f on master.

@andyduncan how is this different to the current activity command?
Also, we use to have all the repos against 'staff', I had a script that made sure they were all part of that team - let me find it.

I think that 'activity' will just show recent.
The 'git log -S' will show any commits in the history that contain the search term. Useful for trying to find if a usage of a variable was removed (not evident in a blame/annotate).

Got it.


fix: fixed cachine

#171  👈
develop

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @bob-obringer and the rest of your teammates on  Graphite


HBASE-27826 Refactor code to move creation of Ref files to SFT interface apis

Modifies HStoreFile/StoreFileInfo constructors to take SFT interface as a parameter.
Refactors direct interactions of Reference/HFileLink creations to SFT interface. Also moves getStoreFiles/hasReferences from HRegionFS to SFT impls.
Use the SFT interface to list files of store everywhere instead of using FS objects directly

Modifies HStoreFile/StoreFileInfo constructors to take SFT interface as a parameter. Refactors direct interactions of Reference/HFileLink creations to SFT interface. Also moves getStoreFiles/hasReferences from HRegionFS to SFT impls. Use the SFT interface to list files of store everywhere instead of using FS objects directly

This POC PR is primarily to get a high level overview of the list of changes and is intended to be broken once the initial review is done.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 27s
Docker mode activated.

-0 :warning:
yetus
0m  3s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+1 :green_heart:
mvninstall
2m 47s
master passed

+1 :green_heart:
compile
0m 39s
master passed

+1 :green_heart:
shadedjars
5m 32s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 24s
master passed

_ Patch Compile Tests _

-1 :x:
mvninstall
1m 14s
root in the patch failed.

-1 :x:
compile
0m 36s
hbase-server in the patch failed.

-0 :warning:
javac
0m 36s
hbase-server in the patch failed.

-1 :x:
shadedjars
4m  7s
patch has 616 errors when building our shaded downstream artifacts.

-0 :warning:
javadoc
0m 22s
hbase-server generated 3 new + 22 unchanged - 0 fixed = 25 total (was 22)

_ Other Tests _

-1 :x:
unit
0m 36s
hbase-server in the patch failed.

17m 39s

Subsystem
Report/Notes

Docker
ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5834/1/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/5834

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux cbb4cdc31d98 5.4.0-1103-aws #111~18.04.1-Ubuntu SMP Tue May 23 20:04:10 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / d7f9ba6548

Default Java
Temurin-1.8.0_352-b08

mvninstall
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5834/1/artifact/yetus-jdk8-hadoop3-check/output/patch-mvninstall-root.txt

compile
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5834/1/artifact/yetus-jdk8-hadoop3-check/output/patch-compile-hbase-server.txt

javac
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5834/1/artifact/yetus-jdk8-hadoop3-check/output/patch-compile-hbase-server.txt

shadedjars
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5834/1/artifact/yetus-jdk8-hadoop3-check/output/patch-shadedjars.txt

javadoc
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5834/1/artifact/yetus-jdk8-hadoop3-check/output/diff-javadoc-javadoc-hbase-server.txt

unit
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5834/1/artifact/yetus-jdk8-hadoop3-check/output/patch-unit-hbase-server.txt

Test Results
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5834/1/testReport/

Max. process+thread count
67 (vs. ulimit of 30000)

modules
C: hbase-server U: hbase-server

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5834/1/console

versions
git=2.34.1 maven=3.8.6

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 39s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ master Compile Tests _

+1 :green_heart:
mvninstall
3m 30s
master passed

+1 :green_heart:
compile
3m 18s
master passed

+1 :green_heart:
checkstyle
0m 57s
master passed

+1 :green_heart:
spotless
1m  6s
branch has no errors when running spotless:check.

+1 :green_heart:
spotbugs
1m 53s
master passed

_ Patch Compile Tests _

-1 :x:
mvninstall
2m  7s
root in the patch failed.

-1 :x:
compile
1m 19s
hbase-server in the patch failed.

-0 :warning:
javac
1m 19s
hbase-server in the patch failed.

-0 :warning:
checkstyle
0m 42s
hbase-server: The patch generated 2 new + 55 unchanged - 2 fixed = 57 total (was 57)

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

-1 :x:
hadoopcheck
2m 26s
The patch causes 612 errors with Hadoop v3.3.6.

+1 :green_heart:
spotless
0m 50s
patch has no errors when running spotless:check.

-1 :x:
spotbugs
0m 48s
hbase-server in the patch failed.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ master Compile Tests _

+1 :green_heart:
mvninstall
3m 39s
master passed

+1 :green_heart:
compile
3m  9s
master passed

+1 :green_heart:
checkstyle
0m 55s
master passed

+1 :green_heart:
spotless
0m 54s
branch has no errors when running spotless:check.

+1 :green_heart:
spotbugs
1m 59s
master passed

_ Patch Compile Tests _

-1 :x:
mvninstall
2m  3s
root in the patch failed.

-1 :x:
compile
1m 11s
hbase-server in the patch failed.

-0 :warning:
javac
1m 11s
hbase-server in the patch failed.

-0 :warning:
checkstyle
0m 40s
hbase-server: The patch generated 2 new + 55 unchanged - 2 fixed = 57 total (was 57)

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

-1 :x:
hadoopcheck
2m 16s
The patch causes 612 errors with Hadoop v3.3.6.

+1 :green_heart:
spotless
0m 47s
patch has no errors when running spotless:check.

-1 :x:
spotbugs
0m 50s
hbase-server in the patch failed.

_ Other Tests _

+1 :green_heart:
asflicense
0m 10s
The patch does not generate ASF License warnings.

20m 40s

Subsystem
Report/Notes

Docker
ClientAPI=1.45 ServerAPI=1.45 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5834/2/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/5834

Optional Tests
dupname asflicense javac spotbugs hadoopcheck hbaseanti spotless checkstyle compile

uname
Linux 468a939842f1 5.4.0-176-generic #196-Ubuntu SMP Fri Mar 22 16:46:39 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / d7f9ba6548

Default Java
Eclipse Adoptium-11.0.17+8

mvninstall
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5834/2/artifact/yetus-general-check/output/patch-mvninstall-root.txt

compile
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5834/2/artifact/yetus-general-check/output/patch-compile-hbase-server.txt

javac
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5834/2/artifact/yetus-general-check/output/patch-compile-hbase-server.txt

checkstyle
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5834/2/artifact/yetus-general-check/output/diff-checkstyle-hbase-server.txt

hadoopcheck
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5834/2/artifact/yetus-general-check/output/patch-javac-3.3.6.txt

spotbugs
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5834/2/artifact/yetus-general-check/output/patch-spotbugs-hbase-server.txt

Max. process+thread count
78 (vs. ulimit of 30000)

modules
C: hbase-server U: hbase-server

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5834/2/console

versions
git=2.34.1 maven=3.8.6 spotbugs=4.7.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 33s
Docker mode activated.

-0 :warning:
yetus
0m  3s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+1 :green_heart:
mvninstall
3m 38s
master passed

+1 :green_heart:
compile
0m 56s
master passed

+1 :green_heart:
shadedjars
6m 43s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 29s
master passed

_ Patch Compile Tests _

-1 :x:
mvninstall
1m 54s
root in the patch failed.

-1 :x:
compile
0m 51s
hbase-server in the patch failed.

-0 :warning:
javac
0m 51s
hbase-server in the patch failed.

-1 :x:
shadedjars
5m  1s
patch has 612 errors when building our shaded downstream artifacts.

-0 :warning:
javadoc
0m 30s
hbase-server generated 4 new + 93 unchanged - 3 fixed = 97 total (was 96)

_ Other Tests _

-1 :x:
unit
1m  2s
hbase-server in the patch failed.

22m 43s

Subsystem
Report/Notes

Docker
ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5834/2/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/5834

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 993919b8a98e 5.4.0-1103-aws #111~18.04.1-Ubuntu SMP Tue May 23 20:04:10 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / d7f9ba6548

Default Java
Eclipse Adoptium-11.0.17+8

mvninstall
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5834/2/artifact/yetus-jdk11-hadoop3-check/output/patch-mvninstall-root.txt

compile
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5834/2/artifact/yetus-jdk11-hadoop3-check/output/patch-compile-hbase-server.txt

javac
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5834/2/artifact/yetus-jdk11-hadoop3-check/output/patch-compile-hbase-server.txt

shadedjars
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5834/2/artifact/yetus-jdk11-hadoop3-check/output/patch-shadedjars.txt

javadoc
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5834/2/artifact/yetus-jdk11-hadoop3-check/output/diff-javadoc-javadoc-hbase-server.txt

unit
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5834/2/artifact/yetus-jdk11-hadoop3-check/output/patch-unit-hbase-server.txt

Test Results
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5834/2/testReport/

Max. process+thread count
79 (vs. ulimit of 30000)

modules
C: hbase-server U: hbase-server

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5834/2/console

versions
git=2.34.1 maven=3.8.6

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

Checked the POC, overall good.
It is a pain that we need to touch the MOB related code.
Anyway, I think first we could first do some refactorings, to move the reference file related logic to StoreFileTracker, without changing any real logic. And then, another refactoring to move HFileLink, and back reference files related logic to StoreFileTracker. And finally, we start to change the implementation of StoreFileTracker, to implement the special logic for FileBasedStoreFileTracker, and also consider how to migrate between different store file tracker implementions.
WDYT?
Thanks.

Checked the POC, overall good.
It is a pain that we need to touch the MOB related code.
Anyway, I think first we could first do some refactorings, to move the reference file related logic to StoreFileTracker, without changing any real logic. And then, another refactoring to move HFileLink, and back reference files related logic to StoreFileTracker. And finally, we start to change the implementation of StoreFileTracker, to implement the special logic for FileBasedStoreFileTracker, and also consider how to migrate between different store file tracker implementions.
WDYT?
Thanks.

Totally agree, we can target them 1 pr at a time.

to move the reference file related logic to StoreFileTracker
then, another refactoring to move HFileLink, and back reference files
then we start to change the implementation of StoreFileTracker, to implement the special logic for FileBasedStoreFileTracker to create virtual links
and commit all the ref files as part of split in 1 go

Thanks a lot for the review :)


Edit button on saved config are not working ? not yet implemented?

I'm having an issue with the edit button on the saved config card. Is the feature properly working for anyone else ?

It's broken indeed... fixing.

@cthiounn it's fixed. Sorry for the long delay. I didn't noticed the issue.


SonyLIV

Checklist

[x ] I'm reporting a broken site support
[x ] I've verified that I'm running youtube-dl version 2020.06.06
[x ] I've checked that all provided URLs are alive and playable in a browser
[x ] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x ] I've searched the bugtracker for similar issues including closed ones

Verbose log

PASTE VERBOSE LOG HERE

youtube-dl6 -v https://www.sonyliv.com/shows/the-kapil-sharma-show-1700000083/kapil-s-wonderful-audience-1000044877
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-v', 'https://www.sonyliv.com/shows/the-kapil-sharma-show-1700000083/kapil-s-wonderful-audience-1000044877']
[debug] Encodings: locale cp1252, fs mbcs, out cp437, pref cp1252
[debug] youtube-dl version 2020.06.06
[debug] Python version 3.4.4 (CPython) - Windows-10-10.0.19041
[debug] exe versions: ffmpeg git-2020-04-13-59e3a9a
[debug] Proxy map: {}
[generic] kapil-s-wonderful-audience-1000044877: Requesting header
WARNING: Falling back on generic information extractor.
[generic] kapil-s-wonderful-audience-1000044877: Downloading webpage
[generic] kapil-s-wonderful-audience-1000044877: Extracting information
WARNING: [generic] kapil-s-wonderful-audience-1000044877: Failed to parse JSON Extra data: line 15 column 19 - line 17 column 17 (char 691 - 743)
ERROR: Unsupported URL: https://www.sonyliv.com/shows/the-kapil-sharma-show-1700000083/kapil-s-wonderful-audience-1000044877
Traceback (most recent call last):
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpd0boo4lk\build\youtube_dl\YoutubeDL.py", line 797, in extract_info
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpd0boo4lk\build\youtube_dl\extractor\common.py", line 530, in extract
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpd0boo4lk\build\youtube_dl\extractor\generic.py", line 3370, in _real_extract
youtube_dl.utils.UnsupportedError: Unsupported URL: https://www.sonyliv.com/shows/the-kapil-sharma-show-1700000083/kapil-s-wonderful-audience-1000044877
Description

WRITE DESCRIPTION HERE
Downloading is not working since SonyLiv updated their website
official new MPD URL
https://securetoken.sonyliv.com/show_set_thekapilsharmashow_ep127_20200327T145303.mpd
but this is working MPD url
https://streaming.sonyliv.com/show_set_thekapilsharmashow_ep127_20200327T145303.mpd
seems like, need t modify to get working MPD URLs
Please update,
Thanks

i am too facing a similar problem but with a different show

when will ytdl is going to update

Same issue, latest youtube-dl. @remitamine Please look into this, if you have time :)

A working Indian VPN or proxy to work on this issue.

A working Indian VPN or proxy to work on this issue.
How Do i send it?
By the way, send one on<EMAIL_ADDRESS>

if dstftw expressed his interest in working on this issue then no need to send it to me, otherwise you can contact me on the E-Mail from my profile.

email sent, please check

you need to use the following:
youtube-dl --add-header "x-playback-session-id: THEXPLAYBACKID" "MPDLINK"
Just replace THEXPLAYBACKID and MPDLINK by searching for the respective data in the network tab in developer mode. You can open it by pressing f12. Don't forget to reload the site after that and simply search 'mpd' and 'x-playback-session-id'

you need to use the following:
youtube-dl --add-header "x-playback-session-id: THEXPLAYBACKID" "MPDLINK"
Just replace THEXPLAYBACKID and MPDLINK by searching for the respective data in the network tab in developer mode. You can open it by pressing f12. Don't forget to reload the site after that and simply search 'mpd' and 'x-playback-session-id'

i am able to find the playbackid but where is the mpdlink? can u give an example?

you need to use the following:
youtube-dl --add-header "x-playback-session-id: THEXPLAYBACKID" "MPDLINK"
Just replace THEXPLAYBACKID and MPDLINK by searching for the respective data in the network tab in developer mode. You can open it by pressing f12. Don't forget to reload the site after that and simply search 'mpd' and 'x-playback-session-id'

i am able to find the playbackid but where is the mpdlink? can u give an example?

Did u try searching for mpd in the search bar? This is called mpd link cuz it ends with .mpd. You can read https://www.youtube.com/watch?v=xViWU57xDk4 for more help.

you need to use the following:
youtube-dl --add-header "x-playback-session-id: THEXPLAYBACKID" "MPDLINK"
Just replace THEXPLAYBACKID and MPDLINK by searching for the respective data in the network tab in developer mode. You can open it by pressing f12. Don't forget to reload the site after that and simply search 'mpd' and 'x-playback-session-id'

i am able to find the playbackid but where is the mpdlink? can u give an example?

Did u try searching for mpd in the search bar? This is called mpd link cuz it ends with .mpd. You can read https://www.reddit.com/r/youtubedl/comments/jzd7ig/cant_download_from_sonyliv_website/ for more help.

thanks a lot. Got it working but hope the extractor get's fixed soon.

@remitamine hey are you still working on this?

no, the VPN provided by @Mouse753 didn't work for me(problem with OpenVPN config), and i did try later to access to the website with proxies which did allow to access the website(didn't test if access to the manifests is possible), now the website seems to require an account to access any video, i will try later to check again.


homebridge-ewelink can not be installed on new Factory reseted Hoobs 4

What issue do you have? Please be as thorough and explicit as possible.
I have reset my Hoobs to Factory Default.
I tried to install the ewelink plugin but no success.
it asking me to upgrade Homebridge to 1.3.8, however Hoobs does not offer the automatic upgrade.
12/21/2021, 1:50:10 PMeWeLink BridgeERRORThe plugin "homebridge-ewelink" requires a Homebridge version of ^1.3.8 which does not satisfy the current Homebridge version of 1.3.4. You may need to update this plugin (or Homebridge) to a newer version. You may face unexpected issues or stability problems running this plugin.
Details of your setup.

Do you use (1) Homebridge UI-X (2) Homebridge CLI or (3) HOOBS?

HOOBS

Which version of Homebridge/HOOBS do you have?

HOOBS: 4.0.110
Node version: 16.13.1
Homebridge: 1.3.4

Which platform do you run Homebridge/HOOBS on (e.g. Raspberry Pi/Windows/HOOBS Box)? Please also mention your version of Node.js/NPM if known.

HOOBS

Which version of this plugin (homebridge-ewelink) do you have? Has the issue started since upgrading from a previous version?

ewelink: 8.8

Which eWeLink devices do you have that are causing issues? Please include product models if applicable.

Please paste any relevant logs below.
12/21/2021, 1:18:35 PMhub is running on port 80
12/21/2021, 1:48:55 PMeWeLink Bridge starting
12/21/2021, 1:49:13 PMeWeLink BridgeWARNINGNo plugins installed.
12/21/2021, 1:49:14 PMeWeLink BridgeBridge is running on port 51826.
12/21/2021, 1:49:17 PMeWeLink BridgeConfiguration change
12/21/2021, 1:49:17 PMewelinkbridge stopping
12/21/2021, 1:49:22 PMeWeLink Bridge starting
12/21/2021, 1:49:35 PMeWeLink BridgeERRORThe plugin "homebridge-ewelink" requires a Homebridge version of ^1.3.8 which does not satisfy the current Homebridge version of 1.3.4. You may need to update this plugin (or Homebridge) to a newer version. You may face unexpected issues or stability problems running this plugin.
12/21/2021, 1:49:35 PMeWeLink BridgeLoaded plugin 'homebridge-ewelink'
12/21/2021, 1:49:35 PMeWeLink BridgeLoading 1 platforms...
12/21/2021, 1:49:35 PMeWeLink BridgeeWeLinkInitialising plugin v8.8.0 | Node v16.13.1 | HB v1.3.4 | HOOBS v4...
12/21/2021, 1:49:35 PMeWeLink BridgeeWeLinkPlugin initialised. Setting up accessories....
12/21/2021, 1:49:35 PMeWeLink BridgeeWeLinkWARNING***** Disabling plugin. *****
12/21/2021, 1:49:35 PMeWeLink BridgeeWeLinkWARNING***** eWeLink credentials not configured. *****
12/21/2021, 1:49:36 PMeWeLink BridgeBridge is running on port 51826.
12/21/2021, 1:50:01 PMeWeLink BridgeConfiguration change
12/21/2021, 1:50:01 PMewelinkbridge stopping
12/21/2021, 1:50:06 PMeWeLink Bridge starting
12/21/2021, 1:50:10 PMeWeLink BridgeERRORThe plugin "homebridge-ewelink" requires a Homebridge version of ^1.3.8 which does not satisfy the current Homebridge version of 1.3.4. You may need to update this plugin (or Homebridge) to a newer version. You may face unexpected issues or stability problems running this plugin.
12/21/2021, 1:50:10 PMeWeLink BridgeLoaded plugin 'homebridge-ewelink'
12/21/2021, 1:50:10 PMeWeLink BridgeLoading 1 platforms...
12/21/2021, 1:50:10 PMeWeLink BridgeeWeLinkInitialising plugin v8.8.0 | Node v16.13.1 | HB v1.3.4 | HOOBS v4...
12/21/2021, 1:50:10 PMeWeLink BridgeeWeLinkPlugin initialised. Setting up accessories....
12/21/2021, 1:50:11 PMeWeLink BridgeBridge is running on port 51826.
12/21/2021, 1:50:11 PMeWeLink BridgeeWeLinkSending HTTP login() request.
12/21/2021, 1:50:12 PMeWeLink BridgeeWeLinkNew HTTP API host received [as-apia.coolkit.cc].
12/21/2021, 1:50:12 PMeWeLink BridgeeWeLinkSending HTTP login() request.
12/21/2021, 1:50:17 PMeWeLink BridgeeWeLinkWARNING***** Disabling plugin. *****
12/21/2021, 1:50:17 PMeWeLink BridgeeWeLinkWARNING***** Cannot find module 'es-abstract/2021/IsCallable'
Require stack:
- /var/lib/hoobs/ewelinkbridge/node_modules/promise.prototype.finally/implementation.js
- /var/lib/hoobs/ewelinkbridge/node_modules/promise.prototype.finally/index.js
- /var/lib/hoobs/ewelinkbridge/node_modules/websocket-as-promised/src/requests.js
- /var/lib/hoobs/ewelinkbridge/node_modules/websocket-as-promised/src/index.js
- /var/lib/hoobs/ewelinkbridge/node_modules/homebridge-ewelink/lib/connection/ws.js
- /var/lib/hoobs/ewelinkbridge/node_modules/homebridge-ewelink/lib/index.js
- /usr/lib/hoobsd/node_modules/homebridge/lib/plugin.js
- /usr/lib/hoobsd/node_modules/homebridge/lib/pluginManager.js
- /usr/lib/hoobsd/node_modules/homebridge/lib/api.js
- /usr/lib/hoobsd/bridge/server.js
- /usr/lib/hoobsd/bridge/index.js
- /usr/lib/hoobsd/main.js
- /usr/bin/hoobsdRequire stack:. *****

Hi @alsolaim94
Can you try deleting the bridge/uninstalling the plugin and starting again?
I think there was a problem with the initial plugin install - the homebridge version warning shouldn’t have an effect on the plugin.
Thanks!


Added inital support for Ubuntu 18.04 LTS

This closes #40. This adds a copy paste and confirmed working version of Ubuntu 18.04.

Your tests are failing.

Updated to Ubuntu 18.04 LTS since https://github.com/William-Yeh/docker-ansible/commit/367cf1dca5daec01c4b2aa15ea9b00abe5ca319d