kf.coreaddons: no metadata found in ... Failed to extract plugin meta data from ...

Describe the bug
Since I upgraded my nixos unstable (to a4bf4434570), I get strange errors from time to time about kf.coreaddons. For instance, if I start Dolphin from the Konsole, I get:
$ dolphin
Qt: Session management error: Could not open network socket
kf.coreaddons: no metadata found in "/run/current-system/sw/lib/qt-5.15.2/plugins/kf5/kio/desktop.so" "Failed to extract plugin meta data from '/nix/store/8b9xb90nzyi79fv2h70amkql6xyqidh2-plasma-workspace-5.22.5/lib/qt-5.15.2/plugins/kf5/kio/desktop.so'"
kf.coreaddons: no metadata found in "/run/current-system/sw/lib/qt-5.15.2/plugins/kf5/kio/applications.so" "Failed to extract plugin meta data from '/nix/store/8b9xb90nzyi79fv2h70amkql6xyqidh2-plasma-workspace-5.22.5/lib/qt-5.15.2/plugins/kf5/kio/applications.so'"
kf.coreaddons: no metadata found in "/nix/store/q052xvyb8b6ib2mj2a55jfwzlzc2xqnz-plasma-workspace-5.21.5/lib/qt-5.15.2/plugins/kf5/kio/applications.so" "Failed to extract plugin meta data from '/nix/store/q052xvyb8b6ib2mj2a55jfwzlzc2xqnz-plasma-workspace-5.21.5/lib/qt-5.15.2/plugins/kf5/kio/applications.so'"
kf.coreaddons: no metadata found in "/nix/store/q052xvyb8b6ib2mj2a55jfwzlzc2xqnz-plasma-workspace-5.21.5/lib/qt-5.15.2/plugins/kf5/kio/desktop.so" "Failed to extract plugin meta data from '/nix/store/q052xvyb8b6ib2mj2a55jfwzlzc2xqnz-plasma-workspace-5.21.5/lib/qt-5.15.2/plugins/kf5/kio/desktop.so'"
kf.coreaddons: no metadata found in "/nix/store/58a3h83r65fb5l7ic34q9r25vbiiys78-kdelibs4support-5.81.0/lib/qt-5.15.2/plugins/kf5/kio/metainfo.so" "Failed to extract plugin meta data from '/nix/store/58a3h83r65fb5l7ic34q9r25vbiiys78-kdelibs4support-5.81.0/lib/qt-5.15.2/plugins/kf5/kio/metainfo.so'"
kf.coreaddons: no metadata found in "/nix/store/76khzgyz5459f6vs5hmlxxkf6rld0i1y-baloo-5.81.0-bin/lib/qt-5.15.2/plugins/kf5/kio/timeline.so" "Failed to extract plugin meta data from '/nix/store/76khzgyz5459f6vs5hmlxxkf6rld0i1y-baloo-5.81.0-bin/lib/qt-5.15.2/plugins/kf5/kio/timeline.so'"
kf.coreaddons: no metadata found in "/nix/store/76khzgyz5459f6vs5hmlxxkf6rld0i1y-baloo-5.81.0-bin/lib/qt-5.15.2/plugins/kf5/kio/tags.so" "Failed to extract plugin meta data from '/nix/store/76khzgyz5459f6vs5hmlxxkf6rld0i1y-baloo-5.81.0-bin/lib/qt-5.15.2/plugins/kf5/kio/tags.so'"
kf.coreaddons: no metadata found in "/nix/store/76khzgyz5459f6vs5hmlxxkf6rld0i1y-baloo-5.81.0-bin/lib/qt-5.15.2/plugins/kf5/kio/baloosearch.so" "Failed to extract plugin meta data from '/nix/store/76khzgyz5459f6vs5hmlxxkf6rld0i1y-baloo-5.81.0-bin/lib/qt-5.15.2/plugins/kf5/kio/baloosearch.so'"
kf.coreaddons: no metadata found in "/run/current-system/sw/lib/qt-5.15.2/plugins/kf5/kio/desktop.so" "Failed to extract plugin meta data from '/nix/store/8b9xb90nzyi79fv2h70amkql6xyqidh2-plasma-workspace-5.22.5/lib/qt-5.15.2/plugins/kf5/kio/desktop.so'"
kf.coreaddons: no metadata found in "/run/current-system/sw/lib/qt-5.15.2/plugins/kf5/kio/applications.so" "Failed to extract plugin meta data from '/nix/store/8b9xb90nzyi79fv2h70amkql6xyqidh2-plasma-workspace-5.22.5/lib/qt-5.15.2/plugins/kf5/kio/applications.so'"
[...]
kf.coreaddons: no metadata found in "/nix/store/76khzgyz5459f6vs5hmlxxkf6rld0i1y-baloo-5.81.0-bin/lib/qt-5.15.2/plugins/kf5/kio/tags.so" "Failed to extract plugin meta data from '/nix/store/76khzgyz5459f6vs5hmlxxkf6rld0i1y-baloo-5.81.0-bin/lib/qt-5.15.2/plugins/kf5/kio/tags.so'"
kf.coreaddons: no metadata found in "/nix/store/76khzgyz5459f6vs5hmlxxkf6rld0i1y-baloo-5.81.0-bin/lib/qt-5.15.2/plugins/kf5/kio/baloosearch.so" "Failed to extract plugin meta data from '/nix/store/76khzgyz5459f6vs5hmlxxkf6rld0i1y-baloo-5.81.0-bin/lib/qt-5.15.2/plugins/kf5/kio/baloosearch.so'"
inotify_add_watch(/tmp/nix-build-bespokesynth-1.0.0-4a74e80.drv-0) failed: (Permission denied)
inotify_add_watch(/tmp/systemd-private-c222aaf4bf5249aaad0eb6bb05709729-cups.service-9sDrcT) failed: (Permission denied)
inotify_add_watch(/tmp/systemd-private-c222aaf4bf5249aaad0eb6bb05709729-ModemManager.service-UtkAeg) failed: (Permission denied)
inotify_add_watch(/tmp/systemd-private-c222aaf4bf5249aaad0eb6bb05709729-nscd.service-99uqoK) failed: (Permission denied)
inotify_add_watch(/tmp/systemd-private-c222aaf4bf5249aaad0eb6bb05709729-systemd-logind.service-up0h0w) failed: (Permission denied)
inotify_add_watch(/tmp/systemd-private-c222aaf4bf5249aaad0eb6bb05709729-systemd-timesyncd.service-sjnSQY) failed: (Permission denied)
inotify_add_watch(/tmp/systemd-private-c222aaf4bf5249aaad0eb6bb05709729-upower.service-BpRfSz) failed: (Permission denied)
inotify_add_watch(/tmp/systemd-private-c222aaf4bf5249aaad0eb6bb05709729-vboxnet0.service-AIBo8f) failed: (Permission denied)

I can't see any special misbehavior after (Konsole seems to work fine), but I may not have tested all functionalities.
Steps To Reproduce
Steps to reproduce the behavior:

From KDE Plasma 5, start Dolphin in Konsole.

Notify maintainers
@NixOS/qt-kde

Is this still the case? I'm not able to reproduce.

I'll try to upgrade my system to check when I have some time (and I wanted to wait for blender 3 to be merged first). But if you can't reproduce it is maybe solved.

Please re-open if this is still an issue.


How to programmatically shut down the app?

