Typo "GitHub Action workflows"→"GitHub Actions workflows"

Summary
Describe your changes here.
Typo "GitHub Action workflows"→"GitHub Actions workflows"
https://docs.microsoft.com/en-us/dotnet/devops/create-dotnet-github-action
https://github.com/dotnet/docs/blob/main/docs/devops/create-dotnet-github-action.md
#PingMSFTDocs
Fixes #Issue_Number (if available)

MerhabaBurada çok yeniyim bu yuzd3n çok karısık 06:23, 11 Temmuz 2022, Hiroshi Yoshioka @.***>:Summary
Describe your changes here.
Typo "GitHub Action workflows"→"GitHub Actions workflows"
https://docs.microsoft.com/en-us/dotnet/devops/create-dotnet-github-action
https://github.com/dotnet/docs/blob/main/docs/devops/create-dotnet-github-action.md
#PingMSFTDocs
Fixes #Issue_Number (if available)
You can view, comment on, or merge this pull request online at:
  https://github.com/dotnet/docs/pull/30154
Commit Summary
92cadba  Typo "GitHub Action workflows"→"GitHub Actions workflows"
File Changes  (1 file)
M
docs/devops/create-dotnet-github-action.md
(10)

Patch Links:
https://github.com/dotnet/docs/pull/30154.patch
https://github.com/dotnet/docs/pull/30154.diff
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you are subscribed to this thread.Message ID: @.***>
-- Yandex Mail mobil uygulamasından gönderildi


Better describing object settings config

Thank you for the update @jdeoliveira ! Do you want to make any further changes in this PR, or is it ready to merge?

I'm merging now so this can be published today. If there are any further updates, I'll be happy to review and merge new PRs.


.editorconfig file

Adding editorconfig file. Makes it easier for developers to get their editor to follow your rules if they use some kind of editorconfig plugin.

Wow, this was all wrong


Post author flair disappears from post after accessing post.comments

Issue Description
As also reported by other user on reddit.
When accessing post.comment on a post from subreddit.new() (for example by doing post.comments.replace_more() or post.comments.list()) the post.author_flair_* properties are cleared
Example code (reddit is a Reddit instance from apraw.Reddit()):
# Get post with flair set
post = [post for post in reddit.subreddit("mechmarket").new() if post.author_flair_css_class is not None][0]
print(post.author_flair_css_class)  # Prints i-[value]
post.comments.list()  # Get list of comments, result not important
print(post.author_flair_css_class)  # Prints None

System Information
PRAW Version: 5.0.1 (also reported on 5.4.0)
Python Version: 2.7.12
Operating System: Ubuntu 16.04

For the record: I just upgraded PRAW to 5.4.0 and I can still reproduce the bug.

So I did some sleuthing and it seems like you only get the author_flair_* properties from the API if you explicitly add /r/[subreddit]/ to the API endpoint.
When accessing submission.comments the first time PRAW updates the submission __dict__ with the information gained from the new API call, which is done without the /r/[subreddit]/ part which clears all author_flair_* properties as they are all None in response from the new call.

Yeah, it feels really weird to get different response depending on the path. I'm pretty sure this is a new thing too as things started breaking just last week...
Thank you for the fast response and fix.

The upstream bug report is here: https://www.reddit.com/r/bugs/comments/8g265d/api_submission_api_returns_different_results/

This has now escalated to also include author flair properties on comments.
I've replied to bsimpson on the bug report but thought I'd also update you here as this fix just got a little more relevant if reddit won't/can't fix their issues soon enough.


Enable additional CI jobs

VS 2017 is no longer available on GitHub runners. We need to figure out either how to install it, or add it to our self-hosted runners.

VS 2017 is no longer available on GitHub runners. We need to figure out either how to install it, or add it to our self-hosted runners.

... or let us all switch to VS 2022 and stop worrying about VS 2017. For how long do we need to keep supporting VS 2017?


SMTPMailServiceHealthCheck can easily "run-away" and use up e-mail quotas

Required Information

[X] AEM Version, including Service Packs, Cumulative Fix Packs, etc: 6.5.5
[X] ACS AEM Commons Version: 4.7.0
[X] Reproducible on Latest? yes

Actual Behavior
It has been reported from several users, and observed by myself (via access to commonly used, shared SMTP accounts) that the SMTPMailServiceHealthCheck can be easily misconfigured and "run-away" sending thousands of emails over the course of a day.
It would be nice to add extra safeguards to prevent this from occurring.

Do not send an email if the default email address is used (as no one can actually verify if this email is, in fact, receiving anything vs. going into a black hole)
Put a max limit per day, with a default of 24 (eg. 1 per hour)

That means we should introduce a rate-limit (e.g. 1 email per hour) and ignore all subsequent invocations until at least an hour has passed.
I don't think that we need to be too precise, so we keep the statistics in-memory and if the bundle/systems restarts within the hour, you start again with the full limit. Everything else makes it too complicated.
@davidjgonzalez  would that suffice?

Ok, looks like this has already been fixed. Closing.


How to check a WebSocket handle is live healthily?

How to check a WebSocket handle is live healthily?
Did I do the right thing with the code below?
if ( WebSocket.isOpen() &&
! WebSocket.isClosing() &&
! WebSocket.isFlushAndClose() &&
! WebSocket.isClosed() )
{
//	indicates that this socket connection is healthy
}

Each websocket can be in a single state (all possible states are listed here: https://www.javadoc.io/doc/org.java-websocket/Java-WebSocket/latest/org/java_websocket/enums/ReadyState.html).
Just use isOpen(), all other checks are unnecessary.

Question answered. Closing issue!


Windows 7

Does this work with Win 7?
Thanks.

I have not tested with Windows 7. It will probably run as .NET 6 is currently supported on Windows 7 according to this.
Please try the latest artifact in Actions and let me know if any problems.


Example not working correctly

Describe the issue as clearly as possible:
I think the example code is broken in syntax, and also the results are not correctly working.
Steps/code to reproduce the bug:
import outlines

model = outlines.models.transformers("mistralai/Mistral-7B-v0.1")

prompt = "1+1="
answer = outlines.generate.format(model, int)(prompt)

prompt = "sqrt(2)="

generator = outlines.generate.format(model, float)
answer = generator(prompt)

print(answer)
print(type(answer))

Expected result:
1.414
float

Error message:
1.414
str

Outlines/Python version information:
0.0.25.dev15+g0cd9608
Python 3.10.13 | packaged by conda-forge | (main, Dec 23 2023, 15:36:39) [GCC 12.3.0]
Context for the issue:
Firstly, I think the example code is wrong:
prompt = "1+1="
answer = outlines.generate.format(model, int)(prompt)

prompt = "sqrt(2)="

generator = outlines.generate.format(model, float)
answer = generator(prompt)

I assume it should be:
prompt = "1+1="
generator = outlines.generate.format(model, int)(
answer = generator(prompt)

prompt = "sqrt(2)="

generator = outlines.generate.format(model, float)
answer = generator(prompt)

Secondly, for both inferences, 'answer' is a string, not an int or float.
Lastly, I also tried the "Efficient JSON generation following a Pydantic model" example, and was returned a python object. Should this directly be a json string? it's not clear.

I agree, it does seem reasonable for the format function to return the object itself. Of course, we should avoid using eval(), so there's a need for a function that converts strings to objects, in line with the definitions provided in the module below that defines each types regex:
https://github.com/outlines-dev/outlines/blob/main/outlines/fsm/types.py

A case of "The Principle of Least Astonishment" :)

I agree we should change the interface. SequenceGenerator has a method that post-processes the generated string so that would be the way to go about it.
Thank you for pointing this out!


Next release date?

Hello,
I see lots of activity from @XhmikosR in the last month and thanks for it, but the last release was more than 2 years ago.
When do you plan to release a new release date?

There's no ETA because there's still quite a few things to address.
@jkphl friendly ping to have a look at the open PRs so that we move forward.

Don't you wanna release in smaller chunks? I like the smaller releases than the big bums.
And not easy resolve 2 years tech dept.

Nah, at this point I'd rather finish with the outstanding issues before v2.0.

@XhmikosR hello, can you share those issues? I'll check if I can help.

Hey, @Kreeg! I'd say at this point we should choose how to proceed with puppeteer in #447.
@ert78gb has made some significant work, it's just that it feels like having puppeteer in dependencies is too much because it's just used in the case where the dimensions cannot be determined.
That being said, I'd be happy to go with puppeteer and see if we can make it optional later.

@XhmikosR I see no problems with going with the puppeteer since phantomjs was also here only for the dimension checking. Since PhantomJS was deprecated I'd say it's more than OK to make puppeteer optional later and go straight with the working solution now :)

The thing is that puppeteer is huge. It just doesn't make sense to have it by default :/
Also, I'm just one person with limited time. If you have time, happy to give you push rights to help with the PRs and further tweaks.

The definition of "having time" may differ, but since my actual work depends on this project I'll do all my best to help with it.
So, yes. I have time. :)

Going to close this since we have already released v2.0.0-beta2. After we finish with tests and covering all modes, we can release v2.0.0 stable.


Build libusb 1.0.23 for Windows binaries

libusb 1.0.23 is needed for Windows 10 to work, but pre-build binaries are not provided for it, so we need to build our own deterministically.
Fixes #259

For testing: hwi-1.0.2-windows-amd64.zip

enumerate tests on Windows 10:
Works

Trezor T
Trezor One
Ledger Nano S
Coldcard
BitBox 1.0

Problematic

KeepKey

I have never tried to use this KeepKey so I am not sure it's a HWI issue, but I plugged in an uninitialized KeepKey and HWI refuses to respond until I plug it out. When I do that it responds with a trezor type with no model instead:
C:\Users\user\Downloads\hwi-1.0.2-windows-amd64>hwi.exe enumerate
[{"type": "trezor", "path": "hid:\\\\?\\hid#vid_2b24&pid_0001#6&c739c8b&0&0000#{4d1e55b2-f16f-11cf-88cb-001111000030}", "error": "Could not open client or get fingerprint information: read error", "code": -13}]

Console Log
C:\Users\user\Downloads\hwi-1.0.2-windows-amd64>hwi.exe enumerate
[]

C:\Users\user\Downloads\hwi-1.0.2-windows-amd64>hwi.exe enumerate
[{"type": "trezor", "path": "webusb:001:4", "model": "trezor_t", "needs_pin_sent": false, "needs_passphrase_sent": false, "error": "Not initialized", "code": -18}]

C:\Users\user\Downloads\hwi-1.0.2-windows-amd64>hwi.exe enumerate
[{"type": "trezor", "path": "hid:\\\\?\\hid#vid_534c&pid_0001&mi_00#7&a5e9b90&0&0000#{4d1e55b2-f16f-11cf-88cb-001111000030}", "model": "trezor_1", "needs_pin_sent": false, "needs_passphrase_sent": true, "error": "Could not open client or get fingerprint information: Passphrase needs to be specified before the fingerprint information can be retrieved", "code": -12}]

