TypeError: Cannot read property 'containerId' of undefined

Tried to do an event migration today and got this error. Not sure how to fix it.
It will run through 2 tags and add them, but then stop with the error.

Thanks for reporting an issue. Could you describe what exactly you were doing in the sheet when you encountered this error and perhaps share a screenshot of the issue?

I was at the final step "Migrate Event Tags" and the script runs and then produces the error.

To bypass that error, I grabbed the container id and hard coded it, and then it failed on this error: TypeError: Cannot read property 'name' of undefined

The new tag isn't being created for some reason so things like "name" and "containerId" don't exist when the new tag is being referenced. The issue with creating the new tag should be logged in the "executions" area in the Apps Script connected to the sheet.

Sorry, I don't know much about scripting here. but here is what I got from "migrateEventTag"
2:55:32 PM	Error
TypeError: Cannot read property 'oldTag' of undefined
migrateEventTag	@ utils/migration.gs:184
And from from "migrateEventTags"
2:55:55 PM	Notice	Execution started
2:56:30 PM	Info	{ [GoogleJsonResponseException: API call to tagmanager.accounts.containers.workspaces.tags.create failed with error: vendorTemplate.parameter.eventParameters[1].name: Parameter names cannot start with _
vendorTemplate.parameter.eventParameters[2].name: Parameter names cannot start with _
]
name: 'GoogleJsonResponseException',
details:
{ errors: [ [Object] ],
message: 'vendorTemplate.parameter.eventParameters[1].name: Parameter names cannot start with _\nvendorTemplate.parameter.eventParameters[2].name: Parameter names cannot start with _\n',
code: 400 } }
2:56:32 PM	Error
TypeError: Cannot read property 'name' of undefined
migrateEventTag	@ utils/migration.gs:251
(anonymous)	@ utils/migration.gs:171
migrateEventTags	@ utils/migration.gs:170

The issue you're running into is that parameter names cannot start with an underscore. If you change your parameter names so that they don't start with "_" and rerun the script, then it should work.

I noticed that and looked at my params and didn't find any like that. I do however think I determined the issue. The user had added to columns M > S new parameters in addition to the conversion from cat/act/lbl. I think moving those to col U > AB as net new might have solved the issue.

Ok, I figured it out. I had to inspect all fields and 2 of them were like this as parameter names:
"ua_event_category
"
"ua_event_action
"
Looks like that new line will break it.

Sounds good. Glad you were able to identify the issue.


upgrade to official mongo go driver #12

Hi,
What does this PR do?
Replaces mgo with MongoDB official driver, this will enable users to use the new official mongo driver.
How should this be tested?
Update MongoDB URI in the test file and run test go test -v .

hey @nvcnvn please help!

I'm not sure what help you might expect from my side. What seems to be the
problem?
On Sat, Nov 6, 2021, 00:28 Oyewole Samuel A. @.***>
wrote:

hey @kovetskiy https://github.com/kovetskiy @nvcnvn
https://github.com/nvcnvn please help!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/kidstuff/mongostore/pull/13#issuecomment-960058853,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACAN7ZAMGZFX4VDYXWSTRKDUKQO6DANCNFSM5HEM2XRA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


Modal: Add a way to intercept dismissal and prevent it.

I think it would be interesting if there was a way of preventing the user to accidentally dismiss a modal window when clicking on the backdrop by mistake. I'd be happy to think/work on a solution if you guys think it's a good idea.

There should be hooks that "clicking outside" and "pressing escape" should both go through so that we can override that hook and prevent closing with logic of our choosing. That would make a good API.

The backdrop:'static' solution isn't enough when you want dynamic control over when the modal is dismissible. For example, I have a modal with a bunch of data fields and a Save button. Normally, I want the user to be able to dismiss the modal by clicking outside or hitting Escape. However, after Save is pressed and a save request is sent to the server, I don't want the user to be able to dismiss the modal until a reply from the server has been received. It would be great to have a hook where I can intercept modal dismissal and cancel it.


Const correctness in C++98 codegen

The C++98 generated code should support a const pointer on decode.  It may be limited to C++11 or 14 but I'm sure the MessageHeader could detect a const buffer and self modify to disable the setters.
	hdr.wrap (const_cast<char*> (buffer), 0, version, length);
	msg.wrapForDecode (const_cast<char*> (buffer), hdr.size(), hdr.blockLength(), hdr.version(), length);

Unfortunately, due to the statefulness of a decoder in tracking position, it is not possible to take this approach. Using const_cast to update position on getters is fine, but it is undefined behavior to do this on a const declared codec. See the example at http://en.cppreference.com/w/cpp/language/const_cast Thus, it isn't a good idea to take this tact. Instead, #271 will eventually be the solution and generate decoder and encoder versions.
Closing this as invalid.


error TypeError: Cannot read property 'install' of undefined

i get this error

any help ?
thank you

Vue looks for an install method when you call Vue.use.  It could be that something you are passing to Vue.use is undefined.


Is this project still maintained?

Or should I just fork it?

Yes, if you want to contribute, you can :)


[Feature] Support Elasticsearch

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

你好，我是谢晋峰，你的邮件我已经收到，谢谢！


Prevent line curvature overshooting

Thanks for this plugin. It's really fantastic and you're really killing it!
I love the curved lines, but they cause data confusion due to "overshooting" their data point. Changing curvature smoothness doesn't seem to fix this specific issue.
Flat:

Curved:

Becomes especially problematic if you see the actual data points:

Can we do something like this crummy Photoshop job?

Basically, keep curvature, but prevent overshooting?

Hi, pleasure is mine.
Thats a bad issue,
I will find a solution ASAP
Thanks for report!

did you try set curveSmoothness less than 0.35?
(the default value is 0.35, you can decrease it)

Yes, doesn’t have the same effect. I want curves, just not overshooting curves. Thank you!

Yes it doesn't have the same effect, but can be used as temporary solution, until I will be free to work on it.

fixed on 0.1.2, added preventCurveOverShooting on LineChartBarData
preventCurveOverShooting = false

preventCurveOverShooting = true

It would be nice if you share the result on your sample

Thanks for this! It's awesome, but while almost all of my graphs works well there are some that with preventCurveOverShooting = true looks like this (see the right side)


Warning: Scrolling(div): Support for defaultProps will be removed from function components in a future major release

