[Bug report]: Popover based elements break when rendered with Angular @for

Requirements before reporting

[X] No duplicated issue reported.
[X] I have checked the latest version if the bug exist there. See all available packages at npmJS.com
[X] I have followed the installation guide.

Package versions
@scania/tegel: 1.13.0
@scania/tegel-angular: 1.13.0
Browser
Chrome
Framework
Angular
Version
17
Reproduction steps

Create an array of objects. Each object should have an id field that uniquely identifies it.
Render an array of objects using Angular's @for directive. Each rendered item should contain a tds-popover-menu
Make sure to use the track function of the @for loop and track the items by their unique ID property
Change the order of items

Code example
import { Component } from "@angular/core";
import { TegelModule } from "@scania/tegel-angular";

@Component({
  selector: "example-product-list",
  standalone: true,
  imports: [TegelModule],
  template: `
    @for (product of products; track product.productId) {
      <span>{{ product.description }}</span>

      <tds-button size="sm" [attr.id]="product.productId">
        <tds-icon name="kebab" slot="icon"></tds-icon>
      </tds-button>

      <tds-popover-menu [selector]="'#' + product.productId">
        <tds-popover-menu-item>
          <button (click)="moveProductDown(product.productId)">
            Move down
          </button>
        </tds-popover-menu-item>
      </tds-popover-menu>
      <tds-divider></tds-divider>
    }
  `,
})
export class ExampleProductListComponent {
  public products = [
    {
      productId: "product-id-1",
      description: "Truck",
    },
    {
      productId: "product-id-2",
      description: "Bus",
    },
  ];

  public moveProductDown(productId: string): void {
    const productIndex = this.products.findIndex(
      product => product.productId === productId
    );

    if (productIndex >= this.products.length - 1) {
      return;
    }

    const newProducts = structuredClone(this.products);
    const matchingProduct = newProducts[productIndex];
    newProducts.splice(productIndex, 1);
    newProducts.splice(productIndex + 1, 0, matchingProduct);
    this.products = structuredClone(newProducts);
  }
}

Expected behavior
The component above renders two items. Each item contains a button that opens a popover menu. After opening the popover menu of the "Truck" item and clicking on the "Move down" button, both menu buttons should still open their respective tds-popover-menu.
Actual behavior
After opening the popover menu of the "Truck" item and clicking on the "Move down" button, the popover menu of the "Bus" item no longer opens when clicking on the menu button.
This only seems to happen when the track function of the @for  loop is used to track items by their unique IDs. When using track $index instead of track product.productId this does not occur, but using track $index causes performance issues, since angular has to re-render the whole list instead of just the elements that changed.

@adarean5 based on your example code I made an test PR here: https://github.com/scania-digital-design-system/tegel-angular-17-demo/pull/34
I have used track product.productId and it seems it looks OK. Can you please check it?
I did an update on the popover-core component a release or two ago that might affect this issue too. So I need your pair of eyes to double-check if I implemented the example correctly and if the problem is still there.


[keymgr] Update programmers model

Removed init and merged its functionality directly into 'advance'
This also removes the init_wip status and merges everything into op_status
Streamline the states shown to software and used by hardware.  Software only sees the logical end states and not the transition states.

please have a look at the last two PRs, it's what we discussed last week about simplifying the init.
@weicaiyang i made some changes to DV side since the init field is gone. Please have a look and let me know how it can be done better.

please have a look at the last two PRs, it's what we discussed last week about simplifying the init.
@weicaiyang i made some changes to DV side since the init field is gone. Please have a look and let me know how it can be done better. Of course, feel free to change it however you like afterwards too!

design update LGTM. I will update the DV side later. I rerun the CI (private) as I can't see the detail of failure and this PR is passing in my local.

o dont' worry about the failure. It's not keymgr. I think it's some package
ordering issue related to keymgr/kmac.
I should have fixed it in the EDN pr.
On Tue, Dec 8, 2020 at 10:52 AM weicaiyang<EMAIL_ADDRESS>wrote:

please have a look at the last two PRs, it's what we discussed last week
about simplifying the init.
@weicaiyang https://github.com/weicaiyang i made some changes to DV
side since the init field is gone. Please have a look and let me know how
it can be done better. Of course, feel free to change it however you like
afterwards too!
design update LGTM. I will update the DV side later. I rerun the CI
(private) as I can't see the detail of failure and this PR is passing in my
local.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/lowRISC/opentitan/pull/4467#issuecomment-740844140,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAH2RSV4HCR5XB5DN4LDHMTSTZYV7ANCNFSM4URRV2WA
.

Apologies for the late response. Thanks @tjaychen, this looks great.

Apologies for the late response. Thanks @tjaychen, this looks great.


Create an example of a request graph with labels

This may not be needed now that we have a hover state


Downgrade to go 1.18, optimize code

Codecov Report

Merging #13 (8212471) into main (95dad34) will decrease coverage by 0.77%.
The diff coverage is 70.45%.

@@            Coverage Diff             @@
##             main      #13      +/-   ##
==========================================
- Coverage   89.88%   89.11%   -0.78%
==========================================
  Files          55       55
  Lines        1868     1993     +125
==========================================
+ Hits         1679     1776      +97
- Misses        162      170       +8
- Partials       27       47      +20

Impacted Files
Coverage Δ

codec/api_versions_resp.go
30.90% <0.00%> (-3.10%)
:arrow_down:

codec/util_kafka_int32.go
75.00% <ø> (+13.88%)
:arrow_up:

codec/util_kafka_string.go
73.58% <50.00%> (+2.99%)
:arrow_up:

codec/metadata_resp.go
85.22% <78.78%> (-12.08%)
:arrow_down:

codec/join_group_resp.go
95.12% <100.00%> (ø)

codec/leave_group_resp.go
100.00% <100.00%> (ø)

codec/metadata_req.go
85.71% <100.00%> (ø)

codec/offset_commit_resp.go
100.00% <100.00%> (ø)

codec/offset_fetch_resp.go
100.00% <100.00%> (ø)

... and 4 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


usertype with C style array  function return type

Hi,
I've seen that in order to bind c style array you need to use std::ref but this doesn't seems to work for member function return.
What would be the syntax for this ?
I'm interested for dimension array as well :)
like for
class Foo
{
public:
float **data() { return mpData; }
private:
float **mpData;
};

Thanks !

 Pointers are not arrays.
Please show me the errors when you use std::ref with a c-style array type. (What you have put here is not a c-style array, it is just a pointer to a pointer. Which will get serialized as... a pointer to a pointer.)

Well, in that case, I should update the title then.
Is there a way to access a float** as a 2D array in lua then ?

Yes. Write a wrapper for it as a proper container: http://sol2.readthedocs.io/en/latest/containers.html

(For example, grab gsl::span or something similar.)

I see.
Is this the right direction ?
class ChannelData
{
public:
  ChannelData(float* pData, size_t size)
  : mpData(pData)
  , mSize(size)
  {
  }

  size_t size() const
  {
    return mSize;
  }

  float& operator[](size_t pos)
  {
    assert(pos < mSize);
    return mpData[pos];
  }
  const float& operator[](size_t pos) const
  {
    assert(pos < mSize);
    return mpData[pos];
  }

private:
  float* mpData;
  size_t mSize;
};

namespace sol
{
  template <>
  struct is_container<ChannelData> : std::true_type
  {
  };
  template <>
  struct container_traits<ChannelData>
  {
  };
} // namespace sol

  lua.new_usertype<dataflow::AudioBuffer>("AudioBuffer",
                                          "numChannels",
                                          sol::property(&AudioBuffer::numChannels),
                                          "channelData",
                                          [](AudioBuffer& self, int channel) {
                                            return ChannelData(self.channelData(channel),
                                                               self.maxBlockSize());
                                          });

Thanks !

gsl::span works out of the box without declaring is_container so it looks like I am missing some stuff.
  lua.new_usertype<dataflow::AudioBuffer>("AudioBuffer",
                                          "numChannels",
                                          sol::property(&AudioBuffer::numChannels),
                                          "channelData",
                                          [](AudioBuffer& self, int channel) {
                                            return gsl::make_span(self.channelData(channel),
                                                                  self.maxBlockSize());
                                          }

Documentation lacks a simple example.
No hurry but that could be helpful to other people
Thanks !


Best approach to "only-svg" view

Hi everywere:
I need to find the best implementation for a OSD viewer showing a electrical circuit loaded from a SVG file.
That is, all the info comes from a SVG file and so is not a "overlay", in a proper way, but the main information container, as the background "image" is only a "dummy" solid color (white) actually.
In order to generate that background, I can think in several options:

hack it in a really dirty way, by means of a "premeditated" invalid tileSource URL;
using an image placeholder consisting in a solid color PNG with an enough resolution;
render the SVG as a rasterized image (i.e. again PNG) with an enough resolution.

All these alternative solutions do not appear to be very elegant, so... any advice on it?

I guess the first question is why use OSD for this? I assume you want the zooming/panning navigation, but it's best to be sure.
How big is your SVG in bytes? Is it okay to load it all at once, or is the whole thing big enough that it would cause a long wait? If it is that big, you're probably better off rendering it to a rasterized image and turning it into a tile set.
If you did want to pursue breaking the SVG up into SVG-based tiles, that would probably be a bit of work. I've got some thoughts here:
https://twitter.com/DenubisX/status/1006698407564460033
If the SVG is small enough to load all at once, you could certainly use the SVG overlay, but yes you would still need an image in OSD, I think, to generate the boundary the user zooms within. You could do this with a super tiny image though, probably even just a single pixel. Like so:
maxZoomPixelRatio: Infinity,
tileSources: {
    tileSource: {
        type: 'image',
        url: 'pixel.jpg'
    },
    width: 1000
}

It would certainly be more elegant to not require an image at all, but that would involve changes to OSD itself. If you're interested in pursuing that, I can help point you in the right direction. That said, the single pixel image solution doesn't seem like too terrible of a hack :)

BTW, I'm tagging this issue as a question. If you do want to pursue being able to load SVGs directly into OSD without raster images, you can file an issue in the OSD repository, and we can tag that with "enhancement".

