Cannot Retrieve CF Internal Data Needed for Billing / Debugging (Diego)

#We run a large Cloud Foundry environment, and have for some time now.
We have two major problems:

We need to be able to log into the Bosh VMs as root for debugging /troubleshooting purposes (including CF bugs which crop up from time to time with production impact).  This includes pulling logs, strace, etc.

We need to be able to pull CF data for various accounting and billing purposes, and in the past this has sometimes required pulling data which only exists on the DEAs. (ie. /var/vcap/data/dea_ng/instances.json)

The problem today is that it seems we can't get root on any of the Diego VMs.  From what I understand -- and based on what I've been reading today -- this is a design decision and root is entirely disabled.
This would not be such a problem if it was only restricted to the Diego Cells, but from what I can tell, this is the case on the other supporting infrastructure as well (such as the Diego Brain etc).
Is this correct (that there is no root on any of these VMs)?  Is there a workaround for this?
The second issue I described is probably even more important:   Traditionally, we pulled some rather critical information which we needed for mapping utilization data for billing from the DEAs and collecting it in one place.  In the past , while there was no database (that I could find) which mapped from quasi-random Warden container IDs to Instance IDs, I was able to work around this problem by gathering data directly from the DEAs.
However, now this seems like it's going to be a major problem for us... in addition to having less ability to troubleshoot issues via standard debugging within these Diego cells,  I don't see how we can map from the Diego container IDs  (the internal Diego hostname GUIDs) to CF app GUIDs?

Is there a way that to get root on the Diego VMs?  Which VM types (router, diego cell, diego brain, api ,etc) are included in this change (root disabled) and which will remain unaffected?

What is the appropriate (and most straightforward) way to convert from Diego internal hostname GUIDs to CF application ID?  Is this available internally via some sort of API?

Hi, @bitshark,
If by getting root you mean being able to run something like sudo -i to become the root user after logging onto a VM with bosh ssh, there shouldn't be anything fundamental about Diego that prevents you from doing that. If it doesn't currently work for you, that sounds like a BOSH configuration issue.
Diego's implementation is fundamentally different from the DEAs, especially in terms of how it tracks application instances. That said, Diego offers APIs that are stable, if not publicly routable, from which you would likely be able to extract the information you used to obtain from the instances.json files on the DEAs. The BBS API would be the most productive source of this information, and we even have a command-line tool, cfdot, that makes it easier for human operators to interact with the BBS API from within the BOSH deployment. If you're looking to correlate instance IDs to application IDs, cfdot actual-lrp-groups should have all of that information for the entire deployment, encoded as a stream of JSON objects. The manifest-generation templates in diego-release also even deploy and auto-configure cfdot on all the Diego VMs by default, so you should be able to use bosh ssh to access a Diego VM and then run cfdot commands even as the auto-provisioned BOSH user.
Hope that helps,
Eric, CF Diego PM

@ematpl Thank you for your feedback - this has been very helpful.
For a few years now the default CF root password for the cflinux2 stemcells has been cloudcow  as you're probably familiar.  .   What it appears now is that there is actually a capability to login and sudo to root, but it's just that the root password is randomized and not stored in /etc/passwd
I've realized that this process now only works via bosh ssh, while before it would work with a valid ssh rsa identity file from the time of PaaS provisioning.
I found a great link here that describes the changes in some additional detail:
http://www.cloudengineeringblog.com/
I do have a final question though, and that is the following:

Where is the  instance IDs to application IDs  data actually stored, physically speaking... does this go in the blob store or somewhere else?

Does CF / Diego keep a record of this data  (instance <-> app_ids) over time, and if so, where is it stored?

Can the data you are referring to above (cf dot ) and the BBS API be reached from outside of the Diego cells, or is this information only present locally on each Diego Cell VM?

thanks again

Hi, @bitshark,
That correspondence between IDs is part of the state that the Diego system uses to ensure eventual consistency of workloads across the cluster, and as such is stored in the relational database that backs the BBS API. Only the current state of the system is stored in the database and presented through the BBS API, although the Diego component logs would contain a trace historical record of those associations.
In a typical deployment, the BBS API is not routed publicly, and is protected with mutual TLS authentication, so clients must present TLS credentials signed with the shared internal CA used across the CF components (or at least for the BBS API). Client components also expect to resolve it through internal DNS at the bbs.service.cf.internal domain, and so the certificate that the BBS API server hosts will contain that as a domain SAN for clients to verify. In practice, this means that you can communicate with the BBS API from anywhere in the CF deployment, so long as you have those client credentials and either skip server certificate verification or resolve it through DNS (typically provided by the local Consul agent). I think Consul agents are still colocated everywhere for DNS, and the cfdot job can be configured to carry those client credentials, so you'd be able to colocate that job on any CF VM and use it to access the BBS. As I mentioned, the manifest-generation script in diego-release already colocates that cfdot job on all the VMs in the manifest it generates (well, I think we skipped the cc_bridge VMs), and the cf-deployment base manifest does the same thing on the diego-api, diego-brain, and diego-cell instance groups.
Finally, the BBS API even provides a stream of all events that alter DesiredLRPs (the specification that CC submits to Diego to run app instances) and ActualLRPs (the logical records of the individual instances), which would be another way to track those app/instance association changes over time. The cfdot tool exposes that via the lrp-events command, which provides those events as a stream of JSON-encoded objects.
Best,
Eric

Hi, @bitshark,
I'm closing out this issue now. If you have further questions, please feel free to open a new issue.
Best,
Eric


Flexible update interval based on speed

Default Update interval, make it longer eg. 3-5 seconds, users might not bother setting it up and 1 sec is rather aggressive, battery wise.

The device will use a lot of battery power for location services if the interval is set to 3 seconds too (maintaining a fix). 1 second was chosen because it provides the smoothest real-time view on the map given that there is currently no movement prediction on the frontend. I'm not really inclined to change this default right now

It's complicated, 1 sec is smooth in a car/train/bus where the position changes a lot so precision is needed, but...

walking speed, 1-2-3-4 sec are not that different (what is the GPS precision anyway, 1-2 meters?), the position changes rather slow
stopped/no gps lock, you're consuming energy without any benefit by continuously sending network data

What crossed my mind writing this is that, based on speed maybe you can vary the interval
Eg. Say walking speed is 5km/h, have interval set at 1s

if speed >=5 then use interval 1s
if 0< speed <5 then use between 5s and 1s
if speed=0 or no lock then use 10s or 30s

...if interval 3s then maybe multiply, 3s, between 15s and 3s, 30s or 90s etc albeit this doesn't scale that nice, maybe use a lower factor than actual interval (eg. numbers would go awry fast for 10s)

It's possible to set the GPS provider to not receive updates if the device hasn't moved at least X meters since the last update. I can make this a configurable option. That way if the device stops moving, location updates aren't sent unless the device has moved at least e.g. 5 meters since the last update, regardless of the update interval.

Minimum distance is configurable in settings.


[Bug] Source array not long enough

I'm generating a screenshot slideshow similar to the steps described in #426 ... oddly today I started getting an error:
Setup: Win7x86
Steps:

Grab some screens of various sizes with Greenshot
Load into STG, add a progress bar and a watermark
Save as weither GIF or APNG

Defect:

Attempted workarounds: I tried deleting the watermark but no difference.

Are the pictures the same size? If you can share the pictures I can fix this bug, Thanks.

Sorry for the delay on this, having some system issues that are preventing a re-test.

It's alright. :)

Can I close this issue? I'm not sure if it still happens.

Closing this issue as inactive. I believe this issue should not happen anymore,
I added some checks when importing frames with different sizes.


changes in direct memif implementation

Signed-off-by: Mikhail Avramenko<EMAIL_ADDRESS>Related issues
https://github.com/networkservicemesh/deployments-k8s/issues/2381
https://github.com/networkservicemesh/sdk-vpp/issues/373
Description
To fix problem with nse-composition i made and tested rework for direct memif.

@Mixaster995 I'm a little curious why you are fixing this in mechanism/memif/client.go instead of in mechanism/memif/memifproxy ?  It looks like you are simply replicating logic the memif proxy has for not recreating proxies for existing connections, only in memif/ rather than memif/proxy.

Hello, @edwarnicke.
It is not related to memifproxy chain element.
Current direct memif logic working like this:

Create memif on nse
Create memif on client side of forwarder
On server side of forwarder check if client-side memif created delete it and not create server-side memif.
create memif on nsc
every step happens on way back

With this fixes current logic is:

On way forward add left mechanism on server side of forwarder
On way back on client side of forwarder check left and right mechanism to be memif (which is direct memif scenario)
In direct memif case not create memif on client side of forwarder and just save socket url to metada
On server side of forwarder not create memif and set mechanism socketurl from metada

Thus, we removing excessive memif creation on client side of forwarder, because it is not actually needed in direct memif scenario. And it is also eliminates leak reported in https://github.com/networkservicemesh/deployments-k8s/issues/2381

@Mixaster995 After much poking at it, this looks good.  Could you flip to ready to review so we can merge it?


How to use packer_windows_libvirt_amd.box in Vmware / Virtualbox?

Hello, I'm sangjun who is very interested in your awesome project.
There is one thing that is being set for convenience while using kAFL.
Instead of using packer_windows_libvirt_amd64.box to run a vm using libvirt on Linux, I want to run the same image on a Windows host machine through virtual box or vmware.
( packer_windows_libvirt_amd64.box is generated from make biuld command )
https://github.com/IntelLabs/kafl.targets/tree/master/templates/windows
However, no matter how much I set up the vagrantfile, it eventually failed. Any good ideas?
I would like to reproduce the crash using the same box image in vmware or virtualbox rather than qemu. However, changing the provider of vagrant to virtualbox or vmware instead of libvirt failed.
Thanks
If there is a solution to this, I would appreciate it if you could write the command.

Hi @5angjun,
unfortunately, kAFL is only based on a modified QEMU/KVM virtualization stack, you wouldn't be able to run it on Windows, even if you could generate the Packer Vagrant box for VirtualBox or VMware.
And nested setup scenarios (Windows VMware > Linux QEMU/KVM kAFL > target ) aren't supported yet https://github.com/IntelLabs/kAFL/issues/142


Move compiler warnings settings to target level for modules

@Bidski can I grab a review pls xx

Alright. Unrelated changes have been stricken from this PR lol

I have added the warnings for tests and fixed the existing warnings that created :+1:

@TrentHouliston does this PR match what you had in mind?

@TrentHouliston two things:

