move docs into monorepo

Move documentation into the main repo.  The two major benefits of this are:
Pull requests can contain both code and related docs, together at last.  Docs can even be required before approving a PR, similar to how tests are/should be required.
With docs as a first-class member of the main repo, everyone will have closer proximity to them and a better understanding of how to update; it's currently an oft-forgotten repo that few people use.

npm run start failed with the following:
patternfly-elements  (docs-docs-docs) $ npm run start

> @patternfly/patternfly-elements@ start /Users/carobert/repos/patternfly-elements
> npm run doc-listing-inject && spandx

> @patternfly/patternfly-elements@ doc-listing-inject /Users/carobert/repos/patternfly-elements
> node scripts/doc-listing-inject.js

/Users/carobert/repos/patternfly-elements/scripts/doc-listing-inject.js:29
  if (err) throw err;
           ^

Error: ENOENT: no such file or directory, open '/Users/carobert/repos/patternfly-elements/doc/wrapper.html'

Thanks, fixed!

Yes that is awesome!!!


fix: updated deprecated type usage

Description
Code Cleanup: Updated deprecated type usage
Why
After migrating to v8, there are still instances of a deprecated type that has been removed. A replacement type is available.
Issue
#305
Checklist
Please delete options that are not relevant.

[x] I have followed the contributing guidelines
[x] I have performed a self-review of my own code
[x] I have successfully tested my changes locally

Run yarn pretty manually and then push the changes to fix pr checks issue

Kindly resolve conflicts occurring due to PR #309 enforcing import order

Conflicts are resolved now


Support for Amazon linux

Thanks for the PR!

Leave the versioning out of the PR please - I handle it with a separate one
Is this Rubocop fixes or adding support for Amazon Linux? Could we split commits?

Sure, I'll split it.

You can use the same PR. It's even a good idea. Just rewrite the branch


”Colibași”, ”Câmpurelu” are listed in the drop-down menu

Description:
When user writes ”Colibași”, ”Câmpurelu” the locations appear in the drop-down menu next to the comune and the county witch is a part according to the law.
Precondition:
The website is up an running.

Step 1
Write in the search bar "Colibași"

Expected results
The location appears in the drop-down menu as "Sat Colibași, GIURGIU (Colibași)".


ENH: Refocus on Windows

The procedure to test in case someone is curious, only on Windows (for now):

On master, run sync_test.py with response_device='keyboard':

Press any key (like space) and notice the flashing speeds up (because it does not wait the 0.5 sec.)
Press alt-tab to switch to any other process
Press any key (like space) and notice the flashing does not change, and pressing "ctrl" does not close the window.
Click in the expyfun window
Press "ctrl" to close the window (it closes).

On this branch, repeat the above, but steps (iv) is not necessary to get step (v) behavior -- it happens already on step (iii) (i.e., focus is regained automatically).

Codecov Report

Merging #362 into master will decrease coverage by 0.22%.
The diff coverage is 20%.

@@            Coverage Diff             @@
##           master     #362      +/-   ##
==========================================
- Coverage   89.01%   88.78%   -0.23%
==========================================
  Files          44       44
  Lines        5689     5699      +10
  Branches      920      921       +1
==========================================
- Hits         5064     5060       -4
- Misses        428      438      +10
- Partials      197      201       +4


add import job owners

add import_job owners

please approve the PR

plz approve the PR

/check-enforcer override


Use PYTHON expressions

Right now it uses a mix of PYTHON and VB expressions. No big deal and no issue when you are in ArcMap. When you try to run outside of ArcMap that's the only time it would be an issue. I like to debug outside of ArcMap. In this case VB adds nothing that PYTHON couldn't do.

There's also a few limitations to using VB as well:

Field calculations with a VB Expression type are not supported on 64-bit products, including ArcGIS for Desktop—Background Geoprocessing (64-bit) and ArcGIS for Server. To successfully use Calculate Field in these products, expressions should be converted to Python, or in the case of Background Geoprocessing (64-bit), background processing can alternatively be disabled.


Update not found handlers recursively fix #110

It solves 2 problems:

It sets not-found custom handler for all subtrees if they don't have custom handler already
You can define custom not-found handler before or after any mounts.

nicely done. thanks for the PR!


Set ELYRA_RUNTIME_ENV variable in exection environment for a component to determine it's exection environment

