Minor performance update to the splitting API

I noticed a difference of about a second (or two) between standard splitting call and splitting/update_only.
Also remove lheInputFiles from the updatable splitting parameters.

I'm removing the do not merge label, but let's wait for the cmsweb production upgrade. We can then push it in May deployment.


withFolderProperties out of node [phase 1]. Add unit tests

Add unit tests so we can refactor the project much safer

@mig82 How about this change?


How to disable plugin for a particular filetype?

I looked through the docs but couldn't find this. I want to disable all pairs for some filetypes, for example TelescopePrompt.
Looking at the code it seems this should work, but it doesn't for me:
require("ultimate-autopair").setup {
  default_pairs = {
    ft = {
      TelescopePrompt = { disable = true },
    },
  },
}

It might also be handy to be able to supply a function that returns a bool which enables or disables the plugin. That would make it easier to do more complex logic with buftypes/filetypes and other factors.

Added the option to disable filetypes in the ft option and added the TelescopePrompt={disable=true} to the defaults.

@altermo Is this still a feature? I tried adding this
require('ultimate-autopair').setup({
  default_pairs = {
    ft = {
      DressingInput = { disable = true },
    },
  },
})

to disable ultimate-autopair in dressing.nvim's input window, but it doesn't work.
I also tried browsing through the documentation and code and imitating that, but with no success.

It has been changed in version 0.5 to:
require('ultimate-autopair').setup({
  extensions={
    filetype={
      nft={'DressingInput'},
    },
  },
})

@altermo Thank you, that worked for me!
Is it also possible to disable buffer types?

Yes:
require('ultimate-autopair').setup({
  extensions={
    cond={
      cond=function ()
        return vim.bo.buftype~='your_buffer_type'
      end,
    },
  },
})


nixos/virtualisation.oci-containers: follow podman-generated systemd units more closely

Description of changes

This PR makes podman virtualisation.oci-containers follow the layout of units generated by podman with podman generate systemd --new --name more closely.
The changes mean that fewer process reside in the systemd-unit cgroup and setting PODMAN_SYSTEMD_UNIT allows the use of podman-auto-update.
Things done

Built on platform(s)

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

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

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

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 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
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

I tested the changes locally, but I had trouble running the tests properly.

+1 to making podman autoupdate support easier.
cc @bjornfor @zowoq @szlend @flokli (based on the last PR that touched this module)

Any updates?

Yeah, let's get this in.

podman-auto-update: set ODMAN_SYSTEMD_UNIT to %n

@mlyxshi I don't quite remember why I didn't use it, but it should also expand to the unit name.

I have create a new PR
https://github.com/NixOS/nixpkgs/pull/197950
Please review and approve, my server is broken now

This PR broke rootless podman for me, since it now tries to write a cidfile to /run/ which the podman user does not have permissions for.


Create a Website for this Repo

Hi there.
Is there a website for this repo?
Because if you don't have, well, this repo can simply be turned into a website right away. Others will discover this project in that website.
Steps:

Go to Settings and look for GitHub Pages, scroll down. That's almost at the bottom.

You will see there: Branch:none, so you should change that to master because you have a README.md file in the master repo. This will be your page. Click Save first.

Then click Choose a theme, you select a predefined theme of your site.

Visit your site now! The URL will be https://crev-dev.github.io/cargo-crev.

If you were amazed by that, simply read the documentation about GitHub Pages.

I own a crev.dev doman, and we do have a web.crev.dev, but I do plan to eventually make http://crev.dev into something informative. I do know GitHub Pages, but haven't considered using it. Maybe I should, until something better is ready. Thanks.

GitHub Pages on its own is actually great already. Thanks for the response.

Try the website:
https://web.crev.dev/crates/


Attendance component scaffolded out

Description
This PR adds an attendance page to the sidebar.  Further functionality fleshed out in the near future(next week).
Video Link
https://www.loom.com/share/c3dc23d7568d4dd4b19b8f28e4660983
[Loom Video](Add your loom video link here)
Trello Link
https://trello.com/c/u0rWH6ac
Type of change

[x] New feature (non-breaking change which adds functionality)

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have removed unnecessary comments/console logs from my code
[x] My changes generate no new warnings
[x] I have checked my code and corrected any misspellings
[x] No duplicate code left within changed files
[x] Size of pull request kept to a minimum
[x] Pull request description clearly describes changes made & motivations for said changes

Overall this Pull-request is clean. There are a few that I have pointed out directly inline that I would love addressed.

Please go and fix the linter warnings listed in by our GitHub action. They're easy to find if you open the Files Changed tab and scroll down bit you'll see a Warning from a check that looks like this:

The way to fix those warnings is go to the file and do what the linting warning says.
For the one above, you'd simply delete the definition of PendingApplications in the file.
Remember, if we're not using it, we don't want it in main.

Ok Ryan.  I believe I've made all the changes you've requested.  Please let me know if there is anything else to change for this PR to be merged.  Thanks!


Non-idempotent results for recurring CI check runs

Describe the bug

We currently have the ink! master CI failing due to some words which were previously not found, even though the CI was green for the PR's. This is the job from master: https://gitlab.parity.io/parity/ink/-/jobs/1012438.

I've created this PR branched from the above master to fix those issues. The CI for that PR is green. This is the job: https://gitlab.parity.io/parity/ink/-/jobs/1011563.
I've created another PR from the same master commit. The CI there fails due to a spellcheck error which hasn't shown up anywhere else before. This is the job: https://gitlab.parity.io/parity/ink/-/jobs/1012289.
Expected behavior
I would have expected the last spelling mistake ("reorg") to show up in either the master CI run or the run on https://github.com/paritytech/ink/pull/855, but it did in neither.

Could we add a cargo spellcheck --version? That would simplify diagnostics quite a bit since I started doing smaller and faster iterations.

I'm running it slightly different: adding -r $(git diff --name-only ${CI_COMMIT_SHA} ${CI_DEFAULT_BRANCH}) at the end. But evidently, it does not work: the amount of changed files in the PR is 230 and growing. One of the reasons might be in GitLab, it can't find the default branch to compare with:
$ time cargo spellcheck check -vvv --cfg=scripts/gitlab/spellcheck.toml --checkers hunspell --code 1 -r $(git diff --name-only ${CI_COMMIT_SHA} ${CI_DEFAULT_BRANCH})
fatal: ambiguous argument 'master': unknown revision or path not in the working tree.
Use '--' to separate paths from revisions, like this:
'git <command> [<revision>...] -- [<file>...]'

Parsing the git diff won't be necessary as soon as the whole code is covered by the check, but it seems like something makes spellcheck quit early sometimes.
Same I can conclude from @cmichi 's example, he doesn't parse the git diff.

With #190 all files checked will be visible with -vv or higher logging verbosity. So this should become easier to investigate.

v0.8.9 got released just now, which should provide enough information to resolve this. If it turns out it's not gitlab's env lacking, please re-open!


CodepenLogo icon is not exported from  'phosphor-react'

Describe the bug
There is an icon for Codepen logo but when imported gives an error that  CodepenLogo is not exported.

I just verified it is exported and working as expected in 1.4.1, and if the icon is present on our website, it is working as expected in the React package, since we use it there.
Can you share some more information? What version are you on? How are you consuming the package? What bundler?

Hi, version "phosphor-react": "^1.3.1",
I am using it in a react component

Will try 1.4.1 version

it works as you said in 1.4.1

Yep, this icon was not part of the library yet in 1.3.x.

I'll include the release version on the website, if it's helpful in the future!

Version info is now included for each icon on the site:


GetController or GetxController?

I am confused, the document refer all to GetxController, but the example code using GetController.
What is the different between these 2? Which one should we use?

The sample code for the repository is from version 2.0 that still used Get controller.
In version 3 you must use GetxController

@jonataslaw is it possible to have more than one instance of the same type controller?
Example:
In a browser the state controller will be the same kind for every page you open.

@peekpt you don't need multiple instances of your controller
you can use the same single instance in the entire app, using dependency injection
and what do you mean the same state controller for every page? you mean that in all pages you gonna use the exact same data displaying the exact same thing?
or you gonna use the controller to show different data in every page? if that's the case, use one controller per page

@jon

well, i did not understand yet but i am deviating from the subject.
@jonataslaw can you confirm if what he asked is possible on get? as far as i know, it is not

There are several ways to do this.
The first:
Controller controller = Controller (); // without Get.put()
The second, if you are injecting dependency directly on the widget:
GetBuilder<Controller>( // or GetX
init: Controller(),
global: false, // remembering that this will make the controller available only within GetBuilder
builder: (controller) => Text(controller.name)
[...]

The third is using tags:
Controller c1 = Get.put (Controller(), tag: 'ct1');
Controller c2 = Get.put (Controller(), tag: 'ct2');
Controller c3 = Get.put (Controller(), tag: 'ct3');

There is still a longer way using abstractions and override: true, but it is not documented because I am not sure if I will keep the api, so I will not explain about it here.

@jasonlaw https://www.reddit.com/r/FlutterDev/comments/hmjff1/people_who_are_you_using_getx_how_do_you_feel/?utm_source=ifttt
I will delete this post once you put a icon reaction

Example updated. Closing this


Text Alignment to right

        contentStream.beginText();
        contentStream.setFont(font, 10);
        contentStream.setLeading(12f);
        contentStream.newLineAtOffset(myPage.getMarginLeft(), 700);

        String temp="";

        String[] sentences = data.split("\n");
        for (String sentence : sentences){
            ArrayList<String> lines = wrap(sentence, font, myPage);
            for (String line : lines){
                contentStream.showText(line);
                contentStream.newLine();
            }
        }
        contentStream.endText();

I want to align the text to right. I have lot of strings in ArrayList lines. How can I align it to right?

There's no method that causes the text to align to the right. A similar effect can be achieved with something like this code. You will need to shift each line over so that they align to the right by computing the length of each line.


Enable Discussions on this Repo Please

Can discussions be enabled on this repo please? This will provide a better approach for many to ask questions/seek help as opposed to these being opened as issues and then being closed. Since it will be attached to the repo - information will remain for others to search etc.
Very simple to enable under Settings and categories etc. can be setup with Discussions.
Am aware of the Reddit community but there its barely active!
Thanks

Thanks for the idea, will do :)


ImportError: No module named 'core.rpn_generator

Hi, when I try to run the same command in #1 , an error occurs:
ImportError: No module named 'core.rpn_generator'
And I cannot find any codes with rpn_generator.
Thanks.

Sorry about that, added now.


Add spring-integration-feed & spring-integration-file dependencies

Hi,
The Spring Integration getting started suggests to:

Download and unzip the source repository for this guide, or clone it using Git: git clone https://github.com/spring-guides/gs-integration.git
cd into gs-integration/initial
Jump ahead to Define an Integration Flow.

This path skip the Add to the Build Files section, which ask to add spring-integration-feed & spring-integration-file dependencies.
Therefore, someone following the guide might encounter an error. Based on #14, it looks it is expected to have already these dependencies declared in the build files.