Hi Ian, and as always thank you very much for your immediate support.
Answering your first question: yes I want to zoom/pan my SVGs and, well, as I'm using OSD already in the case of rasterized images, it would be nice not need to use/learn another library for the vectorized ones.
The SVGs I'm using are really small (around 250KB), that's a very good thing of the vectorized graphics in the case of CAD-like designs. That way I don´t need to split in tiles my files.
About the single pixel image, I find it very convenient and BTW similar as texture techniques used with 3D texturing in using in other kind of projects. The problem here (and of course with any other solution not using a "featured" rasterized image) is the lack of image in the Navigator view, as I think the SVG-overlay is not shown on it (if I'm not wrong).
Best regards.

Good point! Well, perhaps the simplest thing would be to make another copy of the SVG and lay it in the navigator element. Since the navigator doesn't zoom or pan, you only have to position it once. You can probably do this without messing with the OSD code at all.

I'm thinking just straight JavaScript:
        var img = document.createElement('img');
        img.src = 'foo.svg';
        img.style.position = 'absolute';
        img.style.left = 0;
        img.style.top = 0;
        img.style.width = '100%';
        viewer.navigator.canvas.appendChild(img);

Hi Ian:
First of all I'd like to apologize for my lack of answer to your info. I'm out of office at the moment since last week and could't read this thread.
Said this, thanks for so valuable info, I understand now that it is done under a basic DOM schema. The case is that I thought I needed to "hook" something with the OSD engine in order to not breaking the whole functionality.
Best regards.

Hi Ian:
First of all I'd like to apologize for my lack of answer to your info. I'm out of office at the moment since last week and couldn't read this thread.
Said this, thanks for so valuable info, I understand now that it is done under a basic DOM schema. The case is that I thought I needed to "hook" something with the OSD engine in order to not breaking the whole functionality.
Best regards.

No worries; thank you for the follow up. Yes, in this case I think it's perfectly safe to use basic DOM.

Hi again Ian:
I'm trying to fine-tune the behavior of this approach to the SVG viewer we've been talking about along this thread.
This way, one think I'd wish to implement is to scale the display-region-rectangle of the navigator according to the size of the SVG in the viewport, the same way that is implemented natively by OSD in the case of raster images.
Any point about this?

Hi again:
First of all, as you can see I've mistakenly closed this thread and immediately reopened it. Can you fix/delete those events?
In order to complete my needs (and I think of everybody) for this "only-SVG" scenario we are talking about, I see now that I can´t zoom-in nor out freely (I can only zoom-out a bit indeed).
Maybe this can be related to the mentioned "problem" with the "display-region"? I presume is needed to tweak more properties of the OSD engine that the little ones I've considered.
Best regards.

Ok, I think that in order to use the 1-pixel bg (raster) solution and at the same time getting a correct OSD interactive scaling behavior, I need to scale the SVG according to my needs of visualization.
I'm trying to understand that numbers now.

I'm happy to take a look if you can share a link... perhaps a CodePen?

Hi Ian:
As always, thank you very much for your support. As within the open source sw world that is the real difference between an engine that behaves a standard "de facto" (as OSD is now) or not.
Said this, a codepen is a bit complicated in my case, but I could try it if necessary.
However, maybe it could be enough to explain the problematic behavior (some degree of "dis-matching" between the part of the SVG shown in the "viewport" and the one in the "navigator") with the help of some graphical assets, namely:
a.1) viewer.open({ ..., width : 5});

a.2) viewer.open({ ..., width : 5});

a.3) viewer.open({ ..., width : 5});

b.1) viewer.open({ ..., width : 10});

b.2) viewer.open({ ..., width : 10});

b.3) viewer.open({ ..., width : 10});

Best regards.

Yes, I think we can talk through it here.
Basically you need to align your "bg" image to your SVG. One thing I'm realizing is a single pixel actually won't do, as the aspect ratios need to agree. Looks like your SVG is 11 inches by 8.5 inches. That's a 11:8.5 aspect ratio, or 22:17 when converted to whole numbers. So, your BG image should be 22px by 17px, and that way the aspect ratio will match.
As for the width you give it, it's funny adding inches into the mix, but it does seem to be working. When you specified width: 10 you seem to be getting most of the image, so I imagine width: 11 (to match the 11 inches of the SVG) would be perfect.
As for not being able to zoom in or out very much, the default zoom limits are based on the pixels of your BG image, so you'll have to change the limits. There are a few options for this you can see listed here:
http://openseadragon.github.io/docs/OpenSeadragon.html#.Options
If you want to completely disable zoom limits, you can use:
minZoomImageRatio: 0,
maxZoomPixelRatio: Infinity

There may be other things that need tweaking, but that's a good start.

Good morning (CET) Ian:
Ok, I see. Your tips put me definitively in the right way.
However, one think I'm trying to figure out how can be done is setting "zoom image ratio", min and max not in the Viewer constructor but dynamically, as I'm using the same viewer object to toggle from raster to vector image. If I'm not wrong there's not way of passing them (zoom rations) as a parameters to the Viewer.open() method and, on the other hand, I can´t find a setMinZoomImageRatio nor setMaxZoomImageratio method for the viewer object.
Kind regards.

Hi again Ian:
Please let me return now briefly to the positioning of the SVG in the navigator viewport.
The case is that I'm following the code previously suggested by you in this same thread:
    ...
    viewer.navigator.canvas.appendChild(img);

This way, I experiment a vertical offset for the SVG shown in the Navigator viewport, as it is styled to be positioned, no centered vertically (as the white BG is indeed shown), but at a specific distance (0 in this case) from its parent (canvas) element:

Inspecting now the markup of the Navigator element I find the following hierarchy is (identified by the applied class when available):
openseadragon-container
-->openseadragon-canvas
---->canvas
---->div
---->img
-->displayregioncontainer
---->displayregion
I'm going to try with the several techniques for centering an img vertically into its container div, as you are doing for the canvas (I suppose so), but I prefer to consult you before, so maybe you can see something I may have missed at this point.
Best regards.

Hi Ian, I'm tweaking a bit here and there the navigator styling and, well, I've got to something that is more close to an ideal solution.
The case is that I'm implementing the following CSS solution for centering unknown-height things vertically (I'll go for a more cross-browser and probably know-height solution when I get rid of this first approach):
...
position: absolute;
top: 50%;
transform: translateY(-50%);
width: 100%;
...

...in order to get this:

As you can see, I'm still experimenting some kind of vertical offset, I think is related with the existence of a padding in the openseadragon-canvas container div, but the case is that I can´t see nothing of that when inspecting that element.
Best regards.

For your first question: Yes, it appears there's no explicit API for changing the minZoomImageRatio (perhaps there should be). It looks like you can do so by setting a new value into viewer.viewport.minZoomImageRatio however.
Regarding the navigator placement: Is your navigator a dynamic size? If not, the easiest solution would be to explicitly set it with top and left values that work. Otherwise, seems like you're on the right track. You may have to tweak it manually... for instance, top: calc(50% + 5px).
It is odd that it's off in the viewer, since it looks pretty perfectly centered in the navigator. Anyway, if you tweak it, is it possible to get it adjusted properly (and therefore it's just a matter of positioning), or does it also need to be scaled some?

Hi Ian, and as always thank you very much for your immediate response.
About setting the minimum (maximum) ZoomImageRatio by means of viewport, I'll do so.
The v-offset I've been experimenting was my fault, as I was doing tests with other SVG files and I forgot to change later the background file size to the necessary 22x17px. After fixing it it is centered now all right:

About the positioning of it all via CSS styling I need a responsive behavior, so the following is working well for such a dynamic scenario:
position: absolute;
top: 50%;
transform: translateY(-50%);

Of course, It can be done also with the help of flexbox in the parent (viewer.navigator.canvas) but I prefer not to touch the internal OSD engine styles.
Best regards.

Fantastic. I'm glad we're documenting this process here in this thread, in case others want to do something similar.

Yes, I think so Ian.
With the help of this thread and the other one about Loading an external SVG file (started by Salitehkat)  I think it's possible to have a good idea of how to implement real-world SVG pan-zoom functionalities to any web development, with the aid of OSD.
Again, thank you very much for your support.
Best regards.

Excellent. When your project is ready to share, I'd love it if you could add it to http://openseadragon.github.io/examples/in-the-wild/ :)


Grafana Dashboard: effective_cache_size is wrong displayed

an effective_cache_size=3GB is shown as

because the metric value is not multiplied by block_size
"expr": "cnpg_pg_settings_setting{name=\"effective_cache_size\",namespace=~\"$namespace\",pod=~\"$instances\"}",
please assign the issue to me

can you submit a pull request please?

I added it to the open one
https://github.com/cloudnative-pg/cloudnative-pg/pull/2839


Forwarder Multiple instances not working

[x] I confirm that this is an issue rather than a question.

Hi guys,
Thanks for the great work on this amazing piece of software!
I am trying to setup a multiple hyperion instance configuration, but after a few days of trying I just can't figure it out. I've read this https://hyperion-project.org/threads/forwarder-on-coreelec-proto-hyperionrequest-missing-command.3585/ and followed the steps.
Your help is very much appreciated!
Bug report
Not sure whether it's a bug or I just missed something in my configuration.
Steps to reproduce

Two Raspberry PI 3B, resp. hyperion-1 and hyperion-2

hyperion-1

2.0.0-alpha.6 installed
Forwarder activated

json target 1: <IP_ADDRESS>:19444
flatbuffer target 1: <IP_ADDRESS>:19400

Platform Capture  deactivated
USB Capture activated
LED device activated

Controller Type: adalight

This instance controls a ws2813 ledstrip (271 leds) behind my TV perfectly fine, while playing a movie or using the hyperion app on my iPhone.
hyperion-2

2.0.0-alpha.6 installed

Forwarder deactivated

Platform Capture  deactivated

USB Capture deactivated

LED device activated

Controller Type: adalight

Json Server port: 19444

Flatbuffers Server port: 19400

This instance controls a ws2813 ledstrip (140 leds). Using the hyperion app on my iPhone the effects and color wheel controls the ledstrip as expected.
What is expected?
That hyperion-1 forwards to json target and flatbuffer, while hyperion-2 receives this data and controls the secondary ledstrip.
What is actually happening?
From the hyperion-1 logs is seems it connects to hyperion-2:
 [hyperiond FLATBUFCONNECTION] (INFO) Connecting to Hyperion: <IP_ADDRESS>:19400
 [hyperiond NETFORWARDER] (INFO) Forward now to json targets '<IP_ADDRESS>:19444'
 [hyperiond NETFORWARDER] (INFO) Forward now to flatbuffer targets '<IP_ADDRESS>:19400'

However at hyperion-2 there is nothing to see in the logging related to the connection and the ledstrip isn't controlled as expected.
System

Hyperion-1
Hyperion Server:

Build:       (HEAD detached at f5583de8) (Sersch-f7865b45/f5583de8-1590611619)
Build time:  May 27 2020 21:11:33
Git Remote:  https://github.com/hyperion-project/hyperion.ng
Version:     2.0.0-alpha.6
UI Lang:     auto (BrowserLang: en-gb)
UI Access:   expert
Avail Capt:  dispmanx,v4l2,framebuffer,qt

Hyperion Server OS:

Distribution: Raspbian GNU/Linux 10 (buster)
Arch:         arm
Kernel:       linux (4.19.118-v7+ (WS: 32))
Browser:      Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_5) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/13.1.1 Safari/605.1.15

Hyperion-2
Hyperion Server:

Build:       (HEAD detached at f5583de8) (Sersch-f7865b45/f5583de8-1590611619)
Build time:  May 27 2020 21:11:33
Git Remote:  https://github.com/hyperion-project/hyperion.ng
Version:     2.0.0-alpha.6
UI Lang:     auto (BrowserLang: en-gb)
UI Access:   expert
Avail Capt:  dispmanx,v4l2,framebuffer,qt

Hyperion Server OS:

Distribution: Raspbian GNU/Linux 10 (buster)
Arch:         arm
Kernel:       linux (4.19.118-v7+ (WS: 32))
Browser:      Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_5) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/13.1.1 Safari/605.1.15

I've got a setup similar to this with adalight and forwarding to a Raspberry Pi and I experienced the same problems. Today I compiled #820 and my setup is working fine since.

I've got a setup similar to this with adalight and forwarding to a Raspberry Pi and I experienced the same problems. Today I compiled #820 and my setup is working fine since.

