Support external database

[Validation Helper] It is possible to use the component with entities from external database?

Hi,
For what i've seen, entities from external database doesn't have an espace id, so when i want to create the validations and access the detail screen it doesn't appear anything and says 'Oops, Espace or Entity not found. '.
I've tried to investigate what it is done and change something to let me create the validations without espace id or name but i couldn't make it.

Acceptance Criteria

Entity search screen shows external DB entities
Espace search criteria have been removed.
Espace and Extension can be searched by Module search criteria
External Entity (Entity in external database) can be searched
There is no column for Espace in the search results
There is a column of Module in the search result.
Espace and Module names are displayed in the Module column of search results.
Can move to the detail screen of the external Entity from the search result.

Detail screen

Rules of an external Entity can be displayed.
Rules of an external Entity can be saved
Rules of an external Entity can be deleted.

Action - IsValidEntity

Validation works correctly (Element)
Validation works correctly (Existence)
Validation works correctly (Format)
Validation works correctly (Length)
Validation works correctly (Numecarity)
Validation works correctly (Uniqueness) @SQL Server
Validation works correctly (Uniqueness) @Oracle


Fix for start.sh

This pull request provides fix for #39 and #31

First of all, thanks for the PR. Ill test it ASAP, in the meantime please check my questions.

@meMo-Minsk Any news on this?

@meMo-Minsk any updates?

Closing in favor of #50


[FORSETI] Audit Logging scanner functionality

Related to Forseti Security Issue #3624
Scanner
Audit Logging Scanner
Description: You can configure Cloud Audit Logging to save Admin Activity and Data Access for Google Cloud Platform (GCP) services. The audit log configurations for a project, folder, or organization specify which logs should be saved, along with members who are exempted from having their accesses logged. The audit logging scanner detects if any projects are missing a required audit log, or have extra exempted members.
Python Violations Summary
Generate a violation if the log_types IamAuditConfig resource does not match the log_types defined in audit_logging_rules.yaml. Can choose to select specific organization/folders/projects to scan.
1:1 Rego | Python Parity Status
The Rego rule has the same features and will look for the log_types and exemption users. HOWEVER, it does not look like you can select what projects/folders/organizations-to filter for and no ability to choose only Folder/Project asset type.
Rego work to do
Create or edit existing Rego rule to do the following:

Allow for specifying between Organization, Folder, and Project asset types --> current rule uses all three asset types together:

asset_types := {
           "cloudresourcemanager.googleapis.com/Organization",
           "cloudresourcemanager.googleapis.com/Folder",
           "cloudresourcemanager.googleapis.com/Project",
           	}

@morgante : Utilize the "target" option in Rego in order to filter out which Organization, Folder, Project, etc. to parse. Functionality is complete here.


Import d613ac4668: Parse .glb urls properly (by Maki)

This is an automated proposal to look at a commit made by Maki and import it into Overte

Commit:
d613ac4668da3ea06407c41f21316bc62beee38c

Author:
Maki

Date:
Thu, 02 Apr 2020 22:02

Parse .glb urls properly

Stats:

Filename
Stats
Lines
Added
Removed
Lines in blame

libraries/fbx/src/GLTFSerializer.cpp

6
3
3
3

1 files
-
6
3
3
3

To work on this, please assign the issue to yourself, then look at the commit and decide whether this would be a good addition to Overte.

If the commit is useful, tag it with "Tivoli: Keep", and keep it open until it's merged.
If the commit is not useful, tag it with "Tivoli: Discard", and close it.
If the commit is not useful right now, but might be later, tag it with "Tivoli: Maybe later", and close it.
If it's hard to decide, tag it with "Tivoli: Discuss", and keep it open.

Useful commits should be submitted as a PR against Overte. Tag this issue in PR, so that it's automatically closed once the PR is merged.
You can cherry-pick this issue with this command:
git cherry-pick d613ac4668da3ea06407c41f21316bc62beee38c

This might be interesting, especially since full url text might be longer than the path it contains, so it doesn't have to end with "glb" for a glb file.

During the meeting 74 suggests that later commits might be dealing with problems from this.


Try-Component, add Context to catch case

Pull Request Test Coverage Report for Build 1075

14 of 14 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 1072:
0.0%

Covered Lines:
615

Relevant Lines:
615

💛  - Coveralls


Upgrade to ReactNative 0.31

Please verify the following:

[x] Everything works on iOS/Android
[x] ignite-base ava tests pass
[x] fireDrill.sh passed

Describe your PR
Upgrades from ReactNative 0.30 to 0.31
Fixes linter errors in ignite-cli/index

@ryanlntn  - I get 0 warnings on props after a newclear with the changes I made.


Publish new mantle binary release for cork

Current situation
The last binary release lacks some feature which we rely on for future releases.
One example is the cork verify command which now can check if the correct tags are checked out.
Impact
Users have to compile from source or will run into an error message.
Ideal future situation
A new mantle binary release is done on GitHub and mentioned in the SDK development docs.
Implementation options
Additional information

Done


Avoid saving user-specific data in instance scripts

Task instanceUp is saving e.g JAVA_EXECUTABLE absolute path in control/\*.\[bat|sh] scripts and  it makes instances not usable directly when extracted externally (outside of GAP usage).
Consider passing user-specific data to instance scripts using arguments or env vars instead of writing them down to files that could be later a part of instance backups.
Thx @szymon-owczarzak for reporting it!

Or a system variable for the username could be used in JAVA_EXECUTABLE:
C:\Users\%Username%\.gradle\jdks\jdk8u275-b01\bin\java.exe

Or a system variable for the username could be used in JAVA_EXECUTABLE:
C:\Users\%Username%\.gradle\jdks\jdk8u275-b01\bin\java.exe

How about backup made on Unix but reused on windows? 😁 Nope

How about backup made on Unix but reused on windows? 😁 Nope

for *.sh we could use $HOME/.gradle/jdks... and for *.bat %userprofile%/.gradle/jdks..`

for *.sh we could use $HOME/.gradle/jdks... and for *.bat %userprofile%/.gradle/jdks..`

Distro of Java determined by Gradle could be different for each OS.


How does the block_size work?

Hi, I would like to know how does the input data look like during training?
Let's say if I set my block_size to be 256, I have two different inputs, one with 128 length and another with 150 length. Would the model be trained with

concat input #1 and input #2 and their final length truncated to 256, or
input #1 + padding to 256 length and another input #2 + padding to 256 length?

But I am wondering how would this affect the result? My observation is that the second approach is most consistent with its output. Have you tried the second option before?

Has anyone tried the padding option? The doc is not clear. I tried disable_group_text option but got error.

Please take a look at https://github.com/OptimalScale/LMFlow/pull/577

This issue has been marked as stale because it has not had recent activity. If you think this still needs to be addressed please feel free to reopen this issue. Thanks


Appears the bot is unable to connect for data

Upon request it seems to not have any data, maybe someone is blocking the request?

Hi, thanks for letting me know! I have fixed the issue, rescaling the dynos on Heroku had caused the processes to stop. A simple restart of dynos fixed the issue. It should work now.


[HOLD for payment 2024-07-24] [$250] [Violations] Distance - Incorrect error message when distance amount is changed to smaller amount

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.69-0
Reproducible in staging?: Y
Reproducible in production?: N
Logs: https://stackoverflow.com/c/expensify/questions/4856
Issue reported by: Applause-Internal team
Action Performed:

Go to staging.new.expensify.com
Go to workspace chat.
Submit a distance expense.
Go to transaction thread.
Modify the amount to smaller than the calculated amount.

Expected Result:
The error message should correctly reflect the difference in the new amount.
It should follow Old Dot style where it shows "Expense amount differs from the calculated distance amount",
Actual Result:
When the distance amount is manually changed to the lower amount, the error message is "Amount greater than scanned receipt", which is generic and untrue.
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

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

Screenshots/Videos
https://github.com/Expensify/App/assets/115492554/df6b8495-03f3-4a25-a8d5-161176c54eb2

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01888703ab728af885
Upwork Job ID:<PHONE_NUMBER>242878976
Last Price Increase: 2024-05-02

Issue OwnerCurrent Issue Owner: @
Issue OwnerCurrent Issue Owner: @strepanier03

Production

@strepanier03 I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors.

We think this issue might be related to the #collect project.

Proposal
Please re-state the problem that we are trying to solve in this issue.
Incorrect error message when distance amount is changed to smaller amount
What is the root cause of that problem?
The modifiedAmount violation message is worded differently
What changes do you think we should make in order to solve the problem?
We will change the modifiedAmount violation to the OldDot style message: Expense amount differs from the calculated distance amount

Okay testing now, one sec.

I'm unable to repro. Changing the amount to smaller or larger doesn't produce any error message as shown in the video. Will try a few more times to repro.

I've lost the ability to edit the amount of the distance expense, not sure what changed but I'm stuck at the moment. I need to shift focus to some other testing so I'll circle back on this.

Violations are still in beta and this is not breaking any functionality, I'm going to remove the blocker label.
/cc @Krishna2323 @cead22, this sounds like it could potentially come from https://github.com/Expensify/App/pull/37813 but I haven't double-checked.

I've lost the ability to edit the amount of the distance expense

I'm seeing the same. I think @Krishna2323 is right about the violation we're returning, but I haven't figured out what part of the code is wrong 🤔

Ok, the code is right and calculating a modifiedAmount violation for the distance expense, but

We don't use the message from the backend
We're putting the message for this violation under the receipt every time, which isn't ideal since we have several cases

Expense amount differs from the calculated distance amount
Amount greater than card transaction
Amount $displayPercentVariance% greater than scanned receipt
Amount greater than scanned receipt

I can now edit the amount again but the error doesn't populate at all when making a change to the amount, whether its larger or smaller.

Expense

Smaller amount

Larger amount

This has been labelled "Needs Reproduction". Follow the steps here: https://stackoverflowteams.com/c/expensify/questions/16989

Discussed with Jason on new dot. Let's do the following

Let's show all these cases under the amount field, since a) we think it's more impactful/intuitive to highlight that the amount is greater than the receipt, than the other way around; and b) the field you need to take action on in order to fix the violation is the amount field, not the receipt field
I'll have the backend send data for the transaction violation as follows

{data: {type: 'distance'}}
{data: {type: 'card'}}
{data: {type: 'smartscan'}}
{data: {type: 'smartscan', displayPercentVariance: $displayPercentVariance}}

Let's update the front end such that

We display these notices under the amount field

Update violationsByField so filteredViolations includes the modifiedAmount violation which has type notice
Update MoneyRequestView.tsx so noticeTypeViolations excludes modifiedAmount