Thanks @NeverMorewd for this handy library! It seems to work well so far and allowed me to port my .NET MAUI application MauiProgram.cs almost verbatim to Program.cs.
I do have one question: How do I shut down the application programmatically? My app needs to shut down after launching the updater. Looking at the source, I see that an AvaloniauiApplicationLifetime` is added to the service pipeline, so I injected that into the viewmodel that controls the update procedure.
I have not tested yet if this works but I did notice that doing this causes the app to no longer shut down when the main window is closed. Perhaps I'll have to shut it down manually?
I suppose what I am really looking for is how does this library play with the application lifetime as described in the official Avalonia docs?

Similar to WPF, Avalonia's IClassicDesktopStyleApplicationLifetime provides the ShutDownMode property and the ShutDown() method.
You can call ShutDown() directly to terminate the application. Alternatively, you can set the ShutDownMode property to one of the following values, depending on your use case:

OnLastWindowClose
OnMainWindowClose
OnExplicitShutdown

For Hosting.Avaloniaui, the application lifetime is accessed via AppBuilder.SetupWithLifetime(). You can find an example here:
Hosting.Avaloniaui - ApplicationLifetime Setup
On the other hand, Avalonia provides an alternative approach for managing application lifetime:
    [STAThread]
    public static void Main(string[] args)
    {
        BuildAvaloniaApp().Start(AppMain, args);
    }

    private static void AppMain(Application app, string[] args)
    {
        app.Run(new MainWindow());
    }

    // Avalonia configuration (don't remove; also used by the visual designer)
    public static AppBuilder BuildAvaloniaApp()
        => AppBuilder.Configure<App>()
            .UsePlatformDetect()
            .LogToTrace();

Thank you @NeverMorewd, I'm finally getting back to this issue. It would be nice if the IApplicationLifetime instance that is set up in RunAvaloniauiApplicationCore could be registered with the service collection so that it could be injected into view models later.
For now, to get me going, I'll just handle this with a static property on the App class.


cleanup

You shouldn't use the 'g' namespace in a taglib - it's better to use a custom value to make it clear that the taglib isn't from Grails.  How about 's' ?

Thanks Burt! Good suggestion, I'll change the namespace.


async start

I've spend a few hours on this, and hoping there's an easy answer that I'm just missing somehow. Is there a way to wait to start listening for webhook events until some async things have happened, such as getting secrets from paramstore and setting them as envs?

I think I finally got it
module.exports.probot = (event, context) => new Promise(async (resolve, reject) => {
    try {
      const result = await setEnvs();
    } catch (e) {
      console.error("Unable to set required environment variables");
      return reject(e);
    }
    const handler = serverless(appFn);
    return resolve(handler(event, context));
  }
);

Closing, but suggestions still welcome


Deployment of Ktrain based Albert model

Hi ,
My environment details:
Centos 7.5
Python 3.7
Tensorflow 2.1
Ktrain 0.14.5
I have built a classifier (in training instance) using ALBERT transformer. (The model directory contains three files viz  json,tf_model.h5 and .preproc files)
Im able to do the prediction as follows. ( I get class name as result )
myModel = ktrain.load_predictor('model/')
question = sys.argv[1]
myAnswer =  myModel.predict(question)
I have deployed the model  to QA instance now. I run the same prediction script and I get the following error
Traceback (most recent call last):
File "albertPredict.py", line 7, in
myModel = ktrain.load_predictor('model/')
File "/usr/local/lib/python3.7/site-packages/ktrain/core.py", line 1328, in load_predictor
preproc = pickle.load(f)
File "/usr/local/lib/python3.7/site-packages/transformers/tokenization_albert.py", line 185, in setstate
self.sp_model.Load(self.vocab_file)
File "/usr/local/lib/python3.7/site-packages/sentencepiece.py", line 118, in Load
return _sentencepiece.SentencePieceProcessor_Load(self, filename)
OSError: Not found: "/home/xxxx/.cache/torch/transformers/dd1588b85b6fdce1320e224d29ad062e97588e17326b9d05a0b29ee84b8f5f93.c81d4deb77aec08ce575b7a39a989a79dd54f321bfb82c2b54dd35f52f8182cf": No such file or directory Error #2
Please let me know whether any prereq file needs to be available in QA instance. Apart from Python3.7, Ktrain, Tensorflow I do not have anything in this instance. What is it looking for in Cache dir ? Any vocab file for creating embeddings ???
Thanks
Sridhar

If you're using a transformers model and loading a Predictor instance on a new machine, the vocab files need to be present in the cache on the new machine, which seems to be missing in your case.  Typically, the transformers library will download it to cache of new machine if it doesn't exist.  However, the error is saying that the it can't file one of the cached files required for tokenization on deployment instance.
Not sure why transformers didn't download the required files.   But, the solution is to copy the cache folder (i.e., <home_directory?/.cache/torch/transformers to the QA instance).  Specifically, this file is important for Albert:
dd1588b85b6fdce1320e224d29ad062e97588e17326b9d05a0b29ee84b8f5f93.c81d4deb77aec08ce575b7a39a989a79dd54f321bfb82c2b54dd35f52f8182cf

If you open that file with vim or nano, you'll probably see that it's the vocab file for ALBERT.
If the deployment instance is being used to train/re-train a model, and doesn't have internet connection or is behind a firewall, then there are additional considerations


o1VM: renaming trace related structures

SingleInstructionTrace into Trace
DecomposableTrace into DecomposedTrace
DecomposableMIPSTrace into DecomposedMIPSTrace
Keep DecomposableTracer for trait

Addresses https://github.com/o1-labs/proof-systems/pull/2208#pullrequestreview-2048133087

Fix conflicts. Will merge when CI is happy again.


curl | bash is bad

curl https://raw.githubusercontent.com/dirkpetersen/froster/main/install.sh | bash

Please no. Piping directly to bash is not okay, if network fails, you'll get a corrupted installation. In general, it is not a good practice to do this for many other reasons. I advice you to update it to:
curl -O https://raw.githubusercontent.com/dirkpetersen/froster/main/install.sh
bash install.sh

A warning that advices the user to read the script before running it would be even better!

Piping to bash has a bad reputation (ChatGPT agrees with you) but in practice it makes really no difference. If you get a corrupted install you can just run it again (so far, knock on wood, did not happen to me in 15 years of Github) and the download, review first then execute paradigm is nice in theory but < 1% of the users check this before they are running something and they can still do this by leaving out the pipe to bash thing.

closing


Fix lock_resolver to use BSD file locks.

This avoids spurious deadlock detection errors under Linux.
Fixes #1693

See #1693 for the blow by blow, but the original issue is now completely understood and this fix has the advantage over Joshua's stop-gap fix that we know it works for all Pants users without the specter of potential infinite retries looming over things.

LGTM. Super annoying about the seemingly historical bug (?).

We'll, historical, but kernel call APIs are reliable! They will generally never break user code. At least Linux won't (https://lkml.org/lkml/2012/12/23/75)! So there are newer Linux APIs Pex could use, but then I'm not sure what we'd do for OSX and others. It would be a new thing for Pex to go down the road of getting OS specific.
But, from my reading list in the associated issue, see: https://www.samba.org/samba/news/articles/low_point/tale_two_stds_os2.html
Very, very good reading.


[Mini RFC] Detect if a layer has loaded all of its assets

Background
deck.gl introduced async props support in v6. Currently, all layers have the ability to load some of its assets asynchronously, including data, image (BitmapLayer), iconMapping and iconAtlas (IconLayer), texture (SimpleMeshLayer), etc.
In most applications, we just want deck to rerender as soon as new assets become available. However, it may be also desirable to know when a layer has fully loaded its assets and performed a complete draw. In render tests, we should only compare screenshots when all layers are done loading. In kepler.gl, video frames should only be saved when the content is fully rendered.
Proposal
Add a public method to the base Layer class Layer.isLoaded().
The method should return true when:

All async props are resolved - this is the default implementation of the base Layer's method
All async requests that affect render (downloads, data processing, etc.) are resolved - this should be implemented by each layer if needed
All sublayers are loaded - this is the default implementation of the base CompositeLayer's method

Note that isLoaded() would still return true even if some assets fail to load. It merely indicates that all async operations are done -- not necessarily successfully -- and the layer has come to an "idle" state util future prop updates.
In practice, an application can use the feature as follows:
Deck({
  ...
  layers,
  onAfterRender: () => {
    if (layers.every(layer => layer.isLoaded())) {
      // capture screenshot
    }
  }
})

Alternative names

isIdle
isAwaitingAsset

To Do List

[x] Add label and assign to milestone
[ ] Coding
[ ] Doc update
[ ] What’s new update
[ ] Test

@ibgreen @chrisgervang @heshan0131

This is great start and much needed for a use case we are working on.

I was thinking it would be nice to have a callback, not just a query, e.g. for test cases etc.

it would be neat if this "rolls up" to a method and callback on Deck

Deck.isLoaded(),

Deck.onLoaded(true/false)

Especially in TileLayer and Tile3DLayer, we have layers that can be loaded and then no longer loaded when view props change, so being able to track loading status before each frame is important for e.g. video rendering, leading to a flow like:

deck.setProps({layers: new Tile3DLayer(...)});

// `deck.onLoaded` called with `false`
// ... loading tiles
// `deck.onLoaded` called with `true`

deck.setProps({ViewState})

// `deck.onLoaded` called with `false`
// ... loading missing tiles
// `deck.onLoaded` called with `true`

Why wouldn't your Tile3DLayer use case be covered by my code sample in the proposal? You already know that a reload is required when you initiate setProps. Even if you get a callback onLoaded(true), you still need to listen to onAfterRender` to be able to capture a video frame.

That is true, it could all be done with the simpler mechanism you propose so this is more an API design suggestion.
My comments and example are based on a sense that:

Loading status is a core concept that deserves its own top-level API, bringing to the user's attention that we support this important and non-trivial feature.

There are other things than video frames that could be updated when load status changes, such as UI elements etc indicating that things are still loading.

Allowing apps listen for such changes directly in a callback with a self-documenting name rather than adding a layer loop in onAfterRender would be clearer in my opinion.

Unlike onDataLoad, which exposes the loaded data array to the user, the “assets loaded” event does not have any tangible results. When the assets are loaded, the layer is simply marked “needs redraw.” Any texture/state that has been updated is internal to the layer and subject to change without documentation. So what exactly would be accomplished by listening to this event? Is your app manually tracking the loaded states of all the layers?

When the assets are loaded, the layer is simply marked “needs redraw.”

Good point, a bit more detail around how the loading tracking should be done would probably be good thing to add to the RFC.
I.e. how would it look for Layer authors? Can we do it with no impact?
I get the sense that you have an idea around a minimal system, and I am probably imagining something a bit more extensive that is aware of when things load.

For the use cases section in your RFC, these are the 3 use cases currently in my mind around "rendering individual frames":

testing
video animation
using deck as an image/texture rendering engine for use e.g. with other rendering engines (will share some code soon around this)

So what exactly would be accomplished by listening to this event?

A loading spinner

deck.setProps({
  onLoaded: isLoaded => dispatch(showSpinner(!isLoaded))
});

rendering individual frames

While this can certainly be done by animation loop callbacks, I feel using callbacks on top of the animation loop does not feel natural when rendering individual frames, and would prefer to see a simpler, more "linear" API for these cases:
  deck.setProps({layers, viewState, renderLoop: false});
  await deck.loaded(); // TBD pseudo-code, we could ideate on how to get a promise here.
  await deck.render();

Also in the animation loop case (unless extra logic was added), deck would potentially render lots of wasted frames while deck was loading, whereas here we would explicitly only render when we actually want the frame.

Looks to me that your last two example use cases can be solved if setProps returns a promise.
I think this is what we can agree:

Regardless of what API we add to the Deck class, the loaded check needs to be supported on the layer level.
For a layer, this is a passive check, i.e. no callbacks will be exposed.

I prefer to keep this proposal small and actionable. Discussion regarding the Deck API can be a separate thread.

Looks to me that your last two example use cases can be solved if setProps returns a promise.

Yes they could, but is it intuitively/unambigously clear what a setProps promise should represent? Loaded? Rendered? Transitions complete?

Regardless of what API we add to the Deck class, the loaded check needs to be supported on the layer level.

Agreed. (An architectural change that might be desirable in the future is being able to define separate "deck.props.dataSources", so that e.g. multiple layers can share the same data. Of course, the layers loaded() methods could still refer to the dataSources, so that would still work with this API).

For a layer, this is a passive check, i.e. no callbacks will be exposed.

Understood. This requires busy-polling but that is in line with how dirty checks in general work at the moment.

I prefer to keep this proposal small and actionable. Discussion regarding the Deck API can be a separate thread.

That works for me. I am totally fine with the proposal, mainly wanted to engage/show interest/encouragement.
And in fairness, the current RFC mainly talked about the top-level API rather than the implementation, so I suppose that is what I ended up trying to provide my perspective on.

Maybe also need consider how to surface to users when some assets failed to load.
https://github.com/uber/deck.gl/issues/4149


various modifications

Hi, can you take a look at my 'features' branch on my fork. I generated the component page, link is on my master branch.
To summarize :

I was inspired by the google autocomplete system : the input is always focus when we navigate with the up/down arrows in the results list. With IronMenuBehavior, items are focused and input lose focus, so I used iron-selector and write the arrows handlers myself. Items are still selectable with tab key.
I've merged the 2 files because I don't think paper-typeahead-results will be used standalone.
I called the filtredItems "predictions" (fancy thing from Place Autocomplete API) I thought it was cool from the user side.
To increase API, the user can to control the result layer manually (select/display/hide), but as we don't want the user to display the result layer when the result array is empty, I set it private (_hideResults) and add methods tryDisplayPredictions(), hidePredictions(), selectPrediction(selectedIndex).
I used hidden$="[[_hideResults]]"  instead of dom-if, to use the $.id selector instead of $$('#id'), I'm sure about the performance on this one ?

