Render thread blocking with larger schematics

Attempting to render larger voxelizations often causes the program to hang and crash, especially if Ambient Occlusion isn't disabled.

Sometimes it's just the rendering that crashes and you can still try and apply blocks to the model, but nothing will export past that point so you can't actually get the litematic.
Here's the file I'm attempting to use. Extremely small height sizes can often work but anything sizeable seems to cause the issue.
Large Obj Test.zip

Once the /refactor branch has been merged, improving the UI responsiveness is my next goal. Currently, the work of the program is performed in the same thread as the UI so when heavy work is performed it blocks the UI from being responsive.
In addition, complicated meshes with thousands of triangles will always be a performance issue, I have plans to add the ability to simplify a mesh to lower the number of triangles in it and hence reducing the time to voxelise the mesh.

This should be resolved once /worker is finished and merged.

Now all meshes no longer use the old slow RenderBuffer code. On my PC, a mesh of 3 million triangles is loaded in 2.5s since c58a03e70cb6c620b55bbb8947816f75e77f38d6, whereas previously it would crash after 10s. This whole process still runs in the same process and still blocks the render thread which is not ideal but that's what /worker will fix but is a large refactor.


Display mobile plaintext notification doesn't work on Android

Turned it on, and it still only says "You got a message" so I have to open the app etc to see it.

Is this a known issue affecting all users, or is it occasional? It's affecting a friend of mine. Is the preference just not hooked up?


Expand documentation for prefilter and prefilterM

24 Pull Requests (https://24pullrequests.com)

Excellent work!  Thank you :)


Notify error logs on exit (recipe) (hashtag601)

Originally this post was a question - then I figured out a solution while writing, and now it's sharing the solution in case it helps someone else (plus welcoming improvements). Feel free to add to the recipes page or completely ignore :)
When the program encounters an error, I'd like to be notified with the full log to properly debug it.
Using the notifiers integration I can send myself the error messages themselves, but not the surrounding context of the rest of the log (eg, the version number and config of the program that's failing).
This approach sends the full log text when the program finishes running with at least one error.
Improved version from Delgan (with minor fixes):
class ErrorReporter:
    def __init__(self):
        self._log_lines = []
        self._has_encountered_error = False

    def write(self, message):
        self._log_lines.append(message)
        if message.record["level"].no >= 40:  hashtag ERROR
            self._has_encountered_error = True

    def stop(self):
        if self._has_encountered_error:
            log = "".join(self._log_lines)
            notifiers.notify(...)

logger.add(ErrorReporter())

My original:
import notifiers
from loguru import logger

def set_encountered_error():
    global has_encountered_error
    has_encountered_error = True

hashtag  ... Other logger setup
logger.add(lambda message: log_lines.append(message))
logger.add(lambda _: set_encountered_error(), level="ERROR")  hashtag Set whichever level

try:
    hashtag Your main function here
finally:
    if has_encountered_error:
        log = "".join(log_lines)
        notifiers.notify(...)  hashtag Your notifier

Hi  thanks for sharing. :+1:
You can also use a custom sink with a stop() method:
class ErrorReporter:
    def __init__(self):
        self._log_lines = []
        self._has_encountered_error = False

    def write(self, message):
        self._log_lines.append(message)
        if message["level"].no >= 40:  hashtag ERROR
            self._has_encountered_error = True

    def stop(self):
        if self._has_encountered_error:
            log = "".join(log_lines)
            notifiers.notify(...)

logger.add(ErrorReporter())

Note that keeping every message in  _log_lines can end up consuming a lot of memory.
Maybe you can use a file sink and send it using a custom compression function:
def send_file(filepath):
    if has_encountered_error:
         notifiers.notify(filepath)

logger.add("file.log", compression=send_file)
logger.add(lambda _: set_encountered_error(), level="ERROR")


[New] 申请

😀 申请人 Applicant

GitHub: mstao

🌏 领养宠物 Adopt pets

alpaca
dinosaur
panda
whale
unicorn

🌈 Tip

After completing the adoption, it is recommended to close the issue yourself. 完成领养后，建议自行关闭 issue。
How to show? 如何展示？
What we can? 我可以做什么？

Invited. Please check your email.

ok, thanks


Support relative paths for images in markdown/rich output cells (of untitled notebooks)

https://github.com/microsoft/vscode/issues/99139

Closing in favor of https://github.com/microsoft/vscode/issues/99139


chore: automated PR to update ReleaseNotes for 202306.13.0 VHD

This is an automated PR to bump ReleaseNotes for the VHD release with image version 202306.13.0

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall First build on releaseNotes/202306.13.0 at 61.476%

Totals

Change from base Build<PHONE_NUMBER>:
61.5%

Covered Lines:
2558

Relevant Lines:
4161

💛  - Coveralls


Update bsp4j to 2.0.0-M14

Updating in the hope to fix https://github.com/com-lihaoyi/Ammonite/issues/1175
I already updated guava in bsp4j https://github.com/build-server-protocol/build-server-protocol/pull/176. Apparently I will also need to do a PR in Ammonite https://github.com/com-lihaoyi/Ammonite/blob/master/build.sc#L76

Closing in favor of https://github.com/com-lihaoyi/mill/pull/1378 which I missed earlier :)


Deploy your Docker image to Kubernetes

As a [role]
I need [function]
So that [benefit]
Details and Assumptions
* [document what you know]

Acceptance Criteria
gherkin
Given [some context]
When [certain action is taken]
Then [the outcome of action is observed]

As a [role]
I need [function]
So that [benefit]
Details and Assumptions
* [document what you know]

Acceptance Criteria
gherkin
Given [some context]
When [certain action is taken]
Then [the outcome of action is observed]

As a developer
I need to deploy the docker image to kubernetes
So that I am able to deploy it to the customer
Details and Assumptions
* the environemnt is set up and is running and the docker image is created

Acceptance Criteria
gherkin
Given the environment is working and the docker image is created
When start the deployment to kubernets
Then the docker image is deployed


crypto__default.default.randomFillSync is not a function

Describe the bug
show error message: Uncaught TypeError: crypto__default.default.randomFillSync is not a function built by rollup
How to reproduce
npm install uuid, and build a project by rollup without even though specified a global config
Expected behavior
show error message: Uncaught TypeError: crypto__default.default.randomFillSync is not a function built by rollup
Runtime

OS: macOS
Runtime: Node.js, Chrome
Runtime Version: 14.12.0

Additional information
where is the crypto dependency defined (included in rng.js line.1 ) in this project?

Do you run the code in bowser?

See https://github.com/rollup/plugins/tree/master/packages/node-resolve#browser

Do you run the code in bowser?

yup

Then you need to bundle for browser. By default rollup/plugin-node-resolve resolves node versions. See the option in link above.

See https://github.com/rollup/plugins/tree/master/packages/node-resolve#browser

it worked successfully, ty!😊


Misaligned window controls on nonstandard display

Description
On a secondary display (mine has different aspect ratio and scaling factor, the close, maximize, and minimize controls are unaligned).
Include screenshot / video recording if possible.

Configuration

Firefox Version: 106.0.3 (64-bit)
I'm using the msix package (Windows Store)
OS: Windows 11 22h2

Huh. I think the window controls are CORRECT when it isn't in focused. Not only that, but it seems to reflect the Windows 11 design.

It seems like it renders properly when the window isn't in focus. The style of the maximize button looks more consistent with Windows 11 too.

Another edit: I tested unplugging the primary display, and it still happened. My laptop screen is 3:2 2256x1504 175% scaling. My primary monitor (on which it works) is 16:9 1920x1080 100% scaling.

When using the Mica tweak and the window is focused, the icons are drawn by windows but the button background when you hover over them is drawn by Firefox. The theme hides the window icons because the Mica appearance forces a set of window icons to appear. When the window is inactive the icons are drawn by Firefox, as Mica is disabled and the window background becomes a solid colour.
The misalignment seems to be caused by inconsistent button sizes when using fractional scaling settings. But there isn't any way to get the proper size of the icons. So I think the only way to fix this would be to add different sizes for different scaling settings. But I'll have to work on this another time, as Windows is not the operating system I use.

Closing this due to Mica being no longer supported as this issue only occurred with Mica enabled.


Deal with tenancy

This PR makes the orchestrator pass a tenant header with each of its requests.
To query for sources we have to first use the internal API to query for tenants. For this call (and all others to the internal API) we use a special tenant identifier which won't collide with existing users. After we have the list of tenants we can then query for sources and their dependent objects tenant-by-tenant.
Also of note here, this PR changes the #each_resource method so that it can deal with both paginated and non-paginated responses as the internal API doesn't use paginated responses.
This will resolve https://projects.engineering.redhat.com/browse/TPINVTRY-282

Overall looks good to me...some structural questions, but I'd be ok with merging this now and cleaning up later.


Compander should not save upon selection

Since the compander option is grouped with the other channel settings, it would be logical to assume that it does not save to the channel automagically.
@ In app/menu.c line 654:
SETTINGS_UpdateChannel(gTxVfo->CHANNEL_SAVE, gTxVfo, true, false, true);
should be changed to
SETTINGS_UpdateChannel(gTxVfo->CHANNEL_SAVE, gTxVfo, true, false, false);
Same for the ScAdd menu items, but in that case automagic saving makes sense, as there is no need (and no way) to "preview" the changes. Perhaps they should be moved down past the channel settings (right before SList) to indicate they don't need manual saving...

Just a little quirk with the change I suggested. It seems the compander setting does not reset back to the original setting when you change channels and then go back. Looks like it may need some other change(s) to match the behavior of the other settings...

The real question would be whether the compander function is actually used by anyone. I never use it. And you ? 🤗

I use it every now and then, which is how I found the "bug"... Although it's no big loss for me if it's gone, I know lots of radios (including very cheap ones) have this function, and it does make long distance communication more clear.

I'd proposed removing it to free up program memory. This is a 5W HT, and I'd guess 80% of the user base are non-hams wanting to use it as a family walkie-talkie, or listen to airband. I respect that the three of us are hardly "representative", so I'm prepared to be corrected on this, but if anyone was taking the feature seriously, they'd be on here looking down the feature list and making suggestions.

Compander is an attribute specific to each channel. I won't change anything here 😌


Add IE 11 as a target

Attempt to fix async/await issues

@JamesAPetts I'm going to merge this to see if it resolved my issue. Feel free to poke/prod at it, and let me know if I need to revert or address any issues 👍


Support custom json scalars.

I want to store a mailing address in a single field.  The best option I've found so far is to store it in a json field.
type Contact {
   ...
   optional mailing_address: json;
   ...
}

But if I do that, i basically loose all the type info of the address..  It would be nice if I could preserve it.  Something along the lines of
scalar type Address extending json {
   line1: str;
   line2: str;
   city: str;
   state: str;
   zip_code: str;
}

type Contact {
   ...
   optional mailing_address: Address;
   ...
}

I understand that if you want to support modeling any valid json in scalar (number, string, arrays, in addition to objects), the syntax likely needs to be a little more complicated.  The above is just an attempt to illustrate the desired outcome.

Or maybe this would be better described by letting the json type accept a generic type param.  Kinda like:

a json string: json<str>
a json string array: json<array<str>>
a json object: json<tuple<title: str, published: bool, upvotes: int64>>

So in my original example the Address scalar is defined as:
scalar type Address extending json<tuple<
   line1: str,
   line1: str,
   line2: str,
   city: str,
   state: str,
   zip_code: str
>>

Can you speak a bit more about the "I want to store a mailing address in a single field." part? From my perspective without knowing a bit more about your motivation here, it seems like a less powerful way of expressing the existing type language.
Some alternatives to consider:

Use a tuple here. Tuples cannot have optional properties so maybe that is why you're not currently do that?
Make this a real type. Gives you way more tools for your address model (constraints, indexing, query flexibility, migrations, triggers, mutation rewrites, etc)

A scalar that extends a named tuple with optional types would be awesome.   But I don't think you can extend a tuple right now.  And can you define a type field as being a named tuple?


feat: bump deps

Bump all major deps.
Time spent on this PR: 1 day
Resolves: #541 and #536
Pull Request type

Please check the type of change your PR introduces:

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[x] Refactoring (no functional changes, no API changes)
[ ] Build-related changes
[ ] Documentation content changes
[ ] Testing

What is the new behavior?
Bumps the deps to the latests for dojo.

Does this introduce a breaking change?

[ ] Yes
[x] No

increase the nonce of the contract account as well plz


Handle compositions of compositions correctly

When composition points to claims or XRs instead of managed resources, we should display and handle that correctly

@candonov FYI


[SttingsV2] Updated design spec

We need an updated design/layout specification for Settings V2.
At the moment we have various mock ups, issues and ideas. For me it is not clear how the final setting should look and behave.
It should be based on the MS Design Guidlines.
It should also include the results off #2700, #2705, #2710, #2688 and other interesting mock ups/ideas.
@enricogior, @crutkas, @niels9001, @laviusmotileng-ms: Thoughts?

Not sure. The examples look outdated (Windows 8ish, with e.g. Windows Mobile mentioned). We want to look as native as possible and based our current design on the current Windows 10 Settings app in terms of padding, margins and fontsizes. Only tweak we have made is the page title: that now follows the Windows 10X page title style.
I think, with the current implementation, we are really close to the design guidelines you mentioned. Even inbox apps like Mail or Your Phone do not follow these to the exact pixel, which is fine since they are _guide_lines.
Being as close as the native Settings app makes most sense, since we both have not a single settings page but basically an entire 'app'.
My 2 cents :).

