Shibboleth.sso/Login requires redirect url as ?target=url, not as ?next=url

I have set the LOGIN_URL to mydomain/Shibboleth.sso/Login
If I enter
mydomain/Shibboleth.sso/Login?target=redirectUrl

I am being redirected corretly. Juding from my browser history, the automatic link created by this package is
mydomain/Shibboleth.sso/Login?next=redirectUrl

Strangely enough, I can't find the bug in the code, so it might as well have another cause.
See documentation here https://wiki.shibboleth.net/confluence/display/SHIB2/NativeSPSessionCreationParameters
This might be different for older versions of shibboleth.

It has been a couple of years since this issue was submitted (as well as the PR). @bcail and @KonstantinSchubert do you think this is still an issue or have things changed?

?next= definitely does not work with default Shibboleth implementations, it expects it to be ?target=

Uncertain if the conditions are the same as OP, but this happens when the admin site is protected by Shibboleth. Basically, follow the setup in the readme with /admin in the Shibbolized Location directive, and add a urlconf like url(r'^admin/login/$', RedirectView.as_view(url=settings.LOGIN_URL, permanent=False, query_string=True)). This generates a redirect to the IdP with ?next= as parameter.
In this particular case, it seems to be due to a combination of an admin quirk (it does not honor LOGIN_URL) and not being able to pass redirect_field_name to RedirectView


Add support for kubernetes pod labels and annotations for KFP pipelines

This PR

enables the pod labels and annotations support that is delivered by https://github.com/elyra-ai/kfp-notebook/pull/83
attaches metadata to containers before running a pipeline
attaches metadata to containers when exporting a pipeline
closes https://github.com/elyra-ai/elyra/issues/1277
closes #1289

Example of metadata that is attached to each KFP pipeline node:
metadata:
  ...
  labels:
    elyra/experiment-name: my-labeled-pipeline-g-0210142652
    elyra/node-name: notebook
    elyra/node-type: notebook-script
    elyra/pipeline-name: my-labeled-pipeline-g
    elyra/pipeline-version: my-labeled-pipeline-g-0210142652
    ...
  annotations:
    elyra/node-file-name: notebook.ipynb
    ...

Values that are derived from user provided input (experiment-name,  node-name, pipeline-name, pipeline-version / future, and node-file-name) don't necessarily match the user provided input, due to restrictions imposed by Kubernetes. For example, if a pipeline was named by a user %off-analysis, the name cannot be used as-is as a label value, because % is not a supported character. Elyra takes care of converting this string into a format that meets the restrictions. However, that means that the listed values might deviate significantly from the one a user might be familiar with.

User workflow scenarios that are covered:

S1: Run pipeline from Pipeline editor
S2: Export pipeline to YAML > upload YAML in KFP UI > create an experiment > run pipeline
S3: Export pipeline to Python DSL > run generated Python DSL in terminal window to create an experiment and run the pipeline

More in-depth discussion about issues in the comments below.
Developer's Certificate of Origin 1.1
   By making a contribution to this project, I certify that:

   (a) The contribution was created in whole or in part by me and I
       have the right to submit it under the Apache License 2.0; or

   (b) The contribution is based upon previous work that, to the best
       of my knowledge, is covered under an appropriate open source
       license and I have the right under that license to submit that
       work with modifications, whether created in whole or in part
       by me, under the same open source license (unless I am
       permitted to submit under a different license), as indicated
       in the file; or

   (c) The contribution was provided directly to me by some other
       person who certified (a), (b) or (c) and I have not modified
       it.

   (d) I understand and agree that this project and the contribution
       are public and that a record of the contribution (including all
       personal information I submit with it, including my sign-off) is
       maintained indefinitely and may be redistributed consistent with
       this project or the open source license(s) involved.

elyra/node-name:
Identifies the pipeline node that the Kubernetes pod is processing.
elyra/node-name: notebook

Issue: The node name is currently the notebook's | script's basename (e.g. notebook for notebook.ipynb) and not the user-provided value. @lresende is working on a fix that should pass through the correct information.

elyra/pipeline-version:
Pipeline versioning is currently not supported. Therefore the value is '' for every pipeline.
elyra/pipeline-version: ''

elyra/experiment-name:
In KFP an experiment identifies a "workspace" in which a pipeline is executed. Currently there is a 1-to-1 relationship between pipelines and experiments (and their names) if a pipeline is executed through the Elyra UI.
elyra/experiment-name: my-labeled-pipeline-g-0210142652

elyra/pipeline-name:
Identifies the pipeline that the Kubernetes pod is associated with.
elyra/pipeline-name: my-labeled-pipeline-g

elyra/node-type:
Identifies the pipeline node type, which initially is always notebook-script to indicate that the pod processes a Python script or notebook. Additional types can be introduced in the future, once pipelines can be assembled from components.
elyra/node-type: notebook-script

elyra/node-file-name (annotation):
This annotation identifies the notebook | Python script file name for pods that are labeled with elyra/node-type: notebook-script. This annotation complements the elyra/node-name label.
elyra/node-file-name: notebook.ipynb

Basic test pipeline:

Download i1284.tar.gz
Extract the archive in your JupyterLab workspace
Open my-labeled-pipeline

Example screen shots taken during test of scenario 1 (run from Elyra UI)
KFP Pipelines view:

KFP Experiments view:

KFP component view (for one of the nodes):

The tests are failing https://github.com/elyra-ai/elyra/pull/1284/checks?check_run_id=1891031113#step:12:1407 because the outdated kfp-notebook package is used. The tests should succeed once elyra-ai/kfp-notebook#83 was merged and the updated package published.

I guess I view the experiment to be like an instance of a pipeline. A runnable instance.

In KFP an experiment is a workspace and can contain runs from different pipelines. For example, a user can create an experiment X1 and then run pipeline P1 and pipeline P2 in that experiment. If the user chooses to create another experiment X3, he/she can run P1, P2, or any other pipeline P3 in this experiment.

In KFP an experiment is a workspace and can contain runs from different pipelines. For example, a user can create an experiment X1 and then run pipeline P1 and pipeline P2 in that experiment. If the user chooses to create another experiment X3, he/she can run P1, P2, or any other pipeline P3 in this experiment.

Thank you for this clarification.  I had nearly the opposite definition.
Today, Elyra only supports single-pipeline experiments.  Are there plans to support multi-pipeline experiments?  I haven't heard us talk about that kind of thing and a quick search doesn't find anything, but let's not rule that out.
Since the "experiment" is really an end-user notion, it seems to me that, by default, the experiment name should be the thing provided by the user and, if not provided, default to the basename of the pipeline filename (if multi-pipeline experiments are supported, this default would need to be revisited).  It seems like when the user runs a pipeline, they should have the option of entering a name (that defaults to the pipeline name).
pipeline_name is purely static and is the basename of the pipeline file.  There would really be no need to prompt for the user to change this value, it should be derived directly from the pipeline filename.  They also get to set/change the name when they create or rename the pipeline file.
And the run_name would be the timestamped pipeline_name and no need for the -run suffix.
I also believe that experiment_name should be something passed into the pipeline processor (assuming a given processor processes a single pipeline) so that when/if we support true (multi-pipeline) "experiments", that same name is conveyed to each processor.
This could be summarized as:

experiment_name is provided at runtime and defaults to the pipeline file basename.
pipeline_name is the basename of the pipeline filename.
run_name is purely the timestamped pipeline_name.
And, in the KFP UI, the pipeline_name is not surfaced.

Hmm - since experiments can be run multiple times (and is why we timestamp the pipeline name) I wonder if the experiment name passed to create_experiment should be the run-name prefixed with experiment_name?  That is experiment_name-pipeline_name-timestamp.
I think it would be good to get a solid definition of these terms in particular and align them with the roadmap.  We should also consider how they might equate to Airflow parlance since we essentially want the same pipeline editor for both/all platforms.  If this means naming "experiments" to be more like a "workspace" (although I would view a workspace to be a container of experiments), then we could define our own terms with the platform-specific terms mapped in the respective platform-specific processors.

Based on the discussion've we had this morning, I updated processing behavior if a pipeline is run from the pipeline editor.
First time run:

Pipeline file name: version-test3.pipeline
Pipeline name: version-test3 (the default when the job was submitted)
KFP UI pipeline name: version-test3
KFP UI version name: version-test3
KFP UI experiment name: version-test3
KFP UI run name: version-test3-0215144656
COS artifact location: .../version-test3-0215144656

Second/n-th time run (property values that have changed are marked in bold):

Pipeline file name: version-test3.pipeline
Pipeline name: version-test3 (the default when the job was submitted)
KFP UI pipeline name: version-test3
KFP UI version name: version-test3-0215153755 (Note the appended timestamp to achieve uniqueness)
KFP UI experiment name: version-test3
KFP UI run name: version-test3-0215153755 (Note the updated timestamp to achieve uniqueness)
COS artifact location: .../version-test3-0215153755 (Note the updated timestamp to achieve uniqueness)

From a user's point of view the only thing that changes is that all pipeline runs are now grouped under the same pipeline name and experiment. Previously each pipeline run was treated as a separate (logical) pipeline and experiment.

With this new behavior, a user can finally compare performance metrics across multiple runs:

Quick update - process() - as far as I can tell - currently has no awareness of what the original pipeline file name is. Therefore, if a user overrides the pipeline name when the job is submitted, the implementation currently does not have awareness that this has happened.

Quick update - process() - as far as I can tell - currently has no awareness of what the original pipeline file name is. Therefore, if a user overrides the pipeline name when the job is submitted, the implementation currently does not have awareness that this has happened.

I guess this makes sense. Does the server ever need to know the pipeline's filename since the pipeline is actually sent in the request?  The file is never read on the server (other than the new CLI tool) - to the best of my knowledge.
Has all the basename talk been conflated with node and notebook file names? :smile:
If so, that's probably "on me" and I apologize!

Export as YAML update. I tested two scenarios:

upload exported YAML as new version
upload exported YAML as new pipeline

The results surface an issue that, as is, cannot be resolved properly without loosing information.
Pod metadata for upload-as-new-version scenario:
    elyra/experiment-name: ''     # this is correct because at compile time we don't know which experiment the user will choose eventually
    elyra/node-name: notebook
    elyra/node-type: notebook-script
    elyra/pipeline-name: version-test3 # this is correct because the pipeline was manually uploaded as a new version
    elyra/pipeline-version: '' # this is correct because at compile time we don't know which experiment the user will choose eventually
    elyra/node-file-name: ...

Pod metadata for upload-as-new-pipeline scenario:
    elyra/experiment-name: '' # this is correct because at compile time we don't know which experiment the user will choose eventually
    elyra/node-name: notebook
    elyra/node-type: notebook-script
    elyra/pipeline-name: version-test3 # this is **not correct** because the pipeline was manually uploaded as new pipeline `version-test4`
    elyra/pipeline-version: ''
    elyra/node-file-name: notebook.ipynb

From a conceptual point of view it'd probably be better to set elyra/pipeline-name to the empty string, which is at least not wrong. However, with pipeline name and version not set, there is no good way to associate the pod(s) with a pipeline at all ...
    elyra/experiment-name: ''
    elyra/node-name: notebook
    elyra/node-type: notebook-script
    elyra/pipeline-name: ''
    elyra/pipeline-version: ''
    elyra/node-file-name: notebook.ipynb

Does the server ever need to know the pipeline's filename since the pipeline is actually sent in the request?

Not the server, but it's the operator/monitor that is potentially loosing context information. The results of the export-to-YAML test highlight that nicely. In that scenario having another piece of metadata would help. Maybe add one more annotation that contains the original pipeline file?

Yes, an annotation makes sense for this, but where is that value going to come from?  The processor doesn't "see" the filename since the body of the request is the file content.

Yes, an annotation makes sense for this, but where is that value going to come from? The processor doesn't "see" the filename since the body of the request is the file content.

Is this an HTTP request? If it is, could the information be enclosed in the header?

Is this an HTTP request? If it is, could the information be enclosed in the header?

Yes.  I suppose it could, but I don't know enough about web-programming to answer definitively.   Anyone else?

To resolve the "export as YAML" issue I'm changing the value of  elyra/pipeline-name to be empty because I believe no information is better than (potentially) incorrect information.


Social Media Blocker Extension

Description
This will be an awesome JavaScript Extension some what similar to Ad Blocker but this Social Media Blocker will block all the social media while someone is focusing on his/her work.
@harshita214, please assign me this extension
Consider this under GSSoC'22

@Atharv-110 there are 2 site blockers already present Block Site and AD Blocker, you can take a look at both of them and think of enhancing feature of any one of them. Ping me on discord if you have any queries! I will be happy to help

This will not be site blocker, it will be only for social media blocker which doesn't make any coder get distracted by them. @harshita214

@Atharv-110 kindly check this and view its readme file this is also blocking sites and focus features also.
I can see you are willing to develop a social media blocker then, this this is the task which you can do, that is here as of now only some specific websites which are added to the code base are being blocked, what you can do is implement your social media blocker feature here where user can select social media websites which he/she wants to block. Do let me know your views on this.


Integrate OpenAI into the chat bot #8.

Changes description

Adds /message endpoint for slack message events.
Adds OpenAI integration for chat completion. Implements OpeanAI configuration from JSON file.
Adds unit tests. Updates CI to run tests with coverage and post coverage report as PR's comment.
Fixes version check CI.
Updates README with instructions on how to setup the slack app.

What else do I need to know?
There are few known issue which should be investigated and fixed.

Slack resends message events very often since even though message processing is async, often it is taking more than 3 seconds. I've added the hack to not send those duplicate messages responses back to slack but this still will be processed and sent to openai. To be investigated why it's taking so long and how to handle it better.

Added a few TODOs in the code to be implemented later:

Messages should be streamed if possible to do it with slack or user should be notified somehow that the response generation is in progress.
Logging should be refactored to be configured from one place and stored to the file.
Add deployment instructions and mekt deployment run automatically from CI.

How was it tested? / Demo

https://github.com/COXIT-CO/RecruitFlowAI/assets/6651168/0bca3f14-234b-4bb9-83dc-bfa7ad6df817
Quality check

[x]  Set base branch to dev.
[x]  Pull request is prepared: Added to sprint, added yourself as assignee, reviewers are added.
[x]  README or any other documentation is added or updated.
[x]  Clear instruction on how to run it, what are input/output, and configuration is provided.
[x]  Comments are added to my code, particularly in hard-to-understand areas.
[x]  Logs are added for observability, particularly in error or exception cases.
[x]  Unit tests and/or sample app are added.
[x]  All added tests/sample app is run in CI.
[ ]  PR description is clear and demo screens or recoding are attached.
[ ]  PR is not very big (about 400 lines of change ideally).
[ ]  Doesn't create any new FIXME or TODO comments. If not, please explain.
[x]  CI checks passed
[ ]  Followed best practice rules:https://google.github.io/styleguide/pyguide.html

Converting it to a draft since need to fix conflicts first.


Add new & improved glob matcher

I've added a new matcher using gobwas/glob side-by-side to the existing glob matcher
This might hinder performance due to the fact that the matcher is compiling the pattern each time.
I was thinking of using a cache for the compiled matchers (something like patrickmn/go-cache).
@magiconair what do you think?
https://github.com/fabiolb/fabio/issues/452

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Shay Arbov seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Hey @magiconair,
I've implemented the fixes you've requested.
As far as I can tell the two glob packages are compatible,
I used the same test to validate it (and extended it to test the new capabilities).


Struct types

Infer types for struct fields, and add basic field completions. There's also some code for enums, but I focused on getting structs working.
There's still ways to go before this becomes useful: There's no autoderef (or even reference types) and no inference for self, for example.

@matklad I've added a hir::TypeRef as an intermediate between ast::TypeRef and ty::Ty, and used that to resolve the field types lazily.
I also added reference / pointer types since I was already working on them and it intersected a bit :smile:

bors r+
Super!

@matklad I've added a hir::TypeRef as an intermediate between ast::TypeRef and ty::Ty, and used that to resolve the field types lazily.

One thing to think about is how would we map back from hit::TypeRef to ast::TypeRef (this will be needed in IDE bits I think, and for error reporting). An obvious solution would be to store spans/syntax ptrs inside the hir, but I'd rather avoid that: this information takes space, and is only rarely needed. Ideally, we invent some kind of side-table approach here....


REACH-157 NodeView for Watch node

Use WatchNodeView for watch node.
To work correctly, this pull request requires https://github.com/Benglin/Dynamo/pull/255.

Hi @pboyer,
Please, do not merge this pull request until we get clear with https://github.com/Benglin/Dynamo/pull/255#discussion_r22651432

Status update? Looks like you're unifying 255 and 256?

Yes, Peter, we are going to unify 255 and 256.

Hi @pbidenko,
Can you bring this up to date?

ping @pbidenko

Hey @pboyer , thanks for the reminder :)

To work correctly, this pull request requires https://github.com/DynamoDS/Dynamo/pull/3730


Change date in filenames to year-month-day format

