Sidebar not scrollable anymore with Giscus comment presented at after body.

Describe the bug
As I described in the title. I find that when 1) the left sidebar (possibly right sidebar as well, but haven't test it) has content exceeding the current viewable area and 2) when the giscus comment area presents. We lost the ability to scroll on the left side bar.
To Reproduce
Steps to reproduce the behavior:

extend your sidebar so that it doesn't fit one page
enable the comment
try to scroll, either on the main area or on the sidebar
See that the side bar stops moving even it is not depleted yet.

Expected behavior
I would expect  either

if the main content is depleted, the sidebar should start scrolling down (which was the case when no giscus widget has been loaded yet.
or if I place my cursor on the sidebar or click on that area, the focus should shift to the area and scroll that part.

Screenshots and Source
I can provide upon request.
You can visit my site www.lumeny.io for a look at the problem. A fun finding is that the sidebar is in a short period of time scrollable before the giscus component is loaded (feel free to try on my site). My quartz codebase is at here.
Desktop (please complete the following information):

Quartz Version: v4.4

Additional context
Thanks for this amazing work! I can't think of blogging to be sooo rewarding if not with quartz.

Thanks for reporting. I'll investigate tomorrow.

Thanks! I'll be looking forward to see it fixed!

Thanks! I'll be looking forward to see it fixed!

Hey, I have posted a potential fix, can you check if https://github.com/jackyzha0/quartz/pull/1574 fixes your issue?

Hi @saberzero1, thanks for the prompt fix.
I manually applied #1574 and it works! Now my site is working properly with the giscus!
Thanks for the help and I'll close this issue.

Hey @LumenYoung, I'm glad it fixes your issue.
I'm going to reopen this issue until the proposed fix has been merged into Quartz for all users to prevent confusion for others running into this issue.


this project is disfunctional and dead ;)

Yeah, I know that. I had 2 versions and uploaded the wrong one. I'm working on it. Probably final february I'll up a working version.

Well, a year latter.. here we are, a first version, working.


ng xi18n silently fails to extract tags

Bug Report or Feature Request (mark with an x)
- [x] bug report -> please search issues before submitting
- [ ] feature request

Versions.
Angular CLI: 1.5.0
Node: 8.7.0
OS: win32 x64
Angular: 5.0.0
... animations, common, compiler, compiler-cli, core, forms
... http, language-service, platform-browser
... platform-browser-dynamic, router

@angular/cli: 1.5.0
@angular-devkit/build-optimizer: 0.0.32
@angular-devkit/core: 0.0.20
@angular-devkit/schematics: 0.0.35
@ngtools/json-schema: 1.1.0
@ngtools/webpack: 1.8.0
@schematics/angular: 0.1.1
typescript: 2.4.2
webpack: 3.8.1

Repro steps.
ng xi18n --verbose > xi18n.log
The log given by the failure.
there is no log generated by ng
Desired functionality.
a log should be generated containing the errors or at least they should be logged to console. my project builds. but extracting i18n from the templates silently fails. i created a repro with an app component with i18n tags and the libraries i use.
Mention any other details that might be useful.
https://github.com/JanEggers/repro

i tracked it down to ngx-bootstrap if i comment that out in vendor.ts messages.xlf is generated.
@hansl can you please have a look / verify my findings. and find out whats causing the error and why it is not shown?

Perhaps a duplicate of: https://github.com/angular/angular-cli/issues/8065

related because i dont get an error.
but it is no duplicate because my packages are uptodate and ng build works without errors

Yes it's a duplicate, my issue was similar: it would build just fine but not extract because of an error only in the extraction mode. And I believe that the solution would be the same so I'm closing this one, please follow up on the other issue.


Allows seeking to zero

We should be able to restart the currently playing audio by setting this.audio.seek(0)

Hi, thanks  a lot for making this PR. But I think you need to add one more check
if (pos || pos === 0) {
  ...
}

Otherwise, you still can't seek to 0

Ah thanks for pointing that out: I had been hacking it with this.audio.seek(1e-12) so I didn't catch it :-)

Thanks. I'll cut a release tonight


feat: improve box render

We're currently processing your upload.  This comment will be updated when the results are available.


View insertions, deletions, and files changed by commit

Is your feature request related to a problem? Please describe.
I really like the command for querying author contributions:
SELECT author_name, COUNT(author_name) AS commit_num FROM commits WHERE datetime > "2024-01-01 00:00:00" GROUP BY author_name ORDER BY commit_num DESC
I especially like that this can be run across many repos at once.  That is an excellent and unique feature.
However, from this I cannot view information about insertions, deletions, and files changed.
I see that there is a diffs table, but it cannot be filtered by datetime or branch.  It is also very expensive to query.  I also don't see a way to join commits to diffs.
Describe the solution you'd like
Include insertions, deletions, and file count in commits table.
Describe alternatives you've considered

Include a way to filter diffs by date and branch.
Include a way to join commits and diffs

Additional context
Add any other context or screenshots about the feature request here.

Hello @Danation,
Thanks you suggestion, Now i added datetime to the diffs table and also you will be able to join commits with diffs using commits id, author name...etc, but still working on ON expression for join
Thank you

Solved on 0.25.0


Bug in Text splitting while using langchain.text_splitter.split_text_on_tokens¶

System Info
Langchain Version = 0.0.311
Python Version = 3.9
Tried it on my local system as well on Company's hosted Jupyter Hub as well
Who can help?
@eyurtsev @agola11
Information

[X] The official example notebooks/scripts
[X] My own modified scripts

Related Components

[ ] LLMs/Chat Models
[ ] Embedding Models
[ ] Prompts / Prompt Templates / Prompt Selectors
[ ] Output Parsers
[X] Document Loaders
[ ] Vector Stores / Retrievers
[ ] Memory
[ ] Agents / Agent Executors
[ ] Tools / Toolkits
[ ] Chains
[ ] Callbacks/Tracing
[ ] Async

Reproduction
from langchain.text_splitter import TokenTextSplitter
token_splitter_model_name = "gpt-3.5-turbo"
SPLIT_CHUNK_SIZE = 1024
CHUNK_OVERLAP = 256
text_splitter = TokenTextSplitter.from_tiktoken_encoder(model_name=token_splitter_model_name, chunk_size=SPLIT_CHUNK_SIZE , chunk_overlap = CHUNK_OVERLAP)

blog_content= ' your text here'
blog_splits=text_splitter.split_text(blog_content)

Expected behavior
The ways this token text splitter works , isn't how it is intended to work .
For Example :-
When specified chunk_size = 1024 and overlap = 256 with input text tokens = 991 , it made two chunks ;
first = token[0 : 991]
second = token[768 : 991]
but logically it should work this ways .
If input text token >1024 , there should be two chunks .
first = token[0 : 1024]
second = token[768 : ]
and so on ..............
If input text token <=1024 , there should be one chunk only .
first = token[ 0 : ]
More details on this issue raised previously :- https://github.com/langchain-ai/langchain/issues/5897

I am working on a PR for this issue. It should be up within 24 hours.

@sweetkane nice . good to go . I have resolved this issue in my codebase .


Added TagBots and todo

fixing #3

@Jay-sanjay coming in with another PR! Wowzers! I'll take a look at this ASAP! Thanks!

Hey @Jay-sanjay ! This is ready to go! I removed the todo.yml workflow as I actually decided to deprecate that across JuliaHealth. I just hadn't mentioned it anywhere yet and I apologize for not letting you know. Otherwise, let's merge this! :smile:

Thanks ! for that :)


Update TooT_T.R

To accommodate the special case where the sequence length is shorter than 31.

I somehow missed that there was an actual PR, but had added exactly this after our email exchange.


Feature Request: Add a compilation blacklist of sorts

Incremental compilation is super cool. Pyro is a cool tool that already makes this happen.
However, it does this by parsing the .ppj file and feeding Caprica curated old-school command-line options. Although Caprica can now work with .ppj files itself, this feature is redundant because bypassing that functionality is the most optimal way to use Capria.
If Caprica had a way of blacklisting already-compiled scripts, though, that'd change things. Pyro could pass Caprica the .ppj and the blacklist, and that'd be that.
Alternatively, Caprica could handle that on its own, though this would be redundant and a pain to implement.

One issue is Pyro's remote Import feature. There'd be no "good" way to slot that into Caprica anywhere.


Add an unique id for each unclaimed fund/state.

This is a comment from the audit-team.

