[Feature Request] Best time to buy and sell stocks_1-5 Dynamic programming advance problems

🚀 Best Time to Buy and Sell Stocks
Featured Problem
The "Best Time to Buy and Sell Stocks" problems are essential in dynamic programming, commonly featured in coding interviews. By adding a structured series of these problems with varying difficulties (1-5), learners can progressively build their skills while gaining practical insights into stock trading strategies.
Describe the solution you'd like
I would like a set of dynamic programming problems focused on the "Best Time to Buy and Sell Stocks" concept, categorized into five different levels of difficulty (1-5). Each problem should include:
A clear statement of the problem.
Example inputs and outputs.
A step-by-step explanation of the optimal solution, including the dynamic programming approach.
Time and space complexity analysis.
Describe alternatives you've considered
I have considered creating simpler problems that focus solely on the basic buy/sell mechanics without incorporating dynamic programming techniques.

Would you like to work on this feature?

[X] Yes
[ ] No

@ajay-dhangar shouldn't these problems be labeled as level2


Use tsup as bundler

Use https://github.com/egoist/tsup#tsup as bundler, so the bundling process is faster, more stable and provide the possibility to serve cjs + esm at the same time.
I will create a PR to demonstrate it.

I also want to introduce vitest instead of jest, and tsup and vitest share similar dependencies, which results in smaller node_modules/sub-dependencies.
However vitest doesn't make sense in this repo, because there are not tests at all 🤔

Use https://github.com/egoist/tsup#tsup as bundler, so the bundling process is faster, more stable and provide the possibility to serve cjs + esm at the same time.
I will create a PR to demonstrate it.
I also want to introduce vitest instead of jest, and tsup and vitest share similar dependencies, which results in smaller node_modules/sub-dependencies.
However vitest doesn't make sense in this repo, because there are not tests at all 🤔

For vitest I think https://github.com/dreamit-de/otel-json-logger might be a good project to try it out. It has just one test class but many features we use in other projects are used here.
When I think of vite or vitest I thought frontend projects might benefit most of it. I don't know if and what benefits it might provide, so a PoC/MR to try it out might give some answers. Especially points like developer experience I have to try out myself.

For vitest I think dreamit-de/otel-json-logger might be a good project to try it out. It has just one test class but many features we use in other projects are used here.

I will look into it :+1:

When I think of vite or vitest I thought frontend projects might benefit most of it. I don't know if and what benefits it might provide, so a PoC/MR to try it out might give some answers. Especially points like developer experience I have to try out myself.

You are right about Vite :+1:
Vite is a bundler for mainly frontend and additionally have the "lib"-mode with that you can create e.g. UI Component Libraries
However vitest is a full replacement for jest (& mocha & chai) which even brings its compatibility layer, so if you are lucky, you can 1-to-1 replace jest with vitest and just swap the imports
vitest is a highly optimized testing framework with which you can e.g. run tests in isolation and parallel out of the box
jest has that too to some kind, but is slower and "more outdated"
vitest uses Vite under the hood for its HMR support in watch-mode
vitest is not a UI testing tool, but a JS/Node testing tool
as an example, you could have a look into some of my projects like

https://github.com/faker-js/faker/tree/next/test
https://github.com/prettier/plugin-pug/tree/main/tests
https://github.com/gtm-support/core/tree/main/tests

as you can see, non of these are frontend related and are pure Node projects


[$500] IOU - Distance request with vague location created offline shows incorrect error online

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

Version Number: 1.4.41
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/4314136
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Action Performed:

Launch app
Turn off mobile data
Tap fab
Tap request money--- distance
Tap start
Enter SF and tap save
Tap finish
Enter LA and tap save
Tap next
Select a workspace
Tap request
Turn on mobile data

Expected Result:
In offline, with vague location distance request created after turning online must show correct error message "Unable to find the location for this address. Please edit the address and try again. ”
Actual Result:
In offline, with vague location distance request created after turning online shows incorrect error message "Unexpected error requesting money, please try again later" instead of "Unable to find the location for this address. Please edit the address and try again.”
Workaround:
Unknown
Platforms:

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

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

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/11fd4292-7e6c-47c4-9053-98b4de0dab8c
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~014b08208fbad4a9ac
Upwork Job ID:<PHONE_NUMBER>611659264
Last Price Increase: 2024-02-14

We think that this bug might be related to #wave5
CC @dylanexpensify

Asked Dylan here - https://expensify.slack.com/archives/C05DWUDHVK7/p1707961175788869

Proposal
Please re-state the problem that we are trying to solve in this issue.
Incorrect error message displayed in app for offline-created distance request when online.
What is the root cause of that problem?
In buildOnyxDataForMoneyRequest we always use a genericCreateFailureMessage for transaction failure: https://github.com/Expensify/App/blob/935c44ae97f3dbe28b0bb4b237df4534ba594e86/src/libs/actions/IOU.ts#L612 https://github.com/Expensify/App/blob/935c44ae97f3dbe28b0bb4b237df4534ba594e86/src/languages/en.ts#L654
What changes do you think we should make in order to solve the problem?
We need to add a "Unable to find the location for this address. Please edit the address and try again." at anguages/en.ts, sometjhing like:
error: {
    // ... (existing error messages)
    invalidLocation: 'Unable to find the location for this address. Please edit the address and try again.',
    // ... (other error messages)
}

And properly use it when the request fails due to invalid waypoints.
What alternative solutions did you explore? (Optional)

@brunovjk
Could you be more specific about which conditions (with variables) will wel add this invalidLocation error. Also, isn't this error sent by the backend?

@c3024 With the same steps for reproduction, this issue has a different Expected Result. I assume we're dealing with a kind of new feature here, is that correct?

isn't this error sent by the backend?

Yes. But it sends a "Please select a valid waypoint." message. While we can use this, it seems the message isn't yet expected for this particular issue.

invalid waypoint offline

If we repeat the steps for reproduction, but return Online in step 8, we see that we cannot proceed, and this error message above appears:

invalid waypoint online

Should the behavior be similar in both reproductions above?

@c3024 With the same steps for reproduction, this issue has a different Expected Result. I assume we're dealing with a kind of new feature here, is that correct?

isn't this error sent by the backend?

Yes. But it sends a "Please select a valid waypoint." message. While we can use this, it seems the message isn't yet expected for this particular issue.

invalid waypoint offline

If we repeat the steps for reproduction, but return to online in step 8, we see that we cannot proceed, and this error message above appears:

invalid waypoint online

Should the behavior be similar in both reproductions above?
Thank you :D

Bump @c3024

I continue investigating and testing, I will update my proposal soon.

I've been investigating and couldn't find a solution completely through the frontend. In addition to the new message "Unable to find the location for this address. Please edit the address and try again.", I believe there is a problem with the backend here, the issue report video doesn't show it but the request appears to be deleted without even the user "dismissing the error".

Bump @c3024 - can you confirm if this is a back-end issue? Thanks!

Actually should be #wave6 - posted: https://expensify.slack.com/archives/C02MW39LT9N/p1708900139785349

@jliexpensify external like me do not have access to this archive right?

@brunovjk - if it's a back-end issue, we'll apply the Internal label for an Expensify Engineer to pick up.
From that point on, the engineer can determine if we need help from Contributors.

Very good @jliexpensify, thank you very much!

@jliexpensify
Yes, the validation happens on backend so backend sending the correct error message is appropriate. Please apply Internal label. Thanks.

Great, thanks for jumping in Neil. Closing (also requested by Greg)!


Make hexidecimal named scripts execute without requiring .\ prefix

PR Summary
In #15543 it was discovered that if you have a case where a script is named in a way that can be interpreted as hexadecimal (such as 0x.ps1 and you execute it without the dot backslash convention (.\0x.ps1 the script will always return null.
@vexx32 identified the problem as being an oversight in an if statement that doesn't account for a number appended a few lines above.

PR Context
This aims to solve #15543.
PR Checklist

[x] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[x] Summarized changes
[x] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[ ] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: or [ WIP ] to the beginning of the title (the WIP bot will keep its status check at Pending while the prefix is present) and remove the prefix when the PR is ready.

Breaking changes

[x] None
OR
[ ] Experimental feature(s) needed

[ ] Experimental feature name(s):

User-facing changes

[x] Not Applicable
OR
[ ] Documentation needed

[ ] Issue filed:

Testing - New and feature

[ ] N/A or can only be tested interactively
OR
[x] Make sure you've added a new test if existing tests do not effectively test the code changed

Tooling

[x] I have considered the user experience from a tooling perspective and don't believe tooling will be impacted.
OR
[ ] I have considered the user experience from a tooling perspective and opened an issue in the relevant tool repository. This may include:

[ ] Impact on PowerShell Editor Services which is used in the PowerShell extension for VSCode
(which runs in a different PS Host).

[ ] Issue filed:

[ ] Impact on Completions (both in the console and in editors) - one of PowerShell's most powerful features.

[ ] Issue filed:

[ ] Impact on PSScriptAnalyzer (which provides linting & formatting in the editor extensions).

[ ] Issue filed:

[ ] Impact on EditorSyntax (which provides syntax highlighting with in VSCode, GitHub, and many other editors).

[ ] Issue filed:

The change is cuausing some unexpected behavior - https://dev.azure.com/powershell/PowerShell/_build/results?buildId=111934&view=ms.vss-test-web.build-test-results-tab&runId=1961790&resultId=108814&paneView=debug
I'm not sure how this can be related at all to the changes I made.

I'm not sure how this can be related at all to the changes I made.

I hope @vexx32 and @SeeminglyScience could help.

🤔 Not seeing anything in those tests that feels really relevant here, unsure what's going on there.

In conclusion:
On Windows the following tests fail:

Import-Alias DRT Unit Tests.Import-Alias Resolve To Multiple will throw PSInvalidOperationException
Group-Object.Should output an array when piped input is used

On Linux and MacOS, both of the new test cases fail for this reason:
The term '0x.ps1' is not recognized as a name of a cmdlet, function, script file, or executable program. / CommandNotFoundException

@vexx32 @iSazonov I'm unable to get the tests to work after multiple attempts and speaking with people in the Powershell Discord.
Alll pipeline tests pass when I exclude edits to the Parser.Tests.ps1 file, so I am sure that the issue isn't the changes we made, but the test cases I created.
Is there any protocol for excluding tests when they prove to be too cumbersome to reasonably implement? If so I strongly believe that it's the best option.

Alll pipeline tests pass when I exclude edits to the Parser.Tests.ps1 file, so I am sure that the issue isn't the changes we made, but the test cases I created.

I suggest to pull new draft test PR with only the new your tests and without the code change and see CI results.
Expectation is all tests pass except new.

I suggest to pull new draft test PR with only the new your tests and without the code change and see CI results.
Expectation is all tests pass except new.

Please see failures here: https://github.com/PowerShell/PowerShell/pull/18398

Not waiting on author. Waiting on experienced pester user input on broken tests.

Ready to review?

Ready to review?

Yes!

@SteveL-MSFT Please update your review.

/rebase


Version Increment to 1.3.3 OpenSearch Dashboard release

Signed-off-by: pgodithi<EMAIL_ADDRESS>Description
https://github.com/opensearch-project/opensearch-build/issues/2101
Issues Resolved
https://github.com/opensearch-project/opensearch-build/issues/2101
Check List

[ ] New functionality includes testing.

[ ] All tests pass, including unit test, integration test and doctest

[ ] New functionality has been documented.

[ ] New functionality has javadoc added
[ ] New functionality has user manual doc added

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #86 (207b50f) into 1.3 (60c60c6) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              1.3      #86   +/-   ##
=======================================
  Coverage   95.30%   95.30%
=======================================
  Files           5        5
  Lines         149      149
  Branches       45       45
=======================================
  Hits          142      142
  Misses          7        7

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 60c60c6...207b50f. Read the comment docs.


Add option to show video player controls on Web and make double click go fullscreen when controls are invisible

Hi,
I would like to propose that:

video player controls are added in VideoPlayerOptions as an option in the video_player package. Without it is really unintuitive to the end user as the double click doesn't make it full screen when the controls are not visible.
Double click would make the video go full screen when the controls are not visible (like it does when the controls are visible)

I am currently using a VideoElement() directly to achieve the same thing but that will hit me when I go native. Chewie at the time had issues with fullscreen web video.
final video = VideoElement()
..src = URL!
..autoplay = widget.autoplay
..controls = widget.controls

@zambetpentru
Check this open issue and see if it's similar to your request.

Greetings,  it looks like that other issue is to get fullscreen video working on web? Which it now does as of a few weeks ago :-)

Thanks for confirming.


timeString: always plural

1 secs -> 1 sec.

I will likely provide a localization string that accepts all the fields from a Difference object, allowing localization of this output. I won't be able to take care of this for a while, potentially. I'm waiting on my laptop to come in from repairs, and I'm hoping to do a major overhaul of the localization backend soon.


<IP_ADDRESS>	Feature single campaign on front page of app

Feature an upcoming/current campaign with its picture, summary description and link to campaign page on the home page.  For this release we can just select first campaign that fits the criteria as we will have only one campaign ‘in production’ and for future we will likely add filters, possibly site admin tagging for highlight and featuring more than one campaign.
See below for an example of how to display/feature on home page (subject to UX discussion in #693)

@tonysurma I can take a look at this one. Please assign to me.

Update from me on this. The work to implement is mostly complete. I have some tests to write and a little tidy up and should be able to get my PR in soon.


Update: Adobe.CreativeCloud version <IP_ADDRESS>1

Created by 🥟 Dumplings in workflow run #8318.

Log
The hash was changed
Submitting WinGet manifests
Found existing pull requests:
New version: Adobe.CreativeCloud version <IP_ADDRESS>1 - https://github.com/microsoft/winget-pkgs/pull/174306
This task is configured to ignore existing pull requests
Creating manifests
Uploading manifests and making commits
Creating a pull request

Microsoft Reviewers: Open in CodeFlow

Automatic Validation ended with:

(Automated response - build 896.)


How to setup the Roslyn code generator

Hi there,
I'm using the Asset store version and try to setup the Roslyn code generator. Things have changed since Simon's video on how to set it up
https://youtu.be/L-18XRTarOM?t=232
So after unzipping the Jenny.zip file I do this

I run mono Jenny/Jenny.exe
Select "Use Jenny.properties"
Select ??????
Run "Save and continue (auto import)"
Start the server via ./Jenny-Server
Generate via mono Jenny/Jenny.exe gen

But what needs to be selected from the CLI dialog? By that I mean this dialog

I tried to select just everything but this is wrong and leads to errors #962
The rest of my setup steps should be fine I think?

So I guess this means Entitas just isnt supported anymore? When Documentation on something this important is missing and nobody is answering here, its just not worthwhile using anymore I am afraid.


Calf Position Protection

We are in low-level development mode (L2+A, L2+A, L2+B, L1+L2+START, the robot is lying on the ground), and trying to run our low-level control script on it. But when we run a roslaunch, we keep geeting Position Protection Error, is that supposed to happen? Sample output:
leg 0 calf: -159.495187
leg 1 calf: -158.930801
leg 2 calf: -157.501350
leg 3 calf: -159.381854
Error: Position Protection.

I also have the same problem (sometimes, maybe depending on the exact leg placement when starting)


Add more systemd hardening params and update systemd readme

This PR adds a bunch more systemd hardening params to lock the service even further.
A section on the systemd readme was added with instructions for development environments (someone said on Slack they had to figure this out by themselves, so might as well have it in the readme).
The new systemd service was tested on a single replica cluster and a 3-replica cluster. Upgrades weren't tested, but I hope to do that once a version greater than 0.15.4 lands due to the unique upgrade process from 0.15.3 to 0.15.4.
I read through TB's source code for any mentions of posix and linux to check the involved syscalls and other things the TB code was relying on, and tried to lock down almost everything else that I could understand from systemd's docs. The testing I did wasn't extensive, so I appreciate any further testing from others.

Thanks @DanielSidhion!
Looks like the spell check is trying to check the systemd unit file: https://github.com/tigerbeetle/tigerbeetle/actions/runs/10278086700/job/28441172616
Could you try

@cb22 I added more words to the spellcheck dictionary, looks like the CI job for that succeeds now. Thanks for calling it out :)


Create ShadowView.pyui

To match ShadowView.py ... Some modifications also need to be made to Popup.pyui.  I can submit those once ShadowView.py & .pyui are in the repo.

I reworked the structure.


Update BulkLoadFile class

What are these for? Maybe I missed a discussion somewhere?

@oblodgett linkml_schema_version is already in our Java model.  There was a request to track which MOD release corresponds to each submitted file.  This slot has already been added to the Ingest class, but it needs to be stored on the bulkLoadFile too.


Export GraphQlDefinitionWidget

Hey, I just made a Pull Request!
This is important for reuse of the widget.

:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

Could not fix checks as I have this error
gitpod /workspace/backstage $ yarn build:api-reports plugins/api-docs
yarn run v1.22.1
$ yarn build:api-reports:only --tsc plugins/api-docs
$ ts-node -T -P scripts/tsconfig.json scripts/api-extractor.ts --tsc plugins/api-docs
$ tsc --project /workspace/backstage/tsconfig.tmp.json --skipLibCheck false --incremental false
node_modules/@asyncapi/react-component/lib/types/helpers/marked.d.ts:1:18 - error TS7016: Could not find a declaration file for module 'highlight.js/lib/core'<EMAIL_ADDRESS>implicitly has an 'any' type.
  Try `npm i --save-dev @types/highlight.js` if it exists or add a new declaration (.d.ts) file containing `declare module 'highlight.js/lib/core';`

1 import hljs from 'highlight.js/lib/core';
                   ~~~~~~~~~~~~~~~~~~~~~~~

Found 1 error.

Yeah i think you may be hitting a bug in the api report generator, that only occurs when run on a subset of packages. I tried running over the entire project and that seems to have worked out. I rudely pushed the updated report on top of your branch, let's see if it goes green


[Feature request] Form validation: "dependable validation" with callbacks

Hello.
Form validation works great on forms with independent fields.
But if fields are dependent?
For sample, I have 4 fields. Form should be valid, if:
function(field1) < function(field2) < function(field3) < function(field4) if all of them not empty.
If field is empty, it should be skipped from this inequality.
Or field is required only if another field have a specific value...

Any native PHP function that accepts one parameter can be used as a rule, like htmlspecialchars(), trim(), etc.

So I need to use functions which accepts many parameters as rule.

This is already possible with anonymous functions (or in some cases with clever usage of existing rules). We can't add more and more logic for every edge case like this one, and I'm pretty sure it would break some existing rules like regexp_match anyway.

Than there is a big lack of documentation. Can you provide me the sample how it's possible with anonymous functions? I see only sample with one parameter in documentation.
$this->form_validation->set_rules(         'username', 'Username',         array(                 'required',                 array(                         'username_callable',                         function($str)                         {                                 // Check validity of $str and return TRUE or FALSE                         }                 )         ) );

There's no lack of documentation - anonymous functions are a PHP feature and we document what CI provides, not how PHP works.
$field1 = $this->input->post('field2');
$field2 = $this->input->post('field2');
$field3 = $this->input->post('field3');
$callback = function() use ($field1, $field2, $field3) {
    // Your logic here
};

... and it's not like a controller callback couldn't lookup other fields via $this->input->post().


MAM function doesn't return anything

Hi,
Thank you for your work on this plugin.
The flutterXmpp.requestMamMessages(userJid, requestSince, requestBefore, limit) function does not return any messages. Is that correct? How do I receive the archived messages?
  Future<void> requestMamMessages(
      String userJid, String requestSince, String requestBefore, String limit) async {
    print(
        " Plugin : User Jid : $userJid , Request since : $requestSince , Request Before : $requestBefore, Limit : $limit ");
    final params = {
      "userJid": userJid,
      "requestBefore": requestBefore,
      "requestSince": requestSince,
      "limit": limit
    };
    await _channel.invokeMethod('request_mam', params);
  }

Thanks,
Robert

You should expect the message that on the receivemessage listener. @robertscally
let me know if that doesn’t work.


MGMT-18458: Simplify helm charts and add UI helm chart as dependency

This PR reduces the amount of configuration needed for deploying FCTL as a Standalone or ACM plugin.
Notable changes:

we allow choosing multiple auth methods - none, oidc (external OIDC), built-in (our Keycloak deployment), openshift
Keycloak (built-in auth) is enabled by default
The built-in Keycloak helm chart is very bare-bones. Most of the values are now hardcoded. It is expected that for the production-grade deployments, users will have their own Keycloak instance (and will use oidc auth method, instead of built-in).
UI helm charts are added as dependency, currently referencing charts built from https://github.com/flightctl/flightctl-ui/pull/112 - this will need to be updated once we merge the UI PR

If you want to try the deployment, here are the example values.yaml
Standalone
global:
  baseDomain: "fctl-test.apps.rw-fctl.nvss.p1.openshiftapps.com"

flightctl-ui:
  enabled: true
  api:
    insecureSkipTlsVerify: true
  image:
    image: quay.io/rawagner/flightctl-ui
    tag: standalone

ACM
global:
  baseDomain: "fctl-test-acm.apps.rw-fctl.nvss.p1.openshiftapps.com"
  auth:
    type: openshift
    openShiftApiUrl: https://api.rw-fctl.nvss.p1.openshiftapps.com:6443
  networkPolicyAllowList:
    - openshift-console

flightctl-ocp-ui:
  enabled: true
  api:
    insecureSkipTlsVerify: true
  image:
    image: quay.io/rawagner/flightctl-ui
    tag: plugin

the api.insecureSkiptTlsVerify will go away once we have the endpoint to download the CA certs
UI image section will also go away once we merge these PRs and can get the updates images from fctl quay repositories.

Alternative proposal:
Standalone
global:
  baseDomain: "fctl-test.apps.rw-fctl.nvss.p1.openshiftapps.com"
  ui: "standalone"

This part I'd make it the default (so we avoid the CA setup step UI->backend) I don't think that buys us much for now. We can automate it later.
flightctl-ui:
  api:
    insecureSkipTlsVerify: true

ACM
global:
  baseDomain: "fctl-test-acm.apps.rw-fctl.nvss.p1.openshiftapps.com"
  auth:
    type: openshift
    openShiftApiUrl: https://api.rw-fctl.nvss.p1.openshiftapps.com:6443
  ui: "acm"

Do you think there could be a way to use helm inspection of the cluster to detect "openShiftApiUrl"?

Helm dependencies conditions can reference only 'boolean' values, you cannot do functions like eq or similar. In order to implement the suggested changes, we would need to have only one UI helm chart that can determine if standalone or ACM plugin should be deployed.
If we will have only one UI helm chart we could take this a bit further:
Standalone
global:
  baseDomain: "fctl-test.apps.rw-fctl.nvss.p1.openshiftapps.com"

and UI helm chart can detect global.auth and if it is built-in or oidc then deploy standalone UI
ACM
global:
  baseDomain: "fctl-test-acm.apps.rw-fctl.nvss.p1.openshiftapps.com"
  auth:
    type: openshift
    openShiftApiUrl: https://api.rw-fctl.nvss.p1.openshiftapps.com:6443

since global.auth.type is openshift, UI helm chart would deploy ACM plugin
Im not sure if we can use helm inspection to get the k8s API and use it in values. I will take a look.

@rawagner yesterday I remember looking it up somewhere but can't find it now, I saw you could set conditions like:
condition: eq global.some.value "blah"
I'll try to verify and ping back

After talking to @mangelajo , i've created a helper function to get the base domain of the current openshift cluster and also merged the UI charts into one.
Now when deploying the Standalone version, the default values.yaml is ok, there is no need to define/override anything.
For the ACM deployment, we only need to specify auth.type = openshift
global:
  auth:
    type: openshift

This looks good, let's finish and merge https://github.com/flightctl/flightctl-ui/pull/112 so we can reference it from here.

UI PR is merged. I've updated chart.yaml/lock to reference the released UI chart


issues with "show replies" timeline mode

"show replies" timeline mode doesn't stay on but rather keeps resetting back ( to not showing replies ) even from just clicking on someone's profile and then going back to timeline, but also anytime you close the browser.
also when in "show replies" mode the icon to reply to the notes disappears and i have to open the thread or drawer view in order to reply.
also in thread / drawer view i can't reply to any note above the one i clicked on - no reply icon to press. i have to copy Note ID, search for it and then i can reply.

Might be good to split this issue.
I confirmed that show replies resets.
I confirmed that show replies doesn't show in note view.
As a workaround to the last thing, "Write reply" does show for replies above by clicking on the note.


Difference between first two components of snapshot image

I'm confused about the difference between the first two components in the snaspshot_<time_step> image panel being saved after each episode's run.

Above are the two side-by-side images i'm talking about.
They both seem to be depicting object detection confidence score + mask segmentation.
The code seems to suggest the first is the raw rgb_frame, however, I don't understand why it also consists of the segmentation mask (if it's the raw rgb_frame) and detection confidence threshold.
Appreciate any help understanding this diff, thanks!

The right one just overlays the mask - for each predicted object Detic also provides us with masks. If you do not like the box around the object in the left frame, you can try setting use_detic_viz=False https://github.com/facebookresearch/home-robot/blob/5eca3968af0451058795c8e160600aa50ccf0a67/projects/habitat_ovmm/configs/env/hssd_demo.yaml#L22

Hi, that makes sense. two questions here:

The left frame seems to have a beige color mask too over the duck.
I want to replace the left frame with the ground truth masks, how do you suggest I do this?

Hi,

I think the beige color mask comes from the Detic visualisation too. Setting use_detic_viz = True should remove it too.
If env_config.ground_truth_semantics = False (which is the case for you), GT is not passed along - fields like object_segmentation have detector outputs. To visualize both, firstly you can add separate GT fields for these observations here. Let's call them gt_object_segmentation. Then include it in the info dict here. This dict will get passed to visualize function where you can overlay this on top of the rgb person frame  here.

thank you. that worked. had to tweak a few more things, but ground truth is displaying now:


MIP12c2-SP3: Domain Work Requirements for the Onboarding of MANA

MIP12c2-SP: 3
Title: Domain work requirements for onboarding the MANA collateral type
Author(s): Cyrus Younessi (@DonutJr), Niklas Kunkel (@NiklasKunkel), Mariano Conti (@nanexcool)
Contributors: Charles St.Louis (@CPSTL)
Status: Formal Submission (FS)
Date Proposed: 2020-07-08
Date ratified:

LGTM!


Update README.md

add a new jailbreak defense method by two-stage adversarial tuning

Thank you for adding the paper!


multiple locations in Python?

great tool!
is there a way to query multiple locations from a Python3 script?
import requests     payload = {'format': '4'}     r = requests.get("http://wttr.in/{Nuremberg,Hamburg,Berlin}" , params=payload)
does not seem to work (will return the last location with name Nuremberg,Hamburg,Berlin)
thanks!

The examples given in the README are shell syntax that performs multiple location lookups, and not a specific capability of wttr.in to perform more than one location lookup in a single request.
When the shell sees the following command:
curl -s 'wttr.in/{Nuremberg,Hamburg,Berlin}?format=3'
It gets processed into:
curl -s 'wttr.in/Nuremberg?format=3'
curl -s 'wttr.in/Hamburg?format=3'
curl -s 'wttr.in/Berlin?format=3'
And each command is run consecutively.
If you wanted to do the same in Python, it would look more like this:
import requests

payload = {'format': '4'}
locations = ["Nuremberg", "Hamburg", "Berlin"]
for loc in locations:
    resp = requests.get(f"http://wttr.in/{loc}", params=payload)
    ...

makes sense. thanks!!


Use Mutex instead of RwLock in ProgressBar

Nearly every call to RwLock was write() defying it's advantage over
Mutex. Additionally, Mutex implements Sync for T: Send, contrary
to RwLock which implements Sync for T: Sync, which is a bit harder to
satisfy and thus less versatile.

Makes sense, thanks!

Thanks for your quick responses! :)


Handlebars file detected as Mustache

Description
All *.hbs files are being detected as Mustache, and using setting for Mustache instead of Handlebars.
The root of the problem is that I want to use Pretty Diff for Handlebars instead of JS Beautify, but it's not one of the supported beautifiers for Mustache.
Input Before Beautification
{{header}}
{{outlet}}
{{footer}}

Expected Output
Original File Path: C:\temp\format\app\application\template.hbs

Original File Grammar: Handlebars

Original File Language: Handlebars

Language namespace: handlebars

Supported Beautifiers: JS Beautify, Pretty Diff

Actual Output
Original File Path: C:\temp\format\app\application\template.hbs

Original File Grammar: HTML (Mustache)

Original File Language: Mustache

Language namespace: mustache

Supported Beautifiers: JS Beautify

Steps to Reproduce

Go to .hbs file
Run command Atom Beautify: Beautify Editor
Code is beautified with Mustache settings

Debug
Here is a link to the debug.md Gist: debug.md
Checklist
I have:

[x] Tried uninstalling and reinstalling Atom Beautify to ensure it installed properly
[x] Reloaded (or restarted) Atom to ensure it is not a caching issue
[x] Searched through existing Atom Beautify Issues at https://github.com/Glavin001/atom-beautify/issues
so I know this is not a duplicate issue
[x] Filled out the Input, Expected, and Actual sections above or have edited/removed them in a way that fully describes the issue.
[x] Generated debugging information by executing Atom Beautify: Help Debug Editor command in Atom and added link for debug.md Gist to this issue

It looks like this can be done by simply modifying this file https://github.com/Glavin001/atom-beautify/blob/master/src/beautifiers/prettydiff.coffee at around line 50.  Just make the change and submit a pull request!


⚠ Orpheus has degraded performance

In a291e9a, Orpheus ($OPS) experienced degraded performance:

HTTP code: 200
Response time: 1289 ms

Resolved: Orpheus performance has improved in edb9797 after 5 minutes.


reclassification of bovine dairy liquid manure under a new term bovine manure

Current Classification:
Bovine dairy liquid manure [http://purl.obolibrary.org/obo/ENVO_01001116] is currently defined as "Manure which has been collected directly or indirectly from dairy cattle, as from flushing of manure from concrete feed lanes, free stalls, and milking facilities," and is classified under animal manure.
Suggested Revision:
Create a new entry "bovine manure" defined as "The excreta of Bos taurus, with or without an admixture of bedding or litter, fresh or at various stages of further decomposition or composting." This definition is derived from: 1) the definition of animal manure in the Glossary of Soil Science Terms [https://www.agronomy.org/publications/soils-glossary], and 2) the scientific name of bovines in Wilson & Reader's Mammal Species of the World, 3rd Edition [http://www.departments.bucknell.edu/biology/resources/msw3/]. This term is also consistent with the suggested revision of the term "animal manure" [https://github.com/AgriculturalSemantics/agro/issues/67]. The current entry [http://purl.obolibrary.org/obo/ENVO_01001116] would become a subclass of "bovine manure."
Rationale:
Unlike the other terms at this same hierarchy, such as pig manure, horse manure, and poultry manure, "bovine dairy liquid manure" contains information about not only the animal species, but also the animal sex, reproductive state, and end product (dairy) and the physical form of the manure (liquid). It is therefore at least a subclass of bovine manure.

I realized after submitting this issue here that I needed to submit it to the Environment Ontology, which I subsequently did.


Implement Pluggable Widgets 1 feedback

I tried to implement most of @r0b1n 's feedback. Could both @r0b1n and @Andries-Smit  review my changes? Also, I waited to implement the following points until Andries could weigh in. Please advise on these points:
Point 1: 3.1 Creating a Test Project
a. Press F4 to refresh your app.

It is not clear where I have to press F4. Add or select "Project > Synchronize Project Directory" in Studio Pro?

Point 2: 3.3 Adding the Attribute
2 Edit src/TextBox.xml to remove the 'sampleText' property and add the property 'Text attribute':

This feels slightly inconsistent, it uses sampleText which is a key of a prop with caption 'Default value' which is not mentioned. And then it ask to create an attribute with caption 'Text attribute'.

Point 3: Override the class lines in TextBox.tsx until they look like this

The code snippet looks incomplete. I miss exports of the class.

Would you both be able to align on this Monday? @r0b1n @Andries-Smit

I am going to resolve conflicts and do the final touches on the PR, then merge it 8/1 or 8/2.


:parrot: :link: Integrate E2B into LangChain

:parrot: :link: Integrate E2B into LangChain
To see all bounties and learn everything about the bounty program, go HERE.
Description
LangChain is a framework for developing applications powered by language models. LangChain already offers many integrations with other AI products.
Integrating E2B means using E2B sandboxes as a runtime environment for the agents created with LangChain.
Useful links
Each integration works differently, and it's up to you to think of the necessary steps. These resources might help you better understand LangChain and E2B.

Existing LangChain integration of E2B - data analysis use case
LangChain docs
E2B docs
Guide - using AI data analyst in cloud sandbox with LangChain & E2B

Existing integrations
For inspiration and to understand how the integrations may work, see:

E2B Sandbox used with OpenAI Assistants API
Custom Data Analysis Sandbox for LangChain
E2B as a tool for code execution in Superagent

I'm interested in this bounty

I'm interested in this bounty

Hey @dbmanifest, thanks! Do you want to start working on the bounty immediately, or learn more first?
You can jump on our Discord server to discuss your approach or ask us questions anytime. We are also happy to schedule a call if needed.
Let me know here or on Discord, and we can confirm your starting date.

Hey , I would to contribute to the  bounty , I have relevant past experience with langchain

The bounty program has undergone changes and this is no longer the way to apply for paid contributions.


Are complex variables supported in eager mode?

System information

TensorFlow version (you are using): 2.0.0-rc1
Are you willing to contribute it (Yes/No): No

Describe the feature and the current behavior/state.
Currently (unless I'm not doing things correctly) it seems that there is no support for complex variables. This fails:
    a = tf.Variable(4.0+1.0j) # a = tf.Variable(4.0) works

    def f(x):
      return x*a

    loss = lambda: tf.abs(f(5)-5)

    optimizer = tf.optimizers.Adam(learning_rate=0.1)
    optimizer.minimize(loss, [a])

WARNING:tensorflow:The dtype of the watched tensor must be floating (e.g. tf.float32), got tf.complex128

and

ValueError: Invalid type tf.complex128 for Variable:0, expected: [tf.float32, tf.float64, tf.float16, tf.bfloat16].

Will this change the current api? How?
Potentially yes
Who will benefit with this feature?
All of the users who used to rely on complex numbers!

I was able to reproduce the issue in 2.0.0rc2. Please check the gist here. Thanks!
Here is the error trace
WARNING:tensorflow:The dtype of the source tensor must be floating (e.g. tf.float32) when calling GradientTape.gradient, got tf.complex128
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
<ipython-input-23-f65531aefcec> in <module>()
      5 loss = lambda: tf.abs(f(5)-5)
      6 optimizer = tf.optimizers.Adam(learning_rate=0.1)
----> 7 optimizer.minimize(loss, [a])

2 frames
/usr/local/lib/python3.6/dist-packages/tensorflow_core/python/keras/optimizer_v2/optimizer_v2.py in _assert_valid_dtypes(self, tensors)
    830       if dtype not in valid_dtypes:
    831         raise ValueError("Invalid type %r for %s, expected: %s." %
--> 832                          (dtype, t.name, [v for v in valid_dtypes]))
    833
    834   def _valid_dtypes(self):

ValueError: Invalid type tf.complex128 for Variable:0, expected: [tf.float32, tf.float64, tf.float16, tf.bfloat16].

Unassigning, as I have no context on complex variable support. Sorry.

@fchollet what do you mean by context?

might be related https://github.com/tensorflow/tensorflow/pull/30107

@ziofil: That issue is slightly unrelated. I have a fix that add support for complex values in (some) optimizers that should be landing shortly.

@jaingaurav Is there a timeframe on this fix? I am currently looking at a fix myself and wonder if it is worth it.

@Davidvdrm: The fix is already committed and available in the nightly pip package. It will also be included in the upcoming 2.1 release.

I built the nightly package and still get a NotFoundError concerning the needed ApplyGradient ops for Adam, SGD and AdaDelta. This is the same issue as before the upgrade to the nightly build and I created a new issue https://github.com/tensorflow/tensorflow/issues/34098 concerning this problem. Should the change fix this issue or are they unrelated?
Full error code:
> NotFoundError                             Traceback (most recent call last)
> <ipython-input-41-0c0c662a0b5d> in <module>
>      26         # Optimize the model
>      27         loss_value, grads = grad(model, x, y)
> ---> 28         optimizer.apply_gradients(zip(grads, model.trainable_variables))
>      29
>      30         # Track progress
>
> ~\AppData\Roaming\Python\Python37\site-packages\tensorflow_core\python\keras\optimizer_v2\optimizer_v2.py in apply_gradients(self, grads_and_vars, name)
>     442           functools.partial(self._distributed_apply, apply_state=apply_state),
>     443           args=(grads_and_vars,),
> --> 444           kwargs={"name": name})
>     445
>     446   def _distributed_apply(self, distribution, grads_and_vars, name, apply_state):
>
> ~\AppData\Roaming\Python\Python37\site-packages\tensorflow_core\python\distribute\distribute_lib.py in merge_call(self, merge_fn, args, kwargs)
>    1947     if kwargs is None:
>    1948       kwargs = {}
> -> 1949     return self._merge_call(merge_fn, args, kwargs)
>    1950
>    1951   def _merge_call(self, merge_fn, args, kwargs):
>
> ~\AppData\Roaming\Python\Python37\site-packages\tensorflow_core\python\distribute\distribute_lib.py in _merge_call(self, merge_fn, args, kwargs)
>    1954         distribution_strategy_context._CrossReplicaThreadMode(self._strategy))  # pylint: disable=protected-access
>    1955     try:
> -> 1956       return merge_fn(self._strategy, *args, **kwargs)
>    1957     finally:
>    1958       _pop_per_thread_mode()
>
> ~\AppData\Roaming\Python\Python37\site-packages\tensorflow_core\python\keras\optimizer_v2\optimizer_v2.py in _distributed_apply(self, distribution, grads_and_vars, name, apply_state)
>     486           update_ops.extend(
>     487               distribution.extended.update(
> --> 488                   var, apply_grad_to_update_var, args=(grad,), group=False))
>     489
>     490       any_symbolic = any(isinstance(i, ops.Operation) or
>
> ~\AppData\Roaming\Python\Python37\site-packages\tensorflow_core\python\distribute\distribute_lib.py in update(self, var, fn, args, kwargs, group)
>    1541       kwargs = {}
>    1542     with self._container_strategy().scope():
> -> 1543       return self._update(var, fn, args, kwargs, group)
>    1544
>    1545   def _update(self, var, fn, args, kwargs, group):
>
> ~\AppData\Roaming\Python\Python37\site-packages\tensorflow_core\python\distribute\distribute_lib.py in _update(self, var, fn, args, kwargs, group)
>    2172     # The implementations of _update() and _update_non_slot() are identical
>    2173     # except _update() passes `var` as the first argument to `fn()`.
> -> 2174     return self._update_non_slot(var, fn, (var,) + tuple(args), kwargs, group)
>    2175
>    2176   def _update_non_slot(self, colocate_with, fn, args, kwargs, should_group):
>
> ~\AppData\Roaming\Python\Python37\site-packages\tensorflow_core\python\distribute\distribute_lib.py in _update_non_slot(self, colocate_with, fn, args, kwargs, should_group)
>    2178     # once that value is used for something.
>    2179     with UpdateContext(colocate_with):
> -> 2180       result = fn(*args, **kwargs)
>    2181       if should_group:
>    2182         return result
>
> ~\AppData\Roaming\Python\Python37\site-packages\tensorflow_core\python\keras\optimizer_v2\optimizer_v2.py in apply_grad_to_update_var(var, grad)
>     468       if "apply_state" in self._dense_apply_args:
>     469         apply_kwargs["apply_state"] = apply_state
> --> 470       update_op = self._resource_apply_dense(grad, var, **apply_kwargs)
>     471       if var.constraint is not None:
>     472         with ops.control_dependencies([update_op]):
>
> ~\AppData\Roaming\Python\Python37\site-packages\tensorflow_core\python\keras\optimizer_v2\adadelta.py in _resource_apply_dense(self, grad, var, apply_state)
>     139         coefficients['epsilon'],
>     140         grad,
> --> 141         use_locking=self._use_locking)
>     142
>     143   def _resource_apply_sparse(self, grad, var, indices, apply_state=None):
>
> ~\AppData\Roaming\Python\Python37\site-packages\tensorflow_core\python\training\gen_training_ops.py in resource_apply_adadelta(var, accum, accum_update, lr, rho, epsilon, grad, use_locking, name)
>    1100         pass  # Add nodes to the TensorFlow graph.
>    1101     except _core._NotOkStatusException as e:
> -> 1102       _ops.raise_from_not_ok_status(e, name)
>    1103   # Add nodes to the TensorFlow graph.
>    1104   if use_locking is None:
>
> ~\AppData\Roaming\Python\Python37\site-packages\tensorflow_core\python\framework\ops.py in raise_from_not_ok_status(e, name)
>    6596   message = e.message + (" name: " + name if name is not None else "")
>    6597   # pylint: disable=protected-access
> -> 6598   six.raise_from(core._status_to_exception(e.code, message), None)
>    6599   # pylint: enable=protected-access
>    6600
>
> ~\Anaconda3\lib\site-packages\six.py in raise_from(value, from_value)
>
>
> NotFoundError: No registered 'ResourceApplyAdadelta' OpKernel for 'CPU' devices compatible with node {{node ResourceApplyAdadelta}}
> 	 (OpKernel was found, but attributes didn't match) Requested Attributes: T=DT_COMPLEX64, use_locking=true
> 	.  Registered:  device='CPU'; T in [DT_HALF]
>   device='CPU'; T in [DT_BFLOAT16]
>   device='CPU'; T in [DT_FLOAT]
>   device='CPU'; T in [DT_DOUBLE]
>  [Op:ResourceApplyAdadelta]

`


unknownfieldexception

Hello
Thank you for useful library
I want to  use feature ignoreUnknownKeys = true
when using MsgPack.decodeFromByteArray  , I get an error  unknownfieldexception

This is a missing feature actually, I will try to make a 0.4.4 release including it.

thank you @esensar  🙏


How Long does It Take to Train Your Model

I saw from ur readme doc that you take ~100k images for training. I would like to know more abt the training details, such as:

What is the resolution of the training images?
What GPU device you use on training?
How long did you take to train ur model?

Thank you,
Alex

Hello!
Sorry for the late reply.
I will merge the discussion with #15; planning to perform a major refactoring of the project, then add detailed info regarding your questions.

Reopening.
Ref - https://github.com/alexandru-dinu/cae/wiki/Running-times.


adapter-cloudflare does not bundle service-worker.js since around next.207 - results in 404 and thus failed SW registration

Describe the bug
Since around next.207 adapter-cloudflare seems not to bundle service-worker.js
When testing locally, everything seems OK. The SW file is present in the ./svelte-kit/cloudflare builld folder - as it was with older versions of Sveltekit. When running npm run preview the SW is being successfully registered according to Chrome Dev Tools.
After deployment to CF, and running the site, the following error shows in the console log:
Uncaught (in promise) TypeError: Failed to register a ServiceWorker for scope ('https://sw-cloudflare-repro.pages.dev/') with script ('https://sw-cloudflare-repro.pages.dev/service-worker.js'): A bad HTTP response code (404) was received when fetching the script.
Also, according to Dev Tools, the SW is not present.
Reproduction
https://github.com/codepainting/sw-cloudflare-repro
Basically it's the standard demo app, with the following additions:

service-worker.js is included in /src
import adapter from '@sveltejs/adapter-cloudflare'; is configured

Demo site:
https://sw-cloudflare-repro.pages.dev/
Run demo site.
Look into the console logs.
Logs
Uncaught (in promise) TypeError: Failed to register a ServiceWorker for scope ('https://sw-cloudflare-repro.pages.dev/') with script ('https://sw-cloudflare-repro.pages.dev/service-worker.js'): A bad HTTP response code (404) was received when fetching the script.

System Info
System:
    OS: Linux 4.19 Ubuntu 20.04.3 LTS (Focal Fossa)
    CPU: (4) x64 Intel(R) Core(TM) i5-7600K CPU @ 3.80GHz
    Memory: 19.42 GB / 25.03 GB
    Container: Yes
    Shell: 5.8 - /usr/bin/zsh
  Binaries:
    Node: 16.13.0 - ~/.volta/tools/image/node/16.13.0/bin/node
    Yarn: 1.22.4 - ~/.volta/bin/yarn
    npm: 8.1.4 - ~/.volta/tools/image/npm/8.1.4/bin/npm
  npmPackages:
    @sveltejs/adapter-auto: next => 1.0.0-next.7
    @sveltejs/adapter-cloudflare: ^1.0.0-next.5 => 1.0.0-next.5
    @sveltejs/kit: next => 1.0.0-next.213
    svelte: ^3.44.0 => 3.44.3

Severity
blocking an upgrade
Additional Information
No response

This behavior is happening in adapter-vercel as well.

The issue seems to be with the internal manifest of the workers.
export const manifest = {
	appDir: "_app",
	assets: new Set(["favicon.png", /*removed for brevity...*/,"svg/Youtube.svg","svg/YoutubeRed.svg" /* no service-worker.js :( */]),
	_: {
		mime: {".png":"image/png",".jpg":"image/jpeg",".svg":"image/svg+xml"},
		entry: {"file":"start-d1b74b63.js","js":["start-d1b74b63.js","chunks/vendor-873c0a70.js"],"css":["assets/start-d5b4de3e.css"]},
		nodes: [
			() => import('../output/server/nodes/0.js'),
			() => import('../output/server/nodes/1.js'),
			() => import('../output/server/nodes/2.js'),
			() => import('../output/server/nodes/3.js'),
			() => import('../output/server/nodes/4.js'),
			() => import('../output/server/nodes/5.js'),
			() => import('../output/server/nodes/6.js'),
			() => import('../output/server/nodes/7.js'),
			() => import('../output/server/nodes/8.js'),
			() => import('../output/server/nodes/9.js'),
			() => import('../output/server/nodes/10.js')
		],
		routes: [
			{
				type: 'page',
				pattern: /^\/$/,
				params: null,
				path: "/",
				a: [0,2],
				b: [1]
			},
			/*...*/
			]

This is created from builder.generateManifest (link to impl), and does not seem to include service workers.

I stumbled into the same issue  a few days ago. My current workaround is to add an additional service-worker.js into  folder /static, such that the generated service worker will be included in the final bundle.
(Yes, THE SERVICE WORKER being served is compiled from src/service-worker.ts rather than static/service-worker.js)
the demo: https://pwa-test-1mm.pages.dev/service-worker.js
the repo: https://github.com/Envl/pwa-test
Notice the code in static/service-worker.js is different from what that is in src/service-worker.ts. It seems what's inside  static/service-worker.js doesn't matter as long as it exists.

Great to see work dedicated to this issue. Unfortunately, the update seems to have no effect - at least in the case of Cloudflare, which I was testing thoroughly (different browsers to avoid SW caching problems).
Reproduction:
I updated my initial demo site with a clean new version of the Sveltekit demo app.
When the service-worker.js file is included in /src alone, the following error persists:
A bad HTTP response code (404) was received when fetching the script.
sw-cloudflare-repro.pages.dev/:1 Uncaught (in promise) TypeError: Failed to register a ServiceWorker for scope ('https://sw-cloudflare-repro.pages.dev/') with script ('https://sw-cloudflare-repro.pages.dev/service-worker.js'): A bad HTTP response code (404) was received when fetching the script.

But I can confirm that the workaround proposed by @Envl is working also with the updated Sveltekit.
I added another copy of the service-worker.js file into the /static folder and console logged respectively to see which version is being used when deployed on Cloudflare.
I can confirm that the sheer existence of a service-worker.js file in the static folder somehow triggers the use of the SW, and it is     the one from /src and not from /static.
So while the workaround seems valid also with the new version of Sveltekit, the issue unfortunately seems (at least on Cloudflare) not yet resolved.


action_text_value property type must be object,null

I'm working on setting up this repo to run tests against the Bullet Train starter repo and in the process I've discovered that v2.6.4 of this gem is causing errors in the starter repo.
There are several errors like this:
[2] tmp/openapi.yaml:1650:38 at #/paths/~1scaffolding~1absolutely_abstract~1creative_concepts~1{absolutely_abstract_creative_concept_id}~1completely_concrete~1tangible_things/get/responses/200/content/application~1json/example/0/action_text_value

Example value must conform to the schema: `action_text_value` property type must be object,null.

1648 | super_select_value: three
1649 | text_area_value: MyText
1650 | action_text_value: MyText
1651 | created_at: '2023-01-01T00:00:00.000Z'
1652 | updated_at: '2023-01-02T00:00:00.000Z'

referenced from tmp/openapi.yaml:1630:15 at #/paths/~1scaffolding~1absolutely_abstract~1creative_concepts~1{absolutely_abstract_creative_concept_id}~1completely_concrete~1tangible_things/get/responses/200/content/application~1json

Warning was generated by the no-invalid-media-type-examples rule.

And one like this (which might be normal?):
[1] tmp/openapi.yaml:11:10 at #/servers/0/url

Server `url` should not point to example.com or localhost.

 9 |   version: "V1"
10 | servers:
11 |   - url: http://localhost:3001/api/v1
12 | components:
13 |   securitySchemes:

Warning was generated by the no-server-example.com rule.

You can see a failing run here: https://github.com/bullet-train-co/jbuilder-schema/actions/runs/6907378005/job/18795175127#step:18:53

For the first warning, there is an issue in Jbuilder, that instead of ERB, which calls to_s on ActionText objects, it doesn't do it and parses the whole object as it is.
We already monkey patch Jbuilder to return us strings instead of objects for ActionTexts, but it didn't work for Jbuilder::Schema, as it also overwrites Jbuilder methods. Here I added a PR which patches Jbuilder::Schema the same way, I think with it these errors should go.
--
I tried to avoid parsing ActionText in Jbuilder::Schema directly to mimic the original Jbuilder behaviour.
Thinking on how to update Jbuilder better, because also want to avoid adding dependencies.


Proposal: Add a syntactic sugar for 'this' or/and optionally relax its usage when possible.

Background
One of the things that I really dislike about TypeScript is that whenever I need to point to a field or a function I need to use this.something, the this keyword is used heavily whereas in other languages I can avoid it by doing _something.
Clarification: by avoiding it I really mean that the compiler can figure it out for me and free me from having to type it.
I fully understand the need for it but can't we do better than having to type it all the time? it's quite verbose.
So in order to reduce the amount of verbosity in the code I propose a syntactic sugar to it.
Syntax
I thought of two syntactic sugars that can be attractive, it's very subjective but we can have a discussion about the style if you don't like it.
I've added a 3rd syntax that I know that many of us would prefer over any syntactic sugar where the compiler does the dirty work for us.
So here is an example to a class that does nothing useful but it's for the purpose of demonstrating the syntax.
class Message {
	constructor(public message: string = "Empy message.") {
	}

	public toLowerCase() {
		this.message = this.message.toLowerCase();

		return this;
	}

	public print() {
		console.log(this.message);
	}
}

1st syntax (at sign):
class Message {
	constructor(public message: string = "Empy message.") {
	}

	public toLowerCase() {
		@message = @message.toLowerCase();

		return @;
	}

	public print() {
		console.log(@message);
	}
}

2nd syntax (period):
class Message {
	constructor(public message: string = "Empy message.") {
	}

	public toLowerCase() {
		.message = .message.toLowerCase();

		return .;
	}

	public print() {
		console.log(.message);
	}
}

Notes

When it's used before a variable it's going to be emitted as this.
When it's used after a return statement then it's going to be emitted as this

3nd syntax (no this unless we need it.):
This is the preferable syntax that we would like to write because it's close to how we would write this in other languages.
For this to be possible the compiler will need to be smarter so if something is not declared in the scope of the class/function then the compiler can assume we're accessing a global otherwise it can assume we're accessing a local and emit this. for it; if something is declared within a local scope but it also exists at the global scope then we will need to use the existing this keyword to access the local scope, once we use this to access something the compiler will assume that something without this is global and therefor it won't emit this. for it.
I'm sure that there are some challenges with this but this is really the desired syntax that many would prefer over any syntactic sugar.
class Message {
	constructor(public message: string = "Empy message.") {
	}

	public toLowerCase() {
		message = message.toLowerCase();

		return this;
	}

	public print() {
		console.log(message);
	}
}

Dear Mr. eyalsk,
I'm sorry that I have to be the one who has to tell you. But 2 of of your 3 suggestions suck.
Your first suggestion (Using the "@" character) is a total disaster. Do you know where they did hide that character on a german keyboard? I bet not. I'm ten times faster in typing a "this" than entering the right key combination for the "@" character.
You second suggestion isn't much better. I can hear the programmers crying when they find out that the bug they hunted for days, was a tiny dot which they forgot somewhere in 5000 lines of code. By the way, did you ever try a search and replace for a dot? That's no fun, I can tell you that. If this dot has a special meaning in your code you have to do that some day. That's for sure.
So there is one suggestion left. (Using no this or that or whatever at all) That one looks fine to me. But as long as the 'this' keyword (the current context) is such a beast in javascript I don't expect that before the next millennium. But hey, there should always be a place for dreaming.
Kind regars, Lord Saumagen

@lord-saumagen, you don't need to be sorry and if you read carefully, I intentionally used the word can and also noted that we can have a discussion about the style.
I'm not German so I couldn't possibly know anything about the layout of a German keyboard but if I was I'd probably picked something else that made more sense.
I agree that a period or at-sign isn't the most attractive and I might not be as creative as some people but it was just to have something instead of nothing.
I don't believe anyone in his right mind would use find and replace to change any code at this point in time! refactoring is much more reliable so I wouldn't worry about that.
Luckily TypeScript is a superset of JavaScript, so we can build concepts that don't exist in JavaScript and still emit valid JavaScript so I proposed two styles that might make sense and one that I know from experience that people would really like because it's closer to how we would write it in other languages.
p.s.

The third syntax would be a huge change in mental model.
Classes in js/ts is just a fancy syntax to put stuff on the prototype and the prototype is not in scope of the methods. By not requiring this the prototype would somehow look to be in scope without actually not being. That feels weird to me.

Dear Eyal Solnik,
I beg your pardon for the use of the rude expression. I didn't know that this expression might actually hurt somebodies feelings. My understanding of the English language is base on Hollywood action movies. May be that's not the best reference for a civilised conversation.
My point is, that this feature you suggested wouldn't help anybody in anyway. (In other words, it's totally pointless.)
When you use a development environment which earned that name, you don't save that much typing.
I'll give you an example.

So, its two keystrokes to type a this. First you type a "t" and than you hit the return or dot key. That depends on what you want to express. You can't shorten that much more. (Except for the two key strokes you still have to do.) Do you really think it's a good idea to add a compiler function to shorten that amount of work down to one key stroke?
To be honest, I didn't type a complete keyword or classname for ages. Even the longest class name shows up after 3 matching characters. Code completion became my best friend. It did not only reduce the amount of keystrokes I have to do, it also reduced the amount of misspelled words in my code.
How should a compiler function which can save you 2 keystrokes on only one keyword impress anybody who uses code completion?
So you must agree that this is only a wast of time. You should try code completion yourself and you will see there is already a solution for your problem.
Kind regard, Lord Saumagen.

@Pajn, TypeScript already supports many things that are not part of the mental model that a typical JavaScript programmer understands, given that the programmer is programming only in JavaScript.
Many of the features in TypeScript, many of the features of ES6 and ES7 is already a huge shift in the mental model.
However, to programmers that are coming from other languages that supports multi-paradigms especially OOP it isn't so odd to ignore the pointer whenever it's redundant.
What you're actually implying is that because classes are implemented through prototypical inheritance behind the scenes the programmer needs to use the exact same mental model but the truth is we shouldn't, unless again you're programming in JavaScript (ES5).
Now what I'm proposing is completely optional! if you will use this. the compiler will do the exact same thing as before, what I'm asking is basically for the compiler to be slightly smarter for these of us who want to ignore it possible.
Maybe we can add an explicit flag for it in the compiler for these of us who wants it.
@lord-saumagen, I understand, no offense taken.
Okay, I understand where you're coming from but then when I proposed it I didn't really thought about intellisense and the amount of keystrokes but making the code more readable and reduce some of the noise that exists in the code.
A syntactic sugar usually means an alternative syntax or an addition to the current syntax not in place of the existing syntax, so you can always use the old syntax and it will work just fine!
I'm using many editors and many IDEs due to my work with multiple languages so I do know about code completion and intellisense but then again, my proposal is about making the code more readable, we can speak about the style to do it later.
I don't believe it's a waste of time but everyone is entitled to his own opinions, I have mine, you have yours. :)

Dear Eyal Solnik,
so it's not the keystrokes which give you that bad mood, it's a crusade against the keyword itself. (That's a joke).
You are telling that you want to make the language easier to read and to understand. That's a noble motivation, but have you proof that reducing the verbosity, noise, ambiguity, redundancy or how ever you call it has that effect?
Because every linguist in the world would tell you the opposite. The fuzziness and redundancy in human languages makes it easier for a human mind to understand a language. Because that gives your more options in the interpretation of a text or speech. New words are invented every day. Words come and go and some of them come back after ages. Others are in a constant change of meaning and every vocable can change it's meaning into the complete opposite depending on the context or situation it is used in. Even the grammatic, phonetic and the rhythm is changing. Human languages are in constant flux, but the human brain is able to handle that.
But there is on thing a human brain can't do. As Wittgenstein pointed out, humans can only think of things they have a word for. The human brain needs words to focus thoughts, ideas, concepts or other abstractions. If you don't have a word for a thing or abstraction, you don't have a chance to get a grip on it. Your mind is simply not able to handle such a situation. That's the way a human brain works.
So, if you eliminate the "this" keyword, you are also going to hide an important concept in the javascript language. That doesn't make it easier to understand the code. The opposite is true. Because without the keyword, you have to make assumptions about the current context inside a function class or program. But the anchor to fix that assumption and proof it is missing. So you will lose the track. You can't avoid that. As soon as the word leaves your sight, the abstraction it stands for is draining from your brain.
Programming languages are not made for machines, they are made for humans.  Computers interpret sequences of differences in electric voltage.
Of course it seems to be a logical assumption that programs become easier to understand or read when you go to reduce the amount of sentences, words, characters or other building blocks of a language.
But the human mind isn't working logical. That is the reason why your concept fails.  You can't tell a story by simply listing the facts. There has to be noise in a human language, because that is the glue between the facts that give them meaning. What you are going to do is trying to make the language easier to understand for computers. But no computer asked for that.
You may still disagree with my opinion, but at least you should look for a proof of your assumption. Changing the compiler because somebody things that may be a good idea is pointless.
Kind regards, Lord Saumagen.

I think this proposal runs into some of the non-design goals of TypeScript.  In particular:

Exactly mimic the design of existing languages. Instead, use the behavior of JavaScript and the intentions of program authors as a guide for what makes the most sense in the language.

TypeScript is a superset of JavaScript.  Most of this proposal would be surprising to a JavaScript developer.  We have to deal with the reality of this in the underlying JavaScript and the way JavaScript behaves.  TypeScript tries not to break the way JavaScript operates, but provide a type safe way of which to develop JavaScript.

@lord-saumagen I'm not doing crusades here and I'm not against the keyword itself! you clearly missing the point of my post.
I don't need a proof to make a proposal, certainly not this kind of proposal, it bothers me that I know that the compiler can be smarter when it comes to again, do the dirty work for me and I need to type it again and again regardless to what you guys think.
It's plain logic, reducing the amount of repeated keywords in the code reduces its verbosity, you don't need to be a genius to know that although I agree verbosity does not necessarily mean readable code but in this case, in my opinion, it is.
I'm not eliminating or trying to eliminate anything, this still exists and you can still use it, what I'm proposing is completely optional! and like I said in my previous post it's possible to even add a flag to turn this feature for some of us who don't need the pointer to know where we are.
Please don't go off-topic here because what you said is purely philosophical and these discussions never ends, it's also not the place here, even though I understand what you're trying to say, it doesn't hold any water in this case.
I disagree with most of the philosophical things you said though because pardon my language it's nonsense.
@kitsonk, I don't think so because I'm not trying to mimic exactly the design of existing languages, I'm trying to relax the typing when it comes to the this keyword whenever possible in order to reduce some of the noise in the code and copy/paste JavaScript to existing TypeScript file should still work.
I wrote that the 3rd syntax is close to how we would write it in other languages but I never said it should or would behave the same.

@eyalsk I guess I am just wondering what the value is in making something more confusing to save 4 characters of typing which will have to be emitted anyways.  I am assuming you would propose what you are proposing as an opt in, which would make it only more confusing for those that choose to opt in and opt out.  If you propose it as mandatory, you will break just about everyones code and directly conflicts with TypeScript being a superset of JavaScript.
Option 1 directly conflicts with decorators.
Option 2 is confusing in my opinion.
Option 3, how do you propose to handle this?
var message;

class Message {
    constructor(public message: string = "Empy message.") {
    }

    public toLowerCase() {
        message = message.toLowerCase(); /* which message? */

        return this;
    }

    public print() {
        console.log(message);
    }
}

@kitsonk, I think I've covered this in my post in case of ambiguity you will have to use this to make it work.
However, yeah, I see the place for confusion and even more when people share their code, if person A shares his code with person B and person B be didn't opt-in, it can cause some undesired side-effects so this feature need to be all in or all out to make it work and that ship has sailed long time ago.
Admittedly, I proposed it to see whether people will like the idea or not, I didn't know people will take it so personally!

@eyalsk I am not taking it personally.  Just because I don't like the idea and am trying to express reasons why I don't like the idea doesn't mean I am taking it personally.  Hopefully by asking peoples thoughts you didn't expect everyone to agree with you.

@kitsonk it was a general statement not directed at you, at all.
I don't mind people to disagree with me, in fact, I don't even mind to be wrong and I certainly respect people's opinion, so it's all good.

Thanks for the suggestion @eyalsk, it's always worth discussing. :smiley:


Django 4.0+ deprecated django.utils.encoding.force_text

The following error occurs with Django 4.0+ because force_text has been changed to force_str:
File "/usr/local/lib/python3.9/site-packages/nested_inline/admin.py", line 13, in <module> from django.utils.encoding import force_text ImportError: cannot import name 'force_text' from 'django.utils.encoding' (/usr/local/lib/python3.9/site-packages/django/utils/encoding.py)
The issue is documented there:
https://docs.djangoproject.com/en/4.0/ref/utils/#module-django.utils.encoding
According to this articicle we need to change to 'force_str'

@jnegro It is fixed in and in master, we are just awaiting a release of 0.4.5
You can use git+https://github.com/s-block/django-nested-inline.git@617f5aa8ce940688f702e0b32a30816473c54ab4´ in your requirements` file for the time being.

Thanks for the quick response.  Sorry I did not see it within that ticket!

Just made a release to pypi. You can now download version 0.4.5

Thank you for the update!
On Thu, Apr 7, 2022 at 6:28 PM Daniel Alexander Brenot <
@.***> wrote:

Just made a release to pypi. You can now download version 0.4.5
—
Reply to this email directly, view it on GitHub
https://github.com/s-block/django-nested-inline/issues/141#issuecomment-1092268549,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAHKYXHMSWG4UNCS7CE2JTTVD5OP7ANCNFSM5SX3DTWA
.
You are receiving this because you modified the open/close state.Message
ID: @.***>


Pointer type support

Unfortunately sometimes you do need to use raw pointer types (especially when dealing with Qt on the C++ side). cxx should support T* -> *mut T, const T* -> *const T (with special case void* -> *mut () / const void* -> *const ()). Currently I'm circumventing this restriction by using usize as a pointer type but that's kind of a major hack :P

You can also use UniquePtr as an alternative hack. https://docs.rs/cxx/0.5.10/cxx/struct.UniquePtr.html#method.from_raw, https://docs.rs/cxx/0.5.10/cxx/struct.UniquePtr.html#method.into_raw and C++ side equivalents.

Pointer support is tracked in #164 so I'll go ahead and close this as a duplicate. I am definitely interested in this feature -- I think this just needs a PR...

Ah, my bad. I did look for open issues regarding this but must have missed that one.


Improve Backend Onboarding Documentation

Devs will need to have SLS installed to run the backend. Make note of this in the README
npm install -g serverless

Moving this to the backend repo


Switches to use npm

Hi,
Had some trouble getting this running - looks like it's a yarn issue: https://github.com/storybookjs/storybook/issues/22431
I think the old node_modules style of yarn is near-deprecated at this point it probably makes sense to move to npm, plus it fixes that issue
I think I got everything that needs to move over including docs

Im going to make this change across all the repos today and release them w/ npm.

Thanks for doing this - I ended up doing this as part of another PR.


Migrate to Swift 2

Blocks #69. Migrate over so that we can pull in the new version of XcodeServerSDK.

Hi Honza,
are you currently working on this? Because I have it running in a very hacky state (and I'm especially unhappy with the way I did the new error handy part). And also I based it on your experimental Xcode 7 brach, so before the XcodeServerSDK was created.
To make a long story short: I'm thinking about adding atlassian stash support to Buildasaur. So if you are working on Xcode 7 I will just wait, otherwise I could also put some effort into Xcode 7.

Hi @sofacoder, yeah actually last night I just got it to compile with Swift 2. But there are a couple of things I had to break (specifically the way you pick test devices) due to changes in XcodeServerSDK during migration to Xcode 7 API. I will be working on this branch now to fix them and mostly things in the UI will change.
To answer your question - if you'd like to add more git server support, feel free to start working of this branch, because it's already migrated to Swift 2 and it uses XcodeServerSDK. I plan no changes to the GitHub communication code, so that's probably what you'll be changing (somehow parse out common code between GitHub and Stash and add the differences).
And please keep me in the loop so that I know what parts of code to try to avoid changing now. And thanks for helping out with Buildasaur! :)

Ah, should have looked at the commit before commenting here. ;-)
Nice to see that you have this already working.
Ok, I will fork an base my stuff at this branch. No time indication yet, since the weather is good this week. ;-)

Yeah, I myself was out playing frisbee when you commented here (guilty as charged) :+1:

Migrated to Swift 2 on branch hd/swift-2.


MapOverlay support on ios and android

Adding ios google maps support
Adding android support

Hello everyone,
I should have checked there are already few pending PRs with same implementation.
I think this PR, if I am right then only this PR supports all the platforms in one.
It's been a good learning experience with the MapKit and Google Maps.
Please comment on this. If I need to continue working or not. Appreciate your time.
This is my first big contribution to a major project in Open Source. Looking forward for feedback.

Any update on this pull-request

I think there's other similar pull requests, would love to see them merged.
GJ @srikanthkyatham

I think since #2022 all the platform are supporting Overlay. @rborn should we close this one ?

@srikanthkyatham thanks a lot for your PR 🤗
Somehow there were other PRs with the same funcionality pushed so I had to close this.


Cleanup Go source code, bring back gofmt

Fixing the issue with removal of gofmt binary.
More cleanup to decrease image size.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

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

Needs approval from an approver in each of these files:

OWNERS [abhay-krishna]

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

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

Needs approval from an approver in each of these files:

OWNERS [abhay-krishna,jaxesn]

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


Installation issue

Hi
I can't get it installed on my phone, it shows "App not installed as package appears to be invalid".
Is there any workaround?
Thanks!

Checkout app-debug.apk

Hi, I am Mohsin Shahid from FAST National University in Pakistan, I am making a similar application to you that blocks or locks the application. But, we are facing some permission issues, so we wanted to try your application but we are facing some errors while running your application. Are there any dependencies that we want to fulfill before running your application?. Kindly guide us through that how can we run your application. Thank you.


Fix SAMP_NUM and SAMP_SIZE feature arg documentation in README

In README.md, it claims that SAMP-NUM and SAMP-SIZE are the correct argument to flow2features(). However, it should be SAMP_NUM and SAMP_SIZE

Thank you!


What's the output node names?

I tried to convert model to frozen graph, but couldn't find the output names need in freee_graph tool
freeze_graph --input_saved_model_dir=saved_model --output_node_names= --output_graph=frozen_graph.pb
thanks a lot!

Last layer is a dense layer, shooooould just be called dense
https://github.com/bumble-tech/private-detector/blob/32c04fab0404dca1566cfe37aa2c9110b199f80c/private_detector/utils/effnetv2_model.py#L569-L572
>>> print(model.model.layers[-1].name)
dense

Does that work?

Thank you
it doesn't work out, here is the error message:
AssertionError: dense is not in graph

here is the script I used to do convert
from tensorflow.python.tools import freeze_graph
from tensorflow.python.saved_model import tag_constants

input_saved_model_dir = './saved_model'
output_node_names = 'dense'
input_binary = True
input_saver_def_path = False
restore_op_name = None
filename_tensor_name = None
clear_devices = False
input_meta_graph = False
checkpoint_path = './saved_checkpoint/ckpt-0.9375-14.data-00000-of-00001'
input_graph_filename = None
saved_model_tags = tag_constants.SERVING
output_graph_filename='frozen_graph.pb'

freeze_graph.freeze_graph(input_graph_filename,
  input_saver_def_path,
  input_binary,
  checkpoint_path,
  output_node_names,
  restore_op_name,
  filename_tensor_name,
  output_graph_filename,
  clear_devices,
  "", "", "",
  input_meta_graph,
  input_saved_model_dir,
  saved_model_tags)

solved as per #7


Update 150 using prisma with cockroachdb to fix #5888

removing native type declaration on line 5 of prisma example schema causing " "Native type VarChar is not supported for cockroachdb connector." error. This would fix issue https://github.com/prisma/docs/issues/5888

Thank you again for your contribution @IAmLuisJ !


Cannot build Android App on Bitrise

Hi,
when I try to build my Android App on Bitrise I got the following error:
Could not determine the dependencies of task ':app:collectLogBuchReleaseDependencies'.
> Could not resolve all task dependencies for configuration ':app:logBuchReleaseRuntimeClasspath'.
   > Could not resolve eu.simonbinder:sqlite3-native-library:3.32.3.
     Required by:
         project :app > project :sqlite3_flutter_libs
      > Could not resolve eu.simonbinder:sqlite3-native-library:3.32.3.
         > Could not parse POM https://bintray.com/larpoux/CanardouxMaven/eu/simonbinder/sqlite3-native-library/3.32.3/sqlite3-native-library-3.32.3.pom
            > The markup in the document preceding the root element must be well-formed.
      > Could not resolve eu.simonbinder:sqlite3-native-library:3.32.3.
         > Could not get resource 'https://google.bintray.com/exoplayer/eu/simonbinder/sqlite3-native-library/3.32.3/sqlite3-native-library-3.32.3.pom'.
            > Could not HEAD 'https://google.bintray.com/exoplayer/eu/simonbinder/sqlite3-native-library/3.32.3/sqlite3-native-library-3.32.3.pom'. Received status code 403 from server: Forbidden
      > Could not resolve eu.simonbinder:sqlite3-native-library:3.32.3.
         > Could not get resource 'https://dl.bintray.com/sbinder/sqlite3-native-library/eu/simonbinder/sqlite3-native-library/3.32.3/sqlite3-native-library-3.32.3.pom'.
            > Could not HEAD 'https://dl.bintray.com/sbinder/sqlite3-native-library/eu/simonbinder/sqlite3-native-library/3.32.3/sqlite3-native-library-3.32.3.pom'. Received status code 403 from server: Forbidden

This does not happend before (Last build on Friday).
My versions of moor are:
moor: 3.1.0
moor_ffi: ^0.8.0
Can you help me with this?

I think you may need to update moor and/or moor_ffi.
Bintray is no longer. See https://jfrog.com/blog/into-the-sunset-bintray-jcenter-gocenter-and-chartcenter/
It may work locally due to cached dependencies.
You can also try adding a newer version of https://pub.dev/packages/sqlite3_flutter_libs directly.
This was changed in https://github.com/simolus3/sqlite3.dart/commit/2b0c1ffa9e869144680e4b31914d7c81d7044397#diff-30bdf6c5328f273f91b808d80d0401195a367700e0854c7363956a2c4e553182

Ok. Thanks for your quick feedback :)

Note that the change to simonbinder.eu/maven was temporary (never published to pub) and no longer works. But sqlite3-native-library is on maven central now, so if you upgrade your version of sqlite3_flutter_libs (use 0.4.0+1 or later) it should work.

@simolus3 , is it possible to publish a version that of sqlite3_flutter_libs that fixes this issue but doesn't require Dart > 2.12? We have an app from last year that we are unable to migrate to null safety yet, but need to publish an update for, and can no longer do so through CI due to this issue :/

Please note that you can use Flutter 2/Dart 2.12 without migrating your app to null safety, just keep something like this in your pubspec:
environment:
  sdk: '>=2.11.0 <3.0.0'

Regardless, version 0.4.0+1 of sqlite3_flutter_libs  uses the new libraries from Maven Central without having a tight lower sdk constraint.


[Feature Request]: Upload image from HA instead of using a URL

The request
The example code given to send an image is this :
service: whatsapp.send_message
data:
clientId: default
to<EMAIL_ADDRESS>body:
image:
url: "https://dummyimage.com/600x400/000/fff.png"
caption: Simple text
I want to send a snapshot on my media folder of homeassistant.  Since HA is not available to the public I can't use a URL
Is your feature request related to a problem?
No response
Additional information
No response

Nevermind.  I was able to figure it out.  In a nut shell I ended up using www folder and added it to external dirs via yaml.

thank you for such a great add on


Make opencv optional

cv2 (formerly cv) could be removed:

I'll definitely check this out within the next few days. I'm currently working on something big: removing cv2 as a dependency.

-https://github.com/anrayliu/pyvidplayer2/pull/38#issuecomment-2292653317
but maybe make it optional, since it is handy for advanced operations on the video.
Such as:
In the module:
ENABLE_CV2 = True
try:
    import cv2
except ImportError:
    ENABLE_CV2 = False
. . .
if ENABLE_CV2:
    # ... do stuff
if ENABLE_CV2:
    # ... more stuff in some other part of the code

...then split off a different requirements.txt called requirements-opencv.txt and just add opencv_python to that.
And pip install pyvidplayer2[opencv] can be defined if you do:

setup.py:

    extras_require = {
        'opencv':  ["opencv_python"]
    }

pyproject.toml:

[project.optional-dependencies]
opencv = ["opencv_python"]

However, I didn't get a good look at the part of the code that actually requires opencv, so if there is some way for me to use opencv on the pyvidplayer2 video efficiently without the pyvidplayer2 module using opencv, that's fine too.

Cv2 will indeed be an optional package. It seems like ffmpeg can do almost everything that cv2 can, except ffmpeg can also accept videos in memory. My biggest motivation for removing cv2 is because that would reduce the dependencies down to just pygame and ffmpeg, which would be really simple and lightweight. Also, what kind of video operations were you thinking of doing with cv2?

Well I want to use pyvidplayer2 for some simple video editing (I know all the drawbacks, I am wanting to make a unique rotoscoping program, probably not an NLE. A rationale is at the readme here: https://github.com/Hierosoft/rotocanvas). I'm likely to want to want to have:

Scene detection: https://github.com/Breakthrough/PySceneDetect
Object tracking: https://www.geeksforgeeks.org/getting-started-with-object-tracking-using-opencv/

What kind of interaction would you need with the video and cv2 for this? Reading video frames?

The examples I gave are python_opencv examples, but only used opencv briefly. When I save the video, I will merely use ffmpeg. pyvidplayer2 won't be involved in that step I don't think.
Therefore, I don't know enough yet. I don't know if I will need to use opencv with pyvidplayer2. I will likely just use opencv to analyze the source video apart from the GUI, save the metadata, then overlay my own widgets based on the metadata (such as boxes the user can move if the object tracking is wrong).
The reason pyvidplayer2 is useful regardless of that is:

pyvidplayer2 could preview ffmpeg filters (and maybe opencv filters) as part of the realtime preview. I don't even know how much control I will have over ffmpeg without subclassing pyvidplayer2 heavily. If I find a neat way to do it and put most of the code in my program, I can submit a small PR. I haven't looked at the ffmpeg part of pyvidplayer2's code much yet either, so I don't know what that would take.
pyvidplayer2 can use the same timecodes as ffmpeg uses so I assume I can do frame-accurate editing (Though I understand I may have to re-encode portions if using mp4 and similar formats due to keyframe issues etc). In other words, if I can go to a timecode in pyvidplayer2, I can use that same timecode in ffmpeg commands. Therefore I can add markers or cut points that I can store in a project and later pass along to ffmpeg. Unedited clips will remain lossless (-c:v copy -c:a copy). Then I will splice the edited clips with the unedited clips.

My old program rotocanvas-cs very slow as it was using the ffmpeg command & saving each preview to a file (I wasn't good at using stdout back then, but should have probably used a library anyway): https://github.com/Poikilos/rotocanvas-cs. It used the code from another repo for the timecode construction function (can be used in -ss option in ffmpeg command for cutting): https://github.com/Poikilos/RetroEngine-cs/blob/f43c73a24b01ea37fbb5c1fa1d86c03031ef01bf/RConvert.cs#L257 . This code goes to the "middle" of the frame. As in, if it is 60 fps, to go to the second frame I have to convert 1.5/60 seconds to a timecode that works reliably for -ss (1.0/60 gets the previous frame all or most of the time). Maybe I made some other mistake, but that is the only way I could get it working.

Interesting. Have you browsed the documentation for things that may help you? For example I just added (hasn't been pushed to pypi yet but is available in the repo) a seek_frame function, which should quite accurately seek to a frame. In addition, frame_data and frame_surf both contain the video's current rendered frame as a numpy array and pygame surface, respectively. Also, check out the built-in post processing (which is what I was planning to have cv2 be an optional package for). However, yes, there isn't an audio post processing feature yet. How pyvidplayer2 uses ffmpeg is inaccessible to users.

For accuracy I should say I found out today that AVIDemux supports "smart rendering" (a combination of filtered clips and lossless clips): https://www.avidemux.org/admWiki/doku.php?id=using:video_filters


Consider upgrading to v3 of API

I'm not sure if this package is still maintained, but it looks like WPVulnDB recently opened up v3 of their API which requires all requests to be sent with an auth token (you are still using v1).
I reached out to the WPScan team who have indicated that they won't be maintaining the older versions indefinitely... It is unclear if they will be blocking access completely.
Maybe the token could be accepted as a command line option (e.g. --token=asd123) and the readme could encourage users to set a default value via WP-CLI config?

Hi @ssnepenthe thanks for your suggestion!
I tried out v2 and it seems requests can be done without an authorization token too. Maybe we can update the package to use v2 instead of v1 if the WPScan team decides to stop maintaining v1. If they also stop maintaining  v2, then your suggestion sounds like the most reasonable step to follow.
Definitely something we can improve on!

Yeah looking back at this there probably is no rush considering v1 is still available. Should this issue be closed altogether?

v1 seems to be no longer available and the endpoint returns 404. At the same time, wp vuln scan happily reports no issues.
We need to update the package for it to work again. A token parameter seems the proper way to address this.

Seems like this is done but the issue was left open: https://github.com/10up/wp-vulnerability-scanner/commit/59bc7422ee29cf63fc78273d5034e59211f4a656


Deduplicate checks in ld16 macro

Structure macro a little better to de-duplicate some checks

Fixed in another branch.


Sampler app - mobile toggle switch does not work in some instances

This is in the Sample App from OpenNTF included in the download. Recreate using Chrome on desktop.
Open Contacts and select Add from list. 'Mobile Enabled' green toggle switch works.
Close new doc, click/tap on an entry and then re-tap Add; 'Mobile Enabled' green toggle switch no longer works - may have to do this for a couple of entry to see it not working.

Could you have a retest of this one please? I've tested in multiple browsers and it all seems to be working.
We have upgraded the version of Bootcards since this was reported so maybe it was fixed with that?


Enable stream, quality settings for screen sharing in Discord?

Is your feature request related to a problem? Please describe.
Discord desktop client allows you to alter stream settings when sharing screen, Ferdi does not expose this.

Describe the solution you'd like
Ability to set stream settings (resolution and refresh rate) when screen sharing

Yes - and unfortunately, unless Discord exposes this in their browser-based site, Ferdi will not be able to support this at all.


Dataverse Synapse Link - ALM - unable to configure the destination storage account

In the ALM section it is mentioned you can move configuration between environments (from dev, to test, to acc to prd). What I have seen for our customers, they have created an synapse workspace, storage account etc for each step in the DTAP. Sop each dataverse environment has a corresponding environment within Azure. As a result I should be possible to deploy the configuration in the next environment and configure what datalake will be used as a target. At this moment this is not possible and you can only deploy the link to the same storage account. It is nice that it will be in a different datalake, but this is not enough segregation.
It would be best to mention this in the features/ limitations. Otherwise you have to install a perform all steps until you find out it will not work for your specific scenario.
[Enter feedback here]

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 5eaf46c4-e66c-96c6-e780-42ed464a6db3
Version Independent ID: 5eaf46c4-e66c-96c6-e780-42ed464a6db3
Content: Transport Azure Synapse Link for Dataverse configuration - Power Apps
Content Source: powerapps-docs/maker/data-platform/azure-synapse-link-solution.md
Service: powerapps
Sub-service: dataverse-maker
GitHub Login: @sabinn-msft
Microsoft Alias: matp

Hi pieterh, you are right. ALM approach only applies to Dataverse setting and brings no impact on Azure side. I will update the doc accordingly

Hoping the product group can improved this functionality. In the meantime, my team has created a CLI tool that you can install in your pipelines / workflows to configure environment-level settings prior to importing a solution:
https://github.com/oops-i-git-it-again/dsyn-cli

We are planning to do ALM enhancement around Apr 23 to allow user update Azure side configuration before importing solution. Thanks

Closing issue.


DBZ-3671 Add property to exclude users in logminer query

https://issues.redhat.com/browse/DBZ-3671

@Naros, that's one for you :)

@gunnarmorling @jpechane Should we consider adjusting this to support reg-ex for username matching?

Hi @infinitesnowboy, I went ahead and pushed 2 commits here, one including the test I requested.  Once we decide whether we want to add reg-ex support for this or not, that will then determine if this is ready to go or not.

Thanks @Naros for adding the test

@infinitesnowboy, would you mind pulling down the few commits I pushed & rebase against master?  I'd prefer not to push a rebase to your branch if you don't mind.

@Naros Sure, I rebased against master

@Naros, if you wanted to address that one last pending comment, feel free to rebase and merge this one. I've pushed a comment for the casing change.

Did that last fix and applied. Thanks a lot, @infinitesnowboy!


Xbox controller support

Have to move the game controller support for next milestone because mine does not seems to work on my laptop


Disk resize with snapshots

When using the resize command, it seems __disk_resize() does not exclude snapshots, resulting in an error:
iohyve resize name disk0 25G
Resizing disk0 to 25G
[: /iohyve/zroot: unexpected operator
cannot set property for 'zroot': 'volsize' does not apply to datasets of this type

Also for roll-back the parent directory needs to be unmounted!

Long time no talk. I'm finally closing this issue.
We are going to completely re-write snapshotting/rollback functions in the near future.
We are going to have it function much like zfstools and also allow automatic snapshotting.