Show progress of buildSrc build on CLI progress bar

Allow BuildProgressFilter to send task graph complete and
after task executed events to BuildProgressLogger for nested builds.
BuildProgressLogger will increment progress during initialization
phase.
Issue: #1538
Context
Branch build
Contributor Checklist

[ ] Link to Design Spec for changes that affect more than 1 public API (that is, not in an internal package) or updates to > 20 files
[x] Provide integration tests (under <subproject>/src/integTest) to verify changes from a user perspective
[x] Provide unit tests (under <subproject>/src/test) to verify logic
[ ] Update User Guide, DSL Reference, and Javadoc for public-facing changes
[x] Ensure that tests pass locally: ./gradlew quickCheck <impacted-subproject>:check

Gradle Core Team Checklist

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation including proper use of @since and @Incubating annotations for all public APIs

@eriwen What's the status on this PR? Would be good if we could get this fixed up/merged soon.

@bmuschko I was going to wait on this change until I could get other console cleanup done because it could affect it, however, given that grouping plain logs output will be more work than expected I think I'll address your feedback now.

@bmuschko I've addressed your feedback and rebased this change atop very latest master. Please give another look when you have a bit.

Rebased and merged with ffd75f2


Installing nuget package "Microsoft.Extensions.Hosting" is necessary when building in docker

I make an app by following this example.  It builds successfully in VS 2017, but fails in docker. The error message is in the follwing:
error CS0234: The type or namespace name 'Extensions' does not exist in the namespace 'Microsoft' (are you missing an assembly reference?).
error CS0246: The type or namespace name 'IHostedService' could not be found (are you missing a using directive or an assembly reference?)
After installing nuget package, the issue is solved.  Just for reference if anyone would like to build this in docker.

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

ID: c3460b0d-9c60-3a06-ac63-18cf990e53c3
Version Independent ID: b2dece1f-15b8-03af-6d52-7cc070432b4b
Content: Background tasks with hosted services in ASP.NET Core
Content Source: aspnetcore/fundamentals/host/hosted-services.md
Product: aspnet-core
GitHub Login: @guardrex
Microsoft Alias: riande

Hello @Julian-Chu ... Thanks for commenting.
Can you share your package references from your project file? I'm curious to see what packages you ending up referencing.

Hi @guardrex ,
Here is my packages:

Microsoft.Extensions.Hosting package is installed for docker, my VS 2017 can build without this.
docker image tag: microsoft/dotnet

Right-click the project and select Edit <PROJECT_NAME>.csproj. Grab the <PackageReference> elements and paste them in here.
Our Web Host sample goes with Microsoft.AspNetCore.App, which includes Microsoft.Extensions.Hosting. The Generic Host sample directly references the Microsoft.Extensions.Hosting package. You can see those two samples here ...
https://github.com/aspnet/Docs/tree/master/aspnetcore/fundamentals/host/hosted-services/samples
The Docker sample app goes with the App package. Here's the project file of that sample ...
https://github.com/dotnet/dotnet-docker/blob/master/samples/aspnetapp/aspnetapp/aspnetapp.csproj

Hi @guardrex ,
I think Microsoft.AspNetCore.App or Microsoft.Extensions.Hosting must be referenced in project correctly like docs and samples, so I suspect it is due to my windows environment, but I have no idea how to identify it. It's not big issue, just to add package manually, and docs/samples are great and correct, so you can close this as well.
Here are 2  .csproj files.  I implemented hosted service in individual project and inject to web project.
Timestealer.HostedService.csproj (service is implemented)
<Project Sdk="Microsoft.NET.Sdk">

  <PropertyGroup>
    <TargetFramework>netcoreapp2.1</TargetFramework>
  </PropertyGroup>

  <ItemGroup>
    <PackageReference Include="Microsoft.EntityFrameworkCore" Version="2.1.3" />
    <PackageReference Include="Microsoft.EntityFrameworkCore.InMemory" Version="2.1.3" />

    // without this,  build can be successful in VS2017 and dotnet CLI
    <PackageReference Include="Microsoft.Extensions.Hosting" Version="2.1.1" />

  </ItemGroup>

  <ItemGroup>
    <ProjectReference Include="..\Timestealer.DataAccessLayer.EFcore\Timestealer.DataAccessLayer.EFcore.csproj" />
    <ProjectReference Include="..\Timestealer.DataAcessLayer\Timestealer.DataAccessLayer.csproj" />
  </ItemGroup>

  <ItemGroup>
    <Reference Include="Microsoft.Extensions.Hosting.Abstractions">
      <HintPath>..\..\..\..\..\..\..\..\Program Files\dotnet\sdk\NuGetFallbackFolder\microsoft.extensions.hosting.abstractions\2.1.1\lib\netstandard2.0\Microsoft.Extensions.Hosting.Abstractions.dll</HintPath>
    </Reference>
  </ItemGroup>
</Project>

Timestealer.Web.csproj (service is injected)
<Project Sdk="Microsoft.NET.Sdk.Web">

  <PropertyGroup>
    <TargetFramework>netcoreapp2.1</TargetFramework>
  </PropertyGroup>

  <PropertyGroup>
    <GenerateDoucmentationFile>true</GenerateDoucmentationFile>
  </PropertyGroup>

  <PropertyGroup Condition="'$(Configuration)|$(Platform)'=='Debug|AnyCPU'">
    <DocumentationFile>bin\$(Configuration)\$(TargetFramework)\$(AssemblyName).xml</DocumentationFile>
  </PropertyGroup>

  <ItemGroup>
    <Folder Include="wwwroot\" />
  </ItemGroup>

  <ItemGroup>
    <PackageReference Include="Microsoft.AspNetCore.All" Version="2.1.4" />
    <PackageReference Include="Swashbuckle.AspNetCore" Version="3.0.0" />
  </ItemGroup>

  <ItemGroup>
    <DotNetCliToolReference Include="Microsoft.VisualStudio.Web.CodeGeneration.Tools" Version="2.0.4" />
  </ItemGroup>

  <ItemGroup>
    <ProjectReference Include="..\Timestealer.DataAccessLayer.EFcore\Timestealer.DataAccessLayer.EFcore.csproj" />
    <ProjectReference Include="..\Timestealer.DataAcessLayer\Timestealer.DataAccessLayer.csproj" />
    <ProjectReference Include="..\Timestealer.HostedService\Timestealer.HostedService.csproj" />
  </ItemGroup>
</Project>

The topic doesn't call out the Microsoft.Extensions.Hosting package near where it describes the IHostedService interface. I'll add language to the topic that specifies the package. Thanks for opening this issue.
RE: Running VS/CLI but not Docker
My guess is that it's transitively referenced by another package. However it comes in, you're right, let's not leave this up to chance or guessing. It's best if the topic is explicit about referencing the package.


Bluetooth: audio: lc3: Fix build error

This fixes build error caused by redundant comma without expression lc3.h:269:74: error: expected expression before ',' token
269 |          (_max_frames_per_sdu)))),
|

@MariuszSkamra I guess this must have come from the fact that we've never set _max_frames_per_sdu to anything but 1, right?
Should we add that in some sample or BSIM test or something, so that we can verify this new change in CI, as well as catch it if it happens again?


[QUESTION]

Question

This could be:
appConfig:
  auth:
    users:
    - user: alicia
      hash: 4D1E58C90B3B94BCAD9848ECCACD6D2A8C9FBC5CA913304BBA5CDEAB36FEEFA3
      type: admin
    - user: bob
      hash: 5E884898DA28047151D0E56F8DC6292773603D0D6AABBDD62A11EF721D1542D8

I try and find authentication · Lissy93/dashy Wiki · GitHub is wrong. Which could be work is from Authentication | Dashy.
Category
Setup and Deployment
Please tick the boxes

[X] You are using a supported version of Dashy (check the first two digits of the version number)
[X] You've checked that this question hasn't already been raised
[X] You've checked the docs and troubleshooting guide
[X] You agree to the code of conduct

If I understand you correctly, you are talking about some error in the Github Wiki?
I can see that your link has an odd number of random characters at the end,if you navigate normally to the github wiki all is fine.
Can you try manually navigating to the wini and then to auth?
Or here the url:
https://github.com/Lissy93/dashy/wiki/authentication
Can you explain where you got the url which leads to that faulty text, from?

Cool! Apparently I made a mistake and didn't use the latest version of the documentation. I guess I saw someone else's tutorial online. Thanks for your answer.


No database type found for the type string specified: 'SqlServerDatabaseType'. Make sure the relevant assembly NPoco.SqlServer is referenced.

Hello dears!
I made the adjustment you indicated but even so when trying to connect to the bank, I am receiving the following exception from the application:
System.Exception: 'No database type found for the type string specified:' SqlServerDatabaseType '. Make sure the relevant assembly NPoco.SqlServer is referenced. '
Could you tell me what I'm doing wrong?
Follow my code:
public List<Employee> Retreave(Employee entity)
{
	using (IDatabase db = new Database(@"Server=SERVER\SQL2019; Database=tEST;User Id=sa;Password=sa@2019;", DatabaseType.SqlServer2012, System.Data.SqlClient.SqlClientFactory.Instance))
	{
		return db.Fetch<Employee>("select * from Employee");
	}
}

My database is Sql Serve 2019, in this wiki: https://github.com/schotime/NPoco/wiki/Database-Constructor-Alternates it is clear that it would not be my version of Sql 2019 that would be causing the failure.

You must reference the NPoco.SqlServer nuget package as suggested.
Its probably easier to also call new SqlServerDatabase()
I'll make a wiki for it.

It worked!!!!
Thank you very much @schotime !
I will close this issue!


PERF: fast inf checking in to_excel

Adds new functions to check for infinity rather than calling np.isposinf and np.isneginf, which were (surprising to me) a significant drag on to_excel.
I also added xlsxwriter to the asv build configuration.  openpyxl is still failing, I'm assuming something do with the specific version on conda?
    before     after       ratio
  [472e6e0e] [8002555d]
   156.36ms   154.16ms      0.99  packers.packers_read_excel.time_packers_read_excel
     failed     failed       n/a  packers.packers_write_excel_openpyxl.time_packers_write_excel_openpyxl
   469.69ms   357.49ms      0.76  packers.packers_write_excel_xlsxwriter.time_packers_write_excel_xlsxwriter
   368.96ms   270.60ms      0.73  packers.packers_write_excel_xlwt.time_packers_write_excel_xlwt

Looks good to me!

I'd rather make a bigger change to how the xls writing is done, to make it much more performant. The idea is to copy what is done for csv writing.
You chunk write, then convert the columns by blocks with to_native_types() (you would have to expand this to for example handle the inf conversions, it already handles the nan and float_format for FloatBlock). these already return object arrays.
Then you simply iterate and yield the ExcelCells, but no conversion is done (as it was all done before), so should be faster/simpler.

Yeah, I had actually started looking into that - this particular change was low hanging fruit, so I thought it might make sense to do first.
There probably is some performance to be picked up still, but an awful lot of the time is spent in the actual writing steps now.
In [1]: df = pd.DataFrame({'b': np.linspace(0, 1000, 100000), 'c': np.linspace(0, 100, 100000)})