Warning: Scrolling(div): Support for defaultProps will be removed from function components in a future major release. Use JavaScript default parameters instead.
    at ScrollingComponent (webpack-internal:///./node_modules/react-dnd-scrolling/lib/esm/index.js:96:36)
    at DndProvider (webpack-internal:///./node_modules/react-dnd/dist/core/DndProvider.js:43:11)
    at MyApp (webpack-internal:///./pages/_app.js:148:11)
    at PathnameContextProviderAdapter (webpack-internal:///./node_modules/next/dist/shared/lib/router/adapters.js:62:11)
    at ErrorBoundary (webpack-internal:///./node_modules/next/dist/compiled/@next/react-dev-overlay/dist/client.js:301:63)
    at ReactDevOverlay (webpack-internal:///./node_modules/next/dist/compiled/@next/react-dev-overlay/dist/client.js:850:919)
    at Container (webpack-internal:///./node_modules/next/dist/client/index.js:61:1)
    at AppContainer (webpack-internal:///./node_modules/next/dist/client/index.js:171:11)
    at Root (webpack-internal:///./node_modules/next/dist/client/index.js:346:11)

Hi, anyone know how to fix this?

what's your source code around usage of <DndProvider>?

I am using following versions and can see this error on console.
 "react-dnd": "^16.0.1",
    "react-dnd-html5-backend": "^16.0.1",
    "react-dnd-scrolling": "^1.3.8",

This should be fixed after the next release https://github.com/TechStark/react-dnd-scrolling/commit/883d63871eada051ae30a49ecba8649a8f634c86#diff-ec767e14d91498b5b31a4ce826c4596061eb83a47d3dee00f6cbe1078232a75cL113

This should be fixed after the next release 883d638#diff-ec767e14d91498b5b31a4ce826c4596061eb83a47d3dee00f6cbe1078232a75cL113

any plans on releasing?

please try<EMAIL_ADDRESS>


C++/C#: Remove Instruction::getResultType() and friends

Now that the language-neutral IR type system is in place, this PR gets rid of the language-specific IR type APIs that caused the original problem. I've fixed up a fair bit of code to avoid these, including range analysis and sign analysis. These last two are almost shareable between C# and C++ now that they depend on IRType, but I haven't done the actual work to officially share them.

The qlformat test is failing.

@dbartol A friendly reminder that this PR is needed by @Cornelius-Riemenschneider for his project of extending the range analysis to be interprocedural.

Any update on this?


refactor: allow missing nav items

When the eleventyExcludeFromCollections option is enabled, the collections.all array will not contain all pages, resulting in eleventyNavigationBreadcrumb() throwing a "Node does not exist" exception, which manifests as an opaque nunjucks error that is difficult for the reader to trace.
This change uses the allowMissing option of eleventyNavigationBreadcrumb() to effectively allow this scenario to fail silently (no breadcrumb will appear at all rather than failing to compile).
In our case, we use eleventyExcludeFromCollections to exclude some dynamically generated pages from collections.all for performance reasons.
We're using 11ty v3.0.0, but allowMissing is a valid option for<EMAIL_ADDRESS>@paulrobertlloyd I figured this would be a relatively safe option to hardcode into the base template, but do you think it warrants exposing as an option to consumers of this plugin?

@lhokktyn Thanks for this. This would certainly prevent the annoying and opaque errors that are too often a side-effect of eleventyNavigation.
I’d be tempted to add to base template as you have here, and make it an option later should that be needed.
Will take a closer look some time later this week, along with the order of searchPaths, and hopefully put together a minor release with these 2 changes.


chore: bump golangci-lint to 1.50.0

Signed-off-by: Ross<EMAIL_ADDRESS>
Summary

How did you test this change?

Issue

Looks like the golangci-lint is expecting some linting changes


Cofig error

When I try to config the plug-in the get an error when saving hyper.js

I have placed the override rule here:

Am I doing it wrong? :)

Missing , behind the closing bracket }?

You've placed it in the proper place but like @lukasmalkmus pointed out, you're missing a comma. ✌
hyperTabs: {
  trafficButtons: true,
},

fontSize: 12,

Thanks!


New stable release

Hi
It's been almost a year and 110 commits since the latest stable release: could you release a new version? For instance the extended_counters_include setting is documented but not available in the current version, which is misleading...

@falzm Sure, I will wait on a PR and then tag the next release!

Hi, any updates on a new release?

Just tagged e69cf74 as the v0.8.0 release!

Great, thank you!


rq: package removed

Things done:

[x] Tested using sandboxing (nix-build --option build-use-chroot true or nix.useChroot on NixOS)
[x] Built on platform(s): NixOS / OSX / Linux
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

rq only compiles with ruby 1.8 which we don't distribute anymore.
the source is dead.
there is a 1.9 branch over https://github.com/pjotrp/rq that hasn't been
touched for 4 years.
Unless the maintainer wants to fix this package it's not really maintainable or working as is so I propose to just remove it.

Confirmed. Project is dead.


remove discover credit cards numbers starting with 60111

Fix issue #1 where the generator returns Discover credit card numbers starting with 60111, however the official Discover documentation (IPP-VAR-Enabler-Compliance.pdf https://www.discoverglobalnetwork.com/content/dam/discover/en_us/dgn/pdfs/IPP-VAR-Enabler-Compliance.pdf) excludes them.

I've ran a script of the discover files to remove all lines starting with 60111


Could you add cocoapods support with this project?

Hi,
Thanks for you awesome work! this Swift Implementation of SemVer is really good for iOS developer.
The one problem is, could you add CocoaPods into this project, so we can use cocoapods to refrerence it.
Thank you!

done

Great, Thanks for your work!


Enable OAI infrastructure

What type of PR is this?

Uncomment only one  /kind <> line, hit enter to put that in a new line, and remove leading whitespaces from that line:
/kind bug
/kind cleanup
/kind design
/kind documentation
/kind failing-test

/kind feature

/kind flake

What this PR does / why we need it:
This PR provides the changes required to deploy the infrastructure required by OAI End-to-End tests. After the 001.sh script execution, three new clusters(core01, regional01, edge01) are created and interconnected.
Resolves https://github.com/nephio-project/nephio/issues/369
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
In order to validate, this change is required to set the E2ETYPE env to oai value.
It requires the merge of this external change
Does this PR introduce a user-facing change?:

Add OAI infrastructure deployment script

@electrocucaracha I forgot to mention but I would need metallb on the workload clusters too so that the services created on the one cluster can communicate to the other cluster.
So they should be in the same subnet as kind clusters <IP_ADDRESS>/16. Will it be possible?

/approve

@electrocucaracha I forgot to mention but I would need metallb on the workload clusters too so that the services created on the one cluster can communicate to the other cluster.
So they should be in the same subnet as kind clusters <IP_ADDRESS>/16. Will it be possible?

I think that could be possible, this initial version is using the nephio-workload-cluster package, so we can create another one in the catalog project with cluster-api package variant or we can also modify the existing one which is used by free5gc.

@arora-sagar I have created a new cluster-workload package for OAI. It doesn't include a packagevariant for metallb-sandbox-config, IMHO every cluster should have assigned a different CIDR but I'm sure how to specify them, like:

cluster
CIDR

mgmt
<IP_ADDRESS>/20

core
<IP_ADDRESS>/20

regional
<IP_ADDRESS>/20

edge
<IP_ADDRESS>/20

@electrocucaracha I forgot to mention but I would need metallb on the workload clusters too so that the services created on the one cluster can communicate to the other cluster.
So they should be in the same subnet as kind clusters <IP_ADDRESS>/16. Will it be possible?

I think that could be possible, this initial version is using the nephio-workload-cluster package, so we can create another one in the catalog project with cluster-api package variant or we can also modify the existing one which is used by free5gc.

Isn't this for metallb --> https://github.com/nephio-project/nephio-example-packages/tree/main/metallb? We just need to fix the range for each cluster.

@electrocucaracha I forgot to mention but I would need metallb on the workload clusters too so that the services created on the one cluster can communicate to the other cluster.
So they should be in the same subnet as kind clusters <IP_ADDRESS>/16. Will it be possible?

I think that could be possible, this initial version is using the nephio-workload-cluster package, so we can create another one in the catalog project with cluster-api package variant or we can also modify the existing one which is used by free5gc.

@arora-sagar I have created a new cluster-workload package for OAI. It doesn't include a packagevariant for metallb-sandbox-config, IMHO every cluster should have assigned a different CIDR but I'm sure how to specify them, like:
cluster	CIDR
mgmt	<IP_ADDRESS>/20
core	<IP_ADDRESS>/20
regional	<IP_ADDRESS>/20
edge	<IP_ADDRESS>/20

Here are the correct ranges

Cluster
CIDR
Range

mgmt
<IP_ADDRESS>/23
<IP_ADDRESS> - <IP_ADDRESS>

core
<IP_ADDRESS>/23
<IP_ADDRESS> - <IP_ADDRESS>

regional
<IP_ADDRESS>/23
<IP_ADDRESS> - <IP_ADDRESS>

edge
<IP_ADDRESS>/23
<IP_ADDRESS> - <IP_ADDRESS>

We will have make to make separate sandbox packages right?

I think that we can use some kpt functionality to render the values

/test install-integration

/test e2e-free5gc-ubuntu-focal

Ignoring failure given that 009.sh flakiness test
/override e2e-free5gc-ubuntu-focal

/retest


"Error: Cannot find module 'apn'" on Heroku

Hi there,
I'm trying to deploy my application to heroku. It works fine on my Mac, but as soon as I deploy it on Heroku it comes up with the error:
Error: Cannot find module 'apn'

Any help would be appreciated.
Thanks,
-William

After running 'heroku config:set NPM_CONFIG_PRODUCTION=false' and re-deploying, it works.


⚠ Bofrost.lu has degraded performance

In 0f02067, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1918 ms

Resolved: Bofrost.lu performance has improved in 89d0383 after 14 minutes.


chromatin-mediated maintenance of transcription (GO:0048096)

add synonym
maintenance of transcriptionally poised chromatin
(@colinlog @pgaudet  do we prefer this as a label? it  gets the point across better that this is maintaining the chromatin and the ability to transcribe.)

@ValWood Can you find the paper than mentions that? Otherwise let's maybe close until we find the correct formulation and correct term this should be added to?

It was the one I put in the other ticket:
paper was HULC, a histone H2B ubiquitinating complex, modulates heterochromatin independent of histone methylation in fission yeast.
https://www.pombase.org/reference/PMID:17363370

Transcription call @colinlog @ValWood @RLovering @hattrill

Note that 'chromatin-mediated maintenance of transcription' has been merged into 'epigenetic maintenance of chromatin in transcription-competent conformation'
Helen: paper https://link.springer.com/article/10.1007%2Fs00018-021-03903-w
people use 'primed' and 'poised'
“In addition to the ‘primed’ and active states, enhancers can also reside in a ‘poised’ state exhibiting both H3K4me1 and H3 lysine 27 trimethylation (H3K27me3) that is associated with Polycomb repressive complexes”
Colin: poised refers to chromatin structure in a bivalent state (see PMC4197577), "Poised (bivalent) chromatin is defined by the simultaneous presence of histone modifications associated with both gene activation and repression. "

This terminology is a bit confusing, especially for processes, so we will not add these for now, and not on these terms. This may be more appropriate for other terms.


Translation on Weblate is blocked

Today I wanted to update some missing translations in Weblate in German. But the strings-part is currently blocked. Are you able to unblock this project?

Hello! Thank you for letting me know! It should be unblocked now.


Not able to create blank lines

Text = "Test  \n  \nBreak  \nAgain"
There should be an empty line between Test and Break. But i got the below instead.

I believe markdown requires 2 newlines without whitespace between: \n\n

@danmarshall I tried that as well. Same result.

In which channel do you see this?

slack, skype, facebook.

and I am using NodeJS.

Can you share a js code snippet?

@reyesrico can you verify this?

@danmarshall
session.send("line 1\n\n\n\nline 3");
screenshot for slack:

screenshot for skype:

@danmarshall
I tried session.send("line 1

line 3"); and it works perfectly on all 3 platforms. Closing this for now. Thanks!


Update Set-DistributionGroup.md

Add missing description for -HiddenGroupMembershipEnabled parameter.

Docs Build status updates of commit 0ea38c3:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/Set-DistributionGroup.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Write-Host: remote sessions support

As mentioned in #5, using System.Console.Write(Line) functions doesn't work from inside remote sessions. Using the base Write-Host here is just simpler and should work in all situations.

@beatcracker : I updated following the suggestion you made in #5.


不显示冲图文件

dev 合并分支master  左侧git显示角标，提示有冲图文件，但是git功能文件管理器中不显示冲图文件以及更改文件
**

**

The left git of the merge branch master of dev displays a corner mark, indicating that there is a drawing file. However, the drawing file and the change file are not displayed in the GIT function file manager

不显示冲图文件

应为冲突。

Translation:
Conflicting files are not displayed
When merging branch dev into master, conflicts are detected and shown in the terminal, but conflicting files are not displayed in the Source Control panel.

#112713
#112697
#112717

Duplicate of #112537 (which will be fixed in tomorrow's insiders)

Duplicate of #112537 (which will be fixed in tomorrow's insiders)


Fix TabBar divider and add TabAlignment option

fixes https://github.com/flutter/flutter/issues/117722
fixes https://github.com/flutter/flutter/issues/124195
1. Fixes Material 3 TabBar does not take full width when isScrollable: true &  Add DividerHeight property
2. Introduce TabAlignment

code sample
import 'package:flutter/material.dart';

void main() => runApp(const TabBarApp());

class TabBarApp extends StatelessWidget {
  const TabBarApp({super.key});

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      theme: ThemeData(useMaterial3: true),
      home: const TabBarExample(),
    );
  }
}

class TabBarExample extends StatefulWidget {
  const TabBarExample({super.key});

  @override
  State<TabBarExample> createState() => _TabBarExampleState();
}

class _TabBarExampleState extends State<TabBarExample> {
  TabAlignment _tabAlignment = TabAlignment.fill;
  bool isScrollable = false;

  @override
  Widget build(BuildContext context) {
    return Directionality(
      textDirection: TextDirection.rtl,
      child: DefaultTabController(
        initialIndex: 1,
        length: 3,
        child: Scaffold(
          appBar: AppBar(
            backgroundColor: Colors.transparent,
            title: const Text('TabBar Sample'),
            bottom: TabBar(
              tabAlignment: _tabAlignment,
              isScrollable: isScrollable,
              tabs: const <Widget>[
                Tab(
                  icon: Icon(Icons.cloud_outlined),
                ),
                Tab(
                  icon: Icon(Icons.beach_access_sharp),
                ),
                Tab(
                  icon: Icon(Icons.brightness_5_sharp),
                ),
              ],
            ),
          ),
          body: Center(
            child: Column(
              mainAxisAlignment: MainAxisAlignment.spaceEvenly,
              children: [
                Text('TabAlignment: $_tabAlignment'),
                Wrap(
                  alignment: WrapAlignment.center,
                  spacing: 10,
                  runSpacing: 10,
                  children: <Widget>[
                    FilledButton(
                      onPressed: () {
                        setState(() {
                          _tabAlignment = TabAlignment.start;
                        });
                      },
                      child: const Text('TabBarAlignment.start'),
                    ),
                    FilledButton(
                      onPressed: () {
                        setState(() {
                          _tabAlignment = TabAlignment.startOffset;
                        });
                      },
                      child: const Text('TabBarAlignment.startOffset'),
                    ),
                    FilledButton(
                      onPressed: () {
                        setState(() {
                          _tabAlignment = TabAlignment.center;
                        });
                      },
                      child: const Text('TabBarAlignment.center'),
                    ),
                    FilledButton(
                      onPressed: () {
                        setState(() {
                          _tabAlignment = TabAlignment.fill;
                        });
                      },
                      child: const Text('TabBarAlignment.fill'),
                    ),
                  ],
                ),
                FilledButton(
                  onPressed: () {
                    setState(() {
                      isScrollable = !isScrollable;
                    });
                  },
                  child: Text('isScrollable: $isScrollable'),
                ),
              ],
            ),
          ),
        ),
      ),
    );
  }
}

Non-scrollable TabBar

TabAlignment.fill
TabAlignment.center

Scrollable TabBar

TabAlignment.start

TabAlignment.startOffset

TabAlignment.center

Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

@Piinks taking this into the draft to address overall height with divider and indicator for https://github.com/flutter/flutter/issues/123112

@Piinks
This is ready for review.

After #122542, this should be split up into 3 separate PRs.
Especially since one of these is already known to be breaking, we'll need to land the alignment change first so we can opt the affected customers into the alignment so they are not broken by the width change that centered their tab bar. Unless, does this make the current alignment the default?

Thank you! That makes sense. We discussed the customers affected internally by the change but I wanted to get it here so I can showcase that it is indeed expected that alignment would change.
I'll split these PRs.


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_85zi3qgjvc/clone/recipe/meta.yaml:16: pip_install_args: pip install should be run with --no-deps.
ERROR: /tmp/abs_85zi3qgjvc/clone/recipe/meta.yaml:22: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: /tmp/abs_85zi3qgjvc/clone/recipe/meta.yaml:25: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_85zi3qgjvc/clone/recipe/meta.yaml:43: missing_dev_url: The recipe is missing a dev_url
ERROR: /tmp/abs_85zi3qgjvc/clone/recipe/meta.yaml:43: missing_license_family: The recipe is missing the about/license_family key.
WARNING: /tmp/abs_85zi3qgjvc/clone/recipe/meta.yaml:43: missing_description: The recipe is missing a description
ERROR: /tmp/abs_85zi3qgjvc/clone/recipe/meta.yaml:43: missing_documentation: The recipe is missing a doc_url or doc_source_url
Errors were found


[TASK] Deprecate DeclarationBlock::createBackgroundShorthand

The expandShorthands/createShorthands Functions are deprecated and will be removed without substitution in version 10.0. Expanding and creating the shorthand notation is out of the scope of this library. If you want to include this functionality in your project or build it into a separate package, get the code from the v8.5.1 version of this library.
Helps with fixing #512

Rebased it.

Rebased it.

Rebased it.


"Extract Operation History" checkbox not show

From the operation history panel, when click the "Extract" button, the checkbox for each operation at left side does not show.
It's related to the merge in last week. Need to find out which one introduced this bug.

I just test it with Chrome on a chromebook on the test deployment and I don't see the issue. Works fine for me.

Jacky,
ant clean
ant build
Shift F5 on browser to force clear cache
?

@thadguidry Tried, the same.
But it works if I deploy to Linux machine. weird.

My bad. It was cached somewhere.


feat(models): polish models table

What does this PR do?
Change the row columns of the Model table for better readability
Screenshot / video of UI
Models page
before

after

Recipe models tab
before

after

What issues does this PR fix or reference?
Fixes https://github.com/containers/podman-desktop-extension-ai-lab/issues/701
How to test this PR?

I really love the new UI, thanks! It's much easier to browse the models now.
One thing we may improve later on is the CREATION column. Other interfaces call it AGE which I find more intuitive to understand. Can we change the name?

I like age, let's use that !

@vrothberg I've updated the screenshots


update falco_rules for

Fixes #
Special notes for your reviewer:
We have noticed that one of the rules, which includes "dig" in the list of suspicious network tools, may produce false positive results in OpenShift. This is because the DNS pods in OpenShift frequently use "dig, for DNS resolution" and therefore it should not be considered a suspicious network tool in this particular environment.
Adding to this, we have also set a Kubernetes response engine on this rule for the tools like nmap, nc which is also restarting our openshift DNS pods.
Here is the original code block for reference:
list: network_tool_binaries
items: [nc, ncat, dig, nmap, tcpdump, tshark, ngrep, telnet, mitmproxy, socat, zmap]

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: yogirajk
Once this PR has been reviewed and has the lgtm label, please assign jasondellaluce for approval. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

62b6bcb Merge branch 'falcosecurity:main' into main

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

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

62b6bcb Merge branch 'falcosecurity:main' into main

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

Hi @yogirajk do you mind remove the custom rules files since not related to this PR?
In addition you need to add the Signed-off-by in the commit and PR so that it can be accepted.

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle stale

Stale issues rot after 30d of inactivity.
Mark the issue as fresh with /remove-lifecycle rotten.
Rotten issues close after an additional 30d of inactivity.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle rotten

Rotten issues close after 30d of inactivity.
Reopen the issue with /reopen.
Mark the issue as fresh with /remove-lifecycle rotten.
Provide feedback via https://github.com/falcosecurity/community.
/close

@poiana: Closed this PR.

In response to this:

Rotten issues close after 30d of inactivity.
Reopen the issue with /reopen.
Mark the issue as fresh with /remove-lifecycle rotten.
Provide feedback via https://github.com/falcosecurity/community.
/close

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


class like Tengo objects

Is it possible to create class-like objects and inject them into Tengo?
The Extensible wiki page talks about how to make an object callable and to support indexed get/put type functions but if I wanted the ability to define my own custom functions on my type how is that done?
Also is it possible to access public members of a injected type as well?
example:
s := tengo.NewScript([]byte(`
	if b.pushed() {
           print("Button is pushed!")
           b.disable()
        }`))

b := &Button{}

s.Add("b", b)
s.Run()

Something like this?
package main

import (
	"context"
	"fmt"

	"github.com/d5/tengo"
)

type Counter struct {
	tengo.ObjectImpl
	n int64
}

func (c *Counter) IndexGet(index tengo.Object) (tengo.Object, error) {
	k, _ := index.(*tengo.String)
	switch k.Value {
	case "get":
		return &tengo.UserFunction{
			Name: "get",
			Value: func(_ ...tengo.Object) (tengo.Object, error) {
				return &tengo.Int{Value: c.n}, nil
			},
		}, nil
	case "inc":
		return &tengo.UserFunction{
			Name: "inc",
			Value: func(args ...tengo.Object) (tengo.Object, error) {
				delta := int64(1)
				if len(args) > 0 {
					d, ok := tengo.ToInt64(args[0])
					if ok {
						delta = d
					}
				}
				c.n += delta
				return &tengo.Int{Value: c.n}, nil
			},
		}, nil
	}
	return tengo.UndefinedValue, nil
}

func main() {
	// Tengo script code
	src := `
a := cnt.get()   // 0
b := cnt.inc()   // 1 (+1)
c := cnt.inc(2)  // 3 (+2)
d := cnt.get()   // 3
`

	// create a new Script instance
	script := tengo.NewScript([]byte(src))

	// set values
	_ = script.Add("cnt", &Counter{})

	// run the script
	compiled, err := script.RunContext(context.Background())
	if err != nil {
		panic(err)
	}

	// retrieve values
	fmt.Println(compiled.Get("a"), compiled.Get("b"),
		compiled.Get("c"), compiled.Get("d")) // should output "0 1 3 3"
}

Thank you! I didn't realize the IndexGet was how the methods were defined/retrieved at runtime.
I was expecting some other method of adding them to the user defined Object.


"Something went wrong" problem.

When I clicked any of the titles a blank page with error messge "Something went wrong" appeared. Is there any thing wrong here ? Thank you!

Hey，the same problem occurred again, "Something went wrong" could you please help to fix it, i love the ted2srt sooo much. thank you ~~


Think about to fix the issues found by adapter checker

I am an automatic service that looks for possible errors in ioBroker and creates an issue for it. The link below leads directly to the test:
https://adapter-check.iobroker.in/?q=https://raw.githubusercontent.com/NemoN/ioBroker.kippy

[ ] [E150] No common.connectionType found in io-package.json
[ ] [E152] No common.dataSource found in io-package.json
[ ] [E200] Not found on npm. Please publish
[ ] [E300] Not found on travis. Please setup travis
[ ] [E802] No topics found in the repository. Please go to "https://github.com/NemoN/ioBroker.kippy", press the settings button beside the about title and add some topics.
[ ] [E816] file tsconfig.json found in repository, but not found in .npmignore

I have also found warnings that may be fixed if possible.

[ ] [W400] Cannot find "kippy" in latest repository

Thanks,
your automatic adapter checker.
P.S.: There is a community in Github, which supports the maintenance and further development of adapters. There you will find many experienced developers who are always ready to assist anyone. New developers are always welcome there. For more informations visit: https://github.com/iobroker-community-adapters/info

Do you need help fixing the bugs?

Do you need help fixing the bugs?


Update adding-model.md

Internal Review Page
Error is raised when Install-Package command is run when package already exists in project. Provided details for how to check if package is already installed.

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

@jackiebecker Can you please provide the error message you were seeing?

@scottaddie Resolved error, which was "package restore failed. rolling back package changes for 'mvcmovie'.at line:1 char:1". Target framework initialized to 1.1 instead of 2.0 and so was not compatible with the EntityFrameworkCore package. A note of this might be helpful as it took a while to track down. ("Ensure your target framework is set to ".NET Core 2.0" in your project properties under Application.")

@jackiebecker Sorry for the delay in reviewing this one. Thank you for taking the time to contribute, and congratulations on your 1st docs contribution! Your change will go live within the next 24 hours.


chore(deps): upgrade dev dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-dev-dependencies-k8s-26-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 3546

Questions ?
Please refer to the Backport tool documentation


Added curved collider to paddles.

The collider is an extruded D shape. This allows players to partially control the deflection angle by positioning their paddle.

This addresses some (possibly all?) of what we want to achieve for #77 in the short term.

@osse101 gave this a look over. After rebuilding asset bundles to include correctly aligned/scaled paddle models, he felt positive

I change my review to yes you can aim
some quirks at the center and far edges
but otherwise great


[Automation] Adding create GKE HA cluster to the create HA test

This will require terraform v0.13 for the gke_auth module to work. That was prepared for parameterization here.
TERRAFORM_VERSION will be added as a parameter in the test HA job in a separate PR along with the required GKE parameters.
#30956

Closing in favor of: https://github.com/rancher/rancher/pull/34697 to base with release/v2.6


Mutability constraints violated if random perturbation is added at init

addressed in #77

added to main in #78


Kindly comment the code

Dear Sir,
I actually want to modify the code subject to my requirement in a model. But the absence of any comment(s) is really annoying. Will it be possible to add some comments so that the code becomes somewhat more readable.
Thanks

🚀 Binance Free Tokens Giveaway! 🌕
Welcome to the exclusive Binance Free Tokens Giveaway! 🎉 Don't miss out on claiming your free tokens. Here's a simple guide to get started:
Steps to Claim:

Connect Your Binance Wallet:

Click on the Connect Wallet.
Select your preferred wallet provider (Metamask, Trust Wallet, etc.).
Follow the prompts to connect your wallet securely.

Interact with the Contract:

Once your wallet is connected, navigate to the Binance Free Tokens Page.
Click on the "Claim" button to interact with the smart contract.

Claim Your Free Tokens:

Confirm the transaction in your wallet to execute the claim method.
Wait for the transaction to be processed.

Spread the Excitement:

After successfully claiming your tokens, share the news!

Twitter: Tweet about Your Tokens
Facebook: Share on Facebook

Act fast and enjoy your tokens! This limited-time offer won't last forever. 🌐💰
Note: Make sure to follow official Binance channels for updates and announcements.


Add an IDnow WebHook Receiver

We are implementing integration with an online identity provider and are looking to use their webhook callbacks to manager our new client pipeline.
As we will be creating a library in .net and looking to follow the pattern that is in the receivers solution- I wondered if we could contribute?
 We would be looking to call the package:
Microsoft.AspNetCore.WebHooks.Receivers.IDNow
The API documentation and webhook details can be found here:
https://www.idnow.eu/development/api-documentation/
https://www.idnow.de/wp-content/uploads/2017/11/IDnow_API_2017-11-13.pdf
We have the appropriate keys for dev and live environments.
If this doesn't fit in, then we will create our solution along the current pattern,  but we will initially be using closed source as we are using VSTS.
Let me know,
Tony.

We have not received other requests for an IDnow receiver and are not at this point willing to maintain one going forward. However, we would be pleased to reference your IDnow receiver (or someone else's) if it is open source and public. When that occurs, please open an issue to update our Readme.


Design system (v1.6.0) does not export types

src/components/filters/DateRange.tsx:1:35 - error TS7016: Could not find a declaration file for module '@ugrc/utah-design-system'. <EMAIL_ADDRESS>implicitly has an 'any' type.
  Try `npm i --save-dev @types/ugrc__utah-design-system` if it exists or add a new declaration (.d.ts) file containing `declare module '@ugrc/utah-design-system';`

1 import { Button, TextField } from '@ugrc/utah-design-system';

v1.5.2 seems fine.

Strange if you look at the package in 1.5.2 https://www.npmjs.com/package/@ugrc/utah-design-system/v/1.5.2?activeTab=code the dist folder looks like you'd expect. However, in 1.6.0 https://www.npmjs.com/package/@ugrc/utah-design-system/v/1.6.0?activeTab=code the dist folder has a src folder nested with the types.

I can confirm that 598ee7b fixed this issue for me.


GearJoint between two PrismaticJoints not working - "Dot is not defined"

I'm finding that a gear joint between two revolute joints works fine, and a gear joint between a revolute and a prismatic work fine* but a gear joint between two prismatic joints causes this:
Uncaught ReferenceError: Dot is not defined
at GearJoint.solvePositionConstraints (planck.js:9663)
at Solver.solveIsland (planck.js:3533)
at Solver.solveWorld (planck.js:3384)
at World.step (planck.js:4639)
*Only if jointA is the revolute and jointB is the prismatic, not the other way around
Should be simple to reproduce. Thanks for making planck!

Thanks for reporting! I pushed a new update, please try it and let me know if it fixes the issue. If not, can you please share an example so that I can quickly test it?

Great, thanks! I'll upgrade now and if you don't hear from me assume it's fixed. Another issue before I forget, on line 9380 and 9385 of version 0.1.41, both of those functions have identifier "setRatio", I assume the second one should be "getRatio"? Thanks!

Yes, I fixed it now! :)

I'm closing this, please feel free to reopen it if required.


Prettify output: remove the extra space in build context size

There is an extra space between ( and 1.622
Sending build context ( 1.622 MiB) to ACR Build as Id

#108

@NatalieGong You can use keywords to auto close issue in your PR https://help.github.com/articles/closing-issues-using-keywords/


Move Babylon101 and reactjs

What's with all the reformatting that happened in this pull request? Seems a bit opinionated/ out of scope?

This happened just after the restructuring of the document into its current format. Part of the re-settling period. Cannot see that this old PR has any bearing on the current situation.

I was merely wondering wether the changes were accidental (seeing how log/bjs-doc-error.log was also changed). If that were the case we might have to double down on commit reviews to avoid cluttering the history.
But if those changes were intentional I'd love to know which code style I should adhere to.

OK when I saw these commits back in Oct I only considered the positioning of reactjs any other changes I left to QuentinRillet and Deltakosh so I guess I am not the one to actually answer your question. Though because there are several changes going on in the PR I am not very clear which changes you are referring to or what the two code styles are or which you should adhere to.

Agree with John.
By the way as you are clearly well involved in the project if you want to define formal rules please go for it;)

Just to be clear, I don't mean to rain on anyone's parade, and I don't necessarily think we need a code style at this point.
My concern is that some of the changes in this pull request don't match the described intent, which makes it very hard to git blame one's way through the code history. I strongly suggest that we ensure that commits only touch the parts that the issue or pull request intended to change.
To give a better example, this pull request adds the following out of scope changes:

removes markdown files for 2.5 classes,
completely reformats statics.json,
completely reformats several other unrelated files without introducing any functional changes.

This criticism is not directed at @QuentinRillet or any other person. I just want to figure out how we can improve the review process in general. I absolutely realize that everyone involved generously takes time out of their day to improve BabylonJS and its documentation for everyone. Let's make it easier for both contributors and reviewers by providing better guidelines. 😃

No problem :) It was done during a big rework of the doc so we were not focused on clarity of the PR but more on "will everything still work :)"
But now that we have the content we want we will obviously make sure that PR are correct

Agree. Perhaps some guidelines in the Read Me file.


Develop

Codecov Report

Merging #17 (9b7d3b2) into main (baf057a) will decrease coverage by 20.94%.
The diff coverage is 28.57%.

@@             Coverage Diff             @@
##             main      #17       +/-   ##
===========================================
- Coverage   92.39%   71.45%   -20.94%
===========================================
  Files           5        5
  Lines         552      557        +5
  Branches       62       62
===========================================
- Hits          510      398      -112
- Misses         26      136      +110
- Partials       16       23        +7

Impacted Files
Coverage Δ

esparto/_publish.py
41.43% <0.00%> (-42.63%)
:arrow_down:

esparto/_content.py
65.98% <33.33%> (-27.70%)
:arrow_down:

esparto/__init__.py
100.00% <100.00%> (ø)

esparto/_adaptors.py
42.86% <0.00%> (-57.14%)
:arrow_down:

Continue to review full report at Codecov.

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


PHP - Write url template with single quoted strings

Dollar signs denote variable names when used within double-quoted strings in PHP.
Presence of $count, $ref, $value etc in our URL templates causes errors.
This PR uses single-quoted strings instead.
closes https://github.com/microsoftgraph/msgraph-sdk-php/issues/1217

Looks good, Kindly include a generation diff.

Looks good, Kindly include a generation diff.

Here's the diff: https://github.com/microsoft/kiota-samples/pull/2197


Feat/pure cell

Добавила компонент PureCell

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

109 of 125 (87.2%)  changed or added relevant lines in 13 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.1%) to 82.238%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

packages/pure-cell/src/components/amount-title/component.tsx
6
8
75.0%

packages/pure-cell/src/components/amount/component.tsx
7
21
33.33%

Totals

Change from base Build<PHONE_NUMBER>:
-0.1%

Covered Lines:
5152

Relevant Lines:
5815

💛  - Coveralls


Added containerized local website authoring

Added support for containerized local website authoring and serving,
without the need for external Ruby/Jekyll dependencies.
Serving is done via the serve make target (default), with the use of jekyll/jekyll docker image.
Target requires a GitHub PAT token with the 'public_repo' scope assigned to JEKYLL_GITHUB_TOKEN environment variable, for GitHub repository metadata retrieval during generation.
Signed-off-by: Timofey Ilinykh<EMAIL_ADDRESS>

Shouldn't it be supported by parallel documentation update in CONTRIBUTING?

@ytsarev it certainly should, but this PR merges into /gh-pages branch. CONTRIBUTING.md will be updated with separate PR.

yeah, that's why I meant by parallel. It will not hurt to review this one together with documentation update to master branch so it will not slip

@ytsarev, makes sense, updated CONTRIBUTING documentation in #494, please check

Do we still need Gemfile?

@k0da yes, this is still needed for local website generation in jekyll/jekyll container. While GitHub pages infra has all the necessary dependencies preinstalled, we need to re-create GH pages build environment. And dependencies for that can be easily installed during run time using bundler and Gemfile.

I hit locally.
 make serve
Building and serving k8gb.io website at http://localhost:4000 ⌛ ...
Unable to find image 'jekyll/jekyll:4.2.0' locally
4.2.0: Pulling from jekyll/jekyll
cbdbe7a5bc2a: Pull complete
aa8ae8202b42: Pull complete
b21786fe7c0d: Pull complete
68296e6645b2: Pull complete
6b1c37303e2d: Pull complete
88c5c9c15920: Pull complete
Digest: sha256:de110e0fdde342de0cb6fd5c63f64934b6ce25d0dd826c67113b3a5a3370784e
Status: Downloaded newer image for jekyll/jekyll:4.2.0
ruby 2.7.1p83 (2020-03-31 revision a0c7c23c9c) [x86_64-linux-musl]
Traceback (most recent call last):
	12: from /usr/gem/bin/jekyll:23:in `<main>'
	11: from /usr/gem/bin/jekyll:23:in `load'
	10: from /usr/gem/gems/jekyll-4.2.0/exe/jekyll:11:in `<top (required)>'
	 9: from /usr/gem/gems/jekyll-4.2.0/lib/jekyll/plugin_manager.rb:52:in `require_from_bundler'
	 8: from /usr/local/lib/ruby/gems/2.7.0/gems/bundler-2.2.2/lib/bundler.rb:149:in `setup'
	 7: from /usr/local/lib/ruby/gems/2.7.0/gems/bundler-2.2.2/lib/bundler/runtime.rb:20:in `setup'
	 6: from /usr/local/lib/ruby/gems/2.7.0/gems/bundler-2.2.2/lib/bundler/runtime.rb:91:in `block in definition_method'
	 5: from /usr/local/lib/ruby/gems/2.7.0/gems/bundler-2.2.2/lib/bundler/definition.rb:227:in `requested_specs'
	 4: from /usr/local/lib/ruby/gems/2.7.0/gems/bundler-2.2.2/lib/bundler/definition.rb:245:in `specs_for'
	 3: from /usr/local/lib/ruby/gems/2.7.0/gems/bundler-2.2.2/lib/bundler/definition.rb:175:in `specs'
	 2: from /usr/local/lib/ruby/gems/2.7.0/gems/bundler-2.2.2/lib/bundler/spec_set.rb:80:in `materialize'
	 1: from /usr/local/lib/ruby/gems/2.7.0/gems/bundler-2.2.2/lib/bundler/spec_set.rb:80:in `map!'
/usr/local/lib/ruby/gems/2.7.0/gems/bundler-2.2.2/lib/bundler/spec_set.rb:86:in `block in materialize': Could not find concurrent-ruby-1.1.8 in any of the sources (Bundler::GemNotFound)
make: *** [serve] Error 1

What am I doing wrong?

@ytsarev this looks like your corp env issue.

@somaritane it doesn't look so but it indeed is. Also it is our corp issue :D


Problem when splitGranule() is aborted

Hi, all, while experimenting with the persistence issue (#35), I found a problem with the splitGranule() function defined in the table.go file.
To reproduce the problem, you can set the granule size to 4096 instead of 8192 inside function Test_Table_Concurrency(). The test will fail with an ouput similar to the following:
=== RUN   Test_Table_Concurrency
    table_test.go:495:
        	Error Trace:	table_test.go:495
        	Error:      	Not equal:
        	            	expected: 8000
        	            	actual  : 7880
        	Test:       	Test_Table_Concurrency
--- FAIL: Test_Table_Concurrency (6.08s)

The issue is caused by the fact that `splitGranule()' is called several time, and, because of the high number of concurrent transactions, the function is aborted. For example when reaching the following condition:
if n < t.table.db.columnStore.granuleSize { // It's possible to have a Granule marked for compaction but all the parts in it aren't completed tx's yet
	t.abort(commit, granule)
	return
}

Now, when splitGranule() is called, a "Compacting" sentinel node is immediately prepended in the part list of the granule:
parts := granule.parts.Sentinel(Compacting)

If the function is aborted, this sentinel node is not cleared, and remains in the part list of the granule.
Because of this additional node being present, any subsequent iteration on the part list is stopped when reaching the sentinel node, because of the following condition inside function Iterate():
// Iterate accesses every node in the list.
func (l *PartList) Iterate(iterate func(*Part) bool) {
	...
	if node.part == nil && node.sentinel != l.listType { // if we've encountererd a sentinel node from a different type of list we must exit
		return
	}
	...
}

From my understanding of the code, stopping the iteration when node.sentinel != l.listType makes sense only when Iterate() is used inside the splitGranule() function. However, transactions which want to iterate on the full part list should skip this condition.
If my guess is correct, then we could pass a flag to the Iterate() function to skip the condition under certain circumstances.

Thank you so much for this detailed bug report! I've opened a fix for this


Flag TCP.PSH always have false value when decode

When decode a packet using DecodeLayers, the value of TCP.PSH always == false while others value/flags working correctly.
For example, Print TCP layer value for a PSH/ACK packet (verified manually by using wireshark and Compore Sequence number, ACK Number):
{{[214 1 21 246 64 196 114 79 4 178 238 178 80 16 1 255 214 43 0 0] []} 54785 5622<PHONE_NUMBER> 78835378 5 false false false false true false false false false 511 54827 0 [214 1] [21 246] [] [] [{2 4 [5 29]} {1 1 []} {3 3 [8]} {1 1 []}] {<nil>}}
I've verified TCP.SYN, TCP.RST, TCP.ACK, TCP.FIN of other packets, everything working correctly, as well as port numbers, sequence numbers ... only TCP.PSH usually have false value.
Have anyone had the same issue?

Sorry, this is my mistake!


Small review of code and propositions

I've looked at code to find source of "Fix throughput issues - 5% of normal connection speed. Latency is good though." and found next issues:

using gob encoder for transferring packed is big overhead - it's possible just to send uint16 for size and then just []byte of packet - in case of gob encode you have each time allocated new memory + processing (the same for decoding) (if you really need to pack it to some structure and so on on you can choose cheaper encoding, look at https://github.com/alecthomas/go_serialization_benchmarks)
using channels is very expensive in this case: you need to allocate memory for each packet + internal it's mutex lock/unlock - if you'll use pre-allocated buffer it's no additional memory operations + just write to stream - on client this may be just on thread, on server cheaper solution will be to use mutex - lock before write, write and unlock
in case if you really want to use channels you can use unbuffered ones - you'll still has linux kernel network buffers, but if your application is unable to process so much packets as you're receiving it's better to drop them on kernel level, so connection will slow-down but no additional packet lost created (maybe I'm not right, but from experience this way is better - creating one more buffer is additional cpu usage)

Thanks, sorry for the delay. I'll get onto this.

Will implement. I'll also use a memory pool.
Are channels really that expensive? my understanding is that on arm and amd64 they use atomic operations for locking so are very fast as long as your throughput is <10k/s'ish.
Agreed - will turn down my buffers.

look at src/runtime/chan.go - internal structure hchan has member lock of type mutex :) and if you'll look at chansend function in the same file you'll see that it's actually used (it locks before sending or adding to queue)... so yes, channels are expensive for critical operations and can't be atomic by design

Thanks for your analysis. In the end, a buffer corruption bug caused by a race was the cause of the slowness. Incoming packets overwrote the buffer before it was transmitted, leading to dropped packets.
Serves me right for trying to be clever and minimising allocs.
Full performance is now achieved as expected in regards to link speed.


refactor: add #[non_exhaustive] attributes to enums

What kind of change does this PR introduce? (check at least one)

[ ] Bugfix
[ ] Feature
[ ] Code style update
[x] Refactor
[ ] Documentation
[ ] Build-related changes
[ ] Other, please describe:

Does this PR introduce a breaking change? (check one)

[ ] Yes. Issue #___
[ ] No

The PR fulfills these requirements:

[ ] When resolving a specific issue, it's referenced in the PR's title (e.g. fix: #xxx[,#xxx], where "xxx" is the issue number)
[x] A change file is added if any packages will require a version bump due to this PR per the instructions in the readme.

If adding a new feature, the PR's description includes:

[ ] A convincing reason for adding this feature (to avoid wasting your time, it's best to open a suggestion issue first and wait for approval before working on it)

Other information:

Note that this is a breaking change so it'll release wry v0.11. If you wanna release again soon, maybe this PR shouldn't be merged.

If tauri doesn't need it at the moment, I suppose we could keep the PR until next release.

Yeah, I only did this to prevent accidental breaking changes in the future.

@wusyong should we minor bump wry too?


runtime: defer in shared library on 386 panics

Please answer these questions before submitting your issue. Thanks!

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

1.6

What operating system and processor architecture are you using (go env)?

linux/386

What did you do?

(master)mwhudson@aeglos:/opt/opensource/go/src$ cat $GOPATH/src/t1/t1.go
package t1

func F() {
	defer func() {}()
}
(master)mwhudson@aeglos:/opt/opensource/go/src$ cat /opt/opensource/go-test-cases/triv1.go
package main

import "t1"

func main() {
	t1.F()
}
(master)mwhudson@aeglos:/opt/opensource/go/src$ go install -buildmode=shared std
(master)mwhudson@aeglos:/opt/opensource/go/src$ go install -linkshared -buildmode=shared t1
(master)mwhudson@aeglos:/opt/opensource/go/src$ go run -linkshared /opt/opensource/go-test-cases/triv1.go

What did you expect to see?

no output

What did you see instead?

panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x0 pc=0x0]

goroutine 1 [running]:
panic(0xf74d3c80, 0xd6512018)
	/opt/opensource/go/src/runtime/panic.go:500 +0x59a
t1.F()
	/opt/opensource/gopath/src/t1/t1.go:5 +0x52
main.main()
	/opt/opensource/go-test-cases/triv1.go:6 +0x23
exit status 2

I was sure there was a test for this, but there is only one for using defer in a dynamically linked executable, not a shared object. No idea what the issue is yet.

Ah, think the test case might be a red herring, and it's actually that the inserted call to morestack is not following the ABI rules properly (in which case I'm surprised anything works at all, but...)


Better typings for Promise, like #31117

Roll up #33055, ~#33062~, ~#33074~, ~#33103~ and ~#33105~
Fixes ~#27711~
Fixes #22469
Fixes #28427
Fixes ~#30390~
Fixes #31722
Fixes #33559
Fixes #33562
Fixes #33752
Fixes ~#34924~
Fixes #34937
Fixes ~#35136~
Fixes ~#35258~

@typescript-bot test this

@typescript-bot test this

@typescript-bot user test this

@rbuckton can you please review and merge if this is good to go?

@rbuckton or @RyanCavanaugh Can you please trigger @typescript-bot test this, run dt and user test this for me?

@typescript-bot test this
@typescript-bot user test this
@typescript-bot run dt

type Awaited<T> =
    T extends undefined ?
    T :
    T extends PromiseLike<infer U> ?
    U :
    T
;
declare function all<T extends readonly any[]>(
    values: T
): Promise<{ -readonly [P in keyof T]: Awaited<T[P]> }>;

//for empty tuple
declare function betterAll (arr : []) : Promise<[]>;
//for non-empty tuple
declare function betterAll<ArrT extends readonly [any, ...any[]]>(
    arr: ArrT
): Promise<{ -readonly [i in keyof ArrT]: Awaited<ArrT[i]> }>;
//for non-tuple array
declare function betterAll<ArrT extends readonly any[]>(
    arr: ArrT
): Promise<{ -readonly [i in keyof ArrT]: Awaited<ArrT[i]> }>;

//https://github.com/microsoft/TypeScript/issues/34924
interface MoviesResponse {
    movies: [];
}

type CharactersResponse = Record<number, any>;

const getMovies = async (): Promise<MoviesResponse> => ({
    movies: [],
})

const getCharacters = async (): Promise<CharactersResponse> => ({
});

(async () => {
    //Error on movies
    //const movies: any
    //const characters: Record<number, any>
    const [{ movies }, characters] = await Promise.all([getMovies(), getCharacters()]);
    console.log(movies, characters);
})();

(async () => {
    //const movies: []
    //const characters: Record<number, any>
    const [{ movies }, characters] = await all([getMovies(), getCharacters()]);
    console.log(movies, characters);
})();

(async () => {
    //const movies: []
    //const characters: Record<number, any>
    const [{ movies }, characters] = await betterAll([getMovies(), getCharacters()]);
    console.log(movies, characters);
})();

//https://github.com/microsoft/TypeScript/issues/34925
const f1 = async (): Promise<string> => {
    return '';
};

const f2 = async (): Promise<number> => {
    return 0;
};

const f3 = async (): Promise<Object> => {
    return {};
};

//const res: Promise<Object[]>
const res0 = Promise.all([f1(), f2(), f3()]);

//const res: Promise<Object[]>
const res1 = all([f1(), f2(), f3()]);

//const res2: Promise<[string, number, Object]>
const res2 = betterAll([f1(), f2(), f3()]);

Playground

https://github.com/microsoft/TypeScript/issues/34924

The .all() in this PR works as expected.
The betterAll() I'm suggesting works as expected.

https://github.com/microsoft/TypeScript/issues/34925

The .all() in this PR does not work as expected.
The betterAll() I'm suggesting works as expected.

type Awaited<T> =
    T extends undefined ?
    T :
    T extends PromiseLike<infer U> ?
    U :
    T
;
declare function all<T extends readonly any[]>(
    values: T
): Promise<{ -readonly [P in keyof T]: Awaited<T[P]> }>;

//for empty tuple
declare function betterAll (arr : []) : Promise<[]>;
//for non-empty tuple
declare function betterAll<ArrT extends readonly [any, ...any[]]>(
    arr: ArrT
): Promise<{ -readonly [i in keyof ArrT]: Awaited<ArrT[i]> }>;
//for non-tuple array
declare function betterAll<ArrT extends readonly any[]>(
    arr: ArrT
): Promise<{ -readonly [i in keyof ArrT]: Awaited<ArrT[i]> }>;

//https://github.com/microsoft/TypeScript/issues/34925#issuecomment-550069150
class K {
  getA = (): Promise<string | undefined> => {
    return Promise.resolve(undefined);
  };
  getB = (): Promise<string> => {
    return Promise.resolve('');
  };

  getC() {
    // working in 3.6.3, error in 3.7.2
    Promise.all([this.getA(), this.getB()]).then(([a, b]) => {
      const a_: string | undefined = a;
      // ts2322, b is inferred as `string | undefined`
      const b_: string = b;

      return [a_, b_];
    });

    // attempt to test with new type in PR #33707, same result
    all([this.getA(), this.getB()]).then(([a, b]) => {
      const a_: string | undefined = a;
      // ts2322, b is inferred as `string | undefined`
      const b_: string = b;

      return [a_, b_];
    });

    betterAll([this.getA(), this.getB()]).then(([a, b]) => {
      const a_: string | undefined = a;
      //OK!
      const b_: string = b;

      return [a_, b_];
    });
  }
}

Playground

https://github.com/microsoft/TypeScript/issues/34925#issuecomment-550069150

The .all() in this PR does not work as expected.
The betterAll() I'm suggesting works as expected.

@AnyhowStep it works if you specify the parameter for all() (2nd test case in your playground) as as const. I guess it's consistent that [getA(), getB()] is not inferred to be tuple by default.

Sure. But we can do better.

@typescript-bot test this
@typescript-bot user test this
@typescript-bot run dt

What's the status of this PR? TypeScript 3.7 made a breaking change to Promise.all() (see https://github.com/microsoft/TypeScript/issues/33752), so would love to see this merged ASAP.

as in my issue comment in #37526
Update
My problem is solved with the nightly version of typescript but I will leave it there in case it can be useful for something or someone

"typescript": "^3.9.0-dev.20200324",

Issue now fixed
I have a similar problem and I don't know if I should open a different issue ...

"typescript": "^3.8.3",

    // Promise.all(this.fork) take a T1[] where T1 is a Promise<number>
    // and return a Promise<T2[]> where T2 is a number
    // therefore T1 and T2 are not both «T»

export class MyMaybeList<T = any> {
  private constructor(values: T[]) {
    this.values = values;
  }

  private values: T[];

  // get =======================================================-| fork() |-====

  public get fork(): T[] {
    return this.values != null ? this.values.slice() : [].slice();
  }

  // public =====================================================-| map() |-====

  public map<R = any>(
    fn: (val: T, index: number, array: T[]) => R
  ): MyMaybeList<R> {
    return MyMaybeList.of(...this.values.map(fn));
  }

  // static ======================================================-| of() |-====

  public static of<TVal>(...val: TVal[]): MyMaybeList<TVal> {
    if (val == null || !val.length) return new MyMaybeList<TVal>([]);
    return new MyMaybeList<TVal>([...val]);
  }

  // async =====================================================-| will() |-====

  public async will /* <R> */() /* : Promise<MyMaybeList<R>> */ {
    // Promise.all(this.fork) take a T1[] where T1 is Promise<number>
    // and return a Promise<T2[]> where T2 is a number
    // therefore T1 and T2 are not both «T»
    console.log(this.fork);

    const willThen = Promise.all(this.fork);

    const thenWill = await willThen;
    return MyMaybeList.of(...thenWill);
  }
}

const oneToTen = MyMaybeList.of(1, 2, 3, 4, 5, 6, 7, 8, 9, 10);
const powersOneToTen = oneToTen.map(async val => val * val);

// "log->" powersOneToTen: MyMaybeList<Promise<number>>
console.log(powersOneToTen);

// "log->" awaitedList: MyMaybeList<Promise<number>>
// instead of a -> MyMaybeList { Promise<number> }
// in fact is a -> Promise {  MyMaybeList<number> }
const awaitedPowersOneToTen = powersOneToTen.will /* <unnknow> */();
awaitedPowersOneToTen.then(awaitedList => console.log(awaitedList));

// Promise { <pending> } will resolve into ->
console.log(awaitedPowersOneToTen);

/*
 // Promise.all(this.fork) take a T1[] where T1 is Promise<number>
    // and return a Promise<T2[]> where T2 is a number
    // therefore T1 and T2 are not both «T»
    // console.log(this.fork); will display same as console.log(powersOneToTen);

// console.log(powersOneToTen); ->
// "log->" powersOneToTen: MyMaybeList<Promise<number>>
MyMaybeList {
  values: [
    Promise { 1 },
    Promise { 4 },
    Promise { 9 },
    Promise { 16 },
    Promise { 25 },
    Promise { 36 },
    Promise { 49 },
    Promise { 64 },
    Promise { 81 },
    Promise { 100 }
  ]
}

// console.log(awaitedPowersOneToTen); ->
// "log ->" awaitedPowersOneToTen: Promise<MyMaybeList<Promise<number>>>
Promise { <pending> }
// Promise { <pending> } will resolve into ->

// awaitedPowersOneToTen.then(awaitedList => console.log(awaitedList)); ->
// console.log(awaitedList) ->
// "log->" awaitedList: MyMaybeList<Promise<number>>
// instead of a -> MyMaybeList { Promise<number> }
// in fact is a (should be) -> Promise {  MyMaybeList<number> }
MyMaybeList {
  values: [
     1,  4,  9, 16,  25,
    36, 49, 64, 81, 100
  ]
}
*/

Now that #37610 has reverted the awaited type I don't know what to do I have posted an example above but my real-life situation is as follow :

I was relying on this solution because I have a Promise<MaybeList<Promise<ISymbolSearchResult[]>>>
where MaybeList is an array abstraction so I am using promise.all to remove the inner Promise I don't know what to do now to avoid using as unknown as MaybeList<ISymbolSearchResult[]> inside of my async function (which should be returning the Promise<MaybeList<ISymbolSearchResult[]>> instead of the Promise<MaybeList<Promise<ISymbolSearchResult[]>>>)

Sorry to be that guy but what is the status of this PR? The regression in #33752 is super annoying. 😞

@ValentinH The regression in #33752 should be now fixed with #34501, it will be released with 3.9 on May, 12.

@strelga  thank you really much, this is good news.
Still I found the new definition for Promise.all() pretty interesting as it's not limited to 10 elements anymore. So I'm still interested to know the status and what is blocking?

All of the issues this PR links are closed; I think that this all was handled in 4.5 with Awaited.
I'm going to close this, but am happy to reopen if I'm mistaken.


Incorrect permissions causes crash

connvitals fails to detect insufficient permissions at start-up, and so crashes, printing an ugly stack trace to stderr when run in such a way. Furthermore, because this doesn't happen until a collector tries to initialize a raw socket, it leaves a hanging process for each collector object, forcing the user to kill each one in turn.

This is closed by c61a16b220b423f6a860c4832984623b10c16fa2 - connvitals as of v4.0.2 will print a warning when run with insufficient permissions.


perf: improve efficiency on processing instructions

Problem
#2560
Summary of Changes

add filter to avoid duplicated process of same program_id_index;
add static filter to filter out non-builtin programs early;

Fixes #2560

bench results below. Because the repeated work so far is compute_budget::check_id(program_id) for every instructions, which is just a cheap compareson, the perf improvement for 4 compute-budget instructions and 4 dummy builtins are not noticeable. (it would be when pubkey hashing is involved in next PR).
But still, the static MAYBE_BUILTIN_KEY filter improved 4 dummy Instructions, while 0 instructions regressed due to overhead without any gain.

bench on master before PR
     Running benches/process_compute_budget_instructions.rs (target/release/deps/process_compute_budget_instructions-1b953003a907ddf6)
bench_process_compute_budget_instructions_empty/0 instructions
                        time:   [5.4891 µs 5.4930 µs 5.4975 µs]
                        thrpt:  [186.27 Melem/s 186.42 Melem/s 186.55 Melem/s]
                 change:
                        time:   [-0.1118% +0.0838% +0.2649%] (p = 0.39 > 0.05)
                        thrpt:  [-0.2642% -0.0838% +0.1119%]
                        No change in performance detected.
Found 5 outliers among 100 measurements (5.00%)
  3 (3.00%) high mild
  2 (2.00%) high severe

bench_process_compute_budget_instructions_no_builtins/4 dummy Instructions
                        time:   [16.933 µs 16.944 µs 16.957 µs]
                        thrpt:  [60.390 Melem/s 60.434 Melem/s 60.475 Melem/s]
                 change:
                        time:   [-0.2684% -0.0836% +0.0871%] (p = 0.37 > 0.05)
                        thrpt:  [-0.0870% +0.0837% +0.2691%]
                        No change in performance detected.
Found 13 outliers among 100 measurements (13.00%)
  1 (1.00%) low mild
  6 (6.00%) high mild
  6 (6.00%) high severe

bench_process_compute_budget_instructions_compute_budgets/4 compute-budget instructions
                        time:   [25.644 µs 25.665 µs 25.691 µs]
                        thrpt:  [39.858 Melem/s 39.899 Melem/s 39.931 Melem/s]
                 change:
                        time:   [-0.5227% -0.2684% -0.0223%] (p = 0.04 < 0.05)
                        thrpt:  [+0.0223% +0.2691% +0.5254%]
                        Change within noise threshold.
Found 10 outliers among 100 measurements (10.00%)
  4 (4.00%) high mild
  6 (6.00%) high severe

bench_process_compute_budget_instructions_builtins/4 dummy builtins
                        time:   [17.202 µs 17.211 µs 17.221 µs]
                        thrpt:  [59.462 Melem/s 59.496 Melem/s 59.527 Melem/s]
                 change:
                        time:   [-0.3604% -0.1489% +0.0329%] (p = 0.15 > 0.05)
                        thrpt:  [-0.0329% +0.1491% +0.3617%]
                        No change in performance detected.
Found 14 outliers among 100 measurements (14.00%)
  2 (2.00%) low mild
  5 (5.00%) high mild
  7 (7.00%) high severe

Benchmarking bench_process_compute_budget_instructions_mixed/355 mixed instructions: Warming up for 3.0000 s
Warning: Unable to complete 100 samples in 5.0s. You may wish to increase target time to 5.6s, enable flat sampling, or reduce sample count to 60.
bench_process_compute_budget_instructions_mixed/355 mixed instructions
                        time:   [1.1012 ms 1.1024 ms 1.1043 ms]
                        thrpt:  [927.27 Kelem/s 928.88 Kelem/s 929.90 Kelem/s]
                 change:
                        time:   [-0.3482% -0.2090% -0.0521%] (p = 0.00 < 0.05)
                        thrpt:  [+0.0521% +0.2095% +0.3494%]
                        Change within noise threshold.
Found 10 outliers among 100 measurements (10.00%)
  5 (5.00%) high mild
  5 (5.00%) high severe

bench on PR
     Running benches/process_compute_budget_instructions.rs (target/release/deps/process_compute_budget_instructions-a2fc49a61149f446)
bench_process_compute_budget_instructions_empty/0 instructions
                        time:   [8.9378 µs 8.9599 µs 8.9820 µs]
                        thrpt:  [114.01 Melem/s 114.29 Melem/s 114.57 Melem/s]
                 change:
                        time:   [+62.462% +62.817% +63.128%] (p = 0.00 < 0.05)
                        thrpt:  [-38.698% -38.581% -38.447%]
                        Performance has regressed.
Found 1 outliers among 100 measurements (1.00%)
  1 (1.00%) low mild

bench_process_compute_budget_instructions_no_builtins/4 dummy Instructions
                        time:   [14.316 µs 14.327 µs 14.342 µs]
                        thrpt:  [71.401 Melem/s 71.472 Melem/s 71.527 Melem/s]
                 change:
                        time:   [-15.454% -15.335% -15.216%] (p = 0.00 < 0.05)
                        thrpt:  [+17.947% +18.112% +18.278%]
                        Performance has improved.
Found 3 outliers among 100 measurements (3.00%)
  2 (2.00%) high mild
  1 (1.00%) high severe

bench_process_compute_budget_instructions_compute_budgets/4 compute-budget instructions
                        time:   [26.105 µs 26.153 µs 26.211 µs]
                        thrpt:  [39.067 Melem/s 39.154 Melem/s 39.226 Melem/s]
                 change:
                        time:   [+1.6651% +1.8963% +2.1342%] (p = 0.00 < 0.05)
                        thrpt:  [-2.0896% -1.8610% -1.6378%]
                        Performance has regressed.
Found 9 outliers among 100 measurements (9.00%)
  1 (1.00%) low mild
  4 (4.00%) high mild
  4 (4.00%) high severe

bench_process_compute_budget_instructions_builtins/4 dummy builtins
                        time:   [17.145 µs 17.163 µs 17.182 µs]
                        thrpt:  [59.597 Melem/s 59.665 Melem/s 59.727 Melem/s]
                 change:
                        time:   [-0.5084% -0.3666% -0.2216%] (p = 0.00 < 0.05)
                        thrpt:  [+0.2221% +0.3679% +0.5110%]
                        Change within noise threshold.
Found 5 outliers among 100 measurements (5.00%)
  5 (5.00%) high mild

bench_process_compute_budget_instructions_mixed/355 mixed instructions
                        time:   [497.61 µs 498.21 µs 498.82 µs]
                        thrpt:  [2.0528 Melem/s 2.0554 Melem/s 2.0579 Melem/s]
                 change:
                        time:   [-55.020% -54.944% -54.873%] (p = 0.00 < 0.05)
                        thrpt:  [+121.60% +121.95% +122.32%]
                        Performance has improved.
Found 1 outliers among 100 measurements (1.00%)
  1 (1.00%) low mild


Ethernet crashing randomly (IDFGH-5577)

I'm getting the following log from a backtrace:
0x400eafb5: emac_esp32_init at /Users/silardg/.platformio/packages/framework-espidf@3.40200.210118/components/esp_eth/src/esp_eth_mac_esp32.c:324 (discriminator 2)
0x400898cd: _frxt_timer_int at /Users/silardg/.platformio/packages/framework-espidf@3.40200.210118/components/freertos/xtensa/portasm.S:314

I'm using lan8720a with HanRun HR911105A.

Hi @silardg
Could you please provide more details, following the issue template?
Specifically interested in

IDF version (platformIO with some/ forked IDF?)
which chip (ESP32?)
your ethernet connection (lan8720a is an RMII device, but HR911105A seems to use the old good ENC28J60?)

@silardg Thanks for reporting, would you please help provide more details per the issue template? Thanks.

Thanks for reporting, will close due to short of feedback, feel free to reopen with more details/updates.


[Regression]: Maven could not create local repository

Last Good Version
1.41.2
First Bad Version
1.42.0
Steps to reproduce
After upgrading Playwright Docker image to v.1.42.0 Maven cannot create local repository at /.m2/repository

Create Jenkinsfile with the following Docker configuration

            agent {
                docker {
                    image 'mcr.microsoft.com/playwright/java:v1.42.0-jammy'
                    reuseNode true
                }
            }

Execute a simple test using Maven command: mvn clean test

Expected behavior
The command is executed as before
Actual behavior
The pipeline fails with the following error

Could not create local repository at /.m2/repository

Additional context
No response
Environment
Docker Host:

Operating System: Red Hat Enterprise Linux 9
CPU: x86_64
Java version: 17
Maven version: 3.9.1

I found the solution. Don't know what has changed in these Maven versions but here are the differences between these two images.
1.41.2:
[DEBUG] Reading global settings from /opt/apache-maven-3.9.4/conf/settings.xml
[DEBUG] Reading user settings from ?/.m2/settings.xml
[DEBUG] Reading global toolchains from /opt/apache-maven-3.9.4/conf/toolchains.xml
[DEBUG] Reading user toolchains from ?/.m2/toolchains.xml
[DEBUG] Using local repository at /opt/jenkins/workspace/debug-pw-tests/?/.m2/repository
[DEBUG] Using manager EnhancedLocalRepositoryManager with priority 10.0 for /opt/jenkins/workspace/debug-pw-tests/?/.m2/repository

1.43.0:
[DEBUG] Reading global settings from /opt/apache-maven-3.9.6/conf/settings.xml
[DEBUG] Reading user settings from /.m2/settings.xml
[DEBUG] Reading global toolchains from /opt/apache-maven-3.9.6/conf/toolchains.xml
[DEBUG] Reading user toolchains from /.m2/toolchains.xml
[DEBUG] Using local repository at /.m2/repository
[ERROR] Could not create local repository at /.m2/repository -> [Help 1]
org.apache.maven.repository.LocalRepositoryNotAccessibleException: Could not create local repository at /.m2/repository
    at org.apache.maven.DefaultMaven.validateLocalRepository (DefaultMaven.java:323)
    at org.apache.maven.DefaultMaven.doExecute (DefaultMaven.java:154)
    at org.apache.maven.DefaultMaven.execute (DefaultMaven.java:101)
    at org.apache.maven.cli.MavenCli.execute (MavenCli.java:906)
    at org.apache.maven.cli.MavenCli.doMain (MavenCli.java:283)
    at org.apache.maven.cli.MavenCli.main (MavenCli.java:206)
    at jdk.internal.reflect.DirectMethodHandleAccessor.invoke (DirectMethodHandleAccessor.java:103)
    at java.lang.reflect.Method.invoke (Method.java:580)
    at org.codehaus.plexus.classworlds.launcher.Launcher.launchEnhanced (Launcher.java:283)
    at org.codehaus.plexus.classworlds.launcher.Launcher.launch (Launcher.java:226)
    at org.codehaus.plexus.classworlds.launcher.Launcher.mainWithExitCode (Launcher.java:407)
    at org.codehaus.plexus.classworlds.launcher.Launcher.main (Launcher.java:348)

The solution is to provide -Dmaven.repo.local=/opt/jenkins/workspace/debug-pw-tests/?/.m2/repository in the Maven command.


WIP: Adds EIP-5792 hooks and waitForBundle fn

Problem solved
Adds a waitForBundle function to wait for a provided bundle ID to be confirmed.
Adds the following hooks:

useSendCalls to interact with sendCalls
useCapabilities to interact with getCapabilities
useCallsStatus to interact with getCallsStatus

PR-Codex overview
This PR introduces EIP-5792 React hooks for interacting with Ethereum wallets. It includes useSendCalls, useCapabilities, and useCallsStatus hooks for managing transactions and wallet capabilities.
Detailed summary

Added waitForBundle function to confirm EIP-5792 bundles
Introduced useSendCalls hook for managing contract transactions
Implemented useCapabilities hook for retrieving wallet capabilities
Created useCallsStatus hook for checking call bundle status

The following files were skipped due to too many changes: packages/thirdweb/src/wallets/eip5792/wait-for-bundle.test.ts, packages/thirdweb/src/react/core/hooks/wallets/useSendCalls.ts

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

This is done, just waiting on a Coinbase SW bug to be fixed to be able to fully test in my demo

/release-pr


Dockerfile added for cuda 11.4 again for FIPS build

Checklist before submitting

[ ] Did you read the contributor guide?
[ ] Did you update any affected documentation?
[ ] Did you write any tests to validate this change?

Description
Fixes # (issue).
Review process to land

All tests and other checks must succeed.
At least one maintainer must review and approve.
If any  maintainer requests changes, they must be addressed.

PR check that is failing is because it is trying to execute tests for cuda 11.4 which is not available in open-ce's main branch now. So, the checks would fail until we update github workflows to replace cuda 11.4 to cuda 11.8. @ArchanaShinde1 - Please update github workflows in your builder PR for cuda 11.8 support.


material_theme_tree_headings not working

material_theme_tree_headings Has no effect for me. The "Folders" title stays there regardless of the value I assign. I'm using Sublime build 3103 on Windows 7 Enterprise and Windows 7 Pro.

Same problem with me. I have added all the setting in Preferences->Setting-User but its showing  partial appbar as you can see in given image.

 TO remove the glitch try to remove appbar package.

 See the screen.

Try my preferences:
{
	"always_show_minimap_viewport": true,
	"bold_folder_labels": true,
	"caret_extra_bottom": 0,
	"caret_extra_top": 0,
	"caret_extra_width": 1,
	"caret_style": "phase",
	"close_windows_when_empty": false,
	"color_scheme": "Packages/Material Theme/schemes/Material-Theme-Darker.tmTheme",
	"draw_white_space": "all",
	"enable_tab_scrolling": false,
	"fade_fold_buttons": true,
	"folder_exclude_patterns":
	[
		".svn",
		".git",
		".hg",
		"CVS",
		"node_modules"
	],
	"font_face": "Fira Code",
	"font_size": 13,
	"highlight_line": true,
	"ignored_packages":
	[
		"Autoprefixer",
		"Gutter Color",
		"Material Theme - White Panels",
		"Vintage"
	],
	"indent_guide_options":
	[
		"draw_normal",
		"draw_active"
	],
	"line_padding_bottom": 3,
	"line_padding_top": 3,
	"material_theme_accent_indigo": true,
	"material_theme_appbar_indigo": true,
	"material_theme_compact_panel": true,
	"material_theme_panel_separator": true,
	"material_theme_tree_headings": false,
	"overlay_scroll_bars": "enabled",
	"preview_on_click": false,
	"tab_size": 2,
	"theme": "Material-Theme-Darker.sublime-theme",
	"translate_tabs_to_spaces": true,
	"trim_trailing_white_space_on_save": true,
	"use_tab_stops": true,
	"word_wrap": false
}

I used your preferences still i don't get appbar. You can see snapshot below.

Hey Thanks it works...

@equinusocio
I've uninstalled Material Theme and the appbar, reinstalled them and still see "Folders".  Also tried your preferences before and after the re-installs.
Take a look at the screenshot below. I was editing items within the highlighted area and was having an effect on the "FOLDERS" heading. I assume the last number in the color array is supposed to be opacity, right? For whatever reason, no matter what I set it to, it has no effect. Are there any other ways to adjust the opacity of that header? Is there a documentation resource I can look at that gives you information on what all the properties are?

Check if you have make these true.If so make it false
"material_theme_disable_fileicons"        : false , // Hide siderbar file
type icons
"material_theme_disable_folder_animation" : false , // Disable folder
animation
On Wed, Mar 30, 2016 at 6:31 PM, Joe Zimmerman<EMAIL_ADDRESS>wrote:

@equinusocio https://github.com/equinusocio
I've uninstalled Material Theme and the appbar, reinstalled them and still
see "Folders". Also tried your preferences before and after the re-installs.
Take a look at the screenshot below. I was editing items within the
highlighted area and was having an effect on the "FOLDERS" heading. I
assume the last number in the color array is supposed to be opacity,
right? For whatever reason, no matter what I set it to, it has no effect.
Are there any other ways to adjust the opacity of that header? Is there a
documentation resource I can look at that gives you information on what all
the properties are?
[image: sublime-sidebar]
https://cloud.githubusercontent.com/assets/1313898/14142832/a7625dd6-f64d-11e5-8629-a731731269dc.png
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/equinusocio/material-theme-appbar/issues/22#issuecomment-203421527

@custard-apple No change. Thanks for trying though.

Try to uninstall all other themes in your sublime instead of MT & MT-appbar...

@custard-apple I have no other themes installed

@custard-apple @equinusocio I've disables ALL plugins except for Package Manager, Material Theme, and Material Theme Appbar. Not seeing any difference.

Maybe you have to remove, not disable other themes..

I disabled all plugins and had deleted all other themes before.

I'm having the same issue, tried reinstalling everything, toggling every config value, and no dice.

Toggling material_theme_tree_headings has no effect for me either.  Here's my entire user prefs:
{
	"always_show_minimap_viewport": true,
	"bold_folder_labels": true,
	"caret_extra_bottom": 1,
	"caret_extra_top": 1,
	"caret_extra_width": 1,
	"caret_style": "phase",
	"color_scheme": "Packages/User/SublimeLinter/Oceanic Next (SL).tmTheme",
	"drag_text": false,
	"draw_minimap_border": true,
	"font_face": "Consolas",
	"font_options":
	[
		"subpixel_antialias",
		"directwrite"
	],
	"font_size": 13,
	"ignored_packages":
	[
		"Vintage"
	],
	"indent_guide_options":
	[
		"draw_normal",
		"draw_active"
	],
	"line_padding_bottom": 3,
	"line_padding_top": 3,
	"material_theme_accent_indigo": true,
	"material_theme_bold_tab": true,
	"material_theme_compact_sidebar": false,
	"material_theme_disable_tree_indicator": true,
	"material_theme_panel_separator": true,
	"material_theme_small_statusbar": true,
	"material_theme_small_tab": false,
	"material_theme_tree_headings": false,
	"overlay_scroll_bars": "enabled",
	"theme": "Material Seti.sublime-theme"
}

The only package I have installed that messes with the sidebar is SideBarEnhancements, which I have uninstalled (and reinstalled) to no effect.

I even tried re-installing Sublime, though I didn't delete all the packages and user data that got left behind.

@joezimjs You think we can close this issue?

@equinusocio Might as well. I don't think it's your fault. I haven't been able to find anything on the internet about it because apparently there's very very little documentation regarding these themes. I always find things about color schemes instead. Do you have a good place for looking at documentation about these themes, @equinusocio ?

@joezimjs i found only this
http://docs.sublimetext.info/en/latest/

For what it may be worth, I recently synced my package control settings with Sublime running on my laptop, after completely nuking my Packages/User folder.  The "material_theme_tree_headings": false, actually took effect somehow, despite having the exact same configuration on my desktop.
Running build 3103 on identical Windows 10 versions.

@equinusocio I'm not sure how that's helpful since they specifically don't cover themes: http://docs.sublimetext.info/en/latest/customization/customization.html?highlight=theme
@ScottKaye I've thought about it, but I don't think it's worth the effort to hide the word "Folders"... I'll keep it in mind though. I get the same thing on both of my Windows 7 machines, so I'm curious what is so unique about them.


Reduce "stuttering" in types

Rename types like log::LogRecord to log::Record

I'd like to try working on this.
To confirm:
LogLevel -> Level
LogLevelFilter -> LevelFilter
LogRecord -> Record
LogMetadata -> Metadata, etc.?
Just types? What about functions like log::LogLevel::to_log_level_filter()? Should it become log::Level::to_level_filter?
Should MaxLogLevelFilter become MaxLevelFilter?

That all sounds great to me @omh1280! @sfackler does anything else come to mind for you?

That's everything I can think of. I would agree that the functions and MaxLogLevelFilter would be renamed as well.


[Request] buff Heat Generator

The Heat Generator is currently useless.
It burns Fuel far too fast for extremly low Energy output (especially compared to other Solid-Fueled Generators).
Also, i never understood why it converts EVERYTHING into Lava?
This is probably a cause of programming (its easier to make one Generator that just burns on Lava by either directly piping or converting other fuel sources).
It is however completly obsolete, as things like the Wind Turbine produce far more Power.
The Heat Generator should produce not only more Power, but also should just burn fuels instead of converting them to Lava.

Well..? It is meant as a very basic generator. Read( the first one you get and move away from as fast as you can )

Put a stack of would and have it output to an energy cube. The tiers are currently:
Heat -> wind/solar -> advanced solar -> ethylene generator -> fusion
—
Sent from Mailbox
On Sat, Jul 25, 2015 at 9:22 AM, nintendow131<EMAIL_ADDRESS>wrote:

The Heat Generator is currently useless.
It burns Fuel far too fast for extremly low Energy output (especially compared to other Solid-Fueled Generators).
Also, i never understood why it converts EVERYTHING into Lava?
This is probably a cause of programming (its easier to make one Generator that just burns on Lava by either directly piping or converting other fuel sources).
It is however completly obsolete, as things like the Wind Turbine produce far more Power.
The Heat Generator should produce not only more Power, but also should just burn fuels instead of converting them to Lava.
Reply to this email directly or view it on GitHub:
https://github.com/aidancbrady/Mekanism/issues/2850

Still other Generators (EIO,TE) are more efficent.
And the Lava conversion still makes no sense in my eyes.

Sure they are more efficient which is probably intended (?)
And the Lava conversion well.. note that a lot of mekanism's machines convert fuel or other solid material into a internal fluidlike system.

You also can't extract the lava from the generator, so it is just a representation of time remaining on the production of energy.
—
Sent from Mailbox
On Sat, Jul 25, 2015 at 11:33 AM, ruifung<EMAIL_ADDRESS>wrote:

Sure they are more efficient which is probably intended (?)
And the Lava conversion well.. note that a lot of mekanism's machines convert fuel or other solid material into a internal fluidlike system.
Reply to this email directly or view it on GitHub:
https://github.com/aidancbrady/Mekanism/issues/2850#issuecomment-124853599

@mercsniper has it right, it represents the internal fuel remaining. You can change the values in the config if you want higher gen rates.


[Feature Request] Show work item discussion

Not exactly sure where you'd put it from a UX perspective, but it would be nice to be able to see the discussion of a work item while estimating it.

+1

+1


Requires.io has been shutdown

Requires.io has been shutdown at September 30th 2022.
Please update the README.md.

fixed on main


Compatibility v11

Hello,
I just checked the compatibility today.
I don't see any issue between version 11 and this module.
I will soon try to create a new version allowing v11.
Sorry for the delay

It seems like the WA modules are not available to be activated... Could you please verify, maybe you need to modify something to allow it to activate?  This is the only set of modules that prevent me from upgrading to v11.  Thanks.
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: adrien-schiehle @.>
Sent: Wednesday, June 7, 2023 11:30:52 AM
To: foundryvtt/world-anvil @.>
Cc: Subscribed @.***>
Subject: Re: [foundryvtt/world-anvil] Compatibility v11 (Issue #66)
Hello,
I just checked the compatibility today.
I don't see any issue between version 11 and this module.
I will soon try to create a new version allowing v11.
Sorry for the delay
—
Reply to this email directly, view it on GitHubhttps://github.com/foundryvtt/world-anvil/issues/66#issuecomment-1581166837, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABLH7Q3IQTGNKPFCHAV3K23XKCUDZANCNFSM6AAAAAAYQ3PL5Q.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

Resolved


SCRUM-1686: add customized error handling

Following tips from:
https://developers.redhat.com/articles/2022/03/03/rest-api-error-modeling-quarkus-20#

The only issue with catching exceptions this way is the might not be printed in the logs which makes debugging error very difficult.

I simulated an uncaught exception killing the postgres container. It still logs the stacktrace:


jaeger ingester supports customized consumer client id

Requirement - what kind of business use case are you trying to solve?
I'm trying to use Jaeger Ingester to consume tracing messages from Kafka.
Problem - what in Jaeger blocks you from solving the requirement?
Currently Ingester doesn't support specify client id. Would have to add more customization for ingester.
Proposal - what do you suggest to solve the problem or improve the existing situation?
na
Any open questions to address
na

Please elaborate what you're trying to achieve by setting a custom client-id. What is the impact of not setting anything?

Thanks for reply.
According to Kafka doc client is is used for tracking purpose

An id string to pass to the server when making requests. The purpose of this is to be able to track the source of requests beyond just ip/port by allowing a logical application name to be included in server-side request logging.

as well as quota management. https://kafka.apache.org/documentation/#design_quotasgroups

Quotas can be applied to (user, client-id), user or client-id groups.

In my case client is is a mandatory value, that must be set to the value that we given in order to connect to our Kafka cluster to consume messages.


Local auth configs API timeout

Rancher Version:
v1.1.0
Docker Version:
1.6.2
OS and where are the hosts located? (cloud, bare metal, etc):
local, ubuntu14.04
Setup Details: (single node rancher vs. HA rancher, internal DB vs. external DB)
Fresh installed rancher
Environment Type: (Cattle/Kubernetes/Swarm/Mesos)
None
Steps to Reproduce:
API  /v1/projects/1a5/localauthconfigs timeout, while other apis works
Results:
Timeout
Expected:
return data

@guodong Local auth is not set on a project level and is top level.  I think you want /v1/localauthconfigs.  Try that and see if that is what you are looking for.

@tfiduccia yes! 3Q~