Check accuracy of mapped tech desc for big utilities

hey @aesharpe is this one still relevant? Now that we are moving towards using EIA as the source of the tech description?

hey @aesharpe is this one still relevant? Now that we are moving towards using EIA as the source of the tech description?

We can close this


Prettier to format code on demand

Thanks for the great tool. I user Carbon regularly and am a super fan of prettier as well. It would be nice to be able to integrate them together.
I've been tweaking around with prettier codebase lately and also got a demo running for a prettier rest api server and also a static version of prettier standalone UMD packages.
I've been so excited about this that I started to add it to carbon locally :)
This is just a basic demo from what I've got so far:

Since prettier has a finite list of supported languages and technologies, I've been thinking about integrating dynamic parsers of prettier with the language that highlight.js detects from the editor.
Also prettier configurations might be good to be added to the settings dropdown menu.
I'd be happy to send a WIP PR if this looks good enough 👍

@farskid this looks cool, however I'm not sure we want to start implementing all of these editor features within Carbon. Yes Prettier is pretty easy to maintain, but we are not trying to make Carbon into another CodeSandbox (@compuives has you covered there 🙂) and could create a slippery slope.

Totally fair.
My use case on a daily basis is such I use carbon a lot! and non pretty indented code pasted in carbon editor bugs me sometimes. I used to use my local projects or code sandbox for prettifying the code sample, but having in inside the carbon sounds like an enormous win 👍
I'd be more than happy to help with the integration if you want this to happen sometimes in future :)

@farskid after talking some more with @mfix22, we think you make a good case for this. I'll defer to @jakedex as far as what the interface should look like, but go ahead and submit a WIP PR if you like 😃

@briandennis Super Cool! I'm on it. will send a WIP very soon. Thanks for the feature people :)


API add ability to filter deployments by application name

What this PR does / why we need it:
Currently, we only have a way to filter deployments by Env, Kind, Status, and ApplicationId (display as application name on UI) which is hard to use in case there are many applications with the same name. This change adds ApplicationName as a filter, the change for the client will be addressed by another PR.
Which issue(s) this PR fixes:
Part of #2448
Does this PR introduce a user-facing change?:

NONE

/kapentanios reload

Honestly I'm not sure why this change solves that issue. I think the root cause is that it contains disabled applications as filter options.

That makes sense. Now I get it.
/lgtm

/approve

Honestly I'm not sure why this change solves that issue. I think the root cause is that it contains disabled applications as filter options.

Has your question been answered yet?

It has 👍

It has 👍

nice 🤝


dynamic linking

Currently the tpm2.0-tools build clones the TPM2.0-TSS sources directly, compiles them, and then statically links against a now missing library. The better long term solution to decouple these builds is to have the tpm2.0-tools treat the TPM2.0-TSS libraries and headers like any others on the build system.
This is the ideal solution to #1. Likely we'll do a work around first and then get to this.

I broke this out into two tickets, one for SAPI #7 and one for the TCTI #6


Quarantine test generates events for downloading artifacts

@bot-gradle test and merge please

OK, I've already triggered a build for you.


Building docker-compose for dev environment

Pour la review de cette PR, je vous suggère de suivre le README
Je n'ai pas changé les commandes de création/suppression de site, juste ce qu'il y a autour 👍

Codecov Report

Merging #5 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master       #5   +/-   ##
=======================================
  Coverage   85.71%   85.71%
=======================================
  Files           3        3
  Lines          14       14
=======================================
  Hits           12       12
  Misses          2        2

Continue to review full report at Codecov.

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

Updates to follows with following issues:

https://github.com/camptocamp/epfl-os-wp/issues/13 : to have python3 in container
https://github.com/camptocamp/epfl-os-wp/issues/14: to have virtualenv
https://github.com/camptocamp/epfl-os-wp/issues/15: to enter the local container in same condition as in C2C (home dir and .bashed sourced)

Manu on supprime la branche ?

Manu on supprime la branche ?

yes! done 👍


proposal: use web workers for some kiln services

would this noticeably speed up api calls? and component re-rendering?

The synchronization makes this harder than necessary


Skip building HotROD for all platforms for pull requests

Which problem is this PR solving?

Resolves #5743

Description of the changes

Similar to the functionality we have for all-in-one image I tried to achieve same build configuration supported by corresponding script file.

Checklist

[x] I have read https://github.com/jaegertracing/jaeger/blob/master/CONTRIBUTING_GUIDELINES.md
[x] I have signed all commits
[ ] I have added unit tests for the new functionality
[x] I have run lint and test steps successfully

for jaeger: make lint test
for jaeger-ui: yarn lint and yarn test

@yurishkuro I am bit confused about adding the integration test for local build image and the support for debug image using add_debugger flag as we have in all-in-one-build script.

lgtm, one small change needed

Done. Thanks for the review 😄

@yurishkuro I have made changes for some failing checks too that I witnessed from the previous commit.

Thanks!


Game crashed in 1.16.4

Minecraft version: 1.16.4
Fabric Loader version: 0.10.8
Fabric API version: 0.26.3
GeckoLib version: 3.0.0+beta.7
Desolation version: <IP_ADDRESS>-beta
Here's the crash text:
crash-2020-11-25_17.28.40-client.txt

Although this may be bothering you, I sincerely hope you can update it to be compatible with the latest version of fabric API. I like the idea of this mod very much!

Hey, it looks like the issue is being caused by Geckolib - the mod uses version 2.0.0, not 3.0.0. I just tested it with the versions you listed and it works fine with Geckolib 2.0.0. Try downgrading and let me know if it still crashes!

aha, it works! Thank you!


Unhandled exception in circuit after migrating to 3.0 preview 9

After migrating to Asp.net core 3.0 preview 9 I started getting this exception when starting my Blazor server app:
16:26:57 ERR] Unhandled exception in circuit 'ntIvJ1Q9N86qaaRbnuCYqrSio__1WUzSsTQEcDUJw2E'.
System.AggregateException: One or more errors occurred. (InvalidCharacterError: Failed to execute 'setAttribute' on 'Element': '@onclick' is not a valid attribute name.)
 ---> System.InvalidOperationException: InvalidCharacterError: Failed to execute 'setAttribute' on 'Element': '@onclick' is not a valid attribute name.
   at Microsoft.AspNetCore.Components.RenderTree.Renderer.InvokeRenderCompletedCallsAfterUpdateDisplayTask(Task updateDisplayTask, Int32[] updatedComponents)
   --- End of inner exception stack trace ---

I could finally find out that it works again if I remove the second @onclick attribute from the NavMenu.razor.
Broken version:

    TestApp
    <button class="navbar-toggler" @onclick="ToggleNavMenu">

    </button>

Working version:

    TestApp
    <button class="navbar-toggler" @onclick="ToggleNavMenu">

    </button>

What's strange is that if I create a new app with dotnet new blazorserver, the template contains the second @onclick attribute and doesn't throw the exception!
In my app I'm importing some packages (e.g.: Blazored.Modal).
Can someone explain the reason for this weird behavior?

Thanks for contacting us, @fleed .
Please refer to the release notes in here showing you the migration steps to Preview9, which includes multiple preview breaking changes.
In this particular case, you're missing a using for Microsoft.AspNetCore.Components.Web: https://devblogs.microsoft.com/aspnet/asp-net-core-and-blazor-updates-in-net-core-3-0-preview-9/#blazor-web-attributes-moved

A bit of help for others which have done the upgrade like @mkArtakMSFT as suggested but still finding the problem.
I had a .razor component in a shared assembly.
The shared assembly compiled fine, but as there is no _Imports.razor I had to manually add the @using Microsoft.AspNetCore.Components.Web to the top of the razor component in the shared library.


In .NET Core, libMan install 'Failed to download resource'

Hello,
I am working with .NET core in Ubuntu 18.04 behind a proxy server. When I try to install a library using 'libMan' utility (version 1.0.172+g2a33cb3eeb) got ''Failed to download resource' error message.
Tried command: libman install @aspnet/signalr -p unpkg -d wwwroot/lib/signalr --files dist/browser/signalr.js --files dist/browser/signalr.min.js
Error message: Microsoft.Web.LibraryManager.Contracts.ResourceDownloadException: Failed to download resource from "https://api.npms.io/v2/search?q=signalr+scope:aspnet" at Microsoft.Web.LibraryManager.WebRequestHandler.GetStreamAsync(String url, CancellationToken cancellationToken) at Microsoft.Web.LibraryManager.Extensions.GetJsonObjectViaGetAsync(IWebRequestHandler webRequestHandler, String url, CancellationToken cancellationToken) at Microsoft.Web.LibraryManager.Providers.Unpkg.NpmPackageSearch.GetPackageNamesWithScopeAsync(String searchTerm, CancellationToken cancellationToken) at Microsoft.Web.LibraryManager.Providers.Unpkg.NpmPackageSearch.GetPackageNamesAsync(String searchTerm, CancellationToken cancellationToken) at Microsoft.Web.LibraryManager.Providers.Unpkg.UnpkgCatalog.SearchAsync(String term, Int32 maxHits, CancellationToken cancellationToken) [LIB002]: The "@aspnet/signalr" library could not be resolved by the "unpkg" provider
Apparently present version of libman utility does not offer 'config' command for configuring proxy settings. Does anyone know a workaround to this proxy issue?

It has to be supported – see pull request #474.

The libman version 1.0.172+g2a33cb3eeb does not have  'config'. Here is output of libman --help
1.0.172+g2a33cb3eeb
Usage: libman [options] [command]
Options:
--help|-h  Show help information
--version  Show version information
Commands:
cache      List or clean libman cache contents
clean      Deletes all library files defined in libman.json from the project
init       Create a new libman.json
install    Add a library definition to the libman.json file, and download the library to the specified location
restore    Downloads all files from provider and saves them to specified destination
uninstall  Deletes all files for the specified library from their specified destination, then removes the specified library definition from libman.json
update     Updates the specified library
Use "libman [command] --help" for more information about a command.

@jimmylewis, I see the same message after updating (and even uninstall\install) the libman tool.
I use the dotnet tool install --global Microsoft.Web.LibraryManager.Cli command to install the tool into the C:\Users\username\.dotnet\tools directory, so maybe it (the dotnet tool) uses another, not updated source...

I'm working on getting the updated package pushed to NuGet, which will contain the new libman config command.  I believe the new package version will be 2.0.48 once it's up.
I'm closing this issue for now as it's a dupe of #399.  I'll resolve that issue when the new package is available.

@jimmylewis thanks and looking forward to find my way out of this bottle-neck with the new package.

@chjatala, you can try to apply my workaround solution as described in https://github.com/aspnet/LibraryManager/issues/399#issuecomment-472760343.

@chjatala, the new version is available so you can update the libman (using the dotnet tool update --global Microsoft.Web.LibraryManager.Cli) command and try to use it as described in the #399. In my case, proxy login is 'domain\username'.


Track DeepL API usage

Motivation
We have an automatic translation feature via DeepL. However, we have to pay for translations. We do want to keep track of our used quota.
Proposed Solution
Query our used words quota before and after sending translation requests and store the difference in our database (InfluxDB?)
Alternatives
The total amount of words translated could be stored in an attribute for the region.