Also, if you want to follow up on some of the ideas and thoughts around this topic, check out issue #889

@niels9001 You are rigth.
But we have to ensure, that changes on link/button style and other generally things are done for all settings pages.
In which way are they coordinated?

I would have thought the PowerToys WinUI app would have a shared Xaml ResourceDictionary, so pages using a shared resource, will update as that style resource is updated.
That should enable consistency throughout all PowerToys Xaml UIs

@mdtauk, little by little

@mdtauk That could bee. I don't know that.
What I mean with the design spec is, that I can use one different settings pages

different controls for the same setting. (Example: on/off switch & checkbox; dropdown & radio buttons)
different namings for the same setting.
I have seen PRs which overrite changes from other PRs. (Example: opensource notice - #2676 override #2176)

I'm going to close this out, i think a better work item here is "Clint sync's with the design team".  Based on that, we can get official docs updated if they differ.  I agree we should be true to docs but those are guidance as well.


Add exitstat vmtest

Summary:
Add exitstat vmtest to make sure we keep the bpf program working on
different kernel versions.
Differential Revision: D54585856

Closed with 3d08aef


Improve performance of large, inefficient partial aggregations

Sometimes partial aggregations can be particularly inefficient in terms of reducing the row count of the original input batch.  For example, computing a distinct result of many input batches that are already distinct.  In these situations, it may be more efficient to send the batched partial aggregations directly to the shuffle as they are computed rather than attempting to cache and merge them for minimal shuffle size savings.  The final merge after the shuffle will still have to do the merge, but the partial aggregation could be much more efficient for that scenario.
The row count of the batch aggregations could be compared to the original input batch row counts, and some heuristic could be used to determine that the input appears not worthy of merge-aggregating in the partial phase.  This could be made even more efficient if we realize the aggregated schema matches the input schema, meaning we could pass the input batches straight to the shuffle without attempting a partial aggregation at all.

"we could pass the input batches straight to the shuffle without attempting a partial aggregation at all."
DataBricks runtime did the exact thing: eliminating map-side combine under certain situations.

Closing as won't fix for now


TrySingle : xs → Zero | (x, One) | (x, Many)

I'm wondering whether the community would think that TrySingle would be a useful addition to MoreLINQ.
public static Cardinality TrySingle<T>(this IEnumerable<T> values, out T result)
public enum Cardinality { Zero, One, Many }

Where the behaviour of TrySingle is to attempt to return the first, single element from the enumerable. If the enumerable has a single element, out result is populated with the item, and Cardinality.One is returned. However, if there are zero items in the enumerable then the method returns Cardinality.Zero; if there are multiple items in the enumerable then the method returns Cardinality.Many. In both of these cases the out result is null.
A consumer of this method can then distinctly handle the Zero or Many cases in whichever special way is required.
I'm happy to submit a PR if this is worthwhile.

True, that could be the implementation perhaps.

If we are going to add this, I would like to avoid introducing the Cardinality enum. I think the following is a lot more generic, a pure function and avoids by-ref, output, parameters:
public static TResult
    TrySingle<T, TCardinality, TResult>(this IEnumerable<T> values,
        TCardinality zero, TCardinality one, TCardinality many,
        Func<TCardinality, T, TResult> resultSelector)

In other words, the caller defines the three cardinality cases and the resultSelector will get to know which one was the case. The original signature and enum you proposed can then be implemented locally as follows:
public static Cardinality TrySingle<T>(this IEnumerable<T> values, out T result)
{
    Cardinality cardinality;
    (cardinality, result) =
        values.TrySingle(Cardinality.Zero,
                         Cardinality.One,
                         Cardinality.Many,
                         ValueTuple.Create);
    return cardinality;
}

very nice suggestion, but I think Cardinality is overkill. what about this?
public static IEnumerable<T> ExtractFirst<T>(this IEnumerable<T> values, out T result)
{
    result = default;
    using (var enumerator = values.GetEnumerator())
    {
        var move = enumerator.MoveNext();

        if (move) result = enumerator.Current;

        return GetRest();

        IEnumerable<T> GetRest()
        {
            if (move)
            {
                while (enumerator.MoveNext())
                {
                    yield return enumerator.Current;
                }
            }
        }
    }
}

@MkazemAkhgary Your ExtractFirst implementation (which appears to be CADR-like) has two major flaws:

It disposes the enumerator.
It changes the enumerator into an enumerable when they're not the same thing.

If null is returned, source contains no element.
If returned enumerable is empty, source contains single element.

If these are the rules, then it suffices to have something like this:
public static IEnumerator<T> TryMoveToFirst<T>(this IEnumerable<T> values)
{
    var enumerator = values.GetEnumerator();
    try
    {
        return enumerator.MoveNext() ? enumerator : null;
    }
    catch
    {
        enumerator.Dispose();
        throw;
    }
}

The Current property of the returned enumerator will contain the first element so you don't need a separate out-argument to capture it. Disposing the returned enumerator will be the caller's responsibility.

#590 could address this too and since there's been no further follow up from @jimmcslim, I'm going to close this issue.

Re-opening this issue now that there is a PR (#612) open for it.

I have moved this to the experimental namespace as I feel this needs more thought in terms of naming.


GH-199: Do not wakeup consumer after queueing ack

Waking up the consumer too often will trigger a rebalance and messages are not actually committed
Please patch this on the 1.0.x branch as well for kafka 0.9 users

@martindam Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

@martindam Thank you for signing the Contributor License Agreement!

Hello, @martindam !
To be honest, I'm fine with your solution. It really looks better to have a short poll period instead of wake up, rather than don't have heartbeats and finish with always rebalance.
We thought with @garyrussell to have something like immediate flag to do existing logic for BATCH and RECORD modes. But that looks like switching on that option for your workload will lead you to the same bad system state. So, for me that smells weak and bug-aware anyway.
So, my position is as +1 to drop wakeup() where you do that already.
But in addition I'd add commitOffsets() call after poll() invocation one more time. Just because during poll() blocking we may have added new offsets for commit.
Although, tell me, please, if you receive the same records during the next poll, while we haven't committed yet because we don't wakeup() any more.
@garyrussell , your thoughts ?

As long as you do not trigger a rebalance, you will not get duplicate messages from the poll() independent of you committed the offset or not. Committing the offset is committing it externally, but the consumer keeps track of the next offset to read internally anyway.
You could add the commitOffsets() after the poll(), but you just did that before the poll and this loop should continue so I don't see the need. Unless you do auto commit you are just passing the batch of records to the queue and then calling the commitOffsets() so I don't think it is needed.

Thank you, @martindam , for the explanation!
Several more points to sanctify:
When we have several consumers in the same group, only one of them process single partition.
And when happens rebalance a new partition consumer consults stored offset to start with.
If that correct, then I won't worry a lot about non-committed offsets, since we always can reply them again and filter in a new consumer.

Yes that is correct, after a rebalance it will clear its internal offsets and consult the external stored offsets and start from there even if the same consumer is reassigned a partition it already had.

LGTM; thanks @martindam

Merged as https://github.com/spring-projects/spring-kafka/commit/c245cbbe8905af1334ce268ebb05420c3afb1265 and cherry-picked to 1.0.x.
Thank you for valuable contribution, @martindam ! As always :smile:

You are welcome. What are your plans for releasing 1.0.5.RELEASE?

Well, @martindam ,
Frankly we have only one commit (your) there since the last 1.0.4 release.
Or everybody has moved to 1.1 already with Kafka 0.10 support, or they use some other configuration or don't use Spring Kafka at all :smile:
Although we are flexible enough and can release 1.0.5 anyway. Even if it is useful only for you.
@garyrussell , you thoughts ?

I have no issues making a 1.0.5 release, even with just 1 commit.

Good. Then it is fine with me to release just today!
If there are any objections, you have 10 min until I start the release process :smile:

@martindam ,
The version 1.0.5 has been released: https://repo.spring.io/release/org/springframework/kafka/spring-kafka/1.0.5.RELEASE/
Enjoy. And thank you for contribution!


Trait

maybe im just new to this, but i keep getting the following:
Trait 'matting\RecordsActivity' not found (matting being my namespace)
I have followed your instructions perfectly.

Sorry for the slow response - didn't get a notification for some reason.
It sounds like you simply need to add a "use" statement at the top of the class that you are utilizing the trait in.
It should look like this:

use Chaseconey\ActivityRecorder\RecordsActivity;

Class Tweet extends Model
{
  use RecordsActivity;
}

Let me know if that works!

I updated the README to display this information as well. Thanks for pointing that out.

@cardi777 Does this solve the problem for you?

I'll let you know. Have to reinstall, I had given up :)
On Thu, 2 Jul 2015 11:40 pm Chase Coney<EMAIL_ADDRESS>wrote:

@cardi777 https://github.com/cardi777 Does this solve the problem for
you?
—
Reply to this email directly or view it on GitHub
https://github.com/chaseconey/laravel-activity/issues/1#issuecomment-118036748
.

--
Cheers,
Doug

ok i think we are good :)

Ok cool - glad it was that easy!

i love this plugin, its perfect for what i want.


Update to ng-annotate 1.0.0

Hi v1.0.0 is out, could you please try to update this plugin?

@shprink sure. I'll get to it after wor.

Awesome thanks :)
On Jun 29, 2015 9:08 PM, "Jeffrey Ling"<EMAIL_ADDRESS>wrote:

@shprink https://github.com/shprink sure. I'll get to it after wor.
—
Reply to this email directly or view it on GitHub
https://github.com/jeffling/ng-annotate-webpack-plugin/issues/12#issuecomment-116799646
.

merged. will publish after a bit more testing.


Chore/alert style cleanup

Some hygiene work on alert component. Grouping similar style as mixin and remove z-index so the component can be used in sticky header container or within a page.

any print screen?

It got no visual change at all, but you might find current style guide got this glitch when open the site menu in alert demo page.

looks like the package also affecting here


Upgrade Microsoft.Net.Compilers.Toolset to 4.6.0

After upgrading my Visual Studio yesterday to the latest I got this error:
Severity	Code	Description	Project	File	Line	Suppression State
Error	CS9057	The analyzer assembly 'C:\Program Files\dotnet\sdk\7.0.302\Sdks\Microsoft.NET.Sdk\codestyle\cs\Microsoft.CodeAnalysis.CodeStyle.dll' references version '<IP_ADDRESS>' of the compiler, which is newer than the currently running version '<IP_ADDRESS>'.	Mapsui (netstandard2.0)

The upgrade fixed it.

The error message means something in the windows app sdk isn't anmyore compatbile i'll upgrade the analyzers and the windows app sdk to see if it runs

intrestingly locally this works. So it seems that to merge this upgrade we have to wait until the Image gets upgraded to Visual Studio 17.6

false was the fix for the build.
Should I make a second pull request with the changes for winui (removing workarounds, and fixing dark mode).

Thanks a lot for helping out. This is the kind of issue I struggle with. I am very eager to upgrade everything to .NET 6. Very unfortunate that simply upgrading VS breaks the build. That should not happen.
Fine that just work with me in the same MR.
The release nugets actions still fails for this branch on dotnet workload install maui macos android ios maccatalyst wasm-tools wasm-tools-net6, but that command is identical to the windows build:
https://github.com/Mapsui/Mapsui/actions/runs/5097832115/jobs/9164604923

I forget to add the .net 7 install to the release yml. will do it now.

I forget to add the .net 7 install to the release yml. will do it now.

Thanks, that fixed it. Now there is an error later in the script
error MSB4018: The "Microsoft.DotNet.ApiCompat.Task.ValidatePackageTask" task failed unexpectedly https://github.com/Mapsui/Mapsui/actions/runs/5103128241/jobs/9173132346#step:10:479

I set
    <RunApiCompat>false</RunApiCompat>