C:\Users\user\Downloads\hwi-1.0.2-windows-amd64>hwi.exe enumerate
[{"type": "coldcard", "model": "coldcard", "path": "\\\\?\\hid#vid_d13e&pid_cc10&mi_00#7&38f11f5a&0&0000#{4d1e55b2-f16f-11cf-88cb-001111000030}", "needs_passphrase": false, "fingerprint": "f143e112"}]

C:\Users\user\Downloads\hwi-1.0.2-windows-amd64>hwi.exe enumerate
[{"type": "ledger", "model": "ledger_nano_s", "path": "\\\\?\\hid#vid_2c97&pid_0001&mi_00#7&e45ae20&0&0000#{4d1e55b2-f16f-11cf-88cb-001111000030}", "fingerprint": "4054d6f6", "needs_pin_sent": false, "needs_passphrase_sent": false}]

C:\Users\user\Downloads\hwi-1.0.2-windows-amd64>hwi.exe enumerate
[{"type": "digitalbitbox", "model": "digitalbitbox_01", "path": "\\\\?\\hid#vid_03eb&pid_2402&mi_00#7&149acf39&0&0000#{4d1e55b2-f16f-11cf-88cb-001111000030}", "error": "Could not open client or get fingerprint information: Password must be supplied for digital BitBox", "code": -6}]

C:\Users\user\Downloads\hwi-1.0.2-windows-amd64>hwi.exe enumerate
[{"type": "trezor", "path": "hid:\\\\?\\hid#vid_2b24&pid_0001#6&c739c8b&0&0000#{4d1e55b2-f16f-11cf-88cb-001111000030}", "error": "Could not open client or get fingerprint information: read error", "code": -13}]