@nhumblot ,
Thank you for your contribution; looking forward for more!
I think at the point this Guide was developed those deps were included into Spring Boot management.
That's how they were not required.
Things has been changed over years,  and now dependency management on Spring Boot level is more strict.
I don't think it is a big problem though to have these deps added as a part of this guide: it is going to demonstrate a version management for this kind of Spring Integration modules or similar in other Spring portfolio projects.


[+] Feature Request : File Extension Detection Patterns

Feature Request:
A way to detect the files based on the given File Extension patterns?
Ex:
    match:       ".pem",
    description: "Potential cryptographic private key",

    match:       ".pcap",
    description: "Network traffic capture file",

Hi @zricethezav ,
Any update on the feature?

Hi @zricethezav,

Please provide an update regarding the feature,

Thanks!

It’s not my project, but IMO rules based on file path and/or extension alone would not likely provide much value. For example, pem files can contain public keys (non sensitive) as well as private keys, and I can’t really think of many examples of file extensions that are exclusively for secrets.
That said, I have considered making a feature request for adding the git commit hash, file path, and line number to the whitelist and blacklist patterns, kind of like the git-secrets Project.

but IMO rules based on file path and/or extension alone would not likely provide much value.

Yes, There would be high chances for FP since we are just checking the file extensions & filing an offender leak,
How about keeping this feature  as optional,
if the user defines a set of extensions in gitleaks.toml then only gitleaks look for leaks along with extensions.
Please look at how Gitrob does the extension based leaks.

https://github.com/michenriksen/gitrob/blob/master/core/signatures.go

@arbazkiraak I agree with what you're saying. This would be a useful option. No progress on this feature yet, but I have it on my roadmap.

This would be a very welcome addition! We recently dropped a similar library (diffence) that offered this due to performance and maintenance issues. Would be great to rely entirely on gitleaks.
I might suggest expanding the scope of this feature more generically to "filename matches" rather than just extensions.
Beyond gitrob signature list linked above, here's diffence's rule list.

https://github.com/zricethezav/gitleaks/wiki/config-toml you can now check match file regexes with filetypes in the config

See https://github.com/zricethezav/gitleaks/#rules-summary


template engine also for errors

We have to come up with a better error handling, we want to be able to use the template engine for ALL errors, except maybe the internal server errors, but we do not want to recreate the object in the exception.

I think it already does that...


Make chameleon fibers mysterious

About the PR

Made chameleon fibers be listed as "mysterious fibers" instead of "chameleon fibers" to the detective's scanner.
Why / Balance

Makes it so that people don't get banned for metagaming because they literally saw "chameleon fibers" in the det scanner, which is admittedly a shitty way to discover someone, but also good for them to prevent them getting banned. Good for everyone involved!
Technical details

Media

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

:cl:

tweak: Chameleon gloves have been made undiscoverable by the Detective's scanner.

This is Huge for thiefs.


In memory security contexts are not supported

See https://github.com/dask/distributed/blob/3754a6694f72207b69aaf661445aecf18fb5f0cb/distributed/security.py#L211
i.e. one cannot use Security.temporary() to create a cluster
distrinuted allows using in memory keys where the content is in memory. It does so by checking if the value is multiline.
the expansion to the extra arguments does not support this:
https://github.com/dask/dask-jobqueue/blob/79a9a3bee55c56195f97b3e70bda15c12c3b19be/dask_jobqueue/core.py#L224
in this case a temporary file needs to be created

Just tracking this because I will need to fix this in #519 to work on #496

I guess the main challenge will be handling the temporary files as they need to survive until the cluster is closed. Maybe also writing the whole cert into the arg works...

And like @jacobtomlinson said we are assuming a shared file system anyways.

I was actually thinking this also the whole time until you reminded me of /tmp. I started digging into the code to find where this assumption is actually encoded. It seems that HTCondor is explicit about this:
https://github.com/dask/dask-jobqueue/blob/26fd3ae8e8c18d26f5e203f2c7f1a1be91e113d4/dask_jobqueue/htcondor.py#L236
However, when setting up the new CI I think I completely ignored this:
https://github.com/dask/dask-jobqueue/blob/9944654374a4e06d3a3289069d2ec0a61ae96162/ci/htcondor/docker-compose.yml#L56-L57
So I think it might probably not depend on this.
Here is a hint that this can be very different on different setups:
https://github.com/dask/dask-jobqueue/blob/1f9ae1ecc79a5b76930e56c8801f3b8ace659877/dask_jobqueue/lsf.py#L201
Here you find hints that even python might not be shared:
https://github.com/dask/dask-jobqueue/blob/58e0ce2111e74484dca4e2469698bbe68a2240b2/docs/source/debug.rst#checking-job-script
Bottom line:  the testing of security should actually be done on all schedulers and the assumption should be dropped, IMHO. It might well be the case that also file based certificates need to be copied to the worker.#524
What do you think.
@guillaumeeb : thanks for nagging me here. I think this led to much clearer thinking. One positive outcome should be that the testing strategy should be overhauled a bit, since the CI run on #519 for all the schedulers should be a warning...

I confirmed that actually file based certificates also won't work on the HTCondor and slurm CI: #532 . Now the price question: is this a bug of the ci or the code. It should be at least document that one has to be smart enough to put the certificates in some accessible place. On the other hand: how would code fixing this issue know where this is...

Sorry for the delay @riedel, and as always thanks for your investments in dask-jobqueue!

Now the price question: is this a bug of the ci or the code

I'd said neither.
I think all dask-jobqueue Cluster can work without any shared file system in some specific setups. Some CI environments show this. It should remain the same.
The code for generating a temporary file should just try to do it in the user folder, and be explicit (with some logging?) that the place where the file is generated should be accessible from all workers. Maybe we can add a configuration option for this location (as we do for worker log directory). All this should also be clearly documented.
For the CI, we could test this only in CI environment with shared folders. I don't know if we want to add a shared file system for all tests, just to test this. I don't think so.


fix: unset output.file when input is "index.html"

Sorry @yyx990803, I should've noted that I was dealing with an error still. 😅
vite-plugin-mobile is broken until this is merged. 😢

Merged into #1074


Compiling MxNet: Problems with CMake

Hello,
I want to add a new operator to MxNet and for that I need to able to compile the MxNet library. I am running into some problems in the CMake stage and could not build the Visual Studio solution.
I installed the prebuilt MxNet library for Windows and the python package. In addition, I downloaded and extracted the source code for the MxNet. I configured the CMake so that it points to the source code coming from the extracted code.
Firstly, I have installed the OpenCV library: I set the environment variable OpenCV_DIR to the build folder as "C:\Users\ufuk.bicici\Desktop\OpenCV\opencv\build" and I add %OpenCV_DIR% to the Path variable. This stopped CMake to complain about finding OpenCV.
Then I downloaded the mshadow tensor source code. I extracted the contents under the mshadow folder, which is empty by default in the MxNet source code. CMake seemed happy to see related mshadow .cmake files and errors about mshadow library ceased.
In the next step, I added CUDNN_ROOT environment variable, pointing to "C:\Users\ufuk.bicici\Desktop\cuDNN\cuda" where the cuDNN,s bin, include and lib folders reside.
Lastly, I downloaded the OpenBlas library. I placed it in a separate folder at "C:\Users\ufuk.bicici\Desktop\OpenBLAS" and set its environment variable "OpenBLAS_HOME" to this path. But CMake finds an another OpenBlas under the MxNet  installation and it detects where the include files are, under the folder: "C:/Users/ufuk.bicici/Desktop/MxNet/20160419_win10_x64_gpu/3rdparty/openblas/include". This is the folder where the prebuilt package is installed. Then it throws the following error:

Could not find OpenBLAS lib. Turning OpenBLAS_FOUND off
CMake Error at cmake/Modules/FindOpenBLAS.cmake:59 (MESSAGE):
Could not find OpenBLAS
Call Stack (most recent call first):
mshadow/cmake/mshadow.cmake:13 (find_package)
CMakeLists.txt:17 (include)
Configuring incomplete, errors occurred!
See also "C:/Users/ufuk.bicici/Desktop/MxNet/mxnet-20160419/build/CMakeFiles/CMakeOutput.log".

Looking to FindOpenBLAS.cmake, it seems that the following line fails:
FIND_LIBRARY(OpenBLAS_LIB NAMES openblas PATHS ${Open_BLAS_LIB_SEARCH_PATHS})
Despite I set the related OpenBlas environment variable and the needed lib files are in place, CMake refuses to detect lib folder for OpenBLAS and it fails. I am simply stuck at that point. What can be the problem here?  Is there any special steps for OpenBLAS installation that I am missing? Thanks in advance.

The file name for OpenBLAS library is libopenblas.dll.a, which can not be recognized by CMake correctly. I am creating a pull request to handle this correctly. Before the PR is merged, you can manually point OPENBLAS_LIB to the libopenblas.dll.a file.

Changed the related line to:
FIND_LIBRARY(OpenBLAS_LIB NAMES libopenblas PATHS ${Open_BLAS_LIB_SEARCH_PATHS})
Unfortunately, this didn't help, the error continues.

Cmake cannot recognize the .a suffix on Windows. So you will need to use
FIND_FILE and specify the full file name.
You can take a look at the PR here:
https://github.com/dmlc/mxnet/pull/1548/files#diff-a16f56795f78049b03a273dcc09b7bc9R38
On Tue, May 31, 2016 at 11:30 PM, ufukcbicici<EMAIL_ADDRESS>wrote:

Changed the related line to:
FIND_LIBRARY(OpenBLAS_LIB NAMES libopenblas PATHS
${Open_BLAS_LIB_SEARCH_PATHS})
Unfortunately, this didn't help, the error continues.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/dmlc/mxnet/issues/2284#issuecomment-222725840, or mute
the thread
https://github.com/notifications/unsubscribe/ABFI4XzXJD4cDaXQe1nyonDEms7pIucaks5qHFQcgaJpZM4IpmpP
.

--
HONG Chuntao
System Research Group
Microsoft Research Asia

@ufukcbicici The PR is already merged. You can checkout the latest code and it should be able to find libopenblas.dll.a correctly.


Does SD have a macro to distinguish between different versions like this

- (id<WXImageOperationProtocol>)downloadImageWithURL:(NSString *)url imageFrame:(CGRect)imageFrame userInfo:(NSDictionary *)userInfo completed:(void(^)(UIImage *image,  NSError *error, BOOL finished))completedBlock
{
  #if SDVersion > 4.0

return (id<WXImageOperationProtocol>) [[SDWebImageManager sharedManager] loadImageWithURL:[NSURL URLWithString:url] options:0 progress:^(NSInteger receivedSize, NSInteger expectedSize, NSURL * _Nullable targetURL) {

    } completed:^(UIImage * _Nullable image, NSData * _Nullable data, NSError * _Nullable error, SDImageCacheType cacheType, BOOL finished, NSURL * _Nullable imageURL) {

        if (completedBlock) {
            completedBlock(image, error, finished);
        }
    }];
#else

return (id<WXImageOperationProtocol>)[[SDWebImageManager sharedManager] downloadImageWithURL:[NSURL URLWithString:url] options:0 progress:^(NSInteger receivedSize, NSInteger expectedSize) {
    } completed:^(UIImage *image, NSError *error, SDImageCacheType cacheType, BOOL finished, NSURL *imageURL) {
        if (completedBlock) {
            completedBlock(image, error, finished);
        }
    }];
#endif
}

