SSR + axum + >0.4.5: panic - ExtractorHelper apparently does not exist in the context

Describe the bug
SSR + axum + --release build produces a panic:
thread '<unnamed>' panicked at 'should have had ExtractorHelper provided by the leptos_axum integration', /mnt/dev-data/work/git/projects/leptos/integrations/axum/src/lib.rs:1369:10

This does not happen with a debug build.
When I pin all leptos versions to =0.4.4 is does not appear and everything is fine, but as soon as I switch to 0.4.5, it panics and only with --release flag. This happens on the current main branch too.
I already found out, that it has nothing to do with the new nonce feature I am using. I am still trying to nail it down further which action exactly causes this. This is a bit tedious and time consuming though since it only happens with a release build.
As soon as I hopefully find out more about the cause, I will add a comment here.
Leptos Dependencies
SSR + start-axum example + leptos >0.4.5
To Reproduce
I am still trying to find the cause of this, which is not too easy.
If i remember correctly, there was a memory leak issue with axum, right? Seems like it could have something to do with the fix.

Is there any chance it's the same as #1397?

Could be. I nailed it down to a section where it happens in my testing project, but I am stil trying to reproduce it in a minimal example currently.
What I am doing on this scenario is a manual .set for a create_resource when the component is running on the server to actually get initial values out of an additional_context.
As soon as I am commenting out these lines, the panic does not happen again.

Okay, I managed to reproduce it, actually in 2 different versions:

The "good" thing is, it happens in debug mode too. I just did not see it before, because of a lot of debug logs and axum just restarted the worker silently
The root cause might be the same as in #1397 but I don't know enough about the deep internals of leptos yet to be sure
It has definitely something to do with leptos_axum::extract

The base for this example is the start-axum, just paste the following content into the app.rs:
use leptos::*;

#[server(AccessServer, "/api")]
pub async fn access_server(cx: Scope) -> Result<String, ServerFnError> {
    let _headers = leptos_axum::extract(
        cx,
        |headers: axum::headers::HeaderMap| async move { headers },
    )
    .await
    .map_err(|err| ServerFnError::ServerError(format!("extraction error: {:?}", err)))?;

    Ok("Hi from Server".to_string())
}

// create panic version 1
#[component]
pub fn App(cx: Scope) -> impl IntoView {
    let res = create_resource(
        cx,
        || (),
        move |_| async move { access_server(cx).await.expect("access_server") },
    );

    // if we do the manual `.set` here AND extract the headers in `access_server` -> panic
    // if we do not extract the headers and just `.set` -> fine
    // if we extract only the headers and don't do the `.set` here -> fine
    res.set("Manual Hi".to_string());

    view! { cx,

"We don't even need to access the result to get the panic"

    }
}

// // create panic version 2
// #[component]
// pub fn App(cx: Scope) -> impl IntoView {
//     let res = create_resource_with_initial_value(
//         cx,
//         || (),
//         move |_| async move { access_server(cx).await.expect("access_server") },
//         // if we just create the resource with an initial value other than None, the same
//         // as above applies: if Som(..) + extractor in the server fn -> panic
//         Some("Manual Hi".to_string()),
//         // None,
//     );
//
//     view! { cx,
//
"We don't even need to access the result to get the panic"

//     }
// }

I stumbled about this issue while creating an i18n component which absolutely needs the initial translations before the very first render on the server for a good SEO, so that the values do not end up in the <template> section in the browser, but actually in the  directly.

Yes that's what I do.
The result of the resource will alter on be used during client side rendering / routing, because then the translations will be fetched async. The same happens when I use create_resource_with_initial_value. The result is the same, I just have it wrapped in an if on_server() { } block to make it better readable in my code.
This example is really stripped down. In the project where I found the error, the translations are inserted into the additional_context in a custom handler for the leptos routes. This means instead of setting a default value, I can retrieve it from the context directly as long as I am on the server while still using OutOfOrder streaming, which gives me exactly the result I want.
I did some experiments with create_blocking_resource too, but then I have the values in the <template> again and not in the body, which means that not a single translation would work without Javascript / WASM on the client side.
I will take a look at SsrMode::PartiallyBlocked too. If I can specify the "i18n " to block and all other's to resolve out of order like before, this might work too.

When I switch to create_blocking_resource + render_app_to_stream_with_context_and_replace_blocks as you suggested, it is giving me the same result without panicking.
The panic does still appear when you just use create_resource_with_initial_value which gives the same result, but this is now working in my case. Thank you a lot!

Yeah makes sense to me. If it's all right with you, I'll close the issue. I think it falls under the annoying category of "you're holding it wrong" leading to an unclear result, rather than a bug per se. Although I'd be very open to a suggestion of how to document this kind of thing better, if you have any thoughts.

Yes sure we can close it then.
But shouldn't the create_resource_with_initial_value not panic in that case? Basically my second example in the code above.  Just out-comment the first <App> and use the second one. That should throw the same panic.
I cleary was a bit stupid, which is why I created this issue, but when I am using create_resource_with_initial_value I would not expect it to panic if I am accessing a server fn which has a custom extractor.
How to document this a bit better is a really good question. I guess at some point we could add a section to the docs with "typical use cases or tasks", or something like that. But on the other hand it kind of falls into the section "hydration bugs" and it could maybe be mentioned there (if it not already is and I just missed it).

I just tested it with the 0.5.0-alpha and I was unable to reproduce the problem with this version.
I would say we can close this issue.

Oh, excellent. I love finding all the little unexpected bugs this fixed.


Programmatically Set Device Tree Configuration

In order to enable the USB Power on the Wavehsare Carrier board we use for the Raspberry PI CM4, a dt_overlay variable must be set. Wiki link to the documentation.
dtoverlay=dwc2,dr_mode=host
Previously I've been setting this via the Balena Web UI, however, this makes this configuration obscure and not obvious to the Open Source community. We should consider setting this programmatically in code.

We should add the changes made as part of #61 in this changeset. Currently the settings below are set via the balena web UI.
t_params = "i2c_arm=off","i2c-gpio=on"
dt_overlay = "dwc2,dr_mode=host","i2c-gpio,bus=11,i2c_gpio_delay_us=100,i2c_gpio_sda=2,i2c_gpio_scl=3"

@djgood fyi in case you are ever messing with anything until I finish this item.

We're moving to an esp32-based frog for the foreseeable future, so closing this won't fix fow now. Perhaps once the global supply chain clears up a bit, we will revisit this.
New software repo below:
https://github.com/Ribbit-Network/ribbit-network-frog-software


javascript_redirect refactor

Ideally, methods calling javascript_redirect should be oblivious to the fact that a controller is RESTful or not. That way we can eliminate the conditional if restful_routed?(model) in multiple places every time we have to redirect to a URL.
javascript_redirect now does the RESTful check for us.
Fixes https://github.com/ManageIQ/manageiq/issues/1002
(This refactor is mostly applicable for the show route)

/cc @martinpovolny

Current: https://github.com/ManageIQ/manageiq/pull/11318
Query to find previous related work: https://github.com/ManageIQ/manageiq/pulls?utf8=✓&q=is%3Apr author%3Amartinpovolny is%3Aclosed RJS
e.g.: https://github.com/ManageIQ/manageiq/pull/10030
Counting the number of render :update do is an imperfect way to measure the conversion, sure so for Darga, we have:
devil - [~/Projects/manageiq] (HEAD)$ git grep  'render :update do |page|'  | wc -l
562

I am more concerned about the types of transactions that go/don't go through RJS.
Redirects are a candidate for conversion. It would be cool if you could follow the PRs that I reference above and reimplemen the redirects using JSON. You can also use plain HTTP redirects in some cases.

@AparnaKarve : what about leaving the javascript_redirect as is (esp. keeping it a low-level method that does not know about our special controllers and restful/non restful) and creating a wrapper around it that would do this decision and pass the correct URL to the javascript_redirect?
That would allow each of the methods to solve only one task and would allow us to change the low level javascript_redirect from RJS to JSON like we did with javascript_flash and other methods.

@martinpovolny Per your suggestion, I've now left javascript_redirect as is, and have created a wrapper called javascript_process_redirect_args.
Please review when you get a moment. Thanks.

@martinpovolny Can you review this when you get a chance? Thanks.


Remove session vars for redirect

This is a better approach for handling the redirect after someone logs in.
It also adds Schoology support to the auth/login page.
It does not handle the case where the portal is iframe'd and they user logs in with Schoology, I'm not sure when that is used.
This also does not fix the issue with the strange messages when an anonymous user first tries to access a restricted page, and then another user logs in who doesn't have access to that page.
See this PR for more details:
https://github.com/concord-consortium/rigse/pull/296
I think this might be good enough to release to production, but I'm going see about taking the approach suggested by @dougmartin of passing an extra param to pages when they are redirected to after login. This way the non-authorized exception handling can see this param and not show a confusing message.

OK I'm done with this now.
@pjanik I removed some code you just added. The remove is in this commit: https://github.com/concord-consortium/rigse/pull/304/commits/34883d6112eac5e2e947607c5d4ac5350ac8d8f7
If you can, take a look at the documentation I added:
https://github.com/concord-consortium/rigse/blob/remove-session-vars2/doc/logging-in.md and see if it makes sense. If there is something that you want more details on, I'll try to update the doc.

I forgot the corresponding change when I modified the approach for RESEARCHER_REPORT_HOST. It is in there now.

Looks good 👍 I won't merge in case you want to make the suggested refactoring first.


Improved Docker build & deployment

Files are now compatible with version 3 of docker compose.
Saved disk space and some time building/deploying containers.
Added volumes in order to preserve database data.
Removed exposed ports.

Closing, as these commits/changes are now contained within #367, and should be going into upgrade/mern-v2.4.0.
Thanks! 💯😀


UI: render sequence refactoring

Hi @larshp,
I've got one though about current rendering architecture.  Almost all pages have wrapping - header and footer. And this is repeated in each render.  Recent settings page does not call footer and so html is actually ends incorrectly.
No blaming, I've fixed actually, just thinking how to make it more automatic. I think it would be a good idea to call header and footer implicitly. This would mean e.g. that only page_super will implement render which will call header/footer and + some common method (render_page?) which will be specific and redefined for each page.
At the same time some pages have redirect, some have menu, some don't, etc ... So that render_page method should actually return a number of instructions that will control main renderer (show/hide menu, include script, actual html output ...).
Or there should be some separate class - page_renderer - which will care about header/footer/mainmenu and html consistency ...
What do you think about it ? The above approaches still don't look like a lean solution for me, maybe you have better ideas ?