In [2]: %prun df.to_excel('temp.xlsx')

   Ordered by: internal time

   ncalls  tottime  percall  cumtime  percall filename:lineno(function)
        1    0.707    0.707    5.212    5.212 excel.py:1410(write_cells)
   300000    0.699    0.000    1.773    0.000 xmlwriter.py:129(_xml_number_element)
     1842    0.612    0.000    0.612    0.000 {built-in method compress}
   300001    0.568    0.000    0.685    0.000 format.py:1817(_format_regular_rows)
   300000    0.543    0.000    1.092    0.000 worksheet.py:482(write_number)
   300002    0.542    0.000    2.490    0.000 worksheet.py:5085(_write_cell)
        1    0.512    0.512    3.912    3.912 worksheet.py:4911(_write_rows)
600004/300002    0.471    0.000    2.439    0.000 worksheet.py:51(cell_wrapper)
   300002    0.430    0.000    2.175    0.000 worksheet.py:349(write)
   300002    0.400    0.000    0.400    0.000 {pandas.json.dumps}
  1500255    0.398    0.000    0.398    0.000 {isinstance}
   502071    0.390    0.000    0.390    0.000 {method 'write' of 'file' objects}
   500157    0.285    0.000    0.923    0.000 codecs.py:353(write)
   300002    0.267    0.000    0.267    0.000 worksheet.py:3459(_check_dimensions)
       11    0.267    0.024    0.267    0.024 {method 'close' of 'file' objects}
   300002    0.265    0.000    0.388    0.000 format.py:1707(_format_value)
   500157    0.252    0.000    0.252    0.000 {_codecs.utf_8_encode}
   400143    0.238    0.000    0.365    0.000 xmlwriter.py:180(_escape_attributes)
   300003    0.207    0.000    1.280    0.000 format.py:1931(get_formatted_cells)
   500157    0.188    0.000    1.111    0.000 codecs.py:692(write)
   300002    0.184    0.000    0.225    0.000 excel.py:494(_conv_value)
   300002    0.157    0.000    0.302    0.000 utility.py:604(supported_datetime)
   300002    0.136    0.000    0.136    0.000 utility.py:37(xl_rowcol_to_cell_fast)
   100001    0.134    0.000    0.507    0.000 worksheet.py:5038(_write_row)
   100001    0.132    0.000    0.354    0.000 xmlwriter.py:61(_xml_start_tag_unencoded)
   400167    0.127    0.000    0.127    0.000 {method 'search' of '_sre.SRE_Pattern' objects}
   300002    0.116    0.000    0.116    0.000 format.py:1644(__init__)
        1    0.116    0.116    0.136    0.136 worksheet.py:4989(_calculate_spans)
   300000    0.111    0.000    0.189    0.000 <string>:8(__new__)
   300004    0.077    0.000    0.077    0.000 {built-in method __new__ of type object at 0x000000001E2A0670}
   100037    0.059    0.000    0.238    0.000 xmlwriter.py:70(_xml_end_tag)
   300000    0.054    0.000    0.054    0.000 worksheet.py:4515(_isinf)

@chris-b1 agreed. Ok rebase this (prob a whatsnew conflict). The can look at other improvements if desired.

@jreback  - rebased


Cannot join discord, getting kicked at Verified Level

Did you follow the steps outline in the README?

[X] Yes I did
[ ] No, I found them confusing

Describe the bug
I completed the appeal form and the captcha but cannot click submit. It sent an error and forwarded me to this github page
To Reproduce
Steps to reproduce the behavior:

Go to 'https://appeal.zksync.dev/form'
Fill all the question
Scroll down to and click submit.
See no error and it doesnt move and respond

Expected behavior
A clear and concise description of what you expected to happen.
It should return an message said you have submitted the form
Screenshots
If applicable, add screenshots to help explain your problem.

Attached
Desktop (please complete the following information):

Browser [e.g. chrome, safari] Chrome
Version [e.g. 22] 118.0.5993.71

Additional context
Add any other context about the problem here.
My account is numezp#0 - michael06#1448
My appeal form is as below:
Why do you think you were you banned?
Using VPN and tried to join again after getting kicked out while using VPN. Love privacy tools though like what zk-rollups can enable but well, it got me kicked
Why do you feel you should be unbanned?
I am just a dude trying to join. I am not here to spam or scam anyone. Not here to impersonate anyone just being myself
Just want to join to share vibe + knowledge about zk rollups and zk meme with cool features like paymaster or layer 3 on top of zksync
What will you do to avoid being banned in the future?
use VPN less and only use 1 account to log in. Don't try to rejoin with a different account.

Sometimes, when it responded,(if it is not frozen) it showed error
Failed to post message to appeals channel using bot token. Please contact and admin or open a ticket here https://github.com/jcsumlin/discord-ban-appeal/issues/new?template=bug_report.md


Add support for Passport-Stripe

Add login support using Passport-Stripe

On hold pending stable Keystone 0.3.0


Prepares 3.5.4 release

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

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

Comparison is base (1d50325) 83.49% compared to head (fbc315d) 83.49%.

:exclamation: Current head fbc315d differs from pull request most recent head 06d00ff. Consider uploading reports for the commit 06d00ff to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #630   +/-   ##
=======================================
  Coverage   83.49%   83.49%
=======================================
  Files          37       37
  Lines        1805     1805
=======================================
  Hits         1507     1507
  Misses        298      298

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


metadata; fmt

Terraform changes from this PR:
Terraform used the selected providers to generate the following execution
plan. Resource actions are indicated with the following symbols:
~ update in-place
Terraform will perform the following actions:
google_cloud_run_service.us_covid_stats will be updated in-place
~ resource "google_cloud_run_service" "us_covid_stats" {
id                         = "locations/us-east1/namespaces/stanke-covid-stats/services/us-covid-stats"
name                       = "us-covid-stats"
# (4 unchanged attributes hidden)
  ~ metadata {
      ~ labels           = {
          + "foo"                           = "bar"
            # (1 unchanged element hidden)
        }
        # (6 unchanged attributes hidden)
    }

  ~ template {

      ~ spec {
            # (2 unchanged attributes hidden)

          ~ containers {
                # (3 unchanged attributes hidden)

              - env {
                  - name  = "CUSTOM_VAR" -> null
                  - value = "foo" -> null
                }

                # (2 unchanged blocks hidden)
            }
        }
        # (1 unchanged block hidden)
    }

    # (1 unchanged block hidden)
}

Plan: 0 to add, 1 to change, 0 to destroy.


[ca/node] Maybe update the root CA when renewing the TLS cert