How do you want it to be configurable? I have a couple ideas: add a ${NUCLEAR_COMPILER_WARNINGS} variable which is what we use when we include(${NUCLEAR_COMPILER_WARNINGS}) the function, or alternatively perhaps a NUCLEAR_COMPILER_WARNINGS_DIR variable which is added by nuclear/CMakeLists.txt?
If you're keen to add this to nuclear proper, woudl you like me to add back the MSVC support? The repo I got this off had a list of MSVC warnings which I removed because we'll never use MSVC on this repo.

@Bidski FYI I removed Wsign-conversion, because it warns on this snippet (courtesy trent), which is silly

This dismissal was approved by Alex


hostap: Fix enterprise configs

Move Enterprise only configs and files under Enterprise guards to save memory when not using Enterprise.

FYI @fengming-ye  @MaochenWang1

I am also working on adding configs for below and anything that is optional, will raise a PR soon.

ECDH - WPA3-SAE-PK only
Conditional MD4 (not MbedTLS guards but hostap guards)


Skip WildFly dist unpacking when skipping GWT compilation

No need to unpack the WildFly distribution when the GWT compilation is
skipped. It is not being used and only slows down the build (and takes
additional disk space)
@dgutierr could you please review?

merged


Update cacheEnabled doc

cacheEnabled doesn't actually work on iOS, the closest one can get is setting incognito={true}.
Ref: https://github.com/react-native-webview/react-native-webview/issues/1979

cacheEnabled doesn't actually work on iOS, the closest one can get is setting incognito={true}.
Ref: #1979

Only on iOS, incognito={true} can delete all web content including cache, local storage and session storage.
On Android, it can only delete cache and the local storage is been remained.
How can I delete cache only on iOS?


Crash on a randomization (not on a stable release)

Description
Tried the latest commit, 73961c9e590b44770fbc4f9ded562845efd3d69a, and got a randomization that crashes the game on title screen (right when they are supposed to say "Fire Emblem: Engage").
Tried loading the randomization through LayeredFS and through Cobalt, and both make the game crash.
I know this is not supposed to be a stable release, but sharing this at it might possibly help
Going to verify if this works with another randomization (I turned on most settings)
Output
Here is the Output folder uploaded on mega.nz: https://mega.nz/file/qYoEhKqJ#zggDH9zOgPnzkY_eDarMR7LMiTDm6Z1Gz6C_-qaFXE8

Other randomizations seem to work, I disabled the character size scaling setting, might have been that causing it.

Yeah, slight stability issues with asset shuffles for some reason. Some combination of fields apparently results in a crash... under investigation for the time being.

ouch, good luck with that!
and once again, a million thanks for the love poured in this! never expected so many new features to release so quick, super excited about trying out different starting classes, inventory swap, etc 🙂

Finally found the cause for this particular crash: It turns out that the game straight up crashes on the title screen if the protagonist's starting level is anything but 1... Fixed in newest commit. Don't hesitate open issues if similar cases occur. 👍


feat: Dask add pod template support

TL;DR
This PR change the dask plugin to use ToK8sPodSpec to create the basis of the pod template to be used for all dask components. This is coherent with other plugins (such as the ray plugin) and enables the use of common features such as pod templates.
Type

[ ] Bug Fix
[x] Feature
[ ] Plugin

Are all requirements met?

[x] Code completed
[ ] Smoke tested
[x] Unit tests added
[ ] Code documentation added
[ ] Any pending items have an associated Issue

Complete description
This has been pretty straight forward. The only downside was that I had to remove the interruptible config from the pod spec after it has been added, as I couldn't see a way of not adding it in ToK8sPodSpec.

@hamersaw I'm done with the work on this, but haven't smoke tested this in a local cluster yet, should hopefully get to that tomorrow.
The test and lint failures are a bit puzzling to me as both pass locally, and it seems like they flag something that's not even in the code (e.g. dask_test:543 does not contain the function mentioned)

should hopefully get to that tomorrow.

@bstadlbauer have you had a chance to test this locally yet? Everything looks good to me, but always nice to double-check.
As far as the CI checks, I'm not quite sure what's going on. Will plan on addressing as we iterate on this.

@hamersaw This would be ready from my end now - again sorry for the incredibly long delay here.
I've swapped the removeInterruptibleConfig function in favor of a nonInterruptibleTaskExecutionMetadata wrapper as you've suggested above.
In addition, I've written a small end-to-end test suite in https://github.com/bstadlbauer/flyte-dev-setup which can automatically end-to-end test the plugin which should hopefully decrease the iteration time on future changes as I was mostly blocked by testing this. I've also found a small bug (an incorrect restart policy due to a dask kubernetes update) which I've fixed along the way.
In addition I've noticed that Labels are somehow not propagated (i.e. this task will fail), but I'm not 100% sure yet what the root cause is - I've done some debugging and it somehow seems like
https://github.com/flyteorg/flyteplugins/blob/fba2d0b4d829c9822bbe0f6015d9920a36aa7f28/go/tasks/plugins/k8s/dask/dask.go#L116
the output GetLabels() here does not contain the label, which would mean it's outside of the plugin.


feature request: skip verifying builtin extensions cache for improved performance

Problem description
When launching vscode on my computer, 4737 files or folders are opened. For example

/home/simon/.config/Code/User/globalStorage/storage.json
/home/simon/.config/Code/User/globalStorage/state.vscdb
/home/simon/.config/Code/User/settings.json

Out of these 4737 files, 4199 are builtin extension files, for example:

/snap/code/138/usr/share/code/resources/app/extensions/configuration-editing/package.json
/snap/code/138/usr/share/code/resources/app/extensions/css-language-features/package.json
/snap/code/138/usr/share/code/resources/app/extensions/diff/package.json
/snap/code/138/usr/share/code/resources/app/extensions/github-authentication/package.json

It seems to me a lot of these files are opened for verifying the builtin extensions cache.
Proposal
Perhaps it might be a good idea to add a setting to skip verifying the builtin extensions cache for improved performance.
I have a draft here https://github.com/microsoft/vscode/pull/192631 where I disabled verifying the extensions cache, which reduced the number of opened builtin extension files by ~4000 from 4199 to 190.
Additional information
For seeing which files vscode accesses, I used strace like this:
sudo strace -u $(whoami) -E PATH=$PATH -E HOME=$HOME -f -t -e trace=file code > /tmp/log.txt 2>&1

We closed this issue because we don't plan to address it in the foreseeable future. If you disagree and feel that this issue is crucial: we are happy to listen and to reconsider.
If you wonder what we are up to, please see our roadmap and issue reporting guidelines.
Thanks for your understanding, and happy coding!


Customizable PV info - Added missing mongo PVC

Description
This PR provides a mongo PVC that was previously missing from the deployment. Also made PV/PVC information more customizable.
Fixes # (issue)
Type of change
Please delete options that are not relevant.

[ ] New feature (non-breaking change which adds functionality)

Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] My changes generate no new warnings
[ ] New and existing unit tests pass locally with my changes
[ ] I have not reduced the existing code coverage
[ ] I have updated any sections of the app's documentation that are affected by the proposed changes, if applicable

Also looks fine to me. Thanks @zagganas. Also can you please squash it to one commit?

I will squash the commits when merging.


No tournament API methods

Waiting on API key (asked for one for both stub/regular) still!

this is currently in-progress! looking good so far

