Improve Binance getTicker log entry

Resolves #1953

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Log improvements

What is the current behavior? (You can also link to an open issue here)

Binance log code consistency.
Binance getTicker call logs the full response from API causing too much noise in log files.

What is the new behavior (if this is a feature change)?
The getTicker call logs how many symbols were loaded.

Other information:

great stuff!
I actually want to remove all binance output logs at some point, they were added to deal with the many problems people were encountering with the binance API (a while ago).

Sure let me know if you need a hand. Thanks.


(C) C23's digit separators are highlighted incorrectly

Describe the issue
C23 has added digit specifiers to the language (', just like C++). They are not highlighted correctly in the code. It seems as if the engine treats it as the start of a string/character
Which language seems to have the issue?
c
Are you using highlight or highlightAuto?
Whichever one StackExchange uses.
Sample Code to Reproduce
#define _POSIX_C_SOURCE 200'809L
#define _XOPEN_SOURCE   700

Expected behavior
It should recognize it as a digit separator instead of a character or a string opening quote.
Additional context
This was found at this code review post: https://codereview.stackexchange.com/revisions/292149/6
And it was fixed by such a hack:
#define _POSIX_C_SOURCE 200'809L // '

Does our C++ grammar properly support this already.  If so might be an easy port.

@joshgoebel based on the release over on Stack Overflow, neither C or C++ support the grammar.

Looks like both C++ and C already have support for digit separators. For example the regex for NUMBERS in C++ allow usage of single quotes as seprators:
https://github.com/highlightjs/highlight.js/blob/da79da636baa790cfceda931554186ac17f7f48b/src/languages/cpp.js#L52-L95
From my understanding the actual issue is that the contains for PREPROCESSOR does not include NUMBERS but does have STRINGS causing the beginning of the single quotes to be considered a string:
https://github.com/highlightjs/highlight.js/blob/da79da636baa790cfceda931554186ac17f7f48b/src/languages/cpp.js#L104-L116
I am not familiar with Highlight.js but my guess is that adding NUMBERS in PREPROCESSOR.contains should fix the issue

I am not familiar with Highlight.js but my guess is that adding NUMBERS in PREPROCESSOR.contains should fix the issue

I think that would do a bit too much (for some directives), but that's the right idea.  I think the pre-processors may need to be handled more precisely... for example #define x CODE might deserve full highlighting (for CODE) but not others... I'm trying to think of what false positive cases we might have.

is this still open
i recently made a commit regarding a similar issue i believe i can contribute

@joshgoebel  upon further inspection the PR i made yesterday
handles all digit seperators. I believe we can mark this issue as resolved

Resolved by https://github.com/highlightjs/highlight.js/pull/4094


side-scrolling with off-set for long input

Sometimes I find myself deleting too much of a text when for example renaming a file in nvim-tree.lua because I can't see the characters to the left. It would be really nice if I could configure the offset of characters to the left so that I can see what I'm deleting next.

This is a great use case for :help sidescrolloff. You can set that by passing it in as a window option
require("dressing").setup({
  input = {
    win_options = {
      sidescrolloff = 4,
    },
  },
})

Uuuuhhh and here I thought I looked through all the docs. I didn't know Neovim comes packed with this! Thank you!

@jan-xyz How did you get the input to extend beyond the nvim-tree buffer? Mine is always limited to the width of that buffer.


GD-309: push_error() should be captured when it occurs in an assert_error() lambda

Is your feature request related to a problem? Please describe.
When using assert_error along with is_push_error, the error shows up in Godot's error list when running tests. This is a bit confusing, and may potentially mask other, actual error that occur during the tests. Enabling "report of push_error as failure" doesn't work as a workaround, as that makes even expected errors (is_push_error) cause failures.
Describe the solution you'd like
Ideally push_error would not show up as an actual error when called in the context of an assert_error lambda.
I'm not sure how hard this is to do or it's even possible with Godot currently (can push_error be temporarily overridden with a custom implementation?), but wanted to file the feature request regardless.
Describe alternatives you've considered
There's no real workaround as far as I know, but it's not the biggest problem so at the moment it just requires more careful manual inspection of the errors list after running tests.
Additional context
N/A

This is already fixed in V4.2.1.
Close as duplicate to #303

Oh thanks!

No problem, feedback and improvements always welcome.

I updated my project to the latest gdUnit4 master, and one small thing I noticed is that the error seems to still go uncaptured when running the "Run overall tests" command, but it will correctly capture if you just run the unit test directly. I don't exactly know the difference between the two run modes beyond which tests they have selected, but this seemed a bit surprising (although it's easy to work around).

I updated my project to the latest gdUnit4 master, and one small thing I noticed is that the error seems to still go uncaptured when running the "Run overall tests" command, but it will correctly capture if you just run the unit test directly. I don't exactly know the difference between the two run modes beyond which tests they have selected, but this seemed a bit surprising (although it's easy to work around).

Hi @voithos, that sounds like an error. The test results should be the same regardless of how you perform the tests.
Please create a new bug issue, and add steps to reproduce.


[trax] Improve handling of data_dir.

Give it a default.
Let it be specified on the command line unconditionally, rather than
silently ignoring it sometimes.
Remove data_dir = None from model configs now that data_dir has a default.

cc @afrozenator

@jlebar this is a little more subtle (see the internal code), but I believe Anselm's change should fix the issue you see here -- https://github.com/tensorflow/tensor2tensor/commit/9db59c5cba0c847b0e4e76d174f19147e044a0c1
Long story short, the default internally should be placer, which we do now if we find data_dir is set to None, externally Anselm's CL will kick in and give it a reasonable default.
Let me know if that works - I believe it should, unless I'm missing something? We can chat internally as well.
@lukaszkaiser

I think this became obsolete, closing for now. Please reopen if the issue is still there!


Extend websockets to full Websocket support by enabling queries and mutations over wss

Right now, only subscriptions are supported over wss. It would be helpful for a number of use cases to have support for queries and mutations over wss. It's supported by the latest Apollo Subscription Client (https://github.com/apollographql/subscriptions-transport-ws#full-websocket-transport).

I'm really looking forward for this feature...
Surely I'm not aware of all the details involved in making this happen, but I guess it shouldn't be too hard to make this happen.
I've been waiting for this like for several months now, been waiting secretly on the emails to inform me of this thing being implemented. Sadly, it hasn't happened yet. Hopefully it will happen soon though. I know this probably isn't a priority for you guys, but for countries like mine (Madagascar), the reliability this provides is huge.


[chore] prefer interfaces to types

I remember we had done this in Sapper. Makes sense to do it here too probably

@benmccann The big difference here is that interfaces can be extended while types can't. So mostly user-customizable interfaces and internal types is mostly the right rule of thumb!

@ignatiusmb you might be interested in this PR too given all the TypeScript stuff you've done. If it looks good to you or I don't hear anything I'll probably merge it in the next day or so

I'm certainly no TS expert, but as I understand the only remaining difference in latest TS is that interface supports implicit open typing and merging whereas type will error - https://www.typescriptlang.org/docs/handbook/2/everyday-types.html#differences-between-type-aliases-and-interfaces
Seems to me type would then be less error prone as extending the type def would always need to be explicit.
Again, no TS expert.

I think interfaces give some flexibility when it comes to customization, being able to extend types without having to create new ones can be useful in some situations.

I think interfaces give some flexibility when it comes to customization, being able to extend types without having to create new ones can be useful in some situations.

For sure, in some situations. Just seems like a huge footgun. I see this as changing from const to var because extending a const value is useful sometimes. Seems overkill to make all const values vars.

Yeah, but you can only extend an interface by redeclaring it (sort of), and if people take the time to redeclare an interface they usually know what they are doing


Raster reprojection example does not work

Describe the bug
Raster reprojection fails with Uncaught TypeError: Cannot read property 'x' of null
To Reproduce
Steps to reproduce the behavior:

Go to https://openlayers.org/en/latest/examples/reprojection.html?q=reproject
set view projection to EPSG:3413
See error in console Uncaught TypeError: Cannot read property 'x' of null

Expected behavior
Valid example

This is caused by an issue in proj4. See proj4js/proj4js#332 for a fix.

I don't think it is needed in the example if proj4 is being updated but if you are encountering reprojection errors elsewhere and cannot wait for a new release using addCoordinateTransforms to wrap error catching around the transforms is a simple workaround https://codesandbox.io/s/reprojection-m05mt

Thank you for the clean workaround ;) I will use this until proj4 is updated.


[WIP] Switch dashboard queries to client metrics where appropriate

