A couple fixes for dynamically expanding containers with a FlowLayout

Fixed calculations to determine the measuredHeight property, and added a fix to invalidate the layoutTarget's size

Danny,
Geez.  I've been away from Flex for much of 2011, but I'll be back in 2012
(just in time to watch it wither and die, thanks Adobe!).  One of the first
things I'm planning in January is to get this pulled in, tested, and merged.
Sorry I've been such a bum, but I just wanted to say thank you for your
work on this patch, and the making the effort to send a pull request.  I
promise I'll get it in soon.
-Justin
<EMAIL_ADDRESS>On Fri, Nov 4, 2011 at 4:51 PM, Danny Kopping <
<EMAIL_ADDRESS>
Fixed calculations to determine the measuredHeight property, and added a
fix to invalidate the layoutTarget's size
You can merge this Pull Request by running:
git pull https://github.com/dannykopping/flexlayouts master
Or you can view, comment on it, or merge it online at:
https://github.com/saturnboy/flexlayouts/pull/4
-- Commit Summary --

Fixed calculations to determine the measuredHeight property, and added a
fix to invalidate the layoutTarget's size

-- File Changes --
M libs/flexlayouts/src/org/flexlayouts/layouts/FlowLayout.as (53)
-- Patch Links --
https://github.com/saturnboy/flexlayouts/pull/4.patch
https://github.com/saturnboy/flexlayouts/pull/4.diff

Reply to this email directly or view it on GitHub:
https://github.com/saturnboy/flexlayouts/pull/4

Hey Justin
Haha, no worries mate - yup, Flex is being put out to die like a beautiful
stallion with a broken leg. I feel like Adobe raped the whole community
with a pineapple (ok, those are my two weird metaphors for the day.)
If you don't mind my asking, i've been doing an informal survey of the Flex
devs I know to find out in which direction they're going... What ship have
you jumped to?
Cheers
Danny Kopping Flash Platform Consultant
Adobe Certified Expert (Flash, Flex, AIR)
http://dannykopping.com/http://dannykopping.com | (+27) 84 941 4422
On Tue, Dec 20, 2011 at 12:28 AM, Justin Shacklette <
<EMAIL_ADDRESS>
wrote:

Danny,
Geez.  I've been away from Flex for much of 2011, but I'll be back in 2012
(just in time to watch it wither and die, thanks Adobe!).  One of the first
things I'm planning in January is to get this pulled in, tested, and
merged.
Sorry I've been such a bum, but I just wanted to say thank you for your
work on this patch, and the making the effort to send a pull request.  I
promise I'll get it in soon.
-Justin
<EMAIL_ADDRESS>On Fri, Nov 4, 2011 at 4:51 PM, Danny Kopping <
<EMAIL_ADDRESS>
Fixed calculations to determine the measuredHeight property, and added a
fix to invalidate the layoutTarget's size
You can merge this Pull Request by running:
git pull https://github.com/dannykopping/flexlayouts master
Or you can view, comment on it, or merge it online at:
https://github.com/saturnboy/flexlayouts/pull/4
-- Commit Summary --

Fixed calculations to determine the measuredHeight property, and added
a
fix to invalidate the layoutTarget's size

-- File Changes --
M libs/flexlayouts/src/org/flexlayouts/layouts/FlowLayout.as (53)
-- Patch Links --
https://github.com/saturnboy/flexlayouts/pull/4.patch
https://github.com/saturnboy/flexlayouts/pull/4.diff

Reply to this email directly or view it on GitHub:
https://github.com/saturnboy/flexlayouts/pull/4

Reply to this email directly or view it on GitHub:
https://github.com/saturnboy/flexlayouts/pull/4#issuecomment-3210764

Danny Kopping Flash Platform Consultant
Adobe Certified Expert (Flash, Flex, AIR)
http://dannykopping.com/http://dannykopping.com | (+27) 84 941 4422

Danny,
As part of my job I actually did a month long research project into mobile
options (html5, flex, phonegap, appcelerator, unity, C++ everywhere,
native).
And honestly, we concluded write it twice (once for iOS, once for Android).
  Might as well write it native from the beginning, do things the
native way, test things the native way, and just be happy.   If you do
something especially painful, like image processing or speech recognition,
you can always do it once in C and wrap it with the NDK for Android.
 I feel like Adobe raped the whole community
with a pineapple (ok, those are my two weird metaphors for the day.)
If you don't mind my asking, i've been doing an informal survey of the Flex
devs I know to find out in which direction they're going... What ship have
you jumped to?
Cheers
Danny Kopping Flash Platform Consultant
Adobe Certified Expert (Flash, Flex, AIR)
http://dannykopping.com/http://dannykopping.com | (+27) 84 941 4422
On Tue, Dec 20, 2011 at 12:28 AM, Justin Shacklette <
<EMAIL_ADDRESS>
wrote:

Danny,
Geez.  I've been away from Flex for much of 2011, but I'll be back in
2012
(just in time to watch it wither and die, thanks Adobe!).  One of the
first
things I'm planning in January is to get this pulled in, tested, and
merged.
Sorry I've been such a bum, but I just wanted to say thank you for your
work on this patch, and the making the effort to send a pull request.  I
promise I'll get it in soon.
-Justin
<EMAIL_ADDRESS>On Fri, Nov 4, 2011 at 4:51 PM, Danny Kopping <
<EMAIL_ADDRESS>
Fixed calculations to determine the measuredHeight property, and added
a
fix to invalidate the layoutTarget's size
You can merge this Pull Request by running:
git pull https://github.com/dannykopping/flexlayouts master
Or you can view, comment on it, or merge it online at:
https://github.com/saturnboy/flexlayouts/pull/4
-- Commit Summary --

Fixed calculations to determine the measuredHeight property, and
added
a
fix to invalidate the layoutTarget's size

-- File Changes --
M libs/flexlayouts/src/org/flexlayouts/layouts/FlowLayout.as (53)
-- Patch Links --
https://github.com/saturnboy/flexlayouts/pull/4.patch
https://github.com/saturnboy/flexlayouts/pull/4.diff

Reply to this email directly or view it on GitHub:
https://github.com/saturnboy/flexlayouts/pull/4

Reply to this email directly or view it on GitHub:
https://github.com/saturnboy/flexlayouts/pull/4#issuecomment-3210764

Danny Kopping Flash Platform Consultant
Adobe Certified Expert (Flash, Flex, AIR)
http://dannykopping.com/http://dannykopping.com | (+27) 84 941 4422

Reply to this email directly or view it on GitHub:
https://github.com/saturnboy/flexlayouts/pull/4#issuecomment-3210895

Hey man
Cool, yeah a lot of people seem to be going that route. Personally, I'm
moving exclusively to the backend (PHP, MySQL, REST APIs) and outsourcing
the frontend because it's a) too much effort to get anything done right and
b) the game is changing too fast.
What's your company called?
If I come across a project that requires the skillset you described, I may
hit you up for a quote :)
Cheers
Danny Kopping Flash Platform Consultant
Adobe Certified Expert (Flash, Flex, AIR)
http://dannykopping.com/http://dannykopping.com | (+27) 84 941 4422
On Tue, Dec 20, 2011 at 1:07 AM, Justin Shacklette <
<EMAIL_ADDRESS>
wrote:

Danny,
As part of my job I actually did a month long research project into mobile
options (html5, flex, phonegap, appcelerator, unity, C++ everywhere,
native).
And honestly, we concluded write it twice (once for iOS, once for Android).
  Might as well write it native from the beginning, do things the
native way, test things the native way, and just be happy.   If you do
something especially painful, like image processing or speech recognition,
you can always do it once in C and wrap it with the NDK for Android.
 I feel like Adobe raped the whole community
with a pineapple (ok, those are my two weird metaphors for the day.)
If you don't mind my asking, i've been doing an informal survey of the
Flex
devs I know to find out in which direction they're going... What ship
have
you jumped to?
Cheers
Danny Kopping Flash Platform Consultant
Adobe Certified Expert (Flash, Flex, AIR)
http://dannykopping.com/http://dannykopping.com | (+27) 84 941 4422
On Tue, Dec 20, 2011 at 12:28 AM, Justin Shacklette <
<EMAIL_ADDRESS>
wrote:

Danny,
Geez.  I've been away from Flex for much of 2011, but I'll be back in
2012
(just in time to watch it wither and die, thanks Adobe!).  One of the
first
things I'm planning in January is to get this pulled in, tested, and
merged.
Sorry I've been such a bum, but I just wanted to say thank you for your
work on this patch, and the making the effort to send a pull request.
I
promise I'll get it in soon.
-Justin
<EMAIL_ADDRESS>On Fri, Nov 4, 2011 at 4:51 PM, Danny Kopping <
<EMAIL_ADDRESS>
Fixed calculations to determine the measuredHeight property, and
added
a
fix to invalidate the layoutTarget's size
You can merge this Pull Request by running:
git pull https://github.com/dannykopping/flexlayouts master
Or you can view, comment on it, or merge it online at:
https://github.com/saturnboy/flexlayouts/pull/4
-- Commit Summary --

Fixed calculations to determine the measuredHeight property, and
added
a
fix to invalidate the layoutTarget's size

-- File Changes --
M libs/flexlayouts/src/org/flexlayouts/layouts/FlowLayout.as (53)
-- Patch Links --
https://github.com/saturnboy/flexlayouts/pull/4.patch
https://github.com/saturnboy/flexlayouts/pull/4.diff

Reply to this email directly or view it on GitHub:
https://github.com/saturnboy/flexlayouts/pull/4

Reply to this email directly or view it on GitHub:
https://github.com/saturnboy/flexlayouts/pull/4#issuecomment-3210764

Danny Kopping Flash Platform Consultant
Adobe Certified Expert (Flash, Flex, AIR)
http://dannykopping.com/http://dannykopping.com | (+27) 84 941 4422

Reply to this email directly or view it on GitHub:
https://github.com/saturnboy/flexlayouts/pull/4#issuecomment-3210895

Reply to this email directly or view it on GitHub:
https://github.com/saturnboy/flexlayouts/pull/4#issuecomment-3211302

Danny Kopping Flash Platform Consultant
Adobe Certified Expert (Flash, Flex, AIR)
http://dannykopping.com/http://dannykopping.com | (+27) 84 941 4422

Danny,
I did the research for Rosetta Stone (they make language learning
software), but I'm going to work for Gorilla Logic in January.  They do
Flex, iOS, Android consulting, and also make some nice open source testing
tools (FlexMonkey and FoneMonkey).
-Justin
<EMAIL_ADDRESS>On Mon, Dec 19, 2011 at 4:10 PM, Danny Kopping <
<EMAIL_ADDRESS>
wrote:

Hey man
Cool, yeah a lot of people seem to be going that route. Personally, I'm
moving exclusively to the backend (PHP, MySQL, REST APIs) and outsourcing
the frontend because it's a) too much effort to get anything done right and
b) the game is changing too fast.
What's your company called?
If I come across a project that requires the skillset you described, I may
hit you up for a quote :)

Great! I'm familiar with both those companies actually
Thanks Justin - I'll keep you guys in mind. We're based in South Africa and
there aren't a lot of guys offering both Android and iOS development
in-house.
Cheers
Danny Kopping Flash Platform Consultant
Adobe Certified Expert (Flash, Flex, AIR)
http://dannykopping.com/http://dannykopping.com | (+27) 84 941 4422
On Tue, Dec 20, 2011 at 1:28 AM, Justin Shacklette <
<EMAIL_ADDRESS>
wrote:

Danny,
I did the research for Rosetta Stone (they make language learning
software), but I'm going to work for Gorilla Logic in January.  They do
Flex, iOS, Android consulting, and also make some nice open source testing
tools (FlexMonkey and FoneMonkey).
-Justin
<EMAIL_ADDRESS>On Mon, Dec 19, 2011 at 4:10 PM, Danny Kopping <
<EMAIL_ADDRESS>
wrote:

Hey man
Cool, yeah a lot of people seem to be going that route. Personally, I'm
moving exclusively to the backend (PHP, MySQL, REST APIs) and outsourcing
the frontend because it's a) too much effort to get anything done right
and
b) the game is changing too fast.
What's your company called?
If I come across a project that requires the skillset you described, I
may
hit you up for a quote :)

Reply to this email directly or view it on GitHub:
https://github.com/saturnboy/flexlayouts/pull/4#issuecomment-3211569

Danny Kopping Flash Platform Consultant
Adobe Certified Expert (Flash, Flex, AIR)
http://dannykopping.com/http://dannykopping.com | (+27) 84 941 4422


New install - where is the QR code in Windows 11?

Ignore, I found it

Found, in HA notifications. D'oh!


Port the setup command tree from infludb

Port the setup command (and any subcommands) from the influxdb repo.

Replace the use of any hand-rolled HTTP clients with code-generated clients
Interesting business logic should "live" outside of the cmd/ module
Tests should also be ported / extended as part of this

After discussing with @russorat, I'm not going to port the (undocumented) influx setup user subcommand over to this repo. The subcommand's behavior is fully covered by other APIs/CLI commands, and it's never been publicly documented.

Ran into issues with our code generator while working on this. Pausing to work on #19


reorg documentation structure on website (docs.zephyrproject.org)

Right now and to be compatible with how documentation was before we migrated to docs.zephyrproject.org we have the latest docs in the root and all old releases in sub-directories on the same level. This is not optimal, because we not able to do a sync and remove old files that do not apply anymore, so changes are we will have some stale files if we move things around in the future.
Additionally, we want to put other information and documents in the same structure, so this becomes a bit unorganised.
One way we could resolve this would be to put the latest in a sub-directory as well and have 1 top level index page that links to the various documentation and information hosted, this however can't be a document that is maintained within the current documentation, it needs to be independent.
Other suggestions?

should be implemented now. Please take a look....


IsTrivialMove interacts poorly with Aggressive Delete [JIRA: RIAK-2232]

When aggressive delete feature identifies .sst file for grooming, IsTrivialMove can actually cause .sst file to migrate to highest level instead of running a compaction to clear out the data.  Likely need to create a flag in Compaction object that is tested in DBImpl::BackgroundCompaction() when deciding whether or not to use move operation.  A flag needs to be set in VersionSet::Finalize() then passed to Compaction object within VersionSet::PickCompaction().

addressed on [mv-expiry e5ffc55], db/version_set.cc and db/version_set.h changed.  To be included in first expiry pull request.

mv-expiry committed to develop on July 6, 2016


Horizontal support?

Coming soon (TM)

Version 2.0.0 is out. Introduces support for horizontal scroll.


core(devcontainer): move to docker-compose

Codecov Report

Merging #440 into master will increase coverage by 0.32%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #440      +/-   ##
==========================================
+ Coverage   61.45%   61.78%   +0.32%
==========================================
  Files          77       72       -5
  Lines        4483     4223     -260
==========================================
- Hits         2755     2609     -146
+ Misses       1509     1408     -101
+ Partials      219      206      -13

Impacted Files
Coverage Δ

pkg/stream/common.go
17.24% <0.00%> (-27.59%)
:arrow_down:

pkg/object/value.go
39.35% <0.00%> (-2.59%)
:arrow_down:

pkg/object/list.go
59.25% <0.00%> (-2.47%)
:arrow_down:

pkg/object/object.go
79.41% <0.00%> (-0.93%)
:arrow_down:

pkg/streammanager/streammanager.go
30.44% <0.00%> (ø)

internal/generator/pubsub/pubsub.go
100.00% <0.00%> (ø)

pkg/object/references_load.go

pkg/exchangemock/exchangemock_simple.go

pkg/object/references_unload.go

pkg/sqlobjectstore/filters.go

... and 11 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 7c185c9...851ff82. Read the comment docs.


Incorrect ecnIPv4DataLen on FreeBSD

#4059 added ECN support for Darwin (MacOS), FreeBSD and Linux but incorrectly configured ecnIPv4DataLen on FreeBSD.  Whilst 4 is the correct length for use with setsockopt(2), when used as a sendmsg control message, the length should be 1.  I can't find this documented anywhere but https://cgit.freebsd.org/src/tree/sys/netinet/udp_usrreq.c#n1144 shows that it checks for a length of 1.
 Would you mind submitting a PR?


Data is being dropped from Anchor claim reward transactions

This is an example of claiming rewards from the Anchor Govern tab. Inspecting the transaction in Terra Finder shows that data is being dropped regarding the reward value. Looks like the entire from_contract event log is missing.

Here is an example of an Anchor claim reward tx that contains the data:

This likely is not the right repo for this issue, as this seems like a data problem, so please transfer this issue to the proper repo if appropriate. Thank you.

This just happened to me again on another transaction to Claim Rewards from Anchor. Any update on this issue?


For a custom auth integration with Dexie cloud, how does one initiate a login from a server after user info has been collected?

(I posted this in discussions, but reposting here, as I'm not clear what the correct place is to post questions.)
I am currently looking at adding a custom auth integration and reviewing the example provided in the docs.
Based on the example, it appears the only way to initiate a login flow in Dexie Cloud is from the client, by calling db.cloud.login({ email<EMAIL_ADDRESS>which then results in a call from dexie cloud to the endpoint added in fetchTokens in cloud.config.
However, based on the provided example, it appears that the server needs to respond immediately to this call with a token, which does not allow for completion of a custom login flow. (Eg an OTP roundtrip.)
It seems there needs to be a "/login" endpoint that is equivalent to calling db.cloud.login({ email: "foo@bar.com"}), but allows a server to do so after the needed info (eg their email) has been collected in a custom flow.  Or am I not understanding how this should work?
Any clarifications would be appreciated!

sorry for late reply. Even if the fetchTokens callback expects a Promise of a TokenResponse in return, there's nothing requiring this to be returned immediately. You can implement any custom flow of multiple requests/responses between your client and server before returning a result. You could even redirect the user away from your app to a 3rd part login flow using oidconnect or similar (in that case there won't be any TokenResponse returned but it doesn't matter because the login flow would reddirect you back to your app when completed and att that time, you'd have a local endpoint that will have login information immediately this time.


Cfg target has atomic v1.6.0

The best way forward would be for the upstream PR to be completed in the end.
Apparently there was some progress here at the beginning of the year, and there doesn't seem to be that much left to do. But the original author seems to be too busy at the moment.
So, please try to create a new upstream PR based on https://github.com/tokio-rs/bytes/pull/467.
See the two comments:
https://github.com/tokio-rs/bytes/pull/467#issuecomment-1904026824
https://github.com/tokio-rs/bytes/pull/467#issuecomment-2180970767


allow BigInt?

There is a new primitive type implemented in Chrome browser (also in Node.js):
https://developers.google.com/web/updates/2018/05/bigint
so, when i compile
a = 123n

the result is
var a;
a = 123;

should this truncation be disabled or fixed? why does it exist in LiveScript? to avoid mis-typings?

We allow any suffix, i.e. 126_euros

It's a little clunky, but there's always a = ``123n``.

Also a change like this wouldn't be backwards compatible for all use cases such as 12_euros, 23cm & 560m etc.

i still dont get, why this numeric literal parse mechanism exist
for example:
a = 0xFFFFn # BigInt vs truncated
b = 0123 # 83 vs compile error
c = 0b01111111100000000000000000000000 #<PHONE_NUMBER> vs truncated
d = 1_euro # syntax error vs truncated

doesn't work like it's intended for JS. if it's not bound to anything, maybe it could be cut safely from livescript compiler, why not? what back-compatibility are you referencing?

It exists as a terser, easier-to-read way to add comments to code that uses a lot of numeric constants. For example:
seconds-in-day = 60 /* secs per min */ * 60 /* mins per hr */ * 24 /* hrs */
# or...
seconds-in-day = 60secs_per_min * 60mins_per_hr * 24hrs

Given that this feature exists, removing it could break code that has been using it.

oh my.. this is not functional at all, not real..
i vote with both hands to drop it in version 2.0! so those who made such comments will not be affected until compiler upgrade.

It doesn't seem any less real to me than any other form of comment in code. Either way, you have characters that the machine ignores but that humans reading the code will find helpful in explaining what's going on. It's just another option available to the programmer for documenting their thoughts, in a less obtrusive way than more general comment syntaxes.

the machine doesnt ignore those characters, for now it's 123n, but earlier it was 0b001 or 0123, as i showed in the example. the one, who ignores/truncates it is the compiler, so they never reach the machine. there are multiple ways to make comments and there should be a good reasoning about why one more method is required/appropriate.
this is a big limitation to primitives and a design mistake, imo. you will never be able to run ahead of engines interpreting native/raw values/literals. it happend in LESS compiler, when css variables appeared, as it tries to interpret any color function, it has to be written like ~'primitive', it's not convenient/better for human who follows engine/underlying language docs, it's worse.
the compromise solution could be some special suffix like _someComment

The medley of 0x, 0b, octal prefix 0, and suffix n is not exactly an inspiring, future-safe design for literal syntax, and one of LiveScript's (and its predecessors') goals is to paper over such historical bodges with a cleaner syntax. The first three were replaced in LiveScript with the general radix notation (16~FFFF, for example). BigInt is a new kind of thing for JavaScript, and in my opinion merits being thought about on its own terms.
The question you're implicitly raising seems to be whether there will be more BigInt-like ‘modes’ to number literals that get supported by JavaScript engines, and whether the popularity of such modes will be greater than the popularity of number comments in LiveScript, such that the latter should move out of the way to clear syntax space for the former. If not, there are possibly other ways we could notate BigInt-ness (and for now, as I said, ``123n`` will always work). But I think if JavaScript ended up taking a hard turn in the direction of C, and started differentiating such beasts as floats, doubles, and small ints, then reintroducing suffixes could make sense. I don't know how likely that is.
Most of the value of number comments could probably be preserved by requiring that they start with an underscore, I'll grant you that. So in a hypothetical release where we wanted to make some larger breaking changes, that's a compromise I'd consider, if there's no better way to notate BigInts for LiveScript. I'm not yet convinced that there isn't, though.

n~123 currently binds the  123  method of n.

Okay, i got your point. Think about the benefits - no need to write more docs and more code to support those "innovations". Maybe it's not inspiring, future-safe or well designed things, but the first responsibility lays on JS makers.


add labwc

Although labwc is still a work-in-progress, I think it's already stable enough to be added to the list (it seems to be more advanced than waybox from what I've seen, anyway).

Thank you!


Add Gin example

Addresses #63

Coverage remained the same at 93.955% when pulling bba697ba00f39fbe5e1dc544335ae5d5c83df2ee on circa10a:add-gin-example into<PHONE_NUMBER>097a6dc6fd830ac268a6f1953194f1 on gavv:master.

Thanks!

Please also squash commits after fixing all the issues.

LGTM, thanks!


Validate HHS Protect interface (STRAC, SAFE, SimpleReport)

The HHS Protect interface is new.  Test it thoroughly.  Very important to HHS.
Not a bug: QA task.
Dual pathing SAFE to verify everything gets there by Kevin in HHS Protect.
Additional context
Add any other context about the problem here.

Done.


fixed:Search box doesn't render well in small views

i have fixed issue #83  Search box doesn't render well in small views please add hacktobefest label

Can you show a screenshot with a mobile view

Ah mobile view is different. This fix seems OK but i think we might want to improve the search bar there- its super small on md. I think maybe we could use a shadcn command modal when someone clicks the box on small screens?

i think it seems perfect on mobile devices

@seveibar i also made dashboard responsive on mobile kindly review and merge

I would run "bun format" after editing, the first pr for a person on github i have to manually approve workflows

Hi @seveibar,
I sincerely apologize for the confusion caused. As a beginner, this is my first time contributing to a large open-source project, and I appreciate your understanding. Please kindly disregard any mistakes made.

No worries! Just letting you know the first pr is always tricky and you might have to ping me more often 😁

Haha, I think I’ve already pinged you more than 20 times! 😅 I really love your project and I’m excited to contribute more, especially with frontend bugs. You can definitely expect more pings from me!


Always specify :dimension when constructing a permutation-gate

I am not sure if this fix is semantically correct, but it fixed make test in the qvm repo :).
Note that there exists a wrapper function cl-quil:make-permutation-gate. As far as I can tell it's not used anywhere, but in theory the handful of individual calls to (make-instance 'permutation-gate ...) could be replaced with calls to make-permutation-gate, which handles setting :dimension for you.
Fixes rigetti/qvm#85

