Circular dependency handling is not working on libcrux ml-dsa

The implementation for #396 probably has a bug.

I found another issue. Here is a reproducer:

pub mod a {
    pub fn f () {super::b::h()}
    pub fn g () {}
}
pub mod b {
    pub fn h () {}
    pub fn i () {super::a::g()}
}

Open this code snippet in the playground
The problem is that the we look for chains of items making a module depend on itself but we don't need a chain, we can have the dependency from a to b and from b to a use completely disjoint items.
In this example we need to bundle only one item to break the cycle (g or h).
One way to break these cycles that has an arbitrary part is to look for module level cycles (M1 -> M2 -> ... -> Mn -> M1). Take one module dependency that is in the cycle ( for example Mn -> M1). Look for all items in M1 that are in the dependencies of Mn. Bundle them together with all there dependencies in any module that is part of the cycle. This is not so nice because choosing arbitrarily which part of the cycle we should break can lead to bundle many things when there can be another solution minimizing the amount of items to bundle.
A less naive solution would be to look for the best edges to break in the module dependency graph (minimum number of edges, we could even do a weighted version where an edge in the module dependency graph has a weight that is the number of edges in the item dependency graph that correspond). This is the minimal feedback arc set problem. And there is a solution for that in the ocamlgraph library.

A less naive solution would be to look for the best edges to break in the module dependency graph (minimum number of edges, we could even do a weighted version where an edge in the module dependency graph has a weight that is the number of edges in the item dependency graph that correspond). This is the minimal feedback arc set problem. And there is a solution for that in the ocamlgraph library.

This is still an imperfect solution because the module dependency graph is missing the information of how much subsequent items will need to be bundled to break an edge.


Thousands separator

It would be interesting in face of the counter to have an option of thousand separator.
For very large numbers is hard to read them.

+1 on this feature

I'm just doing this now for a project—here looking at scaling—so this isn't final, or overly pretty. But, add this in a separate JS before you call it.
(function($) {

	/**
	 * Counter Clock Face
	 *
	 * This class will generate a generice flip counter. The timer has been
	 * disabled. clock.increment() and clock.decrement() have been added.
	 *
	 * @param  object  The parent FlipClock.Factory object
	 * @param  object  An object of properties to override the default
	 */

	FlipClock.CashFace = FlipClock.Face.extend({

		/**
		 * Tells the counter clock face if it should auto-increment
		 */

		shouldAutoIncrement: false,

		/**
		 * Constructor
		 *
		 * @param  object  The parent FlipClock.Factory object
		 * @param  object  An object of properties to override the default
		 */

		constructor: function(factory, options) {

			if(typeof options != "object") {
				options = {};
			}

			factory.autoStart = options.autoStart ? true : false;

			if(options.autoStart) {
				this.shouldAutoIncrement = true;
			}

			factory.increment = function() {
				factory.countdown = false;
				factory.setTime(factory.getTime().getTimeSeconds() + 1);
			};

			factory.decrement = function() {
				factory.countdown = true;
				var time = factory.getTime().getTimeSeconds();
				if(time > 0) {
					factory.setTime(time - 1);
				}
			};

			factory.setValue = function(digits) {
				factory.setTime(digits);
			};

			factory.setCounter = function(digits) {
				factory.setTime(digits);
			};

			this.base(factory, options);
		},

		/**
		 * Build the clock face
		 */

		build: function() {
			var t        = this;
			var children = this.factory.$el.find('ul');
			var time 	 = this.factory.getTime().digitize([this.factory.getTime().time]);

			if(time.length > children.length) {
				$.each(time, function(i, digit) {
					var list = t.createList(digit);

					list.select(digit);
				});

			}

			$.each(this.lists, function(i, list) {
				list.play();
			});

			if ( time.length > 3 ) {
				this.createDivider();
				$(this.dividers[0]).insertBefore(this.lists[this.lists.length - 3].$el);

				if ( time.length > 6 ) {
					this.createDivider();
					$(this.dividers[1]).insertBefore(this.lists[this.lists.length - 6].$el);

					if ( time.length > 9 ) {
						this.createDivider();
						$(this.dividers[2]).insertBefore(this.lists[this.lists.length - 9].$el);

						if ( time.length > 12 ) {
							this.createDivider();
							$(this.dividers[3]).insertBefore(this.lists[this.lists.length - 12].$el);
						}
					}
				}
			}

			this.base();
		},

		createDivider: function(label, css, excludeDots) {
			if(typeof css == "boolean" || !css) {
				excludeDots = css;
				css = label;
			}

			var dots = [
				','
			].join('');

			if(excludeDots) {
				dots = '';
			}

			label = this.factory.localize(label);

			var html = [
				'',
					''+(label ? label : '')+'',
					dots,
				''
			];

			var $html = $(html.join(''));

			this.dividers.push($html);

			return $html;
		},

		/**
		 * Flip the clock face
		 */

		flip: function(time, doNotAddPlayClass) {
			if(this.shouldAutoIncrement) {
				this.autoIncrement();
			}

			if(!time) {
				time = this.factory.getTime().digitize([this.factory.getTime().time]);
			}

			this.base(time, doNotAddPlayClass);
		},

		/**
		 * Reset the clock face
		 */

		reset: function() {
			this.factory.time = new FlipClock.Time(
				this.factory,
				this.factory.original ? Math.round(this.factory.original) : 0
			);

			this.flip();
		}
	});

}(jQuery));

CSS
.flip-clock-wrapper .flip-clock-dot.comma {
	display: block;
	width: auto;
	height: auto;
	position: absolute;
	border-radius: 0;
	box-shadow: none;
	background: transparent;
	bottom: -10px;
	font-size: 70px;
	margin-left: -5px;
}

Call the Clock
var clock = $('#drawvalue').FlipClock( 250000, {
	clockFace: 'Cash'
});

@rjksn Thank you for this! I was able to modify some of the code to add styled commas and dollar signs.
My problem is when we setValue() from say $10000.00 down to $100.00, it reads as $00100.00. Where in this code would I remove the digits that are no longer there? Also, have you found a way to make this work with cents? ex $100.50. It works but the commas are misplaced.

I didn't need to for my use case, and thankfully I'm only counting up! I
believe it happened there as well going from say 1e5 to 1e6, I don't think
I bothered solving that, since I was counting donations I didn't break into
the tens of millions.
On Mon, Jun 13, 2016, 1:50 PM Steve Chester<EMAIL_ADDRESS>wrote:

@rjksn https://github.com/rjksn Thank you for this! I was able to
modify some of the code to add styled commas and dollar signs.
My problem is when we setValue() from say $10000.00 down to $100.00, it
reads as $00100.00. Where in this code would I remove the digits that are
no longer there? Also, have you found a way to make this work with cents?
ex $100.50. It works but the commas are misplaced.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/objectivehtml/FlipClock/issues/91#issuecomment-225656825,
or mute the thread
https://github.com/notifications/unsubscribe/ABHSUSAGq6ClKS7MI_pHKom23fhz4Go4ks5qLZhOgaJpZM4BuYzj
.

--
Ryan Jackson |<PHONE_NUMBER> |<EMAIL_ADDRESS>

New version of this library can handle alpha characters. In fact anything with a toString() method can be digitized and set in the clock face. Closing this old issue, but will try to get an example of this for the new version.


Cannot read property '_id' of undefined at Datastore.prepareDocumentForInsertion

<EMAIL_ADDRESS>Not quite sure what the repro is (somewhat big stack on top of NeDB), but there is a promise that fails (possibly not returned to caller?), that causes node to complain:
(node:6320) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 1): TypeError: Cannot read property '_id' of undefined
(node:6320) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

and here is the call stack
unhandledRejection TypeError: Cannot read property '_id' of undefined
    at Datastore.prepareDocumentForInsertion (/home/andre/projects/greeny/services/node_modules/nedb/lib/datastore.js:385:21)
    at Datastore._insert (/home/andre/projects/greeny/services/node_modules/nedb/lib/datastore.js:348:24)
    at /home/andre/projects/greeny/services/node_modules/nedb/lib/executor.js:40:13
    at Object.process (/home/andre/projects/greeny/services/node_modules/async/lib/async.js:731:21)
    at next (/home/andre/projects/greeny/services/node_modules/async/lib/async.js:728:27)
    at Immediate._onImmediate (/home/andre/projects/greeny/services/node_modules/async/lib/async.js:24:16)
    at runCallback (timers.js:785:20)
    at tryOnImmediate (timers.js:747:5)
    at processImmediate [as _immediateCallback] (timers.js:718:5)

It might not be NeDB's fault (e.g. caller passed some bogus parameters) but should at least null checks be in place and an explicit exception thrown instead?
I'm trying to get a repro with only NeDB - I'll update this issue if I am able to.

I'd check downstream first. Looks like someone isn't handling rejected
promises correctly anyway.
On Nov 4, 2017 10:17 PM, "Andre Esteve"<EMAIL_ADDRESS>wrote:

<EMAIL_ADDRESS>Not quite sure what the repro is (somewhat big stack on top of NeDB), but
there is a promise that fails (possibly not returned to caller?), that
causes node to complain:
(node:6320) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 1): TypeError: Cannot read property '_id' of undefined
(node:6320) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.
and here is the call stack
unhandledRejection TypeError: Cannot read property '_id' of undefined
at Datastore.prepareDocumentForInsertion (/home/andre/projects/greeny/services/node_modules/nedb/lib/datastore.js:385:21)
at Datastore._insert (/home/andre/projects/greeny/services/node_modules/nedb/lib/datastore.js:348:24)
at /home/andre/projects/greeny/services/node_modules/nedb/lib/executor.js:40:13
at Object.process (/home/andre/projects/greeny/services/node_modules/async/lib/async.js:731:21)
at next (/home/andre/projects/greeny/services/node_modules/async/lib/async.js:728:27)
at Immediate._onImmediate (/home/andre/projects/greeny/services/node_modules/async/lib/async.js:24:16)
at runCallback (timers.js:785:20)
at tryOnImmediate (timers.js:747:5)
at processImmediate [as _immediateCallback] (timers.js:718:5)
It might not be NeDB's fault (e.g. caller passed some bogus parameters)
but should at least null checks be in place and an explicit exception
thrown instead?
I'm trying to get a repro with only NeDB - I'll update this issue if I am
able to.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/louischatriot/nedb/issues/532, or mute the thread
https://github.com/notifications/unsubscribe-auth/ACA19pxxIUawAZHXf6oLDeqLhBc3VSQRks5szRqigaJpZM4QSO89
.


dependabot fails to update indirect dependencies of nested go modules (cmd/kafka-sampler and cmd/neblictl)

Describe the bug
PRs that dependabot creates fail CI checks due to not updating the nested modules indirect dependencies by runnning go mod tidy.
Expected behavior
dependabot should update indirect dependencies as well all in the same PR

