App crash when width/height are too bigger in encodeBarcode()

Hello :)
I want to generate a barcode in aztec format.
So I use the encodeBarcode() method, to which I send the data and set the parameters.
The default width/height (120px) doesn't work for me, so I increase it.
class BarcodeGeneratorService implements BarcodeGeneratorInterface {
  final Zxing _barcodeGenerator;

  BarcodeGeneratorService({Zxing? flutterZxing}) : _barcodeGenerator = flutterZxing ?? Zxing();
  @override
  Uint8List? encode({
    required String textToEncode,
    required int format,
    required int width,
    required int height,
    required int margin,
  }) {
    final encoded = _barcodeGenerator.encodeBarcode(
        contents: textToEncode,
        params: EncodeParams(format: Format.aztec,  width: 150, height: 150 , eccLevel: EccLevel.high));

    if (!encoded.isValid || encoded.data == null) {
      return null;
    }
    final imglib.Image img = imglib.Image.fromBytes(
      150,
      150,
      encoded.data!,
    );
    return Uint8List.fromList(
      imglib.encodeJpg(img),
    );
  }
}

When I want to display my barcode, when the encodeBarcode() method is triggered, my application crashes at this point.
The crash occurs on several physical iOS devices, in 16 and 16.5.
On the other hand, everything works fine on simulator.
The error is not very clear:

Do you have any idea what it could be?
Thanks :)

I got the same Problem when I try to encode a QR-Code. For me it works in the Debug build when I set the width&height to 128 in the Debug build, but on the release build the App crashes.
Luckily I have FirebaseCrashlytics configured.
Please take a look:
Fatal Exception: java.lang.IllegalArgumentException: CaptureRequest contains unconfigured Input/Output Surface!
       at android.hardware.camera2.CaptureRequest.convertSurfaceToStreamId(CaptureRequest.java:738)
       at android.hardware.camera2.impl.CameraDeviceImpl.submitCaptureRequest(CameraDeviceImpl.java:1187)
       at android.hardware.camera2.impl.CameraDeviceImpl.setRepeatingRequest(CameraDeviceImpl.java:1235)
       at android.hardware.camera2.impl.CameraCaptureSessionImpl.setRepeatingRequest(CameraCaptureSessionImpl.java:312)
       at z5.q.g0(:33)
       at z5.q$b.onConfigured(:41)
       at android.hardware.camera2.impl.CallbackProxies$SessionStateCallbackProxy.lambda$onConfigured$0$CallbackProxies$SessionStateCallbackProxy(CallbackProxies.java:53)
       at android.hardware.camera2.impl.-$$Lambda$CallbackProxies$SessionStateCallbackProxy$soW0qC12Osypoky6AfL3P2-TeDw.run(:4)
       at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1167)
       at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:641)
       at java.lang.Thread.run(Thread.java:923)


This project is awesome, thought I might contribute a pretty readme

Love the work you've done on this, but the instructions weren't particularly noob friendly. Thought I might contribute a pretty readme. Keep up the awesome work!

That's a much more comprehensive readme for sure, very nicely done. Thanks for the pull request!


Clarify mocks versus test mocks

"Mocks" and "examples" should not expose dangerous features by default.
For example, superuser burning.

#217 Is this sufficient?

@fulldecent I also added a readme update.


Update react and react-dom peerDependencies

Update both react and react-dom to 16.7.0
Fixes #32

@abhijitparida so sorry about the delay but I didn't notice the PR at all!
Thanks for opening this, I just edited your change to include React 15 as it's still supported and some people haven't updated to React 16.


#547 Dashboard UI Components: Button & storybook foundations

Acceptance Criteria
Overview

Button component for dashboard use. Has primary, secondary, cta variations.
Storybook for this button also means some boilerplate to set up storybook on the dashboard side of stories.
"@storybook/addon-actions": "^5.2.8" package added to simulate actions in storybook.
Constants for Dashboard started with button variations (primary and secondary).
Theme adds colors, button font, text styling for reuse and eliminates the h1 global font family.

Testing
Run npm run storybook and see that there's a new section of storybook added.