Do not review until community discussion is done.
Please provide a description for what this PR is for. https://github.com/istio/istio/issues/27595
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[x ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[x ] User Experience
[ ] Developer Infrastructure
Pull Request Attributes
Please check any characteristics that apply to this pull request.
[ ] Does not have any changes that may affect Istio users.

@carolynhu: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
4704f89cf7c7befe9f5e5530f5cb806734ecefb4
link
/test release-notes_istio

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

@carolynhu: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
5c65cd4708cee914c98c97da3500aec2adca832c
link
/test release-notes_istio

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

@carolynhu: PR needs rebase.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@carolynhu: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
5c65cd4708cee914c98c97da3500aec2adca832c
link
/test release-notes_istio

integ-pilot-multicluster-tests_istio
5c65cd4708cee914c98c97da3500aec2adca832c
link
/test integ-pilot-multicluster-tests_istio

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

@carolynhu: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-pilot-multicluster-tests_istio
5c65cd4708cee914c98c97da3500aec2adca832c
link
/test integ-pilot-multicluster-tests_istio

release-notes_istio
22295a39a12f66b44c2f39bf9306ed2af39c7e4c
link
/test release-notes_istio

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

Service Dashboard:
(1) General Section
1-1: "reporter=source"
https://snapshot.raintank.io/dashboard/snapshot/1YEYl9cfTBhra04ZQwGnXkyFWxn46p2g?orgId=2
1-2: "reporter=destination"
https://snapshot.raintank.io/dashboard/snapshot/FO5W114pn5EKJdenHYlHF9TS8z4dEH1D?orgId=2
(2) Client Workloads Section:
2-1: "reporter=source"
https://snapshot.raintank.io/dashboard/snapshot/mMqSprQQEUrQyELzYJnRVmMaaSJJ1wLQ?orgId=2
2-2: "reporter=destination"
https://snapshot.raintank.io/dashboard/snapshot/LekV026uU567CxOCyUbzDJUdQnr3CPzu?orgId=2

@carolynhu: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
37033caa1650b4b955804a155c796d300f332171
link
/test release-notes_istio

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

Workload Dashboard:
(1) General Section
1-1: "reporter=source"
https://snapshot.raintank.io/dashboard/snapshot/P2XpzaejDh31BfguiGHxuZyDGr9i0ZsZ?orgId=2
1-2: "reporter=destination"
https://snapshot.raintank.io/dashboard/snapshot/Ow0zpBPrxT1YiHpoC5K2gR2v2NWH8qi0?orgId=2
(2) Inbound Workloads Section:
2-1: "reporter=source"
https://snapshot.raintank.io/dashboard/snapshot/qivlqYUpKMLgwQ0o0AaBT2VYMMa2UT3V?orgId=2
2-2: "reporter=destination"
https://snapshot.raintank.io/dashboard/snapshot/dOZp344KjQJG475rN3UXTEiQPoGkfvaa?orgId=2

@carolynhu: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
efdc9eb023593704a366abe37884390e03d3bcdb
link
/test release-notes_istio

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

Looking quickly at this, my reaction is that having multiple drop-down selectors for reporter is too much. If we go the drop-down route, let's keep it to a single one.

I agree with Doug, I was a bit confused about what each drop down did. I
think a single one will be more clear
On Mon, Nov 23, 2020 at 11:54 AM Douglas Reid<EMAIL_ADDRESS>wrote:

Looking quickly at this, my reaction is that having multiple drop-down
selectors for reporter is too much. If we go the drop-down route, let's
keep it to a single one.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/istio/istio/pull/28564#issuecomment-732390297, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEYGXL6VGBV22KV526UFYLSRK4WHANCNFSM4TJR4DIQ
.

hmm... my initial intention is to let user select reporter for different sections, okay, if that seems too much, let me shrink them into one

Update:
(1) Service Dashboard:
"reporter=source"
https://snapshot.raintank.io/dashboard/snapshot/tgll8pZMRgEebOVo0d37dORnsiaz4bb2?orgId=2
"reporter=destination"
https://snapshot.raintank.io/dashboard/snapshot/ZwmPqHU7YhG5QpxFigM90Imh97nIFBhZ?orgId=2
(2) Workload Dashboard:
"reporter=source"
https://snapshot.raintank.io/dashboard/snapshot/zVcSextJ1cqPOM43YQDARk60VpNmgt4n?orgId=2
"reporter=destination"
https://snapshot.raintank.io/dashboard/snapshot/33pqQoAXeTJ6vT1P3HVi6PXJsR60GXXG?orgId=2

Update:
(1) Service Dashboard:
"reporter=source"
https://snapshot.raintank.io/dashboard/snapshot/SVuGenHApGGUkFuLUkN7KqpSqJAsHYfx?orgId=2
"reporter=destination"
https://snapshot.raintank.io/dashboard/snapshot/GescHkKEWx4L2F5Y3tIdpgQO5r8VbfNT?orgId=2
(2) Workload Dashboard:
"reporter=source"
https://snapshot.raintank.io/dashboard/snapshot/PuVjs0p9SgQ2oyfdWVmCxO89VJZQALOo?orgId=2
"reporter=destination"
https://snapshot.raintank.io/dashboard/snapshot/I4DuUpRNdJFPM5CY6LhhydosoGczPvjw?orgId=2

@douglas-reid @howardjohn ptal

I think we need to figure out a better term for it. Is it possible with those drop-downs to have a different value than the text displayed?
Maybe we could call the drop-downs Proxy instead ?  Would that be obvious to someone not intimately familiar with Istio?
@jmazzitelli what do you think here?


Add support for sub-header text in Login component

Related to https://github.com/patternfly/patternfly-next/issues/354
Need to add support for the "Log in to Application" text as seen on https://rawgit.com/patternfly/patternfly/master-dist/dist/tests/login-single-sign-on.html
Details to come.
cc @tlabaj @mceledonia

Let's go with small text style (14px) underneath the header with spacing: 16px top, 32px below..

@mceledonia the bottom spacing is a bit different. After the header text currently, there is 16px bottom padding on the header, then the hidden error message, and then 24px (lg) spacer as the gap between the error message and first form field.
Here I just left all of the existing spacing the way it is and added the sub-header text (description) with 16px between it and the content above it. Then on mobile, there is 16px between the title, description, and dropdown.
Desktop

Desktop with error

Mobile

Mobile with error

I think it looks OK, but the spacing between the description and error on desktop seems too tight. Here it is with an 8px (sm) spacer below the description on desktop only.
Regular

Error

This all looks great, thanks @mcoker !!


All-time return static value

This will be fixed when we fix #2
All-time return is currently a static value in the code. I am waiting for communities' input on how to calculate all-time return before we implement it in the plugin.

Your all time return is your capital gains + dividends earned


[feature] Support aggregate functions in select

This little change adds support to aggregate functions directly to select.
Example:
$rows = Flight::db()->select('table',
[ 'column1', 'column2', 'COUNT'],
['GROUP'=>['column1','column2']);

Fix #191 & #192

I am missing a function to get the columns of a table. Even if the table is empty!
It is possible to add this function?

GROUP BY  multiple columns is very useful.
I usually use this way count data.

Actually, group by only one column is a restriction, not a functionally.
El 28/03/2015 06:41, "姚振"<EMAIL_ADDRESS>escribió:

GROUP BY multiple columns is very useful.
I usually use this way count data.
—
Reply to this email directly or view it on GitHub
https://github.com/catfan/Medoo/pull/190#issuecomment-87209919.

This request is closed because it is now able to do that via raw object.


Copywriting

Hello!
I've been watching your project and I want to make the copywriting stuff: Contact Us, About Us, FAQ, etc
Best Regards!

Nice! But It needs a discussion. Because all TOAST UI products are written using a common README template. I will leave a comment when it is determined. Thanks!

Great! I'll wait. Thank You

Thanks for your attention @navxx.
By the way, I wonder where does need copywriting stuff? tui-calendar is under MIT licensed as you know. So we don't receive any additional stuff.
What do you think?

No discuss. I close this.
If you have any question, please don't hesitate to leave a issue.


investpy data is not being placed in the package directory on installation

It has been spotted that when installing investpy, the data contained as resources/ is not being placed inside the Python package directory and it is being placed in the root directory.
Described issue is presented in the following image:

So a way to include that data inside the Python package directory (.../site-packages/investpy/...) should be developed. Anyways the package works properly, but this should be fixed since it is not the pythonic way of doing it.

Anyways, data is being duplicated, which means that the original data is downloaded both in resources/ (which is the correct directory) and in the base directory. So on, there is an issue affecting the installation of investpy (I guess) that is duplicating the data.
Any help is appreciated! Thank you!

It has been found out that the problem was in the setup.py file since it was including the package data twice by using both include_package_data and data_files keys in the setup dict. So on, removing the data_files values seemed to fix the error. Anyways further tests need to be done before this issue is considered closed.


checkout-ui-extensions version 0.27.0 (pre-publish)

Background
This brings over the latest from checkout-web. This will be the last version of checkout-ui-extensions, before we move to the unified ui-extensions package. It will be matched with our corresponding, first, stable 2023-04 version.
Preview the generated docs (will be published in a separate PR, after this merges to main).

Note: the preview of the docs includes both an unstable and a 2023-04 version which are the same right now.

Solution
Followed the steps in releasing libraries.
There were follow-up commits made in this PR to bring in last minute changes from the following PRs:

https://github.com/Shopify/checkout-web/pull/22782
https://github.com/Shopify/checkout-web/pull/22588
https://github.com/Shopify/checkout-web/pull/22693
https://github.com/Shopify/checkout-web/pull/22736

🎩
Tophat changes by copying the packages from Spin into a real scaffolded app / extension.
Change into the app directory on your local machine and run this:
VM=direct.$(spin show -o fqdn)
rsync -av --delete $VM:~/src/github.com/Shopify/ui-extensions/packages/checkout-ui-extensions node_modules/@shopify/ && \
   rsync -av --delete $VM:~/src/github.com/Shopify/ui-extensions/packages/checkout-ui-extensions-react node_modules/@shopify/

You can tophat the generated docs on this spin instance.
Checklist

[x] I have :tophat:'d these changes
[x] I have updated relevant documentation

There are two PRs that we are going to wait for to include these changes:

https://github.com/Shopify/checkout-web/pull/22588
https://github.com/Shopify/checkout-web/pull/22736 (optional)

@jamesvidler I have a change coming up that would be great if we can include as well:
#1041

Included! 👍


DependencyResolver / DependencyResolverProxy alias #128

Hello!
Just implemented https://github.com/scribd/Weaver/issues/128

Thank you @MontakOleg,
I will do some testing and merge if I don't find any issue.

Hi @MontakOleg,
After trying it out, it looks like your changes make Weaver create an extra typealias for every type, which is a lot of added boilerplate code.
Since we're trying to solve a scenario which maybe happens ~5% of the time, I think we should find a way to solve this by only impacting the Proxy structs and not the rest of the code. It looks like a more scalable solution to me.
I'm thinking that a better solution to this problem could be to rename the concerned dependency resolvers only to AInternalDependencyResolver and rename their proxy struct to ADependencyResolver. It would scale better this way.
We could even go further by having the proxy structs implementing the internal dependency resolver. It would allow us to make the value attribute private and hide the proxy aspect entirely to developers.

Here is an example of what I'm talking about above.
protocol AInternalDependencyResolver {
    var anAmbiguousDependency_ad0737d39be39483b642e7dbfef913a379412836: String { get }
}

typealias ADependencyResolver = ADependencyResolverProxy

struct ADependencyResolverProxy {

    let value: AInternalDependencyResolver

    init(_ value: AInternalDependencyResolver) { self.value = value }

    var anAmbiguousDependency: String {
        return value.anAmbiguousDependency_ad0737d39be39483b642e7dbfef913a379412836
    }
}

Hi @trupin,
Got it.
Can't fix this right now, will look at this closer a bit later.

@trupin done, please look at this

@trupin This situation should be indicated in the README to avoid confusion until the issue being solved. I thought to use DependencyTypeDependencyResolverProxy as a workaround by exploring the generated code file. It can be challenging for new Weaver users.

Thanks for these changes @MontakOleg.


WIP: Main loop api and singlelogparser class

This adds:

SingleLogParser class to parse a full log by delegating to the section handlers
Main loop with both parse("*.log").summary() and get_dataframe(["*.log"]) alternatives for compatibility with the old API

@maliheha there is still a bit to do here but could you take a look at single_log_parser.py and api.py?
Some other changes:

Removed feather files, regression tests now run directly against the old code still in the repo
Updated grblogtools.py to v1.3.1 (merged in master branch) to test against this code directly

Also - integrating the termination parser with the different termination messages from barrier and nodelogs is looking tricky. Switching the termination parser settings on the fly once we determine the model type doesn't seem to fit very well. What do you think of the following?

Include "Optimal objective" in the barrier parser
Include "Best objective/best bound/solution count" as well as "Optimal solution found" in the nodelog parser, since they are specific to models with a node log
Instead of the termination parser, have a status message parser which picks up "Time limit reached", "Interrupted" and so on (without interrupting other parsers).

In the case of a MIP where barrier solves the relaxation but the node processing times out, when merging all the log info, Status="TIME_LIMIT" from the status parser would then taken precedence over Status="OPTIMAL" coming from the barrier parser, and we would get the right result.


[Bug]: Email addresses containing non-latin characters like ö are not treated correctly

Is there an existing issue for this?

[X] I have searched the existing issues

Description
When I put an email address in a Text that contains an ö (for example), only the part after the special character is turned into an email link, like so:
<EMAIL_ADDRESS>The link target is similarly mangled.
Steps To Reproduce
Make a Text element that contains an email address containing the letter ö.
Public Sample App
No response
Issue video log
No response
Version
Appsmith Community v1.9.2

Today, appsmith doesn't render email addresses as links so this issue is not valid anymore


refactor(Vehicle_cmd_gate): rework parameters

Description
Implement the ROS Node configuration layout described in https://github.com/orgs/autowarefoundation/discussions/3371 for the vehicle_cmd_gate package.
Remove the default value from the source code in order to ensure all parameter values are passed from the parameter files.
Create the schema
Tests performed

Not applicable.
Effects on system behavior
More reliable and faster parameter configuration file creation.
Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[ ] I've confirmed the contribution guidelines.
[ ] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

Hi @karishma1911 Thanks for your contribution. But this PR seems to include unnecessary commits. Could you rebase this PR with latest main branch?

Not applicable.

Due to uncommitted changes has been pushed in this PR, I want to close this PR
Same changes has been pushed in this PR, I requested to go through this https://github.com/autowarefoundation/autoware.universe/pull/5652

@karishma1911 OK. Please close this PR. I'll review #5652.

@karishma1911 OK. Please close this PR. I'll review #5652.


Use pypy3 to run all days

Pypy3 is faster than Python. In order to work properly, the following tasks must be performed:

find a suitable Docker image
investigate why pypy3 is not working for day 19 (it produces a seg fault which seems quite random, further investigation is needed)

Finally I had to change day 19 solution by implementing an iterative function instead of a recursive one.


Debugging angular code with Visual Studio & asp.net core

Is it possible to debug typescript code in the integrated with asp.net core angular-cli project (VS 2015)?

I'll leave this open for the community to answer, but don't have an answer myself.

Closing as there doesn't seem to be any discussion going on.


Just checking out Milligram for the first time. Maybe I'm missing something, but how do I have a grid column be 100% on small screens and then switch to 50% on large screens? Is that built into the CSS with a class that I'm not seeing or do I have to override that in my project?

Thanks for bringing this to my attention @mikeriley131 I was about to start using Milligram on a project but that's a big missing feature. It's not too hard to manually add it, but isn't the purpose of a css framework that you don't need to do this by hand :-).

Currently Milligram does not support this functionality but I liked the idea. Initially Milligram was built with the purpose of being the simplest framework, defining only the minimum of possible styles. So it does not have so many features easily found in large frameworks.
I'll think of something but if you have any idea how we could implement this functionality and want to submit a PR, feel free.
Thank you for sharing your thoughts with us.

@cjpatoilo I hear you and I agree that's exactly what I liked about Milligram...just the bare minimum...but for me this a basic and essential responsive design feature.

OK, thanks @cjpatoilo. Happy to create a PR, just wanted to make sure I wasn't missing something first.

@mikeriley131 happy to help, review PR, etc...

Created pull request 155 to add the responsive grid feature: https://github.com/milligram/milligram/pull/155

I played around with a media query, bear with me. I'm a guy who likes back-end stuff and plays around with Linux most of my time so take my front-end development with a grain of salt. Also I haven't done anything in any css pre-processor.
@media (min-width: 495px) and (max-width:800px) {
.row .column[class*=" column-"] {
max-width: 100%;
}
}
It matches a .row with a .column in the child or grand-child(why would you want even want an element in between of .row and .column?) which is followed by a .column-* where the * is a wild-card.
What do you think?
remarks : 495 pixels is the limit because that's the minimal resized amount chrome allowed me. 800px has no significant remark.

Hello Sir,
I would like to contribute to this project,
I am new to hacktoctoberfest and this would be a great beginning.
Thank you...

Hello Sir, I would like to contribute to this project, I am new to hacktoctoberfest and this would be a great beginning. Thank you...

@Zaid-Parkar that's off topic, please send a message directly to cjpatoilo.


Egg Dropping Problem in Python

💥 Proposal
I want to implement the Egg Dropping Puzzle problem in Python.
Have you read the Contributing Guidelines on Pull Requests?
Yes.

/assign


building OpenBLAS 0.2.20 with LAPACK 3.7.1 fails with unresolved symbols

We usually building OpenBLAS with an official release of LAPACK included, rather than using the lapack-netlib directory as is.
This approach, where we basically just replace lapack-netlib with the unpacked source tarball for the (usually latest) LAPACK release, has worked fine in the past (e.g. with OpenBLAS 0.2.19 & LAPACK 3.7.0, or OpenBLAS 0.2.18 & LAPACK 3.6.1).
When trying to build the latest OpenBLAS 0.2.20 with the latest LAPACK 3.7.1, I'm running into problems however:
make[1]: Entering directory `/tmp/OpenBLAS/0.2.20/GCC-6.4.0-2.28-LAPACK-3.7.1/OpenBLAS-0.2.20/exports'
make[1]: warning: -jN forced in submake: disabling jobserver mode.
perl ./gensymbol linktest  x86_64 _ 0 0 0 0 0 0 "" "" 1 > linktest.c
gcc -O2 -march=native -O2 -DMAX_STACK_ALLOC=2048 -Wall -m64 -DF_INTERFACE_GFORT -fPIC -DSMP_SERVER -DNO_WARMUP -DMAX_CPU_NUMBER=24 -DASMNAME= -DASMFNAME=_ -DNAME=_ -DCNAME= -DCHAR_NAME=\"_\" -DCHAR_CNAME=\"\" -DNO_AFFINITY -I. -O2 -DMAX_STACK_ALLOC=2048 -Wall -m64 -DF_INTERFACE_GFORT -fPIC -DSMP_SERVER -DNO_WARMUP -DMAX_CPU_NUMBER=24 -DASMNAME= -DASMFNAME=_ -DNAME=_ -DCNAME= -DCHAR_NAME=\"_\" -DCHAR_CNAME=\"\" -DNO_AFFINITY -I.. -L/prefix/software/GCCcore/6.4.0/lib64 -L/prefix/software/GCCcore/6.4.0/lib -shared -o ../libopenblas_haswellp-r0.2.20.so \
-Wl,--whole-archive ../libopenblas_haswellp-r0.2.20.a -Wl,--no-whole-archive \
-Wl,-soname,libopenblas.so.0 -lm -lpthread -lgfortran -lm -lpthread -lgfortran
gcc -O2 -march=native -O2 -DMAX_STACK_ALLOC=2048 -Wall -m64 -DF_INTERFACE_GFORT -fPIC -DSMP_SERVER -DNO_WARMUP -DMAX_CPU_NUMBER=24 -DASMNAME= -DASMFNAME=_ -DNAME=_ -DCNAME= -DCHAR_NAME=\"_\" -DCHAR_CNAME=\"\" -DNO_AFFINITY -I. -O2 -DMAX_STACK_ALLOC=2048 -Wall -m64 -DF_INTERFACE_GFORT -fPIC -DSMP_SERVER -DNO_WARMUP -DMAX_CPU_NUMBER=24 -DASMNAME= -DASMFNAME=_ -DNAME=_ -DCNAME= -DCHAR_NAME=\"_\" -DCHAR_CNAME=\"\" -DNO_AFFINITY -I.. -L/prefix/software/GCCcore/6.4.0/lib64 -L/prefix/software/GCCcore/6.4.0/lib -w -o linktest linktest.c ../libopenblas_haswellp-r0.2.20.so -L/prefix/software/GCCcore/6.4.0/bin/../lib/gcc/x86_64-pc-linux-gnu/6.4.0 -L/prefix/software/GCCcore/6.4.0/bin/../lib/gcc -L/prefix/software/GCCcore/6.4.0/lib64/../lib64 -L/prefix/software/GCCcore/6.4.0/lib/../lib64 -L/prefix/software/GCCcore/6.4.0/bin/../lib/gcc/x86_64-pc-linux-gnu/6.4.0/../../../../lib64 -L/lib/../lib64 -L/usr/lib/../lib64 -L/prefix/software/binutils/2.28-GCCcore-6.4.0/lib -L/prefix/software/GCCcore/6.4.0/lib64 -L/prefix/software/GCCcore/6.4.0/lib -L/prefix/software/GCCcore/6.4.0/bin/../lib/gcc/x86_64-pc-linux-gnu/6.4.0/../../..  -lgfortran -lm -lquadmath -lm -lc   && echo OK.
/tmp/eb-vlSJIi/ccsLO4XL.o:linktest.c:function main: error: undefined reference to 'LAPACKE_clatms'
/tmp/eb-vlSJIi/ccsLO4XL.o:linktest.c:function main: error: undefined reference to 'LAPACKE_clatms_work'
/tmp/eb-vlSJIi/ccsLO4XL.o:linktest.c:function main: error: undefined reference to 'LAPACKE_dlatms'
/tmp/eb-vlSJIi/ccsLO4XL.o:linktest.c:function main: error: undefined reference to 'LAPACKE_dlatms_work'
/tmp/eb-vlSJIi/ccsLO4XL.o:linktest.c:function main: error: undefined reference to 'LAPACKE_slatms'
/tmp/eb-vlSJIi/ccsLO4XL.o:linktest.c:function main: error: undefined reference to 'LAPACKE_slatms_work'
/tmp/eb-vlSJIi/ccsLO4XL.o:linktest.c:function main: error: undefined reference to 'LAPACKE_zlatms'
/tmp/eb-vlSJIi/ccsLO4XL.o:linktest.c:function main: error: undefined reference to 'LAPACKE_zlatms_work'
/tmp/eb-vlSJIi/ccsLO4XL.o:linktest.c:function main: error: undefined reference to 'LAPACKE_clagge'
/tmp/eb-vlSJIi/ccsLO4XL.o:linktest.c:function main: error: undefined reference to 'LAPACKE_clagge_work'
/tmp/eb-vlSJIi/ccsLO4XL.o:linktest.c:function main: error: undefined reference to 'LAPACKE_dlagge'
/tmp/eb-vlSJIi/ccsLO4XL.o:linktest.c:function main: error: undefined reference to 'LAPACKE_dlagge_work'
/tmp/eb-vlSJIi/ccsLO4XL.o:linktest.c:function main: error: undefined reference to 'LAPACKE_slagge'
/tmp/eb-vlSJIi/ccsLO4XL.o:linktest.c:function main: error: undefined reference to 'LAPACKE_slagge_work'
/tmp/eb-vlSJIi/ccsLO4XL.o:linktest.c:function main: error: undefined reference to 'LAPACKE_zlagge'
/tmp/eb-vlSJIi/ccsLO4XL.o:linktest.c:function main: error: undefined reference to 'LAPACKE_zlagge_work'
/tmp/eb-vlSJIi/ccsLO4XL.o:linktest.c:function main: error: undefined reference to 'LAPACKE_claghe'
/tmp/eb-vlSJIi/ccsLO4XL.o:linktest.c:function main: error: undefined reference to 'LAPACKE_claghe_work'
/tmp/eb-vlSJIi/ccsLO4XL.o:linktest.c:function main: error: undefined reference to 'LAPACKE_zlaghe'
/tmp/eb-vlSJIi/ccsLO4XL.o:linktest.c:function main: error: undefined reference to 'LAPACKE_zlaghe_work'
/tmp/eb-vlSJIi/ccsLO4XL.o:linktest.c:function main: error: undefined reference to 'LAPACKE_clagsy'
/tmp/eb-vlSJIi/ccsLO4XL.o:linktest.c:function main: error: undefined reference to 'LAPACKE_clagsy_work'
/tmp/eb-vlSJIi/ccsLO4XL.o:linktest.c:function main: error: undefined reference to 'LAPACKE_dlagsy'
/tmp/eb-vlSJIi/ccsLO4XL.o:linktest.c:function main: error: undefined reference to 'LAPACKE_dlagsy_work'
/tmp/eb-vlSJIi/ccsLO4XL.o:linktest.c:function main: error: undefined reference to 'LAPACKE_slagsy'
/tmp/eb-vlSJIi/ccsLO4XL.o:linktest.c:function main: error: undefined reference to 'LAPACKE_slagsy_work'
/tmp/eb-vlSJIi/ccsLO4XL.o:linktest.c:function main: error: undefined reference to 'LAPACKE_zlagsy'
/tmp/eb-vlSJIi/ccsLO4XL.o:linktest.c:function main: error: undefined reference to 'LAPACKE_zlagsy_work'
collect2: error: ld returned 1 exit status
make[1]: *** [../libopenblas_haswellp-r0.2.20.so] Error 1
make[1]: Leaving directory `/tmp/OpenBLAS/0.2.20/GCC-6.4.0-2.28-LAPACK-3.7.1/OpenBLAS-0.2.20/exports'
make: *** [shared] Error 2

This surprised me a bit, since the lapack-netlib copy included in OpenBLAS 0.2.20 is basically LAPACK 3.7.0 + a couple of backported patches on top.
Are any other changes required in order to use an official LAPACK release as a drop-in replacement for the lapack-netlib directory as of recent?
On the side: are there any plans for updating the shipped lapack-netlib to LAPACK 3.7.1?
I'm willing to try and look into this myself, is there any documentation available on the procedure to follow for this?
I noticed that an update is required to lapack-netlib/LAPACKE/src/Makefile to avoid build issues with mingw (cfr. #1057, #1060), but I was hoping for a more complete outlined procedure that could help?

Maybe @wernsaar has some comments on this, cfr. the last update to LAPACK 3.7.0 + upstream patches in #1046?

Why would you expect a newer/different lapack to be a simple drop-in replacement, when the "new" version is very likely to come with new, deprecated or rearranged functions ? I believe the Makefile fixes you mentioned were applied upstream as well (but later superseded by a rewrite), what you are looking at now is the OpenBLAS-specific link test (in "exports) that gets run on the static library before its dynamic counterpart is built. The full build log will tell you if the missing lapacke functions were compiled at all, and perhaps also what if anything happened to them.
There are certainly plans to update to 3.7.1 at some point (or at least I have them...), but most of the fixes in that release have already been applied in the OpenBLAS copy and it is not entirely trivial to merge a new lapack (as you are seeing for yourself). Given the current lack of available developers
(wernsaar you are trying to contact has not been seen here since january) and the number of recent issues this is probably not the top priority.

Most likely you introduced a problem in lapack-netlib/LAPACKE/src/Makefile if/when you tried to reapply the mingw fixes - from looking at the file history in the netlib repository (apparently the netlib maintainers have not gotten around to providing a changelog for 3.7.1 yet) lapacke_clagge et al. have grown an additional interface for switching off NaN detection so depend on a new nancheck module now.
(I believe netlib nowadays relies predominantly on cmake, so maybe you actually found a bug in the netlib lapacke Makefile - I have not checked this)

@martin-frbg Using a LAPACK release as a drop-in replacement has worked fine before, so I don't think it's unreasonable to expect it would still work now (but maybe I'm being naive).
I literally only replaced lapack-netlib with an unpacked LAPACK 3.7.1 tarball, I didn't make any other changes. Since I'm building on Linux with GCC, I don't need to tweak the Makefile to split the list of objects in two different lists.
Thanks for the clarifications, I think we'll go ahead and just use the shipped LAPACK going forward.
I'm still interested in helping out with updating lapack-netlib to LAPACK 3.7.1, but I understand this is not top priority. Is there any write-up available on the procedure to follow for this, which (potential) changes need to be made both inside & outside of lapack-netlib?

I suspect your previous drop-in replacements worked only insofar as the subset of functions already present in the older LAPACK were still accessible, but none of the additional ones would have been
available in the shared libopenblas.so at least.
There is no writeup available as far as I know, your best bet is to look at what the relevant earlier commits touched. Basically you'd need to read the netlib changelog (if/when available, or their git history) to see what is new, check that it all builds with make and cmake after replacing the lapack-netlib tree and adapt the exports/gensymbol perl script as needed. This has the list of functions that should be present in the library, and linktest.c is generated by it.
Did you check if there were any earlier error messages in your build log, if you have a LAPACKE_clatms.o somewhere under lapack-netlib/LAPACKE in your unsuccessful build, and if libopenblas.a contains any symbols named LAPACKE_clatms etc. ?

A quick check shows several changes in the Makefiles distributed with 3.7.1. Apart from the addition of new functions and a splitting of the "make clean" target into "cleanobj" and "cleanlib" , the lapacke functions you were missing in your build have been made conditional on a LAPACKE_WITH_TMG that the OpenBLAS build system obviously does not yet know it has to define.
I will prepare a PR for updating to 3.7.1 later, possibly next week.

One can build openblas without builtin lapack, and then laapack/lapacke with '-lopenblas'

FWIW of the impressive list of 780 files that differ between the current OpenBLAS snapshot of lapack-netlib and 3.7.1, the overwhelming majority are changes to the function documentations in the comments at the start of each file (additions, typo fixes, version number or year updates). We may be missing a few recent LAPACKE fixes and enhancements that were committed to netlib since May.

PR #1366 has the majority of changes for updating to 3.8.0 (testers needed, also cmake updates not done yet)


Add more collaborators or transfer this repository's ownership

This repository fails on install for now and the original maintainers of this seems not able to resolve issues ATM. I recommend that new collaborators shall be added, or this repository's ownership to be transferred.

Ping. @ecordell

Hi @evanslify, sorry for the delay.
So here's a little backstory: I wrote this quickly a couple of years ago with the intention of using it, but never ended up doing that. It's under localmed's namespace, as I was working there at the time. During that time I've tried to merge PRs that claim to fix compatibility with newer protagonist versions, but I wasn't doing what a maintainer should do and testing those claims, so keeping up hasn't been very successful.
I'm no longer at LocalMed and I don't think they use this either. I'd be happy to transfer ownership personally but I'm not comfortable doing that on a project in localmed's namespace. Maybe @petebrowne could weigh in on whether or not he'd be willing to do that?
That said, there are lots of similar projects listed on [API Blueprint's website](https://apiblueprint.org/tools.html#mock servers) though I admit I've never tried them - is there something about this particular take on the idea that makes it worth maintaining over the other projects?
All this project does is parse the blueprint (using protagonist), and then walk through the AST to add routes to a standard express server. It would be easy to replicate or fork if that's desirable?

Hey @ecordell Thanks for your reply!
I don't have any particular idea, but since this project is listed on npm
and you have to install it to know it's broken due to outdated
dependencies, it would be great if we could do something to improve this
situation. As you mentioned, there are several similar projects listed on
apib's website, are as wonderful as your project (and I use some of them on
daily basis,) maybe we should turn this project into maintenance mode, or
at least edit README.md to indicate that this project is not under active
development?
Regardless of your choice, I could help out with a PR.
On Mon, Nov 14, 2016 at 10:29 PM, Evan Cordell<EMAIL_ADDRESS>wrote:

Hi @evanslify https://github.com/evanslify, sorry for the delay.
So here's a little backstory: I wrote this quickly a couple of years ago
with the intention of using it, but never ended up doing that. It's under
localmed's namespace, as I was working there at the time. During that time
I've tried to merge PRs that claim to fix compatibility with newer
protagonist versions, but I wasn't doing what a maintainer should do and
testing those claims, so keeping up hasn't been very successful.
I'm no longer at LocalMed and I don't think they use this either. I'd be
happy to transfer ownership personally but I'm not comfortable doing that
on a project in localmed's namespace. Maybe @petebrowne
https://github.com/petebrowne could weigh in on whether or not he'd be
willing to do that?
That said, there are lots of similar projects listed on API Blueprint's
website https://apiblueprint.org/tools.html#mock servers though I
admit I've never tried them - is there something about this particular take
on the idea that makes it worth maintaining over the other projects?
All this project does is parse the blueprint (using protagonist), and then
walk through the AST to add routes to a standard express server. It would
be easy to replicate or fork if that's desirable?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/localmed/api-mock/issues/55#issuecomment-260348581,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMeDEsPdMn-Z3SXJLF71JpLtG8GhYJnUks5q-HAsgaJpZM4Kene5
.

Yes, I would happily transfer ownership, or at the very least add some collaborators - we're not using the project anymore

@evanslify Are you interested in helping out?


[c++] Forward declare GenericWriteVariableUnsigned

Some users are reporting that they are receiving error "C3861:
'GenericWriteVariableUnsigned': identifier not found" when building.
I think this happens when custom streams that do not have their own
WriteVariableUnsigned member function.
Add a forward declaration of GenericWriteVariableUnsigned.
Fixes https://github.com/microsoft/bond/issues/1115

This is exactly what fixed this issue in my use case.
Thanks!


fix: Remove "Edit Follows" button from notification screen

Description
Removing the "Edit Follows" button from notification screen. I've accidentally added it again because I haven't seen this card from the QA session.

Context

PR Checklist

[ ] I have tested my changes on iOS and Android.
[ ] I hid my changes behind a feature flag, or they don't need one.
[ ] I have included screenshots or videos, or I have not changed the UI.
[ ] I have added tests, or my changes don't require any.
[ ] I added an app state migration, or my changes do not require one.
[ ] I have documented any follow-up work that this PR will require, or it does not require any.
[ ] I have added a changelog entry below, or my changes do not require one.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Cross-platform user-facing changes

iOS user-facing changes

Android user-facing changes

Dev changes

Need help with something? Have a look at our docs, or get in touch with us.

Warnings

:warning:

❌ An error occurred while validating your changelog, please make sure you provided a valid changelog.

  Generated by :no_entry_sign: dangerJS against 2459c0a29fab30f14bd5904c725906011075a594

Thank youuu


[discussion only] a bunch of api changes

A PR to help discuss API changes in #436 #427 #261 (maybe more). For now, this PR will only contain changes to docs/sphinx/source/api.rst.

ideas here have been implemented or rejected.


test project on Jupyter

Jupyter has 3.8 python version --> when doing a conda install with 3.11 version it's working fine!
see quickstart documentation


Document the accept property

This will fix #32

Looks like maybe the most recent version of the module loader isn't compatible with Ember 1.13?
At what point do we drop support?

@cowboyd what's interesting is this happened randomly. There has to be a dep that changed???

Yeah, seems that way. loader.js maybe?

Hm, I can't replicate this locally.

This is after a nombom


Use caching to speed up CICD

Issue #, if available:
Description of changes:
Creates a new ecr repo for caching and uses it during build process
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Hey seems like there's a bunch of misc changes in here that repeat work? Please double check and let's merge it into the files that already exist.

Some minor comments, please take a look.
Was it tested? Just trying to figure out if there may be any regression.
Will the warmer workflow be run at provisioning time and if yes, will it run docker build as part of it?

Yes, the warmer workflow runs as part of a workflow-of-workflows after the provisioner workflow.  The warmer workflow runs a full docker build to populate the cache in ECR(names the image cache-init).  I did test the full workflow successfully with dotnet(build down to 1.5 min), and tested the two workflows separately with rust(build down to 3 min).


Update using-latest-daily.md

Microsoft Reviewers: Open in CodeFlow

Thanks for taking care of this, @eerhardt. With my recent changes on the templates we can probably remove the tip section around conflicts as those won't be happening any more 😃

It would be great to make these docs changes in the same PRs that are changing the behavior/experience.


Sign backend JWT via Tenant Key Store Certificate

Describe your problem
With the new KM implementations in APIM v3.2.0, backend JWT is signed by only using default gateway certificate. No configuration to select tenant keystore certificates for JWT signing.
Describe your solution
Implement a configuration to enable the backend JWT is signed by the per tenant keystore certificate.

Needs to be frontported to master - in progress


ESP32-S2: enter uf2 bootloader from application

Is there a way to enter the uf2 bootloader by code?

Ok, found it: https://github.com/adafruit/tinyuf2/blob/8a54e0ed4c3373765903ca6d80eeb54dbcfc54d5/ports/esp32s2/README.md#usage


Anyway to load in WebVTT file directly?

I like what you've put together with this.
Not a real issue per say, but is there a way to load in a webvtt file and still get the vertical text you've added?

It's definitely possible. You can transform a WebVTT file to my caption data standard. I'll put that on my TODO list.


(maint) Removed support for Ubuntu 10.04

@ahenroid woot! Good cleanup.
Btw, two minor niceties wrt to commit messages. (We don't consistently do either of these things but good to know about them.)

Put the commit message in the imperative, e.g. (maint) Remove support for 10.04 rather than ... Removed ....  See the example in https://github.com/puppetlabs/puppet/blob/master/CONTRIBUTING.md
Use [skip ci] or [ci skip] in the body of the commit message to avoid travis and appveyor. We can do this for changes we know won't affect travis/appveyor output, e.g. changes under the acceptance directory.
** https://docs.travis-ci.com/user/customizing-the-build/#Skipping-a-build
** https://www.appveyor.com/docs/how-to/filtering-commits#skip-commits

CLA signed by all contributors.


[Bug] VPS filtering not always consistent -- VPS data may be inconsistent

Describe the bug
VPS filtering not always consistent -- VPS data may be inconsistent:  a game may be missing its "features" even though the tables have them
To Reproduce
Consider "Shooting the Rapids":

https://virtualpinballspreadsheet.github.io/games?game=5RaindsR7o

In VPin Studio this does not show up under the VR filter, but it does on the VPS website:

It looks like VPin Studio uses the features on the game itself for the first pass of filtering, but that is not always a union of the individual entries.  I noticed that there were a couple games where it showed under the VR feature but none of the tables had VR.  Shooting the Rapids is the opposite: the table has it but it doesn't show using the VR filter in VPin Studio.
Expected behavior
I am not sure if it is a bug or a feature request but I expect that the filtering would honor the union of the features on the tables (at the game level) as it does on the website.  I do think the VPS data is inconsistent, but maybe the game level feature data should be ignored and computed instead.

Yes, we missed to analysize the features of all version here too.

Fixed with 3.9.2


Extend the commons/protocol schema to enable linkage to sub-protocols

Include dcterms:hasPart property shape in commons/protocol schema to allow for sub-protocols to be linked

This issue is related to #69

Duplicate


Can't open a file with backslash in file name

Prerequisites

[x] Put an X between the brackets on this line if you have done all of the following:

Checked that your issue isn't already filed: https://github.com/icetee/remote-ftp/issues

Description
Can't open a file with backslash in file name.
Steps to Reproduce

Right click on a folder, select "Add File", type "test".
Right click on the file, select "rename", change "test" to "\test".
Open the file.

Expected behavior: Ability to open files with backslashes in file names.
Actual behavior: Can't open a file with backslash in file name.
Versions
Atom 1.20.1
Remote-FTP 1.1.3
Additional Information

Another special symbols can also reproduce the issue.
Changing "test" to "te\st" produces "st" filename in Remote-FTP tree.
Also can't rename and delete the file.
It's able to open, rename or delete the file in Project tree.

Why does the backslash in the filename? :)
Backslash is represents folder in Window and replace in code.
https://github.com/icetee/remote-ftp/blob/3075b21f00386893181559842c7522bda5cd0365/lib/directory.js#L217-L231
https://github.com/icetee/remote-ftp/blob/3075b21f00386893181559842c7522bda5cd0365/lib/file.js#L58-L72

I supported. I think do not use backslash in file/folder name.

I added backslash to the file accidentally and decided report the issue. I don't think the backslash in filename is a good idea

I thought and corrected it. Next version will include. Thank you very much for the report!


bot permissions

make sure that the bot is able to ban and kick people using commands

Added both kick and ban commands


Add data-hover on Select.Item

Is your feature request related to a problem? Please describe.
Mentioned in https://github.com/fabien-ml/kobalte/issues/57

fixed by #60

fixed in v0.4.0.


Problems with getting started: WinError 2

Hy nuitka people,
I'm a newby to nuitka and i would really like to try it, but i'm running into some troubles. If i try to compile a simple HelloWorld.py file i always end up with the following error:
C:\Python34\Scripts>nuitka --module HelloWorld.py
Traceback (most recent call last):
File "C:\Python34\Scripts\nuitka", line 195, in
MainControl.main()
File "C:\Python34\Lib\site-packages\nuitka\MainControl.py", line 737, in main
main_module = main_module
File "C:\Python34\Lib\site-packages\nuitka\MainControl.py", line 654, in compi
leTree
quiet       = not Options.isShowScons()
File "C:\Python34\Lib\site-packages\nuitka\MainControl.py", line 524, in runSc
ons
return SconsInterface.runScons(options, quiet), options
File "C:\Python34\Lib\site-packages\nuitka\build\SconsInterface.py", line 215,
in runScons
return subprocess.call(scons_command, shell = False) == 0
File "C:\Python34\lib\subprocess.py", line 535, in call
with Popen(*popenargs, **kwargs) as p:
File "C:\Python34\lib\subprocess.py", line 848, in init
restore_signals, start_new_session)
File "C:\Python34\lib\subprocess.py", line 1104, in _execute_child
startupinfo)
FileNotFoundError: [WinError 2] Das System kann die angegebene Datei nicht finde
n
Here is what i did to install nuitka: I downloaded the nuitka installation .msi file for windows x64 and python 3.4. I started the installation and added the path to the python 3.4 installation. Corresponding to that I use python 3.4 for testing nuitka (i also use a python 3.6 version with anaconda, altough i have a python 2.7 installation). I also installed Visual Studio 2015 professional. The same error occurs if i try to test this in python 3.6.
I found a similar post to this problem here http://bugs.nuitka.net/msg1381 but it seems that this question was not answered, at least i cant find it.
Furthermore i would like to obtain a corresponding c++ file out of my .py file. What is the best way to obtain this in nuitka? I'm not sure about this...
Would be nice if someone could help me! :)

Please read the user manual. You need to have a Python2 installation too.


[stable/owncloud] Add javsalgar to the list of OWNERS

Signed-off-by: Javier Salmeron Garcia<EMAIL_ADDRESS>
What this PR does / why we need it:
Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

fixes #

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[ ] DCO signed
[ ] Chart Version bumped
[ ] Variables are documented in the README.md
[ ] Title of the PR starts with chart name (e.g. [stable/chart])

/ok-to-test
/lgtm


Added first attempt of repr for TensorMap and TensorBlock

In the spirit of issue #32. I added an idea for two __repr__ functions for TensorMap and TensorBlock.
In TensorMap it basically prints the keys and its names:
>> Tensor
TensorMap
keys: [(0, 1, 1), (0, 1, 6), (0, 1, 8), (0, 6, 1), (0, 6, 6), (0, 6, 8), (0, 8, 1), (0, 8, 6), (0, 8, 8)]
key_names: ['spherical_harmonics_l', 'species_center', 'species_neighbor']

For the TensorBock, the function prints the names of the samples, components and properties:
>> Tensor.block(0)
TensorBlock
samples: ['structure', 'center']
component: ['spherical_harmonics_m']
properties: ['n']

The example are taken from a spherical harmonic expansion Tensor.

The build failure is unrelated, and should be fixed on master. Can you rebase this branch to bring in the fix?
Some small suggestions for the output
TensorMap with 9 blocks  # add the number of blocks
# how about a vertical layout? takes more space but might be easier to read
keys: ['spherical_harmonics_l', 'species_center', 'species_neighbor']
               0,                        1,               1,
               0,                        1,               6,

                                        ... # skip some keys if they are more than 10?

               0,                        8,               6,
               0,                        8,               8,

If you agree with this, maybe we can skip keys only in __repr__ and not in __print__ (similarly to how numpy prints arrays)
For the tensor block, I would also add some information about the gradients

I agree for sure with the addition of the number of blocks.
the vertical layout how you would suggest aligning the numbers with just tabs?
I mean if the keys_names changes it would be difficult are different to put the number under the label.
Regarding the skip, I agree to put it only in repr (actually that's why I haven't put it, normally I do not like when it skips stuff.
I can complete the TensorBlock with gradients. Here do we put full infos or just if there are gradients or not? and then a repr (or similar) function for the gradient only?

the vertical layout how you would suggest aligning the numbers with just tabs?
I mean if the keys_names changes it would be difficult are different to put the number under the label.

There is some code doing this kind of alignment in rust, here: https://github.com/lab-cosmo/equistore/blob/8823476d864409618afb23c96dc86affc767b160/src/labels.rs#L229-L244
It is not too bad: you can use the length of the different variables as padding when writing the values. I think python formatting can take the padding width as an extra parameter (not sure what's the syntax for it though).

I can complete the TensorBlock with gradients. Here do we put full infos or just if there are gradients or not? and then a repr (or similar) function for the gradient only?

I think just having which gradients are present or not is fine, and then having another __repr__ function in the Gradient proxy object is a great idea!

The TensorMap prints the number of blocks and the values in a table-like way.
The padding for sure can be improved (also the code is not super good-looking).
If the number of blocks is < 10, both functions do not skip any value.
>> Tensor
TensorMap with 9 blocks
keys: ['spherical_harmonics_l' 'species_center' 'species_neighbor]
                  0                   1                1
                  0                   1                6
                  0                   1                8
                  0                   6                1
                  0                   6                6
                  0                   6                8
                  0                   8                1
                  0                   8                6
                  0                   8                8

>> print(Tensor)
TensorMap with 9 blocks
keys: ['spherical_harmonics_l' 'species_center' 'species_neighbor]
                  0                   1                1
                  0                   1                6
                  0                   1                8
                  0                   6                1
                  0                   6                6
                  0                   6                8
                  0                   8                1
                  0                   8                6
                  0                   8                8

If the number of blocks is > 10.
>> Tensor
TensorMap with 63 blocks
keys: ['spherical_harmonics_l' 'species_center' 'species_neighbor]
                  0                   1                1
                  1                   1                1
                  2                   1                1
               ...
                  4                   8                8
                  5                   8                8
                  6                   8                8

print function never skip.
The TensorBlock output now gives info on the gradients
If there are no gradients:
TensorBlock
samples: ['structure', 'center']
component: ['spherical_harmonics_m']
properties: ['n']
gradients: No

If there are gradients, it gives the list of parameters wrt the gradients are computed:
>> Tensor.block(0)
TensorBlock
samples: ['structure', 'center']
component: ['spherical_harmonics_m']
properties: ['n']
gradients: ['positions']

Finally the gradients block:
>> Tensor.block(0).gradient('positions')
Gradient TensorBlock
parameter: 'positions'
samples: ['sample', 'structure', 'atom']
component: ['gradient_direction', 'spherical_harmonics_m']
properties: ['n']

I like the new output!
The build will fail again, you'll need to reformat the code with black. You can do this manually with tox -e format or integrate it automatically in your editor with https://github.com/microsoft/vscode-black-formatter (I know you are using vscode). The automatic integration will automatically format all python files when you save them.


Automatically send digital goods when payment received

Right now a vendor selling digital goods receives payment and then manually enters a link and (optionally) a password when they confirm they've received payment.
Ideally we could automate the sending of the link and password as soon as payment is received (though probably after one or two confirmations). The vendor would obviously need to put in the link and password at the time of the listing.
@morebrownies might have ideas on how best to implement this.
I'll create a duplicate issue on the server side since it would require changes there as well.

As you're going to integrate ipfs anyway, the file could be just added to ipfs.
The hash would be revealed only when the payment verifies. This wouldn't even require a password.

This feature is planned for v2.


[FR] add unbound-checkzone to check syntax of response policy zones (RPZ)

first of all thanks to @gthess for his help to find very useful link (see here - closed)
I was having some trouble with converting an IPv6 address into a valid rpz entry, but succeeded
hapy to share the bash conversions, I've been using:
convert IPv4 address:
entry=$(echo "${addressArray[$j]}" | awk -F. '{OFS="."; print "32",$4,$3,$2,$1,"rpz-ip CNAME ."}')

result (example):
<IP_ADDRESS>.104.rpz-ip CNAME .

convert IPv6 address:
addr=$(echo "${addressArray[$j]}" | sed 's/::/:zz:/')
# need to remove trailing : if address ends with :zz: thus ${addr%:}
entry="128."$(echo "${addr%:}" | awk -F: '{for (i=NF; i>1; i--) printf("%s.",$i); print $1; }')".rpz-ip CNAME ."

result (example):
128.f8f9.6810.zz.4700.2606.rpz-ip CNAME .

The question: (FR):
after looking around (duckduckgo), I was made aware of several programs, able to check zonefiles.
this includes NSD, another great NLnetLabs product!
I'm lucky to have NSD installed, thus can use /usr/local/sbin/nsd-checkzone rpz file, but assume not all unbound users also run NSD.
Would it be possible to make unbound-checkzone available, this to check rpz (and other zone files), since zone files are used by several unbound users.
Thank you for your time and effort.

I guess that one can use ldns-read-zone, in ldnsutils, from nlnetlabs, that doesn't run any additional stuff.


can I use a  with child components to render the header ?

As per the animated gif here - how can I render a <View> as the header, so that i can (for example) use a background Image, use color gradient (<LinearGradient>) ? Pretty urgent so appreciate quick feedback. Thanks!

If there is a way, I would like to know too. It is possible to add a custom component below the title?

+1

Anyone?

+1

does anyone know if this is possible yet ?

+1 for Gradient

+1 for gradient
the react-native-router-flux can easily set a background image in nav bar but still cant fide a workaround here

+1 linear gradient

+1

+1

You can accomplish this by creating your own header component. The existing header is a good starting point.


fix: apply margins in nvd3

🐛 Bug Fix
The NVD3 visualization does a bunch of operations on the margin object, depending on the visualization parameters. For example, if we're showing the labels on bars we need to increase the top margin by 24px so that it doesn't overlap with the legend:
      if (showBarValue) {
        // Add more margin to avoid label colliding with legend.
        margins.top += 24;
      }

The problem is that the margins object is never assigned to the chart. This PR fixes it.
Before:

After:

This PR fixes https://github.com/apache/incubator-superset/issues/8480.

Once you rebase, the build should be fixed

also, it looks like a bunch of extra formatting changes were applied here. could you remove those? (maybe you're on an old version of prettier?)


Front-Matter

Consider Implementation of credits, colophon, or whatever.

This is an interesting one. How many separate things to do you think are necessary? Should they live as .md files in the root?

The way it is now seems okay... There is a credits.md file by default. Whatever shows up there ends up on page 2 of the book ... I'll likely make a template just for that page with 2 columns and small text. Having a dynamic indexes in the back as we planned is nice (URLs). I could see adding a feature that is an index of all proper nouns (all capitalized words and what pages they are on). We could add a closing version of credits, but no need to do so right now.


Update .md grammar and comments

Change strings to spanish, as well as minor grammar corrections

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Thanks!


build: Add -fdebug-types-section

While this may slow builds down a little and result in slightly larger debug builds, it should reduce the size of production bins.
Adding here to unblock #30394 which has a related issue.

Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

this didnt resolve the issue that i was trying to address - may be desirable anyway - so ill leave open other PR


fix(deps): update module github.com/jmoiron/sqlx to v1.3.5

This PR contains the following updates:

Package
Type
Update
Change

github.com/jmoiron/sqlx
require
patch
v1.3.4 -> v1.3.5

Release Notes

jmoiron/sqlx (github.com/jmoiron/sqlx)
v1.3.5
Compare Source

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

⚠ Artifact update problem
Renovate failed to update an artifact related to this branch. You probably do not want to merge this PR as-is.
♻ Renovate will retry this branch, including artifacts, only when one of the following happens:

any of the package files in this branch needs updating, or
the branch becomes conflicted, or
you click the rebase/retry checkbox if found above, or
you rename this PR's title to start with "rebase!" to trigger it manually

The artifact failure details are included below:
File name: go.sum
Command failed: go get -d -t ./...
go<EMAIL_ADDRESS>reading github.com/tetsumaru/go-private-sandbox/go.mod at revision v0.0.2: git ls-remote -q origin in /go/pkg/mod/cache/vcs/86de18e818d2ff3f863245d1730caa64352667da1c19832c71c23235b2a97863: exit status 128:
	fatal: could not read Password for<EMAIL_ADDRESS>terminal prompts disabled


Fix/instrumentations

coverage: 67.871% (+0.2%) from 67.716%
when pulling a8721a432879166a7dcb2e75cc7684b13be58d32 on fix/instrumentations
into c2e00d130cfa6294105415d8c5b03c4dcc7da988 on main.


Small fixes to CppInteroperabilityManifesto.md

Fix link to C interop doc
Fix the type for CxxConstPointer<T> (this should be an UnsafePointer, not UnsafeMutablePointer, right?)

@gribozavr Can you review?

@swift-ci Please smoke test and merge


Create a Debian package and submit to mentors.debian.net

pingo is not available on Debian yet and the first move to have it on official repositories is to create a package. Then, we need to find a Debian Developer to sponsor the packaging of this software and sign the package before uploading it to Debian Mentors Server so it can be approved (or not) by the FTP-Masters team and then enter in the official repositories.
@kretcheu, can you please do the packaging stuff? :) I can help with Python-related stuff.

Hi, I began a Debian process of packaging, creating a ITP bug:
http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=800393

Hi, working on packaging to Debian, I saw that it's interesting thinks to do.

changelog is recommended.
signing tarball it's good practice.
pingo/examples/dual7seg.py and pingo/examples/dualboard.py are identical.


HTML to PDF - Font become bold in PDF

I am just converting HTML to PDF using HTML Renderer. However i am getting bold letters when we generate PDF in Demo Application. However it works correctly browser though Show Browser button or Open External button.
I am trying this only in Demo Application not any specific written code by myself.
Left Side of Screenshot is PDF and Right Side is Browser.
I have attached HTML Code here
HTML.txt
This is my screenshot

Please help me to identify root cause of the issue.

Hi! Have you resolved?
I have the same problem, bold is not show in any way, I tried with b or strong tag and style font-weight: bold.
Nothing!

@itkmanoj -  please remove in style "font-weight: 400;" and you won't get bold letters.


Issue with merge conflict resolution

Gitlens version: 10.2.1
Vscode version: 1.45.1
Issue: 30s + Lag for "Accept Current Change" | "Accept Incoming Change" | ... menu to appear
When it does appear, when I select "Accept Incoming Change" instead of it adjusting the code in the diff section showing the incoming/current change, it instead modifies other code in my script leaving the conflict in place.
Not clear on what has changed. I've been using VsCode/GitLens for 6months+ without issue.
Looking at the errors in dev console I see:
(node:66468) [DEP0005] DeprecationWarning: Buffer() is deprecated due to security and usability issues. Please use the Buffer.alloc(), Buffer.allocUnsafe(), or Buffer.from() methods instead.
Also:
Error Python Extension: 2020-06-08 16:34:00: Failed to check if file needs to be fixed [EntryNotFound (FileSystemError): Unable to read file

GitLens doesn't provide the merge conflict code lens -- those are provided by VS Code itself.


Add coolhigh canbus code and its config file

add coolhigh canbus code and add /apollo/modules/calibration/data/ch

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

Please sign the CLA and remove those configs/launch files first.


fix: update shopware to 6.5, quickstart and defaults to 6.5+composer

The Issue

Our shopware quickstart didn't work with Shopware 6.5, as described in discord conversation
Shopware 6.5 is probably better installed with composer, as done here.
In https://github.com/ddev/ddev/issues/4701 we're trying to use cleaner repos for testing, so created ddev/test-shopware6

Manual Testing Instructions
Use these artifacts to go through the updated Shopware 6 quickstart. https://github.com/ddev/ddev/blob/82560d25598f4c2721501428d1ca71749cae99c2/docs/content/users/quickstart.md#shopware
Automated Testing Overview
TestDdevFullSiteSetup will now use new artifacts and Shopware 6 code.
Related Issue Link(s)
Release/Deployment Notes

@rfay @mattstein The recommend way of installing Shopware now is via symfony flex (i.e. composer create-project ...) Do you think it is worth while to update the quick config accordingly? Happy to do so, in case.

@vanWittlaer Please take a look at this PR. That's one of the fundamental things it does. Your feedback is very welcome. You can read the proposed quickstart at https://github.com/ddev/ddev/blob/7b65f91683737d9e943469157ca2460fde6284d9/docs/content/users/quickstart.md#shopware - would love your feedback.

@rfay Thx for your speedy response! In fact I'd have 2 issues re. the quick start, both related to symfony flex:

I think it's "cleaner" to install Shopware in its own subfolder (same level as .ddev)
The logic of how the .env files are used for different environments, and which of these are commited to the repo - in no case should we change or remove the .env file generated and maintained by symfony flex, instead generate a .env.dev and .env.local

A very basic script that does it all is here https://gist.github.com/vanWittlaer/def64280969ff3e8d2934ac03b206828 - what would it need to better integrate it with ddev? I presume it is very close to what you have documented in the quick start anyhow.

I'd appreciate it if you'd discuss more:

The logic of how the .env files are used for different environments, and which of these are commited to the repo - in no case should we change or remove the .env file generated and maintained by symfony flex, instead generate a .env.dev and .env.local

I thought it was always inappropriate to commit a .env to any project...
Unfortunately shopware refuses to do these standard steps if a .env file already exists, which is why the .env file created by composer has to be removed. I'm surprised at that and will be interested in your feedback.
ddev exec console system:setup --database-url=mysql://db:db@db:3306/db --app-url='${DDEV_PRIMARY_URL}' --mailer-url='smtp://<IP_ADDRESS>:1025?encryption=&auth_mode='
ddev exec console system:install --create-database --basic-setup

I guess you'd rather have to blame Symfony for the policy change re. .env files (as far as I understand). See
https://symfony.com/doc/current/configuration.html#overriding-environment-values-via-env-local.
From my experience, with only a couple of projects so far, the new policy however makes perfect sense.
I appears to me that Shopware has just not yet updated their system:setup command accordingly. That's why I create the .env.dev and .env.local manually. As far as I see system:setup does nothing else.

Resources:

Official composer instructions

@stasadev I still have test troubles with this but if you could give it another run-through it would be appreciated, thanks.

Just my quick thoughts:

I'd strongly recommend to setup Shopware in a subfolder
ES is not required/needed
echo "DATABASE_URL=mysql://db:db@db:3306/db" >>.env  is not necessary, should be in the .env.local already
.env.local should/needs only contain 4 vars: APP_ENV, APP_URL, DATABASE_URL and MAILER_DSN

Thanks so much for following this and contributing your valued expertise @vanWittlaer !

I'd strongly recommend to setup Shopware in a subfolder

I did understand that you recommended that. However, you gave nothing other than personal preference or practice to support your opinion, and all other quickstarts install in the root of the project by default. Remember that with a quickstart we aren't necessarily promoting best practices, we're trying to get people started.

ES is not required/needed

I see that from https://docs.shopware.com/en/shopware-6-en/first-steps/system-requirements so will remove that step.

echo "DATABASE_URL=mysql://db:db@db:3306/db" >>.env is not necessary, should be in the .env.local already

I would love it if it weren't necessary, but please do try it out. Without that step (and with DATABASE_URL in .env.local) you get
$ ddev exec console system:install --basic-setup

In DatabaseConnectionInformation.php line 54:

  [Shopware\Core\Maintenance\System\Exception\DatabaseSetupException]
  Environment variable 'DATABASE_URL' does not contain a valid dsn.

If you'll please try it and suggest a better technique it will be welcome. Perhaps ddev exec DATABASE_URL=mysql://db:db@db:3306/db console system:install --basic-setup would work.

.env.local should/needs only contain 4 vars: APP_ENV, APP_URL, DATABASE_URL and MAILER_DSN

I wasn't able to get it to run at all without APP_SECRET. The install runs through, but then when you launch the site you get

errors":[{"code":"0","status":"500","title":"Internal Server Error","detail":"Environment variable not found: \u0022APP_SECRET

Anyway, if you could please test this and suggest other things that would be great.
If you leave out LOCK_DSN you get this at install time:

In EnvVarProcessor.php line 193:

[Symfony\Component\DependencyInjection\Exception\EnvNotFoundException]
Environment variable not found: "LOCK_DSN".

If you could please go through the quickstart manually it would be very much appreciated.

Thanks, DDEV is super easy. You need docker if you don't have it, https://ddev.readthedocs.io/en/latest/users/install/docker-installation/
And you install DDEV on any platform, https://ddev.readthedocs.io/en/latest/users/install/ddev-installation/
I didn't realize you weren't already a DDEV user?

Thx, I am great fan and a heavy user of ddev in fact :-) just missed testing-a-pr doc, thx again! Already on it.

@rfay ok just tested the pr. I omitted the ES install and the echo "DATABASE ..
The .env.local generated is broken. The .env.local should look like this (proved to work):
APP_ENV=dev
APP_URL="https://my-shopware6.ddev.site"
DATABASE_URL="mysql://db:db@db:3306/db"
MAILER_DSN="smtp://<IP_ADDRESS>:1025?encryption=&auth_mode="

The LOCK_DSN and APP_SECRET env vars are in the .env already and need not to be replicated.
Hope this is of help.

Perhaps not generated by ddev itself - it comes from Symfony/Shopware via composer. Weird if you don't have it? I just followed the quickstart instruction as set forth in the PR. Starting from a new, blank folder.

To summarize:
I use https://ddev--5489.org.readthedocs.build/en/5489/users/quickstart/#shopware
And after ddev composer create shopware/production:^v6.5 there is two files:
.env:
###> symfony/messenger ###
# Choose one of the transports below
# MESSENGER_TRANSPORT_DSN=amqp://guest:guest@localhost:5672/%2f/messages
# MESSENGER_TRANSPORT_DSN=redis://localhost:6379/messages
# doctrine://default?auto_setup=0
###< symfony/messenger ###

###> symfony/mailer ###
# MAILER_DSN=null://null
###< symfony/mailer ###

###> symfony/lock ###
# Choose one of the stores below
# postgresql+advisory://db_user:db_password@localhost/db_name
LOCK_DSN=flock
###< symfony/lock ###

###> shopware/core ###
APP_ENV=prod
APP_URL=http://<IP_ADDRESS>:8000
APP_SECRET=143a978bd1099c65da67b64be2d54bf3
INSTANCE_ID=891b33cb29b18ba886d641ec8a3ee4bc
BLUE_GREEN_DEPLOYMENT=0
DATABASE_URL=mysql://root:root@localhost/shopware
# With Shopware <IP_ADDRESS> the MAILER_DSN variable will be used in this template instead of MAILER_URL
MAILER_URL=null://null
###< shopware/core ###

###> shopware/elasticsearch ###
OPENSEARCH_URL=http://localhost:9200
SHOPWARE_ES_ENABLED=0
SHOPWARE_ES_INDEXING_ENABLED=0
SHOPWARE_ES_INDEX_PREFIX=sw
SHOPWARE_ES_THROW_EXCEPTION=1
###< shopware/elasticsearch ###

###> shopware/storefront ###
STOREFRONT_PROXY_URL=http://localhost
SHOPWARE_HTTP_CACHE_ENABLED=1
SHOPWARE_HTTP_DEFAULT_TTL=7200
###< shopware/storefront ###

And .env.local:
DATABASE_URL="mysql://db:db@db:3306/db"
LOCK_DSN="flock"
MAILER_DSN="smtp://<IP_ADDRESS>:1025?encryption=&auth_mode="
APP_SECRET=""
APP_URL="https://my-shopware6.ddev.site"

As @vanWittlaer said, some values from .env.local are already defined in .env, and a minimal .env.local should look like https://github.com/ddev/ddev/pull/5489#issuecomment-1824667507
APP_ENV=dev
APP_URL="https://my-shopware6.ddev.site"
DATABASE_URL="mysql://db:db@db:3306/db"
MAILER_DSN="smtp://<IP_ADDRESS>:1025?encryption=&auth_mode="

I tried to run ddev exec console system:install --basic-setup with ⬆ and everything worked.

When I run it I get no .env created using the current strategy. I must be starting over with a ddev composer install and it doesn't get generated that way.  Will go back to it again and do the whole thing. I wish I knew what part of the process is generating the .env so I didn't have to go through the entire process.

I wish I knew what part of the process is generating the .env

ddev exec "rm -f .env symfony.lock && composer require symfony/flex"

I sure appreciate you both hanging in there with me and coaching me! Not sure how I got to where .env wasn't being created, but here's the current situation, and hopefully tests will pass:

Artifacts: https://github.com/ddev/ddev/pull/5489#issuecomment-1789806464
Quickstart: https://ddev--5489.org.readthedocs.build/en/5489/users/quickstart/#shopware

I'd sure appreciate you taking another run-through.

one minor observation. at one step of the composer create process there is the prompt asking if i want to include the docker configuration from recipes. the default is yes. but it might be a good thing to add a single comment to recipe perhaps about that? as someone not familiar with shopware it made me uncertain if i should follow the default recommendation when i tested? or might it cause problems with ddevs docker setup?

Thanks for testing! The quickstart does say "# If it asks Do you want to include Docker configuration from recipes? answer x, as we're setting it up."
How would you improve that?

i think i should better go to bed now. i completely missed that line. :/ apologies.

I'd sure appreciate you taking another run-through.

Just tested your latest changes - all ok!

Thanks so much! We may try to get this into next release, since it should only affect shopware anyway.


E5560: nvim_get_option must not be called in a lua loop callback

I've hit this after the latest updates, while using nvim-dap (never happened before):
Error executing luv callback:
.../site/pack/packer/start/impatient.nvim/lua/impatient.lua:138: E5560: nvim_get_option must not be called in a lua loop callback

nvim 0.5

Thanks. Just fixed this but note that require in a fast callback currently has a known issue in Neovim so nvim-dap should probably be fixed as well.

Working fine now, thx.


updating version details and ossp master for AppFlowy, Codeclimate, t…

tailwind-dashboard-windster#226
AppFlowy#224
codeclimate#228


Deprecation warnings with numpy 1.20

Numpy 1.20 has deprecated np.bool, np.int etc: https://numpy.org/devdocs/release/1.20.0-notes.html#deprecations
This means Cherab and raysect scripts using 2D observers now display the following warning:
DeprecationWarning: `np.bool` is a deprecated alias for the builtin `bool`. To silence this warning, use `bool` by itself. Doing this will not modify any behavior and is safe. If you specifically wanted the numpy scalar type, use `np.bool_` here.

It looks like the only place these deprecations exist are in sampler2d.pyx:
(cherab) jlovell@freia028 Cherab> grep -r "np\.bool[^0-9]" --include="*.py" --include="*.pyx" --include="*.pxd" raysect
raysect/raysect/optical/observer/sampler2d.pyx:            self._mask = value.astype(np.bool)
raysect/raysect/optical/observer/sampler2d.pyx:            self.mask = np.ones(pixels, dtype=np.bool)
raysect/raysect/optical/observer/sampler2d.pyx:            self._mask = value.astype(np.bool)
raysect/raysect/optical/observer/sampler2d.pyx:            self.mask = np.ones(pixels, dtype=np.bool)
raysect/raysect/optical/observer/sampler2d.pyx:            self.mask = np.ones(pixels, dtype=np.bool)
raysect/raysect/optical/observer/sampler2d.pyx:            self._mask = value.astype(np.bool)
raysect/raysect/optical/observer/sampler2d.pyx:            self.mask = np.ones(pixels, dtype=np.bool)
raysect/raysect/optical/observer/sampler2d.pyx:            self.mask = np.ones(pixels, dtype=np.bool)
raysect/raysect/optical/observer/sampler2d.pyx:            self._mask = value.astype(np.bool)
raysect/raysect/optical/observer/sampler2d.pyx:            self.mask = np.ones(pixels, dtype=np.bool)
raysect/raysect/optical/observer/sampler2d.pyx:            self.mask = np.ones(pixels, dtype=np.bool)

These instances should have np.bool replaced with bool.

Thanks, @jacklovell. It looks like the code that triggers these warnings was added by me. I'll fix it.


Emoji categories break after search

After performing a text search the emoji categories break down and do not operate properly (a lot of emoji categories become missing from the picker).
Steps to reproduce:

Click the emoji button to open the picker
Type a search term such as "apple"
Click anywhere outside the picker (the picker will close)
Open the picker again to see the missing emojis

The search text box will be cleared at this point and the picker can be restored to proper operation by typing some other text and then hitting backspace until all text is deleted.

Also, I have noticed a similar issue occurs when highlighting the text you type in the search box and pressing delete.

Similar issue to this is the anchors break too, seems to always happen if you search for something, click an anchor.. If you then remove all text and try click any anchor, it will not scroll to the right sections. And randomly happens after searching even if not clicking any of the anchors.

This might not be the "correct" fix as far as how the plugin is meant to work, but I found a fix by changing https://github.com/mervick/emojionearea/blob/99129f789f2e7619ce553c038783a13f47cbc846/dist/emojionearea.js#L1357 to:
categoryBlocks.filter('[data-tone="' + tones.find("i.active").data("skin") + '"]:not([name="recent"])').children().show();

Has this bug been fixed?

ref https://github.com/mervick/emojionearea/issues/341

@HanFox could you submit a pull request? The bug still exists but I believe your fix is correct

@mervick, why do you thumb-down my reply? You don't think it's the correct solution?


error text: sh: sshfs: command not found

When I mount , i got error text: sh: sshfs: command not found. Anyway to solve it?
thanks

Read the documentation, you need sshfs since this is just a GUI for the command line utility:
You need Mac OS X 10.9+ and at least one of the utilities listed below in order to use the sshfs gui:
sshfs from https://github.com/osxfuse/sshfs/releases / http://osxfuse.github.io/
MacFUSE from https://code.google.com/p/macfuse/
or a custom compiled sshfs from brew, macports, ..
Otherwise this GUI won't function.

Thanks. I will install these ultility.
BTW, can you help me to show how to implement sshfs in cocoa app? Thanks much

Same error:sh:     sshfs: command not found
I have sshfs and MacFUSE installed, on cmd:
$ which sshfs
/usr/local/bin/sshfs
Thanks in advance!

I get a similar issue:
sudo port install sshfs-gui
restart terminal
sshfs-gui
-bash: sshfs-gui: command not found

The problem is probably caused by path being in /usr/local/bin which is not included in the default PATH variable in Mac OS X. So GUI would probably be unable to locate sshfs in this directory. I would suggest adding frequent locations into PATH at application start in the source code by doing somehing like "putenv("PATH=" + getenv("PATH") + ":/usr/local/bin/:/opt/local/bin");"

@jleaders the shfs-gui is not available in port. You should try sudo port install sshfs instead. Or alternatively brew install sshfs if you use homebrew.
For graphical installations of sshfs you should check out the pages i recommended in my first comment:

sshfs from https://github.com/osxfuse/sshfs/releases / http://osxfuse.github.io/ MacFUSE from https://code.google.com/p/macfuse/

It is available in port for me
$ port search sshfs-gui

sshfs-gui @1.3 (fuse)
    OS X GUI for sshfs

Oh, thats seems to be another project.

This should be fixed now. Thanks to @YuriyNasretdinov for your path suggestion.


[TACHYON-814] Refactored more error messages

https://tachyon.atlassian.net/browse/TACHYON-814

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/Tachyon-Pull-Request-Builder/4566/
Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/Tachyon-Pull-Request-Builder/4593/
Test PASSed.

+1 LGTM

A small patch for this PR: #1490


[FEATURE REQUEST] Media tags (audio i.e.)

requested by @curezthree
ability to see the tags if they exists

we will use mutagen probably

from mutagen.mp3 import MP3
from mutagen.easyid3 import EasyID3
from mutagen.mp4 import MP4
from mutagen.flac import FLAC
# ... other necessary imports ...

def get_audio_metadata(file_path):
    file_ext = file_path.split('.')[-1].lower()
    audio_meta = {}

    try:
        if file_ext in ['mp3']:
            audio = MP3(file_path, ID3=EasyID3)
        elif file_ext in ['m4a', 'aac']:
            audio = MP4(file_path)
        elif file_ext in ['flac']:
            audio = FLAC(file_path)
        # ... other formats ...
        else:
            LOGGER.error(f"Unsupported file extension: {file_ext}")
            return None

        # Here you need to adjust how you access metadata based on file format
        audio_meta['duration'] = int(audio.info.length)  # This is usually the same across formats
        # For tags like artist or title, you might need to access them differently per format
        audio_meta['performer'] = audio.get('artist', [None])[0] if file_ext in ['mp3', 'm4a', 'aac'] else None
        audio_meta['title'] = audio.get('title', [None])[0] if file_ext in ['mp3', 'm4a', 'aac'] else None
    except Exception as e:
        LOGGER.error(f"Error extracting metadata: {e}")

    return audio_meta


[Task] ShenYu-admin all Synchronous buttons plus anti-shake function

Description

Synchronous buttons synchronize data to the gateway to avoid concurrent operations. Increase throttling function
Task List
No response

Please assign it for me, i will try it. thanks


Show Multi-file size

What feature or improvement do you think would benefit Files?
When you select multiple files and folders, it should calculate the storage size (It currently doesn't show within the properties menu)

Requirements
Not sure...
Files Version
<IP_ADDRESS>
Windows Version
10.0.23481.1000
Comments
No response

Thanks for the feedback. I am unable to reproduce this, Can you give the following information.

Were are the file/folders located (local drive, network, cloud storage etc.)

#12664 I encountered the same problem but when I put it in my favorites bar it does not crash


Fix issue 62 and 64, use patch instead of update

Fixes #62 #64

Proposed Changes

Use patch for updating subscriber statuses.
Add a webhook, default the proper duck shape so that the channel works.

Release Note

- 🐛 Fix bug Both the reconciler and dispatcher were updating the status. Change to use patch.
- 🐛 Fix bug Channel was not correctly specifying that it is a v1 / v1beta1 duck shape, and hence was not working correctly when used by things like Broker. Basically it was not conforming to the channel spec.
- 🐛 Fix bug Channel was not being validated because the webhook was not implemented.

Docs

/cherry-pick release-0.19

/cherry-pick release-0.19


TransformMol does not consider numpy.ndarray flags?

Hi there,
I'm using rdkit.Chem.AllChem.TransformMol from python. This takes a molecule and a numpy array. For various reasons, I'm getting a matrix (from other code) that needs transposing before it's passed to this function, but this generates "unexpected" results.
I think what's happening is that, for a simple array, calling ndarray.T just creates a view on the original with the C_CONTIGUOUS and F_CONTIGUOUS flags swapped. I think at some point between the python and C++ code, the underlying array gets read assuming it's a 4x4 C-ordered array.
The simplest demonstration is to take a e.g. a rotation around Z with a transform of 10 (M in the code below) - I would expect transforming by M and M.T to produce different final coordinates, but they don't.  (Aside: I think that the way rdkit parses the array, M.T does not actually represent the inverse of M, but that's not the problem).
Given some 3D input file (e.g. ATP from RCSB: https://files.rcsb.org/ligands/view/ATP_ideal.sdf)
from rdkit.Chem import MolFromMolFile
from rdkit.Chem.AllChem import TransformMol
import numpy as np
mol = MolFromMolFile("ATP_ideal.sdf")

M = np.array([[0., 1., 0., 10.], [-1., 0., 0., 10.0], [0., 0., 1., 10.0], [0., 0., 0., 1.,]])

print("Original position of first atom:")
print(mol.GetConformer().GetPositions()[0])

print("Transform by M:")
TransformMol(mol, M)
print(mol.GetConformer().GetPositions()[0])

print("Reload and transform by M.T - Would expect a different answer, even if this isn't actually a sensible thing to do")
mol = MolFromMolFile("ATP_ideal.sdf")
TransformMol(mol, M.T)
print(mol.GetConformer().GetPositions()[0])

print("array flags for M:")
print(M.flags)

print("array flags for M.T:")
print(M.T.flags)

My output looks like:
Original position of first atom:
[ 1.2   -0.226 -6.85 ]
Transform by M:
[9.774 8.8   3.15 ]
Reload and transform by M.T:
[9.774 8.8   3.15 ]
array flags for M:
  C_CONTIGUOUS : True
  F_CONTIGUOUS : False
  OWNDATA : True
  WRITEABLE : True
  ALIGNED : True
  WRITEBACKIFCOPY : False
  UPDATEIFCOPY : False

array flags for M.T:
  C_CONTIGUOUS : False
  F_CONTIGUOUS : True
  OWNDATA : False
  WRITEABLE : True
  ALIGNED : True
  WRITEBACKIFCOPY : False
  UPDATEIFCOPY : False

A workaround for now is to force C-ordering in the transposed matrix, there may be a neater way but this works:
M_transposed = np.zeros((4,4), dtype=float)        # C-ordered by default
np.copyto(M_transposed, M.T)                       # Still C-ordered

RDKit version: 2021.03.05  (  - rdkit=2021.03.5=py37h13c2175_0
OS: CentOS7
Python version (if relevant): 3.7.12
Are you using conda? Yes
If you are using conda, which channel did you install the rdkit from? conda-forge

@fredludlow
As you mention, the numpy documentation for ndarray.transpose() states that it only returns a different view on the underlying data, while the underlying data is unchanged.
RDKit's transConformer Python wrapper RDKit uses the underlying array data to compute the transformation; as this is unchanged, it will yield the same transformation.
Until this is is fixed in RDKit, a neater way to get the desired transformation is to force a copy of the array:
TransformMol(mol_copy, np.ndarray.copy(M.T))

Indeed, there are various ways to copy an array and the re-ordering behaviour differs:
The method on ndarray (as per your example or equivalently (M.T).copy()) defaults to C ordering, whereas np.copy(M.T), which I'd tried first, defaults to order='K', which means match original.
I got caught out as my immediate thought was "oh, this is a memory order thing" and then took me a lot longer to actually open the docs and read that np.copy preserves memory layout by default. 🤦♂