Add a prometheus label mapping component

PR Description
This PR add a prometheus component to create a label based on a source_label value and a static mapping table.
Which issue(s) this PR fixes
For a large mapping table, using regex with prometheus.relabel is inefficient
Notes to the Reviewer
PR Checklist

[x] CHANGELOG.md updated
[x] Documentation added
[x] Tests updated

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Thanks for contributing.

For a large mapping table, using regex with prometheus.relabel is inefficient

Could you share the use case and performance differences? In many cases users can use discovery.relabel to add the team=X labels - and then the overhead is much smaller, because it's not samples, but targets that are relabeled.
If the reason to add this is that prometheus.relabel it's not efficient, we will need some more data on how inefficient it is and whether it could be optimised to work faster. Any new solution we would consider will require benchmarks to prove that it's more efficient. We also generally want to avoid having many ways of doing the same thing - but there can be exceptions.

Thanks for sharing some performance data. Have you tried to use discovery.relabel instead?
The issue with prometheus.relabel is that you are relabelling every single sample. With discovery.relabel you can relabel the targets before they are scraped.
So, for example, in a cluster of 1k targets and each exposing 1k metrics, you would have 1 million relabels with prometheus.relabel as opposed to 1k relabels with discovery.relabel.
If we find that for some reason discovery.relabel cannot be used, I would like to understand better why. If we need to optimise prometheus.relabel I would like to explore that option too before committing to create a new feature to do a very similar job. We may also need to go through proposal process to let team/people comment on this.

Thanks for your answer.
I'm not using alloy to scrape targets but I'm receiving data thru remote_write from multiple data producers.
The data producers only have the information about the service_id but not the tenant_id, therefore they cannot apply the correct label at the scraping time.
I'm unable to see how we could optimize prometheus.relabel to be as performant as a key lookup. We have made some attempt to share cache between instances https://github.com/grafana/alloy/pull/1692 which aims to lower the memory consumption.
Here is a extract of my test configuration, using prometheus.relabel:
prometheus.receive_http "ingestion_head" {
    http {
        listen_address = "<IP_ADDRESS>"
        listen_port    = 9000
    }
    forward_to = [prometheus.relabel.allow_list.receiver]
}

prometheus.relabel "allow_list" {
    forward_to = [prometheus.remote_write.ingestion_tail.receiver]
    rule {
        action        = "keep"
        regex         = "(.+)"
        source_labels = ["service_id"]
    }

    // One rule block per service_id
    rule {
        source_labels = ["service_id"]
        regex         = "^000001$"
        target_label  = "tenant_id"
        replacement   = "tenant_42"
    }

    rule {
        source_labels = ["service_id"]
        regex         = "^000002$"
        target_label  = "tenant_id"
        replacement   = "tenant_190"
    }

    // ...

    rule {
        source_labels = ["service_id"]
        regex         = "^12345678$"
        target_label  = "tenant_id"
        replacement   = "tenant_8876"
    }
}

prometheus.remote_write "ingestion_tail" {
    endpoint {
        url = "http://mimir/"
    }
}

The prometheus.relabel can be fairly heavy on the resources especially when we do a 1:1 match, hence the proposal to have a mapping. Here is an example of a working proof of concept based on the code in the PR.
Here is a schema:

prometheus.receive_http "ingestion_head" {
  http {
    listen_address = "<IP_ADDRESS>"
    listen_port    = 9000
  }
  forward_to = [prometheus.relabel.allow_list.receiver]
}

prometheus.relabel "allow_list" {
  forward_to = [prometheus.mapping.tenants_mapping.receiver]

  // First ditch everything that don't have service_id
  rule {
    action        = "keep"
    regex         = "(.+)"
    source_labels = ["service_id"]
  }
}

prometheus.mapping "tenants_mapping" {
  forward_to = [prometheus.remote_write.ingestion_tail.receiver]

  src_label_name = "service_id"
  output_label_name = "tenant_id"

  mapping = {
   "000001" = "tenant_42",
   "000002" = "tenant_190",
   // ....
   "12345678" = "tenant8876",
  }
}
prometheus.remote_write "ingestion_tail" {
        endpoint {
                url = "http://mimir/"
        }
}

What we forgot to explicit with @vaxvms is that our initial design would benefit from https://github.com/grafana/alloy/issues/521 we would remove the need for cortex-tenant

@vaxvms Did the doc topic at components/prometheus/prometheus.relabel accidentally fall off this PR in one of the force/push updates? Or was it removed intentionally? I don't see it in the file list anymore.

Are you talking about components/prometheus/prometheus.mapping ? I accidentally removed it. It back again.
I've updated the way the component work: The target_label isn't fixed anymore. each value of the source label can be mapped to severals labels

Thanks for review @clayton-cornell
Most of the phrasing came from prometheus.relabel.md

Most of the phrasing came from prometheus.relabel.md

Yeah that topic needs some attention too. I've got an open task to review and refactor the component docs.

@dehaansa @clayton-cornell @thampiotr Is here anything I have to fix/document/explain/change ?

So I have to wish them to have a nice holiday !


adding new techniques (part 2)

@UlrikeS91 this is ready now for a review :)

@apdavison We can yes. For now I was just focusing on terms that were already present in some way in the old system. We need to go as deep as it would make sense for queries. I think large/medium/small makes sense. is large-scale not a synonym to HPC? or are there large-scale simulations that can run without an HPC?

Hi Lyuba, I think "small", "medium" and "large" are problematic, because there is no agreed upon definition for how to define the scale of a simulation (number of neurons, number of synapses, volume of tissue simulated, number of processor-hours required...?), and for sure the boundaries change with time (what was large-scale 10 years ago is now medium- or small-scale).
I suggested "HPC simulation" as a more concrete, well-defined term that is approximately equivalent to the intent of "large-scale", but really I would be inclined to just have "simulation".

@apdavison just "simulation" seems to vague for me to be of real use. I've entered "large-scale simulation" because it has a meaning to people from the field (discussed it with some NEST people), but HPCsimulation would make them also happy (I guess). Could you have a look at https://search.kg.ebrains.eu/instances/069e4718-2b4a-4056-9c8f-e646a841b2dd and tell me which controlled techniques would you filter out of the given methods? (I've kept now "communication profiling", "large-scale simulation", "quantitative analysis" and "qualitative analysis")

@apdavison just "simulation" seems too vague for me to be of real use. I've entered "large-scale simulation" because it has a meaning to people from the field (discussed it with some NEST people), but HPCsimulation would make them also happy (I guess). Could you have a look at https://search.kg.ebrains.eu/instances/069e4718-2b4a-4056-9c8f-e646a841b2dd and tell me which controlled techniques would you filter out of the given methods? (I've kept now "communication profiling", "large-scale simulation", "quantitative analysis" and "qualitative analysis")

The PR is basically ready, but @lzehl and I would like to wait for @apdavison feedback about the question around the "simulation" technique. @apdavison would you be able to have a look and tell us which techniques would be suitable?

Having now reviewed the level of detail in the other "technique" instances, I definitely think that plain "simulation" would be best. If you want more specificity, then I would put both "HPC simulation" and "neuromorphic simulation" for that specific dfataset. "Large-scale" should be avoided, as the term is subjective and doesn't have a stable meaning across the field or across time (i.e. different people have a different idea about what it means, and people's idea of its meaning changes as computing technology improves).
I would be inclined to remove "communication profiling", as this is a technique from computer engineering, not from neuroscience.

I've also updated the image registration techniques and the transformation techniques according to the feedback I received from Timo.


qdiscs: Fix extra backslashes in commands examples

These extra backslashes were added when copying from the original
script.
Signed-off-by: Vinicius Costa Gomes<EMAIL_ADDRESS>

This PR looks good to me. Since it is very simple I'll go ahead and merge it now. Thanks @vcgomes!


Fix deleteGJAccComment20.php

The page for deleteGJAccComment20.php was inaccurate and missed the targetAccountID as a required parameter.

I also changed a couple other small things

now adding 2.207 stuff

If you wanna know, the event level response is not the same as the weekly and daily one:
>>> req = req.post(...) # get event level
>>> req.text
200001|10|VSIAccw1AW2EPDgAKBwINCwoCDwoLAB4BCR0OHgQVAAseBA==|956d9d1de892597b92a988570c20002dc0e8831e

Also, the binary version is 44, not 43

Also, also, you named the getGJsecretRewards endpoint's doc file to getGJsecretRewards.php instead of getGJsecretRewards.md

For whatever reason if I omit the udid parameter then valid codes always return -1 (I only tested with wellmet and thickofit but it's probably true for the rest of the known codes)

About getting icons as rewards:
This is server response for "thickofit": dbycz:696969:18:1:7,1
This is server response for "bussin": IXfrK:696969:17:1:7,69
This is server response for "touchgrass": HKjcW:690626:19:1:10,1
As we can see, on thickofit, which grants 1 orb, bussin, which grants 69 orbs, and touchgrass, which grants one Earth shard, there are five colon-separated values (7 if you include the commas, which I do.)
            | thickofit    | bussin    | touchgrass
            | ------------ | --------- | ----------
Unknown     | 1            | 1         | 1
Reward Type | Orb          | Orb       | Earth Shard
Amount      | 1            | 69        | 1

However, this is server response for "backondash": L4DRv:696969:6:2:1001,409
Here we can see, that Unknown is now 2 instead of 1. And the Reward ID is 6.
Now for the reward field, there is "1001,409". Too lazy to check what this means for now, but it's probably the icon reward.

I did some research, the golden key key is 43 in GS_value. However gd overwrite my changes to this key but i'm SURE that key 43 is for golden keys, so gd seems to overwrite it
Strange

For some reason I also needed to change the amount of golden keys the wraith reward gave me for gd to actually register a change in the amount of golden keys I had


Potential Bug Risks and Anti-Patterns

Description
Hi 👋
I ran DeepSource Static Code Analysis upon the Project, the results for which are available here.
The Static Code Analysis Tool found potential bugs and anti-patterns in the Code, that can be detrimental at a later point in time with respect to the Project. DeepSource helps you to automatically find and fix issues in your code during code reviews. This tool looks for anti-patterns, bug risks, performance problems, and raises issues.
Some of the notable issues are:

Function literal can be simplified here
Empty fallthrough can be avoided here
Empty String Test can be improved here
Calls to exit/fatal made in function using defer here
Shadow of builtin detected here

There are plenty of other issues in relation to Bug Discovery and Anti-Patterns which you would be interested to take a look at.
If you would like to integrate DeepSource to autofix some of the commonly occurring issues, I can help you set that up :)