[] Add basic tests and fix all the failing tests due to Request having  a new field. A workaround is to just straight up delete body if the method is GET lol
[] Also send PR to riotratelimiter after adhering to code standards / cleaning up code / discussions (I'm just throwing code at it atm to see if everything works)
then i'll release
yarn add kayn@beta

Released in 0.7.6. I wasn't able to test the Tournament endpoints because my key is still processing, but it should work since TournamentStub basically works.
It is still using a fork for now until I clean up that fork to submit the PR.


Check nutanixmachine resources when deleting nutanixcluster

What this PR does / why we need it:

Wait for nutanixcluster deletion until all nutanixmachine resources are deleted
Retry nutanixcluster creation in case of category creation failure instead of raising fatal error.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
N/A

How Has This Been Tested?:
Special notes for your reviewer:
Please confirm that if this PR changes any image versions, then that's the sole change this PR makes.
Release note:

- Wait for nutanixcluster deletion until all nutanixmachine resources are deleted
- Retry nutanixcluster creation in case of category creation failure instead of raising fatal error.

Codecov Report

Merging #304 (acb0d84) into main (556905e) will decrease coverage by 0.02%.
The diff coverage is 0.00%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@           Coverage Diff            @@
##            main    #304      +/-   ##
========================================
- Coverage   2.64%   2.63%   -0.02%
========================================
  Files          4       4
  Lines        982     988       +6
========================================
  Hits          26      26
- Misses       956     962       +6

Files Changed
Coverage Δ

controllers/nutanixcluster_controller.go
6.83% <0.00%> (-0.27%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

/lgtm
/approve


Docker image missing directories

Description
I keep getting errors when trying to install "A Core Company" or any other app.
The app does not seem to create the required temp and transfer directories in the r3 container's /opt/r3/data bind mount directory.
The app only works after stopping the containers, clearing all bind mounts, and creating the temp + transfer directories.
Here is my docker-compose config
version: '3.3'

services:
    r3_db:
        image: postgres:latest
        container_name: r${R3_VERSION:-3.4.2}_db
        restart: always
        environment:
            POSTGRES_PASSWORD: app
            POSTGRES_USER: app
        volumes:
            - ./data:/var/lib/postgresql/data
    r3:
        build:
            context: https://github.com/r3-team/r3_docker.git#main
            args:
                r3_db_host: r3_db
                r3_os_arch: x64
                r3_version: ${R3_VERSION:-3.4.2}
                im_policy: /etc/ImageMagick-6/policy.xml
        container_name: r${R3_VERSION:-3.4.2}_app
        image: "r3/${R3_VERSION:-3.4.2}"
        depends_on:
            - r3_db
        ports:
            - "14000:80"
        restart: always
        volumes:
            - ./data_app:/opt/r3/data

Steps to Reproduce Issue

Create an empty folder somewhere.
Create the docker-compose.yml file.
Inside your terminal / command-line run docker compose up

Screenshots
No temp directory.

...after creating the temp directory, refreshing localhost in the browser, and retrying the Core Company installation...

Additional Info
Operating System: Windows 11 Pro x64 Version 22H2 Build 22621.1992
Container Engine: Docker version 24.0.2, build cb74dfc

Hello Heartnett,
I´ve had a look and ran the default Docker Compose file from our website without issues.
At this step of the process, the required directories are created:
 => [r3 2/5] RUN bash -c 'mkdir -p /opt/r3/data/{certificates,files,temp,transfer}'  && apt-get update ...
It looks like your compose file is different to the default one. Because I do not work with Docker daily, I am not super versed with the syntax - but you seem to have have changed this line:
Default: volumes: - data_app:/opt/r3/data
Your version: volumes: - ./data_app:/opt/r3/data
Is there a reason for the change? Does the default version work for you?

Hello Heartnett,
I´ve had a look and ran the default Docker Compose file from our website without issues.
At this step of the process, the required directories are created:
 => [r3 2/5] RUN bash -c 'mkdir -p /opt/r3/data/{certificates,files,temp,transfer}'  && apt-get update ...
It looks like your compose file is different to the default one. Because I do not work with Docker daily, I am not super versed with the syntax - but you seem to have have changed this line:
Default: volumes: - data_app:/opt/r3/data
Your version: volumes: - ./data_app:/opt/r3/data
Is there a reason for the change? Does the default version work for you?

No, I did not use the default config as-is, because I do not use docker volumes for testing. I use bind mounts instead .

The Docker image we offer is for testing/development purposes. I am sorry, but I cannot help you with specific changes you make to make it fit your environment.
Setting up REI3 is very straightforward - so as long as you setup everything following the steps outlined in the docs, it should work just fine.

Hi Heartnett,
trying your Dockerfile you just have to create manually a 2 directories named "temp" and "transfer" inside your local directory "./dat_app". It's important to keep the root permissions for those directories since R3 is running inside the container as root.
For example, you could use from a local terminal:
sudo mkdir temp
or open a terminal within your container with:
docker exec -ti r3.4.2_app /bin/bash
And the directory tree structure must be as follows:
./data_app
    /temp
    /transfer
    config.json
I hope this helps. For me it's enough to run R3 and install apps properly.
Regards.


Feature Request: Rotate app

The current implementation of Badger Mobile will not rotate on an Android or iOS. I talked with an active user of Badger Mobile who said it 'doesn't work' on an iPad. After getting it clarified, he meant that it doesn't rotate, and so was 'unusable' by him.
I thought this was a bit odd, but he was very sincere. I'm sure there are other people who would like to use this app on tablet and really want it to rotate.

Not going to implement at least for the foreseeable future.  Rotation has been turned off on purpose as there are not enough resource to support rotation across all devices and keep the quality.

Sounds reasonable.


Add example utterances to AMAZON.HelpIntent.

This expands the Help response to include random example utterances the user can try.
The spoken response includes one random utterance and another for the reprompt.  The same reprompt is used for ask.launch (skill opened with no Intent requested, as in, open kodi).
The Card presented in the Alexa app provides 5 randomly-chosen utterances.
The example utterances are chosen from the speech_assets/SampleUtterances.LANG.txt file.  A new file sample_slotvals.LANG.txt was created to fill slot references with something to make it sound more natural than something like insert title here.
Needs German translations
Needs testing, especially for AWS and Heroku deployments

There are several things in this PR that require German translations:

sample_slotvals.de.txt -- the media titles don't necessarily need to differ, but things like thirty seconds do need localization,
optional: Extensions to sample utterances for AMAZON.HelpIntent in utterances.de.txt,
The new/modified template strings:

help: "You can ask me whether there are any new shows, to play a movie, t. v. show, or artist, control playback of media, and more. For example, {{ example }}. I've sent more examples to your Alexa app."

help_short: "What would you like to do?  Try something like, {{ example }}."

help_text: "Kodi-Alexa allows you to fully control one or more Kodi installations with your voice.  Here are some examples of things you can say:\n\n{{ examples }}\nFor a complete list of supported requests, please see the README file on Github (https://github.com/m0ngr31/kodi-alexa/README.md)."

Ping @mcl22 and @ausweider :)

@mcl22 and @ausweider, I forgot to mention.. the values for these in sample_slotvals.de.txt should all be related:
Artist
Album
MusicGenre
Song
MusicVideoGenre
MusicVideo

That is, they should ideally all be from the same album and by the same artist, otherwise you could get some weird suggestions like:
Play Death Metal music by The Beetles
Play Barbie Girl by Metallica

;)

Ok, here we go.  Zum Beispiel {{ example }}. Ich habe dir weitere Beispiele an deine Alexa App geschickt."
help_short: "Was möchtest du tun? Probiere etwas wie {{ example }}.""
help_text: "Kodi-Alexa ermöglicht es dir eine oder mehrere Kodi Installtionen mit deiner Stimme zu steuern.
utterances:
AMAZON.HelpIntent (/nach) was ich es fragen kann
AMAZON.HelpIntent was es (/alles) kann
sample slotvals:
Addon YouTube
MovieGenre Komödie
Movie Männertag
Artist Helene Fischer
Album Farbenspiel
MusicGenre Schlager
AudioPlaylist Partymusik
Song Atemlos
Show Farscape
VideoPlaylist HD-Videos
ShowGenre Science-Fiction
MusicVideoGenre Pop
MusicVideo Die Hölle morgen früh
Volume Fünfundsiebzig
Season zwei
Episode acht
ForwardDur eine Minute
BackwardDur 30 Sekunden

Thanks again @mcl22 :)

Just a thought, but given the double meaning of the word "alternative" is it going to be a little confusing when used in context?
ListenToGenre listen to alternative genre
ListenToGenre listen to alternative music
ListenToGenre listen to alternative songs
ListenToArtist listen to alternative artist Sia
ListenToArtist listen to alternative music by Sia
ListenToArtist play alternative artist Sia
ListenToArtist play alternative music by Sia

It doesn't sound weird to me, but honestly it doesn't matter to me either what it is so long as it fits, so I'll change it :)

Have you tried this PR with AWS?  If no one gets around to it, I can do it, but it'd save me some trouble.

It works.

Could quotes be added around each command to help with formatting?
I'm all for providing the user with help, but the randomness of the examples might not be that helpful, especially for a beginner.
Thoughts on using a curated list of the basics to randomly pick from?

Yes, I can add quotes around them.  I actually hadn't looked at it using the mobile app.
As for the list of things, I prefer a random set because it shows variations.  Certain intents can be excluded, however.  I've already excluded the PlayerZoom intents and had considered excluding the navigation commands because things like "right" without any context are kind of confusing, particularly in the spoken response.

Had to insert a non-breaking space in between two newlines to get it to show an empty line on the mobile app.. kinda weird, but it works.  Lemme know if that looks better.

As an aside, it'd be nice if Amazon allowed hypertext links, though I can understand why they don't (security risk).

Peachy.
Maybe just say "for a complete list please see the GitHub repo"

Thoughts on including the navigation/system commands?  Particularly these:
Home (/navigate/go) home
Back (/navigate/go) back
Up (/navigate/go) up
Down (/navigate/go) down
Right (/navigate/go) right
Left (/navigate/go) left
PageUp (/navigate/go) page up
PageDown (/navigate/go) page down
PageDown show more
Select select
Menu (/open/context) menu
Fullscreen (/toggle) (fullscreen/visualizer)
Hibernate (hibernate/deep sleep) (/system)
Reboot reboot (/system)
Shutdown shut down (/system)
Suspend (suspend/sleep) (/system)
Suspend put system to sleep
Suspend go to sleep

That's fine with me

Meaning you'd prefer to include or exclude them?

Sorry, I think it's fine to include those.

Ok, all that needs to be done I think then is to test on Heroku.

I'll give it a test real quick

Works great on Heroku

Awesome, thanks for checking.


[Bug] Can't seem to create a theme

Bug report
What I did
Trying to create a theme so I can override some colour choices of tabler (not found an easy way to do this yet!)
I've gone through the updated docs in (https://github.com/Laravel-Backpack/docs/issues/417) and can't get my theme to be picked up by backpack.
In ui.php I commented out the view_namespace_fallback variable so that I could check if my files were being read and I get a 500 error.
The view: [backpack.ui::blank] was not found in any of the following view paths: [my-theme-folder.::blank ], [ backpack.theme-coreuiv2::blank ], [ backpack.ui::blank]- Called in: /storage/framework/views/9bd51d16f96d06902e0c9283968e1990.php on line: 21
even though I have a blank.blade.php file in the root of the theme folder.
My theme lives in resources -> views -> my-theme-folder.
If I reinstate the view_namespace_fallback variable then the error goes away but I'm left with no customisations, just the fallback theme.
What I've already tried to fix it
Going through the docs on the github issue, moving the theme folder to various different positions in the resources path
Is it a bug in the latest version of Backpack?
Yes, as far as I can tell
Backpack, Laravel, PHP, DB version
When I run php artisan backpack:version the output is:
PHP VERSION:
PHP 8.1.12-1ubuntu4.2 (cli) (built: Jun 28 2023 13:56:12) (NTS)
Copyright (c) The PHP Group
Zend Engine v4.1.12, Copyright (c) Zend Technologies
with Zend OPcache v8.1.12-1ubuntu4.2, Copyright (c), by Zend Technologies
with Xdebug v3.2.0, Copyright (c) 2002-2022, by Derick Rethans
LARAVEL VERSION:
<IP_ADDRESS>
BACKPACK PACKAGE VERSIONS:
backpack/basset: 1.1.1
backpack/crud: 6.1.3
backpack/generators: v4.0.2
backpack/logmanager: v5.0.0
backpack/permissionmanager: 7.0.0
backpack/pro: 2.0.10
backpack/theme-coreuiv4: 1.0.5
backpack/theme-tabler: 1.0.6
Cheers for any help/clarification if this isn't a bug!

I have a custom theme open right now - here's how the file structure looks in mine:

And in config/backpack/ui.php I have:

    /*
    |--------------------------------------------------------------------------
    | Theme (User Interface)
    |--------------------------------------------------------------------------
    */
    // Change the view namespace in order to load a different theme than the one Backpack provides.
    // You can create child themes yourself, by creating a view folder anywhere in your resources/views
    // and choosing that view_namespace instead of the default one. Backpack will load a file from there
    // if it exists, otherwise it will load it from the fallback namespace.

    'view_namespace'          => 'theme-bootstrap.',
    'view_namespace_fallback' => 'backpack.theme-coreuiv4::',

Maybe it helps 🤷♂


socket_io_transmit_bytes counter doesn't change

HI,
I have been using your library, works great but there is 1 metrics thats not being updated. Its the

socket_io_transmit_bytes counter

I am emitting things to a room that the client joins. But the counter doesn't increase.
Here is an abstract of what I am getting, as you can see the receive bytes work fine.
Maybe the library isn't listening to room emits ?
Any ideas ?

TYPE socket_io_events_sent_total counter
HELP socket_io_recieve_bytes Total socket.io bytes recieved
TYPE socket_io_recieve_bytes counter
socket_io_recieve_bytes{event="AUTHENTICATE"} 10194516
socket_io_recieve_bytes{event="JOIN_CHANNEL"} 235065
socket_io_recieve_bytes{event="SEND_EXCHANGE_RATES"} 0
HELP socket_io_transmit_bytes Total socket.io bytes transmitted
TYPE socket_io_transmit_bytes counter