@svenseeberg I'm a bit unsure why we'd want to store the used quota. Wouldn't it make more sense to just query the DeepL API (see Ulli's link) for our usage any time it's relevant (e.g. before sending translation request, in the planned translation budget widget, ...)? That way it's always up to date.
So for now I'd add a function to check the usage, and raise an exception in the deepl_translation  function if the translation would exceed the limit.

Wouldn't it make more sense to just query the DeepL API (see Ulli's link) for our usage any time it's relevant (e.g. before sending translation request, in the planned translation budget widget, ...)? That way it's always up to date.

I think DeepL only tracks our global credit count, but we want to keep track of which portion of that is used by which region... this is not only done to prevent errors during the automatic translation, but also for our business calculation to see whether the usage roughly matches the price we charge the regions for this feature. This way, we could identify e.g. "power-user" regions which use the feature above average and maybe need an individual rate or whatever...

So for now I'd add a function to check the usage, and raise an exception in the deepl_translation function if the translation would exceed the limit.

Not sure whether that's required - can we just "try" to translate and then handle the error thrown by DeepL when the quota is reached? I'm also not sure whether we're able to calculate the limit in exactly the same way as DeepL... what about special HTML characters, white spaces etc., are they billed exactly like translatable text?

from what I can see in the wiki, the plan is to offer a budget of 50.000 words. However, as far as I can tell, DeepL tracks usage by character. Is this an oversight, or are we actually tracking and displaying the translation usage of each community by word, regardless of word lengths/character counts?

I'm not 100% sure, but yes, how I understood it this is intentional, so we're offering the regions a different payment plan than how we pay DeepL. We somehow have to calculate internally that the external per-character costs for DeepL + our support hours + our developer hours are covered by the per-word fees by the regions. Maybe @dkehne knows more...

But yes good point, this kind of means that we need to keep track of two separate counters? The character count by DeepL which we need for our internal calculations and the word count which we need to show the regions their limit?
Then your suggestion earlier to check the limit beforehand also makes absolutely sense, because the DeepL API won't throw an exception if there is enough global credits left, but an individual region has exceeded the limit we gave them? :thinking:

Yeah, if this is how it's supposed to be tracked, then I think we have to check the word usage beforehand, and then update it in the DB after successfull translation.

@charludo @timoludwig maybe to clarify the word/character counting mechanism. We are aware that DeepL counts characters but bcs the translation offices charge per word and all our translation reports are word based, we decided to go with words for our DeepL package as well. We took the average of 7 characters per word which is why we are buying 7 Mio character from DeepL per Region and giving them 1 Mio words.
What we need to be able to track is how many words each region has used/uses, bcs 50.000 words are free of charge and after that the Kommunen need to sign a contract to get 1 Mio words for 1000€.
I am not sure whether it is actually necessary to track the character usage in DeepL?
But what we need to be able to do is tell a Kommune that the amount of words they are trying to translate exceeds their budget of words (if that is ever the case) so I think we need to track usage/know usage before sending it for translation, right?

I am not sure whether it is actually necessary to track the character usage in DeepL?

If you're not interested in knowing whether your assumption of 7 characters per word is correct, then not. I just have a few small concerns that our own calculation might be completely off and we need to adjust our business calculation in the future.
E.g., consider the following page fragment:
word

Then, this would be counted as two "words" (word) of length 2 and 138, so the average char count per word would be off by the factor of 10.
Maybe we can improve this method during PR review, but the underlying problem will stay: I's always hard to calculate the number of words in HTML source code and we don't exactly know how much DeepL charges for sending special characters to the API which are not translated (maybe this has to be tested), so in this case I kind of expect our calculations to be wrong no matter what and it could be useful to track the character count from DeepL so e.g. after half a year, we can say "ok, X translated words were billed by DeepL with Y characters, maybe the value Z would be a better average word count than what we thought initially.

But what we need to be able to do is tell a Kommune that the amount of words they are trying to translate exceeds their budget of words (if that is ever the case) so I think we need to track usage/know usage before sending it for translation, right?

Yes, this is exactly what @charludo implemented in #1909 I think :+1:

@timoludwig yes, that's basically what I implemented, though I now think there's some potential for improvement, e.g. storing the translated character count as well, like you said.
I'm not sure of we should calculate the character count ourselves (potentially error-prone compared to DeepL's calculation), or query DeepL (also potentially error-prone if another region happens to translate at the same time).
So maybe storing the total amount of characters/words over all regions makes the most sense?

I just have a few small concerns that our own calculation might be completely off and we need to adjust our business calculation in the future.

True, I always forget that we are sending HTML code - so maybe it would be interesting to know, also to know whether we are right in our assumption of 7 characters per word or not. If we can test how DeepL behaves with HTML code, even better!

So maybe storing the total amount of characters/words over all regions makes the most sense?

But then we would not be able to determine how many words each region used, right?

query DeepL (also potentially error-prone if another region happens to translate at the same time).

Theoretically we could use a simple threading.Lock() for this, shouldn't be too much effort.

So maybe storing the total amount of characters/words over all regions makes the most sense?

But yes, this is the simpler version. I'm just not sure whether we even need to store this - then we can probably just query the usage after a specific point in time and just sum all consumed words of all regions to calculate the global average?

But then we would not be able to determine how many words each region used, right?

@osmers No, I think the word counter for each region is not affected by the new field where we track the global character count from DeepL.

we don't exactly know how much DeepL charges for sending special characters to the API which are not translated (maybe this has to be tested)

I just did some quick tests. The following event description:

Das ist ein Test, um zu sehen, ob DeepL HTML Zeichen berechnet.

Beschreibung des Links

Und hier noch ein Bild:

has a length of 277 characters excluding spaces, and that is exactly the amount of credits DeepL consumes for the translation (I already accounted for the event title), even though far less characters actually get translated:

This is a test to see if DeepL calculates HTML characters.

Description of the link

.

And here's another picture:

As you can see, DeepL is context-aware enough to translate the alt text of an image, but leave e.g. the link alone, but the characters that make up the link still get counted.
According to TinyMCE, the original snippet contains 20 words, meaning in this (not representative) example, the average word length is just under 14 characters.

Two other "weird" things I noticed:

I don't know why the . is added between the last two
 tags. Adding a . after Beschreibung des Links to mark the end of the "sentence" doesn't stop this.
Words with Umlaute are not translated, at all, but I think that's due to us sending HTML-espaced characters instead of the actual Umlaut. @timoludwig should I open a new issue for this or fix it in my current PR?

By the way, the character count for actual words, including the img alt text, is 141, so pretty much exactly 7 characters/word.
As I said, I don't think my example is representative, since most pages contain a lot more translatable text than non-translatable links, images, and so on; but add in a couple of links that look like the one @timoludwig mentioned, and we could be looking at significantly more than 7 characters per "word".
@osmers @timoludwig I know HTML parsing isn't exactly easy, but do you think the discrepancy here is large enough to justify implementing some kind of (e.g.) beautifulsoup solution to gather all the actual text ourselves, then only send those words off to DeepL and but the result back into the original HTML structure?
I can see a very real (monetary) benfit in this, but also some danger in fragmenting text and grammatical structure in e.g. inline-links.

Ok, I was afraid of that, but it kind of makes sense and at least makes it easy for us to know exactly how much credits a translation consumes.

Does it make sense? They are charging for characters they are intentionally discarding.

since this is an independent problem, please open another issue for it, but feel free to fix it in your PR as well smile

👍🏼

This is pretty much what I did for SUMM.AI since their API does not support HTML yet. For SUMM.AI, we just accepted the fact that inline formatting is gone, but I guess we could also improve this solution to work with smaller placeholders which can be re-inserted after the translation is complete?
Definitely not without trip hazards, but I think it's worth considering this possibility.

Neat, I didn't know that!
Fortunately, I think I just stumbled upon an easier solution: Their API offers the option to directly translate a number of document formats, including HTML, and using that method to translate the same HTML snippet as before, only 110 character credits are used! 🥳
The discrepancy to the 141 from before is most likely the now not-translated img alt tag, though it isn't adding up exactly, I'm not sure why.
Also I just tried a translation the normal way and simply put a single German word as the href in a link, and this actually does get translated, which is far from ideal.
I think the most sensible way forward is to either:

write the description/content of a page/poi/event to a tempfile, send it to DeepL, the python DeepL library automatically writes the result to a new file. Read it, delete both. This is probably the more simple solution, but I'm not crazy about constantly creating tempfiles. (Though we could probably just wrap the text in StringIO? Not 100% sure on this.)
ditch the DeepL python library for this one type of translation, post the request manually, get the result without ever storing it in a file. If DeepL significantly changes their API, we will have to manually adjust this, though I'm not sure that is entirely likely.

AFAIK image alt tags are the only thing that won't be translated with this solution. Maybe that is acceptable?

write the description/content of a page/poi/event to a tempfile, send it to DeepL, the python DeepL library automatically writes the result to a new file. Read it, delete both. This is probably the more simple solution, but I'm not crazy about constantly creating tempfiles. (Though we could probably just wrap the text in StringIO? Not 100% sure on this.)

As I understand the Readme of the Python client, the translate_document() works with IO objects and does not write the files to disk?
Also, there is this tag_handling="html" option which can be passed to the translate_text() function, maybe also worth trying out? (seems to be in beta though)
But both opinions definitely sound better than doing the html parsing ourselves 😅

the translate_document() works with IO objects and does not write the files to disk

Does it not? For that example, an input and output file are opened before calling translate_document, but it looks like that function then handles reading and writing? Anyways, doesn't really matter, I'll just try it out.

Also, there is this tag_handling="html" option which can be passed to the translate_text() function, maybe also worth trying out? (seems to be in beta though)

Once again, reading the entire docs seems like it would have been a good idea... 😅

Does it not? For that example, an input and output file are opened before calling translate_document, but it looks like that function then handles reading and writing?

Exactly, in the example the files from the disk are converted to IO objects via with open(input_path, "rb") as in_file, but we can just skip this step and directly create in-memory IO objects.

Once again, reading the entire docs seems like it would have been a good idea... 😅

Well, yeah, don't ask me why I didn't notice this in the review of the initial implementation 🙈

AFAIK image alt tags are the only thing that won't be translated with this solution. Maybe that is acceptable @osmers?

I'm not even sure they get translated right now with MemoQ and DeepL translations - I'd have to check. Also, most Kommunen don't use that many pictures which is why it should be ok. Would it be something that can be changed in case there are a lot of complaints?


Not open source anymore?

The code in this repository contains version 36 of the extension while the version published on the Chrome Web Store right now is version 44. There are lots of differences between the version here and the one downloaded from the store.
Is this extension still open source then? If so, update the code here please. If not, please state it explicitly here and on the store.

Now is version 46 and this repo seems unattended for quite a long time.


Game.Create width when using a square font

I've gone through "How a Font works" on the SadConsole website (https://sadconsole.com/articles/basic-font-information.html) and I have it working with a square font wanted to try.  The font is a 16 x 16 pixel square font.
To test the font,  I'm creating a 16 x 16 console and populating it with an array of Cells with the Glyph property set to incrementing values from 0 - 255.
using System;
using SadConsole;
using Microsoft.Xna.Framework;
using Console = SadConsole.Console;

