Can we get a wiki page or something for how to get a minimum working start project?

Whenever I want to try get a simple project started quickly I end up either going around in circles trying to get it working,
or copy pasta/cargo culting in a whole bunch of extraneous stuff just to get something working, and it just looks messy and bloated.
My last attempt at something bare bones just crashes/exits without error when ImGui.Text is called.
Followed that down into the internals a bit more to ImGuiNative.igText(numPtr); but got no further that.
Something along the lines of:
a list of nuget packages
a chunk of code or Program.cs to set it up.
with a small section of extra stuff you might want. (i.e. implot)
I realize there's a couple of ways to do this depending on what render you're wanting to use, but that shouldn't complicate stuff too much I don't think.

https://github.com/ImGuiNET/ImGui.NET/tree/master/src/ImGui.NET.SampleProgram is not enough?

not really,
there's six other projects in that, which is fine if you're wanting to futz around with imgui.net project itself, but not so great if you just want to get something bare bones working for a project.

Why can't we have instructions for doing it ourselfs?
I want to start from zero and add stuff, not start from bloat and remove stuff.

I want to start from zero and add stuff, not start from bloat and remove stuff.

Define zero? I think your definition of zero is very different from (users of this library) definition of zero.

I think my op and following comments make it clear what I'm asking for.

I think my op and following comments make it clear what I'm asking for.

Actually it does not, what backend you want in that wiki? Imgui doesn't work without a backend, so what does ZERO mean over here? Does it mean with or without backend?
FYI: Following are the list of backends imgui supports.
https://github.com/ocornut/imgui/tree/master/backends
Imgui doesn't work without keyboard, mouse, controller input system, what input system you want in that wiki? There are 1000s of configuration that be different when using imgui so which configuration should be considered ZERO and included in the wiki?

I really feel like I'm being deliberately misunderstood and dismissed here.
I challenge you to open visual studio/code/rider or whatever your editor of choice is, create a new project and figure out exactly what nuget packages you need, drop in the code from the SampleProgram and get it working.
If I understand correctly, veldrid libraries handle most of the backend stuff, use those. and/or use openGL just for the cross platform ability.

Se5a, you're not being misunderstood.
You're failing to understand.
Let me try to explain: ImGui is a immediate Gui library. ImGui.Net is a dot net wrapper for that.
Imgui (and therefore imgui.net) has no way of opening windows, nor drawing shapes to the screen. It doesn't know rendering, it simply allows for structured creation of a GUI.
If you want to manage windows, you need a separate library for that. If you wanna render fonts, you need a dedicated graphics library for that. ImGui does not support that by default
It doesn't have sdl2 as a dependency, nor does it has opengl as a dependency. You can use whatever you want
So there is no MVP with ImGui.Net alone. Conversely, there's no required dependencies either.
If you wanna do a barebones project, you absolutely need to understand at least one of the samples. You don't need to use it in your project, but you need to understand what it's doing, how it's doing, and how to adapt it to your own project.
This is not a math library that you can just drop in any project. You need to study and experiment with it.
And, for the record, before I tried anything with ImGui, I was a proficient C# developer. And yes, it was challenging as hell, but what's provided in the samples is more than enough to get you going, all you need is to be willing to learn, and not copy paste everything.
Lastly: a window manager library and a graphics rendering library should be everything you need to get started with ImGui. Check the original repo (in cpp). It's not hard to adapt, and you'll see they'll have a lot of examples, for a plethora of different window managers and graphics libraries. Study, learn, and one day, you'll see we were right, afterall

I agree that this project really needs an introduction on what to add to make the library work.
Requiring the users to find themselves out what to include is just unreasonable. End-users of UI frameworks are those who design user UI apps, not necessarily familiar with all the layers of app rendering pipeline. Having these template projects is not enough. They really combine together few things that are unnecessary detail to people trying the framework out for the first time to see if its approach fits them. Reading these on plain github repo view page is not much helpful and downloading the
I don't say the entire library should be graspable within 10 minutes. If it does some crazy shit under the hood with lots of specific detail for fine tune then no one could understand it in these 10 minutes. But an entry level tutorial to start using the product is crucial. You won't get large userbase if everybody entering will be met with the 10-metre high brickwall of undocumented sample projects. If you want people to enjoy your product you must tell them how to get first experience with the product, not the poor exception traces. I may not assume the reason for the people behind ImGUI.Net to create and support it. But if they want to make something great and share it with people, pretending the current non-existence of tutorial is fine is plain hypocrisy. The tools and libraries you used to create this wrap had their documentation and abstractions good enough to let you focus on the port, not finding out the ways how to implement CPU and GPU-specific optimizations, hadn't they?
I have no doubt that it took great effort to make the original DearGUI work and this .Net wrap run, but it is embarassing how low grade the entry level experience is when working with it for the first time. Thousands of hours spent are suffering extreme loss of value because no one made a get-and-run page which could take no more than 10 hours to write. So that people could at least see that the library works and they would start learning it before sinking their own hours into selecting lower levels of their app. Which may be completely irrelevant for small applications where UI performance and other lib differences won't be noticeable.

because no one made a get-and-run page which could take no more than 10 hours to write

Listen, if you're so keen on having this sort of documentation, you are very welcome to contribute to the repo. Take a moment to learn it as it is and write about your findings along the way. Then compile those findings in a document and submit a PR. Everyone would appreciate the initiative.

Yeah taking a "moment" to learn means I will need to dedicate a full day or few to first learn, then write a guide and then check that I didn't miss something in the process.
The amount of time it would take anyone new to make such a guide is way more than for someone who has experience with it and has already created projects that use it. I found it out that for me it is too time-demanding and unreasonable to relearn from the available "documentation" to bother with this library. Why should I bother pointing out X if it has already been done by others? This is past for breaking point for me to get the thing working. It makes me ask questions about the creators are being interested in maintaining and supporting the product. About having a chance to get help if I encounter an issue.
I do respect the effort to build a wheel, but it sinks in futility if everyone else is forced to reinvent it anyway.

I am going to lock this issue now, if anyone has some time to do this, feel free to create a PR. This should be a good first issue to solve. :)


Tags in ops metric client

Add tags to metric-client-ops.
Depends on https://github.com/openshift/openshift-tools/pull/1640 and https://github.com/openshift/openshift-tools/pull/1621

Tested all scenarios locally.
@yaacov @joelddiaz PTAL

LGTM 👍

@twiest this one has been rebased off of prod, and it's ready on my part.
It's based on a commit from https://github.com/openshift/openshift-tools/pull/1640, but the other 2 commits simply adds 2 clis (very similar to ops-zagg-client & ops-zagg-pcp-client)


MINT function is not working

Giving error while executing MINT function in cw20 smart contract (git clone https://github.com/CosmWasm/cw-plus.git)
raw_log: 'failed to execute message; message index: 0: Unauthorized: execute wasm contract failed'

On contract State i am also getting this
{"name":"stablecoin","symbol":"GBP","decimals":6,"total_supply":"1000000000000","mint":null}

This is my INITD
INITD='{"decimals":6,"initial_balances":
[{"address":"cosmoszsfnu6dk9n8gneftup9tqgasjm0ysrssr3xty3","amount":"1000000000000"}],
"name":"stablecoin","symbol":"GBP"}'

Let me know if i am missing something or doing anything wrong.

"mint":null

->
Unauthorized

I tried, but no replies though.


jhu.edu is official website of Johns Hopkins University

Is there any wrong about this site?

False positive removed! Thanks for the heads up.


USMON-1152: Remove Kafka monitor tests optimization

What does this PR do?
Remove cleanProtocolMaps optimization from kafka_monitor_test.go.

Motivation
While investigating the flaky Kafka tests, we discovered that the cleanProtocolMaps function is failing with per-cpu maps used by the Kafka monitoring eBPF programs, which could potentially cause map leaks. Therefore, we are removing this optimization temporarily until a fix is identified for this issue

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge


Header gets hidden in safari when zoomed

When zoomed in on mobile safari, the header gets half-hidden sometimes, and stuck in place.

I've seen this on safari on an ipad.


InfluxDB store execution failure

In ILPF environments, we noticed that, eventually, the data appears in the GUI and is retrieved from the history, but is not returned in the retriever. In the project, influxDB is the main source we use in context, through retriever.
In a preliminary analysis, it was observed that errors in the log of the persistence service, indicating failure of communication with the message.
Restarting the dojot_influxdb-storer_1 container, the communication was re-established and functioning to be successful.
A suspicion is that an order to ascend the services may generate a communication failure that cannot be re-established later. Something similar to the suspected related flow broker in Jira DEVOPS-234.
The call is being opened to investigate this situation.

hj(21/03) ocorreu o cenário: "o retriever não está respondendo esses dados de hoje, mas o history está"
Após reiniciar o influxdb-store os dados ficaram disponível.
anexo o log do influxdb-store
log_influxdb-store-210311-0900.log


chore: skip starship prompt for customized image

Signed-off-by: Keming<EMAIL_ADDRESS>

/lgtm


Unable to execute the sample code

When I run the sample code on my pc i get this error:
{
status: 'error',
message: 'Invalid session credentials',
error_type: 'TokenException'
}

@srinicipher it just means the token was not valid. If you could paste the code here we could help you out


Optimal Stagger per hardware