I imagine this would've been caught if the test coverage for src/gates.lisp wasn't poor.

What about :name? Ok to be nil in these cases, or should I provide a value in the call to make-instance?

Pushed a commit that initializes the permutation-gate's dimension based on the permutation's length in initialize-instance. Couple of follow-up comments.

I limited this change to permutation-gates, but it looks like simple-gate might also be a candidate? At least based on how simple-gates are used in the test suite, it seems to always be the case that

(with-slots (dimension matrix) some-simple-gate
  (= dimension (magicl:matrix-rows matrix) (magicl:matrix-cols matrix)))

I was too timid to tinker with the class hierarchy, especially since these types are part of the exported API, but maybe permutation-gates should not have a dimension slot, and instead just (defmethod gate-dimension ...) similar to controlled-gate, dagger-gate, etc. I gather from the fact that only :readers are defined for these slots that they are meant to be read-only after initialization, so I guess setfing the slot-values directly is a mostly theoretical concern, but it still seems a little awkward that permutation-gates now have a vestigial :dimension initarg.

I conditionally approve on there being a PR to fix QVM issues.

I have a QVM diff ready that was just waiting on approval here. I will open a PR later after a final rebase and testing.


Doc Feedback: Link Not Working

Location : https://is.docs.wso2.com/en/latest/compliance/electronic-identification-authentication-and-trust-services-regulation/

Link eIDAS SAML Attribute Profile Support via WSO2 Identity Server Not working.

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


feat: Ability to remove images that I know are being replaced

Describe the feature you'd like to request
If I am uploading an image to replace a previously uploaded image, I'd like to pass in the previous image URL so that this can be deleted since I know it will be replaced by the new image being uploaded.   This would ensure my file store usage remains low.
Describe the solution you'd like to see
On uploading a new image, I'd like to set a property of the image URL to delete when this image is successfully uploaded.
Additional information
No response
👨👧👦 Contributing

[ ] 🙋♂ Yes, I'd be down to file a PR implementing this feature!

We do have some plans to do things like this, for now though, you can pass that information to onUploadComplete() and call deleteFiles() there.
Something like this could work:
export const uploadRouter = {
  videoAndImage: f({
    image: {
      maxFileSize: "4MB",
      maxFileCount: 4,
      acl: "public-read",
    },
    video: {
      maxFileSize: "16MB",
    },
  })
    .input(z.object({
      prevFile: z.string().optional(),
    }))
    .middleware(({ req, input }) => {
      // check for auth
      const user = auth(req);
      if (!user)  throw new Error("Unauthorized");

      // Return some metadata to be stored with the file
      return { userId: user.id, input };
    })
    .onUploadComplete(async ({ file, metadata: { prevFile }}) => {
      if(prevFile) await utapi.deleteFiles(prevFile);
    }),
} satisfies FileRouter;


Feature/multiple instructions

Adding support for multi-instructions in the multisig

DO NOT merge or delete this branch yet. It needs to go together with other updates in the payment-streaming program.


Method to list states of given country?

Hello,
Is there any method to list the states of given country?

@Keerthi-harish Thanks for the idea. I will add this to the roadmap.

@Keerthi-harish Sorry for the delay. I updated the package with new methods. It is possible now to have access to states of a given country using: getStatesForCountry.


Technical/finish up package

compose
readme
workflow

Due to the reorganization of the ouster-lidar repositories (https://github.com/ouster-lidar), the Dockerfile and demo folder need the additional changes proposed in the next commit


add websocket

add webSocket to response, or corsify will fail.
RangeError: init["status"] must be in the range of 200 to 599, inclusive.
Web Socket request did not return status 101 Switching Protocols response with Web Socket

Early return works perfectly. PR closed.


Enable footnotes for README.md files

There's an actual missing feature still, looking at this diff — footnotes. Do you know if these are actually used in Markdown files parsed by docs.rs? They're not supported by GitHub's own README.md renderer, for what it's worth.

Footnotes are now a part of github-favored markdown[^1], is it now possible to add this feature to the docs?
[^1]:https://github.blog/changelog/2021-09-30-footnotes-now-supported-in-markdown-fields/
This mostly involves enabling in Comrak, where they're already supported.
Originally posted by @zStruCat in https://github.com/rust-lang/docs.rs/issues/117#issuecomment-1147559768

I'm confused. I've used footnotes in my rustdocs before and they're rendered on docs.rs: https://docs.rs/minecraft-assets/latest/minecraft_assets/versions/index.html#fn1.
What's the issue? Is it when you do #[doc = include_str!("README.md")]?

@BGR360 this is specifically for the README, which docs.rs renders even for crates which don't get documented by rustdoc (e.g. binaries): https://docs.rs/crate/bat/latest

This looks like the right part of the code: https://github.com/rust-lang/docs.rs/blob/a43b0fb8a9fecb68f92c8e348874e989e8e02687/src/web/markdown.rs#L51-L70

Ah, I see. Thanks.
@rustbot claim

I think the docs.rs build environment is busted on M1 macs. It's trying to run docker x86 docker containers.
running `Command { std: "docker" "create" "-v" "/Users/ben/Documents/Rust/rust-lang/docs.rs/.workspace/builds/essential-files-20221130-1.67.0-nightly-c97b539e4/target:/opt/rustwide/target:rw,Z" "-v" "/Users/ben/Documents/Rust/rust-lang/docs.rs/.workspace/builds/essential-files-20221130-1.67.0-nightly-c97b539e4/source:/opt/rustwide/workdir:ro,Z" "-v" "/Users/ben/Documents/Rust/rust-lang/docs.rs/.workspace/cargo-home:/opt/rustwide/cargo-home:ro,Z" "-v" "/Users/ben/Documents/Rust/rust-lang/docs.rs/.workspace/rustup-home:/opt/rustwide/rustup-home:ro,Z" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-e" "CARGO_TARGET_DIR=/opt/rustwide/target" "-e" "DOCS_RS=1" "-e" "CARGO_HOME=/opt/rustwide/cargo-home" "-e" "RUSTUP_HOME=/opt/rustwide/rustup-home" "-w" "/opt/rustwide/workdir" "-m" "3221225472" "--user" "501:20" "--network" "none" "ghcr.io/rust-lang/crates-build-env/linux-micro" "/opt/rustwide/cargo-home/bin/cargo" "+nightly" "rustdoc" "--lib" "-Zrustdoc-map" "-Z" "unstable-options" "--config" "build.rustdocflags=[\"-Z\", \"unstable-options\", \"--emit=unversioned-shared-resources,toolchain-shared-resources\", \"--resource-suffix\", \"-20221130-1.67.0-nightly-c97b539e4\", \"--static-root-path\", \"/-/rustdoc.static/\", \"--cap-lints\", \"warn\", \"--disable-per-crate-search\", \"--extern-html-root-takes-precedence\"]" "--offline" "-Zunstable-options" "--config=doc.extern-map.registries.crates-io=\"https://docs.rs/{pkg_name}/{version}/aarch64-apple-darwin\"" "--target" "aarch64-apple-darwin", kill_on_drop: false }` log.target="rustwide::cmd" log.module_path="rustwide::cmd" log.file="/Users/ben/.cargo/registry/src/github.com-1ecc6299db9ec823/rustwide-0.15.2/src/cmd/mod.rs" log.line=479
2022-12-01T05:36:52.536450Z  INFO log: [stderr] WARNING: The requested image's platform (linux/amd64) does not match the detected host platform (linux/arm64/v8) and no specific platform was requested log.target="rustwide::cmd" log.module_path="rustwide::cmd" log.file="/Users/ben/.cargo/registry/src/github.com-1ecc6299db9ec823/rustwide-0.15.2/src/cmd/mod.rs" log.line=621
2022-12-01T05:36:52.536514Z  INFO log: [stdout] 06e3ad4d5d4c6c5807228ae22b3e4b4ec36a48b33376da0338c7fa25be7ddd53 log.target="rustwide::cmd" log.module_path="rustwide::cmd" log.file="/Users/ben/.cargo/registry/src/github.com-1ecc6299db9ec823/rustwide-0.15.2/src/cmd/mod.rs" log.line=621
2022-12-01T05:36:52.550261Z  INFO log: running `Command { std: "docker" "start" "-a" "06e3ad4d5d4c6c5807228ae22b3e4b4ec36a48b33376da0338c7fa25be7ddd53", kill_on_drop: false }` log.target="rustwide::cmd" log.module_path="rustwide::cmd" log.file="/Users/ben/.cargo/registry/src/github.com-1ecc6299db9ec823/rustwide-0.15.2/src/cmd/mod.rs" log.line=479
2022-12-01T05:36:52.851696Z  INFO log: [stderr] standard_init_linux.go:228: exec user process caused: exec format error log.target="rustwide::cmd" log.module_path="rustwide::cmd" log.file="/Users/ben/.cargo/registry/src/github.com-1ecc6299db9ec823/rustwide-0.15.2/src/cmd/mod.rs" log.line=621

I think the docs.rs build environment is busted on M1 macs. It's trying to run x86 docker containers.

yep, it definitely is. On intel macs you needed to run the build inside the web container, which worked, but not any more on M1.

sorry about that.
Btw, if it's only about this issue, the readme is stored in the database (releases.readme), so you might be able to just put in a record into releases and crates, and test the Markdown rendering accordingly.


Inside on_url cb http protocol version is always 0

Hi,
executing the following code in every cb, you can see that http_parser::http_major and http_parser::http_minor are always 0 inside on_url callback.
`void dumpHttpParserInfo(http_parser* p){
auto method = http_method_str(http_method(p->method));
auto errorName = http_errno_name(http_errno(p->http_errno));
auto errorDescription = http_errno_description(http_errno(p->http_errno));

cout << "METHOD:\t" << string_view(method) << endl
     << "HTTP:\t" << to_string(p->http_major) << "." << to_string(p->http_minor) << endl
     << "ERROR:\t" << string_view(errorName) << " - " << string_view(errorDescription) << endl;

};`
Is it normal?
Tahnks,
Dario

That's normal because URL is parsed before the protocol version when applying parser to HTTP requests.

OK, so I close this issue.
Thank you very much

No problem.


Add NodeJS version

Here is a simple NodeJS implementation.

Hum, you might want to change the MAX_STATION constant and try then benchmark ruby bench.rb "node .", but you would waste your time ;)
Loading top stations
. done

