retrieveDynamicLink() is always null

URL: https://pub.dartlang.org/packages/firebase_dynamic_links
i'm using version 0.0.5 and on iOS.  i haven't ever been able to use:
final PendingDynamicLinkData data = await FirebaseDynamicLinks.instance.retrieveDynamicLink();
and get a non-null value for that data var.  that includes clicking on links i've generated in the firebase console, and thru buildUrl() and/or manually.  it doesn't matter if the app is open in the bg, closed, or coming out of a hard reboot.  and i've used both the simulator and a real device, and clicked on the link from sms and email.
generating links works as expected, and clicking on the link(s) always opens the app as expected (everything is configured properly in firebase and xcode).  to make sure i wasn't missing anything i reduced my app to this entry point:
void main() async {

  final PendingDynamicLinkData data = await FirebaseDynamicLinks.instance.retrieveDynamicLink();
  print('main link = $data');

  runApp(MaterialApp(
    home: new LinksTest(data: data),
  ));

and rendered LinksTest like so:
import 'package:flutter/material.dart';

class LinksTest extends StatelessWidget {
  var data;
  LinksTest({this.data});

  @override
  Widget build(BuildContext context) {
    return new Center(
      child: new Text(data.toString(), style: new TextStyle(color: Colors.white, fontSize: 14.0)),
    );
  }
}

no matter what i do, the app always opens but data is always null.

@blaneyneil: please file such request to the package's repository, in this case flutter/flutter. I'm moving this now.

This issue was moved to flutter/flutter#19506


Password mismatch wile unzipping

I have tried to make zip file with password protection. Zip creation succeeded but while unzipping the password is not matching anymore. I have tried to unzip with Finder on macOS Catlina(version 10.15.6) but still no luck.
let url = getDocumentsDirectory().appendingPathComponent("log.txt")
try fileBody.write(to: url, atomically: true, encoding: .utf8)
let zipURL = self.getDocumentsDirectory().appendingPathComponent("log.zip")
try Zip.zipFiles(paths: [url], zipFilePath: zipURL, password: "123", compression: .NoCompression) { (progress) in
print("zip progress: \(progress)")
}

Environment Informations:
Zip: 2.1.1
Swift version: 5.2
Xcode version: 12.2 (12B45b)
Tested On: iPhone X(iOS 14.0.1), iPhone 6s Plus(13.4.1)

We are also facing the same issue. Environment details are same.

same issue😓.

I ended up rewriting the library using SSZipArchive..

Hi Patrasupratim,
I had the same issue with Zip and I am trying to use SSZIpArchive and I haven't had any luck.
I am using swift 5.3 in Xcode 12.
Do you have a sample code using SSZipArchive for me to look at or the exact steps to follow to make it work.
Thanks,
Much appreciated

@ajc-1924 check this issue on SSZipArchive
https://github.com/ZipArchive/ZipArchive/issues/601

Hi @ajc-1924,
@sz-ashik440 pointed it right. It needs to disable the default AES encryption from other overloaded methods. Link is here -
https://github.com/ZipArchive/ZipArchive/issues/601#issuecomment-731170085.

Thank you @patrasupratim  and @sz-ashik440 for your replay. However, I still cannot unzip the file using SSZipArchive.
I am using swift 5.3, xocde 12. I am using the emulator. the file I am trying to unzip is password protected. this file gets unzipped properly in linux, windows as well as mac when I double click on the file it the default archive application tries to open ti. it prompts me for the password, I enter it and it gets successfully unzipped.
the file was zipped using AES encryption.
this is the call to the unzip method:
let success: Bool = SSZipArchive.unzipFile(atPath: srcURL.path,
toDestination: destURL.path,
preserveAttributes: true,
overwrite: true,
nestedZipLevel: 1,
password: !password.isEmpty ? password : nil,
error: nil,
delegate: nil,
progressHandler: nil,
completionHandler: nil)
and it fails here:
switch (zip->file_info.compression_method)
{
case MZ_COMPRESS_METHOD_STORE:
case MZ_COMPRESS_METHOD_DEFLATE:
#ifdef HAVE_BZIP2
case MZ_COMPRESS_METHOD_BZIP2:
#endif
#ifdef HAVE_LZMA
case MZ_COMPRESS_METHOD_LZMA:
#endif
err = MZ_OK;
break;
default:
return MZ_SUPPORT_ERROR;
}
I returns MZ_SUPPORT_ERROR
I Have been trying to make this work for days now with no luck.
I appreciate any help anyone can give me. I am trying to learn Swift.

Hi @ajc-1924,
I used following code for zipping the file -
let success = SSZipArchive.createZipFile(atPath: zipUrl.path,                                    withContentsOfDirectory: stagingDirUrl.path,                                    keepParentDirectory: false,                                    compressionLevel: zlib.Z_BEST_COMPRESSION,                                    password: password,                                    **aes: false,**                                    progressHandler: nil)
This way I can unzip in mac and windows as well. The thing is if the file is protected using AES, then there is need an unzip application written using the same library. Can you please try using  aes: false,.

Thank you again @patrasupratim and @sz-ashik440, the things is I  do not have controlled over the zipped file. I just need to unzip it. I know the zipped file is password is password protected and it was protected using the AES method. It can be unzipped on linux and windows using java libraries as well as 7zip and on mac it can be unzipped using the default unzip application. I need to unzip it using swift 5.3 using xcode 12 on ios.
regards,
I appreciate your input.

Any news on this issue...
I don't know if disabling AES when zipping using this library could help...
Thanks in advance.

same issue😓.


mopidy: use headlessGApps wrapper, cleanup meta.description

Description of changes

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

I deployed this on actual hardware and tested mopidy-youtube streaming with it. We have environment.noXLibs turned on.


observers: update on using trap

After the update to entrypoint and entrypoint wrapper the error code of the observers could be propagated (ci-tools/#3560, ci-tools/#3575), but changes to the observer script need to be made to run properly (see release/#43489 ) and the doc should be updated then.
/cc @bbguimaraes

/ok-to-test

/lgtm


Property File Validation

Integrate property file validation so that warnings or errors are reported prior to application startup.

This was implemented with the LFH processor consolidation/streamlining


Crash when calling upload()

Summary
If you call upload() in the SDK before queueing an event for processing, the SDK will throw a runtime exception.
Details
The uploader may be null when called on this line. https://github.com/mParticle/mparticle-web-sdk/blob/4740596cb9f19e968e74c4d3c103d50c846d0888/src/mp-instance.js#L577
This is because it has not yet been initialized within queueEventForBatchedUpload on the APIClient. https://github.com/mParticle/mparticle-web-sdk/blob/4740596cb9f19e968e74c4d3c103d50c846d0888/src/apiClient.ts#L39-L50
The uploader is also listed as a nullable type in its type definition. https://github.com/mParticle/mparticle-web-sdk/blob/4740596cb9f19e968e74c4d3c103d50c846d0888/src/apiClient.ts#L13
Proposed fix
// if your JS toolchain supports the optional chaining operator
self._APIClient.uploader?.prepareAndUpload(false, false);

// if your JS toolchain does not support the optional chaining operator
if (self._APIClient.uploader) {
  self._APIClient.uploader.prepareAndUpload(false, false);
}

Thanks for flagging this @paulrehkugler!.  Our JS toolchain does support the optional chaining operator.  We can make a ticket and add to our backlog, but since you have a proposed fix and since we are open source, you could also open a PR to get credit for it if you are comfortable with that :).  Let me know what you prefer.

I do think think that a workaround for not is wrapping the mParticle.upload() in the mParticle.ready function would also work for now before this gets worked on.  You can view docs here for an example: https://docs.mparticle.com/developers/sdk/web/initialization/#listen-for-sdk-initialization


Wheather or not we could get File(flow-rule.json) from fat jar in the spring-boot-sentinel project in the current version

Execuse me ,please!
Associate issue 428
I want to confirm wheather the spring-cloud-alibaba can support getting File from fat jar in the current version.That means we can't write our properties like this :
spring.cloud.sentinel.datasource.ds1=jar:D:/sentinel.jar!/BOOT-INF/classes/config/sentinel/FlowRule.json
class path resource cannot be resolved as enhancement in #91 #428

Refer: Sentinel Example Project.


Finetune Elli not to give discounts and not to talke much.

finetuned Elli by adjusting the temperature and  maxTokens.


feat: users table

This change also:

Makes the site text size slightly smaller. Previously, it seemed too big.
Adds a <TabsBar /> component.

Closes #370

@hashrock, would you happen to have any suggestions regarding my previous comment?

@iuioiua hmm I'm not familiar with table styling, so I'm not sure how to solve it.
just a workaround, but how about combine these columns?

@iuioiua hmm I'm not familiar with table styling, so I'm not sure how to solve it.
just a workaround, but how about combine these columns?

Done!


Can't use Google to log in

It throws an error message--"It looks like your browser is currently blocking popups. Please allow pop-ups to continue"--then opens an external browser tab that doesn't load:
https://www.notion.so/verifyNoPopupBlockerHtmlAndRedirect?redirectUri=https%3A%2F%2Fwww.notion.so%2Fgooglepopupredirect%3FcallbackType%3Dpopup%26redirectToAuth%3Dtrue

The popup doesn't work.
don't use the google login button, but type in your gmail instead in the bar below. Notion will send you a password to your gmail, which you can use to login.
Works for me on linux.


UI IMPROVEMENT IN "PERSONALIZED RECOMMENDATIONS" #1258

Fixes:  #(1258)
Description
so basically i just recreate the "PERSONALISED RECOMENDATIONS " SECTION  which gives the interactive interface for going  smoothly. This page deals with to collecting reviews from users regarding your website.  Transitions ,keyframes are used in these making it more interactive and the the things are unable to show in one picture just go and check  out  the transitions for hovering effects if you like the designing then  please rate it as "level3" or "level 2" label 👍 .
AS I ADDED:-
-3D effects like they moving up-down by time

redesign the  whole section
provide" EYE-COMFORT " colors
ADD HOVER EFFECTS
ADD TRANSITIONS
BUILD A interactive user interface
and many more you need to check it out

#UI IMPROVEMENT IN "PERSONALIZED RECOMMENDATIONS" #1258

[ ] Bug fix
[x] Feature enhancement
[ ] Documentation update
[ ] Other (specify): _______________

Screenshots / videos (if applicable)
--BEFORE:-

---AFTER:-

Checklist:

[x] I have made this change from my own.
[x] My code follows the style guidelines of this project.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generate no new warnings.
[x] I have tested the changes thoroughly before submitting this pull request.
[x] I have provided relevant issue numbers and screenshots after making the changes.

KINDLY MERGE THIS PR  UNDER GSSOC EXTD.
AND LET ME KNOW IF THERE IS ANY ISSUE

@PriyaGhosal

@isimarjitsingh Your pull request has been successfully merged! Don't stop here—continue contributing by addressing more issues and aiming to become a top contributor. Exclusive swag awaits for those who go the extra mile!
Great contribution ! 🎉


Getting Error Sass::SyntaxError - Undefined variable: "$filter-field-seperator-width"

I am using activeadmin (1.0.0). I found active_bootstrap_skin pretty simple using in my new project but when I try it on my existing project it crashed let me share logs.
sass::SyntaxError - Undefined variable: "$filter-field-seperator-width".:
sass (3.4.19) lib/sass/script/tree/variable.rb:49:in _perform'   sass (3.4.19) lib/sass/script/tree/node.rb:50:in perform'
sass (3.4.19) lib/sass/tree/visitors/perform.rb:395:in visit_prop'   sass (3.4.19) lib/sass/tree/visitors/base.rb:36:in visit'
sass (3.4.19) lib/sass/tree/visitors/perform.rb:158:in block in visit'   sass (3.4.19) lib/sass/stack.rb:79:in block in with_base'
sass (3.4.19) lib/sass/stack.rb:115:in with_frame'   sass (3.4.19) lib/sass/stack.rb:79:in with_base'
sass (3.4.19) lib/sass/tree/visitors/perform.rb:158:in visit'   sass (3.4.19) lib/sass/tree/visitors/perform.rb:431:in block (2 levels) in visit_rule'
sass (3.4.19) lib/sass/tree/visitors/perform.rb:431:in block in visit_rule'   sass (3.4.19) lib/sass/tree/visitors/perform.rb:179:in with_environment'
sass (3.4.19) lib/sass/tree/visitors/perform.rb:429:in visit_rule'   sass (3.4.19) lib/sass/tree/visitors/base.rb:36:in visit'
sass (3.4.19) lib/sass/tree/visitors/perform.rb:158:in block in visit'   sass (3.4.19) lib/sass/stack.rb:79:in block in with_base'
sass (3.4.19) lib/sass/stack.rb:115:in with_frame'   sass (3.4.19) lib/sass/stack.rb:79:in with_base'
sass (3.4.19) lib/sass/tree/visitors/perform.rb:158:in visit'   sass (3.4.19) lib/sass/tree/visitors/perform.rb:431:in block (2 levels) in visit_rule'
sass (3.4.19) lib/sass/tree/visitors/perform.rb:431:in block in visit_rule'   sass (3.4.19) lib/sass/tree/visitors/perform.rb:179:in with_environment'
sass (3.4.19) lib/sass/tree/visitors/perform.rb:429:in visit_rule'   sass (3.4.19) lib/sass/tree/visitors/base.rb:36:in visit'
sass (3.4.19) lib/sass/tree/visitors/perform.rb:158:in block in visit'   sass (3.4.19) lib/sass/stack.rb:79:in block in with_base'
sass (3.4.19) lib/sass/stack.rb:115:in with_frame'   sass (3.4.19) lib/sass/stack.rb:79:in with_base'
sass (3.4.19) lib/sass/tree/visitors/perform.rb:158:in visit'   sass (3.4.19) lib/sass/tree/visitors/perform.rb:431:in block (2 levels) in visit_rule'
sass (3.4.19) lib/sass/tree/visitors/perform.rb:431:in block in visit_rule'   sass (3.4.19) lib/sass/tree/visitors/perform.rb:179:in with_environment'
sass (3.4.19) lib/sass/tree/visitors/perform.rb:429:in visit_rule'   sass (3.4.19) lib/sass/tree/visitors/base.rb:36:in visit'
sass (3.4.19) lib/sass/tree/visitors/perform.rb:158:in block in visit'   sass (3.4.19) lib/sass/stack.rb:79:in block in with_base'
sass (3.4.19) lib/sass/stack.rb:115:in with_frame'   sass (3.4.19) lib/sass/stack.rb:79:in with_base'
sass (3.4.19) lib/sass/tree/visitors/perform.rb:158:in visit'   sass (3.4.19) lib/sass/tree/visitors/perform.rb:431:in block (2 levels) in visit_rule'
sass (3.4.19) lib/sass/tree/visitors/perform.rb:431:in block in visit_rule'   sass (3.4.19) lib/sass/tree/visitors/perform.rb:179:in with_environment'
sass (3.4.19) lib/sass/tree/visitors/perform.rb:429:in visit_rule'   sass (3.4.19) lib/sass/tree/visitors/base.rb:36:in visit'
sass (3.4.19) lib/sass/tree/visitors/perform.rb:158:in block in visit'   sass (3.4.19) lib/sass/stack.rb:79:in block in with_base'
sass (3.4.19) lib/sass/stack.rb:115:in with_frame'   sass (3.4.19) lib/sass/stack.rb:79:in with_base'
sass (3.4.19) lib/sass/tree/visitors/perform.rb:158:in visit'   sass (3.4.19) lib/sass/tree/visitors/perform.rb:326:in block (2 levels) in visit_import'
sass (3.4.19) lib/sass/tree/visitors/perform.rb:326:in block in visit_import'   sass (3.4.19) lib/sass/stack.rb:88:in block in with_import'
sass (3.4.19) lib/sass/stack.rb:115:in with_frame'   sass (3.4.19) lib/sass/stack.rb:88:in with_import'
sass (3.4.19) lib/sass/tree/visitors/perform.rb:323:in visit_import'   sass (3.4.19) lib/sass/tree/visitors/base.rb:36:in visit'
sass (3.4.19) lib/sass/tree/visitors/perform.rb:158:in block in visit'   sass (3.4.19) lib/sass/stack.rb:79:in block in with_base'
sass (3.4.19) lib/sass/stack.rb:115:in with_frame'   sass (3.4.19) lib/sass/stack.rb:79:in with_base'
sass (3.4.19) lib/sass/tree/visitors/perform.rb:158:in visit'   sass (3.4.19) lib/sass/tree/visitors/base.rb:52:in block in visit_children'
sass (3.4.19) lib/sass/tree/visitors/base.rb:52:in visit_children'   sass (3.4.19) lib/sass/tree/visitors/perform.rb:167:in block in visit_children'
sass (3.4.19) lib/sass/tree/visitors/perform.rb:179:in with_environment'   sass (3.4.19) lib/sass/tree/visitors/perform.rb:166:in visit_children'
sass (3.4.19) lib/sass/tree/visitors/base.rb:36:in block in visit'   sass (3.4.19) lib/sass/tree/visitors/perform.rb:186:in visit_root'
sass (3.4.19) lib/sass/tree/visitors/base.rb:36:in visit'   sass (3.4.19) lib/sass/tree/visitors/perform.rb:157:in visit'
sass (3.4.19) lib/sass/tree/visitors/perform.rb:8:in visit'   sass (3.4.19) lib/sass/tree/root_node.rb:36:in css_tree'
sass (3.4.19) lib/sass/tree/root_node.rb:20:in render'   sass (3.4.19) lib/sass/engine.rb:278:in render'
compass-rails (3.0.0) lib/compass-rails/patches/sass_importer.rb:48:in evaluate'   tilt (2.0.1) lib/tilt/template.rb:96:in render'
sprockets (3.5.2) lib/sprockets/legacy_tilt_processor.rb:25:in call'   sprockets (3.5.2) lib/sprockets/processor_utils.rb:75:in call_processor'
sprockets (3.5.2) lib/sprockets/processor_utils.rb:57:in block in call_processors'   sprockets (3.5.2) lib/sprockets/processor_utils.rb:56:in call_processors'
sprockets (3.5.2) lib/sprockets/loader.rb:134:in load_from_unloaded'   sprockets (3.5.2) lib/sprockets/loader.rb:60:in block in load'
sprockets (3.5.2) lib/sprockets/loader.rb:318:in fetch_asset_from_dependency_cache'   sprockets (3.5.2) lib/sprockets/loader.rb:44:in load'
sprockets (3.5.2) lib/sprockets/cached_environment.rb:20:in block in initialize'   sprockets (3.5.2) lib/sprockets/cached_environment.rb:47:in load'
sprockets (3.5.2) lib/sprockets/bundle.rb:23:in block in call'   sprockets (3.5.2) lib/sprockets/utils.rb:183:in dfs'
sprockets (3.5.2) lib/sprockets/bundle.rb:24:in call'   sprockets (3.5.2) lib/sprockets/processor_utils.rb:75:in call_processor'
sprockets (3.5.2) lib/sprockets/processor_utils.rb:57:in block in call_processors'   sprockets (3.5.2) lib/sprockets/processor_utils.rb:56:in call_processors'
sprockets (3.5.2) lib/sprockets/loader.rb:134:in load_from_unloaded'   sprockets (3.5.2) lib/sprockets/loader.rb:60:in block in load'
sprockets (3.5.2) lib/sprockets/loader.rb:318:in fetch_asset_from_dependency_cache'   sprockets (3.5.2) lib/sprockets/loader.rb:44:in load'
sprockets (3.5.2) lib/sprockets/cached_environment.rb:20:in block in initialize'   sprockets (3.5.2) lib/sprockets/cached_environment.rb:47:in load'
sprockets (3.5.2) lib/sprockets/base.rb:66:in find_asset'   sprockets (3.5.2) lib/sprockets/environment.rb:30:in find_asset'
sprockets (3.5.2) lib/sprockets/base.rb:92:in []'   sprockets-rails (2.3.3) lib/sprockets/rails/helper.rb:230:in lookup_asset_for_path'
sprockets-rails (2.3.3) lib/sprockets/rails/helper.rb:190:in check_errors_for'   sprockets-rails (2.3.3) lib/sprockets/rails/helper.rb:159:in block in stylesheet_link_tag'
sprockets-rails (2.3.3) lib/sprockets/rails/helper.rb:158:in `stylesheet_link_tag'

solved by adding these in active_admin.scss
$filter-field-seperator-width: 20px;
$side-by-side-filter-input-width: 20px;


[Transport] HTTP goes out of memory when writing large file with slow back-end

Description:
At the moment highlighted queue in the image is controlled by the writabitliy of the socket buffer. But ideally the highlighted queue should have it is own controller.
New tasks are added to the queue. Using below code.
this.channel.eventLoop().execute(())

To reproduce the issue use below ballerina code.
import ballerina/http;
import ballerina/io;

http:Client jpgEnpoint = new ("http://localhost:9000/abc");

# Prints `Hello World`.

public function main() {
    io:println("Started");

    http:Request mp4file = new();
    mp4file.setFileAsPayload("/home/shaf/Documents/balFiles/main-mp4/ReceivedFile.mp4");
    var response = jpgEnpoint->post("/", mp4file);

    io:println("Payload sent");

    if (response is http:Response) {
        var payload = response.getJsonPayload();
        if (payload is json) {
            io:println(payload);
        } else {
            io:println(payload);
        }
    } else {
        io:println(response);
    }

    io:println("Done");
}

Use the jar in the link as the back-end.
java -jar netty-in-action-1.0-SNAPSHOT-jar-with-dependencies.jar
https://drive.google.com/file/d/13lTiBGMrBd6gmET9EejtdXVkrifNxOFZ/view?usp=sharing

Closing as this not a high priority issue.


fix: add "Content-Type: application/json" haader when body is nil

Call DeleteField to get an error, http_status: 400, code: 91401
The header Content-Type: application/json; charset=utf-8" will be added, when the body is nil.
The server parses the body according to the header, and then reports an error

thanks


Support extended reports (HTML section & items section) + varius fixes

Added HTML section support.
Added items section to support a section with column styled items.
Added grouped list section that uses list section
Added error boundary on sections to avoid empty react render on failure
fixed line chart sort by string rather than date.
fixed list section key issues

Coverage decreased (-8.4%) to 68.799% when pulling c75ab346024cb5cf70a016fc377b4a035d8fab4c on reimagine_reports into 0c2538da25f000374fd11d9dc13d1503f4a826b2 on master.

@dorsha will work as before, just missing a lot of new sections.

👍


Add boot_volume_size option to -B in knife rackspace server create

When using the -B option with knife rackspace server create, the size
of the created volume is hardcoded to 100GB.
In order to specify the volume size, I added the boot_volume_size option
to fog's create_server call (merged in fog 1.35).
This commit leverages this new fog option and allows to use smaller
volumes (miminum 50GB) as well as larger ones.
This commit also clarifies the wording of one misleading error message.

Hi. I am an automated pull request bot named Curry. There are commits in this pull request whose authors are not yet authorized to contribute to Chef Software, Inc. projects or are using a non-GitHub verified email address. To become authorized to contribute, you will need to sign the Contributor License Agreement (CLA) as an individual or on behalf of your company. You can read more on Chef's blog.
Non-GitHub Verified Committers
There are 1 commit author(s) whose commits are authored by a non-GitHub verified email address. Chef will have to manually verify that they are authorized to contribute.
Please sign the CLA here.

signed ICLA

@adecarolis The bot will update the PR once the ICLA is signed -- it looks like you've signed your commit from an email address unknown to Github: https://0.gravatar.com/avatar/9f23246304c8386223caa897d53b5fca?d=https%3A%2F%2Fassets-cdn.github.com%2Fimages%2Fgravatars%2Fgravatar-user-420.png&r=x&s=140

Hi. Your friendly Curry bot here. Just letting you know that all commit authors have become authorized to contribute. I have added the "Signed CLA" label to this issue so it can easily be found in the future.


Freezes weights

closes #20
I didn't write a specific test for this, but if you run it for one epoch you'll see that the first four weights don't change from what they are initialized to.
Also, Nathaniel mentioned that grad_remove should not be necessary here, but at the moment it's the only way to get grad_replace to work. He said he'll be writing tests tomorrow for it.
I'm wondering if this should be configurable? As in whether the weights are frozen or not? Which weights are/aren't frozen?

I'm also wondering if this should be configurable? As in whether the weights are frozen or not? Which weights are/aren't frozen?

Only the first four weights may be frozen. I haven't implemented the pre-train module, where the first four weights will be trained.

Also, Nathaniel mentioned that grad_remove should not be necessary here, but at the moment it's the only way to get grad_replace to work. He said he'll be writing tests tomorrow for it.

OK, I will wait for his test.

https://github.com/burn-rs/burn/pull/695

Nice work! I will review the code tomorrow.

I wrap your code into freeze_initial_stability() function to make it more readable.
By the way, I don't know whether it is possible that we can save a config to turn on/off freeze_initial_stability. The current implementation hasn't been prepared for freeze_initial_stability.

Sounds good. It's certainly possible to make it configurable through the ModelConfig struct :-). I was able to do it locally, but I didn't include it in the PR because I was unsure of what you were envisioning. Let me know if that's what you would like and I can add it to the PR

Thanks! Please include it in the PR.

Added. Basically, instead of calling ModelConfig::new(), you initialize it like ModelConfig: { freeze_stability: true }. Right now, it defaults to false when initializing with ModelConfig::new().
In Burn's examples, they add the configurable paramaters inside new, but in my opinion that is less readable. e.g it would be ModelConfig::new(false) for this particular case. What are your thoughts?

I prefer the struct form too, though would recommend you have ModelConfig derive Default and set the default value to whatever we're more likely to use, so we can just do ModelConfig::default().

I prefer the struct form too, though would recommend you have ModelConfig derive Default and set the default value to whatever we're more likely to use, so we can just do ModelConfig::default().

Burn seems to like setting defaults with their Config derive macro. So when calling ModelConfig::new(), it's using the defaults set with #[config(default = false)]. Is this sufficient, or would you also like to be able to use default?

The nice thing about default is when you have multiple options, and can do eg ModelConfig { freeze_stability: true, ..Default::default) } to get the defaults for the rest of the options.
Is freezing the most common use case? If so it would probably make sense to make true the default instead. Note that the config(default=...) I think only applies when creating a config from a JSON file, so we'd need to manually derive Default on it to get the right values.
One other possible improvement if other options are anticipated in the future: storing ModelConfig inside Model, instead of passing its individual options in to Model::new().

That makes sense. My understanding is that this feature is staying off until the pre-train stage implemented. Admittedly I'm pretty new to FSRS, so I'm not sure what the pre-train stage entails, but I set the default to false for that reason. It sounds like once that's implemented freezing would be the normal behaviour, so yeah, maybe I should switch it the default to true?
What would training initialization look like if ModelConfig was inside Model? Are we planning to configure from a json file? Sorry if these are silly questions, still getting aquainted with the project.

Will defer to @L-M-Sherlock on sensible defaults; for the latter questions:

instead of Model::new(freeze_stability) it would be Model::new(config)
re JSON, it may be useful when testing/experimenting, but probably not so much when using this in third-party crates

The pre-train stage will be used to optimize the values of initial stability. When it is finished, we will pass the first four values to the initialization of model and set freeze_stability: true.


Allow Empty Value in Cookie

Allows vapor-session=; as a valid cookie, resulting in:
let cookie: (name: String, value: HTTPCookieValue) = HTTPCookieValue.parse("vapor=; Secure; HttpOnly")
// cookie.name == "vapor"
// cookie.value.string == ""

Hey @calebkleveter, you just merged a pull request, have a coin!
You now have 352 coins.


Tidy with ESLint

This resolves a few small errors found with the default recommended
ESLint config, and adds a comment explaining a piece of code that I
found confusing (and appears to lack any kind of tests or
documentation).

Thanks for the merge!

Thank you both for cleaning up the code :heart:  Looks much nicer. I pushed a small change for my commit after the merge.

Works for me!


Enabled configuration of ids alert threshold

@buzztroll So we can push in this value as needed.

SonarQube analysis reported 1 issue:

 1 minor

Watch the comments in this conversation to review them.

:+1:


Merging of video and audio

I am trying to capture video from the camera and save the movie mixed with an audio file. Any Ideas ?

Hi @jasudev,
this pull request should slove your problem: https://github.com/BradLarson/GPUImage/pull/1887


[#1043] Deprecate lower camel case method names

Resolves issue #1043 by renaming the methods to reflect the method names of BindableEvent.
Signal:fire() -> Signal:Fire()
Signal:connect() -> Signal:Connect()
Signal:wait() -> Signal:Wait()
The old methods still exist for compatibility, but are deprecated

You can clean up your code by instead of doing...
function sig:connect(f)
	self:Connect(f)
end

...just do
sig.connect = sig.Connect

@Kampfkarren People using the old API did sig:connect(), not sig.connect(), so this would not work.

@stillunt1tled x:y(...) is the exact same as x.y(x, ...).
Thus, if you have this code
local X = {}

function X:A()
    print("hello")
end

X.B = X.A

Then attempting to call X:B() will do the exact same thing as X:A(). This is because X:B() will now be equivalent to X.A(X.A).
My point is, sig.connect = sig.Connect is the exact same as what you're doing, just more efficient and readable.

@Kampfkarren while I will take your suggestion, if this is done, self will never be passed to the method (not that its used in this script, I just concerned about future implementations that use self)

@stillunt1tled The colon operator automatically passes self as a parameter to the function. That's the whole point of the colon operator in the first place.
https://www.lua.org/manual/5.1/manual.html#2.5.8
"A call v:name(args) is syntactic sugar for v.name(v,args), except that v is evaluated only once."

^ this is why I don't contribute to this repository.
Thats the point. If you dont have the : operator, self wont be passed. I'm saying if they use it in a future version of this script, all of the

Er, no. self is automatically passed.
https://repl.it/repls/IdealisticWebbedLogins

Hi there, thank you for your feedback. We're going to be closing this repository as we've chosen the Roblox Dev Forum as our venue for developer feedback. Please feel free to migrate your feedback to a DevForum post if you feel it is still relevant.


Bug with Beam Search Generation

When using model.generate() with num_beams > 1, the pixel_values input is duplicated num_beams times. However, since the pixel_values for Qwen2-VL does not have a batch_size dimension, the duplicated pixel_values end up with an incorrect memory format compared to what is expected. This performance is much worse than greedy search generation.

transformers version = 4.46.3, found same problem when using beam search.
Beam search will use repeat_interleave function to expand the inputs to num_beams times. For the inputs key with batch size in first dim, such as input ids, the expanding is like this:
>>> a
tensor([[0., 0., 0.],
        [1., 1., 1.]])
>>> a.repeat_interleave(2,dim=0)
tensor([[0., 0., 0.],
        [0., 0., 0.],
        [1., 1., 1.],
        [1., 1., 1.]])

But for the pixel_values of Qwen2 VL, which has shape of [patch_count, patch_pixels], the expand is not done as expected. The first dim of pixel_values will be repeated and leading to a wrong order. Here is a simplified example:

The two inputs (if num beams=2) should be the same two images as the original image as expected, but result in two wrong, different images.

Above is a real image example, the image ① is crossing and merging the upper left and upper right corners of the original image in rows, and the image ② is crossing and merging the bottom left and bottom right corners of the original image in rows. When using num_beams=2 and query is Describe the image in detail., the output of Qwen2-VL-7B-Instruct is:
The image appears to be a highly pixelated and distorted photograph of a cityscape. The main subject of the image is a row of tall, green trees lining what seems to be a street or avenue. The trees are densely packed, and their leaves are a vibrant green, indicating that the photo was likely taken during the spring or summer months.

In the background, there is a large, modern building with a metallic facade. The building has a series of windows and appears to be part of a larger complex, possibly an office building or a commercial structure. The sky is overcast, and the overall color palette of the image is muted, with shades of green from the trees and grays from the building and sky.

The distortion in the image makes it difficult to discern finer details, but the general layout and elements of the scene are still recognizable. The pixelation suggests that the image may have been manipulated or is of low resolution. Despite the distortion, the image conveys a sense of urban environment with a touch of nature provided by the trees.

This bug disrupts the proper functioning of the beam search decoding. Hope this bug can be fixed soon.


Updating Hackrank/Java

Added Solved Problems In HackRank/Java And New Hello World Program Using Assembly

And Vector sort and erase in c++ algorithm


Regenerate ARM resources libraries to use @azure/core-http

This change regenerates @azure/arm-resources to use the new changes to autorest.typescript that outputs code to use @azure/core-http and @azure/core-arm.  The other libraries are all generated as a result of this since their specs are all related.
This spec repo commit was used to generate them all to avoid unwanted API changes:
https://github.com/Azure/azure-rest-api-specs/commit/85ca88d23974f60963533605bc5a53fc44866652#diff-ac43489f896d1c83759d8fb63fc46175

I've verified that these changes all build locally for me and that the code only contains changes to the new dependencies and import names.

I soooo need a tool that will take a diff and exclude certain word replacements!

Closing this PR as we plan to use the new code generator that is still in development in order to re-generate the management libraries


React warning when adding the same course multiple times in collection course selection UI

Related to #391
If the user adds the same course multiple times to the collections course selection field it will cause a react warning since the key prop needs to be unique in the list. It may be easiest to filter out courses already added from the dropdown so they can't be added again. However there is a product decision which needs to be made since there could be a use case for having multiple instances of a course in a collection.

@pdpinch @Ferdi wondering what your thoughts are on this - is there a use-case for adding the same course to a course collection more than one time?

No, no reason to have the same course twice in the same list.

if we want to add this constraint, we'll need to add it as a general constraint on our relation field


🐛 Fix Parsing of MQT-Bench Circuits

Description
🚧 Should probably have opened an issue but I wanted to directly add the failing test.
What I did
I downloaded a circuit for AE with 2 qubits compiled hardware-agnostically with qiskit from MQT Bench and inserted this circuit as input to qc::QuantumComputation::fromQASM(), which should not throw an error but it does, see the respective test
https://github.com/cda-tum/mqt-core/blob/f91a0526617dbb68e2ce3589a8994f220bc44347/test/test_operation.cpp#L263-L287
Checklist:

[x] The pull request only contains commits that are related to it.
[ ] I have added appropriate tests and documentation.
[ ] I have made sure that all CI jobs on GitHub pass.
[ ] The pull request introduces no new warnings and follows the project's style guidelines.

Also, I am not sure whether the place for the new test is appropriate, feel free to suggest something different.

This was a pretty quick fix. Although, technically, this is not even a bug on our side, but on the Qiskit side. A well known one in fact.
The Qiskit OpenQASM 2.0 exporter is known to hallucinate the existence of the u gate in the standard library, while, in fact, the gate is not part of the standardized qelib1.inc, but was only part of Qiskit's extension of said file.
So, technically, our parser is right to reject the circuit since it is not valid OpenQASM 2.0.
However, many people have come to rely on these circuits to be readable and Qiskit itself has implemented a respective workaround for reading the files.
Given that it's a one line change, I think it should be fine to also adopt the workaround here.

Also, I am not sure whether the place for the new test is appropriate, feel free to suggest something different.

I moved the test to the OpenQASM parser test suite and extended it slightly.
Also adjusted the labels of this PR. The "bug" label is typically reserved for issues (=bug reports), while the "fix" label is intended for PRs (=bug fixes). And given that this is only a bug fix and not a breaking change of any sort nor a new feature, it qualifies as a patch release contribution and does not really warrant a new minor version. Hope that makes sense 😌

@burgholzer Thanks for taking care of this!


Error  when rewriting url if is server relative

Category
[x ] Bug
[ ] Enhancement
Problem Area
[ ] Page Transformation: Error during the setup/use of the Page Transformation UI solution (did you check our troubleshooting guide?)
[x ] Page Transformation: Error during the use of page transformation from PnP PowerShell
[ ] Page Transformation: Error during the use of page transformation from .Net
[ ] Page Transformation: Page is not looking correct after transformation
[ ] Modernization Scanner: something went wrong...
Expected or Desired Behavior
Working with the new Url rewrite. I am getting the following error

This is the input to the RewrireUrl method:
error.txt
The source site is  TronetAU20190424-A-Temp
Destination is TronetAU20190424-A

the text attached to above was wrong. The input to the RewriteUrl Method is /sites/TronetAU20190424-A/btly/sitepages.
This happens at line 115. The url is already relative:

Thanks for logging this @russgove, will fix it asap

I apologise to you both for not acting on this, I spotted this possibility last night and made a comment in a commit rather than just fixing it.

Just fixed it

Closing this...please re-open if the problem still occurs


libpng 1.6.19: Fix for linuxbrew, link the package, otherwise packages

libpng 1.6.19: Fix for linuxbrew, link the package, otherwise packages
depending on libpng will failed to build.

I thought keg_only was ignored for Linuxbrew anyway? @sjackman

It seems it is not ignored. When I try to upgrade / install a formula that depends on libpng, I run into this issue.

What do you see, specifically? Linux shouldn't even be reading provided_pre_mountain_lion because Mountain Lion is definitely not a Linux distro release :sweat_smile:.

ok, this is what I just did.
$ brew uninstall libpng
$ brew install libpng (this is with the unchanged libpng.rb)
==> Downloading
ftp://ftp.simplesystems.org/pub/libpng/png/src/libpng16/libpng-1.6.19.tar.xz
Already downloaded: /home/yuhuang/.cache/Homebrew/libpng-1.6.19.tar.xz
==> ./configure --disable-silent-rules
--prefix=/home/yuhuang/.linuxbrew/Cellar/libpng/1.6.19
==> make
==> make test
==> make install
==> Caveats
This formula is keg-only, which means it was not symlinked into
/home/yuhuang/.linuxbrew.
OS X already provides this software in versions before Mountain Lion.
Generally there are no consequences of this for you. If you build your
own software and it requires this formula, you'll need to add to your
build variables:
LDFLAGS:  -L/home/yuhuang/.linuxbrew/opt/libpng/lib
CPPFLAGS: -I/home/yuhuang/.linuxbrew/opt/libpng/include

==> Summary
/home/yuhuang/.linuxbrew/Cellar/libpng/1.6.19: 17 files, 1.3M, built in 27
seconds
$ vi
vi: error while loading shared libraries: libpng16.so.16: cannot open
shared object file: No such file or directory
$ brew link --force libpng
Linking /home/yuhuang/.linuxbrew/Cellar/libpng/1.6.19... 19 symlinks
$ vi
(now everything works fine)
On Fri, Dec 4, 2015 at 1:40 PM, Dominyk Tiller<EMAIL_ADDRESS>wrote:

What do you see, specifically? Linux shouldn't even be reading
provided_pre_mountain_lion because Mountain Lion is definitely not a
Linux distro release [image: :sweat_smile:].
—
Reply to this email directly or view it on GitHub
https://github.com/Homebrew/linuxbrew/pull/675#issuecomment-161881747.

--
With Regards,
Yu Huang
Software Engineer,
Google Inc.

Looks like a bug in Linuxbrew to me. I'm on vacation and back in the new year.

I can't reproduce this issue. Possibly it was fixed in the last merge.

This case is handled in the core code here: https://github.com/Homebrew/linuxbrew/blob/master/Library/Homebrew/formula_support.rb#L16-L17


feat: add GitHub oauth login

part of #808

try koa-passport with passport-github

try koa-passport with passport-github

The author of Passport-Github has not maintained the original module for a long time. Features in his module don't work since Github upgraded their API to version 3.0. We forked it and re-published it to NPM with a new name passport-github2.


feat: add optional env to enable logging of incoming messages

This commit adds an optional flag to log incoming messages.

Just a thought -- You may want to consider logging just the last element of messages, since that array will grow as the conversation history grows. Or perhaps make that an option, otherwise logs could grow quite quickly

@nkrumm Thanks for the suggestion, I have incorporated them and set trim messages by default to true while enabling logging is optional (default=false).

I've added https://github.com/rhnvrm/chatbot-ui/pull/2 @rhnvrm it may be relevant, if you can, please take a look :)


Fix argument annotation name not being processed

You get a funny NPE when you use processed arguments names. Lemme fix that :)

Mmmh, now that I'm on it, not every command annotation value is processed...


What is the Chip Grapher meant to be?

I saw that it's in the works, but I don't know what it is / what it's being made with. If we could get more details, some people might contribute to make it.

Sorry for the late answer, but I was working on a circuit grapher, similar to this one. I couldn't keep my motivation and left it in an unfinished state.

Ah. So like a way to test out circuits online? Something like that is really complex, and is even harder to fit into a website. I can see why you stopped working on it, but if we ask around, get some help and a team, we could bring it back if you think it's worth reviving.

Well, you're not really testing the circuits, but it's basically for people who either want to draw out their graphs without going in game, or for those who have to explain things to the "visual learners".


Add linux-s390x

s390x works, too -- keep in parallel with other Linux architectures.

Thanks @utzb :+1:


Sidekiq.schedule= not found

Looks like there's a require missing for sidekiq-scheduler/extensions/schedule as well.

Hi @aaronjensen
I think I found the issue. From the backtrace you posted above, it seems that your have some sort of initializer file. I found an issue in the README.md file related to an old way of requiring the gem.
See the diff here: https://github.com/moove-it/sidekiq-scheduler/commit/3b5d6ed843bfa2fd9d783ecf9ec851759e71362e
So, in summary, you need to have this:
require 'sidekiq-scheduler'

instead of:
require 'sidekiq/scheduler'

Hope that helps

That did, thank you.

This works fine


feat: support detect pnpm

When using pnpm create preact in a pnpm monorepo, it's necessary to support automatic detection of pnpm for installing dependencies. On the other hand, if within a monorepo, the dependencies should preferably be installed using the package manager employed by the monorepo.
If necessary, I can introduce the testing framework Vitest to add relevant unit tests.

Apologies for the long delay, should now be available in 0.4.0


Exception escape during trinity full node sync

What is wrong?
 WARNING  11-10 20:16:56       StateDownloader  Task <coroutine object StateDownloader._request_and_process_nodes at 0x7f1c8c46b938> finished unexpectedly: b'\x1fs\xce^\xcd\x822\x14p\x14\xd2\xc4\xd6\x18\x1e\xb7b\xd6\xf4\xd4\xccxL\xe7\xfbz\xfc\xa8\xa3\x84-`'
   DEBUG  11-10 20:16:56       StateDownloader  Task failure traceback
Traceback (most recent call last):
  File "/home/ubuntu/py-evm/p2p/service.py", line 141, in _run_task_wrapper
    await awaitable
  File "/home/ubuntu/py-evm/trinity/sync/full/state.py", line 218, in _request_and_process_nodes
    await self._process_nodes(node_data)
  File "/home/ubuntu/py-evm/trinity/sync/full/state.py", line 134, in _process_nodes
    await self.scheduler.process([(node_key, node)])
  File "/home/ubuntu/py-evm/trinity/sync/full/hexary_trie.py", line 227, in process
    await self.commit(request)
  File "/home/ubuntu/py-evm/trinity/sync/full/hexary_trie.py", line 242, in commit
    await self.commit(ancestor)
  File "/home/ubuntu/py-evm/trinity/sync/full/hexary_trie.py", line 238, in commit
    self.requests.pop(request.node_key)
KeyError: b'\x1fs\xce^\xcd\x822\x14p\x14\xd2\xc4\xd6\x18\x1e\xb7b\xd6\xf4\xd4\xccxL\xe7\xfbz\xfc\xa8\xa3\x84-`'

How can it be fixed
:see_no_evil: :hear_no_evil:

Currently, the plan is to skip implementing "standard fast sync" altogether. So I'm going to close as wontfix, until that changes.

If the plan from #511 gets reversed, then this should be reopened.


[ICE] crash on array initializer with misleading number of braces

Expected behavior
h is initialized to a 1x3 array of char.
Actual Behavior
rcc panics during initialization.
Code
    char h[] [3] = {1, 2, 3};
thread 'main' panicked at 'assertion failed: `(left == right)`
  left: `3`,
 right: `8`: destination and source slices have different lengths', src/libcore/slice/mod.rs:2217:9

Backtrace

  14: core::slice::<impl [T]>::copy_from_slice
             at /rustc/73528e339aae0f17a15ffa49a8ac608f50c6cf14/src/libcore/macros.rs:52
  15: rcc::ir::static_init::<impl rcc::ir::Compiler>::init_expr
             at src/ir/static_init.rs:163
  16: rcc::ir::static_init::<impl rcc::ir::Compiler>::init_symbol
             at src/ir/static_init.rs:254
  17: rcc::ir::static_init::<impl rcc::ir::Compiler>::init_array
             at src/ir/static_init.rs:285
  18: rcc::ir::static_init::<impl rcc::ir::Compiler>::init_symbol
             at src/ir/static_init.rs:213
  19: rcc::ir::static_init::<impl rcc::ir::Compiler>::store_static
             at src/ir/static_init.rs:86
  20: rcc::ir::compile
             at src/ir/mod.rs:89
  21: rcc::compile
             at src/lib.rs:98
  22: rcc::real_main
             at src/main.rs:99
  23: rcc::main
             at src/main.rs:164

Thanks to @byter09 for the fuzzing :)