Is your feature request related to a problem? Please describe.
I'm developing the elyra component library and need to know if I'm running inside Kubeflow or in local mode as I need to switch between local data storage and COS data storage for example
Describe the solution you'd like
Please provide a ELYRA_RUNTIME_ENV env variable set to either
local, kubeflow, airflow, ....
Describe alternatives you've considered
Find some other way (based on env variables already set where I am
Additional context
Main purpose now is to support https://github.com/elyra-ai/component-library to change behavior of different components depending on their runtime context

Thanks for opening this issue.

Please provide a ELYRA_RUNTIME_ENV env variable set to either local, kubeflow, airflow, ....

Please note that the implementation will likely set the value of the system-defined env (proposed as ELYRA_RUNTIME_ENV) to the name of the corresponding runtime configuration, with "local" being a special case (as there is no formal runtime configuration named "local" today).  As a result, the known values will be "local", "kfp" and "airflow".  I just want to clarify that "kubeflow" will not be one of the "built-in" options.


Fix issue with ng-bootstrap popup closing when clicking on attached element

Hi! I am quite interested in the status of this PR... is there any way I could give you a hand so it can be moved forward??
thanks in advance

Thanks for the PR!


[FancyZones Editor - Edit layout (Templates)]: Button 'Increment number of zones' and 'Decrement number of zones' are not visible in HC white and black theme.

Microsoft PowerToys version
"0.35.0"
Running as admin

[ ] Yes

Area(s) with issue?
FancyZones Editor
Steps to reproduce
Test Environment:
OS: Windows 10 Build 21354.1
App version: 0.35.0
App name: PowerToys
Screen Reader: Narrator
Tool: Accessibility Insight for Windows
Pre-Requisite:
Set High contrast using Settings -> Ease of Access -> High contrast -> High contrast black and white
Repro Steps:

Launch 'PowerToys' application.
Application will get open with default list item 'General' selected.
Navigate to 'FancyZones' at the left side of the pane and activate it.
Now navigate to 'Launch layout editor' button and activate it.
FancyZones editor window will get open.
Now navigate to any 'Edit layout' button of any layout present under Templates and activate it.
Edit layout dialog will get open.
Navigate to 'Increment number of zones' and 'Decrement number of zones' buttons.
Observe the issue.

User Impact: Priority-1
If any image , icon , hyperlink , text or visual focus disappear when high contrast mode is enabled , will make the UI confusing and difficult for users with low vision to navigate easily to the desired element.
MAS Reference:
https://microsoft.sharepoint.com/:w:/r/sites/accessibility/_layouts/15/Doc.aspx?sourcedoc={3870F7C4-7C43-43C6-A9F7-FD73D800BFD0}&file=MAS 4.3.1 – No Disruption of Accessibility Features.docx&action=default&mobileredirect=true&cid=fa626c36-15d8-4169-911f-29216b0140e8

✔ Expected Behavior
Button 'Increment number of zones' and 'Decrement number of zones' should be properly visible in HC white and black theme.
❌ Actual Behavior
Button 'Increment number of zones' and 'Decrement number of zones' are not visible in HC white and black theme.
Other Software
No response

Outdated due to recent UX changes.


how to create my tokens  with omni??

how to create tokens to represent custom currencies or assets by omni core???I did not find any documents.

watch the spec:  https://github.com/OmniLayer/spec and look for 'Smart Property'

To make a token on the Omni protocol you need to login to your Omniwallet account (or create a new one if you don't have one).
Then you need to have some BTC on your the address created by your Omniwallet (around 0.001-0.002 should be more than enough).
You can then go to 'My Assets' and access the token creation page by clicking the drop down button to choose the type of token you wish to create.
Simply fill in the form (take note of the ecosystem drop down in the top right) with your token details and provide an appropriate miner fee  and once the tx is completed and confirmed the token is created and will be visible in your wallet.
---this is reply from omni help desk .thank you .


componentDidMount is not called when child item is added

Problem Description
I have the following code. I am just playing around trying to understand everything. I have child items being added successfully on the react side, but i have also have a jquery extension that needs to be run on each child item as it is placed in the visual builder. I thought componentDidMount would be the perfect place to do that. I have placed breakpoints in the function and if you create a new page and add the plugin you can add the child items and see them visually but componentDidMount is not called until the page is saved and reloaded. Intermittently you can get it to call in the right place everytime if you add my module after another standard module but it doesn't work everytime
Steps To Reproduce

write a react script to show child items and try to use a jquery extension in componentDidMount to add some functionality.

Screenshot and/or gif
import React, { Fragment, Component } from 'react';
import jQuery from 'jquery';
// Internal Dependencies
import './style.css';
class HelloWorld extends Component {
componentDidMount(){
	console.log('hello');
	jQuery(".main").tiltedpage_scroll({
		sectionContainer: "> section",
		angle: 50,
		opacity: true,
		scale: true,
		outAnimation: true
	});
}

static slug = 'etl_hello_world';
renderImages(){
	const children = this.props.content;
	console.log(children);
	if(this.props.content) {
		return this.props.content.map((content) => {
			return (

			);
		});
	} else{
		return '';
	}
}
render() {

	return (

			{this.renderImages()}

	);
}

}
export default HelloWorld;

QA Checklist
Reporter

[ ] Helpful title used

Helpful title format: Scope(s) :: One Sentence To Describe The Issue
Example: VB :: Floating Toolbar :: Jumps To Top Of Page

[x] Detailed description
[x] Steps to reproduce included
[ ] Support Ticket URL included
[ ] Image and/or gif

Developer

[ ] Steps to reproduce proven solved
[ ] Test included and coverage acceptable and all Tests passing?
[ ] PR description has layman's description of problem and solution and "Areas Affected" listed/explained?
[ ] PR branch includes latest commits from master branch (at the time of checking)?
[ ] Estimated points on issue
[ ] Proper code documentation and conventions?
[ ] Proper RTL appearance
[ ] i18n used if applicable
[ ] Ajax/Form used? Nonce used? Perms checked? All user input sanitized and validated? Intentional unsanitized/unescaped functions used?
[ ] Database saved/updated? Data sanitized and validated?
[ ] Variable output properly escaped and validated?

Peer Review

[ ] PR branch includes latest commits from master branch (at the time of checking)?
[ ] Steps to reproduce proven solved
[ ] Affected areas are tested and ok
[ ] Proper code documentation and conventions
[ ] Proper RTL appearance
[ ] i18n used if applicable
[ ] Test(s) included and coverage acceptable
[ ] Ajax/Form used? Nonce used? Perms checked? All user input sanitized and validated? Intentional unsanitized/unescaped functions used?
[ ] Database saved/updated? Data sanitized and validated?
[ ] Variable output properly escaped and validated?

Quality Assurance

[ ] All tests passed on CircleCI.
[ ] Steps to reproduce passed, the fix works.
[ ] No errors in console or output
[ ] Affected areas are tested and ok
[ ] The fix does not change a customer's website in any surprising or unintended way

[ ] If Divi's design is adjusted, settings are migrated gracefully and invisibly

Design Review

[ ] UI is to spec
[ ] UX is to spec
[ ] Design is to spec
[ ] No surprising visual changes to a customer's website will be experienced after updating

[ ] Any/all visual changes are migrated gracefully

Dev Review

[ ] Test(s) included and coverage acceptable
[ ] Ajax/Form used? Nonce used? Perms checked? All user input sanitized and validated? Intentional unsanitized/unescaped functions used?
[ ] Database saved/updated? Data sanitized and validated?
[ ] Variable output properly escaped and validated?
[ ] i18n used if applicable
[ ] Implementation acceptable, DRY, Scaleable, future safe
[ ] Best practices used, cleverness and grokage, ES6 use, etc.

How about using componentDidUpdate() instead? It runs after the render() function has finished and I use it for example to setup the lightbox effect on my images:
componentDidUpdate() {
    const lightbox = findDOMNode(this.refs.lightbox);
    if (!lightbox) return;

    $(lightbox).magnificPopup({
        type: 'image',
        removalDelay: 500,
        mainClass: 'mfp-fade',
        zoom: {
            enabled: true,
            duration: 500,
            opener: function (element) {
                return element.find('img');
            }
        }
    });
}

Thanks! that did it.


Edge with equal SourceConnectionPoint and TargetConnectionPoint

WPF:
GraphX.Controls.EdgeControlBase.PrepareEdgePath method
We can place items when SourceConnectionPoint of edge equals TargetConnectionPoint of edge.
https://github.com/panthernet/GraphX/blob/PCL/GraphX.Controls/Controls/EdgeControlBase.cs#L778
That cause a little bit odd behavior of edge.
I think i can add check equal condition and change point little.

Hello, thanks for report.  This scenarion wasn't the expected one :)  I don't see much sense in assigning the same connetion point to surce and target. I'm in for not allowing to do so, will it be correct for your case?

My main goal is prevent showing alone arrow (in [0;0] point) when source and target points are equal (that can happened accidentally after manual graph generation and sometimes after dragging).
It is not cool, but now for a while i just added:
        if (p1 == p2)
        {
            p1.X = double.PositiveInfinity;
            p1.Y = double.PositiveInfinity;
            p2.X = double.PositiveInfinity;
            p2.Y = double.PositiveInfinity;
        }
        SourceConnectionPoint = p1;
        TargetConnectionPoint = p2;

I've checked how it works and it looks like self looped edge. If you have an edge where source and target vertices are the same then the self looped edge indicator will be drawn in the top left corner of the vertex. On the other side the code execution should not pass to the place you'd made the fix in if it is the self looped edge we talk about.  Unless you'd changed something in original logic. Originally edge can't have equal source and target connection points.

I have made sample to show what i mean:
https://github.com/galakt/GraphXSample2

Thanks for the sample, unfortunately i'm quite busy right now. Can't promise much but will try to spare some time on the issue.

Fixed! That was an intersting case where vertices were so close together that edge were almost zero length making rotation calc return NaN value messing up the final pointer position. Also i've implemented the EdgeControlBase::HideEdgePointerOnVertexOverlap property which allows you to hide edge pointer in such cases (default) or display them at the edge pos if set to false.


Doesn't work in IntelliJ

Everything works fine in sbt, but IntelliJ keeps saying libraries missing when the app is run from the IDE. Is there any way to configure IntelliJ to make everything work?

Do you mind explaining a bit more what type of error is what you are getting in your IDE? ... or put an screenshot of it?
I seldom use IntelliJ but from what I can imagine, if your problem is when running the application (what you would do from the terminal with an sbt run), then most likely it could fail if IntelliJ is not adding the Spark dependencies in the running classpath (Spark dependencies need to be in the provided scope). If this is the case I would suggest running it using the SBT Shell feature in IntelliJ and see what happens.


misc comments

I like that I can specify a single center point rather than specifying bounds. The user often has no good idea of the bound, but a center guess is far more reasonable. I can set a scale guess by multiplying the input space by a scale; maybe that's helpful? (For example, I know that LR has a tighter scale than init, and per-layer LRs are less impactful than global LRs.)
The readme could link your paper: https://arxiv.org/abs/2306.08055
In the readme, notebooks/carbs_demo.sync.ipynb probably should be notebooks/carbs_demo.ipynb.
I'm not sure how to use is_suggestion_remembered. In my current setup, I pickle my hyperoptimizer to a file, and load+resave it whenever I make a suggestion or observation. This is necessary because it uses a quasirandom sequence, so that parallel suggestions need to increment the optimizer state. Should this flag be used in a similar way? The consequence of "whether to store the suggestion, which be fit with a surrogate value until it is observed" is unclear. It seems odd to fit an arbitrary value as the source of truth to control future sampling locations. And what if my run crashes, and a wrong surrogate value is left inside?
You appear to be sampling from torch's Normal distribution. A low-discrepancy sequence could perform a faster search. Vizier uses Halton: https://github.com/google/vizier/blob/main/vizier/_src/algorithms/designers/quasi_random.py
HEBO uses Sobol: https://github.com/huawei-noah/HEBO/blob/bd1461c752883ca0fe79daa4cf107d9db3160ba2/HEBO/hebo/optimizers/hebo.py#L49

Thanks! I made some updates to the README.

There is a scale parameter on each search space you can set -- typically we set the LR scale to 0.5 for the reasons you suggest.
I added a description of is_suggestion_remembered to the README -- this is actually for parallel execution. We do save a pickled version of the optimizer to wandb on each observation by default, this is controlled by is_saved_on_every_observation in the CARBSParams.
We have tried using a Halton series for the initial sampling, but we didn't find much difference in our testing, and the code was quite a bit more complicated. We removed it for simplicity.


Questions about the test accuracy of only 0.8

Hello, you have done an excellent job. We have encountered some problems in surfacing your work. There are frequent interruptions of downsample error and many error patches when extracting slices, so there may be errors when extracting slice features accordingly. So we would like to know if you can share the npy file of camelyon16 after feature extraction? We are looking forward to your reply!

Hi @hemo0826 and thank you for your question. You test set performance seems to be very different from the results in issue #2. But anyway, I will upload the preprocessed .npy files later.

The preprocess dataset is about 16GB and currently I have run out of Google Drive space. I will find an alternative platform to upload these files, and when the uploading is done you will receive a notification.

Hello, first of all, thank you for your insights. However, I'm currently facing a situation where I'm achieving a maximum accuracy of only 0.81 on the 129 test samples in the 16-challenge. I've used the data preprocessing methods provided by CLAM, and I'm using a pre-trained ResNet50 on ImageNet to extract features. Would it be possible for you to share the trained model parameters after your training? I'd like to further test to see if the issue lies with the data. Looking forward to your response, and thank you very much.


Targeting S+ (version 31 and above) requires that one of FLAG_IMMUTABLE or FLAG_MUTABLE be specified when creating a PendingIntent

When running on Android 12
E/flutter ( 7301): Strongly consider using FLAG_IMMUTABLE, only use FLAG_MUTABLE if some functionality depends on the PendingIntent being mutable, e.g. if it needs to be used with inline replies or bubbles., , null) E/flutter ( 7301): #0      StandardMethodCodec.decodeEnvelope (package:flutter/src/services/message_codecs.dart:607:7) E/flutter ( 7301): #1      MethodChannel._invokeMethod (package:flutter/src/services/platform_channel.dart:167:18) E/flutter ( 7301): <asynchronous suspension> E/flutter ( 7301): #2      FlutterCallkitIncoming.showCallkitIncoming (package:flutter_callkit_incoming/flutter_callkit_incoming.dart:39:5) E/flutter ( 7301): <asynchronous suspension> E/flutter ( 7301): #3      FirebaseMessagingService.notificationOnMessageHandler.<anonymous closure> (package:vinciio/core/services/firebase_messaging_service/firebase_messaging_service.dart:113:7) E/flutter ( 7301): <asynchronous suspension>

Hi @TouseefAQ
I just deployed version 1.0.1.
please update it

Resolved , Thank you very much. I really appreciate.


Only First User Group Is Added Using Hidden Group's "Add User"

Summary
When editing a Hidden SeAT Group, only the first of the list of selected users is added when Add User is pressed.
Discussion
The Hidden SeAT Group user configuration block allows you to pick multiple users ("groups") from those available. When clicking "Add User", only the first is added. Granted the button says "Add User" and not "Add Users", but other similar configuration areas, like choosing SeAT groups for adding managers, work fine with multiple selections.

Thank you for reporting. Will be solved in next release


Add GCP PubSub Sample to Wiki

I've added a GCP PubSub sample to the org which needs to be added to the wiki under Samples and Blog posts

GCP PubSub messages consumed by a Go Container

@jeffhollan 	Can you pick this up or shall we move this to the GitHub repo as well?

@patnaikshekhar can you send a PR to our samples repo, if not done already please?
If we're good to go, I'd like to close this issue.


fix(plugin-react): fix missing ResolveConfig type

Description
This fixes type error when running pnpm run dev in plugin-react.
Additional context
Steps to Reproduce:

$ cd packages/plugin-react
$ pnpm run dev
You will see type error as following

What is the purpose of this pull request?

[x] Bug fix
[ ] New Feature
[ ] Documentation update
[ ] Other

Before submitting the PR, please make sure you do the following

[x] Read the Contributing Guidelines.
[x] Read the Pull Request Guidelines and follow the Commit Convention.
[x] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[x] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[x] Ideally, include relevant tests that fail without this PR but pass with it.

please add a reproduction

please add a reproduction

Added steps to reproduce.

Closing this as it was fixed by https://github.com/vitejs/vite/pull/8139/files#diff-d1f942b9a74f586a80bca3733150b7c692a6460e8350bc19cbbad1d53f95b356.
Thanks for the PR.


Syntactic checks against comparing to null/undefined in strictNullChecks mode

🔍 Search Terms
nullish, strictNullChecks, equals, undefined, null
✅ Viability Checklist

[x] This wouldn't be a breaking change in existing TypeScript/JavaScript code
[x] This wouldn't change the runtime behavior of existing JavaScript code
[x] This could be implemented without emitting different JS based on the types of the expressions
[x] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, new syntax sugar for JS, etc.)
[x] This isn't a request to add a new utility type: https://github.com/microsoft/TypeScript/wiki/No-New-Utility-Types
[x] This feature would agree with the rest of our Design Goals: https://github.com/Microsoft/TypeScript/wiki/TypeScript-Design-Goals

