custom sling.properties not picked up during instanceSetup

Followed the documentation and placed my custom sling props in src/aem/localInstance/common/crx-quickstart/conf but it is not being picked up during any of subtasks of instanceSetup task. No errors in the logs. Tested on 14.2.24 as well as some older versions.

@mierzwid you worked in this area more recently than I

Looks like a BUG - I will verify it with one of our simple examples: https://github.com/Cognifide/gradle-aem-plugin/tree/master/docs/blog/2020-02-aem-on-java-11

how about:

so the answer is: src/aem/localInstance/override/common


apollo_canbus&chassis&teleop  adaption bug report

最近负责Apollo与小车底盘适配工作，在调试过程中遇到一些问题。
大概情况描述如下:我们根据<Apollo车辆适配教程>完成了CAN协议设计，DBC转换等工作。测试时使用Apollo自带的底盘联调工具teleop（用键盘按键生成控制命令），发现控制命令无法下发给底盘。根据打印的调试日志来看，应该是canbus模块没有成功调用onControlCommand回调函数，初步猜测是控制命令不符合回调函数要求，导致调用不了回调函数。
请问有大佬可以帮忙看看什么问题吗？
Recently, I was responsible for the adaptation of Apollo and trolley chassis, and encountered some problems during the commissioning process.
The general situation is as follows: We have completed the CAN protocol design, DBC conversion and other work according to the. During the test, Teleop, the chassis joint debugging tool provided by Apollo, was used to generate control commands with the keyboard keys. It was found that the control commands could not be issued to the chassis. According to the printed debug log, it should be that the canbus module failed to call the onControlCommand callback function. The initial guess is that the control command did not meet the requirements of the callback function, resulting in the failure to call the callback function.
Excuse me, can you help me to see what the problem is?

鉴于你提供的信息，大概率是没有添加相应的注册函数，你可以参考以下文档。
Apollo车辆适配教程

1、有没有将新的车辆controller添加到工厂模式？
2、message manager上有没有sender message？
3、具体的执行接口（转向、驱动、制动）等有没有写具体实现；
4、checkresponse 是检查底盘在自动驾驶，如果使能后底盘是否进入自动驾驶？


[Event Request] Codeunit 80 "Sales-Post" – New Event “OnRunOnBeforeCheckTotalInvoiceAmount”

Could we please have a new publisher “OnRunOnBeforeCheckTotalInvoiceAmount” in Function “OnRun”, Codeunit 80 "Sales-Post”?
    trigger OnRun()
    var
        SalesHeader: Record "Sales Header";
        TempInvoicePostBuffer: Record "Invoice Post. Buffer" temporary;
        TempItemLedgEntryNotInvoiced: Record "Item Ledger Entry" temporary;
        CustLedgEntry: Record "Cust. Ledger Entry";
        TempServiceItem2: Record "Service Item" temporary;
        TempServiceItemComp2: Record "Service Item Component" temporary;
        TempVATAmountLine: Record "VAT Amount Line" temporary;
…
        FillTempLines(SalesHeader, TempSalesLineGlobal);
        TempServiceItem2.DeleteAll();
        TempServiceItemComp2.DeleteAll();

*** start new code ***
        OnRunOnBeforeCheckTotalInvoiceAmount(SalesHeader);
*** stop new code ***

        // Check that the invoice amount is zero or greater
        if SalesHeader.Invoice then
            CheckTotalInvoiceAmount(SalesHeader);

*** start new code ***
    [IntegrationEvent(false, false)]
    local procedure OnRunOnBeforeCheckTotalInvoiceAmount(SalesHeader: Record "Sales Header")
    begin
    end;
*** stop new code ***

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


Palette should have open/close animation

Currently the palette opens/closed immediately but should have some animation.  Most likely productive from
https://v10.carbondesignsystem.com/guidelines/motion/overview/

Not sure when it as implemented but the palette opening and closing is now animated
https://github.com/user-attachments/assets/658f74cb-bbc5-4b44-89ab-fe36fec2a60e


How to optionally allow negative numbers?

Hi! Is there any way to optionally allow negative numbers? For example allow both 99 and -99 as valid inputs?
Thanks

Hi, you can try something like this:
<input type="text" ng-model="maskModel" mask="*99?" restrict="reject" validate="false" data-ng-pattern="/^(-(\d){1,2}|(\d){1,3})$/" />

Thank you so much! I will give it a try. :)


added coming soon page

I have added coming soon page with respect to an issue #86

https://github.com/amupedia2021/Project-Amupedia/assets/91741581/3d875961-7dcf-4c26-b98c-912d7d4535c5
Checklist before requesting a review

[x] I have performed a self-review of my code.
[x] Starred the repository.
[x] Added My Name to the Contributor's List.
[x] Followed the repository's Contributing Guidelines.
[x] I ran the app and tested it locally to verify that it works as expected.

i have added to my review. will review soon thanks

i have added to my review list. will review soon thanks

Here it is showing that the branch has conflicts but Resolve Conflict option is disabled. How can I solve?

try to resolve conflicts at your local. pull the new commits and there conflicts will come to your local too. resolve those and push.https://www.simplilearn.com/tutorials/git-tutorial/merge-conflicts-in-git

Hi @anmode I have solved the merge conflicts.

congrats!!

congrats!!

I didn't like the UI, I have better update Fix ?


[fix] Update level property to change transport level #1191

This should fix #1191, see the added testcase.

Also related #1167

@ChrisAlderson @DABH opt'ed for the latter – the change wasn't exactly the way I predicted, but https://github.com/winstonjs/winston-transport/pull/26 is close.


Bump firetv to 1.0.8

Description:
This is pull request 1 of 3 that I have planned for the Fire TV component.

Bump the version of the firetv package.
Allow for using an ADB server to send ADB commands, in addition to the current method (the Python implementation of ADB).
Register a media_player.firetv_adb_cmd service.

Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass
[x] There is no commented out code in this PR.

If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated in home-assistant.io

If the code communicates with devices, web services, or third-party tools:

[ ] New dependencies have been added to the REQUIREMENTS variable (example).
[ ] New dependencies are only imported inside functions that use them (example).
[ ] New or updated dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[ ] New files were added to .coveragerc.

If the code does not interact with devices:

[ ] Tests have been added to verify that the new code works.

I'm closing this in favor of https://github.com/home-assistant/home-assistant/pull/21221.


Listing Management - Org Steward Tabs do not work on first attempt

In Listing Management the first time that you click on an Organizational tab the screen appears to update, but does not actually change the apps displayed.
If you then choose My Listings and then go back to Org tab, the results are updated as you would expect.

Passed testing in Master (IE9/11, FF, CH).  OrgSteward1 goes to Listing Management, selects TO3 (displays 11), selects My Listing, selects TO3.  Displays TO3 (11), TO2 (18), TO (24) as expected.


badge

Username and Discrim: Parzival#5075
User ID:<PHONE_NUMBER>80750336
Image Link: https://i.imgur.com/rC5TAWU.gif

done~


Add bold: variant

Most colors have a bold variant by appending _bold to the end (ex.: yellow -> yellow_bold).
See http://doc.cogciprocate.com/colorify/colorify/macro.colorify!.html or use: colorify!(help); for a full list.

Yes, but how to get bold without any specific color (i.e. with default color)? And even if I have to specify some color, there is no "black" one in the list.

I'll add a black and black_bold for you as soon as time permits.

25787115e09764fe916206cfe603971d96e20e9b


Render trailing punctuation marks after a block equation on the same line as the equation

The problem
When a block equation is part of a sentence and followed by a punctuation mark, like a period, you have to put it inside the equation for it to not be rendered on the line bellow the equation (see examples bellow). This is frustrating because (1) the period is not technically part of the equation, and (2) the math font is used instead of the text font.
Example 1
Notice how the period is inside the equation in the Typst code, and rendered correctly: on the same line as the equation, right after it.
Binomial coefficients are defined for all $n in NN$ and $k in [|0, n|]$ by the formula
$ binom(n, k) := (n!) / (k! (n - k)!). $

Example 2
Notice how the period is atfer the equation in the Typst code, and rendered incorrectly: on the line bellow the equation.
Binomial coefficients are defined for all $n in NN$ and $k in [|0, n|]$ by the formula
$ binom(n, k) := (n!) / (k! (n - k)!) $.

The solution
Although fixing #1125 would likely provide a way to solve (2), it still won't solve (1).
My suggestion it that punctuation characters (which I recognize might not be well defined for non-latin languages) that are right after the closing $ of a block equation should be displayed on the same line as the equation. A potential problem with this specific solution would be languages that require a space before some punctuation marks (like french with the colon), and it could maybe increase the complexity of the parser.
Use case
Having cleaner markup because punctuation marks do not have to be included within an equation to be rendered at the right place.

See my comment here https://github.com/typst/typst/issues/3936#issuecomment-2057008693

I wrote an example here https://github.com/typst/typst/issues/3936#issuecomment-2057470137 that illustrates one logical place that a resolution to this issue would be useful.


[Draft] Add Optional Score Filtering to AniList UserList Builder

Description
Add Score Filtering functionality to the AniList Builder for UserList Queries, which allows setting a specific score_format for the query to return, a score value used for comparison, and a modifier field. AniList allows different score formats to be used on a per account basis, so this will allow the query to transform the score based on the user's choice.
Since the GraphQL query does not support score filtering by default, I appended the functionality within the _userlist function within anilist.py. Because I have done None checks for specific arguments, this should be a non-breaking change and will not affect current configurations.
Main reason for adding this feature is so I can auto filter based on my own Completed List and only put shows that have been rated higher than a certain score within the Collection. Because this was a quick code change done late at night, I haven't tested it and have marked the PR as a Draft. I will continue to work on this to fully test the functionality and document the changes. I did also want to get feedback on whether this feature would be in the scope of the project.
Type of Change
Please delete options that are not relevant.

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

Checklist

[X] My code was submitted to the nightly branch of the repository.
[ ] I have tested the code to ensure functionality with no regression.
[ ] I have documented the functionality on the corresponding/relevant files.

I notice there are no updates to the wiki pages, I'm guessing any new functionality the user can choose from should be detailed in the wiki

@ZhunCn ive implemented it in a different way see the nightly wiki for how to use can you test it for me?
https://metamanager.wiki/en/nightly/builders/anilist/#anilist-userlist


Zone switches don't always show up?

I'd be surprised if there was anything you're doing in here which would cause this glitch, but I've been using HomeKit and Homebridge devices within it for several years and have never seen anything like this. Perhaps some sort of race condition? It's not easy to reproduce.
https://user-images.githubusercontent.com/208647/163857428-d7491789-2ff5-4f41-a8de-743ba67def14.mp4

This appears to be effecting any app or plugin using an irrigation system device following the list IOS update.
It seems that if you click the tile it will be blank, yet if you "long" press it will show fine.
This bug does not seem to have any effect if you do not display as an irrigation system. May that will work for a bit until Apple fixes this.

This is also happening to me - regardless of long press or short tap.

I have the same issue.