I have a fix for the immediate ICE, but I'm still not sure why {1, 2, 3} should initialize the inner array instead of being an error.

So it appears this can nest arbitrarily deep.
char h[][1][1][1][3] = {1,2,3};
<stdin>:1:25: warning: suggest braces around initialization of subobject
      [-Wmissing-braces]
char h[][1][1][1][3] = {1,2,3};
                        ^~~~~
                        {    }
<stdin>:1:25: warning: suggest braces around initialization of subobject
<stdin>:1:25: warning: suggest braces around initialization of subobject
<stdin>:1:25: warning: suggest braces around initialization of subobject


Downloading stash for Hardcore, details logged to DebugInfo.log, please open a ticket

keeps crashing hear I need pro to work so I can sell stuff

Downloading stash for Hardcore, details logged to DebugInfo.log,

A pull request has been started to resolve this issue.
[https://github.com/Stickymaddness/Procurement/pull/315]

Procurement 1.9.7 has been released which fixes this issue!
https://github.com/Stickymaddness/Procurement/releases/tag/1.9.7


Teradata connection string

Hi All,
Could anyone provide me with sample Teradata connection string that I can use in NBi (or some kind of guide for dummies)?
I am not able to get it working from the documentation provided.
Thanks for any advice.

I've solved it in the meantime (using ODBC driver).
In case anyone would know, the general connection string is below:
Driver={Teradata Database ODBC Driver 17.10};DBCname=database_name;uid=username;pwd=password;
Cheers.


Circle CI json-spec

Starting Oct 14, there is an issue with the Circle CI step sudo pip install json-spec
https://app.circleci.com/pipelines/github/dkhamsing/open-source-ios-apps/532/workflows/0bf9ef5f-ccd3-451b-9397-6e64b8674f58/jobs/3316

Looks like it was a temporary issue, now fixed
https://app.circleci.com/pipelines/github/dkhamsing/open-source-ios-apps/543/workflows/c254ecf2-9cd4-4687-b062-8c59523b3094/jobs/3332


Keyboard Manager: 0.81.1 regression prevents several shortcuts from working

Microsoft PowerToys version
0.81.1
Installation method
PowerToys auto-update
Running as admin
Yes
Area(s) with issue?
Keyboard Manager
Steps to reproduce
This is all my remapping keys:

but some of them not working now after update, like alt+f to search, alt+v to paste, alt+c to copy, alt+y to go to parent fold in vscode. unforturnately.
✔ Expected Behavior
I hope the shortcuts remapping works normally.
❌ Actual Behavior
It just not works, including alt+? mappings.
Other Software
No response

I have same issue

This resembles issue #33230 a lot. Please retitle this to something like:

Keyboard Manager: 0.81.1 regression prevents several shortcuts from working

/dup https://github.com/microsoft/PowerToys/issues/17344


Link to correct version of CoreUI in config

WHY
BEFORE - What was wrong? What was happening before this PR?
This PR fixes issue https://github.com/Laravel-Backpack/CRUD/issues/4475
It appears that provided CoreUI CDN link<EMAIL_ADDRESS>now points to the most recent version, v4.1.6, but bundle.js includes older CoreUI v2.1.16.
AFTER - What is happening after this PR?
If I replace CoreUI CDN link with the correct version link - https://unpkg.com/<EMAIL_ADDRESS>it works as expected.
HOW
How did you achieve that, in technical terms?
I used CDN link to the same version that is used to build bundle.js
Is it a breaking change?
Not this isn't a breaking change
How can we test the before & after?
Comment line with bundle.js and uncomment CDN lines. Before - the sidebar doesn't collapse, after - it works just like when using bundle.js

Thanks @viperet ! Merged, will be available to everyone starting Monday. Though... those who have already published the config... will basically never get it... 👀


Replace underscores with hyphens in HTML data attributes name

This makes the specific data attributes can be easily found in the editor.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1169

Relevant Lines:
1169

💛  - Coveralls


Unable to go to first page

Changed goToNextStep to accept nextStep(0) when you are on the last step
by changing the if statement if (hasNextStep.current || stepIndex) to
if (hasNextStep.current || stepIndex != null), this way, when we try to
go to the first step while being on the last step, the activeStep will be
changed successfully

Hi, thanks for creating this PR. You're absolutely right, didn't take into account the falsy values.


Question about multi-table join query

I created two tables, one is base and one is base_info, they are one-to-one related. I want me to return everything from both of their tables in one query, is there any way to achieve this? I looked through all the related issues and couldn't find a corresponding answer, hoping to have a good solution.
My question translated into sql probably needs to implement a one-time query like this
select * from base left join base_info on base.id=base_info.base_id

This is probably the operation in ent
clent.Base.Query().LeftJoin(baseinfo.Table).On(baseinfo.FieldBaseID, base.FieldID).Where(xxx).Select(xxx).Scan(xxx)

Hi @Mickls,
I think what you are looking for is eager-load
for your case you can do:
bases, err := client.Base.
    Query().
    WithBaseInfo().
    All(ctx)

Hi @Mickls, I think what you are looking for is eager-load for your case you can do:
bases, err := client.Base.
    Query().
    WithBaseInfo().
    All(ctx)

Thanks a lot for your help, it's useful.

withxxxx is used to one-to-one releated model?

withxxxx is used to one-to-one releated model?

It should, it's a one-to-one edge right?


added the possibility to exclude a list of pokemons to encounter

Because  there are less pokemons that we don't want to be encountered

May be better to merge exclusions with the "all" param than creating a specific exclude param/condition ?

Good idea I will try to make it when I am back from vacation


Example theme broken after upgrade

I have been using the example theme for a long time, i recently got a new machine and tried to use it and it doesnt work.
ZSH version on my old machine = 5.0.5
ZSH version on new machine = 5.5.1
I also just updated oh-my-zsh on my old machine and it broke the example theme that machine too.

Fixed in master. Please run upgrade_oh_my_zsh


Refactor tests

separate batch, extensions and the regular apis into separate modules
added delete for some tests

Current coverage is 94.46% (diff: 100%)

Merging #115 into master will not change coverage

@@             master       #115   diff @@
==========================================
  Files             9          9
  Lines           668        668
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits            631        631
  Misses           37         37
  Partials          0          0

Powered by Codecov. Last update dbf9938...80dde2d

@mbohlool Done!


Optional parameters in DaphneDSL UDFs

User-defined functions in DaphneDSL should support optional parameters with default values.
Example:
def foo(x, y = 0) {...}
def bar(x, y:si64 = 0) {...}
def baz(x, y:matrix<f64> = seq(1, 10, 1)) {...}

The usefulness of such default values would be increased, if DaphneDSL supported the specification of function parameters by key, such as in:
def foo(a, b, c=3, d=4, e=5, f=6, g=7, h=8) {
  ...
}
foo(1, 2, f=99); # c, d, e, g, and h be default

Otherwise, for functions with many parameters (as foo above), one would always need to specify all parameters up until the last one that shall deviate from the default, as in:
foo(1, 2, 3, 4, 5, 99); # only g and h by default; c, d, and e need to be repeated


Computed value for text and cleanup

About
Refactored the Vue logic inside core.js

Renamed the value key to enabled for semantic purposes.
Set the text data to be computed from the enabled value
Moved the reloadPage function into Vue instance under methods.

Dont pull. Issue with reloads


Use of ^ for dependencies in definition files for updating

First, there  might be a difference for use in Rust vs NPM / Javascript.
I am of the opinion that we should use ^ where possible and sensible to ensure we run with the most recent updated dependencies.  The lock file should be committed and updated at regular intervals.  However, let s use this issue to put forward the pros and cons and feed the (eventual) conclusion in the developer guidlines.
Note this is not urgent.

First thing that comes to mind is https://github.com/dominictarr/event-stream/issues/116

Two more points

Web3 1.0.0-beta.38 has breaking changes from beta.37
We may also differentiate between internal and external dependencies.

Ref. https://github.com/OpenSTFoundation/brandedtoken.js/pull/115


Server content.

Hello.
Didn't find some "official" server content, because this addons gaves me errors :c. Thanks you if you find it, please give it in readme or smth 😋
Thanks 😎

friend found the file, sorry for pings.


[FEATURE] Support sha512 digests in soci rebuild-db

Description
soci rebuild-db only scans for content in blobs/sha256/
Describe the solution you'd like
scan in blobs/sha512/ also
Describe any alternative solutions/features you've considered
No response
Any additional context or information about the feature request
Support for SHA512 is optional in https://github.com/opencontainers/image-spec/blob/main/descriptor.md#registered-algorithms

I think we do that already - https://github.com/awslabs/soci-snapshotter/blob/main/cmd/soci/commands/rebuild_db.go#L47 , walks through the blobs directory entirely.

Sorry, I must have misread.


Export models

I am unable to stub the models (such as message) returned by the various methods of the slack client here because those models cannot be imported.
Please expose the models used here so I can properly mock this library.

Would you mind giving me an example of what you'd like to do here? I don't quite understand why you can't do something like:

var Team = require('./src/team');

... so I'm guessing I'm missing something in what you're trying to do?

you mean like require('./node_modules/slack-client/src/team') I suppose I could do that, but this makes the test a bit brittle. This file is technically considered a "private" module, because it doesn't get exported. If it were to ever be moved/replaced, my tests would break. I suppose I could do that, but I think it's not a good practice to refer directly to src files.

You should be able to do require('slack-client/src/group'); I believe.
I recognize it's not ideal and the general structure of the client is something we'll be looking into more going forwards. As part of that I'll define a clearer top level contract for what the module offers that can define a clearer way of doing this.

Closing this one for now - I'll drop you an update once we have a plan in place for how to work with this longer term.


canberra-boot and systemd services are missing from libcanberra

Describe the bug
canberra-boot, which is offered by most Linux distributions where libcanberra is installed, is not present in NixOS. Its systemd services aren't available either. I'd like to be able to have an easy way to play sounds on boot and shutdown and this seems like the usual way to do it on other Linux distributions :grin:
Steps To Reproduce
Steps to reproduce the behavior:

Install libcanberra (whether via nix-env, environment.systemPackages option, etc)

Expected behavior
canberra-boot should be available and executable.
systemd services such as canberra-system-bootup.service, canberra-system-shutdown.service, and canberra-system-shutdown-reboot.service should be available to enable.
Notify maintainers
@RossComputerGuy
Metadata
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.6.33, NixOS, 24.11 (Vicuna), 24.11.20240622.a71e967`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.23.0`
 - nixpkgs: `/nix/store/yqy82fn77fy3rv7lpwa9m11w3a2nnqg5-source`

Add a :+1: reaction to issues you find important.

Probably a NixOS module would be good so it'll be easy to customize.

That doesn't really have a bearing on this PR, as this would still need to be built anyway.

woot! thank you for your work on this! i'm so excited to get this working on my PC

FYI it'll take a couple of weeks to end up in unstable, and you'll then have to add it to systemd.packages as well, but it should work at that point.


Proper HTML scaffolding

The default with soupault --init is:

Welcome!

Welcome to my homepage. It's under construction.

This is missing some required minimum scaffolding, namely:
<!doctype html>

   <meta charset="utf-8">
   <title>Sunday</title>

Welcome!

Welcome to my homepage. It's under construction.

https://developer.mozilla.org/Web/HTML/Quirks_Mode_and_Standards_Mode
https://validator.w3.org
https://w3.org/tr/html5/syntax

Could you tell me the reproducing procedure?
If that's what you get in build/, then it's definitely a bug. However, if you are referring to the page in site/index.html, then it's just a page body that is inserted in templates/main.html at build time.
$ soupault --init
[WARNING] Configuration file soupault.conf not found, using default settings
Initializing the project
Creating site directory site
Initialization complete.
...

$ cat templates/main.html

    <title>  </title>
    <meta name="viewport" content="width=device-width, initial-scale=1">

$ cat site/index.html

Welcome!

Welcome to my homepage. It's under construction.

$ soupault --verbose
[INFO] Entering directory site/
[INFO] Processing page site/index.html
[INFO] Processing widget generator-meta on page site/index.html
[INFO] Processing widget no-blink on page site/index.html
[INFO] Processing widget page-title on page site/index.html

$ cat build/index.html
<!DOCTYPE html>

  <title>
   Welcome! — My Homepage
  </title>
  <meta name="viewport" content="width=device-width, initial-scale=1">
  <meta name="generator" content="soupault 1.5">

   Welcome!

   Welcome to my homepage. It's under construction.

Looks like I was looking at a fragment rather than complete document. It does seem that
<meta charset="utf-8">

Is missing though

This is already resolved for people building from source, binary 1.6 should be available shortly too.


⚠ Jellyfin has degraded performance

In 464df6b, Jellyfin (https://jellyfin.underthere.xyz) experienced degraded performance:

HTTP code: 502
Response time: 18820 ms

Resolved: Jellyfin performance has improved in 5da1d32 after 1 minute.


Update comment in test file

Codecov Report

Merging #150 into dev will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##                dev     #150   +/-   ##
=========================================
  Coverage     18.49%   18.49%
  Complexity       76       76
=========================================
  Files            58       58
  Lines          2157     2157
  Branches        201      201
=========================================
  Hits            399      399
  Misses         1738     1738
  Partials         20       20

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 d76af40...91ad56f. Read the comment docs.


query index file changes with path and query

Please always provide the GitHub issue(s) your PR is for, as well as test URLs where your change can be observed (before and after):
Fix #
Test URLs:

Before: https://main--world-bank--aemsites.hlx.live/
After: https://WRLDBAEMS-226-query-index--world-bank--aemsites.hlx.live/


help me with controller API

i dont understand how to use the controller API program.
i have installed the ardrone-autonomy modules
and now how i can get the repl.js to use the controller API program?
when i run the repl.js in the examples folder, it doesnt change anything
please help me

Hey @rezaprasetyo have you found a solution to this?

no i haven't found a solution on this :(
On Tue, Mar 1, 2016 at 3:14 PM, RPEK001<EMAIL_ADDRESS>wrote:

Hey @rezaprasetyo https://github.com/rezaprasetyo have you found a
solution to this?
—
Reply to this email directly or view it on GitHub
https://github.com/eschnou/ardrone-autonomy/issues/19#issuecomment-190607638
.

--
Reza Prasetyo
Engineering Physics
Institut Teknologi Bandung

Hey! I managed to resolved it. You've to download the date format packag and it works.
Btw have you tried to mission."log(path)? I've got a hell lot of numbers on excel, but unable to decipher anything out of it! ):


Update bsv2-series.md

Correcting typo on M8ps to MBps

@danielharkerdp - thanks for this correction!


incompatible with Gradle 8.0

react-native: 0.67.2
it works perfectly with ios.

Any Updates on that?
My App works on iOS but Android is not working at all

@raajjemv Hi if this is still a problem for you then the author has posted the fix. As the error stated, libc++_shared.so was found on 2 files.
This snippet was included in the README to remove one of the duplicate libc++_shared.so, on jetified-react-native file.
tasks.whenTaskAdded((tas -> {
    // when task is 'mergeLocalDebugNativeLibs' or 'mergeLocalReleaseNativeLibs'
    if (tas.name.contains("merge") && tas.name.contains("NativeLibs")) {
        tasks.named(tas.name) {it
            doFirst {
                java.nio.file.Path notNeededDirectory = it.externalLibNativeLibs
                        .getFiles()
                        .stream()
                        .filter(file -> file.toString().contains("jetified-react-native"))
                        .findAny()
                        .orElse(null)
                        .toPath();
                Files.walk(notNeededDirectory).forEach(file -> {
                    if (file.toString().contains("libc++_shared.so")) {
                        Files.delete(file);
                    }
                });
            }
        }
    }
}))

Be mindful though as on the latest react-native version the file is not named jetified-react-native anymore if you enabled hermes. It is jetified-react-android instead.

I get the following error when adding this block of code to android/app/build.gradle:
* What went wrong:
Execution failed for task ':app:mergeDebugNativeLibs'.
> Could not get unknown property 'Files' for task ':app:mergeDebugNativeLibs' of type com.android.build.gradle.internal.tasks.MergeNativeLibsTask.

Tried to figure it out by myself but my Java and Groovy skills are lacking.
Using react-native 0.70.6
Any ideas?
i also get a syntax error in the editor but i'm not sure it is linked:

You need to import Files. Add this line to the top of your build.gradle:
import java.nio.file.Files


Allow running pickle on unit tests

I've tested this with our project. I had lots of other problems with Robolectric but that is out of scope. This seems to work perfectly. 👍 It was able to generate code for unit tests.
I have a suggestion for the extension. What about something like below:
androidTest {
    enabled = true
    featuresDir = ....
}
unitTest {
   enabled = true
   featuresDir = ....
}

@tasomaniac do you mean
pickle {
  androidTest { }
  unitTest { }
}

?

Exactly 👍 It is just an idea.

It does sound good! I'll give it a try this weekend :+1:


Allow KubeAdm config customizing and port forwarding.

Instead of hard coding the kubeadm.conf settings as part of the
dind-cluster.sh script, we'll use a template file and customize
it with the known settings needed based on the IP mode and CNI
mode. This allows users to add additional customization desired
(e.g. for istio), without altering scripting.
For minion nodes, allow port forwarding, using the EXTRA_PORTS
configuration setting.
As part of setting the static route to the NAT64 server for IPv6,
set the docker flag to remove the temp container used.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.vagrant seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

There's a problem with this approach, namely that separate tmpl file is not compatible with the idea of having a self-contained fixed/ scripts. So after build/genfixed.sh the fixed/ scripts will be broken. Maybe we can somehow move the template into the image?


Suggestions on container tags

Current container tags are hard to use and don't follow the convention of common images.
https://github.com/v2fly/v2ray-core/pkgs/container/v2ray/versions?filters[version_type]=tagged
A few suggestions on container tags:

please remove arch from the tag and build multi-arch images
add some non-fixed tags such as latest, v5, v4 to track the latest version or a range of versions

Thanks!

c.c. @xiaokangwang

Currently the container building pipeline don't support multi-arch images, and a separate step would be necessary to get it working.  Pull request on this is welcomed, so long as it don't modify exist building steps.
Tags such as latest will be considered once multi-arch tags are available.

https://github.com/redhat-actions/buildah-build/issues/121#issuecomment-1683691170
seems quite tedious. to keep most of current pipeline implementation intact, another job outside of the strategy matrix is required to merge manifests and push them all together.
or do you mind that we simply remove the strategy matrix and use one single script step to build all archs in one shot. this way the pipeline can be simpler but we will have to invoke podman commands manually instead of using existing actions


[SPARK-35583][DOCS] Move JDBC data source options from Python and Scala into a single page

What changes were proposed in this pull request?
This PR proposes move missing JDBC data source options from Python, Scala and Java into a single page.
Why are the changes needed?
So far, the documentation for JDBC data source options is separated into different pages for each language API documents. However, this makes managing many options inconvenient, so it is efficient to manage all options in a single page and provide a link to that page in the API of each language.
Does this PR introduce any user-facing change?
Yes, the documents will be shown below after this change:

"JDBC To Other Databases" page

Python

Scala

Java

How was this patch tested?
Manually build docs and confirm the page.

Test build #139134 has started for PR 32723 at commit 5853560.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/43654/

Test build #139140 has started for PR 32723 at commit 2126a67.

cc @maropu FYI

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/43654/

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

Test build #139142 has started for PR 32723 at commit f453070.

Kubernetes integration test unable to build dist.
exiting with code: 1
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/43660/

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

Test build #139134 has finished for PR 32723 at commit 5853560.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/43662/

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

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/43662/

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

Test build #139140 has finished for PR 32723 at commit 2126a67.

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

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

Test build #139142 has finished for PR 32723 at commit f453070.

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

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

The proposal looks fine. Have you checked R ones? e.g., https://github.com/apache/spark/blob/master/R/pkg/R/SQLContext.R#L612-L621

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/43707/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/43707/

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

Thanks, @HyukjinKwon .
Also updated the PR description!

Test build #139193 has started for PR 32723 at commit f70c28e.

Test build #139193 has finished for PR 32723 at commit f70c28e.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds no public classes.

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

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/43713/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/43713/

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

Test build #139198 has started for PR 32723 at commit bd91a48.

Test build #139187 has finished for PR 32723 at commit 13f9f86.

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

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

Merged to master.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/43720/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/43720/

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

Thank you for this update, @itholic late lgtm.

Test build #139198 has finished for PR 32723 at commit bd91a48.

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

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


v0.2.1 redux

Updating the Github Actions workflow based on failures to publish.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 97.85%. Comparing base (674d8d2) to head (4bcf8a5).
 Consider uploading reports for the commit 3568d4f to get more accurate results

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main      #22   +/-   ##
=======================================
  Coverage   97.85%   97.85%
=======================================
  Files           5        5
  Lines         280      280
=======================================
  Hits          274      274
  Misses          6        6

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


Environment export in CI not working

In CI we have a step that dumps out the current environment being used right before we run tests
https://github.com/coiled/coiled-runtime/blob/0667395184944e17968a1211672de3e8b24c497a/.github/workflows/tests.yml#L60-L64
This is useful for debugging CI builds. Unfortunately, when I look at the output on CI builds it's unexpectedly empty
--
--Conda Environment (re-create this with `conda env create --name <name> -f <output_file>`)
--
name: test
channels:
  - defaults

EDIT: low priority, but would be good to fix at some point

Thanks for reporting this James, I can't reproduce it locally, so it must be a CI problem.
I opened a PR to switch to conda on this line and see what happens.

Closed via https://github.com/coiled/coiled-runtime/pull/78


Hyphen upsets IPTC ApplicationRecord Tags - get something terrible error

MacOS 10.13.6 High Sierra
Reading IPTC tags from a .dng file and the hyphenated ones don't work
Code fragment -
     var reado =
    `Make: ${tags.Make},
     Model: ${tags.Model},
     Sub-location: ${tags.Sub-location},
     City: ${tags.City},
     Province-State: ${tags.Province-State},
     Country-PrimaryLocationName: ${tags.Country-PrimaryLocationName},
     UserComment: ${tags.UserComment}`;

    console.log('READO ', reado);
    })
    .catch(err => console.error("Something terrible happened: ", err))