Again this is a great project and my sincerely tanks for the time you put in this.
I literally know nothing about all this and started all in a rush but for newbies like me is there a way to present a auto stagger mode with the script checking the existing hardware, could this be added on enhancement list, if to much no trouble at all.
If not possible could you advise for a not computer tech guy ho would i stagger the best for my set up
AMD Ryzen 3960x - 48 Threads
96 gb ram
3 x 2tb NVME (1 Sabrent Q and 2 Samsung 970 EVO Plus) not in raid
15 x 8 or 10 tb portable hdds all connected  and sleep mode disabled
What is the way as i cant get this time under 10h i was running 12 plots in parallel in different way (3 in pase 1 then another r 3 after 32% and another 3 after 78% and Another 3 at 85% or i run the 1 at interval of 60 minute  whatever i tried nothing diminished that time.
Any advice would be much appreciated.

I'm sure you know but your phase 4 is going to be limited to the write speed or interface speed of your portable drives. With that said I have not done much testing on NVME raid vs single. However one of my servers is getting 20 plots per day with 8 600gb 15k SAS drives in a raid 0. I have not done much testing to max or perfect the timing and just run 8 plots in parallel with 120 minute delay, 6 cores each, and 6144MB RAM.
I do not have any NVME drives that I am willing to sacrifice (not even my enterprise grade ones) on this coin. I didn't do the exact math but I wouldn't expect more than ~160 days of usage out of your 3 NVME drives before they die. Someone can correct me if I am way off on that number.
You could try putting your 3 NVME drives in a storage pool and setting them up in a raid 0. See if your speed increases ( I would assume it would). I will link a great video on how to optimize your plots by looking at logs and using PowerShell.
Optimizing The Delay Between Parallel Plots Using Log Files and Powershell - Chia Plotting Part 3

@sk00t3r   thanks a million for reply, in terms of the NVMe ware I'm aware and is fine as long as i can get my 200tb full quickly, in terms of the copy phase it takes rafely 9-10 minutes they are connected all through usb 3.2 gen ports and the hdd's are usb 3
i had don one plot only and used 12 threads and 6750 memory and it took 6h 35min

Someone please correct me if I am wrong here but I think only phase 1 uses multithreading and there are diminishing returns after using 4 threads. Your RAM is on the high side many people say that with bitfield enabled (default settings) the max is ~3400, and after reading up on your setup, you should be aiming for 7 plots on each 2tb nvme (1862GiB/239GiB=7.79). A good starting might be trying to stagger them 40 minutes apart and start all three nvme's at the same time. From there you can look at logs or trial and error and see if its any faster to stager each nvme's starting time or not.

I think sk00t3r provided a great starting point a good optimization of settings. I think from there it will just come down tweaking it a bit.


Problems with options

500 issues with options.
I have been having issues with place_option_orders.  I'm not sure what the issue is.  I can get it to work by matching the API I see with the browser, but not with the current setup.

Thanks, looks like the URL and the lzone are different.
Could you give the two a try and let me know which it is?

Looks like changing dc_core_r001 to dc_core1 in webull.py was all that was needed.
Thank you for the help!
Does on_order_message work for you?

I haven't used it lately, but the last time I used it was arond mid last year? I remembered it worked okay.

You're right that mid last year it did work, but not since then for me.  It's fairly important, because I try to keep my hits to the servers as few as possible.  Because I'm not able to see when a order fills, I have to check periodically, which is more hits to the server than I'd regularly carry out.  It's still not a lot, but it's also slower using that method.  I tried to find where that's being carried out through chrome developer tools, but have yet to find anything other than I think it's using a different mechanism, possibly.

I fixed on_order_message, needs an update to streamconn.py.  Do you support active developers contributing directly, or do is it better to just open an issue and put the updates there?

There are a couple of contributors. If you have a bugfix, feel free to send a PR and one of us will look at it


Documentation for setup

Similar to #12, I was confused on how to use this code due to the file picker dialog.
I think a short documentation can help to get this working for users unfamiliar to the specifics of MacOS virtualisation.

In addition, I'd like to thank you all for your amazin tool. I just tried setting up VM Ware Fusion for ARM and I didn't get it working. Meanwhile I got your tool running within a few minutes. Amazing and thanks again.


Fix rubocop violations

Summary
Partial fix for #1021
Details

Hash#has_key? => Hash#key? due to DEPRECATION
Lint/BlockAlignment
Lint/DefEndAlignment

Awesome, thanks for your contribution @madundead!


Publish image for arm64

At the moment it is not possible to run the existing image on arm64 and OS X due to a qemu bug (anyway, it would be painfully slow).
Fortunately I've just successfully built an image for amd64 (OS X) with one minor amendment:
RUN pip3 install --upgrade pip && \   # pip should be upgraded to support multiple patforms
    pip3 install empy \
    ...

It would be helpful if arm64 image will be published on Docker Hub. It can be done by a simple change in GitHub pipelines config.

I'd suggest to make a PR here that adds what you need to CI.

I've added PR: #27.

Thanks @Sitin.


update KubeEdge as incubating project in landscape

Signed-off-by: Kevin Wang<EMAIL_ADDRESS>Pre-submission checklist:
Please check each of these after submitting your pull request:

[x] Are you only including a repo_url if your project is 100% open source? If so, you need to pick the single best GitHub repository for your project, not a GitHub organization.
[x] Is your project closed source or, if it is open source, does your project have at least 300 GitHub stars?
[X] Have you picked the single best (existing) category for your project?
[X] Does it follow the other guidelines from the new entries section?
[X] Have you included a URL for your SVG or added it to hosted_logos and referenced it there?
[X] Does your logo clearly state the name of the project/product and follow the other logo guidelines?
[X] Does your project/product name match the text on the logo?
[X] Have you verified that the Crunchbase data for your organization is correct (including headquarters and LinkedIn)?
[x] ~15 minutes after opening the pull request, the CNCF-Bot will post the URL for your staging server. Have you confirmed that it looks good to you and then added a comment to the PR saying "LGTM"?

Deploy preview for landscape ready!
Built with commit 4bf071f2629f994a9137a7c04e8b69352101bf89
https://deploy-preview-1844--landscape.netlify.app

LGTM


Provide ability to disable Notebook start page when running tests

Currently this steals editor focus and our tests fail as a result of this (window.activeNotebookEditor is undefined after we open notebooks).
The tests are complex and we can't just rely on a return value of openNotebookDocument or the like, these are end to end tests in Jupyter extension.
Is there a way way to disable this start page?
@rebornix /cc

We are disabling this for the release so we don't have to worry about it for now.
In the future we can keep the setting or add a CLI flag that you can pass

Just to confirm @roblourens did you mean the notebook start page is being disabled for the upcoming release?

Yesterday I meant just the auto-open, but this morning we decided to not show the profile picker on the getting started page at all for this release. It will be back next month.

Not sure whether the auto-open is coming back. I'll close this. If it comes back we would definitely need a way to disable it.


enum mapping in lowercase fails by default

I have a legacy API that takes an enumeration as argument, here is a yaml fragment of OpenAPI spec :
...
components
    schemas:
       Mode:
          type: string
          enum:
            - a
            - b

Such enumeration should be used as string in API, but is translated to Enum with processor (why not, this is not the problem) as :
@Generated(value = "openapi-processor-spring", version = "2023.4", date = "2023-10-05T14:30:44.211810700+02:00")
public enum Mode {
    A("a"),
    B("b");
...

Internally spring use StringToEnumConverterFactory to convert the string to enumeration:
Daemon Thread [parallel-3] (Suspended)
	owns: FluxConcatMapNoPrefetch$FluxConcatMapNoPrefetchSubscriber<T,R>  (id=741)
	owns: FluxConcatMapNoPrefetch$FluxConcatMapNoPrefetchSubscriber<T,R>  (id=742)
	Enum<E>.valueOf(Class<T>, String) line: 268
	StringToEnumConverterFactory$StringToEnum<T>.convert(String) line: 54
	StringToEnumConverterFactory$StringToEnum<T>.convert(Object) line: 39
	GenericConversionService$ConverterFactoryAdapter.convert(Object, TypeDescriptor, TypeDescriptor) line: 436
	ConversionUtils.invokeConverter(GenericConverter, Object, TypeDescriptor, TypeDescriptor) line: 41
	DefaultFormattingConversionService(GenericConversionService).convert(Object, TypeDescriptor, TypeDescriptor) line: 192
	TypeConverterDelegate.convertIfNecessary(String, Object, Object, Class<T>, TypeDescriptor) line: 129
	SimpleTypeConverter(TypeConverterSupport).convertIfNecessary(Object, Class<T>, TypeDescriptor) line: 73
	SimpleTypeConverter(TypeConverterSupport).convertIfNecessary(Object, Class<T>, MethodParameter) line: 53
	BindingContext$ExtendedWebExchangeDataBinder(DataBinder).convertIfNecessary(Object, Class<T>, MethodParameter) line: 729
	RequestParamMethodArgumentResolver(AbstractNamedValueArgumentResolver).applyConversion(Object, AbstractNamedValueArgumentResolver$NamedValueInfo, MethodParameter, BindingContext, ServerWebExchange) line: 193

and it fail, because Enum.valueOf(Mode.class, "a") delegates to internal enumConstantDirectory which contains only uppercase values "A" and "B" !
So the only workaround is to inject custom converter, but this is not desirable.

say the same shortly :
 // processor-spring impl
 Mode.fromValue(source);

is not the same as
Mode.valueOf(source);

yes, this is annoying.
the question is how to improve it.

generate lowercase enums

would need some switch to be backward compatible
maybe configure by enum

provide/generate converter

StringToEnumConverter implements Converter<String, MyEnum>

any other idea?

I would like specify what is annoying :

the need to add specific spring behavior
per enum spring behavior (converter) injection

Second point is far more annoying, because each enum must have its converter registered. Having a special interface on the enum i.e like "Supplier<String>" would allow :
- not break the backward compatibility
- reduce '2)' to '1)' by :
* allowing to identitfy that enum must converted another way.
* avoid to depends on specific openapi-processor type

If breaking backward compatibily is an option, then perhaphs, because initial openapi spec declare a string type, we can just keep it as a string. enum declaration in spec could be interpreted a special validator constraint with something like :

    public Mono<String> someEndpoint(@RequestBody @DictionaryValue(allowedValues = {"a", "b"}) String mode)

with :
    import javax.validation.Constraint;
    import javax.validation.Payload;
    import java.lang.annotation.*;

    @Target({ElementType.FIELD, ElementType.PARAMETER})
    @Retention(RetentionPolicy.RUNTIME)
    @Constraint(validatedBy = DictionaryValueValidator.class)
    @Documented
    public @interface DictionaryValue {
        String message() default "Invalid value. Please choose from the dictionary.";
        Class<?>[] groups() default {};
        Class<? extends Payload>[] payload() default {};
        String[] allowedValues() default {};
    }

and
    public class DictionaryValueValidator implements ConstraintValidator<DictionaryValue, String> {

        private String[] allowedValues;

        @Override
        public void initialize(DictionaryValue constraintAnnotation) {
            this.allowedValues = constraintAnnotation.allowedValues();
        }

        @Override
        public boolean isValid(String value, ConstraintValidatorContext context) {
            return value != null && Arrays.asList(allowedValues).contains(value);
        }
    }

This is only suggestions.
Best regards

I have successfully created a prototype ConverterFactory that creates enum converters for generated enums that implement Supplier<>. Not much code which is nice  :-)
import org.springframework.core.convert.converter.Converter;
import org.springframework.core.convert.converter.ConverterFactory;
import org.springframework.stereotype.Component;

import java.util.EnumSet;
import java.util.function.Supplier;

public class StringToEnumConverterFactory<T extends Enum<T> & Supplier<String>> implements ConverterFactory<String, T> {

    @Override
    @SuppressWarnings({"unchecked", "rawtypes"})
    public <E extends T> Converter<String, E> getConverter(Class<E> targetType) {
        return new StringToEnumConverter(targetType);
    }

    static class StringToEnumConverter<T extends Enum<T> & Supplier<String>> implements Converter<String, T> {

        private final Class<T> enumType;

        public StringToEnumConverter(Class<T> enumType) {
            this.enumType = enumType;
        }

        public T convert(String source) {
            String sourceValue = source.trim();

            for (T e : EnumSet.allOf(enumType)) {
                if (e.get().equals(sourceValue)) {
                    return e;
                }
            }

            throw new IllegalArgumentException(
                    String.format("No enum constant of %s has the value %s", enumType.getCanonicalName(), sourceValue));
        }
    }
}

It requires a one time registration using Web[Mvc|Flux]Configurer.addFormatters().
Your String version is also something I'm considering. I'm thinking about a global setting to switch between enum  and String. enum would also generate the converter factory  and it can be registered (or not if it is not needed).
Thanks for your input :-)