shrug the gui is not perfect, but we also do not create a lot of new pages
feel free to come up with a suggestion

doing some refactorings in html stuff and pages current. I'd be greatfull if there are no changes to them for a while to avoid code conflicts :))

closing then.


[review] accuracy increase for longer simulations?

Is this the case?
Can the authors give some rules of thumb how fast the accuracy increases with simulation length? Is it always as simple as /sqrt(N)?

It's basically as simple as 1/sqrt(N), where N is the number of uncorrelated samples.  In most cases, the uncertainty of free energy calculations does behave as normally distributed, and thus the 1/sqrt(N) calculation applies. This needs a bit more thought, though, to make it clear why it's the case. As a rule of thumb, this is fine, though.

Was this a comment on a particular line number? Could help with edits along these lines.

I would imagine this is referring to section 7.2.5

Added this at the end of section 7.2.5 on Overleaf. Please review and close this issue if you're happy with the changes.

Tweaked a bit more, looks good to me now.

Great thanks! Will take a look in my big sweep.


Suggested changes for irc.rb to make it more easier to find out who owns the bot (add user/repo in realname).

Hi,
On our IRC network we have an issue with the Github IRC hook, a bot that keeps identifying to a name it does not own generates both annoyance for us as well the owner of the account. Because it is impossible for network operators to identify who owns the bot. I have written a small suggested change that adds the repository name and github account owner to the realname of the bot.
However, as I was writing the patch, my inexperience with ruby made it impossible to finish the originally intended pull request, and without the ability to properly test this, I'd like to appeal to the maintainer of the service (which seems to be @technoweenie) to verify if this will work, or is willing to integrate this on their own.
With that in mind, there is an issue at hand.  Private repositories will be visible in the realname. I think this issue is pretty trivial and it is the users own choice to enable the hook.  However I am willing to see if there are better alternative ways to implement this instead (perhaps by not mentioning the repo name, but just the user name). Alternatively it may be an idea to include the channel, but those can come in quantity, and may reveal secret channels due to this.
With this in mind, the suggested patch is below. But I was stuck at the stage where I could fully test my patch to make sure it actually works.
diff --git a/lib/services/irc.rb b/lib/services/irc.rb
index 4596bef..98bbc89 100644
--- a/lib/services/irc.rb
+++ b/lib/services/irc.rb
@@ -61,7 +61,7 @@ class Service::IRC < Service

     irc_password("PASS", data['password']) if !data['password'].to_s.empty?
     irc_puts "NICK #{botname}"
-    irc_puts "USER #{botname} 8 * :GitHub IRCBot"
+    irc_puts "USER #{botname} 8 * :#{irc_format_realname}"

     loop do
       case irc_gets
@@ -276,6 +276,12 @@ class Service::IRC < Service
     "#{fmt_name commit['author']['name']}: #{short}"
   end

+  def irc_format_realname
+    "#{repo.name}/#{sender.login} - Github IRCBot"
+  rescue
+    "Github IRCBot"
+  end
+
   def irc_issue_summary_message

Please consider implementing this patch to make it easier for IRC administrators and operators around the world to identify who the bot belongs to, so that they can contact the owner, and make them aware of the issue.

@Zarthus thanks for this! @technoweenie no longer works on this repository. I'm hopeful someone else in the community can work on this and transform it into a Pull Request.


Add support for both EnumStore and FormulaType for OptionSet in one build

We need a build of PowerFx that supports two parameters to PowerFxConfig (EnumStore and CultureInfo) AND be able to get the FormulaType for OptionSet.

Does this solve your issue: https://github.com/microsoft/Power-Fx/pull/182

Thanks @MikeStall  - @ToddN - can you please check?

@MikeStall @ckkashyap , yes, a build from this should meet our needs. By making the enum store the second argument, you force people to either provide CultureInfo or a null.  If you'd made the enum store the first argument, they could have left off the CultureInfo if they didn't have it.


No shadow on slider cirle on iOS

There should be no shadow on the slider circle. It is only reproducible on iOS.

Closed by #321


Bug - Invalid filtering type with service eks

This is an excellent tool. Thanks for creating it :)
I have filtered valid instances for eks like this but EKS complained that instance c6i.xlarge is not a supported one. I am using terraform resource aws_eks_node_group
❯ ec2-instance-selector --version
v2.0.3

❯ ec2-instance-selector --service eks --memory 8 --vcpus 4 --cpu-architecture x86_64 -r ap-southeast-2
c5.xlarge
c5a.xlarge
c5ad.xlarge
c5d.xlarge
c6i.xlarge
inf1.xlarge

Are you using a Managed Node Group?

Yes, I am using managed node groups. It failed when I had used Terraform to provision my code, which uses AWS APIs directly.

I wasn't able to replicate your issue with Managed Nodegroups. I used the AWS CLI to create a nodegroup with c6i.xlarge and my nodes are healthy.
What's the specific error you're getting back from the CreateNodegroup API? You can find me on the Kubernetes Slack too if  that would be faster.

Hey,
I'm having the same problem with eksctl that is using amazon-ec2-instance-selector.
I'm looking for managed node group, 4cpu, 8mem in eu-west-2, and this also results in c6i.xlarge that is refused by the node group. I'm using the latest eksctl version 0.76.0. (Sorry I can't say which version of aws-ec2-instance-selector they are using at the moment).

Hey, can you provide more details on the configuration you did use to start you node group please ? I tried to update AMI to the latest version without success, the same error did happen. At the moment, I didn't see any c6 started with node groups.
We have an uncompatibility with EKS atm.
Note that, to me, aws-ec2-instance-selector behaviour seems correct, maybe eksctl needs to provide a different filtering when using aws-ec2-selector so we maintain that compatibility.

Hey @bwagner5 @suket22 @vikas027 , as confirmed there, I think this ticket could be closed as resolved now no ?

Sure, we can close the ticket now.


Move to alternate async upload flow

Fixes
Fixes WB-NNNN
Fixes #NNNN
Description
What does the PR do?
Testing
How was this PR tested?
Checklist

[ ] Include reference to internal ticket "Fixes WB-NNNN" and/or GitHub issue "Fixes #NNNN" (if applicable)
[ ] Ensure PR title compliance with the conventional commits standards

copilot:poem

Current dependencies on/for this PR:

main

PR #5507   👈

This comment was auto-generated by Graphite.


ForegroundServiceStartNotAllowedException when playing from a home screen widget

Due to the behavior changes of Android 12, the app crashes when I play a track then the app loses AudioFocus, for which I have a callback when the app gains audio focus AudioManager.AUDIOFOCUS_GAIN that's where the media player crashes
The MediaNotificationHandler.java#L129   throws a ForegroundServiceStartNotAllowedException.
I assume when the app loses audio focus stopForegroundServiceIfNeeded(); is called
Would we need to update the playing media to WorkManager or what is the official guide to support playing media while the app is in the background on Android 12?

AndroidX Media 3
Android version 12
Android device Pixel 5

Hello, we have the exact same issue. What should we do?
Thanks!

Can you describe in a bit more detail how you are handling audio focus? Are handling it yourself in your app or are you using the internal audio focus support of ExoPlayer?
From the description above it looks like that playWhenReady changes which then stops the service from the foreground and later wants it to start in the foreground again when resuming after the audio focus is requested again. From this I conclude you are receiving a AudioManager.AUDIOFOCUS_LOSS.
If this is the case then it needs a user interaction from the foreground to start playback again, so you can't just automatically start playing again.
If my above guess is not correct, then you are receiving a AudioManager.AUDIOFOCUS_LOSS_TRANSIENT. In this case you will automatically get the audio focus back after the transient loss. In such a case you should not stop the service from foreground. instead keep it in the foreground and when you get the focus back you can continue playback without needing to start in foreground.
Can you let me know what you setup is in detail and who is calling player.play after the audio focus loss?

We are handling the audio focus ourselves with the following code when a user plays a song:
private void requestFocus() {
        if(hasAudioFocus) return;
        Log.d(Utils.LOG, "Requesting audio focus...");

        AudioManager manager = (AudioManager)service.getSystemService(Context.AUDIO_SERVICE);
        int r;

        if(manager == null) {
            r = AudioManager.AUDIOFOCUS_REQUEST_FAILED;
        } else if(Build.VERSION.SDK_INT >= 26) {
            focus = new AudioFocusRequest.Builder(AudioManager.AUDIOFOCUS_GAIN)
                    .setOnAudioFocusChangeListener(this)
                    .setAudioAttributes(new AudioAttributes.Builder()
                            .setUsage(AudioAttributes.USAGE_MEDIA)
                            .setContentType(AudioAttributes.CONTENT_TYPE_MUSIC)
                            .build())
                    .setWillPauseWhenDucked(alwaysPauseOnInterruption)
                    .build();

            r = manager.requestAudioFocus(focus);
        } else {
            //noinspection deprecation
            r = manager.requestAudioFocus(this, AudioManager.STREAM_MUSIC, AudioManager.AUDIOFOCUS_GAIN);
        }

        hasAudioFocus = r == AudioManager.AUDIOFOCUS_REQUEST_GRANTED;
    }

In our foreground service we implemented the handling of the audio focus change that sends events to our javascript side (we have a react native app):
    @Override
    public void onAudioFocusChange(int focus) {
        Log.d(Utils.LOG, "onDuck");

        boolean permanent = false;
        boolean paused = false;
        boolean ducking = false;

        switch(focus) {
            case AudioManager.AUDIOFOCUS_GAIN:
                paused = false;
                break;
            case AudioManager.AUDIOFOCUS_LOSS:
                permanent = true;
                paused = true;
                abandonFocus();
                break;
            case AudioManager.AUDIOFOCUS_LOSS_TRANSIENT:
                paused = true;
                break;
            case AudioManager.AUDIOFOCUS_LOSS_TRANSIENT_CAN_DUCK:
                if (alwaysPauseOnInterruption)
                    paused = true;
                else
                    ducking = true;
                break;
            default:
                break;
        }

        if (ducking) {
            //playback.setVolumeMultiplier(0.5F);
            wasDucking = true;
        } else if (wasDucking) {
            //playback.setVolumeMultiplier(1.0F);
            wasDucking = false;
        }

        Bundle bundle = new Bundle();
        bundle.putBoolean("permanent", permanent);
        bundle.putBoolean("paused", paused);
        bundle.putBoolean("ducking", ducking);
        service.emit(MusicEvents.BUTTON_DUCK, bundle);
    }

I do not know how the users are able to reproduce the issue since we see in the crash logs this stacktrace:
Fatal Exception: java.lang.RuntimeException: Unable to create service com.guichaguri.trackplayer.service.MusicService: android.app.ForegroundServiceStartNotAllowedException: Service.startForeground() not allowed due to mAllowStartForeground false: service com.esound/com.guichaguri.trackplayer.service.MusicService
       at android.app.ActivityThread.handleCreateService(ActivityThread.java:4953)
       at android.app.ActivityThread.access$1800(ActivityThread.java:310)
       at android.app.ActivityThread$H.handleMessage(ActivityThread.java:2300)
       at android.os.Handler.dispatchMessage(Handler.java:106)
       at android.os.Looper.loopOnce(Looper.java:226)
       at android.os.Looper.loop(Looper.java:313)
       at android.app.ActivityThread.main(ActivityThread.java:8663)
       at java.lang.reflect.Method.invoke(Method.java)
       at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:567)
       at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1135)