https://github.com/zkemail/email-wallet/blob/aa7892bc1f96a5c11a8679c55e06933c36f40c30/packages/contracts/src/handlers/UnclaimsHandler.sol#L237
since UnclaimsHandler.registerUnclaimedState can be called by anyone, a transaction could be front-run such that require(unclaimedStateOfEmailAddrCommit[recipientEmailAddrCommit].sender == address(0) will fail in https://github.com/zkemail/email-wallet/blob/aa7892bc1f96a5c11a8679c55e06933c36f40c30/packages/contracts/src/handlers/UnclaimsHandler.sol#L292, right?

To protect transactions to register unclaimed funds/states against the front-run attacks, this PR allocates a unique id for each unclaimed funds/states based on the total number of registered ones and allows users to register duplicated email address commitments at the same time.
I updated the test codes and confirmed that all tests except for integration tests passed.
After I confirm that all integration tests pass as well, this PR is ready to merge.

All integration tests passed.
It is ready to merge.


Authorization bug

After the user authorizes pushbullet, he's redirected to a error page with this text "api pushbulle {relative_path} pushbullet settings not found"

Resolved in a0ac685, thanks!


Provide getAsset method

@juliawarnke I used the getAsset method throughout the code now. If you want to overwrite it, keep that in mind.
Does that align with what you wanted to do with it?


Fix: cannot hide admiralty-checkbox label to be accessible

This fixes the issues listed below. I've introduced a new property called labelHidden which will allow the user to visually hide the label text, beside the checkbox and still be accessible (read by screen-readers). Also whilst working on this issue, I've also fixed the behavioural differences when clicking inside and outside the checkbox.

[BUG] ARC Tool issue - Aria attribute is not allowed
[BUG] ARC Tool issue - Empty Label Text

Arc tool results (checkbox hiding label)

Arc tool results (checkbox unchecked)

Arc tool results (checkbox checked)

Arc tool results (multiple checkboxes)

Need to fix the failing snapshot tests for the checkbox component


Remove testing-e2e

as we only backport fixes ... it's not important to fix upcoming browser changes that will result in slightly different web-UI behavior

#25682


stateful session persistence

Http session persistence is achieved through hash-based (consistent hash algorithm) load balancing. When the state of the backend servers change (new server is added, existing server is deleted, server state is updated, etc.) it breaks the session persistence result.
Is it possible to consider adding a cookie-based stateful session persistence? When this feature is turned on, the host id or something is added to the cookie. When new requests arrive, the corresponding host is resolved from the cookie and passed to the LoadBalancer via Upstream::LoadBalancerContext. The LoadBalancer can return this host first, if possible.

cc @alyssawilk

The Upstream::LoadBalancerContext may needs to extend an interface similar to the one shown below:
struct PrimaryHost {
   std::string host_id_or_hash_or_address_or_something; // something can uniquely identify a host
   size_t expected_status; // expected health status of host
};

PrimaryHost primaryHostShouldSelected()

The Http Router need add some new API to enable or disable this function and some code to handle cookie.
The LoadBlancer can optionally select the host based on the above interface. Of course, when this feature is enabled, we need an additional hash table, which will bring some additional memory overhead.
In general, it is not complicated and does not require much new code.
I am not sure but this may be can close #7218.

Add a doc to describe more detail and some possible solutions.

@alyssawilk @mattklein123  Is this plan feasible for #7218

for cookies, can you comment where this doesn't meet your needs?
https://www.envoyproxy.io/docs/envoy/latest/api-v3/config/route/v3/route_components.proto#envoy-v3-api-msg-config-route-v3-routeaction-hashpolicy-cookie

@alyssawilk The most important reason is that no matter what hash policy is used, the existing session persistence is stateless. A suitable host is obtained according to the algorithm based on the value calculated by the hash.
When the state of the back-end servers changes (New servers are added, some servers’ state is updated, etc.), the hash ring will be rebuilt, the session persistence is broken.
The requirement in #7218 is actually a good example. When host x is marked as draining, for the existing session requests, we hope that they will still be routed to host x. If we have the ability to maintain stateful session persistence, and for each session, save the relevant host information, then we can do this.

so at some point if a host is draining it's going to go away.  It sounds like what you really want here is a period where no new "sticky" sessions are assigned, but sessions already sticky to an upstream will be routed there?
You say on the doc that maglev might meet your needs if only you could route while draining, but I don't think either of your design options include routing to a host while it's draining, which I don't think your cookie solution would necessarily do either.
That said, encoding ip:port of backend in cookies and using them for direct routing is a pretty common method of LB, and if you want to implement that I think it would be a useful addition to Envoy.

so at some point if a host is draining it's going to go away. It sounds like what you really want here is a period where no new "sticky" sessions are assigned, but sessions already sticky to an upstream will be routed there?

Yes.

You say on the doc that maglev might meet your needs if only you could route while draining, but I don't think either of your design options include routing to a host while it's draining, which I don't think your cookie solution would necessarily do either.

The draining host is special. Because it has been removed from the hosts set of the cluster, we may need to think more about it. But unhealthy host and degraded host can use the cookie scheme to maintain the existing sticky session.

That said, encoding ip:port of backend in cookies and using them for direct routing is a pretty common method of LB, and if you want to implement that I think it would be a useful addition to Envoy.

I am happy to implement it.

Ok reposting some of the stuff shared with @wbpcode offline, since it might be useful for future cases of custom load balancing.
We've used two methods for achieving something similar to what could be called stateful session persistence.
Case 1:
As described in [0], we route requests based on a cookie value which is extracted and then applied as dynamic metadata using a filter similar to the header-to-metadata filter [1]. This metadata is in turn used by the subset LB to find the right endpoint. For @wbpcode's use case, you could have one subset per endpoint to achieve this. The case of missing endpoints could be addressed by computing an availability map from the main thread or just checking directly from the data path if the endpoint is available. For the use case described in the blog post, we compute a map of capacity per version from the main thread. The additional work from the main thread might not be needed for wbpcode's case.
Case 2:
We have another use case where we balance requests by hashing on the output of a regex applied to a given header. This might not be directly applicable, since there isn't – currently at least – a way to handle fallbacks in a custom way when using a hash policy for routing.
Happy to expand more on any of these details.
[0] https://medium.com/pinterest-engineering/simplifying-web-deploys-19244fe13737
[1] https://www.envoyproxy.io/docs/envoy/latest/api-v3/extensions/filters/http/header_to_metadata/v3/header_to_metadata.proto
[2] https://github.com/envoyproxy/envoy/pull/11819

Reposting some reply to @rgs1
custom filter + dynamic metadata + subset lb is a powerful method that can be used to implement various complex routing.
But think the problems custom filter + dynamic metadata + subset lb  solved are still not consistent with stateful session sticky. The problem I hope to solve is how to persist the session.
For example, we have three upstream instances of A B C, and the client X request is routed to A at the beginning. After that, A was marked as degraded due to the health check. I hope that at this time, traffic from X is still routed to A. The traffic from the new client Y is normally routed to B or C.
In essence, subset lb is still matching routing rules and then selecting host. (By setting clever matching rules, you can achieve a result similar to stateful session sticky in some scenarios, but I think this way of using it is too complicated for persisting the session, and it only covers some of the scenarios)
The stateful session sticky is to record the result of the first match, and then directly route to the corresponding host.

working

working

working


[Impeller] In Playground, implement a swap to Skia backend for comparison

Perhaps pressing 's' to quick swap to Skia while pressed and/or 'S' - one could swap while pressed, the other could toggle.

Good suggestion, we could parameterize the display list unittests harness to begin with.

Looking into this some more, I don't think this makes sense for any playground other than maybe the display list one because all other variants use Impeller specific APIs.


Why / Balance

It feels more natural that a writing utensil would be more casually accessible than a workplace ID. Im hoping that by swapping them around, people will think about their pens more and maybe be more willing to leave notes and engage in some bureaucracy.
Media
I have attached a video to respond to feedback concerning the ease of accessing an ID in a combat scenario. I understand this concern and wouldnt want to make a change that would disrupt combat, but I dont think that this change affects combat significantly. I think that stealing IDs mid-fight is a bit niche of a scenario to prioritize quality of life for over the general station gameplay experience.
https://github.com/user-attachments/assets/9c94e5fb-7577-4b10-92e3-7f856b6cf6fe
Requirements

[X] I have read and am following the Pull Request and Changelog Guidelines.
[X] I have added media to this PR or it does not require an ingame showcase.

Changelog

:cl: SpaceRox1244

tweak: Pens now eject from PDAs before ID cards.

I'm personally not a fan of the change because of the aforementioned scenarios; I can't recall a situation where I have had to be fast with ejecting my pen so this would be a QOL downgrade for myself. I do echo the sentiment of wanting people to be more aware of their pens though - if it's an option, perhaps adding a secondary eject keybind might be a good middle ground?

I have attached media and a description to respond to this feedback

I'm personally not a fan of the change because of the aforementioned scenarios; I can't recall a situation where I have had to be fast with ejecting my pen so this would be a QOL downgrade for myself. I do echo the sentiment of wanting people to be more aware of their pens though - if it's an option, perhaps adding a secondary eject keybind might be a good middle ground?

Syndies have use for it for quick pen weapons and bombs, but other than that ID first would be preferable so option to select priority would be nice, otherwise I feel lots of crew are going to eject their pens from now...

Nobody is going to throw out their pens so that they can quick draw their IDs.

Nobody is going to throw out their pens so that they can quick draw their IDs.

Everyone with a braincell will during malf AI if they do not want to get shocked by doors.

Nobody is going to throw out their pens so that they can quick draw their IDs.

Everyone with a braincell will during malf AI if they do not want to get shocked by doors... though I guess throwing whole PDA is option too.

What?

Nobody is going to throw out their pens so that they can quick draw their IDs.

Everyone with a braincell will during malf AI if they do not want to get shocked by doors... though I guess throwing whole PDA is option too.

What?

If AI harms crew with doors, you can avoid touching doors by activating them with PDA/ID rather than hand.

Nobody is going to throw out their pens so that they can quick draw their IDs.

My first thought when I read the PR was that if it annoys me, I'll just keep my pen elsewhere.

Im hoping that by swapping them around, people will think about their pens more and maybe be more willing to leave notes and engage in some bureaucracy.

Personally I can't say the reason I've ever forgone paperwork or notes is because my pen isn't the first thing available. Having paper nearby in a convenient place is much more a nuisance. And because I prefer directly communicating with people.
And whenever I've given people paperwork (I play a lot of QM/HOP), the reason they don't do it isn't usually because they don't have a pen.. a bunch of people (LRP) just find it tedious/annoying. (Part of this is needing a better paperwork system like SS13 with proper fields and such)
That being said, I think my preference would be that a system is setup that prioritizes the last inserted item. Then it doesn't matter which it starts as.

I echo the sentiment that there should be some mechanic allowing the player to effectively set the ejection priority. I think this would be the best of both worlds, allowing for players who are using pen weapons/bombs to quickdraw, while maintaining the ability of HoP and Nukies to quickly manage IDs.

On its own, this change isn’t really great as quick management of ID is much more critical than quick management of a pen, and would put that as against this PR on this alone. Plus you can always alt click again? And there’s a pen ejection icon up at the top right of the PDA UI.
Pens are usually used in rather calm situation, except for antag pen-like items. But IDs are much more likely to get pulled out of the PDA in an emergency, etc.
In the context of wider game though, I think being able to set specific slot as preferred for alt click ejection would be a much better change. Something similar to backpack favorite slots/etc would be nice. Could reuse the same (default) keybind for it and such.

These are some good points actually yall. The calmness of either use case is admittedly important regardless of the frequency of the use case.

Perhaps for my desired effect, I should find a way to make something to write on more readily available


How to customize search input

I need to customize the search input, in example adding icon or customize label, how to do that?

You can do that using pure css. See this example:
https://bootsnipp.com/snippets/featured/custom-search-input


Implement the async scan algorithm based on the cuda implementation

Implement the async scan algorithm based on the cuda implementation

Can one of the admins verify this patch?


Implemented the concept of 'modes' which allows pugs to be set up as …

…std, pro, or insta

Looks ok, can fix later if not - merged.


Missing mocked method AsyncStorage.flushGetRequests

This method is used in particular by the Raven (ie. Sentry) client.
https://facebook.github.io/react-native/docs/asyncstorage.html

Thank you for report this :) I will fix this fast as i can :)

