New configuration option needed for specific but popular setups eg. LED strips

I think about mulitiplier/factor which can be used with drivers - for driver used with LED strips or even integrated low-voltage lamps (but for standard bulbs or lamps it just should always be 1, so it should have default value 1 only with option to modify when desired).
This multiplier should be used only with power data from LUT file (and not for standby_power).
Hardware examples
integrated lamps (could be used one or more pieces with one driver):
https://www.ikea.com/gb/en/p/urshult-led-cabinet-lighting-nickel-plated-30260402/
https://www.ikea.com/gb/en/p/lindshult-led-cabinet-lighting-nickel-plated-10260436/
drivers:
https://www.ikea.com/gb/en/p/tradfri-driver-for-wireless-control-grey-50356187/
https://www.ikea.com/gb/en/p/tradfri-driver-for-wireless-control-grey-60342656/
also LED stips in standard starter-kits have driver and predefined length of strip (eg. 5m), but available extension sets contain only smaller quantity and in case of strip it is possible to cut strip to desired size.
It is a kind of compromise (driver power usage counted properly only in case of standby), but I think, that LUT file should contain data for 1m of strip or for 1 piece of integrated lamp (measured or calculated from measure more than one piece connected together to driver or measured strip of any length; best measurements would be in standard setup - so I mean LED strip of leghth just like out of the box or typical count integrated lamps used with given model of driver).
It is of course possible to measure only driver working idle as separate file but it really make no sense because power losses in driver  are different without load then in real work.
Some variation of that factor could be used also for groups of identical bulbs controlled always together (as group), but in such case standby power should be also multiplied.

Would be easy to implement a "multiply factor", but how does a user determine the correct multiply factor? And will this be consistent over the whole brightness / color temp spectrum. Not sure about these things.

how does a user determine the correct multiply factor?
it is very simple - LUT file must contain power value for 1m of strip or 1 piece of lamp - calculated after measure.

Some words about measuring:
eg. if LUT file is created using 5m of strip then every power value in ready made LUT file must be calculated by division by 5
if it was measured for 2 lamps connected then values in final LUT file should be divided by 2
Of course power draw used by driver will be also divided - so I've suggested measuring just after opening a box - it will minimize differences in real use - I think when someone buys 5m starter kit, he won't use only 0,5m of strip, but rather 4,5m or so.
And about use multiplier - if someone uses 4.5m of strip then he stets multiplier to 4.5 if 7.5m then 7.5

I will give this another thought.
Not sure if we should divide values to have the LUT files converted to 1 meter (for hue ledstrip for example), or we use the default length as base. For the Hue lightstrip the default length (as you buy it in the store) is 2 meters, so imo the LUT file should be for the stock model (which is 2 meters). When the user has extended the strip he can apply the multiply factor accordingly. But he shouldn't need to do anything when using the default model, because this will be error prone.
But also see this post, he said power should be the same even when the led strip is extended.
I know we don't have laboratorium conditions at home, when the measured values are close enough users have a good indication about there power consumption which is what this component is all about. It does not has to be scientifically perfect.

For the Hue lightstrip the default length (as you buy it in the store) is 2 meters, so imo the LUT file should be for the stock model (which is 2 meters). When the user has extended the strip he can apply the multiply factor accordingly. But he shouldn't need to do anything when using the default model, because this will be error prone.

Maybe it is really better solution, but linked above Ikea lamps are sold separately (and only requirement is not to exceed the driver specification), so how they should be measured?
I think about conversion to unitary value, because it does not matter if starter-kit contains 2m of strip or 5m of strip or 2 lamps or 3 lamps, but maybe better is to measure standard contents of the package as single unit.
It is just to be documented properly, but  some kind of multiplier/factor is a must (because I do not know any person who use standard length of strip :P).