Thanks for the tip! I'll try that branch as well 👍

@szlop , your right #820 fixes this issue as well, thanks! 👍🏽


Properly create the _controller attribute to handle the autowiring of controller actions parameters.

1. Why is this change necessary?
This change allows to create the autowired services as controllers action paramters with currently used (and future) symfony version.
See: https://github.com/symfony/http-kernel/blob/4619cd0da8b1528ee3d75e0187ac2349a523817e/Controller/ControllerResolver.php#L119
2. What does this change do, exactly?
Fixes the format of the _controller attribute of the request.
3. Describe each step to reproduce the issue or behaviour.

Create controller which uses autowired service.
Try to use such controller action.
System is unable to find the service and breaks.

4. Please link to the relevant issues (if any).

5. Which documentation changes (if any) need to be made because of this PR?

6. Checklist

[ ] I have written tests and verified that they fail without my change
[x] I have squashed any insignificant commits
[x] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/issues/SW-26298
Please use this issue to track the state of your pull request.


The model architecture may different from paper.

Hi, @sercant !
Thank you for sharing this fantastic project!
I found that in core/shufflenet_v2.py line 69 "layer_stride = 1" will cause the parameter "rate" as 1 instead of 2 in shufflenet V2 stage 4.
Could you check this small issue?
Thanks

Hello @songdoyouho,
Thank you for your interest. Yes, in fact, you are correct. One other person also pointed out the error to me via email. I didn't fix it since the checkpoints use the code as it is. In the meantime, I am training the network again with the correct stride adjustment. The new checkpoints might be available later on when I finish experimenting. The person pointed out the mistake also said that they tried the network with the correct stride and it didn't contribute to the accuracy. (But, the error causes the network to underperform if the output stride value is lowered. i.e. 8 while it should increase the accuracy)

Thank you for your instant reply!
Wish you have a nice year.


Review nginx and mysql configurations to ensure optimal performance

What happened (or feature request):
Per #195 We should review our nginx and mysql configurations to ensure configurations that relate to or impact performance are configured in the most optimal way possible.
What you expected to happen:
When we upgraded the web containers to PHP7, time was spent to focus on providing configurations for opcache, memory limits, etc to ensure we provided optimal performance. The same should be done for nginx and mysql, reviewing if there are any configurations that can be adjusted or introduced that would improve general web application performance. We should also check to see if new releases improve performance at all, and if so upgrade to them.
How to reproduce it (as minimally and precisely as possible):
Anything else do we need to know:
Related source links or issues:

Thanks for moving on this! A couple ideas:

Is it worth documenting any refence material that justifies any recommendations? That would help in the review process as well as ongoing performance audits in the future. Example: there are many Apache tuning articles on how to change number of threads based on RAM pool and php.ini limits.
Not sure if it's worth expanding this to address docker container resource allocations.

The fairly standard nginx config for Drupal is Perusio's - I think it's been holding as "normative" for many years now. Not sure if it's worth fiddling with using it, especially since we span more than one distro.

I originally created this ticket as part of an initial effort to break out the tasks in #195. However, when we discussed the tasks in sprint planning, @beeradb suggested that this task wasn't worth pursuing at this time. Since we know our biggest performance issues relate to bind mount performance on Mac, it seems questionable that we would make any notable gains in terms of nginx or mysql configuration changes. I also don't have any specific configuration changes in mind at this time to evaluate in terms of performance. Given this, I've marked this incubate and needs decision. We should decide if this task is worth pursuing (possibly at a later time), and if so, research/review would be useful to determine if there are any configurations worth evaluating.

Not worth addressing unless/until we have a specific need here.


onFailure(...) was not overridden, but callback was received

logcat has warning W/JsonHttpRH﹕ onFailure(int, Header[], String, Throwable) was not overriden, but callback was received in my code. I haved try to solved and read this issued https://github.com/loopj/android-async-http/issues/944 but i can't solve it.
My code
AsyncHttpClient client = new AsyncHttpClient();
client.get(address, null, new JsonHttpResponseHandler() {
      @Override
      public void onSuccess(int statusCode, Header[] headers, JSONObject response) {
               //Code
      }
      @Override
       public void onFailure(int statusCode, Header[] headers, Throwable throwable, JSONArray errorResponse) {
               Log.d(TAG , "onFailure : "+statusCode);
    }
});

You have not overriden all methods of JsonHttpResponseHandler().
Check the documentation:
https://github.com/loopj/android-async-http/blob/master/library/src/main/java/com/loopj/android/http/JsonHttpResponseHandler.java