from polydoc
Since it is generally much faster to hide and show elements rather than destroy and recreate them, conditional templates are only useful to save initial creation cost when the elements being stamped are relatively heavyweight and the conditional may rarely (or never) be true in given usages. Otherwise, liberal use of conditional templates can actually add significant runtime performance overhead.

Possible improvements inspired from google autocomplete system:

An attribute as "live-input-update" could do the paper-input's value to update each time the 'iron-selected' changes, but still remembering the initial user input.
hover an item could directly select it (so the keyboard navigation continues at this point) but if something like "live-input-reload" was in place, the paper-input's value should not update.

Sorry for the long speech ;)

hey @Zecat I just took a look, this seems great! would you want to open up a PR? :tada:
awesome work here, i have made a ton of similar modifications locally I just have not synced everything yet


Typegen - predicate typegen to support configurable constants

Now that predicates support configurable constants in #998, we should look to support them in typegen.
Predicates with configurable constants are instantiated like so:
https://github.com/FuelLabs/fuels-ts/blob/28c1cda969c2cf97519d9289798c18843dc3364b/packages/fuel-gauge/src/predicate-with-configurable.test.ts#L77
However the autogenerated predicate factory created using typegen doesn't offer any way of passing configurables:
https://github.com/FuelLabs/fuels-ts/blob/28c1cda969c2cf97519d9289798c18843dc3364b/packages/abi-typegen/src/templates/predicate/factory.hbs#L49-L62
We should be able to infer from the ABI that the predicate contains configurable constants and the types of them, and provide a way for them to be easily passed to the factory instance. This could be done by amending the createInstance signature should configurables be present.

@FuelLabs/sdk-ts I'd be quite keen to pick this one up over the next week or so unless anyone has any objections?

Fantastic, this is the issue I said I'd pick up yesterday on the sync, but you're more than welcome to do it yourself! ⚡
Predicates, Scripts, and Contracts will be practically the same. Since the code sharing between them should be prevalent, the three might fit in a single PR. Do you want to do all three?
I suspect we will end up with something like this:
export type {{capitalizedName}}Cofigurables {
  ...
}

static createInstance(provider?: Provider, configurable?: {{capitalizedName}}Cofigurables) {
  ...
}

@arboleya Ah apologies, I do remember you mentioning it now. If that's alright with you I'd really like to!

No worries, man. Thank you.
If you need anything, just let me know.


Take active_slot_coeff into account for sync progress calculation

Issue Number
#1197
Overview

[x] Added active_slot_coeff to SlotParameters
[x] Let remainingBlocks = active_slot_coeff * remainingSlots in calculation

Comments

⚠ Haven't managed to test this at all yet

bors r+


Update deps

Coverage remained the same at 97.796% when pulling 2e419b2782cbb8d3c8ca0ddf121ca0a6bb20baf6 on update-deps into 452b16fba34b43a50e65358897b96634d78a0135 on master.


Pass dtype to cumsum and cumprod

Currently the APIs keras.ops.cumsum and keras.ops.cumprod are calling numpy backend where in all int dtypes are being converted into int64 if specific dtype is not provided explicitly for numpy.cumsum and numpy.cumprod as by default numpy will be converting this unspecified int dtype to int64.
Hence proposing to infer dtype and the pass it explicitly to numpy backend.
#Fixes #18730

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

Comparison is base (d8a06ff) 78.49% compared to head (315fb7f) 64.35%.
Report is 46 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff             @@
##           master   #18734       +/-   ##
===========================================
- Coverage   78.49%   64.35%   -14.14%
===========================================
  Files         335      336        +1
  Lines       33096    33939      +843
  Branches     6488     6651      +163
===========================================
- Hits        25978    21842     -4136
- Misses       5546    10658     +5112
+ Partials     1572     1439      -133

Flag
Coverage Δ

keras
64.35% <0.00%> (-14.04%)
:arrow_down:

keras-jax
?

keras-numpy
?

keras-tensorflow
?

keras-torch
64.35% <0.00%> (-0.80%)
:arrow_down:

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

Files
Coverage Δ

keras/backend/numpy/numpy.py
0.00% <0.00%> (-99.24%)
:arrow_down:

... and 103 files with indirect coverage changes

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

Hi @SuryanarayanaY Can you please resolve conflicts? Thank you!

Hi @SuryanarayanaY Can you please resolve conflicts? Thank you!

Hi @gbaned
It seems like this PR has already been addressed by #18813

Hi @SuryanarayanaY Can you please take a look on above comment from @james77777778. Thank you!

Closing this PR, since the above linked PR addresses the concern for all the backends. Thanks!


Basket threshold not consistently enforced

oscar's is_quantity_allowed bases its answer on the basket's num_items before an update and the quantity in the proposed update. This is flawed because if you're updating the quantity for a line, it is double-counting the previous quantity for the line.
Steps to reproduce bug:
0) Set OSCAR_MAX_BASKET_QUANTITY_THRESHOLD in your settings to any number - say, 10.

add a single item to your cart.
update the quantity to be something less than the threshold, say 5
try to update the quantity to something more than threshold - current quantity - say, 6
Result:
Quantity: 6
Error message: "Due to technical limitations we are not able to ship more than 10 items in an order."

Relevant code:
    if basket_threshold:
        total_basket_quantity = self.num_items
        max_allowed = basket_threshold - total_basket_quantity
        if qty > max_allowed:
            return False, _(
                "Due to technical limitations we are not able "
                "to ship more than %(threshold)d items in one order.") \
                % {'threshold': basket_threshold}

in oscar/apps/basket/abstract_models.py

Fix:
In basket/models, change is_quantity_allowed to also take a line argument and use that to calculate the correct limit:
def is_quantity_allowed(self, qty, line):
"""
Test whether the passed quantity of items can be added to the basket
"""
# We enfore a max threshold to prevent a DOS attack via the offers
# system.
basket_threshold = settings.OSCAR_MAX_BASKET_QUANTITY_THRESHOLD
if basket_threshold:
total_basket_quantity = self.num_items
max_allowed = basket_threshold - total_basket_quantity + line.quantity
In basket/forms, change check_max_allowed_quantity to supply the line:
def check_max_allowed_quantity(self, qty):
is_allowed, reason = self.instance.basket.is_quantity_allowed(qty, self.instance)

You also need to update do_reorder in customer/views:
    for line in lines_to_add:
        is_quantity_allowed, reason = basket.is_quantity_allowed(
            line.quantity, line)
        if not is_quantity_allowed:
            messages.warning(self.request, reason)
            self.response = redirect('customer:order-list')
            return

Actually, the fix for do_reorder will have to be a little more complicated than that.

Cool, thanks. Can you supply a PR with an unittest?

Fixed by @sasha0


Can't find variable: EMPTY

I'm not sure if this issue should go into purescript-aff or this repo, but when I use purescript-aff#4.0.0 and use the makeVar function, the Control.Monad.Eff.AVar module is missing the EMPTY variable in the compiled JavaScript. I think these steps can reproduce the issue.  Here's a sample module...
module Test.Main where

import Prelude

import Control.Monad.Aff (message, runAff_)
import Control.Monad.Aff.AVar (makeEmptyVar, makeVar)
import Control.Monad.Eff.Console (log)
import Control.Monad.Eff.Exception (stack)
import Data.Either (Either(..))

main = runAff_ (case _ of
    Left err -> do
      log $ "ERROR: " <> message err
      log $ show (stack err)
    Right r -> log "OK"
  ) $ do
    a <- makeVar "s"
    pure unit

If you compile using pulp build --main Test.Main --to test.js, the resulting test.js has this compiled Control.Monad.Eff.AVar module.
(function(exports) {
  /* globals exports, setTimeout */
  /* jshint -W097 */
  "use strict";

  // ...

  function drainVar (util, avar) {

    // ...

    if (value === EMPTY && (p = takeHead(ps))) {
      avar.value = value = p.value;
    }
  }
  // ...
})(PS["Control.Monad.Eff.AVar"] = PS["Control.Monad.Eff.AVar"] || {});