Currently we do not have such macro to detect version. If you want to write an library that supports both 3.7.x and 4.0. Maybe curently the only solution is to use runtime dynamic invoke such as this tricky code:
- (id<WXImageOperationProtocol>)downloadImageWithURL:(NSString *)url imageFrame:(CGRect)imageFrame userInfo:(NSDictionary *)userInfo completed:(void(^)(UIImage *image,  NSError *error, BOOL finished))completedBlock
{
    SEL loadImageWithURL = NSSelectorFromString(@"loadImageWithURL:options:progress:completed:");
    SEL downloadImageWithURL = NSSelectorFromString(@"downloadImageWithURL:options:progress:completed:");
    if ([[SDWebImageManager sharedManager] respondsToSelector:loadImageWithURL]) {
        return objc_msgSend([SDWebImageManager sharedManager], loadImageWithURL, [NSURL URLWithString:url], 0, ^(NSInteger receivedSize, NSInteger expectedSize, NSURL * _Nullable targetURL) {}, ^(UIImage * _Nullable image, NSData * _Nullable data, NSError * _Nullable error, SDImageCacheType cacheType, BOOL finished, NSURL * _Nullable imageURL) {

            if (completedBlock) {
                completedBlock(image, error, finished);
            }
        });
    } else {
        return objc_msgSend([SDWebImageManager sharedManager], downloadImageWithURL, [NSURL URLWithString:url], 0, ^(NSInteger receivedSize, NSInteger expectedSize) {}, ^(UIImage *image, NSError *error, SDImageCacheType cacheType, BOOL finished, NSURL *imageURL) {
            if (completedBlock) {
                completedBlock(image, error, finished);
            }
        });
    }
}

Also, since 4.0, SD changed many methods and this compatibility layer maybe hard to implement. You'd better consider to provide an special branch or tag just for SD3.7.x and keep another branch or tag for SD4.0.

@songxing10000 objc_msgSend is Objective-C runtime method. You should import "objc/runtime.h"(or specify clearly "objc/message.h")...

@dreampiggy thanks,I'll be try

Closing per comments from @dreampiggy


T 2905

[x] - Does the code comply with the style guide? (Run make lint from the Wagtail root)
[x] - For new features: Has the documentation been updated accordingly?

@tomdyson updated

I'm unable to set a focal point after having removed the existing focal point. It is possible to edit the existing focal point but not to set a new one after removing it.

@HvandenBerg re-test please, its fixed. Point is now area.

re-test OK

Thanks! Merged in 270ac66.
In addition I noticed that the focal point was showing up visually without having it set. Probably due to the enhanced visual border. Turns out there was invalid check within the edit template to determine whether the focal point was set or not. See 4ab57dd


Publish Scala 2.12.0 artifact

Can you please publish 2.12.0 artifact ?

I'm still waiting on typeleve/cats#1450 to update. They should be ready soon.


issue-260 - reduce TBT

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 #260
Test URLs:

Before: https://main--famous-smoke-cigaradvisor--hlxsites.hlx.page/cigaradvisor/must-reads
After: https://issue-260--famous-smoke-cigaradvisor--hlxsites.hlx.page/cigaradvisor/must-reads

We get 100/100 PSI with this change
https://pagespeed.web.dev/analysis/https-issue-260--famous-smoke-cigaradvisor--hlxsites-hlx-live-cigaradvisor-must-reads/rbvyr3t4bx?form_factor=mobile


Automatic deletion of audio transcript

In order for GPT to remember the chat context, i have a lengthy prompt, which it reads before listening to follow up questions.  This is causing it to exceed the allowed token limit.  Is there anyway you can add a new  function that a user can select or not select, which will automatically delete audio transcript every 10 seconds.

Which model are you using currently?
The default model is gpt-3.5-turbo-0301
as specified in parameters.yaml file
ai_model: gpt-3.5-turbo-0301

This model has a token limit of 4096 tokens.
Using a different model with higher token limit could alleviate the issue you are having as well.
See this article for token limits of various models
https://platform.openai.com/docs/models/gpt-4
The model gpt-4-32k has 32,768 token limit. That should alleviate the issue I believe.

Thanks!, changing the model helps, however it still reads the whole context/previous prompts/previous input.  This adds to the cost of usage.  Automatic deletion after each input, would be huge benefit in terms of cost as well as performance, as I noticed, the response from chatgpt is faster when I clear audio transcript.

I think there are 2 different issues here

Cost - It is correct that with a large prompt the cost will go up and automatically clearing the transcript will alleviate the cost concerns. I will look into this further.

Performance - I have been investigating the performance issues reported here and in an earlier issue.
As the size of transcript increases the performance of Transcribe starts to degrade after a couple of minutes in specific scenarios.

E.g. If a single speaker is doing most of the talking, the performance degrades over time. This situation is not observed if input from speaker, microphone alternates with a few second gaps in the two.
Another parameter that plays a part in the performance side of things is PHRASE_TIMEOUT in AudioTranscriber.py but I haven't been able to determine the exact mechanics as yet.
Really appreciate the feedback and I will look into alleviating the concerns.

The original ask to provide a way for automatic deletion of audio transcripts should be resolved by the PR #73
Looking forward to feedback on the same.

Works like a charm !  It is auto deleting which has improved performance ! Thank you !

Appreciate the feedback !
Glad we were able to resolve the issue for your use case.
Resolved with PR #73.


python311Packages.pyvesync: 2.1.9 -> 2.1.10

Changelog: https://github.com/webdjoe/pyvesync/releases/tag/2.1.10
Description of changes

Things done

Built on platform(s)

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

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

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

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

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

[ ] Fits CONTRIBUTING.md.

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

5 packages built:

home-assistant-component-tests.vesync
python310Packages.pyvesync
python310Packages.pyvesync.dist
python311Packages.pyvesync
python311Packages.pyvesync.dist


Always run flake8 through pre-commit, and with doctests

Description:

make all flake8 invocations go through pre-commit
enable doctests in all invocations

Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass
[x] There is no commented out code in this PR.
[x] I have followed the development checklist

If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated in home-assistant.io

If the code communicates with devices, web services, or third-party tools:

[ ] The manifest file has all fields filled out correctly. Update and include derived files by running python3 -m script.hassfest.
[ ] New or updated dependencies have been added to requirements_all.txt by running python3 -m script.gen_requirements_all.
[ ] Untested files have been added to .coveragerc.

If the code does not interact with devices:

[ ] Tests have been added to verify that the new code works.

Removed stuff that was taken care of or obsoleted by #28538, this one's ready to continue review now.


BVTCK-88 Retrieve method parameter names via new API in Java 8

https://hibernate.atlassian.net/browse/BVTCK-88

I didn't add any new tests as updating the existing one seem to be sufficient.

Rebased and applied. Thx, @gsmet!


require Julia 0.4 instead of a point release

Test passes: https://travis-ci.org/JuliaCloud/LibCloud.jl/builds/157948480

Current coverage is 58.80% (diff: 100%)

Merging #4 into master will not change coverage

@@             master         #4   diff @@
==========================================
  Files             8          8
  Lines           284        284
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits            167        167
  Misses          117        117
  Partials          0          0

Powered by Codecov. Last update 3b21d44...ec7ec8e

Coverage increased (+13.2%) to 58.803% when pulling ec7ec8e6daac60e21f31170116b840b70e998fcf on tanmaykm into 3b21d4477f09aa158d03a00dbb319aa29de82747 on master.


Reduce useless async cache request when read from remote worker

What changes are proposed in this pull request?
reduce useless async cache request when read from remote worker
Why are the changes needed?
It is useless to trigger async cache when the remote worker has already cached block.
Does this PR introduce any user facing changes?
No user facing changes.

/ping @dbw9580

alluxio-bot, merge this please


Added a note

I have added a note regarding the SCP script which is not needed to be executed.
Problem: https://github.com/MicrosoftDocs/windows-itpro-docs/issues/4776

Ready to go live

@jvsam : Do we need a copy review on this document before checking with the author or document manager?

@illfated yes since it's a content update that impacts the context/messaging.
@nenonix @JohanFreelancer9 for copy review. Thank you.

Also tagging @RobG-MSFT here. What do you think about proposed changes/updates to the doc?
Reference: #5305

Ready to go live.

Hi @mapalko, this pull request has been copy edited and is ready for your final review and approval. Thank you.

@officedocsbot assign @mapalko

@RobG-MSFT : Thanks for visiting this Pull Request page and giving feedback on it.
Took me a little while to recognize that you are the issue ticket author. 😄

Looks good to me. Thanks for making the change!

Thanmks @mapalko can you please merge the updates? Thank you.

nenonix suggestions applied.

Hi again @mapalko, just a friendly follow-up. Kindly merge this PR. Thank you.

Fair enough, that means this requires follow-up from the DM/PM Daniel Simpson instead.

Hello again @Dansimp. This is another PR that has been copy reviewed and approved by mapalko. Kindly merge, thank you.
@officedocsbot assign @Dansimp

Hi @Dansimp, just following up on this PR. I hope you'll find time to merge this. @mapalko already approved the proposed changes. Thank you in advance for your assistance.

#sign-off


support aot？

support aot？

It supports Aot (Ready To Run) and NativeAot.

Interesting performances 👀


Phoenix-Stats Correct incorrectly formed UTs with wrong sized slotSpan.

Corrects incorrectly sized slotSpan which should be the same size as the number of elemetns in the List<<List> passed to ScanRanges.create.

@ChinmaySKulkarni simple fix please review.

+1

@dbwong Can you update the commit message to contain the jira number so it automatically links to the jira?

As discussed offline as this is a dev branch and these issues are in our dev code we do not need a JIRA associated with these sharing fixes.

Merged closing.


Issue #101: Persist run-time crawl metrics across crawler restarts

Coverage increased (+0.2%) to 47.35% when pulling 7a01194c49f38547679667e50d2418f979a38212 on anudeepti2004:metrics into<PHONE_NUMBER>98aaafe638b8f65079f06b2f7dfb52 on ViDA-NYU:master.


Prototype/auth-state

Handles authentication state changes, by showing/hiding the login button and features as needed.
Signed Out:

Signed In:

Dependent on PR #17

Handles authentication state changes, by showing/hiding the login button and features as needed.

Please add depends on #<PR num> to the end of your description

