Upon editing a cluster owner member is not populated

I'm pulling this single item from this issue https://github.com/rancher/dashboard/issues/3727 as it still needs to be tracked but it's not related to the other items.
I created a cluster with all defaults in the UI. When editing, the following fields are not populated and I can't save the cluster. I'm marking it as a release-blocker since I can't actually edit the cluster.
Note: I have permissions for the cluster as I am able to view the cluster in my cluster dropdown, navigate to it and create k8s resources through the ui.
Member Roles - I am the cluster creator/owner of the cluster and can see it when creating the clsuter, but it doesn't populate anything when I go to edit the cluster. I have the permissions to view the cluster in the cluster dropdown, so not sure why it's not populating here.

As I noted elsewhere. This issue appears to be coming from the backend. The backend doesn't appear to create the owner clusterroletemplatebinding which this issue presents itself.

@codyrancher , is this V2 (RKE2) provisioning specific or happens for RKE1 too?

I think this has been resolved and working now? Can you double check this @codyrancher ?
I believe I validated this use case for standard users and it was working towards the end of last week.

@deniseschannon looks fine on my end. We do see system accounts now which is captured in https://zube.io/rancher/dashboard-ui/c/3815.

When I edit RKE2 DO cluster that I created as Standard-User in dashboard I can see 3 owners in Member Roles.
I see the same when I edit cluster as Admin.

So cluster owner roles are populated, but I reopened https://github.com/rancher/dashboard/issues/3256 for the principal

Reopening, it worked fine yesterday, there are no roles today again.

v2.6-head (2778eca)

@kravciak was the cluster still being provisioned when you edited?
The backend provisions the binding so if it hasn't done that yet we don't show the member. It seems that we do display it once it exists.

Nevermind, I just realized that it was requiring a page refresh to show.

As Standard-User I can't see them even after page refresh (on DO RKE2 cluster that I created as Standard User)

Works fine on v2.6-head (b8a44fa)


Support loglevel.getLogger(Symbol())

Passing a string to getLogger does not ensure that your logger won't collide with existing loggers, leading to unexpected behaviour. In tests or for libraries that don't know much about what environment they're running in, it can be helpful to use Symbol to ensure there can never be collisions.

Great suggestion! I've added support for that in https://github.com/pimterry/loglevel/commit/ae3a282a02ce377edad43e129c83a6d53cd7ed0c.
The one tricky bit is that we previously used names for log level persistence: in a browser, the level for each logger is stored as loglevel:${logname} in local storage. We can't uniquely do that for symbols, so I've just dropped persistence for symbol-named loggers entirely, we effectively treat them as fresh every time. Can you take a look at at that, and let me know if that would work for you?

Thanks for the quick turnaround! And yes, that definitely works for me.

@pimterry
index.d.ts should fix:
getLogger(name: string): Logger;
to
getLogger(name: string | symbol): Logger;

Good catch, thanks @dixdiydiz! Now fixed in v1.7.1.
In future do feel free to throw in a PR, I'm very happy to merge tiny fixes like that straight away.


BrowserWebDriverContainer fails to determine Selenium version for Selenium v3.9.1

When starting a BrowserWebDriverContainer it tries to determine Selenium version from classpath, looking for the Manifest file from selenium-api.jar.
Using Selenium 3.9.1, the detection fails and it uses the default 2.45.0 version, thus pulling a selenium/standalone-chrome-debug:2.45.0 for Chrome.
Looking at the code of SeleniumUtils class, we can see that it reads Selenium-Version value of the Build-Info section in the Manifest file to infere the version to use.
That is ok in selenium-api-2.45.0.jar:
Manifest-Version: 1.0
Built-By: linman
Build-Jdk: 1.7.0_65
Created-By: Apache Maven 3.1.1
Archiver-Version: Plexus Archiver

Name: Build-Info
Selenium-Revision: 5017cb8e7ca8e37638dc3091b2440b90a1d8686f
Selenium-Version: 2.45.0
Selenium-Build-Time: 2015-02-27 09:10:26

But in selenium-api-3.9.1.jar, the format is not the same:
Manifest-Version: 1.0

Name: Build-Info
Build-Revision: 63f7b50
Build-Time: 2018-02-07T22:25:02.294Z
Build-User: shs

Name: Selenium
Selenium-Version: 3.9.1

Thus, reading Build-Info section, the Selenium-Version attribute can't be found.
However, looking at selenium-api-3.10.0 & selenium-api-3.11.0, the Selenium-Version is back in the Build-Info section... I didn't check all selenium-api version, but it looks like the format of the Manifest file is not really a standard.
To be more adaptable to format change, I think the code should check for the Selenium-Version entry in all the sections of the Manifest file and not only the Build-Info section, hoping we can at least rely on the fact the Selenium-Version will be present somewhere...

Fixed in #632


Moved to app styles to config

Created separate styles object in config. Addresses #11.

Thanks :+1:


Support for required arguments?

Would it make sense to support required arguments? For example:
const args = arg({
	'--help':    Boolean,
	'--port':    arg.req(Number),
	'--name':    String,
	'--tag':     arg.req([String]),
});

Alternatively:
const args = arg({
	'--help': Boolean,
	'--path': String,
});

const {
	'--help': printHelp = false,
	'--path': thePath = arg.required('--path', String),
} = args;

arg.required() would print an error message and exit Node.js. Downsides of this approach: Doesn’t handle aliases, redundancy.
If this is out of scope, it would be a good FAQ, because many shell argument parsing libraries support this feature.

Pretty much the existing scope of features is the intended scope of features, honestly. Any sort of conditional logic is considered out of scope. Usually "required" arguments are handled by if (!args['--some-arg']) throw new Error('--some-arg is required') - at least, that's how we intend.
I'm :+1: for an FAQ, however :)

Finally got around to this, thanks for pointing it out @rauschma. Also super weird coincidence, I just came back to this repo to look for something right after finding your openurl module and decided to take a look at this issue. Had to go back and check who the author was!
Anyway, FAQ in the readme now. :)


Allow defining Lambda functions of the project root

CDK throws a recursive error if the Lambda function srcPath is the project root. Currently, we throw an error saying that the srcPath is invalid.

Fixed in #17

Fixed in #17


Ability to configure ToTypeScriptType to handle nullable types differently.

Currently, NTypewriter appends " | null" to nullable types:
https://github.com/NeVeSpl/NTypewriter/blob/018ead1c6d6abd87f4125972919e905483d544a7/NTypewriter.CodeModel.Functions/TypeFunctions.ToTypeScriptType.cs#L25
I would like to be able to use any of following TypeScript notations in my templates:
myProperty: string | null;
myProperty: string | undefined;
myProperty: string | null | undefined;
myProperty?: string;

I could attempt to write my own implementation of ToTypeScriptType, but I'd rather not copy-paste 100+ lines to change a single detail. I'd like to help out if possible, but I'm not sure where to start - I'm not sure what the best approach is here. Is there already an example of something similar to this in the code base, e.g. a configuration option that affects the behavior of a static function such as ToTypeScriptType?

At the current stage, breaking changes are not a problem. But in that case, I would leave the behaviour of ToTypeScriptType  as it is, because  | null is the most accurate equivalent of nullable type.
The function that you described already exists, only needs a few tweaks: add null checking, documentation, make it public
https://github.com/NeVeSpl/NTypewriter/blob/018ead1c6d6abd87f4125972919e905483d544a7/NTypewriter.CodeModel.Functions/TypeFunctions.ToTypeScriptType.cs#L31
But the name is awful and it shows that I did not have an idea how to name it. Something better is needed, that will underline distinction. And most obvious choices seem too long :
ToTypeScriptTypeNotNullable
ToTypeScriptTypeRaw
ToTypeScriptTypeExtant
and more ideas on how to replace the phrase not null:
https://stackoverflow.com/questions/14273820/is-there-a-single-word-meaning-not-null

I know you just said you want to leave the behavior of ToTypeScriptType as-is, but I'll make the following suggestions anyway:
How about renaming the existing function to ToTypeScriptUnionType and use ToTypeScriptType for the non-nullable variant?
Or, perhaps even better, an optional boolean argument named "nullable". It could be true by default if that's the default behavior you'd want.

I would go with the proposed nullable parameter, it seems most obvious from a user perspective what it does.

After doing some work on this, I realized that having an option to exclude the nullable postfix isn't going to be sufficient for my use case. Arrays and generics make things a little more complicated. The function currently produces the following:
myProperty: int | null[]
myProperty: MyGeneric<int | null>

In these cases, I'd like to be able to configure NTypewriter to use  | undefined instead.
By the way, int | null[] is a bug. This is a type that can either be an int, or an array containing nulls. This does not represent an array that contains nullable ints.. It should be (int | null)[] or Array<int | null> instead. I will create a pull request soon that fixes this problem.


Replace Word (docx) with comparable markdown (md) file for adding artifact repo

Add markdown file
Remove Word (docx) file
Add necessary images into the Documentation/images subfolder
Redo the Word document as markdown

Hi @d3r3kk, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
We will now validate the agreement and then real humans will evaluate your PR.
TTYL, AZPRBOT;

Please squash your commits

LGTM.

@d3r3kk, please do squash your commits for future PRs :-)


Can't read buffer that was not empty before opening serial port (Windows)

I'm using Windows 10 with a serial port I can manually write data to via a terminal to test the following code:
SerialPort port = SerialPort("COM42");

port.openReadWrite();
port.flush(); // not working ??
SerialPortReader reader = SerialPortReader(port);

reader.stream.listen((data) {
  print(data);
  port.write(data);
});

It works when the serial port's buffer is empty: the raw data is sent back to the serial port.
When I send 3 bytes or more via my device BEFORE starting the previous code, nothing happens: dart/flutter can't read any new byte i send via my device.
With 2 bytes before starting the code, it's ok, but if it's more than 3, the data somehow gets stuck.
I tried using the flush() and the drain() methods but none of them change anything.
If I try to print port.bytesAvailable, I get 0.
The only way to get my code to work again is to:

unplug and plug the device
connect to the device with Putty which displays the bytes I send, then closing Putty.
These 2 workarounds are just 2 ways of flushing the buffer before starting the code ...

Please note that this is not related to my device as I don't have this issue while using PySerial on Python.
Any idea ?

Solved ... I did not configure the port correctly (at all, in fact).
Made some attempts and looks like I had to configure the dtr to 1 with libserialport while pyserial somehow did it for me.
I still don't know why not configuring the baudrate does not cause any problem, but I do it anyway.
SerialPortConfig conf = port.config;
confg;baudrate = 9600;
conf.dtr = 1;
port.config = conf;

This solved everything.

Does it work with 115200 baudrate?

If I set my serial device to 115200, yes of course.
Le lun. 13 nov. 2023 à 22:44, mihailacusteanu @.***> a
écrit :

Does it work with 115200 baudrate?
—
Reply to this email directly, view it on GitHub
https://github.com/jpnurmi/flutter_libserialport/issues/96#issuecomment-1809177183,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APTKNH45JIHWB5IFVJZSSKTYEKID3AVCNFSM6AAAAAA7ILKVNWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMBZGE3TOMJYGM
.
You are receiving this because you modified the open/close state.Message
ID: @.***>