I've since tried out other sprinkler controllers in HomeKit and even Rachio, which natively does their own HomeKit, has this problem. I doubt there's anything that can be done here.

I was able to confirm this issue of not displaying an irrigation system correctly is fixed in IOS16.0 (public beta)

Thank you! I noticed this as well.On Jul 23, 2022, at 10:37 AM, valiquette @.***> wrote:
I was able to confirm this issue of not displaying an irrigation system correctly is fixed in IOS16.0 (public beta)
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you commented.Message ID: @.***>

Had this issue with iOS 15... this is no longer an issue with iOS 16. im running the first public beta

Fixed in IOS 16


Bump to laravel dusk 5.0

Signed-off-by: Mior Muhammad Zaki<EMAIL_ADDRESS>

Coverage remained the same at 83.594% when pulling 316a28fb45c8c73bbb6516705407ba9d01f0af54 on dusk-5 into 3eeafbb5ec12afd77febf4dd8dc604357658e1d1 on 3.7.

Merged

thanks @crynobone - lots of stuff on here today!

@Keoghan no problem, just have some available time today to look into this. But I don't have any project using orchestra/testbench-dusk. Might want to look if anything doesn't work as expected.

BTW master is failing right now, but I suspect is due to https://github.com/laravel/framework/commit/c37702cbdedd4e06eba2162d7a1be7d74362e0cf not yet available in 5.9


Throw an exception if the input GPX file doesn't contain trackpoints

Otherwise the result returned by the map matching API would be an GPX
file without any trackpoints.
Without this patch:
michael@hatano:~/git/github.com/graphhopper/map-matching$ curl -XPOST -H "Content-Type: application/gpx+xml" -d @matching-core/src/test/resources/test-only-wpt.gpx "localhost:8989/match?vehicle=car&type=gpx"
<?xml version="1.0" encoding="UTF-8" standalone="no" ?><gpx xmlns="http://www.topografix.com/GPX/1/1" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" creator="Graphhopper version 0.11.0" version="1.1" xmlns:gh="https://graphhopper.com/public/schema/gpx/1.1">
<metadata><copyright author="OpenStreetMap contributors"/><link href="http://graphhopper.com"><text>GraphHopper GPX</text></link><time>2018-06-14T11:34:16Z</time></metadata>
<trk><name>GraphHopper Track</name><trkseg>
</trkseg>
</trk>
</gpx>

With this patch, an error message will be returned (the wrong format is a different issue and will be fixed by https://github.com/graphhopper/graphhopper/pull/1390
curl -XPOST -H "Content-Type: application/gpx+xml" -d @matching-core/src/test/resources/test-only-wpt.gpx  "localhost:8989/match?vehicle=car&type=gpx"
{"code":500,"message":"There was an error processing your request. It has been logged (ID 7fb0f76e14f13322)."}

Returning an empty GPX response is confusing.

Hmmmm. How is an empty GPX response to an empty GPX request confusing? Am I being too literal-minded here? Map-matching is an operation on lists, and an empty list is a list, too. How is 0 points more special than, say, 1 point?
I'd say we return an empty list for the same reason that "".reverse() returns "" instead of throwing an Exception.
Or am I missing something, like an empty GPX file being really illegal (not an expert)?

Arguments that would convince me otherwise would be, for example:

If I convert it to a LineString and try to draw it, it fails, because empty LineStrings are illegal. (Is that true?)

I also think an empty GPX file should not be handled like an error. Especially not as a server error 5xx.
Maybe we handle it as a usage problem and throw a 400 for an empty list i.e. wrong usage? (We do this for /route if no points)

If I convert it to a LineString and try to draw it, it fails, because empty LineStrings are illegal. (Is that true?)

No. An empty LineString is ok as I recently found out :)

Maybe we handle it as a usage problem and throw a 400 for an empty list i.e. wrong usage? (We do this for /route if no points)

But my point was: What's wrong about it? Literally, what's wrong with empty input -> empty output? I tend to think that it's a good thing when code does not have corner cases (like throwing exceptions on n=0), so I'm a bit reluctant to put them in explicitly, without need.

I'm in traffic modelling, so I think of a GPX file as, for example, "things a person was known to be doing on a day". What's wrong with "nothing"?

There is nothing wrong :)
Still we could prevent certain incorrect usage (not saying that we should). What if someone intends to map match a GPX file with wpt and as we consider trk only we would return an empty response? So we would return nothing to a GPX input file filled with content without a warning.
(But we could also just document this behaviour explicitly and change nothing else)

Or am I missing something, like an empty GPX file being really illegal (not an expert on GPX files)?

I am no GPX expert either but please keep in mind that GPX files can contain waypoints, tracks (with track segments containing trackpoints) and routes.
I am developing map matching for my railway routing. Unfortunately, my customer took "supports GPX" too literal. He converted a CSV file of measurements into a GPX file using ogr2ogr and had a file of waypoints, not of trackpoints. Luckily, my MatchResource class is implemented a bit different than yours and that's why he got the error message of an exception raised at a later step in the processing (of the map matching). Default GraphHopper would have returned an empty track and my client application to render it—or fail (not tested).
Did you notice that if someone sends a GPX file with more than waypoints or routes, the response only contains the result of the matching of the trackpoints to the network?
My aim is to throw an error if a GPX file is valid (in terms of XML and the schema) but does only contain unuseable items (waypoints and routes). GraphHopper only reads trackpoints. The check if the list of GPXEntry is empty was the easiest solution. Shall I check if the XML document contains any waypoints or routes and throw an error only if no trackpoints but at least one route or waypoint is in the GPX file?

I also think an empty GPX file should not be handled like an error. Especially not as a server error 5xx.

The 500 response is caused by a minor incompatibility of the map matching module and current GraphHopper master branch. I added it to the unit test in order to make the unit tests pass. It is related to https://github.com/graphhopper/graphhopper/issues/1389
You can leave this pull request open until https://github.com/graphhopper/graphhopper/pull/1390 is merged. I will change my pull request in order to return code 400 then.

My aim is to throw an error if a GPX file is valid (in terms of XML and the schema) but does only contain unuseable items (waypoints and routes). GraphHopper only reads trackpoints.

Okay got it, thanks!

Shall I add NodeList wpl = doc.getElementsByTagName("waypoint"); and look if it is empty (and the same for route points)?

Shall I add NodeList wpl = doc.getElementsByTagName("waypoint"); and look if it is empty (and the same for route points)?

Almost -- I moved your test to the front, and now consider a GPX input without tracks to (very probably) be a user error.
Properties of that solution:

A user (like your customer) who doesn't convert their data to tracks, but to waypoint or routes gets an error.
A user who does  convert their data to tracks, but sends an empty dataset, will not get an error, because it contains a track, but without trackpoints.
We don't say that an empty list of points is an illegal map-matching input.
We don't say that a GPX file without trackpoints is an illegal GPX file.


build: bump version

Codecov Report

Merging #85 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #85   +/-   ##
=======================================
  Coverage   85.19%   85.19%
=======================================
  Files           8        8
  Lines         608      608
=======================================
  Hits          518      518
  Misses         51       51
  Partials       39       39

Flag
Coverage Δ

#unittests
85.19% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Continue to review full report at Codecov.

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

/status can-merge


Testing pages

I am currently testing fixes, tweaks and additions against client sites which often have a lot going on and may not have a need for what I'm working on. This adds overhead to testing new changes.
I'd like some form of testing within this repo, not currently sure to what extent.
The basic would be simply a set of pages that load the latest version and apply it in several simple use-cases that can be manually checked.
Could also look into testing frameworks to make it more automated, I wonder if I can see how Slinkity handles their tests for inspiration.

Added in 0.3.0


Fix typos

I created https://github.com/gradle/gradle/pull/21704 which is identical to this one, except its commit contains "Signed-off-by:", so it ought to pass the DCO test.

Superseded by https://github.com/gradle/gradle/pull/21704


Remove kernel_registry and call backends directly.

Remove kernel_registry and call backends directly
Allow for forward function to save tensors that will be needed by the backend function. We pass dy and the saved tensors to the backprop function.
Remove logic in tape to handle multiple outputs. We can bring it back if needed.
consolidate tape_types.ts and tape_util.ts into a single file tape.ts
in debug mode, log the name of the activeScope, which should point to the last operation which called the kernel. The aggregation of these and showing hierarchical debug view is still a TODO.

This change is

Awesome work, glad you're killing the tape types :)
Only major comment is in tensor.ts

Reviewed 49 of 79 files at r3.
Review status: 49 of 58 files reviewed at latest revision, 3 unresolved discussions, some commit checks failed.

src/engine.ts, line 37 at r3 (raw file):

export type ForwardFunc<T extends Tensor> =
    (backend: KernelBackend, save?: <S extends Tensor>(tensor: S) => S) => T;

it'd be nice to document "save" somewhere, maybe here. Just say that these are tensors computed in the forward pass that we need in the backwards pass.

src/tape.ts, line 121 at r3 (raw file):

      const prunedNode = Object.assign({}, node) as TapeNode;
      prunedNode.inputs = prunedInputs;
      prunedNode.output = prunedOutputs;

you can consolidate this block to:
prunedNode.output = node.output
and completely remove "prunedOutputs"

src/tensor.ts, line 598 at r3 (raw file):

    return ops.subStrict(this, x);
  }
  pow<T extends Tensor>(this: T, exp: Tensor): T {

this change seems a little weird to me - why are the chained ops now taking this as the first argument, and why not all of them?

Comments from Reviewable

Review status: 49 of 58 files reviewed at latest revision, 4 unresolved discussions, some commit checks failed.

src/engine.ts, line 102 at r3 (raw file):

      return x;
    };
    const kernelName = this.activeScope.name;

this is kind of funny, won't the active scope name be the operation name because of @operation and the scope around the operation, and not the actual kernelName?

Comments from Reviewable

Review status: 49 of 58 files reviewed at latest revision, 4 unresolved discussions, some commit checks failed.

src/engine.ts, line 102 at r3 (raw file):
Previously, nsthorat (Nikhil Thorat) wrote…
this is kind of funny, won't the active scope name be the operation name because of @operation and the scope around the operation, and not the actual kernelName?

One idea is to keep passing a string for the kernel name just for logging.

Comments from Reviewable

Review status: 48 of 58 files reviewed at latest revision, 4 unresolved discussions, some commit checks failed.

src/engine.ts, line 37 at r3 (raw file):
Previously, nsthorat (Nikhil Thorat) wrote…
it'd be nice to document "save" somewhere, maybe here. Just say that these are tensors computed in the forward pass that we need in the backwards pass.

Done.

src/engine.ts, line 102 at r3 (raw file):
Previously, nsthorat (Nikhil Thorat) wrote…
One idea is to keep passing a string for the kernel name just for logging.

Great observation. So I was thinking about this a little.
There will be a skew between kernels and user-ops and I think for now, before we have aggregation done, it's better to report user-ops since that is what the user controls (we report the inner-most named scope, with ability to aggregate).
Couple reasons for this argument:

When we backpropagate, if we see that a tapeNode is missing gradFunc, (meaning we haven't implemented gradient yet), it's more useful to report an error that this opName is missing gradients, so we can't backpropagate, as opposed to this kernelName is missing grad, since from the user perspective, the culprit is dl.opName(), not dl.kernelName().
When reporting timing in debug mode, kernelNames might be confusing, e.g. imagine we reuse a conv2d kernel for higher-order convs, reporting the logical conv is more helpful than reporting conv2d (from the user perspective).

Once we have aggregation done, we can come back and decide maybe to report both kernel and op, at which point, we can add a required name argument to engine.executeKernel().
WDYT?

src/tape.ts, line 121 at r3 (raw file):
Previously, nsthorat (Nikhil Thorat) wrote…
you can consolidate this block to:
prunedNode.output = node.output
and completely remove "prunedOutputs"

Done.

src/tensor.ts, line 598 at r3 (raw file):
Previously, nsthorat (Nikhil Thorat) wrote…
this change seems a little weird to me - why are the chained ops now taking this as the first argument, and why not all of them?

this: T is not a real argument, it's a special signal to the TS compiler that if this is of type T, when this.pow() will return the same type. Previously we had no constraint that a.pow() will return the same rank as a. Now we do.

Comments from Reviewable

Review status: 46 of 58 files reviewed at latest revision, 4 unresolved discussions, all commit checks successful.

src/engine.ts, line 102 at r3 (raw file):
Previously, dsmilkov (Daniel Smilkov) wrote…
Great observation. So I was thinking about this a little.
There will be a skew between kernels and user-ops and I think for now, before we have aggregation done, it's better to report user-ops since that is what the user controls (we report the inner-most named scope, with ability to aggregate).
Couple reasons for this argument:

When we backpropagate, if we see that a tapeNode is missing gradFunc, (meaning we haven't implemented gradient yet), it's more useful to report an error that this opName is missing gradients, so we can't backpropagate, as opposed to this kernelName is missing grad, since from the user perspective, the culprit is dl.opName(), not dl.kernelName().
When reporting timing in debug mode, kernelNames might be confusing, e.g. imagine we reuse a conv2d kernel for higher-order convs, reporting the logical conv is more helpful than reporting conv2d (from the user perspective).

Once we have aggregation done, we can come back and decide maybe to report both kernel and op, at which point, we can add a required name argument to engine.executeKernel().
WDYT?

That sounds reasonable to me. Maybe let's name this field to that effect: s/kernelName/scopeName or something.

Comments from Reviewable

Review status: 46 of 58 files reviewed at latest revision, 4 unresolved discussions, all commit checks successful.

src/engine.ts, line 102 at r3 (raw file):
Previously, nsthorat (Nikhil Thorat) wrote…
That sounds reasonable to me. Maybe let's name this field to that effect: s/kernelName/scopeName or something.

Done.

Comments from Reviewable


Add pair support, fix CompatibleObject conversions (fixes #600)

I've put the tests in some weird places, @nlohmann do you have any better spot to put them?

Coverage decreased (-0.03%) to 99.688% when pulling 550045d41fdbc572aa4a1771b6d49f1cbe6b2663 on theodelrieu:develop into 1a9d76679a3f41a038a89742fba29d81ee24ce82 on nlohmann:develop.

About the test cases - I also have no better idea.

Could you please add a test to cover line 1119 (see https://coveralls.io/builds/11848200/source?filename=src%2Fjson.hpp#L1119)?

Coverage increased (+0.002%) to 99.722% when pulling cea39dfaa892115adb05985221cfa2090f92bab9 on theodelrieu:develop into 92ef19696a18187112512b8361b822d817d01e95 on nlohmann:develop.

Thanks! I shall close the PR once Travis finishes.


Not able to complete backup on Redmi

The app is not able to make complete backup of sms on redmi phones, it just backup person to person sms and not other sms. Also, it doesn't auto backup in redmi devices, you have to manually update every time

You can check out the following Step:
Security->Permissions->Other Permission->SMS Backup+->Service SMS(check) (read orther sms)
Security->Battery->App battery saver->SMS Backup+->No restrictions (run automatically)
After the step,press the Backup button may not backup all the sms,Press the Reset button on the upper right corner menu then you can re-backup all the sms

Closing as duplicate (#572)


Access Variable file module

Hi. I am using aws glue etl job as a compute environment to run my long running test cases. For this purpose i have to create an .egg file of my robot framework keywords and attach this egg file with aws ETL job. so my veriables file script is in a module. So my question is

is it possible to add my variable file as we add module
current working solution:
Variables     path/folder/verifiable.py    ${input}
Expectation:
Variables     Module_name.veriable      ${input}

Thanks

This isn't possible at the moment. Variable files, like resource files, are always taken into use by using a path to the variable/resource file. Python (and Java) variable files could also be imported as modules same way as libraries, and I agree that would be a good idea. It would make it possible to include variables in normal Python packages, possibly even bundling them to same packages with libraries.
If the given path to resource/variable file doesn't match any file, Robot will automatically check is such a file found from directories in PYTHONPATH. This is not the same as importing discussed above, because the value still needs to be a path or file name like my_variables.py, not only the module name like my_variables. We are also kind of misusing PYTHONPATH, and should probably introduce our own environment variable like ROBOTPATH and search imported files from there instead.
Although I agree this would be a good idea, I cannot give promises when, if ever, this will be implemented. I'd like to have this in, but I doubt I have time in the near future. If someone else is interested to look at implementing this, I'd be happy to help. Shouldn't be overly complicated.

I am ready to work on this issue.

Cool! The first step would probably be changing Importer('variable file').import_class_or_module_by_path to use import_class_or_module instead. After that needs to see what acceptance tests possibly break (see atest/README.rst for details running then) and new tests need to be added.

How does the implementation look like @OzairKhanBro? Anything I could already review?


How to load image from a remote url

Hi,
As I understand, it takes images from raw folder. I want to load images from a url. Has anyone tried it ?
Thanks

You mean this ?
        val plHotspot1 = ActionPLHotspot(
            this,
            hotSpotId,
            PLImage(BitmapFactory.decodeResource(resources, R.raw.hotspot)),
            0f,
            0f,
            PLConstants.kDefaultHotspotSize,
            PLConstants.kDefaultHotspotSize
        )

It needs a bitmap and here you find it

Not just the hotspot, but the panorama itself. I found in old issues there was some comment of loading it from cache.

Then show me the comment.
Btw, when you show me what you want to do exactly, I need no guessing, what you want to do.

I mentioned clearly in the topic, loading images from remote url, I didn't
left you to the guess anything. One of the closed issue has the similar
query, and their they mentioned to work with cache.
On Wed, 31 Jan, 2024, 19:48 Hannes Achleitner, @.***>
wrote:

Then show me the comment.
Btw, when you show me what you want to do exactly, I need no guessing,
what you want to do.
—
Reply to this email directly, view it on GitHub
https://github.com/hannesa2/panoramaGL/issues/280#issuecomment-1919195492,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A43TBXHIKOISP47VKSL7VMTYRJHCZAVCNFSM6AAAAABCSLRHLWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMJZGE4TKNBZGI
.
You are receiving this because you authored the thread.Message ID:
@.***>

I don't search the code for you. If you needs help, you have to point me to the right place

Nevermind, got it working what I was trying to do. I understand that you are not creator of this library so not well versed with its nuances.


Warp the GLTF model to project into spherical shape (1/2 dome) - projection mapping

There are a number of techniques for this.  Projection mapping is a technique that can be used.  There are a number of ways to do this.  You could also use a sphere with a metallic surface and use a reflection from that , placing the gltf object beyond the field of view.

We will probably delay this one as we can use the prototype without warping the projection.


convert multibandsplit and eq3 to typescript

This converts multibandsplit and the EQ3 to typescript. Hopefully this time you won't need to do so many fixes. I'm a little unsure of the low, mid and high parameters on the EQ3. Not sure if they should be a Param or Signal.

I've pushed a few more commits around linting. Let me know if you would prefer that in a separate PR.

thanks for the PR very much appreciated! Could you submit the linting changes in a separate PR?
Also no worries about me needing to clean things up, it still saves me a bunch of time to start from your contribution!

@tambien sure, I've removed the linting from this PR and created #505 . Let me know if you need me to fix anything else.

looks great thanks!

 If you making lots of changes on the PR's. You can add a co-author to the commit so we are both listed as authors in the commit: https://help.github.com/en/articles/creating-a-commit-with-multiple-authors

oh no! i would never want to remove you or any contributor from the commit history! those commits in your screenshot are just merges though, created from incorporating your changes into the codebase which included some changes already. your PRs are just a little earlier in the history

and your name will also appear on the files you contributed to

i want to fully credit contributors as much as possible. i've always just used the default mechanisms github provides for merging and attributing contributors. i looked quickly to see if other OSS authors add a "co-author" to merges, i couldn't find an example of it looking quickly here and here.
Do other repos use the co-author feature for merging PRs? i'm happy to start if that's the best way to highlight the work of contributors

Oh man, I'm so sorry. I didn't look far enough down. Really sorry. Keep doing what you doing. You can look to have less commits if you want. It depends on how strict you are about git history. An example is on the pouchdb repo https://github.com/pouchdb/pouchdb/commits/master Where each commit is linked to a PR or issue so we can track all history. But if you happy how it is then keep going.
Again really sorry for complaining unnecessarily.  I'm on vacation for the next few weeks. I'll look to help more with the typescript once I'm back.

that's no problem at all! glad we cleared it up!
i will take a look at pouchdb for reference/inspiration on how i can keep a better git history with clearer attribution. thanks for the reference!


Favicon 120x120

Updated the favicon for the site with the logo of the repository with dimensions 120x120

@akshitchowdhury
Please, actualise your fork-branch and check the diff of this PR ;)

@akshitchowdhury
I am closing this PR as inactive and having redundant edits.


Issue with Replacing Existing Subtitle in ExoPlayer

Hi,
I'm encountering an issue when trying to replace an existing subtitle in an ExoPlayer-based Android project. When attempting to replace an existing subtitle, the player continues to use the old subtitle or does not reflect the updated configuration. However, adding a subtitle to a video that does not already have a subtitle works fine.
Steps to Reproduce:

Attempt to Replace the Subtitle:

Clear the media items, create a new SubtitleConfiguration, and set it as the new subtitle for the media item.
Example:MediaItem.SubtitleConfiguration newSubtitle = new MediaItem.SubtitleConfiguration.Builder(newSubtitleUri)
       .setMimeType(MimeTypes.APPLICATION_SUBRIP)
       .setSelectionFlags(C.SELECTION_FLAG_DEFAULT)
       .build();

MediaItem newMediaItem = new MediaItem.Builder()
       .setUri(videoUri)
       .setSubtitleConfigurations(ImmutableList.of(newSubtitle))
       .build();

player.clearMediaItems(); // Clear any existing media items
player.setMediaItem(newMediaItem); // Set the new media item with the updated subtitle
player.prepare();

Play the Video:

Resume playback with the expectation that the new subtitle will be displayed.

Expected Behavior:

The player should display the new subtitle, effectively replacing the existing subtitle.

Actual Behavior:

When replacing an existing subtitle, the player continues to display the initial subtitle or does not reflect the new subtitle configuration as expected.
Adding a subtitle to a video that does not already have a subtitle works fine.

Additional Context:

The issue occurs consistently when trying to replace an existing subtitle with a new one.
The problem does not occur when adding subtitles to a video that does not already have subtitles.
Clearing the media items and setting a new media item does not appear to update the subtitle as intended.

Thank you for your response

What do you mean by "resume playback"? Are you clearing the media items during playback? Are you keeping the same videoUrl, hoping for a seamless continuation?

I meant: when I select another subtitle file from storage and try to add it to the video it doesn't reflect it's the old one that's reflecting ,
As you can see below I created a new MediaItem with the new subtitle file but still it's the subtitle that comes with the video shows
MediaItem.SubtitleConfiguration newSubtitle = new MediaItem.SubtitleConfiguration.Builder(newSubtitleUri)
.setMimeType(MimeTypes.APPLICATION_SUBRIP)
.setSelectionFlags(C.SELECTION_FLAG_DEFAULT)
.build();
MediaItem newMediaItem = new MediaItem.Builder()
.setUri(videoUri)
.setSubtitleConfigurations(ImmutableList.of(newSubtitle))
.build();
player.clearMediaItems(); // Clear any existing media items
player.setMediaItem(newMediaItem); // Set the new media item with the updated subtitle
player.prepare();

I'm not able to reproduce this at the tip of the main branch. What version of the library are you using? I modified the demo app to add the following code to PlayerActivity.initializePlayer just after the call to player.prepare():
MediaItem mediaItemWithDifferentSubtitles =
    mediaItems
        .get(0)
        .buildUpon()
        .setSubtitleConfigurations(
            ImmutableList.of(
                new MediaItem.SubtitleConfiguration.Builder(
                        Uri.parse(
                            "https://storage.googleapis.com/exoplayer-test-media-1/ttml/japanese-ttml.xml"))
                    .setMimeType(MimeTypes.APPLICATION_TTML)
                    .setSelectionFlags(C.SELECTION_FLAG_DEFAULT)
                    .build()))
        .build();
player
    .createMessage(
        (messageType, message) -> player.setMediaItem(mediaItemWithDifferentSubtitles))
    .setLooper(player.getApplicationLooper())
    .setPosition(5000)
    .send();

I then played the Subtitles > TTML Positioning sample and 5s in (as expected) playback restarted and showed Japanese subtitles instead.
I tried modifying my code to include the additional clearMediaItems() and prepare() calls from your example, but I still was unable to reproduce your problem:
player
    .createMessage(
        (messageType, message) -> {
          player.clearMediaItems();
          player.setMediaItem(mediaItemWithDifferentSubtitles);
          player.prepare();
        })
    .setLooper(player.getApplicationLooper())
    .setPosition(5000)
    .send();

I also tried with legacy subtitle decoding (defaultMediaSourceFactory.experimentalParseSubtitlesDuringExtraction(false)), but was still unable to repro.
One thing that's not clear in your repro steps: Is your original media item configured with sideloaded subtitles via MediaItem.Builder.setSubtitleConfigurations or are the subtitles part of the stream (e.g. muxed into an MP4 file, or configured as part of a DASH/HLS playlist)?
If it's the former case (which is what my repro steps recreate), please can you try my repro steps in the demo app and see if you still see an issue?
If it's the latter case, please can you send the media file to<EMAIL_ADDRESS>with subject Issue #1696 so we can take a look, and update this issue when you've done so.


V6

Please more the docs more clear for table expansion

This PR needs to be redone.

Do not include built files
Only include changed files


Compatibility issues with Safari 14

Detailed description
We are noticing when clicking on some components will crash the browser, Safari version 14 in particular. We are not sure if that is caused by the component or the browser, but the problem can be reperduce in the storybook.
Steps to reproduce the issue

Go to https://web-components.carbondesignsystem.com/?path=/story/components-content-switcher--default-story with Safari version 14
The default selected switcher isn't highlighted
Click on any options, the browser crashes and reloads the page.

Additional information

The same problem can be found in checkbox component
https://web-components.carbondesignsystem.com/?path=/story/components-checkbox--default-story

Thanks @ericwwsun for reporting! Fixed browser crash. Wrt selection, no selection at the initial state is consistent with Chrome and is intentional. You can go to Knobs tab and set something like staging to The value of the selected item (value) knob to see the selection.


Issue while creating an react app

the showing error
Creating a new React app in C:\Users\ADMIN\Desktop\sucees\my-app.
Installing packages. This might take a couple of minutes.
Installing react, react-dom, and react-scripts with cra-template...
changed 1491 packages in 2m
253 packages are looking for funding
run npm fund for details
Missing dependencies in package.json

same error

Thanks for your support, I hope this will definitely solve my problem.
On Fri, Mar 8, 2024 at 9:14 PM Rohit Badekar @.***>
wrote:

It seems like you're encountering an issue with missing dependencies after
creating a new React app. Here's a step-by-step solution to address this
problem:
Check Package.json: First, navigate to your project directory (my-app in
your case) and open the package.json file in a text editor.
Verify Dependencies: Inside package.json, there should be a section called
"dependencies" listing packages like "react", "react-dom", and
"react-scripts". Ensure that these dependencies are properly listed.
Install Dependencies: If any of these dependencies are missing, you can
manually install them using npm or yarn. Open your terminal or command
prompt, navigate to your project directory, and run:
npm install react react-dom react-scripts
or if you prefer using yarn:
yarn add react react-dom react-scripts
This command will install the necessary dependencies for your React app.
Restart Development Server: After installing the dependencies, restart
your development server by running:
npm start
or
yarn start
This will ensure that your React app runs with the newly installed
dependencies.
Verify Installation: Once the development server is running without
errors, open your web browser and navigate to http://localhost:3000 (or
whichever port your app is running on) to see if your React app is working
properly.
By following these steps, you should be able to resolve the missing
dependencies issue and continue developing your React application. If you
encounter any further errors, feel free to ask for more assistance!
—
Reply to this email directly, view it on GitHub
https://github.com/facebook/create-react-app/issues/13528#issuecomment-1985922654,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWMMRSXJ7DKLAIBA7BVTDOLYXHME7AVCNFSM6AAAAABD3PTAF2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOBVHEZDENRVGQ
.
You are receiving this because you commented.Message ID:
@.***>

Nevertheless, it is displayed as in the screenshot below and only the
package.json is displayed in the folder, Please solve my Problem
On Sat, Mar 9, 2024 at 6:58 AM Sejal Kathane @.***> wrote:

Thanks for your support, I hope this will definitely solve my problem.
On Fri, Mar 8, 2024 at 9:14 PM Rohit Badekar @.***>
wrote:

It seems like you're encountering an issue with missing dependencies
after creating a new React app. Here's a step-by-step solution to address
this problem:
Check Package.json: First, navigate to your project directory (my-app in
your case) and open the package.json file in a text editor.
Verify Dependencies: Inside package.json, there should be a section
called "dependencies" listing packages like "react", "react-dom", and
"react-scripts". Ensure that these dependencies are properly listed.
Install Dependencies: If any of these dependencies are missing, you can
manually install them using npm or yarn. Open your terminal or command
prompt, navigate to your project directory, and run:
npm install react react-dom react-scripts
or if you prefer using yarn:
yarn add react react-dom react-scripts
This command will install the necessary dependencies for your React app.
Restart Development Server: After installing the dependencies, restart
your development server by running:
npm start
or
yarn start
This will ensure that your React app runs with the newly installed
dependencies.
Verify Installation: Once the development server is running without
errors, open your web browser and navigate to http://localhost:3000 (or
whichever port your app is running on) to see if your React app is working
properly.
By following these steps, you should be able to resolve the missing
dependencies issue and continue developing your React application. If you
encounter any further errors, feel free to ask for more assistance!
—
Reply to this email directly, view it on GitHub
https://github.com/facebook/create-react-app/issues/13528#issuecomment-1985922654,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWMMRSXJ7DKLAIBA7BVTDOLYXHME7AVCNFSM6AAAAABD3PTAF2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOBVHEZDENRVGQ
.
You are receiving this because you commented.Message ID:
@.***>

Nevertheless, it is displayed as in the screenshot below and only the package.json is displayed in the folder, Please solve my Problem
…
On Sat, Mar 9, 2024 at 6:58 AM Sejal Kathane @.> wrote: Thanks for your support, I hope this will definitely solve my problem. On Fri, Mar 8, 2024 at 9:14 PM Rohit Badekar @.> wrote: > It seems like you're encountering an issue with missing dependencies > after creating a new React app. Here's a step-by-step solution to address > this problem: > > Check Package.json: First, navigate to your project directory (my-app in > your case) and open the package.json file in a text editor. > > Verify Dependencies: Inside package.json, there should be a section > called "dependencies" listing packages like "react", "react-dom", and > "react-scripts". Ensure that these dependencies are properly listed. > > Install Dependencies: If any of these dependencies are missing, you can > manually install them using npm or yarn. Open your terminal or command > prompt, navigate to your project directory, and run: > > npm install react react-dom react-scripts > > or if you prefer using yarn: > yarn add react react-dom react-scripts > > This command will install the necessary dependencies for your React app. > > Restart Development Server: After installing the dependencies, restart > your development server by running: > > npm start > > or > > yarn start > > This will ensure that your React app runs with the newly installed > dependencies. > > Verify Installation: Once the development server is running without > errors, open your web browser and navigate to http://localhost:3000 (or > whichever port your app is running on) to see if your React app is working > properly. > > By following these steps, you should be able to resolve the missing > dependencies issue and continue developing your React application. If you > encounter any further errors, feel free to ask for more assistance! > > — > Reply to this email directly, view it on GitHub > <#13528 (comment)>, > or unsubscribe > https://github.com/notifications/unsubscribe-auth/AWMMRSXJ7DKLAIBA7BVTDOLYXHME7AVCNFSM6AAAAABD3PTAF2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOBVHEZDENRVGQ > . > You are receiving this because you commented.Message ID: > @.***> >

Did you try with pnpm?


add check for extension being md to verify the last active file was a…

Fixes #837
Changes Made:

Checked to make sure that the extension of the file is one of a markdown file

This still needs to be tested in Obsidian to verify it works.

This has been tested. All that is left is just to confirm that this works.


grpc-thrift compat

For Code review

Link to thrift codegen https://github.com/chedeti/thrift/pull/1

I really don't see the point of 57cc8f8 . Why isn't this a submodule? I don't think we should carry it around like this - better to make it a submodule pinned at a specific commit and then we can only take it when we want to.  Let it be a submodule of the actual thrift repo pinned at a specific commit and then just check in your changes. Bringing their entire repo into ours is going to make maintainability impossible.

Dude, seriously!  Who do you expect to properly review 2,104 files?!?!?  Yes, I know it's funny to create this set of commits, but still...
This PR definitely needs to be broken up into smaller, more manageable chunks.  Even GitHub is complaining:

I am sorry, give me sometime. I am working on it.
On Sun, Jul 31, 2016 at 3:12 PM, Paul Grosu<EMAIL_ADDRESS>wrote:

Dude, seriously! Who do you expect to properly review 2,104 files?!?!?
Yes, I know it's funny to create this set of commits, but still...
This PR definitely needs to be broken up into smaller, more manageable
chunks. Even GitHub is complaining:
[image: github-too-many-files]
https://cloud.githubusercontent.com/assets/6555937/17279694/362c57f4-5749-11e6-979e-41a4063f1b7e.png
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/grpc/grpc/pull/7523#issuecomment-236459903, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ASfuIhvWNxxIirDsHqzkYua5WssWQ-kAks5qbR24gaJpZM4JUsim
.

@chedeti No worries, it's understandable :)  Thank you for helping us out.


[Removal]:

Hotspot Name
Long Scarlet Buffalo
Hotspot b58 Address
112Rn1KgPuvwArVpfnS1uFqewVX2iNfJRNTn3a8tvRH9pm3LmGqL
Discord Handle
ken340#1015
Hotspot Manufacturer
Nebra
Removal Reason
I bought my Nebra 18 days ago. I am a legit user not fakers or spoofer.
Modifications
Yes
Extra forwarders
No
Extra antennas
No
Additional Information
I replace the SD CARD as my eMMC is almost full and using RAK 5.8 dbi. I had to change the antenna location a few times as there is no guidance on whether I should put based on what altitude reference (e.g. above ground, above mean sea level, GPS altitude, etc).
Also, I am worried about what data are being sent out. on average the data usage is extremely high but no activities are being registered. Can anyone explain why so much data was uploaded?

How can I be gaming or faking with such low earnings?

Thank you for your submission. This has been reviewed and may be considered for a future change to the denylist. If any change will be made, it'll likely be in the next 1-2 weeks assuming (1) it is proposed to be removed, (2) denylist signers agree on the change, and (3) the release is published to all Hotspots. Please do not open additional issues regarding this Hotspot within the next 3 months.

Dear Abhay,
Thank you for your prompt reply. I would like to advise you that due to the low earnings and I will be moving my miner to another location. Would it affect my removal from the deny list?
I will assert to location when I do that. If you check in have transferred 1 HNT into my wallet for the transfer.


Is it possible to use this plugin on bucket Aggregations?

This plugin returns value for single value if want to calculate for whole field then what we have to do
.

Was interested in this question as well - I've been trying to figure out how to do reports/charts for weighted averages it seems it's straightforward with bucket_script but can't find anything that let's kibana work the arbitrary elasticsearch queries e.g. that use bucket_script.
Any answer?

Is there a way to use this in Kibana 6?


Add PTMA AA FF to polyply library [arXiv:2209.02072]

PR for contributing the OPLS-based AA FF developed for PTMA in https://arxiv.org/abs/2209.02072.
I will shortly add a test. Also open a separate PR to contribute CG model(s).

@ricalessandri if you register here you can get a free T-shirt if you make 4 PRs in Oktober for the Hacktoberfest.

Sure thing we just have to know if it gives an atactic or isotactic polymer. Either way is fine for me.

Finally getting to close this, some things to fix:

[ ] update atom type names (used standard OPLS atom type names)
[ ] update citation as soon as I get the DOI
[ ] do the plot @fgrunewald asked for :)

Alright, I'm finally at it. This is one of the 3 things left to close this PR: tacticity.
@fgrunewald  we're getting atactic PTMAs, see:

Average enantiomer fractions over 100 chains of 30-mers:
enantiomer #1 : 0.39399930000000005
enantiomer #2 : 0.6059994800000001

With a 40-60% ratio. I guess this would converge to 50-50 with more (and/or longer) chains.
Also, chains are not syndiotactic. This I have printing the dihedrals and check them within and across chains. See for yourself for the first 10 chains:
  0 36.5249         |  0 36.0532    |  0 34.7708    |  0 -36.5657   |  0 35.2323    |  0 36.9282    |  0 36.1829    |  0 36.1268    |  0 -36.0353
  1 34.0585         |  1 32.8891    |  1 35.7504    |  1 34.8827    |  1 38.3678    |  1 -41.1577   |  1 40.1183    |  1 -35.7562   |  1 37.7092
  2 -37.2684        |  2 34.5314    |  2 -36.087    |  2 36.5295    |  2 38.1303    |  2 32.8347    |  2 37.3767    |  2 -35.1527   |  2 -39.0089
  3 36.8553         |  3 34.2684    |  3 -36.7091   |  3 36.5085    |  3 -36.8466   |  3 -40.7319   |  3 35.2229    |  3 40.2213    |  3 40.0856
  4 -33.6193        |  4 -32.8512   |  4 -42.2365   |  4 -37.6941   |  4 -34.7345   |  4 -37.4294   |  4 -36.0298   |  4 -36.4733   |  4 35.8881
  5 37.2838         |  5 33.4698    |  5 -32.5688   |  5 37.3802    |  5 35.678     |  5 36.2165    |  5 37.1842    |  5 37.5854    |  5 -32.3946
  6 35.3496         |  6 -36.0319   |  6 38.4035    |  6 39.7357    |  6 -41.5397   |  6 38.9863    |  6 40.6138    |  6 36.6165    |  6 37.2975
  7 32.874          |  7 -36.0374   |  7 36.9036    |  7 36.3968    |  7 38.5483    |  7 -39.3083   |  7 -35.7891   |  7 37.9968    |  7 34.7671
  8 43.9699         |  8 35.4649    |  8 -37.1433   |  8 -35.7173   |  8 37.0971    |  8 36.0991    |  8 -33.8231   |  8 31.2062    |  8 36.5176
  9 35.8624         |  9 37.4412    |  9 38.3855    |  9 -39.6864   |  9 -41.669    |  9 -38.5243   |  9 38.1871    |  9 40.1058    |  9 -37.2323
  10 37.6172        |  10 39.4304   |  10 34.4706   |  10 -39.4187  |  10 35.063    |  10 39.1183   |  10 33.5543   |  10 -34.6115  |  10 38.5975
  11 41.7453        |  11 33.5612   |  11 35.5474   |  11 34.1137   |  11 -36.8871  |  11 35.875    |  11 -38.4814  |  11 37.2663   |  11 41.8513
  12 39.0329        |  12 -34.4066  |  12 36.363    |  12 31.7394   |  12 36.6417   |  12 -35.4788  |  12 -33.0636  |  12 -35.5622  |  12 -36.0089
  13 35.9318        |  13 39.8196   |  13 -39.2512  |  13 35.2016   |  13 -33.0821  |  13 -36.4384  |  13 39.1468   |  13 37.0456   |  13 36.0463
  14 -34.3356       |  14 -36.6234  |  14 37.2354   |  14 39.4331   |  14 41.5223   |  14 37.71     |  14 -38.5657  |  14 37.4491   |  14 37.891
  15 -34.7859       |  15 -33.715   |  15 -36.5078  |  15 -38.4706  |  15 31.7546   |  15 37.0417   |  15 36.2768   |  15 -33.8178  |  15 -37.1269
  16 33.7626        |  16 -36.9238  |  16 33.5581   |  16 40.7569   |  16 36.9485   |  16 34.397    |  16 35.0809   |  16 36.8588   |  16 37.7889
  17 39.1062        |  17 38.5846   |  17 33.7556   |  17 38.6473   |  17 37.9576   |  17 36.227    |  17 33.3178   |  17 38.425    |  17 36.5993
  18 -36.0831       |  18 33.9731   |  18 -39.3478  |  18 -36.6049  |  18 -35.2307  |  18 -35.9973  |  18 33.6778   |  18 38.6039   |  18 34.586
  19 -32.5932       |  19 -34.7784  |  19 -34.793   |  19 -34.527   |  19 38.7232   |  19 -33.607   |  19 33.8859   |  19 35.1522   |  19 -35.5263
  20 -39.7191       |  20 38.3118   |  20 35.6131   |  20 36.365    |  20 37.1533   |  20 39.9286   |  20 37.0797   |  20 -36.2315  |  20 36.6258
  21 39.7267        |  21 39.9699   |  21 38.0917   |  21 37.1777   |  21 -42.0094  |  21 36.3877   |  21 -36.6893  |  21 -36.845   |  21 -34.415
  22 34.2142        |  22 36.3733   |  22 35.9077   |  22 -35.3277  |  22 36.8763   |  22 36.5706   |  22 36.445    |  22 -33.9113  |  22 40.454
  23 -39.275        |  23 37.1109   |  23 -37.8667  |  23 38.5636   |  23 -35.4948  |  23 -39.1039  |  23 36.604    |  23 34.715    |  23 -35.9212
  24 36.3931        |  24 37.9223   |  24 37.0983   |  24 37.749    |  24 -33.4265  |  24 35.8632   |  24 -37.5184  |  24 36.9496   |  24 -38.328
  25 -34.1988       |  25 -39.3313  |  25 38.5158   |  25 37.0989   |  25 -37.5111  |  25 37.9106   |  25 36.1155   |  25 -38.0322  |  25 38.8457
  26 34.4993        |  26 29.5963   |  26 -35.9228  |  26 37.5261   |  26 36.3587   |  26 37.3568   |  26 -37.3268  |  26 -35.1047  |  26 39.0637
  27 33.1792        |  27 35.752    |  27 -37.0754  |  27 -38.6848  |  27 37.4816   |  27 -35.7833  |  27 38.576    |  27 -34.3623  |  27 37.3753
  28 -34.6558       |  28 42.5101   |  28 -40.0315  |  28 37.6345   |  28 -38.7036  |  28 35.2331   |  28 -39.6963  |  28 37.8905   |  28 32.9077
  29 31.8568        |  29 31.8861   |  29 40.188    |  29 -37.5955  |  29 33.4013   |  29 -38.7681  |  29 -36.739   |  29 37.571    |  29 38.1437

Not sure if you want this more quantitative. In case, what's a smart metric for this?

@fgrunewald I resolved the previous issues. Let me add a test.

One test fails but I think it's not related to this .ff file.

@ricalessandri you can also consider adding a line on the news section of the readme file to promote your paper

Thanks for the nice ideas!
I've added the warning, it works:
https://github.com/marrink-lab/polyply_1.0/blob/006e37ed44d2f8730e806cd66d5de8b71c4859f0/polyply/data/oplsaaLigParGen/links_PTMA.ff#L164-L174
I didn't include that [ non-edges ] sections of P3HT, which would have looked like this:
[ link ]
[ atoms ]
C01 {"resname": "PTMA"}
[ warning ]
You should patch PTMA with a CH3-terminal residues. Use -seq CH3a:1 PTMA:4 CH3b:1
[ non-edges ]
-C1 C01
-C17 C01

[ link ]
[ atoms ]
C17 {"resname": "PTMA"}
[ warning ]
You should patch PTMA with a CH3-terminal residues. Use -seq CH3a:1 PTMA:4 CH3b:1
[ non-edges ]
C17 +C1
C17 +C01

because it didn't make any difference in the resulting itp and I wasn't sure about their purpose. Let me know if it should be there.
As a side note, I played with the behavior and the warnings will be printed twice for each residue (i.e., 8 times for PTMA:4). While the twice is fine because it's one for CH3a and one for CH3b, printing it for each residue seems unnecessary. Also, if I use: PTMA:4 CH3b:1 or CH3a:1 PTMA:4 I was expecting polyply to print only one warning, but it still prints two. Let me know if there's something wrong with my definitions. Otherwise, these are anyway minor things because we want the user to use CH3a:1 PTMA:4 CH3b:1 anyway.
I will include the news piece in the other PR later today.

@ricalessandri the non-edges are specifically to reduce the number of warnings being printed and in principle, you could even use them to specifically target only one end, like if you miss CH3b it would only do 1 warning. However, there is a small syntax mistake in the non-edges of the first residue. It should first mention the atom that is also in the atoms directive. So it should be:
[ link ]
[ atoms ]
C01 {"resname": "PTMA"}
[ warning ]
You should patch PTMA with a CH3-terminal residues. Use -seq CH3a:1 PTMA:4 CH3b:1
[ non-edges ]
C01 -C1
C01 -C17

If you adjust this, the PR should be ready to go.

@fgrunewald done! I implemented and tested and works as expected (1 warning for CH3a if I miss that, 1 warning for CH3b if I miss that, of 2 warnings if I missed both) - fantastic!


tests: fix test_command_output

The with_setup decorator does not appear to be designed to be used
without parenthesis.  Remove it, since it is not needed/used anyway.
Fix the actual test.

btw: this makes the tests run on pytest.
What do you think about using pytest as main test runner?

What do you think about using pytest as main test runner

Definitely in favor, the work was done in https://github.com/neovim/python-client/pull/266 but it's blocked by a build failure that no one has investigated.


[Question] Should flutter root widget always be StatelessWidget?

When I read the doc in flutter, I have a question that should flutter root widget always be StatelessWidget?
 class MyApp extends StatelessWidget {
   @override
   Widget build(BuildContext context) {
     return MaterialApp(
       title: 'Flutter Code Sample for Navigator',
       // MaterialApp contains our top-level Navigator
       initialRoute: '/',
       routes: {
         '/': (BuildContext context) => HomePage(),
         '/signup': (BuildContext context) => SignUpPage(),
       },
     );
   }
 }

Because I think there's sometime need init function to call, and maybe not want the code of that write in HomePage.  For example: check token expire or not, and decide go to HomePage or LoginPage.
Then the best option: should I change the root Widget to StatefulWidget, and just include the logic above in its initState function ?

Hi @jerryzhoujw
I don't see any reason why not to use StatefulWidgets right after
void main() => runApp();
Please see https://flutter.dev/community for resources for asking questions like this,
you may also get some help if you post it on Stack Overflow.
Closing, as this isn't an issue with Flutter itself,
if you disagree please write in the comments and I will reopen it
Thank you

Thank you


Add re-exports for modules, make module organization more rusty

A lot of our modules are quite nested in an unhelpful way, e.g. shimmer::bvh::Bvh. Should we just re-export Bvh to get shimmer::Bvh?
Generally look for places that might benefit and do them

There's a good section here: https://doc.rust-lang.org/book/ch14-02-publishing-to-crates-io.html

I'm going to repurpose this to say, let's make this more Rust-y. That might mean combining structs into one file rather than each struct getting their own file, etc. Look for more idiomatic organization.

I'm closing since #54 duplicates


Android gradle plugin 1.5.0  not compatiable

Please, provide update

looks like it's been updated to 2.0.0-alpha2

yes and we are at spoon runner 1.3.1

Should work with version 1.0.4 (will be in Maven Central in few hours). Please reopen if it does not work for you.


Why don't you show the using statements?

You have code displayed without any of the imports/using statements.  This helps me how?
[Enter feedback here]

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

ID: 97d622b2-1e4b-4d76-b3df-fefaa5ba99cb
Version Independent ID: ec833af2-275e-981f-689f-8c65edfc00cd
Content: Generated Values - EF Core
Content Source: entity-framework/core/modeling/generated-properties.md
Product: entity-framework
Technology: entity-framework-core
GitHub Login: @AndriySvyryd
Microsoft Alias: avickers

Duplicate of #729


[Bug]: Expired permissions can be selected

ODC version
odc431
OB version

What happened?

What did you expect to happen?

How can we reproduce it (as minimally and precisely as possible)?
Enter the project, click on members, click on manage permissions, and then click on expired permissions
Anything else we need to know?
No response

Fixed In Commit d7addf8113ff4e6e6fd30003509e26d4d8ebac08
------
畅晚
Wed Jul 10 13:58:11 2024 +0800

pass


WIP: adding docs for Texture2DArray, Cubemap, CubemapArray, and overhaul TextureLayered

will add docs for Texture2DArray and Texture3D as requested in issue #30298

[x] TextureLayered
[x] Texture2DArray
[x] Cubemap
[x] CubemapArray

applying feedback to the class documentations above

[x] TextureLayered
[x] Texture2DArray
[x] Cubemap
[x] CubemapArray

TextureLayered
overhaul

brief_description
description

adding descriptions for

_get_images()
create_from_images()
get_layers()
update_layer()

Texture2DArray
adding

brief_description
description

Cubemap
adding

brief_description
description

CubemapArray
adding

brief_description
description

#30298 is related to using layered textures in shaders, Shader specific documentation should be added to http://docs.godotengine.org/en/latest/tutorials/shading/shading_reference/shading_language.html

#30298 is related to using layered textures in shaders, Shader specific documentation should be added to http://docs.godotengine.org/en/latest/tutorials/shading/shading_reference/shading_language.html
But this is great so far. I will do a review tonight :)