no in
Directory.Build.props
so that in every project the runapicompat task is disabled. (We don't use it currently anyway). The task checks for breaking changes between Versions. It enforces the Correct Version numbering for Releases.

I set
 <RunApiCompat>false</RunApiCompat>
now in Directory.Build.props so that in every project the runapicompat task is disabled. (We don't use it currently anyway). The task checks for breaking changes between Versions. It enforces the Correct Version numbering for Releases.

I know nothing about this. Read a bit now here: https://learn.microsoft.com/en-us/dotnet/fundamentals/package-validation/overview

So can you try again?

Fixed it!


Spring interpolation throws an exception when the to value is null

When interpolation occurs for a null value, an exception TypeError entry.interpolation._interpolation is not a function is thrown at line 144 of Spring.js.
// Deal with interpolations
const previous =
    entry.interpolation &&
    entry.interpolation._interpolation &&
    entry.interpolation._interpolation(entry.animation._value)

I'm not sure if this is expected behaviour, but I would have thought it could either

fail gracefully and warn, or
skip the animation and instead just pass the value of null through to the child function.

This sandbox reproduces the bug: https://codesandbox.io/s/8lll45290l
Looking at the code snippet above, it seems odd that it expects to safely call the _interpolation function.

The from-to type should always be the same. It would be good to have invariant checking in dev mode at some point, though.


Email not working on Sparkpost

I'm having trouble sending email using sparkpost. My test email route works fine and also other user emails, but the ticketit emails are not working.

Have you configured the from setting in config/mail.php?

Hi. I've configured the from setting 2 days ago. I solved it. Thanks for the reply.  Mail sending works fine now.


Logwatcher

Is possible to run the logwatcher as a daemon or to make it run continuosly?
How the logwatcher graph should be read?
You show the occurence on a per minute base?

Any plans to make this an npi compatible plugin?


Can I ClaudeDev on Windows machine?

Can I ClaudeDev on Windows machine?
If yes: how exactly can I do that?

It's a VSCode extension, download VSCode and then the extension. Linked from Readme
https://marketplace.visualstudio.com/items?itemName=saoudrizwan.claude-dev


BiomeJS extension and LSP not working properly on Zed #13492

Issue Link (Zed Issues): https://github.com/zed-industries/zed/issues/13492#issue-2371850613

You should set up settings.json like this:
{
  "code_actions_on_format": {
    "source.fixAll.biome": true,
    "source.organizeImports.biome": true
  }
}

See https://github.com/zed-industries/zed/issues/13492#issuecomment-2246243131 :)


build: Update API-Extractor configs in client packages under packages

Updates configs to use the new shared api-extractor config (from build-common), which is stricter and enables trimmed type-rollup generation. Nothing consumes these types yet.
For now, it disables some of the stricter rules in a number of the packages (any where it required fixing more than a few comments, or involved missing exports). The required changes in them will be handled in follow-up PRs.
One noteworthy change resulting from the config update: .api.md reports have moved from the repo root to instead live directly under the associated package.
AB#5694

Looks like the driver-web-cache api docs are duplicated in a lot of places.

Looks like the driver-web-cache api docs are duplicated in a lot of places.

What on earth... No idea how that happened. I'll investigate. Marking this as a draft for now.


Add Prometheus metrics endpoint

We want to be able to monitor a cxMate service using Prometheus. We need to add a "/metrics" endpoint that serves various metrics about cxMate's behavior.

If at all possible, please use our client libraries; it will make your life a lot easier.


Application is not running. Application.exit() failed

Issue
It seems to be working (it injects the changes) in watch files mode but getting this error when i make a change in --watch-files mode
I do have to press enter in the aider terminal to get it proceed though, not sure if that is normal
File "[REDACTED]/.local/share/uv/python/cpython-3.12.8-linux-x86_64-gnu/lib/python3.12/threading.py", line 1075, in _bootstrap_inner
self.run()
File "[REDACTED]/.local/share/uv/python/cpython-3.12.8-linux-x86_64-gnu/lib/python3.12/threading.py", line 1012, in run
self._target(*self._args, **self._kwargs)
File "[REDACTED]/.local/share/uv/tools/aider-chat/lib/python3.12/site-packages/aider/watch.py", line 130, in watch_files
raise e
File "[REDACTED]/.local/share/uv/tools/aider-chat/lib/python3.12/site-packages/aider/watch.py", line 125, in watch_files
self.io.interrupt_input()
File "[REDACTED]/.local/share/uv/tools/aider-chat/lib/python3.12/site-packages/aider/io.py", line 397, in interrupt_input
self.prompt_session.app.exit()
File "[REDACTED]/.local/share/uv/tools/aider-chat/lib/python3.12/site-packages/prompt_toolkit/application/application.py", line 1289, in exit
raise Exception("Application is not running. Application.exit() failed.")
Version and model info
Aider v0.70.0
Model: deepseek/deepseek-coder with diff edit format, prompt cache, infinite
output
Git repo: .git with 0 files
Repo-map: using 1024 tokens, auto refresh

Thanks for trying aider and filing this issue.
Are you able to reproduce this error?

Yes, it happens everytime.  It works as I mentioned above, though I have to hit enter in terminal.

How did you install aider? What OS/environment are you using?

I installed aider using pip and aider-install as per the instructions.  I tried fixing it by uninstalling and then reinstalling, but didn't seem to help.
I am using linux ubuntu.

Feel free to close this if no one else reports it.  I tried googling around but didn't see anyone else say anything.  It could be something wrong with my environment.


how to predict answers for different question and context by reusing loaded model.

Hi:
After i reload the mrc model,
how can I implement the  function like predict to get the answers for different input questions and contexts.
Meanwhile, i want to reuse the loaded mrc model and session. I do not want to reload model and initialize the session many times.
Thanks

because graph_handler depends on the model from get_multi_gpu_models(), while the model from get_multi_gpu_models() depends on the input questions and contexts.

In chinese： 如果做到load 存储的模型一次，就可以测试任意多个文章／若干个question，输出答案。而不用每一次都run_single,加载模型多次。
现在，有n个很小的测试数据，分别在mytest1.json,,,mytestn.json.  如下脚本跑，每一次都会加载模型，耗费不少时间。
basic/run_single.sh $HOME/data/squad/mytest1.json single.json
basic/run_single.sh $HOME/data/squad/mytest2.json single.json
...
basic/run_single.sh $HOME/data/squad/mytestn.json single.json
有没有方法，写点python脚本 prepare 准备好everything，然后 , 而prepare中只会load 模型一次。
prepare(mytest1.json, single.json)
prepare(mytest2.json, single.json)
...
prepare(mytestn.json, single.json)
恳请给点指导。新手膜拜你的代码中！

Could you tell me what "mrc model" is?

"mrc model"  i mean the  the restored the model by "saver.restore(sess, save_path)" .
My question is :
I have many json files which is the same to the standard train file or dev file, but in each of my json file, there is only one paragraph, one question. I use the mrc to predict the answer.
Since  the following scrip will cost so much time:
basic/run_single.sh $HOME/data/squad/mytest1.json single.json
basic/run_single.sh $HOME/data/squad/mytest2.json single.json
...
basic/run_single.sh $HOME/data/squad/mytestn.json single.json
Can i use a script  to restored the model and initialize  everything once, then i can use the it  on different json files?

Basically you want to set up a server which can take in question and output prediction, is that correct?
As you pointed out, calling script every time will be costly. At this point, probably the easiest thing without touching anything is use our demo. You can host your own demo by using demo branch (https://github.com/allenai/bi-att-flow/tree/demo). Then you can perform GET request to directly get the answer from the server.

That's what I need. Thanks a lot. Let me have a try.  👍

@rubby33 Were you able to run the demo?
I tried to run it and faced the problem that the pretrained model at out/basic/00/save/basic-18000 wasn't provided and instead there is a file named basic-20000 under the same location.
However, now I face a new problem while trying to run python run-demo.py, the program throws a:
NotFoundError (see above for traceback): Tensor name "model_0/prepro/u1/BW/BasicLSTMCell/Linear/Matrix/ExponentialMovingAverage" not found in checkpoint files out/basic/00/save/basic-20000
is there anything else that I should set into the running configuration? Thank you

Same issue here. Thank you.

Are you running on tensorflow < 1.0?

yes, installed r0.11 to match the readme. tensorflow.version gives me 0.11.0.

Ohhh I see, you're running on the demo branch right?
If I remember rightly, I copied all the flags in basic/cli.py and replaced all the flags in basic/demo_cli.py for the one in cli.py, then I changed the following flags to pick the data from the correct location:
flags.DEFINE_string("data_dir", "path-to-bi-att-flow/data/squad", "Data dir [data/squad]") flags.DEFINE_string("eval_path", "data/squad", "Eval path []") flags.DEFINE_string("load_path", "path-to-bi-att-flow/save/37/save", "Load path []") flags.DEFINE_string("shared_path", "path-to-bi-att-flow/save/37/shared.json", "Shared path []") flags.DEFINE_string("mode", "forward", "trains | test | forward [test]")
You can also change the flags on the command.
I hope it helps.

Thank you! I ran:
python run-demo.py --data_dir=data/squad --eval_path=data/squad --load_path=save/37/save --shared_path=save/37/shared.json --mode=forward and got the server running.
This is perhaps a different matter in need of a separate issue (please advise):
In the online demo, we're able to write our own paragraph and ask a question about it (as opposed to only select one of the predefined paragraphs).  The UI on the demo running on my machine doesn't seem to allow writing my own paragraph.  Is this possible somehow?  Is there perhaps some curl command I could run passing in a paragraph and a question and getting an answer in response?
Apologies in case my question exposes me missing some obvious matter.
Thank you

Yeah!
The demo only works with text from the dataset. If you look into the code, the client is only sending the paragraph_id instead of the text itself so even if you change the text, the backend will go and pick the original paragraph.
If you want to predict new text, I’d suggest to look at the function basic/main.py::_forward and pay attention at all the stuff is going on from line 186 to 211. You will have to preprocess your new data to work with that workflow.
I hope it helps

On 6 Apr 2017, at 20:54, Norris<EMAIL_ADDRESS>wrote:
Thank you! I ran:
python run-demo.py --data_dir=data/squad --eval_path=data/squad --load_path=save/37/save --shared_path=save/37/shared.json --mode=forward and got the server running.
This is perhaps a different matter in need of a separate issue (please advise):
In the online demo, we're able to write our own paragraph and ask a question about it (as opposed to only select one of the predefined paragraphs). The UI on the demo running on my machine doesn't seem to allow writing my own paragraph. Is this possible somehow? Is there perhaps some curl command I could run passing in a paragraph and a question and getting an answer in response?
Apologies in case my question exposes me missing some obvious matter.
Thank you
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/allenai/bi-att-flow/issues/13#issuecomment-292296166, or mute the thread https://github.com/notifications/unsubscribe-auth/AAfBQRHTJ0EC6VFhzjM-KZdRnpPvCsr7ks5rtULhgaJpZM4Mn3xU.

@webeng Hmm, in fact, it seems the demo branch is not up to date.. It used to be like that, and we changed it later (current demo). @shmsw25 Could you please take a look at this?

@bluelight773 I tried running your command and the server now works but when I press submit on a question I can an internal server error. Did you also experience this problem?

I'm also interested in adding my own paragraphs - did you manage to edit the demo to use your own text?

@nikhilaravi I'm not getting the server error you're getting.  When I submit one of the pre-defined questions on one of the pre-defined paragraphs (which seems to be all that's supported in the current version of the demo on the repo), I get an answer as expected.  Not sure how our set-ups may differ. For what it's worth, I'd set up a conda environment where I ensured the required packages listed in the readme here are installed with the specified versions (where mentioned): https://github.com/allenai/bi-att-flow/blob/master/README.md.
@shmsw25 I second @nikhilaravi 's request on updating the branch if possible :)  I'd imagine many people would be interested in the functionality of specifying own paragraphs and own questions using the pretrained model as is already supported in the online demo. :)

@bluelight773 thanks I managed to fix the issue! @bluelight773 I managed to get user input of paragraphs working - just clone @shmsw25's fork of the repo and use the demo branch (from #22 I noticed it had been updated in her version and is just waiting to be merged into this repo)

Tested, it works as the online demo! Thank you very much for the hardwork!

Thanks . It works fine after several test.

@seominjoon @rubby33 @salmedina  How can we return the top 5 answers for a question along with the scores instead of only the top answer?

@rubby33 If the sentence doesn't fit into the threshold, the sentence gets chopped (if I remember correctly). So this will lose accuracy. Otherwise, the accuracy should be the same (i.e. larger threshold doesn't hurt). Of course, larger thresholds needs more GPU RAM.
@nikhilaravi I want to add this functionality in future, but for now probably the easiest way to do it is looking into the eval-XX.json (or eval-XX.pklz) files. How to use this file can be seen basic/ensemble.py of dev branch. https://github.com/allenai/bi-att-flow/blob/dev/basic/ensemble.py#L94 This line of code computes the best span, but you can sort it and get best 5 if you want to.

@seominjoon thanks very much, I'll look into it!

Hi,
I have been trying to figure out why the model doesn't work so well for short sentences. For instance if my input paragraph is "I go to school" and the question is "Where do I go to" , I dont get an answer.
http://squad.budianto.id which is an extension of your BiDaf seems to be able to work well for short sentences but performs poorly in comparison to yours for longer paragraphs. This makes me wonder if there is a window parameter which I can play around to make your Bi-Daf work better for smaller sentences.

@diamondspark This is bug actually, related to #18
Fixing this is easy, although I didn't update the master branch yet.
For now, probably the easiest way around this is put a word such as period at the end (e.g. "I go to school." instead of "I go to school").

@seominjoon back to the question regarding thresholds, 1) can you confirm that when running the demo, that these are the thresholds that are in effect?
(From basic/demo_cli.py)
flags.DEFINE_integer("word_count_th", 30, "word count th [100]")
flags.DEFINE_integer("char_count_th", 150, "char count th [500]")
flags.DEFINE_integer("sent_size_th", 1000, "sent size th [64]")
flags.DEFINE_integer("num_sents_th", 1000, "num sents th [8]")
flags.DEFINE_integer("ques_size_th", 100, "ques size th [32]")
flags.DEFINE_integer("word_size_th", 48, "word size th [16]")
flags.DEFINE_integer("para_size_th", 1000, "para size th [256]")
Above values are different than what they are are in basic/cli.py

Could we change those values, in basic/demo_cli.py, before running the demo to any desired value and they'd take effect once the server is up and running?

Can you confirm, we don't actually need to train a new model with new threshold values. We can just update those values and they'd apply regardless of the model we're using?

I don't wanna make this too long, but if you're able to specify the meaning of those 7 parameters, it'd be quite helpful as for instance it's not obvious in some cases whether the numbers are referring to number of chars or words, or whether that number applies in the context of a word/sentence/paragraph.

Thank you

@bluelight773

The params are right. You can ignore num_sents_th because right now entire paragraph is considered as a single sentence. So basically, you can put up to 1000-word context into the demo, and you can increase it arbitrarily. Note that this will slow down in CPU and could result in OOM in GPU.

Yes, you can change them to any values before running the server and they will be effective right away.

Yes, those params can be arbitrarily changed without retraining.

word_count th, char count th: ignore these for now
sent_size_th: maximum length (# of words) of each sentence
num_sents_th: maximum number of sentences. This is not used now, because the entire paragraph is considered as a single sentence.
ques_size_th: maximum # of question words
word_size_th: length (# of chars) of each word. If exceeds, only take first 48 chars.
para_size_th: maximum number of words in the context. Since the entire para is considered as one sentence, this should equal sent_size_th

Does this demo have to be run on tensorflow 0.11 ?

@lbbc1117 Yes, but if you want to make it compatible to v1.2, perhaps the best way is to look at the dev branch, which is compatible with v1.2.

@rubby33 When you create the json files whose format are the same to the standard train file or dev file. Do you put, for each question, the answer_start parameter?

I am trying to predict the answer to my own custom question using the pre-trained model.
The test1.json file following the Squad format is as follows:

{"data": [{"title": “News”, "paragraphs": [{"context": "jordan 's crown prince hassan ibn talal arrived tuesday for his first visit to jerusalem and was to pay his condolences to the widow of assassinated prime minister yitzhak rabin .", "qas": [{"answers": [{"answer_start": 81, "text": “jerusalem”}, {"answer_start": 81, "text": "jerusalem"}, {"answer_start": 81, "text": "jerusalem"}], "question": “where did the crown prince visit?”, "id": “1”}]}]}], "version": "1.1"}

However, when I run the command basic/run_single.sh test1.json single.json, I get the following error
raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 1 column 21 (char 20)

Can someone help me out?

The quotes around the title name "News" in the json doesn't look right.
There are a few other places in the json which has the wrong quotes.
Thanks,
Smita
On Sat, Dec 2, 2017 at 6:12 PM, Bhavika Reddy Jalli <
<EMAIL_ADDRESS>wrote:

I am trying to predict the answer to my own custom question using the
pre-trained model.
The test1.json file following the Squad format is as follows:
{"data": [{"title": “News”, "paragraphs": [{"context": "jordan 's crown
prince hassan ibn talal arrived tuesday for his first visit to jerusalem
and was to pay his condolences to the widow of assassinated prime minister
yitzhak rabin .", "qas": [{"answers": [{"answer_start": 81, "text":
“jerusalem”}, {"answer_start": 81, "text": "jerusalem"}, {"answer_start":
81, "text": "jerusalem"}], "question": “where did the crown prince visit?”,
"id": “1”}]}]}], "version": "1.1"}
However, when I run the command basic/run_single.sh test1.json single.json,
I get the following error
raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 1 column 21 (char 20)
Can someone help me out?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/allenai/bi-att-flow/issues/13#issuecomment-348734865,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHFlo_J7j5U_rCYirz8uRawK1cmBSRdTks5s8gN4gaJpZM4Mn3xU
.

