Documentation: Added extra param for Flink streaming

Minor change in the documentation to add the parameter "shutdownSourcesOnFinalWatermark", which is required to run NEXMark test with flink runner (streaming).
Reference: https://lists.apache.org/thread.html/re71e8298e0c13180a4ab0ac6a65e808e1d82ce85e955778cf1089553%40<user.beam.apache.org>
Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
Post-Commit Tests Status (on master branch)

Lang
SDK
Apex
Dataflow
Flink
Gearpump
Samza
Spark

Go

---
---

---
---

Java

Python

---

---
---

XLang
---
---
---

---
---

Pre-Commit Tests Status (on master branch)

---
Java
Python
Go
Website

Non-portable

Portable
---

---
---

See .test-infra/jenkins/README for trigger phrase, status and link of all Jenkins jobs.

@mxm I think you changed recently Flink to not need this anymore, can you PTAL.

Indeed, this parameter is not needed anymore and has been removed: https://jira.apache.org/jira/browse/BEAM-9900

@sruthiskumar Sorry for the timing but we have just removed the parameter and changed the default behavior to always shutdown, unless checkpointing is enabled.

Here is the commit which removed the flag: https://github.com/apache/beam/commit/d106f263e625e5d7c4f3848f16da301871f65142

Closing the PR as the change is not required anymore


Check if boundary condition falls in inactive idomain

Upon configuring flopy packages (several functions within nlmod/mfpackages) check if boundary condition is placed in inactive idomain to prevent errors such as placing drains/lakes in inactive cells.

When recalculating IDOMAIN take into account inactive zones in model.
The following two variables define where a model is active:

IDOMAIN
first_active_layer

flopy checks this
nlmod ignores data in inactive zones (this should be logged)
include check function nlmod.check_idomain(var) --> return list of inactive cellids ?


New Icons and Delivery Module

I created new Icons for the survival GUI and started working on a delivery module for the framework. I'm not sure if the delivery module I've started working on meets your needs or expectations. I've also started working on implementing more features to the survival GUI i.e hygiene, bladder, and energy.

I'll review the changes to the UI and see what I'm going to merge. I was working on some delivery module already, but your code will help a lot with that.
Not sure if hygiene, bladder, and energy will be included in the core, but if they can be made into modules, that would be perfect.

Sounds good, the only issue I'm having is deleting the unit that spawns in, in which you deliver the package to.

I'll download your repository as it is and update the icons when I update the HUD a little bit. I'll include it all in the next release. Hope you won't mind if I mention you in the contributors?

K, and I don't mind if you mention me.  And if you'd like to you can add me your Dev Team here on GitHub and on Trello.

I'm planning to keep the Trello board just to my personal development of modules and the core.
Switched RPFramework to its own organization and invited you to it.
With this, I'll close the pull request and implement the changes I can to the new update.


Server Eslint

This PR gets the server code to pass eslint, and it also requires eslint to pass in order to run npm run dev.

@dionjwa Check this out, I mainly just cleaned up a million little spaces and commas and stuff, but I did move a few things to separate files.  Also just to check, I'm ok to add eslint to the npm dev script here, right?  It's only run in development?

Why don't we create a git pre-commit hook? It's pretty straightforward to create them, and link them locally, and to me it makes a bit more sense to have the linting check on the commit step rather than the dev server, since you can run the server, make a lint-breaking change, them commit.

I think I'll move the lint/nodemon combo into another script that I can use if I want, since the frontend webpack-dev-server already runs eslint on each code change too.  I was playing just playing around with pre-commit hooks and this seemed to work for me in .git/hooks/pre-commit:
cd ./server/
exec npm run lint

@dionjwa Done.

Looks good!


Global update starts 1-2 more times after cancelling

Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open issue.
[X] I have written a short but informative title.
[X] If this is an issue with an extension, I should be opening an issue in the extensions repository.
[X] I have tried the troubleshooting guide.
[X] I have updated the app to version 1.7.0.
[X] I have updated all installed extensions.
[X] I will fill out all of the requested information in this form.

Tachiyomi version
Preview r402
Android version
Android 11
Device
Galaxy Tab S7 plus
Steps to reproduce
Global update starts. Cancel it.
Expected behavior
Global update cancels and notification disappears
Actual behavior
Another global update starts, sometimes even another one starts (but doesn't update) after cancelling the second time.
Crash logs
No response
Other details
No response

Its a main Tachi bug if its still happening


MQTT fixes

Remove subscribers iterator is broken. Needs a fix.

Fixed this issue in another pull request.


[Q] Need details on entry updated

Hi there,
I am using redisson and just imagined a possible behavior on scaled environments.
I have a Redisson Map like that:
public abstract class BaseAuthMapEntry implements MyDataSerializable {

    /**
     * The website for the session
     */
    private BaseWebsite website;

    ...

    public BaseWebsite getWebsite() {
        return website;
    }

    public void setWebsite(BaseWebsite website) {
        this.website = website;
    }

    @Override
    public ObjectNode toJson() {
        ObjectNode node = mapper.createObjectNode();
        node.put("website_id", website.getId().toString());

        return node;
    }

    @Override
    public void fromJson(ObjectNode json) {
        verifyString(json, "website_id");
        website = BaseWebsite.getById(json.get("website_id").asText());
    }
}

Where MyDataSerializable is my custom serializer for redisson.
Where BaseWebsite is a bean stored in database
Imagine I have two instances which have in memory this map entry.
Now one of them update the BaseWebsite's object and then do a "map.fastput(id, object);"
Since the website_id didn't change, when the other instance will do "map.get(id);", will the map entry be refetched and the website's object reloaded or not?
Thanks,

The map entry will be reloaded

Thanks for your answer!


Accordion has forced H4 in heading

The heading of panel in accordion (accordion-group.component.ts) automatically includes a H4 type heading, forcing everything inside to be a H4. Can you change this to a div and allow flexibility to change this.
If H4 is needed this can be provided by using the accordion-heading attribute.

Will be merged tomorrow


ArgGroup with exclusive=false and multiplicity=1 not behaving as expected

Here's a minimal example to illustrate the problem:
import java.util.concurrent.Callable;
import picocli.CommandLine;
import picocli.CommandLine.ArgGroup;
import picocli.CommandLine.Command;
import picocli.CommandLine.Option;

@Command(name = "multiplicity")
class Multiplicity implements Callable<Integer> {

  @ArgGroup(exclusive = false, multiplicity = "1")
  private RequiredGroup requiredGroup;

  static class RequiredGroup {
    @Option(names = "--option_one")
    private String optionOne;

    @Option(names = "--option_two")
    private String optionTwo;
  }

  @Override
  public Integer call() throws Exception {
    return 0;
  }

  public static void main(String... args) {
    int exitCode = new CommandLine(new Multiplicity()).execute(args);
    System.exit(exitCode);
  }
}

With this example, I would expect that the command without any options (i.e. multiplicity), would throw a MissingParameterException. However, in the current version (4.7.4) the command executes and the call() method is executed. Note, that the requiredGroup object is null.
On a much older version of picocli (4.1.4), running this command works as expected. It throws a MissingParameterException with the text Error: Missing required argument(s):.
I am not sure exactly where the regression happened or if this is even a regression? From my reading of the docs, it does seem like one. In the ArgGroup documentation:

define a set of arguments that must co-occur. Set exclusive = false to define a group of options and positional parameters that must always be specified together. Picocli will throw a MissingParameterException if not all the options and positional parameters in a co-occurring group are specified together.

Groups may be optional (multiplicity = "0..1"), required (multiplicity = "1")

When the parent group is required, at least one subgroup must be present.

Maybe this could be added to the test cases to prevent the regression, also to test it on previous versions.

None of the options in the RequiredGroup group is required, so even if the group itself is required (because it has multiplicity = "1"), the validation logic will only raise an alarm if any of the required options/parameters in that group is missing.
Should this be changed? I am not sure... The current logic does make logical sense to me, it allows groups that have one optional and one required option to work as expected.
For your use case, I would recommend making all options in the group required.
I am not sure when the validation changed; I don't remember intentionally changing it but I could have forgotten... 😅

The use case I am trying to solve for is when at least one of the options is required. I.e. a user should be able to say multiplicity --option_one or multiplicity --option_two or multiplicity --option_one --option_two, but should not be able to say multiplicity.
If I set all the options to required, then the user would necessarily have to specify both options right? Or am I misunderstanding how required works?
A colleague also helped track down this past issue: https://github.com/remkop/picocli/issues/947 which addresses our exact use case. The approach suggested there is exactly the one I have outlined in the minimal code above.
I can work around this problem fairly easily by adding custom validation to throw a ParameterException if requiredGroup==null. However, I wanted to confirm whether this is a regression first.

Ouch! Thanks for that link. Now it does look like a regression! 😅
I'll be traveling the next 2 weeks but I'll look at it when I get back.
Thank you for raising this! 🙏

No problem! Thanks in advance for looking into it (and creating picocli)! We love picocli in our team :)

@remkop really sorry to re-ping you here, but I was wondering if you already had / already still planning to have a look at this one?

Thanks for the reminder! I'll try to take a look this weekend.

Looks like the regression was introduced by this commit: https://github.com/remkop/picocli/commit/4b41a217982f98fce15b95b17e06ae147d676662 (PR #2030).
I will look into a fix that addresses both this ticket and #1848.

@remkop thank you! Were you hoping to find the time for fixing this yourself?
Or do you need or would welcome contributions, if anyone (not me...) was motivated to?

I have been poking at this problem this week. I have a potential solution which probably requires a fair bit of polish to actually work. But basically, I would remove the method added in [4b41a21] and instead add logic in the ArgGroupSpec constructor to recalculate the multiplicity based on whether any default values were provided.
Example:
int defaultedCount = 0;
for (ArgSpec arg : args) {
    if (arg.calcDefaultValue(true) != null) {
        defaultedCount++;
    }
}
multiplicity = multiplicity.min(multiplicity.max() - defaultedCount);

I.e. if you have an ArgGroup with multiplicity=1 and an Option within it that has a default value, the multiplicity will become 0..1. If you have multiplicity=3 and 2 options are defaulted, the multiplicity becomes 1..3. This calculation probably needs some more thought to cover all the edge cases.
@remkop What do you think? Am I breaking some implicit assumption within the codebase by having the "actual" multiplicity be different to the "defined" multiplicity?

@utmittal Thank you for helping think this through.
I agree that [4b41a21] needs to be rolled back, or refined. Still working on a solution...

Should be fixed in main now. Thank you again for raising this!


Unable to use logstash output rabbitmq plugin

Version: logstash 6.7
Operating System: Linux
Config File (if you have sensitive info, please remove it):
Sample Data:
Steps to Reproduce:

output {
rabbitmq {
id => "file.atlas.metrics.all"
host => "host.url.com"
port => 3030
exchange => "mq.rabbit.exchange.metrics"
exchange_type => "direct"
vhost => "vhost.rmq"
user => "rmq.user"
password => "password"
key => "metrics"
durable => true
automatic_recovery => true
heartbeat => 0
codec => line {
format => "
<domain_stats>
<as_of_ts>%{[sample][output_time]}</as_of_ts>
<metric_name>%{[metric][metric]}</metric_name>
<client_code>-</client_code>
<client_code_type>pricingNumber</client_code_type>
<metric_value>%{[metric][value]}</metric_value>
<metric_host>%{[sample][contents][metrics][host]}</metric_host>
</domain_stats>"
}
}
}
getting the following error:
[2019-04-08T13:46:38,471][ERROR][logstash.outputs.rabbitmq] RabbitMQ connection error, will retry. {:error_message=>"Connection to localhost:5672 refused", :exception=>"MarchHare::ConnectionRefused"}
[2019-04-08T13:46:39,490][ERROR][logstash.outputs.rabbitmq] RabbitMQ connection error, will retry. {:error_message=>"Connection to localhost:5672 refused", :exception=>"MarchHare::ConnectionRefused"}
[2019-04-08T13:46:40,501][ERROR][logstash.outputs.rabbitmq] RabbitMQ connection error, will retry. {:error_message=>"Connection to localhost:5672 refused", :exception=>"MarchHare::ConnectionRefused"}
why it's trying to connect to local host. what i am doing wrong?

host => "host-url:port", i don't know why the documentation is not updated. this is frustrating.. i spend entire day to find this problem.


Impl AsyncFileWriter by object_store

Is your feature request related to a problem or challenge? Please describe what you are trying to do.
Follow up of https://github.com/apache/arrow-rs/issues/5738, add AsyncFileWriter support for object_store.
Describe the solution you'd like
I plan to add a ParquetObjectWriter like we do for ParquetObjectReader
Describe alternatives you've considered
None
Additional context
I'm willing to help implementing this.

Was this closed in https://github.com/apache/arrow-rs/pull/5753 ?

Was this closed in #5753 ?

https://github.com/apache/arrow-rs/pull/5753 adds the trait for parquet, and this issue will impl trait for object_store.

How are you getting on with this, is there any way I can assist?

Hi @tustvold, I've reported a block issue here: https://github.com/apache/arrow-rs/issues/5834.


An error occurred during deployment

I have tried to deploy the current Azure template and get an error.
{
  "error": {
    "code": "InvalidTemplate",
    "message": "Die Vorlagensprachausdrücke für Ressource \"/subscriptions/a95a59fe-c7c7-4792-93a9-1c2c630899ce/resourceGroups/NewNavExperience/providers/Microsoft.Compute/virtualMachines/navexperience/extensions/vmextension1\" in Zeile 1 und Spalte 17845 konnten nicht verarbeitet werden. \"Die Sprachausdruckseigenschaft \"templateLink\" ist nicht vorhanden, verfügbare Eigenschaften sind \"template, parameters, mode, debugSetting, provisioningState\".\""
  }
}

Not sure what went wrong here, but typically the problem is, that the VM Name needs to be globally unique and cannot contain special characters nor be more than 15 characters long.
I typically use my initials followed by dev and some number - that typically works.

I think this is not the problem. The error says the TemplateLink property does not exist in the deployment script.
Typ: Microsoft.Compute/virtualMachines/extensions
Reesource: navexperience/vmextension1
VMName: navexperience

A VM Name of navexperience is probably not globally unique - try navexp followed by some number or your initials followed by dev and some number


Conversion with convert results in strange output

While building tests for https://github.com/paazmaya/grunt-sass2scss, came across strange behaviour.
The given Sass file:
/**
 * This is third file
*/
@import 'styles-2'

// Imports include the other file

label
  // Use the variable from the other file
  color: $color-white

with sass2scss -c < input.sass > output.scss, the resulted SCSS file:
/*** This is third file */*/ {}@import 'styles-2';/* Imports include the other file */label { /* Use the variable from the other file */color: $color-white; }

You can actually shorten this to:
/**
*/

Have you tried compiling this with ruby sass? Because it does not! Somehow ruby sass expects the multiline comment closer to be indented. And my tests indicate that sass2scss mirrors that behavior pretty much perfectly. Unfortunately the sass specs are not very clear on such issues ...

Just retried on sassmeister and the given code does still not compile.


Doc - Navbar toggle button not visible in mobile view

Although not visible, it is interactive.

should be

cc @hakenr @crdo

There were some dances around CSS variables, maybe check there. (Found 1 mistake yesterday.)


Seeing a segfault when running mapper/examples/gaussian.json

Looks like it is segfaulting somehow on creating consts.

Fixed by #473


rocket chat will not load

My Setup

Operating System:
App Version:
Installation type:

[ ] I have tested with the latest version
[ ] I can simulate the issue easily

Description

Current Behavior

Expected Behavior

rocketcht willnot load

Meaning what exactly?

Maybe he is expiring the same issue as I do after applying the latest updates on server side. Right after I have updated RC on server side, all connected (Windows Desktop) clients become dysfunctional.
All I can see than on client side is a grey screen with the "3 dot animation". The fix is easy, just do a "clear cache" followed by a "reload server". But as not all users understand how to get it fixed, I consider this as a big Issue ...

Maybe he is expiring the same issue as I do after applying the latest updates on server side. Right after I have updated RC on server side, all connected (Windows Desktop) clients become dysfunctional.
All I can see than on client side is a grey screen with the "3 dot animation". The fix is easy, just do a "clear cache" followed by a "reload server". But as not all users understand how to get it fixed, I consider this as a big Issue ...
I have observed this behaviour for the first time after I have updated RC to version 6.4.1. Mobile clients seems not to be affected by this.

Same problem.

Maybe he is expiring the same issue as I do after applying the latest updates on server side. Right after I have updated RC on server side, all connected (Windows Desktop) clients become dysfunctional.

This problem is not limited to the Electron client: RocketChat/Rocket.Chat#30717

ive got on rc seekingalpha chat
On Mon, Nov 6, 2023 at 12:35 AM Paul Staroch @.***>
wrote:

Maybe he is expiring the same issue as I do after applying the latest
updates on server side. Right after I have updated RC on server side, all
connected (Windows Desktop) clients become dysfunctional.
This problem is not limited to the Electron client:
RocketChat/Rocket.Chat#30717
https://github.com/RocketChat/Rocket.Chat/issues/30717
—
Reply to this email directly, view it on GitHub
https://github.com/RocketChat/Rocket.Chat.Electron/issues/2758#issuecomment-1793883099,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BAG72TQMIDRCHP43VOSB7PLYDAPFXAVCNFSM6AAAAAA6RH4SWCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOJTHA4DGMBZHE
.
You are receiving this because you authored the thread.Message ID:
@.***>

Maybe he is expiring the same issue as I do after applying the latest updates on server side. Right after I have updated RC on server side, all connected (Windows Desktop) clients become dysfunctional.
All I can see than on client side is a grey screen with the "3 dot animation". The fix is easy, just do a "clear cache" followed by a "reload server". But as not all users understand how to get it fixed, I consider this as a big Issue ...
I have observed this behaviour for the first time after I have updated RC to version 6.4.1. Mobile clients seems not to be affected by this.

Our team is experiencing the same issue as well.
Operating System: Mix of Windows (10/11) and Ubuntu (22.04) machines.
App Version: 6.4.5
Installation type: Snap install (Ubuntu 22.04)

This "constant refresh" affects both the Desktop Application as well as the Webapp (Firefox/Brave).
The "clear cache and server refresh" does fix the issue temporarily but the problem comes back.

This is issue is not specific to the electron app, it is a core issue, so let's move the conversation to https://github.com/RocketChat/Rocket.Chat/issues/30990
Closing this for now, thanks!


Fix bug in dm.utils.fs

Checklist:

[ ] Add tests to cover the fixed bug(s) or the new introduced feature(s) (if appropriate).
[ ] Update the API documentation is a new function is added or an existing one is deleted.
[x] Added a news entry.

copy news/TEMPLATE.rst to news/my-feature-or-branch.rst) and edit it.

Just got this bug manipulating s3 paths. Can I merge here?

yep, sorry, haven't paid attention.


Feature update t4 i82

for masking, I also constrained it over time and extract region. Making cube small.
add some comments to explain what np.where does. and also about np.ma.sum


feat(parser): the 1st version of a new combinator style parser

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
Introducing a new parser with winnow.
This is the first version of the new parser, with only the datatype parsing. However, it's also the most complicated part of our Parser. Due to our wrong design previously, we use < and > to define struct type, then introduce a necessary state angle_number during parsing. Fortunately, the new framework handle it perfectly.
winnow-rs is a parser-combinator crate, and it's forked by me for a simple patch https://github.com/winnow-rs/winnow/pull/525.
The new framework can progressively replace the original parser, see Parser::parse_v2 for detail.

Checklist