improved in 2023.6, added enum-type mapping option default (like before), string (just use string) and framework (enum with spring converter factory)


How to make model fast?

I compare the 4 models you supplied，as the same condition Linknet looks fast , but 'decoder_filters's length is  5 (default is None,None,None,None,16), how can i change it to 3 or 2 ? And the same problem is the default 'encoder_features' is 4 , it comes from BackbonesFactory , how can i choose 2 or 3 to make model fast ? Thank you !

I found that LinkNet was the fastest, but the accuracy was significantly lower than U-Net. You can look at deceasing the size of the images?


Supporting HAIP as a mandatory?

This issue is meant to open a discussion with respect to the use of HAIP in EWC.
In RFC001 v2 we mention supportitng HAIP.
The ARF v1.4 includes HAIP  but only as a reference , we should make a decision as to if HAIP is mandator or optional  (with respect to EWC piloting).
This obviously affects various parts of the RFC001 such requirement for Pushed Authorisation Request  and so on...
Let us try to get a concensous ....

The intention of HAIP was also to restrict the options and therefore increase the likeliness of interop. I am in favor of making it mandatory. Any issues to be aware of here?

Yes, I fully agree but we have to keep in mind that there is a misalignment between HAIP and OIDC4VCI which needs to be considered.

Hey all, @andreasabr would you be kind enough to point to these issues so we can discuss (to be honest I was not aware)


Gitignore + Another Update to the README

Changes in this pull request:

Added a .gitignore file
Updated the README

Moved the GIF references to the bottom
Added local development instructions
Moved line 57 outside of the code block
Fixed contributors screenshot positioning
Added line separators between sections
Added re-fork message

Thanks!


Allow public invites for alice/faber demo

Signed-off-by: Ian Costanzo<EMAIL_ADDRESS>Addresses https://github.com/hyperledger/aries-cloudagent-python/issues/1571
Note - added a --reuse-connections parameter to faber - this will trigger faber to use a public did in the didexchange connection (and alice will always "reuse connection" if possible) and the connection indeed gets reused, however right now there are no webhooks for the "reuse" messages so the alice/faber controllers are getting hung ...

Codecov Report

Merging #1574 (d492419) into main (cd5588f) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1574   +/-   ##
=======================================
  Coverage   95.76%   95.76%
=======================================
  Files         527      527
  Lines       32312    32312
=======================================
  Hits        30942    30942
  Misses       1370     1370

Hmm...I ran this:
./run_demo faber --public-invites
Preparing agent image...
sha256:94c44484e255f32387e52e2b69f8f7e767625c3a36235cd1239b5481dbb67cc9
Trying to detect ngrok service endpoint
ngrok not detected for agent endpoint
<IP_ADDRESS>
Starting [faber] agent with args [--port 8020 --public-invites]
usage: faber.py [-h] [--no-auto] [-p <port>] [--did-exchange] [--revocation]
                [--tails-server-base-url <tails-server-base-url>]
                [--cred-type <cred-type>] [--aip <api>] [--timing]
                [--multitenant] [--mediation] [--multi-ledger]
                [--wallet-type <wallet-type>]
                [--endorser-role <endorser-role>] [--arg-file <arg-file>]
faber.py: error: unrecognized arguments: --public-invites

Hmm...I ran this:
./run_demo faber --public-invites

...

faber.py: error: unrecognized arguments: --public-invites

You don't need to add a new parameter to faber, the --public-invites param is added to the aca-py agent that faber (and alice) start up under the hood

So it works, but each time I do a new invitation, I get a new connection on both sides.  That means that reuse is not being used, which is the goal.  Alice should be responding with a "reuse" message, and her "new" connection should be removed.  Likewise on Faber -- once she responds with the reuse, the hanging connection should be removed.  This (pretty likely :-) ) works because there was a follow up tweak to DID Exchange to do the clean up after someone tried this out.
Thoughts?

I think the reuse should be automatic, handled by ACA-Py, but am not certain how it works.

Also, I noticed that the "their_public_did" item on the connection object was not set, which is a bit odd.  Not sure why it wouldn't be, but I think that is a pre-requisite for "reuse" on a connection.

I believe the auto-reuse only works when you use public DIDs for connections, which the demo is not doing

I believe the auto-reuse only works when you use public DIDs for connections, which the demo is not doing

I guess it depends on the intent, I though we were just trying to unblock Peter (issue https://github.com/hyperledger/aries-cloudagent-python/issues/1571 where he is using the API).
If we want to configure alice/faber to use connection reuse for the demo then there is more work required.  Let me know ...

Let's leave it for now.  Peter's goal is to enable connection reuse, so that's really what we are after.
I think it would be nice to have the demo optional support connection reuse, so it would have an optional parameter, making it easy to see what the controller has to do to support reuse.  How about we leave this as a draft PR and when you get a chance, you can update it?  I'm hoping it is a pretty small change.

Let's leave it for now. Peter's goal is to enable connection reuse, so that's really what we are after.
I think it would be nice to have the demo optional support connection reuse, so it would have an optional parameter, making it easy to see what the controller has to do to support reuse. How about we leave this as a draft PR and when you get a chance, you can update it? I'm hoping it is a pretty small change.

Sure that works, I've converted the PR to draft.

Fixed demo - run with ./run_demo faber --reuse-connections - this will use faber's public DID for the connection and allow connection reuse.
(Alice will always try to reuse connections if possible.)


Footer Disable if records are less than rowsPerPageOptions array

If total records are less than provided rowsPerPageOptions array than i want to disable footer, and if i want to show all records from want one option in rowsPerPageOptions as All than how it possible ?

Expected Behavior

Disable if records are less than rowsPerPageOptions.
Current Behavior

Display Footer even if records are less than rowsPerPageOptions.
Steps to Reproduce (for bugs)
This is codesandbox link
https://codesandbox.io/s/datesorting-jntyu?fontsize=14&hidenavigation=1&theme=dark

Your Environment

Tech
Version

Material-UI

MUI-datatables

React

browser

etc

https://codesandbox.io/s/footer-1ogfh

@wdh2100  now my question is i want default functionality if record is graterthan rowPerPage. i want to handle both case than it's how ?

@DhruvSakhiya
here Ref
https://github.com/gregnb/mui-datatables/blob/4f836096a56be93ee1322f3c98cbaca485f0b064/examples/customize-footer/CustomFooter.js#L11


Feature: Making Top 4 Magitek commands an objective result

Changed default Magitek commands for all characters to be just the 3 elemental beams + Heal Force. Added new "Magitek Upgrade" Objective Result. When complete, the Top 4 Magitek commands (Terra's extra commands in vanilla) are available for all.
Video: https://www.youtube.com/watch?v=_8J3g3D1n5E&ab_channel=SilverThorn
Tested with these flags:
py wc.py -i ../rom/ff3.sfc -oa <IP_ADDRESS>.0 -ob <IP_ADDRESS>.0 -oc <IP_ADDRESS>.0 -od <IP_ADDRESS>.0 -oe 59.0.0 -sc1 terra -sc2 locke -sc3 sabin -com<PHONE_NUMBER>2998989898989898 -lsa 1 -hmced 0.5 -as -frw -mca -smc 3 -sn
-oe 59.0.0 is what enables upgraded Magitek.
I also double checked the resulting asm in a debugger to confirm it looked as expected. Here it is with -oe 59.0.0 and -of 59.0.0 (so that two objective bits get checked):

Merged into ff6wc fork


name 'SequenceClassifierOutput' is not defined

Try to run in Colab the Bert-explainability
! pip install -r requirements.txt
I got the following error
ERROR: Could not find a version that satisfies the requirement opencv_python==<IP_ADDRESS> (from versions: <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>)
ERROR: No matching distribution found for opencv_python==<IP_ADDRESS>
When run the example cells, I got the persistent error: name 'SequenceClassifierOutput' is not defined

Hi @songhuadan, thanks for pointing this issue out.
It seemed to be a problem with opencv versions with colab (I was able to run the notebook on my machine), and I fixed the requirements file.
There seems to be another issue with visualization using captum, but now you should be able to get the relevance scores.
I will try to work on resolving the captum issue as well, say tuned :)
Best,
Hila.

Hi, Thanks for the great effort.
I was wondering how I can use the VIT visualization code with a fine-tuned VIT on my own custom dataset? I used a pre-trained VIT based on the "google/vit-base-patch16-224-in21k" on my own data. Now that I am trying to use your code on my own trained Vit I get errors here and there! Do you have any suggestions?

Hi @Amirhosein2c, thanks for your interest!
As mentioned in this issue you can load your own weights as long as they are compatible with the implementation from timm.
does this help?

@songhuadan, issue fixed, thanks for pointing this our again :)

@Amirhosein2c  If you please, did you manage to visualize your google/vit model because I'm facing the same problem and it seems that the vit/model architecture is not the same as the model implemented in this repo


Update readme to include screenshoot of the tool

We should add screenshots for different functionalities of the tool.

@ttannis @danwom

Agreed, however I'd consider this task to be in waiting until we have a richer example data set that mimics real data.

@ttannis , just create the issue here to remind us :)


Feature/dynamically generate pages

Instead of adding a file for all the pages we want to have, we now query ContentfulPage with graphql and create a new page from a template for each page found
We move the global style to the template to avoid refetching of the font and therefore the "flicker" effect.
You might need to use gatsby clean before yarn start for these changes to work properly