Sorry for that delay. I created the method. I think for us enough just an empty function. Because in the real AsyncStorage has more complex implementation and they need this method, i think for mocking, just enough to not throw an exception. But if you had some ideas please feel to free to contribute.
I just created a branch, and i create another one for fix some silly naming stuff in tests and code, after i publish the 2.0.1 version.

Seems perfect to me: I use the same technique in my jest setup file to add an empty method, and everything works as expected.
Thanks for the fix!


Look only for the latest event when deploying an API

Issue
https://gravitee.atlassian.net/browse/APIM-3193
https://github.com/gravitee-io/issues/issues/9364
Description
Look only for the latest event when deploying an API.
Also did some cleaning in the EventService interface

📚  View the storybook of this branch here

Need to check if it will be the same changes on 4.0+


Child injectors not sharing singletons (and affecting each other when binds are present)

The documentation for createChildInjector states that elements of the child injector are not visible to the parent, but the state from the parent injector is inherited by the child.
Now, to demonstrate the issue, consider the following example:
    @Singleton
    private static class MySingleton {}

    @Override
    public void run() {
        Injector root = Guice.createInjector();

        Injector firstChild = root.createChildInjector();
        Injector secondChild = root.createChildInjector();
        Injector thirdChild = root.createChildInjector();

        MySingleton singleton1 = firstChild.getInstance(MySingleton.class);
        MySingleton singleton2 = secondChild.getInstance(MySingleton.class);
        MySingleton singleton3 = thirdChild.getInstance(MySingleton.class);

        assert singleton1 == singleton2;
        assert singleton2 == singleton3;

        // As expected: singleton1 = singleton2 = singleton3
    }

A single instance of the MySingleton object is created and shared by all of the child injectors. So far, so good. The issue occurs when defining a module containing a binding for the MySingleton on one of the child injectors, but not the others.
    @Override
    public void run() {
        Injector root = Guice.createInjector();

        Injector firstChild = root.createChildInjector(new AbstractModule() {

            @Override
            protected void configure() {
                this.bind(MySingleton.class).toInstance(new MySingleton());
            }
        });
        Injector secondChild = root.createChildInjector();
        Injector thirdChild = root.createChildInjector();

        MySingleton singleton1 = firstChild.getInstance(MySingleton.class);
        MySingleton singleton2 = secondChild.getInstance(MySingleton.class);
        MySingleton singleton3 = thirdChild.getInstance(MySingleton.class);

        assert singleton1 != singleton2;
        assert singleton2 == singleton3;

        // Expected outcome:
        // - singleton1 is the instance specified in the binding
        // - singleton2 and singleton3 are the same instance

        // Actual outcome:
        // - singleton1 is the instance specified in the binding (good)
        // - singleton2 and singleton3 are not the same instance (bad)
    }

Specifying a single binding on one of the child injectors causes the other child injectors to no longer adhere to the parent's singleton.
First child bindings:
InstanceBinding{key=Key[type=GuiceExample$MySingleton, annotation=[none]], source=GuiceExample$1.configure(GuiceExample.java:20), instance=GuiceExample$MySingleton@63753b6d}

Second child bindings:
ConstructorBinding{key=Key[type=GuiceExample$MySingleton, annotation=[none]], source=class GuiceExample$MySingleton, scope=Scopes.SINGLETON}

Third child bindings:
ConstructorBinding{key=Key[type=GuiceExample$MySingleton, annotation=[none]], source=class GuiceExample$MySingleton, scope=Scopes.SINGLETON}

It looks as though a constructor binding is being created on the other two child injectors when one of the three child injectors specify bindings to the singleton.
Surely the child injectors shouldn't affect each other as they inherit from the parent and not from one another?

This is working as intended
in the second and third child MySingleton is a jit binding.  for child injectors we intentionally try to resolve jit bindings in parents first.  so both secondChild and thirdChild will ask the root for the binding and thus share the singleton.
firstChild is different though, it is an explicit binding in the child so that is used rather than asking the parent to create a jit binding.

firstChild is different though, it is an explicit binding in the child so that is used
both secondChild and thirdChild will ask the root for the binding and thus share the singleton

The first child is resolving the binding and the explicit binding is being used, correct. Contrary to your statement, the second and third children do not share the singleton. They each get their own instance of the singleton in their own scopes, completely unrelated to the parent.
It's a JIT conundrum lacking in documentation where children aren't allowed to have bindings overriding their parent, but parents are allowed to have JIT bindings passed down to children which have bindings themselves.
public class GuiceTest {

    @Test
    public void testChildren() {
        Injector root = Guice.createInjector();

        Injector manualChild = root.createChildInjector(new AbstractModule() {

            @Override
            protected void configure() {
                this.bind(MySingleton.class).toInstance(new MySingleton());
            }
        });

        // Pass
        MySingleton nonJIT = manualChild.getInstance(MySingleton.class);

        // Fail
        Assert.assertEquals(
            root.createChildInjector().getInstance(MySingleton.class),
            root.createChildInjector().getInstance(MySingleton.class)
        );
    }
}


Implement gaussian splatting ply file saving with o3d

Ply saving based on open3D. That's a quick way to generate ply files.
I think this is a highly requested feature. Should work with default and mcmc densification strategies.

I try my own save_ply with plyfile version save_ply with this draft. But, the color seems a little bit different between two method, despite I loaded it from exact same checkpoint.

Interesting. The arm chairs are equally colored? 🙂

Interesting. The arm chairs are equally colored? 🙂

seems the result is more washed out in open3D. The most noticeable is black chair (third image).
I will check for another scene later.

Here is it, the code comes from gsplat v1.3.0, then I modified it a little bit, so it saves PLY with plyfile library.
The command is easy
python3 gsplat/examples/simple_trainer_save_ply.py default --disable_viewer --data_factor 1 \
            --data_dir data/my_dataset \
            --result_dir output/my_dataset \
            --ckpt output/my_dataset/ckpts/ckpt_29999.pt

Then it will create splats.
simple_trainer_save_ply.zip

Should be fixed @ichsan2895.
I am also down to add a separate script so create a ply from the ckpt. It could be also done additionally. I don't mind. The maintainers should decide what they want if at all.

Should be fixed @ichsan2895. I am also down to add a separate script so create a ply from the ckpt. It could be also done additionally. I don't mind. The maintainers should decide what they want if at all.

Well, creating PLY from checkpoints is just adding a single line of code.
if cfg.ckpt is not None:
        # load checkpoint
        ckpts = [
            torch.load(file, map_location=runner.device, weights_only=True)
            for file in cfg.ckpt
        ]
        for k in runner.splats.keys():
            runner.splats[k].data = torch.cat([ckpt["splats"][k] for ckpt in ckpts])
       .....................................................
       # add this line
       save_ply(runner.splats, f"{cfg.result_dir}/point_cloud_{step}.ply")

It is not about adding a line of code. There is some reasoning behind why it writes the checkpoint in the way it does and I am not aware of it. The ply file should be directly saved during training and not evaluation. There could be a script added that converts old checkpoints to ply files if desired.
Beside saving the trained scene in an alternative format, the ckpts are pretty useless. They don't serialize the full training state (optimzer, scheduler) so it is not possible to resume the training properly. In fact, the ply file contains the same information.
Additional input from the maintainers is needed.

@bolopenguin Good catch. I bake the appearance into the shs so it can be opened by any viewer.
@liruilong940607 What does this pr require to go into gsplat?

Not arguing for it to block this PR, but as an FYI this will break things for folks running Python 3.12, which the latest open3d release doesn't support. (afaik this is the main thing that's preventing Python 3.12 support in the main nerfstudio repo)
It looks like the open3d folks merged their Python 3.12 PR a few days ago though, so maybe this will resolved soon? https://github.com/isl-org/Open3D/pull/6717

Ah, interesting!
We could either wait until open3D is compatible with Python 3.12 or we write the ply file directly similarly as it is done in Nerfstudio. Any preferences?

I removed open3D in commit 36e3df32a997300f384990722e368010ba08e09a in case that's blocking.

Is there some difference in implementation from the 3DGS official version? I tried to save the training result as a ply file and then view it in the web viewer SuperSplat. I observed some color artifact, but it is fine in the gsplat viewer.
SuperSplat:

gsplat viewer[gsplat/examples/simple_viewer.py]:

I am not sure what supersplat is doing behind the curtains. I can compare the output of the last two commits. They should be identical and were tested.

Encountered a strange issue. Everything working when training on previous datasets. However, when training on a new dataset and when save it as a ply file, an error is displayed.
 Files of ply generated, but open with Postshot show “PlyReader: unexpected table size”. MeshLab: "Error encountered while loading file. Unexpected EOF".

@hecodeit
I didn't filter for NaNs and Inf which might cause your issue. Please check again and confirm.
Moreover, I found that there was a slight misalignment in the order the attributes are saved which might have caused the slight color issues that @lambdald reports. Please test as well.

@MrNeRF Yes, pull the commit and the error don't show again. Thanks for the updating.