I must admit i haven't tested it with rooms so you might be right about it not listening to room emits. If you have a fix feel free to make a pull request or else i can look into it as soon as i can :).


[SUGGESTION] JavaScript Fade Animation

Is your feature request related to a problem? Please describe.
nope it is not. it's just an idea that i had for a while now
Describe the solution you'd like
Every single time you load the page, all the images and buttons (excluding the background image) will load in a fade animation
Describe alternatives you've considered
as of now, nothing
Additional context
Add any other context or screenshots about the feature request here.

could also use bootstrap's fade animation...

this could be done with bootstrap's built in fade animation

although a nice touch, i see no point of include on. the reason why is that it's supposed to load instantly, so it gives that feeling of the site being fast. and every single time someone would want to check on another site within the hub, the fade animation would play, which would be rlly annoying


Sync eng/common directory with azure-sdk-tools for PR 3735

Sync eng/common directory with azure-sdk-tools for PR https://github.com/Azure/azure-sdk-tools/pull/3735  See eng/common workflow

/check-enforcer override


Can't retry request after awaiting response

I'm trying to retry a request if I detect an error in the response. The server returns this specific error as a successful response with a specific error code inside the message. I know its not ideal but its what we have now.
This is the code I have now:
export default (): RpcInterceptor => ({
  interceptUnary(
    next: NextUnaryFn,
    // eslint-disable-next-line @typescript-eslint/no-explicit-any
    method: MethodInfo<any, any>,
    input: object,
    options: RpcOptions,
  ): UnaryCall {
    let result = next(method, input, options);

    let defHeader = new Deferred<RpcMetadata>();
    let defMessage = new Deferred<object>();
    let defStatus = new Deferred<RpcStatus>();
    let defTrailer = new Deferred<RpcMetadata>();

    void (async () => {
      try {
        await result;

        defHeader.resolve(result.headers);
        defMessage.resolve(result.response);
        defStatus.resolve(result.status);
        defTrailer.resolve(result.trailers);

        const response = await defMessage.promise;
        if (isAuthError(response)) {
          console.warn(`[${method.name}] auth error, unauthenticating & retrying...`);

          // manually reset the auth status of the call
          result = next(
            method,
            {
              ...input,
              authStatus: {},
            },
            options,
          );

          await result;

          defHeader = new Deferred<RpcMetadata>();
          defMessage = new Deferred<object>();
          defStatus = new Deferred<RpcStatus>();
          defTrailer = new Deferred<RpcMetadata>();

          defHeader.resolve(result.headers);
          defMessage.resolve(result.response);
          defStatus.resolve(result.status);
          defTrailer.resolve(result.trailers);
        }
      } catch (err) {
        console.log(err);
        defHeader.rejectPending(err);
        defMessage.rejectPending(err);
        defStatus.rejectPending(err);
        defTrailer.rejectPending(err);
      }
    })();

    return new UnaryCall(
      method,
      options.meta ?? {},
      input,
      defHeader.promise,
      defMessage.promise,
      defStatus.promise,
      defTrailer.promise,
    );
  },
});

I can definitely detect the error in the response and trigger the second RPC call but its response is ignored. It always returns whatever the first RPC call returns.
I've been going through other issues where interceptor examples are posted (that's how I've gotten this far) but still can't do what I need. Any help is appreciated.

The first problem is that you shouldn't reassign the def[Whatever] variables since we're already returning references to their initial values (which is why you're seeing it only ever returning the first response). The second problem is that you're not throwing an error if the second attempt fails for the same isAuthError(response) logic. Here's an attempt to fix your interceptor:
export default (): RpcInterceptor => ({
  interceptUnary(
    next: NextUnaryFn,
    // eslint-disable-next-line @typescript-eslint/no-explicit-any
    method: MethodInfo<any, any>,
    input: object,
    options: RpcOptions,
  ): UnaryCall {
    let defHeader = new Deferred<RpcMetadata>();
    let defMessage = new Deferred<object>();
    let defStatus = new Deferred<RpcStatus>();
    let defTrailer = new Deferred<RpcMetadata>();

    void (async () => {
      try {
        const first_try = await next(method, input, options);

        // If the response doesn't have an auth error then resolve and exit.
        if (!isAuthError(first_try.response)) {
          defHeader.resolve(first_try.headers);
          defMessage.resolve(first_try.response);
          defStatus.resolve(first_try.status);
          defTrailer.resolve(first_try.trailers);
          return;
        }

        // Otherwise try again with a reset auth status
        console.warn(`[${method.name}] auth error, unauthenticating & retrying...`);

        const second_try = await next(
          method,
          {
            ...input,
            authStatus: {},
          },
          options,
        );

        // If the second attempt's response also has an auth error then throw a custom RpcError.
        if (isAuthError(second_try.response)) {
          const error = new RpcError(
            'The request does not have valid authentication credentials for the operation',
            'UNAUTHENTICATED',
            {...second_try.headers, ...second_try.trailers}
          );
          error.methodName = method.name;
          error.serviceName = method.service.typeName;
          throw error;
        }

        defHeader.resolve(second_try.headers);
        defMessage.resolve(second_try.response);
        defStatus.resolve(second_try.status);
        defTrailer.resolve(second_try.trailers);
      } catch (err) {
        defHeader.rejectPending(err);
        defMessage.rejectPending(err);
        defStatus.rejectPending(err);
        defTrailer.rejectPending(err);
      }
    })();

    return new UnaryCall(
      method,
      options.meta ?? {},
      input,
      defHeader.promise,
      defMessage.promise,
      defStatus.promise,
      defTrailer.promise,
    );
  },
});

Worked like a charm. Thank you very much for the pointers. I couldn't wrap my head around the def[Whatever] variables.


When treeConfig change defaultCollapsed is true drap and drop limit

    angular.module('ui.tree', [])             ...             defaultCollapsed: true         });
Why is drap only limit level 4?

Will close this because your question is unclear, please re-open the ticket with more detail if this is still a problem in the latest release.


Feature/dsdev 1909 binding failures

Generally looks good. Minor updates again. Needs the build.sbt update with a (pending) vault-common bump, a squash, and good to go.


iOS cannot use FCM credential type

Related : https://github.com/twilio/voice-quickstart-objc/issues/109
Description
I fall into the same conclusion, we cannot use fcm credential type for voip push notification even if we've added APN certificates into the FCM application.
First of all, is that correct ? Then, is this possible to make such a thing (that would simplify a lot the registration / make call process). Actually we're creating a Flutter application and we are creating a twilio programmable voice package and want to not make a huge difference between iOS and Android. And with things as it, it's a huge mission.
Steps to Reproduce

Create an FCM Credential type
Register with a generated accessToken that has this properties :

{
  "jti": "SKxxxxxx",
  "grants": {
    "identity": "testId",
    "voice": {
      "incoming": {
        "allow": true
      },
      "outgoing": {
        "application_sid": "APxxxxx"
      },
      "push_credential_sid": "CRxxxxx"
    }
  },
  "iat":<PHONE_NUMBER>,
  "exp":<PHONE_NUMBER>,
  "iss": "SKxxxxxx",
  "sub": "ACxxxxxxx"
}

Try to make a dialing out call to the testId client
See the error in the Twilio debugger :

Versions
Latests

Thanks for reaching out @izio38.
We do have research tasks to expand our call notification delivery alternatives, but currently the Programmable Voice iOS SDK only supports Apple's VoIP Service certificate for incoming call push notifications. To provide the best and seamless VoIP experience on iOS, especially when the app is not active or not running, it's highly recommended to integrate CallKit and PushKit for incoming calls.

@maningyt we don't have any committed plans to implement right now, though we have added it to the list for official consideration for 2023 roadmap. We've recently released a React Native Voice SDK, and this request is something worth considering as we see more and more developers look to share common code between platforms


suppress unwanted stack traces and add --stacktrace option

also fix test names
Addresses issue TS-27903

[x] Changes are tested adequately
[x] Teamscale documentation updated in case of relevant user-visible changes
[x] CHANGELOG.md updated
[x] Present new features in N&N

Please respect the vote of the Teamscale bot or flag irrelevant findings as tolerated or false positives. If you feel that the Teamscale config or architecture file needs adjustment, please state so in a comment and discuss this with your reviewer.

I think the one long method finding is still tolerable for now. I'd delay splitting up the test class until we have additional tests, so we can split in a way that makes the most sense then.


Record as Samples

I need to record input of my Mic and record as 1 minute wave files.The 1 minute file saving should be done while recording.
Any helps?
Please

Yes, quite easy, just use WaveIn and in the DataAvailable event handler, write the recorded data into a WaveFileWriter. if the size of the current WAV file is 1 minute, then close it, and write create a new WaveFileWriter.

Thanks a lot Mark.
:)


Is there still interest in this project?

I have a personal interest in furthering this project a bit and thought I'd ask and possibly open up discussion for some of my own ideas and questions.

I am interested in this project for potential riscv microcontroller emulation. I'm curious to see if it might be possible to use this library/tool for the CPU but have peripherals that I design for it (GPIO, PWM, UART, I2C, SPI, etc). Also, I'm curious about the clock rate... if it is possible to make it fixed (like 1MHz, 8Mhz, 16Mhz, etc).

Also, I'm curious about the clock rate... if it is possible to make it fixed (like 1MHz, 8Mhz, 16Mhz, etc).

You could call Cpu::tick at your choice of some fixed rate.

I'm interested in this for WASM usage, and will maintain a version here: https://github.com/LeoDog896/wessel


Create a minimalistic UI client using the chat-e2ee/service SDK

Use the SDK @chat-e2ee/service and build a minimalistic UI
https://github.com/muke1908/chat-e2ee/tree/master/service
Scope of this task:

Create a separate folder for the client.
Create the project (typescript + any UI framework)
Install the SDK  and initialize it.

Hey, Mukesh just came across your project and this issue.
I am at a very beginner level for open source but I would like to contribute in this issue.
So I just wanted to make sure that what you want to do in this issue is change the app's frontend ?

I would like to work on this issue

@EmilioBerumen910 Sure, please go ahead.

@muke1908 Thank you very much for the opportunity to participate in this project. I wanted to share my concept for how the minimalist client UI might appear and would appreciate any feedback before beginning the implementation process. This are some drafts of how I visualize the project UI:

Looks great!
Some suggestions:
1 - let's keep the primary color (which is orange here) configurable in the code.
2 - It can display the QR code along with the generated link - no separate button needed.

Hey @muke1908 why is this issue still open? I'd like to contribute on your project! Is this a good place to start?

@lumariaap15 sure, please go ahead.

@muke1908  Is the issue still open? Can I contribute to it?

@Abirami04M , yes. Just make sure that you create a separate folder for your UI code. It needs to be unique so that it doesn't conflict with others (if any).
Probably ./ui-abirami04M/ ?
@lumariaap15 , if you are also making an UI, can you also please make sure that it's in a unique folder like './ui-lumariaap15 ?

Ok, just to clarify, can I use React for the UI?  and is there any time
span within which i should complete the work?
On Thu, Aug 15, 2024 at 12:14 PM Mukesh @.***> wrote:

@Abirami04M https://github.com/Abirami04M , yes. Just make sure that
you create a separate folder for your UI code. It needs to be unique so
that it doesn't conflict with others (if any).
Probably ./ui-abirami04M/ ?
@lumariaap15 https://github.com/lumariaap15 , if you are also making an
UI, can you also please make sure that it's in a unique folder like
'./ui-lumariaap15 ?
—
Reply to this email directly, view it on GitHub
https://github.com/muke1908/chat-e2ee/issues/297#issuecomment-2290766483,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A47IL4VIIZMCYXJ4EFXF3SLZRRE5PAVCNFSM6AAAAAAY64KLU6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEOJQG43DMNBYGM
.
You are receiving this because you were mentioned.Message ID:
@.***>

Sure, feel free to use any framework.


[FIX] split MEG files should be listed separately in scans.tsv

As discussed in #692 in the mne-bids repo, MEG recordings that consist of split files, all parts should be listed separately in the scans.tsv file.

@eort could you please add yourself to the contributors list here --> https://github.com/bids-standard/bids-specification/wiki/Recent-Contributors

I haven't read the context, but it should be clear that is only about FIF files.
For other CTF and 4D MEG recordings, the split files are within the folder referenced in the scans.tsv.
(for other manufacturers, I don't know...)
Note that the split FIF files are linked with hard-coded file names (tag FIFF.FIFF_REF_FILE_NAME).
Ideally, renaming them should be done carefully by a dedicated program.
Does MNE-BIDS does this?

Ideally, renaming them should be done carefully by a dedicated program.
Does MNE-BIDS does this?

I'm actually not entirely sure how well this is currently being handled, but if we're not doing this properly in MNE-BIDS, then it's a bug we need to fix. :)

Ideally, renaming them should be done carefully by a dedicated program.
Does MNE-BIDS does this?

I can't say anything about how consistently MNE-BIDS does that, but for my data it works fine.

I'm actually not entirely sure how well this is currently being handled, but if we're not doing this properly in MNE-BIDS, then it's a bug we need to fix. :)

pretty sure we are just using MNE-Python for that in mne-bids, and MNE-Python does properly rename multi-part FIF files. It's even mentioned as an example/recommended tool to use in the bids spec

We should clarify:

Multi-part runs should be discussed in scans.tsv, not just in split, and the same acq_time is REQUIRED across parts of the same run (e.g., echo-, part-, split-)
Explicit note in split that acq_time should be the same for all splits, not acq_time(split2) == acq_time(split1) + duration(split1)
scans.tsv remains OPTIONAL

Multi-part runs should be discussed in scans.tsv, not just in split, and the same acq_time is REQUIRED across parts of the same run (e.g., echo-, part-, split-)

My personal opinion is that the idea that all files should be listed in scan.tsv (when provided), should specifically be brought up in the description for scans.tsv and not in the definition for split.
Something like:

When the OPTIONAL scans.tsv file is provided, split files MUST have the same acquisition time (acq_time).

@tsalo wrote

...all files should be listed in scan.tsv (when provided), should specifically be brought up in the description for scans.tsv ...

For FieldTrip data2bids and examples I have also been pondering about what is to be listed in the scans.tsv file.  That is most explicit here. But I don't find it so clear what constitutes data (to be listed) and what metadata (not to be listed).
E.g. for PET there are the pet.nii files (obviously), but should the blood.tsv files also be listed in the scans.tsv? And for fMRI the bold.nii files would be listed in case scans.tsv is provided, but should physio, stim and events also be listed in that case?
However, for the concrete case here (split/part/echo) I don't think this confusion applies. I do like the suggestion to require acq_time to be the same for all.

all files should be listed in scan.tsv (when provided), should specifically be brought up in the description for scans.tsv and not in the definition for split

Yes, I agree. But should these two things be only mentioned in the scans.tsv section or also in the split sections?

E.g. for PET there are the pet.nii files (obviously), but should the blood.tsv files also be listed in the scans.tsv? And for fMRI the bold.nii files would be listed in case scans.tsv is provided, but should physio, stim and events also be listed in that case?

That's a good point that we should clarify as part of a bigger issue/PR :+1:

Multi-part runs should be discussed in scans.tsv, not just in split, and the same acq_time is REQUIRED across parts of the same run (e.g., echo-, part-, split-)

This should be specified in this section: https://github.com/bids-standard/bids-specification/blame/master/src/03-modality-agnostic-files.md#L326
@eort can you make a proposal please?

Nice!
Once this is merged, does the validator need to be updated to test for that change?

Once this is merged, does the validator need to be updated to test for that change?

Taylor has opened an issue for that: https://github.com/bids-standard/bids-validator/issues/1192
but it will be non-trivial to implement, unless you are well versed in JS :) (in the latter case it may or may not be easy)

@tsalo Makes perfect sense! I adopted your suggestions.

According to ci/circlci, there seems to be a certificate issue for https://www.cognitiveatlas.org/task/id/trm_54e69c642d89b
I don't know what to do about it.

According to ci/circlci, there seems to be a certificate issue for https://www.cognitiveatlas.org/task/id/trm_54e69c642d89b
I don't know what to do about it.

that's nothing we can solve - we'll ignore it for now and revisit if it doesn't fix itself within a couple of weeks.

Sorry about that, cert should be renewed now.

Restarted the Circle job.

Thanks @eort


Add support for defining status as a subresource of flyteworkflow CRD

TL;DR
Adding support for setting status as a subresource on the flyteworkflow CRD. This should reduce traffic to the kubernetes API server by reducing update message sizes.
Type

[X] Bug Fix
[ ] Feature
[ ] Plugin

Are all requirements met?

[X] Code completed
[X] Smoke tested
[X] Unit tests added
[X] Code documentation added
[ ] Any pending items have an associated Issue

Complete description
Setting status as a subresource on the flyteworkflow CRD introduces a separate kubernetes rest API which partitions updates of the CRD spec and status subfields. This functionality seems straight-forward, but introduces complexities when trying to provide backward compatible support for existing deployments where the CRD doesn't have the status defined as a subresource. Specifically, the cases are described below for both the existing and non-existing CRD status subresource and the effect of the Update and UpdateStatus API calls.
case 1: subresource exists on CRD
1.1 client.Update(...) -> succeeds, but status does not change - update doesn't care about changing status subresource
1.2 client.UpdateStatus(...) -> succeeeds
case 2: no subresouce on CRD
2.1 -> client.Update(...) -> succeeds (status does change)
2.2 -> client.UpdateStatus(...) -> throws k8s error NotFound  - impossible to distinguish from 1.2 where the resource does not actually exist
Therefore, we determine if the status subresource API is available and use it to inform the correct commands to update flyteworkflow CRD status and spec subfields. This means the implementation works for the existing flyteworkflow CRD definition (ie. without status subresource) and offers advantages once the subresource is added.
Tracking Issue
https://github.com/flyteorg/flyte/issues/126
Follow-up issue
NA

@EngHabu is it sane to structure the hasStatusSubresourceAPI handling in the controller logic? i'm trying to convince myself this approach is better than burying it in the workflowstore Update function. is there a compelling reason why workflowstore has separate Update and UpdateStatus functions besides mimicing the kubernetes API? logically, the controller shouldn't care what API calls update the flyteworkflow CRD, just that it happens correctly.

It was decided that at the moment support for this feature faces complications in deployment and upgrading of flytepropeller and flyte kubernetes components. See https://github.com/flyteorg/flyte/issues/126 for further explanation.


[Feature request]
Animation retargeting and inverse kinematics

I need to share/retarget animation from one model to another sharing same skeleton structure.It would be nice if libgdx provide an easy api.

You might get some good ideas and/or usable inverse kinematics code from @Raxorg 's Alejandria repo: https://github.com/Raxorg/Alejandria/blob/master/core/src/com/epicness/alejandria/showcase/modules/kinematics/InverseKinematics.java
I haven't used the code yet, but the demo that shows Alejandria is impressive.


Added test cases for rancher-compose up options

@alena1108 @ibuildthecloud , can you review the rancher-compose test cases .
Currently test cases are written for "up" option.
As part of rancher-compose test plans , I have moved the validation and setup libraries of services, lb services , link services , dns services and external services to common_fixtures.py.

@sangeethah Sangeetha, validation for the following seems to be missing:

instance healtcheck
load balancer config (including healtcheck/stickiness policies)

Also you might want to look at the latest docker-compose ref to see if there is anything else missing (and perhaps find out what is not implemented yet on cattle/rancher-compose side): http://docs.docker.com/compose/yml/

I will be adding more test cases for instance healtcheck and load balancer config (including healtcheck/stickiness policies) in the next PR . There cases will not have validation of the health/LB rules but will have validations at the API level to make sure that the healtcheck and stickiness related configurations are set.

LGTM


Address stability issues when receiving signal interrupts

Currently, we have some stability issues when receiving signal interrupts (such as CTRL-C on the command line) that may cause a core dump from the Device Client (as reported by @janborch). We need to review our handling of these interrupts and provide stability improvements to the code base.

This issue is been resolved now. Please feel free to reopen this issue or create a new one if you see any segmentation faults.
PR: https://github.com/awslabs/aws-iot-device-client/pull/281


Create LICENSE

Links: https://fivetran.height.app/T-748713/description
Adds a LICENSE file that was previously missing

Addressed by PR #113. Therefore, closing this PR out.


[Web Install] Declarative vs Imperative version of cross-origin Web Install

Web Install is an imperative (JS) API. Is this better than declarative? What are the reasons for this?

I think everything would be so much easier if installing was as simple as writing
Install Google

and after clicking it, a browser would offer the user to install Google.com (the icon could be taken from the "href" URL from some  tag, or its manifest.json). You would be able to install a webapp from an online store of webapps, without even opening a webapp.
I think a browser shouldn't have any trouble preventing the abuse of this mechanism. Just like browsers prevent the abuse of alert() for the past 30 years. Why does everything have to be done in the most complicated way?

hola @photopea! Thanks for the feedback. We definitely want a declarative entry point for the Install API. It is not discarded, and I think it's a perfect complement to the imperative version. We've certainly discussed a declarative version of Web Install, and for the first version we want to push out we've carefully considered how to avoid having multiple versions of the same app installed. You can see these slides that summarize a bit the complexity.
Specially for cross-origin installs, which is where this issue is relevant, we want to make sure that only web apps are allowed to be installed. This implies having a mandatory manifest_id and reinforcing the concept of an install_url as well. Potentially this could all be stated in a declarative way as follows:
<a href="//www.webapp.com" target="_install" app_id="<app id>" install_url="<install url>">Install Web App

Even declaratively things start getting a bit more complex if we want to navigate away from a world where end users have N versions of the same app where N-1 of them aren't being able to be updated by the developer.
Working on the declarative approach allows us to get feedback for a navigator.install(<app id>,<install url>); and gives plenty of flexibility for developers to for example, query if an origin has permission to install apps, and then proactively remove the install UX and display a link using JS.
Nonetheless, I 100% agree and want a declarative version of this API, and I don't see any reason why both, declarative and imperative versions coexist. Appreciate the feedback and further discussion you might have on the topic!

hola @photopea! Thanks for the feedback. We definitely want a declarative entry point for the Install API. It is not discarded, and I think it's a perfect complement to the imperative version. We've certainly discussed a declarative version of Web Install, and for the first version we want to push out we've carefully considered how to avoid having multiple versions of the same app installed. You can see these slides that summarize a bit the complexity.
Specially for cross-origin installs, which is where this issue is relevant, we want to make sure that only web apps are allowed to be installed. This implies having a mandatory manifest_id and reinforcing the concept of an install_url as well. Potentially this could all be stated in a declarative way as follows:
<a href="//www.webapp.com" target="_install" app_id="<app id>" install_url="<install url>">Install Web App

Even declaratively things start getting a bit more complex if we want to navigate away from a world where end users have N versions of the same app where N-1 of them aren't being able to be updated by the developer.
Working on the declarative approach allows us to get feedback for a navigator.install(<app id>,<install url>); and gives plenty of flexibility for developers to for example, query if an origin has permission to install apps, and then proactively remove the install UX and display a link using JS.
Nonetheless, I 100% agree and want a declarative version of this API, and I appreciate the feedback or further comments you might have on the matter.
cc @dmurph @amandabaker

@photopea following up on this, do you have any use cases you require with the install API that the shape of the API wouldn't be able to fulfill? Please let me know.

Good to document this idea. I think it could compliment things if it's needed. I don't want to block the existing more flexible API from shipping on this though. This is a good follow-up if there is enough demand.
Cases where this might not be as suited:

Sites like Adobe Creative Suite, MSFT 360, game sites like GeForce Now etc, which want to somtimes install a number of apps all at once.

The JS API makes this possible, although currently we are being conservative with user activation so only one would happen at a time - but possible for future. Especially possible for enterprise devices, where they may 'allow' their internal sites to call this more in this pattern.

I think "installing a webapp" should not be anything more than adding a website into bookmarks, but a bookmark will be on the homescreen, with a special icon, which opens the website without a brower UI. Manifest files should be optional.
I think it is a very wrong path to treat installed apps differently than websites, and to create a "new, parallel standard", making webapps which would exist "parallel" to websites, and developers would basically have to learn a new technology. I am afraid you want to turn "good old websites" into "Android", where apps would be extremely hard to get "verified" to be installed, and a website would require tons of effort to turn into a webapp.
So basically, an installed webapp should be a URL (similar to bookmarks). A specific URL can be "installed" (added to homescreen) just once.

I think that generally matches how we are treating them as well. Some things 'only' make sense when you have OS integration, like handling files now can work (e.g. photopea can be a file handler for someone double clicking on .png file on their OS). And then now it's an OS entity, let it do things like specify the little 'shortcuts' / 'jumplist' things that appear when you right click on the dock icon on window/mac. But otherwise - yeah it's just a website / tab in a different dressing.

I have a JSON with web games, that I have created. I want to make a simple site, which loads this JSON and lets the visitor play these games, I made this: photopea.com/games.
I want to let the user install each game. Each game has a different URL, e.g.

https://www.photopea.com/games/peasmaker
https://www.photopea.com/games/dino-hunt-2