waiting for an answer in this dependabot issue: https://github.com/dependabot/dependabot-core/issues/6678


code to fine-tune

How can I fine-tune using model.py with my custom dataset and the pre-trained model.pt?

We don't currently plan to release training or fine-tuning examples in this repo, but in general fine-tuning is done with:

Load the pre-trained model, usually with a prediction "head" for your fine-tuning task.
Initialize an optimizer, with an appropriate learning rate which you'll need to tune.
Repeat: feed the data through the model (and the head) and update your model to minimize your task-specific loss.
Stop the updates after a fixed number of iterations or when your early stopping criterion is met.

We don't currently plan to release training or fine-tuning examples in this repo, but in general fine-tuning is done with:

Load the pre-trained model, usually with a prediction "head" for your fine-tuning task.
Initialize an optimizer, with an appropriate learning rate which you'll need to tune.
Repeat: feed the data through the model (and the head) and update your model to minimize your task-specific loss.
Stop the updates after a fixed number of iterations or when your early stopping criterion is met.

Load the pre-trained model, usually with a prediction "head" for your fine-tuning task.
Initialize an optimizer, with an appropriate learning rate which you'll need to tune.
Repeat: feed the data through the model (and the head) and update your model to minimize your task-specific loss.
Stop the updates after a fixed number of iterations or when your early stopping criterion is met.

May I ask some implement details about the finetune on ImageNet？
In the page 14, you write as below :
"Although adapting CLIP to the ImageNet distribution increases its ImageNet accuracy by 9.2% to 85.4% overall, and ties the accuracy of the 2018 SOTA from Mahajan et al. (2018), average accuracy under distribution shift slightly decreases"
In this version of model, the text encoder is still working or not working? It seems that you only use the clip features which extracted by the image encoder.
Thank you for your work.

We don't currently plan to release training or fine-tuning examples in this repo, but in general fine-tuning is done with:

Load the pre-trained model, usually with a prediction "head" for your fine-tuning task.
Initialize an optimizer, with an appropriate learning rate which you'll need to tune.
Repeat: feed the data through the model (and the head) and update your model to minimize your task-specific loss.
Stop the updates after a fixed number of iterations or when your early stopping criterion is met.

What do you mean by "with a prediction "head"?  As I understood, CLIP encodes images and texts with its backbone network and then the feature maps are compared by cosine similarity, and the distance between image-text pairs is tried to reduce by the back-propagation algorithm. So, CLIP does not make a classification, it aims to reduce the distance between ground-truth image-text pairs regardless of their class. So, is there a prediction head in the original CLIP, if not so why I should add to fine-tune it with my own data?


Make copying context easier (for pasting into an editor or shell).

A lot of the time I'm looking at the context of templates, the next step is top paste it into an editor for reasons[1] - especially including pasting into a python shell or notebook to explore.
At the moment it's inconvenient: You have to manually select the data - making it easy to select things that aren't the context data, and the data isn't exactly in a useable format ready to paste into a python list.
See this example - the first and second items are not comma separated and there is no start and end bracket.:

It would be nice to have a "Copy to clipboard" button, perhaps that floats in the top right of the context.
It would also be good if the text was in a format that was likely to work in a python shell, but maybe that is a slightly different issue (at the
[1] - scrolling up and down in the browser is less convenient with everything there, searching is more fiddly etc

This is a solid idea. I suspect we need to print the !r version rather than the string version. The copy logic should be manageable. I believe the Django error page does this for the stacktrace already.
@stuaxo do you have the availability to create a PR?

I can probably start with the !r bit, it's a small thing - useful.
Do you have pointers in DDT codebase I can look at?

This is the part of the code that renders that html: https://github.com/jazzband/django-debug-toolbar/blob/main/debug_toolbar/templates/debug_toolbar/panels/templates.html#L23
We'll want to add a button or something to it that allows copying the context here. Then hook into the click event handler in a template specific javascript file. You can see how history.js is defined and included, as well as how it handles button clicks: https://github.com/jazzband/django-debug-toolbar/blob/main/debug_toolbar/static/debug_toolbar/js/history.js

Something that's not entirely clear is what exactly are you hoping to copy and paste? The examples in the original issue are valid python dictionaries.
Another challenge is that some of the context values are not going to be python values (see assignments to temp_layer[key] https://github.com/jazzband/django-debug-toolbar/blob/main/debug_toolbar/panels/templates/panel.py#L143-L170)

Hello,
I'm new to open source contributions and would love to start contributing to this project. I've noticed that this issue is already assigned to @elijah0kello, but if there are any other beginner-friendly tasks or issues available, I would appreciate the opportunity to help out. As a newbie, I'm eager to learn and contribute, so any guidance or work you could assign to me would be great.
Thank you!

Hi @lav6301, I think it's fair for you to pick this issue up. As for other introductory issues we should have a beginner friendly label on them.

Hi @tim-schilling,
Thank you for assigning the issue to me! I'll start working on it right away. If I have any questions while working on this issue, I'll reach out for clarification. Thanks again for the opportunity!


L.geoJson.setZIndex doesn't seem to work

Trying to control layer order with L.geoJson but it seems that map.addLayer(layer) always adds the layer on top, if the layer was created with L.geoJson.
I've tried layer.setZIndex(foo) and this doesn't affect it, and also creating the layer with L.geoJson(data, {zIndex: foo}).

No wonder - only TileLayers implement the setZIndex method. Read the docs.
Unless we get more info, this will probably be closed as a duplicate of #2086 (always check for dupes before reporting a new bug!)

Thanks @IvanSanchez I did read that issue but not carefully enough to determine that it's the same problem...

How would I be able to control the zIndex via some other means (like CSS) ?

The only way to control zIndex in vector layers is bringToBack and bringToFront methods. This is a technical limitation — SVG lacks the concept of zIndex.

Thank you for the suggestion @mourner, I was able to do what I needed with bringToFront. Essentially sorting my layers by zIndex and calling bringToFront() on all of them, whenever a layer is added or removed.

I just stumbled over the same issue in 1.0rc1 and found it very confusing. If setZIndex has no effect in a geojson layer then it should throw an exception as it is clearly a programmer error. Of course, it would be nice to actually get setZIndex working since it is possible to sort layers with the bringToBack and bringToFront methods. Can't the geoJson (or other suitable) class not override setZIndex and implement such logic?

@neothemachine
I would go for a warning instead of exceptions. Since L.geojson inherits from L.LayerGroup, and L.layerGroup has setZIndex, it's not unreasonable for user to expect that calling setZIndex on any object that inherits from L.layerGroup will not throw an error.
As for sort layers, I would love to have that. It would be nice to have a way to disable that functionality (like setZIndex to null) so that maps with many geojson layers do not pay the cost of sorting them if the user don't care about their order.


Remove out of date doc comment

Was reading through docs and noticed that #547 appears to be resolved now, so the doc comment in abstractBatchSystem is wrong.
Commit 67c7cf7 appears to have fixed the underlying issue.

Can one of the admins verify this patch?

Continued in #848


Stablecoin update operation is not working

Describe the bug
I have an error INVALID_TREASURY_ACCOUNT_FOR_TOKEN when trying to update the newly created token.
To Reproduce

Go to https://stablecoinstudio.com
Create a new token.
Open "Stablecoin details" view.
Change token name.
Click "Save" button.

Expected behavior
Token name should be updated.
Desktop:

OS: macOS Sonoma 14.3.2
Browser: Chrome 124.0.6367.94

Additional context
Token: https://hashscan.io/testnet/token/0.0.4350783
Transaction: https://hashscan.io/testnet/transaction/1715604700.363997005

@dimasmoliar this looks like a hedera precompiled contract (0x167) issue, I have created an issue on their github repo : https://github.com/hashgraph/hedera-services/issues/13341.
I will keep you posted.


Meta 3154 - CLASSIFICATION_ONLY_PROPAGATION_DELETE task traversal optimization(Part 2)

Implemented the BFS parallelization

We release the first iteration of optimization in CLASSIFICATION_ONLY_PROPAGATION_DELETE now we need to work on optimizing it more. We will update rest of the work in this ticket

Type of change

[x] Enhancement

I believe META-3158 caused the conflict here with this PR

So in PR which was closed in staging Dillion asked why we are maintaining traversed vertices, so it is there and it doesn't have the entity start vertex. So we can just subtract it but found it causing some delay in computing that. So its better to have the traversed vertices there.


Automatically include ServiceCIDR as an argument to Windows nodes bootstrap

Tell us about your request
After Windows support is implemented, it will be ideal and preferable to automatically include ServiceCIDR as an argument to bootstrap file for Windows nodes. It is possible we can retrieve this value along with the DNSClusterIP as is part of the DescribeCluster request operation against the EKS API. The value will be set in serviceIpv4Cidr in the kubernetesNetworkConfig.
Tell us about the problem you're trying to solve. What are you trying to do, and why is it hard?
If the ServiceCIDR is not automatically retrieved or provided, the default value will be used as described in EKS docs Bootstrap script configuration parameters and will default to <IP_ADDRESS>/16 or <IP_ADDRESS>/16 based on the IP address of the primary interface.
In a case where a custom service CIDR was used, pods behind a Kubernetes service not have network connectivity as it would have been misconfigured with the default values. If this issue is not fixed, it would become a blocker for anyone wishing to adopt Karpenter as an autoscaler for Windows nodes in a case where they need a custom service CIDR.
Are you currently working around this issue?
The only workaround is to use the default IP address range for the service CIDR.
Additional Context
At the current time of opening the issue, support for Windows is still yet to be released but has been merged in PR https://github.com/aws/karpenter/pull/3698.
Attachments
No response
Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

I am interested in working on this issue and will take it up.

This is kind of a show-stopper if the VPC CIDR range is not  <IP_ADDRESS>/16 or <IP_ADDRESS>/16

We had the same issue, with Karpenter v0.30.0 which added windows userdata support this issue can be worked around.
Just add this one line of user data to AWSNodeTemplate, replacing <EKSClusterName> to your EKS cluster name.
  userData: |
    $global:EKSCluster = Get-EKSCluster -Name <EKSClusterName>

The user data will be prepended before the EKS bootstrap script which could make use of the $global:EKSCluster.
Of course a proper Karpenter level support for ServiceCIDR is still very nice to have.

@jzhn This is awesome! Would you be open to adding this into the upstream documentation for users to be aware of until we get the proper ServiceCIDR feature merged into the project?

@jonathan-innis : I'm happy to PR for this workaround, but I hesitate to do that because the usage of $global:EKSCluster is internal to EKS Windows AMI's bootstrap script. I doubt it's opensourced (please point me to the repo if I'm wrong), so it's hard for me to verify the compatibility of this workaround.
If you could provide some insights that this workaround is safe and won't cause confusion I'll make a PR ASAP. Thanks.

Going to aim to prioritize this and hopefully have a PR up soon given how this is a show-stopper for some cases.
I am afraid that @jzhn is right that the Windows bootstrap is not open sourced but the contents are publicly readable on the node if you run an instance with the EKS Windows Optimized AMI.
@jzhn is also right regarding the fact that the global variables are internal to the Powershell bootstrap so they will only be accessible within the script itself. If you set the global variable outside the script it is overwritten internally in the script so this workaround will not work. Support for ServiceCIDR will have to be at Karpenter level for ease of use. At the moment the only workaround is manually update the VPC CNI json config on the Windows node and set the ServiceCIDR which is not sustainable to apply to all worker nodes.

@jzhn Ah I see. When I referred to the value being overwritten I was referring to $global:ServiceCIDR which is overwritten when the bootstrap starts. I have looked at the code and what you have mentioned should work!
Please run a test locally and see if setting the cluster name results in the the ServiceCIDR value being correctly retrieved as we expect. I expect that it should work and if so, we can specify that as a workaround like @jonathan-innis suggested in the docs. I would give my thumbs up to for that workaround, and will also test a bit later myself to make sure it doesn't break anything else in the bootstrap flow.
To verify that the ServiceCIDR was set correctly, please check the file $env:ProgramData\Amazon\EKS\cni\config\vpc-bridge.conf on the node.

Yes, I've been running this workaround for my cluster for ~1 week with Karpenter v0.30.0 + Windows_Server-2019-English-Core-EKS_Optimized-1.27-2023.08.17. It's been working consistently that the right serviceCIDR is set on C:\ProgramData\Amazon\EKS\cni\config\vpc-bridge.conf and it resolved our weird windows pod networking issue.
Just for the ease of reproduce and test on your side, we used 10.x.0.0/16 for VPC range and <IP_ADDRESS>/16 for serviceCidr range (without overlaps). When cluster VPC is in 10.* range, EKS bootstrap script defaults to <IP_ADDRESS>/16 for serviceCidr range which caused networking pain.
Let me prepare a quick doc update PR then.

This change will be not be implemented within Karpenter, as discussed in closed PR #4625. The fix will be applied in the Windows bootstrap to extend it to accept an environment variable that can be set with custom userdata. Will share more information regarding the environment variable name and examples of how to use it at a later time within the next few weeks.

This issue can now be closed as the EKS Windows powershell bootstrap script now accepts an environment variable for Service CIDR which can be set with custom user data. This is available in the latest EKS Windows Optimizd AMIs, that is, version 1.28-2023.10.19 and later.
Example of setting the service CIDR:
apiVersion: karpenter.k8s.aws/v1alpha1
kind: AWSNodeTemplate
...
spec:
  userData: |
    [System.Environment]::SetEnvironmentVariable('SERVICE_IPV4_CIDR', '<IP_ADDRESS>/16', 'Machine')
...


⚠ Bofrost.at has degraded performance

In ba1f154, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1801 ms

Resolved: Bofrost.at performance has improved in 5eb2b20.


Incomplete Chinese Translation

The following strings are currently not translated in master:

View Shed (inc. all expanded options)
Filter (inc. all expanded options)
Euclideon Logo option
License at login option
Invert Mouse and Controller options
Convert AllSpaceLabel
Convert SpaceECEF
Poly Model Mesh
Poly Model Texture
Poly Model Material Colour
{polyModelInvertFaces}
{scenePOIAttachModel}
{scenePOIAttachModelURL}

Are you doing these @Hongzhi-liu?

Also these:

Oriented Image
{polyModelInvertFaces}
{scenePOIAttachModel}
{scenePOIAttachModelURL}

@CarlCFX, could you get someone to check the latest master and see if this can be closed?

@OsiJr Did you intend to merge #138 before making this comment?

Yeah, I clicked merge but must have missed the step where I tell it that I really want to merge

@OsiJr Looks like they are all in Client <IP_ADDRESS>10607

Closing this issue based on previous comment.


Fix warnings, static checks, add collect_rules unit tests

Rework metrics-allowlist schema based on collect groups
Fix warnings, static checks
Add collect_rules unit tests

/retest-required

/retest-required

/retest

/approve
/lgtm

/lgtm


Cannot find declaration of: remember { ... }

Workaround:
add to imports
import androidx.compose.runtime.getValue
import androidx.compose.runtime.setValue

Did you try to sync the project with Gradle?

Also can you provide archive with the project and tell what version of IDEA/Kotlin plugin are you using?

I've tried to:

sync as instructed
upgraded to latest compose (0.1.0-build119)
invalidate & restart

The project:
https://github.com/atoktoto/compose-experiment
Intellj version:
IntelliJ IDEA 2020.2.3 (Ultimate Edition)
Build #IU-202.7660.26, built on October 6, 2020
Subscription is active until December 7, 2020
Runtime version: 11.0.8+10-b944.34 amd64
VM: OpenJDK 64-Bit Server VM by JetBrains s.r.o.
Windows 10 10.0
GC: ParNew, ConcurrentMarkSweep
Memory: 1979M
Cores: 12
Registry: documentation.show.toolbar=true
Non-Bundled Plugins: org.antlr.intellij.plugin, org.jetbrains.plugins.hocon, org.jetbrains.plugins.vue, io.ktor.intellij.plugin, com.intellij.lang.jsgraphql

Thanks! Reproduced the issue

The same thing happens in Android studio and iirc it's intentional since it's an intrinsic.

The thing is, it makes it hard to understand what is going on without prior experience in Android dev. I finally found the documentation:
https://developer.android.com/reference/kotlin/androidx/compose/runtime/package-summary#remember
https://developer.android.com/jetpack/compose/state
It also has a kotlin implementation in source code:
https://cs.android.com/androidx/platform/frameworks/support/+/androidx-master-dev:compose/runtime/runtime/src/commonMain/kotlin/androidx/compose/runtime/Remember.kt?q=remember&ss=androidx%2Fplatform%2Fframeworks%2Fsupport:compose%2Fruntime%2Fruntime%2F

I have the same error. Some more information:

and

Treated with replacement
import androidx.compose.runtime.remember
to
import androidx.compose.runtime.*
IDEA version:
IntelliJ IDEA 2020.3 RC (Community Edition)
Build #IC-203.5981.114, built on November 26, 2020
Runtime version: 11.0.9+11-b1145.21 amd64
VM: OpenJDK 64-Bit Server VM by JetBrains s.r.o.
Linux 5.8.18-1-manjaro
GC: ParNew, ConcurrentMarkSweep
Memory: 977M
Cores: 2
Non-Bundled Plugins: com.atlassian.bitbucket.references, no.tornado.tornadofx.idea
Current Desktop: KDE

(and earlier versions)
Compose v. 0.2.0-build131 and earlier
Construction val a = remember { .... } works fine...

Error "has no method" happened because there are no these imports in the file:
import androidx.compose.runtime.getValue
import androidx.compose.runtime.setValue

getValue/setValue are kotlin delegates needed for "var test by" expression.
IDEA doesn't offer to import them if we wrap mutableStateOf in remember, so will have to write them manually.

https://youtrack.jetbrains.com/issue/KT-39199

New Kotlin ticket - https://youtrack.jetbrains.com/issue/KTIJ-20402

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


Add --no-partialeq  flag

Related to #965.

[x] Add a new RegexSet member to bindgen::Builder (similar to the whitelisted_types set).

[x] A Builder method to add strings to that RegexSet.

[x] Plumbing in src/options.rs to convert --no-partialeq  CLI flags into invocations of the builder method.

[x] Make the MonotoneFramework::constrain function in src/ir/analysis/derive_partialeq.rs check if the given item is explicitly marked not to be Partialeq, and if so, insert it into the self.cannot_derive_partialeq set via return self.insert(id).

[] Tests!

[] When the no-partialeq type is transitively referenced by a whitelisted item

[] When the no-partialeq type is explicitly whitelisted

[] When the no-partialeq type is marked opaque

This is my first pass at implementing this functionality, I haven't implemented the tests yet. I wanted to make sure I am on the right track, particularly when it comes to updating MonotoneFramework::constrain.
r? @fitzgen

:umbrella: The latest upstream changes (presumably #1000) made this pull request unmergeable. Please resolve the merge conflicts.

r? @fitzgen
I've added the required tests, but I am not quite sure I did it right. I use --with-derive-partialeq for all three tests and then add the additional flags per your description. When I explicitly whitelist or make opaque a particular type PartialEq still gets derived.

It makes sense that opaque types still derive(PartialEq) because the no_partialeq_by_name check comes after the opaque check in the constrain function. We should move the no_partialeq_by_name above the opaque check.
The explicitly whitelisted types always getting derive(PartialEq) even when we request they shouldn't sounds more problematic. Let me look at the code a little more...

Hm yeah the test headers look how I'd expect them to, but the generated bindings don't. I'd try adding some debug println!s in the CanDerivePartialEq implementations and lookup_item_id_can_derive_partialeq methods. Then, I'd run with extra logging:
$ RUST_LOG=bindgen::ir::analysis RUST_LOG_LEVEL=trace ./tests/test-one.sh <one of your new tests>

If you still can't figure out what's going on after that, I can try pulling these changes down myself and investigating.

BTW, I'm not sure if you're aware that the "impl period" has just begun, but the folks hacking on bindgen during the impl period are hanging out in https://gitter.im/rust-impl-period/WG-dev-tools-bindgen if you want to join us :)

:umbrella: The latest upstream changes (presumably #1002) made this pull request unmergeable. Please resolve the merge conflicts.

We should move the no_partialeq_by_name above the opaque check.
I've tried it just now, but the result is still the same. Makes me wonder if no_partialeq_by_name is doing the right thing at all. I'll try to figure it out.

Thanks for the invite! :)

@bors-servo r+
Thanks @alexeyzab !

:pushpin: Commit ec8456b has been approved by fitzgen

:hourglass: Testing commit ec8456bd74026bf5a0f1ff530b8fa36d893c2933 with merge 48429734f10165d53584486af2d584ebde19577c...

:sunny: Test successful - status-travis
Approved by: fitzgen
Pushing 48429734f10165d53584486af2d584ebde19577c to master...


Strange Bug ! Apache restart

What steps will reproduce the problem?
step 1 : create a class in any folder of yii framework.
step 2 : structure of class code must be like under :
namespace app\components; #or any thing
class Bug{
	public function load(){
		foreach($examples as $e){
			call_user_func($a,&$b);
		}
	}
}

call class 'Bug' in any place:
<?php
namespace app\controllers;

use Yii;
use yii\web\Controller;

class SiteController extends Controller
{
    public function actionIndex()
    {
        new \app\components\Bug;
    }
}
?>

What is the expected result?
show error .
What do you get instead?
Apache restart !!!
Additional info
apache error log :
[Wed Apr 27 19:53:36.208403 2016] [mpm_winnt:notice] [pid 9064:tid 400] AH00428: Parent: child process 6816 exited with status<PHONE_NUMBER> -- Restarting. [Wed Apr 27 19:53:36.813088 2016] [ssl:warn] [pid 9064:tid 400] AH01909: www.example.com:443:0 server certificate does NOT include an ID which matches the server name [Wed Apr 27 19:53:36.843255 2016] [mpm_winnt:notice] [pid 9064:tid 400] AH00455: Apache/2.4.10 (Win32) OpenSSL/1.0.1i PHP/5.6.3 configured -- resuming normal operations [Wed Apr 27 19:53:36.843255 2016] [mpm_winnt:notice] [pid 9064:tid 400] AH00456: Apache Lounge VC11 Server built: Jul 17 2014 11:50:08 [Wed Apr 27 19:53:36.843255 2016] [core:notice] [pid 9064:tid 400] AH00094: Command line: 'c:\\xampp\\apache\\bin\\httpd.exe -d C:/xampp/apache' [Wed Apr 27 19:53:36.845257 2016] [mpm_winnt:notice] [pid 9064:tid 400] AH00418: Parent: Created child process 8772 [Wed Apr 27 19:53:37.539436 2016] [ssl:warn] [pid 8772:tid 380] AH01909: www.example.com:443:0 server certificate does NOT include an ID which matches the server name [Wed Apr 27 19:53:37.885145 2016] [ssl:warn] [pid 8772:tid 380] AH01909: www.example.com:443:0 server certificate does NOT include an ID which matches the server name [Wed Apr 27 19:53:37.917167 2016] [mpm_winnt:notice] [pid 8772:tid 380] AH00354: Child: Starting 150 worker threads. [Wed Apr 27 19:53:39.539831 2016] [mpm_winnt:notice] [pid 9064:tid 400] AH00428: Parent: child process 8772 exited with status<PHONE_NUMBER> -- Restarting.
php error log :
[27-Apr-2016 19:53:37 Asia/Tehran] PHP Fatal error:  Call-time pass-by-reference has been removed in C:\xampp\htdocs\ziaratban\components\Bug.php on line 7

Q
A

Yii version
2 Basic

PHP version
5.6.3

Operating system
Windows 8.1

looks like a problem with your php setup, this is not caused by yii.

also as the php messages says, call time pass by reference does not work, so do not use it.

thanks for reply.

looks like a problem with your php setup, this is not caused by yii.

i test this class in a single php file and see php error but in yii framework the above problem(apache restart) has occurred
why this error in yii is different ? but in single php file is ok !
maybe a program like antivirus cause this error.


Adding ControllerName struct, and test cases

Description
Please include a summary of the changes and the related issue. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes # (issue)
Feature # (details)
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration. Is it a breaking change which will impact consuming tool(s)?

[ ] Test A
[ ] Test B

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules

unit tests are failing right now

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
3.8%

Covered Lines:
1857

Relevant Lines:
2068

💛  - Coveralls

/ok-to-test sha=83bd476

/ok-to-test sha=0e5659b


badtouch: init at 0.7.3

Motivation for this change
Scriptable network authentication cracker
https://github.com/kpcyrd/badtouch
Related to #81418
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

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

1 package built:

badtouch


test_sync_async_add: After closing the default event loop, set a new one

TestAsyncCase.test_sync_async_add leaves the default loop closed. If TestAsyncCaseWithCustomLoop.test_await_async_add runs right after it, it will fail.
As far as I can see from the other test methods, the default loop should be reset if it's closed.
Usually, this issue is masked by tests that run in between these two and re-set the default event loop as a side effect. It can be reproduced with pytest -k 'test_await_async_add or test_sync_async_add', or on Python 3.11 with #21 merged (cc @hroncok).

Thanks.


Right click copy doesn't work

Description of the problem or steps to reproduce
Specifications
Version: 1.2.0
Commit hash: be81241
Compiled on May 27, 2017
Terminal: Mate Terminal 1.18.0
Right click copy doesn't work, when selecting something and wanting to copy the right mouse button doesn't work and is grayed out.

Is right mouse button to copy something enabled by your terminal? Micro has no bindings for the right mouse button. I'm also not sure how the right mouse button can be "grayed out."

@zyedidia I don't believe he's trying to just right click to copy, but is rather referring to the right click menu. Most terminals include a right click menu which include a copy function.
On Xfce4-terminal, the right click menu does not open when running Micro. I guess it detects that Micro is capturing mouse clicks, and decides to drop its own right click menu functionality.
I've just installed Mate-terminal to test @themotu 's issue, and have found that the right click menu does open, with the copy option greyed out.
My recommendation @themotu would be to use use Ctrl + C to copy, and Ctrl + V to paste. I don't know much about how terminals work, so I don't really know how big of a change it would take to make the right click -> copy work, but my uneducated guess would be that it's not feasible. Perhaps @zyedidia will correct me on this.

Unfortunately I don't think this is possible to fix. I'm guessing it only works on Nano because Nano doesn't use mouse support (please correct me if I'm wrong).
The core of this issue has to do with the question of who gets mouse events. If micro gets the mouse events, then the terminal can't copy/open a menu on right click. If the terminal gets the mouse events then micro can't respond to the mouse.

This basically means I can't copy and paste across terminals

Yeah, it keeps me from using micro on my servers.

Maybe I'll look into implementing a way of disabling and enabling mouse support so that you could temporarily disable mouse support to paste using right click and then re-enable it.

@zyedidia As far as I know, nano has some basic mouse support...

Yes but I don't believe that you can paste with right click while nano's mouse support is enabled (when on a server over ssh). Please correct me if I'm wrong.

I use ctrl+shift+V and middle click all the time in nano over ssh. Just want the same functionality in micro. Copy and paste is too unreliable as is.

@themotu do you have xclip installed? If you have xclip installed, Micro's Ctrl+C and Ctrl+X should copy to your system's clipboard, so you can paste in another terminal with Ctrl+Shift+V. And the other way around as well, you'll be able to copy in other programs and paste into micro as expected.

Yes, to expand on what @DanielPower said:
since micro runs inside the terminal, each program (the terminal emulator and micro) has its own handling of copy and paste. When you press Ctrl-V, the terminal sends this key event to micro and micro then interfaces with the system clipboard (if you have xclip or xsel installed -- or a local clipboard otherwise) and pastes. If you press Ctrl-Shift-V (or whatever the keybinding is to paste using through the terminal emulator) then the terminal emulator accesses the system clipboard and sends the string of characters to micro as a "paste event." The same works for copy and paste but since a selection is required to copy, mouse support must be disabled in micro (atm this is not possible) to use the terminal emulator's copy so that a selection can be made using the terminal emulator's method of selection instead of selecting through micro (in that case, line numbers might become selected because the terminal emulator only knows about the characters being displayed).
Generally it doesn't matter if you use the terminal's copy/paste or micro's copy/paste.
However, when using micro over ssh, it is important. Since micro runs on the remote server, when copying and pasting it accesses the server's clipboard (in fact, I believe most servers do not even have system clipboards), so it is not possible to copy from your local computer and paste using Ctrl-V (micro's paste). However, if you use the terminal emulator's paste (Ctrl-Shift-V), since it is running on your local computer, it will access your system clipboard and send the paste event across ssh to micro on the server and everything will work.
I think the reason you see nano working over ssh is because nano's mouse support is disabled (even if mouse support is possible I think it is disabled by default), so mouse events like middle click aren't being captured by the editor. I think adding an option to disable mouse support in micro would fix these issues, provided people understand what is going on and when to have mouse support enabled or disabled.