Hi,
I ran the localhost demo (demo branch) and tested using both pre-defined and my own paragraphs. However, I don't get the similar results like your online demo (http://allgood.cs.washington.edu:1995/). The main problem is that, after writing the first question and getting its answer, I keep getting the same answer for the next questions. Whereas your online-demo works well.
Any advice is appreciated, thanks.

Hi,
I am also using demo and successfully able to run it but I am not getting even a single answer correctly. The model is giving same answer from any question.
Note : I didn't do anything else just ran python run-demo.py and it get started. Also I didn't use pre-trained model or anything else. Can someone help me please. @salmedina  Thanks a lot


Allow PHP 8 (and future PHP versions)

@marcaube How about it??

I've done it here https://github.com/94noni/ObHighchartsBundle#obhighchartsbundle


Move entities from Chunk to Cube

Fixed in https://github.com/OpenCubicChunks/CubicChunks/commit/6f322a0ba98384153c18002445de4d2ef55c8fec


Backlog 22976

SSL capability for IBM MQ

Build Completed
:x: This pull request has errors. They will need to be addressed before it can be accepted. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
mvn -B -fn -f 'pom.xml' -pl 'ui,plugins/streaming/impls/jms' -P '!assemblies' -Dsurefire.runOrder=alphabetical -Daudit -amd clean install

Cleanup Commands
mvn -B -f 'pom.xml' -pl 'ui,plugins/streaming/impls/jms' -P '!assemblies' -amd build-helper:remove-project-artifact

Changed files
  plugins/streaming/impls/jms/src/main/java/org/pentaho/di/trans/step/jms/JmsDelegate.java
  plugins/streaming/impls/jms/src/main/java/org/pentaho/di/trans/step/jms/context/WebsphereMQProvider.java
  plugins/streaming/impls/jms/src/main/java/org/pentaho/di/trans/step/jms/ui/ConnectionForm.java
  plugins/streaming/impls/jms/src/main/java/org/pentaho/di/trans/step/jms/ui/JmsConsumerDialog.java
  plugins/streaming/impls/jms/src/main/java/org/pentaho/di/trans/step/jms/ui/JmsDialogSecurityLayout.java
  plugins/streaming/impls/jms/src/main/java/org/pentaho/di/trans/step/jms/ui/JmsProducerDialog.java
  plugins/streaming/impls/jms/src/main/resources/org/pentaho/di/trans/step/jms/messages/messages_en_US.properties
  ui/src/main/java/org/pentaho/di/ui/core/widget/CheckBoxTableCombo.java

Unit Test Coverage
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.

:warning: Coverage Changes (click to expand)
:new: org.pentaho.di.ui.core.widget.CheckBoxTableCombo

Coverage: 0% :exclamation:

org.pentaho.di.trans.step.jms.context.WebsphereMQProvider

Branch Change: -41.7% :small_red_triangle_down:
Complexity Change: -29.7% :small_red_triangle_down:
Instruction Change: -19.4% :small_red_triangle_down:
Line Change: -13.1% :small_red_triangle_down:

:new: org.pentaho.di.trans.step.jms.ui.JmsDialogSecurityLayout

Coverage: 0% :exclamation:

License header violations

:heavy_exclamation_mark: Couldn't find acceptable copyright on ui/src/main/java/org/pentaho/di/ui/core/widget/CheckBoxTableCombo.java

:heavy_exclamation_mark: Couldn't find acceptable license on ui/src/main/java/org/pentaho/di/ui/core/widget/CheckBoxTableCombo.java

Checkstyle violations
The code submitted is not complying with one or more of our guidelines. To perform these checks locally, please use this tool.
/home/buildguy/project/plugins/streaming/impls/jms/src/main/java/org/pentaho/di/trans/step/jms/ui/ConnectionForm.java

:heavy_exclamation_mark: Unused import - org.pentaho.di.ui.core.widget.PasswordTextVar. at Line: 41 Column: 8

/home/buildguy/project/ui/src/main/java/org/pentaho/di/ui/core/widget/CheckBoxTableCombo.java

:heavy_exclamation_mark: ',' is not followed by whitespace. at Line: 174 Column: 54

Build Completed
:x: This pull request has errors. They will need to be addressed before it can be accepted. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
mvn -B -fn -f 'pom.xml' -pl 'ui,plugins/streaming/impls/jms' -P '!assemblies' -Dsurefire.runOrder=alphabetical -Daudit -amd clean install

Cleanup Commands
mvn -B -f 'pom.xml' -pl 'ui,plugins/streaming/impls/jms' -P '!assemblies' -amd build-helper:remove-project-artifact

Changed files
  plugins/streaming/impls/jms/src/main/java/org/pentaho/di/trans/step/jms/JmsDelegate.java
  plugins/streaming/impls/jms/src/main/java/org/pentaho/di/trans/step/jms/context/WebsphereMQProvider.java
  plugins/streaming/impls/jms/src/main/java/org/pentaho/di/trans/step/jms/ui/ConnectionForm.java
  plugins/streaming/impls/jms/src/main/java/org/pentaho/di/trans/step/jms/ui/JmsConsumerDialog.java
  plugins/streaming/impls/jms/src/main/java/org/pentaho/di/trans/step/jms/ui/JmsDialogSecurityLayout.java
  plugins/streaming/impls/jms/src/main/java/org/pentaho/di/trans/step/jms/ui/JmsProducerDialog.java
  plugins/streaming/impls/jms/src/main/resources/org/pentaho/di/trans/step/jms/messages/messages_en_US.properties
  ui/src/main/java/org/pentaho/di/ui/core/widget/CheckBoxTableCombo.java

Unit Test Coverage
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.

:warning: Coverage Changes (click to expand)
org.pentaho.di.trans.steps.salesforceupdate.SalesforceUpdateMeta

Instruction Change: -0.2% :small_red_triangle_down:

:new: org.pentaho.di.ui.core.widget.CheckBoxTableCombo

Coverage: 0% :exclamation:

org.pentaho.di.trans.step.jms.context.WebsphereMQProvider

Branch Change: -41.7% :small_red_triangle_down:
Complexity Change: -29.7% :small_red_triangle_down:
Instruction Change: -19.4% :small_red_triangle_down:
Line Change: -13.1% :small_red_triangle_down:

:new: org.pentaho.di.trans.step.jms.ui.JmsDialogSecurityLayout

Coverage: 0% :exclamation:

Checkstyle violations
The code submitted is not complying with one or more of our guidelines. To perform these checks locally, please use this tool.
/home/buildguy/project/ui/src/main/java/org/pentaho/di/ui/core/widget/CheckBoxTableCombo.java

:heavy_exclamation_mark: ',' is not followed by whitespace. at Line: 196 Column: 54

Build Completed
:fire: This pull request has some issues. It would be preferable to fix them in order for it to be just perfect. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
mvn -B -fn -f 'pom.xml' -pl 'ui,plugins/streaming/impls/jms' -P '!assemblies' -Dsurefire.runOrder=alphabetical -Daudit -amd clean install

Cleanup Commands
mvn -B -f 'pom.xml' -pl 'ui,plugins/streaming/impls/jms' -P '!assemblies' -amd build-helper:remove-project-artifact

Changed files
  plugins/streaming/impls/jms/src/main/java/org/pentaho/di/trans/step/jms/JmsDelegate.java
  plugins/streaming/impls/jms/src/main/java/org/pentaho/di/trans/step/jms/context/WebsphereMQProvider.java
  plugins/streaming/impls/jms/src/main/java/org/pentaho/di/trans/step/jms/ui/ConnectionForm.java
  plugins/streaming/impls/jms/src/main/java/org/pentaho/di/trans/step/jms/ui/JmsConsumerDialog.java
  plugins/streaming/impls/jms/src/main/java/org/pentaho/di/trans/step/jms/ui/JmsDialogSecurityLayout.java
  plugins/streaming/impls/jms/src/main/java/org/pentaho/di/trans/step/jms/ui/JmsProducerDialog.java
  plugins/streaming/impls/jms/src/main/resources/org/pentaho/di/trans/step/jms/messages/messages_en_US.properties
  ui/src/main/java/org/pentaho/di/ui/core/widget/CheckBoxTableCombo.java

Unit Test Coverage
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.

:warning: Coverage Changes (click to expand)
:new: org.pentaho.di.ui.core.widget.CheckBoxTableCombo

Coverage: 0% :exclamation:

org.pentaho.di.trans.step.jms.context.WebsphereMQProvider

Branch Change: -41.7% :small_red_triangle_down:
Complexity Change: -29.7% :small_red_triangle_down:
Instruction Change: -19.4% :small_red_triangle_down:
Line Change: -13.1% :small_red_triangle_down:

:new: org.pentaho.di.trans.step.jms.ui.JmsDialogSecurityLayout

Coverage: 0% :exclamation:


Compounds complain about invalid config names, even though they are valid

Open vscode launch.json see warning about non-existent launch configurations
They exist, there should be no warning

Good catch @joaomoreno
Can you please verify?


Improve how commands are internally executed

Use const CommandContext for execution
Update error handling
Fix SonarQube issues
Remove duplicate code
Use mutex instead of atomic_bool

@akuker Less changes this time ;-).
I have not managed to add @dialtr to the list of reviewers. Any idea why? I thought anybody could be added.


MNT: The cycler recipe has some lint :(

This is the friendly conda-forge-admin automated user.
I've ran the conda-smithy linter and found some lint in this feedstock :cry:.
Here is what I have got:

The top level meta keys are in an unexpected order. Expecting ['package', 'source', 'build', 'requirements', 'test', 'about', 'extra'].
When defining a source/url please add a sha256, sha1 or md5 checksum (sha256 preferably).

Thanks!

We can also inline the build/script nowadays so that the bld.bat and build.sh can be removed.

Just a friendly reminder that this is still a thing. 😄

Thoughts on addressing? 😉


Unable to set a specific solc compiler version

Component
Forge
Have you ensured that all of these are up to date?

[X] Foundry
[X] Foundryup

What version of Foundry are you on?
forge 0.2.0 (551bcb5 2024-02-28T07:40:42.782478000Z)
What command(s) is the bug in?
forge build / forge test
Operating System
macOS (Apple Silicon)
Describe the bug
Despite declaring a specific solc version in my foundry.toml, when I run forge build or forge test it will return an error saying that there is no solc version matching the version I have declared (0.8.26).
My foundry.toml looks like this:
[profile.default]
src = "src"
out = "out"
libs = ["lib"]
solc = "0.8.26"
auto_detect_solc = true

# See more config options https://github.com/foundry-rs/foundry/blob/master/crates/config/README.md#all-options

And the error I receive after forge build is the following:
[⠃] Compiling...2024-07-27T11:30:28.131852Z ERROR foundry_compilers::resolver: failed to resolve versions                                                                                │
│[⠊] Compiling...                                                                                │
│Error:                                                                                │
│Encountered invalid solc version in src/Example.sol: No solc version exists that matches the version requirement: =0.8.26                                                                    │

In addition to the above, I've also tried to run the command passing the --use flag to enforce solc version, but it won't solve the issue.
forge build --use solc:0.8.26
-> Error: Unknown version provided

forge 0.2.0 (551bcb5 2024-02-28T07:40:42.782478000Z)

could you please update your forge version? 0.8.26 didn't exist then

forge 0.2.0 (551bcb5 2024-02-28T07:40:42.782478000Z)

could you please update your forge version? 0.8.26 didn't exist then

my bad, I was doing forge update instead of foundryup
by the way, another compile error pops up now, which is related to the latest Solidity version (0.8.26):
[⠊] Compiling...                                                                                │
  33     │[⠰] Compiling 26 files with Solc 0.8.26                                                                                │
 34 cont│[⠔] Solc 0.8.26 finished in 1.36s                                                                                │ but no r
  35     │Error:                                                                                │
  36     │Compiler run failed:                                                                                │
  37     │Error: Unimplemented feature (/solidity/libsolidity/codegen/ExpressionCompiler.cpp:1266):Require with a cu                                                                                │
  38     │stom error is only available using the via-ir pipeline.                                                                                │
  39     │UnimplementedFeatureError: Require with a custom error is only available using the via-ir pipeline.

This has to do with the newly implemented syntax for require statements, namely:
require(sent, TransferError())

pls check if you're using via-ir pipeline, require with custom err is available only for such https://soliditylang.org/blog/2024/05/21/solidity-0.8.26-release-announcement/

Solved by updating foundry to latest version and adding the --via-ir flag to the forge test command:
forge test --mt testBlaBlaBla -vvv --via-ir


terraform: exit 0 on non-hardened environments

Signed-off-by: Steven Danna<EMAIL_ADDRESS>

@srenatus I like that suggestion, I left it as-is since it matches a pattern used elsewhere.  Perhaps we can follow up by fixing them all at once.


Placeholder for expiration group

Please add placeholder %luckperms_highest_group_by_weight_expiration% or %luckperms_exipration_group%
So that "VIP" players know when their best VIP ends. + so that it is possible to somehow set the expiration format eg 1w 1d 1h to another language for example or just limit the format to 1w / 1d only, ... From what I've looked at placeholders, none of that is what I imagine. Just to show you the expiration time of the highest group you have, and if it doesn't find your expiration time, it will tell you that you have that group forever.

+ so that it is possible to somehow set the expiration format eg 1w 1d 1h to another language for example or just limit the format to 1w / 1d only, ... From what I've looked at placeholders, none of that is what I imagine.

Maybe add a setting to decide how to view the time remaining formatted?
Example config from PlaceholderAPI config.yml:

  luckperms:
    time:
      weeks: w
      days: d
      hours: h
     minutes: m
     seconds: s

Display the time in some measure such as seconds and let each admin format it as they see fit.
Example in seconds without breaking compatibility with users who are currently using the placeholder:

raw (new placeholder): %luckperms_group_expiry_time_<group>_raw% = 86399
formatted (current form): %luckperms_group_expiry_time_<group>% = 23h 59m 59s
So we can display the value in several languages, not everyone wants to show the time in the USA standard.

@SrBedrock You're by far not the only one asking for this so I took it into my own hands to make something to help with this. If you don't mind using a javascript, perhaps give this one a try https://github.com/ZeromaniacMc/LuckPerms-Time-Placeholder-Script

But I need the highest group the player has. Not that I have to enter a group in the placeholder.


data placement structure of the dataset?

Dear author
what is the data placement structure of the dataset?
such as seamantic kitti, like,  are there images and lidar data under sequence00?

For convince, we use the same structure as SemanticKITTI (original 360 one)

For convince, we use the same structure as SemanticKITTI (original 360 one)

Dear author,
How can I set the "nuscenes" dataset structure like the Semantic KITTI dataset?
LIKE scene-0000 ,scene-0001, ...?
Is there any script?
Really thanks.

Hmm, it's kind of hard to set nuScenes dataset structure like the SemanticKITTI dataset, because nuScenes is based on the token to sample data.


It will be great to have Phone and pincode validation

It will be great to have Phone and pincode validation.... Thanks for the awesome library

What do you mean by pincode validation? Could you use @NotEmpty @Digits?
For phone numbers you could use @Pattern with a regex to match a phone number in the format you're expecting.

There is no standard format for telephone numbers and pin codes (widely known as zip code in other countries). Both of these differ from country to country. @hardikamal You could use the annotations just like @inktomi mentioned or you could also create your own custom annotations for reusability. I also should bring to your attention, the lesser know Saripaar X project for country specific annotations. You can send pull requests for region-specific annotations.


Adding Alert Box

Describe the solution you'd like
I would like to add a alert box just below the navbar saying something like. "Good Landing! Create a profile to get started". Will provide functionality that it will appear for only 5 seconds and then will disappear.
Please assign this to me. Thankyou!

Somewhat like this below the navbar. Please assign this to me. Thankyou!

@vanshika2906  Assigned :)
Extremely sorry for the late response. You can add whatever backend as well as stuff left. Will get back to in few hours with update from my side.


my live servers arn't displaying live or showing any changes.

