Client Allocation Request for: FileDrive

Client Allocation Questions
Core Information

Name: FileDrive
Website / Social Media: filedrive.io
Region: Greater China
DataCap Requested: 5TiB
Addresses to be Notarized:  f3s3dwh67ps4kxui4mziwvus5ue7p6p324aiebrrpe24st5t334ib5gwlxql6auo7ibrccmppjxxavah46wdqa
Notary Requested: steven004

Hey @laurarenpanda  , thanks for your request. Can you kindly add more information / context here on:

yourself/ your organization (with links to official website/social media/news mentions etc)
Your history working on IPFS/Filecoin (notable contributions to the community, type of engagement shown over time etc)
Use case: type/source/quantity/diversity/scarcity of the data you plan to store on Filecoin
Miners you intend to allocate DataCap with allocation proportion(with redundancy/location/specific requirement of storage miners)
What's your next plan for the data you plan to store in Filecoin Network? For-profit or not?
Would you be OK in posting deal transactions here?

And, I have a new request, for further verification and better support, I need you share your company's business certificate, it could be the business license or equivalent material. For privacy, you can share me via mail.
Thanks

Hi @steven004 , thanks for replying, here are my answers.
Yourself/ your organization (with links to official website/social media/news mentions etc)
FileDrive team has been in Filecoin ecosystem from 2019, and conscientiously concentrate on development distributed storage solution. Our vision is to build a data transaction platform on Filecoin Network. Our team is composed of talents with years development experience. We have self-developed underlying codes, and technical architecture and solutions of distributed storage. In 2019, we independently developed DriveUp, which is a beta version cloud storage application based on IPFS. Over hundreds of users participated in testing phase. In 2020, we established Shanghai Xuanling Technology Co., Ltd, and began to work as start-up company. Since September 2020, we participated in Slingshot competition by Protocol Labs and developed an application named FileDrive.
FileDrive is a open source databank powered by IPFS/Filecoin, users can retrieve any data by file-name or content hash. Now, there are over 2.9k files from global public datasets stored on it with capacity of more than 57TiB storage space.

Project website: http://filedrive.io
Team website: https://honghupro.com
Twitter: https://twitter.com/FileDrive1
WeChat Official Account: FileDrive
Github: https://github.com/beeleelee
Slack: Laura Panda/ Felix
Participated in Filecoin Virtual Community Meetup - January 2021: https://www.youtube.com/watch?v=0aVghq-syoo

Your history working on IPFS/Filecoin (notable contributions to the community, type of engagement shown over time etc)
Contribution to the Filecoin Community:

Open-source tool Go-graphsplit: https://github.com/filedrive-team/go-graphsplit

Dashboard of DataCap: <filplus.info>

Participating in Slingshot Competition

Slingshot 2.2: 1st Place / Booster Award
Slingshot 2.1: 1st Place / Booster Award
Slingshot 1: 3rd Place / Community Choice Awards: Personal User Storage(Third Place)

Start-up project: DriveUp

Cloud storage application based on IPFS
Self-developed by FileDrive team

Use case: type/source/quantity/diversity/scarcity of the data you plan to store on Filecoin
Type: Web3 DApp / user content / public dataset
Source: open source / non-private data from users / DApp
Quantity: over 1 TiB
Diversity: no more than 10 copies
Scarcity: <5 similar data source
Miners you intend to allocate DataCap with allocation proportion(with redundancy/location/specific requirement of storage miners)

Redundancy: at least 3 copies
Proportion: at least 3 miners
Location: intend to store data in Asia
Miners: have great reputation and contribution in Filecoin Community, accept real data deals during Slingshot Competition, or are willing to store real data on Filecoin Network.

What's your next plan for the data you plan to store in Filecoin Network? For-profit or not?
All data on filedrive.io are non-profit. Everyone can search, browse and download these data by using FileDrive.
Would you be OK in posting deal transactions here?
Yes, of course.
Company's business certificate
The business license has already sent to you via email, please check it.

Hey @laurarenpanda , Your Datacap Allocation Request has been approved
Message sent to Filecoin Network

bafy2bzacecviicjlhh2xej2lol5qee7rhwbn54nedydbtkwjbwufognn7bjv6

Address

f3s3dwh67ps4kxui4mziwvus5ue7p6p324aiebrrpe24st5t334ib5gwlxql6auo7ibrccmppjxxavah46wdqa

Datacap Allocated

5 TiB

This user’s identity has been verified through filplus.storage


Capital letters in input alphabet

Why capital letters are part of the input alphabet and affect the size of embedding layer if input text is always lowcase()'ed ?
Here is the alphabet:
https://github.com/NVIDIA/tacotron2/blob/master/text/symbols.py#L12
And here is the english cleaner:
https://github.com/NVIDIA/tacotron2/blob/master/text/cleaners.py#L86

You can train with both uppercase and lowercase characters by changing the cleaners.


add server rpc mode

Resolves https://github.com/godotengine/godot-proposals/issues/4592
Note: I'll keep this a draft until the proposal has sufficient support

I'd love to take this on, what further support does this feature need?

If by support you mean people wanting it, I think it has more than enough already. The proposal is currently at 16 thumbs up and zero thumbs down.


Initial Zoom to defined Position (or save current view)?

Hi @nbelyh ,
i'm not sure if this is possible - we have some visio shapes to use as process description. If I embed them it looks like:

So there very small, so the zoom feature is nice. But when zooming into 120% for example we have to scroll a bit up to not have a blank page, because zoom is always centered.

Is it possible to save the "viewing" location / scrollbar location and use this in combination with the zoom level to have a nice view  like:

Thanks in advance,
Constantin

Yes I got the issue 😅
There is Visio Online API to control the View, so technically this must be possible:
https://learn.microsoft.com/en-us/javascript/api/visio/visio.pageview?view=visio-js-1.1#visio-visio-pageview-setposition-member(1)
An easy option may be to provide option for the default view, more "advanced" could be to analyze the content of the diagram and auto-adjust the diagram accordingly.
I'm actually currently working on "SVG" version of the web part, that would NOT use Visio to render diagrams (basically porting my SvgPublish extension to a  the web / wordpress part, that provides the default view out of the box), but currently it's at a very early alpha stage: https://github.com/nbelyh/SvgPublishWebPart

ah yeah with getPosition() and setPosition() that should be possible. (or maybe: centerViewportOnShape() is evend better)
could you give me a hint how I could call that funciton within the webpart? the zoom thing is only an attachment to the get url, that's what I already understood. But i'm a bit unclear how the other things get rendered :/

the svgPublish looks like the draw.io features :o) i know this from confluence, that's more awesome than visio by design, but currently aswell not an integration possibility found (even if its free) ;-)


jsii-docgen seems to use javadoc syntax instead of tsdoc

tsdoc and javadoc have slightly different syntax.
For example, tsdoc wants a hyphen at the start of @param descriptions, while javadoc does not:
tsdoc
 * @param first - The first parameter
javadoc:
 * @param first The first parameter

I've got automated tooling to enforce tsdoc syntax, but because jsii-docgen assumes the javadoc syntax, then the documentation ends up with errant - characters at the start of each parameter.
Another difference is in links. For example, tsdoc links use | to separate the text from the target, but javadoc does not.
tsdoc   {@link SomeClass | click here}
javadoc {@link SomeClass click here}

However, the generated remarks sections when using -l typescript -f json re-print the links in javadoc syntax, even if the format is typescript.

jsii-docgen doesn't parse your TS code, instead it reads from a pre build jsii assembly file (usually .jsii).
Can you please post the relevant section from your jsii assembly. If it's already wrong in the assembly, the issue is with aws/jsii-compiler and not jsii-docgen. (And because these repos live in a different organizations we unfortunately cannot move the issue.)

Sure thing! I've created this repository illustrating both problems.
The relevant part for this issue is the parsing of this line:
    /**
     * This documents a constructor
     *
     * @param someParameter - this documents the parameter
     */

Which results in this line in the assembly, which is from this segment:
        "parameters": [
          {
            "docs": {
              "summary": "- this documents the parameter."
            },
            "name": "someParameter",
            "type": {
              "primitive": "string"
            }
          }
        ]

(note that the summary inappropriately starts with "- ")

Oh, I forgot to include the repro for the link issue - in the generated json documentation for typescript, this line is in javadoc syntax -
{@link https://github.com/cdklabs/jsii-docgen/issues/1634 this link}

Which seems odd, as it was originally from a properly formed tsdoc comment here.
{@link https://github.com/cdklabs/jsii-docgen/issues/1634 | this link}

I'm not sure if there are other issues like this, these are just the two I noticed.

Also, let me know if you want me to reopen these issues somewhere else.

That's a great reproduction. Both look like issues with aws/jsii-compiler.
Thank you that would be great if you could reopen them.

Thanks! Opened https://github.com/aws/jsii-compiler/issues/1498 to track this.


ETH Bridge

@t-z-u add design file in this issue when you finish it

@creed-victor
https://youtu.be/nH2KsQ-P7Po
please reach out with any questions
I will work on the gif animations for the window (mm and liquality)
eth bridge v1.1.zip

eth bridge v1.2.zip
https://youtu.be/aV0TiS_0qVg
@creed-victor
really great to thrash this one out with you, I think this is working much better now
let me know if you have any issues or suggestions

https://youtu.be/ZZMMbdBio_g
eth bridge v1.2.zip
@creed-victor really great to thrash this one out with you, I think it works much better now :)

@creed-victor I think I need to add this to it as well

https://youtu.be/Z9Omufa41bA
eth bridge v1.2.zip
@creed-victor sorry about that!
was great to thrash this out with you, I think it works much better now

eth bridge v1.3.xd.zip
@creed-victor updated with external address based on your suggestions

eth bridge v1.3.xd.zip
@creed-victor updated design for external address based on your suggetions


Release 0.4.0

Cut a 0.4.0 release with the following changes:

Swift 4.0.2 (Xcode 9.1) support
Remove deprecated stackViewContainer property

Note that CI currently fails. This should be fixed with #41 (specifically 32ebf4d8ca9e6e3fbb3f756354c673486b2996af).


Syntax highlighting: Operators

@joshgoebel I can't find a way to make operators work and I can't find any documentation on regex.either( )
But I did fix the keyword pattern, so that's working now.
Originally posted by @KTSnowy in https://github.com/exercism/cobol/issues/1#issuecomment-1182195885

Maybe show me the last thing you tried?  It's not documented, but look in the regex.js source and it should be easy to see - it just takes an arrray of strings or regex - or you can manually write the regex out - it's just nicer to use the helper.

Will this work?

match (sugar for begin) is nicer, but yes that should work IF it doesn't need any escaping.  It looks right at a glance.

Hey @joshgoebel it worked, we now have operator and type highlighting

There's one more issue, COBOL comments start with a *> and COBOL also has * and > operators. This means that we'll have to use regex negative lookahead and negative lookbehind, and lookbehind is not supported on Safari.

This is to keep the operator regex from overwriting the hljs.COMMENT() regex and not displaying comments properly

I uploaded the latest version to NPM: https://www.npmjs.com/package/highlightjs-cobol

I see that the operator regex also broke the keywords with a -

negative lookbehind, and lookbehind is not supported on Safari.

Yeah, generally we don't recommend anyone use lookbehind (and it's forbidden in core) because of Safari.  There are various tricks, did you figure it out?
Why do you think you need look-behind, for -?  Doesn't look-ahead cover comments?

