Add id.Slice.ToMap and id.Slice.ToStringSlice

Outsourced from #1148

@kwk snapshot fabric8-wit image is available for testing.  docker pull docker.io/fabric8/fabric8-wit:SNAPSHOT-PR-1980-1

Codecov Report

Merging #1980 into master will increase coverage by 0.01%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master   #1980      +/-   ##
=========================================
+ Coverage   64.69%   64.7%   +0.01%
=========================================
  Files         147     147
  Lines       13818   13823       +5
=========================================
+ Hits         8939    8944       +5
  Misses       4045    4045
  Partials      834     834

Impacted Files
Coverage Δ

id/slice.go
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

@kwk snapshot fabric8-wit image is available for testing.  docker pull docker.io/fabric8/fabric8-wit:SNAPSHOT-PR-1980-2


Handle max_coins field in deployer

@Ben-Rey I saw you put value 0.1MAS in max_coins, but you are not estimating the storage cost. What will happen if I deploy a big contract that it worth more than 0.1MAS?
related to formula in the issue: https://github.com/massalabs/thyra/issues/640

It's just an example value for now and we have to discuss how it will be handled in the future. Could be estimated but we chose to leave it to the user for now.


#920 add slash when eureka.client.service-url.defaultZone does not contain one

Have you signed the Contributor License Agreement?

Nice tests!

@spencergibb yes, I have signed the aggreement.


Linked Auction Houses

Needs linked auction house support

Fixed in https://github.com/Tauri-WoW-Community-Devs/TauriApiWrapper/commit/87eca07e4fa30fd5e9a9549f2ef0657041093c29


Escaped pipe and replaced comma solving #1420

went through .md files and replaced unescaped pipe with escaped pipe where I found it. Also replaced a few occasions of commas between command options with pipe and added back-tics where missing.

Coverage remained the same at 100.0% when pulling 0240fcedd01b0b2e97de86ddd843b9ec4ab76c92 on jhagstrom:fix-pipes-in-doc-1420 into f7f194d36ec571e84edc736ac0c7d821534abd54 on pnp:dev.

Awesome! Thank you! 👏

Replaced the escaped pipe pattern with an html encoded representation of the pipe char, as suggested by @garrytrinder

@jhagstrom any pointer to the discussion/suggestion? What was the issue with \. Expecting contributors to use html escaping is pretty excessive imho. If that's the only way forward due to some limitations, then I'd suggest we reconsider the change and consider simplicity of maintenance and contributions.

Merged manually. Thank you! 👏


Bug: The option StartIn is invalid

Hi @KristofferStrube,
There is bad news and good news here 😃
The bad news is that StartIn is invalid, because the new keyword makes it impossible to get the data covered by the child class in the parent class.
The good news is that I fixed the issue and submitted a PR.

Also, I noticed that some async method signatures are optimized.
Can a patch version be released, I want to apply the changes asap.

Thanks for the fix. I see that my refactoring of these option classes were a bit too harsh. Had thought the base class method would use the property of the derived class but I see your solution fixes this. I will merge and make a Nuget release with this and the Async naming fixes in a bit.

Excellent! I have updated to the latest version and everything is normal 👏
Thanks!


deis create in a non-git repo should run with --no-remote

As reported in https://github.com/deis/builder/issues/499, the Heroku client will run heroku create with --no-remote if the directory we are running in is not a git directory:
[~:dom] ⚡  heroku create
Creating app... done, ⬢ thawing-forest-56676
https://thawing-forest-56676.herokuapp.com/ | https://git.heroku.com/thawing-forest-56676.git
[~:dom] ⚡

Whereas with the deis CLI, this does not happen:
[~:dom] ⚡  deis create
Creating Application... done, created haptic-farmland
Error: Error when running 'git remote add deis ssh://git@deis-builder.cluster.fluid.ly:2222/haptic-farmland.git'
fatal: Not a git repository (or any of the parent directories): .git

I'm not sure if this was "fixed" in later Heroku versions, but this wasn't the case a few years ago. We should try to mimic Heroku's workflow here, seeing how we are an "open source Heroku".

Part of why I found this confusing wasn't just the difference with Heroku's behaviour, it was also conceptual. I'm creating a deis app on a remote box - It didn't seem obvious to me that my location in a local shell would affect that.
I though that deis create would have created a remote repo that I could then point to locally in a separate step if I so wished.
Example work flow
# create deis app for my service
$ deis create
$ cd my-service
$ git remote add production PATH_TO_DEIS_REPO

Ah I can understand that point of view. While there is a remote git repository stored at the builder, I wouldn't rely on it as a persistent git repo store for your application source code. Leave that for Github/bitbucket/your own git server etc. If the builder is rebooted, the repositories are gone because we don't have them backed by persistent disk or anything. The database WAL logs, registry images, built Heroku tarballs, and build caches are, though. The git repos in the builder are created on-the-fly when you push the local contents of the repository to the builder via git push deis.
Does that help clarify some things?

Absolutely, I though I never intended to use the git repo for a source control, just as a deployment pathway.

I thought that deis create would have created a remote repo that I could then point to locally in a separate step if I so wished.

For what it's worth, this is do-able, but it requires flipping on a few extra option flags (again, tooting the "Heroku compatibility" horn again):
$ deis create myappname --no-remote
$ deis git:remote --app myappname

This issue was moved to teamhephy/workflow-cli#9


Issue with custom IPipelineBehavior

Describe the bug
We added a custom IPipelineBehavior that appends the tenantId in the meta.security property.
public class AddTenantIdBehavior : IPipelineBehavior<CreateResourceRequest, UpsertResourceResponse>
    {
        private readonly IMediator _mediator;
        private readonly ResourceDeserializer _deserializer;

        public AddTenantIdBehavior(
            IMediator mediator,
            ResourceDeserializer deserializer)
        {
            _mediator = mediator;
            _deserializer = deserializer;
        }

        public async Task<UpsertResourceResponse> Handle(CreateResourceRequest request, CancellationToken cancellationToken, RequestHandlerDelegate<UpsertResourceResponse> next)
        {
            // just a simplification of what we are doing
            var deviceId = observation.Device.Reference.Split("/")[1];
            var resourceElement = await _mediator.GetResourceAsync(new ResourceKey(ModelInfo.GetFhirTypeNameForType(typeof(Device)), deviceId), cancellationToken);
            var device = resourceElement.ToPoco<Device>(_deserializer);

            var tenant = device.Meta.Security.Find(c => c.System == "our-tenant-system");
            observation.Meta.Security.Add(tenant);

            return await next();
        }
    }

The creation of the resource randomly fails throwing a null pointer exception at line 53 of BaseResourceHandler when trying to generate a new GUID:
https://github.com/microsoft/fhir-server/blob/77dd1cc76ca7e3fea0756631589832b7e5af1aae/src/Microsoft.Health.Fhir.Shared.Core/Features/Resources/BaseResourceHandler.cs#L49-L54
The Create property that should return the lambda that generates the GUID does seem to return null:

If we replace the DI registration type of the ResourceIdProvider
https://github.com/microsoft/fhir-server/blob/77dd1cc76ca7e3fea0756631589832b7e5af1aae/src/Microsoft.Health.Fhir.Shared.Api/Modules/PersistenceModule.cs#L28
to AddTransient the issue never happend again.
FHIR Version?
R4
Data provider?
SQL Server
To Reproduce
It randomly happens after we added the new pipeline middleware 😕
Expected behavior
The resource should awlays be created.
Actual behavior
The resource is not getting created as the GUID generations randomly fails.

Thanks @giacomodeliberali.
There could be a subtle bug in there, possibly the use of ResourceIdProvider's AsyncLocal and having a Scoped lifetime. So at a guess it might be possible for AsyncLocal to go out of context, while the IoC's scoped instance of ResourceIdProvider is still around.
I know the BundleHandler relies on ResourceIdProvider to set IDs, so any fix would need to ensure we don't break this.

AB#79320

@giacomodeliberali we have merged a PR that should have addressed this, can you verify if this is resolved for you?

@giacomodeliberali  I'm going to close this but please let us know if you have any additional questions


chore: bdd-nh-1592568964 to 0.0.1

chore: Promote bdd-nh-1592568964 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


css nesting rules seems not work

Hi, this may not an issue, but a question.
i configured gridsome.config.js as follows:
{
      use: "gridsome-plugin-tailwindcss",
      options: {
        tailwindConfig: './tailwind.config.js',
        presetEnvConfig: {
          features: {
            'nesting-rules': true
          }
        },
        shouldImport: true,
        shouldTimeTravel: true
      }
    }

then wrote  css by  nesting style like this:
.nesting__dot button {
  background: #888111;
}
.nesting {
	color: aqua;
  &__actions {
    margin-top: 20px;
  }
  &__nav-button {
    background: transparent;
    border: none;
    color: #ccc;
    cursor: pointer;
    font-size: 24px;
    transition-duration: 0.3s;
    &:hover {
      color: #888;
    }
  }
  &__dot {
    margin: 0 10px;

    button {
      @apply bg-green-400;
      border: none;
      border-radius: 50%;
      cursor: pointer;
      display: block;
      height: 10px;
      font-size: 0;
      line-height: 0;
      margin: 0;
      padding: 0;
      transition-duration: 0.3s;
      width: 10px;
    }
    &--current,
    &:hover {
      button {
        background-color: #888;
      }
    }
  }
}

No error reported, but the style didn't work.
Is something else i need to do?

@kuleyu Wrong nesting style.
This plugin includes https://github.com/jonathantneal/postcss-nesting
not https://github.com/postcss/postcss-nested

@brandonpittman Thank you!


Add render_component render

[x] renderer implementation
[x] specs
[x] docs update

solving #41

:+1:


[Request] Direct Windows Support

Hi there, is it possible for you to support windows directly.
Ollama has it on windows.
If possible in the future an installation script shall be perfect.
Thanks for the project.

Hi, thank you for your interest in Harbor!
Direct support for Windows (outside of running in WSL) is not planned, as most of the Harbor relies on a POSIX compliant unix shell atm. I have plans on making it more portable, but it's very far away and likely along with a full rewrite


Feature Request: sibling-swap.nvim integration

It would be awesome to have sibling-swap.nvim integration.

I don't see any movements defined there, only the swap action. If so, I would suggest using something like dot-repeat for making it repeatable as dot is by convention used to repeat actions.
But maybe I have missed something, can you tell what would you like this integration to have?

Thank you for the explanation! I'll actually go ahead and close this.


Implementation adapter for redis

Q
A

Is bugfix?
❌

New feature?
✔

Breaks BC?
✔

Fixed issues
https://github.com/yiisoft/queue-redis/issues/1

Hello Alexander!
I really needed an adapter for queues in yii3 and I had to write the implementation myself.
I hope my solution will help you. If there are any shortcomings in it, I will be glad to hear feedback.
I used the library structure from https://github.com/yiisoft/queue-amqp .
Methods for working with Redis from  https://github.com/yiisoft/yii2-queue/blob/master/src/drivers/redis/Queue.php.
Coverage 100%
Psalm level 1 without errors.

@TalismanFR thank you very much for contributing. Do you have time to ensure tests are running?