When renewing the TLS certificate, if we get an x509.UnknownAuthorityError,
try to update the root CA such that it validates against the old TLS creds
before renewing again.
This can help with edge cases where a manager might have been demoted, and
thus not be able to push the latest root certificate to its agent, but
the node needs to renew the TLS cert to get one for the worker role (but
root rotation has already finished, and hence any certificate it gets will
no longer have the intermediate and won't validate).
Signed-off-by: Ying Li<EMAIL_ADDRESS>Still needs test.  The ideal scenario @aaronlehmann and I discussed was to separate the Root CA used to sign new certs from the root CA the node itself uses to validate incoming connections and external servers, but that may be a bigger change.
This functionality may still be useful, however, to account for edge cases.

This isn't compiling yet but the concept looks sound.

Ok, test written and CI passing, ready for another look.

LGTM


missing 3.0.5 tag on GitHub

Will create the release and tag after @bvenners review and merge the following:
https://github.com/scalatest/scalatest/pull/1318

Closing as release tag now available:
https://github.com/scalatest/scalatest/releases/tag/release-3.0.5


it seems like this code is really using the pool - just creating new ones each time...

https://github.com/supabase/grid/blob/e216168b5a35d9dedc812f3917cdcfa886e05d09/example/pages/api/sql-query.ts#L10
I have some other sample code that creates a const pool - that way the pool is reused..
// node-pg returns numerics as strings by default. since we don't expect to
// have large currency values, we'll parse them as floats instead.
types.setTypeParser(1700, val => parseFloat(val));
// These environment variables are set in the repo root's docker-compose.yml
const { DB_PORT, DB_HOST_NAME, POSTGRES_USER, POSTGRES_PASSWORD } = process.env;
// Create a connection pool. This generates new connections for every request.
const db = new Pool({
host: DB_HOST_NAME,
port: DB_PORT,
user: POSTGRES_USER,
password: POSTGRES_PASSWORD,
max: 5,
min: 2,
idleTimeoutMillis: 1000, // close idle clients after 1 second
connectionTimeoutMillis: 1000, // return an error after 1 second if connection could not be established
});
module.exports = db;

I see you shut it down at the bottom - but wouldn't it be better to keep the pool around?

It's ok for an example app.
For prod app, Yes you should keep the pool connection and reused it for better performance.


How to hold down mouse left click?

Can't hold mouse left button down (especially for drawing).
any suggestions?

You can use MouseToggle:
robotgo.MouseToggle("up")
robotgo.MouseToggle("down")

Use toggle():
robotgo.Toggle("left"),
...
robotgo.Toggle("left", "up")


[Question] Rendering  sgl intro a framebuffer - assertion failure

Hi there,
I'm working on a small text app and I managed to render sokol_fontstash into a framebuffer, it works well, but I'm getting an assertion error when validating the pipeline when the sample_count and depth format is different from main default pipeline and pass.
here's more or less what I'm doing,

creating the offscreen image:

sg_image_desc fbo_desc = (sg_image_desc){
        .render_target = true,
        .width = OFFSCREEN_WIDTH,
        .height = OFFSCREEN_HEIGHT,

        .pixel_format = SG_PIXELFORMAT_RGBA8,
        .min_filter = SG_FILTER_LINEAR,
        .mag_filter = SG_FILTER_LINEAR,
        .wrap_u = SG_WRAP_REPEAT,
        .wrap_v = SG_WRAP_REPEAT,

        .sample_count = OFFSCREEN_SAMPLE_COUNT,
        .label = "color-fbo",
    };

offscreen_state.fbo_color_img = sg_make_image(&fbo_desc);

fbo_desc.pixel_format = SG_PIXELFORMAT_DEPTH_STENCIL;
fbo_desc.label = "depth-image";
sg_image fbo_depth_img = sg_make_image(&fbo_desc);

creating the pass for the offscreen buffer:

sg_pass_desc desc = (sg_pass_desc){
            .color_attachments[0].image = off_state.fbo_color_img,
            .depth_stencil_attachment.image = fbo_depth_img,
            .label = "offscreen-attach",
        };
offscreen_state.pass = sg_make_pass(&desc);

Drawing into the FBO

sgl_load_pipeline(offscreen_state.pipeline);
sgl_push_pipeline();

sgl_begin()
....
sgl_end()
sgl_pop_pipeline();

sg_begin_pass(off_state.pass, off_state.pass_action);
sgl_draw();
sg_end_pass();
sg_commit();

drawing the FBO image:

sgl_load_pipeline(app_state.sgl_pip); //default pipeline
sgl_push_pipeline();

sgl_pop_pipeline();

sg_begin_default_pass(&app_state.pass_action, width, height);
sgl_draw();
sg_end_pass();
sg_commit();

if OFF_SCREEN_SAMPLE_COUNT is the same used in main_pass_action and the main sgl_pipeline, no assertion fails.
if it's not the same, I get this error:
sg_apply_pipeline: pipeline MSAA sample count doesn't match render pass attachment sample count
^^^^  VALIDATION FAILED, TERMINATING ^^^^
Assertion failed: (0), function _sg_validate_end, file fips-workspace/sokol/sokol_gfx.h, line 13690.

stepping into the sgl_draw function, it looks like the pipeline being applied is the default one and not the offscreen one ( with samples = 4 )

I'm not sure if I'm doing something wrong, or I'm supposed to use the same settings for the offscreen_pipeline/pass and main pipeline/pass

Hi, the sgl_load_pipeline() needs to go after the sgl_push_pipeline() (because push basically creates a new entry on the pipeline stack, and sgl_load_pipeline() loads a pipeline handle on the current top of the pipeline stack).
After that you will run into a more serious problem though: currently you can't call sgl_draw() twice in the same frame, because of the sg_update_buffer() call here:
https://github.com/floooh/sokol/blob/9aa7891121f4495974c54cd12449c04ad9512e4d/util/sokol_gl.h#L3248
So currently you can only draw in a single render pass.
The proper solution for this are contexts like I'm using the sokol_debugtext.h header, but I didn't get around yet adding context support to sokol_gl.h

Hey,
unfortunately swapping sgl_push_pipeline() with sgl_load_pipeline() didn't fix the assertion error... I still get the same break
here's my offscreen buffer draw code, if it helps clarify anything:
void draw_fbo(){
    const int width = OFFSCREEN_WIDTH;
    const int height = OFFSCREEN_HEIGHT;
    float radius = 100.0f;

    sgl_push_pipeline();
    sgl_load_pipeline(off_state.pipeline);

    sgl_viewport(0, 0, width, height, true);
    sgl_defaults();
    sgl_matrix_mode_projection();
    sgl_ortho(0, width, height, 0, -1.0, 1.0);

    sgl_matrix_mode_modelview();
    int num_of_sides = 32;
    sgl_c3b(255, 30, 50);
    sgl_begin_triangles();
    for(int i = 0; i < num_of_sides; i++){
        float radians = ((float)i / num_of_sides) * 2.0 * M_PI;
        float next_radians = ((float)(i + 1) / num_of_sides) * 2.0 * M_PI;
        float x =  cos(radians) * radius;
        float y =  sin(radians) * radius;
        float nx = cos(next_radians) * radius;
        float ny = sin(next_radians) * radius;
        sgl_v2f(0.0f, 0.0f);
        sgl_v2f(nx, ny);
        sgl_v2f(x, y);
    }
    sgl_end();
    sgl_pop_pipeline();

    sg_begin_pass(off_state.pass, off_state.pass_action);
    sgl_draw();
    sg_end_pass();
    sg_commit();
}

After that you will run into a more serious problem though: currently you can't call sgl_draw() twice in the same frame
Yeah, that's why I ended up calling sg_commit() twice ( 1. at the end of the draw_fbo() function, 2. at the end of the default pass )
The proper solution for this are contexts like I'm using the sokol_debugtext.h header, but I didn't get around yet adding context support to sokol_gl.h.
got it! I was mainly curious to why the pipeline was not being applied, but maybe it's not even worth to take a look at this issue if rendering sgl_ to an fbo is not supported and the way that I got it to work is a bit hackish

unfortunately swapping sgl_push_pipeline() with sgl_load_pipeline() didn't fix the assertion error... I

Oops, yes you're right. It's because I'm patching in the default framebuffer attributes into the pipeline desc here:
https://github.com/floooh/sokol/blob/abadc11723c860ac17a325b7e272c30696710e38/util/sokol_gl.h#L2257-L2265

Yeah, that's why I ended up calling sg_commit() twice

Uh oh, that's "unsupported". It probably works by accident because in some backends.
I've started to work on the context support in this branch:
https://github.com/floooh/sokol/tree/sgl-contexts
With test code and a new sample I should be able to finish this over the weekend. This will provide "proper" support for your use case.

Ok, a bit later then intended, because some real-world stuff got in the way, but sokol_gl.h now has context support, see this sample for how to render both into an offscreen pass and the default pass with sokol_gl.h:
https://floooh.github.io/sokol-html5/sgl-context-sapp-ui.html
...and the updated header documentation:
https://github.com/floooh/sokol/blob/master/util/sokol_gl.h

that's awesome! thank you!

@Hperigo would you mind sharing or explaining how you got sokol_fontstash rendering to an offscreen buffer?
I have difficulties that seem to be similar to what you have achieved here (My problem: https://github.com/floooh/sokol/issues/749) I think it might be the pipeline, which isn't provided in this issue :sweat_smile:


net/http: perhaps merge IsDir checks in fs.go

What version of Go are you using (go version)?

$ go version
go version go1.12.6 linux/amd64

Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
~ go env
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/vagrant/.cache/go-build"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/vagrant/go"
GOPROXY=""
GORACE=""
GOROOT="/usr/local/go"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/linux_amd64"
GCCGO="gccgo"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD="/dev/null"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build301219300=/tmp/go-build -gno-record-gcc-switches"

What did you see instead?
In net/http/tranfers.go: https://github.com/golang/go/blob/master/src/net/http/transfer.go#L254
        if t.ContentLength > 0 {
		return true
	}
	if t.ContentLength < 0 {
		return false
	}
	// Many servers expect a Content-Length for these methods
	if t.Method == "POST" || t.Method == "PUT" {
		return true
	}
	if t.ContentLength == 0 && isIdentity(t.TransferEncoding) {

Since we have already checked t.ContentLength > 0 and t.ContentLength < 0, the t.ContentLength == 0 in the fourth if branch looks useless.

In net/http/fs.go: https://github.com/golang/go/blob/master/src/net/http/fs.go#L586
	if d.IsDir() {
		url := r.URL.Path
		if url[len(url)-1] != '/' {
			localRedirect(w, r, path.Base(url)+"/")
			return
		}
	}

	// use contents of index.html for directory, if present
	if d.IsDir() {
		index := strings.TrimSuffix(name, "/") + indexPage
		ff, err := fs.Open(index)
		if err == nil {
			defer ff.Close()
			dd, err := ff.Stat()
			if err == nil {
				name = index
				d = dd
				f = ff
			}
		}
	}

Maybe we could merge this two if d.IsDir() { into one.
Please correct me if I am wrong.

In the first example, the check may seem useless but it’s explicit, which can be more clear to people when reading the code. I don’t think opening a change for that is necessary.
The second example seems more appropriate to fix since it’s checking d.IsDir() three times in a row.

@andybons , can I work on this?

@andybons
I am not sure if the three IsDir checks can be merged into one, because the d maybe be changed in the second IsDir block. And /path/to/index.html might be a directory which can be opened successfully. Maybe we should merge them into two.
I will give some time this weekend to submit a patch if a PR is welcome.

@bharaththiruveedula @spacewander I will leave it up to both of you to determine who will author a patch.
Keep in mind we're still in the 1.13 freeze right now so the changes won't actually be merged until the 1.14 tree opens.

@andybons
Just submit a PR with a few lines.

@spacewander please follow first come first serve and wait if no activity is there. Please don't override anyone here. Anyways it's fine. Hope we follow some rules in the community!

@bharaththiruveedula as @spacewander filed the issue, it’s fair for them to contribute the change to fix it. There are plenty of other issues to help with.

oh is it.. my apologies @spacewander


render_modes key error

I cant figure out a error I am getting when  setting up a PerceptualDecisionMaking-v0 task.
It gave me an error for not having a 'render_modes' field, so I tried adding one, but it still gave the same error. And I can't quite figure out where to go from here with the error.
# We will import the task from the neurogym library
import neurogym as ngym

# Environment
task = 'PerceptualDecisionMaking-v0'
timing = {
    'fixation': ('choice', (50, 100, 200, 400)),
    'stimulus': ('choice', (100, 200, 400, 800)),
}
kwargs = {'dt': 20, 'timing': timing, 'render_modes':None}
seq_len = 100

# Make supervised dataset
dataset = ngym.Dataset(task, env_kwargs=kwargs, batch_size=16,
                       seq_len=seq_len)

# A sample environment from dataset
env = dataset.env
# Visualize the environment with 2 sample trials
_ = ngym.utils.plot_env(env, num_trials=2)

# Network input and output size
input_size = env.observation_space.shape[0]
output_size = env.action_space.n

Error:
---------------------------------------------------------------------------
KeyError                                  Traceback (most recent call last)
Input In [8], in <cell line: 14>()
     11 seq_len = 100
     13 # Make supervised dataset
---> 14 dataset = ngym.Dataset(task, env_kwargs=kwargs, batch_size=16,
     15                        seq_len=seq_len)
     17 # A sample environment from dataset
     18 env = dataset.env

File /storage/users/MTrappett/neurogym/neurogym/utils/data.py:35, in Dataset.__init__(self, env, env_kwargs, batch_size, seq_len, max_batch, batch_first, cache_len)
     33     if env_kwargs is None:
     34         env_kwargs = {}
---> 35     self.envs = [gym.make(env, **env_kwargs)
     36                  for _ in range(batch_size)]
     37 for env in self.envs:
     38     env.reset()

File /storage/users/MTrappett/neurogym/neurogym/utils/data.py:35, in <listcomp>(.0)
     33     if env_kwargs is None:
     34         env_kwargs = {}
---> 35     self.envs = [gym.make(env, **env_kwargs)
     36                  for _ in range(batch_size)]
     37 for env in self.envs:
     38     env.reset()

File ~/conda_voltar/envs/data-science/lib/python3.9/site-packages/gym/envs/registration.py:625, in make(id, max_episode_steps, autoreset, new_step_api, disable_env_checker, **kwargs)
    623 # If we have access to metadata we check that "render_mode" is valid
    624 if hasattr(env_creator, "metadata"):
--> 625     render_modes = env_creator.metadata["render_modes"]
    627     # We might be able to fall back to the HumanRendering wrapper if 'human' rendering is not supported natively
    628     if (
    629         mode == "human"
    630         and "human" not in render_modes
    631         and ("single_rgb_array" in render_modes or "rgb_array" in render_modes)
    632     ):

KeyError: 'render_modes'

This sounds like it could be a gym version issue. The render_modes key is supposed to be part of the task's metadata, so it's not something you can just override by passing it in kwargs.

Thank you! I will look into my version of gym and see whats going on.

Is this still relevant? Otherwise, we can close it @nacloos @manuelmolano


Remove transaction around consumer loop

We have a case where one system creates data then sends a message to another
system that reads it. Since both actions occur in one of these transaction
loops, the second system cannot see the data created by the first (if it hasn't
yet committed its transaction).
Provided there are no downsides, removing the transaction(s) should be an
overall simplification and resolve the issue.
The original benefits of the transaction are tenuous at best. Because the
offset is being moved before the transaction is opened, this change doesn't
change the retry semantics in the case of errors (those retry semantics may not
be what they want, but this change at least preserves them for now). This
change would make it possible for partial updates being made within the
processing loop to not get entirely rolled back in the case of errors. But our
systems need to be robust to this anyway and should not rely on multi-document
transactions in general.
/cc @codeclimate/review

LGTM


Added ESPOL to Educational domains

Escuela Superior Politécnica del Litoral

@noeliapasaca I am sorry, the domain espol.edu.ec is removed from our trusted domains list because it was abused in the past. If you are the student of this school, please enroll to GitHub Student Developer Program. Then apply for JetBrains educational subscription with your GitHub account (GitHub tab at https://www.jetbrains.com/shop/eform/students).


Add Support for Path Level Parameters and avoid Name Clashes

This PR does 3 things actually:

support path level parameters (including the override mechanic)
adds bodies to generated kdocs
renames all parameters if a name conflict is detected by prefixing all parameters with the location (body for request bodies)

Compatibility:
In some places the parameter order changed which is probably breaking.

the "additionalHeaders" parameter is not yet part of the name clash avoidance, I think a generic way to generate parameter default values would be needed first. It's definitely desirable, since it would also get rid of a few special cases.

Nice work

the "additionalHeaders" parameter is not yet part of the name clash avoidance, I think a generic way to generate parameter default values would be needed first. It's definitely desirable, since it would also get rid of a few special cases.

I think that's fine. We can address that in another PR if anyone hits the issue.

In some places the parameter order changed which is probably breaking.
That's ok, we have another PR merged that is also a breaking change. We can rev a major version for both


[Bug] [seatunel-transform-flink-udf] udf prepare method  properties analysis error

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
`
public void prepare(FlinkEnvironment prepareEnv) {
    final Properties properties = new Properties();
    PropertiesUtil.setProperties(config, properties, UDF_CONFIG_PREFIX, false);

    classNames = new ArrayList<>(properties.size());
    functionNames = new ArrayList<>(properties.size());

    properties.forEach((k, v) -> {
        classNames.add(String.valueOf(k));
        functionNames.add(String.valueOf(k));
    });
}

`
udf {     function.test_1 = "com.example.udf.flink.TestUDF"     function.test_2 = "com.example.udf.flink.TestUDTF"   }
classNames.add(String.valueOf(k))  error,  is not  k ,
it should be   v   , classNames.add(String.valueOf(v))
SeaTunnel Version
dev
SeaTunnel Config
empty

Running Command
empty

Error Exception
empty

Flink or Spark Version
empty
Java or Scala Version
No response
Screenshots
No response
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

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

This is an obvious mistake. Looking forward your PR!

What‘s happend ? If this is a bug, I will reopen it.

If you don't have enough time to fix it, maybe others can help to do it.

Hi, assign me please.

If you don't have enough time to fix it, maybe others can help to do it.

What‘s happend ? If this is a bug, I will reopen it.

Sorry ,  I thought it was fixed  in #3171

I to fix it


dialogCloseClass does NOT remove dialogOpenClass

Hi everyone. I would like to first thank you for the awesome work you put into this lightweight modal plugin. It's very versatile (for me at least) and I love how it integrates with animate.css.
I found a problem - when use dialogOpenClass: 'animated zoomIn' and then you use dialogCloseClass: 'animated zoomOut', everything works fine. But when you try to open the modal again, the modal doesn't show up because the old classes still remain for those elements - in this case, fadeIn and fadeOut still exists, as you can see via Inspect Element.
Here is an example: http://next.plnkr.co/edit/1xszc0nNrAsPLm7G
Click on "Show modal" and then click on "Close". Then try to to click on "Show modal" again and you will see that the modal doesn't actually show up. If you inspect the element, you can see that the old animate classes are still there.

Hi, unfortunately it only works with a single class for now.
You need to specify the following and it should work the way you want to:
{
     dialogClass: 'animated',
     dialogOpenClass: 'zoomIn',
     dialogCloseClass: 'zoomOut'
}

This would probably be fixed in the next release.

Here's what I've been doing that's a working solution for me:
Use removeClass() in afterOpen:
afterOpen: function() {
	$("#modal > div.modal-dialog").removeClass("zoomOut")
}

and do the same thing for afterClose:
afterClose: function() {
	$("#modal > div.modal-dialog").removeClass("zoomIn")
}


[WHLSL] Consider removing constant and device address spaces

Migrated from https://bugs.webkit.org/show_bug.cgi?id=189120:
At 2018-08-29T23:31:44Z<EMAIL_ADDRESS>wrote:
Pointers and array references may refer to constant and device address spaces. HLSL represents them by distinct types, StructuredBuffer and RWStructuredBuffer. We could remove pointers in favor of slices, with * being a shorthand for [0]. This would increase compatibility with existing HLSL source.

Don't think we can do this. Pointers to device memory need to retain their device-ness, and pointers shouldn't be represented by RWStructuredArrays.


Capacity for Dataverse for Teams API calls

[Enter feedback here]
Is capacity for Dataverse for Teams "2000 API Calls /day/user" is correct?
It seems a new limitation because of there is no mention about this limit below.
https://docs.microsoft.com/en-us/powerapps/teams/data-platform-compare#environments
If this is correct, please share how to be counted the API call and what happens if the usage is over the limit.

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

ID: 92a7fa1a-8aeb-414a-fe50-ffe4efe4246b
Version Independent ID: c9051ed9-a131-0f32-4a8b-705830ef2c15
Content: Comparing Microsoft Lists, Dataverse for Teams, and Dataverse - Power Apps
Content Source: powerapps-docs/teams/compare-data-sources.md
Service: powerapps
GitHub Login: @mmercuri
Microsoft Alias: mmercuri

@mgoto-git - The article was updated some time ago to remove the limit and point to this article section for information about API request limits
Closing issue.


Update README.md

Dummy PR
https://devopsmx.atlassian.net/browse/OP-11496
We prefer small, well tested pull requests.
Please refer to Contributing to Spinnaker.
When filling out a pull request, please consider the following:

Follow the commit message conventions found here.
Provide a descriptive summary for your changes.
If it fixes a bug or resolves a feature request, be sure to link to that issue.
Add inline code comments to changes that might not be obvious.
Squash your commits as you keep adding changes.
Add a comment to @spinnaker/reviewers for review if your issue has been outstanding for more than 3 days.

Note that we are unlikely to accept pull requests that add features without prior discussion. The best way to propose a feature is to open an issue first and discuss your ideas there before implementing them.

This PR  triggered this build in this execution, please merge to master only if this succeeds:  https://spin.prod.opsmx.com/#/applications/gcp-gitprtrigger/executions/details/01FTFHDK7VPHFTQJZYHTZ6S1ZP

RunTestJar RESULT: Total tests run: 281, Failures: 10, Skips: 0 Reviewers , please check that the changes in this PR did not cause the failures in  http://jenkins.opsmx.net:8181/jenkins/job/Dev-run-tests-on-gitpr-parallel/ws/emailable-report-1772.html


signal buttons cannot be interacted with on walls

players cant click buttons on walls as of today. seem to be obscured by the wall for some reason, no issue with other wall mounts.

this applies to buttons on the floor as well so its just buttons in general are fucked

It's likely the non-fixture codepath in InRangeUnobstructed is doing something funky because buttons aren't anchored.

Alright so buttons have fixturescomponent but no fixtures so it should just be a prototype update to fix.


Content Audit (Static Site) - Home

Why should we do this?

Current experience: Currently, the SimpleReport homepage uses COVID-19-specific language throughout, limiting its perceived functionality and excluding users seeking support for other point-of-care tests.
Why we want to make this change: By switching to generic language like "rapid point-of-care tests", we accurately reflect the application's broader capabilities and welcome users seeking results for any supported condition. This aligns with the project's goal of supporting wider disease and condition reporting in the future.

User story

As a potential user unfamiliar with SimpleReport, I want the homepage to clearly communicate its function beyond COVID-19 testing, so I can understand its potential value and explore using it for the tests I need.

Action Requested
1. Tagline

Existing text: "A better way to report COVID-19 tests"
New text copy: A simpler way to report rapid point-of-care tests

2. Screenshot

Update the "Conduct Tests" screen image with a current version showing multiple diseases on one test card.
Screenshot of "Conduct Tests" screen image is attached. Check with Kenny that screenshot is right size/looks right on page.
Screenshot: https://www.figma.com/file/LglkDsGZDKMZm3XCkJPhfR/SimpleReport-2023%2B?type=design&node-id=4849%3A14876&mode=design&t=nQ1S7vozAL9v1r6W-1

3. Light Blue Section

Existing text: "Ready to simplify your COVID-19 reporting?"
New text copy: Ready to simplify your rapid point-of-care test reporting?

Acceptance Criteria
Design

[ ] All three updates are implemented on the homepage.
[ ] Visual consistency is maintained with existing design elements.

Review

[ ] Reviewed with design and/or design lead
[ ] Reviewed with product and engineering

Additional Context

No Figma design files are associated with this specific content update.
Screenshots of the impacted areas of the homepage are attached for reference.

@mehansen @jayna-SkylightDigital Updated the ticket with a link to the updated screenshot!


Error: rpc error: code = Unknown desc = no pods matching the input request

Bug Report

General Information
root@kindling-master01:/home/lz/myCode/kubearmor-client# ./karmor summary -p nginx
local port to be used for port forwarding discovery-engine-746fccb855-fmxw6: 32813
Error: rpc error: code = Unknown desc = no system summary info present for the requested pod name

Environment description (GKE, VM-Kubeadm, vagrant-dev-env, minikube, microk8s, ...)

kind

Kernel version (run uname -a)

Linux kindling-master01 5.15.0-46-generic #49~20.04.1-Ubuntu SMP Thu Aug 4 19:15:44 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux

Orchestration system version in use (e.g. kubectl version, ...)

WARNING: This version information is deprecated and will be replaced with the output from kubectl version --short.  Use --output=yaml|json to get the full version.
Client Version: version.Info{Major:"1", Minor:"26", GitVersion:"v1.26.3", GitCommit:"9e644106593f3f4aa98f8a84b23db5fa378900bd", GitTreeState:"clean", BuildDate:"2023-03-18T02:16:17Z", GoVersion:"go1.19.7", Compiler:"gc", Platform:"linux/amd64"}
Kustomize Version: v4.5.7
Server Version: version.Info{Major:"1", Minor:"25", GitVersion:"v1.25.3", GitCommit:"434bfd82814af038ad94d62ebe59b133fcb50506", GitTreeState:"clean", BuildDate:"2022-10-25T19:35:11Z", GoVersion:"go1.19.2", Compiler:"gc", Platform:"linux/amd64"}

Link to relevant artifacts (policies, deployments scripts, ...)
Target containers/pods

To Reproduce
follows: https://github.com/kubearmor/KubeArmor/blob/main/getting-started/workload_visibility.md#how-to-get-the-application-behavior
Install KubeArmor
curl -sfL http://get.kubearmor.io/ | sudo sh -s -- -b /usr/local/bin && karmor install
Install Discovery-engine
kubectl apply -f https://raw.githubusercontent.com/kubearmor/discovery-engine/dev/deployments/k8s/deployment.yaml
Next you install the workload. If the workload is already installed, then the behavior will be shown based on the new system events only.
Get the application summary
karmor summary -n NAMESPACE
Expected behavior
A description of what you expected to happen.
Screenshots

If applicable, add screenshots to help explain your problem.

Could you please tell me how to install nginx pod

Could you please tell me how to install nginx pod @zhy76

apiVersion: v1
kind: Pod
metadata:
  name: nginx
spec:
  containers:
    - name: nginx
      image: nginx

We are aware about KubeArmor not working on Kind and already have an issue for adding kind support: https://github.com/kubearmor/KubeArmor/issues/903

We are aware about KubeArmor not working on Kind and already have an issue for adding kind support: #903

Thanks for pointing that out！

Closing this issue as duplicate. Please include any additional details in the original issue itself.


Add config option to disable/enable line numbers in the status buffer

See https://github.com/NeogitOrg/neogit/discussions/820#discussioncomment-6982271

Great! Thanks for the PR :)


chore(contracts)(sdk): Optimize gas efficiency when fetching a Portal

What does this PR do?
To optimize gas efficiency 2 functions are added in portal registry along with getPortalByAddress
getPortalOwner(address portalAddress) -> address
getPortalRevocability(address portalAddress) -> bool

Related ticket
Fixes #809
Type of change

[x] Chore
[ ] Bug fix
[ ] New feature
[ ] Documentation update

Check list

[x] Unit tests for any smart contract change
[ ] Contracts and functions are documented

Codecov Report
Attention: Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.

Project coverage is 48.59%. Comparing base (ba69bac) to head (da23d74).

Files with missing lines
Patch %
Lines

sdk/src/dataMapper/PortalDataMapper.ts
0.00%
2 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##              dev     #838      +/-   ##
==========================================
- Coverage   48.84%   48.59%   -0.25%
==========================================
  Files          18       18
  Lines         389      391       +2
  Branches       63       63
==========================================
  Hits          190      190
- Misses        196      198       +2
  Partials        3        3

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


glz::reader/glz::writer for incremental reading/writing

Glaze doesn't easily support incremental reading and writing. There are times when we want to just parse the first element of an array, perform logic in our program, and then decide how what item to use to parse the rest of the array. This is common for formats that need header information.
Another use case is appending data to a buffer incrementally.
This issue proposes the addition of two stateful classes: gz::reader and glz::writer, which will enable incremental reading/writing.
Initially this will focus on array support, but object support will be considered in the design.
Example syntax:
glz::reader reader{buffer};
reader.read(glz::element<0>(value));
reader.read(glz::rest(value));

glz::writer writer{buffer};
writer.append(value);
write.append(value);

Addresses: #917

Note that partial reading, which is supported for JSON, handles this problem in another manner.

JSON pointer syntax reading also supports this in part. The main difference is that we want a core way to maintain iterators at the parsed location, for the best possible speed. I think this will also take a good step in the direction of handling streams of data.

The new read_allocated option in #1027 also provides a solution for partial reading.

read_allocated has been renamed to partial_read in #1028

The new API design proposal looks nice.

This would be a very welcome addition.


enable JDK_REPO and JDK_BRANCH

Signed-off-by: Sophia Guo<EMAIL_ADDRESS>

@sophia-guo Should we close this PR?

Yes, close it.


sap_hana_install: Add tag for only creating the hdblcm configfile

With the additional tag sap_hana_install_create_hdblcm_configfile the role can be used for creating the hdblcm configfile only. This can be useful for error analysis.

@marcelmamula Do you have any comments on this one?


[Bug]: git not found

Error Message and Logs
bash: line 4: git: command not found
Steps to Reproduce

Create a new resource
Choose "Private Repository" (with or without app, doesn't matter) - this could probably be public too
Choose build pack as "Docker Compose"
Attempt to save or refresh the compose file
The toast error "bash: line 4: git: command not found" shows up

Example Repository URL
No response
Coolify Version
v4.0.0-beta.360
Are you using Coolify Cloud?
Yes (Coolify Cloud)
Operating System and Version (self-hosted)
No response
Additional Information
Currently having to deploy a service stack for my docker compose setup, rather than hooking it up to a git workflow with deployment history etc

Being a tool - needed git installed on my host 😆

Did you not use the install script?

@peaklabs-dev using Coolify cloud - just installed docker + went through the walkthrough on the dashboard.


HTML

The bug
When using a HTML <template> element in lume, its contents are cleared when building with a special configuration of plugins.
Demo
Project setup
src/index.njk
<!doctype html>

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

	<template>

Hey

        </template>

_config.ts
import lume from "lume/mod.ts";
import basePath from "lume/plugins/base_path.ts";
import url from "lume/plugins/url.ts";

const site = lume({
  src: "src",
  dest: "build",
});

site
  // Use some random plugins here
  .use(basePath())
  .use(url());

export default site;

Test run
lume && cat build/index.html

Expected
🔥 /index.html /index.njk

🍾 Site built into build
<!doctype html>

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

	    <template>

Hey

        </template>

Actual
🔥 /index.html /index.njk

🍾 Site built into build
<!DOCTYPE html>

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

	    <template></template>

Culprits
In _config.ts above, comment out the line .use(basePath()) and the build will work as expected. Replacing basePath with the inline plugin will also cause the bug. Not the date plugin, however.
Versions
$ lume --version
v1.4.2
$ deno --version
deno 1.17.1 (release, x86_64-apple-darwin)
v8 <IP_ADDRESS>
typescript 4.5.2

I thought I was going mad while developing my code, since my <template> elements kept going empty on build, whereas they worked all over the rest of the internet 😄😄😄

Yes, this is an known bug in deno-dom (https://github.com/b-fuze/deno-dom/issues/18) that wasn't fixed yet and I'm not sure when is going to be fixed, it's going slow :(

Aha! Alpha status things ✨ Then I know, at least. Thanks!

I think this was fixed in the latest deno-dom version, which is included in the latest Lume version (1.7.3).


xmr-stak: 2.4.4 -> 2.4.5

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools.
This update was made based on information from https://repology.org/metapackage/xmr-stak/versions.
These checks were done:

built on NixOS
/nix/store/wbq2l97g7y24hnbz1zzs17yl8qh1csd3-xmr-stak-2.4.5/bin/xmr-stak passed the binary check.
Warning: no invocation of /nix/store/wbq2l97g7y24hnbz1zzs17yl8qh1csd3-xmr-stak-2.4.5/bin/libxmrstak_opencl_backend.so had a zero exit code or showed the expected version
1 of 2 passed binary check by having a zero exit code.
0 of 2 passed binary check by having the new version present in output.
found 2.4.5 with grep in /nix/store/wbq2l97g7y24hnbz1zzs17yl8qh1csd3-xmr-stak-2.4.5
directory tree listing: https://gist.github.com/d748f1490c29ab43e9426b5d283a5e4e
du listing: https://gist.github.com/06e416d3c3db5caf733655c9ab632eea

meta.description for xmr-stak is: '"Unified All-in-one Monero miner"'.
cc @fpletz for testing.
Experimental: this build is cached with Cachix. To use the cache follow these experimental instructions:
One time setup in nixpkgs Git checkout:
cachix use r-ryantm
git remote add r-ryantm https://github.com/r-ryantm/nixpkgs.git

Test this build:
git fetch r-ryantm && git checkout 1c479b27fa41e0b92374d59a2847f5488586fcd9 && nix-shell --pure -I nixpkgs=. -p xmr-stak

Failure on aarch64-linux (full log)
Attempted: xmr-stak
Partial log (click to expand)
make[2]: *** [CMakeFiles/xmr-stak-c.dir/build.make:115: CMakeFiles/xmr-stak-c.dir/xmrstak/backend/cpu/crypto/c_skein.c.o] Error 1
gcc: error: unrecognized command line option '-maes'
gcc: error: unrecognized command line option '-maes'
make[2]: *** [CMakeFiles/xmr-stak-c.dir/build.make:76: CMakeFiles/xmr-stak-c.dir/xmrstak/backend/cpu/crypto/c_groestl.c.o] Error 1
make[2]: *** [CMakeFiles/xmr-stak-c.dir/build.make:89: CMakeFiles/xmr-stak-c.dir/xmrstak/backend/cpu/crypto/c_jh.c.o] Error 1
make[2]: *** [CMakeFiles/xmr-stak-c.dir/build.make:102: CMakeFiles/xmr-stak-c.dir/xmrstak/backend/cpu/crypto/c_keccak.c.o] Error 1
make[1]: *** [CMakeFiles/Makefile2:68: CMakeFiles/xmr-stak-c.dir/all] Error 2
make: *** [Makefile:130: all] Error 2
builder for '/nix/store/qx7rqszqfzf4iksmb33gajc87dl6sx9d-xmr-stak-2.4.5.drv' failed with exit code 2
error: build of '/nix/store/qx7rqszqfzf4iksmb33gajc87dl6sx9d-xmr-stak-2.4.5.drv' failed

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: xmr-stak
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

Success on x86_64-linux (full log)
Attempted: xmr-stak
Partial log (click to expand)
-- Installing: /nix/store/wbq2l97g7y24hnbz1zzs17yl8qh1csd3-xmr-stak-2.4.5/bin/libxmrstak_opencl_backend.so
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/wbq2l97g7y24hnbz1zzs17yl8qh1csd3-xmr-stak-2.4.5
shrinking /nix/store/wbq2l97g7y24hnbz1zzs17yl8qh1csd3-xmr-stak-2.4.5/bin/xmr-stak
shrinking /nix/store/wbq2l97g7y24hnbz1zzs17yl8qh1csd3-xmr-stak-2.4.5/bin/libxmrstak_opencl_backend.so
strip is /nix/store/21ymadblbmsbb2bk4q7gl4kjasp8zmgd-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/wbq2l97g7y24hnbz1zzs17yl8qh1csd3-xmr-stak-2.4.5/bin
patching script interpreter paths in /nix/store/wbq2l97g7y24hnbz1zzs17yl8qh1csd3-xmr-stak-2.4.5
checking for references to /build in /nix/store/wbq2l97g7y24hnbz1zzs17yl8qh1csd3-xmr-stak-2.4.5...
/nix/store/wbq2l97g7y24hnbz1zzs17yl8qh1csd3-xmr-stak-2.4.5


GCN disassembler tools

Hi, recently I wanted to disassemble a kernel binary file *.co into assembly code *.s. I have tried the tools listed in https://rocmdocs.amd.com/en/latest/Programming_Guides/gcn-assembler.html, but it seems that these tools could not generate a complete assembly code file. So, does ROCm provide such a disassembler tool?

Don't know such a tool. llvm-objdump can show disassembly, but cannot produce source file for (re-)assembly.
Only known method for producing *.s files known to me is setting environment variables before running OpenCL/HIP application according to #726
export KMDUMPISA=1
export AMD_OCL_BUILD_OPTIONS_APPEND="-save-temps -g"


Initial draft. Need to add links and pics.

Work in progress here for now.

OK now i think @ejjick
I've added you as a reviewer @jokershaw so you can accept whenever you want.

nudge here @jokershaw


Error when testing Reflex Cloud

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Code/Link to Repo:

Expected behavior
A clear and concise description of what you expected to happen.
When following the steps to test the reflex cloud I received this error
Screenshots
If applicable, add screenshots to help explain your problem.

Specifics (please complete the following information):

Python Version: 3.12.7
Reflex Version: reflex==0.6.6.post2
OS:
Browser (Optional): Brave linux in fedora 41

Additional context
Add any other context about the problem here.

out of curiosity, what's the output when you do pip freeze | grep reflex-hosting-cli?

Thank you very much for replying, I have the following result
( pip freeze | grep reflex-hosting-cli = reflex-hosting-cli==0.1.29 )

I changed the Python version to 3.12.0 and changed the browser to run correctly.


Bump jinja2 version

[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Pull Request Attributes
Please check any characteristics that apply to this pull request.
[ ] Does not have any changes that may affect Istio users.

Is this because a bot suggested it and that bot can't sign the CLA?

A bot suggested it in private since its a security update - but not one that tangibly impacts istio

and in public too: https://github.com/istio/istio/pull/31591

Oops, didn't see that. It has the CLA signed so that one should work too

@howardjohn: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-telemetry-mc-k8s-tests_istio
49574a953fc46d6bf2c22063eba66402ca40b38e
link
/test integ-telemetry-mc-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-telemetry-mc-k8s-tests_istio
49574a953fc46d6bf2c22063eba66402ca40b38e
link
/test integ-telemetry-mc-k8s-tests_istio

integ-telemetry-k8s-tests_istio
49574a953fc46d6bf2c22063eba66402ca40b38e
link
/test integ-telemetry-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest


If speakers does not add a title it creates unwanted space in speaker description

If a speakers adds both title and company things look fine on the schedule and tracks page. But, if a speaker does not add a job title there is an unwanted space after brackets, e.g. "Daniel Blueman ( FOSSASIA)" instead of  "Daniel Blueman (FOSSASIA)".
Compare below:

Can you write steps to reproduce this.

Is this issue is still open, If yes then I want to fix this.

It's open

Can I work on this issue??

Yes

@iamareebjamal Seems like no one is working on this at present. I would like to take up this issue. Can you please assign me this issue?

Please send a PR

@mariobehling @iamareebjamal I wanted to manually test my changes for this issue, so I tried running the app locally and got an error while extracting zip. Then I tried on development-version and production-version to check if there is some error in the local setup or the error is in the codebase. I am getting the same error on both the development-version and the production-version. I am able to figure out the bug. Please do let me know if a new issue needs to be created.

Use API method, not ZIP


Public access to feature Advanced analytics (beta)

I'm unsure whether it's a bug or not, but the new [Advanced analytics (beta)] feature is publicly accessible.

Hi @rommansabbir,
Thank you for bringing this to our attention.
Currently, the new Advanced Analytics (Beta) feature is designed to be publicly accessible. This aligns with how our analytics have traditionally been shareable.
However, we understand the importance of privacy and are planning to introduce a configurable option that will allow users to set the analytics to private by default. This upcoming enhancement will give you more control over who can access your analytics data.
We appreciate your feedback and are continuously working to improve our features to better serve our users.
Best regards,
Harsh

That's awesome!


Page has no attribute site_id [Docs]

There is a bug in the docs that concerns missing attribute. To get information about site_id in ExtensionToolbar you have to use page.node.site_id

Coverage remained the same at 78.189% when pulling 173e8ad4fc87d47a74ad97c281d8773733d55b18 on pawelmarkowski:develop into 8007ce3d2faff541eb2d1e7efba9df615a26649e on divio:develop.

Thanks, @pawelmarkowski. We forgot to reflect that change in our documentation.


New version: RobotOS v0.8.0

Registering package: RobotOS
Repository: https://github.com/jdlangs/RobotOS.jl
Created by: @jdlangs
Version: v0.8.0
Commit: 233a1c643c364b664e93eedc22ea05200191ed2c
Reviewed by: @jdlangs
Reference: https://github.com/jdlangs/RobotOS.jl/commit/233a1c643c364b664e93eedc22ea05200191ed2c#commitcomment-41573694

This PR may be closed, I will do some cleanup of auto-merge incompatibilities.


TH16 "crazy Humidity" after update to v2.3.0

Hi.
After the update to version 2.3.0, the humidity values went "crazy".
Attached screenshot and debug

AlexxIT Sonoff Humidity problem v230.txt

I had the same problem, my setting was at 1 minute, I changed for 2 minutes, restart HA, and everything came back ok

I had the same problem, my setting was at 1 minute, I changed for 2 minutes, restart HA, and everything came back ok

I will try
Thanks

Not work for me 😔
Modify to 2 minutes and restart HA
Humidity 0.81%  😲

I also see some strange updates since the last version. For example the temperature reads 41 degrees, but for a moment it changes to 0.41, then goes back to 41.

I also see some strange updates since the last version. For example the temperature reads 41 degrees, but for a moment it changes to 0.41, then goes back to 41.

With mine, all Sonoff sensors gone after update to v2.3.0. Why do you still have sensors?

I've had a similar issue with my humidity sensor since the update. Until a fix is released, I'd recommend reverting back to 2.2.3.

Yeah, back to 2.2.3. Works again!

I had the same problem, my setting was at 1 minute, I changed for 2 minutes, restart HA, and everything came back ok

I will try
Thanks

this fix not work , is has temporise , i wait for new fix to.

Bug is fixed now. Update to latest version


sensors: Add definition for ambient temperature

Some device include a temperature sensor that measure the die temperature
and can be used to compensate other sensor's drift. This temperature IS NOT
related to the the ambient temperature for which a new separated definition
is required.
Signed-off-by: Armando Visconti<EMAIL_ADDRESS>

@ydamigos
Yes, I also took into consideration that (i.e. defining DIE instead of AMBIENT). Since I work a lot on Android I decided to go for AMBIENT: it seemed more clear to me.
To be honest I would not change other sensors implementation, as I'm not sure for each sensor whether the referred temperature is DIE or AMBIENT (should not be that difficult though). Rather I wanted to take an Android approach, i.e. older device would report SENSOR_CHAN_TEMP while newer device should report the new definitions. If this is not acceptable I would try to evaluate how many additional changes I should do.

Codecov Report

Merging #6763 into master will increase coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #6763      +/-   ##
==========================================
+ Coverage   54.39%   54.39%   +<.01%
==========================================
  Files         444      444
  Lines       42257    42257
  Branches     8000     8000
==========================================
+ Hits        22986    22987       +1
+ Misses      16058    16057       -1
  Partials     3213     3213

Impacted Files
Coverage Δ

kernel/posix/pthread.c
70% <0%> (+0.5%)
:arrow_up:

Continue to review full report at Codecov.

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

@nashif @ydamigos
OK, I will do that.
Now, what is the best according to you?

Leaving SENSOR_CHAN_TEMP for ambient temperature and define new SENSOR_CHAN_DIE_TEMP (Less effort for alignment but maybe less clear define)
Defining SENSOR_CHAN_AMBIENT_TEMP and leaving SENSOR_CHAN_TEMP for die temperature

Please let me know and I'll proceed.

OK, I decided to go for #1 described above.
Following sensors have been left as SENSOR_CHAN_TEMP because I know they measure
the die temperature and are used for compensating the drift:

lis3mdl
lsm6ds0
lsm6dsl

My impression is that there are other sensors (like bmi160 and bmg160) that do the same, but I'm not
sure.

Now, what is the best according to you?

Leaving SENSOR_CHAN_TEMP for ambient temperature and define new SENSOR_CHAN_DIE_TEMP (Less effort for alignment but maybe less clear define)
Defining SENSOR_CHAN_AMBIENT_TEMP and leaving SENSOR_CHAN_TEMP for die temperature

IMO, defining SENSOR_CHAN_AMBIENT_TEMP for ambient temperature and SENSOR_CHAN_DIE_TEMP for die temperature, is the best option.

IMO, defining SENSOR_CHAN_AMBIENT_TEMP for ambient temperature and
SENSOR_CHAN_DIE_TEMP for die temperature, is the best option.

MMmmh. OK, but maybe we can just extend the sensor_channel types, so leaving in the SENSOR_CHAN_TEMP definition (with a comment that indicates its obsolescence).
is it ok for everybody?

re-pushed including @ydamigos suggestions.

 So repushed again.
The list of DIE temp sensor I'm aware of is the following:

lis3mdl
lsm6ds0
lsm6dsl
lsm9ds0

@nashif
Is the new commit matching now your expectations?
@bogdan-davidoaia   @erwango
Can you please take a look at it as well?
Thx!!


fix(ocp-names): shorten generated names

Kustomize is configured to add integration-service prefix to each name. However many objects had already integration-service prefix defined which lead into duplicated names like
integration-service-integration-service-controller-manager
This had also another side effect causing that pod name was longer than 63 characters and it was truncated
generateName: integration-service-integration-service-controller-manager-6c6c5dcb65-
name: integration-service-integration-service-controller-managerc8sww

Removing extra integration-service prefix will solve this issue.
Maintainers will complete the following section

[ ] Commit messages are descriptive enough (hints)
[ ] Code coverage from testing does not decrease and new code is covered (check the PR coverage on codecov)
[ ] Controllers diagrams are updated when PR changes controllers code  (if applicable)

I was too optimistic that this would be so easy :D
 Error: accumulating components: accumulateDirectory: "recursed accumulation of path '.components/integration/k-components/manager-resources-patch': no matches for Id Deployment.v1.apps/integration-service-controller-manager.system; failed to find unique target for patch Deployment.v1.apps/integration-service-controller-manager.system

This should work after changing components/integration/k-components/manager-resources-patch/manager_resources_patch.yaml in infra-deplyoments, however I don't know who to do separated stage/prod deployments, chaging it for stage would break prod. It needs more work in infra repo, keeping this as a draft because of it.

/retest

Tests are failing on this, however it's from infra repo, so I need to merge this and then update infra repo with the new integration-service image and reference
[{"lastTransitionTime":"2024-01-18T14:07:43Z","message":"rpc error: code = Unknown desc = `kustomize build .components/integration/development` failed exit status 1: # Warning: 'bases' is deprecated. Please use 'resources' instead. Run 'kustomize edit fix' to update your Kustomization automatically.\n# Warning: 'patchesStrategicMerge' is deprecated. Please use 'patches' instead. Run 'kustomize edit fix' to update your Kustomization automatically.\n# Warning: 'patchesStrategicMerge' is deprecated. Please use 'patches' instead. Run 'kustomize edit fix' to update your Kustomization automatically.\nError: accumulating components: accumulateDirectory: \"recursed accumulation of path '.components/integration/k-components/manager-resources-patch': no matches for Id Deployment.v1.apps/integration-service-controller-manager.system; failed to find unique target for patch Deployment.v1.apps/integration-service-controller-manager.system\"","type":"ComparisonError"}]

https://github.com/redhat-appstudio/infra-deployments/pull/3094 this PR will allow me to do rolling changes, but still I need to merge this with broken deployment first

/test integration-service-e2e


Config flow - missing attribute labels

Version of the custom_component
1.0.0

Configuration
N/A
Describe the bug
When I go to Configuration -> Integrations and select tesla_custom, the attribute labels are missing in the config flow form, so I can't tell what I'm supposed to enter into these fields. See below screenshot.

HA version: 2021.9.3
Debug log
N/A

What language is HA set to?
You say you select tesla_custom? Not “Tesla Custom Integration”? It sounds like HA hasn’t loaded translations at all. Have you tried a forced refresh in the browser?
(The first field is email, second is refresh token)

What language is HA set to?

English.

You say you select tesla_custom? Not “Tesla Custom Integration”? It sounds like HA hasn’t loaded translations at all. Have you tried a forced refresh in the browser?
(The first field is email, second is refresh token)

Yes, it appears as "tesla_custom". I tried a hard refresh but no change. Tried restarting HA multiple times. Also tried in an incognito window, with a different browser entirely, and with a different computer.

I figured it out. I had put the integration in custom_components in a folder called tesla, not tesla_custom. Didn't know this mattered.


ci: add and configure golang-lint

Codecov Report
Attention: Patch coverage is 15.00000% with 34 lines in your changes are missing coverage. Please review.

Project coverage is 48.59%. Comparing base (863c69d) to head (e8c2a03).

Files
Patch %
Lines

crowdin/model/string_translations.go
0.00%
13 Missing :warning:

crowdin/model/source_files.go
0.00%
11 Missing :warning:

crowdin/model/translations.go
0.00%
10 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #27      +/-   ##
==========================================
- Coverage   48.63%   48.59%   -0.04%
==========================================
  Files          23       23
  Lines        1094     1093       -1
==========================================
- Hits          532      531       -1
  Misses        519      519
  Partials       43       43

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


Data Table

Hi
Can you please explain the data table entries and examples of the values that need to be populated or does the flow populate them?

@NagWolf it really depends on what you want to show in the UI for the customer. This example is built on the ability to show both the "Transcript" & "Survey" as buttons at the end of the session. So for that you need a "KEY" which is the conversationId,  the transcriptUrl as well as the surveyUrl as additional columns. This is so that the Buttons in the popup can link out to these locations, info in is the readMe here: https://github.com/mcphee11/genesys-cloud-messaging-survey?tab=readme-ov-file#step-4---create-datatable if you also wanted to show additional buttons of options then you may want to have additional columns for more info. Its really the survey that causes this level of complexity due to the way the policy runs today. If you only wanted the transcript i would do it a different way to make life simplier. Remember this is designed as an example and you can make it into your own through the APIs. Any more quesitons let me know.
Cheers.


Version Bump

Hi
I see that the Windows app has a few version bumps since the Mac one had
https://github.com/veler/DevToys
Did someone work on updating this repo to include the fixes?
Thank you

Hi,
I'm the author of the original DevToys for Windows. I feel like this DevToysMac repository is not maintained anymore.
On my side, I'm working on DevToys 2.0, which will run on Windows Mac and Linux. Stay tuned!
@ObuchiYuki , if you do not maintain this app anymore, I would suggest to consider archiving the repository so people don't get too confused by its state.

Thanks @veler
To you have any timeline or place to be updated once the app will be out?
Thanks

Thanks @veler
To you have any timeline or place to be updated once the app will be out?
Thanks

We're aiming for Spring 2024. Best is to follow our repository and website https://devtoys.app


Fix/sync app and ros param

Fixes #224.
Fixes #221.
It also fixes some errors regarding the GetMapUuids service.

@smits Thanks for your review. I addressed your comments.
Here is the current behavior:
The preferences are set from the ROS params when:

starting the app
connecting to Tango
The ROS params are set from the app preferences when:
pressing the RESTART TANGO button on the snackbar.

@mcopejans Thanks for your review.


Add the ls-extension  - Metadata JSON to record converter

Purpose

Add the language server extension, Metadata JSON to record converter. This will be used to generate Ballerina records from JSON objects which contains metadata related to CRM platforms such as Salesforce
Fixes
https://github.com/ballerina-platform/ballerina-extended-library/issues/229
Approach
The new Custom Record generator LS-Extension is intended to generate Ballerina records for the metadata obtained from CRM platforms. This will provide the capability to the developer to generate Ballerina records from metadata of CRM platforms

Proposal
https://docs.google.com/document/d/1PL1VcT4vw1FEwn7gY2kVSWukBxjvSedh9MVtW7vIqIs/edit?usp=sharing
Specification
https://docs.google.com/document/d/1RJWjafSo9F-o3KP59n-ZUA8mA5mCOMLXjYd-isyRkNo/edit?usp=sharing

Remarks
Related PRs

https://github.com/wso2-enterprise/ballerina-low-code-editor/pull/1071
https://github.com/wso2-enterprise/ballerina-plugin-vscode/pull/446

Check List

[x] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[x] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

This PR is not needed because the suggested LS-Extension for Metadata to JSON converter development is discontinued development due to design decisions taken. Please refer the meeting notes here for more information
https://docs.google.com/document/d/1PL1VcT4vw1FEwn7gY2kVSWukBxjvSedh9MVtW7vIqIs/edit?usp=sharing


Rename package from javax to jakarta

Signed-off-by: tvallin<EMAIL_ADDRESS>

can you rebase this on master so it can be merged, please?


Fix get_matrix returning the same value twice

Questions
Answers

Description?
Now that autoupgrade results are pushed on the nightlyboard, these results were used in the get_matrix function leading to duplicate jobs. This PR fixes it

Type?
bug fix

BC breaks?
no

Deprecations?
no

Fixed ticket?

How to test?
php .github/get_matrix.php should return a json table with only 2 objects.

Possible impacts?

Hi @atomiix !
Without the PR :

With the PR :

QA :heavy_check_mark:
Thanks !

Thank you @atomiix


Fix version checks in shell bundle install and upgrade

@Microsoft/ostc-devs
Installed omi was getting downgraded during shell bundle install and upgrade on unix.
Also version check of omi was not done during install on Linux.
Added version checks to resolve the issue.

Taking this fix to SCOM shiproom. Will merge this after getting required approvals.


Add user authentication

This allows

Grabbing the user photo for the nav drawer.
Starring and unstarring repos from the app.
Concrete demonstration of contextual debug actions.

Do you have specific examples of contextual debug actions that you would want to show for a "Logged In" user?
cc @mattprecious

The action is of logging in on the sign in screen itself.

Yes, this would be a mock-mode specific action that would just enter sane data to exercise the UI and get you through the screen with as few taps as possible.


PEP8 import optimizer

I noticed a few unused import statements. This is potentially problematic because it clutters up the name space and adds overhead when importing sncosmo. The latter of these is especially important when running on systems like NERSC where imports are already painfully slow. I took the liberty of running an automatic PEP8 import optimizer.
NERSC is down for maintenance while they prepare for the Perlmutter machine, but I get a small ~1.6 % improvement in runtime when importing sncosmo on my local machine. The improvement is tiny, but it's essentially free.
Before optimization: 0.682 ms
After optimization: 0.671 ms

Looks good!


A way to hide my real IP ?

Hi everyone,
I just installed PiVPN, and if I understand correctly it works like this (I'm not familiar at all with VPNs) : I connect on the Internet without PiVPN and if I check my IP on a website like https://whatismyipaddress.com/ it shows my real IP, but if I connect to my raspberry, and then go to the same site, it shows the IP where my raspberry is connected (in my case : at home).
My question is : is it "risky" to show our real IP ? Because in my case, with or without PiVPN, I have the same IP (I'm in my home and my raspberry too). And by the way, does my Internet access provider can see all of my browser data and all of these stuff, even if my IP is still visible ? My main goal was to protect me against big websites and my Internet access provider, but as my IP is still visible, I'm asking many questions and to be honest, I don't really know which Internet data is visible for Internet access provider / big sites like Twitter or Google.
Can you explain me how it works and in which proportions am I protected ? It would be very nice ! Thanks ;)

Yes, you're still "visible" for your ISP. The main usecase for PiVPN (an installer script for OpenVPN) is to be able to securely connect to your home network from an untrusted location. For example, if you're in a restaurant and don't trust the wifi, you'll still be able to have a secure connection. PiVPN isn't meant to keep you anonymous: there exist VPN providers like PIA, or the TOR network for that. Please reply if you have any questions.

Well, no, not quite. Your connection between you and the VPN server is secure, but the connection between the VPN server and the ISP is as secure as it would be if you were using the network at home. PiVPN secures against for example someone intercepting the traffic between you and the VPN server.


tests: migrate tests to pytests

Instead of writing assertions using unittest, the tests now use
Python's built-in assert.
Updates test script so poetry runs pytest instead of coverage.
Resolves #24.

Nice work!
Just had a bit of a look at coverage, and pytest-cov seems to be good. Running it locally, I made the following changes:
Under [tool.poetry.dev-dependencies] in pyproject.toml
pytest = "^6.2.5"
pytest-cov = "^2.12.1"

And in scripts.py, changed the coverage call to:
    subprocess.call('pytest --cov=royston tests/', shell=True)

Nice work!
Just had a bit of a look at coverage, and pytest-cov seems to be good. Running it locally, I made the following changes:
Under [tool.poetry.dev-dependencies] in pyproject.toml
pytest = "^6.2.5"
pytest-cov = "^2.12.1"

And in scripts.py, changed the coverage call to:
    subprocess.call('pytest --cov=royston tests/', shell=True)

Thank you for your review! Just pushed an update with those two changes :)

Thank you very much! 🙇♂


Filtered calendar view of plenary doesn't work

Describe the issue
Go to the 115 HTML agenda and filter for the plenary. It doesn't show up in the view, and the query string is ?show=None.
(using Safari latest)
Code of Conduct

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

https://datatracker.ietf.org/meeting/115/agenda?show=ietf-plenary is what you wanted (there's a button for it under the Plenary grouping in the filters), but it's clear why this changed from just "plenary" and why there is a separate "planary" thing that causes show=None to happen. Will investigate.

Aha - thanks!

So - this was an issue with the filter choser for the older agenda (which was just removed). The root was that it was gathering slugified session names and session purpose names for sessions that weren't regular wg/rg sessions.
>>> Session.objects.filter(meeting__number='115',name__icontains='plenary').values_list('name','purpose__name')
<SessionQuerySet [('IETF Plenary', 'Plenary')]>

The chooser for the new agenda handles this better. I'm going to close this issue with no code changes. Please reopen if you find the same problem in the new agenda in a place I haven't managed to see.


I'd like to use a game score history api.

I'd like to use a game score or history api.
Hello. I want to search my entire game score or history while playing dead by daylight, but I don't have any related sites. So I want to make it myself. If you don't mind, could you tell me if there is an api related to the game score or history? If you have, could you get the api? I'm a beginner in development, so I can't create api, so please get that api.

Ah, sorry but I have not come across any game history API's and most of the reported player stats we get are mainly from the Steam API.


Too many open issues

EDIT
I have added the Hacktoberfest label to this, so that we can get some more outside feedback. If you are here because of this: Welcome!
This is not a coding related issue but goes more in the direction of community building, project maintenance and guidelines. If you have experience with open source projects, we would appreciate your input and even if you don't we'd still like to hear from you.
END EDIT
This project has grown quite a bit over the last year. And also recently the number of new issues picked up a lot. While it is great that more people use the project and get involved, it quickly can get overwhelming getting a bunch of new issues every day, many of which would take quite a lot of time to address. Having this many issues open can also lead to an issue getting lost next to all the other open ones.
With this thread, I want to start a conversation and get feedback on how we can reduce the numbers of issues opened as well as the average time it takes to process issues (especially more trivial ones).
This will give us maintainers more time to address more critical issues.
I will start listing some ideas that I had but I also want to hear from the other maintainers @jaagr, @NBonaparte and @skystrife and of course you, the people using polybar and reporting issues, what you think we can improve, so that people don't necessarily have to open an issue and can solve their problem on their own, as well as, if they do open an issue it can be processed by us more quickly.
Contributing Guidelines
Creating a Contributing.md file will prompt a user creating a new issue to review the contributing guidelines, for more info click here.
This file could contain instructions on how to structure an issue depending on what kind of issue it is (bug, build error, feature request, etc.). It could also tell the user what we expect they have already done before actually opening the issue. Some of which are

Having looked at the wiki
Using the latest stable release or the latest commit
Tried to solve it with a custom/script module
...

Issue Templates
This is very close to the point before. Here we could have the user confirm that they read the Contributing.md file and other things (like read the Wiki, etc.). See the Atom Issue Template as an example.
Preventing Question Issues
Many tickets are of the form "How can I do X?" and "Is it possible to do Y?", often such questions can be resolved by responding with a link to the wiki or a simple "This is not possible.". We want to keep those types of questions to a minimum by enabling the user to find the information themselves. Github allows us to create a Support.md file and it will also prompt the user to read that before opening an issue, as described here.
Adding an FAQ
This would be another way to decrease question style issues.
Closing Old Issues
If the original author stops responding to requests, the issue is often forgotten about and remains open. This happens frequently if the author does not tell you if a proposed solution to their issue worked. In the past, once we stumbled upon such an issue, we just closed it, if we felt enough time had passed. I think we should formalize, how and when we close such issues.
I propose the following rules:

If the author was able to reproduce the issue and fix it on their machine, they can close the issue once the PR with the fix is merged or immediately, if the fix doesn't require any code changes to the project.
Else close the issue after 30 days. Maybe consider using stalebot for this.

Use the invalid label
If an issue comes in where the author clearly has not read or followed the Contributing Guidelines, simply makr it invalid with a note telling them to try again.
What do you think about this so far?
I will draft some PRs to add contributing guidelines, the issue template and maybe even start the FAQ.

I agree with all of this.
Issue templates would be very helpful, as there have been multiple occasions where we have asked for more information.
We should definitely have automatic issue closing. I recall a while back there were a lot of old issues that were resolved which we had to close manually.

Great initiative. I also agree with all of this. I don't know how many times I've asked for more information regarding an issue such as config, version, etc. Having a bit more restrictive policy to issue reports (list of requirements for example) would solve part of it. I do however think that there's a fine line between useful restrictions and restrictions that make the reporter think twice about actually reporting - which wouldn't be good.
There might be other bots (a part from stalebot that, from what I can tell, only handles inactivity) that could help with commenting on missing details, auto closing, etc. Worth looking into.


Add option to end the stream when to option is reached

Adds an option to stop processing the stream when the number of lines
indicated by options.to is reached. This allows for only the requested
number of lines to be parsed, preventing parsing errors from later
lines.
Depending on the expectations this may fix #132

Thanks, this subject has been raised in the past and never yet coved. Do we really need a new option for this functionality? To me, it looks more like a bug than a missing feature. There two purpose to end processing when to is achieved:

prevent error in later CSV code, as you mentionned
speed up processing, potentially drastically if a lot of records remains to be parse

Also, could you please add a test which illustrate the first point. In current source code, the test shall fail since error will be present after to, and the test shall pass with your enhancements.
What's your opinion?

Sounds good. Would you prefer that I remove the option and make this the normal behaviour when to is defined?
Once we're in agreement I'll update the code and add the needed tests.

Sounds good. Would you prefer that I remove the option and make this the normal behaviour when to is defined?

This is my opinion. I ask you yours to make sure my arguments are valid, my understanding correct and that I'm not missing something. You can go ahead on my side.

I've removed the endStreamWithTo option as this feels more like a bug. Added a test where parsing the last line would result in an error, and have adjusted the parser to ignore additional calls once to has been reached.

What is the reason of moving down those four line after emitting records ?

Moving that block allows the current parsed record to be emitted, before checking if count >= to, otherwise I found that the tests would lose the last result.

Strange, the currently shouldn't pass then. Are you sure that you swith the condition >= back to > when moving the code above ?

I believe the tests are accurate.
In the original position, line 3 would get parsed when to: 2 so a parsing error could occur if line 3 was malformed.
In the new position, after parsing and emitting line 2 it will check if count >= 2 and then emit null to end the stream, set the isEnded which will prevent parsing of line 3+.

OK, thank you for your feedback

A patch version has been published.


Providing a custom seccomp profile

Hi, we're looking to run playwright in CI for end to end testing, and the docs mention we can run chrome without disabling sandbox by providing their custom seccomp profile to docker run: https://playwright.dev/#version=v1.3.0&path=docs%2Fdocker%2FREADME.md&q=run-the-image
I've verified that this works great locally.
However I can't seem to find a way supply the required args using docker-buildkite-plugin: --security-opt seccomp=seccomp_profile.json
Would you be willing to accept a PR to add this functionality?
In the mean time we can disable chrome sandboxing in CI, but that's not ideal as we would not be testing in the same environment as users.

Hey @lewisl9029, we support passing in args like (taken from the readme):
steps:
  - plugins:
      - docker#v3.5.0:
          image: "mesosphere/aws-cli"
          always-pull: true
          command: ["s3", "sync", "s3://my-bucket/dist/", "/app/dist"]
    artifact_paths: "dist/**"

Would this work for your usecase?

Hi @jayco apologies for the late follow up. --security-opt seccomp=seccomp_profile.json is an arg to docker itself, not to the program running in the container. See https://docs.docker.com/engine/security/seccomp/
It looks like the suggestion you posted is for passing args to the program running inside the container, correct?

This has been implemented in #171 and released in version v3.14.0 :)


add wikipedia links for teilnehmer

use vorname/nachname fileds from #2 to search for them automatically..
If wikipedia page is available in multiple languages, either choose the longest one (most often German, perhaps) or make it dependent on the language selection of the site.

fixed by #278


Uncaught TypeError: Cannot read property 'slice' of undefined

When I create object library cat this error.
My code:
`const EnjoyHint = require('../../../node_modules/enjoyhint/enjoyhint');
var EnjoyTest = /**@class*/ (function () {
function EnjoyTest() {
    var enjoyhint_instance = new EnjoyHint({});

    var enjoyhint_script_steps = [
        {
            'click .x-btn-inner' : 'Click the "New" button to start creating your project'
        },
        {
            'next .fa-eye' : 'Click the "New" button to start creating your project'
        }
    ];
    enjoyhint_instance.set(enjoyhint_script_steps);

    enjoyhint_instance.run();
}

return EnjoyTest;
}()

);`

Can you clarify the line of code triggering this exception? I can't image it would be arrays.prototypes.slice

This error is coming from inside the jquery.scrollTo plugin when it tries to scroll the page to the selectors you have provided.  Have you confirmed that those selectors can be found if you manually type $('.x-btn-inner') and $('.fa-eye')?
Also I'd recommend that if you're using node packages you use https://www.npmjs.com/package/enjoyhint.js not the other one which has not been updated in 2 years.

Yes this selectors really exist.
And I try used https://www.npmjs.com/package/enjoyhint.js, but problem not resolved.

Will you create a fiddle to reproduce this issue?

I created fiddle for you
I can't see any error on same script
https://jsfiddle.net/darron1217/pd40u9to/1/

@darron1217 Thank you. Really all work. I Will be search problem in my project when build project by webpack.


missing 3rd argument 'sendResponse' for onMessage event listener

Concerning the the typing of onMessage event handler, sendResponse function is missing as the 3rd argument.
Here is the MDN and Chrome documentation about this parameter.
BTW thanks for your good work 👍

duplicate

duplicate


PhysicsProcessor ParentScene may be initialized to null

Release Type: GitHub
Version: master
Platform(s): N/A
Describe the bug
I'm not currently sure why or if always but on the below line the parentScene is not set, which results in later inability to display debug colliders in game.
https://github.com/stride3d/stride/blob/74b2b4ad07d7811688e540259cedb8abb95ead9f/sources/engine/Stride.Physics/Engine/PhysicsProcessor.cs#L215
To Reproduce
Steps to reproduce the behavior:

Create a new project
Add a static collider to the ground
Add Debug physics shapes script
Launch game and press Ctrl+Shift+P

Expected behavior
Debug collider shapes are displayed.
Additional context
The issue can be easily mitigated by adding
SceneSystem.SceneInstance.GetProcessor<PhysicsProcessor>().ParentScene = Entity.Scene;

to the beginning of the debug script.

Hi! Is this still an issue? If so, I would like to take this one. @manio143 @Eideren

@kizilsum Sure, go ahead !
Just to clarify, the help wanted message removed above is because anyone can now freely contribute to the engine. You can open PRs for any issue you want, right now the engine doesn't have a lot of traffic so you can just do so without notifying us first if you don't want to :)

Hello, I'd like to try working on this issue and was wondering if anyone had some advice or ideas on where to start.  Thanks!

https://doc.stride3d.net/4.2/en/manual/engine/entity-component-system/index.html I assume this will be an issue within the Entity processor? I havent messed with Bullet too much but each of these physics shapes should be getting initialized in the EntityProcessor if it works like other systems.
I assume it should be around here where it gets set up.

Fixed through #2262


time.Format prunes trailing zero decimals from format

I'm not sure it's a bug, but it caught me by surprise:
time.Unix(0, 1).Format(time.RFC3339Nano) // 1970-01-01T00:00:00.000000001Z
time.Unix(0, 0).Format(time.RFC3339Nano) // 1970-01-01T00:00:00Z

I think if the format asks for 9 decimal places, it should show them. Logs look horrible otherwise.
With nanoseconds it's not that likely to happen, but with a custom format up to milli- or microseconds the same thing happens.

I just realized RFC3339Nano uses 9s for the second decimals, instead of 0s, and that I can use that in custom formats. I guess this is by design then.


Issue/executor log cleanup

Description
Final log cleanup, based on working with systemtenant
closes #7692
Self Check:
Strike through any lines that are not applicable (~~line~~) then check the box

[x] Attached issue to pull request
[x] Changelog entry
[x] Type annotations are present
[x] Code is clear and sufficiently documented
[x] No (preventable) type errors (check using make mypy or make mypy-diff)
[x] Sufficient test cases (reproduces the bug/tests the requested feature)
[x] Correct, in line with design
[ ] End user documentation is included or an issue is created for end-user documentation (add ref to issue here: )
[ ] If this PR fixes a race condition in the test suite, also push the fix to the relevant stable branche(s) (see test-fixes for more info)

Processing this pull request

Merged into branches master in ce4de1a7a82c21ee243b39f8e2b1a18dd77d0696