Environment (please complete the following information):

OS (e.g., Windows 10):
Extension version (e.g., 1.2.3):

Describe the bug:

Repro steps:

Expected behavior:

Additional context:

Thanks @HpFontenot1993, could you please describe in more details the issue that you are facing?

Hello,
The problem has sense then been fixed. I uninstalled the extension on vs
code. At the time it was showing on my tasks that the window was being
suspended. Causing the functions not to work. So, I just reinstalled the
extension. Thank you.
Hpf
On Mon, Jan 2, 2023, 2:20 AM Patrick Brosset @.***>
wrote:

Thanks @HpFontenot1993 https://github.com/HpFontenot1993, could you
please describe in more details the issue that you are facing?
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode-edge-devtools/issues/1305#issuecomment-1368770392,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A344VK2D5FX2FM26FILWHNTWQKMVXANCNFSM6AAAAAATFIZ5S4
.
You are receiving this because you were mentioned.Message ID:
@.***>


Number sorting with a twist

Feature ✅
Description
The task is to sort n numbers in ascending format based on following conditions:

ith number can be exchanged only with i-1th position number.
Complete operation can be performed in (n(n-1)/2 -1) exchanges.
If both conditions are met print "YES" else print "NO"

Example
Input: 9 7 1 12 8
Output: YES
Input: 2 1
Output: NO

@pratik-choudhari I would like to contribute. I will be using java.


Correct Svg.Benchmark namespaces and cleanup csproj

Reference Issue

Split from #786
What does this implement/fix? Explain your changes.

Correct Svg.Benchmark namespaces and cleanup csproj
Any other comments?

@mrbean-bremen Can we merge this too, it's just cleanup PR

@mrbean-bremen Can we merge this too, it's just cleanup PR


Add metrics around IOHandle times

Moved to https://issues.apache.org/jira/browse/HUDI-62

Hi, This task has been unassigned. I would like to pick this up. Could you please help me out here? Where to start

@chandresh-pancholi if you give us your jira id, I can assign HUDI-62 to you. is that something you are interested in?


Poetry builds wheel package with wrong version

[x] I am on the latest Poetry version.
[x] I have searched the issues of this repo and believe that this is not a duplicate.
[x] If an exception occurs when executing a command, I executed it again in debug mode (-vvv option).

OS version and name: Debian GNU/Linux 9
Poetry version: 0.12.15
Link of a Gist with the contents of your pyproject.toml file: https://gist.github.com/mshenhera/ced9c3f98d494eb435552eec5b7b3d00