Scroll-2-focus input issue with rendering on iOS

Resources:
Before submitting an issue, please consult our troubleshooting guide (http://ionicframework.com/docs/troubleshooting/) and developer resources (http://ionicframework.com/docs/developer-resources/)
Please make sure you are posting an issue pertaining to the Ionic Framework. If you are having an issue with the Ionic Pro services (Ionic View, Ionic Deploy, etc.) please consult the Ionic Pro support portal (http://support.ionicjs.com)
Ionic version:  (check one with "x")
(For Ionic 1.x issues, please use https://github.com/ionic-team/ionic-v1)
[ ] 2.x
[x] 3.x
[] 4.x
I'm submitting a ...  (check one with "x")
[x] bug report
[ ] feature request
Please do not submit support requests or "How to" questions here. Instead, please use one of these channels: https://forum.ionicframework.com/ or http://ionicworldwide.herokuapp.com/
Current behavior:
On iOS, when focusing and scrolling to the top element that is half hidden, cloning and rendering breaks in a way that looks like duplicate element see image
1st image before focusing

And second in the middle of scrolling to focus

The end state:

Expected behavior:
No duplication or artefacts
Steps to reproduce:
have multiple input items, scroll down a bit and focus the top most input. To easily see what happens, have inputs as non-transparent (like in my example). Additionally, you can set breakpoint on subtree modification of input-wrapper

Related code:
I managed to mitigate the issue with setting visibility setting on cloned input while the clone input was present: add these lines to input.js:
-->
//add to cloneInputComponent function
//after line: srcComponentEle.style.pointerEvents = 'none';
srcComponentEle.style.visibility='hidden';

//add to removeClone function:
//after line: srcComponentEle.style.pointerEvents = '';
srcComponentEle.style.visibility='';

The scrolling is also much smoother when visibility is modified.
However, this causes a different issue with keyboard not popping up every time when trying to focus input.

Yea, visibility doesn't work because of the iOS touch and keyboard behaviour. opacity is better.
focusedInputEle.style.opacity = '0';

However, for some reason, it seems much smoother with visibility. That one would be ideal

Hello! Thank you for opening an issue with us! Would you be able to provide a sample application via GitHub that demonstrates the issue you are having? Thanks for using Ionic!

It's literally present on any form with inputs. You can see it on untappd app, register form, but it's less noticeable because inputs are transparent. The way it manifests itself in that app is that labels tend to show after the scroll event is finished, but in reality the textbox pushes the label outside the div with overflow hidden.


Document new widget generation setting

IMPORTANT: This PR is currently in the process of internal review by WTF team.
The PR includes:
New Feature: WTF-1679

Sounds good @alihcsumer . Can you give a bit more info about the feature in the section, and do you know when this feature might go live?

If data grid 2, combo box and image widgets are available in the projects with this setting , they will be used for content generation. The setting is true by default and will be a step towards modernization of core widgets.  A separate documentation for this process is also planned. The setting mentioned will be available in the next release.


Update SimulateTokenRequest test helper to enable returning mocked responses when GetToken is called on an underlying credential type that doesn't match the input credential

This case shows up when a credential type like AzurePipelinesCredential or WorkloadIdentityCredential defers the GetToken call to another credential (ClientAssertionCredential) rather than doing the work directly.
See https://github.com/Azure/azure-sdk-for-cpp/pull/5802#discussion_r1678802661

Fixed in https://github.com/Azure/azure-sdk-for-cpp/pull/5802#discussion_r1680447767


Apply Identity property to Id column with custom name

For #51.

Looks good :+1:

Thanks! Hopefully I'll have time to get back to the .Net core stuff (#53) soon...


Memory management for array allocation in nopython

Introduces a numba runtime or NRT
New option to jit, njit nrt=True to enable NRT.
support np.empty() in nopython+nrt
return array, return slices of array in nopython+nrt
return any array in nopython+nrt

How does this work? Do you use the memory allocator of Python, or a separate one? Do you use reference counting or garbage collection?

This implementation uses a custom atomic reference counting.  The atomic operations are generated by LLVM for portability so you won't see it in the C code.  Cyclic reference is not yet handled.  Our first and major usecase is for arrays, which cannot have cyclic reference.  A separate reference count is maintained by NRT so that incref/decref do not need the GIL.
At https://github.com/sklam/numba/commit/f75cdfd548474caf994da5dc15a5ef3d9c3fc287, the implementation is not completely threadsafe yet.  Internally, NRT maintains some data structures (the freelist for MemInfo) that are subject race condition.  I am working on a threadsafe version by using atomic compare-and-swap.

So, from a quick glance, the implementation adds a lot of complication that doesn't sound strictly necessary for a reference-counting scheme:

the reference count is a separately allocated structure rather than an inline member
it has its own freelist scheme
it has a "deferred destructor" scheme (what is that, and why is it useful?)
it has its own spinlock implementation (this sounds like a bad idea, IMHO)
it maintains function pointers for the atomic increment and decrement functions

Ideally:

the reference count field is all we have to add to existing structures, and it can be inlined (similar to PyObject_HEAD for CPython objects)
the object type is known at compile time, so is the destructor
there is no need to customize the increment and decrement functions at runtime
if we want to handle lock-free structures, it sounds better to use an existing library :-)

So, from a quick glance, the implementation adds a lot of complication that doesn't sound strictly necessary for a reference-counting scheme:

the reference count is a separately allocated structure rather than an inline member

In the emitted llvm, numba treats most data as immutable.  Structures are passed around within each function as LLVM structs.  To have a mutable field, we will need to make structures pass by reference (by pointer to the LLVM struct).  My implementation makes the MemInfo an opaque pointer that can be treated as an immutable field.  This avoids substantial changes to the compiler.
As to why we pass structures by value:

it is faster (due to fewer indirection) on GPU architectures where memory bandwidth are scarce.
easier ABI for functions since we only pass around non-aggregate, primitives types (and there are so few of them).

it has its own freelist scheme

This is to avoid too much malloc/free calls.  But, I haven't proven it to be useful.

it has a "deferred destructor" scheme (what is that, and why is it useful?)

Memory backed by a PyObject requires the GIL for Py_DecRef, which is called once the NRT refct drops to zero.  I am considering if it would be useful to allow the destructor be deferred so that we can call perform multiple Py_DecRef in a GIL acquire+release.  Otherwise, we are calling one Py_DecRef per GIL acquire+release.

it has its own spinlock implementation (this sounds like a bad idea, IMHO)

I am using LLVM to generate the atomic operations for portability reason.  That way I don't have to change anything when we port to a new architecture.  Of course, we can always substitute it with a portable atomic+lock library.

it maintains function pointers for the atomic increment and decrement functions
see comment above.

Ideally:

the reference count field is all we have to add to existing structures, and it can be inlined (similar to PyObject_HEAD for CPython objects)

see comment above

the object type is known at compile time, so is the destructor

We only differentiate between NRT allocated vs Python allocated at this time.  The compiler needs to insert type-specific destructor for types containing other allocated objects.  But, a function does not know who allocated the memory.

there is no need to customize the increment and decrement functions at runtime
if we want to handle lock-free structures, it sounds better to use an existing library :-)

Can you suggest a library?

In the emitted llvm, numba treats most data as immutable. Structures are passed around within each function as LLVM structs

Oh, yes, sorry for that. I always forget it (my mistake for often dealing with PyObject pointers :-)).
That said, the MemInfo could still be stack-allocated in the Numba function (a bit like generators), rather than heap-allocated. I wonder if that would be complicated to implement...

Memory backed by a PyObject requires the GIL for Py_DecRef, which is called once the NRT refct drops to zero.

I don't understand how a Numba-compiled function can ever call Py_DecRef in nopython mode. That means the object was passed as parameter to the function, but then we shouldn't need to memory-manage it ourselves...

I am using LLVM to generate the atomic operations for portability reason