eg: typical error is -
Something terrible happened: ReferenceError: State is not defined at exiftool.read.then (app.js:730) etc
Sub-location - FAIL
City - OK
Province-State - FAIL
Country-PrimaryLocationName - FAIL
Cheers,
Blane

Could you please attach the DNG?

Thanks, I'll look at it now.

@Blatman you can't reference tags with dashes like that.
Javascript interprets tags.Province-State as "the value of tags.Province minus the value of State," which explains your errors.
You want tags["Province-State"].

Ahhh - yes - too much brainfade at my end sorry for the waste of time
Cheers.


Bible Linker not functioning

I've installed Bible Linker from Google Marketplace on both Chrome and Edge and both times got the same message:
Message details
Script function not found: dfv_en_jw_en_jw_kjv
This occurs when I try to select either a different bible version or try to insert the links.
Help?

Hi @Mswiftney! Thanks for dropping by and reporting this error.
This has been an ongoing issue and I've been tracking this in Google Apps Script (GAS) error logs.
The problem with the issue is it is intermittent. Personally, I have never encountered the issue myself, but the logs throw that issue from time to time. A friend once had the same issue, and without changes to the code, it later resolved for him.
Since the code is the same, and it works for most but not for others, I am guessing that that there might be something in the way the GAS interpreter runs the code. Because I have never encountered the error myself, it is not possible for me to debug it (since there's nothing to debug if everything works on my install). I'm actually waiting for a coder who will identify the error and help debug it from their end.
So for now all I can do is guess what went wrong. My initial "fixes" focused on the JS IIFE function that is necessary for dynamic custom menus. My hunch was that the IIFE did not complete before the main function started. But my "mitigations" did not to work.
So now I'm back to guessing again why the error is happening, albeit not consistently. I'm still waiting for someone who could point me to where this bug is coming from. 😅


First try at replacing partials lines

Concerns #1

Currently it screws up the first example…
-                mem::transmute::<*mut c_void, &mut Box<FnMut(&Button)>>(data)(&button)
+                &mut *(data as *mut Box<for<'r> std::ops::FnMut(&'r controls::Button)>)Button)>)&button)