@TalismanFR thank you very much for contributing. Do you have time to ensure tests are running successfully?

Yes, sure! I am now making changes to the code based on the comments above.

Time to merge it to master. Let's continue work in separate branches / PRs. And thank you!


Stop UWP renderer from crashing due to Image loading/XamlBuilder exceptions.

Exceptions thrown while building the Xaml tree were not being caught and reported to the async operation. This resulted in the Async operation never completing.
This change adds two error results to the async operation:
Image Loading related Errors
All other XamlBuilder errors.
This will be overhauled once we switch the renderer to return a RenderResult instead of the Xaml elements directly.

@riarenas,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

Fixes #556


Support columns with non-nullable data types

Currently all columns have nullable data types. If a user has data that explicitly should not be null, this makes their code more verbose (they have to add ! everywhere) and doesn't enforce non-null input.
To support non-nullable data types, we should add a non-nullable column and/or non-nullable DataFrame.

0.4.0-nullsafety.0 is published on pub


Add asteroid to jcenter repository

Add asteroid to jcenter repository, and start looking into how to synchronize with mavenCentral. At the moment it is ok with jcenter only.

Bintray provides integration with Maven Central itself. It would be good to have Asteroid there.

I've already sent a request to Bintray. It shouldn't take long.

Request approved. In theory version 0.2.4 should be available through jcenter as well.

Older versions are already visible.
Btw, have you enabled Maven Central synchronization by the way?

Nope, I didn't notice. I'll check it out tomorrow. Thanks for the info :)

@szpak  I'm closing this issue with the jcenter syncrhonization, and I'm opening a new one with the maven central sync for next milestone.


updated versions for powertoys to 0.41.3, openvpn to 2.5.3

Only exe file available, no MSI provided by repo

Hi, just wanted to check-in about this PR since it's been a bit

Hi, just wanted to check-in about this PR since it's been a bit

Should be good to go!

This is actually totally broken because of the change from an exe to an msi. Scoop just extracts MSIs rather than actually trying to install them. I get the following error:
Installing 'openvpn' (2.5.3) [64bit]
Loading OpenVPN-2.5.3-I601-amd64.msi from cache
Checking hash of OpenVPN-2.5.3-I601-amd64.msi ... ok.
Extracting OpenVPN-2.5.3-I601-amd64.msi ... done.
Running pre-install script...
Running installer... error.
ERROR Exception calling "Start" with "0" argument(s): "The system cannot find the file specified"

Even if the manifest is simplified to just allow contents to be extracted, OpenVPN won't work because it's not properly installed (missing registry entries, TAP drive not installed, etc).


Switch to using static libs on Windows

Switching to the static version of sentry-native libs on Windows should allow bypassing issues with publishing to UE Marketplace (obfuscation of certain binaries required by Sentry plugin).
Related to #424

Why do we need specifically VS 2019 to build sentry-native?

In order to preserve the compatibility of sentry-native static libs with Unreal we have to stick to VS 2019. Afaik VS 2019 version remains available only at windows-2019 VMs so that's why we have to hold up with using windows-latest at this point.


Mitigating AWS Lamdbda limits

I'm sure you have already seen this AWS Lambda Limits.
I see the 1.5Gb total limit, and 50mb deploy limits as huge problems for the current deploy script.
Some things that I believe can help mitigate the impact of these limits are:

Implementing a build pipeline to minify the packaged source, here is a great example in Gulp
Using webpack or another compile time bundler to avoid copying external node_modules

Please correct me if I missed something, just trying to look ahead and identify areas that I might be able to contribute.

Austin

@austinrivas These are great suggestions.  You've been active with this project and it's much appreciated.
This weekend I'm going to publish the JAWS Road Map which will identify priorities for everyone to focus on.  The Road Map will include your suggestions.  Please know that I will be publicly acknowledging all contributors everywhere JAWS has a presence (e.g., in the repo, on the website, etc.).  Hopefully, we will assemble an awesome team to build the ultimate server-less framework.

In a pull request I have coming hopefully today, it will partially address the 50mb deploy limit. You will be able to setup file/dir ignores on a function by function basis in the lambda.json metadata file for each function.  I'm of course open to other ideas, just want to prevent you from spending time on solving the problem if you end up liking my solution.

@doapp-ryanp Awesome, this sounds super useful.  Can't wait to see it.

addressed in https://github.com/jaws-stack/JAWS/pull/42

Update 8/13/15:
We’re currently working on a document that outlines the specifications for JAWS v1.0.0.
A proposed resolution to this issue in particular is included in that document.
Today or tomorrow, we will share the document for everyone to comment on because your feedback is essential.

AWS recently pumped up the limit to 75GB ... Closing this for now.
Thanks guys! :blush:


Use new methods in Bocho

Codecov Report

Merging #37 into master will increase coverage by 0.26%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #37      +/-   ##
==========================================
+ Coverage    96.3%   96.56%   +0.26%
==========================================
  Files         144      144
  Lines        9035     9006      -29
  Branches      699      698       -1
==========================================
- Hits         8701     8697       -4
+ Misses        333      308      -25
  Partials        1        1


OWL-2646: Re-style the validation parts of Account, Reset Pass, and New Account

https://jira.uwo.ca/browse/OWL-2646
This pull request realigns the error messages and strength bars in:

Edit account
Edit individual user's account (As admin)
Create New Account
Password Reset

There was a little bit of work done to make the responsiveness look okay as well

@fostersdesign, when you're ready to merge this you can use the 'Squash and Merge' feature in the drop down to squash all the commits into one and edit the commit message accordingly.

This PR needs to be rebased before it can be merged.

@fostersdesign, conflicts and commit strangeness resolved. Might want to double check with @jenSadlerUWO that all her code changes are here.


Change the default cassette record mode to once

The current default cassette record mode is: none (see: support/vcr.rb) which:

allows previously recorded responses to be replayed but raises an error for any new requests and no new HTTP requests will be made

I find it's annoying since I have to change this option every time I want to record a new response AND change it back to none but there were many times I forget to revert the change and mistakenly commit that file.
I think we should change the default option to: once (document), which:

Replay previously recorded interactions and record new interactions if there is no cassette file.

another point of view is:

The upside of saibling recording is that left unchecked it forces dev to force it on purpose so they are aware that network requests are triggered. In terms of organization, it forces devs to name properly recordings. But the downside is that it’s manual to enable it.

so let's do a vote, which option do you prefer:

@longnd How about keeping to none in the config but instead document the use of RSpec metadata at the example level:
describe "Using an options hash", vcr: { record: :once } do
  it 'uses the provided cassette name' do
    #...
  end
end

Official documentation 👉 https://relishapp.com/vcr/vcr/v/2-4-0/docs/test-frameworks/usage-with-rspec-metadata#pass-a-hash-to-set-the-cassette-options

setting as one could leak a case that we forgot to commit the vcr file and on CI it will do a real request without noticing us, so we don't know about that case and it keeps requesting to the external server.
But setting it as none if we don't commit the vcr file, on CI it will raise the error so we can fix that.
What do you think @longnd

setting as once could leak a case that we forgot to commit the vcr

I think it's a valid point @anduonghien. Besides that, the voting results show that there're more people prefer keeping the once option.

setting as once could leak a case that we forgot to commit the vcr

I think it's a valid point @anduonghien. Besides that, the voting results show that there're more people prefer keeping the none option.

Thanks @longnd I think we can close this issue for now :)


Unable to broadcast transaction

Hello,
When I try to broadcast a transaction after succesfully creating the transaction hex, I'm unable to broadcast it, the "Broadcast" button is unresponsive.
EDIT: iOS version 11.3.1
EDIT 2: Ran test, and it returns OK

Fixed!
SSL cert expired on pushtx endpoint. Renewed it.
TODO: use several independent endpoins for pushtx

several endpoins implemented in https://github.com/Overtorment/BlueWallet/projects/1#card-10453427


Keep invalidated data on blur, but clear it if it's empty.

I know you can keep invalidated data with ui-options="{clearOnBlur:false}", but if the textbox is empty, this leaves a blank mask. Is it possible to show the placeholder if it's empty ?
Let's say ui-mask="+9<PHONE_NUMBER>", and placeholder="Phone". I want to keep a user's incomplete number but show "Phone" if they have no number input when they blur. Thanks

Is the functionality in this plunkr what you are trying to get?

Yes, exactly what I was looking for. I will look into it. Thanks!

Okay I'll try and get a PR for that change tomorrow. That was a quick, dirty, untested change in that plunkr just to check I understood you properly.


plugins/lsp: use a no-default option when there is no default provided

As per https://github.com/nix-community/nixvim/pull/2365#discussion_r1793846911

Hmm eval failing with those accessed but not defined errors

Ah this requires adaptations in the tests to pass null to the package it seems like

Hmm eval failing with those accessed but not defined errors

Ah this requires adaptations in the tests to pass null to the package it seems like

Should all be resolved now @khaneliman @traxys

@mergify queue


After Creating a new Company , creating a new Dispatcher User gives pop-up error of '{"user":"User with given username is already registered."}' in Create New User.(Company Panel)

Describe the bug
After Creating a new Company , creating a new Dispatcher gives pop-up error of '{"user":"User with given username is already registered."}' in Create New User. While this is a new company , creating new Dispatcher  should not give any such error.(possible reason is that I might created the dispatcher with the same user name in another company ).
To Reproduce
Steps to reproduce the behavior:
Steps to reproduce the behavior:
1.Enter in the address bar-pulseptt.com
2.Login With Id and Password.
UserName-tata
Password-12345678
3.Click On 'Create User' and Create a new Dispatcher.
Expected behavior
While this is a new company and for a new company creating new Dispatcher should not give any such error.
Desktop (please complete the following information):
OS: Windows 10
Browser: Version 100.0.4896.60 (Official Build) (64-bit)
Version: [0.3.8]

Creating a new dispatcher which was created in another company causing a server error.

Same server issue


Font resolution differences

Hello! Thanks for gemini, we love it. We're running into a problem where images generated on Mac OSX seem to have a different DPI/resolution than images generated headless on Ubuntu with xvfb-run (X11 server):

Anyone have an intuition or idea as to how we might fix this?
Thanks!

We have same problems due to different OS of our devs (mac, ubuntu, windows). We solved this problem by using dockerised browser. We using now selenium/standalone-chrome:3.0.1-germanium

@puzankov thanks for the tip.  Do you happen to have a link to something you followed to set that up, or an open source example of a project that is set up that way?
A bit of searching found this: https://hub.docker.com/r/roukmoute/testing-gemini/ which seems like a promising setup, but interested to know what easy to set up solutions worked for others (and how to replicate them).

also you can use some common grid to avoid such problems. For example, https://saucelabs.com

That's not necessary Gemini related. I can tell you, that happens with every form of visual regression testing. I recommend that you create the reference screenshots on the same system that is making the actual tests. So when you're using a CI/CD system that tests your screenshot for differences, also use the CI/CD system to generate the reference screenshots.
In my case with GitLab, I define a job to create new screenshots and commit/push them with a bot and never create local screenshots.


Box data support + cleanup

I've implemented a basic support for the box data from Tesseract (v3, but I guess the format hasn't changed since v2). I've also done a small cleanup (comments + PEP8 + Pylint).

Thanks... I'll merged the code but I'm really not interested in this project anymore. Don't you wanna do the maintenance?


(packaging) Update facter version to 3.6.7

Need to update the translations file as well

I think there's also a reference to the version in the POT file too.

@Magisus @branan updated


DR-1026 - Add ":" in all fields of Activation form

Hi team,
I added ":" in all name field of the registration form.
Could you Review?

Please, verify it because I have seen some places where : is set directly in the view, not in the literal.

I verified and doesn't found in the view that a field have ":",  the translation files have this characters


Ubuntu 1604 desktop access denied issue

Hi,
When trying to download the ubuntu 1604 ( https://vagrantcloud.com/box-cutter/boxes/ubuntu1604-desktop/versions/2.0.26/providers/virtualbox.box) getting access denied error
Response I am getting
Error>
AccessDenied
Access Denied
69683B94B5EAE6F5

ESPMhbpHKNnPAv5kIHXOtwhiFqukSY2EFfuQQCBx3fXuAPLtjAPdhzB7M0RHqCRuY1oqom9i2wI=

It was available few days back

Me to.
It's from http://cdn.boxcutter.io/ubuntu/virtualbox5.0.20/ubuntu1604-desktop-nocm-2.0.18.box and looks like a AWS S3 error.

See #138

We no longer provide pre-built binaries.


i18n(ko-KR): update deno.mdx

Description (required)

update deno.mdx
#9949

Related issues & labels (optional)

Closes #
Suggested label:

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

File
Note

ko/guides/deploy/deno.mdx
Localization changed, will be marked as complete.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.


docs: Clarify that knn_match limit is per shard

Or maybe more accurate:
-index per shard, and each shard will return at most `k` items. To limit the
-total query result, add a :ref:`LIMIT clause <sql-select-limit>` to the query.
+index per shard, and on each shard the function will match at most `k` records.
+To limit the total query result, add a :ref:`LIMIT clause <sql-select-limit>` to
+the query.

The function itself is not really returning rows, it's matching on them.

Or maybe more accurate:

-index per shard, and each shard will return at most `k` items. To limit the

-total query result, add a :ref:`LIMIT clause <sql-select-limit>` to the query.

+index per shard, and on each shard the function will match at most `k` records.

+To limit the total query result, add a :ref:`LIMIT clause <sql-select-limit>` to

+the query.

The function itself is not really returning rows, it's matching on them.

I think both are fine. The shard/lucene index still returns records matching the condition.


feat(connectors): add umbrel connector

Link this PR to an issue
Fixes #674
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update

Describe the changes you have made in this PR -
Begin to add the Umbrel connector, I repurposed the LND one for this because the backend should be the same. I still need to add the Umbrel specific steps to this so if anyone knows those can they please post them here.
I also changed the connectors screen back to having 4 connectors per row, we originally had it this way and only made it 5 because we thought having only 1 connector on the bottom row did not look good. Now with the new connectors being added the bottom row will be populated.
Screenshots of the changes (If any) -

How Has This Been Tested?
I added a LND node successfully.

[ ] Test A
[ ] Test B

Checklist:

[x] My code follows the style guidelines of this project and performed a self-review of my own code
[x] New and existing tests pass locally with my changes
[x] I have made corresponding changes to the documentation (If Any)

Note: this is not ready to be merged - still need to add the Umbrel steps in here.


no error is thrown if you try to update a habit which hasn't been added

Out of interest, I tried to add a habit with more than one word by executing:
atomify new "music practice" 1

this enters the task into atomify correctly however I made a mistake in my next command but no error was thrown and nothing was listed when I accidentally mistyped the habit again.
atomify ++ "music practive"

If you don't have time to fix this then I'm quite happy to fix this over the weekend :)

Error handling not a priority for now


The code is not compatible with torch 2.3, but is set to require the latest torch

Pytorch v2.3.0 no longer defines function _accumulate in torch/_utils.py. The function is called from mlcolvar/data/datamodule.py.
Either the code needs to be changed to be executable with the new pytorch, or an older pytorch should be used in requirements.

Ciao Alexander, thanks, we also noticed that, and it should have been fixed in #129.
The tests on the main branch are indeed running fine and they require torch 2.3.
To be sure, are you using an older version of the library with the new pytorch?  For example, the PyPi version is bound to the latest stable release.

We should make a bugfix in the pypi version

Hi Enrico, hi Luigi,
thank you!
Yes, the issue was with the PyPi version. After that, I used locally installable version and it worked fine.
I also believe that this issue concerns tutorial colabs, now the only way to use them is to manually add lines to install pytorch 2.2.0.
But it is all solvable, just a small nuicance.
Thank you for the fix!
Alex

Thank you Alexander for making us notice it!
Thanks also to @luigibonati for addressing the issue.


Don't use strings.repeat - use * instead

LGTM, but this package is deprecated (as noted on the readme).


Is there any clear way to have multiple app servers?

Thanks for your great clean work here. Is it possible to scale the app servers with a single database server? here is the scenario:

Multiple app servers each of the have multiple IP addresses attached to them (their own IP Pool).
Single MySQL database
Single RabbitMQ
Putting a loadbalancer in front of the web interface (for open and click tracking)
Putting a loadbalancer in front of the SMTP server or routing in the client application itself.

See this topic:
https://github.com/atech/postal/issues/41
But if you just need multiple ips, you can use IP Pools

This is also confirmed by https://github.com/atech/postal/issues/334#issuecomment-326765678
If each IP Pool was actually one server, it would be good to confirm that Postal can handle such a "physical" separation and each server only send emails it was supposed to.

@catphish Do we have any update on this?
Let's say I have a common MySQL and RabbitMQ server, and connect multiple postal instances to the same server, would that be enough? Or is there anything else that I need to add to the setup to retain the same data across both servers?

This should work fine. There is no documented installation process, but if you install Postal on 2 servers and point them to a common MySQL an Rabbit server, it should work fine.

You can either loadbalance SMTP, or just send emails to one server in the cluster and and they will all send.
Servers should be able to share an IP pool, as long as the each have at least one IP from that pool.

This is also confirmed by #334 (comment)
If each IP Pool was actually one server, it would be good to confirm that Postal can handle such a "physical" separation and each server only send emails it was supposed to.

Any additional information about that? Using IP Pools with smtp server running on distributed servers.
If it works, how to setup additional smtp servers?


feats: immediate & carryover response split

updated ci range calculation
added ci plot to onepager
new robyn_calibrate() function to replace calibrate_mmm(), enabling calibration on only immediate & future carryover effect, excluding historical carryover
added immediate carryover plot to onepager


NO Language Syntax Available

Please add Syntax for _.ahk_ file I can't use it's extenstion because all extenstion's are dead & outdated (last modified 2016 & now is going on 2022, still not updated). Also, add in it Emmet.
Thank You

Emmet is mainly for HTML and CSS. Your use case is better handled by user snippets.

Emmet is mainly for HTML and CSS. Your use case is better handled by user snippets.

Ok, Please add Syntax Highlighting

We try to keep VS Code lean and we think the functionality you're asking for is great for a VS Code extension. Maybe you can already find one that suits you in the VS Code Marketplace. Just in case, in a few simple steps you can get started writing your own extension. See also our issue reporting guidelines.
Happy Coding!


Howdy yall, I updated readme & added requirements.txt

Howdy yall, I updated readme & added requirements.txt

Howdy pardner! That's a mighty fine job you did there! Thanks so much!


Error: INSTALLATION FAILED: cannot re-use a name that is still in use

Using the ConfigMap option to create a certificate, seems to fail when trying to add a second host.
This succeeds: helm install 1.2.0 nabsul/kcert-configmap -n kcert --debug --set name=kcert,hosts="test1.mydomain.com"
This fails: helm install 1.2.0 nabsul/kcert-configmap -n kcert --debug --set name=kcert2,hosts="test2.mydomain.com"
Any idea why?

I think this is a misunderstanding of help and the configmap. You have created a helm deployment named 1.2.0 in the above example, so trying to create a second config map with the same name is giving you this error.
Two solutions:

Use a different name than 1.2.0 for the second host name.
Upgrade/Update the existing helm deployment and provide a comma separated list of all your hosts.


RandomFlip is not in the pipeline registry

Hi, great work. I see 'RandomFlip' in some configs, i.e., imagenet_128.py, I wonder if it is the same as 'Flip' in augmentation.py. Since I cannot find any function related to 'RandomFlip', could you please provide some details about it? Thx.

Oh, I figured out that it should be from mmcls. Sorry.


[Bug]: How to unify grids to resolve this error?

SBAS and PSI analysis on Otmanbozdagh Mud Volcano, Azerbaijan, 2020-2023
Fails at "SBAS Trend Correction":   ---------------------------------------------------------------------------
AssertionError                            Traceback (most recent call last)
in ()
4 inc = decimator(sbas.incidence_angle())
5 yy, xx = xr.broadcast(topo.y, topo.x)
----> 6 trend_sbas = sbas.regression(unwrap_sbas.phase, [topo, topoyy, topoxx, topoyyxx, yy, xx, inc], corr_sbas)
/usr/local/lib/python3.10/dist-packages/pygmtsar/Stack_detrend.py in regression(data, variables, weight, valid_pixels_threshold, **kwargs)
233             vshapes = [v[0].shape if v.ndim==3 else v.shape for v in variables]
234             equals = np.all([vshape == dshape for vshape in vshapes])
--> 235             assert equals, f'ERROR: shapes of variables slices {vshapes} and data slice {dshape} differ.'
236             #assert equals, f'{dshape} {vshapes}, {equals}'
237             variables_stack = [v.chunk(dict(y=chunk2d[0], x=chunk2d[1])) for v in variables]
AssertionError: ERROR: shapes of variables slices [(776, 940), (776, 940), (776, 940), (776, 940), (776, 940), (776, 940), (776, 940)] and data slice (775, 940) differ.
Processing Log
If applicable, attach a log file from notebook cell or terminal output or relevant error messages
[Please note you can use the ChatGPT4-based InSAR AI Assistant for guidance or troubleshooting. Visit https://insar.dev/ai for more information.]

Use xarray reindex_like() function to unify the grids: array1.reindex_like(array2).

Thanks Alexey.  Can you please provide code with actual array1 and array2?  Perhaps provide an updated Colab notebook?

Here is the Xarray documentation: https://docs.xarray.dev/en/latest/generated/xarray.DataArray.interp_like.html
Example notebooks with the function usage are shared on my Patreon, but it is pretty obvious.

Thanks Alexey,
I modified using the Xarray function (as suggested by ChatGPT) and the notebook successfully executed that step (SBAS Trend Correction), and the remaining SBAS steps. GREAT.
Unfortunately there was an incomplete red progress indicator at a later PS processing step during materialization:

In subsequent steps the PS plots looked empty:

Here is a my revised Colab notebook:  https://drive.google.com/file/d/100g6LRKmWc-geL6OMFLv6Ytoa59VFNy4/view?usp=sharing

There are some pixel-wise inconsistencies between the interferograms valid pixels in the stack, which can be resolved as follows:
# check pixel stacks
validmask_ps = np.isfinite(intf_ps).all(axis=0)
# optionally, materialize to disk and open
validmask_ps = sbas.sync_cube(validmask_ps, 'validmask_ps')
validmask_ps.where(validmask_ps).plot.imshow()

disp_ps_pairs = sbas.los_displacement_mm(sbas.unwrap1d(
    intf_ps.where(validmask_ps)
    ))
disp_ps_pairs

My notebook completed and I calculated rate (mm/year) from disp_sbas output by linear regression (is there an export function for mm/year??):
But I have very different SBAS results to the Imperial Valley example  for the same area, same scenes):
Otmanbozdagh_mud (mm/yr):

Imperial_Valley (mm/yr):

You can see this makes a big difference in the maps (points labelled mm/year):
Otmanbozdagh_mud (mm/yr):

Imperial_Valley (mm/yr):

Can you provide an explanation for these different results?  What parameters can I adjust?

The examples are very different. In the Imperial Valley case, we remove very strong stratified plus turbulent atmospheric noise using Gaussian filtering, whereas the Otmanbozdagh mud volcano shows pretty good results with linear regression trend correction. How do you propose we unify the nature of these processes? :)

It would be very useful to have a general purpose example that can be applied to a variety of areas, and where the parameters are well documented.  StaMPS has been very good in this respect, as results with default parameters are very good (agree with convectional levelling contours).  I have spent a great deal of time trying to adapt these examples to my own areas but perhaps they are too specific?  For example, I gave up trying to modify the Otmanbozdagh mud volcano because the results were so different from the more simple Imperial Valley (I cant imagine it is different atmospheric filtering).  But the Imperial Valley example has very coarse resolution (60 m), with a large wavelength (400 m).  I have spent many hours trying to decrease the Imperial Valley wavelength and grid size (to reduce the spatial smoothing) so I can examine smaller-scale phenomenon, without success.  Is there anywhere I can access more documentation to allow me to better modify the examples?  Would your book be helpful?  As a Patreon subscriber do I have access to the book?  Thanks, Mark

Mark, StaMPS does not have any parameters to process distributed scatterers at all, and it is primarily a persistent scatterers (PS) analysis tool, as mentioned in its name. The PyGMTSAR Imperial Valley notebooks illustrate DS SBAS analysis in the absence of persistent scatterers, making it impossible to produce complete area coverage results using StaMPS and other PSI tools. The Otmanbozdagh mud volcano is a completely different case, having a great density of PS. As illustrated in the notebook, both PSI and SBAS methods can produce very similar results.
If you apply DS analysis as in the Imperial Valley case to sparse persistent scatterers, it does not work due to exact physical reasons. PyGMTSAR allows for both PS and DS analysis using PSI and SBAS techniques, but it is impossible to unify the different physics into a single standalone example. PS are smaller than a single Sentinel-1 pixel and are affected by speckle noise and often sparse, whereas DS are larger than a pixel, and speckle noise is not an issue because it is effectively removed using anti-aliasing filtering and proper DS analysis scale selection typically can produce dense but lower resolution pixels.
There are many practical cases where not-so-rare PS can be grouped into DS, and all analysis methods work well enough, although they still have some limitations, as explained in my publications available on Patreon.

Thanks Alexey,Your explanations are useful and appreciated.  Imperial Valley:  Is it possible to reduce the grid spacing from 60 to 30 meters (or 15 meters)? If so, which parameters?  Please note this is for my own study area in New Zealand, there are many PS in the area.  I have already used the Imperial Valley example in this area and the results are consistent with StaMPS.  However, I would like to see if results can be improved; e.g. by reducing averaging of DS, using a smaller grid size.   If not possible to do this, then perhaps the Otmanbozdagh_mud example will be a better template?Thanks,MarkSent from my iPhoneOn 11/05/2024, at 12:39 AM, Alexey Pechnikov @.***> wrote:
Mark, StaMPS does not have any parameters to process distributed scatterers at all, and it is primarily a persistent scatterers (PS) analysis tool, as mentioned in its name. The PyGMTSAR Imperial Valley notebooks illustrate DS SBAS analysis in the absence of persistent scatterers, making it impossible to produce complete area coverage results using StaMPS and other PSI tools. The Otmanbozdagh mud volcano is a completely different case, having a great density of PS. As illustrated in the notebook, both PSI and SBAS methods can produce very similar results.
If you apply DS analysis as in the Imperial Valley case to sparse persistent scatterers, it does not work due to exact physical reasons. PyGMTSAR allows for both PS and DS analysis using PSI and SBAS techniques, but it is impossible to unify the different physics into a single standalone example. PS are smaller than a single Sentinel-1 pixel and are affected by speckle noise and often sparse, whereas DS are larger than a pixel, and speckle noise is not an issue because it is effectively removed using anti-aliasing filtering and proper DS analysis scale selection typically can produce dense but lower resolution pixels.
There are many practical cases where not-so-rare PS can be grouped into DS, and all analysis methods work well enough, although they still have some limitations, as explained in my publications available on Patreon.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

The Imperial Valley example uses a wavelength of 400 meters for Gaussian filtering (antialiasing and speckle removal) and spatial averaging (multilooking) into a ~60-meter square grid (16x4=64 original pixels aggregated into one) to enhance the signal-noise ratio and correlation, plus a Goldstein filter is applied. This processing is targeted at highly decorrelated areas with high radiometric noise and strong atmospheric effects. It's possible to apply a much lower wavelength, starting from about 4 meters, and to exclude multilooking and the Goldstein filter if you have more frequent and higher-quality interferograms in a well-coherent area like in the Golden Valley example. But why do you need to improve the SBAS results? Use these to start your PSI processing, utilizing SBAS-detected phase ramps to produce higher resolution results, as demonstrated in the Otmanbozdagh mud volcano example. PSI 1D unwrapping is more accurate but requires a priori known phase corrections, which we define using less accurate and lower-resolution SBAS analysis.

Thanks Alexey!
To answer your question: why do I need to improve SBAS results?
Here is my situation - I have a study: a two year time interval in a
geothermal area with a subsidence bowl.
StaMPS results in color shading (see legend).  Note: ascending and
descending combined to give an estimate of vertical.  White contours are
from a ground-based levelling survey (same time period) showing a maximum
of -90 mm/year.  StaMPS is in good agreement with -85 mm/year (at bowl
center).
[image: image.png]
Now, Imperial SBAS results in color, showing a maximum of* -60 mm/year*,
quite nice, but I am hoping for better agreement with StaMPS and
levelling.  So maybe we can just reduce the wavelength to ~100m, and reduce
the grid size to ~30 m?  I have tried, but get errors because I don't know
what parameters to change.
[image: image.png]
On Mon, May 13, 2024 at 5:22 AM Alexey Pechnikov @.***>
wrote:

The Imperial Valley example uses a wavelength of 400 meters for Gaussian
filtering (antialiasing and speckle removal) and spatial averaging
(multilooking) into a ~60-meter square grid (16x4=64 original pixels
aggregated into one) to enhance the signal-noise ratio and correlation,
plus a Goldstein filter is applied. This processing is targeted at highly
decorrelated areas with high radiometric noise and strong atmospheric
effects. It's possible to apply a much lower wavelength, starting from
about 4 meters, and to exclude multilooking and the Goldstein filter if you
have more frequent and higher-quality interferograms in a well-coherent
area like in the Golden Valley example. But why do you need to improve the
SBAS results? Use these to start your PSI processing, utilizing
SBAS-detected phase ramps to produce higher resolution results, as
demonstrated in the Otmanbozdagh mud volcano example. PSI 1D unwrapping is
more accurate but requires a priori known phase corrections, which we
define using less accurate and lower-resolution SBAS analysis.
—
Reply to this email directly, view it on GitHub
https://github.com/AlexeyPechnikov/pygmtsar/issues/128#issuecomment-2106319789,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMOB4Q5T573RYLKESEIKBJTZB6QL5AVCNFSM6AAAAABHE6PHX6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMBWGMYTSNZYHE
.
You are receiving this because you authored the thread.Message ID:
@.***>

Thanks Alexey!
Please disregard last email that contained an error
To answer your question: why do I need to improve SBAS results?
Here is my situation - I have a study: a two year time interval in a
geothermal area with a subsidence bowl.
StaMPS results in color shading (see legend).  Note: ascending and
descending combined to give an estimate of vertical.  White contours are
from a ground-based levelling survey (same time period) showing a maximum
of -90 mm/year.  StaMPS is in good agreement with -85 mm/year (at bowl
center).
[image: image.png]
Now, Imperial SBAS results in color, showing a maximum of* -30 mm/year*,
the shape is good, but I am hoping for better agreement with StaMPS and
levelling in terms of rate (mm/year).  So maybe we can just reduce the
wavelength to ~30m and grid size to ~30 m?  I have tried, but get
errors because I don't know what parameters to change.
[image: image.png]
On Sun, May 12, 2024 at 10:22 AM Alexey Pechnikov @.***>
wrote:

The Imperial Valley example uses a wavelength of 400 meters for Gaussian
filtering (antialiasing and speckle removal) and spatial averaging
(multilooking) into a ~60-meter square grid (16x4=64 original pixels
aggregated into one) to enhance the signal-noise ratio and correlation,
plus a Goldstein filter is applied. This processing is targeted at highly
decorrelated areas with high radiometric noise and strong atmospheric
effects. It's possible to apply a much lower wavelength, starting from
about 4 meters, and to exclude multilooking and the Goldstein filter if you
have more frequent and higher-quality interferograms in a well-coherent
area like in the Golden Valley example. But why do you need to improve the
SBAS results? Use these to start your PSI processing, utilizing
SBAS-detected phase ramps to produce higher resolution results, as
demonstrated in the Otmanbozdagh mud volcano example. PSI 1D unwrapping is
more accurate but requires a priori known phase corrections, which we
define using less accurate and lower-resolution SBAS analysis.
—
Reply to this email directly, view it on GitHub
https://github.com/AlexeyPechnikov/pygmtsar/issues/128#issuecomment-2106319789,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMOB4Q5T573RYLKESEIKBJTZB6QL5AVCNFSM6AAAAABHE6PHX6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMBWGMYTSNZYHE
.
You are receiving this because you authored the thread.Message ID:
@.***>

Unfortunately, the images are not visible in your last messages.
In the Imperial Valley case, we use a wavelength of 400 meters to detect larger DS instead of the default ("gold standard") value of 200 meters. For small DS, we can apply a wavelength of 100 meters and even 60 meters for PS detection. However, this is tricky for SNAPHU unwrapping, so I do not recommend using a wavelength less than 100 meters. It's preferable to use PSI analysis after SBAS to enhance your results.
# Gaussian filtering 400m cut-off wavelength with multilooking 1x4 on Sentinel-1 intensity
intensity = sbas.multilooking(np.square(np.abs(data)), wavelength=400, coarsen=(1,4))
phase = sbas.multilooking(sbas.phasediff(pairs), wavelength=400, coarsen=(1,4))

Additionally, we can specify a more detailed output grid like 30 meters using sbas.decimator(30) instead of the default 60-meter one:
# use default 60m resolution
decimator = sbas.decimator()

Commonly, there is no reason for a higher resolution output SBAS grid due to SNAPHU unwrapping error accumulation.
The Goldstein filter can introduce some spatial inaccuracies while it helps to clean up the interferograms' spatial spectrum:
# Goldstein filter expects square grid cells produced using multilooking
intf_filt = sbas.interferogram(sbas.goldstein(phase, corr, 32))

Typically, we do not need to exclude the filter because it can be adjusted to use a smaller processing window of 16 pixels as sbas.goldstein(phase, corr, 16).
As shown in my other examples, for many InSAR cases, the best approach is to apply SBAS analysis with default or nearly default parameters to produce stable results on a rough grid and enhance it using PS analysis with default or close to default parameters too, to calculate the most accurate displacements on the original Sentinel-1 resolution grid. Of course, SBAS and PSI results need to be consistent to validate the analyses. If you only need an intermediate output resolution like 30 meters, sometimes it is sufficient to apply only SBAS processing.

Perfect, I'll give it a try with your suggestions.
I have attached the images FYI.
Mark
On Sun, May 12, 2024 at 11:11 PM Alexey Pechnikov @.***>
wrote:

Unfortunately, the images are not visible in your last messages.
In the Imperial Valley case, we use a wavelength of 400 meters to detect
larger DS instead of the default ("gold standard") value of 200 meters. For
small DS, we can apply a wavelength of 100 meters and even 60 meters for PS
detection. However, this is tricky for SNAPHU unwrapping, so I do not
recommend using a wavelength less than 100 meters. It's preferable to use
PSI analysis after SBAS to enhance your results.
Gaussian filtering 400m cut-off wavelength with multilooking 1x4 on Sentinel-1 intensity
intensity = sbas.multilooking(np.square(np.abs(data)), wavelength=400, coarsen=(1,4))
phase = sbas.multilooking(sbas.phasediff(pairs), wavelength=400, coarsen=(1,4))
Additionally, we can specify a more detailed output grid like 30 meters
using sbas.decimator(30) instead of the default 60-meter one:
use default 60m resolution
decimator = sbas.decimator()
Commonly, there is no reason for a higher resolution output SBAS grid due
to SNAPHU unwrapping error accumulation.
The Goldstein filter can introduce some spatial inaccuracies while it
helps to clean up the interferograms' spatial spectrum:
Goldstein filter expects square grid cells produced using multilooking
intf_filt = sbas.interferogram(sbas.goldstein(phase, corr, 32))
Typically, we do not need to exclude the filter because it can be adjusted
to use a smaller processing window of 16 pixels as sbas.goldstein(phase,
corr, 16).
As shown in my other examples, for many InSAR cases, the best approach is
to apply SBAS analysis with default or nearly default parameters to produce
stable results on a rough grid and enhance it using PS analysis with
default or close to default parameters too, to calculate the most accurate
displacements on the original Sentinel-1 resolution grid. Of course, SBAS
and PSI results need to be consistent to validate the analyses. If you only
need an intermediate output resolution like 30 meters, sometimes it is
sufficient to apply only SBAS processing.
—
Reply to this email directly, view it on GitHub
https://github.com/AlexeyPechnikov/pygmtsar/issues/128#issuecomment-2106728418,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMOB4Q37YZ7PJHTJP4O3GMTZCBKR3AVCNFSM6AAAAABHE6PHX6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMBWG4ZDQNBRHA
.
You are receiving this because you authored the thread.Message ID:
@.***>

Thanks Alexey,
I took your advice and tried changing the Imperial Valley to a 30m grid:
decimator = sbas.decimator(30)
This gives an error at the later step:
tqdm_dask(unwrap_ll := sbas.ra2ll(unwrap.phase).persist(), desc='Geocoding')
Error:  .Transforming grid spacing (grid_dy, grid_dx) is smaller than
transform matrix spacing (trans_dy, trans_dx),   call Stack.geocode()
with less coarsing
I tried changing coarsen=(1,4) to coarsen=(1,1),
then I tried changing coarsen=(1,4) to coarsen=(1,2)
..but still get the same error
What other parameters do I need to change Imperial Valley to a 30m grid?
Thanks,
Mark
I tried changing
On Sun, May 12, 2024 at 11:11 PM Alexey Pechnikov @.***>
wrote:

Unfortunately, the images are not visible in your last messages.
In the Imperial Valley case, we use a wavelength of 400 meters to detect
larger DS instead of the default ("gold standard") value of 200 meters. For
small DS, we can apply a wavelength of 100 meters and even 60 meters for PS
detection. However, this is tricky for SNAPHU unwrapping, so I do not
recommend using a wavelength less than 100 meters. It's preferable to use
PSI analysis after SBAS to enhance your results.
Gaussian filtering 400m cut-off wavelength with multilooking 1x4 on Sentinel-1 intensity
intensity = sbas.multilooking(np.square(np.abs(data)), wavelength=400, coarsen=(1,4))
phase = sbas.multilooking(sbas.phasediff(pairs), wavelength=400, coarsen=(1,4))
Additionally, we can specify a more detailed output grid like 30 meters
using sbas.decimator(30) instead of the default 60-meter one:
use default 60m resolution
decimator = sbas.decimator()
Commonly, there is no reason for a higher resolution output SBAS grid due
to SNAPHU unwrapping error accumulation.
The Goldstein filter can introduce some spatial inaccuracies while it
helps to clean up the interferograms' spatial spectrum:
Goldstein filter expects square grid cells produced using multilooking
intf_filt = sbas.interferogram(sbas.goldstein(phase, corr, 32))
Typically, we do not need to exclude the filter because it can be adjusted
to use a smaller processing window of 16 pixels as sbas.goldstein(phase,
corr, 16).
As shown in my other examples, for many InSAR cases, the best approach is
to apply SBAS analysis with default or nearly default parameters to produce
stable results on a rough grid and enhance it using PS analysis with
default or close to default parameters too, to calculate the most accurate
displacements on the original Sentinel-1 resolution grid. Of course, SBAS
and PSI results need to be consistent to validate the analyses. If you only
need an intermediate output resolution like 30 meters, sometimes it is
sufficient to apply only SBAS processing.
—
Reply to this email directly, view it on GitHub
https://github.com/AlexeyPechnikov/pygmtsar/issues/128#issuecomment-2106728418,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMOB4Q37YZ7PJHTJP4O3GMTZCBKR3AVCNFSM6AAAAABHE6PHX6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMBWG4ZDQNBRHA
.
You are receiving this because you authored the thread.Message ID:
@.***>

For geocoding the results, you need to create the proper geocoding matrix like:
# use the original Sentinel-1 resolution (1 pixel spacing)
sbas.compute_geocode(1)

Select a suitable coarsening factor instead of 1 (it works too but creates a matrix much larger than you actually need).

Thanks Alexey, this solved my issue!Sent from my iPhoneOn 15/05/2024, at 12:34 AM, Alexey Pechnikov @.***> wrote:
For geocoding the results, you need to create the proper geocoding matrix like:
use the original Sentinel-1 resolution (1 pixel spacing)
sbas.compute_geocode(1)
Select a suitable coarsening factor instead of 1 (it works too but creates a matrix much larger than you actually need).
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

@AlexeyPechnikov, Hello Alexey, I tried a few times on the colab file (otmanbozdagh) shared in this discussion for my own field and found it successful. After that, since this notebook is commercial (exclusive to Patreon members), I wanted to buy a 1-month subscription from your patreon account. However, when we wanted to make a payment in Turkey, it was rejected by the system. I want to pay you for a month and I want to become a member of patreon. Is there another way for this, can you help me?
@wharfbanger ,Thank you for sharing your code, I realized that I made changes to the same file because I didn't realize it was editable. I'm sorry that I messed up your file, I tried to restore it as much as I could but there must have been parts that were changed and deleted. I'm so sorry.

@oguzhannysr
There are several payment methods listed here: Patreon Support. Additionally, it could be a recurring payment issue, but you might still be able to purchase the single file using the same payment method.

@wharfbanger , ... I'm sorry that I messed up your file

This shouldn’t be an issue, as Google Colab notebooks are stored on Google Drive and are versioned. The author can easily revert to a previous version of the notebook if needed:
https://knowledge.workspace.google.com/kb/how-to-recover-versions-of-a-google-drive-document-000007648
And of course, we should avoid sharing important files publicly on the internet where they can be edited by anyone

@AlexeyPechnikov ,Alexey, I purchased an essential membership but I could not reach otmanbozdag.ipnyb. I signed up for this reason, can you help me? I couldn't find the link to Colab.


When I run gestureCNN.py,I have meet errors

When I run gestureCNN.py on windows platform, I meet the following errors:
ImportError:Failed to import pydot.You must install pydot and graphviz for 'pydotprint' to work.
After I pip install pydot,pip install graphviz.The errors are still.
How can I to do this?
Thank you .

Hi there,
Sorry just now noticed this issue. Can you please tell me at which line are you getting this error msg ??

Are you getting it for following line of code ?? If yes, then a quick workaround would be to comment out these two lines. Its only required to display model configuration visually. This info is anyhow being printed in the console. So not that critical requirement for the code.
from keras.utils import plot_model
plot_model(model, to_file='new_model.png', show_shapes = True)

Yes ,I have solved that problem.Another error appeared again.
ljy1@ubuntu:~/CNNGestureRecognizer$ KERAS_BACKEND=theano python trackgesture.py
Using Theano backend.
What would you like to do ?
1- Use pretrained model for gesture recognition & layer visualization
2- Train the model (you will require image samples for training under .\imgfolder)
3- Visualize feature maps of different layers of trained model
1
Will load default weight file

Layer (type)                 Output Shape              Param #
conv2d_1 (Conv2D)            (None, 32, 198, 198)      320

activation_1 (Activation)    (None, 32, 198, 198)      0

conv2d_2 (Conv2D)            (None, 32, 196, 196)      9248

activation_2 (Activation)    (None, 32, 196, 196)      0

max_pooling2d_1 (MaxPooling2 (None, 32, 98, 98)        0

dropout_1 (Dropout)          (None, 32, 98, 98)        0

flatten_1 (Flatten)          (None, 307328)            0

dense_1 (Dense)              (None,<PHONE_NUMBER>2

activation_3 (Activation)    (None, 128)               0

dropout_2 (Dropout)          (None, 128)               0

dense_2 (Dense)              (None, 5)                 645

activation_4 (Activation)    (None, 5)                 0
Total params: 39,348,325
Trainable params: 39,348,325
Non-trainable params: 0

loading  ori_4015imgs_weights.hdf5
Traceback (most recent call last):
File "trackgesture.py", line 300, in
Main()
File "trackgesture.py", line 175, in Main
mod = myNN.loadCNN(0)
File "/home/ljy1/CNNGestureRecognizer/gestureCNN.py", line 176, in loadCNN
model.load_weights(fname)
File "/home/ljy1/anaconda2/lib/python2.7/site-packages/keras/models.py", line 715, in load_weights
f = h5py.File(filepath, mode='r')
File "/home/ljy1/anaconda2/lib/python2.7/site-packages/h5py/_hl/files.py", line 272, in init
fid = make_fid(name, mode, userblock_size, fapl, swmr=swmr)
File "/home/ljy1/anaconda2/lib/python2.7/site-packages/h5py/_hl/files.py", line 92, in make_fid
fid = h5f.open(name, flags, fapl=fapl)
File "h5py/_objects.pyx", line 54, in h5py._objects.with_phil.wrapper (/home/ilan/minonda/conda-bld/work/h5py/_objects.c:2696)
File "h5py/_objects.pyx", line 55, in h5py._objects.with_phil.wrapper (/home/ilan/minonda/conda-bld/work/h5py/_objects.c:2654)
File "h5py/h5f.pyx", line 76, in h5py.h5f.open (/home/ilan/minonda/conda-bld/work/h5py/h5f.c:1942)
IOError: Unable to open file (File signature not found)
And I have downloaded ori_4015imgs_weights.hdf5.

Can you tell me what is the size of ori_4015imgs_weights.hdf5 file you are seeing? It should be ~150MB.

Yes,the error was ori_4015imgs_weights.hdf5,I have downloaded another time.
Thank you!

Hi:
I met another problem again.
ljy1@ubuntu:~/CNNGestureRecognizer$ KERAS_BACKEND=theano python trackgesture.py
Using Theano backend.
What would you like to do ?
1- Use pretrained model for gesture recognition & layer visualization
2- Train the model (you will require image samples for training under .\imgfolder)
3- Visualize feature maps of different layers of trained model
1
Will load default weight file
......
Total params: 39,348,325
Trainable params: 39,348,325
Non-trainable params: 0

loading  ori_4015imgs_weights.hdf5
ori_4015imgs_weights.hdf5
OpenCV Error: Assertion failed (size.width>0 && size.height>0) in imshow, file /root/mc-x64-2.7/conda-bld/opencv-3_1482256854890/work/opencv-3.1.0/modules/highgui/src/window.cpp, line 281
Traceback (most recent call last):
File "trackgesture.py", line 318, in
Main()
File "trackgesture.py", line 244, in Main
cv2.imshow('Original',frame)
cv2.error: /root/mc-x64-2.7/conda-bld/opencv-3_1482256854890/work/opencv-3.1.0/modules/highgui/src/window.cpp:281: error: (-215) size.width>0 && size.height>0 in function imshow
But,I have installed opencv,and I can use opencv to read videos and images.
ljy1@ubuntu:~/CNNGestureRecognizer$ pkg-config --modversion opencv
2.4.8
I don't know why.

This error msg is actually pointing that camera input is not getting captured actually. Can you try this:
At following line in trackgesture.py file where I am setting up VideoCapture using input value as 0. Pass in value as 1 and check if it helps
Grab camera input
cap = cv2.VideoCapture(1)

Any update @liujinyan12 ??

@liujinyan12  I am closing this issue. Feel free to reopen in case of any other issue you come across.


How to use the docker image on a developer machine?

By default on Docker for Windows, any of the commands don't seem to mount the local source directory where you start the command.
Maybe add a few lines to say how to use the docker image to scan your local source files?
(Edit: I also notice that the included psalm v1.1.9 is 1.5 years old)

1) Mount the local source directory
You might need to use different environment variable. -v $PWD:/app probably does not work for Windows.

-v $PWD : Passes docker the existing working directory for analyzing
https://hub.docker.com/r/eko3alpha/docker-phpqa/

2) Tools in docker image
Please see https://github.com/EdgedesignCZ/phpqa/issues/159#issuecomment-452794397 and https://github.com/EdgedesignCZ/phpqa/issues/168#issuecomment-489180974

Thanks for the reply. I'd still vote for including some of this in the readme 👍


HyperLink Bad Performance

Description
I'm generating an excel file with only 3000 rows and 15 columns. Two columns have hyperlink which I set up as in docs said. The problem is that this task is taking so long (almost 9 minutes). The problem is that If I remove the hyperlinks then everything work fine (between 10 - 15 seconds). Is there anyway that I can set hyperlinks withou the name? or convert a range of rows to hyperlinks?

Steps to reproduce the issue:

Loop 3000 items.
Set rows values.
Set hyperlink row values.

Describe the results you received:
2 minutes of waiting for the excel file.
Describe the results you expected:
Less time generating the excel.
Output of go version:
go version go1.10.1 windows/amd64
(paste your output here)

Excelize version or commit ID:
1.4.0
(paste here)

Environment details (OS, Microsoft Excel™ version, physical, etc.):

Maybe a temporary solution: would it make sense to do the everything except hyperlink first, and then generate hyperlink actually in Excel using the HYPERLINK function?
https://support.office.com/en-us/article/hyperlink-function-333c7ce6-c5ae-4164-9c47-7de9b76f577f

Its not a good solution, since there is more than 3000 rows. I can’t just write the links manually. For now Im just writing those links as plain text

you can generate the url and the name programmatically as helper columns, and then simply pass them to the HYPERLINK and fill all the rows? That probably only take a minute.

Hi @jorgeb12, sorry for my late reply. I have optimized the performance of add hyperlink, please upgrade the library to the last version.


Result is bad

when I used  " python inference.py --checkpoint_file best_ckpt/g_best " for tested my noisy.wav, The result.wav is bad .The noise is still preserved. Did I make a mistake in my steps？

Is your noisy speech data from your own dataset or from the VoiceBank+DEMAND dataset?
Does it at a sampling rate of 16 kHz?

Is your noisy speech data from your own dataset or from the VoiceBank+DEMAND dataset? Does it at a sampling rate of 16 kHz?

It's my dataset and 16kHz. But universality is required. Does it rely strongly on training data?

The universality is also related to the dataset size.
Our model was trained on the Voicebank+DEMAND dataset, which is not that big.
You can fine-tune our model with your data for better performance.


T58955 Export animated FBX to glTF 2.0

Bug from blender tracker: https://developer.blender.org/T58955

If it's helpful I've tested and documented a workflow for FBX files from the source mentioned, Mixamo, in the past: https://www.donmccurdy.com/2017/11/06/creating-animated-gltf-characters-with-mixamo-and-blender/. I haven't re-tested this since Blender 2.8, however, and it would be an important workflow to keep intact. 👍

@UX3D-nopper I'm moving this to MS VII because it seems to be skinning-related, and I assume that the more examples we have, easier it will be to identify the correct fixes. If you think this should be left out of the milestone please let me know.

^the material issue can be considered out a separate problem for now.

also fixed by #412


Make sure our unit tests break if we drop regexp support.

(Hey Manu, this is not urgent at all, putting up the PR just so I don't forget)
This is an undocumented feature about how our config options work for matching e.g. package names. I think is it worthwhile to have something warn us if we drop support accidentally, but not sure if we are committing to supporting this non-standard regexp format (The format is just Java regexp but . is interpreted as the literal dot character, not "any character" because of how normal package names look like).

@lazaroclapp can you update the docs to indicate our support for this?


Use correct types namespace when building namedtuple core schemas

Change Summary
Fixes https://github.com/pydantic/pydantic/issues/10336
Regression introduced in https://github.com/pydantic/pydantic/pull/10063
See diff without whitespaces
Related issue number

Checklist

[ ] The pull request title is a good summary of the changes - it will be used in the changelog
[ ] Unit tests for the changes exist
[ ] Tests pass on CI
[ ] Documentation reflects the changes where applicable
[ ] My PR is ready to review, please add a comment including the phrase "please review" to assign reviewers

Feel free to remove that benchmark for now (let's comment it out), it's too flaky to be valuable at this point.

I have a pending PR that I'll finish this afternoon to tackle this

Sounds good. Also, could you please add a test before we merge?


Does not work SKIP, TAKE and ORDER BY in queries without filtering, when DynamicFilter is used

Hello. We use dynamic filter and we faced up problem. All relevant information about used versions provided at the end.
We found some problem with using dynamic filter. When we trying to get all values from database with pagination and ordering, without any additional filtering (except dynamic filter), we get all rows from DB (pagination and ordering does not work).
However, if we add some filtering (even mindless), sorting and pagination works.
Our class looks like
public class Entity {

	public long Id { get; set; }

	public long SortOrder { get; set; }

	public bool IsDeleted { get; set; }

}

Here is example of query without filtering

var query = this
	.Repository
	.GetAll()
	.OrderBy(entity => entity.SortOrder)
	.Skip(1)
	.Take(1);

here is sql command for this query (this is query for this point, I mean we have breakpoint here and we get this command from 'query' variable)
SELECT "Var_1"."Id", "Var_1"."IsDeleted" FROM "public"."Entities" AS "Var_1" WHERE "Var_1"."IsDeleted" = @DynamicFilterParam_000001 OR @DynamicFilterParam_000002 IS NOT NULL ORDER BY "Var_1"."SortOrder" ASC  OFFSET 1 LIMIT 3
hovewer further, when the actual query is executed, the sql command looks like
SELECT "Var_1"."Id", "Var_1"."IsDeleted" FROM "public"."Entities" AS "Var_1" WHERE "Var_1"."IsDeleted" = @DynamicFilterParam_000001 "
as you can see the last part of query lost.

Here is example of query with mindless filtering

var query = this
	.Repository
	.GetAll()
	.Where(entity => !entity.IsDeleted)
	.OrderBy(entity => entity.SortOrder)
	.Skip(1)
	.Take(1);

here is sql command for this query
SELECT "Extent1"."Id", "Extent1"."IsDeleted" FROM "public"."Entities" AS "Extent1" WHERE ("Extent1"."IsDeleted" = @DynamicFilterParam_000001 OR @DynamicFilterParam_000002 IS NOT NULL) AND "Extent1"."IsDeleted" != TRUE ORDER BY "Extent1"."SortOrder" ASC  OFFSET 1 LIMIT 3
and further, when the actual query is executed, the sql command looks like
SELECT "Extent1"."Id", "Extent1"."IsDeleted" FROM "public"."Entities" AS "Extent1" WHERE ("Extent1"."IsDeleted" = @DynamicFilterParam_000001 ) AND "Extent1"."IsDeleted" != TRUE ORDER BY "Extent1"."SortOrder" ASC  OFFSET 1 LIMIT 3
As you can see in this case everytinhg is OK.
We investigated this problem a little bit, and found out, that probably the reason in your RemoveFilterDisabledConditionFromQuery function in DynamicFilterExtensions class.
...

int endIdx = command.CommandText.IndexOf(")", paramIdx);

if (endIdx == -1)
{
	//  PostgreSQL does not wrap all conditions in () (see below for example) so we may not find this.
	//  So assume end of line.
	endIdx = command.CommandText.Length - 1;
}

....

command.CommandText = command.CommandText.Remove(startIdx, endIdx - startIdx + 1);

...

Since our first example does not contain any parentheses, this function removes all query part after @DynamicFilterParam_000002, as a result LIMIT, OFFSET and ORDER BY also removed.
Additional information:

DynamicFilter version is 1.4.11
We use postgres 9.5.5
NpgSql 3.2.5
EntityFramework6.NpgSql 3.1.1

We have single filter, presented below
protected override void OnModelCreating(DbModelBuilder modelBuilder)
{
	modelBuilder.Filter(FilterName.SoftDelete, (ISoftDelete sd) => sd.IsDeleted, false);
}

Best Regards,
dopusteam

Hello @JonathanMagnan,
I will try to provide example project in few days
Best Regards,
dopusteam

Hello @JonathanMagnan,
I found that the problem is not reproduced starting from 2.0 version of dynamicFilters, so probably we will use newer version.
However, since it can be useful, I attach sample project with 1.4.11 version, where the problem is reproduced.
Thank you!
Best Regards,
Jonathan

Oh, I should have looked at the version you specified first ;)
Thank for the project

Hello @Jon,
This issue will be closed since it has been resolved in latest version
Feel free to reopen it if you feel otherwise.
Best Regards,
Jonathan


Corrected a typo in {% page_header_buttons %}

Fixes #10730

Added to 5.1 d75d6e6f5dc5496e69b8b857b24a3c0476b4d771
This appears to have been broken before 5.1 but I think it would make sense to fix this in the same release as we fixed a similar issue with the page_listing_buttons template tag. See wagtail/admin/templatetags/wagtailadmin_tags.py (diff)


Black Background after removing thumbnail

Hello. In my adapter I set up an ImageView in front of the player to be used as a thumbnail and remove it when the video starts playing. However after hiding the thumbnail, the video takes a few seconds to load and turns black and only then starts playing.
How can I make a smoother transition?
Thanks in advance and thanks for this library.

If your ViewHolder implements ToroPlayer.EventListener  you have a method to implement called onPlaying(), in this method you can hide your thumbnail.
To add the listener you need to call  helper.addPlayerEventListener(this) . on initialize Toro
Greetings!


added split and train_test... function code

Thanks!
Tested successfully!
Re-coded and will send another PR through with code changes so far.


bug v3: Error viewing astro file

Did you check docs and existing issues?

[X] I have read all the trouble.nvim docs
[X] I have searched the existing issues of trouble.nvim
[X] I have searched the existing issues of plugins related to this issue

Neovim version (nvim -v)
NVIM v0.9.5
Operating system/version
Arch Linux x86_64
Describe the bug
When I open an Astro file using LazyVim and Trouble-v3 it starts with a lot of errors like this
..hare/nvim/lazy/trouble.nvim/lua/trouble/view/section.lua:98: Error executing lua: ...e/nvim/lazy/lualine.nvim/lua/lualine/utils/nvim_opts.lua:77: E539: Illegal character <]>
stack traceback:
[C]: in function 'nvim_win_set_option'
...e/nvim/lazy/lualine.nvim/lua/lualine/utils/nvim_opts.lua:77: in function 'setter_fn'
...e/nvim/lazy/lualine.nvim/lua/lualine/utils/nvim_opts.lua:50: in function 'set_opt'
...e/nvim/lazy/lualine.nvim/lua/lualine/utils/nvim_opts.lua:74: in function 'set'
...irio/.local/share/nvim/lazy/lualine.nvim/lua/lualine.lua:432: in function 'refresh'
.../.local/share/nvim/lazy/trouble.nvim/lua/trouble/api.lua:150: in function 'on_update'
...hare/nvim/lazy/trouble.nvim/lua/trouble/view/section.lua:108: in function 'update'
...hare/nvim/lazy/trouble.nvim/lua/trouble/view/section.lua:72: in function 'finder'
...hare/nvim/lazy/trouble.nvim/lua/trouble/view/section.lua:66: in function 'fn'
...hare/nvim/lazy/trouble.nvim/lua/trouble/view/section.lua:99: in function <...hare/nvim/lazy/trouble.nvim/lua/trouble/view/section.lua:98>
[C]: in function 'nvim_win_call'
...hare/nvim/lazy/trouble.nvim/lua/trouble/view/section.lua:98: in function 'main_call'
...hare/nvim/lazy/trouble.nvim/lua/trouble/view/section.lua:64: in function 'fn'
....local/share/nvim/lazy/trouble.nvim/lua/trouble/util.lua:119: in function <....local/share/nvim/lazy/trouble.nvim/lua/trouble/util.lua:115>
[C]: in function 'xpcall'
....local/share/nvim/lazy/trouble.nvim/lua/trouble/util.lua:115: in function <....local/share/nvim/lazy/trouble.nvim/lua/trouble/util.lua:114>
If I use old version of Trouble there are no errors
Steps To Reproduce