Hm.. I believe @akien-mga meant the 3 classes I want to document...
I reckon we are both right: add docs for the 3 classes + adding sampler2dArray (+ its two functions) to the shading language data types
What's your take on this, @TheDuriel?

I replaced some enum and method tags with code, because of several errors:
ERROR: Unresolved enum 'TextureLayeredType.TEXTURE_LAYERED_2D_ARRAY', file: Texture2DArray

ERROR: Unresolved enum 'DataType.TYPE_SAMPLER2DARRAY', file: Texture2DArray

ERROR: Unresolved type reference 'ResourceFormatLoaderTextureLayered' in method reference 'ResourceFormatLoaderTextureLayered.load', file: TextureLayered

ERROR: Unresolved type 'basisu_backend.compute_slice_crcs', file: TextureLayered

ERROR: Unresolved method 'RenderingDevice.texture_create_shared_from_slice', file: TextureLayered

ERROR: Unresolved enum 'RenderingDevice.TextureSliceType', file: TextureLayered

I guess I won't document Texture3D, as there is no class for that. That one really is purely for shaders.
I might do that at a later time

Where is VS::TEXTURE_TYPE_CUBEMAP defined?
I reckon it stands for TextureType.TEXTURE_TYPE_CUBE as well as TextureType.TEXTURE_TYPE_CUBE_ARRAY??

