OAuth?

While working on the inital developer documentation I noticed that the app actually "handles" the username and password of the app users. I was surprised by that.
Does Commons/Mediawiki not support OAuth so the app can use a token based authentication mechanism?

I was indeed wondering about this. The current login mechanism is a slightly-modified version of the legacy code from several years back. It's entirely possible that Commons/Mediawiki might support OAuth now.

One data point: Wikipedia for iOS also has an integrated login, not Oauth.

There is a mediawiki extension OAuth that is used by Wikipedia. Perhaps a thing for a library to share with the Wikipedia app? I made one of several python packages to connect with mediawiki and can explain some of the basics.

Awesome. Do you know any apps that actually use this? Is this also available on Commons?

The Oauth token works for all projects and you can customize what permissions the consumer needs. Here are the existing clients: https://m.mediawiki.org/wiki/Special:MyLanguage/OAuth/For_Developers
I don't know if there is a Java library, but add it on that page if you find one. The python mwoauth library is very readable and could be a template for a Java implementation.

Nice, looking good and pretty straightforward.
Is there a list of apps, websites or projects that are using this already somewhere?

Last point at https://m.mediawiki.org/wiki/Special:MyLanguage/OAuth/For_Developers#content-collapsible-block-3 might be a problem as well.

Are you mentioning this (The link to the headline does not work for me)?

The application secret must be kept secret. Submitting it to source control or putting it into user-accessible code (such as mobile app or desktop application; even if it is obfuscated) undermines the security model and will result in admins forcefully disabling the application.

https://www.mediawiki.org/wiki/OAuth/For_Developers#Security_benefits_and_trade-offs
The solution seems to be:
https://stackoverflow.com/questions/4419915/how-to-keep-the-oauth-consumer-secret-safe-and-how-to-react-when-its-compromis
We could set up a web service on tool labs (Which I guess got renamed again and is now Tool forge) that handles the authentification between the app and Wikipedia? I have some practice with that and could help.

Yep, that was the one.
Is this "normal" for OAuth mobile apps?
(Or is this a OAuth 1 vs 2 thing?)

I am not totally sure, but my current understanding is that there always need top be 3 parts involved. The Oauth provider, the consumer and the client. If the mobile app is both consumer and client the security can never work. Maybe that ist why there is no android library?

No Android Library?

https://gist.github.com/JakeWharton/f26f19732f0c5907e1ab (OAuth 1.0a)
https://gist.github.com/alex-shpak/da1e65f52dc916716930
https://github.com/pakerfeldt/okhttp-signpost
https://github.com/corcoran/okhttp-oauth2-client

That's a quick sample of code that utilize OkHttp (a transitive dependency that already exists in this app)
Do any of them cover the cases we need here?

He meant the list here: https://m.mediawiki.org/wiki/Special:MyLanguage/OAuth/For_Developers#Libraries
If the Mediawiki implementation is standard, any of the ones you posted matching the Mediawiki OAuth version should be fine though.

Thanks for the correction @janpio, I was barking up the wrong tree on that one!

Adding this as high priority due to a request from Michael Holloway (WMF).

I was trying to submit our app's details for OAuth consumer registration but am not sure about the callback URL column as our app is a native Android app. Can anyone help with it? https://meta.wikimedia.org/wiki/Special:OAuthConsumerRegistration/propose

Tagging @whym and @nicolas-raoul :)

I have always wondered how non-web applications handle OAuth.
From what I read at https://developer.android.com/training/id-auth/authenticate.html it seems that no callback URL is needed?

The Callback URL column was mandatory so have put https://commons-app.github.io/ in it. Have been assigned a consumer token and secret token for the app

The way ahead isn't very clear to me so i have created a phab task hoping someone could help us with it. https://phabricator.wikimedia.org/T179519

Someone replied to my phab task from WMF.

