Reduce automate callers

Goal: reduce calls into automate so we can more easily stub it out when we are working through various scenarios like provisioning, where it is not practical to use automate for all the callback functionality.
/cc @gmcculloug low hanging fruit

@mkanoor Please review.

:+1:


Increase max retries to 1000

100 is not enough for some slow Sonatype repository.
This PR should fix #363

Even increasing the retry limit here will not handle CircleBreakerOpenException, whose default handling is fail-fast. We need to add a further retry around making an  HTTP request.

I don't understand. Shouldn't an exception be thrown from the HTTP client only after that number of retries?

Do you suggest that withMaxRetry is actually a no-op?

The default retry logic of the HTTP client prefers fail-fast behavior and does not perform a retry when the circuit breaker opens when Sonatype is totally down https://github.com/wvlet/airframe/blob/master/airframe-http/src/main/scala/wvlet/airframe/http/HttpClientException.scala#L162

withMaxRetry is actually a no-op
Yes, when the circuit breaker is open. For other types of errors this setting is effective.

Would it help if the HTTP client instead consider 500 errors as a temporary error and do not treat the host as totally down?

Rather, disabling the circuit breaker at clientConfig would work like this:
      // Disables the circuit breaker, because Sonatype can be down for a long time https://github.com/xerial/sbt-sonatype/issues/363
      .noCircuitBreaker

1000 might be a bit long (~ 8 hours) but Sonatype is still down today, so it might be ok https://status.maven.org/


[SPARK-40773][SQL] Refactor checkCorrelationsInSubquery

What changes were proposed in this pull request?

This PR refactors checkCorrelationsInSubquery in CheckAnalysis to use recursion instead of foreachUp.
Why are the changes needed?

Currently, the logic in checkCorrelationsInSubquery is inefficient and difficult to understand. It uses foreachUp to traverse the subquery plan tree, and traverses down an entire subtree of a plan node to check whether it contains any outer references. We can use recursion instead to traverse the plan tree only once to improve the performance and readability.
Does this PR introduce any user-facing change?

No
How was this patch tested?

Existing unit tests.

cc @cloud-fan

thanks, merging to master!


criterion: 2.4.1 -> 2.4.2, add maintainer, migrate by-name, add passthru tests

Description of changes

Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[X] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/3875

Result of nixpkgs-review pr 308951 run on aarch64-darwin 1

2 packages built:

criterion
criterion.dev

Result of nixpkgs-review pr 308951 run on x86_64-linux 1

3 packages built:

criterion
criterion.dev
siril

Result of nixpkgs-review pr 308951 run on x86_64-linux 1

3 packages built:

criterion
criterion.dev
siril

@AndersonTorres done :smile:

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/1629


In Merge.js, enable in the Merged API the ability to get the merged chunks

So I am trying to get the merged chunk in Merge.js split view for selecting purposes.  Currently, I can only get the left and right chunks.  Going through the code, I notice that we track the merged chucks in a private variable.  Is there a way to make this variable public via an interface function?

What exactly do you mean by 'merged chunks'?

Example: https://codemirror.net/demo/merge.html. In the paragraph beneath the example, click "three-way"
Now, open up a developer console with an immediate window/console/watch variables in your browser of choice and inspect the variable "dv".
"dv" has access to the following variables

dv.leftChunks (returns an array of 8 Objects)
dv.rightChunks (returns an array 3 Objects)

These objects represent the highlight "chunks" of modified text in the corresponding view (Example: the 8 objects of leftChunks represent the 8 highlighted blocks in the left view).  What I am looking for is a way to access the merged chunks in the center pane of the 3-way view (there is no way to access them).
Does that make sense?  I read through the source code of Merge.js and see no way of accessing the center highlighted blocks (aka chunks).

These merged chunks are visually there, but there are not really a concept in the merge addon's code. You could write your own logic to overlay the left and right chunks on top of each other, I guess.


Dropdown list items rendered multiple times when bulk adding options

It seems that render() is called every time a choice is added and render removes all innerHTML DOM elements from the dropdown and recreates the list leading to poor performance with large sets, O(n2).
setChoices calls _addChoice in a loop which in turn calls store.dispatch and store calls render on every state change.
The pattern seems to look like: render 1 - clear - render 1, render 2 - clear render 1, render 2, render3

Hey!
There is certainly room for improvement here. There is also going to need to be a loop within the render() method but I'm wondering whether it would be better to dispatch to the store with a bulk of data...

Hi! Idk, the bulk dispatch seems to make the most sense to me, ADD_CHOICES and REPLACE_ITEMS or something. I like the idea of only making one call to update the state per user/api interaction. If render is only called once there's less reliance on the state checks within the function.

Using https://github.com/google/incremental-dom or https://github.com/patrick-steele-idem/morphdom Mmight be a nice way to improve rendering performance (less GC), particularly while searching since many choice results will remain the same as more search characters are added?

How to import if you use webpack, its saying,
import  {Choices} from 'choices.js/assets/scripts/src/choices';
error:
Uncaught TypeError: Cannot assign to read only property 'exports' of object '#<Object>'
But if i use this way nothing is happening,
window.Choices = require('choices.js');

new Choices('#portTag',{

});

error look like,
Uncaught TypeError: Cannot read property 'type' of null

@MicroDreamIT If you have installed Choices via npm, you just need to do this:
import Choices from 'choices.js'

please add the pinned label or whatever makes @stale shut up.

@flying-sheep can you provide a jsfiddle demonstrating Choices affecting load time please? Would be interested to know the size of your dataset too.

I've got a functioning work-in-progress of Choices using morphdom. Would be great if somebody could take it for a spin. The branch name is integrate-morphdom

looking good! sadly I have not time to compare times, so how much did they improve?
if you e.g. provide 10000 choices to select from, the JS needed a few seconds to execute before. is that zippy now?

@stale no


Meeting Link Expiration Inefective

There needs to be a way for our admin to expire specific meetings by link or by the Conference ID #. We have students, that are minors, accessing and sharing old meeting join links. In some cases, the organizer sets the meeting options to not allow anyone to bypass the lobby, but these options revert to allowing all to bypass on the following day. I suspect this has something to do with setting up the meeting as reoccurring. (I'm sorry if this is not the correct place for this, but I am struggling with how to find a solution to this problem.)

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

ID: ee6d4c1c-6f16-b1a4-c4a9-be4e6d1f95b7
Version Independent ID: 50ad835d-ed7f-18c6-34ba-322c1574f9c2
Content: Meeting policies and meeting expiration in Microsoft Teams - Microsoft Teams
Content Source: Teams/meeting-expiration.md
Service: msteams
GitHub Login: @KarliStites
Microsoft Alias: kastites

Hello @mclaughlin716
Thanks for your feedback, here, we are limited to changes/improvements against the documentation, for your inquiry, the Feedback Portal is a resource that can be used to submit your idea and it can reach the product group, for example, I found a very similar suggestion here
Hope this helps you,

@scanum


The Eye

The eye is a platform deicatded to archving any and all kinds of data.
They say they have 140 Tb in total in assorted formats and a good fraction seems to be in text format.
https://the-eye.eu/public/
unfortuatly due to the fact that all of their size estimates seem to be "pending update" it is dificult to give exact estimats on how much of this is textual

I believe the team has contacted folks at the Eye. The Bibliotik component is from them. Do they have other big text datasets that you know of?

Indeed, we are in contact with them and have gotten datasets from them. Long term we are working on hosting a copy of all of the data in the Pile on their systems.
Are there any specific datasets you recommend?

Not Really
On Tue, Oct 13, 2020 at 4:05 PM Stella Biderman<EMAIL_ADDRESS>wrote:

Indeed, we are in contact with them and have gotten datasets from them.
Long term we are working on hosting a copy of all of the data in the Pile
on their systems.
Are there any specific datasets you recommend?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/EleutherAI/The-Pile/issues/51#issuecomment-707803731,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APLK3R66YGSL2DUQVAYFMBDSKRUCNANCNFSM4SLEDAPQ
.

Okay. I’m going to tentatively close this issue, but feel free to suggest additional data sets in the future either on GitHub or on Discord.


Update Memcached Chart to increase image portability

This commit updates the Memcached chart to allow use with images other than Stackanetes, define ImagePullPolicy, and provide config params in the values.yaml.

This change is

Review status: 0 of 2 files reviewed at latest revision, 1 unresolved discussion.

memcached/templates/deployment.yaml, line 18 at r1 (raw file):

command: ["sh", "-xec"]

This method requires a shell to be present in the container - meaning that minimal ACI images are not supported, but they are still pretty rare.
I used this method as it's possible that the memcached binary may be in /bin, /usr/bin, or /usr/local/bin depending on the Distro or install method used by the container maintainer.

Comments from Reviewable

Review status: 0 of 2 files reviewed at latest revision, 1 unresolved discussion.

memcached/templates/deployment.yaml, line 18 at r1 (raw file):

command: ["sh", "-xec"]

This method requires a shell to be present in the container - meaning that minimal ACI images are not supported, but they are still pretty rare.
I used this method as it's possible that the memcached binary may be in /bin, /usr/bin, or /usr/local/bin depending on the Distro or install method used by the container maintainer.

Comments from Reviewable

Reviewed 2 of 2 files at r1.
Review status: all files reviewed at latest revision, 1 unresolved discussion.

memcached/templates/deployment.yaml, line 18 at r1 (raw file):
Previously, intlabs (Pete Birley) wrote…

command: ["sh", "-xec"]

This method requires a shell to be present in the container - meaning that minimal ACI images are not supported, but they are still pretty rare.
I used this method as it's possible that the memcached binary may be in /bin, /usr/bin, or /usr/local/bin depending on the Distro or install method used by the container maintainer.

I think this is a reasonable sacrifice to allow the most flexibility.

Comments from Reviewable


Service Fabric Stateful Service in Azure - RAM usage keeps growing. Possible memory leak

I am running a Service Fabric application on a cluster in Azure. The cluster has two scale sets:

4x B2ms nodes where a stateful service type is placed with Placement Constraints (Primary Scale Set)
2x F1 nodes where a stateless service type is placed.

There are two types of services in the application

WebAPI - stateless service used for receiving statuses from a system via HTTP and sending them to the StatusConsumer.
StatusConsumer - stateful service which processes the statuses and keeps the last one. Service instance is created for each system. It communicates via RemotingV2.