But I have not yet investigated why that is.
 This might be clippy's fault: Column 78 is just before (&button) (cutting off the ( is my fault, though!)
cc @mcarton

Yes, clippy forgets to add parenthesis: foo(bar)(baz) is transformed into &mut *(…)(baz) while it should be (&mut *(…))(baz). This will be fixed. https://github.com/Manishearth/rust-clippy/pull/1060 correctly handles parenthesis inside the suggestions, and I'm planning to correctly handle parenthesis around them (and also whether ; is needed) in another PR.


Remove uniqness of phone verification

Users should be able to open multiple accounts using the same phone number.

[x] on server set env allowDuplicateUserData to true

@AnastasiiaOdnoshevna this can be tested on QA/PROD
DEV always allowed this

Checked on the QA env. v0.22-1.0
I cough create the two accounts with the same phone numbers.

@AnastasiiaOdnoshevna please also verify you can edit the profile and change your number to an existing number but only if you have verified it correctly (ie via the sms code)

@sirpy
The user is able to change the number on the edit profile page using an existing number
Checked on the QA v 0.22.1-1
Desktop: Windows 10 // Google Chrome 83.0.4103.61


Change the Global object

Hi,
I would like to know if I can replace the actual Global object with the one I have defined ?
I am thinking about the navigators window object, that is the this in global scope, and each window functions are available as global functions.
Thanks.