@gbuenoandrade Take a look again - it should appear. You may need a test account to use this (unless if you're okay using your @google acct.)


Unit Test Cases: cleanup, static imports for Assertions

Pull Request Etiquette

[x] I have checked the PRs for upcoming features/bug fixes.
[x] I have read the contributing guidelines.

Changes

[ ] Internal code
[ ] Library interface (affecting end-user code)
[ ] Documentation
[x] Other: Unit Test Cases

Closes Issue: NaN
Description
I have noticed an overwhelming amount of Assertions.assertEquals(), Assertions.assertTrue() or other similar static method calls from the Assertions class. I also noticed that some test classes have static imports for Assertions, and some don't. I decided to clean up the code by adding static imports for Assertions for the classes that did not import them already.
Another minor change I made within the code cleanup was that I extracted out a huge builder chain into a separate private method in MessageSerializationTest, to ensure code tidiness.

Thanks, but I've decided to move to AssertJ instead. See #2638. Feel free to leave a review there!


Implicit Conversion Error when POSTing to stubbed request

I'm getting the following error when attempting to post data to a stubbed request:
no implicit conversion of ActiveSupport::HashWithIndifferentAccess into String
/Users/dfre/.rvm/gems/ruby-2.4.3/gems/webmock-3.1.1/lib/webmock/util/json.rb:27:in `initialize'
/Users/dfre/.rvm/gems/ruby-2.4.3/gems/webmock-3.1.1/lib/webmock/util/json.rb:27:in `new'
/Users/dfre/.rvm/gems/ruby-2.4.3/gems/webmock-3.1.1/lib/webmock/util/json.rb:27:in `convert_json_to_yaml'
/Users/dfre/.rvm/gems/ruby-2.4.3/gems/webmock-3.1.1/lib/webmock/util/json.rb:14:in `parse'
/Users/dfre/.rvm/gems/ruby-2.4.3/gems/webmock-3.1.1/lib/webmock/request_pattern.rb:264:in `body_as_hash'
/Users/dfre/.rvm/gems/ruby-2.4.3/gems/webmock-3.1.1/lib/webmock/request_pattern.rb:246:in `matches?'
/Users/dfre/.rvm/gems/ruby-2.4.3/gems/webmock-3.1.1/lib/webmock/request_pattern.rb:38:in `matches?'
/Users/dfre/.rvm/gems/ruby-2.4.3/gems/webmock-3.1.1/lib/webmock/stub_registry.rb:58:in `block in request_stub_for'
/Users/dfre/.rvm/gems/ruby-2.4.3/gems/webmock-3.1.1/lib/webmock/stub_registry.rb:57:in `each'
/Users/dfre/.rvm/gems/ruby-2.4.3/gems/webmock-3.1.1/lib/webmock/stub_registry.rb:57:in `detect'
/Users/dfre/.rvm/gems/ruby-2.4.3/gems/webmock-3.1.1/lib/webmock/stub_registry.rb:57:in `request_stub_for'
/Users/dfre/.rvm/gems/ruby-2.4.3/gems/webmock-3.1.1/lib/webmock/stub_registry.rb:50:in `response_for_request'
/Users/dfre/.rvm/gems/ruby-2.4.3/gems/webmock-3.1.1/lib/webmock/http_lib_adapters/net_http.rb:79:in `request'
/Users/dfre/.rvm/gems/ruby-2.4.3/gems/faraday-0.12.2/lib/faraday/adapter/net_http.rb:80:in `perform_request'
/Users/dfre/.rvm/gems/ruby-2.4.3/gems/faraday-0.12.2/lib/faraday/adapter/net_http.rb:38:in `block in call'
/Users/dfre/.rvm/gems/ruby-2.4.3/gems/faraday-0.12.2/lib/faraday/adapter/net_http.rb:85:in `with_net_http_connection'
/Users/dfre/.rvm/gems/ruby-2.4.3/gems/faraday-0.12.2/lib/faraday/adapter/net_http.rb:33:in `call'
/Users/dfre/.rvm/gems/ruby-2.4.3/gems/faraday_middleware-0.12.2/lib/faraday_middleware/response_middleware.rb:31:in `call'
/Users/dfre/.rvm/gems/ruby-2.4.3/gems/faraday_middleware-0.12.2/lib/faraday_middleware/instrumentation.rb:26:in `block in call'
/Users/dfre/.rvm/gems/ruby-2.4.3/gems/activesupport-5.1.4/lib/active_support/notifications.rb:168:in `instrument'
/Users/dfre/.rvm/gems/ruby-2.4.3/gems/faraday_middleware-0.12.2/lib/faraday_middleware/instrumentation.rb:25:in `call'
/Users/dfre/.rvm/gems/ruby-2.4.3/gems/faraday-0.12.2/lib/faraday/rack_builder.rb:141:in `build_response'
/Users/dfre/.rvm/gems/ruby-2.4.3/gems/faraday-0.12.2/lib/faraday/connection.rb:386:in `run_request'
/Users/dfre/.rvm/gems/ruby-2.4.3/gems/faraday-0.12.2/lib/faraday/connection.rb:186:in `post'

This is occurring when when I POST, however it only is thrown when I've set up a body in the mock call.
This finishes (the test correct fails):
stub_request(:post, "https://domain.com/path")
        .with(headers: basic_headers)
        .to_return(status: 200, body: { token: '1234' }.to_json)
post "path", params: { status: 'reboot' }, headers: basic_headers }

This should succeed but throws the above stacktrace:
stub_request(:post, "https://domain.com/path")
        .with('body' => {"status"=>"reboot"}), headers: basic_headers)
        .to_return(status: 200, body: '')
post "path", params: { status: 'reboot' }, headers: basic_headers }

My best guess is that there's a weird interaction with Faraday and webmock?
I'm still researching to get more information.

I had a bug in my code that should have been converting the has to a json string before sending it to faraday. I'm Not sure if this is valid or not, but if no no else is seeing this i'll close this out as user error.


Crash in FlurryContentLoadingManager deRegisterPreCachingDelegate (array out of bounds)

Related to #43 (but different).
Version 7.6.3 of the FlurrySDK and FlurryAds frameworks.
We're seeing another crash during runtime that looks like:
Fatal Exception: NSRangeException
0  CoreFoundation                 0x181552db0 __exceptionPreprocess
1  libobjc.A.dylib                0x180bb7f80 objc_exception_throw
2  CoreFoundation                 0x181432dfc -[__NSArrayM removeObjectAtIndex:]
3  CoreFoundation                 0x18144b068 -[NSMutableArray removeObject:]
4  Flurry_iOS_SDK                 0x100cb0664 __60-[FlurryContentLoadingManager deRegisterPreCachingDelegate:]_block_invoke (FlurryContentLoadingManager.m:1385)
5  Flurry_iOS_SDK                 0x100ca9ed0 __65-[FlurryPreCachingDelegatesMapManager processReadBlockWithBlock:]_block_invoke (FlurryContentLoadingManager.m:88)
6  libdispatch.dylib              0x180f9d4bc _dispatch_call_block_and_release
7  libdispatch.dylib              0x180f9d47c _dispatch_client_callout
8  libdispatch.dylib              0x180fa7d50 _dispatch_async_redirect_invoke
9  libdispatch.dylib              0x180f9d47c _dispatch_client_callout
10 libdispatch.dylib              0x180fab914 _dispatch_root_queue_drain
11 libdispatch.dylib              0x180fab0b0 _dispatch_worker_thread3
12 libsystem_pthread.dylib        0x1811b5470 _pthread_wqthread
13 libsystem_pthread.dylib        0x1811b5020 start_wqthread

The exception thrown is:
Fatal Exception: NSRangeException
*** -[__NSArrayM objectAtIndex:]: index 254 beyond bounds [0 .. 253]

What version of the Flurry SDK are you using?

  pod 'Flurry-iOS-SDK/FlurrySDK', '~> 7.6.3'
  pod 'Flurry-iOS-SDK/FlurryAds', '~> 7.6.3'

@zacwest , what type of ads have you integrated with? are these interstitials, banners or native ads?

@abhandary We're integrating with native ads

Same here. Version 7.6.0.
The stack
Crashed: com.Flurry.PreChachingDelegatesMap 0  libobjc.A.dylib                0x20bffa66 objc_msgSend + 5 1  TapatalkFree                   0x9e53e5 __60-[FlurryContentLoadingManager deRegisterPreCachingDelegate:]_block_invoke + 9475045 2  TapatalkFree                   0x9de871 __65-[FlurryPreCachingDelegatesMapManager processReadBlockWithBlock:]_block_invoke + 9447537 3  libdispatch.dylib              0x20fc5823 _dispatch_call_block_and_release + 10 4  libdispatch.dylib              0x20fd002d _dispatch_async_redirect_invoke$VARIANT$mp + 1360 5  libdispatch.dylib              0x20fd45e9 _dispatch_root_queue_drain + 1560 6  libdispatch.dylib              0x20fd3fcd _dispatch_worker_thread3 + 96 7  libsystem_pthread.dylib        0x21189b29 _pthread_wqthread + 1024 8  libsystem_pthread.dylib        0x21189718 start_wqthread + 8

This has been fixed as of 7.8.0.  Please let us know if you have any further issues.


DocC documentation for JavaKit and related modules

Our general user guide is somewhat of a "getting started" guide, but we should move toward a more comprehensive model for documentation, which involves integrating DocC into the build and migrating reference content over there. This issue covers adopting DocC and starting that migration, not necessarily writing new content (that can be covered by separate issues). We'd like this documentation to be available on the Swift Package Index as well.

hi @DougGregor ! may I take up this issue? I think it’d be a great way to get familiar with the codebase 👍

Sure, please do. If you have any questions let us know here or on the forums.
A basic docc page to get us started would be good, and we'll likely want an article for javakit and for jextract, but we can fill in those later.
You could have a look at the https://github.com/swiftlang/swift-java/blob/main/USER_GUIDE.md to seed some contents into it.


Support bit-wise operators in formula generation

ban, bor, bxo.
Does PARI support bitwise operators? Is the semantics the same? What is the notation?

added with limitations


passing autoPlay 'true' crashes the app

Hey Leo, great job with the package, very cool. I found it really useful for my project!
Here is one thing that I found not working.
Attempt to use autoPlay={true} crashes the app with ReferenceError: goToNextSlide is not defined.
Steps to reproduce:

pass autoPlay={true} to your instance of Gallereact

Tested both on my app, as well as on the example provided in the /examples directory. It looks like it gets triggered in src/Gallereact.js line 28 but I'm not sure.

Did you fix it?

well, not, but did a dirty workaround
const Slider = () => {
  const [index, setIndex] = useState(0);

  useEffect(() => {
    const timeoutId = setTimeout(() => {
      if (index < IMAGES_FOR_SLIDER.length - 1) {
        setIndex(index + 1);
      } else {
        setIndex(0);
      }
    }, 4000);
    return () => clearTimeout(timeoutId);
  }, [index]);

  return (<Gallereact inputIndex={index} />);
}

Hey @rafwit!
Sorry, it has been a while that I didn't check the project 😅
I must have missed it during the last refactor, this is why I should have had tests...
Feel free to make a PR if you want otherwise I'll try to fix it in the next weeks, at the latest during the Xmax holidays!
Thank you so much for the comment, hope you're doing great!

Hey, @div-Leo! all good, enjoying the dev life a lot 🌈🚀
Hope all is good on your end too!
I opened PR #36 🎁 with my proposition of the fix and a little adjustment of the contribution guide to (I hope) make it easier for others to contribute.
All the best to you!