There are several onFailure methods each with different parameter sets.
According to your log you need to override the one with the following parameters: (int, Header[], String, Throwable).
             ```

@Override
public void onFailure(int statusCode, Header[] headers, String responseString, Throwable throwable) {
Log.d(TAG , "onFailure : "+statusCode);
}

How to solve it, explain

@Mr-LYF override the methods you want to override, thats correct solution


chore(chain-adapters): do not export bn utils

Description
This removes the bn exports from chain-adapters utils.
The reason for that is that currently, they are available for consumers, which makes TypeScript Language Server confused and it will often pick the @shapeshiftoss/ exports over the local ones in consumer(s) (which for now is just web, see https://github.com/shapeshift/web/pull/1760).
Other PRs will follow for investor-foxy and investor-yearn packages.
Notice

[x] Have you followed the guidelines in our Contributing guide?
[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?

Pull Request Type

[ ] :bug: Bug fix (Non-breaking Change: Fixes an issue)
[x] :hammer_and_wrench: Chore (Non-breaking Change: Doc updates, pkg upgrades, typos, etc..)
[ ] :nail_care: New Feature (Breaking/Non-breaking Change)

Issue (if applicable)
N/A
Risk
N/A, we just consume bn utils directly from utils/bignumber vs. re-exporting them in utils and consuming them from there
Testing

Tests should still be happy

Screenshots (if applicable)

nice


DrgnParser: Don't error out for certain bad DWARF

Match the behaviour of CodeGen v1 and just log warnings when certain debug info has issues.
Example warning:
W0811 10:20:40.967265 112689 DrgnParser.cpp:522] Error looking up parent (1) for type 'EventBase': 2 DW_TAG_member has invalid DW_AT_data_member_location

Codecov Report

Merging #283 (f82dbd9) into main (e5a83a3) will decrease coverage by 0.08%.
The diff coverage is 7.14%.

@@            Coverage Diff             @@
##             main     #283      +/-   ##
==========================================
- Coverage   67.42%   67.34%   -0.08%
==========================================
  Files         105      105
  Lines       10520    10531      +11
  Branches     1761     1764       +3
==========================================
- Hits         7093     7092       -1
- Misses       2508     2520      +12
  Partials      919      919

Files Changed
Coverage Δ

oi/type_graph/DrgnParser.h
90.00% <ø> (ø)

oi/type_graph/DrgnParser.cpp
71.29% <7.14%> (-2.77%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Add ES6 String.prototype.repeat()

[x] Rebase once #1049 is merged
[x] Basic implementation
[x] Small footprint variant
[x] Internal fixed buffer zeroing optimization (very often zeroing not needed)
[x] Note on UTF-8 safety: if input is UTF-8, output is also UTF-8; if input is not UTF-8 it's possible for initial and last bytes of the repeated string to combine in the repetition => noted in test case
[x] Testcase
[x] Releases entry

There's a (probably) harmless warning in master related to this change:
1>duk_bi_string.c(1345): warning C4267: '=': conversion from 'size_t' to 'duk_uint_t', possible loss of data (compiling source file ..\src\shared\duktape.c)

Thanks, I'll fix that.


upgrade form sqlds/v2 to sqlds/v4

Could you please double check and test it before merging?

tested and it all works as expected, thanks!


Email verification Function Remove when user will sign up using Email ID system

Feature description
Email verification Function Remove when user will sign up using Email ID system
👀 Have you spent some time to check if this issue has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I agree to follow this project's Code of Conduct

I want to work on this issue under JWOC


[WIP] AssetsListener

Allow to use assets (css, js etc.) from modules, or other not public folders
Web server has not access to module folder directly, but this is possible through AssetsListener.
Zend\View\Helper\Assets helper can build link to module folder (through Router),
AssetsListener can detect this link (through Router too) and route it to module name and asset.
When AssetsListener detected asset link, it caching module asset to public folder and
browser can request this asset directly from public folder (it can be disabled).
Also AssetsListener can filter assets content, as sample .less to .css.
This is extention for https://github.com/zendframework/zend-view/pull/106

I am going to close this PR now due to lack of activity.
Additionally, this functionality is nice to have but I would argue it belongs to a module, not this package.


tootle: fix build

Description of changes
Fixing Tootle since it's still the only GTK mastodon client.
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Yeah I am not fixing this previously with the reason listed in https://github.com/NixOS/nixpkgs/pull/194135#issuecomment-1264815647, we should really start looking for new alternatives :see_no_evil:


refact with Generator

1.Remove "--benchmarks=all" will skip the benchmark test. Previous behavior will skip all test cases.
2.Refact rrelu with Generator which is introduced in Random number generation
3.Add graph test support

How could I disable flake8 error E722 do not use bare 'except'

@googlebot I consent

I've merged master into your branch to update it and fixed any formatting/conflicts it might have. If you need to do some more modifications, please do git pull beforehand.

cc @gabrieldemarmiesse


filter dropdown

new filter dropdown that filters by perun columns

Overall I'm still re-thinking this. Czech comment will be on redmine.


rpm spec file doesn't install man pages

The current version bump in RPM doesn't install the man pages.
Need to look at and adjust so it does.


How to penalty repeating some parts of the answers?

I noticed some answers were incorrect due to repetition. Is there a way to add some penalty to the repetition?

Thank you very much in advance!
Best regards,
Shuyue
Dec. 21st, 2023

https://github.com/vllm-project/vllm/blob/925f3332cac488e5ad2dbc8f5c6d5f42d2556816/vllm/sampling_params.py#L47-L50


Possible to use multiple layout.erb files?

I'm attempting to make a landing page. I can manage with a single template, but it would be ideal if I would use another template/layout. I haven't seen any examples in the wild so I was hoping someone could point me in the right direction.

I figured this out. :+1:
http://blog.konstantinpavlov.net/2015/04/12/slate-static-api-documentation-generator/


Pass original app kwargs so that un-pickling doesn't fail for celerybeat.

What is the unpickling error you're experiencing with celerybeat? I don't think this is the right approach. init() in that class is only meant for Flask, not Celery. The proper celerybeat implementation can be found here: https://github.com/Robpol86/Flask-Large-Application-Example/blob/master/manage.py#L224


Updated tokenizers to match transformers requirement

tokenizers version must be >= 0.14

We don't have a transformers dependency, I didn't quite understand the use case. Can you share some context?

Here is the conflict:
transformers 4.34.0 depends on tokenizers<0.15 and >=0.14
fastembed 0.1.1 depends on tokenizers<0.14 and >=0.13


[Proposal] Allow to specify dtype for Discrete

Proposal
Add dtype argument to spaces.Discrete (similar to MultiDiscrete and Box).
Motivation
Currently the dtype is fixed to numpy.int64. However, often Discrete spaces are much smaller, resulting in a waste of memory.
Pitch
No response
Alternatives
No response
Additional context
No response
Checklist

[X] I have checked that there is no similar issue in the repo

Sure, if you can update all the functions and update the example testing spaces with dtypes

I would like to work on the issue.
Can you guide me on what other functions needs to be updated if we default it to int8 or int32 in constructor?
With example testing spaces do you mean tests/spaces/test_discrete.py? There is no explicit dtype checks in multidiscrete test, can you tell what needs to be done there?


install

I am relatively new to all of this but,  I have made the database connection changes needed and uploaded all of the files to my server and every time I try to run the install I am getting a 404 file not found do I have the files in the wrong spot on my server or am I missing something else. any help wouldld be appreciated?

figured it out

Please share your fix. Responding "figured it out" helps no one in the future who comes across the same issue.

I'm guessing that the 404 error ended up that the files were in the wrong spot and you fixed it by moving them or updating the server config.
I'm closing the bug, but I'm trying to watch github more often for bugs (I wasn't getting notifications for the last month, so I missed this when it happened).


valid args for -K / --rmulti2

I'm guessing valid args for this are either 1 or 2?
Or maybe not.. I passed 30 to it just for testing, and during

[P2] Table 7 scan
[P2] Table 7 rewrite

it appeared to only be using 1 thread.. and I just looked back and it seems to be using all 48 cores..
Trying to figure out what the best setting is for this for my setup.
Phase 2 times
Prior to merging https://github.com/madMAx43v3r/chia-plotter/pull/670: 430.402s
-K 2: 544.311s
-K 30: 482.178s

30 is too high a value, 2 is twice as many threads put in the -r argument, the way it is in the code is num_threads * g_thread_multi

-K 1 = same as -r (today has little load in phase 2)
-K 2 = double the -r
-K 3 = triple of -r
...
at a certain point, the number of threads becomes a bottleneck that will take more time, you have to test to see how far this limit is compensated on your hardware

1 or 2 makes most sense, it multiplies the -r value

From my testing with a dual xeon 2678v3 system, phase 2 performs best at -r 48, and phase 3 does best at -r 24 with phase 1 about the same for both. Phase 3 is the dominant factor, so my previous fastest time was 24 minutes at -r 32 (best balance of phase 2 and 3).
Now with -r 24 -K 2 I get the best of both, and my plot creation time got reduced by just over a minute.

I tried it with a value of 2 and I didn't get any difference

From my testing with a dual xeon 2678v3 system, phase 2 performs best at -r 48, and phase 3 does best at -r 24 with phase 1 about the same for both. Phase 3 is the dominant factor, so my previous fastest time was 24 minutes at -r 32 (best balance of phase 2 and 3).
Now with -r 24 -K 2 I get the best of both, and my plot creation time got reduced by just over a minute.

I appreciate it. It also looks like you've been doing a lot of testing and benchmarking. I stumbled across https://github.com/sharkdp/hyperfine , and it's been a treasure.
hyperfine -r 2 -P threads 20 48 '~/bin/chia_plot -c $SCK -f $FPK -n 1 -r {threads} -u 256 -v 256 -t /mnt/ssdarray1/ -2 /mnt/ssdarray2/ -d /mnt/ssdarray1/'
A command like the above results in this (if you chose to output as md) https://gist.github.com/3b430d7cac9260a56a7435a5fbecc309

@stotiks - appreciate you may be busy, but do you know when this option will be added to the Windows build?

@stotiks - appreciate you may be busy, but do you know when this option will be added to the Windows build?

Even just some working instructions on how to build for Windows from Source would be ideal, then I'd look at making changes myself


SeasonHUD crashes with newer FTB Chunks releases

Describe the issue

SeasonHUD crashes with newer FTB Chunks releases.
Using build 2101.1.0 of FTB Chunks-

Minecraft Version

1.21.1

Mod Version

1.9.8

Modloader(Fabric/Forge/NeoForge)

NeoForge

Modloader Version

21.1.23

Additional Info (Crashlog/Modlist)

https://mclo.gs/N3EKAaf

Should be fixed in 1.10.0


Fix forceFetch on SSR

Fix for #292
TODO:

[x] Make sure all of the significant new logic is covered by tests
[x] Rebase your changes on master so that they can be merged easily
[x] Make sure all tests and linter rules pass
[x] Update CHANGELOG.md with your change

Interesting. We added a ssrMode: true to AC which simply ignores forceFetch to get around this problem originally.
I don't recall if we had an idea at the time of use cases where people might validly want to force fetch in SSR (and thus not set the option). Probably not huh? If so, we should remove ssrMode as an option (and make sure the angular SSR works the same way).


wg_engine: require code refactoring

CompositeMethod::ClipPath will be replaced with the Scene::clip() (currently -> Paint::clip()) method. gl_engine needs to refactor to remove the enum value usage before proceeding with its removal.

see:
    /**
     * @brief Clip the drawing region of the paint object.
     *
     * This function restricts the drawing area of the paint object to the specified shape's paths.
     *
     * @param[in] clipper The shape object as the clipper.
     *
     * @retval Result::NonSupport If the @p clipper type is not Shape.
     *
     * @note @p clipper only supports the Shape type.
     * @note Experimental API
     */
    Result Paint::clip(std::unique_ptr<Paint> clipper);

@hermet
wg_engine did not use CompositeMethod::ClipPath enum value at all.
wg_engine uses only clips count in RenderMethod::prepare stage.
So nothing to change in a case of enum values

@SergeyLebedkin it does. Please check the screenshot.


Enhancement: List multiple locations

It is often useful to know the weather in multiple locations such as when travelling.

This is already a feature present in the application you can choose how many locations you want and use the a replicant for each location...

Example


Elasticsearch nodes disk usage above high watermark

This issue appears as soon as I start my docker instances.
Log from the datanode
2024-10-09T09:13:26.607Z INFO  [OpensearchProcessImpl] [2024-10-09T09:13:26,607][WARN ][o.o.c.r.a.DiskThresholdMonitor] [datanode] high disk watermark [90%] exceeded on [5ii93UqvQoC-Vnxt6Fx28g][datanode][/var/lib/graylog-datanode/opensearch/data/nodes/0] free: 4.1gb[8.6%], shards will be relocated away from this node; currently relocating away shards totalling [0] bytes; the node is expected to continue to exceed the high disk watermark when these relocations are complete

{"error":{"root_cause":[{"type":"cluster_block_exception","reason":"index [.ds-gl-datanode-metrics-000003] blocked by: [TOO_MANY_REQUESTS/12/disk usage exceeded flood-stage watermark, index has read-only-allow-delete block];"}],"type":"cluster_block_exception","reason":"index [.ds-gl-datanode-metrics-000003] blocked by: [TOO_MANY_REQUESTS/12/disk usage exceeded flood-stage watermark, index has read-only-allow-delete block];"},"status":429}
                at org.graylog.shaded.opensearch2.org.opensearch.client.RestClient.convertResponse(RestClient.java:376) ~[graylog-storage-opensearch2-6.0.7.jar:?]
                at org.graylog.shaded.opensearch2.org.opensearch.client.RestClient.access$1800(RestClient.java:120) ~[graylog-storage-opensearch2-6.0.7.jar:?]
                at org.graylog.shaded.opensearch2.org.opensearch.client.RestClient$1.completed(RestClient.java:426) [graylog-storage-opensearch2-6.0.7.jar:?]
                at org.graylog.shaded.opensearch2.org.opensearch.client.RestClient$1.completed(RestClient.java:422) [graylog-storage-opensearch2-6.0.7.jar:?]
                at org.graylog.shaded.opensearch2.org.apache.http.concurrent.BasicFuture.completed(BasicFuture.java:122) [graylog-storage-opensearch2-6.0.7.jar:?]
                at org.graylog.shaded.opensearch2.org.apache.http.impl.nio.client.DefaultClientExchangeHandlerImpl.responseCompleted(DefaultClientExchangeHandlerImpl.java:182) [graylog-storage-opensearch2-6.0.7.jar:?]
                at org.graylog.shaded.opensearch2.org.apache.http.nio.protocol.HttpAsyncRequestExecutor.processResponse(HttpAsyncRequestExecutor.java:448) [graylog-storage-opensearch2-6.0.7.jar:?]
                at org.graylog.shaded.opensearch2.org.apache.http.nio.protocol.HttpAsyncRequestExecutor.inputReady(HttpAsyncRequestExecutor.java:338) [graylog-storage-opensearch2-6.0.7.jar:?]
                at org.graylog.shaded.opensearch2.org.apache.http.impl.nio.DefaultNHttpClientConnection.consumeInput(DefaultNHttpClientConnection.java:265) [graylog-storage-opensearch2-6.0.7.jar:?]
                at org.graylog.shaded.opensearch2.org.apache.http.impl.nio.client.InternalIODispatch.onInputReady(InternalIODispatch.java:87) [graylog-storage-opensearch2-6.0.7.jar:?]
                at org.graylog.shaded.opensearch2.org.apache.http.impl.nio.client.InternalIODispatch.onInputReady(InternalIODispatch.java:40) [graylog-storage-opensearch2-6.0.7.jar:?]
                at org.graylog.shaded.opensearch2.org.apache.http.impl.nio.reactor.AbstractIODispatch.inputReady(AbstractIODispatch.java:121) [graylog-storage-opensearch2-6.0.7.jar:?]
                at org.graylog.shaded.opensearch2.org.apache.http.impl.nio.reactor.BaseIOReactor.readable(BaseIOReactor.java:162) [graylog-storage-opensearch2-6.0.7.jar:?]
                at org.graylog.shaded.opensearch2.org.apache.http.impl.nio.reactor.AbstractIOReactor.processEvent(AbstractIOReactor.java:337) [graylog-storage-opensearch2-6.0.7.jar:?]
                at org.graylog.shaded.opensearch2.org.apache.http.impl.nio.reactor.AbstractIOReactor.processEvents(AbstractIOReactor.java:315) [graylog-storage-opensearch2-6.0.7.jar:?]
                at org.graylog.shaded.opensearch2.org.apache.http.impl.nio.reactor.AbstractIOReactor.execute(AbstractIOReactor.java:276) [graylog-storage-opensearch2-6.0.7.jar:?]
                at org.graylog.shaded.opensearch2.org.apache.http.impl.nio.reactor.BaseIOReactor.execute(BaseIOReactor.java:104) [graylog-storage-opensearch2-6.0.7.jar:?]
                at org.graylog.shaded.opensearch2.org.apache.http.impl.nio.reactor.AbstractMultiworkerIOReactor$Worker.run(AbstractMultiworkerIOReactor.java:591) [graylog-storage-opensearch2-6.0.7.jar:?]
                at java.base/java.lang.Thread.run(Unknown Source) [?:?]

Just as a test, I logged on to the shell of datanode and created 6.3GB file with /dev/random.

Testing the docker-compose file I've used on another machine does not yield this same issue.
I would let this go and just use that machine, but I am afraid I am just kicking this issue forward to future me.
The compose-file I am using
version: "3.8"

# For DataNode setup, graylog starts with a preflight UI, this is a change from just using OpenSearch/Elasticsearch.
# Please take a look at the README at the top of this repo or the regular docs for more info.

services:
  mongodb:
    image: "mongo:5.0"
    volumes:
      - "mongodb_data:/data/db"
    restart: "on-failure"

  # For DataNode setup, graylog starts with a preflight UI, this is a change from just using OpenSearch/Elasticsearch.
  # Please take a look at the README at the top of this repo or the regular docs for more info.
  datanode:
    image: "${DATANODE_IMAGE:-graylog/graylog-datanode:6.0.7}"
    hostname: "datanode"
    environment:
      GRAYLOG_DATANODE_NODE_ID_FILE: "/var/lib/graylog-datanode/node-id"
      GRAYLOG_DATANODE_PASSWORD_SECRET: "${GRAYLOG_PASSWORD_SECRET:?Please configure GRAYLOG_PASSWORD_SECRET in the .env file}"
      GRAYLOG_DATANODE_ROOT_PASSWORD_SHA2: "${GRAYLOG_ROOT_PASSWORD_SHA2:?Please configure GRAYLOG_ROOT_PASSWORD_SHA2 in the .env file}"
      GRAYLOG_DATANODE_MONGODB_URI: "mongodb://mongodb:27017/graylog"
    ulimits:
      memlock:
        hard: -1
        soft: -1
      nofile:
        soft: 65536
        hard: 65536
    ports:
      - "<IP_ADDRESS>:8999:8999/tcp"   # DataNode API
      - "<IP_ADDRESS>:9200:9200/tcp"
      - "<IP_ADDRESS>:9300:9300/tcp"
    volumes:
      - "graylog-datanode:/var/lib/graylog-datanode"
    restart: "on-failure"

  graylog:
    hostname: "server"
    image: "${GRAYLOG_IMAGE:-graylog/graylog:6.0.7}"
    depends_on:
      mongodb:
        condition: "service_started"
    entrypoint: "/usr/bin/tini --  /docker-entrypoint.sh"
    environment:
      GRAYLOG_NODE_ID_FILE: "/usr/share/graylog/data/data/node-id"
      GRAYLOG_PASSWORD_SECRET: "${GRAYLOG_PASSWORD_SECRET:?Please configure GRAYLOG_PASSWORD_SECRET in the .env file}"
      GRAYLOG_ROOT_PASSWORD_SHA2: "${GRAYLOG_ROOT_PASSWORD_SHA2:?Please configure GRAYLOG_ROOT_PASSWORD_SHA2 in the .env file}"
      GRAYLOG_HTTP_BIND_ADDRESS: "<IP_ADDRESS>:9000"
      GRAYLOG_HTTP_EXTERNAL_URI: "http://localhost:9000/"
      GRAYLOG_MONGODB_URI: "mongodb://mongodb:27017/graylog"
    ports:
    - "<IP_ADDRESS>:5044:5044/tcp"   # Beats
    - "<IP_ADDRESS>:5140:5140/udp"   # Syslog
    - "<IP_ADDRESS>:5140:5140/tcp"   # Syslog
    - "<IP_ADDRESS>:5555:5555/tcp"   # RAW TCP
    - "<IP_ADDRESS>:5555:5555/udp"   # RAW UDP
    - "<IP_ADDRESS>:9000:9000/tcp"   # Server API
    - "<IP_ADDRESS>:12201:12201/tcp" # GELF TCP
    - "<IP_ADDRESS>:12201:12201/udp" # GELF UDP
    #- "<IP_ADDRESS>:10000:10000/tcp" # Custom TCP port
    #- "<IP_ADDRESS>:10000:10000/udp" # Custom UDP port
    - "<IP_ADDRESS>:13301:13301/tcp" # Forwarder data
    - "<IP_ADDRESS>:13302:13302/tcp" # Forwarder config
    volumes:
      - "graylog_data:/usr/share/graylog/data/data"
      - "graylog_journal:/usr/share/graylog/data/journal"
    restart: "on-failure"

volumes:
  mongodb_data:
  graylog-datanode:
  graylog_data:
  graylog_journal:

The available storage on the host machine.

Docker host system info and storage

HI fredrikjag! We are using GitHub issues for tracking bugs in Graylog itself, but this looks like a support question. Please post this issue to our discussion forum.
Thank you!

HI fredrikjag! We are using GitHub issues for tracking bugs in Graylog itself, but this looks like a support question. Please post this issue to our discussion forum.
Thank you!

I don't really think this is only a support question.
This must be a bug, how can starting the containers and completing the initial setup cause the application to 7 days later, crash?
I have not put any data on the graylog instance at all.
Now, restarting the instances it will result in application traceback..
{"error":{"root_cause":[{"type":"cluster_block_exception","reason":"index [.ds-gl-datanode-metrics-000003] blocked by: [TOO_MANY_REQUESTS/12/disk usage exceeded flood-stage watermark, index has read-only-allow-delete block];"}],"type":"cluster_block_exception","reason":"index [.ds-gl-datanode-metrics-000003] blocked by: [TOO_MANY_REQUESTS/12/disk usage exceeded flood-stage watermark, index has read-only-allow-delete block];"},"status":429}
                at org.graylog.shaded.opensearch2.org.opensearch.client.RestClient.convertResponse(RestClient.java:376) ~[graylog-storage-opensearch2-6.0.7.jar:?]
                at org.graylog.shaded.opensearch2.org.opensearch.client.RestClient.access$1800(RestClient.java:120) ~[graylog-storage-opensearch2-6.0.7.jar:?]
                at org.graylog.shaded.opensearch2.org.opensearch.client.RestClient$1.completed(RestClient.java:426) [graylog-storage-opensearch2-6.0.7.jar:?]
                at org.graylog.shaded.opensearch2.org.opensearch.client.RestClient$1.completed(RestClient.java:422) [graylog-storage-opensearch2-6.0.7.jar:?]
                at org.graylog.shaded.opensearch2.org.apache.http.concurrent.BasicFuture.completed(BasicFuture.java:122) [graylog-storage-opensearch2-6.0.7.jar:?]
                at org.graylog.shaded.opensearch2.org.apache.http.impl.nio.client.DefaultClientExchangeHandlerImpl.responseCompleted(DefaultClientExchangeHandlerImpl.java:182) [graylog-storage-opensearch2-6.0.7.jar:?]
                at org.graylog.shaded.opensearch2.org.apache.http.nio.protocol.HttpAsyncRequestExecutor.processResponse(HttpAsyncRequestExecutor.java:448) [graylog-storage-opensearch2-6.0.7.jar:?]
                at org.graylog.shaded.opensearch2.org.apache.http.nio.protocol.HttpAsyncRequestExecutor.inputReady(HttpAsyncRequestExecutor.java:338) [graylog-storage-opensearch2-6.0.7.jar:?]
                at org.graylog.shaded.opensearch2.org.apache.http.impl.nio.DefaultNHttpClientConnection.consumeInput(DefaultNHttpClientConnection.java:265) [graylog-storage-opensearch2-6.0.7.jar:?]
                at org.graylog.shaded.opensearch2.org.apache.http.impl.nio.client.InternalIODispatch.onInputReady(InternalIODispatch.java:87) [graylog-storage-opensearch2-6.0.7.jar:?]
                at org.graylog.shaded.opensearch2.org.apache.http.impl.nio.client.InternalIODispatch.onInputReady(InternalIODispatch.java:40) [graylog-storage-opensearch2-6.0.7.jar:?]
                at org.graylog.shaded.opensearch2.org.apache.http.impl.nio.reactor.AbstractIODispatch.inputReady(AbstractIODispatch.java:121) [graylog-storage-opensearch2-6.0.7.jar:?]
                at org.graylog.shaded.opensearch2.org.apache.http.impl.nio.reactor.BaseIOReactor.readable(BaseIOReactor.java:162) [graylog-storage-opensearch2-6.0.7.jar:?]
                at org.graylog.shaded.opensearch2.org.apache.http.impl.nio.reactor.AbstractIOReactor.processEvent(AbstractIOReactor.java:337) [graylog-storage-opensearch2-6.0.7.jar:?]
                at org.graylog.shaded.opensearch2.org.apache.http.impl.nio.reactor.AbstractIOReactor.processEvents(AbstractIOReactor.java:315) [graylog-storage-opensearch2-6.0.7.jar:?]
                at org.graylog.shaded.opensearch2.org.apache.http.impl.nio.reactor.AbstractIOReactor.execute(AbstractIOReactor.java:276) [graylog-storage-opensearch2-6.0.7.jar:?]
                at org.graylog.shaded.opensearch2.org.apache.http.impl.nio.reactor.BaseIOReactor.execute(BaseIOReactor.java:104) [graylog-storage-opensearch2-6.0.7.jar:?]
                at org.graylog.shaded.opensearch2.org.apache.http.impl.nio.reactor.AbstractMultiworkerIOReactor$Worker.run(AbstractMultiworkerIOReactor.java:591) [graylog-storage-opensearch2-6.0.7.jar:?]
                at java.base/java.lang.Thread.run(Unknown Source) [?:?]

Hi @fredrikjag adding to Jan above, if you believe this is a bug, can you share the series of steps you took that led you to this state?
Can you also confirm if your disk did indeed exceed the default watermark of 85% disk space used?
Can you also share the output of the following which specifically looks for a log line:
Be sure to replace docker-datanode-1 with your datanode container name or container id
sudo docker logs docker-datanode-1 2> /dev/null | grep "space"
should output something like:
10-30T16:13:24.116Z INFO  [OpensearchProcessImpl] [2024-10-30T16:13:24,116][INFO ][o.o.e.NodeEnvironment    ] [datanode] using [1] data paths, mounts [[/var/lib/graylog-datanode (/dev/mapper/ubuntu--vg-ubuntu--lv)]], net usable_space [18.1gb], net total_space [29.3gb], types [ext4]

What state is your graylog and data node in now? Do they start? Are they reachable via their bound ports?
Thanks!

Hi @fredrikjag adding to Jan above, if you believe this is a bug, can you share the series of steps you took that led you to this state?
Can you also confirm if your disk did indeed exceed the default watermark of 85% disk space used?
Can you also share the output of the following which specifically looks for a log line: Be sure to replace docker-datanode-1 with your datanode container name or container id sudo docker logs docker-datanode-1 2> /dev/null | grep "space"
should output something like:
10-30T16:13:24.116Z INFO  [OpensearchProcessImpl] [2024-10-30T16:13:24,116][INFO ][o.o.e.NodeEnvironment    ] [datanode] using [1] data paths, mounts [[/var/lib/graylog-datanode (/dev/mapper/ubuntu--vg-ubuntu--lv)]], net usable_space [18.1gb], net total_space [29.3gb], types [ext4]

What state is your graylog and data node in now? Do they start? Are they reachable via their bound ports?
Thanks!

At the moment it's working as I tried re-created the environment.
Currently there is no watermark, but give it a couple of days and we are back at square one.
This is the output.
2024-10-31T09:12:31.556Z INFO  [OpensearchProcessImpl] [2024-10-31T09:12:31,556][INFO ][o.o.e.NodeEnvironment    ] [datanode] using [1] data paths, mounts [[/var/lib/graylog-datanode (/dev/mapper/ubuntu--vg-ubuntu--lv)]], net usable_space [73.1gb], net total_space [97.8gb], types [ext4]```