You can modify the global object, but you can't replace it entirely.
Probably the way to do it is to instantiate a new instance of your global object, then transfer the functions across.
For example, to add the myfunction() function to the global object, do something like this (not tested):
var myObj = new MyGlobalObject(engine);
engine.Global["myfunction"] = myObj["myfunction"];


router_check_tool: fix ASSERT in path_rewrite test

Signed-off-by: Jyoti Mahapatra<EMAIL_ADDRESS>Description: When multiple test validations are mentioned in the validation section,
eg:
"validate": {
        "host_rewrite": "api.lyft.com",
        "virtual_host_name": "api",
        "path_rewrite": "/pre/test",
        "cluster_name": "www2"
      }

finalizeRequestHeaders is called multiple times causing ASSERT failures. None of the existing tests covered this scenario. Now few tests have been added to simulate the error.
Before the fix the failure looked like this
[2019-05-29 19:47:48.210][1405009][critical][assert] [source/common/router/config_impl.cc:557] assert failure: case_sensitive_ ? absl::StartsWith(path, matched_path) : absl::StartsWithIgnoreCase(path, matched_path).

Risk Level: Low
Testing:

Added breaking tests and fixed them using the fix.
Tested the tool against lyft's routes

#730


Installer

Where is the easy installer download link?

Currently you need to download the zip: https://github.com/NamelessMC/Nameless-Installer/archive/master.zip