Starting the application
. done

Loading the data
. done

Starting the benchmark
FATAL ERROR: CALL_AND_RETRY_LAST Allocation failed - process out of memory

Pull request now integrates tests and JS implementation that cover functional limitation indicated in original readme

Closing this, as we have https://github.com/trainline-eu/csa-challenge/blob/master/csa.js.


Update dependencies to address dependabot alerts

https://github.com/LibreTexts/conductor/security/dependabot
You may also consider enabling Dependabot security updates.

Will be closed by #3


coupling with scrapy

Hello, your project is very interesting. I wanted to know if it was possible to use it with scrapy library.
Thank you for your help :blush:

anything is possible! this is a standalone library, though.


Alert conflicting with Bootstrap alert

When an alert is created, a classname notyf alert is assigned to the element. The class alert is a special class in Bootstrap so it's setting an additional padding making the card look pretty big.
Fix this by either changing the class name or manually setting padding:0 to the alert element.

Nice lib!
About this issue, can it be solved by changing card.className to Notyf_alert (for example) here and in scss source?
As the matter of fact, all notyf's css class names may be prefixed this way ( Notyf_ ), to avoid conflicts.

You're right man thank you for suggesting that. Been away these holidays, but now I'm back to it. Gonna fix that in no time. I think I'm going to follow BEM methodology for consistency.

Resolved in https://github.com/caroso1222/notyf/commit/45e327273fb8d3524f15ab56f75dcb36d35d489a by using BEM methodology for CSS classes.


Predicting a conv1d model in keras that takes index of words in a sentence as input

So I have prepared a sentiment analysis model and I am trying to predict it with new input but I am faced with an error:
data = getFile('Cleaned Data.xlsx')

data['Description'] = data['Description'].apply(lambda x: x.lower())
data['Description'] = data['Description'].apply((lambda x: re.sub('[^a-zA-z0-9\s]','',x)))

print(data[ data['Classification'] == 1].size)
print(data[ data['Classification'] == 0].size)

for idx,row in data.iterrows():
    row[1] = row[1].replace('rt',' ')

tokenizer = Tokenizer(split=' ')
tokenizer.fit_on_texts(data['Description'].values)

#vectorizer = CountVectorizer()
#X = vectorizer.fit_transform(jobSpec['Description']).toarray()

X = tokenizer.texts_to_sequences(data['Description'].values)
X = pad_sequences(X, maxlen=100, value=0.)

display(X.shape)

vocab_size = len(tokenizer.word_index) + 1
max_length = max([len(s.split()) for s in data['Description']])

Y = pd.get_dummies(data['Classification']).values
X_train, X_test, Y_train, Y_test = train_test_split(X,Y, test_size = 0.33, random_state = 42)

tokens_docs = [doc.split(" ") for doc in data['Description'].values]
all_tokens = itertools.chain.from_iterable(tokens_docs)
my_dict = {token: token if token.isdigit() else idx for idx, token in enumerate(set(all_tokens))}

print(X_train.shape,Y_train.shape)
print(X_test.shape,Y_test.shape)

raw_embedding = load_embedding('glove.6B.100d.txt')

embedding_vectors= get_weight_matrix(raw_embedding, tokenizer.word_index)
embedding_layer = Embedding(vocab_size, 100, weights=[embedding_vectors], input_length = X.shape[1], trainable=False, name="Embeddings")

display(embedding_vectors.shape)

# define model
sequence_input = Input(shape=(100,), dtype='int32')
embedded_sequences = embedding_layer(sequence_input)
x = Conv1D(128, 5, activation='relu')(embedded_sequences)
x = MaxPooling1D(2)(x)
x = Conv1D(128, 5, activation='relu')(x)
x = MaxPooling1D(2)(x)
x = Conv1D(128, 5, activation='relu')(x)
x = MaxPooling1D(18)(x)
x = Flatten()(x)
x = Dense(128, activation='relu')(x)
preds = Dense(2, activation='softmax')(x)
model = Model(sequence_input, preds)
model.compile(loss='categorical_crossentropy',
              optimizer='adam',
              metrics=['acc'])

# input for which we need the embedding
input_str = "the company our client is a renowned civil actor who have consistently and safely delivered major civil infrastructure  ts across a"

# build index based on our `vocabulary`
word_to_idx = OrderedDict({w:all_tokens.index(w) for w in input_str.split() if w in all_tokens})

ynew = model.predict([1],[3],[5],[7])
display(ynew)

when I try to predict this model with new input:
ynew = model.predict([1],[3],[5],[7])
display(ynew)

it gives me an error message:
ValueError: Error when checking input: expected input_29 to have shape (100,) but got array with shape (1,)
I have tried to change the shapes of the model to None and 1 but it gives me other new errors. I am quite new to the machine learning stuff so really not sure how to fix this one.
Any help will be appreciated

Your shape error is because your model is expecting an input of size 100.  You should pass in a tensor of size [batch_size, 100]

Thank you. I managed to fix it by adding paddings to the input data.
text = ['Construction workers needed for this company who has qualifications ']
#vectorizing the tweet by the pre-fitted tokenizer instance
text = tokenizer.texts_to_sequences(text)

text = pad_sequences(twt, maxlen=100, dtype='int32', value=0)
print(text)
sentiment = model.predict(text, batch_size=1)[0]
display(sentiment)

This solved my problem. I was wondering what you meant by changing the input size to 100 but I see that padding it with the 0 allows me to make up for the empty sequence values.

I have a question regarding this input on android now I will be using this on Android. So, let's say I have an input of some text sequences of a sentence but some words in the sentence are not in the sequence dictionary that I saved in a JSON format from this model's whole text data. So, for the words that are not in the dictionary, would I need to add a 0 in their place or can I just remove that word from the sequence and add only the sequence ID of the words that exist in the dictionary then filling the rest with 0?

The typical solution is to use an "Out of Vocabulary" (OOV) integer.  Sometimes developers will use exactly one integer for their OOV tokens.  Other times developers will use a hash function to map words to one of several "OOV buckets"

The typical solution is to use an "Out of Vocabulary" (OOV) integer. Sometimes developers will use exactly one integer for their OOV tokens. Other times developers will use a hash function to map words to one of several "OOV buckets"

Okay, for now, I will use 0 for all of my Out of vocabulary Tokens. Thank you for your assistance.


Widget not implemented?

Anyone have any idea how to fix this?
error[E0277]: the trait bound `impl ratatui::widgets::Widget + '_: Widget` is not satisfied
  --> src\components\text_entry.rs:43:21
   |
43 |     f.render_widget(tw, rect);
   |       ------------- ^^ the trait `Widget` is not implemented for `impl ratatui::widgets::Widget + '_`
   |       |
   |       required by a bound introduced by this call
   |
   = help: the following other types implement trait `Widget`:
             BarChart<'a>
             Canvas<'a, F>
             Chart<'a>
             Gauge<'a>
             LineGauge<'a>
             List<'a>
             Paragraph<'a>
             Sparkline<'a>
           and 7 others
note: required by a bound in `ratatui::Frame::<'a, B>::render_widget`
  --> C:\Users\haydu\.cargo\registry\src\github.com-1ecc6299db9ec823\ratatui-0.21.0\src\terminal.rs:88:12
   |
86 |     pub fn render_widget<W>(&mut self, widget: W, area: Rect)
   |            ------------- required by a bound in this associated function