Is it compatible with supersplat or mkkellogg gaussian-viewer?
No, it is not that straight forward. It would be best to support it directly.

I think currently is busy season in terms of getting the publications camera ready... likely there is not much time left for open source projects.


An unknown error

I run a bot after a while it was stopped working and showed this error ;
Exception ignored in: <_io.TextIOWrapper name='' mode='w' encoding='UTF-8'>
BrokenPipeError: [Errno 32] Broken pipe
[ 0][t 0][1561986013.779947758][MessagesManager.cpp:10127][!Td][&last_new_message_id.get() > d->last_new_message_id.get()]       server message 432473 server message 432473 on_get_channel_dialog 40

You need to apply this patch to TDLib to fix this:
0001-Fix-setting-last_new_message_id.txt.

You need to apply this patch to TDLib to fix this:
0001-Fix-setting-last_new_message_id.txt.

It worked thanks a lot

This was fixed in TDLib 1.5.0.


Documentation Manuals

[How to find the relevant documentation, manuals in the current present command line interface one is interacting with inside of any one of the Microsoft Windows Operating Systems either or any Office or whatever Server whatever or what ever else word is associated with the device low level software accessible through the command line, terminal, UI, virtual or whatever word is associated with a non graphical user interface  .]

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

ID: 6d382ca0-6538-5e1d-3e83-e31c9ab6a911
Version Independent ID: 11eff296-6158-f89a-4941-8735529183e9
Content: Windows commands
Content Source: WindowsServerDocs/administration/windows-commands/windows-commands.md
Product: windows-server
Technology: windows-commands
GitHub Login: @JasonGerend
Microsoft Alias: jgerend

You may want to look elsewhere for answers, unless you have suggestions for improving one or more specific pages in the MS Docs repository.
Disclaimer: I am not affiliated with Microsoft or the Microsoft Docs teams here on GitHub.
The intention of this repository is to improve on the existing documentation by providing the users a way to report factual errors, improvement suggestions and request updates for new versions.
The MS Docs team members might not have time to answer your question, both due to the small size of the team and the big workload.
Try the following resources to get in touch with users who may be able to help you:

TechNet forums: https://social.technet.microsoft.com/Forums/en-US/home
MSDN: https://social.msdn.microsoft.com/Forums/en-US/home
Microsoft Community: https://answers.microsoft.com/

I do apologize for what I did write. I should have known better from previous experience in open source that to find answers takes a lot of researching. I do not have the clock time to write scripts nor data storage to accommodate w-get to pull all of Microsofts HTMl or XML files and sort them by OS. And then index the sub-subjects in each OS. And create tables of reference for the command-lets that are available in same OS`s And Servers. Interesting enough , when the computers were made one of their purposes was storing data after accumulation. Yes we know that. Yet I take it for granted that a Corporation would assimilate their data into comprehensible indexing by OS and its sub subjects and so on. We could take a hint from biology in animal and plant families and as to how they are cataloged by family etc. Or a modern day Library. Thank you


How to force an element to left or to right while using 2-columns layout

As the title suggests I want to know if there is any way to force an element to a particular side .

I'm aware that this is a rather hacky solution, but I recently found out that by putting an empty div before a VerticalTimelineElement you can change the alternation of all the VerticalTimelineElement elements below that div. In order to apply this effect for just one element, simply add another empty div after it. See the example:
<VerticalTimelineElement className="left-element1"> // assume this is placed left
    // stuff
</VerticalTimelineElement>

<div/> // this changes the VerticalTimelineElement alternation

<VerticalTimelineElement className="right-element1"> // it will be placed left instead of right
    // stuff
</VerticalTimelineElement >

<div/> // this breaks the effect and restores the original alternation

<VerticalTimelineElement className="left-element2"> // this will be placed normally on the left side
     // stuff
</VerticalTimelineElement>

I hope that helps!


update README.md with import/export supported by node

It describes how to import and export with the es modules, but they are not supported by current versions of nodeJS. It's experimental. You can use them with babel (or another transpilator), but it is not included in the project and will add a layer of complexity. I updated the readme with the currently supported native syntax.

That's a good idea.
For reference: https://nodejs.org/api/esm.html


--cookies-from-browser for a chrome/firefox like browser or non-default install location

Checklist

[X] I'm reporting a feature request
[X] I've looked through the README
[X] I've verified that I'm running yt-dlp version 2022.02.04. (update instructions)
[X] I've searched the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Description
The command --cookies-from-browser works fine on regular Opera but it does not support Opera GX. Opera GX support only requires a few additions in cookies.py to look in Opera Software\Opera GX Stable similar to how regular Opera is dealt with. This will allow Opera GX users like me to be able to avoid having to extract cookies manually every time.

Assuming opera and GX store cookies similarly, You can do --cookies-from-browser "opera:path/to/gx/profile". This allows us to not have to support the different variants of each browser explicitly. Let me know if this works

Oh nice it works. Thanks a lot.

Assuming opera and GX store cookies similarly, You can do --cookies-from-browser "opera:path/to/gx/profile". This allows us to not have to support the different variants of each browser explicitly. Let me know if this works

I know this is old but just wanted to say your instructions helped solve my issue with Librewolf with a bit of adjustment
--cookies-from-browser "Firefox:path/to/.librewolf
Maybe it will help others out.


Add support for queries with a parameter

Dear Brian,
I have a TCPIP instrument whose API includes some parametrized SCPI queries in the form of :STUFf:THINg?[<wsp>MIN|MAX|DEF|SET|ACT|ALL] and I can't find how to perform those queries using easy_scpi.
Trying:
my_instrument.stuff.thing('MIN')

would return a fixed int (let's say 24 for instance). easy_scpi does no processing of the response so I believe this integer is genuinely sent back by the instrument although its documentation does not mention it.
Am I doing it wrong or is it not yet supported by easy_scpi?
Thank you for your time and your great abstraction work in this module.

Thanks you :)
For this you probably need to use the query method.
result = my_instrument.query("STUFF:THING? MIN")

If arguments are passed it is assumed the statement is a command.

PR #18 includes functionality for this, now.


Sort is broken since 3.0.0 with namespaced-json format

In 3.0.0, CLI option --sort --format namespaced-json is no longer working. Keys are sorted in random order.

Hi,
After looking into this, I don't think it actually ever worked when using namespaced-json.
unflatten messes with the ordered keys:
https://github.com/biesbjerg/ngx-translate-extract/blob/7d1bcd2a80eaa06dec278446bb2c86dfba0ec526/src/compilers/namespaced-json.compiler.ts#L20

Yes it was working previously, we are using it for one year now without any issue. Thanks for the fix!


feat(derive): Track Current Channel Size

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#331  👈
#330
#327
#326
#325
main

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

Join @refcell and the rest of your teammates on  Graphite

Codecov Report
Attention: Patch coverage is 6.66667% with 14 lines in your changes missing coverage. Please review.

Project coverage is 77.7%. Comparing base (f1dedb5) to head (0967d78).

Files
Patch %
Lines

crates/derive/src/stages/channel_bank.rs
11.1%
8 Missing :warning:

crates/derive/src/types/channel.rs
0.0%
6 Missing :warning:

Additional details and impacted files

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


[RFE] Additional Tasks

Tasks:

clone-pvc
clone-template
modify-vm-template

/remove-lifecycle stale

/remove-lifecycle rotten

most of these tasks have been already implemented


Merge in changes from hackathon

We're hoping to add these changes to processors instead of eidos.  This PR has been made in part to facilitate discussion.

moved code to conceptdiscovery so closing for now, we can def reopen as needed


Fix gcloud config syntax for e2e test setup

Changes

This commit fixes the argument passed in the gcloud key creation command.
Thanks @XinruZhang for spotting this.
Submitter Checklist
As the author of this PR, please check off the items in this checklist:
/kind bug

[n/a] Has Docs included if any changes are user facing
[n/a] Has Tests included if any functionality added or changed
[x] Follows the commit message standard
[x] Meets the Tekton contributor standards (including
functionality, content, code)
[x] Has a kind label. You can add one by adding a comment on this PR that contains /kind <type>. Valid types are bug, cleanup, design, documentation, feature, flake, misc, question, tep
[n/a] Release notes block below has been updated with any user facing changes (API changes, bug fixes, changes requiring upgrade notices or deprecation warnings)
[n/a] Release notes contains the string "action required" if the change requires additional action from users switching to the new release

Release Notes
NONE

/retest


Hover over image to see social icons with color effect #367

Description
After hovered over image the user can see social icons and also after hovering over those social icons , the color of icons changes.
Fixes # (issue no.)
#367
Type of change

[x] New feature (non-breaking change which adds functionality)

Explain the Testing instructions
Test Configuration:

Operating system: Windows
Version: 10
Text-editors used: Sublime editor

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] My changes generate no new warnings

ATTACH SCREEN-SHOTS / DEPLOYMENT LINK

@VidhiBhatt01 I have done the changes whatever you told , have a look.

@VidhiBhatt01 title size changed and justified the content in center have a look.

Everything is perfect but the text is getting cut at this dimension. Make a media query for the same 😶

@VidhiBhatt01 I did the changes , kindly look and let me know if anything else to do.

Finally done!! Kudos to your patience and dedication. Usually, contributors back of due to frequent changes but you sailed through! 👏

Finally done!! Kudos to your patience and dedication. Usually, contributors back of due to frequent changes but you sailed through! 👏

Thank You @VidhiBhatt01 🤗


I can't Get trained models

when i run bash /scripts/get_reference_models.sh  just got no such   file or directory
did i missed something?

run bash scripts/get_reference_models.sh if you are under temporal-segment-networks-master/

thanks, got it

Closing this. Please feel free to reopen it if you meet any further problem.


⚠ Bofrost.at has degraded performance

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

HTTP code: 200
Response time: 969 ms

Resolved: Bofrost.at performance has improved in 08922aa after 13 minutes.


Fix locality aware routing to be applied when proxy has empty locality config set

Fix for issue: https://github.com/istio/istio/issues/11609

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: anilutcs
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: zackbutcher
If they are not already assigned, you can assign the PR to them by writing /assign @zackbutcher 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:

pilot/pkg/networking/OWNERS

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

Hi @anilutcs. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

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.

/ok-to-test
@anilutcs  Fix the lint error

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

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests.sh
388531bd92f6c47f85c430ebedceac0665281049
link
/test e2e-simpleTests

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.

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

Test name
Commit
Details
Rerun command

prow/e2e_pilotv2_auth_sds.sh
388531bd92f6c47f85c430ebedceac0665281049
link
/test istio_auth_sds_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.

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

Test name
Commit
Details
Rerun command

prow/e2e_pilotv2_auth_sds.sh
388531bd92f6c47f85c430ebedceac0665281049
link
/test istio_auth_sds_e2e

prow/istio-pilot-multicluster-e2e.sh
388531bd92f6c47f85c430ebedceac0665281049
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.

@ymesika thanks, will do that.

/retest

Comment for CLA to work

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

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
40415edbe85e9d8af1c591ee069f49997d0d7fcb
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.

I signed the CLA.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: anilutcs
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: rshriram
If they are not already assigned, you can assign the PR to them by writing /assign @rshriram 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:

OWNERS

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

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

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
3d07543731331926f95be0dc619417f492ef210e
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.

PR has lots of unnecessary changes like mixer update, API update, etc.

@rshriram working on correcting the PR today, thanks.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: anilutcs
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: zackbutcher
If they are not already assigned, you can assign the PR to them by writing /assign @zackbutcher 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:

pilot/pkg/networking/OWNERS

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

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

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
3d07543731331926f95be0dc619417f492ef210e
link
/test istio-pilot-multicluster-e2e

prow/istio-integ-local-tests.sh
99e6a1b5472184dbf49c67b3ed12ef2decb64522
link
/test istio-integ-local-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.

I suggest opening a new one..

@ymesika I fixed the commit, I see only my changes now.

/retest

Comment for CLA.

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

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
3d07543731331926f95be0dc619417f492ef210e
link
/test istio-pilot-multicluster-e2e

prow/istio-integ-local-tests.sh
99e6a1b5472184dbf49c67b3ed12ef2decb64522
link
/test istio-integ-local-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.

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

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
3d07543731331926f95be0dc619417f492ef210e
link
/test istio-pilot-multicluster-e2e

prow/istio-integ-local-tests.sh
99e6a1b5472184dbf49c67b3ed12ef2decb64522
link
/test istio-integ-local-tests

prow/istio-integ-k8s-tests.sh
99e6a1b5472184dbf49c67b3ed12ef2decb64522
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.

/lgtm

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: anilutcs, ymesika
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:

pilot/pkg/networking/OWNERS [ymesika]

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

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

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
3d07543731331926f95be0dc619417f492ef210e
link
/test istio-pilot-multicluster-e2e

prow/istio-integ-k8s-tests.sh
99e6a1b5472184dbf49c67b3ed12ef2decb64522
link
/test istio-integ-k8s-tests

prow/istio-integ-local-tests.sh
ceeae19a18061602d2f477dde238af560e2b74b3
link
/test istio-integ-local-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.

Wait for https://github.com/istio/istio/pull/11677 to fix your test error.

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

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
3d07543731331926f95be0dc619417f492ef210e
link
/test istio-pilot-multicluster-e2e

prow/istio-integ-local-tests.sh
ceeae19a18061602d2f477dde238af560e2b74b3
link
/test istio-integ-local-tests

prow/istio-integ-k8s-tests.sh
ceeae19a18061602d2f477dde238af560e2b74b3
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.

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

Test name
Commit
Details
Rerun command

prow/istio-integ-local-tests.sh
ceeae19a18061602d2f477dde238af560e2b74b3
link
/test istio-integ-local-tests

prow/istio-integ-k8s-tests.sh
ceeae19a18061602d2f477dde238af560e2b74b3
link
/test istio-integ-k8s-tests

prow/istio-pilot-multicluster-e2e.sh
ceeae19a18061602d2f477dde238af560e2b74b3
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.

New changes are detected. LGTM label has been removed.

/lgtm

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: anilutcs, ymesika
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:

pilot/pkg/networking/OWNERS [ymesika]

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

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

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
733067300549946e86eea853c40d62933085befd
link
/test istio-integ-k8s-tests

prow/istio-pilot-multicluster-e2e.sh
733067300549946e86eea853c40d62933085befd
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.

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

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
733067300549946e86eea853c40d62933085befd
link
/test istio-integ-k8s-tests

prow/istio-pilot-multicluster-e2e.sh
733067300549946e86eea853c40d62933085befd
link
/test istio-pilot-multicluster-e2e

prow/e2e_pilotv2_auth_sds.sh
733067300549946e86eea853c40d62933085befd
link
/test istio_auth_sds_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.

Closing this in lieu of https://github.com/istio/istio/pull/11681


BMI and AHI Graph Tool

From @susanredline:

What ever happened to the tool we had to enter height and weight- BMI and see how changes in BMI influence AHI? Would be great to see that back in.

Take a look at making an interactive page where members can plot their BMI, (or calculate it) to see potential changes in AHI with an increase/decrease in BMI.

@krand7 The tool is a bit rough around the edges and needs more educational material to correctly frame what it's showing. If you have time to meet about it next week and decide how to upgrade it and update it's styling, let me know.


chore(main): release 0.4.4

:robot: I have created a release beep boop
0.4.4 (2024-10-10)
Bug Fixes

test rust cache (c674fda)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v0.4.4
:sunflower:


Collecting WiFi information without using it

The application collects the user WiFi information to the database but doesn't make any use of it.

Hey, yes! We  locally collect the WIFI BSSIDs  around. The main idea is that GPS is great for outdoor intersection detection, but terrible for indoor. COVID-19 usually passes by people in confined spaces (indoor). Collecting BSSIDs would allow us, to some extent, an indoor intersection detection.
There're more ideas we're considering for indoor detection - bluetooth, data over sound, etc.
We don't have an ETA for this feature yet.

react-native-background-geolocation already uses the WiFi information when configured to DESIRED_ACCURACY_HIGH, see the documentation.
You are not using the GPS directly, you are using the OS location API which already takes the WiFi information into account.

I don't understand your point. The output of react-native-background-geolocation is GPS coordinates - regardless of the accuracy we set. Also, we are agnostic to the WIFI info that the underlying OS location api (e.g. FusedLocation) uses.
This is a simple feature that was meant to assess distance between people. We'll probably use BLE and retire it anyway.

Hi @MiriShalev what was the reason for retiring this feature? Was the wifi feature unreliable?


BlobTrigger throw exception Cannot convert input parameter 'myBlob' to type 'System.IO.Stream' from type 'System.ReadOnlyMemory`1[[System.Byte, System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e]]'.

