fix(twap/upgrades): geometric twap accumulator initialization; testnet bug reproduction

Closes: #XXX
What is the purpose of the change
In https://github.com/osmosis-labs/osmosis/issues/3542, it was concluded that we do not need to perform the migrations to initialize the geometric twap accumulator.
However, the testnet halted due to the geometric twap accumulator ending up uninitialized.
The reason why our e2e test did not catch this problem was because we neve swapped against a pool that was created pre-upgrade after the upgrade itself.
This PR reproduced the testnet halt bug and fixed it by introducing twap record migrations that were originally spearheaded  in: https://github.com/osmosis-labs/osmosis/pull/3729
Brief Changelog

reproduce testnet halt
introduce twap migrations from: https://github.com/osmosis-labs/osmosis/pull/3729
test more thoroughly and document

Testing and Verifying
This change added e2e and migrations tests.
Documentation and Release Note

Does this pull request introduce a new feature or user-facing behavior changes? no
Is a relevant changelog entry added to the Unreleased section in CHANGELOG.md? no
How is the feature or change documented? not applicable

Closing in favor of: https://github.com/osmosis-labs/osmosis/pull/3972


fix: Use IHostedZone to work with imported zones

Also allow specifying record name, may differ from zone name

I needed to regenerate API.md


Update README.md

Add new website for win-acme

Coverage remained the same at 90.559% when pulling 3cfc69d0ec0667645cf9ab8687c214eb63f55296 on WouterTinus:patch-1 into 5de21916a32769bdc36af0485ac128832b2cbd90 on joohoi:master.

Lookin' good!


amend: updated deps

:tada: This PR is included in version 2.10.8 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[python-package] take shallow copy of dataframe in predict (fixes #6195)

Takes a shallow copy of the dataframe in the _data_from_pandas function to avoid modifying it when converting categorical features to their codes. This used to always be done (implicitly) for predict until #6066, since the function is called with feature_name='auto' and the copy argument of pandas.DataFrame.rename is passed as the deep argument to pandas.DataFrame.copy, however that behavior wasn't obvious.
Fixes #6195.
Also updates the test that verifies the original dataframe isn't modified to now consider the case of unseen categories, which fails in the current master and should prevent #6195 from happening again.

Haha no worries. I missed it too when reviewing, the predict path wasn't clear (feature_name='auto' leading to the shallow copy).


feat(badge): updated horizontal padding

Update Badge Horizontal Padding

These types will increment the version number on release:

[ ] Fix
[x] Feature
[ ] Performance Improvement
[ ] Refactor

These types will not increment the version number, but will still deploy to documentation site on release:

[ ] Documentation
[ ] Build system
[ ] CI
[ ] Style (code style changes, not css changes)
[ ] Test
[ ] Other (chore)

:pencil: Checklist

For all PRs:

[x] I have ensured no private Dialpad links or info are in the code or pull request description (Dialtone is a public repo!).
[x] I have reviewed my changes.
[x] I have added all relevant documentation.
[x] I have considered the performance impact of my change.

For all CSS changes:

[x] I have used design tokens whenever possible.
[x] I have considered how this change will behave on different screen sizes.
[x] I have visually validated my change in light and dark mode.
[x] I have used gap or flexbox properties for layout instead of margin whenever possible.

"Lint pull request commit message when it has a single commit"

I still understand this.

"Lint pull request commit message when it has a single commit"

I still don't understand this.

If your PR has a single commit, the commit message should adhere to commit convention
If it has multiple, the PR title would be used instead.

"...in this case GitHub will use this commit for the commit message when your PR is merged..."

🙃 Still flying right over my head. So it won't use my title, feat(badge): updated horizontal padding instead using the commit message of that single commit?

So it won't use my title, feat(badge): updated horizontal padding instead using the commit message of that single commit?

Yep it's pretty silly

WOW! look at this new option I just found.

noice.


Updating styles for nav sidebar

New expansion indicators
New spacing
New colour for page links
New style for nav to deprecated content

larger screens

smaller screens

@plaindocs Yeah, sorry, that was unclear. Just the change in link style for deprecated.
These H3s are as per Figma, but I agree that indentation feels better. How about this?


Add cli option to set titleBarStyle

Hello,
This PR is to include a CLI option to specify an alternative title bar style for macOS. Available values include 'hidden' and 'hiddenInset' as specified here.
Would you please consider merging?
This PR partly addresses #35.

When using this option. You'll probably need to inject (via --inject) to add some custom CSS to add a draggable area. As an example, I nativefied YNAB. This web app has a nav bar on the left so I made it draggable. I also added some extra padding on the top left so the traffic light does not obstruct any element on the page.

@ronjouch I'm using Visual Studio Code. Any prettier extension/plugin that you could recommend?

@khaister I don't know, the big one Prettier - Code formatter by Esben Petersen, with 2.1M installs, look okay and maintained.

@ronjouch Thanks! I'll take a look.

@ronjouch Added the items you requested.

@khaister merged! 👍👍👍 documentation, thank you 🙂.


Xml file not found

while running sumo_config in sumo it is showing the following error.
Error: The route file 'E:\intelligent_traffic_lights-master\intelligent_traffic_lights-master\src\cfg\episode_routes.rou.xml' is not accessible
can you please send this file episode_routes.rou.xml to<EMAIL_ADDRESS>or upload it in the github
Thank you

Hi @Mahesh020699!
I believe you may use generator.py to generate a file with vehicle routes. Tell me if it doesn't work for you.

closing for inactivity

while running sumo_config in sumo it is showing the following error. Error: The route file 'E:\intelligent_traffic_lights-master\intelligent_traffic_lights-master\src\cfg\episode_routes.rou.xml' is not accessible
can you please send this file episode_routes.rou.xml to<EMAIL_ADDRESS>or upload it in the github
Thank you

Hello. Did you run this model?


Feature: New PreventAccessFromTenants Middleware

As discussed in #help you were looking for an additional solution to the domain functionality in routes, with this middleware which is the inverse of PreventAccessFromCentralDomains should suffice. Let me know your thoughts.

(I don't have time to deal with this right now, but for future reference: The reason why this is a draft is that using a different way than domain() to register central routes leads to the issue where you can't use / for both central and tenant routes)


Add GCSFileSystem.get_dir

Recursively download remote directory to local

Thanks @martindurant, I tried to take your suggestions into account in my last commit.

Thanks @martindurant, it should be good now, please let me know if you see anything else.

Thank you, this is going in.
If you have time, you might be interested in https://filesystem-spec.readthedocs.io/en/latest/ , an attempt to make an abstraction of the various file-systems that are around (including gcsfs). There is an issue there around using globs in various operations, and this idea of recursion could fall there too. If you have the impetus, you might like to sketch out or even make a PR for how to generalise what's where to fsspec, to globs and directory trees, and to all the methods that might use them. btw: walk in the sense you have used it here is called find in fsspec, where walk produces iterators as os.walk does.

Thanks for the merge and for the guidance @martindurant! I will look at filesystem_spec. Also, a put(recursive=True) could be useful.

Indeed - and I imagine you will want to factor out the functionality which gets the common root of a set of files and applies a different root on the target file-system.


Error handling

I need error handling.
Ex: Object({"errors": Array([Object({"code": Number(89), "message": String("Invalid or expired token.")})])})

Now there is no error handling for decoding errors.
It seems to be good to separate cases based on whether a response can be decoded to the expected response type by serde_json.


Hold to record & release to send _ Voice not send/save until lock button press

I want to send voice messages when the user holds the long press and release like what's app. but it will not work until it holds the long press and lock.  how can I resolve this?

okay I will update my package soon

Okay I will update my package soon

I want to send voice messages when the user holds the long press and release like what's app. but it will not work until it holds the long press and lock. how can I resolve this?

Please use update version and check new parameter releaseToSend
RecordButton(
color: Colors.black,
allTextColor: Colors.white,
controller: controller!,
arrowColor: Colors.white,
releaseToSend: false,
onRecordEnd: (String value) {
setState(() {
filepath = value;
});
},
onRecordStart: () {
=
},
onCancelRecord: () {
              },
            ),

Thank you . I like your package

Thank you . I like your package

Then can close this issue?


Messages sent with /me show as normal text unless C2 is restarted

^ Title
Bug reproduction: Send a message with /me
Notice how it doesn't look colored
Restart C2, and message now is colored

Just realized fourtf just opened an issue about this, closing


Dart/ Flutter Fat Arrow Syntax Causes Error In Expand Selection

VSCode Version: 1.24.1
OS Version: MacOS 10.13.5

Steps to Reproduce:

Open a dart application containing a tree of nested objects, inside which the 'fat arrow' syntax is used (as shown below)

Use the 'Selection->Expand Selection (ctrl+shift+cmd+rightArrow) to keep expanding the selection

Upon reaching the arrow, the selection becomes incorrect (as shown below). The Expand Selection should include the contents past the arrow operator.

Does this issue occur when all extensions are disabled?: Yes (And with only the flutter/ dart extension enabled)

/duplicate of #4795


Crash on Install

Hyprland Version
idk
Bug or Regression?
Bug
Description
If I try to run the last command "sudo make install", I get the error
cmake --build ./build --config Release --target all -j16
[0/2] Re-checking globbed directories...
ninja: error: '../subprojects/wlroots/build/libwlroots.so.12032', needed by 'Hyprland', missing and no known rule to make it
make: *** [Makefile:156: legacyrenderer] Error 1
How to reproduce
Probably a me issue
Crash reports, logs, images, videos
Would appreciate some help!

1- that's not a crash
2- you have to git clone --recursive https://github.com/hyprwm/Hyprland
https://wiki.hyprland.org/Getting-Started/Installation/

Then I get this error:
CMake Error at CMakeLists.txt:115 (add_executable):
Cannot find source file:
protocols/cursor-shape-v1-protocol.c

Tried extensions .c .C .c++ .cc .cpp .cxx .cu .mpp .m .M .mm .ixx .cppm .h
.hh .h++ .hm .hpp .hxx .in .txx .f .F .for .f77 .f90 .f95 .f03 .hip .ispc

what distro?

I will try that

is this resolved for you ??


[BUG] Concurrent Segment Search doesn't work if a Plugin Overrides the QueryPhaseSearcher

Describe the bug
As part of Enabling the Normalization and Score Combination Feature(RFC: https://github.com/opensearch-project/neural-search/issues/126) in OpenSearch via Neural Search plugin, we used the Extension point of QueryPhaseSearcher(https://github.com/opensearch-project/OpenSearch/blob/main/server/src/main/java/org/opensearch/search/SearchModule.java#L1248-L1259), but the problem with that extension point is it enables the Searcher at Cluster Level and not at the Index level. This make the Concurrent Search un-usable because another implementation of QueryPhaseSearcher is there.
So cutting this issue to start a conversation that, the fix needs to be done before Concurrent Segment Search can become GA.
To Reproduce
Steps to reproduce the behavior:

Use the full distribution of OpenSearch 2.10
Enable the Concurrent Segment Search Feature.
Try doing the search, the concurrent Segment search does't work.

Expected behavior
Ideally there should be a way in which Plugins can define when to use their QueryPhaseSearcher at Index level. Also, core needs to make sure that all the QueryPhaseSearcher which are registered are initialized and during runtime based on the index/cluster setting right Searcher is picked up.
Plugins
k-NN, MLCommons, Neural Search.
Screenshots
NA
Host/Environment (please complete the following information):
NA
Additional context
The same issue was raised as a feature request too: https://github.com/opensearch-project/OpenSearch/issues/7020

@navneet1v I don't think this is a bug instead it is how the QueryPhaseSearcher is exposed to the plugin and uses either the core implementation of QueryPhaseSearcher or the one provided by plugin. It is upto a particular installation with set of plugins to see which one to use. If the use case here is to have multiple QueryPhaseSearcher support (i.e. core + ones provided by plugins), then that functionality needs to be built based on such use case. For example: With concurrent search,  since it is part of core we have created this QueryPhaseSearcherWrapper which will hold on to both DefaultQueryPhaseSearcher and ConcurrentQueryPhaseSearcher and use one or the other based on which path to take for a request. A plugin can extend this or compose it based on the use case. Like plugin can provide a QueryPhaseSearcher, which can keep map of core provided searchers and its own and then based on request state decide which one to use. Or even build this in the core to make that decision. Not sure why it is tied to concurrent search GA. Based on https://github.com/opensearch-project/OpenSearch/issues/7020 my understanding was you or @martin-gaievski were doing some PoC for the same ?

@sohami

A plugin can extend this or compose it based on the use case
The problem here is the QueryPhaseSearcherWrapper is marked as opesearch.internal so it is subject to change. If it is marked as opesearch.api then it would make more sense to extend QueryPhaseSearcherWrapper.

Based on the code shared here, seems like it needs to extend QueryPhaseSearcherWrapper now instead of DefaultQueryPhaseSearcher ?

Yes we can change the neural search class to extend the QueryPhaseSearcherWrapper, but still it doesn't solve the problem. What will happen if some other backend plugin comes up and want to implement another query phase searcher. Thing is it cannot happen. Hence I think we should start thinking about this and not come up with these intermediate solutions. Reason why I am tieing this up with Concurrent Search is because we are adding it as a new Feature in OpenSearch Core and it changes the QueryPhaseSearcher.
Also, my main point here is the interface provided for extending QueryPhaseSearcher is not an actual extension.

It is upto a particular installation with set of plugins to see which one to use.
Yes that is correct. But from OpenSearch project standpoint we provide full distribution and min distribution. So we need to make sure that in both those installation we are doing it correctly when more than 1 plugin is trying to override the QueryPhaseSearcher.

This is similar to the Codec extension point we had earlier in the OpenSearch. We extended that interface to make sure that more than one codec can be be present in OpenSearch. I was thinking we should come up with a similar solution here too.

Yes we can change the neural search class to extend the QueryPhaseSearcherWrapper, but still it doesn't solve the problem. What will happen if some other backend plugin comes up and want to implement another query phase searcher. Thing is it cannot happen. Hence I think we should start thinking about this and not come up with these intermediate solutions

@navneet1v I didn't say in my last comment that extending the OpenSearch internal implementation is the best way to go. The way it was done currently is as I explained above where it expects depending on the installed plugin to either use plugin provided implementation or use the core ones. Given there are use cases to have multiple composite searchers we should definitely extend it and you can create PR with the change in core which solves the plugin use case in best way.

Reason why I am tieing this up with Concurrent Search is because we are adding it as a new Feature in OpenSearch Core and it changes the QueryPhaseSearcher

Concurrent search is changing the QueryPhaseSearcher used internally in the core. It doesn't affect how the plugins are implementing the QueryPhaseSearcher or the current extension points.

Also, my main point here is the interface provided for extending QueryPhaseSearcher is not an actual extension.

Agreed and I would think when QueryPhaseSearcher interface was added in plugin this use case was not there (@reta
can correct me here) and hence it was provided in its current form. But again you can expand on the current implementation to provide a better one to help with the new use case and probably a generic one to help other plugins as well (if we know about those use cases).

Agreed and I would think when QueryPhaseSearcher interface was added in plugin this use case was not there (@reta
can correct me here) and hence it was provided in its current form.

@sohami correct, we'd been looking to the implement one specific case, the concurrent search, in a way that does allow injection of different implementations by plugins.
@navneet1v what @sohami is saying is certainly one of the options to implement the desired flow: provide your own QueryPhaseSearcher but delegate to the ConcurrentQueryPhaseSearcher / DefaultQueryPhaseSearcher when needed (it is possible now without any changes in core). Also, the QueryPhaseSearcher is not about just search anymore, it also includes AggregationProcessor.
However, what we probably need is to have the contextual QueryPhaseSearcher that could complement (or may be even replace) the way QueryPhaseSearcher is provided currently, something roughly like that:
interface ContextualQueryPhaseSearcherProvider {
    Optional<QueryPhaseSearcher> forContext(SearchContext context);
}

That would allow plugins to register many  providers but only one us going to be picked for the context. Obviously, the issue is that since QueryPhaseSearchers are not composable, plugins could provide competing implementations that somehow have to be resolved deterministically (same one always wins, others loose).


deliver task fixes

prevents deliver from relying on publishTo or packagedArtifacts values
add makeIvyXml as a more sensible name for the deliver task

The deliver and deliverLocal tasks (which seem to be identical in function, incidentally) may possibly be deprecated at a later time, in favor of makeIvyXml.
Closes #3419

To validate this pull request against other sbt modules, please visit this link.

Could you rebase your branch on top of 1.0.x branch, plz?

To validate this pull request against other sbt modules, please visit this link.

Merged!


generate report button not required for "APIM Monthly API Usage Report" widget

"APIM Monthly API Usage Report" widget in admin dashboard used to create reports. So having a generate report button doesn't add any value.

I don't see a button as such in this widget.

I think here the pdf export button in the header need to be removed. Considering the usage of this widget, we can remove the header itself as the maximize option also not necessary for this one.


not wotking in 1.9.2

thanks for the code but its not working. I added it in my custom theme folder but still... i cannot make it work.. please let me know if any suggestions.
P:S i see no changes on frontend and no new module in admin side.
Thanks

Is it working with the default or rwd theme?
On 20 Sep 2017 12:37, "saraasghar"<EMAIL_ADDRESS>wrote:

thanks for the code but its not working. I added it in my custom theme
folder but still... i cannot make it work.. please let me know if any
suggestions.
P:S i see no changes on frontend and no new module in admin side.
Thanks
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/czettnersandor/magento-quick-view-ajax/issues/15, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAQEBtyFBf1Q8b_hKyTGA8OQfRDx99yFks5skPkRgaJpZM4PdvFs
.

I have not tried it on default or rwd.. Our website is using its own custom
built theme .
On 20 September 2017 at 18:43, Sándor Czettner<EMAIL_ADDRESS>wrote:

Is it working with the default or rwd theme?
On 20 Sep 2017 12:37, "saraasghar"<EMAIL_ADDRESS>wrote:

thanks for the code but its not working. I added it in my custom theme
folder but still... i cannot make it work.. please let me know if any
suggestions.
P:S i see no changes on frontend and no new module in admin side.
Thanks
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/czettnersandor/magento-quick-view-ajax/issues/15,
or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AAQEBtyFBf1Q8b_
hKyTGA8OQfRDx99yFks5skPkRgaJpZM4PdvFs>
.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/czettnersandor/magento-quick-view-ajax/issues/15#issuecomment-330911309,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Ab51gJsnWhREIF8m3i2CLofRVetS42FJks5skUDJgaJpZM4PdvFs
.


Issue #178: Split up known issues

Split up known issues in a "deliberate deviations from the guidelines" part and a more "ephemeral" part, due to tooling errors etc.
In addition, implement some changes in the tools to handle calculating effective cardinalities better.

Graag even jullie blik om te kijken of jullie het eens zijn met de wijzigingen in known-issues en in de profielen.
NB. alle verplichte vinkjes in deze pull request staan eindelijk op groen :) Er is nog wel het rode vinkje van de "strengere" handmatige test, maar dat is in dit geval loos alarm.

Nice! I like the way the file is now split up. I do not completely understand the new syntax yet with regards to the wildcard '*' but it works so for now I think I can approve the PR. Perhaps bonus points can be won by having a bit of documentation (maybe in the README on the root folder)?

That's a good suggesion, I'll add some documentation.

Nice! I like the way the file is now split up. I do not completely understand the new syntax yet with regards to the wildcard '*' but it works so for now I think I can approve the PR. Perhaps bonus points can be won by having a bit of documentation (maybe in the README on the root folder)?

That's a good suggesion, I'll add some documentation.

I think I'll track back from this statement. At the moment, proper documentation of known issues in general is missing so I'll create a task for this. The wildcard syntax is described in the repo's for the tools that are used.


CssSelectorComplexBridge: Use cookies everywhere (#3827)

Fixes #3827

Suggested reviewer: @LarsStegman

I haven't run the code, but this is a good change. Do you want me to run it?

download this patch and apply it:
wget -O /tmp/3870.patch "https://github.com/RSS-Bridge/rss-bridge/pull/3870.patch"
git apply /tmp/3870.patch

looks good i guess. havent tested it.

oopsie. this generates error.
the second argument is $ttl and must be an int
@ORelio


Only incrementally sync Providers that have been updated since last sync

Context
The current process of our incremental TTAPI sync which runs every 10 minutes is as follows:

Query the API for the providers updated since our last sync
Update or create providers from the list
update the last sync timestamp to current time - 1 hour
Trigger course sync jobs for the provider list retrieved
Query the API for the courses updated since the new timestamp (the one just updated) for the provider
Update or create the courses from the list
Trigger subsequent sites sync

Querying the courses changed since with the updated timestamp meant we were not actually ever getting the updated courses because the TTAPI being one hour behind, meant that we actually missed 10 minutes each time the course sync jobs ran, missing our window to retrieve the correct courses.
A quick fix was merged yesterday to set the last sync timestamp to two hours behind https://github.com/DFE-Digital/apply-for-teacher-training/pull/4904 to factor in the courses that could come in even when we update the last sync time
However a robust solution would be to use the same updated_since across all sync jobs. After a discussion with @duncanjbrown the suggestion was to pass the old updated_since to every job we trigger.
I started going down this path but realised the complexity we are adding with introducing yet more parameters to pass around between different jobs. Instead I resurrected https://github.com/DFE-Digital/apply-for-teacher-training/pull/4765 which removes any syncing from courses/sites and only retains it for the provider list we pull in.
Changes proposed in this pull request

Remove incremental syncing from courses
Remove unnecessary logging now we found the issue

Guidance to review
The price for syncing all courses for updated providers is small to reduce complexity/chance of bugs in the sync. Are there any issues we foresee in the future around this, where we will be syncing a higher number of courses per provider?
Link to Trello card
https://trello.com/c/59TtliPE/3776-incremental-sync-not-adding-new-courses
Things to check

[ ] This code does not rely on migrations in the same Pull Request
[ ] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[ ] API release notes have been updated if necessary
[ ] Required environment variables have been updated added to the Azure KeyVault

Given we will need to stub a response from the TTAPI, given we have test coverage elsewhere I'm not sure if adding a system spec for incremental sync will result a significant increase in coverage / confidence but happy to go with the consensus :-)

Maybe it's not a system spec we want... Ideally we'd have a test that fails without this fix

Looks good, might be nice to add to /create a new system spec that checks we sync a newly added course from the TTAPI in incremental mode

I think as @TobyRet says we have coverage elsewhere — for testing that we correctly request providers with timestamps, and that we correctly request courses (which is now much simpler). I think at this point we can reasonably confident that if a course didn't show up in the incremental sync that would be due to a bug on the TTAPI side where the provider was not marked changed, rather than because we were not making the correct requests?


Issue debugging Blazor WASM in Visual Studio Code

I think I followed everything from the article but without success.
I am currently working on
MacBook Air (M1, 2020)

macOS Big Sur v11.2.3 (20D91)
.NET SDK v5.0.201
Visual Studio Code v1.54.3

C# v1.23.9
.NET Install Tool for Extension Authors v1.0.0
Microsoft.AspNetCore.Razor.VSCode.BlazorWasmDebuggingExtension v1.0.0

Microsoft Edge v89.0.774.54

I am attempting to debug a simple Blazor WASM dotnet new blazorwasm -o MyBlazorApp

launch.json

{
    "version": "0.2.0",
    "configurations": [
        {
            "name": "Launch and Debug Standalone Blazor WebAssembly App",
            "type": "blazorwasm",
            "request": "launch",
            "browser": "edge",
            "cwd": "${workspaceFolder}/src/MyFirstBlazorApp"
        }
    ]
}

This message keeps popping on:
Couldn't find a debug adapter descriptor for debug type 'blazorwasm' (extension might have failed to activate)

I also can't get rid of this from the Problems tab:
{
	"resource": ".../MyBlazorApp/src/MyBlazorApp/Program.cs",
	"owner": "csharp",
	"code": "CS0246",
	"severity": 8,
	"message": "The type or namespace name 'App' could not be found (are you missing a using directive or an assembly reference?) [MyBlazorApp]",
	"source": "csharp",
	"startLineNumber": 18,
	"startColumn": 40,
	"endLineNumber": 18,
	"endColumn": 43
}

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

ID: ec2f61c7-9f51-6735-e328-3c7b495d70c3
Version Independent ID: 4d06e565-282d-6387-aa90-cb2c2986d35a
Content: Debug ASP.NET Core Blazor WebAssembly
Content Source: aspnetcore/blazor/debug.md
Product: aspnet-core
Technology: aspnetcore-blazor
GitHub Login: @guardrex
Microsoft Alias: riande

Hello @RicardoValero95 ... Open this for the product unit at ...
https://github.com/dotnet/aspnetcore/issues
... please add a cross-link to this docs issue and and ping for me so that I can follow the discussion. I'll re-open this if we end up adding content to the doc. Here's a cut-'n-paste for your engineering issue ...
Cross-reference: https://github.com/dotnet/AspNetCore.Docs/issues/21825
cc: @guardrex


[Feature Request] Change all (on/off), (true/false) to three state flag

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I cannot create a project..
Describe the solution you'd like
A clear and concise description of what you want to happen.
e.g. If you want a new command to be added, what would it do?
Additional context
Add any other context or screenshots about the feature request here.

Done


no such property exception in gh-pages plugin

When the origin of build/ghpages has changed (e.g. after changing the config), an exception no such property: logger for class: org.ajoberstar.gradle.git.ghpages.GithubPagesPlugin is thrown.
In GithubPagesPlugin:122 it should be project.logger instead of just logger.

Workaround: manually deleting build/ghpages

Fixed in master. Thanks for catching that!


torch.hub.load example clones and installs fairseq master, ignoring system-wide version

🐛 Bug

Example here: https://github.com/pytorch/fairseq/blob/master/examples/language_model/README.md says:
en_lm = torch.hub.load('pytorch/fairseq', 'transformer_lm.wmt19.en', tokenizer='moses', bpe='fastbpe')

This clones a fresh version of fairseq under ~/.cache/pytorch_fairseq_master, completely ignoring the system-wide fairseq version.
To Reproduce
Steps to reproduce the behavior (always include the command you ran):
Follow the tutorial.
import torch
en_lm = torch.hub.load('pytorch/fairseq', 'transformer_lm.wmt19.en', tokenizer='moses', bpe='fastbpe')

Code sample

Expected behavior

No need to reinstall fairseq, especially not in a temp directory (.cache?!).
Environment

fairseq Version (e.g., 1.0 or master): master
PyTorch Version (e.g., 1.0): 1.7.1
OS (e.g., Linux): Ubuntu 18.04
How you installed fairseq (pip, source): source
Build command you used (if compiling from source): pip install -e ./
Python version: 3.8.5
CUDA/cuDNN version: 10.2
GPU models and configuration: Titan X
Any other relevant information: N/A

Additional context

real nice advance in this bug


fix python relax deps hook

python/hooks: fix python-relax-deps-hook
python/hooks: add sourcing notice to python-relax-deps-hook

Description of changes

Not notifying the hook was sourced while silently not working implies an unnecessary debugging detour; fix it and notify the hook was sourced.
Globbing is brittle and stopped working; use proper find command (shellchecked :heavy_check_mark:).

Things done

[ ] Tested a private package making use of pythonRelaxDepsHook and while it did not work prior to this PR, it now works.

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Root cause was: Version number normalised: '15.0.0-dev' -> '15.0.0.dev0' (see PEP 440)
version=15.0.0-dev is not part of the filename anymode
Not fixed by this PR.


Tests are complicated... is it possible to re-write basic test to support new approach without $GOPATH?

Hello,
i'm trying to understand all this stuff with GO, and have confuse:

GOPATH is not necessary due to new approach: https://stackoverflow.com/questions/21001387/how-do-i-set-the-gopath-environment-variable-on-ubuntu-what-file-must-i-edit/53026674#53026674
and when i'm trying to run basic test (https://github.com/gruntwork-io/terratest/tree/master/examples/terraform-basic-example)  - i see error connected to GOPATH.

are there any best practicies for pointing that GOPATH for this project terratest?
This would be great to hear, cause i would like to introduce terratest to our team, but... i can't understand how to setup it correctly, so everyone will use it by somewhat template.

mrpsy@LVAGPLTP2798:~/repos/terratest/test$ go test -v -run TestTerraformBasicExample
# _/home/mrpsy/repos/terratest/test
packer_basic_example_test.go:10:2: cannot find package "github.com/aws/aws-sdk-go/aws" in any of:
	/usr/lib/go-1.10/src/github.com/aws/aws-sdk-go/aws (from $GOROOT)
	/home/mrpsy/repos/terraform_conf/src/src/github.com/aws/aws-sdk-go/aws (from $GOPATH)
FAIL	_/home/mrpsy/repos/terratest/test [setup failed]

this what i see.

ok, i figured it out.
but:
mrpsy@LVAGPLTP2798:~/src/github.com/gruntwork-io/terratest$ dep ensure
dep: WARNING: Unknown field in manifest: prune
dep: WARNING: Unknown field in manifest: require
ensure Solve(): No versions of google.golang.org/api met constraints:
	master: unable to update checked out version: : command failed: [git checkout 19ff8768a5c0b8e46ea281065664787eefc24121]: exit status 1
	v0.5.0: Could not introduce<EMAIL_ADDRESS>as it is not allowed by constraint master from project github.com/gruntwork-io/terratest.
	v0.4.0: Could not introduce<EMAIL_ADDRESS>as it is not allowed by constraint master from project github.com/gruntwork-io/terratest.
	v0.3.2: Could not introduce<EMAIL_ADDRESS>as it is not allowed by constraint master from project github.com/gruntwork-io/terratest.
	v0.3.1+alpha: Could not introduce<EMAIL_ADDRESS>as it is not allowed by constraint master from project github.com/gruntwork-io/terratest.
	v0.3.1: Could not introduce<EMAIL_ADDRESS>as it is not allowed by constraint master from project github.com/gruntwork-io/terratest.
	v0.3.0: Could not introduce<EMAIL_ADDRESS>as it is not allowed by constraint master from project github.com/gruntwork-io/terratest.
	v0.2.0: Could not introduce<EMAIL_ADDRESS>as it is not allowed by constraint master from project github.com/gruntwork-io/terratest.
	v0.1.0: Could not introduce<EMAIL_ADDRESS>as it is not allowed by constraint master from project github.com/gruntwork-io/terratest.
	v0.3.1-alpha: Could not introduce<EMAIL_ADDRESS>as it is not allowed by constraint master from project github.com/gruntwork-io/terratest.
	master: unable to update checked out version: : command failed: [git checkout b50168921e183c95644f04416271d702a730550c]: exit status 1
	dartman: Could not introduce google.golang.org/api@dartman, as it is not allowed by constraint master from project github.com/gruntwork-io/terratest.

why?

github.com/gruntwork-io/terratest (download)
package github.com/gruntwork-io/terratest: no Go files in /home/yn/src/github.com/gruntwork-io/terratest

its something strange, that it is so complicated to install this :(

Hi,
Apologies for the difficulties you are running into. I understand the pain and we are actively working on figuring out ways to improve this.
With that said, here are a few answers and pointers:

GOPATH is not necessary due to new approach

Unfortunately, our project has not been updated yet to use gomodules (which is a requirement for that approach). In fact, all of the issues you ran into beyond that are because the source repo isn't cloned in the GOPATH src path. You can see the discussion on this PR.
With that said, I would not recommend running the terratest examples directly from the terratest repo unless you are contributing to the project, as it is not a good starting point for your own modules. Here is what I would suggest trying:

First create a new blank folder in the gopath. This can be in any folder structure as long as it is in the GOPATH, which based on the error messages you have it will be /home/mrpsy/src/.
In the folder, create two folders: example and test.
In the example folder, copy the basic terraform example.
In the test folder, copy the corresponding test.
In the test folder, create a Gopkg.toml file with the following content:

[[constraint]]
  name = "github.com/gruntwork-io/terratest"
  version = "0.16.0"

Then, here are the steps to run the tests:

In the test folder, run dep ensure
Once that finishes, run go test -v .

Once you have the above, that can be a good starting point / scaffold for all your terraform modules going forward. Note that the same thing can be done for the helm if you intend on using those (see https://github.com/gruntwork-io/terratest-helm-testing-example).

cool, thanks)
also found, that it is not working with go 1.12.
but for my purpose it is ok.

Can you share the error message you get with go 1.12? I want to create a new issue for it since we should be compatible.

yes.
forgot to copy&paste original one... but just upgraded GO and see this:
mrpsy@LVAGPLTP2798:~/go/src/test$ go test -v .
warning: GOPATH set to GOROOT (/home/mrpsy/go) has no effect
# test
package test (test)
	imports github.com/gruntwork-io/terratest/modules/terraform
	imports github.com/gruntwork-io/terratest/modules/shell
	imports bufio: cannot find package "bufio" in any of:
	/home/mrpsy/go/src/test/vendor/bufio (vendor tree)
	/home/mrpsy/go/src/bufio (from $GOROOT)
	($GOPATH not set. For more details see: 'go help gopath')
FAIL	test [setup failed]

same command on go.1.10 (which was installed by apt install golang) works.

i'm sorry, it was my fault
it works with go 1.12.

Any ideas? whats the corrcet way to define project id as per this point?

Hi @jawoponlenuigalwaycom  can you file a new issue with this exact content, since it is unrelated to this topic which is closed? Thanks!

I'm kind of new to raising issues😅, thanks done
Get Outlook for Androidhttps://aka.ms/ghei36

From: Yoriyasu Yano<EMAIL_ADDRESS>Sent: Wednesday, February 26, 2020 9:43:18 PM
To: gruntwork-io/terratest<EMAIL_ADDRESS>Cc: AWOPONLE, JESSICA<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [gruntwork-io/terratest] Tests are complicated... is it possible to re-write basic test to support new approach without $GOPATH? (#316)
Hi @jawoponlenuigalwaycomhttps://github.com/jawoponlenuigalwaycom can you file a new issue with this exact content, since it is unrelated to this topic which is closed? Thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/gruntwork-io/terratest/issues/316?email_source=notifications&email_token=ANODAEB5JOBGUSZXAMOPUN3RE3O7NA5CNFSM4HSHVSHKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOENCAD3I#issuecomment-591659501, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ANODAEAMLJGYJCCDQOPIA7TRE3O7NANCNFSM4HSHVSHA.


account: allow managment

steps:

[ ] change logout button to profile
[ ] add small menu including "logout", "change password", "setup user data" or similar
[ ] implement change password
[ ] implement change user data

moved to trello


ensure machine_options and configs are symbolised for chef13

This is required to work with cheffish5 and cheffish13.  Also requires a seperate patch which is in a pull request for chef provisioning itself, but with these two we're provisioning on chef 13 reliably

@jjlimepoint i believe this is what you are looking for for core chef-provisioning -> https://github.com/chef/chef-provisioning/pull/576

Actually was thinking of https://github.com/chef/chef-provisioning/pull/572 - which along with the same hash fix has some 13 specific general fixes


[6.x] Use isset but not !empty for changing redis database

But why? Could you provide some information? Does this fix a bug?

I assume this allows explicitly selecting connection 0, which would usually already be the client's default.

Great, I did not know.
@davidhhuan Please provide a description to prevent the "...it lacks sufficient explanation..." answer.


iOS 热更新不生效

iOS 集成好了  但是热更新没反应啊  怎么调试  或者看是什么错误

using google translation :"
Integration is good, but the hot update does not respond. How to debug or see what is wrong."


Overwrite color definitions in rendermime package

Some packages (e.g., @jupyterlab/rendermime) do not use CSS color variables defined as public API in JupyterLab themes. This PR is a monkey-patch for such packages.

It turns out that fileeditor, which I thought needs to be tuned, does not have custom color definition. It just does not have enough color definition, and therefore does not go well with dark theme. I'll merge this PR though the branch is now a bit too broad.

Other patches for package with hard-coded color definition will be submitted as separate PRs.


(QENG-3811) Add AIX and Solaris SPARC platforms

Oh my god these fixture files (rageguy)

Yep, I changed 839 files

Ping @waynr @johnduarte for review.
Apologies for the terrible reviewing experience, GitHub gets slow when dealing with 839 changed files in a PR.

CC @kevpl

We still need https://tickets.puppetlabs.com/browse/QENG-3946 (arbitrary hypervisors) before we can completely get off static config files for the AIX and Solaris SPARC testing.

:+1:  FGTM. Thanks @nwolfe

We still need https://tickets.puppetlabs.com/browse/QENG-3946 (arbitrary hypervisors) before we can completely get off static config files for the AIX and Solaris SPARC testing.

Done -- see #34

@nwolfe I just mentioned this in hipchat, but it's also worth mentioning on the PR--alphabetically sorting the platform mappings is an aesthatic change that would be better-suited to its own PR. Maybe it's not worth spending the time to separate it out here.
Once you remove the alphabetical sort, I think the best way to prevent shuffling fixtures around is to append new fixtures to the platform map--if I remember correctly.

I'm going to extract most of the PR into a separate PR that isolates the test fixture reordering, and also take a stab at improving the fixture generation code so it doesn't produce unnecessary and irrelevant fixture changes.

I'm going to extract most of the PR into a separate PR that isolates the test fixture reordering, and also take a stab at improving the fixture generation code so it doesn't produce unnecessary and irrelevant fixture changes.

Done -- see #35

I'll rebase this PR once #35 is merged

@waynr Rebased.  Changed files down from 800 to 500 😄
CC @kevpl


ClassLabel not working properly

Hi @aseemw ,
I converted my .pb into mlmodel,
`input_tensor_shapes = {'Const_6:0':[1,32,64,3]} # batch size is 1
coreml_model_file = 'DJ3.mlmodel'
output_tensor_names = ['Softmax_2:0']
frozen_model_file = 'modified_deepLogo.pb'
class_labels = [
'Adidas', 'Apple', 'BMW', 'Citroen', 'Cocacola', 'DHL', 'Fedex', 'Ferrari',
'Ford', 'Google', 'HP', 'Heineken', 'Intel', 'McDonalds', 'Mini', 'Nbc',
'Nike', 'Pepsi', 'Porsche', 'Puma', 'RedBull', 'Sprite', 'Starbucks',
'Texaco', 'Unicef', 'Vodafone', 'Yahoo', 'Background'
]
coreml_model = tfcoreml.convert(
tf_model_path=frozen_model_file,
mlmodel_path=coreml_model_file,
image_input_names = 'Const_6:0',
red_bias = -1,
green_bias = -1,
blue_bias = -1,
is_bgr = True,
image_scale = 2.0/255.0,
input_name_shape_dict=input_tensor_shapes,
output_feature_names=output_tensor_names,
class_labels=class_labels,
predicted_feature_name=none,
predicted_probabilities_output=' ')`
While this works properly and converts into mlmodel. But when I integrate that model , then the prediction is always the first element of class label.
i.e in this case the prediction always comes "Adidas".
Can you please help me what am I doing wrong here?

Can you check whether the output of the mlmodel matches the TF output for the same output. You can look at the python notebooks on how to do that.
If TF and Coreml match then there is no issue. (Maybe the TF model itself always gives adidas). If not, then something went wrong during conversion.

Yes, The TF model and mlmodel both of them is giving "adidas", as this is the first element of my classLabel file.

Okay. Then the converter did its job!
I guess you need to look into getting a better model:)


macvim install

Looks like custom-icons is no longer valid and current build errors:
~ » brew install macvim --custom-icons --with-override-system-vim --with-lua --with-luajit
Warning: macvim: this formula has no --custom-icons option so it will be ignored!
==> Downloading https://github.com/macvim-dev/macvim/archive/snapshot-145.tar.gz
Already downloaded: /Users/cdenneen/Library/Caches/Homebrew/macvim-8.0-145.tar.gz
==> ./configure --with-features=huge --enable-multibyte --with-macarchs=x86_64 -
==> make
Last 15 lines from /Users/cdenneen/Library/Logs/Homebrew/macvim/02.make:
	17  libdyld.dylib                       0x00007fff6a939115 start + 1
	18  ???                                 0x0000000000000009 0x0 + 9
)
libc++abi.dylib: terminating with uncaught exception of type NSException
Command /Applications/Xcode.app/Contents/Developer/usr/bin/ibtool failed with exit code 6

** BUILD FAILED **

The following build commands failed:
	StripNIB English.lproj/MainMenu.nib
	StripNIB English.lproj/FindAndReplace.nib
(2 failures)
make[1]: *** [macvim] Error 65
make: *** [first] Error 2

READ THIS: https://docs.brew.sh/Troubleshooting.html

[Running] brew install macvim --custom-icons --with-override-system-vim --with-lua --with-luajit
Warning: macvim: this formula has no --custom-icons option so it will be ignored!

Seems to have installed in recent update...

That was just warning, not an error. But nevertheless I have just merged a PR that fixes it.

[Running] brew install macvim --custom-icons --with-override-system-vim --with-lua --with-luajit
Warning: macvim: this formula has no --custom-icons option so it will be ignored!

Seems to have installed in recent update...

@cdenneen How could I uninstall/undo " brew install macvim --custom-icons --with-override-system-vim --with-lua --with-luajit"?


rename arrowuprighttip to arrowrightuptip

Previously the proposed name for uni21B1 was arrowuprighttip. When looking at similar arrows in the list (see below), I think it makes sense that it is called arrowrightuptip. Moving line 69 to 71 in rangeProcessors/arrows.py fixes this, without causing any other changes.


Disable the OK button until a node is selected.

Fixes #62893.
The OK button of SceneTreeDialog is disabled until a node is selected

.

You need to squash the commits.
Also you accepted my suggestion, which automatically added me as co-author; you should remove that.
Also also, if you are changing the style, the other condition needs a change too.

I tried to fix the issues you mentioned. But I'm not sure if this is the right thing to do with the commits. Thanks for your suggestion.

Cherry-picked for 3.6.

Cherry-picked for 3.5.1.


Server Error when searching for places which doesn't exist

Hey,
I think there is a problem when searching for an address which doesn't exist or at least where the GoogleAPI returns nothing for the request in the DistanceDataExtension.
It causes the DB Query to have an empty "radians()"  which returns an sql error.
For example if you put in: "sobiret". I have found very few inputs which throw this error but this one and some others do.
Thank you

@michaelwolz thanks for reporting this bug. We'll investigate.


Local.testAccounts vs. noUncheckedIndexAccess

With noUncheckedIndexedAccess enabled (best practice), the TS checker errors on the inner destructuring of testAccount elements.
let [{ privateKey, publicKey }] = Local.testAccounts
//   ~~~~~~~~~~~~~~~~~~~~~~~~~
//   ^
//   Property 'privateKey' does not exist on type '{ publicKey: PublicKey; privateKey: PrivateKey; } | undefined'. ts(2339)

Is there a reason not to type the length of Local.testAccounts? We could do so with the TupleN type right here. This would spare the developer of needing to override the checker.
let [{ privateKey, publicKey }] = Local.testAccounts // no TS error, sweet!

Damn i didn't know noUncheckedIndexAccess. I love it!


Memory leak through use of HttpClient in SeqSink

SeqSink leaks memory through creating new instances of HttpClient. HttpClient is intended to be instantiated once, and then reused for all future requests. Creating new HttpClients will leak memory and network connections over time.
You can read more details on this subject in this article: https://aspnetmonsters.com/2016/08/2016-08-27-httpclientwrong/
You can also find more details in the Windows azure performance manual here: https://docs.microsoft.com/en-gb/azure/architecture/antipatterns/improper-instantiation/
The fix is updating the variable here: https://github.com/serilog/serilog-sinks-seq/blob/dev/src/Serilog.Sinks.Seq/Sinks/Seq/SeqSink.cs#L65
to be static scope, and making the necessary changes to the use of the object so that it works through reuse (ie. set BaseAddress on the request, and not on the HttpClient). There is also no need to call Dispose on the object in the destructor as it has no effect (HttpClient cannot be disposed).
See this image for example of how hundreds of HttpClient objects are created (and leaked) after logging through Seq:

Hi @hjnilsson,
Thanks for dropping by.  Just wondering how you are instantiating Serilog and the Seq Sink?

@hjnilsson as @merbla is suggesting, it looks like you're creating multiple instances of the Seq sink. Just like HttpClient, only a single Seq sink instance (per server, at least) should be created within an app. Check out:
https://github.com/serilog/serilog/wiki/Lifecycle-of-Loggers
for some info on how Serilog expects loggers to be created and reused.
Closing as this thread has gone quiet, but please get back in touch if you need more information or still think there's a problem here. Thanks!


Option to make World space not have inverted y-axis with origin at bottom left

Currently, world space by default has origin set to top-left with an inverted y-axis. This means that for objects to move up in world space, they need to move in the negative y-direction. Although this mirrors how the canvas is drawn, this feels unintuitive when programming how objects move in the game world and makes server code more confusing when representing player/game object velocities and coordinates.
It would be nice to have an option for world space to have a non-inverted y-axis with the origin at the bottom left.

This will never happen for Phaser 3, so please add it to the Phaser 4 repo
I can't move issues between repos, otherwise, I would do so to save you posting it again.


No server API docs

There are no docs describing the communication with the game server. The doc must consist of two separate parts:

Stage communication (how to: quick game, create game, list games, join game)
Gameplay communication (all events emitted and received by Socket.IO with data definitions)

Initial plaintext server API doc is committed: bbb72b3e7d09d1f5b5ae0d6df35b68001e59f921


Disable large file tests in test_mmap

The large file tests (test_large_filesize and test_large_offset) in test_mmap create very large files and can be somewhat slow. These were disabled (on purpose) in IronLanguages/main because the test was run via the regrtest launcher. It would be nice to be able to disable them using the new test running (without having to modify the source).

This is fixed in #170

Fixed!


How to disable FFMPEG logger?

Sometimes there will be such a task is terminated due to an error
    Input/output error @ io_write - <STDERR>

After the restart the resque and retry tasks will can be return to work.
Here's my task code:
  transcode_cmd =  %Q(-threads 0 )

  transcode_cmd << %Q(-codec:v libx264 )

  transcode_cmd << %Q(-vf scale="#{resolution}:trunc(ow/a/2)*2" )

  transcode_cmd << %Q(-f hls -hls_allow_cache 1 )

  transcode_cmd << %Q(-hls_time 10 -hls_list_size 0 )

  transcode_cmd << %Q(-hls_key_info_file "#{key_info_file}" )

  transcode_cmd << %Q(-hls_base_url "//endpoint/" )

  transcode_cmd <<  %Q(-hls_segment_filename "#{output_m3u8_dir}/stream%03d" )

  movie = FFMPEG::Movie.new(video.filename)

  movie.transcode(output_m3u8, transcode_cmd, validate: false) do |progress|
    video_file.update_attribute :progress, progress*10000
    Rails.logger.debug "transcoding #{file_basename}:#{resolution}p in: #{progress*100}"
  end

I found out this err is from logger, How to disable logger?

Refer to answer in #105 .


License

Hi There,
Is Pingmesh code open source under any license?
Wonder what permissions do I have to use this code?

Yes
Free


state a custom inventory file must be at the same directory as group_vars

SSIA
Fixes: #1864

This change is

lgtm

Thanks for the review

Should we do any checks in init.sh

@eparis what could work too

@eparis PTAL. I have rephrased the statement in README to be more open. The group_vars/all.yml is not the only solution. Individual rules can have its own group_vars files. Extending init.sh and underlying script files with check as well.

@eparis what about now?

/lgtm


Reference Android App for Custom Settings profile

See README.MD

@HawaiiRyan - Please make sure to click through the CLA so that I can merge this request.

Will do. Thanks Mike

Closing to reopen PR and verify CLA bot does not prompt for license agreement.


[BUG] output files can't be loaded into correct GMT grid files

Describe the bug
Output files have unequal spacing and inconsistent geotransform information.
To Reproduce
Steps to reproduce the behavior:

Download
wget https://aria-share.jpl.nasa.gov/20200623_Oaxaca_Mexico_EQ/Displacements/S1-GUNW_COSEISMIC-A-R-107-tops-20200625_20200619-003043-16794N_15045N-RR-35f2-v2_0_3.nc
Command used

ariaExtract.py -f "products/S1-GUNW-A-R-107-tops-20200625_20200619-003043-16794N_15045N-RR-35f2-v2_0_3.nc" -l all -d download -w products -m download --outputFormat ENVI
gdal_calc.py -A products/unwrappedPhase/20200625_20200619.vrt -B products/connectedComponents/20200625_20200619.vrt --outfile results/S1_A107_20200625_20200619_disp_cm.tif --calc='((A*(B>0))/(4*pi)*5.546576)' --format=GTiff --NoDataValue=0 --overwrite
gdal_translate -of GMT S1_A107_20200625_20200619_disp_cm.tif S1_A107_20200625_20200619_disp_cm.grd
gmt grdlandmask -Glandmask.grd -RS1_A107_20200625_20200619_disp_cm.grd
gmt grdinfo landmask.grd

Error Output

grdinfo [ERROR]: (x_max-x_min) must equal (NX + eps) * x_inc), where NX is an integer and |eps| <= 0.0001.
grdinfo (gmtlib_read_grd_info): Use grdedit -A on your grid file to make region and increments compatible [landmask.grd]
landmask.grd: Title: z
landmask.grd: Command: grdlandmask -Glandmask.grd -RS1_A107_20200625_20200619_disp_cm.grd
landmask.grd: Remark: Derived from the low resolution shorelines
landmask.grd: Pixel node registration used [Geographic grid]
landmask.grd: Grid file format: nf = GMT netCDF format (32-bit float), CF-1.7
landmask.grd: x_min: -96.5451065365 x_max: -93.9876553945 x_inc: 0.00083331741351 (3 sec) name: longitude n_columns: 3069
landmask.grd: y_min: 15.046045981 y_max: 16.7804535652 y_inc: 0.000833449103383 name: latitude n_rows: 2081
landmask.grd: z_min: 0 z_max: 1 name: z
landmask.grd: scale_factor: 1 add_offset: 0
landmask.grd: format: netCDF-4 chunk_size: 134,131 shuffle: on deflation_level: 3

Expected behavior
There should be a way to get the output into a correct GMT grid for further processing. The output file has been resampled from the original 3-arcsecond spacing to have different "x_inc" and "y_inc" values. In addition the "xmin" and "xmax" that don't exactly fit the "xinc" times the number of columns. This makes it an invalid GMT grid that can't be used for other calculations.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

ARIA-tools git tag: [e.g. git show --summary]
commit f39a8ee74acf8c40e641560a2d2ef2eb220acdb4 (HEAD -> dev, origin/dev, origin/HEAD)
Merge: 561950c 95dc360
Author: sssangha<EMAIL_ADDRESS>Date:   Fri Jun 19 18:15:38 2020 -0700
Merge pull request #176 from sssangha/sss_avgrasterfix
Fixed bug in average raster generation and reporting failed IFGs

OS
MacOS 10.14.6

Additional context
I need to use GMT grdmath to do additional calculations but it can't use the files.

hello @EJFielding,
I wasn't able to reproduce your error because (I think) there are some inconsistencies in the file names you gave in your commands. Also I have GMT6 but the grdlandmask throws the error due to -R not being a valid format:
grdlandmask [ERROR]: Could not parse S1_A107_20200625_20200619_disp_cm.grd into Geographical, Cartesian, or Temporal coordinates!grdlandmask [ERROR]: Syntax error -R option. Correct syntax:
	-R<xmin>/<xmax>/<ymin>/<ymax>[/<zmin>/<zmax>]
	  Append r if giving lower left and upper right coordinates
	-Rg or -Rd for global domain
	-R<grdfile> to take the domain from a grid file
grdlandmask [ERROR]: Offending option -RS1_A107_20200625_20200619_disp_cm.grd

However, I think you can define the final product output format as GSBG in gdal which is the grd format I successfully use with gdal and GMT. I am leaving the command below.
gdal_calc.py -A products/unwrappedPhase/20200625_20200619.vrt -B products/connectedComponents/20200625_20200619.vrt --outfile results/S1_A107_20200625_20200619_disp_cm.tif --calc='((A*(B>0))/(4*pi)*5.546576)' --format=GSBG --NoDataValue=0 --overwrite

I have not heard of GSBG format before.

Golden Software Binary Grid
https://gdal.org/drivers/raster/gsbg.html

Thanks.
It is also possible to extract the full-resolution layers directly from the original ARIA NetCDF files with:
gmt grdconvert 'S1-GUNW_COSEISMIC-A-R-107-tops-20200625_20200619-003043-16794N_15045N-RR-35f2-v2_0_3.nc=gd?NETCDF:"S1-GUNW_COSEISMIC-A-R-107-tops-20200625_20200619-003043-16794N_15045N-RR-35f2-v2_0_3.nc":/science/grids/data/amplitude' -Gamp.grd

This keeps the original exact 3-arcsecond spacing.

Here is the info about the GMT grid created directly with grdconvert:
grdinfo amp.grd
amp.grd: Title: Produced by grdconvert
amp.grd: Command: grdconvert S1-GUNW_COSEISMIC-A-R-107-tops-20200625_20200619-003043-16794N_15045N-RR-35f2-v2_0_3.nc=gd?NETCDF:"S1-GUNW_COSEISMIC-A-R-107-tops-20200625_20200619-003043-16794N_15045N-RR-35f2-v2_0_3.nc":/science/grids/data/amplitude -Gamp.grd
amp.grd: Remark:
amp.grd: Gridline node registration used
amp.grd: Grid file format: nf (# 18) GMT netCDF format (float)  (COARDS-compliant) [DEFAULT]
amp.grd: x_min: -96.5533333333 x_max: -93.91 x_inc: 0.000833333333333 name: x nx: 3173
amp.grd: y_min: 15.045 y_max: 16.795 y_inc: 0.000833333333333 name: y ny: 2101
amp.grd: z_min: 0 z_max:<PHONE_NUMBER> name: z
amp.grd: scale_factor: 1 add_offset: 0

@EJFielding @ehavazli  Would be happy to welcome an example notebook on how to make a GMT6 figure:
e.g. how to plot up an unwrapped IFG in cm units in GMT with a color figure and scalebar.
e.g. how to plot up a wrapped IFG in rad with a colorbar and scale bar
The Hawaii IFG we use for the extract notebook would be well suitable example.
it already captures how to download the product, how to stitch it, and would then we great to see how it gets plotted.

@dbekaert I can help with a notebook on GMT plotting. We still need to fix the ariaExtract.py output. Did you mean using the gmt grdconvert command to make GMT files directly and then plotting those? When I use gmt grdconvert on the GUNW_COSEISMIC pair, I get a grid that is 3173 by 2101, but the ariaExtract output is 3069 by 2081, so the size changes somewhat. That means that I can't use the LOS vector files from ariaExtract with the unwrapped phase from grdconvert without doing some additional resampling.

@EJFielding it would be best to use the outputs of ariaExtract.py directly and plot through GMT, making sure to specify the GSBG format with the -of argument as Emre suggested so you can directly leverage the outputs of ariaExtract.py. Makes sense?

@sssangha Yes, the GSBG format helps avoid an extra step of running gdal_translate to make a file that GMT can read, but it does not fix the problem that the pixel spacings and image limits are not quite correct, so GMT refuses to work with the files from ariaExtract.py.

I tried using the -of GSBG in the ariaExtract.py and now I am getting errors:
***Extract Product Function:***
All (1) GUNW products meet spatial bbox criteria.
Group GUNW products into spatiotemporally continuous interferograms.
All (1) interferograms are spatially continuous.
All layers are to be extracted, pass all keys.
Thread count specified for gdal multiprocessing = 2
***Downloading water mask... ***
Downloaded 3 arc-sec SRTM DEM here: products/DEM/SRTM_3arcsec.dem
Generating: unwrappedPhase - [==================================================] 20200625_20200619
Generating: coherence - [==================================================] 20200625_20200619
Generating: connectedComponents - [==================================================] 20200625_20200619
Generating: amplitude - [==================================================] 20200625_20200619/Users/fielding/anaconda3/envs/ARIA-tools/lib/python3.8/site-packages/ARIAtools/extractProduct.py:344: RuntimeWarning: overflow encountered in multiply
  update_file.GetRasterBand(1).WriteArray(mask.ReadAsArray()*gdal.Open(outname+'.vrt').ReadAsArray())

Generating: bPerpendicular - [==================================================] 20200625_20200619/Users/fielding/anaconda3/envs/ARIA-tools/lib/python3.8/site-packages/ARIAtools/extractProduct.py:633: RuntimeWarning: overflow encountered in multiply
  out_interpolated = mask.ReadAsArray()*out_interpolated

Generating: bParallel - [==================================================] 20200625_20200619
Generating: incidenceAngle - [==================================================] 20200625_20200619
Generating: lookAngle - [==================================================] 20200625_20200619
Generating: azimuthAngle - [==================================================] 20200625_20200619

@EJFielding I was unable to duplicate this error:
***Extract Product Function:***
All (1) GUNW products meet spatial bbox criteria.
Group GUNW products into spatiotemporally continuous interferograms.
All (1) interferograms are spatially continuous.
All layers are to be extracted, pass all keys.
Thread count specified for gdal multiprocessing = 2
Downloaded 3 arc-sec SRTM DEM here: test_dir/DEM/SRTM_3arcsec.dem
Generating: unwrappedPhase - [==================================================] 20200625_20200619
Generating: coherence - [==================================================] 20200625_20200619
Generating: connectedComponents - [==================================================] 20200625_20200619
Generating: amplitude - [==================================================] 20200625_20200619
Generating: bPerpendicular - [==================================================] 20200625_20200619
Generating: bParallel - [==================================================] 20200625_20200619
Generating: incidenceAngle - [==================================================] 20200625_20200619
Generating: lookAngle - [==================================================] 20200625_20200619
Generating: azimuthAngle - [==================================================] 20200625_20200619
(ARIA-tools) [ssangha@leffe Eric_test]$ ariaExtract.py -f "products/S1-GUNW_COSEISMIC-A-R-107-tops-20200625_20200619-003043-16794N_15045N-RR-35f2-v2_0_3.nc" -l all -d download -w test_dir --outputFormat GSBG

We did push a series of commits, some of which addressed minor issues with the metadata fields, so could you please try to update to the latest release and relaunch? Thanks!

@sssangha I did an update last night and ran it again with the same result. It seems to be a problem with the masking. I notice that you did not include the masking flag. This is what I used ariaExtract.py -f "products/S1-GUNW_COSEISMIC-A-R-107-tops-20200625_20200619-003043-16794N_15045N-RR-35f2-v2_0_3.nc" -l all -d download -w GSBG -m download --outputFormat GSBG
The same command with the output format set to ENVI works fine, so it is some kind of bug when the GSBG output is used.

@EJFielding very strange, looks like a numpy issue. The PNG previews indeed look bad, but the phase fields in the actual rasters are fine. But the problems seems to be that with the GSBG format, for some reason the masked areas are inconsistently set to infinity (see navy blue values in the attached image), which is what is raising this warning. I can come up with a bandage solution to simply replace infs with 0s, but I will work to see if I can come up with a cleaner permanent solution:

@sssangha Yes, you are right. In fact, there are areas of positive and negative infinity in the ocean, which gives that strange pattern with autoscaling.

@EJFielding thanks for helping me hash out what I wouldn't necessarily call a bug per se, but an oversight.
It seems for some data formats the nodata value for the mask file is set to Inf/-Inf. I now make sure to always define the nodata value as 0 for maskfiles. Please test out the PR I just posted.

@sssangha Can you confirm that the extract product do no longer have the default standard product sampling?
If that is the case it would mean that the DEM does not have the -tap command in its gdal warp.
The DEM as we download should have regular spacing.

@sssangha @dbekaert
to cut some of the backtracking work,
here is the pixel size from the product:
Pixel Size = (0.000833333333333,-0.000833333333333)

here is the geotransform value from the unwrappePhase vrt file
 (0.00083331741351013496, -0.00083344910338289198)

please check the code starting from line 593 to line 602 in extractProduct.py:
https://github.com/aria-tools/ARIA-tools/blob/7d173b0aa2b4a3a6459ac674f253c7e4058b803f/tools/ARIAtools/extractProduct.py#L593:L603

when tap is enforced it cannot be used together with width and length. @sssangha suggest to potentially reverse engineer the with and length to be integer multiples of the original posting.

@ehavazli cn you confirm that the sampling is now identically?  see PR #192

@dbekaert @EJFielding @sssangha
#192 solved the problem in my environment.

@ehavazli could you show some figures of the results here?
Just to clarify the error is gone, but did you also have a workaroudn for the +- inf

@EJFielding The files should not be compatible in terms of grid-sampling. Thanks @sssangha for implementing. This should at least allow not to use ARIA-extract to anotehr format when loading into GMT.

The new version now works great! I was able to set the output format to ENVI and get a file that I can successfully convert to GMT with gdal_translate. I had to do one extra grdmath step to set the zero pixels to NaN, but it did not complain about the grid spacing and extent. Then I was able to load into the Python fault modeling software that I use (Classic Source Inversion). Here is the output from the matplotlib call in that software.

That great @EJFielding

One other comment while I still have a browser tab open about GSBG (https://desktop.arcgis.com/en/arcmap/latest/manage-data/raster-and-images/supported-raster-dataset-file-formats.htm). It seems that the standard file extension for GSBG files is .grd and other formats have standard extensions. This is usually not a problem on MacOS or Linux, but people trying to use Windows might have problems if the ARIA-tools output files don't have the standard extension. I add this comment in case it helps someone later.


TDP: Activity detail page tweaks

Additions

Adds a link to the learn the building blocks page in the sidebar
Styles <small> elements inside an <h4> to have a font-size of 14px
Adds a paragraph of text in the Download activity section
Adds a horizontal line with spacing above the added paragraph

How to test this PR

Pull branch
Build front-end assets
Visit http://localhost:8000/practitioner-resources/youth-financial-education/teach/activities/saving-post-secondary-education/

Screenshots
Sidebar before:

Sidebar after:

Download activity before:

Download activity after:

Checklist
[Feel free to delete any checkboxes that are not applicable to this PR.]

[x] PR has an informative and human-readable title
[x] Changes are limited to a single goal (no scope creep)
[x] Code follows the standards laid out in the CFPB development guidelines

Front-end testing
Browser testing
Visually tested in the following supported browsers:

[ ] Firefox
[x] Chrome
[x] Safari
[ ] Edge 18
[ ] Internet Explorer 11 and 8 (via emulation in 11's dev tools)
[ ] Safari on iOS
[ ] Chrome on Android

Further guidance on browser support
Accessibility

[x] Keyboard friendly (navigable with tab, space, enter, arrow keys, etc.)
[x] Screen reader friendly
[x] Does not introduce new errors or warnings in WAVE

Other

[x] Is useable without CSS
[x] Is useable without JS
[x] Does not introduce new lint warnings
[x] Flexible from small to large screens

@Scotchester Since I can’t request you as a reviewer in the GitHub UI I’m tagging you here.

One other minor food-for-thought comment: Given that even with properly-styled small text, the parenthetical link in the heading will wrap to a second line if the window is between 900 and 1180ish px, are we sure that the small-within-heading approach is the route we want to take? I would consider just putting it on its own line, outside of the heading. This would also avoid the situation where a screenreader reads out "building block what are the building blocks" if a user is cycling through the headings on the page.

@Scotchester I refactored the new link to show up below the heading rather than next to it. It should now look like this:

I also made the changes you requested to the download links.


Use @vscode/test-cli to run tests

Migrates to the new @vscode/test-cli package to download the appropriate VS Code and run the tests. This streamlines the test setup code, and should be a more reliable way to run the tests as VS Code evolves.
Hopefully this also helps with the occasional SIGSEGV errors that happen in CI on startup.

I have this setup with my WIP for unit testing, I'll push that and we can reconcile this patch with that

@award999 ah, sorry I didn't realize you already had some code going for this. I wrote most of this a while ago and revived it after seeing @matthewbastien getting those SIGSEGV errors in CI hoping this would help.

@award999 ah, sorry I didn't realize you already had some code going for this. I wrote most of this a while ago and revived it after seeing @matthewbastien getting those SIGSEGV errors in CI hoping this would help.

NP, that patch will take longer to get in so let's use this one for just the test runner portion

@swift-server-bot test this please

@swift-server-bot test this please

Thanks @award999 for providing the code you’d written for this PR

@swift-server-bot test this please

@swift-server-bot test this please

@swift-server-bot test this please

@swift-server-bot test this please


ANSI SQL Adapter

Which languages to target?

Node
Python
....

Intrigued by what this is - are we talking general adapters using native language libs/support?


Bodies/shapes do not support negative scale

Hello, i am making a small physics based game and currently trying out Jolt as a physics engine.
The game has a lot of StaticBodies with their collision placed offcenter, that often get inverted on a single axis for variation, since i don't want to have to make a separate scene for every inverted one.
However unlike GodotPhysics, Jolt does not handle these correctly, as you can see in the reproduction project.
MinReproduction.zip

Thank you for reporting this, and for providing the minimal repro.
Negative scaling is something I seem to have forgotten to test altogether, and is indeed broken.
Jolt does seem to support it as well, so it's likely some small thing on my end that needs to be fixed.

It did turn out to be a small thing, thankfully.
The next release (containing this fix) will likely drop some time next week, but you can always build from source if you need the fix sooner.

Just curious, in Godot physics, scaling any collision shape is a terrible idea according to rburing. Does jolt work correctly with scaled collision shapes? Why does having a negative scale make any sense at all, does it swap the faces with backfaces?

I honestly have no idea. I was expecting it to not be supported when I first tried it, since it does mess with my head a bit as well, but even ConcavePolygonShape3D works just fine.
Jolt seems to have been designed with scaling in mind from the start though, unlike Godot Physics, allowing for things like scaling compound shapes (which I use to scale bodies, even dynamic ones) as well as being able to scale most shapes non-uniformly. I assume it has something to do with the fact that Jolt always takes scale separate from the rest of the transform.

This is included in the newly released 0.3.0-rc1. I'd appreciate it if you could give it a try @ettiSurreal and report back how it works out for you, either here or in the release's dedicated discussion.

Tried it out, works perfectly now. Thanks!


Drawing web

It's a website where we can draw any design with any colour and also background change available.After that you may download your work.

Please assign ASAP..


Cannot download BASE_BOX

In Vagrantfile I see:
$BASE_BOX    = "windows2012r2min-wmf5-virtualbox"
But a vagrant up dc01 shows:
==> dc01: Adding box 'windows2012r2min-wmf5-virtualbox' (v0) for provider: virtualbox     dc01: Downloading: windows2012r2min-wmf5-virtualbox An error occurred while downloading the remote file. The error message, if any, is reproduced below. Please fix this error and try again.
How can I obtain this base box? or should I use another name so vagrant will actually know about it?
Thank you!

I cannot publish those images as it would most probably be a violation of the Windows EULA. However, you can easily build an image by using the following project:
https://github.com/dbroeglin/packer-templates
and entering this command (after installing packer):
packer build -force -only virtualbox-iso vbox-2012r2-wmf5.json

For detailed explanations about how the project works you can read Matt Wrock's blog posts: http://www.hurryupandwait.io/blog/creating-a-windows-server-2016-vagrant-box-with-chef-and-packer
If you want the lab to work with the final image you will need some of my changes:
https://github.com/mwrock/packer-templates/compare/master...dbroeglin:master
especially keeping Windows Features in the base box.
However, some might annoy you like
https://github.com/dbroeglin/packer-templates/commit/5146a13248f842d6e8dae81a4221f306ae379dba which changes the keyboard to Swiss-French layout ;-)
You can customise those to your own preferences.
Please don't hesitate to contact me if you need more help!

FYI, I added a bit of documentation at the bottom of the README: https://github.com/dbroeglin/windows-lab

Thank you very much!

@icassina were you able to make the build work ? Please feel free to give me some feedback about any issue you might have encountered. I created this lab as an exercise for my own learning. It might not be easily reusable for somebody else. I would be happy to improve it though!

I used a published 2012r2 box with "wmf5" preinstalled and it worked fine, but I imagine others might struggle with it in the future for the same reason.
I just needed an environnment to test windows SSO (so I never tested IIS nor NetScaler) and I got it thanks to you! … at least until the base box I'm using will expire, then I'll have to use packer anyway. I'll let you know how it went when I try.


fix: strict equality in useLocalStorage

Hi.
I changed equality to strict equality operator. I think it was typing mistake :)

Hi, good catch 👍 Thanks


Align Command Palette aliases to k9s aliases

As Sebastian and Mathew I would like to be able use the same shortcuts as K9s application.
Acceptance Criteria

[ ] Follow Project DOD
[ ] We align our shortcuts to follow the k9s aliases
[ ] We have a dynamic way of updating our aliases depending on the resources running in the cluster (CR can be accessed by their name )
[ ] Tests are present testing the functionality
[ ] Handle commands for navigation to custom resource list (e.g. cluster/shoot--kyma-prow--nkyma/customresources/clusteraddonsconfigurations.addons.kyma-project.io) and custom resource details (/cluster/shoot--kyma-prow--nkyma/customresources/compassconnections.compass.kyma-project.io/a)
[ ] 2 actions for Namespace - switch as a default and go to overview

Tasks

Attachments

Related #1212, but it won't close this issue.


[Function Expose] Page 5471 RegisterFieldSet

Hey,
could you please expose the function RegisterFieldSet so that we can add fields to this page with pageextensions.

The pages of type API are intended to be extended as we would like them to identical across tenants. You can however create your own API as an extension.
Thanks
Anders

The fix was rejected. I've removed the "ships-in-future-update" tag to reflect the correct status.


add notes on sizeconfig and multiformat

This is a PR for the updates introduced by this Prebid.js fix:
https://github.com/prebid/Prebid.js/pull/3938
Add some clarifying notes about how sizeConfig works with multi-format type bids; namely that it only applies when the banner mediaType is part of the adUnit.

thanks @jeanstemp

went out with 2.23


Send notifications about certificates that are about to expire

Right now there are no e-mails sent about certificates that are going to expire. The notification machinery could also distinguish server certs from client certs.

Finally implemented with 14b5761a3686fc04e00ee66faa2d4bad424d27cc


Odd quoting in output

It looks like all the double quotes in p4harmonize's output are escaped in the log output, which is a little confusing. For example:
2023.12.15-20:13:45   --> Exec: p4 -p ssl:perforce.example.com:1666 -u jorgen -c ec2_ondemand_engine_sync_harmonize -x \"/tmp/p4harmonize_delete_3938411009.txt\" delete -c 45400
2023.12.15-20:13:45   --> Exec: p4 -p ssl:perforce.example.com:1666 -u jorgen -c ec2_ondemand_engine_sync_harmonize add -c 45400 -t text -I -f \"/home/ubuntu/ws/org-harmonize/Engine/Source/ThirdParty/CEF3/Build/linux/Toolchain-clang.cmake\"
2023.12.15-20:13:45   --> //proj/engine/Engine/Source/ThirdParty/CEF3/Build/linux/Toolchain-clang.cmake#3 - opened for add
2023.12.15-20:13:45   --> Exec: p4 -p ssl:perforce.example.com:1666 -u jorgen -c ec2_ondemand_engine_sync_harmonize revert -a -c 45400 \"/home/ubuntu/ws/org-harmonize/...\"
2023.12.15-20:13:45 Success! All changes are waiting in CL #45400. Please review and submit when ready.
2023.12.15-20:13:45 Remember to delete workspace \"ec2_ondemand_engine_sync_harmonize\"
2023.12.15-20:13:45 and local folder \"/home/ubuntu/ws/org-harmonize\"

Both the arguments to p4 delete are escaped in the terminal and the workspace name & local folder in the final status message, along with other messages. I wouldn't expect backslashes here -- it makes it hard to copy-paste the output to e.g. run a command manually or copy the path name.

Yeah, I agree with you. The escapes are happening in the logger, frog, but I don't think that the logger should escape those quotes in that case. I'll push a new version of frog, then update p4harmonize to use it.

OK, should be fixed in the latest release: https://github.com/danbrakeley/p4harmonize/releases/tag/v0.6.0


Improving Rendering Performance on Town12 Map

Hi CARLA team,
I'm making use of the excellent and massive Town12 ( https://carla.readthedocs.io/en/latest/map_town12/ ) map but am experiencing poor rendering performance with it! I have few ideas on how to improve it and welcome any recommendations:

Town12 uses Large Maps and the tile_stream_distance can be reduced to reduce the # of tiles loaded in ( https://carla.readthedocs.io/en/latest/large_map_overview/ ) however, when I set this value very low (ex: to 1) I still see 9ish tiles loaded in at any given time when driving through the environment! How could I reduce the number of tiles to the tile I'm in + any tiles within tile_stream_distance away?
When visualizing the shader complexity the building interior's seem really expensive (both the fully 3D interiors and cubemaps), is there any way to set the minimum LODs of all the building higher such that these interiors aren't loaded in? Alternatively, is there a way to improve the glass material used for these windows?

In the layered maps (https://carla.readthedocs.io/en/latest/core_map/#layered-maps) there are methods to turn off certain levels in the API. Does there exist an easy way to do this for the tiled maps? Specifically, is there an easy way to not load all the decals on the roadway?

Many of the objects, especially the trees, are integrated into the map using Hierarchical Instanced Static Mesh Components (HISMC). I've never used HISMCs before, but is there an easy way to scale the LOD of these objects? So I could generally use LOD1 at the lowest, but prefer LOD2 more frequently without changing the base asset?

Most (all?) of the fences use the wallmaster material which uses a masked blend mode. Given the number of fences in the environment do you think it'd be worth doing something to replace this with an opaque blend mode?

The street lights and lamps utilize a Blueprint that exposes a pointlight and spotlight component. During the daytime I don't believe these are activated. Should I expect that there is no performance cost during the daytime?

Any other ideas are welcome! We'd love to use this environment.

Could you tell me which GPU you used to open town12? I can't open the map with 3090, I don't know the reason.


[UI]: Fix Search Input Field

Current Behavior
Current behavior of input field, the color when we focus or type in it is not as expected for keppel color. This issue ticket is sub-ticket of #7863
Desired Behavior
Expected behavior of every input field should be when we focus or type something, the color should be of keppel as it is action item.
Screenshots/Logs
Current behavior screenshot:

Environment

Browser: Chrome Safari Firefox
Host OS: Mac Linux Windows
Meshery Server Version: stable-v
Meshery Client Version: stable-v
Platform: Docker Kubernetes

Contributor Guides and Handbook

🎨 Wireframes and designs for Meshery UI in Figma
🖥 Contributing to Meshery UI
🙋🏾🙋🏼 Questions: Discussion Forum and Community Slack

@Yashsharma1911 I would like to work on this,
Please note that this will be the last milestone to fix the #7863 Report. Rest all fixes are done 👌

I've worked on a similar issue in this report itself. If @Yashsharma1911 is not ready, I would like to work on it.

Hey @HarshDobariya79 thanks for raising your hands let's assign this to @samyakshah3008 as he asked first though we have other similar issues where you can help ^^

Oh sorry, I thought he asked if you want to work on it. Go ahead @samyakshah3008 :)

if no body is working on this issue, can you assign me?

I want to work on this issue too, if possible?

Since I am occupied with some other issue ticket, I have a load to share.
Thanks for raising your hands @ranjanmangla1 and @deepchauhan2002
Assigning in chronological order, please go ahead @ranjanmangla1

@ranjanmangla1  are you working on this issue?

@priyanshu-kun yes

How are we progressing on this issue ticket, @ranjanmangla1 ? Any blockers you are facing?

If no one is working on this issue. I can take it up.

@samyakshah3008 sending pr in a hour

Hi, @Ghat0tkach can I work on this issue?

@hemantmm can you confirm two things

have you able to run meshery locally
If yes then is this issue still persist, if it persist then please raise PR and you can inform here

@Yashsharma1911 can i get this assigned

can I work on this @Yashsharma1911 !!


The plugin asks me to update it to version 0.4.0 or higher

I'm sorry to open an issue again, but I have a new problem that came up.
This morning, I was able to share my note without any problem. I didn't touch anything in particular, and now, when I try to share a note, it asks me to update it to version 0.4.0 or higher.
I've tried forcing the update with BRAT, there isn't one, I'm currently on 0.3.20. There is no more recent release currently on GitHub. What could this be due to?
I'd be embarrassed to have to reinstall share note because we've lost our user ID and I'd have to change all the note urls.

No problem, let me know if any issues :)


0.3 release logo

in 0.3 we'll have a GUI that might be a reason to move away from ASCII logos? ;)

that was some time ago..., by now we have 0.4 and a logo for that


Joypad joystick input sensitivity regression (3.2.4 beta 2)

Godot version:
3.2.4 beta 2
OS/device including version:
Windows 10 64bit
Issue description:
I may have a slightly drifting analog stick on my joypad, but in 3.2.3 I am getting none or hardly any random joystick inputs, whereas in 3.2.4 beta I am getting loads (causing interference with other input events.)
Steps to reproduce:
The minimal reproduction project spits out any unhandled events, I am getting none in 3.2.3, but here is what is happening in 3.2.4 beta 2 (ignore the meta/shift presses, that was me capturing the image):

Minimal reproduction project:
InputEcho.zip

Likely culprit: https://github.com/godotengine/godot/pull/43233 cc @madmiraal

We removed the filter of 0.01 that was preventing changes of around 0.004 getting through (see #42876) with the expectation that this was the smallest change that would be received i.e. 256 steps. However, it appears that your joystick is sending changes of 0.000336 (around a 1/10 of the size). So perhaps a filter of 0.001 is needed.

Alternatively, the problem may not be with the joystick sensitivity, but the lack of a functioning deadzone; since it appears that these values are from stationary joysticks.
I'm not in favour of imposing Godot specified limits on the sensitivity of the joystick. What I think we really need is user configurable project settings with Godot defaults for both the joystick sensitivity and the joystick deadzone.


Update a shopcart

As a User
I need a function to update a shopcart
So that I can update the shopcart resource in one go
Details and Assumptions

We take a JSON input and update all fields at once?

Acceptance Criteria
Given the shopcart exists
When `PUT /shopcarts/{shopcartID}`
Then the shopcart will be update with the provided content

I found deserialize() in Item currently not using id from data. So when I use that in my tests, something like this keeps showing up:
- It should update a shopcart with JSON content (FAILED)

======================================================================
FAIL: It should update a shopcart with JSON content
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/app/tests/test_routes.py", line 195, in test_update_shopcart
    self.assertDictEqual(resp.get_json(), shopcart.serialize())
AssertionError: {'customer_id': 68, 'id': 1, 'items': [{'co[185 chars] 1}]} != {'id': 1, 'customer_id': 68, 'items': [{'id[185 chars]w'}]}
  {'customer_id': 68,
   'id': 1,
   'items': [{'color': 'green',
-             'id': 1,
?                   ^

+             'id': 7,
?                   ^

              'name': 'mouse',
              'price': 199999.2,
              'quantity': 1,
              'shopcart_id': 1},
             {'color': 'yellow',
-             'id': 2,
?                   ^

+             'id': 8,
?                   ^

              'name': 'pc',
              'price': 199999.2,
              'quantity': 5,
              'shopcart_id': 1}]}
-------------------- >> begin captured logging << --------------------

So basically the only diffs are ids.
I add self.id = data["id"] to deserialize() and it seems working without breaking other codes. I'll include that in my PR.


[2.2.0] Can't bind to 'ngClass' since it isn't a known property of 'div'

I'm submitting a ...
[x ] bug report
[ ] feature request
[ ] question about the decisions made in the repository

Do you want to request a feature or report a bug?
A bug report with the 2.2.0 just updated. These errors didn't occur with the 2.1.0.

What is the current behavior?
These errors are displayed: (not whole but part of the errors.)

Can't bind to 'ngClass' since it isn't a known property of 'div'. ("

][ngClass]="[toast.type, toast.theme]">

            <div *ngIf="t"): ToastComponent@2:17
Property binding ngIf not used by any directive on an embedded template. Make sure that the property name is spelled correctly and all directives are listed in the<EMAIL_ADDRESS>("

            [ERROR ->]

            <div *ng"): ToastComponent@2:12

I believe taking out CommonModule from index.ts by this commit causes the errors.
https://github.com/akserg/ng2-toasty/commit/7d7c0a837991f5bf7e2e448944d9e285d20725f1

If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem via https://plnkr.co or similar.

What is the expected behavior?
No error should be displayed to console.

What is the motivation / use case for changing the behavior?

Please tell us about your environment:

Angular version: 2.4.1
Angular CLI: 1.0.0-beta.24
ng2-toasty: 2.2.0

Other information (e.g. detailed explanation, stacktraces, related issues, suggestions how to fix, links for us to have context, eg. stackoverflow, gitter, etc)

I have the same issue - fixing.

Fixed in 2.2.1
Please confirm.

It got fixed. Thanks a lot!

I'm still busy testing the latest build, so please, if you find other issues - let me know.


[Event Request] COD 80 / 90 - OnBeforePostItemTrackingErrors

COD 80 - PostItemTracking - We would like to skip the errors in case RemQtyToBeInvoiced is higher than the Qty to receive / Qty to ship.

[ ] Solved in BC14

Therefore, can you please move the code below to 2 new procedures:

'ThrowInvoiceGreaterThanReturnReceiptErr' with a OnBefore... publisher with access to SalesLine and include the Handled pattern to skip the error
'ThrowQuantityToInvoiceGreaterErr' with a OnBefore... publisher with access to SalesLine and include the Handled pattern to skip the error.

COD 80 - PostItemTrackingForShipment - We would like to skip the ShipmentInvoiceErr error.

[ ] Solved in BC14

Therefore, can you please move the code below to a new procedure:

'ShipmentInvoiceErr' with a OnBefore... publisher with access to SalesLine and include the Handled pattern to skip the error.

COD 90- PostItemTracking - We would like to skip the errors in case RemQtyToBeInvoiced is higher than the Qty to ship / Qty to receive.

[ ] Solved in BC14

Therefore, can you please move the code below to 2 new procedures:

'ThrowInvoiceGreaterThanReturnShipmentErr' with a OnBefore... publisher with access to PurchLine and include the Handled pattern to skip the error
'ThrowQuantityToInvoiceGreaterErr' with a OnBefore... publisher with access to PurchLine and include the Handled pattern to skip the error.

COD 90 - PostItemTrackingForReceipt - We would like to skip the ReceiptInvoicedErr error.

[ ] Solved in BC14

Therefore, can you please move the code below to a new procedure:

'ReceiptInvoicedErr' with a OnBefore... publisher with access to PurchLine and include the Handled pattern to skip the error.

COD 90 - PostItemTrackingForShipment - We would like to skip the ReturnShipmentInvoicedErrerror.

[ ] Solved in BC14

Therefore, can you please move the code below to a new procedure:

'ReturnShipmentInvoicedErr' with a OnBefore... publisher with access to PurchLine and include the Handled pattern to skip the error.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


fixes icon import issue and bumps version

📝 Changes
Issue
currently icons cant be imported from @easypost/easy-ui-icons https://www.npmjs.com/package/@easypost/easy-ui-icons?activeTab=code . we've identified that it might be caused by specifying the files prop in package.json in the easy-ui-icons project. unfortunately, i was unable to test this scenario using npm pack/publish with --dry-run and it seemed unwise to preform trial and error and bump versions. i created a roughly equivalent scenario to test by forking easy-ui:

new version is live


from instance seg to semantic seg

How do I migrate you model from an instance segmentation task to a semantic segmentation task?

I am interested in migrating this project to semantic segmentation too. Do you have any progress?

You can try change the decoder head


refactor!: remove task specific data

Description

Remove task specific data.

TODO:

[x] migration to drop data column
[x] set logsPermissions
[x] adapt e2e tests

How has this been tested?

Checklist

[x] changelog was updated with notable changes
[ ] documentation was updated

Please run the e2e tests for significant changes
/e2e

End to end tests: :hourglass_flowing_sand: STARTING

Please note that the migration in this PR conflicts with https://github.com/Substra/orchestrator/pull/48

Companion PRs that need this branch to work, as they use the /task endpoint and do not fill the extra Data fields (setting category as unknown for now in the backend until https://github.com/Substra/orchestrator/pull/48 is merged):
https://github.com/Substra/substra/pull/299
https://github.com/Substra/substra-backend/pull/513
https://github.com/Substra/substrafl/pull/31
https://github.com/Substra/substra-tests/pull/221
https://github.com/Substra/substra-documentation/pull/197

End to end tests: :hourglass_flowing_sand: STARTING


#50 : k8s acceptance test - increase wait time for pv and pvc

(to avoid timeout due to k8s slow scheduling)
because sometime the acceptance test failed because 10 seconds not enough for k8s to provision PVC. So i would like to avoid false positive so i increasing the wait-time from 10 to 30 seconds

This change is

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


recent discord update broke the theme

only the colours work, and only sometimes. all of the layout changes are broken because discord re-rolled their css classes (to my understanding)

Wow...
FWIW, DNOME is wildcard targeting classes, for example, if the sidebar class is sidebar-abc123, we are targetting [class*="sidebar-"], allowing DNOME to work every time they compile their frontend and the suffix changes. But this time they changed the dashes on some classes turning sidebar-abc123 to sidebar__abc123. So now I have to go back and check every single wildcard in DNOME to see which ones changed :(
A quick workaround is to replace all instances of -"] to _"] in your compiled DNOME, but that might either not target everything or target more than it should

I did a sweeping change of - to _ on my end and so far have not encountered any issues so hopefully it should be straightforward in the end.


Add lookup tables to database definitions

Currently nothing in day_lookup or agent_hours, they need to be populated on proprietary values

If I run all the database sync jobs at all the same time, the redundancy of having the same definitions may be compensated for because it is substantially less hassle. On the other hand, you do not want diverged to happen.