This happens only when the app is in the background and we guessed that it might happen when the users are listening the music, they receive a call and the app loses the audio focus and then after they ended up the call the click play again in the notification to resume the music and the app tries to start the foreground service again from the background and this exception happens.
This is the code we use when a user taps the play button in the notification and we requestFocus again:
    public void onPlay() {
        Log.d(Utils.LOG, "onPlay");

        requestFocus();

        if(!receivingNoisyEvents) {
            receivingNoisyEvents = true;
            service.registerReceiver(noisyReceiver, new IntentFilter(AudioManager.ACTION_AUDIO_BECOMING_NOISY));
        }

        if(!wakeLock.isHeld()) wakeLock.acquire();

        metadata.setActive(true);
    }

Thanks!

I'm also developing music player and already faced the same problem.
Classical scheme is to synchronize active playing state with service foreground state. Notification appears at the first play and keeps visible even when playback is stopped for the sake of usability.
So the only way to fix the problem for Android 12 is to keep foreground state forever (along with notification of course). As a user I'm quite disappointed with applications using that scheme even for old androids.

@vitamin-caig are you suggesting to never call stopForeground when we receive an interruption and to always keep the foreground service active?
What if the operating system stops the foreground service itself? I read somewhere that if you stay in a call for example for more than one minute Android may stop the foreground service.

@marf I would suggest to get rid of such a hard limitation in Android:) At least for multimedia playback services.
To be serious, of course this is impossible. But androidx can give an example of proper implementation for that cases. For example via WorkManager (extending it if required).

@vitamin-caig so do you confirm that if a user clicks the 'Play' button on app's notification this exception may happen?
If the OS stops the service the notification associated should disappear too, right? Hence the user will not be able to click the 'Play' button on the notification and get the exception if I am not mistaken.
Because I tried with my phone on Android 12 and the issue is not happening but it seems to happen for other users (don't know how they manage to make the issue happen).
Where I can find an implementation of the notification via WorkManager?

so do you confirm that if a user clicks the 'Play' button on app's notification this exception may happen?

Unfortunately, yes. Moreover, the same distilled issue happens when user tries to start playback from home screen widget - usually it uses intents bound directly to service. So no UI -> no foreground -> no permission for service to move itself to foreground state

If the OS stops the service the notification associated should disappear too, right?

It's up to an app. You may keep notification visible after service disabling its foreground mode. It's just convenient for users.

Where I can find an implementation of the notification via WorkManager?

I was pretty sure you are from androidx team so can contact with other maintainers:)

the same distilled issue happens

I hope the Android team patches this issue otherwise how can developers implements widgets or notification that are bounded to a service using Android 12? Do we need to use a WorkManager?
I have also put this in the manifest for the MusicService specifying also the android:foregroundServiceType as "mediaPlayback" thinking maybe the OS could avoid arising this exception if the foregroundServiceType is mediaPlayback but without any success.
        <service android:name=".service.MusicService"
            android:enabled="true"
            android:foregroundServiceType="mediaPlayback"
            android:exported="true">
            <intent-filter>
                <action android:name="android.intent.action.MEDIA_BUTTON" />
            </intent-filter>
        </service>

So the only way to fix the problem for Android 12 is to keep foreground state forever

I don't think this is correct. Can you let me know your reasoning why this is the case?
On Android 12 the System UI (notification) is sending a PendingIntent to your service and this pending intent creates a window of some seconds while your app is on a exemption list and hence is allowed to start a foreground service from the background.
You find an appropriate entry in the log in such a case which reports an allow list entry for 30 seconds:
2022-02-25 16:37:51.378 1824-8258/? I/ActivityManager: Background started FGS: Allowed [callingPackage: androidx.media3.demo.session; callingUid: 10314; uidState: SVC ; intent: Intent { act=android.intent.action.MEDIA_BUTTON cmp=androidx.media3.demo.session/.PlaybackService (has extras) }; code:START_ACTIVITY_FLAG; tempAllowListReason:<setPendingIntentAllowlistDuration,reason:NotificationManagerService,pendingintent:u0a229:android.intent.action.MEDIA_BUTTON,reasonCode:NOTIFICATION_SERVICE,duration:30000,callingUid:10229>; targetSdkVersion:33; callerTargetSdkVersion:33; startForegroundCount:8; bindFromPackage:null]

Classical scheme is to synchronize active playing state with service foreground state.

I agree to that.

When you receive a pause command you should stop the service from the foreground. The service is now running in background until the system decides to stop it (around 2 minutes). You stopForeground(/* removeNotification= */ false) and post the updated notification with the notification manager (updated notification now shows a Play button, for instance).

When then the user presses the play button within these 2 minutes from the notification. The PendingIntent is sent to your service and the service can be started in the foreground while on the allowlist. So you put the service into the foreground again.

That's how the Media3 MediaSessionService/MediaLibraryService is doing it and I think this works on S and as well on T. We removed all the `` we were seeing and I'm actually confident it works and works as intented with the pattern from above. I tested this and we were able to work around all issues with the most recent Media3 sources and with S and the most recent T build. For this to work you would have to build Media3 from the main branch. The binary releases don't have all the fixes needed to work for targeting API 29 through 33 and working on S and T.
The problem with the audio focus is that you don't have the exemption from the PendingIntent because you want to start the service in the foreground again, but because this is triggered by the audio focus and not a PendingIntent you are not on the allow list and you crash.
So I think when handling the audio focus and you get a loss that is  tentative, then you should keep the service in the foreground until you receive the audio focus again. If you get a permanent focus loss, then you should not start automatically again. Instead a user needs to resume from the foreground (like from the app or notification) which again gives you the chance to start in the foreground.
I believe that the case from above
case AudioManager.AUDIOFOCUS_GAIN:

only happens automatically if you lose the audio focus tentatively. Can you let me know if this is correct or if you experience it differently?
Please let us know whether you think I am wrong. :) We really want to make sure this works according to the best practices and we may have missed some corner cases that you guys see and for which we may have to file bugs internally to make sure it is working nicely on all OS version. Until that I would insist I am right and the pattern above should be used and works. :p

android:foregroundServiceType

Yes, confirming this does not help.

Thanks @marcbaechinger!
The foreground service is restarted only when the user press the play button again from the notification. We do not restart it again automatically so we should already do that. The onPlay method is only called when the user press the play button from the notification, the music is not resumed automatically when the call finishes it is the user which manually resumes it :)

Cool! A user pressing a button on the notification will allow you to start in the foreground as per exemption.
As far as this case
case AudioManager.AUDIOFOCUS_GAIN:
                paused = false;
                break;

is not called from the background you should not crash. If this case is triggered in the background (like when getting back the focus automatically after a transient loss) and paused = false makes you app call player.play() and then service.startForeground(int, not) later, then you may have the crashes you see in the crashlog.

I'm closing this issue with a note:
Starting with Android 12 a service needs to be started in the foreground from the foreground. Pausing the player causes the service to be taken off the foreground. If this hap[pens because of a custom implementation of audio focus, playback can not be resumed from the background, because this start the service in the foreground again which causes the ForegroundServiceStartNotAllowedException.
ExoPlayer provides integrated support for audio focus management. The implementation keeps playWhenReady true when playback is suppressed and provides a suppression reason. With this, the service is not taken out of the foreground when playback is suppressed which avoids the need for starting in the foreground when playback suppression is removed.
For these reasons audio focus support provided by ExoPlayer is the preferred approach in such a scenario.
Please re-open if you think that's indicated.


Not all entities respect FOV

All entites are opening up the map as visible, and some of them aren't being marked as not visible properly when they leave the FOV.
Here are some screenshots:

Entities being marked outside the FOV at start up:

Entites not properly respecting fov visible and not visible calls:

But depending from where you see the entity, they are disappearing:


Browsing IIS .xml log files no longer formats in browser

It used to be that if I clicked download on an application failed request trace.xml file (/LogFiles/W3SVNnnnnnnnnnn/frnnnnnn.xml) from the debug console that the file would just open in the browser formatted automatically for display. This is no longer working and instead the file shows in the download bar whereupon I then have to open outside the browser session. So the autoformat is no longer working
Have there been any changes in this area, have tried this on my usual computer and another fresh install one.

See this issue https://github.com/projectkudu/kudu/issues/1634
We're currently investigating the best way to handle this, either roll back the change, change the UI for both actions or have an exception for XML Failed Request Tracing.
If you also download the .xsl template your downloaded XMLs should show up nicely formatted.
Sorry for the inconvenience.

Many thanks for the response and it's good that this is in hand. So far I haven't managed to get it to display formatted though, opening the file in chrome is showing blank, but not looked any further as to what's happening.

You're right. Now it only works in IE when both XML and XSL are downloaded in the same folder.
I'm working on a fix as we speak.

I have found this extension which has fixed it for now and reversed the header:
https://chrome.google.com/webstore/search/undisposition?hl=en

Ok, @snobu reverted the change, so this issue will go away on the next Kudu deployment. And for now, you have that Chrome extension :)


