Repo count

Create a function in GitHub.py to get the total number of repositories that a user has

Create a function in GitHub.py to get the total number of repositories that a user has

Hey, just created the function get_total_repositories() in github.py, I have submitted a pull request to merge.

Create a function in GitHub.py to get the total number of repositories that a user has

Hey, just created the function get_total_repositories() in github.py, I have submitted a pull request to merge.

Can you check and authorize my pull request?

Thanks for your contribution❤


Proposal: authentication and authorization API for 1.0

Type of change

Documentation

Description
Proposal: authentication and authorization API for 1.0
Checklist

[x] Update/write design documentation in ./documentation/design
[ ] Write tests and make sure they pass
[ ] Update documentation
[ ] Check RBAC rights for Kubernetes / OpenShift roles
[ ] Try your changes from Pod inside your Kubernetes and OpenShift cluster, not just locally
[ ] Reference relevant issue(s) and close them after merging
[ ] Update CHANGELOG.md

@lulf @rgodfrey  I am considering the following change:

AccessControlService -> AccessControlPlan.
ServiceAdmins define one or AccessControlPlan (prod, uat, sit1) etc.  Messaging tenants will not be able to define these objects.
AccessControlPlan keeps its selector - this is used to assign a default AccessControlPlan to the MessagingTenant. However, a MessagingTenant may override that default using an accessControlPlanRef
For the use-case where a MessagingTenant needs to override settings (say a host), the MessagingTenant object would have an inline accessControlOverride object (structure follows that of the plan), that allows the user to override the required value(s).
The ability for the tenant to override the plan or fields within the plan would be controllable from the AccessControlPlan.

How does that sound?

@lulf @rgodfrey I am considering the following change:
1. AccessControlService -> AccessControlPlan.

I kind of like the former better, but either are ok with me.

2. ServiceAdmins define one or more AccessControlPlans (prod, uat, sit1) etc.  Messaging tenants will _not_ be able to define these objects.

I think they should still be possible to create these outside of the enmasse-infra namespace, as the 'service-operator' persona would need that ability. Or are you referring to the default RBAC roles?

3. AccessControlPlan keeps its selector - this is used to assign a default AccessControlPlan to the MessagingTenant. However, a MessagingTenant may override that default using an `accessControlPlanRef`

I'm also thinking this is probably the way we should go with plans as well. However, what should be the default if you have overlapping selectors for AccessControlPlans?

4. For the use-case where a MessagingTenant needs to override settings (say a host), the MessagingTenant object would have an inline `accessControlOverride` object (structure follows that of the plan), that allows the user to override the required value(s).

If the tenant, if granted permission by the system admin (making the tenant a service operator as well), have the ability to create their own AccessControlPlan, would that not cover this use case? Or is this to get finer grained overrides?

5. The ability for the tenant to override the plan or fields within the plan would be controllable from the AccessControlPlan.

How does that sound?

@lulf @rgodfrey I am considering the following change:
1. AccessControlService -> AccessControlPlan.

I kind of like the former better, but either are ok with me. Naming it Plan makes me think, should this have been part of the MessagingPlan?

I changed the name partly because the Service had the implication of the existence of a running entity that might consume resources.  I used the word Plan just to follow the convention from MessagingPlan and MessagingAddressPlan.
I don't think merging AccessControlPlan and MessagingPlan.  You might want to use MessagingPlan small in dev and sit, but to the two would need different access control configuration (dev ldap, sit ldap)..

2. ServiceAdmins define one or more AccessControlPlans (prod, uat, sit1) etc.  Messaging tenants will _not_ be able to define these objects.

I think they should still be possible to create these outside of the enmasse-infra namespace, as the 'service-operator' persona would need that ability. Or are you referring to the default RBAC roles?

I meant RBAC. I don't mean to restrict AccessControlPlans to enmasse-infra namespace.    The TenantAdmin won't be able to create/update/delete these objects.

3. AccessControlPlan keeps its selector - this is used to assign a default AccessControlPlan to the MessagingTenant. However, a MessagingTenant may override that default using an `accessControlPlanRef`

I'm also thinking this is probably the way we should go with plans as well. However, what should be the default if you have overlapping selectors for AccessControlPlans?

I think we just define a rule that says if overlapping, the alphabetically first resource will be used so that the system is deterministic.  I'd prefer to do this to the alternative which would be to make the referrers invalid with an "ambiguous plan ref".

4. For the use-case where a MessagingTenant needs to override settings (say a host), the MessagingTenant object would have an inline `accessControlOverride` object (structure follows that of the plan), that allows the user to override the required value(s).

If the tenant, if granted permission by the system admin (making the tenant a service operator as well), have the ability to create their own AccessControlPlan, would that not cover this use case? Or is this to get finer grained overrides?

The steer I understood from @rgodfrey was that we wanted to give the Tenant Admin the ability to selectively override values (such as hosts).    That said, there if the  Tenant Admin had the RBAC ability, there is no reason why it should not work.

5. The ability for the tenant to override the plan or fields within the plan would be controllable from the AccessControlPlan.

How does that sound?

@lulf @rgodfrey I am considering the following change:
1. AccessControlService -> AccessControlPlan.

I kind of like the former better, but either are ok with me. Naming it Plan makes me think, should this have been part of the MessagingPlan?

I changed the name partly because the Service had the implication of the existence of a running entity that might consume resources. I used the word Plan just to follow the convention from MessagingPlan and MessagingAddressPlan.

I admit I have been constantly fighting this confusion every time we have discussed this :) - to me "service" definitely triggers the impression of a distinct process (or service) rather than just a set of configuration.  I'm not totally happy with plan, because it is more of a config... but I prefer it to service :-)

I don't think merging AccessControlPlan and MessagingPlan would be good. You might want to use MessagingPlan small in dev and sit, but to the two would need different access control configuration (dev ldap, sit ldap)..

+1

2. ServiceAdmins define one or more AccessControlPlans (prod, uat, sit1) etc.  Messaging tenants will _not_ be able to define these objects.

I think they should still be possible to create these outside of the enmasse-infra namespace, as the 'service-operator' persona would need that ability. Or are you referring to the default RBAC roles?

I meant RBAC. I don't mean to restrict AccessControlPlans to enmasse-infra namespace. The TenantAdmin won't be able to create/update/delete these objects.

+1

3. AccessControlPlan keeps its selector - this is used to assign a default AccessControlPlan to the MessagingTenant. However, a MessagingTenant may override that default using an `accessControlPlanRef`

I'm also thinking this is probably the way we should go with plans as well. However, what should be the default if you have overlapping selectors for AccessControlPlans?

I think we just define a rule that says if overlapping, the alphabetically first resource will be used so that the system is deterministic. I'd prefer to do this to the alternative which would be to make the referrers invalid with an "ambiguous plan ref" message.

+1

4. For the use-case where a MessagingTenant needs to override settings (say a host), the MessagingTenant object would have an inline `accessControlOverride` object (structure follows that of the plan), that allows the user to override the required value(s).

If the tenant, if granted permission by the system admin (making the tenant a service operator as well), have the ability to create their own AccessControlPlan, would that not cover this use case? Or is this to get finer grained overrides?

The steer I understood from @rgodfrey was that we wanted to give the Tenant Admin the ability to selectively override values (such as hosts). That said, there if the Tenant Admin had the RBAC ability, there is no reason why it should not work.

Yeah - the RBAC mechanism is fine.  The per-field override is only required because of the current way the auth services are defined

5. The ability for the tenant to override the plan or fields within the plan would be controllable from the AccessControlPlan.

How does that sound?


Add a Marker with a popup , to store later in my database

Hello everyone , I would like to add a marker whenever i click on the image in my panorama , with a popup to enter the infos I want any help please

You can get the coordinates of a click as follows:
var viewer = pannellum.viewer(...);
viewer.on('mousedown', function(e) {
    var coords = viewer.mouseEventToCoords(e);
    var pitch = coords[0],
        yaw = coords[1];
    // Your code goes here...
});

Thank you Sir , but whenever i turn the image the marker is marker on the photo , so the photo is now filled with markers :/ I tried both mousedown and mouseup none of them work for me any help please . Thank yiu

I don't understand what you're saying. The code snippet I posted doesn't add any markers; it just gives you the mouse coordinates to do with as you see fit.

here is my code ,
viewer.on('mouseup', function(event) {
viewer.addHotSpot({"pitch":viewer.mouseEventToCoords(event)[0], "yaw":viewer.mouseEventToCoords(event)[1], "type":"info", "text":"infoText"});
});
so now it's pretty fine , I can add a marker when I click on the photo , but when I turn the photo looking for some points of interest to mark ,  ,, the marker is add too  , even I didn't do a single click .

You need to check the mouseup events too and only add the marker if the positions match.


🧠 Logic: 🐛 Fix consult/1 base64 decoding

Fixes #384.
To avoid breaking change on wasm uri handler, we decide to implement our own consult/1 predicate by adding the base64
decoding from the wasm response into this predicate and remove the decoding from the VirtualFile system.

Apologies for the delay on this matter. I have some concerns about the proposed approach.
I would have incorporated instructions for Base64 decoding in the URI itself, as it carries the protocol elements. IMHO, the responsibility for decoding should lie with the VFS rather than the consuming function (as initially proposed by #384). And we could consider to have Base64 decoding set as the default option for the considered URI, to avoid any breaking changes, I presume. as it stands, the current approach appears to be quite limiting. What are your thoughts on this?

@ccamel Yes I'm agree, I prefer to make the logical on URI that will hold this decoding information but we have choose this approach to avoid breaking change. Otherwise, make the decoding by default on this URI and add an option to remove the decoding should be a good compromise.
Before continue, which URI parameter could be used to deactivate encoding ? &options=no_decoding, &decoding=none, or something else ?

I would have incorporated instructions for Base64 decoding in the URI itself, as it carries the protocol elements. IMHO, the responsibility for decoding should lie with the VFS rather than the consuming function (as initially proposed by #384). And we could consider to have Base64 decoding set as the default option for the considered URI, to avoid any breaking changes, I presume. as it stands, the current approach appears to be quite limiting. What are your thoughts on this?

Well, I'm divided..
Regarding the Base64 at the VFS level, the need appeared using a consult loading sources from the objectarium smart contract which return the data in Base64, this seems to me far from the nominal case and it hardly justify this behavior on the VFS, in my opinion.
On the other side this is weird to have the Base64 decoding in the consult, but to prevent any limitations the implementation here supports both Base64 and raw data.
However, I'm not against going on the URI param way I you both prefer :)

Before continue, which URI parameter could be used to deactivate encoding ? &options=no_decoding, &decoding=none, or something else ?

One way to declare the intention of base64 decoding right in the URI would be: ?base64Decode=true. Or the opposite if the default behavior is systematic decoding: ?base64Decode=false.

:tada: This PR is included in version 5.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Installation issue in Ubuntu 20.04.3 LTS

Hello,
I am trying to install this feature in OpenFOAM v 1906 in Ubuntu 20.04.3 LTS,  I am facing the following issues:

Using pip install -r requirements.txt, I am getting the following error:

ERROR: spyder 3.3.6 requires pyqt5<5.13; python_version >= "3", which is not installed.
ERROR: spyder 3.3.6 requires pyqtwebengine<5.13; python_version >= "3", which is not installed.
ERROR: astroid 2.3.1 requires typed-ast<1.5,>=1.4.0; implementation_name == "cpython" and python_version < "3.8", which is not installed.
ERROR: astroid 2.3.1 has requirement six==1.12, but you'll have six 1.14.0 which is incompatible.
Installing collected packages: terminado, notebook, opt-einsum, protobuf, pybind11, pydot, pydotplus, qtconsole, scipy, scikit-learn, sklearn, Werkzeug, tensorboard, wrapt, termcolor, tensorflow-estimator, tensorflow
  Found existing installation: terminado 0.8.2
ERROR: Cannot uninstall 'terminado'. It is a distutils installed project and thus we cannot accurately determine which files belong to it which would lead to only a partial uninstall.

Could I use coanda install -r requirements.txt to rectify this issue, I am using Python 3.7.4 while coanda environment on.
2. At the end of step 2, during compiling using: g++ hello_tf.cpp -ltensorflow, getting the error:
/usr/bin/ld: /usr/local/lib/libtensorflow.so: .dynsym local symbol at index 1603 (>= sh_info of 1)
/usr/bin/ld: /usr/local/lib/libtensorflow.so: .dynsym local symbol at index 2710 (>= sh_info of 1)
/usr/bin/ld: /usr/local/lib/libtensorflow.so: .dynsym local symbol at index 2711 (>= sh_info of 1)

But while executing giving me the output: Hello from TensorFlow C library version 1.15.0

In running the step 1 of ML_RANS: python ML_Model.py gives me error:

Traceback (most recent call last):
  File "ML_Model.py", line 5, in <module>
    import tensorflow as tf

Could you please help.

Hello, Sorry for the multiple posts, for the issue 1 getting after pip install -r requirements.txt
I  managed to forcefully install the terminado using pip install terminado==0.8.3  --user --ignore-installed

Can you try running the same python script but uncomment keras.utils.plot_model(model, 'ml_model.png', show_shapes=True) - it looks like you don't have GraphViz on your machine.

Hello Romit,
Many thanks for your prompt reply and help, I commented it, and it worked. But in the last step when I am trying to the RANS case using the simpleFoam_ML in Testing, I am getting the following error:
2021-12-16 09:07:54.970500: I tensorflow/core/platform/cpu_feature_guard.cc:142] Your CPU supports instructions that this TensorFlow binary was not compiled to use: AVX2 AVX512F FMA
2021-12-16 09:07:55.013029: I tensorflow/core/platform/profile_utils/cpu_utils.cc:94] CPU Frequency:<PHONE_NUMBER> Hz
2021-12-16 09:07:55.014298: I tensorflow/compiler/xla/service/service.cc:168] XLA service 0x56211c521560 initialized for platform Host (this does not guarantee that XLA will be used). Devices:
2021-12-16 09:07:55.014346: I tensorflow/compiler/xla/service/service.cc:176]   StreamExecutor device (0): Host, Default Version
/* Value of relative step height (hh_) is */1.9
*** stack smashing detected ***: terminated

Could you please help further? Also if I wanted to run the LES case, what needs to be done, since in the README.md pdf it is mentioned that work in progress.

Hmm - I have never seen that error - it may be related to the use of OpenFOAM v 1906 instead of OpenFOAM 5. I believe there were some other issues (although they never had that specific error).

To have a fresh start, I tried to repeat the instructions, and instead of installing TensorFlow C library version 1.15.0 I tried to install the TensorFlow C library version 2.7.0-dev20211101, thereby I am getting no warning while executing g++ hello_tf.cpp -ltensorflow. Now, while trying to generate the data using python training_data_maker.py I am getting the following error:
Velocity reader file
Traceback (most recent call last):
  File "training_data_maker.py", line 58, in <module>
    U = read_vector(f'U_{i}')
  File "training_data_maker.py", line 32, in read_vector
    file = open(filename,'r')
FileNotFoundError: [Errno 2] No such file or directory: 'U_1'

I guess instead of generating the data, Total_dataset.npy is already there in OF_Data, should I use that ? Inside the training_data_marker.py , is the code reads and writes NumPy error corresponding to the cases of varying height and not with varying velocities? What are the cx and cy scalars in the code? Sorry for the multiple queries, but wanted to understand so that I could implement this for our case study, hope you understand.

Yes the training dataset is already available in Total_dataset.npy for different heights. The cx, cy scalars correspond the x,y coordinates in the computational domain.

Hello,  I tried again but unfortunately got a similar kind of error (a bit changed):
2021-12-19 08:27:05.455261: I tensorflow/core/platform/cpu_feature_guard.cc:151] This TensorFlow binary is optimized with oneAPI Deep Neural Network Library (oneDNN) to use the following CPU instructions in performance-critical operations:  AVX2 AVX512F FMA
To enable them in other operations, rebuild TensorFlow with the appropriate compiler flags.
/* Value of relative step height (hh_) is */1.9
*** stack smashing detected ***: terminated

Hi again, maybe I am not able to compile the ML turbulence model  correctly, in Step 3: Compile ML turbulence model , using wclean && wmake libso I am  getting the warnings/errors:
Making dependency list for source file tf_utils.C
wmkdepend: parse error while scanning 'tf_utils.H' ... perhaps missing a final newline
wmkdepend: parse error while scanning 'tf_utils.C' ... perhaps missing a final newline
Making dependency list for source file maketurbulentTransportModels.C
wmkdepend: parse error while scanning 'tf_utils.H' ... perhaps missing a final newline
wmkdepend: parse error while scanning 'maketurbulentTransportModels.C' ... perhaps missing a final newline

However, I managed to get the  ML_SA_CG.so file in $FOAM_USER_LIBBIN
Is this which could be causing the error? Any help/suggestions ?

Yes - this looks like a version issue - are you still trying this on OpenFOAM v 1906?

Yes, I am using OF v 1906, but if I remember correctly then I managed to work this utility perfectly using your help in https://github.com/argonne-lcf/TensorFlowFoam/issues/6
but I am not sure what had changed-- an only thing which I noticed had changed is the Ubuntu version from 18 to 20.

Yes, I am using OF v 1906, but if I remember correctly then I managed to work this utility perfectly using your help in https://github.com/argonne-lcf/TensorFlowFoam/issues/6
but I am not sure what had changed-- the only thing which I noticed had changed is the Ubuntu version from 18 to 20.

To start afresh, I am trying to re-install this utility. In step 2 of README.md inside TensorFlowFoam-master/ML_RANS, when I am trying to wmake  I am getting the following error:
Making dependency list for source file tf_utils.C
wmkdepend: parse error while scanning 'tf_utils.H' ... perhaps missing a final newline
wmkdepend: parse error while scanning 'tf_utils.C' ... perhaps missing a final newline
Making dependency list for source file maketurbulentTransportModels.C
wmkdepend: parse error while scanning 'tf_utils.H' ... perhaps missing a final newline
wmkdepend: parse error while scanning 'maketurbulentTransportModels.C' ... perhaps missing a final newline

Could you kindly help in this regard?


Use global paths in macro expansions

Closes #1058
Switching to global paths (the ones that start with ::) means the intended symbols can't be shadowed by user code except maybe by more advanced things like aliasing crates via extern.

I'm wondering if we can somehow lint for this?


add Instagram to learn footer and list social and modify learn page

Preview 1

Preview 2

Everything must matched hehe

PREVIEW on Learn Page

@tzarebczan and @jsigwart here my latest ideas

@tzarebczan and @jsigwart Whoop Thanks My auntie for merge my PR and also my Teacher was assigned my auntie to review.


#129 Remove empty space between trucks lineup and dealer locator section

Please always provide the GitHub issue(s) your PR is for, as well as test URLs where your change can be observed (before and after):
Fix #129
Test URLs:

Before: https://develop--vg-volvotrucks-us-rd--netcentric.hlx.page/
After: https://129-truck-lineup-space--vg-volvotrucks-us-rd--netcentric.hlx.page/

I would do:
.section:has(+ .v2-dealer-locator-teaser-container) {
    padding-bottom: 0;
}

So you can prevent more blocks, not only dealer locator.


Update to deprecate in readme.

As requested by GDS; this is no longer compliant/maintained and as such should be marked.
Checklist:

[ ] Documentation N/A
[ ] Tests N/A
[x] Ready to be merged

Deploy preview for govuk-react processing.
Building with commit 8ef1c9d5685efd17e8b810de994cc3bc531ccbaf
https://app.netlify.com/sites/govuk-react/deploys/5e67b60e3ae7380008da2653

As requested by GDS

more information on this please, who and what was requested?

no longer compliant
does not meet current gov.uk standards

Please cite

no longer maintained/deprecated

The project is open source and we're always open to valid contributions


Offline Functionality not working

Hi thanks for the great work here , but i was wondering about the offline functionality , i tried to download the commit where it was still implemented , tested it didn't work , i tried also the dev branch but couldn't figure out also , can you please help ?

Offline functionality is not completed yet. I'm facing few problems while working with database. Once I fix that, offline functionality should work correctly.

Thanks , for the reply , i actually tried to make it work and had to go few commits back , anyway i noticed that you had a problem with the favorites and watchList which depend on the tmdb login which was removed a while ago , anyway keep the good work man 👍


[ENH] Implemented SimpleImputer

Reference Issues/PRs
Fixes #2293

What does this implement/fix? Explain your changes.
Implements SimpleImputer class in aeon.transformations.collections, along with test cases.

Does your contribution introduce a new dependency? If yes, which one?

Any other comments?

PR checklist

For all contributions

[ ] I've added myself to the list of contributors. Alternatively, you can use the @all-contributors bot to do this for you.
[x] The PR title starts with either [ENH], [MNT], [DOC], [BUG], [REF], [DEP] or [GOV] indicating whether the PR topic is related to enhancement, maintenance, documentation, bugs, refactoring, deprecation or governance.

For new estimators and functions

[x] I've added the estimator to the online API documentation.
[ ] (OPTIONAL) I've added myself as a __maintainer__ at the top of relevant files and want to be contacted regarding its maintenance. Unmaintained files may be removed. This is for the full file, and you should not add yourself if you are just making minor changes or do not want to help maintain its contents.

For developers with write access

[ ] (OPTIONAL) I've updated aeon's CODEOWNERS to receive notifications about future changes to these files.

thanks but this is still not logically correct, or rather not how I would like it. I want mean/median interpolation to happen independently on each channel and I dont think it is doing that at the moment. Logically
n_channels = X[0].shape(0)
for i in range(len(X)):
    for j in range(n_channels):
           interpolate any missing in X[i][j]

Its obviously possible you can do it more efficiently without loops, but I think the outer loop is fine because it generalises the case for list and 3D array collections

I think the parameter axis=-1 takes care that we need the mean of every channel independently.

@all-contributors add @notaryanramani for code


splits like ogarul

There must be a way to change them without affecting the doubleplit

@PsycoYT Nope, sorry.


[LiveComponent] Tweak live collection rendering

Q
A

Bug fix?
yes

New feature?
yes?

Tickets
-

License
MIT

I like this a lot! Would you mind adding a small section to the docs with an example of overriding live_collection_widget (e.g. to render the add button first then the widget) and live_collection_entry_row. (e.g. to add a <div> around each row)?

Thanks! I will.

Thank you Gábor!

Thank you!


[core] Fix jss v10 types being used

Problem: <workspace_root>/package.json forces every jss version to v10 which has breaking changes compared to v9. However @material-ui/core still uses v9 at compile-time (and run-time if the stable styling solution is used).
Solution: use path mapping in tsconfig to resolve the version used in core to v9 while any other package remains on v10.
Closes #14040

@eps1lon When using<EMAIL_ADDRESS>next to<EMAIL_ADDRESS>you end up with a build, containing two different major versions of jss.
I have the feeling this is triggering double runtime css compilation, resulting in conflicting styling rules. For us, this has only negative impact in production builds where all code is compiled and minified.
Have you seen similar negative impact?

@Sewdn This sounds like you're not properly installing @material-ui/styles. You should end up with only one major version of jss in your actual bundle. Do you have a screenshot of the treemap of your build?
If you just bundle all of your node_modules/ then yes you end up with two versions. I don't think I've seen anybody do that. I'm mostly familiar with webpack and it should not bundle dependencies that are not used.

We're (still) using<EMAIL_ADDRESS>to create our (production) builds.
I can confirm that we end up with 2 different major versions. When I pin to<EMAIL_ADDRESS>we end up with one single major version of<EMAIL_ADDRESS>where we can avoid the conflicting styling rules.
So I can only assume we have an issue with tree shaking when using Meteor to create our production builds: https://github.com/meteor/meteor-feature-requests/issues/206, since Meteor probably will not resolve to a single dependency on JSS when 2 major versions are referenced...
when using<EMAIL_ADDRESS>git:(develop) ✗ yarn list | grep jss@
│  ├─<EMAIL_ADDRESS>│  ├─<EMAIL_ADDRESS>│  ├─<EMAIL_ADDRESS>│  ├─<EMAIL_ADDRESS>│  │  ├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>
after pinning to<EMAIL_ADDRESS>git:(develop) ✗ yarn list | grep jss@
│  ├─<EMAIL_ADDRESS>│  ├─<EMAIL_ADDRESS>│  ├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>

yarn list i.e. your node_modules/ is not what matters here. What should matter what ends up in the bundle your users have to download.


Find and replace "line break"

In other IDEs I used to simple copy a line break and paste it to the find text box. Then I could replace it with some other string.
I tried to do it with regex find using "\n" or "\r" and I had no luck either.
This is very helpful when creating db queries or csv files on the fly by replacing line breaks with strings like: '");' or whenever you detect  some char (like >) you can replace it with a breakline.
Thanks a lot!

Duplicate of https://github.com/Microsoft/vscode/issues/313

@Tyriar thanks - closing as duplicate of #313


Create Plaindrome.java

please add hacktoberfest  too on pull request  and hacktoberfest-acceepted on repo . It is saying excluded on my hacktoberfest dashboard


deleting files actually moves them to trash, is there a way to actually delete them completely?

deleting files actually moves them to trash, is there a way to actually delete them completely?

from https://github.com/private-it/flysystem-google-drive/issues/4

@Jossnaz I'll make an option to delete or into trash can be chosen.
Thanks! 👍

@Jossnaz "deleteAction" was added to the option specified by the third argument when creating the adapter.
$googleDrive = new Hypweb\Flysystem\GoogleDrive\GoogleDriveAdapter(
	new \Google_Service_Drive($client), // Client service
	'root',                             // Folder ID as root ('root' or Folder ID)
	[ 'deleteAction' => 'delete' ]      // Delete action 'trash' (Into trash) or 'delete' (Permanently delete)
);


SCSS no longer auto compiling

I had a number of EuroPython sprint participants tell me that SCSS was not automatically recompiling.

This might be due to changes introduced in https://github.com/pypa/warehouse/pull/4358.

After investigation @nanuxbe found that the version of gulp should be   3.9.1 instead of 4.0.0 due to a bug introduced in the newer version:
https://github.com/gulp-community/gulp-less/issues/283

That bug looks like it affects gulp-less and less.js, neither of which we use as dependencies.

I'm also experiencing this while editing Javascript files, seems the watch task is no longer triggered on file changes.

Something like:
diff --git a/Gulpfile.babel.js b/Gulpfile.babel.js
index b73aeaf0..3be0331a 100644
--- a/Gulpfile.babel.js
+++ b/Gulpfile.babel.js
@@ -17,11 +17,9 @@ import brotli from "gulp-brotli";
 import composer from "gulp-uglify/composer";
 import del from "del";
 import gulp from "gulp";
-import gulpBatch from "gulp-batch";
 import gulpCSSNano from "gulp-cssnano";
 import gulpImage from "gulp-image";
 import gulpSass from "gulp-sass";
-import gulpWatch from "gulp-watch";
 import gulpWebpack  from "webpack-stream";
 import gulpConcat from "gulp-concat";
 import gzip from "gulp-gzip";
@@ -465,19 +463,17 @@ gulp.task("dist", gulp.series(
 ));

-gulp.task("watch", gulp.series("dist", () => {
+gulp.task("watch", function(done) {
   let watchPaths = [
-    path.join(staticPrefix, "**", "*"),
-    path.join("!" + distPath, "**", "*"),
-    path.join("warehouse/admin/static", "**", "*"),
-    path.join("!warehouse/admin/static/dist", "**", "*"),
+    "warehouse/static/**/*",
+    "!warehouse/static/dist",
+    "warehouse/admin/static/**/*",
+    "!warehouse/admin/static/dist/**/*",
   ];

-  gulpWatch(
-    watchPaths,
-    gulpBatch((_, done) => { gulp.start("dist", done); })
-  );
-}));
+  gulp.watch( watchPaths, )
+    .on( "change", gulp.series("dist") );
+});

 gulp.task("default", gulp.series("dist"));

Seems to work, but I think it needs a debounce function added, if you save another file while the compile is in progress, you get an error.


When in debug REPL, tab key changes focus instead of indents

Does this issue occur when all extensions are disabled?: Yes

ersion: 1.74.2
Commit: e8a3071ea4344d9d48ef8a4df2c097372b0c5161
Date: 2022-12-20T10:26:09.430Z (1 day ago)
Electron: 19.1.8
Chromium: 102.0.5005.167
Node.js: 16.14.2
V8: <IP_ADDRESS>-electron.0
OS: Darwin x64 21.6.0
Sandboxed: No
Steps to Reproduce:

Put the cursor into the Debug REPL
Press the tab key. The focus will change to the bottom bar instead of adding an indentation

Is there any way to change this behavior so that the tab key adds an indent?

In chrome devtools, tab will tab out of the input when the cursor is on the first line, and insert a tab when it's on the 2nd or below line. Do you think this is sufficient for accessibility requirements @isidorn?
Or, should we have a mode like the editor that you have to toggle manually with ctrl+m?

@roblourens yes, I think that is a reasonable experience. An alternative idea -> only when the cursor is at the very end of the debug input content the tab would focus out.
But whatever we do I would love if we do it for all SimpleEditorWidgets so we have a consistent behaviour across all our input boxes.
fyi @meganrogge

Here is another surprising behaviors that I'll add here since it's related (but happy to make new issues if that is easier to track)

If you hit the up arrow to get your last statement, you can then hit command+down-arrow to get to the end of a multi-line statement. I thought that this would now put me into normal cursor editing mode where I could use up, down, left and right to navigate my command but that's not the case. If it hit up-arrow, the text in my REPL gets replaced with the previous command.

In Chrome, the behavior is different. If I hit up arrow, it places my cursor at the end of the inserted statement and I can use my up, down, left and right keys as normal unless I key all the way up to the top line and hit the up arrow. That UX feels more intuitive to me since my expectation is that if my cursor is, say, on line 3 of 5 and I hit the up arrow, the cursor will go to line 2 and not replace my statement with something else.

In short, if the REPL could have the same UX as chrome dev tools that would be helpful for muscle memory and for navigating these expectations of how the interface will react.

Hi, Just wanted to check in to see if it would be possible to have this added to a release target? I run into this issue frequently.

Just checking in to note that this would still be a useful feature. Another area where it doesn't match Chrome is the REPL area is very short on the bottom. It would be easier to use if you had a larger text area to type in, like how Chrome lets you immediately after your last returned statement instead of in a bar at the bottom. Any time you can spend on this would be much appreciated.


Added code for pyramid pattern from 46 to 55.

Included Pattern 27.

Related Issues -
Issue Number - #58
Issue Number - #59
Issue Number - #60
Issue Number - #61
Issue Number - #62
Issue Number - #63
Issue Number - #64
Issue Number - #65
Issue Number - #66
Issue Number - #67
Sample Outputs -
Pyramid Pattern 46-

Pyramid Pattern 47-

Pyramid Pattern 48-

Pyramid Pattern 49-

Pyramid Pattern 50-

Pyramid Pattern 51-

Pyramid Pattern 52-

Pyramid Pattern 53-

Pyramid Pattern 54-

Pyramid Pattern 55-

@aryashah2k Please review this PR.


Hvplot yaxis range not updating with groupby and datashader=True

In Hvplot when I do a groupby='variable' and datashade=True, the y-axis is not updated, when i select a different value in the dropdown.
See also: https://stackoverflow.com/questions/58182484/y-axis-is-not-updated-on-dropdown-selection-when-i-use-datashader-in-hvplot
@jbednar suggested to use .opts(framewise=True). The effect is now that when I switch via the dropdown from col1 to col2, the y-axis range is updated, but when I switch back from col2 to col1, the y-axis is not updated.
Example code:
# import libraries
import numpy as np
import pandas as pd

import hvplot
import hvplot.pandas

import holoviews as hv
hv.extension('bokeh')
from holoviews.operation.datashader import datashade, dynspread

# create sample data
col1 = np.random.normal(loc=0.0, size=[50, ])
col2 = np.random.normal(loc=300.0, size=[50, ])

df = pd.DataFrame(data={'COL1': col1, 'COL2': col2}).melt()

# argument datashade=True causes that y-axis isn't automatically updated, which I would like it to be, so i added framewise=True but now the y-axis is updated only once
df.hvplot(
    kind='scatter',
    x='index', y='value',
    groupby='variable',
    dynspread=True, datashade=True
).opts(framewise=True)

I have the following packages:
hvplot 0.4.0
holoviews 1.12.5
datashader 0.7.0
jupyterlab 1.1.1

I can reproduce this issue on current master of the relevant projects, in Jupyter:
hvplot=0.4.0.post49+g6b12482
holoviews=1.13.0a4.post2+gae695356b
datashader=0.7.0.post19+ge42694b
bokeh=1.3.4

The problem here is that the RangeXY stream persists the previously set range. I don't have a good solution here yet but this isn't really an hvPlot issue.


Reasoning behind no_log: true

What was / is the reasoning behind this line, setting no_log: true as default. My playbook was failing at this step, and it wasn't clear why this line was not being not logged or by setting CI=true it allowed me to overwrite. For me it was failing because I had mistyped a config value.
https://github.com/prometheus-community/ansible/blob/4af5c9bd3092d64b3e284fd5b9d044893f31a323/roles/alertmanager/tasks/configure.yml#L20
Proposed Solution
I'd like to move the validate step to a different step so it is really clear if the validation is failing, I understand that we'd don't want to log secrets to the log but this seems like a good halfway step.

As you correctly assumed this is to avoid leaking credentials to the logs.

But I could be wrong, how do you propose we'd do it in a dedicated task?

I was thinking we could use the shell task to validate and fail if the validate failed.
I'm happy to contribute a PR but wanted to know if that's something would be well received.

Any task that fails may print the variables passed to it, depending on the verbosity and callback plugin settings. Even if we separate the validation into two tasks—one for the actual validation configured with failed_when: false, and another to determine if the validation should be marked as failed—there is still a risk of credentials being leaked if the first task fails due to a connection error.
In my opinion, the best approach would be to create an Ansible module for the validation. This way, we can mark sensitive parameters as secret.

Sorry if I wasn't clear, I think having a separate task that looks similar to the following:
- name: Validate AlertManager variables
   shell: "{{ _alertmanager_binary_install_dir }}/amtool check-config %s"
   nolog: true

Thinking about it maybe renaming the current task could be useful, currently when it fails it could be for a number of reasons, does the file have space, is it a permission issue. Separating these out makes it clear why the task is failing rather than a generic 'Copy alertmanager config'.
A separate module could be a good idea, although I'm not sure where it would fit.


"File to import not found or unreadable" using relative imports

Hi, as mentioned in the title, I'm getting that error while importing a library from  node_modules having the following folder structure:
asset/
└── scss
    └── app.scss
node_modules/
└── boostrap
    └── scss
        └── boostrap.scss

app.scss's content:
@import "../node_modules/bootstrap/scss/bootstrap";

I can use @import "node_modules/bootstrap/scss/bootstrap"; but then the autocomplete in the IDE will not work anymore.
Command: node-sass asset/scss/ --output public/css/ --output-style expanded --source-map-embed --source-map-content.
Full error:
{
  "status": 1,
  "file": "/srv/asset/scss/app.scss",
  "line": 1,
  "column": 1,
  "message": "File to import not found or unreadable: ../node_modules/bootstrap/scss/bootstrap.",
  "formatted": "Error: File to import not found or unreadable: ../node_modules/bootstrap/scss/bootstrap.\n        on line 1 of asset/scss/app.scss\n>> @import \"../node_modules/bootstrap/scss/bootstrap\";\n   ^\n"
}

NPM version (npm -v): 6.4.1
Node version (node -v): v10.13.0
Node Process (node -p process.versions):

{ http_parser: '2.8.0',
  node: '10.13.0',
  v8: '6.8.275.32-node.36',
  uv: '1.23.2',
  zlib: '1.2.11',
  ares: '1.14.0',
  modules: '64',
  nghttp2: '1.34.0',
  napi: '3',
  openssl: '1.1.0i',
  icu: '62.1',
  unicode: '11.0',
  cldr: '33.1',
  tz: '2018e' }

Node Platform (node -p process.platform): linux
Node architecture (node -p process.arch): x64
node-sass version (node -p "require('node-sass').info"):

node-sass	4.9.4	(Wrapper)	[JavaScript]
libsass  	3.5.4	(Sass Compiler)	[C/C++]

npm node-sass versions (npm ls node-sass): `--<EMAIL_ADDRESS>

Your relative path is only going up one level rather than 2 to get to the root of your code folder.
EX: @import "../../node_modules/bootstrap/scss/bootstrap";

Use https://github.com/sass/dart-sass instead of https://github.com/sass/node-sass resolve this problem for me.


Support sqlalchemy  >= 1.4.0

Is your feature request related to a problem? Please describe.
sqlalchemy 2.0 has breaking API changes that can be previewed in 1.4. The highest version of sqlalchemy supported by ptyhon-spanner-sqlalchemy is 1.3.23. Currently when using python-spanner-sqlalchemy, users must use the deprecated API, because 1.3.23 does not support the new API.
Describe the solution you'd like
python-spanner-sqlalchemy supports a version of sqlalchemy >= 1.4.0
Describe alternatives you've considered
Not sure

@aderk I think this is a reasonable request. Based on #105, @ilya started work on #123 as this was something we wanted to support but we then refocused our efforts on GA'ing the support that we had already built.
We will certainly consider continuing this work next year, just need to figure out priorities with respect to some other projects as well.

The PR which prepared a test suite for SQLAlchemy 1.4 is now merged. All the tests passed, so I guess SQLAlchemy 1.4 support is implemented.
Closing the issue, feel free to open in case something's acted up during migration to 1.4 version.

setup.py limits the version to < 1.4.0, can this version restriction be changed to allow 1.4.*?
https://github.com/googleapis/python-spanner-sqlalchemy/blob/main/setup.py#L24


Support text hyperlinks

Feature
Allow users to link objects to a URL
UX proposal
Option 1
I thought the UX could be something similar to what happens in Miro:

Click on the object
Show a toolbar with a link icon (overtime more formatting options could be added to the toolbar)
Click the link icon
Show input box to write the URL and make the link

As in Miro, I would just allow to link text objects or objects with text inside.
Option 2
Less work and still nice: show the linking functionality as part of the set of actions shown in the object panel.

Click on object
Click on link action icon in panel
On the panel show an input box to write URL as well as showing "link" and "unlink" buttons
If you click the link icon again, it hides the input box and "link" and "unlink" buttons.

@dwelle @lipis I am planning to allocate some time to work on this. Do you have any suggestions or comments on the proposed implementations?

My thinking is this should be part of #1126, and only allow for making selected ranges into hyperlinks, not whole elements (you could always select all the text).
Reasoning:

We want to keep UI minimal, and as the features pile on we'd quickly end up with a complex UI resembling Photoshop.
To battle this, I'd hide specific use cases under context-sensitive actions, in this case by selecting text when editing it. When you did that, a floating toolbar (as you suggested in (1)) would open, where you could change color (#1126) and potentially other properties, such as background (#1317), or make the range into a hyperlink.
As such, most of the UI complexity would be hidden until after you trigger it by a specific action (and not just by selecting a text element by clicking on it, where I'd argue the common use would simply be to move it around).
As for discoveribility: when you edit an existing text, we by default select the whole text input which means the floating menu would appear and thus users would discover this feature sooner or later, by themselves.

/cc @kbariotis

Yes! Indeed, with #1126 that top toolbar will come out of the box, all we have to do is enable it. Please feel free to fork that and experiment, lots of work has been done there @paufabregat.
@dwelle correctly pointed out a few issues we will face in the process but an experiment I think will surface all of them.

Since #1126 is on hold for now, I believe this issue shouldn't be blocked on that.
In other words, it should work atomically on whole text elements (or potentially other elements/groups).
That said, we need to figure out how it'll work with text element editing. Should a click on hyperlinked text go to link directly, or should it required -click?

if with a modifier, which one. Cmd/Ctrl-click and Alt-click are both taken, kinda.
if without a modifier, how will you be able to select the text element so that you can edit it?

I would suggest clicking without a modifier will send users directly to the link, boards have more people viewing them than editing and many viewers will not think to use a modifier and some will be confused by the idea.
Selecting and editing can use a modifier. I think the Macro/Win/Cmd key is unmapped so far?

If we go with modifier-to-edit, then Cmd/Ctrl-click could be used for single elements. It's currently taken by exclusive-select action, so it wouldn't work for hyperlinked groups (if we support that).
Apart from Cmd (mac)/Ctrl (win) (those are identical), Option/Alt, Shift, we can't use any other modifier. Mac also has Control but it has no matching key on Windows (Ctrl on Windows is what Cmd is on Mac).

Probably best to start with what Figma does: on click/hover show a popup:

Click-to-visit only works for people which do not have edit permission, or in presentation mode (neither of which applies to excalidraw yet so this part wouldn't be implemented yet).
Safest solution.

hi,
an other possibility (i means complementary) is to drag and drop the URL on the object. With a shadow on the object you select.

It would be good if more than just text was able to be made in a hyperlink. In practice you probably want some text to be included, but having graphical elements be part of the clickable link could be valuable as well

Hi
How can I delete the link information. I removed the diagram but when I open the shared link the diagram appears. Is there any way to delete the link?

@aygulsibel not possible from within the app yet. But you can send us an email on support at excalidraw.com and we'll remove it manually. Be sure to send the id portion of the link (no need to send the key): https://excalidraw.com?json=id,key

Hi David,
Id is Ct5XpLLK_86i1ugx0JuSj
Could you please remove the link.
Thanks
On Mon, 20 Dec 2021 at 15:12, David Luzar @.***> wrote:

@aygulsibel https://github.com/aygulsibel not possible from within the
app yet. But you can send us an email on support at excalidraw.com and
we'll remove it manually. Be sure to send the id portion of the link (no
need to send the key): https://excalidraw.com?json=id,key
—
Reply to this email directly, view it on GitHub
https://github.com/excalidraw/excalidraw/issues/1205#issuecomment-997869280,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AW7UC63X4D7IJAWXPSRGD3LUR4MTPANCNFSM4L4LAH3Q
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Sibel Aygül

@aygulsibel done. Btw, next time please open a separate issue, send us an email, or a DM on twitter, thanks!

closing this as Hyperlinks support is available now https://twitter.com/aakansha1216/status/1489256535817854977

@ad1992 Many thanks for hyperlinks!
Is adding links to objects within the same drawing still on the table, as described here?

Is adding links to objects within the same drawing still on the table, as described here?

Definitely!

@ad1992  Hey, thanks for adding hyperlinks!
I noticed when exporting to SVG the links are not getting exported. Any way this can get implemented?

Could you please remove me from the mail?
On Sat, 5 Feb 2022 at 08:14, exaktly @.***> wrote:

@ad1992 https://github.com/ad1992 Hey, thanks for adding hyperlinks!
I noticed when exporting to SVG the links are not getting exported. Any
way this can get implemented?
—
Reply to this email directly, view it on GitHub
https://github.com/excalidraw/excalidraw/issues/1205#issuecomment-1030567547,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AW7UC6YUAORRC2FCS3BI7UDUZTE43ANCNFSM4L4LAH3Q
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Sibel Aygül

@exaktly there's a tracking issue https://github.com/excalidraw/excalidraw/issues/4699
@aygulsibel since you subscribed on your own (when you commented), you'll have to do it yourself using the unsubscribe button on the right (but there should be one in the email as well)

Hyperlinks on shapes would be great. Any plans to add this?

hyperlinks on individual words would be great, Any plans to add this? :)

@825i this issue is about adding hyperlinks to text shapes which is supported. What you're likely asking about is adding links to individual word/sentences inside a text element, which falls under https://github.com/excalidraw/excalidraw/issues/6678. It will come, but it's not simple, lots of moving parts. We will be splitting it into sub-tasks and start working on it soon.


[Profile] Allow user to view consent document from Profile screen

When tapping the "View Consent Form" button on the Profile screen, the user should be shown a copy of the document they agreed to during onboarding.

Completed by @kellynguyvn


Update B2 env variable information

What is the purpose of this change? What does it change?
Fixed and updated the environment variables for Backblaze B2 to show the correct information + add the information that these variables can also be used for the newly introduces support for B2s Application Keys.

Was the change discussed in an issue or in the forum before?
No.

Checklist

[x] I have read the Contribution Guidelines
[ ] I have added tests for all changes in this PR
[ ] I have added documentation for the changes (in the manual)
[ ] There's a new file in changelog/unreleased/ that describes the changes for our users (template here)
[ ] I have run gofmt on the code in all commits
[ ] All commit messages are formatted in the same style as the other commits in the repo
[x] I'm done, this Pull Request is ready for review

Codecov Report

Merging #1975 into master will decrease coverage by 4.88%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1975      +/-   ##
==========================================
- Coverage   50.16%   45.27%   -4.89%
==========================================
  Files         170      165       -5
  Lines       13559    13516      -43
==========================================
- Hits         6802     6120     -682
- Misses       5733     6447     +714
+ Partials     1024      949      -75

Impacted Files
Coverage Δ

internal/backend/b2/b2.go
0% <0%> (-78.54%)
:arrow_down:

internal/backend/swift/swift.go
0% <0%> (-78.45%)
:arrow_down:

internal/backend/gs/gs.go
0% <0%> (-73.61%)
:arrow_down:

internal/backend/azure/azure.go
0% <0%> (-69.46%)
:arrow_down:

internal/backend/swift/config.go
36.95% <0%> (-54.35%)
:arrow_down:

cmd/restic/cmd_mount.go
12.98% <0%> (-44.16%)
:arrow_down:

internal/archiver/blob_saver.go
92.59% <0%> (-4.94%)
:arrow_down:

internal/checker/checker.go
64.54% <0%> (-3.87%)
:arrow_down:

internal/archiver/file_saver.go
84.21% <0%> (-1.76%)
:arrow_down:

internal/restic/node.go
39.01% <0%> (-1.3%)
:arrow_down:

... and 6 more

Continue to review full report at Codecov.

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


Support GCE google_compute_https_health_check (TF)

Description
Add GCE support for google_compute_https_health_check (TF)
Sources

Terraform Resource Documentation
GCP Resource Documentation

Same than for #994 - you're right, let's schedule this later based on user need & feedback


Use unsupported config map to force the cert rotation

/wip

/retest

/hold

/retest

/test e2e-snc

/retest

/test e2e-snc

/retest

/unhold

/retest

/test e2e-snc

/retest

I took the bundle from the CI and the bundle is really better.
find /etc/kubernetes -iname "*.crt" -exec /bin/sh -c "echo {}; openssl x509 -checkend 2160000 -noout -in  {}" \; | grep "will expire" -B1

returns only the kubelet serving cert to be expired. Compared to before it is a huge improvement.

/lgtm


[Add a Play]: Video Preview

What's the Play about?
User will search a video and when the user hovers over the video thumbnail for 2-5 seconds then user will see a preview of the video
What ReactJS concept will be used to create this play?
Consuming APIs to render images conditionally and to use timer efficiently in react js.
Are you willing to take it up for implementation?
Yes
Code of Conduct

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

Can I work on this one?

Hi @suzy-g38 I have picked this one up

Hi @suzy-g38 I have picked this one up

Sorry, I misunderstood. I really wanted to work on something, all the other issues were taken by someone and then saw this one and was not assigned to anyone so thought maybe this can be my chance at last. Anyway sorry.

Hey @suzy-g38 please don't be sorry. You can contribute your own play. Please check react-play website for once.

Hey @suzy-g38 join our discord. Lots going on there for you to contribute. Ping me on discord, I'll guide you.

Hey @suzy-g38 join our discord. Lots going on there for you to contribute. Ping me on discord, I'll guide you.

Hey, @atapas thank you. Yesterday I already joined the discord channel as Vizard. I have pinged you.

Hi @atapas, can you give me discord link please.

Hi @atapas, can you give me discord link please.

https://discord.gg/UfYj4MvW9A

Hey @suzy-g38 join our discord. Lots going on there for you to contribute. Ping me on discord, I'll guide you.

Hey, @atapas thank you. Yesterday I already joined the discord channel as Vizard. I have pinged you.

Pinged you there


fix issues in webhook use case

Address issues #296, including

use placeholder instead of real data in deployment.yml
use default namespace _ other than guest namespace in deployment.yml

I will close it and create a new PR.


DOCS: Require Developer Documentation for running Cypress Tests in local env development

Require Detailed documentation for running and integrating Cypress Tests during local Development.
Probably we can also add this under Developer Section in https://docs.open-metadata.org/

https://docs.open-metadata.org/developer/build-code-and-run-tests/cypress-integration-tests


[url_launcher] grey screen after return from a launched app

After launching to another app
await launch('youtube://www.youtube.com/watch?v=' + id,
            forceSafariVC: false);

and when you come back the screen is just grey

Hi @EliasDeuss
Could you please provide more information and flutter doctor -v and code simple to reproduce the issue.
Thank you.


Cvebin2vex plugin

Summary
This PR is for plugin enhancement #170
If merged this pull request will add a plugin that leverages the cve-bin-tool (https://github.com/intel/cve-bin-tool) and a custom conversion process to generate OpenVEX vulnerability statements from binary files. This plugin supports analyzing binary strings to identify known vulnerabilities and outputs them in the standardized CycloneDX and OpenVEX format that is added to the "(sha256hash)_additional_metadata.json" file.
Proposed changes
Add the cvebin2vex plugin as well as a minor update to how the binary2string and angrimportfinder plugins search for the "_additional_metadata.json" files.
This adds the cve-bin-tool as a required dependency for this plugin.

I had a couple errors while trying out the plugin. Not sure if others ran into the same thing.
First, I had to change the dnfile version to 0.14.1 since the current one seems to break surfactant. Then, ran pip install -e ..
Then, when running cve-bin-tool --update now I got errors saying ERROR    cve_bin_tool - InsufficientArgs: Please specify a directory to scan or an input file required.
Reran that command again and got a warning that said  WARNING  cve_bin_tool.CVEDB - Updating cachedir /Users/tenzing1/.cache/cve-bin-tool. Since it said updating cachedir, that seemed like the update command worked. But when that stopped running, got the same ERROR    cve_bin_tool - InsufficientArgs: Please specify a directory to scan or an input file required.

I tried again running surfactant again,

It did create a lot of ...metadata.json files.
Let me know if there's anything else I need to try. I've tried with the ShareX portable zip file and with the HELICS linux tar gz file.

Everything is working with the new commits! No changes on my end.
The only thing remaining is the one failed precommit check --related to inconsistent return statements:
plugins/cvebin2vex/surfactantplugin_cvebintool2vex.py:14:0: R1710: Either all return statements in a function should return an expression, or none of them should. (inconsistent-return-statements)
Otherwise, ready to merge!


Named secrets profile support

@ryanjbaxter ready for review. Documentation will be added when I add support for profiles for labeled secrets also, in my next PR.

Yeah, it will make sense when I open my next PR :)


removed extra blank line from this file.

Checklist:

[ ] I have read freeCodeCamp's contribution guidelines.
[ ] My pull request has a descriptive title (not a vague title like Update index.md)

Closes #XXXXX

Thank you for opening this pull request.
This is a standard message notifying you that we've reviewed your pull request and have decided not to merge it. We would welcome future pull requests from you. If you have not done so, please read the contributing documentation at contribute.freecodecamp.org.
Thank you and happy coding.


Typo in Changeset 'optimistic locking' example code

The example code for optimistic locking in Changesets assigns a changeset to a variable named 'stable_change':
iex> stable_change = Post.changeset(:update, post, %{title: "baz"})
Three lines later, it attempts to update the Repo, but refers to a 'stale_change' variable, which does not exist:
iex> Repo.update!(stale_change)
The variable should probably be changed from 'stable_change' to 'stale_change'.

@JohanVanZyl perfect. could you please send a pull request? THank you!

Sorry about that, it seems to have been fixed already. I was looking at hexdocs v1.1.5

evenbetter, thank you!


Users can choose where to save the converted file

We settled eventually to save the file immediately in the same directory


Update threat-explorer.md

Updated real-time detections with Real-time detections

@JohanFreelancer9 - kindly do the copy editor review for this PR. Thanks

@andypunt Please implement the review comments. Thanks.


fix(steps): purge dropped categories from metadata

Builds on #74
Previously, the final step used the ml.everything() selector, because ml.numeric() was erroneously not picking up encoded columns (former categories).

Codecov Report
Attention: Patch coverage is 73.91304% with 6 lines in your changes are missing coverage. Please review.

Project coverage is 84.29%. Comparing base (35eb9c2) to head (790cabc).

Files
Patch %
Lines

ibisml/__init__.py
64.28%
5 Missing :warning:

ibisml/steps/encode.py
83.33%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #76      +/-   ##
==========================================
- Coverage   84.39%   84.29%   -0.11%
==========================================
  Files          21       21
  Lines        1519     1541      +22
==========================================
+ Hits         1282     1299      +17
- Misses        237      242       +5

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


install apt-transport-https on debian 8 and debian 9

The package apt-transport-https does not look present on old docker images.
We have debian8 and debian9 jobs that are failing with message: The method driver /usr/lib/apt/methods/https could not be found.
Details :
https://github.com/voxpupuli/puppet-nodejs/pull/416

Can one of the admins verify this patch?

This is likely due to a regression in puppetlabs-apt 7.4.0 (https://github.com/puppetlabs/puppetlabs-apt/commit/f862889455c4c9994d0afc21a84715f4bc051a9c) when lsb-release is not installed. In that case the lsbdistrelease fact is missing and puppetlabs-apt no longer installs the dependency.
IMHO the images should only install the bare minimum

It was pointed out to me that the puppet-release deb has switched its URLs to https so Puppet itself wasn't getting installed. In that case, :+1: since it fixes a bootstrap issue.


Breaks on older split podlove feed

I've encountered an issue with this feed that spans multiple pages: https://omegataupodcast.net/feed/mp3/
It seems that the validation fails when trying to fetch the next feed page. Anyway, thanks for the great tool!

Fixed, thanks for reporting!

Thank you for fixing it


Question: optional/wildcard url parameter

Hi,
I would like to have a route with a param in it but not having to have to specify it in the method level, like:
Class ComponentList(Resource):
    def get(self):  # note that there is no some_random_string param here
        ...
api.add_resource(ComponentList, '/api/<some_random_string>/components')

For the moment I couldn't find any keyword in the code, * doesn't work either. Is it possible?

Hi @bastianb , fortunately there isn't an option to do so.
I would suggest to create a middleware that catches this endpoint and then returns your Resource response:

from flask_restful import Api, Resource
from flask import Flask, request

app = Flask(__name__)
api = Api(app)

class ComponentList(Resource):
    def get(self):  # note that there is no some_random_string param here
        ...
        return "my_response"

@app.route('/api/<string:ignored>/components')
def middleware(ignored):
    method = request.method.lower()
    if hasattr(ComponentList, method):
        return getattr(ComponentList(), method)()

if __name__ == '__main__':
    app.run(debug=True)

Notice that if you need a scaleable solution you can use decorator to the class


Jpeg quality

Dear Developers,
Can I set jpeg quality somewhere?
Thanks: Bela

No, this is currently not possible.
Would you like to contribute this feature? It should be as easy as:

Adding a setting and a command line option for the encoder quality value
Replacing null with EncoderParameters for the preview here and doing the same for the save here

Are you still considering implementing this feature?

Hi @eltos, are you still considering implementing this feature?

I won't implement this anytime soon myself, but I would accept a reasonable PR on the matter. The default quality (whatever that is) should be kept unless the user changes it.


mysql: small privileges refactor

What problem does this PR solve?
While fixing https://github.com/pingcap/tidb/issues/22703, I noticed two problems:

I need to check if a set of privileges has a specific privilege for several times. But we only have raw slices []PrivilegeType, which means I either write a loop or a function on my own - too verbose. The privileges verification package, however, is using bitmask(AllPrivMask) to do that - not consistent.
It is very annoying that TiDB str, ok := Priv2UserCol[v] followed by three lines of if !ok { return "invalid priv" }. We should use the global set, e.g.AllGlobalPrivs, to check if it is a valid privilege, then just use String() to get a string.

After checking code in parser/mysql, I decided to do some small refactors to:

Hide direct usage of all maps like AllXXXPrivs, or Priv2UserCol by invoking methods. Possibly replace maps with switch.
Make one bitset type for collections of privileges. Then executor and privileges packages in TiDB could share one type.

The logic will be more self-contained. And TiDB will get a consistent API and a less verbose solution for my fix on tidb#22703. However, the refactor will break the compilation of TiDB.
So this PR will only try to prepare for removing all direct accesses to maps. The refactor should be possible to be done in several PRs without breaking others' PR merging.
Check List
Tests

Unit test
Integration test

Code changes

Has exported function/method change
Has exported variable/fields change

LGTM

/lgtm

/merge


Allow no argument?

Sometimes the name of the concatenated file doesn't matter. Like:
var gulp = require("gulp");
var dox = require("gulp-dox");
var doxme = require("gulp-doxme");
var concat = require("gulp-concat");
var rename = require("gulp-rename");

gulp.src("**.js")
  .pipe(concat("anything"))
  .pipe(dox())
  .pipe(doxme())
  .pipe(rename("Readme.md"))
  .pipe(gulp.dest("."))
  ;

I understand "" can't ever be a valid filename. But undefined casts to "undefined", same with null. Why not allow them?
It shouldn't do any harm. When someone sees a file named undefined output to his gulp.dest, it's quite obvious what happened. Without throwing any error.

I think it's confusing and not good API design to default an argument to "undefined" - sorry :(

A year later: I agree!
:+1:


Consumer example not working

I'm trying to use latest pysher version, however the example in README.md is not working:
error from callback <bound method Connection._on_open of <Connection(PysherEventLoop, started daemon<PHONE_NUMBER>60096)>>: _on_open() takes exactly 2 arguments (1 given) error from callback <bound method Connection._on_message of <Connection(PysherEventLoop, started daemon<PHONE_NUMBER>60096)>>: _on_message() takes exactly 3 arguments (2 given) error from callback <bound method Connection._on_error of <Connection(PysherEventLoop, started daemon<PHONE_NUMBER>60096)>>: _on_error() takes exactly 3 arguments (2 given) Connection: Connection closed
Versions:
websocket-client==0.53.0
Pysher Version: 1.0.3
in addition, the example code misses imports of time and sys.

Fixed in 1.0.4 (via 8bec81616be2b6c9a9c0e2774b42890426a73745).


BUG: fix alignment in series ops (GH14227)

[x] closes #14227
[x] tests added / passed
[ ] passes git diff upstream/master | flake8 --diff
[ ] whatsnew entry

Current coverage is 85.24% (diff: 100%)

Merging #14230 into master will increase coverage by <.01%

@@             master     #14230   diff @@
==========================================
  Files           140        140
  Lines         50559      50561     +2
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits          43099      43102     +3
+ Misses         7460       7459     -1
  Partials          0          0

Powered by Codecov. Last update 5e2f9da...82fb2ab

@jorisvandenbossche can you update

@jreback I moved this to align (and therefore also fixing the bug in align), but it is still not yet that clean ...
So the problem for tz aware indexes is that Index.join does not returns indexers when the result does not need to be reindexed (although in case of tz aware indexes the actual indexes can be different and have to be adapted), and based on that the index of the result is adapted in align or not. So I still have to check for this specific case. I tried to solve it in Index.join itself, but that caused other errors.

if this works (and you want to push in 0.19.0), ok.. maybe just add a TODO / issue to fix/clean later.

Well, it was a hack before, and it is still the exact same hack now, only moved down the stack into align (fixing the behaviour of align as well), so merging for now.


CAPI cluster autoscaler (for MachineDeployment) possibly broken on k8s 1.21+

Reported by @ShaunakJoshi1407
On test environment, the CallHomeConfig custom resource is stuck in the retrying state. See error message in Status field:
$ kubectl -n capi-old-argo get callhomeconfig cluster-autoscaler -oyaml
apiVersion: core.arlon.io/v1
kind: CallHomeConfig
metadata:
  annotations:
    kubectl.kubernetes.io/last-applied-configuration: |
      {"apiVersion":"core.arlon.io/v1","kind":"CallHomeConfig","metadata":{"annotations":{},"labels":{"app.kubernetes.io/instance":"capi-old-argo"},"name":"cluster-autoscaler","namespace":"capi-old-argo"},"spec":{"kubeconfigSecretKeyName":"value","kubeconfigSecretName":"capi-old-argo-kubeconfig","managementClusterUrl":"https://<IP_ADDRESS>:42943","serviceAccountName":"cluster-autoscaler","targetNamespace":"kube-system","targetSecretKeyName":"kubeconfig","targetSecretName":"cluster-autoscaler-management-kubeconfig"}}
  creationTimestamp: "2022-09-12T17:12:10Z"
  generation: 1
  labels:
    app.kubernetes.io/instance: capi-old-argo
  name: cluster-autoscaler
  namespace: capi-old-argo
  resourceVersion: "15918"
  uid: fc0b3e12-f6a7-47f3-bb61-e50c621aa3bd
spec:
  kubeconfigSecretKeyName: value
  kubeconfigSecretName: capi-old-argo-kubeconfig
  managementClusterUrl: https://<IP_ADDRESS>:42943
  serviceAccountName: cluster-autoscaler
  targetNamespace: kube-system
  targetSecretKeyName: kubeconfig
  targetSecretName: cluster-autoscaler-management-kubeconfig
status:
  message: serviceaccount cluster-autoscaler does not have a token, retrying in 10
    seconds
  state: retrying

Inspecting the serviceaccount resource shows that it indeed has no Secrets:
$ kubectl -n capi-old-argo get sa cluster-autoscaler -oyaml
apiVersion: v1
kind: ServiceAccount
metadata:
  annotations:
    kubectl.kubernetes.io/last-applied-configuration: |
      {"apiVersion":"v1","kind":"ServiceAccount","metadata":{"annotations":{},"labels":{"app.kubernetes.io/instance":"capi-old-argo"},"name":"cluster-autoscaler","namespace":"capi-old-argo"}}
  creationTimestamp: "2022-09-12T17:12:10Z"
  labels:
    app.kubernetes.io/instance: capi-old-argo
  name: cluster-autoscaler
  namespace: capi-old-argo
  resourceVersion: "12943"
  uid: 2a5e72c2-363e-4397-babf-74be87a342fd

Looking at the latest Kubernetes documentation, it appears that the association between serviceaccounts and token secrets could have changed in v1.21+:
https://kubernetes.io/docs/reference/access-authn-authz/service-accounts-admin/#bound-service-account-token-volume
https://cloud.google.com/blog/products/containers-kubernetes/kubernetes-bound-service-account-tokens
Aha! Link: https://pf9.aha.io/features/ARLON-294

@bcle are you already looking into this or should I have someone else take this up?

I have not investigated beyond a cursory read of the above documents. I think it's a potentially tricky problem to solve. If someone in your team would like to own it, go for it. Otherwise I'll gladly take it.


Private init

Hi, first of all thanks you very much for this repo, I found it very useful. Just, I wonder if class constant approach could be incomplete. Isn't necessary a private init? with this current implementation you could create several references of the same class, with a private init you only could create one instance (the static property).
Thanks.

You are correct, in fact the .swift example is already updated. I made a pull request for this: https://github.com/hpique/SwiftSingleton/pull/14


AZ LOGIN failing

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az login
Errors:
local variable 'token_entry' referenced before assignment
Traceback (most recent call last):
python3.8/site-packages/knack/cli.py, ln 215, in invoke
    cmd_result = self.invocation.execute(args)
cli/core/commands/__init__.py, ln 654, in execute
    raise ex
cli/core/commands/__init__.py, ln 717, in _run_jobs_serially
    results.append(self._run_job(expanded_arg, cmd_copy))
cli/core/commands/__init__.py, ln 710, in _run_job
    six.reraise(*sys.exc_info())
...
cli/command_modules/profile/custom.py, ln 150, in login
    subscriptions = profile.find_subscriptions_on_login(
azure/cli/core/_profile.py, ln 212, in find_subscriptions_on_login
    subscriptions = subscription_finder.find_from_user_account(
azure/cli/core/_profile.py, ln 851, in find_from_user_account
    if not token_entry:
UnboundLocalError: local variable 'token_entry' referenced before assignment

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az login --user {} --tenant {}

Expected Behavior
Environment Summary
macOS-11.5.2-x86_64-i386-64bit
Python 3.8.6
Installer: HOMEBREW

azure-cli 2.15.0 *

Additional Context

@jiasli for awareness

This issue has been fixed by https://github.com/Azure/azure-cli/pull/15962 and released in 2.15.1.
Please update to the latest version.


Android compile error transformClassesWithJarMergingForDebug

hello
I am trying to add this plugin for ionic framework (android) but when compiling i get different errors,
firstly i was getting multiple dex file error and i added multiDexEnabled true
now i am getting the following error
:transformClassesWithJarMergingForDebug FAILED

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':transformClassesWithJarMergingForDebug'.
> com.android.build.api.transform.TransformException: java.util.zip.ZipException: duplicate entry: android/support/annotation/AttrRes.class

i tried following ways to fix but i am not able to fix it

tried to exclude android support v4 for branch by adding dependencie

compile('io.branch.sdk.android:library:1+') {
	exclude group: 'com.android.support', module: 'appcompat'
        exclude group: 'com.android.support', module: 'support-v4'
    }

plugins i have installed are
com.darktalker.cordova.screenshot 0.1.5 "Screenshot"
cordova-plugin-admobpro 2.14.0 "AdMob Plugin Pro"
cordova-plugin-console 1.0.2 "Console"
cordova-plugin-datepicker 0.9.3 "DatePicker"
cordova-plugin-device 1.1.1 "Device"
cordova-plugin-extension 1.3.0 "Cordova Plugin Extension"
cordova-plugin-file 4.1.1 "File"
cordova-plugin-file-opener2 2.0.2 "File Opener2"
cordova-plugin-file-transfer 1.5.1 "File Transfer"
cordova-plugin-inappbrowser 1.2.1 "InAppBrowser"
cordova-plugin-splashscreen 3.1.0 "Splashscreen"
cordova-plugin-statusbar 2.1.2 "StatusBar"
cordova-plugin-whitelist 1.2.1 "Whitelist"
cordova-plugin-x-socialsharing 5.0.12-dev "SocialSharing"
cordova-plugin-x-toast 2.5.0 "Toast"
es6-promise-plugin 3.0.2 "Promise"
io.branch.sdk 2.1.10 "branch-cordova-sdk"
ionic-plugin-keyboard 2.0.1 "Keyboard"
phonegap-facebook-plugin 0.12.0 "Facebook Connect"

I tried to change android support v4 with android suppoert v4:13
I added  renderScriptTargetApi 23 to config
I tried configurations { all*.exclude group: 'com.android.support', module: 'support-annotations' }
I tried to refresh the cordova by removing and adding back also

@hussainanver52 , can you try doing this workaround?
Hopefully this will resolve the issue.

no it didnt worked its a issue with support v4 jar file replacing it by support v4:13 and updating the calls in facebook core to this api woked for me


Rails/HasManyOrHasOneDependent flagging has_many: associations with foreign_key on_delete: cascade

Rails/HasManyOrHasOneDependent seems to be wrongly flagging has_many associations that have a foreign key with options {on_delete: :cascade}. We prefer to have the database to delete associated VerifyReminders, when we destroy a Signature and not Rails.
I have the following model:
class Signature < ApplicationRecord
  has_many :verify_reminders
end

I have the following migration:
class CreateVerifyReminders < ActiveRecord::Migration[5.1]
  def change
    create_table :verify_reminders, id: :uuid, default: -> { 'uuid_generate_v4()' } do |t|
      t.references :signature, type: :uuid, foreign_key: {on_delete: :cascade}
    end
  end
end

I have the following schema.rb after migration:
ActiveRecord::Schema.define(version:<PHONE_NUMBER>5158) do
  create_table "verify_reminders", id: :uuid, default: -> { "uuid_generate_v4()" }, force: :cascade do |t|
    t.uuid "signature_id"
    t.index ["signature_id"], name: "index_verify_reminders_on_signature_id"
  end

  add_foreign_key "verify_reminders", "signatures", on_delete: :cascade
end

Expected behavior
I expect that running rubocop on this file would not report any offense.
Actual behavior
I get the following report:
app/models/signature.rb:11:3: C: Rails/HasManyOrHasOneDependent: Specify a :dependent option.
  has_many :verify_reminders

Steps to reproduce the problem
Run rubocop on a fail containing the code above.
RuboCop version
Include the output of rubocop -V. Here's an example:
➔ bundle exec rubocop -v
0.51.0

You can disable this cop with .rubocop.yml or ignore the offenses in your case.
# .rubocop.yml
Rails/HasManyOrHasOneDependent:
  Enabled: false

RuboCop cannot know presences of foreign keys. So RuboCop cannot have this feature.
But we can probably add another cop to check foreign key.
# db/migrate/2017xxxxxx_some_migration.rb
create_table :users do |t|
  t.belongs_to :repository
# ^^^^^^^^^^^^^^^^^^^^^^^^ Did you forget `foreign_key`?
end

I know that there is no perfect solution to my issue but maybe we could improve the situation. In the given scenario it could be possible to check for the foreign key in the schema.rb file. Though, I have no overview about other possibilities within rubocop.

True. But we don't have a cop that reads other files such as schema.rb. So I think we cannot implement the feature immediately.


Regression: Cannot build loft of Ariane 5

TiGL Viewer crashes on the current master with a vector subscript out of range caused by this line:
https://github.com/DLR-SC/tigl/blob/7320a3c8531a332b6cfbd0c64940b5278e1252ff/src/fuselage/CTiglFuselageSegmentGuidecurveBuilder.cpp#L97
when opening the Ariane 5 with tiglviewer.

Do you have any clue, what causes the bug /when the bug was introduced?

At least cool, that this was detected by your script now ;)

You were faster than me, I am still git bisecting.
:D I like that the commit message is "prevent hard crash ..."

I like that the commit message is "prevent hard crash ..." 😆

Yeah, lets just replace one crash with another 😆


lldb-out: improve testing with lldb debugger attached

Trying to get the stack trace for a flaky test. Having the output corrupted for each test run is making things a lot complicated

@jianchun Thanks for the review


can't 'open component in editor '  when I use wsl environment

Version
6.4.1
Browser and OS info
Chrome 105/ Windows 10
Steps to reproduce

vite serve a project and visite localhost
open dev tools in chrome,
click open component in editor

What is expected?
open file in editor
What is actually happening?
error in command

In the windows environment, this function can be used normally

Same issue

Same, no one found a solution?

I'm using Visual Studio Code as my default editor and it required me to add the following to my shell configuration file (~/.zshrc) in WSL :
export EDITOR="code"


Issue with swapped dependency

Describe the bug
Running npm i @swc/core causes the following error:
❯ npm i @swc/core
npm ERR! Failed to fetch dependencies:
npm ERR! -<EMAIL_ADDRESS>
npm ERR! A complete log of this run can be found in:
npm ERR!     /home/.npm/_logs/2023-10-20T23_44_48_973Z-debug-0.log

My assumption is that "@swc/core" is being swapped for "@swc/wasm", with the matching version. The reason I am assuming that is because if I install one of the projects, that gives me the issue, locally then I get<EMAIL_ADDRESS>The issue is that the only 1.3.93 versions of @swc/wasm are nightlys, so<EMAIL_ADDRESS>does not exist.
I noticed the issue, because I have been unable to load any Storybook sandboxes from "https://storybook.new/". I did end up finding a workaround, once I realized what was happening, which is to add "@swc/core": "1.3.92" to my dependencies, so my child dependencies don't install<EMAIL_ADDRESS>Link to the blitz that caused the error
https://stackblitz.com/edit/stackblitz-webcontainer-api-starter-pprezj
Steps to reproduce

Run npm i @swc/core in a web-container.

Expected behavior
Since my child dependency has "@swc/core": "^1.3.82" in it's dependencies, I would expect a valid version of the replacement, which in this case would be "<EMAIL_ADDRESS>Parity with Local

[X] I have run the project in my local machine and I could not reproduce the issue.

Screenshots
No response
Platform

OS: [Windows]
Browser: [Chrome, Edge, Firefox]
Version: [118.0.5993.89, 119.0.2151.2, 119.0b8]

Additional context
No response

:wave: Thanks for the report, this looks like an error on our side, we assumed that @swc/core and @swc/wasm would be released in lockstep.
Your particular issue might solve itself on its own if they happen to bump the Wasm version, but at any rate, we probably need to look into handling this more graciously.


chore: Capitalize Nerdlet and Nerdpack

Description
Per marketing specs, Nerd* should be capitalized in public docs.

Is this good to merge @alexronquillo ?


Unable to connect to the server: i/o timeout after Cilium installation

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
Hello there.
I've initialized a bare-metal cluster with kubeadm. I want kube-proxy, and I deploy that as well, and everything is good.
I tried to install Cilium as my CNI only (not a replacement of kube-proxy), and after running the below command:
cilium install --version 1.15.4

After deployment of all Pods, now nothing works and kubectl commands return the below error:
$ kubectl get pod -A
Unable to connect to the server: dial tcp <IP_ADDRESS>:6443: i/o timeout

Even the cilium status command not working.
Cilium Version
$ cilium version --client
cilium-cli: v0.16.5 compiled with go1.22.2 on linux/amd64
cilium image (default): v1.15.4
cilium image (stable): v1.15.4

Kernel Version
$ uname -a
Linux master-1 5.15.0-89-generic #99-Ubuntu SMP Mon Oct 30 20:42:41 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Kubernetes Version
1.30.0

Regression
No response
Sysdump
No response
Relevant log output
No response
Anything else?
No response
Cilium Users Document

[ ] Are you a user of Cilium? Please add yourself to the Users doc

Code of Conduct

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

Thanks for this issue @alifiroozi80. Did you follow the steps in https://docs.cilium.io/en/stable/installation/k8s-install-kubeadm/ ?
That looks like your kubectl is unable to connect to the apiserver, which could be caused by many issues, not all of them related to Cilium. Without further info, I don't think we will be able to help you, sorry. (And obviously, if you can't connect to the cluster, I understand it will be difficult to collect further info).
I'd suggest starting over following the directions in the docs, and come back here if there are still problems.


[AIRFLOW-161] New redirect route and extra links

Dear Airflow maintainers,
Please accept this PR. I understand that it will not be reviewed until I have checked off all the steps below!
JIRA

[ ] My PR addresses the following Airflow JIRA issues and references them in the PR title. For example, "[AIRFLOW-161] My Airflow PR"

https://issues.apache.org/jira/browse/AIRFLOW-161

Description

[ ] With this change different operators would be able to customize the
task instance model view with extra links, those can be used to
redirect users to systems which are out of Airflow.

An operator which is not having any extra links to show (same as the present):

An operator with extra links to show:

Also modified call_modal function a bit to accept operator name instead of boolean for subdag_operator, it'll help in removing unnecessary if-else and keep code more DRY.
Tests

[ ] WIP

Commits

[ ] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

LGTM. +1

+1

I've overlooked the GUI screenshots, looks great! +1

So a better way of doing this is to let the webserver construct the redirect and have it verify the target against a whitelist. It should also make clear in the user interface where it is redirecting to, e.g. by showing the url below the button.
I do like the functionality: it would allow to link to a spark history server/ui etc

I also like the functionality, does it need a redirect though? Am I missing something but what about if it was something like BaseOperator.extra_urls defined as:

def spark_console_url(context):
    return 'spark.org/{dag_id}/{task_id}/'.format(**context)

def spark_log_url(context):
    return 'spark.org/{dag_id}/{task_id}/'.format(**context)

class SparkOperator(BaseOperator):
    extra_urls = [
        {
            'button_text': 'Spark Console',
            'url_generator': spark_console_url,
        },
        {
            'button_text': 'Spark Logs',
            'url_generator': spark_log_url,
        }
    ]
    # ...

@bolkedebruin I like the idea of having whitelisted domains only via config, but showing urls doesn't seem to be possible. For ex a DAG having 100s of tasks, then the webserver has to prepare urls for all those 100 operators's via calling get_redirect_url method, which would be too much CPU intensive and mostly unneeded.

@mistercrunch If we can achieve it the way you describe, it would be great. But will all tasks in a DAG need to prepare the urls in advance? If yes, then it would be a problem, cause all operators might not have very simple logic as you have in the example. In the current form we are doing the same, but lazily. i.e. deciding how to make an url on the basis of which button was clicked.

@msumit what you are saying doesn’t make sense. The web server doesn’t need to do it for 100s of tasks, because it will only do it for the task/operator being displayed. That is just 1.
I also think it doesn’t need to be a redirect, it can just be a link. @mistercrunch I’m not sure I understand your approach. If we launch a spark task we need to parse the output to get the right information to link directly to the relevant location. Thus there is a method involved in the operator that gives back dynamic information. And maybe stores this.
What I am imagining is that the Operator stores some extra info in a Json field, we then use a static method from The Operator (derived from BaseOperator) that translates this information to a url and the web server can make that visible when it is matched a against a whitelist.

@bolkedebruin that is not true. Airflow webserver prepares a json dump of the whole dag (i.e all tasks) and sends it to UI. When you click a node in UI, it just uses that json information to render the modal dialog. So it can't be done 1 node at a time, unless you load the model dialog via ajax. For better understanding take a look at nodeobj variable from javascript console on tree view page of a dag.

@bolkedebruin added domain whitelist support.

ping @mistercrunch @bolkedebruin @Fokko

@msumit The tests are failing. Please check Travis. For me, this is good to go if @mistercrunch and @bolkedebruin do not have any objections.

somehow tests are failing with below error:
File "/home/travis/build/apache/incubator-airflow/airflow/www/views.py", line 1766, in redirect
   dttm = dateutil.parser.parse(execution_date)
NameError: global name 'dateutil' is not defined
while many other methods using dateutil in views.py, also it passes in my local.

@msumit You need to rebase with apache:master, you're behind 52 commits.

Can you remove the redirect? I don't think it is required. The button can be generated directly instead of being a redirect. Please also rename the Jira, because a redirect from a security point of view does not sound good.
Please rebase indeed, because dateutil should not be used anymore for parsing, but rather airflow.utils.timezone.parse

Please make sure to run against latest master so that flake8 tests also run

@bolkedebruin will work on all the suggestions. But I am still not certain how to generate all links before rendering the page. Restating that, to achieve that we've to prepare all such links for all tasks of a DAG beforehand, which would be time-consuming.

Hey @msumit this is something I'm interested in. Would it be okay if I took hold of your changes and tried to drive it to conclusion?

@ArgentFalcon yeah sure, plz go ahead.


version bumps (sbt 1.5.5, Scala.js 1.7, setup-scala v13)

and remove sbt-dotty, not needed anymore on sbt 1.5

ready for review

ping

@alexandru feel free to reopen and merge :-)


To change twitter logo as X

As the twitter logo is changed I would like to make change in the website.
I will add the new twitter logo !
Assign me the issue if u like my idea @phyleria

@phyleria have a look

Hi @ayan-joshi
Which line would you like to make the changes to?

This twitter logo is old we can change it to the new one

Ok, go ahead

Hi, @phyleria and @Khadeeejah I would like to work on this issue since it seems it hasn't been worked on yet even though it was assigned someone. I will be happy to help.

Go ahead @octocatblain 👍🏾


Deprecate type

Currently, the user needs to define type before certain tests can pass (e.g. min or max).
It would be more intuitive if e.g. a min specification did not require a type specification, but instead tried to:

Coerce the input value (considered a string) to the required type (a number)
Then test the min specification

The reporting could indicate at which step the test failed.
Effects:

type: json and type: url need to become stringformat: json and stringformat: url
type: boolean is no longer supported, but that was a tricky one anyway, as it can be coerced in non-intuitive ways. User can use allowed: [0, 1, True, False] instead.
Number tests no longer require a type
Difference between integers and floats can be tested with numberformat
type: string was useless anyway, as every value passed it.

@stijnvanhoey, in the specifications you wrote:

Remark:
The behaviour of length (minlength and maxlength) depends on the usage of the length validation in combination with the type validation or not. When no type validation is added (or tests for string, which is default), the length will interpret the field as string:
maxlength: 2
type : string

will invoke an error for the field : {'individualCount' : '100'}. However, when requiring an integer value:
maxlength: 2
type : integer

the field {'individualCount' : '100'} will be converted to {'individualCount' : 100} (100 as  integer) and length will ignore the integer. It makes more sense to test this with the min and max validators (see further).

I have removed that whole section. I think minlength and maxlength should always assume strings and test for those.

type has been removed from the specification, closing here


drivers: dma: stm32 disabling stream waits for disable

Add the loop to wait until the stream is really disable after disabling and also cleared the TCIF flag.
This is a specifity of the dma of type V1.
Fixes https://github.com/zephyrproject-rtos/zephyr/issues/54805
Signed-off-by: Francois Ramu<EMAIL_ADDRESS>

also fixes the tests/drivers/uart/uart_async_api/ running on the nucleo_f429zi:
Running TESTSUITE uart_async_write_abort
===================================================================
START - test_write_abort

    Assertion failed at WEST_TOPDIR/zephyr/tests/drivers/uart/uart_async_api/src/test_uart_async.c:526: uart_async_write_abort_test_write_abort: (sent not equal to received)
Sent is not equal to received.
 FAIL - test_write_abort in 0.120 seconds

Rebase on b91aa2e28c1004141e452dee8cc1d1a8712e7d35
Add a break line before return


Make tests fail on warnings

Problem
Users seeing warnings.
Solution
This is a technique I've rolled out against my own open source projects and Django REST Framework.
By failing tests as soon as new deprecation warnings are created in the library, they aren't passed on to users who then have to report them back in issues like #1195, saving everyone some time, effort, and noise.
Acceptance Criteria
There are no warnings at current so the tests don't fail. To check I reverted the code changes from commit e855fcf, ran Django 3.1 tests, and saw the warnings fail the test run at import time:
GLOB sdist-make: /.../django-import-export/setup.py
py38-django31-tablibstable inst-nodeps: /.../django-import-export/.tox/.tmp/package/1/django-import-export-2.4.1.dev0.zip
py38-django31-tablibstable installed: asgiref==3.2.10,certifi==2020.6.20,chardet==3.0.4,coverage==5.3,coveralls==2.1.2,defusedxml==0.6.0,diff-match-patch==20200713,Django==3.1.2,django-import-export @ file:///.../django-import-export/.tox/.tmp/package/1/django-import-export-2.4.1.dev0.zip,docopt==0.6.2,et-xmlfile==1.0.1,idna==2.10,isort==5.6.4,jdcal==1.4.1,MarkupPy==1.14,mysqlclient==2.0.1,odfpy==1.4.1,openpyxl==3.0.5,psycopg2==2.8.6,pytz==2020.1,PyYAML==5.3.1,requests==2.24.0,sqlparse==0.4.1,tablib==2.0.0,urllib3==1.25.10,xlrd==1.2.0,xlwt==1.3.0
py38-django31-tablibstable run-test-pre: PYTHONHASHSEED='903442151'
py38-django31-tablibstable run-test: commands[0] | python -W error::DeprecationWarning -W error::PendingDeprecationWarning /.../django-import-export/tests/manage.py test core
Traceback (most recent call last):
  File "tests/manage.py", line 12, in <module>
    execute_from_command_line(sys.argv)
  File "/.../django-import-export/.tox/py38-django31-tablibstable/lib/python3.8/site-packages/django/core/management/__init__.py", line 401, in execute_from_command_line
    utility.execute()
  File "/.../django-import-export/.tox/py38-django31-tablibstable/lib/python3.8/site-packages/django/core/management/__init__.py", line 377, in execute
    django.setup()
  File "/.../django-import-export/.tox/py38-django31-tablibstable/lib/python3.8/site-packages/django/__init__.py", line 24, in setup
    apps.populate(settings.INSTALLED_APPS)
  File "/.../django-import-export/.tox/py38-django31-tablibstable/lib/python3.8/site-packages/django/apps/registry.py", line 122, in populate
    app_config.ready()
  File "/.../django-import-export/.tox/py38-django31-tablibstable/lib/python3.8/site-packages/django/contrib/admin/apps.py", line 24, in ready
    self.module.autodiscover()
  File "/.../django-import-export/.tox/py38-django31-tablibstable/lib/python3.8/site-packages/django/contrib/admin/__init__.py", line 24, in autodiscover
    autodiscover_modules('admin', register_to=site)
  File "/.../django-import-export/.tox/py38-django31-tablibstable/lib/python3.8/site-packages/django/utils/module_loading.py", line 47, in autodiscover_modules
    import_module('%s.%s' % (app_config.name, module_to_search))
  File "/Users/chainz/.pyenv/versions/3.8.5/lib/python3.8/importlib/__init__.py", line 127, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1014, in _gcd_import
  File "<frozen importlib._bootstrap>", line 991, in _find_and_load
  File "<frozen importlib._bootstrap>", line 975, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 671, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 783, in exec_module
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "tests/../import_export/admin.py", line 25, in <module>
    from .signals import post_export, post_import
  File "tests/../import_export/signals.py", line 3, in <module>
    post_export = Signal(providing_args=["model"])
  File "/.../django-import-export/.tox/py38-django31-tablibstable/lib/python3.8/site-packages/django/dispatch/dispatcher.py", line 36, in __init__
    warnings.warn(
django.utils.deprecation.RemovedInDjango40Warning: The providing_args argument is deprecated. As it is purely documentational, it has no replacement. If you rely on this argument as documentation, you can move the text to a code comment or docstring.
ERROR: InvocationError for command /.../django-import-export/.tox/py38-django31-tablibstable/bin/python -W error::DeprecationWarning -W error::PendingDeprecationWarning tests/manage.py test core (exited with code 1)
___________________________________________________________________________________________________ summary ____________________________________________________________________________________________________
ERROR:   py38-django31-tablibstable: commands failed

Coverage decreased (-5.2%) to 91.507% when pulling a5b59c73d77b7853299cd60d95a67422bce1d924 on adamchainz:issue_1195_warnings into 691baadc9ea9258ce6859fa010cfb434ebbc381c on django-import-export:master.