Add R Ratio Filter

Clinical Summary
Add a filter for R Ratio/R Score.
Details
Notes below from @JimBuchanan:
The other ALP filter uses what is called the “R ratio” or “R score”. The R ratio is calculated as:
(ALT value/ULN for ALT) / (ALP value/ULN for ALP)
An R value of ≥5 indicates hepatocellular injury, >2 to <5 mixed-type injury and ≤2 cholestatic injury.

This would be an important feature to include in v1.0.

Initial implementation is almost done - Added a new control allowing a user to set a minimum R Ratio value. All values below this cut point are hidden - default value = 0. This is controlled by 2 settings:
config.r_ratio_filter - boolean indicating whether the R ratio filter should be shown.
config.r_ratio_cut - numeric indicating the default value of the R ratio cut point. User can change this value


Packit is blocked by rust crates.io for download crate tarball

In https://github.com/rust-netlink/netlink-sys/runs/11770471551 , we noticed a 403 error when downloading the rpm source tarball from https://crates.io/api/v1/crates/netlink-sys/0.8.5/download#/netlink-sys-0.8.5.crate .
Manually download that URL works well.
According to https://github.com/rust-lang/crates.io/issues/1361 , it seems your bot is retrying to frequent which caused this block.
Any idea.

Hi @cathay4t, thanks for reporting!
Firstly, I can see that you had before configured both pull_from_upstream and propose_downstream jobs. I see now you removed the configuration in GitHub, so just to clarify, they cover the same functionality, the difference is that the propose_downstream is configured in GitHub and reacts to the GitHub webhooks about the new release and the pull_from_upstream reacts to upstream-release-monitoring messages (which means there is usually some delay) and is configured in the dist-git repo directly so you do not have to touch the GitHub repository.
As for the downloading and retrying: if we are not able to download the source tarball, we retry after 1 minute and then after 2 minutes, these retries are meant for use cases when the source is being uploaded e.g. to some package manager or other place and is not done yet when we receive the GitHub webhook about the new release. With pull_from_upstream this is less probable to happen, as there is some delay. This time limit seems quite sensible to me.
We also utilise some auto-retries that happen in seconds.
But we could probably bump into this issue again if e.g. processing multiple packages that require downloading sources from crates.io at a time.
I will add this to discuss at our team meeting tomorrow and will provide updates on this.