For my tests I am using Application Insights and Service Fabric Analytics to track performance. I am observing the following parameters:
Metrics for the stateful scale set: Percentage CPU, Disk Read/Write operations/sec
Applicaiton Insights: Server Response Time - which corresponds to the execution time of the method that receives the statuses in the stateful StatusConsumer.
Service Fabric Analytics: Performance Counters with log analytics agent on the stateful nodes - used to observe the RAM usage of the nodes.
Each simulated system sends its status every 30 seconds.
In the beginning of the test the RAM usage is around 40% on each node, Avg Server Response time is around 15ms, CPU usage is around 10%, and Read/Write operations are under 100/s.
Immediately after the start of the test the RAM usage starts slowly building up, but there is no difference in the other observed metrics.
After about an hour of simulating 1000 systems the RAM usage is around 90-95% and problems start to show in the other metrics - Avg Server response time peaks with values around 5-10 seconds and Disk Read/Write operations reach around 500/sec.
This continues for 1-3 minutes then the RAM usage drops and everything goes back to normal.

On the images you can see that the RAM peak corresponds to the server response time peak. In the end of the RAM graph the usage is flat in order to show what is the behaviour without simulating any systems.
The number of the systems simulated only reduces or increases the time it takes for the RAM to reach critical levels - in one of the tests 200 systems were simulated and the RAM usage rise was slower.
As for the code: In the first tests the code was more complicated, but to find the cause of the problem I started removing functionality. Still there was no improvement. The only time the RAM usage did not rise was when I commented all the code and in the body of the method that receives the status were only a try/catch block and a return. Currently the code in the StatusConsumer is this:
public async Task PostStatus(SystemStatusInfo status){
	try{
		Stopwatch stopWatch = new Stopwatch();

		IReliableDictionary<string, SystemStatusInfo> statusDictionary = await this.StateManager.GetOrAddAsync<IReliableDictionary<string, SystemStatusInfo>>("status");

		stopWatch.Start();

		using (ITransaction tx = this.StateManager.CreateTransaction())
		{
			await statusDictionary.AddOrUpdateAsync(tx,"lastConsumedStatus",(key) => { return status; },(key, oldvalue) => status);
			await tx.CommitAsync();
		}

		stopWatch.Stop();

		if (stopWatch.ElapsedMilliseconds / 1000 > 4) //seconds
		{
			Telemetry.TrackTrace($"Queue Status Duration: { stopWatch.ElapsedMilliseconds / 1000 } for {status.SystemId}", SeverityLevel.Critical);
		}

	}
	catch (Exception e)	{Telemetry.TrackException(e);}
}

After connecting to the nodes with remote desktop, in Task Manager I can see that when the RAM usage is around 85% the "Memory" of the SystemStatusConsumer process, which "holds" the microservice's instances, is not more than 600 MB. It is the highest consumption but it is still not that high - the node is with 8 GB of RAM. However I don`t know if this is useful information in this case.
How can I diagnose and/or fix this?

Closing this as it looks more like questions at this point vs an issue/bug. Please post your questions to https://docs.microsoft.com/en-us/answers/topics/azure-service-fabric.html


use consistent syntax for examples

Hey :wave:!
First of all, your tutorial is awesome. I'd liked to read the tutorial and will share and promote it in the future.
This Pull Request is about to improve the experience for beginner and professionals. Of course there are different types of syntax guidelines. But somehow the missing space before the { and } just feel wrong.
Just to be consistent to common patterns and increase the readability.
:sunflower: Please be nice. Thank you for your work. :sparkles:
:book: http://sass-lang.com/guide
:book: https://houndci.com/configuration#scss
:book: http://www.w3schools.com/css/css_syntax.asp

@jgthms - any kind of feedback?

I don't think this change is necessary. It's not an innocent one actually: the space has a meaning in a CSS selector, and also in some CSS properties. I don't want to add them just because it might "feel wrong".
Thanks for your suggestion anyway.


referenced in BY but not declared in USING in group by off of backlink

EdgeDB Version: 3.2 and 3.3
EdgeDB CLI Version: 3.4.0+4d95a2c
OS Version: macos 13

Steps to Reproduce:

Throws error (with helpful guidance) UnsupportedFeatureError: may not group by a field named id Hint: try 'using id_ := .id'

group(
  select AuditLog.user
) by (.id);

Aliasing as recommended works as expected,

group(
   select AuditLog.user {id_:=.id}
) by (.id_);

Add a backlink in the path,

group(
  select ActivitySource.<context[is AuditLog].user {id_:=.id}
) by (id_);

error: InvalidReferenceError: variable 'id_' referenced in BY but not declared in USING
  ┌─ <query>:3:7
  │
3 │ ) by (id_);
  │       ^^^ error

Schema:
type User {}
type AuditLog {
    user: User;
    context: ActivitySource;
}
type ActivitySource {}

I think this one is actually OK. You just need a leading dot on your by:
group(
  select ActivitySource.<context[is AuditLog].user {id_:=.id}
) by (.id_);

or if you define your alias in using instead, you can forego the leading dot:
group(
  select ActivitySource.<context[is AuditLog].user
) using id_ := .id by (.id_);

Feel free to re-open if I've missed something here.

Ahhhhh I misinterpreted this as somehow an error coming out of postgres from a corrupt query. EdgeDB also has the concept of using.


Next.js dynamic routes under a rewrite fail to regenerate via ISR

Before opening, please confirm:

[X] I have checked to see if my question is addressed in the FAQ.
[X] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.

App Id
dgpf8z9zdyr4o
Region
ca-central-1
Amplify Console feature
Build settings
Describe the bug
Next.js dynamic routes that are accessed via a rewrite fail to trigger ISR regeneration. In my app I have multiple dynamic routes that aren't accessed via a rewrite and they trigger ISR successfully. I also have non-dynamic routes that are not accessed via a rewrite and they also trigger ISR normally. Only dynamic routes that are accessed by a rewrite fail to trigger ISR, so content stays stale on those routes until a new build is ran.
Expected behavior
Dynamic routes should trigger ISR if accessed via a rewrite. This has been tested to work properly locally.
Reproduction steps

Create a Next.js app
Add a dynamic route (e.g. /test/[id].js) with getStaticPaths/getStaticProps calls configured for ISR
Add a rewrite (e.g. /rewrite/:id -> /test/:id)
Try to trigger ISR on the dynamic route

Build Settings
version: 1
backend:
  phases:
    build:
      commands:
        - '# Execute Amplify CLI with the helper script'
        - amplifyPush --simple
frontend:
  phases:
    preBuild:
      commands:
        - nvm use
        - npm ci
    build:
      commands:
        - npm run build
  artifacts:
    baseDirectory: .next
    files:
      - '**/*'
  cache:
    paths:
      - node_modules/**/*

Additional information
No response

Hi @julesrenaud, this is an interesting observation, thank you for reporting it. We will look into it.


Can't change current library from old library to new

Hi, recently i changed my library on the ibm i server, so now i don't have access to that old library, lets call it LIBOLD. The new library is LIBNEW.
Here is my problem. I can't seem to find a way to remove LIBOLD form the vscode for i lib list and also can't set LIBNEW as the new current lib. Which means, everytime i try to compile something, it tries to use LIBOLD, which i have no access anymore.
I would aprecciate any advice. Thanks
This is LIBOLD

This is LIBNEW

This is the error when trying to compile

It looks like it is in limbo

This doesn't work when you right click on a library?

This doesn't work when you right click on a library?

I can remove the new library but not the old one (which is set a current library)

You may have discovered something the developers didn't think of, so it may be a bug.
You did try right clicking and doing this?

Does doing CHGCURLIB on the green screen and then reconnecting make any difference?
If you are really stuck, you could:

Create a new connection.

Or if you are comfortable with it, you could manually edit setting.json: But be careful...

Right click + change current library gives this error

Changing the curlib on the green screen does nothing
I don't wanna try editing the settings right now cuz i have other connections and i don't know if that is gonna mess something else up and give me more headache

Please share the contents of your Code for IBM i output channel after the error appears.

Hi @kraudy,
this is indeed a bug. If your settings' current library doesn't exist anymore, then it cannot be removed.
I'll work on a fix, but in the meantime, you should follow @SJLennon advice and replace your connection's currentLibrary by something else (like QPGL). This will get you going while the fix is being worked on and you have no risk of breaking anything.
Alternatively, you can use the Copy... action to duplicate the faulty connection and make a copy of everything but the library list.

Here is the output @worksofliam

@kraudy this problem has been fixed in v2.10.1 that was released today.
After you've updated the extension, you'll be able to change you current library.

Thanks @sebjulliand


Proposal: Switch to Conventional Commits

We've been looking at a few ideas and tools around shoring up our commits, making breaking changes more clear, and generally making our release, versioning, and documentation process more automatable.
Introducing: https://www.conventionalcommits.org/en/v1.0.0/
We can use that with tools like commitlint and other Conventional Changelog tooling. We'll still want to write separate notes, but this could help make things a LOT more clear.
I'd like to get the core team's thoughts on this (@alenkacz, @kensipe, @zen-dog especially), and if so, I will PR updating the CONTRIBUTING process and add commitlint as a check and a hook.
Another big benefit, beyond the automation, is it will all help get us into the same mindset with writing good commits.

love the idea of improving changelog from commits ❤ . I see potentially two paths here:

conventional commits
something close to what k8s does (have changelog block in PR that is mandatory plus maybe use labels for categorization). Then generate changelog from closed PRs not commits.

The advantage I see in second option is that you can edit PR after it was merged and change labels/changelog message. You cannot do that with commits without force push.

I do like that second option too! Could we combine these two things, and have that block used for our longer form release notes?

My concern is that this convention requires fixes/refactors and features to be in different commits, but our GitHub configuration only allows squash merges. This means that if I encounter a need to refactor or cleanup something while working on a feature, I'd need to create a separate base PR and wait for it to be merged before even raising the feature PR. I'm afraid that with our 24h delay rule it would either:

introduce a lot of latency compared to current situation where both changes could be reviewed in parallel, or
discourage applying the boyscout rule (without prejudice to girl scouts)

I can see the following ways around this:

allow merge commits, or
raise the latter PR against the branch of the former one, and be careful not to merge it before changing back the target against master

Also, I would expect that a tested way to validate commit correctness on developer workstation is provided before this rule is enforced.


Hello I have problem with using multiple java file

when I use multiple java file by Import keyword or library error say that file not found
can you guide me how to use it for multiple import comand.