[ ] I have written necessary rustdoc comments
[ ] I have added necessary unit tests and integration tests
[ ] I have added test labels as necessary. See details.
[ ] I have added fuzzing tests or opened an issue to track them. (Optional, recommended for new SQL features #7934).
[ ] My PR contains breaking changes. (If it deprecates some features, please create a tracking issue to remove them in the future).
[ ] All checks passed in ./risedev check (or alias, ./risedev c)
[ ] My PR changes performance-critical code. (Please run macro/micro-benchmarks and show the results.)

[ ] My PR contains critical fixes that are necessary to be merged into the latest release. (Please check out the details)

Documentation

[ ] My PR needs documentation updates. (Please use the Release note section below to summarize the impact on users)

Release note
If this PR includes changes that directly affect users or other significant modifications relevant to the community, kindly draft a release note to provide a concise summary of these changes. Please prioritize highlighting the impact these changes will have on users.

Can we have a better error display with this new parser framework? 🤔

taken from #15091

Can we have a better error display with this new parser framework? if it provides span in some way. 🤔

It's possible, but must after we finish the whole refactor.

The combinator style makes the code more readable and maintainable

Why do you think the combinator is more maintainable? I think now everyone can work with the parser without learning, but not afterwards.

Why do you think the combinator is more maintainable? I think now everyone can work with the parser without learning, but not afterwards.

Let's use Golang to rewrite RisingWave 😄


Machine translate 2nd hands-on material from English to Thai

Machine translate 2nd hands-on material from English to Thai
ref #5
Acceptance criteria:
After translation, the following must be confirmed

[ ] Review the translation for obvious errors or duplicate content.
[ ] Reverse machine translation is applied to check for obvious errors.

Don't forget to add it to the README table of contents and VitePress config.

Add TIPS about File based routing (I forgot to add )


Unable to run program using PowerShell in VSCode CMakeTool plugin 1.14.29/1.14.30

Brief Issue Summary
1.14.29/1.14.30 Cannot use powershell run  Qt 5.6.3 for Desktop (MinGW 4.9.2 32 bit) program
I am using the VSCode CMakeTool plugin version 1.14.29/1.14.30 to build and run my project. However, I am unable to run the program using PowerShell in the plugin's integrated terminal. When I try to run the program, it does not start and I do not receive any error messages.
I have confirmed that the program runs successfully when launched from File Explorer or when running PowerShell outside of VSCode. Therefore, I believe that the issue is specific to the CMakeTool plugin.
I have tried updating the plugin to version 1.14.30, but the issue still persists.
but Versions before 1.14.29 do not have this issue. The issue with the previous version disappeared before returning
Could you please investigate this issue and provide a solution or workaround? Let me know if you need any additional information or if there are any troubleshooting steps I can try.
Thank you.
CMake Tools Diagnostics
No response
Debug Log
No response
Additional Information
No response

I also encountered the same problem, so when I rolled back to 1.13.45, it worked properly

This issue sounds similar to #3123, which has a fix in progress (#3132). In the meantime before we get the fix out, can you try the workaround mentioned in that issue and let me know if it works for you? Specifically, the workaround is to make sure that targets is specified for all cmake tasks in tasks.json.

The fix from #3132 should be in pre-release version 1.15.1, which is now available on the marketplace. Please let me know if you're still seeing issues with this version.

A potential fix for this issue from #3138 is now publicly available in pre-release channel extension version 1.15.3. Let us know if you're still seeing this issue in the pre-release version.

This fix is included in version 1.14.31 that was just released to the marketplace.


add extra parameter "extra_systemd_config"

This add an extra parameter to make it possible to add lines to /etc/systemd/system/portgresql-.service.
This is needed for instance to provide for extra open files for postgres.
LimitNOFILE=10000
This is implemented as a multi line string. This way you can add multiple lines to the systemd file.

Hi @veninga , thank you for your contribution !


I/tflite: Created TensorFlow Lite delegate for GPU.
I/BufferQueueProducer: SurfaceTexture-0-23554-0 queueBuffer: slot 2 is dropped, handle=0x7463442200
I/BufferQueueProducer: SurfaceTexture-0-23554-0 queueBuffer: slot 3 is dropped, handle=0x74634422c0
I/BufferQueueProducer: SurfaceTexture-0-23554-0 queueBuffer: slot 4 is dropped, handle=0x7463441cc0
I/BufferQueueProducer: SurfaceTexture-0-23554-0 queueBuffer: slot 5 is dropped, handle=0x7463443640
I/BufferQueueProducer: SurfaceTexture-0-23554-0 queueBuffer: slot 0 is dropped, handle=0x7463443040
I/BufferQueueProducer: SurfaceTexture-0-23554-0 queueBuffer: slot 6 is dropped, handle=0x7463443c40
I/BufferQueueProducer: SurfaceTexture-0-23554-0 queueBuffer: slot 2 is dropped, handle=0x7463442200
I/BufferQueueProducer: SurfaceTexture-0-23554-0 queueBuffer: slot 3 is dropped, handle=0x74634422c0
I/BufferQueueProducer: SurfaceTexture-0-23554-0 queueBuffer: slot 4 is dropped, handle=0x7463441cc0
I/BufferQueueProducer: SurfaceTexture-0-23554-0 queueBuffer: slot 5 is dropped, handle=0x7463443640
I/BufferQueueProducer: SurfaceTexture-0-23554-0 queueBuffer: slot 0 is dropped, handle=0x7463443040
I/BufferQueueProducer: SurfaceTexture-0-23554-0 queueBuffer: slot 6 is dropped, handle=0x7463443c40
I/BufferQueueProducer: SurfaceTexture-0-23554-0 queueBuffer: slot 2 is dropped, handle=0x7463442200
I/BufferQueueProducer: SurfaceTexture-0-23554-0 queueBuffer: slot 3 is dropped, handle=0x74634422c0
I/BufferQueueProducer: SurfaceTexture-0-23554-0 queueBuffer: slot 4 is dropped, handle=0x7463441cc0
I/BufferQueueProducer: SurfaceTexture-0-23554-0 queueBuffer: slot 5 is dropped, handle=0x7463443640
I/BufferQueueProducer: SurfaceTexture-0-23554-0 queueBuffer: slot 0 is dropped, handle=0x7463443040
I/BufferQueueProducer: SurfaceTexture-0-23554-0 queueBuffer: slot 6 is dropped, handle=0x7463443c40
I/BufferQueueProducer: SurfaceTexture-0-23554-0 queueBuffer: slot 2 is dropped, handle=0x7463442200
I/BufferQueueProducer: SurfaceTexture-0-23554-0 queueBuffer: slot 3 is dropped, handle=0x74634422c0
I/BufferQueueProducer: SurfaceTexture-0-23554-0 queueBuffer: slot 4 is dropped, handle=0x7463441cc0
I/BufferQueueProducer: SurfaceTexture-0-23554-0 queueBuffer: slot 5 is dropped, handle=0x7463443640
I/BufferQueueProducer: SurfaceTexture-0-23554-0 queueBuffer: slot 0 is dropped, handle=0x7463443040
I/BufferQueueProducer: SurfaceTexture-0-23554-0 queueBuffer: slot 6 is dropped, handle=0x7463443c40
I/BufferQueueProducer: SurfaceTexture-0-23554-0 queueBuffer: slot 2 is dropped, handle=0x7463442200
I/BufferQueueProducer: SurfaceTexture-0-23554-0 queueBuffer: slot 3 is dropped, handle=0x74634422c0
I/BufferQueueProducer: SurfaceTexture-0-23554-0 queueBuffer: slot 4 is dropped, handle=0x7463441cc0
I/BufferQueueProducer: SurfaceTexture-0-23554-0 queueBuffer: slot 5 is dropped, handle=0x7463443640
I/BufferQueueProducer: SurfaceTexture-0-23554-0 queueBuffer: slot 0 is dropped, handle=0x7463443040
I/BufferQueueProducer: SurfaceTexture-0-23554-0 queueBuffer: slot 6 is dropped, handle=0x7463443c40
I/BufferQueueProducer: SurfaceTexture-0-23554-0 queueBuffer: slot 2 is dropped, handle=0x7463442200
I/BufferQueueProducer: SurfaceTexture-0-23554-0 queueBuffer: slot 3 is dropped, handle=0x74634422c0
I/BufferQueueProducer: SurfaceTexture-0-23554-0 queueBuffer: slot 4 is dropped, handle=0x7463441cc0
I/BufferQueueProducer: SurfaceTexture-0-23554-0 queueBuffer: slot 5 is dropped, handle=0x7463443640
I/BufferQueueProducer: SurfaceTexture-0-23554-0 queueBuffer: slot 0 is dropped, handle=0x7463443040
I/BufferQueueProducer: SurfaceTexture-0-23554-0 queueBuffer: slot 6 is dropped, handle=0x7463443c40
I/BufferQueueProducer: SurfaceTexture-0-23554-0 queueBuffer: slot 2 is dropped, handle=0x7463442200
I/BufferQueueProducer: SurfaceTexture-0-23554-0 queueBuffer: slot 3 is dropped, handle=0x74634422c0
I/BufferQueueProducer: SurfaceTexture-0-23554-0 queueBuffer: slot 4 is dropped, handle=0x7463441cc0
I/BufferQueueProducer: SurfaceTexture-0-23554-0 queueBuffer: slot 5 is dropped, handle=0x7463443640
I/BufferQueueProducer: SurfaceTexture-0-23554-0 queueBuffer: slot 0 is dropped, handle=0x7463443040
I/BufferQueueProducer: SurfaceTexture-0-23554-0 queueBuffer: slot 6 is dropped, handle=0x7463443c40
I/BufferQueueProducer: SurfaceTexture-0-23554-0 queueBuffer: slot 2 is dropped, handle=0x7463442200
I/BufferQueueProducer: SurfaceTexture-0-23554-0 queueBuffer: slot 3 is dropped, handle=0x74634422c0
I/BufferQueueProducer: SurfaceTexture-0-23554-0 queueBuffer: slot 4 is dropped, handle=0x7463441cc0
I/BufferQueueProducer: SurfaceTexture-0-23554-0 queueBuffer: slot 5 is dropped, handle=0x7463443640
I/BufferQueueProducer: SurfaceTexture-0-23554-0 queueBuffer: slot 0 is dropped, handle=0x7463443040
I/BufferQueueProducer: SurfaceTexture-0-23554-0 queueBuffer: slot 6 is dropped, handle=0x7463443c40
I/BufferQueueProducer: SurfaceTexture-0-23554-0 queueBuffer: slot 2 is dropped, handle=0x7463442200
I/BufferQueueProducer: SurfaceTexture-0-23554-0 queueBuffer: slot 3 is dropped, handle=0x74634422c0
I/BufferQueueProducer: SurfaceTexture-0-23554-0 queueBuffer: slot 4 is dropped, handle=0x7463441cc0
I/BufferQueueProducer: SurfaceTexture-0-23554-0 queueBuffer: slot 5 is dropped, handle=0x7463443640
I/BufferQueueProducer: SurfaceTexture-0-23554-0 queueBuffer: slot 0 is dropped, handle=0x7463443040
Thanks in advance:))

I'm not sure what the question is..
Does the app run at all?
Are you only having an issue with hair segmentation app?
Please describe more of what you are seeing.
Also, these BufferQueueProducer logs are from the android OS...
Can you provide more logs that include messages from MediaPipe and the app?
Are you running the most recent version of MediaPipe?
Thanks

You may run setup_android.sh to install Android SDK and NDK.
android_ndk_repository(
name = "androidndk",
api_level = 21,
)
android_sdk_repository(
name = "androidsdk",
)
i am using AAR file for hair segmentation app
it run perfectly almost in all devices like redmi note 5 ,realme 5 pro, oppp a57 but when it comes to oppo a1k it takes too much time to load camera not only hair app bt all other apps like facedetection, hand detection etc..
after, camera load successfully, app is not working properly means it's not applying color in hair
i am using princep repository

the princep repository may not be the most updated one, please try the official mediapipe repo.
can you provide more of the logcat output?

We are closing this issue for now due to lack of activity.

Facing same issue, When click image by camera it start buffing and it shows continually this line in log call
BufferQueueProducer(18387): SurfaceTexture-0-18387-1 queueBuffer: slot 0 is dropped, handle=0x7499b072c0


net::ERR_INVALID_URL when intercepting http

I have a simple app that intercepts the http protocol, and depending on the parsed URL, either modifies the request, or leaves it as it was. In the test app below, I don't even do anything in the handler, just return null:
var app = require('app'),
    BrowserWindow = require('browser-window');

var mainWindow = null;

app.on('window-all-closed', app.quit);

app.on('ready', function() {
  mainWindow = new BrowserWindow({
    width: 800,
    height: 600
  });

  mainWindow.openDevTools({detach: true});

  var protocol = require('protocol');
  protocol.interceptProtocol('http', function(request) {
    return null;
  }, function(error, scheme) {
    protocol.registerStandardSchemes(['http']);
    mainWindow.loadUrl('https://www.google.com');
  });
});

When the program loads, it works fine because it is loading the https URL for google.  However, when I type document.location = 'http://www.google.com' in the console, I get a net::ERR_INVALID_URL error and the page becomes blank.  I would expect the URL to load normally because I returned null as stated in the docs.
I am running v0.30.0 on Ubuntu 14.04.

It seems the problem occurs when there is a redirect.  If I navigate to http://www.example.com, there is no problem.


unclear/wrong error message?

messages.rflx.txt
test.rflx.txt
On the attached file, RFLX issues this error message:
test.rflx:26:46: parser: error: Expected 'for', got 'First'

The file contains neither "for" nor "First", so what is this about? I probably did something wrong, what's the proper syntax for such sequence comprehensions?

Thanks for the quick answer!

I'm reopening the issue because your code is accepted by Recordflux, but:

It triggers the assertion at generator/session.py, line 2495 (because the expression is a Selected, not a Variable; this is only visible when running with assertions obviously, e.g. via the testsuite)
The generated code doesn't compile:

rflx-test-session.adb:63:45: error: "In_Msg" is undefined (more references follow)
rflx-test-session.adb:81:10: error: "Seq" is not visible (more references follow)
rflx-test-session.adb:81:10: error: non-visible declaration at rflx-messages-seq.ads:18