From my perspective, I'm done

Great job! I love the attention to detail and the deep-fullsome explanations.
Most of the notes I left are about removing references to how the engine uses the resources internally.
Keep in mind that these resources map onto Vulkan resources. What the engine uses is the Vulkan resources and not these ones directly. These are provided for the convenience of users so that they can create the underlying resource (e.g. CubemapArray) from images without having to access the Vulkan API directly. The engine just access the Vulkan API directly.
While you aren't wrong about how Sky/ReflectionProbes utilize these resources, its information that will confuse users and it won't help them in using the resource.

Thanks! I know what it means having little to none documentation... Therefore, I don't want Godot's growth to suffer because of it
I'll make several commits to improve the texts and clean up at the end, so that it's easier to see what I changed

@MJacred I understand your concern. And I also don't like the thought of losing the valuable information you have put together. There is definitely a need for documenting more low-level engine stuff. I just don't think that the user-facing class reference is the right place for it.
An alternative would be to add a section to the docs that goes into depth like that "here is how it is used in the engine, here are some low-level considerations". But it would have to be visually distinct from the rest of the documentation. IMO even more so than added a bolded Engine Excursion warning.
@Akien what do you think about including engine details in the class reference? Would be good to get your opinion so we can think about doing something similar for other docs.

@clayjohn: This PR completely slipped my mind. Cubemap.xml is updated according to cut requests. I'll update the rest in the upcoming days.
The cut extra details are stored locally on my disk (and already backed up externally). I'm currently preparing to open a "state-of-godot" repo containing such info and more. If sth. useful comes from it, then it will find its way back

