[Installation] Error when running any of the examples

After following the installation procedure to the letter (only change is regarding the Meshcat-python installation. sudo apt-get libzmq3 couldn't be found but I managed to install from libzmq3-dev), I can't seem to run any of the examples as I always get this error:
ImportError: /home/cayovalsamis/pychoreo/conrob_pybullet/utils/ikfast/kuka_kr6_r900/ikfast_kuka_kr6_r900.so: undefined symbol: Py_InitModule4_64
The path changes depending on which example I'm trying to run but the undefined symbol is always the same.

@cayovalsamis sorry for taking so long for the reply... Would you mind giving me more info on your machine and OS? Which python version you are using? Have you compiled the IKfast modules?


fix(grid): handle direction object prop for responsive design

[x] I have followed (at least) the PR section of the contributing guide.

resolves https://github.com/mui/material-ui/issues/34571

@vanyaxk Thanks for the fix! Can you add a test to make sure that we don't mess it up in the future?

Added the test :)


refactor(cp): remove cards json tag

What type of this PR
Add one of the following kinds:
/kind bug
What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Specified Reivewers:
/assign @your-reviewer

/approve


Make the "open" tab as default active in HR Recruitment

Describe the task
When clicked on the main menu HR > Recruitment, it shows the open job application listing. However, the "Open" status in the tabs is not highlighted.

Expected behavior

[ ] The "Open" tab should be highlight by default as it was before when the UI was not updated.

This looks great. Thanks @suntwalriya


Truncated string output

We have found some Cerner Blobs containing compressed RTF, where the decoded output is truncated by your library. We would expect all RTFs to end with } char or }<whitespace>. Around 20% of the CE_BLOBS in a collection of 10M PowerChart RTF documents are being truncated and do not end with } - documents are being cut-off mid-sentence. There seems to be no discernable pattern as to what is truncating them and I have no idea how to debug this.
Is it possible there is a limit to the total size of the blob?
We experimented with another library implemented in Python and confirmed that no truncation occurred on the same files - so we are certain this isn't a problem with the source data. I also tried the Bruce Jackson alternative, but I can't find any suitable column in the Cerner Millennium schema to provide the stringSize parameter - we tried CE_BLOB.BLOB_LENGTH but this resulted in an $Index was outside the bounds of the array error.
Any suggestions you could provide would be greatly appreciated!

I'm not aware of those issues. Can you send some example code?
Remember that the notes in PowerChart can be spread across multiple rows.

Thanks for the example. I too am using a clr in a similar way.
I'll double check the documents I've got to see if I can see the same issue.