comma three: excessive CPU usage after "fatal modem error"

This was much more common on the comma two, where we ran a daemon that would monitor the modem crash count and offline it if it got too high.
6480130cc87ac628|2023-03-02--06-58-37--13
[820.956275] MAIN 0 kernel - Fatal error on the modem.
[820.956327] MAIN 0 kernel - modem subsystem failure reason: (unknown, smem_get_entry_no_rlock failed).

Another case
https://connect.comma.ai/f2a06f696f9517b3/1688227804903/1688227886668


R package: feature_pre_filter vanished

Description
Version: LightGBM 3.3.2 in R.
In a grid search, I am trying to optimize (amonst others) the parameter min_data_in_leaf. When the value of this parameter is getting smaller, an error is launched that feature_pre_filter=FALSE should be set in dataset construction. However, that parameter does not exist anymore (it used to be part of the ..., which was removed in the latest version).
Here is the message:
[LightGBM] [Fatal] Reducing min_data_in_leaf with feature_pre_filter=true may cause unexpected behaviour for features that were pre-filtered by the larger min_data_in_leaf.
You need to set feature_pre_filter=false to dynamically change the min_data_in_leaf.
Error in doTryCatch(return(expr), name, parentenv, handler) :
Reducing min_data_in_leaf with feature_pre_filter=true may cause unexpected behaviour for features that were pre-filtered by the larger min_data_in_leaf.
You need to set feature_pre_filter=false to dynamically change the min_data_in_leaf.
Reprex
library(lightgbm)

dtrain <- lgb.Dataset(
  data.matrix(iris[, c("Sepal.Length", "Petal.Length")]),
  label = iris$Sepal.Width,
  #feature_pre_filter=FALSE
)

# Grid with decreasing min_data_in_leaf
grid <- expand.grid(
  iteration = NA,
  score = NA,
  learning_rate = 0.05,
  objective = "regression_l2",
  metric = "rmse",
  min_data_in_leaf = c(10, 5),
  stringsAsFactors = FALSE
)

for (i in seq_along(grid$iteration)) {
  cvm <- lgb.cv(
    params = as.list(grid[i, -(1:2)]),
    data = dtrain,
    nrounds = 5000,
    nfold = 5,
    early_stopping_rounds = 2,
    verbose = -1
  )

  grid[i, 1:2] <- as.list(cvm)[c("best_iter", "best_score")]
}

Thanks for the report and reproducible example, @mayer79 !
I've reduced the size of the example a bit, and can confirm the following code run on the latest commit of LightGBM (https://github.com/microsoft/LightGBM/commit/dad391574137958402bec4b417bc594fa1c5fbed) produces the error you mentioned.
library(lightgbm)

dtrain <- lgb.Dataset(
  data.matrix(iris[, c("Sepal.Length", "Petal.Length")])
  , label = iris$Sepal.Width
)

cv_booster <- lgb.cv(
  params = list(
    objective = "regression_l2"
    , min_data_in_leaf = 10L
  )
  , data = dtrain
  , nrounds = 10L
  , nfold = 2L
)

# run cv again with a smaller min_data_in_leaf
cv_booster <- lgb.cv(
  params = list(
    objective = "regression_l2"
    , min_data_in_leaf = 9L
  )
  , data = dtrain
  , nrounds = 10L
  , nfold = 2L
)

[LightGBM] [Fatal] Reducing min_data_in_leaf with feature_pre_filter=true may cause unexpected behaviour for features that were pre-filtered by the larger min_data_in_leaf.

As mentioned in https://lightgbm.readthedocs.io/en/latest/R/reference/lgb.Dataset.html#arguments, any of the Dataset-specific parameters found at https://lightgbm.readthedocs.io/en/latest/Parameters.html#dataset-parameters can be provided to lgb.Dataset() via that params list.
I tried running the same code, but just changing the lgb.Dataset() call to the following.
dtrain <- lgb.Dataset(
  data.matrix(iris[, c("Sepal.Length", "Petal.Length")])
  , label = iris$Sepal.Width
  , params = list(
    feature_pre_filter = FALSE
  )
)

Using the R package built from the latest commit of LightGBM (https://github.com/microsoft/LightGBM/commit/dad391574137958402bec4b417bc594fa1c5fbed) and v3.3.2 installed from CRAN, that example works for me without any errors.

The params argument... thanks a lot for that hint! This will solve the problem.


Cant install face_recognition (stuck at bdist_wheel for dlib...)

face_recognition version: latest
Python version: 3.5.2
Operating System: Ubuntu Server 16.04

Description
Trying to install the face_recognition package on the server running ubuntu 16.04. But it gets to the following line of the install Running setup.py bdist_wheel for dlib ...  and then stays there indefinetely.
I have installed dlib etc seperately too. Had to install dilb without cuda support due to the server not having any gpus. See below for terminal output and where it gets stuck.
What I Did
xxxxx@xxxxx:/$ pip3 install face_recognition
Collecting face_recognition
  Using cached https://files.pythonhosted.org/packages/3f/ed/ad9a28042f373d4633fc8b49109b623597d6f193d3bbbef7780a5ee8eef2/face_recognition-1.2.3-py2.py3-none-any.whl
Collecting numpy (from face_recognition)
  Using cached https://files.pythonhosted.org/packages/d4/64/7619774f0bd8ef364d46a5df8eb1bc78784cd787324b9624f6793e72f787/numpy-1.17.1-cp35-cp35m-manylinux1_x86_64.whl
Collecting Pillow (from face_recognition)
  Using cached https://files.pythonhosted.org/packages/d6/98/0d360dbc087933679398d73187a503533ec0547ba4ffd2115365605559cc/Pillow-6.1.0-cp35-cp35m-manylinux1_x86_64.whl
Collecting face-recognition-models>=0.3.0 (from face_recognition)
Collecting Click>=6.0 (from face_recognition)
  Using cached https://files.pythonhosted.org/packages/fa/37/45185cb5abbc30d7257104c434fe0b07e5a195a6847506c074527aa599ec/Click-7.0-py2.py3-none-any.whl
Collecting dlib>=19.7 (from face_recognition)
  Using cached https://files.pythonhosted.org/packages/05/57/e8a8caa3c89a27f80bc78da39c423e2553f482a3705adc619176a3a24b36/dlib-19.17.0.tar.gz
Building wheels for collected packages: dlib
  Running setup.py bdist_wheel for dlib ... /

Please help.

You should wait until it finishes. It normally takes some time to install :)

So turns out the problem was the server was just running out of memory. Stopped other applications and reran the install and it went through no problem.
FYI this server has 2 core cpu and 4GB ram.


updated to use Requests.jl

resp.status and resp.data

cc @mdpradeep

cc: @mdpradeep , please review.

txs... great package! has been part of our production systems for well over a year.


Runtime error

An attempt has been made to start a new process before the
current process has finished its bootstrapping phase.
    This probably means that you are not using fork to start your
    child processes and you have forgotten to use the proper idiom
    in the main module:

        if __name__ == '__main__':
            freeze_support()
            ...

    The "freeze_support()" line can be omitted if the program
    is not going to be frozen to produce an executable.
ForkingPickler(file, protocol).dump(obj)

BrokenPipeError: [Errno 32] Broken pipe
Namespace(b1=0.5, b2=0.999, batch_size=1, channels=3, checkpoint_interval=-1, dataset_name='facades', decay_epoch=100, epoch=0, img_height=256, img_width=256, lr=0.0002, n_cpu=8, n_epochs=200, sample_interval=500)
Namespace(b1=0.5, b2=0.999, batch_size=1, channels=3, checkpoint_interval=-1, dataset_name='facades', decay_epoch=100, epoch=0, img_height=256, img_width=256, lr=0.0002, n_cpu=8, n_epochs=200, sample_interval=500)
Abnormal to me, this line was printed twice

Duplicate issues. See #7.

same problem when using srgan and esrgan
what is the solution ?


Abuse report viewer (bug 1162160)

First iteration of abuse report viewer.
(I'll collapse the commits shortly)

It needs a bug :)

bug # added

commits squashed, etc.

r+wc


It doesn't work for in-app video

Is this by default that this plugin doesn't work when I build my app on iPhone. I tried to build an app directly on iPhone and it doesn't play video inline. When I open it via Safari it plays inline

You don't need this in your app, but it's only a setting
https://stackoverflow.com/a/39662773/288906


fix(vue-gtag.d.ts): fix for typescript types & deduping utils.js

Fix for issue #314 (In Vue 3.x plugin installations now use the application instance as the first parameter - not Vue).
Removing functions isObject and isFn and replacing with exported utility functions available in Vue 3.x (@vue/shared). The exported functions now used are isPlainObject and isFunction - I just recently found out about these. To see what other utility functions check the vue-next shared package source code.

:tada: This PR is included in version 2.0.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Missing semi column at the end of the plugin

A semi column is missing at the end of the definition of the plugin.
... e.fn._reverse=[].reverse})(jQuery) [missing ; here]
when I compiled all my js files, nivo slider crashed with this error
"TypeError: (intermediate value)(...) is not a function"

You mean a SEMICOLON. Please, put the error message in the issue title for easier finding.


Javascript Generated websites

I tried to use website agent with https://www.mobirum.com/en/knightschronicle, but the output keep returning empty, so I came here and searched issues/wiki, found Browser Emulation with PhatomJS
I removed the first part since I don't have rss to give. I did something wrong, because it doesn't seem to work. Scenario
Thank you in advance.

You need to set url_from_event to {{url}}, but sadly it looks like the phantomjscloud service does not extract the website properly. If you open the url of the Event created by the PhantomJsCloudAgent in your browser it will not contain much HTML.

Yeah, it doesn't. So bothersome, I have no Idea what I could do with my limited skills.

https://www.browserless.io/ seems to be working well, their code is open source and they have a docker image: https://github.com/joelgriffith/browserless#-docker-quickstart
curl -X POST \
  http://localhost:3000/content \
  -H 'Cache-Control: no-cache' \
  -H 'Content-Type: application/json' \
  -d '
{
  "url": "http://www.mobirum.com/en/knightschronicle"
}'| pup ".c_atc_tt:nth-child(1) strong"

  [Notice]

 Novice Knights Guide - 3

  [Notice]

 Arena Recommended and Banned Heroes List (2nd week of August)

  [Notice]

 KnightsGram #3

.....

I'm the free-ish Heroku to host huginn, as it was the best I could find for this.  I'm not sure If I will be able to use browserless together with huginnI(Both by lacking how-to and Heroku itself)
I will try to read on it. Appreciate your help.

Great, this completely solved my original problem, I just need to change the URL to https://localhost:3000/{{url}} in the website agent. I immediately tried it with docker, which is much better than PhatomJS.


Not able to create Virtual Machine

What happened:
Got : 27: mapping values are not allowed in this context
while implementing: https://github.com/kubevirt/kubevirt/blob/main/examples/vm-alpine-datavolume.yaml
Reference :

What you expected to happen:
A virtual machine should be created instead.
How to reproduce it (as minimally and precisely as possible):
Apply : https://github.com/kubevirt/kubevirt/blob/main/examples/vm-alpine-datavolume.yaml
Additional context:
The Code seems to be fine  I didn't get where could be the error :