Hopefully it will be on the NamelessMC website soon, but for now the link Sam gave you is the best way of getting it. Thanks!


32_stonks.py improvement

#hactoberfest
Format stock_prices list in a more readable way, consitant with 'lists' section list formating

Post October Monthly Challenge cleanup. It's changing 24 files here :( Have to close this one.


Add /nostr route and .md

Placeholder.[^fn]
[^fn]: Consider this PR an announcement of an announcement.

ACK

👀


fix issue#9912 : helm list  panic when release.Chart is nil

fix issue #9912 : helm list  panic when release.Chart is nil

You only have one commit incorrectly signed off! To fix, first ensure you have a local copy of your branch by checking out the pull request locally via command line.

what do you think of my pr https://github.com/helm/helm/pull/9924 ? @technosophos


Moved _common to default.

Moving _common.rb to default.rb makes this cookbook easier to include
in other cookbooks.  This also brings this cookbook back in line
with its README.

Relevant Gerrit Review

Should the include_recipe 'ceph::conf' be moved too? Most anything that talks to ceph needs to have both the software and the config file.

I think you're right.  Just pushed that change now.

Missed the ceph::conf in resources/cephfs.rb, but otherwise I like it!

Fixed.  My grepping is bad, and I should feel bad.

Running the kitchen tests reveal that each of the *_install recipes includes the _common_install recipe.
Should _common_install be left in place, and have the default recipe include it? I think having each of the *_install recipes include the default recipe looks weird otherwise...