I'm creating functions in a MacOS. I have a few bugs. But for now I want to log this one.

Executed 'Functions.BlobTriggerCSharp' (Failed, Id=2964977b-296a-4a58-8930-69a8bfcfdcbb, Duration=1296ms)
[2021-04-18T01:15:34.633Z] System.Private.CoreLib: Exception while executing function: Functions.BlobTriggerCSharp. System.Private.CoreLib: Result: Failure
[2021-04-18T01:15:34.633Z] Exception: Microsoft.Azure.Functions.Worker.Diagnostics.Exceptions.FunctionInputConverterException: Error converting 1 input parameters for Function 'BlobTriggerCSharp': Cannot convert input parameter 'myBlob' to type 'System.IO.Stream' from type 'System.ReadOnlyMemory`1[[System.Byte, System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e]]'.
[2021-04-18T01:15:34.633Z]    at Microsoft.Azure.Functions.Worker.Context.Features.DefaultModelBindingFeature.BindFunctionInput(FunctionContext context) in D:\a\1\s\src\DotNetWorker.Core\Context\Features\DefaultModelBindingFeature.cs:line 70
[2021-04-18T01:15:34.633Z]    at Microsoft.Azure.Functions.Worker.Invocation.DefaultFunctionExecutor.ExecuteAsync(FunctionContext context) in D:\a\1\s\src\DotNetWorker.Core\Invocation\DefaultFunctionExecutor.cs:line 37
[2021-04-18T01:15:34.633Z]    at Microsoft.Azure.Functions.Worker.OutputBindings.OutputBindingsMiddleware.Invoke(FunctionContext context, FunctionExecutionDelegate next) in D:\a\1\s\src\DotNetWorker.Core\OutputBindings\OutputBindingsMiddleware.cs:line 13
[2021-04-18T01:15:34.633Z]    at Microsoft.Azure.Functions.Worker.GrpcWorker.InvocationRequestHandlerAsync(InvocationRequest request, IFunctionsApplication application, IInvocationFeaturesFactory invocationFeaturesFactory, ObjectSerializer serializer, IOutputBindingsInfoProvider outputBindingsInfoProvider) in D:\a\1\s\src\DotNetWorker.Grpc\GrpcWorker.cs:line 169
[2021-04-18T01:15:34.633Z] Stack:    at Microsoft.Azure.Functions.Worker.Context.Features.DefaultModelBindingFeature.BindFunctionInput(FunctionContext context) in D:\a\1\s\src\DotNetWorker.Core\Context\Features\DefaultModelBindingFeature.cs:line 70
[2021-04-18T01:15:34.633Z]    at Microsoft.Azure.Functions.Worker.Invocation.DefaultFunctionExecutor.ExecuteAsync(FunctionContext context) in D:\a\1\s\src\DotNetWorker.Core\Invocation\DefaultFunctionExecutor.cs:line 37
[2021-04-18T01:15:34.633Z]    at Microsoft.Azure.Functions.Worker.OutputBindings.OutputBindingsMiddleware.Invoke(FunctionContext context, FunctionExecutionDelegate next) in D:\a\1\s\src\DotNetWorker.Core\OutputBindings\OutputBindingsMiddleware.cs:line 13
[2021-04-18T01:15:34.633Z]    at Microsoft.Azure.Functions.Worker.GrpcWorker.InvocationRequestHandlerAsync(InvocationRequest request, IFunctionsApplication application, IInvocationFeaturesFactory invocationFeaturesFactory, ObjectSerializer serializer, IOutputBindingsInfoProvider outputBindingsInfoProvider) in D:\a\1\s\src\DotNetWorker.Grpc\GrpcWorker.cs:line 169.