We discussed this thoroughly and even did a few experiments.
I personally checked how can we obtain the archive without accessing crates.io API, I'm assuming the API endpoint has those strict rules. My assumption was the archives will be on a cold CDN storage. Let's try:
$ curl -v https://crates.io/api/v1/crates/aleph/0.0.1/download
...
< HTTP/2 302
< location: https://static.crates.io/crates/aleph/aleph-0.0.1.crate
...
* Issue another request to this URL: 'https://static.crates.io/crates/aleph/aleph-0.0.1.crate'
*   Trying <IP_ADDRESS>:443...
* Connected to static.crates.io (<IP_ADDRESS>) port 443 (#1)
...
< content-type: application/x-tar
< server: AmazonS3
< content-length: 312

Yup, the tarballs are on AWS S3.
$ dig static.crates.io
;static.crates.io.              IN      A
static.crates.io.       214     IN      CNAME   fastly-static.crates.io.
fastly-static.crates.io. 5      IN      CNAME   n.sni.global.fastly.net.
n.sni.global.fastly.net. 5      IN      A       <IP_ADDRESS>

One thing that could help here would be to set static.crates.io as the Source0 in the spec files to bypass the crates.io API and directly access the archives.

One thing that could help here would be to set static.crates.io as the Source0 in the spec files to bypass the crates.io API and directly access the archives.

The S3 URLs are probably not stable.
Using %{crate_source} macro (which expands to the crates.io API endpoint for the download) is mandatory:

https://docs.fedoraproject.org/en-US/packaging-guidelines/Rust/#_package_sources_2
In my years of doing Rust packaging, I have never hit the API rate limit for crates.io endpoints ... but rust2rpm properly sets the HTTP user agent. That packit does not set a user agent for HTTP requests seems like an oversight to me ...

With the morning's production update, we should now be able to download the tarball. Therefore closing this.


repeated name in body array, Object name = B_first

Hi,
I'm a student and I'm trying to use MuJoCo for my project.
I'm looking for some help with using cable to simulate a cable robot.
Here is a model which explains my question:
<body name="cable1">
    <composite type="cable" curve="s" count="41 1 1" size="1" offset="0 0 .6" initial="ball" >
      <plugin plugin="mujoco.elasticity.cable">
	<!--Units are in Pa (SI)-->
	<config key="twist" value="1e7"/>
	<config key="bend" value="4e6"/>
	<config key="vmax" value="0.05"/>
      </plugin>
      <joint kind="main" damping=".015"/>
      <geom type="capsule" size=".005" rgba=".8 .2 .1 1" condim="1"/>
    </composite>

In my work at first I want to add four cables in my model but when I add the second it gives me an error because the name of body are same. you know when we add a cable composite to model it actually add n (the number of geom) geom and create a cable model and the first geom name will be B_first and the n'th will be B_last, now when I add the second cable composite model I give error below:
Error: repeated name in body array, position 43
Object name = B_first, id = 43
I suppose that this error refer to the name of first and last body of cable but don't know how to fix it.
I will be thankful for any help or suggestion to solve my problem.

This is what you want.


some PVDAQ sites are missing

The # sign in these names causes pandas to treat the rest of the row as a comment, so all of the remaining metadata is set to NaN.
https://github.com/SolarArbiter/solarforecastarbiter-core/blob/5dfed355bee24f2192e169d0a596f2b5e3f97823/solarforecastarbiter/io/reference_observations/sfa_reference_sites.csv#L224-L225
site_file = reference_data.DEFAULT_SITEFILE
all_sites = pd.read_csv(site_file, comment='#')
network_filtered_sites = all_sites[all_sites['network'].isin(network)]

Looks like this applies to 8 systems. Easiest thing seems to be to delete the # and in one case replace it with a space. @alorenzo175 any API/database things to be aware of while fixing this?

I don't think so. The sites didn't get created so should be fine, otherwise I would say we should delete the old ones asap.


Allow rows generated from queries to ignore the query column data types

Currently if you use addRows() or one of the convenience methods (e.g. downloadFileFromQuery()) to generate rows from a query, the cell data type will be set according to the column types in the query (e.g. VarChar, Date etc), rather than the individual cell values.
In most cases this will be fine, but VarChar columns may contain values that are not strings and which may not display properly when typed as strings.
Add an option in each of the relevant methods to ignore the query column types when populating rows and set the type according to the cell value, which is how population methods taking non-query data work (e.g. addRow(), addColumn()).

Solution: add an optional ignoreQueryColumnDataTypes argument to the following methods:
addRows()
binaryFromQuery()
downloadFileFromQuery()
workbookFromQuery()
writeFileFromQuery()

Using ignoreQueryColumnDataTypes may result in data loss.  I wrote a QuerytoCSV UDF and added a similar feature so that I could pass instructions as to how data should be outputted.  Some values are incorrectly detected and cast and it was better IMHO to intentionally cast them so it wouldn't result in any data loss.
Example: A 16 digit varchar string is converted to scientific notation.  (I don't pass CCs, but "4444333322221111" is displayed in Excel as "4.44433E+15" and the underlying data is rounded to "4444333322221110".)  Passing "3.14159265359" returns a rounded 3.141592654, but retains the correct data.  I'm sure there's other examples.
DATE/TIME AS TEXT EXAMPLE: I have client that hosts an annual event and requires millisecond accuracy.  Is there a way to output a varchar column with date/time values like "2017-06-23 18:39:49.206" and and not have Excel strip the accuracy?  (This is one of the primary reasons that we've been using CSV.)  I was previously using a 3rdparty command-line converter and would only have to escape with quotes for the value to be "displayed as text" and retain the accuracy.

I think this due to Excel's limits, which include a max 15 digit precision for numbers.
This and other quirks will apply whenever you add values that aren't in a query object - which is currently the only way of controlling the types ahead of time (by manipulating the query's Column Types).
As I say, ACF does give you the option of specifying which of Excel's limited data types you want to use when adding values. I guess that would give greater flexibility in the end so I should probably just knuckle down and try and implement that instead of this.
On date/time accuracy, I think I answered that before?

Although... being able to specify column types wouldn't solve your original problem, which was to have some values in a column as numbers and others in the same column as strings. Not sure how you could do that without writing your own pre- or post-logic.
Unless the number of likely "quirks" is small enough to write workarounds for them (e.g. if( number.Len() GT 15 ) cellType = "string" )?
What do you think?

Yes, You did answer date/time question before... which is why I wouldn't want that to be blindly converted from VARCHAR to Date or Time.
What are your thoughts on a "data type override"?
On a QueryToCSV UDF that I wrote, I pass an optional struct containing column names and the date type that should be used.  Since I'm outputting a text file, it's not as strict as the POI library where you may need to test the value first and fallback if failed.  For example, if {"score"="numeric"}, but one of the VARCHAR scores is not numeric, you'd need to fallback to string (instead of blank or throw an error).
The benefit is that querydata like this ['1','1','1','-','0','0','0'] (all as VARCHARs) are outputted as "numeric when possible", then you can select the Excel column of data and count 3 and average 0.5.  If the column consists of "number stored as text", you can't perform any mathematical functions.
I believe that this optional level of granularity would enable developers to identify potentially unsafe values as strings to ensure that they aren't converted to scientific notation as well as coerce strings-to-numbers when desired.  (NOTE: Here's a whole thread from 2015 dedicated to the issue that has 1.8k votes on a "started" feature request.

Did you look at ColdFusion's datatype parameter (introduced in ACF11)? This is basically an override you can use when adding single or multiple rows, or columns.
Rather than setting the type per named column (like your UDF), it allows you to specify the datatypes of each column in each row in the case of spreadsheetAddRows(). So you could pass an array of custom row specifications equal to the number of rows in the data. I can't imagine that's something you'd want to do with a big dataset, but it does in theory give you completely granular control.
With spreadsheetAddColumn() you can also set the type for each row.
From some quick testing, it appears to "fallback" in the way you describe, i.e. if you specify NUMERIC for the value "-" it will ignore it and set it to STRING.
But as I've said before, I'm anticipating it being a lot of work to implement, which is why I haven't tackled it up to now, with the exception of setCellValue(), which does have a type option.
Incidentally, it seems my setCellValue() differs from ACF's in that if you specify a type, it will be applied regardless, whereas ACF's will only apply it if the value matches the type (like the fallback above). In the case of setting a single cell value, I think my approach is correct! If the developer has explicitly asked for the value to be cast as a number, then that's what should happen. But it potentially makes trying to implement the datatype option across the board even more complicated.

I really don't like how Adobe implemented it... the format is different than anything else AND requires using positional instructions.  If you use exports that may change fields or field order, this is a huge mess. (The example code appears to be invalid since line 31 only displays "DATE:" and no column number.)
The first patch that I submitted performed a generic upgrade of any VARCHAR values to a blanket NUMERIC, DATE, TIME and/or BOOLEAN for all columns.  (You could select... it's wasn't an all-or-nothing option.)  This approach may not be entirely safe, in my one use-case where I generating query data for an unknown amount of response-related columns, it made sense.
Alternatively my QuerytoCSV UDF uses the data type from getmetadata, but before outputting it checks to see if an override exists (ie, {"Score"="numeric"} and whether the value is valid to export as the type.   If it's not valid (ie, a non-numeric value like empty or '`'), it defaults to the fallback column-type behavior.  No harm done and it's name-based rather than relying on a static, non-changing column order (default) or writing extra logic to determine this and pass it for every row (new CF11 datatype attribute).
I need this as this is the only thing that my clients have complained about when I've used this CFC versus using a Windows CSV-to-XLSX EXE that I was using.

I don't like the ACF implementation either: I think it's complex to use and difficult to replicate - I wonder how many people are actually using it (I can't find it mentioned online other than in the docs). Agree that field order changes could mess up the positional instructions, but OTOH not all data sets have column names (what if I pass an array to addRows()?).
I'm not really sure though what your preference is? This fix #199 does what your first patch does, i.e. lets the default auto-type detection work for queries, ignoring the column types.
But then you're saying your UDF does something different to address other potential issues, but as I say it can only apply to query data with column names, so it's not a universal solution.
Are you happy for me to release this change and then think about addressing problems that remain as a separate fix? Or is it better to hold off and think this through a bit more before committing to anything?

I do think this needs to be applicable to array data as well as queries and have come up with an implementation that does that, while also I hope addressing your concerns. Could you give it a spin on the develop branch? Thanks.

This works correctly. Thanks.


basenc, base64 and base32 have an ABOUT that's formatted differently than the other utils.

For example:
static ABOUT: &str = "
 With no FILE, or when FILE is -, read standard input.

 When decoding, the input may contain newlines in addition to the bytes of
 the formal alphabet. Use --ignore-garbage to attempt to recover
 from any other non-alphabet bytes in the encoded stream.
";

should be:
// Note the backslash here to remove the first newline
//                    |
//                    V
static ABOUT: &str = "\
With no FILE, or when FILE is -, read standard input.

When decoding, the input may contain newlines in addition to the bytes of
the formal alphabet. Use --ignore-garbage to attempt to recover
from any other non-alphabet bytes in the encoded stream.
";

The difference is the backslash and leading spaces.
Discovered here: https://github.com/uutils/coreutils/pull/3014#issuecomment-1027175034

so we have to just add the backslash?

And remove the leading spaces, but yes, it's very straightforward!

@tertsdiepraam can you assign me , I would like raise PR for this?

You can just open the PR with the text "Closes #3022" in the description and it will show up as a PR that closes this issue

@tertsdiepraam i cannot see any leading space here in the code , which space you are talking about

static ABOUT: &str = "
 With no FILE, or when FILE is -, read standard input.

The spaces at the start of each line. The static and With don't line up. The same goes for the other lines.

Ah you already got it, nice!

thanks,  got it , please review this PR @tertsdiepraam


Save buffer on the host when attempting to save it on the guest

Closes https://github.com/atom/teletype/issues/240
This pull request uses the APIs introduced in https://github.com/atom/teletype-client/pull/45 to:

Perform a save request when attempting to save a remote buffer on the guest (d186a3c).
Honor the save request on the host by flushing changes to disk. Requests to save in-memory buffers are ignored by the host. (0edbe49)

In addition, 3a30144 takes care of moving TextBuffer's monkey-patches down into BufferBinding.
/cc: @nathansobo @jasonrudolph

@as-cii: Thanks for putting this together! :zap:
Can I talk you into describing a verification process for these changes?

I tried out these changes locally in dev mode, and I'm seeing a weird interaction with the popular file-icons package:

Host opens a.txt in their workspace

Host shares portal

Guest joins portal and sees editor for a.txt

Host closes a.txt

Guest sees exception:
Uncaught (in promise) TypeError: Cannot assign empty path
    at File.setPath (~/.atom/packages/file-icons/node_modules/atom-fs/lib/resource.js:232)
    at bufferDisposables.CompositeDisposable.buffer.onDidChangePath.to (~/.atom/packages/file-icons/node_modules/atom-fs/lib/file.js:185)
    at Function.module.exports.Emitter.simpleDispatch (~/github/atom/node_modules/event-kit/lib/emitter.js:25)
    at Emitter.module.exports.Emitter.emit (~/github/atom/node_modules/event-kit/lib/emitter.js:141)
    at BufferBinding.restoreOriginalBufferMethods (~/github/teletype/lib/buffer-binding.js:63)
    at BufferBinding.dispose (~/github/teletype/lib/buffer-binding.js:22)
    at bufferDestroySubscription.buffer.onDidDestroy (~/github/teletype/lib/buffer-binding.js:40)
    at Function.module.exports.Emitter.simpleDispatch (~/github/atom/node_modules/event-kit/lib/emitter.js:25)
    at Emitter.module.exports.Emitter.emit (~/github/atom/node_modules/event-kit/lib/emitter.js:141)
    at TextBuffer.module.exports.TextBuffer.destroy (~/github/atom/node_modules/text-buffer/lib/text-buffer.js:1503)
    at TextBuffer.module.exports.TextBuffer.release (~/github/atom/node_modules/text-buffer/lib/text-buffer.js:1553)
    at TextEditor.destroy (~/github/atom/src/text-editor.js:593)
    at lastEditorProxyChangePromise.lastEditorProxyChangePromise.then (~/github/teletype/lib/guest-portal-binding.js:86)

I don't observe this exception when using the current Teletype release (0.3.0).

Do you see those errors when you run the tests locally?

I am seeing those errors on master as well. I will look into them shortly.

I tried out these changes locally in dev mode, and I'm seeing a weird interaction with the popular file-icons package:

I ended up going back to monkey-patching everything in EditorBinding, as there seem to be issues when clearing the path of a buffer and emitting a did-change-path event after doing so. I've added some verification steps in the body of this pull request and I will go ahead and merge these changes into master as soon as we get a green build.

Thanks, this is great!


select box style issue in Firefox 58

1.The select box text is cutoff

Select optgroup text has the same color as background color

The text color and background should be fixed now. As far as text being cut off I can only assume it's the font-size that is at fault.
If you create a pull request with a fix for this I will gladly accept it! Thanks!

This has been fixed!


Graph API: Sharepoint sites pages API @odata.nextLink stops after a few times, no error

Target SharePoint environment
other: Python code, call to Graph API: https://graph.microsoft.com/beta/sites/{siteId}/pages/
What SharePoint development model, framework, SDK or API is this about?
Sharepoing Graph API:
other: https://graph.microsoft.com/beta/sites/{siteId}/pages/
Developer environment
Windows
What browser(s) / client(s) have you tested

[ ] 💥 Internet Explorer
[ ] 💥 Microsoft Edge
[ ] 💥 Google Chrome
[ ] 💥 FireFox
[ ] 💥 Safari
[ ] mobile (iOS/iPadOS)
[ ] mobile (Android)
[X] not applicable
[X] other (enter in the "Additional environment details" area below)

Additional environment details

python code to call Graph API
Databricks notebook

Describe the bug / error
I am not getting all the files available.
Below is my code to ask all files. I use<EMAIL_ADDRESS>to make the next requests.
  def process_pages(accessToken, siteId) -> DataFrame:
      items = []
      headers = {"Authorization": f"Bearer {accessToken}"}
      next_link = None
      while True:
          try:
              url = f"https://graph.microsoft.com/beta/sites/{siteId}/pages/?top=100"
              if next_link:
                  url = next_link  # Use the nextLink if available
              print(url)
              response = requests.get(url, headers=headers)
              response.raise_for_status()

              data = response.json()
              items.extend(data.get('value', []))
              next_link =<EMAIL_ADDRESS>              if not next_link:
                  break
          except Exception as e:
              print(f"Error: {str(e)}")
      return items
  token = connector.get_access_token(TENANT_ID, CLIENT_ID, CLIENT_SECRET)
  items = process_pages(token, SITE_ID)

When i use ?top=50, I get 166 total files. ?top=100 gives 234, ?top=200 gives 472, ?top=300 gives 384 and ?top=400 gives 654. From 500 it gives me a timeout error.
Everything looks normal, the last request does not give any error (200 OK). There is just a lack of nextLink in the response.
This looks like a bug to me, unless someone can point out what I am doing wrong.
Putting a time.sleep(60) in between does not have any effect.
Steps to reproduce

Add a lot of site pages (500+)
Make a request with ?top=xx
Keep making requests to the<EMAIL_ADDRESS>property until not available anymore
Count total pages
Repeat 2-4 and see how the total pages differ.

Expected behavior
Get all files, same amount of files with different batch sizes (different top query parameter).

@nick-pape Could someone look at this bug? I am still stuck on this.

@nick-pape Could someone look at this bug? I am still stuck on this more than three months later.

@nick-pape  @VesaJuvonen
Could someone add the "area:pages" label, which seems to be appropriate here, I believe?
Still waiting for a solution 6 months later.
Thank you.


Can't set temperature for heating less than 18°

I have several rooms, where I want to heat in winter only up to a temperature of 15. And also in one room I have to set the temperature for heating with an offset of -3. So if I want 20 I have to st it to 17 because of the geometry of the room. But setting a temperature less than 18 from node-red results in an error
Error: Value (15) must not be smaller then 18
As far as I understand the code, all three temperatures (heating, cooling and auto) are set regardless which mode is active. When I look at the possible ranges I see that for heating 10..31, for cooling 18..33 and for auto 18..30 is possible.
{"/operationModes/heating/setpoints/roomTemperature":{"settable":true,"value":15,"unit":"°C","stepValue":0.5,"minValue":10,"maxValue":31},"/operationModes/cooling/setpoints/roomTemperature":{"settable":true,"value":30,"unit":"°C","stepValue":0.5,"minValue":18,"maxValue":33},"/operationModes/auto/setpoints/roomTemperature":{"settable":true,"value":30,"unit":"°C","stepValue":0.5,"minValue":18,"maxValue":30}}
So setting in heating mode to a temperature of 15 results in an error because Daikin does not allow a cooling and auto of less than 18. In my opinion the temperatureControl should only be set to the active mode of the unit to allow heating also in a range from 10 to 18 degree.

Hi @EriHau could you please test the latest release 0.4.0. There is now a config switch in the node config where you could disable the temperature sync feature.

Afterwards please make sure to set msg.payload.dataPointPath to "/operationModes/heating/setpoints/roomTemperature" or others when you want to switch the temperature. It is no longer possible then to switch temperature without dataPointPath set. If you want to set it for all three you need to make this for all 3 modes seperately.

It works perfect! Thank you for the quick and easy solution.


abysmal, capretto

This pull request was generated by the 'mq' tool
[test]
flake rate: 0.4
logical conflict every: 100
sleep for: 300s
[pullrequest]
requests per hour: 30
deps=[a,c]

/trunk merge


Schema Writer - support enums

Add support for emitting enum declarations from the FlatBuffersSchemaWriter.

Dependency for #20


Aparat Downloads Lowest Res by Default

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like this: [x])
Use the Preview tab to see what your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2017.08.09. If it's not, read this FAQ entry and update. Issues with outdated version will be rejected.

[x ] I've verified and I assure that I'm running youtube-dl 2017.08.09

Before submitting an issue make sure you have:

[ x] At least skimmed through the README, most notably the FAQ and BUGS sections
[ x] Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

[x ] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
$ youtube-dl http://www.aparat.com/v/XeL6n -v
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['http://www.aparat.com/v/XeL6n', '-v']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2017.08.09
[debug] Python version 3.4.5 - Linux-3.10.0-514.26.1.el7.x86_64-x86_64-with-centos-7.3.1611-Core
[debug] exe versions: ffmpeg 2.6.8, ffprobe 2.6.8
[debug] Proxy map: {}
[Aparat] XeL6n: Downloading webpage
[Aparat] XeL6n: Testing video URL 0
WARNING: unable to extract isFamilyFriendly; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
[debug] Default format spec: bestvideo+bestaudio/best
[debug] Invoking downloader on 'http://as6.cdn.asset.aparat.com/aparat-video/04905cf4a6e499c617161464d1f11f997736172-144p__81483.apt?direct=1'
[download] Destination: Nintendo 64 U.S. Unveiling @ E3 '96  in  Los Angeles, California-XeL6n.mp4
[download] 100% of 15.36MiB in 00:03

Description of your issue, suggested solution and other information
Hello, third GitHub issue, but anyway. Aparat downloads by the lowest res (144p) by default,and i want to, let's say, get the highest resolution available. Here's an example video:
http://www.aparat.com/v/XeL6n
Thanks.

whoops realized youtube-dl was outdatted--sorry--i edited the log

I was supposed to open this issue one more time because it's not working.
I've Already described the problem via a comment on the merged code.
[https://github.com/ytdl-org/youtube-dl/commit/70851a95c307880f016fcb6f37427a8eeae73cad#commitcomment-39029171](here is the link)

@dstftw
I Found that Aparat isn't working correctly but someone has changed it a bit to work, you can the code below.
https://raw.githubusercontent.com/HanifBirgani/youtube-dl/fix/aparat/youtube_dl/extractor/aparat.py


New package: WGPUCanvas v0.1.1

Registering package: WGPUCanvas
Repository: https://github.com/JuliaWGPU/WGPUCanvas.jl
Created by: @arhik
Version: v0.1.1
Commit: 5ab9d523f6f0f9296e52ff9a17be80cdc61eef93
Reviewed by: @arhik
Reference: https://github.com/JuliaWGPU/WGPUCanvas.jl/commit/5ab9d523f6f0f9296e52ff9a17be80cdc61eef93#commitcomment-132648359
Description: Graphics Interface Library for JuliaWGPU

https://github.com/JuliaWGPU/WGPUCanvas.jl/commit/5ab9d523f6f0f9296e52ff9a17be80cdc61eef93#r132750414


clean up pages and vendor assets

Summary of Changes

Vendored GitHub/Colab badges as that significantly increased page load time when pulling from URL
Re-organized pages to be in their own folder
Moved favicon into assets

/preview


Setup a package for ODF extensions

Refactors ODF Dashboard
Signed-off-by: Bipul Adhikari<EMAIL_ADDRESS>Screenshot:

/test odf-console-e2e-aws

/test odf-console-e2e-aws

/lgtm

/test odf-console-e2e-aws

@bipuladh just to confirm once, did u test create/details page routes for noobaa resources as well ??
I think "Create" button on list page is still pointing to "/odf/resource/..."

@bipuladh just to confirm once, did u test create/details page routes for noobaa resources as well ?? I think "Create" button on list page is still pointing to "/odf/resource/..."

same for breadcrumbs on details pages...

@SanjalKatiyar this PR is getting confusing I will send another PR with route changes for now reverting those changes.

/lgtm


Question about Panics in spacepackets-rs

Hello!
I've been quite interested in a rust implementation of spacepackets and have enjoyed using your project.
I have been grokking the codebase of the Rust spacepacket implementation and noticed the use of panics in the builder methods, specifically in the const_new and new functions of various structs. While I understand that panics are a part of Rust's error handling mechanism and can be useful for catching exceptional cases, I have concerns regarding their usage in these particular scenarios.
First, in the UnixTimestamp struct, the const_new function panics if the subsec_millis value exceeds 999. Similarly, in the PacketSequenceCtrl struct, both the const_new and new functions panic if the seq_count value exceeds MAX_SEQ_COUNT. Although panics can be appropriate for handling unexpected or invalid inputs, I'm unsure whether they are the best approach in these cases. I do understand for the latter example that these panic conditions are checked before calling, so they won't panic in those cases.

is this pattern, const_new and new, common for Rust 2021 projects? Would there be preferable ways to call const_new without requiring the caller to check the panic conditions?
what is the rationale for checking  subsec_millis greater than 999? Would there be cases where the value may be user-specified?

Thank you,
Ethan

Hmm, I don't think const_new is something that common. This is just a variant which allows creation of structs in const contexts, which might be useful for some cases. The caveat here of course is that I can not perform regular error handling, so I don't really have any other choice than to panic on invalid/unsanitized input.
About the subsec_millis being greater than 999: It is assumed that the subsec_millis only contain the millisecond part on top of the full UNIX seconds, so a value greater than 999 would be  an additional second.

Big changes for everything you mentioned are coming in version 0.11.
I renamed the const_new methods to new and renamed the former new methods to new_checked. This is more in line with how it is done in the Rust ecosystem.
The new UnixTime timestamp type now consists of a second part (i32) and a nanoseconds part (i64). The same rules still apply though: A nano second value larger or equal to  1_000_000_000 is considered invalid.

@robamu great to know, thanks!
Also, I will close this issue as my questions have been answered (a long time ago :) )


Show only current visible month dates

I want to show only current visible month dates and exclude other dates from previous and next month
I want to use some simple method to do this task.

You can do that just by applying the following css:
  .react-datepicker__day--outside-month {
    color: white;
  }

It will make the days outside of the current month "white" meaning that they will not be visible. It worked for me, hope it will work for you.

You can do that just by applying the following css:
  .react-datepicker__day--outside-month {
    color: white;
  }

It will make the days outside of the current month "white" meaning that they will not be visible since it will have the same color as the background. It worked for me, hope it will work for you.

I used maxData and minData props.. It worked for me without changing any css. Btw Thanks for the help.

You can simply add prop
showOutsideDays={false}


ERROR :: plex.py:48 :: There was an exception in process_to_db

New install on Windows 10, and things have been running well.  Just got this error, though, and wanted to report it and make sure it wasn't anything to be concerned about.
2019-01-14 10:33:17,313 :: bw_plex :: DEBUG :: plex.py:657 :: Checking if we can reach The.Real.Housewives.of.Atlanta.s11e11 directly
2019-01-14 10:33:17,313 :: bw_plex :: WARNING :: plex.py:670 :: Downloading from pms..
2019-01-14 10:33:17,315 :: bw_plex :: DEBUG :: misc.py:260 :: Calling find_offset_ffmpeg with command ffmpeg -i http://<IP_ADDRESS>:32400/library/parts/706409/1547479942/file.mkv?download=1&X-Plex-Token=<hidden> -t 600 -vf blackdetect=d=0.5:pix_th=0.1 -af silencedetect=n=-50dB:d=0.3 -f null -
2019-01-14 10:33:33,279 :: bw_plex :: DEBUG :: misc.py:195 :: final_video [['00:00', '00:02', '00:02'], ['09:50', '09:52', '00:02']]
2019-01-14 10:33:33,279 :: bw_plex :: DEBUG :: misc.py:196 :: final_audio [['00:00', '00:01', '00:01'], ['00:06', '00:07', '00:00'], ['03:18', '03:19', '00:00'], ['09:49', '09:52', '00:02']]
2019-01-14 10:33:33,280 :: bw_plex :: DEBUG :: misc.py:198 :: fin v [[0.0, 2.002, 2.002], [590.006, 592.133, 2.127]]
2019-01-14 10:33:33,280 :: bw_plex :: DEBUG :: misc.py:199 :: fin a [[0.0, 1.94796, 1.94796], [6.29579, 7.21585, 0.920063], [198.887, 199.409, 0.522229], [589.922, 592.084, 2.16206]]
2019-01-14 10:33:33,280 :: bw_plex :: DEBUG :: misc.py:227 :: Matching windows are [['09:50', '09:52', '00:02']]
2019-01-14 10:33:33,282 :: bw_plex :: DEBUG :: misc.py:765 :: Checking if The.Real.Housewives.of.Atlanta.s11e11 has a recap with phrase previously on, last season, last episode using subtitles
2019-01-14 10:33:33,298 :: bw_plex :: DEBUG :: misc.py:704 :: Downloading subtitle from PMS
2019-01-14 10:33:34,398 :: bw_plex :: DEBUG :: plex.py:657 :: Checking if we can reach The.Real.Housewives.of.Atlanta.s11e11 directly
2019-01-14 10:33:34,400 :: bw_plex :: WARNING :: plex.py:670 :: Downloading from pms..
2019-01-14 10:33:34,400 :: bw_plex :: DEBUG :: credits.py:312 :: Trying to find the credits for http://<IP_ADDRESS>:32400/library/parts/706409/1547479942/file.mkv?download=1&X-Plex-Token=<hidden>
2019-01-14 10:33:34,838 :: bw_plex :: ERROR :: plex.py:48 :: There was an exception in process_to_db
Traceback (most recent call last):
  File "c:\bwplex\src\bw-plex\bw_plex\plex.py", line 43, in inner
    return func(*args)
  File "c:\bwplex\src\bw-plex\bw_plex\plex.py", line 137, in process_to_db
    check=-1)
  File "c:\bwplex\src\bw-plex\bw_plex\credits.py", line 324, in find_credits
    recs = locate_text(frame, debug=False)
  File "c:\bwplex\src\bw-plex\bw_plex\credits.py", line 244, in locate_text
    rect = cv2.boundingRect(contour)
error: OpenCV(4.0.0) C:\projects\opencv-python\opencv\modules\imgproc\src\shapedescr.cpp:741: error: (-215:Assertion failed) npoints >= 0 && (depth == CV_32F || depth == CV_32S) in function 'cv::pointSetBoundingRect'`

Fixed.


podvm/github-action: Support building public s390x qcow images

The cd-rom option for qemu doesn't work for s390x as it can't auto-detect the layout. Uses virtio-scsi instead works for both s390x and amd64 for the cloud-init data. Addtionally to make sure it boots from the qcow2 drive instead of the cloud-init, need to explictly define the device and drive.
Fixes: #512
Signed-off-by: James Tumber<EMAIL_ADDRESS>

Can check docker.io/jamesnelson/podvm-ibmcloud-ubuntu-amd64 and docker.io/jamesnelson/podvm-ibmcloud-ubuntu-s390x for test runs of this change

@jtumber-ibm - Sorry to ask a simple question, but after this are we going to make https://quay.io/repository/confidential-containers/podvm-ibmcloud-ubuntu multiarch, or have two separate images for amd64 & s390x?

@jtumber-ibm - Sorry to ask a simple question, but after this are we going to make https://quay.io/repository/confidential-containers/podvm-ibmcloud-ubuntu multiarch, or have two separate images for amd64 & s390x?

No worries it is a good question. I think not - as there is no intention that this is a runnable image so having the convenience of the manifest list might lead people to mistakenly use the wrong image.


Notif Name + Location

Notification Settings, adding the Location of the Merchant would be very useful. Dorella (Feiton) etc.

To clarify, you want to see the general continent/region the merchant is in?
Because the location should already be in there: