Add optional InitForJob functions for Ruby makers

Set neomake#makers#ft#ruby#project_root_files to find either Gemfile or
Rakefile for a Ruby project.

neomake#makers#ft#ruby#use_bundler(jobinfo) function will always use
bundle exec for the associated maker. This would be used for a maker
(like Rubocop) that breaks if a specific version is not used for a project.
Use as follows:
call neomake#config#set('ft.ruby.rubocop.InitForJob',
    \ function('neomake#makers#ft#ruby#use_bundler'))

neomake#makers#ft#ruby#try_bundler(jobinfo) function will read the project
Gemfile to see if there is a gem line for the maker executable (this is
generally safe, but if the name of the gem is not the same as its
command-line program, use neomake#makers#ft#ruby#use_bundler). If found,
it uses bundle exec for the associated maker.
Use as follows:
call neomake#config#set('ft.ruby.InitForJob',
    \ function('neomake#makers#ft#ruby#try_bundler'))

The two paragraphs talking about the InitForJob functions would be added to the Makers wiki page for Ruby.

Related to #2078.

I also made a quick edit on the suggested text in the wiki because I realize that I didn’t properly document the use of try_bundler.

Rebased from current master.

Is there any chance of this work being completed? I could really use this.

I am not currently using NeoMake (of the various async runners, it’s currently my favourite, but I am not using Vim as heavily as I did a few months ago), but I consider this PR to be complete and working.

Do you think you could take a crack at fixing the failing tests?

@cstyles
Is it working for you, i.e. have you tried it?  (I've just rebased it)

No; I think there’s something wrong with the CI configuration based on what I remember.

The CI error is:
    (354/690) [EXECUTE] Instantiate all default makers
      > SKIP: Skipping haskell - expensive setup
      > [D +1.18]: Failed to get cargo metadata for workspace using 'cargo metadata --no-deps --format-version 1'.
    (354/690) [EXECUTE] (X) Errors when loading makers (per file):
 - autoload/neomake/makers/ft/ruby.vim:
   - Could not load maker 'try_bundler' for filetype ruby: Vim(let):E119: Not enough arguments for function: neomake#makers#ft#ruby#try_bundler
   - Could not load maker 'use_bundler' for filetype ruby: Vim(let):E119: Not enough arguments for function: neomake#makers#ft#ruby#use_bundler
 - autoload/neomake/makers/ft/chef.vim:
   - Could not load maker 'try_bundler' for filetype chef: Vim(let):E119: Not enough arguments for function: neomake#makers#ft#ruby#try_bundler
   - Could not load maker 'use_bundler' for filetype chef: Vim(let):E119: Not enough arguments for function: neomake#makers#ft#ruby#use_bundler
      > /home/neomake/repo/tests/makers.vader:238: Assert 0, "Errors when loading makers (per file):".msg
    > NOTE: 1 error(s) in teardown.

I.e. try_bundler and use_bundler gets recognized as a maker, which should not be the case really.
I think it would be better to control this through (a) generic setting(s) instead.
But in general I am still on the fence to just setup the maker's exe/args manually if required, but I can see that it is nice to have wrappers for this.

But in general I am still on the fence to just setup the maker's exe/args manually if required, but I can see that it is nice to have wrappers for this.

Hi, I've been looking for a way to do this. Do you mean to just override the specific maker in your .vimrc instead?
Like for example, if I wanted to use bundler for my rubocop, I would just copy paste this
https://github.com/neomake/neomake/blob/8805e1547424d165fa909604a370fe21663e89fb/autoload/neomake/makers/ft/ruby.vim#L7-L22
and update it to have
...
        \ 'args': ['bundle exec', '--format', 'emacs', '--force-exclusion', '--display-cop-names'],
...

@corroded see :h neomake-makers-properties (https://github.com/neomake/neomake/blob/5e140db568eb2f7b88e4110cd4b651562e8878ab/doc/neomake.txt#L507-L519)


Original proto files

Can you add original protobuf files in repository?

Sorry, I'd like to only check in generated files, as I'm worried about the legal ramifications of checking in original schemas.
Feel free to shoot me an email to discuss this further.


Skin changer issue

Not workin! it says found for everyting but not injecting

#442 check this and be patient


⚠ Bofrost.at has degraded performance

In 0ad6578, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2267 ms

Resolved: Bofrost.at performance has improved in a5de4a4.


Avoid HDF5 warnings

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Pin libnetcdf != 4.9.1 to avoid HDF5 warnings.

@conda-forge-admin, please rerender


How to update and get specific user

Hi,
I would like to create profile page and for that I need to have the possibility to get a specific user and update this information. I searched functions in FirebaseChatCore dart file but I couldn't found it.
Thanks for your help,

You don't need FirebaseChatCore for that :) just use firebase directly :)


Add Job Cancelling and More Unit Tests

Added:

Job cancelling + unit test
More unit tests for torchx-mcad jobs

Can you also make cluster default to None so the user isn't required to pass any args?
https://github.com/project-codeflare/codeflare-sdk/blob/826630703841fdda12ff624b565faffbe0ac8e33/src/codeflare_sdk/job/jobs.py#L156

/LGTM


once insertting a link and choosing an option it closes:

Once i insert my link for the video and choose my option it just instantly closes python, my chrome drive and chrome version are the sanme (both 99). I have everything in the same folder aswell, im not srue what to do.

also when i try to open chrome driver manually my pc blocks it, maybe thats the problem?

Hello! Can you please run the script in your terminal to see what errors you are getting.

oh my i feel dumb for asking this, but how do i actually do that?

You open your terminal/cmd, drag and drop the tiktok-bot.py file in the terminal/cmd then click "enter"

You open your terminal/cmd, drag and drop the tiktok-bot.py file in the terminal/cmd then click "enter"

I just opens the code in python

I understand, but I want you to run the code via your terminal to see why it's not working for you.

I am just now starting out with python and thought it would be cool to try something like this that is pre-built and all I have to do is look at the code myself to get a better understanding of how the code works. However whenever I try and run it this always comes up, I'm on MacBook and don't know what to do because I am not running windows. Im new to this wouldn't be surprised if I did something wrong LOL

Hello! Looks like you didn't put the chromedriver in the correct path. Install the correct chromedriver, unzip the downloaded zip file, copy and paste the executable chromedriver in the tiktok-bot.py directory.

Hello! Looks like you didn't put the chromedriver in the correct path. Install the correct chromedriver, unzip the downloaded zip file, copy and paste the executable chromedriver in the tiktok-bot.py directory.

Is there anyway you could help me to show me how to do this? Im not sure how to do it, do you have any videos that I could look up, or if you have any you could share so I could have a better understanding of what to do? I don't know how to put the chrome driver in the correct path as well as those other steps you told me to do.

I have a video, but it's not published on youtube or elsewhere and I won't probably publish it. I'm only sending it privately to those who need it. Do you have telegram so I can send it to you?

Yeah I have a telegram, @ElChapoGottaGlocko is my username on telegram. I appreciate all the help :)

J’ai une vidéo, mais elle n’est pas publiée sur youtube ou ailleurs et je ne la publierai probablement pas. Je ne l’envoie qu’en privé à ceux qui en ont besoin. Avez-vous un télégramme pour que je puisse vous l’envoyer?

Hello i have the same problem, can you send me the video? https://t.me/ethan610


Error when debugging in Visual Studio Code

I've been using W&B v0.6.34 successfully for a few months, and I could debug within Visual Studio Code, but I'd get a warning that W&B wouldn't log anything because it hadn't been executed via this:
wandb run python <script>
Warning given was:

wandb: ERROR: Headless mode isn't supported on Windows. If you want to use W&B, please use "wandb run ..."; running normally.

That was fine, as I didn't want to log out when I was debugging anyway, but now after updating to v0.8 I get an error when I try to debug:
wandb: Tracking run with wandb version 0.8.19
wandb: ERROR W&B process (PID 3988) did not respond
wandb: ERROR W&B process failed to launch, see: C:\Users\USER\AppData\Local\Temp\wandb-debug.log

  File "c:\Development\Coding\Test.py", line 778, in main
    wandb.init(project="tester")
  File "C:\Anaconda3\envs\pytorch-gpu\lib\site-packages\wandb\__init__.py", line 1067, in init
    _init_headless(run)
  File "C:\Anaconda3\envs\pytorch-gpu\lib\site-packages\wandb\__init__.py", line 296, in _init_headless
    "W&B process failed to launch, see: {}".format(path))
wandb.run_manager.LaunchError: W&B process failed to launch, see: wandb\debug.log

Am I right in thinking there was a recent change so that it doesn't need to be executed like this
wandb run python <script>
and would work by simply doing:
python <script>
?
If so, I suspect that's why it's failing as it can't successfully attach to the VSC debug process.  How do I get around this please?
Thank you.

I've just tried updating to latest version of W&B and it seems to work fine now.  So not entirely sure what the issue was, but all's good now.  Thanks.


Capstone project

Add mobile menu

Hi @Bornittah,
While you made a great effort in this project, unfortunately, I cannot proceed to review your code.
Invalid Code Review Request
You have submitted a project, that does not include a readme file describing the project. Please be sure to add a descriptive readme file to your project so that you meet the general requirements 📝
Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.


Fix modal enum member names

The interaction type is MODAL_SUBMIT and the interaction response type is MODAL

Thanks advaith!


Sendtoken

how can send token when reciver not have associated with token owner send?

Optimally, the wallet would give the ability to the user to crate a token account if he didn't have one. It's not up to the sender.

i think they should AssociatedTokenAccount for revicer. on my web site i do the same with TransactionInstruction


Automatically Download the Output

Is there a possibilty to configure the script or change something, so that the Output which comes out down below, to automatically download? I want the Output file automatically got created after generating the subititle file.
Thank you.

Hi. If you look at the outputs folder, you can see the subtitle files that are generated after transcription.


[json] Show default value from JSON schema in IntelliSense window

Currently when a JSON schema contains the default keyword, that information is not shown in the IntelliSense window. It would be nice to show that.
Example of default not being shown (in the far-right window):

Example of the default being set when pressing Enter on the completion item:

Reason for this new, proposed logic is that when users write JSON schemas with defaults, they might want to show and use that default even if they do not set the property. AJV has the option useDefaults https://ajv.js.org/#assigning-defaults which means just this, that if a prop is missing that is still taken as a default. As such it would be nice to also show the default keyword for users in IntelliSense even if they do not chose to autocomplete a property.
Note that default thus has two meanings:

Default for when doing auto-complete in VS Code (by pressing Enter).
Default as documentation to the user (in how AJV's useDefaults works). I.e. that default will be used if the property is not set at all.

@aeschli I assume this is not good enough to be be brought into the backlog anyway, even though it has not received 20 upvotes? I think it'd be a great usability feature to show the default.

As you observed, JSON schemas are used in various ways and the default value can have different purposes.
Wouldn't it be better to let the schema author add the relevant information for the hover to the documentation field? Appending the default value info could also be unwanted or a duplication of the information that's already there.
I would rather leave things as they are.

With the risk of duplicating the info, I still think it would be preferred to show it. It would guide users into not writing the defaults in the description, which is error-prone. Users often forget to update descriptions along with if they update the defaults. It also gets rid of having to double-document it in description + with the default keyword. This would save all users time.
Are there valid reasons to document it in description as well?

It would guide users into not writing the defaults in the description, which is error-prone.

But that's not clear from an generated string likedefault: null. Settings have a hierarchy, so if you are in workspace setting, not having the setting will mean the user setting is used, not necessarily null
Also what to do with defaults that are objects or arrays?

In VS Code it has a hierarchy, that is true. Users use schemas for other purposes than the VS Code settings. But if you are talking about the hierarchy, that is equally true for writing defaults in the description. Hierarchy (cascading) is something a user has to be aware of as a quirk of VS Code, at least I feel like that.
I don't see the problem defaults with objects arrays. This is already shown right now when the options pop up. Example for array:
"fieldPatterns": {
          "description": "Some description.",
          "type": "array",
          "default": [],
          "items": { "type": "string" },
          "examples": [["field1", "*version*"]]
        }

This shows:


PyPI tarball lacks license

Please include the license file in your PyPI distribution to make it redistributable.

Released under 0.3, thanks


Installation docs are confusing

Hi
The section of the docs that says
Important: This doesn't work out of the box with Materialize versions lesser than v0.97.8 anymore since they removed leanModal.js.
Instead, add the following files to your project root directory: rbuild.js and materialize-css.js
These files contain leanModal.
However, The two files that don't contain leanModal are the two links above that sentence.
I ended up downloading the wrong ones. I believe this is why the is a comment from Markus Dessler
saying the same.
(Hope this makes sense)
Thanks

Yes, you're definitely not the only one. And I agree.
I was thinking of some variations to make it better but I didn't settle on anything specific.
Then I thought, now with au import available I could first create an importer so that all of this isn't necessary (you'd just do au import aurelia-materialize-bridge and be ready).
But that's just delaying improvement of the docs. So now I will do that first. 😃

@glennpierce Please check the updates docs if you find some time. I hope this is less confusong now. 😃

That looks good to me.

See updated docs


Bug: Rust build failed on Windows PC

Describe the bug
My PC with i5-10500 CPU, does not compile properly with rust bindings (using default features).
This pull may solve some problems.
build.log
Steps to reproduce
Add package and build
Expected behavior
Build success
USearch version
2.15.0
Operating System
Windows 10
Hardware architecture
x86
Which interface are you using?
Other bindings
Contact Details
No response
Are you open to being tagged as a contributor?

[X] I am open to being mentioned in the project .git history as a contributor

Is there an existing issue for this?

[X] I have searched the existing issues

Code of Conduct

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

Thanks for noting, @CCnut! I'm merging the mentioned PR. Please let me know if it doesn't solve your issue ;)

It would help if anyone could add a Rust test into the prerelease.yaml CI for Windows 🤗
cc @brkp, @CCnut

Thank you @brkp! Much appreciated ;)


Unable to override name or path of Dockerfile in .PublishAsDockerFile()

.PublishAsDockerFile() hardcodes the path to the Dockerfile and the name of the Dockerfile.
The path and name cant be overridden using DockerBuildArgs.
Why this matters: When building a monorepo, such as a Vercel turbo monorepo, the working directory for the docker build must be the parent folder at the root of the monorepo (so the docker build process can access packages and etc. in adjacent folders), but there can only be one file called Dockerfile in that root location, preventing publishing multiple apps from the monorepo.
The problematic line is in ExecutableResourceBuilderExtensions.cs:
private static async Task WriteExecutableAsDockerfileResourceAsync(ManifestPublishingContext context, ExecutableResource executable, IEnumerable<DockerBuildArg>? buildArgs = null)
{
    ...
    var appHostRelativePathToDockerfile = Path.Combine(executable.WorkingDirectory, "Dockerfile");
    ...
}

We now have AddDockerFile and WithDockerFile that allow specifying the path to the docker file https://learn.microsoft.com/en-us/dotnet/api/aspire.hosting.containerresourcebuilderextensions.adddockerfile?view=dotnet-aspire-8.0
@mitchdenny We need to decide if we update PublishAsDocker file (or do we deprecate it).

Unassigning until it bubbles to the top of the stack.

Assuming PublishAsDockerfile gets deprecated, how are we supposed to run node apps in dev and docker containers in prod?
Something like this?
var astroWebApp = Builder.ExecutionContext.IsPublishMode
    ? Builder.AddDockerFile(...)
    : Builder.AddNpmApp(...);

This doesn't work for a few reasons; They don't have the same type, so the program simply doesn't compile. There's also no longer a shared place to set various configuration, like WithReference (Granted, I haven't tested this enough yet to see how it currently works, as I tried using Dockerfiles with Aspire for the first time just now)

We might have a new overload that does something like this:
var builder = DistributedApplication.CreateBuilder(args);

builder.AddNodeApp("app", @"node/app.js")
       .WithHttpEndpoint(targetPort: 8080)
       .PublishAsDockerImage(c =>
       {
           c.WithEnvironment("NODE_ENV", "production");
       });

builder.Build().Run();

public static class DockerFileExtensions
{
    public static IResourceBuilder<ExecutableResource> PublishAsDockerImage(this IResourceBuilder<ExecutableResource> builder, Action<IResourceBuilder<ContainerResource>>? configure = null)
    {
        if (!builder.ApplicationBuilder.ExecutionContext.IsPublishMode)
        {
            return builder;
        }

        // Bait and switch the ExecutableResource with a ContainerResource
        builder.ApplicationBuilder.Resources.Remove(builder.Resource);

        var container = new ExecutableContainerResource(builder.Resource);
        var cb = builder.ApplicationBuilder.AddResource(container);
        cb.WithImage(builder.Resource.Name);
        cb.WithDockerfile(builder.Resource.WorkingDirectory);

        configure?.Invoke(cb);

        return builder;
    }

    class ExecutableContainerResource(ExecutableResource er) : ContainerResource(er.Name)
    {
        public override ResourceAnnotationCollection Annotations => er.Annotations;
    }
}

I tried this extension method in conjunction with AddNpmApp, but that caused some internally-defined runtime args (WithArgs) to be added to the resulting manifest, which seemingly breaks the docker runtime in Azure ( which was very avoiding to debug :/ ). That blocker aside, this seemingly is a much better, more extensible API.

Good catch!

Tweaked the code above


Feature/implement api layer for civilians

What
Implemented API layer for civilians accounts, added few middlewares and viewmodels, refactored old code.
Also added extension methods for startup configuration.
How
Added controllers two controllers for passport: for authorized users and administrators.
Added three controllers for unauthorized users, authorized users and admins.
Added configuration methods for application services, infrastructure (repos), auth and utilities.
Added middlewares for db migrating, seeding and handling global exception.
Affected Area
Civilians.Api.*
Civilians.Infrastructure.Repositories
Civilins.Infrastructure.DbContext
Civilians.Core.Models

About lines, brackets and etc.
You have to write code according code style.
It makes you code more readable and understandable for other.
And you should write clean code straightway, without leaving it for refactoring.
If you send me pr, that means, that here i should see logical and clean code


Update AWS documentation to v1.2

On behalf of AWS, update AWS documentation to v1.2 in general
Help https://github.com/kubeflow/website/issues/2322
/cc @8bitmp3 @RFMVasconcelos

@8bitmp3 Right, that's minimal change I think
Seems like we need root owner to approve the PR
@Bobgy Can you help approve the PR?
Let me know if you think it's necessary to add OWNERS file in subfolder layouts, thus improve our bar of fine grained access.

Thank you for this update @PatrickXYS!
/lgtm

Yes, please add OWNERS whenever you are updating anything aws specific

/lgtm
/approve
You can do that in a separate PR


Tell interested suppliers that a question has been answered

Suppliers want to be notified when there's a new answer for opportunities they are interested in
Trello ticket: https://trello.com/c/2noI0EFB/472-tell-interested-suppliers-that-a-question-has-been-answered

I just noticed on the Trello card that you're planning to send the emails at 8am, the same time as the cut-off for working out which things to include.  I'd be tempted to have at least an hour's gap between the the time of sending and the latest changes to pick up as a "just in case" for time weirdness, daylight saving stuff and out-of-sync clocks.  I have no concrete examples to offer of how things could go wrong, just a gut feel that there's a risk of sending the same notifications twice if you have all the times at 8 on the dot.

Re: scheduling: we've agreed to schedule the job at 9.01am to avoid any potential duplication (or wrinkles around daylight savings). Regards the api load - the script makes three queries via the api client. I'm not sure how expensive each one will be, but we can keep an eye on it and potentially move the times back a couple of hours if necessary?

OK I left a final comment about exit codes and catching errors during the run.  It's not a massive deal-breaker,m but I do think we need to be able to ensure that the Jenkins build will be marked red and ping Slack if any of the emails fail to send, and I'm not sure that will happen with the current setup (i.e. I'm not 100% sure if, e.g. an API error halting the script, would trigger not ok and cause the script to return 1).

So on reflection, of course this will fail red if it dies with an uncaught error.
The question is whether we want to give up immediately there's a single failure or if it might be worth continuing with the rest of the job.
That's up to you really so I'm going to approve this now, although it feels to me like we might not want to give up for a single failed request/email.
There's an example of one of our scripts counting errors and returning the count here: https://github.com/alphagov/digitalmarketplace-scripts/blob/master/scripts/upload-counterpart-agreements.py#L79

I did think of suggesting passing in IDs to send to rather than exclude but didn't have the heart at this stage :)
Better for trying things out too - you can just send to yourself rather than everyone.


PTM sites of BRAF

There are many PTM sites from dbPTM, see http://dbptm.mbc.nctu.edu.tw/info.php?id=BRAF_HUMAN, Experimental PTM Sites tab, but not in Genome Nexus: https://www.genomenexus.org/variant/7:g.140453136A>T

This seems like an id mapping issue on our side. In https://github.com/genome-nexus/genome-nexus-importer/blob/53b39dceaff7d69c009340605363954bc2f886fe/data/common_input/CCDS2Sequence.current.txt we have two lines for CCDS5863.1 (which is the CCDS id for BRAF_HUMAN):
CCDS5863.1	1	1	NCBI	NM_004333.5	NP_004324.2	Accepted	1
CCDS5863.1	1	1	EBI,WTSI	ENST00000646891.1	ENSP00000493543.1	Accepted	1

Currently, genome nexus importer only parses the ids starting with ENST:
https://github.com/genome-nexus/genome-nexus-importer/blob/53b39dceaff7d69c009340605363954bc2f886fe/scripts/add_enst_id_to_ptm.py#L15
Therefore the importer sees ENST00000646891 as the only valid transcript for BRAF. However, this is not the canonical transcript. Actually, we don't even have this transcript in our transcript dropdown menu. That's why we don't see any PTM sites for BRAF.
I think we need to update the existing logic to accept NM_004333 as a valid transcript. We may need another mapping from NM to ENST to make this work because we always query with ENST in the frontend like this:
https://v1.genomenexus.org/ptm/experimental?ensemblTranscriptId=ENST00000288602
CC: @inodb


mat-menu missing API to set aria-label

<mat-menu> doesn't have an API that lets the user set an aria-label on the role="menu" element.
I'd like to fix this long term by moving to a different menu API, something like
<mat-menu *matMenuDef>
  ...
</mat-menu>

(where mat-menu is the role="menu" element)
In the interim, we should just add an input for this.

Note there are similar related attributes which should have a similar api, such as aria-labelledby and aria-describedby

Right- that's what I meant by the long-term fix; I don't want to add an explicit API for every attribute, but we do need a shorter-term workaround for just aria-label


Fix MouseTracker annotation leak

Description
MouseTracker had a bug where Map.remove was being called with a value instead of a key, causing widgets and their state to leak (if it used a Listener).
Changed the existing pointer listener test to check if the annotation has been removed from the MouseTracker.
Related Issues
None
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] My PR includes tests for all changed/updated/fixed behaviors (See Test Coverage).
[x] All existing and new tests are passing.
[x] I updated/added relevant documentation (doc comments with ///).
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require Flutter developers to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (Please read Handling breaking changes).
[x] No, this is not a breaking change.

@ds84182 Thanks for your contribution. It's good practice to make only one change per PR. That way, we can easily roll a specific change back if it causes trouble.
Also, it looks like you change is causing some existing tests to fail. Did you look at those failures?
/cc @gspencergoog

Gotcha, I'll move the double painting to another PR. Seems like the issue comes from marking RenderPointerListener as always composited.

Update, PTAL

Should be good to go now

I'll land this as soon as the tests are done running.
Thanks so much for the PR!

Thanks for merging that, @dnfield! I just forgot to check back on this to see that it had finished.


Исправлено описание в демке

Исправлено описание в демке

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 83.986%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
9854

Relevant Lines:
10817

💛  - Coveralls


Add tests

I really meant to develop more specific tests but just haven't got around to it.
pip install pytest && pytest
Other things I'd like to see added to aid development:

Continuous Integration (Travis and AppVeyor)
Makefile for common commands i.e. make lint, make coverage, make publish, etc.
requirements.txt for development dependencies
tox.ini configuration for pre-deployment tests

I can send a PR for some or all of this if desired.

Thanks. I'd definitely appreciate a PR for all of those.
I'd like to use pipenv to handle dev dependencies and testing. Feel free to set it up as well if you like. If not, or if you're not a fan of pipenv, I can create the Pipfile later. I might also add some pre-commit hooks.

I'm a big fan of pipenv. :smiley:


Copy-DbaLogin fix - needed to add ParameterSets so it was valid option

Type of Change

[X] Bug fix (non-breaking change, fixes #6037
[ ] New feature (non-breaking change, adds functionality)
[ ] Breaking change (effects multiple commands or functionality)
[ ] Ran manual Pester test and has passed (`.\tests\manual.pester.ps1)
[ ] Adding code coverage to existing functionality
[ ] Pester test is included
[ ] If new file reference added for test, has is been added to github.com/sqlcollaborative/appveyor-lab ?
[ ] Nunit test is included
[ ] Documentation
[ ] Build system

Purpose
Fix issue using -OutFile parameter
Approach

Previously the only parameter with the File ParameterSet was Outfile, so when you try and pass in Source and File it got upset.

        [parameter(ParameterSetName = "File", Mandatory)]
        [string]$OutFile,

Added File parameterset to Source... which fixed it, but broke piping.

        [parameter(ParameterSetName = "File", Mandatory)]
        [parameter(ParameterSetName = "SqlInstance", Mandatory)]
        [DbaInstanceParameter]$Source,

Added two ParameterSets to Destination. Now all the scenarios below work.

        [parameter(ParameterSetName = "SqlInstance", Mandatory)]
        [parameter(ParameterSetName = "InputObject")]
        [DbaInstanceParameter[]]$Destination,

I'm not sure if there is a cleaner approach?
Commands to test
Copy-DbaLogin -source sourceServer -OutFile "test.txt"
Copy-DbaLogin -Source sourceServer -Login loginName -Destination destServer
Get-DbaLogin -SqlInstance sourceServer | Out-GridView -Passthru | Copy-DbaLogin -Destination destServer

We started avoiding parameter sets because of the awfully confusing error message - then we handle them in the begin. but i'll take this update! thank you

So would a better long term fix to be to change it to not use them?

shoot, this was the PR @wsmelton - my bad. the appveyor failures were so common, I just accepted it. I'll fix the test and ping @jpomfret to make sure i did it right


Cleanup misc to reflect kubefed rename

Partial fix for #936

This moves the ball forward I'm sure we'll continually comb over docs so I'd prefer to get this swath merged.
/lgtm


@Hardwaregore is inserting invisible charecters so the code breaks and i have to fix this

ARGH @Hardwaregore QUIT IT

ACTUALLY, IT WAS A CERILIC "E" IN THE indEx.html and the stylE.css!!!!!


bump python-yeelight version

Description:
This PR just bumps the required version of python-yeelight library, which most notably fixes the music mode of these bulbs when it's active on multiple bulbs. Upstream issue: https://gitlab.com/stavros/python-yeelight/issues/7

cherry-picked


Add Armagetron Advanced

Armagetron Advanced is a 3D multiplayer game with Tron-style lightcycles.
Microsoft Reviewers: Open in CodeFlow

Blocked by https://github.com/microsoft/winget-pkgs/issues/523.

This PR is still valid, it's just blocked by #523.

/AzurePipelines run


Finalized Burn and Transaction Processing

Also created new migrations to add table for tokens in postgres
Next steps: Create token logging service and create token processing service

⚠ GitGuardian has uncovered 1 secret following the scan of your pull request.
Please consider investigating the findings and remediating the incidents. Failure to do so may lead to compromising the associated services or software components.
🔎 Detected hardcoded secret in your pull request
🛠 Guidelines to remediate hardcoded secrets
🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.

scared me man


Improved text, rounded images corners

Description
I fixed some syntax, punctuation and vocabulary errors while also rounding the corners of the first, third and fourth files in the images folder (previously, only the second image had rounded corners). The corners are 8px in radius.
Type of change
:x: Bug fix (non-breaking change which fixes an issue)
:x: Icon addition (non-breaking change that adds/modifies Lawnicons's icons)
:white_check_mark: General change (non-breaking change that doesn't fit the above categories like copyediting)

Actually, now that I think about it, is the radius of the contributing-image-2's corners equal to 8px or 6px? Considering that the icon guidelines mention that corners should be 6px, does this image follow the same rules?

Hmm, it still says it has conflicting files, yet I just updated the image to fit the new official changes.


Error when the toml or database can't be found.

Fixes #1652

Interesting, these tests pass locally. It seems I have an empty gitignored database.gpkg in ribasim_qgis tests data folder. @visr can you confirm the existence of said file?

This was created when we started testing if that TOML is valid in Ribasim Python, which produced the GeoPackage. I assume that should be fixed now that you split the reading and writing context var?

Hmm, I don't follow. An empty file was created previously? Anyway, I propose to either commit the empty geopackage file or to touch the file from the test?

Ok sounds good. It's a bit odd that read works with an empty database but not no database.

Ok sounds good. It's a bit odd that read works with an empty database but not no database.

There's some complexity in the FileModel, it can use a spring cleaning as I mentioned in another issue. Initialization also passes some of these codepaths, so an empty db is almost identical to initializing an empty model. Which actually makes some sense 😉


Loaded scripts shows empty tree node

testing #31107

debugging an extensions
looking the tree and see an empty node (representing the root folder on my machine I believe)

most likely a duplicate of #31523

Not a duplicate of #31523.
On unix and macOS the first segment of a path is '/' which results in an empty string when using a path.split('/').


Documentation on note clusters

I would like to add some documentation on note clusters to the guidelines. We can mark a chord as cluster by using the @cluster attribute. The values from data.CLUSTER specify which keys are to be pressed. So in theory it should be sufficient to encode the top and bottom note of the clustered chord, correct? Is this the preferred way, or should every "sounding" note be encoded? The latter could potentially contradict the value in @cluster. If both is equally good, we could add a schematron rule to give a warning for illogical encodings.
Another thing is is the visual appearance. There are different forms of cluster notation, most commonly used a) the notation introduced by Cowell for keyboard music, and be) the block notation (often used for "divisi" parts).

But there are more possibilities:

How can the appearance of clusters be encoded? One way would be to use @glyph.* or @head.shape on every note, but that would be a very extensive encoding, also error prone. Usually the look of clusters in sheet music is fixed, so maybe it would be a good idea to add a @cluster.form attribute to scoreDef/staffDef?

Corresponds to #1153

As I understand the documentation for @cluster, the chord should still encode every “sounding” pitch. The attribute only indicates a different rendition. Benefit if a rendering engine would not support @cluster, there would still be a cluster of notes in the rendition. I see the problem with the potential contradiction, though.
Nevertheless, with @glyph.auth, @glyph.num this could be done, too.
It's always a tough decision to rely on external standards exclusively, though…

Did we decide to punt this to post-5.0?

There is another solution to this issue, namely adding a <cluster> element as a child, or possibly a sibling, of <chord> that can take the cluster related attributes from <chord>.  So, I think this should be taken up post-5.0.

2023 Developer Workshop:

We could leave it open to the encoder to encode all notes or just the top and bottom notes.
Nevertheless, the documentation should also contain a hint towards audio rendition, which, of course, would need all the notes.

I like the idea of <cluster>.
A close analogy would be with <fTrem>
 <fTrem beams="2" unitdur="16">
          <note dur="4" oct="4" pname="c" accid="n" />
          <note  dur="4" oct="4" pname="d" accid="n" />
</fTrem>

For a cluster element, it could be done either as an alternate element to a chord, or as a child element of a chord.   If it is a child element, then that would imply that part of the chord could be a cluster, but then additional notes could be added to the chord outside of the cluster range (which i have not seen, but in theory would be acceptable notation, but not many cases could be performable on a keyboard).   Theoretical example:

<chord>
    <note pname="a" oct="5" accid="f" />
    <cluster>
         <note pname="d" oct="5" accid="n" />
         <note pname="g" oct="4" accid="n" />
    </cluster>
    <note pname="c" oct="4" accid="s" />
</chord>

The two notes in the cluster would be used to infer the notes in between the two outer notes.  If both outer notes are white keys, then the notes in between are white notes.  If the outer notes are black keys, then then inner notes are all the black notes between them.  If one extrema note is black and the other is white, then notes in between are all chromatic notes.
There will be exceptions where the extrema notes could be the same color, but all chromatic notes should be included between.  In that case cluster@chromatic="true" should be used.
<cluster> also needs @accid (and accid.ges) to indicate the accidental sign above or below the cluster (so also @place or @accid.place or something like that).
And then some sort of styling option for the visual display of the cluster symbol as given in the first message in this thread.

I prefer <cluster> as a child of <chord> rather than a sibling.  Adding attributes for accidentals is a good idea.  For the cluster "type" -- white-note, black-note, or chromatic -- I think an attribute is better than relying on the top and bottom pitches.  It's better to be explicit than make inferences that could lead to contradictions.
As an aside, Gardner Read's Thesaurus of Orchestral Devices (https://archive.org/details/thesaurusoforche00read) covers tone clusters on pg. 239.  This is a great reference for music notation and we should rely on it more.

Just found an example of part of a chord using a cluster-like sign at http://harpnotation.com/notation-manual/damping-techniques/muting-strings-with-one-hand/

That example is a good demonstration of the  need for <cluster> as a child of <chord>.   For orchestral harp, clusters could include both "black" and "white" notes, since each string could be tuned to one of three pitches (such as C-sharp, C-natural, C-flat).  So there might be a need of listing multiple pitches for harp clusters  (otherwise it will need to be inferred from previous tuning elements (or there could be a pointer to the last tuning element given in the cluster attributes to allow calculating of the notes in the cluster if only the lowest and highest notes are given).
In this particular case there should be some attribute for the cluster to indicate that it is muting rather than sounding the strings.  Informally cluster@type="mute" might be good.   And ideally there would be a parameter for <note> to say that the notated pitches are not literally sounding at that pitch (this is useful for pitch analysis, where the sound in this case is percussive more than pitched). In other words some note parameter to indicate that it should be ignored for pitch analyses.

That example is a good demonstration of the need for  as a child of .

I think of a cluster as a sort of specialized "chord" (one where the notes are specified by range, instead of by discrete pitches) so I was thinking <cluster> could be an alternative to <chord>, rather than a child.
In the example posted by @pe-ro, the first chord / cluster could be expressed as something like:
.... layer and other stuff ...
<chord xml:id="c1">
    <note pname="a">
    <note pname="c">
    <note pname="d">
</chord>
... another layer and more stuff ...
<cluster stemwith="#c1" pname.low="a" pname.high="d" />
...

Other attributes (like @mute proposed by @craigsapp) could be used as well, or we could make <note> allowed as a child of <cluster> but I'm not sure that's necessary.
(I was just taking the example at face value, but on closer inspection I'm not really sure if this is a good example of the use of <cluster>... The notation looks like it's saying "mute A-D, but also pluck them as discrete strings in a chord" rather than "sound all strings from A-D simultaneously", like you would expect from an actual cluster. So this might be a case where the harp notation is co-opting cluster notation for its own uses.)

How would you encode the harp example?

I see something like this:
<staff n="1">
   <layer n="1">
      <space dur="4"/>
      <rest dur="4"/>
      <beam>
        <chord dur="8">
           <note pname="a" oct="4" />
           <cluster staff="2" tstamp2="0m+5" form="mute">
              <note pname="a" oct="4" />
              <note pname="d" oct="5" />
           </cluster>
        </chord>
        <note pname="d" oct="5" dur="8" />
        <note pname="b" oct="4" dur="8" />
        <note pname="c" oct="5" dur="8" />
      </beam>
   </layer>
</staff>
<staff n="2">
   <layer n="1">
      <note staff="2" pname="d" oct="5" />
      <note staff="2" pname="d" oct="5" />
      <note staff="2" pname="d" oct="5" />
      <cluster tstamp2="0m+3" form="mute">
         <note pname="a" oct="f" />
         <note pname="d" oct="5" />
      </cluster>
      <space dur="2" />
   </layer>
</staff>

I might do something like this:
<staff n="1">
   <layer n="1">
      <chord xml:id="c1" dur="4">
         <note pname="a" />
         <note pname="c" />
         <note pname="d" />
      </chord>
      <rest dur="4" />
      <beam xml:id="b1">
         <note pname="a" dur="8" xml:id="n1" />
         <note pname="d" dur="8" />
         <note pname="b" dur="8" />
         <note pname="c" dur="8" />
      </beam>
   </layer>
</staff>
<staff n="2">
   <layer n="1">
      <cluster form="mute" pname.low="a" pname.high="d" stem.with="above" />
      <cluster form="mute" pname.low="a" pname.high="d" stem.with="above" />
   </layer>
</staff>

Recognizing that there are a few things that are a bit off, but that's the general gist of it. (namely, the ability to target specific notes or chords to stem with, and the fact that the durations probably aren't correct.)
But, like I said, the harp example might simply be co-opting cluster notation for its own purposes, rather than actually being a "true" cluster.

(I must admit I'm having a hard time trying to figure out your encoding... Where do the three ds come from on the second staff? And the space and initial rest on the first staff?)

The three d's are because I was lazy and typing by hand.  Two of those should be C5 and A4.   The space and the initial rests is because I was placing the music on the staff indicated by the stem directions (using cross-staff chord with the first cluster and notes being attached to staff 2, and a space in staff 1 where the notes are displaying).
This example is a cluster of mute notes.
It would be possible to have something like that for piano music: the cluster could be depressed silently and then other notes struck, causing sympathetic resonances on the depressed notes (so intermediate between a mute cluster and a regular cluster).

Here is an interesting case of clusters where the notes are written out, but a box is placed around them to indicate a cluster:
Charles Ives, 114 songs (no. 1):
https://vmirror.imslp.org/files/imglnks/usimg/1/14/IMSLP34908-PMLP66688-Ives-114Songs1922op.pdf

PDF page 23:


lottie: use internal append rect/ellipse func

The lottie format defines the starting point for
drawing rectangles and ellipses differently than
the API available in the TVG. In the case of using dashes or trim paths, this leads to discrepancies. Therefore, when using the lottie builder, internal functions for appending rectangles and circles are utilized.
@issue: https://github.com/thorvg/thorvg/issues/1997

Is this a better approach that compromises renderer compatibility and then modifies the Lottie? Or can we simply tweak the SVG alone?
Have you reviewed the specifications for rectangles here? If not, we may need to update only the problematic circle.

Is this a better approach that compromises renderer compatibility and then modifies the Lottie? Or can we simply tweak the SVG alone?
Have you reviewed the specifications for rectangles here? If not, you may need to update only the problematic circle.

I'm not sure if I understand the question, but I'll try to answer.
The renderer is set to a certain starting point for the circle and rectangle. In TVG, it is consistent with the SVG standard, though it could well be different. But as long as it's different from the Lottie standard, we need to do something about it so that Lottie files are displayed correctly.
Here, I proposed such a solution because it's very simple and we can easily change it in the future. In this way, we can also merge prs #1987 and #1996
As for the ultimate solution - I've read more about the Lottie standard and I think that the implementation of 'direction' will result in the solution proposed here being able to be modified as well.
And ofc I checked the lottie specs for rects. That's why I included some changes regarding rects into this pr as well.
@hermet if you think this isn't a good approach, please close this pr. I'm new in the lottie format, I'm probably missing many aspects of it right now.

The renderer is set to a certain starting point for the circle and rectangle. In TVG, it is consistent with the SVG standard, though it could well be different.

@mgrudzinska The renderer itself is a open method, it could not follow any specification, I've never designed it from the get go, it is a just a pure graphics engine.

I'm considering the less change as possible for the best of both worlds.


What are the best practices for cancellation?

I've seen a design called cancellation token in ASIO, how about this project?

I am not sure what's the best practices for cancellation. Personally it may be std::executors.
I can't speak for ASIO.
Also, this may not be a async_simple issue. So I'd like to close this. If you want to chat topics like this, I think Discussions may be a better place.


My server wont show up on ogartracker

my servertracker is set to 1, how ever i still can not see my server on ogartracker
help would be appreciated

Have you viewed this tracker?

are you using the latest version of MultiOgar?


Custom generic @ExceptionHandler gets overriden by spring-security-oauth

In my project, I have a @ControllerAdvice class with two @ExceptionHandler methods. One of these methods catches an abstracted exception unique to the project, but the other catches generic Exceptions. These methods log whatever exception is passed to them in a db table for later review, which is of course critical to our QA process. This used to work fine, but since integrating spring-security-oauth I only get the custom exceptions logged. Generics exceptions just get printed to console as usual. I've confirmed that spring-security-oauth is the cause of this by disabling/enabling. As far as I can tell, the offending code is this snippet from oauth2.provider.endpoint.TokenEndpoint:
@ExceptionHandler(Exception.class)
public ResponseEntity<OAuth2Exception> handleException(Exception e) throws Exception {
    logger.info("Handling error: " + e.getClass().getSimpleName() + ", " + e.getMessage());
    return getExceptionTranslator().translate(e);
}

Although my usage is probably an edge case, this is pretty frustrating behaviour. No framework should be catching generic errors.

I'm pretty sure that handler is only used for exceptions in the token endpoint, which seems entirely reasonable to me. If you think the handler behaves differently, maybe a sample project with some steps to reproduce the problem would help get to the bottom of it.


Introduce two new updated adapters.

The 'denier' adapter replaces the previous denyChecker. It implements all of the CHECK
and QUOTA templates such that it can be used to deny any of those mesh functions. Eventually,
we'll auto-generate this adapter but for now it is manually curated.

The 'list' adapter replaces the previous genericListChecker and ipListChecker adapters. It
can be used for IP-centric or generic lists. It supports both a remote list fetched from a
server, as well as a local override.

Release note:

This change is

@geeknoid: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/mixer-presubmit.sh
1327f1a720fe02dcabe33b85b219e91734f933d9
link
@istio-testing bazel test this

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

@geeknoid: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/mixer-presubmit.sh
ba3c91713fe428b36f6fbbd76e854c3eccf6122b
link
@istio-testing bazel test this

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

@geeknoid: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/mixer-e2e-smoketest.sh
33bf20f406d98e6fb3591849869ce401fc9ab454
link
@istio-testing smoke test this

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

/test prow/mixer-e2e-smoketest.sh


Legalization with isplit as last instruction generates invalid clif ir

What are the steps to reproduce the issue? Can you include a CLIF test case,
ideally reduced with the bugpoint clif-util command?

Add the following as legalization:

for &(ty, ty_half) in &[(I64, I32), (I128, I64)] {
    let inst = ireduce.bind(ty_half).bind(ty);
    expand.legalize(
        def!(a = inst(x)),
        vec![
            def!((a, xh) = isplit(x)),
        ]
    )
}

Try to compile:

test run
target x86_64

function u0:0() -> i64 system_v {
ebb0:
    v0 = iconst.i64 0
    v2 = iconcat v0, v0
    v3 = ireduce.i64 v2
    return v3
}
; run

What do you expect to happen? What does actually happen? Does it panic, and
if so, with which assertion?
I expected the ireduce to be legalized to v3, v999 = isplit v2. The result however was a verifier error:
function u0:0() -> i64 [%rax] system_v {
                                ebb0:
[RexOp1pu_id#b8]                    v0 = iconst.i64 0
[-]                                 v2 = iconcat v0, v0
[Op1ret#c3]                         return v3
;~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
; error: inst3: v3 is defined by inst2 which has no EBB

}

The problem is that the following legalization code is generated:
pos.func.dfg.clear_results(inst);
let curpos = pos.position();
let srcloc = pos.srcloc();
let (a, xh) = split::isplit(pos.func, cfg, curpos, srcloc, x);
let removed = pos.remove_inst();
debug_assert_eq!(removed, inst);
return true;

The inst results are never attached to an instruction again, nor turned into aliases.

Which Cranelift version / commit hash / branch are you using? master

This legalization is added in #1616, using an intermediate copy.

The new backend framework doesn't use legalizations as much as the old backend framework anymore.


possible bug, hanging on getting eigenvalues?

Hi,
I’m trying to find the roots of a polynomial, and nalgebra has overall worked great.
Recently, I’ve found that my calculations hang on certain inputs, for which numpy immediately returns a value.
You can see my code for finding roots at https://github.com/hwchen/roots.
A pathological input is in this test case, which you can also see in the repo's unit tests:
   #[test]
    fn hanging() {
        let input = &[
            -770.9700059294701,
            0.0,
            0.0,
            0.0,
            0.0,
            0.0,
            0.0,
            0.0,
            0.0,
            0.0,
<PHONE_NUMBER>081459,
            0.0,
        ];

        let input: Vec<f64> = input.into_iter().rev().cloned().collect();
        dbg!(&input);
        let rs = roots(input.as_slice());
    }

Thanks for any help.

Hi! This is a bug. It seems like the following matrix makes the Schur decomposition loop indefinitely:

  ┌                                                                                ┐
  │  0 0 0 0 0 0 0 0 0 0 1.7748092111017977 │
  │  1 0 0 0 0 0 0 0 0 0 │
  │  0 1 0 0 0 0 0 0 0 0 │
  │  0 0 1 0 0 0 0 0 0 0 │
  │  0 0 0 1 0 0 0 0 0 0 │
  │  0 0 0 0 1 0 0 0 0 0 │
  │  0 0 0 0 0 1 0 0 0 0 │
  │  0 0 0 0 0 0 1 0 0 0 │
  │  0 0 0 0 0 0 0 1 0 0 │
  │  0 0 0 0 0 0 0 0 1 0 │

For what it's worth, note that you can use Schur::try_new to specify a maximum number of iterations. This does not fix your problem, but at least this givens you a way to avoid the infinite hanging (though the result will be None in those cases).

Thanks! try_new will get me through for now, looking forward to the bugfix. Let me know if there's anything I can do to help.

I can reproduce this issue with multiple matrices. I also tested your solution @hwchen which works great on most occasions. However, there are instances in which the function complex_eigenvalues() returns NaN while eigenvalues() returns correct eigenvalues. One such case is with:
fn test_real_roots_1() {
    dbg!(roots(&vec![-225., 0., 224., 0., 0.]));
}

This yields:
roots(p) = [
    Complex {
        re: 0.0,
        im: NaN
    },
    Complex {
        re: 0.0,
        im: NaN
    },
    Complex {
        re: 0.0,
        im: 0.0
    },
    Complex {
        re: 0.0,
        im: 0.0
    }
]

While just calling eigenvalues() yields:
Matrix {
    data: VecStorage {
        data: [
            0.9977753031397177,
            -0.9977753031397177
        ],
        nrows: Dynamic {
            value: 2
        },
        ncols: U1
    }
}

I might be wrong here but complex_eigenvalues should return all real eigenvalues in complex form right? Thanks for any help.


Remove /operations

As per @smarterclayton's comment https://github.com/GoogleCloudPlatform/kubernetes/pull/3548#issuecomment-70715714, "Operations should be removed altogether."
@smarterclayton: Can you please clarify why is it not required anymore?
It seems to be used for listing the ongoing actions on the server.

@nikhiljindal feel free to assign to yourself if you plan to work on this.

We reached agreement at the last Kube face to face that the original design (that operations could be long running on the apiserver, so we need to track them) would be better accomplished by a) controller loops and status on the objects, and b) breaking actions up to never be long running.  In that world, we do not need operations because no operation can be slow.  @brendandburns / @lavalamp just for sanity unless I'm misrepresenting the discussion we had.
----- Original Message -----

As per @smarterclayton's comment
https://github.com/GoogleCloudPlatform/kubernetes/pull/3548#issuecomment-70715714,
"Operations should be removed altogether."
@smarterclayton: Can you please clarify why is it not required anymore?
It seems to be used for listing the ongoing actions on the server.

Reply to this email directly or view it on GitHub:
https://github.com/GoogleCloudPlatform/kubernetes/issues/3644

We do need to satisfy the use cases described in #1899. However, we did agree that operations as currently implemented are not useful (they don't wait for the conditions deployment tools and/or users care about), not reliable (they aren't persisted),  and not friendly to replication or sharding (since they are server-local). I'm also not sure they are currently compatible with namespaces. So, I'm happy for them to be removed.

Removing operations would also address one of the inconsistencies in the API (#2740).

Also not secure (leak information if you can guess operation id).

On Jan 20, 2015, at 6:52 PM, Brian Grant<EMAIL_ADDRESS>wrote:
We do need to satisfy the use cases described in #1899. However, we did agree that operations as currently implemented are not useful (they don't wait for the conditions deployment tools and/or users care about), not reliable (they aren't persisted), and not friendly to replication or sharding (since they are server-local). I'm also not sure they are currently compatible with namespaces. So, I'm happy for them to be removed.
—
Reply to this email directly or view it on GitHub.

Please remove this from the client library:
https://github.com/GoogleCloudPlatform/kubernetes/blob/master/pkg/client/restclient.go#L134

And here:
https://github.com/GoogleCloudPlatform/kubernetes/blob/master/pkg/client/request_test.go#L802
That must not be exercised?

Thanks for pointing it out.
Sent https://github.com/GoogleCloudPlatform/kubernetes/pull/3921 for that.

This is basically done.

Actually, the objects (type Operations, OperationsList) still exist in the API. We should remove them.

Sent https://github.com/GoogleCloudPlatform/kubernetes/pull/4150 to delete those API objects.


Broken builds: build cache is disabled by GOCACHE=off, but required as of Go 1.12

All the builds break because of this change:

build cache is disabled by GOCACHE=off, but required as of Go 1.12

https://travis-ci.org/purpleidea/mgmt/jobs/470949378#L1610
All the specific info is available here.
We use this variable because it stops the tests from being cached. We do NOT want test results cached ever! We want to re-run the tests when we run go test.
Priority: critical
Blocker: 1.12

Thanks for the report.
This change is documented in the 1.12 release notes:

Build cache requirement
The build cache is now required as a step toward eliminating $GOPATH/pkg. Setting the environment > variable GOCACHE=off will cause go commands that write to the cache to fail.

https://tip.golang.org/doc/go1.12#gocache
Regarding:

We use this variable because it stops the tests from being cached.

The idiomatic way to bypass test caching is to use -count=1; not to disable the build cache. I suggest you enable the build cache, as required by Go1.12, and use that to disable test caching.

The idiomatic way to bypass test caching is to use -count=1; not to disable the build cache. I suggest you re-enable the build cache, as required by Go1.12, and use that to disable test caching.

How do I set that in my .envrc ? Or is the solution to take away an upstream feature and make typing go test even longer?? It's already got an fqdn in there...

(.envrc just stores shell variables, so any normal env setting will work)

You can use GOFLAGS="-count=1".
From https://tip.golang.org/cmd/go/#hdr-Environment_variables :
GOFLAGS
	A space-separated list of -flag=value settings to apply
	to go commands by default, when the given flag is known by
	the current command. Flags listed on the command-line
	are applied after this list and therefore override it.

Closing because I don't think there is anything to do.

-count=1 looks like a hackish IMO. Its better clean testcache before running the test cases
go clean -testcache.


setSendImmediatly(false)  don't work.

Hi. I'm create new Ziggeo instance in Button onClickListener in Fragment like this
Ziggeo ziggeo = new Ziggeo(getString(R.string.ziggeo_appid));
ziggeo.setSendImmediately(false);
ziggeo.createVideo(getActivity(), ziggeo_video_duration);
and use EventBus for OnVideoSent event. But video always sent after recording. What's wrong?
And next question. How can I sent video not immediately after recording?

@wandersage
Hi Anton,
Everything seems working fine.
When you set
ziggeo.setSendImmediately(false);

Video will not be sent right after you've recorded it and you will have ability to play the video first.
Then when you press ok button the video will be sent.

Please, tell me if you still have any questions,
Thanks

Hi:
I am trying to work with the method attachRecorder setting the value setSendImmediately(false) but it does not seems to work. Checking the source code, the boolean ARG_SEND_IMMEDIATELY is passed to the class VideoRecorderFragment but never taked into account: The super class BaseRecorderFragment does use it to set the boolean mSendImmediately but this boolean is never used.
The reason why I am using this is fragment recorder instead of the fullscreen recorder is because the fullscreen recorder crashes when it tries to upload the video. Thus, I don't have any other option.
Please, could you fix it?

Hi @jiahaoliuliu,

I couldn't reproduce your crash but tried to fix. Please check.
Why do you think setSendImmediately is not working? I've rechecked, everything working as expected. mSendImmediately is used in VideoRecorderFragment. This argument just allow you to select cover shot before send a video. After cover shot selection the video will be sent.
Can you please describe how do you expect it should work?


event: implement source muxing on Windows

Implement the cases in _dispatch_unote_register_muxed() and
_dispatch_unote_unregister_muxed() for when multiple event sources are
open on a handle and we need to combine them.
The test suite doesn't hit these codepaths anywhere and we haven't run
into issues with Foundation yet, so I added a dispatch_io_muxed test
which opens multiple sources on a file/pipe/socket and checks that
events fire correctly.

cc @compnerd @ktopley-apple

@swift-ci please test


DDSaaS: Shopify: Crawler Integration v1.0.0

What does this PR do?
This is an initial release PR of Shopify integration including all the required assets.
Additional Notes

Crawler code for this integration has been committed in its respective repo
Pipeline created for this integration are available in our sandbox and would be shared separately with the required teams.
Samples for the pipeline review would also be shared separately with the required teams.
Since during the standard attribute remapping we are not preserving the source attributes as per suggested best practices, it would result in filters using these standard attributes populating the values of other integrations as well as per current Datadog behavior.

Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] Changelog entries must be created for modifications to shipped code
[ ] Add the qa/skip-qa label if the PR doesn't need to be tested during QA.
[ ] If you need to backport this PR to another branch, you can add the backport/<branch-name> label to the PR and it will automatically open a backport PR once this one is merged

@akaila-crest Is this ready for review? If so please remove the PR from draft status.

@cmlaverdiere, I have resolved the conflict, and yes PR is ready for the review.


Raster processing tools that take a raster OR a constant fail

There are many tools in the Math and Stats Tool toolset that have parameters that take a raster OR a constant. If try to run any of these tools they all fail as you cannot proceed as they reject a constant value.
Take the Divide tool as example you cannot divide your test data DEM.tif by a number.
For these tools expanding the allowable list of data types resolves the issue, I tried the following:
datatype=["GPRasterLayer","Long","Double"]

@Hornbydd This bug has been fixed. It should be working now. Thanks for reporting!

@giswqs your edits have not resolved the problem, the problem is in the Parameter definition and the datatype. It need's updating to something like:
datatype=["GPRasterLayer","Long","Double"]
The tool validation step does not allow you to enter a number, try it.

I only tested in ArcGIS Pro, and it works fine. See below. So it seems ArcGIS Pro and ArcMap are handling tool validation differently. I will revise the code and tested in ArcMap.

It should work with ArcMap now.

@giswqs  No it's not! You have fixed the issue with the validation by using the GPDouble, but you are using a method isnumeric() which is not compatible with the version of python installed with ArcMap, so the code crashes with:

Executing: Divide DEM.tif 5.3 C:\Scratch\oo.tif
Start Time: Tue Mar 10 18:46:31 2020
Running script Divide...
Traceback (most recent call last):
File "", line 27680, in execute
File "c:\program files (x86)\arcgis\desktop10.7\arcpy\arcpy_init_.py", line 1260, in Describe
return gp.describe(value, data_type)
File "c:\program files (x86)\arcgis\desktop10.7\arcpy\arcpy\geoprocessing_base.py", line 376, in describe
self._gp.Describe(*gp_fixargs(args, True)))
IOError: "5.3" does not exist
Failed to execute (Divide).
Failed at Tue Mar 10 18:46:31 2020 (Elapsed Time: 0.42 seconds)

See here and here.

@Hornbydd Oops, I only tested integer numbers. Thanks for the catch. It should be truly fixed now. Please try again.

@giswqs Just tested some of the tools and they all appear to be running well! Parameters are accepted and no crashing, so well done. I can't get over how quick WBT runs, its great!


Fix docker run when using docker swarm

docker only accepts -H before the subcommand on recent versions.
In order to fix this I've removed socket_connect from the run flags and added
it to the docker_command variable.
This has the added benefit of prefixing every command for docker
run in the init scripts, with the socket_connect option. This can
prevent weird situations where docker stop/kill will run against a container
the local docker swarm node doesn't know about.

@heliocentric thanks, nice fix. Much appreciated.


user level autoattach overrides network level autoattach

On user level I have:
2014-10-23 16:27:33+0300 < *autoattach> +-----+------+---------+------+
2014-10-23 16:27:33+0300 < *autoattach> | Neg | Chan | Search  | Host |
2014-10-23 16:27:33+0300 < *autoattach> +-----+------+---------+------+
2014-10-23 16:27:33+0300 < *autoattach> |     | *    | *%nick%* | *    |
2014-10-23 16:27:33+0300 < *autoattach> +-----+------+---------+------+

on network level at OFTC I have
2014-10-23 16:26:33+0300 < *autoattach> +-----+----------+--------+------+
2014-10-23 16:26:33+0300 < *autoattach> | Neg | Chan     | Search | Host |
2014-10-23 16:26:33+0300 < *autoattach> +-----+----------+--------+------+
2014-10-23 16:26:33+0300 < *autoattach> | !   | #supybot | *      | *    |
2014-10-23 16:26:33+0300 < *autoattach> +-----+----------+--------+------+

What I expect to happen: I get autoattached everywhere on highlight with exception of #supybot at OFTC.
(There is relaybot between #supybot at oftc and freenode, so I only get highlights from myself there.)

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/5472261-user-level-autoattach-overrides-network-level-autoattach?utm_campaign=plugin&utm_content=tracker%2F1759&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F1759&utm_medium=issues&utm_source=github).

The current behaviour (having it as user level makes sense) and in case users have issues it can be putten into FAQ.


Expose the deployment of SQL CLI workflows in the Astro CLI

Context
Using cobra, implement the command astro flow deploy in the Astro CLI. As a starting point, this command will work on Astro Cloud (ticket to support deploying to software: #1522), and it will deploy using the image-based approach (ticket to improve to use --dags: #1523).
For this first version of deployment, we are focused on deploying DAGs with expanded tasks (without using the render approach).
To deploy a single workflow:
astro flow deploy –all

To deploy multiple workflows:
astro flow deploy <workflow-name> --env dev

If the parameter --env is not defined, it should use the environment default.
There should be an optional flag -y which, if passed, will give empower the application to make decisions on behalf of the user, without prompting.
Behaviour

Identify the SQL CLI deployment/environment configuration using #1517. If this task is not completed yet, have a function with the same interface and hard-coded values.

Identify the Python SDK availability/version compatibility using #1473. If this task is not completed, have a function with the same interface and hard-coded values.

If the folder <project-dir>/.deploy/<env>/ does not exist, create it

Copy any dags from <project-dir>/dags into the folder (3) - so we don't delete them while deploying. Python DAGs are being run in Astro CLI. Perform this action silently.

Generate the SQL workflows dags of interest (one or all) into the folder (3). Perform this action silently.

#1529
#1528

(Future, post 0.3.0) check if the connections used in the SQL workflow are available in the target deployment environment. Notify users if they are not.

#1524
#1525
#1526

(Future, post 0.3.0) if the connections are defined locally, but no remotely, prompt users if they would like for them to be created.

#1527

Build the Docker image. Perform this action silently, reusing the existing logic in the astro deploy command.

Check if there are errors in the DAGs in (3), reusing the existing logic in the astro deploy command.

Push the Docker image to the container registry, reusing the existing logic in the astro deploy command.

Deploy reusing the existing logic in the astro deploy command.

Notify users about the success with the following message:

The deployment was successful!

The SQL workflows can be accessed using the following links:
* example_dag_basic: https://astronomer.astronomer.run/djhb6ppo/dags/example_dag_basic/graph
* example_dag_advanced: https://astronomer.astronomer.run/djhb6ppo/dags/example_dag_advanced/graph

I just extracted the following part, reducing the scope of this ticket:

There should be an optional flag -y which, if passed, will give empower the application to make decisions on behalf of the user, without prompting.

To a ticket of its own:
https://github.com/astronomer/astro-sdk/issues/1618
As agreed with @feluelle and @pankajkoti


Fix perf-tests

@sjug FYI: I'm fixing it for the second time.

/lgtm

@wojtek-t I'm sorry did I revert this? I didn't see it in the delta?

It is in the diff.

@wojtek-t I see, sorry I missed that, and thank you for spotting & fixing. Won't happen again.

Fixed compile error.


More characters need to be escaped in serialization

URLEncodedFormSerializer does not percent-encode the ampersand character (& U+0026) which MUST be percent-encoded as it is used as a separator between fields.
URLEncodedFormSerializer does not percent-encode the semicolon character (; U+003B) which MUST be percent-encoded as HTTP server implementors are encourage to accept it in place of & according to RFC 1866.
URLEncodedFormSerializer does not percent-encode the following characters which SHOULD be percent-encoded as most implementations (i.e. WebKit, Blink and Gecko) do so: !, $, ', (, ), ,, /, :, =, ?, @ and ~.

Ran into this issue with AWS SES (Simple Email Service). Specifically with = not being encoded.

I put up a PR fixing Vapor 4 here: https://github.com/vapor/vapor/pull/1965. If this seems like a good approach, we can cherry pick to Vapor 3.
I chose to only add percent encoding to ?&=[]; since those seem like the most critical issues. We can consider adding support for percent encoding the "should" characters as a separate PR.

@tanner0101 would appreciate this being cherry picked to Vapor 3 🙏

Can this repo be updated as well? I would suggest using Alamofire's implementation as a reference:
extension CharacterSet {
    /// Creates a CharacterSet from RFC 3986 allowed characters.
    ///
    /// RFC 3986 states that the following characters are "reserved" characters.
    ///
    /// - General Delimiters: ":", "#", "[", "]", "@", "?", "/"
    /// - Sub-Delimiters: "!", "$", "&", "'", "(", ")", "*", "+", ",", ";", "="
    ///
    /// In RFC 3986 - Section 3.4, it states that the "?" and "/" characters should not be escaped to allow
    /// query strings to include a URL. Therefore, all "reserved" characters with the exception of "?" and "/"
    /// should be percent-escaped in the query string.
    public static let afURLQueryAllowed: CharacterSet = {
        let generalDelimitersToEncode = ":#[]@" // does not include "?" or "/" due to RFC 3986 - Section 3.4
        let subDelimitersToEncode = "!$&'()*+,;="
        let encodableDelimiters = CharacterSet(charactersIn: "\(generalDelimitersToEncode)\(subDelimitersToEncode)")

        return CharacterSet.urlQueryAllowed.subtracting(encodableDelimiters)
    }()
}


Azure storage and Sharepoint

Which of these storage types can you use with Sharepoint?

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

ID: 312ba22f-685c-d17f-438f-9656f019012a
Version Independent ID: a93993be-f687-16e1-ff5b-b526699c009f
Content: Introduction to Azure Storage - Cloud storage on Azure
Content Source: articles/storage/common/storage-introduction.md
Service: storage
GitHub Login: @tamram
Microsoft Alias: tamram

@smushtaq1 Thanks for your question. We are checking on this and will respond to you soon.

@smushtaq1 Since we have not heard back from you we will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly reopen the issue and continue the discussion.

Hi Sumanth,
I apologize for not replying, for some reason I never got notification of you reply.  I will look into the articles you provided, thank you for the help.
Thank You,
Shahab Mushtaq | ECS Senior Specialist Applications
Information Technology | Regional Hub AMS – Saint Louis
Office Phone: +1<PHONE_NUMBER> | Team Sitehttp://thenest-eur-hq.nestle.com/GL/GL_GLTS/ECSHomepage/Pages/EnterpriseComputingSolutions.aspx
We make IT possible
[http://phonebook.us.nestle.com/PhoneBookPhoto.asp?mail=Shahab.Mushtaq@nestle.com]http://phonebook.us.nestle.com/?mail=Shahab.Mushtaq@nestle.com[http://phonebook.us.nestle.com/mysite_icon.gif]http://mysite-ams.nestle.com/Person.aspx?accountname=nestle\USMushtaSh[http://phonebook.us.nestle.com/chatter_icon.png]http://thenest.my.salesforce.com/[http://phonebook.us.nestle.com/mail_icon.gif]mailto:Shahab.Mushtaq<EMAIL_ADDRESS>From: SumanthMarigowda-MSFT<EMAIL_ADDRESS>Sent: Tuesday, February 12, 2019 10:25 AM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Mushtaq,Shahab,ST LOUIS,GLOBE IS/IT NA<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] Azure storage and Sharepoint (#24488)
@smushtaq1https://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fsmushtaq1&data=02|01|shahab.mushtaq%40nestle.com|82cc603b3e25466e023408d69106b2e2|12a3af23a7694654847f958f3d479f4a|0|0|636855855332472580&sdata=q6jFgJIFdRwqBTX8z1hlcL12OyQLr1v87masYCmqZsM%3D&reserved=0 Since we have not heard back from you we will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly reopen the issue and continue the discussion.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F24488%23issuecomment-462828263&data=02|01|shahab.mushtaq%40nestle.com|82cc603b3e25466e023408d69106b2e2|12a3af23a7694654847f958f3d479f4a|0|0|636855855332472580&sdata=glWJ1%2Fn6leWv2OD6oKcEK0ix%2FD%2FvqYwgKXfqMMZJZ4I%3D&reserved=0, or mute the threadhttps://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAtOvMefySfaeXKsXtfPV7adkthYxch95ks5vMur2gaJpZM4anw2D&data=02|01|shahab.mushtaq%40nestle.com|82cc603b3e25466e023408d69106b2e2|12a3af23a7694654847f958f3d479f4a|0|0|636855855332482585&sdata=JIBOI1KtHIojbI3RAfH5O0QsTGLXgN3nNxKxu633qlk%3D&reserved=0.


[release/3.1] Fix fetch-microsoft-sdk script

The script internally uses dotnet-install. The output of dotnet-install has changed: it now prints multiple urls and only one of them is the one we want.

dotnet-install: Note that the intended use of this script is for Continuous Integration (CI) scenarios, where:
dotnet-install: - The SDK needs to be installed without user interaction and without admin rights.
dotnet-install: - The SDK installation doesn't need to persist across multiple CI runs.
dotnet-install: To set up a development environment or to run apps, use installers rather than this script. Visit https://dotnet.microsoft.com/download to get the installer.
dotnet-install: Payload URLs:
dotnet-install: Primary named payload URL: https://dotnetcli.azureedge.net/dotnet/Sdk/3.1.110/dotnet-sdk-3.1.110-linux-x64.tar.gz
dotnet-install: Repeatable invocation: ./dotnet-install.sh --version "3.1.110" --install-dir "$HOME/.dotnet" --architecture "x64 "

Fix the grep expression to pick up the payload URL.

@crummel @dagood @dseefeld Can we merge this in? This fixes release/3.1's smoke-tests to work again.

macOS is expected to fail (https://github.com/dotnet/source-build/issues/1819), although earlier than it did in this PR. Maybe the pool got fixed? Anyway, good to merge.

@omajid was the error you saw something like this?
 dotnet-install: To set up a development environment or to run apps, use installers rather than this script. Visit https://dotnet.microsoft.com/download to get the installer.
 curl: (6) Could not resolve host: dotnet-install: To set up a development environment or to run apps, use installers rather than this script. Visit https; Unknown error
/src/build.proj(250,5): error MSB3073: The command "./smoke-test.sh --minimal --projectOutput --configuration Release --archiveRestoredPackages" exited with code 6.

We just saw it in a branch build, and wondering if we just need to merge this change into it. 🙂

The error log looked like this on Fedora:
  dotnet-install: To set up a development environment or to run apps, use installers rather than this script. Visit https://dotnet.microsoft.com/download to get the installer.
  curl: (3) URL using bad/illegal format or missing URL
/home/tester/source-build/build.proj(250,5): error MSB3073: The command "./smoke-test.sh --minimal --projectOutput --configuration Release --archiveRestoredPackages" exited with code 3.

And this on RHEL:
  dotnet-install: To set up a development environment or to run apps, use installers rather than this script. Visit https://dotnet.microsoft.com/download to get the installer.
  curl: (3) Port number ended with ' '
/home/tester/source-build/build.proj(250,5): error MSB3073: The command "./smoke-test.sh --minimal --projectOutput --configuration Release --archiveRestoredPackages" exited with code 3.

The error message here looks close enough that this PR should fix it.
That said, I have seen dotnet-install fail recently just downloading an SDK all by itself.

Interesting... yeah, seems similar enough. (We're adding another || true just to make sure this doesn't block us in the future, FYI. Until we get this working using a more direct download rather than the download script (#1631).)
I faintly remember seeing issues reported with dotnet-install recently, network issues. Not sure what's going on there.


incorrect TS generated when param name is like page[number]

when a param name (in this case query param) contains square brackets, it ends up generating types like this:
 {
    page[number]?: number;
    page[size]?: number;
    filter[search]?: string;
}

which is invalid TS. they need to be quoted. this style of query param is standard for JSONAPI

Hey 👋
I just released a new version of the cli and generator that fixes your issue (v3.1.5)


Batch edit taking incredibly long

I'm mass editing a byte tag in all of the chunks of a 2000x2000 world that I pregenerated for my server. I expected it to take a few seconds but its been going at it for more than 4 hours now.
gif: https://gyazo.com/e1241503dcc5efd077f4fdb5f3ce23b9
I have an Intel i5-8600K @3.6GHz

I have been trying to edit about 836 command blocks and came here to suggest that they add a search/replace function that allows regex.  Your request seems similar enough, so I didn't make my own post.
I am trying to remove the run tell portion from my sorter commands
/execute if block ~ ~1 ~4 hopper{Items:[{id:"minecraft:oak_sign", Count:1b, Slot:0b}]} run tell @p Oak Sign Sorted
If I could have it search for all lines containing 'run tell' and truncate them after the last '}' using a regular expression, this would be done already.

The problem is that NBTExplorer simulates clicking on the GUI while searching, and the rendering of Windows widgets is very slow. Try minimizing the main window to stop rendering it and you're likely to get things done in 10 minutes (but not a few seconds because NBTExploer is still doing something on the GUI).


alias vf="vim $(fzf)" can't work!

If I write  alias vf="vim $(fzf)" into my zshrc, the command vim $(fzf) will run automatically whenever I open a new terminal.
What should I do?

Use single quotes to avoid immediate evaluation: alias vf='vim $(fzf)'

Thanks for your good answer! @junegunn

Thanks!


Nested tabs swiping

After Update react-native-tab-view #1327 . Nested tabs on android don't swipe correctly. Video below and example added to the playground in the fork below.
Maybe related to https://github.com/react-native-community/react-native-tab-view/issues/225
Other swiping examples work just not nested.
Video: https://www.youtube.com/watch?v=dGDYaI6fsbE
Code: https://github.com/onlydave/react-navigation/tree/nestedTabs
@satya164

This looks like the same as https://github.com/react-native-community/react-native-tab-view/issues/225
This seems to be a very old bug with ViewPager https://github.com/expo/ex-navigation/issues/415

Also when you say nested, is it swipeEnabled={true} inside swipeEnabled={false} or swipeEnabled={false} inside swipeEnabled={true} ?

swipeEnabled={true} inside swipeEnabled={false}

Thanks. Yeah, seems like that setup should trigger the bug

Some problem.
Swipe of nested tab navigator not works in first tab of main tab navigator.
Helps only config: {   swipeEnabled: false,   animationEnabled: false } for nested tab navigator.
@satya164 have ideas?

Same issue!!! Please fix it! Thanks!

Same issue here, any progress on this one?

We have resources (time) to help with this issue, but I would need some hints on where to start.

Same issue here

This appears to be an issue for me with swipeEnabled={true} inside swipeEnabled={false} on release builds of Android only, not on debug Android builds or any iOS builds.
react-native: version: 0.48.4
react-navigation: 1.0.0-beta.15
Tested on

Device: Samsung Galaxy S7 edge, Android 6.0.1
Emulator: Nexus 5X, Android 7.1.1

Sadly this looks like it's only fixable at react-native && react-native-tab-view level. Hopefully v70 of tab-view has the fix, and it will be integrated into beta.16.

current:
react-native-cli: 2.0.1
react-native: 0.49.3
<EMAIL_ADDRESS><EMAIL_ADDRESS>it  runs normally when i seted swipeEnabled={false} inside swipeEnabled={false},but the inside one just can't swipe,it's not what i want. did i miss something and  what should i do?

react : 16.0.0
react-native - 0.50.4
react-navigation- 1.5.10
activeTintColor stuck while tap and swipe, please fix this issue?

hi there! if this is still an issue, can someone who would like to see this issue resolved please create a new issue with a reproducible example on https://snack.expo.io or in a github repository, with a minimal amount of superfluous code? (please don't post your entire app, just a small one to reproduce the problem). thanks!


Fixed #34038 -- Improve link readability and identifiability

To improve the readability of links in the Django admin:

The default colour has been slightly darkened in light mode (-1% lightness in HSL colour space)
The links are underlined on hover for all default links (add/create links, breadcrumbs...) to be more identifiable for people with colour vision deficiencies.
Links in the form rows have been underlined by default as - since they bend into the help text - they are very difficult to distinguish from plain text.

@CLWilliam Do you have time to keep working on this? (see https://github.com/django/django/pull/16101#pullrequestreview-1135339000)

@felixxm it's on the menu for this evening!

Closing due to inactivity.


Rocket.Chat Plugin - Removing "Dies habe ich in vergangenen Konversationen gefunden“ and the User ID when using a suggestion

Issue*
Part 1:
When copying a smarti suggestion (historic conversation) into the input field in order to edit the text, the description "Dies habe ich in vergangenen Konversationen gefunden“  is added.
Usually the user will remove this text because it might not fit to the message the user likes to post.
Part 2:
Moreover, the text includes the user id of the user who originally posted the answer. When copying just one answer from the knowledge base this information is not relevant as well.
Solution
Remove the description and user id.

@janrudolph MSZ has requested this feature as well.
@westei Can we implement this for version 0.7.1?


Use Roslyn to dynamically re-compile in test cases

As far as I can tell CodeDom is pretty much deprecated and Roslyn is the future. This is .NET Standard compatible.

(Also allows recompilation with mono ;) )

👍


update project automation version

https://github.com/leonsteinhaeuser/project-beta-automations/issues/34


HTML Considerations

This skill is documented to extract text from various file formats, but for HTML, there is no expectation set as to what it will extract.  An example would be greatly appreciated.  New to skillsets for Cognitive services, but I'm sure extracting text from HTML documents are a major use case.  In addition, HTML file should be retrieved using a URL rather than stored as a blob.  Both are retrieved by URL, so why not reach across the internet, versus just the URL for a blob - latency may be the consideration.... but that too should be discussed and provide a best practise for the use case.

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

ID: 888362f6-45c3-2bb6-47b5-80c756ac77ad
Version Independent ID: 5ca9f578-ad8c-90a6-9ceb-a23b41427451
Content: Document Extraction cognitive skill - Azure Cognitive Search
Content Source: articles/search/cognitive-search-skill-document-extraction.md
Service: cognitive-search
GitHub Login: @Careyjmac
Microsoft Alias: chalton

@Richard-Painworth Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.

Hi @Richard-Painworth - I'll take following up on the HTML example as an action item and see if I can find an owner for this.
The second issue (file retrieval by URL) is a non-starter.  Skillsets are subordinate to indexers, and indexers provide all of the connection, retrieval, serialization, and change detection logic.  It's more than just file retrieval.  For this reason, indexers only work with a finite list of supported data sources, listed here: https://docs.microsoft.com/en-us/azure/search/search-indexer-overview#supported-data-sources.

Ok, I review.  Did a weekend worth of work on this pipeline, and made some progress.   I'm updating the v11 sample app, using the tutorial au enrichment....by adding an documentextractor for an HTML document I stored in a blob.
Would help if I knew what a typical result would be for an HTML file running through that new skill.  Can u point me to any documentation, or the repo as to the code for that skill?
R
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: Heidi Steen @.>
Sent: Monday, November 29, 2021 10:35:38 AM
To: MicrosoftDocs/azure-docs @.>
Cc: Richard Reukema @.>; Mention @.>
Subject: Re: [MicrosoftDocs/azure-docs] HTML Considerations (Issue #84507)
Hi @Richard-Painworthhttps://github.com/Richard-Painworth - I'll take following up on the HTML example as an action item and see if I can find an owner for this.
The second issue (file retrieval by URL) is a non-starter. Skillsets are subordinate to indexers, and indexers provide all of the connection, retrieval, serialization, and change detection logic. It's more than just file retrieval. For this reason, indexers only work with a finite list of supported data sources, listed here: https://docs.microsoft.com/en-us/azure/search/search-indexer-overview#supported-data-sources.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/issues/84507#issuecomment-981905822, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AWRP45MRYN2TWR7LNW3OIJLUOPBXVANCNFSM5I4UC55A.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Hi @Richard-Painworth, sorry for the delayed response. HTML and other markup is stripped from files if you run the skill with parsingMode = default. If you use parsingMode=text, then html tags are not stripped out and you’ll get all of the document content back as if it were raw text. I'll make sure the docs are clear about this.
#please-close


Add instructions to resize root volume for tensorflow install

Looks like tensorflow image has grown in latest release and can no longer be accommodated on t3.small Cloud9 instance. We'll have to modify root volume to increase the size and increase filesystem to install tensorflow
https://eksworkshop.com/advanced/420_kubeflow/inference/

closing this issue because I didn't experience this in my last run. Will reopen if it occurs again


:four_leaf_clover: Proposal: Refactor dev version check

What Would You Like to Add? Why Is This Needed?
I opend #821 some days ago, and refactored dev version check.
@hxcGit opened a new PR, using the semver package to validate version, I think semver covers more situations. So refactoring code are very necessary.
https://github.com/devstream-io/devstream/blob/7edf2749cf3f64c90ccfed626fd58cae6376f043/internal/pkg/version/version.go#L13-L16
He will refactor this in #840. Thanks!

@iyear  I've checked out this problem. In semver we can use semver.NewConstraint() to add some constraints and validate them, but the common use is to constrain the version number.
https://github.com/Masterminds/semver
https://semver.org/
I think it's simpler and clearer to use regular expressions, because we only need to distinguish dev version from release version. I'm a new user of semver, do you have any suggestions?

My mistake. I'm sorry.😥
I originally thought that our dev version number was not semver, but now it seems that it is also semver(0.7.0-130-g805bf48). So let's stick with the current style. Thanks for your reply!