However, if I substitute makeEmptyVar for makeVar, then I get this output...
(function(exports) {
  /* globals exports, setTimeout */
  /* jshint -W097 */

  "use strict";

  var EMPTY = {};
  // ...

I'm using compile 0.11.6.

I was hit by this too.  That's an issue with purs bundle --main Test.Main --module Test.Main.  The DCE performed removed the EMPTY variable.   (zephyr)[https://github.com/coot/zephry] does not suffer from this.

purs bundle is extremely brittle when it comes to DCE on FFI code 😞. It's designed to perform DCE on PureScript code which has very specific patterns, completely ignoring other common JS things (like switches). I had tested the bundler with Test.Main and everything worked, so I assumed it was OK, but clearly there are bad cases where DCE will remove the var even though it's in use. I fixed this for Aff by putting everything important in a big IIFE, which purs bundle seems to leave alone.

Can you try out the changes in #6 and confirm it fixes your problem?

The EMPTY is now preserved. With zephyr we'll stop having this kind of problems, since DCE in foreign modules can be done by javascript tooling, while DCE on corefn is much easier to do.

#6 fixed it for me. Thanks, Nate!


APE_SPD.1-2 requirements are not met for Threats defined in Section I.3

As per APE evaluation of the CL and responses from the Canadian Scheme, below observation require HIT's attention:
HDcPP Section I.3 does not define the following threats in terms of an asset (as defined in I.2) in the threat definition.
CEM v3.1r5 APE_SPD.1.2C -  "All threats shall be described in terms of a threat agent, an asset, and an adverse action."
T.TSF_FAILURE.
T.UNAUTHORIZED_UPDATE.
T.WEAK_CRYPTO.

Agree, not sure why the issue isn't able to be assigned to me....

Howdy folks,
Reworded the threat statements using what was in accordance with the assets defined in I.2.  We tried to stay with the conventions used for the other threat statements in I.3.  Throw darts
T.TSF_FAILURE
[Original] 	A malfunction of the TSF may cause loss of security if the TOE is permitted to operate.
[Suggested] A malfunction of the TSF may compromise the device security status if the TOE is permitted to operate.
Agent: malfunction of the TSF
Adverse action:  TOE is permitted to operate
Asset: TSF Data (Device security status)
T.UNAUTHORIZED_UPDATE
[Original] 	An attacker may cause the installation of unauthorized firmware/software on the TOE.
[Suggested] An attacker may install unauthorized firmware/software on the TOE to modify the Device security status.
Agent: Attacker
Adverse Action: Installation of unauthorized firmware/software
Asset : TSF Data (Device security status)
T.WEAK_CRYPTO
[Original]	An attacker may exploit poorly chosen cryptographic algorithms, random bit generators, ciphers or key sizes.
[Suggested]	An attacker may exploit poorly chosen cryptographic algorithms, random bit generators, ciphers or key sizes to access (read, modify, or delete) TSF and User data.
Agent: Attacker
Adverse action: exploit poorly chosen cryptographic algorithms, random bit generators, ciphers or key sizes
Asset: User and TSF data

Resolved in errata,
line 6115, 6118, 6124

This issue is addressed by the following TD:

HCD0002

The TD above is located at the following location:

https://github.com/HCD-iTC/HCD-iTC-Template/tree/Working/TD


Need way to override client config values per-resource

Scenario/Intent
To be clear, I think this is more of a feature request, not a bug.
I'm trying to override the API version for a specific resource, but still be able to use my OneView config variable with the credentials, etc. defined. I'd really like to define the API version at the resource level, because the data properties of that resource are tied to the API version. For example:
My ov_config.json:
{
  "ip": "<IP_ADDRESS>",
  "credentials": {
    "userName": "Administrator",
    "password": "secret123"
  },
  "api_version": 600
}

And my playbook (main.yml):
- name: Create network
  oneview_fc_network:
    config: "ov_config.json"
    # Override the api version for this resource (doesn't work):
    api_version: 1600
    state: present
    data:
      name: 'New FC Network'
      fabricType: 'DirectAttach'

It looks like the code currently does an if-else, so I understand why it's not currently possible. I think it would be great if this code actually built out the client configuration in a layered approach though, allowing the use case above.
Environment Details

Module Version: 5.6.1
Ansible Version: 2.9.10
OneView Appliance Version: 5.20.00
OneView Client API Version: Any
Python Version: Any
python-hpOneView SDK Version: Any
Platform: Any

Steps to Reproduce
# See the above code snippets for the contents/context
$ ansible-playbook main.yml

Expected Result
It executes the oneview_fc_network API calls using API version 1600.
Actual Result
The API version from the config file is used. The api_version attribute in the resource itself is ignored.
Current Workaround
I think it's possible to do this with some filters like the following, but it's ugly:
vars:
  ov_config: "{{ lookup('file', 'ov_config.json') | from_json }}"
tasks:
  - name: Create network
    oneview_fc_network:
      config: "{{ ov_config | combine({ 'api_version': 1600 }) }}"
      state: present
      data:
        ...

This will be taken up in future SDK release

Our oneview ansible module follows the below priority order.

module params
config file
environment variables

So to override client config values per resource, you have to provide the config details in the task module params so that it will override the client config values. Refer to below example to get better understanding.
I hope this comment will resolve your issue and hence closing the issue.
- name: Getting network set info
  oneview_network_set_facts:
    hostname: "{{ ov_hostname }}"
    username: "{{ ov_username }}"
    password: "{{ ov_password }}"
    api_version: "{{ ov_api_version }}"
    name: "{{ network_set_name }}"
  delegate_to: localhost
  register: network_set_info


New feature request: quadratic programming

Would it be possible to add quadratic programming as a new feature?
Most already-integrated solvers are capable of solving (SCIP, Gurobi, CPLEX, and CBC it seems as well)

It is being done, with the same API as the sos.  Not sure if it is usable
for now.
Le ven. 19 juil. 2019 à 03:18, Roger Lloret-Batlle<EMAIL_ADDRESS>a écrit :

Would it be possible to add quadratic programming as a new feature?
Most already-integrated solvers are capable of solving (SCIP, Gurobi,
CPLEX, and CBC it seems as well)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/google/or-tools/issues/1434?email_source=notifications&email_token=ACUPL3M6F2VIAVRZ2CAJEG3QAGIFVA5CNFSM4IFESAL2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HAHOTKQ,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACUPL3LHYB6ZNSO3EL3N2PTQAGIFVANCNFSM4IFESALQ
.

Thank you. I could not find an open feature on this.

Will it work on CPLEX?
When will the next major version be released? (i.e. 8.0?)

Cplex, most likely not without user contributions.
With the next version (7.4), we will have quadratic support for SCIP and
Gurobi (I believe) using a proto 2 input. Because it is a proto2, C# will
not be supported.
With 8.0, I intend to replace the proto2 by a proto3, breaking
compatibility. But we have not yet started implementation, so do not hold
your breath.
Laurent Perron | Operations Research |<EMAIL_ADDRESS>| (33) 1 42 68 53
00
Le lun. 9 sept. 2019 à 16:48, Roger Lloret-Batlle<EMAIL_ADDRESS>a écrit :

Will it work on CPLEX?
When will the next major version be released? (i.e. 8.0?)
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/google/or-tools/issues/1434?email_source=notifications&email_token=ACUPL3KHRTCDRTNTXCI5723QIZO3BA5CNFSM4IFESAL2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD6H3MWI#issuecomment-529512025,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACUPL3MP3TOBIRAHJLR26H3QIZO3BANCNFSM4IFESALQ
.

Now that 8.0 has been released, and SCIP comes with the binary, can quadratic programming be considered?

It is available if you use the protobuf API for Gurobi and SCIP.
It is still proto2, so no .NET though.
Laurent Perron | Operations Research |<EMAIL_ADDRESS>| (33) 1 42 68 53
00
Le lun. 19 oct. 2020 à 16:03, Roger Lloret-Batlle<EMAIL_ADDRESS>a écrit :

Now that 8.0 has been released, and SCIP comes with the binary, can
quadratic programming be considered?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/google/or-tools/issues/1434#issuecomment-712179125,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACUPL3J45WFGFKRQZ255RCTSLRBMLANCNFSM4IFESALQ
.

Is there any example or documentation talk about the proto API?

Any news on QP with proto3?

None :-(
Laurent Perron | Operations Research | @.*** | (33) 1 42 68 53
00
Le lun. 17 janv. 2022 à 10:32, Roger Lloret-Batlle @.***>
a écrit :

Any news on QP with proto3?
—
Reply to this email directly, view it on GitHub
https://github.com/google/or-tools/issues/1434#issuecomment-1014316198,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACUPL3MQKDZXFUXSWWU4JJTUWPO2XANCNFSM4IFESALQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were assigned.Message ID:
@.***>

Hi everyone,
is there any progress on support for quadratic programming?
Particularly for the support of cplex?
As poying mentioned, it is quite hard to find any documentation.
The only example i found is this java example.
Thanks in advance!

A few answers:

we do not support/touch/modify cplex interface. You can send a PR to
support it.
genuine quadratic support is implemented in math_opt. Currently it is
C++ bazel only. We will soo add other languages and cmake.

Laurent Perron | Operations Research | @.*** | (33) 1 42 68 53
00
Le mar. 18 juil. 2023 à 23:22, JannikDroegemueller @.***>
a écrit :

Hi everyone,
is there any progress on support for quadratic programming?
Particularly for the support of cplex?
As poying mentioned, it is quite hard to find any documentation.
The only example i found is this java example
https://github.com/ogunheper/google-or-tools-sample/blob/55bc62064db87e33194ce3cd200a0f3a5c15b1e9/src/test/java/com/ortools/OrToolsDemoApplicationTest.java#L237
.
Thanks in advance!
—
Reply to this email directly, view it on GitHub
https://github.com/google/or-tools/issues/1434#issuecomment-1641487099,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACUPL3OWDOVQKHDXZ45G63DXQ54KPANCNFSM4IFESALQ
.
You are receiving this because you were assigned.Message ID:
@.***>


Godot can't find wine (Flatpak)

Godot version:
3.3 (Flatpak version)
OS/device including version:
Ubuntu 21.04
Issue description:
Godot can't find wine for the rcedit command (Windows export). I tried to leave the wine option in the editor settings blank, I tried "wine", "/usr/bin/wine", ... When I click on browse I can't find it either, but it works if I run it in the terminal. It works in the normal download version of Godot but not in the Flatpak version so I think this is a permission thing.
Steps to reproduce:

Install Godot via Flatpak
Install wine
Try to find and use it in an export

That's a normal limitation of flatpaks, they're sandboxed so they don't give you access to other system binaries.

Ok, I understand. Would be helpful if the whole command would be printed in the output console to run the command by hand in the terminal as a solution.

It should be possible to implement support for rcedit with the Flatpak as long as WINE is installed on the host: https://github.com/flathub/org.godotengine.Godot/pull/38

You shouldn't have to use host wine anymore, Godot should be able to use org.winehq.Wine.
Unfortunately the way it's currently distributed is as a base app, but Godot can choose to distribute its own Wine as an extension (similar to how I did in the Fightcade Flatpak: com.fightcade.Fightcade.Wine


Can't define url with embedded or command for manifest schema

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest version
[X] Search the existing issues.

Steps to reproduce
While updating the resource manifest for Microsoft/OSInfo in #168, I had to remove the schema.url property from the manifest for tests to pass - my understanding was that the schema url was strongly suggested for resources that define schema.command, but still usable for schema.embedded.
Double-checking the type definition:
https://github.com/PowerShell/DSC/blob/5a1f8b582edfbe8a040195baa7ff4bdb7231addd/dsc_lib/src/dscresources/resource_manifest.rs#L41-L43
https://github.com/PowerShell/DSC/blob/5a1f8b582edfbe8a040195baa7ff4bdb7231addd/dsc_lib/src/dscresources/resource_manifest.rs#L56-L67
It looks like you have to choose between the three options. You can define the value in the manifest as embedded or command or url, but not embedded and url or command and url, which contradicts prior conversations about using url and command together.
Expected behavior
The following manifest snippets are valid:
{
    "schema": {
        "embedded": {
            // schema definition
        }
    }
}

```jsonc
{
    "schema": {
        "command": {
            // schema command definition
        }
    }
}

{
    "schema": {
        "url": "https://schemas.tailspintoys.com/tstoy.json"
    }
}

{
    "schema": {
        "embedded": {
            // schema definition
        },
        "url": "https://schemas.tailspintoys.com/tstoy.json"
    }
}

{
    "schema": {
        "command": {
            // schema command definition
        },
        "url": "https://schemas.tailspintoys.com/tstoy.json"
    }
}

Actual behavior
The following snippets are valid:
{
    "schema": {
        "embedded": {
            // schema definition
        }
    }
}

{
    "schema": {
        "command": {
            // schema command definition
        }
    }
}

{
    "schema": {
        "url": "https://schemas.tailspintoys.com/tstoy.json"
    }
}

The following snippets are invalid:
{
    "schema": {
        "embedded": {
            // schema definition
        },
        "url": "https://schemas.tailspintoys.com/tstoy.json"
    }
}

{
    "schema": {
        "command": {
            // schema command definition
        },
        "url": "https://schemas.tailspintoys.com/tstoy.json"
    }
}

Error details
No response
Environment data
Name                           Value
----                           -----
PSVersion                      7.3.6
PSEdition                      Core
GitCommitId                    7.3.6
OS                             Microsoft Windows 10.0.22621
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Version
Latest build from main
Visuals
No response

Consider moving url out to a higher level that is required as that was intended to be used by configuration authoring tools where the user probably has internet access while command and embedded are for deployment scenarios where internet is not always available.

I think we can get away with requiring the resource instance schema defined in schema.embeddded or returned by schema.command to define the $id keyword pointing to the canonical URI for the schema.
For example:
https://github.com/PowerShell/DSC/blob/5a1f8b582edfbe8a040195baa7ff4bdb7231addd/schemas/2023/08/config/document.json#L2-L3
This reuses the semantics of JSON Schema instead of defining a new extra set of information.
Alternatively, if we are adding it as an annotation to the manifest, we could use a key like instanceSchemaID (or whatever) at the top-level of the manifest[^1].
[^1]: Worth noting, I think we should always recommend people to author their manifests with the schema embedded - this makes integration and usage much easier overall and makes it so folks don't need to publish their schemas separately. So in that model, schema.command is supported for development purposes, but schema.embedded is preferred. If you want to reference an external schema from embedded, you can do something like:
  ```jsonc
  {
    // rest of the manifest
    "schema": {
      "embedded": {
        "$ref": "https://schemas.tailspin.com/dsc/tstoy.json"
      }
    }
  }
  ```

Url was removed, so this issue is moot


Lime test not working

Describe your problem here.
I tried using source code for the first time, i followed every step and verified everything from the page, but when i try running the "lime test windows" in the command prompt it says that lime is not recognized.
Are you modding a build from source or with Lua?
Source
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
No response

haxelib run lime setup -alias -y after ur command

haxelib run lime setup -alias -y after ur command

that actually worked but now when i run it this message appears
C:/HaxeToolkit/haxe/lib/flixel-ui/2,5,0/flixel/addons/ui/FlxUICursor.hx:564: characters 9-11 : Warning : Potential typo detected (expected similar values are flixel.addons.ui.SortMethod.ID)
C:\Users\Mateus\Documents\Friday Night Funkin'\FNF-PsychEngine-main\export\release\windows\obj>setlocal enabledelayedexpansion
Warning: Could not find Visual Studio VsDevCmd
Missing HXCPP_VARS
Error: Could not automatically setup MSVC


Implementation for 'Audit' action in Authorization Policy [WIP]

This will update the pilot implementation of Authorization Policy to support the Audit action which was added to the authorization policy api. A third RBAC filter will be built with the Log action based on the configuration of authorization policies with the Audit action. See: Audit action RFC
This PR is WIP until a proxy update.
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[x] Policies and Telemetry
[x] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

@davidraskin: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
9fb137a5b171ba67e9ef0f4847cab1f9e3a2dcbe
link
/test release-notes_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

cc @liminw @yangminzhu

@davidraskin: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
6d544b27557b7300b9e0ee14d8c9818bb281ab4f
link
/test release-notes_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@davidraskin: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
6d544b27557b7300b9e0ee14d8c9818bb281ab4f
link
/test release-notes_istio

analyze-tests_istio
6d544b27557b7300b9e0ee14d8c9818bb281ab4f
link
/test analyze-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@davidraskin: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

analyze-tests_istio
9bb394233d4dd8c8adcb23d24a9e57d6d0198fa2
link
/test analyze-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@davidraskin: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-security-k8s-tests_istio
504f8dd7f7e77c7059d4490b082754949941a657
link
/test integ-security-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@davidraskin: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-telemetry-k8s-tests_istio
7e0a6bdb2a75c3225286acfd7ad898f43f45d698
link
/test integ-telemetry-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@davidraskin: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-telemetry-k8s-tests_istio
7e0a6bdb2a75c3225286acfd7ad898f43f45d698
link
/test integ-telemetry-k8s-tests_istio

integ-distroless-k8s-tests_istio
7e0a6bdb2a75c3225286acfd7ad898f43f45d698
link
/test integ-distroless-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

Thank you!

/cherrypick release-1.7

@xulingqing: #26439 failed to apply on top of branch "release-1.7":
Applying: trying again
Applying: support LOG action in Authz policy
.git/rebase-apply/patch:106: trailing whitespace.
time (mkdir -p /work/out/linux_amd64/docker_build/docker.pilot && cp -r pilot/docker/Dockerfile.pilot /work/out/linux_amd64/pilot-discovery tests/testdata/certs/cacert.pem /work/out/linux_amd64/docker_build/docker.pilot && cd /work/out/linux_amd64/docker_build/docker.pilot && chmod 755 pilot-discovery cacert.pem && docker build --build-arg BASE_VERSION=1.7-dev.0 --build-arg BASE_DISTRIBUTION=default -t docker.io/davidraskin/pilot:davidraskin -f Dockerfile.pilot . );
.git/rebase-apply/patch:107: trailing whitespace.
Sending build context to Docker daemon  557.1kB
Sending build context to Docker daemon  24.51MB
Sending build context to Docker daemon  48.46MB
Sending build context to Docker daemon  72.42MB
Sending build context to Docker daemon  95.81MB
Sending build context to Docker daemon  102.5MB

warning: 2 lines add whitespace errors.
Using index info to reconstruct a base tree...
M	bin/init.sh
M	go.mod
M	go.sum
M	pilot/pkg/model/authorization.go
M	pilot/pkg/model/authorization_test.go
M	pilot/pkg/security/authz/model/model_test.go
M	pkg/config/xds/filter_types.gen.go
M	tests/integration/security/authorization_test.go
Falling back to patching base and 3-way merge...
Auto-merging tests/integration/security/authorization_test.go
Auto-merging pkg/config/xds/filter_types.gen.go
CONFLICT (content): Merge conflict in pkg/config/xds/filter_types.gen.go
Auto-merging pilot/pkg/security/authz/model/model_test.go
Auto-merging pilot/pkg/model/authorization_test.go
Auto-merging pilot/pkg/model/authorization.go
Auto-merging go.sum
CONFLICT (content): Merge conflict in go.sum
Auto-merging go.mod
Auto-merging bin/init.sh
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0002 support LOG action in Authz policy
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

In response to this:

/cherrypick release-1.7

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@xulingqing: new issue created for failed cherrypick: #27050

In response to this:

/cherrypick release-1.7

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.


Improve grammar

Since some of the questions either didn't seem to make sense when
combined with the intensity rating or had wording which may not have had
clear meanings, improve the grammar of some of the questions and answers
so we can have more consistent survey data and user experiences.
Signed-off-by: Ryan Y<EMAIL_ADDRESS>

'Have you ridden public transit in the past three months'
'Would you ride public transit in the near future?'),

I think the adjustment changes the question. What if we changed ridden to used or something of the lack

The problem lies in that "how strongly do you feel about this" makes no sense for the original question.  Using used instead doesn't really help much either.
What else would work here?

Agree with you, the question probably needs to be reworked in general.

Okay.  Let me know tonight or tomorrow what to change it to if you have any suggestions and I'll commit again.

Be sure to check the G-form so we stay in Sync. G-form is "master". I'll
work on that queations
On Friday, April 1, 2016, Ryan Y<EMAIL_ADDRESS>wrote:

Okay. Let me know tonight or tomorrow what to change it to if you have any
suggestions and I'll commit again.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/Code4HR/okcandidate/pull/60#issuecomment-204600698

--
Kevin Curry
Virginia Beach, VA
https://comparityins.com

The challenge you are facing comes from changing the rating question to "How strongly do you feel about this?" The original was "How important is public transportation to you?" Public transportation can be important or not important regardless of whether the respondent uses it regularly or at all. I'm conflicted about switching from "How important" to "How strongly do you feel" but I'm pretty certain the rating question should restate the choice question instead of using the ambiguous "this"


python311Packages.boschshcpy: 0.2.79 -> 0.2.83

Diff: https://github.com/tschamm/boschshcpy/compare/0.2.79...0.2.83
Description of changes

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 273152 run on x86_64-linux 1

5 packages built:

home-assistant-component-tests.bosch_shc
python310Packages.boschshcpy
python310Packages.boschshcpy.dist
python311Packages.boschshcpy
python311Packages.boschshcpy.dist


[sdl2] alias targets break multiple find_package calls on CMake < 3.18

When calling find_package(SDL2) from multiple subdirectories on CMake versions below 3.18, the following snippet runs:
https://github.com/microsoft/vcpkg/blob/025e564979cc01d0fbc5c920aa8a36635efb01bb/ports/sdl2/0004-sdl2-alias-on-static-build.patch#L1-L17
The existence of this alias triggers a failure in SDL2Targets.cmake when it attempts to create SDL2::SDL2-static. This should be fixed by amending this patch to check early on in SDL2Config.cmake if the version is less than 3.18 and SDL2::SDL2 already exists. When this is the case, a FATAL_ERROR should result that explains the issue and encourages users to upgrade CMake.

Hi @alexreinking
Thanks for posting this issue.
@RT222
Could you please help confirm if the patch is needed here? Seems the changes in this patch cannot be merged to upstream.

Sorry about the late answer, I must have missed the notification. On dynamic linkage the sdl2 cmake file create the SDL2::SDL2 target, but on static linkage it create the SDL2::SDL2-static target instead. The problem is that libraries depending on the sdl2 (sdl2-ttf, sdl2-image, sdl2-net...) will only look for the former target, even on static linkage, and so will fail to build in this case. That's what this patch fixes.

so if I want to build SDL statically I have to use SDL2::SDL2-static? Which breaks on other platforms non vcpkg or is this upstream behaviour? If i want to build statically on vcpkg I use the -static triplet.

if I want to build SDL statically I have to use SDL2::SDL2-static?

@cjwijtmans The discussion is about using SDL2, not building. Some packages offer to build static and dynamic at the same time. That's why they offer different targets. In vcpkg, this property isn't used, and often packages are patched to offer a single target for uniform usage.

@cjwijtmans The discussion is about using SDL2,

You can't be serious. If I don't build my project with SDL2 how can I "use" it? Do not ever reply to me again.

@cjwijtmans The discussion is about using SDL2,

You can't be serious. If I don't build my project with SDL2 how can I "use" it? Do not ever reply to me again.

Slow, please. Building your project with SDL2 is using SDL2, in contrast to building SDL2 itself.
FTR this issue is about find_package breaking on multiple calls. It is not clear to me if you are actually suffering from this issue, or if your question is about another topic.


Create a plugin to check if hosts have the defined plugins installed inside the MAAS api

Do NOT include any confidential information, this issue will be created on a public repo
This section is completed by the issue reporter
On which version(s) do you encounter this issue?
Please provide the output to 'git describe --all' for  all deployments where you have confirmed it occurs.

What type of issue is this? Please tick the appropriate box.

[ ] bug - existing functionality that does not work for you as documented
[x] feature - new functionality that you would like to see added

What is the expected behavior?
Create a plugin to check if hosts have all defined plugins installed inside the MAAS api.
The initial check could currently rely on the configuration inside /etc/rackspace-monitoring-agent.conf.d and make sure that all plugins are installed.
Ideally the plugin would be able to determine what is configured inside /opt/rpc-openstack/rpcd/playbooks/roles/rpc_maas/defaults/main.yml and determine the necessary checks, in
What is actually happening?
Occasionally we see clouds where MAAS checks are missing due to maas api or other limits and we would not recognize if a check never installed in the first place.
What were you attempting to do when you encountered this error (give as much detail as possible)?
How can we reproduce this issue?

How can we verify your issue is addressed?
Check is rolled out for the rackspace-monitoring-agent but not installed inside the maas api
If you are using or know of a workaround please describe the necessary steps?

What is your evaluation on the importance of this issue, and why?
Prio 3

This section is completed during the issue resolution process. If you are reporting an issue, please do not modify the section below.
Which branches are impacted by this issue?
Remove branches that are not impacted.
For each pull request, update the branch line to reference it.

[ ] master
[ ] newton-14.0
[ ] mitaka-13.1
[ ] liberty-12.2

Upgrade impact. Consider the impact on moving between major, minor and patch releases.

[ ] This change does not impact upgrades
[ ] This change addresses a potential upgrade impact
[ ] The upgrade impact is being tracked by another issue.

The issue tracking the upgrade impact is:
Testing impact

[ ] An issue/pull-request has been create or exists to track the testing gap, and is referenced below
[ ] This change includes additional testing
[ ] This change is tested by the existing setup

Additional testing links:
Release note

[ ] This change includes a release note.
[ ] This change does not require a release note.

Doc Impact
Does your change require documentation?

[ ] Yes, and I submitted an issue to Docs issues
[ ] Yes, and I contributed a documentation patch to docs-rpc
[ ] No documentation required for this change

Additional documentation links (please add links if you have ticked yes):

Thanks @BjoernT for the feedback!
I think @b3rnard0 and @jamiramontez will have to take a look at this and tell us where it fits into our release roadmap. I'm marking this as type-enhancement-small. IMO, this request isn't something that is large enough for the intake process. However, a decision still needs to be made regarding what milestone this fits into.


Add river images

I can take this :)

Posted from our discord channel:
https://discord.com/channels/1202304549294190602/1202304549944316007/1211712518481911808
https://discord.com/channels/1202304549294190602/1202304549944316007/1211719825403936848

I have been trying to figure out how to juggle personal life things that have come up, but im finding myself losing more of my free time atm while I deal with these things :/

I will still be following the project and if I find myself having free time I will attempt to contribute to already made images


Sent tokens to exchange 20 minutes not showing

Hi greymass, sent tokens to exchange. Not showing after 20 minutes. Please advise.
https://eostracker.io/transactions/b17d7d08dc31a8834196b9dc8e1ecc184c7c13fc8e5c6f41cd3117392fe80882
Thanks...

From the looks of the transaction, it successfully left your wallet and went to kraken. If it's not showing there, contact their support. There's nothing we can do from our side.


plan to add workflow labels to the metric?

Summary
add workflow/workflowTemplate's label to the metric
Use Cases
When I have two workflowTemplates (A,B), I want to know the overall situation of workflow of A and B respectively. The current metric does not support me to do this. Since there is no label for workflow.
NOW:
argo_workflows_count{container="workflow-controller", endpoint="metrics",
instance="<IP_ADDRESS>:9090", job="workflow-controller-metrics", namespace="argo",
pod="workflow-controller-7d69c4b564-t6xn8",
service="workflow-controller-metrics", status="Error"}
BETTER:
argo_workflows_count{container="workflow-controller", endpoint="metrics",
instance="<IP_ADDRESS>:9090", job="workflow-controller-metrics", namespace="argo",
pod="workflow-controller-7d69c4b564-t6xn8",
service="workflow-controller-metrics", status="Error",templateName="test"}

Message from the maintainers:
Love this enhancement proposal? Give it a 👍. We prioritise the proposals with the most 👍.

Duplicate https://github.com/argoproj/argo-workflows/issues/9849


Feature multi-model

Extends the API so that:

A trained model can be selected for an annotation request (eg. HPO, MONDO, etc...)
Trained models can be listed (/lsmodels)
Trained models can be deleted (/delete_model)
A script can be generated for model training on HPF (/submit_training_job)

Modifications were made to the model training code so that it executes on HPF.

Thank you for your feedback. I have revised this PR so that it only includes the changes needed for using multiple models and does not include any model training or HPF code.


Investigate providing both WebAssembly- and vanilla JS-based decoders

When we replaced the asm.js image decoder with a WebAssembly-based one (#20), we gained performance and resilience† at the expense of increased bundle size and breaking compatibility. This was fine for our use case--but what if people want to use our library in a runtime that does not support WebAssembly (or asm.js for that matter)? What if bundle size is critical?
† asm.js is dead tech and there are development tools (e.g. esbuild) that do not support it. See https://github.com/webtoon/psd/pull/20#issuecomment-1153571437
In our internal experiments, a properly optimized vanilla JS decoder was ~15% slower in Chrome (V8), and ~50% slower in Firefox and Safari. Since Node.js uses V8, we may expect similar performance differences in Node.js. These differences may be acceptable to those who want a tiny bundle--or don't have the luxury of WebAssembly.
Perhaps we could provide two decoders (vanilla JS and WebAssembly) and let the user choose?
Brainstorming Strategies
Create separate bundles for vanilla JS and WebAssembly, possibly exposing them as separate endpoints:
import Psd from '@webtoon/psd'         // WebAssembly
import Psd from '@webtoon/psd/vanilla' // Vanilla JS

Create two parse() methods and export them:
import { parse, parseWithJs } from '@webtoon/psd'

const psd1 = parse(arrayBuffer)
psd1.composite() // WebAssembly

const psd2 = parseWithJs(arrayBuffer)
psd2.composite() // Vanilla JS

Create two composite() methods:
import { parse, composite, compositeWithJs } from '@webtoon/psd'

const psd = parse(arrayBuffer)
composite(psd)       // WebAssembly
compositeWithJs(psd) // Vanilla JS

We might be able to bundle asm.js code while still supporting esbuild (or Vite) by wrapping the asm.js code in an indirect eval (or a Function constructor). Code transformers usually leave string literals as-is.
It would make the code slightly larger (and complicate the build pipeline) but it could work.


fix: thubmnails loading - Talisman default config

SUMMARY

We have THUMBNAILS feature flag enabled and thumbnails are not loaded with the default security settings

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

@Khrol thanks for the PR. Would you mind adding testing instructions—preferably with the inclusion of unit/integration tests?

@john-bodley, I've updated manual testing instructions.
Meanwhile, this feature test requires the real browser and all components running. Looks like this is where e2e/cypress tests are used.
I don't see how to configure cypress tests to enable THUMBNAILS feature flag only for a specific test.


NPM build and publish

PR for publishing ESM module to NPM unders ./dist/nostr.js

Is this working? Will it break things?


Add Flask Conf 2018

https://2018.flask.python.org.br/

@rennerocha Thanks for the contribution!
If you want to enhance it you could add the talk slides as related_urls as a new pull request.


Add new specs

@JamesSmartCell The travis build fails due to insufficient memory heap allocation used in gradle properties.


docs: update tecent games images

Changes:

Redo and update 2 images in the English blog.
Screenshots of the change:

Preview: https://deploy-preview-951--apache-apisix.netlify.app/blog/2021/05/24/tencent-games/
Actual: http://localhost:3000/blog/2021/05/24/tencent-games


Make compatible with SpaCy 3.0+

I see that the SpaCy version is constrained to <3.0 in requirements.txt, but I'm wondering if that restriction can be lifted. At present I have to set up a service to deliver predictions from mordecai to another environment that requires SpaCy 3.0. Thank you!

I'm working on it! 🤓

Any updates? :smile:


I get this error when i try to open the addon panel

Message: Interface/AddOns/StatWeightsClassic/shared/ui.lua:630: attempt to call method 'Show' (a nil value)
Time: Mon Feb  5 14:25:20 2024
Count: 2
Stack: Interface/AddOns/StatWeightsClassic/shared/ui.lua:630: attempt to call method 'Show' (a nil value)
[string "=(tail call)"]: ?
[string "=[C]"]: in function `Show'
[string "@Interface/AddOns/StatWeightsClassic/shared/ui.lua"]:630: in function <Interface/AddOns/StatWeightsClassic/shared/ui.lua:626>
[string "@Interface/AddOns/StatWeightsClassic/shared/core.lua"]:367: in function <Interface/AddOns/StatWeightsClassic/shared/core.lua:366>
Locals: (*temporary) = "LeftButton"
(*temporary) = false
(*temporary) =  defined =[C]:-1

I think you might be trying to open it on a non-caster class which the addon doesn't implement.
Feel free to re-open if this is happening on a caster class.


Add type annotations to fform in more places

Can you benchmark this @wouterwln , in my benchmarks it shows up to 2x speed up in some cases, which looks suspicious.
It allocates slightly more in GraphPPL benchmarks, but benchmarks in RxInfer do not show any significant change in the performance nor in memory allocations.

For me it also shows 40% in some cases (memory is worse though in make node but not in model creation so should be fine), so looks fine to me


Margin doesn't work in slide number object

Failed to set margin of slideNumber block to zero.
Category
Bug
Versions
PptxGenJS version: 3.3.1
PowerPoint version: PowerPoint for Mac 16.40
Steps to Reproduce
Try to create new slide number block with zero margins:
import pptxgen from "pptxgenjs";

pptx.defineSlideMaster({
  title: "Text Slide",
  slideNumber: {
    x: "90%",
    y: "10%",
    w: 0.5,
    margin: 0,
    fontSize: 12,
    lineSpacing: 16
  }
});

slide = pptx.addSlide({ masterName: "Text Slide" });

pptx.writeFile("sample.pptx");

Expected Behavior
slideNumber block has zero margins.
Result
margin: 0 has no effect on slide number margin.

Fixed, thanks for reporting this @ax2mx !


merging changes for anonymous functions and will create tests

failed travis build, going to bring this into its own branch instead so I can debug it


How to tracing GraphQL query with skywalking ?

Question
Does skywalking support to trace a GraphQL query and requests to other services?

We have provided the GraphQL service side plugin for several months. Tests are here, https://github.com/apache/skywalking/tree/master/test/plugin/scenarios

We have provided the GraphQL service side plugin for several months. Tests are here, https://github.com/apache/skywalking/tree/master/test/plugin/scenarios

Please read the documentation before submit the issue.  https://github.com/apache/skywalking/blob/v8.3.0/docs/en/setup/service-agent/java-agent/Supported-list.md

Please read the documentation before submit the issue.  https://github.com/apache/skywalking/blob/v8.3.0/docs/en/setup/service-agent/java-agent/Supported-list.md


Need to get js-jose added to http://jwt.io/

Now that js-jose also support JWT it should be added to http://jwt.io/

Do you want to send them a pull request?

I will, I started but have not had enough time to test it and not sure when I will so anyone else is free to do this :) I will try to get to it soon.

Closing this out. Feel free to complete this if you think it will add value.


How do I exclude my custom lock screen layer when sharing a screen?

I have a mobile workstation.
I want to see it run and operate remotely.
But my app has a lock screen that hides the phone's screen.
How do I filter out this mask when I look at it remotely?
livekit sdk has an API same this code?
val localVideoTrackOptions = LocalVideoTrackOptions.Builder()
    .excludeApps(listOf("com.example.customlock"))  // exclude this
    .build()

I'm not sure what you want to do. Do you want to exclude an app from being shared from the screen share?
If it helps, you can write your response in Chinese, I can understand a bit.

好的兄弟，我说一下我的思路。
我在做的是手机群控系统，但是手机托管在第三方的机房中，我需要这一个自定义的app来锁定这个屏幕，让第三方机房无法看到我手机的界面，但是我在屏幕共享的时候没有排除这个锁屏界面的显示，我之前的方案是要修改 framework层达到目的，但是毕竟手机系统不是我们的，所以 VisualDisplay 加上修改底层 framework 的方案不行。按理说通过 VisualDisplay 可以让虚拟显示器不显示这个自定义锁屏activity，然后推流这个虚拟显示器的内容即可达到需要的效果。但是 livekitsdk for android 将 visualdisplay 封装在内部了吧，不知道我要的这个方式如何能实现。
谢谢你。

Do you know my mean? @davidliu

Yes, I need create a custom VirtualDiaplay object transmitted to room.localParticipant.publishVideoTrack(track)?
@davidzhao

Yes. You can reference the code I linked above to make your own implementation.

I created a custom VideoCapturer. But what the remote video is seeing is not what my video is about. How to make this video replace the screen recorded video stream?
My code:

    private fun startScreenCapture(intentData: Intent) {
        val localParticipant = room.localParticipant

        GlobalScope.launch {
            val resolution = VideoResolution.H240
            val frameRate = 6
            val captureParams = getCaptureParams(context, resolution, frameRate)

            val excludedPackages = setOf("com.xxx.remote")

            val screenShareVideoCapturer = ScreenShareVideoCapturer(context, intentData, captureParams, excludedPackages)

            val track = localParticipant.createVideoTrack(
                capturer = screenShareVideoCapturer,
                name = "screenshare"
            )

            localParticipant.publishVideoTrack(
                track = track,
                options = VideoTrackPublishOptions(
                    simulcast = false,
                    videoEncoding = createCustomVideoEncoding(context, resolution, frameRate)
                )
            )

            screenShareVideoCapturer.startCapture(captureParams.width, captureParams.height, frameRate)
        }
    }

Your custom video capturer needs to pass the frames to the capturerObserver that's given in the initialize method.

Yes, it's can capture video, now!
thanks ;)
but, I want shared screen.
How to create ScreencastTrack?

My custom VideoCapturer can capture android screen video publish to livekit server.
but, the video have big black background. why?
I use room.localParticipant.setScreenShareEnabled(true, intentData) not black background.


Commands for updating package name are not working

Issue
The following commands for updating the package names (from github.com/sgnl-ai/adapter-template to something) are not working as expected.
sed -e 's,^module github\.com/sgnl-ai/adapter-template,github.com/your-org/your-repo,' -i go.mod
find pkg/ -type f -name '*.go' | xargs -n 1 sed -n -e 's,github\.com/sgnl-ai/adapter-template,github.com/your-org/your-repo,p' -i

Bugs that I see are:

nit: module is missing in the second half of the first command.
find should run on cmd/main.go too.
The -n flag is causing sed to empty the files in an attempt to replace a line.
The ending p in sed expression is causing to duplicate the line.

Tested Environment
> uname -a
Linux Bhargav-Ravuri 6.5.0-15-generic #15~22.04.1-Ubuntu SMP PREEMPT_DYNAMIC Fri Jan 12 18:54:30 UTC 2 x86_64 x86_64 x86_64 GNU/Linux

> sed --version
sed (GNU sed) 4.8
Packaged by Debian
Copyright (C) 2020 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later <https://gnu.org/licenses/gpl.html>.
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.

Written by Jay Fenlason, Tom Lord, Ken Pizzini,
Paolo Bonzini, Jim Meyering, and Assaf Gordon.

This sed program was built with SELinux support.
SELinux is disabled on this system.

GNU sed home page: <https://www.gnu.org/software/sed/>.
General help using GNU software: <https://www.gnu.org/gethelp/>.
E-mail bug reports to<EMAIL_ADDRESS>
For Maintainers
Maybe the issue is specific to my OS or the sed versions. But I'll be creating a PR with updated commands. Please check if it is working on your end.

/assign


Test: status bar accessibility

Refs: https://github.com/microsoft/vscode/issues/94677

[ ] win @dbaeumer
[x] linux @bpasero

Complexity: 2

If you are on Windows setup NVDA screen reader.
If you are on Linux setup Orca latest from source (tested that it work on Ubuntu 18, ArchLinux, Fedora 31. Ubuntu 19 showed some issue) docs. Some troubleshooting
Turn on your screen reader, open VS Code. For NVDA press Insrt + PgDown and the status bar content should be read. Make sure that what is read makes sense and captures what you see in the status bar.
For Orca you can orca -s from the command line and configure the keybinding for the read status command. Trigger that command and make sure that the status bar gets read out.
In both cases no duplicated content should be read out, and what is being read should be concise and descriptive enough.

For setting up Orca from master this comment has all the steps
https://github.com/microsoft/vscode/issues/90446#issuecomment-585782600

Since @bpasero is having issues setting up Orca.
@marcozehe might you know if VoiceOver has a command to focus the status bar? I searched and could not find one?
@joanmarie is there an easier way to setup orca with vscode then to take one from Master? In Ben's VM he can not check off Canocical Packages in his Ubuntu 18 and thus can not install latest Orca.

No, there is no command to read the status bar. Status bars aren't as common on MacOS it seems.

I could get Orca to run (from master) and verified that pressing the keybinding for reading status bar worked for me on Ubuntu 18 VM.

Great thanks @bpasero
@joanmarie please ignore this thread then.


migrate SwiftTestTool to Observability APIs

motivation: adopt new Observability APIs
changes: replace DiagnosticsEngine with Observability APIs in SwiftTestTool

needs #3777 first
replaces #3752

@swift-ci please smoke test

@abertelrud @neonichu this is rebased on #3777 and ready to go


[Feature]: Add better syntax highlighting to ComponentPreview

Describe the feature
There is currently a black & white theme for code highlighting in Documentation, I'd suggest changing this to a more vibrent theme, like the current theme used in radix-vue documents.
Additional information

[X] I intend to submit a PR for this feature.
[ ] I have already implemented and/or tested this feature.

I agree with that but I thought it would be better to be like main shadcn-ui that's why I didn't use shiki themes

I understand your point, but with the Vue ecosystem, we have the ability to incorporate features that are not available in shadcn-ui (the React version). We are not limited to only support what  shadcn-ui supports.
I have cloned and modified shadcn-vue to suit my own needs, and some of the changes I've made could benefit the end-users that are not currently implemented in shadcn-ui. I am working on creating some feature improvement requests based on my experiments.
Regarding this particular feature request, I may have a bias towards certain styles. If you believe that the current styling is better, feel free to close this issue as resolved. ❤


[Bug] Incorrect output of Palindrome Calculator.

Description
The output of Palindrome Calculator is wrong . For instance for the string "Eye" output is that it is not a palindrome , but we know that it is a palindrome string , and many other strings like this .
Kindly assign me and I will fix this issue .
Screenshots

Checklist

[X] I have checked the existing issues.
[X] I follow Contributing Guidelines of this project.
[X] I am a GSSOC'24 contributor.
[ ] I am a IWOC'24 contributor.
[ ] I am a JWOC'24 contributor.
[ ] I am a SWOC'24 contributor.

Code of Conduct

[X] I agree to follow this project's "Code of Conduct".

@Rakesh9100 this issue can be assigned

Then assign me @Rakesh9100

This is not any bug since palindrome checks for the case as well, so Eye is not palindrome but eye or EyE is plaindrome @meetarora10 @jessicachawla


Switching context produces errors.

When trying to switch to a new context, while being on an active context which is not present in the config file anymore, the switch command produces errors.
Example (problematic) config:
version: 1
current_context: oldctx
contexts:
  newctx:
    bootstrap_servers:
      - broker01.my-host.com:9094
      - broker02.my-host.com:9094
    security_protocol: SASL_PLAINTEXT
    sasl_params:
      mechanism: PLAIN
      user: root
      password: toor

Then, when executing the ctx command, I get following error:
$ esque ctx
ValidationException: Context oldctx does not exist. Available contexts ['newctx']
Run with `--verbose` for complete error.

It's a minor issue, because it is obvious, but it would still be nice to have this fixed :)

I think that is actually the expected behavior. Your config is no longer valid since your current_context doesn't exist and esque fails during initialization of the internal config object. I added a new feature in #140 which could potentially resolve this by allowing esque to fix those error itself.
For the future you could avoid those config errors by running esque config edit which opens an editor with the config for you and validates it afterwards.

For the future you could avoid those config errors by running esque config edit which opens an editor with the config for you and validates it afterwards.

Yeah, I opened it up manually, and that's why it happened. My bad 😅


macOS 13.0 Beta (22A5331f) DBeaver doesn't show dropdown menus

Description
macOS 13.0 Beta (22A5331f) DBeaver doesn't show dropdown menus
DBeaver Version
Community edition last version
Operating System
No response
Database and driver
No response
Steps to reproduce
No response
Additional context
No response

Hello @joaofconde
DBeaver is based on the Eclipse platform, and this is the Eclipse bug: https://bugs.eclipse.org/bugs/show_bug.cgi?id=580322
It seems to be fixed already, but this fix will be available in our product only in Autumn with the new Eclipse release.
As a workaround, for now, you can use DBeaver versions <= 22.1.0.
I am closing this as a duplicate of #17204

Hi all, with last version now works perfectly

Thanks for the report @joaofconde


Potential Issues with WSAWaitForMultipleEvents and High Concurrent TCP Connections

Due to the constraints imposed by WSA_MAXIMUM_WAIT_EVENTS, high concurrent TCP connections can potentially give rise to issues with WSAWaitForMultipleEvents.
https://github.com/wiresock/ndisapi/blob/805fe6976a5fc680387126431fa7e1fb02cc818b/examples/cpp/common/proxy/tcp_proxy_server.h#L525
a possible solution from https://stackoverflow.com/a/11989138:
    DWORD WSAWaitForMultipleEvents(DWORD count, const HANDLE* pHandles,
                                   DWORD millisecs) {
        DWORD retval = WAIT_TIMEOUT;

        // Check if objects need to be split up. In theory, the maximum is
        // MAXIMUM_WAIT_OBJECTS, but I found this code performs slightly faster
        // if the object are broken down in batches smaller than this.
        if (count >= MAXIMUM_WAIT_OBJECTS) {
            // loop continuously if infinite timeout specified
            do {
                // divide the batch of handles in two halves ...
                DWORD split = count / 2;
                DWORD wait = (millisecs == INFINITE ? 2000 : millisecs) / 2;
                int random = rand();

                // ... and recurse down both branches in pseudo random order
                for (short branch = 0; branch < 2 && retval == WAIT_TIMEOUT;
                     branch++) {
                    if (random % 2 == branch) {
                        // recurse the lower half
                        retval =
                            WSAWaitForMultipleEvents(split, pHandles, wait);
                    } else {
                        // recurse the upper half
                        retval = WSAWaitForMultipleEvents(
                            count - split, pHandles + split, wait);
                        if (retval >= WAIT_OBJECT_0 &&
                            retval < WAIT_OBJECT_0 + split)
                            retval += split;
                    }
                }
            } while (millisecs == INFINITE && retval == WAIT_TIMEOUT);
        } else {
            // call the native win32 interface
            retval = ::WSAWaitForMultipleEvents(count, pHandles, FALSE,
                                                millisecs, FALSE);
        }

        // done
        return (retval);
    }

Thanks for the hint, I have updated the tcp_proxy_server.h.


Educational area and degree

I am currently a student attending university and I'm trying to convert my jrs resume to the FRESH format. I am a bit confused about the education field: what is the difference between "level" and "degree"? Also, what is "title" used for? I am an Electrical Engineering student working towards my BS, any advice on how I should represent that? Is there an equivalent to jrs's "area" field?
Btw, great work on this whole project, it's fantastic when compared with JSON Resume (no offense to them, jrs is great too!).

Hi @D1SC0tech thanks for the note of support and welcome to the FRESH community! The education section will receive some polish in the next version (this week, actually), but here's essentially what the fields mean:

The title field is the freeform display name of your degree, program, or role at the institution. For most cases, this should just be the name of your degree. Most FRESH themes will associate this closely with the name of the institution:
  [title], Somewhere University
  Somewhere University ([title])

So if title is (for example) BSEE you'd get:
 BSEE, Somewhere University
 Somewhere University (BSEE)

Think of title as the friendly display name of your degree, cert, program, role, etc. We make this a freeform field because there's so much variation in academic degrees, to model this any other way would increase the complexity of the education section beyond what makes sense for typical resume use.

The level is an indication of your overall level of education and corresponds to the drop-downs or multiple choices you may have seen on some employment submission forms: 'some college', '4-year degree', 'high school diploma', 'advanced degree', etc.

The degree is the formal statement of your degree's (or cert's, or program's, etc.) name. Where title is really more for display purposes, degree is informational.

So assuming a Western-style resume, probably the simplest thing for your case would be:

Set title to BSEE.
Set level to some college or in progress (no themes currently display this, so it won't matter yet).
Set degree to Bachelor of Science, Electrical Engineering or whatever your school uses on official letterhead.
Ignore other fields until the next version of FRESH/FRESCA later this week : )

Wow, fantastic response, thank you! That clears up a lot, and I look forward to the next version!


Clarlars development

Update raw zip files to match app-designer

Looks like you accidentally opened this to master, closing.


Seed.random should not generate seeds longer than 81 trytes

Seed.random generates 82-tryte seeds by default.  However, beyond 81 trytes, you can't increase the security of your seed — not even against brute force attacks (see Why aren't seeds longer than 81 trytes more secure? for more info).
Using urandom to generate random bytes is a clever approach, but because we have to translate bytes into trytes, we are limited to generating even numbers of trytes.  An alternate approach is needed.
Modify Seed.random so that it generates exactly 81 trytes each time:

Remove the length argument.
Use a different method of generating random trytes (see http://stackoverflow.com/a/2257449/ for some ideas — note that using a CSPRNG is mandatory!).

Scheduled for release: 1.2.0


USHIFT-1422: use the regular RPM repositories for dependencies in image-builder script

Instead of downloading dependencies and manipulating the local repo,
use the regular repositories normally. This allows composer to find the
pinned version of openvswitch3.1 needed right now.
Includes some other cleanup changes.
/assign @ggiguash @jogeo

I was able to build an ISO using RPMs from source and boot the VM successfully.
[root@localhost ~]# rpm-ostree db list e211ad60c69b42703c430f88f64b07a1c20a2a2fe6a99287b41706266bd9c1a2 | grep openvswitch
 openvswitch-selinux-extra-policy-1.0-31.el9fdp.noarch
 openvswitch3.1-3.1.0-14.el9fdp.x86_64

[root@localhost ~]# oc get pods -A
NAMESPACE                  NAME                                       READY   STATUS    RESTARTS      AGE
kube-system                csi-snapshot-controller-75b9b8c9ff-wghtt   1/1     Running   0             22m
kube-system                csi-snapshot-webhook-84684f46f6-x2nxv      1/1     Running   0             22m
openshift-dns              dns-default-jc7pg                          2/2     Running   0             21m
openshift-dns              node-resolver-wdvjf                        1/1     Running   0             22m
openshift-ingress          router-default-77dbd6c6b6-qlzgx            1/1     Running   0             22m
openshift-ovn-kubernetes   ovnkube-master-k24j9                       4/4     Running   1 (21m ago)   22m
openshift-ovn-kubernetes   ovnkube-node-b7djt                         1/1     Running   1 (21m ago)   22m
openshift-service-ca       service-ca-ff55d8996-wzsw4                 1/1     Running   0             22m
openshift-storage          topolvm-controller-ffd4fbff9-l588d         5/5     Running   0             22m
openshift-storage          topolvm-node-bk84t                         4/4     Running   0             21m

@ggiguash I think I have addressed all of your comments.

/lgtm


Test classify

Hi @manjumegh what's missing is remove the comments  that were already classified from the potential-bad-01.csv . THe app should do that for you. When you click append results, it should show the following screen and you should say yes..


ServiceOptions#getApplicationName() - performance issue

Dear Google Team,
During our company internal performance testing we found that there are some places for improvement (although it's not critical but very nice to have). Method com.google.cloud.ServiceOptions#getApplicationName() is used with each request (for example from code of method com.google.cloud.datastore.spi.v1.HttpDatastoreRpc#getHttpRequestInitializer() )
. This method is used each time during EACH request. Basically what it does under the hood: it loads some resource from classpath with EACH request. It is leading to not necessary memory and CPU consumption. It is especially notable on a high-loaded project with millions requests per day.
Proposal is to cache result of method ServiceOptions#getApplicationName(). It's constant value anyway(by it's nature) - so there's no need to load it each time with each request (it won't change). Or load it once on startup.
Bug found in a library: com.google.cloud:google-cloud-datastore:1.2.3
Best Regards,
Dionis Argiri

cc @pongad another perf issue

This looks like https://github.com/GoogleCloudPlatform/google-cloud-java/issues/2234. I thought I fixed this, but apparently not. I'll look into it.


Add parsed magnets as soon as meta is downloaded

This is somewhat related to #280.
I've tried to add 4 torrents via magnet links, 3/4 were parsed instantly while 4th one is stalled (been like that for last 5 mins or so, could be that there are connection issues or something else). If I click on X to terminate the parsing, even those 3, successful ones are lost as well.

Magnet parser shouldn't wait for all of the magnet links to parse and download metadata before adding them to the list of 'active' torrents. IMHO it should add 'em as soon as metadata is received. That way we'll prevent loss of already viable torrents for download. It's easier to re-add just that one instead of all of 'em.
edit: forgot to mention that i'm using latest available artifact from AppVeyor.

Agreed - the easiest way to fix it would be to show the add torrent dialog for the magnet links that were actually downloaded and perhaps show an alert for the ones which weren't. This lets the user add the successful ones with the option to retry the failed ones later.

Yes, that would be great if it's doable.
Thanks :)

Fixed in #324.


Release stubs for v2.11

Happy to help with this going forward :-)

I just don't want to see "WordPress" on my screen (and its unsolvable problems).

https://github.com/php-stubs/wp-cli-stubs/releases/tag/v2.11.0


Difficulty distinguishing the color of redirects

It is a funny idea that uses 2 green colors very close to each other.
On the browser, it is very difficult to distinguish which links are correct (200) and which links are redirects (301).

+1