Also not that I am leaving the country so I won't have access to the devices anytime soon, so I won't be able to reproduce this test :(

I have never tried to use this KeepKey so I am not sure it's a HWI issue, but I plugged in an uninitialized KeepKey and HWI refuses to respond until I plug it out. When I do that it responds with a trezor type with no model instead:

The problem is probably due to not being initialized. The Keepkey implementation just reuses the Trezor implementation (since they're basically the same, just the Trezor library was better) with some things changed to Keepkey for display purposes. The implemented way to differentiate is to send a specific message and check the response.
In general, there have been some issue with Keepkey because of this, and I'll look into it.


Opera detection is returning chrome

Opera browsers are returning as chrome (tested in OSX, multiple opera versions, windows tested with latest opera), it looks like you are parsing the UA to find the word 'opera', which is not in the opera UA, however the word 'chrome' actually is in the user agent so it returns chrome. As far as I can tell the only unique identifier to opera is the string 'opr/26.0.1656.60' (replace the numbers with the version) and really old versions contain the string 'Presto/'
The regex that works for me is /OPR/\d{2}.\d{1}.\d{4}.\d{2}|Presto// however it would be nice if we didn't have to have a separate rule from the Detectizr stuff.

Is this ever going to be fixed?!

@lepinejf It doesn't seem like the library is maintained anymore, I recommend opening a pull request with a fix and try and breath new life into the project or just fork it like I did and apply your own fixes

@merrickfox I don't know, I've seen commits that are 2 month old. Pretty sure it's still maintained, just very slowly.


fix(dashboard): removed functions section

Codecov Report

Merging #1548 (941f0f5) into main (f34702f) will not change coverage.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@           Coverage Diff           @@
##             main    #1548   +/-   ##
=======================================
  Coverage   85.97%   85.97%
=======================================
  Files          83       83
  Lines        9236     9236
  Branches      516      516
=======================================
  Hits         7941     7941
  Misses       1295     1295

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Editor: Read-Only mode doesn't allow keyboard selection of text

From<EMAIL_ADDRESS>on 2010-09-27T15:19:55Z
What steps will reproduce the problem?

Open a file that is read-only.
The keyboard can no longer be used to select text, it scrolls instead. What is the expected output? What do you see instead? I still want to be able to select text with the keyboard as usual.  I'd like to see this as an option.  I'm okay leaving the buffer read-only so no text can be changed. What version of the product are you using? On what operating system? Spyder 2.0.0beta3, Windows XP, python 2.6.1.

Please provide any additional information below
.
Original issue: http://code.google.com/p/spyderlib/issues/detail?id=393

From pierre.raybaut on 2010-10-26T04:53:15Z
Unfortunately, the problem is that it seems that's the way Qt deals with read-only editors (QPlainTextEdit objects): http://bugreports.qt.nokia.com/browse/QTBUG-3799 But there may be a way: I have to test textInteractionFlags attribute to see if has something to do with it.
Status: Accepted
Labels: Milestone-Release2.0

From pierre.raybaut on 2010-10-28T02:27:52Z
That's what I thought: I can't do anything about it easily (I could implement my own read-only mode, ignoring text insertion or any other change but it would certainly be a continuous source of bugs...).
However note that even if the cursor is not visible, you can select things by holding Ctrl and Shift.
Status: WontFix


Bug: GenAI script making changes to every git repo opened (even if not used)

After installing the GenAIScript VSCode extension every code repo I open in VSCode has two new files created:
 create mode 100644 .genaiscript/.gitattributes
 create mode 100644 .genaiscript/.gitignore

This occurs even if GenAIScript was not used in the repo or even the VSCode session.
This is an issue as users will not want to make changes to every git repo they contribute to for one extension they happen to have installed.

Yes that's not the intended behavior. It should only do this when finding .genai.mjs files.

Thanks :)


[proxima-beam] [proxima-direct] disable rate limiting before closing reader

Closes O2-Czech-Republic#292


overlapblock Creator - Web design #3

https://images01.nicepagecdn.com/page/42/39/html-template-42393.jpg Design a overlapblock like the above image
Create a file "overlapblock_NUM_by_(YourName).html" in the modules/overlapblock folder.
Use simple HTML and CSS and JAVASCRIPT to create the overlapblock animation. You can use the interface development https://kwickerhub.com/ (This project) or you can write the lines of code yourself.
Step by Step Guide.
Fork this project(Use the 'fork' button in the top right corner) and Clone your Fork.
git clone https://github.com/YOUR_USERNAME/frontend
Open your code Editor and Create a file "overlapblock_NUM_YOUR_NAME_.html" in the "modules/overlapblock" folder of this project you just cloned. No need to create head, title and body tags, Just Add a div tag with some embedded style(i.e use the style tag) and the script tag where necessary.
If you want to add an image resource, please add it in the folder ••
"modules/overlapblock/images_and_icons"
We recommend you use an svg for your image/icon.
Push your Code: You need to push your recent changes back to the cloud. Use the command below in the main directory of this Repository
git push origin dev
or use a GUI tool to avoid mistakes or complexity. LOL.
Then make you Pull Request...
Below is a example of the code structure we need:
'''

'''
For more examples:
modules->buttons
modules->cards
for examples of contributions or Pull request accepted in to the project.
And please do not wait for this issue to be assigned to you as we have limited hands but have a lot to do/cover. Please send in your PR.
Good-luck.

Hi, I am working on this PR. I will merge the changes as soon as possible.

Hi @NtemKenyor, I have finally completed this issue and pushed the PR-https://github.com/KwickerHub/WebCraftifyAI/pull/833. Can you please review and merge it.
Thanks,
Jagathi


Project Catalogue Submission: Test jacob

A new project has been submitted to the project catalogue!

Publication DOI/Project URL: Test jacob,
Contact name: Jacob Windsor
Contact email<EMAIL_ADDRESS>Comments: some comments

Was just a test.


1004 redesign contributor profiles

https://user-images.githubusercontent.com/1512805/134863203-948463ce-7783-4287-9061-7af253e0c023.mov

@tillprochaska Where should we display an alternative email address for a user? From my UE: Stammdaten bearbeiten >  E-Mail-Adresse (and delete PLZ). Your thoughts?

@tillprochaska Where should we display an alternative email address for a user? From my UE: Stammdaten bearbeiten > E-Mail-Adresse (and delete PLZ). Your thoughts?
I think there are two use cases for alternative email addresses:

A contributor uses multiple email addresses, e.g. a personal email address and a work email address or email aliases, and replies from both of them. Or they have email aliases (e.g<EMAIL_ADDRESS>vs. max.mustermann@googlemail.com).

A contributor uses Signal, but we want them to receive requests via email as well.

Regarding 1: I think I’d prefer to add an option for this in the email section -- that way, everything related to the communication channel would be in one place.

Regarding 2: As this use case is a workaround until we have a better understanding of how reliable Signal delivers our messages, I think it would be a good solution to add the email address via the admin panel. The contributor profile would then display something like this:

@drjakob Those aren’t really two different options, but different use cases…

"Ausgewählter Kanal" anstelle von "Kommunikation via"


feat(compaction): do consolidation for deprecated SSTs

What's changed and what's your intention?
do consolidation for deprecated SSTs
Checklist
Refer to a related PR or issue link (optional)

I'll write a new picker after other parts of this PR is refined.

I'll write a new selector after other parts of this PR is refined.

done @zwang28

Any updates? Shall we close this?

Any updates? Shall we close this?

closed


Problem connecting to CockroachDB....

Hi,

DBeaver <IP_ADDRESS>808130751 Community Edition

On my Windows 10 system I followed the instructions to spin up a 3 node cluster in secure mode (using their SSL generation option not OpenSSL) https://www.cockroachlabs.com/docs/stable/secure-a-cluster.html which ultimately means I am looking to use the following connection string:
postgresql://root@localhost:26257?sslcert=certs%5Cclient.root.crt&sslkey=certs%5Cclient.root.key&sslmode=verify-full&sslrootcert=certs%5Cca.crt

However after configuring a PostgreSQL connection as I think they should be I get:

Any suggestions on how to get this working?

Will check this in #4140

Try to install SSHJ extension, it will setup advanced security libraries.
However I can't initiate SSL connection to Cockroach.

I've configured certificates:

Set default connection settings:

When I connect I see error The server requested password-based authentication, but no password was provided.
Just our of curiosity I've tried to set any password and now I see error ERROR: user root must use certificate authentication instead of password authentication.
So I stuck here. Once I enable SSL it complains that no password specified. Once I specify password itr says that I can't use password in secure mode.

The original error reported (Could not read SSL key file ...) is due to the fact that JDBC wants PKCS#8 keys. You can convert DER-encoded PEM keys using:
openssl pkcs8 -topk8 -inform PEM -outform DER -in client.root.key -out client.root.pk8 -nocrypt

Having had this same issue I can confirm that creating the .pk8 key file and connecting as above but with that instead of the .key file does indeed work.


Improve Test coverage

Tests for the Runner class in particular.

Currently up to 70% - CLI needs testing, but this will be done as part of figuring out the chaining for build-lambda/upload/deploy.


Black screen when ExSlideMenuController is open after a main screen

Hi, I have a primary screen in my app that contains a simple button.
When this button is clicked, I create a ExSlideMenuController and presents it.
But it takes about 40 seconds in black screen until this view controller appear.
Can you help me please?
Here is my button click source code:
`@IBAction func btnAction_Click(_ sender: UIButton) {
let storyboard = UIStoryboard(name: "Main", bundle: nil)
            let dashboardViewController = storyboard.instantiateViewController(withIdentifier: "dashboardController") as! DashboardController
            let leftViewController = storyboard.instantiateViewController(withIdentifier: "leftViewController") as! LeftViewController
            let rightViewController = storyboard.instantiateViewController(withIdentifier: "rightViewController") as! RightViewController

            let nvc: UINavigationController = UINavigationController(rootViewController: dashboardViewController)

            UINavigationBar.appearance().tintColor = UIColor(hex: "689F38")

            leftViewController.mainViewController = nvc

            let slideMenuController = ExSlideMenuController(mainViewController:nvc, leftMenuViewController: leftViewController, rightMenuViewController: rightViewController)
            slideMenuController.automaticallyAdjustsScrollViewInsets = true
            slideMenuController.delegate = dashboardViewController

           self.present(slideMenuController, animated: true, completion: nil)

}`

Sorry, this is not a issue.
I was calling let task = session.dataTask with URLSession.shared and it was my problem to update the screen. For imediatly update session.dataTask must be called in the main thread (UI thread) like this:
let sessionConfig = URLSessionConfiguration.default

                let session = URLSession(configuration: sessionConfig, delegate: nil, delegateQueue: OperationQueue.main)

                let task = session.dataTask...


Here is the log
latest.log

https://github.com/Electroblob77/Wizardry/issues/402

Looks like it was fixed on their end


Some input types missing ability to add description to Composition::Option

Looks like the library allows Composition::Option objects to be created with a description parameter (ref), but several input types are missing the ability to do so:

StaticSelect
MultiStaticSelect
RadioButtons

While some others include it:

Checkboxes

Happy to take care of this just wanted to put an issue up in case there was more context.

Hey @gabichuelas -- I think this is just an oversight!

Closed via #195 -- Thanks @resputin 🚀
Shipped in v0.25.0


Add cf_update

use mount file instead of env (dotenv) to store secret
can merge gcp codebase and define entry point when deploying functions


Upgrade sacura to include output improvements

Proposed Changes

Upgrade sacura to include output improvements

/test pull-knative-sandbox-eventing-kafka-broker-integration-tests

/test pull-knative-sandbox-eventing-kafka-broker-integration-tests

/lgtm


Move abstract functional tests to src projects

fixes #5055
Move abstract classes for function tests to src folder
Package name changes for provider-writers
MS.EFCore.FunctionalTests -> MS.EFCore.Specification.Tests
MS.EFCore.Relational.FunctionalTests -> MS.EFCore.Relational.Specification.Tests
MS.EFCore.Relational.Design.FunctionalTests -> MS.EFCore.Relational.Design.Specification.Tests
Above packages are class libraries. They contain only abstract test bases & fixtures which providers can use to test by plugging in their implementation. They will be published on nuget.org.
Other changes:
Provider specific tests are derived from *.Specification.Tests directly.
EFCore.FunctionalTests contain only runnable tests. Other 2 project did not have any concrete test classes and hence removed.
Class level skip conditions are moved to derived classes instead of testbase so that if tests are running for other providers then they can verify them.
build.cmd runs successfully.
csproj based solution builds & runs all tests successfully.
xproj based solution builds fine, fails to discover any tests in my system.
Haven't incorporated namespace changes according to project name changes yet. (my resharper crashed doing so). Are namespaces supposed to be changed? @divega , @natemcmaster

@natemcmaster - travis build failing due to "integrated security" though there is nothing in this PR which is changing connection strings. Any insights?

@smitpatel Hoping that other than the package name change this will not impact how I use the test libraries...

@ErikEJ - It shouldn't impact the way tests are used.
The changes I had to make to SqlServer.FunctionalTests are below

Package name change
Adding dotnet-test-xunit to project.json which was earlier coming from Core.FunctionalTests transitively.
MonoVersionConditions on tests which fails on lower version of Mono. (depends on provider if their implementation can work on Mono below 4.2.0 or not)
Minor class rename to UpdatesRelationalTest to UpdatesRelationalTestBase
If you still face issue while moving to new package names, I would be happy to assist you.

Don't forget to update nugetpackageverifier.json

cc @muratg heads up: we are adding new packages.

travis build failing due to "integrated security" though there is nothing in this PR which is changing connection strings.

@smitpatel we need to take a closer look at the xunit test utilities. It appears to me the test conditions are not actually being evaluated correctly so the tests are running on Linux when they shouldn't.

@smitpatel Got it - you are just making these test libraries an official part of the "product", and as such, they belong in src folder, not in test

you are just making these test libraries an official part of the "product"

Correct. We don't want to ship packages that have a dependency on the coreclr test runner, and we also want to be consistent with how other aspnet packages ship their test specs. For example: https://github.com/aspnet/DependencyInjection/tree/dev/src/Microsoft.Extensions.DependencyInjection.Specification.Tests

@natemcmaster Please make sure to update packages/packages.csv in Coherence repo as well as the nuget package verifier JSON file.
cc @pranavkm heads up

If the packages are being moved to src, we should move these to the AdxShipping category in NuGetPackageVerifier.json

Updated nugetpackageverifier.json
Submitted PR to coherence & coherence-signed

LGTM. Assuming we get the Linux tests passing, I don't see anything less that needs to change. (Not a :shipit: untill we get Linux passing and ask-mode permission.)

travis passed 😌


Enable Setup/Update of OIDC Auth for Arkime Portal

Description
As part of enabling OIDC auth (https://github.com/arkime/aws-aio/issues/75), we want to enable users to setup OIDC Auth on the Arkime Portal. To accomplish this, we will need to support one or more new CLI commands to allow them to pass in the necessary configuration and pipe it through to the containers.
This task assumes the completion of https://github.com/arkime/aws-aio/issues/76
Acceptance Criteria

User is able to perform initial configuration of OIDC Auth
User is able to update their initial OIDC Auth configuration
Demo'd with a real OIDC provider

Updating task description based on design discussion in this RFC (https://github.com/arkime/aws-aio/issues/81).  We no longer want separate commands for this.

Implemented in this PR [1]; resolving
[1] https://github.com/arkime/aws-aio/pull/117


AudioPlayer is skipping through playlist on error response

I'm currently investigating the issue that the player is skipping through the playlist when it reaches the maximum retries per item (because of a backend error). I'd like to have it just stopping/pausing at the current item with the error state and have the user deciding to try the next item in the queue.
I could patch it myself but I'm wondering what the best approach would be. Maybe I'm also missing something? Hope you can help out here. I really like the implementation and API, good work!

Yup, I see your point and I totally agree that the player should not decide how to behave on error.
I didn't think of that when developing that feature.
Anyway, 4e1d49d3ad4835560dc2b53488d2a40c54e9d971 fixed this and I'm going to tag this release as 0.8.0 as it's a change that affects everyone, even if it's for the best.

Just so you know, you should tell how the player to behave when the delegate to alert status changed is called with toState being .Failed.

Nice. I like your solution. Will give it a try later. Thanks a lot, much appreciated!

I just tried it but it still skips to the next one.
I don't do anything in the didChangeStateFrom delegate on .MaximumRetryCountHit as I just want it to stay in this state.
When I put in an audioPlayer.pause() it continues to call .MaximumRetryCountHit for the same item. Also the app get's stalled during the retry but that's probably a different issue.
Any idea?

That's pretty weird but maybe I don't understand your issue.
I tried setting a wrong URL to 5 items that I queued up (title are "0", "2", "3", "4" & "4"):
Queue (item.title): ["0", "1", "2", "3", "4"]
State: Buffering
WillStartPlaying: Optional("0")
Failed(KDEAudioPlayer.AudioPlayerError.FoundationError(Optional(Error Domain=NSURLErrorDomain Code=-1100 "The requested URL was not found on this server." UserInfo={NSLocalizedDescription=The requested URL was not found on this server., NSUnderlyingError=0x7fadab4f4920 {Error Domain=NSOSStatusErrorDomain Code=-12938 "(null)"}})))

That's it, the player waits.

Maybe the difference here is that I break it during playback. I have items queued up and playing and during playback I enable geoblocking which leads to a bad-gateway response.


[feature request] some way to make an iOS virtual machine inside iOS

A way to virtualize iOS in iOS. Maybe a pre-made image or a tutorial... it'll need support for the gpu that iPhones use that QEMU doesn't support but since it's all running on iOS maybe you wouldn't need that much work? It would probably be easier to start with like the iPhone 2G because iOS wasn't as advanced (TSS/SHSH didn't exist, needs less RAM, etc.) and there are definitely tons of exploits for iOS 2.x

Pretty sure QEMU does not emulate iOS.

you can do it, just install mac os 10.5 in utm and then run xcode and run iphone simulator and done. other than that you currently can't.

Pretty sure QEMU does not emulate iOS.

I know that, I literally said that QEMU doesn't support emulating any GPU compatible with iOS but because it's all running under iOS you probably wouldn't need to do much to get the GPU to work

you can do it, just install mac os 10.5 in utm and then run xcode and run iphone simulator and done. other than that you currently can't.

Doesn't the iPhone Simulator not support App Store or really any of iOS's features that aren't needed for the app that you are building to run?

because it's all running under iOS

This doesn't matter much for QEMU; it's running in TCG. You may have luck compiling this but it's nowhere near being able to run the App Store and I doubt anything will change there soon.

https://github.com/devos50/qemu/


iOS issue - dropdown does not trigger for the second time

Doing a mutliselect in iOS devices, the dropdown triggers fine for the first time and does the selection like it is supposed to . But the dropdown fails to trigger for the second time and never triggers there after .
Anybody else facing the same issue ?

Simply open this url "http://valor-software.com/ng2-select/" on any iOS device,

Go to the "Multiple" tab .
Tapping on a country name , the country gets selected and the drop down goes away.
Now try triggering the drop-down by tapping on the same select box to select another country , the drop-down fails to trigger.


feat: force use of libPod if nvidia device requested

What does this PR do?
Requesting an nvidia device through the container toolkit is only supported through the libPod API not the compatibility API. Recognize the request and force libPod in a similar manner to what's done when the container is associated with a pod (it seems like a similar situation in that association with a pod is only available through libPod and not the compatibility API) .
This is the issue I used to ask about how to request an nvidia device available through the container toolkit in podman - https://github.com/containers/podman/issues/24687. From that, my understanding is that it only works through the libPod API and I don't see an interest on the podman side to change that.
See, https://github.com/podman-desktop/podman-desktop/pull/10166 for more context as to one of the use cases where this is needed.
Screenshot / video of UI
N/A
What issues does this PR fix or reference?

How to test this PR?

[X] Tests are covering the bug fix or the new feature

Rebased

@benoitf I'm off on PTO starting Thursday. If you need me to rebase before them just let me know.


[Feature] I hope to support more bitmap functions with the same functions as Clickhouse

Search before asking

[X] I had searched in the issues and found no similar issues.

Description

bitmapSubsetInRange

bitmapSubsetInRange(bitmap, range_start, range_end)
参数：
bitmap – 位图对象.
range_start – 范围起始点（含）.
range_end – 范围结束点（不含）

bitmapSubsetLimit

bitmapSubsetLimit(bitmap, range_start, limit)
参数：
bitmap – 位图对象.
range_start – 范围起始点（含）.
limit – 子位图基数上限

subBitmap
subBitmap(bitmap, offset, limit)
参数：
bitmap – 位图对象.
offset – 跳过多少个元素.
limit – 子位图基数上限

Use case
需求：
我们有一个系统是基于用户画像的推送系统，根据用户画像的标签构建用户群组bitmap，然后根据构建的bitmap导出用户id，
基于用户id去做推送。
遇到的问题：
目前遇到的问题是，当构建的bitmap数据量比较大时（比如达到百万级别的基数），我们无法通过bitmap_to_string函数导出
所有的用户id，所以我们想的办法是通过分页的方式，每次构建子bitmap，然后将子bitmap进行导出
Related issues
#bit
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

I also hope to support more bitmap functions with the same functions as Clickhouse！

I'm hoping so eagerly that there are more support for bitmap functions!

Coming soon

Coming soon

Will it be released in version 0.15？

Yes, maybe in 0.15.x

There are also bitmapmax and bitmapandnot functions

If the bitmap is too large, to_string  does have a problem.
A temporary solution is to use select into file to export the query results to a file.
However, this method is not directly obtained by segmentation. So I very much support the addition of this similar function.

Coming soon

Will it be released in version 0.15？

Yes, maybe in 0.15.x

When was 0.15 officially released? Yesterday's attempt to compile 0.15 failed

My compilation script is as follows:
docker run -it -v /Users/liujinliang/.m2:/root/.m2 -v /Users/liujinliang/doris-src/incubator-doris-branch-0.15/:/root/doris-0.15/ apache/incubator-doris:build-env-1.4.0
cd doris-0.15
sh build.sh

Coming soon

Will it be released in version 0.15？

Yes, maybe in 0.15.x

When was 0.15 officially released? Yesterday's attempt to compile 0.15 failed

My compilation script is as follows: docker run -it -v /Users/liujinliang/.m2:/root/.m2 -v /Users/liujinliang/doris-src/incubator-doris-branch-0.15/:/root/doris-0.15/ apache/incubator-doris:build-env-1.4.0 cd doris-0.15 sh build.sh

You need to update your docker image

Coming soon

Will it be released in version 0.15？

Yes, maybe in 0.15.x

When was 0.15 officially released? Yesterday's attempt to compile 0.15 failed

My compilation script is as follows: docker run -it -v /Users/liujinliang/.m2:/root/.m2 -v /Users/liujinliang/doris-src/incubator-doris-branch-0.15/:/root/doris-0.15/ apache/incubator-doris:build-env-1.4.0 cd doris-0.15 sh build.sh

You need to update your docker image

The latest image version I found in docker hub is 1.4.0. I think the latest version in the official document is 1.4.1, but I can't get 1.4.1 from anywhere

@yangzhg


Add missing Log pages back.

https://github.com/false-spring/gbfr-logs/commit/41c17fc26ee5f8043fafc28025ac6f93e56703f5 refactored some pages and moved Log View and Log Index out of Log.tsx, but I'm guessing they got ignored by the .gitignore when checking them back in.
I pulled these from the previous commit and it appears to be working.
Feel free to close if you've made other changes that I don't have to these files.

That's what I get for doing multiple things at the same time.. added back in https://github.com/false-spring/gbfr-logs/commit/503dbeccd5c37c4a966a6e5c2cee2279ce4181ae


go mod depends on 1.12

The go module currently depends on an unreleased version of go. Can we bump the requires go version down to a released version?

Is the use of 1.12 negatively affecting you in some way?
It's in RC1 so it's the default version of Go internally to Google at this point, so should be fine to use in pipelines.

We use go 1.11 in all of our builds and don't wish to upgrade to 1.12 until
it's released. In general, it's probably wise for libraries to have a lower
bound on the go compiler version that they depend on since not all users
are able to upgrade.
On Thu, Feb 21, 2019, 2:30 PM JP Robinson<EMAIL_ADDRESS>wrote:

Is the use of 1.12 negatively affecting you in some way?
It's in RC1 so it's the default version of Go internally to Google at this
point, so should be fine to use in pipelines.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/NYTimes/gziphandler/issues/79#issuecomment-466193948,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACu-MgY78GQEmup7E_kfFafjBDrU1pjqks5vPx3sgaJpZM4bIdOQ
.

@jared2501 the version of go specified in go.mod is informative and doesn't actually do anything. It's used by Go to suggest that you may need to upgrade you version in case it doesn't compile, but gziphandler doesn't actually require 1.12.
See https://tip.golang.org/doc/go1.12#modules

The go directive in a go.mod file now indicates the version of the language used by the files within that module. It will be set to the current release (go 1.12) if no existing version is present. If the go directive for a module specifies a version newer than the toolchain in use, the go command will attempt to build the packages regardless, and will note the mismatch only if that build fails.

Hmm I'm not seeing this on go 1.11:
✓ jnewman@Jareds-MacBook-Pro:~/dev/front-door | jn/gzip● $ CGO_ENABLED=0 GOOS=darwin go build -v -o build/darwin-amd64/front-door ./cmd/front-door
github.com/NYTimes/gziphandler
go build github.com/NYTimes/gziphandler: module requires Go 1.12
✕ jnewman@Jareds-MacBook-Pro:~/dev/front-door | jn/gzip● $ go version
go version go1.11.2 darwin/amd64

If you're not keen to downgrade the suggested version, no worries, I can use the replace direct and depend on my fork until go 1.12 comes out and then move over.

@jared2501 oh interesting. We can downgrade, but if the Go docs are telling the truth, the building is failing for some other reason and Go is trying to guess it's because of the language version? Is that code open source?

(and if I have to guess, I'd go with golang/go#29278, which will likely require you to upgrade to Go 1.11.4)

Reopening as we wait for more info.

Ah nice one @fsouza, that was it!

I upgraded to go 1.11.5, and could build v1.1.0 successfully

Cool, thanks for confirming!


Rename paper.pdf to paper.pdf

https://github.com/ABirHAsan123-beep/yaml-test-suite

https://github.com/ibm-granite/granite-code-models/pull/13/commits/4aaea3891792c7cd36aab7c54715d3f335071c5a

#13

SPAM


fix #4579

[ ] Tests added / passed
[ ] Passes flake8 dask

Thanks for the PR @HSR05! Looking forward to having this added
Could you please add a test to dask/array/tests/test_random.py to ensure that the new dtype keyword is behaving as expected. Also, it looks like there are some code linting issues causing the CI builds to fail. You can render these locally by running flake8 dask.

@jrbourbeau I'll work on adding the tests. Can you guide me for adding the tests? I will be more than thankful. 👍

@HSR05 you should read through http://docs.dask.org/en/latest/develop.html which explains in detail how we test.  If you have any particular questions then please ask them.
Asking for general guidance is less likely to be successful.

@HSR05 you should read through http://docs.dask.org/en/latest/develop.html which explains in detail how we test. If you have any particular questions then please ask them.
Asking for general guidance is less likely to be successful.

Thankyou for an heads up. I'll go through that.  👍

Just wanted to check in here @HSR05. Are there any issues you've run into?

@HSR05 wanted to gently ping again and see if you were interested finishing up this PR ?

Closing.  I've handled this elsewhere.


Updated Nerfing Hacker Plan section , added my quotes and minor changes

Changed last line of Nerfing Hacker Plan

UPDATE: Hacker plan is no longer available, the only choice for Private Repls & the other boosts is the $20/mo "Core" plan which is priced at 4x the Hacker plan but you still have to pay more

Added my quote
Minor changes like grammatical updates and added X in brackets

@codingMASTER398  please review

The reason I haven't already merged this is because some of the wording, like " which is priced at 4x the Hacker plan but you still have to pay more", is a bit improper and doesn't fit well with the rest of the text.
More things like "Retweet (X)" isn't needed either.

Oh okay. I will edit and update branch soon (sorry i didnt view your comment)

Updated bro.. I think it is okay to be merged now

Please shorten the quote and remove the bit about hacker plan costing 4x more, as that's previously explained

Yeah okay but I really think  people should know about 4 times  extra pay

Shortened my quote and removed that part

@codingMASTER398

it’s not the overspending that worries me — it’s that I already spend $220 a year for Replit Core and I still have to pay more.

( https://ask.replit.com/t/flexible-credits-usage-based-billing-updates/105372/32?u=prakarsh17-coder)


[BugFix]VMIController: creation should be observed

Creation should be observed when there is any creation error to avoid skipping reconciliation loop forever for some vmis.

What this PR does
Before this PR:
After this PR:

Fixes #
Why we need it and why it was done in this way
The following tradeoffs were made:
The following alternatives were considered:
Links to places where the discussion took place:
Special notes for your reviewer

Checklist
This checklist is not enforcing, but it's a reminder of items that could be relevant to every PR.
Approvers are expected to review this list.

[ ] Design: A design document was considered and is present (link) or not required
[ ] PR: The PR description is expressive enough and will help future contributors
[ ] Code: Write code that humans can understand and Keep it simple
[ ] Refactor: You have left the code cleaner than you found it (Boy Scout Rule)
[ ] Upgrade: Impact of this change on upgrade flows was considered and addressed if required
[ ] Testing: New code requires new unit tests. New features and bug fixes require at least on e2e test
[ ] Documentation: A user-guide update was considered and is present (link) or not required. You want a user-guide update if it's a user facing feature / API change.
[ ] Community: Announcement to kubevirt-dev was considered

Release note

NONE

I think this one should be backported

/cherry-pick release-1.3
/cherry-pick release-1.2
/cherry-pick release-1.1
/cherry-pick release-1.0

@Barakmor1 Could you do manual backports?

@Barakmor1 Could you do manual backports?

Yes :)


room modal rev 1

Looking over the branch a few things:

The db should be a file under assets (eg. room.db)
Using an ORM would abstract the process of querying the db (sqfentity pub)

Changes to be done:

Create SQLite schema for the db
Generate a model in dart using an ORM builder
Update the relevant widgets to support the ORM

Changes I made:

[x] Created SQLite schema for the db (might change in the future)
[x] Generated a model in dart using sqfentity
[x] Updated widgets and screens to work with the new model

Changes to be done:

[ ] Importing data from OSM
[ ] Database versioning and importing newer versions of the db (sqfentity only imports the db from assets first time the app runs or on clear data)
[ ] Update travis script with build_runner for generating the room model


Computation time

Hi,
Based on the paper, the time complexity of the algorithm should be O(N). But in practice, it took around 15 seconds to process the image to get the static cost. I am wondering if this is the normal case to process the image.
Also, did the path cooling and on the fly training included here?
Thanks for any heads-up.

Hi,
The algorithm can process the image in real time. However, my implementation only includes static/dynamic cost and a naive graph search. Are you sure that calculating static features takes so much time? Do you mean graph search? This part is very inefficient now. I am going to improve performance and add additional features within a couple of weeks.

`import pickle
import numpy as np
from collections import deque
from scipy.ndimage.filters import gaussian_filter1d
from skimage.filters import gaussian, laplace, sobel_h, sobel_v, sobel
import skimage
import time
from PIL import Image, ImageDraw
from scissors.graph import PathFinder
from scissors.feature_extraction import StaticExtractor, DynamicExtractor, Scissors
t1 = time.time()
image_path = 'Image.jpg'
image = Image.open(image_path)
t2 = time.time()
w,h = image.size
gray_scaled = skimage.color.rgb2gray(np.asarray(image))
static_extractor = StaticExtractor()
static_cost = static_extractor(gray_scaled)
t3 = time.time()
finder = PathFinder(image.size, static_cost)
t4 = time.time()
dynamic_extractor = DynamicExtractor()
dynamic_features = dynamic_extractor(gray_scaled)
scissors = Scissors(static_cost, dynamic_features, finder)
t5 = time.time()
print('t2-t1:',t2-t1)
print('t3-t2:',t3-t2)
print('t4-t3:',t4-t3)
print('t5-t4:',t5-t4)`
t2-t1: 0.011044979095458984
t3-t2: 0.4484446048736572
t4-t3: 14.301629066467285
t5-t4: 0.20930242538452148
Right, the graph search took the most of the time.

The Pather Finder here is an O(N^2) forloop, should be much fast with the unrestricted graph search.

Thanks for your reply. There's still a lot of work to do. You may look at GIMP implementation of this tool. It works in real time and includes a lot of additional features.


Update the-xliff-interchange-file-format-in-sharepoint.md

Category

[ X] Content fix
[ ] New article
Related issues: fixes #2570

What's in this Pull Request?
Fix invalid link.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/general-development/the-xliff-interchange-file-format-in-sharepoint.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Footnotes don't work

If you click a footnote or the return, it tries to open the Microsoft store.

This issue should be fixed with the new markdig 0.30 version in release 0.7.0.

issue can be reproduced

Bug has been fixed in version 0.7.1 https://github.com/mohzy83/NppMarkdownPanel/releases/tag/0.7.1


Update Turkish localization strings

Hi @nesimtunc - thanks for this but we don't accept translation PRs as we use a third party to do all our translations.  We need to update our contributions doc to reflect this, so sorry about that.  I will also add a note to feedback these to our translators next time the app goes through a translation cycle.

Hi,
Yeap, no worries. They may use it as a reference.
Thank you

On Jun 22, 2021, at 21:08, Christopher Brind @.***> wrote:

Hi @nesimtunc - thanks for this but we don't accept translation PRs as we use a third party to do all our translations. We need to update our contributions doc to reflect this, so sorry about that. I will also add a note to feedback these to our translators next time the app goes through a translation cycle.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.


[docs] Fix broken tutorial-doc symlinks in release-1.15

Issue and/or context: Post-1.15.0 :( I realized that when I synced main and release-1.15 (modulo necessary differences of course) the files that had been symlinks in main became plain files in release-1.15 -- worse, files containing only the path to their referees. :(
[sc-59686]
Changes: Make these symlinks as they should be.
Notes for Reviewer:

Thanks @ryan-williams !


Feat: Import old repo from @thenocturnaldevgypsy/gypsyshards-bootstrap-template-linkinbio to @thenocturnaldevgypsy-io/gypsyshards-bootstrap-template-linkinbio

Feat: Import old repo from @thenocturnaldevgypsy/gypsyshards-bootstrap-template-linkinbio to @thenocturnaldevgypsy-io/gypsyshards-bootstrap-template-linkinbio
Since my 10 year old GitHub account @thenocturnaldevgypsy got "flagged" by GitHub for no apparent reason (GitHub even not responding to my support ticket), I am recreating my old repos to the newly created GitHub account @thenocturnaldevgypsy-io . This is also a good practice/excuse to use GitHub Projects in preparation for most of my freelance work requirements.
I'm just thankful that I still have access to my old account in order to clone the content to my local Git,
Old repo: https://github.com/thenocturnaldevgypsy/gypsyshards-bootstrap-template-linkinbio
New repo: https://github.com/thenocturnaldevgypsy-io/gypsyshards-bootstrap-template-linkinbio

Progress/Updates:

Linked this issue to https://github.com/users/thenocturnaldevgypsy-io/projects/1 for task tracking
Created feat-import-old-repo-issue-1 branch to cover this ticket
Updated .gitignore to have Git/GitHub ignore unnecessary files inside the repo
Updated README.md, kept the old logs from the old repo (especially links) at the CHANGELOG.md
Imported/migrated all necessary files from the old repo to the new one


ship engine doesnt return bucket

when putting a lava bucket into the ship engine, it does not return a bucket.
side note:
please make it so that you can connect a pipe to it, and pump in lava

I would suggest that instead of adding the ability to connect pipes and pump in lava, there should be a separate engine for fluids. A liquid fuel engine you could say. I would like to see it take in liquid fuel and water to produce trust, like a steam engine. In return, it can grant higher power and longer burn time than putting a bucket in a solid-fuel engine.

I would suggest that instead of adding the ability to connect pipes and pump in lava, there should be a separate engine for fluids. A liquid fuel engine you could say. I would like to see it take in liquid fuel and water to produce trust, like a steam engine. In return, it can grant higher power and longer burn time than putting a bucket in a solid-fuel engine.

unfortunately, that requires a bit of custom Fluid Handling as Architectury does not have its own past making new liquids

I would suggest that instead of adding the ability to connect pipes and pump in lava, there should be a separate engine for fluids. A liquid fuel engine you could say. I would like to see it take in liquid fuel and water to produce trust, like a steam engine. In return, it can grant higher power and longer burn time than putting a bucket in a solid-fuel engine.

unfortunately, that requires a bit of custom Fluid Handling as Architectury does not have its own past making new liquids

I would imagine, just wanted to make sure my thought about the side note is logged before this issue is closed and I forget about it. I believe that the side note should be its own separate issue. It might be more fitting for such a feature to be a separate addon.


container ID changed after save setting info

Expected behavior
Actual behavior
Information about the Issue
Steps to reproduce the behavior

chang setting container name
container ID change

This behaves properly - When you change settings, kitematic re-creates a new container with the provided settings.
Containers cannot be modified once created. Why are you relying on the Container ID?

thanks

If the container ID is modified, the docker-compose up will not be able to start the previous container

Docker website description:
https://docs.docker.com/compose/overview/#multiple-isolated-environments-on-a-single-host
Preserve volume data when containers are created
Compose preserves all volumes used by your services. When docker-compose up runs, if it finds any containers from previous runs, it copies the volumes from the old container to the new container. This process ensures that any data you’ve created in volumes isn’t lost.


Update to support F4-based streaming

Currently this code is targeting our DAMS4 file naming conventions.
We need to update the code to support the F4 binary file path as described in https://github.com/ucsdlib/dams5-cc-pilot/issues/27

@mcritchlow @rstanonik: In dams4, we are encrypted the stream name with three fields (ark, file, and ip) and decoded it for Wowza. While in fedora, binary is stored in pair tree ( 3 level deep) with its MessageDigest (md5) as the file name instead. So to build the stream name for Wowza from a Fedora 4 binary file, we need to specify the media type: mp3, mp4, and flv etc.
How about encrypted the fields like object id, file id, md5, media type (or source file name?), and ip for Fedora 4? I think object id and file id are mainly used for logging reference.

I've added a PR to include object ID, fileset ID, Fedora binary stream name, file name, and request id in the encrypted text, which will allow the Wowza server to load the Videos/Audios from the Fedora binary store.

It will also be necessary to write/modify the wowza module that understands the additional information, so wowza can locate and stream the file.

@stanonik Yes, we need to mount the Fedora binary store to the VM where the Wowza server is running so that the Wowza server can located the file by the stream name, which is a pair tree structure handled at line#L286-L294.
I'll create a PR for ticket https://github.com/ucsdlib/dams5-cc-pilot/issues/43 sso that can test the Wowza support.

Going to close this out and we can do more testing when we have a staging environment


tensorflow pyspark

I am an experienced TensorFlow/Keras user. I am trying to learn how to feed the pyspark data frame into the tf/keras project. To understand how to feed pyspark data frame into keras model, I created a toy dataset and model using sequential API that has one input and one output. I am using Colab to run code. You can see the code below. In my case, I could not figure out how to separate the input and output columns to train the model. The code below somehow takes the pyspark dataframe and starts to run, however, It does not finish training the model. I believe I am making a mistake in defining the inputs and output of the model using petastorm. I can also convert the pyspark data frame into pandas data frame, however, I want to learn how to feed the model using pyspark dataframe. Is there any simple solution or simple example to solve this problem?
import numpy as np
import pandas as pd
!pip install --quiet pyspark py4j
!pip install --quiet petastorm

from pyspark.sql import SparkSession
spark  = SparkSession.builder.appName('practice').getOrCreate()
import numpy as np

from petastorm import TransformSpec
from petastorm.spark import make_spark_converter,SparkDatasetConverter
import tensorflow as tf
from  tensorflow import keras
df = spark.createDataFrame([(-1, -3),
                            (0, -1),
                            (1, 1),
                            (2, 3),
                            (3, 5),
                            (4, 7),
                            ], ['A', 'B'])

df.show()
spark.conf.set(SparkDatasetConverter.PARENT_CACHE_DIR_URL_CONF, "file:///dbfs/tmp/petastorm/cache")
converter = make_spark_converter(df.select('A','B'))
def get_model():
  model = tf.keras.Sequential([keras.layers.Dense(units=1, input_shape=[1])])
  return model

def get_compiled_model():

  model = get_model()

  model.compile(optimizer='sgd', loss='mean_squared_error', metrics = ['accuracy'])
  return model
def train_and_evaluate():

  model = get_compiled_model()

  with converter.make_tf_dataset() as dataset:
    dataset= dataset.map(lambda x: (x.A, x.B))
    model.fit(dataset,epochs=2)
train_and_evaluate()

In the documentation it states that for converter.make_tf_dataser() the default value for num_epochs is None, meaning that "Setting num_epochs to None will result in an infinite number of epochs.". This leads to the dataset to keep giving data to your model and training never ending since your model is not aware that the first epoch has ended. Specifying a num_epochs = 2 or another value for num_epochs in make_tf_dataset() should solve your problem I think.
I hope this helps!

@RobindeGrootNL , Yes, you are right. Thanks for your help.  Adding num_epochs solved the problem.
As an additional note, num_epochs in make_tf_dataset just let the finish training process. Real epoch number is given in model.fit command. The working version is shared below.
import numpy as np
import pandas as pd
!pip install --quiet pyspark py4j
!pip install --quiet petastorm

from pyspark.sql import SparkSession
spark  = SparkSession.builder.appName('practice').getOrCreate()
import numpy as np

from petastorm import TransformSpec
from petastorm.spark import make_spark_converter,SparkDatasetConverter
import tensorflow as tf
from  tensorflow import keras
df = spark.createDataFrame([(-1, -3),
                            (0, -1),
                            (1, 1),
                            (2, 3),
                            (3, 5),
                            (4, 7),
                            ], ['A', 'B'])

df.show()

spark.conf.set(SparkDatasetConverter.PARENT_CACHE_DIR_URL_CONF, "file:///dbfs/tmp/petastorm/cache")
converter = make_spark_converter(df.select('A','B'))
def get_model():
  model = tf.keras.Sequential([keras.layers.Dense(units=1, input_shape=[1])])
  return model

def get_compiled_model():

  model = get_model()

  model.compile(optimizer='sgd', loss='mean_squared_error', metrics = ['accuracy'])
  return model
def train_and_evaluate():

  model = get_compiled_model()

  with converter.make_tf_dataset(num_epochs=1) as dataset:
    dataset= dataset.map(lambda x: (x.A, x.B))
    model.fit(dataset,epochs=1000)
    print(model.predict([4]))
train_and_evaluate()


Facebook Comments Plugin Not Working in The TWA

Describe the bug
Thank you very much for this Lab. After building a TWA, I have noticed that the Facebook Comments plugin does not function in the app. In my app, users can view articles on item pages where the Facebook Comments plugin typically displays, and then users can add comments there.
To Reproduce
Steps to reproduce the behavior:

First, I scrolled to the comments section and wrote a comment and clicked "Log In to Post"

A new tab opens inside of the TWA and takes me to the Facebook site to log in

I enter my credentials to log in

The app is then stuck in a continuously flashing blank tab

Expected behavior
I expected the plugin to behave as it normally does with Chrome and allow for comments to be posted.
Smartphone

Device: Galaxy S8
Android Version 9
Chrome 87.0.4280.141

Closing this as I was able to resolve my own issue by enabling third party cookies in chrome, which then caused the Facebook comments plugin to function normally within my TWA.


In the collections repo Readme file , collections build steps are present however we don't know how to create a git release and push build artifacts there for kabanero-pipelines testing

Describe the bug

In the collections repo Readme file , collections build steps are present however we don't know how to create a git release and push build artifacts there for kabanero-pipelines testing
2.Steve GROEGER  updated in one of the thread below information for setting environment parameters , which is not mentioned clearly in the readme file in case of local build step

The only one that needs to be worried about if you are building locally is: IMAGE_REGISTRY_ORG. As that will determine which org the images get created under.

To Reproduce
Steps to reproduce the behavior:

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

Expected behavior
A clear and concise description of what you expected to happen.
Actual behaviour
What is the actual behaviour.
Environment Details (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

If applicable please specify:

CLI version:
Stack you are using:

Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

There is an automated build process that will generate a full release. This includes:

The collections artifiacts - stored within a Git Release (based on the git tag)
The docker images which will be pushed to the Kabanero docker hub

For the work that you are doing to test the pipelines step 2) is not needed.
All that should be needed are the artefacts in a Git Release.
The only way to do that is to build the artefacts manually and to upload them to a Git Release.
To do that you will need to follow the README.md  here: https://github.com/kabanero-io/collections/blob/master/README.md
Because this is a local build that you will be using as if it were a production release then the artefacts will need to be generated as if Travis has built them. This will require just a few extra environment variables to be setup. These are:
TRAVIS_REPO_SLUG=kabanero-io/collections
IMAGE_REGISTRY_ORG=kabanero
TRAVIS_TAG=v0.1.1.beta1
The value of TRAVIS_TAG will vary depending on the the git release you want the artificats in.
Once these are setup then run ./ci/build.sh from the base directory of your git project on your local machine, ie <some_path>/collections
If all goes well then you should have a set of files generated in your ./ci/assets directory.
These are the files that need to be uploaded to a git release.
You can use the same release I generated previously (if you dont need to keep that for something else). Go to http://github.com/kabanero-io/collections......
If you want to use the

@aadeshpa Hopefully this is sufficient to allow you to build the test release artefacts.

Thanks @groeges : sure let me add some more steps after above steps

Once you upload all the artifacts in the Release goto the assets inside that release and find the file kabanero-index.yaml and copy link to that file
example link : https://github.com/kabanero-io/collections/releases/download/v0.1.1.beta1/kabanero-index.yaml
Go to your environment and create a sample.yaml based on this example
https://raw.githubusercontent.com/kabanero-io/kabanero-operator/master/config/samples/full.yaml
In above file sample.yaml update the url link to what you have from point 7 which is as below example
url: https://github.com/kabanero-io/collections/releases/download/v0.1.1.beta1/kabanero-index.yaml
and kubectl apply -f sample.yaml
This will update the collections and pipelines in your environment. New collections and pipeline should get activated as a result of this step.

Instructions on creating a release have been provided.


DD2 - Game forcing wrong resolution

I was expecting to be able to play in full screen mode, however, when playing the game, it forces me into a resolution that i'd rather not play in (1280-720), despite me setting 1920/1080
I've had a mess around in the game config settings but does not seem to be able to work
Because the game is not supporting controllers at the moment, i've not actually been able to mess with the Framework settings in my headset (which is so much easier). It's been a couple of years since I messed around with it so i'm unfamiliar with the keybindings (apart from the insert key) so perhaps i'm just doing something wrong. As I say, unfamiliar with the menu buttons as i've spent so much time using UEVR.
Can someone remind me of the buttons as i've had a look around and can't seem to find them
Thanks

Have just been checking out the troubleshooting, couldn't see any of the settings mentioned regarding resolution, although the in headset resolution is fine, just on monitor its wrong and doesnt respond to swapping in game settings


Support for Either type in tact

Tact should add support for Either TL-B type.
I propose we use the ? operator: TL-B Either X Y -> Tact X ? Y. If it's too confusing we could also use a more explicit syntax Either<X, Y>.

Yes, this is needed to implement #322. I'd propose to use the pipe symbol | instead of ? because it's essentially a type analogue of logical disjunction and there won't be grammar conflicts with the optional type operator: https://github.com/tact-lang/tact/blob/da6a7f6e3eb56a20f5daafa84e50d095f923f3ba/src/grammar/grammar.ohm#L24.
We should also outline the concrete syntax for the constructors and destructors for the Either type.
To construct an expression of an Either type, we need two constructors, we can reuse the Left and Right names that TL-B borrows from Haskell, because it's more general than what, for instance, Rust does with Ok and Error.
To destruct an expression of an Either type, we need pattern-matching:
let foo: X | Y = ...;
match foo {
  Left(l) => {
    return 42;
  }
  Right(r) => {
    return 24;
  }
}

or, in Ohm.js-like notation:
StatementMatch ::= "match" Expression "{" ListOf<MatchClause, ","> ","? "}"

MatchClause ::= Pattern "=>" "{" Statement* "}"

// patterns can be nested
Pattern ::= Constructor ("(" NonemptyListOf<Pattern,","> ","? ")")?
        | TactIdentifier
        | "_"

...

This could be good enough for an initial implementation, later we could support things like | for grouping patterns, etc. The initial implementation does not have to support the general algebraic data types, just the Either type.

The only problem I have with using | is that, in many languages with sum types, | is either straight away commutative, or implementation details are hidden well enough so developers can treat it as commutative.
TL-B on the other hand, makes a clear distinction with the selector bit. And because we want Tact code to interoperate with TL-B definitions, there's no way to abstract that implementation detail away.

Well, because it is commutative. In the sense that you can always build an isomorphism between Either X Y and Either Y X.

Yup, but serializing type X in Either X Y would result in 0b0xxxxxxxx, and serializing X in Either Y X would result in 0b1xxxxxxxx. If people take the type from FunC and change the order without realizing what is under the hood it would lead to errors.

Yeah, that's a good point, but still more verbose syntax won't prevent this. I'm happy to change my opinion if you have a convincing concrete example, though

Random thought: using |> over | may help showing the serialization layout and its "direction". I don't think that we'll be adding pipes in Tact (at least as of now), so that shouldn't be too confusing.
let foo: X |> Y = ...;

Alternatively, we may come up with an or keyword (as in Either ... or ...). It won't describe the layout, but it also won't be confused with general ADTs (not nearly as much as | would).
let foo: X or Y = ...;

As a bonus it pairs nicely with as serialization modifier, introducing a stronger visual bond between left and right parts as opposed to | or comma in Either<..., ...>

// Alternative Maybe of Eithers'
let foo2: X or? Y = ...;
@novusnota what’s that in TL-B notation?

 Y = ...; // Maybe of Either's, like here:
                        // https://github.com/ton-blockchain/ton/blob/140320b0dbe0bdd9fd954b6633a3677acc75a8e6/crypto/block/block.tlb#L155-L157

// TL-B: Either (Maybe X) (Maybe Y)
let ex3: X? or Y? = ...; // Either of Maybe's

// TL-B: Maybe (Either (Maybe X) (Maybe Y))
let ex4: X? or? Y? = ...; // Maybe of Either of Maybe's (inception!)

@novusnota We should just allow ? to be used for any type, not only type idents, hence X or? Y becomes (X or Y)?

I tried to implement this, but I came across the fact that in the type ABITypeRef from @ton/core there is no way to define either type.

Just remembered that Kotlin (the language we name in the first commandment of Tact) uses as for type casts. See "unsafe" cast operator and "safe" nullable cast operator in their docs.

I'm not saying that we should have type casts (we probably don't, except for our !! operator), but the general idea of as? looks reminiscent of my ideas of or? syntax prior.
But this is also fine:

We should just allow ? to be used for any type, not only type idents, hence X or? Y becomes (X or Y)?

Yes, obviously unsafe type casts have no place in security/finance applications.
Maybe and Either types are boolean-blind and exist in functional programming languages mostly for implementation of generic libraries that do not care about semantics of stored values. Contracts are not generic, do care about semantics, and shouldn't use either Either or Maybe.
I understand how adding Either might sound simpler than full-fledged algebraic data types, but winging it here will degrade code quality of the whole Tact ecosystem in the long run, and in my opinion just isn't worth it.

@verytactical but the thing is that token standards have Either in their message schemas so we have to give a good way to define such structs.
Maybe we should add some flag for that, so that it's possible to strictly check which of the Either options was used during parsing?


DRQN-AttributeError: 'zip' object has no attribute 'append'

I run the code and meet a problem like this:

Target Set Success
5000 0.65 1
98%|█████████████████████████████████████████▏| 50/51 [00:00<00:00, 665.63it/s]
Traceback (most recent call last):
File "Deep_Recurrent_Q-Network.py", line 235, in
summaryLength,h_size,sess,mainQN,time_per_step)
File "/home/jimmy/desktop/tensorflowcode/DRL/helper.py", line 52, in saveToCenter
images.append(bufferArray[-1,3])
AttributeError: 'zip' object has no attribute 'append'

I have no idea to solve it and need a help, my friends.
Thanks a lot.

same issue here.

I suspect that you are running this with Python3, then the zip() returns an iterator see here: https://docs.python.org/3/library/functions.html#zip to fix this you could change line 51 in helper.py to
images = list(zip(bufferArray[:,0]))


Include ciphers in CMake build

Just including the cs1c cipher into the CMake-based build by default, since it was not present before. This allows this project to be used in a CMake-based build as a dependency.
Just a quick fix to get cs1c working, does not attempt to include cs3a which requires external dependencies.

Will create new one


F-Droid

Please publish to F-Droid, so that we get notifications of new versions.

Someone submitted it to f-droid awhile back, obviously we don't control how long it takes them to add it to their repo.


Fix AutoSuggestBox not respecting common TextBox properties

Description
This PR makes the AutoSuggestBox control consume the following properties now:

Foreground
Background
BorderBrush
BorderThickness

Motivation and Context
Fixes #2249.
How Has This Been Tested?
Tested visually.
Screenshots (if appropriate):
The following XAML gives the look below:
<AutoSuggestBox Header="AutoSuggestBox" MinWidth="200"
                Foreground="Blue"
                Background="#FF404040"
                BorderBrush="Orange"
                BorderThickness="2"/>

/azp run

@Felix-Dev as expected the visual tree verification tests failed. Can you please update the masters?

@StephenLPeters should I wait until commit #2198 is merged into master, we ran the tests here again and I can use the new visual tree masters now containing #2198 changes too?

/azp run

There was a test failure in the previous run of the build. However the builds get cleaned up after a while, so it is no longer available.
I've kicked off another run of the build. Once that is available we will be able to investigate. Likely an xml master file needs to be updated.

/azp run

I've merged the latest from master and updated the xml masters. I've kicked off another run of the build which should pass the tests now.

Missed a set of masters in the previous push. I think it should be good now.

/azp run

Hmmm. Another xml dump verification failure. I'll take a look.

/azp run

/azp run


MT90A issues

I send  "$$d138,863158020440865,AAA,35,60.000000,130.000000,150408203314,A,7,18,0,0,0,49,3800,24965,510|10|0081|4F4F,0000,000D|0010|0012|0963|0000,,*BF" to my traccar 5020 TCP. It not work. There is server log
2015-04-08 20:41:14 DEBUG: [5020 <- <IP_ADDRESS>] - HEX:<PHONE_NUMBER>382c3836333135383032303434303836352c4141412c33352c36302e3030303030302c3133302e3030303030302c3135303430383230333331342c412c372c31382c302c302c302c34392c333830302c32343936352c3531307c31307c303038317c344634462c303030302c303030447c303031307c303031327c303936337c303030302c2c2a4246
2015-04-08 20:41:18 DEBUG: [5020 <- <IP_ADDRESS>] - HEX:<PHONE_NUMBER>382c3836333135383032303434303836352c4141412c33352c36302e3030303030302c3133302e3030303030302c3135303430383230333331342c412c372c31382c302c302c302c34392c333830302c32343936352c3531307c31307c303038317c344634462c303030302c303030447c303031307c303031327c303936337c303030302c2c2a4246
2015-04-08 20:41:18  INFO: bh: 33, time: Thu Apr 09 04:33:14 CST 2015, lat: 60.0, lon: 130.0
2015-04-08 20:41:18  WARN: Value too long for column "OTHER VARCHAR(255)": "'meitrack3571803800<... (289)"; SQL statement:
    INSERT INTO positions (device_id, time, valid, latitude, longitude, altitude, speed, course, power, address, other)
    VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?);
 [22001-184] - org.h2.jdbc.JdbcSQLException (DbException.java:345)

2015-04-08 20:41:20 DEBUG: [5020 <- <IP_ADDRESS>] - HEX:<PHONE_NUMBER>382c3836333135383032303434303836352c4141412c33352c36302e3030303030302c3133302e3030303030302c3135303430383230333331342c412c372c31382c302c302c302c34392c333830302c32343936352c3531307c31307c303038317c344634462c303030302c303030447c303031307c303031327c303936337c303030302c2c2a4246
2015-04-08 20:41:20  WARN: java.lang.NullPointerException (MeitrackProtocolDecoder.java:264)
2015-04-08 20:41:20  INFO: Closing connection by exception
2015-04-08 20:41:20  WARN: java.lang.NullPointerException (MeitrackProtocolDecoder.java:264)
2015-04-08 20:41:20  INFO: Closing connection by exception
2015-04-08 20:41:20  WARN: java.lang.NullPointerException (MeitrackProtocolDecoder.java:264)
2015-04-08 20:41:20  INFO: Closing connection by exception
2015-04-08 20:41:20  WARN: java.lang.NullPointerException (MeitrackProtocolDecoder.java:264)
2015-04-08 20:41:20  INFO: Closing connection by exception
2015-04-08 20:41:20  WARN: java.lang.NullPointerException (MeitrackProtocolDecoder.java:264)
2015-04-08 20:41:20  INFO: Closing connection by exception
2015-04-08 20:41:20  WARN: java.lang.NullPointerException (MeitrackProtocolDecoder.java:264)
2015-04-08 20:41:20  INFO: Closing connection by exception
2015-04-08 20:41:20  WARN: java.lang.NullPointerException (MeitrackProtocolDecoder.java:264)
2015-04-08 20:41:20  INFO: Closing connection by exception
2015-04-08 20:41:20  WARN: java.lang.NullPointerException (MeitrackProtocolDecoder.java:262)
2015-04-08 20:41:20  INFO: Closing connection by exception
2015-04-08 20:41:20  WARN: java.lang.NullPointerException (MeitrackProtocolDecoder.java:264)
2015-04-08 20:41:20  INFO: Closing connection by exception
2015-04-08 20:41:20  WARN: java.lang.IndexOutOfBoundsException (ByteBuffer.java:375)
2015-04-08 20:41:20  INFO: Closing connection by exception
2015-04-08 20:41:20  WARN: java.lang.IndexOutOfBoundsException (ByteBuffer.java:375)
2015-04-08 20:41:20  INFO: Closing connection by exception
2015-04-08 20:41:20  WARN: java.lang.NullPointerException (MeitrackProtocolDecoder.java:264)
2015-04-08 20:41:20  INFO: Closing connection by exception
2015-04-08 20:41:20  WARN: java.lang.NullPointerException (MeitrackProtocolDecoder.java:264)
2015-04-08 20:41:20  INFO: Closing connection by exception
2015-04-08 20:41:20  WARN: java.lang.NullPointerException (MeitrackProtocolDecoder.java:264)
2015-04-08 20:41:20  INFO: Closing connection by exception
2015-04-08 20:41:20  WARN: java.lang.NullPointerException (MeitrackProtocolDecoder.java:264)
2015-04-08 20:41:20  INFO: Closing connection by exception
2015-04-08 20:41:20  WARN: java.lang.NullPointerException (MeitrackProtocolDecoder.java:264)
2015-04-08 20:41:20  INFO: Closing connection by exception
2015-04-08 20:41:20  WARN: java.lang.NullPointerException (MeitrackProtocolDecoder.java:262)
2015-04-08 20:41:20  INFO: Closing connection by exception
2015-04-08 20:41:20  INFO: Closing connection by disconnect

I see two problems here:

Database column is too short. This is a known issue, but it's not fixed yet. You can either not save "other" data or change column size manually in the database.

Messages are incorrect. Carriage return and line break symbols are missing which means length of the message is incorrect.

Thank you for your reply. I had disable "other" data, it work well.
One more question, where i can change column size?

By default Traccar uses H2 database, so you need to open the database file in any JDBC viewer and edit column length. Probably you need to have some basic knowledge of SQL to do that.

Thank you very much, i will try it


Stable

Issues associated with this PR
Your PR should present related issues you want to solve.
Sign the CLA
Make sure you have signed the CLA
Solutions
You should show your solutions about the issues in your PR, including the overall solutions,
details and the changes. At this time, Chinese is allowed to describe these.
UT result
Unit Test is needed if the code is changed, your unit test should cover boundary cases, corner cases, and some exceptional cases.
And you need to show the ut result.
Benchmark
If your code involves the processing of every request, you should give the Benchmark Result
Code Style

Make sure Goimports has run
Show Golint result

Codecov Report

Merging #798 into master will increase coverage by 3.84%.
The diff coverage is 54.02%.

@@            Coverage Diff             @@
##           master     #798      +/-   ##
==========================================
+ Coverage   51.64%   55.49%   +3.84%
==========================================
  Files         205      232      +27
  Lines       17052    23625    +6573
==========================================
+ Hits         8807    13110    +4303
- Misses       7352     9313    +1961
- Partials      893     1202     +309

Impacted Files
Coverage Δ

pkg/types/network.go
0% <ø> (ø)
:arrow_up:

pkg/module/http2/ciphers.go
100% <ø> (ø)

pkg/types/upstream.go
59.37% <ø> (ø)
:arrow_up:

pkg/api/v2/types.go
71.42% <ø> (ø)
:arrow_up:

pkg/network/idlechecker.go
63.15% <ø> (ø)
:arrow_up:

pkg/metrics/store.go
88.88% <ø> (ø)
:arrow_up:

pkg/xds/v2/xdsconfig.go
0% <ø> (ø)
:arrow_up:

pkg/types/tls.go
0% <ø> (ø)
:arrow_up:

pkg/metrics/downstream.go
0% <ø> (ø)
:arrow_up:

pkg/module/http2/hpack/encode.go
98.07% <ø> (ø)

... and 88 more

Continue to review full report at Codecov.

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


Error Code 134 on Deploy of Base Repo

Have you experienced this bug with the latest version of the template?
Yes
Steps to Reproduce
Initialize project using npx create-remix@latest --template remix-run/blues-stack
run through first steps in README
push to repo
GHA fails to deploy
Expected Behavior
Deploy succeeds without any modifications
Actual Behavior
GHA fails with the following logs:
Error release_command failed running on machine 4d89605f907287 with exit code 134.
Check its logs: here's the last 100 lines below, or run 'fly logs -i 4d89605f907287':
  Pulling container image <OMITTED>
  Successfully prepared image <OMITTED>(4.673965443s)
  Configuring firecracker
  [    0.032823] PCI: Fatal: No config space access function found
   INFO Starting init (commit: 5b8fb02)...
   INFO Preparing to run: `docker-entrypoint.sh bash ./scripts/migrate.sh` as root
   INFO [fly api proxy] listening at /.fly/api
  2024/04/16 00:18:13 listening on [<OMITTED>]:22 (DNS: [fdaa::3]:53)
  Machine created and started in 5.362s
  Setting up swapspace version 1, size = 2 GiB<PHONE_NUMBER> bytes)
  no label, UUID=<OMITTED>
  npm WARN exec The following package was not found and will be installed<EMAIL_ADDRESS>  <--- Last few GCs --->
  [324:0x5453850]     6142 ms: Mark-sweep (reduce) 252.5 (258.2) -> 252.0 (258.7) MB, 122.6 / 0.0 ms  (average mu = 0.570, current mu = 0.271) allocation failure; scavenge might not succeed
  [324:0x5453850]     6269 ms: Mark-sweep (reduce) 253.0 (258.7) -> 253.0 (259.7) MB, 124.4 / 0.0 ms  (average mu = 0.368, current mu = 0.024) allocation failure; scavenge might not succeed
  <--- JS stacktrace --->
  FATAL ERROR: Reached heap limit Allocation failed - JavaScript heap out of memory
   1: 0xb9a330 node::Abort() [npm exec prisma migrate deploy]
   2: 0xaa07ee  [npm exec prisma migrate deploy]
   3: 0xd71ed0 v8::Utils::ReportOOMFailure(v8::internal::Isolate*, char const*, bool) [npm exec prisma migrate deploy]
   4: 0xd72277 v8::internal::V8::FatalProcessOutOfMemory(v8::internal::Isolate*, char const*, bool) [npm exec prisma migrate deploy]
   5: 0xf4f635  [npm exec prisma migrate deploy]
   6: 0xf61b0d v8::internal::Heap::CollectGarbage(v8::internal::AllocationSpace, v8::internal::GarbageCollectionReason, v8::GCCallbackFlags) [npm exec prisma migrate deploy]
   7: 0xf3c1fe v8::internal::HeapAllocator::AllocateRawWithLightRetrySlowPath(int, v8::internal::AllocationType, v8::internal::AllocationOrigin, v8::internal::AllocationAlignment) [npm exec prisma migrate deploy]
   8: 0xf3d5c7 v8::internal::HeapAllocator::AllocateRawWithRetryOrFailSlowPath(int, v8::internal::AllocationType, v8::internal::AllocationOrigin, v8::internal::AllocationAlignment) [npm exec prisma migrate deploy]
   9: 0xf1db40 v8::internal::Factory::AllocateRaw(int, v8::internal::AllocationType, v8::internal::AllocationAlignment) [npm exec prisma migrate deploy]
  10: 0xf155b4 v8::internal::FactoryBase<v8::internal::Factory>::AllocateRawWithImmortalMap(int, v8::internal::AllocationType, v8::internal::Map, v8::internal::AllocationAlignment) [npm exec prisma migrate deploy]
  11: 0xf17868 v8::internal::FactoryBase<v8::internal::Factory>::NewRawOneByteString(int, v8::internal::AllocationType) [npm exec prisma migrate deploy]
  12: 0x1057999 v8::internal::JsonParser<unsigned short>::MakeString(v8::internal::JsonString const&, v8::internal::Handle<v8::internal::String>) [npm exec prisma migrate deploy]
  13: 0x10596b6 v8::internal::JsonParser<unsigned short>::ParseJsonValue() [npm exec prisma migrate deploy]
  14: 0x105a19f v8::internal::JsonParser<unsigned short>::ParseJson() [npm exec prisma migrate deploy]
  15: 0xdf7983 v8::internal::Builtin_JsonParse(int, unsigned long*, v8::internal::Isolate*) [npm exec prisma migrate deploy]
  16: 0x1710839  [npm exec prisma migrate deploy]
  ./scripts/migrate.sh: line 9:   324 Aborted                 npx prisma migrate deploy

(note that the swapfile is set to 2GB in this log, but it fails on the default swapfile size as well)
Related: #170

+1 Seeing the same in a fresh clone / Fly setup following the Readme.

Swap space now has to be set via the swap_size_mb option in fly.toml - https://fly.io/docs/reference/configuration/#swap_size_mb-option

I'm still getting this error after https://github.com/remix-run/blues-stack/pull/241 was merged in. Anyone else?

@drewpereli are you saying that your fly.toml has a swap_size_mb field, and you're still seeing JavaScript heap out of memory?

@benwaffle Yes. I think it's related to this issue, because I fixed it by adding NODE_OPTIONS="--max-old-space-size=4096" to the [env] section of my fly.toml


Add float and double to nvs_partition_gen.py (IDFGH-9377)

This brings the partition generator up to compatibility with the ESP32-Arduino Preferences library.
Both types are converted to blobs then written as binary entries, same as the Preferences library does.

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

what kind of floats are supported?
IEEE754? For persistent storage that should be documented IMO.

It's IEEE754 single (for float) or double (for double) precision.
The C code used by the Arduino library would already be using those, and the Python functions used force IEEE754 regardless of the platform (see note 4 here).

@jclds139 I'm afraid we can't add float and double to the partition generator tool and the docs without also implementing nvs_get_float, nvs_set_float, nvs_get_double, nvs_set_double functions in IDF. Otherwise the readers of the documentation might think that they can actually create key-value pairs using these types and later access them in the program.
However, if we do implement these NVS functions for the new data types, we wouldn't be aliasing them to the blob type — they would be added as new types. This means that Arduino Preferences library will have to change the implementation of these NVS types, which might be a breaking change for existing projects.
cc @me-no-dev


Apply debounce to DependencyDataProvider's refreshing

Although #182 solves the NPE problem, the direct cause of the exception is the too frequent refresh request.
This PR applys debounce to refresh request, make sures that only one refresh request is sent every 2 seconds.

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

We also support customizing the auto refresh delay time via the user setting java.dependency.refreshDelay. The default value is 2000 ms.


Refactor all pollingEpic by using exhaustMap

…store isFetching check.
Remove isFetching reducers, fetch_start and fetch_end action of polling reducers.

Tested and working. There's an unused import of 'store' in lastblockinfo.js but, other than that, everything is good 👍

Thanks. Will remove it and merge.


Refine grouping condition for single assertion conflict

This PR further refines the grouping condition updated in PR #229 to add function name to the key. This is particularly useful when the text of the producer and consumer messages is not sufficient to distinguish the conflicts.

Overall looks good to me (we can also refine this logic to perhaps include a full AST path from root to the diagnostic to further distinguish the diagnostics, but the function name heuristics work for majority of the cases).
I only have one request for some comments and an small optimization :)

