Add DataContainer superclass

Closes #272.
In this implementation, the superclass is not very functional. It only inherits from Equivalence, removing the requirement therefore from the subclasses.
Ideally, the superclass would take define some common fields (attributes). However, any field (attribute) defined in the superclass would be first in de init function. label and name have default values in EITData, which would not work with non-default fields like path. Also, path would no longer be the first argument, potentially breaking existing code. To move fields toDataContainer, they would have to be the first arguments, or we would have to switch to keyword-only arguments.
This PR also removes superfluous inheritance from Sequence.

@DaniBodor This is a super simple PR. I'd like not to complicate this PR, so we can use this superclass for #269.

Ideally, the superclass would take define some common fields (attributes). However, any field (attribute) defined in the superclass would be first in de init function. label and name have default values in EITData, which would not work with non-default fields like path. Also, path would no longer be the first argument, potentially breaking existing code. To move fields toDataContainer, they would have to be the first arguments, or we would have to switch to keyword-only arguments.

I agree with this, and I think this should be implemented. I also think that the first fields should be identical for each container, and then the specific arguments per container should follow after that. Also, I think that nomenclature should be aligned more. For example, vendor and pixel_impedance in EITData in my mind is the equivalent of category and values (respectively) in the other containers.
Regarding breaking existing code, this is an argument that you have made a few times now (e.g. in issue #262). I don't think we have a large enough user base at this point to make that a prohibitive reason not to change things. We are very much still in the developmental phase of this project, and it's more important to "get things right", even if that means breaking changes, than ensuring backwards compatibility.
I think these kinds of changes can be implemented in a separate PR though, so that this one can stay light, as you suggest.

I agree with this, and I think this should be implemented. I also think that the first fields should be identical for each container, and then the specific arguments per container should follow after that.

I created an issue for this. We can discuss how to approach this in #279.

Also, I think that nomenclature should be aligned more. For example, vendor and pixel_impedance in EITData in my mind is the equivalent of category and values (respectively) in the other containers.

category is going to point to what type of data is stored in an object (e.g. impedance, pressure or breaths), so algorithms can check whether they support this type of data. I'm working on a proposal for a proper implementation of categories. A vendor is not a data type, and should not be a category.
We decided to explicitly make EITData.pixel_impedance different from values because it is not immediately apparent what the values are you get from EITData.values. Is it pixel impedance, or functional impedance, or...? I think one advantage of data-specific classes is that you can customize things like this.

Regarding breaking existing code, this is an argument that you have made a few times now (e.g. in issue #262). I don't think we have a large enough user base at this point to make that a prohibitive reason not to change things. We are very much still in the developmental phase of this project, and it's more important to "get things right", even if that means breaking changes, than ensuring backwards compatibility.

I agree the user base is not large, but it is annoying enough for a few people here working on multiple projects at the same time. I think we should at least be more clear about what breaks and how to fix it in release notes, and minimise the number of times things break.


Correct the Instant Client 18.3 build syntax

Add the missing 'current directory' argument to docker build.

Thanks. I meant to do this but decided to spend the weekend in Yosemite National Park instead. 😁


Sign-up & Login flow improvement

Design
https://www.figma.com/file/7dCC4YlzF8tFOooA5M8b5y/G%24-Wallet?node-id=149%3A1478
The latest version of design is located the right side, in the block diagram
Context
https://app.zenhub.com/workspace/o/gooddollar/gooddapp/issues/3593
P1 Overview
The purpose of this ticket is to simplify the existing signup flow by replacing the initial screens with one simple screen and replacing the log in/sign up flows with a single “Continue” screen. We want to know if removing friction from the explanatory steps will increase or decrease sign-up conversion.
We need to provide the user with a simple guide on what is GoodDollar for those needing more info, and to track clicks.
We also need to still be able to differentiate a new user from a returning user.
Tasks
Logic

[x] move checkExisting to separate hook
[ ] add this hook usage to the AuthTorus, call it just after SSO login <----- This will not work
[ ] useAlreadySignedUp should use new 'you already have wallet' screen
[x] according to user's existence perform login or go to signup
[x] current "Welcome" (InviteWelcome ?) should be kept without changes
[x] signup screen to be replaced with the new "get started" screen https://www.figma.com/file/7dCC4YlzF8tFOooA5M8b5y/G%24-Wallet?node-id=272%3A6197
[x] login screen to be updated with "continue with". But on the dev we need to add a "Self custody" link and "Recover from seed" link. All self custody flow - without changes
[ ] new screen "you already have wallet". show it
a) after login when we checked for existense and found account but with the differen login type
b) if after complete profile we re checkend existence  and found account but with the differen login type
[x] profile screens wthout changes
[x] phone is not mandatory
[ ] new screen 'preparing the wallet'. instead of 'preparing' popup. show 'preparing' message until login (or account create). once done, show 'congrats/ welcome' wait a few sec and restart
[ ] make 'we're preparing wallet' a separate reusable component, render it onto login screnn while torus is working (now we're showing popup in this case).
[ ] Check verification method and change the text of this screen to the corresponding one https://www.figma.com/file/7dCC4YlzF8tFOooA5M8b5y/G%24-Wallet?node-id=272%3A6074 It could be email or phone

Design

[x] Add first (welcome) screen according the new design
[x] Adjust signup screen according the new design
[x] Adjust profile details according the new design
[ ] Add screen preparing the wallet
[ ] Add screen you already have wallet
[ ] Adjust confirmation screen according the new design
[ ] adjust final screen according the new design

The latest version of design is located the right side, in the block diagram

[x] move checkExisting to separate hook
[ ] add this hook usage to the AuthTorus, call it just after SSO login - [ ] useAlreadySignedUp should use new 'you already have wallet' screen

this would not work as intended, what the hook does is to show the user already has a wallet popup **during the singup flow ** it makes sense to have that hook there, but there is no sense on having it in the authTorus screen as it will not apply the new signin flow.

@TomasGonzalez then the hook should be adjusted. it should take some object
{
 fullName?: string;
 email?: string;
 mobile?: string;
}

and work with it. check for the existence by email / phone / name.
after SSO you pass data received from torus, after signup - the data from the signup screens

@TomasGonzalez actually it already works like i described.
the funciton it exports is taking searchBy as agrument. so you could pass any name/email/phone object despite you got it from SSO or after signup screens

@TomasGonzalez i understand what do you meaning. Will describe what to do a bit later and also we will discuss it or the call

So:

useCheckExisting should:

take name/email/phone/typeOfLogin object.
check for existence
if not exists return false
if exists and type of login matches return true
if exists and type of login not matches redirect to AlreadyExists screen

useAlreadySignedUp  is not needed, it should be replaced with AlreadyExists screen

do not use screen state to keep signup profile details, add AuthContext and keep data here.

@patpedrosa

The message "something went wrong" is misleading, like an error happened.
also the image is misleading as some kind of fatal error happened.
Maybe "Account with same email/mobile already exists" is better, hmm i think thats actually the current message:)

@johnsmith-gooddollar regarding invite welcome I think it was decided to remove those screens, please verify with @patpedrosa
@TomasGonzalez @johnsmith-gooddollar I dont think new route is a good way to handle it. please keep current code with minimal changes. I want to remind that we refactor only if you get significant improvement.

@johnsmith-gooddollar regarding invite welcome I think it was decided to remove those screens, please verify with @patpedrosa

have asked her in the PR's comments: https://github.com/GoodDollar/GoodDAPP/pull/3612#issuecomment-1014384663

@TomasGonzalez @johnsmith-gooddollar I dont think new route is a good way to handle it. please keep current code with minimal changes. I want to remind that we refactor only if you get significant improvement.

ok, i will think today how to avoid the new route here

@danielsha22 Could you please find a solution for Hadar's comment above regarding the "You already have a wallet" screen?


fix(replace static mut with static AtomicBool)

Hey, this replaces global static mut with AtomicBool which is thread-safe

Thanks for the contribution! You probably figured out, that the whole API is not threadsafe (does not implement Sync), but still, the acquiring of the API guard should be threadsafe.


Cannot connect to customized odata service

I have an odata backend. https://mobiledev-sdkforandroid-integration-projcomp-service.cfapps.sap.hana.ondemand.com and when i call  client = pyodata.Client(SERVICE_URL, session) i always get PyODataModelError
This backend is No auth.
Which is generated by Web IDE.
raise PyODataModelError('Property {} does not exist in {}'.format(proprty, entity_type.name))
pyodata.exceptions.PyODataModelError: Property ParentProjectId does not exist in Action

Thank you for taking the time to report this issue.
Unfortunately does not support type inheritance and the type Action inherits from the type Item in your metadata.
We will try to fix it ASAP.

closing as not reproducible / not actionable +  no communication.


The error message page does not have a scroll bar

Describe the bug
I encountered an error while creating a virtual machine, which is normal.
But the error message was so long and there was no scroll bar that I couldn't even click the close button.
To Reproduce
Any mistake is fine, as long as it is long enough.
But I met it when I was creating nixos.
Expected behavior
Screenshots

Logs
Although there is an error log, I don't think it's important. Just solve the problem of ui first.
Info report

Additional context
Thank you very much

By the way, whether orbstack follows the proxy settings of the system, because I do not see where the proxy can be set, I think my error is a network error, I need an proxy.

Fixed for the next version.
As for proxy support, that was added the week after you reported this issue: https://github.com/orbstack/orbstack/issues/2

Released in v0.11.1.


Add bypass_cmek_check requirements to plugins that do not have CMEK s…

…upport

Are these the only plugins that need CMEK?

These are the GCP plugins that do not support CMEK yet, thus require bypass_cmek_check to be enabled on cdap instance otherwise user would not be allowed to use them.


What's new add experimental features

Overview
Adds several missing new experimental features to the "what's new in 2.3" page.
Changes

Fixes: #7241

/azp run mrtk_docs

Thank you@julenka!


feat: load fbridge & fswap store

Description

This patch adds setupUpgradeStoreLoaders(), which handles store upgrades.
How has this been tested?

Start v3-Rose chain.
Upgrade the chain to v4-Marigold.
Access the new module's state.

e.g. fnsad q fswap swaps

Expected results:

Without the patch: panics
With the patch: returns an empty array of swaps.

Checklist:

[ ] I followed the contributing guidelines and code of conduct.
[x] I have added a relevant changelog to CHANGELOG.md
[ ] I have added tests to cover my changes.
[ ] I have updated the documentation accordingly.

How can we test this modifications? Because there is no unittest.

I've added the test in the PR description.


Allow "vertical" behavior for function calls

(Closest issue to this I could find was #65)
Currently, fn_args_layout = "Vertical" will force all arguments on the function definition to be on their own line. I would like to have a way to get the same "vertical" behavior applied at callsites.
I'm not proposing that this be a default behavior, I'd just like to be able to opt-in to it.
Current Behavior
draw_circle(radius, position, color);

LogicalPosition::new(p0.x - p1.x, p0.y - p1.y)

Desired Behavior
draw_circle
    radius,
    position,
    color
);

LogicalPosition::new(
    p0.x - p1.x,
    p0.y - p1.y
);

Rationale