@clayjohn: Ok, I think I got everything. With godot 4, I guess the basisu tool got replaced with the Godot Editor re-import presets doing the slicing, mipmaps, etc.
What I'm not sure about is in CubemapArray.xml:

In Godot, the [CubemapArray] can be used for e.g. the [Sky], if you set [b]ProjectSettings > Rendering > Quality > Reflections > Texture Array Reflections[/b] to [code]true[/code]. Otherwise, the sky expects a [Cubemap].

I have left a review with some comments and suggestions. Once you update the doc, you will also need to rebase and squish your comment (described in the docs here

I applied all the feedback. Even the currently unresolved discussion. If my proposal is fine, I'll add it back in. If not, this PR can be merged.
branch is up-to-date with master

ping @clayjohn

And thank you for sticking with me! I hope you'll stick with me again in future PRs in your area :smile:

Thanks!


Feature request: Display CPU model

Frequently performance issues are cause by anemic hardware,
being able to display the CPU model along side some other metrics like cores available to the JVM, enabled mitigations and current clocks would go a long way helping diagnose this.

I did consider looking into this a while back, ended up not persuing it as I was somewhat offput by the idea of having to chunk something like oshi into paper (https://github.com/oshi/oshi) and co

For the short term, what about querying a few things like lscpu?

lscpu is a unix command. That doesn't work for windows.
There should be a platform independently solution for this.

I never said use lscpu only, I said use things like lscpu.
With powershell you can do Get-WmiObject –class Win32_processor | select * which outputs something similar.
Of particular note from lscpu:

NUMA nodes
Sockets
Model/Model name
Cache
CPU(s)
CPU MHz

This should cover most use cases and provide enough useful information into how resources are allocated to whatever (virtualised) environment MC is being ran in.  Querying Runtime.getRuntime() should be considered more powerful here for what its worth.
Additionally, querying lspci and filtering for Network controller, SATA Controller and Non-Volatile memory controller should provide enough information about storage models.
What are your thoughts on this @electronicboy ?

Throwing in oshi would be a much better solution vs tryna deal with parsing that mess ourselves

You to you, just providing an alternative


Get list of management clusters for the current product

This fixes an issue where all the management clusters are listed while exploring a downstream cluster by getting a list of management clusters for the current product. This means that Cluster Management => Advanced => Mgmt Clusters should remain unchanged, as the product is still management, while Cluster Management => { downstreamCluster } Explore => More Resources => Rancher => Mgmt Clusters should only list the management clusters for the given product.
#4667

@richard-cox  the difference we're looking for isn't when you pick local vs a different cluster to explore, but the cluster management 'product' vs explorer in the top left nav.
It was previously using the management store even when you were looking at that type in a downstream clearer.

@vincent99 Sorry yes, good point. I should have read the description properly


Enabled Analytics tracking

We've currently got the placeholder in until we launch.

• Check the tracking ID
• Enable the analytics

we need to pull out the slugs of posts and attach them to CTA, Newsletter signup and Callouts directives

See commit: 3ad84dfbab077769
for references to analytics tracking which need changing

Done in build.


Missing dependency?

I tried using csv2json, here is what i get:
/usr/lib/ruby/1.9.1/rubygems/custom_require.rb:36:in `require': cannot load such file -- orderedhash     (LoadError)
    from /usr/lib/ruby/1.9.1/rubygems/custom_require.rb:36:in `require'
    from /var/lib/gems/1.9.1/gems/csv2json-0.3.0/lib/csv2json.rb:4:in `<top (required)>'
    from /usr/lib/ruby/1.9.1/rubygems/custom_require.rb:36:in `require'
    from /usr/lib/ruby/1.9.1/rubygems/custom_require.rb:36:in `require'
    from /var/lib/gems/1.9.1/gems/csv2json-0.3.0/bin/csv2json:6:in `<top (required)>'
    from /usr/local/bin/csv2json:23:in `load'
    from /usr/local/bin/csv2json:23:in `<main>'

seems like a missing dependency to me, but im no rubyist =/ any ideas?

I did a gem i orderedhash to get this working


guest inviter role still not supported in Teams?

Is it still correct that the guest inviter role is not supported in Teams?
Because if I disable "Members can invite" in the External collaboration settings in Azure AD and I give an non-admin user the guest inviter role he can add guests form Teams Client. If I remove the guest inviter role he can't. So it seems to work for me. Or am I understanding anything wrong?

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

ID: 93906e31-cd28-702d-bb7e-fcc2529df363
Version Independent ID: afe4d6ed-cdcb-77e0-0951-f3c69e2463e1
Content: Add a guest to a team - Microsoft Teams
Content Source: Teams/add-guests.md
Service: msteams
GitHub Login: @MikePlumleyMSFT
Microsoft Alias: mikeplum

@alexjeitler Thank you for submitting feedback.
I think the best way forward is if you open a service ticket in your tenant so this can get resolved ASAP. Based on the outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:https://docs.microsoft.com/office365/admin/contact-support-for-business-products
Please keep us posted here on the resolution so we can feed whatever information you discover into the content.

Hi Manuela,
I saw there was a change recently and now the mention of the limitation is gone.
Can you conform that my observation was right?
Thanks and br,
Alex
Von: Manuela Scanu<EMAIL_ADDRESS>Gesendet: Dienstag, 8. September 2020 06:15
An: MicrosoftDocs/OfficeDocs-SkypeForBusiness<EMAIL_ADDRESS>Cc: Alexander Jeitler<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Betreff: Re: [MicrosoftDocs/OfficeDocs-SkypeForBusiness] guest inviter role still not supported in Teams? (#5706)
@alexjeitlerhttps://github.com/alexjeitler Thank you for submitting feedback.
I think the best way forward is if you open a service ticket in your tenant so this can get resolved ASAP. Based on the outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:https://docs.microsoft.com/office365/admin/contact-support-for-business-products
Please keep us posted here on the resolution so we can feed whatever information you discover into the content.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/OfficeDocs-SkypeForBusiness/issues/5706#issuecomment-688607916, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AKGQSF7KAOHWR4GWNKOC5LLSEWVULANCNFSM4QXTV4DA.


Pass bazel java path via JAVACMD to scalac

Yeah, just got this failure on my own stuff. Tracking it down.

Squashed.

Awesome! thanks @smparkes !


Fixed for an upcoming version, I hear.


lunanod() in nodule classification

in line 132 of main_nodcls.py the function lunanod receives the following parameters:
testset = lunanod(tefnamelst, telabellst, tefeatlst, train=False, download=True, transform=transform_test)
put in dataloader.py it says they should be:
def init(self, npypath, fnamelst, labellst, featlst, train=True,
transform=None, target_transform=None,
download=False)
so npypath is not refereced. Should it be? what goes there?
Also, would you provide the trained model for dpn3d.py for nodule classification? do you have other already loaded checkpoints for noudule classification? If there are, where are them? In case of being the ones in the dpnmodel or resmodel folder, why are there so many? which one should I use for classification o any CT scan? (not necessarily from the luna data set)
Thank you very much!

Yes. You should do detection first, then classification. The classification is based on detection, detected box. Try to run detection and get a sense of how to process the lung CT images.


daemon: Allow releasing builder while waiting for proxy ACKs

Sidecar proxy ACKs can take a long time due to container
initialization. Allow other Endpoints to build while another Endpoint
is waiting for proxy completions.
Build queue is implemented with a semaphore to simplify the implementation. Queueing for the build new makes the calling go-routine wait when Acquiring the "build permission". A function for releasing the permission is returned if a permission is granted, nil otherwise. The permission is released after the endpoint bpf compilation and before waiting for the proxy completion so that endpoint builds are not blocked when other endpoints are waiting for proxy completions. This allows the overall build timeout to be increased so that endpoint builds need not fail because of slow container initialization.
Signed-off-by: Jarno Rajahalme<EMAIL_ADDRESS>

This change is

Coverage decreased (-0.04%) to 43.483% when pulling 930920ef20f1aeccea510d323ba57250748743da on pr/release-builder-while-waiting into e28cfc8d76f4d8fb65a17db1dcc1191c2a0cb3e3 on master.

test-me-please

Apparent CI flake in https://jenkins.cilium.io/job/Cilium-PR-Ginkgo-Tests-Validated/7986/, but the failed test can't be found from the artifacts.

test-me-please

This eliminated "not-ready" PODs in my local testing with the Istio GSG and allowed other endpoints to regen while Istio app pods were booting up.

@aanm Had to rebase to master, but the last commit contains all changes due to the comments so far.

test-me-please

Upped priority to high as it is possible Istio users get not-ready endpoints due to this.

Hit by #5867 (likely). testing again.

test-me-please

test-missed-k8s

@aanm You added 1.4 tags, do you think this should NOT be backported to 1.3? Istio pods initially get to "not-ready" state because of the too short timeout to accommodate for image pulls.

@jrajahalme adding 1.4 tags doesn't mean it should not be backported - the needs-backport/1.3 label indicates that it should be backported to 1.3.


Not a bug, but something "visually" beautiful to describe how a single point is expanded with gaussian splatting.

Check out this nice thing. I was curious to see how a single point is expanded in the gaussian splatting when the splatting is done. For example, in this case, I have colorized each unique point in the initial splat, and tracked respective single point, during training while indexing its duplication and splitting to its "initial parent point", to see how it is modified. Some very colorful things appear and it is a nice way to visualize how a single point is proceessed through the engine. :)
The image below shows a finished gaussian splatting model with two million points, from an initial 103000 points

@abrahamezzeddine , do you mean that each individual color here (e.g. blue) is either the original point or one of its spawned children (due to splitting/dup)?

@maturk
The colors of the duplicated and split Gaussian points (which you see here) are not determined by the finished Gaussian splatting process. Instead, they are based on the initial sparse point cloud.
Initially, 103,000 unique RGB colors were assigned to the points in the sparse point cloud. During the Gaussian splatting process, these points are duplicated and split. The indices of the original points are tracked and kept through these operations, so even if a point is split, they both receive the same index number as in the initial sparse point cloud. After training, the colors of the final Gaussian points are set based on the indices from the initial point cloud. This ensures that the colors of the final Gaussian points correspond to their originating points in the initial sparse point cloud.
I hope it made sense.

Hi @abrahamezzeddine ,  This is indeed really good work. Would you mind sharing the code for how you kept track of each gaussian? It would be really helpful for me to understand this more clearly. Only if it's okay for you.


GEN3: Many frequent connection timeouts

PR #138 brings some improvements that make the connection setup more reliable.
However, MODBUS timeouts repeatedly occur, which then lead to disconnections. Adapted timeouts and regular synchronization of the timing with the messages sent by the inverter may alleviate these problems

improved with #147


tsconfig.json expected in js projects

I was using ts-simple-ast with js and it took me a long time to figure out why I got the following error: throw this.getFileNotFoundErrorIfNecessary(err, filePath); as it cannot find the tsconfig.json. But I don't need it in my js project...
Creating an empty config with touch tsconfig.json solved the issue. When I find the time I will post a more exact desciption in the future, but I guess the main problem is clear.

@marikaner could you post some reproduction code? You shouldn't need to specify a tsconfig and it should only be loaded when specified.

@dsherret Sorry for the sloppyness, I found the error in our own code.
Just FYI: we basically had a code generator (written in ts) using ts-simple-ast, that was relying on a ./tsconfig.json in the project (which I missed).
When packing it and sharing it with other colleagues, they got the mentioned error.
The fix was simple:
const project = new Project({
    tsConfigFilePath: path.resolve(__dirname, './tsconfig.json')
});

Thank you for your support, though!


Added JUnit course

Added a beginner friendly JUnit 5 course. Kaushik Sir from Java Brains is well known to explain concepts in meaningful manner.

By submitting this pull request I confirm I've read and complied with the below requirements.
Failure to properly do so will just result in the pull request being closed and everyone's time wasted.

[x ] I have read the Contribution guidelines and made sure my pull request follows it, specifically the part about promotions and markdown format.
[x] Any resources I am adding in this PR have been used by me and I can personally vouch for their quality.
[x] This pull request has a descriptive title. For example, Add [Resource Name] or Add [Topic Name], not Added new entries or update.
[x] I have double checked that I have only updated the minimum files that are required by this PR.

Thanks @prashanttejura for the contribution.


Support LEFT and RIGHT gravity values.

Hello .
An approach which only provides the implementation of the component at the bottom looks kinda forced. It would be useful to apply some feature which would allow changing the position where the view is in (or on the side ...). That would open up a wide range of opportunities for new layouts.
What do you think about this?

Congratulations for this handy library.
That will be great LEFT and RIGHT support, cause I'm trying to implement bottom-top on portrait and right-left on landscape. For now, I'll import both libraries separately.

+1 for the left/right support.
Unless I've missed something, SlidingPaneLayout doesn't support the setOverlayed/umanoOverlay behvior like this library does.

Anyone happened to make this lib work form LEFT to RIGHT?

+1 for left/right

+1 for left/right

+1 for left/right


Visually differentiate the various states of the payment status (e.g.Paid, Outstanding, Split buttons, ...)

@ksennr

1662e1303cae6344c5027fd9edf221d082325f79


Unable to Build Realm 4.4.1

Goals
I am trying to build the latest version of realm due to the fact that the latest version available via Cocoa Pods was built using Swift 5.1.2 and the latest version of Xcode is compiling with 5.2.2.
Expected Results
I am expecting that running the command sh build.sh build complete successfully.
Actual Results
Instead the build is failing.  I see the following at the end of the run:

** BUILD FAILED **

The following build commands failed:
	CompileC /Users/rodney/src/github.com/realm/realm-cocoa/build/DerivedData/Realm/Build/Intermediates.noindex/Realm.build/Release-iphoneos-static/Realm\ iOS\ static.build/Objects-normal/armv7/work_queue.o /Users/rodney/src/github.com/realm/realm-cocoa/Realm/ObjectStore/src/sync/impl/work_queue.cpp normal armv7 c++ com.apple.compilers.llvm.clang.1_0.compiler

Steps to Reproduce

Clone the master repository (As of April 19, 2020 - 4:47 PM PST)
Run: git submodule update --init --recursive
Run: sh build.sh build

Code Sample

Version of Realm and Tooling

Realm framework version: ?
4.4.1
Realm Object Server version: ?
Xcode version: ?
11.4.1 (11E503a)
iOS/OSX version: ?
Dependency manager + version: ?

There should be an actual error message which says why it failed to build earlier in the build log.

I can't install too. When I downloading dependency: sync 4.9.5 from https://static.realm.io/downloads/sync/realm-sync-cocoa-4.9.5.tar.xz is very very long time then terminated.
-> Installing Realm (4.4.1)
 > Git download
 > Git download
     $ /usr/local/bin/git clone https://github.com/realm/realm-cocoa.git
     /var/folders/c1/x91t9vjs3m37jd7zgr66dm7w0000gn/T/d20200502-63444-d8kbxe --template= --single-branch --depth 1 --branch v4.4.1
     Cloning into '/var/folders/c1/x91t9vjs3m37jd7zgr66dm7w0000gn/T/d20200502-63444-d8kbxe'...
     Note: switching to 'fa43b8e2909334c79f233ce472332c136ca108da'.

     You are in 'detached HEAD' state. You can look around, make experimental
     changes and commit them, and you can discard any commits you make in this
     state without impacting any branches by switching back to a branch.

     If you want to create a new branch to retain commits you create, you may
     do so (now or later) by using -c with the switch command. Example:

       git switch -c <new-branch-name>

     Or undo this operation with:

       git switch -

     Turn off this advice by setting config variable advice.detachedHead to false

     $ /usr/local/bin/git -C /var/folders/c1/x91t9vjs3m37jd7zgr66dm7w0000gn/T/d20200502-63444-d8kbxe submodule update --init --recursive
     Submodule 'Realm/ObjectStore' (https://github.com/realm/realm-object-store.git) registered for path 'Realm/ObjectStore'
     Cloning into '/private/var/folders/c1/x91t9vjs3m37jd7zgr66dm7w0000gn/T/d20200502-63444-d8kbxe/Realm/ObjectStore'...
     Submodule path 'Realm/ObjectStore': checked out '92503d750cab072b3b78cf1dd696e63081c83d6a'
     Submodule 'external/catch' (https://github.com/catchorg/Catch2) registered for path 'Realm/ObjectStore/external/catch'
     Cloning into '/private/var/folders/c1/x91t9vjs3m37jd7zgr66dm7w0000gn/T/d20200502-63444-d8kbxe/Realm/ObjectStore/external/catch'...
     Submodule path 'Realm/ObjectStore/external/catch': checked out '7c9f92bc1c6e82ad5b6af8dff9f97af880fae9c6'
 > Running prepare command
   $ /bin/bash -c  set -e sh build.sh cocoapods-setup
   core is not a symlink. Deleting...
   Downloading dependency: sync 4.9.5 from https://static.realm.io/downloads/sync/realm-sync-cocoa-4.9.5.tar.xz

@rodneywitcher @JourKung Are you still experiencing this with the latest version of Realm?

@leemaguire I solved by latest version. Thank you


Train tabular models with many frameworks and import to Vertex AI using Pipelines

These pipelines are:

Working out of the box (code runs with zero modifications)
End-to-end (from nothing to a Vertex Model)
Feature multiple ML frameworks (TensorFlow, PyTorch, XGBoost, Scikit-learn)
Feature multiple training objectives: tabular classification and tabular regression

The main user artifacts are Python-based pipelines (pipeline.py), but pipelines in the standard component.yaml format are also included.
REQUIRED: Fill out the below checklists or remove if irrelevant

If you are opening a PR for Official Notebooks under the notebooks/official folder, follow this mandatory checklist:

[ ] Use the notebook template as a starting point.
[ ] Follow the style and grammar rules outlined in the above notebook template.
[ ] Verify the notebook runs successfully in Colab since the automated tests cannot guarantee this even when it passes.
[ ] Passes all the required automated checks. You can locally test for formatting and linting with these instructions.
[ ] You have consulted with a tech writer to see if tech writer review is necessary. If so, the notebook has been reviewed by a tech writer, and they have approved it.
[ ] This notebook has been added to the CODEOWNERS file under the Official Notebooks section, pointing to the author or the author's team.
[ ] The Jupyter notebook cleans up any artifacts it has created (datasets, ML models, endpoints, etc) so as not to eat up unnecessary resources.

If you are opening a PR for Community Notebooks under the notebooks/community folder:

[ ] This notebook has been added to the CODEOWNERS file under the Community Notebooks section, pointing to the author or the author's team.
[ ] Passes all the required formatting and linting checks. You can locally test with these instructions.

If you are opening a PR for Community Content under the community-content folder:

[x] Make sure your main Content Directory Name is descriptive, informative, and includes some of the key products and attributes of your content, so that it is differentiable from other content
[x] The main content directory has been added to the CODEOWNERS file under the Community Content section, pointing to the author or the author's team.
[x] Passes all the required formatting and linting checks. You can locally test with these instructions.

If there are no issues, then let's merge this community sample?


Adopt style guide

It would be nice to early on adopt an existing style guide so the conventions are well established. I use Invenia's Blue Style for my repos. JuliaFormatter.jl's defaults match Blue Style pretty well.

I feel strongly about https://github.com/jrevels/YASGuide/issues/13#issue-484616702, otherwise I feel flexible

I feel strongly about jrevels/YASGuide#13 (comment), otherwise I feel flexible

Hmm, I definitely prefer spaces around *. Less whitespace makes it much harder for me to read code. Somehow I don't have an issue with no space around ^, probably because visually it's really unique.
Is the rule basically spaces around + but all other operators have no spacing?
I feel pretty strongly about https://github.com/invenia/BlueStyle#module-imports. Module imports/exports/using should be in the top file of the package so the API and dependencies are obvious. import should be avoided, and packages should in general overload using the dot syntax i.e.
Base.eltype(x) = ... # good

import Base: eltype
eltype(x) = ... # bad

If a function/constant/struct not in the standard library is used, it should be prefixed by the module name. If it's used so much that that would make the code cluttered, then it can be explicitly included in the top file for clarity with (using ModuleName: functionname).
This makes it a lot easier to track whether a function was correctly overloaded or where the used functionality is coming from.

By convention, we tend to space operators more tightly if they get applied before other nearby operators.

https://docs.julialang.org/en/v1/manual/mathematical-operations/

I'm very on-the-fence about style guides. They're a good thing in principle, but it seems to sometimes be a pedantic slippery slope. I'd be strongly in favor of anything that can be automated, like JuliaFormatter.jl in vscode, and maybe a few other guidelines.
I do have some maybe unusual preferences, like I'd rather have exports near the function definition than in a giant list. I'm never going to read through the list, but if I'm thinking about a particular function, it's helpful to see whether it's exported. But it's not such a strong preference.
I could go either way on things like operator spacing, but hopefully it can automated so it doesn't take a lot of time or attention.
I agree that Base.eltype(x) = ... style is much better than eltype(x) = ....

Bumping this to say that in my (ongoing) docs revamp, I'm trying to generally adhere to JuliaFormatter / BlueStyle, especially by grouping imports and exports

Thanks @gdalle . I'd be very open to a PR to automate this in github actions


Fixed typo

Fixed the spelling of 'Neural Networks' again ;-)

Also please change https://github.com/octoai/octo-micro-site/blob/master/app/public/assets/images/octo6.png to correct the typo.


EC2 instances getting constant "refresh failed"

What happened?
We are trying to create EC2 instances using the aws provider. The instance does get created, but after it is up, there are constant error events in the resource.
Message:               observe failed: cannot run refresh: refresh failed: Conflicting configuration arguments: "cpu_threads_per_core": conflicts with cpu_options.0.threads_per_core
Conflicting configuration arguments: "cpu_core_count": conflicts with cpu_options.0.core_count
Conflicting configuration arguments: "cpu_options.0.core_count": conflicts with cpu_core_count
Conflicting configuration arguments: "cpu_options.0.threads_per_core": conflicts with cpu_threads_per_core

This doesn't let us update or remove the VM. We have to do it manually.
How can we reproduce it?
This is the manifest we are using:
apiVersion: ec2.aws.upbound.io/v1beta1
kind: Instance
metadata:
  name: test
spec:
  forProvider:
    ami: ami-07c781a9ff16018ea
    keyName: test-key
    instanceType: t3.medium
    subnetId: subnet-0b61449cc93c48841
    region: us-west-2
    vpcSecurityGroupIdRefs:
      - name: test-security-group
  providerConfigRef:
    name: default

What environment did it happen in?
Crossplane version: 1.12.2
provider-aws: v0.36.0
kubernetes: 1.26
distro: EKS

Also, the instances shows:
Ready = False
Synced = False
I can see the instance on the EC2 console


Guest Entry submission doesn't seem to trigger a cache invalidation?

My observation has been that submitting an anonymous entry using this plugin does not trigger a cache invalidation. So when the page reloads, the user's content is not visible yet. However, if I reload a CP URL, that seems to trigger the cache invalidation.
My workaround was to use the Cache Clear plugin, but it's kind of a blunt instrument, I'd rather have the appropriate cache keys cleared when the user submits their entry data.
If this can't be added to the roadmap, it might be worth pointing out in the readme?
Thanks!

I assume by “cache invalidation” you mean the “Delete stale template caches” background job. Craft only runs the queue when accessing the control panel, but you can fix that by disabling runQueueAutomatically and setting up your own queue worker that is either constantly listening for new jobs, or blindly running the queue once every minute, etc. See nystudio107’s Robust queue job handling in Craft CMS article for more info.


Flask server doesn't works correctly with JS

$ python simpleapi/example_project/server/flask1/app.py

$ curl -XGET 'http://<IP_ADDRESS>:5000/api/?_call=test
{"errors": ["Method must be provided."], "success": false}%

None for Python/Flask examples work for me, e.g calculator gives me error:
AssertionError: HTTP-Server returned http code 405 (expected: 200)