I didn’t know about DeepSource before, but from the “issues” it seems to be another linter, right?
I’m not really sure if these are worth fixing tbh. I wouldn’t be opposed either, but some of these suggestions seem to be more a matter of taste than anything else (for example, the empty string comparison).


Cant access lightzoneproject.org

It was error somethong about botnet and logging my actions. I did not save the  page
not just Error 503 : Service Temporarily Unavailable

Sorry, this issue list is for software itself, but not for the website. If the problem continues, please contact<EMAIL_ADDRESS>directly. Recently the website had been attacked by spammers, and I guess that was the cause of this problem.


Automatization of LaTeX/pdf output.

Automatize classical LaTeX ouput using QaDiL/qadil/toLaTeX.py minimizing human intervention. A suitable zip-file:  QaDiL/Notes/IMO21/Kompendium.zip has been added (pull)  with further details. This zip file contains the LaTeX version of the IMO21 notes curated by a human (after being processed by toLaTeX.py).

Can you provide a short example of how this works? The makefile in Kompendium does not seem to work. How should the script QaDiL/qadil/toLaTeX.py be run?


doesn't work with cli generated project

Running Angular v4 generated by cli.  Directive is installed but does nothing, no error. Any ideas?

@rob-balfre Did you sove this? I'm new to angular-cli and don't know how to add external directives

Ended up not using it.
This is all you need mate.  Generate a directive using the CLI and then paste in this gist.
https://gist.github.com/rob-balfre/0c989ed6a4e131a09a684dabb649e0cd

I found that this works for chrome with the ng cli, but not in firefox

@rob-balfre : well, your example works if I have only one of the kind...
in my case I have 3 dropdown components, all have the same clickOutside directive, but it gets triggered only when clicking anywhere outside of the dropdowns.... so if I have one open.... and click outside it gets triggered..... but if I have one open... and click on another.... it doesn't get triggered and both are now open

@demiro I have the same exact issue with multiple dropdowns. Have been trying for a few hours now.

I'm not sure if it's the same issues outlined above by @demiro and @fahdi but the directive doesn't seem to like being applied to elements that are inside an *ngFor loop, specifically all clicks are registered as "outside clicks", anyone experienced this or have a work around?

doesnt work on my non-CLI project.

it doent work because it bundled to SystemJs
my work around is do to:
import { ClickOutsideDirective } from 'angular2-click-outside/clickOutside.directive';
and adding the directive to your module declarations.

That actually would probably solve it, but I ended up just using this one which seems to work. https://www.npmjs.com/package/ng-click-outside


Generator: Update SDK /services/iaasalpha

Generated from GitHub run<PHONE_NUMBER>6

There is a newer PR with the same changes https://github.com/stackitcloud/stackit-sdk-go/pull/992


Fix: Tutorial Pills Should Scroll On Overflow

What changed? Why?

added overflow scroll for tutorial pills

Notes to reviewers
How has it been tested?

🟡 Heimdall Review Status

Requirement
Status
More Info

Reviews

🟡
0/1

Denominator calculation

Show calculation

1 if user is bot
0

1 if user is external
0

From .codeflow.yml

1

Additional review requirements

Show calculation

Max
0

0

From CODEOWNERS

0

Global minimum
0

Max

1

1

1 if commit is unverified
0

Sum
1


DOCS-#3904: Improving Modin README

Signed-off-by: Naren Krishna<EMAIL_ADDRESS>
What do these changes do?
Fix links in the Modin README and improve some of the wording in both the FAQ and README.

[x] tests added and passing
[x] module layout described at docs/developer/architecture.rst is up-to-date

I'd probably mention explicitly somewhere that performance effect could better be seen on large datasets and on small data sets perf can be worse than pandas

We also miss mentioning of current conda-forge version

@naren-ponder : Once we finalize the README content (add GIF and TOC), can we also migrate the same content over to our home page for ReadTheDocs?


 content not rendered in Chrome for W7

When I try to copy a simple SVG, everything is fine excepting the TEXT tag that is rendered in Chrome (v34 for Windows 7) with a very small font size.
Rendering is OK with Firefox and with Internet Explorer.

Workaround : force a font-size attribute in the TEXT element.
Anyway the font-family is still not handled correctly in Chrome for Windows.
Another convertor seems to suffer the same way: http://bl.ocks.org/biovisualize/raw/8187844/


Drop Frame Support for Timecode

This update provides for some fixes in the progress tooltip, to show full length timecodes, as well as providing the calculations required for supporting drop frame timecodes (e.g. fractional frame-rate videos using NTSC SD standards).  Indicates dropframe using semi-colon, e.g. 00:00:00;00.  I've had this in MEJS since version 2, but updated it for 3.  One note: due to how MEJS sets up features, this particular item requires the player to be re-initialized if changing video between DF and non-DF videos in the same player.  MEJS is outstanding so I felt compelled to share this with everyone, if you're interested please review and merge.  Enjoy.

Thanks for your work on this PR; is people like you that makes this plugin outstanding. This work LGTM but I'll need you to fix the unit tests so I can completely approve this PR and also answer the question posted above.

Or if you are not opposed I can make the changes for you

Thanks @ron666.  The .066666 represents the number of drop frames to drop on the minute marks.  Essentially 6%.  You need to calc it as a decimal like this for accuracy before turning it into an integer.
The conversions are based on pseudocode from:
//Code by David Heidelberger, adapted from Andrew Duncan
While there is no explicit license on his blog, http://www.davidheidelberger.com/blog/?p=29 I did want to ask you about properly attributing this (I figured a readme.txt addition would be appropriate).  Let me know how you think we should handle it.
With regards to the tests, if you want to fix them up, that would be fine, my answers to the three failed ones are:  1) is typical of what the frame index is as a counter, where 0 isn't the first frame it's no frame, and the first frame is 1.  2)  I suspect this is caused by  the format conversion being incomplete.  SMPTE is usually seen in it's four-pair form.  3)  For frames, it doesn't make much sense to have fractional (.decimal) frames.  In SMPTE while it looks like time, the purpose is simply to give each frame a unique identifier.  The dropframe support is to make it align with a realtime clock, but the purpose is just labelling.  So the toFixed functions were set to (0).  If people wanted MM:SS and MM:SS.sss, that makes sense, so perhaps what I changed hampers that ability?
Let me know if you need more info., or if I really need to get my hands dirty on the tests.

On a side note, what's the best way to run the tests in my local environment for MEJS?  Looking at the tests themselves, I can probably come up with something a bit more robust, and make sure the drop frame side gets tested properly.

@ron666 - Regarding the tests:
The first test which failed is due to rounding errors when creating the test values.  The actual values that come up, are off by a frame due to the original code probably truncating the value instead of rounding.
The second test revealed a minor bug where I returned frames instead of seconds (and it would partially affect the third test).
The third test, "can show numeric value with decimals when fps is indicated" is no longer valid since I return toFixed(0).  We should ask the community perhaps if anyone needs the FF component with .decimals?  To keep the feature we could adapt the timeformat option to allow for HH:MM:SS:FF.sss, or simpler yet, add an option for ffNumberOfDecimals, which would just go directly into the toFixed() call.  I could use 0 they can use 3, default 0.  This choice will determine how the third failed test gets handled.
Should we cancel this PR, and I can put up the fixed bug, along with the updated test values?  We could then discuss implications/resolution for the .decimal function of timeCodeToSeconds and the third test?

Yeah I agree let's bring the fix that you mention and to run the test use npm install and then n run test

Sorry npm run test

Coverage increased (+0.5%) to 45.855% when pulling 79db7faf391748eb5d2a288228ff5e3caaf8e552 on dmongrel:master into ae819c67b691cc5a2bd87b8cb5422fbe77e8a5a6 on johndyer:master.

@ron666 Looks like everything passed.  Once I can get npm to run the test in my environment I'll create some more specific dropframe-related tests.  That really only leaves the topic regarding FF.sss in the display and whether to leave it off or create an option for it.

I'd say add it as an option and with that this PR will be covered

OK, I will look into it Sunday.

Sounds like a plan. Let me know when you are ready to review this

@ron666 Not sure how the builds became a conflict, but the final changes I've uploaded add a user option:  secondsDecimalLength.  If frames are NOT shown, then the seconds value will show the number of decimal places.  Default = 0, no max, though 2 or 3 is probably optimal.  If frames are shown then the values have 0 decimal places.

Thanks for your work on this. I'll review it one more time, fix the issues and merge

PR merged. Thanks for all the work you did on this


Explicit return type not required in TypeScript anymore

Description
Not sure when this changed, but we used to enforce an explicit return type. We're no longer doing that, and we should re-enable enforcing it it.
Steps to reproduce

In the code, remove the return type from a function

Expected results

Linter should fail, requiring a return type to be specified

Actual results

Linter does not fail

I think previously one of the configs we extended had explicit-module-boundary-types enabled. This is for anything exported and the benefit is it makes it more clear if a change was made to a potentially public facing API
There is also explicit-function-return-type which I don't think we had enabled. This requires it for all functions, even if they are not exported


SSR nuxt

Hi does this work with nuxt / ssr?
I've got
document is not defined

Not sure if I implemented the virtual grid in the right way
  return styleElement
}