I'll try to update it later today without lookbehind if I find a way to make it work.

I can't make it ignore *> in the operator regex without lookahead

Negative lookahead, no?  * not followed by >... looking \*(?!>)
You can use look-ahead everywhere, the issue is with look-behind (in Safari).

Is there a way to make highlightjs choose the comment and keyword regex first over the operator regex?

contains rules are scanned in the order specified, but keywords is a whole sep engine that runs "inbetween" other things...

same thing with the keywords and the - operator.

Exactly example please?

Here, function-id for example didn't get completely highlighted because it would choose the - operator instead of the keyword.

I saw originally you were using \W, was that not helping?  IF you never need the character before to have it's own highlight you could use a multimatch:
{
match : [ /\W/, /-/ ],
scope: { 1: null, 2: "operator" }

Or even multiple multi-matches (for diff cases)... but if you need the content immediately before to match another rule - this trick no longer works.
Otherwise you may need to fall back to specifying a custom match rule for ONLY - keywords (or move all your keywords to rules)... once you are just using normal rules you defeat the "weirdness" of the keywords engine.  This has performance implications if you have a zillion keywords, but I think your keyboard could would be "ok".  The inbetween ground is to only use rules for the - keywords...
See angelscript for an example of using rules for keywords... the keywords engine is just a lot more convenient if it works...

If you'd like to see the test here's the codepen: https://codepen.io/ktsnowy/pen/vYRXxov

The negative lookbehind was also removed in version 0.2.11, so it should be all working fine now.

so requiring a space around the - operator would be best practice anyways.

Oh sure if you can just say "use best practices" and then code for the space that's even simpler! :-)  Awesome.


Include other contextual covariates from GHSL UCDB

Other relevant contextual covariates are present in the GHSL UCDB, including average elevation, climate zones, average temperate and rainfall.  While these are historic (eg in latest version, the latest observations are for 2014 or 15) it may be fair to assume these aspects have not changed too much in general in past 8 years.
So, will be good to include in the example to show this can be done (as per suggestion of @marcdmallafre; thanks Marc, good idea!)
It will require a slight modification of code, as some of these variables are strings, and currently the code assumes they are numeric.

This is implemented -- eg in the GUI when looking at region statistics, these look like this for Las Palmas:

Of course, it would be nice if a description including units were provided insead of the obscure variable name, but that's another issue!


websocket bad handsake

panic: couldn't dial whatsapp web websocket: websocket: bad handshake

Is there any solution for "websocket bad handsake" problem ?

Well, I take it there is no solution as of yet. Will keep checking. Thanks.

Please read the README.md file. This project is no longer maintained and will not work with newer versions of WhatsApp.


Optimize StableLM model inference

What does this PR do?

Works

Optimized StableLM model inference.
Validated stabilityai/stablelm-zephyr-3b inference on single card.

Validation results
static shape generation
python run_generation.py --model_name_or_path stabilityai/stablelm-zephyr-3b --use_kv_cache --max_new_tokens 100 --bf16 --batch_size 4 --use_hpu_graph

Input/outputs:
input 1: ('DeepSpeed is a machine learning framework',)
output 1: ('DeepSpeed is a machine learning framework that allows you to build and train deep learning models at high speeds.\n\nIt is designed to handle large datasets and complex models, making it suitable for applications such as image and speech recognition, natural language processing, and autonomous driving.\n\nOne of the key features of FastSpeed is its ability to parallelize computations across multiple CPUs and GPUs, enabling it to achieve significantly faster training times than other frameworks.\n\nAnother important aspect of FastSpeed is its support for a wide range of',)

input 2: ('He is working on',)
output 1: ("He is working on a project to create a new type of material that can be used in various industries, including construction, aerospace, and automotive.\n\nStep 1: Research existing materials\nTo create a new type of material, it's essential to understand the properties and limitations of existing materials. The researcher will conduct a thorough analysis of various materials, including metals, polymers, ceramics, and composites.\n\nStep 2: Identify gaps in existing materials\nAfter researching existing materials, the researcher will identify any",)

input 3: ('He has a',)
output 1: ('He has a lot of experience in the field of education, having worked as a teacher, principal, and administrator for many years.\n\nHe has a deep understanding of the needs of students, having spent countless hours observing and interacting with them. This has given him a unique perspective on how to create an environment that fosters learning and growth.\n\nHe is also well-versed in the latest educational trends and practices, having kept up with the latest research and developments in the field. This allows him to provide',)

input 4: ('He got all',)
output 1: ('He got all the information he needed from the source, which was the official website of the organization. and he also checked the references of the author of the document. He wanted to make sure that the information was accurate and came from a reliable source. This is a good practice to follow when using information from external sources., especially when it comes to important or sensitive topics., verifying the information and checking the references of the author can help ensure that the information is accurate and comes from a reliable source.',)

Stats:
--------------------------------------------------------------------------------------------------------------
Throughput (including tokenization) =<PHONE_NUMBER>354841 tokens/second
Number of HPU graphs                = 12
Memory allocated                    = 5.46 GB
Max memory allocated                = 5.53 GB
Total memory available              = 94.62 GB
Graph compilation duration          = 10.75790829397738 seconds
--------------------------------------------------------------------------------------------------------------

Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[x] Did you make sure to update the documentation with your changes?
[x] Did you write any new necessary tests?

Hi, can you please review this PR?

@XinyuYe-Intel can you add a test case in txt-gen for ci slow test?

@libinta ci test is added for stabilityai/stablelm-zephyr-3b.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

Can you run
pip install -U ruff
make style

from the root of the repo please?

It's done.


[Feature Request]: Handling loading states

Requirements before reporting

[X] No duplicated issue reported
[X] I have read the contribution guidelines
[X] I have checked the lastest version if the feature exist there

Type of feature request
Functionality
What is your feature request?
The components such as cards/tooltip/loaders don't have a loading state, thus they appear from nothing; sometimes some of the data which needs to be populated in these is awaited from the backend or is received asynchronously.
There should be a way to have a loading state while the data is getting populated. Examples shimmer effect used by facebook or pulsing animation by linkedin for example.
Background
This is a modern UX pattern which if available in our design system will be great for Scania's own platforms plus other external developers who intend to use this design system. This will be a good UX where network data is slow.
Idea
Corporate UI had a generic loader which could be installed in the places, but modern UXs have the skeletal content of the expected data ad not a generic loader. This could  be a good start.
Goals and objectives
A better UX
Timeline
No response
Resources
https://pencilandpaper.io/articles/ux-pattern-analysis-loading-feedback/#:~:text=Loading states are just that,is effectively being worked on.
Contact information
<EMAIL_ADDRESS>

Great request - we are already on to it! We are now fixed on the design of skeleton screens, and we will very soon start to take that into code. The idea -for now- is to have a skeleton state/variant for each of our components.

@alexlioliopoulos Thanks for the response, I can also contribute if needed once you have set some examples

https://www.figma.com/proto/KkyoRT71TbX95sM7nEwsUF/Skeleton-text-field-exploration?page-id=1%3A166&node-id=425%3A28075&viewport=-1176%2C-3628%2C0.14&scaling=min-zoom
Here you can see an example of what we are aiming for. We are now considering how to take that to code; if skeleton will be a component variant or a state, and how it will all work together. If you'd like to give it a try, we can talk more :)

Looks real nice, and should be a good visual experience, and yes, I would like to be part/contribute in building this :)


[Bug Report] Prevent to call onChange of TextField when Keyboard hide (Back Press)

I am working on a Sample Application in which I am searching using TextField and displaying data in ListView.
I found some issue in this scenario that when I press the back button to hide the keyboard, it calling onChange() callbacks again. It's happening on the back press button only, working perfectly on the Done button of Keyboard.
You can try the sample gist (without API Calling and printing simple logs). Kindly run on the device to hide the keyboard by pressing the back button.
Here is the video to clear more about it:
https://user-images.githubusercontent.com/3306366/113987983-99d9a680-986c-11eb-8c1a-7f37e92f9737.mp4
I have tried to run it on both stable and master branches of flutter.
MASTER : flutter --version:

Flutter 2.1.0-13.0.pre.524 • channel master • https://github.com/flutter/flutter.git
Framework • revision 98a9618993 (2 hours ago) • 2021-04-07 22:34:03 -0700
Engine • revision cf86823971
Tools • Dart 2.13.0 (build 2.13.0-213.0.dev)

STABLE : flutter --version:

Flutter 2.0.4 • channel stable • https://github.com/flutter/flutter.git
Framework • revision b1395592de (6 days ago) • 2021-04-01 14:25:01 -0700
Engine • revision 2dce47073a
Tools • Dart 2.12.2

I got a temporary answer on Stackoverflow but its a hack only not a solution.
I think many developers are not aware of it and don't know even they are calling their APIs +1 time. Hope it will be solved soon.

Related to https://github.com/flutter/flutter/issues/79982. However I cannot reproduce this one on iOS simulators.

code sample
import 'package:flutter/material.dart';

void main() => runApp(MyApp());

class MyApp extends StatelessWidget {
  late final TextEditingController _controller = TextEditingController();

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      home: Scaffold(
        body: Center(
          child: TextField(controller: _controller, onChanged: (v) => print('Change: $v\n'),),
        ),
      ),
    );
  }
}

flutter doctor -v
[✓] Flutter (Channel master, 2.1.0-13.0.pre.524, on macOS 11.2.3 20D91 darwin-x64, locale en-AO)
    • Flutter version 2.1.0-13.0.pre.524 at /Users/pedromassango/Code/flutter_master
    • Framework revision 98a9618993 (4 hours ago), 2021-04-07 22:34:03 -0700
    • Engine revision cf86823971
    • Dart version 2.13.0 (build 2.13.0-213.0.dev)