gnumake 4.4.1 cant be used to build the project

PETSc is currently pinned to version 3.15.5 which has a build issue with never gnumake versions.
Currently on ubuntu this is not a problem yet, but this is likely to change in the future.
This issue could be addressed by bumping the petsc version or by just applying the patch.
It was reported here upstream https://gitlab.com/petsc/petsc/-/issues/1336 and fixed with the MR https://gitlab.com/petsc/petsc/-/merge_requests/6140

I did not see any issues with GNU make version 4.3. Do you have any log files ?

i updated the title to make this more clear. and the petsc issue also specifies that it breaks with gnumake 4.4.1


Provide a prop to enable/disable rotation on active.

In reference to #17,
The developer should have the option to decide whether or not to rotate the FAB on click.
In certain situations, it may not make sense to rotate, i.e: there are no FAB actions and there is an action bound to @click.native
It should default to always rotate but if the developer decides, they may disable with a prop.

@Edujugon I can take this one if you'd like


Adds: keysRotateUp/keysRotateDown (Free Camera)

Possibility to rotate Free Camera up and down using the keyboard.
https://forum.babylonjs.com/t/free-camera-rotation-keys-cameras-feature-reques/38891

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Snapshot stored with reference name:
refs/pull/13628/merge
Test environment:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/13628/merge/index.html
To test a playground add it to the URL, for example:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/13628/merge/index.html#WGZLGJ#4600
Links to test babylon tools with this snapshot:
https://playground.babylonjs.com/?snapshot=refs/pull/13628/merge
https://sandbox.babylonjs.com/?snapshot=refs/pull/13628/merge
https://gui.babylonjs.com/?snapshot=refs/pull/13628/merge
https://nme.babylonjs.com/?snapshot=refs/pull/13628/merge
To test the snapshot in the playground with a playground ID add it after the snapshot query string:
https://playground.babylonjs.com/?snapshot=refs/pull/13628/merge#BCU1XR#0

Please merge from master, we had a small issue with one of the tests.


"Import Devices" is not populating F5 Hosts section

Per https://clouddocs.f5.com/training/community/automation/html/class03/module1/lab01.html a method of importing a host via a json declaration:
[
   {
   "device"<EMAIL_ADDRESS>   "password": "admin"
   }
]

by right-clicking an open file editor containing the highlighted declaration and selecting "Import Devices" fails to add the referenced BIG-IP to the F5 Hosts window. The Output Tab produces:
233 [2021-09-29T15:06:37.111Z] [INFO]: ------ Environment Variables ------
234 [2021-09-29T15:06:37.112Z] [INFO]: F5_CONX_CORE_EXT_HTTP_AGENT=The F5 VScode Extension
235 [2021-09-29T15:06:37.112Z] [INFO]: F5_CONX_CORE_CACHE=/home/ubuntu/.local/share/code-server/extensions/f5devcentral.vscode-f5-3.1.0/cache
236 [2021-09-29T15:06:37.112Z] [INFO]: F5_CONX_CORE_LOG_LEVEL=INFO
237 [2021-09-29T15:06:37.112Z] [INFO]: F5_VSCODE_TEEM=true
238 [2021-09-29T15:06:38.333Z] [INFO]: ------ Environment Variables ------
239 [2021-09-29T15:06:38.333Z] [INFO]: F5_CONX_CORE_EXT_HTTP_AGENT=The F5 VScode Extension
240 [2021-09-29T15:06:38.333Z] [INFO]: F5_CONX_CORE_CACHE=/home/ubuntu/.local/share/code-server/extensions/f5devcentral.vscode-f5-3.1.0/cache
241 [2021-09-29T15:06:38.334Z] [INFO]: F5_CONX_CORE_LOG_LEVEL=INFO
242 [2021-09-29T15:06:38.334Z] [INFO]: F5_VSCODE_TEEM=true
243 [2021-09-29T15:09:41.115Z] [INFO]: authToken expired -> will refresh with next HTTPS call

F5 Extension version 3.1.0

As worked in the AGILITY-2021-AS3-FAST-ACC version 14 UDF Lab


PPPoE client over Bridge interface: problem with the ifname

Hi,
When you like to configure one PPPoE Client connection with LUCI, the UI doesn't work if your physical interface for the PPPoE connection is a Bridge.
Here some part of my current "/etc/config/network":
config interface 'eth-pppoe'
        option type 'bridge'
        option bridge_empty '1'
        option ifname ' '
        option proto 'static'

config interface 'test'
        option proto 'pppoe'
        option password '******'
        option username 'user****'
        option ifname 'br-eth-pppoe'

So, when using option ifname 'br-eth-pppoe' that's the name of the bridge (note the "br-") it works. However, the UI of LUCI always try to use the physical interface of the bridge; and it adds to the configuration only the name, like option ifname 'eth-pppoe'. This is true even when the brdige is empty, or when using a dummy interface on it. And then the client connection fails.
I request to remove this limitation and support bridge interfaces for PPPoE Client connections.
Regards.

Should work fine in master, arbitrary ifnames may be specified


Help on Dead by Daylight

I am trying to set rank to just 15, and I know that to give yourself rank 1, you change the number in SAM to 90.  So I did math stuff to try to find out what number rank 15 would be.  I did 90 divided by 20 (20 ranks in game), which is 4.5.  4.5 times 15 (my wanted rank) equals 67.5.  So I tried entering this number in SAM but I get a red alert icon next to the box and cannot store items.  Why is this? I want rank 15 because I don't want to go all the way to rank 1 just like that.  And I always get paired with people that are new since ranks are reset monthly I think and always end up dying because of teammates.  (I'm not trying to say I'm better than all of them)

It's probably an integer, not a float.

I've never used SAM.  Can you explain to me what you just said means.  Thanks and sorry.

Ok thanks, it works now lol.  I feel dumb.

Great.


Add a shortcut to toggle fullscreen mode. Fix #1187

This pull request fixes #1187 .
Add a shortcut(Super+Enter) to toggle fullscreen mode.
Could you help review the patch? Thanks.

What's Super?

@tbodt super is just a universal name for either the Windows key on Windows / Linux or the Command key on Mac.

Normal keyboard shortcut for Fullscreen on Windows is F11 or sometimes ALT+ENTER too. WIN+ENTER toggles narrator and guess should not be replaced.
I have no idea how it works but is it possible to have OS specific and different keyboard shortcuts in Electron/Hyper?

I think this should be configurable? cc: @matheuss

I don't think it's a good idea to set a global shortcut like that 🤔
Other than that, that's doable via a plugin, and as a recent policy we're not merging this kind of change (simple change, doable as a plugin) into the core codebase 🙌

I do think it's a good idea to set a global shortcut like that  ;)
More seriously, this is in the main menu so should have a Keyboard shortcut too and as it is common probably F11  (or/and maybe platform specific to Windows ALT+ENTER). A core functionality should be Keyboard accessible and be part of the core and not be implemented in a plugin.


Familiarize with literature and with the -xtevent- package

@rayhuang11 will:

[ ] Read FHPS (2021)
[ ] FHS (2019) could be useful too
[ ] Learn about -xtevent- funcionality with the test file here: https://github.com/JMSLab/xtevent/blob/main/test/test.do

Summary: In this issue @rayhuang11 learned about event studies and the xtevent package.


StudentServiceNeeded

Please accept my  request

@yaseen20051 There are several files in the request, along with file.zip. It must not be there. Please remove file.zip and make pull request again.  I have to close this request without merging. Thank you for your understanding.


add new syscalls collecting support

Motivation
Some intrusion method:

inserting kernel module for persistence
make a device node on disk utilizing mknod, for gaining larger attack surface

Examples in container escape:

container starts with CAP_SYS_MODULE: inserting kernel module
container starting with CAP_SYS_ADMIN: mknod can make a device node on disk, providing debugfs a chance to overwrite host's filesystem, with ability from the dangerous CAP_SYS_ADMIN config:   mknod syscall

Feature

https://falco.org/docs/reference/rules/supported-events/
👆 from the doc here, it seems that mknod and init_module has not been supported.
Apply for the support.

ei @MagpieRYL thank you for that, I added these syscalls into our umbrella issue https://github.com/falcosecurity/falco/issues/1998!
You are right! these syscalls are not supported in Falco or at least they are supported in "generic" mode (i.e. you can see when these syscalls are called but not which are their parameters)
Here you can find a more detailed report https://github.com/falcosecurity/libs/blob/master/driver/report.md

the green ones are the "fully" supported
the yellow ones are supported only in "generic" mode

Btw since these syscalls can have a relevant syscall impact maybe a generic implementation is not enough we will work on that ASAP :)

We should address this issue with:

https://github.com/falcosecurity/libs/pull/1242
https://github.com/falcosecurity/libs/pull/1270

(Thanks @therealbobo !)


got AP=0 ,when I use nuScenes dataset with only pedestrain and car.

I use nuScenes dataset for 3D detection, but when I test, I got the result of AP=0.
Here are some changes I make for 2 categories.
1、rewrite the contants.py and utils.py files under the nuscenes/eval/detection folder.
contants.py file, change as follows：
DETECTION_NAMES = ['car','pedestrian']
PRETTY_DETECTION_NAMES = {'car': 'Car',
                          'pedestrian': 'Pedestrian'}

2、utils.py file, change as follows：
function category_to_detection_name:
detection_mapping = {
        'vehicle.bus.bendy': 'car',
        'vehicle.bus.rigid': 'car',
        'vehicle.car': 'car',
        'human.pedestrian.adult': 'pedestrian',
        'human.pedestrian.child': 'pedestrian',
        'human.pedestrian.construction_worker': 'pedestrian',
        'human.pedestrian.police_officer': 'pedestrian',
        'vehicle.truck': 'car'
    }

function detection_name_to_rel_attributes:
if detection_name in ['pedestrian']:
        rel_attributes = ['pedestrian.moving', 'pedestrian.sitting_lying_down', 'pedestrian.standing']
    elif detection_name in ['car', 'bus', 'truck']:
        rel_attributes = ['vehicle.moving', 'vehicle.parked', 'vehicle.stopped']
    else:
        raise ValueError('Error: %s is not a valid detection class.' % detection_name)

3、convert_nuScenes.py file, change as follows：
CATS = ['car', 'pedestrian']