If you mean that you are getting errors when trying to convert java files that contain imports to non-standard libraries, it's normal. Java2Smali needs to have all the required sources inside the directory that you are converting.
So if you use a third party library, you should include it in the sources manually.


Added example using Tailwind, CSR (only) and Trunk

The title is self-explanatory.
I wasn't able to run the original Tailwind example with Trunk, so here is a simple one that should work out of the box.

Nice, thanks! I added this to the root Makefile.toml so it will be checked during CI, looks good otherwise. Much appreciated.


[go_router] If you use Scaffold.showBottomSheet, page transitions will no longer be possible.

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce

Create a main page and subpage, and implement the process to display the bottom sheet on the subpage.
With the bottom sheet displayed on the subpage, press the back button on the AppBar to close the bottom sheet.
If you press the AppBar back button again, an exception will occur and you will not be able to return to the main page.

Expected results
I can return to the main page by pressing the AppBar's back button.
Actual results
The following exception occurs.

E/flutter (29106): [ERROR:flutter/runtime/dart_vm_initializer.cc(41)] Unhandled Exception: Bad state: No element
E/flutter (29106): #0      List.last (dart:core-patch/growable_array.dart:348:5)
E/flutter (29106): #1      _PagePopContext.onPopPage (package:go_router/src/builder.dart:596:57)
E/flutter (29106): #2      NavigatorState.pop (package:flutter/src/widgets/navigator.dart:5045:28)
E/flutter (29106): #3      NavigatorState.maybePop (package:flutter/src/widgets/navigator.dart:5005:9)
E/flutter (29106):

Code sample
Code sample
import 'package:flutter/material.dart';
import 'package:go_router/go_router.dart';

/// This sample app shows an app with two screens.
///
/// The first route '/' is mapped to [HomeScreen], and the second route
/// '/details' is mapped to [DetailsScreen].
///
/// The buttons use context.go() to navigate to each destination. On mobile
/// devices, each destination is deep-linkable and on the web, can be navigated
/// to using the address bar.
void main() => runApp(const MyApp());

/// The route configuration.
final GoRouter _router = GoRouter(
  routes: <RouteBase>[
    GoRoute(
      path: '/',
      builder: (BuildContext context, GoRouterState state) {
        return const HomeScreen();
      },
      routes: <RouteBase>[
        GoRoute(
          path: 'details',
          builder: (BuildContext context, GoRouterState state) {
            return const DetailsScreen();
          },
        ),
      ],
    ),
  ],
);

/// The main app.
class MyApp extends StatelessWidget {
  /// Constructs a [MyApp]
  const MyApp({super.key});

  @override
  Widget build(BuildContext context) {
    return MaterialApp.router(
      routerConfig: _router,
    );
  }
}

/// The home screen
class HomeScreen extends StatelessWidget {
  /// Constructs a [HomeScreen]
  const HomeScreen({super.key});

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(title: const Text('Home Screen')),
      body: Center(
        child: ElevatedButton(
          onPressed: () => context.go('/details'),
          child: const Text('Go to the Details screen'),
        ),
      ),
    );
  }
}

/// The details screen
class DetailsScreen extends StatelessWidget {
  /// Constructs a [DetailsScreen]
  const DetailsScreen({super.key});

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(title: const Text('Details Screen')),
      body: const DetailInnerScreen(),
    );
  }
}

class DetailInnerScreen extends StatelessWidget {
  /// Constructs a [DetailInnerScreen]
  const DetailInnerScreen({super.key});

  @override
  Widget build(BuildContext context) {
    return Center(
      child: Column(
        mainAxisAlignment: MainAxisAlignment.center,
        children: [
          ElevatedButton(
            onPressed: () => context.go('/'),
            child: const Text('Go back to the Home screen'),
          ),
          ElevatedButton(
            onPressed: () {
              showBottomSheet(
                context: context,
                backgroundColor: Colors.orange,
                builder: (context) {
                  return SizedBox(
                    height: 300,
                    width: double.infinity,
                    child: TextButton.icon(
                      onPressed: () {
                        Navigator.pop(context);
                      },
                      icon: const Icon(Icons.close),
                      label: const Text('Close'),
                    ),
                  );
                },
              );
            },
            child: const Text('Show the Bottom sheet'),
          ),
        ],
      ),
    );
  }
}

Screenshots or Video
No response
Logs
No response
Flutter Doctor output
Doctor output
[✓] Flutter (Channel stable, 3.13.3, on macOS 13.5.2 22G91 darwin-arm64, locale ja-JP)
    • Flutter version 3.13.3 on channel stable at /Users/user/.asdf/installs/flutter/3.13.3-stable
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision<PHONE_NUMBER> (3 days ago), 2023-09-06 14:32:31 -0700
    • Engine revision b8d35810e9
    • Dart version 3.1.1
    • DevTools version 2.25.0

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.1)
    • Android SDK at /Users/user/Library/Android/sdk
    • Platform android-34, build-tools 33.0.1
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b829.9-10027231)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.3.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14E300c
    • CocoaPods version 1.12.1

[✓] Android Studio (version 2022.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b829.9-10027231)