I think they should actually be removed since default now includes it and everything should now include default.

You mean the *_install recipes should all include the default recipe? sounds good.

Okay, that change has been made in the PR @hufman.  Does that look good to you?

ping @hufman.  Anything I can do to move this patch along?

Using default looks like a cleaner approach to me.  Appears ready to go.

The test kitchens all passed on my end, +1 from me


No pcad

I replaced pcad-pipe-macro_compiler & pcad-pipe-list_macros. I don't think we'll ever need pcad-pipe-infer_decoupled. Depends on ucb-bar/barstools#24

Will take a look at this and probably merge after #29 is merged.


[Model Update]: TractionBatteryCode BAMM -> SAMM

Update Reason
Update the model from BAMM to SAMM.
https://github.com/eclipse-tractusx/sldt-semantic-models/tree/main/io.catenax.traction_battery_code
MS1 Criteria

[ ] The model that should be updated exists
[ ] The referenced use case exists
[ ] The potential updates are discussed with all stakeholders
[ ] A decision on versioning was made in consensus with the affected parties according to the guidelines.
[ ] The potential update will be either

[ ] backward compatible or
[ ] a (chain of) migration strategy(-ies) from all non-deprecated previous versions to the new model will be developed

[ ] A modeling steward has been assigned

modeling_team
FYI @BenediktMuellerIML @johannsvarela @jacewski-bosch


FluentFTP.GnuTLS possible memory leak

FTP Server OS: Unknown
FTP Server Type: Unknown
Client Computer OS: Debian
FluentFTP Version: 50.0.1
FluentFTP.GnuTLS Version: 1.0.30/1.0.31
Framework: .NET 7
Hi, I am working on an app that periodically connects to customer FTPS server and downloads some files (using the FluentFTP.GnuTLS package). Everything works well apart for increased memory usage in over time (days). After taking some memory dumps it seems that the unmanaged memory is the one that keeps increasing.
I can not provide any logs at this time but I will try to generate a sample project to reproduce the issue.
I have looked in the FluentFTP.GnuTLS and repository and I saw that the FunctionLoader.Free() call has been removed and that function is not called anywhere else. Could this be the cause of the issue?
I know there is not that much information in the issue.

Thanks for reporting this.

Everything works well apart for increased memory usage in over time (days). After taking some memory dumps it seems that the unmanaged memory is the one that keeps increasing.

This is important information

After taking some memory dumps it seems that the unmanaged memory is the one that keeps increasing.

Not surprising

I will try to generate a sample project to reproduce the issue

See next...

FunctionLoader.Free() call has been removed and that function is not called anywhere else. Could this be the cause of the issue?