namespace sadconsolefonttest
{
    class Program
    {
        static void Main()
        {
            // Setup the engine and create the main window.
            SadConsole.Game.Create(16, 16); // <--- **** I need to change this to (32, 16) to work correctly ****

            // Hook the start event so we can add consoles to the system.
            SadConsole.Game.OnInitialize = Init;

            // Start the game.
            SadConsole.Game.Instance.Run();
            SadConsole.Game.Instance.Dispose();
        }

        static void Init()
        {
            Cell[] map = new Cell[256];
            for (int gly = 0; gly < 256; gly++)
            {
                map[gly] = new Cell(Color.White, Color.Gray, gly);
            }

            var fontMaster = SadConsole.Global.LoadFont(@"Font\16x16_sm_ascii.font");
            var normalFont = fontMaster.GetFont(SadConsole.Font.FontSizes.One);
            var bigFont = fontMaster.GetFont(SadConsole.Font.FontSizes.Two);
            var console = new Console(16, 16, normalFont, map);
            // var console = new Console(16, 16, Global.FontDefault, map);
            // var console = new Console(16, 16);
            // console.FillWithRandomGarbage();

            SadConsole.Global.CurrentScreen = console;
        }
    }
}

When I run the program with both the Game.Create width and the new Console width set to 16, then I only get a 8 character wide console.

I have to bump up the Game Create width to 32 while keeping the new Console width at 16 in order for it to display the full 16 character wide window.

This is the content of the .font file:
{
  "Name": "16x16_sm_ascii",
  "FilePath": "16x16_sm_ascii.png",
  "GlyphHeight": 16,
  "GlyphPadding": 0,
  "GlyphWidth": 16,
  "SolidGlyphIndex": 219,
  "Columns": 16,
  "IsSadExtended": false
}

And here is a link to the png file for the font.
https://1drv.ms/u/s!AhHT7_5wGG0VkskyQ_OYKiRMLvCHtw?e=DOrbDb
Here's a screenshot I took of it. For some reason, when I try to embed the png image here, it shows as all white or all transparent so you can't see it.

Hello! The create width/height is how many cells the window will show based on the default font size, which is 8x16 pixels. When you change the default font in Init, it's window size was already determined, based on default font. The Create method actually has a parameter for you to pass in "Font\16x16_sm_ascii.font" as the new "default" font and then sizes the window according to that.
SadConsole.Game.Create("Font\16x16_sm_ascii.font", 16, 16);

That should fix your problem. That also sets the SadConsole.Global.FontDefault to your provided font and all consoles created that don't specify a font will use that font.
Let me know if this fixes your problem! Feel free to join us on discord too!

Okay, that makes sense.  Thanks,
Dave


add PB-MTL evaluation?

pball_evaluation.csv has 145 lakes (likely the original source lakes tested on themselves), but should we also have the PB transfer learning results for the 305 (or more) lakes?

yes, those will be coming in hopefully today. The tricky part with the PB-MTLs is that they have to extend the depths to match the target lake, which is a new processing step I haven't yet added here. But soon!


Add helpers for 'visualising-xmaps'

Function Family
Exported:

[ ] plot() generic
[ ] viz_as_matrix() or xmap_viz_matrix()
[x] viz_as_bigraph() or xmap_viz_bigraph()

Internal/Helpers (for converting implicit zero links)

[ ] complete_bigraph(), connect_zeroes(), fill_links()
[ ] simple_bigraph(), prune_links()

Design Notes

interface should take an xmap as arguments xmap_viz(x)
variants: xmap_ggplot_graph(), xmap_ggplot_matrix(); or xmap_viz_as_graph()
ggbump is available on CRAN, and has MIT license
matrix visualisation requires explicitly encoding the "zero-weight" links (i.e. the missing links)
data wrangling should happen outside of the plotting function, though it might be sensible to provide convenience wrapper functions
facets based on OUTGOING and/or INCOMING link types:

outgoing: one-to-one, and one-to-many relations
incoming: one-from-one and one-from-many

Would like to plot an xmap as:

bipartite graph between from_set and to_set
matrix mapping from_set rows, to_set columns

Implementation notes (based on viz-panel-maps attempts)

identify must have channels/aes arguments
avoid wrapping theme/palette code into functions
enable passthrough of arugments?
aim for composable plots (multi-step graphs)?
shared colour palette between representations
pull example data from conformr

Customisation

[ ] theme_xmap() for modified minimal background bigraph

Matrix plot
library(tidyverse)
library(ggplot)
library(ggbump)

edges <- tibble::tribble(~key1, ~key2, ~share,
                 "BLX", "BEL", 0.5,
                 "BLX", "LUX", 0.5,
                 "E.GER", "DEU", 1,
                 "W.GER", "DEU", 1,
                 "AUS", "AUS", 1)

Make missing links explicit:
inc_tbl <- expand_grid(key1 = edges$key1, key2 = edges$key2) |>
	left_join(edges, by = c("key1", "key2"))

Add cases to facilitate colour of matrix cells, and plot matrix.
plt_inc_long_mtx <- function(inc_long, from, to, weights) {
  gg <- inc_long |>
    dplyr::mutate(src_case = dplyr::case_when(
      {{weights}}==1 ~ "one-to-one",
      is.na({{weights}}) ~ "none",
      {{weights}} < 1 ~ "one-to-many")) |>
  ggplot(aes(x={{to}}, y={{from}})) +
    geom_tile(aes(fill=src_case), col="grey") +
    scale_y_discrete(limits=rev) +
    scale_x_discrete(position='top') +
    scale_fill_brewer() +
    coord_fixed()  +
    labs(x = element_blank(), y = element_blank(), fill="source-to-target") +
    theme_minimal()
  return(gg)
}

plt_inc_long_mtx(inc_tbl, key1, key2, share)

Alternative to retain control of fill aesthetic & labels:
plt_inc_long_mtx <- function(inc_long, from, to, weights, fill) {
  gg <- inc_long |>
  ggplot(aes(x={{to}}, y={{from}})) +
    geom_tile(aes(fill={{fill}}), col="grey") +
    scale_y_discrete(limits=rev) +
    scale_x_discrete(position='top') +
    scale_fill_brewer() +
    coord_fixed()  +
    labs(x = element_blank(), y = element_blank(), fill="source-to-target") +
    theme_minimal()
  return(gg)
}

inc_tbl |>
    dplyr::mutate(src_case = dplyr::case_when(
          share==1 ~ "one-to-one",
          is.na(share) ~ "none",
          share < 1 ~ "one-to-many")) |>
    plt_inc_long_mtx(from=key1,to=key2,weights=share,fill=src_case)

Wrapped matrix plotting function skeleton:
plt_xmap_as_mtx <- function(xmap, from, to, weights){
  ## check is xmap
  stopifnot(inherits(xmap, "xmap_df"))

  ## expand missing links
  inc_tbl <- xmap |>
    dplyr::select({{from}}, {{to}}) |>
    base::expand.grid(KEEP.OUT.ATTRS = FALSE, stringsAsFactors = FALSE) |>
    dplyr::left_join(xmap, by = c(deparse(substitute(from)), deparse(substitute(to))))

  ## add fill variable
  inc_long <- dplyr::mutate(...)

  ## plot
  gg <- plt_inc_long_mtx(inc_long, from, to, weights, fill)

  return(gg)
}

For more on ggplot internals:

https://emitanaka.org/datavis-adv-workshop/day1.html
https://ggplot2-book.org

How to handle labels?

This is way more complicated than I anticipated... Just collate usable code examples into #80 and think about designing flexible viz functions later.... maybe never.

For modifications:
https://stackoverflow.com/questions/13407236/remove-a-layer-from-a-ggplot2-chart

Maybe use layouts from: https://cran.r-project.org/web/packages/ggraph/index.html

Instead of a fully wrapped plotting function; could I start by offering conversion from an xmap_df to ggplot inputs e.g.:
xmap_to_ggplot_df() with some option for "completing" the graph -- with NA or 0 links. -- maybe .complete = 0

Existing packages:

https://nbarrowman.github.io/vtree for nested subsets

Attempt at defining new stat... didn't go very well. I don't understand that well how StatEdgeDiagonal works
StatFracWeight <- ggplot2::ggproto(`_class` = "StatFracWeight",
                          `inherit` = ggraph:::StatEdgeDiagonal,
                          compute_group = function(data, scales){
                            edge_linetype <- ifelse(data$weights < 1, "dashed", "solid")
                            edge_label_pos <- ifelse(data$weights < 1, 0.8, 0.2)
                            data.frame(x = data$x, y = data$y,
                                       xend = data$xend, yend = data$yend,
                                       circular = data$circular,
                                       edge_linetype = edge_linetype,
                                       edge_label_pos = edge_label_pos)
                          },
                          required_aes = c("x", "y", "xend", "yend", "circular")


bm3d wrapper changing colour range from limited to full.

using bm3d wrapper changes the colour range from limited to full.
Example code- denoise=vsdenoise.BM3DCPU.denoise(clip,1,1,profile=Profile.LOW_COMPLEXITY)
before

after

Additional information gleaned from the Discord conversation here:

The user passes a Limited Range, 16 bit clip to BM3DCPU.
BM3DCPU returns a Full Range 16 bit clip.

https://github.com/Jaded-Encoding-Thaumaturgy/vs-tools/commit/29d487b2abbb5aadc29610ac81bc2025e587f296
it seems like this commit is actually what breaks it. idk why though.


Help!

Not a issue
Any docs or
support to deploy on online platforms like heroku, vercel, others ❤

there are instructions to setup locally: Setup
there is no possibility to deploy on heroku & other platforms as the project is currently in beta. read: https://github.com/gaowanliang/DownloadBot/issues/29#issuecomment-850237456


AttributeError: 'str' object has no attribute 'decode'

Continued from https://github.com/kennethreitz/pipenv/issues/541#issuecomment-329518287

This issue is likely to show up on libraries that need to support both Python 2 and 3 where pipenv is being used to manage tooling. The feature:

Tell pip we're using the required Python version, with trickery, for dependency resolution.

is keyed off python_version = "2.7", but to test that this project works with multiple versions of Python, my build matrix also runs with version of Python 3.

Install Python 2
Install Python 3 and set this as your global Python version
Install pipenv 7.0.2
Set a required Python version of 2.7 in Pipfile
Run $ pipenv install --python=python2

@kennethreitz here are some build logs demonstrating the issue: https://travis-ci.org/jacebrowning/verchew/builds/275563258

if you have version_version set, there's no need to specify --python now.

you shouldn't be using python_version is sounds like, if you're testing against multiple pythons.

I didn't have python_version previously, but you said I needed to add it (https://github.com/kennethreitz/pipenv/issues/541#issuecomment-329512659) in order for pipenv to detect the Python 2 dependencies correctly.

that would be correct.

Why aren't you committing a pipfile.lock?

@kennethreitz Simply to show this exception occurring in build logs.

i think removing the pip requests session will fix this

that's what's adding the logging

can you try master?

@kennethreitz That seems to either hit the same error or NameError: name 'unicode' is not defined based on the Python version: https://travis-ci.org/jacebrowning/verchew/builds/275601507

@jacebrowning can you share you pipfile with us? I cannot reproduce this.

try updating travis' pip to latest, which is what we require anyway.

e.g. travis' pip is ancient. I'll cut a release depending on latest pip.

7.0.5 released, which depends on latest pip. hopefully that'll solve this issue.

The Pipfile is here: https://github.com/jacebrowning/verchew/blob/2099e17c5f8dee0319d4311915a4a86eca43397b/Pipfile
I just gave you push access if you want to create your own throwaway branch to use this project as a test bed for pipenv with multiple Python versions and version-specific dependencies.

thanks. this is impossible to reproduce locally.

@jacebrowning looks like 7.0.5 works!

i take that back.

So... I'm thinking that pipenv should refuse to run if all of the following are true:

Pipfile specifies a Python version
Pipfile.lock doesn't exist
python is a different version than what is specified in Pipfile

7.0.5 looks like it might be working, but he generated Pipfile.lock will only work with Python 2 if it was originally generated with Python 2 (for "fancy" dependencies).

I'm not what to do if new "fancy" dependencies are added later. Maybe it should always fail hard if python is different from the required version listed in Pipfile?

that sounds reasonable. We do issue a warning currently.

trying an alternative approach first.

yeah, cannot reproduce locally, even with your exact Pipfile.

well, your solution is to commit your lockfile.

got it working.

You're welcome.

v7.0.6 released, which fixes this strange edge case.


add instructions for testing to documentation

Before I push to a public repository I run tests. In particular I like to run the automatic tests which will be run be the continuous integration. For k-9 I check the most recent run by the CI and try to run the test locally before pushing my commits.
I had problems as I did not use the correct Java SDK. I tried JDK 11 and JDK 13 before I was successful with JDK 8. I could have saved some time if I had known what the proper test setup requirements are.
I suggest to document some system requirements for running the unit tests.
I orientated the documentation to fit the Android CI build on the most recent commit on master (2505a7b0cdeeda10e8b38e8c4ae21b4d963306f1).

Thanks.
The documentation describing the app design feels like the wrong place for this, though. I updated the build instructions in the wiki: https://github.com/k9mail/k-9/wiki/BuildingK9
Feel free to add to the instructions there if you find something else is missing.


ignoreBlur bug causing can't lose focus

hover on the last item of the menu, then type something that will shrink the menu.

i think when mouse leaves the menu without triggering the onMouseLeave callback it makes the component mistakenly think the mouse is still inside the input which cause the component always ignore blur.

Thanks for reporting this! I've started work on a fix, but sadly it will be very messy. I also notices that the current restore-scroll behaviour does not work as expected if Autocomplete is inside a scrolled container that's not <body>. My plan is to rethink how the focus management logic works, and hopefully that will negate the need for these dirty tricks :)

@CMTegner Have you given additional thoughts to this problem?
I am having a similar issue with my setup. I have a button in my autocompleter dropdown that hides the dropdown by toggling the open props.
This of course circumvent the onMouseLeave event, therefore never unsetting _ignoreBlur.
This causes the autocompleter field to hog focus forever because you can't onMouseLeave an element that is already hidden.

@vinhboy I'm afraid the entire logic needs a rethink. I hope to have time to address it soon, but I can't say for sure when this may be completed. If you can, try to work around the problem in your app so you don't get impeded by my ability to work on this issue.


Channel's menu keep visible even after click item

Issue
When opening "Connectors" from channel's (meatballs) menu, menu is still on top (visible) other content, and not closed by itself.
Expectation
After clicking item, the menu should be hidden (closed)
Version
Teams for Mac v<IP_ADDRESS>71

@enkhbatd - Could you please confirm if your issue has resolved or still looking for any help?


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_dbmsp1j6_x/clone/recipe/meta.yaml:31: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: /tmp/abs_dbmsp1j6_x/clone/recipe/meta.yaml:34: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
WARNING: /tmp/abs_dbmsp1j6_x/clone/recipe/meta.yaml:55: missing_description: The recipe is missing a description
Errors were found

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_22jz7unxtd/clone/recipe/meta.yaml:31: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: /tmp/abs_22jz7unxtd/clone/recipe/meta.yaml:34: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
WARNING: /tmp/abs_22jz7unxtd/clone/recipe/meta.yaml:55: missing_description: The recipe is missing a description
Errors were found

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_017s8ep4ot/clone/recipe/meta.yaml:31: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: /tmp/abs_017s8ep4ot/clone/recipe/meta.yaml:34: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
WARNING: /tmp/abs_017s8ep4ot/clone/recipe/meta.yaml:55: missing_description: The recipe is missing a description
Errors were found


Now you can define "id" for input