[✓] VS Code (version 1.82.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension can be installed from:
      🔨 https://marketplace.visualstudio.com/items?itemName=Dart-Code.flutter

[✓] Connected device (2 available)
    • sdk gphone64 arm64 (mobile) • emulator-5554 • android-arm64 • Android 13 (API 33) (emulator)
    • macOS (desktop)             • macos         • darwin-arm64  • macOS 13.5.2 22G91 darwin-arm64

[✓] Network resources
    • All expected network resources are available.

• No issues found!

This issue has been occurring since go_router 10.1.1.
For your reference:
https://github.com/flutter/packages/blob/main/packages/go_router/lib/src/builder.dart#L596-L597
This can be avoided by changing lines 596 to 597 of go_router/lib/src/builder.dart to the following.
  bool onPopPage(Route<dynamic> route, dynamic result) {
    final Page<Object?> page = route.settings as Page<Object?>;

    final RouteMatch match = _routeMatchesLookUp[page]!.last;

    // If a popup is opened using something like Scaffold.showBottomSheet,
    // it will be registered in LocalHistoryRoute._localHistory,
    // so it should be deleted from _routeMatchesLookUp only if
    // LocalHistoryRoute.willHandlePopInternally is false.
    if (route is LocalHistoryRoute && route.willHandlePopInternally == false) {
      _routeMatchesLookUp[page]!.removeLast();
    }

    return onPopPageWithRouteMatch(route, result, match);
  }

(Diff)
    bool onPopPage(Route<dynamic> route, dynamic result) {
      final Page<Object?> page = route.settings as Page<Object?>;

      final RouteMatch match = _routeMatchesLookUp[page]!.last;
-     _routeMatchesLookUp[page]!.removeLast();
+
+     // If a popup is opened using something like Scaffold.showBottomSheet,
+     // it will be registered in LocalHistoryRoute._localHistory,
+     // so it should be deleted from _routeMatchesLookUp only if
+     // LocalHistoryRoute.willHandlePopInternally is false.
+     if (route is LocalHistoryRoute && route.willHandlePopInternally == false) {
+       _routeMatchesLookUp[page]!.removeLast();
+     }

      return onPopPageWithRouteMatch(route, result, match);
    }

I'm posting this as an issue because I can't accurately judge whether this is the correct solution.

Thanks for the report @lynrin
I ran the code sample using latest master and package version but unable to replicate the reported error:
https://github.com/flutter/flutter/assets/67046386/dbad472a-c077-4d13-a2ae-913804207e22
The console log didn't throw any error. Can you use the latest package version and try again ?

Thanks for checking @darshankawar.
I'm using go_router version 10.1.2.
I also shot a video here.
Steps are

Click the "Go to the Details screen" button.
Click the "Show the bottom sheet" button.
Click the back button on the AppBar. (The bottom sheet closes.)
Click the back button on the AppBar. (An exception occurred.)

I get an exception at "4. Click the back button on the AppBar."
https://github.com/flutter/flutter/assets/12870525/4e06bab1-903a-438f-84ce-dba51b046223

Thanks for the update. Yes, I now see the same error as reported.

 stable, master flutter doctor -v
[!] Flutter (Channel stable, 3.13.2, on macOS 12.2.1 21D62 darwin-x64, locale
    en-GB)
    • Flutter version 3.13.2 on channel stable at
      /Users/dhs/documents/fluttersdk/flutter
    ! Warning: `flutter` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/flutter, which is not inside
      your current Flutter SDK checkout at
      /Users/dhs/documents/fluttersdk/flutter. Consider adding
      /Users/dhs/documents/fluttersdk/flutter/bin to the front of your path.
    ! Warning: `dart` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/dart, which is not inside your
      current Flutter SDK checkout at /Users/dhs/documents/fluttersdk/flutter.
      Consider adding /Users/dhs/documents/fluttersdk/flutter/bin to the front
      of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision ff5b5b5fa6 (7 days ago), 2023-08-24 08:12:28 -0500
    • Engine revision b20183e040
    • Dart version 3.1.0
    • DevTools version 2.25.0
    • If those were intentional, you can disregard the above warnings; however
      it is recommended to use "git" directly to perform update checks and
      upgrades.

[!] Xcode - develop for iOS and macOS (Xcode 12.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    ! Flutter recommends a minimum Xcode version of 13.
      Download the latest version or update via the Mac App Store.
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] VS Code (version 1.62.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.21.0

[✓] Connected device (5 available)
    • SM G975F (mobile)       • RZ8M802WY0X • android-arm64   • Android 11 (API 30)
    • Darshan's iphone (mobile)  • 21150b119064aecc249dfcfe05e259197461ce23 •
      ios            • iOS 14.4.1 18D61
    • iPhone 12 Pro Max (mobile) • A5473606-0213-4FD8-BA16-553433949729     •
      ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
    • macOS (desktop)            • macos                                    •
      darwin-x64     • Mac OS X 10.15.4 19E2269 darwin-x64
    • Chrome (web)               • chrome                                   •
      web-javascript • Google Chrome 98.0.4758.80

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.

[!] Flutter (Channel master, 3.14.0-14.0.pre.212, on macOS 12.2.1 21D62
    darwin-x64, locale en-GB)
    • Flutter version 3.14.0-14.0.pre.212 on channel master at
      /Users/dhs/documents/fluttersdk/flutter
    ! Warning: `flutter` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/flutter, which is not inside
      your current Flutter SDK checkout at
      /Users/dhs/documents/fluttersdk/flutter. Consider adding
      /Users/dhs/documents/fluttersdk/flutter/bin to the front of your path.
    ! Warning: `dart` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/dart, which is not inside your
      current Flutter SDK checkout at /Users/dhs/documents/fluttersdk/flutter.
      Consider adding /Users/dhs/documents/fluttersdk/flutter/bin to the front
      of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision ffe1ebd207 (67 minutes ago), 2023-09-11 03:46:15 -0400
    • Engine revision 1aa11b6448
    • Dart version 3.2.0 (build 3.2.0-134.0.dev)
    • DevTools version 2.27.0
    • If those were intentional, you can disregard the above warnings; however
      it is recommended to use "git" directly to perform update checks and
      upgrades.

[!] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Users/dhs/Library/Android/sdk
    ✗ cmdline-tools component is missing
      Run `path/to/sdkmanager --install "cmdline-tools;latest"`
      See https://developer.android.com/studio/command-line for more details.
    ✗ Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/macos#android-setup for
      more details.

[✓] Xcode - develop for iOS and macOS (Xcode 13.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 13C100
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] IntelliJ IDEA Ultimate Edition (version 2021.3.2)
    • IntelliJ at /Applications/IntelliJ IDEA.app
    • Flutter plugin version 65.1.4
    • Dart plugin version 213.7228

[✓] VS Code (version 1.62.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.29.0

[✓] Connected device (3 available)
    • Darshan's iphone (mobile) • 21150b119064aecc249dfcfe05e259197461ce23 • ios
      • iOS 15.3.1 19D52
    • macOS (desktop)           • macos                                    •
      darwin-x64     • macOS 12.2.1 21D62 darwin-x64
    • Chrome (web)              • chrome                                   •
      web-javascript • Google Chrome 109.0.5414.119

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.

[!] Xcode - develop for iOS and macOS (Xcode 12.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    ! Flutter recommends a minimum Xcode version of 13.
      Download the latest version or update via the Mac App Store.
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] VS Code (version 1.62.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.21.0

[✓] Connected device (5 available)
    • SM G975F (mobile)       • RZ8M802WY0X • android-arm64   • Android 11 (API 30)
    • Darshan's iphone (mobile)  • 21150b119064aecc249dfcfe05e259197461ce23 •
      ios            • iOS 14.4.1 18D61
    • iPhone 12 Pro Max (mobile) • A5473606-0213-4FD8-BA16-553433949729     •
      ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
    • macOS (desktop)            • macos                                    •
      darwin-x64     • Mac OS X 10.15.4 19E2269 darwin-x64
    • Chrome (web)               • chrome                                   •
      web-javascript • Google Chrome 98.0.4758.80

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.

Thank you very much.
I have also sent the corrected code written in the comments as a Pull Request(flutter/packages#4882).

I didn't see error when pressing back button, only when clicking Go back to home screen.

Thank you @chunhtai for checking.
Could you please post the steps and video of the operation you performed?

@lynrin Was also facing a similar issue in v10.1.2, upgraded the package to v12.0.0 and the issue seems to be resolved, however, you might want to find out from which version onwards was this issue fixed

@darshankawar @chunhtai @keithcwk ,
It seems that this problem has stopped occurring since the release of v11.0.0.
https://github.com/flutter/packages/blob/55b7293656e01eabc96b3ef8d9c641b832e803ff/packages/go_router/lib/src/builder.dart#L596-L599
I will close this issue.


Support batches with > 10 messages

Is your feature request related to a problem? Please describe.
Yes. For a while now, it's possible to receive batches with more than 10 entries in Lambda (ref). However, the calls from SendMessageBatch and DeleteMessageBatch in AWS PowerTools are made with the entire set of messages that are being sent to the DLQ or deleted from the SQS.
Those APIs, however, are limited to 10 entries each, which results in exceptions like TooManyEntriesInBatchRequestException: Maximum number of entries per request are 10. You have sent 226.
Describe the solution you'd like
Batch API calls should be made in chunks of 10 to satisfy the SDK restrictions.
Describe alternatives you've considered
So far, reducing the batch size to 10, which is not ideal for my solution due to the amount of concurrent lambdas.

Thanks for opening the issue. I believe we made a similar fix in python powertools as well. I will look into fixing it this week.

We’ve just added support yesterday for the new native partial response that
is a much better alternative now
https://awslabs.github.io/aws-lambda-powertools-python/latest/utilities/batch/
I’ve documented the mechanics, events, responses and I posted the details
on the RFC too (on roadmap repo)
On Tue, 21 Dec 2021 at 16:07, Pankaj Agrawal @.***>
wrote:

Thanks for opening the issue. I believe we made a similar fix in python
powertools as well. I will look into fixing it this week.
—
Reply to this email directly, view it on GitHub
https://github.com/awslabs/aws-lambda-powertools-java/issues/663#issuecomment-998853641,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAZPQBBVKTSPRUFRF3VOYM3USCJZLANCNFSM5KN6IYUQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Yeah. This one is for Java which does not have the new batch implementation yet :p .. but this is a bug/limited functionality in older util

Yeah, for the older one we still have to fix that too - we have a PR at an
okay stage but we’re fixing it for the next release.
We’ll add a warning for them to move
To the new one as their cost and latency will decrease too
On Tue, 21 Dec 2021 at 16:16, Pankaj Agrawal @.***>
wrote:

Yeah. This one is for Java which does not have the new batch
implementation yet :p .. but this is a bug/limited functionality in older
util
—
Reply to this email directly, view it on GitHub
https://github.com/awslabs/aws-lambda-powertools-java/issues/663#issuecomment-998861440,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAZPQBFBFN6R5WLKV3YPHKLUSCK4RANCNFSM5KN6IYUQ
.
You are receiving this because you commented.Message ID:
@.***>

Yeah, for the older one we still have to fix that too - we have a PR at an okay stage but we’re fixing it for the next release. We’ll add a warning for them to move To the new one as their cost and latency will decrease too
…
On Tue, 21 Dec 2021 at 16:16, Pankaj Agrawal @.> wrote: Yeah. This one is for Java which does not have the new batch implementation yet :p .. but this is a bug/limited functionality in older util — Reply to this email directly, view it on GitHub <#663 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAZPQBFBFN6R5WLKV3YPHKLUSCK4RANCNFSM5KN6IYUQ . You are receiving this because you commented.Message ID: @.>

Ah ok.. so thats not merged yet in python as well. Good to know. I will look if I can get this fix out first and then plan for newer version post that.

Related python issue https://github.com/awslabs/aws-lambda-powertools-python/issues/824

Available in https://github.com/awslabs/aws-lambda-powertools-java/releases/tag/v1.10.0


Foundation 6 Table Scroll

I loved this feature. I implemented it. It worked. But later stopped working. Even if i delete my own classes it doesn't work.
Here is my demo page. There is a big table there.
http://fantasycore.com/clients/hosting/app/seo-vps-eu.html
Edit: If i add max-width:100% to container column, it works.
Also if i add small-12 medium-12 large-12 it works. On F5 if i add large-12 it was ok on small screen. Grid works different then old or this is a flex grid thing.
I will close the issue after some info. There is not good detailed information about this on docs.

I use custom grid exported css. I don't use scss. Sorry i am in love with stylus. :)

When did it stop working? When you updated Foundation versions?

I did not update Foundation. I am aware it is not a good idea to update middle of a project.

Ah okay. In that case, we need to know how you changed your HTML or CSS to make it stop working.

I would say, i did not change a thing but i am not sure. When it was working, i stopped playing on that part.
As i said it works if i add small-12 to parent column. The problem is column. Large-12 doesn't have max-width or it doesn't do the magic.

In general the grid should work exactly the same as it did in 5. If it doesn't, consider it a bug.

I will close the issue after some info. There is not good detailed information about this on docs.

What do you need clarity on?

I don't know. Better close this one now. I will make a demo later and put it here.

I'm having same issue.  can't get .scroll or .table-scroll to work when using flex grid. I've turned off all of my own styles.

@asecondwill see the workaround in https://github.com/zurb/foundation-sites/issues/9658


Same AKS VNet in multiple Subscriptions

I have a doubt with respect to AKS networking in Azure.
I have multiple subscriptions and one of then is using network range 10.x.x.x - 10.30.x.x
As Azure recommendation is keep unique (diff from on-premise and already in use) vnet range. As all clusters will be in different subscriptions, so there should not be any issue.
Just wanna confirm, Can I create multiple aks with range 10.0.x.x in different subscriptions?

@techsaau Thank you for your feedback. Since this channel is more for driving improvements towards MS Docs, could you provide us with a URL of the documentation that your feedback is most applicable to? That way, we're able to locate the right team that's better suited to address your feedback.

I was following below URL for networking setup:
https://docs.microsoft.com/bs-latn-ba/azure/aks/configure-azure-cni

Thanks for providing additional info, @techsaau! I have assigned an engineer who can help with the next steps.

any update guys?

@techsaau You can do it as long as you are not planning to do Vnet peering or create an internal load balancer in the Vnet. As long as you are accessing it with Public IP, it will be fine but accessing via internal load balancer would be an issue because of overlapping IPs.

We will now close this issue. If there are further questions regarding this matter, please tag me in a comment. I will reopen it and we will gladly continue the discussion.

Thanks @Karishma-Tiwari-MSFT for your response.


Hover fallback logic

If there is no precise hover, we'll fall back to a search provider for hover text, which calls the search provider definition and scans the area around that text.
What should happen is that we ask for the best definition available and use that to scan the text for a heuristic hover. The search-based hover provider should call the top level definition provider, not just its own.

Fixed by https://github.com/sourcegraph/code-intel-extensions/pull/335.


delete post controller done

I dont really know how to do this so I used createpost controller for inspiration, please lmk if this does not work

At a quick glance this looks good. However, I don't think you need a DeletePostState. When an individual post view is opened, the PostState is updated to contain the entryID of the post currently displayed. I feel that you can simply extract the entryID from the state object and pass that as the input to the controller's input data.
Were you able to implement a view for this?

Were you able to implement a view for this?
I have a view "done" but I have questions as to if i am doing the right thing. I will ask about it later

Were you able to implement a view for this?
I have a view "done" but I have questions as to if i am doing the right thing. I will ask about it later

Sures sure - however, even without a view were you able to delete a post successfully from the database? You can login to MongoDB atlas, go the the Posts Collection and get one of the entryIDs for one of the posts and delete it (pls don't delete any posts that have "Java" as the category as I kind of need that to display the filtering functionality) 😄


Localization (Phase 2 of 3)

Phase 2
Support for Translated Instance Names / Class Names / Spec Names.
Support for Translated LFG Messages.
NO Support for Matching of Translated Instance Names / Localized Abbreviations of Instances.
Also Relevant

https://github.com/Gogo1951/Groupie/issues/156
https://github.com/Gogo1951/Groupie/issues/158

Closing as stale.


First Cloudflare API can validation, but sec can't validation

Hello,
I am using KVM to test k8s and used cert-manager static manifests to for deployment.
kubeadm version: &version.Info{Major:"1", Minor:"18", GitVersion:"v1.18.3", GitCommit:"2e7996e3e2712684bc73f0dec0200d64eec7fe40", GitTreeState:"clean", BuildDate:"2020-05-20T12:49:29Z", GoVersion:"go1.13.9", Compiler:"gc", Platform:"linux/amd64"}
OS version: CentOS Linux release 7.8.2003 (Core)
I have two domain testing.hkitblog.com and wilson.hkitblog.com, the first one can validation, but second can't validation. (Cloudflare API error)
Here is testing.hkitblog.com manifest:

Here is wilson.hkitblog.com manifest:
Same secret

Thanks

This appears to be printing "invalid API token" in your last image, and looking at your Issuer manifests, they are each authenticating with cloudflare using different methods. One is using an API key and the other is using an API token. It seems like your API token is incorrect 🙂
/triage support

Sorry, I have changed to API key, but using same token still have problem

The screenshot change back to API token, also tried API key not work.

seems to be related: https://github.com/jetstack/cert-manager/issues/3013#issuecomment-650792368

Hello,
I have followed this guide: https://cert-manager.io/docs/configuration/acme/dns01/cloudflare/ again to create another Cloudflare token,
but Still Error presenting challenge: Cloudflare API Error:

Thanks

Hello,
I just tried same clusterissuer still Cloudflare API error (the clusterissuer working with first cert)
Thanks

Hello,
This issue was presented https://github.com/jetstack/cert-manager/issues/2857
But I am using v0.15.1
Events:
Type    Reason          Age                From               Message

Normal  SandboxChanged  26h (x2 over 26h)  kubelet, sealos04  Pod sandbox changed, it will be killed and re-created.
Normal  Pulled          26h                kubelet, sealos04  Container image "quay.io/jetstack/cert-manager-controller:v0.15.1" already present on machine
Normal  Created         26h                kubelet, sealos04  Created container cert-manager
Normal  Started         26h                kubelet, sealos04  Started container cert-manager
Normal  SandboxChanged  25h                kubelet, sealos04  Pod sandbox changed, it will be killed and re-created.
Normal  Created         25h                kubelet, sealos04  Created container cert-manager
Normal  Pulled          25h                kubelet, sealos04  Container image "quay.io/jetstack/cert-manager-controller:v0.15.1" already present on machine
Normal  Started         25h                kubelet, sealos04  Started container cert-manager
Normal  SandboxChanged  25h                kubelet, sealos04  Pod sandbox changed, it will be killed and re-created.
Normal  Started         25h                kubelet, sealos04  Started container cert-manager
Normal  Pulled          25h                kubelet, sealos04  Container image "quay.io/jetstack/cert-manager-controller:v0.15.1" already present on machine
Normal  Created         25h                kubelet, sealos04  Created container cert-manager
Normal  SandboxChanged  3h28m              kubelet, sealos04  Pod sandbox changed, it will be killed and re-created.
Normal  Pulled          3h28m              kubelet, sealos04  Container image "quay.io/jetstack/cert-manager-controller:v0.15.1" already present on machine
Normal  Created         3h28m              kubelet, sealos04  Created container cert-manager
Normal  Started         3h28m              kubelet, sealos04  Started container cert-manager
Normal  SandboxChanged  5m18s              kubelet, sealos04  Pod sandbox changed, it will be killed and re-created.
Normal  Pulled          5m5s               kubelet, sealos04  Container image "quay.io/jetstack/cert-manager-controller:v0.15.1" already present on machine
Normal  Created         5m4s               kubelet, sealos04  Created container cert-manager
Normal  Started         5m                 kubelet, sealos04  Started container cert-manager

Hello all,
I am trying http-01 and successfully created cert.

But the cert is Fake

Here is yaml manifest:

Ready is True:

By the way, if I change back yo testing.hkitblog.com (Cloudflare API) that work...
Thanks

OK, I don't play with dns-01 anymore, I just tested http-01 working find, but have to separate clusterissuer, thanks.


Offline cache support not working

Describe the bug
When i close the app and reopen it with the internet turned off, i get this error message in spite of  using the CacheProvider : [Failed host lookup: --url-- : Undefined location]
To Reproduce
turn off internet and then open the application, wherever Query widgets are used, they return this error.
Expected behavior
Display the cached data instead of throwing an error.
Screenshots

As i proceeded to debug, the result.data will be null for the first call of the build function! and then the data will be returned by the subsequent calls.

@dopecoder To be honest the offline support is very experimental, but will have a look and see what's going on.

Any update on this? @HofmannZ Also, when do you expect to release v1?

@micimize I see that you add this Queued for Work. Any particular design in mind yet? Should we instead discuss in #153 and implement different responseType?

Is this related to #290 as well?

@truongsinh it is - I'm actually closing this in favor of #290, #153, and the newly created #319.
As @lucasavila00 noted, it's expected behavior, it's just confusing and unclear to the user what the best way to handle it is


Server Crash when placing Globe

The server crashes when the Globe is placed down. Globe was made of Cobblestone, if that matters.
Crash report here:
https://pastebin.com/NPQxeiZm
This crash happens when starting the server and trying to enter the Globe:
https://pastebin.com/UPEvkw1X

Humm, ill have to look into this.

Fixed in dev, I will release a new version soon.


PrintLog shouldn't call print

because print uses stdout, but we may want to output the text another way. More precisely, using print breaks tqdm gui.
The better way is to compose strings and output them when they are finished using a callback function.

Agreed. A better logging system using callback functions is in the works.


Pull requests: clarification

Hello,
I have forked a test project from an organization to my individual account. I have then committed to the individual repository, and from the Gogs UI i have pushed the small green button (Pull request).
The page that I’m seeing is pretty neat, but I thought of two improvements:

On the top section, the branch comparison makes close to no sense, because there is no way to tell from which repositories they belong (Comparing master with master, but what repo is on the left, what repo is on the right?).
In the middle section, the tabs allowing to write a summary for the PR are not translated (I see release.write and release.preview instead of the titles) (Solved in  <IP_ADDRESS>6 Beta).
On the bottom section (Files diffs), I previously experienced major issues with this kind of view: when the PR has lots of changes (Say 20k lines), it is most likely to fail to display the page. There should be a test on the server-side, if there is more than X lines, don't display the diff.
Then, once validating the PR, the next screen shows the sentence username wants to merge username/master with organization/master, but the two last indicators are in blue (Links) without a link.
Edit: Also, when merging a pull request, it appears that the originator user is "Gogs" when navigating to the main repository page (See screenshot), while when inspecting the PR page the correct user is shown.

I would like to finish this issue by congratulating all Gogs foundators and contributors. When I tried the PR feature, I was truly amazed by the "click-click-go" feeling, and I have to say that the pages overall offer an awesome UX. Keep up the good work, the result is just amazing.
Gogs version <IP_ADDRESS>6 Beta from docker image.
Partially persists in  <IP_ADDRESS>6 Beta.

@hickscorp
Regarding your first question. Since Gogs does not support pullrequest in the same repo, The Branch on the left is always the the one from the repo you forked and the left is your repo.
btw. Which version of Gogs are you using?

@stonerl Yes, i'm used to this behavior, but what i'm saying is that a small indication of the repository name might really clarify what the user is actually doing. Thanks for your explaination!

@stonerl  Hi, Why Gogs does not support pullrequest in the same repo? this is my expect function.

Not implemented yet. But there is annopen issue: #1597

@linphy I'm not sure I understand what would be the benefit of such feature, unless Gogs also implements per-branch authorizations. Anyone with push access on a full repo can directly merge branches. Gitlab solves this by introducing per-branch per user rights. So far, having each of our team member fork the project and issue PRs on the original repo solves all the problems.

@stonerl Please see my updated original post, with one new issue and screen-shot.

On the top section, the branch comparison makes close to no sense, because there is no way to tell from which repositories they belong (Comparing master with master, but what repo is on the left, what repo is on the right?).

This will be fixed along with #1597 so you should follow on that one.

On the bottom section (Files diffs), I previously experienced major issues with this kind of view: when the PR has lots of changes (Say 20k lines), it is most likely to fail to display the page. There should be a test on the server-side, if there is more than X lines, don't display the diff.

Maxline of git diff default is 10k lines, you can change it in settings.

Then, once validating the PR, the next screen shows the sentence username wants to merge username/master with organization/master, but the two last indicators are in blue (Links) without a link.

Indeed an enhancement.

Also, when merging a pull request, it appears that the originator user is "Gogs" when navigating to the main repository page (See screenshot), while when inspecting the PR page the correct user is shown.

This is an old issue, you had wrong setting in your server I think. "Gogs" is a fake one, but is never used.

@Unknwon Regarding your answer to the last point, I don't see any particular settings in the configuration file. Here is why (Along with a suggestion):
The default gogs docker image has a single config file: app.ini. Therefore, when new to Gogs, the immediate reflex is to use app.ini and customize it, therefore overwriting any new app.ini when future Gogs docker image releases will come up.
May I suggest that you adopt the same strategy as you did for the old "regular release" before docker, using conf/app.ini and custom/conf/app.ini? This way, it would always be possible to see differences between custom and vanilla config.

Regarding your answer to the last point, I don't see any particular settings in the configuration file. What is this setting you're telling me about? Here is why (Along with a suggestion):

I mean it needs to go another issue because it's irrelevant to this one (title).
Also, the one you're talking is already the custom app.ini: https://github.com/gogits/gogs/blob/master/Dockerfile#L12

Then, once validating the PR, the next screen shows the sentence username wants to merge username/master with organization/master, but the two last indicators are in blue (Links) without a link.

In the case when forked repo/branch is deleted, the translation string will break, and it adds unnecessary complicity to the I18N (two exactly same strings but with link) and a database query.
Since all point are invalid to this thread, I'm closing it. But thanks your feedback!


Postgres driver ignores supplied database name

The offending line seems to be here:
https://github.com/DHI-GRAS/terracotta/blob/964a72b022bea565a10a73d6cce0a40542304a90/terracotta/drivers/postgresql_meta_store.py#L67
       engine = sqla.create_engine(
            # `.set()` returns a copy with changed parameters
            self.url.set(database=self.DEFAULT_CONNECT_DB),
            echo=False,
            future=True,
            isolation_level='AUTOCOMMIT'
        )

The supplied name is replaced with DEFAULT_CONNECT_DB.

Can you tell us what you're trying to do / what kind of error you get? That line is fine - using the default DB is intended behavior here because the target DB doesn't exist yet.

You're right, I misunderstood the functionality of set. I thought it also changed the underlying class property. Apologies.


Committing first iteraiton of classy CI test org creation

Just checking in, @winstan. I am still doing more work, but I wanted to get some feedback about what you would like to change.
The problem with this approach is that we are importing a couple of libraries that require a pip3 install. I do not know if we want to open that possibility yet.

It was necessary to test this script against working CI tests to ensure that the classytest organization setup worked. To test the script with working CI tests, I had to merge it with the feature/circle-ci-github-dev-make-script-new-env branch.
I am therefore going to close this pull-request.


Can you help me with a touchscreen issue?

Hi daumemo,
I have the same display with the same FT6236U based chip. However in your example script, the left, right and top side of the screen is not working correctly. If I slide on the edges the touchscreen will be off by 20 px on the top, 50 px on the left and 35 px on the right side, do you have any idea where the problem might be?
With best regards! And thank you for your great work on Github!

Hi. It might be a little hard for me to fully help you as my display is currently in use and I don't have a physical ability to test how the display behaves in reality. But I will try to help you in theoretical way.

In the FT6236U  datasheet it is written that this IC supports gestures. If you slide a finger, the behavior might be different than just tapping on the screen on the edges.
My test code reads only X and Y values of the first touch point detected and ignores all others. Do you press on the screen with one finger? Or maybe you are holding the screen in the had while tapping? Then try to lay it on the table and tap only with one finger.
If the screen surface is dirty, there might be also some issues.
Check out the FT6236 datasheet. The IC has some offset values which might be set by the MCU. You could try playing with them, but I don't believe that it would help.

So, this is all I could think of at the moment.

Hi Daumemo, thank you for the quick reply :)

By sliding around with the finger, the touchscreens works very accurate. However I can not use my keyboard on the display, because single touches are very inaccurate at the edges.
No change.
I bought several displays and they all behave exactly the same.
I tried that, but it did not help with the accuracy on the edges.

I also made a new test board with very short wires (3 cm) and soldered everything together. I used ILI9481 displays with the same pin layout and the same touchscreen and driver as the R61529 display. I can switch between the ILI9481 display and the R61529 display.  I also tried different libraries with the same inaccuracy. However if I use two displays (one only for graphics and the other as touchscreen). I have no problems and the touchscreen works just fine.
Do you have a hint what this could mean? I used your schematics and I am using 3.3k pull up resistors on i2c.

ok, I found a solution for my ILI9481 screen with FT6236 CTP driver.
By hours of tinkering by trial and error. The CTP driver is working very accurate now. For reasons I do not understand the problem disappears when I comment out the following part in the init file:
writecommand(0xD1);
writedata(0x00);
writedata(0x07);
//   writedata(0x10);

Hi, I am happy to hear that you have found a solution. I check out mine R61529 init file : https://github.com/daumemo/IPS_LCD_R61529_FT6236_Arduino_eSPI_Test/blob/master/lib/TFT_eSPI/TFT_Drivers/R61581_Init.h
you will find that there are no such writedata(0x10) at all in D1 register...
And as I understand, the problem has nothing to do with the touchscreen itself, it seems like an LCD driver problem? As the writecommand(0xD1); is used to set a VCOM voltage (according to the ILI9481 datasheet) in the LCD itself. But still don't get why it was related to the touch screen. Maybe if you set wrong VCOM amplitude, the LCD's calibration becomes with an offset?
Well, I personally don't have such deep knowledge what the VCOM and its alternation amplitude does to the LCD behavior... If you find it out - please share your knowledge :D


Add acknowledgements

Hi @StRigaud,
I'm proposing to add a section acknowledgements to the readme. Feel free to expand the text to also acknowledge other funding sources.
Thanks!
Best,
Robert

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

Comparison is base (af4b499) 49.74% compared to head (bc1fd23) 49.74%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #114   +/-   ##
=======================================
  Coverage   49.74%   49.74%
=======================================
  Files          15       15
  Lines        1172     1172
=======================================
  Hits          583      583
  Misses        589      589

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

Done :)


Review Lifecycle Document during TAC Call

Please share any additional details on this topic
It was decided to do a live review/edit of the lifecycle document during a TAC call
Detail what actions or feedback you would like from the TAC
N/A
How much time do you need for this topic?
At least 30 minutes

Current draft : https://docs.google.com/document/d/1NV-0vNgXWdc81oqT0jv0C-9Funb8dySS06u90ghF-X4

This was done. So closing.


Disable 600+ channel json_run_localhost scenarios from RBE TSAN runs

Per conversation in https://github.com/grpc/grpc/pull/17539 (which this PR is an alternative to)
One way to do this that might be better is to just have a blacklist of scenario names; I wasn't sure, if there are thoughts on that.

closing in favor of https://github.com/grpc/grpc/pull/17575


Imports FOLIO Holdings Records

Adds support for importing FOLIO Holdings JSON records.
Fixes #12

Closing in favor of https://github.com/sul-dlss/folio-airflow/pull/25


sidecar-deployment-user is not authorized to perform: lambda:GetLayerVersion in me-south-1

this happens only in me-south-1 region  , it works fine in  us-west-2
when run : php artisan sidecar:deploy --activate
AccessDeniedException (client): User: arn:aws:iam::**:user/sidecar-deployment-user is not authorized to perform: lambda:GetLayerVersion on resource: arn:aws:lambda:me-south-1:821527532446:layer:sidecar-browsershot-layer:1 because no resource-based policy allows the lambda:GetLayerVersion action - {"Message":"User: arn:aws:iam::13286269**944:user/sidecar-deployment-user is not authorized to perform: lambda:GetLayerVersion on resource: arn:aws:lambda:me-south-1:821527532446:layer:sidecar-browsershot-layer:1 because no resource-based policy allows the lambda:GetLayerVersion action"}
the user policy :
`
"Version": "2012-10-17",
"Statement": [
	{
		"Sid": "VisualEditor0",
	"Effect": "Allow",
		"Action": "iam:PassRole",
		"Resource": "*",
		"Condition": {
			"StringEquals": {
				"iam:PassedToService": "lambda.amazonaws.com"
			}
	}
	},
	{
		"Sid": "VisualEditor1",
		"Effect": "Allow",
		"Action": [
			"states:*",
			"ecr:SetRepositoryPolicy",
			"lambda:*",
			"ecr:GetRepositoryPolicy"
		],
		"Resource": "*"
	},
	{
		"Sid": "VisualEditor2",
		"Effect": "Allow",
		"Action": [
			"logs:DescribeLogStreams",
			"logs:GetLogEvents",
			"logs:FilterLogEvents"
		],
		"Resource": "arn:aws:logs:*:*:log-group:/aws/lambda/*"
	},
	{
		"Sid": "VisualEditor3",
		"Effect": "Allow",
		"Action": "s3:*",
		"Resource": [
			"arn:aws:s3:::sidecar-*",
			"arn:aws:s3:::sidecar-*/*",
			"arn:aws:s3:::*"
		]
	}
]

`

Hi Basem,
Both chrome-aws-lambda-layer and sidecar-browsershot-layer have not been deployed to me-south-1.
chrome-aws-lambda-layer is not maintained by me. You would have to create an issue in their repository to request a deployment to me-south-1.
sidecar-browsershot-layer is maintained by me. One chrome-aws-lambda-layer has been updated, I can deploy the layer to me-south-1 as well. (Would be great if you could create an issue in that repository as well.)


Expose ability to set HTTP/HTTPS in Agent

Currently the only transport protocol supported during install is HTTP.
The user should be able to define what transport protocol to use - e.g. HTTPS or HTTP.

Actually, this might not be needed. Thoughts,  @matisku ?

It all depends on a setup. Might me needed as a option.

I tried connecting over HTTP to a HTTPS bamboo server to pull down the JAR file - appears to work, although my setup uses a 301 redirect from HTTP to HTTPS that only changes the protocol.
I imagine most people would also be using a 301 redirect that only changes the protocol as well.

I don't think this is actually needed any more


Update twofa.entitlements

Remove duplicate com.apple.application-identifier to attempt fixing #4, specifically —

Failed to parse entitlements: AMFIUnserializeXML: duplicate dictionary key near line 10

This is great, thanks!


Roll engine 2e8e96fad1..a8890fdccd667 and fix paragraph tests

git log 2e8e96fad1ada324adca718d13410a98d9bb1036..a8890fdccd667d0bf0fa0a81ac350dadef60bd6c --no-merges --oneline
a8890fdcc SkImageFilter needs to be imported in the header (#6296)
9ca666868 Roll src/third_party/skia c9873a5f8b8e..44c6167c4125 (23 commits) (#6295)
8471862cf Dart SDK roll for 2018/09/20 (#6294)
0036a3ac2 Revert "Roll src/third_party/skia c9873a5f8b8e..f3ebd312f213 (14 commits) (#6290)" as it broke Windows buildbot. (#6292)
ff787072f Remove runtime/dart_vm_entry_points.txt (#6291)
09afb2b8c Roll src/third_party/skia c9873a5f8b8e..f3ebd312f213 (14 commits) (#6290)
2302bea9c Roll src/third_party/skia 9369031fde2d..c9873a5f8b8e (11 commits) (#6287)
71457a67d Roll dart to c688d0c0c3ad3dece3a79ce0e115d787a94707ea. (#6285)
150135929 Roll src/third_party/skia 1d6281d4bb47..9369031fde2d (27 commits) (#6284)
f3d51b0cf Fix Top, Left, and Right padding for fullscreen android apps. (#6282)
7ec1d211e Compute maxIntrinsicWidth based on the width of styled runs added to the line breaker (#6281)
2b25fd7da Don't use unix or win namespaces (#6277)
60c5bb86d Return null instead of throwing if an instance method is passed to PluginUtilities.getCallbackHandle (#6260)
Fix paragraph_tests.dart to match now correct engine

Thanks for fixing.


[T4A3][T16-B4]MuhammadArifBinSyedNasser

Ready for review

@Halo3fanz added several comments, please read through them and familiarise yourself with inheritance, there are many resources online that talk about this topic, such as this tutorial by Sun Oracle https://docs.oracle.com/javase/tutorial/java/IandI/subclasses.html
Please close this PR after you're done reading the comments.

@Halo3fanz added several comments, please read through them and familiarise yourself with inheritance, there are many resources online that talk about this topic, such as this tutorial by Sun Oracle https://docs.oracle.com/javase/tutorial/java/IandI/subclasses.html
Please close this PR after you're done reading the comments.


Possible DriverLoader.load(className, pathToDriver) issue

According to the javadoc:
@param pathToDriver the path to the JAR file containing the driver; note, this can be a semi-colon separated list of paths
However in the code pathToDriver is being splitted using File.pathSeparator which I think is wrong and could cause really weird errors (URLClassLoader is including any file from the root path)
final String[] paths = pathToDriver.split(File.pathSeparator);

Going back to review this I don't think this is an issue because while yes ODC does split the provided paths - however, it converts each part into a URI and re-combines them:
https://github.com/jeremylong/DependencyCheck/blob/78cca2d0a5fcc57f0fe6d8efabcbca35b85577c5/core/src/main/java/org/owasp/dependencycheck/data/nvdcve/DriverLoader.java#L134


PWM Pin option is shown only for PWM capable pins

fixes #1201

PWM pin option only displayed when a PWM-capable pin is selected
PWM option now also available for multi-selection when all checked pins support PWM
Fixed Dock-property and added padding to multiselect
Multi-select option is displayed correctly

Added IsArcazeSerial as method to make it clear and similar to all the other boards. Arcaze Serial is detected when all other Serial Types are returning false (MobiFlight, Joystick, MidiBoard)
Added unit tests (also for Midi Board Serial)

I guess I approve this 😆


fix(package): stop using typings on postinstall

Now, Installation is fail when an user isn't using typings.
Please stop using postinstall for developing.
Thanks.

hi.
This is required when developing this package.
are you referring to when developing using this package your project?

Same here. Seems to be an issue when using TypeScript2 in your project and trying to install angular2-infinite-scroll.

this is confirmed as Typescript2 use the npm '@types' package for typings.

hi.
i published the new 0.2 version at 2d784323640063da9d0ea33a3f56b4c016269a77 - so this pr is included.


Faucet - Request funds wording

As a user of the faucet, I want to have user friendly wording, so that I can better interact with the interface
Given I am in https://faucet.okp4.network/
When I want to request funds by using a chain address
Then I want the wording in the section to be :

Request funds with OKP4 chain address
0xxxxxxxxxxxxxxxxxxxxxxxxxxx

Example :

To release at the same time as the recaptcha

@ChaimaaZaoui the faucet is internationalized in english and french so we need a french description as well. Are you ok with: Demander des fonds avec une adresse de la chaine OKP4 ?

ça me convient merci

@ChaimaaZaoui @ErikssonJoakim The placeholder is not correct for the expected format of okp4 addresses. We should use okp4xxxx...:

Test 20/06 OK ✅


[event-exporter] Fix event exporter spam

Watch framework periodically resets the connections (~one every 15 minutes) which causes a lot of messages about lost events in Stackdriver. Though this change is raising the chances that some events will be lost without a note, the probability of this event is extremely low (it requires to lose whole apiserver event history, e.g. in case of a really long network outage). Note, that in case container is restarted, the message will still be generated.
/cc @piosz

This change is

/lgtm

/approve no-issue

/approve no-issue

@piosz Thanks!


Add UserDefinedLogicalNodeCore

Which issue does this PR close?
Related to #5400
Follow-up of the discussion from PR #5515 where @alamb added new docs to UserDefinedLogicalNode.

Rationale for this change

Will make boilerplate from dyn_eq and dyn_hash unnecessary.
What changes are included in this PR?
Added the trait UserDefinedLogicalNodeCore and a blanket implementation UserDefinedLogicalNodeCore -> UserDefinedLogicalNode.
Refactored existing UserDefinedLogicalNode to use UserDefinedLogicalNodeCore - which showcases the value of this PR.

Are these changes tested?
It's a refactor. The expected behaviour is in already existing tests.

Are there any user-facing changes?
Added new trait UserDefinedLogicalNodeCore.
No changes to the existing API (including UserDefinedLogicalNode).

Given this is backwards compatible, I plan to merge this in in the next few hours so we make the release. Thanks again @mslapek

I took the liberty of adding a bit ore documentation and merging up from main; Once this PR passes CI I intend to merge it.
Thanks again @mslapek

@alamb Thanks for the review! 💫


Support -include flag

From GCC's man page:

-include file
Process file as if "#include "file"" appeared as the first line of the primary source file. However, the first directory searched for file is the preprocessor's working directory instead of the directory containing the main source file. If not found there, it is searched for in the remainder of the "#include "..."" search chain as normal.

This currently isn't handled correctly by clangd. I get error diagnostics on symbols from the headers passed via the -include option. Would it be possible to support this?

Could you provide more details on how you tried and what diagnostics you got? As clangd does support implicitly including headers via -include.
e.g. have a compile_flags.txt with (the same should work with compile_commands.json):
-include
abc.h

and then having abc.h with:
int abc;

and a.c with:
void foo() { abc = 2; }

you should not get any errors in a.c.

Could you provide more details on how you tried and what diagnostics you got? As clangd does support implicitly including headers via -include.
e.g. have a compile_flags.txt with (the same should work with compile_commands.json):
-include
abc.h

and then having abc.h with:
int abc;

and a.c with:
void foo() { abc = 2; }

you should not get any errors in a.c.

You're right, I just tried again and it works. I must have gotten something wrong with the paths. Thanks for the support!

You're right, I just tried again and it works. I must have gotten something wrong with the paths. Thanks for the support!


feat(options): add ale_fix_* options

Add 3 new options for better control of ALEFix: ale_fix_on_insert_leave, ale_fix_on_save and ale_fix_on_enter. Their defaults are the same as their ale_lint_* equivalents.

ale_fix_on_insert_leave = 0
ale_fix_on_enter = 1
ale_fix_on_save = 1

Also added docs.
Sorry, but don't have tests because I'm new to Vim world, can you handle them?

@w0rp ookey, i'll change them. It would be good to add your thoughts about ALEFix on the docs too :)

For fixing files when leaving insert mode or saving files, we should probably be a little more clever. The autocmd commands for checking and fixing the files will be competing with one another. If both are on at the same time, we should make it so files are checked only after fixing them, which won't be straightforward to handle. It might not be possible at all. Vim doesn't let you modify a buffer which isn't open, but we can read a buffer which isn't open. So, we could fix and then check a file if the buffer is open, but we can only check a file if the buffer is closed, unless we are writing the changes to disk and then reloading them when the buffer opens again.
I suggest passing an even argument to Fix like for what is currently done for Queue, and then saving that event string in the buffer Dictionary for fixing files. Then we can load that string back again when files are fixed, and we can test that string for how files should be checked. 'lint_on_save', etc.
I can work on all of this later. Here are the two workflows.
For fixing files when leaving insert mode:

Leave insert mode.
Fix a file, and also check it.
Apply changes to buffer when buffer is open.
Check file again, running only the linters with lint_file set to 0.

For fixing files when saving them:

Save a file.
Fix a file, and also check it.
Save the result of fixing the file to disk.
If the buffer ever opens again, reload the buffer and run all linters for the file.

Mostly, don't know. As I said I'm new to Vim, so don't understand the whole things too deeply.

I'll give the handling of saving files and so on a go when I get some time.

I'll close this as I'm working through these options now. I'm not sure about fixing files on enter yet. I have added fixing files on save for now, which is the hardest to implement. I'll work on storing the buffer variables for ale#Var in case the variables are read after the buffer is deleted.

Cool! No problem. It's great that progress is happening.


Opera 11.52 crashes

Opera 11.52 crashes on "Gentoo Linux amd64" when testing for "Event 'progress' " with an "segmentation fault".

Does this happen on the test suite or the single page http://areweplayingyet.org/event-progress ?
If it happens on the single page you could report this directly to Opera: https://bugs.opera.com/wizard/
Once done you can create a pull request to indicate this bug in the reports object of the test. (described in the readme https://github.com/soundcloud/areweplayingyet/)
Thanks!

I have reported the bug to the vendor. :)

Thank you!

\o/ — Closing this issue :)


[Project title here] : Implementation of Heap memory management system in C++ (using first fit strategy)

Prerequisites

[X] I am a GSSoC-22 Participant
[X] I checked to make sure that this issue has not already been filed

Description
I would like to add a c++ project which is an implementation of Heap Memory Management System using First Fit Strategy.
Which also merges continuous free blocks.
Tech Stack
C++

@Siddhesh777 what else can we do aside from memory allocation
Coz this can be done using malloc
Add some other features like using gui and stuff

@abhijeet007rocks8
Its a heap visualizer.
This is the stander project given in colleges


shobjidl_core.h

Hello there,
Would it be possible to provide wrappers to interfaces from shobjidl_core.h? I am particularly interested in IShellLinkA/IShellLinkW.
Thanks.

Of course! I've added IShellLink and a few related ones to the main branch. At the moment, I'm still adding these interfaces manually as I improve and verify the generation code, so please let me know if you need other ones and I'll add them as quickly as I can.

v2.1.2 is published, including these changes: https://pub.dev/packages/win32

Thanks!
Can we also have factory method just as in https://github.com/timsneath/win32/blob/main/lib/src/com/IShellItem.dart#L92?

Oh -- good catch. The Unicode name (IShellLinkW) messed up my code generator. I've fixed this in b91ad90, but I'd like to write a small test to make sure that this works correctly. I'll do that and then publish.

v2.1.3 includes a working version.
I've created a small example of its use: https://github.com/timsneath/win32/blob/d8f04121d5ed5c42b9e4c89098adf626f471b267/example/shortcut.dart

Thanks for quick response. I was able to create and persist a link.
BTW, the example above gives me STATUS_HEAP_CORRUPTION due to free(ppf) call. I didn't look much into it – is the memory already released at that point?

Yeah, I think you may be right. Looks like there's a double free bug there.
I'll take a proper look this evening.
On Tue, Jun 1, 2021 at 12:40 AM youk @.***> wrote:

Thanks for quick response. I was able to create and persist a link.
BTW, the example above gives me STATUS_HEAP_CORRUPTION due to free(ppf)
call. I didn't look much into it – is the memory already released at that
point?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/timsneath/win32/issues/231#issuecomment-851901655,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AARWL66A32T7AO7XJRDNUPDTQSFILANCNFSM45KZ72UQ
.

I think I've caught it -- can you try again?

It's ok now.


engine.create_schema() fails to create tables

When there are more than 100 table in user's account and a table for a model already exists engine.create_schema() will fail with:
ResourceInUseException: Table already exists: table-name
This happens because in engine.py line 204 the call to:
        tablenames = set(self.dynamo.list_tables())

will return 100 first tables and in model_meta.py line 440:
        if tablename in tablenames:
            return None

will miss the fact that the table already exists.
Proposed solution:
For each table to create deliberately validate that it doesn't exist. Like (proto-code):
        for model in self._ddb_engine.models.itervalues():
            if not self._ddb_engine.dynamo.describe_table(model.meta_.ddb_tablename()):
                model.meta_.create_dynamo_schema(self._ddb_engine.dynamo, tablenames=[], wait=True)

I think the solution might be easier than that. Old versions of dynamo3 had a default limit on list_tables of 100, but since 0.4.6 the default limit has been removed. Can you try pip install dynamo3>=0.4.6 and see if that fixes your problem?


Replace github.com/juju/errors with github.com/pkg/errors

github.com/pkg/errors is used in the rest of the project and does not have a LGPL license

replaced with https://github.com/jetstack/jetstack-secure/pull/570 which uses a branch on this repo.


Unify STDOUT/STDERR logging

STDOUT and STDERR are logged three separate ways, to the terminal, to the local output.log, and to the back end. Currently it's possible for the two streams to race differently in each case, resulting in slightly different output. We should make sure all three of these representations look exactly the same.

In this run, downloading output.log gives different different results than the terminal on the run page, and both of them look different than the local terminal output:
wandb: Syncing https://app.wandb.ai/adrianbg/garbage/runs/4pj90kcr
wandb: Run directory: wandb/run-20180214_194906-4pj90kcr

Using TensorFlow backend.
/Users/adrian/.pyenv/versions/3.6.0/Python.framework/Versions/3.6/lib/python3.6/importlib/_bootstrap.py:205: RuntimeWarning: compiletime version 3.5 of module 'tensorflow.python.framework.fast_tensor_util' does not match runtime version 3.6
  return f(*args, **kwds)
Train on 60000 samples, validate on 10000 samples
2018-02-14 11:49:16.104273: I tensorflow/core/platform/cpu_feature_guard.cc:137] Your CPU supports instructions that this TensorFlow binary was not compiled to use: SSE4.1 SSE4.2 AVX AVX2 FMA
{'batch_size': 32, 'epochs': 3, 'steps': None, 'samples': 60000, 'verbose': 1, 'do_validation': True, 'metrics': ['loss', 'acc', 'val_loss', 'val_acc']}
Epoch 1/3
48544/60000 [=======================>......] - ETA: 4s - loss: 0.3329 - acc: 0.8962^Cwandb: Ctrl-c pressed; waiting for program to end.
Traceback (most recent call last):
  File "./mnist-cnn.py", line 54, in <module>
    TensorBoard(write_images=True, write_grads=True, histogram_freq=1)])
  File "/Users/adrian/.pyenv/versions/3.6.0/envs/wandb-3.6/lib/python3.6/site-packages/keras/models.py", line 960, in fit
    validation_steps=validation_steps)
  File "/Users/adrian/.pyenv/versions/3.6.0/envs/wandb-3.6/lib/python3.6/site-packages/keras/engine/training.py", line 1657, in fit
    validation_steps=validation_steps)
  File "/Users/adrian/.pyenv/versions/3.6.0/envs/wandb-3.6/lib/python3.6/site-packages/keras/engine/training.py", line 1213, in _fit_loop
    outs = f(ins_batch)
  File "/Users/adrian/.pyenv/versions/3.6.0/envs/wandb-3.6/lib/python3.6/site-packages/keras/backend/tensorflow_backend.py", line 2357, in __call__
    **self.session_kwargs)
  File "/Users/adrian/.pyenv/versions/3.6.0/envs/wandb-3.6/lib/python3.6/site-packages/tensorflow/python/client/session.py", line 889, in run
    run_metadata_ptr)
  File "/Users/adrian/.pyenv/versions/3.6.0/envs/wandb-3.6/lib/python3.6/site-packages/tensorflow/python/client/session.py", line 1120, in _run
    feed_dict_tensor, options, run_metadata)
  File "/Users/adrian/.pyenv/versions/3.6.0/envs/wandb-3.6/lib/python3.6/site-packages/tensorflow/python/client/session.py", line 1317, in _do_run
    options, run_metadata)
  File "/Users/adrian/.pyenv/versions/3.6.0/envs/wandb-3.6/lib/python3.6/site-packages/tensorflow/python/client/session.py", line 1323, in _do_call
    return fn(*args)
  File "/Users/adrian/.pyenv/versions/3.6.0/envs/wandb-3.6/lib/python3.6/site-packages/tensorflow/python/client/session.py", line 1302, in _run_fn
    status, run_metadata)
KeyboardInterrupt

wandb: Program failed with code 255. Press ctrl-c to abort syncing.
wandb: Waiting for final file modifications.
wandb: Syncing files in wandb/run-20180214_194906-4pj90kcr:
wandb:   wandb-metadata.json
wandb:   diff.patch
wandb:   wandb-events.jsonl
wandb:
wandb: Verifying uploaded files... verified!
wandb: Synced https://app.wandb.ai/adrianbg/garbage/runs/4pj90kcr
(wandb-3.6) adrian@Adrians-MBP:~/src/wandb-examples/keras$

Uh.. this is still a bug, isn't it?

Ah, I thought you had fixed it.

Closing in favour of https://github.com/wandb/core/issues/2607

Closing in favour of https://github.com/wandb/core/issues/2607

error 404

I want to the wandb logs to go do stdout. How do I do that?


feat: closes #71 - add configurable trace resolution

I've added a parameter to set the resolution on the trace image.  I called it traceResolution to avoid any confusion with that and the output resolution.  I don't think the process is explained in the docs, so it might be helpful to include a brief description of how the tool works.  I made a brief mention in the parameter documentation.

Thanks alot for this PR!. Everything looks good so I will be releasing this in v2.1.0.


How to set the value of the stepper?

Hi.
I am using your library with Swift 2 for iOS9.3
I am trying to set the value of the stepper from my controller. Is there a method to do that?
I saw the setValue method but it requires "key" for which the value is set. I am not sure what the key should be to set the stepper value. Can you give me command to set the value of the stepper?
Thx
-Vijay

I guess I found it.
stepper.setValue(3, forKey: "Value")


Possibility to get further information of a tweet

Hi, would it be possible to extend the Tweet class to be able to get the following information:

The geographic location of a tweet
The name of the author of a tweet
The hashtags of a tweet
The mentioned URLs in a tweet
The media of a tweet (picture, video, GIF)

Hi @rahelarnold98 ,

The geographic location of a tweet

I think that currently this is not possible, you have to use tweet.getUser().getLocation() to have the user location.
For the tweet location, I will try to implement it this week-end.

The name of the author of a tweet

Use tweet.getUser().getName()

The hashtags of a tweet

Use tweetV2.getData().getEntities().get("hashtags")

The mentioned URLs in a tweet

Use tweetV2.getData().getEntities().get("urls")

The media of a tweet (picture, video, GIF)

I think this is not possible, a development will probably be needed too, I'll check that.

Missing tweet fields will be accessible from the next release :
tweetv2.getAttachments().getMediaKeys() to get medias ids.
tweetv2.getGeo() to get geo-localisation information.

Hi @redouane59
Thanks a lot for the fast support.

But by calling tweet.getUser().getName() the only thing that happen is, that UnsupportedOperationException() gets thrown...

Can you tell me how did you get the user ? I think that this is available from TweetV2 but not TweetV1

Just to follow up - though I've never used github and don't know if this will be seen given it's closed,
TweetSearchResponseV2's data list is of type TweetV2.TweetData, so when a user calls searchForTweetsFullArchive and loops through TweetSearchResponse's getTweets() and calls getUser() then I believe you will reach UnsupportedOperationException().  Perhaps I'm not understanding the code correctly, but I've run into a similar issue as I do not see how to reach TweetV2's Includes information from searchForTweetsFullArchive.
Hopefully this helps.
J

Hi @JustinBPhillips , don't worry I saw your comment. I'm going to debug it now. I'll keep you informed

Ok I see the problem, the Twitter API is returning an object which looks like :
{
   "data":[
      {
         // tweet 1
      },
      {
         // tweet 2
      },
      {
        // tweet X
      }
   ],
   "includes":{
      "users":[
         {
            // user 1
         },
         {
            // user 2
         },
         {
           // user X
         }
      ]
   }

and my method just returns a list of tweets, which is based on the data property.
I don't really like the object that Twitter is returning. To solve the problem, I should maybe return the whole object instead of a List<Tweet> an let user consume it with result.getData() and result.getIncludes().getUsers() .
I will add @SuhemParack into the discussion to have his advice because it's his scope :)

Fantastic, thanks for looking into this.
While I have you, there are some missing fields in the TweetV2.TweetData class.  Off the top of my head, I think it's missing "source", and "withheld", and Attachments is missing "polls_ids".  Thanks for putting all of this together, I'm looking forward to using this!

there are some missing fields in the TweetV2.TweetData class

Could you create another issue for this ? I'll try to implement it soon.

I will add @sparack into the discussion to have his advice because it's his scope :)

For this structure issue, @sparack advised me to create a feedback for his product team here . Please put +1 on it to make the topic going up :)


Multiple silent kits scanning in multi-root workspace

When opening a multi-root project, if there is a breaking change in the kits definition, the extension will trigger as many silent scans as the number of projects.

This should be fixed in CMake Tools 1.4.1 which was published today.
Let us know if you encounter any issues with this release.