4、rewrite the config file as detection_person_car.json
"class_range": {
      "car": 50,
      "pedestrian": 40
    }

What's the problem? Why I got AP=0？
Can you help me ?

Are you able to reproduce the performance without changing it to just car and pedestrian? The default nuScenes eval API should print metrics for each category separately, and you can directly read from that.

In my project, car include "car, bus, truck"。 So I can't read these informations from the default nuScenes eval API.


Hygiene: enabled presets and various linters.

Changes
Context: https://github.com/tektoncd/pipeline/issues/5899
There are no expected functional changes in this PR.
Explicitly enabled:

maintidx
makezero
mustag
nakedret
nilerr
nilnil
noctx
nolintlint
nosprintfhostport

Implicitly enabled: all other default linters in the presets.

fixed presets configuration, which wasn't formated properly and did not in fact activate the presets
explicitly disabled some linters (enabled by presets) that highlight too many issues to resolve in this PR
fixed host-port issues identified by nosprintfhostport
removed defunct nolint:revive pragmas
fixed format of //nolint directives in conformance with nolintlint
added needed //nolint directives.

Note: hub resolver now passes context.Context through to http GET call. It is possible that, prior to this
change, the http GET would succeed despite a context timeout or cancellation; after this change, that call
would fail with a context timeout or cancellation error. I consider this a pathological condition and not a
functional change.
/kind cleanup
Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[N/A] Has Docs if any changes are user facing, including updates to minimum requirements e.g. Kubernetes version bumps
[N/A] Has Tests included if any functionality added or changed
[x] Follows the commit message standard
[x] Meets the Tekton contributor standards (including functionality, content, code)
[x] Has a kind label. You can add one by adding a comment on this PR that contains /kind <type>. Valid types are bug, cleanup, design, documentation, feature, flake, misc, question, tep
[N/A] Release notes block below has been updated with any user facing changes (API changes, bug fixes, changes requiring upgrade notices or deprecation warnings)
[N/A] Release notes contains the string "action required" if the change requires additional action from users switching to the new release

Release Notes
NONE

/hold
Annoying issues with linter versions and golangci-lint versioning.
My desktop has 1.52.2 where this configuration works, but our CI is using 1.50.1 where it doesn't. 👎
I may just abandon this; putting it on hold for now.

/retest-required

/retestI suspect thath

/unhold

/retest

/approve


Wallpaper not working

The retro wallpaper isn't visible unless i turn on the VPN and restart spotify. Is there any way to locally store and access it everytime rather than turning on my VPN for the same??

Without VPN

With VPN

When installing the theme without marketplace, you can edit the link in user.css to point to another image provider.
There used to be a way to use local assets in spicetify, but it was removed for some reason, I think.

Thankss!
On Sat, 28 Jan 2023 at 20:27, Martin Prokoph @.***>
wrote:

When installing the theme without marketplace, you can edit the link in
user.css to point to another image provider.
There used to be a way to use local assets in spicetify, but it was
removed for some reason, I think.
—
Reply to this email directly, view it on GitHub
https://github.com/Motschen/Retroblur/issues/13#issuecomment-1407415564,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AYQU45OQQEILXF3UMDF7KLLWUUXXLANCNFSM6AAAAAATTFIPX4
.
You are receiving this because you authored the thread.Message ID:
@.***>


No integration with navigation options

Doesn't support the use of react-router-dom or I haven't found a way to connect them.
No documentation to explain how it would work with navigation of any type even though the package specifies it is used for navigation (which hamburger menus often are).
The biggest issue is documentation as there is no way to know how to correctly use the package nor can it be found anywhere on the internet as far as I've looked. So applying CSS styles seems impossible, routing through the use of menu items seems impossible. In fact, adding any type of functionality to the menu items seems impossible.

Please check the example.

Example was checked. It is compatible with HTML native href elements for navigation.
Possible future feature could be enabling the use of react-router-dom elements (such as  and UseNavigate()).


drop xtend dependency

As far as I see it, xtend does the same job as the browsers' native Obejct.assign and is therefore obsolete as a dependency.
If there is no obvious reason, I might send a PR and drop the xtend- dependency.

Object.assign is not supported in Internet Explorer, unfortunately. I'd like to keep browser support in line with React, which supports IE9 and up (given you polyfill Map, Set and requestAnimationFrame)

Then i think a polyfill for object.assing should be used instead, then better browser would be able to garbe collect the things it don't need

xtend is already being used by upstream dependency remark-parse and the like, so changing to a different polyfill would actually add to the size of the bundle instead of reducing the size.


this context not auto-completing

When using the monaco-editor, I would expect the following to work:
function myFunc() {
    this.property = 'bar'
    this. // here I would expect auto completions to show up (including property)
}

Is there a way to get the monaco editor to autocomplete this properties?

solved my own question it seems to work by adding this:
 monaco.languages.typescript.javascriptDefaults.setCompilerOptions({
      target: monaco.languages.typescript.ScriptTarget.ES5,
})


Third Party Service IDs propose in-place update without code change

I'm facing an issue where my pipeline (terraform plan) tells me there is an in-place change for third_party_service_ids.
It happens on different resources, in my case these are:

site24x7_website_monitor
site24x7_ssl_monitor

I would expect no change, because code has not changed.
third_party_service_ids = [
          - "132963000043304102",
            "132963000043304106",
          + "132963000043304102",
        ]

This is a bug and it happens in multiple places https://github.com/site24x7/terraform-provider-site24x7/issues/269
Not sure, does anybody from team check this issue


Fix broken links in README file

This pull request fixes broken links in the README file that were pointing to outdated URLs. The updated links now correctly direct users to the appropriate resources.

Hi Felix, that you very much for fixing these. I completely forgot about the links, while moving the blog to the new domain :)

Thanks! Glad I could help.


VLAZ: Watch nordenvation.com

VLAZ requests the watch of the watch_keyword nordenvation\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

Approved by Jeff Schaller in Charcoal HQ


CAMEL-17175: camel-mllp option 'logPhi' with value 'false' does not work for exception logging

@luigidemasi can you please backport to 3.11.x too? Thanks.

@luigidemasi can you please backport to 3.11.x too? Thanks.

sure


When focus shifts to the bug when runnig and debugging code, do not close the file that I started running the code from.

When I am running and debugging the code, and an error shows up on the code, the focus goes to the area with the error, and the file that I was on is closed, so I have to search for the file I was using before the error showed up to start running it again. I would like it to happen that when the focus shifts to another file based on an error, that the original file that I was on does not go away from being able to be interacted with.

Are files that are closed have filename in italics and do you have Workbench › Editor: Enable Preview enabled? ("workbench.editor.enablePreview": true)?


Fallback method for ID validation

Creating a fallback methode for ID verification for users who cannot user their institute, Dutch Bank or eIDAS.

Design in process!


 I kill both at once with a swipe. The mad scientist's deathrattle should go off, and set duplicate.  Then, the duplicate should fire on the death of the Sludge Belcher

It already does, as far as I can see. The test (expected failure) you introduced in 7e2c5a358 has never failed.

This was apparently fixed by 9b7704f. (Assuming that 7e2c5a358 was supposed to be the test for this bug.)
What previously happened was that the Mad Scientist was duplicated instead of the Magma Rager.

Very true, it was fixed.  However, further experimentation indicates that the fix I gave isn't right.  Basically, I can either fix this or #92, but not both.  I'll close this one though, and leave 92.

You might want to remove the "expected failure" from the test as well.

I think I did already?  Unless I am missing something.  https://github.com/danielyule/hearthbreaker/blob/master/tests/card_tests/mage_tests.py#L623

Oh, sorry. :P


Add liquidity provision/commitment examples

Only merge when testnet has active LP deployment

9a56038 removes liquidity commitment bit from market proposal
55a8b7e removes the liquidity commitment amendment (as not working properly yet)
These can be reverted once needed.


Revamp accordion borders to fix pixel jumping

This should fix the issue once and for all—this takes a simpler approach to the borders and seems to fix all the movement I could see (which was actually visible without even zooming in, at least in Safari).

Pushes the borders to the .accordion-item instead of the .accordion-button
Simplifies the .accordion-flush's border resets
Recreates the .accordion-button's border-bottom with an inset box-shadow
Moves the border-radius to the parent, which necessitated the new $accordion-inner-border-radius variable

Fixes #32250, closes #32251.
Preview: https://deploy-preview-33149--twbs-bootstrap.netlify.app/docs/5.0/components/accordion/

Definitely better from a quick look!
It seems there's some new issues, like double border in the first item, missing border-bottom is the other items, and a slight border-radius issue in the last item

Pushed an update that I hope will address the FF zoom issue.

@ffoodd Any chance you could give me a quick review here? <3

I still see a little glitch but it's better.
FF zoom issue persists (I start to see weirdness from 110% to 170%, either a missing border or doubled ones).
I guess our main issue relates more to animating height than to borders. Shouldn't we try something based on visibility & max-height instead? My main reference on the topic is this van11y hide/show plugin (check the "Animation" section in "Bonuses").
Not quite sure this is the way to go, though…

@ffoodd I think we merge this as-is and then look at revamping the animations separately. Thoughts?

Agreed, :shipit:
Should probably open a new issue dedicated to animation though.


Expecting value: line 1 column 2 (char 1)

Hi,
Giving this a spin i'm not getting any data in either get_user_tweets, get_list_tweets, search, get_user_tweets, get_liked_tweets or get_user_timeline.
I like the pickled session resuming, good ideea.
The get_user_id returns correctly
Traced it back to the make_request inside request_util that raises the exception in the title, after the retries.
The response dictionary is as follows:
{'data': [], 'cursor_endpoint': None, 'has_next_page': True}
for the get_user_timeline these are the urls that get hit:
https://twitter.com/i/api/graphql/Uv42IObcWFhsgKHYEh5Brw/HomeTimeline
https://twitter.com/i/api/graphql/Uv42IObcWFhsgKHYEh5Brw/HomeTimeline
https://twitter.com/i/api/graphql/Uv42IObcWFhsgKHYEh5Brw/HomeTimeline
Expecting value: line 1 column 2 (char 1)
EDIT:
Just saw the closed issue.
I can confirm i'm not using proxies and I can use other twitter libraries. Python3.10.11