87 |     where
88 |         W: Widget,
   |            ^^^^^^ required by this bound in `Frame::<'a, B>::render_widget`

For more information about this error, try `rustc --explain E0277`.
error: could not compile `wd` (lib) due to previous error

This turned out to be because Cargo was downloading 0.20.1 of ratatui for tui-textarea, while I was using 0.21.0. Fixed by forking and updating the toml file. Don't think my change to return concrete type was necessary.
https://github.com/rhysd/tui-textarea/compare/main...haydenflinner:tui-textarea:main

@haydenflinner thank you for your fork. Using it works for now.

You can use ratatui-textarea a fully supported tui-textarea for ratatui

Closed due to v0.2.1: https://github.com/rhysd/tui-textarea/releases/tag/v0.2.1


fix(raw_vehicle_cmd_converter): fix parameter files to parse path to csv files (#6136)

Description
backport from https://github.com/autowarefoundation/autoware.universe/pull/6136
Related links
https://tier4.atlassian.net/browse/RT0-30609
Tests performed
I confirmed in https://tier4.atlassian.net/browse/RT0-30609?focusedCommentId=155746
Notes for reviewers

Interface changes

Effects on system behavior

Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[ ] I've confirmed the contribution guidelines.
[ ] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.
[ ] The PR has been properly tested.
[ ] The PR has been reviewed by the code owners.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.
[ ] The PR is ready for merge.

After all checkboxes are checked, anyone who has write access can merge the PR.

@mergifyio backport beta/v0.19.2

@Mergifyio backport beta/v0.19.2


Investigate possibility of deploying apps from within R-Studio

Looks like it's possible, though, it could be restricted to shinyapps.io.

Done via R Connect, which is a paid solution, and a bit too much expensive for us right now.


add centroid calculator

With reference to issue #1212
Added centroid calculator

@khareyash05 I think you have added this centroid calculator in between the parallel line checker calculator by mistake.

The lines which are in box are a part of parallel line calculator.

Ohhh sorry sorry @RaveenaBhasin

@sairish2001 Resolved merge conflicts

@sairish2001 REsolved merge conflicts


Added counts into @ToString

I added "counts" param into ToString annotation, which substitute all arrays, collections and maps by their items count.

I like that you added your name to the AUTHORS file, that you created a config key and added End-to-end tests.
Bad news first: we are not going to merge this. We have some guidelines we consider when adding features:
1) Is the feature solving a common problem?
2) Does it help developers to create better code?
3) Does the code become unpredictable/easy to misunderstand?
4) Is it worth the added burden of maintaining?
5) Does it conflict with other ideas or solutions?
In this case, especially (1) and (4) are a problem.
The after-ecj seems to indicate that it didn't work though. It looks like you didn't get the eclipse side working at all.
I'm still considering if it is going to work, especially when someone creates their own collections. Using resolution is very brittle. Especially in Eclipse. Especially when using method local classes.
Consider adding the following to your test cases:
class X {
    public void method() {
        class StringList extends ArrayList<String> {}
        @ToString(count=true)
        class MethLocal {
            StringList x = new StringList ();
        }
    }
}

Also, it only keys off the declared type of the field, not the actual value. I think that that's a good thing, but still need to think this through.
Did you consider a different output format?
class Foo {
    int[] a = {};
    String b = "x";
}

In your code this would result in:
"Foo(a.lenght=0, b=x)"

But an alternative implementations could be
"Foo(a[0], b=x)"
"Foo(a(0), b=x)"
"Foo(#a=0, b=x)"
"Foo(|a|=0, b=x)"

One other problem in your implementation is that it is an all-or-nothing property. You cannot say: this field I want to see, the other fields I want counted.
I hope that you continue to contribute to this project. Next time, please send a mail to the mailing list first, so we can discuss the guidelines, possible solutions and pitfalls.


[REVIEW] Naresh-chandanbatve

GitHub Profile Link
https://github.com/Naresh-chandanbatve

Profile Review
TLDR;
Your profile looks awesome! See the full details with my suggestions below.

Full details
Your profile looks awesome! Here some suggestions from my side:

remove the profile view counter due its inaccurate.


Add next.js sceduler example

What this PR does / why we need it?
Add Next.js example with react-calendar
Any background context you want to provide?
#557
Add dynamic-editing Next.js Sceduler example with react-calendar

display joined peers list
can create & delete scedule with comments dynamically
write by next.js latest version (v13.4)

What are the relevant tickets?
Feature #557
Checklist

[ ] Added relevant tests or not required
[ ] Didn't break anything

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

my god.... my vscode prettier extension overwrite yorkie's prettier options... 😢
I'll make a request again after solving the problem

@banma1234 Please reopen this PR after you fix your problem 😄


Add recyclerview-ktx

Draft:
inline fun RecyclerView.requireAdapter(): RecyclerView.Adapter<RecyclerView.ViewHolder> = checkNotNull(adapter)

inline fun <reified T : RecyclerView.LayoutManager> RecyclerView.requireLayoutManager(): T {
    return checkNotNull(layoutManager) as T
}

fun RecyclerView.setupLinearLayoutManager(
    @RecyclerView.Orientation orientation: Int = RecyclerView.VERTICAL,
    reverseLayout: Boolean = false,
) {
    layoutManager = LinearLayoutManager(context, orientation, reverseLayout)
}

Also to prevent memory leaks would be useful to add a delegate for an adapter clearing on view destroy.


action.remove on Controls with validators cause issue with intents

The Problem
I create a set of new measurements(system, quantity, and unit) in an array model using actions.push. Each Control in the set also include validators. If I then attempt to remove a set using actions.remove, I get the error below:
Uncaught TypeError: Cannot read property 'intents' of undefined

If I type into the fields OR if I remove the validators on each Control then I do not get this error.
Reproducible Code Example
http://codepen.io/sanchezedgar/pen/NjvdGK?editors=0110

Can you try this in the latest version?


are you going to release the pretrained model

There is a pretrained ResNet50 at https://github.com/ppwwyyxx/GroupNorm-reproduce/tree/master/ImageNet-ResNet-TensorFlow and pretrained ResNet50 & ResNet101 at https://github.com/facebookresearch/Detectron/tree/master/projects/GN


nft transfer event

Fixes #570
Changes proposed in this PR:

Tracking NftTransferHistory
Updating creator, owner and transferHistory in Order
Updating tests

looks good so far, see https://github.com/oceanprotocol/aquarius/pull/943

@jamiehewitt15 did I miss anything?
I thought order entity will also store nftOwner on entitycreation?

@jamiehewitt15 did I miss anything? I thought order entity will also store nftOwner on entity creation?

Yeah, I had it in initially but removed it due to Alex's comment above. Let me know if you need it and I'll add it back in

We already have it:
https://github.com/oceanprotocol/ocean-subgraph/blob/1bf645541fbc4292b38568db76f7cf79c44cf002/schema.graphql#L152
https://github.com/oceanprotocol/ocean-subgraph/blob/1bf645541fbc4292b38568db76f7cf79c44cf002/src/mappings/erc20Templates.ts#L44

Looks like we don't have them :)
I will push soon

@jamiehewitt15 did I miss anything? I thought order entity will also store nftOwner on entity creation?

Yeah, I had it in initially but removed it due to Alex's comment above. Let me know if you need it and I'll add it back in

I was under the impression that nftOwner was under nftData, and not Order.  that's why I said that we don't need it.  my  mistake, sorry

@jamiehewitt15 did I miss anything? I thought order entity will also store nftOwner on entity creation?

Yeah, I had it in initially but removed it due to Alex's comment above. Let me know if you need it and I'll add it back in

I was under the impression that nftOwner was under nftData, and not Order. that's why I said that we don't need it. my mistake, sorry

No worries, I've added it back in


[KAFKA-13687]  Allowing dumping logs for a small segment

Summary
This PR allows to limit the output batches while they are inspected via the kafka-dump-log.sh  script.
The idea is to take samples from the logsegments without affecting a production cluster  as the current script will read the whole files, this could create issues in term of page.
More information can be found in the ticket here
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

WIP

sorry @mimaison looks like I don't have permission for the KIP creation, how it is supposed I get access ? thanks

@sciclon2 To get permissions, you need to follow the steps in Getting Started:
4. Send an email to the dev mailing list ([dev@kafka.apache.org](mailto:dev@kafka.apache.org)) containing your wiki ID and Jira ID requesting permissions to contribute to Apache Kafka.

If you give me your JIRA ID, I can add you. Also you need to open a PR against the branch named trunk, your other PR was correct.

thanks for the reply @mimaison ,  I sent earlier the email with both Ids, so I will wait,
jira user<EMAIL_ADDRESS>wiki di: sergio_troiano
In the other hand I opened the PR to the 2.8 as the files are different and they output different data.
I worked in the 2.8 branch as it is the one we use now in PRO
2.8
trunk
Does it mean I have only to commit in trunk? sorry again and thanks for the information

I've granted you permissions on the wiki and in JIRA. You should now be able to create KIPa and assign JIRAs to yourself.
Yes new features are only committed to trunk.

It's probably best to close this PR and reopen the other one on trunk.

sure, thanks a lot @mimaison you were really helpful !!!


Add OpenNMT-py in dependencies

The onmt_modules are already included. The installed onmt may not work with our repo.


Mismatch between rate law symbols and state variable names

Hi folks,
In the AMR format, rate laws are expressed as symbols using ASKENET State id not ASKENET State name,
https://github.com/indralab/mira/blob/82421e4465edfb6eb27f811726d9745afb3d28e4/mira/sources/askenet/petrinet.py#L101
but MIRA only preserves the name field (unless the name is not present), and not the id when reading the AMR into a Mira model.
https://github.com/indralab/mira/blob/82421e4465edfb6eb27f811726d9745afb3d28e4/mira/sources/askenet/petrinet.py#L229
This makes it difficult to automatically associate each sympy symbol with its corresponding state variable.
Is there a way to associate AMR ids with state variables in the MIRA model?  Maybe just add id as another key in the variable.data dictionary or perhaps as an askenet:<id> in the identifiers or context attribute?

Yes, this can indeed happen if both an id and a name are specified and they are different. We will look into the best option to fix this.

The main issue is that I need a reliable way to associate a symbol with a concept. One possibility is to include the symbol as an additional attribute of the Concept


New 2F140 gripper implementation + franka2f140 robot + some changes on creating pybullet clients

Overview of changes:

Full refactor of Robotiq2F140 gripper class in eetool. Based on p.createConstraint instead of a background mimic thread. This should help the system be useable from MacOS (which has extra complications multithreading with pybullet)
added 'server' argument to the BulletClient to create a p.GUI_SERVER or p.SHARED_MEMORY_SERVER instead of the regular modes. These are also needed for proper multithreading
New URDF + mesh assets + cfg file for panda robot (panda.urdf, panda_2f140.urdf, franka_2f140_cfg.py)
modified check for pulling URDFs from pybullet data, use a new 'pybullet_urdf' flag instead.
some small updates to the franka_cfg and assets/franka_arm cfg file

@taochenshh @richardrl would appreciate a review on this. @richardrl I think hopefully this should fix the issues you faced a very long time ago using this gripper on your mac.


eventbuilder.exe is now xplat (tm) and always uses the latest SDK's.

Don't merge yet.
Progresses issue #1043
Next steps is

:fire: all $platform_XS.csproj conventions
:fire: all prebuild mdtool/msbuild conventions
Configure EventBuilder (this) into CI
Create ReactiveUI-Events packages
Progress iOS CI story.

Through QA completed on generating events. All platforms generate the same output as the old implementation regardless of Mac|Windows.


fix panic by handling error message

This PR for addressing panic message below:
2022/09/03 09:15:13 http: panic serving [::1]:50741: runtime error: invalid memory address or nil pointer dereference
goroutine 146 [running]:
net/http.(*conn).serve.func1()
	/opt/homebrew/Cellar/go/1.19/libexec/src/net/http/server.go:1850 +0xb0
panic({0x100c77440, 0x100fb0230})

I must have forgotten to handle err there. Thank you for this PR


Markdown preview window is empty when splitting right

Issue Type: Bug
Open an *md file in preview and right click on the tab.
Select split right (or click the split button in upper right of VS Code.
Markdown preview window is empty when splitting right.
Also, right clicking and selecting copy doesn't work.
VS Code version: Code 1.69.1 (b06ae3b2d2dbfe28bca3134cc6be65935cdfea6a, 2022-07-12T08:21:24.514Z)
OS version: Windows_NT x64 10.0.19044
Restricted Mode: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-7700HQ CPU @ 2.80GHz (8 x 2808)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: unavailable_offvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
31.83GB (20.57GB free)

Process Argv
--crash-reporter-id 22414da9-2e90-42d1-8701-27fd81972ce5

Screen Reader
no

VM
0%

Extensions (38)

Extension
Author (truncated)
Version

aws-toolkit-vscode
ama
1.45.0

doxdocgen
csc
1.4.0

es7-react-js-snippets
dsz
4.4.3

gitlens
eam
12.1.2

prettier-vscode
esb
9.5.0

auto-rename-tag
for
0.1.10

go
gol
0.34.1

terraform
has
2.23.0

better-cpp-syntax
jef
1.15.19

webpack
jer
2.2.0

cmake-language-support-vscode
jos
0.0.4

vscode-language-babel
mgm
0.0.36

file-downloader
min
1.0.12

vscode-docker
ms-
1.22.1

dotnet-interactive-vscode
ms-
1.0.3357010

vscode-dotnet-runtime
ms-
1.5.0

jupyter
ms-
2022.6.1101950301

jupyter-keymap
ms-
1.0.0

jupyter-renderers
ms-
1.0.8

remote-containers
ms-
0.241.3

remote-ssh
ms-
0.84.0

remote-ssh-edit
ms-
0.80.0

remote-wsl
ms-
0.66.3

vscode-remote-extensionpack
ms-
0.21.0

cmake-tools
ms-
1.11.26

cpptools
ms-
1.10.8

cpptools-extension-pack
ms-
1.2.0

powershell
ms-
2022.6.3

pwa-studio
PWA
1.0.5

vscode-yaml
red
1.9.1

rust-analyzer
rus
0.3.1123

vscode-odata
sta
0.1.0

even-better-toml
tam
0.16.4

cmake
twx
0.0.17

vscode-lldb
vad
1.7.0

vscode-icons
vsc
11.13.0

vscode-import-cost
wix
3.3.0

markdown-all-in-one
yzh
3.4.3

(1 theme extensions excluded)

A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
pythonvspyl392:30443607
vserr242:30382549
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
vscorecescf:30445987
pythondataviewer:30285071
vscod805cf:30301675
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
cmake_vspar411:30525688
vsaa593cf:30376535
vsc1dst:30438360
pythonvs932:30410667
wslgetstarted:30449410
vscscmwlcmt:30465135
cppdebug:30492333
pylanb8912:30520716
vsclangdf:30486550

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!

This feature request has not yet received the 20 community upvotes it takes to make to our backlog. 10 days to go. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

:slightly_frowning_face: In the last 60 days, this feature request has received less than 20 community upvotes and we closed it. Still a big Thank You to you for taking the time to create this issue! To learn more about how we handle feature requests, please see our documentation.
Happy Coding!


Add prometheus jmx javaagent thrown exception 'Address already in use' when monitor starting

Alluxio Version:
2.9.0
Describe the bug
I added prometheus javaagent in alluxio-env.sh:
ALLUXIO_WORKER_JAVA_OPTS="$ALLUXIO_WORKER_JAVA_OPTS -Xms16G -Xmx32G -XX:+UseG1GC -javaagent:${ALLUXIO_HOME}/extensions/jmx_prometheus_javaagent-0.15.0.jar=13753:${ALLUXIO_HOME}/conf/jmx-prometheus.yaml "

when I start worker, the exception is thrown:
Exception in thread "main" java.lang.reflect.InvocationTargetException
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:498)
	at sun.instrument.InstrumentationImpl.loadClassAndStartAgent(InstrumentationImpl.java:386)
	at sun.instrument.InstrumentationImpl.loadClassAndCallPremain(InstrumentationImpl.java:401)
Caused by: java.net.BindException: Address already in use
	at sun.nio.ch.Net.bind0(Native Method)
	at sun.nio.ch.Net.bind(Net.java:433)
	at sun.nio.ch.Net.bind(Net.java:425)
	at sun.nio.ch.ServerSocketChannelImpl.bind(ServerSocketChannelImpl.java:223)
	at sun.nio.ch.ServerSocketAdaptor.bind(ServerSocketAdaptor.java:74)
	at sun.net.httpserver.ServerImpl.<init>(ServerImpl.java:100)
	at sun.net.httpserver.HttpServerImpl.<init>(HttpServerImpl.java:50)
	at sun.net.httpserver.DefaultHttpServerProvider.createHttpServer(DefaultHttpServerProvider.java:35)
	at com.sun.net.httpserver.HttpServer.create(HttpServer.java:130)
	at io.prometheus.jmx.shaded.io.prometheus.client.exporter.HTTPServer.<init>(HTTPServer.java:179)
	at io.prometheus.jmx.shaded.io.prometheus.jmx.JavaAgent.premain(JavaAgent.java:31)
	... 6 more

To Reproduce
add the following env in alluxio-env.sh and run worker:
ALLUXIO_WORKER_JAVA_OPTS="$ALLUXIO_WORKER_JAVA_OPTS -Xms16G -Xmx32G -XX:+UseG1GC -javaagent:${ALLUXIO_HOME}/extensions/jmx_prometheus_javaagent-0.15.0.jar=13753:${ALLUXIO_HOME}/conf/jmx-prometheus.yaml "

Are you planning to fix it
Please indicate if you are already working on a PR.
Additional context
Why ALLUXIO_WORKER_MONITOR_JAVA_OPTS extends ALLUXIO_WORKER_JAVA_OPTS?
I think it is unnecessary because worker could use a lot of memory(32G), but the monitor just need 4G or less memory. If I set -Xms16G -Xmx32G in ALLUXIO_WORKER_JAVA_OPTS that means monitor will run with -Xms16G -Xmx32G.

I see you are trying to attach a java agent profiler via the environment variable. You can do so by specifying the lib in a dedicated env var ALLUXIO_WORKER_ATTACH_OPTS, see the documentation here https://docs.alluxio.io/os/user/stable/en/administration/Troubleshooting.html#debugging-alluxio-processes
Also see related discussion here https://github.com/Alluxio/alluxio/issues/15168#issuecomment-1072641897

The monitor env vars were introduced in https://github.com/Alluxio/alluxio/pull/13576 to address a similar issue. The PR was mostly a workaround which removes JDWP specific options. I am not sure why ALLUXIO_WORKER_MONITOR_JAVA_OPTS must extend from ALLUXIO_WORKER_JAVA_OPTS, but from the context of #13576 I guess is that it's mostly aimed to stay backward compatible.
Judging from the contexts, I'd say ALLUXIO_WORKER_MONITOR_JAVA_OPTS should not duplicate ALLUXIO_WORKER_JAVA_OPTS and instead should have a different set of defaults. @humengyu2012 I'd really appreciate it if you can come up with a fix PR and test if the a dedicated set of defaults for the monitor should be appropriate.

Fixed by https://github.com/Alluxio/alluxio/pull/16688


Remove labels and unused VC generation code

This PR removes labels inside the verification conditions generated by Boogie.  Labels were used to generate error traces but are obsolete now given that SMT solvers are reasonably good at generating models.  The new mechanism uses extra "control-flow" instrumentation in the verification to generate error traces.
This PR also removes a bunch of algorithms for generating verification conditions that were not being used.
Finally, the tests inside stratifiedInline and extractLoops folder have been disabled because they are dependent on implementation of stratified inlining which is going to be eliminated from Boogie (#200) and which works only with labels.

Changes look good. Goodbye labels.

@RustanLeino : I double checked that with the current version of this branch, all tests in Dafny pass (with Z3 4.8.4) except those in comp folder and a couple others that require some .net dlls to be installed (I am working on a MacOS machine).  So I think we are good to land this PR.

I have removed /vc option entirely.  So its mention in the help message is also eliminated.
I have removed the doomed code and tests.
I tried to remove doomedInjectedAssumptionVariables in a couple of different ways (without really understanding the point of this variable) but the test Test/snapshots/runtest.snapshot was failing.  So I gave up on this endeavor. Perhaps @RustanLeino  can explain the purpose of this field and whether it can be eliminated.  If it cannot be eliminated, perhaps it should be renamed.

@shazqadeer @bkragl doomedInjectedAssumptionVariables definitely doesn't have anything to do with the doomed VC generation option. I wrote that code a long time ago for the verification result caching feature that is used by the Dafny IDE, but I don't fully remember the reason for picking that name... Probably that any attempt at reusing cached results is doomed. :)