My code is simple like it:
[FunctionName("BlobTriggerCSharp")]
        public static void BlobTriggerCSharp([BlobTrigger("imagestest/{blobName}.{blobExtension}", Connection = "AzureWebJobsStorage")]
        Stream myBlob, string name, ILogger log)
        {
            log.LogInformation($"C# Blob trigger function Processed blob\n Name:{name} \n Size: {myBlob.Length} Bytes");
        }

Hello,
I can confirm that I am experiencing the exact same error when executing a BlobTrigger function with the same method signature as above.
Tech stack is:

Visual Studio for Mac 8.9.6 (build 6).
.Net 5.0.
Azure Function Core Tools Version 3.0.3442.

The BlobTrigger function was generated using Azure Function Core Tools commands, followed by changing the default input parameter "myBlob" from a string to a Stream.
Happy to upload and reference a sample project if required.
Thanks.

Based on the comments in azure-functions-dotnet-worker/issues/378 ...am I correct in thinking that Streams aren't actually support when using BlobTriggers in .Net 5.0?

It works changing stream to byte[]. At least as a workaround.
On Tue, 20 Apr 2021 at 10:34 am, Simon Gilbert @.***>
wrote:

Based on the comments in azure-functions-dotnet-worker/issues/378
https://github.com/Azure/azure-functions-dotnet-worker/issues/378#issuecomment-820767774
...am I correct in thinking that Streams aren't actually support when using
BlobTriggers in .Net 5.0?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Azure/azure-functions-dotnet-worker/issues/398#issuecomment-822883240,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABJYXHGWMN6E4OJUPKULCEDTJTDXZANCNFSM43DV2MFA
.

--
Regards,
Anderson Fetter
www.afetter.com

I can confirm that using a byte[] is correct.
Also worth mentioning this other linked comment here which explains the reason that Streams are not supported in .Net 5.0 currently.


how to change the node size at synchronous time

while using such as method, how to change the node size at synchronous time
addNodeInput (id)
addNodeOutput (id)
removeNodeInput (id, input_class)
removeNodeOutput (id, output_class)

By default the size is changed

Adding editor.addNodeInput(7) in demo.

Thanks,jero !
It is wrok finely.


[r2.4-rocm-enhanced] Fix for JIRA ticket 280792

Updating rocm_configure.bzl to do the right thing based on whether or not the softlinks to <rocm_path>/include/[hiprand|rocrand] exist. Removing the workaroung to handle the same in Dockerfile.rocm, since that is no longer required (as a consequence of the change in rocm_configure.bzl

/cc @sunway513
I will merge the PR once the local build I am doing with this change completes

local build passed...merging PR

http://ontrack-internal.amd.com/browse/SWDEV-280792


Pattern Detection in Declarative Deployment Models

Enable pattern detection in topologies by extending and repurposing PRMs.

Start and end date: 2020-10-28 to 2021-04-28
Contributor: Adrian Wersching @awersching
Supervisor: Lukas Harzenetter @lharzenetter
[ ] Ensure that you followed http://eclipse.github.io/winery/dev/ToolChain#github---prepare-pull-request.
[x] Branch name checked. That means, the branch name starts with thesis/, fix/, ...
[ ] Ensure that the commit message is a good commit message
[x] Ensure to use auto format in all files
[x] Ensure that you appear in NOTICE at Copyright Holders
[x] Tests created for changes
[x] Documentation updated (if needed)
[x] Screenshots added (for UI changes)

Looks good! Please create the final PR in eclipse/winery. :)

Follow up at https://github.com/eclipse/winery/pull/593


[Breaking change]: System.Configuration.ConfigurationManager no longer references System.Security.Permissions

Description
The System.Configuration.ConfigurationManager package no longer references the System.Security.Permissions package.
Version
.NET 8 Preview 3
Previous behavior
The System.Configuration.ConfigurationManager referenced the System.Security.Permissions package.
New behavior
The System.Configuration.ConfigurationManager does not reference the System.Security.Permissions package.
Type of breaking change

[ ] Binary incompatible: Existing binaries may encounter a breaking change in behavior, such as failure to load or execute, and if so, require recompilation.
[x] Source incompatible: When recompiled using the new SDK or component or to target the new runtime, existing source code may require source changes to compile successfully.
[ ] Behavioral change: Existing binaries may behave differently at run time.

Reason for change
This avoids a dependency on System.Drawing.Common when System.Configuration.ConfigurationManager is referenced which is primarily an issue for non-Windows operating systems.
The dependency to System.Drawing.Common was caused by the following package dependencies:
System.Configuration.ConfigurationManager ->
System.Security.Permissions ->
System.Windows.Extensions ->
System.Drawing.Common.
Recommended action
If the System.Configuration.ConfigurationManager package is referenced and a dependency is needed to System.Security.Permissions or any of its dependencies, which may include System.Windows.Extensions, System.Security.AccessControl, or System.Drawing.Common then ensure those packages are referenced directly or indirectly.
Feature area
Other (please put exact area in description textbox)
Affected APIs
No response

I marked this as "source incompatible" instead of "binary incompatible" because the problem will only be observed on a rebuild when referencing a new version of the package.  Existing binaries / builds will work.


update model data collector to include counts of agents for each risk attitude

If we update model data with total number of agents in each risk attitude, then we can use that to compare proportions relative to total agents (based on grid size), and have better numeric values for use use in regression analysis (use the same buckets as our categories for risk moderate, risk averse, risk inclined). This means we can skip collecting agent-level data for the large scale batch runs - so maybe add a parameter to the hawk/dove multi batch run script to control whether agent data is collected.

implemented in #66


Getting the FileNotFoundException even when using the absolute path.

CMD:
java -jar JS2PDFInjector-1.0.jar IndigoERP.pdf temp.js
ERROR:
[*] JavaScript Payload: temp.js
[*] Output File Path: null/js_injected_IndigoERP.pdf
java.io.FileNotFoundException: null/js_injected_IndigoERP.pdf (No such file or directory)
	at java.base/java.io.FileOutputStream.open0(Native Method)
	at java.base/java.io.FileOutputStream.open(FileOutputStream.java:293)
	at java.base/java.io.FileOutputStream.<init>(FileOutputStream.java:235)
	at java.base/java.io.FileOutputStream.<init>(FileOutputStream.java:184)
	at org.apache.pdfbox.pdmodel.PDDocument.save(PDDocument.java:1305)
	at com.cornerpirate.js2pdfinjector.JS2PDFInjector.main(JS2PDFInjector.java:107)```

Please let me know if any solutions found?

CMD: java -jar JS2PDFInjector-1.0.jar IndigoERP.pdf temp.js
ERROR:
[*] JavaScript Payload: temp.js
[*] Output File Path: null/js_injected_IndigoERP.pdf
java.io.FileNotFoundException: null/js_injected_IndigoERP.pdf (No such file or directory)
	at java.base/java.io.FileOutputStream.open0(Native Method)
	at java.base/java.io.FileOutputStream.open(FileOutputStream.java:293)
	at java.base/java.io.FileOutputStream.<init>(FileOutputStream.java:235)
	at java.base/java.io.FileOutputStream.<init>(FileOutputStream.java:184)
	at org.apache.pdfbox.pdmodel.PDDocument.save(PDDocument.java:1305)
	at com.cornerpirate.js2pdfinjector.JS2PDFInjector.main(JS2PDFInjector.java:107)```

Please let me know if any solutions found?

Found Solution:
it's said in README.md file itself,

When running in headless mode if you see a FileNotFoundException like this:
It means you used relative paths to the files. Please supply the absolute path as command line arguments.
So instead of this command:
java -jar JS2PDFInjector.jar dummy.pdf test.js

You should use absolute paths like this:
java -jar JS2PDFInjector.jar /tmp/dummy.pdf /tmp/test.js


Structure projectors not normalized during Kremlin extraction

For the following code:
module Test
open FStar.All

inline_for_extraction
type t1 = {
  b : bool;
}

let  test_fun1 () : Tot bool =
    ({Mkt1 true with b = true}).b

The F* normalizer produces the following AST when extracting with Kremlin, while we would expect the function body to have been normalized to true:
function Debug_Noise_Debug1_test_fun1 <0>(uu____26(0, ): ()): bool
{
  match
  let uu___4_27(0, ): Debug_Noise_Debug1_t1 =
    { b = true; }:
    Debug_Noise_Debug1_t1
  in
  { b = true; }: Debug_Noise_Debug1_t1
  with | λ b(0, ): any.  { b = @0; } ->  @0
}

It can be very annoying when dealing with lifted pure computations and inlined lets, for example when a structure field is used in a conditional branching (which is then not reduced).

duplicate of 2038? :)


Rewritten .timeline files are not accepted by the Timeline applicaton

When rewriting .timeline files, the elements within the <event> subtree may be disordered.

After #2 is fixed, the cause of #1 can also be better narrowed down:
When a scene description is subsequently created in yWriter, the corresponding new <description> entry is appended to the end of the existing <event> subtree during timeline synchronization.
Now a way must be found to insert the new entry at the correct position in the subtree instead.


Find with Guid property

I'm trying to do a Find using a Guid property. It fails using both Query and Linq, no results are returned. This worked previously in v2, but not in v4. Am I doing something wrong, or are guids no longer supported?

Guid are supported in Bson specs. You can query using both Query class or Linq expression. Can you paste your code so I see what's going wrong

Is ServerId property your _id document field? Are you sure that your database constains only data that, in this field, are Guid data type. Because I don't see any problem in your code. This should be woks.... what exception/stacktrace are you getting?

