Strange behavior with custom parameter check using withParameterOfType

Because I am new here, I have put an example with the strange behavior together on my
github.com/cs6502/cpputest---withParameterOfType-
This is because I hesitate to open an issue here that might become no issue at all.
If needed, I can put it here too. Please advise.
The example runs out of the box for everyone who compiles it in his CppUTest Suite as main module without any other stuff needed. In the example the installed custom comparator is called twice where it is expected it should be called just once.

Could you ask the issue here?

Intention of test
A mocked function PostMessage() is called twice in a row somewhere in the code under test with difference in parameter content from first to second call. A test should ensure the mocked function is called in the correct order as well as with the correct parameter content. Therefore the test setup is choosing strictOrder() and the test itself sets the expectations for first and second call of mocked function. There is a custom comparator installed for the parameter pMessage of PostMessage().
Issue during test
On the first call to PostMessage() from code under test the installed comparator for pMessage is called twice from the MockPlugin where it is expected from my point of view it should be called just once. Even though the attached test passes, this might become an issue from my point of view and the MockPlugin should call the custom comparator just once at this point with the expected and posted data that have been set first from the test.
It looks like the false result returned by the CustomComparator on the second unexpected call to itself is ignored from the MockPlugin. If this is guaranteed one could live with this issue as long as the CustomComparator knows about this and takes care for this. But if the CustomComparator for instance saves failed false tests for some reasons it might become an issue again.
Glue for the root cause of potential issue
I guess the MockPlugin saves a kind of signature for PostMessage() with its name and its parameters. These are the same for both calls of PostMessage() in the constructed test and just the content of the parameter pMessage is different. And therefore the plugin offers both saved sets to the custom comparator? But the comparator does not know the context when it is called, so this second call should be prevented by the MockPlugin already. If one of the other not custom compared parameters is different between first and second call the custom comparator is just called once on the first call to PostMessage() as I would expect. This expected behavior is shown in the attached code by uncommenting the define SHOW_EXPECTED_BEHAVIOR.
Example
I have put an example together that should run out of the box for everyone who compiles it in his CppUTest Suite as main module without any other stuff needed. There is console output for explanation. The output from my run is also attached below.
#include "CppUTest/CommandLineTestRunner.h"
#include "CppUTest/TestHarness.h"
#include "CppUTest/TestRegistry.h"
#include "CppUTestExt/MockSupport.h"
#include "CppUTestExt/MockSupportPlugin.h"
#include "stdio.h"

//#define SHOW_EXPECTED_BEHAVIOR

#ifdef SHOW_EXPECTED_BEHAVIOR
	#define eDeviceOther eDeviceC
#else
	#define eDeviceOther eDeviceB
#endif

/// General types
typedef unsigned char byte;

typedef enum {
	eMsgXPosition,
	eMsgYPosition,
	eMsgDiameter,
} tMessageId;

typedef struct {
	tMessageId nMsgId;
	unsigned int uiData;
} tMessage;

typedef enum {
	eDeviceA,
	eDeviceB,
	eDeviceC,
} tDeviceId;

bool PostMessage (tDeviceId nSender, tDeviceId nReceiver, tMessage *pMessage);

/// Debug console
void DebugOut(const char *szTemplate, ...)
{
	va_list ap;

	va_start (ap, szTemplate);
	vprintf (szTemplate, ap);
	va_end (ap);
}

/// Code to be tested
bool UpdateDiameter(unsigned int uiDia)
{
	tMessage sMessage;
	sMessage.nMsgId = eMsgDiameter;
	sMessage.uiData = uiDia;

	return PostMessage( eDeviceA, eDeviceB, &sMessage );
}

bool UpdateXPosition(unsigned int uiX)
{
	tMessage sMessage;
	sMessage.nMsgId = eMsgXPosition;
	sMessage.uiData = uiX;

	return PostMessage( eDeviceA, eDeviceOther, &sMessage );
}

bool CodeUnderTest(void) {
	bool bSuccess = 1;

	if( !UpdateDiameter(42) )
		bSuccess = 0;

	if( !UpdateXPosition(220) )
		bSuccess = 0;

	return bSuccess;
}

/// Mock
bool PostMessage (tDeviceId nSender, tDeviceId nReceiver, tMessage *pMessage)
{
	DebugOut("\nPostMessage() - pMessage=%p\n",pMessage);

	mock().actualCall("PostMessage")
		  .withParameter("nSender", nSender)
		  .withParameter("nReceiver", nReceiver)
		  .withParameterOfType("tMessage", "pMessage", pMessage);

	return (bool) mock().boolReturnValue();
}

/// Test
TEST_GROUP(OrderTest) {
	void setup()
	{
		mock().strictOrder();
	}
};

TEST(OrderTest, SimilarMockCallsInARow) {

	tMessage oDiameter = { .nMsgId = eMsgDiameter, .uiData = 42 };

	mock().expectOneCall("PostMessage")
		  .withParameter("nSender", eDeviceA)
		  .withParameter("nReceiver", eDeviceB)
		  .withParameterOfType("tMessage", "pMessage", &oDiameter)
		  .andReturnValue((bool) 1);

	tMessage oXPosition = { .nMsgId = eMsgXPosition, .uiData = 220 };

	mock().expectOneCall("PostMessage")
		  .withParameter("nSender", eDeviceA)
		  .withParameter("nReceiver", eDeviceOther)
		  .withParameterOfType("tMessage", "pMessage", &oXPosition)
		  .andReturnValue((bool) 1);

	CHECK_EQUAL(1, CodeUnderTest());
}

/// Custom comparator
class MessageComparator : public MockNamedValueComparator
{
	virtual bool isEqual(const void* oExpected, const void* oPosted)
	{
		bool bSuccess = 1;

		DebugOut("\tMessageComparator() for pMessage - oExpected=%p, oPosted=%p\n", oExpected, oPosted );

		/* on different pointer do custom check is desired */
		tMessage *psGeometryExpected = (tMessage*)oExpected;
		tMessage *psGeometryPosted = (tMessage*)oPosted;

		if( psGeometryExpected->nMsgId != psGeometryPosted->nMsgId )
			bSuccess = 0;
		if( psGeometryExpected->uiData != psGeometryPosted->uiData )
			bSuccess = 0;

		const char *szMsgId[] = { "eMsgXPosition", "eMsgYPosition", "eMsgDiameter" };
		const char *szSuccess[] = { "FAIL", "PASS" };

		DebugOut("\t\tID expected=%s, posted=%s -> %s\n", szMsgId[psGeometryExpected->nMsgId], szMsgId[psGeometryPosted->nMsgId], szSuccess[bSuccess] );

		return bSuccess;
	}
    virtual SimpleString valueToString(const void* object) {
        return StringFrom((void*)object);
    }
};

/// CppUTest main function

int main(int ac, char** av)
{
    MockSupportPlugin mockPlugin;
    MessageComparator comparator;
    mockPlugin.installComparator("tMessage", comparator);
    TestRegistry::getCurrentRegistry()->installPlugin(&mockPlugin);
    return RUN_ALL_TESTS(ac, av);
}

Suspected console output:
PostMessage() - pMessage=0069faa8
	MessageComparator() for pMessage - oExpected=0069fb30, oPosted=0069faa8
		ID expected=eMsgDiameter, posted=eMsgDiameter -> PASS
	MessageComparator() for pMessage - oExpected=0069fb28, oPosted=0069faa8
		ID expected=eMsgXPosition, posted=eMsgDiameter -> FAIL

PostMessage() - pMessage=0069faa8
	MessageComparator() for pMessage - oExpected=0069fb28, oPosted=0069faa8
		ID expected=eMsgXPosition, posted=eMsgXPosition -> PASS

Expected console output: (uncomment define SHOW_EXPECTED_BEHAVIOR)
PostMessage() - pMessage=0069faa8
	MessageComparator() for pMessage - oExpected=0069fb30, oPosted=0069faa8
		ID expected=eMsgDiameter, posted=eMsgDiameter -> PASS

PostMessage() - pMessage=0069faa8
	MessageComparator() for pMessage - oExpected=0069fb28, oPosted=0069faa8
		ID expected=eMsgXPosition, posted=eMsgXPosition -> PASS


Fix assertion failure in grpc_channel_create_with_builder

This fixes https://bugs.chromium.org/p/chromium/issues/detail?id=986179

@AspirinSJL

Friendly ping. CI failures look not relevant, what to do next?

@AspirinSJL, can you please record the known failures and get this merged?  Thanks!

Friendly ping, do you still need anything from me?

The only test failure should be an unrelated timeout.
Thanks for the fix!


how to use ssl_certificate_id with this module

When I tried to pass ssl_certificate_id in listener
listener = [
{
instance_port      = "389"
instance_protocol  = "TCP"
lb_port            = "636"
lb_protocol        = "SSL"
ssl_certificate_id = "${module.ldap_elb_certificate.arn}"
},
]
Error: module.ldap_elb.module.elb.aws_elb.this: "listener.0.instance_port": required field is not set
Error: module.ldap_elb.module.elb.aws_elb.this: "listener.0.instance_protocol": required field is not set
Error: module.ldap_elb.module.elb.aws_elb.this: "listener.0.lb_port": required field is not set
Error: module.ldap_elb.module.elb.aws_elb.this: "listener.0.lb_protocol": required field is not set

+1 ^
Ref of same issue with solution : https://github.com/hashicorp/terraform/issues/16582#issuecomment-342570913)

@amitgera what is the solution? Waiting for Terraform 0.12?

Yes, this is now possible in Terraform 0.12.
I have just updated this module to support it.
Please give v2.0.0 a try and open an issue if something is not working as expected.


Fix delete application command API route

Update route to match Discord documentation

Thanks!
Released in https://github.com/disgoorg/disgo/releases/tag/v0.12.6


Take into account rules with empty contition (for tutorial rule)

Description
All the rules that have metadata.yaml file have a condition field. But the tutorial rule (ccx-rules-ocm)
does not. This change fixes the behaviour where errorContent is not filled correctly if condition field is not
provided.
Fixes #218
Type of change

Bug fix (non-breaking change which fixes an issue)

Testing steps

Manual testing. This is not a normal behaviour.
Since tutorial_rule is the one affected, this behaviour will be checked in rule_content integration tests.

Looks ok (but TBH it would be nice to have some tests for such scenario ;)

It takes 10 minutes max to add them, so I'll do it =)

Codecov Report

Merging #217 (7c288d2) into master (ad208d7) will decrease coverage by 0.15%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #217      +/-   ##
==========================================
- Coverage   72.37%   72.22%   -0.16%
==========================================
  Files           9        9
  Lines         362      360       -2
==========================================
- Hits          262      260       -2
  Misses         81       81
  Partials       19       19

Impacted Files
Coverage Δ

content/content.go
86.48% <100.00%> (-0.19%)
:arrow_down:

Continue to review full report at Codecov.

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


Ensure that ownership follows parent album

This is a first attempt to fix an issue where admin could create/move photos and albums around resulting in mixed-ownership album content, etc.
This patch ensure that all add, move, and merge operations performed by admin change the ownership of moved objects to that of the destination album, if any.
I tried to take advantage of the new session infrastructure but I wasn't always successful, e.g., in AlbumFunctions (is it possible to invoke SessionFunction from them?).

Codecov Report

Merging #244 into master will decrease coverage by 0.17%.
The diff coverage is 9.8%.

I tried to take advantage of the new session infrastructure but I wasn't always successful, e.g., in AlbumFunctions (is it possible to invoke SessionFunction from them?).

If you have it in the constructor then it should be fine.

OK, I'll test it again tonight with @ildyria's changes and if it continues working fine, I'll merge it.
Note that it would help with the testing if #232 (and its front end partner https://github.com/LycheeOrg/Lychee-front/pull/95) were merged first.