See https://stackoverflow.com/a/48729396/14144258 [and https://stackoverflow.com/a/6040556/14144258].
Change in https://github.com/SuffolkLITLab/ALKiln/blob/releases/v4/lib/utils/files.js
[Should we change the date format in the after-test report? Should we make a different issue for that? The main report could use the GitHub action environment variable to better match up with the GitHub test folder name. That wouldn't work for local tests, though. Ugh. How about the individual scenario reports? Maybe not necessary.]

For time, we're using the format ##hrs ##mins ##secs ###ms because windows path names can't have a : character in them.


chore: Point graddle to official GAPIC artifact

This updates our build.gradle to point to the published GAPIC artifacts. On a follow up I will clean up the local projects previously referenced in the build.

lgtm

Should probably also remove all the vendored code in this change as well right? *-google-maps-fleetengine-v1-*

Thanks Bo for reminding us on this, was too focused on the gradle file.

Yup, was thinking of doing a follow up PR to remove those, but can do here since it's a pretty small change

Yea I ask because submitting atomically will make absolutely sure build is consuming the gradle dependencies and not the vendored dependencies.

Thanks for the reminder @bshi
I've amended my commit PTAL when you get a chance @shawnlu-g

@ArturoSalazarB16 go/github to register your github account


androguard and android-21

Looks like there is an issue with androguard and android-21. Specifically the get_android_resources() function seems to be having an issue parsing the resources.arsc file successfully. We discovered issue on some internal builds and I'm going to try to recreate a simple apk to demonstrate, but wanted to go ahead and create issue in case anyone has any ideas.

@adesnos Have you had a chance to look at this?

@kvarga @subho007 I met the same problem!!! It occur in  the "> or = Android-SDK-21" apps.I think the problem might be that  "self.header = unpack('<h', buff.read(2))[0]"   is 0,it is not correct.

Don't know if it's related, but I get a similar error when open an apk file, the stack trace shows the error happened in zipfile.py (http://bugs.python.org/issue14315). The androguard included version (md5: 1e042f0661f8966e7a15c81651b4dfb9) generate an "unpack" error, but the (previously mentioned) patched version of zipfile.py works without any problems.

This seems to be fixed at uTest/Androguard
Although, a small bug remains which can be patched according to my comments at https://github.com/uTest/Androguard/pull/1

I have implemented the fix in my repo too at https://github.com/xysec/androguard/pull/8


No way to get back to landing page when waiting for other user

https://github.com/GoogleChrome/webrtc/issues/445
When you enter a room and no one else has joined, or if the remote side hangs up, there is no way to get back to the landing page. (You can hit back in a browser, but not in the Chrome App)
jiayliu commented 5 days ago
There is a shortcut key 'q' to hangup while waiting, but we probably should make it a visible UI element.

There are now rejoin and new room buttons at the bottom after hanging up, is this enough?


Infinite rerender in admittedly niche use case

So I am building a table right now, and within this table there can be nested items ... this is being accomplished by adding a row to the data set that has a single td with colSpan=number of columns. Pretty common hack.
Anyways, I naturally created a component that would nest inside of this full-width td, and when I add it in the component infinitely rerenders. A stripped-down version of what I mean:
renderItem(i, key) {
...
    if (isExpandedRow) {
        return

                    Test

;
    } else {
        // build rows
        return

            {rows}

;
    }
}

Obviously this lacks the keys and all that jazz, but you get the idea. Now, this setup works just fine, however when I switch out "Test" for my component ... boom.
renderItem(i, key) {
...
    if (isExpandedRow) {
        return

                    <MyComponentThatWorksJustFine />

;
    } else {
        // build rows
        return

            {rows}

;
    }
}

Now, when I say boom, its not automatic. Only when I scroll down and updateFrame is triggered does the infinite re-render begin. My hunch is that its due to the fact that the itemsPerRow differs each time, and you appear to be calling eventually calling setState on componentDidUpdate ... so it updates ad-infinitum.
Any ideas?

Sounds like you're using the uniform type? If so, are all of your items always an identical size? That is a prerequisite for using the uniform type, and this situation can arise when it's not met.

Ah bugger ... you're right, I'm using uniform. I guess I'm learning how to use the itemSizeGetter so that I can ensure the scrollbar doesn't "magically grow".
Thanks man, appreciate the help.


FileNotFoundError in Colab Demo

Using the unmodified Google Colab Demo.
I think it has something to do with the zip file not being downloaded correctly. The size of the zip is only around 3KB. Maybe the Google Drive download is being rate limited?
Error:
initialize network with normal
initialize network with normal
initialize network with normal
model [FusionModel] was created
load Fusion model from checkpoints/coco_finetuned_mask_256_ffs/latest_net_GF.pth
---------------------------------------------------------------------------
FileNotFoundError                         Traceback (most recent call last)
<ipython-input-11-e948e7e9ae19> in <module>()
      1 model = create_model(opt)
----> 2 model.setup_to_test('coco_finetuned_mask_256_ffs')

3 frames
/content/InstColorization/models/fusion_model.py in setup_to_test(self, fusion_weight_path)
     95         GF_path = 'checkpoints/{0}/latest_net_GF.pth'.format(fusion_weight_path)
     96         print('load Fusion model from %s' % GF_path)
---> 97         GF_state_dict = torch.load(GF_path)
     98
     99         # G_path = 'checkpoints/coco_finetuned_mask_256/latest_net_G.pth' # fine tuned on cocostuff

/usr/local/lib/python3.6/dist-packages/torch/serialization.py in load(f, map_location, pickle_module, **pickle_load_args)
    582         pickle_load_args['encoding'] = 'utf-8'
    583
--> 584     with _open_file_like(f, 'rb') as opened_file:
    585         if _is_zipfile(opened_file):
    586             with _open_zipfile_reader(f) as opened_zipfile:

/usr/local/lib/python3.6/dist-packages/torch/serialization.py in _open_file_like(name_or_buffer, mode)
    232 def _open_file_like(name_or_buffer, mode):
    233     if _is_path(name_or_buffer):
--> 234         return _open_file(name_or_buffer, mode)
    235     else:
    236         if 'w' in mode:

/usr/local/lib/python3.6/dist-packages/torch/serialization.py in __init__(self, name, mode)
    213 class _open_file(_opener):
    214     def __init__(self, name, mode):
--> 215         super(_open_file, self).__init__(open(name, mode))
    216
    217     def __exit__(self, *args):

FileNotFoundError: [Errno 2] No such file or directory: 'checkpoints/coco_finetuned_mask_256_ffs/latest_net_GF.pth'

Do you run the section 4 for downloading the model?
!sh scripts/download_model.sh
I've just checked the notebook, and still not found any problem.

Yes, I ran the script. This is it's output:
Downloading...
Finish download.
Archive:  checkpoints.zip
  End-of-central-directory signature not found.  Either this file is not
  a zipfile, or it constitutes one disk of a multi-part archive.  In the
  latter case the central directory and zipfile comment will be found on
  the last disk(s) of this archive.
unzip:  cannot find zipfile directory in one of checkpoints.zip or
        checkpoints.zip.zip, and cannot find checkpoints.zip.ZIP, period.

Got the same issue when ran from localhost. But works for on Colab.

I am getting the error on Colab. I guess outdated link, and still I can't find another location.


add pump location to comments

Thanks for this awesome extension, it's handy. :)
I wanted to know how the header was generated and what the syntax was, and it took me 30 minutes or so to find where "pump" was.  (In particular, googling was of no use.)  I suggest pointing to it in the comments.

Sorry for late response. I'm glad you find it handy. Especially this project is a result of hacking born from  a curiosity ;)


Opening hours editor should ask for time by default

Should ask for times ba default instead of assuming full day open. Then add a "full day open"-button. 99% of the shops are not open 24 hours a day

I understand the reasoning, but no. It won't be a good UI. This one extra tap is needed to ease a user into the UI switch. Also many people follow the order "weekdays → hours", and your suggestion reverses it, adding for an unnecessary confusion that would take more time than tapping the switch.

How about getting rid of any 24 hours and just letting the user set the opening time from 0:00 to 24:00?


(BKR-1174) Add support for SLES 12 on Power8

Note: Power8 platforms make use of ABS templates.

👍


autocompletion isn't work at starting insert

FAQ

[X] I have checked the FAQ and it didn't resolve my problem.

Issues

[X] I have checked existing issues and there are no open or closed issues with the same problem.

Neovim Version
NVIM v0.7.0-dev+863-g8f27c4a041
Minimal reproducible config
vim.cmd('set rtp+=~/.local/share/dein/repos/github.com/hrsh7th/nvim-cmp')
vim.cmd('set rtp+=~/.local/share/dein/repos/github.com/hrsh7th/cmp-buffer')

local cmp = require('cmp')
cmp.setup({
    sources = {{name = 'buffer'}},
    completion = {
        autocomplete = {
            require('cmp.types').cmp.TriggerEvent.InsertEnter,
            require('cmp.types').cmp.TriggerEvent.TextChanged
        }
    }
})

cmp.register_source('buffer', require('cmp_buffer'))
vim.cmd('set completeopt+=menu,menuone,noselect')

Description
Sometimes the menu does not appear when I do InsertEnter. It might be skipped because there is no change from the last menu.
Steps to reproduce

Open with nvim -u mini.lua foo.txt
Type "foo fo". the completion menu has "foo"
Escape from insert mode
Type 'a' to insert
The menu will not be shown

Expected behavior
I want the completion menu to appear when InsertEnter.
Actual behavior
It seems like the display is skipped when cursor_before_line is the same.
Additional context
"ifoo fo<ESC>i<ESC>$a" will show the menu.
"ifoo fo<ESC>i<ESC>$a<ESC>a" will not show the menu at the last a.

Thank you. I've fixed it.


you do not intent -> you do not intend

#hacktoberfest

@potatoqualitee : Thanks for your contribution! The author, @MikeRayMSFT, has been notified to review your proposed change.

Merging typo fix
@potatoqualitee Thanks!

Thank you!


fix(plex): fix font loading performance

Overview
Resolves #405.
Applies a similar change to https://github.com/IBM/type/issues/34 to reduce HTTP requests for fonts.
Changed
The order of @font-face definition for various Unicode ranges.
Testing / Reviewing
Testing should make sure no font rendering is broken.

Much faster! 👍 ✅

👍 ✅


Goo Gauntlet Model Rendering instead of Held Mitten

Make the gauntlet render as a sort of armor layer sleeve rather than a held item like a mitten you're holding. The whole thing was just a placeholder and was intended for improvement.
The item representation is still relevant for what renders in inventories, but in world we want it to look more glovely.

This has been done for a long time. Not sure how I missed this.


Output basic Sense info to logs on Butler startup

It would be nice to get some basic info about the underlying Qlik Sense environment written to the Butler logs, whenever Butler is started.
Things such as overall Sense version, QIX engine version, info about the hardware Sense is running on etc.

Started working on this, targeting 2.2.1 release.


Encourage to the developer to create/switch to a virtual environment

If the chosen Python version is not related to a virtual environment, suggest to the developer to create a new one to avoid problems with global dependencies and possible ones she needs to install.
As next steps, it'd be really nice to allow creating of virtual environments from VSCode directly.

Thanks for the PR, however please create an issue first so we can discuss on whether we want this and how to approach this. We're reducing the number of pop ups which show up at startup, so it's unlikely we'll accept the PR at this state. Maybe we can show it in the problems window: https://github.com/microsoft/vscode/issues/155673.

As next steps, it'd be really nice to allow creating of virtual environments from VSCode directly.

https://github.com/microsoft/vscode-python/issues/19676 may just be what you suggest, feel free to leave your ideas there.

Sure! Thanks for your feedback @karrtikr


Deploy application into AWS

Deploy application into AWS
Acceptance Criteria:

[x] Application gets deployed into AWS dev namespace via Terraform or Cloudformation
[x] Frontend deployed on S3/Cloudfront
[x] Postgres database deployed on Aurora RDS
[x] NestJS API deployed on Fargate ECS
[x] Frontend/Database/API communicate with each other and have accessible public route

Bonus:
Depending on how smoothly this goes we may want to break this into another issue though it would be amazing if this was deployed via our GitHub actions pipeline.

This is completed along with deployment pipeline to AWS via GitHub actions.
Frontend: https://dz37sfd9duckr.cloudfront.net/
Swagger API docs: https://qmxe1tmuig.execute-api.ca-central-1.amazonaws.com/api/docs

app is up! moving this one to done.
@marcellmueller  can you add the swagger doc link to our readme? I couldnt find it so ignore if it's already there


Baro Knob issue

"Push STD" does not work.
It could be nice to enable this function.

Work after FL180
It's by simulator design

Is it possible to override that?
What I'm expecting is that the push knob function is always available, and generally used before crossing TA and TL.

Not sure, due it required rewrite G3000 mod. When WT g3000 realise STD switch function, then i can change know function


Release Cadence

There have been a handful of upstream releases from Twitter. Is there anything blocking a release for 21.8.0?
Going forward, would it be possible to automatically release upon updates from Twitter, assuming everything passes in CI?

Oh, from https://github.com/typelevel/catbird/pull/344#issuecomment-852253444 it looks like maybe Travis is the only person who can release at the moment. Would a PR converting over to sbt-ci-release be welcome?

I'd personally prefer sbt-ci-release, so pushing a version tag will publish a release. Makes life easier and now that the project is hosted in typelevel, it makes sense to have a less centralized approach.

Apologies, just saw this. I can publish a release manually today, and I think we should go ahead and change the coordinates and packaging to org.typelevel as soon as possible so that others can publish. I'd also support switching to sbt-ci-release, although I don't have much experience with it and someone else would need to set it up.

Okay, 21.8.0 is on its way to Maven Central. I probably won't have time to make the packaging and coordinate changes this week, but would be more than happy to review that or an sbt-ci-release PR.

Ok I obviously didn't get to this last week, but I haven't forgotten about it. Hopefully coming soon (again, unless someone else beats me to it).


change 'segmindapi' to 'segmind'.

Let us use pip install segmind and from segmind import xxx to standardise the syntax.

Changed the Package origin and imports from pip install segmindapi to pip install segmind and from segmindapi import xxx to from segmind import xxx


Modify database options

Add cf options.
Modify db open method to enable cf options.

todo:

additional options
options with config file
docs of config options

Issue: #280

Codecov Report
Patch coverage: 100.00% and project coverage change: +0.05 :tada:

Comparison is base (d37d014) 78.14% compared to head (d7e7f7d) 78.19%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #289      +/-   ##
==========================================
+ Coverage   78.14%   78.19%   +0.05%
==========================================
  Files          27       27
  Lines        4008     4018      +10
==========================================
+ Hits         3132     3142      +10
  Misses        876      876

Impacted Files
Coverage Δ

src/storage.rs
68.50% <100.00%> (+1.84%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Change the methods of Duktape to be synchronized.

https://github.com/square/duktape-android/issues/69

How about this, then?


Add support for Yooksmart blinds with inverted reporting percentage

For some reason, they accept a standard percentage for commanding the blinds, but invert the reported percentage.

Codecov Report

Merging #971 (b6c4ffe) into dev (b7e6173) will decrease coverage by 0.00%.
The diff coverage is 80.00%.

@@            Coverage Diff             @@
##              dev     #971      +/-   ##
==========================================
- Coverage   82.55%   82.54%   -0.01%
==========================================
  Files         189      190       +1
  Lines        4798     4813      +15
==========================================
+ Hits         3961     3973      +12
- Misses        837      840       +3

Impacted Files
Coverage Δ

zhaquirks/yooksmart/D10110blinds.py
80.00% <80.00%> (ø)

Continue to review full report at Codecov.

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

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

12 of 15 (80.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.008%) to 82.547%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

zhaquirks/yooksmart/D10110blinds.py
12
15
80.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.008%

Covered Lines:
3973

Relevant Lines:
4813

💛  - Coveralls

These are the motors for Yoolax and Graywinds blinds on Amazon. These are the ZF version.

This change appears to have completely broken my Yoolax shades (which were correctly reporting the state prior to this change). With this change, they believe they are closed when they are open (and vice-versa) leading to them not accepting commands to move from Home Assistant.
I wonder if this behavior is dependent on the firmware version. My shades are Firmware: 0x11006780. @bballwiz5 what version are yours?

Hmm... my firmware is Firmware: 0x11006770 so maybe they did fix it in more recent firmware versions? As far as I know, there is no way to update the firmware on these. Both the Hubitat and SmartThings community drivers invert these percentages too.
I don't believe the quirks are setup to include Firmware version in the signature yet. Maybe we can add that as a PR?

Yeah - it certainly seems firmware dependent. Mine were just manufactured (arrived ~1 week ago). I think if we could add firmware version as part of the signature that would solve this problem for sure. @dmulcahey @Adminiuga is that something that could be added?

Post the signature string from the device (or better log from ZHA reading the basic cluster then paring the device as being added in the quirk signature and not stripped as in this PR) so the devs can looking at it if its gets difference that ZHA can using for selecting the quirk.

Here's the signature for mine.
{
  "node_descriptor": "NodeDescriptor(logical_type=<LogicalType.EndDevice: 2>, complex_descriptor_available=0, user_descriptor_available=0, reserved=0, aps_flags=0, frequency_band=<FrequencyBand.Freq2400MHz: 8>, mac_capability_flags=<MACCapabilityFlags.AllocateAddress: 128>, manufacturer_code=4169, maximum_buffer_size=82, maximum_incoming_transfer_size=82, server_mask=11264, maximum_outgoing_transfer_size=82, descriptor_capability_field=<DescriptorCapability.NONE: 0>, *allocate_address=True, *is_alternate_pan_coordinator=False, *is_coordinator=False, *is_end_device=True, *is_full_function_device=False, *is_mains_powered=False, *is_receiver_on_when_idle=False, *is_router=False, *is_security_capable=False)",
  "endpoints": {
    "1": {
      "profile_id": 260,
      "device_type": "0x0202",
      "in_clusters": [
        "0x0000",
        "0x0001",
        "0x0003",
        "0x0004",
        "0x0005",
        "0x0020",
        "0x0102"
      ],
      "out_clusters": [
        "0x0003",
        "0x0019"
      ]
    }
  },
  "manufacturer": "yooksmart",
  "model": "D10110",
  "class": "zigpy.device.Device"
}

Mine, unfortunately, has the same signature. Seems they updated the software version without changing anything else.
It seems zigpy is getting the firmware version from an OTA update check from the device, then stores it in the HA device registry. Zigpy doesn't store it anywhere itself and does not expose it to quirks. Not sure yet how to connect the two. Probably should open a ticket for this.

You can using local quirks that is overriding the quirk that is in HA #693 (comment).
Download the quirk and do the change in the local quirk folder. The local quirks is not being updated with HA and you need "manager" your self.

The whole yooksmart directory and custom quirk in /zhaquirks didn't exist prior to the August 25th commit, how would I handle removing that file entirely via the custom_zha_quirks directory?

Copy the quirk file (its ending with .py from the commit and doing the changes in it.
Then you is installing it in the local quirk folder is ZHA using it from the local folder and you dont need doing any changes in the HA container.
You cant deleting quirk in HA with one local quirk only overriding the installed ones so do one local quirk without the changes that is braking your device.

Interestingly enough, I made the suggested change and the the directions are back to working normally BUT I still only ever get 0 or 100 out of the position. Now its at 0 when closed, but immediately jumps to 100 as soon as you start to open it. Additionally, if you send a command for it to open/close to a certain percentage, it just closes all the way and goes to 0.
There is definitely still something weird going on.

@HEAT16 - try running a ZHA reconfigure on the device. That's what I had to do to get the position sorted out after swapping out the quirk.


Format the log

Format the log and remove unused codes.

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@4746bcb). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #47   +/-   ##
=======================================
  Coverage        ?   45.57%
=======================================
  Files           ?        6
  Lines           ?      814
  Branches        ?        0
=======================================
  Hits            ?      371
  Misses          ?      414
  Partials        ?       29

Flag
Coverage Δ

unittests
45.57% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


ds004024 misalignment of BOLD and fieldmap

What happened?
Checking w/ @jbwexler. Will record more details as I get them. Screenshot attached.
Possibly related: the fieldmap reportlet had weird coordinate listing
What command did you use?
tacc-openneuro submitter

What version of the software are you running?
fmriprep 23.2.0
How are you running this software?
Singularity
Is your data BIDS valid?
Yes
Are you reusing any previously computed results?
No
Please copy and paste any relevant log output.
No response
Additional information / screenshots

@psadil Does this look like #461 to you?

@jbwexler It's probably too much to hope you can find the dataset that this happened in?

Unfortunately it's long gone as far as I can tell

I could rerun it though if that would be helpful.

I'm not sure. I haven't see doubling of the brain like that. But I have seen cases with fieldmaps that look like what's shown, by which I mean something like "squished into a line".

And the resulting SDC

And FWIW, here's the fieldmap on that case

I haven't see doubling of the brain like that.

I think the screenshot was just during the transition.

I think the screenshot was just during the transition.

Oh, of course. Then, yes, this does seem related. I would be curious whether the coregistration works better with the dilated masks.

One aspect of what I was seeing is that the step was highly variable (which I attributed to the sampling_percentage). If there's a way to regenerate the working directory, it could be interesting to see what repeated runs of that coregister node produce


Writing Brainf*** programs to achieve certain tasks.
How do you play your game?
a/d -> move around menus/code
w/s -> change operand
k -> select level/stop simulation
i -> start simulation
j -> show level task
l -> go back to level selection
Code
Check off the items that are true.

[x] The game was made using the Sprig editor.
[x] The game is placed in the in the /games directory.
[x] The code is significantly different from all other games in the Sprig gallery (except for games labeled "demo").
[x] The game runs without errors.
[x] The name of the file/game contains only alphanumeric characters, -s, or _s.
[x] The game name is not the same as the others from gallery

Image (If an image is used)

[x] The image is in the /games/img directory.
[x] The name of the image matches the name of your file.

Thanks for your PR!

This is actually pretty impressive.  I did not play deeply but I totally get what the author was going for.  Great job!  If you're a teen, and you haven't already received a Sprig device, please fill out this form to receive one!  Also, feel free to join Hack Club's Slack and check out the #sprig channel!

Thanks a ton!
I'll be filling out the form rn :)


fixed bug in shifted instantaneous rate peak (was not centered at origin)

Fixes #360
clamp instantaneous rates to zero from below, because a convolution of non-negative vectors is non-negative by definition, and the warning is not reachable. Previously, the user saw annoyingly small negative values due to the FFT.

Coverage increased (+0.02%) to 89.918% when pulling 25812aef2ffbd8826d9962b0e67742d8218bce13 on INM-6:fix/instant_rate into 11a9e12b749b6e614d4592d4b0fa8757b4f71169 on NeuralEnsemble:master.

Differences with the previous implementation for a gaussian kernel
Typical cases (small sampling period)

Non typical cases (large sampling period)

What happens with the kernel sigma is small and the sampling period is large?

The first obvious observation is that the previous implementation results in a zero vector estimate. But why now it's overestimated?
AlphaKernel convolved with a single spike at the origin


EntryToken example returning an error upon attempting to unlock

When running the TokenScript for our EntryToken example on Ethereum mainnet: 0x63cCEF733a093E5Bd773b41C96D3eCE361464942, it displays this after attempting to unlock after signing. Retrieving the challenge seems to work correctly:
Failed with: {"timestamp":1577940453791,"status":500,"error":"Internal Server Error","exception":"org.web3j.ens.EnsResolutionException","message":"Unable to determine sync status of node","path":"/api/checkSignature"}

0x63cCEF733a093E5Bd773b41C96D3eCE361464942.xml

0x63cCEF733a093E5Bd773b41C96D3eCE361464942.xml


Org Sort doesn't seem to do anything, no docs!

Describe the bug
When I launch Org Sort List it doesn't do anything! And I couldn't find anything in the docs to explain what it should do.
To Reproduce

Put the cursor in a header
Select Org Sort List from the command palette
Nothing happens!

Expected behavior
Like emacs I expected to be presented with a choice of how I want my items sorted
Screenshots

Platform:
PC Win 11 ST4
** Sublime debug console output**
None
Additional context
None

See this section of the Org Manual: https://www.gnu.org/software/emacs/manual/html_node/org/Structure-Editing.html#index-C_002dc-_005e

C-c ^ (org-sort)
Sort same-level entries. When there is an active region, all entries in the region are sorted. Otherwise the children of the current headline are sorted. The command prompts for the sorting method, which can be alphabetically, numerically, by time—first timestamp with active preferred, creation time, scheduled time, deadline time—by priority, by TODO keyword—in the sequence the keywords have been defined in the setup—or by the value of a property. Reverse sorting is possible as well. You can also supply your own function to extract the sorting key. With a C-u prefix, sorting is case-sensitive.

There are some docs now!
https://github.com/ihdavids/orgextended_docs/blob/master/lists.org#sorting-lists
Thanks for using it. Sadly that command only works on lists not headings.
Even more annoying is this is not org mode. I can't even begin to approach that system with the many devs contributing to its awesomeness. This is just my personal little immitation.
If you would like to expand the command to work with headings and be a little more DWIM I would certainly appreciate the help.


[WIP] Fix copy for ask and tel.

Hello @chschroeder! Thanks for submitting the PR.

In the file skopt/tests/test_optimizer.py, following are the PEP8 issues :

Line 200:70: W291 trailing whitespace


How does translation work?

I've found translation.json file and changed it to my language, but changes are not seen.
What am I doing wrong?
This is my translation.json file:
{
  "Today#!1": "Danas",
  "Tomorrow#!2": "Sutra",
  "星期日#!3": "Nedjelja",
  "星期一#!4": "Ponedjeljak",
  "星期二#!5": "Utorak",
  "星期三#!6": "Srijeda",
  "星期四#!7": "Četvrtak",
  "星期五#!8": "Petak",
  "星期六#!9": "Subota",
  "%M月%D日 %W#!10": "",
  "添加待办事项#!11": "Add to do",
  "昨天#!12": "Jučer",
  "前天#!13": "Prekjučer",
  "本%W#!14": "",
  "上%W#!15": "",
  "周日#!16": "Nedjelja",
  "周一#!17": "Ponedjeljak",
  "周二#!18": "Utorak",
  "周三#!19": "Srijeda",
  "周四#!20": "Četvrtak",
  "周五#!21": "Petak",
  "周六#!22": "Subota",
  "%D天前#!23": "% dana prije",
  "很久以前#!24": "mnogo prije",
  "待办事项#!25": "za učiniti",
  "已办事项#!26": "je bilo učinjeno",
  "上一页#!27": "Prošla stranica",
  "下一页#!28": "Sljedeća stranica",
  "更新 %V#!29": "Update",
  "下载中 %R#!30": "downloading",
  "暂停中 %R#!31": "Suspended",
  "下载完成#!32": "download complete",
  "今天 %T#!33": "Danas"
}

Also, can you please do something about letters [čćđž]?
Apparently - current font doesn't support them or smth. because they fall back to other font.

Hi,

translation.json must be put in the storage folder, not the resources. I think this might be the reason which cause your issue.
You can add a file named custom.css in the storage folder with content below:

body {
  /* you can change the font to any other font installed in your computer */
  font-family: sans-serif;
}

Thanks for reply. I've fixed it.
I'd be willing to make a PR if you'd merge translation to Croatian language.


setting version for kafka

Description
Setting Kafka to earlier version.
Issue reference
We strive to have all PRs being opened based on an issue, where the problem or feature have been discussed prior to implementation.
Please reference the issue this PR will close: #[issue number]
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

[ ] The quickstart code compiles correctly
[ ] You've tested new builds of the quickstart if you changed quickstart code
[ ] You've updated the quickstart's README if necessary
[ ] If you have changed the steps for a quickstart be sure that you have updated the automated validation accordingly. All of our quickstarts have annotations that allow them to be executed automatically as code. For more information see mechanical-markdown. For user guide with examples see Examples.

@pravinpushkar why can't we use latest kafka?

@pravinpushkar why can't we use latest kafka?

@yaron2 Not completely sure as of now, but looks like we need to do some change in our kafka setup steps in README. The latest kafka does not comes with zookeeper by default and also it has SASL auth enabled by default. So, probably something related to this as our app sidecar could not connect with Kafka saying "client has run out of available brokers".


Codeception improvements

A more full featured codeception plugin. Parses the XML return from codeception test results and uses some of them in the display. Stores all of them in the meta table.

I've removed the bootstrap changes. The reason they were in there was because the original file seemed to have omitted part of the bootstrap package that did the accordion stuff I use. It seems that's now in there and it works.
I've added the changes you mention above. It seems to work fine now.

Hey @cooperaj - I'm really sorry this has taken such a crazy amount of time to get merged.
Could I ask you once again to update these changes to the latest master branch? I promise we'll get it merged ASAP after that!

Changes as requested.

@cooperaj can you rebase it (and ping a maintainer if done to get it merged)

You'll have to clarify rebase I'm afraid. What am I rebasing onto what?

Rebased as requested. With translation fixes.

@dancryer @tvbeek @mavimo I think thats it done now. Passes and everything.

@mavimo Added the Lang call back in.

Once this is in I think I'm going to abstract out the XML parsing to a more generic way of doing it so I can implement an xUnitParser plugin. That way I can have my tests run as part of a build process and use the plugin to show the results.

@dancryer @tvbeek @mavimo Any news on this? Sorry to pester.

I check it, can you do a rebase? Then it is good

Another one? At 17 commits it takes a good half hour since I get merge
conflicts on every commit. Unless I’m doing something wrong?
On Wed, 22 Apr 2015 at 14:42 Tobias van Beek<EMAIL_ADDRESS>wrote:

I check it, can you do a rebase? Then it is good
—
Reply to this email directly or view it on GitHub
https://github.com/Block8/PHPCI/pull/588#issuecomment-95182168.

As far as I see you only get a couple of merge conflicts. But if that is done I can give the ready-to-merge flag (or someone else) and it can be pulled to master

@tvbeek @mavimo @dancryer Rebased as requested.

@cooperaj Thanks for the create work you have done

@tvbeek Aww crap. This may need rolling back. Looking through the commit that went in tells me that the bootstrap.css files that I swear I reverted still sneaked through. I'm not sure whats happened but not pulling it out will leave a 3.2 bootstrap css file in there.

hmm, @cooperaj can you create a new pull request that fixed it? Because it is more easy to do that then to revert a request

Will do.

@tvbeek Done as part of PR #935

Thanks, it's merged


feat: Accept "string only" versions

Loosens up our version related things so we can use string based versions if we need to. The rule it uses is "if major doesn't parse as a number, then its a 'string version'". String versions will never ask for updates as they are more likely to be dev versions and suchlike.
I switched a couple of old store-type things to use env vars whilst I was here.
The entire thing assumes that we never deal with actual full semvers, only 3 numbers i.e. we never need to be aware of carets and tildes, only 3 numbers joined by dots ~3.0.0 vs 3.0.0.
This PR makes it so we only need to be aware of "3 numbers joined by dots" and "full strings" - we still don't need to be aware of ~ or ^ prefixed semvers.
Signed-off-by: John Cowen<EMAIL_ADDRESS>

Gonna merge this shortly, we can always come back and revisit in a follow up if we need to tweak slightly.


package not working with<EMAIL_ADDRESS>

The global declaration of Element {
host: any
}
is conflicting with the new typescript version(3.0.1) where host is of string type.
Getting the following errors while using this package
/node_modules/typescript/lib/lib.dom.d.ts:5547:11 - error TS2320: Interface 'HTMLAnchorElement' cannot simultaneously extend types 'HTMLElement' and 'HTMLHyperlinkElementUtils'.
Named property 'host' of types 'HTMLElement' and 'HTMLHyperlinkElementUtils' are not identical.
/node_modules/typescript/lib/lib.dom.d.ts:5658:11 - error TS2320: Interface 'HTMLAreaElement' cannot simultaneously extend types 'HTMLElement' and 'HTMLHyperlinkElementUtils'.
Named property 'host' of types 'HTMLElement' and 'HTMLHyperlinkElementUtils' are not identical.

I don't know why this haven't shown up on the TS tests done on this repo, as well as on scroll-into-view-if-needed and scroll-into-view-if-needed-website, but I have created a PR to attempt fixing it by removing this global Element.host definition.

Looks like it's because none of the tests are testing a <a> element, that would make sense 🤔

:tada: This issue has been resolved in version 1.0.8 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Fixed a bugs that lvm does not support repetition volume/snapshot name

What this PR does / why we need it:
Fixed a bugs that lvm does not support repetition volume/snapshot name
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Fixed issue: #304
Special notes for your reviewer:
Release note:

Coverage increased (+0.03%) to 56.713% when pulling 4bd9b8241be7ae3a3e6e45a14c60e13c985d317e on xxwjj:lvm_bugs into 3bce85f0a05cf17a463673fbb7ac17e963883808 on opensds:development.

Coverage increased (+0.03%) to 56.713% when pulling 4bd9b8241be7ae3a3e6e45a14c60e13c985d317e on xxwjj:lvm_bugs into 3bce85f0a05cf17a463673fbb7ac17e963883808 on opensds:development.

Coverage increased (+0.03%) to 56.713% when pulling 4bd9b8241be7ae3a3e6e45a14c60e13c985d317e on xxwjj:lvm_bugs into 3bce85f0a05cf17a463673fbb7ac17e963883808 on opensds:development.

Coverage increased (+0.03%) to 56.713% when pulling 4bd9b8241be7ae3a3e6e45a14c60e13c985d317e on xxwjj:lvm_bugs into 3bce85f0a05cf17a463673fbb7ac17e963883808 on opensds:development.

Why do you define volumePrefix volume_ but snapshotPrefix _snapshot-?

@leonwanghui there will be bugs if you use the snapshot as the snapshot prefix.

Ok


npc indicators: add server tile and outline width

Added a server tile option and a stroke width config

We are not interested in adding "true position" highlighting (see #8702), however a customizable outline size would be acceptable. If you want to remove the former feature, we'd be interested in adding that.

My bad I didn't see that one. I can't update it right now but I'll try to send the update in tomorrow.

My branch got kind of broken but I think that works. not too certain though


[Snyk] Upgrade @mdx-js/mdx from 1.6.19 to 1.6.22

Snyk has created this PR to upgrade @mdx-js/mdx from 1.6.19 to 1.6.22.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 3 versions ahead of your current version.
The recommended version was released 3 months ago, on 2020-12-01.

Release notes

Package name: @mdx-js/mdx

1.6.22 - 2020-12-01
Bump dependencies
Update documentation

1.6.21 - 2020-11-07v1.6.21

1.6.20 - 2020-11-07v1.6.20

1.6.19 - 2020-10-20v1.6.19

    from @mdx-js/mdx GitHub release notes

Commit messages

Package name: @mdx-js/mdx

510bae2 v1.6.22
6c1bff1 chore(deps): update dependency eslint-plugin-react to v7.21.5 (#1288)
083e3fd chore(deps): update dependency eslint-config-prettier to v6.15.0 (#1282)
b5c31c1 chore(deps): update dependency preact-render-to-string to v5.1.12 (#1310)
a5941f0 fix(deps): update dependency mdast-util-to-hast to v10 (#1293)
5b7fb73 chore(deps): update dependency eslint-plugin-import to v2.22.1 (#1287)
cd0843f chore(deps): update dependency eslint to v7.14.0 (#1281)
0c66a86 chore(deps): update dependency eslint-config-xo to v0.33.1 (#1303)
60824ea chore(deps): update dependency @ vue/test-utils to v1.1.1 (#1321)
15922c8 chore(deps): update babel monorepo (#1301)
4e93b8d chore(deps): update dependency remark-preset-wooorm to v8 (#1306)
22e6271 chore(deps): update dependency hast-util-select to v4.0.1 (#1322)
8749cb8 chore(deps): update dependency yarn-deduplicate to v3.1.0 (#1323)
d586a84 Add mdnext link
48642e9 Add MDNEXT to projects page (#1309)
02e7c8c fix(deps): update dependency @ emotion/core to v10.1.1 (#1324)
83e6e81 chore(deps): update dependency preact to v10.5.7 (#1334)
7a9476c fix(deps): update dependency @ theme-ui/components to v0.3.4 (#1335)
d158484 fix(deps): update dependency detab to v2.0.4 (#1336)
c355547 fix(deps): update dependency gatsby-plugin-theme-ui to v0.3.4 (#1337)
3925219 Correct descriptions/tags in Table of components (#1331)
ed19998 Fix `delete` -> `del` in docs on typescript
889eb4b Update `typescript.mdx`
97ec1a8 v1.6.21

Compare

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

C'est une arnaque
Le ven. 19 mars 2021 à 22:33, John P Vajda @.***> a
écrit :

@.**** requested changes on this pull request.
there are conflicts that need to be resolved
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/newrelic/opensource-website/pull/708#pullrequestreview-616807383,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AS3GLCVWH3AK4UPOJUIFEXTTEO7MFANCNFSM4ZHJD43A
.

I had to mess with the lockfile to get this to resolve correctly. Theres a dependency that is relying on a dependency which is now outdated. The solution was to run npm install --save --legacy-peer-deps.
If someone could take review this (to double check it looks right), that would be great.

@zstix I did a quick visual check on the amplify preview link, site seems fine to me. That's a huge change but I assume it's due to the npm command you had to run.


Duplicate Contributors on README

What file is affected?
The README has duplicate contributors, it seems the all-contributors bot appears to have a bug.

Additional information

Thanks for letting us know @Ruth-ikegah!

Is there a way to solve this. I will love to take it up!

Is there a way to solve this. I will love to take it up! @janiceilene

ohh i see someone has solved it!

I will close it then!


Fix release notes build failure and PR preview

Moved deconst-requirements.txt and _deconst.json files to the source
directory
Update content ID in  _deconst.json
Change to single page template build in conf.py
Add oslosphinx to deconst-requirements.txt

Addresses https://github.com/rackerlabs/nexus-control/issues/555

:eyes: Preview

@meker12 it works! :)

@git-harry @b3rnard0 can we merge this PR? Thanks!

@git-harry should this have a swimlane? It's in needs-review

👍  LGTM

👍 Thanks @darrenchan


Zoom to Graphic Extent after Graphic Created

after creating a graphic, the map should be zoomed to graphic extent by default. If the user is drawing within their current extent, they can see their graphic being created. If they are manually inputting parameters and those parameters fall outside their current extent, they should be automatically taken to the graphics extent.

@mcalamito, please verify with next Jenkins build.

Testing Procedure:

Zoom into California (Monterey area).
Enter lines/circles/ellipses/range rings around 13SCU7353073185 (Kirtland AFB, Albuquerque, NM)
Does it zoom?

[ ] ArcMap 10.3.1
[ ] ArcMap 10.4.1
[ ] ArcMap 10.5
[ ] ArcGIS Pro 1.4


#10874: enable initial line allgather testing in TG frequent pipelines

Ticket
#10874
Problem description
We don't have dedicated CCL (in this case line all-gather) testing for TG in our CI pipelines. This PR adds some initial tests to the frequent pipeline.
What's changed
Added several line all-gather tests to TG frequent pipeline
Checklist

[x] Post commit CI passes: https://github.com/tenstorrent/tt-metal/actions/runs/10380932125
[ ] tg frequent: https://github.com/tenstorrent/tt-metal/actions/runs/10380918054
[ ] tg unit: https://github.com/tenstorrent/tt-metal/actions/runs/10390337203
[x] Blackhole Post commit (if applicable)
[x] Model regression CI testing passes (if applicable)
[x] New/Existing tests provide coverage for changes

Note that the frequent pipeline is currently timing out due to several failing tests. All tests are passing locally:

@tt-rkim is there an alternative place I can put this test list in the meantime?


New file placeholder should not have an icon

VSCode Version: Code 1.14.0 (b8cd5b9556d8b70ea560d35b903422363f6c5c40, 2017-07-10T18:36:31.751Z)
OS Version: Windows_NT ia32 10.0.14393
Extensions:

Extension
Author (truncated)
Version

vue
liu
0.1.5

csharp
ms-
1.11.0

sublime-keybindings
ms-
2.9.1

debugger-for-chrome
msj
3.1.6

vetur
oct
0.9.1

Steps to Reproduce:

Click New File
Before naming the file, click on the icon to the left of the file name input field (if it's not meant to be a "move me" icon, it certainly looks like one) and drag to move the pending file.
Instead of moving the pending new file, VS Code will abort new file creation and instead move the file that was in that spot before "New File" was clicked.

Optional additional step: User is very confused.

Reproduces without extensions: Yes/No

@bpasero the issue is that the new file placeholder gets an icon, is it possible to specify that a tree entry does not get an icon? Where do you add these icons in the explorer?

@isidorn I disagree that we should not show an icon for the new-file placeholder, because you will notice that while you type the extension, the icon updates and reflects the file type.
I suggest to restore the previous title of this issue because that seems like the issue to resolve (e.g. do not close the input box if the user is clicking still somewhere into the row of the tree item.

I'd like to try to fix this. Is PR welcome or someone is already working on this?

@4O4 a PR is welcome
Code pointer https://github.com/Microsoft/vscode/blob/master/src/vs/workbench/parts/files/browser/views/explorerViewer.ts#L326

Thanks @isidorn , I'm going to work on this then :)

Hi, I'd like to take this up if you are no longer working on it @4O4

@jeanp413 Unfortunately I haven't made any progress with this so feel free to take this 🙂


[Sign-in] When user moved to 'Sign-in' window, appeared scroll bar.

Environment: Firefox 121.0 (64-bit.
Reproducible: always.
Build found: last commit from https://github.com/ita-social-projects/GreenCityClient/tree/dev
E.g. "commit 147fa44"
Preconditions
Moved to https://www.greencity.social/#/greenCity
Steps to reproduce

Click on 'Sign-in' bttn.

Actual result
When user moved to 'Sign-in' window, appeared scroll bar.
Expected result
Scroll bar must be appeared only when user make scalking-up.
Postconditions
A clear and concise description of actions to be done after bug reproducing. E.g.:
"Delete created item from database using SQL query:
DELETE FROM table1
WHERE id=<item1_id>;"
Labels to be added
"Bug", Priority ("pri: "), Severity ("severity:"), Type ("UI, "Functional"), "API" (for back-end bugs).

The bug is not fixed:
When an user navigates to the 'Sign in' window, a scroll bar still appears

The bug keeps reproduce on Mozilla 124.0.2 (64-bit)

The bug keeps reproduce on Chrome 127.0.6533.100 (64-bit)

Added a fix to the login and registration modal so that the scrollbar is displayed only on a small screen when necessary
Pull Request: https://github.com/ita-social-projects/GreenCityClient/pull/3272
Result:
Normal size of window:

Small size of window:


custom nameResolver could ends as error

Brief bug description
When I run the model generator with codename _3_section_items and using nameResolver as asny value camelCase, pascalCase, snakeCase I get an error.
camelCase or pascalCase
SyntaxError: An identifier or keyword cannot immediately follow a numeric literal. (8:6)
   6 | */
   7 | export type MainMenu = IContentItem<{
>  8 |     3SectionItems: Elements.LinkedItemsElement<IContentItem>;

snakeCase
SyntaxError: Numeric separators are not allowed here. (8:6)
     7 | export type MainMenu = IContentItem<{
>  8 |     3_section_items: Elements.LinkedItemsElement<IContentItem>;

Usage:
    await generateModelsAsync({
        sdkType: 'delivery',
        projectId: process.env.KONTENT_PROJECT_ID,
        addTimestamp: true,
        nameResolver: 'camelCase',
    })

Expected behavior
The model generator does not fail and generates the codename with some prefix.
Additional context
Generation could also fail with a more polished error.

This has been fixed in the name resolvers in delivery SDK. There is also a test for this specific name (starting with underscore - name )


A new solution for Android Developers!

Unfortunately this library is no longer maintained by the creator. But.
I just released the 1.0.0 version of my own react-native library that includes AdMob modules.
My code is based on this lib, but i made some refactoring, bug fixes and adaptations for Androidx libs and new react-native framework(0.60.3).
My library is for Android Only, so it is not a solution for IOS :/
If you are developing a react native App for Android Only, check my lib:
https://github.com/MatheusAlvesA/react-native-androide/

#399
#462

Your fork doesn't work for me, using RN 0.60.5

Issues regarding new versions of React Native (and thus AndroidX) are being addressed with #472


Download all content of a user(add new feature)

Please put a solution in the program to download all the posts, including photos and videos of other users, so that when there is no access to the Internet or when a person has sent educational materials, etc., all the contents of the person can be downloaded in full.

What is the reason for doing it with a mobile app?
That's a request minor enough to feel complicated.
I believe that request use PC and the network environment that cost not depends on the transfer amount.

also, Since ST handles so many columns, it omits even saving the posts themselves in the in-app DB.

I think it's not job for a normal mobile client app.
2023年7月23日(日) 20:17 Wjxfi @.***>:

I really need that, too
—
Reply to this email directly, view it on GitHub
https://github.com/tateisu/SubwayTooter/issues/245#issuecomment-1646813069,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACRQ6A7LRCRLRPLYCN4NKTXRUB5DANCNFSM6AAAAAAYIYHIP4
.

So I haven't looked into existing tools.

This kind of scraping tool will always be criticized, which puts a heavy burden on the person who published the tool.


Error: Not a valid CRS

Hi,
I'm using sentinelhub.BboxSplitter to split my AOI, and I have an issue with providing a coordinate reference system to the function. I get the following error:
ValueError: CRS.from_wkt('PROJCS["GDA_1994_Albers",GEOGCS["GDA94",DATUM["Geocentric_Datum_of_Australia_1994",SPHEROID["GRS 1980",6378137,298.257222101004,AUTHORITY["EPSG","7019"]],AUTHORITY["EPSG","6283"]],PRIMEM["Greenwich",0],UNIT["degree",0.0174532925199433,AUTHORITY["EPSG","9122"]],AUTHORITY["EPSG","4283"]],PROJECTION["Albers_Conic_Equal_Area"],PARAMETER["latitude_of_center",0],PARAMETER["longitude_of_center",132],PARAMETER["standard_parallel_1",-18],PARAMETER["standard_parallel_2",-36],PARAMETER["false_easting",0],PARAMETER["false_northing",0],UNIT["metre",1,AUTHORITY["EPSG","9001"]],AXIS["Easting",EAST],AXIS["Northing",NORTH]]') is not a valid CRS
I'm not sure what CRS the function is expecting?
I'm using version 2.6.1.

Hello,
At the moment sentinelhub-py supports WGS84, POP_WEB and UTM coordinate systems, as described in the documentation here.
If you convert your geometry to one of the supported CRS it should work correctly.

Thanks, I was a bit slow to get that part in the documentation. I converted to UTM 55S and it works fine.


JsSerializeScript

Hi Limin,
When trying to serialize a script, I am getting an error code "CannotSerializeDebugScript". The sample is uploaded @ https://onedrive.live.com/redir?resid=9C4512A9CBBBEB98!115&authkey=!AM81bn-hC92WIDQ&ithint=file%2Czip .
Could you please help me with this?
Regards,
Srinivas.

Sure I'm here to help @SrinivasGourarum. Basically we have slightly different bytecode for normal script and script in debug mode so we disable serialization in debug mode (otherwise your serialized bytecode in debug mode would not run normally later). Comment out or delete JsStartDebugging call and you should be fine.

Since we do not know the size of the buffer to be created, should we first call JsSerializeScript to get the size of the buffer and again calling the same api to load the buffer as in the below code?
   ulong buffSize = 0;
   byte[] buff = null;
   JavaScriptErrorCode errCode = Native.JsSerializeScript(script2Serialize, buff, ref buffSize);
   if (errCode == JavaScriptErrorCode.NoError)
   {
        buff = new byte[buffSize];
        errCode = Native.JsSerializeScript(script2Serialize, buff, ref buffSize);
   }

Yes your understanding is correct. If you don't have a big enough buffer when you call JsSerializeScript, the function would return the correct buffer size. Since there is no way of knowing how large the buffer needs to be at first, in practice, you usually call JsSerializeScript twice, the first time figuring out the buffer size.
We've done similar thing in our ch.exe host (note it uses ChakraCore not Chakra but it's the same idea).

Thanks Limin.
Did you get a chance to look at https://github.com/Microsoft/Chakra-Samples/issues/16? It has been long pending :(.


Upgrade cereal to newest version?

Hello
A new minor version of cereal has been released: <IP_ADDRESS>.
In order for a package of mine (bitcoin-payment-channel) to be accepted into the stackage repo, murmur3 - being a haskoin-core dependency - needs an update to use the newest cereal: https://github.com/fpco/stackage/pull/1750#issuecomment-236729636
Would it be possible for you to release a new version of murmur3 on Hackage, with the cereal dependency updated to accept <IP_ADDRESS>?

Hi, sure. I've uploaded murmur3-1.0.3 with relaxed dependencies.

Perfect! Thank you sir.


[HUDI-4508] Repair the exception when reading optimized query for mor in hive and presto/trino

[HUDI-4508] Repair the exception when reading optimized query for mor in hive and presto/trino, if the fileSlice with log file has not base file.
Tips

Thank you very much for contributing to Apache Hudi.
Please review https://hudi.apache.org/contribute/how-to-contribute before opening a pull request.

What is the purpose of the pull request
(For example: This pull request adds quick-start document.)
Brief change log
(for example:)

Modify AnnotationLocation checkstyle rule in checkstyle.xml

Verify this pull request
(Please pick either of the following options)
This pull request is a trivial rework / code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end.
Added HoodieClientWriteTest to verify the change.
Manually verified the change by running a job locally.

Committer checklist

[ ] Has a corresponding JIRA in PR title & commit

[ ] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

@5herhom can you please update PR description so that it follows the template and clearly states the purpose of this PR?

@5herhom can you please update PR description so that it follows the template and clearly states the purpose of this PR?

Ok, I update more detail in "What is the purpose of the pull request' in RP description"

@hudi-bot run azure


Cleanup configuration of bazel RBE jobs

Followup for https://github.com/grpc/grpc/pull/29353

cleanup and simplify .cfg and .sh files for linux bazel RBE jobs (make use of env variables instead of helper .sh scripts)
enable uploading of XML report artifacts for selected bazel jobs (the idea is to enable the bazel_report_helper.py reports to show up in resultstore/sponge).
use "respawn in moved src tree" for linux bazel RBE jobs (to speed up job finalization).

Bazel C++ iOS bazel report: https://source.cloud.google.com/results/invocations/303bc042-6e79-4ea4-9d3f-233aac2995f0/targets (from adhoc test run).

@lidizheng  this is now ready for review.

LGTM to the script structure changes. Can you also ask a C++ team member to review? So they will be aware of the change.

CC @veblush  for visibility.
I feel like this change is pretty noncontroversial (it's just moving config args around), so I don't think we need extra pair of eyes to formally review.
As a side effect, this change also fixes the problem introduced by https://github.com/grpc/grpc/pull/29353 (thus superseding https://github.com/grpc/grpc/pull/29387),
and since this change is already approved, I think I'm going to merge once I doublecheck that it actually fixes the broken tests on master.

Adhoc run of grpc/core/master/linux/bazel_rbe/grpc_bazel_rbe_opt job (to see if this PR fixes the breakage currently on master): https://source.cloud.google.com/results/invocations/fdb3c227-6f80-4cfb-8d44-0438c4291329

Merging to fix b/229025694


⚠ Samsung has degraded performance

In 1cb82e7, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2464 ms

Resolved: Samsung performance has improved in 73ad586.


Replace the primary button color "green" to "cod"

Cod #141414
https://zpl.io/ag7YvpO
https://zpl.io/AmkONme

Dark Mode
https://zpl.io/ag7YvpO
https://zpl.io/GEkne6r

@hboon Would this be a good place to suggest a UIKitFactory?

@hboon Would this be a good place to suggest a UIKitFactory?

Nope. Changing the 2 button colors should be quite easy currently.

So do you want light/dark mode to be implemented as well? @hboon

Separately. This  button color change #3843 and then #3819 should be a quick change, then Dark mode after some refactoring

I can do this with dark/light mode with just one extra line…

Nah, better to ship this first. Dark mode is a bigger change. Could have complications.

I can do this with dark/light mode with just one extra line…

Also, functionality-wise, they are separate features/changes hence separate PRs.

Is there a inactive dark theme colour for foreground and text of buttons? @colourfreak @hboon

Primary - Light https://zpl.io/ag7YvpO
Secondary - Light https://zpl.io/V1gKq1w
Primary - Dark https://zpl.io/aRA5Y8e
Secondary - Dark https://zpl.io/bemqYpp

I'm asking for Dark Inactive primary and Dark inactive secondary. Like https://zpl.io/2vQJYAv but for Dark.

Alright.
Primary Active

Light
https://zpl.io/ag7YvpO
Dark
https://zpl.io/aRA5Y8e

Primary Inactive

Light
https://zpl.io/MdkpW5K
Dark
https://zpl.io/5EkpJe3

Secondary Active

Light
https://zpl.io/V1gKq1w
Dark
https://zpl.io/bemqYpp

Secondary Inactive

Light
https://zpl.io/1ykQOeX
Dark
https://zpl.io/RMgzxon


[Core] Fix resolution of client_arg_name

Fix #5509.

This checklist is used to make sure that common guidelines for a pull request are followed.
General Guidelines

[X] The PR has modified HISTORY.rst describing any customer-facing, functional changes. Note that this does not include changes only to help content. (see Modifying change log).

Command Guidelines

[N/A] Each command and parameter has a meaningful description.
[N/A] Each new command has a test.

(see Authoring Command Modules)

View a preview at https://prompt.ws/r/Azure/azure-cli/5553
This is an experimental preview for @microsoft users.

This looks good but it's making an assumption that my custom command handlers will be in custom.py.
I believe there may be cases where the custom handlers are spread over multiple files (?).
Is there an approach that doesn't have this assumption?

@derekbekoe that's a great point (and why I don't like when custom commands are spread over multiple files).  I'll try another approach.

@derekbekoe please take another look at the updated heuristic.  Should work if extension developers adhere to our guidelines.

Verified using the subscription extension that this fixes the issue.


EventHubs tracing: don't modify unmodifiable app properties

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


DRAFT - Adding ASA and AireOS support

Description
Adding Cisco ASA and Cisco AireOS support to scrapli community drivers.
NOTE: I've put the reasoning for my naming in #86 . As I said, happy to change the naming if you want.
Type of change

[x] New feature (non-breaking change which adds functionality)
[x] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also
list any relevant details for your test configuration
Checklist:

[x] My code follows the style guidelines of this project (no GitHub actions compalints! run make lint before
committing!)
[x] I have commented my code, pydocstyle and darglint are happy, docstrings are in google docstring format, and all
docstrings include a summary, args, returns and raises fields (even if N/A)
[x] I have added tests that prove my fix is effective or that my feature works, if adding "functional" tests please
note if there are any considerations for the vrnetlab setup
[x] New and existing unit tests pass locally with my changes

Test evidence
(venv) ➜  scrapli_community git:(feature/asa-aireos) make lint
python -m isort .
Skipped 3 files
python -m black .
All done! ✨ 🍰 ✨
111 files left unchanged.
python -m pylama .
python -m pydocstyle .
python -m mypy --strict scrapli_community/
Success: no issues found in 92 source files
(venv) ➜  scrapli_community git:(feature/asa-aireos) make test
python -m pytest \
tests/
=========================================================================================== test session starts ===========================================================================================
platform linux -- Python 3.8.10, pytest-6.2.5, py-1.10.0, pluggy-1.0.0
rootdir: /mnt/c/Users/dfjt/Documents/networking/projects/scrapli_community
plugins: pylama-7.7.1, cov-2.12.1
collected 84 items

tests/unit/aethra/atosnt/test_aethra_atosnt.py ...                                                                                [  3%]
tests/unit/aruba/test_aruba_aoscx.py ...                                                                                [  7%]
tests/unit/cisco/aireos/test_cisco_aireos.py ..                                                                                [  9%]
tests/unit/cisco/asa/test_cisco_asa.py ........                                                                                [ 19%]
tests/unit/cisco/cbs/test_cisco_cbs.py .......                                                                                [ 27%]
tests/unit/edgecore/ecs/test_edgecore_ecs.py ...                                                                                [ 30%]
tests/unit/eltex/esr/test_eltex_esr.py .......                                                                                [ 39%]
tests/unit/fortinet/wlc/test_fortinet_wlc.py ...                                                                                [ 42%]
tests/unit/huawei/vrp/test_huawei_vrp.py .....                                                                                [ 48%]
tests/unit/mikrotik/routeros/test_mikrotik_routeros.py .......                                                                                [ 57%]
tests/unit/nokia/srlinux/test_nokia_srlinux.py ...                                                                                [ 60%]
tests/unit/nokia/sros/test_nokia_sros.py ..........                                                                                [ 72%]
tests/unit/paloalto/panos/test_paloalto_panos.py ......                                                                                [ 79%]
tests/unit/ruckus/fastiron/test_ruckus_fastiron.py ........                                                                                [ 89%]
tests/unit/scrapli/networkdriver/test_scrapli_networkdriver.py ......                                                                                [ 96%]
tests/unit/siemens/roxii/test_siemens_roxii.py ...                                                                                [100%]

=========================================================================================== 84 passed in 0.50s ============================================================================================
(venv) ➜  scrapli_community git:(feature/asa-aireos)

Apologies for the stuff around @carlmontanari , it's now ready to review. I couldn't get this to work for aireos, then re-read the doco relating to auth_bypass and on_open.
Have come up with a solution, but not 100% sure whether its acceptable.
Below is the demo script which you can use if you have access to an ASA or a WLC:
from scrapli import Scrapli
from scrapli.logging import enable_basic_logging

enable_basic_logging(file=True, level="debug")

asa_device = {
    "host": "<your_asa_ip>",
    "auth_username": "<your_username>",
    "auth_password": "<your_password>",
    "auth_strict_key": False,
    "platform": "cisco_asa",
    "channel_log": True,
}

aireos_device = {
    "host": "<your_wlc_ip>",
    "auth_username": "<your_username>",
    "auth_password": "<your_password>",
    "auth_strict_key": False,
    "platform": "cisco_aireos",
    "channel_log": True,
}

with Scrapli(**asa_device) as asa_conn:
    result = asa_conn.send_command("show version")
    print(result.result)

with Scrapli(**aireos_device) as aireos_conn:
    result = aireos_conn.send_command("show inventory")
    print(result.result)

Awesome, looking good!
I had a thought regarding the WLC -- sorry I didn't think of this earlier, I forgot they have that weird auth but not normal auth thing going on ... Wonder if you can use the already built in auth methods in the channel so you don't need to do the sleeping and sending of creds yourself. Check this out (there is a sync flavor as well).
The only downside to doing it with those methods is if/when I ever make this even simpler and turn community into just yaml/json (so it can be used w/ scrapligo as well) then we'd have to figure out some mechanism for doing that (could be a special string that calls this method or something). Buuuuut I can cross that bridge when I get to it 😜 !
So yeah, if you wanna give that a crack cool, if not lemme know and I will merge as is! Either way is good with me!
Thanks for all the work on this!
Carl

hey @carlmontanari, I think we just leave it as is for now and if/when you plan on doing a refactor, I can help with testing the refactored drivers. So if you're happy to merge it, that would be great :)

hey @carlmontanari, I think we just leave it as is for now and if/when you plan on doing a refactor, I can help with testing the refactored drivers. So if you're happy to merge it, that would be great :)

Sounds good to me!


There is a problem. If using apiBaseUrl, the domain name should be filled in, but error 404 was reported. Please fill in the /v1 Error 401 reported again

Describe the Bug

Where are you running VSCode? (Optional)
Windows
Which OpenAI model are you using? (Optional)
gpt-3.5-turbo
Additional context (Optional)
No response

If using apiBaseUrl, fill in the domain name https://chatgpt.zhangzhiyu.live , error 404, fill in https://chatgpt.zhangzhiyu.live/v1 Error 401 reported again。

I don't understand what 401 means, I have correctly filled in the API KEY

If apiBaseUrl is not used, there will be no problem

It's really strange. I didn't do anything and suddenly it's okay


Manually adding folder to space doesn't refresh space preview

I am forced to reload the application every time I manually add a folder from the native file menu of obsidian.
The file at the root space are appearing immediately, but the folder and the file inside are not displayed.

This is now fixed


Supporting heading-secondary?

The GDS styles have the concept of a secondary heading - typically they appear inside an h1 element as a span with a class of heading-secondary before the text of the primary title.  They are styled grey and in a smaller font size than the primary title.
This can be seen clearly on this page:
https://govuk-elements.herokuapp.com/typography/example-typography/
On this page the secondary heading is "Typography", with the primary title as "Example: Typography".
The heading-secondary class varies it's size depending upon the class given to the heading element itself.  There are only definitions present for .heading-large .heading-secondary and .heading-xlarge .heading-secondary.
It would be good to support these secondary headings - ideally with them being contextually aware and automatically picking appropriate font sizes depending on the size given to their parent.

Discussion in the office between Jon, Ali and myself is that perhaps this should be done by having the Header element support a secondaryHeading prop (or similar).

Arguably a better way to do this would be to use context, allowing a HeadingSecondary component to be aware of the Heading type element it's placed within - although it looks like a proper/decent way of doing this won't be coming until React 16.3.  See: https://reactjs.org/docs/context.html


chore: upgrade to storybook v7

fix(calendars): make controlled props optional
chore: upgrade to storybook v7

@SocketSecurity ignore<EMAIL_ADDRESS>


Patch 1

This is a proposed solution to issue #390
A tags argument is added to KerasLayer.__init__ which is passed to module_v2.load so that a SavedModel with multiple MetaGraphs can identify which MetaGraph to load

@googlebot I signed it!

Thank you Artem for your contribution! Coincidentally as part of the ongoing work to support for Keras, an independent change was recently submitted that's added support for tags (https://github.com/tensorflow/hub/commit/81092be4e1e900473cd008fec50dc49c8af2eed9). Let us know if anything is missing.


CLOWarden Deployment cncf-tags

Prepping configuration data for upcoming deployment of CLOWarden requires the following steps.

For each repo in this org, cncf-tags, note the perms granted by non-CLOWarden means (repo local settings.yml files, manual configuration)
Submit a PR updating https://github.com/cncf-tags/org-admin/blob/main/access-control.yaml to reflect currently granted perms
Re-enable the CLOWarden App on the orgs settings, notify @tegioz to finish out any required deployment steps.
For each repo, remove repo permission settings that are now covered by CLOWarden's  access-control.yaml files submitting a PR to update settings.yml where appropriate.

N.B. Name Changes
In this, the second CNCF deployment of CLOWarden, we want to make CLOWarden's role in access governance clearer.
The name of CLOWarden's configuration repo to org-admin
The name of the configuration file that declares the desired repo access state is access-control.yaml
This has been done to better advertise CLOWarden's role in centralizing governance of repo acess in this org.
On the first CNCF deployment of CLOWarden the repo and file were people and config.yaml

CLOWarden is now in use on cncf-tags.
Access rule changes made by CLOWarden, can be reviewed  here https://clowarden.io/audit/?organization=cncf-tags&page=1


refactor(cat-voices): move api to repository package

Description

Move generated api code from services package to repository as repos should be responsible to IO operations
Moved Storage/Vault/crypto to shared package.
Rename VaultCryptoService to LocalCryptoService as it's just used by Vault but can be user somewhere else too

In melos.yaml this section needs updating:
  build_runner:
    run: |
      melos exec -c 1 \
      --depends-on="build_runner" \
      --ignore="catalyst_voices_services" -- \
          dart run build_runner build --delete-conflicting-outputs
    description: |
      Run `build_runner` in every package which contains the build_runner dependency.
      The catalyst_voices_services is skipped because to run a build_runner there you
      must generate first swagger docs (see related Earthfile).

In justfile this section needs an update:
# Generates gateway services in packages/internal/catalyst_voices_services
generate-gateway-services:
    cd .. && earthly ./catalyst_voices+code-generator --platform=linux/amd64 --save_locally=true

Code looks good, I will validate on my machine then approve

btw, the earthly target to which this justfile section refers no longer exists, we can remove it:
# Test generated gateway services
test-gateway-services:
    cd .. && earthly ./catalyst_voices+test-flutter-code-generator --platform=linux/amd64

In README.md in catalyst_voices_repositories the test-flutter-code-generator target is referenced, we can remove this section from readme too


Question : how to make a real time game for 2 players

Hi,
I am a student in second year of university.
Last year I had a project which was to develop a first website.
I used Wordpress and your chessboardjs.
2 players can play the game.
The dashboard is displayed on a page.
Each game is a post.
Each time the user drop the piece the fen is refreshed and saved on post meta.
It is working but to make the opponent able to see the movement the page is refreshed every x seconds.
I'd like to know if there is another way to use your chessboardjs and make it live to allow players to see the movement in real time?
I read few things on websocket.
Is this a free solution (a student project) ? I do understand I have to install some node.js... buy an IP address. Do you know a free solution that is working with wordpress ?
Best regards,
Thierry

Wordpress is first and foremost a CMS for blog sites. It has, however, advanced to the point that it's possible to make bigger and more complex web sites. And what you are doing is possible, but in my opinion very inefficient. I don't know much about wordpress' internals, but it's written in php and you will most likely have to write your own plugin for it. It was not made with realtime web applications in mind, and I'd recommend writing the system that you want yourself.
For a real-time game php is a bad choice. You could take advantage of javascript and ajax, for most basic use, jQuery could be sufficient. However HTTP requests in essence are not fit for the thing that you are trying to do. You are better off using node.js and sockets for that, or any other server solution that is based on or can use sockets. However I do recommend node.js.
The task is interesting and not extremely hard, but not quite as easy as just plugging in wordpress and dropping a chessboardjs library in...

Hi,
Please don't take it as an advertisement :) .... but if you feel in the mud, take a look here, it is quite old code, not touch since long time, but it could be used for your project.
Cheers