If data.type == distance the copies of the violation should be (I'll double check on the Spanish translations)

EN: Amount differs from calculated distance
ES: Importe difiere del calculado basado en distancia

If data.type == card the copies of the violation should be

EN: Amount greater than card transaction
ES: Importe mayor al de la transacción de la tarjeta

If data.type == smartscan && data.displayPercentVariance the copies of the violation should be

EN: Amount $displayPercentVariance% greater than scanned receipt
ES: Importe $displayPercentVariance% mayor al del recibo escaneado

If data.type == smartscan the copies of the violation should be

EN: Amount greater than scanned receipt
ES: Importe mayor al del recibo escaneado

Yes, agreed. Sounds great!

About 4, and 5, instead of adding getModifiedExpenseViolationTranslation, can we do this instead?

Update ViolationUtils.getViolationTranslation for the case 'modifiedAmount': to pass the violation data to translate
Update src/languages/en.ts and src/languages/es.ts to use the logic in https://github.com/Expensify/App/issues/41401#issuecomment-2091898713 to display the right thing

Or is there anything about that solution that wouldn't work?

Cool, yeah I should've been clearer on that one, but the idea was to make this work with the helper methods we already have

@Krishna2323 While we're on this, I noticed the code grabs any notice and puts it under the receipt
const noticeTypeViolations = transactionViolations?.filter((violation) => violation.type === 'notice').map((v) => ViolationsUtils.getViolationTranslation(v, translate)) ?? [];

If that's the case, let's update this so we only show receiptRequired & receiptNotSmartScanned under the receipt. receiptRequired is a violation, but receiptNotSmartScanned is a notice, so can we:

Remove the code I pasted above
Call getErrorForField('receipt') like we do for all the other fields, and pass the result in the notes prop to ReceiptAuditHeader and ReceiptAuditMessages
Update violationsByField to include receiptNotSmartScanned like we're doing for modifiedAmount

I think that should make it work, and it'll make the violations on the receipt field more consistent with how we compute them for the other fields. What do you think?

Super great questions, thanks @Krishna2323!

we've determined that only receiptRequired and receiptNotSmartScanned will be displayed under the receipt.

That's correct

But what should we do with other notices? In the example below, you can see two notices for the tax and date fields. Where should they be shown?

Missing category should be shown on the category field. We actually don't intend to show the taxRateChanged notice in NewDot, so that's a good find. We updated how the default tax rate is shown and now capture the change via a system message, so that note is not useful on the new platform.

@JmillsExpensify, ok, so we are saying that only show receiptRequired and receiptNotSmartScanned under the receipt and ignore all other notices accept few like modifiedAmount (which will be shown under amount field ), is that correct?

That's correct.

asking because https://github.com/Expensify/App/issues/41401#issuecomment-2093288802 you have clarified that we don't want to show taxRateChanged but what about date notice.

Yes we shouldn't show the taxRateChanged note on NewDot, as it is being deprecated given that we have implemented that feature differently on NewDot.
Then as for date, it will be treated in line with our solution for amount. So for clarity, the note violation will appear in-line on the date field. Quick mock to ensure the point is clear.

On-going convo and work in the linked PR.

Issue not reproducible during KI retests. (Second week)

Issue not reproducible during KI retests. (Third week)

Issue not reproducible during KI retests. (Fourth week)

@Krishna2323 - I sent you an offer via Upwork, I'll keep an eye out for the acceptance notification to finish up.
@sobitneupane - I'll check in later for the checklist and once it's done I'll post the payment summary. Thanks!

Saw you accepted so I paid and closed the contract @Krishna2323, thanks!

@sobitneupane - Gentle bump on the checklist when you get a chance.

BugZero Checklist: The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:

[ ]  [@sobitneupane] The PR that introduced the bug has been identified. Link to the PR:

https://github.com/Expensify/App/pull/37813

[ ]  [@sobitneupane] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment:

Not Required.

[ ]  [@sobitneupane] A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion:

Not Required.

[ ]  [@sobitneupane] Determine if we should create a regression test for this bug.

Yes.

[ ]  [@sobitneupane] If we decide to create a regression test for the bug, please propose the regression test steps to ensure the same bug will not reach production again.

Regression Test Proposal

Go to Submit expense > Scan > Add valid receipt > Select workspace > Submit
Wait for scanning to complete, once completed edit the amount to a higher value
Verify Amount greater than scanned receipt error appears below amount field
Go to Submit expense > Distance > Select 2 waypoints > Select workspace > Submit
Open transaction page > Wait for amount to be calculated > Edit the amount to higher value
Verify Amount differs from calculated distance error appears below amount field

Do we agree 👍 or 👎

Payment Summary

$250 - @sobitneupane via New Expensify
$250 - @Krishna2323 via Upwork (PAID)

@JmillsExpensify - One payment request incoming.

$250 approved for @sobitneupane

Thanks all, closing now.


GH-469: Add Spring Integration support

Fixes https://github.com/spring-projects-experimental/spring-native/issues/469

Introduce org.springframework.integration.IntegrationHints
for reflection and initialization types required in Spring Integration
for compatibility with Native
Mention spring-boot-starter-integration as supported in the support.adoc
Add samples/integration to demonstrate as much as possible of
Spring Integration in a simple app

Forgot to mention, the native-image-maven-plugin fails for me on Windows. Just because it cannot find cl.exe in the PATH.
So, I had to install Microsoft Visual Studio:
https://stackoverflow.com/questions/64197329/cl-exe-missing-when-building-native-app-using-graalvm
Then I tried to set PATH env var for respective cl.exe, but failed with many other missed stuff in the env:
https://stackoverflow.com/questions/84404/using-visual-studios-cl-from-a-normal-command-line
Perhaps GraalVM could run that recommended vcvars64.bat script from VC build before starting to build an image.
That's what VC people recommend instead of "polluting" ENV for many VC vars: https://social.msdn.microsoft.com/Forums/vstudio/en-US/2ebc59de-dc8e-426f-b1ca-885c2709df5f/visual-c-installed-but-cannot-find-clexe?forum=vcgeneral
Right now I give up in favor of working variant with the spring-boot:build-image.

In https://docs.spring.io/spring-native/docs/current/reference/htmlsingle/#_build_the_native_application_2 we document: On Windows make sure to use x64 Native Tools Command Prompt as recommended in the GraalVM native-image prerequisites. Is that the way you run your build?

Thanks @sdeleuze !
That worked for me: when I run vcvars64.bat from command line and cded to my sample I definitely could build it! Now I see that cool integration.exe in the \target! 😸

@artembilan Glad to hear that. Should I merge this PR for inclusion in 0.10.0 from your POV?

We can merge now or wait until Boot 2.5 GA in a couple weeks.
Plus I see CI has failed for this PR. I'm not familiar with Concourse and complex Maven build, so would be great if someone take a look over there and let me know what is wrong and what to fix if that.
Thanks


Added GetReceivedByAddress function

Hi.  I've looked at this.  Good job. The one comment I would make it that, in the two classes you have added to the framework: AddressBalanceModel and AddressBalance, you have the property CoinType in AddressBalanceModel, but not AddressBalance.  I suggest including CoinType in AddressBalance also to maintain symmetry and because you may need to filter on a collection of the those balances downstream in some other context.

And... Why not add a Unit Test for this function?

I've added the CoinType. But yes you're right, I should have added Unit Test.

wehey check who's here!
I'll review this as well, first thing tomorrow.

Good stuff!


Do not use local:// URIs

Package version (if known): current
Describe the bug
Invenio RFC 42 decided that Invenio should use local: URIs for schemas, but:

This is a non-IANA URI scheme, and we're exposing it to consumers and users of an Invenio system. Invenio should be standards-based wherever possible, and this breaks that principle
The implementation assumes that any schema within a jsonschemas directory is actually local, when implementors may want to locally host an external schema (~as we do with the GeoJSON Geometry schema~ this was an untracked file in my clone of invenio-rdm-record)
~This is breaking my invenio-swh tests when calling RecordService.create(), as that leads to InvenioJSONSchemasState.refresolver_store() asserting that schema.get("$id") == schema_uri but 'http://json-schema.org/draft-07/schema#' != ''local://external/geojson/Geometry.json'~ — resolved by removing the untracked file

Screenshots (if applicable)

Expected behavior
Invenio should use HTTPS URIs throughout for schemas. To resolve the JSONSCHEMA_HOST issue targetted by the RFC:

locally-provided schema URIs should have $id properties without URI scheme or host information (e.g. /schema/records/record-v3.0.0.json)
the resolver should canonicalise to absolute URIs, created by calling urllib.parse.urljoin(BASE_URL, schema["$id"], both when indexing jsonschema/ directories and when finding schemas for records

this means that locally-hosted but external schemas (e.g. GeoJSON) have their external URL preserved
it also means that the layout of the jsonschemas directory is irrelevant, as it uses the $id over the filesystem path. That said, it would be possible to have the collection process issue warnings if the path doesn't align with the $id for non-absolute URLs

records that use schemas that start with the base URL should have their schema URL stored as a relative URL without URI scheme or host, so that these URLs are portable across domain changes
records and schemas can have their $id and $schema properties made absolute on serialization.

Some of the points we've discussed on Discord:

Schemas are for internal use: The JSONSchemas used in Invenio are purely for internal structural validation of the JSON we store in the database (akin to database table definition). The schema contains internal properties we do never want to expose to end-users.
JSONSchemas do not provide sophisticated enough validation, thus a JSON always passes through a Marshmallow schema first, and thus are not useful to provide as an external description.
The local:// was chosen because 1) we did not want to overload an existing URI scheme, 2) wanted to make it explicit that schemas were loaded locally, thus HTTP URIs would create doubts if something was resolved locally or not.


rawTerm for term simplification

fixes: #916

The change here is pretty much exactly what I was expecting to see, thanks Guy - can you run the Pyk and Booster integration test suites to see what (if anything) will need to be updated? My recollection is that:

there are some Pyk tests that will need to be updated similarly to the tests you've changed here
the booster should just work but I don't know precisely the deserialization code path that it uses; it might be the case that some Haskell code is needed to strip the rawTerm{}

cc @goodlyrottenapple @tothtamas28

The pyk integration tests did indeed have a case that needed to be updated. I was able to fix it very quickly, so that won't be an issue.
The booster backend is failing as well, but I'm not sure how to fix it. It plainly states Unknown symbol "rawTerm". So, it needs to use some functionality to strip it there. A grep for rawTerm in the repo doesn't show anything, though, so I guess it doesn't exist yet.

Blocked on: https://github.com/runtimeverification/hs-backend-booster/issues/431


Fix a link to Spring Integration documentation

Fix a link for external Spring INtegration Documentation

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

@YevheniiLutsyshyn Thank you for signing the Contributor License Agreement!

Thank you for contribution; looking forward for more!


Implement cilium clustermesh disconnect for Helm mode

Similar to https://github.com/cilium/cilium-cli/issues/1620 clustermesh connect implementation
Implement cilium clustermesh disconnect using Helm actions.
This is the help message for the non-Helm based existing command:
Disconnect from a remote cluster

Usage:
  cilium clustermesh disconnect [flags]

Flags:
      --destination-context string   Kubernetes configuration context of destination cluster
  -h, --help                         help for disconnect

Global Flags:
      --context string     Kubernetes configuration context
  -n, --namespace string   Namespace Cilium is running in (default "kube-system")

Let's implement the same set of flags (namely --destination-context). This command will remove the cluster identified by --destination-context from the Helm values of the cluster identified by --context, and then apply those Helm values using a helm Upgrade action.
Note: the name of the destination context and the "cluster name" of the destination context may be different (for example, in the case of kind, the context name would typically be kind-c2 while the name of the cluster might be c2.
The implementation primarily needs to iterate through this array in the Helm values, and delete the destination context, then perform the Helm upgrade.
https://github.com/cilium/cilium-cli/blob/main/clustermesh/clustermesh.go#L2089

Hey @asauber, I have started drafting sth at https://github.com/cilium/cilium-cli/pull/1743. Feel free to provide any early feedback :).


Nothing in email

I've been messing around with this script for hours and I finally got it to send me an email. But when I go to check my email, there is nothing inside the message. Could I get some help please.
BTW I have a pretty new machine and have delays on, and I allowed less secure apps on google.
P.S. I also have a few lines of red text right before my second power shell closes. I dont know if that helps you out or not,, just wanted to address that.

yeah I had the same problem here is my fix:
fix code line 22 to:
DigiKeyboard.print(F("(netsh wlan show profiles) | Select-String '\:(.+)$' | %{$name=$.Matches.Groups[1].Value.Trim(); $} | %{(netsh wlan show profile name=$name key=clear)}  | Select-String 'Key Content\W+\:(.+)$' | %{$pass=$.Matches.Groups[1].Value.Trim(); $} | %{[PSCustomObject]@{ PROFILE_NAME=$name;PASSWORD=$pass }} | %{(netsh wlan show profile name=$name key=clear)}  | Select-String 'Key Content\W+:(.+)$' | %{$pass=$.Matches.Groups[1].Value.Trim(); $} | %{[PSCustomObject]@{ PROFILE_NAME=$name;PASSWORD=$pass }} | **Export-Csv temp.csv**" "));
the error u were getting in PS was this one:
Cannot convert argument "item", with value: "temp.csv", for "Add" to type "System.Net.Mail.Attachment": "Cannot
convert value "temp.csv" to type "System.Net.Mail.Attachment". Error: "Could not find file
'C:\WINDOWS\system32\temp.csv'.""
At line:1 char:464

... rds Your Digispark'; $ReportEmail.Attachments.Add('temp.csv'); $SMTPI ...

                     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : NotSpecified: (:) [], MethodException
FullyQualifiedErrorId : MethodArgumentConversionInvalidCastArgument

Ok so now it has this error message:
exit status 1
Error compiling for board Digispark (Default - 16.5mhz).
This report would have more information with
"Show verbose output during compilation"
option enabled in File -> Preferences.
And thats before I changed any code. I haven't touched the arduino software in about a month so I don't know if it has something to do with that, or if its my settings or what. Also, before that message, I was able to get an email so I'm pretty sure my settings are correct unless the software has changed of something. Please help me thanks!

I had the same problem with the attachment file
putting the full file path into a variable and passing it as the argument seems to fix it for me.
replace the e-mail sending line with:
DigiKeyboard.print(F("$path = join-path $env:userprofile 'temp.csv'; $SMTPInfo = New-Object Net.Mail.SmtpClient('smtp.gmail.com', 587); $SMTPInfo.EnableSsl = $true; $SMTPInfo.Credentials = New-Object System.Net.NetworkCredential('GMAIL_USERNAME', 'GMAIL_PASSWORD'); $ReportEmail = New-Object System.Net.Mail.MailMessage; $ReportEmail.From = 'SENDER_MAIL'; $ReportEmail.To.Add('RECEIVER_MAIL'); $ReportEmail.Subject = 'DigiSpark Report'; $ReportEmail.Body = 'Attached is your report. - Regards Your Digispark'; $ReportEmail.Attachments.Add($path); $SMTPInfo.Send($ReportEmail);exit"));


where shall parameters get stored?

As already seen in PR #207 and issue #209 there is some need for more per-repository and/or per-payload parameter storage:

key / encryption type
hash / mac algorithm
compression algorithm + compression algorithm parameters

Currently, there seem to be 2 places where such information can be stored:

first byte of every payload
$repopath/config file (.ini style)

While some stuff could be defined once and be global per repository, it might be more flexible to have it in each payload, so you can switch to new stuff without having to start a new repo.
But: one byte is not enough to represent all we need.

payload currently uses byte0 with values 0x00, 0x01, 0x02 (to give key/enc type). this should be recognized also in future for backwards compatibility (but always write new flexible format, so old format can be dropped at some time in the future).
idea for new flexible format for parameter record:

use byte0 values 0x03 and beyond
if byte0 >= 0x03, lookup parser function from mapping. the parser sets up everything as needed and returns an offset to the data after the parameter record (here: 4)
byte0: TYPE, 0x03 for now
byte1: key/encryption type (0x00, 0x01, 0x02, ... (as previously put into byte0))
byte2: compression method
byte3: hash / hmac method
byte4: everything that was at offset 1 with old format

So, we have 3 bytes more to store, but gain flexibility for the future. Also the code gets simpler than in PR #207. As we have 256 values for compression, we could even map some parameters directly, e.g. use 0-9 for gzip+level, 10-19 for lzma+level, etc. (alternatively: use 1 byte more for compression params).

Some measurements with PR #207 code - all tests on local SSD filesystems:
--compression=6 --mac=0 (zlib default level 6 + sha256)
Duration: 6 minutes 53.29 seconds
Number of files: 247725   6.03 GB              2.36 GB              2.15 GB
--compression=6 --mac=1 ("" + sha512-256)
Duration: 6 minutes 42.46 seconds
Number of files: 247725   6.03 GB              2.36 GB              2.15 GB
--compression=1 --mac=1 (fastest zlib compression + "")
Duration: 4 minutes 29.36 seconds
Number of files: 247725   6.03 GB              2.53 GB              2.31 GB
--compression=0 --mac=1 (no compression + "")
Duration: 4 minutes 15.61 seconds
Number of files: 247725   6.03 GB              6.04 GB              5.49 GB

I changed the 0x03 format again to use msgpack to get more flexibility and easier code.
Also using a Meta namedtuple now to remove the pain with all these hardcoded offset/byte-ranges.


Issue creating 2GP packages without namespace

Describe the bug
When attempting to create a new 2GP package version via the build_feature_test_package flow it errors if the project and subsequent packages do not contain a namespace.
Reproduction steps
Steps to reproduce the behavior:

Initialize CCI project without namespace
Create 2GP package shell
Ensure there is some test metadata in force-app repo
Run build_feature_test_package flow

Context

Your CumulusCI version (3.20.1).
Your Python version (Python 2.7.16)
MacOS version 10.15.7
CCI installed via Brew

Gist
https://gist.github.com/5587fecec7c132aa45bb35f0a2a8ff90

@anspaujd Are you trying to create a managed 2gp or an unlocked 2gp? The build_feature_test_package flow is specifically defaulting to building a 2gp managed beta package version, because we intend to use it to test Salesforce.org's managed packages. And managed packages must have a namespace.
If you're trying to create an unlocked package, you have a couple options:

You can set the package_type option when running the flow: cci flow run build_feature_test_package --org dev -o create_package_version__package_type Unlocked
You can set package_type: Unlocked in the project: package section of cumulusci.yml (same section as the package name)

@anspaujd Are you trying to create a managed 2gp or an unlocked 2gp? The build_feature_test_package flow is specifically defaulting to building a 2gp managed beta package version, because we intend to use it to test Salesforce.org's managed packages. And managed packages must have a namespace.
If you're trying to create an unlocked package, you have a couple options:

You can set the package_type option when running the flow: cci flow run build_feature_test_package --org dev -o create_package_version__package_type Unlocked
You can set package_type: Unlocked in the project: package section of cumulusci.yml (same section as the package name)

@davisagli - Attempting to create unlocked packages. That updated command worked to create a package however it is not pulling the existing package shell and versioning. Currently my sfdx-project.json reads as follows:
{
    "packageDirectories": [
        {
            "path": "force-app",
            "default": true,
            "package": "MetaCI_Test",
           "versionName": "ver 0.1",
            "versionNumber": "0.1.0.NEXT"
        }
    ],
    "namespace": "",
    "sourceApiVersion": "49.0",
    "packageAliases": {
        "MetaCI_Test": "0Ho6g000000XZsjCAG",
<EMAIL_ADDRESS>"04t6g0000083ixWAAQ"
   }}`
But when the updated command runs, it outputs:
Created package version:
2020-10-15 09:18:58:   Package2 Id: 0Ho6g000000XZstCAG
2020-10-15 09:18:58:   Package2Version Id: 05i6g000000XaS7AAK
2020-10-15 09:18:58:   SubscriberPackageVersion Id: 04t6g0000083iy0AAA
2020-10-15 09:18:58:   Version Number: <IP_ADDRESS>
2020-10-15 09:18:58:   Dependencies: None
Am I missing something or is this working as expected? It seems odd that the build package command would ignore the existing package shell and create new package at large?

This is working as expected. Our general philosophy with CumulusCI's 2gp support was to try to create a way of releasing and installing 2gp package versions that does not use sfdx-project.json. We think that package and package version metadata belongs outside of the repository for a few reasons:

For a repository that is open source or shared, it's conceivable that different parties might release different packages from the same source.
It makes it easier to set up an automated release pipeline, because package info doesn't need to be committed back to the repo.
It opens the possibility of putting package info in a registry that can be accessed by parties who don't have access to the repository (though this is more hypothetical at this point: so far we've been planning to put this info in git tags, so it would still require read access, similar to how CumulusCI resolves 1gp package versions).

Since we aren't using sfdx-project.json to get the package id, the create_package_version task queries the Dev Hub to see if there is an existing package with the specified package name and type. If found, it'll make a new package version of that package; otherwise it creates a new package. In your case, perhaps there is a mismatch between the existing package name and the package name configured in cumulusci.yml. There should be some log lines further up in the output that show the task checking for an existing package.


[5.2] updated_at column always updated, regardless of 'timestamps' option

Doing something like:
$user->name = 'user ' . date('H:i:s');
$user->save(['timestamps' => false]);

Results in:
'name' => 'user 12:36:35'
'updated_at' => '2016-05-02 12:36:35'

The reason appears to be that, although in the model's performUpdatemethod the updated_at column is discarded,  Illuminate\Database\Eloquent\Builder adds it again.

As in https://github.com/laravel/framework/issues/12627 and https://github.com/laravel/framework/issues/10028.
This feature is broken, and was removed in 5.3 https://github.com/laravel/framework/pull/12617.
Use this approach instead:
...
$user->timestamps = false;

$user->save();

You can go with @themsaid's solution or you can simply do this in your Eloquent model as well:
public $timestamps = false;

Oops, sorry for the duplication, and thanks for the response.


Add consistent style badges in section Tech Stack

Please assign this issue to me and I will resolve this quickly.


How to rebuild the results of INSHOP dataset

So this is the same question (link below) but for the INSHOP dataset
https://github.com/pierre-jacob/ICCV2019-Horde/issues/4
What do I need to change get your 90%~ recall@1?
With the settings from run_cub.sh I get around 60% recall@1.
Thanks in advance!
Ping! @pierre-jacob who kindly answered the last issue.

By the way, it's really nice to see working code from the authors of the original paper.
Should be made nessecary to submit code when publishing ai/deep-learning papers.
Thank you for submitting well-written code freely on github.
And I also imagine you submitted the paper/code to https://paperswithcode.com/task/metric-learning
Which was the reason I found this project.

In-Shop and Online Products are two large-scale datasets (25k and 60k images respectively) compared to Cub (5k images). You have to increase the number of steps per epoch in the “config.json” file.
Usually, 1k steps for In-Shop and 2k steps for Online Products are enough.
Let me know if this works for you!
Btw, I am happy that this code helps you! If you have any difficulties, don’t hesitate to send me an e-mail!

Since this issue has been opened few months ago, and is not active anymore, I hope the given answer has solved this reproducibility issue.
Fell free to re-open an issue otherwise.


Feature: Adding contributors section to the README.md file.

There is no Contributors section in readme file .
As we know Contributions are what make the open-source community such an amazing place to learn, inspire, and create.
The Contributors section in a README.md file is important as it acknowledges and gives credit to those who have contributed to a project, fosters community and collaboration, adds transparency and accountability, and helps document the project's history for current and future maintainers. It also serves as a form of recognition, motivating contributors to continue their efforts

@AnthonyLatsis Kindly assign me this issue, I want to work on it. Thank You!!

Does the list need to be hardcoded?
What's your motivation for having this section in addition to the contributors displayed on the repository page, and the contributors page itself?

@xedin @hborla @MaxDesiatov Thoughts?

There are at least 1043 contributors right now according to GitHub. Would with this suggestion all of them be displayed in README.md? If not, what subset of those? How would that subset be sorted? How will the subset list be updated? Wouldn't it be easier to link to GitHub's contributors page instead? OTOH, the contributors page is already linked from the default repository view, both on desktop and mobile, and at least on desktop already provides a subset of that list visible by default. What would be the benefit of duplicating that information in README.md?

Yeah, its true ! but all the contributors will display on Readme.md. if you want then i can update it.

@AnthonyLatsis @MaxDesiatov hey can I work on this?

As discussed above in this thread, we need answers to those questions about how this would work answered first.


[BUG] - The overlay gives me this error what can I do

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
every time I run the overlay it asks me to update, I update but it gives an error and doesn't let me use the overlay
How often does the bug occur?
Rarely
What operating system are you using?
Windows 10 21H2
What version of the program are you using?
Latest version
What version of the game are you using?
High-Grade Edition, Non High-Grade Edition
Quest ID
64123

Can you re-send the image? Also send the logs file found in %localappdata%/MHFZ_Overlay/app-0.27.3/logs.


Compiling OpenBLAS using the Nvidia compiler

Dear Sir:
I am having trouble compiling OpenBLAS using the Nvidia compiler, formerly the PGI compiler.  It does not like "-march=native".  In the file "Makefile.system", there is code to exclude "pgcc" from using this flag:
On x86_64 build getarch with march=native unless the compiler is PGI. This is required to detect AVX512 support in getarch.
ifeq ($(HOSTARCH), x86_64)
ifeq ($(findstring pgcc,$(HOSTCC)),)
GETARCH_FLAGS += -march=native
endif
endif
Does anyone know the syntax for excluding both "pgcc" and "nvc" from adding the "-march=native" flag.  Please excuse me if this issue has already been addressed.    I am using
Version 0.3.13
12-Dec-2020

nvc support was added shortly after the 0.3.13 release. Probably best to fetch a snapshot of the current develop branch or wait a (hopefully) few days for 0.3.15.

Dr. Kroeker,
Thank you for your helpful response.  I will wait for 0.3.15 to be released.
Tom Oppe

Thomas Oppe | SAIC
Benchmarking Team Computational Scientist | ERDC DSRC
601-642-4516

From: Martin Kroeker @.***>
Sent: Wednesday, April 28, 2021 10:39:18 PM
To: xianyi/OpenBLAS
Cc: Oppe, Thomas C ERDC-RDE-ITL-MS Contractor; Author
Subject: Re: [xianyi/OpenBLAS] Compiling OpenBLAS using the Nvidia compiler (#3200)
nvc support was added shortly after the 0.3.13 release. Probably best to fetch a snapshot of the current develop branch or wait a (hopefully) few days for 0.3.15.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubBlockedhttps://github.com/xianyi/OpenBLAS/issues/3200#issuecomment-828918848Blocked, or unsubscribeBlockedhttps://github.com/notifications/unsubscribe-auth/AMLWS3CKAY5NROMR25JQHFLTLDIGNANCNFSM43X2MBFABlocked.

0.3.15 released now


Novusk v3 architectures

A list of new architectures for Novusk v3

[x] Arm32
[ ] RISCV

ix86:

[x] i386
[ ] i486
[ ] i586

b18c3ce6 Added arm32 support

8deb273 Worked on arm32/64

Scott Mabin - Xtensa will officially be a supported architecture in Rust. You could compile Xtensa Novusk v2 with this compiler he wrote, but in the future you'll probably just use the Rust compiler.

65c6d58 Added RISCV support

Rust #64 - asm! and global_asm! aren't supported for Xtensa which is weird because the esp libraries use libraries that use assembly macros, so the esp libraries can't be used and no low-level stuff can be done. When Xtensa can be supported it will be.

After Novusk v3-beta Arch v3 I guess this issue could be closed


Add gtc:dace:gpu backend

bors try

What is the status of this? The ai2cm team is depending on this so it'd be good to push it forward and merge. Let me know if there's something I can do to help that.

Closing in favor of another PR that will come on top of #623


Upgrade Travis CI to container-based infrastructure

See multi-ghc-travis for instructions on how to set up Travis with container-based infrastructure and caching of dependencies.

Hi @jpvillaisaza - I updated the .travis.yml today, based on the latest recommendations from Stackage. I think that my changes cover these concerns, so I'm closing this issue, but please feel free to open this or another issue if you have other suggestions.

@jsl Great! I think the changes cover more than I had in mind. I like that you included OS X as well (they seem to take more time, but not too much).


return pull request creation response

+10 for this.

:tada: This issue has been resolved in version 1.2.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Fix support for tags in the 'acs' commands.

Fixes https://github.com/Azure/azure-cli/issues/2693

Codecov Report

Merging #2956 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #2956      +/-   ##
==========================================
+ Coverage   63.33%   63.33%   +<.01%
==========================================
  Files         466      466
  Lines       26539    26541       +2
  Branches     4069     4069
==========================================
+ Hits        16809    16811       +2
  Misses       8617     8617
  Partials     1113     1113

Impacted Files
Coverage Δ

...re-cli-acs/azure/cli/command_modules/acs/custom.py
38.49% <ø> (ø)
:arrow_up:

...e-cli-acs/azure/cli/command_modules/acs/_params.py
69.64% <100%> (+1.12%)
:arrow_up:

Continue to review full report at Codecov.

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

Ping to @derekbekoe on this one.  Thanks!


The README is misleading

I've just upgraded to version 0.2.33. After reviewing the README, I was wondering why my app was failing due the error intercom_shutdown is undefined.
It took me some time to realise that this version doesn't include the code referred in the README.
In order to save time to other users, please:

Release a new version or update the README with the correct behaviour of the current release.
Keep a CHANGELOG file.

Thank you!

Hey,
I've just upgraded to the version 0.2.34. Sorry it took so long and that you had to figure out on your own.


[Hack n Hustle] Get and update common fields across advisories

I wanted a command to update the shipping date of a group of advisories without having to go through ET.
Made it a little general purpose so you can get any field across advisories, if you wanted to.
And then update common fields across advisories like date, manager, qe, owner etc.
I don't see us updating the others as much
and date as basically the only real feature :)
but worth it

Nice! Would be great with a search-and-replace for the title in case, say, 4.8.8 gets renamed as 4.8.9.

Nice! Would be great with a search-and-replace for the title in case, say, 4.8.8 gets renamed as 4.8.9.
Apologies for the scope creep ;)

I added a --version-replace option as a shortcut to bump versions. PTAL
Noticed that we can't do a blanket search and replace in description (because of bug titles being part of it), so I added template lines

LGTM once you fix the conflict.


Support for linux/arm64 (ARMv8, aarch64)

[x] Feature Request or Change Proposal

OS/Container environment:
ARMv8 server is a Packet 2A (Cavium ThunderX, 96-core at 2 Ghz)
Steps or code to reproduce the issue:

Visit releases page, https://github.com/nats-io/gnatsd/releases
Look for linux-arm64 version

Expected result:
linux-arm64 supported release
Actual result:
No files found.
As of 2017-04-04, build works fine, tests fail until timeouts are extended, and we've identified a performance issue on ARMv8 Go 1.8 crypto/tls. Further work pending Go performance improvements on ARMv8.
Feature Requests
Use Case:
Two use cases: one for ARMv8 single-board computers (e.g. Raspberry Pi 3, Odroid C2, Pine64); another for ARMv8 in the data center (e.g. Cavium ThunderX).
Proposed Change:
Build and test for arm64, validate that it works, add as supported release.
Who Benefits From The Change(s)?
Users of arm64 (ARMv8) platforms as listed above.
Alternative Approaches
Planning to build from source and see how that goes; I'll use this issue to identify anything that comes up.

test fails; looking into this:
root@docker-build-test:~/src/nats-io/gnatsd# go build
root@docker-build-test:~/src/nats-io/gnatsd# go test ./...
?       nats-io/gnatsd  [no test files]
?       nats-io/gnatsd/auth     [no test files]
ok      nats-io/gnatsd/conf     0.018s
ok      nats-io/gnatsd/logger   0.618s
ok      nats-io/gnatsd/server   24.935s
ok      nats-io/gnatsd/server/pse       0.104s
--- FAIL: TestServerRestartReSliceIssue (10.01s)
panic: Unable to start NATS Server in Go Routine [recovered]
        panic: Unable to start NATS Server in Go Routine

goroutine 44 [running]:
panic(0x8154a0, 0x482000b7e0)
        /usr/lib/go-1.6/src/runtime/panic.go:481 +0x384
testing.tRunner.func1(0x4820250870)
        /usr/lib/go-1.6/src/testing/testing.go:467 +0x168
panic(0x8154a0, 0x482000b7e0)
        /usr/lib/go-1.6/src/runtime/panic.go:443 +0x4b4
nats-io/gnatsd/test.RunServerWithAuth(0x482027c3c0, 0x0, 0x0, 0xffff9c66e110)
        /root/src/nats-io/gnatsd/test/test.go:102 +0x180
nats-io/gnatsd/test.RunServerWithConfig(0x9c72f0, 0x14, 0x0, 0x482027c3c0)
        /root/src/nats-io/gnatsd/test/test.go:79 +0x2a4
nats-io/gnatsd/test.runServers(0x4820250870, 0x0, 0x0, 0x0, 0x0)
        /root/src/nats-io/gnatsd/test/cluster_test.go:66 +0x4c
nats-io/gnatsd/test.TestServerRestartReSliceIssue(0x4820250870)
        /root/src/nats-io/gnatsd/test/client_cluster_test.go:17 +0x3c
testing.tRunner(0x4820250870, 0xbff288)
        /usr/lib/go-1.6/src/testing/testing.go:473 +0xbc
created by testing.RunTests
        /usr/lib/go-1.6/src/testing/testing.go:582 +0x65c
FAIL    nats-io/gnatsd/test     11.477s
?       nats-io/gnatsd/util     [no test files]
?       nats-io/gnatsd/vendor/github.com/nats-io/nuid   [no test files]
?       nats-io/gnatsd/vendor/golang.org/x/crypto/bcrypt        [no test files]
?       nats-io/gnatsd/vendor/golang.org/x/crypto/blowfish      [no test files]
?       nats-io/gnatsd/vendor/golang.org/x/sys/windows  [no test files]
?       nats-io/gnatsd/vendor/golang.org/x/sys/windows/registry [no test files]

Can you just run from command line to see what happens:
gnatsd

Thanks!

Run from command line works just fine - at least the server comes up.
Is there a particularly good client you'd recommend to exercise the server, @kozlovic ? Happy to bash on it to see if I can trigger whatever this issue is.

Just realized that it worked for server package.
Could you make sure that there is no gnatsd running in the background and then do this just to check:
go test -race -v -p=1 ./...

The -p=1 will ensure that each package is run after the other. I am just wondering if there could be ports conflicts between the tests in different packages. We normally try to use different ports, and it works fine on Travis, but it could be just luck.

go test -race is not available in Go 1.6.x on arm64 on Ubuntu.
With -p=1 I get a lot more tests to pass, but a few still fail, all related to TLS:
root@docker-build-test:~# grep FAIL gnats-test.out
--- FAIL: TestTLSConnz (1.12s)
--- FAIL: TestPingSentToTLSConnection (0.71s)
--- FAIL: TestTLSConnection (1.19s)
--- FAIL: TestTLSBadAuthError (1.11s)
FAIL
FAIL    nats-io/gnatsd/test     73.991s
root@docker-build-test:~# go version
go version go1.6.3 linux/arm64

Looking in a little more detail, here are all of the error messages:
root@docker-build-test:~# grep "version 4552" gnats-test.out
        monitor_test.go:337: Got an error on Connect with Secure Options: tls: received record with version 4552 when expecting version 303
        test.go:128: Error writing command to conn: tls: received record with version 4552 when expecting version 303
        tls_test.go:44: Got an error on Connect with Secure Options: tls: received record with version 4552 when expecting version 303
        tls_test.go:252: Excpected and auth violation, got tls: received record with version 4552 when expecting version 303

You may want to try with a newer version of Go, just to make sure.

Oh, that's because the timeouts are too small.

Let me see in which place you would have to increase this timeout to make sure that's only that.

Two things you could try:

Run those failed tests individually to see if they still fail: go test -v -run=TestTLSConnz ./test
Increase the auth and tls timeout values
Since there may be several places in tests to change that value, I would just override in the server code, again just to make sure that this is simply a timeout issue.
That would be in server/server.go:591:

//ttl := secondsToDuration(s.opts.TLSTimeout)
ttl := 10*time.Second

And server/client.go:1244:
// c.atmr = time.AfterFunc(d, func() { c.authTimeout() })
c.atmr = time.AfterFunc(10*time.Second, func() { c.authTimeout() })

Could you please try and report back?

Single test still fails:
root@docker-build-test:~/src/github.com/nats-io/gnatsd# go test -v -run=TestTLSConnz ./test
=== RUN   TestTLSConnz
--- FAIL: TestTLSConnz (1.11s)
        monitor_test.go:337: Got an error on Connect with Secure Options: tls: received record with version 4552 when expecting version 303
FAIL
exit status 1
FAIL    github.com/nats-io/gnatsd/test  1.134s

My version of Go is 1.6.3 which is older than the one you recommend; I'll report back separately testing under Go 1.8.
root@docker-build-test:~/src/github.com/nats-io/gnatsd# go version
go version go1.6.3 linux/arm64

When you ran the test, have you override the timeouts? For that test specifically, if you do not want to tweak the code, you can modify the config file used in this test:
test/configs/tls.conf

Change both timeout values in this file to 10 instead of 2 and 1.

Wtih longer timeouts, the 10 second times patched in above into client.c and server.c, we pass a test:
=== RUN   TestTLSConnz
--- PASS: TestTLSConnz (2.25s)
PASS
ok      github.com/nats-io/gnatsd/test  2.269s

minio has some accelerated crypto routines which should speed up TLS, if that timeout is due to slow performance.

Ok, now the problem re-running the whole test suite with the override is that you may then get some test failures because the test expect the timeout to occur say within 2 seconds. But we should be able to figure out if that's the case based on the test name.

All the TLS tests now pass, but there's one test that fails:
=== RUN   TestAuthClientNoConnect
--- FAIL: TestAuthClientNoConnect (3.03s)
        test.go:128: Error reading from conn: read tcp <IP_ADDRESS>:43868-><IP_ADDRESS>:
10422: i/o timeout

                2 - /root/src/github.com/nats-io/gnatsd/test/auth_test.go:80
                3 - /usr/lib/go-1.6/src/testing/testing.go:473
                4 - /usr/lib/go-1.6/src/runtime/asm_arm64.s:975

The code reads
        // This is timing dependent..
        time.Sleep(server.AUTH_TIMEOUT)

Yes, like I said. So it means that the only failures you got were due to timeout. What surprises me is that you go the failures in the first place. Even with current values (sometimes as low as 0.5 is some config files), it works even when running the suite on Travis, which sometimes is way slower than when we run on our personal laptops. So it is a bit surprising considering the spec of your machine?

The timeouts are very surprising given the spec of the machine. I'm going to rebuild with Go 1.8 next, because I know I've seen speed improvements overall with that, and maybe that is enough to help.
With one failed test, I get this as an overall test time:
FAIL    github.com/nats-io/gnatsd/test  80.181s

and it looks like the last log on Travis runs the same tests in
ok  	github.com/nats-io/gnatsd/test	68.013s

With Go 1.8 it fails a little faster
FAIL    github.com/nats-io/gnatsd/test  78.930s

still failing in
--- FAIL: TestAuthClientNoConnect (3.03s)

I'm sure that's because Go is using software crypto on arm64, rather than the hardware instructions on the chip. The minio code is at https://github.com/minio/sha256-simd which might help.

Nope, gnatsd doesn't use the sha256 code, but I was able to benchmark Go's crypto/tls and found it wanting on arm64. The open issue is
https://github.com/golang/go/issues/19840
I'll chase this upstream, for the moment let's mark this issue as "on hold", and I'll work to get a performance improvement.

Go 1.9 beta 1 is out and has a binary build for ARM64 (yay).
According to the referenced issue https://github.com/golang/go/issues/19840 the opportunity for this particular performance issue to be resolved in Go for ARM will come in the Go 1.10 timeframe. However there may be other performance improvements in Go 1.9 so that's worth a quick test.

Is it possible for you to summarise the state of the aarch64 server build, we are very interested in using it on our embedded aarch64 platform as a control plane enabler.

@salerio - what are the specs for your aarch64 platform? The concern expressed above was that some of the crypto instruction in Go on aarch64 are not hardware accelerated, and that the soft versions of the algorithms have poor performance on one system (Cavium ThunderX).

Its a Xilinx UltraScale+ MPSoC which has 4 x Cortex-A53 CPU complex. Although there are crypto accelerators in the SoC I doubt anyone (any standard software that is) will make sure of them yet as the part is very new.
See https://www.xilinx.com/products/silicon-devices/soc/zynq-ultrascale-mpsoc.html

@ghost does this MPSoC from Xilinx have an FPGA in it?

We would be interested in what you find, keep us posted.


fix: namespaced type refs resolve

Now, a type reference from something like Deno.connect to Deno.Conn provides a link. Previously, it would not have resolved a link to the type reference.
Fixes #25

@bartlomieju already requested for Deploy, not currently possible.

@kitsonk thanks for the heads up, looking forward to it


Add Maria Shaldybin as Diego approver

I'd like to be added as an approver in the Diego working group.
@ameowlia would you be willing to nominate me?
Submitted substantial PRs:

https://github.com/cloudfoundry/executor/pull/54
https://github.com/cloudfoundry/executor/pull/62
https://github.com/cloudfoundry/executor/pull/58
https://github.com/cloudfoundry/diego-release/pull/595
https://github.com/cloudfoundry/diego-release/pull/596

Reviewed substantial Issues:

https://github.com/cloudfoundry/diego-release/issues/626
https://github.com/cloudfoundry/diego-release/issues/587
https://github.com/cloudfoundry/diego-release/issues/444
https://github.com/cloudfoundry/diego-release/issues/582

Involved in technical discussions:

Dynamic ASGs
C2C over TLS
NATS v2
HTTP healthcheck Improvements

Resolved slack interrupts (in cloudfoundry.slack.com seach in:#diego from:@mariash).
My tenures:

I was an engineer on Volume Services (now part of Diego) team in 2018 - 2019.
I was an engineer on Diego team in mid 2019 - feb, 2020.
I have been an engineer on TAS Runtime team in aug, 2021 - present.

👍 I approve Maria's nomination. Besides the things she listed she was also one of the main architects behind the dynamic ASG feature which spanned cf-networking, silk, and diego.


Adding PER and getMaxValueIndex function

Adding PER, but it's not very efficient right now.
Adding the simple function getMaxValueIndex

@raimannma thanks for adding this! Usually we'd add tests before merging, but this branch is super experimental so we'll just go for it lol
By the way, did you find that the RL agent was able to learn?

@all-contributors please add @raimannma for code

@raimannma thanks for adding this! Usually we'd add tests before merging, but this branch is super experimental so we'll just go for it lol
By the way, did you find that the RL agent was able to learn?

I haven't done any tests, so i can't tell you anything about the RL Agent.
But as you saw i rewrote neataptic to Java and in my java port i already implemented a reinforcement learning agent.


dyld: Symbol not found:

updated the starscream in my project and getting this error.
dyld: Symbol not found: __TWPSSSs8Hashable
Referenced from: /private/var/mobile/Containers/Bundle/Application/9F09CA1E-A5B7-400E-84BF-82ED8C410341/jamesAppV2.app/Frameworks/Starscream.framework/Starscream
Expected in: /private/var/mobile/Containers/Bundle/Application/9F09CA1E-A5B7-400E-84BF-82ED8C410341/jamesAppV2.app/Frameworks/libswiftCore.dylib
in /private/var/mobile/Containers/Bundle/Application/9F09CA1E-A5B7-400E-84BF-82ED8C410341/jamesAppV2.app/Frameworks/Starscream.framework/Starscream
(lldb)
and crashed
Can you please tell me the way how to add starScream Library in my project using swift 1.2

the easiest way with pod install
https://cocoapods.org/

I have used both cocoapods and manual install of the framework without issue. I assume your are trying to deploy this to a real device. You have to add the framework to the copy phase so it will be available to your app. You need to follow:
https://github.com/daltoniam/Starscream#add-copy-frameworks-phase
Might also want to clean your build folder. Hold Option while the Product menu is open in Xcode, which will change Clean to Clean Build Folder. Select the Clean Build Folder.


Update k8s manifests

Title
What does this PR do?

Resolve #4

@kuronosec  please see the code factor comments


Sprite content size should not change on newly applied frame if stretch mode is enabled

axmol version: https://github.com/axmolengine/axmol/tree/1baf5cde3b0ccadf63fef045521723497d6f346f
devices test on:
developing environments

NDK version: r23c
Xcode version: 14.2+
Visual Studio:

VS version: 2022 (17.9+)
MSVC version: 19.39+
Windows SDK version: 10.0.22621.0+

cmake version:
Steps to Reproduce:

Use grossini.plist sprite sheet and texture from cpp-tests.
auto spriteCache = SpriteFrameCache::getInstance();
spriteCache->addSpriteFramesWithFile("grossini.plist");

ax::Vector<ax::SpriteFrame*> animFrames;
for (int i = 1; i < 15; i++)
{
    auto frameName = fmt::format("grossini_dance_{:02}.png", i);
    animFrames.pushBack(spriteCache->getSpriteFrameByName(frameName));
}

auto* animation = ax::Animation::createWithSpriteFrames(animFrames, 1 / 25.f, -1);
auto* animationAction = ax::Animate::create(animation);
auto animatingSprite  = ax::Sprite::createWithSpriteFrame(animFrames.front());
animatingSprite->setPosition(visibleSize / 2);
animatingSprite->setStretchEnabled(true);
animatingSprite->setContentSize(ax::Vec2(200, 200));
animatingSprite->runAction(animationAction);
this->addChild(animatingSprite);

Expected output when animating (size of 200, 200 due to stretch factor):

Actual output while animating:

Not sure about the best way to fix this.
Can add a new FixedSizeAnimate action is added that always sets the content size after each new frame on a sprite, but that means calculations inside the Sprite will be done twice, once in the chain called from setSpriteFrame -> setTextyureRect etc., and then another by the extra setContentSize.  They all end up calling updateStretchFactor(); and updatePoly();, so they'll be called twice for every frame.
Another option would be to add something to Sprite to indicate that the content size is fixed, and should not change on new textures/frames (as long as it's not a content size of 0, 0).

API: change to setAutoSize?

Yes! That makes way more sense. Will update the PR.


Allow cross-token transactions from/to exchange wallet

Issue/Task Number: T467
Overview
This PR fixes the entries generation for cross-token transfers that involve from or to address being the exchange wallet itself.
Changes

Add support for cross-token transfers from/to an exchange wallet into EWallet.TransactionFormatter
Add tests to validate the change in TransactionFormatterTest as well as transaction controller tests.
Fill in some missing test cases between admin user and provider requests.

Implementation Details
Because usual cross-token transfers require 4 debit/credit entries (2 for moving from_token into the exchange account, 2 for moving to_token out of the exchange account), so the existing TransactionFormatter generates a pair of debit/credit that goes to the same wallet address when an exchange wallet is also used as from or to address.
This is fixed by having the TransactionFormatter detects that if the from/to is the same address as the exchange wallet, it should generate only 1 pair of entries that moves the funds from the other wallet into the exchange wallet or vice versa.
Usage
Try /transaction.create with from or to address being the same as the exchange wallet, or /me.create_transaction with to address being the same as the exchange wallet. The operations should be successful.
Impact
Code changes only. No DB migrations or API request/response format changes.

@T-Dnzt Done!


⚠ HawkeUno has degraded performance

In 40c07a2, HawkeUno ($UNO) experienced degraded performance:

HTTP code: 403
Response time: 6246 ms

Resolved: HawkeUno performance has improved in 911bedd after 33 minutes.


Use native system instructions for Gemini 1.5 models

The current code in google_manifold_pipeline adds a "system"-like prompt by prepending the request with a message containing the system prompt:
if system_message:
    contents.insert(0, {"role": "user", "parts": [{"text": f"System: {system_message}"}]})

On Gemini 1.5 models, Google added support for native system instructions. In the Python API it is passed as an argument for the constructor:
model = genai.GenerativeModel(model_name=model_id, system_instruction=system_message)

The equivalent in the REST API is documented here:
curl https://generativelanguage.googleapis.com/v1beta/models/gemini-1.5-flash:generateContent?key=$GOOGLE_API_KEY \
-H 'Content-Type: application/json' \
-d '{ "system_instruction": {
    "parts":
      { "text": "You are Neko the cat respond like one"}},
    "contents": {
      "parts": {
        "text": "Good morning! How are you?"}}}'

In my patch I added support for it when the string gemini-1.5 is part of the model id:
            if "gemini-1.5" in model_id:
                model = genai.GenerativeModel(model_name=model_id, system_instruction=system_message)
            else:
                if system_message:
                    contents.insert(0, {"role": "user", "parts": [{"text": f"System: {system_message}"}]})

                model = genai.GenerativeModel(model_name=model_id)

Gemini 1.0 and Gemma models don't support system instructions, so they will fall back to the other approach. Note: after trying to use system_instruction with Gemini 1.0 I got an error from the server, but both Gemma models (gemma2-9b-it and gemma-7b-it) did accept the instruction without error and showed knowledge of it. Officially they don't support system instructions, so it could be a workaround prompt on the server.
Future models, say, like gemini-2.0 will need to be added manually when they become available on the API, otherwise they will fall back to the other approach.
The information returned by the API unfortunately doesn't indicate if the model supports system instructions. Here is for example, the information for Gemini 1.5 pro (returned via https://generativelanguage.googleapis.com/v1beta/models):
   {
      "name": "models/gemini-1.5-pro",
      "version": "001",
      "displayName": "Gemini 1.5 Pro",
      "description": "Mid-size multimodal model that supports up to 2 million tokens",
      "inputTokenLimit": 2097152,
      "outputTokenLimit": 8192,
      "supportedGenerationMethods": [
        "generateContent",
        "countTokens"
      ],
      "temperature": 1,
      "topP": 0.95,
      "topK": 64,
      "maxTemperature": 2
    },

Excellent, let's get a version number bump to 1.3 and I'll merge this.

The two Gemma models do seem to work with the system instruction passed, but I can't really know if all others will.
It's technically possible to add a check for them as well, but I don't know if it matters, since probably most people using the API are using it for the larger model (except for testing maybe).
I guess the script may need to be edited further in the future for those kind of things.

I think you're right, this should cover probably 90% of most people's usage. Thanks! 🙏

Now I realized gemma-7b-it and gemma2-9b-it actually originated from the Groq API, not Gemini. This change doesn't apply to them at all.
I got confused there due to Gemma models appearing on Google AI Studio. The Gemini API currently only provides Gemini models, not any Gemma models.
So, this change does currently apply to 100% of use cases. No need for any change until a different version of Gemini is released.

I also ported this enhancement to the Functions version:
https://openwebui.com/f/justinrahb/anthropic


files without matches again

i managed to record:
At first, showing the bugged picker, returning a lot of files.
i closed and opened again, the bug disappears.

Continuation of https://github.com/fdschmidt93/telescope-egrepify.nvim/issues/11
It is super rare to happen, but happen sometimes

When reporting an issue, please

provide a minimal config with system details and Neovim version
exact reproduction steps

or otherwise I won't help you. Please understand it is not my burden to scavenger hunt details on your issue.
I'm somewhat puzzled why you keep ignoring repeated instructions of doing so and we have to start over. I was so far very supportive, and still it appears that you keep treating Github issues as shoving problems into a customer support line.
Kindly understand that, going forward, I will be simply closing issues that do not include both of the above.


Obvious cases don't work

I have gotten dexter to work for me, but in throwing some obvious cases at it, it's not giving me new indexes.  I have a snapshot of production I am running in which I have dropped every index except primary keys.  I am trying to see if dexter can build me only the indexes I need.
I am running:
PostgreSQL 9.6.6 on x86_64-pc-linux-gnu, compiled by gcc (Ubuntu 5.4.0-6ubuntu1~16.04.4) 5.4.0 20160609, 64-bit
And hypopg version 1.1.0.
And dexter Ubuntu Xenial package version 0.2.1-1506008296.7423139
Here is my example:
I have a table with 453236 records.  It has a field updated_at which spans several years.  I only have an index on the primary key, which is a serial column.  Here is my query:
SELECT * FROM table WHERE updated_at >= current_date - 1;

Here is my dexter invocation:
tail -F <logfile> | dexter -d foo -p 0000 --min-time 0

But dexter simply says:
Processing 1 new query fingerprints
No new indexes found

Here is the query plan with a full scan (warm cache):
                                                       QUERY PLAN
-------------------------------------------------------------------------------------------------------------------------
 Gather  (cost=1000.00..14730.76 rows=39 width=151) (actual time=72.497..74.809 rows=1317 loops=1)
   Workers Planned: 3
   Workers Launched: 3
   ->  Parallel Seq Scan on foo  (cost=0.00..13726.86 rows=13 width=151) (actual time=68.802..70.700 rows=329 loops=4)
         Filter: (updated_at >= (('now'::cstring)::date - 1))
         Rows Removed by Filter: 112980
 Planning time: 0.608 ms
 Execution time: 79.123 ms
(8 rows)

And if I manually create the index, obviously much faster:
                                                             QUERY PLAN
------------------------------------------------------------------------------------------------------------------------------------
 Bitmap Heap Scan on foo  (cost=27.08..3883.25 rows=1374 width=151) (actual time=0.179..0.511 rows=1317 loops=1)
   Recheck Cond: (updated_at >= (('now'::cstring)::date - 1))
   Heap Blocks: exact=151
   ->  Bitmap Index Scan on foo_updated_at_idx  (cost=0.00..26.73 rows=1374 width=0) (actual time=0.160..0.160 rows=1317 loops=1)
         Index Cond: (updated_at >= (('now'::cstring)::date - 1))
 Planning time: 0.365 ms
 Execution time: 0.592 ms
(7 rows)

Hey @jfinzel, can you run with --log-level debug2 and share the output?

Thanks for all your responses - I'm glad to see willingness to improve!
This is related to the schema_name issue.  However, I tried to workaround it using search_path with no success.  Here is the example (I am masking my names):
tail -F foo-2017-12-21_080000.log | dexter -d my_db -p 60000 --min-time 0 --log-level debug2 --log-sql --interval 5 | tee /tmp/dexter.log

Here is my 2 SQL statements (I made the schema for this table in search_path):
SHOW search_path;
               search_path
-----------------------------------------
 "$user", public, foo_schema
(1 row)

SELECT * FROM foo_schema.foo_table WHERE customer_id = 3913437;

SELECT * FROM foo_table WHERE customer_id = 3913437;

Here is Dexter output:
SQL: SELECT hypopg_reset()
                                                                                134,1         Top
SQL: SELECT * FROM pg_available_extensions WHERE name = 'hypopg' AND installed_version IS NOT NULL
SQL: SET lock_timeout = '5s'
Started
Processing 1 new query fingerprints
SQL: SELECT hypopg_reset()
SQL: SELECT table_name FROM information_schema.tables WHERE table_catalog = current_database() AND table_schema NOT IN ('pg_catalog', 'information_schema') AND table_type = 'BASE TABLE'
No new indexes found
--------------------------------------------------------------------------------
Query 01935edfa9c5e040fffddecfd955563351d55fc649
Total time: 0.0 min, avg time: 1 ms, calls: 1
No candidate tables for indexes

SELECT * FROM pg_available_extensions WHERE name = 'hypopg' AND installed_version IS NOT NULL

Processing 2 new query fingerprints
SQL: SELECT hypopg_reset()
SQL: SELECT table_name FROM information_schema.tables WHERE table_catalog = current_database() AND table_schema NOT IN ('pg_catalog', 'information_schema') AND table_type = 'BASE TABLE'
No new indexes found
--------------------------------------------------------------------------------
Query 017694f81fc17a972a9ed4ac824d4914bcb121dbbc
Total time: 0.0 min, avg time: 0 ms, calls: 1
No candidate tables for indexes

SET lock_timeout = '5s'

--------------------------------------------------------------------------------
Query 019650634ebfcd43f242c7f8c0834df9f0ab2ef39c
Total time: 0.0 min, avg time: 0 ms, calls: 1
No candidate tables for indexes

SELECT hypopg_reset()

Processing 1 new query fingerprints
SQL: SELECT hypopg_reset()
SQL: SELECT table_name FROM information_schema.tables WHERE table_catalog = current_database() AND table_schema NOT IN ('pg_catalog', 'information_schema') AND table_type = 'BASE TABLE'
No new indexes found
--------------------------------------------------------------------------------
Query 01df0e7076d117e49a0576c11e3983373e8371c92f
Total time: 0.0 min, avg time: 6 ms, calls: 2
No candidate tables for indexes

SELECT table_name FROM information_schema.tables WHERE table_catalog = current_database() AND table_schema NOT IN ('pg_catalog', 'information_schema') AND table_type = 'BASE TABLE'

Processing 1 new query fingerprints
SQL: SELECT hypopg_reset()
SQL: SELECT table_name FROM information_schema.tables WHERE table_catalog = current_database() AND table_schema NOT IN ('pg_catalog', 'information_schema') AND table_type = 'BASE TABLE'
No new indexes found
--------------------------------------------------------------------------------
Query 01c08a83631a0c4306ba316bbf4b05706d5c3498cd
Total time: 0.0 min, avg time: 1 ms, calls: 1
No candidate tables for indexes

SELECT pg_catalog.quote_ident(datname) FROM pg_catalog.pg_database  WHERE substring(pg_catalog.quote_ident(datname),1,3)='acc'  LIMIT 1000

Processing 0 new query fingerprints
Processing 0 new query fingerprints
Processing 0 new query fingerprints
Processing 0 new query fingerprints
Processing 0 new query fingerprints
Processing 1 new query fingerprints
SQL: SELECT hypopg_reset()
SQL: SELECT table_name FROM information_schema.tables WHERE table_catalog = current_database() AND table_schema NOT IN ('pg_catalog', 'information_schema') AND table_type = 'BASE TABLE'
No new indexes found
--------------------------------------------------------------------------------
Query 012ca94a1b63a807a498285fd718f4ffc1e7f2c9a0
Total time: 0.0 min, avg time: 1513 ms, calls: 1
No candidate tables for indexes

SELECT * FROM foo_schema.foo_table WHERE customer_id = 3913437;

Processing 1 new query fingerprints
SQL: SELECT hypopg_reset()
SQL: SELECT table_name FROM information_schema.tables WHERE table_catalog = current_database() AND table_schema NOT IN ('pg_catalog', 'information_schema') AND table_type = 'BASE TABLE'
SQL: SELECT schemaname AS schema, relname AS table, last_analyze, last_autoanalyze FROM pg_stat_user_tables WHERE relname IN ('foo_table')
Running analyze: ANALYZE "foo_table"
SQL: ANALYZE "foo_table"
SQL: EXPLAIN (FORMAT JSON) SELECT * FROM foo_table WHERE customer_id = 3913437
SQL: SELECT table_name, column_name, data_type FROM information_schema.columns WHERE table_schema = 'public' AND table_name IN ('foo_table') ORDER BY 1, 2
SQL: EXPLAIN (FORMAT JSON) SELECT * FROM foo_table WHERE customer_id = 3913437
SQL: EXPLAIN (FORMAT JSON) SELECT * FROM foo_table WHERE customer_id = 3913437
SQL: SELECT schemaname AS schema, t.relname AS table, ix.relname AS name, regexp_replace(pg_get_indexdef(i.indexrelid), '^[^\(]*\((.*)\)$', '\1') AS columns, regexp_replace(pg_get_indexdef(i.indexrelid), '.*  USING ([^ ]*) \(.*', '\1') AS using FROM pg_index i INNER JOIN pg_class t ON t.oid = i.indrelid INNER JOIN pg_class ix ON ix.oid = i.indexrelid LEFT JOIN pg_stat_user_indexes ui ON ui.indexrelid = i.indexrelid WHERE t.relname IN ('foo_table') AND schemaname IS NOT NULL AND indisvalid = 't' AND indexprs IS NULL AND indpred IS NULL ORDER BY 1, 2
No new indexes found
--------------------------------------------------------------------------------
Query 013ce1dc4db78af651cb1584fe349cb3b2a76642f0
Total time: 0.0 min, avg time: 0 ms, calls: 3
Could not run explain

EXPLAIN (FORMAT JSON) SELECT * FROM foo_table WHERE customer_id = 3913437

--------------------------------------------------------------------------------
Query 016ec46c3a2d89603a3372884726d3e7ad9972de78
Total time: 0.0 min, avg time: 0 ms, calls: 1
Tables not present in current database

SELECT schemaname AS schema, t.relname AS table, ix.relname AS name, regexp_replace(pg_get_indexdef(i.indexrelid), '^[^\(]*\((.*)\)$', '\1') AS columns, regexp_replace(pg_get_indexdef(i.indexrelid), '.* USING ([^ ]*) \(.*', '\1') AS using FROM pg_index i INNER JOIN pg_class t ON t.oid = i.indrelid INNER JOIN pg_class ix ON ix.oid = i.indexrelid LEFT JOIN pg_stat_user_indexes ui ON ui.indexrelid = i.indexrelid WHERE t.relname IN ('foo_table') AND schemaname IS NOT NULL AND indisvalid = 't' AND indexprs IS NULL AND indpred IS NULL ORDER BY 1, 2

--------------------------------------------------------------------------------
Query 018f71bfdc5277a2f46491fed0004d46abd13ec6a8
Total time: 0.0 min, avg time: 11 ms, calls: 1
Tables not present in current database

SELECT schemaname AS schema, relname AS table, last_analyze, last_autoanalyze FROM pg_stat_user_tables WHERE relname IN ('foo_table')

--------------------------------------------------------------------------------
Query 0198c564148f82d61db2b4328b18949cbcf6e7dffc
Total time: 0.0 min, avg time: 2 ms, calls: 1
Tables not present in current database

SELECT table_name, column_name, data_type FROM information_schema.columns WHERE table_schema = 'public' AND table_name IN ('foo_table') ORDER BY 1, 2

--------------------------------------------------------------------------------
Query 019d656ba1db4a5104a5ef56261cc2e2341defad87
Total time: 0.2 min, avg time: 10745 ms, calls: 1
Could not run explain

ANALYZE "foo_table"

Processing 0 new query fingerprints
Processing 0 new query fingerprints
Processing 0 new query fingerprints

Hey @jfinzel, can you install master and see if it works as expected?
gem install specific_install
gem specific_install ankane/pgdexter

@ankane as I said, I don't have much of a ruby dev env.  I tried, but failed, running into this issue: https://github.com/rdp/specific_install/issues/21
sudo gem specific_install https://github.com/ankane/dexter.git
/usr/bin/git
git installing from https://github.com/ankane/dexter.git
Cloning into '/tmp/d20171226-19790-sawpds'...
remote: Counting objects: 1142, done.
remote: Compressing objects: 100% (103/103), done.
remote: Total 1142 (delta 84), reused 115 (delta 51), pack-reused 987
Receiving objects: 100% (1142/1142), 153.16 KiB, done.
Resolving deltas: 100% (638/638), done.
ERROR:  While executing gem ... (NoMethodError)
    undefined method `build' for Gem::Package:Module

I'm not too thrilled about figuring this env problem out.  Is it difficult for you to package the ubuntu xenial packages?  I normally download these actual debian package files and install them using dpkg.  If you have any suggestions as to how to resolve my ruby issues I'm all ears.

@jfinzel Package instructions are here: https://github.com/ankane/dexter/blob/master/guides/Linux.md

Beautiful.  Didn't realize that the packages were available already.  Thanks.  I will get back to you.

It appears to be working now!
--------------------------------------------------------------------------------
Query 024c3b0749767ad544e002823b0476bb03d20a5d03
Total time: 0.0 min, avg time: 6 ms, calls: 1
No candidate tables for indexes

SELECT pg_catalog.quote_ident(c.relname) FROM pg_catalog.pg_class c WHERE c.relkind IN ('r', 'S', 'v', 'm', 'f') AND substring(pg_catalog.quote_ident(c.relname),1,6)='pg_sta' AND pg_catalog.pg_table_is_visible(c.oid)        UNION
        SELECT pg_catalog.quote_ident(n.nspname) || '.'  USING ([^ ]*) \(.*', '\1') AS using FROM pg_index i INNER JOIN pg_class t ON t.oid = i.indrelid INNER JOIN pg_class ix ON ix.oid = i.indexrelid LEFT JOIN pg_stat_user_indexes ui ON ui.indexrelid = i.indexrelid WHERE schemaname || '.' || t.relname IN ('foo.bar') AND indisvalid = 't' AND indexprs IS NULL AND indpred IS NULL ORDER BY 1, 2
Index found: foo.bar (customer_id)
--------------------------------------------------------------------------------
Query 0211d4ad4b4b4536ea8873999fa4637b2a5b515384
Total time: 0.0 min, avg time: 1544 ms, calls: 1
Start: 943461.81
Pass1: 47.68 : foo.bar (customer_id)
Pass2: 47.68 : foo.bar (customer_id)
Final: 47.68 : foo.bar (customer_id)

SELECT * FROM foo.bar WHERE customer_id = 3913437;

--------------------------------------------------------------------------------
Query 02f8605773dd3aec31308b7e23ae3eaf0e160f30b6
Total time: 0.0 min, avg time: 1 ms, calls: 1
Tables not present in current database

SELECT * FROM pg_stat_activity WHERE NOT pid = pg_backend_pid();

SQL: SELECT pg_try_advisory_lock(123456)
SQL: SELECT schemaname || '.' || t.relname AS table, ix.relname AS name, regexp_replace(pg_get_indexdef(i.indexrelid), '^[^\(]*\((.*)\)$', '\1') AS columns, regexp_replace(pg_get_indexdef(i.indexrelid), '.*  USING ([^ ]*) \(.*', '\1') AS using FROM pg_index i INNER JOIN pg_class t ON t.oid = i.indrelid INNER JOIN pg_class ix ON ix.oid = i.indexrelid LEFT JOIN pg_stat_user_indexes ui ON ui.indexrelid = i.indexrelid WHERE schemaname || '.' || t.relname IN ('foo.bar') AND indisvalid = 't' AND indexprs IS NULL AND indpred IS NULL ORDER BY 1, 2
No new indexes found
--------------------------------------------------------------------------------
Query 020f4251015b74eb7e58755c04749b6bdc5e6142cb
Total time: 0.0 min, avg time: 0 ms, calls: 3
No tables

SET client_min_messages = warning

--------------------------------------------------------------------------------
Query 022134fffa0ec1ab5ca6ecfaa2e8ac08d32a1e4a75
Total time: 0.0 min, avg time: 3 ms, calls: 4
Tables not present in current database

SELECT schemaname || '.' || t.relname AS table, ix.relname AS name, regexp_replace(pg_get_indexdef(i.indexrelid), '^[^\(]*\((.*)\)$', '\1') AS columns, regexp_replace(pg_get_indexdef(i.indexrelid), '.* USING ([^ ]*) \(.*', '\1') AS using FROM pg_index i INNER JOIN pg_class t ON t.oid = i.indrelid INNER JOIN pg_class ix ON ix.oid = i.indexrelid LEFT JOIN pg_stat_user_indexes ui ON ui.indexrelid = i.indexrelid WHERE schemaname || '.' || t.relname IN ('foo.bar') AND indisvalid = 't' AND indexprs IS NULL AND indpred IS NULL ORDER BY 1, 2

--------------------------------------------------------------------------------
Query 022152944e070157a00864594efd3afa1a5f2df1ae
Total time: 0.0 min, avg time: 1 ms, calls: 2
Low initial cost: 0.01

SELECT pg_try_advisory_lock(123456)

--------------------------------------------------------------------------------
Query 02358dd0a711aa176bc3b2931c37a2cae4422db7fa
Total time: 0.0 min, avg time: 7 ms, calls: 2
Tables not present in current database

SELECT schemaname || '.' || relname AS table, last_analyze, last_autoanalyze FROM pg_stat_user_tables WHERE schemaname || '.' || relname IN ('foo.bar')

--------------------------------------------------------------------------------
Query 02b801bf8304d078f264405606b772f832bc3558d7
Total time: 0.0 min, avg time: 2 ms, calls: 2
Tables not present in current database

SELECT table_schema || '.' || table_name AS table_name, column_name, data_type FROM information_schema.columns WHERE table_schema || '.' || table_name IN ('foo.bar') ORDER BY 1, 2

--------------------------------------------------------------------------------
Query 02bebf4f26540f8da6b2f0b0cb03387f3e1fb3e798
Total time: 0.0 min, avg time: 0 ms, calls: 2
Low initial cost: 10.25

SELECT * FROM hypopg_create_index('CREATE INDEX ON "foo"."bar" ("customer_id")')

--------------------------------------------------------------------------------
Query 02c4515bd877f9816617be5a8e02796ee21c538c13
Total time: 0.0 min, avg time: 0 ms, calls: 6
Could not run explain

EXPLAIN (FORMAT JSON) SELECT * FROM foo.bar WHERE customer_id = 3913437

--------------------------------------------------------------------------------
Query 02ec6d7cfd17893bde837a09c75c0e669494d525b9
Total time: 0.0 min, avg time: 1 ms, calls: 2
Low initial cost: 0.01

SELECT pg_advisory_unlock(123456)

--------------------------------------------------------------------------------
Query 02f4a9be8567d2f25ee51695f7560a2ab7f4ad3fdb
Total time: 0.0 min, avg time: 0 ms, calls: 1
Could not run explain

CREATE INDEX CONCURRENTLY ON "foo"."bar" ("customer_id")

Processing 3 new query fingerprints
SQL: SELECT hypopg_reset()
SQL: SELECT table_schema || '.' || table_name AS table_name FROM information_schema.tables WHERE table_catalog = current_database() AND table_schema NOT IN ('pg_catalog', 'information_schema') AND table_type = 'BASE TABLE'
SQL: SHOW search_path
No new indexes found
--------------------------------------------------------------------------------
Query 023d3fe4c2977c63c45b14e2c3fe2ed43a8f96c2c7
Total time: 0.0 min, avg time: 0 ms, calls: 2
No candidate tables for indexes

EXPLAIN (FORMAT JSON) SELECT pg_advisory_unlock(123456)

--------------------------------------------------------------------------------
Query 02d09b3a0e0186e30996ec50ee44d834b1f497ee62
Total time: 0.0 min, avg time: 0 ms, calls: 2
No candidate tables for indexes

EXPLAIN (FORMAT JSON) SELECT pg_try_advisory_lock(123456)

--------------------------------------------------------------------------------
Query 02ed6f30b114223a58064463658319d8bebd3ebe5d
Total time: 0.0 min, avg time: 0 ms, calls: 2
No candidate tables for indexes

EXPLAIN (FORMAT JSON) SELECT * FROM hypopg_create_index('CREATE INDEX ON "foo"."bar" ("customer_id")')

Processing 0 new query fingerprints

Great, going to close this out. Thanks again for all the great feedback (across all the issues you opened), and let me know if anything else comes up as you're using it. 💯


Remove checking minimum version of Boost and OpenSSL at runtime

Summary:
As I was in the process of correctly specifying the correct Boost version minimum from 1.57 to 1.67 in CheckLibraryVersions.cpp, @mellery451 nicely pointed out that having these runtime checks at all doesn't make much sense since we check the minimum Boost and OpenSSL versions at CMake configure time. Having the check at build time is much better than runtime.
This PR is split into two commits as such:

Specify correct Boost minimum version (1.67 instead of 1.57)
Since we require a min Boost version of 1.67 as of recently (for Beast), we also remove the conditional
checks that existed for us to know whether Boost.Process is available or not. We can always
assume it is available now.

Remove CheckLibraryVersions entirely and its associated unit tests.

Requested reviewers:
@wilsonianb @ximinez

Thank you for your submission. It will be reviewed soon and submitted for processing in CI.

Thank you for your submission. It will be reviewed soon and submitted for processing in CI.

@mellery451 any idea why Travis build failed with error "E: Unable to locate package gcc-5"? Is it possible to retry the Travis job?

I'm guessing some transient condition..I restarted it.

Codecov Report

Merging #2583 into develop will decrease coverage by <.01%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##           develop    #2583      +/-   ##
===========================================
- Coverage    70.84%   70.83%   -0.01%
===========================================
  Files          703      703
  Lines        54685    54685
===========================================
- Hits         38739    38735       -4
- Misses       15946    15950       +4

Impacted Files
Coverage Δ

src/ripple/app/main/Main.cpp
45.42% <ø> (ø)
:arrow_up:

src/ripple/basics/impl/CheckLibraryVersions.cpp
93.18% <100%> (ø)
:arrow_up:

src/ripple/basics/DecayingSample.h
77.77% <0%> (-8.34%)
:arrow_down:

src/ripple/beast/core/SemanticVersion.cpp
94.07% <0%> (-0.66%)
:arrow_down:

Continue to review full report at Codecov.

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

LGTM 👍

Thank you for your submission. It will be reviewed soon and submitted for processing in CI.

@mellery451 great point.  I went ahead and removed the file and its unit tests in 176897fbe09adffec3a688ed02de856383187ed6.
I'm happy to reword the commits, squash, or such.  I intentionally didn't mark the second one as "FOLD" as they are slightly separate changes (due to removing Boost Process checks), but it's not a big change, so it can probably all be squashed into one. Better to keep them decoupled until someone tells me otherwise, though.

in 06d0ff6e5


[Feature] Clear notification with right click

Is your feature request related to a problem? Please describe.
No.
Describe the solution you'd like
When there is a notification popup right clicking will clear it.
Describe alternatives you've considered
There's already a clear button on notifications, so just triggering the clear on a right click anywhere should be easy enough to implement.
Additional context
N/A

@Jas-SinghFSU should this be default behaviour?


[TASK] Add failing tests for non-removed class node

@jaapio could you have a look maybe? It seems like even though the  ClassNodeTransformer returns null the node does not get properly removed somewhere in the shadow trees. And I dont really get what the problem is.

Thanks for reporting, this is fixed now.


[@algolia/ui-components-horizontal-slider-js] horizontalSlider not working with recommend-js in UMD

Description
When using ui-components-horizontal-slider-js with recommend-js and both libraries are loaded in UMD, they both import their own bundled version of Preact. Even if the version is the same, the fact that each function uses a different instance of Preact breaks the app.
Reproduction
Preview →
Steps

Visit the sandbox
See the runtime error

Expected behavior
The app should run normally.
Suggested fix
If recommend-js was providing createElement and Fragment in view and that ui-components-horizontal-slider-js accepted a renderer (like autocomplete-js) we could bind the same instance of Preact, which should work.
Environment

OS: macOS Monterey 12.3.1
Browser: Firefox 99.0.1
Library version: @algolia/ui-components-horizontal-slider-js 1.0.0 / @algolia/recommend-js 1.3.0

Hello !
Did any body came up with a solution ? I'm experiencing the same behavior with recommend-js, the documentation indicates :

The horizontalSlider UI component is currently not supported with recommend-js when imported as a UMD bundle
https://www.algolia.com/doc/ui-libraries/recommend/api-reference/recommend-js/relatedProducts/#param-view

I couldn't find a workaround for now.
Environment
OS: Windows 11
Browser: Chrome 121.0.6167.162
Library version<EMAIL_ADDRESS>

The workaround for now is using a bundler or similar instead of UMD. Sorry!

No problem, I eventually managed to display my algolia recommendations in a Swiper.js slider.
Thanks for your answer !


Draft: First tab

Diff:

[x] init solid, solidjs/router, styled-components, three-js, prettier, eslint
[x] init project structure
[x] add header component
[x] add slider component
[x] add file-input, email-input, hash-input, button components
[ ] make request to backend to start training
[ ] validate form fields

Please, resolve conversations when you make changes or just write 'Done', 'Changed', etc.

We have to redirect from /3d-scene-editing-app to /3d-scene-editing-app/train instead of /train

Read how to write self-explainable commit messages. Resources: 1, 2

OAuth
Download files
Scope: https://www.googleapis.com/auth/drive.readonly
Google Picker API


feat: semantic screen utilities range

https://github.com/windicss/windicss/discussions/28#discussioncomment-392853
<sm => smaller than this breakpoint
>sm => greater than this breakpoint
@sm => exactly this breakpoint

Codecov Report

Merging #203 (c7ee9a2) into main (f0aac12) will increase coverage by 0.03%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #203      +/-   ##
==========================================
+ Coverage   86.64%   86.67%   +0.03%
==========================================
  Files          45       45
  Lines        2584     2590       +6
  Branches      682      682
==========================================
+ Hits         2239     2245       +6
  Misses        145      145
  Partials      200      200

Impacted Files
Coverage Δ

src/lib/variants/screen.ts
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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

> and < behave then like + and - ?
shouldn't this be more like this?
<sm => @media(max-width: 639.9999px)
>sm => @media(min-width: 640.0001px)
@sm => @media(min-width: 640px) and (max-width: 767.9999px) // makes only sense when using mobile first

Yeah, I think it's more like a semantic change as - + could be confusing without reading the docs and < > could be more intuitive. Can you explain a bit why the 0.0001 delta is needed? Thanks

Does this mean that you must write +sm for the min width? We should promote mobile-first, and the max and only breakpoints are overrides. I like @sm for between breakpoint.

> and < behave then like + and - ?
shouldn't this be more like this?

<sm => @media(max-width: 639.9999px)

>sm => @media(min-width: 640.0001px)

@sm => @media(min-width: 640px) and (max-width: 767.9999px) // makes only sense when using mobile first

This is correct. Min and max breakpoint can not have the same value or there will be bugs att that exact breakpoint value.

Can you explain a bit why the 0.0001 delta is needed? Thanks

min-height and max-height is always inclusive.
so min-height: 640px is like >= 640px
and max-height: 640px is like <= 640px
when using <sm: i would expect that the variant is smaller than 640px (and not equal)

Does this mean that you must write +sm for the min width? We should promote mobile-first, and the max and only breakpoints are overrides. I like @sm for between breakpoint.

Nope. The bare sm is mobile-first just like Tailwind.

Changed the range to this:
<sm => @media(max-width: 639.9px)
>sm => @media(min-width: 640.1px)
@sm => @media(min-width: 640px) and (max-width: 767.9px)


TSpeedButton Glyph support?

Am I missing something, or is a TSpeedButton's glpyh not supported by skia?
I installed the lateste 5.0.0 release using the installer on my Windows developer computer, enabled skia in the project's properties and I can load SVG images into a TImage. I can also specify the SVG file in a TSpeedButton's glyph, and it gets displayed in the image editor used to loead the glyph. But when I save the glyph, I get an exception in the IDE telling me that th graphic format is wrong.
Thanks
Stefan

@huste As I understand, the vcl glyph only support bmp, or am I wrong? Do any other SVG library add this support?