Keeping old Backend.main() shim for databricks mode

This patch address issue 1461. Latest sparklyr code refactoring caused two issues:

default value for master was removed
main class shim in Backend was removed.

This patch fixes both. For the first one I am copying @kevinykuo's fix here. For the second issue I am adding back the Shim.
For some context, Databricks was supposed to switch to switch to new main class last year. Unfortunately, my patch in our runtime got reverted and I did not realize. I am making sure it happens now, but for backwards compatibility it would be good to keep the shim for a bit longer.
cc @javierluraschi

Looks good! I can regenerate the jars for you. There is a configure.R script which bundles all the versions, livy sources and other related changes.


Remove asList() asSet(), as() from ConfigAccessor #106

Hello , I tried to solve this issue and during refactoring the tests, I did a suggestion a overload of method
ConfigAccessor access(String propertyName);
with
ConfigAccessor access(String propertyName,Class propertyType);
Please , verify if worth it.
Thanks.
Signed-off-by: Breno Pessoa<EMAIL_ADDRESS>

Hi @brenopessoa !
Thanks for your PR!
Those changes already got applied manually in the meantime.
Sorry for not looking at it earlier.


flow js checker - use 'flow' instead of 'flow check' for speed

I changed the command from "flow check" to "flow"
Right now, my editor freezes for 2-3 seconds every time I save, because that's how long it takes to boot up the flow type checker. Flow is designed to turn on a type checking server once, which keeps track of file changes on its own. You run flow and if the server isn't on, it turns it on. Then you just run flow again and it reports updates.
This PR removes "check", which makes subsequent type checks instant. The first time you save, it still freezes, but from then on it's great. I verified that it creates the server in the normal way (running flow in the terminal after saving doesn't start a second server).

Cool! If you don't want to make the user wait for the server to start, you can also do something like --timeout 2 and the server will exit with error code 3 if it takes more than 2 seconds.

That's great, except (1) somebody else suggested something similar a few days ago (see #1341), and (2) the patch is not needed:
let g:syntastic_javascript_flow_exe = 'flow'

(This is not to be confused with g:syntastic_javascript_flow_exec).


[enhancement] Added offline support. PWA FTW!

Now is PWA-enabled

Added two plugins:

gatsby-plugin-manifest (https://github.com/gatsbyjs/gatsby/tree/master/packages/gatsby-plugin-manifest)
gatsby-plugin-offline (https://github.com/gatsbyjs/gatsby/tree/master/packages/gatsby-plugin-offline)

Changed gatsby-config.js
@zainfathoni @resir014 please review
Thanks

added yarn.lock


App close clicking android back button at the end of stack

I want my app to be closed when I click back button at the end of stack.
But the BackHandler is implemented outside of stack.
How can I do that?

facing same issue


[Core] Update Diagnostics.md after upgrading System.Diagnostics.DiagnosticSource

Removing the section from Diagnostics.md about needing to add a reference to System.Diagnostics.DiagnosticSource. Since we have a high enough version to support tracing it's not needed anymore.
Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

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


add-to-app via CocoaPods cannot find flutter_assets on iOS 14

Run https://github.com/flutter/samples/tree/master/add_to_app/ios_fullscreen via the CocoaPods integration path.
Fails to launch the Flutter screen on iOS 14 / Xcode 12
Failed to find assets path for "Frameworks/App.framework/flutter_assets"
[VERBOSE-2:engine.cc(148)] Engine run configuration was invalid.
[VERBOSE-2:shell.cc(487)] Could not launch engine with configuration.
flutter: Observatory listening on http://<IP_ADDRESS>:63033/hwmX53Eqeq0=/

Interestingly, once I launch it once via iOS 13 / Xcode 11, then go back to iOS 14 / Xcode 12, it works.
cc @jmagman

Tried another one. This is pretty suspect.
../[flutter-module].ios/Flutter/flutter_export_environment.sh: line 8: inherited: command not found

Perhaps it didn't run xcode_backend.sh (the next line to flutter_export_environment.sh) at all?

Here is a fix, please merge: https://github.com/flutter/flutter/pull/63568
It's not related to iOS 14. You just need to buidl app once, then execute "pod install" and run again.
"inherited: command not found" is also a bug, but not related to this.

The inherited bug is the problem here.  I fixed up the App.framework path problem (which https://github.com/flutter/flutter/pull/63568 also does) and it still doesn't embed the App.framework into the resulting host app, though it does make the phantom /User/ directory go away.  The Flutter script doesn't run any flutter code, it just bails at flutter_export_environment.sh: line 8: inherited: command not found.  When I escape the export "OTHER_LDFLAGS=$(inherited) -framework Flutter" parenthesis, it embeds as expected.
Let's follow along at duplicate https://github.com/flutter/flutter/issues/63176.

I have found the same error trying to build in Xcode 12.2 on Apple M1.
The difference was that the error only happens when building from the native app, because I am building add-to-app.
What worked for me was to handle the VALID_ARCHS on my iOS Project Podfile as you can see in this CocoaPods issue
Fix all configuration
target 'Demo' do
  post_install do |installer|
    installer.pods_project.build_configurations.each do |configuration|
      installer.pods_project.build_settings(configuration.name)['VALID_ARCHS'] = []
    end
  end
end

Fix one configuration, like Debug
target 'Demo' do
  post_install do |installer|
    installer.pods_project.build_settings('Debug')['VALID_ARCHS'] = []
  end
end

I was able to get my add-to-app project building again from Xcode with the solution mentioned here https://github.com/flutter/flutter/issues/71959#issuecomment-755535401.


License clarification

Hello ! I see that in QVGE you have SVG components licensed under GNU GPL. Is it normal that QVGE is licensed under MIT ? Is it possible to use QVGE in order to edit graphs for our proprietary software ? Our goal is to avoid being obliged to license our software under an open source license.
Thank you very much for your help mucj appreciated !
Best regards
Esther

Hello Esther!
Hm, I'm not exactly sure which SVG components do you mean - I assume, the resources like icons?
They are taken from Inkscape project and are not licensed together with QVGE sources - this means you are free to use your own SVG files, whatever you like. I.e. you could fork QVGE and replace the SVG icons with your ones etc. There is no violation of GPL because the sources are under MIT.
And yes, you can use QVGE in your proprietary software in any way you want.
Maybe you could tell me later a little about your project and how do you suppose to use QVGE within it (but only if you want - I'm just curious how QVGE could still be useful for the community :) )
Kind regards
Ars

Dear Ars,
Thank you very much for your email, I’m going to let our Product Director (@Jonathan @.***>) answer to you regarding the project we may have, he will be more precise than I will!
Thanks also for your answer regarding the SVG components, I was indeed talking about the icons that are under GPL because of viral licensing. Yet after verification, there should be no issue of virality.
Kind regards,
@.***D77A4D.28F786E0]
De : Ars Masiuk @.>
Envoyé : vendredi 16 juillet 2021 00:15
À : ArsMasiuk/qvge @.>
Cc : Esther Renting @.>; Author @.>
Objet : Re: [ArsMasiuk/qvge] License clarification (#156)
Hello Esther!
Hm, I'm not exactly sure which SVG components do you mean - I assume, the resources like icons?
They are taken from Inkscape project and are not licensed together with QVGE sources - this means you are free to use your own SVG files, whatever you like. I.e. you could fork QVGE and replace the SVG icons with your ones etc. There is no violation of GPL because the sources are under MIT.
And yes, you can use QVGE in your proprietary software in any way you want.
Maybe you could tell me later a little about your project and how do you suppose to use QVGE within it (but only if you want - I'm just curious how QVGE could still be useful for the community :) )
Kind regards
Ars
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/ArsMasiuk/qvge/issues/156#issuecomment-881045670, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AUX3TAWTB7EZYNXTQCBCLLLTX5MVBANCNFSM5ANMLVRQ.
-------- Ce message et toutes les pièces jointes sont confidentiels et établis à l'intention exclusive de son ou ses destinataires. Si vous avez reçu ce message par erreur, merci d'en avertir immédiatement l'émetteur et de détruire le message. Toute modification, édition, utilisation ou diffusion non autorisée est interdite. L'émetteur décline toute responsabilité au titre de ce message s'il a été modifié, déformé, falsifié, infecté par un virus ou encore édité ou diffusé sans autorisation. -------- This message and any attachments are confidential and intended for the named addressee(s) only. If you have received this message in error, please notify immediately the sender, then delete the message. Any unauthorized modification, edition, use or dissemination is prohibited. The sender shall not be liable for this message if it has been modified, altered, falsified, infected by a virus or even edited or disseminated without authorization. --------

Dear Ars,
I’m the Product Manager in a startup in France called Airudit. We do vocal assistants to ease communications between humans and machines.
We will use QVGE as a tool to build graphs with a visual interface because editing graphs with a text editor in not convenient at all…
Then our product will parse graph files to deploy Natural Language Processing graphs.
We choose your tool because it is easy to use, cross platform, free, and it exports standard graph file formats 😊
Did you design the icons yourself with inkscape ?
The GPL license is only for inkscape or also for the icons ?
Kind regards,
Jonathan Girardeau
De : Esther Renting @.>
Envoyé : vendredi 16 juillet 2021 14:23
À : ArsMasiuk/qvge @.>; ArsMasiuk/qvge @.>; Jonathan Girardeau @.>
Cc : Author @.***>
Objet : RE: [ArsMasiuk/qvge] License clarification (#156)
Dear Ars,
Thank you very much for your email, I’m going to let our Product Director (@Jonathan @.***>) answer to you regarding the project we may have, he will be more precise than I will!
Thanks also for your answer regarding the SVG components, I was indeed talking about the icons that are under GPL because of viral licensing. Yet after verification, there should be no issue of virality.
Kind regards,
@.***D77C82.80466A10]
De : Ars Masiuk<EMAIL_ADDRESS>Envoyé : vendredi 16 juillet 2021 00:15
À : ArsMasiuk/qvge<EMAIL_ADDRESS>Cc : Esther Renting<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Objet : Re: [ArsMasiuk/qvge] License clarification (#156)
Hello Esther!
Hm, I'm not exactly sure which SVG components do you mean - I assume, the resources like icons?
They are taken from Inkscape project and are not licensed together with QVGE sources - this means you are free to use your own SVG files, whatever you like. I.e. you could fork QVGE and replace the SVG icons with your ones etc. There is no violation of GPL because the sources are under MIT.
And yes, you can use QVGE in your proprietary software in any way you want.
Maybe you could tell me later a little about your project and how do you suppose to use QVGE within it (but only if you want - I'm just curious how QVGE could still be useful for the community :) )
Kind regards
Ars
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/ArsMasiuk/qvge/issues/156#issuecomment-881045670, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AUX3TAWTB7EZYNXTQCBCLLLTX5MVBANCNFSM5ANMLVRQ.
-------- Ce message et toutes les pièces jointes sont confidentiels et établis à l'intention exclusive de son ou ses destinataires. Si vous avez reçu ce message par erreur, merci d'en avertir immédiatement l'émetteur et de détruire le message. Toute modification, édition, utilisation ou diffusion non autorisée est interdite. L'émetteur décline toute responsabilité au titre de ce message s'il a été modifié, déformé, falsifié, infecté par un virus ou encore édité ou diffusé sans autorisation. -------- This message and any attachments are confidential and intended for the named addressee(s) only. If you have received this message in error, please notify immediately the sender, then delete the message. Any unauthorized modification, edition, use or dissemination is prohibited. The sender shall not be liable for this message if it has been modified, altered, falsified, infected by a virus or even edited or disseminated without authorization. --------

Dear Jonathan,
Thanks for choosing QVGE for your project, hope you'll find it helpful.
You could use branch 0.7.dev as it should be more stable than 0.6 ones. It just has to be compiled from sources, but this should be easy enough.
Please feel free to report about any issues here.
According to the icons: no, I haven't designed them by myself - I've just took an original Inkscape icons set (I think one of version 0.91 but I can be wrong here). As far as the whole Inkscape is licensed under GPL, I assume the icons are under GPL too (at least I haven't seen any separate license for them).
But here I would suggest you to contact Inkscape developers to let them answer your question more in detail.
Kind regards
Ars

Hi!
May I close this ticked already?


[5.7] throw explicitly in public method

This one is a suggestion to throw explicitly in a public method body rather than calling return $this->someProtectedMethod() which in fact throws exception internally. It helps in readability of the code.
No functional or BC breaking change in public interface.
little chance to be BC breaking in case where someone is extending & using protected Container::notInstantiable method in their implementation. That is because this method does not throw, but rather returns the exception now.
To address that fact 2 replacements should do (sublime style in regex mode):

normal regex can do in one go in greedy mode.

I am closing this pull request because it lacks sufficient explanation, tests, or both. It is difficult for us to merge pull requests without these things because the change may introduce breaking changes to the framework.
Feel free to re-submit your change with a thorough explanation of the feature and tests - integration tests are preferred over unit tests. Please include it's benefit to end users; the reasons it does not break any existing features; how it makes building web applications easier, etc.
Thanks!


GCE Cloud Provider lists too many images

GCE currently has about 420 operating system images across all officially supported platforms. However, only about 20 of them are considered non-deprecated.  Normally, users will only want to use the select from the most current list.
Incoming fix for this.

@miq_bot add_label providers/google

Forgot to /cc @agrare and @blomquisg

@erjohnso If we don't inventory deprecated images then we won't be able to link instances to the images that were used to created them if that image is now deprecated, but I definitely agree that we show too many images.
I think a good approach would be to add a 'deprecated' field to the images model, that way we can still have a record of the relationship but only show current images.  Thoughts?

Is your suggestion limited to the Google provider, or a general fix?  If it's just Google, then I think I can keep this issue open and work on a new PR.
One of the main reasons we deprecate images is because of CVEs, so limiting inventory access to old images is a good thing.

It would be added for all providers but for now only gce would fill it in.

@agrare I think we already do something like this for amazon, in particular with their public images.

Cool thanks @Fryguy I'll look into that

@Fryguy are you talking about get_public_images: false here https://github.com/ManageIQ/manageiq/blob/master/config/vmdb.tmpl.yml#L73?  Looks like that skips gathering inventory for those entirely.


How to add PDO in non-consecutive Indexes

Hello
I want to add a new PDO,
but this PDO is not consecutive to indexes
0x1800, 0x1801, 0x1802, 0x1803,
is it possible to add, for instance a PDO in.... 0x190A index?
if not, what is the maximun number of PDO that I can use canopenNode library?
Thank you very much in advanced
P.d. I have already found something similar in this post
https://github.com/CANopenNode/CANopenNode/issues/309
but I don´t find were to activate the CO_MULTIPLE_OD  option

Library itself is object oriented and it is possible almost anything.
The "problem" are files CANopen.c and .h, which tries to be flexible and so they may be complex. You can do some "magic" there. Or if you need customized solution, you can make your own.
You can activate CO_MULTIPLE_OD in your CO_driver_target.h or in Makefile. CANopenPIC (PIC32) or CANopenLinux or CANopenDemo contains most up to date examples (but not for CO_MULTIPLE_OD).

Yes, I think I could manage in CANOpen.c File. In deed, I did it in the previous CANOpen node version 6 years ago, but Now the actual code seems too different from the old, version, at some point it was a huge change.
For instance, Now I don´t know how to create CALLBACKs functions when I write in and index.
In the "old Style" Code there was a function called  CO_OD_configure that was easy to all and then implement the call back function
example: CO_OD_configure(CO->SDO, 0x2101       , CO_ODF_nodeId, 0, 0, 0);
and then
you could create the call back function very easily code like this
static CO_SDO_abortCode_t CO_ODF_nodeId(CO_ODF_arg_t *ODF_arg){
}{
I cannot find how to make call backs in this code, neither in the PIC32 example, there is any call back.
do anyone  know where to find any example or documentation for the call backs?
thanks a lot
David.-

Yes, please see CANopenDemo project, there is example.
CANopenNode was actually rewritten. New approach to object dictionary as a core. All code was mostly rewritten. Configuration options allows low ram usage or comfort with functionalities. Documentation is up to date.


Special Characters In SQL Server Connection String

In the "Copy Data Portal" wizard I'm getting an error, "It is failed to decrypt the credential ...".  It's not clear what and why this is.  However, when I use the "Microsoft Integration Runtime Configuration Manager Preview" application I get a slightly different error: "sequence: \s. Path 'properties.typeProperties.connectionString', line 5, position 88."  For me, this is due to a special character in the server name, which is interpreted as an escape character '\s'.  The fix is to use double-backslash, '\s'.  I'm wondering if this is causing encryption/decryption errors in the portal because they are out of sync?

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

ID: abf7e612-a8a4-ff95-2dd1-2c0eef051634
Version Independent ID: edd2ffa9-0b4b-42f6-a3fb-24ef25c9cdec
Content: Copy on-premises data by using the Azure Copy Data tool | Microsoft Docs
Content Source: articles/data-factory/tutorial-hybrid-copy-data-tool.md
Service: data-factory
GitHub Login: @linda33wj
Microsoft Alias: jingwang

@mkrain Thanks for the feedback! We are currently investigating and will update you shortly.

@linda33wj I am unsure if this is something that should be enhanced in the documentation or an update to the product to not allow certain characters to be used when creating a host or service name.

Apologies for the delay in responding.  We have updated our process and resources so we will be more responsive in the future. There shouldn't be such issue now in ADF. Feel free to let know if you still see abnormal behavior.

#please-close


iOS and text: fix UTF-8 support on iOS and improve doc on FFI related text services

This PR fix the support for UTF-8 on iOS by simply updating the encoding and use the correct number of bytes. This does not completely resolve #1945 as the error messages are still present and I suspect that the UTF-8 cleaning does not work, however I'll update the issue (or open a different one) to reflect that.
Intro the service to_s_with_copy_and_length which complements to_s_with_copy and to_s_with_length . The name is a mouthful, but using optional parameters was not desirable since this service will often be used from a foreign language.
Also improves the doc of the existing services to get a Nit String from a NativeString. The new doc is more detailed and oriented towards users of the FFI by putting emphasis on where the raw data resides and wether it is cleaned or not.
Also note the removal of return type declarations in method refinements which was not consistent with the introduction.

@R4PaSs Of course, I would like your opinion on this, and please check if I misunderstood the memory management or the meaning of the length parameter in each method. For example, in to_s_with_length (and I guess to_s) the _items of the String could use either the NativeString itself or reallocate it with the GC, right?

From what I see, this looks cohesive with the rest of the API, :shipit: !
UTF-8 cleaning might not work on several platforms due to a few problems with the way ints are compiled: 0x80808080 is used internally as a mask and the compiler might generate the integer<PHONE_NUMBER> which when used as a long might overflow in the case of 32-bit platforms and break some core string functions.
I'm not sure there's an issue opened for this but I'm supposed to work on this soon.

My first guts question is when are the copied nativestrings freed?

They are managed by the GC, vs being from unknown origin. I was looking for a term to make it more clear, I ended up with Nit memory and Nit controlled memory, but if you have a better idea I'm open to suggestion.

How are memory management stuff called in other FFI and native interface proposals?

+1 for me anyway

Others FFI talk about the language controlled memory, GC heap and such. However they rarely have the same ambiguity since they don't often manipulate native strings in the high level language, it's more the other way around.


Examples for various databases

It would be nice to have specific examples for the major databases such as SAP, DB2 etc.

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

ID: dad6b54f-7b8d-0d9b-0ece-815b8874f751
Version Independent ID: dee28cea-a153-0d80-b3f1-982aa4d16a87
Content: Access external data: ODBC generic types - PolyBase - SQL Server
Content Source: docs/relational-databases/polybase/polybase-configure-odbc-generic.md
Product: sql
Technology: polybase
GitHub Login: @MikeRayMSFT
Microsoft Alias: mikeray

@shawnnesser -- thank you for your feedback.  You may some examples here:

DBA Stack Exchange
Stack Overflow

You are welcome to raise this issue here:

SQL Server Feedback


Updated features for extra_vars passing

These changes add a number features related to passing additional parameters for a job. Those are the ability to:

Specify arguments on the command line, instead of needing to create a text file
Consistency with Ansible's syntax of specifying a file by starting with and "@" character
Add your parameters while still preserving those in Ansible Tower's job template

This is not needed as of Ansible Tower 2.3.0, and CLI will act accordingly

Use any number of --extra_vars flags, allowing you to combine command line input with file contents

We also have tests for all this, as well a few unrelated syntax and test edits to bump up coverage to 100% again after it subtly fell after some other recent commits. The core method remains mostly the same as the pull request #43 by @cedub, which is to parse all the available sources of variables, combine them, then dump them into a single variable definition string that can be passed to the API. So thanks for that contribution.
Usage follows the following pattern for launching a job:

$ tower-cli job launch --job-template=1 --extra-vars="a: 5"<EMAIL_ADDRESS>
This command combines the variables in the file called filename.yml with the command-line set of variables that just says a=5. The job template can also be created with the same syntax:

$ tower-cli job_template create --name=test_job_template --project=1 --inventory=1 --playbook=helloworld.yml --machine-credential=1 --extra-vars="a: 5"<EMAIL_ADDRESS>
Note that it combines the sources of extra_vars and stores it as only one yaml string. This means that the modify command works as it always has, but you will only have the ability to enter one --extra-vars flag in that case.

@AlanCoding Ansible supports the <key>=<val> syntax (e.g. --extra_vars foo=bar).  Will tower-cli honor that as well?

No, it does not support that. To get that level of correspondence with Ansible, my suggested solution would be import Ansible itself. I will look into that.

I haven't tried running this but can you speak to how variables will be handled if they are defined in multiple places? Eg if a variable is defined at the commandline and in extra_vars of the job template will they merged or will one side "win?"

@perzizzle Yes, this will be covered in the updated README as well. The different sources will be merged if there is more than 1. You can use the flag multiple times, and the final one will take priority. All of them take priority over the job template extra-vars. So your command could include --extra-vars="a=1 b=2" --extra-vars="a=3", and you would end up with a=3 and b=2. You can include a file as in<EMAIL_ADDRESS>but it's priority will be set by its order just like all the other flags.
What matters most is that the job template variables are over-written, and that they're not forgotten when even more variables are passed. It will do that correctly.

@AlanCoding :+1: from me. Nice work!


add papertrail to auditing library recommendations

I have no opinion on the Audited, but we use papertrail in production to great success.

Hey @KelseyDH, thanks for the PR. This isn't intended to be a comprehensive list of libraries or services (I'm worried it'll lead to clutter), so I've just included ones I've used in the past.


Question: Tracking Download Statistics

For many developers, keeping track of download statistics (how many downloads, unique users, etc) is useful for justifying one's work to bosses or funding agencies.  Is this something that's in development for either the binstar client or the binstar web dashboard?

We count the amount of downloads. closing


When the plugin is verifying it does not parse syntax

When using Dafny 3.4 and ide upstream I can jump to definitions,
however if verification hangs, then I can not :(

Can you provide source that reproduces the problem?

It may be released to #115 I'l try it some more

Is this issue still showing up for you, @seebees?

I don't think this issue is showing up after the recent refactorings.


Race Condition results in "Incorrect authentication service"

I searched previous issues for "Incorrect authentication service" and it seems like there have been many, as there are a variety of problems that can occur resulting in the backend not being found.  Many of the issues are closed as the resulting problems were misconfigurations with backends, but I believe I've come across a fundamental problem that can occur with how backends are discovered.
Note: It won't be a problem when using Django's development server as it's a single process/thread, but when running under a multi process/thread environment as most production environments will use, a race condition can sometimes occur.
The fundamental problem is that social_auth.backends.BACKENDS is populated at import time, and that may occur prior to all backend classes having actually been defined. Particularly, backends usually require importing base classes from social_auth.backends that they can subclass, prior to them having been defined. The BACKENDS population code then runs, and the backend classes aren't picked up as they haven't been defined yet.
Like I said you won't be able to reproduce this using Django's development server, but if you run a project using gunicorn you will eventually come across this issue after a number of restarts of gunicorn. Sometimes it will work, other times it won't. The behaviour basically sticks to each startup of a project, since the code only runs once when first imported.
A workaround for me has been to explicitly add my backend to social_auth.backends.BACKENDS in my urlconf and this resolves the issue.
As a more permanent solution, I would recommend throwing out the whole approach of walking the filesystem from SOCIAL_AUTH_IMPORT_SOURCES, and just have SOCIAL_AUTH_ENABLED_BACKENDS defined as Python dotted paths (in the same way Django defines INSTALLED_APPS, MIDDLEWARE_CLASSES etc) that are imported inside get_backend, making get_backends, BACKENDS, and SOCIAL_AUTH_IMPORT_SOURCES no longer required. If this sounds like a good approach please let me know, I'd be happy to submit a pull request to fix it, but I thought I'd just gauge the direction you'd like to take first.
Finally, thanks for the awesome app - it really is an amazing piece of work that obviously a ton of effort has been put into, and one of my always-go-to solutions for the problem it solves.

OMG, what a good explanation you drop there, good work. About the fix, I really like it, a sane default will be needed to keep backward compatibility of course. If you send a pull request I'll look into it ASAP.
Thanks!


[5.7][stdlib] Work around binary compatibility issues with String index validation fixes in 5.7

(Cherry picked from #58954)

Explanation:  Swift 5.7 added stronger index validation for String, so some illegal cases that previously triggered inconsistently diagnosed out of bounds accesses now result in reliable runtime errors. Similarly, attempts at applying an index originally vended by a UTF-8 string on a UTF-16 string now result in a reliable runtime error.
As is usually the case, adding new traps to the stdlib exposes some existing code that contains previously undiagnosed / unreliably diagnosed coding issues. This change allows invalid code inside binaries built with earlier versions of the stdlib to continue running with the 5.7 library by disabling some of the new traps based on the version of Swift the binary was built with.
In the case of an index encoding mismatch, allow transcoding of string storage regardless of the direction of the mismatch. (Previously we only allowed transcoding a UTF-8 string to UTF-16.)

Scope: String in the Standard Library; binary compatibility.

Risk: Medium. While the change is relatively large, it merely loosens the Stdlib's preconditions, allowing entry points to succeed in more cases. The primary risk has to do with the introduction of the new bincompat checks, especially as they come with a new stdlib/runtime interface through SwiftShims additions.

Issue: rdar://93379333

Reviewer: @glessard

@swift-ci test


Add suggestion for bad package id.

This adds some suggestions to the error message if a pkgid spec does not match any packages.

r? @alexcrichton
(rust-highfive has picked a reviewer for you, use r? to override)

r? @alexcrichton
(rust-highfive has picked a reviewer for you, use r? to override)

@bors: r+
Nice!

@bors: r+
Nice!

:pushpin: Commit b04c7fb849de72f14cdddb5da7c123abfc5fa6e4 has been approved by alexcrichton

:pushpin: Commit b04c7fb849de72f14cdddb5da7c123abfc5fa6e4 has been approved by alexcrichton

:hourglass: Testing commit b04c7fb849de72f14cdddb5da7c123abfc5fa6e4 with merge b52fc0a8270d671e33dafe8dec355624727e8534...

:hourglass: Testing commit b04c7fb849de72f14cdddb5da7c123abfc5fa6e4 with merge b52fc0a8270d671e33dafe8dec355624727e8534...

:sunny: Test successful - checks-actions
Approved by: alexcrichton
Pushing b52fc0a8270d671e33dafe8dec355624727e8534 to master...

:sunny: Test successful - checks-actions
Approved by: alexcrichton
Pushing b52fc0a8270d671e33dafe8dec355624727e8534 to master...


Implement deploy command for CI/CD

The deploy command should deploy all containers in all shipments defined by compose files. This command is a subset of the up command without support for uploading environment variables or making any other shipment modifications. It's useful for making CI/CD pipelines even easier.
For example, given the following harbor-compose.yml ...
version: "1"
shipments:
  shipment1:
    env: dev
    containers:
      - container1
      - container2
  shipment2:
    env: dev
    containers:
      - container3
      - container4

and the following env vars defined in your build...
SHIPMENT1_DEV_TOKEN
SHIPMENT2_DEV_TOKEN

you could deploy the shipments from CircleCI using:
harbor-compose deploy

and if you wanted to also deploy the shipments to QA, you could add the following env vars and then do:
SHIPMENT1_QA_TOKEN
SHIPMENT2_QA_TOKEN

harbor-compose deploy --env qa

...all with a single harbor-compose.yml.  And your CI/CD build scripts would be nice and clean...

set image versions (either as env vars or defined in docker-compose.yml)
docker-compose build
docker-compose up -d && ./test
docker-compose push
harbor-compose deploy

...and all of your container images would be built, tested, pushed to a registry, cataloged and deployed.

In develop branch now.  Release as 0.10 after catalog command is updated to use customs.


Update readme after first release to Maven Central

Should be merged only when the release is available...

Hi,
should we maybe change the version in build.gradle.kts to 0.1.0-SNAPSHOT again?
Else this build will be published to the staging for maven central again.

Good point, changed...


Improve usage of delete function

@Foltik I would appriciate a quick look from a second pair of eyes before it is merged to master and released, but this has worked fine for me during my testing. Problem is I dont use the config* type of resources any longer as I am working on adding more proper ones.

Looks good and tested to be working!


Docs: Playground does't load pyodide in mobile Firefox (at least on Android)

I don't think there is much I can do, but thought I'd note it here that the playground doesn't work in mobile Firefox on Android. Works great on normal, desktop Firefox, so...🤷🏻.
If anyone runs into this, this issue is to assert the issue is known. I am unaware of any workarounds right now as Pyodide's official repl demo won't load in mobile Firefox either, so I imagine they need to fix something on their end.

This doesn't seem to be an actual problem. I have a newer phone, and docs seem to work fine in firefox. Probably just my old crappy phone.


PATCH requests not supported?

Getting the following error TypeError: _fetchMock2.default.patch is not a function when attempting to:
fetchMock.patch('some-link', {})

Does the fetch-mock api support mocking PATCH requests?

read the docs https://github.com/wheresrhys/fetch-mock/blob/master/README.md#head


fix(issue#25182): eventhub and svervicebus processor client start span independently

processor can start span if events or messages come from upstream with opentelemetry is not enable
Description
Please add an informative description that covers that changes made by the pull request and link all relevant issues.
If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[ ] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

Sorry, the former PR I rebase wrong.  Here is a new PR for fixing issue https://github.com/Azure/azure-sdk-for-java/issues/25182

@lmolkova please take a look. I added the test and fix the same logic in service bus sdk.


Loggie installed as unwanted dependency

Steps to reproduce:

Open a new project
Add the Sentinel package
Choose None for the SentinelLoggie target selection
Wait for the installation to finish
Observe that Loggie is installed in the project even though you opted-out.

Due to internal changes to Sentinel, both Loggie and Collar dependencies will be removed in the next release (#36).
Once the new version is released, this issue will be closed.


Normalize default credentials across ALL platforms

The BOSH release, Docker images, Docker Compose, and software itself do not agree on the default failsafe creds for SHIELD.  Find common ground and bring all systems in-line, for documentation's sake.

The new default credentials, everywhere, are admin:password


HFValidationError

Hello! I downloaded v2-1_512-ema-pruned.ckpt and changed the Line106 of train.py with the corresponding path. However I get
Traceback (most recent call last):
  File "train.py", line 767, in <module>
    main(args)
  File "train.py", line 423, in main
    tokenizer = AutoTokenizer.from_pretrained(
  File ".../conda_envs/IDEGAN/lib/python3.8/site-packages/transformers/models/auto/tokenization_auto.py", line 686, in from_pretrained
    tokenizer_config = get_tokenizer_config(pretrained_model_name_or_path, **kwargs)
  File ".../conda_envs/IDEGAN/lib/python3.8/site-packages/transformers/models/auto/tokenization_auto.py", line 519, in get_tokenizer_config
    resolved_config_file = cached_file(
  File ".../conda_envs/IDEGAN/lib/python3.8/site-packages/transformers/utils/hub.py", line 429, in cached_file
    resolved_file = hf_hub_download(
  File ".../conda_envs/IDEGAN/lib/python3.8/site-packages/huggingface_hub/utils/_validators.py", line 111, in _inner_fn
    validate_repo_id(arg_value)
  File ".../conda_envs/IDEGAN/lib/python3.8/site-packages/huggingface_hub/utils/_validators.py", line 159, in validate_repo_id
    raise HFValidationError(
huggingface_hub.utils._validators.HFValidationError: Repo id must be in the form 'repo_name' or 'namespace/repo_name': '/mnt/cephfs/home/gebaltso/home/gebaltso/SD2ch/2'. Use `repo_type` argument if needed.

Any idea what I can do?

Stable diffusion provides a folder-version way and a ckpt-version way for loading their model.
for example, the folder-version way:
vae = AutoencoderKL.from_pretrained("stabilityai/stable-diffusion-2-1-base", subfolder="vae")
tokenizer = CLIPTokenizer.from_pretrained("openai/clip-xxx")
text_encoder = CLIPTextModel.from_pretrained("openai/clip-xxx")
unet = UNet2DConditionModel.from_pretrained("stabilityai/stable-diffusion-2-1-base", subfolder="unet")
the ckpt-version way:
pipe = StableDiffusionPipeline.from_pretrained("stabilityai/stable-diffusion-2-1-base")
our codes are based on the folder-version way. you could try to change to the ckpt-version way or download the folders in https://huggingface.co/stabilityai/stable-diffusion-2-1-base/tree/main.


(Non-fatal) errors during juliabox image build

During the juliabox image build I note a few failures.
The first few failures are due to sudo failing on apt-get (sudo timeout? container configuration problem?) during Pkg.build. This affects, e.g.,  hfd5-tools , libnlopt0, libcairo2, libmagickwand5, tcl8.5 which report errors such as:
INFO: Building HDF5
Installing dependency hdf5-tools via 'sudo apt-get install hdf5-tools':
sudo: no tty present and no askpass program specified
================================[ ERROR: HDF5 ]=================================

failed process: Process('sudo apt-get install hdf5-tools', ProcessExited(1)) [1]
while loading /opt/julia_packages/.julia/v0.3/HDF5/deps/build.jl, in expression starting on line 24

===============================================================================

Another failure occurs when configure cannot find gmp.h.
configure: error: gmp.h header not found
================================[ ERROR: GLPK ]=================================

failed process: Process(setenv(`/opt/julia_packages/.julia/v0.3/GLPK/deps/src/glpk-4.52/configure --with-gmp --prefix=/opt/julia_packages/.julia/v0.3/GLPK/deps/usr`,Union(UTF8String,ASCIIString)["LDFLAGS= -L/opt/julia_packages/.julia/v0.3/GLPK/deps/usr/lib -L/opt/julia_0.3.11//lib -Wl,-rpath -Wl,/opt/julia_packages/.julia/v0.3/GLPK/deps/usr/lib","PATH=/opt/julia_packages/.julia/v0.3/GLPK/deps/usr/bin:/usr/local/texlive/2014/bin/x86_64-linux:/usr/local/bin:/usr/bin:/bin:/sbin:/usr/sbin:/opt/julia/bin","SHLVL=1","PWD=/home/juser","JULIA_PKGDIR=/opt/julia_packages/.julia","HOME=/home/juser","PKG_CONFIG_PATH=/opt/julia_packages/.julia/v0.3/GLPK/deps/usr/lib/pkgconfig","HOSTNAME=80acce8971ef","_=/usr/bin/julia","CPPFLAGS= -I/opt/julia_packages/.julia/v0.3/GLPK/deps/usr/include -I/opt/julia_0.3.11//include"]), ProcessExited(1)) [1]
while loading /opt/julia_packages/.julia/v0.3/GLPK/deps/build.jl, in expression starting on line 61

================================================================================

These errors are probably because the Dockerfile (JuliaBox/docker/Dockerfile) is based on julialang/julia instead of julialang/juliaboxpkgdist.
julialang/juliaboxpkgdist is a larger image and has all the libraries pre-installed for the packages JuliaBox uses. This is what is being used at juliabox.org.
julialang/julia is much smaller and lacks those packages. This is useful for a development setup.
You could try either of the following:

switch to julialang/juliaboxpkgdist and rebuild (JuliaBox/scripts/install/img_create.sh build /jboxengine/data)
trim down the number of packages being installed from JuliaBox/docker/setup_julia.sh

Got it.
The documentation you have recently produced has been extremely valuable in getting to grips with installing a JuliaBox development environment. I think I could add some helpful details for newcomers.
Would you mind if I fork, open a PR for you to review, and if you think its useful perhaps merge?

@samuelpowell Please do. It will be great to have more people install/contribute to JuliaBox.

@tanmaykm, I will put something together. Thanks for your hep.


Chatbot cannot be imported after update

Hi, I updated from Pypi this morning, and now I am getting this error as soon as I call
from Bard import Chatbot
the error that I get is
line 119, in Chatbot     def load_conversations(self, file_path: str) -> list[dict]: TypeError: 'type' object is not subscriptable
Any help greatly appreciated...

I made the following adjustments, and the loading and importing work fine now... since I only intend to use this for one off questions, I think it should work now, but ultimately I would like to know a better way if possible.
This was the original line 119 in Bard.py, effective this mornings update...
   def load_conversations(self, file_path: str):         print('Load_Conversations was just called with '+ file_path)         return []     # def load_conversations(self, file_path: str) -> list[dict]:     #     # Check if file exists     #     if not os.path.isfile(file_path):     #         return []     #     with open(file_path, encoding="utf-8") as f:     #         return json.load(f)

Duplicate of #43 and already fixed


ENH: format errors nicely on multiline error strings

EDIT: Uses exception chaining so there is no longer a confusing traceback shown to the user. Also color-codes single-line errors and prepends then with meson-python. See this comment below for a real-world example.
Original description:
Before
meson-python: error: A single line error message

meson-python: error: A multiline error message. Eg. X went wrong in the fields:
- aaa
- bbb

And Y went wrong in:
- ccc
- ddd

After
meson-python: error: A single line error message

meson-python: error:
>
> A multiline error message. Eg. X went wrong in the fields:
> - aaa
> - bbb
>
> And Y went wrong in:
> - ccc
> - ddd
>

With the latest commit, I kept single-line error the same, following the review, keeping the change for multi-line errors only.
Note, the error messages get nicely colorized too. The > line prefix is red, and the following is reset text.

@FFY00 thanks. It would be helpful if you could add examples of both the old and new output in the PR description, so it's clear what this PR changed.

I've done some testing here, by comparing the behavior when not specifying project.name. On main I get this (no color-coding):
Found ninja-1.11.1 at /home/rgommers/mambaforge/envs/scipy-dev/bin/ninja
Traceback (most recent call last):
  File "/home/rgommers/mambaforge/envs/scipy-dev/lib/python3.10/site-packages/pep517/in_process/_in_process.py", line 351, in <module>
    main()
  File "/home/rgommers/mambaforge/envs/scipy-dev/lib/python3.10/site-packages/pep517/in_process/_in_process.py", line 333, in main
    json_out['return_val'] = hook(**hook_input['kwargs'])
  File "/home/rgommers/mambaforge/envs/scipy-dev/lib/python3.10/site-packages/pep517/in_process/_in_process.py", line 249, in build_wheel
    return _build_backend().build_wheel(wheel_directory, config_settings,
  File "/home/rgommers/mambaforge/envs/scipy-dev/lib/python3.10/site-packages/mesonpy/__init__.py", line 1034, in wrapper
    return func(*args, **kwargs)
  File "/home/rgommers/mambaforge/envs/scipy-dev/lib/python3.10/site-packages/mesonpy/__init__.py", line 1097, in build_wheel
    with _project(config_settings) as project:
  File "/home/rgommers/mambaforge/envs/scipy-dev/lib/python3.10/contextlib.py", line 135, in __enter__
    return next(self.gen)
  File "/home/rgommers/mambaforge/envs/scipy-dev/lib/python3.10/site-packages/mesonpy/__init__.py", line 984, in _project
    with Project.with_temp_working_dir(
  File "/home/rgommers/mambaforge/envs/scipy-dev/lib/python3.10/contextlib.py", line 135, in __enter__
    return next(self.gen)
  File "/home/rgommers/mambaforge/envs/scipy-dev/lib/python3.10/site-packages/mesonpy/__init__.py", line 829, in with_temp_working_dir
    yield cls(source_dir, tmpdir, build_dir, meson_args, editable_verbose)
  File "/home/rgommers/mambaforge/envs/scipy-dev/lib/python3.10/site-packages/mesonpy/__init__.py", line 717, in __init__
    self._metadata = pyproject_metadata.StandardMetadata.from_pyproject(pyproject, self._source_dir)
  File "/home/rgommers/mambaforge/envs/scipy-dev/lib/python3.10/site-packages/pyproject_metadata/__init__.py", line 224, in from_pyproject
    raise ConfigurationError('Field `project.name` missing')
pyproject_metadata.ConfigurationError: Field `project.name` missing

ERROR Backend subproccess exited when trying to invoke build_wheel

With this PR I get (with color-coding of meson-python: error: and the > symbols):
Found ninja-1.11.1 at /home/rgommers/mambaforge/envs/scipy-dev/bin/ninja
meson-python: error:
>
>  Field `project.name` missing
>

ERROR Backend subproccess exited when trying to invoke build_wheel

So the key improvement is hiding the large and uninformative traceback (color coding is a nicety too) for single-line errors.
Nothing changes when there's multiple errors in the build log as of now. It's not something all that important compared to the single-error logging improvement I'd say.

With this PR I get (with color-coding of meson-python: error: and the > symbols):

This should not be the case, it's a bug in the PR. The > format should only be used when the errors are multiline, per the review above.

I don't think those will arrive any time soon, so I decided to ignore that part. Let's cross that bridge if and when we get to it - it really isn't very interesting either way.
I updated the commit message to explain the actually relevant, and significant, improvement here. I'll also re-title this PR to reflect that.

This broke reporting error messages that contain curly brackets. There are several places where meson-python reports errors about user supplied configuration.If any of these configuration strings contains a curly bracket, the error message results in an exception. meson-python also emits an error when it is not able to map a Meson install prefix into a wheel installation path and Meson install prefixes are defined with curly brackets.

Thank you for noticing it. Feel free to submit a PR if you want, otherwise I will submit one this week.


Refactor to allow multiple initialization listeners to be attached

This allows for multiple listeners to be attached to the same event, avoiding multiple uses of it clashing.

(Re-reviewed by @mrmans0n)


feat(router): handle cookie header internally

PR Checklist

Closes: None.
What is the new behavior?

Cookies will be available on load function of .server.ts files without implementing a custom interceptor.

Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information
Very small test:

In the apps/analog-app/src/app/pages/(home).server.ts file, add a console.log.

Without this change, if you remove the cookies interceptor of apps/analog-app/src/app/app.config.ts, the console log will print an empty object.

What gif best describes this PR or how it makes you feel?

What if they already have a "cookie" header? I think the header should have a unique name at least, such as X-Analog-Cookie.
Is cookie one value or a dictionary of key/value cookies?

What if they already have a "cookie" header? I think the header should have a unique name at least, such as X-Analog-Cookie.
Is cookie one value or a dictionary of key/value cookies?

I was thinking that the cookie header should only be added if the request URL has the format /api/_analog/whatever; I believe it can work in that case.

The Idea is, if we make a request to by example: http://localhost:4200/posts/my-slug, when the load function of the .server.ts file runs, I can get the cookies using the h3 functions (getCookie, parseCoikies, etc). If we set custom headers, we won't be able to use the h3 cookies functions.

cookie

cookie is a complete cookie string, the server with the h3 functions parse internally for us that string

The idea is that if we make a request to, for example, http://localhost:4200/posts/my-slug, when the load function of the .server.ts file runs, we can get the cookies using the h3 functions (getCookie, parseCookies, etc.). If we set custom headers, we won't be able to use the h3 cookies functions, we would have to use the headers functions (that it is valid).
The only cookies this request https://github.com/analogjs/analog/blob/beta/packages%2Frouter%2Fsrc%2Flib%2Froute-config.ts#L71 should have are the server request cookies

I see, that makes sense. We can keep it scoped to start as its only for internal Analog requests.


Refactor development demo to be split up into views

Add react-navigation to split up the development demo into separate views. This improves performance of the overall demo app.
I also cleaned up and upgraded dependencies.

cc @boygirl @chrisbolin @kenwheeler

Thank you!


remove lib merging from yosys with new yosys version

@sebinho FYI

Thanks
Will it be an issue if my different LIB files contain templates with the same name?

@sebinho it shouldn't, if it does we would need to get that fixed in ABC. Please try it out and let me know if you have any issues.

Thats great to hear.


Add words ending with "ty"

Add words ending with "ty"
Refer Contribution.md for more details on how to contribute.
Comment the word/words you're working on  to avoid conflicts.
Add reference to this issue i.e Issue #355 in your pull request description.

I will add Naughty.json file

I will add punctuality_noun.json file

adding Penalty

i will add Entity


Laravel 7.x Not Support

Why you not change composer.json for support Laravel v7 and up?

you can add this in your composer.json and run composer update
"repositories": [{         "type": "git",         "url": "https://github.com/neopheus/laravel-filemanager.git"     }],

Thanks, But "neopheus/laravel-filemanager" repository does not support laravel 7.2

Thanks, But "neopheus/laravel-filemanager" repository does not support laravel 7.2

Are u sure?
Check this https://github.com/UniSharp/laravel-filemanager/pull/898/commits/bbea8b23f652cbe8c6004e8c1e48992fa267992b

Yes, I am sure!
I just tried again, it made errors for laravel 7.2
The code of this file also shows:
    "illuminate/config": "5.4.* || 5.5.* || 5.6.* || 5.7.* || 5.8.* || ^6.0 || ^7.0 || ^7.1 || ^8.0",
    "illuminate/filesystem": "5.4.* || 5.5.* || 5.6.* || 5.7.* || 5.8.* || ^6.0 || ^7.0 || ^7.1 || ^8.0",
    "illuminate/support": "5.4.* || 5.5.* || 5.6.* || 5.7.* || 5.8.* || ^6.0 || ^7.0 || ^7.1 || ^8.0",
    "illuminate/http": "5.4.* || 5.5.* || 5.6.* || 5.7.* || 5.8.* || ^6.0 || ^7.0 || ^7.1 || ^8.0",
    "illuminate/container": "5.4.* || 5.5.* || 5.6.* || 5.7.* || 5.8.* || ^6.0 || ^7.0 || ^7.1 || ^8.0"

Yes, I am sure!
I just tried again, it made errors for laravel 7.2
The code of this file also shows:
    "illuminate/config": "5.4.* || 5.5.* || 5.6.* || 5.7.* || 5.8.* || ^6.0 || ^7.0 || ^7.1 || ^8.0",
    "illuminate/filesystem": "5.4.* || 5.5.* || 5.6.* || 5.7.* || 5.8.* || ^6.0 || ^7.0 || ^7.1 || ^8.0",
    "illuminate/support": "5.4.* || 5.5.* || 5.6.* || 5.7.* || 5.8.* || ^6.0 || ^7.0 || ^7.1 || ^8.0",
    "illuminate/http": "5.4.* || 5.5.* || 5.6.* || 5.7.* || 5.8.* || ^6.0 || ^7.0 || ^7.1 || ^8.0",
    "illuminate/container": "5.4.* || 5.5.* || 5.6.* || 5.7.* || 5.8.* || ^6.0 || ^7.0 || ^7.1 || ^8.0"

This Lines must be changes to:
    "illuminate/config": ">=5.4",
    "illuminate/filesystem": ">=5.4",
    "illuminate/support": ">=5.4",
    "illuminate/http": ">=5.4",
    "illuminate/container": ">=5.4"

or remove this lines and add simply this line only:
    "laravel/framework": ">=5.4"

How to install it on laravel 7.2

Yes, I am sure!
I just tried again, it made errors for laravel 7.2
The code of this file also shows:
    "illuminate/config": "5.4.* || 5.5.* || 5.6.* || 5.7.* || 5.8.* || ^6.0 || ^7.0 || ^7.1 || ^8.0",
    "illuminate/filesystem": "5.4.* || 5.5.* || 5.6.* || 5.7.* || 5.8.* || ^6.0 || ^7.0 || ^7.1 || ^8.0",
    "illuminate/support": "5.4.* || 5.5.* || 5.6.* || 5.7.* || 5.8.* || ^6.0 || ^7.0 || ^7.1 || ^8.0",
    "illuminate/http": "5.4.* || 5.5.* || 5.6.* || 5.7.* || 5.8.* || ^6.0 || ^7.0 || ^7.1 || ^8.0",
    "illuminate/container": "5.4.* || 5.5.* || 5.6.* || 5.7.* || 5.8.* || ^6.0 || ^7.0 || ^7.1 || ^8.0"

This Lines must be changes to:
    "illuminate/config": ">=5.4",
    "illuminate/filesystem": ">=5.4",
    "illuminate/support": ">=5.4",
    "illuminate/http": ">=5.4",
    "illuminate/container": ">=5.4"

or remove this lines and add simply this line only:
    "laravel/framework": ">=5.4"

How to install it on laravel 7.2

This will work in all versions of laravel equal or up to 5.4, Including 7.2 and up

Yes, I am sure!
I just tried again, it made errors for laravel 7.2
The code of this file also shows:
    "illuminate/config": "5.4.* || 5.5.* || 5.6.* || 5.7.* || 5.8.* || ^6.0 || ^7.0 || ^7.1 || ^8.0",
    "illuminate/filesystem": "5.4.* || 5.5.* || 5.6.* || 5.7.* || 5.8.* || ^6.0 || ^7.0 || ^7.1 || ^8.0",
    "illuminate/support": "5.4.* || 5.5.* || 5.6.* || 5.7.* || 5.8.* || ^6.0 || ^7.0 || ^7.1 || ^8.0",
    "illuminate/http": "5.4.* || 5.5.* || 5.6.* || 5.7.* || 5.8.* || ^6.0 || ^7.0 || ^7.1 || ^8.0",
    "illuminate/container": "5.4.* || 5.5.* || 5.6.* || 5.7.* || 5.8.* || ^6.0 || ^7.0 || ^7.1 || ^8.0"

This Lines must be changes to:
    "illuminate/config": ">=5.4",
    "illuminate/filesystem": ">=5.4",
    "illuminate/support": ">=5.4",
    "illuminate/http": ">=5.4",
    "illuminate/container": ">=5.4"

or remove this lines and add simply this line only:
    "laravel/framework": ">=5.4"

How to install it on laravel 7.2

This will work in all versions of laravel equal or up to 5.4, Including 7.2 and up:
composer require/update unisharp/laravel-filemanager

It not working https://github.com/UniSharp/laravel-filemanager/issues/904

It not working #904

😕

It not working #904

confused

do you find the solution?

Yes, install this forked package:
composer require hmoradian/laravel-filemanager for laravel up to version 7.3

I'm waiting for confirm my pull request:
https://github.com/UniSharp/laravel-filemanager/pull/900

do you find the solution?

I'm waiting for confirm my pull request:
Support Laravel 7.3

The package doesn't still support for laravel 7 :(

The package doesn't still support for laravel 7 :(

Of course for now (2020-04-03) :-)


Cannot correctly edit a form header

If you backspace on the editable field, it resets to the original value
Additionally, the field is not populated with the correct overridden value when it exists. (See attached)

duplicate of #2662


Change paths to allow compiling as module

This solves https://github.com/RestKit/RestKit/issues/2356

@jlampa Travis fails due to missing includes. It looks like something is still missing in this PR to make the build work.

This is probably not backwards compatible, if anyone is directly including a sub-part of restkit

Closing in favor of the other open PR

@segiddins what is the other open PR?

https://github.com/RestKit/RestKit/pull/2404, I think


What features are used to train a VQA model? DO you use only 2048-dimension features?

In your code, the image_id, image_h, image_w, num_boxes, boxes,  features were extracted and saved. But in your paper, it seems that only features are used to present the image. Do you use the embedding of the predicted classes or bbox to train a VQA model?

No we didn't use the class labels or the bbox. I did some initial experiments like that but performance didn't change much.


timeline archival: slimmed down timeline object

for archived timelines, we want things like walreceiver and walredo to be shut down. We should shut down/start up both according to the incoming network requests.
Part of https://github.com/neondatabase/neon/issues/8088

This was covered by creating OffloadedTimeline in https://github.com/neondatabase/neon/pull/8907, right?

yeah, it can be closed I'd say.


Auto follow links

See https://github.com/Siderite/lichessTools/issues/303
Some mechanism to allow people to quickly follow a displayed link (like pressing Space) and/or automated and configurable following of the link.
auto:somelink or auto:X:somelink where X is the number of seconds to wait until going there. There could be a countdown on the comment as well.

Complicated to explain to users, even with a ticker like interface. It feels too complicated for no real benefit.


TINKERPOP3-910 - In session transaction opened from sessionless request

https://issues.apache.org/jira/browse/TINKERPOP3-910
Test via:
mvn clean install -DincludeNeo4j
mvn clean -DskipIntegrationTests=false -DincludeNeo4j --projects gremlin-server verify

The summary for this change is in the comments of this commit:
https://github.com/apache/incubator-tinkerpop/commit/43f64c2a4aad965343ef87ce5407cb0b79d96714
VOTE: +1

Simple code changes, tests passed for me, everything looking good.
Vote: +1

@pluradj do you have time to help out with a vote here for gremlin server stuff?

+1 Ran mvn clean install -DincludeNeo4j and then mvn verify -DskipIntegrationTests=false -DincludeNeo4j successfully. Looks good to me.

RESULT: 4 +1 (3 binding, 1 non-binding) - merging


ODE geocoded school dataset added to DB

addition to DB of ODE geocoded school dataset (see link from slack for ode_schools.csv), school_id field joins with the below id fields.

joins with Attending School Institution ID field in ode_enrollment dataset (see github #197 for associated story card, ode_enrollment also listed in Demographics and Schools Sandbox package).
joins with SCHLINSTID field in ode_free lunch dataset (see github #228 for associated story card.

https://github.com/hackoregon/civic/issues/197


How to send files additional headers

Discussed in https://github.com/HapticX/happyx/discussions/225

Originally posted by derekdai December 20, 2023
I have tried with
serve("localhost", 5000):
  "/":
    let headers = newCustomHeaders()
    headers["x"] = "y"
    return FileResponse("index.html", headers = headers)

It compiles without problem, but the headers = header part are discarded.
  of HttpMethod(1):
    if `==`(urlPath, "/"):
      let headers = newCustomHeaders()
      headers["x"] = "y"
      await(answerFile(req, "index.html"))

No luck with outHeaders
serve("localhost", 5000):
  "/":
    outHeaders["x"] = "y"
    return FileResponse("index.html")

  of HttpMethod(1):
    if `==`(urlPath, "/"):
      var outHeaders = newCustomHeaders()
      outHeaders["x"] = "y"
      await(answerFile(req, "index.html"))
```</div>

@derekdai since latest commit outHeaders automatically injects into FileResponse.
answerFile is procedure, so outHeaders didn't injected into it - you should use headers = outHeaders directly when use answerFile procedure.

I see, thank you for the detailed explanation.


Advanced Settings: Mouse-over weird behavior

https://youtu.be/LliOE7XMgyY
OS: macOS Sierra

Unforeseen interaction between the form buttons and the button highlight. Should be easy to fix.

I swear I fixed this on one of my branches. Must have gone missing :(

This is addressed in #175 and #212

This was fixed.


bufio: uninitialised buffer in Writer causes panics and infinite loops

Please answer these questions before submitting your issue. Thanks!
What version of Go are you using (go version)?
go version go1.7.3 darwin/amd64
What operating system and processor architecture are you using (go env)?
GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOOS="darwin"
GOPATH="/Users/albert/go"
GORACE=""
GOROOT="/Users/albert/.gostable"
GOTOOLDIR="/Users/albert/.gostable/pkg/tool/darwin_amd64"
CC="clang"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/wp/sytm29yd1d1dclmf01xgvqbw0000gn/T/go-build808633337=/tmp/go-build -gno-record-gcc-switches -fno-common"
CXX="clang++"
CGO_ENABLED="1"
What did you do?
If possible, provide a recipe for reproducing the error.
A complete runnable program is good.
A link on play.golang.org is best.
If one will create buffer without using a constructor (buf := new(bufio.Writer)), then the further writes will cause panics or infinite loops depending on method one uses.
E.g. https://play.golang.org/p/LBrzU6xw2Z
What did you expect to see?
Buffer will be initialised, if it's not, and the further writes will be successful.
What did you see instead?
buf.WriteString causes an infinite loop: https://play.golang.org/p/LBrzU6xw2Z,
buf.WriteRune causes panic: https://play.golang.org/p/MAioIyqCJs,
buf.WriteByte causes panic: https://play.golang.org/p/e-JIMtF_tc,
and so on.

@bogem the documented way to create a bufio.Writer is via bufio.NewWriter (or bufio.NewWriterSize)
Please also consider asking questions like this in the go-nuts mailing list or other help forums as a first port of call. If the behaviour you are seeing is then confirmed as a bug/issue it can easily be re-raised here in the issue tracker, referencing the original thread.

@myitcv but user can also initalize writer by calling new method. The language permits doing this, so why not? In some cases it's the only way.
For example in pool of bufio.Writers. Pool can create new Writer by new(bufio.Writer), but it leads to this bug. So developer should write bufio.NewWriter(nil). Concrete example
Ok, sorry for this. Next time I will do so.
But many times I post some question or issue in go-nuts and it finds no answer at all :(

The language permits doing this, so why not?

I've always followed the guidance laid out here:
https://golang.org/doc/effective_go.html#allocation_new

Since the memory returned by new is zeroed, it's helpful to arrange when designing your data structures that the zero value of each type can be used without further initialization. This means a user of the data structure can create one with new and get right to work. For example, the documentation for bytes.Buffer states that "the zero value for Buffer is an empty buffer ready to use." Similarly, sync.Mutex does not have an explicit constructor or Init method. Instead, the zero value for a sync.Mutex is defined to be an unlocked mutex.

Hence, I first check to see whether there is an explicit constructor or Init method; if there is neither and there is documentation that confirms the zero value is "ready to use" then I feel safe in doing so. Some data structures cannot be designed in such a way that the zero value is "useful."

Ok, sorry for this. Next time I will do so.

I wouldn't say an apology is needed 😄 I make the comment just to try and help keep the issue tracker focussed on things that are known to be issues (if you look at the thousands of people who potentially get notified of each issue)

Closing because there is nothing to fix.


coqPackages.VST: 2.6 → 2.7.1

Motivation for this change
Support for Coq 8.13 and CompCert 3.8.
Cyril, do you have suggestions on how to improve the selection of the matching version of CompCert? The logic of defaultVersion is replicated at the call-site…
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Thanks for the feedback. Let’s move CompCert inside coqPackages as a future PR.


feat(screenshot): Update screenshot golds for merged PR

Thanks for adding the gold mode. It is really super clear now!


EvenMoreFish v<IP_ADDRESS> generated an exception when a competition ended.,

First time testing a competition with /emf admin competition start 600 largest_fish and at the end, no prizes were given out, but an exception was generated.
[22:26:40] [Server thread/WARN]: [EvenMoreFish] Task #6206 for EvenMoreFish v<IP_ADDRESS> generated an exception
java.lang.NoClassDefFoundError: me/clip/placeholderapi/PlaceholderAPI
	at com.oheers.fish.competition.reward.Reward.run(Reward.java:70) ~[even-more-fish-<IP_ADDRESS>.jar:?]
	at com.oheers.fish.competition.Competition.handleRewards(Competition.java:533) ~[even-more-fish-<IP_ADDRESS>.jar:?]
	at com.oheers.fish.competition.Competition.end(Competition.java:98) ~[even-more-fish-<IP_ADDRESS>.jar:?]
	at com.oheers.fish.competition.Competition.processCompetitionSecond(Competition.java:155) ~[even-more-fish-<IP_ADDRESS>.jar:?]
	at com.oheers.fish.competition.Competition.access$000(Competition.java:21) ~[even-more-fish-<IP_ADDRESS>.jar:?]
	at com.oheers.fish.competition.Competition$1.run(Competition.java:111) ~[even-more-fish-<IP_ADDRESS>.jar:?]
	at org.bukkit.craftbukkit.v1_18_R1.scheduler.CraftTask.run(CraftTask.java:101) ~[purpur-1.18.1.jar:git-Purpur-1564]
	at org.bukkit.craftbukkit.v1_18_R1.scheduler.CraftScheduler.mainThreadHeartbeat(CraftScheduler.java:483) ~[purpur-1.18.1.jar:git-Purpur-1564]
	at net.minecraft.server.MinecraftServer.tickChildren(MinecraftServer.java:1584) ~[purpur-1.18.1.jar:git-Purpur-1564]
	at net.minecraft.server.dedicated.DedicatedServer.tickChildren(DedicatedServer.java:519) ~[purpur-1.18.1.jar:git-Purpur-1564]
	at net.minecraft.server.MinecraftServer.tickServer(MinecraftServer.java:1507) ~[purpur-1.18.1.jar:git-Purpur-1564]
	at net.minecraft.server.MinecraftServer.runServer(MinecraftServer.java:1294) ~[purpur-1.18.1.jar:git-Purpur-1564]
	at net.minecraft.server.MinecraftServer.lambda$spin$1(MinecraftServer.java:322) ~[purpur-1.18.1.jar:git-Purpur-1564]
	at java.lang.Thread.run(Thread.java:833) ~[?:?]
Caused by: java.lang.ClassNotFoundException: me.clip.placeholderapi.PlaceholderAPI
	at org.bukkit.plugin.java.PluginClassLoader.loadClass0(PluginClassLoader.java:153) ~[purpur-api-1.18.1-R0.1-SNAPSHOT.jar:?]
	at org.bukkit.plugin.java.PluginClassLoader.loadClass(PluginClassLoader.java:105) ~[purpur-api-1.18.1-R0.1-SNAPSHOT.jar:?]
	at java.lang.ClassLoader.loadClass(ClassLoader.java:520) ~[?:?]
	... 14 more

Hey there, is your server running the latest version of placeholder API? If not could you let  me know if the issue persists with that version please? Cheers 👍

This was on my test server, and it looks like PlaceholderAPI isnt even installed! So its a hard dependency then!
Maybe put in a check into the plugin if its not installed? I will install it and test again!

That was it, seems to run prizes now. I suggest checking for placeholder API at boot, and if it is 100% required, then disable the plugin maybe and tell people its required as a hard dependency? or just disable placeholderAPI things if its not installed.

Ah glad that's working for you then :) It shouldn't be a hard dependency, so I'll leave this open and have a look what might be causing it to act like that,

Fixed a while back but issue was never closed.


Layout files for Swift Pacakge Manager

Initial work to layout the files/folders in the manner that SPM expects them to be.
Uses the conditional macros to use Glibc or Darwin depending on the platform.

I followed the comments from this guy's blog post so that we don't break carthage/pods support by re-arranging all the files, however, I don't know if this is required by another package, if SPM is smart enough to go into the SwiftPackageManager folder to actually build it.  Alas, I figured I would commit this up anyways as it's a helpful start at attempting to build the package on linux.

It looks like support for excluding paths from the swift build process was implemented on 2015-12-11, once thats in the next distro it would be possible to move the Sources folder back up to the top level, and then just add the ARI folders into the list of excludes so SPM doesn't try and build through them.
@see https://github.com/apple/swift-package-manager/pull/57

This goes against master, lets keep this stuff in the linux feature branch for now as Swift 2.2 is incompatible with 2.1.

Also, lets wait for the 2015-12-11 version then.

:+1:

Any news on this?

Source is now layout for SPM.


Add more plugin-hub search tags

Added a tag for the functionalities added in #63 and #67
I couldn't think of a good tag for Boss Kill Counts so I just skipped it 🎳

I was going to say achievement should be covered by combat achievements, if I understand how the tags work, but I have no clue why this plugin shows up for achievement 🤔

https://github.com/Godofdrakes/barrows-potential/blob/1f26887bd47f8161edf42f9c082f339321a707a2/runelite-plugin.properties#L5
https://github.com/Godofdrakes/barrows-potential/blob/1f26887bd47f8161edf42f9c082f339321a707a2/src/main/java/com/barrowspotential/BarrowsPotentialPlugin.java#L27


Sort tables and graphs

[x] Closes #796 .
[x] I am familiar with the contributing guidelines.
[x] Tests added or updated, no tests made for checking sorting as isn't straightforward to add javascript tests in pytest that I can find
[ ] Updates entries to docs/source/api.rst for API changes.  - NOT APPLICABLE
[x] Adds descriptions to appropriate "what's new" file in docs/source/whatsnew for all changes. Includes link to the GitHub Issue with :issue:`num` or this Pull Request with :pull:`num`. Includes contributor name and/or GitHub username (link with :ghuser:`user`).
[x] New code is fully documented. Includes numpydoc compliant docstrings, examples, and comments where necessary.
[x] Maintainer: Appropriate GitHub Labels and Milestone are assigned to the Pull Request and linked Issue.

Add table and figure sorting, see #796
WIP - only sorting tables done so far, need to add sorting figures still

Following breaking version changes required for CI tests:

Jinja2<3.1 due to Markup safe import change
Scipy<1.10 due to cannot import name 'PearsonRConstantInputWarning' from 'scipy.stats'
importlib-metadata<5 due to Entrypoint object attribute errors

Version 3.7 is no okay but 3.8 and 3.9 fail with what seems like some pandas issues.

pandas was required to be <1.4 to avoid multiple failures.

@dplarson
This should be good to go. The remaining errors were all dependency issues which I've updated setup.py requirements.
I've also added a whatsnew for release 1.0.14, but I think with other changes we should maybe target a minor instead of patch release especially changing to include net load and wind.


Add support for method compartments in xUML diagram

Methods are currently supported in xmm script language, but are not currently acted on during parsing.  Add in support so that the compartments draw with method content on executable uml class diagrams.

Simple enhancement as expected.


Style/OpMethod should not complain about the backticks method

Given the following code:
def `(cmd)
  system cmd
end

Rubocop complains with the following offence:
my_class.rb:40:7: C: Style/OpMethod: When defining the  operator, name its argument other`. (https://github.com/bbatsov/ruby-style-guide#other-arg)
def `(cmd)
      ^^^

I think that "other" is not an intuitive parameter name for the backticks method and that Rubocop should not report it as an offence.
$ rubocop -V
0.41.2 (using Parser <IP_ADDRESS>, running on jruby 2.3.0 java)

@Drenmi @bbatsov Thanks!


Moving to a Modern Build System

I think it makes sense to spend some time migrating asciimath's core functionality to a modern and more modular build system. I was thinking something like rollup. I find myself often requiring only a subset of asciimath's core functionality, and it would be super cool to be able to include only that functionality when I need it.
For example, I recently needed to convert asciimath markup to latex, which requires finding the right file (which has a large amount of duplicated code), and then using that file. It would be great to modularise this codebase for two main reasons:

It makes it much less intimidating to new contributors who might like to use it
It makes the intent of different code sections clearer
It makes it easier to integrate asciimath into your workflow

Any thoughts?

I personally find build systems annoying, but I'm not totally unopposed so long as a compiled version of the script remains in the repo as well.


Keyboard shortcuts don't work unless I explicitly source ~/.bashrc

Category

[ ] fzf binary
[ ] fzf-tmux script
[X] Key bindings
[ ] Completion
[ ] Vim
[ ] Neovim
[ ] Etc.

OS

[ ] Linux
[X] Mac OS X
[ ] Windows
[ ] Etc.

Shell

[X] bash
[ ] zsh
[ ] fish

I did
$ brew install fzf
==> Downloading https://homebrew.bintray.com/bottles/fzf-0.15.5.el_capitan.bottle.tar.gz
Already downloaded: /Users/pt/Library/Caches/Homebrew/fzf-0.15.5.el_capitan.bottle.tar.gz
==> Pouring fzf-0.15.5.el_capitan.bottle.tar.gz
==> Caveats
To install useful keybindings and fuzzy completion:
  /usr/local/opt/fzf/install

To use fzf in Vim, add the following line to your .vimrc:
  set rtp+=/usr/local/opt/fzf
==> Summary
🍺  /usr/local/Cellar/fzf/0.15.5: 17 files, 3.5M
$ /usr/local/opt/fzf/install
Downloading bin/fzf ...
  - Already exists
  - Checking fzf executable ... 0.15.5
Do you want to enable fuzzy auto-completion? ([y]/n)

Do you want to enable key bindings? ([y]/n)

Generate ~/.fzf.bash ... OK
Generate ~/.fzf.zsh ... OK

Do you want to update your shell configuration files? ([y]/n)

Update /Users/pt/.bashrc:
  - [ -f ~/.fzf.bash ] && source ~/.fzf.bash
    - Already exists: line #112

Update /Users/pt/.zshrc:
  - [ -f ~/.fzf.zsh ] && source ~/.fzf.zsh
    - Already exists: line #3

Finished. Restart your shell or reload config file.
   source ~/.bashrc  # bash
   source ~/.zshrc   # zsh

Use uninstall script to remove fzf.

For more information, see: https://github.com/junegunn/fzf

and then it works great in that window. If I reopen it thought, when I press CTRL + R it just puts
$ `__fzf_history__`

as my terminal. If I press Enter it works and brings up a list, but I select one I get:
$ `__fzf_history__`
-bash: !884: command not found

The same thing happens for CTRL + T.
If I type source ~/.bashrc in that same window, now everything works again. Is something you are doing expecting me to source bashrc instead of just doing whatever bash normally does?

If I type source ~/.bashrc in that same window, now everything works again. Is something you are doing expecting me to source bashrc instead of just doing whatever bash normally does?

No. If you start a new shell, ~/.bashrc is automatically loaded, so you don't have to do it manually. Close your terminal emulator completely and restart it and see if it helps.

On OSX you typically need to source ~/.bashrc from ~/.bash_profile
http://apple.stackexchange.com/a/13019
http://hayne.net/MacDev/Notes/unixFAQ.html#shellStartup

@junegunn yes, that is exactly what is broken. The latter part of my report has

If I reopen it thought...

so opening a new terminal is where fzf isn't working.
@mjwestcott I do have that:
$ cat ~/.bash_profile
if [ -f ~/.bashrc ]; then
  . ~/.bashrc
fi

What I don't understand about your initial report is

and then it works great in that window.

It shouldn't, because the code is not yet loaded on the shell. Maybe you missed the step where you sourced the file? Anyway, I think some other part in your configuration files are interfering with fzf key bindings. Can you post the output of bind -p? You should be able to find the following entries:
"\er": redraw-current-line
"\e^": history-expand-line
"\e\C-e": shell-expand-line

BASH on OSX Sierra (10.12) does not appear to source ~/.bashrc on startup. I can confirm that adding source ~/.bashrc to ~/.bash_profile resolves this issue.  This could be fixed by either modifying the install script to put [ -f ~/.fzf.bash ] && source ~/.fzf.bash in ~/.bash_profile instead on OSX, or updating the post-install instructions to instruct the user to source the ~/.bashrc file in their ~/.bash_profile if it isn't already.

@MuschPusch I'm not sure what's going on there. It seems like the bindings from fzf are overridden. Is there any set command after the line that loads fzf script?
Excerpt from https://github.com/junegunn/fzf#key-bindings-for-command-line

If you use vi mode on bash, you need to add set -o vi before source ~/.fzf.bash in your .bashrc, so that it correctly sets up key bindings for vi mode.

Does this help?

i source it as last file in my .bash_profile, so no it should be the last one

Last line in ~/.bashrc is
[ -f ~/.fzf.bash ] && source ~/.fzf.bash

Ctrl+R returns
`__fzf_history__`bash: fzf: command not found

and
bind -p |grep line |grep [e][r\^]
"\e^": history-expand-line
"\er": redraw-current-line
"\e\C-r": revert-line

and the suggestion to set -o vi does not work -- in addition it breaks the shell's "normal" behaviour, ie going back in history with the page up/down keys.

Sourcing of .fzf.bash works from within .bash_profile. Sorry if this issue is only OSX specific. I am on GNU bash, version 4.3.48(1)-release (x86_64-pc-linux-gnu).

@NikosAlexandris Related: https://github.com/junegunn/fzf/issues/282#issuecomment-116651487

@junegunn thanks. I am aware. In fact, I do have the following recommended lines in my .bash_profile:
# source: http://superuser.com/a/183980/128768
if [ -r ~/.profile ]; then . ~/.profile; fi
case "$-" in *i*) if [ -r ~/.bashrc ]; then . ~/.bashrc; fi;; esac

However, for some reason, in my setup, I still have not found, it doesn't work if fzf is sourced from .bashrc. Details in my previous posts.
ps- Kudos for the excellent work.

For anyone who comes here with a similar problem for zsh, I found that I needed to run the fzf scripts after the completion system initializes, and then it works.  If I ran the fzf stuff first, neither the key bindings or the expansion worked.

@jonbarksdale solution works for me.. just have to make sure fzf is loaded after the bash_completion.sh file and everythings working again.

Reading through the comments, I suspect different users are seeing slightly different problems, but anyway...  (@MuschPusch I think we're having the same issue.)
Arch Linux/Bash.  Re-sourcing ~/.bashrc sometimes works for me - it's not consistent for some reason.  Key bindings always work if I'm logged into a virtual console.
I normally have vi mode enabled - I set it in ~/.inputrc, not ~/.bashrc.  @junegunn asked if any set commands were made after sourcing fzf's scripts - in my case there were not.  I'm beginning to suspect Bash reads ~/.inputrc last, preventing key-bindings.bash from figuring out how to configure itself.
If I also enable vi mode in my ~/.bashrc, any issues go away.  (I still enable vi mode in my ~/.inputrc for other programs that use Readline.)

For anyone who comes here with a similar problem for zsh, I found that I needed to run the fzf scripts after the completion system initializes, and then it works. If I ran the fzf stuff first, neither the key bindings or the expansion worked.

In my case (MacOS 10.14.6 with bash 3.2.7) key bindings are not problematic, but expansion did not work until I changed the order as described  by @jonbarksdale!
Relevant excerpt FROM my .bash_profile:
[ -f ~/.fzf.bash ] && source ~/.fzf.bash # <- placed here, **TAB does NOT work as advertised (CTRL+T and CTRL+R does work though)
 # homebrew bash completion
[[ -r "/usr/local/etc/profile.d/bash_completion.sh" ]] && . "/usr/local/etc/profile.d/bash_completion.sh"
[ -f ~/.fzf.bash ] && source ~/.fzf.bash # <- placed here, **TAB DOES work as advertised (CTRL+T and CTRL+R still works)

For clarity I tested with the [[ test ]] directly added to .bash_profile (as opposed to being sourced from .bashrc). The result - naturally - is the same…!
tve@TvE-iMac27:~$ bash --version
bash --version
GNU bash, version 3.2.57(1)-release (x86_64-apple-darwin18)

@junegunn asked if any set commands were made after sourcing fzf's scripts - also in my case there were none of these.
PS.: fzf = fantastic tool, I am so happy to have discovered it (!)


Add a feature to auto setup the repository on new user login

For new user logins, The ui should present an option for automatically setting up the notes repository. Also allow him to select existing notes repository manually if he wishes to.
The auto setup option should create a new repository in user's github account and store it as default notes repo for user.
This should enhance the user experience since otherwise new users would select irrelevant existing repository for storing notes or they would have to go to github account and create a new one and then come back to application and select it.

Done


[question] Questions regarding project aim

Not that gone to this source yet, so I hope I don't ask dumb things.
This is very cool and probably takes a way a lot of the gripes in the current implementation. I also think this can help compile times quite a lot.
As the memory model of arrow is stable. Do you think it would be possible to define traits for Arrow2 arrays and default arrow Arrays that make it possible to generalize the existing kernels between both crates? Or is that a ball and chain you don't want to have for this repo (which I can imagine).
And do you aim publishing to crates.io? :)

Thank you for your interest: great admirer of your work in Polars!
Also great questions and ideas. Regarding traits: technically, it should be possible. I think that we we would need to declare a fair bit of them; for example arrow Bitmaps are not equal to arrow2 bitmaps, and thus we would need a trait for them (e.g. when we read validities from arrays). Buffers are also different: in arrow every buffer is Buffer<u8>, but in arrow2 buffers are typed. Thus, a trait that returns a "buffer" is not easy to achieve (which is required when a kernel clones a buffer, e.g. casting).
We would also need to declare them somewhere: on arrow or arrow2 or a third repo, arrow-traits, and then implement them on both sides. If we place them in the arrow repo, this crate now depends on a large dependency (arrow). The traits' release cycle is also tied to arrow release cycle. If we place it here, then arrow would need to depend on arrow2. If we place it on a separate repo, who manages and takes decisions over it?
My wish is that we would move this to the Apache Arrow project to an separate repo and release cycle independent of everything else (like a normal crate).
Note that this crate's high-end API is very similar to arrow, thus, in terms of migration, it should not be a major issue. I know exactly what to do to migrate DataFusion to it. If you think it is worth, I would gladly take a shot at working it out in Polars.
My plan was to have parquet support before doing any work on integration as IMO parquet IO is a major feature missing here and any work would be blocked by that.
It happens that parquet has its fair share of non-trivial transmutations; I am currently evaluating whether there is a way out of that. I do have an experimental branch that supports primitive types here using parquet crate, but I would prefer to avoid depending on a crate which I sense may have some risk of buffer overflows and/or out of bound reads (which IMO are direct CVE material).
I could also finish the parquet implementation and deal with the safety and performance later. Any suggestions?

And do you aim publishing to crates.io? :)

Nothing stopping us. My idea was to use pre-1 semver 2.0 here, 0.X.Y where X represents backward incompatibility and Y is minor and patch (backward compatible).

Thank you for your interest: great admirer of your work in Polars!

Thank you, I think I can say the same because I learned a lot from the Arrow project. :smile:

My wish is that we would move this to the Apache Arrow project to an separate repo and release cycle independent of everything else (like a normal crate).

That would be great indeed.  Another gripe is the slow release cycle, and no patches.
Regarding the traits.. Yeah I fully understand the complexities there. Maybe its better to stay free from those complexities. Currently you have a clean slate.

Note that this crate's high-end API is very similar to arrow, thus, in terms of migration, it should not be a major issue. I know exactly what to do to migrate DataFusion to it. If you think it is worth, I would gladly take a shot at working it out in Polars.

I would be very thankful for any advice or help making Polars ready for such a change. I believe that the complexity is in ChunkedArray<T> where T can be a PolarsPrimitive: ArrowPrimitive. So there is a dependency there.
I believe the separation of the logical part and physical part truly is worth the refactor. Currently, if I compile for all Series, compilation take very long due to all the duplicates in generics, e.g. Date32 being physically the same as i32.
whilst I am typing this I come to the conclusion that the beforementioned dependency should not matter, and could be abstracted away. Still I could learn from you separation of physical, / logical types!

I could also finish the parquet implementation and deal with the safety and performance later. Any suggestions?

Make it work, make it right, make it fast? :smile:

@ritchie46 , perfect.

I would be very thankful for any advice or help making Polars ready for such a change. I believe that the complexity is in ChunkedArray where T can be a PolarsPrimitive: ArrowPrimitive. So there is a dependency there.

So, the original arrow crate has a bunch of traits Native, ArrowPrimitive, ArrowNumeric, etc. They represent a combination of Logical and physical types (the const DATA_TYPE is the logical, its in-memory representation the physical) and other numeric that are not really needed.
Arrow2 has one type, NativeType, for primitive arrays. It represents any byte-like type that can be represented in Rust (and thus can be interpreted from bytes). All native Rust types that are part of the Arrow spec implement NativeType.
The consequence is that matches over DataType are always done to differentiate:

physical representations
physical operations

as either of them inevitably generate a different assembly / set of instructions.
In practice, this amounts to
match array.data_type() {
     DataType::Boolean => array.as_any().downcast_ref::<Boolean>().unwrap() ...
     DataType::Date32 | Time32(_) => array.as_any().downcast_ref::<PrimitiveArray<i32>>().unwrap(); op specific for any date32 or time32
     DataType::Date64 => array.as_any().downcast_ref::<PrimitiveArray<i64>>().unwrap(); op specific for date64
     DataType::Timestamp(_, _) => array.as_any().downcast_ref::<PrimitiveArray<i64>>().unwrap(); Op that applies to all time units
}

I.e. no more "TimestampMillisecondArray" and the like; they are all PrimitiveArray<i64> (since their physical representation is the same and thus the assembly code is the same, their only differ in the DataType). There is a map between a logical type and a physical type, which I placed as part of the guide.

Make it work, make it right, make it fast? 😄

Roger roger :)
I was able to re-write all functionality without using unsafe and AFAIK without jeopardizing performance, which gives me confidence that things can be indeed done in a safe manner. I can already read to native rust (Vec<Option<i32>>). I am now writing the logic to map parquet's representation directly into arrow.
I have not started writing the parquet writer. Is this something already supported by Polars? If yes, then it is something I also need to work, to ensure feature parity.

@paddyhoran , thank you for your comment and points. I understand that sentiment, which, perhaps paradoxically, I also share.

The release process was discussed a few times on the mailing list and don't think there is anything stopping us (Rust) releasing more frequently, although in practice it's cumbersome.

Patch releases are still tied to all implementations. We still release a major version every X months with all others, release all cargos at once, have no minor releases and a bunch of other practical problems that I raised on the mailing list.

Don't you think that publishing this crate and having people rely on it will create more confusion than anything and risk splitting the community?

but overall I think keeping the community intact is more important.

With +50k -40k code committed to the arrow repo, the Rust implementation of arrow and DataFusion on the main repo is something very dear to me. It has been my personal project over the past 8 months. Not wanting to merge this upstream for now was not a decision that I took lightly; it generated a fair amount of stress in my life over the past 2 months.
Arguing / pressuring that I should not release a Rust implementation of the Arrow specification outside of the Apache Arrow project at the expense of splitting the community is imo a bit unrealistic in an open source context. For me, it would be like asking @ritchie46 to donate Polars to Arrow because it otherwise splits the community around DataFusion and Polars.
IMO the community remains intact for as long as we respect and help each other, which imo has more to do with whether we listen and take our members' opinions into account. Where the code lives is typically a by-product of such behaviors. The fact that I am doing this should be interpreted as a consequence, not cause, of some of my frustrations, which I hope I was sufficiently transparent about in the mailing list.
If other projects are considering depending on this crate, IMO we should offer them the right tools (SemVer and a package manager). I am supportive of this because I know that the arrow crate has a serious soundness and security design problem that will only be addressed with a major redesign, and IMO it is more important to have projects in Rust relying on safe and sound implementations with good development and release processes than to have this code under the Apache Arrow repo. Others may of course disagree with the weights on the equation, and are welcome to address those security concerns within the arrow repo.
So, like I said,

My wish is that we would move this to the Apache Arrow project to an separate repo and release cycle independent of everything else (like a normal crate).

However, given the responses I got in the mailing list over these concerns, it is not something in my top list of priorities atm. I am more concerned in writing an implementation for reading and writing parquet that is safe and fast so that arrow2 has the same feature parity as arrow with a design that leverages Rust's compiler for checking memory-management invariants.

Arguing / pressuring that I should not release a Rust implementation of the Arrow specification outside of the Apache Arrow project at the expense of splitting the community is imo a bit unrealistic in an open source context. For me, it would be like asking @ritchie46 to donate Polars to Arrow because it otherwise splits the community around DataFusion and Polars.

I'm not trying to pressure you into anything.  It's unfortunate that an Arrow PMC is working on an alternative implementation outside the main community (as defined within Apache) though.  I disagree with your comparison to Polars but I do appreciate and understand the point to you trying to make.

and IMO it is more important to have projects in Rust relying on safe and sound implementations with good development and release processes than to have this code under the Apache Arrow repo

You are mentioning "safe and sound implementations" but there is no disagreement here.  I don't think anyone in the community is married to the current implementation (though I probably should not speak for @andygrove, @alamb, @nevi-me, etc.) and all of us would trust you to refactor the current implementation.  I believe your main issues are with JIRA and the release process.  Again, I'm in agreement with you here, I just don't feel as strongly as you do :).
In short, I understand your reasons but I hope that you are able to upstream this implementation to the main arrow repo (I really like some of the changes you have made).

Thanks for the ping @paddyhoran and I'm excited to see some of the amazing work that @jorgecarleitao is doing here and I would very much like to see this upstreamed to the official Apache Arrow repo. I would love to be able to use this from DataFusion and Ballista. I think it could be game-changing in terms of performance and safety.
I'm nearly through the process of contributing Ballista back to Arrow and it has been a slightly painful/stressful process so if there is a plan to contribute this back then I would advise:

Make sure there are CI checks for ASL (not ASF) headers in the project
Make sure that all contributors are onboard with contributing this back and prepared to sign CLAs when the time comes (and maybe consider only accepting future contributions from those that already have Apache CLAs filed)

Given that I myself have gone off and developed something new in a separate repo, I obviously understand the appeal and the need to do this for major undertakings. You can't do this kind of project one PR at a time.
On the subjects of JIRA, release process, release versioning, monorepo, etc, I also would like to see changes here, and although we have made some progress (like voting on source releases for patch releases) I acknowledge there is more to do. I personally have not been able to find the time to help drive this forward so far but once Ballista is merged I will have more motivation to work on this.

I for one would be very supportive of getting some/all of this implementation back into the main arrow/rust implementation. I realize doing so is lots of work -- not just technically but also JIRA / process wise.
I don't especially enjoy doing the JIRA dance for the main apache repo, but my judgement is that the apache/arrow crate is the best, most full featured implementation of Arrow in Rust, and it has the biggest community and potential to attract new contributors, and so it is where I want to spend my time. As @andygrove  said, projects at different phases of their lives are best served by processes that tailored.
I truly hope some / all of the great innovation @jorgecarleitao  is doing in this repo can be used and appreciated by a wider community:  I think it will be under appreciated / under used if it stays as a separate implementation.
Anyhow, let me know what I can do to help. There is so much great work going on.

I.e. no more "TimestampMillisecondArray" and the like; they are all PrimitiveArray (since their physical representation is the same and thus the assembly code is the same, their only differ in the DataType). There is a map between a logical type and a physical type, which I placed as part of the guide.

Yes, I've read it, and it make total sense. Another benefit is the reduction of code bloat. The cast kernel is huge!

I have not started writing the parquet writer. Is this something already supported by Polars? If yes, then it is something I also need to work, to ensure feature parity.

Yes, it is, but I see you already are rocking that as well.. :smile:
With regard to the second discussion that has started on this thread. I believe that as long we are staying compatible with the arrow memory format, a crate that does semantic versioning, regular patch releases etc., is beneficial to the arrow community and acceptance.
At the moment, (unless you are in the main repo), being dependent on Arrow's official released Rust sources (e.g. crates.io) is quite hard. For Polars I started following github master branch, because I was afraid I couldn't get the refactors working otherwise. I believe that regular updated crates, with semantic versioning can make the whole Rust Arrow community bigger, instead of splitting it.


update React Docs Link in 'getting started with redux page'

What docs page is being added or updated?

Section:  How to Read This Tutorial
Page: this page

For Adding New Content
What kind of content category is this page (tutorial, how-to, explanation, reference)?
tutorial
Who is the intended target audience?
What knowledge are we assuming they have?
What are the intended results or takeaways from reading this page?
What is the most critical info they should learn?
For Updating Existing Content
What updates should be made to the page?
update the links to latest react docs
Do these updates change any of the assumptions or target audience? If so, how do they change?

assign it to me I'll update them

@markerikson i found some more outdated links so should i fix them in single PR or should i create separate PRs?

@MahendraBishnoi29 one combined PR, please - thanks!

@MahendraBishnoi29 one combined PR, please - thanks!

Alright 👍🏻