When you use the label tag as the wrapper for ModernDatepicker, you need the id attribute for the input tag to set the relationship between the elements
Now you can define the "id" attribute, for example:
    return (
      <div className="event-box__date">
        <label htmlFor="date">Date
          <ModernDatepicker
            id="date"
            date={this.state.startDate}
            format={'DD-MM-YYYY'}
            onChange={(date) => this.handleChange(date)}
          >
          </ModernDatepicker>
        </label>
      </div>
    );
  }```

@fit thanks for the PR, I have merged this PR and this feature is now available in v0.5.1


It Doesn't Give Me The Option To Mask - Facebook

Tried the program out.. chose facebook (1), and Cloudflare (2).. didn't even give me an option to mask.. just went straight to this error message:

Why is it on eternal blacklist?

I will fix that


OCPBUGS-17309: Updated golang.org/x/net/html dependency.

What type of PR is this?
/kind bug
What this PR does / why we need it:
Which issue(s) this PR fixes:
Part of the upstream patch: https://github.com/kubernetes-sigs/ibm-powervs-block-csi-driver/pull/434
Special notes for your reviewer:
Release note:
none

/cherry-pick release-4.13

Manually cherry-picked into release-4.13 via #44


Home Screen

Refer to notes for build:
https://docs.google.com/document/d/1HBv1MCxfPQZROFvANMPLLxZhLjra6MDwa0HySJ2sKO8/edit

PRIMARY WORK:

[ ] App Icon
[ ] Sort Newest/Trending
[ ] Card Contents - Boards/Posts
[ ] Upvotes - Left bottom of card has likes and dislikes w/ comment count
[ ] Dango menu post/board for flagging/sharing(maybe url)/moderate if its your post
[ ] Empty page if there are no subscriptions

Secondary:

Settings for view modes (card mode/compact mode/photo grid)
Drop-down
Gestures pull down for menus


How to set input seeds in GCAL collector ?

Dear Prof. Bednar,
I am using the model by GCAL collector and I need to change the input seeds for different
orientation maps. But in the submodel gcal.py and earlyvision.py there are not parameters
for input seeds. I can set the seeds in patterncoordinator.py. But it is not convenient to launch
multiple jobs by one time. So I would like to ask, how to set the input seeds in the submodel
GCAL used by the collecotr ? I guess I need to import patterncoordinator there, but I am not
sure how to set it exactly.

Good question.  In a quick glance over the files I don't see where to configure the input seeds when using gcal.ty and don't recall how to do that. Seems like there ought to be a better way, but you should be able to control seeds globally using something like import numbergen as ng ; ng.RandomDistribution.seed=2, and probably something similar for patterncoordinator that would apply only to input patterns. I'm not sure how that would affect launching multiple jobs, seems like you can do anything like this for multiple jobs easily enough...

Hi Bednar,
I tried "import numbergen as ng ; ng.RandomDistribution.seed=2" but it doesn't effect the retina input differences. I think I can just
import patterncoordinator.master_seed to the model GCAL's definition and then change the seed. It seems to work and produce different input patterns set, but I am not quite sure if this way gives problems.

Sounds like the RandomDistribution default seed is being overridden by the PatternCoordinator's master_seed, so setting it at the master_seed sounds like the right thing to do.  Good work!

I tried to set the input_seed as a parameter to give to the ModelGCAL so that I can launch multiple jobs, but I meet an error:
topo.sim.model = ModelGCAL(PatternCoordinator.master_seed=int(args.input_seed))
SyntaxError: keyword can't be an expression
I would like to ask how to solve this ?


console in brokered address space seems to lose communication with broker after broker restart

Scenario:
create a brokered address space, use the console to create some queues and topics, and send and receive messages.
Cause the broker to be restarted.
Attempt to create a new queue through the console - the new queue never becomes ready
Send messages to an existing queue - the stats on the existing queue as seen through the console do not update.
Example logs from agent during such a scenario: https://pastebin.com/raw/TbU1Tu9a

Believed to be already fixed.


[UART/DV]  sanity test fails for excelium

reproduce
make SEED=1243649678 SIMULATOR=xcelium
log included
run.log
wave is massive so I am leaving that out for now

@rasmus-madsen we had some design change on UART recently. I'm updating the TB now. Please uart failure for now. Thanks

@weicaiyang  are you asking me to close the issue? I think you may have forgotten a word or two?

This PR #1482 will fix it.


Psr Integration

Codecov Report
Attention: Patch coverage is 67.66917% with 43 lines in your changes missing coverage. Please review.

Project coverage is 80.59%. Comparing base (ce8d885) to head (b65d820).

Files with missing lines
Patch %
Lines

webfiori/cache/Item.php
36.36%
21 Missing :warning:

webfiori/cache/FileStorage.php
65.78%
13 Missing :warning:

webfiori/cache/Cache.php
84.37%
5 Missing :warning:

webfiori/cache/Storage.php
86.66%
4 Missing :warning:

:exclamation:  There is a different number of reports uploaded between BASE (ce8d885) and HEAD (b65d820). Click for more details.
HEAD has 1 upload less than BASE

Flag
BASE (ce8d885)
HEAD (b65d820)

3
2

Additional details and impacted files
@@              Coverage Diff              @@
##                dev       #3       +/-   ##
=============================================
- Coverage     97.90%   80.59%   -17.32%
- Complexity       60       97       +37
=============================================
  Files             3        4        +1
  Lines           143      237       +94
=============================================
+ Hits            140      191       +51
- Misses            3       46       +43

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


Youtube comment like bot

Hello, do you happen to have a bot that likes YouTube comments?

I can't find it anywhere :(

i will try


fix: do not update unconnected components

Prevent calling update on alive components. resolves #106

I have tested the code on my website and it does fix the issue!


Test issue.

Just a test to check out what happens on trello.
┆Issue is synchronized with this Trello card by Unito

yep, this works.


Feature Request: Only show specific containers

Feature description
Portainer allows you to hide specific containers by using the --label flag. It would be great if also the opposite was possible, i.e. hide all containers except specific ones.
Use cases might include, but are not limited to:

I only want to see the containers that belong to a given group (identified by an infrastructure label, e.g.).
I only want to see the containers that belong to a given application (identified by a common label for all application containers, e.g.).

What to do
This could either be done by introducing a new flag, or by changing the behavior of the existing flag. For consistency and for simplicity I'd vote for the second one. My suggestion is like this:
If you specify

docker run -d -p 9000:9000 -v /var/run/docker.sock:/var/run/docker.sock portainer/portainer -l owner=acme

instead of hiding all containers by owner acme (as up to now), it shows only those. If you want to hide them, you have to prefix the expression with the keyword NOT:

docker run -d -p 9000:9000 -v /var/run/docker.sock:/var/run/docker.sock portainer/portainer -l "NOT owner=acme"

This should also support parentheses and the terms AND and OR to build more complex queries, such as:

docker run -d -p 9000:9000 -v /var/run/docker.sock:/var/run/docker.sock portainer/portainer -l "NOT owner=acme AND (type=infrastructure OR type=application)"

The casing of the keywords not, and and or should not matter.

Hi. I think the use-case for this would also very often be a per-stack limitation. So either a command line flag to limit to one particular stack (which might involve some parsing on the API values), or a limitations not only for nodes, but also for networks/services would be appropriate. Something like this for example
-l "node.labels.env != staging"

or
-l "service.labels.env != production"

Also, why is the flag named -l, it seems to filter, not limit.

Hi guys,
+1 to that!
Portainer currently lets you add/revoke team access to a particular docker host/cluster. This doesn't play well in the scenario  where multiple teams use single docker host as their dev env, each running multiple containers.
Adding another layer on top of host/cluster abstraction would come handy here.
It would be great if you could configure portainer so that team A members would only have access to docker objects that have a particular label.

@SemionPar you can use the integrated access control features that allows you to create a Docker resource and restrict the access to this resource to either the creator or all the team members. This is available via the Access Control panel when creating a container (also as App Templates), a Swarm service or a volume.

@deviantony thank you for your response. If I wanted to use integrated access control features I would need to create containers via Portainer interface, am I right?
The thing is, I would rather deploy containers using CI/CD server and existing infrastructure, independently from Portainer.

@SemionPar Indeed, you would need to use the Portainer interface.
Note that Portainer also expose a HTTP API that can be used to manage your Docker environment as well, but it's not fully documented at the moment. Some examples are available here: https://gist.github.com/deviantony/77026d402366b4b43fa5918d41bc42f8
You can join us on Slack to discuss this if you need to.

+1 ^ 1000 on this one.
For a a bunch of us it's not practical to create the containers via UI due to scale or various other factors, and instead are largely looking to provide an easy means for other people to manage existing containers but not necessarily wanting to allow control to all the containers running on a host. I would love to see this feature happen.

@264nm what about using the existing UAC feature for this? You could create a team called restricted and add your users to that team. Then, switch the ownership of the containers that you want to allow access to to that new restricted team.

@deviantony switching the ownership of containers only works until these containers are not destroyed, obviously.
For an environment, where containers are created/destroyed often (think of a dev environment where services are redeployed multiple times on daily basis, recreating containers at each run) this is not the way to go, sadly (unless you use Portainer API to create and launch containers, as you mentioned before).

Right, we deploy using docker stack deploy from a (GitLab) CI/CD pipeline. This happens many times per day, so containers can come and go. We can tag them, but better would be to define users, and add docker services or - even better - stacks to them. In that case they would be able to see only containers belonging to services/stacks they have authorization for.

Has any progress been made on this front? I just set up LDAP on my Portainer instance and would like to invite people, but my team uses docker stack deploy and have a couple stacks with secrets in their environment variables.
There are a number of solutions to this use case but I haven't found any that work with Portainer.

Make all stacks restricted by default. Allow opening ownership on select stacks.
Make all stacks disabled. This will let my team's engineers at least be able to manage secrets.
Make the "Hidden Containers" feature that hides by label also apply to stacks.
Allow for hiding access to select pieces of the UI so that I can prevent env vars from being seen.

@taiidani what about using Docker secrets instead of placing secrets in environments variables? Ownership can be set on secrets as well.

@deviantony Best laid plans take time to hatch :) I'm making good progress with this fancy shell snippet to load the secrets into environment variables in the entrypoint script. It ensures that the Swarm service definition only sees the secrets and not the env vars while still letting the applications which don't support file-mounted configuration values access them as env vars.
# Export secrets for application to consume, if no env var has been set
if [[ -d "/run/secrets" ]]; then
    for secret in /run/secrets/*; do
        if [[ -e "$secret" ]]; then
            eval "export $(basename $secret)=\${$(basename $secret):=\$(cat ${secret})}"
        fi
    done
fi

But I'm hoping that the rollout of this to each service's images (and modifying our third-party images) does not block the rollout of Portainer.


[kie-issues#913] Upgrade to Quarkus 3.2.10.Final LTS version.

Tracker: https://github.com/apache/incubator-kie-issues/issues/913
Aligns with Quarkus 3.2.10.Final.

PR job #93 was: FAILURE
Possible explanation: Pipeline failure or project build failure
Please look here: https://ci-builds.apache.org/job/KIE/job/kogito/job/main/job/pullrequest/job/kogito-images.build-and-test/93/display/redirect
See console log:

Console Logs
	at org.jenkinsci.plugins.gitclient.CliGitAPIImpl$1.execute(CliGitAPIImpl.java:635)	at org.jenkinsci.plugins.gitclient.CliGitAPIImpl$2.execute(CliGitAPIImpl.java:871)	at org.jenkinsci.plugins.gitclient.RemoteGitImpl$CommandInvocationHandler$GitCommandMasterToSlaveCallable.call(RemoteGitImpl.java:170)	at org.jenkinsci.plugins.gitclient.RemoteGitImpl$CommandInvocationHandler$GitCommandMasterToSlaveCallable.call(RemoteGitImpl.java:161)	at hudson.remoting.UserRequest.perform(UserRequest.java:211)	at hudson.remoting.UserRequest.perform(UserRequest.java:54)	at hudson.remoting.Request$2.run(Request.java:377)	at hudson.remoting.InterceptingExecutorService.lambda$wrap$0(InterceptingExecutorService.java:78)	at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264)	at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1128)	at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:628)	at java.base/java.lang.Thread.run(Thread.java:829)	Suppressed: hudson.remoting.Channel$CallSiteStackTrace: Remote call to builds60		at hudson.remoting.Channel.attachCallSiteStackTrace(Channel.java:1787)		at hudson.remoting.UserRequest$ExceptionResponse.retrieve(UserRequest.java:356)		at hudson.remoting.Channel.call(Channel.java:1003)		at org.jenkinsci.plugins.gitclient.RemoteGitImpl$CommandInvocationHandler.execute(RemoteGitImpl.java:153)		at jdk.internal.reflect.GeneratedMethodAccessor1120.invoke(Unknown Source)		at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)		at java.base/java.lang.reflect.Method.invoke(Method.java:566)		at org.jenkinsci.plugins.gitclient.RemoteGitImpl$CommandInvocationHandler.invoke(RemoteGitImpl.java:138)		at com.sun.proxy.$Proxy269.execute(Unknown Source)		at hudson.plugins.git.GitSCM.retrieveChanges(GitSCM.java:1222)		at hudson.plugins.git.GitSCM.checkout(GitSCM.java:1305)		at org.jenkinsci.plugins.workflow.steps.scm.SCMStep.checkout(SCMStep.java:129)		at org.jenkinsci.plugins.workflow.steps.scm.SCMStep$StepExecutionImpl.run(SCMStep.java:97)		at org.jenkinsci.plugins.workflow.steps.scm.SCMStep$StepExecutionImpl.run(SCMStep.java:84)		at org.jenkinsci.plugins.workflow.steps.SynchronousNonBlockingStepExecution.lambda$start$0(SynchronousNonBlockingStepExecution.java:47)		at java.base/java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:515)		at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264)		at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1128)		at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:628)		at java.base/java.lang.Thread.run(Thread.java:829)[Pipeline] deleteDir[Pipeline] }[Pipeline] // dir[Pipeline] }[Pipeline] // dir[Pipeline] }[Pipeline] // script[Pipeline] }[Pipeline] // stage[Pipeline] stage[Pipeline] { (Declarative: Post Actions)[Pipeline] script[Pipeline] {[Pipeline] sh+ wget --no-check-certificate -qO - 'https://ci-builds.apache.org/job/KIE/job/kogito/job/main/job/pullrequest/job/kogito-images.build-and-test/93/api/json?depth=0'[Pipeline] readJSON[Pipeline] sh

@baldimir can you please rename your fork?
stderr: remote: Repository not found.
fatal: repository 'https://github.com/baldimir/incubator-kie-kogito-images.git/' not found

@ricardozanini We should adjust pipelines rather than asking to rename forks :-) buildchain already can do that for code repos, so it should be done also for others. Consider when we're promoted from incubation - we'd need to ask people to rename again, which is not practical.

The Jenkins CI check is now failing because of https://github.com/apache/incubator-kie-issues/issues/955.

@baldimir can you please rename your fork?
stderr: remote: Repository not found.
fatal: repository 'https://github.com/baldimir/incubator-kie-kogito-images.git/' not found

@ricardozanini We should adjust pipelines rather than asking to rename forks :-) buildchain already can do that for code repos, so it should be done also for others. Consider when we're promoted from incubation - we'd need to ask people to rename again, which is not practical.

Sure, we have an issue for that: #1729. I don't know where to change this, can you help?

PR job #101 was: ABORTED
Possible explanation: Most probably a timeout, please review
Please look here: https://ci-builds.apache.org/job/KIE/job/kogito/job/main/job/pullrequest/job/kogito-images.build-and-test/101/display/redirect
See console log:

Console Logs
[Pipeline] // stage[Pipeline] // stage[Pipeline] // stage[Pipeline] // stage[Pipeline] // stage[Pipeline] // stage[Pipeline] // stage[Pipeline] // stage[Pipeline] // stage[Pipeline] }Failed in branch kogito-base-builder[Pipeline] }Failed in branch kogito-data-index-ephemeral[Pipeline] }Failed in branch kogito-data-index-postgresql[Pipeline] }Failed in branch kogito-jit-runner[Pipeline] }Failed in branch kogito-jobs-service-allinone[Pipeline] }Failed in branch kogito-jobs-service-ephemeral[Pipeline] }Failed in branch kogito-jobs-service-postgresql[Pipeline] }Failed in branch kogito-runtime-jvm[Pipeline] }Failed in branch kogito-runtime-native[Pipeline] }Failed in branch kogito-s2i-builder[Pipeline] }Failed in branch kogito-swf-builder[Pipeline] }Failed in branch kogito-swf-devmode[Pipeline] }Failed in branch kogito-task-console[Pipeline] // parallel[Pipeline] }[Pipeline] // dir[Pipeline] }[Pipeline] // script[Pipeline] }[Pipeline] // stage[Pipeline] stage[Pipeline] { (Declarative: Post Actions)[Pipeline] script[Pipeline] {[Pipeline] sh+ wget --no-check-certificate -qO - 'https://ci-builds.apache.org/job/KIE/job/kogito/job/main/job/pullrequest/job/kogito-images.build-and-test/101/api/json?depth=0'[Pipeline] readJSON[Pipeline] sh

PR job #104 was: ABORTED
Possible explanation: Most probably a timeout, please review
Please look here: https://ci-builds.apache.org/job/KIE/job/kogito/job/main/job/pullrequest/job/kogito-images.build-and-test/104/display/redirect
See console log:

Console Logs
[Pipeline] }[Pipeline] // stage[Pipeline] }Build KIE » kogito » main » pullrequest » kogito-images.build-image PR #1743 - kogito-runtime-native: https://github.com/apache/incubator-kie-kogito-images/pull/1743 completed: SUCCESS[Pipeline] }[Pipeline] // stage[Pipeline] }Build KIE » kogito » main » pullrequest » kogito-images.build-image PR #1743 - kogito-jit-runner: https://github.com/apache/incubator-kie-kogito-images/pull/1743 completed: SUCCESS[Pipeline] }[Pipeline] // stage[Pipeline] }Build KIE » kogito » main » pullrequest » kogito-images.build-image PR #1743 - kogito-swf-builder: https://github.com/apache/incubator-kie-kogito-images/pull/1743 completed: SUCCESS[Pipeline] }[Pipeline] // stage[Pipeline] }Build KIE » kogito » main » pullrequest » kogito-images.build-image PR #1743 - kogito-task-console: https://github.com/apache/incubator-kie-kogito-images/pull/1743 completed: SUCCESS[Pipeline] }[Pipeline] // stage[Pipeline] }Build KIE » kogito » main » pullrequest » kogito-images.build-image PR #1743 - kogito-swf-devmode: https://github.com/apache/incubator-kie-kogito-images/pull/1743 completed: SUCCESS[Pipeline] }[Pipeline] // stage[Pipeline] }Calling Pipeline was cancelledClick here to forcibly terminate running stepsBuild KIE » kogito » main » pullrequest » kogito-images.build-image PR #1743 - kogito-data-index-postgresql: https://github.com/apache/incubator-kie-kogito-images/pull/1743 completed: ABORTED[Pipeline] }[Pipeline] // stage[Pipeline] }Failed in branch kogito-data-index-postgresqlBuild KIE » kogito » main » pullrequest » kogito-images.build-image PR #1743 - kogito-s2i-builder: https://github.com/apache/incubator-kie-kogito-images/pull/1743 completed: ABORTED[Pipeline] }[Pipeline] // stage[Pipeline] }Failed in branch kogito-s2i-builder[Pipeline] // parallel[Pipeline] }[Pipeline] // dir[Pipeline] }[Pipeline] // script[Pipeline] }[Pipeline] // stage[Pipeline] stage[Pipeline] { (Declarative: Post Actions)[Pipeline] script[Pipeline] {[Pipeline] sh+ wget --no-check-certificate -qO - 'https://ci-builds.apache.org/job/KIE/job/kogito/job/main/job/pullrequest/job/kogito-images.build-and-test/104/api/json?depth=0'[Pipeline] readJSON[Pipeline] sh

looks like CI is not helping 😬

The tests failing are expected, we have issues to solve them already. They are not related, but I've restarted just in case.

PR job #105 was: FAILURE
Possible explanation: Pipeline failure or project build failure
Please look here: https://ci-builds.apache.org/job/KIE/job/kogito/job/main/job/pullrequest/job/kogito-images.build-and-test/105/display/redirect
See console log:

Console Logs
[Pipeline] }Build KIE » kogito » main » pullrequest » kogito-images.build-image PR #1743 - kogito-jit-runner: https://github.com/apache/incubator-kie-kogito-images/pull/1743 completed: SUCCESS[Pipeline] }[Pipeline] // stage[Pipeline] }Build KIE » kogito » main » pullrequest » kogito-images.build-image PR #1743 - kogito-management-console: https://github.com/apache/incubator-kie-kogito-images/pull/1743 completed: SUCCESS[Pipeline] }[Pipeline] // stage[Pipeline] }Build KIE » kogito » main » pullrequest » kogito-images.build-image PR #1743 - kogito-jobs-service-ephemeral: https://github.com/apache/incubator-kie-kogito-images/pull/1743 completed: UNSTABLE[Pipeline] unstableWARNING: Tests on kogito-jobs-service-ephemeral seems to have failed[Pipeline] }[Pipeline] // stage[Pipeline] }Build KIE » kogito » main » pullrequest » kogito-images.build-image PR #1743 - kogito-runtime-jvm: https://github.com/apache/incubator-kie-kogito-images/pull/1743 completed: SUCCESS[Pipeline] }[Pipeline] // stage[Pipeline] }Build KIE » kogito » main » pullrequest » kogito-images.build-image PR #1743 - kogito-jobs-service-allinone: https://github.com/apache/incubator-kie-kogito-images/pull/1743 completed: SUCCESS[Pipeline] }[Pipeline] // stage[Pipeline] }Build KIE » kogito » main » pullrequest » kogito-images.build-image PR #1743 - kogito-task-console: https://github.com/apache/incubator-kie-kogito-images/pull/1743 completed: SUCCESS[Pipeline] }[Pipeline] // stage[Pipeline] }Build KIE » kogito » main » pullrequest » kogito-images.build-image PR #1743 - kogito-swf-devmode: https://github.com/apache/incubator-kie-kogito-images/pull/1743 completed: SUCCESS[Pipeline] }[Pipeline] // stage[Pipeline] }Build KIE » kogito » main » pullrequest » kogito-images.build-image PR #1743 - kogito-s2i-builder: https://github.com/apache/incubator-kie-kogito-images/pull/1743 completed: SUCCESS[Pipeline] }[Pipeline] // stage[Pipeline] }[Pipeline] // parallel[Pipeline] }[Pipeline] // dir[Pipeline] }[Pipeline] // script[Pipeline] }[Pipeline] // stage[Pipeline] stage[Pipeline] { (Declarative: Post Actions)[Pipeline] script[Pipeline] {[Pipeline] sh+ wget --no-check-certificate -qO - 'https://ci-builds.apache.org/job/KIE/job/kogito/job/main/job/pullrequest/job/kogito-images.build-and-test/105/api/json?depth=0'[Pipeline] readJSON[Pipeline] sh

It seems an issue fetching maven plugin:
[ERROR] Unresolveable build extension: Plugin org.kie.kogito:kogito-maven-plugin:999-SNAPSHOT or one of its dependencies could not be resolved: Failed to collect dependencies at org.kie.kogito:kogito-maven-plugin:jar:999-SNAPSHOT -> org.kie.kogito:kogito-pmml-dependencies:pom:999-SNAPSHOT -> org.kie:kie-pmml-dependencies:jar:999-SNAPSHOT -> org.kie:kie-pmml-models-mining-model:jar:999-SNAPSHOT @

This doesn't look like a problem connected with the change I did. Could we merge this please?


Error building boring ssl on Linux

While running yarn , the following error occurs
[2] ../third_party/boringssl/crypto/bio/fd.c: In function ‘fd_ctrl’: [2] ../third_party/boringssl/crypto/bio/fd.c:192:11: error: this statement may fall through [-Werror=implicit-fallthrough=] [2]        num = 0; [2]        ~~~~^~~ [2] ../third_party/boringssl/crypto/bio/fd.c:193:5: note: here [2]      case BIO_C_FILE_SEEK: [2]      ^~~~ [2] cc1: all warnings being treated as errors [2] boringssl.target.mk:439: recipe for target 'Release/obj.target/boringssl/third_party/boringssl/crypto/bio/fd.o' failed [2] make: *** [Release/obj.target/boringssl/third_party/boringssl/crypto/bio/fd.o] Error 1 [2] make: Leaving directory '/home/calvin/Git/zap-desktop/app/node_modules/grpc/build' [2] gyp ERR! build error  [2] gyp ERR! stack Error: make failed with exit code: 2
System Linux 4.13.0-16-generic
NPM 5.5.1

Hm, interesting. What's your Node version @calvin-pietersen

Ah, after taking a closer look at your error it seems to be grpc related. Did you generate compatible certs and use electron-rebuild (instructions in the README)

Any update @calvin-pietersen?

Closing


Traffic Light Detection. Detection.

What?
Updated TLClassifier Class.
Added models with weights for detection and classification of TL Lights.
Added JSON model for TL Detection because of Keras bug with model_load with custom objects
[INFO] [1521385727.398925]: [TL_DETECTOR] Loading TLDetector model
/usr/local/lib/python2.7/dist-packages/keras/models.py:291: UserWarning: Error in loading the saved optimizer state. As a result, your model is starting with a freshly initialized optimizer.
  warnings.warn('Error in loading the saved optimizer '

and Adam optimizer. Because Adam Optimizer has been added in the most recent version of Keras, had to update requirements list to remove version for Keras. This will allow to instal the most recent one.
model = load_model('./....h5')  File \"/usr/local/lib/python2.7/dist-packages/keras/models.py\", line 257, in load_model    custom_objects=custom_objects)  File \"/usr/local/lib/python2.7/dist-packages/keras/optimizers.py\", line 689, in deserialize    printable_module_name='optimizer')  File \"/usr/local/lib/python2.7/dist-packages/keras/utils/generic_utils.py\", line 141, in deserialize_keras_object    return cls.from_config(config['config'])  File \"/usr/local/lib/python2.7/dist-packages/keras/optimizers.py\", line 127, in from_config    return cls(**config)  File \"/usr/local/lib/python2.7/dist-packages/keras/optimizers.py\", line 403, in __init__    super(Adam, self).__init__(**kwargs)  File \"/usr/local/lib/python2.7/dist-packages/keras/optimizers.py\", line 63, in __init__    'passed to optimizer: ' + str(k))TypeError: Unexpected keyword argument passed to optimizer: amsgrad

Encountered a problem with Pillow
File "/opt/ros/kinetic/lib/python2.7/dist-packages/cv_bridge/core.py", line 248, in cv2_to_imgmsg
   img_msg.height = cvim.shape[0]

had to upgrade it to the version >4.3. Remove requirements version constraint for Pillow.
Updated TLDetector Class to call initialize and load Classifier.
Before detecting TL I am doing image augmentation to resize it to the proper size for the model.
After the TL was found I pass in data into TL Classifier to detect the light color.
Function extract_image does all the work with predicted TL Light. It takes in an image and a predicted mask and return cropped image of the TL.
Why?
Traffic Light Detection: This can be split into 2 parts:
Part1: Detection: Detect the traffic light and its color from the /image_color. The topic /vehicle/traffic_lights contains the exact location and status of all traffic lights in simulator, so you can test your output.
[x] - works on simulator
[x] - tested locally
[x] - local env version Used Docker container with Mac OS HIgh Sierra 10.13.3

Closing for now, found few more library issues with keras.

Closing for now.


[CHANGE]: Add Search Functionality to the Member Modal

Desired Change
Add an input to filter the users shown in the multiple select
Reason/Motivation for Change
Speed Up Selection of users.
Impact of Change
No response

change implemented in c0f96235c89afc3e4bf35587794b2c857876fd91.


docs: update deploy readme

OSD-11376
Updates deploy readme to make it easier to follow.

/lgtm
/approve


add multipliers and change schema to october

@colourful-land please resume work on return types from this PR and then we can update everything and merge

This issue intends to solve #225 but it's unfished yet

access

@James-Sangalli I've done my changes to add bitmask to the ethereum element. Can you check if every example passes the new schema and if so, update every example to refer to the new schema? Thanks.

@James-Sangalli I've done my changes to add bitmask to the ethereum element. Can you check if every example passes the new schema and if so, update every example to refer to the new schema? Thanks.

@colourful-land schema validation fails on legacy tokenscripts,try it out in the intellij IDE as it will syntax highlight.
Error for compound ZRX token:
`make cZRX.canonicalized.xml
XML Canonicalization
xmlstarlet c14n cZRX.xml  > cZRX.canonicalized.xml
XML Validation
xmlstarlet val --xsd http://tokenscript.org/2019/05/tokenscript/tokenscript.xsd cZRX.canonicalized.xml || (mv cZRX.canonicalized.xml cZRX.canonicalized.xml.INVALID; xmllint --noout --schema http://tokenscript.org/2019/05/tokenscript/tokenscript.xsd cZRX.canonicalized.xml.INVALID)
cZRX.canonicalized.xml - invalid
http://tokenscript.org/2019/05/tokenscript/xhtml1-strict.xsd:33: element import: Schemas parser warning : Element '{http://www.w3.org/2001/XMLSchema}import': Skipping import of schema located at 'http://www.w3.org/2001/xml.xsd' for the namespace 'http://www.w3.org/XML/1998/namespace', since this namespace was already imported with the schema located at 'http://tokenscript.org/2019/05/tokenscript/xml.xsd'.
cZRX.canonicalized.xml.INVALID:1: element token: Schemas validity error : Element '{http://tokenscript.org/2019/10/tokenscript}token': No matching global declaration available for the validation root.
cZRX.canonicalized.xml.INVALID fails to validate
make: [cZRX.canonicalized.xml] Error 3 (ignored)`

@colourful-land might be good to add arrays to this Schema version as well?

@James-Sangalli I've done my changes to add bitmask to the ethereum element. Can you check if every example passes the new schema and if so, update every example to refer to the new schema? Thanks.

@colourful-land schema validation fails on legacy tokenscripts,try it out in the intellij IDE as it will syntax highlight.
Error for compound ZRX token:
$ make cZRX.canonicalized.xml cZRX.canonicalized.xml.INVALID:1: element token: Schemas validity error : Element '{http://tokenscript.org/2019/10/tokenscript}token': No matching global declaration available for the validation root. cZRX.canonicalized.xml.INVALID fails to validate make: [cZRX.canonicalized.xml] Error 3 (ignored)

Look like you didn't change the namespace of your legacy TokenScript?

@colourful-land yup, it's working but the IDE doesn't like it mostly likely because the web path doesn't exist yet

@colourful-land have changed the TS files and removed the JS hack for compound e8 multipliers. Still WIP until we make the tokenscript schema for 2019/10 on the web

@colourful-land also need to add the return-type

@colourful-land also need to add the bitmask

Apologies, see you already have this

@colourful-land ready to merge

@colourful-land iOS completed, just need to do android now

Android complete but needs review and iOS needs to fix issues with new schema

@colourful-land #252 is solved in 579a8d6129de3a5e32aefcdf15755b190136f21b

iOS problem fixed here: https://github.com/AlphaWallet/alpha-wallet-ios/pull/1495 once merged we are ready to go


Change accent color for all styles

Is there an existing issue outlining your problem?

[x] I have searched the existing issues and they do not solve my problem.

Describe your issue.
I can change accent color for 1 theme, but it's too long. Can I change it for all at once?

Mentioned in https://github.com/catppuccin/userstyles/releases/tag/all-userstyles-export (which is linked on our readme as the recommended installation method).

You say, that only approach is import themes a lot of times to change accent color?

How else would you do it? I can add a list of checkboxes so that you can uncheck certain or all userstyles to that generator website?

For the record - I only visit half the sites we have userstyles for, and I use the import generator method. Way easier.

Ok, thank you. I have problem, that in some sites I slightly modify theme. No problem, I just don't want use local version.


Response can not access to array object in 6.2.4

env: Supertest 6.2.4, Windows 11 Pro 21H2, 22000.856
Description
res.body does give the whole object, but res.body.solutions does not give an object, it is undefined.
Code here
      const res = await request(server).get(root);

Log here
^      console.log('res get all:', res.body);

      res get all: [
        {
          _id: '62fc15a51aa9ad1b58f48f5e',
          description: '62fc15a51aa9ad1b58f48f55',
          solutions: [ '62fc15a51aa9ad1b58f48f5c' ],
          __v: 0
        },
        {
          _id: '62fc15a51aa9ad1b58f48f62',
          description: '62fc15a51aa9ad1b58f48f56',
          solutions: [ '62fc15a51aa9ad1b58f48f60' ],
          __v: 0
        }
      ]

^      console.log('res solutions,', res.body.solutions);

      res solutions, undefined

And my project is over here: https://github.com/Alex-Lin5/Algorithm-Coding-Pool

Here, it looks like res.body is an array, not an object. Which res.body.solutions were you expecting it to return, the first one or the second one? Or both?
Since res.body is an array of two objects, you'd can access it a couple of ways.
Access via index:
console.log('first res solutions', res.body[0].solutions) //this should print [ '62fc15a51aa9ad1b58f48f5c' ]
Access via iterator:
res.body.forEach(obj -> console.log('a solution', obj.solutions)) //this should print both on separate lines

Thanks man, I figured out where the bug is now.


[SQL Editor] failed to list query history when there're some chinese in comment

exception thrown when there're some chinese in sql editor, for example:
--- test
--- 这是一条测试语句
show tables;

it is because ace.js cannot get correct charactor width for the line contains chinese, exception will throw in line 73 of 'desktop/core/src/desktop/js/ko/bindings/ko.highlight.js', i think we should catch the exception.

Merged!

@romainr tks 👍

It happened in hue 4.10.0 again，How can I fix it


fix(getObject): null in return type for items

Using getObjects returns an array of records but also null when the object id doesn't match any record. Updating the type to reflect that.
See example of a response where the first object id matches a record and the second (object id "z") doesn't:

seems like it's circle which is partially down, not the tests themselves per se. Can you do a force push?


transparent color of BottomTabNavigator not working.

Current behaviour
Setting tabBarOptions.style.backgroundColor to transparent sets the background color to White.

Expected behaviour
Setting tabBarOptions.style.backgroundColor to transparent should make the color invisible, i.e make the color transparent.

Code sample
createBottomTabNavigator(
    {
      DashboardStack: DashboardStack,
      EditStack: EditStack,
    },
    {
      tabBarOptions: {
        style: {
          backgroundColor: 'transparent',
        },
      },
    },
  ),

Screenshots (if applicable)

What have you tried
I have tried the solution mentioned in a similar issue #46 but the problem with it is that other scrollView don't consider the tab bar as bottom anymore. That is, if I scroll my vertical scroll view to the very end, goes behind the tab bar.
Scroll View looks like this after scrolling to the bottom:

and on Android I am not able to change tabs using this method.

Your Environment

software
version

ios or android
both

react-native
0.60.5

react-navigation-tabs
2.5.3

node
v12.9.1

npm or yarn
6.10.2

Am I the only person who is facing this issue??

Am I the only person who is facing this issue??

Facing the same issue.

just checked on this issue. TabBar should be displayed as absolute in order to render it above the content. But, adding absolute to TabBar will raise problem like @mukeshsharma1201 said that screen isn't aware TabBar anymore. any suggestion on this problem @osdnk @satya164 ? Do you think expose onLayout from TabBar to get height will be good solution for this? but using this approach, user will need to adjust each screen and adding padding based on TabBar height

@adrianha we can export something like TabBarHeightContext which user can use to get the height of the tab bar inside of a screen.

Let me know if you were still interested at looking at this @adrianha 👍
If not, I'd gladly work on this.

hi @samchamberland , you can try to solve this issue


chore: Upgrade to yarn 4.0.0-rc.14, Node 16

Adds support for Node 18
What is the previous behavior before this PR?
As documented by @ronkok in #3681, yarn start (and most yarn operations) crashes on Node 16 as follows:
(node:7640) ExperimentalWarning: Custom ESM Loaders is an experimental feature. This feature could change at any time
(Use `node --trace-warnings ...` to show where the warning was created)
node:internal/errors:477
    ErrorCaptureStackTrace(err);
    ^

Error [ERR_LOADER_CHAIN_INCOMPLETE]: "file:///C:/Users/Ronal/OneDrive/Documents/GitHub/KaTeX/.pnp.loader.mjs 'resolve'" did not call the next hook in its chain and did not explicitly signal a short circuit. If this is intentional, include `shortCircuit: true` in the hook's return.
    at new NodeError (node:internal/errors:387:5)
    at ESMLoader.resolve (node:internal/modules/esm/loader:860:13)
    at async ESMLoader.getModuleJob (node:internal/modules/esm/loader:439:7)
    at async Promise.all (index 0)
    at async ESMLoader.import (node:internal/modules/esm/loader:541:24)
    at async loadESM (node:internal/process/esm_loader:91:5)
    at async handleMainPromise (node:internal/modules/run_main:65:12) {
  code: 'ERR_LOADER_CHAIN_INCOMPLETE'
}

What is the new behavior after this PR?
I followed the workaround in https://github.com/yarnpkg/berry/issues/4629. yarn start etc. are functional again.
I had to upgrade CI from Node 12 to 16 (or anything 14+) for Yarn to keep working.  Node 12 is deprecated, and 16 is the current active version.
Thoughts for consideration:
I'm not really convinced we should be using yarn anymore. I'd be tempted to use NPM or PNPM. But we might want to merge this anyway so at least we're functional again.
@ronkok Can you test that this works for you?

@edemaine Thank you. The Yarn upgrade got me back in business.
Do you know why some of the checks are failing? I'd like to approve this PR, but I hesitate to do so.

Do you know why some of the checks are failing? I'd like to approve this PR, but I hesitate to do so.

Indeed, the website is failing to build. The issue seems to be https://github.com/yarnpkg/berry/issues/2470 but the workaround described there doesn't work. ☹
Unless we can get some Yarn expertise from @ylemkimon, I'd be tempted to move to PNPM. I could try that in a separate PR.

@edemaine add enableGlobalCache: false to your .yarnrc.yml file.

Thanks @merceyz! Switching to stable Yarn 3.2.2 makes sense, and it looks like all tests are passing now! @ronkok could you test/approve/merge?


Added legends for daily confirmed chart

Description of PR
Added legends for daily confirmed chart as requested
Type of PR

[x] Enhancement

Relevant Issues
Fixes #866
Checklist

[x] Compiles and passes lint tests
[x] Properly formatted
[x] Tested on desktop
[x] Tested on phone

Please note the inconsistency in order of total cases and daily cases charts. Let me know if change is required and how to change.

Changing order of legends will change order in the stack of graph bars

@jeremyphilemon @sudevschiz Any change required here?


Allow non-uberjar deploys for speed & simplicity

We currently have lein uberjar running in these lines:
https://github.com/Parsely/streamparse/blob/master/streamparse/ext/invoke.py#L135-L146
That makes the JAR file contain Storm + all its JAR dependencies, which explodes the topology JAR size and makes deploys slow. A Python codebase that is only 20-30KB ends up being a 15MB JAR to submit.
The non-uberjar is built (it's the one without -standalone) but we are uploading the uberjar.

How is the status on this? Can this be solved?

We haven't had any time to work on this yet, but it's definitely solvable.

Storm expects all of the dependencies to be available in your submitted jar. While you could submit something else, your devops environment would be responsible for setting the appropriate dependencies in your classpath. That would also entail global specification of dependency versions, which seems bad.

In our experience, this only matters when you're including other JVM dependencies.  If your topology is pure python, then you can get away with only submitting the python code since it already has the Storm-specific dependencies.  We used to use Petrel, which constructed the jar as a zipfile, without any JVM dependencies included, and the resulting jars were much, much smaller.
That's really the only case we were thinking of.  To do as you're suggesting sounds like a nightmare.

Ah, fair enough. I'm making use of a jvm spout and several jvm-based bolts. A pure-python use case would indeed only have to do minor gymnastics to make sure the JAR was valid.

Reviewing and fixing this logic might be a good issue for a first pull request, @cabiad.


Response Message Failures should show the HTTP status code

In the health panel, it would be nice to show the HTTP status code. Because sometimes it's easier to understand the status code than the status message of an error request.

Implemented. The status code is going to be displayed only for newly uploaded and processed tests.


Text Not Show Content When style both set borderWidth and backgroundColor.

Description
Text Not Show Content When style both set borderWidth and backgroundColor.
if Text style is styles.tag, the '预售' Not Show，only show backgroundColor
else style is styles.tag, the '停止营业' Can show normally
like this

The Code:

style class:

Code link: https://github.com/zdpdsy/reproducer/blob/main/ReproducerApp/App.tsx
Steps to reproduce
1.cd ReproducerApp
2. npm  run start
3. run Xcode
React Native Version
0.73.9
Affected Platforms
Runtime - iOS
Areas
Fabric - The New Renderer
Output of npx react-native info
Node Version : v20.11.1

Stacktrace or Logs
none log

Reproducer
https://github.com/zdpdsy/reproducer

@zdpdsy issue lists  rn version 0.73.9 but reproducer is on 0.74.5, I am not able to reproduce the issue on reproducer, please confirm from your side

@zdpdsy issue lists rn version 0.73.9 but reproducer is on 0.74.5, I am not able to reproduce the issue on reproducer, please confirm from your side

@netmaxt3r  sorry,i push code now. you can try it again
The Result Snapshot is

@zdpdsy thanks for update, i see that this is fabric related issue. issue  is reproducible on current latest  rn 0.74.5 with fabric enabled , i was wrong on previous comment i was on old architecture when i said i was not able to reproduce on 0.74.5 sorry about that

@zdpdsy thanks for update, i see that this is fabric related issue. issue is reproducible on current latest  rn 0.74.5 with fabric enabled , i was wrong on previous comment i was on old architecture when i said i was not able to reproduce on 0.74.5 sorry about that

Thanks for the confirmation @netmaxt3r

@zdpdsy This is fabric related issue. I'm trying to debug this issue.
In Fabric New Renderer:
https://github.com/facebook/react-native/blob/ee25081d208a69e803f8b1975020b1eb2d897f5a/packages/react-native/React/Views/RCTBorderDrawing.m#L186
This is responsible for drawing backgroundColor and borderWidth styles.
I am looking to map same for Old Architecture. @cortinico Can you help me pointing the code responsible for handling border and backgroundColor for Old Architecture?
This will help me in finding the root cause.

@zdpdsy thanks for update, i see that this is fabric related issue. issue is reproducible on current latest  rn 0.74.5 with fabric enabled , i was wrong on previous comment i was on old architecture when i said i was not able to reproduce on 0.74.5 sorry about that

OK，Thanks,i find a temporary solution is change  border style To Super element Like as ，like th snapShot:

but this solution is temporary，can't Real problem solving。

@shubhamguptadream11 issue might not be on that function , since it is reproducible with dotted border style as well probably with layer zIndex for border layer and main layer.
https://github.com/facebook/react-native/blob/5ced5f8363a8c26bdcf70aa03c6696a681710f4b/packages/react-native/React/Fabric/Mounting/ComponentViews/View/RCTViewComponentView.mm#L737

I have found one more issue related to this when border width is not specified or computed to zero with dotted style we loose background color
 <Text style={{
        backgroundColor: '#F00',
        borderWidth: 0,
        borderStyle: 'dotted'
    }}>hello</Text>

this is due to early return without checking bgcolor https://github.com/facebook/react-native/blob/ee25081d208a69e803f8b1975020b1eb2d897f5a/packages/react-native/React/Views/RCTBorderDrawing.m#L469-L471

@netmaxt3r  I found one more issue related to border in Text Component only.
Here: https://github.com/facebook/react-native/issues/45925

I did some comparison with old and new arch implementation, i see border styling is not working properly even on old arch
     <View style={{
        backgroundColor: '#F00',
        borderColor: "black",
        borderWidth: 1,
        borderStyle: "dashed",
        minHeight: 30,
      }}/>
      <Text
      style={{
        backgroundColor: '#F00',
        borderColor: "black",
        borderWidth: 1,
        borderStyle: "dashed",
        minHeight: 30,
        marginTop: 10,
      }}
      >hello</Text>

text view render with solid border rather than dashed
I also see in the new arch RCTTextView has been replaced by RCTParagraphComponentView with improved logic for border rendering. i see now we render border and background on single CALayer and setting its zIndex to -1024 hopping it would render borderLayer behind the root layer of RCTParagraphComponentView, but it is not , you can verify this by applying some transparency to border layer (zindex works with relative to multiple sublayers). I am not sure issue is with iOS or RN, I believe we might have to reconsider the  logic for border and background layer
@cortinico since TextView is a extremely important component can we have someone from meta team take a look and suggest a way forward
possible options i see are

move rendering text to new sub layer and add on top of borderLayer
separate  border and background.

Any one can try this #45972 ?


Issues with regex

This plugin is very useful, excellent idea!
I'm only having some issues with the regex matching.
First of all, it won't let me escape the first slash:
 in 'services':
  - in entry 3 of 'services':
    in 'routes':
      - in entry 1 of 'routes':
        in 'plugins':
          - in entry 2 of 'plugins':
            in 'config':
              in 'allow_paths':
                - in entry 1 of 'allow_paths': should start with: /
                - in entry 2 of 'allow_paths': should start with: /
                - in entry 3 of 'allow_paths': should start with: /

And also a regex that I validated in some regex testers does not work as well:
- in entry 1 of 'allow_paths': invalid path: '/path/api/v1/site/[^\/ \#]+$' (characters outside of the reserved list of RFC 3986 found)

I think it’s okay to start with / because it can avoid many problems.
the second issuse:
Can you provide steps to reproduce？

Yeah, I've found a post staging that it's okay to start with the non-escaped slash, sorry.
As for the second issue, I'm trying to match the following route
/path/api/v1/site/SaksQZd2vdA1vmxdwpLvqCpyvpkmHrp8ltY7jvs3rYFb-0a2R
This id in the last part of the url is random, does not have a fixed size, and the regex must stop with it, as I need to match other routes that have other stuff in front of the id.

@PauloGDPeixoto Please tell me what you want and what you got, It would be better if there are steps to reproduce.

So I've added this configuration:
- name: path-allow
  config:
    allow_paths:
    - /path/api/v1/site/[^\/ \#]+$
    - /path/api/v1/site/[^\/ \#]+/[0-9a-f]{8}\b-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-\b[0-9a-f]{12}$
    - /path/api/v1/site/[^\/ \#]+$\/[0-9a-f]{8}\b-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-\b[0-9a-f]{12}/submit$
    regex: true

And with these I expect to match the following routes:
/path/api/v1/site/SaksQZd2vdA1vmxdwpLvqCpyvpkmHrp8ltY7jvs3rYFb-0a2R
/path/api/v1/site/SaksQZd2vdA1vmxdwpLvqCpyvpkmHrp8ltY7jvs3rYFb-0a2R/19673ac0-1d75-4fba-b289-7dfb9c67462a
/path/api/v1/site/SaksQZd2vdA1vmxdwpLvqCpyvpkmHrp8ltY7jvs3rYFb-0a2R/19673ac0-1d75-4fba-b289-7dfb9c67462a/submit

For reference, I don't want to allow something like:
/path/api/v1/site/SaksQZd2vdA1vmxdwpLvqCpyvpkmHrp8ltY7jvs3rYFb-0a2R/19673ac0-1d75-4fba-b289-7dfb9c67462a/other_value
The regexes work as intended in a validator.

Thanks for your prompt action!


Torrentio issues linking to all debrid key

Hi there
I have an issue were my all debrid keys can pull cached files from other addons but not torrentio.
Is there a particular reason why.
Used to work but no longer does.
Every key generated on all debrid works with say media fusion to bring back addon results.
But with tortentio it’s an immediate no streams found.
This is for new content and old
And used to be aces

Make sure you are not selecting smth not intended in quality exclude filter.

apologies sir
not understanding what’s smt
which option is that in the quality drop down?
Regards,
Feemo

From: TheBeastLT @.>
Sent: Monday, June 24, 2024 8:13:38 AM
To: TheBeastLT/torrentio-scraper @.>
Cc: faeem-mahommed @.>; Author @.>
Subject: Re: [TheBeastLT/torrentio-scraper] Torrentio issues linking to all debrid key (Issue #259)
Make sure you are not selecting smth not intended in quality exclude filter.
—
Reply to this email directly, view it on GitHubhttps://github.com/TheBeastLT/torrentio-scraper/issues/259#issuecomment-2185692563, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BJM3ZLYANXNE7QZIOBPNMO3ZI62JFAVCNFSM6AAAAABJZBE2KOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCOBVGY4TENJWGM.
You are receiving this because you authored the thread.Message ID: @.***>

me again
have only selected 1080p in quality filter
add on gets added to stremio
doesn’t get fetched when pulling up a show
does so for the likes of torrentjacket and media fusion though
Regards,
Feemo

From: TheBeastLT @.>
Sent: Monday, June 24, 2024 8:13:38 AM
To: TheBeastLT/torrentio-scraper @.>
Cc: faeem-mahommed @.>; Author @.>
Subject: Re: [TheBeastLT/torrentio-scraper] Torrentio issues linking to all debrid key (Issue #259)
Make sure you are not selecting smth not intended in quality exclude filter.
—
Reply to this email directly, view it on GitHubhttps://github.com/TheBeastLT/torrentio-scraper/issues/259#issuecomment-2185692563, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BJM3ZLYANXNE7QZIOBPNMO3ZI62JFAVCNFSM6AAAAABJZBE2KOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCOBVGY4TENJWGM.
You are receiving this because you authored the thread.Message ID: @.***>

Uninstall all previous configurations of Torrentio from you addons, and then install a fresh configuration where the only thing you change is enter your RD api key, nothing more. And see if that helps.

thanks guys that helped looks like the quality filter poked the actual add on
Regards,
Feemo

From: TheBeastLT @.>
Sent: Monday, June 24, 2024 9:55:38 AM
To: TheBeastLT/torrentio-scraper @.>
Cc: faeem-mahommed @.>; Author @.>
Subject: Re: [TheBeastLT/torrentio-scraper] Torrentio issues linking to all debrid key (Issue #259)
Uninstall all previous configurations of Torrentio from you addons, and then install a fresh configuration where the only thing you change is enter your RD api key, nothing more. And see if that helps.
—
Reply to this email directly, view it on GitHubhttps://github.com/TheBeastLT/torrentio-scraper/issues/259#issuecomment-2185850044, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BJM3ZL32PJNYK7Y3AI4DQNDZI7GHVAVCNFSM6AAAAABJZBE2KOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCOBVHA2TAMBUGQ.
You are receiving this because you authored the thread.Message ID: @.***>