Duplicate : Check #2


feedback

The initial description of the project, paper, and goals was not particularly
clear.
The main goal appears to be to reproduce results from the paper - identified
specific results to attempt to replicate (image from paper), but you didn't
present any preliminary analysis towards that aim.
Identified struggles with applying preprocessing and linear modelling concepts
from class to their data.  This is a good first step and effort should be
redoubled to accomplish this ASAP.
Seemed to have a good grasp of their overall goal, but no specific, short-term,
managable goals toward that end. You need to complete the regression analysis
to better understand the data and the challenges of reaching their goal to
reproduce the paper's results.
You appear to be using statsmodels, but haven't mentioned it as a dependency.
In order for me to be able to run your code, you need to explicitly mention
dependencies as discussed during lecture 18.
You still have a lot of work to do, but there is still time.

I am worried about your teams progress.  It appears that you haven't done anything since last week.  Time is running short and you still have a lot of work to do.  If you have run into a major problem, please let me know immediately.
It appears that you've only added one code file so far:
https://github.com/berkeley-stat159/project-eta/blob/master/code/logistic regression.py
And it is essentially one or two lines of code.  If you don't start working on your project soon, I am concerned that you will have nothing to turn in for your project.  Remember the project is 55% of your grade.
Please review the grading criteria:
http://www.jarrodmillman.com/rcsds/notes/rubric.pdf
As well as my comments from lecture 22:
http://www.jarrodmillman.com/rcsds/lectures/day22.html

Thanks for the feedback and reminder, @jarrodmillman. We have been working on a few things, but have not pushed any changes to our own forks. Still, we understand we have a lot to do. All of us are making this a priority.
Again, thanks for the concern. We are taking it seriously.


Publish to /artifactory/ url instead of /gradle/

until repo.gradle.org is fixed again.

We don't need to do this, the problem was timeouts, not the URL.


Add a note about rebooting in WireGuard installation

WireGuard depends on a kernel module that may not be available until node reboot. Adds a note to the installation instructions regarding this.
Description
Documentation update for WireGuard preview.
Release Note

None required

Deploy preview for calico ready!
Built with commit 836bf6f9a6f685a04d118b5b261e81008259030f
https://deploy-preview-3607--calico.netlify.app


Tidy Budget

Requirements 💯

Design