Any idea how the code should be written to be accepted?

can be solved by using variables instead of the `Selected´ expression directly:

      state Copy is
         Seq : Messages::Seq;
         C : Messages::Integer;
         D : Messages::Integer;
      begin
         C := In_Msg.C;
         D := In_Msg.D;
         Seq'Append (C);
         Seq'Append (D);
         Out_Msg := Messages::Msg_Seq'(A => Seq);

We could make the unsupported expression clearer by calling _unsupported_expression, if no variable is used.

points to an actual bug:

rflx-test-session.adb:103:10: error: "Seq" is not visible (more references follow)
rflx-test-session.adb:103:10: error: non-visible declaration at rflx-messages-seq.ads:10

103          Seq'Size <= 32768
104          and then Seq'Size mod RFLX_Types.Byte'Size = 0

Seq'Size should be substituted by Messages.Seq.Size (Seq_Ctx). The fix is probably quite simple: Handling that specific case in _substitution in rflx/generator/session.py.
Do you think you can fix that?

Sure, I can try.


Provide a prepared astro skeleton starter project

Describe what feature you'd like. Pseudo-code, mockups, or screenshots of similar solutions are encouraged!
Similar to the npm create skeleton-app@latest project it would be nice to offer a base project instead of letting the user work through the guides which is always error prone.
This way the user can start right away instead of setting up the basic configurations, which is a way better starting experience.
For the moment we could promote something based on my little https://github.com/delight/astro-base-skeleton project.
On the longer run I could look into building a astro template. Astro templates are fine, but doesn't offer much of a higher value over a base project + also can go invisible quickly. But I think it still would be worth it creating one for skeleton.
What type of pull request would this be?
None
Any links to similar examples or other references we should review?
No response

@delight thanks for the suggestion, however we will be going in a different direction. Our long term goal is to update the new Skeleton CLI to generate a completely up to date project on the fly. While we currently only support SvelteKit, Astro and Vite support are planned.
You can view directions for this as part of our soft launch announcement in the latest release notes:
https://github.com/skeletonlabs/skeleton/discussions/605
And we have plans to make this our default for onboarding for SvelteKit projects in the next release:
https://github.com/skeletonlabs/skeleton/issues/628
If you're interested in contributing to an Astro version, please reach out to @niktek here or on Discord. He's new to Astro so support would be welcome! You can view the project here:
https://github.com/skeletonlabs/create-skeleton-app


A quick DFTB calculator

The current ase-dftb calculator suffers from high I/O load.
We need a simple dftb calculator that does not save the following files at each relaxation step.

band.out
detailed.out
charges.bin

Being fixed in https://github.com/qzhu2017/PyXtal/commit/8516cfa0f9f7b1c584b2631040ddabf8821dec84


pass form data back on fail in register

Most likely the user is going back to "forgot password". But, we want to cover the possibility that she entered the incorrect email address

completed with PR #158


feat(builder): bypass nodes

I've found out much to my chagrin that ComfyUI is in fact not turing complete, and the bypass functionality is only on the ComfyUI frontend, and is in fact not programmable.
This necessitates handling turning on and off nodes before they are sent to the API. I've added a bypass() and reinstate() method to the prompt builder, enabling users to conditionally bypass nodes.
This change necessitated some changes to types, most significantly the type of prompt has been specified to extend NodeData. It also requires querying the server for node definitions in order to determine the types of outputs and inputs of a node before it can be bypassed.
I've also modified prompt builder to clone the workflow every time prompt builder is cloned, as previously the original workflow was modified by input() and would have been further modified by the bypass functionality, which led to some strange mutation issues when two jobs of the same workflow were sent to the server. I've used structuredClone() for this, which should be covered by the lib settings already present in tsconfig.json.
NodeDef has also been extended to include optional properties, as these are necessary for bypassing. A new error class called MissingNodeError was added, in case bypassed nodes are not present in the workflow.

There was a bug in call wrapper that in some circumstances, if an output node was cached, and another one was not (setting a different seed in FaceDetailer but keeping the old seed in the base generation), the totalOutput count would not be zero at executedEnd, as it would fire before those output events would fire. I've added a getHistory() check in executedEnd to see if an output is really missing.

LGTM

@btaens this cause PromptBuilder not working correctly, i will re-check at this


GoDesk quickly builds your desktop application

I really enjoy building my own applications using Wails. This is the basic template I developed, which allows developers to quickly build desktop applications based on Wails. Project address: https://gitee.com/xbuntu/go-desk

Your address is wrong.
correct address
https://github.com/xbuntu/godesk

Please open a PR for adding it to https://wails.io/docs/next/community/templates


generate_observer is broken

$ php pestle.phar generate_observer
Full Module Name? (Pulsestorm_Generate)]
Event Name? (controller_action_predispatch)]
Observer Name? (pulsestorm_generate_listener)]
Class Name? (Pulsestorm\Generate\Model\Observer)]
PHP Fatal error:  Call to undefined function Pulsestorm\Magento2\Cli\Library\addSchemaToXmlString() in phar:///home/dio5/www/site/mage2local/pestle.phar/modules/pulsestorm/magento2/cli/library/module.php on line 92
PHP Stack trace:
PHP   1. {main}() /home/dio5/www/site/mage2local/pestle.phar:0
PHP   2. main() /home/dio5/www/site/mage2local/pestle.phar:9
PHP   3. Pulsestorm\Pestle\Runner\main() /home/dio5/www/site/mage2local/pestle.phar:7
PHP   4. ReflectionFunction->invokeArgs() phar:///home/dio5/www/site/mage2local/pestle.phar/modules/pulsestorm/pestle/runner/module.php:333
PHP   5. Pulsestorm\Magento2\Cli\Generate\Observer\pestle_cli() phar:///home/dio5/www/site/mage2local/pestle.phar/modules/pulsestorm/pestle/runner/module.php:333
PHP   6. initilizeModuleConfig() phar:///home/dio5/www/site/mage2local/pestle.phar/modules/pulsestorm/magento2/cli/generate/observer/module.php:35
PHP   7. ReflectionFunction->invokeArgs() /tmp/pestle_cache/2d27f575ea4837c003f8467c72c32131/reflection-strategy/baf3662faf17339378692b6967fc8c4d.php:7
PHP   8. Pulsestorm\Magento2\Cli\Library\initilizeModuleConfig() /tmp/pestle_cache/2d27f575ea4837c003f8467c72c32131/reflection-strategy/baf3662faf17339378692b6967fc8c4d.php:7

@dio5 Hmmm, I"m not seeing this one either.   Could you do two things to help me debug this remotely?

Could you paste the contents of md5 /path/to/your/pestle.phar here?
Remove the pestle namespace/module cache folder at /tmp/pestle_cache (should happen automatically, but if there's a bug there Bad Things can happen) and let me know if the problems persist
If the above doesn't help -- try re-downloading the phar? curl -LO http://pestle.pulsestorm.net/pestle.phar

Re: #3 -- I just downloaded the latest version of the phar and didn't see the same issues you're seeing.  A re-download would be a nice sanity check that could fix the problem, or will let us move on to the next stage of debugging.

Sorry, this is fixed as well. Apparently the second time I upgraded I got the link from the wiki
curl -LO http://alanstorm.com/2016/projects/pestle/pestle.phar
which breaks it.
While this one works:
curl -LO http://pestle.pulsestorm.net/pestle.phar

Wiki's updated with the right link.  I was originally distributing via my personal site before I decided the phar needed a real home.


send /update requests with outageIntervalUpdate=true on perfect within outage + outageBuffer interval in scheduler

This will prevent update retries from being incorrectly perfectly counted as that counter will only increase for missed retries from the perfectly scheduled script.
A global variable for updateFast should be set and then another script should run at that rate.
Both update scripts should execute commands.

The chart level result is to not have a high rate of update retries while updateFast was set to true on a former response.

Or chart when a host is being viewed and has updateFast enabled for all hosts, proving why there are many update retries at a certain time.

This makes sense with connection failures, what update retries was renamed to.


great idea

Sounds good to me

Please don't do that. You just created THREE meaningless issues.

Sorry man I'm new to this figured I was in tutorial mode still and had to
comment to get the git command
On Mar 9, 2015 11:23 PM, "Inndy"<EMAIL_ADDRESS>wrote:

Please don't do that. You just created THREE meaningless issues.
—
Reply to this email directly or view it on GitHub
https://github.com/tdenniston/bish/issues/8#issuecomment-77989421.

Haha, no worries. Glad you like the project. However, I will close this issues now :-).


fix documentation content

Purpose

Update the code for flow.dag.yml file in the prompt flow.
Does this introduce a breaking change?
When developers merge from main and run the server, azd up, or azd deploy, will this produce an error?
If you're not sure, try it out on an old environment.
[ ] Yes
[X] No

Does this require changes to learn.microsoft.com docs?
This repository is referenced by (https://azure.microsoft.com/products/phi-3)
which includes deployment, settings and usage instructions.
[ ] Yes
[X] No

Type of change
[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[X] Documentation content changes
[ ] Other... Please describe:

Hi @yuting1008, thank you for submitting the PR for this E2E sample. However, as far as I know, it worked fine without these variables. Could you explain why these variables are needed? Also, I'd like to understand what kind of bug occurs when this change is not applied.

@yuting1008  can you please confirm what the proposed changes do?


Fix mistranslation of external sources

Remapping of external sources in the index is done by replacing the path with xcodeOutputBase, which is built by removing paths from xcodeExecutionRoot. However, when split() is called, the leading / from xcodeExecutionRoot is lost, which was causing index-import to not translate these files correctly.

You'll need to sign-off on your git commit to pass the DCO check.

Thanks @rockbruno!


Add null check consistent with the rest of the handler.

Re-instate a null check. Need to get this into MRTK so we don't have to keep adding it.

/azp run

/azp run

This is actually unnecessary. Rather than creating a ManipulationHandler and trying to access its RotationConstraint, create (AddComponent) a ManipulationHandler and then, before ManipulationHandler's Start() can get called, AddComponent a RotationAxisConstraint and setup that, then when ManipulationHandler.Start() gets called, it will grab that existing RotationAxisConstraint. Fragile, but works.

This is actually unnecessary. Rather than creating a ManipulationHandler and trying to access its RotationConstraint, create (AddComponent) a ManipulationHandler and then, before ManipulationHandler's Start() can get called, AddComponent a RotationAxisConstraint and setup that, then when ManipulationHandler.Start() gets called, it will grab that existing RotationAxisConstraint. Fragile, but works.


Add data migration for updating NHS Trust and NHS Region Geography names and geography codes

Description
This PR includes the following:

Adds a data migration to update the NHS Region and trust geography records

Fixes #CDD-2101

Type of change
Please select the options that are relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Tech debt item (this is focused solely on addressing any relevant technical debt)

Checklist:

[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests at the right levels to prove my change is effective
[ ] I have added screenshots or screen grabs where appropriate
[ ] I have added docstrings in the correct style (google)


Feat/alias for docs site to catch import issues

Aliases actually do work!
For all of the docs site, we can now use Skeleton exactly as an end developer would via:
import {Card} from '@braindandbones/skeleton
Any internal ones like CodeBlock can just be imported from the $lib.  Also comes with intellisense too.
This closes #103 and should go a long way to making sure it doesn't happen for other new components.

Still a bit on the fence about this one, but I suppose having the option to go relative or alias is valid. We'll give it a go!


Unable to implement ee_to_geopandas

I have been trying to incroporate ee_to_geopandas in my project using Jupyter. It produced an error showing that ee_to_geopandas is not a valid
it was something like:
module 'geemap' has no attribute 'ee_to_geopandas'

Use ee_to_gdf


Why is there not a queue set here  / URLSession(configuration: configuration, delegate: self, delegateQueue: nil)

    session = URLSession(configuration: configuration, delegate: self, delegateQueue: nil) ???

and why not use AVAssetDownloadURLSession
https://developer.apple.com/documentation/avfoundation/avassetdownloadurlsession
instead of URLSession ?

There is no need for queue to be set, please read Apple's docs first before opening an issue https://developer.apple.com/documentation/foundation/urlsession/1411597-init.
Which benefit does AVAssetDownloadURLSession have over the URLSession in this case?

ok thanks -
not actually sure if AVAssetDownloadURLSession is of use - seems to be available in later os - thought it may supercede older code


Updated the request header sent to the OIDC endpoint so it doesn't result in a redirect response when an invalid system access token is provided.

Fixes https://github.com/Azure/azure-sdk-for-net/issues/46130

/azp run net - identity

/azp run net - identity - tests

/azp run net - identity - tests

API change check
API changes are not detected in this pull request.

/azp run net - identity - tests

/azp run net - identity - tests

@christothes I keep seeing a seemingly unrelated test fail on Ubuntu:
Failed KubectlExecuteIdentityAKSTests
It fails on retry.
Is this a known issue? It fails on identity test runs outside this PR, which makes me believe it isn't related to it.
Any recommendations on how to unblock this PR? I have gotten enough confidence of correctness from the passing tests (and new test addition) in the net - identity - tests legs, so I could reset and run just the CI pipelines/unit tests. Does that work?
2024-09-24T20:33:20.0037474Z   Failed KubectlExecuteIdentityAKSTests [754 ms]
2024-09-24T20:33:20.0038481Z   Error Message:
2024-09-24T20:33:20.0041045Z    System.InvalidOperationException: ERROR: AADSTS700024: Client assertion is not within its valid time range. Current time: 2024-09-24T20:33:19.6123783Z, assertion valid from 2024-09-24T20:16:20.0000000Z, expiry time of assertion 2024-09-24T20:26:20.0000000Z. Review the documentation at https://learn.microsoft.com/entra/identity-platform/certificate-credentials . Trace ID: 4edaea99-30f4-4c2f-83f5-a7b4e83f3000 Correlation ID: 951d6530-5f14-4a20-a9a5-aba45e411644 Timestamp: 2024-09-24 20:33:19Z
2024-09-24T20:33:20.0044185Z Interactive authentication is needed. Please run:
2024-09-24T20:33:20.0046014Z az login
2024-09-24T20:33:20.0048084Z    at Azure.Identity.ProcessRunner.Run() in /_/sdk/identity/Azure.Identity/src/ProcessRunner.cs:line 66
2024-09-24T20:33:20.0050003Z    at Azure.Identity.Tests.ManagedIdentityAKSIntegrationTests.RunCommand(String fileName, String args) in /mnt/vss/_work/1/s/sdk/identity/Azure.Identity/tests/ManagedIdentityAKSIntegrationTests.cs:line 76
2024-09-24T20:33:20.0052414Z   Stack Trace:
2024-09-24T20:33:20.0054820Z      at Azure.Identity.Tests.ManagedIdentityAKSIntegrationTests.RunCommand(String fileName, String args) in /mnt/vss/_work/1/s/sdk/identity/Azure.Identity/tests/ManagedIdentityAKSIntegrationTests.cs:line 83
2024-09-24T20:33:20.0058030Z    at Azure.Identity.Tests.ManagedIdentityAKSIntegrationTests.SetupKubernetesEnvironment() in /mnt/vss/_work/1/s/sdk/identity/Azure.Identity/tests/ManagedIdentityAKSIntegrationTests.cs:line 41
2024-09-24T20:33:20.0061136Z    at Azure.Identity.Tests.ManagedIdentityAKSIntegrationTests.KubectlExecuteIdentityAKSTests() in /mnt/vss/_work/1/s/sdk/identity/Azure.Identity/tests/ManagedIdentityAKSIntegrationTests.cs:line 60
2024-09-24T20:33:20.0063491Z
2024-09-24T20:33:20.0067912Z 1)    at Azure.Identity.Tests.ManagedIdentityAKSIntegrationTests.RunCommand(String fileName, String args) in /mnt/vss/_work/1/s/sdk/identity/Azure.Identity/tests/ManagedIdentityAKSIntegrationTests.cs:line 76
2024-09-24T20:33:20.0070993Z    at Azure.Identity.Tests.ManagedIdentityAKSIntegrationTests.SetupKubernetesEnvironment() in /mnt/vss/_work/1/s/sdk/identity/Azure.Identity/tests/ManagedIdentityAKSIntegrationTests.cs:line 41
2024-09-24T20:33:20.0074273Z    at Azure.Identity.Tests.ManagedIdentityAKSIntegrationTests.KubectlExecuteIdentityAKSTests() in /mnt/vss/_work/1/s/sdk/identity/Azure.Identity/tests/ManagedIdentityAKSIntegrationTests.cs:line 60
2024-09-24T20:33:20.0075791Z
2024-09-24T20:33:20.0076432Z
2024-09-24T20:33:20.0078061Z   Standard Output Messages:
2024-09-24T20:33:20.0079856Z  Running command: which az
2024-09-24T20:33:20.0081463Z  output:
2024-09-24T20:33:20.0082969Z  /usr/bin/az
2024-09-24T20:33:20.0084595Z  Running command: which kubectl
2024-09-24T20:33:20.0086169Z  output:
2024-09-24T20:33:20.0087866Z  /usr/bin/kubectl
2024-09-24T20:33:20.0094844Z  Running command: /usr/bin/az login --federated-token *** --service-principal -u *** --tenant ***
2024-09-24T20:33:20.0099150Z  System.InvalidOperationException: ERROR: AADSTS700024: Client assertion is not within its valid time range. Current time: 2024-09-24T20:33:19.6123783Z, assertion valid from 2024-09-24T20:16:20.0000000Z, expiry time of assertion 2024-09-24T20:26:20.0000000Z. Review the documentation at https://learn.microsoft.com/entra/identity-platform/certificate-credentials . Trace ID: 4edaea99-30f4-4c2f-83f5-a7b4e83f3000 Correlation ID: 951d6530-5f14-4a20-a9a5-aba45e411644 Timestamp: 2024-09-24 20:33:19Z
2024-09-24T20:33:20.0102560Z  Interactive authentication is needed. Please run:
2024-09-24T20:33:20.0104500Z  az login
2024-09-24T20:33:20.0106630Z     at Azure.Identity.ProcessRunner.Run() in /_/sdk/identity/Azure.Identity/src/ProcessRunner.cs:line 66
2024-09-24T20:33:20.0110149Z     at Azure.Identity.Tests.ManagedIdentityAKSIntegrationTests.RunCommand(String fileName, String args) in /mnt/vss/_work/1/s/sdk/identity/Azure.Identity/tests/ManagedIdentityAKSIntegrationTests.cs:line 76
2024-09-24T20:33:20.0127239Z
2024-09-24T20:33:20.0127903Z

From analytics, this particular test has failed intermittently over the last few weeks:

@christothes I keep seeing a seemingly unrelated test fail on Ubuntu: Failed KubectlExecuteIdentityAKSTests It fails on retry.
Is this a known issue? It fails on identity test runs outside this PR, which makes me believe it isn't related to it. Any recommendations on how to unblock this PR? I have gotten enough confidence of correctness from the passing tests (and new test addition) in the net - identity - tests legs, so I could reset and run just the CI pipelines/unit tests. Does that work?
Please let me know if you'd like me to file a tracking issue for it (or optionally disable the test).

Yes, let's file a tracking issue. I think this is just a timing related issue since we are using the OIDC token that was set at the begining of the pipeline and sometimes we just happen to execute those tests after the 10 minute lifetime expiration. I have some ideas for how to resolve it that we can discuss in the issue.
We can Ignore the test in the meantime, if it is blocking now consistently, and cite the issue in the reason.

Yes, let's file a tracking issue.

Done https://github.com/Azure/azure-sdk-for-net/issues/46209


Zr ti

Large amount of file changes for updated experimental sample coming in nov

This PR included a set of decent model weights, and an updated approach to using XCA as an agent


@CasePathable fails to compile when nested in private type

Describe the bug
@CasePathable generates code that fails to compile when the enum is nested in a private type.
To Reproduce
struct A {
    private struct B {
        @CasePathable
        enum C {}
        // Expanded Macro
        extension A.B.C: CasePaths.CasePathable, CasePaths.CasePathIterable { // 🛑 Error: 'B' is inaccessible due to 'private' protection level
        }
    }
}

Expected behavior
Successful compilation.
Environment

swift-case-paths 1.5.6
Xcode 16.0
Swift 6.0
OS: iOS 18.0.1

Additional context
Relaxing private to fileprivate can be used as a workaround.

Hi @pietrasm, unfortunately I'm not sure there is anything we can do to fix this. Our macro is merely expanding an "extension macro" to add some conformances to the C type, and Swift is interpreting that to mean extension A.B.C. I think for now using fileprivate is a good workaround, or you can add the conformances yourself and then the macro will skip them:
@CasePathable
enum C: CasePathable, CasePathIterable {}

Since this isn't an issue with the library I am going to convert it to a discussion. Please feel free to continue the conversation over there! And if you have the time I recommend filing an issue with Apple because it does seem like something that should be allowed.


cannot udpate settings in vscode

Does this occur consistently?
Repro steps:

Action: azureFunctions.appSettings.download
Error type: Error
Error Message: Failed to run "func" command. Check output window for more details.
Version: 0.22.1
OS: darwin
OS Release: 18.7.0
Product: Visual Studio Code
Product Version: 1.46.0
Language: en

Call Stack
Object.<anonymous> extension.bundle.js:1:133816
fulfilled extension.bundle.js:1:132699
processTicksAndRejections task_queues.js:85:5

Hi @helloanh
Thanks for reporting.  Could you provide repro steps please?  Just by running the Download Remote Settings... command, I'm not able to reproduce this bug.


[Blog] Spotlight for k8s upstream training in Japan

Description
We are the organizing team for k8s upstream training in Japan. see https://github.com/kubernetes-sigs/contributor-playground/tree/master/japan for more details
We would like to feature our efforts in k/website blog!
Important Date

We plan to move this PR out of draft status by early October
We would like to publish this blog post within October early November (November 10?)

Let me know if you’d like any adjustments!
Issue
Closes: #

/cc @shu-mutou @ziyi-xie @atoato88

@Okabe-Junya Thanks for submittin this!
LGTM for now. I look forward to hearing more about your interview! :smile:

LGTM.
Thank you for creating this blog post.

LGTM Thank you so much for the amazing content you created!

/auto-cc

/retitle [Blog] Spotlight for k8s upstream training in Japan

/assign @nate-double-u @sftim

Special thanks @mochizuki875 @ffjlabo !!

The November 10th publication target date is plausible, but because there is a very small pool of reviewers for both sites, I don't think it's likely.

I feel like Contributor Comms can help make that happen.
Cc @kaslin @fsmunoz @mfahlandt

Thank you, Tim and chris-short!
I see, considering our topic, it makes sense to post it on https://k8s.dev/blog/.
I would like to create a PR later to post it on k8s.dev

but because there is a very small pool of reviewers for both sites, I don’t think it’s likely.

Ah, I saw a post about this topic on Slack.
Of course, that’s not an issue. I want to take this opportunity to once again express my gratitude to you and your team for the tremendous contributions in running k/website blog.

OK to unhold if: ...

And the canonical url is the K8s.dev site, right @sftim?

Opened https://github.com/kubernetes/contributor-site/pull/541! Thanks!

Thank you for your review, @sandeepkanabar !!

Looks good - great read! Thanks for the effort on this! Just one nit - shouldn't we use Sentence Case for the headings? (Ref - style guide)

Please provide feedback on https://github.com/kubernetes/contributor-site/pull/541 (which this PR mirrors)
but yes, [having the] headings in sentence case is better

Thanks, Tim! Frederico has already suggested the required edits there :)

Yes, it was the only thing I could think that was an improvement 😀

/lgtm
/approve

Previewed locally; all looks good
/lgtm
/approve
/hold cancel


MicroHR installation

https://sentinelcustomer.gemalto.com/sentineldownloads/

Closing as this is going to be handled by yaqd-horiba going forward

(Also means the sentinel link is no longer needed !!)


django-admin-relation-links 0.2.5

What problem does this feature will solve?
How can one use this feature with import-export feature.
I assume the existence of a certain DB already in place as links are needed.
Can it work also with import-export CSV files? where the links are established after the import?
If yes, how one should apply
class AdminMember(ImportExportModelAdmin)
with class GroupAdmin(AdminChangeLinksMixin, admin.ModelAdmin)
Or should the import be done via a script of separated tables?
or else, how it differs from linking say members and groups in manytomany relationship via dedicated script that adds a linking model, say membership and via get_or_create command?

Hi, this issue tracker is for issues with the codebase behind https://pypi.org itself, not the projects hosted on PyPI.
You should report this issue to the tracker for the project in question instead.


Define OnyxSupportButton

Related to

#1008

Acceptance criteria

[ ] A new created refined Implement OnyxSupportButton ticket
[ ] Replaced OnyxIconButton in packages/sit-onyx/src/components/OnyxDataGrid/features/sorting/SortAction.vue
[ ] Ticket was created

See #2139


Issue 5 and 6

Pullrequest for issues 5 and 6:

boolean attributes are converted to string
text nodes with value false and 0 are not rendered

Would be great to have some tests.
Also you can use the predefined isArray and change this to use Array.isArray like so
var isArray = Array.isArray || function isArray(thing) {
  return Object.prototype.toString.call(thing) === '[object Array]';
}

tests added and isArray modified

just released 0.1.8
Thanks for contributing!


desktop icons fences !?

Description of the new feature / enhancement
hi there, I have an idea about the icons.
what about putting some fences on certain icons to separate them as groups?
there's an application named Fences, very helpful to make the desktop cleaner and to find the icons easier.
thank you
Masoudi
Scenario when this would be used?
it makes the desktop very clean and will be easier to find the icons, files, and folders when there are a lot of icons on the desktop.
the user can sort everything on the desktop as groups for example | shortcuts, Folders, Text documents, photos, and even Media such as Videos or Music.
Supporting information
some tunning settings such as Fence opacity, free size adjustable, icon size, Fence Color \ Name
Fence free drag on the desktop.
<3

Hello and thanks for your message. There's a program that acutally does this: Stardock Fences https://www.stardock.com/products/fences

/dup #1392
If a 3rd party app like fences solves the problem, check them out.

Hi,
Yes i mentioned that, there's program named Fence.
I meant to add something similar and light and will be affordable for all
users.
Fence software is paid.
And such tool will help with the workflow.
Thanks 🙏
On Sun, Jan 22, 2023, 10:45 PM Jay @.***> wrote:

Hello and thanks for your message. There's a program that acutally does
this: Stardock Fences https://www.stardock.com/products/fences
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/PowerToys/issues/23532#issuecomment-1399616878,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A5M432KULPWRX42HWSIRU53WTWS77ANCNFSM6AAAAAAUDEQL7U
.
You are receiving this because you authored the thread.Message ID:
@.***>


Extension without publisher should be able to run / debugged

yo code
Remove publisher field
Error

[/Users/octref/Code/language-extension-samples/grammar-hello-world]: property `publisher` is mandatory and must be of type `string`"

Why do I have to have a publisher to run/debug an extension?

@octref Is this coming from yo code?

@sandy081 No the error comes from VS Code. VS Code literally doesn't allow me to run the extension using launch.json, if its package.json doesn't contain a publisher.

@octref It comes from [here[(https://github.com/Microsoft/vscode/blob/master/src/vs/workbench/services/extensions/node/extensionPoints.ts#L370)
Publisher id is mandatory because it is used to generate the id of the extension. So I think this cannot be made optional.


Corrected mut function in Neural_Network.pde file

Hello! I think there is a little error in 'mut' function of 'Neural_Network.pde' file in coding challenge 100. From my knowledge the if condition on line 69 should be:
if(random(1) < rate)
instead of:
if(random(1) > rate)
Suppose the rate is 0.9, then what our intention is to mutate 90% of the time. But according to me, the function is performing other way round right now i.e the probability that the random(1) > 0.9 is just 10%.
Also when I checked the P5 version of this coding challenge, I found the 'mutate' function correct there.
Here are the snippets from both the versions.
This snippet is from P5 version which I think is correct:

And this is from Processing version:

Correct me if I'm missing something here.
Cheers!

@GypsyDangerous What are your thoughts?

I agree it needs to be changed.


Add AWS protocol tests

This commit adds the start of AWS protocol tests for REST XML and JSON
RPC 1.1. More protocol tests to follow.
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.

This is going to fail to build until we cut a release of the Gradle plugin. This is the first time we've used the Gradle plugin in this repo, and I think we can feel free to use it more. It's nice.

Broke this into two separate PRs, one for each protocol


Keep a history of "Find All References"

When studying a large code base written by somebody else, I make heavy use of the built-in "Find All References" command to understand where and how symbols are used, and I often need to follow from one symbol to another symbol. E.g. I look for references of a class Foo and then, when studying where and how Foo is used, I see another interesting class Bar and I want to find references for Bar.
Unfortunately, VS Code currently does not allow me to easily backtrack to the "Find All Reference" results of a previous symbol.
It would be really nice if VS Code could keep a history of the "Find All Reference" results that I can easily go back and forth in, especially if you can keep it as a stack in the side panel.

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

This feature request has not yet received the 20 community upvotes it takes to make to our backlog. 10 days to go. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

:slightly_frowning_face: In the last 60 days, this feature request has received less than 20 community upvotes and we closed it. Still a big Thank You to you for taking the time to create this issue! To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

It would be must-have feature and really nice if VS Code could keep a history of the "Find All Reference" results.

It would be must-have feature and really nice if VS Code could keep a history of the "Find All Reference" results.
Hope this extension helps you.
https://marketplace.visualstudio.com/items?itemName=happiness86.vscode-references-plus


Correct for cartographer PR #227.

Also includes some clang-format fixups.

Although I haven't tested this, I think this PR could land before Cartographer 227 and no builds would break.

@macmason Travis agrees with you. @wohe will be back in the office today.


index.js - fix typo

approved - but has conflicts. either resolve the conflict and re-fresh this PR. or open with branches on this repo (I can add you as a collaborator.
And thank you, for you PRs, and issues  :heart:

Conflicts resolved.
Thank you for building this important project. 👏
I just try to do my best to help.

No response from owner... 😢
Closing.


Multiple user registration with multiple providers

I use Devise Token Auth gem, and ng-token-auth. When signing up a new user with regular signup, then try to login with an OmniAuth provider such as GitHub, I get an exception:
ActiveRecord::RecordInvalid (Validation failed: Email has already been taken):
That's because DeviseTokenAuth tries to create a new user instead of updating the current one. I have no idea how to configure DeviseTokenAuth to just update the existing user.

FWIW I use the following strategy, this might not be totally accurate and not the expected behavior but I find it very useful.

Catch RecordNotUnique in OmniauthCallbacksController
Login the user manually with the authenticated email

class Api::V1::OmniauthCallbacksController < DeviseTokenAuth::OmniauthCallbacksController
  rescue_from ActiveRecord::RecordNotUnique, with: :user_exists_with_other_providers

  # RecordNotUnique is thrown when the email has already been used with another
  # provider, sign that user in since we know the user own the email through
  # their provider.
  def user_exists_with_other_providers
    @resource =<EMAIL_ADDRESS>
    create_token_info
    set_token_on_resource
    create_auth_params

    if resource_class.devise_modules.include?(:confirmable)
      @resource.skip_confirmation!
    end

    if @resource.avatar_uid.nil?
      @resource.avatar_url = get_large_image(auth_hash['info']['image'])
    end
    if @resource.nickname.nil?
      @resource.nickname = auth_hash['info']['nickname']
    end

    sign_in(:user, @resource, store: false, bypass: false)
    @resource.save!

    yield @resource if block_given?

    render_data_or_redirect('deliverCredentials', @auth_params.as_json, @resource.as_json)
  end
end

Workaround posted, so closing for now. I do agree though that a more official way to login through multiple providers would be nice!

Just wanted to mention that for the workaround to work (catching RecordNotUnique), you'll need a unique index on the email column


fuzz test fail due to unreliable timings

Test run on main fails, though at least this time it's not obviously a configuration error.
Pulling out the multiline message

raise Flaky(
hypothesis.errors.Flaky: Hypothesis test_does_not_crash_on_any_valid_code(self=<tests.test_flake8_trio.TestFuzz testMethod=test_does_not_crash_on_any_valid_code>, syntax_tree=<ast.Module object at 0x7f6cdd1faf80>) produces unreliable results: Falsified on the first call but did not on a subsequent one
Falsifying example: test_does_not_crash_on_any_valid_code(
syntax_tree=parse('assert  {}\nclass A: pass\ndef A(): pass\n'),
self=<tests.test_flake8_trio.TestFuzz testMethod=test_does_not_crash_on_any_valid_code>,
)
Unreliable test timings! On an initial run, this test took 608.31ms, which exceeded the deadline of 200.00ms, but on a subsequent run it took 6.86 ms, which did not. If you expect this sort of variability in your test timings, consider turning deadlines off for this test by setting deadline=None.

Is this due to xdist/CI servers being unreliable, or something weird in hypothesis, or is it an actual bug in the underlying code? Would be great to figure out why tests sometimes take 600ms
I'll probably not investigate this on my own until you've given your opinion since you're about a million times more experienced with hypothesmith/hypothesis.
=================================== FAILURES ===================================
________________ TestFuzz.test_does_not_crash_on_any_valid_code ________________
[gw1] linux -- Python 3.11.2 /home/runner/work/flake8-trio/flake8-trio/.tox/flake8_6/bin/python
Traceback (most recent call last):
  File "/home/runner/work/flake8-trio/flake8-trio/.tox/flake8_6/lib/python3.11/site-packages/hypothesis/core.py", line 878, in _execute_once_for_engine
    result = self.execute_once(data)
             ^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/runner/work/flake8-trio/flake8-trio/.tox/flake8_6/lib/python3.11/site-packages/hypothesis/core.py", line 817, in execute_once
    result = self.test_runner(data, run)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/runner/work/flake8-trio/flake8-trio/.tox/flake8_6/lib/python3.11/site-packages/hypothesis/executors.py", line 47, in default_new_style_executor
    return function(data)
           ^^^^^^^^^^^^^^
  File "/home/runner/work/flake8-trio/flake8-trio/.tox/flake8_6/lib/python3.11/site-packages/hypothesis/core.py", line 813, in run
    return test(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^
  File "/home/runner/work/flake8-trio/flake8-trio/tests/test_flake8_trio.py", line 647, in test_does_not_crash_on_any_valid_code
    @given((from_grammar() | from_node()).map(ast.parse))
               ~~~~~~~~~~~~^~~~~~~~
  File "/home/runner/work/flake8-trio/flake8-trio/.tox/flake8_6/lib/python3.11/site-packages/hypothesis/core.py", line 781, in test
    raise DeadlineExceeded(runtime, self.settings.deadline)
hypothesis.errors.DeadlineExceeded: Test took 608.31ms, which exceeds the deadline of 200.00ms

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/home/runner/work/flake8-trio/flake8-trio/tests/test_flake8_trio.py", line 647, in test_does_not_crash_on_any_valid_code
    @given((from_grammar() | from_node()).map(ast.parse))
               ^^^^^^^
  File "/home/runner/work/flake8-trio/flake8-trio/.tox/flake8_6/lib/python3.11/site-packages/hypothesis/core.py", line 1396, in wrapped_test
    raise the_error_hypothesis_found
  File "/home/runner/work/flake8-trio/flake8-trio/.tox/flake8_6/lib/python3.11/site-packages/hypothesis/core.py", line 842, in execute_once
    raise Flaky(
hypothesis.errors.Flaky: Hypothesis test_does_not_crash_on_any_valid_code(self=<tests.test_flake8_trio.TestFuzz testMethod=test_does_not_crash_on_any_valid_code>, syntax_tree=<ast.Module object at 0x7f6cdd1faf80>) produces unreliable results: Falsified on the first call but did not on a subsequent one
Falsifying example: test_does_not_crash_on_any_valid_code(
    syntax_tree=parse('assert  {}\nclass A: pass\ndef A(): pass\n'),
    self=<tests.test_flake8_trio.TestFuzz testMethod=test_does_not_crash_on_any_valid_code>,
)
Unreliable test timings! On an initial run, this test took 608.31ms, which exceeded the deadline of 200.00ms, but on a subsequent run it took 6.86 ms, which did not. If you expect this sort of variability in your test timings, consider turning deadlines off for this test by setting deadline=None.
Highest target scores:
              17  (label='(hypothesmith from_node) number of unique ast node types')
              21  (label='(hypothesmith) number of unique ast node types')
             112  (label='(hypothesmith) total number of ast nodes')
             181  (label='(hypothesmith) instructions in bytecode')
            1031  (label='(hypothesmith from_node) total number of ast nodes')
            2331  (label='(hypothesmith from_node) instructions in bytecode')

=========================== short test summary info ============================
FAILED tests/test_flake8_trio.py::TestFuzz::test_does_not_crash_on_any_valid_code
=================== 1 failed, 2 passed in 930.96s (0:15:30) ====================
flake8_6: exit 1 (931.58 seconds) /home/runner/work/flake8-trio/flake8-trio> pytest --onlyfuzz --no-cov -n auto pid=1800
.pkg: _exit> python /opt/hostedtoolcache/Python/3.11.2/x64/lib/python3.11/site-packages/pyproject_api/_backend.py True setuptools.build_meta __legacy__
  flake8_6: FAIL code 1 (937.07=setup[5.49]+cmd[931.58] seconds)
  evaluation failed :( (937.15 seconds)
Error: Process completed with exit code 1.

I think this test is probably just slow for some inputs, and we could disable the deadline. The existence of a cache for something slow is sufficient to explain the variability part.


return type () => Type is rendered as two separate returns

Doc:
/**
 * @returns returns doc
 */
export function fn(): () => Promise<any> {
}

Output:

It's a bit confusing.

While somewhat confusing at first, this is expected behavior. Newer versions render this in a way that at least makes it clear what's going on:


Access to /tmp directory?

Hi,
Thanks for the great work! It's been very helpful so far.
I was wondering if there is a way to access the /tmp directory?
Thanks,
Andrea

@andreacab you mean the temporary directory on iOS? (NSTemporaryDirectory)  That is not implemented at the moment but I'm sure it could be added. Do you know whether there is a temporary directory available on Android too?


Fix service scale to set service running state

Split out of #3277
Fixes #2290 GridServices::Scale can leave the service in the initialized state
Set the GridService.state to running for GridServices::Scale when deploying the service.
Includes GridServices::Deploy refactoring for #3277.
Services will no longer get deployed in the initialized state, removing some special cases.

This is the first PR in the associated series of PRs to fix kontena stack rm --wait. The remainder are WiP waiting for this to get merged.


Accelerate mh-sha1 by ASIMD on aarch64

This patch use the Advanced SIMD of aarch64 to accelerate mh_sha1
Change-Id: I321c68a8175e1ac8a3ffa74c9fb916f8bba4832d
Signed-off-by: Daniel Hu<EMAIL_ADDRESS>

I'll schedule for integration.

Integrated


.toLocaleString throws error in Chrome on macOS 14

After updating to macOS 14 the polyfill throws an error in Chrome when calling toLocaleString on a PlainDateTime. The polyfill functions correctly in Firefox.
Reproduction here https://stackblitz.com/edit/temporal-polyfill-macos-chrome
macOS 14 Sonoma
Chrome 117
"@js-temporal/polyfill": "^0.4.4"

macOS 14 Chrome 117

macOS 14 Firefox

macOS 13 Chrome 117

Thanks for the report! Could you share the stack trace of the error?

Stacktrace for the above example:
ecmascript.ts:173 Uncaught (in promise) TypeError: Cannot read properties of undefined (reading 'getPossibleInstantsFor')
    at GetMethod (ecmascript.ts:173:18)
    at GetPossibleInstantsFor (ecmascript.ts:2833:65)
    at GetInstantFor (ecmascript.ts:2689:28)
    at extractOverrides (intl.ts:539:16)
    at DateTimeFormatImpl.format (intl.ts:193:32)
    at PlainDateTime.toLocaleString (plaindatetime.ts:384:49)

I'm also seeing another similar issue in Chrome 117 on macOS 14 calling Temporal.Now.plainDateISO(), happy to report this as a separate issue if preferred.
runtime-core.esm-bundler.js:221 Uncaught RangeError: Invalid time zone specified: undefined
    at new DateTimeFormat (<anonymous>)
    at getIntlDateTimeFormatEnUsForTimeZone (ecmascript.ts:356:16)
    at GetCanonicalTimeZoneIdentifier (ecmascript.ts:3148:21)
    at ParseTemporalTimeZone (ecmascript.ts:462:24)
    at ToTemporalTimeZoneSlotValue (ecmascript.ts:2592:10)
    at plainDateTimeISO (now.ts:19:14)
    at Temporal.Now.plainDateISO (now.ts:38:36)
    at PlainDate.now (PlainDate.ts:18:52)

Looking into the Invalid time zone specified error I have found that Temporal.Now.timeZoneId() is returning undefined

After working my way through the polyfill code, I've found the issue to be Chrome not setting timeZone

Closing the issue here, will report to the Chrome team.

Hi @twoBoots - I'm glad you've identified the root cause. Could you link the Chrome bug here after you've filed it?
Also, what does Firefox report in the same case?

Will link details here when I have them.
macOS 14 Firefox

macOS 13 Chrome 117
{
    "locale": "en-GB",
    "calendar": "gregory",
    "numberingSystem": "latn",
    "timeZone": "Australia/Melbourne",
    "year": "numeric",
    "month": "2-digit",
    "day": "2-digit"
}

I'm not sure I understand the results in your comment above. What changed between your earlier comment where Chrome reported undefined and when it reported Australia/Melbourne?  Could it be that the OS needed some time after upgrading in order to find which time zone it was in?

In the last comment, Chrome on macOS 13 (previous) reported Australia/Melbourne, but Chrome on macOS 14 (latest) reports undefined.
This was probably unclear as the earlier comment didn't have the OS version beside the screenshot, I've updated it.

Another report of the same issue https://support.google.com/chrome/thread/237667177/macos-sonoma-14-chrome-is-reporting-wrong-time-due-to-undefined-timezone?hl=en

https://bugs.chromium.org/p/chromium/issues/detail?id=1487920
https://bugs.chromium.org/p/chromium/issues/detail?id=1490767


Author CONTRIBUTING documentation

Is your feature request related to a problem? Please describe.
When bootstrapping the repo, an empty contributing document was created. This document needs to be written.
Describe the solution you'd like
First-pass of a contributors document to be added. As a point-of-reference Spotify does a great job with documentation in their public repos, and this might represent a good model to build from (we don't, however, want to simply copy their docs verbatim).
Additional context

The current placeholder doc is here (it should remain in this location, versus the root of the repository).

In progress.


User is no longer prompted with all build tasks when there are multiple default built tasks in a multi-root workspace

Does this issue occur when all extensions are disabled?: Yes

Version: 1.63.2 (user setup)
Commit: 899d46d82c4c95423fb7e10e68eba52050e30ba3
Date: 2021-12-15T09:40:02.816Z
Electron: 13.5.2
Chromium: 91.0.4472.164
Node.js: 14.16.0
V8: 9.1.269.39-electron.0
OS: Windows_NT x64 10.0.19043
Steps to Reproduce:

Create a multi-root workspace like this:

{
    "folders": [
        {
            "path": "subworkspace1",
            "name": "subworkspace1"
        },
        {
            "path": "subworkspace2",
            "name": "subworkspace2"
        }
    ]
}

Within each folder, create .vscode/tasks.json files with the following content:

{
    "version": "2.0.0",
    "tasks": [
        {
            "label": "Subworkspace 1 build task",
            "type": "shell",
            "command": "echo Subworkspace1",
            "group": {
                "kind": "build",
                "isDefault": true
            }
        }
    ]
}

{
    "version": "2.0.0",
    "tasks": [
        {
            "label": "Subworkspace 2 build task",
            "type": "shell",
            "command": "echo Subworkspace2",
            "group": {
                "kind": "build",
                "isDefault": true
            }
        }
    ]
}

Click ctrl+shift+p and select Tasks: Run Build Task
The build task for subworkspace 1 is run

I'm not sure what the expected behavior is in this case, but I've noticed it was changed between version 1.62 and 1.63. Previously, Tasks: Run Build Task would show all build tasks when there were multiple default ones, but now it runs the first default build task in the list of folders.
Let me know if any information is missing, or if this is actually expected behavior now

We should still prompt here. Thanks for pointing this out!

Looks like this is already fixed in insiders.

Looks good. I see the prompt now.


[MINOR] Fix Badges with links

Fix badges to only reflect status on push to master branch
Make badges link to the tests conducted.

This looks good to me. Thank you.

If the commit is ready to be merged adding a line closes #903.  helps us merge on the github itself.

since the OSS release in 2015, we try to avoid direct merges (to prevent merge commits and other anomalies) - please, follow https://gist.github.com/deroneriksson/e0d6d0634f3388f0df5e

thanks @Baunsgaard and @j143


Text doesn't render properly

Attach (reccomened) or Link to PDF file here:
https://drive.google.com/open?id=0B2Z7u3CUJ0-NSkJPNTd1NThkWUk
chrome-extension://oemmndcbldboiebfnladdacbdfmadadm/https://drive.google.com/a/texthelp.com/uc?id=0B2Z7u3CUJ0-NSkJPNTd1NThkWUk&export=download&filename=bad pdf.pdf&share=true
Configuration:

Web browser and its version: Version 61.0.3163.100 (Official Build) (64-bit)
Operating system and its version: mac v10.13
PDF.js version:  1.8.557
Is an extension: yes

Steps to reproduce the problem:

the text gets drawn on top of other text

What is the expected behavior? (add screenshot)

What went wrong? (add screenshot)

Closing as fixed by PR #8938.


init jobscript + rule specific tmpdirs.

make specific tmpdir per rule under /scratch/local (hardcoded)
job shell script with trap cleanup similar to slurmeasy

👍


[T3A3][T14-A1] Low Zheng Heng Henry

Split Address class into Block, Street, Unit and PostalCode classes.
Update code segments referencing Address.value to use Address.toString()

Ready for review

you have aptly split the address class to four component classes.
however, please take note that you need to update the user guide, in program help and tests that correspond to the change.

@e0003801
Some comments added. Please ack & close the PR after reading comments.


refactor: test and cleanup injector

Description
This change adds tests for the injector flow and cleans up the code.
Related Issue
N/A
Checklist before merging

[x] Test, docs, adr added or updated as needed
[x] Contributor Guide Steps followed

Lets wait for pkg PRs to merge first.
https://github.com/defenseunicorns/pkg/pull/105
https://github.com/defenseunicorns/pkg/pull/104


[stable/sapho] bump dependency mysql 0.2.7

/ok-to-test

/cc @lodotek

How do we get this merged?


fix: Align lines with emojis correctly

Alignment issues occur when some lines have certain emojis in them. This is caused by incorrectly calculating the width of a string.
For a string with emojis in them, it is important to understand that the string width is different to the rune width. Many functions within Lipgloss base the string length on the printable rune width which is incorrect. The correct method requires finding the string width.
The reflow package has a printable rune width function. While the ANSI sequences need to be removed, we do not want the rune width as this function returns.
This change adds a new function to get the printable string width. The code was taken from reflow and modified for use in Lipgloss. Since the printable rune width function in reflow works correctly, it was more reasonable to add the function to Lipgloss.

Please do fix this upstream if you found any issues. At first glance I don't see a difference however?

The key change is the use of runewidth.StringWidth instead of runewidth.RuneWidth
I will provide a repository shortly showcasing the problem and why this pull request resolves the problem.

I want to clarify that there is no bug with reflow. It is just providing a different answer to the question. Lipgloss needs to know the printable string width, not the printable rune width. Are you willing to accept a pull request to reflow to add this function? Lipgloss still needs to be updated to use it however.

Yeah, definitely willing to accept PRs there, so we don't end up copying similar code all over the place.

I will provide a repository shortly showcasing the problem and why this pull request resolves the problem.

Cool, a few test cases might be nice here.

Test case with both code and images.
https://github.com/mikelorant/emoji-alignment
Hopefully these results are consistent with other platforms and terminals.

Unfortunately, I don't believe this is a valid solution. I'm seeing different results with kitty, however I believe you will also see different results with multi-sequence emojis such as the following. I'm providing both emoji literals and unicode sequences here, which Go will evaluate equally.
// alembic
"⚗"
"\U00002697\U0000FE0F"

// kiss: woman, man, medium-dark skin tone, light skin tone (with zero width joiners)
"👩🏾❤💋👨🏻"
"\U0001F469\U0001F3FE\U0000200D\U00002764\U0000FE0F\U0000200D\U0001F48B\U0000200D\U0001F468\U0001F3FB"

Tested the two emojis you provided into my example code.

I think the complexity was always going to be the terminals and operating system support. However, on a Mac you can see this improves the results.

Even more interesting is my patch seems to get much closer to handling the 12 byte emoji. Normally the main issue is table borders break. This did not occur which indicates this may just be an issue with calculating maximum line length.

If we can identify more emojis that fail this will allow us to build a small bubbletea app that can be used for testing purposes. Its hard to identify all the edge cases.

Finding some very unusual results when using kitty. Providing some comparison shots with iTerm2 and Terminal for reference.

Not sure what all this means except more research is required.

Kitty has its own wcwidth implementation while Terminal and iTerm2 use the system wcwidth. This may not be solvable if rendering is done differently by terminals.

What I would recommend for now is to simply avoid any emojis that are causing you problems.
The crux of the issue here is that emoji widths must both be correctly calculated in both Go and on the terminal level, and neither side is perfect yet. Even Kitty, which is among the best renderers I've seen, doesn't render every single Emoji correctly. Going further, some desktop environments don't support newer emojis.
We have some tooling in the works that should alleviate the problem a bit on the Go side, but there's not ETA yet.

Got it, thanks for the additional context @meowgorithm.

simply avoid any emojis that are causing you problems.

Just a clarification on that front, almost all the emojis I tried are suffering from the same issue.
Some of them include: ✨, 🌊, 💫, ✅, 🚀

@praagyajoshi which terminal, OS, and window manager (if relevant) are you using? As I mentioned this varies widely depending on terminal and can be affected by other OS-level factors.
Also, we still need to test properly but I suspect that switching from go-runewidth to rivo/uniseg directly under the hood may solve for this, particularly as rivo/uniseg now supports both the latest version of Unicode (15.0.0) and the ability to return cell widths.

I'd like to bring this issue up again based on some feedback from @rivo.

That's because a single code point (rune) is not always a complete character. That's the whole basis of the uniseg package. The README explains this in detail. One example given there is the country flag emoji. These emojis must always consist of two runes. What's the width of only one of these runes then? It's basically undefined because it's an incomplete grapheme cluster. The mattn/go-runewidth package gets this fundamentally wrong and I have tried for a long time to help them do it right but there was never much interest in following up on it.
https://github.com/rivo/uniseg/issues/48#issuecomment-1904871545

Using the rune width anywhere is something we need to be very careful about. In most cases I would expect we should only be using the string width.
It is likely we will not see the full benefits unless we replace RuneWidth functions across nearly all of the package as well as its dependencies. This makes this quite a complicated and time consuming task.
My recommendation would be to create a new issue for either bubbles or bubbletea and determine all the dependencies that would need to be fixed. This would then allow for issues and hopefully pull requests in each of the repositories.

Hey! We actually began work on switching to rivo.StringWidth today. It's not too daunting of a task as we maintain a fairly tight level of control over our dependency chain, at least as far as rendering is concerned. Here’s a few PRs for it so far:
https://github.com/muesli/reflow/pull/69
https://github.com/muesli/termenv/pull/164
https://github.com/charmbracelet/bubbles/pull/462

@meowgorithm Been tracking the great work on the StringWidth improvements. 👍
This is why I have now turned my attention to RuneWidth. This is just so much more complex because we can't just swap it out. While we can convert the rune to a string we now know it is not always going to be valid.
As you are one of the project maintainers, would be great to get some guidance on where we should start. Something experimental to prove this will give us improvements.

So I think ultimately be counting and iterating over graphemes like you suggest. For example, ⚗ is 6 bytes, 2 runes, and 1 grapheme cluster playground.
I'd also, have a look at this excellent writeup on grapheme clusters in terminals by Mitchell Hashimoto. It's the best reference I've seen on this subject.

I've put up a preliminary pull request for muesli/reflow to iterate over grapheme clusters instead of runes. There are so complexities that will likely require further discussion but we can deal with them via an issue on that repository instead.

Awesome. For reference that's https://github.com/muesli/reflow/pull/71 (also FYI @muesli is on the Charm team).
@maas, heads up re: textarea.

Closing this pull request as this is going to be fixed with muesli/reflow#71.


[1.13.x] Fix CLI build

Fix https://eng-jenkins-csb-business-automation.apps.ocp-c1.prod.psi.redhat.com/blue/organizations/jenkins/KIE%2Fkogito%2F1.13.x%2Fnightly%2Fkogito-operator-deploy/detail/kogito-operator-deploy/113/pipeline
Many thanks for submitting your Pull Request :heart:!
Please make sure your PR meets the following requirements:

[ ] You have read the contributors' guide
[ ] Pull Request title is properly formatted: [KOGITO-XYZ] Subject
[ ] Pull Request contains a link to the JIRA issue
[ ] Pull Request contains a description of the issue
[ ] Pull Request does not include fixes for issues other than the main ticket
[ ] Your feature/bug fix has a unit test that verifies it
[ ] You've ran make before-pr and everything is working accordingly
[ ] You've tested the new feature/bug fix in an actual OpenShift cluster
[ ] You've added a RELEASE_NOTES.md entry regarding this change

How to retest this PR or trigger a specific build:

Run operator BDD testing
Please add comment: /jenkins test

Run RHPAM operator BDD testing
Please add comment: /jenkins RHPAM test

/jenkins test


Add the ability to specify an overrides file directory.

A number of NixOS options allow a user to specify a file instead of writing one as a nix expression for configuration, e.g. services.usbguard.ruleFile, security.pki.certificateFiles, etc. This can be useful if a user, for example already has a large number of pre-written configurations for whatever program they are using.
In my case, I have a number of Flatpak override config files already present on my device. It would be very useful if I could simply drop them in a folder as part of my OS config. This would also allow users to use tools they are already familiar with, when configuring Flatpak overrides, e.g. flatseal, and the built in configuration tool present on KDE.

Hey @Spaghetto392,
Good proposal. I think this is useful and should not be hard to implement.
Currently, services.flatpak.overrides is just a Nix attrSet. I need to experiment a bit, but you should be able to split the configs across multiple files and assemble them at build time.
However, this doesn’t directly solve your problem. To load unmanaged (= not declared in nix-flatpak) overrides, you'll need to convert flatpak ini files to nix. There’s some ini-wrangling in #78 that might be useful for this feature.
Just to understand your use case better: how do you currently manage override files? Do you keep them version controlled outside of $XDG_DATA_HOME/flatpak/overrides`?

Hi @gmodena,
As it currently stands I don't really manage override files. I mostly just set them and forget them, which I know, isn't particularly great, but I haven't really had the time to setup anything more sophisticated. I've considered using something like the nextcloud sync client to add them to each of my devices, but have to consider if any of the configuration would need to be changed on a by device basis.
What I would like to do, is put my premade overrides into a directory as part of my nix repo, and have them be auto deployed system wide or perhaps per user.

What I would like to do, is put my premade overrides into a directory as part of my nix repo, and have them be auto deployed system wide or perhaps per user.

The tricky part is managing attrset merges between configurations loaded from Flatpak ini files and those declared in nix. Under the hood, nix-flatpak uses jq to handle overrides state. I’m still undecided on whether to add additional logic to this process or refactor the overrides management directly within nix expressions instead of relying on jq. jq is great, but it makes things harder to test compared pure nix.
tl;dr: this issue is still in scope, but might require a bit more effort than initially estimated.