It's not uncommon to insert new parameters, or change parameters, being passed into a function. Allowing vertical behavior makes for clean diffs and potentially avoids some merge conflicts. (Guiding principle 3 "Specifics")
In some cases there is symmetry that is helpful to see. When the the symmetry is broken, it's more conspicuous. This can make bugs easier to spot, or if it's not a bug, help the reader to see this. (Guiding principle 1 and 2 "Readability" and "Aesthetics")
In my opinion, this would be consistent with current fn_args_layout behavior (Guiding Principle 4 "Application")

Implementation
I think allowing the same Tall/Compressed/Vertical choice for function calls as already exists for function definitions would be a consistent, intuitive approach.

A simple, low impact solution would be to add a new config variable like fn_call_args_layout. This would make this change strictly opt-in.
Another possibility would be to add fn_definition_args_layout and fn_call_args_layout and have them both default to whatever value is set for fn_args_layout. I think this would make the change only impact those who already opted into customized fn_args_layout behavior. It's probably a matter of opinion if this is a good thing or not.

I think this is a fair ask so long as the default still complies with style guide prescriptions, but it would probably be better suited to the rustfmt repo where the implementation could be discussed in more detail (I think I can transfer it if you'd like)
I realize it's often unclear which issues should be opened in which repo, but in general I think a request for a specific new feature in rustfmt is probably a better fit for that repo.
Let me know if you're still interested in this option and if you'd like me to attempt the transfer, otherwise I'll close it here

Working on triaging issues on the style team repo (formerly fmt-rfcs).
Agreed with @calebcartwright that this seems to be a request for an option enabling a non-default style, rather than a request for a change to the default style. Given that, I'm going to close this, and recommend filing an issue on the rustfmt repo if you want to ask for an option.


[CHECKLIST] QA - Hubble Stats

Here is the list of tasks that need to be tackled after the QA Engineering session on September 13th.

[ ] #1625
[x] #1626
[ ] #1627
[ ] #1628
[ ] #1629

There are also some other tasks created to fix existing bugs, increase UX and performance, and improve clarity in code:

[x] #1621
[ ] #1630
[ ] #1631
[ ] #1609

Replace #1621 with #1646, reasons are mentioned in #1621

Remove #1631 from this checklist since that task is only about optimizing performance!

Remove #1628 from this checklist, this task is deprioritized since there are not many real-time updates on the site at the moment!

Removed #1630 from the list since this is refactoring task, not a major issue!

Re-add https://github.com/webb-tools/webb-dapp/issues/1628 since we have successfully migrated to the latest version of Next.js and can now begin investigating this.


implementing percentage metrics computation to pivot table

This PR allow to use percentage metrics in pivot table visualization.
Then an user has the capability to compute percentage on metrics in the pivot table by column or line.

:+1:

Hello,
just a remind about this feature which can be interesting for pivot-table charts.
Waiting for merge :-)


Base fee type mismatch

The MIN_BASE_FEE is int in AbstractTransaction.java but the setBaseFee() is long in TransactionBuilder.java..

Hi, @lijamie98, thank you for reporting this, I will include it in the next release.


Remove PLATFORMS - arm64-darwin-22

I think it's wise to remove this line immediately from Gemfile.lock so it doesn't sneak into any branches and disable production.
It's on main now, so it appeared on one of my branches when I merged main into it. It doesn't seem very foolproof to have to keep checking that.

coverage: 94.406%. remained the same when pulling ca0386d042eaeacfee56dbaabc241e1ce65fc295 on nimmo-remove-arm-darwin-22 into 4b4be0039fa55420cf98b5e5fa3b78693782d983 on main.

I think this is a wise short-term measure to prevent accidentally clobbering the production server.
But when I asked @mo-nathan about that said (in an email in which I forgot to cc mo-developers),
@mo-nathan said:

I’d actually be interested in trying to reproduce the behavior, but I don’t fully understand what was done.  I’m having a hard time understanding why an additional line in the platforms section of the Gemfile.lock could cause the problem.  I’m wondering if it’s something subtler in the gem configuration or something that’s wrong in the deploy.sh script.  Unfortunately the only way I think we can reproduce it is by potentially breaking production.  My guess is that if we add that line back in, rerun bundle install and restart unicorn, it will just work.  However, I want to do that test sometime when I have some time to really bang on it which is not today (I’m heading to a wedding at the moment).

I still think it's best to remove it to prevent accidentally killing the web site. It can always be restored for test purposes.
2. Is there any magic to removing via bundle-lock --remove-platform vs simply deleting the line in Gemfile.lock. ()
3. Will it inadvertently reappear? For instance, what happens if you change Gemfile, then bundle install.

I believe it can only inadvertently appear if Nathan runs bundle install, because he's the one running Ruby with the new architecture. It did not appear for me when I ran bundle install.

Re:

Is there any magic to removing via bundle-lock --remove-platform vs simply deleting the line in Gemfile.lock. ()

I don't know. I believe Jason just deleted the line manually?

Perhaps a more fail-safe approach would be also changing Gemfile to impose
platform constraints on nokogiri.
nokogiri and its dependency libv8-node are the only gems that have arm64
versions installed.)
ChatGPT claims that one can do
gem 'nokogiri', '~> 1.12.3', platforms: [:x86_64_linux] But maybe it's just
making stuff up.
On Sat, Oct 7, 2023 at 1:12 PM andrew nimmo @.***>
wrote:

Re:

Is there any magic to removing via bundle-lock --remove-platform vs
simply deleting the line in Gemfile.lock. ()

I don't know. I believe Jason just deleted the line manually?
—
Reply to this email directly, view it on GitHub
https://github.com/MushroomObserver/mushroom-observer/pull/1663#issuecomment-1751808492,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAALDFAIVV7SZ3KXXPXLAZ3X6GZSPAVCNFSM6AAAAAA5XEGEXCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONJRHAYDQNBZGI
.
You are receiving this because your review was requested.Message ID:
@.***>

Interesting... although I feel like ideally, I wouldn't want to lock the nokogiri version.
Let's wait if Nathan turns up something.


ReferenceError: head is not defined

Hello & thank you for your work on peggy and for picking up where pegjs left off.
I'm one of the maintainers of amqplib and investigating https://github.com/amqp-node/amqplib/issues/453. amqplib uses pegjs via bitsyntax to generate a parser from the grammar specified here.
bitsyntax's last working parser was generated using<EMAIL_ADDRESS>and upgrading to<EMAIL_ADDRESS><EMAIL_ADDRESS>or peggy@latest results in a broken parser.
/Users/steve/Development/squaremo/bitsyntax-js/lib/parser.js:256
  var peg$f17 = function() { return head + tail.join(''); };
                             ^

ReferenceError: head is not defined

I'm completely new to pegjs/peggy and was hoping it's an easy fix. Can you give me any pointers please?
Thanks

I grabbed the grammar, ran it through peggy's CLI with the default options, then called parse('"foo", "bar"'), and got [ { string: 'foo' }, { string: 'bar' } ].  I suspect the issue may be here.  Try making that like be:
	 $(PEGJS) $(GRAMMAR) -o $@

(add -o to direct the output to a file).  My suspicion is that you're getting an error instead of generating the new implementation.  Assuming it's not that, can you tell us how you're calling peggy, and an example input that's failing for you?
I agree that the line you've highlighted looks wrong.  Here's what I get:
var peg$f0 = function(head, tail) { tail.unshift(head); return tail; };

Hi @hildjj,
Thanks for the quick response. I also noticed that the Makefile is out of date. I've generated the parser by hand as follows
rm lib/parser.js
./node_modules/.bin/peggy -o lib/parser.js lib/grammar.pegjs

Example input, please?  And can you check that you're using Peggy 2.0.1, which was just released yesterday?

OK, it has nothing to do with the build process.  Remove the parens on line 43:
identifier
    = head:[_a-zA-Z] tail:[_a-zA-Z0-9]* { return head + tail.join(''); }

which ends up being "a" + ["b", "c"] which is a roundabout way of getting the string that matched.  You could just use:
identifier
    = [_a-zA-Z] [_a-zA-Z0-9]* { return text(); }

Some old version of peg.js must have changed how parens are handled.  In today's world, they hide labels from the containing rule.  Once I got that working, there are a bunch of other test errors, but that's a separate issue.

Thanks @hildjj. I'll close this, and try to work out why the tests are failing.

Let us know if you'd like more help with the grammar.  Happy to pitch in as needed.

Actually, your original grammar works because of a bug, fixed in<EMAIL_ADDRESS>
Fixed label scoping so that labels in expressions like (a:"a") or (a:"a" b:"b" c:"c") aren’t visible from the outside.

Also, you could simplify your grammar much more using a new @ operator, for example, here:
segmentTail
    = ws ',' ws @segment

and using a $ operator instead of return text():
identifier
    = $([_a-zA-Z] [_a-zA-Z0-9]*)


Made flycheck work over TRAMP

Made flycheck work over TRAMP. Tidied a couple of other odds and ends from previous TRAMP work.
Key work in intero-parse-errors-warning-splices. flycheck wasn't correctly identifying the buffer to display errors in, because it was comparing a local and remote path (with string=). I made the paths remote (when remote), and wrote a comparison function for paths to replace string=, since TRAMP paths have a few variations.
Furthers solving #303. Builds on #340.
Commit notes
The background process was finding errors, but the flycheck backend was not identifying the buffer to display errors in. This was because it was comparing the absolute path from the buffer, and the local path from the background process. Fixed this, and provided a general utility function for comparing paths (since paths over TRAMP may differ despite representing the same file).
Did a couple of small improvements to the TRAMP compatibility code also (docstrings, factoring out TRAMP path identification).
Squash of:
Got flycheck to work over tramp
Flycheck was finding the errors, but because it didn't recognize the temporary file from emacs' perspective and from intero's perspective as being the same, it didn't display the errors.
Improved factoring
Made a function to compare paths, rather than relying on standardized paths being used as strings.
Tidying
Refactored path comparison function, so the "representation" function could be removed completely. Improved a couple of docstrings.