[x] Design guidelines >> (Snapscan)[https://www.behance.net/gallery/19759151/Snapscan-iOs-design-and-branding?tracking_source=], including:

Colors.
Typography: font face, size, and weight.
Layout: composition and space between elements.

Interactions

[x] Splash screen

[x] Sign up and log in pages

[x] Home page (categories page)

[x] Transactions page

[x] "Add a new category" page

[x] "Add a new transaction" page

Testing

[x] Create unit and integration tests for all the most important components of the application.

Technical

[x] Use Postgres as database.

[x] Use devise for authentication.

[x] Validate all user input to make sure that anyone with bad intentions cannot compromise the app.

[x] App deployed and accessible online.

Like previously explained please follow instructions the flow you have used is only allowed for a group capstone. Your work wont get reviewed until you do the right thing which is compare your feature branch to development not master to development. Thanks

Hi @Abdelaziz-Mahdi ,
While you made a great effort in this project, unfortunately, I cannot proceed to review your code.
Invalid Code Review Request
You have submitted a project, that does not follow Gitflow as requested the project requirement
Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.
Invalid Code Review Request does not count into the code reviews limit.
As requested by code reviewer kindly compare feature branch to development for this is not a group capstone.
Maybe this will help you understand things better:
https://microverse-crs.slack.com/files/TJG70AZ1N/F05RRALN6CF?origin_team=TJG70AZ1N


Appearance swither only visible in light mode

Issue description
Whenever I open my website in the dark mode, either by setting the preference with the appearance switcher or with OS toggle, the theme switcher is not visible.
Theme version
v2.9.0
Hugo version
hugo v0.121.2+extended linux/amd64 BuildDate=2024-02-07T00:00:00+00:00 VendorInfo=Fedora:0.121.2-1.fc40
Which browser rendering engines are you seeing the problem on?
Chromium (Google Chrome, Microsoft Edge, Brave, Vivaldi, Opera, etc.), Firefox (Mozilla Firefox)
URL to sample repository or website
No response
Hugo output or build error messages
There is no Hugo error, I also checked the browser console.

I solved this, as well as some other CSS issues like broken padding, by deleting the assets folder.


How to get language code correctly?

Hi!
I am trying get a code as 'es-ES' to pass like it parameter.
I tried with "window.navigator.language". Firefox and Edge return "es-ES". But Chrome returns only "es".
Any solution?
Thanks in advance.

I have it!
var lang = (window.navigator.languages) ? window.navigator.languages[0] : window.navigator.language;


Terraform took forever to load

i m trying to run the AWSGoat but the Terroform took long time to load and i end up end the workflow.

Any idea why this happen?

After running the second time, there was an error

Please help me

nvm suddenly it works


Prepare 3.0.0-rc.2

Pull Request Test Coverage Report for Build<PHONE_NUMBER>2
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 80.868%

Totals

Change from base Build<PHONE_NUMBER>9:
0.0%

Covered Lines:
6408

Relevant Lines:
7924

💛  - Coveralls


Apply new config instantly

I thought it would be nice if the new config generated by the Custom Positioning Tool would apply immediately.
This would make it easier to change the position of multiple moons without restarting or manually touching the positions file

Thank you for this proposal !! that's super convenient ! (Sorry for the long delay...)


[Extensions] Kubectl Plugin for Kanvas Snapshot

Signed-off-by: Lee Calcote<EMAIL_ADDRESS>

Oh, wait. I was expecting to see a coming soon flag or ribbon on top of this. Do we have one?


add current profile annotations to CVO manifests

This is matches openshift/enhancements#510 and doesn't change existing behavior

Description of the change:
Motivation for the change:
Reviewer Checklist

[ ] Implementation matches the proposed design, or proposal is updated to match implementation
[ ] Sufficient unit test coverage
[ ] Sufficient end-to-end test coverage
[ ] Docs updated or added to /docs
[ ] Commit messages sensible and descriptive

this looks good, but the manifests are generated from /deploy, so the manifests will need to change there in order to merge.

Thanks for the advice @ecordell !
I hope what I did is not too far from what you think. I tried to keep the annotation as close as possible from ocp and not leak it in upstream.
This script adds the annotations at the end of the generation. The main issue we that is we need to deserialize/serialize the YAML. It changes the indentation, etc. With yq v3, comments are preserved but indents changed in some places.
Let me know what you think!

/ok-to-test
/lgtm
/approve

Almost all green. How can I relaunch the failed github action?


Revert "Implement binary units"

Reverts raynigon/unit-api#553 because Binary Units are not part of the SI Units

Fixed in https://github.com/raynigon/unit-api/pull/560


FR: Force automatic search if job already exists in ARR

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
If a job already exists in in Sonarr/Radarr and a requests is made w/in Petio an automatic search is not triggered.
Describe the solution you'd like
A clear and concise description of what you want to happen.
If a job exists w/in Sonarr/Radarr and is monitored, if a user requests that movie/series Petio should trigger an automatic search forcing the files to be grabbed.

So I tested this yesterday and found out that when adding a series that was already in Sonarr but not in Plex, that Sonarr was actually searching for the missing series. So I think Petio is already doing this, but maybe others have had difference experiences?

I tried this with radarr. Didn't test in sonarr. This could be the case.

Can we test this in Radarr and I can implement a change if it's not searching

Just did a test on both sonarr and radarr and as unable to invoke a search from petio for something that was already in sonarr/radarr but not in plex.


route.segment as a way to get the specific path to the current +layout.*.js file

Describe the problem
Sometimes it would be useful to know "why" a certain load function  within a +layout.*.js file gets executed.
A specific use-case is: breadcrumb navigation, where each +layout.*.js file can contribute its own metadata (url, title, etc.) to an array. In this case each layout would need to know it's specific part of the url. Currently this requires manual parsing of the url.pathname. in each file
Describe the proposed solution
It would be great to have a way to get the information for which part of the url a specific layout file was called.
I think a new segment (or better name) property inside the route object for all load functions would make sense.
e.g. with this folder structure
 /products
	/[id]
		/edit
			/+layout.server.js
			/+page.js
		/+layout.js
	/+layout.server.js

when accessing the url /products/123/edit
following route.segment schould be defined in those files:

/products/+layout.server.js: /products
/products/[id]/+layout.js: /products/123
/products/[id]/edit/+layout.server.js: /products/123/edit

For all +page.*.js the segment would be equal to url.pathname.
Alternatives considered
Leave it as it is.
Importance
would make my life easier
Additional Information
No response

I've been looking for this very thing, I'll paraphrase the discord thread:
Can each +layout.js access their own route id?
I want to use it for some breadcrumb logic.
ie: assuming I am accessing the URL /users/david/edit I would do:
/users/+layout.js
  $breadcrumbs.add(theRouteIdForThisLayout, 'Users')
  ie:
  $breadcrumbs.add('/users', 'Users')

/users/[username]/+layout.js
  $breadcrumbs.add(theRouteIdForThisLayout, 'Users')
  ie:
  $breadcrumbs.add('/users/[username]', 'David King')

/users/[username]/edit/+layout.js
  $breadcrumbs.add(theRouteIdForThisLayout, 'Users')
  ie:
  $breadcrumbs.add('/users/[username]/edit, '✏ Edit')

This ^ logic would populate my breadcrumb data:
[
{ path: /users, title: 'Users' },
{ path: /users/david, title: 'David King' },
{ path: /users/david/edit, title: '✏ Edit' },
]
It feels similar to $types in that it's context aware / specific to the file.

A layout has no id, only pages have it. Something breadcrumb-y built in would help you with that better probably, something like "the current page has the following upper pages in its tree" - although that's probably not a solution for some cases since the folder tree and the perceived route hierarchy wouldn't necessarily correlate.

@dummdidumm that would be ideal. Having access to "parents" could be a great solution. Not sure how typing would work though.
Thinking out loud, a Breadcrumbs component may look like:
<script>
  import parents from '$app/stores';
</script>

{#each parents as parent}
  <a href={parent.url.pathname}>
    {parent.data.title}
  </a>
{/each}

As you can see, the title param is problematic as each parent can have different data structures.

A layout has no id, only pages have it.

I thought routes had ids? And they're made up of pages and layouts...

The route.id to the current layout/load function would be pretty nice. However I don´t think it´s very obvious how to implement this. Statically replacing a string like CURRENT_ROUTE_ID in files within routes could be possible...
With that route id it should be possible to extract the actual path slice using sveltekits router.

This feels like something that could be done in userland, no?
const _depth = 3;

export function load({ parent, url }) {
  const { breadcrumbs } = await parent();

  const breadcrumb = url.pathname.split('/').slice(0, _depth).join('/');

  return {
    breadcrumbs: [...breadcrumbs, breadcrumb]
  };
}

The one bit that's missing is knowing the depth without having to hard-code it. How important would that be? (I prefer depth to segments since it feels more general — could be used with route.id as well as url.pathname, etc.)

This feels like something that could be done in userland, no?

Almost everything can be done in userland ^^.
But like you ready mentioned, you need to hardcode the depth and that is not ideal for refactoring reasons. And you will end up with a line of code that get's repeated in every singe load function, where you need to manually parse the segment.
So if SvelteKit would provide that functionality, it would eliminate 2 lines of code (which have some potential to mess them up) per load function.
-const _depth = 3;

-export function load({ parent, url }) {
+export function load({ parent, route }) {
  const { breadcrumbs } = await parent();

-  const breadcrumb = url.pathname.split('/').slice(0, _depth).join('/');

  return {
-    breadcrumbs: [...breadcrumbs, breadcrumb]
+    breadcrumbs: [...breadcrumbs, route.segment]
  };
}

I just experimented a bit and additional to segment I would also wish that SvelteKit would provide a depth property on the route object. With the depth property it would be possible to keep the parallel data loading feature (no await parent()) intact and still be able to implement a breadcrumb array with the correct order.
https://stackblitz.com/edit/sveltejs-kit-template-default-gbdmhv?file=src/routes/a/+layout.server.ts
Already totally possible in userland, but requires additional effort to maintain and could cause some issues when refactoring or copy and pasting code from another route.

It doesn't feel like it belongs on route, to be honest — the route doesn't change between load functions. There is some precedent for event changing between load functions, because of the data property, so if it belongs anywhere it's there.
On reflection, 'depth' is a rather ambiguous concept though — what is its value inside src/routes/foo/[...bar]/+layout.js when you visit /foo/x/y/z?

is it 0, because there are no parent layouts?
is it 2, because there are two directories between it and src/routes? (Useful for manipulating route.id)
is it 4, because the path has four segments? (Useful for manipulating url.pathname)

If it's based on path segments, in +page.js is the depth affected by the value of trailingSlash?
I'm asking these questions less to figure out the correct answer, more to point out that it's inherently confusing to have such a thing in the framework.
The concept hitherto referred to as 'segment' is less confusing, though 'segment' is a bad name (that usually refers to a part of a pathname that doesn't include a / character). Not sure what a better one would be.

It doesn't feel like it belongs on route, to be honest — the route doesn't change between load functions.

Right, this would not rerun unless someone uses await parent(). Similar to #6315 something is needed that rereuns each time a +page load get's executed.

I'm asking these questions less to figure out the correct answer, more to point out that it's inherently confusing to have such a thing in the framework.

I see. Thanks for your detailed response. The depth concept certainly does not belong into the framework.


Toggle a display on and off

Description of the new feature / enhancement
An addon that will either tell windows that an HDMI output going to a receiver is audio only so it doesn't show as a display or an easy way to move all windows to a specific monitor.
Scenario when this would be used?
A lot of people use an AVR (Audio, Video Receiver) via HDMI to get the sound enhancements. I have always had issues as windows sees this as a 2nd (or 3rd in my case) display screen.  When the computer goes to sleep and I wake it up the screens flash for a good 15 seconds as they try to determine resolution, refresh rate, whatever. When the annoying display light show ends more often then not, some windows are placed on what windows thinks is that 3rd display but in actuality is my audio receiver.
This display dance is most likely a result of my 2 gaming monitors lack of a DP deep sleep toggle.
So a way to toggle a display on or off (so windows doesn't see it) without affecting the audio coming from that input so it is just an audio out.
Or a way to snap all windows to a specific monitor. The 1st option fix would be way better than the 2nd.  :)
Supporting information
No response

possible dup of https://github.com/microsoft/PowerToys/issues/1052

Yes and no. This is an audio only request. So a little different... :)

/feedbackhub


Laravel development

Hey there. Do you guys need help with getting this ready for Laravel?
it was mentioned in this video: https://youtu.be/qwz5XVtbfSY

Hello @cmosguy,
Thank you for reaching out!
We can definitely use help! Right now we are working on modernizing NeoEloquent, but it might be even simpler to have a basic package to integrate the client or driver as well.
What do you think?
Kind regards,
Ghlen

Closed due to inactivity


docs: changed commandhandler.md to warn that it is deprecated.

This PR changes the warning in commandhandler.md to state that it is deprecated and should not be used anymore.

[x] My pull request is made against the dev branch.
[x] I have ensured that the changed library can be built on your target system. I did not introduce any platform-specific code.
[x] I have ensured that all methods and functions are fully documented using doxygen style comments.
[x] I tested my commits, by adding a test case to the unit tests if needed
[x] I have ensured that I did not break any existing API calls.
[x] My code follows the coding style guide (if you are not sure, match the code style of existing files including indent style etc).
[x] I have not built my pull request using AI, a static analysis tool or similar without any human oversight. Where I have generated this pull request using a tool, I have justified why this is needed.

Closed PR. I've come to the agreement with CommandServer that the old notice was much more informational. This new notice does tell the user that it's deprecated, however, if we move to adding [[deprecated]] to the function then that can do the purpose of the notice.


How to test to reproduce the numbers in the paper?

@ckkelvinchan Thanks, which videos (i.e., test set) are used to reproduce the numbers?

All 50 videos in our VideoLQ dataset are used.

@ckkelvinchan Did you compute the metrics on the "00000000.png", "00000049.png", "00000099.png"?

I just checked. It should be 00000000.png, 00000050.png, 00000099.png

Could you tell me the frame numbers of the vid4 dataset used to calculate the metrics as the total frames in those videos is less than 100?

@ckkelvinchan  Thanks for the fast response. Just one more thing, I could not find the matlab function for PI metric. Could you direct me towards some resource to calculate the same

@ckkelvinchan Thanks for the fast response. Just one more thing, I could not find the matlab function for PI metric. Could you direct me towards some resource to calculate the same

If I remember correctly, it should be here.

Thanks it worked.... Do you by any chance have x2 model?

Thanks it worked.... Do you by any chance have x2 model?

Recently I don't have enough resources for the x2 model. We are working on that and will upload it when it is ready. Stay tuned!

When can I expect it? Any estimated date in mind?

When can I expect it? Any estimated date in mind?

I cannot be sure :( Maybe I can spend some time later to update the model so that it supports 2x.

why you only choose 3 frames for evaluation? why not testing on the whole video frames?

why you only choose 3 frames for evaluation? why not testing on the whole video frames?

Since computing NRQM is very time-consuming. We cannot afford calculating almost 5000 frames for 9 methods (= about 45000 frames).

Thanks for the response！

@ckkelvinchan Thanks for the fast response. Just one more thing, I could not find the matlab function for PI metric. Could you direct me towards some resource to calculate the same

If I remember correctly, it should be here.

When calculating PI, the GT images are needed, but there is not gt in VideoLQ. Could you tell me what did you use as the gt when calculating PI?

I found an interesting thing. I tested the bicubic images with brisque and got the same result as in the paper. But when I tested the same bicubic data with niqe, I got 7.9805, which is higher than 5.2762 in the paper. Maybe there is some problem with my niqe code. [http://live.ece.utexas.edu/research/quality/niqe_release.zip](niqe code)  I just find it in the directed niqe paper. Could you give me some information about how you calculate niqe?

@ckkelvinchan Thanks for the fast response. Just one more thing, I could not find the matlab function for PI metric. Could you direct me towards some resource to calculate the same

If I remember correctly, it should be here.

When calculating PI, the GT images are needed, but there is not gt in VideoLQ. Could you tell me what did you use as the gt when calculating PI?

PI is a weighted sum of NIQE and NRQM. I think no GT is needed.

I found an interesting thing. I tested the bicubic images with brisque and got the same result as in the paper. But when I tested the same bicubic data with niqe, I got 7.9805, which is higher than 5.2762 in the paper. Maybe there is some problem with my niqe code. [http://live.ece.utexas.edu/research/quality/niqe_release.zip](niqe code) I just find it in the directed niqe paper. Could you give me some information about how you calculate niqe?

Yes you are correct. In the original paper I computed the NIQE using the MATLAB built-in function, which is different from the one you are using.
I have updated the results using the official code on the Y-channel (which is the normal practice). I will release the updated version soon.

but the code you provided need the GT, which should be applied in "self_validation_HR" folder.

I found an interesting thing. I tested the bicubic images with brisque and got the same result as in the paper. But when I tested the same bicubic data with niqe, I got 7.9805, which is higher than 5.2762 in the paper. Maybe there is some problem with my niqe code. [http://live.ece.utexas.edu/research/quality/niqe_release.zip](niqe code) I just find it in the directed niqe paper. Could you give me some information about how you calculate niqe?

Yes you are correct. In the original paper I computed the NIQE using the MATLAB built-in function, which is different from the one you are using.
I have updated the results using the official code on the Y-channel (which is the normal practice). I will release the updated version soon.

Thanks!!

It is used to calculate RMSE. For NIQE, NRQM, and PI, we don't need GT.

hello~Could you give me some info about which function you used for video compression?

hello~Could you give me some info about which function you used for video compression?

Please refer to here for the details.


Reconcile newly created / deleted hosted clusters

Why
This PR addresses the following architectural changes

Main manager only responsible for init clf-controller/hlf-controller and hosted-cluster controller
Move all active hosted-cluster connections to hypershift-log-forwarder controller
Each hosted cluster watch via submanager with its own context
All hosted cluster submangers start via go routine
hosted-cluster controller watch create/delete of hosted clusters
When a new cluster is created, start a new sub-manager watching the hlf
When the cluster is deleted, cancel submanger context

/lgtm
/approve


SLExpandableTableView how to use in Swift 2.0?

Can you provide me an example shows how to use this in Swift 2.0?

Github is for reporting issues. Please refer to stack overflow for general usage questions.


dnf module install virt

Azure CLI                                                                                8.3 MB/s |  18 MB     00:02
Docker CE Stable - x86_64                                                                                107 kB/s | 7.8 kB     00:00
Extra Packages for Enterprise Linux Modular 8 - x86_64                                                                                361 kB/s | 537 kB     00:01
Extra Packages for Enterprise Linux 8 - x86_64                                                                                3.2 MB/s | 8.8 MB     00:02
nginx stable repo                                                                                136 kB/s |  20 kB     00:00
Oracle Linux 8 BaseOS Latest (x86_64)                                                                                73 MB/s |  29 MB     00:00
Oracle Linux 8 Application Stream (x86_64)                                                                                59 MB/s |  22 MB     00:00
PostgreSQL common RPMs for RHEL/CentOS 8 - x86_64                                                                                988 kB/s | 534 kB     00:00
PostgreSQL 13 for RHEL/CentOS 8 - x86_64                                                                                445 kB/s | 218 kB     00:00
PostgreSQL 12 for RHEL/CentOS 8 - x86_64                                                                                796 kB/s | 410 kB     00:00
PostgreSQL 11 for RHEL/CentOS 8 - x86_64                                                                                1.0 MB/s | 606 kB     00:00
PostgreSQL 10 for RHEL/CentOS 8 - x86_64                                                                                850 kB/s | 422 kB     00:00
PostgreSQL 9.6 for RHEL/CentOS 8 - x86_64                                                                                811 kB/s | 429 kB     00:00
PostgreSQL 9.5 for RHEL/CentOS 8 - x86_64                                                                                654 kB/s | 365 kB     00:00
Latest Unbreakable Enterprise Kernel Release 6 for Oracle Linux 8 (x86_64)                                                                                57 MB/s |  13 MB     00:00
Installing module 'virt:rhel' from Fail-Safe repository @modulefailsafe is not allowed
No default profiles for module virt:rhel. Available profiles: common
Error: Installing module from Fail-Safe repository is not allowed

It's dnf module install virt:ol in Oracle Linux. You can see a list of the modules with dnf module list.

It's dnf module install virt:ol in Oracle Linux. You can see a list of the modules with dnf module list.

dnf module disable virt:rhel

dnf module disable virt:rhel

This line Installing module 'virt:rhel' from Fail-Safe repository @modulefailsafe is not allowed implies that you have packages from the virt module installed, so you either have to remove them or switch the module stream so that you can continue to get updates for them.

This line Installing module 'virt:rhel' from Fail-Safe repository @modulefailsafe is not allowed implies that you have packages from the virt module installed, so you either have to remove them or switch the module stream so that you can continue to get updates for them.


Minified images

Using https://imagecompressor.com and https://github.com/shssoichiro/oxipng

Thanks!

@Motschen I now realized that I had renamed controller_buttons.png to controller_buttons-min.png, please fix that 😅


Update scenario-bgp-peering-hub.md

#sign-off

@prmitiki : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@prmitiki: I'm sorry - only the author of this article, @cherylmc, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

#sign-off


No resource found that matches the given name

Information

Apktool Version (apktool -version) -Apktool v2.2.4
Operating System (Mac, Linux, Windows) - osx 10.11.6
APK From? (Playstore, ROM, Other) -simple helloworld, see attached

Stacktrace/Logcat
Jons-Mac-Pro:test jcase$ apktool d hello.apk
I: Using Apktool 2.2.4 on hello.apk
I: Loading resource table...
I: Decoding AndroidManifest.xml with resources...
I: Loading resource table from file: /Users/jcase/Library/apktool/framework/1.apk
I: Regular manifest package...
I: Decoding file-resources...
I: Decoding values */* XMLs...
I: Baksmaling classes.dex...
I: Copying assets and libs...
I: Copying unknown files...
I: Copying original files...
Jons-Mac-Pro:test jcase$ apktool b hello
I: Using Apktool 2.2.4
I: Checking whether sources has changed...
I: Smaling smali folder into classes.dex...
I: Checking whether resources has changed...
I: Building resources...
W: /Users/jcase/android/working/Concealer/test/hello/res/values-v24/styles.xml:7: error: Error retrieving parent for item: No resource found that matches the given name '@android:style/Animation.OptionsPanel'.
W:
W: /Users/jcase/android/working/Concealer/test/hello/res/values-v24/styles.xml:8: error: Error retrieving parent for item: No resource found that matches the given name '@android:style/Animation.LockScreen'.
W:
Exception in thread "main" brut.androlib.AndrolibException: brut.androlib.AndrolibException: brut.common.BrutException: could not exec (exit code = 1): [/var/folders/j6/hczx8zx139s9cbc0s7nq8shc0000gn/T/brut_util_Jar_8698522024412882631.tmp, p, --forced-package-id, 127, --min-sdk-version, 19, --target-sdk-version, 25, --version-code, 1, --version-name, 1.0, --no-version-vectors, -F, /var/folders/j6/hczx8zx139s9cbc0s7nq8shc0000gn/T/APKTOOL7490291614340390359.tmp, -0, arsc, -0, arsc, -I, /Users/jcase/Library/apktool/framework/1.apk, -S, /Users/jcase/android/working/Concealer/test/hello/res, -M, /Users/jcase/android/working/Concealer/test/hello/AndroidManifest.xml]
	at brut.androlib.Androlib.buildResourcesFull(Androlib.java:496)
	at brut.androlib.Androlib.buildResources(Androlib.java:430)
	at brut.androlib.Androlib.build(Androlib.java:329)
	at brut.androlib.Androlib.build(Androlib.java:267)
	at brut.apktool.Main.cmdBuild(Main.java:230)
	at brut.apktool.Main.main(Main.java:83)
Caused by: brut.androlib.AndrolibException: brut.common.BrutException: could not exec (exit code = 1): [/var/folders/j6/hczx8zx139s9cbc0s7nq8shc0000gn/T/brut_util_Jar_8698522024412882631.tmp, p, --forced-package-id, 127, --min-sdk-version, 19, --target-sdk-version, 25, --version-code, 1, --version-name, 1.0, --no-version-vectors, -F, /var/folders/j6/hczx8zx139s9cbc0s7nq8shc0000gn/T/APKTOOL7490291614340390359.tmp, -0, arsc, -0, arsc, -I, /Users/jcase/Library/apktool/framework/1.apk, -S, /Users/jcase/android/working/Concealer/test/hello/res, -M, /Users/jcase/android/working/Concealer/test/hello/AndroidManifest.xml]
	at brut.androlib.res.AndrolibResources.aaptPackage(AndrolibResources.java:441)
	at brut.androlib.Androlib.buildResourcesFull(Androlib.java:482)
	... 5 more
Caused by: brut.common.BrutException: could not exec (exit code = 1): [/var/folders/j6/hczx8zx139s9cbc0s7nq8shc0000gn/T/brut_util_Jar_8698522024412882631.tmp, p, --forced-package-id, 127, --min-sdk-version, 19, --target-sdk-version, 25, --version-code, 1, --version-name, 1.0, --no-version-vectors, -F, /var/folders/j6/hczx8zx139s9cbc0s7nq8shc0000gn/T/APKTOOL7490291614340390359.tmp, -0, arsc, -0, arsc, -I, /Users/jcase/Library/apktool/framework/1.apk, -S, /Users/jcase/android/working/Concealer/test/hello/res, -M, /Users/jcase/android/working/Concealer/test/hello/AndroidManifest.xml]
	at brut.util.OS.exec(OS.java:95)
	at brut.androlib.res.AndrolibResources.aaptPackage(AndrolibResources.java:435)
	... 6 more
Jons-Mac-Pro:test jcase$

hello.zip
Steps to Reproduce

apktool d hello.apk
apktool b hello

APK
Attached
Questions to ask before submission

Have you tried apktool d, apktool b without changing anything? yes
If you are trying to install a modified apk, did you resign it? n/a
Are you using the latest apktool version? i think so, 2.2.4

➜  Bug1616 apktool d hello.apk
I: Using Apktool 2.2.4-98138b-SNAPSHOT on hello.apk
I: Loading resource table...
I: Decoding AndroidManifest.xml with resources...
I: Loading resource table from file: /Users/connortumbleson/Library/apktool/framework/1.apk
I: Regular manifest package...
I: Decoding file-resources...
I: Decoding values */* XMLs...
I: Baksmaling classes.dex...
I: Copying assets and libs...
I: Copying unknown files...
I: Copying original files...
➜  Bug1616 apktool b hello
I: Using Apktool 2.2.4-98138b-SNAPSHOT
I: Checking whether sources has changed...
I: Smaling smali folder into classes.dex...
I: Checking whether resources has changed...
I: Building resources...
I: Building apk file...
I: Copying unknown files/dir...
➜  Bug1616

Give apktool empty-framework-dir a shot. I don't have a reliable way to detect framework version installed yet.

Worked for me as expected FWIW;
rigby:Apktool diff$ java -jar ./brut.apktool/apktool-cli/build/libs/apktool-cli-all.jar d hello.zip
I: Using Apktool 2.2.5-d00e60-SNAPSHOT on hello.zip
I: Loading resource table...
I: Decoding AndroidManifest.xml with resources...
S: WARNING: Could not write to (/Users/diff/Library/apktool/framework), using /var/folders/29/g4wdlpzs17df5v8jbp88mnnc0000gn/T/ instead...
S: Please be aware this is a volatile directory and frameworks could go missing, please utilize --frame-path if the default storage directory is unavailable
I: Loading resource table from file: /var/folders/29/g4wdlpzs17df5v8jbp88mnnc0000gn/T/1.apk
I: Regular manifest package...
I: Decoding file-resources...
I: Decoding values */* XMLs...
I: Baksmaling classes.dex...
I: Copying assets and libs...
I: Copying unknown files...

I: Checking whether resources has changed...
I: Building apk file...


Add compiling condition for Windows

Wait, does it even compile for Windows?
I must confess that I never tried, but this is great news :)

Yeah! I compiled it successfully with rust-1.61.0-i686-pc-windows-msvc and MSVC19. And it works well.
I'm trying to reproduce the TCP locking/DoS ...