Issue
When package version has local version identifiers (https://www.python.org/dev/peps/pep-0440/#local-version-identifiers) for example 0.1.2+77cc516 poetry creates wheel package with wrong version: replaces "+" by "_". Steps to reproduce the problem:
# Install poetry, jq, wheel-inspect
# Clone simple python project
git clone https://github.com/pypa/sampleproject.git
cd sampleproject
# Create pyproject.toml  from github gist
curl -o pyproject.toml https://gist.githubusercontent.com/mshenhera/ced9c3f98d494eb435552eec5b7b3d00/raw/c33b8107773ca549fab72255527440f4492617e8/gistfile1.txt
# Build wheel package
~/.poetry/bin/poetry build -f wheel
# Inspect the package
wheel2json dist/sampleproject-0.1.2_77cc516-py3-none-any.whl  | jq .version
"0.1.2_77cc516"

# Build wheel package with setutools to show correct behavior:
# Change version in setup.py to something like "0.1.3+77cc516"
python setup.py bdist_wheel
wheel2json dist/sampleproject-0.1.3+77cc516-py2.py3-none-any.whl | jq .version
"0.1.3+77cc516"

As result of wrong version generation it is not possible to install package after uploading it to pypi repo. Uploads works fine. However Install fails with error:
ERROR: Could not find a version that satisfies the requirement sampleproject==0.1.2_77cc516

poetry generates poetry.lock with content like this one:
[[package]]
category = "main"
description = "A sample Python project"
name = "sampleproject"
optional = false
python-versions = "*"
version = "0.1.20_dc43e19"

After changed version in poetry.lock manually from 0.1.20_dc43e19 to 0.1.20+dc43e19 I was able to install the package.

Can you please provide the pyproject.toml for the project that is trying to use sampleproject?
In PEP 427 it states (regarding wheel filenames):

Each component of the filename is escaped by replacing runs of non-alphanumeric characters with an underscore _:
re.sub("[^\w\d.]+", "_", distribution, re.UNICODE)

So Poetry is doing the right thing per the PEP. But in the pyproject.toml you must still specify the version like <public version>+<local version>.

pyproject.toml: https://gist.github.com/mshenhera/ced9c3f98d494eb435552eec5b7b3d00.
PEP 427 specifies filename format that does nothing with version, I guess. And there is no problem with filename as I can see. Poetry can install it without any issue as soon as version is manually corrected in poetry.lock.

This issue is related to #756 (that goes on the file name causing issues).
@chrahunt I do find PEP 427 interesting though, hadn't seen that when implementing PR (#950) to fix #756.
As can be seen from @mshenhera's post, the filename setuptools uses contains +.

~/.poetry/bin/poetry build -f wheel
...
wheel2json dist/sampleproject-0.1.2_77cc516-py3-none-any.whl
...
python setup.py bdist_wheel
wheel2json dist/sampleproject-0.1.3+77cc516-py2.py3-none-any.whl
...

Given that the current behavior breaks multiple implementations (as mentioned in #756), I think it makes sense to do as the Romans do - i.e. produce same result as setuptools.

Ok, looking at how wheel-inspect extracts version from filename, not the wheel itself (wheel_inspect/filename.py#L13), I believe this issue is a duplicate of #756.
The discussion here is interesting though, and a decision on how to go about filenames should be taken.


Changed placeholder for better understanding

Placeholder changed to "e.g. go for a walk"

I kindly request you to please merge my PR🥲, as some of my PR got excluded for HF.


ci(*): update publish/deploy to use ghcr.io; untrack static/data.json

Updates publish/deploy logic to use an OCI registry (currently: docker.io). In doing so, we generate the search index (static/data.json) at time of publishing, removing the need to track in version control. (Ref https://github.com/fermyon/developer/issues/369).
The first choice was ghcr.io, however, in my testing I nearly always encountered rate limits when attempting to push this developer website app (for example: https://github.com/vdice/developer/actions/runs/4892021649/jobs/8733252133).  I can create a follow-up to revisit.
Successful deploy in https://github.com/fermyon/developer/actions/runs/4892766363/jobs/8734931145.  See live version here: https://canary.developer.fermyon.com.  You can also run the website locally from a sample tag pushed from this branch, eg spin up -f fermyon/developer:vdice-oci-deploy-d17b043d0884dab9fc2197709cd3035bd4fe4667
TODO:

[ ] reviews/feedback
[ ] address remaining TODOs in deploy.yml prior to merge

Supersedes https://github.com/fermyon/developer/pull/423
Closes https://github.com/fermyon/developer/issues/369
Pre-Merge Content Checklist
This documentation has been checked to ensure that:

[ ] The title, template, and date` are all set
[ ] Does this PR have a new menu item (anywhere in templates/*.hbs files) that points to a document .md that is set to publish in the future? If so please only publish the .md and .hbs changes in real-time (otherwise there will be a menu item pointing to a .md file that does not exist)
[ ] File does not use CRLF, but uses plain LF (hint: use cat -ve <filename> | grep '^M' | wc -l and expect 0 as a result)
[ ] Has passed bart check
[ ] Has run npm run build-index
[ ] Has been manually tested by running in Spin/Bartholomew (hint: use PREVIEW_MODE=1 and run npm run styles to update styling)
[ ] Headings are using Title Case
[ ] Code blocks have the programming language set to properly highlight syntax and the proper copy directive
[ ] Have tested with npm run test and resolved all errors

One item to note on the Nomad side.  With the updated website job (now running spin up -f <oci ref>), CPU spikes quite high when starting up, I assume from the high number of concurrent downloads when pulling the app/image.  Once running, CPU usage drops back down to the usual ~10 MHz idle baseline.

Note: Build failure currently tracked in https://github.com/fermyon/developer/issues/590

This looks great @vdice. The search index seems to be working as expected. I will wait to approve until we get all the todos cleaned and fixed but it looks good to me!

This is ready for a final review @karthik2804 @rajatjindal.  Also note https://github.com/fermyon/developer/pull/588#discussion_r1187518287.  Thanks!


Publishing assets via Composer

Hi! Thanks for awesome Admin control panel theme!
I wrote little script for publishing assets via Composer https://github.com/kamilsk/Common/blob/master/docs/TipsAndTricks.md#adminlte-assets-publishing. @almasaeed2010, can I integrate the script in this repository? Or can you add a link to it?

404 the link is not found.

@Tjoosten, sorry about that, I moved it to the wiki: https://github.com/kamilsk/Common/wiki/Tips-and-tricks#adminlte-assets-publishing

and refactored for PHP 7 version https://github.com/kamilsk/Common/blob/3.x/src/Composer/Script/AdminLte/Publisher.php

example of usage is located here https://github.com/kamilsk/TinkoffCurrencyAnalyst


docs(readme): fix link to example app

Hey! The latest changes look cool 🎉
I just fixed a small typo in the readme, and I take this opportunity to ask you a question
Can we expect to have a changelog for each release, or at least the list of breaking changes for the major versions ? I'm about to migrate to version 1.0.0 and I think I'll have to dig a bit into the code to do my migration. Not a big deal, but in the future it would be really cool to have this 😋
Thank you for your work!

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

hi @Julien-R44,
thanks for the fix!
what we will surely provide is a release announcement under https://github.com/livekit/livekit-react/releases with mentions of changes from previous to latest version.
If you kept up to date with the previous releases so far, then the changes required for 1.0 should be minimal to nonexistent.
Basically @livekit/react-components can work as a drop-in replacement for livekit-react. Note that the package name livekit-react will be deprecated soon on npm.
There will also be a blog post soon outlining the reasoning for the new repo structure and explaining when to use what!


Clean state command

An inline command to reset state of the component when the value changes. Similar to what React does with key property

:bind directive does exactly that


require the entire s3 prefix from users

This is not a completely tested edit, and you couldn't accept it without breaking existing uses.
PivNet is now using a bastion s3 bucket for uploading the bits, and they internally move them across to the pivnet s3 bucket after the release is created.
The new bucket has the prefix product-files/your-slug/
I needed to use this new format, so we provide our s3_filename_prefix to the pivnet-resource as product-files/application-watchdog-for-pcf and it works beautifully with this edit.
@crawsible paired with me so can also provide context if you need it.

@squeedee thanks for bringing to our attention! We hadn't yet heard from pivnet about this change. Chatting with @timlombardo and @crawsible we will need to implement this to allow teams to move over to the new product-files path.
I think it's important that we make this change in a backwards-compatible manner, however, as not all teams will move over at once, and we don't want to force everyone to pin to an old version until they're moved over to the new path.
We already had a story to capture making the product_files optional - this has been prioritized and will also allow you to provide product-files instead of product_files.
If you want to work through this PR to support backwards compatibility, that would be great! But if you'd rather not, that's fine too, we'll close this issue and work off the existing story. Either is fine for us.

@squeedee new release v0.21.0 should support your use-case, so I'll close this PR. Please let us know if you encounter issues when using a s3_filepath_prefix beginning with product-files.


Description of Algorithm

Hi,
We are using optuna for a research.  Could you describe the algorithm or better, show us a reference to a paper?  We know you are using Bayesian optimization but IIRC there are many methods which fall under the term of Bayesian.

Hi @yoriyuki,
Thank you for using Optuna!
Optuna uses Tree-structured Parzen Estimator (TPE) as a default sampling algorithm. Please take a look at the following paper for further details of TPE.
James S. Bergstra, Rémi Bardenet,  Yoshua Bengio, Balázs Kégl: Algorithms for Hyper-Parameter Optimization, in Proceedings of Advances in Neural Information Processing Systems 24 (NIPS 2011), BibTeX


pnpm to version 9

#962
#963  👈
master

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

Join @steveluscher and the rest of your teammates on  Graphite


Where to add directives

I've looked a little bit into the headroom.js issue (http://wicky.nillia.ms/headroom.js/) and so far, have been unsuccessful. Looking at the instructions, we have to add headroom as a module.
I believe you talked about the ui-router in the last meeting, but have not found where to actually add the code. (for example: angular.module('headroom');) Tried to add the code in the ui-router files but it didn't work, sadly.
Any ideas?
Will take another look at it after dinner.
Thanks!

Hi Kevin,
Apologies for the delay. Let's work on this during the conference call. Also, Shin will be here at 2pm with computer so he will be joining us.
The README.md update looks good. I remember you had to copy the scripts/mecab-config to your PATH somewhere (usr/bin/path) and install mecab-ipadic from source. Can you add a note about that as well?
Thanks,
Sam

Okay, will add that!

Hey Kevin, I fixed the basic sorting for the stories, can you duplicate the functionality for Articles and figure out if there is a better way to do this? Thanks, Sam


Use Poetry 1.2

Update to Poetry 1.2
Use new plugin mechanism for installing the dynamic-versioning-plugin
Use poetry publish instead of twine

@darthtrevino , I'm not sure how to judge this PR.  On the one hand, fixing the causalml install is good.  On the other hand, relying on a private fork for very long doesn't seem good.  How should we think about this?

This is a stopgap until this PR is merged. I agree that I don't want to maintain a fork as a long-term solution

If causalml takes that PR, then that will trigger us to remove this dependency on the fork and go back to causalml.  So, I see that as one possible "way out" of depending on your fork.
What if causalml doesn't take the PR?  What will be the trigger for us to revisit this situation?  Should we want to set a timeline --- like saying that if we cannot resolve this in 45 days, we'll remove the dependency entirely, or take some other action to avoid a long-term dependency on your forked version of causalml?
It seems easier to decide this now.  Otherwise, I worry we will end up maintaining a fork for much much longer than we intend.

I pinged the CausalML team, and they accepted the patch. I'll update our pyproject.toml to point back at their repo.


nixos: Deprecate passing pkgs to specialArgs and lib.nixosSystem

Description of changes
passing pkgs to specialArgs and lib.nixosSystem is a common anti-pattern that new users fall into which causes unexpected behavior and complete ignorance to the nixpkgs. option set
Since readOnlyPkgs has merged there's a easy immutable way to set pkgs
I think it's time to do away with alternative handling methods of pkgs
opinions are appreciated, thanks

TODO: add documentation

I mostly agree with you roberth
Mainly put this out there to get opinions

i'll make a similar PR later


Add transitive overrides and npm update

Resolve open security alerts from transitive dependencies.  Build, docs and tests all pass locally showing no breaking change to functionality used.

Coverage drop will be fixed in a later PR.  I believe this used to be a flakey test but it is now more consistently failing.


Neovim cursor shape does not change on insert mode

[x] I am on the latest Hyper.app version
[x] I have searched the issues of this repo and believe that this is not a duplicate

Debian WSL on Windows 10:
2.0.0-canary.14:
The issue is reproducible in vanilla Hyper.app:

Issue

This is an old  issue #270, that was supposedly fixed by #2437, but that was under hterm I believe.  The cursor remains a block regardless of mode in Neovim.  Not sure if the change to xterm has caused a regression here or I am missing something.

I'm using Ubuntu and have the same problem.

I am using Hyper 3.0.2 on MacOS 10.13.6 with Vim 8.1.950 and I have this issue. I have the below set in my .vimrc. I can change the cursor shape in hyper.js, but it will not change as it should in an xterm terminal.
let &t_SI = "\<Esc>[6 q"
let &t_SR = "\<Esc>[4 q"
let &t_EI = "\<Esc>[2 q"

When I tested Neovim, the cursor was changing correctly.


[5.2] Use Arr::exists and Arr::accessible in the data_get helper

Now that we have Arr::exists and Arr::accessible, we don't need a special case for ArrayAccess.

Do we have decent test coverage here to verify this is 100% good?

@grahamcampbell it's the exact same thing: https://github.com/laravel/framework/pull/6571#issuecomment-67077076

k ;)


Changed mapcar to mapc in execute-parallel function of the spectral-norm codes to reduce consing

Thanks


gRPC ebook Chapter 7

gRPC In Production
Covers self-hosted, Docker, Kubernetes, service mesh, load balancing and APM.

Just the final section on Application Performance Management (APM) to write, will include:

Healthchecks
Logging
Metrics
Tracing

Regarding the Service mesh section: I used Linkerd to create the example screenshot, and it's really easy to add it to Kubernetes and enable in the service. I could cover it in a single paragraph. Istio, on the other hand, is a lot more complicated to set up and more complicated to use. And I haven't even tried Consul.
Q: Should I include an example using Linkerd, but not Istio or Consul? Or leave examples out altogether and direct readers to the Getting Started section for each of the options?
cc @nishanil @mairaw

@markrendle, IMO, Linkerd example is fine to support the service mesh explanation.

Ready for review. cc @glennc @saurabh500 @mconnew @JamesNK @JunTaoLuo @mairaw


notSent as input name

https://stackoverflow.com/questions/3008035/stop-an-input-field-in-a-form-from-being-submitted/61648470#61648470
I was wondering whats the suport for notSent  when you put it in input like from this link ?

Looks like that example is pointing out that the input field inside the nested <form> element won't be sent. The "name" value can be anything and is defined there as "notSent" simply as an example.

Looks like that example is pointing out that the input field inside the nested <form> element won't be sent. The "name" value can be anything and is defined there as "notSent" simply as an example.


Sign packages

Hello, @dang03
Can you point me or write here what's is needed to sign packages?
Thanks,

Hello @jbonnet,
When a signed package is submitted to the GK API, the extra data it will include is the signature hash.
It will be included as a header:
headers['signature'] = signature
So the GK API will receive:
Headers:
{'Authorization': "Bearer %s" % access_token}
{'signature': signature}

Body:
{'package': pkg_file}

Ok, @dang03, great!
Is son-access already sending this header?

@jbonnet
Yes, it is already implemented. Remember that signing is an optional argument in the son-access, the GK API won't always receive the signature.

@dang03
And what's the signing algorithm to be applied on the GK's side?

@jbonnet
In the GK side, it should be applied the Signature Verification, however, as mentioned in Paris F2F, this functionality will be inside the son-validate, which will be called by the GK.
The GK should get the received signature, the User's public key from the UM, and the hash value of the original package file OR the package file itself, and send them to the validation function.
How the Verification works:
The receiving side calculates the hash value of the package file (binaries) and then uses the User's public key with the verify() method to validate its origin:
Using Python (son-validate) Crypto lib
from Crypto.Hash import SHA256
from Crypto.PublicKey import RSA

def verify(son-package, public_key, signature):
  hash = SHA256.new(son-package).digest()
  result = public_key.verify(hash, signature)

It returns True or False

Closed, signing is done by son-validate


바텀 시트 최대폭 수정

🤔 해결하려는 문제가 무엇인가요?
close #228
###🎉 변경 사항

bottom sheet의 최대 폭을 모바일 기준으로 설정하고, 중앙으로 배치했습니다.

Codecov Report
Patch coverage: 100.00% and project coverage change: +0.02 :tada:

Comparison is base (03e7c7c) 91.90% compared to head (8483553) 91.92%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #234      +/-   ##
==========================================
+ Coverage   91.90%   91.92%   +0.02%
==========================================
  Files          38       38
  Lines         284      285       +1
  Branches       52       52
==========================================
+ Hits          261      262       +1
  Misses         23       23

Impacted Files
Coverage Δ

src/components/bottomSheet/BottomSheet.tsx
100.00% <100.00%> (ø)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.