No it's not, it's just an id property to another model. Yes, I'm sure. As mentioned, it all worked correctly back in v2. The exception is an invalid operation on the First call due to the results being empty.

If I retrieve all records and then filter by the Guid outside of the database logic I get the appropriate result back.

After further testing, I've determined it's not just guids. I tried doing a Find on a string property in a different model as well and it still doesn't work. If I change it to a FindAll and then do a Where clause outside of it then it works. This was all functional in v2 though...

This made a big difference... LiteDB v4 uses Linq.Expressions to compile string expressions. This is not work in Unity. You must check last working version of LiteDB for Unity (see Unity3D branch)

That's using v3 right, which has concurrency issues? Do you have any plans to get v4 or future compatible with Unity again?

In v3 you must use your own lock system. Unfortune is not possible be compatible with Unity (net 3.5 compatible) because there are lots of Linq Expressions. I was reading about some next version of Unity that works withs this linq expressions

Unity should support .NET Standard 2.0 (according to this blog post) - hopefully Unity will properly support modern C# once it moves to Roslyn soon.

YOU can use Query().where(x=>x.yourGuid==id)


awating promises  in tearning up/down till the promises be resolved

I am using jest to test creation and deletion of files/folder in cloud provider.
but the issue is that the service provider did not allowed to delete file/folder after creation immediately it need to be after 3-5 min.
I tried various ways to achieve this action in tearning down using  but did not worked for me. my point is to let afterEach holdon for sometime to tear down but it finishes quickly  and quit the test.
and here is a code snippet.
afterEach(async () => {
	try {
		const promise = retryPromiseWithDelay(deleteCompanyFolder, body, 40, 10000);
		await promise;
		console.debug(promise);
		const result = await Promise.all([
			promise
		]);
		console.debug(result);

	} catch (error) {
		console.error(`error afterEach ${error}`);
	}
}, TIMEOUT * 100000);

I have used even
const TIMEOUT = 55000;
jest.setTimeout(TIMEOUT);

and did not worked for me.

So you would want to wait 5 minutes between tests so you are able to cleanup?

So you would want to wait 5 minutes between tests so you are able to cleanup?

Yes, I want to clean up after certain time.

I hope you are kidding :D
As your own words it takes 3-5 minutes to be able to do the cleanup, so in case you have 5 tests it would take you 25 minutes to run it
You should rather mock that service, or build some kind of in memory aproach and only test your bussines logic instead

I used the following code and it worked for me.
const nthTime = 5 ;
jest.retryTimes(nthTime);

// test goes here if failed retry


Option to set custom container image repositories for init containers

The container name and tag can be customized for ngix and galaxy, e.g. through nginx.image.tag and nginx.image.repository. For init containers, the same parameters should exists. Currently hard coded to alpine:3.16.
      initContainers:
        - name: {{ .Chart.Name }}-wait-postgres
          image: alpine:3.16

        - name: {{ .Chart.Name }}-init-cloud-repo
          image: alpine:3.16

and so on.

There's an extraInitContainers option that allows you to specify your own init containers: https://github.com/galaxyproject/galaxy-helm/blob/59c68b17210dcc7dac99f9236916ac89c39b7429/galaxy/values.yaml#L242
Would that be sufficient for your use case? If not, can you describe it in greater detail?

Thanks. I don't necessarily want to specify additional init containers. For my use case, I would like to specify the container image used for the default init containers. The reason is, that I'd like to have them pulled from my private repository.

Right, that makes sense. We should add this in.


Issue with jQuery UI datepicker inside colorbox