Haha nice laboratory ;-).
I have also found this article https://atomstar.tweakblogs.net/blog/17888/measuring-calibrated-hue-energy-usage-via-bridge.
This guy had measured 2m and extended 4m strip, but the 4 meter strip only uses 40% more power (not 100% more). So this feature will be almost undoable to implement :-(

Haha nice laboratory ;-).
I have also found this article https://atomstar.tweakblogs.net/blog/17888/measuring-calibrated-hue-energy-usage-via-bridge.
This guy had measured 2m and extended 4m strip, but the 4 meter strip only uses 40% more power (not 100% more). So this feature will be almost undoable to implement :-(

Could be a standardvalue for 2 meters and then a configure option for addional meters.

Another case when the multiplier would be useful is in my house I have a room with 6 RS125 downlights. I don't need to have a sensor for each of them so currently I have a powercalc sensor for one of them and then I have a template sensor which multiplies that one by six for the total room power when that one is on (since they all get turned on and off as a group). It would be great if I didn't need the template sensor but I could just use the multiplier. This isn't an isolated case for me, My dining room has two identcal bulbs and I have corridors with 3 and 4 identical bulbs....

Haha nice laboratory ;-).
I have also found this article https://atomstar.tweakblogs.net/blog/17888/measuring-calibrated-hue-energy-usage-via-bridge.
This guy had measured 2m and extended 4m strip, but the 4 meter strip only uses 40% more power (not 100% more). So this feature will be almost undoable to implement :-(

Could be a standardvalue for 2 meters and then a configure option for additional meters.
The reason it doesn't take 100% more power is that with addional meters it will affect the brightness because total lumen is still the same with addional meters. 40% additional more power sounds infact much for me.
Would be nice with results from other configurations but maybe it's 20% more power for each meter?

I have installed some led strips myself (just the standard ones) with a custom led driver and a gledopto controller (so I can control them via zigbee). Each led just draws a certain amount of power on full brightness so I just make sure the LED driver has enough amps to control the full length on full brightness. So in this circumstance doubling the amount of meters will also double the amount of watts.
The Hue driver (adapter) is rated for a max amount of amps, so when you make the strip to long it each individual light will not have enougt power, thus resulting in a lower brightness output.
The problem is we don't know which kind of configurations the user has, so it will be a lot of guesses.

Another case when the multiplier would be useful is in my house I have a room with 6 RS125 downlights. I don't need to have a sensor for each of them so currently I have a powercalc sensor for one of them and then I have a template sensor which multiplies that one by six for the total room power when that one is on (since they all get turned on and off as a group). It would be great if I didn't need the template sensor but I could just use the multiplier. This isn't an isolated case for me, My dining room has two identcal bulbs and I have corridors with 3 and 4 identical bulbs....

Yes this is a perfect usecase for this feature request. I also have a room with 6 GU spots and now have 6 power sensors (and a template sensor to sum them up), but it would be much nicer to have a single power sensor as I also control these as a group.

See #103

I have added some information to the readme about the new configuration option.
See:
https://github.com/bramstroker/homeassistant-powercalc/tree/feature/multiply_factor#sensor-configuration
https://github.com/bramstroker/homeassistant-powercalc/tree/feature/multiply_factor#multiply-factor
Let me know if you have any additions.

Nice option that should fix the above example with a factor of 1.4 for example.

Nice option that should fix the above example with a factor of 1.4 for example.

Yes exactly, if you want 40% more power you use 1.4.

I have also found this article https://atomstar.tweakblogs.net/blog/17888/measuring-calibrated-hue-energy-usage-via-bridge.

In this case results of the measurement of the strips look unbelievable, efficiency of driver depends on load, but such a big difference is not possible (it is likely only in case of overload when driver is full saturated) as it @Leatherface75 said before.

The problem is we don't know which kind of configurations the user has, so it will be a lot of guesses.

It is not our problem - when configuration is build within specs (driver isn't overloaded) - additional (or substracted) load could be estimated as linear change, so multiplication factor is OK.

Feature has been merged to master. Closing this issue


feat: add construct library for Application AutoScaling

Adds a library for application autoscaling. Should be consumed and re-exposed in a better way by other services. To start with, the DynamoDB implementation now uses it.
BREAKING CHANGE: The API for enabling AutoScaling for DynamoDB tables has changed.
Fixes #856, #861, #640, #644.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license.

If you're wondering why this new package doesn't have any integ tests: it's because the dependency arrows would point the wrong way. But the library is integ tested in each individual service package that exposes autoscaling constructs.

'nother bug in JSII, looks like:
@aws-cdk/aws-iam: [ERROR] Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.6.1:compile (default-compile) on project iam: Compilation failure: Compilation failure:
@aws-cdk/aws-iam: [ERROR] /tmp/jsii-pacmak-codeUbHgJB/src/main/java/software/amazon/awscdk/services/iam/IRole.java:[26,69] variable _principal is already defined in class software.amazon.awscdk.services.iam.IRole.Builder
@aws-cdk/aws-iam: [ERROR] /tmp/jsii-pacmak-codeUbHgJB/src/main/java/software/amazon/awscdk/services/iam/IRole.java:[52,24] method withPrincipal(software.amazon.awscdk.services.iam.PolicyPrincipal) is already defined in class

Blocked by https://github.com/awslabs/jsii/issues/264

Can you work around this somehow? Perhaps add a fake method to the interface so it's not identified as a datatype?

Blocked on https://github.com/awslabs/jsii/pull/265

Can you work around this somehow? Perhaps add a fake method to the interface so it's not identified as a datatype?

I could, but I don't really want to. I'd rather fix the root cause.

Also fixes #673.

Congratulations! 🎉  beautiful design!


Website deploy failed

It failed with: Error: Config file 'mkdocs.yml' does not exist.
https://github.com/elixir-lsp/elixir-ls/runs/4420319963?check_suite_focus=true
Probably easy to resolve but I wanted to file an issue for tracking.

@axelson was this resolved? It might have been failing because no checkout step was present before the built attempt.

I think it's resolved now https://github.com/elixir-lsp/elixir-ls/actions/runs/2472979277


[BUG] Footer is not getting added in the Downloaded Reports

The footer that is added manually in report generation is not shown in the downloaded report.
Created a Report definition, selected a dashboard for which a report needs to be generated, and added a header and footer for the report.
In the downloaded PDF / PNG able to see only the header and the Dashboard content alone.
In plugins/reportsDashboards/server/routes/utils/visual_report/visualReportHelper.js file, the custom footer node is not appending to the parent node when the Report HTML format is created. Whereas inserting a DOM node before a parent node works fine.

If it's the same as the bug I'm encountering, it seems like despite margin-space being allocated for the footer, the footer itself is being added to the bottom of the dashboard content rather than to said margin-space, resulting in the footer being cut off, the inevitable scroll bar pasted onto the side and the footer space left empty. This mis-ordering in printing could also explain why text coded for right-alignment in the header doesn't get aligned all the way to the right.
To fix this, the bottom margin should be minimized, much how (assumedly) the top margin is.
(I know this isn't the place for this, but I'm also gonna complain how dark mode reporting doesn't but should default to a matching dark-grey background for the PDF and how light/dark mode should be a toggle option for reports and how this would be easily solved if dark mode were a URI parameter)


Enable tests back

Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge


Illegal Map comparison when using isEqualToComparingFieldByFieldRecursively

Summary
Map comparison is broken when using isEqualToComparingFieldByFieldRecursively
May be i misunderstand this feature, but i think that Map's (and any java Collection) must be handled by equals without comparing them field-by-field, at least this is common behavior which everybody will expect - imho.
Example
class A {
    Map<Integer, Boolean> map;
}

@Test
void test1() {
    A a = new A(), b = new A();
    a.map = new HashMap<>();
    a.map.put(1, true);

    b.map = new TreeMap<>();
    b.map.put(1, true);

    assertThat(a).isEqualToComparingFieldByFieldRecursively(b);
}

Produces this output:
Path to difference:  <map>
- expected: <{1=true}>
- actual  : <{1=true}>

The library that is used for recursive comparison comes from https://github.com/jdereg/java-util, it does compare collection and maps by their content, it is not using their equals method.
Is that a desired behaviour ? I think so as the recursive comparison is about content, the javadoc should be improved to make this clear. Would that work for you ?
You can use usingComparatorForType(comparator, type) if you need to introduce a specific comparison strategy.

I've tried  usingComparatorForType(comparator, type) and usingComparatorForFields(comparator, fields) without success - its just ignores this.

Oh, i've take a look at difference detection code..
org.assertj.core.internal.DeepDifference
  if (key1 instanceof SortedMap) {
    if (!(key2 instanceof SortedMap)) {
      differences.add(new Difference(currentPath, key1, key2));
      continue;
    }
  } else if (key2 instanceof SortedMap) {
    differences.add(new Difference(currentPath, key1, key2));
    continue;
  }

  if (key1 instanceof Map) {
    if (!(key2 instanceof Map)) {
      differences.add(new Difference(currentPath, key1, key2));
      continue;
    }
  } else if (key2 instanceof Map) {
    differences.add(new Difference(currentPath, key1, key2));
    continue;
  }

It's makes a difference between SortedMap and Map. As i understand this is by design but why and for what?

This code comes from https://github.com/jdereg/java-util, my guess is that if you have a sorted map you want to take the order into account but it should be able to compare the content of a sorted map and a regular one, I consider this is a bug then.
We will also check usingComparatorForType behaviour, should work.

Yes SortedMaps are only considered equal if they both have the same elements in the same order. The instanceof checks are just a performance optimization.
Taking this into consideration I do not think it makes sense to compare a sorted to a unsorted map (at least if one map contains more that one element).

https://github.com/joel-costigliola/assertj-core/pull/764 makes it possible to use a custom comparator for a concrete collection or map type.
Adding support for registering a comparator for an interface like Map that is then used for all classes implementing it would be a lot more work. The simple check in https://github.com/joel-costigliola/assertj-core/blob/ff021db1c1cbe02b24998228d5659f5eca00d0a6/src/main/java/org/assertj/core/internal/DeepDifference.java#L305 would have to be significantly extended.

Taking this into consideration I do not think it makes sense to compare a sorted to a unsorted map (at least if one map contains more that one element).

Well, may be...
But i think that comparison must be consistent with equals, which doesnt take any order into account at all (if you compare two TreeMap's with different Comparators but same elements - equals will return true)

I haven't made up my mind on what the proper behaviour should be, for the time being I think it should compare map content as the recursive comparison is about content.

My thoughts on this one have changed a bit, I tend to think that the actual behavior is ok (I now agree with @PascalSchumacher comment).
If we were to allow comparing sorted and unsorted maps then for the sake of consistency we should also comparing set and sorted set, and why not list vs set. Basically isEqualToComparingFieldByFieldRecursively is expecting fields of actual and expected to have the same type.
What we could do instead is to improve the error message for two unequal fields wit the same string representation by adding the field's type.

Error message has been improved and now show differences more accurately when standard fields description happened to be the same, in the given example that gives:
Path to difference: <map>
- expected: <{1=true} (TreeMap@4ce)>
- actual  : <{1=true} (HashMap@4ce)>

The type and hash code have been added to disambiguate the fields description when their default representation was identical.

Basically isEqualToComparingFieldByFieldRecursively is expecting fields of actual and expected to have the same type.

Ok. Then following test must fail:
interface XAble {
    int getX();
}

class A implements XAble {
    final int x;

    public A(int x) {
        this.x = x;
    }

    public int getX() {
        return x;
    }
}

class B implements XAble {
    final int x;

    public B(int x) {
        this.x = x;
    }

    public int getX() {
        return x;
    }
}

class C {

    final XAble v;

    public C(XAble v) {
        this.v = v;
    }

    public XAble getV() {
        return v;
    }
}

public class IsEqualToComparingFieldByFieldRecursively {

    @Test
    public void testIsEqualToComparingFieldByFieldRecursively() throws Exception {
        final C c1 = new C(new A(15));
        final C c2 = new C(new B(15));
        assertThat(c1).isEqualToComparingFieldByFieldRecursively(c2);
    }
}

But it doesnt... Passed


Repo sync for protected CLA branch

The pull request is created from public637915383455595593 to public to fix git push error for protected CLA branch

@JohanFreelancer9

Hi @chrfox! I reviewed the article that's mine in this batch. It looks like the other one is yours.


Cannot read property 'WorkerMessageHandler' of undefined Why?

Attach (recommended) or Link to PDF file here:
Configuration:

Web browser and its version: 80.0.3987.149
Operating system and its version:MacBook Pro (13-inch, 2017, Two Thunderbolt 3 ports
PDF.js version:2.2.228
Is a browser extension:

Steps to reproduce the problem:

This is the demo of the recurrence problem
https://codesandbox.io/s/quizzical-sanderson-1zte2

What is the expected behavior? (add screenshot)
I want to know where the problem is and how to solve it
What went wrong? (add screenshot)

Link to a viewer (if hosted on a site other than mozilla.github.io/pdf.js or as Firefox/Chrome extension):

Using https in front of the url seems to resolve the problem for me:
pdfjs.GlobalWorkerOptions.workerSrc = "https://cdn.bootcss.com/pdf.js/2.2.228/pdf.worker.js";

Web browser and its version: 80.0.3987.149

The actual browser name seems to be missing here (there's only a version number).

PDF.js version: 2.2.228

That version is no longer supported, please find the latest releases at https://github.com/mozilla/pdf.js/releases

This is the demo of the recurrence problem
https://codesandbox.io/s/quizzical-sanderson-1zte2

That example seems to be using React, which I don't think any of the PDF.js core contributors know much about. Can you reproduce the error when using the PDF.js library directly without React?

Ninja-edit: Looking briefly at your example, there's this line:
pdfjsLib.GlobalWorkerOptions.workerSrc = require("pdfjs-dist/build/pdf.worker.js");

That's not correct, since as you can see in the JSDocs that parameter must be a String: https://github.com/mozilla/pdf.js/blob/4fe92605b75d7e0952738b7f1575d78145b69aeb/src/display/worker_options.js#L27-L33

Closing as answered/solved based on the above.


Websocket connections fail

Describe the bug
When trying to connect through websockets to a remote url I get a CORS error:

Access to fetch at 'https://socketsbay.com/wss/v2/2/demo/' from origin 'https://node-xgyunu.w.staticblitz.com' has been blocked by CORS policy: No 'Access-Control-Allow-Origin' header is present on the requested resource. If an opaque response serves your needs, set the request's mode to 'no-cors' to fetch the resource with CORS disabled.

StackTrace:
performHTTPCall @ https://node-xgyunu.w.staticblitz.com/blitz.31afbfcfe1b0985e8cbdb27d44e16766166a118e.js:15
_write @ https://node-xgyunu.w.staticblitz.com/blitz.31afbfcfe1b0985e8cbdb27d44e16766166a118e.js:15
writeLatin1String @ https://node-xgyunu.w.staticblitz.com/blitz.31afbfcfe1b0985e8cbdb27d44e16766166a118e.js:15
writeLatin1String @ https://node-xgyunu.w.staticblitz.com/blitz.31afbfcfe1b0985e8cbdb27d44e16766166a118e.js:15
(anonymous) @ https://node-xgyunu.w.staticblitz.com/blitz.31afbfcfe1b0985e8cbdb27d44e16766166a118e.js:6
writeGeneric @ https://node-xgyunu.w.staticblitz.com/blitz.31afbfcfe1b0985e8cbdb27d44e16766166a118e.js:6
Socket._writeGeneric @ https://node-xgyunu.w.staticblitz.com/blitz.31afbfcfe1b0985e8cbdb27d44e16766166a118e.js:6
Socket._write @ https://node-xgyunu.w.staticblitz.com/blitz.31afbfcfe1b0985e8cbdb27d44e16766166a118e.js:6
doWrite @ https://node-xgyunu.w.staticblitz.com/blitz.31afbfcfe1b0985e8cbdb27d44e16766166a118e.js:6
clearBuffer @ https://node-xgyunu.w.staticblitz.com/blitz.31afbfcfe1b0985e8cbdb27d44e16766166a118e.js:6
Writable.uncork @ https://node-xgyunu.w.staticblitz.com/blitz.31afbfcfe1b0985e8cbdb27d44e16766166a118e.js:6
OutgoingMessage._flushOutput @ https://node-xgyunu.w.staticblitz.com/blitz.31afbfcfe1b0985e8cbdb27d44e16766166a118e.js:6
OutgoingMessage._flush @ https://node-xgyunu.w.staticblitz.com/blitz.31afbfcfe1b0985e8cbdb27d44e16766166a118e.js:6
onSocketNT @ https://node-xgyunu.w.staticblitz.com/blitz.31afbfcfe1b0985e8cbdb27d44e16766166a118e.js:6
(anonymous) @ https://node-xgyunu.w.staticblitz.com/blitz.31afbfcfe1b0985e8cbdb27d44e16766166a118e.js:15
(anonymous) @ https://node-xgyunu.w.staticblitz.com/blitz.31afbfcfe1b0985e8cbdb27d44e16766166a118e.js:15
_0x455038 @ https://node-xgyunu.w.staticblitz.com/blitz.31afbfcfe1b0985e8cbdb27d44e16766166a118e.js:15
_0x988000.<computed>.<computed> @ https://node-xgyunu.w.staticblitz.com/blitz.31afbfcfe1b0985e8cbdb27d44e16766166a118e.js:15

The app also crashes with the following error message, instead of gracefully treating the error:
TypeError: Failed to fetch
    at _0x1874c8.performHTTPCall (https://node-xgyunu.w.staticblitz.com/blitz.31afbfcfe1b0985e8cbdb27d44e16766166a118e.js:15:383215)

This request works fine by using devtools to run a normal new Websocket('...') connection.
Additionally, trying to use a raw http socket to create a websocket also doesn't work. The websocket connects, but the server crashes with:
~/projects/node-xgyunu
❯  npm run server
$ node ./server.js
{ port: 9898, address: '::', family: 'IPv6' }
Client connected
TypeError: n.writeAsciiString is not a function
    at https://node-xgyunu.w.staticblitz.com/blitz.31afbfcfe1b0985e8cbdb27d44e16766166a118e.js:6:790399
    at writeGeneric (https://node-xgyunu.w.staticblitz.com/blitz.31afbfcfe1b0985e8cbdb27d44e16766166a118e.js:6:790583)
    at Socket._writeGeneric (https://node-xgyunu.w.staticblitz.com/blitz.31afbfcfe1b0985e8cbdb27d44e16766166a118e.js:6:237560)
    at Socket._write (https://node-xgyunu.w.staticblitz.com/blitz.31afbfcfe1b0985e8cbdb27d44e16766166a118e.js:6:237718)
    at https://node-xgyunu.w.staticblitz.com/blitz.31afbfcfe1b0985e8cbdb27d44e16766166a118e.js:6:878804
    at _write (https://node-xgyunu.w.staticblitz.com/blitz.31afbfcfe1b0985e8cbdb27d44e16766166a118e.js:6:878865)
    at Socket.Writable.write (https://node-xgyunu.w.staticblitz.com/blitz.31afbfcfe1b0985e8cbdb27d44e16766166a118e.js:6:882522)
    at WebSocketRequest.accept (/home/projects/node-xgyunu/node_modules/websocket/lib/WebSocketRequest.js:458:21)
    at WebSocketServer.eval (/home/projects/node-xgyunu/server.js:12:30)
    at WebSocketServer.EventEmitter.emit (https://node-xgyunu.w.staticblitz.com/blitz.31afbfcfe1b0985e8cbdb27d44e16766166a118e.js:6:155505)

on the line:
const connection = request.accept(null, request.origin);

The client connection then hangs.
Link to the blitz that caused the error
https://stackblitz.com/edit/node-xgyunu?file=index.js&terminal=server&terminal=client
Steps to reproduce

open the project
if terminals didn't open, open 1 terminal with npm run server
open second terminal with npm run client
comment out the specific setupWs to test local/remote individually

Expected behavior
Application should run normally and you should see a couple of messages being exchanged back and forth.
If any connection fails, you should see the message:

if everything was working correctly, this message should show up regardless of socket errors

Parity with Local

[X] I have run the project in my local machine and I could not reproduce the issue.

Screenshots
No response
Platform

OS: macOS
Browser: Chrome
Version: 100.0.4896.75 (Official Build) (arm64)

Additional context
No response

Hey there, thanks for reporting this issue. I can confirm that this is not behaving as it should and it's most likely a bug on our side. I have created a ticket in our internal ticket system, and we'll keep you posted on any progress or once it's resolved.

I can't reproduce this...
I managed to link this Stackblitz https://stackblitz.com/edit/nodemon-n6nrt7?file=vite.config.ts to this CodeSandbox
https://codesandbox.io/s/socket-io-vanilla-c2gh56
I just had to configure cors { origin: '*' } on the socket.io server.
This is not related to issue #668

I made an example of a WebSockets relay in StackBlitz... it's not pretty, but it works
https://stackblitz.com/edit/nodemon-xyvji4?file=vite.config.ts

@edusperoni I tried to reproduce your issue but it looks like it works for me. Is this still an issue?

Nope! Thanks!


Cannot reproduce the ideal accuracy of NVAN

Hi,
I reproduce the training procedure and get the results:

baseline: R1=88.54%(+1.12%), mAP=80.71% (2 GPUs)
NVAN: R1=89.55%(-0.45%), mAP=83.05% (3GPUs)
NVAN: R1=89.60%(-0.40%), mAP=82.31% (2GPUs)
STE-NVAN: R1=88.08%(-0.61%), mAO=79.83%(2GPUs)

The training setting and pytorch version is exactly the same as given, the only difference might be that i use several GPUs to train.
The baseline is 1.12% higher than you reported, but NVAN and STE-NVAN is lower. Do you have any ideas on increase NVAN accuracy?

For NVAN, I think the number of GPU may influence the acc (owing to the separate batch norm on each GPU).
For STE-NVAN, you can use the weight after NVAN as initialization and use a smaller lr for fine-tuning.


I'm unable to run this on my repo, please help.

Here is my repo - https://github.com/sahilrajput03/js-object-proxy
I tried everything I can for past 3 hours but it never worked on the github actions.
Here is my github workflow file - https://github.com/sahilrajput03/js-object-proxy/blob/master/.github/workflows/markdown-autodocs.yml
But I can see some error like this in when my actions run -

Please help me.

I resolved the issue now.
The issue was:

README.md was not actually renamed by local git because I renamed it from Readme.md to README.md but it wasn’t detected. So I relsolved the issue by renaming the file to README-temp.md and push to github. Then again rename it to correct name as README.md file.
Other issue was branch name was incorrect as I specified mainbranch in my workflow of auto docs but I was using master branch on my git/github setup.

Thanks.


Metric 8 - # of Births

Creating GitHub ticket to track ZEIR metric 8 - # of births. The complete list and detail can be found here:
https://drive.google.com/open?id=1dOy5H_fy8bYaJFL-Oa53p_rNgR9gZOQi6qpFUE8UHCA
We'll use this ticket to track the status of and issues with the metric as we build it out in Superset.
Current dependency is ETL development.

This metric is built in Superset


fix usage with different name from robot_logger_device

It fixes #37
@Nicogene and I added the possibility to load a mat file with a variable name different from robot_logger_device.

Thank you @marcello-goccia


Give more helpful error when trying to set default values on an interface.

Fixes #5173 - Give more helpful error when trying to set default values on an interface.

Hi @delta-nry, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla.microsoft.com.

TTYL, MSBOT;

Hey @delta-nry - thanks for the PR. I notice that there's no test case affected - can you add a few tests as per  our contribution guidelines?

Thanks for the quick reply @DanielRosenwasser. As I am part of a group of students from the University of British Columbia, I am waiting for my partners to implement the test case(s) you requested. Thus, we may get back to you sometime within two weeks. Thank you for your patience on resolving this PR.

No worries, just keep us posted since GitHub doesn't send updates for new commits.

@delta-nry, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.
Thanks, MSBOT;

I have implemented the required tests. In the interest of full disclosure, I am the sole person who implemented these tests and none of the content of these tests are original works of my partners. Thank you for your patience regarding this issue.

@CyrusNajmabadi can you take a quick look?

@delta-nry, I left some feedback. Basically, you should perform the grammar check in checkGrammarProperty.

@DanielRosenwasser, @CyrusNajmabadi how do these fixes look?

I think this is great - just address the last comments I left and we should be good to merge. :smiley:

:100:


Add Long-form API documentation

Add long form API documentation using FORD: https://cambridge-iccs.github.io/FTorch/

[X] Create GH action
[X] Create GH page under ICCS
[X] Add URL to repo
[X] Run FORD on code
[ ] Add additional md pages
[ ] Update source with comments for FORD where missing
[ ] Move action to rebuild docs for main, not branch

Will close #20

Currently building at https://cambridge-iccs.github.io/FTorch/
Deployment seems to be working so now need to work on docs.

Suggestions for additional pages:

[ ] Dome pleasant documentation for the examples
[ ] Methods of installing libtorch/pytorch and where to find it onn the system etc - can be more verbose than README

It was also suggested that we reduce the README to make it more manageable whilst maintaining the key content there.

According to https://forddocs.readthedocs.io/en/latest/user_guide/writing_documentation.html#non-fortran-source-files we should be able to use FORD for the C code. Not sure how well it works though

Indeed, I am looking at this locally and will push once I have something.

(Mostly a note for myself)
In the GPU section, it might be worth adding something about moving non- parameters/buffers to GPU, as I intend to do in Cambridge-ICCS/FTorch-benchmarks#27.

(Mostly a note for myself)
In the GPU section, it might be worth adding something about moving non- parameters/buffers to GPU, as I intend to do in Cambridge-ICCS/FTorch-benchmarks#27.

I have added a GPU page - feel free to add to it - maybe on a branch if we want to be safe and then merge back into here.

As discussed offline, perhaps we could work towards getting this PR merged and open new issues for the remaining checklist items? The content here is already a really good starting point. I'd be happy to review this PR if that'd be helpful?

As discussed offline, perhaps we could work towards getting this PR merged and open new issues for the remaining checklist items? The content here is already a really good starting point. I'd be happy to review this PR if that'd be helpful?

Yes, please do!

Thanks @jwallwork23 I think those points are all covered in recent commits, including the moving 'Other docs' to 'User Guide'.
Also updated the landing page with latest resources and modified the workflow for main in anticipation of merging.
After casting an eye over I think this is probably now good to go subject to a re-review.

This will also now close #98 as I have removed/simplified large chunks of the README to point at the online documentation.


Delete question for a note

This PR adds a feature to delete a question for a note, backlog item n24416.

Great! Thanks!


⚠ Bofrost.lu has degraded performance

In 9db9a34, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 864 ms

Resolved: Bofrost.lu performance has improved in 1e1f9de after 8 minutes.


Update sbt-release to 1.0.12

Updates com.github.gseitz:sbt-release from 1.0.11 to 1.0.12.
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.
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 = "com.github.gseitz", artifactId = "sbt-release" } ]

labels: semver-patch

Pull Request Test Coverage Report for Build 90

0 of 0   changed or added relevant lines in 0 files are covered.
3 unchanged lines in 1 file lost coverage.
Overall coverage remained the same at 75.274%

Files with Coverage Reduction
New Missed Lines
%

scassandra/src/main/scala/com/evolutiongaming/scassandra/CreateClusterJ.scala
3
44.44%

Totals

Change from base Build 89:
0.0%

Covered Lines:
344

Relevant Lines:
457

💛  - Coveralls


Cannot upload files

I cannot upload files to https://github.com/IBM/qfa which I created.

It says "Uploads are disabled."


feat!: improve hooks parsing and escaping

Currently, we use Bash to interpret tmux commands through eval tmux <commands>. This method is generally unsafe and may introduce mental overhead due to syntax differences between Bash and tmux (e.g., a session name like $aaa may be expanded as a variable in Bash). This PR introduces a new parser for a more robust solution.
Internally, we use xargs to parse arguments into sequences and printf %q to escape each one. This allows users to input ; directly as the command delimiter without worrying about Bash's interpretation. For the precisely supported syntax, please refer to the updated README.
Additionally, this PR adds the missing parse process for the --on-init, --before-open, and --after-close arguments in @popup-toggle. @cenk1cenk2, the tmux_escape(...) function may no longer be needed in your plugin, and you can directly pass user-defined hooks to @popup-toggle :P
BREAKING CHANGE: The new parser may produce results that differ from the previous version.BREAKING CHANGE: The new parser may produce results that differ from the previous version.

I have given this branch a go, and it still works a charm, I suppose the only thing that needs to be gotten rid of is escaping the semicolons, thanks for giving me a heads up.

I just think that https://github.com/loichyan/tmux-toggle-popup/issues/10 this issue also got exaggerated here, where it creates many terminals in the background.
However, I am still not sure that where it is coming from the initial look.

I just think that #10 this issue also got exaggerated here, where it creates many terminals in the background.

I've just merged #11 into this branch. Are you still experiencing this issue?

I've just merged #11 into this branch. Are you still experiencing this issue?

It seems that the issue has not yet been resolved. I'll investigate it further.

I just think that #10 this issue also got exaggerated here, where it creates many terminals in the background.

This should be fixed now :D. The new parser will return nothing if a hook is empty, and running tmux without any commands will start a dummy session. I never caught this because it was unconsciously fixed in #9 lol. Thanks for reporting though!

Thank you for fixing it, I can confirm that with this branch that is no longer a problem.


libregraph: allow provisioning users with legacy usernames

When provisioning users we may encounter usernames like max.    mustermann. Yes that is multiple blanks. And users actually log in with that at the IDP. By moving to a uuid based identity property the username can be changed, allowing cleanup of these bad usernames.
We need a test case for this kind of legacy username. I'm so sorry.

@butonic @rhafer  Since this is already for the use-case of cleaning up things later, how about changing the username with URL encoding if it fails the name test? If we do that, it's also easy to identify the ones that are like this for the cleanup.
WDYT?


IE8 still not work in v1.0.1 :/

Thank u so much for looking into this,but it seems still not work  :-(
----v1.0.1  existing list----
http://codepen.io/NewGuyUseGit/pen/zrmaA
http://codepen.io/NewGuyUseGit/full/zrmaA
----v1.0.2  fuzzy search----
http://codepen.io/NewGuyUseGit/pen/wzajt
http://codepen.io/NewGuyUseGit/full/wzajt

I have the same error at line 723 (val.valueOf()) with IE8. I tried downloading directly from the link provided above but it failed as well. I'm using list.js on Safari, Chrome and Firefox without problem. However I need to support IE8 as well... Any clue on this issue would be greatly appreciated.
Thank you!


Blender-like shortcuts for moving, scaling and rotating Meshes in 3D

Would love to see some Blender like shortcuts for manipulating objects in 3D space in Godot. But only for Scale, Move, Rotate (same as Gizmos)
Keys
G = Grab/Move
S = Scale
R = Rotate
ESC = exit move/scale/rotate mode and cancel changes
left mouse click or ENTER = accept changes
mouse pos = freeform move/scale/rotate
KEY <optional: AXIS> <optional: UNITS>
Of course these should also be remap-able in shortcuts.
These would be different than Q  Tool Select, W  Tool Move, E Tool Rotate, R Tool Scale
The above shortcuts can be changed to keys 1 Tool Select 2 Tool Move 3 Tool Rotate 4 Tool Scale or R Rotate can be moved to Shift+R instead. Either way
For Example
press G to begin grab/move mode (freeform grab/move based on mouse pos)
press ESC to escape grab/move mode and cancel changes
click L mouse or press ENTER to accept
press R to begin rotate (freeform rotate in any direction based on mouse pos)
press ESC to escape rotate and cancel all changes
click L mouse or press ENTER to accept
press G X  to grab/move in X axis only (grab/move based on mouse pos)
click L mouse or press ENTER to accept
press G Y -2 to grab/move -2 units in Y axis
click L mouse or press ENTER to accept (however if you move the mouse it reverts to G Y and mouse pos )
press S X 1.5 to scale by  x 1.5 in X axis
click L mouse or press ENTER to accept (if you move the mouse reverts to S X and mouse pos)
press R Y 45 to rotate about Y axis 45 degrees
click L mouse or press ENTER to accept (if you move the mouse reverts to R Y and mouse pos)
Actions change dynamically as you type
R Y 45 <backspace erase
R Y 4
RY 49

Similar to #3995. I would also really appreciate this. After getting used to Blender's way of manipulating objects, it's hard/mildly annoying to go back to using gizmos for everything. It just feels like such a faster and more seamless workflow manipulation objects like Blender does, compared to using gizmos all the time.

Duplicate of #3995 indeed :slightly_smiling_face:


Play sound from ISM rendered entities

[ ] Go with the approach here: https://github.com/HaywireInteractive/OnAllFronts/commit/c2d683152e06828350e4db32bd6db80f41999d2d. Create a separate PR for continuous sounds and for voice lines.
[x] For calculating if a sound is close enough to any viewer to hear, there is FMassViewerInfoFragment.ClosestViewerDistanceSq. But for entities which use UMassScreensizeLODCollectorTrait such as soldiers and tanks, we set ClosestViewerDistanceSq to an inverse screen size value instead of a closest viewer distance. Since we need the actual closest viewer distance, we should update TMassScreensizeLODCollector::CollectLODInfo to also calculate and store a real ClosestViewerDistanceSq in a new fragment called FMassScreensizeLODViewerInfoFragment. Then where ever we need to use the real ClosestViewerDistanceSq we should check if the entity has FMassScreensizeLODViewerInfoFragment then use FMassScreensizeLODViewerInfoFragment.ClosestViewerDistanceSq, otherwise use FMassViewerInfoFragment.ClosestViewerDistanceSq.

~Use same approach as https://github.com/HaywireInteractive/OnAllFronts-Public/issues/524 where we have an audio-only version of the actor that we use in this case. Use a shared component so we don't need to duplicate voice line logic between Human_Character and voice only actor.~
~We should check "Keep Low Res Actor" in visualization trait to have both ISM and voice only actor enabled.~
~One other way we could implement this:
Instead of having an actor that plays the sounds (which doesn't help us if they're ISM), maybe we create a new trait UMassAudioPlayerTrait which has properties to configure at what maximum distance we should play sounds from this entity. Then to play sounds we always go through some helper method which checks the closest viewer distance compared to that maximum distance and if close enough, plays a sound.~
~Note because of https://github.com/HaywireInteractive/OnAllFronts-Public/issues/686 we should maybe rearchitect how we play sounds so that they can be played for ISMs as well. Maybe they shouldn't go through our MassNotificationComponent at all. But we need some way to make them only play near players. Maybe we need a separate entity. Please discuss your approach with Leroy before implementing.~

Currently blocked by Character_MassNotification lock.

DevNote: #776 Should be closed with this Issue.

DevNote: 726 has potential logic that can conflict with this PR.


How to add new code (I'm new here!)

Basically I wanna buy summer shades item for a set price of 15.33. I can't always be around and don't have enough time to learn this c#/ I understand that i need to work out the value in scrap but how much of these codes and files do i need... I just want it to do one thing and that's buy!
If somebody could please pm me and guide me through this it would be great! thanks!

#427 #527

Thankyou


Holograms randomly dissapear

So I've had for two months now this plugin installed and here and there I would casually get a hologram that dissapeared, but I recently updated the Hologram to V 2.9.1 and have restarted my server a few time here and there but the last restart removed 95% of my holograms and they refuse to be put back, what could be the issue of dissapearing Holograms?

same for me in 1.8.9 spigot version and 2.9.1 holograms

Same for me, I use Multiverse and every time I come back to the place where the Holograms are, they just dissapear...


Mocha isn't as dark as the preview

This is how the preview looks

And this is how it looks in my case

The preview seems to be darker.

Hey :wave:,
Are you using the files from the latest release (v2.0.1) of the theme? It's worth noting that I'd consider your screenshots to actually align more with Catppuccin with base taking up more of the background and mantle taking up just the search box.
I made the the decision to slightly go against our style guide for consistency reasons in the underlying framework. I wonder if these differences are just from the fact that my screenshots were taken on Linux and you seem to be running Windows?
I don't have a Windows machine at the moment ready to test this unfortunately. @arminius-smh do you happen run Windows to test this out?

do you happen run Windows to test this out?

Suure, but unfortunately I can't reproduce it, in the image I am using the mocha theme from the latest release: https://github.com/catppuccin/qbittorrent/releases/download/v2.0.1/catppuccin-mocha.qbtheme, which is consistent with how it looks on my linux machine
I've noticed Windows has this extra 'style' setting in the options that doesn't exist in linux, I've tried all the options in there but also couldn't reproduce it with any of those, so that doesn't seem to effect.
Still, maybe try going through these options and see if it maybe fixes it @FrostedVolcano?
You could also check if Windows Dark/Light mode and these settings are interfering with something? I remember that qbit uses the windows colors/color accents to style the qt elements in the newer releases, maybe something is meddling there with qt?

It's a bit odd that the wrong colors in your image are also catppuccin colors, so I don't really know why it would look any different from my setup or has something to do with the window settings, but might be worth a try, otherwise I currently have no other ideas

Hi @sgoudham and @arminius-smh,
Thanks for responding and I want to apologize for my late response. I was in fact not using the latest release, I was using the first release.
What happened was I had downloaded some .qbtheme files from various places a long time ago to try out. I would use one theme for some time and then change to another. Now, I had the mocha.qbtheme from release 1.0.0. After reading your comments I realized that I should be using the latest one, and after doing so now the interface colors are similar to the preview.
Thanks a lot for you help.

No worries, thank you for following up!


KAFKA-17990: Flaky test improvements

Several of the ShareConsumerTest integration tests are a bit flaky. This PR tightens up the logic with the aim of eliminating the flakes. Annoyingly the tests seem rock solid locally so this might take some experimentation.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

We recently changed the default starting position for consumption from earliest to latest to match the KIP, which necessitated alter configs for the tests to "earliest". The point at which this takes effect is non-deterministic and I believe that's responsible for the increased flakiness.


Featurte request: Line numbers etc.

Optional metadata like source file attributes, line numbers and annotations are currently not translated.

All of this would be really useful for debugging/reverse engineering apps... Would it be hard to add?

I'm not sure. You could try to add it and see.

Have you tried this PR? https://github.com/Storyyeller/enjarify/pull/5


EnvironmentPostProcessor implementations are not invoked at build-time

Conditional beans whose properties are defined by an EnvironmentPostProcessor are not available.
For example, if I create a EnvironmentPostProcessor that adds a PropertySource containing the property foo:
public class CustomEnvPostProcessor implements EnvironmentPostProcessor {
    @Override
    public void postProcessEnvironment(ConfigurableEnvironment environment, SpringApplication application) {
        environment.getPropertySources().addLast(new MapPropertySource("example", Map.of("foo", "bar")));
    }
}

Beans dependent on this property such as:
@Bean
@ConditionalOnProperty(name="foo")
Bar bar() {
  return new Bar();
}

Will not be available, and will throw a NoSuchBeanDefinitionException exception at runtime.
Injecting the property with a @Value annotation seems to work without issue.
Example repo demonstrating the problem:
https://github.com/bdemers/spring-native-aot-problem
NOTE: The example above works when the native plugins are disabled

Thanks for the sample. AotApplicationContextFactory does not invoke EnvPP when preparing the environment at build-time. I don't remember why that is or what was the constraints. Do you @bclozel?

@snicoll I don't know if we wanted to start with something minimal there and never got to this use case, or if there were concerns about EnvPP instances doing "clever" things that could be incompatible with build time processing.
In any case I think we can try and fix this issue.

We've been discussing this one and it's a tough call at this stage. Our preparation of the environment is pretty basic as we don't really have a way to distinguish build-time vs. runtime processing of the environment. Until that's the case, I think it's wiser to move this one to backlog.

Are there any updates on this? As things currently stand, the Okta Spring Boot starter does not work in native mode.

@mraible It's in the backlog so I am afraid we have no plan to fix this in Spring Native itself. We'll pick this up in Spring Boot 3.

Spring Native is now superseded by Spring Boot 3 official native support, see the related reference documentation for more details.
As a consequence, I am closing this issue, and recommend trying your use case with latest Spring Boot 3 version. If you still experience the issue reported here, please open an issue directly on the related Spring project (Spring Framework, Data, Security, Boot, Cloud, etc.) with a reproducer.
Thanks for your contribution on the experimental Spring Native project, we hope you will enjoy the official native support introduced by Spring Boot 3.


create some hooks

Hi,
I think your package would be more useful if you added some event hooks to it.
onDocumentLoad, onDocumentError etc.
Thanks!

Do you know how to do that? because I am not able to catch any error with the mozilla library https://github.com/javascriptiscoolpl/npm-simple-react-pdf/pull/3. The try and catch also not works. The mozilla library is very very wrongly written.


Scrapoxy with own proxy list?

I read all the documentation of scraproxy. I have an idea in my mind and km not sure if I can do it with this library.
I don't want to pay for services such as ec2.. I would like to scrape the internet for proxies such as https://www.sslproxies.org and add proxies from there to scrapoxy for it to manage my proxies and check then fron now and then.
I'm not sure how do I add custom proxies not via  popular suppliers via the rest api. And where will scrapoxy save the custom proxies (if for example I'll need to restart server or something).
Thanks in advanced.

@dinbrca The purpose of scrapoxy is to run your own proxies using cloud servers. What you see on https://www.sslproxies.org are proxies run by other people.

I don't see any reason why scrapoxy shouldn't be able to do that..
If I'm scraping I need a lot of IP's on EC2 in-order to not get blocked, I can easily do that if I use Scrapoxy to validate and ping proxies I find on the internet.
Why must I have my own cloud when the whole internet provides a lot of proxies that I could be adding to Scrapoxy.
For me the "Create your own proxies" statement was a bit blurred:


Provide options --cache/--no-cache and --cache-file

currently the CSV data is always downloaded and written into the file coiv19.csv
provide a boolean flag --cache/--no-cache where the default should be --no-cache
provide --cache-file where the default should be "covid19.csv" being the path and name of file for the downloaded CSV data
log the options

Done.


Typed querySelector with Strict Mode

Introduction
Recently I've refactored and enhanced typed-query-selector module, together with a new feature called "Strict Mode".
The strict mode is implemented but not released at the time I create issue, but I'm planning to release v2.3.0 which includes this feature.
Details of strict mode can be found at readme but in a short word it performs a simple selector syntax checking and if selector syntax is invalid it will return never type. This feature won't be enabled by default and can be opted-in.
Benchmark
To be able to detect syntax errors of selector string, the selector parser has become complex, so I've run benchmark.
 However, memory usage is increased. For this project, it increased about 8 to 9 MiB.
Running type checking with strict mode will spend more time than non-strict mode.

Questions

How about this new feature?
Are there anything can be improved?

That sounds interesting but this project uses the package through select-dom, which probably can’t be strict. Maybe after #3855 we can use it directly.

That sounds interesting but this project uses the package through select-dom, which probably can’t be strict. Maybe after #3855 we can use it directly.

Why cannot select-dom be strict?

Why cannot select-dom be strict?

I don’t know what this change entails exactly, but if I make it strict, suddenly any breaking issues/incompatibility with typed-query-selector suddenly become my issues. The parser isn’t fully compliant so I can’t make it a subdependency.

I don’t know what this change entails exactly, but if I make it strict, suddenly any breaking issues/incompatibility with typed-query-selector suddenly become my issues. The parser isn’t fully compliant so I can’t make it a subdependency.

As select-dom being a library, switching to strict mode will be a breaking change for TypeScript users, and that's why strict mode in typed-query-selector won't be enabled by default. (but not affect JavaScript users because typed-query-selector only works at type level and there're no runtime code)
The reason why it's a breaking change is that if users pass invalid selector string, it will return never type, which may break their TypeScript type checking, but for valid selector string, everything works as before.

As select-dom being a library, switching to strict mode will be a breaking change for TypeScript users, and that's why strict mode in typed-query-selector won't be enabled by default. (but not affect JavaScript users because typed-query-selector only works at type level and there're no runtime code)
The reason why it's a breaking change is that if users pass invalid selector string, it will return never type, which may break their TypeScript type checking, but for valid selector string, everything works as before.


Add severity text to widgets

Split to severity with localized text of severity
accident_count_by_accident_year (severity of accidents)
injured_count_by_accident_year (severity of injured)
@MichalOren FYI

Hi @atalyaalon ,
The current data should be replaced by a dict with severity levels as keys and values the relevant items from current data?

@ziv17 what do you think?
Here we had for example text separated from count- perhaps we should keep it as better practice? Not having translations as keys - since then do need to differentiate them (killed should be in red - and the FE needs to know how many killed using the key - in any language

Also here: https://github.com/hasadna/anyway/blob/dev/static/data/widgets/accident_severity_by_cross_location.json
We will soon need to decide on templates we want to use - and we'll need to adjust best practices.
What do you think?

Hi @atalyaalon ,
We want two levels of grouping? The severity should be the outer level or the inner?

@ziv17 what do you think?
Here we had for example text separated from count- perhaps we should keep it as better practice? Not having translations as keys - since then do need to differentiate them (killed should be in red - and the FE needs to know how many killed using the key - in any language

Hi @atalyaalon , I don't understand

Also here: https://github.com/hasadna/anyway/blob/dev/static/data/widgets/accident_severity_by_cross_location.json
We will soon need to decide on templates we want to use - and we'll need to adjust best practices.
What do you think?

Here too, I do not understand.

@ziv17
I suggest the following structure no.3 here: https://github.com/hasadna/anyway/blob/dev/docs/WIDGETS.md
Meaning we'll have this for accident_count_by_accident_year (and create a similar one to injured_count_by_accident_year):
{
    "name": accident_count_by_accident_year,
    "data": {
                "items": {
                            "label1":
                                    [
                                        {
                                            "category1_name": 2017,
                                            "category1_value": 3
                                        },
                                        {
                                            "category2_name": 2018,
                                            "category2_value": 2

                                        {
                                            "category3_name": 2019,
                                            "category3_value": 2
                                        }
                                    ],
                            "label2":
                                    [
                                        {
                                            "category1_name": 2017
                                            "category1_value": 10
                                        },
                                        {
                                            "category2_name": 2018,
                                            "category2_value": 15
                                        },
                                        {
                                            "category2_name": 2019,
                                            "category2_value": 15
                                        }
                                    ],
                            "label3":
                                    [
                                        {
                                            "category1_name": 2017
                                            "category1_value": 40
                                        },
                                        {
                                            "category2_name": 2018,
                                            "category2_value": 55
                                        },
                                        {
                                            "category2_name": 2019,
                                            "category2_value": 34
                                        }
                                    ],

                        },
              "text": {
                          "title": "כמות התאונות לפי שנה במקטע כניסה למצפה שלם - צומת שדי תרומות"
                           "labels": {
                                            "label1": "קטלנית",
                                            "label2": "קשה",
                                            "label3": "קלה"
                                      }
                        }
            }
    "meta": {
        "rank": 8,
        "information": ""
        type: "stacked_bar"
    }
}

Also on the way - please add information for each of them (localized of course)
Let me know if I missed anything.
Thanks!

@ziv17 also - when count of accidents/injured in specific year and severity is 0 - we should still display the category.
Thanks!

@atalyaalon OK. Why are we naming the first level label, and the second level category? Both are attributes of accident.

@ziv17 what do you think?
Here we had for example text separated from count- perhaps we should keep it as better practice? Not having translations as keys - since then do need to differentiate them (killed should be in red - and the FE needs to know how many killed using the key - in any language

@atalyaalon  I think it is a good idea to not use translated text as labels, and not use it in any logic.
It will be even better not to use the English string, but a code, or Enum, and only translate it just before it is sent outside.

@ziv17 sounds good! Go for it.

Hi @atalyaalon , regarding the labels part.

Can seriese<i> be the actual code, i.e. actual accident severity code?
I understand that value is the translated name of the accident severity. What is name? Is it the English name of the accident severity?

Hi @atalyaalon ,
in anyway/infographics_utils.py:1621, there is a filter for injury severity. It seems to use non-existing values, and looks like it can be replaced by testing for isnot(None).
I will implement it as part of this issue. Just take a brief look check whether my assumptions are correct.

Hi @atalyaalon , regarding the labels part.

Can seriese<i> be the actual code, i.e. actual accident severity code?

I don't see any value in using the actual severity code.
I think that ideally this template should be used for any new stacked bar we create - hence we can either:

Keep a generic name for the series keys (series1, series2, etc) (I think that this option is better)
Replace them with the series value localized (same for category1 with the name) - in that case we need to make sure that the FE can receive from the BE, the colors,icons etc - for example - so they can differ the categories no matter what language we're in.
Perhaps you have a better suggestion?

You can also consult with the FE team regarding this template (I suggest starting with Sergey)

I understand that value is the translated name of the accident severity. What is name? Is it the English name of the accident severity?

The goal of having the name is, for example, make sure that the FE team can tell the difference between fatal and severe no mater what language we're in (for example, differ colors, icons, etc)

Hi @atalyaalon , regarding the labels part.

Can series<i> be the actual code, i.e. actual accident severity code?

I don't see any value in using the actual severity code.
I think that ideally this template should be used for any new stacked bar we create - hence we can either:
* Keep a generic name for the series keys (series1, series2, etc) (I think that this option is better)

* Replace them with the series value localized (same for category1 with the name) - in that case we need to make sure that the FE can receive from the BE, the colors,icons etc - for example - so they can differ the categories no matter what language we're in.

I thought we want to avoid using localized values as keys. In addition, in this case, The labels will have redundant data, ,and should change to {value1: name1, value2: name2, ...

* Perhaps you have a better suggestion?

The series<i> values appear twice in the widget output, and its sole function is to link the name and value from the labels to the outer level items dict keys. So as long as these values are unique, they fulfill their goal. So I would use the entity codes as it is minimal, simple and. I do not see what value generic names brings (Maybe I do not understand what do you mean).

You can also consult with the FE team regarding this template (I suggest starting with Sergey)

I understand that value is the translated name of the accident severity. What is name? Is it the English name of the accident severity?

The goal of having the name is, for example, make sure that the FE team can tell the difference between fatal and severe no mater what language we're in (for example, differ colors, icons, etc)

@ziv17 as we discussed - what ever you think is best :)

Closed by PR 1875


Decoding an empty string fails with "deflate" and "br"

Steps to reproduce the problem:
In [15]: from netlib import encoding

In [16]: encoding.decode('', 'deflate')
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
<ipython-input-16-074267748746> in <module>()
----> 1 encoding.decode('', 'deflate')

/home/ames/dev/mitmproxy/netlib/encoding.pyc in decode(encoded, encoding, errors)
     54             type(e).__name__,
     55             repr(encoded)[:10],
---> 56             repr(encoding),
     57         ))
     58

ValueError: error when decoding '' with 'deflate'

What is the expected behavior?
Return an empty string. gzip works as expected:
In [18]: encoding.decode('', 'gzip')
Out[18]: ''

What went wrong?
deflate and br are unable to decode empty strings.
Any other comments? What have you tried so far?
Not sure what the preferred way of solving this is. Possibly a check to return an empty string if an empty string is attempted to be decoded?

Mitmproxy Version: master (0.18)
Operating System: Ubuntu 16.04

@Schamper If this happens only for empty strings, then yeah we could just return an empty string back.

Actually I think brotlipy can handle empty strings just fine (at least the latest version 0.5.1):
>>> import brotli
>>> brotli.decompress(b"")
b''

Ah yes, the problem is with empty strings. Empty byte arrays work just fine.
Did you experience this problem somewhere while running mitmproxy, or just by using the mitmproxy/netlib API?

It happened at least once while using it normally (which caused me to investigate). I don't have a URL or anything for you to replicate though, sorry.


Side Nav Redesign

Description

This PR adds new navigation changes to Flow Framework Dashboards
Testing Recording 1 -b10ccf87-958a-4894-9f10-f505a4969701.webm

Testing Recording with MDS feature merged  - latest
screen-capture (6).webm
Issues Resolved
Related Issue: https://github.com/opensearch-project/OpenSearch-Dashboards/issues/7029
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.
[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

b10ccf87-958a-4894-9f10-f505a4969701.webm

@SuZhou-Joe, Could you please review this PR? This PR only includes the new side nav changes. I will raise followup PR for header redesign.

LGTM UX-wise! Thank you!


Add Polish localization - Home page

This is a part of the initial Polish localization - see: https://github.com/kubernetes/website/pull/18419

/milestone 1.17-pl.1

/assign @daminisatya

/hold
pending approval & merge for #18419. Once that merges, it's OK to /unhold
People can /lgtm (for localization review) and /approve, even whilst this is held.

@nvtkaszpir: to mark this as reviewed and acceptable, comment with /lgtm on a line by itself

/lgtm

/unassign @daminisatya
(this will be taken care of by the Polish localization team once the initial PR merges)

Thanks @mfilocha & @nvtkaszpir
/approve
(Once the initial localization lands, there will be a set of approvers for Polish who can self-manage the localization)

Approved based on @sftim being an sig-docs approver (however, the dev-1.17-pl.1 branch is a bit behind master)

/hold cancel

/language pl


Async Scope Computation

At Jayvee, we were evaluating whether we could implement dynamic document loading on imports (https://github.com/jvalue/jayvee/issues/593).
However, we ran into a roadblock as the ScopeProvider has a synchronous interface for the getScope method where we ultimately would need to await a promise the loads a document via services.shared.workspace.LangiumDocuments.getOrCreateDocument(documentUri).
If there is a better way to do it in a different place, I'd appreciate your expertise :)
Otherwise, would it be possible to make the interface of the ScopeProvider asynchronous by returning promises?
/**
 * Language-specific service for determining the scope of target elements visible in a specific cross-reference context.
 */
export interface ScopeProvider {

    /**
     * Return a scope describing what elements are visible for the given AST node and cross-reference
     * identifier.
     *
     * @param context Information about the reference for which a scope is requested.
     */
    getScope(context: ReferenceInfo): Promise<Scope>;

}

This would also affect some methods of the Linker and its clients.
Let me know if I can contribute anything to that change!

Hey @georg-schwarz,
The decision to make the getScope method sync was very deliberate, as otherwise every reference resolution, i.e. every call to ref becomes async.
There are other ways of solving this, see https://github.com/eclipse-langium/langium/discussions/1308. The main idea is to hook into the document builder and to load referenced documents before performing the linking.


add support to skip empty string

Fixes #244

You also need to update index.d.ts. That one and the readme should be in sync. I would also like to see more tests.


feat: add listing the nicommons IDs of the installed packages

Description

Add listing the nicommons IDs of the installed packages.

Related Issue

close #271
Motivation and Context

Refer to #271
How Has This Been Tested?

I will add nicommons ID into the data.
I tested this feature using team-apm/apm-data#190.

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

Great UI, very easy to read. The code is also basically fine.
There are a few things I would check. modList.ts has an existing bug that can be fixed as follows.
async function getInfo() {
  const modFile = await ipcRenderer.invoke('exists-temp-file', 'list.json');
  if (modFile.exists) {
    try {
      return await parseJson.getMod(modFile.path);
    } catch {
      return null;
    }
  } else {
-    return null;
+   await downloadData();
+   const downloadedModFile = await ipcRenderer.invoke('exists-temp-file', 'list.json');
+   try {
+     return await parseJson.getMod(downloadedModFile.path);
+   } catch {
+     return null;
+   }
  }
}

If I make this modification to PR and then launch apm from the yarn package with a clean install, it does not seem to work. However, the one with this modification on the main branch does work with a clean install. I do not know why.

The cause seems to be ClipboardJS.

I fixed them.
I want to merge this PR after merging #570.

I rebased.


update versions in pom.xml

Codecov Report

Merging #959 into develop will decrease coverage by 0.03%.
The diff coverage is 100%.

@@              Coverage Diff              @@
##             develop     #959      +/-   ##
=============================================
- Coverage      41.74%   41.71%   -0.04%
+ Complexity       774      773       -1
=============================================
  Files            153      153
  Lines           5534     5531       -3
  Branches         781      780       -1
=============================================
- Hits            2310     2307       -3
  Misses          3055     3055
  Partials         169      169

Impacted Files
Coverage Δ
Complexity Δ

...java/com/microsoft/azure/auth/AzureAuthHelper.java
79.26% <100%> (-0.74%)
34 <0> (-1)

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 cc6468a...2d39723. Read the comment docs.

The fix is at https://github.com/Azure/autorest-clientruntime-for-java/commit/d5b30ff98995ca76ce3e8754e5d6bcfbf868b34e#diff-02989d3b8308ec4591c357a4b94fc537


Not all alarm_control_panel entities shown

hi! I'm struggling to have all my alarm_control_panel entities get listed in the device control list. I'm running the latest version of hikaxpro_hacs (2.1.5), home assist and HACS.
below is a screenshot of the Hikvison pro control unit:

As you can see there are 7 devices (movement sensors) connected, but only 5 are listed in the "controls". I've searched everywhere within Home Assist, and all entities of the 2 sensors seem to be listed, except the alarm_control_panel entities. Maybe someone here has a suggestion of what it could be? Below 2 screenshots of the sensors that, as far as I know, are exactly the same.

Controls are "zones" of your "AXPro".
Under it you have connected devices. Devices != zones.
The terminology is important.
You will not see every device as zone because zone can have multiple devices. You can configure this in your hikvision app or in Web interface of "AXPro".

ok thanks for the reply @petrleocompel !
Check, I'm mixing up terminology. But I've configured the 2 living room sensors to be in the living room zone, and from the Hikvision app I can control them. But the alarm control entities for this area/zone don't show up in Hikvision, while they do for all others. Sorry to bother you this way, but I'm quite puzzled and stuck with this now.

You are still looking on "alarm_control_panel" aka "Zones" and you are looking on Entity tabs...
Just look on "devices" filter integration from "hikvision ax-pro".

hey, thanks again.
I guess that is the point I try to make. The 2 living room sensors do show up in the device list, but I somehow can't make the link to arm them

??? Living room sensor A and B ? are there.. 🤔
What sensors ?  what kind of sensors?

motion and temperature sensors. Somehow the arm status and the temperature are picked up, but I cant arm them.

If there is motion in front of the motion sensor you will not be able to arm them.
You can check the hikvision app why you cannot arm the zone it would tell you if you press "diagnose button".
I am trying my best to understand but. You cannot "arm them"?

I can the "living room" from my Hickvision app, so that works. I can't arm this zone/region in HA because arming the living room is not show in the dashboard.

You have there office twice and one entity ID is living room second one office.
Still think it is your naming of problem.

aha! YES! That's the problem, somewhere I mixed that up.
Thanks! sorry again for bothering you with this one, next time I'll report a real bug! :)

hey, just to clarify my confusion. I've been struggling with this a bit longer, and it looks like I had 2 zones of 2 different alarm systems wrongly mixed up in home assistant. This explains why renaming certain devices had strange like renaming other devices also. After deleting and re-adding all devices everything worked as expected.
Thanks again for your time thinning with me.


Headless interaction

Added a command-line interface which allows to do the graph database population and FHIR export without invoking the UI.

This implements #7


Add initial support for user deletion request

Progress on https://github.com/NuGet/Engineering/issues/921.
The changes were getting big so I stopped and made a PR. Hopefully easier to review.
This PR:

Update the "report my package" form to support a new hidden drop-down (delete decision) and confirmation checkbox.
Add new notification message so that all owners know a version was deleted.
Add method to IPackageDeleteService to determine if a package can be deleted. It always returns false right now.
Add logic to soft delete the package and auto-close the support request.

TODO:

Implement "can I delete this?" logic.
Introduce configuration for the aforementioned logic.
Functional test (hopefully)

Without these TODOs, the new code paths introduced by this PR will not be available yet.

can you attach screenshots?

Select a reason

After choosing a delete reason

After choosing the delete decision

After choosing the contact support decision

After deleted

@scottbommarito, I will address these comments in a follow-up PR.


Update quick-create-java.md

Line number 228 should end with a semi-colon that is missing. Added semi-colon after import statement.

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

#label:"aq-pr-triaged"

Nice catch, @ivaturia! Thank you for your contribution. #sign-off


Build

footer; side-nav fixed
projects data added

no conflicts were detected between build and main - merging the lastest changes about footer, project data, and minor improvements


GitHub is now our primary repository hosting environment

Now that we have switched to Git & GitHub as our primary VCS and repository hosting environment. This updates the community and installation pages to make GitHub the primary location for our repositories and our internally hosted Mercurial repositories to be the mirrors.
A second patch removes all mentions Mercurial.

I will add a separate commit that just removes mention Mercurial, then it can be applied or not, or applied and easily reverted...

Rebased with master.

Added Reviewed-by: tags.
$ git range-diff 50bb207...f42c65f
1:  cdc5041 ! 1:  5d0ae53 GitHub is now our primary repository hosting environment
    @@ Commit message
         mirrors.

         Closes: https://github.com/nginx/unit-docs/issues/70
    +    Reviewed-by: Andrei Zeliankou<EMAIL_ADDRESS>    +    Reviewed-by: Timo Stark<EMAIL_ADDRESS>         Signed-off-by: Andrew Clayton<EMAIL_ADDRESS>
      ## source/community.rst ##
2:  50bb207 ! 2:  f42c65f Remove all mentions of Mercurial repositories
    @@ Commit message

         With having moved to GitHub, we don't intend to keep these around.

    +    Reviewed-by: Timo Stark<EMAIL_ADDRESS>         Signed-off-by: Andrew Clayton<EMAIL_ADDRESS>
      ## source/community.rst ##


Fix buffer overrun issue with MySQL

Vec::reserve ensures that self.capacity() >= self.len() + additional, not self.capacity() >= original_capacity + additional. If
we don't set the length before this call, we aren't necessarily actually
allocating to be the size we want to be, and risk a buffer overrun.

Somewhat humorously, this bug was unlikely to ever cause a segfault. As long as the allocator is laying things out as compactly as possible, there will always be allocated memory in the address after the end of the vector by the time the overrun could occur.


"... run Node.js, entirely inside your browser"

Describe the bug
This article https://blog.stackblitz.com/posts/introducing-webcontainers/ claims

run Node.js, entirely inside your browser

Is that claim true and correct?
Is there an example of the code required to do that locally?
Link to the blitz that caused the error
https://stackblitz.com/edit/node-uccx6z
Steps to reproduce
Try to reproduce this claim locally, offline:

run Node.js, entirely inside your browser

Expected behavior

run Node.js, entirely inside your browser

Parity with Local

[X] I have run the project in my local machine and I could not reproduce the issue.

Screenshots
No response
Platform

OS: [e.g. macOS, Windows, Linux]
Browser: [e.g. Chrome, Safari, Firefox]
Version: [e.g. 91.1]

Additional context
No clear, step by step exaple of running WebContainers ("run Node.js, entirely inside your browser") locally.

@guest271314 Thanks for the report.
I'm not exactly sure what is the scope of your question. The blog post you linked and the docs in this repo go into some detail of what WebContainer is, and what it can do. The project you linked above does show those capabilities. If you have a more concrete question, we'd be happy to help.

I couldn't detrmine if the linked stackblitz is working asc excpected. The goal is to run Node.js in the browser - outside of stacckblitz. How to do that?

It is working as expected. If you type node index.js in the embedded terminal, the code will be executed as in a Node environment.

outside of stacckblitz

Not sure if you're asking how to use this outside of https://stackblitz.com, but the answer is that this is not possible, WebContainer is part of our online IDE.

Not sure if you're asking how to use this outside of https://stackblitz.com/, but the answer is that this is not possible, WebContainer is part of our online IDE.

Yes, that is the question.
The advertised claim does not include that qualification.

node-v17.8.0-linux-x64/bin/node into N discrete files here https://github.com/guest271314/node_executable.
To reconstruct the executable, pertinent part
while (true) {
    readable = await fetch(`${path}node_${n++}.txt`)
      .then((r) => r.body)
      .catch((e) => false);
    if (readable === false) {
      break;
    }
    await readable.pipeTo(writable, { preventClose: true });
  }
  await writable.close();

Now I can dynamically fetch and run node in the browser utilizing Native Messaging, without WebAssembly. Then delete the file when done with the requirement.
WebContainers needs to be at least as simple as that procedure.
First we need to deal with the universal claims made without making sure it is known that claim applies only you your Website.

Node.js, and 680 other languages can be run in the browser at https://tio.run/#javascript-node (https://github.com/TryItOnline).
Node.js can be run in the browser using Native Messaging https://github.com/simov/native-messaging?
What is novel about WebContainers?

WebContainers run Node.js entirely inside your browser, not on a remote server or local binary.
We currently do not expose a way to use WebContainer outside of StackBlitz.com, but this is something we're looking into. We'll post updates to this repo over the next quarter or so with our progress.
Closing this issue for now but lmk if you have other questions!

WebContainers run Node.js entirely inside your browser, not on a remote server or local binary.

We currently do not expose a way to use WebContainer outside of StackBlitz.com

I don't understand that.
On the one hand you claim Node.js is running entirely inside my browser.
On the other hand you say that only works on your Website.
So folks don't have to scour through your source files and reverse engineer your gear, FWIW, why not just openly release the Node.js WASM files as FOSS?


It's so... blue

What happened?
The bootstrap5 update is good, but the data tracker buttons are SO blue -- it makes it difficult to read the rest of the page, as they're so prominent, and much of the actual data values are grey.
There's also very little contrast in the top navber.
What browser(s) are you seeing the problem on?
Firefox, Safari
Code of Conduct

[X] I agree to follow the IETF's Code of Conduct

This will evolve, of course. But the basis of the color set is to outsource that set of decisions to bigger teams as best we can - we are, to the extent possible, using bootstrap defaults.
We have contracted a common bootstrap theme for the IETF (already part of the website, and will be integrated to the datatracker soon) that did involve real usability and accessibility testing. (It's at https://www.npmjs.com/package/@ietf-tools/common-bootstrap-theme).
Would it be alright with you to close this issue, knowing we've taken it as input into how the styling should evolve?

Sure thing. The main thing is that this isn't (just) an aesthetic issue -- it's affecting usability (at least for me).
Cheers,


[Feature] Add support for variable to disable scorecards support

Is there an existing feature request for this?

[X] I have searched the existing issues

Describe the Feature
Add support for var:
greenhouse_using_scorecards: false
Describe alternatives you've considered
No response
Are you interested in contributing this feature?

[ ] Yes.
[ ] Yes, but I will need assistance and will schedule time during your office hours for guidance.
[X] No.

Anything else?
No response

Hi @raphaelvarieras thanks for opening this issue! I believe adding a variable to disable the scorecard source should be simple enough.
Before accepting this Issue I did want to clarify the reason for the scorecard removal. My understanding of the interview process within Greenhouse is that Interviews are accompanied by a scorecard to properly evaluate the interview. Are there situations where a scorecard is not utilized?

I am honestly not sure - I just don't see anything related to scorecards in our database schema and an error is preventing us from using the package as a result.

Yes, we do use scorecards. They are in fact disabled in Fivetran UI but they are greyed out so I don't have the ability to enable them (yet I am connected with Site Admin credentials).

Thanks for confirming this @raphaelvarieras. This seems to be the core of the issue. I would prefer we get the table synced into your warehouse and you are able to leverage accurate data as opposed to building the workaround within the package.
Unfortunately, I do not have visibility into the inner workings of the connector behavior and why your credentials may not be working. I would recommend opening a Support Ticket so our customer support team will be able to help you sync your scorecard table.

Okey dokey - ticket open with support.

Sounds great, thanks @raphaelvarieras and please let me know if you continue to encounter any issues once you are able to sync this table.
For the time being, this is not a feature we will integrate into the package. As such, I will close this ticket. Please feel free to open a new issue if you encounter any other errors once you are able to sync the scorecard table.


Sails js Views are not loading..

Hi there,
I am working on sails.js v0.11 with mongodDB. I am using simple ejs forms as my views. All my views are working fine since yesterday. When i opened in the morning the views are not loading even the sail.js homepage. All my controller actions, routes  and models are working fine, even i am getting correct Json response for all requests. The only things are not working is views.  This is the third time i am facing this problem and i am really sick of creating new project and moving all my code to newly created project. I don't know, why sails.js views are suddenly stops working after few days. please can anyone help me with this problem.
Thanks in advance..
Regards
Anil

Maybe the controller has no return res event:
return res.view('',{});
or no res.end();
All in all, the code is not easy to find a certain details of the code error

Thanks wizzercn for your suggestion .
I checked all my code by moving all my code to new project, there it is working fine.
i didn't change anything in my code which was there in my old project, after some days, the views won't work and this is the problem i am facing..
i really don't know, what's wrong with my sails project.
please help me with this.
thanks in advance.

@anil-codewave are you still having issues?
Could you provide a repo and environmental details so that we might duplicated the issue?


Feature: Add keymanager voluntary_exit

Problem description
Eth Docker now offers signing exits with key manager, which does not yet work with Lodestar
Solution description
Implement keymanager function /eth/v1/validator/<pubkey>/voluntary_exit
Additional context
No response

Acknowledge! Should be easy to add


Package is not installable via composer: The archive may contain identical file names with different capitalization

$ docker-compose exec php composer require knplabs/doctrine-behaviors:~1.5
./composer.json has been updated
Loading composer repositories with package information
Updating dependencies (including require-dev)
Package operations: 1 install, 0 updates, 0 removals
  - Installing knplabs/doctrine-behaviors (1.5.0): Loading from cache
    Failed to execute unzip -qq  '/srv/api/vendor/knplabs/doctrine-behaviors/3d9b6237588659f3873aa8a1081df68a' -d '/srv/api/vendor/composer/8a9bb8db'

unzip: short read

    The archive may contain identical file names with different capitalization (which fails on case insensitive filesystems)
    Unzip with unzip command failed, falling back to ZipArchive class

As result, I've empty vendor/knplabs/doctrine-behaviors directory.
Apparently my docker image runs on case insensitive FS and there are files with same names, but in different case in the repository.
There are same style bugs in other projects, so I decided to open it here as well.
I'm still debugging this issue, so I'll update it if I'll found a workaround or it will turn up to be invalid.

@diimpp did you figure out the solution for this?

@opassion I don't quite remember, most likely I didn't use it in the end, but you always can fork the repository and rename conflicting files and use your repository as vcs source for the composer package.

Should by fixed now with PSR-4 used everywhere.
If not, please report exact file that is causing this


Add Nong Song

Song Name
Chilled 1
Artist Name
Recoil
Source
Direct file download
Youtube Link
No response
Direct File Link
/strorage/emulated/0/download/alucin mp3
Song ID
1
Extra
Layout 114

Okey


Force new connections to relayer on the application side

ClusterIP isn't suitable as a TCP Load Balancer because it's a headless construct. It's just a set of ip-tables rules.
So once a TCP connection is established it lives on until it's closed explicitly.
The solution I initially prototyped using HAProxy as a load balancer was also misguided in the sense that it did indeed achieve round robin access but it did so by explicitly terminating TCP connections which could potentially cause in-flight requests to be terminated as well because multiple requests being served by the onboarding service would reuse the same TCP connection to the relayer, at least in the current setup.
In order to mitigate these we can either:
(a) Move to a more robust transport mechanics - some form of queue (redis/kafka/rabbit/ampq) which was the long term plan anyway but it didn't seem worth it for the MVP
(b) Use service discovery in K8s via DNS lookup to periodically get available relayer endpoints and maintain an application level pool of connections and manage load-balancing on the application level.
(c) Take a slight performance hit by making NestJS instantiate some of our application's dependency tree on a per-request basis. Thus when a relayer interaction occurs during a request it creates a new TCP connection because a new provider is instantiated for that request.
This PR takes path (c) in order to validate the hypothesis on staging and because it's the smallest lift.

I tested this implementation in the East-US cluster and can confirm that the expected behaviour occurred.


Issue with $localize() after upgrading to angular 9

Hello,
After upgrading angular version to 9 and single-spa 5 i get the following error:

The error was caused due to angular 9 version using global $localize() function.
The solution here:
https://stackoverflow.com/questions/57953317/angular-9-introduced-a-global-localize-function-that-needs-to-be-loaded
fixes the problem by adding the import '@angular/localize/init'; to your polyfills.ts
After reading this:
https://single-spa.js.org/docs/ecosystem-angular#polyfills
I have to import @angular/localize/init and its dependencies in the index.html file.
Can you help me importing the modules in the root-html-file.
I dont want to import by npm. I want similar to this:
<script src='https://unpkg.com/core-js-bundle/minified.js'></script>
Thanks

I would try adding import '@angular/localize/init to your main.single-spa.ts file. Angular 9's Ivy Compiler makes it quite hard to load angular libs via script tag.

Thanks! You solved my issue!


Add testing with mochajs and coverage report with istanbul

@jlengstorf sending in case this is useful, maybe in an alt-config/ branch? I added testing with mochajs and test coverage report with istanbul.
The tests and coverage report apply to the src/ tree, not to the generate bundle. An alternate or hybrid approach would be to write tests applying to generated bundle.
Cheers,
Alex

Hmm I seem to have broken the source map! I will have to look into this. Closing PR for now.


can I use api to access h2ogpt？

If my Cloud Server's ip is <IP_ADDRESS>,the open port is 7860. can I use https://<IP_ADDRESS>:7860 to access h2ogpt？Thanks！

Hi, @yanmengxiang1 , Can you try the following steps, and give it a try ?
https://github.com/h2oai/h2ogpt/blob/main/client/README.md

@the-consultant-git
Thanks. I can run it.
If I already have running h2oGPT server, how do I use my own words to input the model?
I want to use a linux command to give an output on the command line. what should I do?


Lithium battery can be stacked.

The lithium battery can be stacked and charged together in IC2 machines. Multiple batteries consumes the same amount of energy as a single battery, but they can be split and the energy is copied without any cost.

Fixed! There are some very minor rounding errors, but I'm not sure there is too much I can do :)