Well, just typed up a super long-winded response and my browser crashed. So, this is a recap of what I wrote.
Sorry for necro'ing such an old issue, but this is the top result on the issue tracker and Google, so wanted to provide a solution.
So basically, browsers have something called "Mouse tracking". Most let you override this by using a modifier, which by default is SHIFT. Pressing SHIFT+Right Click should then perform the action you have for your terminal, whether it be copying (Alacritty, wezterm, etc) or opening a context menu (Konsole, xfce4-terminal, etc).
I've tested this with a bunch of terminal editors (Konsole, xfce4-terminal, wezterm) on Linux, and it works when pressing SHIFT+Right Click.
In terms of an actual solution, it would be neat if Micro had a config option to add "ignored keys" such as right click, middle click, etc. I'm looking into implementing it, so might make another issue about that.
tcell (the library micro uses) has a few issues following up on this:

There was an issue about mouse wheel, which is very similar
gdmaore (maintainer of tcell) looked into it and implemented it - https://github.com/gdamore/tcell/pull/427

I think that should solve this problem?


[Bug] FileNotFoundError: [Errno 2] No such file or directory:

1. Summary
I want to create my journal.txt file in D:\Киролайна\jrnl path. But I get stack trace.
2. Expected behavior
Like path name contains only Latin letters.
[Journal created at D:\Киролайна\jrnl/journal.txt]
[Entry added to default journal]

3. Actual behavior
D:\sublime-jrnl>jrnl yesterday: Kristina unthinkable!
[Journal created at D:\РљРёСЂРѕР»Р°Р№РЅР°\jrnl/journal.txt]
Traceback (most recent call last):
  File "c:\python36\lib\runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "c:\python36\lib\runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "C:\Python36\Scripts\jrnl.EXE\__main__.py", line 9, in <module>
  File "c:\python36\lib\site-packages\jrnl\cli.py", line 181, in run
    touch_journal(config['journal'])
  File "c:\python36\lib\site-packages\jrnl\cli.py", line 98, in touch_journal
    open(filename, 'a').close()
FileNotFoundError: [Errno 2] No such file or directory: 'D:\\РљРёСЂРѕР»Р°Р№РЅР°\\jrnl/journal.txt'

4. Settings
Part of my .jrnl_config file:
"journals": {
  "default": "D:\\Киролайна\\jrnl/journal.txt"
},

5. Steps to reproduce
I create folder D:\Киролайна\jrnl → I install jrnl → I set .jrnl_config file → if my path contain Cyrillic symbols, I get stack trace.
If I run command chcp 1251 in console → I get same behavior.
6. Environment
Operating system and version:
Windows 10 Enterprise LTSB 64-bit EN
Python:
3.6.0
jrnl:
version 1.9.8
Thanks.

I can duplicate the problem, but I'm not sure how to solve it

@Kristinita  that last forward slash in D:\Киролайна\jrnl/journal.txt is suspicious. Wouldn't replacing it with a backslash help? I'm not a Windows user, so I cannot reproduce the issue, but I doubt Windows likes mixing slashes.

@radomirbosak , if I try "default": "D:\\Киролайна\\jrnl\\journal.txt", I get same behavior.
Thanks.