I'm basically injecting raw HTML into colorbox's HTML property and calling the onComplete function to add datepicker. It works for inline, but not when using the HTML property. Is this an existing bug or is it an issue with the datepicker itself?
onComplete: function() {
     $(".datebox").datepicker({
              changeMonth: true,
              changeYear: true,
              ...

I got the same issue.
I used an id as selector, but colorbox duplicate the whole html content and the id's too.
So if I use $('#myid').doSomething() I got an error, because "myid" was duplicated.
Thats the issus.
Use a better selector like $('#colorbox .myidasclass').doSomething() to solve the issue.
The same problem is with class selector without the '#colorbox' as additional selector.


Update README's guide on how to link with RNPM in the postinstall hook

Ah, @geof90 but keep in mind postinstall hook is only intended for your project, not for the dependency, so there's no need to ../../ if I get that right.

I think he wants to run an rnpm link on plugin install

so the default dir would be xxx/node_modules/rnpm-plugin-code-push

@grabbou, I thought the whole point of putting rnpm link inside a postinstall hook is so that when a React Native developer wants to add a plugin to his/her app, he/her simply needs to npm install that plugin and it will be automatically linked.
In this case, my project (i.e react-native-code-push) IS the plugin, and therefore IS the dependency of another project that wants to use it as a plugin. I thought this was the whole point of using postinstall. Perhaps I'm mistaken.

Ah, fair enough. I get the use case for now.
I still believe that postInstall hooks should stay on user project level only so that they have full control over what manipulates their projects (and whether they want to use rnpm or not). All in all, postInstall hook will run after every single npm install, so there's no difference for end users whether it's defined on their level, or on dependency level.
If you still want to do it, I'd see that as an extra readme section, although as I said - I am still not sure if that's the behaviour we want to promote / expose.
What do you think?
Also, re: your snippet, in order to avoid rnpm being fired too many times during npm install, I think developers should use the below postinstall hook instead:
cd ../../ && rnpm install package-name

so it does not scan the entire tree but goes directly to the installed project.

Are you perhaps saying that rnpm + postinstall is meant for the package.json of the RN project itself? Because if so, I fail to see how that is a practical scenario. Nobody would want to add that to their package.json dependency. Even in the scenario where multiple people are working on a project, the gradle xcodeproj files would be checked into their code repositories, therefore there's no need to add the rnpm link step to postinstall since the dependencies would already be linked. I always thought that my scenario (letting plugin developers simplify plugin linking by adding rnpm link to their postinstall hooks) is the main scenario this solution was trying to tackle.

Are you perhaps saying that rnpm + postinstall is meant for the package.json of the RN project itself?

Yes.

Even in the scenario where multiple people are working on a project, the gradle/xcodeproj files would be checked into their code repositories, therefore there's no need to add the rnpm link step to postinstall since the dependencies would already be linked.

Well, that way you don't need package.json as long as you commit node_modules folder? To me that's quite reasonable to automatically perform some extra actions after dependencies are installed (similar to what people do with gulp build tasks etc.).

I always thought that my scenario (letting plugin developers simplify plugin linking by adding rnpm link to their postinstall hooks) is the main scenario this solution was trying to tackle.

That's very interesting possibility, although I'd prefer avoiding rnpm link from being run that way. It not only makes the development trickier (clone repo, npm install - rnpm tries to link to ../../ which can be anything) but also produces a chaos in what's going on when you run npm install. Having a single point of entry makes it easy to control rnpm version for the user and to decide whether he wants automatic linking, no linking at all or anything.

Deleted my last comment. I see what you are saying. I had no idea that the postinstall would also run when you install dependencies, like what you are suggesting. So it seems your advice is, we should prescribe to our users to add rnpm link directly to their own package.json configs so that the automated linking will happen when they install our plugin as a dependency? If that works, awesome! And I guess I learnt something new about NPM today :p
I'll close this PR for now. Thanks for the reviews and help!

Actually sorry - turns out I was wrong - postinstall won't run on the parent project when you explicitly specify dependency name to install. Leaving that opened then, because the issue you described was actually valid - in order for automatic linking to work, package authors have to add postinstall hooks in their package.jsons.

Ah... ok. Thats a bummer.
Actually I suggested to @Kureev on Discord about the possibility of you guys creating a simple script that does the cd ../../ and calls rnpm link for us, so we could point the postinstall to that script instead. Like maybe rnpm link-self. That would be the ideal solution. In the absence of that, this workaround would have to do, unless you have better ideas!

Another solution could be adding logic in the rnpm link to detect if it is in the node_modules folder. Something like:
if (/node_modules\/plugin-name/.test(process.cwd())) {
  process.chdir('../..');
}

@grabbou just saw your comment edit. Your solution definitely works too. The only tradeoff is that it forces the user to install rnpm globally. Which is okay I guess.

Well, I think rnpm should be installed globally. So I'm also up for @grabbou's variant. Make an install plugin and aggregate npm install <package> + rnpm link <package>. I'll take care about it.
ETA: this weekend

@Kureev I think it should be rnpm install  to avoid breaking changes and make it super light-weight.

In this case, is there any value in attaching RNPM to postinstall hooks at all then? Seems like the answer going forward is no

Yep, for me making something like cd ../../ && rnpm link <package> is a bit tricky. I think rnpm install will do the trick without any additional efforts from the plugin authors.

@Kureev I'll add a warning when one calls rnpm link <abc> and we cannot locate abc that oops, you have probably wanted to npm install it first. Do it or use rnpm install

Agree

Anyway, I'm going to close this PR. Hope everybody agree on it.

see https://github.com/rnpm/rnpm/pull/87


Github team management + setting and enforcing standards on all repos

This is covered in #16, #19, and #23.


Roe 2020 api model structure versioning

JIRA link
https://companieshouse.atlassian.net/browse/ROE-2020
Change description
Changes to facilitate calling API end-points and retrieving older versions of ROE submissions. Going forwards, JSON
data written to Mongo will have a schema version set. The schema version will be inferred for older JSON records. The
version is used to convert JSON to the latest version of the model, before being used/returned by the API.
Work checklist

[X] Tests added where applicable

Merge instructions
We are committed to keeping commit history clean, consistent and linear.  A BREAKING CHANGE can be part of commits of any type,
types other than fix: and feat: are allowed, for example build:, chore:, ci:, docs:, style:, refactor:, perf:, test:, and others,
footers other than BREAKING CHANGE: <description> may be provided.

I'm back and from my perspective this can be merged, as soon as that missing unit-test is added.


Beta 0.87.0b0 show wrong color in history for HmIP-SWSD

Home Assistant release with the issue:
0.87.0b0
Last working Home Assistant release (if known):
0.86.4
Operating environment (Hass.io/Docker/Windows/etc.):
Hassio
Component/platform:
Homematic IP
Description of problem:
Sine Beta 0.87.0b0 the HmIP smoke detector is now ok from state. No longer smoke detected shown BUT
No if you click on the sensor or in History the "good" (no smoke) is red und de "bad" state (smoke detected) now green. Is more a cosmetic topic but should be fixed if possible.

@danielperna84
May cause by #20397

@awarecan
Nope, I'm pretty sure he is using the homematicip_clound component, not the homematic (non-cloud) one. The non-cloud variant has different attributes. I think @sander76 or @mxworm take care of that component.

Please have a look on the binary_switch documentation under https://www.home-assistant.io/components/binary_sensor/ . I'm not sure if this is a real bug.

@mxworm what I can say, bevor the "smoke detected" bug the color was vice versa .. green for good (no smoke) and red for bad (smoke detected) and from my point of view this makes more sense.  But what I always mentioned , is more cosmetic issue.

I think that's a front-end bug and not a component bug.
The "door" is also inverted (closed=Red, open=green)

0.87.0 Das Problem ist nicht behoben. Wenn kein Rauch dann Rot, Alarm- Grün, bei dem Wassermelder ist genauso: Trocken-Rot, Nass- Grün. Das soll doch andersrum sein.

Hi,  from component level I don't have any influence on the history color - at least I think.

Hi @MartinHjelmare i think this is a ui issue, and could be moved to polymer repository.
Thanks

/move to home-assistant-polymer


Implement JPN country plugin

I read through the country plugin documentation and it seems like the main thing to define is long_term_capital_gain_period(). However, Japan has no long term capital gains benefits. Everything crypto is taxed as short term. To be more specific, it is taxed as regular income.
So, how should I define this?

You can implement it as:
return sys.maxsize

The value is so large that every taxable event will be short term.

This is now merged.


Grid view like Windows 11 "Installed app" settings view

Add a toggle to enable more views, like Windows ones in "Installed apps" in settings:

Yeah, why not


Any plan to add Kotlin Multi-platform Mobile support?

This seems great, compared to MPAndroidChart, since KMM is reaching stability, Any plans to support KMM?

Hi @kaushalyap,
KMM support is not our priority currently, but we’ll look into supporting it in the future.


Replay2 or Relax2

Do you mean Relax2 ?

This is my first issue, so bare with me if i do something wrong
Yeah i meant Relax2, Started using a map at the conversion to minimize errors like this


Add fultons

Stackable entity that applies fulton to target and after 45 seconds goes to the specified beacon.
Stackable and somewhat cheap to craft (2 cloth) as it'll be necessary for salvage.

https://github.com/space-wizards/space-station-14/assets/31366439/a69da49a-20bc-43ee-867b-2cfa504dccbf
Video shows:

Folding
Invalid fultoning
Successful fultoning

:cl:

add: Added fultons for salvagers.

Can you fulton crit/sleeping mobs?

"We're gonna extract him too, boss?"

this seems to be missing all the animation work from the /tg/ implementation that actually communicates what's happening to the player
you also shouldn't be able to fulton something without linking to a beacon, and you should be able to pick your beacon from the balloon pack itself


dart2js unfork code size regression

This issues track unfork dart2js code size metric
2020/04/20 criteria
Benchmarks below are currently blocking on the 'production' configuration.
AsyncStressTest benchmarks
Several members (Stream and Futures) at -50% which should be no worse that -10%
Iteration benchmarks:
Iteration.concat.manual2.typed -38%
Iteration.deeptree.syncstar -27%
Iteration.concat.syncstar -15%
These iteration benchmarks need to be understood and mitigated.

production CompileAcsGallery: +2.041%
spec mode CompileAcxGallery: +1.597%
CM main: +0.584%

We should get #s for Flutter Gallery as well.
Do you think we can bring the ACX overhead down soon?  Do you have a sense of why that's higher than CM?

My ancient copy of flutter gallery is 1.12% (production).
The size regression seems to reduce as the app grows larger. Several contributing factors:

There are more types in the type pool - e.g. int and int*. This is mostly because the sdk is opt-in, but other code is opt-out. The effect is somewhat limited by the size of the sdk vs other code. CM is the only app where the sdk is insignificant.
There are more type operations - 25%-50% more to dynamically 'star' a type since type the type variable access T is now T*.
The type relationship A<T> extends B<T> is now A<T> extends B<T*> which takes extra space to encode.

No update

No update

Per my update here - https://github.com/dart-lang/sdk/issues/41431#issuecomment-615500143 - Flutter gallery appears well within 1%.
Is there anything still here we would block unforking on?

production CompileAcxGalleryMinify: +1.118%
production CM main: +0.591%

Synced with @sigurdm And @rakudrama , the code side is within range to unfork.  If require, new issues can be open in future milestone

@sigurdm you can ignore this, @franklinyow was referring to me :)


2.1.50 released

Since the new version is finally released, would it be possible, that anybody updates the addon? That would be amazing 🥇

Potential fix here

On Linux 2.1.50 anki-2.1.50-linux-qt5.tar.zst
When loading 'No-Distractions-Full-Screen-master':
Traceback (most recent call last):
  File "aqt.addons", line 244, in loadAddons
  File "/home/user/.local/share/Anki2/addons21/No-Distractions-Full-Screen-master/__init__.py", line 623, in <module>
    addon_view_menu = getMenu(mw, "&View")
  File "/home/user/.local/share/Anki2/addons21/No-Distractions-Full-Screen-master/toolbar.py", line 12, in getMenu
    menu = a.menu()
AttributeError: 'QAction' object has no attribute 'menu'

Same message at Win10 2.1.50 Qt6, but it's working :D

@omega3 I'm on Qt6 if that helps. I updated the release. See if that helps

@mobedoor mobedoor
Thank you very much. It even removed issue 42 that I reported.

@omega3 I'm on Qt6 if that helps. I updated the release. See if that helps

In my case it didn't change anything. The error stills occurs, but it's working.


2d z-ordering issue

currently 2d z ordering seems borked, the interpolation automatically sets everything too z = 0 and in 2d layering is usually done with the z axis of transform. is this accounted for somewhere else that im missing?

another interesting issue i ran into was some weird transform/position discrepancies related too collider spawning that stop happening when i disable the plugin, but im pretty sure this is system ordering related


Support placeholders

Currently, placeholders are not supported in the search or replacement text. It would be really nice to be able to use them.

The problem with placeholders is that they're dynamic. The regexp are compiled ahead of time for performance. If they weren't, then it would be much slower.
While the replacements don't need to be compiled, they are prepared ahead of time to be passed to https://pkg.go.dev/golang.org/x/text/transform which performs the actual replacements (via https://github.com/icholy/replace which implements the regexp part). It might be possible to use replace.RegexpStringFunc instead of replace.RegexpString, but the tricky bit is getting the request into this function, @mholt might have ideas.

Yeah, so, we can add placeholders BUT, as Francis said, the values are compiled ahead of time (so we could only perform replacements on things like env variables and other, non-HTTP-request-specific variables), for performance reasons. Is that sufficient?

My use case has to do with, among other things modifying javascript returned through a reverse proxy. It does require HTTP-request specific variables but no regex - although regex would help :-)
My case, there will probably never be more than 30 versions of a regex so lazy creation of the compiled regex and keeping a lru capped at 100 or 1000 of them would be more than enough for our code.

@mheyman To clarify, all replacements are processed at config load (startup), not at every request, whether using regular expressions or not:
https://github.com/caddyserver/replace-response/blob/9d5652c0256308fddaef1453d463d2a281498cb6/handler.go#L88-L99
Unless there's a clever solution I haven't thought of, it's likely that implementing this feature will cause a performance regression.

I would hope there would be no performance regression because in the case where the information is available at config load, no change in ServeHTTP() is required. But, if using placeholders, a different (slower-running) ServeHTTPWithPlacehoders() stage could do the transform in a more flexible way - perhaps with an lru of compiled-as-needed regex (I'm not a go coder so I wouldn't presume to know the best way to go about this).
In our use case, we probably wouldn't notice the slowdown this alternative middleware would cause over the fast path because we only have a few users hitting the pages over an already-slow link. Also, like I mentioned earlier, there would only be 30ish different possible searches in our case so a reasonably sized LRU cache would easily hold all our possible searches. We can code these 30ish search strings manually in the current implementation but it means rewriting the config and restarting the reverse proxy when any of the 30ish proxied services change (happens often). Additionally, we may not always have insight into when things change except when things break. And we'd rather not run with that method of operation :-)

I guess that's technically possible, but it adds a lot of complexity.

@mheyman @mholt  I posted my 2 cents in another thread of issue discussion titled "placeholders not support {http.request.host}" please check that out and share your comments and thoughts.

Should be closed by #35 I think! Thanks @jshenguru and @icholy!


Add speaker

Edit from Jason: add Sebastien Tosi as a speaker

Builds well and looks good on https://snoopycrimecop.github.io/www.openmicroscopy.org/
Ready to merge fmpov.
There could be a release of www pages on Monday ?

@lunson Please note that the registration opened just now https://forum.image.sc/t/ome-community-meeting-may-26-28-dundee/32924/4

@pwalczysko I'll add registration and another edit from Jason to this PR. Thanks.

Builds well and looks good on https://snoopycrimecop.github.io/www.openmicroscopy.org/.
If you @lunson and @jrswedlow are happy, I can merge and deploy www site with this tomorrow.

@pwalczysko Jason asked me to remove the registration button from Home page only. This is good to go live now. Thanks!

@lunson and @jrswedlow understood and looks good and according to what you indicated on https://snoopycrimecop.github.io/www.openmicroscopy.org/

Merging to deploy