I checked about 250,000 decoded RTF values, and couldn't find the same issue of truncated RTF files.
The code I use is an internal fork to this public one. There's only one difference I could find with my private version of the decoder vs this public one and that's Line 115 of OcfLzw](https://github.com/pgodwin/OcfLzw/blob/c1fc26071add9047680a938c525ad8090c6c5903/OcfLzw2/OcfLzw.cs#L115):
My internal version has:
if (nextCommand == -1)
   break; // EOF?

instead of
if (nextCommand < 0)
    break; // EOF?

@pgodwin thanks again for all your suggestions - so helpful!

Things to try:
You could try modifying the code with the change above and try decompressing the problem blobs again. I'd be very curious to see if that fixes it.

Tried this - same issue

Check the BLOB_CONTENTS values in your database. They should each end with ocf_blob\0 (0x6F63665F626C6F6200). You can check this with SQL:

Thanks I didn't know this - but have checked and all affected files do end with the correct byte sequence.

Check your joins between CE_BLOB and CE_BLOB_RESULT. VALID_UNTIL_DT_TM field should be included on your joins. Perhaps your MAX_SEQUENCE_NUMBER is for an old version of the CE_BLOB record?

Going to look into this shortly with a colleague who wrote the ETL scripts.
But it does strike me that for exactly the same BLOB_CONTENTS, we get a full extract using the Python port of your code. It seems very strange that that would work based on the fact that it's a port, so I would assume designed to be functionally identical.
I'm going to try to compare the C# and py scripts to see if I can spot the difference.

@pgodwin - actually on closer inspection, the Python port is not of your code, but of the Bruce Jackson version you posted on the README.
I asked ChatGPT to analyse the differences between your OcfLzw2 and the Python port. Here's what it had to say:

Certainly! Here are some of the differences between the two scripts:

The C# script uses a LzwTable class to store the LZW dictionary, whereas the Python script uses a list of lzwItem objects to store the lookup table. The LzwTable class has a fixed size and is implemented using an array, whereas the Python list can grow dynamically.

The C# script uses a NBitStream class to read the compressed data, whereas the Python script reads the data byte by byte from the input buffer.

The C# script uses a StringBuilder to accumulate the decompressed string, whereas the Python script appends the decompressed data to a list.

The C# script uses the Encoding.Default property to decode the decompressed byte array into a string, whereas the Python script does not decode the data into a string.

The C# script uses a fixed dictionary size of 8192 codes, whereas the Python script does not have a fixed dictionary size.

The C# script uses a dynamic bit-width scheme to adjust the number of bits needed to represent each LZW code, whereas the Python script uses a fixed bit-width scheme.

The C# script has a Decode method that takes a Stream object as input, whereas the Python script has a decompress method that takes a byte array as input.

The C# script uses bitwise operations to read and manipulate the compressed data, whereas the Python script uses arithmetic operations and bit shifting.

Overall, both scripts implement the LZW algorithm to decompress the OCF compressed data, but they use different data structures and algorithms to achieve this. The C# script is optimized for performance and uses a fixed dictionary size and a dynamic bit-width scheme to minimize the memory overhead. The Python script is simpler and more flexible, using a list to store the lookup table and a byte-by-byte input scheme to handle the compressed data.

So it looks like they are completely different implementations - I think the key part is that yours is more performant.
I have also tried to use the Bruce Jackson C# code on my side. I couldn't get it to work out-of-the-box as it needs a stringSize input and I could not find this in the Cerner tables (only BLOB_LENGTH, which doesn't seem to work).
I adapted his code to use a MemoryStream instead, to avoid having to predeclare the length of the bytes array. When I ran it as a CLR function, I got the same result as your C# script (i.e. truncated RTF on some records) - but much slower.

and some more...
Author:
Comparing the 2 scripts, can you suggest why the C# version sometimes truncates the decoded OCF? Tested on 10M files, we found 2M of them were truncated by the C# version - but they are fine in the Python version
Based on the code, it's difficult to determine exactly why the C# version is truncating the decoded OCF for some files, but one possibility is that it has a bug in how it handles the LZW dictionary.
ChatGPT:
In particular, it's possible that the LzwTable class used by the C# script has a bug that causes it to miss certain codes, resulting in an incomplete dictionary. This could cause the C# script to terminate prematurely when it encounters an undefined code. The Python script, on the other hand, uses a simple list to store the lookup table, which may be more resilient to bugs or errors.
Another possibility is that there may be differences in how the two scripts handle byte order or endianness. If the compressed data is stored in a different byte order than what the C# script is expecting, it could cause the C# script to misinterpret the data and truncate the output.
Without more information about the specific files that are being processed and the details of the bug or error that is causing the C# script to truncate the output, it's difficult to provide a more precise explanation or solution.

I'm no expert at compression, so I'm not ruling out a bug in the implementation of LZW in this code. Without an example of the blob I can't say for certain. I've checked the CRCs of my decompressed versions and they are matching the CRC in CE_BLOB_RESULT, suggesting it is decompressing items correctly.
How are you calling the Python version? Can you call the C# implementation through the same method?
How are you bringing the blobs into SQL Server? Could there be an issue there?

I'm on LinkedIn but based in Australia so might be a bit difficult.
I think the fact you're getting the same result in two separate, independently developed C# implementations suggests it maybe something in your dataset.
I think it'd be useful to compare how you pass the compressed blobs to the Python code vs C#.

@pgodwin
Apologies for the delay, I had to have some internal discussions about this.
I have managed to isolate the truncation issue happening on documents attached to TEST PATIENTs.
I have sent you an email to the gmail address specified in git log on this repo - if you don't receive it please let me know how best to contact you.

I am the author of the Python port you mentioned, and just want to offer any assistance, I have run the python code on about 3.2m blobs with no issues (other than rtf being another thorn in my side...)

@thegoatherder yeah, I was not concerned about performance, for me this was a one time dump.  Though if I have time I might try to see where I could improve performance.  Our bottleneck when doing this was pulling from oracle.  Or if you have any suggestions on performance, that would be wonderful as well.


Proposal: policy on handling bad commits

Occasionally committed PRs may introduce bugs that were not caught during review or CI. Presently such bad commits are fixed in the open if they are not older than 7 days. Older commits may be treated as CVEs if they introduce a security vulnerability.
However there is currently no policy on how tainted commit ranges are identified.
This presents a challenge for Envoy operators that deploy code from the main branch. Specifically when advancing the main branch internally there is currently no reliable way to tell if the internal main branch head is in the middle of a tainted commit range.
This proposal is to establish a policy on handling bad commits that would provide Envoy operators with a (more) reliable way of ensuring that internal main branch head is not broken. Specifically:

Ordinarily a bad commit is reverted.
The revert PR adds the following message: Fixes bad commit #12345
In some cases a commit may be infeasible to revert, such that it is in itself is critical to Envoy stability (i.e. security bug fix) or may require chain reverts that affect a significant portion of the code.
In this case a PR that fixes the problem will be applied.
An additional line is placed in the PR message: Fixes bad commit #12345

NOTE: this mechanism relies on internal main branch to be at least 7 days behind OSS main branch.

@mattklein123 @lizan @alyssawilk

I think this policy makes sense. If we agree on it, we should add this exact text as an optional field in the PR template. Also, it should be clear whether the reference is a commit hash, PR number, or both.

SGTM.  for ease of tooling do we want the Pr number for the commit, the hash or both?
optional nit,  I'd lean towards a simpler "fixes commit #12345" since I think the fact it was a bad commit is implicit from it needing to be fixed :-P

Sure SGTM in general. Let's make sure to clearly document this somewhere in addition to the PR template per @ggreenway

PR # is I think easier. But tooling can really work with both.

First test run of the policy-in-progress: #17522


Kalimdor Balance Notes

Link to document:
https://docs.google.com/document/d/16JyX0eippDbmpvdYk0CcjrDSf_Yi0OKVB0UJ2kqq1hY/edit

@Ripzss Is this document still relevant? If not can we close?

You can close this @YakaryBovine


Make TravisCI Run Tests on Multiple Operating Systems

Summary
As of 11 October 2018, Travis CI now supports Windows build environment. In light of #213  this PR enables multiple OS build environments (Linux, Windows, and MacOS).
The Windows build currently fails because #213. They should build correctly when this issue is resolved.

Thanks for the PR @cr0wst. Putting this on hold until #213 is fixed.


Lookup a child and parents relationship ID when given the child ID and mother or father ID

This use case comes from the pedigree where we have person IDs but not relationship IDs. Currently the sample app is catching the add events from the pedigree and looking up the child and parents ID when necessary. We should port that functionality into this element.
See https://github.com/fs-webcomponents/sample-app/issues/22
There are a number of use cases for adding persons and wanting certain relationships to be constructed (as documented). We differentiate between the use cases by examining the data that's provided.
In this use case we either have just a childId (adding the first parent to a child) or we have a childId with a fatherId or motherId (adding a second parent to the child). The former use case is already covered.
<fs-add-person child-id="STGH-123"></fs-add-person>

We can cover the second use case by allowing for this:
<fs-add-person child-id="STGH-123" mother-id="MMMM-123"></fs-add-person>

We'll have to update the first case to skip creating a new capr if a motherId or fatherId is provided. But there's already a use case that expects a motherId or fatherId without a childId so we'll have to update that one to skip if there's a childId present.

[ ] Update the new-parent-create-capr use case to skip if a motherId or fatherId is also present.
[ ] Update the new-child-create-capr use case to skip if a childId is also present.
[ ] Accept childId and motherId or fatherId to update an existing capr.

The code that creates the relationships, after the person is created or selected, assumes that we can create all requests before we start executing any requests. But this use case is going to break that assumption: we have to make one request to get data before we can construct the next request to update the child-and-parents relationship.
This means we'll need to refactor the most complicated piece of code.

I've done a little refactoring that significantly increases readability and will make it easier to make further modifications. However in looking at how the code is full of if statements and no if else statements, I remember that there was a specific reason that we wanted to allow for multiple use cases to be met at the same time. Yet I can't remember why that is. If we make the two adjustments to existing use cases, as documented above, then we may preclude any possibility of meeting multiple use cases at once.

I just compared the use cases described in the docs to the use cases allowed in this element as well as the use cases that the sample app uses. They're all distinct. There's no situation that hits multiple scenarios. Perhaps there was in the past before we added allowed a couple relationship to be created with caprIds and intentionally prevented that from trigger the add-child use case; see https://github.com/fs-webcomponents/fs-add-person/commit/69cd8c255280c3a12dabe87f483a6dcb908a1725.
I guess we're past that hurdle.
One more thing I'd like to do is assign names to each use case. I'm frequently getting them confused in my mind. It's hard to look at the use case examples and know exactly what's expected to happen without reading the explanation. I'd like to at least have a short phrase for each use case that's easy to remember.

I've done even more refactoring of _createRelationship(). Now we have a method that just generates a list of requests and executes them serially. Now I need to add a use case where we must issue one request and process the return data before being able to create additional requests.
I just realized that our use case fits an existing use case after the first request is finished. The first request is issued to calculate the list of caprIds that match the childId + fatherId + motherId combination. Once we have the list of caprIds then our use case fits the existing add-parent use cases. So when that first request returns and we have the list of caprIds, we can clear the childId property and set the caprIds property then run through the code we have now.


Paste Image is using backslash instead of slash

When using Ctrl-V to paste in an image the following markdown code will be generated:
[](assets\20210623_162030_image.png)

Unfortunately the backslash will not be rendered correctly in Github.
How is it possible to have a slash instead of the backslash during the Paste in?

fixed in v0.1.10


Ensure "dependencies" are possible

Carrying over https://github.com/opencontainers/image-spec/issues/35#issuecomment-223011729
Like appc/spec ACI dependencies https://github.com/appc/spec/blob/master/spec/aci.md#dependency-matching
Ensure that OCI image spec can support an image (config plus descriptors) that includes references to dependent layers that may themselves include its own config.
To some extent, this would satisfy #39

Are we interested in dependency or history?
The main issue with dependencies is that it requires a way to merge configuration. This is much easier if this merge is done at build time, where it can be controlled and deterministic.
In general, dependencies really belong at build time. We can always trace back common components through content-addressing, but carrying dependency information into the image doesn't really provide much.
Otherwise, the image format degrades into an linked list.

@stevvooe the objects required for the linked list could be collected ahead of time by the person signing the version. I agree in general that things should be immutable. Walking the linked list post-transport isn't expensive. This was always the intention with AppC, we just didn't get to it as the linked list is generally "short".

@philips So, does this become a side-chain? Where does that reference land?

Here is a use case that I think should be possible: I want to compose an image out of a binary that dynamically links against libssl so that

I can audit versions of libssl I have running, and
I can push out an update without having to rebuild the blobs every container image I have out there.

Another is the one I mentioned in #39, of composing an image out of a server binary and served data blobs. I would like to be able to update each independently of the other. In Kubernetes, this would happen one level up by composing two images as a pod, but for other systems that abstraction is not available.

(There was a fantastic blog post about container packaging systems that I can't find now, but i gave lots of ideas of things you would like to do , and know about your running system.)

@kamalmarhubi So, are you willing to trust the metadata indicating which version of libssl is present in the image? Deep scanning is the only way to guarantee this.

Another is the one I mentioned in #39, of composing an image out of a server binary and served data blobs. I would like to be able to update each independently of the other.

This really seems like a runtime concern. Because the references are content addressable, updating one or the other component really introduces a new artifact. Implementations can find creative ways to distribute that artifact efficiently.

In Kubernetes, this would happen one level up by composing two images as a pod, but for other systems that abstraction is not available.

Exactly, but this has no place in the image format specification. Compose them at runtime.

@stevvooe I would argue if you can't trust the metadata of the person pushing the binary then you shouldn't trust the binary either. But, I don't think this is really the forum for this discussion.

I would argue if you can't trust the metadata of the person pushing the binary then you shouldn't trust the binary either.

While my statement above mentioned trust, this is less about malice than it is ensuring the right metadata lands in the right place. A small bug or incorrect metadata can be costly and the only way to catch it is to actually fingerprint and scan the content.

@vbatts I think this should be moved out of the v1.0 path. Anything we do here will likely break compatibility with Docker v2.2 and we should strive for compatibility in the first version. Any objection to me moving this to post-v1.0.0?

I'm not opposed to that. I think I'd hoped the naming discussion would have
yielded something that this issue could lean on.
On Wed, Jun 8, 2016, 18:56 Brandon Philips<EMAIL_ADDRESS>wrote:

@vbatts https://github.com/vbatts I think this should be moved out of
the v1.0 path. Anything we do here will likely break compatibility with
Docker v2.2 and we should strive for compatibility in the first version.
Any objection to me moving this to post-v1.0.0?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/opencontainers/image-spec/issues/102#issuecomment-224766087,
or mute the thread
https://github.com/notifications/unsubscribe/AAEF6a8SCq9S6Z89Alfgn76-TBmdHnNkks5qJ1avgaJpZM4IrzTQ
.

@vbatts It might be okay to include a parent reference. There is already a history of rootfs diffIDs. I think if we clarify whether these dependency references are to manifests, image config or raw layers, we can make a clearer decision.

@vbatts OK, moving it to post-v1.0 for now. I think we should focus on the required (base) layers in v1.0 and compatibility. After that we can discuss optional features and layers.

also for the record, just history of the rootfs-diffIDs is not a full enough history for many audit processes.

Ideally, this use-case would be covered by the references API


Add run_exports

@conda-forge-admin, please rerender

@isuruf, is this how one adds run_exports to a package with multiple outputs like this?


style: add favicon.ico file to public directory

I have just simply added switch icon as missing site favicon

@NecipAkgz great, thanks for this!


feat: add docker compose deployment configuration

Description
This PR adds docker-compose configuration files and documentation how to run it
Please delete options that are not relevant.

[x] New feature

How Has This Been Tested?
Manual testing of the working deployment
Checklist

[x] My commit message is conventional
[x] I have run pre-commit on all files before creating the PR
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] I have checked my code and corrected any misspellings

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 84.80%. Comparing base (d26ba44) to head (416c065).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #973   +/-   ##
========================================
  Coverage    84.80%   84.80%
========================================
  Files           28       28
  Lines         2020     2020
========================================
  Hits          1713     1713
  Misses         307      307

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

:tada: This PR is included in version 1.11.0-beta.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Support pasting multiline expressions into the REPL

Currently, when a multiline expression is pasted into ammonite repl, it is parsed line by line, which results in different behavior from the how the compiler would interpret it in a file.
Scala REPL in paste mode
scala> :paste
// Entering paste mode (ctrl-D to finish)

Seq
 .apply(1)

// Exiting paste mode, now interpreting.

res1: Seq[Int] = List(1)

Ammonite
@ Seq
res11: scala.collection.Seq.type = scala.collection.Seq$@4f920ec4
@  .apply(1)
<console>:1: illegal start of definition
 .apply(1)
 ^

This is interesting. I'm not willing to add a :paste special command, but I can provide a
block{
   ...
}

syntax that should be equivalent

Fixed by https://github.com/lihaoyi/Ammonite/commit/46fd86de9825558b920cebdbae91878b6cbeb576, which provides the following syntax
        @ {
            val x = 1
            val y = 2
            x + y
          }
        x: Int = 1
        y: Int = 2
        res0_2: Int = 3

Thanks :) My primary use case for paste mode is trying examples code off people's wikis, blogs etc and trying them out. So, will this allow the @ { to be entered, then the code pasted, then }, and then eval it all?