The problem not solved.
Thanks.

@Kristinita Can you try this on v2.0? I don't see the error running the v2.0, but I'm not sure if it's a Windows-specific problem.

This appears fixed, but it would be good to have a test for it after we've set up a proper Windows testing pipeline.

We recently brought the Windows pipeline into full parity with the other pipelines, so it would be good time to add this test.


Url-key for product should be auto-generated

BUGS:
Url-key for product should be auto-generated #1189

Changes done.
https://prnt.sc/pl3tby

This cannot be merged as this thing need to solved from server side.


Upgrade vendored pq library to support sslrootcert option

I was not able to get the sslrootcert postgres connection option (in the drone docs here) to work -- it seems to accept any value and have no effect. Digging deeper, it appears that the vendored version of pq that drone currently uses doesn't appear to support the sslrootcert option: there's no mention of it in the drone repo except for in documentation. Looks like this is supported in more recent versions of pq.

fixed by pull request #1264

thanks for letting me know


Update devenv-onvalidate-page-fields-trigger.md

Docs says: "You cannot call the UPDATE from this trigger.". This sentence is not true. In Base App you can find multiple implementations of CurrPage.Update called inside OnValidate trigger. For example. In Page 51 "Purchase Invoice", in field "Buy-from Vendor No.", in OnValidate trigger, CurrPage.Update is called.

Thanks for your contribution!


adds views/urls for check uninstall

Plan was to move this code from cmscloud api, figured might as well clean it up a bit.

LGTM, but we should probably protect the view somehow