OAuth 1.0 does not support clients without a backend component, sadly (as they have no way to keep the consumer secret secret). See #5 of the [[https://meta.wikimedia.org/wiki/OAuth_app_guidelines#Review_guidelines|review guidelines]].
You can use [[https://www.mediawiki.org/wiki/OAuth/Owner-only_consumers|owner-only OAuth]] to authenticate without passwords, but the setup process is very user-unfriendly at the moment.
Who did you talk to about WMF login requirements?

Our app doesn't have a backend component as far as I know. Even if we register as a Owner only app, the app will only be able to act in the name of the user who registered them. See Owner Only apps section of OAuth app guidelines
@tobias47n9e @nicolas-raoul @misaochan Any insights?

Anyways the OAuth Consumer request also got rejected. https://meta.wikimedia.org/wiki/Special:OAuthListConsumers/view/efc15ccf765eb39aad7c6a3c25f73826?markasread=2870231

I will email Corey and Aeryn about it and point them to the Phab task. Don't worry about it for now @maskaravivek

We made some progress with the OAuth implementation using oauth library that Corey Floyd from WPF suggested.
Am now stuck at some E006[1] error. This seems to be related to issues other have run into while using OAuth on mobile[2][3]. My suspicion is that it might have to do something with redirection to mobile version of wiki URL. See WIP PR[4]
Any help with it would be really great. :) @tobias47n9e @nicolas-raoul @whym
[1] https://www.mediawiki.org/wiki/Help:OAuth/Errors#E006
[2] https://phabricator.wikimedia.org/T74186
[3] https://phabricator.wikimedia.org/T119343
[4] https://github.com/commons-app/apps-android-commons/pull/935
Attached screenshot for ref.

Also attached app logs for more context.
app_logs.txt

The discussion is continuing on https://phabricator.wikimedia.org/T179519 for anyone who might be interested.

This has been found to be impossible with WMF's current version of OAuth (see https://phabricator.wikimedia.org/T179519 ), and it has also been discovered that adding the Key to the app will open up the possibility for 3rd parties to impersonate the Commons app.
Consequently the requirement for us to use OAuth has been waived - based on Corey Floyd (WMF)'s email to me. I will close this for now, but feel free to reopen if WMF updates its OAuth version.

Good outcome.

Small info, not sure whether it is useful: I just realized that the OSMTracker for Android app uses Oauth with the OpenStreetMap server. It is a bit awkward as the app opens Firefox at the server's /oauth URL but then you are not sent back to the app after authorizing.


GEODE-5574: unit test coverage for RegionMapDestroy

This is a major rewrite of RegionMapDestroyTest. The test was changed enough that I doubt the diffs will be helpful. I changed it to use mocks except for the class under test.
Each test method now has the pattern "given, do, verify".
This unit test now covers 90% of the class (used to be 60%) and the uncovered code is debug logging and what I think is dead code.
I made one small change to the product to use "putEntryIfAbsent" instead of "getEntryMap().putIfAbsent".
Thank you for submitting a contribution to Apache Geode.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[ ] Is there a JIRA ticket associated with this PR? Is it referenced in the commit message?

[ ] Has your PR been rebased against the latest commit within the target branch (typically develop)?

[ ] Is your initial contribution a single, squashed commit?

[ ] Does gradlew build run cleanly?

[ ] Have you written or updated unit tests to verify your changes?

[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and
submit an update to your PR as soon as possible. If you need help, please send an
email to<EMAIL_ADDRESS>

the integrationTest failure was a timeout and I see that other ci runs also had this issue. Since all this PR does is modify a unit test, I'm not going to hold it up the integrationTest timeout.


[Feature Request]

Can you pleade add support for the new SPMC?
http://spmc.semperpax.com/
Thanks

Hi,
As far as I am aware SPMC already supports being set as the launcher. Is that no longer true?

Also, I thought I already added support for SPMC in the past. Have you verified that the current SPMC entry doesn't work anymore?

yes the activity has been changed to  com.semperpax.spmc16/com.semperpax.spmc16.Splash

I see. I'm away on vacation this week but I'll try to remember to update it when I get back.

thank you :)

A new update was just published on the Play Store. Give it a few hours to propagate and let me know.


Encode attributes

HI
I played with your library a bit, and it is really a great job! However there is a huge technical gap in the decoding part: We are losing the information that the property was an attribute or an element. For decoding it works well, but it can not save the information in the same way.
Example:
struct Test {
var name: String?
var information: String?
}

Input:
<test name="tag">
<information>tag information</information>
</test>

Output:
<test>
<name>tag</name>
<information>tag information</information>
</test>

Figured out a way to encode attributes. By default, it treats all Encodable props as XML elements. The  XMLEncoder.attributeEncodingStrategy property controls this. That enum is set to default (by default) but can be changed to custom, where you then provide a closure to control if a particular piece of data should be an attribute or not. =Return true from the closure to make it be an attribute. I have used it in a test app and works perfectly

@stonedauwg Can you show how did you use XMLEncoder.attributeEncodingStrategy  please? I am interested in this part too.
Also, I think an encoding example (with attributes) should be added to the README.md 👍

@nacho4d attributeEncodingStrategy has since been superseded by nodeEncodingStrategy.
https://github.com/MaxDesiatov/XMLCoder/blob/master/Tests/XMLCoderTests/NodeEncodingStrategyTests.swift should give you an idea on how to use it for the time being.


Legacy context API has been detected within a strict-mode tree.

I have
"react-breadcrumbs-dynamic": "^1.2.1",
"react-through": "^1.1.4",
And have the Warning in console
Legacy context API has been detected within a strict-mode tree.

The old API will be supported in all 16.x releases, but applications using it should migrate to the new version.

Please update the following components: ThroughAgent

Learn more about this warning here: https://fb.me/react-legacy-context
    in ThroughAgent

Is there any chance for update or alternative package?

Yes sure. It is just warning or something more? This fix is planned (in my free time) after next release of react-app-rewired-alias. I hope it will be soon. Thanks

Hi @oklas, is there any chance you get this fixed anytime soon ?


Jetbrains Mono - 400, 500, 700 specific stylesheets give an error

Describe the bug
I'm not able to load the 400, 500, and 700 weights specifically via their respective stylesheets in JavaScript. I've used this same technique to load specific font weights from Inter (open source) and Sora (google font) via fontsource.
To Reproduce
Steps to reproduce the behavior:

install fontsource-jetbrains-mono (I'm using yarn if that matters)

Add stylesheets to JavaScript:

import 'fontsource-jetbrains-mono/400.css'
import 'fontsource-jetbrains-mono/500.css'
import 'fontsource-jetbrains-mono/700.css'

Console error:
Module not found: Error: Can't resolve 'fontsource-jetbrains-mono/400.css'
Module not found: Error: Can't resolve 'fontsource-jetbrains-mono/500.css'
Module not found: Error: Can't resolve 'fontsource-jetbrains-mono/700.css'

Expected behavior
I expected those weights to load based on the Fontsource page here that says they are available.
If I do a generic import (import 'fontsource-jetbrains-mono') there is no error.

Hey @austriker27, thanks for checking in.
#92 should address why this issue exists and that it is on the agenda to be fixed. If you refer to the package readme on NPM or GitHub, it has a filler subset called 'all'.
The current method to import this would be:
import 'fontsource-jetbrains-mono/all-400.css'
import 'fontsource-jetbrains-mono/all-500-normal.css'
import 'fontsource-jetbrains-mono/all-700-italic.css'

I'll be closing this for now since #92 will address how to improve the issue you will be seeing.


ttnn implementation for yolox_m

reference implementation: https://github.com/Megvii-BaseDetection/YOLOX/tree/main

Yoloapfpn reference is being tested against torch.
Tested focus, dark<2,3,4> sub modules successfully.
dark5 sub module has mismatch in outputs. I'll generate graphs and verify the implementation.
Subsequently, I'll be working on ttnn implementation of the above sub-modules and then I'll start with the head module.

Yolox_m has two modules YOLOAPFPN and YOLOHEAD
Currently working on YOLOAPFPN
1, Reference implementation of  yolopafpn and its submodules focus, dark2, dark3, dark4, dark 5  completed
2. Tested reference against torch: dark<2,3,4> sub modules successful.
3. Test reference against torch: dark5, other convs of yoloapfpn in progress
Upcoming tasks

Write reference implemetation of YOLOHEAD
Start with ttnn implementation

Profile perf results:
Focus submodule: ops_perf_results_yolox_focus_2024_06_03_13_31_07.csv
Dark2 submodule: ops_perf_results_yolox_dark2_2024_06_03_13_09_36.csv
Dark3 submodule: ops_perf_results_yolox_dark3_2024_06_03_13_12_49.csv
Dark4 submodule:ops_perf_results_yolox_dark4_2024_06_03_13_13_36.csv
Dark5 submodule:ops_perf_results_yolox_dark5_2024_06_03_13_30_19.csv
CSPDarknet submodule: ops_perf_results_yolox_cspdarknet_2024_06_03_13_35_28.csv


allow user to register Kontena account without invitation to Kontena server

Currently user is not able to register Kontena account if he/she has not invitation to a Kontena server. This PR fixes this issue.
Fixes #8

I Close this PR now since code structure has been changed. I will make a new PR.


[Desktop] Editor loses focus when toolbar buttons are pressed.

I experienced this on desktop, it may be the same for web. Current master a855f59922a8efef1d654a23ea1d325e8f587eda
Title pretty much says it. Whenever a toolbar button is pressed, the editor loses focus. This makes to so that if you click bold, for example, the cursor is lost and you need to click back into the editor to recover it. When you do click the editor, the active text is no longer bolded. (This is when there is no selection.)

Flutter 3.3.0 • channel stable • https://github.com/flutter/flutter.git
Framework • revision ffccd96b62 (3 weeks ago) • 2022-08-29 17:28:57 -0700
Engine • revision 5e9e0e0aa8
Tools • Dart 2.18.0 • DevTools 2.15.0

Closing this since, this can now be sovled by requesting focus in the afterButtonPressed callback of the editor.

Possibly a better solution: set onTapOutside to QuillEditorConfigurations to manually control focus.
The focus thief is QuillRawEditorState _defaultOnTapOutside

Possibly a better solution: set onTapOutside to QuillEditorConfigurations to manually control focus.
The focus thief is QuillRawEditorState _defaultOnTapOutside

Hi, I have this issue, can you show how you would do that? I don't understand what you exactly mean by "manually control focus". Thanks

@maelchiotti, this can be done
    QuillEditor.basic(
      focusNode: _focusNode,
      configurations: QuillEditorConfigurations(
        controller: _quillController,
        onTapOutside: (_, __) {
          // _focusNode.unfocus(); or doNothing
        },
      ),
    )

If onTapOutside is set then QuillEditor will not control focus. We can control it ourselves through _focusNode.


LAV Filters Megamix 0.77.1-2

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[x] Does your manifest conform to the 1.4 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Like PDFill, the manifest has to be deleted or updated with each update because theres only one url in the author's site that always directs to the latest version. No 32-bit version from 0.77.1-2 and hence no ReClock.
Microsoft Reviewers: Open in CodeFlow

How is this different from https://github.com/microsoft/winget-pkgs/pull/1467 ?

I'm unable to tell if this is the correct domain for this package.

https://github.com/microsoft/winget-pkgs/issues/95155

I'm unable to tell if this is the correct domain for this package.
* [[Package Issue]: ranpha.LavFiltersMegamix Unofficial package released #95155](https://github.com/microsoft/winget-pkgs/issues/95155)

It's just a codec pack like K-Lite.
https://github.com/microsoft/winget-pkgs/tree/master/manifests/c/CodecGuide/K-LiteCodecPack
Also clsid2.mpc-hc contains LAV Filters
https://github.com/microsoft/winget-pkgs/tree/master/manifests/c/clsid2/mpc-hc
https://github.com/clsid2/mpc-hc

The internal codecs are developed in an external project (LAV Filters). That project is still actively maintained, and MPC-HC will be updated periodically with the latest codecs. Same applies to code/files from other external projects that are used by MPC-HC, such as MediaInfo.


Illegal string offset

Cool, I will look into it... thanks for letting me know

I've spent the last hour trying to resolve this but I can't get it to work, then I thought... if the user is inactive they would not be able to login to Kanboard (I'm guessing).... so I just removed the Status field.  Sorry about that.


Update SysState to Java 8

JDK 6 is EoL and soon JDK 7 will be as well.
Upgrade SysState to Java 8

Fixed!


(CLOUD-1005) create vm with ARM fails

Fixed:

calls to .get() on azure clients needs to be in a begin rescue block, as if it fails to find the target, it throws an exception and breaks the code.
build_params had to be moved to before the storage account, due to dependencies.
def create_virtual_network(args)

     Puppet.debug("Creating vnet '#{virtual_network.name}'")  <= virtual_network doesnt exist? and caused an exception in the success path.

@gregohardy This looks good to me.


Allow passing multiple file patterns

What kind of change does this PR introduce?

[x] Feature
[x] Test
[x] Docs

Description:
Adds support for passing multiple file patterns to options.filesPattern. Addresses #101

LGTM 👌

@all-contributors please add @Stosiu for code


(PUP-7053) shorten username in utf-8 test for aix

By default AIX usernames are 8 chars. To ensure we pass in the default
configuration, shorten the username length in test utf8_user_comment to 8
characters.
Signed-off-by: Moses Mendoza<EMAIL_ADDRESS>

@kurtwall this is good to merge

CLA signed by all contributors.


Run CI when adding a PR to a merge queue

Let's give a try to the new Github Merge Queues feature - Pull request merge queue (public beta) and Managing a merge queue

The change looks good to me. Re-running some Windows tests while it won't be related to the change.


Horizontal annotations

Checklist

[X] I am running the latest version of the card
[X] I checked if there isn't another feature request opened with the same request
[X] I checked that the feature is not already available in the dev branch

Is your feature request related to a problem? Please describe.
Annotations are available as part of the original chart API. I am using them in the following way:
    apex_config:
      annotations:
        yaxis:
          - y: 0
            borderColor: '#000'
            strokeDashArray: 0

This is useful for me when I'm displaying temperatures, the freezing point (0°C) is a thicker line than the others. Now the problem is that this line is always drawn, even if the temperatures rise high enough so that the zero line is not visible anymore. The result is that the line is drawn on the legend, where the zero line is in the background.

Describe the solution you'd like
If you think it would be easy to incorporate this in your API, maybe you could only draw the annotation when you know it is visible.
Describe alternatives you've considered
I'm using it as it is right now, I did not find any workaround.

+1 for this. Would be great to have on Air Quality graphs as well

That's a bug in the library, but I can probably find a way to counter that.
I have very little time at the moment so bear with me 😊

this one is still needed, but low prio

still needed


Create dataset loader for KoPI-CC_News

NusaCatalogue: https://indonlp.github.io/nusa-catalogue/card.html?kopi_cc_news

Dataset
kopi_cc_news

Description
KoPI(Korpus Perayapan Indonesia)-CC_News is Indonesian Only Extract from CC NEWS Common Crawl from 2016-2022(july) ,each snapshots get extracted using warcio and filter using fasttext

License
CC0

#self-assign


Could not get Rust code hightlighted

(never mind the Chinese words)
As the screenshot shows, it is so weird that hightlight.js could highlight Python code, but could not work with the Rust code. This is my website, is there something wrong with my website?

Are you loading the rust.js syntax? I don't think it's built-in.

If you're still having issues a reproducible example fo this not working would be great, such as a jsfiddle, etc...
You can even fork my template:
https://jsfiddle.net/ajoshguy/nagkqytv/18/

Are you loading the rust.js syntax? I don't think it's built-in.

Where should I find the info about rust.js, it seems the official doc doesn't make mention of the rust.js syntax file

https://highlightjs.org/usage/
Read Getting Started and Getting the library.  You need to add a second JS script tag or else build a custom library that includes Rust.

It works now, thanks so much for your help.

Glad to help!


backup create with vfs has case-sensitivity issues

When using convoy v0.3 to create a backup of a snapshot with a vfs destination, it appears that convoy does some sort of toLower() function on the path name causing an invalid path to be attempted.
Example:
~# convoy backup create jenkins-initial-snapshot --dest vfs:///mnt/nfs/mynfsserver/Backup/docker/convoy
ERRO[0000] Error response from server, VFS path /mnt/nfs/mynfsserver/backup/docker/convoy doesn't exist or is not a directory

{
        "Error": "Error response from server, VFS path /mnt/nfs/mynfsserver/backup/docker/convoy doesn't exist or is not a directory\n"
}
~#

Notice in the vfs path I provide, Backup is uppercase, and when convoy returns it back, it's lowercase. I believe this is the reason it's not finding the vfs target.

I am looking into this.

Hi @prelegalwonder
Could you try the binary below to see if it solves your issue? Only replace "convoy" binary from tarball is fine.
https://s3-us-west-1.amazonaws.com/sheng/convoy.tar.gz

@yasker Thank you! Confirmed, I was able to use the directory with the upper-case name.

Thanks @prelegalwonder , I would push the fix and include it in the next release. In the meantime you can still using the binary, it's based off master and should be stable.


Add a simple label element

Describe what problem your feature request solves
It is necessary in order for text to appear on the screen, to draw some element you may not need. Diagrams often have "areas" or need for such text.
Describe the solution you'd like
A simple label tool would save lots of time. It would be a box dragged out which could contain text, or simply one click to get a text cursor- the way you can do for each element today. It would not need any associated threats, but I'd be completely satisfied if they came with everything they don't need for simplicity sake right now.
Additional context
I have been using trust boundaries to be able to draw labels- but even those do weird things with the text after a while. I'm sure many people are doing the same thing. The drawing area is already unnecessarily limited- why fill it with unnecessary objects when all you want is a label.

Hi @storkinsj
Any thoughts on the following implementation? It's a resizable / movable box with user-defined text.  I think it'd make sense having the background be a different color (currently yellow-ish) to signify that it's not a regular entity but more of metadata.
The left is the item picker to drag/drop from, the right is on the graph:

Hello @lreading and @storkinsj - I think anything rectangular with a defined border is going to look like a threat model actor? When I have seen labels applied to diagrams then they are floating text without background or border. Here is a slide from one of my presentations :

Could we have text with transparent background and no border?

Could we have text with transparent background and no border?

We can have anything we want! :wink: I'll play with some options and post a screenshot later.  This is a great point, thank you!

Per this discussion, here's the text block with a transparent background and no border:

Yes, this looks great, thanks for doing this

Sorry I missed your questions and comments- THANK YOU. The transparent box achieves the desired effect. I think part of the original feedback states we are trying to avoid the whole object being in the diagram. Perfect - thank you!

Closing as this is complete in v2


incorrect impl method get

Client using get method
returns only one result, which is not entirely true. According to the protocol, you can specify several keys and obtain several values. I would like to add a little to this feature.
https://github.com/memcached/memcached/blob/9dc8b148345b27dd86580641d17f4268885b5894/doc/protocol.txt#L293

so, im sorry, you impl in get_many)
https://github.com/emcache/emcache/blob/449dad934ab81d5b821a9b19c9fb92f8cacb17a0/emcache/client.py#L315

please, remove this issue

No problemo.


feat: start instructlab session

What does this PR do?
After discussion happening in https://github.com/containers/podman-desktop-extension-ai-lab/issues/1617, here is the frontend implementation for the New Instruct Lab session page
Screenshot / video of UI
https://github.com/user-attachments/assets/66421239-8a15-426f-8f04-d1dbdb491e3e
ℹ I took inspiration from the Play Kubernetes YAML page for the radio selection. It give better consistency has it was the only place with similar design

What issues does this PR fix or reference?
Fixes https://github.com/containers/podman-desktop-extension-ai-lab/issues/1617
How to test this PR?

[x] unit tests has been provided

I'm questioning the Skills or Knowledges option, I don't see where in InstructLab this is a requirement.

@axel7083 looks good - I'm glad to see this coming together! Couple minor things I'd like to request:

Can the +add knowledge/skill to use button be left-aligned as the rest of the UI elements, since there's no input field next to it?
The suggestion was for section to be called "Fine-tune with InstructLab" - I see the UI is structured a bit differently from the mock, but I think it will benefit from this title.
On that note, let's agree on the consistent spelling: my suggestion is Fine-tune and fine-tuning, as on Hugging Face

@jeffmaury let's check with the InstructLab team - do you know who would be the best person to contact there for more info?

I would change the "not selected" knowledge/skill faCircleCheck to some blank one or just some circle, because since we have two faCircleCheck, it seems to me like you have both enabled and you are just adding skills/knowledge to both of them in a same time

I would change the "not selected" knowledge/skill faCircleCheck to some blank one or just some circle, because since we have two faCircleCheck, it seems to me like you have both enabled and you are just adding skills/knowledge to both of them in a same time. Or use just radio buttons

@axel7083 I second that - the pattern I added in the mock already exists in Podman Desktop UI, soo I think that element can be reused.

@axel7083 looks good - I'm glad to see this coming together! Couple minor things I'd like to request:
* Can the `+add knowledge/skill to use` button be left-aligned as the rest of the UI elements, since there's no input field next to it?

* The suggestion was for section to be called "Fine-tune with InstructLab" - I see the UI is structured a bit differently from [the mock](https://private-user-images.githubusercontent.com/14108590/373563894-91dd53f4-423a-41f7-be61-0d7fa16e6a86.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjgzMzI5ODMsIm5iZiI6MTcyODMzMjY4MywicGF0aCI6Ii8xNDEwODU5MC8zNzM1NjM4OTQtOTFkZDUzZjQtNDIzYS00MWY3LWJlNjEtMGQ3ZmExNmU2YTg2LnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDEwMDclMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQxMDA3VDIwMjQ0M1omWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTJkNDViMDZmNzBlODI0YjEzY2I2YWFhOTgyZDJjZTU1ZmU0YzhjNDQ3OGY3NmZlMjQzYjQzZDlmZjgzNDE3NTcmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0In0.dI_1_cGZqQVZ66pbnOcMQDHTeRSJcedxHvQY155_GCg), but I think it will benefit from this title.

* On that note, let's agree on the consistent spelling: my suggestion is Fine-tune and fine-tuning, as on [Hugging Face](https://huggingface.co/docs/transformers/en/training#fine-tune-a-pretrained-model)

@jeffmaury let's check with the InstructLab team - do you know who would be the best person to contact there for more info?

Best is to ask on one of their Slack channels

Folks, the InstructLab beta UI is out, and they have nice short descriptions for knowledge and skills contributions, shall we consider adding those to fields helper text?
🌟
Knowledge in InstructLab is represented by question and answer pairs that involve facts, data, or references.
🌟
Skills are performative. When you create a skill for the model, you are teaching it how to do something: "write me a song," "rearrange words in a sentence" or "summarize an email."
🌟

Adding a comment from Máirín Duffy on the InstructLab side:

for laptop training, full pipeline, right now - you can train knowledge or skills. not both in a training run.

reviewing that PR I noticed that we have a store to get the modelInfos and a method on the studioClient studioClient.getModelsInfo and the store delegates to the method, so I'm finding this confusing

We cannot subscribe to the studioClient.getModelsInfo. We could argue that we could get those value in the onMount, and there is little chance that the models will change on this page, but this is not the choice made in other forms, for reactivity purpose.

Like here, in the component NewInstructlabSession, we import both studioClient and the store but there are overlap between the two

The store is automatically updated every time the backend notify the webview through the Messages.MSG_NEW_MODELS_STATE  channel
https://github.com/containers/podman-desktop-extension-ai-lab/blob/c165dbbb689773b2c264dbbb311a9820357d7da2/packages/frontend/src/stores/modelsInfo.ts#L24
This is why we have two components here, they don't overlap has the store has an auto update logic based on event, that we do not have with a simple studioClient.getModelsInfo.
We import the studioClient to access to two method studioClient.openDialog (open the explorer to select a folder) and studioClient.openUrl to open in the browser the documentation url.
In AI Lab we do not declare global method like in Podman Desktop, where all the communication between the renderer and the preload are made through the window.* object.

This is why we have two components here, they don't overlap has the store has an auto update logic based on event, that we do not have with a simple studioClient.getModelsInfo.

From consumer side, you've two different equivalent as it's from the same source. Exposure is changing.
Someone onboarding into the project might think it's more convenient to import only the client and then call client.getModelsInfo() But then, why I should have access to something not being reactive ? Also you can use get(modelsInfo) to get the value for one time. (no subscribe/unsubscribe)
I'm also offtrack of this PR, I don't require any change on this PR about that, but probably need to have a follow-up issue
my point is more : it looks that there is something more 'internal' to grab data from the backend side but client should never consume this endpoint directly. And then there are methods exposed to the client that he can use.
While here, it's a mix of two. Because if you need to ask something like 'publicly'
And stores being deprecated over $state() it could be revisited at that point.

my point is more : it looks that there is something more 'internal' to grab data from the backend side but client should never consume this endpoint directly. And then there are methods exposed to the client that he can use.
While here, it's a mix of two. Because if you need to ask something like 'publicly'

I get your point, and totally understand it, there is clearly room for improvement, especially with the svelte 5 new requirements.

I'm also offtrack of this PR, I don't require any change on this PR about that, but probably need to have a follow-up issue

I agree, I opened https://github.com/containers/podman-desktop-extension-ai-lab/issues/1883 to continue this conversation, as I think this is a valid point, and I would welcome change to improve this system.
I want to precise however that this system has been design and put together the first days of the hackathons in January, and has not been touch much since.

Can we make the session name optional ?

Feel reasonable to require a name cc @MariaLeonova

Can we make the session name optional ?

Feel reasonable to require a name cc @MariaLeonova

@jeffmaury why do you suggest to make it optional?
The idea there was for people to be able to easily find their session in a list later. That's something that's done in the IBM Cloud flow, as well. I suppose it can be pre-filled with an automatically generated name, if we don't want users to have to type anything in?

That's something that's done in the IBM Cloud flow, as well.

Personal preference to have it mandatory

We don't require it for playgrounds and services

@jeffmaury ultimately it's up to you, as I don't have enough background info on user behaviour in playgrounds and services. Just as long as users are allowed to change the name there to something that makes sense to them - I think in this case it's important to give them control over their session list. Which seems to be the case for playgrounds, at least.

Yes I agree session name should stay

The +add button shouldn't be centered, if we are going with this pattern.

Updated :rocket:

@MariaLeonova: I have a couple of UX feedback.
When the box is not selected / no files added
It really feels that we are duplicating the labels. There is "Add Knowledge" title and below "Add Knowledge to use", that feels redundant to me.
I'd also callout that the "Add Knowledge to use" does not look like a button and that does not feel very intuitive.
When a file has been added
The description of what the section is about feels not at the right location. That should probably be before the action.

@MariaLeonova: I have a couple of UX feedback.
When the box is not selected / no files added It really feels that we are duplicating the labels. There is "Add Knowledge" title and below "Add Knowledge to use", that feels redundant to me. I'd also callout that the "Add Knowledge to use" does not look like a button and that does not feel very intuitive.

I agree! It used to say 'add another knowledge file' in the mock.

When a file has been added The description of what the section is about feels not at the right location. That should probably be before the action.

@axel7083 can you please add a screenshot of what it looks like when there are 2 files added already? We might want to shuffle things a bit - the addition of +add more knowledge/skills button has rearranged elements in the form.

@axel7083 I am not opposed to opening a follow-up issue for design improvement, so we don't have to go back and forth and can change all at once, if you'd rather merge this now.


Use recent version of code formatter

Upgrade black formatter to fix dependency issues. Details see #176

:tada: This issue has been resolved in version 5.3.0 :tada:
The release is available on:

v5.3.0
GitHub release

Your semantic-release bot :package::rocket:


initial noble migration

The link does not seem to work: https://bosh.io/docs/noble-migration

Found here: https://bosh.io/stemcells/bosh-aws-xen-hvm-ubuntu-noble

I think the doc needs to be listed here: https://github.com/cloudfoundry/docs-bosh/blob/master/mkdocs.yml
What would be the best section to include it then I would prepare a PR for it

@benjaminguttmann-avtq you are correct. i forget that part.


lncli: add extra confirmation dialogue to payinvoice+sendpayment

Currently there's no easy way for a user to verify the integrity of a payment request they're about to pay, unless one first uses the decodepayreq method each time before sending on the command line. It would be nice if this became the default, as since many users are new to the system, it's easy for a site to trick a user into paying a higher invoice. To avoid this, we should add a default mode where we always display the invoice to the user for confirmation before attempting to pay the invoice.
Steps To Completion

[ ] Modify payinvoice and sendpayment to always first call DecodePayReq with the payreq. The response of this should be displayed to the users w/ a confirmation dialogue w.r.t if we should proceed or not.

[ ] Add a --force (or -f) flag to both of these commands to skip the user verification

Looks like a good first issue, can I take this ?

I always decode first. This will save a step. Great!

I can take it myself as well, since it can save on a my application half of the time (of course only if isn't going to take it rawtxapp, I'm ok with FIFO policy :D )


revamp script ID_sets_of_duplicate_records.R

The size of ForC is getting too big for this code.
I was able to use it with the GROA data but I fear the memory limit will be reached next time we use it. (problem occurs at line 162)
At some point, I need to sit down and revamp it, but that will take a while.

I found a way to make the script faster so I don't think we will run into running time issues anymore.
The current situations is that every time this script is run, it will generate a pop-up window if it finds conflicts that are different than what was already in the data.
In that case, someone needs to go through each of those (usually a hand full unless a large number of records were added/modified) and decide if the new conflict is correct or if the old one was better (usually when it was manually edited).
My current way of doing that is to list the measurement.ID (separated by a comma) that belong to the same conflict in one of two different groups: 1. keep old conflict and 2. keep new conflict. If nobody goes through this, the old conflict is saved (will be flagged again next time).
Currently this list is in the R script but I could create a csv file instead.
One issue with that is if someone edits the conflicts of a group that is already listed under the "2. keep new conflict" group....
as the manual edit would be overwritten...
I think it is safe for me to delete what is under group "2. keep new conflict" once those are saved.... that way, if someone edits them, they will be flagged in the pop-up window and their measurement ID can be added to 1. keep old conflict....
I don't currently see a way to make that automatic. I may just add a note in the pop-up window to say that that list needs to be emptied once MEASUREMENT table is updated with new conflicts...
Sorry for this long comment, this is for me to keep track of my thoughts...

I think I can close this now. The next step is to resolve site duplicates.


Flutter iOS add2app always have Flutter.podspec with version as 1.0.0

Steps to Reproduce

Create a flutter module
Integrate it in an existing iOS app using option A from the list of options available here (https://flutter.dev/docs/development/add-to-app/ios/project-setup)
Run pod install, Flutter sdk installed always point to version 1.0.0

Flutter.podspec after running flutter pub get
Pod::Spec.new do |s|
s.name             = 'Flutter'
s.version          = '1.0.0'
s.summary          = 'High-performance, high-fidelity mobile apps.'
s.description      = <<-DESC
Flutter provides an easy and productive way to build and deploy high-performance mobile apps for Android and iOS.
DESC
s.homepage         = 'https://flutter.io'
s.license          = { :type => 'MIT' }
s.author           = { 'Flutter Dev Team' =><EMAIL_ADDRESS>}
s.source           = { :git => 'https://github.com/flutter/engine', :tag => s.version.to_s }
s.ios.deployment_target = '8.0'
s.vendored_frameworks = 'Flutter.framework'
end
Problem
How can I update the version of podspec to point to an updated Flutter sdk version (such as 1.20.x) to be downloaded while performing pod install
Flutter Doctor
[✓] Flutter (Channel stable, 1.20.3, on Mac OS X 10.15.7 19H15, locale en-AE)
• Flutter version 1.20.3 at /Users/murtuza.saify/Documents/tools/flutter
• Framework revision 216dee60c0 (10 months ago), 2020-09-01 12:24:47 -0700
• Engine revision d1bc06f032
• Dart version 2.9.2
[!] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
• Android SDK at /Users/murtuza.saify/Library/Android/sdk
• Platform android-30, build-tools 30.0.3
• Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
• Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)
✗ Android license status unknown.
Try re-installing or updating your Android SDK Manager.
See https://developer.android.com/studio/#downloads or visit https://flutter.dev/docs/get-started/install/macos#android-setup for detailed instructions.
[✓] Xcode - develop for iOS and macOS (Xcode 12.0)
• Xcode at /Users/murtuza.saify/Documents/tools/Xcode.app/Contents/Developer
• Xcode 12.0, Build version 12A7209
• CocoaPods version 1.10.0
[!] Android Studio (version 4.2)
• Android Studio at /Applications/Android Studio.app/Contents
✗ Flutter plugin not installed; this adds Flutter specific functionality.
✗ Dart plugin not installed; this adds Dart specific functionality.
• Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)
[✓] VS Code (version 1.57.1)
• VS Code at /Applications/Visual Studio Code.app/Contents
• Flutter extension version 3.23.0
[✓] Connected device (2 available)
• iPhone 11 Pro (mobile)     • CC78A4AE-D08D-4EA1-B404-A3B92D0EAFF2 • ios • com.apple.CoreSimulator.SimRuntime.iOS-14-0 (simulator)
• iPhone 11 Pro Max (mobile) • 201E5C37-7CAD-4FD6-922A-338AD950DC93 • ios • com.apple.CoreSimulator.SimRuntime.iOS-14-0 (simulator)

@chinmaygarde @xster not sure who to tag here for this. But will appreciate your support on this 🙏

Hi @murtuzasaify
This Podspec is generated when you execute flutter pub get in a module, this is not published, this is simply generated for the native app.
s.version          = '1.0.0' doesn't present any particular Flutter release, this is probably just a default version source

How can I update the version of podspec to point to an updated Flutter SDK version (such as 1.20.x) to be downloaded while performing pod install

Can you please explain why you wanna see the actual  Fluttter version in this podspec?  Provide some use cases
Thank you

Without additional information, we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact description of what happens, logs, and the output of flutter doctor -v.
All system setups can be slightly different, so it's always better to open new issues and reference related issues.


Rename and restructure tabs and content in righthand panel

Pull request 949 can be deleted. I messed up something because I wanted to use live reload so I forked @serapath's repo and deleted old one. Thought it will automagically update if I call repos and branches the same, but nope.
So, here is the first draft of right hand panel.

Icons are replaced,
we have a new tab for contract,
content of the tabs is a bit bigger and stretched and design is slightly tightened

TODO:

review the UI
decide how to nicely redesign it for real

Here are snapshot of @yann300 's comments from previous pull request


API Examples update: test.com -> example.com

The domain test.com is actually owned by a company. In accordance with https://tools.ietf.org/html/rfc2606#section-3 we should be using example.com instead as it's reserved for this purpose.

Thanks!


Create org not working

The create scratch org process has been stuck like this

According to the logs, there appear to be an issue with the worker starting:

This is my heroku configuration:

The worker crash is in https://github.com/SFDO-Tooling/Metecho/pull/2042/commits -- if you go to .heroku/start_worker.sh and comment out the "mkdir -p" and "ln -s" lines the worker no longer crashes.

Unfortunately at the moment I still can't create scratch orgs:
metecho.exceptions.SubcommandException: click.exceptions.UsageError: This project requires CumulusCI version 3.86.0 or later. To upgrade, please run this command: pip install --upgrade cumulusci
Perhaps this means my project is ahead of the latest buildpack in Heroku? Hopefully an update is coming soon?


Added podman as experimental driver

Thank you @medyagh! Added podman as an experimental driver.

/assign @savitharaghunathan
/assign @medyagh

@medyagh: Added missing bracket and now deploy preview looks better

Assigning @sftim since he is already engaged on this.
/assign @sftim

/sig cluster-lifecycle

@sftim


Boilerplate HTML for typesetting purposes

This morning @markboulton tweeted:

Anyone point me in the direction of a really good boilerplate html file (no classes, css, id’s or any additional html - just the elements) which shows most content elements (paragraphs, lists, Blockquotes, tables etc etc) for the purposes of typesetting? Something I can just grab

I suggested to have something like that on Wakamai Fondue, which received positive feedback.
Perhaps  @cbracco could allow for using https://github.com/cbracco/html5-test-page here?

Hey! It’s under MIT license so feel free to do with it what you will. It is based off the test file in the normalize.css project.
A small credit/backlink/mention would be cool, but not required. ✌🏻 😄


Open shift configuration does not proceed correctly

Hello.
When i install Openshift using Assistant Installer according to the github's oci-openshift procedure, the state changes to the "Installing Pending User Action" state after a certain period of time and does not proceed any further.
Do I need to do any additional settings?

Hi @kimsh92. Thanks for your inquiry.  Can you share a screenshot of what you're seeing?
Red Hat OpenShift on Oracle Cloud Infrastructure with Virtual Machines (VMs) is currently available for Developer Preview.
Note that Red Hat OpenShift on Oracle Cloud Infrastructure requires the use of Red Hat Enterprise Linux (RHEL) certified Oracle Cloud Infrastructure shapes (instance types) with yourOpen Shift subscription to deploy Red Hat OpenShift on Oracle Cloud Infrastructure.
Please complete this Red Hat OpenShift on Oracle Cloud Infrastructure Developer Preview Request Form to request access to deploy OpenShift on Oracle Cloud Infrastructure.

Version : 4.14 (Dev Preview)

and i can not access Request Form

There is an old issue and has no comments lately. Closing the issue.


sample Azure Functions to automate platform updates for maintenance window

"Automate platform updates for your maintenance window using Azure Functions"
Is sample code available?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: b69a7e2e-dfa7-1fcf-e2f6-781ecd0a7ccf
Version Independent ID: 4c6c22d7-0c67-47be-d241-de0d01c8603a
Content: Overview of Maintenance control for Azure virtual machines using the Azure portal - Azure Virtual Machines
Content Source: articles/virtual-machines/maintenance-control.md
Service: virtual-machines
GitHub Login: @cynthn
Microsoft Alias: cynthn

@kazshi
Thanks for the feedback! We are currently investigating and will get back on this.

@cynthn Can you please check and add your comments on this.

@kazshi Apologies in delayed response on this.
I got update from internal team on this, there is an effort going on for putting the code samples.
This should be available in couple of weeks.

@kazshi
Hope the provided information is helpful.
We will now close this issue. If there are further questions regarding this, please tag me in a comment. I will reopen it and we will continue the discussion.

FYI - Functions sample is here: https://github.com/Azure/azure-docs-powershell-samples/tree/master/maintenance-auto-scheduler and I'll be adding links into the docs as well.


Deprecated:  class_exists(): Passing null to parameter #1 ($class) of type string is deprecated

I'm getting this with PHP 8.2.x

Deprecated:  class_exists(): Passing null to parameter #1 ($class) of type string is deprecated in /var/www/html/site/modules/ProcessGraphQL/src/Utils.php on line 204
{"errors":[{"message":"Syntax Error: Unexpected \u003CEOF\u003E","locations":[{"line":1,"column":1}],"extensions":{"file":"\/var\/www\/html\/site\/modules\/ProcessGraphQL\/vendor\/webonyx\/graphql-php\/src\/Language\/Parser.php","line":433}}]}

I have added a suggest fix as PR: https://github.com/dadish/ProcessGraphQL/compare/main...tiefenb:ProcessGraphQL:patch-3


Setup appveyor for windows builds

Appveyor up and running. detektCheck not used for now, as detekt does now only work with unix like paths.


Replace HTTPBin with Kube Conformance Echoserver

Currently, the conformance tests use kube ingress conformance echoserver image for testing. https://github.com/envoyproxy/gateway/pull/558 requires the echo server to demonstrate HTTP routing. Instead of supporting multiple images, replace httpbin with the kube ingress conformance echoserver image in all examples (only Quickstart at the time of this writing).

xref tls passthrough guide: https://github.com/envoyproxy/gateway/pull/575

xref secure gateways guide: https://github.com/envoyproxy/gateway/pull/542

/assign


Update README URLs based on HTTP redirects

Created with https://github.com/dkhamsing/frankenstein
GitHub Corrected URLs

Was
Now

https://github.com/MobileChromeApps/zip
https://github.com/MobileChromeApps/cordova-plugin-zip

https://github.com/brodysoft/Cordova-SQLitePlugin
https://github.com/litehelpers/Cordova-sqlite-storage

https://github.com/driftyco/ionic-plugins-keyboard
https://github.com/driftyco/ionic-plugin-keyboard

HTTPS Corrected URLs

Was
Now

http://img.shields.io/badge/bower-ngCordova-FFCC2F.svg?style=flat
https://img.shields.io/badge/bower-ngCordova-FFCC2F.svg?style=flat

http://img.shields.io/travis/driftyco/ng-cordova.svg?style=flat
https://img.shields.io/travis/driftyco/ng-cordova.svg?style=flat

Other Corrected URLs

Was
Now

http://ngcordova.com/docs
http://ngcordova.com/docs/

This should be PR'd into dev-next; not master.

Ok


Improve coverage of Speculation Rules custom metrics

In performance.js we're capturing the speculation rules themselves. This tells us how the links should be prefetched/prerendered, but not which ones actually are.
Add new metadata to the custom metric that captures which requests are actually speculatively loaded. We can look for the Sec-Purpose header and log which requests are prefetched or prerendered.
The custom metric also currently only looks for script tags with the Speculation Rules JSON. But it's also possible to configure the rules via the Speculation-Rules header. The custom metric should check there as well as the script tags.

In performance.js we're capturing the speculation rules themselves. This tells us how the links should be prefetched/prerendered, but not which ones actually are.
Add new metadata to the custom metric that captures which requests are actually speculatively loaded. We can look for the Sec-Purpose header and log which requests are prefetched or prerendered.

This doesn't make sense in a lab environment for the following reasons:

Speculations only apply to navigations not sub-resources so will only affect future navigations and not this one.
There is no API for the calling page to measure if a speculation is triggered (though I thought I recalled someone asking for this but can't find it now).
Speculations are disabled in automated testing envs, and attempts to get them to run on WebPageTest have failed, probably for that reason.

The custom metric also currently only looks for script tags with the Speculation Rules JSON. But it's also possible to configure the rules via the Speculation-Rules header. The custom metric should check there as well as the script tags.

This is one we should do!

Wait so if I'm on page A and the rules say to prerender page B, there will be nothing in the network log for page A to say that page B was prerendered? I would have expected to see something in $WPT_REQUESTS with the request to prerender page B.

Well 1) it's not supported on WebPageTest.
Even if it was I'm not sure if it would be observable as it happens in a separate rendering process (you can't see it in the Network tab of DevTools for example - though you can switch to the Network tab of that rendering process in DevTools). WebPageTest may see this as think it looks at a lower level but, as I say, prerender is not supported.
Prefetch does show as a request for this page, so may show however so could monitor those if they work (not sure if they do if also restricted by the same restrictions as prerender).
Also <link rel=prefetch> and <link rel=prerender also prefetch. Just checked and looks like they use the older Purpose: prefetch but not Sec-Purpose: prefetch.

Ok so if prefetches show up, and older link[rel] speculations also show up, then it seems like we should have enough data to collect?
Testing prerendering in a lab environment seems like a valid use case, so maybe it's safe to assume that it'll be supported in WPT at some point?
Maybe @pmeenan can clarify how observable those prerenders would be in $WPT_REQUESTS

Testing prerendering in a lab environment seems like a valid use case, so maybe it's safe to assume that it'll be supported in WPT at some point?

I think it was in the past (I have a video here I generated with it), but then it caused problems in puppeteer and so we blocked it (bug) and I suspect that's what turned it off for WebPageTest.
I'm also not sure we should be spending our resource on this during the crawl if it ever is re-enabled. Like if a page prerenders 10 pages, then we've 10x-ed our crawl time for that page! Well less as should reuse cached resources which are common across the pages but still...

So what I was originally hoping for with this data is to see how many pages would be prerendered if a real user visited the page. The new ruleset options have patterns that obscure the number of affected pages. WDYT if we faked it by trying to apply the rules to the known URLs on the page? For example, for /* except /logout would it work if we count the distinct number of same-origin URLs on the page and omit the logout page?

You'd also need to take into account the eagerness setting. And the chrome limits.

Exactly so the heuristics can only get us an upper limit, which is ok right?

Yeah but maybe a waaaaaay upper limit so not sure what it tells us?
For example, if wikipedia implemented this with document rules and set eagerness to moderate or conservative and you set it able to prerender ALL links on the page then it could say we'll prerender up to 4,816 links on Obama's wikipedia page:
Array.from(document.querySelectorAll('a')).filter(a => a.href.startsWith("https://en.wikipedia.org"));`)

But it won't do that unless a user hovers over every, single link on that page, which is pretty damn unlikely.
So not sure what this is telling us?

Right, we'll still have the context of the eagerness setting, so this will tell us how many links are at best eligible for preloading. If you're saying this data would NEVER be useful, sure, let's skip it. I'm also open to other suggestions for ways to measure API usage.

Picking this back up after discussing offline. There would be value in knowing if a page eagerly prerenders the maximum number of URLs (10 in Chrome). Assigning to myself to prototype a parser to count the number of eligible URLs according to the rules.


Issue launching Roboclaw

I'm having issues when trying to run the robolcaw node. I'm using this roboclaw_ros package and haven't changed any of the source code. I get the follwing error:
[FATAL] [1592845100.478977]: Could not connect to Roboclaw
Traceback (most recent call last):
File "/opt/ros/melodic/lib/python2.7/dist-packages/rospy/core.py", line 568, in signal_shutdown
h()
File "/home/nano/catkin_ws/src/roboclaw_ros/roboclaw_node/nodes/roboclaw_pyscript.py", line 318, in shutdown
roboclaw.ForwardM1(self.address, 0)
File "/home/nano/catkin_ws/src/roboclaw_ros/roboclaw_node/src/roboclaw_driver/roboclaw_driver.py", line 700, in ForwardM1
return _write1(address, Cmd.M1FORWARD, val)
File "/home/nano/catkin_ws/src/roboclaw_ros/roboclaw_node/src/roboclaw_driver/roboclaw_driver.py", line 335, in _write1
_sendcommand(address, cmd)
File "/home/nano/catkin_ws/src/roboclaw_ros/roboclaw_node/src/roboclaw_driver/roboclaw_driver.py", line 126, in _sendcommand
port.write(chr(address))
NameError: global name 'port' is not defined
[WARN] [1592845100.646772]: Problem getting roboclaw version
Traceback (most recent call last):
File "/home/nano/catkin_ws/src/roboclaw_ros/roboclaw_node/nodes/roboclaw_pyscript.py", line 332, in
node = Node()
File "/home/nano/catkin_ws/src/roboclaw_ros/roboclaw_node/nodes/roboclaw_pyscript.py", line 174, in init
if not version[0]:
UnboundLocalError: local variable 'version' referenced before assignment
[roboclaw_node-8] process has died [pid 8459, exit code 1, cmd /home/nano/catkin_ws/src/roboclaw_ros/roboclaw_node/nodes/roboclaw_pyscript.py __name:=roboclaw_node __log:=/home/nano/.ros/log/8e2cd43a-b4a9-11ea-93ad-00044be65023/roboclaw_node-8.log].
log file: /home/nano/.ros/log/8e2cd43a-b4a9-11ea-93ad-00044be65023/roboclaw_node-8*.log
I am able to see the driver when I type the command usb-devices into my terminal window.

Any input would be helpful in debugging this issue.

I ran project with root level privileges (sudo), and everything is working. The issue was ROS didn't have access rights to some usb ports.


"This computer does not have an Intel CPU" but I have an Intel Core i7-9750H

All is in the title
Here is a screenshot of ra1nstorm installer and About Settings page

Will be fixed in latest update


Add C++ wrapper class in minisketch.h header

This provides a safer API for use within C++ code. It's just a headers-only wrapper around the C API, so it does not introduce any further binary compatibility concerns.

Great, thanks!

Doesn't compile for me:
In file included from src/test-exhaust.cpp:7:
src/../include/minisketch.h:272:9: error: missing 'typename' prior to dependent type name 'std::enable_if<std::is_convertible<typename
      std::remove_pointer<decltype(obj.data())>::type (*)[], const unsigned char (*)[]>::value && std::is_convertible<decltype(obj.size()),
      std::size_t>::value, std::nullptr_t>::type'
        std::enable_if<
        ^~~~~~~~~~~~~~~
        typename
1 error generated.

@naumenkogs Oops, forgot to test that function. Fixed.

How to safely use it? I remember minisketch_create could return nullptr in some cases.

I added an operator bool.
So you can use
Minisketch minisketch = Minisketch::CreateFP(...);
if (!minisketch) { ... }

I'm not an expert on C++ wrappers, but I've been using this new code for weeks now in Erlay, and the API seems correct.
Also, I have reviewed the code.
ACK.

Light review ACK a570de26152a0ec224c38f8157f796030cb0bd20

New comments are clearer to me. Thanks!


Issue:  [FEATURE] move connector config to a separate file #1109

Type of Change

[ ] Bugfix
[x] New feature
[ ] Enhancement
[ ] Refactoring
[ ] Dependency updates
[ ] Documentation
[ ] CI/CD

Description

Additional Changes

[ ] This PR modifies the API contract
[ ] This PR modifies the database schema
[x] This PR modifies application configuration/environment variables

Motivation and Context

How did you test it?

Checklist

[x] I formatted the code cargo +nightly fmt --all
[x] I addressed lints thrown by cargo clippy
[x] I reviewed submitted code
[x] I added unit tests for my changes where possible
[x] I added a CHANGELOG entry if applicable

Please take care making the same changes in docker_compose/loadtest configs too.

Hi @MehmetTerc, please let us know if you will be able to make these changes. Thanks!

Hey @MehmetTerc, can you let us know if you will be working on this? Thanks!


Allow creating and adding pygments.styles

Instructions on how to create styles in python are given here: https://pygments.org/docs/styledevelopment/#creating-own-styles
Loading styles into pygments requires tinkering with setuptools, see https://pygments.org/docs/plugins/
I managed to get it to work in python.
It remains to figure out how to load it into PyCall, since the new styles have to be installed as a plugin using setuptools.
Open questions:

Updating a style file requires to rerun the installation of the plugin. Can we do this from within the same julia session using PyCall or do we have to restart the REPL everytime we change the style?
Can we provide an environment variable and/or default directory in which setuptools can look for style files?

There is a discussion going on about alternative ways to register custom lexers or styles in pygments: https://github.com/pygments/pygments/issues/1096


[Docs] GitHub workflow guide - Page not found

Current State
The "guide to a standard GitHub workflow" link under Quickstart on this page leads to a 404 error.

Desired State
The link should redirect to: https://github.com/layer5io/docs/blob/master/CONTRIBUTING-gitflow.md

Contributor Guide and Resources

📚 Instructions for contributing to documentation

Visit the Layer5 documentation site and source

🎨 Access wireframes and designs for the Layer5 site in Figma (open invite)
🙋🏾🙋🏼 Questions? Visit the Layer5 Discussion Forum and join the Layer5 Community Slack

I would like to solve this issue.Please assign .

Go on @YashrajKupekar17  🚀

@YashrajKupekar17 any progress?

yeah I have raised a new pr according to the changes discussed in the meeting .
https://github.com/layer5io/docs/pull/347
this is the link to the pr. check and please merge it


Jamf integration requires password for SSO user?

Does Jamf really require a password for a user authenticating via AzureAD SAML SSO? Doesn't that defeat most of the purpose of using SSO to begin with?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 85cbe739-d436-c414-8ad0-1197ff2af011
Version Independent ID: b0d675ef-1efd-eec6-cf82-a39b07472b98
Content: Tutorial: Azure Active Directory integration with Jamf Pro
Content Source: articles/active-directory/saas-apps/jamfprosamlconnector-tutorial.md
Service: active-directory
GitHub Login: @jeevansd
Microsoft Alias: jeedes

@007 Yes you are right. Once you enable the SSO on JAMF you dont need the password for it. What we have explained in this article is we can auto configure SAML SSO in JAMF and for that you need to login to JAMF using the admin credentials. Also make sure that you install our My Apps browser plugin before performing the OneClickSSO. This will help you configure the SSO in just a minute without even reading the doc and understanding on how to configure the SSO in JAMF.

#please-close

That's... ridiculous. Thanks for confirming details, will follow up with Jamf to try and fix.


number_test_62

Surelog issue: https://github.com/chipsalliance/Surelog/issues/2555

Issue fixed.


how to use this model with whisper.cpp

can u share which parameter to set to use this parameter in whisper.cpp.
is it the n_audio_ctx paramter with 1500 referring to 1.5 seconds. Would have been great if there was an android sample.
// medium
// hparams: {
// 'n_mels': 80,
// 'n_vocab': 51864,
// 'n_audio_ctx': 1500,
// 'n_audio_state': 1024,
// 'n_audio_head': 16,
// 'n_audio_layer': 24,
// 'n_text_ctx': 448,
// 'n_text_state': 1024,
// 'n_text_head': 16,
// 'n_text_layer': 24
// }
//
// default hparams (Whisper tiny)
struct whisper_hparams {
int32_t n_vocab       = 51864;
int32_t n_audio_ctx   = 1500;
int32_t n_audio_state = 384;

Thanks so much for the quick answer. i am curious. did u try distilled models. Specifically with ggml other than using quantized models like u shared are there other optimization gains  one can get easily get to use whisper models for real time streaming usecase


How to uninstall esp-idf? (IDFGH-9768)

Installing is easy enough by just running the install.sh script. But how do you uninstall it? I may have messed up the installation, and I want to revert everything and retry from scratch.
It looks like the script primarily installs a bunch of Python packages and copies files to various directories, but it's difficult to figure out which ones without going down a deep rabbit hole of nested scripts :)

Remove $HOME/.espressif and you should be able to reinstall from scratch.

Yes, everything goes by default to the .espressif folder.
This page contains the official documentation to the installer script: https://docs.espressif.com/projects/esp-idf/en/latest/esp32/api-guides/tools/idf-tools.html

Thanks for reporting, feel free to reopen.

We plan to review and update the documentation. I re-open it as a reminder.


ParameterValidation error - IAM

Getting errors - ParamValidationError: Parameter validation failed:
Invalid length for parameter PolicyArn, value: 8, valid range: 20-inf.
I am trying to trigger a C7N policy on CloudTrail Event CreatePolicy. I see there is no shortcut for the event and thus doing this:
events:
- source: iam.amazonaws.com
event: CreatePolicy
ids: "requestParameters.policyName"
Am i missing something or is it C7N error?

there's not enough context here to understand what the issue is, whats the policy and what's the full traceback.

it looks like the api is expecting an arn, and your sending it the policy name.

Here is the policy:
policies:

name: IAM-Policy-Allow-All-Check
description: |
Check if IAM policy resource(s) have allow-all IAM policy statement block
resource: iam-policy
mode:
role: arn:aws:iam::{account_id}:role/CloudCustodianExecutionRole
type: cloudtrail
events:
- source: iam.amazonaws.com
event: CreatePolicy
ids: "requestParameters.policyName"
filters:

type: value
key: PolicyName
value: AdministratorAccess
op: ne
type: has-allow-all
actions:
type: delete
Traceback:
Parameter validation failed:
Invalid length for parameter PolicyArn, value: 15, valid range: 20-inf: ParamValidationError
Traceback (most recent call last):
File "/var/task/custodian_policy.py", line 4, in run
return handler.dispatch_event(event, context)
File "/var/task/c7n/handler.py", line 91, in dispatch_event
p.push(event, context)
File "/var/task/c7n/policy.py", line 834, in push
return mode.run(event, lambda_ctx)
File "/var/task/c7n/policy.py", line 442, in run
resources = self.resolve_resources(event)
File "/var/task/c7n/policy.py", line 419, in resolve_resources
resources = self.policy.resource_manager.get_resources(resource_ids)
File "/var/task/c7n/query.py", line 474, in get_resources
resources = self.source.get_resources(ids)
File "/var/task/c7n/resources/iam.py", line 164, in get_resources
results.append(client.get_policy(PolicyArn=r)['Policy'])
File "/var/runtime/botocore/client.py", line 314, in _api_call
return self._make_api_call(operation_name, kwargs)
File "/var/runtime/botocore/client.py", line 586, in _make_api_call
api_params, operation_model, context=request_context)
File "/var/runtime/botocore/client.py", line 621, in _convert_to_request_dict
api_params, operation_model)
File "/var/runtime/botocore/validate.py", line 291, in serialize_to_request
raise ParamValidationError(report=report.generate_report())
ParamValidationError: Parameter validation failed:
Invalid length for parameter PolicyArn, value: 15, valid range: 20-inf

I tried using ids: "requestParameters.arn instead but then the trigger does not work.
Traceback:
[INFO]	2018-10-17T19:39:01.743Z	4bb0d8c6-d244-11e8-9b88-a96c7c9d049c	Found resource ids: []
[WARNING]	2018-10-17T19:39:01.743Z	4bb0d8c6-d244-11e8-9b88-a96c7c9d049c	Could not find resource ids

arn is in the responseElements not the requestParameters.

you have to look at the event in the log and specify the correct path to the arn for the ids expression. the one you've done here is not correct, the arn is contained within a policy mapping in the response.

It is working now. Thanks for the help


parse() string into BigUint instead of using BigUint::from_str()

This addresses the issue referenced in https://github.com/Hoverbear/rust-rosetta/pull/392#discussion_r30707638
Using parse() looks more pretty to me as well. I have also noticed that there is no need to specify the type for n, e and d - it gets inferred when the values are passed to mod_exp(BigUint, BigUint, BigUint) or returned as a BigUint tuple (in the tests).

I think parse() as well here.

Looks good to me! Thanks!


GCE "Server misbehabing"

I've been trying to solve this issue for hours now, I am trying to request a cert for 2 domains. The services etc. for the two are set up identically, however one always fails:
http://webhooks.xx.xx.io/.well-known/acme-challenge/_selftest works perfectly externally. Any ideas?
time="2017-03-22T00:31:15Z" level=info msg="authorization successful" context=acme domain=api.xx.xx.io
time="2017-03-22T00:32:34Z" level=warning msg="authorization failed after 1m0s: reachabily test failed: Get http://webhooks.xx.xx.io/.well-known/acme-challenge/_selftest: dial tcp: lookup webhooks.xx.xx.io on <IP_ADDRESS>:53: server misbehaving" context=acme domain=webhooks.xx.xx.io
time="2017-03-22T00:32:34Z" level=warning msg="authorization failed for some domains" context=acme failed_domains=[webhooks.xx.xx.io]

@jaydensmith looks like the kube-lego cannot talk with the dns server (<IP_ADDRESS>).  Try to attach to the kube-lego shell and resolve the dns. Ex:
$host webhooks.xx.xx.io <IP_ADDRESS>

I came back a day or 2 later and the issue had resolved itself... oh well 😄


Bind filter to ng-keypress

Currently filter is bind to ng-change event. I have tried to change it to ng-keypress, The reason behind this, i don't want to call filter for each and every character change, instead bind to  ng-keypress (Enter key) will call filter only after complete input (String) and press Enter.
i tried some thing like this:

Am setting value to ng-model="setString" in controller, the filter working fine, But after clearing ng-model="setString" value tree will not reset to original tree

Would you mind providing an example to demonstrate the issue and your setup? There's a jsbin template in the README: https://github.com/iVantage/angular-ivh-treeview#reporting-issues-and-getting-help.

Closing but feel free to reopen with a demo.


Unnecessary warnings in 0.9.9-rc4 upgrade?

I got the following warnings at the very end of the upgrade logs that would appear fairly serious when upgrading to chocolatey.0.9.9-rc4
Ensuring 'C:\ProgramData\chocolatey\lib\chocolatey' exists.
WARNING: Was not able to remove 'C:\ProgramData\chocolatey\bin\choco.exe.old'. This may cause errors.
WARNING: Was not able to rename 'C:\ProgramData\chocolatey\bin\choco.exe' to 'C:\ProgramData\chocolatey\bin\choco.exe.old'.
Chocolatey (choco.exe) is now ready.
You can call choco from anywhere, command line or powershell by typing choco.
Run choco /? for a list of functions.
You may need to shut down and restart powershell and/or consoles
 first prior to using choco.
Finished installing 'chocolatey' and dependencies - if errors not shown in console, none detected. Check log for errors if unsure.
The batch file cannot be found.

However the date modified timestamps for choco.exe (09 Feb 2015 10:04) and choco.exe.old (13 Jul 2015 19:03) appear reasonable.
I might understand the first warning Was not able to remove 'C:\ProgramData\chocolatey\bin\choco.exe.old' as I don't think I'd done an update before so it never existed. But the second one is more worrying.
I was running my dos prompt as Administrator and the file permissions on choco.exe.old give full permissions to Administrator.

Wrong repository. https://github.com/chocolatey/choco

But also, warnings are a good lead to what may later be an error. Warnings are usually information sake and may require action. Errors mean bad things.


Needs build instructions

Title says it all. Document git submodule installation and build/install process for the library.
Note: It's unnecessary to install/use SWIG since the bindings should be regenerated any time the git submodule is updated.

@Gadgetoid, if I install via pip (pip install rpi_ws281x), do I need to build the submodule with scons?

Since it’s necessary to use sudo to use the library, is it also necessary to use sudo pip install rpi_ws281x when installing?


[rl-reuse_tool-3] Violation against OSS Rules of Play

A violation against the OSS Rules of Play has been detected.
Rule ID: rop-reuse_tool-3
Explanation: Is it registered in REUSE? No
Find more information at: https://wiki.wdf.sap.corp/wiki/display/ospodocs/Repository+Linter+Findings

Registration is done, thanks!


Connection handshake fails with ERR unknown command CLIENT, with args beginning with: SETINFO lib-name

Bug Report

Current Behavior
The connection does not work and fails to connect to Google Cloud Redis (Memory Store) v7.2 with the following error CLIENT 'SETINFO'. The Client Setinfo not supported by GCP Redis

Stack trace
Exception in thread "main" io.lettuce.core.RedisConnectionException: Unable to connect to IP_ADDRESS/<unresolved>:6379
	at io.lettuce.core.RedisConnectionException.create(RedisConnectionException.java:78)
	at io.lettuce.core.RedisConnectionException.create(RedisConnectionException.java:56)
	at io.lettuce.core.AbstractRedisClient.getConnection(AbstractRedisClient.java:350)
	at io.lettuce.core.RedisClient.connect(RedisClient.java:215)
	at io.lettuce.core.RedisClient.connect(RedisClient.java:200)
	at com.wayfair.smart.ds587.demo.bt.graphql.RedisTest.main(RedisTest.java:29)
Caused by: io.lettuce.core.RedisCommandExecutionException: ERR unknown command 'CLIENT', with args beginning with: 'SETINFO' 'lib-name' 'Lettuce'
	at io.lettuce.core.internal.ExceptionFactory.createExecutionException(ExceptionFactory.java:147)
	at io.lettuce.core.internal.ExceptionFactory.createExecutionException(ExceptionFactory.java:116)
	at io.lettuce.core.protocol.AsyncCommand.completeResult(AsyncCommand.java:120)
	at io.lettuce.core.protocol.AsyncCommand.complete(AsyncCommand.java:111)
	at io.lettuce.core.protocol.CommandWrapper.complete(CommandWrapper.java:63)
	at io.lettuce.core.protocol.CommandHandler.complete(CommandHandler.java:745)
	at io.lettuce.core.protocol.CommandHandler.decode(CommandHandler.java:680)
	at io.lettuce.core.protocol.CommandHandler.channelRead(CommandHandler.java:597)
	at io.netty.channel.AbstractChannelHandlerContext.invokeChannelRead(AbstractChannelHandlerContext.java:442)
	at io.netty.channel.AbstractChannelHandlerContext.invokeChannelRead(AbstractChannelHandlerContext.java:420)
	at io.netty.channel.AbstractChannelHandlerContext.fireChannelRead(AbstractChannelHandlerContext.java:412)
	at io.netty.channel.DefaultChannelPipeline$HeadContext.channelRead(DefaultChannelPipeline.java:1410)
	at io.netty.channel.AbstractChannelHandlerContext.invokeChannelRead(AbstractChannelHandlerContext.java:440)
	at io.netty.channel.AbstractChannelHandlerContext.invokeChannelRead(AbstractChannelHandlerContext.java:420)
	at io.netty.channel.DefaultChannelPipeline.fireChannelRead(DefaultChannelPipeline.java:919)
	at io.netty.channel.epoll.AbstractEpollStreamChannel$EpollStreamUnsafe.epollInReady(AbstractEpollStreamChannel.java:800)
	at io.netty.channel.epoll.EpollEventLoop.processReady(EpollEventLoop.java:509)
	at io.netty.channel.epoll.EpollEventLoop.run(EpollEventLoop.java:407)
	at io.netty.util.concurrent.SingleThreadEventExecutor$4.run(SingleThreadEventExecutor.java:997)
	at io.netty.util.internal.ThreadExecutorMap$2.run(ThreadExecutorMap.java:74)
	at io.netty.util.concurrent.FastThreadLocalRunnable.run(FastThreadLocalRunnable.java:30)
	at java.base/java.lang.Thread.run(Unknown Source)

Input Code

Input Code
// your code here;

Expected behavior/code

Environment

Lettuce version(s): 6.3.2.RELEASE
Redis version: Google Cloud Redis (Memory Store) v7.2

Possible Solution

Additional context

Have you tried to disable the Client Info via RedisURI.setLibraryName(null)/RedisURI.setLibraryVersion(…)?
The issue with Redis derivatives is that they pretend a specific version that we use to control whether certain functionality is available while the derivative server software doesn't implement the Redis OSS functionality we're looking for.

Hi Team,
We are currently facing the same compatibility issue with Lettuce 6.3.2.RELEASE when connecting to Google Cloud Redis (Memory Store) v7.2, an issue that does not occur with the earlier v7.0. We are using Lettuce as a transitive dependency through org.springframework.boot:spring-boot-starter-data-redis:3.2.5.
Could you please provide any advice on potential solutions or workarounds for this issue? Additionally, any information on upcoming fixes or patches would be highly appreciated as it would help us plan our development and maintenance activities better.

Hi Team,
We are currently facing the same compatibility issue with Lettuce 6.3.2.RELEASE when connecting to Google Cloud Redis (Memory Store) v7.2, an issue that does not occur with the earlier v7.0. We are using Lettuce as a transitive dependency through org.springframework.boot:spring-boot-starter-data-redis:3.2.5.
Could you please provide any advice on potential solutions or workarounds for this issue? Additionally, any information on upcoming fixes or patches would be highly appreciated as it would help us plan our development and maintenance activities better.

I think this is what the reference ticket is about https://github.com/redis/lettuce/pull/2823

Folks,
My bad. You can unblock yourself for now if you manually disable the reporting of the client name and version:
RedisURI redisUri = RedisURI.Builder.redis("my.redis.instance.com", 6379)
   .withLibraryName("")
   .withLibraryVersion("")
   .build();
RedisClient redisClient = RedisClient.create(redisUri);

This should unblock you until 7.x is released and you are able to consume the solution in #2823

Do you have a hint how to apply the workaround when using Spring Data?

Do you have a hint how to apply the workaround if using Spring Data?

I assume you are not able to bump the version of the Lettuce driver to 6.4.0.RELEASE?
Have you tried the solution from https://github.com/spring-projects/spring-data-redis/issues/2938#issuecomment-2221289568 ?

First, thanks for the hints.

I assume you are not able to bump the version of the Lettuce driver to 6.4.0.RELEASE?

I could, but is that compatible with Spring Boot 3.3? I didn't dare to just update it.

Have you tried the solution from spring-projects/spring-data-redis#2938 (comment) ?

No, I already looked into that, but that is too much copy and paste for me. Instead of doing this, I prefer to wait for Spring Boot 3.4 before upgrading our Redis to 7.

I could, but is that compatible with Spring Boot 3.3? I didn't dare to just update it.

My bad, I thought I saw a release of spring-data-redis consuming Lettuce 6.4.x
Seems it is still in the works.

No, I already looked into that, but that is too much copy and paste for me. Instead of doing this, I prefer to wait for Spring Boot 3.4 before upgrading our Redis to 7.

This is reasonable if you have the luxury to wait for the fix.


no response when deleting gataway in gataway navigate

sometime, the gateway we don't need and want to remove ,then click the delete button,but it's unable to remove


⚠ Bofrost.de has degraded performance

In 549f63b, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1303 ms

Resolved: Bofrost.de performance has improved in 8eca84d after 10 minutes.


Reference automatic link wiring does not work correctly

When a link is generated to access other resources based on a foreign key relationship in cattle if there are multiple references to the same table it will fail.
EX:
accounts has a link for projectMembers and the projectMembers table has two columns account_id and project_id
Both columns are foreign keys to the account table. This will make the collection always 404. A possible fix is to have a link for each resouce/column.

Not sure what the consequence of this is but you should at least look.


Add some missing dependencies

Here are a couple of things I had to add to get this to make locally.

Whoops! I see that brew install fswatch is listed elsewhere, but master is still missing a go dep.

Thanks @danluu :+1:
Ooh, I like that you force pushed. That's a +1 in my books, but a -1 in @icco's books.


use DateTime all the way from the database

See discussion https://discord.com/channels/687207715902193673/687235481154617364/877952104751304735
Replaces https://github.com/backstage/backstage/pull/6900

⭐


Testing CI

Codecov Report

Merging #1 (8dfea7a) into master (90e51c4) will decrease coverage by 0.03%.
The diff coverage is 0.00%.

@@            Coverage Diff            @@
##           master      #1      +/-   ##
=========================================
- Coverage    0.25%   0.21%   -0.04%
=========================================
  Files           9       8       -1
  Lines        1180    1390     +210
  Branches        0     218     +218
=========================================
  Hits            3       3
- Misses       1177    1387     +210

Impacted Files
Coverage Δ

fastai_minima/callback/core.py
0.00% <0.00%> (ø)

fastai_minima/callback/training.py
0.00% <0.00%> (ø)

fastai_minima/callback/training_utils.py
0.00% <0.00%> (ø)

fastai_minima/learner.py
0.00% <0.00%> (ø)

fastai_minima/metrics.py
0.00% <0.00%> (ø)

fastai_minima/optimizer.py
0.00% <0.00%> (ø)

fastai_minima/utils.py
0.00% <0.00%> (ø)

... and 3 more

Continue to review full report at Codecov.

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


Open source deconvolution engine

We need to provide an alternative open source deconvolution option for places that don’t have access to microvolution
It could possibly use the same deconvolution GUI we have now, but allow the user to select which deconvolution engine to use?
This would need to be done before setting deconvolution parameters as some microvolution features may be disabled such as

Blind deconvolutioon
Noise Suppression
Vectorial PSF
???

https://github.com/scopetools/cudaDecon

https://github.com/tlambert03/pycudadecon

https://www.sciencedirect.com/science/article/pii/S1046202316305096

https://www.researchgate.net/publication/312099578_DeconvolutionLab2_An_Open-Source_Software_for_Deconvolution_Microscopy

https://www.degruyter.com/document/doi/10.1515/bams-2020-0027/html

https://github.com/clEsperanto/pyclesperanto_prototype/blob/master/demo/transforms/deskew.ipynb
Deskew by Robert Haase

has this been thoroughly tested?

Nick/James to test cudadecon
Make user selectable: eg. Microvolution, cudadecon, Huygens
Make default only admin settable


I would like to add ThapaTechnical Youtube channel link

Site URL
https://www.youtube.com/@ThapaTechnical
Description of the above provided site
This channel is dedicated to the world of website development, offering a wide range of content related to technical aspects, tips and tricks, design principles, and programming. Whether you're a beginner looking to learn the basics or an experienced developer seeking advanced techniques, this channel aims to provide valuable insights and knowledge.
The channel covers various topics within website development, starting with the fundamentals of HTML, CSS, and JavaScript. It offers tutorials and explanations on how to create web pages, style them, and add interactivity using JavaScript. These videos can help you build a solid foundation in web development.
Programming videos on this channel cover different languages and frameworks commonly used in web development, such as Javascript, React, and more. These videos aim to help you understand the core concepts of each language and demonstrate how they can be used to build dynamic and interactive web applications.
Please assign me this issue under GSSOC 23 @rupali-codes
Category
Youtube
Sub Category
Web Development
Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I am willing to work on this issue (optional)
[X] I am a GSSoC'23 contributor

@Rahul-Bhati Sure, but please use a short & clear description. Describing what they exactly teach :))


Draft metrics/high level goals blog

@lpgoldstein I drafted my sections , I haven't read through it yet but feel free to add your sections here https://doimspp.sharepoint.com/:w:/r/sites/NRRD/Shared Documents/General/Blog/In Progress/metrics blog.docx?d=wd52fbdce62b844a0a0331b6ea6fdc9b4&csf=1&web=1&e=MX2RbR

@cthomasONRR @eelzi-ONRR please review the drafted blog when you have time. Thanks https://doimspp.sharepoint.com/:w:/r/sites/NRRD/_layouts/15/Doc.aspx?sourcedoc={D52FBDCE-62B8-44A0-A033-1B6EA6FDC9B4}&file=metrics blog.docx&action=default&mobileredirect=true&DefaultItemOpen=1&login_hint=faiedm%40mms.gov&ct=1703191871091&wdOrigin=OFFICECOM-WEB.START.EDGEWORTH&cid=768e56ce-d9c0-4b7e-8924-dbeacdecb9fa&wdPreviousSessionSrc=HarmonyWeb&wdPreviousSession=bf46878a-d386-4cde-b9d7-fa02ab0aef5b

@Maroyafaied Could you share the folder location? The link is trying to sign me in as you - never seen a sharepoint link do that before.

@cthomasONRR

@Maroyafaied Done reviewing. I had a few minor typo corrections and some comments

@Maroyafaied I'm done reviewing. Looks great!

@Maroyafaied I incorporated Erin's comments and rebuilt


chore: added script for quarkus ecosystem build

Adds missing script for Quarkus ecosystem CI build

To be merged after #113 and #115


Compact logging mode incorrectly groups log messages

Hi there,
this is a screenshot of log output from my application:

As you can see, the various Starting.../Finished. messages have different values for the middleware key.
Here's the same output in compact mode:

Suddenly all the messages are grouped under the first middleware.
I've reviewed the slog/slog-term code to figure out how this can be and found this function that looks dubious to me: https://github.com/slog-rs/slog/blob/26d13bf2ace81c919f7b5280e6c8f5a506c84537/src/lib.rs#L659
The function returns the id for an OwnedKeyValueList, using the address of self as the id. Although I haven't confirmed it, I believe this is the source of my problem. Even if not, I believe it to be wrong anyway. The function ignores the fact that an OwnedKeyValueList could be dropped, and a new created in its place.
I believe this is what's happening in my case. I'm creating a new Logger with some additional data, use it to log some messages, then it falls off the stack and I create the next one. Since nothing really happens between the dropping of a Logger and the creation of the next one, it makes sense that they would all end up with the same address on the stack, causing all the OwnedKeyValueLists to have the same id.

Thanks for the report. I think you're right on your diagnosis. The way you use logger (from your description) makes it so that different logger can get the same address.
We'll figure out something. Right now I'm guessing that actually comparing the number of addresses and where there point to could be as fast, and eliminate problems like that.

Why not do away with the notion of an id, and make formatting decisions based on the data itself? Performance reasons?

Performance, yes.
I've pushed a branch with fix based on atomic counter.
The cost:
cargo benchcmp old new --regressions
 name                                         old ns/iter  new ns/iter  diff ns/iter  diff %
 log_stream_async_empty_json_blackbox_log_10  143          149                     6   4.20%
 log_stream_async_json_blackbox_10            475          504                    29   6.11%
 log_stream_async_json_blackbox_log_10        136          142                     6   4.41%
 log_stream_json_blackbox_10                  1,378        1,379                   1   0.07%
 log_stream_json_blackbox_i32closure          925          938                    13   1.41%
 log_stream_json_blackbox_i32pushclosure      920          930                    10   1.09%
 log_stream_json_blackbox_i32val              921          923                     2   0.22%
 log_stream_json_blackbox_strpushclosure      958          964                     6   0.63%
 logger_new_empty                             21           25                      4  19.05%
 logger_new_nonempty                          52           57                      5   9.62%
 logger_new_nonempty_10                       60           66                      6  10.00%
 tmp_file_write_1kib                          362          369                     7   1.93%

I can confirm that this fixes my problem. Thanks!

Thanks. So the question now is, is the price worth it (+5ns when creating every Logger instance).
I'm thinking what other approaches could be used to keep the compacting functionality.

I don't think I can really help you with that decision. It would definitely be worth the cost to me, but my use case is not that performance-sensitive. I'd imagine there are use cases that are a lot more demanding, and I can't really speak for those.

I'm still thinking about it. The performance is once thing, the other one is the counter itself, that would only be 32-bit on 32-bit architectures. (and potentially smaller CPUs).
I wish the was a better way.

OK. So it seems to me the current method is a premature and all at all wrong optimization. The code is trying to be smart about not serializing unnecessary things, but it does not work in this case, and it also incorrectly groups messages in case of eg. closures that might change their values.
Instead of ids, everything should be serialized at all times, and results (serialized data) compared. Only then can results be trusted.
The id notion will be deprecated, and slog-term fixed not to use it.

Main fix pushed in https://github.com/slog-rs/term/commit/85d3cf04c35c8eaa7611f33a88fafb99b3f1868a and commits around it.

Closing. Feel free to reopen.

Your fix doesn't work, unfortunately. The problem, as I originally described it, persists. I'm looking into it now to figure out what the cause is.
Unfortunately I can't seem to reopen the issue, for some reason.

Please ignore my last comment. While I was in fact testing with the latest master branch of slog (and had verified that fact multiple times before posting here), I was still using the latest release of slog-term, which of course didn't contain the fixes.
Sorry for the mistake, and thanks a lot for the fix!

👍 Thanks for confirming it.


.fit method could have an argument for deleting broken files

System information.
TensorFlow version (you are using): 2.8.2
Are you willing to contribute it (Yes/No) : yes
Describe the feature and the current behavior/state.
Currently, when trying to fit a VGG16 model with a dataset containing some broken image files, the method simply returns an UnidentifiedImageError by PIL. Instead, it would be good to include an argument into the fit method which allows us to decide whether we would like to have the broken files deleted.

Will this change the current api? How?
Yes. .fit method will now have an argument delete_broken which is False by defualt, but when True, the broken image files are first deleted, and then the model is fitted. This is a way to handle UnidentifiedImageError which might confuse some users.
Who will benefit from this feature?
People who are training a model on custom data and may not be aware that some files are broken or dysfunctional.
Contributing

Do you want to contribute a PR? (yes/no): yes
If yes, please read this page for instructions
Briefly describe your candidate solution(if contributing): I'd like to include an argument, let's say: delete_broken which takes on boolean values, and when True, deletes the broken image files, before executing the actual fitting of the dataset to the model.  When False, the .fit method behaviour stays the same as it is now: it raises PIL's UnideitifiedImageError

@emailic,
Could you please share a reproducible code that supports your statement so that the issue can be easily understood? Thank you!

Hi @tilakrayal ,
thanks for getting back to me.
image_gen_train = ImageDataGenerator(rescale = ./255)
train_data_gen = image_gen_train.flow_from_directory(batch_size = batch_size, directory = train_dir, shuffle= True, target_size = (IMG_SHAPE,IMG_SHAPE), class_mode = 'categorical')
image_generator_validation = ImageDataGenerator(rescale=1./255)
val_data_gen= image_generator_validation.flow_from_directory(batch_size=batch_size, directory=valid_dir, target_size=(IMG_SHAPE, IMG_SHAPE), class_mode='categorical')
vgg_classifier= model.fit(train_data_gen, steps_per_epoch=(6610//batch_size), epochs = 5, validation_data=val_data_gen, validation_steps=(1531//batch_size), batch_size = batch_size, workers=16, use_multiprocessing=True, verbose = 1)
This is the code. In order to reproduce the error statement, you also need an Image dataset containing some broken (unreadable) files.

Thanks for reporting the issue! You can actually usetf.data.experimental.ignore_errors to skip broken samples.

Thanks for getting back to me @chenmoneygithub ! Sounds like a good solution. Take care, Ema

@emailic Closing this issue as it has been resolved by the above comment. Thanks!