See that the existing stories editor sections are still loading appropriately
Navigate to the new Dashboard Button component (http://localhost:56811/?path=/story/dashboard-components-button--default)

See 3 button demos. Default, a "long" button that demos how the button works with overrides from a parent, and a secondary type button. You can use the knobs on any of these three to see how the props work.

@wassgha thanks for the review! I made a good chunk of updates, please take a look when you get a chance. Everything else that i just commented on I will do iteratively - carlos needs the foundation of this PR for what he's doing so i'd like to not delay it unnecessarily given the early stages of development we're in for the dashboard.


test: multires lyrics search

Add integration test for multires lyrics search example, with assertion on chunk-level

LGTM! I think we just need now to add it to the CI workflow!

Added it. Should be running now

LGTM! I think we just need now to add it to the CI workflow!

Added it. Should be running now
EDIT: And it passed partying_face

That is great. Thanks for the effort! We might want to update to jina 0.7.4 in a next iteration, but merge away for now.


Upgrade ESLint & TypeScript ESLint Plugins

FINALLY, after all those PRs, we can now upgrade ESLint. This PR fixes a few more tiny issues and add some linting exceptions where necessary.
In general, this newer version of ESLint + plugins is much stricter and works better parsing with the latest TypeScript version. We still have a few any types that should get fixed eventually, but this is a big, much needed upgrade.

Codecov Report

Merging #6710 into dev will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               dev     #6710   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           15        15
  Lines          671       671
=========================================
  Hits           671       671

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 e43cfe3...793f3a6. Read the comment docs.


content: change Writer/ReaderAt to take OCI descriptor

This change allows implementations to resolve the location of the actual
data using OCI descriptor fields such as MediaType.
No OCI descriptor field is written to the store.
No change on gRPC API.
~contrib/registrycontentstore is added as an example usecase of this.~ (EDIT: will be a separate PR)
Signed-off-by: Akihiro Suda<EMAIL_ADDRESS>
Close https://github.com/containerd/containerd/issues/2129
@tonistiigi @dmcgowan

Codecov Report

Merging #2135 into master will increase coverage by 0.13%.
The diff coverage is 44.04%.

@@            Coverage Diff             @@
##           master    #2135      +/-   ##
==========================================
+ Coverage   45.53%   45.66%   +0.13%
==========================================
  Files          80       81       +1
  Lines        8726     8794      +68
==========================================
+ Hits         3973     4016      +43
- Misses       4107     4121      +14
- Partials      646      657      +11

Flag
Coverage Δ

#linux
50.45% <51.38%> (+0.21%)
:arrow_up:

#windows
40.95% <13.09%> (-0.15%)
:arrow_down:

Impacted Files
Coverage Δ

content/content.go
0% <ø> (ø)
:arrow_up:

images/oci/importer.go
5.3% <0%> (ø)
:arrow_up:

images/oci/exporter.go
0% <0%> (ø)
:arrow_up:

content/helpers.go
29.76% <0%> (-2.71%)
:arrow_down:

content/local/store.go
52.25% <100%> (+1.45%)
:arrow_up:

...ntrib/registrycontentstore/registrycontentstore.go
43.33% <43.33%> (ø)

metadata/content.go
35.61% <77.77%> (+1.59%)
:arrow_up:

... and 1 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 002caba...d4a1dc8. Read the comment docs.

Can we split out the registry test and contib into a separate PR and focus on the interface change. Feel free to point at that work for justification.

@dmcgowan
updated. (bak: https://github.com/AkihiroSuda/containerd/releases/tag/oci-content-store.20180216)

@tonistiigi updated. I changed WriteBlob as well accordingly.

In general, this is a problematic change. The content store is supposed to be purely content addressed with no understanding of types. The resolver/fetcher layer is the system to use for handling type aware content operations.

The content store is supposed to be purely content addressed with no understanding of types.

That does not change. It doesn't change the store but the interface. Types are not stored and are not part of the interface guarantees. It is just a hint that an implementation can take. Data written with a specific digest will always return back for the same digest. It would be problematic if Provider returns back a type but it doesn't.

The resolver/fetcher layer is the system to use for handling type aware content operations.

That's the problem. There are duplicate stacks implemented atm that can't be used together for a lot of useful use-cases nor extended with new implementations. Code for copying content needs to be specific for the implementations and has a separate set of helper functions. This could be implemented with new fetcher/pusher interfaces that bring them to the same functionality of content.Provider/Writer and then removing all the use cases (I guess apart from that internal testcase use-case that you left a comment on) where the old types are currently being used.

Closing temporarily, until we can reach consensus on https://github.com/containerd/containerd/issues/2129

reopened PR
@stevvooe @tonistiigi @dmcgowan

rebased

@stevvooe PTAL?

rebased again

rererebased

re-LGTM, let's get this merged because rebasing vendors like that is not fun

LGTM

opened https://github.com/containerd/cri/pull/799


Implement uninstall command

Problem
We are able to install a package with crawl install -p <pkg> however uninstalling a package is a manual process.
Suggested solution
Implement the uninstall command to be used as crawl uninstall -p <pkg>

Implemented remove command under 0.5.0


Add icon for the threads social network

Import FontAwesome icon v6.4.2 for 'threads' and add it to list of all available icons on the exampleSite.

Thanks for this addition, @ned21, and thanks for updating all the docs pages too! I'll merge this one straight in.


Settings to turn this section off?

Is there any way I can go into the settings for powershell and just turn off this section.?

Create an environment variable POWERSHELL_UPDATECHECK and set it to Off.  See the docs on this.
In your Windws Terminal setting for Windows PowerShell, add the command line arg -NoLogo:

That should get rid of of the startup text.

Perfect - Thank you @rkeithhill


Create  inscriptions.json

closing pull request. reason: existing in /collections/bitcoin-legs-satoshi


Feature Request for parameters specified with :text, :varchar, :nvarchar, etc.

If I try the following:
var properties = XElement.Parse( """<Properties RBLFramework="Evolution"><Live Key="12359" ApprovedOn="2021-10-11T18:41:47.98"<EMAIL_ADDRESS>/></Properties>""" );

var cn = new SqlConnection(this.Connection.ConnectionString);
var qb = cn.QueryBuilder($"UPDATE [File] SET Properties = {properties:text} WHERE UID = {11670}" );
await qb.ExecuteAsync();

I get the following SqlException: Operand type clash: xml is incompatible with text.
To get around this, I can simply put {properties.ToString()} but of course the IDE dims out .ToString() saying I don't need it (since it usually would do a .ToString() during normal string interpolation.  So I have to make sure other users who don't know issue, don't take IDE suggestion of removing it.  Additionally, I have to wrap it with:
#pragma warning disable IDE0071 // Interpoliation can be simplified
...
#pragma warning restore IDE0071 // Interpoliation can be simplified

Maybe be far fetched request, but curious on your thoughts.  If a DB data type is provided and it is a 'string' type (text, ntext, char, varchar, nvarchar, etc.) you first do a .ToString() on the object before creating parameter?  I could take a stab and a PR if you agree and don't have time to implement.  I haven't looked at code base yet, but assume I'd find it pretty easily.

Please disregard my previous suggestion. I checked the code, and this is how explicitly-described string dbtypes are supposed to work:

In InterpolatedSqlParser.TransformArgument() we get the type-specifiers ([n][var]char(SIZE), and [n]text) and wrap the values into a StringParameterInfo. This is where we capture attributes like string length, ansi/unicode, etc. However this code only works if the provided argument is already a string type - your XElement is not a string type and therefore all specifiers (including string lengths) are just ignored and discarded at this point.
In SqlParameterWrapper.AddToDynamicParameters() we get those StringParameterInfo and map it back to Dapper. At this point they are certainly "string types", and therefore no need to change the mapping here.

So basically since your XElement is not previously converted to string (which would also be a quick solution, var properties = XElement.Parse("etc").ToString()), the "text" specifier is just being discarded, and I suppose Dapper passes your XElement as if it was a XML type but then it doesn't match what SQL expects for the field.
So I think the right solution is exactly what you suggested:

In TransformArgument() if we get the type-specifier for a string-type then we should wrap it under StringParameterInfo (like we do), but instead of expecting that the argument is already typeof(string) we should test and ToString() it if it's not.

Would you like to create a PR?

Digging into it.  I've usually just use vscode/tasks.json to do builds.  In attempting to run your build.ps1, I get the following errors:

C:\Program Files\dotnet\sdk\7.0.403\Sdks\Microsoft.NET.Sdk\targets\Microsoft.NET.TargetFrameworkInference.targets(92,5): error NETSDK1013: The TargetFramework
value '"netstandard2.0' was not recognized. It may be misspelled. If not, then the TargetFrameworkIdentifier and/or TargetFrameworkVersion properties must be s
pecified explicitly. [C:\BTR\OpenSource\InterpolatedSql\src\InterpolatedSql\InterpolatedSql.csproj::TargetFramework="netstandard2.0]

C:\Program Files\dotnet\sdk\7.0.403\Sdks\Microsoft.NET.Sdk\targets\Microsoft.NET.TargetFrameworkInference.targets(92,5): error NETSDK1013: The TargetFramework
value 'net7.0"' was not recognized. It may be misspelled. If not, then the TargetFrameworkIdentifier and/or TargetFrameworkVersion properties must be specified
explicitly. [C:\BTR\OpenSource\InterpolatedSql\src\InterpolatedSql\InterpolatedSql.csproj::TargetFramework=net7.0"]

Second error is definitely strange to me as I have .net 7.0 installed (it's current version we are using in my work project).
Then I removed netstandard2.0; and net7.0 and get:

C:\Program Files\dotnet\sdk\7.0.403\Sdks\Microsoft.NET.Sdk\targets\Microsoft.NET.TargetFrameworkInference.targets(92,5): error NETSDK1013: The TargetFramework
value '"net462' was not recognized. It may be misspelled. If not, then the TargetFrameworkIdentifier and/or TargetFrameworkVersion properties must be specified
explicitly. [C:\BTR\OpenSource\InterpolatedSql\src\InterpolatedSql\InterpolatedSql.csproj::TargetFramework="net462]

C:\Program Files\dotnet\sdk\7.0.403\Sdks\Microsoft.NET.Sdk\targets\Microsoft.NET.TargetFrameworkInference.targets(92,5): error NETSDK1013: The TargetFramework
value 'net6.0"' was not recognized. It may be misspelled. If not, then the TargetFrameworkIdentifier and/or TargetFrameworkVersion properties must be specified
explicitly. [C:\BTR\OpenSource\InterpolatedSql\src\InterpolatedSql\InterpolatedSql.csproj::TargetFramework=net6.0"]

So I don't think I have all the .net legacy/current versions installed to support using build.ps1.  I was just going to manually run:
    dotnet build -c release InterpolatedSql.Tests\InterpolatedSql.Tests.csproj
    dotnet test  InterpolatedSql.Tests\InterpolatedSql.Tests.csproj

And I can successfully build the project, but trying to run the tests returns:
Testhost process for source(s) 'C:\BTR\OpenSource\InterpolatedSql\src\InterpolatedSql.Tests\bin\Debug\net5.0\InterpolatedSql.Tests.dll' exited with error: You must install or update .NET to run this application.
App: C:\BTR\NugetPackages\microsoft.testplatform.testhost\15.8.0\lib\netstandard1.5\testhost.dll
Architecture: x64
Framework: 'Microsoft.NETCore.App', version '5.0.0' (x64)
.NET location: C:\Program Files\dotnet\
The following frameworks were found:
  6.0.18 at [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  6.0.20 at [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  6.0.24 at [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  7.0.13 at [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Learn about framework resolution:
https://aka.ms/dotnet/app-launch-failed
To install missing framework, download:
https://aka.ms/dotnet-core-applaunch?framework=Microsoft.NETCore.App&framework_version=5.0.0&arch=x64&rid=win10-x64
. Please check the diagnostic logs for more information.

Test Run Aborted.

Can I just modify the test *.csproj temporarily to remove all target frameworks except for 6?

I've created a PR as you've probably seen.  I was able to temporarily disable the target frameworks that I don't have installed on my laptop to build/test the project without commiting those changes obviously.  Let me know what you think.


[Technical Discussions-Media Push] Metrics

when: bi-monthly
Day: Tuesday
Time: 6 am PDT (change to EU time)
Calendar
Twitter description
Answers: Why should I join and contribute to this project?

I submitted a comment via the agenda today to request the MicroProfilers help unblock this task.
https://docs.google.com/document/d/1HJOg-NOVpcE5X7qhELnyBa_A7OaVTwbOPCsAOp6F1Zc/edit?pli=1#
needed: https://github.com/eclipse/microprofile-marketing/issues/412#issuecomment-806167114

@Emily-Jiang,
Are you involved in Metrics' work? If yes, thank you in advance for looking at the calendar call duplication.
I don't want to delete stuff from the external calendar if the internal is also duplicated.
Can you please help us see why the duplication has started?

@rstjames,
Lets us move ahead and also schedule this API in the mpMedia. Thank you, Triber!
When:
Every Tuesday
3-4pm CEST
Twitter:
Join today's #MicroProfileMetrics discussion & contribute to the API's technical direction. #MicroProfilers' participation ensures the continued growth & development of a well-governed, vendor-neutral #OpenSource API.
https://bit.ly/calendar-zoom-minutes
#ByCommunityForCommunity

@aeiras @rstjames I don't go to their meetings. I've left a message on their gitter chat room for them to confirm.

@aeiras is this ticket for "Technical call" of "#MicroProfileMetrics APIs discussion"? I don't see the info needed for the technical call reminder but see it for APIs.

Hey Ryan,
The media push is for MP metrics call.
I am choosing to use the same description slightly change for the other APIs that are active. Call to action is the same for all, help the API move forward. : )
The information for the tweet is on my July 28th comment.

all set for the year.

Thank you, Ryan!
Please keep in mind to add the issues closed under the mktg "completed tasks" section, always!


Feature: Tabs

Feature request
Thinking of using ui in repository.surf as well - wondering if we can have tabs:
https://tailwindui.com/components/application-ui/navigation/tabs#component-556317db33900f6182a5f45cd9b50300

@joshenlim  what do you think?
I'm thinking we add something like <Tab.Panel> which can support children.
So then when a tab is clicked, the corresponding panel is shown.
<Tabs defaultActiveKey="1" onChange={callback}>
    <Tab.Panel label="Tab 1" key="1">
      Content of Tab Panel 1
    </Tab.Panel>
    <Tab.Panel label="Tab 2" key="2">
      Content of Tab Panel 2
    </Tab.Panel>
    <Tab.Panel label="Tab 3" key="3">
      Content of Tab Panel 3
    </Tab.Panel>
  </Tabs>

We could also support having no children,
in case the tab menu wants to be used alone and the callbacks could be used to control what content is shown.
<Tabs activeKey="1" onChange={callback}>
  <Tab.Panel label="Tab 1" key="1"/>
  <Tab.Panel label="Tab 2" key="2"/>
  <Tab.Panel label="Tab 3" key="3"/>
</Tabs>

// options version with array
// our checkbox and radio groups work like this as well
<Tabs
activeKey="1"
onChange={callback}
options={[
  {
    label:'Tab 1'
    key: '1'
  },
  // ...
]}
/>

@joshenlim  what do you think?
I'm thinking we add something like <Tab.Panel> which can support children.
So then when a tab is clicked, the corresponding panel is shown.
<Tabs defaultActiveKey="1" onChange={callback}>
    <Tab.Panel label="Tab 1" key="1">
      Content of Tab Panel 1
    </Tab.Panel>
    <Tab.Panel label="Tab 2" key="2">
      Content of Tab Panel 2
    </Tab.Panel>
    <Tab.Panel label="Tab 3" key="3">
      Content of Tab Panel 3
    </Tab.Panel>
  </Tabs>

We could also support having no children,
in case the tab menu wants to be used alone and the callbacks could be used to control what content is shown.
<Tabs activeKey="1" onChange={callback}>
  <Tab.Panel label="Tab 1" key="1"/>
  <Tab.Panel label="Tab 2" key="2"/>
  <Tab.Panel label="Tab 3" key="3"/>
</Tabs>

// options version with array
// our checkbox and radio groups work like this as well
<Tabs
activeKey="1"
onChange={callback}
options={[
  {
    label:'Tab 1'
    key: '1'
  },
  // ...
]}
/>

yeah i think this sounds good - having the option for either no children or with children, gives some flexibility.
If thats the case does it also imply that if no children is passed - Each Tab.Panel will have its own styling
But if a children is provided, - Each Tab.panel will not have its own styling, it'll depend on the children's styles for the looks

yeah i think this sounds good - having the option for either no children or with children, gives some flexibility.
If thats the case does it also imply that if no children is passed - Each Tab.Panel will have its own styling
But if a children is provided, - Each Tab.panel will not have its own styling, it'll depend on the children's styles for the looks

first iteration done.
will make a new issue for 'card' tabs, like we have in the supabase dashboard

first iteration done.
will make a new issue for 'card' tabs, like we have in the supabase dashboard


Updates to README

Readme contained specific versions of numba-dpex, dpctl and dpnp packages which were outdated. Updated the readme to always get the latest packages from intel channel.

after pre-commit pass and #171, this can be merged.

Done. Please take a look.

LGTM


Parameterize segment IDs

Issue:
The markUsed API accepts either an interval or segmentIds. When using segment IDs, the underlying query uses an IN clause with the user-provided values inlined directly in the query instead of being parameterized.
Fix:
This patch fixes that by adding the following utilities specifically for queries that contain an IN clause:

getParameterizedInConditionForColumn()
bindColumnValuesToQueryWithInCondition()

Other related changes:
Renamed the following functions for consistency and code hygiene:

appendConditionForIntervalsAndMatchMode() to getConditionForIntervalsAndMatchMode(). Pass only select arguments to the function.
bindQueryIntervals() to bindIntervalsToQuery()

Release note:
The segmentIds filter in the markUsed API payload is parameterized in the DB query.
This PR has:

[x] been self-reviewed.
[x] a release note entry in the PR description.
[x] added Javadocs for most classes and all non-trivial methods. Linked related entities via Javadoc links.
[x] been tested in a test Druid cluster.

@kfaraz, thanks for taking a look!

Thanks for the change, @abhishekrb19 . This is very useful!
It would be nice to have a test to verify that the code throws an exception when passing illegal values for the version or any other column.

What kind of validation and exception did you have in mind? Currently, the code doesn't perform any type checks for the columns.

What kind of validation and exception did you have in mind? Currently, the code doesn't perform any type checks for the columns.

I should think that JDBI would throw some exception if we try to bind am invalid string which contains, say a substring like '); DROP TABLE abc;.
We could try to verify that behaviour. By the way, this is optional and need not block this PR.

Ah, I see. The JDBI library doesn't check for "illegal" values for the named parameters as it's contextual. I ran a test with such values and it seems the library just treats them as any other string literal. That said, the binding itself makes the query safe to execute and I think if we want to detect illegal values or perform semantic validation, it has to be done at a layer on top (I'm not sure if that check can be full-proof though).

That said, the binding itself makes the query safe to execute

Yeah, that's a fair point. Thanks for the clarification.


Check for unlinked files

Waiting on Markcop refactor

@MaxWofford do we still need to do this?

We'd like it as a feature of Markcop, yes.

Hey! Any updates on this?

Nothing yet @jonleung. @zachlatta when you were testing for unlinked images, what script did you use?

It was just a one-off script I wrote. Here another one-line abomination I just
wrote that'll print the names of all of the unlinked files in a git repo.
(export IMG_REGEX="\!\[.*?\]\(\K.*?(?=\))" \
        URL_REGEX="https?:\/\/(www\.)?[-a-zA-Z0-9@:%._\+~#=]{2,256}\.[a-z]{2,6}\b([-a-zA-Z0-9@:%_\+.~#?&//=]*)" &&
 git ls-files |
   grep "\.md$" |
   xargs grep -oP "$IMG_REGEX" |
   sed 's/:/\t/' |
   grep -vP "$URL_REGEX" |
   sed 's/\(.*\)\t\(.*\)/$(dirname \1)\/\2/' |
   xargs -L1 -i bash -c "file={}; if [ ! -f \$file ]; then echo \$file; fi")

Accuracy and completeness of results not guaranteed, but it did work in my test case.

Closing due to inactivity.


Display Pump Status Highlight Message in home view

Display the eventual Pump Statut Highlight Message in home view if state is warning or critical.
fix partially issue #207 with displaying "No pod" in the homeview.
Example in French :

Warning message
Critical message

I suppose Statut is a typo and should be replaced with Status in English?

I agree with @bjornoleh. The variable and function names should use Status: pumpStatutHighlightMessage and displayPumpStatutHighlightMessage.
Start by testing what is there (or not there) in dev, commit 5e513441.
Note - the expiration count-down issue seems to be present in dev and not caused or addressed by this PR.
Test without the patch with pods using an rPi DASH simulator.

Deactivate pod: no message shown
Reservoir notification set to 10 U; use front end to set reservoir level to 5 U

The drop next to reservoir level on main screen is red (instead of blue)

Trigger Fault on pod - shows up on pod screen but not on main screen
Pod age notification set to 70 hours, set rPi pod age to 71 hours

Green dot on main screen changes to red stop-watch starts count-down from 59 minutes

Test after applying the patch with pods using an rPi DASH simulator.

unchanged: indication when pod age exceeds notification, by now says 45 m
unchanged: indication when reservoir is below notification level
increase pod age to 4602 minutes (76.7 hours), main screen did not update, it keeps counting down a minute at a time and now says 38 m

These next things are new and a success:

Trigger fault on pod (0x12) - error message appears, dismiss message - takes me to Omnipod screen (excellent)

main screen shows red "!! Pod Error"

Deactivate pod: main screen show "warning: No Pod"
Pair pod: quit and restart app:

main screen shows finish setup
begin insertion; quit and restart app:
main screen shows finish setup

Trigger out of insulin "fault", error message, dismiss message - takes me to pod screen, main screen shows red "!! No Insulin"
Repeat for 80 hour, same behavior, main screen shows red "!! Pod Expired"

Yet to do: Medtronic pod testing. So far so good.

@marionbarker - I tried to analyse some test results with my rPi simulator :

unchanged: indication when pod age exceeds notification, by now says 45 m :

Seems a correct behavior.

unchanged: indication when reservoir is below notification level

Seems a correct behavior. When reservoir = 0 ➡ message "No insulin" and error alert.

increase pod age to 4602 minutes (76.7 hours), main screen did not update, it keeps counting down a minute at a time and now says 38 m

Seems correct because a alert (Exceeded maximum Pod of Life...) is provided by pump when pod age > max ➡ message "Pod expired".

Updated with the correction of statut ➡ status

My prior test was not valid.
A real pod has an alert set when it has a notification or if it expires.
(My apologies - I often forget that I need to set the alert as well as the time to get reasonable responses with the rPi DASH simulator.)
I repeated the test for what happens near the time the pod reaches expiration.

Set the pod active time to 4315 minutes (72 hours minus 5 minutes)
Observe the main screen as the time counts down

Stages:

main display: red Stop Watch: 4 m
observe main screen at 0 m
observe main screen with red Stop Watch: Replace pod icon

GitHub/Xcode problems slowed me down a bunch, so I'll have to continue this tomorrow.
From my initial test:
Fresh install to iPhone SE (2nd gen) iOS 16.2
"Add Pump" icon
Added Medtronic pump
icon changed to "!! Signal Loss" (much better than staying at "add pump" icon like before)
Added cgm (glucose simulator) and icon changed to typical Medtronic status icons

Okay, I finished some more tests with my Medtronic 722.
It shows ⚠Insulin Suspended when the pump is suspended from the pump module in Trio, when the pump is suspended from the physical pump's menu, and when the pump completely runs out of insulin (actually runs out, not just says 0U)
The typical Medtronic status icons (battery and reservoir) are displayed at all other times, including when the pump claims 0U but actually still has insulin left, when the pump is out of range of the OrangeLink, and when the OrangeLink is out of range of the phone.

Merging with two approvals (several when including those before the last commit with only a trivial change).


chore: update requirements

Contributor Comments
Update all requirements.txt files to use the latest released versions of
dependencies
This remediates https://snyk.io/test/github/seisollc/easy_sast#SNYK-PYTHON-CODECOV-552149
Pull Request Checklist
Thank you for submitting a contribution to easy_sast!
In order to streamline the review of your contribution we ask that you review and comply with the below requirements:

[X] Ensure that make build completes successfully
[X] Keep our reports up-to-date by committing the output of make report
[X] Rebase your branch against the latest commit of the target branch, which likely should be master
[X] If there is an issue associated with your Pull Request, align your PR title with this documentation

Codecov Report

Merging #28 into master will not change coverage by %.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master       #28   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            8         8
  Lines          913       913
  Branches       173       173
=========================================
  Hits           913       913

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 0095009...307452f. Read the comment docs.


Thermostat Beta Update

Updated Thermostat implementation that had some broken merge conflicts in it.

@jfarmer08 this should be a good beta version to merge.

Not trying to be a pain, but your branch is base on the rebase. Since there are so many changes in that and it's more of a playground at this point could you make a copy of what you have and then pull my dev branch down. From there create a branch from it. So when you do a pull request we only see the changes you have for the thermostat. If that does not makes sense let me know.

I don't mind doing it, but I would like to give you credit for this.

 I will have it up tomorrow then.

@jfarmer08 things should be good to go now, let me know if there is still an issue!

Looks good. I will merge this tonight


Make minor cleanup improvements in scheduler.hpp

follow through on using perfect forwarding instead of changing method signature of scheduleAndReplyValue
make previously protected scheduleAndReplyValue method private as all current use-cases do not require it
add trivial compile-time check for TaskFn to be invokable

Thanks for your contribution @bilal614!


[RESTEASY-2410] added check of ResteasyProviderFactory instance in Bu…

…iltResponse

@rsearls , I'm trying looking again at this. I was thinking that, instead of trying cleaning up the default ResteasyProviderFactory instance in the response as you're proposing, we could see if we can completely avoid setting up a new "default" instance if it's not set already (as, again, I think the singleton instance is a source of problems).
Alternatively, we could deal with this by ensuring the ClientFilterResponseBuilderTest leaves the environment in the same state if sees it before running, iow it does not change the default ResteasyProviderFactory instance.
One way or the other, I would need to reproduce the failure this fix is meant to solve, but I couldn't. Can you clarify how to do that?
Thanks

Running ClientFilterResponseBuilderTest without this fix causes a reference to
a ResteasyProviderFactory instance (on the clientside) to be held globally and
used in all tests that runs after it.  Any special code set in that instance of
ResteasyProviderFactory has the potential to negatively affect any following test.
This was a discovery made when debugging the failures with
AllowGzipOnServerAllowGzipOnClientTest and NotAllowGzipOnServerAllowGzipOnClientTest.
While it was determined that this issue was not a direct cause of those test
failures, it has the potential to cause havoc in customer code and future tests.
Currently the only way to see this issue is to use the debugger and record the
address of the generated (clientside) ResteasyProviderFactory instance created in
ClientFilterResponseBuilderTest and check for its reference in the following
test(s).

It is so old as to be obsolete


Error when start project

I can't start screenings
plz help me, thanks
E:\meteor\Screenings>meteor
[[[[[ ~\E\meteor\Screenings ]]]]]
=> Started proxy.
=> Started MongoDB.
=> Errors prevented startup:
While processing files with utilities:compile-svg (for target web.browser):
error: ENOENT, open 'E:\meteor\Screenings\undefined\svg\all_categories.svg'
=> Your application has errors. Waiting for file change.

I got the same error when I start screenings


bundle geocoder, locator

Leaflet geocoder now supports partial importing! Thanks a lot @louh
Since we figured out how to bundle standalone without importing library's dependency (Leaflet), Leaflet-Locator was also replaced to required version of it. It will make maintenance much easier.
related to : https://github.com/mapzen/leaflet-geocoder/issues/154

I wanted to hear @rfriberg 's opinion about  what to do about public-facing api keys. Also there is a separate issue about that https://github.com/mapzen/mapzen.js/issues/369 so we will deal with this separately.
Thanks for comments!


(Sub)Domain-based Routing

Describe the problem
Hi,
first of all, thanks for this excellent piece of software! I truely love it.
I'm currently building a platform where I have multiple tenants. They can configure some settings and provide a frontend for their users. This should live on https://tenant-slug.my-platform.tld. Then, they should also have an admin-dashboard at https://admin.tenant-slug.my-platform.tld.
I'll also have some registration / onboarding ui at https://registration.my-platform.tld.
Currently, this does not seem possible with a single Svelte app. I can't build one for each tenant, because they change quite often.
At DNS level it would be solved via a wildcard subdomain and a wildcard SSL certificate.
Describe the proposed solution
As proxies (see below) are complicated to setup and didn't work for my usecase, I'd like to be able to have different routes for each subdomain and the pure domain.
Maybe this is kind-of linked: https://vercel.com/guides/nextjs-multi-tenant-application
Here are my thoughts:
Currently, there's the routes directory with all the routes inside. In order to support multiple (sub)domains, there could be multiple routes folders like routes_registration., routes_[tenant]. and routes_admin.[tenant].. Notice the dot at the end, which means that it is a subdomain. Without the dot, we could support thinks like site-with-german-title.de and english-site.co.uk as then, it would match against the whole host.
The only thing I don't like about this, however, is the fact that we're breaking the convention, that each folder name equals one segment of the path exactly (as we add the routes_ prefix.
One way to work around that would be to have a new folder called domains in the src directory, and inside there other folders with the names / wildcards of the subdomains. However, we'd then need a way to split that part from where the actual path-based routing begins. This could be done with a convention (like a folder named _routes in there), however, that would prevent people from having a _routes subdomain.
Please let me know what you think about that, I'm happy to hear other opinions!
Alternatives considered
What I've tried so far is using a reverse proxy (traefik in this case) to map the subdomains to path prefixes (e.g. registration.my-platform.tld to my-platform.tld/registration internally. This, however, does not work, as the base path is always wrong in that case.
Maybe this is related to #595, but I don't think its the same case as with that proxy it would need a multiple different base paths (or rather hosts).
Importance
i cannot use SvelteKit without it
Additional Information
This feature is rather important for my application, as there is the requirement for nice urls using subdomains.
I'm also happy to try implementing something once we have a proposed solution and someone can point me in the right direction.

If each subdomain's application has the same basic shape (which it should if you want to use a single application for all of them), then you should be able to just expose the Host header to the rest of your app either in your handle hook or in a top-level +layout.server.js.

Hi @Conduitry,
thanks for the fast response. I haven't thought about that. However, I don't think that it would work for the described usecase.
The routes are not the same for all subdomains. There's the difference between user-frontend and tenant-admin-backend as well as the registration pages.
They do share a lot of UI components and server logic though, so I'd prefer not to move that into a package make separate apps...

It seems possible with Nuxt: https://ynechaev.medium.com/subdomains-with-nuxt-js-483df826c788

I would +1 this, I am looking into making an application where each user has his own subdomain and having a build in way to get it would be really convenient. This is something that is supported in other frameworks as far as I remember.

I have the same use case : ~4 apps that share most of their UI components, stores, utils, but with different routes. The apps are deployed on different domains. (ex1, ex2)
The way I solve it currently is to have a monorepo with a routes directory per app, like routes/app1. and routes/app2 I then use the kit.files.routes config to set the correct directory depending on an env variable PUBLIC_APP=app1 npm run dev.
I deploy these apps on Vercel where I have 4 different "projects" connected to the same repo, but with a different env variable.
This works very well, except that now the deployment is slow. Vercel needs to rebuild the app 4 times and it isn't done in parallel (extra workers are costly on Vercel). If Svelte Kit supported this "multi-tenant application" pattern to build a single "app" to serve my 4 distinct sites, I would be happy to switch to it to simplify the development and deployment processes.

Is there any reason you need those to instead be a single app?

I'd say it's about convenience for the developer. Svelte Kit is already so great at dev experience and I (and if I understand @mquandalle and @georgeiliadis91 correctly them, too) feel like being forced to split apps - just because the routing technique we'd like to use is partly based on (sub)domains and not exclusively on paths - impairs that quite a bit.
Advantages I see with a single app are the following:

easier deployment if done "by hand" (not on vercel, etc) as you'd only have a single application to start and not 3 + load balancer which does the routing
Shorter build time and less complicated build pipeline requirements (see comment by @mquandalle)
no need for custom logic like parsing the host header and accessing that in the template differently to path parameters
no need for a monorepo, which can be complicated to setup, especially with TypeScript typings
benefits of a single app backend instead of multiple (only one db connection, ...)

Don't get me wrong. I could probably achieve the same thing with the current version of Svelte Kit, it would just not be nice code and a good dev experience. You could also completely ignore the router, parse the path segment of the URL and have a single template with a ton of {#ifs. It would just not be as convenient, usable and (most importantly) maintainable then using the router.
I see the (sub)domain routing feature kind of related to this example.
Can you see my points?
Best regards

Couldn't you just deploy a single app, and assign all of your desired subdomains to it in vercel's config?
then you can look at the host header in a hook, and just respond accordingly.
single app, single deployment, multiple assigned domains

Couldn't you just deploy a single app, and assign all of your desired subdomains to it in vercel's config?

I would like to, but I don't know how to benefit from file system routing for multiple apps simultaneously.
routes/
  app1.com/
  app2.fr/
  subdomain.app1.com/

Maybe it's possible to rewrite the query in a hook to append the host name to the query path so that http://app1.com/about is routed into /app1.com/about/?

Honestly, this feels like 'scenario solving' to me. Bottom line, these are separate apps and should be treated as such. Introducing significant extra complexity to the routing system just to cut down on build time a bit is entirely the wrong trade-off.
'I don't want to set up a monorepo' simply isn't a good enough reason to make the framework itself more complicated. This is what monorepos are for.
There are workable approaches to do the things described in this thread, so I'm going to close this issue.

OK, if you say that's out of scope, we need to accept that.
@mquandalle Did you manage to get your approach (multiple routes folders and switching them with an env variable) working with the VSCode Intellisense? I've just tried it and ran into a lot of problems. When having a single .svelte-kit
directory, multiple processes try to write to the same file (I need to have all apps running when developing).
When having different .svelte-kit folders, the Intellisense doesn't like that, as the tsconfig.json at the root needs to point to one of the generated tsconfig.jsons explicitely...

@Rich-Harris is there an example of a monorepo available? I'm struggeling with some points. For example: a .env file at the workspace root is not parsed for $env/static/private. Copying the .env file to all the child packages seems kinda redundant...

Hi @hrueger - please use discord (https://svelte.dev/chat) for usage questions, where people will be better able to discuss and help with your specific requirements.

Discord is fun, but it's hard to find things on it and it's closed (you need a login). I always find it much better if there's a proper documentation on publicly available sites like github for issues like these that many people might want to google for.

I think that is possible now with Universal Hook Reroute:
https://kit.svelte.dev/docs/hooks#universal-hooks-reroute

While the debate is settled, I was exploring this as I had a small personal use.
@hrueger How about handling subdomain via hooks.
My usecase was very specific and persona. This code also handles just that. It's not generic and I do not recommend you use this for production systems without due diligence.
// src/hooks.ts
import { redirect } from '@sveltejs/kit';
import type { Handle } from '@sveltejs/kit';

export const handle: Handle = async ({ event, resolve }) => {
    const host = event.request.headers.get('host');

    // Check if the request is from a subdomain in both localhost and production
    if (host) {
        // Assuming production domain is 'example.com'
        if ((host !== 'localhost:3000' && host.endsWith('localhost')) ||
            (host !== 'example.com' && host.endsWith('support.example.com'))) {
            // Redirect to '/help' when accessed via any subdomain
            throw redirect(307, '/help');
        }
    }

    // Continue with normal flow if no subdomain
    return resolve(event);
};

This is super easy to do with the reroute hook:

https://ramonmalcolm.com/articles/dynamic-domain-subdomain-routing-sveltekit-guide/
https://kit.svelte.dev/docs/hooks#universal-hooks-reroute

I'm planning on using it for the "each user has their own subdomain" use-case.

You can see how I do it in my app here if you want: https://github.com/muni-town/weird/blob/main/src/hooks.ts
You don't need to access the database in that hook. What you do is you check if it's a subdomain, and then you route to a subdomain specific route in your app.
Then when it gets to that route, your route checks the database to see whether or not that subdomain exists in the database or should return a "page not found" because it's not there.

I actually have custom domains working, too!
Basically you could do something like:

In hooks.ts we check if the url.host == env.PUBLIC_DOMAIN, if so, we just return the route unchanged.
Otherwise, we check if it's a subdomain, and we reroute to \/subsite/${subdomain}``
If it's not a subdomain, we treat it as a custom user domain and we reroute to \/subsite/${url.host}``

Then, in our /subsite/[usernameOrCustomDomain] route we check the database to see if usernameOrCustomDomain is a username in our database. If it is, we render a subdomain site from the user's data.
If usernameOrCustomDomain isn't a username, we look in our database for a registered custom domain, and we render the user's site that has that custom domain set.

I also some stuff in there, when the user is configuring their custom domain, it does a little DNS "challenge" to make sure that the domain they are inputting is actually resolving to to the app server, but that's just around how you set the domain, not how you do that routing.
There's also some stuff in there to generate a Traefik reverse proxy configuration for each custom domain, so that we can provide automatic HTTPS certificates for everything. But again, that's kind of unrelated to the routing.

Another solution could be to have a separation at the configuration level. You can create a separate config file for each sub-app and specify parameters such as files, appDir, etc in there. Then you create a separate deployment/development scripts for each app.

Another solution could be to have a separation at the configuration level. You can create a separate config file for each sub-app and specify parameters such as files, appDir, etc in there. Then you create a separate deployment/development scripts for each app.

Apparently separate config file are not supported. The approach is still valid though by using command args and detecting them within the config file as it is shown in this example: https://github.com/sveltejs/kit/issues/2973#issuecomment-986050439


Added ticks on/off in shell/console

Cleaned up a few things in the prompt stuff and added ticks command to shell. You can now turn on/off printing OS Ticks on the console. New Config CONSOLE_TICKS added to sys/console

Hi David,
First and most important - I like the feature and it seems to work perfectly.  Thanks for implementing this!
That said, there are a few nagging comments that I would like to make:

I noticed the default value of CONSOLE_PROMPT got changed from 0 to 1.  I do think most people prefer having the prompt enabled, but I would be leery of changing the default.   My main concern is that we have had a few issues with the prompt interfering with newtmgr commands.  The problem here isn't the prompt; it is that we are overloading the console as a newtmgr transport, which causes no end of weird issues.  As long as we continue to use the console as a newtmgr transport though, I think it is probably best to leave the prompt disabled by default.

There are a few coding standard violations, namely:

{noformat}

Put space after keywords (for, if, return, switch, while).
After a function declaration, the braces should be on a newline.
{noformat}

It is really easy to accidentally violate a coding standard or two.  I wonder if we should look into using some sort of automated check upon commit.
Do you think you could take a quick look at these issues?  Please feel free to voice any disagreements or questions.
Thanks,
Chris

Hi Chris,
Sorry about the change of the default value. That actually wasn't supposed to be in the commit.
I've fixed the coding standards violations. Old habits die hard sometimes. I noticed that I do it the old way about 50% of the time, and the new way the other 50%. Getting there.

Hi David,
I saw your other pull request which addresses these minor issues (https://github.com/apache/incubator-mynewt-core/pull/122).  I didn't merge that one because it appears to target the 1.0.0-b1 branch rather than develop.  It might be easiest to just commit the fixes to your develop branch.  That way, they will be part of the same pull request.
Thanks,
Chris

Hi David,
Looks good.  Just don't forget to set the default CONSOLE_PROMPT value back to 0 :).
Chris

It appears there are some unrelated changes in your development branch, which makes it difficult to merge this pull request.  In particular, there are some changes to bleprph.  Github expects there to be a dedicated branch per pull request, so coordination is difficult when the source branch is something like 'develop'.  The source branch needs to be frozen, apart from changes that directly pertain to the new feature or bug fix.
I hate to make more work for you, but do you think you could create a separate branch that contains only the changes pertinent to this pull request?  Then you would close this PR and open a new one against the new branch.
Please let me know if I have misinterpreted something, or if something I've said doesn't seem right.
Thanks,
Chris


[WIP] Swap package update with Dexhunter integration

since there is a new API, closing it, will keep the branch for reference.


852 passkeys testing safe version compatibility

What it solves
Resolves #852 by scoping the tests to the correct version.
How this PR fixes it
Restrict some of the tests to the version that is applicable only. Adds a test that expects failure for the versions that are incompatbile (if it doesnt exists). The actual verification of the feature on the kit was previously added.

The current branch should be merged on the passkeys branch. Since its aligned with the same branch, it should have some tests failing.


use command  /msg mattermost identify insted of login

for mimic nickserv support

Any technical reasons?

you can run irc bots transparent on mattermost :)

https://gist.github.com/zodman/83b8b4e95fee86892591


Run test button not working

Challenge Change the Color of Text has an issue.
User Agent is: Mozilla/5.0 (Windows NT 10.0; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/50.0.2661.102 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
 See what you guys can do, I really would to go all the way through
Thanks

@smithcoder the code you have is wrong so that is why the Run button appears to be broken.

Change your h2 element's style so that its text color is red.

If you get stuck with a challenge, please first ask questions to the Help Room. Happy coding!

@smithcoder You need to create a new p style and a new p paragraph  for the purr kitty.


Allow release-manifests Target to Specify Image

Add $(IMAGE) to the release-manifests target. This will allow e2e testing of a release prior to cutting the actual release. For example, @Xunzhuo created this test v0.2.0 release but the EG deployment manifest uses a non-existent image.
$ kubectl get deploy/envoy-gateway -n envoy-gateway-system -o yaml | grep "image: "
        image: envoyproxy/gateway:v0.2.0

@arkodg thoughts?

/assign

OK, ping me if this is needed, I have already implemented this locally.

yah this sounds good, thanks for picking this up @Xunzhuo !


add github page for Vitess

github page for vitess.io.
I use Jekyll and grab this guy's (https://github.com/mmistakes/skinny-bones-jekyll) theme, so have to add MIT license.
Some notes:

CNAME is added so that vitess.io could point to youtube.github.io/vitess
author.yml only contains my info but anyone could add it later, so that you can put your name into specific docs if you like.
logo and thumbnails are missing, will add them later (once got a nice design).
to publish the site to github, one need to have ruby installed and may want to follow this tutorial (https://help.github.com/articles/using-jekyll-with-pages/)

All site related stuff are in vitess.io dir and there is a publish-site.sh script in root dir that will publish these contents to github page. This script has to be run manually.

LGTM


Split set secret key and set decrypted content into separate methods …

…of squeak db class

Codecov Report

Merging #1908 (06d0b1a) into master (e827db2) will increase coverage by 0.04%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #1908      +/-   ##
==========================================
+ Coverage   63.26%   63.31%   +0.04%
==========================================
  Files          92       92
  Lines        4647     4653       +6
  Branches      308      308
==========================================
+ Hits         2940     2946       +6
  Misses       1679     1679
  Partials       28       28

Impacted Files
Coverage Δ

squeaknode/db/squeak_db.py
99.07% <100.00%> (+<0.01%)
:arrow_up:

squeaknode/node/squeak_store.py
56.65% <100.00%> (+0.37%)
:arrow_up:

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 e827db2...06d0b1a. Read the comment docs.


Question about using nav mesh areas

Can I use Unity nav mesh areas (like Walkable, Not Walkable, Jump, custom areas) with a NavMesh baked at runtime / if so how do I do it? It seems like right now if I mark a surface as "Not Walkable" and remove this from a nav mesh agent's area mask it will still generate nav mesh on it and the agents will still walk on it...

Yes, each NavMeshBuildSource that you feed into the NavMeshBuilder.UpdateNavMeshData() method can have a different area value that will be taken into consideration when building the NavMesh. "Not Walkable" has the highest priority over all other area types.
If something is not working please submit a bug report to Unity.


re: near-dump-analyzer: Fix clippy warnings about loops index

Let's improve quality, and fix clippy warnings in near-dump-analyzer

@matklad After looking it again, I like the original code better, maybe adding #[disable(clippy::...:)] or whatever the syntax is first, makes more sense.


Gulp.Watch doesn't work on files inside a symlink folder

What were you expecting to happen?
File changes inside a symlink folder to trigger Gulp.Watch
What actually happened?
Nothing.
Please post a sample of your gulpfile (preferably reduced to just the bit that's not working)
gulp.task('watch',function() {
  return gulp.watch('wp-content/themes/pm/less/*.less',gulp.series('less'));
});

If I change "pm" to "twentyseventeen" everything works fine.
Similarly, if I remove the symlink folder "pm" and replace it with a real "pm" folder, gulp.watch also works fine.
Other tasks work, as well. For example, compiling LESS to CSS functions normally. It just seems to be gulp.watch that's the issue.
Folders
wp-content
-themes
--twentyseventeen
---less
--pm (symlink)
---less
What version of gulp are you using?
4.0.0-alpha.2
What versions of npm and node are you using?
npm: 4.2
node: 6.10.0

Stop using alpha 2
This is a thoroughly discussed problem with chokidar. I forget the solution off the top of my head. You can search on their issue tracker.

I did do a search, and found several workarounds here: https://github.com/paulmillr/chokidar/issues/293
However, they don't work. I tried to glob the entire themes folder, and while that worked properly for the twentyseventeen folder (as expected), the "PM" symlink folder still did not trigger gulp.watch.

While I haven't been able to get gulp.watch to work, I did switch to Gaze, which worked straight out of the box.

Glad you found something that worked because we weren't switching back after removing gaze.


fix: navbar items & logo placement to make them vertically center aligned

Fixes #3455
Describe the changes you have made in this PR -
Added display: flex to nav-item class. Then added justify-content property accordingly.
Screenshots of the changes (If any) -

@danger-ahead Thanks for the PR, I agree that it is important for making things aligned but I find initial positions more stabilized.
Also, https://github.com/CircuitVerse/CircuitVerse/pull/3425 PR is under review and we are working on re-designing the navbar.
PS: It is always considered good practice if you discuss the issue before the PR in #community channel and let them accept the feature/bug/UI request.

@danger-ahead Thanks for the PR, I agree that it is important for making things aligned but I find initial positions more stabilized. Also, #3425 PR is under review and we are working on re-designing the navbar.
PS: It is always considered good practice if you discuss the issue before the PR in #community channel and let them accept the feature/bug/UI request.

Thanks. Will keep that in mind. Shall I close this PR then?


Provide some way to reliable know when FileSystemWatcher is listening for events

In our tests it seems when we create a file watcher
const w = vscode.workspace.createFileSystemWatcher(...)

it actually does not start seeing file events for some time after creation.
I went through the docs and code briefly and it seems to really behave that way since the watcher is returned immediately, but in the background there is the machinery of dedicated file watcher process to which the request needs to be delivered by some RPC which is bound to create some delay even if everything else (other vscode internals, parcel, ...) starts the watch instantaneously.
While this is not a huge deal when a real user interacts with the extension (since people are inherently slow and are able to retry any action) it makes it very hard to test any code that depends on file watching in automated tests.
For example we need to do this
const unwrapper = new ProjectUnwrapper() // some code creating file watcher in constructor
await sleep(5000) // give the backing file watchers enought time to actually start watching
await vscode.workspace.fs.rename(j('foo.fizz'), j('bar.fizz'))
expect(unwrapper).reactedToFileEventSomehow()

Which works most of the time but breaks down when the CI is under load because the 5sec sleep is no longer enough.
We also tried watching the system log (by tailing the log file on disk and using this to unblock barriers in the test) with trace logging enabled as described here https://github.com/microsoft/vscode/wiki/File-Watcher-Issues but sometimes that did not work either + some ways of creating the watcher do not seem to produce any message into that log at all (e.g. when given just single string arg to the createFileSystemWatcher).
I think it'd be very useful to have some means provided by vscode API to know when one can reliably expect file events to be emitted e.g.
export interface FileSystemWatcher extends Disposable {
  onWatcherStateChanged: Event<FileWatcherState>;
}

export enum FileWatcherState {
  Preparing, // no events will arrive yet, watch is being set up
  Watching , // fully operational
  Degraded,  // e.g. when the dreaded "No more file descriptors" problem appears,
             // some events might still appear but can't be relied on to be complete
}

If a full blown API is too heavy change, given this may not be used by most extensions in their production code, it might be enough to have a command provided by vscode that, given a watcher instance, returns boolean / enum as described above indicating if that instance is watching. We could then poll this command until it tells us the watcher is ready to go
const watcher = vscode.workspace.createFileSystemWatcher(...)
for(;await vscode.commands.executeCommand("fileWatcherState", watcher);){
    await sleep(500)
}
// now I can start testing code using watcher knowing it will not miss the initial events

this has obviously the downside that I'd have to keep polling the method to find out if the watcher becomes degraded later because e.g.  many files were added into the workspace and I run out of file descriptors, but since it is mostly for testing where we have control over the environment it would be quite sufficient for our case
Originally asked in discussions https://github.com/microsoft/vscode-discussions/discussions/1242 where @theReynald confirmed this is currently not possible, thus this feature request.

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!


Registrator.jl

@JuliaRegistrator register

Registration pull request created: JuliaRegistries/General/106723
Tip: Release Notes
Did you know you can add release notes too? Just add markdown formatted text underneath the comment after the text
"Release notes:" and it will be added to the registry PR, and if TagBot is installed it will also be added to the
release that TagBot creates. i.e.
@JuliaRegistrator register

Release notes:

## Breaking changes

- blah

To add them here just re-invoke and the PR will be updated.
Tagging
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.1.0 -m "<description of version>" 558aef9ff71f1bf813639beedb1fb509ecc6a4e8
git push origin v0.1.0

@JuliaRegistrator register

Registration pull request created: JuliaRegistries/General/107979
Tip: Release Notes
Did you know you can add release notes too? Just add markdown formatted text underneath the comment after the text
"Release notes:" and it will be added to the registry PR, and if TagBot is installed it will also be added to the
release that TagBot creates. i.e.
@JuliaRegistrator register

Release notes:

## Breaking changes

- blah

To add them here just re-invoke and the PR will be updated.
Tagging
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.2.0 -m "<description of version>" f95c00cc4b61b21710b854b0e6d12df3e1f2d77a
git push origin v0.2.0

@JuliaRegistrator register

Registration pull request created: JuliaRegistries/General/113029
Tip: Release Notes
Did you know you can add release notes too? Just add markdown formatted text underneath the comment after the text
"Release notes:" and it will be added to the registry PR, and if TagBot is installed it will also be added to the
release that TagBot creates. i.e.
@JuliaRegistrator register

Release notes:

## Breaking changes

- blah

To add them here just re-invoke and the PR will be updated.
Tagging
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.3.0 -m "<description of version>" 434e8c6447076ba624addd468d53cb3489b333cc
git push origin v0.3.0

@JuliaRegistrator register

Registration pull request created: JuliaRegistries/General/118487
Tip: Release Notes
Did you know you can add release notes too? Just add markdown formatted text underneath the comment after the text
"Release notes:" and it will be added to the registry PR, and if TagBot is installed it will also be added to the
release that TagBot creates. i.e.
@JuliaRegistrator register

Release notes:

## Breaking changes

- blah

To add them here just re-invoke and the PR will be updated.
Tagging
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.4.0 -m "<description of version>" a35c9dd71c3ae11ecf34ef6319b39521bb9b8c29
git push origin v0.4.0


Can't configure where local picotool binary is

I have the picotool binary on my path. cmake (and Findpicotool.cmake) still somehow can't seem to find picotool.
They try to build it, but that fails for other reasons. I don't need to build it. I have a working binary. Why can't it just use it?
I have also tried following picotool's instructions on telling the SDK where the single shared picotool is by setting the picotool_DIR environment variable to the folder that contains the binary (or with -D... to cmake`) and it still tries to build.
I must be doing something silly...

Have you followed the instructions to install picotool in your custom location? The SDK is not actually looking for the picotool binary, it’s looking for the picotoolConfig.cmake file which only gets created when you install picotool.
The develop branch of picotool also has a more fleshed out README on this, so might be worth looking at instead of master if you’re having issues.
Also, as a side note, Findpicotool.cmake isn’t actually used to find picotool, it’s used to build it if it can’t be found - the actual finding is done by the CMake find_package function.

I downloaded the binary for my system from github and put it on my path. Yes, that is not the default install instruction, but I didn't want to build picotool for my system because that's broken for other reasons and I just wanted to use a binary. Figured that should be easy.
In the interim, it looks like the default install instructions might automatically download a binary instead of building (if available) so I should give that another go I guess. Still, I feel like being more clear about how picotool's location can be configured should be a little more obvious, no? Are we ignoring the path too?

Yes, the binary in the path is not used - CMake’s find_package is just looking for the picotoolConfig.cmake file which then points at the picotoolTarget.cmake file, which points at the picotool binary. Hopefully whatever you downloaded came with those files, so if you just use that downloaded directory as-is and set picotool_DIR to that directory it should work - no need to touch the PATH.

I see those files now. I downloaded this from raspberry pi's repo: https://github.com/raspberrypi/pico-sdk-tools/releases
I'm not seeing how then to put picotool on the path and supporting what cmake expects. I guess I could symlink to the binary, but that gets weird on non-linux systems.
I still think the original requests here stands - or at least to get error messages that more helpfully point to the solution.
Is there some reason we couldn't just specify the binary directly?

If you just add that directory you downloaded to your PATH variable (eg in .bashrc on Linux) then that’s the easiest way to add picotool to your path - you could also symlink it as you suggest.

What made it work on MacOS:
When followed the instructions on https://github.com/raspberrypi/picotool/blob/develop/README.md#usage-by-the-raspberry-pi-pico-sdk to install to the standard location with make install, the ~/.local/bin had to be added to the PATH. Also pointed out in the instructions, just not prominent enough.


Add password verification when creating a password for keystore file

When exporting a keystore file from ioPay desktop the app asks the user to create a password for the file: it's highly recommended to also ask to verify the password here otherwise the user could lose access to his wallet due to a typo in the password:

Thanks! moving to v2.0


[GHSA-w98g-5fmx-wm4x] pocketmine/raklib reliable-ordered queue size is unlimited, allowing a session to hog server memory

Updates

Affected products
CVSS

Comments
Quiero Mas Apoyo para gestionar mi empresa Microsoft y ejecutar de mejor manera MI proyecto Científico Empresarial Educar Para Avanzar

Hi there @dktapps! A community member has suggested an improvement to your security advisory. If approved, this change will affect the global advisory listed at github.com/advisories. It will not affect the version listed in your project repository.
This change will be reviewed by our highly-trained Security Curation Team. If you have thoughts or feedback, please share them in a comment here! If this PR has already been closed, you can start a new community contribution for this advisory

This is not correct, the original CVSS was accurate


delete service_name

Please answer these questions before submitting your issue.

Why do you submit this issue?
[ ] Question or discussion
[ ] Bug
[X] Requirement
[ ] Feature or performance improvement

Question

What do you want to know?

[X]  I created one “Your_ApplicationName ”service_name at startup. How can I delete it?

Bug

Which version of SkyWalking, OS and JRE?

Which company or project?

What happened?
If possible, provide a way to reproduce the error. e.g. demo application, component version.

Requirement or improvement

Please describe your requirements or improvement suggestions.

You either understand the storage by yourself or wait for the TTL passed.


Looks for .ssh folder in wrong location?

(I'm posting this here, because GitHub said I should - I encountered this issue when launching the console from the GitHub for Windows app)
I (used to) use MsysGit, which looks in %HOME% for the .ssh folder. posh-git looks in %HOMEPATH%. This leads to two different .ssh folders.
Could posh-git be changed, so it looks for the .ssh-folder in %HOME% if this variable is present and falls back to %HOMEPATH% if it isn't? That way the transition from MsysGit to posh-git (or the use of both) would be a lot easier.

I have changed this to use PowerShell's built-in $HOME variable, which isn't itself an env var; however it is derived from the env vars %HOMEDRIVE%%HOMEPATH%.
In PowerShell ~ is the same as $HOME except when you are located in a provider other than the filesystem e.g.:
12> cd cert:
\
13> resolve-path ~  | fl *
resolve-path : Home location for this provider is not set. To set the home location, call "(get-psprovider
'Certificate').Home = 'path'".
At line:1 char:1
+ resolve-path ~  | fl *
+ ~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidOperation: (:) [Resolve-Path], PSInvalidOperationException
    + FullyQualifiedErrorId : InvalidOperation,Microsoft.PowerShell.Commands.ResolvePathCommand

So it is better to avoid ~ if possible.  However I'm curious how msysGit seems to work better with just %HOMEPATH% i.e. the home path sans drive name - \users\keith?
Is msysGit more *nixy and not like drive specifiers?  If so, we could search $Home first and if .ssh is not found there, then we could search $Home with the drive specified stripped off.

I had forgotten the module still includes:
if (!$Env:HOME) { $Env:HOME = "$Env:HOMEDRIVE$Env:HOMEPATH" }
if (!$Env:HOME) { $Env:HOME = "$Env:USERPROFILE" }

So really it should be pretty safe to use $Env:HOME instead of $Home to align with Git/MSYS.

Is msysGit more *nixy and not like drive specifiers?

Yeah, in Git Bash you will see /c/Windows instead of C:\Windows.

I think 0.7.0 should resolve this but I don't run msysGit, so I can't test if it really is fixed.

I don't run msysGit, so I can't test if it really is fixed.

msysGit of 2012 is what we now call Git for Windows.
At this point I'm basically content to let SSH issues sit (as if I hadn't been before 🙄) until someone gets fed up enough to spin off a project for it (#338).

SSH functionality has been spun off into the posh-sshell module (https://github.com/dahlbyk/posh-sshell) so closing this issue.


[feature request] connection to Microsoft Active Directory

Description
When a user create an account we want to be able to automatically create the account based on the Microsoft Active directory of our company
Steps to reproduce

Go to '...'
Click on '...'
Scroll down to '...'
See error

ILLA Builder Version
2.8.0
Browser
No response
Operating System

[ ] macOS
[ ] Windows
[ ] Linux

Additional Information
No response

Hi team
I want to know more about the requirements of this issue. Let's connect and discuss!!
I'm participating in Hacktoberfest and am excited to contribute to open-source projects.


Kimchi: Enforce number of prev IPA commitments in verifier index

In review: https://github.com/MinaProtocol/mina/pull/11442 and https://github.com/o1-labs/proof-systems/pull/671

still missing this check right? https://github.com/o1-labs/proof-systems/pull/609/files#diff-fc95db4d289fd33bc7fbefe8a487926ce8b47cbe71f047a1dd6933d7444b5d5aR159


eslint

As a developer I would like to have a consistent code style therefore we can use a tool like eslint to highlight deviations and automatically fix simple syntactical errors.
ACs

[x] Add the eslint tool as npm dependency
[x] Write npm scripts to execute eslint as to analyze the code and another one to run exlint to analyze and fix issues which can be fixed automatically
[x] Adjust eslint rules to match project setup
[x] Extend developer docs and introduce eslint

Added eslint configuration and npm dependencies in 3b80eeb053463f3f9aa25e521e373849d691fd1f

Adjusted developer docs and added a new rule to our Definition of Done in a8c23ac6821977f196fecdfd7530c88340a7c183.


SIGABRT Crash in ax::Director::mainLoop on Android

Our Axmol-based game is encountering a SIGABRT crash on Android devices. The crash appears to occur within the ax::Director::mainLoop function. Below are the details of the crash report and the steps we have taken to debug the issue.
Crashed: Thread
SIGABRT
OS: Android: 3.18.14 samsung/on5xeltejv/on5xelte:8.0.0/R16NW/G570FXXS4CTJ7:user/release-keys -14263629-QB35148527 #1 SMP PREEMPT Fri Oct 16 21:36:26 KST 2020 armv8l
CPU: arm: ARMv0
GPU: -
0  libc.so          (Missing)
1  libc.so          (Missing)
2  base.odex        (Missing)
3  libc++_shared.so (Missing)
4  libc.so          (Missing)
5  libc.so          (Missing)
6  libc.so          (Missing)
7  libc++_shared.so (Missing)
8  libc++_shared.so (Missing)
9  libc++_shared.so abort_message (abort_message.cpp:72)
10 libc++_shared.so (Missing)
11 libc++_shared.so (Missing)
12 libc++_shared.so demangling_terminate_handler() (cxa_default_handlers.cpp:62)
13 libc++_shared.so (Missing)
14 libc++_shared.so __unw_get_reg
15 libc++_shared.so __unw_set_reg
16 libc++_shared.so __unw_set_reg
17 libc++_shared.so _Unwind_VRS_Pop
18 libc++_shared.so __unw_set_reg
19 libc++_shared.so _Unwind_VRS_Pop
20 base.odex        (Missing)
21 base.odex        (Missing)
22 linker           (Missing)
23 base.odex        (Missing)
24 linker           (Missing)
25 libc++_shared.so operator delete[](void*, unsigned int, std::align_val_t) (stdlib_new_delete.cpp:259)
26 linker           (Missing)
27 base.odex        (Missing)
28 libdl.so         (Missing)
29 libc++_shared.so libunwind::UnwindCursor<libunwind::LocalAddressSpace, libunwind::Registers_arm>::setInfoBasedOnIPRegister(bool)
30 libc++_shared.so libunwind::UnwindCursor<libunwind::LocalAddressSpace, libunwind::Registers_arm>::step()
31 libc++_shared.so __aeabi_unwind_cpp_pr0
32 libDomino.so     ax::Director::queueOperation(void (*)(void*), void*) (Director.cpp:1531)
33 libDomino.so     ax::Director::mainLoop() (Director.cpp:1565)
34 libc++_shared.so _Unwind_VRS_Pop
35 libDomino.so     (Missing)
36 libc++_shared.so _Unwind_RaiseException
37 libc++_shared.so (Missing)
38 libc++_shared.so (Missing)
39 base.odex        (Missing)
40 libc++_shared.so (Missing)
41 base.odex        (Missing)
42 base.odex        (Missing)
43 libc++_shared.so _Unwind_RaiseException
44 libc++_shared.so _Unwind_RaiseException
45 libc++_shared.so (Missing)
46 libc++_shared.so (Missing)
47 libc++_shared.so operator delete[](void*, unsigned int, std::align_val_t) (stdlib_new_delete.cpp:259)
48 libc.so          (Missing)

@Arvant Triple backtick characters before after should format it for you. The single backtick before and after is for a single line of code.

Our Axmol-based game is encountering a SIGABRT crash on Android devices. The crash appears to occur within the ax::Director::mainLoop function. Below are the details of the crash report and the steps we have taken to debug the issue.

Every single crash will most likely show the ax::Director::mainLoop in the call stack, since it runs the game loop, but it is unlikely, if ever, to have anything to do with the crash.
One thing does stick out though, which is this: operator delete[]
Deleting an object, and then attempting to delete it again, may result in SIGABRT.

i find this function queueOperation on sound engine and the error happen most on old devices.

i find this function queueOperation on sound engine and the error happen most on old devices.

Sorry, I don't understand what you mean by that.
Are you saying that you use the queueOperation method? If yes, then please show how it is being used.

i find this function queueOperation on sound engine and the error happen most on old devices.

I see now, it's being called by alcReopenDeviceOnAxmolThread and _onALEvent, but on Android it would most likely be _onALEvent:
bool AudioEngineImpl::init()
{
...
...
            auto alEventControlSOFTProc  = (LPALEVENTCONTROLSOFT)alGetProcAddress("alEventControlSOFT");
            auto alEventCallbackSOFTProc = (LPALEVENTCALLBACKSOFT)alGetProcAddress("alEventCallbackSOFT");
            if (alEventControlSOFTProc && alEventCallbackSOFTProc)
            {
                // Enable receiving disconnection events
                ALenum event = AL_EVENT_TYPE_DISCONNECTED_SOFT;
                alEventControlSOFTProc(1, &event, AL_TRUE);
                // Set callback
                alEventCallbackSOFTProc(_onALEvent, this);
            }
...
...
}

Does the crash happen when the audio is disconnected in some way? For example, a bluetooth device being connected or disconnected.

this crash happens on queueOperation  function. this function just called on AudioEngineImpl.cpp at line 246 when the audio source is changed.

this crash happens on queueOperation function. this function just called on AudioEngineImpl.cpp at line 246 when the audio source is changed.

Have you managed to reproduce it by attempting to cause the audio device trigger the disconnect event?

i checked it on pc and test this event but no crash happened  on change audio source and try test it  on my android device but also not happened. but in my crash report i get more than 2000 crash for that on this week.

i checked it on pc and test this event but no crash happened on change audio source and try test it on my android device but also not happened. but in my crash report i get more than 2000 crash for that on this week.

Windows goes via a different path, using event ALC_EVENT_TYPE_DEFAULT_DEVICE_CHANGED_SOFT rather than AL_EVENT_TYPE_DISCONNECTED_SOFT.
You need to test this using an Android device.


add ssh key to consul/vault nodes

@chrismatteson I couldn't ssh to the vault and consul nodes to see at the configs, so I added ssh key_pair to the nodes.

I left this off intentionally as I think it violates the security model.


Is there a limit to the number of words that can be merged for a merge filed

Hi All,
I have merged 1500 Chinese characters into a merge filed, but only about 1250 Chinese characters are displayed. Is there a limit to the number of words that can be merged for a merge filed?
Thanks a million

Probably a Word limitation. You can try to work around this by using 2 fields and splitting the text into two parts.


Error "could not find class 'android.support.v4.app.FragmentActivity'  while launching activity

From<EMAIL_ADDRESS>on February 14, 2013 21:09:42
What steps will reproduce the problem? 1. Create a dummy AUT android native app with one activity.
2. Create the test robotium project
3. In the test project add "AndroidTestUtility.jar" file (attached here) in libs folder along with robotium 3.6 jar file.
4. The "AndroidTestUtility.jar" handles the setup and tear down by itself.
5. Add similar cade as below:
private static final String TARGET_PACKAGE_ID ="com.sap.esm.retail";
private static final String LAUNCHER_ACTIVITY_FULL_CLASSNAME2 ="com.sap.esm.retail.MobileRetailingActivity"; // First Activity

 public InitialTest() throws ClassNotFoundException {
			super(TARGET_PACKAGE_ID,LAUNCHER_ACTIVITY_FULL_CLASSNAME2);

 }

Add a sample test method that the robotium would execute.
Run the android junit test.
Error message in IDE shown (screenshot "ErrorScreenshotIDE.jpg") and error message in logcat shown (screenshot "ErrorLogScreenshot.JPG").
I have attached the logcat file content also. What is the expected output? What do you see instead? The test framework "AndroidTestUtility" is extended by the main class of the test roboiutm project. What version of the product are you using? On what operating system? Windows 7 64bit, Android ADT bundle, Robotium 3.6 (checked on 3.4.1 also), Target: Android API 15 Please provide any additional information below. Can anyone help me here ?

Thanks and Regards,
Abhishek
Attachment: ErrorLogScreenshot.JPG ErrorScreenshotIDE.jpg Entirelog.txt AndroidTestUtility.jar
Original issue: http://code.google.com/p/robotium/issues/detail?id=391

From<EMAIL_ADDRESS>on February 14, 2013 21:21:56
The expected output should have been the launch of the activity and the subsequent automated steps for it.
Actualt output is that the activity is launched but no automated steps (solo statements) are executed. The error message is shown in the attached screenshots.

From<EMAIL_ADDRESS>on February 14, 2013 22:26:13
The logcat file Entirelog.txt can be downloaded from : http://www.2shared.com/document/JvKoOJL5/Entirelog.html

From<EMAIL_ADDRESS>on February 15, 2013 06:20:46
You need to export the Robotium jar in the java build path. Please see "Common Robotium issues" in the tutorials wiki page.
Status: Invalid
Owner<EMAIL_ADDRESS>


Kellerkinder

Adding kellerkinder elephpant to the field guide
2 pictures
description and site

Looks like this PR covers elephants added in #143 and #144


Profile-card-{Harsh Pathak}

Profile-card{Harsh Pathak}

updated readme.md file

Your attached website link is not working. Can your check it once again?


Allow to publish views like config, migrations

Awesome work !
I would like to customize a bit the UI, like adding custom panels that redirect to somewhere else but i can't find out how to modify these views as i can not publish them ?
Is this a wanted behavior ?
Félix

Not sure if this is a wanted behavior, but I'll leave this open in case anyone else has a similar opinion about it. If you want to add/view the templates directly that make up the admin panel for Larametrics, you can see them all in vendor/aschmelyun/larametrics/resources/views.


How can I use multi_gpu in tf.keras?

How can I use multi_gpu in tf.keras?
WARNING:tensorflow:From train_conv.py:214: multi_gpu_model (from tensorflow.keras.utils.multi_gpu_utils) is deprecated and will be removed after 2020-04-01.
And in the documentation, the tf.keras.utils.multi_gpu_model has been removed.

@llfzllfz,
You can use Distribution Strategy as a replacement for multi_gpu_model.  Please find the Tutorial for more information. Thanks!


Add BDD for Restock an Inventory Item

Add BDD scenario for restocking an inventory item
Add BDD scenario for not restocking an inventory item which is already above the restock level

@rachitjain2706 the restock scenarios work as expected. Could you resolve the conflicts?


Incorrectly assumed encoding when accessing content with project.resources.text.fromUri on Windows

Expected Behavior
Fetch text file in proper UTF-8 encoding.
Current Behavior
Gradle downloads resources from the remote location in incorrect encoding — probably falls back to the OS default encoding instead of the one provided by the server or UTF-8.
Context (optional)
No response
Steps to Reproduce
Create and run the following task:
tasks {
    register<Sync>("testEncoding") {
from(project.resources.text.fromUri("https://gist.githubusercontent.com/hsz/5bf5c84a6a2e36f2a6716f28f287783c/raw/b0ba02f53f5937643701b848cb3728b0f3dfb144/updates.xml"))
        into(temporaryDir)
    }
}

This task should create the build/tmp/testEncoding/wrappedInternalText....txt file with the following content:
Here's a bullet character: •

Instead, it provides a file with:
Here's a bullet character: �

Gradle version
8.1.1
Build scan URL (optional)
Windows 11
Your Environment (optional)
No response

Just with the project.resources.text.fromUri("...").asFile("UTF-8"), the content is resolved in the expected encoding.

Thank you for providing a valid reproducer.
The issue is in the backlog of the relevant team, but the existence of a workaround makes it non-critical, so it might take a while before a fix is made.

Please also see https://github.com/gradle/gradle/issues/25237


If a storage account secret was given for reading - should remove an existing sas - as this one might fail the reading - or suggest it to user

when reading and there is already secrets on blob storage account and user provides sas we should consider unseting the secret
Or should we consider trying both ? (as in removing one first and try with the other)
Maybe will be better to not setting another one if secret already exists

not a problem

done here
https://github.com/Azure/azure-kusto-spark/pull/102


Orchestration - Services/Instances are not able create after installation

Is this a BUG REPORT or FEATURE REQUEST?:

Uncomment only one, leave it on its own line:
/kind bug
What happened:

Follow the ansible steps for installng Opensds including Orchestration
After successful installation, not able to create services/instances either through dashboard or postman
Developer need to do some manual changes to do the above
What you expected to happen:
No Manual intervetion requied to do Orchestration operations
How to reproduce it (as minimally and precisely as possible):

Anything else we need to know?:
Environment:

Orchestration(release/branch) version:
OS (e.g. from /etc/os-release):
Kernel (e.g. uname -a):
Install tools:
Others:

Moved to installer. /assign @joseph-v

PRs PR #202 and PR #204 should fix this

After  #204 PR merged its listing all the Services..


[zkapps] add new test for large circuits

Adding a test to check that creating a circuit too large in pickles throws an exception.
The exception:
Uncaught exception:

  (monitor.ml.Error (Failure vector_get)
   ("Raised by primitive operation at Kimchi_pasta_basic.Fp in file \"src/lib/crypto/kimchi_backend/pasta/basic/kimchi_pasta_basic.ml\", line 79, characters 24-140"
    "Called from Kimchi_backend_common__Plonk_constraint_system.Make.compute_witness.(fun) in file \"src/lib/crypto/kimchi_backend/common/plonk_constraint_system.ml\", line 461, characters 43-62"
    "Called from Stdlib__Array.iteri in file \"array.ml\", line 129, characters 31-51"
    "Called from Base__List.iteri.(fun) in file \"src/list.ml\", line 544, characters 7-12"
    "Called from Stdlib__List.fold_left in file \"list.ml\", line 121, characters 24-34"
    "Called from Base__List0.fold in file \"src/list0.ml\" (inlined), line 21, characters 22-52"
    "Called from Base__List.iteri in file \"src/list.ml\", line 543, characters 5-62"
    "Called from Kimchi_backend_common__Plonk_constraint_system.Make.compute_witness in file \"src/lib/crypto/kimchi_backend/common/plonk_constraint_system.ml\", line 454, characters 4-597"
    "Called from Kimchi_pasta__Vesta_based_plonk.Proof.Backend.create_aux in file \"src/lib/crypto/kimchi_backend/pasta/vesta_based_plonk.ml\", line 102, characters 8-68"
    "Called from Kimchi_backend_common__Plonk_dlog_proof.Make.create_async in file \"src/lib/crypto/kimchi_backend/common/plonk_dlog_proof.ml\", line 393, characters 6-70"
    "Called from Pickles__Step.Make.f.(fun) in file \"src/lib/pickles/step.ml\", line 746, characters 22-266"
    "Called from Snarky_backendless__Snark0.Run.Make_basic.finalize_is_running in file \"src/lib/snarky/src/base/snark0.ml\", line 1627, characters 14-18"
    "Called from Pickles__Step.Make.f.(fun) in file \"src/lib/pickles/step.ml\", line 729, characters 6-1023"
    "Called from Pickles.Make_str.Make.compile.f.wrap in file \"src/lib/pickles/pickles.ml\", line 811, characters 14-59"
    "Called from Pickles.Make_str.compile.adjust_provers.(fun) in file \"src/lib/pickles/pickles.ml\", line 1178, characters 32-62"
    "Called from Zkapps_examples.compile.go.prover in file \"src/lib/zkapps_examples/zkapps_examples.ml\", line 578, characters 14-32"
    "Called from Async_kernel__Monitor.Exported_for_scheduler.schedule'.upon_work_fill_i in file \"src/monitor.ml\", line 295, characters 42-51"
    "Called from Async_kernel__Job_queue.run_job in file \"src/job_queue.ml\" (inlined), line 128, characters 2-5"
    "Called from Async_kernel__Job_queue.run_jobs in file \"src/job_queue.ml\", line 169, characters 6-47"
    "Caught by monitor block_on_async"))

Raised at Base__Result.ok_exn in file "src/result.ml" (inlined), line 201, characters 17-26
Called from Async_unix__Thread_safe.block_on_async_exn in file "src/thread_safe.ml", line 132, characters 29-63
Called from Async_unix__Thread_safe.block_on_async_exn in file "src/thread_safe.ml" (inlined), line 181, characters 27-54
Called from Dune__exe__Big_circuit.wrapped in file "src/lib/transaction_snark/test/zkapps_examples/big_circuit/big_circuit.ml", line 59, characters 6-49

Looks like I'm getting the exception even with smaller circuit size, investigating why

!ci-build-me

!ci-build-me

we managed to trigger the error:
❯ dune test ./src/lib/transaction_snark/test/zkapps_examples/big_circuit -f
ld: warning: directory not found for option '-L/opt/local/lib'
File "src/lib/transaction_snark/test/zkapps_examples/big_circuit/dune", line 2, characters 7-18:
2 |  (name big_circuit)
           ^^^^^^^^^^^
Transaction_snark.system: 28.599s
Uncaught exception:

  (monitor.ml.Error
   (Failure
     "Constraint unsatisfied (unreduced):\
    \nfield_inverse\
    \nChecked.inv\
    \nassert_non_zero: File \"src/lib/snarky/src/base/utils.ml\", line 185, characters 2-106\
    \nany: File \"src/lib/snarky/src/base/utils.ml\", line 403, characters 6-97\
    \nFile \"src/lib/pickles/step_main.ml\", line 39, characters 15-22\
    \nprevs_verified\
    \nstep_main\
    \n\
    \nConstraint:\
    \n((basic(R1CS(Add(Add(Add(Add(Add(Add(Add(Add(Add(Add(Add(Add(Add(Add(Add(Var 31452)(Var 31450))(Var 31448))(Var 31446))(Var 31444))(Var 31442))(Var 31440))(Var 31438))(Var 31436))(Var 31434))(Var 31432))(Var 31430))(Var 31428))(Var 31426))(Var 31424))(Var 31422))(Var 31454)(Constant 1)))(annotation(field_inverse)))\
    \nData:\
    \nR1CS 0 0 1")
   ("Raised at Stdlib.failwith in file \"stdlib.ml\", line 29, characters 17-33"
    "Called from Snarky_backendless__Checked_runner.Make_checked.add_constraint.(fun) in file \"src/lib/snarky/src/base/checked_runner.ml\", line 224, characters 12-399"
    "Called from Snarky_backendless__Checked_runner.Simple.map.(fun) in file \"src/lib/snarky/src/base/checked_runner.ml\", line 58, characters 29-32"
    "Called from Snarky_backendless__Checked_runner.Make_checked.with_label.(fun) in file \"src/lib/snarky/src/base/checked_runner.ml\", line 177, characters 20-77"
    "Called from Snarky_backendless__Checked_runner.Simple.map.(fun) in file \"src/lib/snarky/src/base/checked_runner.ml\", line 58, characters 29-32"
    "Called from Snarky_backendless__Checked_runner.Make_checked.with_label.(fun) in file \"src/lib/snarky/src/base/checked_runner.ml\", line 177, characters 20-77"
    "Called from Snarky_backendless__Checked_runner.Make_checked.with_label.(fun) in file \"src/lib/snarky/src/base/checked_runner.ml\", line 177, characters 20-77"
    "Called from Snarky_backendless__Snark0.Run.Make_basic.run in file \"src/lib/snarky/src/base/snark0.ml\", line 721, characters 26-51"
    "Called from One_hot_vector.Make.of_index in file \"src/lib/pickles/one_hot_vector/one_hot_vector.ml\", line 24, characters 4-41"
    "Called from Pickles__Step_verifier.Make.side_loaded_domain.(fun).domain in file \"src/lib/pickles/step_verifier.ml\", line 688, characters 11-45"
    "Called from Pickles__Step_verifier.Make.finalize_other_proof.(fun) in file \"src/lib/pickles/step_verifier.ml\", line 915, characters 12-65"
    "Called from Snarky_backendless__Snark0.Run.Make_basic.with_label in file \"src/lib/snarky/src/base/snark0.ml\", line 1233, characters 14-18"
    "Called from Pickles__Step_main.verify_one in file \"src/lib/pickles/step_main.ml\", line 39, characters 4-585"
    "Called from Pickles__Step_main.step_main.main.(fun).go in file \"src/lib/pickles/step_main.ml\", line 451, characters 22-115"
    "Called from Pickles__Step_main.step_main.main.(fun) in file \"src/lib/pickles/step_main.ml\", line 511, characters 16-160"
    "Called from Snarky_backendless__Snark0.Run.Make_basic.with_label in file \"src/lib/snarky/src/base/snark0.ml\", line 1233, characters 14-18"
    "Called from Pickles__Step_main.step_main.main.(fun) in file \"src/lib/pickles/step_main.ml\", line 383, characters 10-1023"
    "Called from Snarky_backendless__Snark0.Run.Make_basic.with_label in file \"src/lib/snarky/src/base/snark0.ml\", line 1233, characters 14-18"
    "Called from Pickles__Step.Make.f.(fun) in file \"src/lib/pickles/step.ml\", line 756, characters 42-77"
    "Called from Snarky_backendless__Snark0.Run.Make_basic.handle in file \"src/lib/snarky/src/base/snark0.ml\", line 1217, characters 14-18"
    "Called from Snarky_backendless__Snark0.Run.Make_basic.mark_active in file \"src/lib/snarky/src/base/snark0.ml\", line 1147, characters 18-22"
    "Re-raised at Snarky_backendless__Snark0.Run.Make_basic.mark_active in file \"src/lib/snarky/src/base/snark0.ml\", line 1152, characters 8-17"
    "Called from Snarky_backendless__Snark0.Run.Make_basic.as_stateful in file \"src/lib/snarky/src/base/snark0.ml\", line 727, characters 14-18"
    "Called from Snarky_backendless__Runners.Make.auxiliary_input.(fun) in file \"src/lib/snarky/src/base/runners.ml\", line 84, characters 21-33"
    "Called from Snarky_backendless__Runners.Make.Run.generate_witness_conv.(fun) in file \"src/lib/snarky/src/base/runners.ml\", line 312, characters 12-98"
    "Called from Snarky_backendless__Snark0.Run.Make_basic.finalize_is_running in file \"src/lib/snarky/src/base/snark0.ml\", line 1247, characters 14-18"
    "Called from Pickles__Step.Make.f.(fun) in file \"src/lib/pickles/step.ml\", line 729, characters 6-1023"
    "Called from Pickles.Make_str.Make.compile.f.wrap in file \"src/lib/pickles/pickles.ml\", line 811, characters 14-59"
    "Called from Pickles.Make_str.compile.adjust_provers.(fun) in file \"src/lib/pickles/pickles.ml\", line 1178, characters 32-62"
    "Called from Transaction_snark.Make_str.Make.of_zkapp_command_segment_exn in file \"src/lib/transaction_snark/transaction_snark.ml\", line 4155, characters 16-125"
    "Called from Async_kernel__Monitor.Exported_for_scheduler.schedule'.upon_work_fill_i in file \"src/monitor.ml\", line 295, characters 42-51"
    "Called from Async_kernel__Job_queue.run_job in file \"src/job_queue.ml\" (inlined), line 128, characters 2-5"
    "Called from Async_kernel__Job_queue.run_jobs in file \"src/job_queue.ml\", line 169, characters 6-47"))

Raised at Base__Result.ok_exn in file "src/result.ml" (inlined), line 201, characters 17-26
Called from Async_unix__Thread_safe.block_on_async_exn in file "src/thread_safe.ml", line 132, characters 29-63
Called from Async_unix__Thread_safe.block_on_async_exn in file "src/thread_safe.ml" (inlined), line 181, characters 27-54
Called from Dune__exe__Big_circuit.(fun) in file "src/lib/transaction_snark/test/zkapps_examples/big_circuit/big_circuit.ml", line 155, characters 28-70
Called from Mina_ledger__Ledger.Ledger_inner.with_ledger in file "src/lib/mina_ledger/ledger.ml", line 250, characters 19-27
Re-raised at Mina_ledger__Ledger.Ledger_inner.with_ledger in file "src/lib/mina_ledger/ledger.ml", line 252, characters 31-40
Called from Dune__exe__Big_circuit in file "src/lib/transaction_snark/test/zkapps_examples/big_circuit/big_circuit.ml", line 144, characters 2-506

!ci-build-me

!ci-build-me

Looks like we might want a better formatting in the string that's in the Failure exception, right?

not sure what you're talking about : o


RD2: Query all fields in RD Custom Field Mapping

As an ISV partner or customer, I need to be able to enable custom field mapping for any field, including large text area fields over 4K in size, to support special integrations and business use cases
Critical Changes
Changes
Issues Closed

Fixes #5674

Community Ideas Delivered
New Metadata
Deleted Metadata

Related To PR: W-7814340
Related To PR: W-7832089

@RadSFDeveloper The code is ready for review

@RadSFDeveloper The code is updated and is ready for second review

@force2b The fields are added and is ready for review