[BEAM-310] Actually Split Root Transforms

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[ ] Make sure the PR title is formatted like:
[BEAM-<Jira issue #>] Description of pull request
[ ] Make sure tests pass via mvn clean verify. (Even better, enable
Travis-CI on your fork and ensure the whole test matrix passes).
[ ] Replace <Jira issue #> in the title with the actual Jira issue
number, if there is one.
[ ] If this contribution is large, please file an Apache
Individual Contributor License Agreement.

Permit the ExecutorServiceParallelExecutor to control its own
ExecutorService by passing only a TargetParallelism parameter. Split
roots into the greater of 3 or the target parallelism.

R: @dhalperi @jkff

Regression test?

LGTM otherwise

Added a test to ensure that split is called with smaller than the estimated size (which only occurs if targetParallelism is greater than 1, which we now enforce)

(Existing LGTM stands, Please self-merge once green)


updated credentials setup to match SOP updates

credential setup readme is updated to match review of OPS-SOP changes

This repo is public and should be general to anyone wanting to use this code base. The linked guide is private, and specific to Red Hat SREP setup. Anyone with a valid AWS credential set can use this repo, so I think keeping the instructions non-specific here is important.

agreed. closing


Core API: Report proper HTTPRoute and Gateway Condition when GatewayClass is invalid or doesn't exist

Background
There is one to one mapping between NKG and GatewayClass resource.
The GatewayClass might be invalid (ex, its controllerName doesn't match NKG controller name). The GatewayClass might not even exist.
For those cases NGK doesn't generate any NGINX configuration and as a result, NGINX will not accept any traffic.
However, NKG still processes Gateways and HTTPRoutes.
Requirements
When GatewayClass is invalid or it is doesn't exist, NGK must report Accepted (false) and appropriate reasons in the statuses of HTTPRoute resources and Gateway resources.
Figure out an appropriate Reason. Check the Gateway API first. If necessary, introduce NKG-specific reasons.
A/C

implement  the requirements
update the doc https://github.com/nginxinc/nginx-kubernetes-gateway/blob/main/docs/gateway-api-compatibility.md#httproute

Aha! Link: https://nginx.aha.io/features/NKG-17

Discussed with @kate-osborn implementation approach:

if the GatewayClass doesn't exist or invalid, NKG will invalidate any listeners of the Gateway resource and report the error about the GatewayClass in the Gateway status.
As a result of invalid listeners, HTTPRoutes will not be able to attach to listeners. NKG will report already implemented invalid listener condition in the HTTPRoute statuses.


sidebar-menu does not work in mobile

Hi @almasaeed2010
I has change the default status in sidebar-menu setting to close

But when I use mobile view in first time, the li it can't open or close,
And I try to use AdminLTE v2.4.
But it will broke... like this...

having this issue too!


Relative Image Paths do not work for docx or epub

In converting a markdown file to docx or epub, I run into PandocResourceNotFound for images with a relative path.
I can duplicate this issue with pandoc itself, by running pandoc from a directory other than the one where the input markdown file is.
With pandoc itself, I can solve the problem, by specifying a nondefault extension. The from file type changes from markdown to markdown+rebase_relative_paths. Here rebase_relative_paths is the nondefault extension.
But there seems to be no way to specify any pandoc extension in this way with pypandoc. The code that normalizes the from file types does not seem to allow it.
In any case, I haven't found a workaround for this issue. Does someone else have a workaround?
Thanks.

Hi there.
When setting the input or output format with pypandoc, you can also set extensions like you did in your example


Android back button is blocked by AnimatedCrossFade

If I remove AnimatedCrossFade ,it becomes ok.

cc @Hixie

You will need to provide some sample code for us to see what's going on. :-)
Ideally, a small main.dart that, when used to run an app, demonstrates the issue.

@realcarlos any chance you can provide some sample code so we can investigate?


tortoise: add voting targets from encoded votes

closes: https://github.com/spacemeshos/go-spacemesh/issues/3608

there is a new data boolean on blockInfo in tortoise state. it is set to true on the direct call to OnBlock to tortoise. tortoise will always vote against the block, if the data is not available locally.
tortoise.Results provides api to query consensus state from a given layer, and returns results in-between (from, verified]. verified is maintained internally. if Data is set to false, with Valid - true, caller is expected to try to fetch data from peers, and then call tortoise.OnBlock
voting targets are also added directly from votes in the ballot. in normal "online" case voting target will be always added together with data with a call to tortoise.OnBlock

bors merge


puppeteer: remove test_not_delete_draft_on_sending().

This test was asserting the draft is not getting cleared
as soon as sending, which is desired behaviour to some extent
because we don't want to delete it until we receive a confirmation
from the server, but given the right amount of delay (or very
low network latency), the draft will be cleared. This inconsistency
was causing failures.
We could better test this with a node test. A node test was
already added in #18827.

Testing plan:
GIFs or screenshots:

Merged, thanks @chdinesh1089!


post analysis not completing

Hi!
I have seen that the time to first binding analysis and dwell time analysis often gets struck for hours and does not proceed to completion. When it works, it takes about 10-15 minutes and would continously show the output lines.  I am not sure if other uses have also experieneced this and I was wondering if there is any bug in the post analysis? Let me know if I can provide any other information to help resolve this.

Hi @nidaf123 ! My suspicion is that there might be a lot of transition between bound and unbound states which slows the analysis. Can you check your p(specific) traces if you indeed have too many such transitions (this might be due to noisy data or under-fitting). Also can you try setting number of posterior samples to 1 (instead of default 2000) and see if that finishes analysis faster?

Hi! Thanks for the quick reply. I have attached the probabilistic rastergrams for channel 0 and 1. A lot of binding events look stable, signal to noise is around 2.5, but let me know if you think that the data might be causing this.

I tried posterior sample = 1. It finished the iterations quickly, but gave an error about tensor dimensions (attached)

I also tried running posterior sample = 500, time to first binding finished, but dwell time got struck or gave an error (attached

)

@nidaf123 can you try out the new 1.1.17 version of Tapqir? You can upgrade by running pip install -U tapqir
It fixed several things that were slow, in particular, made the default num_samples=500 for dwelltime analysis because of the large memory consumption of this command. It should be faster now, but still not super fast. If you see CUDA out of memory error messages try setting smaller num_samples.
This new version has also fixed the error about tensor dimensions.

Hi. I updated tapqir and ran 500 posterior samples for time to first binding and 250 samples for dwell time. For these number of samples, the analysis did not get struck and finished quickly. I will let you know if I encounter very slow/incomplete analysis in future. Thanks!


Migrate to handleResponse over ajaxSuccess

WIP
re #62
Fastboot's initializer/ajax still uses ajaxSuccess and ajaxError.  This begins the process of moving to handleResponse.

I think this is superseded by https://github.com/tildeio/ember-cli-fastboot/pull/95. Thanks for the PR, let me know if I'm misreading. Thanks!


demo video

hey I don't have WoW installed but i'm curious what this looks like!

Hi! At present progress is a bit split on two levels. There's console.log output on all authentication and game packets over WebSockets, such as when a monster moves in your vicinity.
None  of these game packets are currently processed so that any entities appear in the game world. But statically there's been some progress on rendering the game world:

I think that we should add some images to readme. +cool factor

@petersandor Agreed! Have added a preliminary screenshot :smile:

@maxogden It took a while, but there's a demo video in the README now!

excellent :D


Only check configuration for the current config.

We should only verify the configuration for the current selected config; otherwise, we can't use the config file if one config in the config file contains an error (such as a path to a Contextionary that is only referenced in that one config).

Is this still relevant @moretea ?

It can still cause trouble. I'd say that #450 would solve this.


fix: validation error of ngsipayloadattr

closes #350

@SystemsPurge I will add a test case for that

It is done now


[RunIQ] GPS messes up every longer run

Examples

https://www.strava.com/activities/1537109326 no way a lap of 1km goes up to 2'10/km.

Decided to part ways with RunIQ https://github.com/130s/hut_10sqft/issues/157#issuecomment-431757896.


Add autoheight to power grid

For a number of projects I am involved in the transition from smart- to NG client.
As a replacement for the datasetgrid (servoy-plugins.de), the power grid is used.
The datasetgrid however could handle text wrapping where the rowheight was expanding with the number of text lines.
The AG grid seems to have this possibility by using 'autoheight', but this is currently not possible in the power grid.
Could this be added as an option?

Hi Mark,
You can achieve this in the current version by adding a gridOptions property called domLayout and setting it to autoHeight. You should then set the responsiveHeight property to 0. See screenshot.

I currently use this in my solution and it works well

Sorry Mark - misread this but I believe you can do exactly the same thing with the columnDef properties:

In the interests of full disclosure though - I've not used or tested this!!


Fixed html compliance

Fixed html compliance i/em tags

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Hi @TimYagan, these <i> tags are not being used to italicise text, but to denote an icon: https://google.github.io/material-design-icons/#using-the-icons-in-html
Thanks for the contribution, but I don't think it makes sense to change these tags in this case.

No problem at all. Please ignore the pull request.
On Tue, 1 Oct 2019, 13:23 n1474335<EMAIL_ADDRESS>wrote:

Hi @TimYagan https://github.com/TimYagan, these  tags are not being
used to italicise text, but to denote an icon:
https://google.github.io/material-design-icons/#using-the-icons-in-html
Thanks for the contribution, but I don't think it makes sense to change
these tags in this case.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/gchq/CyberChef/pull/651?email_source=notifications&email_token=AHMKZKAIT2B5GYP6MV3WTL3QMMXMJA5CNFSM4I3SHMXKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEAA5QOI#issuecomment-536991801,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHMKZKFFMWKKOP6YN3X7UBLQMMXMJANCNFSM4I3SHMXA
.


fixing the CI in #1142

it fixes #1142
putting extract_version_from_current_wd after sourcing utils.sh for it to be recognised.

As a general rule, pls fill up Milestone, Labels and Assignee fields in the PRs you open 🕵🏻

Whoops this has been merged to master... that would trigger another release v3.0.2 😅 I cancelled that master build, let's get v3.0.1 accepted by JetBrains first (approval still pending). Let's generally open PRs to develop.

Whoops this has been merged to master... that would trigger another release v3.0.2 😅 I cancelled that master build, let's get v3.0.1 accepted by JetBrains first (approval still pending). Let's generally open PRs to develop.

😳 sorry


jhudsl-robot is needed for syncs

Summary
AH. So there is one place that jhudsl-robot is needed. It's for syncs. See #773
So #728 didn't consider this part.
We'll need to add it back to the documentation and issue template too.
Feedback
I haven't tested this on anything. We need something that is external and that jhudsl-robot is given permissions to. I'll probably attempt to send to OCS or something.

OCS might already have the robot set up though- I vaguely remember doing that

yeah it is: https://github.com/orgs/opencasestudies/outside-collaborators I can remove access if that is helpful


Encrypted PEM keys

New API Proposal: Support for encrypted PEM private keys
Motivation:
Currently, the library can read and handle unencrypted PEM-encoded private keys, but it can't directly load encrypted PEM keys. This can be a limitation in scenarios where users need to store their private keys in an insecure place or when working with third-party APIs that give the users encrypted PEM keys (e.g., PassKit/Apple Wallet).
Also, there is a bit of inconsistency within the ecosystem, as NIOSSL supports them, but only for TLS/SSL related stuff.
Importance:
The best solution I know of at the moment is to use an openssl executable (where available) inside a blocking Process. That's how we do it currently in the vapor-community/PassKit library.
cc @0xTim

Thanks for filing this!
One of the wrinkles of supporting encrypted PEM keys is that those encryption schemes often rely on very weak cryptography that we don't otherwise want to support in Swift Crypto. A similar issue occurs with PKCS#12 support.
I think a pragmatic compromise might be to add interfaces in CryptoExtras that directly call into the underlying BoringSSL functions to perform the decryption, rather than attempt to add fully-fledged support for those cryptosystems to Crypto itself.
See-also #263.

What would be the best approach to add this functionality? Maybe create some internal helper methods that decrypt the key, and then add to each individual CryptoExtras algorithm another private key initializer that first decrypts it and then passes it to the traditional initializer?

That approach might work, but it'll require us to reverse-engineer the encrypted PEM format. Probably a suitable strategy though if you wanted to tackle it!

Got it! What other strategies can we consider?

I think the other strategy is to call the higher-level BoringSSL function to parse the encrypted key format, which will return a structured BoringSSL pointer. That can then be either passed directly to a package-level initializer (on Linux) or deserialized and reserialized (on Apple platforms).
I think that's a bit less appealing, even though it's probably easier. Partly it's less appealing because it means there are multiple separate key format parsers, which is a bit rough.


Get-TPPIdentity -IncludeAssociated

When invoking this method with the -IncludeAssociated parameter the following error is issued:
Get-TPPIdentity : Cannot bind parameter because parameter 'UriLeaf' is specified more than
once. To provide multiple values to parameters that can accept multiple values, use the
array syntax. For example, "-parameter value1,value2,value3".
Sample code:
Import-Module -Name VenafiPS
Add-Type @"
using System.Net;using System.Security.Cryptography.X509Certificates;public class TrustAllCertsPolicy : ICertificatePolicy {    public bool CheckValidationResult(    ServicePoint srvPoint, X509Certificate certificate,    WebRequest request, int certificateProblem) {        return true;    }}
"@
[System.Net.ServicePointManager]::CertificatePolicy = New-Object TrustAllCertsPolicy
[System.Net.ServicePointManager]::SecurityProtocol = [Net.SecurityProtocolType]::Tls12
$ErrorActionPreference = "Stop"
$password = ConvertTo-SecureString "xxxxxxx" -AsPlainText -Force
$Cred = New-Object System.Management.Automation.PSCredential("administrator@lab.securafi.net",$password)
New-VenafiSession -Server localhost  -Credential   $Cred -ClientId "WorkflowApp" -Scope @{"Configuration"="manage"}
$Script:VenafiSession
$a = Get-TPPIdentity -ID "AD+ad1:c589dea40536394ea238d73d3919ac04" -IncludeAssociated
$a | Format-List

Please try and use the bug report template instead of free form.  Can you please provide the below?
Operating System:
VenafiPS version:
PowerShell version:

Operating System: Windows 2019
VenafiPS version: 4.1.2
PowerShell version: 5.1.17763.2268


Sankey extension added outside of Repo

Timo,
I extended FSharp.Plotly to draw Sankey diagrams.
http://fssnip.net/7WI/title/Sankey-Diagram-extension-to-FSharpPlotly
It's a minimal extension, missing many features, but I tried to keep the structure as close as possible to the style you have.
Maybe if you time you can integrate it properly into the code base.
Regards,
Faisal

hi @fwaris, I am a little late here but thanks for sharing! I will definitely incorporate your code while i am slowly implementing the missing charts. You can do a PR with the stuff you have so far on the developer branch if you want to. That way you will not only be credited by a comment in the code/release notes 😉


[SB-1020] Fix test typo

The state query test has a typo. Fix it.

I didn't run the test. I just fixed the error reported by VS Code.


F15/14 binding not work

<?xml version="1.0"?>
<root>
  <item>
    <name>F15/14 to Page Up/Down</name>
    <identifier>private.f_15_14_to_page_up_down</identifier>
    <autogen>__KeyToKey__ KeyCode::F15, KeyCode::HOME</autogen>
    <autogen>__KeyToKey__ KeyCode::F14, KeyCode::END</autogen>
  </item>

  <item>
    <name>shift+opt+L/R to HOME/END</name>
    <identifier>private.shift_opt_L_R_to_HOME_END</identifier>
    <autogen>__KeyToKey__ KeyCode::CURSOR_LEFT, ModifierFlag::SHIFT_L | ModifierFlag::OPTION_L, KeyCode::HOME</autogen>
    <autogen>__KeyToKey__ KeyCode::CURSOR_RIGHT, ModifierFlag::SHIFT_L | ModifierFlag::OPTION_L, KeyCode::END</autogen>
  </item>
</root>

Screen shot of EventViewer

This setting works fine on my machine.
Did you enable the new setting?

Yes, I've enabled the setting. Probably the problem exists with my keyboard (logitech K480), where the F15/14 is actually a combination of Fn+UpArrow/DownArrow. The Fn key alone is unrecognizable.

OK. Please show me the raw input event of F15/F14 in debug mode.
https://pqrs.org/osx/karabiner/document.html.en#debugmode

It is very strange that nothing captured by the debug mode when I press Fn+UpArrow/DownArrow (but the EventViewer captured the signal of F15/F14).

I guess other keyboard driver is installed and it conflicts with Karabiner.
Did you install Logitech Control Center or other keyboard drivers?

Yes, you are correct. I've installed Logitech options. Let me uninstall it
and test out Karabiner then. Thanks.
On Wed, Jan 6, 2016 at 10:30 AM, Takayama Fumihiko<EMAIL_ADDRESS>
wrote:

I guess other keyboard driver is installed and it conflicts with Karabiner.
Did you install Logitech Control Center or other keyboard drivers?
—
Reply to this email directly or view it on GitHub
https://github.com/tekezo/Karabiner/issues/547#issuecomment-169201930.

--
Sincerely,
Dinghua (a.k.a vout)
email<EMAIL_ADDRESS>

Without logitech options, Fn+UpArrow/DownArrow become opt_L+UpArrow/DownArrow instead of F15/F14. It seems that there are no perfect way to map Fn+UpArrow/DownArrow to PAGEUP/DOWN with K480. I am closing this issue.

could you leave the logitech thing out, and just map the OPTION_L+arrows to PgUp and PgDown? just an idea...

K480 has a mode switch in keyboard.
You can use fn-arrow keys by using PC mode.

@ethus3h thx but I have to use the driver for my logitech mouse. I am now using OPTION_L+arrows to replace Fn+arrows for PgUp and PgDown.
@tekezo By using PC mode the Fn+Up/Down arrow still map to F15/F14.

@voutcn Thank you for confirmation.
The fn+up/down will be PageUp/Down with PC mode on my Logitech K480, but I understand it is model specific behavior.


[$500] Login - Auto-fill appears for a brief moment in Abracadabra/Here's your magic code screen

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.26-1
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Action Performed:

Navigate to staging.new.expensify.com
Start the sign-in flow by entering an email
Go to the email and locate the magic code link > copy the link
Open a new tab and navigate to the staging link
Verify the "Abracadabra, you're logged in."

Expected Result:
Auto-fill shouldn't appear on the Abracadabra/Here's your magic code screen
Actual Result:
Auto-fill appears for a brief moment in Abracadabra/Here's your magic code screen
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[x] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/14ba011b-1c39-44b3-a70f-b5379368aeaa
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~013401609f515ba854
Upwork Job ID:<PHONE_NUMBER>960612864
Last Price Increase: 2024-01-18

#wave9-collect-signup

Dupe of #32683

cc: @MitchExpensify Seems dupe of https://github.com/Expensify/App/issues/32683 please close this one.


Cannot login to Rancher

Rancher Server Setup

Rancher version: v2.5.9 -> v2.6.0
Installation option (Docker install/Helm Chart): Helm

If Helm Chart, Kubernetes Cluster and version (RKE1, RKE2, k3s, EKS, etc): EKS

Proxy/Cert Details: N/A, AWS Cert Manager

Information about the Cluster

Kubernetes version: 1.18
Cluster Type (Local/Downstream): Local

If downstream, what type of cluster? (Custom/Imported or specify provider for Hosted/Infrastructure Provider):

Describe the bug

Rancher is installed directly in an EKS cluster. After attempting to remove with system-tools and re-install, I can no longer login.
To Reproduce

Install Rancher v2.5.9 via helm (rancher-stable/rancher).
Uninstall Rancher via rancher/system-tools
Re-install Rancher v2.5.9 via helm.
Upgrade to Rancher v2.6.0 and specify bootstrapPassword in values.yml file.

Result

Invalid username or password. Please try again.

Expected Result

I would expect to be able to login.
Screenshots

Additional context
<!--Add any other context about the problem here.—>

Here is the values.yml file contents:

```
hostname: rancher.foo.bar
replicas: 3
tls: external
bootstrapPassword: '[ . . . REDACTED . . . ]'
```

[kubetail_cattle-system.txt](https://github.com/rancher/rancher/files/7228088/kubetail_cattle-system.txt)

Reference(s):
- https://rancher.com/docs/rancher/v2.6/en/installation/install-rancher-on-k8s/amazon-eks
- https://github.com/rancher/system-tools

Did you try the kubectl get secret ... command provided in the screenshot to get the password from the system?
(I might be totally wrong, but I would think bootstrapPassword would not be set when upgrading.)

Did you try the kubectl get secret ... command provided in the screenshot to get the password from the system?
(I might be totally wrong, but I would think bootstrapPassword would not be set when upgrading.)

Yes, I did. I had the same password issue with v2.5.9, so I upgraded to v2.6.0 to see if I could work around the issue by giving it a bootstrap password to use.

I have same problem. But
Nobody cares.

I have same problem. But Nobody cares. Rancher 2.6.2

Are you on an EKS cluster? Did you delete Rancher and then re-install it? Just want to see if it's the same reproduction steps.
I haven't found a resolution yet.

Have you tried to reset the password with:
kubectl -n cattle-system exec $(kubectl  -n cattle-system get pods -l app=rancher | grep '1/1' | head -1 | awk '{ print $1 }') -- reset-password

Have you tried to reset the password with: kubectl -n cattle-system exec $(kubectl -n cattle-system get pods -l app=rancher | grep '1/1' | head -1 | awk '{ print $1 }') -- reset-password

Yeah, that didn't work for me.

Have you tried to reset the password with: kubectl -n cattle-system exec $(kubectl -n cattle-system get pods -l app=rancher | grep '1/1' | head -1 | awk '{ print $1 }') -- reset-password

I setup cluster on bare (1.21.1) by kubespray. After installation, I setup cert manager, Prometheus stack, and try install rancher by helm chart. In first iterations I try setup without bootstrap password in values of chart. And secret bootstrap password in cluster create with empty value. After they I add value using Lens and restart all rancher pod. But I can not login with this password

My setup is different than yours, so I do not have answers, sorry.
Have you tried the Rancher deployment as default with the random password? You would then get the command to track the password within the GUI during the first login?
How about setting an non-empty value with the helm command with --set bootstrapPassword=password
I am not familiar with Lens at all. I would try setting the password with kubectl instead of that in the first place. But still you could either let the system create you a random password or set one your self during the deployment.
(Nothing to do with the password issue, but I would deploy the monitoring stack from inside Rancher.)

I fix this bug in local cluster but very very ugly.
I create the app in the golang for generate new password.
package main

import (
	"golang.org/x/crypto/bcrypt"
	"log"

)

func main() {
	bootstrapPassword:= "testPass";
	hash, _ := bcrypt.GenerateFromPassword([]byte(bootstrapPassword), bcrypt.DefaultCost)
	log.Println("[Password for rancher] ", string(hash))
}

and generate password hash: $2a$10$Gtl3hBxSEwLFDjyxkDHGSejt7PFKdQr4020LDe0OPUSsSIypE6ofq
after that I found Resourse kind: User in Custome Resource, where displayName is "Default Admin" and change password manual using Lens

monitoring stack from inside Rancher

I not use rancher monitoring because I had many problems with using (https://github.com/rancher/rancher/issues/30990, https://github.com/rancher/rancher/issues/34009)

I have not had any password-related issues with any version of Rancher, and setting the password with kubectl has always worked for me. Weird.
Sorry I could not help.

Okay, so it turns out that the admin user account didn't exist! This was remedied via the following command:
kubectl --kubeconfig $KUBECONFIG -n cattle-system exec $(kubectl --kubeconfig $KUBECONFIG -n cattle-system get pods -l app=rancher | grep '1/1' | head -1 | awk '{ print $1 }') -- ensure-default-admin

Thanks to this discovery by my colleague, I can finally login again. I now have a new issue where the local cluster is reporting it's state as "Removing" but that's another bug for another time.

I'm running into this again in a new environment. This time, it's a k3s cluster on bare metal.
I also noticed the command no longer works:
kubectl --kubeconfig $KUBECONFIG -n cattle-system exec $(kubectl --kubeconfig $
KUBECONFIG -n cattle-system get pods -l app=rancher | grep '1/1' | head -1 | awk '{ print $1 }
') -- ensure-default-admin
Error: flags cannot be placed before plugin name: --kubeconfig
Error: flags cannot be placed before plugin name: --kubeconfig

I removed the --kubeconfig $KUBECONFIG parts and it returned this:
kubectl -n cattle-system exec $(kubectl -n cattle-system get pods -l app=ranche
r | grep '1/1' | head -1 | awk '{ print $1 }') -- ensure-default-admin
W0207 22:06:36.415205     162 client_config.go:615] Neither --kubeconfig nor --master was specified.  Using the inClusterConfig.  This might not work.
Found existing default admin user (user-fm27f)
Existing default admin user (user-fm27f) is already enabled
Existing default admin user (user-fm27f) already labeled as admin
Existing default admin user (user-fm27f) is already an admin

So in this new environment, the user account is not missing.

Okay, so it turns out that the admin user account didn't exist! This was remedied via the following command:
kubectl --kubeconfig $KUBECONFIG -n cattle-system exec $(kubectl --kubeconfig $KUBECONFIG -n cattle-system get pods -l app=rancher | grep '1/1' | head -1 | awk '{ print $1 }') -- ensure-default-admin

Thanks to this discovery by my colleague, I can finally login again. I now have a new issue where the local cluster is reporting it's state as "Removing" but that's another bug for another time.

You can delete local cluster completely by patching cluster.management.cattle.io,
kubectl get clusters.management.cattle.io $(kubectl get clusters.management.cattle.io -o jsonpath='{range .items[*]}{<EMAIL_ADDRESS>|  grep local | cut -d ":" -f2) -o=json|jq '.metadata.finalizers = null'| kubectl apply -f -


Move from API keys to route-based authentication for HTTP gateways

Based on the Slack discussion.
We want to get rid of the API keys for HTTP gateways and instead use a randomly generated route for the gateways' endpoints.
What needs to be done (roughly):

Remove apiKey field from the config
Remove API key initialization from Terraform recipes
Turn off API key authentication in the OpenAPI templates
Generate the random path section of the endpoint and update OpenAPI templates to use it
Update the local HTTP gateways implementation accordingly

I managed to test the HTTP and Signed Data gateways by sending a request to the newly-generated addresses via Postman. Since this still needs some documentation, it took me a moment to figure out the fact that an API key is not included in the request headers either but I am sure this will be clear when the docs are done.

@martinkolenic  The docs PR is open here: https://github.com/api3dao/api3-docs/pull/1025 so please feel free to also have a look at that if you have a chance. Your input is always appreciated in case you see something that is not clear :smile:

Hi @vponline, thanks for pointing me in the right direction - I thought the changed docs had already gone live. I've just read your changes and these seem to be exactly what I was missing so we're good to go.


Various improvements to support generating recipes for Noetic

@allenh1 @herb-kuta-lge Please review and merge

Depends on https://github.com/ros-infrastructure/superflore/pull/281 now

@herb-kuta-lge please take a look at this one when you can

Please DON'T SQUASH the commits when merging, it will be impossible to review and understand the commits after they are squashed.


Aggregate Loader - Allow to pass raw values to loader

Currently we are projecting only the id field and passing that to the loader:
https://github.com/entria/graphql-mongoose-loader/blob/master/src/ConnectionFromMongoAggregate.js#L54
We should allow the user to choose to receive the raw values, by adding a raw flag.

it could be useful on connectionFromMongoCursor as well


Rename/Refactor the ProfileView into BuilderHomeView (or something else)?

I think the name profile for the page showing a user's learning paths might be confusing, and also easily confused with the BuilderProfile page.
I propose renaming the ProfileView into BuilderHomeView.
What do you think @carletex and @dgrcode?

Yes, I agree it's confusing. Let's rename it.


Support alternative checkboxes

Many Obsidian themes support "alternative" checkboxes (Minimal, for example), where different characters inside the checkbox brackets give additional meaning to the checkbox.
Currently, this plugin will consider such alternative checkboxes (including the common - [/] checkbox, which means "incomplete") as finished, and not roll them over.
This PR changes this, so only - [x]/- [X] ("done") and - [-] ("canceled") are considered "finished" checkboxes. All other checkboxes are considered "unfinished" and will be rolled over.

Thank you for your contribution @tobiasvl, looks like a neat feature which we should add.
I just merged a bigger change into the codebase, which changes the way we extract ToDos from the documents and added some test.
Can I ask you to check out the newest version, add your changes and extend the test cases that cover the expected behaviour?

@lumoe Okay, I think this should do it. I added a couple of tests I thought made sense (and fixed an unrelated typo in a separate commit). Let me know if you need anything else. (I haven't run the tests locally, and you need to approve running the CI, so we'll see if it works!)

@lumoe Let me know if you need anything else from me!

I think this is hard to standardize. - [ ] and - [x] / - [X] seem to be accepted by a big audience. Looking at the minimal checklist-items, I see multiple ones that could be referred to as "terminal status".
Shouldn't we add more characters to the regex if we are referring to the minimal-theme? Does it make sense for a very wide audience to refer to minimal-theme? Which other themes exist that enlarge the todo-items-functionality?

Those are part of the Markdown standard

Do you mean the GitHub-flavor?

Which other ones do you see that could be referred to as "terminal"?

At least - [>] forwarded seems to signify that there is nothing else to do for this entry.

Do you mean the GitHub-flavor?

Yes, that's what I mean, sorry. It's not part of the official Markdown spec, I forgot since there are so many different ones. But, of course, it's also part of the Markdown supported by vanilla Obsidian (ie. with the standard theme).
By the way, GitLab's Markdown flavor supports an "inapplicable" checbox (- [~]), similar to (and originally suggested to be identical to) the "cancelled" - [-] one: https://docs.gitlab.com/ee/user/markdown.html#task-lists

At least - [>] forwarded seems to signify that there is nothing else to do for this entry.

I can see that. I interpreted it as "forwarded to next sprint/day/whatever". I guess I fell victim to the same semantic interpretation problem!

it's also part of the Markdown supported by vanilla Obsidian

Of course - you're right. My first comment was pointless 🙈

fixes https://github.com/lumoe/obsidian-rollover-daily-todos/issues/87

Hello,
Thank you for the plugin and for the cr.
As shown in this comment, I adapted the regex to match my needs,
Some people may like to only roll-over [ ] items.
Other may want to roll-over [ ] and [/].
Some other may want to roll over [ ], [/] and [>].
Yet other way roll over [?] as well but won't like the [>] to be rolled-over.
I think this is up to every one understanding/use of the different themes, semantic and markdown flavor.
Future will surely bring new things anyway.
Therefore, why not exposing a input box allowing people to enter a list of chars they would like to "roll-over" ?
The javascript would transform this string into the inner part of a regex bracket expression (escaping characters when needed (eg: / instead of /, ] instead of ])).
Having a space in this input box may not be easily visible so we may choose to always rollover [ ] and letting people put in the input box characters other than space that they would like to roll-over.
This would make the implementation very flexible and future proof.

Is there any update on this PR, I'd love to for this functionality to be added to Rollover Daily ToDos.

hello. Are there any updates? Any reasons not to merge this MR?

New plugin version with updated checkbox logic is live. Update available in Obsidian.

How does this work? Is the user able to define which task states rollover and which don't?


drivers/leds/apa102: remove/fix references to I2C/LM-75 (comments only)

Just fixing wrong comments from copy-paste.
Thanks!

@diegoherranz please help fix minor nxstyle warning here:
https://github.com/apache/incubator-nuttx/pull/2321/checks?check_run_id=1410972471
Error: /home/runner/work/incubator-nuttx/incubator-nuttx/nuttx/drivers/leds/apa102.c:141:1: error: Missing blank line after comment
Error: /home/runner/work/incubator-nuttx/incubator-nuttx/nuttx/drivers/leds/apa102.c:184:83: error: Long line found
Error: /home/runner/work/incubator-nuttx/incubator-nuttx/nuttx/drivers/leds/apa102.c:220:78: error: Long line found
Error: /home/runner/work/incubator-nuttx/incubator-nuttx/nuttx/drivers/leds/apa102.c:243:28: error: Operator/assignment must be preceded with whitespace
Error: /home/runner/work/incubator-nuttx/incubator-nuttx/nuttx/drivers/leds/apa102.c:243:28: error: Operator/assignment must be followed with whitespace

Hi @diegoherranz please just fix the Coding Styles issues that was already in the apa102 and push the commit. Thanks for fixing the incorrect reference.


Update Contributor Section in READ.ME

Updating the Contributor section in the READ.ME file to where the title is more accurate to being a Point of Contact section for new users.

I think 'Points of Contact' is more accurate.


[Hotfix] Fix fetchLatestNonOptimizedSnapshotTime

Why are the changes needed?

The condition of fetching the latest non-optimized snapshot is incorrect and the timestamp of expiration is incorrect that must be fixed right away
Brief change log

How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[x] Run test locally before making a pull request

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (not documented)

cc @zhoujinsong


language properties from 008/35-37 and 041 MARC codes

Currently, language properties from the 008/35-37 can have as their object a linked data resource from id.loc.gov, e.g.:
<bf:Work rdf:about="http://example.org/1#Work">
  <bf:language>
    <bf:Language rdf:about="http://id.loc.gov/vocabulary/languages/eng"/>
  </bf:language>
</bf:Work>

This seems a little more elaborate than necessary. Would it be better to just convert like this?
<bf:Work rdf:about="http://example.org/1#Work">
  <bf:language rdf:resource="http://id.loc.gov/vocabulary/languages/eng"/>
</bf:Work>

In addition, the 041 creates entities like this:
<bf:Work rdf:about="http://example.org/1#Work">
  <bf:language>
    <bf:Language>
      <bf:identifiedBy>
        <bf:Identifier>
          <rdf:value rdf:resource="http://id.loc.gov/vocabulary/languages/eng"/>
          <bf:source>
            <bf:Source rdf:about="http://id.loc.gov/vocabulary/languages"/>
          </bf:source>
        </bf:Identifier>
      </bf:identifiedBy>
    </bf:Language>
  </bf:language>
</bf:Work>

In the case where there is no bf:part property of the bf:Language entity, perhaps it would be better to simplify this down to the same pattern:
<bf:Work rdf:about="http://example.org/1#Work">
  <bf:language rdf:resource="http://id.loc.gov/vocabulary/languages/eng"/>
</bf:Work>

That makes more sense to me since that URI resolves to madsrdf and the
bf:Source is the same as the madsrdf:isMemberOfMADSScheme
-Kirk
On Wed, Jun 7, 2017 at 3:16 PM, Wayne Schneider<EMAIL_ADDRESS>wrote:

Currently, language properties for these elements can have as their object
a linked data resource from id.loc.gov, e.g.:
<bf:Work rdf:about="http://example.org/1#Work">
bf:language
<bf:Language rdf:about="http://id.loc.gov/vocabulary/languages/eng">
bf:source
<bf:Source rdf:about="http://id.loc.gov/vocabulary/languages"/>
</bf:source>
</bf:Language>
</bf:language>
</bf:Work>
This seems a little more elaborate than necessary. Would it be better to
just convert like this?
<bf:Work rdf:about="http://example.org/1#Work">
<bf:language rdf:resource="http://id.loc.gov/vocabulary/languages/eng"/>
</bf:Work>
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/lcnetdev/marc2bibframe2/issues/39, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABWqksl3oCg4jZJ5atQLFPJMGKEv-kjuks5sBvcFgaJpZM4NzKKm
.


Add Update repos build configuration for 2.1 branch

/cc @ryanbrandenburg @natemcmaster @pranavkm

/cc @muratg @Eilon can we please triage this? It's important devs can work with latest builds of corefx considering we are closing on preview2

Yeah sure. @ryanbrandenburg / @natemcmaster / @pranavkm - can you guys fight over who can do this the fastest?

I'll just run the update script from my box. IMO no need to make a CI config to run this since we should be locking in 2 days, right?

Script passed successfully. All repos should be on 2.1.0-preview2-30478 now.

I created this config for 2.1.0 releases after preview2. We haven't run it since we don't want to make commits for release/2.1 right now, but it's just a copy of the existing one so I think we're good.


Preview Content API only available if published

Can I double check if the expected behavior is that the preview content api url e.g./preview-content/items/123 is only available when the item is published?
If it is, I'm wondering how to use it for previews? Would it not make more sense to allow the preview api url to be available ONLY to items that are in draft mode?

It's available when is published or draft, think in a static generation site (like nextjs) and even if your content type is published you make changes and instantly can go to see the preview content of that

ok noted on this. thanks for the clarification


Question: why fxhash ?

I found this crate by reading your blogpost Kudos, i much enjoyed it!
I noticed you depend on fxhash. I thought was designed for highly specific use-cases, and is vulnerable to collisions in many others? Did you choose fxhash specifically over alternatives as ahash?
If you could find the time, would you consider explaining a little about this choice?
Thanks in advance

Fxhash was just a fast hasher I knew off the top of my head. I'll have to check out ahash.


add emr scp

Issue number: #3
Description: Protect emr:PutBlockPublicAccessConfiguration
Branch name: Master
Changes proposed:
Add SCP for EMR Block Public Access

Please note the following
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.


Nothing is erverything.

It's the best way for us to code.No code means no bug!!!!!

Too vague. Maybe provide steps to not reproduce?

No code means that you shouldn't do anything.


code tests

At this stage it's crucial to test changes and fixes before pushing to azure, so this issue is here to link to initial build of the testing harness based on https://flask.palletsprojects.com/en/2.0.x/testing/

Beginning work on this. The setup is going to be a combination of:

pytest
pytest-selenium
pytest-flask

I am working on getting the general setup to work. There is an issue with starting the server which I am working out right now

Now have more complete docs and an initial test on 6064ab1ede7e172b4193aba7c5c1f78cd1906ffa

features and issues deferred for rewrite --closing here.


Minor Enhancement in README.md

There are several minor issues in the README.md that need to be fixed for the enhancement of the documentation.

Hey, can I pick this up?


Use internal reg

Fetched upstream
Changed docker registry to internal.

why do we have our own fork?

It's to build our docker image. They don't seem to publish one.


☎ Replace Full Name field with First Name and Last Name fields

Summary of changes
Asana Ticket: ☎ Replace Full Name field with First Name and Last Name fields
The page seems to break on form submit so there's an extra commit meant to address that as well...

Before getting review, please check the following:

[x] Does frontend functionality render and work correctly in IE?
[x] Have we load-tested any new pages or internal API endpoints that will receive significant traffic?
[x] Are interactive elements accessible to screen readers?
[x] Have you checked for tech debt you can address in the area you're working in?
[x] If this change involves routes, does it work correctly with pertinent "unusual" routes such as the combined Green Line, Silver Line, Foxboro commuter rail, and single-direction bus routes like the 170?
[x] Are the changes organized into self-contained commits with descriptive and well-formatted commit messages?

Left a small comment but nothing major.
The other question I have is whether this branch should be merged into the feature branch instead of master?

You're totally right; I updated the target branch to be the feature branch, thanks for catching!

This appears to function fine when I'm in the form, but I'm yet to see any confirmation emails populate from test submissions. Am I missing something needed here?

@ingridpierre with regards to no email response, buried in the docs:

In development, the application is configured to print the contents of the email to the log instead of sending an actual email. This is logged at the info log level, but the default log level is set to warn. Add LOGGER_LEVEL=info to your .env file to see the feedback emails in your logs.

So when developing locally you wouldn't get an email, but you can see the recorded information logged in your terminal window if you have the logger level variable set as specified above. Sorry I hadn't mentioned this when posting this PR :/


Adding regex configuration to the parameter-name-style rule

Adding regex configuration to the parameter-name-style rule, part of issue #2074

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 90.14085% with 7 lines in your changes missing coverage. Please review.

Project coverage is 92.92%. Comparing base (054f273) to head (23d76be).
Report is 62 commits behind head on master.

Files
Patch %
Lines

...log/analysis/checkers/parameter_name_style_rule.cc
89.55%
7 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff            @@
##           master    #2213    +/-   ##
========================================
  Coverage   92.92%   92.92%
========================================
  Files         359      360     +1
  Lines       26740    26912   +172
========================================
+ Hits        24848    25008   +160
- Misses       1892     1904    +12

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

Ok, I think I've addressed all issues, with the regex configuration now extending the existing configuration in a non-breaking way.

Oh, got distracted. Will do another review today.

Thanks! Sorry for the delay, sometimes life is just busy...


Possibility to set error instead of warning

First of all, awesome plugin, one of the most important one, should be a builtin in a danger iOS setup, thank you for that.
I didnt see any option (hidden ?) to set danger as error instead warning in PR message. It would be nice something such as:
periphery.warning_as_error = true (default = false)
WDYT ?

Thank you for using this plugin.

periphery.warning_as_error = true (default = false)

It sounds great. I would like to implement this idea :+1:

Just published 0.3.1 that includes this feature :rocket:
https://github.com/manicmaniac/danger-periphery/releases/tag/v0.3.1

Thank you very much mate !!!!