Just {, the @ is the prompt!
This is assuming the code is well formed, of course. It relies on the parse not-completing/completing to decide whether or not to wait for more.

When the next char is available immediately after a tab,
the tab is taken raw instead of invoking completion.

just curious: is there reason why ammonite does not implement multiline paste support like ipython by waiting for the first blank line until execution?


How to set cache key prefix dynamically multi-tenant scenario

I can see in the example that we can do the following:
services.AddEFSecondLevelCache(options =>
    options.UseEasyCachingCoreProvider(
       (serviceProvider, cacheKey) => "redis-db-" + serviceProvider.GetRequiredService<IHttpContextAccesor>().HttpContext.Request.Headers["tenant-id"],
       isHybridCache: false)
    // `Or` you can set the cache key prefix per tenant dynamically
    .UseCacheKeyPrefix(serviceProvider => "EF_" + serviceProvider.GetRequiredService<IHttpContextAccesor>().HttpContext.Request.Headers["tenant-id"])
    .ConfigureLogging(true)
        // Fallback on db if the caching provider fails.
        .UseDbCallsIfCachingProviderIsDown(TimeSpan.FromMinutes(1))

But in our case, to get service that returns the tenant-id is scoped, and trying to use that here causes an exception. How do you recommend we solve this problem?

Follow this thread

@ahmad-kemsan, did you ever resolve your issue?  I'm having the same issue.  I don't want to create new scope but rather use the same scope as the caller.

When you have access to the current HttpContext via serviceProvider.GetRequiredService<IHttpContextAccesor>().HttpContext, it means you have access to the current executing scope and it will give you  access to the request's service container
and its currently instantiated services via  HttpContext.RequestServices.GetService<IMyService>()


Improvements for uninstall

Uninstall now catches exceptions and tries up to 3 times to remove an object. Exceptions are only raised if after 3 tries some objects still remain.
Tested with https://github.com/larshp/abapPGP
Closes https://github.com/abapGit/abapGit/issues/379 and https://github.com/abapGit/abapGit/issues/3949, and probably #3228
PS: In certain cases with cyclic dependencies it won't be possible to automatically delete.

@mbtools check conflicts

thanks 👍


docker container is trying to bind to host IP address?

It looks like the docker container is trying to bind directly to the host ip rather than localhost or <IP_ADDRESS> which causes issues because I have other containers listening on port 8081 already.  Is there a an env var I'm missing somewhere that let's me try and bind this to <IP_ADDRESS>?
docker run -e HTTP_USER=guest -e HTTP_PASSWORD=guest -e ADDRESS=redis -p 7000:8081 artifactory.epic.com/docker/rediscommander/redis-commander

Creating custom redis-commander config '/redis-commander/config/local-production.json'.
node ./bin/redis-commander
Using scan instead of keys
No Save: false
events.js:200
      throw er; // Unhandled 'error' event
      ^

Error: listen EADDRNOTAVAIL: address not available <IP_ADDRESS>:8081
    at Server.setupListenHandle [as _listen2] (net.js:1289:21)
    at listenInCluster (net.js:1354:12)
    at GetAddrInfoReqWrap.doListen [as callback] (net.js:1493:7)
    at GetAddrInfoReqWrap.onlookup [as oncomplete] (dns.js:65:10)
Emitted 'error' event on Server instance at:
    at emitErrorNT (net.js:1333:8)
    at processTicksAndRejections (internal/process/task_queues.js:81:21) {
  code: 'EADDRNOTAVAIL',
  errno: 'EADDRNOTAVAIL',
  syscall: 'listen',
  address: '<IP_ADDRESS>',
  port: 8081
}

Redis Commander just did what you said it to do...
You mixed up the environment variables - ADDRESS defines the ip address to use to bind its local server to. This env var  should not be set running in docker (it defaults to <IP_ADDRESS> there) . Its for running a local installation checked out from git or installed via npm.
Redis server related configuration is defined via REDIS_* env vars.


Mobile - Swap infiniteHits widget for existing pagination widget

Docs: https://community.algolia.com/instantsearch.js/v2/widgets/infiniteHits.html
Existing implementation of hits template uses allItems feature to display Materialize table-based hits view.
May need to replicate table view using custom HTML/CSS and/or other Materialize helpers to allow for use of a single items template (thus, removing dependency on allItems)

See also https://github.com/algolia/instantsearch.js/issues/2519


Add "latest" tag to images

It would be really great if there would be a "latest" tag. I know it's unsafe for production use, but I'd be glad to have it when testing on my home setup.

Good idea; I've implemented it.


CHANGELOG.md for ChangeLog in VSCode extension view

Can we get a CHANGELOG.md to see whats new on each release? This is common use for github projects and will be picked up by VSCode too.

The problem here is that changes are not done in this repo, but in the language server repo. I am reluctant to duplicate a changelog in both repos.

Maybe use a symlink for this? @felixfbecker

you mean a git submodule?! that only works for directories
is the changelog taken from GitHub or from the extension package?

I think from GitHub, because even unupdated extension gets a changelog  that shows what's new.

This is needed. What is new in 1.3.0? I have this extension disabled due to a memory leaks. Does this fix that? I've no idea!

@rhyek Do you have an idea how to add one without maintaining it in multiple places? The commit log makes it pretty obvious what changed in 1.3.0: https://github.com/felixfbecker/vscode-php-intellisense/commits/master

Basically, it adds the possibility that the intellisense boots up as soon as there is an .php file in the workspace. This is supported in the next insider and in the next stable release. This really improves the performance, because you can just start when you start editing a file and everything is indexed(or at least the index is a bit bigger).

@felixfbecker the language server and vs code extension repos don't have a synced version number and release schedule from what I can see. Not sure how linking a CHANGELOG.md between the two would help? Nevertheless, what @jens1o says should work, no?

cough I created the pr #123 *cough

@rhyek The version number is not synced because both follow semver, but I cut a release here whenever a release is done in the LS. Could you elaborate how you think it would work?

@felixfbecker, last release for php-language-server was on April 24th? Not sure I understand.

But this extension can offer updates as well, for example anything which is isn't handled by the lsp, for example memory restrictions for the server, or a note for vscode when it should launch this extension.

Maybe just keeping a CHANGELOG.md for each project in their respective repo's, linking to the language server's in this one's, and keeping a synced release schedule should do the trick?
In this changelog you could, for example, provide a link in the 1.3.0 entry to a specific release tag or line number on the language server's CHANGELOG.md?
Or just a general link to language server's change log at the top of this one.
You could also consider concatenating the language server's changelog into this repo's.

Awesome!


luci-app-olsr-viz: migrate to JavaScript-based implementation

Hi! I've completed the migration of luci-app-olsr-viz to JavaScript. All the features are functioning just like they did in the old Lua app.
And in the old app, there was an issue of HTTP 413 Content Too Large after running it twice, which resulted in a blank page because the app was saving 130 Cookies at a time as it was saving all the nodes in cookies. I tested it on many routers and was able to reproduce the issue every time.

So, I removed the saving of the nodes in the cookies, as it was making the app unusable.
Now everything is working normally, and I've fixed the error.

my $0.02 - I like this.

@jow- this migrated app looks good to me

Merged, thanks!


Exception in DockContentPresenter

Thanks for the upgrade to Avalonia 11!! We are currently integrating it into our product and noticed an exception:
System.NullReferenceException: Object reference not set to an instance of an object.
at NP.Utilities.ReflectionUtils.SetFieldValue(Object obj, String fieldName, Object val, Boolean includeNonPublic, Type realType)
at NP.Ava.UniDock.DockContentPresenter.OnPropertyChanged(AvaloniaPropertyChangedEventArgs change)
at Avalonia.Animation.Animatable.OnPropertyChangedCore(AvaloniaPropertyChangedEventArgs change)
We tracked it down to the following code:
case nameof(UseLayoutRounding):
case nameof(BorderThickness):
this.SetFieldValue("_layoutThickness", null!, true);
The logic isn't finding _layoutThickness because the .NET reflection logic doesn't look in base classes for some reason.  Can you please take a look? Thanks!

Thanks Adam, I'll look into it on Sunday, Feb 11.

Thanks for reporting this issue. I fixed it. Please, update your unidock reference to NP.Ava.UniDock.1.0.6

Please, also test it and close the issue.

Looks good! Thx!


PS-392 use phr:dbx-subdefs-api

replace phr api me/subdefs by databoxes/subdefs, including "type"

do not merge now


GPU Access?

What's the key to enabling GPU access?  I've cloned your repository and run the examples and CPU works, GPU does not. I have a NVidia GPU with Cuda 8 drivers and other toolkits like ConvNetSharp use it.
Any tips?
Thanks,
Darren

Hello Darren,
There is a " SieNet.Examples
https://github.com/deepakkumar1984/SiaNet/tree/master/SieNet.Examples"
project which is for GPU. In the main method please check if the following
line of code throws an exception:
var devices = DeviceDescriptor.AllDevices().Where(x=>(x.Type ==
DeviceKind.GPU)).ToList();if (devices.Count == 0)       throw new
Exception("No GPU Device found. Please run the CPU examples instead!");
On Wed, Mar 14, 2018 at 1:47 AM, Darren Schroeder<EMAIL_ADDRESS>wrote:

What's the key to enabling GPU access? I've cloned your repository and run
the examples and CPU works, GPU does not. I have a NVidia GPU with Cuda 8
drivers and other toolkits like ConvNetSharp use it.
Any tips?
Thanks,
Darren
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/deepakkumar1984/SiaNet/issues/32, or mute the thread
https://github.com/notifications/unsubscribe-auth/AGCQKYXf6T1Q-r925Du7MWSU7uFASB6Dks5td-MTgaJpZM4So4G9
.

--
Regards,
Deepak

@deepakkumar1984, Of course. Running this example is how I know that your software isn't working. I have CNTK GPU working on several projects but yours won't work. How can I debug it?

@deepakkumar1984, I thought I had it figured out. I thought the problem was that your code only supports CUDA 9.0. I have CUDA 8.0 and CUDA 9.1. But when I rolled back to CNTK 2.3.1 GPU which works on CUDA 8.0, it still doesn't detect the GPU. I'm not sure what's going on now.
Is there really no way to troubleshoot GPU detection?


cannot copy snapshot locally but to remote server ok

Required information

Distribution: Ubuntu
Distribution version: 16.04
The output of "lxc info" or if that fails:

Kernel version: 4.4.0-31-generic
LXC version: 2.0.3
LXD version: 2.0.3
Storage backend in use: ZFS

Issue description
A simple lxc copy from snapshot into a new container that works on our other hosts fails on this one for all containers and snapshots that we tried (at least 5 random ones out of 250).
However what's interesting is that sending these snapshots to a remote LXD server works perfectly, we also tested the copied container's data consistency and it's fine!
root@lxd-us:~# lxc copy PqC-testcontainer/testcontainer-1470606782-autobackup testcontainer-1470606782-autobackup
error: rsync failed: rsync: change_dir "/var/lib/lxd/snapshots/PqC-testcontainer/testcontainer-1470606782-autobackup/rootfs" failed: No such file or directory (2)
rsync error: some files/attrs were not transferred (see previous errors) (code 23) at main.c(1183) [sender=3.1.1]

Edit/Update:
Strange, when adding the server to itself (!) with lxc add remote, the copying works "locally" too...!

Update: on this same machine the following is happening:
du -sh /var/lib/lxd/containers --exclude=/var/lib/lxd/containers/*.zfs
22G	/var/lib/lxd/containers

That's obviously not right, all other machines report a few KB tops for the same command.
What's going on here?
Thanks!

Is the disk full perhaps?

Did you have containers created on that machine before you enabled zfs?

@tych0 nope, it's definitely not.
@stgraber at one time we had to re-import the ZFS pool after rebuilding the server. All containers were stopped prior to the zpool import, and after it all of them have been confirmed to have all the files/folders and general data consistency.

I'm unable to reproduce this here. Also with 2.0.3 on Ubuntu 16.04 with 4.4 kernel and using the ZFS backing storage:
stgraber@dakara:~$ lxc launch ubuntu:16.04 PqC-testcontainer
Creating PqC-testcontainer
Starting PqC-testcontainer
stgraber@dakara:~$ lxc snapshot PqC-testcontainer testcontainer-1470606782-autobackup
stgraber@dakara:~$ lxc copy PqC-testcontainer/testcontainer-1470606782-autobackup testcontainer-1470606782-autobackup

What am I missing?

Can you provide some more details on this?

No, sorry. It was a one time thing and we can't replicate it. The source of the issue was that we rebuilt the host machine and imported the ZFS pool, maybe LXD wasn't stopped prior to that, ZFS wasn't recognized correctly or something. Let's close this now and I'll get back to you as soon as we bump into this again! Thanks.


Fix metric evaluation without inverted target transforms bug

The problem was that metrics were evaluated using the transformed
targets/predictions after performing target transformation
(QuantileTransformer) via EngineerStep

This resulted in metrics that were way too impressive compared
to Experiments without target transformations because they
were being calculated in the transformed target space, rather
than the original/inverted space

To address this, Environment.__init__ has a new
save_transformed_metrics kwarg--moved from settings.G
Default for save_transformed_metrics is chosen based on the
dtype of train_dataset's target data
SparkNotes version of documentation:

If all targets are numeric, save_transformed_metric=False
Else, save_transformed_metric=True

Pull Request Test Coverage Report for Build 749

27 of 27 (100.0%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.2%) to 94.062%

Totals

Change from base Build 747:
0.2%

Covered Lines:
4657

Relevant Lines:
4951

💛  - Coveralls


Update Set-Place.md

Removed SpaceType parameter per email request

Docs Build status updates of commit c14de45:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/Set-Place.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit bfc1c55:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/Set-Place.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


add option to display primary email on public profile

The basics are there, including tests, but i'm not sure if we want to wrap the email address in a mailto:, or how we want to handle emails that are too long.
Fixes #5944.

Sorry, this is for #5944.

Also, I think I'm supposed to run make build-mos to update everything after adding a new string that needs translation, but it looks like there's some other strings getting changed in the diffs as well.  Just wanted to make sure that's to be expected before committing and pushing that.

or how we want to handle emails that are too long.

I don't think you need to handle this in this PR, I created #6880 to capture this.

As for more tests, I'm happy to, but I didn't see any existing tests that let the views run and mutate db entries, so I wasn't sure where it went.  The stuff in tests/unit/manage/test_views.py looks like it's going out of its way to avoid actually letting the view execute, so it seemed like the wrong place.  Maybe somewhere under tests/functional?

or how we want to handle emails that are too long.

I don't think you need to handle this in this PR, I created #6880 to capture this.

Thanks.

I took a stab at a test verifying that it makes the right changes to the User object.  This is my first time with pyramid...not sure if there's a cleaner way to call the view that would be preferrable to what I've hacked up so far.

Just a couple thoughts on this implementation:

Rather than displaying the Primary email address, we should probably allow a user to select any Verified email address. I know for instance I would prefer to publicize my Verified @python.org email address, but prefer to keep a separate personal email address as my Primary.
If possible, only displaying the email address to logged in users might be a good idea? I think the concern there would be for caching user pages.

I also decided in #6880 that the behavior for a long email should be different (it should truncate with an ellipsis), so we should address that in this PR.

I've got this set up, but I'm noticing two things:

the trailing envelope icon that seems to get added to all mailto links gets ellipsized along with the rest of the email address

i hadn't noticed that gets added before...do we want to remove the leading icon on the left?

when it needs to be ellipsized, the email address gets bumped down to the line below the leading envelope icon.  my css isn't all that good, i'm not sure how to fix that.

Just a couple thoughts on this implementation:
* Rather than displaying the Primary email address, we should probably allow a user to select any Verified email address. I know for instance I would prefer to publicize my Verified @python.org email address, but prefer to keep a separate personal email address as my Primary.

Sounds reasonable.  Default to Primary?

* If possible, only displaying the email address to logged in users might be a good idea? I think the concern there would be for caching user pages.

With the proviso that I don't know the codebase well, it doesn't look like there's a way to indicate that the user profile page should only be cached, and served from cache, for non-logged-in users.  If that's the case, is it worth giving up the cache in order to hide the email from non-logged-in folks?

Sounds reasonable. Default to Primary?

I think it'd be reasonable to just default to "not displayed" and force the choice when enabling.

With the proviso that I don't know the codebase well, it doesn't look like there's a way to indicate that the user profile page should only be cached, and served from cache, for non-logged-in users. If that's the case, is it worth giving up the cache in order to hide the email from non-logged-in folks?

This would need to be done via a Client Side Include. An example would be the text on the profile page when the user has no packages, which changes for logged-in users (to do something different on their own profile):
Default, wrapped in a csi block:
https://github.com/pypa/warehouse/blob/9000661ff05e6d7f2cb139abb0ebe55844b3ae90/warehouse/templates/accounts/profile.html#L74-L78
The route:
https://github.com/pypa/warehouse/blob/92b28e69d2cb9af3456d2be047610ec955bc3bc5/warehouse/accounts/views.py#L686-L694
The template that replaces it:
https://github.com/pypa/warehouse/blob/92b28e69d2cb9af3456d2be047610ec955bc3bc5/warehouse/templates/includes/accounts/profile-callout.html#L15-L25

Sounds reasonable. Default to Primary?

I think it'd be reasonable to just default to "not displayed" and force the choice when enabling.

Oh, yeah, sorry if that was unclear.  I meant once you'd checked the "make my email public" checkbox.
Thanks for the explainer on client side includes!

Looks like this is still missing the ability to set a non-primary verified email address as the public address.
I think the checkbox should probably become a dropdown with the ability to select either the default "Not displayed" or any primary address.

Yeah, I was still waffling about the UI and backend options.  Would you recommend adding a public column to the email table, or switching the User.is_email_private column to a nullable User.public_email column instead?

I think it might make the most sense to handle it just like we do for primary email addresses now: a public column to the email table.

I think this is all set, let me know if I missed anything.

Thanks for your patience here @rascalking!

Thanks for all the help!


github.com/schollz/croc/v6 installed as "v6" rather than "croc"

I used gobin to install this binary, but it was installed as v6 (the version) instead of the actual binary name, croc.
Installed<EMAIL_ADDRESS>to /home/jake/dotfiles/gotools/bin/v6

Thanks for the report. This is fixed in #83.

Fixed in #83, released in https://github.com/myitcv/gobin/releases/tag/v0.0.12. Thanks again for the report.

You can use gobin to upgrade itself:
$ gobin -u github.com/myitcv/gobin
Installed<EMAIL_ADDRESS>to /home/myitcv/bin/bingo/gobin


test: add live docker example

This PR adds a live workflow to demonstrate the use of cypress/browsers from Cypress Docker images with the Cypress GitHub Action. This is already described in README > Docker image.

Workflows are successful:

Updated README section for review on https://github.com/MikeMcC399/github-action/blob/add/docker-example/README.md#docker-image

Status badge
will go live after merge


Test Failure

Fixes #47
@robrix You got a favorite tautology you want me to negate in here?

:boat:

Thanks @CodaFi! :heart:


Embed feature: confirmation question asking for output

From the previous UX study, one proposed idea was showing an interactive code example and asking learners about the output. After learners choose an option, it shows an answer with more explanation.
Goal

Encourage users to think through the code example
Help learner better understand the code with further explanation

Prototype

Result
7 out of 10 participants like having this question. They found this “informative, makes me think a little bit more, explains why”. The explanations shown after choosing the correct option also better explain the code.
Confusion
Most participants were unsure if they were supposed to run the code first  or to answer the question first. A few felt this is cheating or redundant if they can run the code before answering it.
P05 didn’t immediately notice the explanation because it was out of the view
Improvement

Move the question above the editor
Set expectation by telling learners what to do: observe it and answer the question first. Then run the code
After answering, automatically scroll down a little to show the explanation section

Status of this issue is also tracked in https://github.com/orgs/flutter/projects/3.

As part of the new, simplified version of the DartPad frontend, the exercise functionality this was related to was removed.
If anyone would like to see similar functionality brought back, please check out https://github.com/dart-lang/dart-pad/issues/2702 or open a new issue. Thanks so much!


[5.0][CodeCompletion] Eat identifier characters after CC position

(Cherry-pick of #20959 for swift-5.0-branch, Not critical for convergence. Not ABI-affecting.)
Text in editors may contain identifier characters following to CC position which should be considered as "filtering text" for the code completion.
Eat identifier characters after CC token in Lexer. By this change, for instance, for @<token>IB class, the parser now can detect this is a start of class declaration and emit class attributes for code completion.
rdar://problem/46103294

@swift-ci test and merge


BitmapIcon does not work

Hello,
I have tried to put a custom icon into a navigationViewItem
I get Access Denied in LoadComponent of the page.
Does BitmapIcon works? How i can put a custom icon in NavigationViewITem
        <muxc:NavigationViewItem Content="Designs" Tag="Designs" x:Name="Designs" SelectsOnInvoked="True">
            <muxc:NavigationViewItem.Icon>
                <BitmapIcon UriSource="ms-appx:///Assets/globe.png"/>
            </muxc:NavigationViewItem.Icon>
        </muxc:NavigationViewItem>

Can you please provide a repro project ?

Hello, i was preparing the project and it worked. sorry for the
inconvenience.
On Fri, 13 Dec 2019 at 21:14, Ranjesh<EMAIL_ADDRESS>wrote:

Can you please provide a repro project ?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/microsoft-ui-xaml/issues/1755?email_source=notifications&email_token=AAIXS4MLRGB4B2ED5HD77YLQYPNKRA5CNFSM4J2JVG62YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEG27AZQ#issuecomment-565571686,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAIXS4JHCLBMB7T4OSSOPPTQYPNKRANCNFSM4J2JVG6Q
.

Hello,
I'm testing two projects:
WinUI 3 with Microsoft.WinUI nuget.
WinUI 3 with Microsoft.ProjectReunion nuget.
Both are running in VS 2019 Preview.
I have followed these docs https://docs.microsoft.com/en-us/uwp/api/Windows.UI.Xaml.Controls.BitmapIcon?view=winrt-19041
The image is in a folder called Assets.
I've tried to place a BitmapIcon in:
<AppBarButton >                 <AppBarButton.Icon>                     <BitmapIcon UriSource="ms-appx:///Assets/Test.png"/>                 </AppBarButton.Icon>  </AppBarButton>
and in:
<NavigationViewItem>                     <NavigationViewItem.Icon>                         <BitmapIcon UriSource="ms-appx:///Assets/Test.png"/>                     </NavigationViewItem.Icon> </NavigationViewItem>
I cannot get the Icon to display in either case.
I have tested this in a UWP project and the icon displays without issue.


Add an inline_wasm! macro or make inline_asm! work on wasm targets

For my use cases, this would have been nice, but not strictly necessary. I expect it will eventually become higher priority for someone.

I'd also be interested in inline web assembly as a corollary to what the test::black_box() (in libtest) function does, but for wasm32 targets, i.e. as a way to disable optimisations.

we probably want to track this in a way that's different than an issue on this repo. going to close (but please reopen if i am incorrect and also apologies!)


Request: Videoplayer in Nativescript 3.0 for Breaking Changes

Hi,
I just upgraded my project to {N} version 3.0.
Since then, It is no longer possible to use your Videoplayer plugin. I am using NativeScript Angular so I registered Video element as you wrote and when I want to run video inside app this error occurs:
Failed to find module: "ui/core/proxy", relative to: app/tns_modules/
I assume it is because of this:

Bindable(“ui/core/bindable” module), ProxyObject (“ui/core/proxy” module) and DependencyObservable ("ui/core/dependency-observable" module) are removed.
Consider using View, ViewBase or Observable instead.

as is wrote in: https://github.com/NativeScript/NativeScript/blob/master/Modules30Changes.md
Can you please help me resolve this issue?
P.S.: before {N} version 3.0 everything worked fine

Hi @KuboStraka - this plugin has not been updated for NS 3+. I'll accept a PR that handles the breaking changes and provides the updates if you want. So for now this plugin will not work on 3+ until it's updated.
Let me know if you're interested in contributing for this.
Thanks

Once 3.0 is officially released we'll do some work to make it compatible.

Hi @KuboStraka this has now been updated to 3.0 .. please test and advise if any issues, thank you.

Hi @bnussey that's great news :). I will test it and let you know.
Thanks again guys for all that great work .


Deformation for view direction?

Hi, author!
Both nerfies and non-rigid nerf use deformation field. However, in non-rigid nerf they recompute view direction after deformation. How does nerfies deal with it? I quickly reviewed your implementation and did not find related operations. Looking forward to your reply!

Similar concerns here.
Quite curious why not apply deformation to the view directions?
Could you explain a little bit? @keunhong
Looking forward to you reply

Hi, sorry for the late response.
This is a very good question. Our method uses observation-frame view directions which remain correct after deformation. With the appearance latent code, this provides a means to somewhat explain view dependent changes. However, you're right that since the scene geometry deforms, the reflected colors should change as well. To truly model view dependence, we'd have to consider local geometry e.g., by using normals to compute reflected ray directions. I'm sure future work will try to do this :)


Plugin sometimes stops working after combining part files.

I've had this happen to me twice today while testing.  I'd say on my machine (MacOS 12.5.1, KiCad 8.0.1) it happens about 10% of the time I try to download a database.  What spurred me to open the issue was that I got the same problem while running in standalone mode, so I have a stack trace:
starting jlcpcbtools standalone mode...
2024.04.25 10:31:23 - DEBUG - download -  Parts db is split into 8 parts. Proceeding to download...
2024.04.25 10:31:23 - DEBUG - download -  Removing any spurios old zip part files...
2024.04.25 10:31:23 - DEBUG - download -  Removing /Users/me/Documents/KiCad/8.0/scripting/plugins/bouni-kicad-tools/jlcpcb/parts-fts5.db.zip.001.
2024.04.25 10:31:23 - DEBUG - download -  Removing /Users/me/Documents/KiCad/8.0/scripting/plugins/bouni-kicad-tools/jlcpcb/parts-fts5.db.zip.002.
2024.04.25 10:31:23 - DEBUG - download -  Removing /Users/me/Documents/KiCad/8.0/scripting/plugins/bouni-kicad-tools/jlcpcb/parts-fts5.db.zip.003.
2024.04.25 10:31:23 - DEBUG - download -  Download parts db chunk 1 with a size of 76.29MB
2024.04.25 10:31:25 - DEBUG - download -  Download parts db chunk 2 with a size of 76.29MB
2024.04.25 10:31:28 - DEBUG - download -  Download parts db chunk 3 with a size of 76.29MB
2024.04.25 10:31:30 - DEBUG - download -  Download parts db chunk 4 with a size of 76.29MB
2024.04.25 10:31:32 - DEBUG - download -  Download parts db chunk 5 with a size of 76.29MB
2024.04.25 10:31:34 - DEBUG - download -  Download parts db chunk 6 with a size of 76.29MB
2024.04.25 10:31:37 - DEBUG - download -  Download parts db chunk 7 with a size of 76.29MB
2024.04.25 10:31:39 - DEBUG - download -  Download parts db chunk 8 with a size of 17.90MB
2024.04.25 10:31:39 - DEBUG - download -  Combining and extracting zip part files...
Traceback (most recent call last):
  File "/Users/me/Documents/KiCad/8.0/scripting/plugins/bouni-kicad-tools/mainwindow.py", line 562, in populate_footprint_list
    if not self.store:
AttributeError: 'JLCPCBTools' object has no attribute 'store'

NOTE: the GUI remains responsive, there is no beachball, but I never get the success message.
The referenced line contains the following code:
if not self.store:
    self.init_store()

Smells like a race condition.  I don't know anything about how python wxwidgets interacts with multithreading, but I believe the error must be something along the lines of this: https://stackoverflow.com/a/31701465

Update, sometimes the success message does manage to show up in spite of the exception.

Is this still an issue or has #455 fixed this as well?


bad failure

new-sprlist should ahve a better failure message all the time but especially if a duplicat elist is attempted

also the exporter wtf


Chore: Add schema for values.yaml

Overview
Since the users are going to define values file, it make sense to have a schema against which we check the config file.
Helm v3 has support for JSON schemas that we can use, https://www.arthurkoziel.com/validate-helm-chart-values-with-json-schemas/
Design
Need the schema to have the following functionality

Have enums for valid chain and relayers names
Valid options for rest for any values config file

Since this was not properly done, have to redo the creation of schema with the latest values of the charts.


Redesign

This is a big PR. Sorry. :/
Brings in the new design we have been working on.
Still lots of things to tidy up but I want to get the new design merged so we start getting feedback, it also makes it much easier for people to contribute.
Changes:

Most content moved to mark down files outside of this repo.
Using speclate to build the html files. See bin/build-markup.js
Using serve-me for local dev env (npm start)
Using standard for linting (https://www.npmjs.com/package/standard).
Bootstrap removed and minimal css thanks to @maxwellito.
new design!!!
Lots of files have been moved around to make it easier to navigate

I’ve left some pretty simple stuff in there to do, the plan being to create tickets and mark them as for beginners.
Speclate also needs to be properly documented. Thats one for me to do. :)
When (if!) you approve this PR please leave a comment saying which browsers you tested in.
Please see README for setup instructions.
Thanks

holy moly!!!

good work tho gents

OSX Safari - Version 9.0.1 (116<IP_ADDRESS>)

OSX Chrome - Version 47.0.2526.73 (64-bit)

lots of small ux imporvements to be made :-)

OSX - Firefox 38.0.1

ok tested - turns out i dont understand breakpoints :-/
this is a +1 from me

merged to get things moving


Fix javadocs

Since java 8 went mainstream today, I thought I might fix all of our java docs, so we could build without passing the skip docs command.

Doclint disabled, issue solved.


Removed unwanted fieldgroup margin

What is the context of this PR?

Removed bottom margin from fields within field group.  This was causing inconsistent spacing noticeable in mutually exclusive examples: date, duration.

How to review
Check spacing in all field groups

 Issue #1556: Inconsistent vertical spacing between duration and date field groups and mutually exclusive checkbox


build: increase jax upper bound

https://github.com/instadeepai/jumanji/blob/ed13d82e2a1c4f734fabe41b173411467e737606/requirements/requirements.txt#LL4C17-L4C17
Is it possible to change this to be jax < 0.5?

Hi, this seems to be a duplicate of #168

Resolve in #174


Implement mure edit subcommand

This command will open the file in the editor.

Codecov Report
Attention: 35 lines in your changes are missing coverage. Please review.

Comparison is base (5987f7c) 77.72% compared to head (fcbb97c) 77.19%.

:exclamation: Current head fcbb97c differs from pull request most recent head bf3aa64. Consider uploading reports for the commit bf3aa64 to get more accurate results

Files
Patch %
Lines

src/app/edit.rs
65.55%
31 Missing :warning:

src/main.rs
0.00%
4 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #418      +/-   ##
==========================================
- Coverage   77.72%   77.19%   -0.53%
==========================================
  Files          17       18       +1
  Lines        1746     1859     +113
==========================================
+ Hits         1357     1435      +78
- Misses        389      424      +35

Flag
Coverage Δ

unittests
77.19% <69.02%> (-0.53%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

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


[bug-787]: Write status code to csi-powerscale

Description

Write the status code back to the driver for csi-powerscale
Log the error when validating the access token

GitHub Issues
List the GitHub issues impacted by this PR:

GitHub Issue #

https://github.com/dell/csm/issues/787

Checklist:

[x] I have performed a self-review of my own code to ensure there are no formatting, vetting, linting, or security issues
[x] I have verified that new and existing unit tests pass locally with my changes
[ ] I have not allowed coverage numbers to degenerate
[ ] I have maintained at least 90% code coverage
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] Backward compatibility is not broken

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Please also list any relevant details for your test configuration
Added unit test for failed token validation for csi-powerscale.

e2e pass

e2e pass


Generic "is" Operator

There are a bunch of equality comparisons in compiler that could be replaced with "is".

The compiler can't use this until constructors are zeroed.


h264dec: only fill referenced dpb pictures for vaapi

Only fill the VAAPI picture reference list with pictures
that are currently referenced in the dpb.  Loading
unreferenced dpb pictures in VAAPI is unnecessary and
only adds to the overhead.
Signed-off-by: U. Artie Eoff<EMAIL_ADDRESS>

One benefit to pass all dpb pictures is it's very convenient to check dbp list in va_trace log. I agree to don't load unreference pictures to ReferenceFrames[16] since it named ReferenceFrames.  But it should have little overhead.

lgtm :+1:

Hi @lizhong1008
So, do you think it's ok to merge this patch or you have other suggestion.

LGTM. Would better don't mention “overhead” in the comment message. The others may think this patch will have obvious benefit to performance, but actually there are litter performance benefit.

Agree, remove “overhead”.
Lgtm

On Jul 20, 2016, at 00:19, lizhong1008<EMAIL_ADDRESS>wrote:
LGTM. Would better don't mention “overhead” in the comment message. The others may think this patch will have obvious benefit to performance, but actually there are litter performance benefit.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/01org/libyami/pull/510#issuecomment-233863167, or mute the thread https://github.com/notifications/unsubscribe-auth/AA0coV9Oy7pig5q67h_rlHwU13O70XVHks5qXcvsgaJpZM4JPZqF.

"overhead" removed from message.

+1

+1

merged as a09e9cba523e40852785d0c3f1b414c0169c208a


Change ordering for Contacts in admin site

Newest first

Current coverage is 100% (diff: 100%)

Merging #16 into master will not change coverage

@@           master   #16   diff @@
===================================
  Files          10    10
  Lines         208   208
  Methods         0     0
  Messages        0     0
  Branches        8     8
===================================
  Hits          208   208
  Misses          0     0
  Partials        0     0

Powered by Codecov. Last update 732484c...53b568c


Please use another AD App password, gives errors in PowerShell

Module: 07
Lab/Demo: 01
Task: 00
Step: 00
Description of issue
az ad app create --display-name "TimeBot" --password "Super$ecretPassw0rd" --available-to-other-tenants
Please don't use "Super$ecretPass0rd". When student use powershell in Visual Studio code, or just PowerShell, Powershell doesn't like $ character in double quote string.
It give strange behaviour when deploying the timebot
az deployment group create --resource-group "YOUR_RESOURCE_GROUP" --template-file "deploymenttemplates/template-with-preexisting-rg.json" --parameters appId="YOUR_APP_ID" appSecret="Super$ecretPassw0rd" botId="A_UNIQUE_BOT_ID" newWebAppName="A_UNIQUE_WEB_APP_NAME" newAppServicePlanName="A_UNIQUE_PLAN_NAME" appServicePlanLocation="REGION" --name "A_UNIQUE_SERVICE_NAME"
I have replaced double quotes into single quotes, but that ended up with single quotes in the Web App configuration.
My suggestion: replace $ with 5
--password "Super$ecretPassw0rd"  -> --password "Super5ecretPassw0rd"
Repro steps:

Visual studio code with PowerShell terminal
PowerShell

Lab instructions are to do this using the CLI, not PowerShell. If you use PowerShell, you'll need to escape the reserved $ character.


TreeView reveal creates a race condition with TreeDataProvider#getChildren

If the tree provider is loading the tree and TreeView#reveal is revealing an element that is not loaded yet (so the parents are loaded) then a race condition can cause the view to break with the message "Element with id X is already registered".
I have also tried to create a promise queue as suggested in this comment but the issue still exists. I think this issue is caused by reveal and refresh of the tree, like it's not reveal statements causing it but rather the tree is not synchronized with reveal.
See the following example repo (based on the tree view sample): https://github.com/EhabY/TreeViewRevealExample/tree/main

Run the extension
Run the command Test View: Reveal

https://github.com/microsoft/vscode/assets/12927950/4ec40555-aefc-4ab1-adb1-569fa418e716
For reference this what the executed command does:
// refresh the parent
provider.refresh();

// Reveal the child which will load the parent
this.promiseQueue = this.promiseQueue.then(async () => view.reveal({ key: "aa" }, { focus: true, select: false }));

// refresh the parent
provider.refresh();

Pinging @alexr00 so it does not get lost 🙏

@EhabY the way for you to solve this from your end would be to keep track of if your tree is currently refreshing and then wait to do the reveal until after the refreshing has finished.

@alexr00 How is it possible to track if the tree is refreshing? IIRC the refresh events do not return a promise so I would need to keep track of all "open" nodes and then make sure that all of those open nodes have been reloaded (making sure to handle deleted nodes). Is there a better approach for this?
But even then, this would reduce the possibility of it happening instead of eliminating it which requires internal syncing.

How is it possible to track if the tree is refreshing?

I was thinking you could track if you have an unfinished getChildren method that was called.

Ah, I actually tried that but it didn't work because refreshing would call getChildren in a recursive manner:

getChildren(undefined)
getChildren(root)
getChildren(child1)
...

So while we can easily check if a getChildren is being executed, we cannot know for sure when it has finished executing all calls unless we manually track all visible tree items (which can be error prone..)

So while we can easily check if a getChildren is being executed, we cannot know for sure when it has finished executing all calls unless we manually track all visible tree items (which can be error prone..)

I think this is still possible.

I just attempted a solution as you described and it still fails, I have added a counter that is incremented first thing in getChildren and a decremented last thing in getChildren and I get the following trace.
"Before" is printed first thing in getChildren, "During" is in the middle,"After" is printed last thing, "Done!" is printed when the counter is 0 and the created promise when refreshing is resolved.
Before:  0
During:  1
After:  0
Done! -- Finishes after only refreshing the root and not the rest of the tree
Before:  0
Before:  1
During:  2
After:  1
Before:  1
During:  2
After:  1
During:  1
After:  0
Done!

Notice how when I execute the command above, it refreshed the top level first (and finishes execution) before refreshing the rest of the tree - thus depending on the counter won't work as it finishes a refresh of the root before refreshing the rest.


⚠ Bofrost.es has degraded performance

In e500af8, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1653 ms

Resolved: Bofrost.es performance has improved in a0e4489 after 8 minutes.


the url pattern is not conform to the URI generic syntax in the newest version 3.1.0

Version
3.1.0
Reproduction link
https://github.com/yangshuan/vue-router-test-3.1.1
Steps to reproduce
Steps are very simple:

Use the vue-cli to generate a new empty project;
Start the project;
Open a browser (e.g. Chrome), and open the application index page;
Everything seems normal, but if you use url with parameters, e.g. http://localhost:8080/?a=1, the page will goto http://localhost:8080/#/?a=1, the uri is not right, it should be 'http://localhost:8080/?a=1#/'

What is expected?
The URI should always be conform to the URI generic syntax 'scheme:[//authority]path[?query][#fragment]', otherwise most of the lib or code which get the parameters from the URI will fail.
What is actually happening?
this url is not conform to the URI generic syntax which is 'scheme:[//authority]path[?query][#fragment]';
The problem the 3.1.0 version brings out is that, when we want to get the parameters from the URL by the default URI generic syntax, we fail all the time, since the uri is not right.

The URI syntax refers to https://en.wikipedia.org/wiki/Uniform_Resource_Identifier#Definition

Actually this issue is brought out from 3.0.0

This was introduced to easily allow some oath services like GitHub as seen in https://github.com/vuejs/vue-router/issues/2125
I'm happy to revert the commit and provide a workaround if this is breaking other libs but I would need a real example that breaks not a link to the URI definition. Did this break anything for you?

Hello @posva .
I work on the wordpress site and on the admin side url to post page looks like this: /wp-admin/post.php?post=123
And for me routes like /wp-admin/post.php?post=123#/home were fine.
But after update vue-router to the new version this was changed to /wp-admin/post.php#/home?post=123 and this breaks everything.
(For example, if I click on tab "details", url will be changed to /wp-admin/post.php#/details, which is in fact another page)
I tried to set base param in config to window.location.pathname + window.location.search, but it didn't helped.
Can you please advice to me any workaround? Thanks in advance.

Interesting, I really thought nobody was using that. Since the query can be accessed on server while the hash cannot, I think it makes sense to revert this behavior since you could theorically send different pages from the server with a different query
The workaround is for the other issue, not for this one

since you could theorically send different pages from the server with a different query

In case of wordpress, it's really so, because here everything is a "post": products, orders, news, etc.
And on the admin side url always looks like /wp-admin/post.php?post=123, but depending on post id page looks absolutely different.

Do you know of wordpress plugins using Vue Router like this? I would like to search a bit more because I'm curious

I suspect, that my case is widespread, because in general I've seen not a lot of examples of using Vue in wordpress plugins.
For combination of using vue-router on the admin side I found only this boilerplate: https://github.com/0aveRyan/wordpress-day-spa
But I searched only briefly, because I needed just an example, whether it's possible at all.
Thank you for the help!

This was introduced to easily allow some oath services like GitHub as seen in #2125
I'm happy to revert the commit and provide a workaround if this is breaking other libs but I would need a real example that breaks not a link to the URI definition. Did this break anything for you?

The lib url-parse (refer to https://www.npmjs.com/package/url-parse) will be break if the uri doesn't follow the uri syntax, you can try it out.
Recently all of my applications in my company met this issue. we have a SSO service which has its own login page, and it sends parameters in the URL. when the other applications integrate with the SSO, code 'window.location.search' helps us to get the parameters from the url for login usage. Unfortunately after upgrade to 3.3.0, everything failed.
Another case is that in Java the URI syntax is '[scheme:][//authority][path][?query][#fragment]'(refer to https://docs.oracle.com/en/java/javase/12/docs/api/java.base/java/net/URI.html), sometimes I need to send the URL to my backend which is built on Java, I can't use the default method to parse the URI generated by vuejs.

I agree parsing the uri in string by ourselves is a work around for this, but following the uri syntax can save a lot time for everyone.
Please think about it, it really help. Thanks.


Incorrect order of messages when polling is used

There is an incorrect order of messages (sent by api.request(SendMessage)), when the polling method is used. How to keep sent messages in order? Is there any way to fix that without using webhooks?

Yes that's the expected behavior, but I see your point, basically you want to enforce a sequential order of requests (maybe for some special part, but not necessarily for everything), could you give me more details? how it breaks? and what is exactly your use case?

Fixed


feat: warn user about missing required dependencies in generated client

Infer client only if all dependencies are present. Log warning to user about missing dependencies that are required.

@mrlubos I have updated the dependencies to match your issues

If you really want this, then if I have just angular installed, it should infer angular and show warning. This implementation would infer fetch and show no warnings (potentially). Again, the primary functionality here is to remove the need to specify client, not to do dependency checks

Addressed in latest commit


Support warnings about potential accessibility issues with Vue components.

What problem does this feature solve?
This will help point out any accessibility flaws with vue components during development. This may not be a core library feature, it could be a set of eslint rules that the CLI generates during scaffold.
What does the proposed API look like?
Could be an eslint plugin, or a different module similar to react-a11y

We only keep core feature requests here - I'd suggest moving this to eslint-plugin-vue instead.


Add reset form state helper. Closes #41

@all-contributors please add @medson10 for code and ideas


Hacking into Android Gradle Plugin is really terrible...!

Terrible transform API...
Terrible proguard-inside integration...
Terrible "Jack maybe used, skip classes generation"...
After version 2.3.0 is updated,
variant.dirname has nothing relation with base bundle dir.
New build-cache is brought in yet specific build-cache dir is not provided by public api.
The code in android plugin is quite designed, jump here, and jump there.


Allow user to dump internal payload files

Motivation
For some tools it can be useful to know what is inside the payload embedded with Sorbet.
One could read the payload from Github but this payload might not be exactly corresponding to the one embedded in the version of Sorbet used.
This PR adds the --print=payload-sources option which writes the content of the payload in RBI files inside the specified directory:
$ sorbet --print=payload-sources=my-dir

Test plan
See included automated tests.

What do you think about using --print for this? Using --print will get some things for free, like the ability to specify an output file for the print output.
--print=payload-sources:out.rb

You can also look at the AutogenAutoloader / --print=autogen-autoloader option to see an example of how to use the --print flag in combination with an output directory.

@jez: Updated to use --print=payload-sources instead 👍

Two questions. If you'd like to decline both of them lmk and I'll click merge.

@jez:

added a check to reject --no-stdlib with --print=payload-sources
using trap EXIT
using mktemp -d


JmeterNamesTemplater v1.1 and UtilityJmeterFunctions v1.1.3

Created JmeterNamesTemplater that allows you to rename items according to the config file

Hi!
I also added another plugin (CustomJmeterFunctions), with your recommendations

Please make the title of PR to reflect what is inside. Look at other PRs to get the idea

Looks like your JSON violate descriptor schema: https://github.com/undera/jmeter-plugins/actions/runs/10546658418/job/29218873882?pr=665

Looks good now


V3.3.x

@robinwyss Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

Was openend by mistake


Possible? failure when the upstream registry is down

After a successful execution of EIB I found my system wasn't deployed correctly as metallb wasn't working:
36m         Warning   Failed                 pod/metallb-controller-7cbd6cf47d-2qrh6    Failed to pull image "registry.opensuse.org/isv/suse/edge/metallb/images/metallb-controller:v0.13.10": rpc error: code = Unknown desc = failed to pull and unpack image "registry.opensuse.org/isv/suse/edge/metallb/images/metallb-controller:v0.13.10": failed to resolve reference "registry.opensuse.org/isv/suse/edge/metallb/images/metallb-controller:v0.13.10": failed to do request: Head "http://localhost:6545/v2/isv/suse/edge/metallb/images/metallb-controller/manifests/v0.13.10?ns=registry.opensuse.org": dial tcp [::1]:6545: connect: connection refused

Turns out, hauler wasn't serving anything:
# journalctl -u eib-embedded-registry
Mar 14 12:27:20 host1rke2 systemd[1]: Starting Load and Serve Embedded Registry...
Mar 14 12:27:21 host1rke2 hauler[2047]: 12:27PM INF loading content from [embedded-registry.tar.zst] to [store]
Mar 14 12:27:21 host1rke2 systemd[1]: Started Load and Serve Embedded Registry.
Mar 14 12:27:21 host1rke2 hauler[2091]: 12:27PM ERR Error: signed image index: stat /opt/hauler/store/index.json: no such file or directory
Mar 14 12:27:21 host1rke2 hauler[2091]: 12:27PM ERR main.go:74: error during command execution: signed image index: stat /opt/hauler/store/index.json: no such file or directory
Mar 14 12:27:21 host1rke2 hauler[2091]: Error: exit status 1
Mar 14 12:27:21 host1rke2 hauler[2091]: Usage:
Mar 14 12:27:21 host1rke2 hauler[2091]:   hauler store serve registry [flags]
Mar 14 12:27:21 host1rke2 hauler[2091]: Flags:
Mar 14 12:27:21 host1rke2 hauler[2091]:   -c, --config string      Path to a config file, will override all other configs
Mar 14 12:27:21 host1rke2 hauler[2091]:       --directory string   Directory to use for backend.  Defaults to $PWD/registry (default "registry")
Mar 14 12:27:21 host1rke2 hauler[2091]:   -h, --help               help for registry
Mar 14 12:27:21 host1rke2 hauler[2091]:   -p, --port int           Port to listen on. (default 5000)
Mar 14 12:27:21 host1rke2 hauler[2091]: Global Flags:
Mar 14 12:27:21 host1rke2 hauler[2091]:       --cache string       Location of where to store cache data (defaults to $XDG_CACHE_DIR/hauler)
Mar 14 12:27:21 host1rke2 hauler[2091]:   -l, --log-level string    (default "info")
Mar 14 12:27:21 host1rke2 hauler[2091]:   -s, --store string       Location to create store at (default "store")
Mar 14 12:27:21 host1rke2 hauler[2091]: 12:27PM ERR exit status 1
Mar 14 12:27:21 host1rke2 systemd[1]: eib-embedded-registry.service: Deactivated successfully.

The embedded-registry.tar.zst file was empty:
# ls -lh embedded-registry.tar.zst
-rw-r--r--. 1 root root 88 Mar 14 12:26 embedded-registry.tar.zst

The build was successful:
$ eib build --config-dir ${CI_PROJECT_DIR}/eib-temp --config-file eib.yaml --build-dir ${CI_PROJECT_DIR}/eib-temp/_build
SELinux is enabled in the Kubernetes configuration. The necessary RPM packages will be downloaded.
Downloading file: rancher-public.key 100% |█████| (2.4/2.4 kB, 14 MB/s)
Generating image customization components...
Identifier ................... [SUCCESS]
Custom Files ................. [SUCCESS]
Time ......................... [SKIPPED]
Network ...................... [SUCCESS]
Groups ....................... [SKIPPED]
Users ........................ [SUCCESS]
Proxy ........................ [SKIPPED]
Resolving package dependencies...
Rpm .......................... [SUCCESS]
Systemd ...................... [SUCCESS]
Elemental .................... [SKIPPED]
Suma ......................... [SKIPPED]
Embedded Artifact Registry ... [SUCCESS]
Keymap ....................... [SUCCESS]
Configuring Kubernetes component...
 [SUCCESS]
Certificates ................. [SKIPPED]
Building ISO image...
Kernel Params ................ [SKIPPED]
Image build complete!

I think it is maybe related to the fact that the upstream registry where the images were mirrored was down... but I'm not 100% sure about this.
I'm opening this just in case we want to investigate more.

Hello, this is a known issue with the old version of Hauler, the new version of Hauler has a fix for this so we just need to upgrade the package in OBS and then Hauler should fail properly when it isn't able to pull the images during the EIB build.

Hauler is now upgraded to v1.0.1 which should resolve this issue.
Both OBS and IBS builds are updated and the respective EIB container images are rebuilt. This should no longer be a problem.


Add Form example to components like Select, MultiSelect, Combobox, Checkbox, ...

Hi, Thanks for this awesome project ❤
Can we have react-hook-form examples for those components?

@devongovett said: in many cases we don't need react-hook-form or zod because the builtin validations are enough.
References can be found here: https://react-spectrum.adobe.com/react-aria/forms.html

Right. but can you share a MultiSelect playground for reference?

I was also surprised that the example didn't exist, now you can see it here.


common: update to new appc/spec aci.TarReadCloser.

Now we have to pass to functions wanting a *tar.Reader the embedded *tar.Reader
in aci.TarReadCloser and call the Close() method when finished.
See appc/spec#461

I guess this is exactly the question I'd raised. Thanks!  Lgtm

Thanks! :+1:


Update Java samples, documentation, fix Windows issue.

I update the java implementation on these points.
Update

Fix Windows issue (Windows requires cacert.pem file and more attribute requires for the annotation)
Update azure-maven-plugin version to 1.5.0 (latest)
Update functions core tools from v2 to v2 or above with v3 for java sample
Documentation update for works both on Mac and Windows
Provide shell/powershell script for installing Kafka Extension and pem file (windows only)
Adding the section, how to deploy to Azure (Premium Linux/Windows)

Test

Windows (bash, powershell)  client
Mac client
Function App (Premium plan Linux EP1)
Function App (Premium plan Windows EP1)

@TsuyoshiUshio Thank you so much sir for your answers!


update PR link checker workflow

Fixes #345.
Use a different changed-files checker (the old one was no longer maintained) and also make the link checking more fine-grained by only checking added lines for broken links.
We now use a matrix to parallelize over changed files. This slows down the complete workflow a bit in some cases, but it does offer a nicer user experience, because the files that are checked will now be shown in the GitHub UI itself, instead of only in the workflow run logs.
Lychee is installed directly from the GitHub nightly release. This is really fast. The original reason, though, was because there is no Ubuntu package for Lychee. Works out well, though.
Determining the base SHA is a bit convoluted, because GitHub does not provide this in a usable way. See
https://www.kenmuse.com/blog/the-many-shas-of-a-github-pull-request/ for a good overview of this problem. The git rev-parse command was suggested there. The fetch depth must be at least two, otherwise the base SHA cannot be used.

@ewan-escience ah that's sloppy, thanks. I hadn't noticed because the workflows themselves seem to pass, the error comes from the Actions execution backend it seems.

Solution here https://github.com/dorny/paths-filter/issues/66

Nice, now it explicitly says it skips it.
Btw, I tested the solution itself in my own fork: https://github.com/egpbos/guide/pull/1 You can see there how it should work here as well if I squashed things correctly 😄 I didn't want to pollute this PR with tests.


Release BotKube 0.15

Overview

This task aggregates all the to-dos which should be done when it comes to 0.15 release.
AC

[x] Go through the release process instruction and release 0.15
[ ] Describe any breaking changes in the GitHub release
[ ] Testing

Test manually interactivity for Slack
Test telemetry
Aggregate all stuff for manual testing described by @mszostok in https://github.com/kubeshop/botkube/issues/595

[ ] Schedule all community activities - blog post, sharing the news in social media, livestream, etc. (@brampling)

Bugs:

[ ] Slack UI does not wrap filter values in quotes.

Ok created PR here

So far, all release candidate required fixes have been addressed in PR: https://github.com/kubeshop/botkube/pull/826


Publish to OpenVSX?

I would love to use your extension with Gitpod, but they have to use OpenVSX instead of the official marketplace: https://www.gitpod.io/blog/open-vsx-in-gitpod
Are you maybe open to publishing to OpenVSX?

published https://open-vsx.org/extension/mondersky/tabscolor

Unfortunately:

https://code.visualstudio.com/docs/editor/vscode-web#_extensions

too bad. In fact the extension operates by patching vs code installation files, it's unlikely that it will function on web

Makes sense then.
Publishing to OpenVSX is of course still useful for all the other people not wanting (or being able) to use the official marketplace 👍


Scrolling menu parallax + less tiling

It at least considers control size so it doesn't look awful at 4k.
Needs https://github.com/space-wizards/space-station-14/pull/9677 IMO because the master values for it aren't good.
Edit: I have just added a reverted parallax in https://github.com/space-wizards/space-station-14/pull/9786 that's gonna be used in 2 out of 3 places.
https://user-images.githubusercontent.com/31366439/179257710-b825ffc8-629a-4c1f-810a-f1b5a7881655.mp4

Use the new high-speed parallax and you're good-to-merge.


Upgrade to angular 9

Following changes were made:

Remove @angular/flex-layout, but as it was only used in one spot, i added some css to replace it.
Upgrade ngx-markdown to 9.0.0-beta.0. Seems it was referencing a local library or something?
Added local copy of markdown-copier.css
Added safe access of ViewContainerRef['_data'].

Everything seems to work so far. I can cleanup the commits etc, but thought i would submit this to get the ball moving.

Thank you for taking the time for the upgrade.
There was something broken when I applied one of the early versions of ng 9 or maybe I thought so somehow. Seems like it is OK now.
I was using my own fork of ngx-markdown that's why you saw a local dependency. Actually I had raised a PR for my fork but appearantly they still couldnt look into it. ngx-markdown is not required for any of the library projects. It is for the showcase application which is the wrapper workspace for libraries.
I'll look into your PR soon. It seems pretty fine at the first sight. Thanks again.

No problem. Might have to replace what i did with ngx-markdown with your version, for the clipboard feature. Let me know if you want me to take a look at it.

Thanks for the effort. I've done the merge. The ones with ngx-markdown is not a big deal. I'll overwrite with my version of them.

Great news! I'm assuming this means we can expect a new release to NPM soon?

Version 9.0.0 is released.


Upgrades - Cost Analysis

What are some interesting stats that can be used to indicate worthwhile upgrades?

Also see example 2 in #9