shaders/tesc/basic.tesc fails

Generated GLSL in /tmp/tmpt1cmqw_0basic.tesc does not match reference /blah/SPIRV-Cross/shaders/tesc/basic.tesc!
$ diff /tmp/*tesc ../shaders/tesc/basic.tesc
3d2
< layout(vertices = 1) out;
5c4,6
< out patch vec3 vFoo;
---
> patch out vec3 vFoo;
>
> layout(vertices = 1) out;
9,14c10,15
<     gl_TessLevelInner[0] = 8.8999996185302734375;
<     gl_TessLevelInner[1] = 6.900000095367431640625;
<     gl_TessLevelOuter[0] = 8.8999996185302734375;
<     gl_TessLevelOuter[1] = 6.900000095367431640625;
<     gl_TessLevelOuter[2] = 3.900000095367431640625;
<     gl_TessLevelOuter[3] = 4.900000095367431640625;
---
>     gl_TessLevelInner[0] = 8.9;
>     gl_TessLevelInner[1] = 6.9;
>     gl_TessLevelOuter[0] = 8.9;
>     gl_TessLevelOuter[1] = 6.9;
>     gl_TessLevelOuter[2] = 3.9;
>     gl_TessLevelOuter[3] = 4.9;
17d17
<

There are three problems:

ordering of the "out patch" vs. "patch out"
location of the layout directive
floating point numbers that can't be represented exactly. For these I recommend using a fractional portion like .875 instead of .9

Is that log from test_shaders.py? If test_shaders.py tests against original shaders that's a bug, but it doesn't do that for me.
The output there does match with the reference files under reference/, it should not try to test against the original shader.

out patch vs patch out. glslangValidator validates this. Is this not allowed according to spec?
Layout directive should not matter if it comes before or after out patch.
I agree on ugly FP numbers in general. Should clean that up in tests to avoid ugly output like this.

This issue should be resolved with https://github.com/KhronosGroup/SPIRV-Cross/pull/7.

Works for me now.  It's a nice cleanup. Thanks.


[FEATURE REQ]Add support for storing Encryption keys in Cosmos Db when using DataProtection service

Library name
Azure.Extensions
Please describe the feature.
When using DataProtection service to encrypt/decrypt data, we store the encryption keys to some distributed resource when using apps that run on distributed systems to help decryption on any of the machine. Currently, the library provides support for storing the keys to Azure blob storage (and Redis as well).
In one of my projects, I needed encrypting auth token that we were storing in Azure Cosmos Db. We use Azure Cosmos db as a distributed token caching resource, a feature provided by MSAL(https://learn.microsoft.com/en-us/azure/active-directory/develop/msal-acquire-cache-tokens). To help different machines use the stored tokens by decrypting them, I needed to store the encryption keys on the same resource as well but the library supports only Redis and Blob storage as Azure resources. I had to write a custom builder extension to add support for writing encryption keys to Azure Cosmos db.
Similar to StackExchangeRedis, I had implemented an extension for reading and writing Encryption keys to Azure Cosmos Db implementing IXmlRepository interface for cosmos db. The client can use an extension similar to PersistKeysToStackExchangeRedis or PersistKeysToAzureBlobStorage for Cosmos db and provide container information in the builder method. This change has been deployed and tested in our solution and it works as expected.

Label prediction was below confidence level 0.6 for Model:ServiceLabels: 'Cosmos:0.52013224,Extensions:0.21414265,Storage:0.12534481'

Hi @MandeepShahi.   Thank you for your suggestion.  This is not something that we're likely to be able to take on in the short-term, but I've added it to our backlog for future consideration.

Hi @jsquire, can I take up this change? I'd be happy to contribute if permitted. I'd already implemented this in our solution and tested it!!

@MandeepShahi :  By all means!   We very much appreciate contributions.   The tricky part here, I believe, will be determining how we want to package things, given that there is no Cosmos library that follows the guidelines for the current generation of Azure SDK packages.
@KrzysztofCwalina, @tg-msft:  Your insight would be very much appreciated.
//fyi:  @JoshLove-msft

@jsquire, should we do the same as we have done for Azure Blob storage? Write the extension and package it in something like Azure.Extensions.AspNetCore.DataProtection.Cosmos?

@MandeepShahi :  I'd wait for feedback from the folks that I mentioned above.   We'd be potentially crossing generations of the SDK, and I want to be sure that our architects would approve before taking action so that we don't ask you to do something that we can't get approved.

Hi @jsquire, any updates on this?

After offline discussion, the Azure SDK team is going to decline adding this as a new library due to not having a Cosmos package from the current Azure SDK generation.   Closing this out.


'tag Level does not exist' error

not 100% sure what I'm doing but when I run the script it throws this error ('tag Level does not exist') and gives a blank "new region" folder with nothing in it.
using different parameters seems to have no effect, do you have any idea of what i"m doing wrong?

Hi, could you check the size of your r.6.-1.mca file?
Or, check if there are any .mca files with a size listed of 0 bytes.

r.6.-1.mca seems to be about 12mb, I went ahead and sorted by size and there's probably like 20 mca files that are 0 bytes in the region folder of my world

What version of the game is it? It could be one of two issues and I'm trying to get both of them fixed right now 🤔

1.18.1

Ok, try running this version of main.py and let mek now if it works or not. I think this should fix the issues with 1.18. https://github.com/Nyveon/MCStructureCleaner/blob/1.5/main.py

Seems to run as intended now, all but 3 of the warnings I was getting for "betternether:nether_city" are gone. not really sure why the other 3 are still there but overall it seems to have done the trick

Great! Glad it's mostly working as intended. Strange that it didn't work for all tags, do you think you could send me your region folder so I could check it out and see why this would be?

doesnt seem to let me attach my region file, I'm new to github am I doing something wrong?
Also I took a look with NBTExplorer in the chucks that the the warnings are claiming the references are but there doesnt seem to actually be anything there anymore.

You might have to upload it somewhere else, or you can send it to me directly via discord Nyveon#9999.
But yeah... very strange. Maybe more changed in 1.18 than I thought at first.


Add option to use type instead of namespace

It would be very useful if there was a generator option to emit PrismaJson["MyType"] instead of PrismaJson.MyType, unless I'm missing something. This would allow me to do something like
  export type PrismaJson ={ [K in keyof Types]: ValueTree<Types[K]> };

What would be the usage of it?
May I ask why you need to write:
declare global {
  export type X = ...
}

?

I'm generating the schema from Typescript and so I just need a mapped type to get the field types.

Well, this should be very straightforward to implement, you're up to do a PR?

I'll work on this asap


Implementation of cross-platform support

Certificate creation is rewritten to use pure .net
Fixes for paths operations and newline checks in tests
Aligned breaking change in UTF8 behavior on classic powershell vs powershell core
Tests are green when run under linux/pwsh and windows/powershell

Fixes #17


Potential bug report - Incorrect 6.390 schedule info

Hi,
I'm teaching 6.3900[6.036] Intro to Machine Learning  in Spring24. I'm writing to report a schedule issue on Hydrant for the class.
We traditionally have Monday/Wednesday class meetings. These are correctly scraped to Hydrant.
We added a Friday class meeting in Spring24, and it was updated on the Registrar http://student.mit.edu/catalog/m6c.html#6.3900 on Jan 2. This update isn't reflected on Hydrant.
I poked around the repo a bit and my understanding is https://github.com/sipb/hydrant/tree/deploy/public has all the raw data but it's not updated for a while. So this seems to be a delayed scraping issue to me? (but then hydrant site shows Last updated: 2024-01-25 13:00 right now, so I'm not sure).
Would appreciate any pointers? Happy to help debug/fix too. Thanks.

seems there to me

perhaps you're looking at fall 2023?

Thanks CJ. Indeed, I refreshed and do the Friday class now. Great!
Curious though. I don't think I was looking at f23 (checked my browsing history, and the URL is the baseURL, doesn't have \?t=f23). I wish I'd thought of taking a screenshot.
I'll see if I can find out more to help understand the issue (or the non-issue 😉) Thanks again.

the "Last updated: 2024-01-25 13:00" thing is directly linked to the data that it shows; latest.json has a key called last updated or something
it's possible that the fireroad data wasn't updated—i'm not sure how often the fireroad backend gets updated, @psvenk ?

Yes, the FireRoad server is still updated manually by running a script; I still need to look into automating it. (And to answer your earlier question, latest.json is deliberately not tracked in this respository because it is updated from the FireRoad server periodically by a cron job; previous terms, OTOH, are frozen.)

By the way, I updated FireRoad and Hydrant earlier today. Let us know if you're still facing issues.
As for what was happening before, it's possible that your browser had cached an older version of the latest.json file; I'm not sure what the TTL for that is on Scripts (where Hydrant is hosted).

for what was happening before, note that, again, the "Last updated: 2024-01-25 13:00" thing is directly linked to latest.json, so whatever hydrant shows under last updated is guaranteed to be when the loaded latest.json was last updated

(closing as cannot repro)


Delete outdated Xcode12b4 builds for reprocessing

Follow-up from #679

Nov 12:

34,591 builds remain having been run with Xcode 12-beta4
14,810 of those failed
30,382 are older than 30 days
13,158 that failed are older than 30 days

Nov 23:

32,622 beta4
13,942 beta4 failed

We've got some idle time today, so I'm starting to delete failed beta4 builds now

We still have quite a number of these in the builds table.
Jan 4th

3,734 beta 2 failed
3,464 beta 2 passed
12,042 beta 4 failed
16,462 beta 4 passed

We still have quite a number of these in the builds table.
Jan 4th

3,734 beta 2 failed
3,464 beta 2 passed
12,042 beta 4 failed
16,462 beta 4 passed

I'd recommend we abandon this as we don't really have the capacity to catch up on these builds.
I think our process should simply be if there's a report of a false negative from a beta build we'll reprocess that specific package. All actively maintained packages pretty much auto-update anyway by way of releases and changes to their default branch.

I'd recommend we abandon this as we don't really have the capacity to catch up on these builds.
I think our process should simply be if there's a report of a false negative from a beta build we'll reprocess that specific package. All actively maintained packages pretty much auto-update anyway by way of releases and changes to their default branch.

I'm going to go ahead and close this. I've deleted a few more batches of 1000 beta builds to day and takes about 2h to build them. We've still got 19k beta builds left and it's quite tedious to keep deleting the batches off and on, when we we don't really have indication that 12b4 had any issues.
It'd be easy if we could just delete them all in one go but that'd then interfere with normal operations (i.e. we'd end up with new releases showing "pending" until we're caught up).


Issue 809 New Champions League tournament

When you have time, could you test this before I merge with master. Thx!

will you still be able to visualize old champions league?
I think it is better to create a tournament_championsleague2015.go file instead of changing the existing one no?

We will still be able to see old CL tournaments. Only the creation is impacted.

ok

Did you test it ?

no


OLM install is not working on CSI 2.6.0

Describe the bug
OLM install is not working on CSI 2.6.0
How to Reproduce?

Install CSI using CSI 2.6.0

Expected behavior
CSI driver should be installed using OLM

Findings:
CSI can be deployed in 3 types of enviroment:

Vanilla Kubernetes
OCP: CNSA(operator) + CSI
OCP: CNSA  + CSI

For Vanilla Kubernetes, SecurityContextConstraints is not required.
For OCP: CNSA(operator) + CSI, CNSA operator.yaml file takes care of installing SecurityContextConstraints.
For OCP:  without CNSA operator:
Following error is observed:
  Warning  FailedCreate  5s (x14 over 46s)  replicaset-controller  Error creating: pods "ibm-spectrum-scale-csi-provisioner-776d78f649-" is forbidden: unable to validate against any security context constraint: [provider "anyuid": Forbidden: not usable by user or serviceaccount, spec.volumes[0]: Invalid value: "hostPath": hostPath volumes are not allowed to be used, spec.containers[0].securityContext.privileged: Invalid value: true: Privileged containers are not allowed, provider "nonroot": Forbidden: not usable by user or serviceaccount, provider "hostmount-anyuid": Forbidden: not usable by user or serviceaccount, provider "machine-api-termination-handler": Forbidden: not usable by user or serviceaccount, provider "hostnetwork": Forbidden: not usable by user or serviceaccount, provider "hostaccess": Forbidden: not usable by user or serviceaccount, provider "node-exporter": Forbidden: not usable by user or serviceaccount, provider "ibm-spectrum-scale-privileged": Forbidden: not usable by user or serviceaccount, provider "privileged": Forbidden: not usable by user or serviceaccount]

Few updates:
For installing CSI using OLM in OCP cluster, user needs to create securitycontextconstraint manually. Documentation needs to be updated for the same. Conveyed the same to Teja.
Page: Performing pre-installation tasks
If installing CSI 2.6.0 using OLM, on a Redhat Openshift Cluster Platform, create securitycontextconstraint in the cluster.
Securitycontextconstraint will ensure that driver and controller pods have sufficient access privileges.

Issue the following command to download the securitycontextconstraint manifest for CSI 2.6.0

curl -O https://raw.githubusercontent.com/IBM/ibm-spectrum-scale-csi/v2.6.0/operator/config/overlays/cnsa/security_context_constraint.yaml

Issue the following command to apply the securitycontextconstraint manifest to create spectrum-scale-csiaccess security_context_constraint.

kubectl create -f security_context_constraint.yaml

@Jainbrt can this be closed ?


grid_load error using Matlab 2018b

Hi Gael,
I ran into the following error using Matlab 2018b Update 2 with a fresh copy of gcmfaces that I downloaded yesterday (12 Feb). I got this error both on MacOS (10.14.2) and a Unix HPC system.
I do not get this error with Matlab 2014b.
Error using dir
Characters adjacent to a ** wildcard must be file separators.

Error in rdmds (line 149)
allfiles=dir( sprintf('%s*.meta',fname) );

Error in rdmds2gcmfaces (line 11)
v0=rdmds(varargin{1:end});

Error in grid_load (line 130)
      eval(['mygrid.' list0{iFld} '=rdmds2gcmfaces([mygrid.dirGrid
      ''' list0{iFld} '*'']);']);

Error in animate_subducted_tracer_cuts (line 33)
grid_load(gloc,5,'compact',1);

I also tried removing the fourth argument "1" from grid_load, but this did not get rid of the error.

I downloaded a fresh copy of gcmfaces (11 Mar), and I'm happy to report that this error no longer occurs.

Note: I discovered that my installation was trying to call an out-of-date version of "rdmds" that was not part of gcmfaces. So this was not a gcmfaces issue after all. Once I deleted the out-of-date "rdmds.m" file, Matlab was able to find the correct version of the function and was able to properly load my data files.


startify and leading space char

When I set the options like
let g:indentLine_fileTypeExclude = ['startify']
let g:indentLine_leadingSpaceEnabled = 0

this works fine, no indent line appears

but when I change the leadingSpaceEnable to 1
let g:indentLine_fileTypeExclude = ['startify']
let g:indentLine_leadingSpaceEnabled = 1

the indent line appears in startity page, fileTypeExclude doesn't work in this case

change the order to load plugins, sorry to bother


Is it possible to specify a max number of history_tracks to maintain for each document?

For example, say for a given model I only want to keep the last 100 revisions and discard the rest.  Is this possible?

AFAIK not directly. The tracker stores data in a collection, so you should be able to create an index on top of it that makes it a capped collection, but I've never tried.
Regardless of the implementation this would be a great feature request! Want to try to implement it? Start with (failing) tests.


Various warnings (PropTypes) in RN 0.45

console:
Warning: PropTypes has been moved to a separate package. Accessing React.PropTypes is no longer supported and will be removed completely in React 16.  Use a plain JavaScript class instead.   We even didn’t have React in dependencies.

@evgenyrodionov please re-open? I'm still seeing the warnings in RN 0.47.1 debugger, and now tracing to redux-logger:
ExceptionsManager.js:73 Warning: PropTypes has been moved to a separate package. Accessing React.PropTypes is no longer supported and will be removed completely in React 16.  Use a plain JavaScript class instead.


Run HMR WebSocket and HTTP server on same port

Run HMR WebSocket and HTTP server on same port.

It now becomes possible to use tools like ngrok to tunnel localhost online with the HMR still working. (This is the main reason for this PR)
No more hardcoded 12321 port for WebSocket server.
One less port used by snowpack.

Trimmed down version of #365

LGTM, thank you


WIP - First Korean l10n work for release-1.16

Target Date: 2019-09-26

Waiting for #16299

@gochist #16299 has been merged to dev-1.16-ko.1 ~ You can go ahead.

@gohist looks nice ! Thank you.
/lgtm

/approve


Jellyseerr changed log format for 1.1.0

This fixes the log check for 1.1.0 of jellyseerr
2022-06-20T09:52:17.637Z [warn][API]: Failed sign-in attempt using invalid Overseerr password {"ip":"::ffff:<IP_ADDRESS>","email":"fakeuser"}

changed to
2022-06-20T09:52:17.637Z [warn][API]: Failed sign-in attempt using invalid Overseerr password {"ip":"<IP_ADDRESS>","email":"fakeuser"}

Also fixed the md file for the setup of the aquis file.

This might be due to the proxy support setting so kept the old log format as well

Any update on when this will be merged?


Add ownerId decorator

Add ownerId decorator and add column to BaseModel

Done in https://github.com/goldcaddy77/warthog/pull/456


Integration tests setup, updated docs

Issue #, if available: #32
Description of changes:

Replaced RF model with scikit-learn
Fixed requirements docs
Setup integration tests run during the build
Removed examples from coverage reports. It is not trivial to track, and it is questionable whether we should be tracking coverage of examples as strict as the core library
As a side effect, updated docs to include recent changes

Testing: made sure tests, integration tests and docs all run locally
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Seems like enum.auto was introduced in 3.6. We can either not support 3.5 or remove the use of auto.

@marpulli I was going to get rid of auto(), feels wrong to rule out 3.5 just because of it

Codecov Report

Merging #51 into develop will increase coverage by 8.81%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop      #51      +/-   ##
===========================================
+ Coverage     81.2%   90.01%   +8.81%
===========================================
  Files           62       58       -4
  Lines         1532     1382     -150
  Branches       154      140      -14
===========================================
  Hits          1244     1244
+ Misses         253      103     -150
  Partials        35       35

Impacted Files
Coverage Δ

emukit/examples/__init__.py

Continue to review full report at Codecov.

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

"will increase coverage by 8.81%."    yay!


jsnext:main field and exports error when transpiling

I'm getting the following when transpiling the jsnext:main entry in my project environment:
index.js?7047:3 Uncaught ReferenceError: exports is not defined
    at eval (index.js?7047:3)
    at Object../node_modules/react-sortable-hoc/dist/commonjs/SortableContainer/index.js (app.js?ffd2f522394b01c333ef:4159)
    at __webpack_require__ (app.js?ffd2f522394b01c333ef:20)
    at eval (index.js?c3b3:17)
    at Object../node_modules/react-sortable-hoc/dist/commonjs/index.js (app.js?ffd2f522394b01c333ef:4198)
    at __webpack_require__ (app.js?ffd2f522394b01c333ef:20)
    at eval (ReactionFeedGuest.js:20)
    at Object../apps/user/components/ReactionFeedGuest.js (app.js?ffd2f522394b01c333ef:271)
    at __webpack_require__ (app.js?ffd2f522394b01c333ef:20)
    at eval (ReactionFeed.js:21)
(anonymous) @ index.js?7047:3
./node_modules/react-sortable-hoc/dist/commonjs/SortableContainer/index.js @ app.js?ffd2f522394b01c333ef:4159
__webpack_require__ @ app.js?ffd2f522394b01c333ef:20
(anonymous) @ index.js?c3b3:17
./node_modules/react-sortable-hoc/dist/commonjs/index.js @ app.js?ffd2f522394b01c333ef:4198
__webpack_require__ @ app.js?ffd2f522394b01c333ef:20
(anonymous) @ ReactionFeedGuest.js:20
./apps/user/components/ReactionFeedGuest.js @ app.js?ffd2f522394b01c333ef:271
__webpack_require__ @ app.js?ffd2f522394b01c333ef:20
(anonymous) @ ReactionFeed.js:21
./apps/user/components/ReactionFeed.js @ app.js?ffd2f522394b01c333ef:258
__webpack_require__ @ app.js?ffd2f522394b01c333ef:20
(anonymous) @ router.js?b5bd:1
./apps/user/router.js @ app.js?ffd2f522394b01c333ef:413
__webpack_require__ @ app.js?ffd2f522394b01c333ef:20
(anonymous) @ index.js?9420:1
./apps/user/index.js @ app.js?ffd2f522394b01c333ef:387
__webpack_require__ @ app.js?ffd2f522394b01c333ef:20
(anonymous) @ app.js?ffd2f522394b01c333ef:63
(anonymous) @ app.js?ffd2f522394b01c333ef:66

I tried both:
import { SortableContainer, SortableElement } from 'react-sortable-hoc;
import { SortableContainer, SortableElement } from 'react-sortable-hoc/src/index';

But soon after I noticed there's no src folder in the installed package! Why this?
Would you consider adding a esnext entry linking the plain files instead of transpiled ones?
Great job!

Solved with:
import { SortableContainer, SortableElement } from 'react-sortable-hoc/dist/es6';


[SAGE-908] Modal add fullscreen variant

Description

[x] add fullscreen variant to sage modal

Screenshots

No overflow
Overflow

Testing in sage-lib

Rails
Visit http://localhost:4000/pages/sandbox and click on the button to see the modal
React
Visit modal storybook
In the options set fullScreen to true
Testing in kajabi-products

(LOW) Adding a new property. No affect on the app.

Related

Closes #908

Will revert sandbox and react storybook before merge


(surge form review) Edits needed to "Specific Topic" quesion

Label: Specific Topics

Already done, will be in next release on staging.


Muaad spectator

Players that join after "WAIT" state are "Spectators" and are "dead" users
Spectators don't have stats applied
Spectators are able to see lobby chat messages throughout the game
Dead users chat messages look different than alive players chat messages.
lobby chat appears for dead users in prompt and chat phase

change the name on this pr to match conventional commits


KeyError when setting TimeRange

I get the KeyError once I set the Time Range on any visualization.
Datasource used is Clickhouse.
Stacktrace :
INFO:superset.connectors.sqla.models:SELECT subcategory AS subcategory, COUNT(*) AS count
FROM ecom.tillnow
WHERE event_time >= toDateTime('2020-06-14 17:00:00') AND event_time < toDateTime('2020-06-15 00:00:00') GROUP BY subcategory ORDER BY count DESC
 LIMIT 50000
DEBUG:urllib3.connectionpool:Starting new HTTP connection (1): clickhouse:8123
DEBUG:urllib3.connectionpool:http://clickhouse:8123 "POST / HTTP/1.1" 200 None
DEBUG:urllib3.connectionpool:Starting new HTTP connection (1): clickhouse:8123
DEBUG:urllib3.connectionpool:http://clickhouse:8123 "POST /?database=default HTTP/1.1" 200 None
DEBUG:urllib3.connectionpool:Starting new HTTP connection (1): clickhouse:8123
DEBUG:urllib3.connectionpool:http://clickhouse:8123 "POST /?database=default HTTP/1.1" 200 None
DEBUG:urllib3.connectionpool:Starting new HTTP connection (1): clickhouse:8123
DEBUG:urllib3.connectionpool:http://clickhouse:8123 "POST /?query_id=2d2b6f18-af52-11ea-9d6a-0242c0a87003&database=default HTTP/1.1" 200 None
Caching 741 chars at key b229be20a1606d62cddbe69a8d1c97cd
INFO:superset.viz:Caching 741 chars at key b229be20a1606d62cddbe69a8d1c97cd
'subcategory'
Traceback (most recent call last):
  File "/usr/local/lib/python3.6/site-packages/superset/views/base.py", line 124, in wraps
    return f(self, *args, **kwargs)
  File "/usr/local/lib/python3.6/site-packages/superset/utils/decorators.py", line 70, in wrapper
    return f(*args, **kwargs)
  File "/usr/local/lib/python3.6/site-packages/superset/views/core.py", line 717, in explore_json
    viz_obj, csv=csv, query=query, results=results, samples=samples
  File "/usr/local/lib/python3.6/site-packages/superset/views/core.py", line 639, in generate_json
    payload = viz_obj.get_payload()
  File "/usr/local/lib/python3.6/site-packages/superset/viz.py", line 406, in get_payload
    payload["data"] = self.get_data(df)
  File "/usr/local/lib/python3.6/site-packages/superset/viz.py", line 2874, in get_data
    levels = self.levels_for("agg_sum", groups, df)
  File "/usr/local/lib/python3.6/site-packages/superset/viz.py", line 2756, in levels_for
    agg_df = df.groupby(groups[:i]) if i else df
  File "/usr/local/lib/python3.6/site-packages/pandas/core/generic.py", line 7894, in groupby
    **kwargs
  File "/usr/local/lib/python3.6/site-packages/pandas/core/groupby/groupby.py", line 2522, in groupby
    return klass(obj, by, **kwds)
  File "/usr/local/lib/python3.6/site-packages/pandas/core/groupby/groupby.py", line 391, in __init__
    mutated=self.mutated,
  File "/usr/local/lib/python3.6/site-packages/pandas/core/groupby/grouper.py", line 621, in _get_grouper
    raise KeyError(gpr)
KeyError: 'subcategory'
ERROR:superset.views.base:'subcategory'
Traceback (most recent call last):
  File "/usr/local/lib/python3.6/site-packages/superset/views/base.py", line 124, in wraps
    return f(self, *args, **kwargs)
  File "/usr/local/lib/python3.6/site-packages/superset/utils/decorators.py", line 70, in wrapper
    return f(*args, **kwargs)
  File "/usr/local/lib/python3.6/site-packages/superset/views/core.py", line 717, in explore_json
    viz_obj, csv=csv, query=query, results=results, samples=samples
  File "/usr/local/lib/python3.6/site-packages/superset/views/core.py", line 639, in generate_json
    payload = viz_obj.get_payload()
  File "/usr/local/lib/python3.6/site-packages/superset/viz.py", line 406, in get_payload
    payload["data"] = self.get_data(df)
  File "/usr/local/lib/python3.6/site-packages/superset/viz.py", line 2874, in get_data
    levels = self.levels_for("agg_sum", groups, df)
  File "/usr/local/lib/python3.6/site-packages/superset/viz.py", line 2756, in levels_for
    agg_df = df.groupby(groups[:i]) if i else df
  File "/usr/local/lib/python3.6/site-packages/pandas/core/generic.py", line 7894, in groupby
    **kwargs
  File "/usr/local/lib/python3.6/site-packages/pandas/core/groupby/groupby.py", line 2522, in groupby
    return klass(obj, by, **kwds)
  File "/usr/local/lib/python3.6/site-packages/pandas/core/groupby/groupby.py", line 391, in __init__
    mutated=self.mutated,
  File "/usr/local/lib/python3.6/site-packages/pandas/core/groupby/grouper.py", line 621, in _get_grouper
    raise KeyError(gpr)
KeyError: 'subcategory'

@med4it what is your Superset version?


No module named 'uploader'

Hello, I'm getting this error. Someone would have an idea how to solve it please?
from uploader import uploadVideo ModuleNotFoundError: No module named 'uploader'

Are you sure that you're running the commands inside the TikTokUploader folder ?

Are you sure that you're running the commands inside the TikTokUploader folder ?

Thanks for your answer! If I do I'm getting another error:
raceback (most recent call last):   File "C:\Users\Nathan\Documents\RSL\index.py", line 36, in <module>     from uploader import uploadVideo   File "C:\Users\Nathan\Documents\RSL\uploader.py", line 7, in <module>     from TikTokUploader.x_bogus_ import get_x_bogus ModuleNotFoundError: No module named 'TikTokUploader'


[DRAFT] distribute as all module types {CJS,ESM,UMD,AMD}

currently the package is a Commonjs module type.
I would love to make it a native module type (ESM).
goal:
have the dist available in multiple flavours:

ESM for browser
commonJS for browser
esm for node
commonjs for node
umd ...
amd ...
... etc ...

implementation:

use https://nodejs.org/api/packages.html#package-entry-points
use https://nodejs.org/api/packages.html#dual-commonjses-module-packages
have multiple typescriot compilers and webpack runs, that generate isolated javascripts for each purpose

are there still plans to produce an ESM for the browser?
I got it to work as an import in my vite project by forking and altering the package.json to:

skip the webpack step in the build phase
point to the index.web.js as main

of course I'd rather see support for this use-case added upstream as opposed to maintaining a fork, so please let me know if I can help.

Moin @agschrei,
yes, the idea is still relevant.
feel free to (draft) pullrequest a (partial) solution.

@agschrei are you still interested in contributing your solution?

I'm not attached to this project but I support this use case. As the front-end industry moves towards ESM for dev builds being able to validate SBOM production as early as possible is very helpful.

This feature is kind of obsolete, now.
There are tests, that show that this library is able to run in ESM context, so no additional work seams to be needed at the moment.
see https://github.com/CycloneDX/cyclonedx-javascript-library/blob/main/examples/node/javascript/example.mjs


feat!: return SendError from send methods allowing replies to be received blocking

ActorRef's send methods which receive replies now return SendResult, allowing message replies to be either received asyncronously, or blocking.
actor_ref.send(msg).await?; // Async recv
actor_ref.send(msg).blocking_recv()?; // Blocking recv

Closes https://github.com/tqwewe/kameo/issues/25

Probably good to be merged, but I can give this a try in a few hours with my code to see how it will work out.


New version don't working stable with WiFi devices

Hi,
I installed new version 0.19.0.
All my WiFi Tuya devices stop working stable.
Generally all my devices - unavailable. Time to time one or two param get info.
See screenshots

After downgrade back to 0.18.2 - all devices working properly.
Please check new version.
How I may help you?

Same with Kogan Wi-Fi convection panel heaters - KAHTP and KAWFHTP models
2022-09-22 03:49:31.620 INFO (MainThread) [custom_components.tuya_local.device] Setting protocol version for topení koupelna to 3.3.
2022-09-22 03:49:31.626 INFO (MainThread) [homeassistant.components.climate] Setting up climate.tuya_local
2022-09-22 03:49:31.626 INFO (MainThread) [homeassistant.components.lock] Setting up lock.tuya_local
2022-09-22 03:49:31.626 INFO (MainThread) [homeassistant.components.number] Setting up number.tuya_local
2022-09-22 03:49:38.632 INFO (MainThread) [homeassistant.components.websocket_api.http.connection] [281472750404032] Connection closed by client

I dunno if that last message is related, but it says unavailable for everything after that.

Thanks for this info.  There are some changes upstream since the tinytuya 1.6.6 release regarding closing and reopening of sockets, so hopefully the next release will fix it.

For now I've marked the last release as a beta to avoid deploying it further until the cause and solution have been found.

I updated my installation.
It look fine after update.
I'll check for 1-2 days.

I still have issues with the latest version:
2022-09-23 18:14:59.193 WARNING (MainThread) [homeassistant.components.switch] Setup of switch platform tuya_local is taking over 10 seconds.
2022-09-23 18:16:32.266 ERROR (SyncWorker_0) [custom_components.tuya_local.device] Failed to refresh device state for Lamparilla Salón.
2022-09-23 18:17:28.288 ERROR (SyncWorker_1) [custom_components.tuya_local.device] Failed to refresh device state for Lamparilla Salón.
2022-09-23 18:17:59.178 ERROR (SyncWorker_0) [custom_components.tuya_local.device] Failed to refresh device state for Lamparilla Salón.
2022-09-23 18:18:28.334 ERROR (SyncWorker_4) [custom_components.tuya_local.device] Failed to refresh device state for Lamparilla Salón.
2022-09-23 18:18:58.645 ERROR (SyncWorker_4) [custom_components.tuya_local.device] Failed to refresh device state for Lamparilla Salón.
2022-09-23 18:19:53.344 WARNING (MainThread) [homeassistant.components.switch] Updating tuya_local switch took longer than the scheduled update interval 0:00:30
2022-09-23 18:20:23.346 WARNING (MainThread) [homeassistant.components.switch] Updating tuya_local switch took longer than the scheduled update interval 0:00:30
2022-09-23 18:20:53.348 WARNING (MainThread) [homeassistant.components.switch] Updating tuya_local switch took longer than the scheduled update interval 0:00:30
2022-09-23 18:21:23.350 WARNING (MainThread) [homeassistant.components.switch] Updating tuya_local switch took longer than the scheduled update interval 0:00:30
2022-09-23 18:21:53.352 WARNING (MainThread) [homeassistant.components.switch] Updating tuya_local switch took longer than the scheduled update interval 0:00:30
2022-09-23 18:22:23.354 WARNING (MainThread) [homeassistant.components.switch] Updating tuya_local switch took longer than the scheduled update interval 0:00:30
2022-09-23 18:23:53.359 WARNING (MainThread) [homeassistant.components.switch] Updating tuya_local switch took longer than the scheduled update interval 0:00:30
2022-09-23 18:24:23.361 WARNING (MainThread) [homeassistant.components.switch] Updating tuya_local switch took longer than the scheduled update interval 0:00:30
2022-09-23 18:24:40.427 ERROR (SyncWorker_4) [custom_components.tuya_local.device] Failed to refresh device state for Lamparilla Salón.
2022-09-23 18:25:23.364 WARNING (MainThread) [homeassistant.components.switch] Updating tuya_local switch took longer than the scheduled update interval 0:00:30
2022-09-23 18:25:53.366 WARNING (MainThread) [homeassistant.components.switch] Updating tuya_local switch took longer than the scheduled update interval 0:00:30

Looks ok so far.

BTW for my Kogan Wi-Fi convection panel heaters - KAHTP and KAWFHTP models I still becomes unavailable for minute or so, after any change, it was doing it even on 18.2 and older. I think it works a bit better with tuya cloud, but it is tuya cloud. I think it might be time to flash esphome on it finally as luckily it is year old and does have espressif chip. Although i noticed Poland made some great progress at open sourcing new tuya arm espressif chip rellacements.

Hello.  you can add this thermostat.  please

Hello.  you can add this thermostat.  please

Please open new issue.
Also you may try to do it by yourself.
Anyway it is nessasary more detailed information about you device.

Failed to refresh device state for glass panel heater koupeln

Last logged: 10:46:44 PM
First occurred: 10:30:35 PM (22 occurrences)
Integration: Tuya Local ([documentation](https://github.com/make-all/tuya-local), [issues](https://github.com/make-all/tuya-local/issues))
Source: custom_components/tuya_local/device.py:132
Logger: custom_components.tuya_local.device

Logger: homeassistant.components.climate
Source: helpers/entity_platform.py:796
Integration: Climate ([documentation](https://www.home-assistant.io/integrations/climate), [issues](https://github.com/home-assistant/home-assistant/issues?q=is%3Aissue+is%3Aopen+label%3A%22integration%3A+climate%22))
First occurred: 10:32:37 PM (7 occurrences)
Last logged: 10:48:37 PM

Logger: homeassistant.components.lock
Source: helpers/entity_platform.py:796
Integration: Lock ([documentation](https://www.home-assistant.io/integrations/lock), [issues](https://github.com/home-assistant/home-assistant/issues?q=is%3Aissue+is%3Aopen+label%3A%22integration%3A+lock%22))
First occurred: 10:31:37 PM (26 occurrences)
Last logged: 10:48:37 PM

Updating tuya_local lock took longer than the scheduled update interval 0:00:30
Updating tuya_local climate took longer than the scheduled update interval 0:01:00

Logger: homeassistant.components.number
Source: helpers/entity_platform.py:796
Integration: Number ([documentation](https://www.home-assistant.io/integrations/number), [issues](https://github.com/home-assistant/home-assistant/issues?q=is%3Aissue+is%3Aopen+label%3A%22integration%3A+number%22))
First occurred: 10:31:37 PM (26 occurrences)
Last logged: 10:48:37 PM

Updating tuya_local number took longer than the scheduled update interval 0:00:30

with 1.8.2 it becomes unavailable, but after while it comes back, with 1.9.1 nope

I updated my installation.
It look fine after update.
I'll check for 1-2 days.

I'm sorry. New version didn't solve the problem. The same trouble. Don't work properly.

1.9.2 reverts the tinytuya library to the previous version. This seems to be working so far for other users.


[Docs] API docs not rendering on docs site

The api documentations page returns a blank page on the docs site.

Thanks @ayushdg - there seems to be a bug in the sphinx build process.
Until it's fixes, please have a look into an older version.

The problem is fixed now and the API docs are visible again. There was a dependency missing during the sphinx build process.
Why I did not receive a mail due to the failed build from readthedocs I do not know so far...

Thanks @nils-braun for the quick response and fix!


Handler::syscall readv implementation should perform at most one VMEXIT

Currently the implementation performs iovcnt VMEXITs with read on each iovec, that is terrible for performance and ideally this should be performed via syscall::Readv or at least single sally.
The main issue is constructing an iterator over &mut [u8] from an iovec passed to us, since we have to validate each iov individually and it is currently a fallible operation.

I gave it a bit more thought and it seems that there are 2 approaches we could try here:

Implement Stage/Call on IntoIterator, and map iovec onto an iterator of syscall::Read to perform just one VMEXIT
Implement a stateful iterator, which would store the returned validation error if it happens and stop iteration once an error is encountered.


user.rs:80-82: Pass port to the base cursor. Now we...

The puzzle 137-66b3c00b from #137 has to be resolved:
https://github.com/h1alexbel/fakehub/blob/5551f71edb028eafb18cbe2a5659099d5dfe666e/server/src/objects/user.rs#L80-L82
The puzzle was created by @rultor on 03-Sep-24.
Estimate: 35 minutes,  role: DEV.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

@0pdd Summary: Quality problem: Lack of clear bug description. Suggestion: Provide a concise exp...

I see you're talking to me, but I can't reply since I'm not a chat bot.


Remove Duplication from Snakemake ETL Workflow / Best Practice Snakemake

Remove tech-debt with duplicate workflows and move ETL/pipelines to best pract Snakemake repository structure

Hopefully now done


Achievement Postman Requests

Add new request definitions to the Observatory Postman collection (.postman/Observatory.postman_collection.json) for the following API routes:

GET /api/achievements
GET /api/achievements/:id
POST /api/achievements
PUT /api/achievements/:id
PATCH /api/achievements/:id
DELETE /api/achievements/:id

Setup:

Download Postman
Import the Observatory.postman_collection.json and *.postman_environment.json files into Postman
Make your additions to the Observatory collection
Export the updated Collection and replace the old version in .postman/Observatory.postman_collection.json
Commit your changes, push to your dev branch, and open a pull request to the dev branch on rcos/Observatory3 (upstream).

Note - this ticket will be assigned to Khalil Fleming (kdot34) when he joins the RCOS GitHub organization.

Closing this issue - irrelevant as Achievements will be deprecated and moved into the RCOS blog


Add metric name prefix to driver options

It would be very useful if we could add a prefix to metric names.
Example:
options.prefix = 'service1_'
// All metrics would have service1_ prefix added before the name
console.log(exporter.poolSize.name); // Would return service1_mongodb_driver_pool_size

Thanks for creating the issue and the related pull request. Before merging can you provide a little more context about the use case for this. Given your example with the service as the prefix I'm wondering if this is really the way to go. Typically and best practice in the prometheus ecosystem is to differentiate between different services/applications using a label and not a prefix on a metric name.

Sure! prom-client library also allows you to set a prefix for the default metrics:
client.collectDefaultMetrics({
prefix: 'service_'
});
I understand it is a better practice to separate services by labels, but sometimes separate services on Grafana with different dashboar

A new release 2.1.0 is now available. Thanks again for your contribution.


Observatory crash in Flutter

As of July 27th, I can't open Observatory on my flutter app built with --profile.
Here's what I see in dev console:

cc @johnmccutchan

@rmacnak-google Any idea?

Chatted with @B3rn475 and discovered we have a version of dart that does have a bug in observatory UI. It should be fixed as of https://github.com/dart-lang/sdk/commit/cb468c2c47d5cbf1d8d5f6cea2d56c102513f86f so we'll ask Flutter to update.
Will close this one, as head of master of dart should be working.

Opened https://github.com/flutter/flutter/issues/5106 to track on the Flutter side.

Looking at the actual JS exception we figured out that it is related to the
Polymer double registration bug that was solved on Sunday.
I'm trying to rebuild the flutter engine on my machine to check that the
latest version is not affected by the bug.

Carlo BERNASCHINA
Google Intern
On Wed, Jul 27, 2016 at 2:46 PM, John McCutchan<EMAIL_ADDRESS>wrote:

@rmacnak-google https://github.com/rmacnak-google Any idea?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/dart-lang/sdk/issues/26976#issuecomment-235731599,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACj_tifNgqfz6xugGLCC2zcUhNYfAXHOks5qZ9HSgaJpZM4JWpvV
.

Have we confirmed that this fixes it?  If so, maybe @whesse could do another roll?

@pq Observatory is built from the runtime that is DEPSed in and not from the SDK

I built the flutter engine using the latest SDK commit and Observatory is
working fine.
Tested on Android.

Carlo BERNASCHINA
Google Intern
On Wed, Jul 27, 2016 at 4:22 PM, John McCutchan<EMAIL_ADDRESS>wrote:

@pq https://github.com/pq Observatory is built from the runtime that is
DEPSed in and not from the SDK
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dart-lang/sdk/issues/26976#issuecomment-235751299,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACj_ttphWfFpwjJl0NcbHhTHJ08YDNTrks5qZ-gdgaJpZM4JWpvV
.

Ah, cool.  Thanks!

Just a friendly update: Observatory work with Flutter again :) Thanks!


Simple styling + Windows (Chrome) fixes

Added some (simple) Bootstrap styling
Fixed some issues in Chrome (Windows):

Loop checkboxes weren't set for audio >= 60 secs
Element positioning was off (overlap)

P.S.: Haven't tested on any other browsers except for Chrome and only on Windows. No idea if this breaks anything on other browsers or on other OSs.

I ended up cherry picking some particular changes like adding preload="metadata" and a width specification to the audio elements in the development of the latest version.


Convert UPPERCASE entities to PascalCase

Convert UPPERCASE entities to PascalCase.
Example: IFCPROPERTYSET becomes IfcPropertySet
This will match the IFC documentation and aid readability.

N.B. Config needs to be updated.
Will update demo and docs.


While there are duplicate LocID don't fetch them >1 time from NBN

While https://github.com/LukePrior/nbn-upgrade-map/issues/180 is still around, use it to cache the NBN status (no need to read it more than once during a cycle).   Many suburbs have lots of duplicates (over 20%) - this will cause similar reduction in requests (and time) to NBN backend.

Ignore. It's already done in NBN api


when I use is checked to get the listbox state ,porpmpt error

    listbox = dlg.ListBox
    print(listbox.items())
    item = listbox.get_item(field)
    if item.is_checked() == True:
        print("T")
    else:
        print("F")

it show error
File "C:\Users\zou-45\AppData\Local\Programs\Python\Python311\Lib\site-packages\comtypes_init_.py", line 274, in getattr
fixed_name = self.map_case[name.lower()]
~~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^
KeyError: 'togglestate_on'

I previously submitted a PR to resolve this issue.
https://github.com/pywinauto/pywinauto/pull/1323
It hasn't been merged yet, so I'd like you to try pulling this branch to your local environment and test it out.


New OneCanvasRenderer

Hey @manuelodelain ,
This is the PR we was speaking about. Please take a look in my OneCanvasRenderer implementation.
Also I added 'x' and 'y' attributes to the animation object, so you can move them inside the canvas.
Please let me know what do you think about the implementation so I can make that x and y attributes work properly on the existing renderers.
Also I think we could add alpha and scale attributes to the animation objects, so we can better control the animations being rendered on canvas. What do you think about?
Best regards,
Miguel

Hi @miguelmoraleda !
Thanks a lot for your addition.
I think it's great to add properties like x and y.
The OneCanvasRenderer is almost the same that CanvasRenderer. Maybe we should just stay with an option for clearing the canvas inside CanvasRenderer.
Instead of assigned dynamicaly an animation property to the render instance we can just pass as an argument the SpriteAnim instance to the renderer (I think it would be more clear).
+1 for scale and alpha properties !

I totally agree with your comments, I'm gonna change those asap.
Maybe I will include scale and alpha too :)

I've updated the code :)
https://github.com/manuelodelain/sprite-anim/commit/99a53cad87708706478d478d9fc4c69571607827
https://github.com/manuelodelain/sprite-anim/commit/1d1c01ebf46d11e30db04216e30f4716c10f5488
But not for the scale and alpha !

Yeah, I saw it.. :) Thanks man. I'm gonna implement that alpha. The scale is kind of tricky. We did it once, but we made it works only for scales between 0 and 1. Upscale wasn't working properly. I'm gonna take a look again.


[Bug]: Potential issue with whitelist mints

Issue description
Hi,
I set up a test CM in mainnet with a whitelist mint with those settings:
{
  "price": 1,
  "number": 25,
  "gatekeeper": null,
  "creators": [
    {
      "address": "6WzAZ5BdVo3YtwwBvBTiTNysUhtKxCVTxzSM7AoRCLWc",
      "share": 100
    }
  ],
  "solTreasuryAccount": null,
  "splTokenAccount": "8ncq2nGqN2kMn3Cc6ADS6x2uGEZ48YXJtQsXagHSkc3b",
  "splToken": "EPjFWdd5AufqSSqeM2qN1xzybapC8G4wEGGkZwyTDt1v",
  "goLiveDate": "21 Jul 2022 20:00:00 +0000",
  "endSettings": null,
  "whitelistMintSettings": {
    "mode": "burnEveryTime",
    "mint": "WL7STrgdAbmFPdDMZrX6QimR1CsdLKrPj7xCvwSVwu3",
    "presale": true,
    "discountPrice": null
},
  "hiddenSettings": null,
  "uploadMethod": "shdw",
  "retainAuthority": true,
  "isMutable": true,
  "symbol": "USGT",
  "sellerFeeBasisPoints": 800,
  "awsS3Bucket": null,
  "nftStorageAuthToken": null,
  "shdwStorageAccount": "EUkV9nmfMM4CzxYAQMpNpwKWgMXFBSTYP8KmMfburPj9"
}

The token has been created with the following instructions
solana-keygen grind --starts-with WL:1
spl-token create-token {ADDRESS.JSON} --decimals=0
spl-token create-account {ADDRESS.JSON}
spl-token mint {ADDRESS} {AMOUNT}

We were trying out to mint with the UI we generated, and we noticed that the "remaining items" count suddenly dropped by 2.
Someone managed to mint our NFTs, without having to burn the whitelist token.
Here is the Tx by the unknown minter.
Here is one of our Txs.
You can see that the 5th step starts for us with "#5.1 Burn", with the address of the WL token.

#5.1 Burn
Interact With
Token Program -TokenkegQfeZyiNwAJbNbGKPFXCWuBvf9Ss623VQ5DAicon copy
Input Accounts
Account
9zb7aNiS8xar4K45NuotESU1bmfHeed6EKtcSat5F75Cicon copyWritable
Amount
1
Authority
AiyvHBEzAANokpK3eEUBDzbAYYbWqpUmFzRG3nFDP4Rbicon copyWritableSignerFee Payer
Mint
WL7STrgdAbmFPdDMZrX6QimR1CsdLKrPj7xCvwSVwu3icon copy

On the other transaction, there is "#5.1 Token Transfer" directly.

#5.1 Token Transfer
Interact With
Token Program -TokenkegQfeZyiNwAJbNbGKPFXCWuBvf9Ss623VQ5DAicon copy
Input Accounts
Source
EWoWngd2MqqWE29ndzQ13FUVAimMmgm3tL2YuqFFKqJ5icon copyWritable
Destination
8ncq2nGqN2kMn3Cc6ADS6x2uGEZ48YXJtQsXagHSkc3bicon copyWritable
Authority
7SkdwF83hds7qbbRfYyxUNFNWTbTBi8Pgz35KEdM7Jazicon copyWritableSignerFee Payer
Amount
1 USDCicon copy

How is this possible ? By chance, we were only doing some tests, but did I miss something ?
I suspect this is a bot scanning for new CM deployed on mainnet, but how was it able to bypass the WL token ?
It paid the $USDC 2, and the NFTs are marked as "test" so no real harm here but I'm glad it didn't happen on the real ones.
Relevant log output
No response
Priority this issue should have
High (immediate attention needed)

Your candy machine seems to be live (public sale), so anyone can mint without a whitelist token. If you want to restrict the mint only to whitelist users before the goLiveDate (presale), you need to set the goLiveDate to a date in the future. Another alternative is to restrict the mint only to  whitelist users, then use "presale": false in combination with "discountPrice": null.
Other combinations are detailed in the docs here:

https://docs.metaplex.com/guides/archived/candy-machine-v2/configuration#whitelist-settings


Rule 942131 False Positive (https://github.com/coreruleset/coreruleset/blob/v4.0/dev/rules/REQUEST-942-APPLICATION-ATTACK-SQLI.conf)

Description
Rule 942131 gives a positive match on PUT request containing the following json-encoded string in body:
Lorem Ipsum is not random text
Audit Logs / Triggered Rule Numbers
Original rule 942131 regex test result, along with a few typical IS NOT SQL query substrings, see URL https://regex101.com/r/6GnIme/1 shows a false positive on harmless string
Suggested solution
Based on the fact, that IS NOT keywords typically come in context with WHERE|AND|OR|HAVING|CASE and possibly other words (thanks @theseion for pointing out the case with CASE) I managed to improve the matching capabilities with modified regex not to give FP anymore, see URL https://regex101.com/r/zbHa2P/1.
The suggested pattern now requires IS NOT to be preceded by WHERE|AND|OR|HAVING|CASE keywords (with a capacity for substrings in between the two parts) to produce true positive, so please consider having the modification consulted with SQL expert for an exhaustive set of values.

Your Environment

CRS version: 4.0:
Paranoia level setting: 2
ModSecurity version: v.3.0.7
Web Server and version: Nginx/1.18.0
Operating System and version: Ubuntu 22.04

Confirmation
[x] I have removed any personal data (email addresses, IP addresses,
passwords, domain names) from any logs posted.

Thanks @nphoe!
@fzipi, @theMiddleBlue I'm not convinced that adding those keywords is a good idea. Alternatively, we could

drop is not entirely
add a stricter rule at PL 3

I lack the expertise in SQL, so I hope you can chime in.

Agree, but unfortunately on "free text" we can't avoid FPs and that's not limited to SQLi but also XSS and RCE.

942131 is a PL2 rule. FPs are unfortunate, but they have to be expected at PL2 and dealt with accordingly.
I also see little interest or initiative to create a PR to fix this or move the remaining problem to PL3.
So I'm closing this.


[BUG] Measurement templates will cause listAllTerrain to fail

Describe the bug
When measurement templates are placed on a scene, listAllTerrain fails with the following stack trace

To Reproduce

Place a circular measurement template on the scene
Execute canvas.terrain.listAllTerrain()

Should be fixed in the latest release.


Exclude generated files from language statistics

Fix to exclude generated files from language statistics

It would be nice if we could also backport this to 1.12 so that there is no need to regenerate all language stats later on, what do you think?

I think backporting to 1.12 is a good idea. It'll save the repeated issues opening about stopping generated files from being included and we'll save having to regenerate later.
I wonder if there is a relatively cheap way of using isGenerated on our diff pages.

It could through it will need reading at least part of the file to detect it

Wait until go-enry/go-enry#24 is fixed upstream

I wonder if there is a relatively cheap way of using isGenerated on our diff pages.

Seconding this, auto-collapsing generated files in diffs would indeed be a nice feature.

I would think it would need to be cached

Wait until go-enry/go-enry#24 is fixed upstream

Merged. For some reason 2.5.0 and 2.5.1 point to the same commit, so I don't know if that was intentional. @mcuadros

Yes, it's intentional, I did a lame move. I deleted the 2.5.0 to avoid creating 2.5.1, 5 seconds after creating it, but then go modules didn't work with this new 2.5.0 so I ended creating the 2.5.1, to make it work, ending with two tags to the same commit :/

Updated go-enry to v2.5.2


The Kitchn: Kitchn's 10 Most Popular New Recipes of September 2022

The Kitchn: Kitchn's 10 Most Popular New Recipes of September 2022.
https://www.thekitchn.com/most-popular-recipes-september-2022-23415313

Should be this instead https://www.thekitchn.com/fried-chickpeas-and-scrambled-eggs-with-garlicky-greens-and-spicy-yogurt-recipe-23415307


cli: --print-commands

Some users would like to run their BLAST distributed. Rather than try to handle that from transrate, we can simply print out the commands to be run and allow them to place the output files in the working directory before running transrate.
Add a --print-commands flag that implements this.

This would also be useful if someone wants to run snap beforehand and then give transrate the bam

+1 for this. I'm trying to investigate some differences in TransRate/Detonate and having an easy way to know how Transrate does the mapping, via snap, would be helpful.
Think --dry-run like in a makefile..

You can get this behaviour currently by using --loglevel debug


Will this plugin ever work for Redmine issues?

It's not very useful for my team, to be used on Forums only.

Same here.
Vote for issue  will be great!

I also agree that will be very useful functionality. I want mentions also about using filters for this field/column on the issues list.
Vote! :+1:

I also agree that voting on issues would be great, especially for features. Voting on features is probably the best way to decide which features should be implemented.


macaddress.one reporting virtualbox port

On linux mint 19.3, node 12.16.1 -
On my desktop until recently, macaddress.one reported the address from the ethernet port used for external networking.
However, the latest version reports the address from a virtualbox port, vboxnet0, of "0a:00:27:00:00:00" (the usual for virtualbox).
Could macaddress.one recognize and not use the virtualbox address?

I pushed a fix. Will release as 0.5.1.


contextmenu append to

After e.preventDefault() and appending changes to context menu, I want to append it to DOM, but I can't find what exactly I need to append (neither items nor ctx are not working), who can help me?

Your question isn't clear.  What are you trying to append to DOM?  The context menu?  What exactly isn't working?

I re-read what you posted, I think you're having difficulty getting things to actually appear in the context menu.  This is from the demo page: https://github.com/TonyGermaneri/canvas-datagrid/blob/master/tutorials/demo.js#L105
It shows how the demo page added items to the context menu.  I hope this help.


Fail migration from CentOS 8 to Rocky on baremetal

I try migrate from CentOS 8.5.2111 to Rocky 8.
CentOS installed on bare metal server and disks added from storage with multipath.
When I try run migration script, I get error
./migrate2rocky.sh: line 562: dm-2/partition: No such file or directory

Unable to gather EFI data: Can't find disk name or partition number for dm-2.

An error occurred while we were attempting to convert your system to Rocky Linux. Your system may be unstable. Script will now exit to prevent possible damage.

my disks:
# ls -ahlp /dev/disk/by-id | grep "dm-2"
lrwxrwxrwx. 1 root root  10 Jan 12  2022 dm-name-mpatha -> ../../dm-2
lrwxrwxrwx. 1 root root  10 Jan 12  2022 dm-uuid-mpath-360060e80223a330050413a3300000600 -> ../../dm-2
lrwxrwxrwx. 1 root root  10 Jan 12  2022 scsi-360060e80223a330050413a3300000600 -> ../../dm-2
lrwxrwxrwx. 1 root root  10 Jan 12  2022 wwn-0x60060e80223a330050413a3300000600 -> ../../dm-2

dmsetup info /dev/dm-2
Name: mpatha
State: ACTIVE
Read Ahead: 8192
Tables present: LIVE
Open count: 3
Event number: 0
Major, minor: 253, 2
Number of targets: 1
UUID: mpath-360060e80223a330050413a3300000600

# multipath -l
Jul 13 15:59:36 | /etc/multipath.conf line 3, duplicate keyword: find_multipaths
mpathb (360060e80223a330050413a3300000606) dm-0 HITACHI,OPEN-V
size=200G features='0' hwhandler='0' wp=rw
`-+- policy='service-time 0' prio=0 status=active
  |- 2:0:1:1  sde 8:64  active undef running
  |- 2:0:0:1  sdc 8:32  active undef running
  |- 12:0:0:1 sdg 8:96  active undef running
  `- 12:0:1:1 sdi 8:128 active undef running
mpatha (360060e80223a330050413a3300000600) dm-2 HITACHI,OPEN-V
size=40G features='0' hwhandler='0' wp=rw
`-+- policy='service-time 0' prio=0 status=active
  |- 2:0:1:0  sdd 8:48  active undef running
  |- 12:0:0:0 sdf 8:80  active undef running
  |- 2:0:0:0  sdb 8:16  active undef running
  `- 12:0:1:0 sdh 8:112 active undef running

What can I do for run this script?

It seems that the script is trying to access a partition on dm-2, which it cannot find. Probably the script expects a certain disk layout, or it might not be fully compatible with multipath devices. But I would suggest you check your multipath.conf for any misconfigurations or any issues, as the output you've provided indicates a duplicate keyword warning. You can also verify the disk and partition layout to make sure the EFI partition is present and correctly identified by the system. You can use lsblk or blkid for this.

Hello there'
I'm under impression I have very similar issue
bash migrate2rocky.sh  -r
migrate2rocky - Begin logging at Wed 19 Jun 2024 11:51:40 AM CDT.
Updating Subscription Management repositories.
Unable to read consumer identity
This system is not registered with an entitlement server. You can use subscription-manager to register.
Removing dnf cache
migrate2rocky.sh: line 562: dm-0/partition: No such file or directory

Unable to gather EFI data: Can't find disk name or partition number for dm-0.

An error occurred while we were attempting to convert your system to Rocky Linux. Your system may be unstable. Script will now exit to prevent possible damage.

 A log of this installation can be found at /var/log/migrate2rocky.log
[root@localhost ~]# cat /etc/redhat-release
CentOS Stream release 8
[root@localhost ~]# ls -ahlp /dev/disk/by-id | grep "dm-0"
lrwxrwxrwx. 1 root root   10 Jun 19 11:41 dm-name-mpatha -> ../../dm-0
lrwxrwxrwx. 1 root root   10 Jun 19 11:41 dm-uuid-mpath-eui.ffffffffffffffff -> ../../dm-0
lrwxrwxrwx. 1 root root   10 Jun 19 11:41 scsi-eui.ffffffffffffffff -> ../../dm-0
lrwxrwxrwx. 1 root root   10 Jun 19 11:41 wwn-0xui.ffffffffffffffff -> ../../dm-0
[root@localhost ~]# multipath -l
mpatha (eui.ffffffffffffffff) dm-0 NVME,SHGP31-1000GM-2
size=932G features='0' hwhandler='0' wp=rw
|-+- policy='service-time 0' prio=0 status=active
| `- 0:1:1:1 nvme0n1 259:2 active undef running
`-+- policy='service-time 0' prio=0 status=enabled
  `- 2:1:1:1 nvme2n1 259:3 active undef running

Is there a known solution or workaround for this issue?


Can't install Weave Gitops on WGE Leaf Clusters

Describe the bug

I'm trying to install Weave Gitops into an EKS cluster that was installed with Weave GitOps Enterprise 0.8.1. I'm currently building a demo to show improved developer experience. The Weave GitOps Core toolsing will be part of that demo.
Environment

Weave-Gitops Version : 0.9.0
Kubernetes version : 1.22.9

The failure is this :
Unable to continue with install: ClusterRole "wego-admin-cluster-role" in namespace "" exists and cannot be imported into the current release: invalid ownership metadata; label validation error: missing key "app.kubernetes.io/managed-by": must be set to "Helm"; annotation validation error: missing key "meta.helm.sh/release-name": must be set to "ww-gitops"; annotation validation error: missing key "meta.helm.sh/release-namespace": must be set to "flux-system
It looks like Leaf Cluster do get the cluster role installed already. This will clash and break the installation of Weave GitOps into leaf clusters.
Expected behavior

I would like to be able to install Weave Gitops into Weave GitOps Enterprise provisioned Clusters. It should just be another application and allow Leaf Cluster only visiblity.

Digging a bit further, I can see that the clash is with the Leaf Cluster base  kustomization :

Yes, this is the resource that blocked your installation.
Simply remove it, and let the HelmController retry.

If you need to install additional resources only on the management cluster, the right place to put the manifests would probably be clusters/management/ instead of clusters/bases/.


I can't autoComplete  the fundamental modules' methods

Environment data

VS Code version: 1.26.1
Extension version (available under the Extensions sidebar): 2018.7.1
OS and version: win7 64
Python version (& distribution if applicable, e.g. Anaconda): python2.7
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): N/A
Relevant/affected Python packages and their versions: os

Actual behavior

Expected  behavior
I just want the module can autoComplete
its method like os.listdir()
Steps to reproduce:

XXX

Logs
Output for Python in the Output panel (View→Output, change the drop-down the upper-right of the Output panel to Python)
XXX

Output from Console under the Developer Tools panel (toggle Developer Tools on under Help)
XXX

I can't reproduce with Anaconda 5.2.0 with Python 2.7.
Was this with Jedi or the new language server?


Codegen README clarification

Thanks! Didn't know we still had that in there 😅
I don't think the Readme needs to contain the history, though. Can you maybe reduce it down to something like this?

It depends on features introduced in Rust 1.15. Make sure to always use the latest stable release for optimal performance and feature support.

I'm new to Rust and Diesel so if you feel the above is better, I have no problem doing it.

Thanks. It's not really a matter of Rust or Diesel, it's more my personal opinion that the Readme only needs to describe what is currently true :)
In general, Rust developers tend to update their compiler version quite often, as there are stable releases every 6 weeks and you only need rustup update to get the latest version. This makes describing the behavior of older compiler versions less important.

Actually, since this is such a small change, I'll just go ahead and do it so you don't have to mess around with adding a commit to that branch (from the "patch-1" name I guessed you used Github's interface). Hope you don't mind.
Thanks again :)


OCPBUGS-30260: Set load balancer target nodes in request serving isolation mode

What this PR does / why we need it:
Adds an annotation on the HostedCluster to allow setting target node labels for control plane router load balancers.
Modifies the request serving node scheduler to set this annotation when scheduling a HostedCluster on a set of nodes.
Only applies subnet annotation to private router loadbalancer, since public load balancers require public subnets.
Which issue(s) this PR fixes (optional, use fixes #<issue_number>(, fixes #<issue_number>, ...) format, where issue_number might be a GitHub issue, or a Jira story:
Fixes #OCPBUGS-30260
Checklist

[x] Subject and description added to both, commit and PR.
[x] Relevant issues have been referenced.

/test verify
/test unit
/test e2e-aws


Accessibility : debugging console should have option to use live regions

Type: Bug
If I'm typing in the debugging console, I should be able to have the option of enabling a live region so that feedback from the evaluation of expressions can be announced immediately upon pressing enter. This may not be appropriate in all situations, thus the request for it to be an option which can be easily toggled.
VS Code version: Code - Insiders 1.90.0-insider (fc8762e773c35d4e52e8a7375704af2e24698e5d, 2024-05-29T05:49:46.157Z)
OS version: Windows_NT x64 10.0.19045
Modes:

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-4800MQ CPU @ 2.70GHz (8 x 2694)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: enabled_ondirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_graphite: disabled_offvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: enabled

Load (avg)
undefined

Memory (System)
15.91GB (9.12GB free)

Process Argv
C:\users\rich\Projects\dsp.git --crash-reporter-id da8aa860-ae68-41ac-aa82-9caa04dc7fc1

Screen Reader
yes

VM
31%

Extensions (5)

Extension
Author (truncated)
Version

vscode-firefox-debug
fir
2.9.10

debugpy
ms-
2024.6.0

python
ms-
2024.6.0

vscode-pylance
ms-
2024.5.1

remote-wsl
ms-
0.88.2

A/B Experiments
vsliv368cf:30146710
vspor879:30202332
vspor708:30202333
vspor363:30204092
vscod805:30301674
vsaa593:30376534
py29gd2263:31024238
c4g48928:30535728
962ge761:30841072
pythongtdpath:30726887
welcomedialog:30812478
pythonidxpt:30768918
pythonnoceb:30776497
asynctok:30898717
dsvsc013:30777762
dsvsc014:30777825
dsvsc015:30821418
pythontestfixt:30866404
pythonregdiag2:30926734
pythonmypyd1:30859725
pythoncet0:30859736
h48ei257:31000450
pythontbext0:30879054
accentitlementst:30870582
dsvsc016:30879898
dsvsc017:30880771
dsvsc018:30880772
cppperfnew:30980852
pythonait:30973460
jchc7451:30973074
showvideot:31016890
chatpanelt:31014475
f3je6385:31013174
a69g1124:31018687
dvdeprecation:31040973
pythonprt:31036556
dwnewjupytercf:31046870
nb_pri_only:31057983
fa862199:31048878
nativerepl2:31062443
refactort:31061298

@jooyoungseo and @rperez030, what do you think of this? My concern is that it could be noisy - especially if one evaluates an object for example.
@RichCaloggero, we are considering dynamic aria announcements for watch variables. #221604

We should also play an audio cue to indicate a warning/error/success

@RichCaloggero I just noticed you said you'd like it to be a setting. Can you think of a case when this would not be preferable?

@jooyoungseo and @rperez030, what do you think of this? My concern is that it could be noisy - especially if one evaluates an object for example.

In the case of arrays or objects in general, something has to be announced, but definitely not the entire content. in the python repl, for example, one gets the reference to the object announced. I'm not even sure what gets presented visually in the vS code output console in these situations. I also agree that it has to be a setting, and idealy it would be ttested carefully before making it to production.

Ok, I will add a setting now. Thanks for the feedback!

@rperez030 I didn't get around to making this a setting, it would be great to get some user feedback on this to see if it's needed. based on my testing, I'm not sure a user would want to disable this.


[Messaging Extensions] Add Client Options Sample

Summary
The focus of these changes is to add samples for the Event Hubs and Service Bus Functions extensions packages demonstrating how to configure the client options
not offered in the host.json settings.

It would be good to update this doc to show how to set the transport - https://docs.microsoft.com/en-us/azure/azure-functions/functions-bindings-service-bus#additional-settings-for-version-5x

Per conversations with Josh, this is actually available in the host settings which is a far more accessible way of setting it.   Closing this out, as it is not needed and will look to update the docs instead.


Terminal rendering issues in Codespaces (iPadOS)

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

VS Code Version: 1.73 - Codespaces / Insiders release
OS Version: iPadOS 15.7

Steps to Reproduce:

Open a Codespaces
Be sure to be using the Insiders Release
Toggle Terminal
The terminal rendering is unstable. Sometimes it only displays the username, sometimes is empty, and very few times it displays the entire shell (username / path / git branch)
Create a new Terminal using the Terminal: Create New Terminal command
A new terminal is created, but the same rendering issue occurs.
You can repeat step 4 as many times as you want, and the same issue in step 5 will happen every time
Type some command inside the terminal area
You won’t see the keys you pressed, but hitting enter, you will notice the command itself was sent, because you can see the decorator at the right of the terminal area that refers to the new shell entry, as the picture below

Switching to Stable version you won’t see such issue.
Thank you

@rebornix could you pls see if you can repro ? I don't have an ipad

It may not a problem with Codespaces, but the iPadOS. I am using code-server and got the same issue.

Disabling terminal.integrated.gpuAcceleration should fix this. the console log from dev tools would be helpful

Disabling gpuAcceleration ("terminal.integrated.gpuAcceleration": "off") indeed fix the terminal issue, thanks!
I wonder what does the Insiders release has that raises this issue, but it's good to know you found that, and probably is aware which feature is responsible for.
Thank you

Not sure which piece of the console log you are looking for, but I noticed a different error message popping up when the gpuAcceleration is enabled.

Is that helpful?
ps: a few more details about the iPad itself. Not sure it makes any difference

8th gen iPad (10.2")
Smart Keyboard
Safari browser

Very helpful, not sure right now how that could happen though. Code it's pointing at: https://github.com/xtermjs/xterm.js/blob/1b13127f0be68b7e4b1a6c71717c74f503ff783d/addons/xterm-addon-canvas/src/BaseRenderLayer.ts#L374-L384

Tracking upstream: https://github.com/xtermjs/xterm.js/issues/4218

Any idea why this is happening only in the Insiders? Because the
gpuAcceleration doesn't seems to be exclusive to Insiders.

@alefragnani the canvas renderer just got a major refactor to share a lot of code with the webgl renderer.

This bug has been fixed in the latest release of VS Code Insiders!
@alefragnani, you can help us out by commenting /verified if things are now working as expected.
If things still don't seem right, please ensure you're on version 9ad069358e9f09d1f49a90db36b73c953505438a of Insiders (today's or later - you can use Help: About in the command palette to check), and leave a comment letting us know what isn't working as expected.
Happy Coding!

@rebornix can you try to verify this on your ipad?

Hi, I just tested and the issue remains.

I’ll try again tomorrow, and post the updates here.
Thank you

@jet082 I reopened and unlocked it

The canvas renderer has been removed from xterm.js/vscode, so this is no longer relevant 🎉


Binary type support

Would be great to see binary type support. Pure JSON could utilize base64 for that, but also besides that same definition could be used to play with MessagePack, that supports binary data natively.

I take it you're proposing an addition to JTD itself? If so:
This was indeed discussed within the IETF, and the rough consensus was that you can just use type: string for that, and then parse the underlying base64 afterwords.
Because this isn't something JTD can quite do "well", because of spotty support for JSON parsers that natively understand base64 JSON strings, and because of the subtleties of base64 (such as the fact that it has four variants, as mentioned above), JTD ultimately did not include this functionality. I don't think that's likely to change anytime soon.

Want to emphasize, however, that I really appreciate you taking the time to make a proposal. Standards-writing is pretty brutal business, and a lot of potentially useful stuff has to be left on the cutting room floor, because the cost of each additional feature is so high (since everyone has to support it in order to support the spec).


Add support for soil parameter options like organic_fract

Add support for the following options:
organic_fract
spatial_snow
spatial_frost
july_tavg_supplied

in which utilities?

grid_params.py has the beginnings of this, but not fully implemented.  So, the wires just need to be finished hooking up.  I'm basically doing this as I make other changes for lake and blowing snow parameters.

this was addressed by PR #40 - for some reason this didn't automatically get closed.  But it can be closed now.

Wait, never mind.  The reason it didn't get closed was because I said it shouldn't be closed yet.  The idea being that this issue pertains to the reverse direction (netcdf to ascii) as well as ascii to netcdf, and this hasn't been addressed yet.

The ascii-netcdf conversion aspect of this issue has been addressed in PR #40.  The netcdf-ascii conversion aspect of this issue has been moved into a new issue #44.  Thus, this issue can be closed.

@jhamman just a reminder that this issue can be closed (see my previous comment in the thread).


Replace time.Before with time.Sub

Also, calculate expiration before next-to-next cronjob run and rotate if required
Fixes #212

@udnay @abhishekdwivedi3060 @prafull01 PTAL


Docs: Added two concepts to backup strategies

I added two concepts to the section on backup strategies:

Replication isn't backup; you still need to consider having a normal, "cold" backup.
Added the idea of disconnecting the backup set, waiting for it to settle, and snapshotting all its file systems.

[Current coverage][cc-pull] is 89.82%

Merging [#418][cc-pull] into [master][cc-base-branch] will not change coverage

@@             master       #418   diff @@
==========================================
  Files            18         18
  Lines          1258       1258
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits           1130       1130
  Misses          128        128
  Partials          0          0

[][cc-pull]

Powered by Codecov. Last updated by [471f032...28b8158][cc-compare]
[cc-base-branch]: https://codecov.io/gh/bigchaindb/bigchaindb/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/bigchaindb/bigchaindb/compare/471f032c1a709c96bbeb3e303e6d57fa6958bc65...28b8158255764bc4a3344f002817668778c4e5f0
[cc-pull]: https://codecov.io/gh/bigchaindb/bigchaindb/pull/418?src=pr

:+1:


Document Items per Page (Pagination) example

Info
In our old storybook documentation we have an example for the Pagination component that details how to add an "Items per Page" option. We need to have that documented in our new website.
Acceptance Criteria:

We have documentation on the Pagination component for how to add an "Items per Page" option

@twjeffery Can you add this example to #1542 and once done, close this story

added example for setting number of items per page to be built out on website to story #1542


feat: git hook pre-push

fix: https://github.com/apache/kvrocks/issues/2301

add-git-hooks sounds wordy and too specific to use. Follwoing npm's practice that uses git hook heavily, we can add a prepare target in x.py. No need for disable and list, that looks strange ...

Sounds good to me. We can just have a ./x.py prepare.

Thanks for your contribution @PokIsemaine!


Honor title on tabs block

If the tabs block has a (block) title, include it in the HTML output. It should be wrapped in a div with the class "title" as the first child of the container (with the class "tabset"). Normal subs should be applied to the text.
This is really just an interim measure until #15 is implemented, which should support a block title just like any other open block.

Resolved by 426b558488027e7fc81e607503f4e00669aecc7e.


Add support for LoadBalancer metrics

Description
This PR enables support for CloudFlare LoadBalancer metrics.
Introduces the following changes:

  feat: Export LoadBalancer analytics metrics

This commit enables support for exporting LoadBalancer analytics
metrics. LoadBalancer's pool health is reported:

1 -> health
0 -> unhealthy

as long as the number of requests each pool receives from each CFL
network location[1]

https://developers.cloudflare.com/load-balancing/reference/load-balancing-analytics/#graphql-analytics

feat: Export browser map page views count

Export browser_map_page_views_count requests metric

Test plan
All changes have been tested locally with a local monitoring stack pulling metrics.

LoadBalancers pool health:

Browser map page views per family

That's really nice :) But should we open this PR upstream and do the review there along with the upstream people? 🤔


[FET-156] Feature: Add recommendation to result page

Add a recommendation to the result page, based on the percentage score.

100%: "Great! Enjoy the colors of life".
60% - 80%: "Good! We recommend you see an opthalmologist".
Less than 60%: "We recommend you see an opthamologist".

Closed because recommendations are not needed as a function of the app. See here for more.


WIP: 2-ci-srv-02: [no-test] Check for NPM dependency updates

Check for any outstanding update of point version of any outstanding NPM package dependency

[ ] npm-update

npm-update in progress on 2-ci-srv-02.
Log: https://logs.cockpit-project.org/logs/npm-update-966-20200517-062323/

npm-update @patternfly/react-styles done: https://github.com/cockpituous/welder-web/commits/npm-update-@patternfly/react-styles-20200517-062424

Please manually check features related to these files:
main.js:import<EMAIL_ADDRESS>package.json:    "@patternfly/react-styles": "3.7.14",


1D

Since I am now focusing on the Stefan problem, it makes sense to do this in 1D instead of 2D. This will help verify that it is easy to switch between 1D/2D/3D in FEniCS.

FEniCS cannot write 1D VTK solutions, so I am outputting a simple table.
This is relevant: https://fenicsproject.org/qa/3932/accessing-the-coordinates-of-a-degree-of-freedom

This works on the stefan_problem branch as of commit bceb66b77624c9f229dc9344abbd5f4d2364f004

Completed with commit bceb66b77624c9f229dc9344abbd5f4d2364f004


feat: Add invalid_target error support

The Token Exchange RFC https://www.rfc-editor.org/rfc/rfc8693.html#name-response defines that if the authorization server is unwilling or unable to issue a token for any target service indicated by the resource or audience parameters the invalid_target error code should be used.
The main goal of this issue is to add these validations to support this error response.

Update: in order to solve this issue an update to the IdentityServer package is also required since the invalid_target isn't supported in the current version of the TokenRequestErrors enum.
https://github.com/IdentityServer/IdentityServer4/blob/main/src/IdentityServer4/src/Models/TokenRequestErrors.cs

Closing.
The project has been archived.
#18


Fix typo lage->large , improve warning message

This pull request fixes one minor typo and improves the warning message so that the end user can have a clue to what is the fault. Prior, the message didnt indicate what size was exceeded.
Thanks
Please merge
George

@apollotonkosmo thanks for contribution. Please sign CLA: http://druid.io/community/cla.html

@leventov Done


Audit and reclassify node catalog

For each node:

Standardize node and parameter names
Classify into new category naming scheme (or remove entirely)
Ensure input invariants are enforced (even when fed via the graph)
Consider if it can be generalized and combined with other nodes

Can we add category for internal/experimental nodes which the user needs to opt in to see?
 I'd imaging something like a toggle in the preferences to opt into that experimental tag/group

Sure, or even just a clearly named category like "Dev Use Only".

Closed by e647ca9f91a5e823137122126fe9e980f65d62ea


API Manager 3.0.0 - drop all changes in carbon.xml file during start server

Hi,
I try to change some values in <APIManager_Home>/repository/conf/carbon.xml.
But after starting WSO2 server this file is overrode by default carbon.xml - all changes are dropped.
It looks strangely, because in API Manager 3.0.0 documentation I found, that it is possible to change this file.
Please, can you give explanation of this situation.
Thanks for your answer in advanced.

https://apim.docs.wso2.com/en/latest/InstallAndSetup/DeployingWSO2APIManager/changing-the-hostname/ &
https://apim.docs.wso2.com/en/latest/Reference/Guides/changing-the-default-ports-with-offset/
have been updated. Hence closing the issue. Feel free to re open the issue if you encounter any issues in the future.


Begin documenting forms in Block Library

Please document all possible blocks and block variants, including templates here so they can be reused easily by Stericycle!

https://main--stericycle-shared--aemsites.aem.page/tools/sidekick/library.html?plugin=blocks&path=/tools/sidekick/blocks/form&index=7


http doesnt work within a bash loop

Looks like a leackage.

I have this file, links.txt:
www.google.com?q=1
www.google.com?q=2
www.google.com?q=3
www.google.com?q=4

Running this:
$ cat links.txt | while read line; do http --print BHhb GET $line; done

Expected result:
4 http calls, one for each line on the file.

Actual:
1 request executed to the first url in the file and request body having the other 3 lines.

GET /?q=1 HTTP/1.1
Accept: application/json
Accept-Encoding: gzip, deflate, compress
Content-Length: 63
Content-Type: application/json; charset=utf-8
Host: www.google.com
User-Agent: HTTPie/0.6.0

www.google.com?q=2
www.google.com?q=3
www.google.com?q=4

HTTP/1.0 400 Bad Request
Content-Length: 925
Content-Type: text/html; charset=UTF-8
Date: Sat, 23 Nov 2013 19:48:51 GMT
Server: GFE/2.0

<!DOCTYPE html>

  <meta charset=utf-8>
  <meta name=viewport content="initial-scale=1, minimum-scale=1, width=device-width">
  <title>Error 400 (Bad Request)!!1</title>

400. That’s an error.

Your client has issued a malformed or illegal request.  That’s all we know.

Running same script with curl instead works as expected.

cheers!

Hi, I have a similar issue which was solved via --ignore-stdin but my context is a bit different:
./run.sh http https://base-url/endpoint\?attributes=all\&nested_attribbutes\=all Authorization:Bearer\ bf54b184d7729ac9bfffb576782e2a1d3cd7bd76 Content-Type:application/json Accept:\ \*/\* -h
I was getting HTTP/1.1 422 Unprocessable Entity
The run.sh script looks as follows: seq 5 | xargs -I {} -n1 "$@"
Adding --ignore-stdin fixes the issue, could you please explain me how it works in this context?

It's because HTTPie by default reads STDIN, if it is redirected (which it is here inside the loop).
There are multiple ways to get rid of it:

The best way is to use  --ignore-stdin (which exists to solve exactly this problem):

cat links.txt | while read url; do
        http --ignore-stdin --verbose $url
done

Or, change STDIN back to terminal:

cat links.txt | while read url; do
        http --verbose $url < /dev/tty
done

Or, do not redirect STDIN in the first place (watch out for spaces though):

for url in `cat links.txt`; do
        http --verbose $url
done

See also #150.

Thank you very much

Thanks a lot for the --ignore-stdin tip :pray:


Configure and makefile updated to use opam depext and specify that icc/gcc/clang are not mandatory.

QoL: ./configure will be executed when using make if src/config.ml is not found. If you build with dune build without ./configure first, that's not our problem.
Should fix #13
cc @odiferousmint

I'll merge this branch when the tests succeed since it doesn't touch the code ;-)