⭐ Suggestion
When strictNullChecks are enabled, disallow == and === comparisons to null/undefined for expressions that are syntactically known to not be undefined.
I'd imagine the definition of syntactically not null/undefined expressions would be similar to that from https://github.com/microsoft/TypeScript/pull/59217.
This is similar to https://github.com/microsoft/TypeScript/issues/11920, but I think would be easier to land because it won't error on defensive programming null checks.
📃 Motivating Example
TypeScript's typeof operator always returns a string, but it can be hard to spot the difference between checking "undefined" and just undefined in code like this:
if (typeof foo === undefined) {

In strictNullChecks mode, TypeScript will catch unnecessary comparisons like this, for expressions that at runtime it knows can never be null or undefined.
💻 Use Cases

What do you want to use this for? catch user errors
What shortcomings exist with current approaches? there's the https://typescript-eslint.io/rules/no-unnecessary-condition/, but that's type-based so does have the problem of flagging defensive programming. it's also nice to have things be in the compiler instead of a separate tool.
What workarounds are you using in the meantime? nothing at the moment, if this is infeasible will probably create a custom lint check.

Duplicate of #29200.

I'd argue this is broader in scope than https://github.com/microsoft/TypeScript/issues/29200. Typeof was the motivating example, but TS could also check other code patterns as it does for nullish coalesing after https://github.com/microsoft/TypeScript/pull/59217.

I mean.. the other cases are covered by #11920.

My understanding is that a major concern about https://github.com/microsoft/TypeScript/issues/11920 was breaking code intentionally checking for null/undefined against an API excluding null/undefined.

We intentionally allow this for the sake of defensive programming (i.e. defending against missing inputs from non-TS code). If there's enough demand we could add a flag or something.

& the discussion on the PR to fix that issue https://github.com/microsoft/TypeScript/pull/59559#issuecomment-2276461058 seems to be concluding that it's too breaking to land without adding an extra flag.
I'd expect this change to be significantly less breaking specifically because it would be syntactic only.
If the TS team is interested in rescoping https://github.com/microsoft/TypeScript/issues/11920 to be about cases only syntactically known to be not null/undefined, or doesn't want to add a syntactic-only check, I understand, but I would like to get feedback specifically on the idea of a syntactic-only check so that it's not lost in the noise in https://github.com/microsoft/TypeScript/issues/11920.

To clarify, I'm not proposing to flag this code from https://github.com/microsoft/TypeScript/issues/11920#issue-185914339:
declare const value: number;
if (value !== null) { // <-- somehow valid, expected to be invalid, since number doesn't have null as a possible value
}
if (value !== '') { // <-- invalid as expected
}

because value is not syntactically known to always be null, that's just a type system construct.

Other cases that could be covered, motivated by the nullish coalesce checks from https://github.com/microsoft/TypeScript/pull/59217:
if (((x) => x) !== undefined) {}
if ("" !== undefined) {}
if ({foo: 3} !== undefined) {}

I think this is a decent idea now that we have a notion of "syntactically provable to not be undefined". I put up a very drafty draft at #60433 to see if this is likely to be useful.


feat: Add support for ffmpeg via snap package

feat: Add support for ffmpeg via snap package, requires ability to specify custom ffmpeg binary names if required
closes: https://github.com/KrumpetPirate/AAXtoMP3/issues/200

Looks good, thanks


Cannot publish app

Cannot publish app with the insturctued command
./publish.ps1 -name MY_APP_NAME
This issue is for a: (mark with an x)
- [x] bug report -> please search issues before submitting
- [ ] feature request
- [ ] documentation issue or request
- [ ] regression (a behavior that used to work and stopped in a new release)

Minimal steps to reproduce
In PowerShell, run the following commands

./provision.ps1 -name <YOUR_APP_NAME> -location westus2
./publish.ps1 -name MY_APP_NAME

Any log messages given by the failure

Expected/desired behavior
The script run without issues
OS and Version?
Windows 11

@Yonet, I have a fix here #12.

Having an issue during publish where it hangs upon .zip file creation. Yarn is installed globally

@nicholasdbrady I notice this symptom after I added Connect-AzAccount in the script. The shell becomes not responsive due to it prompt a page in browser for login. Could you please check your browser to see if it’s the same situation?


Include autoincrement number for (better) UUID support

When defining a muffin, I'd like to be able to define a value that is auto incremented. For instance; I make use of UUID, and when writing tests, it could be very useful to be able to call users like this: 1, 2, 3,4.
I.e:
FactoryMuffin::define(
    'User', [
        'id'                        => 'autoincrement', // would produce: 1, 2, 3, 4, 5, etc for each muffin
        'firstname'     => 'string',
        'lastname'          => 'string',
        'email'             => 'email',
        'size_system'       => 'numberBetween|1;7',
    ]
);

Is this possible as of right now?

@GrahamCampbell FactoryGirl has an idea of Sequences, it's maybe something we can implement:
FactoryGirl.define do
  factory :user do
    sequence(:email) { |n|<EMAIL_ADDRESS>  end
end

FactoryGirl.define do
  factory :user do
    sequence(:id) { |n| n}
  end
end

@canfiax as a work around, you could use a combination of fakers unique() and randomDigitNotNull()

This is already possible using our callable generator.

Just pass in something like this:
function () use ($gen) {
    $gen->next();
};

Where the $gen variable is an instance of a class that looks like this:
class Gen
{
    private $last = 0;

    public function next()
    {
        return ++$this->last;
    }
}

Or, even just this:
function () {
    static $last;

    if ($last === null) {
        return $last = 1;
    }

    return ++$last;
};

@GrahamCampbell I recently updated factory muffin from v2.1 to v3.1
For auto-incrementing ids, we used to use the following function:
function () {
    static $last;

    function () {
        static $last;if ($last === null) {
        return $last = 1;
    }

    return ++$last;
};

But after upgrading the same function always started returning 1, resulting in errors for duplication of primary key in our test database.
Any idea what could be the issue and how to resolve it?


Username must not be null!

what do i do?

If you mean gradle complaining about null username when trying to download dependencies: for the time being it seems like you need an access token to get github hosted gradle packages. To get it to work, I've generated github public access token, and put it in ~/.gradle/gradle.properties along with username:
gpr.user=ja2142
gpr.key=<github_pat>

There's also an option to use GPR_USER and GPR_KEY environment variables instead.


Add unlock-repo command

Since we now have (or soon) the ability to lock the source repo when doing GH->GH migrations, it would be convenient to have an unlock-repo command in gh gei. Today customers would have to use the unlock API if they wanted to unlock a repo that GEI had locked for them (https://docs.github.com/en/rest/migrations/orgs#unlock-an-organization-repository)

Because the unlock repo API requires the (export) migration ID, I see two possible options:

Add gh gei unlock-repo and require the user to provide the right migration ID, or
Add a flag to gh gei migrate-repo that unlocks the source repo on success.

I prefer the latter, but I'm curious to hear others' thoughts.

I don't like any of these. As a customer I just want to give a repo name and have it unlock. What if I did the migration days ago and no longer have the migration ID, but the repo is still locked? What would a customer do today?

if we have the repo name can we lookup the migration id?

Short answer: yes, you can get all migrations, and the response has the repos for each migration; we could then iterate over the migrations and filter that way.
Question behind the question: why are customers wanting to unlock repos in the first place? Are they locking on dry runs, or are they intending to continue using a migrated repo? I guess the most likely answer is just spot-checking data fidelity, which I suppose I understand.

We're going to put this item back on the backlog due to it being more work than anticipated. Will revisit in the future.

@dylan-smith : for what its worth, having the gh gei unlock-repo MIGRATION_ID would suffice for the short term as I myself ran into this issue during a services engagement but the customer was unaware how to unlock it themselves 👍

Is there a way to get numeric GEI Id, all I have is the GEI migration id and I can't unlock repo with this one.

@Mu-Reslife There isn't any way to do that at the moment. Unfortunately, if you need to unlock a repo, you would need to contact GitHub Support. Sorry for the pain!

@timrogers @dylan-smith We have run into this issue as well. There was some issue with the migration, now both the source and destination repos are locked. We are waiting for support team to respond. Is there any other way to unlock the repositories.

@timrogers @dylan-smith We have run into this issue as well. There was some issue with the migration, now both the source and destination repos are locked. We are waiting for support team to respond. Is there any other way to unlock the repositories.

Can you send the URL of the support ticket to the email on my profile? I can take a look.

@timrogers there was a slight miscommunication.. client had locked the source repos using lock repo flag. The destination repos were fine. they still needed to unlock the source repos. Temporarily we are using a workaround to generate a new migration id for the required org and repos using the api.. we are using this migration id to unlock the repos. Thank you for the prompt response


Not compliant with GNU LibreJS (i.e. FSF FreeJS)

Please can the Gratipay website either:

(ideally) not require visitors to execute JavaScript; or
(failing that) comply with GNU LibreJS as per the FSF's FreeJS initiative?

Thanks!

Why are we not already compliant with FreeJS?

Why are we not already compliant with FreeJS?

The JavaScript on gratipay.com is not currently labelled in a way that is detectable by LibreJS.

PRs welcome.


Restructure locksmith integration

Removed Locksmith framework and instead added the necessary Locksmith files.
Takes care of issue #10

Codecov Report

Merging #11 into master will increase coverage by 5.89%.
The diff coverage is 96.42%.

@@            Coverage Diff             @@
##           master      #11      +/-   ##
==========================================
+ Coverage   86.58%   92.48%   +5.89%
==========================================
  Files          16       24       +8
  Lines         671     1290     +619
==========================================
+ Hits          581     1193     +612
- Misses         90       97       +7

Impacted Files
Coverage Δ

CelyTests/ConstantsAndProtocolTests.swift
100% <ø> (ø)
:arrow_up:

Cely/LocksmithAccessibleOption.swift
100% <ø> (ø)

Cely/LocksmithSecurityClass.swift
100% <ø> (ø)

Cely/Cely.swift
96.96% <ø> (+12.12%)
:arrow_up:

Cely/CelyStorage.swift
80.82% <ø> (-1.37%)
:arrow_down:

Cely Demo/ViewController.swift
0% <ø> (ø)
:arrow_up:

Cely Demo/AppDelegate.swift
60% <0%> (+60%)
:arrow_up:

CelyTests/LoginViewControllerTests.swift
99.3% <0%> (-0.7%)
:arrow_down:

CelyTests/CelyStorageTests.swift
86.31% <100%> (+0.14%)
:arrow_up:

Cely/LocksmithError.swift
100% <100%> (ø)

... and 21 more

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 66824c7...272d7c2. Read the comment docs.


assertion failed: edges.remove(&key)

Problem
I'm working on my own RUSTC_WRAPPER and it may slow down some calls to rustc made by cargo. Toying with the amount of --jobs=N, I notice I can fail a cargo assertion:
2024-05-09T07:46:34.9182036Z thread 'main' panicked at src/cargo/util/dependency_queue.rs:191:13:
2024-05-09T07:46:34.9183450Z assertion failed: edges.remove(&key)
2024-05-09T07:46:34.9184589Z note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
2024-05-09T07:46:36.0109959Z ##[error]Process completed with exit code 101.

Now I'm not sure if I'm doing something wrong or if it's on cargo's side?
Steps
No response
Possible Solution(s)
No response
Notes

the mentioned code: https://doc.rust-lang.org/beta/nightly-rustc/src/cargo/util/dependency_queue.rs.html#191
the build logs https://productionresultssa6.blob.core.windows.net/actions-results/fe4adb8f-2924-4ee1-9d51-2ddc2c68553e/workflow-job-run-5d30e883-8d76-5db4-605e-068efef1550f/logs/job/job-logs.txt?rsct=text%2Fplain&se=2024-05-09T08%3A10%3A05Z&sig=eU4enoWXQZTBDKBXjwWMnbSXRC4rLMx1g3pJP0nBhC4%3D&sp=r&spr=https&sr=b&st=2024-05-09T08%3A00%3A00Z&sv=2021-12-02

Version
cargo 1.78.0 (54d8815d0 2024-03-26)
release: 1.78.0
commit-hash: 54d8815d04fa3816edc207bbc4dd36bf18014dbc
commit-date: 2024-03-26
host: x86_64-unknown-linux-gnu
libgit2: 1.7.2 (sys:0.18.2 vendored)
libcurl: 8.6.0-DEV (sys:0.4.72+curl-8.6.0 vendored ssl:OpenSSL/1.1.1w)
ssl: OpenSSL 1.1.1w  11 Sep 2023
os: Ubuntu 23.10 (mantic) [64-bit]

Could you share a minimal reproduction for this, and also a log without authentication to view? Without them it's a bit hard to come out with anything valuable.

Ah well sorry about that link

The failing GA job: https://github.com/fenollp/rustcbuildx/actions/runs/8998781512/job/24766171190?pr=35
Its logs https://github.com/fenollp/rustcbuildx/suites/23560676757/logs?attempt=2
To reproduce pretty deterministically: on that branch wip, starting with commit 9b2587603 run

# Needs docker up and running
jobs=8 ./hack/clis.sh cov clean

I'll try to reproduce with jobs=N, N<8 to help with debugging..

FYI since this clearly doesn't reproduce with jobs=1, this is surely a missed dependency issue on my end!


PIX event name not show up in GPU usage

For ModelViewer sample, even with PIXBeginEvent(...) get called in the right place, GPUusage panel only shows unknown for evey events. Does the pix stuff not working currently?

GPU Usage support for DirectX12 won't be available until Visual Studio 2015 Update 1 is released (which is actually coming up real soon).
If you don't want to wait for the official release, you are welcome to install the Release Candidate which should be pretty close to the official release.


ENH: select_translatable frame argument

[NEW] users can now specify that all sequences in a collection are in
the nominated frame.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>7
Details

12 of 12 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.005%) to 92.01%

Totals

Change from base Build<PHONE_NUMBER>0:
0.005%

Covered Lines:
33117

Relevant Lines:
35993

💛  - Coveralls

Note that the codacy warning is about the scriv markdown, it can be safely ignored since those are reformatted on release


menu example not working on Windows

A menu bar isn't rendered on Windows.
See https://github.com/tauri-apps/tao/issues/343

We've switched back to winit.


edit field editable programmically

Is there any way to edit field editable programmically?
I want to add a "edit" button to enable the all field editable = true

Hi,
You can do it by editing the containers state.
constructor(params) {
        super(params);
        this.state = {
            emailEditable: false
        }
    }

Then on the field:
<GiftedForm.TextInputWidget
  name='email'
  title='Email:'
  editable={this.state.emailEditable}
/>

Then in your button:
_toggleEditable() {
  this.setState({emailEditable: !this.state.emailEditable});
}


Make possible to define bootstrap-brokers via Environment Varible

Copied from #367

Okay, when can I expect this to be merged?

I didn't have time before the last release to determine why it was failing the build. If you would like feel free to make a new PR and resolve that, otherwise it will need to wait until I have some time to dive into it again. If you do create a new PR use a branch on your fork (anything but master) that I can push changes to in case I need to do so before the next release.

Moved to https://github.com/seglo/kafka-lag-exporter/pull/397


[rom_ext,dice] update certs if firmware updates detected

[rom_ext] add ability to update DICE certs
This adds a mechanism to the ROM_EXT to update the DICE attestation certs if the attestation keys have changed due to either:

a ROM_EXT update,
an Owner firmware update, and/or
an ownership transfer.

This partially addresses #21583.

@jwnrt I had to add the last commit to get this to pass CI. Please let me know if I am missing anything.

The last commit looks fine, thanks for the change. I thought we'd been running it in ROM_EXT environments though... was it the otp setting that broke something?

I think it it was that the manual override of the settings previously did not set the rom_ext slot: https://cs.opensource.google/opentitan/opentitan/+/master:hw/top_earlgrey/BUILD;drc=38d0d95f72734ec6c617d22daeb2c12099b4a1b2;l=236

CI failures not related to this PR. Merging.


Feature request: Exporting TensorBoard graphs to graphviz DOT files

As recently discussed on the TensorFlow mailing list (https://groups.google.com/a/tensorflow.org/d/msg/discuss/GUW0KOmN7MM/2lRMD4JVAQAJ), it would be nice if TensorBoard would have an option to export a graph as graphviz DOT file. This would allow users and developers to create e.g., Python packages based on pygraphviz that can further modify the graph structure, such as simplifying or summarizing the graph ops into a publication-ready figure, etc.
For example, scikit-learn has such a function to export decision trees to DOT files; maybe the source code is useful as an example: https://github.com/scikit-learn/scikit-learn/blob/f3320a6f/sklearn/tree/export.py#L74

Hey @rasbt I've recently been using something simple like this:
https://blog.jakuba.net/2017/05/30/tensorflow-visualization.html
It makes mention of the following:
https://github.com/tensorflow/tensorflow/blob/master/tensorflow/tools/quantization/graph_to_dot.py
What do you think?

That's an awesome post, Diego, thanks for sharing! It's really handy, and I didn't know that it was so easy to export graphs via graphviz's "Digraph" already! Of course, for the feature request, it would be even more convenient to have an "export" function in TensorBoard that does exactly this -- saving the graph as dot file for further manipulation.
Best,
Sebastian

On Oct 28, 2017, at 10:58 PM, Diego Mesa<EMAIL_ADDRESS>wrote:
Hey @rasbt I've recently been using something simple like this:
https://blog.jakuba.net/2017/05/30/tensorflow-visualization.html
It makes mention of the following:
https://github.com/tensorflow/tensorflow/blob/master/tensorflow/tools/quantization/graph_to_dot.py
What do you think?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

This is an excellent suggestion and it's something we're already in the process of thinking about. Please follow https://github.com/tensorflow/tensorboard/issues/570. I'm going to close this one out as a duplicate.


Wait for database connection & use init system

We wait for the database to be ready before starting Gerrit. This is a workaround for Issue 1305.
To properly handle sub-processes and shutdown, we use "dumb-init" as a process supervisor.
The Compose YAMLs in this commit can be used for testing this behavior.

Hi there,
By using nc to test the port of DB before startup the Gerrit is a brilliant idea. However, dumb-init as the init system looks kind of unnecessary in this case. I understand there are some arguments about how docker containers may handle signals poorly. I tested this Gerrit container by using docker stop/kill commands. It looks that the Gerrit java process, which run as pid 1 currently, can handle these signals very well. I even built and tried a version with dumb-init installed and ran as the process 1. The behavior looks no differences to me.
So would you please, submit a PR with only the waiting db scripts?

Hi there and thanks for your reply!
Regarding the init system: You are right. Due to your use of exec, the Java process becomes PID 1 and everything is fine from this point of view.
But I still have one doubt: Does Gerrit spawn new processes (i.e. besides the JVM, e.g. some Git process)? In that case, the init-system would be useful after all to reap those processes (as the JVM is not a process supervisor).
Let's first clarify this potential issue please. Then I will create a better PR.

Hi @ju-la-berger
I can't 100% assure that but I think in the JAVA world people usually use multi-threads instead of multi-processes since fork a new JVM might be quit a resource consuming stuff which also isn't a good practice.
However this docker image has an extend feature that you can put some scripts with .nohup as the postfix under the /docker-entrypoint-init.d directory. These scripts will be executed by nohup before starting the daemon. I observed that these processes became zombie/defunct processes after they exit themselves.
The interesting thing is they still become zombie processes even I have dumb-init as the init. That's why I said I found no different behaviors with or without dumb-init. BTW, those processes can be collected after docker stop/kill command. My guess is the upper docker process do the collecting works but I can't confirm that.
My hosts' OS are debian and coreos. You can recreate this issue if you have interests to pursue the root cause of this issue.


Service Name Generation

Buf's default lint settings requires services to end in service i.e. HaberdasherService rather than Haberdasher.
If you follow that convention, TwirpScript generates HaberdasherServiceService

Cool, that makes sense to me. I think this service suffix is coming from google-protobuf, I think the only name manipulation I do is camelCasing.
Speaking of which, because this will be a breaking change, what do you think about camel casing the service and method names while we're in here?


Feature change

Will do Dark mode style change .

please assign me in this work

I want to modify the dark and light button and make it engaging.

can i do this work?

Ok go ahead @arghadipmanna101

thank you


Update to Who We Are

First thoughts on digital paper...

@kyle-spitzer-chr is helping out with this. we are going to be seeking department input on this topic to help build it out. More to come.

Below are suggested edits and stuff I made up. Feel free to tweak and revise as desired. (This is not a pull request be cause I don't have access to create one.)
We are C.H. Robinson IT (CHR IT).  We work to deliver our vision of accelerating commerce through the world’s most powerful supply chain platform.
The role of CHR IT is determined by each of our dedicated CHR IT-ers, or CHRITers*.
At our core, CHRITers are

[I assume you already have descriptors you'd like to go here]
?
?

We want to share our experiences, challenges, and triumphs to help make IT a better place worldwide.

Pronounced critters


chore(deps): update dependency prettier to v1.13.7

This Pull Request updates dependency prettier from v1.13.6 to v1.13.7

Release Notes
v1.13.7
diff

Remove calls to eval("require") in the distributed code (#4766)

Commits
v1.13.7

114b30b Release 1.13.6
1baddf2 Bump Prettier dependency to 1.13.6
9805bab docs(changelog): link -> diff
7a3ac3a Handle JSX whitespace separately from fbt whitespace (#4717)
6c482e2 Expose isPreviousLineEmpty to plugins (#4747). (#4748)
b67fa90 fix(markdown): preserve email autolink (#4740)
2c2283e Update Jest URL to jestjs.io (#4756)
0b659b8 feat: update linguist languages (#4741)
0ce40b1 Add --stdin-filepath to website (#4513)
576e570 Fix custom require's (#4766)

This PR has been generated by Renovate Bot.

Coverage remained the same at 93.878% when pulling 23dd9c1f4c449973aa8cd0335156e64ca33d3d48 on renovate/prettier-1.x into 0b94e20997cbef43380cb7a66a16f43c405348e8 on master.

Coverage remained the same at 93.878% when pulling 23dd9c1f4c449973aa8cd0335156e64ca33d3d48 on renovate/prettier-1.x into 0b94e20997cbef43380cb7a66a16f43c405348e8 on master.


Html export

As discussed in #17 this allows for exporting the scene list and optionally include the images from the save-images command in the html output.

This is absolutely fantastic, thank you so much @wjs018.
Everything seems to be up to the project's standards after a full review, I really appreciate your effort in that regard.  This will make updating the documentation & licenses really easy for me.  I left a few comments in the code review, but they are very minor so I will accept this as-is and work on those separately.
Thank you again for your efforts with this, much appreciated.


What is the best way to prevent OOMKIilled error

Even having the retention time set, the Prometheus pods are failing with OOMKilled error. How to remove WAL? Can I set up Cron Jobs for that?

It makes more sense to ask questions like this on the prometheus-users mailing list rather than in a GitHub issue. On the mailing list, more people are available to potentially respond to your question, and the whole community can benefit from the answers provided.


Proper installation and setup?

This is the first time compiling index.ts and i'm getting stuck, i'm really new to node.js.
Since README didn't explained the installation, i did research myself and found out i need to install node.js and tsc on Windows 10
Then i proceed with these steps as they were required
npm install --save-dev frida-il2cpp-bridge
npm install @types/node --save-dev
npm install @types/frida-gum --save-dev
tsc index.ts

But i'm getting duplicate error
C:/Users/andnix/AppData/Roaming/npm/node_modules/typescript/lib/lib.dom.d.ts:5465:11 - error TS2300: Duplicate identifier 'File'.

5465 interface File extends Blob {
               ~~~~

  node_modules/@types/frida-gum/index.d.ts:2200:15
    2200 declare class File {
                       ~~~~
    'File' was also declared here.

C:/Users/andnix/AppData/Roaming/npm/node_modules/typescript/lib/lib.dom.d.ts:5470:13 - error TS2300: Duplicate identifier 'File'.

5470 declare var File: {
                 ~~~~

  node_modules/@types/frida-gum/index.d.ts:2200:15
    2200 declare class File {
                       ~~~~
    'File' was also declared here.

node_modules/@types/frida-gum/index.d.ts:151:18 - error TS2300: Duplicate identifier 'gc'.

151 declare function gc(): void;
                     ~~

  node_modules/@types/node/globals.d.ts:55:13
    55 declare var gc: undefined | (() => void);
                   ~~
    'gc' was also declared here.

node_modules/@types/frida-gum/index.d.ts:2200:15 - error TS2300: Duplicate identifier 'File'.

2200 declare class File {
                   ~~~~

  C:/Users/andnix/AppData/Roaming/npm/node_modules/typescript/lib/lib.dom.d.ts:5465:11
    5465 interface File extends Blob {
                   ~~~~
    'File' was also declared here.
  C:/Users/andnix/AppData/Roaming/npm/node_modules/typescript/lib/lib.dom.d.ts:5470:13
    5470 declare var File: {
                     ~~~~
    and here.

node_modules/@types/node/globals.d.ts:55:13 - error TS2300: Duplicate identifier 'gc'.

55 declare var gc: undefined | (() => void);
               ~~

  node_modules/@types/frida-gum/index.d.ts:151:18
    151 declare function gc(): void;
                         ~~
    'gc' was also declared here.

Found 5 errors.

If i deleted the node folder which i installed via npm install @types/node --save-dev then i'm getting missing errors. Too long to post here
So what i'm missing there? is it improper way I installed it? If so, what's the proper way to install and setup?

@AndnixSH
There isn't any full-installation guide because this is a frida module. You must be familiar with frida and npm/ts project ecosystem.
You can read more about frida here.
Also, the example folder contains a minimal project sample (index.ts, tsconfig.json, package.json).
Since this isn't really related to my project, let me know if I can close this.

Oh i was expecting more python related. Last question, can't this be used within python? I'm more familar with python and have frida installed
You can close it

Nvm, frida is more related to Typescript.
And closed

@AndnixSH
Frida embeds a JS runtime (v8 or quickjs), so it can only run a single javascript file.
You can look at the example package.json file and look at the npm scripts; it uses frida-compile to bundle a multi module typescript project into a single javascript file.
Btw if you need a further explaination, there's a telegram group for frida: @fridadotre.


Allow configuration of proxy authentication via application.properties

fix #988
fix #987

I have no idea how to set up a proxy using testcontainers and whether it is possible at all.

Maybe something like this would work?  https://hub.docker.com/r/tarampampam/3proxy/

Yeah, thanks for the tip, @shumonsharif , however, 3proxy is not very flexible when it comes to resolving the proxied host names. It accepts only proper external DNS servers, which is far from trivial to set up in Docker. I ended up with mitmproxy/mitmproxy. It resolves docker network aliases OOTB.


Calorie to Power calculation yielded too high watts

The takeaway from #360 and #359 is that when the calorie 2 power logic kicked in it yielded too high power. From HRM 137 it yielded 211W instead of about 150W. There's a lot of leeway the other direction as well as we know because the body's efficiency can vary in the range of 20%-25% or even wider. I may need to revise things here. This can be treated with a calorie factor, but may need to revisit.

Here is the test commit for #360 https://github.com/TrackMyIndoorWorkout/TrackMyIndoorWorkout/commit/649fe8f6c5d04ecfa2480332fda702426d754a36
The heart rate can yield a caloriesPerMinute by the hrBasedCaloriesPerMinute computation. Then the power is calculated by (caloriesPerMinute * 50.0 / 3.0).round() which is just a pure time based integration of the calories rate (50 / 3 = 60 * 1000 / 3600). All is by https://www.braydenwm.com/calburn.htm
So we either need to tune the efficiency, or apply calorie tunes?


[v1.0.0-beta9] Dynamic header titles not rendering immediately

After upgrading to v1.0.0-beta9, I noticed that all of my dynamic header titles are not rendering immediately. Params are coming through as undefined until the scene finishes transitioning, then they take seem to values. This looks really janky. Sample code:
  static navigationOptions = ({ navigation }) => ({
    ...
    headerTitle: `${navigation.state.params.firstName} ${navigation.state.params.lastName}`,
    ...
  });

Result:

I can repro this on the NavigationExample app using StacksOverTabs.

Check out the small tweak from https://github.com/react-community/react-navigation/pull/1341 so headers are visible
Run the NavigationExample app
Open the "Stacks over Tabs" demo
Open various screens to see how the header behaves

The "Navigation" header works because the title is hard-coded
The "Profile" header has no title (it should show "Jordan's Profile!" like in the "Stacks in Tabs" demo, but instead it's blank)

Some combination of actions led to the exact issue in your gif where the header shows the wrong text and then refreshes after the scene finishes transitioning. I can't get it to happen again, but it seems like these issues are related.

I've also confirmed that it used to work on beta.7: https://github.com/cribspot/react-navigation/commits/better-stacks-example-beta-7

Hi! Since this issue has been quiet for a while I'm going to go ahead and close it. If the issue persists in the latest version of React Navigation please re-open!

Still broken


Background image issue #118

Darker background for more contrast.
Opacity background divs increased for readability.

Good to see that there is some activity again! The darker background is more user friendly and shows a good contrast with the visualization now 👍

But I already made the boxes less transparent that was what I meant with: Opacity background divs increased for readability.
I can make de images even darker

Well let's see what the others think

I think it looks good like this. The boxes might be better with a little less transparency, but I think that's more personal taste than readability.

I guess this can be merged now?

I think the darker background image that Clinton changed will show a clear contrast between the front part and the back. I think it looks good.


Support ECDSA keys for ACME accounts

Is your feature request related to a problem? Please describe.
The issue I'm having is that the account we currently use for issuing certificates outside of kubernetes uses ECDSA keys. We'd rather keep using the same account rather than need to make an additional rate-limit request
Describe the solution you'd like
I'd like to avoid this error message
  Conditions:
    Last Transition Time:  2022-08-02T15:51:58Z
    Message:               ACME private key in "prod-issuer-account-key" is not of type RSA
    Observed Generation:   5
    Reason:                ErrVerifyACMEAccount

Describe alternatives you've considered

Creating another account and requesting rate limit adjustment there
Trying out kube-lego
Trying out xDS type solution and setting up customized exteralsecrets

Additional context
I've tested this already
E0802 19:42:55.150182       1 sync.go:190] cert-manager/challenges "msg"="propagation check failed" "error"="DNS record for \"a.query.mongodb-dev.net\" not yet propagated" "dnsName"="a.query.mongodb-dev.net" "resource_kind"="Challenge" "resource_name"="frontend-cert-79k2r-2870715455-3435406281" "resource_namespace"="mongohouse-dev" "resource_version"="v1" "type"="DNS-01"
E0802 19:42:59.225379       1 sync.go:190] cert-manager/challenges "msg"="propagation check failed" "error"="DNS record for \"virginia-usa.a.query.mongodb-dev.net\" not yet propagated" "dnsName"="virginia-usa.a.query.mongodb-dev.net" "resource_kind"="Challenge" "resource_name"="frontend-cert-79k2r-2870715455-835765141" "resource_namespace"="mongohouse-dev" "resource_version"="v1" "type"="DNS-01"
I0802 19:44:01.946217       1 acme.go:216] cert-manager/certificaterequests-issuer-acme/sign "msg"="certificate issued" "related_resource_kind"="Order" "related_resource_name"="frontend-cert-79k2r-2870715455" "related_resource_namespace"="mongohouse-dev" "related_resource_version"="v1" "resource_kind"="CertificateRequest" "resource_name"="frontend-cert-79k2r" "resource_namespace"="mongohouse-dev" "resource_version"="v1"
I0802 19:44:01.946293       1 conditions.go:250] Found status change for CertificateRequest "frontend-cert-79k2r" condition "Ready": "False" -> "True"; setting lastTransitionTime to 2022-08-02 19:44:01.946288117 +0000 UTC m=+114.689935217

/kind feature

We have minted another account with an RSA key instead so this is no longer needed. It would be a nice feature however. Still waiting on a review of the PR


Print expiration_date to secret's versions

Print expiration_date to secret's versions

:tada: This PR is included in version 2.0.16 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Added animated header

Added an animated wave header. using CSS

Looks good to me

Thanks ❤
On Sat, 8 Oct, 2022, 9:15 pm Soumyadip Chowdhury, @.***>
wrote:

Looks good to me
—
Reply to this email directly, view it on GitHub
https://github.com/soumyadip007/hacktoberfest22/pull/5#issuecomment-1272345486,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AST3ITP6DPZ7MFKYVKBAUTLWCGJH3ANCNFSM6AAAAAARAJO6JA
.
You are receiving this because you authored the thread.Message ID:
@.***>


pycparser version?

What version of pycparser is 'cuew_gen.py' using?
I am getting some errors with the latest version of pycparser when trying to
generate files.

Just to confirm, does the issue still happen after changes I've made in 476e8795c32 ?

Just checked and your fix (in your fork) solves it. Thank you very much!

No problem, you're welcome!
Will consider this issue resolved now. But feel free to submit a new one if there is still something we're missing.


Could not determine version for package...

🐞 Bug report
Command (mark with an x)

[ ] new
[ ] build
[ ] serve
[ ] test
[ ] e2e
[ ] generate
[ ] add
[x] update
[ ] lint
[ ] xi18n
[ ] run
[ ] config
[ ] help
[ ] version
[ ] doc

Description
The ng update command fails when a package uses the git+ssh:// or https reference rather than a version number (as you usually have).  While this is definitely not best practice, it happens from now and then inside enterprise environments.
 A clear and concise description of the problem...
🔬 Minimal Reproduction

Add a package referenced via git+ssh, for instance:
"@biesbjerg/ngx-translate-extract": "git+ssh://git@bitbucket.org:r3_gis_dev/ngx-translate-extract.git#3.0.5-r3patch.01",
...
"dependencies": {
    ...
    "mypackage"<EMAIL_ADDRESS>}

Having this in the package.json run ng update @angular/core@next @angular/cli@next.
As a concrete example of an external package, this one should also fail:
...
"dependencies": {
    ...
    "material-icon-font": "https://github.com/petergng/materialIconFont.git",
}

🔥 Exception or Error

An unexpected error happened; could not determine version for package ...

🌍 Your Environment

Angular CLI: 8.3.18
Node: 11.6.0
OS: darwin x64
Angular: 8.2.13
... animations, common, compiler, compiler-cli, core, elements
... forms, language-service, platform-browser
... platform-browser-dynamic, router, service-worker

Package                            Version
------------------------------------------------------------
@angular-devkit/architect          0.803.18
@angular-devkit/build-angular      0.803.18
@angular-devkit/build-optimizer    0.803.18
@angular-devkit/build-webpack      0.803.18
@angular-devkit/core               8.3.18
@angular-devkit/schematics         8.1.2
@angular/cdk                       8.0.2
@angular/cdk-experimental          8.0.2
@angular/cli                       8.3.18
@angular/flex-layout               8.0.0-beta.26
@angular/material                  8.0.2
@angular/material-moment-adapter   8.0.2
@ngtools/webpack                   8.3.18
@schematics/angular                7.3.9
@schematics/update                 0.803.18
rxjs                               6.5.3
typescript                         3.5.3
webpack                            4.39.2

//cc @filipesilva

I think this is a regression, I remember deps we couldn't identify properly were supposed to be ignored. @clydin might know more.

Duplicate of https://github.com/angular/angular-cli/issues/13059

Closing in favor of #13059,


Support listening to specific partitions in the Kafka binder

As a user, I want to be able to set the partition set that the Kafka binder can read from for a given input, so I can have direct control over that.

Duplicates #1.


Update translation strings as of 20201124T212804

Please do not manually merge this pull request.  It will either be automatically merged or withdrawn.

Test errors found. See https://jenkins-aws.indexdata.com/job/folio-org/job/ui-plugin-find-package-title/job/PR-34/1/


YAML: LibSWOC++ update for MemArena.

A significant update to improve the memory characteristics. In particular the memory block list uses IntrusiveDList instead of shared or intrusive pointers. This simplifies the internal logic. The destructor has also been updated so that a MemArena can exist inside its own memory and still destruct cleanly.

Took a different approach with MemSpan.


Sqrt() with imaginary part == -0.0

new Complex(-0.5708, -0.0).sqrt()

returns (0.0, -0.7555130707009641) instead of (0.0, 0.7555130707009641)

Similar problem with Complex.ZERO.cos()
	@Test
	public void testCosZero() {
		Complex cosX = Complex.ZERO.cos();
		System.out.println(cosX);
		assertEquals(cosX, Complex.ONE);
	}

Couldn't we simply convert -0.0 values to 0.0 in the Complex constructors?

I'm on the fence on this.
Signed floating point 0 is a general feature from IEEE754 that has been around for decades and is useful in some places. In fact, I would even say it is more important for complex numbers that for real numbers because it allows for example to select the proper value at branch cuts. The sqrt example you give is a correct result as the point is exactly on a branch cut (the non-positive real axis). So as you are on a singularity you do experience corner-case behavior and in this case the signed 0 is there to help choosing the side of the cut.
The example for cosine is also correct, as per the definition cos(x + i y) = cos(x) cosh(y) - i sin(x) sinh(y): the minus sign implies that for x = 0 the imaginary part has a signed reversed with respect to y, hence the -0.0 you see. In this second example, there are no consequences as for (1 - 0.0 i) and (1 + 0.0 i) are in the same location in the complex plane.

I've only thought about the subsequent calculations which flipped the sign in my (internal) example.
But of course every single result makes sense of if using selecting the proper value at branch cuts.


Prevent black frames during startup

Objective
This PR addresses the issue where Bevy displays one or several black frames before the scene is first rendered. This is particularly noticeable on iOS, where the black frames disrupt the transition from the launch screen to the game UI. I have written about my search to solve this issue on the Bevy discord: https://discord.com/channels/691052431525675048/1151047604520632352
While I can attest this PR works on both iOS and Linux/Wayland (and even seems to resolve a slight flicker during startup with the latter as well), I'm not familiar enough with Bevy to judge the full implications of these changes. I hope a reviewer or tester can help me confirm whether this is the right approach, or what might be a cleaner solution to resolve this issue.
Solution
I have moved the "startup phase" as well as the plugin finalization into the app.run() function so those things finish synchronously before the "main schedule" starts. I even move one frame forward as well, using app.update(), to make sure the rendering has caught up with the state of the finalized plugins as well.
I admit that part of this was achieved through trial-and-error, since not doing the "startup phase" before app.finish() resulted in panics, while not calling an extra app.update() didn't fully resolve the issue.
What I can say, is that the iOS launch screen animation works in such a way that the OS initiates the transition once the framework's didFinishLaunching() returns, meaning app developers must finish setting up their UI before that function returns. This is what basically led me on the path to try to "finish stuff earlier" :)
Changelog
Changed

The startup phase and the first frame are rendered synchronously when calling app.run(), before the "main schedule" is started. This fixes black frames during the iOS launch transition and possible flickering on other platforms, but may affect initialization order in your application.

Migration Guide
Because of this change, the timing of the first few frames might have changed, and I think it could be that some things one may expect to be initialized in a system may no longer be. To be honest, I feel out of my depth to judge the exact impact here.

This looks like it's solving the problem that https://github.com/bevyengine/bevy/pull/9692 supplied a work-around for, but in a more elegant way. If it does, can you revert that change as part of this PR?

@alice-i-cecile Oh yes, someone pointed me to the fix from #9692 on Discord, although it did not solve the iOS issue. I don't think it resolved the flicker I witnessed on Wayland either, but maybe it was trying to? I can revert it and see if I notice anything off, but I don't have a Windows machine to test on, so in that case maybe @hymm could verify if it still works correctly there (I see they tested the previous fix on Win 11)? Just let me know how you would like me to proceed :)

Yikes, those failing DX12 examples does some to be an issue with this branch :(

This should be tested on the other platforms that are peculiar about setup, mostly android and wasm/WebGPU

Doesn't work at all for me in windows 10.
    Finished dev [unoptimized + debuginfo] target(s) in 0.51s
     Running `target\debug\examples\3d_scene.exe`
2023-09-16T18:18:07.189173Z  INFO bevy_winit::system: Creating new window "App" (0v0)
2023-09-16T18:18:07.644783Z  INFO bevy_diagnostic::system_information_diagnostics_plugin::internal: SystemInfo { os: "Windows 10 Pro", kernel: "19045", cpu: "AMD Ryzen 7 5800X 8-Core Processor", core_count: "8", memory: "63.9 GiB" }
2023-09-16T18:18:08.024971Z  INFO bevy_render::renderer: AdapterInfo { name: "NVIDIA GeForce RTX 4080", vendor: 4318, device: 9988, device_type: DiscreteGpu, driver: "NVIDIA", driver_info: "537.34", backend: Vulkan }
thread 'Compute Task Pool (6)' panicked at 'called `Option::unwrap()` on a `None` value', crates\bevy_render\src\pipelined_rendering.rs:144:84
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
error: process didn't exit successfully: `target\debug\examples\3d_scene.exe` (exit code: 101)

Ah, that looks very similar to the CI failure for DX12. I don’t have Windows, but maybe the location of the panic provides a hint.

So it would appear the RenderToMainAppReceiver resource is not added yet:  https://github.com/bevyengine/bevy/blob/c61faf35b8089dd2d6d394aa907d4756de1e31ce/crates/bevy_render/src/pipelined_rendering.rs#L144
Need to figure out why though…

You can't run the schedule before finish.

@hymm I've reverted that part of PR, but unfortunately it's still breaking on Windows. Would you maybe have another suggestion?

I suppose worst case I could just put a #[cfg(platform = "ios")] guard around my change. It would make sure this doesn't affect anything else, but it's a bummer since it might introduce other platform-dependent behaviors in people's applications, I'm afraid.

All green now. Thanks for all the help, @mockersf !

Btw, I was wondering. Maybe I should take the finished_and_setup_done boolean from the winit plugin and let it be passed as an argument to the runner instead. It would save a call to app.ready(), but more importantly it would make the contract a bit more explicit and would provide a place to document what behavior is expected from the runners with regards to the setup.

looks good! could you solve conflicts with main?

@mockersf I've rebased on top of main. Should be all good now.

@mockersf, do you think this change should be controversial now?


Add check for the required number of commits in the PR

Many projects require only one commit per PR. We should add a check that verifies user-defined number commit s per PR. This should also support a range regex if needed. So the config could look like this:
numberOfCommits: 1

or
numberOfCommits: 1..5

Many projects require only one commit per PR. We should add a check that verifies user-defined number commit s per PR. This should also support a range regex if needed. So the config could look like this:
numberOfCommits: 1

or
numberOfCommits: 1..5

Do you have any bounds for the maximum of commits?

@xvanick1 not really, I think possitive short range should be enough. If not, we will change it later.


Allow automatic discovery/failover of the AMQ Broker

The amq: endpoint works exactly like the activemq: endpoint in Apache Camel; only it uses the fabric to automatically discover the broker. So there is no configuration required; it'll just work out of the box and automatically discover whatever ActiveMQ message brokers are available; with failover and load balancing.
[http://fabric8.io/gitbook/camelEndpointAmq.html]

I'm not familiar with Camel and AMQ but there are some pointers that could help you:
An ActiveMQ Artemis JMS connection factory can be configured to either use a static list of servers to connect to or use a discovery mechanism to learn the cluster topology.
This is used to create an initial connection. Once this connection is established, the client is notified of any cluster changes (including failover/failback).
It looks like you would need to use org.hornetq.api.jms.HornetQJMSClient#createConnectionFactoryWithHA(org.hornetq.api.core.DiscoveryGroupConfiguration, org.hornetq.api.jms.JMSFactoryType) to create a JMS ConnectionFactory that discovers the cluster. The DiscoveryGroupConfiguration itself must match the JMS brokers configuration and can be backed by either JGroups or Artemis own UDP stack.

The ActiveMQ component should sufficiently cover this use case
        ConnectionFactory connectionFactory = createConnectionFactory();
        ActiveMQComponent activeMQComponent = new ActiveMQComponent();
        activeMQComponent.setConnectionFactory(connectionFactory);
        camelctx.addComponent("activemq", activeMQComponent);


[Task - EE]: themes should inherit policies from applications when application related permissions are updated for roles.

Is there an existing issue for this?

[X] I have searched the existing issues

SubTasks
No response

reopening the issue As we are facing another critical issue from this fix,
The internal server error is displayed when user tries to create custom role


Wanted: Exception when provided an insufficient buffer

When trying to decode data using uncompress function I receive an Unhandled exception (Access violation).
The size sending to function is too small, and I expected to get NotEnoughStorage exception so the buffer will resize to the right uncompressed size.

I have renamed the issue.

To be clear, currently, you can avoid this problem by providing sufficient buffer memory. For example, if you allocate 2x the original memory plus, say, 1024, you will never run out.

I don't know what is  the original size before it was encoded.
in this case, what should be the multiplier to allocate size by decoded data size.

+1 for how to calculate the decompressed size if you only have compressed size.

This issue is marked as "help wanted" and there is nobody working on it as far as I know.

I'm wondering if the easiest way would be to put the uncompressed length at the start of the buffer (just as a plain uint32_t).

@cs96and

I'm wondering if the easiest way would be to put the uncompressed length at the start of the encoded buffer (just as a plain uint32_t).

It generally works this way already. It is not a format problem, it is an API issue.

I created a new issue https://github.com/lemire/SIMDCompressionAndIntersection/issues/20


bad highDpi support with plugins

Operating system or device - Godot version:
macOS
2.1
Issue description:

when using plugins highDpi has issues. the UI elements themselves are scaled correctly, but the spacing/layout is the same than in loDpi which results in overlapping.
Also icons are loDpi.
Steps to reproduce:
create a project
install a plugins (I used the git plugin)
set the editor to highDpi
open the git panel

This should be opened on respective issues themselves -- we shouldn't support badly-written code with hardcoded values just because it is "plugins".

(Actually, I'm not sure about the icons, but if the plugin creator confirms that it is hard-to-impossible to make those scale right, I can reopen it for them specifically)


Find the Distance between two nodes in Binary Search Tree

Aim

To Find the Distance between two nodes in Binary Search Tree. Find the distance between two keys in a binary tree, no parent pointers are given.

Details

This code will find the distance between two nodes in Binary Search Tree.
The distance between two nodes is the minimum number of edges to be traversed to reach one node from another.

Programming language

[ ] C
[ ] C++
[x] Java
[ ] Python

Link for the problem - https://practice.geeksforgeeks.org/problems/min-distance-between-two-givennodes-of-a-binary-tree/1 ]

Please assign me with this problem as I would like to solve it.

Can I implement in Java? I'm a participant of LGM Soc21.

I can also solve it in Java. I have coded it and it's perfect. Please assign me with this problem

I have implemented it so that anyone coding in java can easily understand since I have used proper comments along with time and space complexity.

@Saurabh4626 please assign me in cpp under LGMSOC21

I jave already created a pull request please look into it


Is it possible to compare to instances by actual value?

In the following scenario
BigNumber.config(6, BigNumber.ROUND_HALF_UP);
var a = new BigNumber("0.000001”);
var b = a.dividedBy(2);

The actual value of a and b are the same, as the e.e. both return 0.000001 from toString().
But when I compare them a.comparedTo(b); it returns 1 as such of course a.equals(b) returns false.
Is that an expected behaviour, because I couldn't find something in the docs of comparedTo or equal that speak about this case.
And if it is, is there a way (in the API of bignumber.js) to compare against the actual value without doing a string conversion?

Hi,
Sorry for the delay. I didn't receive a notification about this issue for some reason.
No, this is not expected behaviour. It is a bug and its disappointing.
I will fix ASAP.

Fixed in v2.0.8
Thanks very much for the report.
Notes:
This is the second bug that has been discovered since the switch to using a higher base internally. The test cases did not reveal it because they mostly just check the string representation and not the actual values of the properties of each big number after an operation. This bug showed that numbers of the form 0.1, 0.01, 0.001 etc. may not have the correct internal representation if they arose as the result of rounding after a division operation. The string representation of the numbers would be correct, but their internal inconsistency (the first value of the coefficient array could be the wrong power of ten) could cause the comparedTo method to produce incorrect results.

are you going to publish this version so it can be updated via npm, or do I need to update it manually?

I've done it. Thanks for the reminder.

:+1: thx


Add www.tinytronics.nl

Could you please add www.tinytronics.nl it's a webshop that sells selectronics in the Netherlands

Added!


Logitech led SDK not longer working with Aurora 0.6.3

Expected behavior:
Logitech LED SDK starting with no issue and keyboard receiving Aurora's lighting
Actual behavior:
Logitech LED SDK not starting and keyboard not receiving Aurora's lighting (still taking Logitech Assistant's lighting instead)
How often does this problem occur?
This problem occurs every time.
Please describe the steps to reproduce this issue:
The issue can be reproduced by doing following steps:

Updating Aurora from 0.6.2b to 0.6.3
Start Aurora

What is the version of Aurora you are experiencing this issue with?

My version is 0.6.3
Does the previous version of Aurora have this issue?
The previous version, 0.6.2b did not have this issue.
Please include any logs, screenshots, or files that are related to this issue:
https://pastebin.com/7wcKCL3b
https://imgur.com/a/HOy40Zy

Are you using Logitech GHub? If so, see #1264

No, I was not using it. I Just installed GHub to try if it would not fix my issue but it's still not initializing

Did you use the G Hub build of Aurora after you installed it?

@simon-wh I have this issue as well after getting 0.6.3 and not using G Hub. Reinstalling fixes the issue, but is concerning updating broke it in the first place.

@JMZ10 Hm, that's really strange, updating shouldn't have broke it

I confirm that uninstalling and re-installing fixed my issue as well.

Turned out that the updater was downloading the G-Hub version that was added in the assets of the release. If anyone experiences this issue then reinstall, should be sorted now

So hey I'm having the same issue here. After I have updated from 0.6.0 to 0.6.3. I have uninstalled both (the Logitech & Aurora) and reinstalled both. Multiple times and does not seem to work for me. And I'm not using G-Hub at all.  Not sure how to upload a log file, but here hopefully this works.
2018-09-27 18.34.45.log

@YaLaMu939 you have the g huh version of Aurora. Reinstall Aurora.

@YaLaMu939 you have the g huh version of Aurora. Reinstall Aurora.

Wow thank you! I have know idea how you knew that, but it worked. Didn't know there where two different installs.

@YaLaMu939 i got the version from the log. glad you got it working.


Support Resource in the Prometheus Exporter

The Resources were added to the default SDK in opentelemetry-js#846. They need to be integrated in the export workflow of the Prometheus exporter similar to opentelemetry-js#843

What is the desired behaviour to support Resource?  If there is clear guidance, I should be able to implement it.
I tried appending resource.attributes to the attributes for each Prometheus metric, but that looks like it might be too many attributes or too much data (especially for the unit tests, which have a full path to Node.js as the service.name).

It is going to be specified at https://github.com/open-telemetry/opentelemetry-specification/pull/2266 to drop all resource attributes, only converting job and instance to Prometheus labels.

Hi! To clarify things on my end: is this about exposing the resource attributes passed, for example to MeterProvider, in the exporter as the "target" info family metric like described here ?

This is about including resource attributes as part of the serialized prometheus output data. As far as I know this behavior is actually not in the specification so this issue might actually just be closed as not planned. It is open so we can determine what other SIGs do with resource data in prometheus and attempt to be consistent.

We have spec texts on how to transform OpenTelemetry resource attributes to Prometheus labels: https://github.com/open-telemetry/opentelemetry-specification/blob/main/specification/metrics/data-model.md#resource-attributes-1.

So we should be able to update the PrometheusSerializer.serialize function so that it also transforms the resource attributes available in the ResourceMetrics input to a target info group. Is there some other places that needs to be updated ?

There is a related spec issue right now to determine precedence of keys https://github.com/open-telemetry/opentelemetry-specification/issues/2535

@legendecas I'm trying to evaluate the work required for our use case atm. I'm still questioning what should happen downstream, in the collectors, as the info type does not appear to be supported. I've tried two parsing libraries and both ended in a failure to parse the metric type info. So I assume this needs a corresponding implementation in the other end ?

We need to transform the resource attributes into an info type in the PrometheusSerializer.serialize and I was wondering how the client pulling this data should interpret them. It's out of this issue's scope but I didn't find relevant informations. Currently the clients choke on the info metric type so my assumption is that clients would have to support this new type and have specific logic tying the attributes of the info block to the other metrics in the payload.


Error: An unknown RPC error occurred - WalletConnect sendTransaction

Well, this error appeared on wagmi docs page, but here is the build with same error @tmm
https://github.com/meri-maki/wagmi-issue
https://wagmi-issue.vercel.app/
To reproduce:

Connect to any wallet via WalletConnect (except for metamask)
Try sending transaction
Get

Error: An unknown RPC error occurred. Request Arguments: from: 0x568A33aA237fFc284FF584A3BAB17A44FAeF4054 to: 0xe8078b5198e572be8d8d412511d48b7d0f5e9a1c value: 0.0001 ETH gas: 104799 Version<EMAIL_ADDRESS>
Originally posted by @meri-maki in https://github.com/wagmi-dev/wagmi/issues/2694#issuecomment-1626945905

Seems to work fine on my wallets on my device, so I suspect this may be a wallet implementation error. What wallet are you using?

I tried Zerion and TrustWallet, transactions are not appearing on my mobile apps (as metamask does)

Just tried connection to zerion via WC on wagmi
Send transaction resulted in this:

Just tried UniSwap and Rainbow wallet, transaction go through fine even with error in console:
Failed to launch 'wc:6c82c0f8e6a85b0c49460a305b0b4a4266edbcf5c5e2c07b68466709758f57c4@2/wc?requestId=1688814266197988&sessionTopic=27b1c0e444a3fbc550a44253b11827ee91bda4b8114af99ddd0bd7f1c887c092' because the scheme does not have a registered handler.

So i guess it might be an error with wallet connect and its connection with zerion.trust?

Hello @meri-maki  have this issue been resolved yet ? i will suggest you contact wallet connect support for assistance Wallet support

Managed to reproduce this with a plain @walletconnect/ethereum-provider, so seems to be an implementation error in the wallet. Let’s close this and we can track this on the WalletConnect repo.

@jxom thank you for taking a look.. I was getting really anxious because my app is getting into production really soon and there is no time to write a whole new logic so zerion wallet works
hope the answer fro mWalletConnect will be there soon

same issue here
i am using wagmi/core
the signMessage function works fine if i connect metamask by browser plugin but if i connect metamask from my mobile scanning the qr code I got this error

NOTE: the address connection works even from mobile/qrcode

add data: "0x" to usePrepareSendTransaction help fix my issue.
https://github.com/wagmi-dev/wagmi/issues/2161#issuecomment-1503030971

adding data: "0x" to usePrepareSendTransaction fixed issues with transactions with Zerion and walletConnect


Semicolon as an alias for REM

Later versions of Microsoft BASIC uses the ‘ (apostrophe) character as a REM substitute, and the semicolon is used in other BASIC commands (specifically in the PRINT statement.)
Wouldn’t it make more sense to use ‘ as a REMark token?

I'd like to use the single quote for character literals in a later version and XC=BASIC doesn't use semicolon in the PRINT statement, that's why I chose the semicolon.
But actually the grammar should be smart enough to recognize whether a character is within an expression or it's the beginning of a statement. So what about using both as aliases for REM?

I have seen some languages allow the semicolon,  but only if it’s the first character on the line or statement.

Implemented in latest develop branch.


Erlang compiled for Windows on Linux.

https://github.com/erlang/otp/issues/5634#issuecomment-1022383187

Postponed, not directly related to VR locomotion and networked object picking UIUX.


Verify the manifest.json file is good

If I want to easily be able to have this install onto android and IOS devices, I need to make sure that the manifest file is in order.  Once in place, update the About page as well to include that.

Home screen loads fine in Android.  Lacks  proper iconography in IOS.


Need to support the new integrity feature

On the webpack branch, something went wrong:

Have you updated to the latest (preview) version of Piral?
There is a fix for this out.

Tried to use the latest version<EMAIL_ADDRESS>but an error occurred while building the app-shell:

Only update piral-core. The webpack plugin is not yet compatible with piral-cli 0.11.6.

OK👌


Adding NEOPIXEL to board definitions

Just a suggestion really :)
I've added the following line to both my version of kfw_s2_board.py and kfw_pico_board.py which has allowed some of my generic RGB code to run:
NEOPIXEL = D11
This should obviously be added below the point where D11 is assigned :)

Hmm, you're 100% right, I'm surprised I missed that myself! Any chance you could submit a PR?


Lack of state of authentication and reconnection

The current library lacks the state of authentication. It is still fine until the library reconnects to bitfinex without letting the rest of the code know. Is there any way to notify the other side of the program to re-issue authentication? Maybe a queue containing reconnection signals?
I am happy to implement it and make a PR if you are not available to do so. Let's agree on a way to achieve this if there isn't an existing way. Many thanks!

Hey @carsonip !
Thanks for contributing!
Issue #49 already reported this,  and the current dev branch should already have fixed this.  Feel free to confirm this is handled (and of course to report if it isn't).
Thanks!

That's great. I'll get back to you after I test it.


Update version: ifedapoolarewaju.IGdm version 3.0.4

Pull request has been created with WinGet Automation :rocket:
Microsoft Reviewers: codeflow:open?pullrequest=https://github.com/microsoft/winget-pkgs/pull/114974&drop=dogfoodAlpha

@wingetbot run

Automatic Validation ended with:


Fixes #5293 (Report number of unknown validators)

PR Description
Now validator status is reported as unknow until explicitly updated otherwise.
Fixed Issue(s)
#5293
Documentation

[x] I thought about documentation and added the doc-change-required label to this PR if updates are required.

Changelog

[x] I thought about adding a changelog entry, and added one if I deemed necessary.

Yes, it seems related to total validators, that are counted using local_validator_counts in the AT. It seems to me AT should be updated, but I dig more.

I fixed the acceptance test. The problem was with the initialization of the network: by default, it starts with 64 validators in total, and they are all owned by the user.
The initial setting of the previous scenario for the acceptance test was: 8 total validators, with 64 owned by the user. This is clearly an error but it was not noticed before because the check is done using local_validator_counts, and that metric did not return unknown values until now, so it was always equal to the validators owned by the user. With unknown status now visible, the bug emerged.
Now the settings for the scenario are correct (or at least it seems to me what was intended for that test, let me know what you think), so there are 64 total validators in the network, with 8 validators owned by the user, and tests pass.


Cannot Extend DimensionEssentials

Is it just me or we cannot extend DimensionEssentials?  The only option is to customize.  We're running 1611.

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

ID: 6606b52c-677f-bf60-880d-3ec4dd03fb36
Version Independent ID: 75150ec2-39e8-64e9-524e-798a2c480bd0
Content: Make a backing table be consumable as a Financial dimension - Finance & Operations | Dynamics 365 | #MSDyn365FO
Content Source: articles/dev-itpro/financial/dimensionable-entities.md
Service: dynamics-ax-platform
Product: unspecified
GitHub Login: @aprilolson
Microsoft Alias: aolson

I assume you are referring to the Security Privilege, DimensionEssentials, Steve.  Security privileges are not extensible by design, although roles and duties are extensible.

Closing this issue. Please let us know if it isn't clear.

Yes, I was referring to the Security Privilege.  So in the topic "make a backing table be consumable as a Financial Dimension", step 1.14, since DimensionEssentials cannot be extended you tell us that we have to do a customization?  What confuses me is that it cannot be done with the latest version since it's "Hard Sealed".  Every changes as to be an extension.

Thanks for the details, Steve.  I'm going to need to do some follow up on this.  I will get back to you as soon as I have a better answer... early next week at the latest.

Apologies for the delay here, but I don't have a final recommendation on this.  Thanks for your patience.


Garbage collector error

Hello,
We face a problem with the garbage collector which prevents the registry to start:
Log of the GC container:
failed to garbage collect: failed to mark: filesystem: filesystem: failed to retrieve tags unknown repository name=xxxxx
The failling GC container prevents the registry container to start.

related to #120

Hi @gitarns , we merged https://github.com/enix/kube-image-keeper/pull/120 , would you mind trying with the version 1.3.0-beta.2, please?


don't echo readfile's return

Since readfile() echoes the given file directly to the output buffer, it is not necessary to echo its return value. In addition, echoing the number of bytes read could mess up the download in environments with output compression.
See http://php.net/manual/en/function.readfile.php

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/#/issues/SW-18241.
Please use this issue to track the state of your pull request.

hey @jeboehm
thanks for your PR :+1:
it will be merged soon into SW 5.2


bug

the gddl search cant be open with the mod integrated demonlist and the integrated demonlist button is hidden by the gddl search button
both of mods latest version
device android 32bit

Doing that should fix the issue (it's quite weird that the integrated demon list button has a higher touch priority even though it's under the gddl search button)

I'm going to assume that it worked since you haven't responded saying that it doesn't and close this issue tomorrow


Weird window closing button being present

For some reason, I have a weird issue of the closing button (top right corner) being present. No idea why it is happening. No changes are made to userchrome.css

What desktop environment are you using?

This is on gnome 3.38.

I'm getting a similar thing on Fedora 33.


v3 vaccine edit

Description
Rejigs the add vaccine screen(s) as per the new design to gather further data.
Full requirements here:
https://www.notion.so/joinzoe/017c3de591604a7b89d0b69e3c37b279?v=0133357df3ed42e1bd42c1e0d7afcf45&p=8c3e4d43cdfe459da2d65f5426b1b94d
Design: https://www.figma.com/file/7TEcX7f6rw3Z0WYR4TpX3R/COVID-Symptom-Study-(App)---MASTER?node-id=6743%3A3918&viewport=-2281%2C354%2C0.5
How to test locally
Run the corresponding BE branch if not merged https://github.com/zoe/covid-app/pull/387 . If merged, point against staging or use master.
Access page: home > profile > not had test.
Observe no vaccines + "Add Vaccine" button
Tap "Add Vaccine" button. US/GB users should have a blue link to another screen with image examples (SE not).
Add a DOSE ONE vaccine. To start, just add a date and save. This will let you see the incomplete vaccine/doses list, plus the prompt edit.
From here, edit vaccine and complete fields as required. Note changing the date in edit mode will prompt alert on save.
Delete vaccine now at bottom of dose in edit mode only.
On what platform have you tested the change?

[ ] Android - Emulator
[ ] Android - Physical device
[x] iOS - Emulator
[ ] iOS - Physical device

Screenshots
Vaccine with a dose that is missing a date (easiest way to replicate is to create a dose, save it, then manually delete its date in django admin)

Dose 1 complete

Both doses complete

Example vaccine doc screen

Checklist

[ ] I have updated mockServer
[ ] I've added analytics as relevent
[ ] I have run npm test
[ ] I have run npm run test:i18n

Out of scope and potential follow-up
Approach followed the Formik form model in which one parent form is passed down to the props, and reused props check if first dose and select properties based on that. Would be good to make this less rigid.
Setting second dose to 'no' on edit mode doesn't delete it yet. Ticket created.

This is a great peace of work @bionara !

This is a great peace of work @bionara !


reduce the number of calls to findTarget

Would you be able to resolve the conflicts against the develop branch ?
thanks

Closing, the better method to use is the Before/after instead of the interceptor.


BUG: Fixing Report multiprocessing

Reference issue
Example: Fixes #9101.
What does this implement/fix?
This fixes the bug appearing when calling Report().parse_folder() with n_jobs>1 by adding projs to the list of pickled attributes in _get_state_params().
Solution suggested by @larsoner

test_report returns 6 errors out of 23, which all seem to related to:
ModuleNotFoundError: No module named 'tvtk.custom.renderer

And they also appear in the main branch.

Locally I can't replicate that issue, at least on main. Can you try:
pip uninstall mayavi
pip install --no-cache mayavi

and see if it helps?

Yes, that helped (although I just seem to have replaced the same version of mayavi (4.7.2))

If you updated vtk in between it breaks Mayavi...

No everything passes (except one Deprecation-Warning:

Maybe pip install --upgrade pyreadline? I have 2.1 and don't see this deprecation warning

If you updated vtk in between it breaks Mayavi...

No everything passes (except one Deprecation-Warning:

Maybe pip install --upgrade pyreadline? I have 2.1 and don't see this deprecation warning

pyreadline is already 2.1 and the deprecation-warning stays. But I use Windows, maybe that makes a difference?
By the way, what method would you recommend to keep the development-conda-environment up to date?
conda env  update -n mnedev -f environment.yml?
And/or something pip-related?

@marsipu can you add an entry in doc/changes/latest.inc to mention this fix? Then I think we can merge. We could in principle add a test but it's often a problem to do so with parallel processing. One potential test would be to pickle and unpickle and ensure .projs is there -- this should fail on master and pass on this PR I think.

I added a test which just tests parse_folder() and save() of Report after pickling the object, which fails on main. What do you think?
Thanks.

thx @marsipu

Great job!!