With the Netlify-Link a page works perfectly without flickering when staying on the page. When you navigate to a different page it still flickers when switching. Do you think it's possible to make the font loading blocking, or to have the font not re-loaded?

I would have to do further research. I think the whole HTML changes including the css when navigating between pages.. I have to find a solution later to keep the css loaded between page changes

Do you prefer then to merge this PR and do the research in a next PR or do you want to continue with this one?

@mtauer i would prefer a separate PR ... i just need to rebase this one again


ignore_facts.rs:51-54: Implement ignores_author. Let's do...

The puzzle 27-1697b171 from #27 has to be resolved:
https://github.com/h1alexbel/ghiqc/blob/2732fe7df00794183019af4ea23e51b706eab570/src/args/ignore_facts.rs#L51-L54
The puzzle was created by @rultor on 02-Aug-24.
Estimate: 35 minutes,  role: DEV.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

@0pdd There is no clear issue statement in the bug report. The description only provides the locatio...

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


helm package behavior difference between helm2.16.0 and helm3.2.4 (permission 755 vs 644)

Observed file permission differences in the packaged charts from 2.16.0 vs 3.2.4
https://github.com/helm/helm/blob/v3.2.4/pkg/chartutil/save.go#L232
In helm 3.2.4, the permission is 644.
https://github.com/helm/helm/blob/v2.16.0/pkg/chartutil/save.go#L216
In helm 2.16.0, the permission is 755.
This has caused some files which are originally executable to be non-executable after the helm package process, should there be an option to preserve the original file permissions instead of using hardcoded permissions of 644 or 755 ?
Thanks.

@adamreese
https://github.com/helm/helm/commit/b6fdd8783bff827ef0598eb709a30492fbafeeec

@adamreese
https://github.com/helm/helm/commit/b6fdd8783bff827ef0598eb709a30492fbafeeec

A security audit prompted that change. You can read the audit here: https://github.com/helm/community/tree/master/security-audit

A security audit prompted that change. You can read the audit here: https://github.com/helm/community/tree/master/security-audit

re-opening.... just wondering would you guys consider a PR for preserving the permissions with helm package CLI flag (instead of hardcode 644) ? this way the permission security would be up to the user instead of helm.

re-opening.... just wondering would you guys consider a PR for preserving the permissions with helm package CLI flag (instead of hardcode 644) ? this way the permission security would be up to the user instead of helm.


chore: log sync aggregate participants when producing beacon block body

Motivation
To better investigate https://github.com/ChainSafe/lodestar/issues/7299
Description
Log sync aggregate participants when producing beacon block body

:tada: This PR is included in v1.24.0 :tada:


Is it possible to have access to the NER part directly?

Hello and thank you for your great work.
I would like to test UmlsBERT on a self supervised model to learn sentence embeddings. I'm looking to simply use your checkpoint to do it. But is it possible for me to have access to the NER part? which given a tokenized sentence, classify each token in the 42 UMLS categories to then use the umls embeddings of BERT that you propose. I saw in the paper that you mention Ctakes to do it but I don't see where it appears in the repository.
The best for me would be to simply download your checkpoint (which I did) and then incorporate a part of your code to my other repo to test it. I looked at the NER jupyter file but I don't see the inference part of tokenized sentence with Ctakes.
I'll be so grateful for any advice on this.
Alexandre

Update : I have seen that vocab_updated.txt is in the repo and corresponds to the umls tags. So if I understand right, the sentences don't go trough a custom NER network (like LSTM or other) to get the classes but if a sentence contains tokens classified as tui they will have a special embeddings right?
Alexandre


fix: focus on Android (v4)

What
This PR fixes a problem of incorrect focus on Android (when position of the focus is driven by gesture-handler).
Changes

This PR passes px value to meteringPointFactory.createPoint method;
This PR adds px for Float extension.

Tested on
Pixel 7 Pro (Android 14).
Related issues
We discussed it internally in Discord channel.
Screenshots

Before
After

Great work kiryl! lgtm


Add compact and detailed mode to ctkDICOMTableView and ctkDICOMTableManager

Introduce "compact and detailed view mode" to ctkDICOMTableView and ctkDICOMTableManager. Basic idea is that you can give the widgets a list of "high priority" tags.
In compact mode all columns of non priority tags will be filter.
In detailed mode everything is shown.
You can define the active mode via property / slot (to e.g. easily connect it with a control widget signal).

@mehrtash  @fedorov @pieper @cpinter  Is this something on DICOM roadmap ?

It's the first time I see this. I am not quite sure I understand this request.

Yes, I worked on that topic while the last ctk hackfest. It is nearly finished (we had some glitsches with the correct column resizing when the compact view was activated).
The reason why it was not already pushed was the combination of pre-holiday-season-crunch-time and paternity leaf till last week.
If I know that someone else is interested in that issue, I will raise the priority on that and see that everything is cleaned up and pushed soon.
Best regards,
Ralf

Von: Jean-Christophe Fillion-Robin<EMAIL_ADDRESS>Gesendet: Donnerstag, 5. Februar 2015 07:01
An: commontk/CTK
Cc: Floca, Ralf Omar
Betreff: Re: [CTK] Add compact and detailed mode to ctkDICOMTableView and ctkDICOMTableManager (#518)
@mehrtashhttps://github.com/mehrtash @fedorovhttps://github.com/fedorov @pieperhttps://github.com/pieper @cpinterhttps://github.com/cpinter Is this something on DICOM roadmap ?
�
Reply to this email directly or view it on GitHubhttps://github.com/commontk/CTK/issues/518#issuecomment-72998755.

@rfloca Thanks for the comment. I was just doing some spring cleaning on the issue tracker and was wondering what was the status.

Closing. Performance and usability of the DICOM browser have been improved in https://github.com/commontk/CTK/pull/886


Fixes #2209: Add a search history to the search bar

Description
A content provider was added to save the most recent search queries of the user and make suggestions as the user performs a new search.
Related issues and discussion
#fixes #2209
Screen-shots, if any
See attached,

Can you use the black material icon instead ? That will be in line with the rest of the app :-)

The search history icon has been changed to material black. Are there any other UI improvements to be done?
See attached,

Looks good. Implemented according to https://developer.android.com/guide/topics/search/adding-recent-query-suggestions#java

Thanks @subhanjansaha & @PrajwalM2212 for the review. Merging :-)


fix(sbb-selection-panel): arrow navigation with radio-button-group should select the radio-button

Preflight Checklist

[x] I have read the Contributing Guidelines for this project.
[x] I agree to follow the Code of Conduct that this project adheres to.
[x] I have searched the pull request tracker for a Pull Request (PR) that matches the one I want to submit, without success.

Issue
This PR Closes #2188
Pull request checklist
Please check if your PR fulfills the following requirements:

[x] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been reviewed and added / updated if needed (for bug fixes / features)

See Review Guidelines for more information what is checked during review process.
Changes
Changes in this pull request:

using a sbb-radio-button-group within a sbb-selection-panel, when a radio-button is selected and an arrow key is pressed, the previous/next is focused and selected instead than only focused.

Browsers
I tested the build on the following browsers:

[ ] Firefox Desktop
[x] Chrome Desktop
[ ] Edge Desktop
[ ] Safari Desktop
[ ] Chrome Mobile
[ ] Safari Mobile

Screen readers
I tested the build on the following browsers:

[ ] JAWS Firefox Desktop
[ ] JAWS Chrome Desktop
[ ] NVDA Firefox Desktop
[ ] NVDA Chrome Desktop
[ ] VoiceOver Safari Desktop
[ ] VoiceOver Chrome Desktop
[ ] VoiceOver Safari Mobile
[ ] Android Accessibility Suite Chrome Mobile

Pull request type

Please check the type of change your PR introduces:

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

Does this introduce a breaking change?

[ ] Yes
[x] No

Other information

Codecov Report
Attention: 89 lines in your changes are missing coverage. Please review.

Comparison is base (638977d) 92.61% compared to head (6710609) 95.95%.
Report is 117 commits behind head on master.

Files
Patch %
Lines

src/components/core/dom/spread.ts
80.48%
8 Missing :warning:

src/components/icon/icon-base.ts
89.74%
8 Missing :warning:

...metable-occupancy-icon/timetable-occupancy-icon.ts
92.45%
8 Missing :warning:

...nents/core/common-behaviors/slot-child-observer.ts
93.33%
5 Missing :warning:

src/components/core/dom/breakpoint.ts
42.85%
4 Missing :warning:

src/components/autocomplete/autocomplete.ts
86.36%
3 Missing :warning:

src/components/card/card-action/card-action.ts
70.00%
3 Missing :warning:

src/components/core/dom/dataset.ts
25.00%
3 Missing :warning:

src/components/core/dom/host-context.ts
40.00%
3 Missing :warning:

...onents/core/eventing/connected-abort-controller.ts
40.00%
3 Missing :warning:

... and 26 more

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2255      +/-   ##
==========================================
+ Coverage   92.61%   95.95%   +3.34%
==========================================
  Files          53      334     +281
  Lines        2409    39172   +36763
  Branches      254     2191    +1937
==========================================
+ Hits         2231    37589   +35358
- Misses        176     1539    +1363
- Partials        2       44      +42

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


catkin_make error

/usr/bin/ld: cannot found-llcm
collect2: error: ld returned 1 exit status
unitree_guide/unitree_guide/CMakeFiles/junior_ctrl.dir/build.make:538: recipe for target '/home/medolic/unitree/devel/lib/unitree_guide/junior_ctrl' failed
make[2]: *** [/home/medolic/unitree/devel/lib/unitree_guide/junior_ctrl] Error 1
CMakeFiles/Makefile2:3547: recipe for target 'unitree_guide/unitree_guide/CMakeFiles/junior_ctrl.dir/all' failed
make[1]: *** [unitree_guide/unitree_guide/CMakeFiles/junior_ctrl.dir/all] Error 2
Makefile:145: recipe for target 'all' failed
make: *** [all] Error 2
Invoking "make -j3 -l3" failed

Hello, your question is about unitree_guide. You can ask your question under the repository https://github.com/unitreerobotics/unitree_guide. The error message you encountered indicates that you have not installed the lcm library. You can refer to this link https://github.com/lcm-proj/lcm for downloading.


bug in documentation: what is QQ Mail?

Line 45 of the readme mentions a a 'QQmail', however it isn't clear what a 'QQmail' is.
Furthermore an example is not given around line 16.
Email:

| Email Schema  | Type_Name          | Function              | :white_check_mark: | :x:         |
| ------------- | ------------------ | --------------------- | ------------------ | ----------- |
| StandardEmail | StandardEmail_Type | is_StandardEmail_Type |<EMAIL_ADDRESS>      | abc         |
| QQMail        | QQmail_Type        | is_QQmail_Type        |<EMAIL_ADDRESS>       |<EMAIL_ADDRESS>   |
| Gmail         | Gmail_Type         | is_Gmail_Type         |<EMAIL_ADDRESS>      |<EMAIL_ADDRESS>|

HaHa.
QQmail is a popular mail format in CHINA. As far as I know, basically every Chinese people have a QQ number and it along with a QQmail. which is like this usually.
<EMAIL_ADDRESS>or<EMAIL_ADDRESS>just like gmail is<EMAIL_ADDRESS>QQ mail is<EMAIL_ADDRESS>QQ


Usage of symbolic icons from the system icon theme

Is your feature request related to a problem? Please describe.

having to use a nerd font for icons

Describe the solution you'd like

allow for using the icons from the system icon theme for modules/widgets

This is a little complex as icons are just images, so have to be added as a separate GTK widget. I could potentially add support for:

Adding images/icons at the start/end of widgets as some common config option (like the tooltip etc)
A standalone image module
Images in the custom module

Being able to just add icons anywhere in place of Nerd Font icons I won't do as it adds quite a lot of complexity.

I think the first option would be best

I don't really need them to be in any specific position

The above PR will get this sorted. I've not  added the prefixes/suffixes since custom will cover that ,but you can now use images in the custom module, and inside workspace.name_map  and music.icons:
let {
  $workspaces = {
    type = "workspaces"
    name_map = {
      1 = "icon:firefox"
    }
  }

  $image = {
    type = "custom"
    //                               \/ http(s) also supported
    bar = [ { type = "image" src = "file:///path/to/image.jpg" size = 32 }  ]
  }
} in {
  icon_theme = "Paper"

  left = [ $workspaces ]  right = [ $image ]
}


⚠ Consejo de Estado has degraded performance