/label good-first-issue

@shubham-cmyk I just tried on my local cluster and I can create the VMI from the example. Which kubectl version are you using? If you add the --dry-run=client flag, does it still complains?

Well, it was to simply make sure it is on client's side. Haven't you changed the example? (Sorry, I need to ask). Otherwise, it is probably due to your local kubectl version. Can you please provide the version? Also you can try to strace it and see where it is launching the error.

@shubham-cmyk : It looks like issue with the source url. Try deploying by removing "docker://registry:5000/" in url.
source:
        registry:
          url: **kubevirt/alpine-container-disk-demo:devel**

@shubham-cmyk Together with @lvyanru8200 we discovered that if you want to use a url with kubectl, the url should return a raw text (https://github.com/kubevirt/kubevirt/blob/main/examples/vm-alpine-datavolume.yaml does not return a raw text). For your specific use case you should use:
k create -f https://raw.githubusercontent.com/kubevirt/kubevirt/main/examples/vm-alpine-datavolume.yaml

Thanks

Yes issue can be closed


Can't run desktop app anymore

After updating all dependecies the desktop app doesn't compile anymore.

Fixed by https://github.com/Gosunet/KrepesMultiPlatform/pull/9


Notification Entry Type correction - "due data" should be "due date"

The following:
"Notification Entry Type | Specify if the workflow notification is triggered by a record change, an approval request, or passed due data."
Should probably read as:
"Notification Entry Type | Specify if the workflow notification is triggered by a record change, an approval request, or passed due date."
Cheers, Ian

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

ID: cca7d126-fb60-435b-0e57-07e84be35c24
Version Independent ID: 6b113e2a-c524-65b7-32c9-fc9b257a9ef2
Content: Create Approval Workflows to Connect Tasks - Business Central
Content Source: business-central/across-how-to-create-workflows.md
Service: dynamics365-business-central
GitHub Login: @brentholtorf
Microsoft Alias: bholtorf

Hello,
Thanks for pointing this out. We've corrected the typo and will soon publish an update.


Is it possible to use electron-vue under android platform?

Hello,
I am new to electron-vue, just curious if it is possible to use electron-vue under android platform? So we could have one code base, and cross-platform, supporting android as well ...
Thanks a lot.
Best regards, Jason

I found this issue:
https://github.com/SimulatedGREG/electron-vue/issues/380
Is it planned that nativescript-vue will be supported in electron-vue in the future which makes android/ios to be target platform supported ?


Link geobadges resources to TeachOSM

Create explicit links from TeachOSM to the relevant badges on http://geobadges.org (The badges will be developed as part of a separate process organized by @jonpmarino).

Duplicate of #86


Error after deploying on NOW

Hey guys,
I have my project:
https://github.com/numbofathma/roninmobile
successfully deployed on now:
https://roninmobile-boaxvpc5o.now.sh/
but in the console I get:
"TypeError: Cannot read property 'react' of undefined".
Any ideas? (Is my first project with next.js and now)
Thanks!

@numbofathma This looks like it's due to the way you're handling react-loader-spinner. It works fine when removing react-loader-spinner. This also occurs locally and not just after deploying to now

Awesome! How did you identified that so I know were to look next time? What did you run? I've built the project locally and I just can get that error :( What did you do? :-S
Thank you! 👍

Fudge... I forgot I ran npm update last night :| Sorry :( Ummm I'll handle it, you can close it! Pffff!
Thank you very much!


[CON-170] This example didn’t do what I expected for a large refactor. Deleted a ton of relevant code (the write model). Command: /edit Modify all the ReadModel types so the only thing they do is extend SequelizeFields, leave the rest of the code as is. 436 lines of code

null
CON-170

@TyDunn is this something you did, or someone else?

@TyDunn bumping. Sorry if you answered my q somewhere else

@sestinj Let's discuss on the call


Allow concurrent connection setup for the maker

Thanks for tackling this! I thought it through and I think it's OK but I find the connection lifecycle hard to follow and understand. I hope we will be able to plug in Quick soon, that will hopefully make the flow easier to understand?

The protocol is called QUIC, not Quick. What QUIC will improve is the routing of messages to particular instances of actors. Additionally, because QUIC implies TLS, we are not going to have the noise handshake. The rest of the connection lifecycle stays pretty much the same. We still need to accept connections, do the Hello handshake and send heartbeats (if we want them).

One question at the start: Was it previously possible to have concurrent connection setups with different takers? I thought that already worked. I am not sure we need concurrent connection setups with the same taker to be honest - what would be the advantage of that? I thought that we would just add a timeout for the handshake and that would do. What are the advantages of this solution?

No it was not possible. We previously blocked the handle_new_connection_impl on the completion of the upgrade so new items were not pulled from the stream of incoming connections until the current upgrade completed.

Here is my understanding of the current solution:
When the taker connects we add a task for setting up the connection. This tasks ends after setup_new_connection completes. The setup_new_connection task first upgrades the connection. Upgrading the connection includes the noise handshake and the hello message. If upgrading fails we just log an error and return from setup_new_connection, which will result in the task completing without setting up the connection. If upgrade worked, we add tasks for reading messages and the heartbeat and return the connection to ourselves via ConnectionReady. Since setup_new_connection is handled as task we can use send for sending ConnectionReady. Handling ConnectionReady notifies the cfd actor of the taker connection and then inserts the connection. The inserted connection is used by sendToTaker and when broadcasting the order.

Correct. Sending a message to ourselves enables the handling multiple connection upgrades.concurrently.

We remove a connection (by calling drop_taker_connection) in three different scenarios:
1. we are unable to send the order upon broadcasting (we clean them up after finishing the loop)

2. we are unable to send a `MakerToTaker` message to the taker

3. we are unable to read from the connection

The lifecycle of the tasks that we add in setup_new_connection is handled by completing them upon failure to send using the connection.
We don't drop the connection when sending the heartbeat fails. I find that a bit odd, but I think it is a conscious decision (?) because we just end the task, the connection cleanup is handled somewhere else. I feel we could also call drop there.

We could clean up the message upon heartbeat failure. Currently, we rely on the taker to drop the connection if they don't receive a heartbeat within a certain timeout.

I am not entirely sure why we have a ReadFail message and not just a message for dropping the connection? - I think that could align dropping the connection for all cases where we run in an error sending/reading.

I'd argue it is cleaner if a task only reports its failure and does not know what the consequence of that is.

I am merging this so we can backport it!

bors r+


Compile LESS Themes

I could not find a guide how to compile UI5 LESS styles to CSS, like the toolset to be configured and commands to use ? Any example available ?

You can find the LESS build module here: https://github.com/SAP/less-openui5
It can be used as plain node.js module (no CLI).
If you are using grunt you can use our openui5_theme task which re-uses the module above.
Regarding the LESS themes of the OpenUI5 libraries in this repo, please have a look into our tooling docs as well as grunt openui5_theme configuration.
Hope this helps 🎉

Thanks!


Make sure we do not proxify our own proxy URL

Inside function proxify_url($url, $base_url = '') we make sure that:

The proxy cannot proxify "itself".
Allow only http, https, ftp in scheme on $url.
Not proxify localhost and internal IP addresses.

Good improvements for security reasons.

This can be closed, I made a simple SecurityPlugin and submitted the PR here:
https://github.com/Athlon1600/php-proxy-plugin-bundle/pull/2


Custom Group Topic name does not work

Perform all steps below and tick them with [x]

[x ] Check the related part of the Documentation
[x ] Update openHASP to the latest version
[ x] Reproduce the issue and describe all steps

Describe the bug
The MQTT group topic is provided to enable sending an MQTT message to multiple selected plates simultaneously.  The default name setting is plates and if this is left unchanged the group topic works.  However, if you change the group topic name in the web interface it does not work.  The GUI shows the new name but the plate still only subscribes to the 'plates' group topic.  Effectively this means it is not possible to use the Group Topic function as intended.
To Reproduce
On a plate create an object such as:
{"page":1,"id":1,"obj":"label","x":10,"y":10,"w":120,"h":60,"text":"---","click":0,"text_font":42,"parentid":10}

Then from MQTT Explorer (or Home Assistant) send the command hasp/plates/command/p1b1.text with the value Hello World and the text will change.
Now rename the Group Topic to mygroup.  Save and restart the plate.
Then from MQTT Explorer (or Home Assistant) send the command hasp/mygroup/command/p1b1.text with the value Have a Nice Day and the text will not change.
Confirmed by smcgann99 on Discord
openHASP rc11
Expected behavior
The ability to use a custom group name as described in the documentation.  Otherwise, the Group Topic has the same functionality as the Broadcast Topic
Screenshots or video

I can't understand that.
I have copied the jsonl line but deleted "parentid". I can send an MQTT group message to the label without any problems.
Please show more of your "pages.jsonl" and the complete text at Group Topic.

If you send a message with the default group name of pages it all works OK.  It is when you change the name of the group it no longer works.

it remains "hasp/plates/command/p1b1.text" despite group topic "hasp/mygroup/%topic%"
I show later in source code.

I have found the place where the problem lies but I have no explanation for the error.
In "src/mqtt/hasp_mqtt_esp.cpp" i have added 2 LOGs
String mqttGetTopic(Preferences preferences, String subtopic, String key, String value, bool add_slash)
{
    String topic = preferences.getString(key.c_str(), value);

    LOG_INFO(TAG_MQTT,"mqttGetTopic 1 key[%s] topic[%s] value[%s]", key.c_str(), topic.c_str(), value.c_str());
    LOG_INFO(TAG_MQTT, "mqttGetTopic 2 key[%s] topic[%s]", FP_CONFIG_GROUP_TOPIC, preferences.getString(FP_CONFIG_GROUP_TOPIC, "notfound").c_str());

    topic.replace(F("%hostname%"), haspDevice.get_hostname());
    topic.replace(F("%hwid%"), haspDevice.get_hardware_id());
    topic.replace(F("%topic%"), subtopic);
    topic.replace(F("%prefix%"), MQTT_PREFIX);

    if(add_slash && !topic.endsWith("/")) {
        topic += "/";
    }
    return topic;
}

This function is called twice in "void mqttStart()". The output of the second log line has exactly the same parameters in both calls but the value of Topic is different. Why?
MQTT: mqttGetTopic 1 key[node_t] topic[hasp/foo/%topic%] value[hasp/%hostname%/%topic%]
MQTT: mqttGetTopic 2 key[group_t] topic[hasp/mygroup/%topic%]
MQTT: mqttGetTopic 1 key[group_t] topic[hasp/plates/%topic%] value[hasp/plates/%topic%]
MQTT: mqttGetTopic 2 key[group_t] topic[notfound]

"preferences.getString" returns different results.

On the second call of "mqttGetTopic" "preferences.getString" returns the error INVALID_HANDLE

Here the backlink to my similar issue: #686

The problem ist multible call of "mqttGetTopic" and in it the call of "preferences.getString".
In first call all ok, in all further calls no longer.
please review my fix.

Thanks all for looking into and fixing this

@MrEcosse Is this working for you now ?
If so you can close the issue ?

For your information: The fix from above is a workaround. I have continued to search for the actual cause.
The actual problem is the parameter transfer from "preferences" to the function mqttGetTopic. The function works with a copy of "preferences". This can be fixed if only the pointer to "preferences" is passed to the function "mqttGetTopic".
String mqttGetTopic(Preferences *preferences, String subtopic, String key, String value, bool add_slash)
Only add the asterisk and call the function with
mqttNodeCommandTopic = mqttGetTopic(&preferences, subtopic, FP_CONFIG_NODE_TOPIC, MQTT_DEFAULT_NODE_TOPIC, false);
Only add ampersand. This means that you are not working with a copy of "preferences" but with the same instance. Working with the copy only works with the first call of "mqttGetTopic" from the second call onwards.


PLUGIN-1520 fix get schema for password with special chars

Fixed the source to properly set the password and username for schema fetching when there is a special character in the password.
Also fixed a validation bug where a password with a special char would incorrectly use the encoded password instead for auth.

cherry-pick of #34


Run benchmark codes

Hi, Mosè. I have a problem in running the benchmark test on julia-1.5.2.
I follow the instructions in the documentation (https://giordano.github.io/Cuba.jl/latest/#Performance-1).
After running "CUBACORES=0 julia -e 'using Pkg; import Cuba; include(joinpath(dirname(dirname(pathof(Cuba))), "benchmarks", "benchmark.jl"))'
" in Terminal, it returns
"
[ Info: Performance of Cuba.jl:
0.383992 seconds (Vegas)
0.572846 seconds (Suave)
0.466027 seconds (Divonne)
0.336353 seconds (Cuhre)
[ Info: Performance of Cuba Library in C:
ERROR: LoadError: UndefVarError: LIBPATH_env not defined
"
I have tried to run the benchmark code on MacOS and Debian. Both of them fail with the same error message. I guess it might be related to the fact that the benchmark code was written for Julia 0.7.0-beta2.3 as mentioned in the documentation. If possible, can you check this problem?

Should work now

It works! Thanks a lot!


Feature/exam mode/student compaints

Checklist

[x] I tested all changes and all related features with different users (student, tutor, instructor, admin) on the test server https://artemistest.ase.in.tum.de.
[x] Client: I documented the TypeScript code using JSDoc style.
[x] Client: I added multiple screenshots/screencasts of my UI changes

Motivation and Context
We want to give the students the opportunity to request more feedback or to complain about an assessment.
Description
Insert the complaint interaction component in the exam summary page.
Propagate the required inputs and handle the StudentReviewPeriod
Steps for Testing

Log in to Artemis
Create an exam in a course which allows complaints. Make sure to set the Student review dates
Add different exercises, always add an assessment due date (for programing exercises a due date & check the manual assessment option)
Take part in the exam
Assess the exam exercises with a different account
log back in as a student and view the summary page (after student review begin)
complain/request feedback.
Interact with the requests as a tutor (in the exam tutor dashboard)

Screenshots
Exam with complain enabled (in course settings)

Text Exercise: complaint rejected

**Programming exercise Feedback given

Modeling Exercise complaint accepted

From the student side everything looks good. I was also able to answer feedback request in the tutor exam dashboard. However I cannot get access to complaints via the tutor exam dashboard. Am I missing a date or something?

You cannot see the complaints made for your own correction. Only other tutors can review the complaints about your correction. Try logging in with a different account which is also a tutor

Should the complaint count not be 2/3 complaint because the complaint was rejected?

Interaction as a tutor with the complaint was nearly fault free except for the error which @sjagla27 mentioned earlier. I wasn't able to detect any other errors

@amahmario Are you referring to the complaint number not falling? This is actually intended as we do not want to limit the number of complaints per user. As long as it is within the review period the student is allowed to submit a complaint. I updated the client to not show that description anymore as it is not relevant for the exam mode

Should the complaint count not be 2/3 complaint because the complaint was rejected?

We do not want to limit the amount of complaints per user for exams. I changed it now so that this line is not visible

Not related, but found while testing this:
The due date on quiz exercises cannot be edited.
To reproduce:

create quiz exercise in exam, save it go back
edit exercise, try to set due date only -> not savable

The student can see the buttons before the review time starts:

When I submit a request the confirmation that we submitted it is shown on multiple alert boxes all over the page
When I submitted a complaint for text exercises and looked into it as a tutor, I couldn't change the assessment anymore. I am not sure how the process looks there, so might be valid - but I don't see how the tutors can react to complaints. If the review time is after the assessment due dates and the assessments can only be changed before the assessment due date -> tutors cannot react to complaints in terms of changing the assessment

In general it worked, well for me. There are some things I noted, but I think most of them are out of scope of this PR.
Things I tested:

complaints correctly shown for programming and text exercises
more feedback request correctly shown for modeling exercises
ability to make complaints / feedback requests was correctly unlocked/locked as a exam participant
did not see any console error

The things I noticed, which are probably out of scope/irrelevant:
The student can see the buttons before the review time starts:

When I submit a request to create a complaint / more feedback request the confirmation that we submitted the request it is shown on multiple alert boxes all over the page (missed the screenshot)
When I submitted a complaint for text exercises and looked into it as a tutor, I couldn't change the assessment anymore. I am not sure how the process looks there, so might be valid - but I don't see how the tutors can react to complaints. If the review time is after the assessment due dates and the assessments can only be changed before the assessment due date -> tutors cannot react to complaints in terms of changing the assessment

I made it so that the buttons are not shown at all, not just deactivated if its not within the review period.
The assessment due date should for no I think include the whole student review period, so assessment due date = end of student review.

I was able to follow the steps successfully.
Confirmed:

complaints can be assessed by other tutors (other than the original tutor who assessed the exercise)
feedback requests can be assessed by the same original tutor
students see the feedback (however students do not see the score - not sure how this is meant to be handled)

The scores are only shown if you have selected a publish results date for the exam.

After a complaint or feedback response from the tutor, the student is not able to read the actual feedback. Just the updated score but not the text why the grading was as it was.
The screenshot should show a complaint which was updated to 4/4 by the tutor with an actual feedback response

I cannot reproduce this issue. This is on the TS1: ExamTestCourse, exam 1865


How to test tressfx using custom assets?

Is there a build of unreal or unity running tressfx 4.0 somewhere that is publically available for me to test out tressfx using my own art assets and groom?

Nothing official, this is my initial unreal engine integration.
https://github.com/lion03/UE4-K4W-Plugin/tree/TressFX-4.16

awesome I'll try this out now! Is it using tressfx 4.0?

It is still in pre pre pre alpha state and barely functional and is still under heavy development.
No lighting/shadowing yet.

Awesome, ill test this out now! Thanks ! This will help a lot! Better than nothing :D

So I get the following errors when I try to compile a build using VS 2017


How to get the 20 test problems

16 + 4
(we have 16 units)

First 16 problems is randomly generated from each unit. (each unit got 1 problem)
The last 4 problem is randomly generate from each of this 4 units(Unit 5 6 15 16,which turns out to be the most difficult units).

Well done!
close for now!

Done


Dark Mode - Ugly White blocks appearing around playing song.

 It looks like it might be that something is highlighted/selected but if I click in an open space, it doesn't disappear.I'm in Dark mode.

This happens on every song, not just on Michael Buble.
Steps to Reproduce: Open Google Play Music app. Be in dark mode. Click on a song to play.

This was fixed in the latest release.  If you are on a proxy or VPN the update won't come through automatically so you might have to download it manually.


Unhandled exception thrown by stream after a delay

Hi!
I am experiencing an issue where this code succeeds but throws also another unhandled FormatException which keeps execution running:
void main() async {
  Stream<int> operation() async* {
    yield 1;
    throw const FormatException();
  }

  final future = operation().last;
  try {
    // removing this will cause the unhandled exception to not occur
    await Future.delayed(Duration.zero);
    await future;

    print("fail");
  } on FormatException {
    print("ok");
  }
}

By commenting the Future.delayed (and avoiding event loop rescheduling), the exception is handled correctly.
It also happens by implementing the same as a Future and with StreamController, Stream.multi, etc..
Issue is also reproducible on DartPad
Dart SDK version: 3.0.5 (stable) (Mon Jun 12 18:31:49 2023 +0000) on "macos_arm64"
Flutter (Channel stable, 3.10.5, on macOS 13.4.1 22F770820d darwin-arm64, locale en-IT)
• Flutter version 3.10.5 on channel stable
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision 796c8ef792 (4 weeks ago), 2023-06-13 15:51:02 -0700
• Engine revision 45f6e00911
• Dart version 3.0.5
• DevTools version 2.23.

This is working as intended.
The call to last starts listening on the stream, for any events until the stream closes. And when the stream computation throws, the future returned by last completes with that error.
As for any other future, there must be a listener on the future when it completes with an error, otherwise the error will be considered, and reported as, unhandeled.
You don't add a listener to the future here until you await it, and awaiting the delayed future first makes that be to late.
What you can do is to call operation.ignore() first. That tells the future that you don't care if it completes with an error. You can still await it later, and get the error there, it will just not be reported as unhandeled.

Thanks @lrhn for the clarification.


Offer setting to customise the menu item

Codecov Report

Merging #23 (b8c8387) into master (839ef65) will decrease coverage by 6.39%.
The diff coverage is 46.66%.

@@            Coverage Diff             @@
##           master      #23      +/-   ##
==========================================
- Coverage   93.00%   86.60%   -6.40%
==========================================
  Files           3        3
  Lines         100      112      +12
==========================================
+ Hits           93       97       +4
- Misses          7       15       +8

Impacted Files
Coverage Δ

wagtail_guide/wagtail_hooks.py
70.96% <46.66%> (-23.77%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Support running different allocation analyses

Needed to have our own allocation analysis for target-specific layout conversion lowerings.


Unable to open the app

Not working in Mac

Can you please follow all the following steps before running app-
Set up the environment
Warning You need a Mac with macOS to write and run iOS-specific code on simulated or real devices. This is an Apple requirement.
To work with this template, you need the following:
A machine running a recent version of macOS
Xcode
Android Studio
The Kotlin Multiplatform Mobile plugin
The CocoaPods dependency manager
Check your environment
Before you start, use the KDoctor tool to ensure that your development environment is configured correctly:
Install KDoctor with Homebrew:
brew install kdoctor
Run KDoctor in your terminal:
kdoctor
If everything is set up correctly, you'll see valid output:
Environment diagnose (to see all details, use -v option):
[✓] Operation System
[✓] Java
[✓] Android Studio
[✓] Xcode
[✓] Cocoapods
Conclusion:
✓ Your system is ready for Kotlin Multiplatform Mobile development!
Otherwise, KDoctor will highlight which parts of your setup still need to be configured and will suggest a way to fix them.


Update substrate to polkadot v0.9.37

Hey guys, why don't he have release branches anymore?

Hey guys, why don't we have release branches anymore?

It's up to @sorpaas or some members of paritytech, others should have no access.

Hi @sorpaas , thanks very much for your hard work. Looks like this PR is going to be merged soon. Can you also have a branch polkadot-v0.9.37 soon? Thanks.


problem with kubeadm-config.yml - apiVersion: kubeadm.k8s.io/v1beta2

When running the ansible playbook for component pack I am getting an error related to used API version :

TASK [setup-master-node : Initialize master for single node installation] ************************************************************************
fatal: [solcon15.solvito.cloud]: FAILED! => {"changed": true, "cmd": ["kubeadm", "init", "--config=/tmp/k8s_ansible/kubeadm-config.yaml"], "delta": "0:00:00.866046", "end": "2023-02-08 15:55:56.572283", "msg": "non-zero return code", "rc": 1, "start": "2023-02-08 15:55:55.706237", "stderr": "W0208 15:55:56.060972    1960 common.go:83] your configuration file uses a deprecated API spec: "kubeadm.k8s.io/v1beta2". Please use 'kubeadm config migrate --old-config old.yaml --new-config new.yaml', which will write the new, similar spec using a newer API version.\nerror execution phase preflight: [preflight]

@MagnaFrisia90, the issue sounds very similar to #221 .  Could you confirm the playbook you're running is the one described here?

I am not sure what you mean by "hear". It jumps directly to "Setting up Component Pack for HCL Connections 8 with its dependencies". But this is not what I am planning.
I am running this command: "ansible-playbook -i environments/examples/cnx7/quick_start/inventory.ini playbooks/setup-component-pack-only.yml" (only want to install Kubernetes with your script)
Inside "setup-component-pack-only.yml" I run

name: Setup Haproxy
import_playbook: third_party/setup-haproxy.yml

name: Setup NFS
import_playbook: third_party/setup-nfs.yml

name: Setup containerd
import_playbook: third_party/setup-containerd.yml

name: Setup Kubernetes
import_playbook: third_party/kubernetes/setup-kubernetes.yml

Same issue after manual removal of containerd & running cleanup-k8s

Which version of k8s are you installing ? I had some issues when I tried to install component pack 7 on k8s Version 1.22 and later.... I prefer 1.21.* for CP7

I can not answer this question - I am only running
"ansible-playbook -i environments/examples/cnx7/quick_start/inventory.ini playbooks/setup-component-pack-infra-only.yml"
I assume it picks the correct version automatically

When I set group_vars\all.yml "kubernetes_version: 1.21.1" the result is pretty much the same:

TASK [setup-master-node : Initialize master for single node installation] ******
fatal: [solcon15.solvito.cloud]: FAILED! => {"changed": true, "cmd": ["kubeadm", "init", "--config=/tmp/k8s_ansible/kubeadm-config.yaml"], "delta": "0:00:00.211092", "end": "2023-02-10 11:48:02.815889", "msg": "non-zero return code", "rc": 1, "start": "2023-02-10 11:48:02.604797", "stderr": "error execution phase preflight: [preflight] Some fatal errors occurred:\n\t[ERROR CRI]: container runtime is not running: output: time="2023-02-10T11:48:02+01:00" level=fatal msg="validate service connection: CRI v1 runtime API is not implemented for endpoint \"unix:///run/containerd/containerd.sock\": rpc error: code = Unimplemented desc = unknown service runtime.v1.RuntimeService"\n, error: exit status 1\n[preflight] If you know what you are doing, you can make a check non-fatal with --ignore-preflight-errors=...\nTo see the stack trace of this error execute with --v=5 or higher", "stderr_lines": ["error execution phase preflight: [preflight] Some fatal errors occurred:", "\t[ERROR CRI]: container runtime is not running: output: time="2023-02-10T11:48:02+01:00" level=fatal msg="validate service connection: CRI v1 runtime API is not implemented for endpoint \"unix:///run/containerd/containerd.sock\": rpc error: code = Unimplemented desc = unknown service runtime.v1.RuntimeService"", ", error: exit status 1", "[preflight] If you know what you are doing, you can make a check non-fatal with --ignore-preflight-errors=...", "To see the stack trace of this error execute with --v=5 or higher"], "stdout": "[init] Using Kubernetes version: v1.21.1\n[preflight] Running pre-flight checks", "stdout_lines": ["[init] Using Kubernetes version: v1.21.1", "[preflight] Running pre-flight checks"]}

Not sure what was the cause, but I have done following steps & it seems like the issue is solved:

yum remove containerd
pkill ssh-agent
rm -Rf /tmp/*
rm -Rf /etc/containerd
ansible-playbook -i environments/examples/cnx7/quick_start/inventory.ini playbooks/hcl/cleanup/cleanup-k8s.yml

I now ran into the next problem - which may be discussed in another topic:

TASK [setup-kubectl : Copy .kube to controller] *******************                                                                    *******************************************************************                                                                    *
fatal: [solcon15.solvito.cloud]: FAILED! => {"changed": false, "cmd                                                                    ": "/usr/bin/rsync --delay-updates -F --compress --archive --rsh=/u                                                                    sr/bin/ssh -S none -o StrictHostKeyChecking=no -o UserKnownHostsFil                                                                    e=/dev/null --rsync-path=sudo rsync --out-format=<>%i %n%L                                                                     solcon15.solvito.cloud:/home/ansible/.kube /tmp/.kube_solcon15.sol                                                                    vito.cloud", "msg": "Warning: Permanently added 'solcon15.solvito.c                                                                    loud,<IP_ADDRESS>' (ECDSA) to the list of known hosts.\r\nprotocol                                                                     version mismatch -- is your shell clean?\n(see the rsync man page                                                                     for an explanation)\nrsync error: protocol incompatibility (code 2)                                                                     at compat.c(178) [Receiver=3.1.2]\n", "rc": 2}

I still would like to know when to set which kubernetes version inside all.yml ?

maybe you need to set { kubernetes_version: "1.21.4" } in your all.yml

Not sure what was the cause, but I have done following steps & it seems like the issue is solved:
recreated git repo
yum remove containerd
pkill ssh-agent
rm -Rf /tmp/*
rm -Rf /etc/containerd
ansible-playbook -i environments/examples/cnx7/quick_start/inventory.ini playbooks/hcl/cleanup/cleanup-k8s.yml

Good the hear the apiVersion issue is fixed.
The Kubernetes version, if not overridden in all.yml, is set in this file:
https://github.com/HCL-TECH-SOFTWARE/connections-automation/blob/84883bf2986b05643857d6255cc2cea5fc92d97a/roles/third_party/kubernetes/kubernetes-install/vars/main.yml#L2
I'd recommend to use the default since that's what was tested.


WIP: Support generation of paged response classes

This is a WIP that can hopefully be simplified further. It adds support for generated page response types, with methods to iterate elements as resource names.

See also https://github.com/googleapis/gax-java/pull/117

I'll let @shinfan do the first pass

The overall approach looks okay to me. Just a few comments.

Current coverage is 82.14% (diff: 97.50%)

Merging #578 into master will increase coverage by 3.26%

@@             master       #578   diff @@
==========================================
  Files           254        257     +3
  Lines          9569       9617    +48
  Methods           0          0
  Messages          0          0
  Branches       1265       1267     +2
==========================================
+ Hits           7548       7900   +352
+ Misses         1695       1386   -309
- Partials        326        331     +5

Powered by Codecov. Last update 28a0dfa...2cf104f

@michaelbausor Have you experimented with putting all the pagedList classes under a top class?

Not yet - I will try this now.

@shinfan Have made that change. The only downside is that now the class names are more verbose, e.g. PagedResponseWrapper.ListTopicSubscriptionsPagedResponse

I am not sure if it is just me...but whenever a PR is large (more than 1000 LOC) I find the review is much harder to follow... So if possible, I would like to encourage to make progressive smaller PRs in the future.

I agree about size - I will try to split changes like this up in future.

PTAL - Passing tests when regenerating into api-client-staging

Looks like the baseline tests are failing. Otherwise looks good.

Fixed baselines.

Addressed comments and added a test for the pagedlistresponse iterateAllAs method

Addressed PR comments, PTAL.

LGTM; Since @shinfan has been giving comments too, he should probably also give an LGTM.

Resolved merge conflicts, PTAL

LGTM


WM model

This PR integrates a WorldModelers model into EMMAA framework. Main differences from biology models: Elsevier is used as a literature source, Eidos reader is used to process text, different assembly steps are used, choice of test corpus is parameterized. All these parameters are configured in the config file. Merging this PR also requires uploading new files to S3, updating lambda function (by running emmaa.aws_lambda_functions.update_lambda.py), merging PR #925 in INDRA, rebuilding INDRA and EMMAA dockers, and restarting EMMAA webservice.

Great!


Increase timeout for seqr aggregation cloud run to 15 minutes.

I have made cloud run function timeout customisable, need larger for seqr aggregate.

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

Please upload report for BASE (main@d74783c). Learn more about missing BASE report.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #260   +/-   ##
=======================================
  Coverage        ?   90.93%
=======================================
  Files           ?        4
  Lines           ?      430
  Branches        ?        0
=======================================
  Hits            ?      391
  Misses          ?       39
  Partials        ?        0

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


[2.5] Update provider detector to include rke.windows

Backport of https://github.com/rancher/rancher/pull/29691

https://github.com/rancher/rancher/issues/28947


correct maven dependencies configuration

What is the purpose of the change
add type=pom for zookeeper dependencies.
Brief changelog
XXXXX
Verifying this change
XXXXX
Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Make sure there is a GITHUB_issue field for the change (usually before you start working on it). Trivial changes like typos do not require a GITHUB issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[ ] Format the pull request title like [Dubbo-XXX] Fix UnknownException when host config not exist #XXX. Each commit in the pull request should have a meaningful subject line and body.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[ ] Run mvn clean install -DskipTests=false & mvn clean test-compile failsafe:integration-test to make sure unit-test and integration-test pass.
[ ] If this contribution is large, please follow the Software Donation Guide.

Codecov Report

Merging #3859 into master will decrease coverage by 0.01%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #3859      +/-   ##
============================================
- Coverage     63.59%   63.58%   -0.02%
  Complexity       71       71
============================================
  Files           708      708
  Lines         31196    31196
  Branches       5048     5048
============================================
- Hits          19840    19836       -4
- Misses         9078     9079       +1
- Partials       2278     2281       +3

Impacted Files
Coverage Δ
Complexity Δ

.../apache/dubbo/qos/protocol/QosProtocolWrapper.java
64.1% <0%> (-17.95%)
0% <0%> (ø)

...ache/dubbo/remoting/p2p/support/AbstractGroup.java
45.45% <0%> (-11.37%)
0% <0%> (ø)

...pache/dubbo/remoting/transport/AbstractServer.java
46.87% <0%> (-1.05%)
0% <0%> (ø)

.../apache/dubbo/remoting/transport/AbstractPeer.java
67.39% <0%> (+2.17%)
0% <0%> (ø)
:arrow_down:

...che/dubbo/remoting/transport/mina/MinaChannel.java
53.94% <0%> (+10.52%)
0% <0%> (ø)
:arrow_down:

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 af8ad8f...f121928. Read the comment docs.