Agreed, we can decide to refine the logic further to include the full path if the need arises in the future.
Addressed your comments :)


feat: Adding codemods in loadable-codemod

Summary
This package is a collection of codemod using jscodeshift that can be used to help making big changes easier to a project, for example: migrating from react-loadable to @loadable/component
This PR setups this in a new package named loadable-codemod and provide the react-loadable-to-loadable-component transform.
Test plan
All tests are put inside codemod/transform/__tests__.
Tests are written as such:
jest.autoMockOff();

const { defineTest } = require('jscodeshift/dist/testUtils');

defineTest(__dirname, 'react-loadable-to-loadable-component', null, 'react-loadable-to-loadable-component_expr');

This will check for the file react-loadable-to-loadable-component_expr.input.js inside @loadable/codemod/transform/__testfixtures__, feed it into the react-loadable-to-loadable-component transformer, and assert that the output is equal to react-loadable-to-loadable-component_expr.output.js
Notes
There might be some cases that I haven't accounted for, so any help in improvement would be great! cc: @neoziro

I like the idea 🤩

I think there is only one step before getting the world's best, as well as the world's first codemod of this type - react-loadable-to-loadable-component migration guide in the docs (which does not exist right now).

the first line of that page would be: "Use codemod"
the second line would be about limitations like we are not able, and not going to convert all cases
the third one would be about which patterns from the previous point could be converted to some loadable variant, and which ones should be just refactored.