function addStyle (obj /* StyleObjectPart */) {
  var update, remove
  var styleElement = document.querySelector('style[' + ssrIdKey + '~="' + obj.id + '"]')

  if (styleElement) {
    if (isProduction) {
      // has SSR styles and in production mode.
      // simply do nothing.

Hello,
Do you mind sending full debug log from your console and the part of the code where you use the grid, thanks :smile:

Hi,
I added virtual grid to my nuxt config
{ src: '~/plugins/vue-virtualgrid'}

within plugins:
vue-virtualgrid.js
import Vue from 'vue'
import VirtualGrid from 'vue-virtual-grid'
Vue.use(VirtualGrid)

then within any nuxt page
<template>
<div>
  <VirtualGrid :items="{}" />
</div>
</template>
<script>
import VirtualGrid from 'vue-virtual-grid'

export default {
  components: { VirtualGrid },

I'm aware i should pass the renderdata as an object but passing an empty object should work for a starter ;-)
this directly results into:

thanks!
And may i ask, will it be possible to add slot content in the future, it would be a lot easier if i could use the item component just within the virtual grid component

First of all, it should be an array of object, so passing [] should work indeed.

And may i ask, will it be possible to add slot content in the future, it would be a lot easier if i could use the item component just within the virtual grid component

Not sure what you are asking, can you rephrase?

So after some searches.
This plugin do not support SSR as of now. You can use it with:
{ src: '~/plugins/vue-virtualgrid', ssr: false}

There is maybe an answer here: https://stackoverflow.com/questions/61409902/vue-how-to-build-bundle-for-nuxt-with-vue-cli-service
Let me know if that helps.

So after some searches.
This plugin do not support SSR as of now. You can use it with:
{ src: '~/plugins/vue-virtualgrid', ssr: false}

There is maybe an answer here: https://stackoverflow.com/questions/61409902/vue-how-to-build-bundle-for-nuxt-with-vue-cli-service
Let me know if that helps.

Awesome, maybe add directly to the main guide ?

@merijnponzo Were you able to get this working with nuxt? If so, would really appreciate if you could let us know the solution.

Alright, got it figured out. I still imported the component in the page I was using it, which doesn't work when it is already imported with Vue.use(VirtualGrid) in the /plugins directory.
Here's the complete solution for nuxt:
plugin file virtualgrid.js (in /plugins)
import VirtualGrid from 'vue-virtual-grid'

import Vue from 'vue'
Vue.use('VirtualGrid', VirtualGrid) // MUST match the name you use when calling the component, e.g. <VirtualGrid ...>

Where you want to use the component.

DO NOT use import 'VirtualGrid from ...' again
DO NOT define VirtualGrid under 'components: []'

Just use <VirtualGrid ... > on your page.
NOTE: The

Hi, i switched to vue-virtual-scroll-list due the lack of time for a deadline :-)
And i prefer using slots.
Best!

Hi, i switched to vue-virtual-scroll-list due the lack of time for a deadline :-)
And i prefer using slots.
Best!

Thanks man for the reply. Mh, I would need Grid Layout support - I guess  vue-virtual-scroll-list doesn't support that, or does it? Yeah, slots are pretty sweet. :)

Seems like I spoke too soon. Even though I don't get a fatal nuxt error now, I get this in the debug console:

[Vue warn]: The client-side rendered virtual DOM tree is not matching server-rendered content. This is likely caused by incorrect HTML markup, for example nesting block-level elements inside < /p/ >, or missing . Bailing hydration and performing full client-side render.
warn @ vue.runtime.esm.js?2b0e:619
patch @ vue.runtime.esm.js?2b0e:6497
Vue._update @ vue.runtime.esm.js?2b0e:3945

vue.runtime.esm.js?2b0e:619 [Vue warn]: Unknown custom element:  - did you register the component correctly? For recursive components, make sure to provide the "name" option.

Alright, got rid of the "not registered" error. Apparently, one has to use Vue.component to register pure components, while Vue.use registers plugins that themselves internally register components.
virtualgrid.js (/plugins folder)
import VirtualGrid from 'vue-virtual-grid'

import Vue from 'vue'
Vue.component('VirtualGrid', VirtualGrid) // instead of Vue.use

Source: https://stackoverflow.com/a/61163860

I'm not sure I follow you @MentalGear , did you manage to make it work with SSR?


Reenable fade transition for Material page transition (#13048)

Fixes #12877 by reverting #9472.
Let's try this again, now that we've tried optimising Opacity.
This may get reverted it if affects the benchmarks badly again.

(The commit in question has already been committed before, so I assume it's ok to commit again.)

LG \o/

Well, this still caused a performance regression. Android-specific; it hurt average_frame_build_time_millis a little and average_frame_rasterizer_time_millis a lot.
I'm going to condition this code on a constant so that it's easier to turn on and off, since some people are going to want it on regardless.


[Feature Request] Localized diagnostics messages for TS 2.6

Since TypeScript 2.6 a new option for localized diagnostics was added (more info).
Is there any possibility to add this option to Atom diagnostics messages?

Since we're not calling tsc directly, we can't really pass CLI options to it. In fact, almost everything, including diagnostics, is handled by tsserver. Maybe try specifying locale in compilerOptions in tsconfig.json?

Oh, I see. Using locale in compilerOptions doesn't seem to work, not even for CLI tsc. Documentation for tsserver mentions --locale option, although it is a CLI option too. Maybe tsserver can be executed with that option at starting (with an appropriate settings window option)?


Have the text property automatically filled when the react property is used when sending emails.

Do you recommend to always send a text version of an email along with the html version? If yes, could we have the SDK automatically set the text property when we use the react one (doing a second render with the plainText: true option)?

https://github.com/resend/resend-node/blob/canary/src/emails/emails.ts#L30
https://github.com/resend/resend-node/blob/canary/src/batch/batch.ts#L27

Do you recommend to always send a text version of an email along with the html version? If yes, could we have the SDK automatically set the text property when we use the react one (doing a second render with the plainText: true option)?

https://github.com/resend/resend-node/blob/canary/src/emails/emails.ts#L30
https://github.com/resend/resend-node/blob/canary/src/batch/batch.ts#L27

Yes, having the text version of the email is usually recommended.
That's a great suggestion, ideally this should be on API side so user never needs to send the text version doesn't matter which SDK they are using. Doing on SDK side with React Email is easier though
We will explore more

It seems this still hasn't been implemented.
In the meantime, is this how you all would recommend working around this?
import { Resend } from 'resend';
import { render } from '@react-email/render';

const resend = new Resend('re_123456789');

async function sendEmail() {
    const jsx = <MyTemplate />;
    const text = await render(jsx, {
        plainText: true,
    });

    await resend.emails.send({
        from: 'Acme<EMAIL_ADDRESS>        to<EMAIL_ADDRESS>        subject: 'hello world',
        react: jsx,
        text,
    });
}


updating CHANGELOG to reflect rubygems releases

Since we skipped 2.0.3, update CHANGELOG to reflect that.
"Could not find devise-2.0.3 in any of the sources"

Sorry. I have done that myself, forgot to push. Pushed now.


Download all submissions on leaderboard button

Dears,
First of all, thank you for the incredible work you do.
Question: On the Codalab platform, for the challenge administrator, on the leader board, there's a "Download all submissions on leaderboard" button. This feature has disappeared on the Codabench palteform. Do you plan to reintroduce this feature in Codabench?
https://www.codabench.org/competitions/1043/?secret_key=391dd24a-9cd7-4496-a4bc-bc6333fe7d2e
https://codalab.lisn.upsaclay.fr/competitions/9090?secret_key=703243e6-f0e8-4014-aeaa-64baab683a36

In the same way, is it possible to add a button in the "submissions" admin tab to download the programs for all submissions?

More generally, a "download all submissions" is an important feature.

Referring to the submissions dialog for a project, I think it would be good to have all the actions which are available for single individual entries available with a bulk action button for all those entries which are checked (download, delete ...)

@nicomy


Merge efforts in one C# library

Hello @markashleybell,
I recently stumbled unto another C# library (https://github.com/goelhardik/ignore) for parsing ignore files and I really like the fact that it was testing the behavior against git (using libGit2sharp).
I don't think C# needs more than one library on the subject so I'm opening a friendly discussion to see if/how efforts could be coordinated.
Let me know your thoughts!

Hey @jairbubbles,
That library looks nice, I also like the method of testing directly against git.
I think I mentioned before (or maybe it was someone else!) that this library also used to compare output directly with git (see this revision). Then the whole approach changed in v3 and it got removed because the tests were rather brittle and required a lot of maintenance, and it wouldn't have worked any more anyway.
There are dependencies on this library built into a few projects now (as it's been around for a while), so I will keep maintaining it on a minimal basis indefinitely. However, it was originally created to fulfil a very specific need for me a long time ago and I don't have that need any more, so being short on time I don't think there will be much active development going forward.
So, if there is a new library with equivalent functionality which is more recent, uses newer tooling and is more accurate in terms of its behaviour, then I suggest we just move over to using that!
I'm happy to give https://github.com/goelhardik/ignore a try at some point and contribute there if I can be of help.

I'm happy to give https://github.com/goelhardik/ignore a try at some point and contribute there if I can be of help.

Awesome, I like the spirit! 🤩

This is a nice initiative! From my understanding do both libraries do exactly the same? When reading the README at https://github.com/goelhardik/ignore it seems like you don't parse existing ignore files? Or am I missing something?

it seems like you don't parse existing ignore files

@nojaf You mean reading the lines of the .gitignore file? I'll have to check... In my application it was done in the user code so I did not need it. But it's very trivial, just a File.ReadAllLines.

Indeed, my current use-case is to check if a file is ignored or not:
https://github.com/fsprojects/fantomas/blob/e92fa7a9201c4ac7c208e8edaea87c124c2a1d40/src/Fantomas.Extras/IgnoreFile.fs#L21-L30
And I was wondering if that functionality is also available in the Ignore project.

And I was wondering if that functionality is also available in the Ignore project.

Yes indeed you can do that, it's at the end of the example in the README:
// Check if a path is ignored
var isIgnored = ignore.IsIgnored("x.user");

Please note thought that it expects relative path.

I've had a quick scan and I can't initially see anything that this library does that Ignore doesn't. They are both pretty tightly scoped, so there is not a lot of surface area to check.
It definitely seems to me that both libraries have exactly the same goal, but the Ignore source looks a lot simpler—I mean that as a compliment—and the testing story is much better.
As I mentioned I'm a little short on time at the moment, but I will pull Ignore down and give it a whirl as soon as I can.


Lots of .rbenv/shims in path...

My path looks a bit like this:
/Users/ian/.rbenv/shims:/Users/ian/.rbenv/shims:/Users/ian/.rbenv/shims:/Users/ian/.rbenv/shims:/Users/ian/.rbenv/shims:/Users/ian/.rbenv/shims:/Users/ian/.rbenv/shims:/Users/ian/.rbenv/shims:/Users/ian/.rbenv/shims:/Users/ian/.rbenv/shims:/Users/ian/.rbenv/shims:/Users/ian/.rbenv/shims:/Users/ian/.rbenv/shims:/Users/ian/.rbenv/shims:/Users/ian/.rbenv/shims:/Users/ian/.rbenv/shims:/Users/ian/.rbenv/shims... < A lot more where those came from! >

I uninstalled using Homebrew and reinstalled using Homebrew, to no affect. Got the same result.
The relevant section of my .zshrc:
echo 'eval "$(rbenv init -)"' >> ~/.zlogin
export RBENV_ROOT=$HOME/.rbenv

MacOS 0.12.4 Beta (16E163f)
I used to have RVM, but deleted it and went with rbenv last October.
I get  the feeling I'm missing something obvious, but some googling didn't turn up anything.
Any ideas?
Thanks!

You can close this one... I found the problem!
My .zlogin looked like this:
eval "$(rbenv init -)"
eval "$(rbenv init -)"
eval "$(rbenv init -)"
eval "$(rbenv init -)"
eval "$(rbenv init -)"
eval "$(rbenv init -)"
eval "$(rbenv init -)"
... etc
Not sure why they were there, but deleting all those eval's did the trick!

Hi @pookah-net, seems like the issue is solved and can be closed?

Sorry for the lengthy delay!
Yep, it can be closed.

@iangrant-2nd since you are creator of the issue you can close it by yourself.

Thanks, @mislav.
Apologies, @ivanovaleksey, I wasn't thinking!


⚠ Processors has degraded performance

In 6e5eb31, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 77 ms

Resolved: Processors performance has improved in 87b75a2.


select-all or find-all

Is your icon request related to a problem? Please describe.
microsoft/vscode#7559
Describe the solution you'd like
An icon for multiple selections, but ideally associated with the current search icon - 🔍
Describe alternatives you've considered
Someone suggested ∀ which is a mathematical symbol and meaningless to most end users.
Additional context
There are several examples of this icon out there.

It's astute that this was opened. For anyone needing this icon, closest analogs that communicate this function are expand-all and plus. Hope this helps someone.


New version: creativeprojects.resticprofile version 0.25.0

Pull request has been created with komac v2.0.3 :rocket:
Microsoft Reviewers: Open in CodeFlow

Hi @Exorcism0666,
The package didn't pass a Defender or similar security scan. This might be a false positive and we can rescan tomorrow.
(Automated response - build 716.)

Automatic Validation ended with:

Installation failed with exit code -1978335187 2024-02-08 19:13:34.058 [FAIL] Installer failed security check. Url: https://github.com/creativeprojects/resticprofile/releases/download/v0.25.0/resticprofile_0.25.0_windows_386.zip Result: 0x80004005

(Automated response - build 716.)

Hi @Exorcism0666,
The package didn't pass a Defender or similar security scan. This might be a false positive and we can rescan tomorrow.
(Automated response - build 716.)

Automatic Validation ended with:

Installation failed with exit code -1978335187 2024-02-08 19:13:34.058 [FAIL] Installer failed security check. Url: https://github.com/creativeprojects/resticprofile/releases/download/v0.25.0/resticprofile_0.25.0_windows_386.zip Result: 0x80004005

(Automated response - build 716.)

Automatic Validation ended with:

No errors to post.

(Automated response - build 729.)

@wingetbot waivers Add Validation-Installation-Error

@wingetbot waivers Add Validation-Executable-Error


Error: Could not find expected browser (chrome) locally. Run npm install to download the correct Chromium revision (901912).

Error: Could not find expected browser (chrome) locally. Run npm install to download the correct Chromium revision (901912).
when build example and start, errors are occurred.
'npm run dev' is ok, but 'npm run build' and 'npm run start' has problem.
reproduce steps

clone this project
npm install
cd example && npm install

after I encountered errors bellow
`It looks like you're trying to use TypeScript but do not have the required package(s) installed.
Please install @types/react by running:`
so i installed @types/react and upgrade next, react
4.   npm install --save-dev @types/react && npm install next@latest && npm upgrade react@latest react-dom@latest

npm build && npm start
browser open and go to url http://localhost:3000/api/basic?myQueryParam=asdf

I also have this problem, but only on production

Same here. Works with dev build, but puppeteer-core cannot find chromium in the production build.

Hey folks @guhyeon @kamenskiyyyy @mvandergrift
The problem is probably only occurring during production app runtime on the local machine since when NODE_ENV is set to production and the library tries to load chrome from chrome-aws-lambda package instead of retrieving chromium path from the local machine.
I'll try to do something like detecting if the code is run on AWS to solve this
Cheers. Igor

@neg4n I found a solution https://github.com/kamenskiyyyy/open-graph-image-example. You just need to remove chrome-aws-lambda.

@neg4n I found a solution https://github.com/kamenskiyyyy/open-graph-image-example. You just need to remove chrome-aws-lambda.

@kamenskiyyyy
Im afraid that this can't be solved like this 😔 Only when hosting/running the next-api-og-image is done on the machine where you have direct access to the physical resources (servers, local hosting) does the problem solve with your solution applied. However, doing so will most likely break library's support for being deployed in the cloud 😭

Hey folks @guhyeon @mvandergrift @kamenskiyyyy
https://github.com/neg4n/next-api-og-image/releases/tag/4.2.2 🚀
Thanks for your contributions, feel free to continue the topic if problem still persists 😄

Serving my Next.js app on DigitalOcean, I'm getting this error after updating to 4.2.2:
Error: Failed to launch the browser process! spawn /usr/bin/google-chrome ENOENT TROUBLESHOOTING: https://github.com/puppeteer/puppeteer/blob/main/docs/troubleshooting.md   at onClose (/workspace/node_modules/puppeteer-core/lib/cjs/puppeteer/node/BrowserRunner.js:197:20)   at ChildProcess.<anonymous> (/workspace/node_modules/puppeteer-core/lib/cjs/puppeteer/node/BrowserRunner.js:189:85)   at ChildProcess.emit (node:events:513:28)   at Process.ChildProcess._handle.onexit (node:internal/child_process:289:12)   at onErrorNT (node:internal/child_process:478:16)   at processTicksAndRejections (node:internal/process/task_queues:83:21)

Serving my Next.js app on DigitalOcean, I'm getting this error after updating to 4.2.2:
Same case with Linode

Error: Failed to launch the browser process! spawn /usr/bin/google-chrome ENOENT
    at process.processTicksAndRejections (node:internal/process/task_queues:82:21)
    at onErrorNT (node:internal/child_process:476:16)
    at ChildProcess._handle.onexit (node:internal/child_process:289:12)
    at ChildProcess.emit (node:events:513:28)
    at ChildProcess.<anonymous> (/home/xxxx/xxxxx/node_modules/puppeteer-core/lib/cjs/puppeteer/node/BrowserRunner.js:189:85)
    at onClose (/home/xxxx/xxxxxxxx/node_modules/puppeteer-core/lib/cjs/puppeteer/node/BrowserRunner.js:197:20)
TROUBLESHOOTING: https://github.com/puppeteer/puppeteer/blob/main/docs/troubleshooting.md

I'm still investigating the problem.

Serving my Next.js app on DigitalOcean, I'm getting this error after updating to 4.2.2:

@wjeliot refer to this discussion, it might help


Add access log sampling percentage to Telemetry api

Describe the feature request

Add a field to AccessLogging.filter randomSamplingPercentage
And to the current CEL access log filter the RuntimeFilter to configure sampling (target source code here)
Define and reserve a new runtime key for configuring this (though ideally should not be set to use the value specified in RuntimeFilter since it doesn't seem like rtds is used)

Describe alternatives you've considered

Sample with request information in the CEL access log filter (e.g. modulo request timestamp), though subject to security concerns
Request Envoy add a CEL random() macro

Affected product area (please put an X in all that apply)
[ ] Ambient
[ ] Docs
[ ] Dual Stack
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[x] Extensions and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Affected features (please put an X in all that apply)
[ ] Multi Cluster
[ ] Virtual Machine
[ ] Multi Control Plane
Additional context

user request

The reason why I haven't switched to Telemetry  was because I didn't find sampling supported for access logs when using it.


[VL] Branch 1.2: Backport fixes for #7243

Backport fixes for https://github.com/apache/incubator-gluten/issues/7243.
The work is partly based on previous efforts https://github.com/apache/incubator-gluten/pull/7921 and https://github.com/apache/incubator-gluten/pull/7648.
Fixes for https://github.com/apache/incubator-gluten/issues/6736 are not included, and https://github.com/apache/incubator-gluten/pull/7165 is ported as a temporary solution.

cc @wForget @zsmj2017

@wForget @zsmj2017
Would you want to help verify this since CI is passing now, Thanks.

@wForget @zsmj2017
Would you want to help verify this since CI is passing now, Thanks.

I am testing some spill hang etls and will give response in 2024/11/15

@wForget @zsmj2017
Would you want to help verify this since CI is passing now, thanks.

Thank you, I will verify this later.

@wForget @zsmj2017
Would you want to help verify this since CI is passing now, Thanks.

I am testing some spill hang etls and will give response in 2024/11/15

@zhztheplayer
Based on the tests, the previously stable spill-hang job no longer hangs.（5/5 jobs passed)

Guys, are we good to merge this? Thanks.


fix(): webserver path should be simply localhost:3000

On my mac I cannot run webserver of example 07 chapter 11, because of different network interfaces.
As a web developer I suggest to simply use localhost:3000 with nodejs for development purposes.

PS: i suggest to duplicate this example to create another ready for raspberry pi. In this way users could run an example on pc + arduino and another one on raspberry + arduino


Development 313 revamp sidebar

Description
#1445
Changes
Changed Portal to Dashboard.
Moved Contracts to Dashboard
Moved 0-hub, 0-bootstrap, grid services & manual to Services
Removed Explorer and placed Stats, Nodes and Farms separately.

This line also needs to be edited.
https://github.com/threefoldtech/tfgrid-sdk-ts/blob/b57b6bc4f1b3d78f9da75af41f0ccfe3ea88d226/packages/playground/src/components/funds_card.vue#L18-L19


Add old default for 'skip_password_complexity?' closes #36

As it was said, the old behavior of gem is damaged by not setting the default for skip_password_complexity?. So, fixing that.

@matthewford, please, review this PR

@mrexox thanks, this totally skipped my mind.


build from dockerfile also results in error

command
docker build -t brainscape .
results in:
Sending build context to Docker daemon  220.2MB
Step 1/6 : FROM python:3.8-alpine
 ---> 8744555ae7bb
Step 2/6 : RUN apk update
 ---> Using cache
 ---> 6e57a4254f4a
Step 3/6 : RUN apk add make automake gcc g++ subversion python3-dev
 ---> Using cache
 ---> 1d26bffbff6d
Step 4/6 : ADD . /brainscapes_client
 ---> fd8a352f7c37
Step 5/6 : WORKDIR /brainscapes_client
 ---> Running in b970d5d565ba
Removing intermediate container b970d5d565ba
 ---> 4d6c90a6a6de
Step 6/6 : RUN pip install -r requirements.txt
 ---> Running in bd29d796eb5f
Collecting requests
  Downloading requests-2.24.0-py2.py3-none-any.whl (61 kB)
Collecting nibabel
  Downloading nibabel-3.2.0-py3-none-any.whl (3.3 MB)
Collecting anytree
  Downloading anytree-2.8.0-py2.py3-none-any.whl (41 kB)
Collecting pandas
  Downloading pandas-1.1.3.tar.gz (5.2 MB)
  Installing build dependencies: started
  Installing build dependencies: still running...
  Installing build dependencies: still running...
  Installing build dependencies: still running...
  Installing build dependencies: finished with status 'done'
  Getting requirements to build wheel: started
  Getting requirements to build wheel: still running...
  Getting requirements to build wheel: finished with status 'done'
    Preparing wheel metadata: started
    Preparing wheel metadata: finished with status 'done'
ERROR: Could not find a version that satisfies the requirement PIL (from -r requirements.txt (line 5)) (from versions: none)
ERROR: No matching distribution found for PIL (from -r requirements.txt (line 5))
The command '/bin/sh -c pip install -r requirements.txt' returned a non-zero code: 1

@marcenko is this resolved meanwhile?

It's resolved now


Fix zero proba interleave datasets

fix https://github.com/huggingface/datasets/issues/7147

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


Multiple archives per file

Add ability to store multiple archives per file.

Not the architecture we're looking for..


feat: add search collator

To make most of this great plugin, you could also provide a Search Collator for the backend to be able to search bulletins from the Backstage search. It's quite easy to do by just providing a CollatorFactory that implements the DocumentCollatorFactory from @backstage/plugin-search-common. You can check my Q&A plugin as a reference https://github.com/drodil/backstage-plugin-qeta/blob/main/plugins/qeta-backend/src/search/collators/QetaCollatorFactory.ts

Good idea! It would make the plugin a lot easier to use. If you want to tackle any of the suggested features, feel free to do it. Otherwise I will definitely look into it.

Great, I will take a look if I have time at some point to contribute!


Websocket Error

As soon as I add websocket connection from NodeRed I get this:
[ERROR:2020-03-27 23:58:34,434] rhasspyserver_hermes: BadRequest(400)
Traceback (most recent call last):
File "/usr/lib/rhasspy-voltron/.venv/lib/python3.7/site-packages/quart/app.py", line 1940, in full_dispatch_websocket
result = await self.preprocess_websocket(websocket_context)
File "/usr/lib/rhasspy-voltron/.venv/lib/python3.7/site-packages/quart/app.py", line 1968, in preprocess_websocket
result = await function()
File "/usr/lib/rhasspy-voltron/.venv/lib/python3.7/site-packages/quart_cors/init.py", line 297, in _apply_websocket_cors
abort(400)
File "/usr/lib/rhasspy-voltron/.venv/lib/python3.7/site-packages/quart/exceptions.py", line 199, in abort
raise error
quart.exceptions.BadRequest: BadRequest(400)
[ERROR:2020-03-27 23:58:31,429] rhasspyserver_hermes: BadRequest(400)
Traceback (most recent call last):
File "/usr/lib/rhasspy-voltron/.venv/lib/python3.7/site-packages/quart/app.py", line 1940, in full_dispatch_websocket
result = await self.preprocess_websocket(websocket_context)
File "/usr/lib/rhasspy-voltron/.venv/lib/python3.7/site-packages/quart/app.py", line 1968, in preprocess_websocket
result = await function()
File "/usr/lib/rhasspy-voltron/.venv/lib/python3.7/site-packages/quart_cors/init.py", line 297, in _apply_websocket_cors
abort(400)

Should be fixed in latest version. Quart has decided for some reason that the HTTP Origin header is required now for websockets, which is against spec. Node-RED does not send this header, so a patch was required on the Rhasspy side to disable websocket origin checks.


Arm build for docker

#3 removed the arm build for docker as it was failing with the following error:
#30 103.4 go build "-asmflags=all='-trimpath=/go'" "-gcflags=all='-trimpath=/go'" -ldflags="-X "github.com/ipfs/go-ipfs".CurrentCommit=67220edaa" -o "cmd/ipfs/ipfs" "github.com/ipfs/go-ipfs/cmd/ipfs"
#30 811.0 # github.com/ipfs/go-ipfs/cmd/ipfs
#30 811.0 /usr/local/go/pkg/tool/linux_arm64/link: running gcc failed: exit status 1
#30 811.0 collect2: fatal error: cannot find 'ld'
#30 811.0 compilation terminated.
#30 811.0
#30 811.2 make: *** [cmd/ipfs/Rules.mk:22: cmd/ipfs/ipfs] Error 2

arm builds should be re-enabled so that arm users can run a natively built image

Bug with the ipfs binary on aarch64, we get a segfault on execution.
ELF Headers all seem to be correct
/dscp-ipfs # readelf -h /usr/local/bin/ipfs
ELF Header:
  Magic:   7f 45 4c 46 02 01 01 00 00 00 00 00 00 00 00 00
  Class:                             ELF64
  Data:                              2's complement, little endian
  Version:                           1 (current)
  OS/ABI:                            UNIX - System V
  ABI Version:                       0
  Type:                              EXEC (Executable file)
  Machine:                           AArch64
  Version:                           0x1
  Entry point address:               0xde05d0
  Start of program headers:          64 (bytes into file)
  Start of section headers:          71879400 (bytes into file)
  Flags:                             0x0
  Size of this header:               64 (bytes)
  Size of program headers:           56 (bytes)
  Number of program headers:         10
  Size of section headers:           64 (bytes)
  Number of section headers:         45
  Section header string table index: 44


[BUG]: CSM Doc's Code block copy button Styling Issue

Bug Description
Copy button is not able to use the css file which is prism.css file.
accessing the css/prism.js instaed of csm-docs/css/prism.css.
Logs
https://dell.github.io/css/prism.css
Screenshots

Additional Environment Information
No response
Steps to Reproduce
just go to any code block you'll see in CSM-DOCs
Expected Behavior

CSM Driver(s)
Applies to all drivers.
Installation Type
No response
Container Storage Modules Enabled
No response
Container Orchestrator
Kubernetes
Operating System
Ubuntu

/sync


Invalid layout name should be valid (cose-bilkent)

Description
Trying to use cose-bilkent layout is invalid, but it appears in suggested layouts in the related error message.
Steps/Code to Reproduce
I have loaded the extra layouts using the code from the documentation - seen below:
import dash_cytoscape as city
cyto.load_extra_layouts()

When trying to set the layout to "cose-bilkent" in Cytoscape component like the following, I get an error:
cyto.Cytoscape(
  id="test",
  layout={
    "name": "cose-bilkent"
  }
)

Expected Results
Would be nice to have a cose-bilkent layout.
Actual Results
The following error popup:
Invalid argument `layout.name` passed into Cytoscape with ID "test".
Expected one of ["random","preset","circle","concentric","grid","breadthfirst","cose","close-bilkent","cola","euler","spread","dagre","klay"].

Versions
Dash 2.3.1
/Users/niels/Documents/GitHub/test/test.py:3: UserWarning:
The dash_html_components package is deprecated. Please replace
`import dash_html_components as html` with `from dash import html`
  import dash_html_components; print("Dash Core Components", dash_html_components.__version__)
Dash Core Components 2.0.2
/Users/niels/Documents/GitHub/test/test.py:4: UserWarning:
The dash_core_components package is deprecated. Please replace
`import dash_core_components as dcc` with `from dash import dcc`
  import dash_core_components; print("Dash HTML Components", dash_core_components.__version__)
Dash HTML Components 2.3.0
Dash Renderer 1.9.1
Dash HTML Components 0.2.0

P.S. there is missing two underscores in the template for this step, so you get the following error at first:
AttributeError: module 'dash_renderer' has no attribute '__version'

They already fixed it but have not yet published a new version.
In the mean time, you can install from the master branch
pip install git+https://github.com/plotly/dash-cytoscape


chore: add @smoya as code owner

Description
Adding me @smoya as code owner. Worked along with @magicmatatjahu adapting this to TS and the new Parser-JS v2 interface.
@magicmatatjahu feel free to suggest your addition as well.
Related issue(s)

Original work: https://github.com/asyncapi/parser-js/pull/578
Moved to this repo: https://github.com/asyncapi/openapi-schema-parser/pull/142

@smoya can you update PR and remove me as codeowner. I was codeowner just to approve Fran's PRs 😄

@smoya can you update PR and remove me as codeowner. I was codeowner just to approve Fran's PRs 😄

Done

/rtm


recording frames frame size error

This issue refers to this conversation
https://community.onion.io/topic/4509/tau-usb-cable-suggestions
Python Library versions
Python 3.9.5
pip 21.1.1
Camera Info
ToF Camera opened successfully
model: 4.0
firmware: 3.3
uid: 9500c6
resolution: 160x60
port: COM8
Describe the bug
example recordRawFrames.py  runs slow and has lots of bad frames.
The shell acquires a frame at sporadic intervals.    I have recorded 2 clips of the frame recording and play back.   The first video shows whats happening in the shell, then I flipped the orientation of the USB-C cable as a test and ran a second video.  Same results.
Steps to Reproduce
step 1 - Use USB A to USB C cable - 3" long,
step 2 - run replayRawFrames.py from shell
What is expected?
frames to be captured at regular intervals
folder filling up with good frames, bad frames not kept
What is actually happening?
frames are captured at irregular intervals,  between 0.5 seconds and 25 seconds
frames are typically bad.   error: 'Data error, actual size: 38356, expected size: 38480
replayRawFrame.py error message 'Bad frame ignored, bytes length 38276'
*** notice the size error on recording and the size error on replay are different sizes.  This is the same for every rejected file.
Any additional comments?
I will add 2 frames, one good and one bad.
1625280643.067339.frame - good frame
1625280658.6192925.frame - bad frame
I will add the two screen recordings
Here is the Zip folder
https://drive.google.com/file/d/1ugfV21nifUp86CioeHIdQ1R-xqKZseBW/view?usp=sharing

@kevinboards To confirm, this only happens with the recordRawFrames.py example program, and the rest of the example programs work as expected?
The same questions from #13 will be helpful here as well:

Does this happen every time you try to run recordRawFrames.py ?
Has it worked as expected before? If so, did it consistently work as expected or only from time to time?
Do you have multiple Tau Cameras, and if so, does this happen with all of them?

@kevinboards To confirm, this only happens with the recordRawFrames.py example program, and the rest of the example programs work as expected?
Correct, recordRawFrames.py acts this way but the other example programs function fine

The same questions from #13 will be helpful here as well:

Does this happen every time you try to run recordRawFrames.py ?
Yes
Has it worked as expected before? If so, did it consistently work as expected or only from time to time?
No, never as expected.
Do you have multiple Tau Cameras, and if so, does this happen with all of them?
I only have 1 camera

UPDATE: Today I was able to get my hands on an old windows 7 machine.   I downloaded Python 3.7, setup everything for the TauLidarServer and it ran  really well!  No problems.
I then downloaded the examples, the recording of frames was incredibly fast and all of my frames were good frames.
The Tau Camera is obviously fine.  There is something going on with my lenovo laptop with python 3.9 that I will need to get to the bottom of.

@kevinboards my guess is the serial port on your lenovo is the culprit here and for #13
Could you look into the details of the COM ports for both machines and see if you can find any differences?
It looks like there's also a fix on the way that might help with this - see issue #14

@kevinboards Looks like this issue and #13 is a combo of two problems:

A bug causing the bad frame ignored error (#14)
A USB issue with Thinkpad laptops (#16)

A fix for the bad frame ignored error has been fixed and released in TauLidarCamera v0.0.4. You can update by running python -m pip install TauLidarCamera --upgrade. (Just make sure v0.0.4 is installed).
And the USB issue can be resolved by updating your Thinkpad's BIOS.

Did not make any changes to BIOS (it should be up to date but I didn't do anything here)
Camera now works as expected, this bad frame bug was the cause for me.
I can now Record frames properly and use TauLidarServer properly.
Thanks for all the hard work team!  You should be able to close this issue and issue #13

@kevinboards awesome, happy to hear everything is working properly now!
Gonna close this issue and #13
It would be much appreciated if you could also post a brief update on the Onion Community that TauLidarCamera v0.0.4 fixes the issue - better visibility for other users who may experience the same thing.


Cloned form causes duplicate sprig values

I'm building a facet filter with Sprig similar to this TailwindUI design: https://tailwindui.com/components/ecommerce/components/category-filters

It works fine, however, they duplicate the DOM elements to build the mobile filters. That means there are two identical forms with sprig filters.
Simplified, I have something like this in my Sprig component:
{# Mobile filter dialog #}

    <form>
        {# for filter here #}
            <input sprig type="checkbox" name="brands[]" value="honda" {{ 'honda' in brands ? 'checked' }}>
            Honda
        {# endfor #}
    </form>

{# Desktop filter sidebar #}

    <form>
        {# for loop here #}
            <input sprig type="checkbox" name="brands[]" value="honda" {{ 'honda' in brands ? 'checked' }}>
            Honda
        {# endfor #}
    </form>

As you can imagine, cloning the DOM elements causes a lot of problems:

When I select two checkboxes, the brands variable starts showing duplicates ($brands = ['honda', 'honda', 'stihl'];) after sprig swaps
When I try to unselect a checkbox, it doesn't persist after sprig swaps, because one form is still sending the checked value 🤡

Did you ever run into such a situation? What's the recommended approach? Have separate state for each filter (brands[] and mobileBrands[])? Keep the forms in sync using something like Alpine model? Is there a way to only let one form submit values and ignore the others?
I tried hx-sync="closest form:abort" but it didn't work, so that's probably not what I'm looking for?

If I add s-preserve="true" to both forms, then each form with checkboxes works fine of course. They are not re-rendered, so they don't affect each other's state. They are not in-sync, so I can toggle checkboxes in form1, and form2 (mobile) stays blank.
It does not work in my specific situation because other filters (e.g. price) update the values of the other filters (e.g. brand) so they need to swap, unfortunately. Will search a bit more if I can let htmx ignore the form name attributes from other forms.

If I add s-preserve="true" to both forms, then each form with checkboxes works fine of course. They are not re-rendered, so they don't affect each other's state. They are not in-sync, so I can toggle checkboxes in form1, and form2 (mobile) stays blank.
It does not work in my specific situation because other filters (e.g. price) update the values of the other filters (e.g. brand) so they need to swap, unfortunately. Will search a bit more if I can let htmx ignore the form name attributes from other forms.
I guess, for now, I will need to look into a solution where I include form1 OR form2 based on a sprig value (mobile filter button click)

Answered by @bencroker on Discord: I had to set s-include="this" on each form element. Each Sprig component element gets that attribute to limit what is included in a request, so you're limiting it further to one table at a time.


cannot download any video from youtube

I run brew install youtube-dl
and
youtube-dl 'https://www.youtube.com/watch\?v\=rywUS-ohqeE'
and got this error:
WARNING: Could not send HEAD request to https://www.youtube.com/watch?v=rywUS-ohqeE: HTTP Error 404: Not Found
[generic] watch?v=rywUS-ohqeE: Downloading webpage
ERROR: Unable to download webpage: HTTP Error 404: Not Found (caused by HTTPError()); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

You're quoting the backslashes, remove them from the command and it should work.


Benchmarks for order event resolving

I want to merge this change because it implements benchmark tests for order events resolving.

Impact

[ ] New migrations
[ ] New/Updated API fields or mutations
[ ] Deprecated API fields or mutations
[ ] Removed API types, fields, or mutations
[ ] Documentation needs to be updated

Pull Request Checklist

[ ] Privileged queries and mutations are guarded by proper permission checks
[ ] Database queries are optimized and the number of queries is constant
[ ] Database migration files are up to date
[X] The changes are tested
[ ] GraphQL schema and type definitions are up to date
[ ] Changes are mentioned in the changelog

Task linked: SALEOR-2739 Optimize events resolver in Order type


Update JDK-21 to JDK-21.0.1 on Jenkins

Description
Update JDK-21 to JDK-21.0.1 on Jenkins
Issues Resolved
Part of https://github.com/opensearch-project/OpenSearch/issues/10746
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

As talk with @reta this does not need a rebuild on images for now as we already have 21.0.0 on images.
We will update them later as it is not prioritized.
Thanks.


Fix 3512 - Hide back button on root view controllers for the tabs

this fixes a UI glitch where the button was shown shortly at the end of the onboarding flow (#3512).
Description

This PR fixes a UI glitch that was reported as #3512. After onboarding, the transition from the tab-specific info screen to its root controller / overview screen was shortly showing a 'back' button.
The fix hides the back button for the three affected view controllers.
All three are used exclusively as root controllers, and don't need this back button.
Link to Jira

Jira: n/a
github: #3512
Screenshots

The video shows the transition for the contact diary as example.
I have tested all three affected root controllers.
https://user-images.githubusercontent.com/6118267/132982821-33c6c568-ca24-4b50-9fae-381b2b4036c6.mov

https://user-images.githubusercontent.com/6118267/132982889-a1d8b537-67ca-440e-9577-eab098f38374.mov


How to update node information outside it

Hello, in my project the changes and configurations of the note are made via a sidebar and I use VUE, so as not to have a sidebar inside each node I put a single sidebar outside the nodes, and when clicking on the node I use an EventBus to populate the correct sidebar.
My problem is when updating the information through the sidebar, how do I update the node?
I'm thinking of doing this using EventBus as well, but I don't know if it's the best way.

There isn't just one way.
You just have to get to the updateNodeDataFromId function.
In vue 3 you can see an example here, in this case without event bus.
https://github.com/jerosoler/drawflow-vue3-example


No auto refresh with Data Viewer

Testing #10915

Open a variable in data viewer
Update the variable value
Variable View auto refreshes it value but Data Viewer doesn't

Data Viewer should have a refresh button? You may not want to auto refresh.

I mean I believe it already has one.

Agreed, i think this was deliberate

Should we close this issue then?

You may not want to auto refresh

For a moment I was thinking why wouldn't I but then realized that users might want to have a lock'ed version of the variable viewed in the data viewer.
With that said, when I open the variable from the variable to the data viewer, having auto refreshed turn on is my expectation. I would prefer by default auto refresh + a lock button.

For this solution I'd like to discuss UI approaches. So, given my time constraints, I'll unassign this issue from me.

Closing this issue as there aren't any upvotes nor feedback from users for this issue.


Redirect to onboard screen on disclaimer agree (#330)

IMPORTANT: Please do not create a Pull Request without creating an issue first.
Any change needs to be discussed before proceeding. Failure to do so may result in the rejection of the pull request.
Do not change the following line aside from replacing YOUR-ISSUE-HERE with the issue this PR fixes
Fixes #330
Please provide enough information so that others can review your pull request:
Explain the details for making this change. What existing problem does the pull request solve?
After agreeing to the disclaimer, the user will now be redirected to the onboarding screen. This creates a more appropriate flow that encourages the user to input their information as soon as possible.
Previously, the disclaimer modal would only appear on the Log screen if the user had not seen it before, but not necessarily agreed to it. Now, the disclaimer modal appears on any screen after logging in, and will only disappear after the user agrees to it. This will help ensure that the user agrees to the disclaimer before using any part of the app.
Test plan (required)
To test locally:

Create a new account, or use an account that has not agreed or disagreed to the disclaimer.
Navigate to any page by typing the URL manually (http://localhost:3000/, http://localhost:3000/education, etc.). The disclaimer modal should appear.
Click the disagree button for the disclaimer. The disclaimer modal should not disappear.
Refresh the page. The disclaimer modal should appear.
Click the agree button for the disclaimer. The disclaimer modal should disappear and the onboarding screen should be navigated to automatically.
Refresh the page now. The disclaimer modal should not appear.

Final Checklist

[x] Have you bumped the version in package.json?

I feel second decimal should be for minor change, and third for patch/bug fix, but I followed the recommendation below.
Second decimal for major change, third decimal for minor change. This can go past 10 i.e. 1.0.9 !=> 1.1.0, 1.0.9 => 1.0.10
Read here for more info

[ ] Have you added any new tests necessary?
[x] Is your PR rebased off the most current master?
[x] Have you squashed all commits, or if you will merge will you squash all commits?
[x] Did you use yarn, not npm?
[x] Did you use Material-UI wherever possible?
[x] Did you format according to Prettier?
[x] Did you run all of your most recent changes locally to make sure everything is working?

🚀 🚀


do not use remote viz.json in tests

image tests need to fetch remote viz.json. I'd create some local fixtures with them

deplicated https://github.com/CartoDB/cartodb.js/issues/375


Can’t use with pouchdb-adapter-node-websql?

I hope I’m not doing something totally stupid, but I have a command-line Node app that uses pouchdb-adapter-node-websql with the SQLite backend—and that part works great.
But when I attach express-pouchdb to that PouchDB instance and try to query over REST, express-pouchdb tries to find a LOCK file “inside” the SQLite file and breaks.
Example:
var PouchDB = require('pouchdb');
PouchDB.plugin(require('pouchdb-adapter-node-websql'));
var db = new PouchDB('mypouchdb.db', { adapter: 'websql' });

var express = require('express');
var app = express();
app.use('/db', require('express-pouchdb')(PouchDB, { mode: 'fullCouchDB' }));
app.listen(3001, () => console.log('Running!'));

Now, when I try to hit a _all_docs endpoint:
$ curl http://<IP_ADDRESS>:3001/db/mypouchdb.db/_all_docs
{"error":"OpenError","reason":"IO error: mypouchdb.db/LOCK: Not a directory"}

If I change the backend to be a leveldb (the default), i.e.,
var PouchDB = require('pouchdb');
var db = new PouchDB('mypouchlevel'); // LevelDB, *not* SQLite

// same Express setup as above

then curl http://<IP_ADDRESS>:3001/db/mypouchlevel/_all_docs prints out my documents—since mypouchlevel is a directory with level stuff inside, while mypouchdb.db is a SQLite db file.

I was able to reproduce the problem. The error looks to me like an issue with pouchdb-adapter-node-websql. I remember problems when setting a path for the db files, where you’d need to create the path manually because the adapters woudn’t do it.
I’m not familiar with the pouchdb-adapter-node-websql adapter myself though and can’t help with it, I’m afraid. Hopefully someone else can. Please let us know if you find the cause of the problem yourself :)

I have the same problem now. 5 years later 🤷


README link to Spark 2.4.8 is a 404

In the README, https://github.com/datastax/cassandra-data-migrator/blob/main/README.md, the link to Spark 2.4.8 is a 404.
https://downloads.apache.org/spark/spark-2.4.8/
Should it now point to one of the 3.x.x version listed on https://downloads.apache.org/spark/ ?

@johnsmartco The broken link to Spark 2.4.8 has been corrected


Folder structure needs to be sorted.

I will update the folder structure, the links on all the files so the pages will be interrelated.
kindly assign me this task under hacktoberfest-accepted lable.

Done


Extend 404 handler to handle /# in URL

E.g.
.../newsroom/somepage/#someanchor
will be redirected into
.../newsroom/somepage#someanchor
Fixes #23
Changelog:

Extend 404 handler to handle /# in URL

Test URLs:

Original: https://old.sunstar.com/newsroom/news/update-of-sunstars-esg-activities-for-2023/#progress-of-esg-activities-at-sunstar-during-2022
Before: https://main--sunstar--sunstar-global.hlx.live/newsroom/update-of-sunstars-esg-activities-for-2023/#progress-of-esg-activities-at-sunstar-during-2022
After: https://issue-23--sunstar--sunstar-global.hlx.live/newsroom/update-of-sunstars-esg-activities-for-2023/#progress-of-esg-activities-at-sunstar-during-2022

We can ignore the LHS here I think since this PR involves a redirect via the 404 page...


#13 Bump @actions/core version

Bumping @actions/core dependency version to eliminate warnings due to new implementations of set-output and save-state functions in github
Fixes #13

Closing. This has been supersed by 1.0.4


fix deployment when target is accounts

Add OptimizeUnassociatedWebACLValue to fms-shield template
StackSets with SERVICE_MANAGED permission model can only have OrganizationalUnit as target  Error when target is accounts, fixed adding RootID parameter and changing conditionals on Deployment Targets
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

hold on this merge, the documentation says ResourcesCleanUp is not valid on Shield Advance but I just test it and it works, will continue testing to commit changes to this PR.
`ResourcesCleanUp
Indicates whether AWS Firewall Manager should automatically remove protections from resources that leave the policy scope and clean up resources that Firewall Manager is managing for accounts when those accounts leave policy scope. For example, Firewall Manager will disassociate a Firewall Manager managed web ACL from a protected customer resource when the customer resource leaves policy scope.
By default, Firewall Manager doesn't remove protections or delete Firewall Manager managed resources.
This option is not available for Shield Advanced or AWS WAF Classic policies.
Required: No
Type: Boolean
Update requires: No interruption`

hi 4ndr4s.
I am going to close this pull request.  I did some more testing and pushed updates that I believe address the issue you called out here.  Appreciate the feedback/identifying this issue!


Reversion of latest and 13.0.0 tags to node-18.15.0 with old browsers versions

https://hub.docker.com/r/cypress/browsers/tags
cypress/browsers:latest has reverted to
cypress/browsers:node-18.15.0-chrome-106.0.5249.61-1-ff-99.0.1-edge-114.0.1823.51-1
after spending the last three weeks being equivalent to
cypress/browsers:node-20.5.0-chrome-114.0.5735.133-1-ff-114.0.2-edge-114.0.1823.51-1
This is probably unintentional and it is certainly disruptive.

This issues affects also:
https://hub.docker.com/r/cypress/base
where
cypress/base:latest
has been reverted from
cypress/base:20.5.0
to
cypress/base:18.15.0
and
https://hub.docker.com/r/cypress/included
where
cypress/included:13.0.0
has been reverted from
cypress/included:cypress-13.0.0-node-20.5.0-chrome-114.0.5735.133-1-ff-114.0.2-edge-114.0.1823.51-1
to
cypress/included:cypress-13.0.0-node-18.15.0-chrome-106.0.5249.61-1-ff-99.0.1-edge-114.0.1823.51-1

This issue has been partially resolved. The following tag is however still incorrect:
https://hub.docker.com/r/cypress/included
where
cypress/included:13.0.0
has been reverted from
cypress/included:cypress-13.0.0-node-20.5.0-chrome-114.0.5735.133-1-ff-114.0.2-edge-114.0.1823.51-1
to
cypress/included:cypress-13.0.0-node-18.15.0-chrome-106.0.5249.61-1-ff-99.0.1-edge-114.0.1823.51-1

@MikeMcC399 I think I can handle that by setting the factory version to 13.0.0. Should be able to play around with that in a few

@MikeMcC399 how about now?

@AtofStryker

how about now?

It looks good now. Thank you!


mc display messed up

Here is what I did:

Updated Win 10 x64 system with BashOnWindows to Creators Update (Microsoft Windows [Version 10.0.15063])
lxrun /uninstall /full
lxrun /install
Run bash.exe
sudo apt-get install mc
mc

Expected to see correct rendering, instead I got this:

More strange behavior: exiting and relaunching mc changes the TrueType font of the console window!
After forcing the font back to Consolas and resizing the window with the mouse (by dragging the border), the rendering appears mostly correct, except

for the new permanently overlaid scroll bars
the background & foreground color of the prompt line

Perhaps this is terminal definition or ncurses related... Seeing the same behavior in dialog...
dialog --yesno "Would you like a dialog?" 10 50

@axelfontaine I can actually explain the peculiarity of the fonts switching back once you close mc!
As it so happens, mc runs in the "Alternate Buffer" of the terminal. In conhost, this buffer just so happens to have it's own separate set of settings, independent of the "main" buffer (which is running your shell that launched mc). So, if you use the property pages to set the settings while you're in the alt buffer, the changes don't propagate to the main buffer.
I'd say for now, the best advice I have is don't change the properties while you're in the alt buffer. I've opened MSFT:11643965 to try and address that.
As far as the bars being messed up lengths, I believe this is a font rendering issue. I've seen it before, but IIRC there was nothing I could to about it. It's kinda related to #1907

@zadjii-msft Thanks for looking into this. My display is scaled at 100% so not related to #1907
Also if you look carefully it clearly shows the wrong characters being printed at various positions in the console, not some scaling issue. I have tried with both raster and TT fonts at various sizes. Same issue.
Also don't forget the weird scrollbar behavior.

@axelfontaine Yea, #1907 says it's a DPI thing, but it's really less about the DPI and more about the fact that those characters are ambiguous width. Some fonts are simply going to work better for this scenario than others.
BUT NEVER RASTER FONTS. Raster fonts have a whole host of problems, so protip, never use them.
I have no idea how you got the scroll bars to show up. I've been playing with an mc window for a bit now and I can't seem to get them to show up.
The prompt line's color change again came from changing the settings - It looks like in mc the settings page picks up the attributes of the character at 0,0 (or more likely, 'mc' left those attributes as the current ones) so submitting the settings page makes those colors the default for the buffer.

More testing keeps on revealing more weird behavior. When mc is running (and only then) resizing the window down to 1 line vertical, causes the whole console window to crash and terminate. This does not happen outside mc.

Ooof. That's definitely a bug too. To avoid polluting this issue, I opened a new issue to track that particularly, #1938. Thanks fir the investigation @axelfontaine!

I solved the problems with the windows console. I use PuTTY (http://www.chiark.greenend.org.uk/~sgtatham/putty/)

@zadjii-msft I'm intrigued by this "alternate buffer"! Do you know of any blog posts/readings on the topic?

@lilred Well I guess there's https://msdn.microsoft.com/en-us/library/windows/desktop/mt638032(v=vs.85).aspx#Alternate_Screen_Buffer__ but it looks like I didn't write all that much there :P
There's also http://invisible-island.net/xterm/ctlseqs/ctlseqs.html#h2-The-Alternate-Screen-Buffer:

Xterm maintains two screen buffers.  The normal screen buffer allows you
to scroll back to view saved lines of output up to the maximum set by
the saveLines resource.  The alternate screen buffer is exactly as large
as the display, contains no additional saved lines.  When the alternate
screen buffer is active, you cannot scroll back to view saved lines.
Xterm provides control sequences and menu entries for switching between
the two.

Different terminals handle it differently, and not all TERM settings support it the same way. I tried to implement it as close to xterm as possible.

Okay, I'm gonna do some cleanup of issues here. There are three issues in this thread:

Box drawing characters don't render right in some fonts
Resizing the buffer to 0 lines causes conhost.exe to crash
Changing the settings in the alt buffer doesn't propagate to the main buffer.

Now, for details.

is a dupe of #2010, #2099 and others, but not #1907. #1907 is for all the ambiguous width chars, while the others are specific to the box-drawing range, 0x2500-0x257F.  This portion of the issue has been checked in, and should make your experience with tmux, mc, and others generally better.

This has also been checked in, and was duped into #1938.

This has NOT been fixed. I'm pulling that out into #2157.

I'm gonna mark this one as fixinbound, #2010, #2099 as closed dupes to this.

I found a temprary work around is to use mintty with wsltty.
Tested with dialog and aptitude, and it works as expected.

FYI, mc works great in the most recent inisder builds...

Small update using Fall Creators Update: first of all, things have improved a lot! Unfortunately there are still some quirks:

See the right and bottom side of the window.
What did I do:

sudo apt-get install mc
mc
wildly resized window in all directions

This issue was moved to Microsoft/console#117


500 Internal Server Error (InvalidPath) on seemingly trivial routes

My app is set up as follows:
let app = Blackfish()
let model = Model()

app.use(renderer: StencilRenderer(), ext: ".stencil")

app.get("/") { request, response in
  response.render("index.stencil")
}
app.get("/students.html") { request, response in
  response.render("students.stencil", data: ["students": model.students])
}

app.use(path: "/api/students", controller: StudentController(model: model))
app.use(path: "/api/courses", controller: CourseController(model: model))

app.listen(port: 3000) { error in
  print(error ?? "BlackFishTest listening on port 3000")
}

The error is related to the paths / and /students.html. The paths starting with api still work as before. Files index.stencil and students.stencil are in the Resources folder. What's wrong here?
I'm using the February 8 build on Ubuntu.

Can I get you to do swift build --clean=dist and then swift build
I pushed a late night fix for a break with the way middleware and routes were handled which might fix this issue.
If that doesn't work, I'll investigate.
The only other thing I can think of is that there's an error with loading the stencil files.

That didn't work.

Does returning JSON instead of .stencil work or do you still get a server error?

Returning JSON works.

Could I get a copy of your index.stencil?

index.stencil:
<!doctype html>

    <meta charset="utf-8">
    <title>Welcome</title>

Students

Courses

students.stencil:
<!doctype html>

    <meta charset="utf-8">
    <title>Students</title>

    {% for student in students %}

{{ student.id }}: {{ student.name }}

    {% endfor %}

I don't seem to be having the issue my side with the index.stencil.
Are the stencil files inside of Resources/ or are they nested in another directory inside of it?
e.g.
Resources/Views/index.stencil

If the latter is the case, then you'll need to do:
response.render("Views/index.stencil")

Is there anyway I can clone your project to give it a go? That way I can figure out what is happening internally.

https://github.com/svanimpe/blackfish-example

Running with your index.stencil example, everything appears to work my side on Ubuntu 14.04
On the student.stencil it renders but it seems to have a problem with the data. If I do this
<!doctype html>

    <meta charset="utf-8">
    <title>Students</title>

 {{students}}

    {% for student in students %}

{{ student.id }}: {{ student.name }}

    {% endfor %}

I get a web page which renders:
[BlackFishTest.Student, BlackFishTest.Student]

I think it's a problem with Stencil handling the Any object. I'll update my renderer this weekend to handle the Any object type.
Looking at the Stencil renderer I can see that there is an issue related to this https://github.com/kylef/Stencil/issues/51

Is there any error's that are produced which would explain why your index page isn't loading?

This appears to be have been an issue with CLion (the IDE I'm using on Linux). If I run the app from the command line, from its root folder, the index stencil works. I think CLion ran the app from the .build/debug folder, in which case the stencils weren't found. Setting the working directory to the project root in the build settings fixed this.

I'll post a workaround shortly on how to process the data using Stencil. It's not ideal but it'll get you the results you want at the moment.

Just want to mention, I've fixed https://github.com/kylef/Stencil/issues/51 in Stencil 0.5.3. Please let me know if there are any released problems.
-Kyle

Oh that's awesome, thanks @kylef
I'll give that a test later and feedback


Fix New Zealand

I'm trying to amend the commit:

updated to include new zealand

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

The updates in the PR appear to be in master. We'll close the PR.


xml:how to check point to nil when xml Unmarshal

when some value is <tt:PTZPosition /> how can I check the value is nil when xml Unmarshal. I try use pointer.but the value is not nil(Zoom attributes is 0 I can not confirm 0 value is pass by xml value).
Example code:https://go.dev/play/p/BrgD4S--Rmf
code:
package main
import (
"encoding/xml"
"fmt"
)
type Vector1D struct {
X float64 xml:"x,attr"
}
type PTZVector struct {
Zoom Vector1D xml:"Zoom"
}
type PTZPresetTourPresetDetailUnmarshal struct {
PTZPosition *PTZVector xml:"PTZPosition"
}
func main() {
str :=           <tt:PresetDetail>             <tt:PTZPosition />           </tt:PresetDetail>
var ans PTZPresetTourPresetDetailUnmarshal
err := xml.Unmarshal([]byte(str), &ans)
if err != nil {
panic(err)
}
fmt.Printf(%#v, ans.PTZPosition)
}
result:
&main.PTZVector{Zoom:main.Vector1D{X:0}}

Unlike many projects, the Go project does not use GitHub Issues for general discussion or asking questions. GitHub Issues are used for tracking bugs and proposals only.
For questions please refer to https://github.com/golang/go/wiki/Questions


feat: add priority to application in manager

Signed-off-by: Gaius<EMAIL_ADDRESS>
Description

Add priority to application in manager

Related Issue

Motivation and Context

Screenshots (if appropriate)
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Documentation Update (if none of the other choices apply)

Checklist

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.

Codecov Report

Merging #1901 (a3f80b0) into main (8f97e92) will decrease coverage by 0.01%.
The diff coverage is n/a.

:exclamation: Current head a3f80b0 differs from pull request most recent head 56ca6c8. Consider uploading reports for the commit 56ca6c8 to get more accurate results

@@            Coverage Diff             @@
##             main    #1901      +/-   ##
==========================================
- Coverage   43.08%   43.07%   -0.02%
==========================================
  Files         132      132
  Lines       16819    16819
==========================================
- Hits         7247     7244       -3
- Misses       9023     9024       +1
- Partials      549      551       +2

Flag
Coverage Δ

Object-compatibility-e2etests
?

e2etests
?

unittests
43.07% <ø> (-0.02%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...ent/daemon/peer/peertask_piecetask_synchronizer.go
49.54% <0.00%> (-0.92%)
:arrow_down:


dosc: update hostclient config hook

What type of PR is this?

docs
Check the PR title.

[ ] This PR title match the format: <type>(optional scope): <description>
[ ] The description of this PR title is user-oriented and clear enough for others to understand.

(Optional) Translate the PR title into Chinese.
(Optional) More detailed description for this PR(en: English/zh: Chinese).

en:
zh(optional):
(Optional) Which issue(s) this PR fixes:

Fixes https://github.com/cloudwego/hertz/issues/952

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


add Tkazer back to exloader!

🔍 Is there already an issue for your problem?

[X] I have checked older issues, open and closed

ℹ Environment / Computer Info
- add Tkazer back to exloader!

📝 Description

add Tkazer back to exloader!

⚙ Configuration Files

add Tkazer back to exloader!

📸 Screenshots

add Tkazer back to exloader!

we need te old version

What happends? Specific this please

Bruh, this repo is for dexterion use only dont open issued about non dedterion issues.


Scheduler: There are tasks scheduled far away to the future in the Queued tasks section in admin panel

Besides Django-Q sometimes completely freezes up (admin panel available, although tasks are not running), sometimes Queued tasks section in admin panel is full of queued tasks that are queued to run in the ... past and in the far far future (like several month ahead of the current date), although these tasks tend to run every 5 minutes or at most daily. Without cleaning this queue section manually, even qcluster process restart won't help to resume normal tasks running. (to be precise: restart without cleaning is not helpful most of the times, you need first to cleanup queued tasks, then restart qcluster, then things are back to normal).
So, what could be the issue there? Which details are necessary to provide?

Sounds like you are caught in this bug?

sometimes Queued tasks section in admin panel is full of queued tasks that are queued to run in the ... past and in the far far future (like several month ahead of the current date),

That's very strange. Haven't heard of that before. Do you have any logs when that happens? Does it keep updating the date/time or is it a one time thing and then it's stuck on that?


Modernize Gradle Configuration, Dependencies, and Upgrade Gradle to 8…

Modernize Gradle Configuration, Dependencies, and Upgrade to Gradle 8.1.4
This pull request resolves build issues and modernizes the project's build system by:

Upgrading Gradle to 8.1.4 and using the Kotlin DSL.
Updating compile and target SDK versions to 34.
Refreshing key dependencies, including AppCompat and Guava.
Improving JNI libraries configuration and packaging options.
Ensuring Kotlin Stdlib version consistency.

These changes enhance compatibility with the latest Android platform, improve build efficiency, and reduce the final APK size. They also leverage modern Gradle features for a more maintainable and optimized build process.
Considerations:

Verify compatibility with the updated dependencies.
Thoroughly test functionality across different devices.

Changes Made:
To be more specific, here's a summary of the changes:

Gradle: Updated to 8.1.4, enabling new features and Kotlin DSL usage.
SDK Versions: compileSdk and targetSdk are now 34 for latest Android support.
Dependencies: AppCompat, Guava, and support design library updated for better compatibility.
JNI Libraries: Improved configuration for proper packaging within the APK.
Kotlin: Resolution strategy ensures consistent Kotlin Stdlib versions.
Packaging: Optimized to reduce APK size and improve debug build efficiency.

could it be implemented to all code examples or only to superresolution ?


Don't generate time zones with leap seconds by default

Since the tz-rs crate only work with Unix times and not TAI, it is better to generate time zones without leap seconds by default to avoid having to compute the Unix leap times associated to the time zones transitions.

Thank you for the explanation, omitted.


fix: temporarily disable pwas

Current dependencies on/for this PR:

main

PR #2705   👈

This comment was auto-generated by Graphite.


GDB's working directory doesn't match launch location

I noticed this at first when I set did set logging on in the startup commands, and found gdb.txt in the directory vscode was launched from... not alongside the executable.
I tried hardcoding a cd command into the generation of the gdb launch, here:
https://github.com/penagos/vgdb/blob/3eff76c107561b427fe1c0894a90c578b6d3b501/src/debuggers/gdb/GDB.ts#L775
So just doing cd /path/to/debuggee/ ; at the beginning of the executed string, before the $.
When I did that, it not only put gdb.txt in a better place, but made other things work that weren't.  I'd been getting messages from gdb of "No source file named..." on paths that did exist.  But it turns out that message really means "Couldn't find symbols for file named...".  By launching gdb in the directory where the output executable was (instead of where vscode was launched) these errors went away.
After doing this I was able to debug chromium built from source.  And vgdb does not seem to have problems watching Blink::Node variables (which causes vscode-cpptools to hang),

All I did was hardcode a cd to the directory I was working with at the head of the invocation.
Doing it correctly you'd have to decide where to get this directory:

the directory of the executable being debugged?
the cwd property from the launch.json file?
some independent overridable property?

Then there is the question of defaulting (if there's no cwd, then should it fall back to the executable's directory?)