Create bucket with region "us-east-1" return error

when using ExAws.S3.put_bucket/3, with the region "us-east-1", the response is always
{:error, {:http_error, 400, "<?xml version=\"1.0\" encoding=\"UTF-8\"?>\n<Error>InvalidLocationConstraint<Message>The specified location-constraint is not valid</Message><LocationConstraint></LocationConstraint></Error>"
I read that with "us-east-1" region, the "LocationConstraint" element is not necessary, then may be in ex_aws/s3.ex file line 269, default case should be handled.
I'm using {:ex_aws, "~> 1.0.0-beta0"}
I attached 2 requests debug info here, they all return the same error.
1/ [debug] Request URL: "https://s3.amazonaws.com/testabcd/"
[debug] Request BODY: "<CreateBucketConfiguration xmlns=\"http://s3.amazonaws.com/doc/2006-03-01/\">\n  <LocationConstraint></LocationConstraint>\n</CreateBucketConfiguration>\n"
2/[debug] Request URL: "https://s3.amazonaws.com/testabcd/"
[debug] Request BODY: "<CreateBucketConfiguration xmlns=\"http://s3.amazonaws.com/doc/2006-03-01/\">\n  <LocationConstraint>us-east-1</LocationConstraint>\n</CreateBucketConfiguration>\n"

Can you gist the code you're running to create this request?

I think it is not only this project's problem:
https://github.com/boto/boto3/issues/125
https://github.com/SaltwaterC/aws2js/issues/64

Yeah basically you want to do put_bucket("bucket_name", ""). However, I should probably handle making "us-east-1" produce an empty string location constraint.
This is a great case of one of the things that makes me rant about how badly designed the S3 API is.

Fixed on master.


Nativewind not working in expo

Discussed in https://github.com/marklawlor/nativewind/discussions/555

Originally posted by grandwaterman September 14, 2023
Followed the doc https://www.nativewind.dev/quick-starts/expo.
The three steps, and it is NOT working. and checking the example on Github, it is very different from the doc.
Got error: Use process(css).then(cb) to work with async plugins
package.json - dep
"dependencies": {
"autoprefixer": "^10.4.15",
"expo": "~49.0.10",
"expo-status-bar": "~1.6.0",
"nativewind": "^2.0.11",
"postcss": "^8.4.29",
"react": "18.2.0",
"react-native": "0.72.4",
"tailwindcss": "^3.3.3"
},
"devDependencies": {
"@babel/core": "^7.20.0"
},
Are postcss and autoprefixer needed?
Document quality -1

Its solved here https://github.com/marklawlor/nativewind/issues/500

use this :
"dependencies": {
"autoprefixer": "^10.4.15",
"expo": "~49.0.10",
"expo-status-bar": "~1.6.0",
"nativewind": "^2.0.11",
"postcss": "^8.4.29",
"react": "18.2.0",
"react-native": "0.72.4",
"tailwindcss": "^3.3.3"
},
"devDependencies": {
"@babel/core": "^7.20.0"
},


Fix check-unused-dependencies

Better support for python3, and fix problem with counting white space as an output.

[approve ci].

[approve ci autest]

Do we need this for 7.1.x too ?

@zwoop I believe so, if python3 is used then it'll break so we better have them in 7.1.x as well.

This script does not seem to exist on 7.x branch.


Initializing task timed out after 30 seconds

I encountered a similar problem once before in Tencent Cloud, when I suspected it was a network environment problem and solved it by switching ap-guangzhou to na-siliconvalley.
After I encountered the problem this time, I tried to switch ap-hongkong to na-siliconvalley, but the problem was not solved
Attached:
https://sls-website-na-siliconvalley-e5yr91-1302257827.cos-website.na-siliconvalley.myqcloud.com

https://blog.csdn.net/weixin_42376823/article/details/113260427


deployment/run-unit-tests.sh is errorneous

The first "cd ../source/services/limitreport" works but then the second "cd ../source/services/slacknotify" fails because it wants to change to: "source/services/source/services/slacknotify".

Yes, thats what I do, I run it from deployment folder.

It looks like each of the paths is constructed relative to the deployment directory, rather than the current directory.
It should look like this:
cd ../source/services/limitreport
npm install
npm test

cd ../slacknotify
npm install
npm test

cd ../tarefresh
npm install
npm test

cd ../customhelper
npm install
npm test

@htrappmann if you replace it with this it should unblock you.

Yes, thank you, did already :)

Addressed with v5.1


Use app label not technical name in app modal

1. Why is this change necessary?
Currently the app modal shows the app's technical name as title. That should be the app's label.
2. What does this change do, exactly?
If makes the modal show the correct title.
3. Describe each step to reproduce the issue or behaviour.

Create an app which opens a modal.
Look at the title.

4. Please link to the relevant issues (if any).
5. Checklist

[ ] I have written tests and verified that they fail without my change
[x] I have squashed any insignificant commits
[x] I have created a changelog file with all necessary information about my changes
[ ] I have written or adjusted the documentation according to my changes
[ ] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/issues/NEXT-16523
Please use this issue to track the state of your pull request.

Hi @niklasbuechner, could you please write the changelog more detailed. You've added a new computed property, changed methods and templates without mentioning it. Furthermore, please write a test for your changes.

I am currently on vacation, but I will take a look at the requested changes once I am back 👍

Ping to keep this issue open. I am still working on it.

One more time: Ping to keep this issue open. I am still working on it.

What's the status here @niklasbuechner?

I ran into a few issues with the tests and have not yet had the chance to get everything going. Unfortunately, since this is just a private project, I have not been able to allocate the time to debug the tests yet.


Bug Fix techniques field as readonly in fusion rule template

ARM API Information (Control Plane)
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Azure 1st Party Service can try out the Shift Left experience to initiate API design review from ADO code repo.  If you are interested, may request engineering support by filling in with the form https://aka.ms/ShiftLeftSupportForm.
Changelog
Add a changelog entry for this PR by answering the following questions:

What's the purpose of the update?

[ ] new service onboarding
[ ] new API version
[x] update existing version for new feature
[x] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When are you targeting to deploy the new service/feature to public regions? Please provide the date or, if the date is not yet available, the month.
When do you expect to publish the swagger? Please provide date or, the the date is not yet available, the month.
By default, Azure SDKs of all languages (.NET/Python/Java/JavaScript for both management-plane SDK and data-plane SDK, Go for management-plane SDK only ) MUST be refreshed with/after swagger of new version is published. If you prefer NOT to refresh any specific SDK language upon swagger updates in the current PR, please leave details with justification here.

Contribution checklist (MS Employees Only):

[x] I commit to follow the Breaking Change Policy of "no breaking changes"
[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

Applicability: :warning:
If your changes encompass only the following scenarios, you should SKIP this section, as these scenarios do not require ARM review.

Change to data plane APIs
Adding new properties
All removals

Otherwise your PR may be subject to ARM review requirements. Complete the following:

[x] Check this box if any of the following apply to the PR so that the label "ARMReview" and "WaitForARMFeedback" will be added by bot to kick off ARM API Review.  Missing to check this box in the following scenario may result in delays to the ARM manifest review and deployment.

Adding a new service
Adding new API(s)
Adding a new API version
-[ ] To review changes efficiently, ensure you copy the existing version into the new directory structure for first commit and then push new changes, including version updates, in separate commits. You can use OpenAPIHub to initialize the PR for adding a new version. For more details refer to the wiki.

[x] Ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If you have any breaking changes as defined in the Breaking Change Policy, request approval from the Breaking Change Review Board.
Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Additional details on the process and office hours are on the Breaking Change Wiki.
NOTE: To update API(s) in public preview for over 1 year (refer to Retirement of Previews)
Please follow the link to find more details on PR review process.

@viskumar-microsoft - Adding new properties to an existing API version is a cross-version breaking change which requires a separate review. I've added the appropriate label-- look for the instructions in PR.

@viskumar-microsoft - Changing a property to readOnly is a breaking change which requires separate review. Adding the appropriate label: look for instructions in the subsequent PR comments.


Normalize line endings to Unix flavor on IO bounraries

bors r+

cc @kjeremy: this risk breaking windows (but it shouldn't, we have a test for this), so if something weird happens, this PR might be to blame

bors r+

bors r+

cc @kjeremy: this risk breaking windows (but it shouldn't, we have a test for this), so if something weird happens, this PR might be to blame

A quick smoke test of cargo test and some light editing in vscode passes on my machine.


Holding altitude while controlling quadcopter with velocity commands

Hi,
I have an algorithm which gives me velocity commands (ROS geometry_msgs/Twist type of message).
Now what I want to achieve with this is to control my quadcopter using those velocity commands with MAVROS package. The problem is that the algorithm expects quadcopter to hold it's altitude. So basically I want to hold given altitude and also control somehow linear velocity to fly somewhere in given altitude.
I've tried using MAVROS /mavros/setpoint_raw/local topic with mavros_msgs/PositionTarget type of message to do this. But this only works if I'm sending either position or velocity, but not both at the same time.
Is there some kind of altitude hold control instead of classic position control which doesn't work with velocity commands at the same time? I don't really want to write my own altitude controller :). I'm using the latest version of px4 firmware.
Thank you for any advice.
Edit: Here I can see setting the altitude hold in position's z-coordinate. This works, but the velocity values are doing nothing in this case no matter what even if this bool operation is false.
This is how I set the type_mask parameter:
    mavros_msgs::PositionTarget position_target;
    position_target.type_mask =
            mavros_msgs::PositionTarget::IGNORE_AFX |
            mavros_msgs::PositionTarget::IGNORE_AFY |
            mavros_msgs::PositionTarget::IGNORE_AFZ |
            mavros_msgs::PositionTarget::IGNORE_PX |
            mavros_msgs::PositionTarget::IGNORE_PY |
            mavros_msgs::PositionTarget::IGNORE_YAW |
            mavros_msgs::PositionTarget::IGNORE_YAW_RATE;

@L4ncelot Ha, I was running into exactly this as well, I think.
I've implemented in #5868 that altitude is held if the z velocity setpoint is 0 and position is held if the x and y velocity setpoint is 0. Can you try with my PR and maybe leave a comment there if something doesn't work for you?

I tried your PR but it doesn't seem to work. Even if I only specify the z-coordinate and everything else leave to 0, the quadcopter can't reach the position. It starts to climb but it tilts forth and back no matter if it already reached it's destination or not. And then the quadcopter flies and acts strangely :).
Changing then the velocity x,y coordinates seems to do nothing.
Am I doing it the right way? I mean, is there any difference if I use perhaps sp_position and sp_velocity topics?

You'll have to share the code that you use to send setpoints, otherwise it's hard for me to judge.

Well, I have a lot of things there which could be misleading to you. But I've commented everything which is not related to publishing position and it boils down to this code basically:
ros::Publisher pub_local_position_raw = nh.advertise<mavros_msgs::PositionTarget>("/mavros/setpoint_raw/local", 1);

mavros_msgs::PositionTarget position_target;
    position_target.type_mask =
            mavros_msgs::PositionTarget::IGNORE_AFX |
            mavros_msgs::PositionTarget::IGNORE_AFY |
            mavros_msgs::PositionTarget::IGNORE_AFZ |
            mavros_msgs::PositionTarget::IGNORE_PX |
            mavros_msgs::PositionTarget::IGNORE_PY |
            mavros_msgs::PositionTarget::IGNORE_YAW |
            mavros_msgs::PositionTarget::IGNORE_YAW_RATE;
position_target.coordinate_frame = mavros_msgs::PositionTarget::FRAME_BODY_OFFSET_NED;
position_target.position.z = 15.0;

ros::Rate rate(20.0);
ros::Time last_request = ros::Time::now();
while (ros::ok()) {
        ros::Duration time_difference = ros::Time::now() - last_request;
        if (current_state.mode != "OFFBOARD" &&
            time_difference > ros::Duration(2.0)) {
            setOffboardMode(srv_set_mode_client, set_mode_offboard);
            last_request = ros::Time::now();
        } else {
            if (!current_state.armed &&
                time_difference > ros::Duration(2.0)) {
                armVehicle(srv_arming_client, arm_cmd);
                last_request = ros::Time::now();
            }
        }

        pub_local_position_raw.publish(position_target);
        ros::spinOnce();
        rate.sleep();
}

Note that even if I set z-coordinate of position to 0 with all other values also at 0, quadcopter keeps behaving strangely as I described above. I can provide full code after some refactoring for you to, but after all comments it really does only the thing listed in code above so I think it's not necessary. In master branch the position control with this code works, but not with velocity commands.

Aha, but you are not setting the velocities? I would add:
mavros_msgs::PositionTarget::IGNORE_PZ |
and then set all 3 velocities.

Nono, I've set velocities and position first like this:
position_target.position.z = 15.0;
position_target.velocity.x = 1.0;
position_target.velocity.y = 0.0;
position_target.velocity.z = 0.0;

But then I encountered the strange flying behavior so I was curious if the flying model is messed up even for position control, which wasn't on master branch. So I commented velocities and tried if it works for z position control. As I said earlier, on master branch if I try to control quadcopter using either position or velocities, it works fine. But using both at the same time doesn't work.

Oh, understood. Well I have not tried that. It would probably need some work in the position controller.

Ok, I'll try something else then. Thank you for advice.

it is implemented and can be closed. you need to use setpoint raw and the right type_mask

I have implemented with setpoint raw and multiple type masks (below) with no success. I get the quad rolling around mostly. Setting just position or velocity works fine (given the right typemask) but not both. I am trying to auto hold a altitude while moving around via x and y velocities.  In all cases I set the velocity and positions, even the ignored ones (not that it made a difference).
coordinate frame is set to 1 if that is important.
Ignore x,y position: 0b0000101111000011
Ignore x,y position and z velocity: 0b0000101111100011 <-- this will rocket straight up if (despite being ignored) I set positionx, poisition.y,velocity.z = 0. When I commented them out, it rolled around.
Is there something wrong with my typemask? Thanks!


Test: Model and integration test for developer uploads

Resolves #21. The PR adds model tests and integration tests for avatar and cover image uploads to developer profiles. Also in doing this, I noticed that we need some upload validations to be added to businesses. The avatar concern should probably own those validations.
Also, I opted to stub out some of the model tests. I think it's efficient and prevents the need for a ton of file fixtures, but it may be a design decision worth taking a second look at.
Pull request checklist

[X] Your code contains tests relevant for code you modified
[X] You have linted and tested the project with bin/check

Made the changes!

And merged! Thanks again Jacob - excited to have tests around this code.


duplicated logic/files?

pkg/podlister/resolve_container.go and pkg/cgroup/resolve_container.go are duplicated?

@SamYuan1990 the good old podlister is gone :D


chore(ci): Attempt to fix TravisCI OSX builds

The pip command is now pip2
Change-Type: minor

This supersedes #1891

After this has been merged, all open PRs will need to be rebased in order to avoid build failures on OSX on TravisCI  @jviotti @jhermsmeier @Shou @lurch

Doh!! No idea why I did Change-Type: minor :confused:  I've just fixed it to Change-Type: patch :+1:


Unable to add Unicorn ORDER to user

Hitting the Order button does nothing, how can I debug ?

Need more info than that. There should be an error in the console, start from there.

I am talking about the browser developer console. You asked how you can debug, the console will give you debug information. Without an error I cannot help you.
Other than that, you should learn to tinker. It's fun :) I cannot replicate your issue, suggest to take a look at the live coding video for some insights on how to debug.

@awsed So now it works a bit better. After I re-did everything, I was able to see the unicorns AND I was able to order one YAY :)
Then disaster -  I logged out & back on to see it reflected, but this time ALL unicorns have disappeared from the app, though I can still see them in DD, but the balance is reflected correctly, but I dont see anything and obviously order button doesnt work anymore. Tried 3 diff. users each time with the hassle of a dozen MFA auths

The only other thing I see each time is the radix error which I am assuming we can ignore?
./src/components/Unicorns.js
Line 83:  Missing radix parameter  radix
Ok although I fixed that, I still encounter no unicorns in the list

Debugger shows an error:
Debugger listening on ws://<IP_ADDRESS>:15454/d14c28de-056e-42a8-83cc-f2bf58e26602
For help see https://nodejs.org/en/docs/inspector
Debugger attached.
/home/ec2-user/environment/unicorn-loyalty/src/components/Unicorns.js:1
(function (exports, require, module, __filename, __dirname) { import React, { Component } from 'react';
^^^^^^
(((( note that the hilite is wrongly placed on filename here but is under import as in the pic)))))
SyntaxError: Unexpected token import
at createScript (vm.js:80:10)
at Object.runInThisContext (vm.js:139:10)
at Module._compile (module.js:616:28)
at Object.Module._extensions..js (module.js:663:10)
at Module.load (module.js:565:32)
at tryModuleLoad (module.js:505:12)
at Function.Module._load (module.js:497:3)
at Function.Module.runMain (module.js:693:10)
at startup (bootstrap_node.js:191:16)
at bootstrap_node.js:612:3
Waiting for the debugger to disconnect...
Process exited with code: 1

I see this:

 cannot appear inside tbody - just a warning

VM828 bundle.js:109631 Uncaught (in promise) {data: {…}, errors: Array(2)}

Haha - now I see this "Download the React DevTools for a better development experience: https://fb.me/react-devtools" - will try that

I see 8 Unicorns being retrieved (in log), and this empty list

There'll be an error in the browser developer console in red if you are not seeing unicorns.
Alternatively run the same query the client is sending to AppSync in the AppSync Console:
query {
  listInventory{
    items{
        itemId
        itemDescription
        price
        count
    }
  }
}

Do you get an error in the AppSync Console?

Like I said earlier (but I blame myself for too much blah), I do see the correct # of unicorns being retrieved in the browser log, they are just not being rendered by the react js script.
I was going over you guys' awesome youtube vid when I realized I did not set 'Auto Generate GraphQL' to true, so thought that might be my mistake (left it at default FALSE as it was not mentioned in the steps here), so I started deleting my DSs in order to re-do them with that flag ON then fell into delete Resolver hell and I just gave up and am starting with a new account.
The disconnect here is your video is manually doing stuff (which is great) while this page tried to automate most of the stuff (which is good of coz) so I am still getting my head comparing the video and these scripts and trying to make out what might be missing.
So by the time I got your post I could not test that out to prove I was indeed seeing the Unics in the logs just not in the web page.
So here goes another attempt - hey I am learning stuff each time so it's all good.

C9 & Amplify dont seem to be ready for prime time; While dropping files into c9 (Upload Local Files) it didnt give any feedback and I thought it was doing it multiple times , since I dropped files multiple times due to no feedback unlike the last time (took 30min) so I cancelled; then I tried again, dropping it just once, it took over 2 hours; then I logged off and when I came back on, my env in the IDE was gone!!
I had a similar experience with c9 in the past but thought it might be better with the AWS hookup.
What a terrible waste of time this has been ! Closing this as I am not interested in proceeding further anymore.
People on Stack & Reddit are right about the cloud - undependable.

Sorry to hear. Suggest to log a support call so they can check your C9 environment.
Mind you C9 is completely optional and you can use a local IDE instead with the SAM CLI to deploy the Lambda function and DDB tables.

Thanks, I forgot about that, ok will try with CLI soon (just was so easy with C9).


Incorrect position of STG implicit place (token)

The implicit place on a straight connection is positioned in the middle between the centres of adjacent transitions.
This may look ugly is the bounding boxes of the transitions are significantly different - the token will look closer to one transition than the other.
A solution is to rely on the coordinates of end points of the arc instead of the centres of the adjacent nodes.

This issue was imported from Launchpad bug.

date created: 2016-02-02T16:29:40Z
owner: danilovesky
assignee: danilovesky
the launchpad url was https://bugs.launchpad.net/bugs/1541000

It is a duplicate of issue #355 and has been fixed by PR #377.


Fixup broken link in changelog

Get's me every time. :see_no_evil:
This deserves a lint running in CI.

Merge?

Merge?


Fix button selector specificity

Currently, the selector for buttons is
[class*="btn"]

which also matches buttons from other modules that prefix their elements, e.g. react-widgets which uses the rw- prefix.
This is unnecessary, the selector
[class^="btn"]

would be preferrable.

Merging your request. Thanks :+1:


[Feature request] Add license dialog support

Clear and concise description of the problem
provide a way to show the license dialog and download some thing:
Use case 1:elementary OS download

Use case 2: agree the agreement then download
Suggested solution
make a new plugin to do this?
Alternative
No response
Additional context
No response

You can register component and support thay on your own


WIP: updating GitHub Actions to adhere to Node 16 support

Should be mostly complete save for token for the release step and possible refactoring of yashanand1910/standard-release-notes

Can we close this as its already merged right?

@rikbrowning - yes, the extract changelog step will need resolved and the whole process will need a test. But it's currently merged


remove global var: btiMap

Codecov Report

Merging #204 (591afb6) into main (e6b4db5) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #204   +/-   ##
=======================================
  Coverage   94.77%   94.77%
=======================================
  Files          12       12
  Lines        5776     5777    +1
=======================================
+ Hits         5474     5475    +1
  Misses        234      234
  Partials       68       68

Impacted Files
Coverage Δ

builtin.go
94.21% <100.00%> (+<0.01%)
:arrow_up:

codebuild.go
95.82% <100.00%> (ø)

Continue to review full report at Codecov.

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


unmanaged-cluster: Implements selecting packages from namespaces on intstall-package

Users can specify the following mapping with the --install-package flag:

pkg-name:pkg-version:config-file-path:package-repo-namespace

Also includes docs updates

Which issue(s) this PR fixes
Fixes: #4165
Describe testing done for PR
Built binary and able to run successfully
$ tanzu unmanaged-cluster create my-cluster secretgen:latest::tkg-system

Still able to run successfully
$ tanzu unmanaged-cluster create my-cluster fluent

cc @jorgemoralespou - would you be willing to try this out and give some feedback? 👀

Sure thing

Some feedback (See last one first):

Test for missing package and NON existing namespace  OK
Input:
tanzu uc create test --install-package secretgen:::my-packages

Output:
Installing package secretgen
  WARNING: failed to install package secretgen. Error: no package was resolved for name secretgen with version

No package with any secretgen in it is available in the repository.

Test for package in repository with no version specified and NON existing namespace KO
Input:
tanzu uc create test --install-package external-dns:::my-packages

Output:
Installing package external-dns
   WARNING: failed to install package external-dns. Error: no package was resolved for name external-dns with version

Package exists in repository but latest version is not installed if version is missing. Help says: package-name should be the fully qualified package name or a prefix to a package name found in an installed package repository. package-version is optional and resolves to the latest semantic versioned package if not specified or latest is entered.

Test for Fully Qualified package in repository with no version specified and NON existing namespace KO
Input:
tanzu uc create test --install-package external-dns.community.tanzu.vmware.com:::my-packages

Output:
Installing package external-dns.community.tanzu.vmware.com
   WARNING: failed to install package external-dns.community.tanzu.vmware.com. Error: no package was resolved for name external-dns.community.tanzu.vmware.com with version

Package exists in repository but latest version is not installed if version is missing. Help says: package-name should be the fully qualified package name or a prefix to a package name found in an installed package repository. package-version is optional and resolves to the latest semantic versioned package if not specified or latest is entered.

Test for Fully Qualified package in repository with latest version specified and NON existing namespace KO
Input:
tanzu uc create test --install-package external-dns.community.tanzu.vmware.com:::my-packages

Output:
Installing package external-dns.community.tanzu.vmware.com
   WARNING: failed to install package external-dns.community.tanzu.vmware.com. Error: no package was resolved for name external-dns.community.tanzu.vmware.com with version latest

Package exists in repository but latest version is not installed if version is latest. Help says: package-name should be the fully qualified package name or a prefix to a package name found in an installed package repository. package-version is optional and resolves to the latest semantic versioned package if not specified or latest is entered.

Test for Fully Qualified package in repository with 0.10.0 version specified and NON existing namespace KO
Input:
tanzu uc create test --install-package external-dns:0.10.0::my-packages

Output:
Installing package external-dns.community.tanzu.vmware.com
WARNING: failed to install package external-dns.community.tanzu.vmware.com. Error: no package was resolved for name external-dns.community.tanzu.vmware.com with version 0.10.0

Package exists in repository but 0.10.0 version is not installed if version is specified. Help says: package-name should be the fully qualified package name or a prefix to a package name found in an installed package repository. package-version is optional and resolves to the latest semantic versioned package if not specified or latest is entered.

Test for Fully Qualified package in repository with 0.10.0 version specified and existing namespace OK
Input:
tanzu uc create test --install-package external-dns:0.10.0::default

Output:
Installing package external-dns.community.tanzu.vmware.com
   Selected package external-dns.community.tanzu.vmware.com
   Using package version 0.10.0
   Installing package with no configuration file

Package installed successfully, so I take that trying to install in a non-existing namespace manifests as the wrong error.

Great feedback. I think that is all valid.
Is there a reason we wouldn't want to automatically create the namespace if it's not there? Probably cuts down on a few extra steps for users. And this is something that many packages already do.
We could log a warning to the user that a new namespace is being created just to surface what's happening.

Having the nameapace be created would be great. I havr a few use cases of packages that docs say should be installed in a specific named namespace but that namespace is not pre created in a cluster. Similar functionality to the --create-namespace flag that exists for the package commands in tanzu cli and it would just make the UX better.


Values are memoised across class/prototype instances

Hi!
On the surface, it's kind of logical that (by default) values are instances are memoised across instances, because they're just function calls.
However: a) that seems almost never like intended behaviour, and b) there's a test that seems like it's explicitly trying to test for this.
Unfortunately, the test is only correct as a side effect. If you pass a new initial value into the constructor, you can break the logic (I've fixed the test on my branch, but not sure how to fix the actual problem!)
Do you know how, either as a library consumer, or within the library itself we could get this working as expected?

Thanks for trying the new decorator and opening an issue. Are you sure that test runs at all? ++this.index sets it to undefined now I think

The tests run ok, but I think you're referring to not defining index on the class? It's using a shorthand syntax for assigning properties directly from the constructor, but is otherwise the same as before. (ref: https://kendaleiv.com/typescript-constructor-assignment-public-and-private-keywords/)

// @Richienb

Should mem be memoizing values return by a function across instances or for each instance?

For each instance by default at least. We could consider adding an option to opt into memorization across instances.


Beacon: Maj and error catching

Some android phones are crashing after onboarding. After investigation this crash id due, on pixel phone, to beacon kotlin SDK crashing during  startBeacon operation.
Please add a catch to prevent the crash like it is suggested here: https://github.com/flutter/flutter/issues/22930#issuecomment-464764077
A new release of beacon has been published 3 weeks ago, please update the plugin.

Check updates of Beacon#1428


Planned Maintenance

Just some tweaks on their way 👍 -MPM

Maintenance now in progress.
Please note that there may be some features that will not function correctly on both MP Productions Services, and Client Services.
These functions include the live current status indicator that displays at the bottom of each website.
These maintenance works will be complete soon.
Thank You. - MPM


Icomoon custom icons for IconNB

Hi, and thanks for a great library :)
It would be great if IconNB could use custom icons as well as all the built in icons. react-native-vector-icons has a factory method for creating icon sets from icomoon.
react-native, react and native-base version
react-native: 0.54.0
native-base: 2.3.9
Expected behaviour
I would like to use the IconNB as this
Actual behaviour
Not implemented
Steps to reproduce (code snippet or screenshot)
not applicable
Screenshot of emulator/device
not applicable
Is the bug present in both ios and android or in any one of them?
not a bug, just a feature request
Any other additional info which would help us debug the issue quicker.
I will make a PR for this

@bjartebore Did you check this?

@SupriyaKalghatgi sorry, but I fail to se the relevance, what I want is the possibility to use custom fonts generated from https://icomoon.io/

@bjartebore you can fork NativeBase and try adding Icomoon
We dont have plans to do this


Popover/Tooltip placement logs tether error

I tried the auto placement like described in the documentation

When "auto" is specified, it will dynamically reorient the popover. For example, if placement is "auto left", the popover will display to the left when possible, otherwise it will display right.

After nothing happens I saw an error in the console.

Error: Tether Error: You must provide an attachment

On a bit debugging I found out that attachment is undefined in the tooltip.js what extends the popover.js cause the AttachmentMap does not provide any auto left
https://github.com/twbs/bootstrap/blob/v4-dev/js/src/tooltip.js#L282

A codepen to show the issue : http://codepen.io/Johann-S/pen/NbojYb


Docstring fix

fixes the docstring for infer_bonds!
the wrong type was indicated for the bonding_rules kwarg

Codecov Report

Merging #103 (2b559fe) into master (dbb688e) will increase coverage by 0.02%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #103      +/-   ##
==========================================
+ Coverage   95.92%   95.94%   +0.02%
==========================================
  Files           9        9
  Lines         981      987       +6
==========================================
+ Hits          941      947       +6
  Misses         40       40

Impacted Files
Coverage Δ

src/bonds.jl
96.71% <100.00%> (+0.06%)
:arrow_up:

src/crystal.jl
95.11% <100.00%> (+0.02%)
:arrow_up:

Continue to review full report at Codecov.

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


@autocomplete

Hi there, first of all kudos for this cool plugin,
I have found a small isuue, I am running latest sublime build on debian linux and when I "@" in a comment, the auto complete window pops up then immediatly closes instead of staying open. Strange. Not 100% sure this is a problem in your plugin or somewhere else in my setup but I can autocomplete any thing else.

I'm not seeing this issue myself (on Ubuntu build 2172). Can you enable command logging and show me the output after you press @? Open the python console (ctrl+`) and type:
sublime.log_commands(1)

Also, is this happening in a PHP or a Javascript file (or something else)?

This is hapenning in a PHP file, here is the log :
command: jsdocs
command: run_macro_file {"file": "Packages/DocBlockr/jsdocs-auto-complete.sublime-macro"}

Same thing in a js file, output :
command: jsdocs
command: run_macro_file {"file": "Packages/DocBlockr/jsdocs-auto-complete.sublime-macro"}

FYI: I've asked about this on the ST forum... no response yet.
http://www.sublimetext.com/forum/viewtopic.php?f=3&t=4987

Tested on my ubuntu box, and the autocomplete box dosent event appear, on debian box it kinda appears the disappears really fast


Parse custom zone mapping information

If ZONE_TO_URL is present in the environment, parse its JSON contents as string->v3.URL

This PR is solved by https://github.com/exoscale/exoscale-csi-driver/pull/4.
The mapping mechanism will be probably reworked in egoscale


Thick Provision SC Parameters has to be updated

Whenever a 'ceph-rbd' SC name is found, we have to create both thin and
thick provision RBD StorageClasses and for both the SC-s' parameters has
to be modified according to the 'data' passed from the input secret.
Signed-off-by: Arun Kumar Mohan<EMAIL_ADDRESS>

@jarrpa , @agarwal-mudit , please take a look

/ok-to-test

/assign @jarrpa

/cherry-pick release-4.8


Great Plugin

Just wanted to say this is one of the best scrolling plugins I've ever used. Keep up the good work 👍

Thanks @jbdivups

@lukehaas any chance you could comment on my horizontal scrollify question?


Image-Cap: Assertion Error

Hi,
@ne7ermore : So I get the following error when I try to load the actor model to the GPU (i.e. in train.py). Any help would be much appreciated.
Traceback (most recent call last):   File "train.py", line 114, in <module>     actor = actor.cuda()   File "/home/user/anaconda2/envs/py35/lib/python3.5/site-packages/torch/nn/modules/module.py", line 216, in cuda     return self._apply(lambda t: t.cuda(device))   File "/home/user/anaconda2/envs/py35/lib/python3.5/site-packages/torch/nn/modules/module.py", line 146, in _apply     module._apply(fn)   File "/home/user/anaconda2/envs/py35/lib/python3.5/site-packages/torch/nn/modules/rnn.py", line 123, in _apply     self.flatten_parameters()   File "/home/user/anaconda2/envs/py35/lib/python3.5/site-packages/torch/nn/modules/rnn.py", line 111, in flatten_parameters     params = rnn.get_parameters(fn, handle, fn.weight_buf)   File "/home/user/anaconda2/envs/py35/lib/python3.5/site-packages/torch/backends/cudnn/rnn.py", line 165, in get_parameters     assert filter_dim_a.prod() == filter_dim_a[0] AssertionError
I checked to make sure that the library versions match with the requirements.

may have at least one gpu

So the issue was that I was trying to run it in PyTorch 0.3.1 but I found that it works with PyTorch 0.3.0. Please consider updating the README here: https://github.com/ne7ermore/torch-light/blob/master/Image-Cap/README.md

So the issue was that I was trying to run it in PyTorch 0.3.1 but I found that it works with PyTorch 0.3.0. Please consider updating the README here: https://github.com/ne7ermore/torch-light/blob/master/Image-Cap/README.md

i see, thank you!


HDDS-8961. [Snapshot] Added check to allow only Ozone admin user to call print compaction dag

What changes were proposed in this pull request?
This change addresses few of the concerns raised in PR #5061.

Added check to allow only Ozone admin user to call print compaction dag API.
Update the response message stating the file name, parent dir's absolute path and the OM leader node info.

Note: This PR doesn't addressed Having a network call that generates a file on the server is strange. because of following reason.
As we talked in here, it is better to be have it as a OM request to get the in-memory picture of the DAG. It can be solved in following two ways.

Return graph POJO to client and generate the image on client side. But it comes with a problem of graph size and might need pagination.
Stream the generated image to the client. But we don't have existed interface to stream a file.
Both the solution will require more time and investigation. So for the interest of time, just addressed the security and debug message for now and create a jira: HDDS-9040 for further enhancement.

What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-8961
How was this patch tested?
Existed unit tests.

@errose28 and @smengcl, please take a look.


(feature) Add environment parameters in the config

Requirements
Add an environment parameter in the config.
This environment field should be added to all users in the param so we can add rules based on the environment and have the same config file for all your environment.

While reading this again I have doubts about the strategy to use to handle multiple environments.
Do we want to have multiple files (one per env) or another strategy?


Context cancelation not being honored

PingContext does not seem to honor context cancellation. I've mainly tested with PingContext and BeginTx, but this issue may be present in other methods as well, and I suspect that it might have to do with how OCIBreak works. However, I am not knowledgeable/familiar enough with Oracle to dig into its C library to further triage the issue.
Sample repro program:
package main

import (
	"context"
	"database/sql"
	"log"
	"time"

	_ "github.com/mattn/go-oci8"
)

func main() {
	log.Println("Calling sql.Open")
	db, err := sql.Open("oci8", "vaultadmin/myreallysecurepassword@localhost:1521/ORCLPDB1")
	if err != nil {
		log.Fatalf("error opening connection: %s", err)
	}
	defer db.Close()
	log.Println("Finished sql.Open")

	timeout := 10 * time.Second
	ctx, cancel := context.WithTimeout(context.Background(), timeout)
	defer cancel()

	done := make(chan struct{})
	go func() {
		ticker := time.NewTicker(time.Second)
		defer ticker.Stop()
		for {
			select {
			case <-ticker.C:
				log.Println("Still processing...")
			case <-done:
				return
			case <-ctx.Done():
				// Context cancellation should print the error, cancel the
				// PingContext call, and exit shortly after. However, the
				// program hangs for a period that is long past our timeout of
				// 10s before exiting.
				log.Println(ctx.Err())
				return
			}
		}

	}()

	log.Println("Calling PingContext")
	err = db.PingContext(ctx)
	if err != nil {
		log.Fatalf("error on ping: %s", err)
	}
	close(done)
	log.Println("Finished PingContext")
}

Output
Run where database is accessible:
› go run main.go
2021/04/28 16:46:32 Calling sql.Open
2021/04/28 16:46:32 Finished sql.Open
2021/04/28 16:46:32 Calling PingContext
2021/04/28 16:46:33 Finished PingContext

Run where database is inaccessible:
› go run main.go
2021/04/28 16:48:01 Calling sql.Open
2021/04/28 16:48:01 Finished sql.Open
2021/04/28 16:48:01 Calling PingContext
2021/04/28 16:48:02 Still processing...
2021/04/28 16:48:03 Still processing...
2021/04/28 16:48:04 Still processing...
2021/04/28 16:48:05 Still processing...
2021/04/28 16:48:06 Still processing...
2021/04/28 16:48:07 Still processing...
2021/04/28 16:48:08 Still processing...
2021/04/28 16:48:09 Still processing...
2021/04/28 16:48:10 Still processing...
2021/04/28 16:48:11 context deadline exceeded
^Csignal: interrupt

My go-to headache tablet with this kind of problem is to see if adding disable_oob=on to the sqlnet.ora (on the machine where you run Go) helps.  (Note if you are using 19c DB and 19c Oracle client libraries, then this line is not needed.)

If you remove the return from case <-ctx.Done(): how long does it go for?
There are other database timeouts that the OCI calls have to deal with so it will not return quickly. This question might help give a little insight:
https://github.com/mattn/go-oci8/issues/417

@MichaelS11 I have it running for 10+ minutes and it's still not terminated.

I'm compiling on MacOS using instantclient_19_8 if that helps.

I was able to get the application to terminate early by specifying SQLNET.OUTBOUND_CONNECT_TIMEOUT=15 in a sqlnet.ora file and setting the TNS_ADMIN environment variable. However, it would be ideal if context cancelation would terminate the process/drop the connection early without waiting for the full duration on the specified timeout value.
› go run main.go
2021/05/03 12:53:45 Calling sql.Open
2021/05/03 12:53:45 Finished sql.Open
2021/05/03 12:53:45 Sleeping for 5s
2021/05/03 12:53:46 Still processing...
2021/05/03 12:53:47 Still processing...
2021/05/03 12:53:48 Still processing...
2021/05/03 12:53:49 Still processing...
2021/05/03 12:53:50 Still processing...
2021/05/03 12:53:50 Calling PingContext
2021/05/03 12:53:51 Still processing...
2021/05/03 12:53:52 Still processing...
2021/05/03 12:53:53 Still processing...
2021/05/03 12:53:54 Still processing...
2021/05/03 12:53:55 Still processing...
2021/05/03 12:53:55 context deadline exceeded
2021/05/03 12:54:05 error on ping: ORA-12170: TNS:Connect timeout occurred

Glad you got it to work.
Yeah, that would be nice. Welcome a PR if you know of a good way to do it.


Allow elements to be moved to start of FirebaseArray (potential fix for #129)

When using FirebaseTableViewDataSource I was seeing something like the following when moving an object to the start of a list.
Assertions: failed: caught "NSRangeException", "*** -[__NSArrayM insertObject:atIndex:]: index<PHONE_NUMBER>854775807 beyond bounds [0 .. 8]"
It looks like it was caused by the FIRDataEventTypeChildMoved being sent a previousChildKey that doesn't exist in the array (presumably ""). This is similar to #129 and is potentially a fix for that.

I've added in the changes. I'm not sure exactly why the CI build failed.. there's quite a few Canceling tests due to timeout in Checking test manager availability messages.


Did I miss something or is there no s3 SDK yet?

Is there any reason it's not there yet? :)
If it's easy to add, can I also request to have it in the maven repo?
Many thanks!

I can work on this


Do not allow to call .normalizeCache() several times

Use case
Do not allow to call .normalizeCache() several times

We have now introduced setRetryOnErrorInterceptor, which gives a specific role to the retry interceptor. We could do the same for cache interceptors. This would avoid some errors at the price of less flexibility (can't add interceptors anywhere in the chain anymore).


Needs a https redirect feature

Need to be able to redirect from http to https easily to force specific services/path to https only.

Say you created a service like this,
rio run --name sample -p 80:80 nginx:alpine
rio ps
NAME      IMAGE          ENDPOINT                                     PORTS     SCALE     WEIGHT    CREATED          DETAIL
sample    nginx:alpine   https://sample-v0-default.somewhere.on-rio.io   80        1         100%      16 seconds ago

Check the virtualservices you got
kubectl get vs
NAME                                 AGE
sample                               110s
sample-tls-somewhere.on-rio.io-tls      110s
sample-v0                            110s
sample-v0-tls-somewhere.on-rio.io-tls   110s

Ok, now we need every request coming on the version of your choosing to go to https.
kubectl edit vs sample-v0
Add the following snippet, under the routes.

- matcher:
        prefix: /
      redirectAction:
        httpsRedirect: true
      routePlugins: {}

Once applied, your spec part of the virtualservice will look something like this
spec:
  virtualHost:
    domains:
    - sample-v0-default.somewhere.on-rio.io
    routes:
    - matcher:
        prefix: /
      redirectAction:
        httpsRedirect: true
      routePlugins: {}
    - matcher:
        prefix: /
      routeAction:
        single:
          kube:
            port: 80
            ref:
              name: sample-v0
              namespace: default

We are going to add this in router.
To add a router with HTTPS redirect, run
rio router add --https-redirect work to sample_service

Then check endpoints of router, access http endpoint will give you redirect link to https
$ curl -i http://xxx.on-rio-io
HTTP/1.1 301 Moved Permanently
location: https://xxx.on-rio-io
date: Tue, 17 Dec 2019 22:46:04 GMT
server: envoy
content-length: 0

To add https redirect to your publicdomain, then register public domain to this router
rio route register foo.bar router/work

Thank you @StrongMonkey

Validated above is working. Small correction to the domain register command: rio domain register foo.bar router/work


Add Window position manipulation

ref #3799
from kivy.app import App
from kivy.clock import Clock
from kivy.core.window import Window

class TestApp(App):
    def show(self, dt):
        pos = Window._get_window_pos()
        Window._set_window_pos(*[p + 1 for p in pos])

    def build(self):
        Clock.schedule_interval(self.show, 0)
TestApp().run()

For now getter and setter only. Not sure if it needs to be a separate thing from Window.top for example, or if it needs to update those two properties.
@dessant @matham ? 😋

Yeah, well... it's SDL2 property to be specific, but if it's necessary, we could have additional kivy property that would be just inverted top or left or something.


Feedback from Dare#0001

From Dare :
I recently missed an announcement about the feedback being hosted on GitHub. Would it be possible to incorporate a role that can be pinged during off-cuff announcements to make sure interested citizens don't miss such announcements when they don't land in <#220424374329671681> ?

From simonizor :
probably not... anything actually important will end up in the announcement channel. small changes like this would annoy most people to get pinged for.


[Patch]: Fix negative tool durability

This just randomly happens for some reason, removing this mixin fixes it.

I had a bit of a look at this, and we cannot completely remove the getMaxDamage mixin, because it will then use the items default durability when figuring out when a tool break, which will make Forgero tools with dynamic durabilty break before they actually run out of durability:
If we think there's something wrong happening here, we can use this as a fallback:
		@Inject(method = "getMaxDamage", at = @At("HEAD"), cancellable = true)
	public void getCustomDurability(CallbackInfoReturnable<Integer> cir) {
		if (this.getItem() instanceof DynamicDurability tool) {
			@SuppressWarnings("DataFlowIssue")
			ItemStack stack = (ItemStack) (Object) this;
			int durability = tool.getDurability(stack);
			if (durability > 0) {
				cir.setReturnValue(durability);
			} else {
				Forgero.LOGGER.error("Somehow the Durability of the tool {}, was {}. This should never happen", stack, durability);
			}
		}
	}

Alright, that seems like a good compromise.
I'll add it to the PR.

Oh, wait can we set the target branch to 1.19? I haven't switch to 1.20 yet

I tested on 1.20.1 myself, works there


Optimise PVC provision time

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
Remove unnecessary filesystem mount checks on the scale gui node during PVC provisioning. This would lead to lesser pvc create time.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

PR to improve PVC provision time : removed unnecessary gui mount fs checks

This has been delivered in 2.12


[Select] fix invalid pointerId in Firefox Cypress test

Description

Fixes https://github.com/cypress-io/cypress/issues/24459

Looks like the invalid pointer ID is thrown If the pointerId does not match any the active pointers and so returns void and throws a DOMException with the name InvalidPointerId . I believe this is happening because the captured pointer is being automatically released by firefox a fraction earlier, so by the time the pointerdown listener is called, the component no longer has a valid pointerID as it has implicitly been released thereby throwing an error.
According to https://www.w3.org/TR/pointerevents3/#implicit-pointer-capture "the hasPointerCapture API may be used (eg. within any pointerdown listener) to determine if `setPointerCapture was called on the target element"

To verify fix, run npx cypress run --headed --browser firefox --spec cypress/integration/Select.spec.ts

Known issues
Headless Firefox does not work. See https://github.com/cypress-io/cypress/issues/24459#issuecomment-1301673757

Hey @myrjola,
Is this a Firefox issue only?
How did you stumble on the symptom?
Why are we not seeing this error when using Firefox manually?

To my understanding, this only affects Radix UI Select inside Cypress tests running on Firefox. It's likely caused by differences in how Cypress' .click() does not work as a real browser click. Essentially, the fix is defensive programming by adding hasPointerCapture check before the releasePointerCapture that can fail in some circumstances. I'd like this fix merged since my team has Firefox tests in our CI pipeline that won't pass if we replace select component with Radix UI Select.
I didn't encounter any regressions by introducing this fix, but do you see any risks in merging it?

No adding it should be fine, it's rather that I wanted to understand why we didn't see it in Firefox itself as a user but I guess that makes sense. I'm not a fan of having to add defensive code to please tools, but in this case I think that's a fair ask.

Should we be standardising on this across our pointer capture code?

Should we be standardising on this across our pointer capture code?

Yes, I'll update other places.


[SPARK-39469][SQL] Infer date type for CSV schema inference

What changes were proposed in this pull request?

Add a new inferDate option to CSV Options. The description is:

Whether or not to infer columns that satisfy the dateFormat option as Date. Requires inferSchema to be true. When false, columns with dates will be inferred as String (or as Timestamp if it fits the timestampFormat) Legacy date formats in Timestamp columns cannot be parsed with this option.

An error will be thrown if inferDate is true when SQL Configuration LegacyTimeParserPolicy is LEGACY. This is to avoid incorrect schema inferences from legacy time parsers not doing strict parsing.
The inferDate option should prevent performance degradation for users who don't opt-in.

Modify InferField in CSVInferSchema.scala to include Date type.

If typeSoFar in inferField is Date, Timestamp or TimstampNTZ, we will first attempt to parse Date and then parse Timestamp/TimestampNTZ. The reason why we attempt to parse date for typeSoFar=Timestamp/TimestampNTZ is because of the case where a column contains a timestamp entry and then a date entry - we should detect both of the data types and infer the column as a timestamp type.
Example:
Seq("2010|10|10", "2010_10_10")
  .toDF.repartition(1).write.mode("overwrite").text("/tmp/foo")
spark.read
  .option("inferSchema", "true")
  .option("header", "false")
  .option("dateFormat", "yyyy|MM|dd")
  .option("timestampFormat", "yyyy_MM_dd").csv("/tmp/foo").printSchema()

Result:
root
 |-- _c0: timestamp (nullable = true)

Also modified makeConverter in UnivocityParser to handle Date type entries in a timestamp type column to properly parse the above example.

Does this PR introduce any user-facing change?

The new behavior of schema inference when inferDate = true:

If a column contains only dates, it should be of “date” type in the inferred schema
--> If the date format and the timestamp format are identical (e.g. both are yyyy/mm/dd), entries will default to being interpreted as Date
If a column contains dates and timestamps, it should be of “timestamp” type in the inferred schema

How was this patch tested?

Unit tests were added to CSVInferSchemaSuite and UnivocityParserSuite. An end to end test is added to CSVSuite
Benchmarks:
inferDate increases parsing/inference time in general. The impact scales with the number of rows (and not the number of columns). For columns of date type (which would be inferred as timestamp when inferDate=false), inference and parsing takes 30% longer. The performance impact is much greater on columns of timestamp type (taking 30x longer than inferDate=false) - due to trying each timestamp as a date (and throwing an error) during the inference step.
Number of seconds taken to parse each CSV file with inferDate true and inferDate false

inferDate=False
inferDate=True

Small file (<100 row/col). Mixed data types
0.32
0.33

100K rows. 4 columns. Mixed data types.
0.70
2.80

20k columns. 4 rows. Mixed Data types.
16.32
15.90

Large file. Only date type.
2.15
3.70

Large file. Only timestamp type.
2.60
77.00

Results are the average of 3 trials with the same machine.

Can one of the admins verify this patch?

Took a cursory look. @MaxGekk do you remember the context here? I remember we didn't merge this change because the legacy fast format parser (Java 8 libraries) did not support the exact matching (e.g., "yyyy" parses "2000-10-12" as "2000")

cc @bersprockets too if you find some time to review.

@Jonathancui123 You probably want to turn on github actions so tests will run.
From https://spark.apache.org/contributing.html:

Go to “Actions” tab on your forked repository and enable “Build and test” and “Report test results” workflows

I've added a inferDate flag that addresses the comments above:

Performance: Logic for date inference during is guarded behind the flag
Parsing behavior inferDate=false: Behavior is identical to prior to changes
Parsing behavior inferDate=true: In timestamp columns, we will attempt to parse as timestamp and then parse as date if timestamp fails.

This is necessary because we can have columns of Date and Timestamp entries inferred as Timestamp. Consider the column of a DateType followed by a TimestampType. We would expect this column to be inferred as a TimestampType column. Thus, when parsing the column, the timestamp converter will fail on the Date entry so we will need to try and convert it with the Date converter. If both converters fail, then we will throw an error.

The logic looks making sense in general .. but would be best to have second look from @bersprockets @MaxGekk @gengliangwang  ....

One test failed at org.apache.spark.sql.execution.datasources.csv.CSVLegacyTimeParserSuite
2022-06-22T01:10:13.0585659Z [info] - SPARK-39469: Infer schema for date type *** FAILED *** (106 milliseconds)
2022-06-22T01:10:13.0592465Z [info]   StructType(StructField(date,DateType,true), StructField(timestamp-date,DateType,true), StructField(date-timestamp,DateType,true)) did not equal StructType(StructField(date,DateType,true), StructField(timestamp-date,TimestampType,true), StructField(date-timestamp,TimestampType,true)) (CSVSuite.scala:2773)
2022-06-22T01:10:13.0598357Z [info]   Analysis:
2022-06-22T01:10:13.0604058Z [info]   StructType(1: StructField(timestamp-date,DateType,true) -> StructField(timestamp-date,TimestampType,true), 2: StructField(date-timestamp,DateType,true) -> StructField(date-timestamp,TimestampType,true))
2022-06-22T01:10:13.0610333Z [info]   org.scalatest.exceptions.TestFailedException:
2022-06-22T01:10:13.0622299Z [info]   at org.scalatest.Assertions.newAssertionFailedException(Assertions.scala:472)
2022-06-22T01:10:13.0627015Z [info]   at org.scalatest.Assertions.newAssertionFailedException$(Assertions.scala:471)
2022-06-22T01:10:13.0634808Z [info]   at org.scalatest.Assertions$.newAssertionFailedException(Assertions.scala:1231)
2022-06-22T01:10:13.0639642Z [info]   at org.scalatest.Assertions$AssertionsHelper.macroAssert(Assertions.scala:1295)
2022-06-22T01:10:13.0647369Z [info]   at org.apache.spark.sql.execution.datasources.csv.CSVSuite.$anonfun$new$379(CSVSuite.scala:2773)
2022-06-22T01:10:13.0655897Z [info]   at org.scalatest.OutcomeOf.outcomeOf(OutcomeOf.scala:85)
2022-06-22T01:10:13.0659677Z [info]   at org.scalatest.OutcomeOf.outcomeOf$(OutcomeOf.scala:83)
2022-06-22T01:10:13.0663352Z [info]   at org.scalatest.OutcomeOf$.outcomeOf(OutcomeOf.scala:104)
2022-06-22T01:10:13.0666989Z [info]   at org.scalatest.Transformer.apply(Transformer.scala:22)
2022-06-22T01:10:13.0670619Z [info]   at org.scalatest.Transformer.apply(Transformer.scala:20)
2022-06-22T01:10:13.0674478Z [info]   at org.scalatest.funsuite.AnyFunSuiteLike$$anon$1.apply(AnyFunSuiteLike.scala:190)
2022-06-22T01:10:13.0678412Z [info]   at org.apache.spark.SparkFunSuite.withFixture(SparkFunSuite.scala:203)
2022-06-22T01:10:13.0694953Z [info]   at org.scalatest.funsuite.AnyFunSuiteLike.invokeWithFixture$1(AnyFunSuiteLike.scala:188)
2022-06-22T01:10:13.0699731Z [info]   at org.scalatest.funsuite.AnyFunSuiteLike.$anonfun$runTest$1(AnyFunSuiteLike.scala:200)
2022-06-22T01:10:13.0703990Z [info]   at org.scalatest.SuperEngine.runTestImpl(Engine.scala:306)
2022-06-22T01:10:13.0708256Z [info]   at org.scalatest.funsuite.AnyFunSuiteLike.runTest(AnyFunSuiteLike.scala:200)
2022-06-22T01:10:13.0712723Z [info]   at org.scalatest.funsuite.AnyFunSuiteLike.runTest$(AnyFunSuiteLike.scala:182)
2022-06-22T01:10:13.0717517Z [info]   at org.apache.spark.SparkFunSuite.org$scalatest$BeforeAndAfterEach$$super$runTest(SparkFunSuite.scala:64)
2022-06-22T01:10:13.0721680Z [info]   at org.scalatest.BeforeAndAfterEach.runTest(BeforeAndAfterEach.scala:234)
2022-06-22T01:10:13.0726397Z [info]   at org.scalatest.BeforeAndAfterEach.runTest$(BeforeAndAfterEach.scala:227)

I added a new QueryExecutionError called inferDateWithLegacyTimeParserError that is thrown when inferDate=true and SQL Configuration LegacyTimeParserPolicy is LEGACY. This prevents the legacy parser from being used with schema inference for date and fixes the failing test

https://github.com/apache/spark/blob/8df7ebbd6c1c0d5bb875ce554026f9b9aeae148e/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/csv/CSVOptions.scala#L165-L172
Added one last commit^^ to fix the case where dateFormat is not specified. When inferDate is true, we can set a default value for dateFormat to ensure that the correct dateFormatter is used:
https://github.com/apache/spark/blob/8c02823b49a6f28005236e4965a25e664d73ebea/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/util/DateFormatter.scala#L182-L184
The defaultDateFormatter class doesn't implement strict parsing, but Iso8601DateFormatter does

I've updated the PR description to include benchmarking results. inferDate=true is bad for large CSVs with many timestamp columns.

Seems fine in general but maybe i'm missing some discussions or details here.
@Jonathancui123 would appreciate it if you summarize any unaddressed comments or concerns. I am fine given that we disable this here by default.

would appreciate it if you summarize any unaddressed comments or concerns. I am fine given that we disable this here by default.

@HyukjinKwon I don't see any remaining unaddressed comments - I believe we are ready to merge this PR. Here are the resolutions to potential concerns:

No backwards compatibility concern: Legacy parser policy cannot be used with inferDate. We can open another ticket to add a LegacyStrictSimpleDateFormatter as Bruce suggested if necessary.
Performance degradation: Benchmark results (in the bottom of PR description) show that date inference greatly slows down schema inference for CSVs with dates. This feature is guarded by the CSV Option inferDate and is off by default.
Logical errors for strict parsing and converting Date to TimestampNTZ have been fixed and have unit tests

Hi folks, @HyukjinKwon @bersprockets @cloud-fan, thanks for reviewing and some great suggestions, is this PR good to go? Thanks!

thanks, merging to master!

Because "1765-03-28" does not match timestamp pattern and the column is inferred as TimestampType, it should be returned as null. However, in the test it is returned as 1765-03-28 00:00:00.0. This is at very least confusing - inferDate should only affect dates, not timestamp columns.

@sadikovi I've verified on an older version of spark prior to this PR: "1765-03-28" in a timestamp column without user specified format is parsed as "1765-03-28 00:00:00.0". So the behavior of parsing default date format in timestamp columns is not due to this PR.
Prior to changes, in a timestamp column:

custom format date: null
default format date: Parsed by fallback

After inferDate PR, in a timestamp column:

custom format date: Parsed if inferDate is true, otherwise null
default format date: Parsed by fallback

After enableParsingFallbackForDateType PR (#37147), in a timestamp column:

custom format date: null
default format date: null

Since default format date was previously parsed by fallback, we thought it was desirable for dates to be parsed in a timestamp column. So we included support for custom format dates in a timestamp column when inferDate=true.
We have two options for target behavior:
OPTION A - Target behavior, in a timestamp column:

custom format date: Parsed if inferDate is true, otherwise null
default format date: Parsed if inferDate is true, otherwise null
--> move the fallback error in #37147 to allow date parsing if inferDate=true

OPTION B - Target behavior, in a timestamp column:

custom format date: null
default format date: null
--> remove redundant dateFormatter.parse behavior for inferDate = true

@cloud-fan  @HyukjinKwon should we go with Option A or Option B?

So am I right that Option A considers the type coercion between timestamps and dates, and Option B does not? I personally prefers Option B so we can switch to Option A in the future. Once we pick Option A, it's difficult to go back to Option B.

I think my main confusion comes from the "inferDate=true" turning an invalid timestamp value into a date and then returning it as a timestamp, the column should have been a DateType column.
@Jonathancui123 Would it be possible to revisit this behaviour?
I am going to fix my PR to work together with inferDate option, no worries.

@Jonathancui123 I addressed this issue in https://github.com/apache/spark/pull/37147/commits/10ca4a4a09623ef9ea217b0fdf33bbdf2e22c227 on my PR. Can you review? Thanks.

@sadikovi mind opening a pr?

It is a small fix so I fixed it in my PR https://github.com/apache/spark/pull/37147.

@Jonathancui123 I fixed this issue in 10ca4a4 on my PR. Can you review? Thanks.

Thanks for the change! I agree with it and I've left a comment


Implement find tasks by date

provides #6

This feature is not going to be implemented, since we have a calendar that provides this function.


just want to clear the existing mediaBlobUrl how can i do that, is there any existing function for that?

No, not at the moment. It gets replaced by the next recording and so on.

hello @0x006F  thanks for replying, so i have a send button on the UI on which i want to clear the video any idea how can we  do that?

Hey @siddhant-reactgeeks sorry for the delayed reply. You could unmount this component on a button click though. Although, it doesn't sound like a straightforward fix. The ideal way would be provide a function from this recorder that nullify the streams / blobs on demand. Let me check.

This would be great! I'm trying to find an easy way to remove the mediaBlobUrl as well and a built-in function would help make my code much cleaner

thanks @0x006F i am using this unmount fix for now, but it would be great if you could implement a callback for this.

https://e2ede998.ngrok.io/project/bBA2doHTR3nP39VJqmaH

https://5b32ce09.ngrok.io/project/vgIuJepfGjTm6HsjUsxl?recalc=true?view=true

https://5b32ce09.ngrok.io/project/5Vm4yWRNCmWrTvjtj8rj

Hey all, I'm sorry for the delayed response, but this has been fixed in latest release V1.3.0.
Let me know

Testing this now


SLF4J: Failed to load class "org.slf4j.impl.StaticLoggerBinder".

If I try to launch the compiled .jar (with IntelliJ) on my Brick it gives me this Error:
`SLF4J: Failed to load class "org.slf4j.impl.StaticLoggerBinder".
SLF4J: Defaulting to no-operation (NOP) logger implementation
SLF4J: See http://www.slf4j.org/codes.html#StaticLoggerBinder for further details.`
It appears if I try to use Motors, Sensors and so on, e.g.
Motor m = new Motor(new LegoPort(LegoPort.OUTPUT_A))
And I am a beginner with ev3dev java, so don't be too harsh :)
Thank you 💯

This is just a warning message that won't affect the functionality. You can probably ignore it. This appears if the logger does not find any SLF4J provider in your application. Probably you did not add a dependency of slf4j-api and a logger e.g. slf4j-log4j.
For more details you can check what the log provided http://www.slf4j.org/codes.html#StaticLoggerBinder

Thank you :)
But my Motor doesn't work anyways :(.
Would this work if I connect a M/L Motor to OUTPUT_A?
Motor m = new Motor(new LegoPort(LegoPort.OUTPUT_A)) m.setDutyCycleSP(50); m.runForever();

Sorry for late reply. In newer kernel version, DutyCycle_SP() is not applicable to runForever() now. To set speed, you have to use setSpeed_SP()
http://www.jev3dev.tk/getting_started-tank.html


When starts debug it hangs and nothing happens it keeps loading code 1.33

When starts debug it hangs and nothing happens it keeps loading I think is related to new debugger subsession on code 1.33
works perfect in 1.30

Same for me...

It works fine on vscode 1.32.3

Please provide:

launch.json contents (significant part)
output of : "showDebugOutput": true, option
general info about setup

I just ran it on 1.33 and 1.34-insider + W10, cannot reproduce.

I have to correct myself, it launches only when i debug the extension.
When runni9ng directly the extension from store, it fails...

This seems to be an issue with VSCode 1.33, since introducing:
ConPTY turned on by default on Windows build 18309+
For supporting versions of Windows 10 Insiders, the new terminal back-end ConPTY will be turned on by default. This should result in a more stable experience closer to the native terminal and adds 256 color support.

Setting "terminal.integrated.windowsEnableConpty"<EMAIL_ADDRESS>fixed the launch with "terminalKind"<EMAIL_ADDRESS>on W10 17763.379 . Not sure if this affects other systems as well, but it might since clearly my system is not 18309+.

for me it still hangs...

@Ikarus76 ,please provide:
launch.json contents (significant part)
output of : "showDebugOutput": true, option
general info about setup

@Ikarus76 : seems to be issue with bash integrated terminal in this case.
"terminal.integrated.shell.windows": "C:\WINDOWS\System32\bash.exe",
Try commenting this out and running with PS.

changes nothing, still hanging

{
"version": "0.2.0",
"configurations": [
{
"type": "bashdb",
"request": "launch",
"name": "Bash-Debug (simplest configuration)",
"program": "${file}"
}
]
}

Windows 10 Home Version 1809 OS build 17763.379
Still using winpty

I will need to investigate this more.
Please verify if "terminalKind": "debugConsole" @launch.json works, so that it is terminal related (keeping in mind that debugConsole does not handle stdin).

I will need to investigate this more.
Please verify if "terminalKind": "debugConsole" @launch.json works, so that it is terminal related (keeping in mind that debugConsole does not handle stdin).

same with "terminalKind": "debugConsole"

I just started to use Visual Code and the extension, got the same hang issue. Did a downgrade of VS Code, and all works as expected.
Got it from here: https://code.visualstudio.com/updates/v1_32

I cannot reproduce the "debugConsole" output issue.
As for "integrated" terminal output, seems that VSCode's this.runInTerminalRequest(...) does not work properly now:
{"title":"Bash Debug Console","cwd":".","args":["wsl.exe","bash","-c","cd \"/mnt/d/Users/rogal_000/Documents/src/BashDebugger/testing\"; while [[ ! -p \"/tmp/vscode-bash-debug-fifo-19228\" ]]; do sleep 0.25; done; \"bash\" \"/mnt/d/Users/rogal_000/Documents/src/BashDebugger/vscode-bash-debug/bashdb_dir/bashdb\" --quiet --tty \"/tmp/vscode-bash-debug-fifo-19228\" --tty_in \"/tmp/vscode-bash-debug-fifo-19228_in\" --library \"/mnt/d/Users/rogal_000/Documents/src/BashDebugger/vscode-bash-debug/bashdb_dir\" -- \"/mnt/d/Users/rogal_000/Documents/src/BashDebugger/testing/test.sh\" "]}

{"type":"response","seq":3,"command":"runInTerminal","request_seq":2,"success":true,"body":{}}

Success is returned, but no debug terminal appears (no terminal run == no bashdb process).

I was using Debian Buster vscode 1.33
@rogalmic

Created an issue for VSCode:
https://github.com/Microsoft/vscode/issues/71844
@JPRO89-Cu The released version of Bash Debug uses :
    "vscode-debugadapter": "1.33.0",
    "vscode-debugprotocol": "1.33.0"

and works with older version of tool. I tried updating the dependency, but it does not make difference.
It seems that the issue is in terminal implementation itself. It is quite possible that there is another issue as well, since there are reports that setting "terminalKind"<EMAIL_ADDRESS>does not help (just I cannot reproduce it currently).
I have attached test project zip in ticket, for me only the option without specified terminal fails (W10 17763.404). On different machine W10 17763.379, "terminal.integrated.windowsEnableConpty": true,  seemed to make a difference.
Pretty annoying bug :)

i just only set : "terminalKind"<EMAIL_ADDRESS>and it worked
but debug console says
/root/.vscode/extensions/rogalmic.bash-debug-0.3.4/bashdb_dir/command/source.sh: line 41: /dev/stdin: No such device or address

@JPRO89-Cu  - this is expected, as debugConsole has no stdin.

Same problem on Ubuntu 16.04, vscode 1.33.0. Just installed the extension, tried the simplest configuration and it hangs loading the script.
Adding "terminalKind": "debugConsole" solves the problem.

Hi, I discovered, that if I use 'args' in the launch.json it hangs. Without args it works. event with integrated terminal.

MacOS 10.14.4
VSCode 1.33.0
Bash: 5.0.3(1)-release
Not working
With: "terminalKind": „debugConsole“, works ok. Do you need any logs?

MacOS 10.14.4
VSCode 1.33.0
5.0.2(1)-release (x86_64-apple-darwin18.2.0)
Not working

There are tickets for VSCode, since running in terminal is done via debugadapter's method.
https://github.com/Microsoft/vscode/issues/71844
https://github.com/Microsoft/vscode/issues/71917
https://github.com/Microsoft/debug-adapter-protocol/issues/43
For now as a workaround terminalKind needs to be specified, default config will fail (https://github.com/Microsoft/debug-adapter-protocol/issues/43).
For problems with terminalKind set explicitly "integrated" or "external", please create separate tickets with:

system version
setting values (if set) for:
"terminal.integrated.shell.*"
"terminal.integrated.windowsEnableConpty"
information from debugging session, if terminal was opened and if there were any errors within terminal window/pane (the actual terminal launch command is visible for user)
launch.json contents, the config part that was launched

When changing settings, restart the VSCode just to be safe.

it's work for me.
my laptop:
ubuntu: 18.04
vscode: 1.33.1
bash: 4.4.19
bashdb: 4.4-0.92
launch.json
{
// Use IntelliSense to learn about possible attributes.
// Hover to view descriptions of existing attributes.
// For more information, visit: https://go.microsoft.com/fwlink/?linkid=830387
"version": "0.2.0",
"configurations": [
{
"type": "bashdb",
"request": "launch",
"terminalKind": "debugConsole",
"name": "Bash-Debug (simplest configuration)",
"program": "${file}"
}
]
}

Setting terminalKind to debugConsole in launch.json fixes the issue for me.
Some information about my system:

VSCode version: VSCodium 1.33.0
Bash version: 5.0.3(1)
bashdb version: 4.4-1.0.1
launch.json:

{
    // Use IntelliSense to learn about possible attributes.
    // Hover to view descriptions of existing attributes.
    // For more information, visit: https://go.microsoft.com/fwlink/?linkid=830387
    "version": "0.2.0",
    "configurations": [
        {
            "type": "bashdb",
            "request": "launch",
            "name": "Bash-Debug (simplest configuration)",
            "program": "${file}",
            "terminalKind": "debugConsole"
        }
    ]
}

i just only set : "terminalKind"<EMAIL_ADDRESS>and it worked
but debug console says
/root/.vscode/extensions/rogalmic.bash-debug-0.3.4/bashdb_dir/command/source.sh: line 41: /dev/stdin: No such device or address

Just as @rgeorgiev583 posted, settting "terminalKind": "integrated" seems to solve the aforementioned error also under the following platform:

OS: Manjaro Linux 18.0.4 updated to kernel 4.19.42
Code: 1.33.1
bash: 5.0.7
bashdb: 4.4_1.0.1

Since nothing is going on debugadapter side, I will need to set default value of "terminalKind".
The issue is that value "integrated" has problems inside VSCode (when path contains nan-ascii chars), while "debugConsole" does not allow interactive strips using stdin.
So - how to choose which is better as default ? Voting? :)

Hmmm...I think I'm having the same problem but setting to "debugConsole" doesn't resolve the issue.
I'm running Ubuntu 18.04, VSC 1.41.1, bash 4.4.19-2ubuntu1.2 amd64. My launch.json file looks like this:
{
    // Use IntelliSense to learn about possible attributes.
    // Hover to view descriptions of existing attributes.
    // For more information, visit: https://go.microsoft.com/fwlink/?linkid=830387
    "version": "0.2.0",
    "configurations": [
        {
            "type": "bashdb",
            "request": "launch",
            "terminalKind": "debugConsole",
            "name": "Bash-Debug (simplest configuration)",
            "program": "${file}",
            "showDebugOutput": true
        }
    ]
}

Debug output looks like this:
(I've trimmed the file paths for where the code resides to /home/dave/code/plugins/lqd-messages/ for readability sake. The full file path would be /home/dave/code/vvv/www/wordpress-one/public_html/wp-content/plugins/lqd-messages
::PROXYID::12183
/home/dave/.vscode/extensions/rogalmic.bash-debug-0.3.6/bashdb_dir/command/source.sh:
line 41: /dev/stdin: No such device or address
(/home/dave/code/plugins/lqd-messages/bin/install-wp-tests.sh:3):
3:	if [ $# -lt 3 ]; then
Debug environment: bash_ver=4.4.20(1)-release, bashdb_ver=4.4-0.94-mod,
program=/home/dave/.vscode/extensions/rogalmic.bash-debug-0.3.6/bashdb_dir/bashdb,
args=12186
############################################################
backtrace
->0 in file `/home/dave/code/plugins/lqd-messages/bin/install-wp-tests.sh' at line 3
##1 source("/home/dave/code/plugins/lqd-messages/bin/install-wp-tests.sh") called from file `/home/dave/.vscode/extensions/rogalmic.bash-debug-0.3.6/bashdb_dir/bashdb' at line 102
##2 main("--quiet", "--tty", "/tmp/vscode-bash-debug-fifo-14185", "--tty_in", "/tmp
/vscode-bash-debug-fifo-14185_in", "--library", "/home/dave/.vscode/extensions
rogalmic.bash-debug-0.3.6/bashdb_dir", "--", "/home/dave/code/plugins/lqd-messages/bin/install-wp-tests.sh") called from file
`/home/dave/.vscode/extensions/rogalmic.bash-debug-0.3.6/bashdb_dir/bashdb' at line 0
12186############################################################
File /home/dave/code/plugins/lqd-messages/bin/install-wp-tests.sh loaded.
break </home/dave/code/plugins/lqd-messages/bin/install-wp-tests.sh:5 >
Breakpoint 1 set in file /home/dave/code/plugins/lqd-messages/bin/install-wp-tests.sh, line 5.
Source files which we have recorded info about:
  /home/dave/code/plugins/lqd-messages/bin/install-wp-tests.sh: /home/dave/code/plugins/lqd-messages/bin/install-wp-tests.sh, 174 lines
Num Type       Disp Enb What
----------------------------
1   breakpoint keep y   /home/dave/code/plugins/lqd-messages/bin/install-wp-tests.sh:5
12186############################################################
Program stopped.
It stopped after being stepped.
Next statement to be run is:
	[ $# -lt 3 ]
examine <$PWD>
/home/dave/code/plugins/lqd-messages/bin
examine <$? # from . "$_Dbg_script_file">
0 # from '. "$_Dbg_script_file"'
12186############################################################
examine <branches>
branches
12186############################################################

Any suggestions?
Thanks!

@davidshq
The log seems ok, everything was exeuted as it should be (this is not the terminal issue). Any chance to get the traces of communication between VSCode-debugadapter?
To get the trace add the line to launch.json:
"trace": true

@rogalmic - sure, its here: https://pastebin.com/MpKxJU6u

@davidshq :
"Debugged program terminated with code 1. Use q to quit or R to restart.\n"
Please try with simple script like echo "Hello world". Also, you can try runing directly with lower layer bashdb tool (after installing it).
This console log is pretty interesting as well, probably limitation of the debugger extension:
usage: /home/dave/.vscode/extensions/rogalmic.bash-debug-0.3.6/bashdb_dir/bashdb <db-name> <db-user> <db-pass> [db-host] [wp-version] [skip-database-creation]

...though it should break at this line. Running bashdb directly will give the answer.

Running the debugger on echo "Hello Word" from within VSC returns the same error regarding /dev/stdin: https://pastebin.com/9as5RN0w
This is my first introduction to bashdb, so not really sure how it should work, I'll try to look into it a bit more. Ran this and received the following output:
 bash bashdb ~/code/vvv/www/wordpress-one/public_html/wp-content/plugins/lqd-messages/bin/test.sh
bash debugger, bashdb, release 4.4-0.94-mod

Copyright 2002-2004, 2006-2012, 2014, 2016-2017 Rocky Bernstein
This is free software, covered by the GNU General Public License, and you are
welcome to change it and/or distribute copies of it under certain conditions.

(/home/dave/code/vvv/www/wordpress-one/public_html/wp-content/plugins/lqd-messages/bin/test.sh:1):
1:	echo "Hello World"
bashdb<0>

Do not worry about stdin error, this is normal for "terminalKind": "debugConsole".
The issue here is the fact that script exits by itself:
Debugged program terminated with code 1. Use q to quit or R to restart.

Most likely setup issue - maybe it is worth to verify bash version and user's bash settings.

I found a similar issue on Arch Linux.
launch.json is as follows.
{
    // IntelliSense を使用して利用可能な属性を学べます。
    // 既存の属性の説明をホバーして表示します。
    // 詳細情報は次を確認してください: https://go.microsoft.com/fwlink/?linkid=830387
    "version": "0.2.0",
    "configurations": [
        {
            "type": "bashdb",
            "request": "launch",
            "name": "Bash-Debug (select script from list of sh files)",
            "cwd": "${workspaceFolder}",
            "program": "${command:SelectScriptName}",
            "showDebugOutput": true,
            "terminalKind": "debugConsole",
            "args": []
        }
    ]
}

Hello i have got a problem (i have got ubuntu 18.04 and visual code 1.49.2.. I launch my script, it opens on debug console, and i don't open in terminal. I add this line
"type": "bashdb",
"request": "launch",
"name": "Bash-Debug (simplest configuration)",
"program": "${file}",
"terminalKind": "debugConsole",
And l launch my script
i have got this message
/home/aip/.vscode/extensions/rogalmic.bash-debug-0.3.9/bashdb_dir/command/source.sh: ligne 41: /dev/stdin: Aucun périphérique ou adresse

@Marcotoulouse3 : debugConsole does not support stdin.
It is there in readme:

Use<EMAIL_ADDRESS>set to integrated or external for interactive scripts (using stdin)

Hello
Whose use it, i don't see to use it, i just read to use it but how install it in visual code (sorry for my no knowledge) :/

In debugged script, if you run any program that requires typing something to console, you effectively use stdin. In this case use other terminalKind, like "integrated", in launch.json.

Hello
When i add this line in my json
"terminalKind": "debugConsole"... don't add fonction terminal ?  i'm so sorry, it 's my first time to use visual code... and who add stin in visual code ?

Hi Marco.
Yes, I see that could be a little confusing.
TerminalKind is connected for input and output for the script, not debug messages.
It's connected to standard input (stdin) and standard output (stdout)
The VS Code debug console is automatically attached to the script + debugger.
View the debug output with Ctrl-Shift-Y or click 'DEBUG CONSOLE'.
A working config for you.
Not this:
    "configurations": [
      {
        "type": "bashdb",
        "request": "launch",
        "name": "Bash simple",
        "cwd": "${workspaceFolder}",
        "program": "${file}",
        "terminalKind": "debugConsole"   <---- **Not this**
      }
    ]

Use this:
    "configurations": [
      {
        "type": "bashdb",
        "request": "launch",
        "name": "Bash simple",
        "cwd": "${workspaceFolder}",
        "program": "${file}",
        "terminalKind": "integrated"   <---- Use this, the integrated terminal.
      }
    ]

HTHs someone.

Hi Marco.
Yes, I see that could be a little confusing.
TerminalKind is connected for input and output for the script, not debug messages.
It's connected to standard input (stdin) and standard output (stdout)
The VS Code debug console is automatically attached to the script + debugger.
View the debug output with Ctrl-Shift-Y or click 'DEBUG CONSOLE'.
A working config for you.
Not this:
    "configurations": [
      {
        "type": "bashdb",
        "request": "launch",
        "name": "Bash simple",
        "cwd": "${workspaceFolder}",
        "program": "${file}",
        "terminalKind": "debugConsole"   <---- **Not this**
      }
    ]

Use this:
    "configurations": [
      {
        "type": "bashdb",
        "request": "launch",
        "name": "Bash simple",
        "cwd": "${workspaceFolder}",
        "program": "${file}",
        "terminalKind": "integrated"   <---- Use this, the integrated terminal.
      }
    ]

HTHs someone.

Hi Marco.
...
Use this:
    "configurations": [
      {
        "type": "bashdb",
        "request": "launch",
        "name": "Bash simple",
        "cwd": "${workspaceFolder}",
        "program": "${file}",
        "terminalKind": "integrated"   <---- Use this, the integrated terminal.
      }
    ]

HTHs someone.

This absolutely helped me. Thank you.

Hi Marco.
Yes, I see that could be a little confusing.
TerminalKind is connected for input and output for the script, not debug messages. It's connected to standard input (stdin) and standard output (stdout)
The VS Code debug console is automatically attached to the script + debugger. View the debug output with Ctrl-Shift-Y or click 'DEBUG CONSOLE'.
A working config for you.
Not this:
    "configurations": [
      {
        "type": "bashdb",
        "request": "launch",
        "name": "Bash simple",
        "cwd": "${workspaceFolder}",
        "program": "${file}",
        "terminalKind": "debugConsole"   <---- **Not this**
      }
    ]

Use this:
    "configurations": [
      {
        "type": "bashdb",
        "request": "launch",
        "name": "Bash simple",
        "cwd": "${workspaceFolder}",
        "program": "${file}",
        "terminalKind": "integrated"   <---- Use this, the integrated terminal.
      }
    ]

HTHs someone.
Debian 11 works!!!

i just only set : "terminalKind"<EMAIL_ADDRESS>and it worked but debug console says /root/.vscode/extensions/rogalmic.bash-debug-0.3.4/bashdb_dir/command/source.sh: line 41: /dev/stdin: No such device or address

Funy lifehack to save nerven :)

i change the setting to         "terminalKind": "integrated"  then everything start working perfectly!

my vscode server says ".vscode-server/extensions/rogalmic.bash-debug-0.3.9/bashdb_dir/lib/filecache.sh: line 131: printf: `B': invalid format character"

and the fix is:
.vscode-server/extensions/rogalmic.bash-debug-0.3.9/bashdb_dir/lib/filecache.sh: line 131
replace printf with normal form:
  typeset find_file="$(printf '%s\n' "$1")"

@all test this PR: https://github.com/rogalmic/vscode-bash-debug/pull/186


Error retrieving data from .com

Thank you for this great package! when i make more than 5 requests getting this error Error retrieving data from .com. Google is pretty hard on scrapers.  After some research, I found that this issue can be fixed with proxies. This Proxy package https://www.npmjs.com/package/simple-proxies can be used with google-index ? This package https://github.com/christophebe/serp also using simple-proxies and https://www.scraperapi.com

Hi @adnanhassan23, the package does not do requests from Node.js directly - it uses headless browser. So I assume if you need proxy - it should be configured on network level, no code

Hi @rodion-arr Thank you for your response. According to this comment https://github.com/puppeteer/puppeteer/issues/1948#issuecomment-582617405 i changed request URL
    const googleDomain = 'https://google.com/search?hl=en-US&rls=en&q='
    const requestUrl = `${scraperApiUrl}${googleDomain}site:${site}`

Is this good practise?

@adnanhassan23, it's OK to pass request URL like you've showed.
Regarding "network level" - I meant to setup proxy on server level and left the code part unchanged, e.g. setup http_proxy env variable.


update versions for march Identity release

API change check
API changes are not detected in this pull request.


PFConfig not returning dictionary

CentOS VPS, iOS
After migration any PFConfig calls return "JSON text did not start with array or object and option to allow fragments is not set.- error code 3840". If i point the app back to the main parse instance everything works great and i get the dictionary returned. Is PFConfig not supported with parse-server?

Config is still in an experimental state until tested fully with the coming dashboard release.  You can enable it now by setting the environment variable PARSE_EXPERIMENTAL_CONFIG_ENABLED=1
If you've done this and are still having a problem, please re-open.  Thanks!

Is PFConfig still in an experimental state? I'm still getting error 3840 even with PARSE_EXPERIMENTAL_CONFIG_ENABLED=1.


Added jacoco and findbugs plugin to make you guys aware of the quality

To run jacoco code coverage run the following command:

gradlew clean build jacoco jacocoTestReport

Build currently fails because of too many vulnerabilities detected by findbugs.
Have fun!

Hi @onimurasame thanks for the PR. Can you update your pull request and add it as part of travis? That will help us make sure we don't merge things that aren't passing those tests.


[Backport release-1.8] [c++] Fix building TileDB if a CMake toolchain is not specified

Backports #2244 to the release-1.8 branch

R CI is a known problem; merging


'wget --credit' does not show anything while it should at least credit Keivan and AppGet

Brief description of your issue

Steps to reproduce
winget --credit

Expected behavior
A message shows up and gives credits to Keivan and AppGet for the original idea.

Actual behavior

The parameter --credit  is not recognized.
Environment
[winget --info]
Windows Package Manager version: any version
Windows: 20H1

Any other software?

Hey @tigerinus, thanks for creating this issue. However, it seems that a better place for this  feedback is #353 where MS is discussing this with the community and Keivan. So I'd recommend closing this issue and adding your input to the one I linked :-)


Add option to allow 'duplicate-cn' (off by default)

This PR adds an option to enable duplicate-cn which might be useful in some cases (if a client has multiple devices and wants to share the same ovpn config on all of them).
https://openvpn.net/community-resources/reference-manual-for-openvpn-2-0/

–duplicate-cn
Allow multiple clients with the same common name to concurrently connect. In the absence of this option, OpenVPN will disconnect a client instance upon connection of a new client having the same common name.

Note that this option has some security implication and this is not enabled by default in the wizard.
Do you want to allow concurrent connection from multiple clients with same common name?
   1) No
   2) Yes (not recommended)
Duplicate CN [1-2]: 1

One certificate should be used per device, this pull request will not be merged.
It is very rare to have a good justification to enable this feature.

I totally agree, but then that's exactly the reason the default value is to not allow, unless one has that rare reason.


(PERSONAL ACCOUNT PAGE)  Updating user's email is not work

When the user changes the email in the profile, it is not saved. This may be because it needs to be confirmed (BA: required).
Screencast:
https://github.com/Sunagatov/Iced-Latte-Frontend/assets/140279745/95386eda-3f0c-47d9-b65a-53fc88afb4d5

Probably it was fixed by somebody. It works as expected.

@iakivpekarskyi
New email is not recorded in the database. If user re-login, old email is presented in profile.


test ignore

Acrolinx Scorecards
A minimum Acrolinx score of 80 is required.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
AcrolinxScore
LowestCategoryScore
Issues
Scorecard
Processed

enterprise-graph/conflation-concepts.md
76
39
34
link
:white_check_mark:

enterprise-graph/conflation-model-tutorial.md
82
56
19
link
:white_check_mark:

enterprise-graph/conflation-validation-tutorial.md
100
100
0
link
:white_check_mark:

More info about Acrolinx

Install Acrolinx locally for VSCode
False positives or issues

OPS Build status updates of commit bcaa9a3:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

enterprise-graph/conflation-concepts.md
:white_check_mark:Succeeded

enterprise-graph/conflation-model-tutorial.md
:white_check_mark:Succeeded

enterprise-graph/conflation-validation-tutorial.md
:white_check_mark:Succeeded

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

closing - was checking for preview links. none.


.ads files not explained at the point they are introduced

I hope you don't mind if I post my impressions as I go. As someone who has written docs, I'm probably more nit-picky than most. If my comments aren't useful, please let me know.
 For anyone who hasn't used Modula, their role and usage is not obvious. I had to google to put them into some context.
https://learn.adacore.com/courses/intro-to-ada/chapters/subprograms.html
To the new user from languages like Python, C# & Java they might look like an irritation. A crisp explanation of their advantages to both the compiler and the developer might help cushion the blow.

I second this. The course is meant to be an introduction to Ada, so explanations, even short ones, on fundamentals like what an ADS file is should not be skipped.


Website Interactiveness

Hi @avinash201199 ,
I have noticed that the future predictions of weather forecast in your website is not that interactive.
I would to add some hover functions and a detailed view for each card(each day) in the future forecast.
Kindly assign this issue to me

I have made the PR. Please check it and let me know how I can deal with the html error I was facing in #29 .


Initial value from API

For initial value, I got error "Invalid constant value" when I put my string from API.
How to set initial for the editor from data from API?

don't make HtmlEditorOptions class const, should solve the issue :)


Honor CFLAGS & LDFLAGS

Currently, the Makefile doesn't honor the environment's CFLAGS & LDFLAGS. This PR addresses that.

Thank you!


Getting value problem

Hi,
Sometimes I get the null value from selectpicker. I always get the successfull result, If I put a delay between setting value and displaying data'. On the other hand  I tried "render" method instead of "refresh". But the result didn't changed. What should I do?
$('#gateway').val(data.GatewayID);
$('#gateway').selectpicker("refresh");
console.log(data.GatewayID);
console.log($('#gateway').val());
Successfull output of console:
1
1
Sometimes I get null value. The output of console:
1
null

.val() is a native jQuery method. If you're running into issues retrieving the value after it has been set, that sounds like a jQuery issue.
Bug reports must include a live demo of the problem. Per our contributing guidelines, please create a reduced test case via Plunker, jsFiddle, CodePen or JS Bin, and report back with your link, Bootstrap version, bootstrap-select version, and specific browser and OS details.


Sidebar social links?

Hi,
what is the trick to make social links to show up in the sidebar?

I have the same question too, couldn't find any thing, ended up modifying sidebar.html to add them in my local copy

I have updated this theme to latest bootstrap 4. I am closing this issue for now. Please open a new one if the latest theme does not do what you need.


fix: landing page broken on small screen

PR Checklist
Please check if your PR fulfills the following requirements:

[ ] Tests for the changes have been added (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] infrastructure changes
[ ] Other... Please describe:

What is the current behavior?
Currently the landing page width is broken when opened on small screens and when pressing the dark mode toggle.
fixed #1572
What is the new behavior?

The width is consistent, not broken, and does not change when pressing the dark mode toggle button.
The logo (used in the hero) is more precise and sharp.

Other information
On 375px screen:

Thank you