Sorry for the late reply, work has been busy these past few weeks 😞
Alright, from what I read, the summary is:

Instead of rendering null make it so we render the component without passing any props.
Logs out warning when there are changes that isn't 100% equivalent, such as the one in point 1.
Add documentation about Use codemod

I will try to find the time to work on this, hopefully soon.

Thanks!


Start/Stop do not automatically read config file

Modify Config.pm to allow this.

Fixed in 11f724b.


React kanban

Q
A

Bug fix?
no

New feature?
no

New sample?
yes

Related issues?
NA

What's in this Pull Request?
Added a new sample for a Kanban Board webpart. This sample shows how to use jQWidgets to show a kanban board using the information from a SharePoint Tasks list

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

Really cool stuff @RamPrasadMeenavalli, let us know if you are interested and available for doing a live demo of this in one of the upcoming community calls.

Thank you @VesaJuvonen for accepting the PR.
Yes, I am interested to do a live demo in any of the upcoming community calls.


Add a per page switcher to crud-manager

Let users switch how many results are shown per page.

Closed in https://github.com/geocola/geocola-crud/commit/3aa02ac20b5f0152a4e704e924a4b4736f5571b2


Update MqttServer.cs

clientID is null => RetainedMessageChangedEventArgs ctor throw new ArgumentNullException(nameof(clientId)

@kimdiego2098 Please accept the CLA so that I can merge the PR.

@kimdiego2098 Please accept the CLA so that I can merge the PR.

is ok @chkr1011

Is ci build failed the reason why pr cannot be merged? Can we release the version as soon as possible
@chkr1011


firefox-esr-91-unwrapped: 91.9.1esr -> 91.10.0esr

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for firefox-esr-91-unwrapped is: "A web browser built from Firefox Extended Support Release source tree"
meta.homepage for firefox-esr-91-unwrapped is: "http://www.mozilla.com/en-US/firefox/"
meta.changelog for firefox-esr-91-unwrapped is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 91.10.0esr with grep in /nix/store/8zxsgv85q0nimnj9rq4zmzjpc03i00hx-firefox-esr-unwrapped-91.10.0esr
found 91.10.0esr in filename of file in /nix/store/8zxsgv85q0nimnj9rq4zmzjpc03i00hx-firefox-esr-unwrapped-91.10.0esr

Rebuild report (if merged into master) (click to expand)

6 total rebuild path(s)

3 package rebuild(s)

3 x86_64-linux rebuild(s)
3 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
firefox-esr-91-unwrapped
firefox-esr-unwrapped
firefoxPackages.firefox-esr-91

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/8zxsgv85q0nimnj9rq4zmzjpc03i00hx-firefox-esr-unwrapped-91.10.0esr \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A firefox-esr-91-unwrapped https://github.com/r-ryantm/nixpkgs/archive/361c7c109fa7df72a8a5526e164e0378067812d8.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/8zxsgv85q0nimnj9rq4zmzjpc03i00hx-firefox-esr-unwrapped-91.10.0esr
ls -la /nix/store/8zxsgv85q0nimnj9rq4zmzjpc03i00hx-firefox-esr-unwrapped-91.10.0esr/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

3 packages built:

firefox-esr
firefox-esr-91-unwrapped
firefox-esr-wayland

Maintainer pings
cc @mweinelt for testing.

https://github.com/NixOS/nixpkgs/pull/175514

Made obsolete by #175514. Closing.


DLL ISSUE

give me a message crash error when i try start program ''api-ms-win-core-synch\1-2-0.dll error''

problem solved but now i cant copy and paste

Please update to Procurement 1.20.0, and create a new ticket if this is still an issue.


Question: How is the consumer config setting max.poll.interval.ms implement and used in franz-go?

In the code I can not find any references to max.poll.interval.ms(link).
Does franz-go have behaviour that will trigger a rebalance if a poll has not been issued for a period of time? It is not a feature I am missing, just curious on how this works in this client.

I didn't know of that config knob (I implemented most of the client from KIPs), so it doesn't exist. It could, but I'm a bit unsure about the need -- it seems like a user may want to auto-fail the group member if it is inactive (gets stuck) for a period of time, but that also increases risk of the group member coming back alive and then the group doubly processing records.
Inside the client right now, the group heartbeats always, and rebalance is triggered in the normal ways: leader detects new partitions, or member detects new topics, or some member times out, or new member joins.
Is there a need for this option? I'm currently erring on this behavior is something that is more suited to an application using the client rather than within the client.

Is there a need for this option

Not in my opinion, at least I dont see the need right now. Thanks for your answer. If others bring it up, lets reevaluate but for now I will close is issue/question. I was just curious.