Well, the idea was to keep the functions loaded (forever) but to initialize and deinitalize the GnuTLS binarys when they are no longer in use.
Certainly there may be a logic error in the code somewhere - probably Dispose(). If you have a reproducible mini-console-app that eats memory, let's isolate the problem step by step.

Hi,
I tried to create a Docker environment to test this and I think I managed to reproduce the issue: https://github.com/validide/FluentFTP.GnuTLS
At the moment the application tries to connect to the server but fails with an exception. After running for 1000 iterations (just creating the client and trying to connect) it gets up to 3GB RAM consumed on my local setup.
I have 0 experience working with FTPS so I am not sure if this test is correct.

Also Hi!
I had a look at your source:
static async Task SimulateWork(int target) {
	var count = 0;
	while (count < target) {
		using (var conn = new FtpClient("ftp-server", "ftptest", "ftptest")) {
			ConfigureConnection(conn);

			await Task.Delay(100);

			try {
				conn.Connect();
			}
			catch (Exception ex) {
				Console.WriteLine(ex.ToString());
			}
		}
		count++;
		Console.WriteLine($"Created {count} clients.");
	}
}

Can you try this with "await using"? and report back? Using "just using" takes things down a different path on dispose (although I thought I was successful in catching this).

I have 0 experience working with FTPS so I am not sure if this test is correct.

Sheesh. All is good, you are on the track of something perhaps.

Also, it would be worth dockering a test with SYNC client - because actually FluentFTP.GnuTLS does not really support async operations. I was hoping it would gracefully block if used in an async environment - but it would be nice if you would see if there is a difference.

Hi, I just pushed an updated version on my repository and there seems to be no difference between the sync and async version.

Good - we'll concentrate on the SYNC version first in order to determine more.

I have finally found time to run your code.
My initial run was 4000 connects (using FluentFTP.GnuTLS) and I took a snapshot at 100 and at 4000. The run took 30 mins. up to then.
Here is the result:

I don't see a leak. I'll let it run for longer, like overnight, see if that show anything up.

After many more hours:

28kB.

When running the project in WSL it seemed like it was leaking 3MB each time it would try to connect. I will try to come-up with better "evidence".

Yes, that would be nice.
I am running some ultra long interations also whenever I have the opportunity.
Note please, that it is also entirely possible that the called external non-managed library might also have a memory leak itself - since we keep it loaded instead unloading and reloading (big overhead).


feat: delay draining of higher order pods while pods are still terminating

Fixes https://github.com/kubernetes-sigs/karpenter/issues/1133
Fixes https://github.com/aws/karpenter-provider-aws/issues/5858
Description
preStop hooks are widely used to allow for the graceful termination of pods. Karpenter currently runs through the list of pods in "priority groups", but moves on to the next group once all evictions in the current group are acknowledged, without waiting for the pods to actually terminate. This can lead to DaemonSets -- like metrics or logging -- terminating before the pods they are serving have had a chance to do so.
To address this issue, this PR introduces a check in the Drain function to identify terminating pods. When such pods are found, Karpenter will queue only the pods belonging to the same or a lower group, ensuring the desired eviction order is maintained. When no terminating pods are found, the queueing continues as normal.
How was this change tested?
Test files: https://gist.github.com/robertoriv/7537e77844d12154437dee0116f79603
I tried using kwok for testing, but it doesn't seem to execute the preStop hooks properly. Terminating a node is nearly instantaneous when using a kind cluster.
I then tried building the controller locally, and deploying it to a live cluster. That worked well enough to test the queueing behavior, but the locally-built controller -- modified or unmodified -- was unable to create new instances to satisfy the node claims due to a bunch of "message":"waiting on cluster sync" messages and launching nodeclaim, resolving ec2nodeclass, object is awaiting reconciliation erros. I'm guessing that's a problem with my setup.
Here are the logs for the feature, printed out using a more verbose version of the Drain function that is included in the gist:
karpenter-58ffc9fcf8-sl7q9 {... "message":"Started with 18 evictable pods.","commit":"d39fdb5-dirty","controller":"node.termination", ...}
karpenter-58ffc9fcf8-sl7q9 {... "message":"Found 3 terminating pods: ","commit":"d39fdb5-dirty","controller":"node.termination", ... }
karpenter-58ffc9fcf8-sl7q9 {... "message":"  - Pod: non-critical-pod-5f4f67fd94-gfx9t, Group: 1","commit":"d39fdb5-dirty","controller":"node.termination", ... }
karpenter-58ffc9fcf8-sl7q9 {... "message":"  - Pod: non-critical-pod-5f4f67fd94-qsh4b, Group: 1","commit":"d39fdb5-dirty","controller":"node.termination", ... }
karpenter-58ffc9fcf8-sl7q9 {... "message":"  - Pod: non-critical-pod-5f4f67fd94-src47, Group: 1","commit":"d39fdb5-dirty","controller":"node.termination", ... }
karpenter-58ffc9fcf8-sl7q9 {... "message":"Highest order pod: non-critical-pod-5f4f67fd94-gfx9t, Group: 1","commit":"d39fdb5-dirty","controller":"node.termination", ...}
karpenter-58ffc9fcf8-sl7q9 {... "message":"Narrowed the list of evictablePods to be queued to 4 pods.","commit":"d39fdb5-dirty","controller":"node.termination", ... }
karpenter-58ffc9fcf8-sl7q9 {... "message":"  - Pod: non-critical-pod-5f4f67fd94-5hjfr, Group: 1","commit":"d39fdb5-dirty","controller":"node.termination", ... }
karpenter-58ffc9fcf8-sl7q9 {... "message":"  - Pod: non-critical-pod-5f4f67fd94-8j5hm, Group: 1","commit":"d39fdb5-dirty","controller":"node.termination", ... }
karpenter-58ffc9fcf8-sl7q9 {... "message":"  - Pod: non-critical-pod-5f4f67fd94-d78pq, Group: 1","commit":"d39fdb5-dirty","controller":"node.termination", ... }
karpenter-58ffc9fcf8-sl7q9 {... "message":"  - Pod: non-critical-pod-5f4f67fd94-rtnqz, Group: 1","commit":"d39fdb5-dirty","controller":"node.termination",  ... }

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

31 of 33 (93.94%)  changed or added relevant lines in 2 files are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.004%) to 77.907%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/controllers/node/termination/terminator/terminator.go
18
20
90.0%

Files with Coverage Reduction
New Missed Lines
%

pkg/scheduling/requirements.go
2
97.98%

Totals

Change from base Build<PHONE_NUMBER>:
0.004%

Covered Lines:
8269

Relevant Lines:
10614

💛  - Coveralls

@robertoriv Just noticed that it's been a few weeks and there's been no movement on this PR. Happy to help in any way that we can. Feel free to ping me or any of the maintainers on slack.

@robertoriv Do you expect to come back to this one? If not, we may take in a similar change to the one that's proposed above (see https://github.com/kubernetes-sigs/karpenter/pull/1280#discussion_r1661796076)

@robertoriv Do you expect to come back to this one? If not, we may take in a similar change to the one that's proposed above (see #1280 (comment))

Hi @jonathan-innis 👋🏻
I’m currently on leave and won’t be able to get back to this for a couple of months. Feel free to close this PR and/or adopt any pieces that are useful from it.


:book: hack: generate cli docs

Summary
Related issue(s)
Fixes #

/lgtm
/approve

@varshaprasad96 please use make update-contextual-logging to pass lint.

Since we are fine with having doc generation for cli, I'll modify the PR to include all cobra commands, and have this run in CI.
/hold

@avinal Thoughts on merging this PR here. Can they later be modified to be Hugo compatible? I'm not sure of what changes are required to do so.

@avinal Thoughts on merging this PR here. Can they later be modified to be Hugo compatible? I'm not sure of what changes are required to do so.

Hi Varsha, I am still looking into the options to integrate everything with Hugo, should take a day or two. I think we can merge this for now. In any case we can always open a PR.

/retest


Documented bundlePath override does not work

The documentation for overriding the Pagefind bundle's path is to configure the bundlePath option. Unfortunately, that option doesn't exist in the currently output pagefind file when writing the bundle from the Node.js getFiles call. Instead, you need to use basePath (as seen in the Index Options below) to override it. I'm not sure where the bug is here, whether it's documentation, source code, or specifically the output from getFiles, otherwise I would have filed a PR.
https://github.com/CloudCannon/pagefind/blob/4a9927764e00fdd4db3a30546de43e1421a6be9e/pagefind_web_js/types/index.d.ts#L7
LMK if you need more information to help track this down!

Good catch! Sorry to delay — this does indeed look like it just got named wrong in a kerfuffle. I'll fix this up for the next patch release 🙂


Throw exception when dropping unknown index

In MongoDB, when trying to drop an index that doesn't exist, the Java Driver throws an exception:
Caused by: com.mongodb.MongoCommandException: Command failed with error -1: 'index not found with name [nonExistingIndex_1]' on server localhost:27017. The full response is { "nIndexesWas" : 2, "ok" : 0.0, "errmsg" : "index not found with name [nonExistingIndex_1]" }

It would be nice if Fongo could mirror this behavior, as it would force my migration tests to drop an actually existing index rather than dropping an arbitrary index. Currently, Fongo silently ignores dropping an unknown index without throwing anything.
I'm willing to submit a pull-request if this is deemed feasible.

Sorry for the late.
Yes, PR are always welcome :)

Hi,
can you check against master or v3 ?

@twillouer: Sorry for the late reply. Works like a charm with 2.0.6, TYVM!


feat(cli): Add cost optimization nudges for Argo CLI

Hi guys, this one closes #4548
Demo:

This is my first contribution to Argo. Would❤   to hear some feedback. Thanks!
Checklist:

[x] My organization is added to USERS.md.

Thanks for the review @simster7!

Not sure that we would want to add a CLI config/data store (e.g. in ~/.argo/config) just for this, but that is a possibility.

I'm happy to implement this. However, a config data store feels like an overkill for this nudges use case unless we have other use cases that can benefit from it as well.
Another option is that we can add a --disable-nudges option for the CLI so that users can choose to disable it.
Let me know what you think. Thanks!

Hi @simster7,
Thanks for the review! I made some changes based on your feedback.

af3f1f656cc47779c6e2d4dcae5e21672ee0a2b3: Added a --no-nudges option for the CLI impacted. User can choose to not print the nudges
ffe487f2fb0b5829345bfa870c02a59d126adfdc: Added a security nudge for argo get

c610490753c22b65285ab00b9b282da83f358700: Discard return values for fmt.Print and fmt.Printf for consistency

Let me know your thoughts. Thanks!

My thoughts on this:

Great stuff!
I'm not sure it is worth it to make it possible to disable nudges in the CLI. Instead, let's see if people ask for that feature.
Should we do security nudges too?
Nudges need to use "exec spec". I think there is Workflow.ExecSpec() for that.

Should we do security nudges too?

I think security nudges are already implemented.

Nudges need to use "exec spec". I think there is Workflow.ExecSpec() for that.

Yes, good point.