In 345141f, Consejo de Estado (https://consejodeestado.gov.co/) experienced degraded performance:

HTTP code: 200
Response time: 7207 ms

Resolved: Consejo de Estado performance has improved in 027b66d.


Get a random assessment, restructuring and loading files, code, problem statement and grading instructions.

Implemented

get a random assessment
load files, code, problem statement and grading instructions from submission that is being assessed
fix authentication (allow empty response)
other minor bug fixes
major restructuring to properly split the functionality of the assessment view model in two

Sorry for the big pull request, but the task unfortunately requires me to work on many of the files.
Created the pull request as soon as possible so that the others can work without us having a big merge conflict.

Thank you @pal03377 for the excellent review. Hopefully everything is fine by now.


Kubernetes monitoring guide

Hi,
I have setup kubernetes with coreOS by following below doc and the setup is perfectly fine.
https://coreos.com/kubernetes/docs/latest/getting-started.html
Now I would like to monitor kubernetes objects(pods,services etc..). Is there step-by-step instruction guide?
I searched for proper guide but none worked for me. Show this error.

Dupe of prometheus/prometheus#2405. Same maintainers over here and there, so closing this one.


Re-fetch/Re-paint issue with BoxDecoration

I think I have found a rather significant issue with re-fetching/re-painting of the advanced network image (ANI) when used as the image within a BoxDecoration. My app has numerous scrolling lists, with most having a 'leading' image (ANI). On one of those lists, I am using a delivered CircleAvatar. What I was seeing (using the Flutter Performance/Repaint Rainbow, etc...) was that the ANI is constantly fetching and re-painting its image. Upon digging into this, I found that the CircleAvatar is really just a Container with a BoxDecoration of BoxShape.circle. Regardless, I found that the ONLY time this issue occurs is when the ANI is used as the image of a DecorationImage, within a BoxDecoration, within a Container. I produced a sample page with only a single container on it - no lists, or scrolling, or anything.  When the image is set to an ANI, it goes crazy, constantly fetching/painting the image. If the image is set to a standard NetworkImage, there is a single fetch/paint. And, if the ANI is removed from the BoxDecoration, and instead is used as the Child of the Container, then there is only a single fetch/paint.
Here is the entire code that demonstrates this issue:
import 'package:flutter/material.dart';

import 'package:flutter_advanced_networkimage/provider.dart';

void main() {
  runApp(MaterialApp(
    title: 'Flutter Example',
    theme: ThemeData(primaryColor: Colors.blue),
    showPerformanceOverlay: true,
    home: TestPage(),
  ));
}

class TestPage extends StatefulWidget {
  @override
  _TestPageState createState() => _TestPageState();
}

class _TestPageState extends State<TestPage> {
  @override
  Widget build(BuildContext context) {
    ImageProvider avImg = AdvancedNetworkImage(
      "https://flutter.io/images/flutter-logo-sharing.png",
      retryLimit: 2,
      retryDuration: Duration(seconds: 5),
      disableMemoryCache: true,
      useDiskCache: true,
      loadedCallback: () {
        print('got photo');
      }, //here just for demonstration purposes
    );
    //avImg = new NetworkImage("https://flutter.io/images/flutter-logo-sharing.png");

    return MaterialApp(
      title: 'Re-paint/fetch Demo',
      home: Scaffold(
        appBar: AppBar(
          title: Text('Re-paint/fetch Demo'),
        ),
        body: Center(
          child: Container(
            width: 100.0,
            height: 100.0,
            decoration: new BoxDecoration(
                image: new DecorationImage(
                  image: avImg,
                ),
                color: Colors.deepPurple,
                shape: BoxShape.circle,
                border: new Border.all(
                  color: Colors.deepPurple,
                  width: 0.5,
                )),
          ),
        ),
      ),
    );
  }
}

Please let me know if I am doing something wrong. Please note that in true practice (not this sample page) I use a FallbackAssetImage, as well as a RetryLimit (2) and a RetryDuration (5 secs). That is why you will see the additional options of DisableMemoryCache (true) and UseDiskCache (true). Both of those were put in on @mchome advice to overcome some other issue with multiple/different cached fallback images.  I have tried changing/removing these options just for this test - sometimes I get good results, but mostly not. I cannot find a pattern that always results in a single fetch/paint.

Hi, i wrote some comments in the source code before. I forgot to post that note to your original issue page, sorry. The disableMemoryCache is not recommended to use, because flutter will call (all kinds of) imageprovider a lot.
In your code, if you remove the disableMemoryCache option, the overlay will stop, it is the same as flutter's NetworkImage.

That did the trick!  Thank you again for a great product.


Time series tooltip bug

It's a weird bug with a timeline tooltip when chart data is loaded dynamically:
http://jsfiddle.net/untit1ed/z1duqgt3/2/ - run the fiddle and mouse over the chart to show the tooltip.
My guess what's going on:
When there's a timeline chart with multiple data sets and some data sets have no value for a specific time then the tooltip stops working properly, it gets stuck on the right side, showing only one dataset value. If all x values are present in every dataset then it works fine - http://jsfiddle.net/untit1ed/xdaff593/
Updated the fiddle, so the bug is reproduced on every run.

:+1: Confirming.

Any updates on this? We're seeing this as well.

Any updates on this?

I think I figured it out. I think if you are missing some time-data one of the data sets, it should be treated as a different x-axis. So use xs. It worked for me. :smiley:


Add MiniZincIDE v2.0.13

Adding a new cask

[x] Checked there aren’t open pull requests for the same cask.
[x] Checked there aren’t closed issues where that cask was already refused.
[x] When naming the cask, followed the token reference.
[x] Commit message includes cask’s name.
[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.
[x] brew cask install {{cask_file}} worked successfully.
[x] brew cask uninstall {{cask_file}} worked successfully.

Thank you for the contribution. It needed some fixes, so they were made in 4edf8875cf1670c263dd0c80165794eb6f0f972c. Your contribution is still included (and still credited to you), with the appropriate modifications. Please feel free to ask about any of the changes.


Docker 1.7.1

I disabled the bottles section because I don't know what that does and it seems to work without it. Let me know what to do if I should re-enable them.

You can delete that section entirely. It refers to Homebrew's binary packages.

Merged in https://github.com/Homebrew/homebrew-versions/commit/2b8fa8d6c3977aecc45e527dc7e99f781f361f6f. Thank you for your contribution to Homebrew, we appreciate it! :smiley_cat:.
Just a nit, but the commit message for new formula should be styled docker171 1.7.1 (new formula), Thanks!


Do not trust the 2D/3D tag in ctab mol files

Sometimes, mol files may have an incorrect 2D/3D tag. RDKit's current behavior is to always honor a "3D" tag, but check the Z coordinate and eventually override the tag if it is "2D".
This PR changes this behavior to distrust the tag, and always assign the conformer dimensionality flag depending on the Z coordinates and the chirality marks on the input.
Basically, the new behavior would be to honor what we see in the Z coordinates, with the exception of all-zero coordinate mols, where we also check the existence of 2D stereo marks (i.e., bond directions) to decide whether the structure should be marked 2D (so that we honor the stereo marks) or 3D (so that assignStereochemistryFrom3D() can be used if changes are made to the mol).

To make sure I understand: the only change in behavior here is if the conformer is labeled as 3D but all the z coordinates are zero and there's a 2D stereo marker?

I can merge this after the conflict is resolved

I can merge this after the conflict is resolved

Should be solved now


Solution of Search in Rotated Sorted Array added

I have added the solution of search in rotated sorted array in Arrays folder

@Mayankrsmk Thank You for your contribution


Implement shortcuts

Description
Based on research conducted on ticket #597, please implement the following list of shortcuts:

Alt + M Toggle the metadata panel
Alt + R Toggle the report panel
Alt + S Toggle the source panel
Ctrl + Z Undo last change
Ctrl + Y Redo last change
Ctrl + Q Revert the changes
Ctrl + S Save the changes
Enter - Confirm action
Esc - Cancel action

Hi @romicolman I see a problem with Ctrl+Q, this is the default shortcut for closing applications. If you do it on Linux it closes the app. Maybe we can find another shortcut for reverting the changes?

Hi! Let's implement the rest and omit the Ctrl+Q for now

Hey! I tested it again today. Apart from ENTER and ESC, shortcuts do not work on Mac.

Hey! I tested it again today. Apart from ENTER and ESC, shortcuts do not work on Mac.

We have decided to merge this as it is working for windows and Linux and will create a ticket for Mac in the future.


trackEvent does not handle null values in JSONObject

If I call trackEvent in MixPanelAPI with a JSONObject containing a null value the method throws an exception. This is caught internally but the result is that the event is not logged to MixPanel.
Line in question is here. properties.get(key) will throw a JSONException if the value is null. The caller to this method could validate input but it seems preferable to handle the nulls and track the event instead of swallowing the exception and not logging the event.
E/MixpanelAPI: Exception tracking event test-event
  org.json.JSONException: No value for key_with_null_value
  at org.json.JSONObject.get(JSONObject.java:354)

Oddly enough, I can only make it throw an exception on devices running Android 4.3 (I used a HTC One). Anything higher than that works, which is perplexing but maybe something changed in implementations somewhere along the way.
Thank you!

@markcerqueira Hey, sorry for the delay!!
If I understand it correctly, what you want is to skip the properties with nulls in our library instead of filtering them out in the caller? That makes a lot of sense. I'll get it done when I get a chance!

@yinfeiru Yep! The current behavior throws an Exception and the entire event is not logged. Unsure if the best behavior is to skip the property altogether or place in a blank String. I'll defer to you all on that call.
Thanks and cheers! 😄

@markcerqueira To me, I feel like skipping the property is the way to go because on Mixpanel you can easily filter out null properties with is not set filter. But I'll discuss with other folks.
Cheers!

@markcerqueira Hey, sorry for the late. The fix will be available in the next release. PR is here.

@yinfeiru Awesome! Thank you!


Add Spinner and disable sign on form while authenticating.

Signon Screen should be disabled after authentication submission.

Fixed


Why the tutorial website is down?

Summary
Dear All,
I am trying to access the tutorial  website as I am currently in the last step but it gives error 404, Please advice
https://www.carbondesignsystem.com/tutorial/react-step-5
Relevant information

The URL seems to have changed slightly:
https://www.carbondesignsystem.com/tutorial/react/step-5

I have the same issue


build: bump java core to 9.20.0

PR summary
Bumps the java core version to 9.20.0, and tweaks the scripts used to publish javadocs slightly so that they consistently use the TRAVIS_TAG environment variable instead of a combination of TRAVIS_TAG and TRAVIS_BRANCH.
PR Checklist
Please make sure that your PR fulfills the following requirements:

[ ] The commit message follows the Angular Commit Message Guidelines.
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

Current vs new behavior

Does this PR introduce a breaking change?

[ ] Yes
[ ] No

Other information

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


Fix User definition (index.d.ts)

Fix issue https://github.com/IdentityModel/oidc-client-js/issues/746

thanks

1.7.0-beta.1 has been published to npm, please test and see if it's working for you.


uboot: 2024.04 -> 2024.07

Description of changes
Build-tested all the uboots in nixpkgs.
Run-tested:

Raspberry Pi 4 (@Svyatoclav)
Orange Pi 5 (@K900, patched to boot SATA)
Orange Pi 3B (@K900, out of tree board)
CM3588 NAS Kit (@K900, out of tree board)

Things done

Built on platform(s)

[ ] x86_64-linux
[x] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Tested on Rock 5 Model B and NanoPC-T6.


Antenatal Bartter syndrome [NTR]

Preferred term label:
Antenatal Bartter syndrome
(ORPHA:93604)
Synonyms
Bartter syndrome, furosemide type
Bartter syndrome, furosemide-amiloride type
Hyperprostaglandin E syndrome
Definition (free text, please give PubMed ID, if applicable, in format PMID:#######)
"A phenotypic variant of Bartter syndrome presenting antenatally with maternal polyhydramnios, pre-term delivery and postnatally with polyuria, and nephrocalcinosis. Hypokalemic alkalosis, increased levels of plasma renin and aldosterone, low blood pressure and vascular resistance to angiotensin II are characteristically associated. Genotypically they comprise Type 1 and Type 2 Bartter syndrome" from Orpha:93604
Parent term (use OLS, or your favorite ontology browser)
MONDO:0015231 Bartter syndrome
Children terms (if applicable) Should any existing terms that should be moved underneath this new proposed term?
Bartter syndrome types 1, 2 and 5 (MONDO:0011127; MONDO:0009424, MONDO:0010503)
Your nano-attribution (ORCID)
If you don't have an ORCID, you can sign up for one here
Note: this is another example for #2693 (the Orpha:93604 is currently lumped in with subtype 1_

@kanems looks like we already have this term in Mondo - I will look more closely at your ticket later though to make sure I'm properly answering the question here
MONDO_0011127 'Bartter disease type 1'


test(borrow): add a test to ensure borrow works

Ran into this when trying to remove some unsafety from hyper.
This works on stable but seems to be a regression on beta and nightly. Placing this here for easy reference/repro.
src/lib.rs:142:17: 142:68 error: cannot infer an appropriate lifetime due to conflicting requirements [E0495]
src/lib.rs:142             map.get::<UniCase<Cow<'key, str>>>(&UniCase(fn_borrow))
                               ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/lib.rs:140:9: 143:10 help: consider using an explicit lifetime parameter as shown: fn get_shorter<'key>(map: &'key HashMap<UniCase<Cow<'static, str>>, u32>,
                     s: &'key str) -> Option<&'key u32>
src/lib.rs:140         fn get_shorter<'map, 'key>(map: &'map HashMap<UniCase<Cow<'static, str>>, u32>, s: &'key str) -> Option<&'map u32>{
src/lib.rs:141             let fn_borrow: Cow<'key, str> = s.into();
src/lib.rs:142             map.get::<UniCase<Cow<'key, str>>>(&UniCase(fn_borrow))
src/lib.rs:143         }
error: aborting due to previous error

Note that it works if you adjust the lifetimes to the suggestion, but that would limit the return value's lifetime to the lifetime of the key which would be undesirable for many cases.

Is this testing something specific about UniCase, or just lifetimes and Cow?

The main intention was to test that Borrow actually worked through unicase, ensuring it works with HashMaps, but you're right in that it's not a very specific test. The verdict on the rust issue seems to be that it's a soundness issue and it's unlikely to compile as-is in the future.

Closing for now, don't think there's a good way to do this currently.


where is the "flow magnitude" frame?

How do I get the "flow magnitude(pixel)" as I could see in this following video?
300 Hz Real-time Optical Flow. ANU campus

I have the same question, how to go to pixel space for the flow magnitude?

In Python, you can use numpy.linalg.norm(flow, axis=2) to get the flow magnitude.


Not able to Install in Mac

Getting this error on my Mac
meson x --buildtype release --strip -Db_lto=true
Error during basic setup:
Neither directory contains a build file meson.build.

See #35

My Bad..it's working now. Thanks ☺


Using_to_llama_similarities() normalization technique may lead to low quality retrieved documents / chunks.

Question Validation

[X] I have searched both the documentation and discord for an answer.

Question
Because of _to_llama_similarities function, the scores are normalized and maximum score is always at 1.0
What is the reason of this normalization technique, and why has this being called by default?
The problem may arise when some query which is not related to a particular document, is used for retrieving the chunks, and still the results will contain scores starting from 1.0 , even when there is no relation of the chunk with respect to the query.
Code snippet:
return VectorStoreQueryResult(             nodes=top_k_nodes,             ids=top_k_ids,             similarities=_to_llama_similarities(top_k_scores),   )
Line no. 623 in

https://github.com/run-llama/llama_index/blob/main/llama-index-integrations/vector_stores/llama-index-vector-stores-elasticsearch/llama_index/vector_stores/elasticsearch/base.py

Isn't this a bug? Why was this specifically made to keep the max value at 1.0?  say we are getting top 5 docs from a query. And the doc at k=1 need not be of score 1.0, it can be of score 0.85  The problem with this is we are giving high scores for those documents which are not that similar to the query.
I had posted similar question earlier but did not get a satisfactory reply.

Please make a PR if you find this to be a bug. I agree, but don't have elastic to test with

as long as they are sorted most to least similar, I don't have a preference


Hp Chromebook 11a G6 ee

Describe the bug
It was installed but didn't work at all. (I'm using ubuntu 23.10)
Boardname
Barla
Logs
debug-logs.tar.gz

Did you not see the warning that says you need a custom kernel?

No.

ubuntu makes me :(


Non-Encryption Subkeys May Be Used to Encrypt Data

This issue was discovered during the audit, see "L5" in https://delta.chat/assets/1907-otf-deltachat-rpgp-rustrsa-gb-reportv1.pdf:
"The RPGP library allows for messages to be encrypted with sub-keys which do not have the encryption key flag set. This may result in users losing the ability to decrypt data, due to loss of a signing key accidentally used for encryption. Additionally, government laws may legally require users to disclose their signing key, as it was accidentally used for encryption."
"The assessment team recommends adding checks to restrict operations that a key can be used for, based on its keyflags."

I am having the same issue, messages encrypted with rpgp cannot be decrypted by Thunderbird because a non-encryption key was used. Here is the used key is not marked for encryption use warning from the gpg and the public key details:
% gpg --decrypt pgpmail.txt
gpg: encrypted with rsa4096 key, ID 44A05CC7A6D9F6B3, created 2023-09-07
      "John Doe<EMAIL_ADDRESS>gpg: used key is not marked for encryption use.
Content-Type: text/plain; charset=iso-8859-1
Content-Transfer-Encoding: 8bit

Zw?lf Boxk?mpfer jagen Viktor quer ?ber den gro?en Sylter Deich.

% gpg --list-keys
[keyboxd]
---------
pub   rsa4096 2023-09-07 [SC] [expires: 2026-09-06]
      EB3B2F84411BCFEBCD85EB5744A05CC7A6D9F6B3
uid           [ unknown] John Doe<EMAIL_ADDRESS>sub   rsa4096 2023-09-07 [E] [expires: 2026-09-06]

me@stalwart /tmp % gpg --with-fingerprint --import-options import-show --dry-run --import ~/Downloads/John\<EMAIL_ADDRESS>pub   rsa4096 2023-09-07 [SC] [expires: 2026-09-06]
      EB3B 2F84 411B CFEB CD85  EB57 44A0 5CC7 A6D9 F6B3
uid                      John Doe<EMAIL_ADDRESS>sub   rsa4096 2023-09-07 [E] [expires: 2026-09-06]

gpg: Total number processed: 1


Fetching FileHandleStatus first and FileStream after leads to NPE

Whenever you request a file and try to get the FileHandleStatus and after that the FileStream to i.e. compose a response with both combined, there will be a NPE thrown at org.lokra.seaweedfs.core.Connection.fetchStreamCacheByRequest(Connection.java:328).
public class SeaweedFile {
	@JsonProperty
	public FileHandleStatus status;
	@JsonProperty
	public String data;
}

@GET
@Path("{id}")
public Response read(@PathParam("id") String fileID) {
	Response response = null;
	FileTemplate template = new FileTemplate(fileSource.getConnection());
	try {
		SeaweedFile file = new SeaweedFile();
		file.setStatus(template.getFileStatus(fileID)); // first get the FileStatus  *X

		StreamResponse fileStream = template.getFileStream(fileID); // *Y
		InputStream is = fileStream.getInputStream();
		file.setData(new String(ByteStreams.toByteArray(is)));
               // do return stuff ...
       } ...
}

If you switch lines marked with *X and *Y, the request will return smoothly.

@rnorden
I think the problem is caused by HttpClient cache policy, this client will using HEAD request when you get the file status (no content, but save at response cache), and then client fetching content using GET request, but HttpClient found this request at cache.
I have been busy recently and I hope someone can help me fix this problem.

fixed #3 at version 0.7.3.RELEASE.


DOC: actually document float_precision in read_csv

So I wasn't 100% correct when I said that float_precision was documented here.  It was well documented internally for TextParser and in a section for io.rst, but it wasn't listed formally in the parameters for the read_csv documentation.

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

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

@@             master     #13377   diff @@
==========================================
  Files           138        138
  Lines         50739      50739
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits          42740      42740
  Misses         7999       7999
  Partials          0          0

Powered by Codecov. Last updated by [b722222...ffca1ec][cc-compare]
[cc-base-branch]: https://codecov.io/gh/pydata/pandas/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/pydata/pandas/compare/b722222f5ea760a3f3df4d063309949eb4956674...ffca1ecd64325073365267aa1bb6c674ac42deca
[cc-pull]: https://codecov.io/gh/pydata/pandas/pull/13377?src=pr

ty


Python support

Add python language support. I've ported the NodeJS into Python and ran it successfully. Working on a PR now.

Look forward to it and thanks for putting the effort!

This was merged. Closing.


[chip dv] Add AST initialization routine

This commit adds the initialization routine to:

Backdoor load the AST configuration partion in OTP with data that will
be read by the test/mask ROM and written to the AST rega registers,
for the SW tests

Do the same for non-SW tests, but also manually write the AST
registers via the TLUL interface in dut_init(), before the tests
begin.

This is an alternate approach to the PR #11275.
Signed-off-by: Srikrishna Iyer<EMAIL_ADDRESS>

o i really like this. So from foundry side, they would just directly populate that array for the real data right?

o i really like this. So from foundry side, they would just directly populate that array for the real data right?

Yes that is correct.
I did a force push, which moves the creator_sw_cfg_ast_cfg_data from the base vseq to chip_env_cfg. You can extend it and modify this array however you want, either in your nuvoton_chip_env_cfg class that extends from chip_env_cfg class. This mechanism is exactly the same as how partner_flash_ctrl_env_cfg overrides the flash_ctrl_env_cfg class in open source.
I am also providing an easier alternative to update this array, via plusargs, such as +creator_sw_cfg_ast_cfg_data[34]=deadbeef. Plusarg lookup is on a word basis. You can set these plusargs in your sim cfg hjson. See update to chip_base_test for details.

sriyerg 25 minutes ago
@Jacob-Levy, are you saying this will not work, because your private repo has proper register names?
We can change this to tl_access instead, which only needs the correct start address.

The AST register's names has nothing to do with the OTP override. Only the offset are needed for the updates.
The real image of the OTP will include the calibrated AST register value and will be set per device after the clalibration process
extract the values on the tester.
For the different simulation enviroment, I need to be able to load/override those values to the OTP (generic or foundry) in a simple way.

sriyerg 25 minutes ago
@Jacob-Levy, are you saying this will not work, because your private repo has proper register names?
We can change this to tl_access instead, which only needs the correct start address.

The AST register's names has nothing to do with the OTP override. Only the offset are needed for the updates. The real image of the OTP will include the calibrated AST register value and will be set per device after the clalibration process extract the values on the tester.
For the different simulation enviroment, I need to be able to load/override those values to the OTP (generic or foundry) in a simple way.

This PR provides two very simple ways to override the values you would backdoor write to the OTP at runtime. This PR works for both, SW and non-SW tests. One way is with plusargs, the other is directly manipulating chip_env_cfg::.creator_sw_cfg_ast_cfg_data array directly. It is not clear why you keep insisting that it will not work.

I will merge this PR for now - we can make amends in future as needed.

As I mentioned before, the OS registers number and location are there to correctly program the XBAR asdress space for
the AST, The names of the registers are not the names of the real AST registers due to a security limitation,
Those registers in the OS model are not connected to any logic therefore do not affect any of the logic inside the
OS' AST. The only exception is REGAL that the write to it will trigger status change, ast_init_done. This means that the
AST registers must not be written randomly in the OS which it true also for the NTIL.
The only time that the AST registers can be written is in boot time from OTP/dut_init or later with secific data (not random).
So, for the OS model, the AST registers can be written only in boot time for AST REGA[0] offset to REGAL offset.
Be aware that REGAL offset will be updated once the the final version of the AST registers in NTIL is done.

@Jacob-Levy, @sha-ron, Are these statements true?

The OTP reserves 32 words for AST cfg, but the ast.hjson sets NumRegsA parameter to 31. This is a bug and will be fixed soon, so that they reflect the same value (after all the adjustments you make in your NTIL repo).
REGAL register will be immediately after the reg_a set of registers

I have made these assumptions and prepared a fix in #11376. PTAL, thanks. Here's the waveform from the non-SW CSR test:


macaroons: specify root key ID in bakery

With the current store, there is only one root macaroon key with ID 0 used.
The bakery should allow users to specify the root key ID that should be used when baking a new macaroon. This would allow for selective revocation of certain macaroons.
The following changes would be needed:

[ ] Add a new optional field root_key_id to the lnrpc.BakeMacaroon request. If no value is provided, the default value (defaultRootKeyID) should be used. If an ID is provided but there is no key in macaroons.db yet, a new random key will be created and stored in the DB for the given ID.
[ ] Add a new RPC operation ListMacaroonIDs that returns all IDs that are used. This operation should require the RPC permission macaroon:read.
[ ] Add new RPC operation DeleteMacaroonID that removes one specific root key ID, invalidating all macaroons derived from the key with that ID. This operation should require the RPC permission macaroon:write. The operation should fail if a user tries to remove the default macaroon root key ID 0.

I'm working on it.

Probably need some feedback before coding, a few questions/thoughts.

it looks like using context is the most feasible way to pass a RootKeyID param? I'm thinking attaching the ID value to a context, pass it to Oven.NewMacaroon,  and extract it within the function RootKey.

It seems that allowing customizing root key ID opens a pandora's box, as,

the current structure of macaroon db has only two keys, defaultRootKeyID and encryptedKeyID. If more root key IDs were to be added, it's better to create a nested bucket under the root, whose purpose is to store macaroon root keys only. Otherwise, any read/write action to a root key IDs would require an extra step to check we are not touching encryptedKeyID.
this also means we need to have migrations of macaroon db? Further, maybe macaroon db could be left to channeldb for management?

My thoughts are, either fix the db structure first then purse this issue, or pause db fix and add the aforementioned extra check against encryptedKeyID. Either is fine with me.

Thank you for investigating this. You are correct on both accounts.

it looks like using context is the most feasible way to pass a RootKeyID param? I'm thinking attaching the ID value to a context, pass it to Oven.NewMacaroon, and extract it within the function RootKey.

Yes, it appears that is the intention of the macaroon-bakery.v2 library. That is a common pattern in go, we also use that in aperture: https://github.com/lightninglabs/aperture/blob/master/lsat/context.go

the current structure of macaroon db has only two keys, defaultRootKeyID and encryptedKeyID.

That is unfortunate indeed. I wasn't fully aware of this when I created the issue. But I think we can get around this without migrating the database (migrations are always painful!).
The easiest option would perhaps be to require a custom key to have a length of 7 bytes or more. Then they couldn't collide with the encryptedKeyID key. But in any case, we'll always want to check we're not overwriting an existing key in RootKeyStorage.RootKey, so we might as well just add a check that it doesn't collide there.

Thanks for the feedback;)

The easiest option would perhaps be to require a custom key to have a length of 7 bytes or more. Then they couldn't collide with the encryptedKeyID key. But in any case, we'll always want to check we're not overwriting an existing key in RootKeyStorage.RootKey, so we might as well just add a check that it doesn't collide there.

Gotya, yeah I think it's a good idea to put some restrictions on the root key ID. Will look into it.


Adding support for s390x and ppc64le for multi-arch image build

Description
Provided support for s390x and ppc64le to create multi-arch image
How Has This Been Tested?
For creating multi-arch build image, use command make docker-buildx.  This will build multi-arch images and will push into your provided image repository
Request review criteria:

Self checklist (all need to be checked):

[X] The developer has manually tested the changes and verified that the changes work
[ ] Commits have been squashed into descriptive, self-contained units of work (e.g. 'WIP' and 'Implements feedback' style messages have been removed)
[ ] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[ ] The developer has added tests or explained why testing cannot be added (unit or cypress tests for related changes)

If you have UI changes:

[ ] Included any necessary screenshots or gifs if it was a UI change.
[ ] Included tags to the UX team if it was a UI/UX change (find relevant UX in the SMEs section).

After the PR is posted & before it merges:

[ ] The developer has tested their solution on a cluster by using the image produced by the PR to main

/ok-to-test

@dpanshug, @christianvogt , @lucferbux Can you please review the requested changes?

@dpanshug , @christianvogt  , @lucferbux  Can you please review the requested changes?


Fix ticker and retry loop in mixer initialization CRD watch

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: bianpengyuan
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: mandarjog
If they are not already assigned, you can assign the PR to them by writing /assign @mandarjog in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

mixer/OWNERS

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

@bianpengyuan: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
54dc27745eeb1d94b176320e2a04eb07dd9a0b90
link
/test istio-pilot-multicluster-e2e

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

/lgtm
/approve

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: bianpengyuan, geeknoid
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

mixer/OWNERS [geeknoid]

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

@bianpengyuan: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
54dc27745eeb1d94b176320e2a04eb07dd9a0b90
link
/test istio-pilot-multicluster-e2e

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

@bianpengyuan: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
54dc27745eeb1d94b176320e2a04eb07dd9a0b90
link
/test istio-pilot-multicluster-e2e

prow/istio-integ-k8s-tests.sh
54dc27745eeb1d94b176320e2a04eb07dd9a0b90
link
/test istio-integ-k8s-tests

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


Сan entt store a vector?

hello again :), I would like to know if it is possible to contain std::vector in my entity, just when I declare a vector in the entity class (I think you remember this class from my last problem), an error appears. And although the error generally says about another library (ImGui), but it seems to me that entt cannot store a vector

It's hard to help without a repro. Even more if you mention a generic error without specifying what you're talking about.
To be honest, I don't remember your implementation of the entity class. Nor do I know what it means to declare a vector in it.
However, I'm pretty sure that EnTT supports standard types on components as well as anywhere else.
As it stands, this issue is invalid. There is no way I can help here. I'm sorry.
Feel free to add more details. The best would be a repro though. I can't assure you that I can help in all other cases.

Ok, here is the full implementation of the "Entity" class: https://gist.github.com/0LGL0/27a588c0e877c8ed0edae196b4639614 . And when I add std::vector<uint32_t> m_group to this class (this vector should store the ids of child entities for this entity), an error appears, although, again, something like this is output in the console: there is no TreePop/PopStyle from ImGui or something like this (if it is not clear, then I apologize, now I'm not near the computer)

Wait, is this the same entity that you use with the registry? As in using my_registry = basic_registry<Entity>;?

Let me rephrase it then 🙂 what's m_reg? Can you tell me the exact type of this member?

m_reg is a type of entt::registry, sorry I didn't understand right away

Ok, I solved this problem by simply creating a new component "GroupComponent", now everything seems to work.


Chrome 108: Large, Small, and Dynamic viewport units 🎉

https://twitter.com/intenttoship/status/1574349759590146049
https://groups.google.com/a/chromium.org/g/blink-dev/c/Qw2XTr0et_U/m/6eucoI_AAQAJ

Estimated milestones
M108

https://tests.caniuse.com/viewport-unit-variants:

🎉
Note:
WPT using the Interop 2022 viewport filter is currently indicating 107, but that's probaby because of

With --enable-experimental-web-platform-features

Thanks!


[18.09] bump libnetwork 872f0a8

carrying #200 for seb
full diff: https://github.com/docker/libnetwork/compare/c9029898e32f7c89bbb81511fbb721df252ce61a...872f0a83c98add6cae255c8859e29532febc0039

docker/libnetwork#2354 [18.09 backport] Cleanup the cluster provider when the agent is closed

backport of docker/libnetwork#2307 Fix for problem where agent is stopped and does not restart
fixes docker/for-linux#495 Docker swarm overlay networking not working after --force-new-cluster

docker/libnetwork#2369 [18.09 BACKPORT] Pick a random host port if the user does not specify a host port

backport of docker/libnetwork#2368 (windows) Pick a random host port if the user does not specify a host port

Signed-off-by: Sebastiaan van Stijn<EMAIL_ADDRESS>

restarted windows RS 1, which was failing on https://github.com/moby/moby/issues/38521 (flaky test)


canceling the cancel confirm doesn't reenable the expanded level card buttons

The level card buttons are disabled when a cancel confirmation is created.  They should be reenabled when you decide not to cancel after all but they are not.  [Ignore color of the pnotify, I was experimenting when I found it, so it will look different once I finish experimenting]

@jennyhavah Disabling these buttons while the confirmation notice is active wasn't in your original spec but it seemed like a good idea (you may notice that I also disabled the edit buttons on all of the collapsed cards). It's to prevent any other action being taken on the card before the confirm is resolved.  I've removed the disabling for now because I realized I need to do it in a different way, however, if you'd prefer to have the buttons remain active, I can leave them active.

Great catch. I agree that all actions should be disabled in the card while you're addressing the confirmation prompt.

@sanjuroj As per our conversation just now: We also need to delete all saved level buttons (Delete, Edit) and indicator links while you're interacting with an "Are you sure you want to delete this result level" prompt.
I just got into a weird state where I rapidly deleted a bunch of result levels, and a single prompt is stuck on the screen:

@jennyhavah Unfortunately, I read your note after coding this up.  I didn't delete the links and buttons while the confirm was showing, I just disabled them.  Let's see how this seems to you, if you want them removed, it's not much more work to do so.

@sanjuroj I totally meant to say "disabled", not "deleted".
Just checked on QA and don't see the change. Are you sure it's merged?

Initiating delete or cancel (i.e. edit one card and try to edit another without saving) should disable all buttons now.

@sanjuroj Found one leetle edge. I can start editing and indicator and then click Delete on another. Clicking Cancel doesn't let me back out. I'm stuck.
While we do want to keep all the Edit buttons active so you can jump from one to another, what do you think about disabling Delete buttons if you've got a level open?

@sanjuroj When I attempted to delete an RF level, the Cancel button didn't work.
The "X" button does, though.

PASSED. The cancel button now works when I click "Delete".
While I am interacting with a Pnotify confirmation dialogue, all Edit and Delete buttons are disabled.

@sanjuroj  Not sure if it's always been this way or if I'm just now noticing, but...
If I edit an RF level and click Edit on a different RF level, I can still type all I want in the first form's two text boxes.
Should we disable those as well, until you make a decision whether to proceed or cancel?

I don't remember having disabled those, but yeah, I think we probably should.  It's a pretty easy change, but I wouldn't say it's low risk for the demo.  Shall I do it after the demo?

It can definitely wait until after the demo. Thanks!

This is what it looks like when I disable the assumptions and name fields when the cancel prompt is up.  The weird thing is that the indicator box looks enabled (even if the drag handles and ordering dropdowns are disabled.  Should that whole section get the disabled treatment?  And I'm assuming the program objectives dropdown should be disabled as well?

@sanjuroj Yes -- anything we don't want users interacting with, we should disable.

Looks great. Everything is inactive while I'm interacting with the warning dialogue. Here's what it looks like:


The library should model that a Group induces a Groupoid

Like a Semigroup induces a Semigroupoid, a Group induces a Groupoid.
The first relation is captured by the instance: Semigroup m => Semigroupoid (Semi m) (link to Hackage docs), but there is no such relationship defined. It would be something like:
instance Group m => Groupoid (Semi m) where
  inv (Semi n) = Semi (groupInv m)

I'm also missing a definition of Group among your packages.
I'm not asking this just so, but because I've run into an application where some concepts are naturally modeled using groupoids and groups (even though I'd need a slightly different definition).

algebra supplies a group class, but is rather deeply unsuitable to this purpose.


Indexer Pipeline

Summary
This PR implements a Jenkins pipeline that can do the indexer builds moving forward.
Test Plan
I ran the Jenkins pipeline and saw the binaries get created.

Codecov Report

Merging #624 (096ccea) into develop (618c5ea) will decrease coverage by 0.13%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop     #624      +/-   ##
===========================================
- Coverage    49.82%   49.68%   -0.14%
===========================================
  Files           25       25
  Lines         3952     3947       -5
===========================================
- Hits          1969     1961       -8
- Misses        1702     1705       +3
  Partials       281      281

Impacted Files
Coverage Δ

cmd/block-generator/generator/generate.go
71.37% <0.00%> (-1.49%)
:arrow_down:

cmd/block-generator/generator/make_transactions.go
100.00% <0.00%> (ø)

Continue to review full report at Codecov.

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


"Abort format on save after 750ms" happens after write 200 lines of code

Issue Type: Bug
I notice that autopep8 works well in my VScode when the number of code lines is below around 200. However, when the code lines is over 200, the auto-format fails. This problems happens both in my Mac and PC.
VS Code version: Code 1.28.0 (431ef9da3cf88a7e164f9d33bf62695e07c6c2a9, 2018-10-04T16:40:40.180Z)
OS version: Darwin x64 18.0.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) M-5Y51 CPU @ 1.10GHz (4 x 1200)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: enabledrasterization: enabledvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
1, 3, 4

Memory (System)
8.00GB (0.04GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (11)

Extension
Author (truncated)
Version

code-gnu-global
aus
0.2.2

jupyter
don
1.1.4

code-runner
for
0.9.4

nbpreviewer
jit
1.0.0

python
ms-
2018.8.0

cpptools
ms-
0.19.0

java
red
0.32.0

vscode-java-debug
vsc
0.13.0

vscode-java-pack
vsc
0.4.0

vscode-java-test
vsc
0.9.0

vscode-maven
vsc
0.11.0

(2 theme extensions excluded)

This is a performance issue with the formatter. Pls file an issue against https://github.com/Microsoft/vscode-python/issues. Thanks!


How does raw_to_Xy() know which column to use to calculate returns

The documentation says

computation of returns (if use_log then logarithmic else simple) - the first timestep is automatically deleted

How does this function know which column is the closing price and therefore which column to use to calculate returns

Hey there! Thank you for your interest.
It is going to compute returns (percentage change) for all of your indicators (e.g. open price, close price,  volume) that are in the input DataFrame.
Hope that helps.

Thank you for the fast reply!
So if i wish to calculate portfolio returns, would you reccomend creating a dataframe that has the percentage returns from closing prices first?

AFAIK taking the closing prices is the  most common approach - I don't really remember the rationale behind it though. However, it is really up to you:)


SQL generation of CLOB is wrong on oracle

If I generate an insert statement of a row with a CLOB value, I get [CLOB] as the value of that cell in the generated script. The script is correctly generated only when I double-click the cell in the grid in order to show the value

By default LOB values aren't read along with regular column values (because they can be huge).
You always can get correct SQL by using "Export ResultSet" function.
Or you can enable LOBs caching (Preferences->Database->Result Sets->Binaries->Cache CLOB Values).

It's ok not to read LOBs along with the rest of the column, but if I have to perform an insert statement in another database, I should be able to copy and paste, also with a LOB


Use tracing instead of log

Before:

After:

Why did you use tracing instead of log?

Why did you use tracing instead of log?

I'd like to add more more instrumentation to the methods to make it easy to build out a tui for mastodon (https://github.com/joshka/tooters). The approach in tracing crate is better for this than log crate. E.g. the #[instrument] attribute, and the ability to log values as the value and not just as a text string (log does have the kv feature, but it's more difficult to support and not as powerful as the values approach). Filtering specific logs and directing them is also easier in tracing. Tracing makes it possible to more easily see which network calls are part of which operation using the concept of spans around groups of calls.
Tracing can be configured to send events to to log (and also the reverse), so there's no real downside to this and a bunch of upside.

OK, thanks.