[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
    • Android SDK at /Users/pedromassango/Library/Android/sdk
    • Platform android-30, build-tools 30.0.2
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.
[!] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.4, Build version 12D4e
    ✗ CocoaPods not installed.
        CocoaPods is used to retrieve the iOS and macOS platform side's plugin code that responds to your plugin usage on the Dart side.
        Without CocoaPods, plugins will not work on iOS or macOS.
        For more info, see https://flutter.dev/platform-plugins
      To install see https://guides.cocoapods.org/using/getting-started.html#installation for instructions.
[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome
[✓] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)
[✓] IntelliJ IDEA Community Edition (version 2021.1)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 55.0.5
    • Dart plugin version 211.6693.108
[✓] VS Code (version 1.54.3)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.16.0
[✓] Connected device (3 available)
    • sdk gphone x86 64 (mobile) • emulator-5554 • android-x64    • Android 11 (API 30) (emulator)
    • macOS (desktop)            • macos         • darwin-x64     • macOS 11.2.3 20D91 darwin-x64
    • Chrome (web)               • chrome        • web-javascript • Google Chrome 89.0.4389.114
! Doctor found issues in 1 category.

can not repro, my device is OPPO RENO 5.


When running replicate.to from browser, express-pouch _local calls all return 404 missing

I have cleared data on either side, am using memory adapter and 6.3.4 version for everything, node 8.4.
When I run a pouch.replicate.to:

I think this may be due to some sort of proxy issue. If I run a replicate from first I get an error on this url:
http://orbit.dev/db/users/_all_docs?conflicts=true&include_docs=true
With a response of {"error":"not_found","reason":"missing"}
But if I load it directly in browser it seems to show a proper JSON response.

Seems like GET works fine, OPTIONS isn't happy:


Need feature to delete installed update

Is there feature to delete installed update?

@abonec do you mean remove all the content at a sync location or roll back the latest merge?

I mean delete whole directory with downloaded and extracted update by ID or by full path.

As of 1.2.0 of the plugin both Android & iOS store their data in the path represented by cordova.file.dataDirectory so you can use the File plugin to remove your cached data.
window.resolveLocalFileSystemURL(cordova.file.dataDirectory + id, function (dirEntry) {
  dirEntry. removeRecursively(function() {
    // success
  }, function(error) {
    // error
  });
} function(e) {
  // handle error
});

Would there be a way in the future to have this functionality included without requiring the file plugin? I'm using the id as a way of keeping track what update version is running, so would be awesome if there was a function which 'cleans up' old IDs.

I think being able to clear cached data would be nice too (although using the File plugin works too). I need to do the same thing as @CookieCookson (clear "old" ids)
I imagined the API would be similar to the ionic-plugin-deploy methods getVersions and deleteVersion:
https://github.com/driftyco/ionic-plugin-deploy/blob/master/www/ionicdeploy.js#L65

maybe the contentsync plugin should keep a history/manifest of synced updates. eg:
contentsync.json
{
	"appid-1": {
		"src": "http://some.server.com/update-package.zip",
		"localPath": "..../files/appid-1",
		"lastUpdate":<PHONE_NUMBER>3,
		"lastSyncType": "merge"
	},
	"appid-2": {
		"src": "http://some.server.com/update-package.zip?v=2",
		"localPath": "..../files/appid-2",
		"lastUpdate":<PHONE_NUMBER>6,
		"lastSyncType": "replace"
	},
	"deep/appid/1.0": {
		"localPath": "..../files/deep/appid/1.0",
		"lastUpdate":<PHONE_NUMBER>6,
		"lastSyncType": "local",
		"copyRootApp": true
	}
}

and then offer a method to retrieve this list. delete could be done with a new sync type delete which wouldn't require a src.

I have a feeling that the plugin already does too much. We could offer the functionality to delete old synced folders but we might end up with a full replacement of the File plugin and I am not sure we want that.


Specify initial position relative to body

I'd like the default position of my draggable element to be, say, 200px down from the top of the screen and centered horizontally. Is this possible?

Yes, just position it that way (position: absolute) then apply <Draggable> to the element. Try it.

Thanks--for anyone else coming in from google:


fix(ci): increase log level for unit test, fix connector node ci

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?

fixed an error in a unit test https://github.com/risingwavelabs/risingwave/pull/9575
increased connector unit test log level to ERROR (less INFO logs...)
modified connector test name

Checklist For Contributors

[x] I have written necessary rustdoc comments
[x] I have added necessary unit tests and integration tests
- [ ] I have added fuzzing tests or opened an issue to track them. (Optional, recommended for new SQL features #7934).
- [ ] I have demonstrated that backward compatibility is not broken by breaking changes and created issues to track deprecated features to be removed in the future. (Please refer to the issue)
[x] All checks passed in ./risedev check (or alias, ./risedev c)

Checklist For Reviewers

[x] I have requested macro/micro-benchmarks as this PR can affect performance substantially, and the results are shown.

Documentation

[x] My PR DOES NOT contain user-facing changes.

IMO we can improve the organization of test cases by collecting all unit tests in a dedicated module and grouping different tests by package. Then all unit tests can share some common configuration. @WillyKidd


How to improve the speed of Kernel Explained

How to improve the speed of Kernel Explained ?
I see that it processes one sample by one sample.
Are there any ways to do it faster, such as batch?
Thanks for answering!!!

#200 should help here a lot by speeding up the building of internal matrices. It does build it's internal matrix one by one, but it calls the model function with a batch. So with #200 moving some of these steps to numpy things should get better.

I'm not quite understanding that #200. Is it talking about squeezing the features into shorter vector, right?
Follow up question is, what do you mean by batch? Can you elaborate it?
Thanks!

@tonnkie each sample in the dataset is explained one at a time, but many evaluations of the model (2000 by default) are done to explain each of those instances. Those evaluations are passed the model as a batch.
#200 just moves some processing from python to numpy to get faster execution.
@GrimLefourbe ...hmmm. Could you see if you get the same redundancy for a single sample? There should not be very many redundant calls by KernelExplainer to the model (only when we randomly sample the same set). If you have lots of samples that can have similar inputs you could consider wrapping you model evaluation function in a memoize function.

Here's a notebook showing my results. The redundancy is much less significant on randomised data than it was in my previous tests but I assume it would behave worse on real data.
Unexpectedly, there still seems to be redundancy when explaining a single sample.
SHAP Values Call overlap.ipynb.zip
I find about 6.5% of samples to be redundant on random data.

@GrimLefourbe thanks for the notebook! I fixed the double call during instantiation, though that is not really going to effect things much. The real redundancy is coming from the random sampling process. Whenever there are enough samples to fill out all subsets of a given size it will do that (and so eliminate redundant samples), but if you don't have quite enough samples for a given subset size then it can't fully enumerate them, yet it is still easy to draw redundant samples.
The best fix would be to check for redundant samples when adding the samples, but I am not sure if using a dictionary would be fast enough for all use cases. I'll leave this issue open to note it is a possible improvement.

I decided the dictionary would fast enough applied to the varying groups mask, so now the latest master shows zero redundant calls.


fix reduce memory cost for cloud compaction of mow table

What problem does this PR solve?
Issue Number: close #xxx
Related PR: #36865
Problem Summary:
#36865 reduced the memory cost for compactions of MoW table
But when we merge the codes for cloud, such optimization is not applied for cloud compaction
We found several cases that compaction of MoW table consume lots of memory on cloud, this PR try to fix this issue
Release note
None
Check List (For Author)

Test

[ ] Regression test
[ ] Unit Test
[ ] Manual test (add detailed scripts or steps below)
[x] No need to test or manual test. Explain why:

[ ] This is a refactor/code format and no logic has been changed.
[ ] Previous test can cover this change.
[ ] No code files have been changed.
[x] Other reason: memory consumption is not easy to test on regression test, it's already tested in our test environment.

Behavior changed:

[x] No.
[ ] Yes.

Does this need documentation?

[x] No.
[ ] Yes.

Check List (For Reviewer who merge this PR)

[ ] Confirm the release note
[ ] Confirm test cases
[ ] Confirm document
[ ] Add branch pick label

run buildall

run buildall

run p0


Update BlogController.php

Does't form->isValid already check id if the form has been submitted? Why do we need to check $form->isSubmitted()

@ZCJ thanks for preparing this pull request!
Technically, you are 100% right and the isSubmitted() method is not necessary. But we include it to improce code readability. In this closed pull request we explain the reasons: #2
To avoid any further issues with this line of code, we've added a help note in 3b1446008770f00b972c93e9205065b5d32184a1


Does BadgeApp allow multiple people to edit the same entry

I cannot seem to find the answer in Issues on the website.
The idea is that any Project Member would be able to edit the project entry on CII website.

Yes, if you have write privileges on GitHub you are able to edit the project.
--
Dan Kohn<EMAIL_ADDRESS>Executive Director, Cloud Native Computing Foundation https://cncf.io
tel:+1-415-233-1000

On Jan 3, 2017, at 17:49, Andrey Prokopenko<EMAIL_ADDRESS>wrote:
I cannot seem to find the answer in Issues on the website.
The idea is that any Project Member would be able to edit the project entry on CII website.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Dan, thanks for answering that.
I wonder if we should document that were clearly somewhere. It's not obvious to me where we would put that information.

It's now documented in this issue.
On Tue, Jan 3, 2017 at 6:55 PM, David A. Wheeler<EMAIL_ADDRESS>wrote:

Dan, thanks for answering that.
I wonder if we should document that were clearly somewhere. It's not
obvious to me where we would put that information.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/linuxfoundation/cii-best-practices-badge/issues/566#issuecomment-270259612,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AC8MBh5jmYlSdSGfk7njCdMdMmQqh9lrks5rOuAGgaJpZM4LaHS2
.

--
Dan Kohn<EMAIL_ADDRESS>Executive Director, Cloud Native Computing Foundation https://cncf.io/
tel:+1-415-233-1000

True - but a closed issue isn't the most obvious place to get this kind of information.  Maybe we could have a static page with some help information like this.

I just find that
https://github.com/linuxfoundation/cii-best-practices-badge/blob/master/doc/implementation.md
is already so voluminous that its unlikely that anyone but a core
contributor will review it all. Much more likely is that they'll search for
the answer, and then the issue will come up.
But, obviously, no harm in adding it twice.
On Tue, Jan 3, 2017 at 8:32 PM, David A. Wheeler<EMAIL_ADDRESS>wrote:

True - but a closed issue isn't the most obvious place to get this kind
of information. Maybe we could have a static page with some help
information like this.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/linuxfoundation/cii-best-practices-badge/issues/566#issuecomment-270273866,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AC8MBm-PCNlpCgjM56gRkVNu2gPm-9ehks5rOvbKgaJpZM4LaHS2
.

--
Dan Kohn<EMAIL_ADDRESS>Executive Director, Cloud Native Computing Foundation https://cncf.io/
tel:+1-415-233-1000

implementation.md  he's not intended for end users. It is intended for long-term code contributors. We really don't have a place to provide information to users like this.


integrations/viem: doctests

Adds tests to ensure the README.md examples from integrations/viem-v2 can be imported.
This will catch any glaring errors. I think we should extend this to all of the packages in this repo.

Nice!

If we'll need snippets that skip some initialization code - copying a comment from slack:
For previous attempts at testing code in markdown we pulled code in from tests:

generated documentation using vuepress snippets https://v1.vuepress.vuejs.org/guide/markdown.html#import-code-snippets
(https://github.com/vuejs/vuepress/issues/2663)

docs.oasis.io generated using docusaurus + https://github.com/oasisprotocol/docs/blob/ded70ed/src/remark/code-block-snippets.ts
https://docs.oasis.io/rofl/app#testing-it-on-sapphire-localnet
-> https://github.com/oasisprotocol/oasis-sdk/blob/d23473d/docs/rofl/app.md?plain=1#L122
-> https://github.com/oasisprotocol/oasis-sdk/blob/d23473d/examples/runtime-sdk/rofl-oracle/src/main.rs#L24-L35

New idea that would work with npm and github markdown without generating them:

keep code as is in markdown
add same code to files in tests
add test that checks that code is found in a test (ignore indentation/whitespace)


Newsletter validation

🛠 Fixes Issue
Closes #720
👨💻 Changes proposed

Added form validation to the newsletter section using pop-up msg.

✔ Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

Video link : https://drive.google.com/file/d/1s6xkByVaZtbyIN_rhJdtm4Muj7lj5S3q/view?usp=sharing
Kindly check @SauravMukherjee44

@soujo nice work dude. 👍


Add hakman to the approvers list

This should allow me to create releases.

/cc @vteratipally
/assign @vteratipally

cc @derekwaynecarr , @wangzhen127 In case you can approve. My team is keen to get this release :-)

/cc @dchen1107
Could you please share the contribution details based on the approver policy. https://github.com/kubernetes/community/blob/master/community-membership.md

@vteratipally The immediate need is for the generation of the 0.8.16 release, to fix CVEs and (I believe) at least one other issue.  @hakman has completed the PR and the tag is applied, but he doesn't currently have permission to make an official release.
If you think this request here to add him to the approvers list may be delayed or not approved, would it be possible for your yourself to generate the Release, as you did for the previous one.
Further discussion, if needed, is here https://kubernetes.slack.com/archives/CJA25LM6D/p1709062180808009

I haven't worked on NPD for the past few years. But I am starting to pick it up again recently. If the request is to make a new release, I can do that this time. I will sync up with @vteratipally on it.
Regarding the approvers, since I lack the knowledge on NPD work progress in recent years, I will defer it to @vteratipally.

Thanks @wangzhen127 .  Yes, that is the request.  To make a new release based on the 0.8.16 tag that @hakman  has already created.

OK. I will likely be able to make the 0.8.16 release early next week. Does that sound good?

Yes, we will be creating a new release early next week. Hope that should be fine.

Thanks @wangzhen127  and @vteratipally. Before you cut that release, can you please take a look at this issue which I just logged: #871.  We are seeing some brand new CVEs in the docker image that @hakman made a couple of days ago.  Can these new CVEs be fixed, with the fixes included in the new release?   They are reported as "HIGH" priority by our scanner.
If those fixes can be included in a release early next week, that would be great. Thanks.

Yes sure, we can fix them

@JohnRusk I understand the need for a new release. Thank you for your support, but I would like to keep this issue about adding myself to the approvers list.
@vteratipally I understand you have concerns about this PR. Would you mind contacting me in private and discussing them? Thanks!

@JohnRusk I understand the need for a new release. Thank you for your support, but I would like to keep this issue about adding myself to the approvers list.
@vteratipally I understand you have concerns about this PR. Would you mind contacting me in private and discussing them? Thanks!

Yes sure. We could set up some meeting.

Yes sure. We could set up some meeting.

Sure, please ping me in Slack and we can find some time that suits us both.


Stress Test - top-level listener for uncaught exceptions/rejections

Work Item
Describe the outcome you expect
Uncaught exceptions and Promise rejections should appear in the logs for the stress test.
Approach
uncaughtExceptionMonitor looks like what we want, and just log an error using the logger when it happens.
Open questions
We should also think through what to do with the runner process itself in these cases - let it crash and the orchestrator restarts it? Catch the exception and "reset" the runner to keep going?
Additional context
Breaking this out as a work item for #5304

right now uncaught exceptions are already handled in .catch block. Mark and I decided to add logging of unhandled promise rejections.


Colon in string misinterpreted

Issue Type

Bug report

Ansible and Ansible Lint details
ansible <IP_ADDRESS>
ansible-lint 3.3.0

ansible installation method: homebrew
ansible-lint installation method: pip

Desired Behaviour
When a quoted string contains a colon, the linter should not consider it to be a key.
Actual Behaviour (Bug report only)
When given the following:
- debug: msg="foo: bar"

ansible-lint returns:
Syntax Error while loading YAML.

The error appears to have been in '/sample.yml': line 7, column 18, but may
be elsewhere in the file depending on the exact syntax problem.

The offending line appears to be:

- debug: msg="foo: bar"
                 ^ here

In the bug report template, there is a section asking for the result of ansible-playbook --syntax-check. I suspect you'll get the same error. The problem is not with ansible-lint, it's that it's not valid YAML.
This one is easily fixed:
- debug:
     msg: "foo: bar"

Given that Ansible best practices say to use the YAML format for module arguments, rather than key=value form, anyway, this is a double win.


Can i suppress type name for OpenSearchservice v2.3

Problem
The _type field has been removed from Amazon OpenSearchService since v2.0.
https://opensearch.org/docs/latest/breaking-changes/
For example, in the fluent-plugin-elasticsearch plugin, the suppress_type_name setting can be used to prevent the _type field from being sent to OpenSearchService, but is there a similar setting planned to be released for the fluent-plugin-aws-elasticsearch-service plugin?
https://github.com/uken/fluent-plugin-elasticsearch/#suppress_type_name
Alternatively, could you please provide a workaround?
Note that even if type_name "" is set, the _type field is still sent to OpenSearchService and the sent logs are rejected.
...
Steps to replicate
  <match os.**>
    @type aws-elasticsearch-service
    include_tag_key true
    type_name ""
    target_index_key "@log_name"
    tag_key "@log_name"
    logstash_format true
    logstash_prefix "projecta"
    logstash_dateformat "%Y.%m"
    <buffer tag>
      @type "memory"
      flush_thread_count 4
    </buffer>
    <endpoint>
      url "https://search-xxxxxxxxxx.ap-northeast-1.es.amazonaws.com"
      region "ap-northeast-1"
      access_key_id "xxxxxx"
      secret_access_key xxxxxx
    </endpoint>
  </match>

Expected Behavior or What you need to ask
...
Using Fluentd and ES plugin versions

OS version
4.14.301-224.520.amzn2.x86_64
Fluentd
td-agent 1.11.1
AWS ES Service plugin version
fluent-plugin-aws-elasticsearch-service (2.4.0)
IAM policies for AWS ES Service (optional)
ES version (optional)
OpenSearchservice v2.3

@hanedatatsufumi any fix for it ? facing the same problem.


Bump golang-lint to 1.61 to support go 1.22

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Type of changes
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Feature/Issue validation/testing:
Please describe the tests that you ran to verify your changes and relevant result summary. Provide instructions so it can be reproduced.
Please also list any relevant details for your test configuration.

[ ] Test A

[ ] Test B

Logs

Special notes for your reviewer:

Please confirm that if this PR changes any image versions, then that's the sole change this PR makes.

Checklist:

[ ] Have you added unit/e2e tests that prove your fix is effective or that this feature works?
[ ] Has code been commented, particularly in hard-to-understand areas?
[ ] Have you made corresponding changes to the documentation?

Release note:

Bump golang-lint to 1.61 to support go 1.22

Re-running failed tests

/rerun-all - rerun all failed workflows.
/rerun-workflow <workflow name> - rerun a specific failed workflow. Only one workflow name can be specified. Multiple /rerun-workflow commands are allowed per comment.

@sivanantha321 are all the changes related with the golint-ci upgrade?

@sivanantha321 are all the changes related with the golint-ci upgrade?

Yes and it also enables lint for tests and fixes previously excluded lint errors

/hold


add logo

Pull Request Template
Description
Add logo folder and logo
Fixes # (issue)
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] Any dependent changes have been merged and published in downstream modules
[ ] I have checked my code and corrected any misspellings

Hi @xroll thank you for your PR!
Do you want to add also a link to it, so it is displayed in the README.md file?

I make a new one

Is it good ?

I'ts perfect! Thank you @xroll !


Onboarding purview account

update mapping
add metadata

Docs Build status updates of commit a8a9ca6:
:warning: Validation status: warnings

File
Status
Preview URL
Details

preview-packages/docs-ref-mapping/reference.yml
:warning:Warning

Details

docfx.json
:white_check_mark:Succeeded

preview-packages/docs-ref-mapping/reference.yml

Line 111, Column 10: [Warning-xref-not-found] Cross reference not found: 'azure.nodejs.sdk.landingpage.services.network.pre'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Setting hubProjectName Causes Failure

It appears that when setting the project name that the file output name is changed, but this causes the plugin to fail. Removing the hubProjectName parameter allows the upload to succeed; however, we don't want to use the default project name.
Version: 'com.blackducksoftware.integration:hub-gradle-plugin:5.0.1'
Settings:
    buildBom {
        hubUrl = 'https://hubqualtrics.blackducksoftware.com'
        hubUsername = 'automation'
        hubPassword = '<REDACTED>'
        hubProjectName = "${project.group}.${project.archivesBaseName}"
        hubIgnoreFailure = true
    }

Output:
Could not deploy the file to the Hub, check the logs for specific issues: Failed to import Bom file: /Users/villek/Workspace/data-relocation-service/build/blackduck/service_bdio.jsonld to the Hub with Error : File '/Users/villek/Workspace/data-relocation-service/build/blackduck/service_bdio.jsonld' does not exist
org.gradle.api.GradleException: Could not deploy the file to the Hub, check the logs for specific issues: Failed to import Bom file: /Users/villek/Workspace/data-relocation-service/build/blackduck/service_bdio.jsonld to the Hub with Error : File '/Users/villek/Workspace/data-relocation-service/build/blackduck/service_bdio.jsonld' does not exist
        at com.blackducksoftware.integration.gradle.task.BuildBomTask.deployHubBDIO(BuildBomTask.java:243)
        at com.blackducksoftware.integration.gradle.task.BuildBomTask.performTask(BuildBomTask.java:153)
        at com.blackducksoftware.integration.gradle.task.BuildBomTask.task(BuildBomTask.java:130)
        at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
        at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
        at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
        at java.lang.reflect.Method.invoke(Method.java:498)
        at org.gradle.internal.reflect.JavaMethod.invoke(JavaMethod.java:73)

And the directory contents in question:
$ ls -l /Users/villek/Workspace/data-relocation-service/build/blackduck/
total 48
-rw-r--r--  1 villek  staff    30 Jun 28 11:37 blackDuckHubProjectName.txt
-rw-r--r--  1 villek  staff    14 Jun 28 11:37 blackDuckHubProjectVersionName.txt
-rw-r--r--  1 villek  staff  6959 Jun 28 11:37 com.qualtrics.core.drs.core_bdio.jsonld
-rw-r--r--  1 villek  staff  7306 Jun 28 11:37 com.qualtrics.core.drs.service_bdio.jsonld

So it seems that the plugin is respecting the project name for generating the file but for some reason then looks for the file name without the customized name. I am not able to reproduce the problem with the 4.0.0 version of the plugin so this seems like a regression with the 5.0.1 version.

I am able to reproduce the problem here in house and I'm looking into it.  I understand what the cause is.

Addressed with the commit:
https://github.com/blackducksoftware/hub-gradle-plugin/commit/f432928069958e1470b4572a74bbdb6300d778f8
Will be in the next release of the plugin 5.0.2

Released 5.0.2 of the gradle plugin containing the fix


Create integration test in core for ScriptExecutor

Tested to work locally, so this test will prove its working in each build


Add: LRT Televizija [LT]

Channel Name
LRT Televizija
Alternative Names (optional)
Lietuvos Programme; LTV; LRT
Network (optional)
No response
Owners (optional)
No response
Country
LT
Subdivision (optional)
No response
City (optional)
No response
Broadcast Area
LT
Languages
Lithuanian
Categories (optional)
Kids
NSFW
FALSE
Launched (optional)
April 30 1957
Closed (optional)
No response
Replaced By (optional)
No response
Website (optional)
http://www.lrt.lt/en/channels/ltv
Logo
https://upload.wikimedia.org/wikipedia/commons/thumb/9/9d/LRT_TV_Logo_2022.svg/1280px-LRT_TV_Logo_2022.svg.png
Notes
No response

Already in the database: https://iptv-org.github.io/channel?id=LRTTV.lt


389 fix types

What: Fix for #389

Why:

How:

Checklist:

[ ] Documentation
[ ] Tests
[x] Ready to be merged

[ ] Added myself to contributors table

Codecov Report

Merging #392 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master   #392   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           4      3    -1
  Lines         335    324   -11
  Branches       90     88    -2
=====================================
- Hits          335    324   -11

Impacted Files
Coverage Δ

src/utils.js
100% <ø> (ø)
:arrow_up:

src/downshift.js
100% <100%> (ø)
:arrow_up:

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 ee64fd8...fc335d3. Read the comment docs.


[SPARK-48391][CORE]Using addAll instead of add function in fromAccumulatorInfos method of TaskMetrics Class

What changes were proposed in this pull request?

Using addAll instead of add function in fromAccumulators method of TaskMetrics.
Why are the changes needed?

To Improve performance. In the fromAccumulators method of TaskMetrics，we should use  tm._externalAccums.addAll instead of tm._externalAccums.add, as _externalAccums is a instance of CopyOnWriteArrayList
Does this PR introduce any user-facing change?

yes.
How was this patch tested?

No Tests.
Was this patch authored or co-authored using generative AI tooling?

No.

@cloud-fan if you have time please take a look？

Retest this please.

thanks, merging to master/3.5!

Thanks for the reivew, all of you.


how different it is from Robyn

how different it is from Robyn
https://github.com/facebookexperimental/Robyn

If someone thinks I am wrong please feel free to correct me.

Lightweight MMM Is written in Python and not in R.
One is an unofficial Google product the other one is an official Meta product.

In Robyn there is a Ridge Regression, the Google Lightweight is a Generalized Linear Regression.
In Robyn you can't get statistical significance of coefficients, with the Lighweight you can. Here on this Stack Question you can find why

This is what you have in Robyn:

In Robyn Adstock-Carryover Effects are modelled trough a Weibull Distribution or a Convergent Geometric Series, while in Lightweight MMM you have or a Convolution or a Geometric Series.

On this point I am still concerned on having three different ways in Lightweight MMM to model adstock-carryover effects, because it seems in that way that are three different things.
Instead is the same phenomenon just treated differently from a mathematical perspective.
This can be misleading for a junior analyst.
From github Lightweight repo:

For both libraries I feel a strong lack of the impact of price elasticity in the sales attribution and consequently price promotion. Don't get me wrong, you can model them, but both documentations doesn't stress this point properly. I noticed that mistake with a lot of modeller in the Robyn user group.
From Lightweight MMM

Both uses the hill function for diminishing returns, a quite complex formula that was used in Jun, Shobowale, Koehler and Case (2012) to describe the relationship between unique reach and Gross Rating Points or TV Impressions.
I am pretty sure Robyn guys studied the Google paper  "Challenges And Opportunities In Media Mix Modelling" or the other Google Paper both co-authored by David Chan.

In general the goal is to understand when the Ad spend is no more efficient. If you show something like that to a client he will understand better.
The hill function assumes also the possibility of zero effects from the media activity, which is something I am still skeptical.

This are some of the key aspects from a modeller perspective, didn't have the time to investigate how on the same dataset how different results can be based on the optimization engine in the back-end.
Hope it helps and there are no big mistakes from my side.

Just as additional note.
At the moment you can run Lightweight MMM on colab, but the same is not possible for Robyn, but you can still run Robyn on Vertex AI


ISPN-6039 NonTxBackupOwnerBecomingPrimaryOwnerTest.testPrimaryOwnerCh…

…angingDuringPutOverwrite random failures
https://issues.jboss.org/browse/ISPN-6039
basically, don't block the put for state transfer :)
ps. I've enabled FunctionalNonTxBackupOwnerBecomingPrimaryOwnerTest.testPrimaryOwnerChangingDuringRemove again (see comment https://issues.jboss.org/browse/ISPN-6040)

hmmm a lot of failures... Could you please look at them and/or rebase this PR?

@slaskawi I can rebase the PR but the failures aren't related to the PR. none of the failures uses BlockingInterceptor

Ok then, integrating...

Integrated, thanks!


build: upgrade rust version to 1.75.0

What was wrong?
We were using a bit stale version of rust in our tools (1.71.1).
I noticed this because on my machine I was using the latest stable version (1.75.0) and clippy was complaining about some issue that weren't caused by me. After a bit of digging I realized that those were checks introduced in recent versions.
How was it fixed?

Updated the rust version in all places that I could find
Fixed clippy warnings

To-Do

[ ] Clean up commit history and use conventional commits.

@ogenev Does that mean we can get rid of using the async_trait library?

@ogenev Does that mean we can get rid of using the async_trait library?

I think so


Add test to ensure dict with enum keys are encoded properly

The issue raised in #829 seems to have been fixed in PR #868
https://github.com/BeanieODM/beanie/blob/0b8d50ba1fff44d7b92d726a1376619cacb4c5fd/beanie/odm/utils/encoder.py#L134-L138
Only the relevant test was missing to ensure the regression was adequately addressed. This PR adds that test.

raised as issue to pydantic

It's good that you're raising this issue in Pydantic, @07pepa. I appreciate that.
This PR, however, is not adding any specific encoding logic to overcome what may be possible Pydantic limitations. It's only adding a test to confirm that the existing Beanie behaviour that had regressed in between 1.23.6 -> 1.24.0 has been fixed.

actualy this is mistage

actualy this is mistage

@adeelsohailahmed  resolve conficts please

@adeelsohailahmed resolve conficts please

I don't see any conflicts.

Thank you for the reviews


GracePeriod in client.Drain() should be configurable

Describe the feature request
In client.Drain() the value of GracePeriodSeconds is hard coded but should be configurable via an environment variable. The default should stay -1 if no other value is set.
Why do you personally want this feature to be implemented?
Letting the Pod decide how long the GracePeriod should be is perfectly fine for production environments in order to not forcefully delete any workload but for test environments this delays the rolling update a lot.
I have a lot of pods with different GracePeriod configurations on a large amount of nodes and thus I would like to increase the rolling update speed by setting a smaller GracePeriod. Setting a small GracePeriod introduces the risk of forcefully killing Pods before they terminated gracefully but one might be totally fine with such behaviour in explained circumstances.
How long have you been using this project?
No response
Additional information
I would be happy to implement this feature myself during the next weeks if @TwiN is fine with the proposal

Released in v1.5.0


DS 2 pre-trained model epoch numbers?

On page: https://nvidia.github.io/OpenSeq2Seq/html/speech-recognition/deepspeech2.html
For WER=6.71%, it says "a pre-trained model which was trained for 200 epochs". However, in the given configuration file "ds2_large_8gpus_mp", it uses 8 GPUs but "num_epochs": 50.
I tried 50 epochs with 4 GPUs and got WER=8.5%. Wondering if the reference WER 6.71% was from 200 epochs or 50 epochs?

We trained the model for 200 epochs using 8 GPUs.

Thanks for the information.


Add support for custom Docker daemon configuration

Docker daemons can be configured with a JSON file. Support for custom data-roots is particularly valuable due to the performance impact of storing Docker layers on performant storage. This commit adds support custom JSON configuration along with the proper SystemD ordering for data-root when it is specified. The assumption that data-root may be provided by the local_ssd_filesystem option is made, but this dependency is soft (After) and does not impose a strict requirement. First commit for Docker daemon config support
Submission Checklist
NOTE: Community submissions can take up to 2 weeks to be reviewed.
Please take the following actions before submitting this pull request.

Fork your PR branch from the Toolkit "develop" branch (not main)
Test all changes with pre-commit in a local branch #
Confirm that "make tests" passes all tests
Add or modify unit tests to cover code changes
Ensure that unit test coverage remains above 80%
Update all applicable documentation
Follow Cluster Toolkit Contribution guidelines #

This option is being deliberately submitted without enabling it on the A3 blueprints (to verify it works without enabling it). We will then add it to the A3 blueprints in a separate PR.


Most demo links don't work

Most of the demo links to apps like chat, fog etc don't work

you can use rawgit just like this

origin: https://github.com/firebase/firebase-angular-starter-pack/blob/master/Crowdsourced-fog/index.html
viewer: https://rawgit.com/firebase/firebase-angular-starter-pack/master/Crowdsourced-fog/index.html

bookmarklet: replace github with rawgit and remove /blob/ or /raw/
javascript:(function(){window.location=window.location.toString().replace(/github\.com|raw.github\.com|raw.githubusercontent\.com/,"rawgit.com").replace(/\/blob\/|\/raw\//,"/")})();


Popover outsideClick is not fully compatible with ngIf

Bug description:
When I click a button, inside a popover with outsideClick as trigger, and the button disappears after the click using ngIf, the popover is closed.
Steps:

Create a popover with outsideClick as trigger.
Add a clickeable element (button, a, etc) with an ngClick inside the popover HTML.

hide me!

The element should disappear using ngIf.

hide me!

The use clicks the element inside the popover.
The popover is closed.

Link to minimally-working plunker that reproduces the issue:
http://plnkr.co/edit/i9FN34lNUNhpOhcCf2ut?p=preview
Version of Angular, UIBS, and Bootstrap
Angular: 1.5.7
UIBS: 2.1.3
Bootstrap: 3.3.6
Research
I made some research and I found why the popover is disappearing. Inside src/tooltip/tooltip.js [line 483] there is this function:
function bodyHideTooltipBind(e) {
    if (!ttScope || !ttScope.isOpen || !tooltip) {
        return;
    }
    // make sure the tooltip/popover link or tool tooltip/popover itself were not clicked
    if (!element[0].contains(e.target) && !tooltip[0].contains(e.target)) {
        hideTooltipBind();
    }
}

This expression is false
tooltip[0].contains(e.target)

The ngIf removes the element (button, a, whatever) from the HTML before the bodyHideTooltipBind is executed, so the element is not part of the popover-content HTML, so this is an outsideClick and the popover is closed.

If you know how can I fix this, please tell me and I will try to send a pull request.

This appears to work for me: http://plnkr.co/edit/UIM4hgSmSmB1toTSWwjw?p=preview - does this not work?

Yes, this work. Thanks for the provisional solution!
But do you consider that this workaround is enough? Do you think that it would be a good idea to implement a fix inside the framework?
In the other hand, I don't find any documentation for this, maybe should be in a known issues page in the wiki.

I need to do a deeper look, but that should be an adequate workaround for now.


[SPARKR][BUILD] AppVeyor change to latest R version

What changes were proposed in this pull request?
R version update
How was this patch tested?
AppVeyor

Test build #80291 has started for PR 18856 at commit 2449e7e.

@felixcheung, BTW, I think optionally we could also consider setting it 3.4.0 a bit more conservatively for now. There was a rather minor problem for using the latest version (see https://github.com/apache/spark/pull/15709) due to sync'ing issue of R version, which was fixed within few hours outside Spark IIRC. Either way is fine to me.

It's the latest but it's not new - 3.4.1 was release a month ago. I think there shouldn't be the sync problem.

Test build #80291 has finished for PR 18856 at commit 2449e7e.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80291/
Test PASSed.

Ah, I meant, the PR I linked actually describes a case when we were using 3.3.1 but it was broken after 3.3.2 release. The reason was, 3.3.2 was released but 3.3.2 for Windows was not synced (but was synced after few hours).
The script we are using checks if it is latest or not and adds ../old/.. part to build the download link correctly. So..

Check given version is latest via https://rversions.r-pkg.org/r-release

If so, don't add ../old/.., e.g., https://cran.r-project.org/bin/windows/base/R-3.3.1-win.exe
if not, add ../old/..,  e.g., https://cran.r-project.org/bin/windows/base/old/3.3.1/R-3.3.1-win.exe

But, the issue was, https://rversions.r-pkg.org/r-release gives 3.3.2 after the release but 3.3.1 for WIndows was not in old repo as the latest.

To cut this short, it was ...

3.3.2 release for Linux only
Gives the latest as 3.3.2 and the download link for 3.3.1 becomes windows/base/old
3.3.2 release for WIndows yet
3.3.1 is still not in windows/base/old but windows/base
Failed to download

@felixcheung, actually, should we try to replace http://rversions.r-pkg.org/r-release in line 29 to http://rversions.r-pkg.org/r-release-win?  I referred https://github.com/krlmlr/r-appveyor/commit/01ce943929993bbf27facd2cdc20ae2e03808eb4

Will try to open a separate PR. LGTM.

Merged to master.

Right, this PR isn't changing how you have made the version to test with sticky - we won't still pick up the latest automatically, which is fine for now.
But we should bump (manually) to the later/latest version in one of the test matrix to make sure it works.

I agree this PR should be okay alone, which fixes what it targets for now and I realised my suggestion is rather an extra bit. I opened another one for replacing r-release to r-release-win.


[WIP] Add virtual NUMA support

Added NUMA topology and processor related HCS schema definitions
New annotations "io.microsoft.virtualmachine.computetopology.numa.defaultnuma"
and "io.microsoft.virtualmachine.computetopology.numa.virtualnodecount" to be
able to specify the number of virtual NUMA nodes to assign the UVM.
New UVM method hostProcessorInfo that will call HcsGetServiceProperties
and return the hosts processor topology as well as the total number of
logical processors on the system. This is to fix the bug where we were
incorrectly capping a users requested CPU count on > 1 NUMA node systems
as runtime.NumCPU only sees the LPs from the processor/node it is run on.
Refactored checking for validations/unsupported configurations when creating
the UVM doc.
Added LCOW test for NUMA validation.

Signed-off-by: Daniel Canter<EMAIL_ADDRESS>

I was given access to a dual socket machine today to test this with but I'd like to list some things related to this work.

I'm not sure what the cutoff for querying for Processor information is but it seems like ProcessorTopology works on RS5 while Processor does not. We can still effectively get the hosts NUMA config and LPs with just ProcessorTopology but it's a little more cumbersome.
Listed above but for adjusting the users requested CPU count for process containers we still have the issue of capping them below the amount but we need a separate way to query for this information to accommodate minroot setups.
Need to find a nice test for WCOW thats available on nanoserver.
There might be more validation we can do input wise in regards to high VNode counts being passed in. For example HyperV will by default try to divide resources evenly across nodes. So in an ideal scenario with 2 physical nodes each with 12 cores and the user requests 2 VNodes (or just uses the hosts count with the defaultnuma annotation), the split would be exactly the same as the host. If they specified 3 VNodes the split would be 4 for each VNode and so on and so forth. For higher VNode counts we can end up with compute-less nodes with no VPs assigned, but this is a completely valid setup as far as Hyper-V's concerned. I don't think there is a scenario where we would end up with memory-less nodes as far as I can tell.
Maybe expose lower level options to configure the per node settings and resource constraints instead of just exposing node count? I can see this being very useful but not sure how this would play into our annotation setup.


Revert "fix ckv_docker errors reported by checkov"

This reverts commit a0e6a1a409d9f690f993ca4c596b37f3f4ceb205.
Caused
+ podman run --rm --env COSIGN_PASSWORD=dIsmAfF3VRRMLtkU --volume /tmp/tmp.4MDvmF5mi3/credentials/manifests/compute/tekton-chains/tmp:/workspace:z --workdir /workspace --entrypoint /usr/bin/cosign quay.io/redhat-appstudio/appstudio-utils:eb94f28fe2d7c182f15e659d0fdb66f87b0b3b6b generate-key-pair
        Error: crun: writing file `/proc/947/setgroups`: Operation not permitted: OCI permission denied
command terminated with exit code 126

per https://redhat-internal.slack.com/archives/C032EJ007C0/p1674243243293849 with @Roming22 we don't yet have a way to take changes to the images and have those images tested in a PR
Currently the tests are always using the main tag of our images


Country only set address on map

"Set to my entered address" does not work when only country is selected. The country field uses country codes (ex: nl,uk,de etc). The google maps api does not know how to convert those. My suggestion would be to use the text of the selected option and send that to the google maps api. This seems to work better.

I cannot reproduce this problem. On the contrary, country codes work better than full names:
Code (relevant):
curl 'http://maps.googleapis.com/maps/api/geocode/json?sensor=false&address=address_country=nl'
{
   "results" : [
      {
         "address_components" : [
            {
               "long_name" : "Netherlands",
               "short_name" : "NL",
               "types" : [ "country", "political" ]
            }
         ],

versus (irrelevant):
david@ginger03:~$ curl 'http://maps.googleapis.com/maps/api/geocode/json?sensor=false&address=address_country=Nederland'
{
   "results" : [
      {
         "address_components" : [
            {
               "long_name" : "41",
               "short_name" : "41",
               "types" : [ "street_number" ]
            },

I mentioned Google Maps Api (assumption) was used. But openstreetmap.org is used when only a country is used as parameter.
Api calls for openstreetmap:
http://nominatim.openstreetmap.org/search?format=json&limit=1&addressdetails=0&q=nl
http://nominatim.openstreetmap.org/search?format=json&limit=1&addressdetails=0&q=Nederland
The google maps api call does indeed not understand "Nederland" instead of "nl".
Should we send both parameters to mod_geo notify and pass different queries to googlemaps or openstreetmap?

Yes, that seems the best way; mod_geo already has access to all address parameters. Can you change this PR?

@ddeboer Changed the pull request so both services have their own set of parameters.

Looks good to me. @arjan Do you agree? If so, can you squash and merge this?


Rounded out command line switches and usage for compat with old cppwinrt.exe

No idea what happened with the PR - committing and will address any build breaks later.


Use jumps to change the order of steps being processed

AS A Service Manager
I WANT TO define in which order DSL steps should be initiated
SO THAT I could do it without moving whole step blocks within YAML file
Expected output: https://github.com/buerokratt/Ruuter/blob/main/samples/backlog/jumps.md

"end" is the keyword used to stop DSL service execution

Linked PR https://github.com/buerokratt/Ruuter/pull/39 where initial jumping logic was implemented


Feature Charts: Add Longhorn 1.6.1 Chart in 2.8.x

Longhorn has an upcoming release 1.6.1 which will be released in March 2024 so we need to add this new release in the feature charts along with Rancher 2.8.x. We will probably need to make this OOB.
Add 1.6.0 LH chart in feature chart 2.8.x - Cluster Manager/Rancher Marketplace
Cc: @cchien816 @gunamata @innobead @khushboo-rancher @yangchiu @PhanLe1010 @rohitsakala @mantissahz
Related issue for 2.7.x: https://github.com/rancher/rancher/issues/44861

The chart was released on April 17th, 2024.


Allow datetime fields in form be empty by default

Now all forms with empty datetime fields are rendered with filled datetime.now values by default. In some places it is not required and should be empty.

removed default datetime value in https://github.com/sanoma/django-arctic/pull/253.
But still needs to be investigated how to specify default value if it is needed. Parameter initial= for form field doesn't work for now.


Added initial TT implementation

Fixes #16
Added a Trusted Types implementation to the ISE Security Bundle. This does not include a full CSP module. The idea for this, after discussion in the ISE group, was to implement TT in a way that would be compatable with other bundles (i.e nelmio security bundle), and would not lead to an overwritten or broken CSP header.

[x] Tests pass
[x] Appropriate changes to README are included in PR

The decrease in test coverage is due to extending the test suite to include some of the base handlers. The logic in these handlers are fully covered by the tests, the only lacking line is a Symfony core line for registering the subscribers. I can add this in but I dont think it gives any value, and only serves to bloat the tests with un-needed line coverage


!./run_local.sh tf resnet50 cpu --scenario Server  - keep getting in valid result

I don't know why Scheduled samples per second is -nan and other invalidation

If this is the notebook example - it uses a handful of images which is not good enough to pass any checks - just meant to give people an idea. To make any of those checks pass you'd need to use the real dataset.

Dear gushmue what do you mean about "notebook example"?

I meant GettingStarted.ipynb. The dataset and settings used in there won't get you VALID results. For that you'd need to use the real datasets decriebed in the readme and you'd need to remove the --time and --count options.


perform rounding in formatFloat()

Ideally, "1" and "1.0" would be printed below:
    print(formatFloat(0.99, 0, 0)); // 0
    print(formatFloat(0.99, 1, 1)); // 0.9

For reference, Java's DecimalFormat uses half-even rounding by default.

For the record, the current truncating behavior is mentioned in the documentation, if only in passing. When we change it, we should take the opportunity to make it a bit more explicit and add examples where rounding makes a difference – i. e., not just use the digits 0 to 4 :)

fixed by #6440


The world view API

This adds methods for manipulating BiomeAreas and BlockVolume, as well as Extents. This include changing bounds, transforming views and copying data. More features will be added later (in future PRs) for iteration and filling (with support for multithreaded operations).
Implementation: https://github.com/SpongePowered/SpongeCommon/pull/102

It's complete, including the implementation.

Anyone against me merging this tonight? Including the implementation.


Port tf-random & QuickCheck

QuickCheck depends on tf-random which fails to install because:
eta: panic! (the 'impossible' happened)
  (Eta version 0.0.5):
	tcCheckFIType: Unsupported calling convention.
  ccall unsafe "static skein.h Threefish_256_Process_Block"

quite a few libraries depend on QuickCheck which makes them in turn unusable.

tf-random has a single C FFI import that needs to be shimmed with a Java import.
Btw, which libraries do you need that are unusable?

I stumbled upon it by trying to depend on megaparsec.

Sure. Will let you know once this issue is resolved.

Aweome, thanks! I'm really excited about eta.

Had a look at both the C code and the relevant java libraries for tf-random.
The code for java can be likely adapted from here: https://github.com/bmalkow/java-bc-threefish/blob/7e70464d70489869209a73ab9d5a9688d6e7cf3c/src/org/bouncycastle/crypto/engines/ThreefishEngine.java#L232

@psibi Finished this. It should now be working.

Great, thanks a ton!


retrieve store from single get

I'm wondering if the following code could be achieved differently (it works but probably needs refactoring).
Case:

project has many features
2 routes from router: projects list and project show
project API request includes nested features
since there are 2 routes, I would prefer to retrieve the store object directly in the project show rather than from the projects list.

<template>

{{ currentProject.name }}

    {{ feature.name }}

</template>


Add string syntax and parse word

note: you have to escape \ with \\ and " with \"
note: print now checks whether the atom contains illegal characters and prints it as a string if it does

parse word: it has the signature of [atom offset -> atom new-offset object] to treat atom as a string and start reading an object at offset in the string and increment the offset on the stack and add the read object

I'd like to keep Forsp a minimalist language. To me it's more interesting that way than a full-featured one.
If people are really interested in adding a lot of core features, then I might suggest adding some sub-directory like "full-featured" that has much more. Or... creating a separate repo with different goals. Personally, I'm not sure I'm interested in building something out, but I encourage others to try if that interests them.
In response to your implementation: you probably want strings to be another object type, not an atom. It's implied that atoms are interned, and "eq" will work on them.

Maybe a directory named the-non-minimalist-parts?
I too would like to see what enhancements to this would look like, while also wanting the core to stay minimalist.
But having worked on a fork of another "minimalist-but-powerful" idea, es-shell https://github.com/pmarreck/es-shell (although sadly not recently- new job and new kid have halted my dev free time), I am unfortunately familiar with the complexity that can emerge when you're trying to merge your idea with non-minimalist changes to upstream. ;)
Not sure if a tweak to make things more "plugin-friendly" would make sense? Some mechanism where someone could add things like 1) new tokens, 2) new elements of state, 3) new root functions (of course), 4) new primitives and low-level primitives 5) new defines. I'm not a C dev by day, so I don't know what the "prior art" is with that idea... The goal would be to NOT (ideally) have to overwrite any of your existing core functions, by adding hooks somehow.
In theory, this should not be terribly complicated to implement, since the only state you really have to worry about is... a stack. Right? (Well, and the parser, and evaluator, and any new functions required...)
To test the mechanism out, you could try implementing a new feature (like... adding string syntax) using it... or... even writing a test for it... or something... (OK, what is it with C devs and a total lack of any test coverage? Unit testing is pretty much de-facto in most other languages at this point.)
Just some thoughts, sorry to stick them on a random PR! As a fan of Forth (which I started writing an interpreter for in Elixir, to the point that I can run things like a factorial function https://github.com/pmarreck/elixir-snippets/blob/master/rpn_forth_thing.exs) and Lisp (at least in theory... even if I stumbled in CS212 at Cornell, which was Scheme-based at the time), this project is nifty!

Well I was thinking just duplicating/forking the source elsewhere. The implementation is very minimalist/proof/of-concept. If you actually wanted to build out a real and practical language you'd have to do much more. E.g. Forsp doesn't even have a GC at the moment. And you'd want some semblance of imports/modules. And lots of additional types: strings, arrays, hash tables, etc. Then you might be adding threads and synchronization primitives. You probably would also want to do a compiled version and generate actually decent machine code. Trying to jam all of these (and more) into a minimalist language is, ummm, going to be a failure. Rather than doing one thing well, it'll be doing two things badly.
Personally, as stated above, my interests are around minimalist and bootstrapping languages. So, I don't have much drive to build out these kinds of things.
If a group of people really want to go down that road, I'll give you my blessing but it's probably a fork or a variant of some kind so the minimalist can remain here.
I would also suggest a strong vision/planning. You don't want to just jam random features in. Instead you should play around with like 5+ approaches and only accept the feature in when it actually feels right. (E.g. I wouldn't say the string impl here "feels" right).

Yeah this string impl is just a hacked together way to generate atoms with otherwise invalid content.
I thought about it and I had an idea. While you could implement strings in the language as a list of integers, that would waste a lot of spaces. That's why I think we should add a string type (could be just a plain array of bytes) and some basic functions like string-length, make-string (creates new zeroed out string of specified length), string-peek and string-poke (to copy between strings). Everything else including stuff like parse can be implemented in the language itself. Having parse allows you to create a kind of module system since you can dynamically load code.
I'm open for suggestions.

I've added my POC implementation for my suggestion

lol @ last commit message. Whether or not original author agrees with this direction, always fun to see developer enthusiasm. But where are the tests? 😉
(also, is Forsp incredibly fast at this point due to being so simple? I want to play with it...)

yeah I forgot I still had this PR
I'll close it now and work on it a bit more until it's polished
(the GC is kinda broken with closures so the forsp.fp fails)


get rid of gvm alias related code

To raise a new Pull Request, the following prerequisites need to be met. Please tick before proceeding:

[x] a conversation was held in the appropriate Gitter Room.
[x] a Github Issue was opened for this feature / bug.

This closes #607

@marc0der I've updated this PR and made two other PRs on website and sdkman-hooks. Please let me know if there's anything more to this


Slim down Mojos

Currently nearly all business logic is contained in Mojos, which can not be easily tested and are tightly couple to the Maven platform.
These Mojos should be stripped down to the bare minimum and all business logic should go into dedicated services.
There are two kind of services:

depending on the Maven API (e.g. MavenProject etc)
independent of the Maven API (no dependency on any Maven artifact)

As much as possible should be moved to Maven independent modules. They are easier to test and could be reused (e.g. for other build systems like gradle).
Ideally the Mojos's task is then to collect the configuration (via  @Parameter injection) and then call into these services.
This backend services could be realized as Plexus components (we already have some)

@rhuss I'd like to start working on this. I've already started doing some refactoring in #815, so I'll start this from that branch (hoping that the watcher abstraction will be merged before this one).

ok. I finally make a d-m-p release this evening. always surprised how fast a backlog of issues can pile up ;)


Using the back gesture anywhere outside a manga chapter closes or crashes the app.

Steps to reproduce

Finish reading a chapter.
Click back to go to the chapters list.
Press back to go to the manga list
The app closes or crashes.

Expected behavior
It would go back to the previous manga list instead of closing.
Actual behavior
It closes or crashes when you use the back gesture outside of a chapter.
Crash logs
NekoLog.txt
Neko version
2.9.2
Android version
Android 13 Beta June 5th
Device
Google Pixel 4a 5G
Other details
This bug has been around the past two or three versions.
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open issue.
[X] I have written a short but informative title.
[X] I have tried the troubleshooting guide.
[X] I have updated the app to the newest version Latest.
[X] I have filled out all of the requested information in this form.

This is a known issue with Android 13 and with how j2k has currently implemented.

Do you still have this issue?

should be resolved


pythonPackages.cysignals: 1.7.1 -> 1.7.2

Motivation for this change
Just a minor version bump.
@GrahamcOfBorg build python2Packages.cysignals python3Packages.cysignals
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build python36Packages.cysignals python27Packages.cysignals

Success on x86_64-darwin (full log)
Attempted: python36Packages.cysignals, python27Packages.cysignals
Partial log (click to expand)
Installing collected packages: cysignals
Successfully installed cysignals-1.7.2
/private/tmp/nix-build-python3.6-cysignals-1.7.2.drv-0/cysignals-1.7.2
post-installation fixup
strip is /nix/store/7ddbq63v97nk8gkbf7gcsfmby37h6gbl-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/yzscjlckn2256hpvwp8fjj8j3csgrsfz-python3.6-cysignals-1.7.2/lib  /nix/store/yzscjlckn2256hpvwp8fjj8j3csgrsfz-python3.6-cysignals-1.7.2/bin
patching script interpreter paths in /nix/store/yzscjlckn2256hpvwp8fjj8j3csgrsfz-python3.6-cysignals-1.7.2
wrapping `/nix/store/yzscjlckn2256hpvwp8fjj8j3csgrsfz-python3.6-cysignals-1.7.2/bin/cysignals-CSI'...
/nix/store/yzscjlckn2256hpvwp8fjj8j3csgrsfz-python3.6-cysignals-1.7.2
/nix/store/73vpzizs1lzwdpxsn50wnb52fks29vf8-python2.7-cysignals-1.7.2

Success on x86_64-linux (full log)
Attempted: python36Packages.cysignals, python27Packages.cysignals
Partial log (click to expand)
shrinking /nix/store/p9dsyfc1h2lkrd4ml1kskfgv311dyyir-python3.6-cysignals-1.7.2/lib/python3.6/site-packages/cysignals/alarm.cpython-36m-x86_64-linux-gnu.so
shrinking /nix/store/p9dsyfc1h2lkrd4ml1kskfgv311dyyir-python3.6-cysignals-1.7.2/lib/python3.6/site-packages/cysignals/pysignals.cpython-36m-x86_64-linux-gnu.so
shrinking /nix/store/p9dsyfc1h2lkrd4ml1kskfgv311dyyir-python3.6-cysignals-1.7.2/lib/python3.6/site-packages/cysignals/tests.cpython-36m-x86_64-linux-gnu.so
strip is /nix/store/4qvrxzxa535y8304mk195x50b6p9607d-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/p9dsyfc1h2lkrd4ml1kskfgv311dyyir-python3.6-cysignals-1.7.2/lib  /nix/store/p9dsyfc1h2lkrd4ml1kskfgv311dyyir-python3.6-cysignals-1.7.2/bin
patching script interpreter paths in /nix/store/p9dsyfc1h2lkrd4ml1kskfgv311dyyir-python3.6-cysignals-1.7.2
checking for references to /build in /nix/store/p9dsyfc1h2lkrd4ml1kskfgv311dyyir-python3.6-cysignals-1.7.2...
wrapping `/nix/store/p9dsyfc1h2lkrd4ml1kskfgv311dyyir-python3.6-cysignals-1.7.2/bin/cysignals-CSI'...
/nix/store/p9dsyfc1h2lkrd4ml1kskfgv311dyyir-python3.6-cysignals-1.7.2
/nix/store/4p6939k8k6zar56290nmw4jfx2qhpyv1-python2.7-cysignals-1.7.2


Draft keynote preview

Post is going live at 8am - http://citycampnc.org/2017/08/30/the-secrets-to-urbanism-and-open-data/
cc @natalienoel1 please help promote via social media

Got it!
On Tue, Aug 29, 2017 at 11:01 PM, Jason Hibbets<EMAIL_ADDRESS>wrote:

Closed #632 https://github.com/codeforraleigh/NCOpenPass/issues/632.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/codeforraleigh/NCOpenPass/issues/632#event-1226828551,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AaBwV-WqWBmw3oOvxLhpMSLvIevCveCFks5sdNCOgaJpZM4PGaCT
.

--
Natalie Noel
Marketing Manager
H/Q Raleigh & Charlotte
www.hq.community
https://twitter.com/HQRaleigh https://www.facebook.com/HubRaleigh
http://www.linkedin.com/company/2783956?trk=NUS_CMPY_TWIT
http://instagram.com/hqraleigh
Is your entrepreneurial team growing? Check out our flexible and
affordable suites at HQ Capital Club! Contact us at<EMAIL_ADDRESS><EMAIL_ADDRESS>or call us at<PHONE_NUMBER> <(919)%20300-6917> to
schedule a hard hat tour! Make history happen!
Check out these renderings of our new space at the Capital Club Building
https://drive.google.com/file/d/0B5Hu4xKF4CvaSHN2eG1selZwN00/view?usp=sharing
!


Add Service Binding for Kubernetes package to TCE

Feature Request
Problem
Today in Kubernetes, the exposure of secrets for connecting application workloads to external services such as REST APIs, databases, event buses, and many more is manual and bespoke. Each service provider suggests a different way to access their secrets, and each application developer consumes those secrets in a custom way to their workloads. While there is a good deal of value to this flexibility level, large development teams lose overall velocity dealing with each unique solution. To combat this, we already see teams adopting internal patterns for how to achieve this workload-to-service linkage.
https://servicebinding.io specifies a Kubernetes-wide specification for communicating service secrets to workloads in an automated way. It aims to create a widely applicable mechanism but /without/ excluding other strategies for systems that it does not fit easily. The benefit of Kubernetes-wide specification is that all of the actors in an ecosystem can work towards a clearly defined abstraction at the edge of their expertise and depend on other parties to complete the chain.
Service Bindings for Kubernetes
Service Bindings for Kubernetes (GitHub - vmware-tanzu/servicebinding: Service Bindings for Kubernetes) implements the  Service Binding Specification for Kubernetes v1.0.0.
How does it work?
This implementation provides support for:

Provisioned Service
Workload Projection
Service Binding
Direct Secret Reference
Role-Based Access Control (RBAC)

What is not implemented?
The following are not implemented:

Workload Resource Mapping
Extensions including:

Binding Secret Generation Strategies

Feature Ask
The ask is to make GitHub - vmware-tanzu/servicebinding: Service Bindings for Kubernetes available as a Tanzu package as part of Tanzu Community Edition.

@kartiklunkad26 is this something the app toolkit team would be doing?

servicebinding releases a package! This would be a very quick and easy to implement I think.

@stmcginnis one of the developer-focused teams (cc: @atmandhol)
App toolkit team will also bring this package into their meta-package (cc: @dambor)

App toolkit team will also bring this package into their meta-package (cc: @dambor)

When is it planned to add the service binding within the app-toolkit ?

@joshrosso @seemiller The team is waiting on approval from TCE engineering team to start work on this. Please let us know if this will be reviewed.


See the differences between a release and the main branch

The release page shows a changelog for every release but doesn't let us see the diff between the latest release and the main branch (i.e. the next release).
More precisely, I'd like to have new links on the release page:

Unreleased commits (commits in main missing from latest release): https://github.com/$user/$repo/compare/$tag...master
Hotfixed commits (released commits but not in main): https://github.com/$user/$repo/compare/master...$tag
"Real" files changes between the latest release and main: https://github.com/$user/$repo/compare/$tag..master

As I'm writing this, I notice that this may not be relevant for all git flows (e.g. the ones where the main branch always match the latest release) but maybe there is a clever way to make this feature helpful to everyone?
Some examples of release pages:

https://github.com/Mottie/GitHub-userscripts/releases (simple raw tags)
https://github.com/sindresorhus/refined-github/releases (full featured releases with title and description)
https://github.com/phoenixframework/phoenix/releases (collapsed releases)
https://github.com/elixir-lang/elixir/releases (pre-releases)

Currently, I use a simple userscript to do that but it has some issues and I'd love to see this feature baked in Refined Github:

// ==UserScript==
// @name         Get next release changelog
// @match        https://github.com/*
// @grant        GM_registerMenuCommand
// ==/UserScript==

// Caveats:
// - only works for tags (not full featured releases).
// - doesn't work with "folded" releases (e.g. https://github.com/phoenixframework/phoenix/releases)
// - hardcoded "master" branch.

const goToCompareUrl = (getRange) => {
  const latestTag = document
    .querySelector(".release-entry .release-entry .commit-title a")
    .innerText.trim();
  window.location.href = window.location.href.replace(
    /\/releases/,
    `/compare/${getRange(latestTag)}`
  );
};

GM_registerMenuCommand("Files changes (..)", () => {
  goToCompareUrl((tag) => `${tag}..master`);
});
GM_registerMenuCommand("Unreleased commits (...)", () => {
  goToCompareUrl((tag) => `${tag}...master`);
});
GM_registerMenuCommand("Hotfixed commits", () => {
  goToCompareUrl((tag) => `master...${tag}`);
});

I think the userscript you have makes the most sense, but let's see what others think.

👍

GitHub already has one of those links in 1 click and let's you pick one of the others in 2 clicks. $tag is arbitrary and dependent on the repo so if you have to pick it you can just do so with the dropdown GitHub provides next to each release.


Stuck need an timeout update.

Scanning got stuck.
=========================================================================
[>>] Crawling URLS from : WaybackMachine, AlienValut OTX, CommonCrawl ...
=========================================================================
[>>] [Total URLs] :  26441
[>>] [Potentially Vulnerable URLs] :  836
[>>] [Unique Potentially Vulnerable URLs] :  114
=========================================================================
[>>] [Total URLs Scanned] : 110/114 | [>>] [Total Error Encountered] : 16

It happened when ran in while loop, you should add an option to add a list of the domain, by the way, great tool 👍


PT Use Weights

Addresses #39 #47 #48
changes:

all Labels have an edgeWeight that is incremented as used to check domination
PtEdges are now converted into weights using PtWeighting. for now this is a simple multiplier but maybe we could do something more sophisticated

TODO:

make sure it works
mess with the various knobs to see what makes good routes

okay testing this out we have good news and bad news
good news: the bike-only route matches the route from the bike2 endpoint, and it's pretty good! nice to see usable bike routes in bikehopper
pt endpoint:

bike endpoint:

bad news: something broke and it's not getting onto public transit at the right time. also it's doing random small trips for reasons i'm not sure about. will have to dig into this tomorrow, bet it's something with currentTime being used to enter the public transit graph


Update openai models

Still need to update llm_handler. Let's run the benchmarks before we actually merge.
Pull Request Checklist

[x] Documentation has been updated, or this change doesn't require that

The results look great. We should merge:
http://benchmarks.mentat.ai/exercism-javascript-results-20240126155034.html
http://benchmarks.mentat.ai/exercism-python-results-20240126155034.html


tests failing because of uninitialized constant GC::INTERNAL_CONSTANTS

I'm having trouble getting the provided tests to run without failures. It seems the memory profiling bits might be missing something in my environment. Has anyone else experienced this issue?
Failures:

Ebooks::Model consumes, saves and loads models correctly
Failure/Error: report = MemoryUsage.report do
NameError:
uninitialized constant GC::INTERNAL_CONSTANTS
./spec/memprof.rb:11:in `report'
./spec/model_spec.rb:28:in `block (2 levels) in <top (required)>'

Are you using Ruby 2.1+? That's when it was introduced

I may not have been! I'll try again with ruby 2.1+ thanks.

Got tests working fine by the way. Apologies for the delay in follow-up.