company completion is actually also working for me, but only if I set company-idle-delay to something other than the default 0.5 (and that's only an issue over TRAMP). Fairly sure that's not an issue with intero (found this: https://groups.google.com/forum/#!topic/gnu.emacs.sources/JUcjGrLMV40), but I might have a go at tracking it down.
A temporary workaround could be to set company-idle-delay to 0.49...

Yes, it doesn't look like intero code is entered at all. It looks like company just doesn't trigger completion. Interesting bug!

Thanks, the changes look very reasonable. Merging and I'll try it out!


properly handle basic types in meta examples

@sanuj @vigsterkr @lisitsyn
Basic types are now not treated as objects (with include paths) anymore

Forgot a change, fixed and squashed into single commit

Merging, as old travis run and new one are green together


SEO Optimization

Hi @teddybradford - a while back Jay Marol set up a Meta Description and a Meta Title field for us for the Universal - Pages content type. (See Github Issue #12 )On a give page what he created looks like this (this one specifically is for the "A close look page":

Anyway, we now have 24 out of our 30 pages with filled-in Meta Descriptions. (5 of the remaining pages I think we'll be able to delete from the CMS entirely and the other one I just have a question about - but anyway, those things are for another time.) With these 24 pages having Meta Descriptions and with these fields filled in in the Settings area of Webhook:

We were hoping you could help us ensure that our search results are coming up appropriately. Right now we have filled-in Meta Description's for a page sometimes showing in a google search and sometimes not. Ex: when you do a google search of: site:congratulations.risd.edu
and you look at the Enrollment and beyond page you see:

but if you just search "new risd students enrollment" in google the same page looks like:

(Please just let me know if you need more examples/specifics)
congratulations.risd.edu's title should be Welcome New Students | Rhode Island School of Design (Rather than Rhode Island School of Design: Welcome New Students) (The other pages titles seem to be working better.)
Thanks,
Carrie

@cmiller02 Unfortunately a lot of this is out of our hands. The best we can do is suggest to Google what we want displayed in their search results, which is what we are currently doing.
Specifically, Google doesn't always show the meta description designated in meta tags:

"…Google will use your search query to populate the description with what it thinks is relevant. So different search queries will pull different descriptions into Google results for the same URL. So what you see isn't what every searcher sees for every query." (source)

I understand, Teddy - thanks very much for the explanation. With meta descriptions now in for almost all pages (all but one right now I believe) and having just made 4 pages "drafts" (per Ruben's instructions) I think we're getting in better shape, SEO-wise, especially in terms of the descriptions that I'm seeing in google.
Not sure if this is similarly out of our hands or not, but is there anything we can do about the sometimes one | and sometimes two || pipes (see Planning, POSE, Neighborhoods) used in the search result page titles. Here's a screenshot:

Also, would it be possible to add the SEO Information fields: Meta Description and Meta Title, to the "Universal - Contact" content type? My thinking is that we'd then have the ability to add a better Meta Description to:

Thanks!

@cmiller02 I just deployed changes that: 1) fix the double pipe issue in titles and 2) adds optional meta title and meta description fields to the Contact page.

Thanks so much, @teddybradford ! Really appreciate these two things. I added a meta description to the Contact page and I've asked Lauren to give me one more description for another page and then I'm going to initiate a google recrawl of the site. If you don't mind, I'll just keep this issue open for a bit longer - hoping we see the improvements after the site recrawl and I can note that here in the issue before closing it. Thanks again!


\\[ should look like \\]

k
v

language
regex-tmLanguage

standard token type
Other

foreground
#56B6C2

background
#282C34

contrast ratio
5.91

textmate scopes
constant.character.escape.tm

...
meta.embedded.block.regexp.tmLanguage

...
source.json.textmate

foreground
constant.character.escape

...
{ "foreground": "#56B6C2" }

k
v

language
regex-tmLanguage

standard token type
Other

foreground
#ABB2BF

background
#282C34

contrast ratio
6.57

textmate scopes
punctuation.definition.tag.tm

...
meta.embedded.block.regexp.tmLanguage

...
source.json.textmate

foreground
meta.embedded

...
{ "foreground": "#ABB2BF" }

"match": "^[ \\t]*;\\s*\\[\\]"
;                      ^^^ #1
;                          ^^^ #2

I think it's a good idea to use the same color, after all, they have the same meaning.
Thanks ≧▽≦

I original reason for that was because the \\ isn't needed and is redundant on ]
^[ \\t]*;\\s*\\[] works fine with no errors
I have since learned about the 'dead code' feature
so I'll implement that feature instead soon™

you are right, I think I'm confusing the implementation details of js-regexp/oniguruma-regexp,
Usually I translate from js-regexp with u-flag back to onig-regexp.
// onig is OK
"match" : "^[ \\t]*;\\s*\\[]"

about js-regexp u flag

https://eslint.org/docs/latest/rules/require-unicode-regexp

Make the regular expression throwing syntax errors early as disabling Annex B extensions.

Because of historical reason, JavaScript regular expressions are tolerant of syntax errors. For example, /\w{1, 2/ is a syntax error, but JavaScript doesn’t throw the error. It matches strings such as "a{1, 2" instead. Such a recovering logic is defined in Annex B.
The u flag disables the recovering logic Annex B defined. As a result, you can find errors early. This is similar to the strict mode.


Submit homework_5 answers

What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)
Add answers to 02_activities/homework/homework_5.sql
What did you learn from the changes you have made?
SQLite does not support dynamic values like current_timestamp as DEFAULT in tables.
Creating tables from existing ones using the CREATE TABLE AS keywords
Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?
No, I didn't see a lot of specifics on what we should use so I was able to implement using my preferred approach.
Were there any challenges? If so, what issue(s) did you face? How did you overcome it?
The last question was pretty challenging.
I was able to complete this one through the lessons I learned from the class, with window functions being the key aspect of the solution.
I only learned this one in this class and I found it very interesting as it gave me a new way of sorting records further and allowing me to easier achieve my desired output.
How were these changes tested?
Ran each query, viewed results using SELECT * <table_name>
select statements were removed prior to check-in
A reference to a related issue in your repository (if applicable)
Checklist

[x] I can confirm that my changes are working as intended

HW5 marked.


update baseRootPath to be happier with npm@3

npm@3 now flattens node_modules directory
This should be happy no matter where the module ends up

Good idea, thank you! Just merged it to master.


completion of import declaration doesn't work on windows.

Fixes #332

I don't like the fix.
Maybe you could just replace the usage of "path/filepath" lib with "path"? I think it does POSIX paths by default everywhere.

Maybe you could just replace the usage of "path/filepath" lib with "path"?

I just replace backslash into slash because the path separator is always backslash on windows.

If you want, I'll add checking of runtime.GOOS.

I'm thinking, filepath.ToSlash doesn't define that it works or doesn't works for partial strings.

No, I don't like the fact of replacement. When there is a function above which does concatenation. My suggestion is to fix concatenation so that it does the right thing.
There is a path lib: https://golang.org/pkg/path/
It always uses the right kind of slash. You can just replace filepath usage with it.

Okay, however, docs should say it works for partial strings too, I think.

@mattn does #334 work for you?


UnauthorizedAccessException when updating on macOS Sequioa 15.0 (x64)

OS
macOS Sequioa 15.0 (x64)
Programming Language
dotnet 8.0
VPK Version
vpk 0.0.626
Library Version
Velopack 0.0.626
What happened?
When trying to apply updates on macOS (installed for everyone) I get an UnauthorizedException Access to the path '/tmp/velopack/{ApplicationName}' is denied.
Additionally it also says There is no local/base package available for this update, so delta updates will be disabled. before trying to update
Relevant log output
No response

removing the directory worked


Update Post “2019-01-15-doom”

Automatically generated by Netlify CMS

👷 Deploy Preview for jade-bunny-a7c9db processing.

Name
Link

🔨 Latest commit
273e7a8cfe1e366afc0c6b8431396463ae882807

🔍 Latest deploy log
https://app.netlify.com/sites/jade-bunny-a7c9db/deploys/6386a3487418330009727cb9


Got a CTD

Got a CTD when clicking the blacklist button, crashlog is here
System.Func11[System.String,System.UInt32,System.Boolean,System.Int64,System.String,System.Int32,System.Nullable1[System.Single],VRage.Game.Entity.MyEntity,System.Boolean,System.Boolean,VRage.Game.Entity.MyEntity], instead it's: System.Func12[System.String,System.UInt32,System.Boolean,System.Int64,System.String,System.Int32,System.Nullable1[System.Single],VRage.Game.Entity.MyEntity,System.Boolean,System.Boolean,System.Int64,VRage.Game.Entity.MyEntity]
at CoreSystems.Api.WcApi.AssignMethod[T](IReadOnlyDictionary2 delegates, String name, T& field)    at CoreSystems.Api.WcApi.ApiAssign(IReadOnlyDictionary2 delegates, Boolean getWeaponDefinitions)
at CoreSystems.Api.WcApi.HandleMessage(Object obj)
at Sandbox.ModAPI.MyAPIUtilities.SendModMessage(Int64 id, Object payload)
at CoreSystems.Api.ApiServer.HandleMessage(Object o)
at Sandbox.ModAPI.MyAPIUtilities.SendModMessage(Int64 id, Object payload)
at CoreSystems.Api.WcApi.Load(Action readyCallback, Boolean getWeaponDefinitions)
at Blacklist.Gui.WeaponDialog.WeaponList()
at Blacklist.Gui.WeaponDialog.ListWeapons()
at Blacklist.Gui.WeaponDialog.CreateListBox()
at Blacklist.Gui.WeaponDialog.RecreateControls(Boolean constructor)
at Blacklist.Gui.WeaponDialog..ctor(String caption)
at Blacklist.Logic.MainLogic.OnBlacklist()
at Sandbox.Graphics.GUI.MyGuiControlButton.HandleInput()
at Sandbox.Graphics.GUI.MyGuiScreenBase.HandleControlsInput(Boolean receivedFocusInThisUpdate)
at Sandbox.Graphics.GUI.MyGuiScreenBase.HandleInput(Boolean receivedFocusInThisUpdate)
at Sandbox.Game.Gui.MyGuiScreenToolbarConfigBase.HandleInput(Boolean receivedFocusInThisUpdate)
at Sandbox.Graphics.GUI.MyScreenManager.HandleInput()
at Sandbox.Graphics.GUI.MyDX9Gui.HandleInput()
at Sandbox.MySandboxGame.Update()
at Sandbox.Engine.Platform.Game.UpdateInternal()
at Sandbox.Engine.Platform.Game.RunSingleFrame()
at Sandbox.Engine.Platform.FixedLoop.<>c__DisplayClass11_0.b__0()
at Sandbox.Engine.Platform.GenericLoop.Run(VoidAction tickCallback)
at Sandbox.Engine.Platform.Game.RunLoop()
at Sandbox.MySandboxGame.Run(Boolean customRenderLoop, Action disposeSplashScreen)
at SpaceEngineers.MyProgram.Main(String[] args)
2021-12-27 04:42:47.993 - Thread:   1 ->  Showing message
2021-12-27 04:42:49.798 - Thread:   1 ->
================================== CRASH INFO ==================================
AppVersion: 01_199_025
GameName: Space Engineers
IsOutOfMemory: False
IsGPU: False
IsNative: False
IsTask: False
IsExperimental: True
ProcessRunTime: 437
PCUCount: 368242
IsHang: False
GCMemory: 3446
GCMemoryAllocated: 3446
HWAvailableMemory: 12637
ProcessPrivateMemory: 17698
AnalyticId: SE
================================== OFNI HSARC ==================================
2021-12-27 04:42:49.799 - Thread:   1 ->  Log Closed

I updated weaponcore api should be working now,

Ok, thank you.

I think the problem is still there as I still seem to be getting the same error when clicking the button. Is there something I need to do on my end?

I think weaponcore author updated his api again lol

Yeah, I think his mods are the most updated; 80-90% of the time when SE mods get an update it's one of his.

Same problem, Is there anyway we can get an update? or A Some guide on how we can fix it ourselves?


DATA INPUT SWARM!

Ok peeps, we've got a data entry blitz again.

Open the new data sheet: (https://docs.google.com/spreadsheets/d/144AkG-RSnB-AntD-7B7T6IGqt3W9Dxa7es6D9LhaAXk/edit?usp=sharing)
Comment here to claim a group of rows. Please take in units of 20, eg "I'll take 2-20!" "I'll take 21-40!" etc.
For each row, FIRST check if dupe by searching our live site at findthemasks.com (don't mess with filters, just search for the name of the hospital with ctrl-F)
if NOT dupe, fill in the form ( https://docs.google.com/forms/d/e/1FAIpQLSfgCpK5coPVFC6rJrE7ZhimiZuDoEaL6fo6gYqxsN_FIpJZhg/viewform) and look up missing info like address.
note: This new data doesn't have info on whether open boxes are ok. Please write "Unsure" in that field in the form.
Repeat 3-4 until you're at the end of your 20.
Come back here and edit your comment to say you're done!
(Optional) claim another 20!

Thank you so much. This one-time pain lets us keep scaling and keeps ongoing data moderation much easier than if we had multiple sources, or tried to merge data into a live form-fed sheet.

Do you want us to finish filling in the info for the places you already have the name of the clinic/hospital?

Hi Rachel, I'll take rows 2-20. =)

Do you want us to finish filling in the info for the places you already have the name of the clinic/hospital?

Not in the sheet (you won't be able to edit it), but yes into the form (step 4). you may need to google the address of the hospital, if they didn't provide it.

Ok thanks.

Looks like I’m the first, I’ll take 1-20

No I took those!

@jlittenberg you aren't! please refresh, and take the next unclaimed batch.

Hey Rachel, Shouldn't we call the hospital and get instructions and exact address for drop-off when filling in the form? Or just fill in what comes up on Google?

Also I'll just do 21-40 so me & @jlittenberg don't replicate

PAUSE THE BLITZ. @alexisjulia good catch. there was supposed to be something in that field.

Haha okay @r-pop!

Ok I think we are ready to unpause. You will need to start at the beginning. This is messier, but complete. Please let me know if you run into more issues.

PS, if you want to help, unclaimed rows start at 40 and we are open for data entry!

Also I'll just do 21-40 so me & @jlittenberg don't replicate

Done!

I'll do 41-60 now

I'll take 61-80

I'll take 81-100

I'll take 61-80

note there are multiple urgent care chains sometimes -- i'm picking the first address unless other ideas

Does dupe mean duplicate? I'll take 81-100

What should we do if we can't find some of the info on the form?

Does dupe mean duplicate? I'll take 81-100

What should we do if we can't find some of the info on the form?

If it supplies I've been checking "other" and writing "not specified"
If it's address, i've been picking the first one
If it's open boxes, write in "unsure"

Done with 81-100. Note that in most cases I couldn't find specific instructions for delivery/drop-off.

I'll take 101-120!

I'll take 121-140

Working on 141-160

I'll take 161-180!

I'll take 181-200!

Whoops, I accidentally entered in two that were already on the site:

Edward Hospital (in Naperville, IL)
AdventHealth Hendersonville

There are some data issues with a few of mine serious enough that I didn't want to enter them:

I skipped 122 since it is for Kaiser Los Altos, and as far as I can figure out there is no Kaiser in Los Altos.
I skipped row 124 for "Kaiser Permanente GA Region" since this is not one specific hospital, and I could not find a specific Kaiser Permanente hospital in Georgia that clearly advertised mask donations on their site after ~10 minutes of searching.
I skipped Row 127, which is as far as I can tell an individual person (visiting nurse). Should that make it over to the site?
I skipped Row 131, which is for a restoration (e.g. water/fire damage) company, not a hospital (!).

Also, when doing this I noticed Kaiser Permanente Santa Clara on the live site is misfiled since the address is in San Francisco.

I'll take 201-220!

I'll take 221-240!

I'll take 241-260!

I'll take 261-280!

I'll take 281-300!

I'll take 301-314!

The spreadsheet is now finished.

you all are AWESOME
On Tue, Mar 24, 2020 at 6:30 AM per1234<EMAIL_ADDRESS>wrote:

The spreadsheet is now finished.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/r-pop/findthemasks/issues/161#issuecomment-603157438,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AO5PBKJCZSAZ4NQYUHOQ65DRJCDS3ANCNFSM4LSIJXYQ
.

I'll do 41-60 now

Done!

 If you would like a mention on findthemasks.com/whoweare.html, I would be delighted to add you there. Just let me know what name or username you want displayed, and (optionally) where to link it to .

You can add my name "Caitlin Sadowski" linked to https://research.google/people/CaitlinSadowski/
Also, pasting my update above here to make sure you saw it:

I'll take 121-140. Done, except:
Whoops, I accidentally entered in two that were already on the site:

Edward Hospital (in Naperville, IL)
AdventHealth Hendersonville

There are also some data issues with a few of mine serious enough that I didn't want to enter them:

I skipped 122 since it is for Kaiser Los Altos, and as far as I can figure out there is no Kaiser in Los Altos.
I skipped row 124 for "Kaiser Permanente GA Region" since this is not one specific hospital, and I could not find a specific Kaiser Permanente hospital in Georgia that clearly advertised mask donations on their site after ~10 minutes of searching.
I skipped Row 127, which is as far as I can tell an individual person (visiting nurse). Should that make it over to the site?
I skipped Row 131, which is for a restoration (e.g. water/fire damage) company, not a hospital (!).

Also, when doing this I noticed Kaiser Permanente Santa Clara on the live site is misfiled since the address is in San Francisco. Not sure how to fix this.

@supertriceratops I did see it, and the data mods read it. we are underwater with comms since this is a rapid response project. Thank you!


feat(kv): add support for WithCursorLimit to ForwardCursor

This introduces the ability to push down a limit for the number of keys returned for a ForwardCursor via the WithCursorLimit(n) option.
Update:
Bringing this back from the dead as it will be crucial for efficient pagination across system wide lookups of resources.

[x] CHANGELOG.md updated with a link to the PR (not the Issue)
[x] Well-formatted commit messages
[x] Rebased/mergeable
[x] Tests pass
[x] http/swagger.yml updated (if modified Go structs or API)
[x] Documentation updated or issue created (provide link to issue/pr)
[x] Signed CLA (if not already signed)

I let this one go stale for a while. However, I am bringing it back to improve the performance of pagination.


docs-bug(Datepicker):  there is no warning about the need to import.

Documentation Feedback
It would be interesting to warn that I need to import MatNativeDateModule to use MatDatepickerModule components.
Thank you.
Affected documentation page
https://material.angular.io/components/datepicker/api

We have a runtime error if no date adapter is provided. Also the date adapter you pick depends on what object you're using to represent dates. There's more info in the docs: https://material.angular.io/components/datepicker/overview#choosing-a-date-implementation-and-date-format-settings


Flight Replay example uses incorrect API

The Flight Replay example sets waypoints using mavutil.mavlink.MAVLink_mission_item_message directly. As I understand the API the correct type for a command in this API is "Command" (which is not even imported)
    cmd = mavutil.mavlink.MAVLink_mission_item_message(0,
                                                     0,
                                                     0,
                                                     mavutil.mavlink.MAV_FRAME_GLOBAL_RELATIVE_ALT,
                                                     mavutil.mavlink.MAV_CMD_NAV_WAYPOINT,
                                                     0, 0, 0, 0, 0, 0,
                                                     lat, lon, altitude)
    cmds.add(cmd)

To fix this I believe the change is trivial, because the Command is just a thin wrapper around MAVLink_mission_item_message

from droneapi.lib import VehicleMode, Command
...
    cmd = Command(0,
                                                     0,
                                                     0,
                                                     mavutil.mavlink.MAV_FRAME_GLOBAL_RELATIVE_ALT,
                                                     mavutil.mavlink.MAV_CMD_NAV_WAYPOINT,
                                                     0, 0, 0, 0, 0, 0,
                                                     lat, lon, altitude)
    cmds.add(cmd)

@hamishwillee did you try this in SITL?

I ran both the original code and with my changes above. In both cases the output "looked reasonable" and was the same.
BUT, I did NOT go through the code fully yet and confirm that it is all working as expected. So "looking reasonable" is not a canonical "this code works perfectly".

Tried this in SITL and the fixed file works. I have made this change. Closing as fix is in #81


LibWeb: Use correct specifier to pad font language override value

Previously the browser would crash when parsing short font-language-override values.
Fixes #2423

I can confirm this PR fixes us 476 subtests


Set repo structure: core and utils

We need to split functionality into utilities and actual core functionality.
In this case I am specifically thinking about extracting the align and bin functions from modules in repo, and place them into a general utils module.

Fixed with https://github.com/vcristiani/galaxy-chop/commit/1c8f85e419e7f307c57082286cca402234c03e44 and https://github.com/vcristiani/galaxy-chop/commit/80b04c53eaf27f83e2272c6b9b34f347b7b7d104


feature: support http put

[ ] I have registered the PR changes.

Ⅰ. Describe what this PR did
使AbstractHttpExecutor类支持PUT方式的请求
Ⅱ. Does this pull request fix one issue?

fixes https://github.com/seata/seata/issues/5067
Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

 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.zsp419 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

我明明签署了cla，但是这里一直显示没有签署，用户名都是zsp419，该怎么办呢


The GCP example in the terratest page points to the wrong file.

The GCP example on the live page points to the wrong url.

Current Link: github.com/gruntwork-io/terratest/test/terraform_gcp_hello_world_example_test.go
Correct Link:  github.com/gruntwork-io/terratest/test/gcp/terraform_gcp_hello_world_example_test.go

added a PR to solve this :)

thanks @brikis98 I will close at its now live 😄


Use semantic versioning

The swift package manager requires that packages use semantic versioning to automatically pull files.  It would be nice if Git2Swift followed this scheme for it's releases:
MAJOR.MINOR.PATCH
MAJOR version when you make incompatible API changes,
MINOR version when you add functionality in a backwards-compatible manner, and
PATCH version when you make backwards-compatible bug fixes.
The current scheme of 0.X, is missing the PATCH which the package manager does not like

Yes you right. It's time to create 1.0.0 version. I will use it after I have reviewed your pull request.


Add Paginator + paginated results

Q
A

Bug fix?
no

Improvement?
no

New feature?
yes

BC breaks?
no

Deprecations?
no

Docs PR
—

As it turns out, the implementation is way more easy than I previously thought / implemented it... Oops.
Sort-of a BC break in the Pagination, fixed by changing the namespace.

About the sort-of BC break: I added a note in the UPGRADING file here https://github.com/Becklyn/RadBundle/commit/4883fffd7a9c0c55667defa15275bc4754894367


Basic E2E test for pod groups

What type of PR is this?
/kind cleanup
What this PR does / why we need it:
Adds a basic E2E for pod groups (admit, check finished, clean up).
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
I'll follow up with more tests about retries and preemption.
Does this PR introduce a user-facing change?

NONE

Still got a timeout on webhooks ready :(
/retest

Flaky #1559
/retest


Experiences for adjusting the weights in the loss when retraining HFNET with 128D local descriptor

Currently I am retraing the HFNET using a teacher model with 128D local descriptor, but the local detector loss cannot reduce to the level as the one with 256D local descriptor. Would you share some experiences for adjusting the weights of global descriptor, local score and detector. Thanks!

Currently I am retraing the HFNET using a teacher model with 128D local descriptor, but the local detector loss cannot reduce to the level as the one with 256D local descriptor. Would you share some experiences for adjusting the weights of global descriptor, local score and detector. Thanks!

Excuse me, do you have any good suggestions?


Feature Request: Provide config to auto open terminals when a workspace is opened

Hi, thanks for making this extension. One feature that would be nice to have is if the extension can automatically open terminals for the configured actions when a workspace is opened.

Hey thanks, that makes sense!
Commands within the "open" action now run on workspace launch when Auto Terminal: Run Open Commands On Startup is enabled in settings. #3

Thanks for implementing this. It's working great.


Update action.yml to add a context to support monorepos

add context field to set working directory

Seconded. I'd use this in our monorepo if it supported it :-)

I published my action, if it helps you can use it

Hi, thanks for your PR! I'm pretty finicky about my projects (see this blog post for details), so I rarely merge a PR as-is. I can move forward on your PR in one of two ways:

I check it out locally, fiddle with it as needed, merge it locally, and simply close this PR. This will preserve at least one commit with your name on it, but the PR will show up as closed in your GitHub stats.
If you enable me to push directly to your fork, I can do my fiddling, then force push to your fork and merge the resulting PR. Again, this will preserve at least one commit with your name on it, but you also get credit for the PR merge in your GitHub stats. The only downside is that I will be force pushing directly to your fork.

Please let me know which approach you'd prefer. If I don't hear from you before I get around to working on this PR I'll go with option #1.
Thanks again for your contribution!

I actually dont care if my name is on it or not. Feel free to check it out or make the same changes and commit

I ended up merging this from the CLI. Thanks!


ERROR with TLS paket again

Hello,
@michaelrsweet
So now i install cups 2.2.5 on Ubuntu Server. With this tutorials :
https://opensource.apple.com/source/cups/cups-69/doc/fr/sam.html#CLIENT_AUTO
I have same error.
[Client 34] Unable to encrypt connection: An unexpected TLS packet was received.  when i do my lpr
Help please, really urgent.
Cordialy,

First, that document is completely out of date (14 years out of date to be exact)
Second, there really isn’t much we can do to help you with the cups on specific Linux distributions. They are responsible for providing a working CUPS installation, which includes all of the supporting libraries such as GNU TLS. It sounds like you probably need the fix from issue #5145 but the Ubuntu folks have not provided a fixed package.


Some programs crashing when disconnect the external monitor

Hyprland Version
v0.29.1
Bug or Regression?
Bug
Description
When I disconnect the external monitor some of the apps crash.
Noticed these apps:

google-chrome-stable
keepassxc
telegram-desktop

How to reproduce

Connect a monitor
Start app keepassxc from the terminal emulator
Disconnect a monitor
See keepassxc process finished with logs:

qt.qpa.wayland: Wayland does not support QWindow::requestActivate()
qt.qpa.wayland: Wayland does not support QWindow::requestActivate()
wl_display@1: error 0: invalid object 26
The Wayland connection experienced a fatal error: Invalid argument

Crash reports, logs, images, videos
I have a core-dump of the google-chrome-stable, but I doubt it is useful.

fixed in git


Added PyInterpreterTool

I have added the Pyodide interpreter to LangChainJS
Example:
import { StringOutputParser } from "../../schema/output_parser.js";
import { OpenAI } from "../../llms/openai.js";
import { PromptTemplate } from "../../prompts/index.js";
import { PythonInterpreterTool } from "../pyinterpreter.js";

const prompt = PromptTemplate.fromTemplate(
  `Can you generate python code that: {input}?`
);

const model = new OpenAI({});

const interpreter = new PythonInterpreterTool({
  indexURL: "../node_modules/pyodide",
});

const chain = prompt
  .pipe(model)
  .pipe(new StringOutputParser())
  .pipe(interpreter);

const result = await chain.invoke({
  input: "print the first 10 numbers of the fibonacci sequence",
});

console.log(result)

// prints:
// {"stdout": "0112358132134", "stderr": ""}

Of course. Where should I put them? In the comments or do you have a special place for tools anywhere? 😄
What should the warning say? The code is executed in secure WASM sandbox...
I also have a question regarding the tests. Should it only test the tool itself or the complete chain like I did there?
Thanks

Of course. Where should I put them? In the comments or do you have a special place for tools anywhere? 😄
What should the warning say? The code is executed in secure WASM sandbox...
I also have a question regarding the tests. Should it only test the tool itself or the complete chain like I did there?
Thanks

Best place would be to add a new folder called tools/experimental and put it there.

Ok done. Let me know where I should put the docs and the warning

Hi so sorry, it's been a crazy week for us - will look at this and write docs tomorrow!

Hey @mishushakov, I moved things around but the integration test seems rather flaky - do you think you could come up with a more consistent one?

Hey, I have been busy this week, but happy to tackle this on the weekend!

Is this better? Or are you looking for something else entirely?
import { test, expect } from "@jest/globals";
import { StringOutputParser } from "../../../schema/output_parser.js";
import { OpenAI } from "../../../llms/openai.js";
import { PromptTemplate } from "../../../prompts/index.js";
import { PythonInterpreterTool } from "../pyinterpreter.js";

describe("Python Interpreter testsuite", () => {
  test("hello langchain", async () => {
    const prompt = PromptTemplate.fromTemplate(
      `Can you generate python code that: {input}?`
    );

    const model = new OpenAI({});

    const interpreter = await PythonInterpreterTool.initialize({
      indexURL: "../node_modules/pyodide",
    });
    const chain = prompt
      .pipe(model)
      .pipe(new StringOutputParser())
      .pipe(interpreter);

    const result = await chain.invoke({
      input: `prints "Hello LangChain"`,
    });

    expect(JSON.parse(result).stdout).toBe("Hello LangChain");
  });
});

The test does require OpenAI API key though

Otherwise, a very basic test, that only tests the tool itself would look like:
import { test, expect } from "@jest/globals";
import { PythonInterpreterTool } from "../pyinterpreter.js";

describe("Python Interpreter testsuite", () => {
  test("hello langchain", async () => {
    const interpreter = await PythonInterpreterTool.initialize({
      indexURL: "../node_modules/pyodide",
    });

    const result = await interpreter._call(`print("Hello LangChain")`)
    expect(JSON.parse(result).stdout).toBe("Hello LangChain");
  });
});

Thank you! It's really neat


[BUG] Failing tests in main branch.

System: Mac OS
GCC version 11.2.0_3 (g++ 11)
cmake version 3.22.2
97% tests passed, 4 tests failed out of 159

Total Test time (real) =   1.81 sec

The following tests FAILED:
	 98 - stream/gz_ostream_test::gz_ostream.regular (Failed)
	 99 - stream/gz_ostream_test::gz_ostream.type_erased (Failed)
	116 - stream/transparent_ostream_test::transparent_ostream.regular_gz (Failed)
	117 - stream/transparent_ostream_test::transparent_ostream.type_erased_gz (Failed)

with
[ RUN      ] gz_ostream.regular
.../b.i.o./test/unit/stream/ostream_test_template.hpp:48: Failure
Expected equality of these values:
  buffer
    Which is: "\x1F\x8B\b\0\0\0\0\0\0\x13\v\xC9HU(,\xCDL\xCEVH*\xCA/\xCFSH\xCB\xAFP\xC8*\xCD-(V\xC8/K-R(\x1J\xE7$VU*\xA4\xE4\xA7\x3\09\xA3OA+\0\0\0"
  compressed<f>
    Which is: "\x1F\x8B\b\0\0\0\0\0\0\x3\v\xC9HU(,\xCDL\xCEVH*\xCA/\xCFSH\xCB\xAFP\xC8*\xCD-(V\xC8/K-R(\x1J\xE7$VU*\xA4\xE4\xA7\x3\09\xA3OA+\0\0\0"
[  FAILED  ] gz_ostream.regular (4 ms)
[ RUN      ] gz_ostream.type_erased
.../b.i.o./test/unit/stream/ostream_test_template.hpp:79: Failure
Expected equality of these values:
  buffer
    Which is: "\x1F\x8B\b\0\0\0\0\0\0\x13\v\xC9HU(,\xCDL\xCEVH*\xCA/\xCFSH\xCB\xAFP\xC8*\xCD-(V\xC8/K-R(\x1J\xE7$VU*\xA4\xE4\xA7\x3\09\xA3OA+\0\0\0"
  compressed<f>
    Which is: "\x1F\x8B\b\0\0\0\0\0\0\x3\v\xC9HU(,\xCDL\xCEVH*\xCA/\xCFSH\xCB\xAFP\xC8*\xCD-(V\xC8/K-R(\x1J\xE7$VU*\xA4\xE4\xA7\x3\09\xA3OA+\0\0\0"
[  FAILED  ] gz_ostream.type_erased (1 ms)

same for stream/transparent_ostream_test

Thanks for the report. Yeah, GZ on Mac produces different files 😑
I will have a look at it!


fix: upgrade to version 3.0.727

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign pmuir
You can assign the PR to them by writing /assign @pmuir in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

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

@jenkins-x-bot-test: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

gsm
877b3e4c8e171299597956485fad4e1421d5bb0a
link
/test gsm

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.

@jenkins-x-bot-test: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

gsm
877b3e4c8e171299597956485fad4e1421d5bb0a
link
/test gsm

vault
877b3e4c8e171299597956485fad4e1421d5bb0a
link
/test vault

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.

@jenkins-x-bot-test: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

gsm
877b3e4c8e171299597956485fad4e1421d5bb0a
link
/test gsm

vault
877b3e4c8e171299597956485fad4e1421d5bb0a
link
/test vault

kube
877b3e4c8e171299597956485fad4e1421d5bb0a
link
/test kube

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.

wonder if we need to combine the jx-cli version upgrade with the version stream upgrade?

@jenkins-x-bot-test: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

gsm
877b3e4c8e171299597956485fad4e1421d5bb0a
link
/test gsm

vault
877b3e4c8e171299597956485fad4e1421d5bb0a
link
/test vault

kube
a98743f7730584d0f70026e312a62fad861ccc46
link
/test kube

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.

@jenkins-x-bot-test: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

gsm
877b3e4c8e171299597956485fad4e1421d5bb0a
link
/test gsm

kube
a98743f7730584d0f70026e312a62fad861ccc46
link
/test kube

vault
a98743f7730584d0f70026e312a62fad861ccc46
link
/test vault

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.

@jenkins-x-bot-test: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

kube
a98743f7730584d0f70026e312a62fad861ccc46
link
/test kube

vault
a98743f7730584d0f70026e312a62fad861ccc46
link
/test vault

gsm
a98743f7730584d0f70026e312a62fad861ccc46
link
/test gsm

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.

/test kube

/retest

/approve

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: jstrachan
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [jstrachan]

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

@jenkins-x-bot-test: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

gsm
a98743f7730584d0f70026e312a62fad861ccc46
link
/test gsm

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.

/retest


Add copyright to GLTFState

This PR adds a copyright property to GLTFState which can be used to set the GLTF copyright string in the asset header (asset.copyright). https://github.com/KhronosGroup/glTF/blob/main/specification/2.0/schema/asset.schema.json
I also organized the asset header code, adding a new method _parse_asset_header and renaming _serialize_version to _serialize_asset_header.
For example, during export this can be done:
gltf_state.copyright = "2014 Godette"

Which results in the GLTF containing this text:
"asset": {
	"copyright": "2014 Godette",
	"version": "2.0"
},

And it can be read back at import time:
print("Copyright " + gltf_state.copyright) # Copyright 2014 Godette

@fire I am working on a system for specifying export settings like copyright, this PR is a part of that work actually.

@aaronfranke I want to mention that the xmp extension is what khronos and adobe, and all the stores recommend for this usage.

Thanks!


[CI/CD] Add support for Buildpacks - Java

Add support for Buildpacks:

Kpack
Buildpacks
Buildpack - Stack

Outcome:

Generic Java Buildpack

Related to: https://github.com/SAP/xsk/issues/182


Mismatch between Groq's latency and throughput

Description
There is a mismatch between Groq's latency and throughput. The latency unit is ms, but the actual value printed seems to be in seconds.
Reproducing
benchit models/transformers/bert.py --device groq
Models discovered during profiling:

bert.py:
        model (executed 1x)
                Model Type:     Pytorch (torch.nn.Module)
                Class:          BertModel (<class 'transformers.models.bert.modeling_bert.BertModel'>)
                Location:       /home/czhang/Documents/mlagility/models/transformers/bert.py, line 18
                Parameters:     109,482,240 (208.8 MB)
                Hash:           d59172a2
                Status:         Successfully benchmarked on GroqChip1 (groq v2.5.2)
                                Mean Latency:   0.002   milliseconds (ms)
                                Throughput:     459.3   inferences per second (IPS)

Note that, given the throughput, the latency should be ~2.1ms.

@jeremyfowers @ramkrishna2910

Related to https://github.com/groq/mlagility/issues/153


Issue with joblib

Traceback (most recent call last):
  File "C:/Users/conta/PycharmProjects/Discord/bot.py", line 11, in <module>
    from profanity_check import predict, predict_prob
  File "C:\Users\conta\anaconda3\envs\Discordbot\lib\site-packages\profanity_check\__init__.py", line 1, in <module>
    from .profanity_check import predict, predict_prob
  File "C:\Users\conta\anaconda3\envs\Discordbot\lib\site-packages\profanity_check\profanity_check.py", line 3, in <module>
    from sklearn.externals import joblib
ImportError: cannot import name 'joblib' from 'sklearn.externals' (C:\Users\conta\anaconda3\envs\Discordbot\lib\site-packages\sklearn\externals\__init__.py)

I've been trying to fix this error for a bit now, tried up & downgrading. Probably a case of dependency hell, has anyone else experienced this before?

Fix for anyone wondering: Downgrade your scikit-learn to 0.20.2 to fix this issue.

scikit-learn 0.22.2.post1 is what worked for me.

Whenever I try to use scikit-learn 0.20.2, it gets stuck at building when running the setup.py
(Python 3.8)

How can I run it on my local?
I cloned the repo on my local, but I don't know what to do next.
Can you help me?

How can I run it on my local to fix the issue?
I cloned the repo on my local, but I don't know what to do next.
Can you help me?

Assuming you're using pip3 (pip may work as well):
pip3 uninstall scikit-learn
pip3 install scikit-learn==0.20.2

Fix for anyone wondering: Downgrade your scikit-learn to 0.20.2 to fix this issue.

Not working for python3.8. py3.8 requires sklearn >= 0.22 .

joblib==0.16.0
scikit-learn==0.20.2
>>> vectorizer = joblib.load(pkg_resources.resource_filename('profanity_check', 'data/vectorizer.joblib'))

>>> model = joblib.load(pkg_resources.resource_filename('profanity_check', 'data/model.joblib'))

Traceback (most recent call last):
  File "<input>", line 1, in <module>
  File "/home/user/anaconda3/envs/LxApp_ContentFiltering/lib/python3.7/site-packages/joblib/numpy_pickle.py", line 585, in load
    obj = _unpickle(fobj, filename, mmap_mode)
  File "/home/user/anaconda3/envs/LxApp_ContentFiltering/lib/python3.7/site-packages/joblib/numpy_pickle.py", line 504, in _unpickle
    obj = unpickler.load()
  File "/home/user/anaconda3/envs/LxApp_ContentFiltering/lib/python3.7/pickle.py", line 1088, in load
    dispatch[key[0]](self)
KeyError: 0

Running Python 3.8.5 (OSX)
Downgraded scikit-learn to 0.22.2
I'm getting a different error
>>> from profanity_check import predict
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/local/lib/python3.8/site-packages/profanity_check/__init__.py", line 1, in <module>
    from .profanity_check import predict, predict_prob
  File "/usr/local/lib/python3.8/site-packages/profanity_check/profanity_check.py", line 3, in <module>
    from sklearn.externals import joblib
  File "/usr/local/lib/python3.8/site-packages/sklearn/__init__.py", line 64, in <module>
    from .base import clone
  File "/usr/local/lib/python3.8/site-packages/sklearn/base.py", line 13, in <module>
    from .utils.fixes import signature
  File "/usr/local/lib/python3.8/site-packages/sklearn/utils/__init__.py", line 14, in <module>
    from . import _joblib
  File "/usr/local/lib/python3.8/site-packages/sklearn/utils/_joblib.py", line 22, in <module>
    from ..externals import joblib
  File "/usr/local/lib/python3.8/site-packages/sklearn/externals/joblib/__init__.py", line 119, in <module>
    from .parallel import Parallel
  File "/usr/local/lib/python3.8/site-packages/sklearn/externals/joblib/parallel.py", line 28, in <module>
    from ._parallel_backends import (FallbackToBackend, MultiprocessingBackend,
  File "/usr/local/lib/python3.8/site-packages/sklearn/externals/joblib/_parallel_backends.py", line 22, in <module>
    from .executor import get_memmapping_executor
  File "/usr/local/lib/python3.8/site-packages/sklearn/externals/joblib/executor.py", line 14, in <module>
    from .externals.loky.reusable_executor import get_reusable_executor
  File "/usr/local/lib/python3.8/site-packages/sklearn/externals/joblib/externals/loky/__init__.py", line 12, in <module>
    from .backend.reduction import set_loky_pickler
  File "/usr/local/lib/python3.8/site-packages/sklearn/externals/joblib/externals/loky/backend/reduction.py", line 125, in <module>
    from sklearn.externals.joblib.externals import cloudpickle  # noqa: F401
  File "/usr/local/lib/python3.8/site-packages/sklearn/externals/joblib/externals/cloudpickle/__init__.py", line 3, in <module>
    from .cloudpickle import *
  File "/usr/local/lib/python3.8/site-packages/sklearn/externals/joblib/externals/cloudpickle/cloudpickle.py", line 167, in <module>
    _cell_set_template_code = _make_cell_set_template_code()
  File "/usr/local/lib/python3.8/site-packages/sklearn/externals/joblib/externals/cloudpickle/cloudpickle.py", line 148, in _make_cell_set_template_code
    return types.CodeType(
TypeError: an integer is required (got type bytes)

Getting same error as @Echooff3 on python37 and scikit-learn==0.20.2

I dont think the downgrade helps. Getting the same error

A few months ago, I had this all working, but now I get the same error.
Ubuntu 20.04
Python 3.8.2
webvtt-py==0.4.5
scikit-learn==0.20.2
joblib==0.16.0
  File "/tmp/youtubeget/lib/python3.8/site-packages/sklearn/externals/joblib/externals/cloudpickle/cloudpickle.py", line 148, in _make_cell_set_template_code
    return types.CodeType(
TypeError: an integer is required (got type bytes)

The problem is compatibility with Python 3.8. I tried Ubuntu 20.04's Python 2.7, but another module I need requires Python 3. Then I used a PPA to install Python 3.7, and it all works. Here's a script
sudo add-apt-repository ppa:deadsnakes/ppa
sudo apt-get update
sudo apt-get install python3.7 python3.7-venv

VENVDIR=/tmp/venv/youtubeget
rm -rf $VENVDIR
python3.7 -m venv $VENVDIR
source $VENVDIR/bin/activate
python -m pip install profanity_check scikit-learn==0.20.2 joblib

scikit-learn 0.22.2.post1 is what worked for me.

it worked for me too, many thanks

The solution to downgrade (@az0 ) worked for me, too. Using poetry, the relevant entry looks like (this installs joblib as a dependency automatically):
[tool.poetry.dependencies]
python = "3.7.9"
profanity-check = "1.0.3"
scikit-learn = "0.20.2"

However, I set this up using pyenv. This worked quite well after initial confusions. It should work and integrate well with poetry, see here or here.

Found this as an alternative:
https://pypi.org/project/alt-profanity-check/


Crashlytics no longer building on 2.1.0

As of 2.1.0, trying to build our iOS app fails when building crashlytics, saying FIRAnalyticsInterop.h cannot be found.
This was added as part of ffba1fc3d01c3facfd8511f6aa34fb9bfffd4716.
    'FirebaseCrashlytics/FIRAnalyticsInterop.h' file not found
    #import <FirebaseCrashlytics/FIRAnalyticsInterop.h>
            ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    ~/.pub-cache/hosted/pub.dartlang.org/firebase_crashlytics-2.1.0/ios/Classes/FLTFirebaseCrashlyticsPlugin.m:12:9: note: did not find header
    'FIRAnalyticsInterop.h' in framework 'FirebaseCrashlytics' (loaded from
    '~/project/build/ios/Release-iphoneos/FirebaseCrashlytics')
    1 error generated.

Indeed, FirebaseCrashlytics.framework does not have the header included:

Flutter doctor output:
$ flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel unknown, 2.0.3, on Mac OS X 10.15.7 19H1217 darwin-x64, locale en-US)
[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
[✓] Xcode - develop for iOS and macOS
[✓] Chrome - develop for the web
[✓] Android Studio (version 3.5)
[✓] VS Code (version 1.52.1)
[✓] Proxy Configuration
[✓] Connected device (1 available)

• No issues found!

Same to me.

Seems that #6623 is a duplicate of this issue.

Closing as duplicate of #6623


[HUDI-5913] Table can not read correctly when computed column is in the midst

Change Logs
The computed column(eg : process_time as proctime() ) will cause a select exception because it is not included in the data file schema, so we need to ignore the computed column
Problem
create table if not exists ods_test_hudi (
    `id` int,
    `name` string,
    `create_date` timestamp(3),
    `process_time` AS PROCTIME(),
    PRIMARY KEY(`id`) NOT ENFORCED
)
WITH (
    'connector'='hudi',
    'table.type'='MERGE_ON_READ'
)
env.sqlQuery("select * from ods_test_hudi /*+ OPTIONS('read.start-commit'='earliest')*/").execute().print();

exception:

 Caused by: java.lang.ArrayIndexOutOfBoundsException: -1
	at org.apache.hudi.table.format.cow.ParquetSplitReaderUtil.lambda$genPartColumnarRowReader$0(ParquetSplitReaderUtil.java:118)
	at java.util.stream.IntPipeline$4$1.accept(IntPipeline.java:250)
	at java.util.Spliterators$IntArraySpliterator.forEachRemaining(Spliterators.java:1032)
	at java.util.Spliterator$OfInt.forEachRemaining(Spliterator.java:693)
	at java.util.stream.AbstractPipeline.copyInto(AbstractPipeline.java:481)
	at java.util.stream.AbstractPipeline.wrapAndCopyInto(AbstractPipeline.java:471)
	at java.util.stream.ReduceOps$ReduceOp.evaluateSequential(ReduceOps.java:708)
	at java.util.stream.AbstractPipeline.evaluate(AbstractPipeline.java:234)
	at java.util.stream.ReferencePipeline.collect(ReferencePipeline.java:499)
	at org.apache.hudi.table.format.cow.ParquetSplitReaderUtil.genPartColumnarRowReader(ParquetSplitReaderUtil.java:120)

Risk level (write none, low medium or high below)
none
Contributor's checklist

[ ] Read through contributor's guide
[ ] Change Logs and Impact were stated clearly
[ ] Adequate tests were added if applicable
[ ] CI passed

Thanks for the feedback, you are right, the test case passed because the computed column is the last column within the schema, I have created a patch and attach it here:
5913.patch.zip
Can you just checkout out a fresh new branch from the latest master, then apply the patch using cmd:
git apply xxx.patch

then force push your branch with cmd:
git push origin xxx_branch --force

Thanks for the feedback, you are right, the test case passed because the computed column is the last column within the schema, I have created a patch and attach it here: 5913.patch.zip
Can you just checkout out a fresh new branch from the latest master, then apply the patch using cmd:
git apply xxx.patch

then force push your branch with cmd:
git push origin xxx_branch --force

I had apply the patch #8154 ，should i close this pr.


Fix typo

This looks like a typo.

@ploeh : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off


ripgrep: 0.7.1 -> 0.8.0, install shell completions

https://github.com/BurntSushi/ripgrep/blob/master/CHANGELOG.md#080-2017-02-11
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build ripgrep

Failure on x86_64-linux (full log)
Partial log (click to expand)
cp: -r not specified; omitting directory 'target/release/.fingerprint'
cp: -r not specified; omitting directory 'target/release/incremental'
cp: -r not specified; omitting directory 'target/release/deps'
cp: -r not specified; omitting directory 'target/release/native'
cp: -r not specified; omitting directory 'target/release/examples'
post-installation fixup
cp: missing destination file operand after '/nix/store/i4igandv5pizi8n8h62pcv62bp6g45za-ripgrep-0.8.0/man/man1/'
Try 'cp --help' for more information.
builder for ‘/nix/store/nmww0mc3yqqc9n4ivywhp4rb7d9s2jgp-ripgrep-0.8.0.drv’ failed with exit code 1
error: build of ‘/nix/store/nmww0mc3yqqc9n4ivywhp4rb7d9s2jgp-ripgrep-0.8.0.drv’ failed

Success on x86_64-darwin (full log)
Partial log (click to expand)
cp: -r not specified; omitting directory 'target/release/examples'
cp: -r not specified; omitting directory 'target/release/deps'
cp: -r not specified; omitting directory 'target/release/build'
post-installation fixup
moving /nix/store/k0afq8w7wwn7kgb5zqzdg2r5lgva1587-ripgrep-0.8.0/man to /nix/store/k0afq8w7wwn7kgb5zqzdg2r5lgva1587-ripgrep-0.8.0/share/man
gzipping man pages under /nix/store/k0afq8w7wwn7kgb5zqzdg2r5lgva1587-ripgrep-0.8.0/share/man/
strip is /nix/store/5a88zk3jgimdmzg8rfhvm93kxib3njf9-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/k0afq8w7wwn7kgb5zqzdg2r5lgva1587-ripgrep-0.8.0/bin
patching script interpreter paths in /nix/store/k0afq8w7wwn7kgb5zqzdg2r5lgva1587-ripgrep-0.8.0
/nix/store/k0afq8w7wwn7kgb5zqzdg2r5lgva1587-ripgrep-0.8.0

Failure on aarch64-linux (full log)
Partial log (click to expand)
cannot build derivation '/nix/store/0jribyf3qxfdl60lgpqgnfali7ai95ib-rust_serde_derive-1.0.18.drv': 6 dependencies couldn't be built
cannot build derivation '/nix/store/kyglwvsprq1in2xnjg6mx9rnccidmy6f-rust_crates-io-0.11.0.drv': 30 dependencies couldn't be built
cannot build derivation '/nix/store/49yfi2522frhdq6i8hms6hjndqhv6sik-rust_docopt-0.8.1.drv': 19 dependencies couldn't be built
cannot build derivation '/nix/store/x3nxrrggzq7znif7b32kb3a7f1rz4a49-rust_git2-curl-0.7.0.drv': 20 dependencies couldn't be built
cannot build derivation '/nix/store/nzrc7j57fzrya5qyl7vqnr1rqhjg40g4-rust_ignore-0.2.2.drv': 18 dependencies couldn't be built
cannot build derivation '/nix/store/m9aq2m4jb4whd51c8jcrhvgp6igwdjqz-rust_cargo-0.22.0.drv': 78 dependencies couldn't be built
cannot build derivation '/nix/store/ir9aw7rf74r0qyxj79wbn3609prgr23c-rust_cargo-vendor-0.1.13.drv': 79 dependencies couldn't be built
cannot build derivation '/nix/store/g8wc77k90c5z5l5gmprr142mh8rnyrf7-ripgrep-0.8.0-vendor.drv': 2 dependencies couldn't be built
cannot build derivation '/nix/store/zfiy5gi560gcfzxb8pw0fry92nnf6csw-ripgrep-0.8.0.drv': 3 dependencies couldn't be built
error: build of '/nix/store/zfiy5gi560gcfzxb8pw0fry92nnf6csw-ripgrep-0.8.0.drv' failed

@GrahamcOfBorg build ripgrep

Success on x86_64-darwin (full log)
Partial log (click to expand)
cp: -r not specified; omitting directory 'target/release/examples'
cp: -r not specified; omitting directory 'target/release/deps'
cp: -r not specified; omitting directory 'target/release/build'
post-installation fixup
moving /nix/store/k2cbdg2p6ba9gwr32kr1a1xgigvna562-ripgrep-0.8.0/man to /nix/store/k2cbdg2p6ba9gwr32kr1a1xgigvna562-ripgrep-0.8.0/share/man
gzipping man pages under /nix/store/k2cbdg2p6ba9gwr32kr1a1xgigvna562-ripgrep-0.8.0/share/man/
strip is /nix/store/5a88zk3jgimdmzg8rfhvm93kxib3njf9-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/k2cbdg2p6ba9gwr32kr1a1xgigvna562-ripgrep-0.8.0/bin
patching script interpreter paths in /nix/store/k2cbdg2p6ba9gwr32kr1a1xgigvna562-ripgrep-0.8.0
/nix/store/k2cbdg2p6ba9gwr32kr1a1xgigvna562-ripgrep-0.8.0

Failure on x86_64-linux (full log)
Partial log (click to expand)
cp: -r not specified; omitting directory 'target/release/.fingerprint'
cp: -r not specified; omitting directory 'target/release/incremental'
cp: -r not specified; omitting directory 'target/release/deps'
cp: -r not specified; omitting directory 'target/release/native'
cp: -r not specified; omitting directory 'target/release/examples'
post-installation fixup
cp: missing destination file operand after '/nix/store/dbpfpjx44p4g9dbw6wqzcdpsa4ac1w4w-ripgrep-0.8.0/man/man1/'
Try 'cp --help' for more information.
builder for ‘/nix/store/6sp2846z16mzyr36asf462k8wkbyjjdk-ripgrep-0.8.0.drv’ failed with exit code 1
error: build of ‘/nix/store/6sp2846z16mzyr36asf462k8wkbyjjdk-ripgrep-0.8.0.drv’ failed

Failure on aarch64-linux (full log)
Partial log (click to expand)
cannot build derivation '/nix/store/abhcmnb13wdgm4x94j8xv70kwkqb5jgx-rust_serde_derive-1.0.18.drv': 6 dependencies couldn't be built
cannot build derivation '/nix/store/q7x2vygi3zx94iznmhhc550v9nwrfyq5-rust_crates-io-0.11.0.drv': 30 dependencies couldn't be built
cannot build derivation '/nix/store/5q0f27nqvgpwlx15pinpvi3b4kvfz2mx-rust_docopt-0.8.1.drv': 19 dependencies couldn't be built
cannot build derivation '/nix/store/gkqnak4fd86jlrjdl84dbvsa4qkran1y-rust_git2-curl-0.7.0.drv': 20 dependencies couldn't be built
cannot build derivation '/nix/store/sid4kn26d0nvhsz1qsx735icn3qq5mal-rust_ignore-0.2.2.drv': 18 dependencies couldn't be built
cannot build derivation '/nix/store/myjdc9w75zws6ngrsf9g0nb4brmka3a6-rust_cargo-0.22.0.drv': 78 dependencies couldn't be built
cannot build derivation '/nix/store/lxkalvjcwxbpnxjcdw08hiwpv1rdkmgv-rust_cargo-vendor-0.1.13.drv': 79 dependencies couldn't be built
cannot build derivation '/nix/store/15bj157l3b9qrrfx552vxvwxfb3asyzj-ripgrep-0.8.0-vendor.drv': 2 dependencies couldn't be built
cannot build derivation '/nix/store/x1mj2i30gx19k2w02rxmrj8bgr9mhz0r-ripgrep-0.8.0.drv': 3 dependencies couldn't be built
error: build of '/nix/store/x1mj2i30gx19k2w02rxmrj8bgr9mhz0r-ripgrep-0.8.0.drv' failed

@GrahamcOfBorg build ripgrep

Success on x86_64-linux (full log)
Partial log (click to expand)
post-installation fixup
moving /nix/store/wg86xpzayw3gfb78isqdzicp0yfsqk0l-ripgrep-0.8.0/man to /nix/store/wg86xpzayw3gfb78isqdzicp0yfsqk0l-ripgrep-0.8.0/share/man
shrinking RPATHs of ELF executables and libraries in /nix/store/wg86xpzayw3gfb78isqdzicp0yfsqk0l-ripgrep-0.8.0
shrinking /nix/store/wg86xpzayw3gfb78isqdzicp0yfsqk0l-ripgrep-0.8.0/bin/rg
gzipping man pages under /nix/store/wg86xpzayw3gfb78isqdzicp0yfsqk0l-ripgrep-0.8.0/share/man/
strip is /nix/store/5qj61lcvzlap87rf6blvf8p577d482bv-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/wg86xpzayw3gfb78isqdzicp0yfsqk0l-ripgrep-0.8.0/bin
patching script interpreter paths in /nix/store/wg86xpzayw3gfb78isqdzicp0yfsqk0l-ripgrep-0.8.0
checking for references to /tmp/nix-build-ripgrep-0.8.0.drv-0 in /nix/store/wg86xpzayw3gfb78isqdzicp0yfsqk0l-ripgrep-0.8.0...
/nix/store/wg86xpzayw3gfb78isqdzicp0yfsqk0l-ripgrep-0.8.0

Failure on aarch64-linux (full log)
Partial log (click to expand)
cannot build derivation '/nix/store/abhcmnb13wdgm4x94j8xv70kwkqb5jgx-rust_serde_derive-1.0.18.drv': 6 dependencies couldn't be built
cannot build derivation '/nix/store/q7x2vygi3zx94iznmhhc550v9nwrfyq5-rust_crates-io-0.11.0.drv': 30 dependencies couldn't be built
cannot build derivation '/nix/store/5q0f27nqvgpwlx15pinpvi3b4kvfz2mx-rust_docopt-0.8.1.drv': 19 dependencies couldn't be built
cannot build derivation '/nix/store/gkqnak4fd86jlrjdl84dbvsa4qkran1y-rust_git2-curl-0.7.0.drv': 20 dependencies couldn't be built
cannot build derivation '/nix/store/sid4kn26d0nvhsz1qsx735icn3qq5mal-rust_ignore-0.2.2.drv': 18 dependencies couldn't be built
cannot build derivation '/nix/store/myjdc9w75zws6ngrsf9g0nb4brmka3a6-rust_cargo-0.22.0.drv': 78 dependencies couldn't be built
cannot build derivation '/nix/store/lxkalvjcwxbpnxjcdw08hiwpv1rdkmgv-rust_cargo-vendor-0.1.13.drv': 79 dependencies couldn't be built
cannot build derivation '/nix/store/15bj157l3b9qrrfx552vxvwxfb3asyzj-ripgrep-0.8.0-vendor.drv': 2 dependencies couldn't be built
cannot build derivation '/nix/store/6cglw3aryziq8w5xw3bf3g7asvmwmng8-ripgrep-0.8.0.drv': 3 dependencies couldn't be built
error: build of '/nix/store/6cglw3aryziq8w5xw3bf3g7asvmwmng8-ripgrep-0.8.0.drv' failed

@GrahamcOfBorg build ripgrep

Success on x86_64-linux (full log)
Partial log (click to expand)
/nix/store/wg86xpzayw3gfb78isqdzicp0yfsqk0l-ripgrep-0.8.0

Success on x86_64-darwin (full log)
Partial log (click to expand)
cp: -r not specified; omitting directory 'target/release/examples'
cp: -r not specified; omitting directory 'target/release/incremental'
cp: -r not specified; omitting directory 'target/release/native'
post-installation fixup
moving /nix/store/zy2q74ri65rqlbnb3y4n78kj97xlcx9v-ripgrep-0.8.0/man to /nix/store/zy2q74ri65rqlbnb3y4n78kj97xlcx9v-ripgrep-0.8.0/share/man
gzipping man pages under /nix/store/zy2q74ri65rqlbnb3y4n78kj97xlcx9v-ripgrep-0.8.0/share/man/
strip is /nix/store/5a88zk3jgimdmzg8rfhvm93kxib3njf9-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/zy2q74ri65rqlbnb3y4n78kj97xlcx9v-ripgrep-0.8.0/bin
patching script interpreter paths in /nix/store/zy2q74ri65rqlbnb3y4n78kj97xlcx9v-ripgrep-0.8.0
/nix/store/zy2q74ri65rqlbnb3y4n78kj97xlcx9v-ripgrep-0.8.0

Failure on aarch64-linux (full log)
Partial log (click to expand)
cannot build derivation '/nix/store/phxvsacm9fscdc34kqhffh4m2q7vbhgh-rust_serde_derive-1.0.18.drv': 6 dependencies couldn't be built
cannot build derivation '/nix/store/83qr95ip9y1rjw538vfvn4893zca3sy3-rust_crates-io-0.11.0.drv': 30 dependencies couldn't be built
cannot build derivation '/nix/store/lba9xlwr92h6bq8gkdzli59chqli3q3i-rust_docopt-0.8.1.drv': 19 dependencies couldn't be built
cannot build derivation '/nix/store/203x9kqfg4hzk4bym4zr1rg5zgra5p3v-rust_git2-curl-0.7.0.drv': 20 dependencies couldn't be built
cannot build derivation '/nix/store/vafvjkb28ms4cc5mz964acr26bkd5hnl-rust_ignore-0.2.2.drv': 18 dependencies couldn't be built
cannot build derivation '/nix/store/c3rvsymm70234ymfdl9vn6a6jrvkv7i2-rust_cargo-0.22.0.drv': 78 dependencies couldn't be built
cannot build derivation '/nix/store/c3fyrgq7w8midwqappsxdxwz2947z4w1-rust_cargo-vendor-0.1.13.drv': 79 dependencies couldn't be built
cannot build derivation '/nix/store/0ihsqw4pfmxgsa6rrgirjiifrj3xp34z-ripgrep-0.8.0-vendor.drv': 2 dependencies couldn't be built
cannot build derivation '/nix/store/d6vkjg91v3x81kzw7n8yv0cpfki906pz-ripgrep-0.8.0.drv': 3 dependencies couldn't be built
error: build of '/nix/store/d6vkjg91v3x81kzw7n8yv0cpfki906pz-ripgrep-0.8.0.drv' failed


Resolves #242: add space between words in button text

remove shouldNotGetCount props from LessonCard components
~render/return single space span for the case where query data property is undefined~
~replace stories with dummy component~
add consistent return values to ReviewCount component (always returns a span containing a number, except for the case of a spinner for a pending/loading query)
simplify word spacing in ReviewButton (move spacing from ReviewCount to ReviewButton)
add mock provider for LessonCard stories
leave comment on strategic decision for UI components (see #275)

How does this PR relate to the name add space between words in button text?

I was asking myself the same thing yesterday 🤣


About new feature : Encrypt sensitive DATA

I call "sensitive DATA" , the data like visited url and stuff like that.

Agree. Thanks for this feature.

Okay, I'm going to code that


"SWS/BR: Copy take media source file path of selected items to clipboard" adds extra /

Regarding: SWS/BR: Copy take media source file path of selected items to clipboard
C:\Users\hans\Desktop\audio files\Footsteps, Human, Footsteps, Ice, Ice Shoes, Crampon 11-glued.wav
As you can see on the file path, i get double \ after desktop. This happens wherever the file is located. Above was just an example.

C:\Users\hans\Desktop\audio files\Footsteps, Human, Footsteps, Ice, Ice Shoes, Crampon 13-glued-01.wav

I don't know why, but it seems like when i publish the post, the double \ \ dissapears after post.
The pathway i get from the script is :
C:\Users\hans\Desktop\audio files\ \ Footsteps, Human, Footsteps, Ice, Ice Shoes, Crampon 11-glued.wav
without the space between the slashes.


RESTClient stopped working after latest firefox update (46.0.1)

RESTClient stopped working after latest firefox update. Can not send any type of request

Same problem here

me too

Same as #220.


Rewrite as symfony-flex

GitHub Issue: https://github.com/Islandora-CLAW/CLAW/issues/1224
Depends on the branch - https://github.com/Islandora-CLAW/Crayfish-Commons/pull/26
What does this Pull Request do?
This PR recreates the Houdini microservice using Symfony/flex.
I will work on a PR for claw-playbook and ansible-role-crayfish
What's new?
Changes to the symfony/flex framework.
To configure the firewall you do that in config/packages/security.yaml
To configure logging you do that in config/packages/monolog.yaml
To configure crayfish commons you do that in config/packages/crayfish_commons.yaml

Does this change require documentation to be updated? yes
Does this change add any new dependencies? yes, changes
Does this change require any other modifications to be made to the repository (ie. Regeneration activity, etc.)? no
Could this change impact execution of existing code? no

How should this be tested?
Currently I was testing by:

Standing up a normal claw-playbook.
Add a binary image to the Islandora 8 and get the URL for Fedora.
Check out this branch.
cd ./Houdini
composer install
php ./bin/console server:start
Get the host and port, then ensure your crayfish_commons.yaml has the correct settings, including the location to the example syn-settings.xml
curl -i -H"Accept: image/png" -H"Apix-Ldp-Resource: <fedora url>" -H"X-Islandora-Args: -resize 100x>" http://localhost:<symfony port>/convert -> 401 Unauthorized
curl -i -H"Accept: image/png" -H"Apix-Ldp-Resource: <fedora url>" -H"X-Islandora-Args: -resize 100x>" -H"Authorization: Bearer bob" http://localhost:<symfony port>/convert -> 403 Forbidden
curl -o test.png -H"Accept: image/png" -H"Apix-Ldp-Resource: <fedora url>" -H"X-Islandora-Args: -resize 100x>" -H"Authorization: Bearer islandora" http://localhost:<symfony port>/convert -> view the resized image in test.png

Interested parties
@Islandora-CLAW/committers

Curse you Gemini!! CURSE YOU!!!

Ok I'm closing this and the associated Crayfish-Commons PR. I did some work on the ansible role and some seemingly correct decisions here, fail when run through ansible.


MUMMNG-2396 : March madness feature

Enabled features in /web by default :)
add in announcement for March madness
add in image for blog post

Coverage remained the same at 28.178% when pulling f239d69690e4e4e426fe4b69c6583f733ba1988c on march-madness-feature into 11b43c27e022d151eb5c703cc4a9e01e7463792e on master.

:+1:

:+1:

:+1:


Lukebailey/infra 386 create a a11y checkbox

Summary
Create a a11y checkbox  per the V3 Library on Figma, with states for:

disabled
unchecked
checked
focused
indeterminate

References
Linear: https://linear.app/contra/issue/INFRA-386/create-a-a11y-checkbox
Loom: https://www.loom.com/share/7c8ef7dc185646bf9d6cdc60c25ed109
Figma: https://www.figma.com/file/owwo3mjL0dCKJijKGaf1XB/V3-Library?node-id=705%3A9117

The new dependencies will need to be added to: https://github.com/contra/ui-kit/blob/main/bin/build-cjs.js#L11

Ah thanks @erictaylor! I didn't realise this was a thing (I'm new to cjs), will add these in today

@lukerohanbailey feel free to reach out to me for a Tuple if you'd like and we can walk through some changes together (if desired). I realize that getting all the context or answers to questions is difficult going back and forth in just comments.

@erictaylor I've made some changes that remove GSAP and replace it with CSS transitions that use the stitches variants feature https://github.com/contra/ui-kit/pull/53/commits/e8cf1a9957284e39f254096da6b96c0e7c8296d5


Actually show clippy

Is there a way to make the plugin actually open a window with clippy giving advice?

Not yet, but it's on our to-do list for quite some time now. Especially now, that when the community expressed interest, we'll try to implement that faster :)


Add ACES endpoints

(related: https://github.com/Infleqtion/server-superstaq/pull/2612)

lgtm!

cool! will merge once https://github.com/Infleqtion/server-superstaq/pull/2612 goes in


Feature: add reinforcement learning as an advanced option for training

UI

Results
0.56 -> 0.65

https://github.com/aigc-apps/sd-webui-EasyPhoto/pull/109/commits/fb49bae1f16b0287bebab381201af3cda21db9fb does the following optimizations:

Update default hyper-parameters (e.g., learning rate and training batch size) for smaller computing resources and improve the training speed.
Fix the problem of not being able to stop training when the time runs out. @bubbliiiing help me test in Windows.
Code and docstring standards @wuziheng

https://github.com/aigc-apps/sd-webui-EasyPhoto/pull/109/commits/dabb5c128d1e2648217cc90553b251ffb4b10dfb:

Use thread.Timer instead of signal to limit the execution time of a function call. It should work in Windows @bubbliiiing.
Save the best checkpoint (be copied to webui) and maintain a heap (except for epoch 0) with length num_checkpoint_limit - 1 by reward_mean.

已经完成测试，faceid 分数来到0.7+

@hkunzhe  @bubbliiiing  Should Notice that
During my testing, I found that the optimized model indeed showed a significant improvement in the similarity scores of the generated characters in end-to-end rendering, for example, increasing from an average of 0.55 to close to 0.7+. However, the visual similarity did not seem to improve as much, and there appeared to be some inconsistency in the identity across multiple renderings, giving the impression of a certain degree of overfitting. All of these aspects would require longer-term and larger-scale testing within the community to validate.

From my experience, the gap between faceID scores and human perception may have the following reasons:

Insufficient network capabilities for extracting facial features: my initial test was based on faceid_SCRFD, and the scores would be more aligned with human perception. And during the test with faceid_retina, it was found that many face images directly generated by LoRA could not be recognized by the model, which rarely happened in faceid_SCRFD.
The training objectives of reinforcement learning and the actual inference pipeline are not completely aligned.


Update bitlocker-countermeasures.md

Added link to Intel Thunderbolt Security documentation for systems that do not support Kernel DMA Protection

@AaDake Thanks for adding these changes!