I'm joining the party only after it already ended.
@shazqadeer Yes, this looks good. Thanks for doing this spring cleaning.
I'm surprised about the deletion of stratified inlining. Doesn't Corral use it?
@bkragl A doomed program point is one from which execution will inevitably fail, regardless of how control reached the doomed point. It was the subject of @martinschaef's PhD thesis.


Default collection types

It seems pretty clear the default collection type is going to be the Arr. Arr is an alias for scala.collection.immutable.ArraySeq. This will be backed up by Arrays and ArrayBuffers. This is still subject to confirmation through successful full implementation, but the conclusion of the process seems almost certain.


fix: trip booking not navigate to T&C Page

Details

Fixed Issues

$ https://github.com/Expensify/App/issues/49870
PROPOSAL:
Tests

Open FAB > Book travel
Click Book travel
Select address and save it
Click Book travel
6 Verify that: T&C Page is shown

[x] Verify that no errors appear in the JS console

Offline tests

QA Steps

Open FAB > Book travel
Click Book travel
Select address and save it
Click Book travel
6 Verify that: T&C Page is shown

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR
    - [x] I added steps for local testing in the Tests section
    - [x] I added steps for the expected offline behavior in the Offline steps section
    - [x] I added steps for Staging and/or Production testing in the QA steps section
    - [x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
    - [x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
    - [x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).
[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:
    - [x] Android: Native
    - [x] Android: mWeb Chrome
    - [x] iOS: Native
    - [x] iOS: mWeb Safari
    - [x] MacOS: Chrome / Safari
    - [x] MacOS: Desktop
[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)
    - [x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
    - [x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
    - [x] I verified that comments were added to code that is not self explanatory
    - [x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
    - [x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method
      - [x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:
    - [x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
    - [x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
    - [x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
    - [x] I verified the JSDocs style guidelines (in STYLE.md) were followed
[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:
    - [x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory
[x] If a new CSS style is added I verified that:
    - [x] A similar style doesn't already exist
    - [x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))
[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:
    - [x] I verified that all the inputs inside a form are aligned with each other.
    - [x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native
https://github.com/user-attachments/assets/b8170ae1-623d-4203-bed6-0bd48f4f5f4c

Android: mWeb Chrome
https://github.com/user-attachments/assets/7fde7887-840f-4f6f-837d-ac4a491017f6

iOS: Native
https://github.com/user-attachments/assets/8f7f8afc-d979-4c81-a561-234d2ed88464

iOS: mWeb Safari
https://github.com/user-attachments/assets/81d9ae3b-b125-4fc1-8e50-8e81e2fb4e7b

MacOS: Chrome / Safari
https://github.com/user-attachments/assets/60ff90fc-12d0-4d38-abbd-d18dd38e5c66

MacOS: Desktop
https://github.com/user-attachments/assets/5e5ddff3-8b38-4be9-8d30-d4680b6c199a

I will review this PR because this is regression from my previous PR

@daledah Prettier check failed

@daledah lint is failing

Reviewer Checklist

[x] I have verified the author checklist is complete (all boxes are checked off).
[x] I verified the correct issue is linked in the ### Fixed Issues section above
[x] I verified testing steps are clear and they cover the changes made in this PR

[x] I verified the steps for local testing are in the Tests section
[x] I verified the steps for Staging and/or Production testing are in the QA steps section
[x] I verified the steps cover any possible failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)

[x] I checked that screenshots or videos are included for tests on all platforms
[x] I included screenshots or videos for tests on all platforms
[x] I verified tests pass on all platforms & I tested again on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] If there are any errors in the console that are unrelated to this PR, I either fixed them (preferred) or linked to where I reported them in Slack
[x] I verified proper code patterns were followed (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick).
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method
[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I verified that this PR follows the guidelines as stated in the Review Guidelines
[x] I verified other components that can be impacted by these changes have been tested, and I retested again (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar have been tested & I retested again)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG)

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR reviewer checklist, including those that don't apply to this PR.

Screenshots/Videos

Android: Native

Android: mWeb Chrome

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari

MacOS: Desktop

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

Confirmed the fix on staging


add Cloudflare

Cloudflare is not exactly frontend focused, but I will add it since I do use it myself as a frontend developer.


[8.x] Throw ModelNotFoundException for sole in Eloquent

Follow up to #35869
As suggested in comment https://github.com/laravel/framework/pull/35869#issuecomment-760482444 by @taylorotwell , this PR throws a ModelNotFoundException when ->sole() is called from an Eloquent Query Builder and no records are found.
This is particularly useful as ModelNotFoundException exceptions are handled automatically in HTTP requests by returning a 404 response.
Maybe we should also add some kind of HTTP handling for the MultipleRecordsFoundException when called from an Eloquent Query Builder.

I think this is going to make things a bit confusing, you'll need to account for ModelNotFoundException and  RecordsNotFoundException in your code based on wether the instance is a base builder or eloquent builder.
I think having RecordsNotFoundException associated with the sole() method makes it easier.

I think this is going to make things a bit confusing, you'll need to account for ModelNotFoundException and  RecordsNotFoundException in your code based on wether the instance is a base builder or eloquent builder.
I think having RecordsNotFoundException associated with the sole() method makes it easier.

Making it extend RecordsNotFoundException is a good idea, but won't make the HTTP handler return a 404.
I can instead of extending it on an Eloquent builder add an if clause to the Illuminate\Foundation\Exceptions\Handler@prepareException to account for a RecordsNotFoundException.
What do you think?

Making it extend RecordsNotFoundException is a good idea, but won't make the HTTP handler return a 404.
I can instead of extending it on an Eloquent builder add an if clause to the Illuminate\Foundation\Exceptions\Handler@prepareException to account for a RecordsNotFoundException.
What do you think?

My bad, now I get what you meant from your first message.
Updated to extend from RecordsNotFoundException. Thanks!

My bad, now I get what you meant from your first message.
Updated to extend from RecordsNotFoundException. Thanks!

I suggest returning a 400 Bad Request for a MultipleRecordsFoundException, but I'll leave it up for discussion.

I suggest returning a 400 Bad Request for a MultipleRecordsFoundException, but I'll leave it up for discussion.


externalName error when using a MongoDB Atlas demo service

I'm testing the service concept and created a basic service for MongoDB Atlas (https://github.com/lucj/acorn-services/tree/main/atlas).
In order to run the webhooks app (https://gitlab.com/web-hook) using this atlas service I've done the following step:

first create the atlas-creds external secret (this one contains the Atlas api keys)

acorn secrets create \
  --type opaque \
  --data public_key=$MONGODB_ATLAS_PUBLIC_API_KEY \
  --data private_key=$MONGODB_ATLAS_PRIVATE_API_KEY \
  --data project_id=$MONGODB_ATLAS_PROJECT_ID \
  atlas-creds

Next  run the latest version of this Acorn image specifying the --atlas flag. This flag allows to use an Atlas generated cluster instead of an internal mongodb instance. The Acornfile used for the webhooks app is https://gitlab.com/web-hook/config/-/blob/main/apps/acorn/Acornfile

$ acorn run -n webhook lucj/webhooksapp --atlas
webhook
STATUS: ENDPOINTS[] HEALTHY[] UPTODATE[]
STATUS: ENDPOINTS[] HEALTHY[0] UPTODATE[0] [defined: waiting on missing secrets [atlas]]
STATUS: ENDPOINTS[] HEALTHY[0/1] UPTODATE[1] [defined: waiting on missing secrets [atlas]] [containers: api pending create; nats ContainerCreating; swagger pending create; wss pending create; www pending create]
STATUS: ENDPOINTS[http://default-webhook-a1a950fb.11jp51.alpha.on-acorn.io => default:8080] HEALTHY[0/1] UPTODATE[1] [defined: waiting on missing secrets [atlas]] [containers: api pending create; nats ContainerCreating; swagger pending create; wss pending create; www pending create]
STATUS: ENDPOINTS[http://default-webhook-a1a950fb.11jp51.alpha.on-acorn.io => default:8080] HEALTHY[0/1] UPTODATE[1] [defined: waiting on missing secrets [atlas]] [containers: api pending create; nats is not ready; swagger pending create; wss pending create; www pending create]
STATUS: ENDPOINTS[http://default-webhook-a1a950fb.11jp51.alpha.on-acorn.io => default:8080] HEALTHY[1] UPTODATE[1] [defined: waiting on missing secrets [atlas]]
STATUS: ENDPOINTS[http://default-webhook-a1a950fb.11jp51.alpha.on-acorn.io => default:8080] HEALTHY[1] UPTODATE[1] pending

The Atlas cluster is correctly created (this can be verified in the Atlas dashboard) but the application remains stuck.
From the acorn controller's logs we can see an error related to the Service externalName:
$ kubectl logs acorn-controller-85cc65ccd4-qd2kx -n acorn-system
...
time="2023-05-17T07:47:32Z" level=info msg="Handling [webhook-atlas-bf6c221f-2d70a4c7-2aa/atlas] [internal.acorn.io/v1, Kind=ServiceInstance]"
time="2023-05-17T07:47:32Z" level=info msg="Triggering [acorn/webhook-atlas-bf6c221f] [internal.acorn.io/v1, Kind=AppInstance] from [webhook-atlas-bf6c221f-2d70a4c7-2aa/atlas] [internal.acorn.io/v1, Kind=ServiceInstance]"
time="2023-05-17T07:47:32Z" level=info msg="Triggering [acorn/webhook-atlas-bf6c221f] [/v1, Kind=Service] from [webhook-atlas-bf6c221f-2d70a4c7-2aa/atlas] [internal.acorn.io/v1, Kind=ServiceInstance]"
time="2023-05-17T07:47:32Z" level=info msg="Triggering [acorn/webhook] [internal.acorn.io/v1, Kind=AppInstance] from [webhook-atlas-bf6c221f-2d70a4c7-2aa/atlas] [internal.acorn.io/v1, Kind=ServiceInstance]"
time="2023-05-17T07:47:32Z" level=info msg="Handling trigger [acorn/webhook-atlas-bf6c221f] [internal.acorn.io/v1, Kind=AppInstance]"
time="2023-05-17T07:47:32Z" level=info msg="Handling trigger [acorn/webhook-atlas-bf6c221f] [/v1, Kind=Service]"
time="2023-05-17T07:47:32Z" level=info msg="Handling trigger [acorn/webhook] [internal.acorn.io/v1, Kind=AppInstance]"
time="2023-05-17T07:47:32Z" level=error msg="error syncing 'webhook-atlas-bf6c221f-2d70a4c7-2aa/atlas': handler acorn-controller internal.acorn.io/v1, Kind=ServiceInstance: failed to create webhook-atlas-bf6c221f-2d70a4c7-2aa/atlas /v1, Kind=Service for acorn-controller webhook-atlas-bf6c221f-2d70a4c7-2aa/atlas: Service \"atlas\" is invalid: spec.externalName: Invalid value<EMAIL_ADDRESS>a lowercase RFC 1123 subdomain must consist of lower case alphanumeric characters, '-' or '.', and must start and end with an alphanumeric character (e.g. 'example.com', regex used for validation is '[a-z0-9]([-a-z0-9]*[a-z0-9])?(\\.[a-z0-9]([-a-z0-9]*[a-z0-9])?)*'), requeuing"
...

Testing the same service with a simple container like in https://gitlab.com/web-hook/config/-/blob/main/apps/acorn/Acornfile works fine though. Any idea of the reason of this error ? Is that linked to the application itself ? Could this be inked to the router definition ?

Closing for now as Atlas service was not correctly specified.
Might add some more details to this issue later on for documentation purposes.


NTR: iron iron sensor

iron iron sensor
standard definition
PMID:11956219
parent
GO:0140299  small molecule sensor activity

duplicate https://github.com/geneontology/go-ontology/issues/19433


Word and Silence combined

With the french model :

This is almost surely not a bug, simply an indication that the alignment is difficult here. This is probably because the file is over 11 minutes long! Try chopping it up into smaller pieces first: I usually have one input file per breath unit.

Thanks for your reply.
But it's possible to split my wav file automaticaly ? I want to do a force
alignement on lot of speech of François Holland or Barack Obama and with
word time informations, split speech for extract single word and make a
dictionnary. If I must split mannualy before force alignement, it's
useless. Actualy, I use EasyAlign, a PraatPlugin and it work with 11min wav
file. But I want a more powerful force alignement tool, I want a perfect
precision time.
Any ideas?
Sorry for my very bad english.
Le 9 avr. 2015 11:59 PM, "Kyle Gorman"<EMAIL_ADDRESS>a écrit :

This is almost surely not a bug, simply an indication that the alignment
is difficult here. This is probably because the file is over 11 minutes
long! Try chopping it up into smaller pieces first: I usually have one
input file per breath unit.
—
Reply to this email directly or view it on GitHub
https://github.com/prosodylab/Prosodylab-Aligner/issues/41#issuecomment-91367093
.

After force alignement, I split speech tout word with ffmpeg, but my force
alignement it's not accurate.
Le 10 avr. 2015 10:24 AM, "Louis Aubert"<EMAIL_ADDRESS>a écrit :

Thanks for your reply.
But it's possible to split my wav file automaticaly ? I want to do a force
alignement on lot of speech of François Holland or Barack Obama and with
word time informations, split speech for extract single word and make a
dictionnary. If I must split mannualy before force alignement, it's
useless. Actualy, I use EasyAlign, a PraatPlugin and it work with 11min wav
file. But I want a more powerful force alignement tool, I want a perfect
precision time.
Any ideas?
Sorry for my very bad english.
Le 9 avr. 2015 11:59 PM, "Kyle Gorman"<EMAIL_ADDRESS>a
écrit :

This is almost surely not a bug, simply an indication that the alignment
is difficult here. This is probably because the file is over 11 minutes
long! Try chopping it up into smaller pieces first: I usually have one
input file per breath unit.
—
Reply to this email directly or view it on GitHub
https://github.com/prosodylab/Prosodylab-Aligner/issues/41#issuecomment-91367093
.

I am simply suggesting that you provide the aligner the same information you provide EasyAlign, namely, approximate utterance boundaries. The way you communicate this information to this tool is to segment long sound files into shorter ones—11 minute chunks are far too long, one minute might work, utterance sized units will definitely work. If you like the EasyAlign workflow, then do this first step in Praat, and then use a Praat script to split the files into shorter audio files, then run the aligner.
Perfect precision is simply not possible by automated means. If you need very precise timing information, then you should manually automated alignments. Most people find this manual correcting much, much faster than doing it manually.

On Apr 10, 2015, at 4:24 AM, aubert-creation<EMAIL_ADDRESS>wrote:
Thanks for your reply.
But it's possible to split my wav file automaticaly ? I want to do a force
alignement on lot of speech of François Holland or Barack Obama and with
word time informations, split speech for extract single word and make a
dictionnary. If I must split mannualy before force alignement, it's
useless. Actualy, I use EasyAlign, a PraatPlugin and it work with 11min wav
file. But I want a more powerful force alignement tool, I want a perfect
precision time.
Any ideas?
Sorry for my very bad english.
Le 9 avr. 2015 11:59 PM, "Kyle Gorman"<EMAIL_ADDRESS>a écrit :

This is almost surely not a bug, simply an indication that the alignment
is difficult here. This is probably because the file is over 11 minutes
long! Try chopping it up into smaller pieces first: I usually have one
input file per breath unit.
—
Reply to this email directly or view it on GitHub
https://github.com/prosodylab/Prosodylab-Aligner/issues/41#issuecomment-91367093
.

—
Reply to this email directly or view it on GitHub.

edit: “you should manually automated alignments” -> “you should manually correct automated alignments”
K

On Apr 10, 2015, at 4:27 AM, aubert-creation<EMAIL_ADDRESS>wrote:
After force alignement, I split speech tout word with ffmpeg, but my force
alignement it's not accurate.
Le 10 avr. 2015 10:24 AM, "Louis Aubert"<EMAIL_ADDRESS>a écrit :

Thanks for your reply.
But it's possible to split my wav file automaticaly ? I want to do a force
alignement on lot of speech of François Holland or Barack Obama and with
word time informations, split speech for extract single word and make a
dictionnary. If I must split mannualy before force alignement, it's
useless. Actualy, I use EasyAlign, a PraatPlugin and it work with 11min wav
file. But I want a more powerful force alignement tool, I want a perfect
precision time.
Any ideas?
Sorry for my very bad english.
Le 9 avr. 2015 11:59 PM, "Kyle Gorman"<EMAIL_ADDRESS>a
écrit :

This is almost surely not a bug, simply an indication that the alignment
is difficult here. This is probably because the file is over 11 minutes
long! Try chopping it up into smaller pieces first: I usually have one
input file per breath unit.
—
Reply to this email directly or view it on GitHub
https://github.com/prosodylab/Prosodylab-Aligner/issues/41#issuecomment-91367093
.

—
Reply to this email directly or view it on GitHub.

ok, thanks.
PS: I have split my wave file in one sentence... poor result, I going to
try to train a new french model with very lot of data of Francois Holland
speech(> 10hours), may be a more accurate result.
2015-04-11 17:33 GMT+02:00 Kyle Gorman<EMAIL_ADDRESS>
edit: “you should manually automated alignments” -> “you should manually
correct automated alignments”
K

On Apr 10, 2015, at 4:27 AM, aubert-creation<EMAIL_ADDRESS>wrote:
After force alignement, I split speech tout word with ffmpeg, but my
force
alignement it's not accurate.
Le 10 avr. 2015 10:24 AM, "Louis Aubert"<EMAIL_ADDRESS>a
écrit :

Thanks for your reply.
But it's possible to split my wav file automaticaly ? I want to do a
force
alignement on lot of speech of François Holland or Barack Obama and
with
word time informations, split speech for extract single word and make a
dictionnary. If I must split mannualy before force alignement, it's
useless. Actualy, I use EasyAlign, a PraatPlugin and it work with
11min wav
file. But I want a more powerful force alignement tool, I want a
perfect
precision time.
Any ideas?
Sorry for my very bad english.
Le 9 avr. 2015 11:59 PM, "Kyle Gorman"<EMAIL_ADDRESS>a
écrit :

This is almost surely not a bug, simply an indication that the
alignment
is difficult here. This is probably because the file is over 11
minutes
long! Try chopping it up into smaller pieces first: I usually have one
input file per breath unit.
—
Reply to this email directly or view it on GitHub
<
https://github.com/prosodylab/Prosodylab-Aligner/issues/41#issuecomment-91367093

.

—
Reply to this email directly or view it on GitHub.

—
Reply to this email directly or view it on GitHub
https://github.com/prosodylab/Prosodylab-Aligner/issues/41#issuecomment-91869909
.

Ok,
All my videos are taken from youtube or Daylimotion, with subtitle. With
the subtitle files in SRT format, I have time information on every sentence
spoken. So I'm going through ffmpeg, break my videos and use
Prosodylab-Aligner  to train a model or use the French model
2015-04-12 3:18 GMT+02:00 Louis Aubert<EMAIL_ADDRESS>
ok, thanks.
PS: I have split my wave file in one sentence... poor result, I going to
try to train a new french model with very lot of data of Francois Holland
speech(> 10hours), may be a more accurate result.
2015-04-11 17:33 GMT+02:00 Kyle Gorman<EMAIL_ADDRESS>
edit: “you should manually automated alignments” -> “you should manually
correct automated alignments”
K

On Apr 10, 2015, at 4:27 AM, aubert-creation<EMAIL_ADDRESS>wrote:
After force alignement, I split speech tout word with ffmpeg, but my
force
alignement it's not accurate.
Le 10 avr. 2015 10:24 AM, "Louis Aubert"<EMAIL_ADDRESS>a
écrit :

Thanks for your reply.
But it's possible to split my wav file automaticaly ? I want to do a
force
alignement on lot of speech of François Holland or Barack Obama and
with
word time informations, split speech for extract single word and make
a
dictionnary. If I must split mannualy before force alignement, it's
useless. Actualy, I use EasyAlign, a PraatPlugin and it work with
11min wav
file. But I want a more powerful force alignement tool, I want a
perfect
precision time.
Any ideas?
Sorry for my very bad english.
Le 9 avr. 2015 11:59 PM, "Kyle Gorman"<EMAIL_ADDRESS>a
écrit :

This is almost surely not a bug, simply an indication that the
alignment
is difficult here. This is probably because the file is over 11
minutes
long! Try chopping it up into smaller pieces first: I usually have
one
input file per breath unit.
—
Reply to this email directly or view it on GitHub
<
https://github.com/prosodylab/Prosodylab-Aligner/issues/41#issuecomment-91367093

.

—
Reply to this email directly or view it on GitHub.

—
Reply to this email directly or view it on GitHub
https://github.com/prosodylab/Prosodylab-Aligner/issues/41#issuecomment-91869909
.

@aubert-creation Hi there, have you managed to train a good french model? Would you be open to sharing it? I'm in the same situation as you and it would save me A LOT of time if I did not have to train the model myself...