My PHP generates a custom manifest.json for each of these games (since Chromium doesn't allow it to be generated dynamically within a webpage). However, when I install one game, it considers all games to be installed. Is there a way to overcome this? This link says that it is discouraged to have multiple PWAs on the same domain.
I think that each URL should be allowed to be installed. The installation should be initiated by a website (by a visitor), and not by a web browser (and sending the "beforeinstallprompt" event). A browser should look for manifest.json only after the installation has been requested.
The current limitations do not help anyone, they only overly complicate everything.

@photopea said:

However, when I install one game, it considers all games to be installed. Is there a way to overcome this?

What considers all games to be installed? You may have an issue with your 'scope' variable - you should make sure that the scope of the apps do not overlap. Nested scopes are highly discouraged - in your example, if photopea is installed, then the browser will not promote your 'nested' app games as those urls are controlled by photopea. In the future, we want to allow developers to have more control over their scope (see https://github.com/w3c/manifest/issues/996 and https://github.com/WICG/manifest-incubations/issues/105) which helps here.
@amandabaker I think we should consider preventing install if there is an app already installed that controls the start_url of the app, or the current document in the same-origin case. This is mostly to prevent this kind of use, which hinders the UA's ability to promote installation / know which urls belong to which app (and other problems cited in here. Then, when we solve https://github.com/w3c/manifest/issues/996, devs can make sure the places they need to installed (if nested) fall outside of the outer app's scope.

I was talking about the current implementation of PWAs in Chromium-based browsers. E.g. these two games:

https://www.photopea.com/games/peasmaker
https://www.photopea.com/games/dino-hunt-2

Each has a different manifest file, but if I install the first game and open the second one, the second game is not available to be installed anymore. Could you tell me what scopes should I give to these games?

It looks like your manifest is here?
(hint: check out chrome://web-app-internals to see properties of your web app).
I would probably recommend a static file, but that works. It looks like you set the start_url to be "//www.photopea.com/games/peasmaker". When parsing this using the spec algorithm, that makes the 'scope' of your app to be "www.photopea.com/games". That is why all URLs under that are controlled by the one game.
Fix options:

Add a '/' at the end of your start_url, so it actually resides in the scope of the game. BUT - this will mean that any existing installations will no longer update, as the manifest id will have changed. To prepare for that, you can set the 'id' field as I outline below.
Or, Specify a scope directly in the manifest, probably as https://www.photopea.com/games/peasmaker/ or https://www.photopea.com/games/peasmaker. Relative urls also work - note they are relative to the manifest url.

I also strongly recommend setting an 'id' field to uniquely identify your app. For backwards compatibility, I recommend setting it to /games/peasmaker (as recommended by dev tools) to make sure any manifest updates will also apply to any existing app installations.


Improve deletion UX

use a ReplyKeyboard to suggest the user a list of his aliases;
use a switch_inline_query_current_chat InlineButton to suggest the user a list of saved objects.

When we send some suggested text, the form is expected a value for the deleteAll field. This is a severe bug.


3.10 support

Codecov Report

Merging #6 (2f23592) into main (49dd7ec) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main       #6   +/-   ##
=======================================
  Coverage   93.68%   93.68%
=======================================
  Files          10       10
  Lines        1156     1156
=======================================
  Hits         1083     1083
  Misses         73       73

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 49dd7ec...2f23592. Read the comment docs.


syslog restart not possible

With the recent changes of running syslog-ng outside of sv, the restart syslog-ng lines in logrotate doesn't actually do anything.
Now that's an other situation where I think abandoning sv wasn't a good idea, I think it'd a better solution to manually control the order of sv services from my_init instead.
But the most important question now:

Why do we need to restart syslog-ng after logrotate?
If we do need it, how do we achieve it now?

I don't know that we do, unless there are memory/leak issues. Otherwise I say we remove the restart of syslog-ng. More research may be required.
Not sure off the top of my head.

Will be fixed with #478

Merged and fixed.


DisplayList should compute bounds directly

When the DisplayList mechanism first landed it used SkPaint to compute the bounds of its rendering operations, but that mechanism has drawbacks because we do not track attributes in a way that always produces optimal results from the helper method on that object. Having to stock the attributes in the SkPaint while interpreting the DisplayList also adds some overhead when most of the bounds calculations are simple to compute. Dedicated bounds computations written directly to the DisplayList dispatch interface would be faster and potentially produce more optimal bounds.
(This is an "after-the-fact" issue for PR https://github.com/flutter/engine/pull/28656 to link from the umbrella DisplayList issue for tracking purposes...)

Fixed by https://github.com/flutter/engine/pull/28656


About code release

Hi, @nintendops ,
When will the code be released?
Thanks~

Hi @amiltonwong ,
I will find a day or two this month or in July to finish the final release. For now, the source code is available in the beta branch but they are not well documented and may need some fixes to the code structure. If you want to use it you may first need to write a setup.py to compile the cuda codes in vgtk/cuda. The final release will restructure the code in this branch and also will include links to download the datasets used in the experiments, as well as several pretrained checkpoints

Thanks @nintendops !


atlanto-occipital joint

Add new term:
abnormal atlanto-occipital joint morphology ( a child of abnormal synovial joint morphology)
DEF: any structural anomaly of either of a pair of condyloid synovial joints between the superior articular facets of the atlas and the condyles of the occipital bone
SYN: abnormal atlanto-occipital articulation
SOURCE: https://books.google.com/books?isbn=9350901994, UBERON:0000220
Annotation in J:228439

Added new term:
abnormal atlanto-occipital joint morphology MP:0030810


Question about hidding the URL bar

Hi :)
I was wondering if it is possible to send options to the browser tab like in InAppBrowser for exemple to hide the URL bar etc.
Thanks.

No, it's not possible, as in-app browser tabs (Chrome Custom Tab on Android, and SFSafariViewController on iOS) don't support such an option.

Ok that's what i was thinking. Thanks for the reply !


Buttons to copy & download outpout.

Add buttons to copy & download repopack output.

Completed by release  v1.0.1


require riotgear cannot find riot

I did require riotgear. it gives runtime error of riot not found. I have done npm riot as well

I'm going to need more info to help.
Server or client?
What does your code look like?

18 days old with no response. Closing.

Same problem here. this happens if you just require riotgear without a riot object in place.
requiring riotgear doesn't give back anything other than an empty object though


feat: update velero version

Automated changes by the update-addons GitHub action

Merging into a branch.


use acvm commit with log directive [DO NOT MERGE]

This uses an unmerged acvm commit. I have pushed this to get my mv/logging PR (https://github.com/noir-lang/noir/pull/630) matched up with the new remote acvm repo and some of the changes I made there.

Does #51 replace this?

Does #51 replace this?

Yes it does, closing this PR


DefinePlugin does NOT inject variables into require() call, if variable was preceded by runtime variable.

Do you want to request a feature or report a bug?
Bug 🐛.

What is the current behavior?
DefinePlugin doesn't replace variable used in require() call, if this variable is preceded by other runtime variable. It looks like problem exists on in the bundle.js file (final result of the bundling process), but not in 'resolving dependencies phase', as you can notice that webpack copies only necessary files.
Thanks to that, there is no error when bundling project, however it's impossible to run generated code, because it has references to undefined (unreplaced) variables.
If the current behavior is a bug, please provide the steps to reproduce.
Here is minimal project, that allows you to reproduce error
As the result you will get bundle.js file, with __FILE__FORMAT__ not being replaced. However, dist/assets will contain only files with configured format.

What is the expected behavior?
All variables passed to DefinePlugin should be replaced in the bundle.js file.
If this is a feature request, what is motivation or use case for changing the behavior?
Please mention other relevant information such as the browser version, Node.js version, webpack version and Operating System.

Webpack 3.5.5 (latest)
Node 6.11.2 & Node 8.0.0
macOs 10.12.6

Yep this bug exists since webpack version 0.


Both sides of ternary operator evaluated in extra_arguments block

I am trying to conditionally set an environment variable depending on the value of a local variable. Simplified terragrunt.hcl:
locals {
  k8s_cluster = "mycluster"
}

terraform {
  extra_arguments "k8s_provider" {
    commands = ["init", "plan", "apply"]
    arguments = []
    env_vars = local.k8s_cluster == "mycluster" ? {
      KUBE_TOKEN = run_cmd("get_k8s_token.sh")
    } : {}
  }
}

What I am finding is, regardless of the value of local.mycluster, the run_cmd is always executed.
Feels like a similar issue to https://github.com/gruntwork-io/terragrunt/issues/2172
I'm wondering whether Terragrunt still does Terraform <0.12-style ternary logic where both sides are evaluated and one discarded?

Hi,
maybe a bug in the HCL parsing implementation

Has there been any news on the topic? Getting the same issue where, in a code I something along :
locals {   is_replicated = var.replicate_from != null   from_bucket = local.is_replicated ? "some-text-${var.replicate_from.element}" : null }
When I test this in local using only terraform, no problem but in our project that uses terragrunt I get a:
**Error: Invalid template interpolation value
var.replicate_from.element is null
The expression result is null. Cannot include a null value in a string**
It prevents us from running a terragrunt run-all plan locally


Add gradle version-2.14.1

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same formula update/change?
[x] Have you built your formula locally prior to submission with brew install <formula> (where <formula> is the name of the formula you're submitting)?
[x] Does your submission pass brew audit --strict --online <formula> (after doing brew install <formula>)?
[x] Does your submission adhere to Versions "Acceptable Formulae" guidelines?

I wish to fetch a historic version of gradle by brew, and I have to add gradle2.14.1.rb for my purpose.

gradle2.14.1 vs Gradle21401

This version of Gradle is too old for us to accept I'm afraid. See https://github.com/Homebrew/homebrew-versions#acceptable-formulae

Hmmm. 2.14.1 was released in June 16 and appears to be within 2 major/minor revisions of the current stable release which is:
3.1 --> 3.0 --> 2.14.1
Am I interpreting the rule incorrectly, or reading the version numbers incorrectly?

@DigitalArcSystems You're right, my mistake. @whvirgil2015 could you remove the older gradle versions as part of this PR?

Gradle version 2.14.1 is needed in Android Development in some cases.

@luckyandyzhang I can't merge this until the CI is green and comments are addressed. If you need it please feel free to open your own PR.


feat: support --watch flag to view status for --target=ibmcloud

status.UI needs to support targets other than Kubernetes..

Related to #63


Adapterrepository.php -> get returns sometimes null

i use laravel 11 on WIN11 in XAMPP Environment. With laravel 10 i had no problems.
but after upgrade it happens sometimes, that he read null from the config settings. not only the app_key, other config settings also.
i debugged a little bit, and see that this function in adapterrepository.php returns sometimes null, that means it is very instabil
the strange thing is, the production is an debian with apache and it works there without problems.
but anyway, there must be a "bug", or what do you thing ?
in the $this->reader there are 3 readers in - is that okay ??
Dotenv\Repository\Adapter\MultiReader {#48   -readers: array:3 [     0 => Dotenv\Repository\Adapter\ServerConstAdapter {#26}     1 => Dotenv\Repository\Adapter\EnvConstAdapter {#25}     2 => Dotenv\Repository\Adapter\PutenvAdapter {#27}   ] }
the echo line is from me for debugging purpose only
`public function get(string $name)
{
if ('' === $name) {
throw new InvalidArgumentException('Expected name to be a non-empty string.');
}
    ****if ($name == "APP_KEY"){
        echo "sdasdasd  -> " . $this->reader->read($name)->getOrElse(null);
    }****

    return $this->reader->read($name)->getOrElse(null);
}`

This is a known issue with putenv and getenv. They are not thread safe, and concurrent calls can corrupt memory. I'd recommend removing PutenvAdapter.

See https://github.com/vlucas/phpdotenv?tab=readme-ov-file#putenv-and-getenv.

Hello, great, finally someone who knows his stuff. laravel uses these functions, not me!! how can I remove the PutenvAdapter adapter? If only via Laravel, right?

With Laravel, you should never be using env directly in your code. Only put env in your config files, and then run the config:cache command. This means that your app doesn't actually call these functions at runtime, and you are never using these functions in a threaded context.

This is somewhat subtext in their docs, but they do point out that the env file is not loaded when config is cached: https://laravel.com/docs/11.x/configuration#configuration-caching.

It is also possible to tell Laravel to not use putenv and getenv. See https://github.com/laravel/framework/blob/v11.23.5/src/Illuminate/Support/Env.php#L26-L46; https://github.com/laravel/framework/pull/28908; https://github.com/laravel/framework/pull/28740.


new project name

i was thinking to use anubi as name. he is the egyptian god of the underworld and its gatekeeper.
other ideas?

Naming is hard ^^
Anubis sounds nice though.


Plugin tooling publishable can have out-of-band errors

See https://github.com/flutter/plugins/pull/6555#issuecomment-1286013980 for an example
In that case, what happened was:

Pre-submit passed, including passing the publishable check of pub publish --dry-run
10 days passed waiting for final review
It landed, and failed publishable in post-submit in pub publish --dry-run.

This is different from the known problem of OOB failures when actually publishing when pub adds new sever-side checks; in this case it appears that the new behavior came in a Flutter roll due to updates in pub. What should happen is that this failure would show up in the roll, but that doesn't happen because pub publish --dry-run doesn't do anything when there's no version change. This is the output from that test in the last roll:
============================================================
|| Running for camera/camera [@0:00]
============================================================
Package camera version: 0.10.0+3 has already be published on pub.

It never tries to run the new analysis, so the new failure didn't show up. I can see two options here:

See if we can get a new flag added to pub publish for "I know there's nothing to actually publish but let's just pretend, and actually run the checks".
Change publishable to locally bump the version before running --dry-run so the checks run.

I'd rather we do the former if possible, since any time we're making local changes to what we test we introduce the possibility of false results.

pub publish --dry-run doesn't do anything when there's no version change

It turns out it's our tooling, not pub publish, that is bypassing the checks.


Fix errors in data model

[ ] Move data product performance rating to application table
[ ] Add notes field to relationship between application and SBA
[x] contribution_to fields should be criticality
[x] satisfaction fields should be performance
[ ] Separate fields for "rationale": One for criticality, one for performance
[ ] Review notes document and thread for more errors https://nsidc.slack.com/archives/C045Y53PP4L/p1666127765240169

Also, consider adding FK for response_id to relationships to ensure that no cross-response relationships are created.

Update data model to match metadata fields: https://docs.google.com/spreadsheets/d/1ILQbBu4_43eId86wu3sIz-UKEHaGmaSlF1BCa5j_lmE/edit#gid=0
(nearly final as of April 28 -- Hazel will add another comment when it's complete)

Closed as OVE because we redid the data model, and many of these features are no longer relevant at this time.


all: fix compiler warnings

@hermet I'm sorry it took so long, I made the changes that you requested. please check now


Padding mismatch from Tacotron2 pre-processing

Hello, I ran into similar trouble as #52 while trying to fine-tune the Universal_V1 checkpoint using ground-truth aligned TC2 outputs.
In my case, the fine-tuning breaks as soon as it reaches the validation phase. The error message is something along the lines of:
Using a target size (torch.Size([1, 80, 132])) that is different to the input size (torch.Size([1, 80, 131])). This will likely lead to incorrect results due to broadcasting. Please ensure they have the same size.
  val_err_tot += F.l1_loss(y_mel, y_g_hat_mel).item()

I found that the padding performed in this line is different than the padding performed in Tacotron2.
I dug deeper and this does introduce a consistent difference of 1 frame between the mel-specs generated by the trained TC2 model and the hifi-gan loss mel-specs.
Technically I could simply edit the padding code in this repo and get the mels to align perfectly but I wonder if this would cause issues since the model is trained using differently padded mel spectrograms. What are your thoughts?

Please understand that we are a bit busy with other work.
The padding is a matter of how to consider the center. You can easily figure out what the center is.
Since it doesn't affect quality significantly, cutting spectrograms to match its lengths can be the simplest way.

No worries! Yes, I realize that the padding issue is not a major problem since there will only be a few samples of mismatch.
For reference, I simply changed the TC2 mel spectrogram generation code to pad using the same scheme as the hifi-gan code. All spectrogram lengths match now, and should also align correctly

Dear @jik876
Can you please provide the code for fixing this issue?


split words via regex

difference can currently split via a single character, where " " is suggested to achieve word-level splits.
However other whitespaces, like tabs would not lead to word-splits. And punctuation does also not lead to a word-split.
Example:

the two strings Ok("hey") and Ok("hey ho!") will be split into words like this

Ok("hey")
Ok("hey + ho!")

Therefore difference will treat these strings as completely different, since no word is identical.
Suggestion:

allow regexes to define what a word is. This way it would be possible to define single punctuation characters as words, as well as all strings that neither contain whitespace nor punctuation. This would lead to the following word splits

Ok + ( + " + hey +           " + )
Ok + ( + " + hey + ho + ! + )

So, difference would be able to detect some overlap in the given example and only treat  ho! as an insertion! :-)
The difference crate could also export a reasonable default regexp to split words. However what you want will most likely very much depend on the context.

This suggestion is based on git diff --word-diff-regex=<regex>

Use <regex> to decide what a word is, instead of considering runs of non-whitespace to be a word.
Every non-overlapping match of the <regex> is considered a word. Anything between these matches is considered whitespace and ignored(!) for the purposes of finding differences. You may want to append |[^[:space:]] to your regular expression to make sure that it matches all non-whitespace characters. A match that contains a newline is silently truncated(!) at the newline.

Sounds reasonable, but I won't have time to work on this in the near future. Happy to take + review PRs. :)

Is this the reason why I get edit distance 2 on a="9", b="99" when split is " " or "\n"?
version 2.0.0
    #[test]
    fn test_diff() {
        init();
        let a = "9";
        let b = "99";
        assert_diff!(a, b, " ", 0);
    }

If the edit distance is based on LCS, this should be 1 the same as split=""


File explorer icons change with file focus

Thanks for taking the time to enter this issue. I do not think that we are going to be able to prioritize this one for the foreseeable future.
I'm going to close for now but if we receive enough interest / feedback we will certainly revisit.


Crash when [FlurryAd dealloc]

Hi guys,
We integrated Flurry in our App and it's helpful. Thanks.
For our current online version. We got crash reports from Fabric maybe related with Flurry SDK.
See the stack of crash here.
Our current version:

Flurry-iOS-SDK/FlurryAds (7.8.2)
Flurry-iOS-SDK/FlurrySDK (7.8.2)

Any idea?

Thanks for the post !
We are working on a fix for this issue already.
Should be out with our upcoming release.

@shruthimr Any update


Update version of OTel Collector

See: https://github.com/open-telemetry/opentelemetry-collector-contrib/issues/7380
Signed-off-by: Goutham Veeramachaneni<EMAIL_ADDRESS>

@gouthamve , seems there is a lot of interest in updating this 🎉
This is the 3rd PR that has been approved but still lacking a merge.
I opened one a few days back which update the version, download url and the metric.addr config to reflect the new changes. Please see here: https://github.com/prometheus/compliance/pull/65
Looking at your code and since it's updating for the *-contrib release, I wonder if there should be 2 targets, 1 for the main release and another for the contrib (this seems to overlap, but would be good to understand with which one we want to maintain this repo updated to.)
wdyt?

Hi, I think #65 is the better PR. I'll merge that one to unblock, and lets open an issue to see if we want contrib or otelcol.


[harness-simulation] add the usage of the CLI command nodeidfilter

This commit adds the usage of the CLI command "nodeidfilter". This command is supposed to only be run via THCI.

This command is supposed to only be run via THCI.

Technically it can also be used for manual experimenting/testing, right?

This command is supposed to only be run via THCI.

Technically it can also be used for manual experimenting/testing, right?

Yes, and it only works for simulation.


Cross-platform distribution

Get a pipeline working that can produce binaries for different OS targets. To be used for releases.

Should be covered by lacking-cli


OSTreeCommit test fails

The OSTreeCommit test consistently fails. I was able to replicate this failure locally and check the image logs. This is the failure message in the image's log:
Output:
/usr/lib/tmpfiles.d/journal-nocow.conf:26: Failed to resolve specifier: uninitialized /etc detected, skipping
All rules containing unresolvable specifiers will be skipped.
Failed to open file "/sys/fs/selinux/checkreqprot": Read-only file system
Initializing root filesystem
Copying data
fuse: device not found, try 'modprobe fuse' first
fuse: device not found, try 'modprobe fuse' first
bwrap: execvp true: No such file or directory
fusermount: failed to unmount /tmp/rpmostree-rofiles-fuseTmK2YZ: Invalid argument
fusermount: failed to unmount /tmp/rpmostree-rofiles-fuseEz4de2: Invalid argument
error: bwrap test failed, see <https://github.com/coreos/rpm-ostree/pull/429>: bwrap(true): Child process killed by signal 1
Traceback (most recent call last):
  File "/run/osbuild/bin/org.osbuild.ostree.commit", line 190, in <module>
    r = main(args_input, args_output, args["options"], args["meta"])
  File "/run/osbuild/bin/org.osbuild.ostree.commit", line 162, in main
    subprocess.run(argv,
  File "/usr/lib64/python3.9/subprocess.py", line 528, in run
    raise CalledProcessError(retcode, process.args,
subprocess.CalledProcessError: Command '['rpm-ostree', 'compose', 'commit', '--repo=/run/osbuild/tree/repo', '--add-metadata-string=rpmostree.inputhash=7194f8c94e47a572d772805b4fbe4820a347e22ddcfc666d28b2572a862db90c', '--write-composejson-to=/run/osbuild/tree/compose.json', '/tmp/tmph1u20mt3.json', '/run/osbuild/tree/tmpdm5njnol']' returned non-zero exit status 1.

@henrywang Have you had any luck replicating this failure? Have you seen a similar failure message?
@gicmo Have you seen any failure like this for ostree?
@martinpitt @marusak Have there been any changes to cockpit's fedora 33 testing images that might cause an issue with fuse or bwrap?

@jkozol : We got several image refreshes, but as TestLargeSystem has failed for so long, we started to ignore these failures. So it's plausible that something in rpm-ostree/bwrap/fuse/etc. regressed recently.

So this is the bwrap selftest inside our build root failing, from the python version I assume this trying to build F34? I wonder if it is working locally.

@gicmo It should be trying to build fedora 33 since it is running on a fedora 33 vm.


Teams Webhook Depricated

From the code mentioned they write for webhook for all communication channels except email, but for workflows they are made the script only for slack only.
Need to a workflow procedure for Teams also

Issue #92 reference same request


Fix sock.js when reconnectin over a unix:///path

var nano = require('axon')
var sock = nano.socket('req')
var address = 'unix:///tmp/some.path'

sock.connect(address)

In sock.connect(port, host) the address is actually the port who is parsed and passed as /tmp/some.path, so on reconnection the port is set without unix:// and the node net.js throws and error.
net.js:921
      throw new RangeError('port should be >= 0 and < 65536: ' + port);
            ^
RangeError: port should be >= 0 and < 65536: NaN
    at lookupAndConnect (net.js:921:13)
    at Socket.connect (net.js:898:5)
    at Socket.connect (net.js:860:37)
    at ReqSocket.Socket.connect (/Users/eugen/code/projects/evea/node_modules/axon/lib/sockets/sock.js:294:8)
    at null._onTimeout (/Users/eugen/code/projects/evea/node_modules/axon/lib/sockets/sock.js:279:12)
    at Timer.listOnTimeout (timers.js:89:15)

@TEugen while it may seemingly fix the issue, this is not a solution to this problem. self.connect still needs to be called in order to re-setup the proper listeners and create a NEW socket. In this function we could potentially keep a reference around of the unix path in order to pass the correct thing back into self.connect. See if you can implement it with that strategy and I'll review


Can't load Exercises_2_1_7.scala

Get errors from :load in scala-shell:
:load Exercises_2_1_7.scala
Loading Exercises_2_1_7.scala...
Exercises_2_1_7.scala:1: error: illegal start of definition
package sofp.unit
^
Exercises_2_1_7.scala:11: error: object eed3si9n is not a member of package com
import com.eed3si9n.expecty.Expecty.expect
^
Exercises_2_1_7.scala:11: error: not found: value munit
import munit.FunSuite
^
Exercises_2_1_7.scala:12: error: not found: type FunSuite
class Exercises_2_1_7 extends FunSuite {
^
Exercises_2_1_7.scala:14: error: not found: value test
test("Exercise <IP_ADDRESS>") {
... and many more

Suggest to de-couple that stuff.

These files are not designed for :load command. They are test files that can be run via sbt test.

Suggest to de-couple that stuff.
@andreas-roehler
Done. Now you can add your solutions to the worksheets/ subdirectory and sbt will ignore them. I moved your solution 1.6 .1.1 to worksheets/ and the tests pass now.


Fix link in README to kickstart a project with Astro

I noticed that when I visit the link "Follow these steps to kickstart a new project with Storyblok and Astro", the Astro tab is not preselected. The cause for this is a malformed URL leading to an exception (see screenshot below). Query params in a URL must always come before a hash location, because everything after the # is interpreted as part of the hash location, including ?.
I ran a global search of (.+)#(.*)\? on the repo and found a few more website links that needed fixing, all for the code of conduct (https://www.storyblok.com/trust-center?utm_source=github.com&utm_medium=readme&utm_campaign=storyblok-js-client#code-of-conduct). However, the hash location #code-of-conduct doesn't exist on that page at all, so there might be more problems with that link.
I also found a sign-up link to the app (https://app.storyblok.com/#!/signup?utm_source=github.com&utm_medium=readme&utm_campaign=storyblok-astro) but I'm not touching that one. Since the app is a SPA with, I assume, a hash-location based router, the rule of putting query params might not apply.
Error
Uncaught DOMException: Element.querySelector: '#astro?utm_source=github.com&utm_medium=readme&utm_campaign=storyblok-astro' is not a valid selector
    <anonymous> https://www.storyblok.com/_astro/hoisted.PgQtRDpH.js:14
    <anonymous> https://www.storyblok.com/_astro/hoisted.PgQtRDpH.js:14

It seems that the other framework repos (react, vue, nuxt...) have the same problem with the link.


pyobjc dependency for osx

missing dependency discussed in issue https://github.com/TaylorSMarks/playsound/issues/5
this should fix but I have not tested yet

This is now fixed on master. I have not added PyObjC as a hard dependency, but I print out a suggestion that you aught to install it. Even without it, the code still works on Python 3 on Mac, by shelling out to Mac OS's built-in python.


[ServiceBus] Discuss how we want to handle assumed/specified encodings across the various string-taking but byte-allowing fields.

azure.servicebus.ServiceBusSession.__init__.param(encoding

Context from apiview discussion
ramya-rao-a
9 weeks ago
What is this encoding for? How come it does not make an appearance elsewhere?
KieranBrantnerMagee
8 weeks ago
It technically makes an appearance elsewhere, just typically hidden in kwargs and not advertised since it's deep control functionality. In this case it's to control the encoding when state is set or fetched. (and as established above we don't expect a user to consume this initializer)
ramya-rao-a
6 weeks ago
What are the other places where one can set encoding?
KieranBrantnerMagee
47 minutes ago
Message kwargs, defaults to UTF-8.
KieranBrantnerMagee
just now
We'll be removing the docstrings to let this float until we settle on a cross-SDK decision on this. Adam has created an issue to track this as a post-GA discussion.

let's wait for customer feedback.


Bug: Fixed multiple things in the app

remove the setting logo
add Confirmation before logout
welcome page before sign up