I am talking about the lock / unlock primitives in the NRT lib. AFAIU, tuning a spinlock is delicate (you want to yield after a certain # of failures, otherwise you may waste a whole timeslice waiting for the unlock).
By the way, if the freelist is just a single-linked list, perhaps you can just atomically replace the list head without any locking? Something like:
list *newhead = malloc(...);
do {
    list *head = global_head;   /* volatile */
    newhead->next = head;
} while (!atomic_cas(&global_head, head, newhead));

Can you suggest a library?

I'll search for one if this is really necessary. But I wonder if we could simply not have any spinlock at all.

I don't understand how a Numba-compiled function can ever call Py_DecRef in nopython mode. That means the object was passed as parameter to the function, but then we shouldn't need to memory-manage it ourselves...

It can happen with container types once we have them.

By the way, if the freelist is just a single-linked list, perhaps you can just atomically replace the list head without any locking? Something like:

Yes, I will try that.

It can happen with container types once we have them.

IMO that's a bit prematurate :-) Also, if we want a specialized representation of container types, not a bunch of PyObject pointers, we wouldn't share the representation with the CPython one...

It is for the case when nrt is borrowing memory owned by a pyobject.
Let's say we have:
class Foo(object):
    def __init__(self, array):
        self.array = array

If we don't want to copy the data of array (a numpy.ndarray), MemInfo needs to keep a reference to the original object.  Decref on MemInfo may trigger a Py_Decref on the original pyobject.

IMO that's a bit prematurate :-)

I agree.  It is not fully developed.  It wasn't turn on by default.  I just wanted to make sure it is possible.
It probably need to trigger the cleanup once awhile to avoid accumulating too many deferred dtors.

Note that the immediate followup to this work is going to be working on JIT classes, so I don't think support for retention of arrays inside a container is that premature.  We certainly want to think through that case before we commit to this approach to memory management.

Needs to check a reference count problem

Important changes since last review:

keyword arg `nrt" renamed to "_nrt" and defaults to False
Buffer object also use NRT if it has a owner object
Refct initialize to 0 for easier code generation (see https://github.com/sklam/numba/commit/e4e1270aa24635678ba929b7093634454833e44b)


Operator Template and Nufft

LinearOperator base class
Operator base class
Simple version of a FourierOperator based on the Nufft

Please add issue that Fourier operator currently treats everything as non-Cartesian and Cartesian speed-up still required.

closes #8, closes #20

Please add issue that Fourier operator currently treats everything as non-Cartesian and Cartesian speed-up still required.

The distincition between pure FFTs and NUFFTS is now implemented

Fourier operator implemented


SSL verification is Running - verification has taken longer than 8 hours

App Id: dikd6h5rpob7c
Region: us-east-1
Step: SSL verification
Status: Running
Issue/question
I added a domain, but it has been "SSL verification" for more than 8 hours.
Error message
It doesn't seem to get any errors. Please tell me if there is a screen to check other than "Amplify".
Checking www.mydomain.com at https://www.whatsmydns.net/ now returns an unstable response. It was because the NS record was changed one day ago. I think that might be the cause.
I wait a little longer.

The problem has been resolved. My DNS record management system was abnormal and I had to rewrite my NS records in two different places.


SCRIPT ERROR: @illenium-appearance/client/zones.lua:61: attempt to index a nil value (field 'job')

Hello,
I have this message when i have do the update from 5.0.1 in 5.3.0.

Can you help me please?
I can no longer access any clothing shops
Thanks you for your job !

@Edvesse use discord for support. This is a place for actual issues
https://discord.illenium.dev


NetStatistics thread safety

According to C# spec, ulong is not necessarily atomic (see https://stackoverflow.com/questions/11745440/what-operations-are-atomic-in-c). Therefore, NetStatistics cannot safely be used from a different thread than it is written to. PacketsReceived and BytesReceived are updated on the ReceiveLogic thread, PacketsSent, BytesSent, and PacketLoss are updated on the UpdateLogic thread, and Statistics are presumably read or Reset on the main thread by the application, (though technically the application could read and write them from any thread, since they are public).
To avoid thread safety issues, I've changed the type to long and modified the access of these values to all use Interlocked. I also removed the #if DEBUG block in ReliableChannel (since it is inconsistent with other usages of StatisticsEnabled), and wrapped the update of PacketLoss in SequencedChannel with if (StatisticsEnabled).
The other major change is the PacketLoss statistic is no longer set to the sum of all currently connected NetPeers (which resulted in the value decrementing on the removal of a peer, causing the PacketLossPercent to be inaccurate in a situation with regular connection and disconnection events), and instead simply increment it whenever the value for each peer is updated.

@TrevorDaschOculus  Thanks!


[test] Disable fixit_stub_editor.swift entirely until fixed.

Tracked by rdar://problem/32804941.
cc @nkcsgexi

Invoking build czar privileges to merge without testing.


Add settings for opening Duck Player in new tab

Task/Issue URL: https://app.asana.com/0/0/1208366641664416/f
Description
Steps to test this PR
Feature 1

[ ]
[ ]

UI changes

Before
After

!(Upload before screenshot)
(Upload after screenshot)

#5110
#5109  👈
develop

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @CrisBarreiro and the rest of your teammates on  Graphite


Add Express Gateway to the project

Hey — you probably remember me, we met at Node Summit and I've been speaking with Chris about featuring Express Gateway as the API Gateway for the project.
This is just a start — I'd love to iterate more with you on this.
Cheers!

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

//cc @seabaylea @BethGriggs

Latest master merged.

Hi @XVincentX. Sorry - I completely missed that you'd raised this PR.
It looks like you've raised a PR to provide an official Helm chart, which is fantastic! Does the deployed gateway support liveness, open tracing and prometheus etc? Are you doing anything to provide annotation based registration of APIs into the gateway?

We have an official Helm chart now.

ping @seabaylea

It seems like the website has been changing since my PR @gdams
I want to make sure the maintainers are going to accept the PR before rebasing.

@gdams Any update on this?


JavaScript

#JavaScript
##User Story
As a boot camp student
I want the prework notes to be structured on a webpage
So that I can easily find and read the information
##Acceptance Criteria
GIVEN  a Prework Study Guide website
WHEN I view the study guide
THEN I can see the four topics I leaned listed along with a suggestion on what I should study first

issue closed


Provide ArmiPlugin.defineSystemGrid builders

Description

Provide ArmiPlugin.defineSystemGrid builders
Goal is to let a plugin define additional sections that
can reside in the systems section, e.g.,
systems:
    core:
        grid name: core
        origin:
            x: 0
            y: 0
            z: 0
    sfp:
        grid name: spent fuel pool
        type: sfp
        ...

The end goal is to have a new system for control drums, pointing to a designated
grid, and allowing assemblies to be added to that grid rather than the core grid.
The plugins basically define a map from the system grid type (defaults to "core"
in the blueprint) and the builder function. The builder function will most likely
be the class of the thing to be built (e.g., "core" returns a Core type
and the blueprint construct method creates the instance given the name of the grid.
In the core example it's not super helpful. But for the spent fuel pool, the
type is sfp but the name is defined by the key one level up and does not
have to match any specific criteria.
The ReactorPlugin defines the existing "core" and "sfp" builders, but is
tagged to go last (or near last) in the YAML construction sequence. This allows other
plugins to define their own core builders and let those have preference over the
default.
Signed-off-by: Andrew Johnson<EMAIL_ADDRESS>Checklist

[ ] The code is understandable and maintainable to people beyond the author.
[ ] Tests have been added/updated to verify that the new or changed code works.
[ ] There is no commented out code in this PR.
[ ] The commit message follows good practices.
[ ] All docstrings are still up-to-date with these changes.

If user exposed functionality was added/changed:

[ ] Documentation added/updated in the doc folder.
[ ] New or updated dependencies have been added to setup.py.

@john-science @jakehader @ntouran any interest in this feature?

Sorry for not looking at this a long time ago.
I've been busy, and just ignore PRs with the red "X" next to them, but I see that's a minor code coverage thing here.

@ntouran Do you have an opinion?

@john-science @ntouran thank you for the feedback. I added some docs in a place that I think makes sense, but let me know if there's a better way to present this information.
In adding the docs, I found some possible shortcomings related to the sfp system documentation. The Systems blueprint documentation has content like
systems:
    core:
        grid name: core
        origin:
            x: 0.0
            y: 10.1
            z: 1.1
    sfp:
        grid name: sfp
        origin:
            x: 1000.0
            y: 12.1
            z: 1.1

which should communicate to the user that a SpentFuelPool object should be created and attached to the sfp grid. But, because the type field is missing, another Core is built instead.
Demoing this with
diff --git a/armi/reactor/blueprints/tests/test_reactorBlueprints.py b/armi/reactor/blueprints/tests/test_reactorBlueprints.py
index 6d47f5a..39f002f 100644
--- a/armi/reactor/blueprints/tests/test_reactorBlueprints.py
+++ b/armi/reactor/blueprints/tests/test_reactorBlueprints.py
@@ -19,6 +19,7 @@ import os
 from armi.reactor import blueprints
 from armi import settings
 from armi.reactor import reactors
+from armi.reactor.assemblyLists import SpentFuelPool
 from armi.reactor.blueprints import reactorBlueprint
 from armi.reactor.blueprints import gridBlueprint
 from armi.reactor.blueprints.tests import test_customIsotopics
@@ -104,6 +105,7 @@ class TestReactorBlueprints(unittest.TestCase):
         core, sfp = self._setupReactor()
         self.assertEqual(len(core), 2)
         self.assertEqual(len(sfp), 4)
+        self.assertIsInstance(sfp, SpentFuelPool)

     def test_materialDataSummary(self):
         """Test that the material data summary for the core is valid as a printout to the stdout."""

yields an assertion error
Traceback (most recent call last):
  File "/Users/ajohnson/codes/armi/armi/reactor/blueprints/tests/test_reactorBlueprints.py", line 108, in test_construct
    self.assertIsInstance(sfp, SpentFuelPool)
  File "/opt/anaconda3/envs/raptor/lib/python3.7/unittest/case.py", line 1276, in assertIsInstance
    self.fail(self._formatMessage(msg, standardMsg))
  File "/opt/anaconda3/envs/raptor/lib/python3.7/unittest/case.py", line 693, in fail
    raise self.failureException(msg)
AssertionError: <Core: sfp id:140658844990032> is not an instance of <class 'armi.reactor.assemblyLists.SpentFuelPool'>

@john-science ah that's on me. I thought, because the upstream main branch had been updated with the click changes, I wouldn't need anything on my end. But I've merged in main and it should work now 🤞

@john-science potentially very minor coverage drop. How do you want to proceed?

@john-science potentially very minor coverage drop. How do you want to proceed?

No worries. It's a 4th decimal-place drop.  I'm not bothered.

@john-science @ntouran thank you for the feedback. I added some docs in a place that I think makes sense, but let me know if there's a better way to present this information.
In adding the docs, I found some possible shortcomings related to the sfp system documentation. The Systems blueprint documentation has content like
systems:
    core:
        grid name: core
        origin:
            x: 0.0
            y: 10.1
            z: 1.1
    sfp:
        grid name: sfp
        origin:
            x: 1000.0
            y: 12.1
            z: 1.1

which should communicate to the user that a SpentFuelPool object should be created and attached to the sfp grid. But, because the type field is missing, another Core is built instead.
Demoing this with
diff --git a/armi/reactor/blueprints/tests/test_reactorBlueprints.py b/armi/reactor/blueprints/tests/test_reactorBlueprints.py
index 6d47f5a..39f002f 100644
+++ b/armi/reactor/blueprints/tests/test_reactorBlueprints.py
@@ -104,6 +105,7 @@ class TestReactorBlueprints(unittest.TestCase):
         core, sfp = self._setupReactor()
         self.assertEqual(len(core), 2)
         self.assertEqual(len(sfp), 4)
+        self.assertIsInstance(sfp, SpentFuelPool)

yields an assertion error
Traceback (most recent call last):
  File "/Users/ajohnson/codes/armi/armi/reactor/blueprints/tests/test_reactorBlueprints.py", line 108, in test_construct
    self.assertIsInstance(sfp, SpentFuelPool)
  File "/opt/anaconda3/envs/raptor/lib/python3.7/unittest/case.py", line 1276, in assertIsInstance
    self.fail(self._formatMessage(msg, standardMsg))
  File "/opt/anaconda3/envs/raptor/lib/python3.7/unittest/case.py", line 693, in fail
    raise self.failureException(msg)
AssertionError: <Core: sfp id:140658844990032> is not an instance of <class 'armi.reactor.assemblyLists.SpentFuelPool'>

So, this is the last question in this PR?  Thoughts? @ntouran ?


Resolve in Merge Editor is Shown in Output Panel

Observed by @NickCraver

@NickCarver Do you have special steps for this? Do you see this with all output channels or only the MySQL one? Can you do the following when this happens: F1 > Inspect Context Keys and then share what's printed on dev tools console

Hi there,
I think I received this email erroneously? I'm Nick Carver, but it looks
like the original msg was from a Nick Craver.
Cheers,
Nick.
On Tue, Oct 18, 2022 at 9:03 AM Nick Craver @.***>
wrote:

No specific repro steps - this happened immediately after upgrading in a
test session - will try to repo on desktop doing it and report back!
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/issues/163922#issuecomment-1281974511,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXJU4GQ7UBJ6NLX3JTOJKK3WDZKVXANCNFSM6AAAAAARHZJY6I
.
You are receiving this because you were mentioned.Message ID:
@.***>

@NickCraver any new insights here?

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!

@jrieken Apologies I was on vacation for a bit - haven't been able to repro this since the latest update landed (but also have to unfortunately rebuild the machine we saw it on for terribad Windows reasons). Recommend we close this and if I can repro, will poke with more info :)

Leaving this open since @hediet saw this once but forgot to inspect context keys (https://github.com/microsoft/vscode/issues/163922#issuecomment-1281970544)

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


⚠ Bofrost.fr has degraded performance

In f23ff55, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1924 ms

Resolved: Bofrost.fr performance has improved in 9ffe573 after 6 minutes.


Support custom grpc dial option for client connecting to datastore emulator

I was trying to use rpcreplay and datastore emulator together.  Found that rpcreplay could not intercept the grpc calls.
After looking into datastore package, it seems NewClient function dial to datastore emulator directly so extra grpc dial options in opts would not work.
This commit make NewClient dial later in the gtransport.Dial function to fix the issue.

Thanks! We don't accepts PRs to this repo but I've made your change in https://code-review.googlesource.com/c/gocloud/+/29350.


explicit colors

if all the categories are valid CSS colors and no range or scheme has been set, the scale type defaults to identity.
Ref: https://talk.observablehq.com/t/specific-coloring-for-bar-plots/5996
Note that with this PR, you can still be surprised by "wrong colors": if any of the channels has an undefined value, the colors will switch back to tableau10. The user needs to use explicit colors everywhere for it to work.
I filed a commit with unit tests and another one with the implementation, because I initially had an alternative implementation where a domain comprised only of valid CSS colors would be promoted to the default range of a categorical scale.

I need to think about this one. One question is whether it should behave more similarly to how we disambiguate temporal vs. quantitative vs. ordinal. Like, perhaps the test should be based on the first non-null channel value? If that’s a valid CSS color, then we use an identity scale (and we don’t test the rest of the values). That feels like it might be more consistent with our other behavior, but then it also feels more error prone since you could have a false positive (mostly with the named colors). And if there is going to be ambiguity, maybe it’s good to require an explicit declaration of the identity scale. I don’t think it’s necessarily a bad thing that people get confused here if it results in them reconfiguring their understanding of Plot. Anyway… like I said I want to think on this a bit.

Yes. As an alternative I thought we could build the scale as keeping the valid colors from the domain, but taking unknown from tableau10. So that if a category is "red" and another one "rabbit", the red category gets the red color, and the rabbit category gets #123456 from tableau10. It doesn't allow the user to return undefined to remove a mark (for this they will need to specify "identity"), but it seems to cover the other cases.

Closing in favor of #673. 🙏


rev task is not cleaning up after ran

I've noticed that in branch 2.0 when run gulp build:production all assets are duplicated
Fonts, css, images twice. so we have hashed files and normal files.

Fixed in #167


Don't initialize all backends when get_backend() is called with a specific backend.

This change refactors the get_backend() logic to also handle some edge cases more clearly, see the unit test changes.
One side-effect of this change not reflected in the test is that there'll be INFO logging for backend factories that return None or backends with no devices. I'm not sure if this is a good or bad thing, but I don't think it should make a huge difference either way.
Partially addresses https://github.com/google/jax/issues/7622.

This is based on https://github.com/google/jax/pull/7627, so I recommend looking at just the second commit until that goes in.

Close this PR? I think we agreed not to do this.


GLTF: use min/max values from position accessor to set the bounding box of meshes

See #4102
As an aside, how can I get rid of the first 3 commits that are displayed in this PR?
They are not related to the PR, I don't know why they are here... I guess it's because I have done them on the master branch of my local repo(?)

Ping @bghgary for second pass review :-) I ll merge it right after.

Looks all good to me let s remove the draft flag and I ll merge it in :-)
Thanks a lot !!!


AttributeError: 'module' object has no attribute 'Controller'

Getting this error while starting the newrelic-plugin-agent. What could you possible reason?
newrelic-plugin-agent -c /etc/newrelic/newrelic-plugin-agent.cfg -f
Traceback (most recent call last):
  File "/usr/bin/newrelic-plugin-agent", line 9, in <module>
    load_entry_point('newrelic-plugin-agent==1.3.0', 'console_scripts', 'newrelic-plugin-agent')()
  File "/usr/lib/python2.7/site-packages/pkg_resources.py", line 378, in load_entry_point
    return get_distribution(dist).load_entry_point(group, name)
  File "/usr/lib/python2.7/site-packages/pkg_resources.py", line 2566, in load_entry_point
    return ep.load()
  File "/usr/lib/python2.7/site-packages/pkg_resources.py", line 2260, in load
    entry = __import__(self.module_name, globals(),globals(), ['__name__'])
  File "/usr/lib/python2.7/site-packages/newrelic_plugin_agent/agent.py", line 23, in <module>
    class NewRelicPluginAgent(helper.Controller):
AttributeError: 'module' object has no attribute 'Controller'

It's because of the recent release of helper on 2nd February. I have made a PR and hopefully the library owner will merge it.
Otherwise the solution is to fork this repo and lock the helper package version in setup.py.

Another workaround I've found was to reinstall and tell pip which version of helper to use with the -c option:
pip install -c constraints.txt newrelic-plugin-agent
Where constraints.txt is a file that contains this line:
helper==4.2.4


bower:install fails with status 127

Both commands bower:install and bower:update remain in a rake abortion with status 127. Don't know what that means.
bundle exec rake bower:install's output:
bower.js files generated
bower install
rake aborted!
Command failed with status (127): [bower install...]
Tasks: TOP => bower:install
(See full trace by running task with --trace)
And because of that i can't precompile my assets.
Mistake in the code or at my side? And if it is on my side, what does the status tell us?

Im still having this same problem even with gem 'bower-rails', github: '42dev/bower-rails'


Change Timezone

Hello, what timezone does loglive use?
How do I change the timezone because I noticed the time is somehow inconsistent.

This is what we use
https://github.com/egoist/loglive/blob/5e213ded2c77fb98b7737b4ce6680e08c1ce954b/src/utils/renderer.js#L2
I think it's using the locale timezone?

Thanks for pointing that out. Although I am still confused as to how to go about changing that because I strongly believe that the time is malfunctioning
Okay, take a look at this: https://gigabytedevelopersinc.com/apps/cometotp/changelog
Under version 5.2.0, it says about 10 hours ago where it's not up to an hour I published that update.
Sometimes the time works, sometimes it doesn't. If at all it works, I have to set the time to an hour behind or an hour ahead my local time for it to look more accurate to the actual time.

Not sure if it's caused by this, but

a zero is missing at the end, it should be 2020-05-07T09:00:00

Wow. I couldn't have figured that out.
Added a zero and cleared my cache, it works. Though it's still an hour behind my local time, it's manageable anyways.
I think it's (loglive) making use of GMT +0 whereas am on GMT +1
Maybe it's something you should consider in #10


GitHub Action?

Given this is supposed to be used from GitHub action workflows (exclusively?), it doesn't seem necessary to tell all users to do npm install pkg-pr-new.
What if we could just add uses: stackblitz-labs/pkg.pr.new@v0 to a GitHub Actions workflow. e.g.
name: ci
on: [push, pull_request]
jobs:
  run:
    runs-on: ubuntu-latest
    steps:
      - uses: actions/checkout@v4
      - run: npm install
      - run: npm run build
      - uses: stackblitz-labs/pkg.pr.new@v0

Under the hood it could probably just do npx pkg-pr-new@latest anyway, but it'd mean that it'd be just one step to add to a repo - also you'd see that it's owned by stackblitz and therefore probably pretty safe to use.

I'm thinking about this, it's solid, how can we handle options at that point? like compact, templates and projects?

how can we handle options at that point? like compact, templates and projects?

      - uses: stackblitz-labs/pkg.pr.new@v0
        with:
          projects: 'packages/*'
          template: foo
          compact: true
          comment: create

Or maybe keep it simpler:
      - uses: stackblitz-labs/pkg.pr.new@v0
        with:
          args: 'packages/* --template foo --compact --comment=create'


🚀 Astro v3.3

Hi 👋!
This PR does the following:

update repo links in README
change port from 3000 to 4321
add .nvmrc
remove @astro/image and replace usage with astro:assets
update integrations
AND most importantly: updates Astro to v3.3 🥳

Reference: https://x.com/srbhr/status/1712492466270769266?s=46
Netlify Deployment

Wow, thanks a lot @xmok, for this PR. I'll test it and will merge it!


Add conformance test for HTTPRoute exact path matching

What type of PR is this?
/ kind conformance-test
What this PR does / why we need it:
Add tests for matching exact path for HTTPRoutes.
Which issue(s) this PR fixes:
Updates https://github.com/kubernetes-sigs/gateway-api/issues/1102.
Does this PR introduce a user-facing change?:
NONE

Thanks!
/lgtm
/approve


51: Expand Morning to 8am

https://github.com/my-badges/my-badges/issues/51#issuecomment-1943506195

I think 8am is still a morning)

I think 8am is still a morning)

Should we leave it?

I'll leave the choice to you. As you decide, so be it. :)


dockerhub proxy not woring

I've recently noticed that some of the image resources are not available, is there a problem?

docker pull dockerhub.azk8s.cn/library/openjdk:11-jdk-slim-bullseye

I also have this issue pulling images from here.

I also see several images apparently missing from the proxy.  It seems to affect only certain versions and is also intermittent as some images used to be available some time ago.
For example:
dockerhub.azk8s.cn/bitnami/contour:1.29.2-debian-12-r0 ✅ works
dockerhub.azk8s.cn/bitnami/contour:1.30.0-debian-12-r6 ❎ not found
gcr.azk8s.cn/datadoghq/cluster-agent:7.56.2 ❎ today: not found
gcr.azk8s.cn/datadoghq/cluster-agent:7.56.2 ✅ 35days ago, worked
gcr.azk8s.cn/datadoghq/agent:7.56.2-jmx ❎ today: not found
gcr.azk8s.cn/datadoghq/agent:7.56.2-jmx ✅ 35days ago: worked

Hi,
The dockerhub in China is blocked by GFW, so that’s the main reason why you cannot pull those images. You could push those images to ACR and then use ACR on mooncake clusters, thanks.

Surry @andyzhangx but we are trying to pull the images from the mirror in china which is mentioned in this repository and not directly from dockerhub. But the images cannot be found in the china registry mirror. Can you tell me who is responsible for pushing the images to the miror?


⚠ Bofrost.lu has degraded performance

In 66801e8, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1420 ms

Resolved: Bofrost.lu performance has improved in 2259ef5 after 9 minutes.


[BUG] The on_empty_queue event did not work

Bug report
Describe the bug
Yesterday started preparatory work to modify the source code to create Pull Request, flashed for the correct config 2023.9, specified as in the documentation:
  on_empty_queue:
    then:
      lambda: |-
        id(rgb8x32)->clock_screen(10,10);
        id(rgb8x32)->date_screen(10,5);

  on_start_running:
     then:
       lambda: |-
          id(rgb8x32)->clock_screen(10,10);
          id(rgb8x32)->date_screen(10,5);

Also from HA, the weather is constantly displayed on the screen. I.e. constant 3 screens, clock, date, weather (updated often because it is tied not only to the weather, but also to the temperature).
But this morning, there was only weather on the screen, no clock or date. It seems that the on_empty_queue event did not happen.
Additional information

used Hardware:

ESP32
EspHoMaTriXv2 version: 2023.9.0

This night a had a similar effect :-(

The on_empty_queue event should not happen. The queue was not empty. There was a screen in the queue.

That's understandable, what's not so clear is where the date and time disappeared to.

They don't disappear. They are created with 10minutes of live.

I see two possible ways for this.

A second queue for permanent screens which don't have a live time.
Permanent screens should only be created within the clock(pulling data)

Well, we basically have no screens that are permanent, since we already have at least two types of watches.
Here it is necessary either to constantly extend the life of the screen,
    this->endtime = this->clock->now().timestamp + 300;

or to set the minimum starting set, which is immortal.

What is the differenz between a permanent screen and a immortal one?

Basically nothing, but we need to understand where the temporary screen is and where it is not, and when temporary can become permanent or vice versa.

We could try replacing the on_empty_queue event with on_expired_screen, then as soon as the clock becomes obsolete it can be returned :)

How do we know which screen expired?

How do we know which screen expired?

Judging by the source code, the information about the screen is transmitted there, but we can not analyze it, just increase the life of the necessary screens, even by an hour, even by a day.

But if we have a weather screen running all the time even a life of a day will run out and the on_empty_screen should never be called.
What a about a on_screen_death event. So we can put a function as a parameter. This would even be a possibility to make it more generic. So a timer for example would make a sound on screen death.

Besides the clock I don't understand the concept of permanent screens. For me, e.g. the wind speed is pushed with a lifetime of two minutes to the display by an automation that triggers on every state change. So the wind speed is "permanent" on the screen but totally controlled by home assistant. If you need something like a "motto of the day" set the lifetime to 24*60 minutes triggers by an automation on home assistant at 12p.m..

I have:
  on_empty_queue:
    then:
      lambda: |-
        id(rgb8x32)->clock_screen(1440,10);
        id(rgb8x32)->date_screen(1440,5);

And weather and temperature from HA, trigger state changes, lifetime 30 min
And a day, two days later, all I have is the weather on my screen. No clock or time

The problem can also happen without any "permanent" screens. Ad some point in time the clock screen dies. An if then any screen is in the queue the clock is gone.
And only if all screens are removed we get a trigger to renew the clock. So if we have homeassistant add 4 10min screens for fun things in the meantime we don't have the time on the clock.


Feature: New method for converting collections/objects to strings

In another project of mine, I tried to implement a simple string template engine through ObjectPath, but I found that I couldn't create legal labels for each element in the collection object.
The code of the template engine is very simple, he extracts the objectpath in ${objectpath} with a regular expression and parses it into a string.
import cc.catman.object.ObjectPath;

import java.util.regex.Matcher;
import java.util.regex.Pattern;

public class ObjectPathTemplate {

    private String regex="\\$\\{(.*?)\\}";

    public String format(String template,Object context){
        Pattern pattern = Pattern.compile(regex);
        Matcher matcher = pattern.matcher(template);
        return matcher.replaceAll((matchResult) -> {
            String expr = matchResult.group(1);
            return ObjectPath.of(expr).eval(context, String.class);
        });
    }
}

Refer to the following use case:
String template= "
${@.names[*].map(''+@+'')}
";
Map<String, ?> context = Map.of("names", Arrays.asList("a", "b", "c"));
ObjectPathTemplate opt = new ObjectPathTemplate();
String format = opt.format(template, context);
System.out.println(format);

The value I expect to get should be:

a  b  c

But the value I actually get is.

[a, b, c]

Obviously, [] in a div is not what I need, and there should be no comma between span tags.
So, I decided to provide a new function that converts collections/objects to strings, and also supports specifying the behavior of the converted string.
.toString(expr,expr,expr,expr)

The function takes up to four arguments, each of which is optional. If there are no arguments, it will trigger the default behavior of java.
The four parameters are, in order.

can be accessed through @ to the elements of the collection, the expression needs to return a string, if the return value is not a string, it will automatically be converted to a string
provide a delimiter to connect the strings in the collection, if no delimiter is provided, it will be filled with null by default.
provide a string, the string will be spliced to the beginning of the return value.
provide a string that will be spliced to the end of the return value.

After adding the method, I can use the following expression to achieve the desired effect:

${@.names.toString(''+ @.value +'')}

output content:

abc

#26


PowerDNS-Admin Docker with nginx proxy

Hi All,
I followed the following tutorial: https://computingforgeeks.com/running-powerdns-and-powerdns-admin-in-docker-containers/
I am using the docker-compose example and it works like a charm as I can access the PowerDNS url via port 8080.
As I have several containers, I decided to use nginx installed on the server as a reverse proxy to proxy the various applications:
https://server.domain.com/pdnsadmin/
https://server.domain.com/portainer/
My nginx reverse proxy to the PowerDNS-Admin container does not work at all (Error 404). I have tried several different configurations, even the configuration in the tutorial, but none seem to work. For now, I am just trying the most basic of configuration to remove complexity, hence no SSL, below is my current config:
upstream pdnsadmn {         server <IP_ADDRESS>:8080; }
`location /pdnsadmin/ {
                    #proxy_set_header Host $host;
                    #proxy_set_header X-Real-IP $remote_addr;
                    #proxy_set_header X-Forwarded-For $proxy_add_x_forwarded_for;
                    #proxy_set_header X-Forwarded-Proto $scheme;
                    proxy_pass http://pdnsadmn;
                    #index  index.html index.htm;
                    proxy_buffering off;
                    proxy_set_header X-Real-IP $remote_addr;
                    proxy_set_header X-Forwarded-Host $host;
                    proxy_set_header X-Forwarded-Port $server_port;
    }`

Is there perhaps someone that can assist me with the nginx reverse proxy config for the PowerDNS-Admin Docker container.
Thank-you very much

                proxy_pass http://pdnsadmn;

Maybe try this:
proxy_pass http://pdnsadmn/;
I think what happens without a trailing slash is that the path /powerdns/ is passed into the proxied service, and powerdns-admin doesn't know about a path /powerdns/.
As far as I know, adding a trailing / to the proxy_pass statement mitigates this.

Given that the OP hasn't responded in some time, I'm closing this question as resolved.


System tests dependency cleanup

Removes dependency to wasmd in system tests. Not needed as tests use the binary
After #1479

@mergifyio rebase


joined and unjoined corner don't have the same stroke result

"Joined" is yellow in the screenshots
By joined and unjoined in source I mean:

And these are different results after export:

joined looks better

unjoined looks better

In any case, whatever the points are joined are not, it should be the same result, right?

Sorry Rosalie, I'm struggling a bit to understand this issue, by unjoined you really mean two paths which intersect or nearly intersect, right?
No, you should not expect that "unjoined" result should be the same, especially if the end points of the "unjoined" paths are not exactly equal. Because, MFEKstroke will draw a cap, not a join.
I recommend pre-processing to forcably join all "near joins" at path terminals.

By unjointed I mean the two points share the exact same coordinates, but the path is open, not closed. So intersect may be the right term then.

Thanks a lot for your help on this 🙏

Now that I'm maintainer, I make the following observations:

We already have circle caps;
Where the circle joins don't work right, the option --segmentwise I propose in #16 will fix this.

Therefore, closing as duplicate of #16 soon to be itself closed by --segmentwise.

I'd like to request reopening this issue; @RosaWagner chatted with me today about releasing "Relief" (https://github.com/google/fonts/issues/3271) and currently this stroker still isn't useful for that release :) I'd like to offer to host a call so we 3 can discuss it.

@davelab6 I'm only seeing this now for some reason—sorry—that I'm looking into #27. Is your comment different than the issue at https://github.com/MFEK/math.rlib/issues/27#issuecomment-1104605712 ?


Fix ocamlformat analysis limiting the other jobs

Attempt to fix #893, instead of failing on the analysis, the fmt job is spawned and the error is notified in it (suggestion from @Julow).

This works if the .ocamlformat file has no version= line 👍.
However, if the version line contains an incorrect version — such as version=0.99.1 — then the (lint-fmt) job is not spawned as no analysis solution is found. @moyodiallo do you think it'd be good to also have this case propagated as an error to a (lint-fmt) job? We'd have to be careful to do this only when the .ocamlformat file is present, so that we don't fail CI for projects without formatting.

However, if the version line contains an incorrect version — such as version=0.99.1 — then the (lint-fmt) job is not spawned as no analysis solution is found. @moyodiallo do you think it'd be good to also have this case propagated as an error to a (lint-fmt) job? We'd have to be careful to do this only when the .ocamlformat file is present, so that we don't fail CI for projects without formatting.

Yeah I think you're right it is logic, when we have .ocamlformat file present in a project, the purpose is to format the project. And also (lint-fmt) job is always spawned.

@benmandrew  Is this OK to your suggestion ?

I'm not sure if the behaviour's changed or I missed it before, but if the .ocamlformat file doesn't have a version= line then the job fails in the extract stage with:
2023-11-29 13:22.17: Exec: "git" "ls-files" "**/.ocamlformat-enable"
                           ".ocamlformat-enable"
2023-11-29 13:22.17: Job failed: Missing 'version=' line in .ocamlformat

My test repo/branch is https://github.com/benmandrew/ci-test-repo/tree/ocamlformat-no-version.

I think you're using the master branch, not the PR one.

My mistake, I was looking at an old job! Feel free to merge.


Commit for PopOverMenuItem3

Was not done earlier, so new component

The issue related to the component was closed by mistake previously. Now it is created.
Ducky-components\PopOverItemMenu3


Microsoft Graph Change Notifications Failing

Intermittently facing issue such as not receiving webhook notification or sometimes webhook subscription is not working i.e. getting subscription request timed out error.
In our setup, we have blocked all inbound traffic and whitelisted only those IPs that are mentioned in the following page:
https://docs.microsoft.com/en-us/microsoft-365/enterprise/additional-office365-ip-addresses-and-urls?view=o365-worldwide
I have listed down few of the IPs which seems to be from Microsoft that are being blocked by the firewall.
<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS>
and more...
It there any change in the list of IPs?

@ivanmaria Thank you for your feedback. We are currently checking these details with Author and PG and will update you shortly.
Thanks
Sri

Hi @msbemba, awaiting for the update.

@kelleyvice-msft  Please add your comment and help us in resolving this issue. Thanks.

Hi @kelleyvice-msft, I am waiting to hear from you. Thanks

@ivanmaria It usually takes a few days to verify and confirm this information with the product group. We have informed the author and you will receive an update once it is made available to us.
Thanks
Sri

@ivanmaria, sorry for the delay; I've been on a leave of absence since March. I'm sure you've figured out how to resolve your issue by now, but if not, here's some information that should help.
The article you reference only contains endpoints that are not already provided by the IP and URL web service. To get the most current list of endpoints you'll need to add to your allow list, you can either use the web service (https://docs.microsoft.com/en-us/microsoft-365/enterprise/microsoft-365-ip-web-service?view=o365-worldwide) or the topic https://docs.microsoft.com/en-us/microsoft-365/enterprise/urls-and-ip-address-ranges. While it is possible you will also need to add endpoints from the topic you referenced, using the methods I outlined above should resolve your issues.
Kelley


CU-85zrn4z99_Auth-register_leonardo-salas

merger auth registration with the main branch

Task linked: CU-85zrn4z99 Auth register


[FINE] Adaptations to have a working refresh for Hawkular's Inventory.v3

This is the same work as ManageIQ/manageiq-providers-hawkular#10 but tailored for fine branch.
If ManageIQ/manageiq-providers-hawkular#10 cannot be backported due to conflicts, please merge this PR.
@miq-bot add_label providers/hawkular
\cc @abonas @pilhuhn

LGTM

@israel-hdez please take a look at Travis failure.

@simaishi well.. as expected, the errors are on the cassettes:

An HTTP request has been made that VCR does not know how to handle:
GET http://jdoe:password@<IP_ADDRESS>:8080/hawkular/inventory/status

As i said in https://github.com/ManageIQ/manageiq-gems-pending/pull/138#issuecomment-297736552, those errors won't go away until fine the other PRs are backported.

@israel-hdez Ah, thanks.  I remember seeing comment about VCR somewhere 😄
@abonas are you good with this PR?

should there be a change also in middleware_deployment.rb ?
making sure because the original PR had this

@abonas Yes, the equivalent change is here: https://github.com/ManageIQ/manageiq/pull/14927/files#diff-4acd9f4e59821ea6f157f0b901ecc670R183

LGTM

This should be https://bugzilla.redhat.com/show_bug.cgi?id=1446329 then (I guess)


Add type / property to identify entities that have map views parsed with TMXLoader

Cannot see the use case for this any more.


ValidationAssist.Suppress does not affect textbox underline

I am using the following style to suppress validation on disabled textboxes.
    <Style x:Key="DefaultTextBox" TargetType="{x:Type TextBox}" BasedOn="{StaticResource MaterialDesignTextBox}">
        <Style.Triggers>
            <!-- Remove the errors when the control is disabled. -->
            <Trigger Property="IsEnabled" Value="False">
                <Setter Property="materialDesign:ValidationAssist.Suppress" Value="True"/>
                <!--<Setter Property="BorderBrush" Value="{StaticResource MaterialDesignTextBoxBorder}"/>-->
            </Trigger>
        </Style.Triggers>
    </Style>

Here is the result of that:

The red underline is still applied to the disabled textbox. The Error text is correctly hidden.
I worked around this by using the commented out setter for the BorderBrush. This is mostly just to make the behavior consistent with other controls. The Combobox does not require the workaround. I haven't tested any other controls either so it may be worth checking each one.

@ButchersBoy what is the expected behavior of the ValidationAssist.Suppress property. In the code there is a comment that this is for framework use only.
Should the various styles be updated to respect this property and not set the ValidationErrorBrush when suppress is true?


Curl success, but Internal Server Error when commit transaction

bigchaindb-driver version: 0.6.2
bigchaindb  version: 2.0.0b9
Python version: 3.6.8
Operating System: Virtual machine of Ubuntu 16, through Window 10 host

Description
I set up bigchaindb, with python driver and mongodb. Then, I try to run the code from Basic Usage Example, and receive Internal Server Error, but when I run curl http://localhost:9984/api/v1/transactions, it shows {"message": {"asset_id": "Missing required parameter in the JSON body or the post body or the query string"}}, which means that my connection is established.
How can I debug form here?
What I Did
from bigchaindb_driver import BigchainDB
from bigchaindb_driver.crypto import generate_keypair
from time import sleep
from sys import exit

bdb_root_url = 'localhost:9984'
# bdb_root_url = 'http://<IP_ADDRESS>:9984'
bdb = BigchainDB(bdb_root_url)

bicycle_asset = {
    'data': {
        'bicycle': {
            'serial_number': 'abcd1234',
            'manufacturer': 'bkfab'
        },
    },
}

bicycle_asset_metadata = {
    'planet': 'earth'
}

alice, bob = generate_keypair(), generate_keypair()

prepared_creation_tx = bdb.transactions.prepare(
    operation='CREATE',
    signers=alice.public_key,
    asset=bicycle_asset,
    metadata=bicycle_asset_metadata
)

fulfilled_creation_tx = bdb.transactions.fulfill(
    prepared_creation_tx,
    private_keys=alice.private_key
)

sent_creation_tx = bdb.transactions.send_commit(fulfilled_creation_tx)

txid = fulfilled_creation_tx['id']

print("tid: " + txid)

asset_id = txid

transfer_asset = {
    'id': asset_id
}

output_index = 0
output = fulfilled_creation_tx['outputs'][output_index]

transfer_input = {
    'fulfillment': output['condition']['details'],
    'fulfills': {
        'output_index': output_index,
        'transaction_id': fulfilled_creation_tx['id']
    },
    'owners_before': output['public_keys']
}

prepared_transfer_tx = bdb.transactions.prepare(
    operation='TRANSFER',
    asset=transfer_asset,
    inputs=transfer_input,
    recipients=bob.public_key,
)

fulfilled_transfer_tx = bdb.transactions.fulfill(
    prepared_transfer_tx,
    private_keys=alice.private_key,
)

sent_transfer_tx = bdb.transactions.send_commit(fulfilled_transfer_tx)

print("Is Bob the owner?",
    sent_transfer_tx['outputs'][0]['public_keys'][0] == bob.public_key)

print("Was Alice the previous owner?",
    fulfilled_transfer_tx['inputs'][0]['owners_before'][0] == alice.public_key)

My stacktrace
vagrant@ubuntu-bionic:/vagrant$ python3 main.py
Traceback (most recent call last):
  File "main.py", line 79, in <module>
    sent_creation_tx = bdb.transactions.send_commit(fulfilled_creation_tx)
  File "/usr/local/lib/python3.6/dist-packages/bigchaindb_driver/driver.py", line 375, in send_commit
    headers=headers)
  File "/usr/local/lib/python3.6/dist-packages/bigchaindb_driver/transport.py", line 82, in forward_request
    backoff_cap=backoff_cap,
  File "/usr/local/lib/python3.6/dist-packages/bigchaindb_driver/connection.py", line 91, in request
    **kwargs,
  File "/usr/local/lib/python3.6/dist-packages/bigchaindb_driver/connection.py", line 128, in _request
    raise exc_cls(response.status_code, text, json, kwargs['url'])
bigchaindb_driver.exceptions.TransportError: (500, '{"message": "Internal Server Error"}\n', {'message': 'Internal Server Error'}, 'http://localhost:9984/api/v1/transactions/')

vagrant@ubuntu-bionic:/vagrant$ curl http://localhost:9984
{"api":{"v1":{"assets":"/api/v1/assets/","blocks":"/api/v1/blocks/","docs":"https://docs.bigchaindb.com/projects/server/en/v2.0.0b9/http-client-server-api.html","metadata":"/api/v1/metadata/","outputs":"/api/v1/outputs/","streams":"ws://localhost:9985/api/v1/streams/valid_transactions","transactions":"/api/v1/transactions/","validators":"/api/v1/validators"}},"docs":"https://docs.bigchaindb.com/projects/server/en/v2.0.0b9/","software":"BigchainDB","version":"2.0.0b9"}

vagrant@ubuntu-bionic:/vagrant$ curl http://localhost:9984/api/v1/transactions
{"message": {"asset_id": "Missing required parameter in the JSON body or the post body or the query string"}}

I have the same problem. Have you solved it now？

@LuffySmile Have you solved it

@LuffySmile @xiaowei6688 Have you solve it? I have same issue here


Custom CA certificate disappears in Windows Container

We are trying to import CA certificate to Cert:/LocalMachine/AuthRoot during application start. The import is successful, however after 35mins~40mins, the certificate is gone from Cert:/LocalMachine/AuthRoot.
At the beginning there are 2 built-in certificates, after import there are 3. But after 35mins~40mins you will find only built-in certificates left, the custom CA certificate is gone.
The code to reproduce:
class Program
    {
        static void Main(string[] args)
        {
            ListCerts();
            var  cert = new X509Certificate2(Path.Combine(AppDomain.CurrentDomain.BaseDirectory, "ca.crt"));
            using (var store = new X509Store(StoreName.AuthRoot, StoreLocation.LocalMachine))
            {
                store.Open(OpenFlags.ReadWrite);
                store.Add(cert);
            }
            while (true)
            {
                ListCerts();
                Thread.Sleep(TimeSpan.FromSeconds(30));
            }
        }

        static void ListCerts()
        {
            using (var store = new X509Store(StoreName.AuthRoot, StoreLocation.LocalMachine))
            {
                store.Open(OpenFlags.ReadOnly);
                Console.WriteLine(DateTime.UtcNow.ToString());
                foreach (var item in store.Certificates)
                {
                    Console.WriteLine("{0} {1}", item.Thumbprint, item.Subject);
                }
            }
        }
    }

Dockerfile:
FROM mcr.microsoft.com/dotnet/framework/runtime:4.7.2-20200714-windowsservercore-ltsc2019
COPY . /app/
ENTRYPOINT ["/app/certtest.exe"]

ca.crt:
ca.zip
docker(Windows Server 2019):
docker run -it --rm certtest:v1

Thanks for bringing this to our attention @zhiweiv.
We have created an internal bug and the team will investigate this issue.

We raised an Azure Support ticket<PHONE_NUMBER>01168, MS support reproduced same issue in his environment. Now he stated the case has been submit to PG team for further investigation.

Can reproduce by PowerShell too:
cert.ps1
Import-Certificate -FilePath ca.crt -CertStoreLocation Cert:/LocalMachine/AuthRoot
cd Cert:/LocalMachine/AuthRoot;
while($true)
{
	Get-Date
	dir;
	Start-Sleep 60;
}

Dockerfile:
FROM mcr.microsoft.com/dotnet/framework/runtime:4.7.2-20200908-windowsservercore-ltsc2019
WORKDIR /app
COPY . /app/
ENTRYPOINT ["Powershell", "-File", "cert.ps1"]

Command:
docker build -t certtest:v1 .
docker run -it --rm certtest:v1

@vrapolinario
Any update from from your side?

@weijuans-msft as @vrapolinario is on leave.

Apologies for the delay in providing an updated on this one. We're still investigating it internally.

@vrapolinario
Any update? It is still a problem for us.

Hi @zhiweiv, I'm so sorry for the delay.
The team did respond and noted that this is an expected behavior, since the recommendation is to not install on the AuthStore as that is managed by the auto root update mechanism. The recommendation is to install it on Cert:/LocalMachine/Root.
Can you please validate that for your application?

Cert:/LocalMachine/Root works for me, thanks.


Tracking of ignored files showing false positives.

Received notice that
Ignored files found that are tracked: .ignore plugin found some files that are tracked with Git which are also listed in .gitignore file.
However, the file being highlighted is gradle-wrapper.jar which is in my .gitignore file as this:
# Avoid ignoring Gradle wrapper jar file (.jar files are usually ignored)
!gradle-wrapper.jar

Earlier I also noticed that log4j2.properties file which is in my build folder was similarly being marked. It is of course included in git in the main project, and my build folder is added to .gitignore, so this rule seems a little confused.

I noticed very similar behaviour across several projects. In one case, my project root (directory!) is being detected as being tracked and ignored at the same time by the plugin. Obviously, my project root directory is not included in .gitignore.

@akomelj Thank you for your report!
When displaying those kind of informations, I'm depending on the following git command:
git ls-files --ignored --exclude-standard

performed in each Git root directory.
This command provides me a list of the files (relative paths to the git-root) that are listed in .gitignore file AND tracked by git (so, they were added to Git repo before a rule was added to the .gitignore).
Can you plase execute this command in your git roots and provide me the output to help on the issue investigation?

$ git ls-files --ignored --exclude-standard
gradle/wrapper/gradle-wrapper.jar
$ cat .gitignore
...
# Avoid ignoring Gradle wrapper jar file (.jar files are usually ignored)
!gradle-wrapper.jar
...

Interesting, I thought I'd try something quickly - and apparently its all git's fault.
If I edit my .gitignore I can avoid this issue...
Previously:
$ cat .gitignore
...
# Avoid ignoring Gradle wrapper jar file (.jar files are usually ignored)
!gradle-wrapper.jar
...
# Package Files #
*.jar
*.war
*.ear
*.zip
*.tar.gz
*.rar

...

Edited, I end up with this:
$ cat .gitignore
...
# Package Files #
*.jar
*.war
*.ear
*.zip
*.tar.gz
*.rar

# Avoid ignoring Gradle wrapper jar file (.jar files are usually ignored)
!gradle-wrapper.jar

...

Previously the Avoid line was placed before the *.jar line
So the order is apparently very important
(Hope that makes sense)

@hsz
The provided command does not list HWE directory (supposedly ignored & tracked):
netek:epay-hwe andrej$ git ls-files --ignored --exclude-standard
MODULES/business_portal/src/main/resources/static/components/qr/.DS_Store
MODULES/issuer_portal/src/main/resources/static/css/.DS_Store
MODULES/modules-furs/src/test/resources/.DS_Store
MODULES/sessionmodule/web/src/main/webapp/META-INF/MANIFEST.MF
MODULES/sessionmodule/web/src/main/webapp/META-INF/context.xml
MPS/StatusImporter/.factorypath
WEBAPP/overlays/rs/ebills/admin/src/main/webapp/static/js/views/.gitignore
WEBAPP/overlays/rs/ebills/portal/src/main/webapp/static/js/views/.gitignore

.gitignore in the root of Git repo and in HWE project directory (direct descendant of the root of Git repo) also does not list HWE as excluded:
netek:epay-hwe andrej$ cat .gitignore
# Compiled source #
target/
build/
bin/
.metadata
.settings/
.apt_generated/
.project
.classpath
.factorypath
.idea/
.codepro/
*.iml
/log/
*.log
pom.xml.versionsBackup
netek:epay-hwe andrej$ cat HWE/.gitignore
cat: HWE/.gitignore: No such file or directory

Since my project root HWE is not in the root of Git repo - is it possible that your plugin processes Git output and wants to display descendants of HWE which are not listed and displays only empty project root directory as a result?
Plugin dialog screenshot:

before press "add unversioned files": https://habrastorage.org/files/559/1e1/17a/5591e117a1374d34a56f4718a670e38b.png
after press: https://habrastorage.org/files/9bb/243/ab5/9bb243ab5287461c8557eeff4d9dcdeb.png
command "git ls-files --ignored --exclude-standard" no output

I'm having the same issue:

.gitignore:
.idea
!.idea/codeStyleSettings.xml

@SUPERCILEX I had the exact same match coming up.
To work around the issue, I changed .idea to .idea/**/*.*
It matches everything inside the .idea folder without having to match on a directory level, which apparently causes git to trip up.

@HeadOnAPlate Thanks for the tip! .idea/**/*.* didn't work for me, but this did: .idea/**. That's weird that git can't do this for us. Anyway, thanks!

I've seen this problem elsewhere: https://github.com/joeblau/gitignore.io/issues/178
It does appear that order matters in the .gitignore file.  Is there a way we could separate the exclusions from templates and include them at the end of the file, maybe after some marker where newly added templates would be placed?  Or maybe have the inclusive portion of templates added to the top of the file and exclusionary portions of templates added to the end?

I'm experiencing false positives on several projects where the plugin is reporting that it found files which are tracked in Git but are listed in .gitignore, yet the "list of commands to be performed with untrack operation" is empty, and git ls-files --ignored --exclude-standard also does not produce any results.

@rscorer @akomelj @bamapookie @SUPERCILEX @HeadOnAPlate @bamapookie @pbienick
Thank you guys for this feedback.
I finally was able to resolve this issue (mostly with reimplementing the indexing part). The main issue that caused this bug was wrongly handled files excluding with ! at the beginning (it was commited to #350).
This fix will be released with v2.0.0 as soon as I'll take care of the other issues (I'm back on track for last 2 weeks).
CC @cjbarth @pvannuff @bamapookie @Hronom


gunicorn - RuntimeError: Working outside of application context.

In a fresh setup from trunk, using gunicorn,
superset does not work, not even the main page.
instead "superset run" works ok.
Command Line:
gunicorn
-w 8
-k gevent
--timeout 120
-b  <IP_ADDRESS>:5000
--limit-request-line 0
--limit-request-field_size 0
superset:app
Error when trying http://<IP_ADDRESS>:5000/
[2019-12-01 12:08:34 +0100] [11510] [ERROR] Error handling request /
Traceback (most recent call last):
File "/home/user/software/venvSS/lib/python3.6/site-packages/gunicorn/workers/base_async.py", line 55, in handle
self.handle_request(listener_name, req, client, addr)
File "/home/user/software/venvSS/lib/python3.6/site-packages/gunicorn/workers/ggevent.py", line 143, in handle_request
super().handle_request(listener_name, req, sock, addr)
File "/home/user/software/venvSS/lib/python3.6/site-packages/gunicorn/workers/base_async.py", line 106, in handle_request
respiter = self.wsgi(environ, resp.start_response)
File "/home/user/software/venvSS/lib/python3.6/site-packages/werkzeug/local.py", line 376, in
call = lambda x, *a, **kw: x._get_current_object()(*a, **kw)
File "/home/user/software/venvSS/lib/python3.6/site-packages/werkzeug/local.py", line 307, in _get_current_object
return self.__local()
File "/home/user/software/venvSS/lib/python3.6/site-packages/flask/globals.py", line 52, in _find_app
raise RuntimeError(_app_ctx_err_msg)
RuntimeError: Working outside of application context.
This typically means that you attempted to use functionality that needed
to interface with the current application object in some way. To solve
this, set up an application context with app.app_context().  See the
documentation for more information.
[2019-12-01 12:08:35 +0100] [11508] [ERROR] Error handling request /favicon.ico
Traceback (most recent call last):
File "/home/user/software/venvSS/lib/python3.6/site-packages/gunicorn/workers/base_async.py", line 55, in handle
self.handle_request(listener_name, req, client, addr)
File "/home/user/software/venvSS/lib/python3.6/site-packages/gunicorn/workers/ggevent.py", line 143, in handle_request
super().handle_request(listener_name, req, sock, addr)
File "/home/user/software/venvSS/lib/python3.6/site-packages/gunicorn/workers/base_async.py", line 106, in handle_request
respiter = self.wsgi(environ, resp.start_response)
File "/home/user/software/venvSS/lib/python3.6/site-packages/werkzeug/local.py", line 376, in
call = lambda x, *a, **kw: x._get_current_object()(*a, **kw)
File "/home/user/software/venvSS/lib/python3.6/site-packages/werkzeug/local.py", line 307, in _get_current_object
return self.__local()
File "/home/user/software/venvSS/lib/python3.6/site-packages/flask/globals.py", line 52, in _find_app
raise RuntimeError(_app_ctx_err_msg)
RuntimeError: Working outside of application context.
This typically means that you attempted to use functionality that needed
to interface with the current application object in some way. To solve
this, set up an application context with app.app_context().  See the
documentation for more information.

Try
gunicorn
-w 8
-k gevent
--timeout 120
-b <IP_ADDRESS>:5000
--limit-request-line 0
--limit-request-field_size 0
"superset.app:create_app()"
instead

Taking a look.


arccheck

Hi Samuel,
Thank you for the great work you performed.
Will you continue to develop this lib ?
Tks
Oliv

Give some points how to develop MADD ..
tks


feat(storage, monitoring): mmap files stats

Pull Request type
Please check the type of change your PR introduces:

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no API changes)
[ ] Build-related changes
[ ] Documentation content changes
[ ] Other (please describe):

What is the current behavior?

Issue Number: N/A
What is the new behavior?

Does this introduce a breaking change?

[ ] Yes
[ ] No

Other information

This change is

crates/papyrus_storage/src/mmap_file/mod.rs line 173 at r1 (raw file):

    offset: usize,
) -> MmapFileResult<(FileHandler<V, RW>, FileHandler<V, RO>)> {
    let file = OpenOptions::new().read(true).write(true).create(true).truncate(false).open(path)?;

Fixing a clippy error
Code quote:
truncate(false).


Anthony  toggle show password

Details
Added a toggle button icon to password fields to change visibility
Fixed Issues
https://github.com/Expensify/App/issues/2734
QA Steps
Web and Desktop:

Type something into a password field
Verify password is hidden
Click eye icon
Verify the password is visible
Click again
Verify it is hidden again

Android and iOS

Type something into a password field
Verify password is shown
Click eye icon
Verify the password is hidden
Click again
Verify it is shown again

Tested On

[x] Web
[x] Mobile Web
[x] Desktop
[x] iOS
[x] Android

Screenshots
setpassword screen web:
https://user-images.githubusercontent.com/47184118/129957727-cff28fb8-f9f5-4d6c-9d69-cd469e2e74e7.mp4
Edge

note the lack of vendor specific show password functionality
Mobile Web

Desktop
https://user-images.githubusercontent.com/47184118/133485179-aa45ff31-6457-4865-a207-6eff4177ae66.mov
iOS

Android

I have yet to upload all the proof of testing as I have an outstanding implementation question in the PR

Where is the question?

it is inside the review I started. you need to click view changes:

@johnmlee101 can you see my review now?

Yes!

I just have to fix what broke in the merge

@johnmlee101 can you please review this PR?

As mentioned this will be on hold, but I'll still review!

there are some bugs introduced in the input field after merging main:
https://expensify.slack.com/archives/C01GTK53T8Q/p1633367678385600?thread_ts=1633367592.384900&cid=C01GTK53T8Q
It was working on all platforms before merging

https://github.com/Expensify/App/issues/5632
this is the related issue

don't need to apologise, family come first 👍 I appreciate the feedback as well.
I need to push the input element downwards but keep the other element in the row in place so I will use something apart form padding to displace it down.
I'll look at the PR you linked and fix it. I'm going on holiday today so I won't get back to this until the 10th Nov.

Any update on this @anthony-hull . I can say you are close.

I'm working on this over the next 24 hours

vs

About the spacing between the input and the icon, what do we think?

Spacing should symmetrical around the icon. So that icon would fit in a rect.

But equal space around the icon on x axis.

in the image you chose the spacing between the input element and the eye is 11.5px and the spacing between the border and the eye is also 11.5px

I was thinking to make it so that the icon looks like a square. But we need more eyes cc: @shawnborton

Hmm I'm a little lost regarding what's being asked of me here, what do you mean more of a square?

I was thinking if we increase the space around the icon horizontally then it will look better. But I tagged you to ask about this https://github.com/Expensify/App/pull/4746#issuecomment-978071261 @shawnborton

Ah, got it. I like the mockups where there is more space between the 1Pass icon and the eye. Maybe we can just use 8px gap between those two icons and see how it looks?

Let's go with the top version.

Just need to test on all platforms  but something has come up this evening. I'll be able to carry on tomorrow

Is this ready for review?

It works on Web, Mobile Web and Desktop. I can't test on the native apps at the moment as they crash due to another issue

The crash is fixed now. You can pull new changes and test.

oh man tired of fixing breaking changes from merges 😆

@parasharrajat testing complete, please review

I hate to say this but overflow issue is back

Just an overflow issue again and a few tweaks. I think that the current implementation could be improved by using an absolute positioned icon over the input. But as this is working fine, I won't ask you to do that unless you are willing to do that change.

If it were absolute, wouldn't any password manager icons render underneath it? as the input element would fill the space due to the icon being out of flow?

If it were absolute, wouldn't any password manager icons render underneath it? as the input element would fill the space due to the icon being out of flow?

Good point. Then we are good.

Hey, @anthony-hull I forgot to mention earlier that you have to sign all of your commits before we can merge the PR.

@anthony-hull  Any update on the requested changes. Let's complete this asap. Let me know if you are facing any challenges.

I've fixed the overflowing input. I'll sign the commits, re-implement the negative margin and then re-test. should be ready soon!

do I have to have all of them to show as verified? I ran a rebase command to sign them all and I was having fix merge issues on all my changes again. git was not happy. it was a nightmare so I aborted.
git rebase --exec 'git commit --amend --no-edit -n -S' -i main

Unfortunately Yes.

much easier to make a new PR with a new branch :)
please see:
https://github.com/Expensify/App/pull/6501
I just need to test on all platforms now. I'll let you know!

@anthony-hull Please close this PR. Thanks.


How Can I set background color to each column in grid?

Is there any way to set the color of each column separately?
Currently, it is possible to change the entire background, but business requirements clearly indicate the need to set the background for each column separately. The ability to add a line (border) between columns would also be welcome!

Did you manage to implement this? If so, could you provide a sample so I can add to the documentation and help others?


Webpack Caching Issue: "Expected end of object" Error during Navigation

Link to reproduction
No response
Environment Info
Binaries:
  Node: 22.7.0
  npm: 10.8.2
  Yarn: N/A
  pnpm: 9.9.0
Relevant Packages:
  payload: 3.0.0-beta.98
  next: 15.0.0-canary.139
  @payloadcms/db-postgres: 3.0.0-beta.98
  @payloadcms/email-nodemailer: 3.0.0-beta.98
  @payloadcms/graphql: 3.0.0-beta.98
  @payloadcms/live-preview: 3.0.0-beta.98
  @payloadcms/live-preview-react: 3.0.0-beta.98
  @payloadcms/next/utilities: 3.0.0-beta.98
  @payloadcms/plugin-form-builder: 3.0.0-beta.98
  @payloadcms/plugin-nested-docs: 3.0.0-beta.98
  @payloadcms/plugin-redirects: 3.0.0-beta.98
  @payloadcms/plugin-seo: 3.0.0-beta.98
  @payloadcms/richtext-lexical: 3.0.0-beta.98
  @payloadcms/translations: 3.0.0-beta.98
  @payloadcms/ui/shared: 3.0.0-beta.98
  react: 19.0.0-rc-7771d3a7-20240827
  react-dom: 19.0.0-rc-7771d3a7-20240827
Operating System:
  Platform: darwin
  Arch: arm64
  Version: Darwin Kernel Version 23.6.0: Mon Jul 29 21:14:30 PDT 2024; root:xnu-10063.141.2~1/RELEASE_ARM64_T6000
  Available memory (MB): 32768
  Available CPU cores: 10

Describe the Bug
I'm encountering a persistent caching issue with Webpack while navigating through different parts of my application, specifically between the admin site and other pages. The following warning is logged in the console:
<w> [webpack.cache.PackFileCacheStrategy] Caching failed for pack: Error: Expected end of object <w> (during deserialization of Object) <w> (during deserialization of webpack/lib/util/registerExternalSerializer webpack-sources/CachedSource) <w> (during deserialization of Map) <w> (during deserialization of Object) <w> (during deserialization of Map) <w> (during deserialization of webpack/lib/cache/PackFileCacheStrategy PackContentItems)
I already tried to remove the node packages and next folder but a reinstall didn't help unfortunately.
I don't know if the problem is Payload or Next in this case and didn't find anything useful on GitHub or StackOverflow.
Reproduction Steps
When navigating through different pages or adding content to my Payload Pages or Collections, I occasionally encounter said caching logs.
Adapters and Plugins
No response

try this
https://gofile.io/d/yMHabA
select the desired compiler in the installer

Okay after updating, removing the node modules, next folder and reinstalling and running pnpm prune the error is apparently gone. Just a heads up for everyone running into the same problems.


oci-image-tool unpack does not preserve permissions and ownership of entries

oci-image-tool does not chmod or chown the extracted files. I wonder if this is something intentional or PR-worthy.

On Thu, Sep 08, 2016 at 07:07:39AM -0700, George Lestaris wrote:

oci-image-tool does not chmod or chown the extracted files. I
wonder if this is something intentional or PR-worthy.

I think this should be fixed.  Unpacking semantics should probably be
just like tar, and from tar(1) 1:
   --no-same-owner
          Extract files as yourself (default for ordinary users).

   --no-same-permissions
          Apply the user's umask when extracting  permissions  from
          the archive (default for ordinary users).

I am happy to pick this up as a PR then @image-spec-maintainers.

This issue was moved to opencontainers/image-tools#17

On Tue, Sep 20, 2016 at 07:47:22PM -0700, Brandon Philips wrote:

This issue was moved to opencontainers/image-tools#17

PR is in opencontainers/image-tools#3.


cocomac: Watch blogsandarticlesgroup.com

cocomac requests the watch of the watch_keyword blogsandarticlesgroup\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

blogsandarticlesgroup\.com has been seen in 2 true positives, 0 false positives, and 0 NAAs.

Approved by Ryan M in Charcoal HQ


Marriage abroad ticket 15. Estonia

Add a callout for Estonia's outcomes when residing in the country.
There is a CNI exception for long-time residents.

Should this also contain updates to the regression test html outcomes in test/artefacts?

Other than my question about html outcomes this looks good to me.

@chrisroos ideally it would, but I'm planning to do that on a tt_ma_ticket_15 -> master PR (note that this is a PR for tt_ma_ticket_15 branch).
I'm not doing that simply because it takes 1h to generate HTML artefacts and I will have 20 or so PRs for this batch of changes.
However, if I could somehow restrict the set of answers to use for a specific question when generating HTML artefacts e.g. ceremony_country=estonia,latvia it would improve the experience for me :)

I'm not doing that simply because it takes 1h to generate HTML artefacts and I will have 20 or so PRs for this batch of changes.

Bah, sorry. I didn't realise it took so long to generate the output. We'll have a think about whether there's anything we could do to speed it up.

Merged manually https://github.com/alphagov/smart-answers/commit/34d1ae59098475b82a43c3edac7ba86c415d95f2


Update README.md

Adjust several sentences, and add the GUIDE.md.

Thx the help of @kezhenxu94 and @JesseZhou-1 .


Fixed: Camera3D has an incorrect shape before opening the 2D tab

Fixed default shape of frustum of camera3D, which had incorrect shape before opening the 2D tab.
Production edit: Fixes #70362

This is what the behavior of frustum should look like, it seems to me (Godot 3.x):

But while I was testing this PR, I found a more detailed pattern. Frustum doesn't change the first time you switch to a 2D tab, it changes every time you switch to a 2D tab. (But I didn't notice this when I created the issue 😞 )
(recorded with v4.0.beta.custom_build [1ea6a5d38])
https://user-images.githubusercontent.com/7782218/209750377-5f2ea46b-74c9-457c-a55d-0c7ec40ba6da.mp4
Also, the size of the Camera3D gizmo itself in godot 4.x changes as correctly as in godot 3.x

It turns out that this PR does not solve the problem yet.

@DmitriySalnikov so the frustum should be independent on viewport size or the frustum should scale without switching to 2D tab?

I do not know how it worked before. But before godot 4, changing the window size parameters always affected Camera3D(gizmo and get_frustum()) on the 3D tab and the size of this area on the 2D tab:

The size of this rectangle was also changed

Current Camera frustum is great for preview mode, but it still only updates when switching to the 2D tab.
By the way, Camera3D inside the SubViewport works correctly, as in godot 3.

@DmitriySalnikov should changing viewport size (in Project setting) affect SubViewport camera frustum ?

@DmitriySalnikov should changing viewport size (in Project setting) affect SubViewport camera frustum ?

No I think not. Even if you look at Camera3D gizmo, you can see that the shape of the camera inside the SubViewport depends only on the size of this subviewport. And a camera without a subviewport changes shape based on the size from ProjectSettings.
https://user-images.githubusercontent.com/7782218/210003476-4d334b30-1967-4db3-85b2-a04c06619fb8.mp4

@DmitriySalnikov i think now it should work like godot 3. :D

This works well, but only as long as the SubViewport has the name SubViewport...

@DmitriySalnikov i am have some trouble solving this. The following code
        bool check = false;
	Node *temp = this->get_node(this->get_path());
	if (temp != NULL) {
		if (temp->get_class_name() == "SubViewport") {
			check = true;
			print_line(temp->get_class_name());
		}
		else {
			check = false;
		}
		temp = temp->get_parent();

	}

is giving the following output for demo project attached in this issue

but the scene tree does not have any subViewport node.

I don't know a best way to find out which SubViewport is the root for the current scene

@DmitriySalnikov finally! i think this should do it. ✌

@akien-mga godot 4.0 has already been released. Is it possible to merge this PR now?

Not yet, I'm not sure this fix is appropriate. Fetching project settings all the time for such a core method doesn't sound like a good option. The real issue might be deeper and this looks like a workaround.


[ADAM-1439] Add inferSequenceDictionary ctr to FeatureRDD.

Fixes #1439.

Coverage increased (+0.06%) to 80.566% when pulling 01af07684669872b5212d177507673ac95c3c374 on heuermh:infer-seqdict into dbf4f85e7a6701edae5a8e465ef6699f07763369 on bigdatagenomics:master.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/ADAM-prb/1892/
Test PASSed.

Coverage decreased (-0.1%) to 80.367% when pulling 3a0d2a579a676b9299b223d95f8468f2d539c319 on heuermh:infer-seqdict into dbf4f85e7a6701edae5a8e465ef6699f07763369 on bigdatagenomics:master.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/ADAM-prb/1893/
Test PASSed.

Merged! Thanks @heuermh!


Update go to 1.15

Update k8s dependencies
Add additional flags:

reproducible builds: -trimpath and -ldlflags=-buildid=
smaller binaries: -w (trim debugger data)

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