Open LazyVim
Open an Astro file
Wait some seconds and they appear

Expected Behavior
It will open the file without errors, now it is unusable. I tried with the repro below, but it doesn't reproduce anything, I don't know how to set it to reproduce (I'm a little newbie in this XD )
Repro
-- DO NOT change the paths and don't remove the colorscheme
local root = vim.fn.fnamemodify("./.repro", ":p")

-- set stdpaths to use .repro
for _, name in ipairs({ "config", "data", "state", "cache" }) do
  vim.env[("XDG_%s_HOME"):format(name:upper())] = root .. "/" .. name
end

-- bootstrap lazy
local lazypath = root .. "/plugins/lazy.nvim"
if not vim.loop.fs_stat(lazypath) then
  vim.fn.system({ "git", "clone", "--filter=blob:none", "https://github.com/folke/lazy.nvim.git", lazypath, })
end
vim.opt.runtimepath:prepend(lazypath)

-- install plugins
local plugins = {
  "folke/tokyonight.nvim",
  "folke/trouble.nvim",
  -- add any other plugins here
}
require("lazy").setup(plugins, {
  root = root .. "/plugins",
})

vim.cmd.colorscheme("tokyonight")
-- add anything else here

I tried also this repro.lua, but it doesn't appear any error
-- DO NOT change the paths and don't remove the colorscheme
local root = vim.fn.fnamemodify("./.repro", ":p")

-- set stdpaths to use .repro
for _, name in ipairs({ "config", "data", "state", "cache" }) do
	vim.env[("XDG_%s_HOME"):format(name:upper())] = root .. "/" .. name
end

-- bootstrap lazy
local lazypath = root .. "/plugins/lazy.nvim"
if not vim.loop.fs_stat(lazypath) then
	vim.fn.system({ "git", "clone", "--filter=blob:none", "https://github.com/folke/lazy.nvim.git", lazypath })
end
vim.opt.runtimepath:prepend(lazypath)

-- install plugins
local plugins = {
	"folke/tokyonight.nvim",
	"folke/trouble.nvim",
	-- add any other plugins here
	{
		"nvim-lualine/lualine.nvim",
		dependencies = { "nvim-tree/nvim-web-devicons" },
	},
}
require("lazy").setup(plugins, {
	root = root .. "/plugins",
})

require("lualine").setup()

vim.cmd.colorscheme("tokyonight")
-- add anything else here

These are my extras installed

These are my dotfiles, if can be of any help [https://github.com/ziriuz84/dotfiles/tree/main/dot_config/nvim]

I think this is probably fixed by https://github.com/folke/trouble.nvim/commit/f6a3fbfb667c3cccea8df9b7471fb9b31b1235fc


Bugfix: remove referee type from support interface if not set

The type of reference wasn't asked early on, so some references don't have this field set. This was throwing an error in the support interface.
This is fixed by removing the whole row if there is no reference type to display.
Fixes bug: https://sentry.io/organizations/dfe-teacher-services/issues/3717069711/?project=1765973

I'm happy if @carlosmartinez is happy!

@carlosmartinez one fail on this still: "Build and Deploy / Collate test coverage reports (pull_request)" - can that be ignored for now, or is there a fix?

I think is relate to https://github.com/dawidd6/action-download-artifact/issues/207
I will downgrade on our build to fix the issue.


customize log out alert

It would be nice if we could change the amount of time to pass before getting the log out notification
at the moment you are notified when you are 15 seconds from being logged, would be cool if we could change that time :)

Maybe we could just honor the idle notifier delay and substract it, that would make everyone happy.


Saleor 4437 refactor tests

I want to merge this change because...

PR intended to be tested with API branch:
Screenshots

Pull Request Checklist

[ ] This code contains UI changes
[ ] All visible strings are translated with proper context including data-formatting
[ ] Attributes [data-test-id] are added for new elements
[ ] Changes are mentioned in the changelog
[ ] The changes are tested in different browsers and in light/dark mode

Test environment config

API_URI=https://master.staging.saleor.cloud/graphql/

Task linked: SALEOR-4437 Refactor all tests

@karola312 you will need to squash the commit removing the screenshots with the one that adds them, otherwise all those files will still be tracked by git history and will be downloaded by anyone cloning the repository.


Chore: New colors for Tokens page

Motivation
We are transitioning to a new set of colors defined in Figma. Not all have been exposed yet. We do it gradually.
In this PR, I expose only the colors needed for the tokens page.
Changes

Add colors to light and dark themes.

Screenshots
No new components, and the new colors are not yet used in any of the current components.

@mstrasinskis please review


Add field of override for component extraArgs

What type of PR is this?

/kind cleanup
What this PR does / why we need it:
I'm using componentsPatches to rewrite the args of kube-apiserver for etcd-servers and etcd-prefix, it's working for me but i found the arg etcd-servers and etcd-prefix is repeated. This PR is working for cleanup the repeate.
config.yaml
kind: KwokctlConfiguration
apiVersion: config.kwok.x-k8s.io/v1alpha1
componentsPatches:
- name: kube-apiserver
  extraArgs:
  - key: etcd-servers
    value: http://<IP_ADDRESS>:3379
  - key: etcd-prefix
    value: /test

cmd of container kube-apiserver
                "--etcd-prefix=/registry",
                "--allow-privileged=true",
                "--etcd-servers=http://kwok-kwok-etcd:2379",
...
                "--etcd-servers=http://<IP_ADDRESS>:3379",
                "--etcd-prefix=/test"

/assign @wzshiming
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?

NONE

Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.:

/release-note none

I think we can add an override field to ExtraArgs to override hardcode args.

@wzshiming
Do you mean add a arg like override=true into ExtraArgs? and for user, they need write something like :
componentsPatches:
- name: kube-apiserver
  extraArgs:
  - key: etcd-servers
    value: http://<IP_ADDRESS>:3379
  - key: etcd-prefix
    value: /test
  - key: override
    value:  true

and do not override args by default.

I think it looks like this.
When the override is true, the same key arguments that already exist are removed.
componentsPatches:
- name: kube-apiserver
  extraArgs:
  - key: etcd-servers
    value: http://<IP_ADDRESS>:3379
    override: true
  - key: etcd-prefix
    value: /test
    override: true

or another way to remove existing arguments
componentsPatches:
- name: kube-apiserver
  extraArgs:
  - key: etcd-servers
    remove: true
  - key: etcd-prefix
    remove: true
  - key: etcd-servers
    value: http://<IP_ADDRESS>:3379
  - key: etcd-prefix
    value: /test

@liangyuanpeng
I think the override may need to be used in a different way without sort args,
I worry that args will lead to unexpected patch args in the future.
WDYT?

Thanks,  Let me go back to this PR this week

@wzshiming  ready to go!

/remove-kind cleanup
/kind feature

/approve
/lgtm

/override pull-kwok-e2e-test-main


Update DRM database

Hi, the current database is 6 years out of date, where can I get a more up to date file?

+1.
@mbyott Would you like to work together to update mutation_db.tsv ?

I found a bunch better resource at hivfacts github page:
https://github.com/hivdb/hivfacts/blob/main/data/
See the different sdrms_*.csv files (they're also in json and yml format which is nice).
Unfortunately they lack the comments which is very helpful. But it's in a very easy to modify format.


 This tooling is useful because it's sometimes difficult to reproduce issues that occur in production, but by enabling this in a production cluster you can get a stack trace of what the program is doing, which will aid in diagnosing possible issues.
Description

Todos

[ ] Tests
[ ] Documentation
[ ] Release note

Release Note

None required

I think instead of using an HTTP server for this we should use a signal handler. Then I think we should be able to enable dumping stack traces much more freely.

I think instead of using an HTTP server for this we should use a signal handler. Then I think we should be able to enable dumping stack traces much more freely.

Enabling it would be the same either way, you need to set an env variable regardless (I don't think we want this enabled all the time).
What signal are you hoping to use?  I'm not opposed to doing that, but I'm not sure that makes dumping this more freely. With the server you don't need to kill the pod, so you can let the issue continue to run and diagnose it. You may need to dump a stack trace multiple times because when you dump it it may not be running the problem. We may also want to run other tools after to get more info, like mem / cpu profiling.
Like I said, I'm not opposed to using a signal but I disagree that it allows us to dump stack traces more freely.

My thought is that if it is a signal handler then we can enable it all the time. If it is an HTTP endpoint then I think the default would need to be disabled and we would need a special flag to enable it.
I would like to have this capability all the time, though granted if it is a signal I think the only way to fire it would to be able to actually get on a node and use docker or podman utilities but that is also the reason I'd be comfortable having it enabled all the time.
As for which Signals, probably SIGUSR1 or SIGUSR2. IIRC Felix already sets up one of those to dump information.

OK that makes sense and sounds good. I had thought that the only signals you could send were kill signals, I don't really know a lot about this but I'll read up on it.

Felix currently uses both SIGUSR1 and SIGUSR2.  One to dump a CPU profile and one to dump a memory profile.
We could just overload those so that we dump a CPU profile and a stack trace, for example.  If we do add a debug server, how about using a unix domain socket instead of a TCP socket; that'd avoid security issues with opening extra ports, which some customers are very worried about.

The Calico team is reorganizing our code and repository structure in order to make development easier, and as such all of the code in this repository is now located at https://github.com/projectcalico/calico. We appreciate your PR and your patience as we work to improve our developer experience, and welcome you to re-open your changes against the new location. Please reach out if you have any questions!


description error show with block

description error show with block :
 this is description

Hello. Please give more information. Also it'll be good to have your allure-generated folder attached

(I'm surely should create issue template)

pre  code block not translate into html show。 it show the pre code string in description

Unfortunately this is a known bug, that is also mentioned here https://github.com/MihanEntalpo/allure-single-html-file/issues/18. Right now, all I've tried to do with this problem had no effect, or just broke the interface.
Some day I'll fix it, or maybe someone else will propose pull request with such a fix

According to this merged pull request allure-framework/allure2#2072 Allure itself now has command line argument --single-file that make it produce report as a single html file. As for now it's not described in official allure docs, but anyway it should make this package (allure-combine) useless. I'm encorage you to use Allure --single-file functionality, as it should work much better that my combiner!


REST API authentication issue

API authentication fails because "username" and "password" variables are of type list which are compared to "suppliedUsername" and "suppliedPassword" which are of type unicode. Converting the list to string fixes the authentication bug.

Any idea as to how soon this will be merged?

I would also like this to be merged, please.

@TheNaterz If you have tested this fix and it works for you, you can use it without it being merged with the upstream master.


Add a block based theme option

Description of the Change

TBD – Very much work in progress

Closes #
How to test the Change

Changelog Entry

Added - New feature
Changed - Existing functionality
Deprecated - Soon-to-be removed feature
Removed - Feature
Fixed - Bug fix
Security - Vulnerability
Developer - Non-functional update

Credits

Props @username, @username2, ...

CC: @10up/editorial-engineering Would love to get your all's input on this :)
Also CC: @darylldoyle we can use this new scaffold to update how we are doing things. So if we want to introduce larger coding style / structural changes this is a time where we can include stuff like that :)

@fabiankaegy
I'll also share with the FEE Best Practices WG - as some of the suggested and similar changes here have been discussed in that WG


Manual update Needed For Artchive token image.

Hey @keone just need to update the token icon for this token #14363 updated .PNG. Please merge manually.

merged

Thank you.


panic: close of closed channel

In your example, when f2 (same will happen when f3, f4 return error) returns an error, the program will crash. for then channel f1 was closed already.

I Noticed you tried to fix the bug, but I don't think it will work out, under multi goroutine conditions some channel still might be closed more than once unless you lock the close procedure. I don't even think lock close procedure is a good idea, the purpose of closing all the channels is that you just want all waiting goroutines to quit, why not send some special return value to the results channel (or even declare a bunch of channels for passing deps excute success/failure) and let waiting goroutines quit respectively then close all channels after that (eg. in the do() func before after wait for all)?

Thanks for your feedback.
I used atomic.Value to store information about the current state of channel (if it's closed). Atomic value guarantee that if state has changed, other goroutines will load them with this state. So, in my opinion there is no an option panic here.
On the question of use quit channel, of course that will works but I think it's a more complicated way.
If I'm wrong, I will be pleased to see some example with failure.


Trainer.test() does not allow to save logs

🐛 Bug
When we use trainer.test() on a fresh trainer, the  call  to self.log_metrics(log_metrics, {}, step) fails to produce logs.
To Reproduce
Steps to reproduce the behavior:
Code sample
model = ...
test_dl = ...
logger = ...
trainer = pl.Trainer(
   logger=logger
)
trainer.test(model, test_dataloaders=test_dl)

Expected behavior
Metrics are logged.

Because of the way aggregate_metrics is implemented, logs are only produced when we are leaving the current step. Because there is only one step in testing, the logs are stored in self._metrics_to_agg = [metrics] but never logged.

@ justusschock any ideas?

Btw, I was able to log my logs by adding self.logger.log_metrics(logs) at the end of test_epoch_end but I believe this should be the default.

@Varal7 would you be able to send a PR with the fix?

results = trainer.test()
results is a dict with the results

if you encounter no logging on end of epoch it may be because you are using ddp, requiring explicit usage of test_step_end see the documentation here:
def test_step(self, batch, batch_idx):
    X, y = batch
    y_hat = self(X)
    return y_hat, y

def test_step_end(self, output_results):
    y_hat, y  = output_results
    loss = criterion(y_hat, y)
    self.log(loss, epoch_end=True)


[hotfix][docs] fix some typos

What is the purpose of the change

I have updated some typos in the docs/content/docs/custom-resource/autoscaler.md file.

Verifying this change

I checked the changes.

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changes to the CustomResourceDescriptors: (no)
Core observer or reconciler logic that is regularly executed: (no)

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

Hi, @mbalassi please take a look in your free time, thanks!

Thank you for contributing!

@mxm @gyfora Thank you for your patience, it gives me a lot of encouragement!


Optimize the log event hander

Optimize the log event hander,add the cancel property to cancel write to the log.

updated v<IP_ADDRESS>


Flashing error while updating to last firmware

Hi,
I have a Raspberry Pi4 and I'm running Hassio on docker from Raspbian.
I've also a Raspbee on /dev/ttyAMA0
I've also change config.txt in adding those:
#enable_uart=1
dtoverlay=miniuart-bt
(activating or not #enable_uart=1 seems not effective)
I've installed the deCONZ add-on on HomeAssistant, but unfortunately the firmware seems not connected and Zigbee channel is stuck to 0 . Also it is impossible to pair any kind of sensor.
I've tried your procedure to update deCONZ to last firmware (after switching off the deCONZ addon) from Home Assistant SSH terminal (disabling protection mode), but unfortunately I'm having this error:
Flashing...
GCFFlasher V3_15 (c) dresden elektronic ingenieurtechnik gmbh
Reboot device /dev/ttyAMA0 (RaspBee)
action: reset device RaspBee
wiringP1 2.52 initialized
Flashing Error! Please re-run script...
Can you please help me? Is there something wrong in my procedure?
Best regards.

Hi,
every time I had an issue like this I could solve this by flashing my stick on a windows system. But even there I had to try it more than once in some cases.
By now I would say the options for retries and timeout would fix this if used when flashing the "docker-way".
@nhulsch proposed a fix for this in #294 - but somehow it didn't make to the scripts though it has been merged ...
In my fork I have done another approach - there I am reading the options you could specify while starting docker with an entrypoint. See #318. Additionally I could specify default in this implementation ...

I just bought a conbee II stick and tried to flash a new firmware following this procedure: https://github.com/marthoc/docker-deconz#updating-conbeeraspbee-firmware
got the same error
david@xps:~/dev/conbee$ docker run -it --rm --entrypoint "/firmware-update.sh" --privileged --cap-add=ALL -v /dev:/dev -v /lib/modules:/lib/modules -v /sys:/sys marthoc/deconz
-------------------------------------------------------------------

             marthoc/deconz Firmware Flashing Script

                       Version: 0.6

-------------------------------------------------------------------

Listing attached devices...

GCFFlasher V3_17 (c) dresden elektronik ingenieurtechnik gmbh
Path             | Vendor | Product | Serial     | Type
-----------------+--------+---------+------------+-------
/dev/ttyACM0     | 0x1CF1 | 0x0030  |            | ConBee II

Enter the full device path, or press Enter now to exit.

Device Path : /dev/ttyACM0

-------------------------------------------------------------------

Firmware available for flashing:
deCONZ_ConBeeII_0x26660700.bin.GCF
deCONZ_ConBeeII_0x26680700.bin.GCF
deCONZ_RaspBeeII_0x26690700.bin.GCF
deCONZ_Rpi_0x26350500.bin.GCF
deCONZ_Rpi_0x26390500.bin.GCF

Enter the firmware file name from above, including extension.
Alternatively, you may enter the name of a firmware file to download
from http://deconz.dresden-elektronik.de/deconz-firmware/
or press Enter now to exit.

File Name : deCONZ_ConBeeII_0x26660700.bin.GCF

-------------------------------------------------------------------

Device: /dev/ttyACM0

Firmware File: deCONZ_ConBeeII_0x26660700.bin.GCF

Are the above device and firmware values correct?
Enter Y to proceed, any other entry to exit: Y

Flashing...

GCFFlasher V3_17 (c) dresden elektronik ingenieurtechnik gmbh
Reboot device /dev/serial/by-id/usb-dresden_elektronik_ingenieurtechnik_GmbH_ConBee_II_DE2219384-if00 (ConBee II)
deCONZ firmware version 26580700
R21B18 Bootloader
Vers: 2.07
build: Jun 17 2019
flashing 164106 bytes: |=error: timeout flashing firmware after 3002 ms

Flashing Error! Please re-run this script...

I was able to recover AND upgrade the firmware -> https://github.com/dresden-elektronik/deconz-rest-plugin/issues/4342

As there hasn't been any new replies and as there are newer versions, i'm closing this issue. If this needs to be re-opened, please ping me :)