Fix tests

PR checklist
Tests were failing at mosdepth, and in later stage of annotation. This PR addresses that.

[ ] This comment contains a description of changes (with reason).
[ ] If you've fixed a bug or added code that should be tested, add tests!
[ ] If you've added a new tool - have you followed the pipeline conventions in the contribution docs
[ ] If necessary, also make a PR on the nf-core/raredisease branch on the nf-core/test-datasets repository.
[ ] Make sure your code lints (nf-core lint).
[ ] Ensure the test suite passes (nextflow run . -profile test,docker --outdir <OUTDIR>).
[ ] Ensure the test suite passes (nextflow run . -profile test_one_sample,docker --outdir <OUTDIR>).
[ ] Check for unexpected warnings in debug mode (nextflow run . -profile debug,test,docker --outdir <OUTDIR>).
[ ] Usage Documentation in docs/usage.md is updated.
[ ] Output Documentation in docs/output.md is updated.
[ ] CHANGELOG.md is updated.
[ ] README.md is updated (including new tool citations and authors/contributors).

No, I haven't had the time to implement those changes. But feel free to take a stab at that PR 😄


Indicate to the player their status to matchmaking server and other players

There should be a notice of some kind when the player connects or disconnects to matchmaking server, and also when other players are connected/disconnected

Implemented with #13


[Improvement] Make ShellCheck Happy

PiVPN is currently failing on Linting tests,
https://travis-ci.com/github/pivpn/pivpn/jobs/475625045
PiVPN scripts should pass on shellcheck and this should be later used to evaluate PR quality.
Since it is not Breaking anything in the script, I have added allow_failures for linting stage until this has been closed.

We should also have a coding style guide. As of now coding style is not consistent.

We should also have a coding style guide. As of now coding style is not consistent.

agree! but i am not sure if am the right one to implement that .... having shellcheck checking for good practices is already a good start.

agree! but i am not sure if am the right one to implement that .... having shellcheck checking for good practices is already a good start.

We could just link to this one: https://google.github.io/styleguide/shellguide.html

We could just link to this one: https://google.github.io/styleguide/shellguide.html

I've had the most success with shfmt when it comes to shell formatters and can highly recommend it.

I'm new to pull requests, do you prefer one mid-sized pull request with a list of of changes och one pull request per change?

@DundarGoc Personally i prefer smaller fixing one issue at a time as they tend to be easier to read and review, also makes easier to track any issue in case it comes up :) this said i don't mind having a thouzand of commits for the same issue :)

I can't find setupVars.conf in the repo, is it created at runtime or something?

@DundarGoc Yes, basically during runtime script echoes the variables and its values to that file which is in turn used later by the pivpn scripts as a way to identify which options were used during setup and by the --debug option to info on whichsettings were used during the setup
That file can also be used for --unattended setups ... 1 example here: https://github.com/pivpn/pivpn/blob/master/ciscripts/ci_wireguard.conf

Chellsheck is now happy on test branch!
CI Job will no longer tolerate failures on Lint/shellcheck stage.

Merged to master today.
https://github.com/pivpn/pivpn/commit/d4e3a63522e4edaa2f9ad9f2d8dd6ffab4117ff9
Closing ...


std/math.d: Explicitly handle zero and inf in scalbn.

Fixes the CTFE test for gdc when x is real.infinity.  It fails as an inf result from ldexp is treated as an overflow in gcc's const fold internals.

Fixes the CTFE test for gdc when x is real.infinity. It fails as an inf result from ldexp is treated as an overflow in gcc's const fold internals.

Should we version it with GDC or __ctfe?

@wilzbach can do __ctfe, as isInfinity is a built-in anyway...


[Suggestion] Add exception to EMB install

Files that are installed in "data" by the installer  (add or replace content) require to be serialised to XML, this causes an issue with EMB files (especially CHARA01.emb) as YAXS can take 5-8 minutes during install to convert all the string binary data to actual binary.
was wondering about the possibly of an exception for EMB files, so the modder can just place a binary EMB file as the source EMB and the installer will attempt to replace data in destination EMB by matching the file names, or add a file if the name doesn't exist in destination.

The only problem I see with that is that binary files in that area are supposed to be directly copied over, so something about that will need to be changed to make this work.
Maybe just checking if InstallPath is ui/texture/CHARA01.emb would be enough? Or possibly adding an extra property to the File object for it, that way it can be done with other EMB files too.

So while implementing this I took the opportunity to rework a few things. Now it is possible to declare what a file entry is supposed to be for, instead of just guessing based on the path.
With this change simply adding Type=”Binary” will allow for most files to be loaded from binary and installed that way. This is a better solution I think.


Add elm-hedley example

Hedley is a demo app that is covering a few common use cases for elm in web apss (login, content creation, JS interop with Leaflet maps, Files drag & drop, routing, etc').
I saw that there are also gif images in the repo, however they are not used - is it still needed?

Very cool, seems like a great addition :) I may drop one of the others to fit it on the front page though.

Alright, new version should be live, thanks!

:smile:


build arm64 distro images

What this PR does / why we need it:

Build both amd64 and arm64 distro images
Which issue(s) this PR fixes:

SC-113140
Does this PR require a test?

NONE
Does this PR require a release note?

NONE

Does this PR require documentation?

NONE

Merging as this does not affect tests.


Fix infinite resolution

Prior to this change, in some corner cases, dependency graph building
would never terminate, with selection of some modules alternating
repeatedly.
With these changes, we now ignore selection changes after a high
threshold. This prevents unstable graph to loop in resolution for ever.

@bot-gradle test RFM

OK, I've already triggered ReadyForMerge build for you.

@bot-gradle test RFM

Sorry some internal error occurs, please contact the administrator @blindpirate


Fix walineJS related questions

fixed walineJS address (it's missing an @)
added requiredMeta for config.toml to avoid spam

e.g.
# {{< version 0.2.0 >}} {{< link "https://waline.js.org/en/" "Waline" >}} comment config
      [params.page.comment.waline]
        enable = true
        serverURL = "https://xxx.vercel.app/"
        visitor = true
        dark = 'auto'
        requiredFields = ["nick", "mail"] # Use " to avoid errors

@all-contributors please add @aozaki-kuro for bug, code


The download is too slow. Is there any way to improve the download speed

I spent several hours downloading model， Is there any way to improve the download speed， or download it locally and then load it

Yes, you can. You can clone any mlc-chat repository from HuggingFace (that has been built for WebGPU) and load them from localhost using a config like this:
import * as webLLM from "@mlc-ai/web-llm";

const modelId = "Some-Model-Name-q4f32_1";

const appConfig = {
  model_list: [
    {
      model_url:
        "https://localhost:8080/params/",
      local_id: modelId,
    },
  ],
  model_lib_map: {
    [modelId]:
      "https://localhost:8080/Some-Model-Name-q4f32_1-webgpu.wasm",
  },
};

const chatConfig = {
  temperature: 0,
  repetition_penalty: 1.2,
  top_p: 1
};

const chat = new webLLM.ChatModule();

await chat.reload(modelId, chatConfig, appConfig);

It works, thanks. I download this modal  mlc-ai/mlc-chat-vicuna-v1-7b-q4f32_0。but  I can't find the wasm file,  i have to use an online address
` const modelId = "vicuna-v1-7b-q4f32_0";
const appConfig = {
model_list: [
{
model_url:
"http://localhost:3000",
local_id: modelId,
},
],
model_lib_map: {
[modelId]:
"https://raw.githubusercontent.com/mlc-ai/binary-mlc-llm-libs/main/vicuna-v1-7b-q4f32_0-webgpu-v1.wasm",
},
};
const chatConfig = {
temperature: 0.7,
repetition_penalty: 1.2
};`

It works, thanks. I download this modal mlc-ai/mlc-chat-vicuna-v1-7b-q4f32_0。but I can't find the wasm file, i have to use an online address。`const modelId = "vicuna-v1-7b-q4f32_0";
const appConfig = {
model_list: [
{
model_url:
"http://localhost:3000",
local_id: modelId,
},
],
model_lib_map: {
[modelId]:
"https://raw.githubusercontent.com/mlc-ai/binary-mlc-llm-libs/main/vicuna-v1-7b-q4f32_0-webgpu-v1.wasm",
},
};
const chatConfig = {
temperature: 0.7,
repetition_penalty: 1.2
};`

Thank you very much, I download this modal mlc-ai/mlc-chat-vicuna-v1-7b-q4f32_0 but I can't find the wasm file. I can only use an online address or download it locally separately. Can you add wasm files to the warehouse
const modelId = "vicuna-v1-7b-q4f32_0";

const appConfig = {
    model_list: [
      {
        model_url:
          "http://localhost:3000",
        local_id: modelId,
      },
    ],
    model_lib_map: {
      [modelId]:
        "https://raw.githubusercontent.com/mlc-ai/binary-mlc-llm-libs/main/vicuna-v1-7b-q4f32_0-webgpu-v1.wasm",
    },
  };

const chatConfig = {
    temperature: 0.7,
    repetition_penalty: 1.2
  };

Great to see you got it working, @LoveSlience!
That's good that you pointed that out. I've updated my comment above to clarify that for future readers.
About mlc-ai/mlc-chat-vicuna-v1-7b-q4f32_0, unfortunately, I can't change that repo, as I'm not part of the mlc-ai team. But you can still download that .wasm and put in the same folder that you're serving the params (http://localhost:3000), so you could load it as
    model_lib_map: {
      [modelId]:
        "http://localhost:3000/vicuna-v1-7b-q4f32_0-webgpu-v1.wasm",
    },


scout:import only one index

I do have 3 different indexes and I would like to import a new one, is there some way I ca do this with only the new one without to re-import all indexes?
Thanks.

The same as in original package https://laravel.com/docs/master/scout#batch-import
Just add class you want to import


Upgraded packages, and prettier config.

Closes #45

Codecov Report

Merging #46 (bc13707) into main (1e9edc3) will not change coverage.
 Consider uploading reports for the commit d88dc65 to get more accurate results

@@            Coverage Diff            @@
##              main       #46   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            1         1
  Lines            1         1
=========================================
  Hits             1         1

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Allow to set custom status code and error details for extract::error::Error

Missing value don't always assume "404 Bad Request" status code. For instance in case of authorization it would be more appropriate to return "403 Forbidden" if access token is required by the application. At the same time, some applications may want to process requests from anonymous users along with authorized ones.
impl_web! {
    impl Resource {
        #[get("/foo")]
        fn foo(&self, t: Token) -> Result<String, ()> {
            Ok(String::from("user"))
        }

        fn bar(&self, t: Option<Token>) -> Result<String, ()> {
            match t {
                Some(_) => Ok(String::from("user")),
                None => Ok(String::from("anonymous")),
            }
        }
    }
}

Currently, that is impossible to specify status code or any additional details of the extract::error::Error
struct Token;

impl<B: BufStream> Extract<B> for Token {
    type Future = Immediate<Token>;

    fn extract(context: &Context) -> Self::Future {
        match context.request().headers().get(http::header::AUTHORIZATION) {
            Some(_) => Immediate::ok(Token{}),
            None => Immediate::err(Error::missing_argument()),
        }
    }
}

The proposal is to change the extract::error::Error to make such error customization possible:
impl<B: BufStream> Extract<B> for Token {
    type Future = Immediate<Token>;

    fn extract(context: &Context) -> Self::Future {
        match context.request().headers().get(http::header::AUTHORIZATION) {
            Some(_) => Immediate::ok(Token{}),
            None => Immediate::err(Error::missing(forbidden())),
        }
    }
}

fn forbidden() -> tower_web::Error {
    tower_web::Error::from(StatusCode::FORBIDDEN)
}

Fixed in #217.


Increase apiserver mem-threshold in density test

Ref: https://github.com/kubernetes/kubernetes/issues/60500#issuecomment-372682659
/sig scalability
/kind bug
/cc @wojtek-t
/cc @crassirostris (for the release-note)
Buffering introduced to audit-logging can lead to increased apiserver memory usage. We saw upto 200MB increase in our 100-node cluster performance tests. Ref: issue#60500

Oops.. Seems like I created it against wrong base branch (tried doing from github :P).
Let me fix it.


Fix race condition and logic bugs in HolographicSpatialMapping sample

This change fixes a race condition between UpdateSurface and CreateVertexResources in the HolographicSpatialMapping sample by explicitly plumbing the pending SpatialSurfaceMesh object to the update thread instead of directly referencing m_surfaceMesh. It also fixes a small logic bug which would cause a surface to be dropped permanently if it ever failed to be located on a given frame.

@oldnewthing what's the process for approving changes to SDK samples? I can't figure out how to add reviewers or assignees, nor is it clear who needs to sign off on this change.

@thetuvix FYI

Thanks David. I've transferred this to the internal workflow.


Sample: Using the EventHubBufferedProducerClient with ASP.NET

Summary
A common usage scenario for publishing events is doing so from within an ASP.NET application.  A sample should be available demonstrating how to create and configure the EventHubBufferedProducerClient in an ASP.NET context.
Scope of Work

A stand-alone sample that demonstrates creating and configuring an EventHubBufferedProducerClient in an ASP.NET application, illustrating a reasonable default configuration and integrating with the ASP.NET dependency injection framework.

The sample should include registering event handlers for success and failure and a stubbed approach for dead-lettering failed sends.

The sample should address the need to extend the shutdown to avoid potential data loss and demonstrate doing so.

Out of Scope

Demonstrating the other Event Hubs clients.

Demonstrating use of the Azure Client Builder extensions (they do not support the full scenario needed by the buffered producer)

Success Criteria

A sample exists, demonstrating creation of the EventHubBufferedProducerClient and integration in to the ASP.NET host environment, as detailed in the scope.

The code for the sample is a stand-alone application that can be executed and run.

The sample project can be built within the Azure SDK repository structure, taking advantage of our engineering system.

The sample project can be built on its own, with no dependencies on the Azure SDK engineering system.

The tests necessary for sample validation have been created in the Event Hubs test suite and ensure that the sample is compliable.

Any functional tests pass reliably.

The existing test suite continues to produce deterministic results and pass reliably.

Hi @jsquire, Is this issue is still relevant? Thanks

@andreyshihov :  I think it's a nice-to-have sample.  It's a bit less important now that there's a processor with ASP.NET host example as this would be the same approach with the details changed.    To be honest, I'm still on the fence of whether or to let the bot close this out.


[Form] Input events validate-on code example missing.

Environment
Operating System: Windows_NT
Node Version: v20.18.0
Nuxt Version: -
CLI Version: 3.14.0
Nitro Version: -
Package Manager: unknown
Builder: -
User Config: -
Runtime Modules: -
Build Modules: -
Version
v3.0.0-alpha.6
Reproduction
Cannot use v3 on stackblitz so no repo.
Description
There is no code example for the validate-on prop. Like how do I stop blur,input and change validations?
https://ui3.nuxt.dev/components/form#input-events
Additional context
No response
Logs
No response

@benjamincanac Is it currently possible to do PR requests for v3 docs?

Closing this as there is actually an example on the docs: https://ui3.nuxt.dev/components/form#input-events.


certs: coverage++

@mhils: There was untested code - which actually didn't work. So I tried to implement the missing bits and make it work as intended.
If you feel more comfortable without that, I can remove the whole thing entirely?

Done - please take another look.


Characters escaped when creating action

Hi again !
I've noticed that sometimes (with cache set to "dummy" or "apc") when creating action, then editing it online, the code is escaped when synced...
Actually, every $data['var1'] are escaped as $data['var1'], even in comments.
Is it the normal behaviour, is it because I'm using single quote, or because I don't have any cache set...
Thank you for your help.
Example: http://pastebin.com/aYbNWqjK
Line 20, 39...

Can you please check your php.ini file for Runtime Configuration directive: magic_quotes_gpc, if it is on try turning it off and see if that fixes the issue.

Solved. Thank you.


Fix Ace editor tooltip positioning

Change the fullscreen modal side panel positioning to use position right instead of transform translateX as transform creates a new stacking context and the containing block is changing the context which results in "wrong" positions.
For more details please see:
https://drafts.csswg.org/css-transforms-1/#transform-rendering
With this change in place we no longer need the hacks to correct the Ace editor tooltip position, as position fixed (used to position the tooltip) is working as expected.

Take that hack out!! 💯  ✨

+1000 thanks for doing this!

@jfurrow why do you want to use pixel values instead of percent?

@orlandohohmeier I pushed a commit, hope you don't mind. The right: -100% value for the side panel needs to change at each breakpoint, because the 100% refers to the whole width of the modal, rather than the width of the side panel itself.

I'll let someone else merge since I had a hand in this as well :)

Looks good 👍


Events are not showing in view since upgrade to v6

Checklist:

[x] I updated to the latest version available
[x] I cleared the cache of my browser

Release with the issue:
Any v6
Last working release (if known):
v5
Browser and Operating System:
Tried on Windows/Edge as well as the Android app

Description of problem:

No events show when tapping the hamburger button on the right of the camera view
Frigate card diagnostic information

card_version: 6.0.0-beta.4
browser: >-
  Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like
  Gecko) Chrome/<IP_ADDRESS> Safari/537.36 Edg/<IP_ADDRESS>
date: 2024-09-30T19:12:54.948Z
frigate_versions:
  4030ef078ae309891cae35223c7c2fc3: 5.4.0/0.14.1-f4f3cfa
lang: en
timezone: America/Chicago
git:
  hash: d907ace
  build_date: Sat, 28 Sep 2024 20:41:35 GMT
  commit_date: Fri, 27 Sep 2024 18:56:59 -0700
ha_version: 2024.9.3
config:
  type: custom:frigate-card
  cameras:
    - camera_entity: camera.front_door_cam

Javascript errors shown in the web inspector (if applicable):

Additional information:
https://i.imgur.com/RVhwx8d.png

Could it be a scroll issue? Try scrolling up in the thumbnail drawer?

I'll be damned. That is indeed the issue! If I scroll down, I now see events!


Cannot run ./build-sdk-sh

Reposting the issue because I can't find any solution among the ones given in the other posts:
I cannot build the .jar files needed to get Rocket.Chat.Android to run. Please help or link me someone who can.
I already updated gradle, invalidate cache and build after cleaning the path, try to compile it using intelliJ, clone the directory multiple times and start from scratch again. I really don't know waht is missing.
(base) MacBook-Pro-di-FB:app paola$ ./build-sdk.sh
CURRENT DIR: /Users/paola/Desktop/Workspace/Rocket.Chat.Android/app
SDK DIR: /Users/paola/Desktop/Workspace/Rocket.Chat.Kotlin.SDK
CURRENT SHA: 570f3dd
not dirty
same hash as before 570f3dd = 570f3dd
FAILURE: Build failed with an exception.

What went wrong:
Could not create service of type ScriptPluginFactory using BuildScopeServices.createScriptPluginFactory().

Could not create service of type PluginResolutionStrategyInternal using BuildScopeServices.createPluginResolutionStrategy().

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

Get more help at https://help.gradle.org

BUILD FAILED in 2s
FAILURE: Build failed with an exception.

What went wrong:
Could not create service of type ScriptPluginFactory using BuildScopeServices.createScriptPluginFactory().

Could not create service of type PluginResolutionStrategyInternal using BuildScopeServices.createPluginResolutionStrategy().

Get more help at https://help.gradle.org

BUILD FAILED in 2s
rm: /Users/paola/Desktop/Workspace/Rocket.Chat.Android/app/libs/common*: No such file or directory
rm: /Users/paola/Desktop/Workspace/Rocket.Chat.Android/app/libs/core*: No such file or directory
/Users/paola/Desktop/Workspace/Rocket.Chat.Kotlin.SDK/common/build/libs/common-0.1-SNAPSHOT.jar -> /Users/paola/Desktop/Workspace/Rocket.Chat.Android/app/libs/common-570f3dd.jar
/Users/paola/Desktop/Workspace/Rocket.Chat.Kotlin.SDK/core/build/libs/core-0.1-SNAPSHOT.jar -> /Users/paola/Desktop/Workspace/Rocket.Chat.Android/app/libs/core-570f3dd.jar

We have lots of closed issues regarding the same inability you've posted here and also the build is working as expected from our side. That said, try following those issues by finding a closest one which fits your case, following the instructions or by using our community channel.


Some changes to readme file

In this box you can write more description.. but I dont need that now.

This is a pull request.

You can close & reopen a pull request.

Now, I open it again.


feat(shipper): add Purolator and Intelcom shippers

Co-authored-by: @glbailey
Proposed change

Adds Purolator and Intelcom Canadian Shippers
Type of change

[ ] Dependency upgrade
[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (which adds functionality)
[ ] Breaking change (fix/feature causing existing functionality to break)
[ ] Code quality improvements to existing code or addition of tests
[ ] Documentation update
[x] Adds a new shipper
[ ] Update existing shipper

Additional information

This PR is related to issue: n/a

Codecov Report

Merging #791 (8744ec4) into dev (5ca4fbf) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files

@@           Coverage Diff           @@
##              dev     #791   +/-   ##
=======================================
  Coverage   97.08%   97.08%
=======================================
  Files           8        8
  Lines        1512     1512
=======================================
  Hits         1468     1468
  Misses         44       44

Impacted Files
Coverage Δ

custom_components/mail_and_packages/const.py
100.00% <ø> (ø)


Added weighted_rigid_align

Implemented weighted_rigid_align (Algorithm 28). Also added "geom_utils.py" file which is taken from pytorch3d to generate random rotation matrices for testing weighted_rigid_align.
Correctness was confirmed for uniform weights. Needs to be tested with non-uniform weights.

@joseph-c-kim another! i have already merged in Felipe's PR but welcome you to check it over to see if you see any inconsistencies
the code looks pretty similar!


[Incubating plugin] Add gradle build cache test

This PR is dependent on https://github.com/apollographql/apollo-android/pull/1849. Do not merge until https://github.com/apollographql/apollo-android/pull/1849 is merged.
It adds this commit: https://github.com/apollographql/apollo-android/commit/ffe330f1b60f4e5919d92995f0d2da9eedef9852
It adds a test to check that build cache (https://docs.gradle.org/current/userguide/build_cache.html) is working correctly with the customIdGenerators.

Using a custom build cache folder is a good idea. I didn't know that this was possible. 👍
I wonder if we can find a generic way to test if build cache is working. We could have one test without even configuring the plugin with custom props. That would be easy.

I wonder if we can find a generic way to test if build cache is working. We could have one test without even configuring the plugin with custom props. That would be easy.

Not sure I get it. What are custom props here? apollo. customIdGenerator  ? This isn't really required but the customIdGenerator triggered the discussion about build cache so I decided to add the test in this context. It can certainly be added for other tests too. Or did you want to add it systematically for all tests ?

Sorry my comment here was just brain-storming and was out of scope of this PR. I was wondering if it would be possible to write (smoke) tests for build cache. They would run over PRs and detect any accidental build cache breakages. It is just an idea.

We could but I don't think build cache would break a lot. While the gradle plugin has some complexity, it should be self contained enough so that it doesn't break too much?

I'm merging this one because it's a small one and I want to open a PR to promote the incubating plugin and this would conflict. Let me know you want me to change anything.

All good. I meant to approve it actually. 👍


Subclass query building

Using Laravel 5.3
I'm not sure if I'm expecting something that should be there or not.
class Base extends Model { }

class Sub extends Base { }

class SubAlt extends Base { }

I would expect doing something like this:
Sub::all()

Expectation:
\Collection {
    all: [
        \Sub { }
    ]
}

Getting:
\Collection {
    all: [
        \Sub { }
        \SubAlt { }
    ]
}

Not sure if this is part of the Laravel 5.3 ticket #23 or not.

I made the following, which seems to work:
class Base extends Model
    public static function allQuery($columns = ['*'])
    {
        if (!in_array(static::class, static::$singleTableSubclasses)) {
            $query = null;
            foreach(static::$singleTableSubclasses as $subClass) {
                if (is_null($query)) {
                    $query = static::where('type', $subClass::$singleTableType);

                } else {
                    $query->orWhere('type', $subClass::$singleTableType);

                }
            }
            return $query;
        }
        return static::where('type', static::$singleTableType);
    }

    public static function all($columns = ['*'])
    {
        return static::allQuery($columns)->get();
    }
}

This allows you to do something like:
Base::all()

\Collection {
    all: [
        \Sub { }
        \SubAlt { }
    ]
}

Sub::all()
\Collection {
    all: [
        \Sub { }
    ]
}

SubAlt()
\Collection {
    all: [
        \SubAlt { }
    ]
}

Believe this could be added to the trait itself and accomplish the same goal, but I'm not sure it's the most elegant solution.

Second iteration:

Uses DB.
Allows columns specification.
Cleaner implementation overall.

    public static function query($columns = ['*'])
    {
        $obj = new static();
        $query = DB::table($obj->table)->select($columns);
        if (in_array(static::class, static::$singleTableSubclasses)) {
            $query = $query->where(static::$singleTableTypeField, static::$singleTableType);
        }
        return $query;
    }

    public static function all($columns = ['*'])
    {
        return static::query($columns)->get();
    }

Going to be adding this to my own trait internal to my project. I have a mental note to submit a PR, but cannot promise timing and want to make sure I'm covering as many use cases as possible using my own project as the guinea pig first.

I can't reproduce the error you describe and have added test that proves it works today dd0668b40b609570d21a8a9991ef6cf1b20bdbb8.


No VB version of 5 minute app?

There must be a VB version of this app mustn't there?

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

ID: faf18d20-d28f-1efa-d329-181fff3630fa
Version Independent ID: 46a7c2e0-4871-491f-02c2-54f7df2237b1
Content: Get started with ASP.NET Core MVC and Visual Studio
Content Source: aspnetcore/tutorials/first-mvc-app/start-mvc.md
Product: aspnet-core
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

@andrewba10  sorry, no.

@andrewba10, if you look the comparaison between asp.net and asp.net core, you can see that VB is not available in .net Core, only C# and F#.
https://docs.microsoft.com/en-us/aspnet/core/fundamentals/choose-aspnet-framework?view=aspnetcore-2.2
HTH

I seems VB is going to be left behind for the new stuff, unfortunately. I am trying to learn c# but its like learning to play guitar all over again switching hands. All the fun is out, struggle with semi columns begin...


Schema v4 Update Guidance: Update Modal

As a PD user opening the latest version of PD, I would like to be informed that PD now supports modules, and that using them requires an App and Robot update.
Acceptance Criteria

[ ] Users opening the latest version of PD are informed in an Update Modal that they can now include modules in their protocols.
[ ] Users are informed that using modules requires their robot on app and robot version > [TBD]
[ ] Users are informed that the app and robot updates are only required if using modules

Notes

users do not have to be made aware of the PD or Protocol Schema version bump
we decided not to include an 'do not show me again' checkbox; this modal will only display once

Design
https://app.zeplin.io/project/5c87a3f55f998e3943ad5d38/screen/5e3a06eda3e25f984e06a9e9
Copy
See design

@pantslakz
"Note: Adding modules to your protocols may require updating the OT-2 app and robot. Protocols with a module will only run on the latest versions (3.17 and higher)."


add params to support skip the nameserver check

Related issue: https://github.com/rancher/rancher/issues/9562
Add an option to be able to skip the host dns nameserver file check.

Environment name generally should follow the format CATTLE_($what_@vincent, @darren, @craig agree on)

LGTM


Parser produces output multiple times per second

The buffer parser receives an input buffer only every 10 seconds, however it continues to trigger the output multiple times per second (so not just when triggered with input). Is that a bug, or did I miss some setting (though it's hard to imagine what the purpose of providing output w/o input for such a transformation node would be...)?

Seems like this is rather a general NodeRed issue (on HomeAssistant) when doing a lot of debugging. Happens to other nodes as well...


Conflict of stack-trace version with the one from clarify

After I have execute an npm i in a clear folder (without package-lock.json and node_modules) I have start obtaining the error right on the app start
F:\test\node_modules\trace\node_modules\stack-chain\index.js:10
    throw new Error('Conflicting version of stack-chain found');
    ^

Error: Conflicting version of stack-chain found
    at Object.<anonymous> (F:\test\node_modules\trace\node_modules\stack-chain\index.js:10:11)
    at Object.<anonymous> (F:\test\node_modules\trace\trace.js:3:15)

STR:

Create an empty folder
Execute in it npm i clarify trace
Create a file with code

require('clarify')
require('trace')

Execute file

I'm posting this issue in trace repo, while probably it's the clarify that is needed to be updated, because update of trace has lead to this problem
npm: 5.5.1
node: 9.3.0

Thanks, should be fixed now.


cove_model.predict with Tensor input

How can we use the output from tf.nn.embedding_lookup() in Tensorflow as the input to the cove_model.predict method?

when I tried:
emb = tf.nn.embedding_lookup(word_mat, sentences)
emb = cove_model.predict(emb)
I got
File ".../lib/python3.4/site-packages/keras/engine/training.py", line 76, in
data = [np.expand_dims(x, 1) if x is not None and x.ndim == 1 else x for x in data]
AttributeError: 'Tensor' object has no attribute 'ndim'
Any idea?
Thanks!


[2pt]Add unit tests for all implemented methods.

The following options should be available:
List of games
List all authors (e.g. 'Stephen King')
Add a game

https://github.com/GordonOtieno/ruby-capstone/pull/26


MS support confirmed this does not work on Intune.

I opened a ticket to Azure support (REG:118083118906763) and after 2 months of escalations they stated this does not work on Intune:
“Basically, the capability of enabling a rule in Firewall that will enable a port such as TCP 3389 using custom policies is not currently possible using a CSP. There, just, aren’t any CSPs currently available to allow ports in Windows Firewall.”

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

ID: bfea4e59-ec8c-cbdf-a039-23612c2266a5
Version Independent ID: 7058544b-7ef2-a398-366a-cf71520aa9d8
Content: Firewall CSP
Content Source: windows/client-management/mdm/firewall-csp.md
Product: w10
GitHub Login: @MariciaAlforque
Microsoft Alias: maricia

Can anyone from the product group confirm this doesn't work?

At first glance, it appears as if the statement from Azure support makes the Firewall CSP page less useful than what is documented on the page itself.
I don't know who specifically to ask for an authoritative answer on this topic , but I will chance a ping to @egallagh to ask if he knows something about this.

@officedocsbot assign @e0i

@egallagh Would you be able to clarify regarding this topic if the time permits? Thank you.

This question can be best answered via Windows 10 Support channels.
Please consider opening a product support ticket by the following link below for your problem so that it gets resolved quickly.
The issues section of this repository is intended for product documentation issues only.
Thank you.

Please read the case.
I already got a response from Microsoft support. And it states that your documentation is Wrong.

@ClaudioRifo An independent contributor is going to follow-up with a PR based on your suggestion. Thank you for the clarification.

@ClaudioRifo - Thank you for sharing the details. Can you please attach the support case response so that I can make adjustments in the document.
Thanks
Imran.

For the second time, please read the case. On the first post is the support ticket number and also quoted the support response.
Also. With the release of the new option to create firewall rules on Endpoint Security, you should just delete this references and point to that functionality.
From: ImranHabib<EMAIL_ADDRESS>Sent: Wednesday, June 17, 2020 3:39 AM
To: MicrosoftDocs/windows-itpro-docs<EMAIL_ADDRESS>Cc: Claudio P. Rifo Wahl<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/windows-itpro-docs] MS support confirmed this does not work on Intune. (#2295)
@ClaudioRifohttps://github.com/ClaudioRifo - Thank you for sharing the details. Can you please attach the support case response so that I can make adjustments in the document.
Thanks
Imran.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/windows-itpro-docs/issues/2295#issuecomment-645208093, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AFUQY42MTQJSDJIHUVTY2BDRXBXIFANCNFSM4GI67DVQ.

@ClaudioRifo The proposed changes based on your feedback has been rejected by the document author via the following rationale,

Per the product team, the statement in the comment is not true. They could test a syncml to add a rule that  would allow TCP port 3389 (both local and remote port worked).

Additionally, Intune has support to be able to configure firewall rules via an endpoint protection device configuration profile as well as through a firewall endpoint security profile.

I am closing this PR as the note added here is not applicable/required. Thanks!

(via https://github.com/MicrosoftDocs/windows-itpro-docs/pull/6993#issuecomment-674971500)
Hope you find this explanation useful.
Thank you.

Hi there.
Just for the record, the statement was correct when the case was opened (2018).
From the moment I opened the case till now this functionality has been correctly implemented on Endpoint protection as stated (and I do believe this a 2020 feature).
From: Onur<EMAIL_ADDRESS>Sent: Monday, August 17, 2020 3:16 PM
To: MicrosoftDocs/windows-itpro-docs<EMAIL_ADDRESS>Cc: Claudio P. Rifo Wahl<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/windows-itpro-docs] MS support confirmed this does not work on Intune. (#2295)
@ClaudioRifohttps://github.com/ClaudioRifo The proposed changes based on your feedback has been rejected by the document author via the following rationale,
Per the product team, the statement in the comment is not true. They could test a syncml to add a rule that would allow TCP port 3389 (both local and remote port worked).
Additionally, Intune has support to be able to configure firewall rules via an endpoint protection device configuration profile as well as through a firewall endpoint security profile.
I am closing this PR as the note added here is not applicable/required. Thanks!
(via #6993 (comment)https://github.com/MicrosoftDocs/windows-itpro-docs/pull/6993#issuecomment-674971500)
Hope you find this explanation useful.
Thank you.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/windows-itpro-docs/issues/2295#issuecomment-675063113, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AFUQY46SH5X73KWOZELZJ6TSBF6XXANCNFSM4GI67DVQ.


Verify Github Pages Deploy flow works correctly

Verify the Github Pages flow works correctly after the NextJS UI project scaffold is updated to use the AppRouter with this PR

Verified with this PR.


Feature

Create objstr function, like here: http://css-blocks.com/

Main contribution of css-blocks is "state selectors" and "sub-state selectors".


Create index.md

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

File
Status
Preview URL
Details
Dependent Files

TOC.json
:warning:Warning

Details

E2E_DocsBranch_Dynamic/index.md
:white_check_mark:Succeeded
View

TOC.json

[Warning] Error happen when converting TOC.json to Pdf. Details: Could not find file 'T:\bh5o\TOC.json'.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

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

File
Status
Preview URL
Details
Dependent Files

TOC.json
:warning:Warning

Details

E2E_DocsBranch_Dynamic/index.md
:white_check_mark:Succeeded
View

TOC.json

[Warning] Error happen when converting TOC.json to Pdf. Details: Could not find file 'T:\o1xm\TOC.json'.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


ignore all directories that start with dot

I am at a loss on how to skip  any directory that starts with a dot.  I put .*/ in my .gitignore, I put exclude = */.*/,.*/ for flake8.  I can not seem to find the combination for isort.
Thank you

There is the --skip-glob option (or could be placed in a config file). To skip any directories starting with a dot I would add --skip-glob '*/.*/*'


can't set imagePullSecrets in Kafka pods

Is your feature request related to a problem? Please describe.
Pods support a imagePullSecrets option to set the secret(s) to use so it can pull the image from a private registry.
This feature is not provided by the KafkaCluster definition.
Describe the solution you'd like to see
Add a imagePullSecrets field in the KafkaCluster CRD which should be an array like :
 imagePullSecrets:
  - name: private-docker-registry-key

Describe alternatives you've considered
The only other solution is to create a ServiceAccount and add the imagePullSecrets to it, then use it in the KafkaCluster
like :
- apiVersion: v1
  kind: ServiceAccount
  metadata:
    labels:
      app: kafkacluster
    name: kafkacluster
    namespace: kafka
  imagePullSecrets:
  - name: private-docker-registry-key

Hi @prune998!
Thanks for opening this issue.
Supporting imagePullSecrets in the KafkaCluster CRD is indeed a good idea.
Would you consider submitting a PR?
I think it is a relatively easy fix and I am happy to help with the process.
In the meantime, please use the alternative ServiceAccount solution.
Configuring a service account for KafkaCluster CRD is supported.

Ok looking into it right now. Will PR soon.

Simple PR done... let me know if I missed something.
I thought it wasn't necessary to add a test for this... let me know if I'm wrong.


Let fixture.js provide a helpful error if the fixture can not be found

If you fat-finger the fixture ID it gives you a nasty 'Cannot call create on undefined' (or something similar) which requires additional debugging.

@justinfagnani for a review

Is it feasible to add a test for this?


Proper schema comment parsing

Schema comment starts with <! and ends with >. LmlParser currently expects !> at the end of any comment. DTD schema comment needs to be properly parsed, taking into account the standards. See #4.

Fixed on new-macro-sign branch.


Show network ID in history

etherlime history is a really handy way to look back later and check what you deployed and which address it went to.
It would be useful if it showed the network ID, so you could see what you deployed to which network.

Hey @edmundedgar Awesome idea. Will implement that shortly :). Anything else that you feel might be improved or done better?

Implemented in 1.0.0


[FLOC 3773] Fix broken doc links

Fixes 3773, which replaces 2 doc-dev links with the new master build from Jenkins.

I tried to fix this problem by making the doc-dev URL map to the S3 hosted site, but it proves to be more difficult than I think it should be.
So, given this page is mainly for internal ClusterHQ use, the change of URLs is acceptable.
The new links work fine, so please merge.


Introduce Centurion::Service

Working towards having multiple services running on a single host, I moved the code a bit and created a Centurion::Service:

It will help later, because we can switch to building multiple services in the DSL and all service attributes, like command, volumes, port mappings and other will have a logical place where they can be put, instead of being just global variables.
I like that it also drastically reduces the length of the parameter list for most of the methods in deploy.rb.
As a bonus, all of the fetch calls are now contained within the rake files and deploy DSL and the variables are passed to the lower layers from there, instead of being pulled from other layers at will.
Ports and volumes are stored verbatim, within the Service instance. Previously, they were immediately converted to the docker API format and then had to be parsed back in order to work with them. Now, they are converted to the docker API format only when needed, which further simplifies some parts of the code.

I tried to maintain the same behavior, and moved all the relevant tests from deploy_spec.rb to service_spec.rb. I did not retain the specific exit codes when wrong cpu shares or memory is specified though and changed that to raising an exception instead. I think that it was only used for testing purposes, but I can move it back if it was critical.
What do you think?

@kremso   YES. This has been badly needed for some time. See the comment here where I was bitching about it, for example. I called it Container, you called it Service but the same idea.
I like how this also works toward supporting multiple containers on the host. How would you feel about calling it Container rather than Service, in order to use more Docker-specific lingo?
I'm otherwise ready to merge. Thanks!

Yes, Container is definitely a better name than Service, I'll change it.
While we're at it, I also don't like the build_host_config name because the config it builds doesn't seem to be a host config, but I couldn't figure out a better name.

@relistan In the existing codebase, the name container is already used to name what you get when you start, well, a container. I tried to change Service to Container, but it leads to weird code, like
  def stop_containers(target_server, container, timeout = 30)

or
  def start_new_container(server, container, restart_policy)
    container_config = container.build_config(server.hostname)
    info "Creating new container for #{container_config['Image'][0..7]}"
    new_container = server.create_container(container_config, container.name)

I'm quite new to docker and I'll rely on your judgement, but it doesn't seem right. Maybe Definition would be a better name? Or ContainerDefinition?

Ok, having spent some time digging into this, I think your naming is right. It includes things that are more general than one container. Particularly if there is more than one container per host per service. So Service seems appropriate. There one place where there's a method that returns sometimes a hash and sometimes a class and I think we should fix that. Otherwise, this is great.

@relistan Good catch, merged your PR. Thanks!

Awesome @kremso, merged. Fixing the one thing @jsuchal pointed out with my code.


X applications interact poorly with returning from locked screen

Windows Version
Microsoft Windows [Version 10.0.22621.2715]
WSL Version
<IP_ADDRESS>
Are you using WSL 1 or WSL 2?

[X] WSL 2
[ ] WSL 1

Kernel Version
Linux version <IP_ADDRESS>-microsoft-standard-WSL2 (root@1c602f52c2e4) (gcc (GCC) 11.2.0, GNU ld (GNU Binutils) 2.37) microsoft/WSL#1 SMP Thu Oct 5 21:02:42 UTC 2023
Distro Version
Ubuntu 22.04
Other Software
xclock installed by whatever the shell suggests when you type 'xclock' at it ("sudo apt-get install x-"something, I think)
Also, emacs, also installed with apt-get.
Repro Steps
I am using the (mostly wonderful) built-in X server in Windows+WSL.  After 15 minutes or so my computer locks automatically, and often I use Windows-L to lock the computer.
To a command prompt, Linux shell, type "xclock &", "emacs &".
Lock the screen.
Unlock the screen.
I've seen three behaviors:

When I unlock the screen, it looks like xclock has reported "X connection to :0 broken (explicit kill or server shutdown)." to the controlling terminal, and xclock has exited with status 1.  Emacs is less forgiving --- it aborts and coredumps.
The second behavior is similar: only, now when I start a new xclock to replace the original, there's no window decorations:

The above two behaviors don't always happen.  Often, my X windows have just disappeared.  I can get them back by poking the X server with an X application (e.g., another invocation of xclock).  Then all the X-gui-using programs come up on the screen (at least one of their windows, at least).

Expected Behavior
I'd like my X programs to survive a locked screen.
When they survive the locked screen, I'd like them to refresh and reappear automatically, just as though they'd gotten an expose-event or something similar.
Actual Behavior
See above.  Locking the screen appears to shut down the X server, and sometimes leaves its internal window-manager confused.
Diagnostic Logs
No response

I've seen this repetitively with my intellij environments, but it doesn't always require a lock screen, I've seen it happen when the Windows desktop background changes as well.


refactor: remove payouts from _computeNewAllocationWithGuarantee

note that previously, information was duplicated, since
newAllocation[i].amount + payouts[i] == allocation[i].amount
always held true

superceded by #3747


Kubelet not starting in Custom Rancher cluster -> Amazon provider

What kind of request is this (question/bug/enhancement/feature request):
Bug
Steps to reproduce (least amount of steps as possible):
Rancher 2.x on AWS.
We are now trying to set up a Custom rancher cluster, by selecting Amazon as the cloud provider.
Result:
When we try to execute the command we get from Rancher UI to start Rancher agent  in our AWS – E2 server, we are noticing that kubelet container is failing to startup. The error its displaying is as below
F0408 00:58:45.614181   15011 server.go:261] failed to run Kubelet: could not init cloud provider "aws": error finding instance i-0b8d20a2b09480f42: "error listing AWS instances: "RequestError: send request failed\ncaused by: Post https://ec2.ap-southeast-2.amazonaws.com/: dial tcp <IP_ADDRESS>:443: i/o timeout""
Other details that may be helpful:
If we setup the cluster ignoring the cloud provider type - Amazon, the setup is successful and we can use it. But in that case if we want to setup a Load Balancer we get "LoadBalancer stuck in Pending" error.
Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI):
v2.1.7
Installation option (single install/HA):
single install
AWS with three EC2 nodes. Selecting custom type cluster and cloud provider as Amazon.
We have followed the pre-reqs required for this kind of setup - AWS role and labels.

Cluster information

Cluster type (Hosted/Infrastructure Provider/Custom/Imported):
Machine type (cloud/VM/metal) and specifications (CPU/memory):
Kubernetes version (use kubectl version):

(paste the output here)

Docker version (use docker version):

(paste the output here)

The hosts need to be able to reach EC2 to look up required information, are you in a proxied environment or any other network restrictions outside or on the host (firewall)?

Thanks for the reply. We are in a private cloud, firewalled from Internet. But the three hosts -master and two workers are all in the same subnet.
Is there any way to check the network restriction?

I have tried something different. This might give some more information about this problem.
I selected new cluster and instead of custom selected 'From nodes in an infrastructure provider' option and Amazon EC2 from there.
Then selected the add node template button, added region and aws key pair and clicked next. After a long wait got a 502 null error message.
Any idea?

Is it only firewalled or also proxied? Connectivity check is basically something like
curl -s https://ec2.$(curl -s http://<IP_ADDRESS>/latest/dynamic/instance-identity/document|grep region|awk -F\" '{print $4}').$(curl -s <IP_ADDRESS>/latest/meta-data/services/domain) -v

You need to open connectivity to AWS as it needs to query resources in AWS as being the cloud provider.

Hello,
I am facing the the same problem.
Creating a custom cluster from Rancher UI ->Chose Amazon as cloud provider -> Rancher UI would provide the command to run in existing instances.
Copied that command and executed in ec2 instance where docker is installed.
Kubelet container failed to start, it keeps restarting.
error listing AWS instances: "RequestError: send request failed\ncaused by: Post https://ec2.ap-southeast-2.amazonaws.com/: dial tcp
I am in private subnet and access to internet is through proxy.
I am able to reach AWS(https://ec2.ap-southeast-2.amazonaws.com/) from EC2 instance using my proxy.
But somehow kubelet which is started in rancher agent is not using the proxy setting.
I have HTTP_PROXY and HTTPS_PROXY in /etc/systemd/system/docker.service.d/http-proxy.conf.
Also I tried to pass proxy variables as env variables to command copied from rancher UI.
Would you please advise where should I include proxy settings so that kubelet can use it?

To provide the proxy configuration environment variables, you can follow https://rancher.com/docs/rancher/v2.x/en/faq/technical/#how-can-i-add-additional-arguments-binds-environment-variables-to-kubernetes-components-in-a-rancher-launched-kubernetes-cluster

Let me know if this still needs help

Hi @superseb ,
Thanks a lot, you are a saviour.
As you said, I have added the proxy details as env variables to each service.
I spent may hours in googling where to include the proxy, the document above is hidden somewhere from googlers.
Cheers,


Use this.constructor instead of Stream.

relates to #348

Thanks @Neppord. However, this is covered by extensibility in 3.0 (#337). If you want to give it a try, you can install from my branch with npm install quarterto/highland#extensibility.


R&D on code coverage tools

[ ] add a label in the readme about how many code convert by this program commulativley

Will do it after all testing will be done


add tests for aggregation mixin methods on resource pools

Specs for resource pools and folders were promised for the recent adding back in of the aggregation methods for folders and resource pools (https://github.com/ManageIQ/manageiq/pull/20423).
Not using the existing shared aggregation mixin specs for these because these rely on with_relationship_type("ems_metadata") and the others do not so the shared specs aren't particularly applicable. The methods that rely on a host association here (which is nonexistent1) will return 0 so I'm only testing one of them to show that which should be okay since the idea behind these specs is merely to show that we're not breaking anything by removing the linked module in the future.
I'll add these same tests for folders once I get approval for these.
@miq-bot assign @kbrock
@miq-bot add_label test
[1]
> ResourcePool.first.hosts
...
Traceback (most recent call last):
...
NoMethodError (undefined method `hosts' for #<ResourcePool:0x00007f80719ec3b8>)
> ResourcePool.first.all_hosts
...
 => []

@NickLaMuro could I ask you to review these since you provided feedback on the original tests (https://github.com/ManageIQ/manageiq/pull/20149) please?

Nope, I refuse.

...
...
...
(kidding... looking now)

That's correct, they're just canary tests.

so, you review those prs to get ancestry on resource pools and folders and then we can talk about the refactoring necessary to fix this one.


2019 Object Detection Labels?

Hello,
Does the 2019 dataset have any object detection annotations?
Also the 2017 Kaggle dataset does not contain images; a download from an S3 bucket for 2017 is the only way currently.
Thanks in advance.

Hi there.
There are only bounding boxes for 2017 - not any other year.
Oisin


Remove use of window object in Date and Time native modules.

Fixes elm-lang/elm-repl#70 by changing a few instances of new window.Date to new Date.

Great, thank you!
In past versions of native modules, I think there was some clash between our Date and JavaScript's Date but I guess that is resolved now.


Document change tracking

I would like to subscribe to changes on this content page.

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

ID: f271eeaf-54e1-5476-bfbf-37f8b899c425
Version Independent ID: b5156471-8259-9cbd-f68b-ad4a9a4c864a
Content: Videos for Finance and Operations - Finance & Operations | Dynamics 365 | #MSDyn365FO
Content Source: articles/fin-and-ops/get-started/videos.md
Service: dynamics-ax-applications
GitHub Login: @josaw1
Microsoft Alias: josaw

You can only subscribe to an RSS feed of all updates to the Unified Operations content. To do that, use the following link from a browser that supports RSS feeds: RSS feed


Do not use ContentTypes in PageChooserPanel

Using ContentTypes can lead to the awkward case where they are used before the database is migrated, leading to errors on start up. Removing ContentTypes and using plain Model classes instead makes this impossible, and also makes the code clearer.
Fixes #2433

Oops, this is a dupe of #2435. The two PRs can fight it out to see which is superior! This one has deprecation warnings for the old attributes, methods, and arguments, for what it is worth.

Thanks @timheap!
@gasman Do you want to review this or shall I? Would be great to have 1.4.3 out today

@kaedroho Go for it! (Agreed re the deprecation warnings.)

Merged in 09878fcfd87f5b5ea67dd8d178667bce344c02e0 + parent
Simplified a little bit in 1e44d78a024405220f20b5c84020b9b07f7140f1
Backported to 1.4.x (without deprecation warnings) in 2b936a00e50e72740810021db49d279371eadabc
Thanks!


[WIP] Add tests using real (not mocked) node/cf-worker adapters

WHY are these changes introduced?
With the abstraction of the network calls in order to make the library isomorphic, we want to add tests that actually validate the real network calls, in both Node and Cloudflare Workers (using miniflare)
WHAT is this pull request doing?

Adds utilities to

determine if we're running tests by checking for E2ETESTS environment variable (node) or global binding (CF worker)
return a platform string, either Node <version> or `Cloudflare Worker, for use in the User-Agent string

Removes dependency on Buffer (Node-specific)
Adds setting of crypto in cf-worker adapter
Bypasses URL validator check if running network tests (E2ETESTS set)

NB: contents of experiments folder will be deleted
How it works
Node
yarn e2etests:node

Using concurrently, this will call

yarn e2etests:http_server, which runs a basic node HTTP server on port 3000 that responds to the test requests
yarn e2etests:node_client, which runs the jest-based test suite src/clients/http_client/__e2etests__/http_client-node.test.ts, which is a copy of src/clients/http_client/http_client.test.ts but uses the node adapter instead of the mock adapter.

Cloudflare Worker
yarn e2etests:cf_worker

Using concurrently, this will call

yarn e2etests:http_server, which runs a basic node HTTP server on port 3000 that responds to the test requests
yarn e2etests:serve:cf_worker, which first runs yarn e2etests:build:cf_worker to bundle the js using rollup and then runs it using miniflare

This creates a Cloudflare worker (a server, on port 8787) that represents a back-end that communicates as a client with the HTTP server
The tests themselves are non-jest implementations of 23 of the 25 tests in the node test suite.

sleep 2 && curl http://localhost:8787 - wait 2 seconds (for the build and processes above to be ready) and then send a get request to the CF worker to initiate the tests

NB: because miniflare runs a script as a server and has no process.exit() like Node to gracefully exit (at least none that I could find!), the concurrently command is run with --kill-others option, which kills all the other processes once the tests have finished.  Hence, the tests always "fail" with error code 1, even when the actual tests themselves pass.
Running all e2e tests
yarn e2etests:test

There's another alternative structure (preferred) that I've added, which overcomes the major deficiencies of how the CF Worker tests are executed as described in the original description:
Cloudflare Worker (alternative
yarn e2etests:cf_worker2

Using concurrently, this will call

yarn e2etests:http_server, which runs a basic node HTTP server on port 3000 that responds to the test requests
yarn e2etests:serve2:cf_worker, which runs yarn node --experimental-vm-modules src/clients/http_client/__e2etests__/experiments/miniflare_test.mjs

miniflare_test.mjs includes the build command (e2etests:build:cf_worker)
Then it runs the bundled script inside a Miniflare() object within a Node script

Benefits:

Being inside a Node script means that the triggering get can be done internally as an event dispatch, removing the need for the external curl command
Being inside a Node script means that the exit code can be set to properly indicate pass/fail of the test suite

Current challenges:

Passing a E2ETESTS environment variable or binding into the script via the Miniflare() object isn't working as I expected - I've temporarily changed it to use globalThis.MINIFLARE to determine if the network tests are being run or not, which is not quite ideal.


Ingestion Connector - DynamoDB

Creating an Ingestion connector source for DynamoDB which fetches all the metadata from DynamoDB and ingests in OpenMetadata

@codingwithabhi @ayush-shah lets finish this first in Milestone and provide documentation for the connector


SmartIR Platform Error - Requirements for smartir not found: ['aiofiles==0.6.0']

Home Assistant version
Core 2024.12.5
Supervisor 2024.12.0
OS 14.1
SmartIR version
1.19.0
SmartIR configuration
media_player 10:
  - platform: smartir
    name: Living room TV
    unique_id: living_room_tv
    device_code: 1000
    controller_data:
      controller_type: Broadlink
      remote_entity: remote.livingroom_remote
      delay_secs: 0.5
      num_repeats: 3
    power_sensor: binary_sensor.living_room_tv_power

Describe the bug
Upon uninstalling the old SmartIR integration (which no longer works under the current version of HA), I have installed this fork of SmartIR (and I thank you for forking and maintining this), I get the following error when checking my config:-
Platform error 'media_player' from integration 'smartir' - Requirements for smartir not found: ['aiofiles==0.6.0'].
I get the same error for climate integrations when I add them to the config, the devices show as unavailable on the dashboard.
Debug log
Logger: homeassistant.util.package
Source: util/package.py:152
First occurred: 14:07:44 (3 occurrences)
Last logged: 14:07:45

Unable to install package aiofiles==0.6.0: × No solution found when resolving dependencies: ╰─▶ Because you require aiofiles==0.6.0 and aiofiles>=24.1.0, we can conclude that your requirements are unsatisfiable.

Here we go, again.
You are NOT using my version of the SmartIR, but the old code instead. My code doesn't requires ANY dependencies as you can clearly see in the Github. You probably messed your installation, incorrectly uninstalled original SmartIR integration, or incorrectly installed mine! This is invalid issue and I am closing it right now.

aiofiles are related to asyncio, media_player.py uses asyncio:-
import asyncio

No it doesn't not!
The issue you are reporting is because your manifest.json has dependency like this:
"requirements": ["aiofiles>=0.6.0"],
https://github.com/smartHomeHub/SmartIR/blob/master/custom_components/smartir/manifest.json
My version should not:
(https://github.com/litinoveweedle/SmartIR/blob/master/custom_components/smartir/manifest.json)
Post here content of your manifest.json from your custom components.

Hi thanks and appreciate your help, manifest.json is:-
{
  "domain": "smartir",
  "name": "SmartIR",
  "codeowners": ["@litinoveweedle"],
  "dependencies": [],
  "documentation": "https://github.com/litinoveweedle/SmartIR",
  "iot_class": "assumed_state",
  "issue_tracker": "https://github.com/litinoveweedle/SmartIR/issues",
  "requirements": [],
  "version": "1.19.0"
}

I've gone down a rabbit hole though, and I think its an issue with the current version of HA (some people are having this issue with a few custom components it seems), so probably not your component thanks.

Sorry, I think I've got it sorted, I had to SSH and manually update some python stuff that the latest version of HA managed to break for some people. Weird. Thanks for your help though and for keeping this component alive

No problem. Do you have more info to share about the issue & resolution? - in case someone will end up here with same problem.

I basically followed the steps in this older thread:-
https://community.home-assistant.io/t/custom-integration-requirements-failing-to-update/772204/22
Download the Advanced SSH & Web Terminal addon GitHub - hassio-addons/addon-ssh: Advanced SSH & Web Terminal - Home Assistant Community Add-ons 3
Turn off Protection mode
Open webui
Enter docker exec -it homeassistant bash
cd /config/deps/lib/python3.12/site-packages
rm -rf msmart*
Restart homeassistant
But I deleted everything in the dependencies folder, and HA redownloaded everything it needed.

OK, thank you for an explanation, it seems plausible - somehow HA is failing to clean old (not used anymore) dependencies, including the one from original SmartIR. And when integration with same name is loaded then it require to load deprecated dependency. A bit of mess. Do you remember uninstalling original SmartIR first, HA restart and only then installing this one?
Thank you for reporting and sharing the resolution and sorry for me being edgy. People report a lot of invalid issues, I will have to work more on my attitude more. ;-)

My old install was a manual install, so I deleted the old smart ir folder (after copying my IR custom codes onto a usb), installed your fork through HACS, then restarted - I don't remember restarting after deleting the old install, so that may have been the issue.
Regardless of your attitude, I still appreciate your help and your work on this, since my home assistant is primarily for lights, TVs, and aircons, if you hadn't carried on the work on this component, most of my home assistant would be useless.


Not compiling when used with Extensions.

I keep getting sharedApplication is unavailable on iOS (App Extension).
This has happened in PFPush, PFApplication, PFPushUtilities, PFLocationManager and PFNetworkActivityIndicatorManager. I am also getting ld: warning: linking against dylib not safe for use in application extensions:

Hey @chayelheinsen, thanks for reporting this.
Feel free to use precompiled packages (from releases page or built from source by instructions in the README) if you are linking with app extensions for now, but we are going to fix this in few upcoming releases, so you can compile from source.
The warning that you see is probably because you are using Cocoapods with use_frameworks! enabled, which produces a dynamic framework instead of static one. Try removing use_frameworks! from your Podfile or again using a precompiled static framework.

@nlutsenko - Removing use_frameworks! doesn't resolve the issue. For example, I ran into an error with PFAlertView saying that UIAlertView is not available. Also, line 53 UIWindow *keyWindow = [UIApplication sharedApplication].keyWindow; gives the same error mentioned above.

use_frameworks! - related to the warning that you were getting aka linking against dylib not safe...
The workaround for extensions right now is to use a precompiled package, until we get there.

@nlutsenko I removed Parse from my pods and now I can get an archive, but their generic. I'm not sure if this is from Parse or just my settings being wrong. It's most likely my build settings being wrong.

Not sure I follow, you can go to and get the releases package - it includes Parse.framework and Bolts.framework
https://github.com/ParsePlatform/Parse-SDK-iOS-OSX/releases
If you are interested in things like Crash Reporting or ParseTwitterUtils - you can get the full download from https://parse.com/docs/downloads


Polish documentation of server agent and continuous profiling policy

[ ] If this pull request closes/resolves/fixes an existing issue, replace the issue number. Closes #.
[x] Update the CHANGES log.

Please remove the go2sky as it is archived.


A nice react native progress plugin recommend

I am so sorry to see so many problem about less than 50 percentage on Android. I recommend a component https://github.com/MrToph/react-native-progress-circle

@UniKylin  This bug has been fixed now. You can try it again.

@JackPu thx


Parallel gif rendering

I don't know if this is possible and even less how to do it, but wouldn't it make sense to try and parallelize the rendering of the frames to speed up the render function?

You mean parallelizing the ffmpeg step or the Javis side. Doing the Javis side it wouldn't work easily due to morphing where parts are computed ones but otherwise I think it should be possible.

Yeah I was thinking about the ffmpeg because I believe some time ago you told me that that takes a lot of the time smth like 50% can't remember exactly.

Yeah it's time consuming. I'm not sure it's possible but if it is it would be super helpful

Can some other software be used to combine images to gifs, like ImageMagic or libvips ?


Clarify the relation of Honeyswap and Gardens for each model

Try to explain it as a way to bridge/align the two communities. Really powerful way of frame it.

Related to #12

Done with: https://1hive.gitbook.io/gardens/requirements/honeyswap


Bug with length restrictions on rep?

I am implementing RFC 8941 and noticed testing my code that length restrictions don't quite seem to work as I expected.
val signedDecIntegral: P[String] =
   (P.char('-').?.with1 ~ digits.rep(1,12)).map {
    case (min, i) =>
        min.map(_ => "-").getOrElse("")+i.toList.mkString
  }
val decFraction: P[String] = digits.rep(1,3).string
val sfDecimal: P[(String,String)] =
   (signedDecIntegral ~ (P.char('.') *> decFraction)).map {
      case (dec: String,frac: String) =>
        (dec,frac.toList.mkString) //todo: keep the non-empty list?
     }

It seems like the length restrictions don't cause the expected errors
scala> decFraction.parse("12312323")
val res17: Either[cats.parse.Parser.Error,(String, String)] = Right((,12312323))
scala> sfDecimal.parseAll("12345678901234567890.22222")
val res18: Either[cats.parse.Parser.Error,(String, String)] = Right((12345678901234567890,22222))

Duh. My error. I was using digits which uses rep

My implementation of RFC 8941 using cats parse only took 2 days. 1.5 to code and .5 to test.

Great to hear it worked out for you!


FCM.On(FCMEvent.Notification) not firing for topic messages when app minimised.

Im am using topics to send messages to a group of android devices. When the app is open the FCM.On(FCMEvent.Notification) is fired and I can handle the message. However when the app is in the background, the notification is shown in the status bar, however the FCM.On(FCMEvent.Notification) is not fired. Whereas normal Notifications (not topic messages) are handled by FCM.On() weather the app is minimised or not

check if this line is triggered
https://github.com/evollu/react-native-fcm/blob/master/android/src/main/java/com/evollu/react/fcm/MessagingService.java#L26
what is your payload

+1

I have the same problem.
FCM.on (FCMEvent.Notification, ~) callback does not work when in only background state.
https://github.com/evollu/react-native-fcm/blob/master/android/src/main/java/com/evollu/react/fcm/MessagingService.java#L26
I tested this code.
"Log.d (TAG," Remote message received ");" This code worked fine.
i need help..
react-native: "0.52.0",
react-native-fcm: "^14.0.1"
app/build.gradle
targetSdkVersion 23
...
compile 'com.google.firebase:firebase-messaging:11.0.4'

AndroidManifest.xml
<uses-permission android:name="android.permission.INTERNET" />
<uses-permission android:name="android.permission.SYSTEM_ALERT_WINDOW"/>
<uses-permission android:name="android.permission.RECEIVE_BOOT_COMPLETED"/>
<uses-permission android:name="android.permission.VIBRATE"/>
<uses-permission android:name="android.permission.WAKE_LOCK" />

<receiver android:name="com.evollu.react.fcm.FIRLocalMessagingPublisher"/>
      <receiver android:enabled="true" android:exported="true"  android:name="com.evollu.react.fcm.FIRSystemBootEventReceiver">
        <intent-filter>
          <action android:name="android.intent.action.BOOT_COMPLETED"/>
          <action android:name="android.intent.action.QUICKBOOT_POWERON"/>
          <action android:name="com.htc.intent.action.QUICKBOOT_POWERON"/>
          <category android:name="android.intent.category.DEFAULT" />
        </intent-filter>
      </receiver>

      <service android:name="com.evollu.react.fcm.MessagingService">
        <intent-filter>
          <action android:name="com.google.firebase.MESSAGING_EVENT"/>
        </intent-filter>
      </service>

      <service android:name="com.evollu.react.fcm.InstanceIdService" android:exported="false">
        <intent-filter>
          <action android:name="com.google.firebase.INSTANCE_ID_EVENT"/>
        </intent-filter>
      </service>

App.js
FCM.on(FCMEvent.Notification, async (notif) => {
  console.log('Notification: ', notif);
});

FCM.on(FCMEvent.RefreshToken, (token) => {
  console.log('refreshToken: ', token);
});

export default class App extends Component {
  componentDidMount = () => {
    FCM.requestPermissions()
      .then(() => console.log('granted'))
      .catch(() => console.log('notification permission rejected'));

    FCM.getFCMToken().then((token) => {
      console.log('getToken: ', token);
    });
  };
`
``

Same here too @evollu.
03-05 11:13:17.816 20187 21158 D MessagingService: Remote message received

The callback does not work on receiving a message. Seems like a major thing, so I hope you'd be willing to spend some time on investigating this issue.

@christianchown if you are using mix of data and notification, the notification piece will be filtered out by FCM SDK, so there will be no data sent to JS callback.
try adding a breakpoint at
https://github.com/evollu/react-native-fcm/blob/master/android/src/main/java/com/evollu/react/fcm/MessagingService.java#L82
and see if data is null

Hi @evollu - think that mention was for @christianversloot rather than me


Integration not campatible with HA core 2024.10.1

After updating Home Assistant to 2024.10, I got a notification reporting that this integration couldn't start.
The error is:

Setup failed for custom integration 'espsomfy_rts': Requirements for espsomfy_rts not found: ['websocket-client==1.7.0']

Anything we can do, apart from rolling back to 2024.9.3?
Is an update to this required/planned?
(It's a crucial integration for my daily use ;) )

My bad. I was running an old version of this integration (2.4.2).
Once updated to 2.4.7, everything works.


[Test] Verify state expiration operations use current ledger number

[Test] Verify state expiration operations use current ledger number and not LCL. Test this for,

[ ] manual bumps
[ ] restore op
[ ] autobump
[ ] minimum expiration

This is resolved by https://github.com/stellar/stellar-core/pull/3984.


proposal: x/tools/cmd/gopls: provide config option for hover documentation placement

What version of Go are you using (go version)?

$ go version
go version devel +c4ca60313e Sun Jul 28 18:09:57 2019 +0000 linux/amd64
$ go list -m golang.org/x/tools
golang.org/x/tools v0.0.0-20190727173135-db2fa46ec33c
$ go list -m golang.org/x/tools/gopls
golang.org/x/tools/gopls v0.1.4-0.20190727173135-db2fa46ec33c

Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE="on"
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/myitcv/.cache/go-build"
GOENV="/home/myitcv/.config/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GONOPROXY=""
GONOSUMDB=""
GOOS="linux"
GOPATH="/home/myitcv/gostuff"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/home/myitcv/gos"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/home/myitcv/gos/pkg/tool/linux_amd64"
GCCGO="gccgo"
AR="ar"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD="/home/myitcv/gostuff/src/github.com/myitcv/govim/go.mod"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build108929039=/tmp/go-build -gno-record-gcc-switches"

What did you do?
I believe (from a quick test) that VSCode places hover-popups above-and-to-the-right of the mouse.
In Vim such popups have historically been below-and-to-the-right.
The text returned for a hover is as follows:
Printf formats according to a format specifier and writes to standard output.
It returns the number of bytes written and any write error encountered.

func fmt.Printf(format string, a ...interface{}) (n int, err error)

This ordering works well when the popup is show above-and-to-the-right, because the signature is near the identifier that trigger the hover.
It doesn't work so well for below-and-to-the-right popup placement, because the distance between signature and identifier is at its maximum.
Could I suggest therefore we have a config option to configure this, to place the type before/after the documentation?
Related discussion: https://github.com/myitcv/govim/issues/421
cc @stamblerre @ianthehat

I have an open issue with LSP about textDocument/hover - I mentioned this there. I feel like it's something that could be fixed really easily in the LSP. I'm a bit wary to add extra configurations like that to gopls, but depending on how quickly changes are actually made in LSP, I might be more open to it.

Thinking about https://github.com/golang/go/issues/33550 made me realize that there are some additional issues here.

Clients will probably want to vary the placement of the hover pop-up per-request, meaning that clients will also want to vary the ordering of documentation per-request. This will have to be done as a change to the LSP. Adding a configuration to gopls will only change the general "rule", but it won't offer per-request flexibility.
There are issues with the ordering even in the top-right configuration. If there is a lot of documentation, as with the json.Unmarshal example provided in https://github.com/golang/go/issues/33550, the client (in that example, VSCode), may limit the size of the hover window, forcing users to scroll to see the signature. Perhaps a solution would be to switch the ordering when the amount of documentation exceeds a certain length. Still, the size of the hover window could vary between clients, so having a strict rule is not the ideal solution.

Sorry I don't understand, why don't just make it in same way, as it was done in all other versions/editors listed in #33550: sourcegraph's go-langserver, bingo, and in the default vscode-go config with godoc or gogetdoc, (btw, including Goland quick documentation feature), signature at the top and the rest is scrollable documentation text?
Right now it makes a problem with viewing docs:

Why not fix it to normal order, as it should be? For example here is how it looks like in VS Code when turning off gopls and switching back to simple godoc:

Clients will probably want to vary the placement of the hover pop-up per-request

Small nit: I'd say "possibly" rather than "probably". Reason being, popup placement (at least in my experience with govim) is generally fixed per function. i.e. hovers go above-and-to-the-right, completion candidates below-and-to-the-right, etc.
Completely agree this is more complex than simply putting the signature at the top or bottom.

Perhaps a solution would be to switch the ordering when the amount of documentation exceeds a certain length

I think we need to determine the structure of what we are returning. There are effectively two parts:

the type/signature
the associated documentation

Currently the hover API bundles all this content together, which I think is part of the issue. The ultimate UI/UX from my perspective would be to have the type/signature always visible, and the associated documentation scroll as required. This might well be implemented using two popups; one for the type/signature, the other (immediately below/above) a max-height X-lines scrollable popup the same width as the type/signature popup, but I only mention this in passing because I think the structure point needs to be established first, the implementation is just a detail
This might even make my request to have the type/signature "at the bottom" redundant, because the max-height of the scrolling would mean the distance between the hover trigger and the type/signature is limited.

@inliquid: I think my main issue with this solution is that, in some cases, as @myitcv said, it's nicer to have the signature of the function pop-up right next to the function call in your code (so, at the bottom).
I certainly prefer the solution of having 2 windows, and I really do think that the best fix is to have structured types for hover. Perhaps a reasonable temporary solution is to add a configuration, and meanwhile to work with the LSP folks on resolving https://github.com/microsoft/language-server-protocol/issues/772.

Sorry I don't understand "it's nicer to have the signature of the function pop-up right next to the function call in your code (so, at the bottom)". Where is it "nicer"? I was wondering why it's moved to the bottom from the very beginning with a single line doc approach, now it just made UX even worse than it was, because user have to scroll every time text is long enough.

From @myitcv's comment:

This ordering works well when the popup is show above-and-to-the-right, because the signature is near the identifier that trigger the hover.

It doesn't work so well for below-and-to-the-right popup placement, because the distance between signature and identifier is at its maximum.

It can be helpful to minimize the distance between the identifier and the signature, to make things clearer to the user. I agree that this is not always the case, particularly when the documentation is long.
My current opinion on this issue is that we should:

Work with the LSP folks on providing structured hover results.
In the meantime, offer some configuration to the user. My thinking is that the signature should go on top if the level of documentation is FullDocumentation, and then on the bottom for the other cases.

Another option: provide a config option that would tell gopls to return a JSON-structured response, including signature, one-liner, synopsis and full documentation. This would effectively allow us to prototype this without any changes to the LSP spec, adding more weight to your proposal @stamblerre in https://github.com/microsoft/language-server-protocol/issues/772#issuecomment-517853089

Just mailed the change to handle part 2 of my suggestions, so @inliquid, you will see the updated behavior on master once it is merged.
@myitcv: Thanks for the suggestion - I think that's a good approach to start with.

It sounds like this discussion has converged to a result. Is this proposal issue still worth keeping open?

I think we can close this. We added workarounds to gopls to handle different cases for hover, and we can wait on responses from the LSP team on https://github.com/microsoft/language-server-protocol/issues/772.


Name correction in README.md

SConstruct calls the header path variable "headers_dir", but README.md (as well as the documentation website) calls it "headers".

I don't know who renamed it to headers_dir but it has been headers for a long time. It still is headers in "Nativescript-1.1" which is set to be merged soon so it may end up turning back to headers.
IMHO the setting should be removed completely now that godot_headers is a submodule


CI is broken since fec20d262f002450a567028d98516dcb5562d24e

The Caliper CI is currently broken : https://github.com/xcompact3d/2decomp-fft/actions/runs/4736855988/jobs/8408933909
It is broken since https://github.com/xcompact3d/2decomp-fft/commit/fec20d262f002450a567028d98516dcb5562d24e

The configuration is a bit hairy with cmake, automating it might be difficult.

I think we have two options  :

Remove the Caliper build from the CI
Fix it

I can not reproduce the error on my workstation so it is not easy to fix it from my perspective.

I haven't tried recently, will see if I can reproduce

I’ll also try the build with caliper

Good news !


dl_with_cache is not striping #/.*

For some reason when url contains rewrite (#/dl.7z) it will result as broken URL.
Broken, which should be OK
https://github.com/Ash258/GithubActionsBucketForTesting/issues/164
When there is no rewrite it is OK.
https://github.com/Ash258/GithubActionsBucketForTesting/issues/162

Fixed in 825ff07cfcde6a44c802ff540827cb24657922e0
After fix: Ash258/GithubActionsBucketForTesting#167
Before fix: Ash258/GithubActionsBucketForTesting#168


expose errors coming from sources API calls

Hi @ahussein3. Thank you for this PR !!
Do you have some examples of the errors exposed by this PR, please?
Just to make sure that we don't show any sensitive information by accident.
cheers!

Hi @chanwit,
Here is an example of error being sent back from API calls clusterName: "default/demo120-broke" message: "failed adding cluster client to pool: failed to create leaf client: Get \"https://<IP_ADDRESS>:6443/api?timeout=32s\": dial tcp <IP_ADDRESS>:6443: connect: no route to host" namespace: ""

We are not currently showing those Errors here in Core we are just Exposing the full response back so that EE can make usage of those errors.


Could not load js config file<EMAIL_ADDRESS>Unexpected token '?'

Installing the plugin on<EMAIL_ADDRESS>running node v12.22.12 (lts) and after building the admin UI via npm run build, I'm unable to start the strapi server with npm run dev.
[2022-07-25 15:01:45.839] debug: ⛔ Server wasn't able to start properly.
[2022-07-25 15:01:45.841] error: Could not load js config file<EMAIL_ADDRESS>Unexpected token '?'
Error: Could not load js config file<EMAIL_ADDRESS>Unexpected token '?'
    at loadJsFile ($folder/node_modules/@strapi/strapi/lib/core/app-configuration/load-config-file.js:18:11)
    at loadFile ($folder/node_modules/@strapi/strapi/lib/core/app-configuration/load-config-file.js:35:14)
    at Object.loadPlugins ($folder/node_modules/@strapi/strapi/lib/core/loaders/plugins/index.js:96:26)
    at async Strapi.loadPlugins ($folder/node_modules/@strapi/strapi/lib/Strapi.js:284:5)
    at async Promise.all (index 1)
    at async Strapi.register ($folder/node_modules/@strapi/strapi/lib/Strapi.js:316:5)
    at async Strapi.load ($folder/node_modules/@strapi/strapi/lib/Strapi.js:414:5)
    at async Strapi.start ($folder/node_modules/@strapi/strapi/lib/Strapi.js:163:9)

Figured out the issue; apparently the plugin doesn't support node v12? I updated to v14.9.1 and it's working properly now. Might want to add it to the requirements?


Switch to new-style Django middleware and update django-reversion

This switches from the deprecated old-style middleware to the new-style Django 1.10+ middleware. All the middleware we are using is supposed to be compatible with MIDDLEWARE (as opposed to MIDDLEWARE_CLASSES).
SessionAuthenticationMiddleware is no longer needed – see https://docs.djangoproject.com/en/1.11/releases/1.10/#features-removed-in-1-10
Switching to new-style middleware also fixed the problem with django-reversion and the metadata views.

Codecov Report

Merging #211 into develop will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #211   +/-   ##
========================================
  Coverage    91.61%   91.61%
========================================
  Files           47       47
  Lines         1669     1669
  Branches       146      146
========================================
  Hits          1529     1529
  Misses         119      119
  Partials        21       21

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 24dad15...3ee093d. Read the comment docs.


Use WARNING text prepended to message

Since Jenkins console does not support colors users may open support tickets saying that they are seeing a error message in the console while using Lizzy client. So it is better to explicitly show that it is just a "warning" message.

Coverage remained the same at 93.298% when pulling<PHONE_NUMBER>51776b33f26305319688416334a49b on rafaelcaricio:less-alarm-for-users into c238521bb65700a7b9e8200025603d68aa4228f5 on zalando:release2.0.

:+1:


Random starting positions for box.py

Currently, the box in box.py always starts at the bottom-left corner. It would be more interesting if the box started from random positions within the white box area :).

Can I get assigned?

Sure thing all the best :)

can i get assigned

Someone is already working on this issue... feel free to chose others :)

hey can I be assigned to this?

hey can I be assigned to this?

Someone is already working on this issue... feel free to chose others :)

Please reassign it to me

@Shaatvika
it has been more than half an hour, Can you please show me some progress
Otherwise, I will be forced to unassign you

@aniruddhajoshi100 You Have been assigned to this issue, Happy Coding!

can I be assigned to this??


Feature Request: Rename "Refresh Mods"

I believe "Refresh Mods" on the mods tab looks a bit confusing, so rename it to "Install Mods" or "Injects Mods" or "Update Mods" or "Update Installation" or something.

Done with https://github.com/affederaffe/BeatSaberModManager/commit/48890f32ebef0b24188a6a576db7571da8223124.


Implement user public profile

Users now only can see their own profile. Goal of this issue is to create public profiles.
This will later be used for sending personal messages, viewing user posts etc.
DoD:
User can view other users profiles by clicking on their picture

Working on it, though I won't be making the PM now.

Thank you for contributing :)
I will make another issue for personal messages.


IPv6 is GA for GWLB.    Update the Limitations section

Azure documentation for Azure Gateway Load Balancer still shows no support for IPv6.   But it GA'ed in September.

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

ID: b0169fb9-0289-2bc9-d934-6bd00a084f14
Version Independent ID: c232010b-2290-091c-70fc-b78760c6cc01
Content: Gateway load balancer - Azure Load Balancer
Content Source: articles/load-balancer/gateway-overview.md
Service: load-balancer
GitHub Login: @mbender-ms
Microsoft Alias: mbender

@stdistef
Thanks for your feedback! We will investigate and update as appropriate.

We fixed this, please close this issue
#please-close


Correct read function to stop trimming of final character of user input

correcting loss of final character during INPUT A$;PRINT A$.

Hey man, thanks a heap for your merge request!
As you've probably noticed, this project is pretty much dead and unmaintained - I haven't even looked at it for nearly a year, let alone actually done some much-needed work on fixing it up to work properly.
For your first commit (correcting the read function), thanks for catching that - from what I can tell, that bug came around when working on basic-web (for whatever reason it requires the last two characters to be removed, but I can't remember why - probably something in the convoluted mess that is IOInterface). I'd be happy to merge that commit in.
For the SHELL command, the purpose of this project was to create a toy interpreter for the BASIC flavour found in the 'Basic!' iOS app, which I was playing around with at the time - the end goal of the project was to have an interpreter that could run (nearly) all projects created in that app.
While node-basic does have a few little extra things, adding an entirely new BASIC command is definitely out of the scope of this project (nevertheless one that provides access to the shell, which doesn't even really exist in iOS).
For these reasons, I can't really merge the SHELL command (but as I said, I would be completely happy to merge the other commit - I believe I will need you to create a new pull request with only that commit for me to do this, however). If you are interested in maintaining this project, you are of course free to do whatever you want with your own fork, and if you want I can add a link to the README of this project to your maintained fork.


bubble up renderTemplate errors

currently we just return null https://github.com/autotelic/fastify-mail/blob/main/index.js#L93

Closed with #33


fix: Clean up margins on In Demand Now section and make layout more responsive on /consign2

Some last minute QA for /consign2 before we launch. This PR:

Truncates long titles for the recent auction result
Centers the recent auction result in the large layout
Shrinks the average price chart to fit the width
Centers the average price chart in the small layout

LGTM 👍
#mergeOnGreen


react slick slider duplicate items when infinite true

Why is slick adding cloned slides to my slider when I set infinite to TRUE and have only one or two items in the slider list? Can anyone help me to resolve this issue, all I need is to set INFINITE TRUE but want to remove duplicate items from slider if I have only one or two items.
I will appreciate your effort and thanks in advance.

same to me.

@salmanbakhtiar786 I'm facing same issue. Any Update on this ?

We found same issue too even without using infinite property by default. And I don't know why another issues closed. Please, does anyone can help about solution?

import Slider from "react-slick";
  const settings = {
    dots: true,
    infinite: false,
    slidesToShow: 5,
    slidesToScroll: 2,
    vertical: true,
    verticalSwiping: true,
    };,

And this is my slider

<Slider {...settings}>

         naveed

         naveed

     </Slider>

butt then i just import Styles file from package
import "slick-carousel/slick/slick.css";
import "slick-carousel/slick/slick-theme.css";

My issue wasn't showing duplicate slides but rather trying to wrap slides in an animated component and having the animation occur more than once.  My solution was to query the DOM to find out if it was a cloned element and safe the result in a ref.
const isAClone = useRef(false)
const ref = useRef(null)

 useEffect(() => {
    let element = ref.current
    while (element) {
      if (element.classList.contains("slick-cloned")) {
        isAClone.current = true
        return
      }
      element = element.parentElement
    }
  }, [ref])

  useEffect(() => {
  if(!isAClone.current){
  ...do some animation if the element is in view
  } else if(isAClone.current){
  ... do the animation even if the element isn't in view
  },[isAClone, other stuff]

  return (
  <animated.div ref={ref}>
  {children
  </animated.div>

const settings = useMemo(() => {
return {
speed: 200,
infinite: productImage.length >= 7 ? true : false,
slidesToShow: productImage.length >= 7 ? 7 : productImage.length,
swipe: false,
adaptiveHeight: true,
slidesToScroll: 1,
arrows: false,
lazyLoad: true,
responsive: [
{
breakpoint: 768,
settings: {
speed: 200,
swipe: false,
infinite: productImage.length >= 3 ? true : false,
slidesToShow: productImage.length >= 3 ? 3 : productImage.length,
adaptiveHeight: true,
slidesToScroll: 1,
arrows: false,
lazyLoad: true,
}
},
]
    }
})

this will help set infinite and slides to show according to amount of images or items you server for example if you have less items than what you had intended to show just show all of it this way it will be less and work perfectly do the same for the infinite, this should work.(worked for me).

Add - adaptiveHeight: true in sliders settings.


update to support chef 13

it gives an error of ERROR: ArgumentError: Please pass in the object class to hydrate or use #edit_hash when trying to save a data bag item, whether encrypted or not.

I'll see if I can investigate - just started a new job, so ramping up on that for now, but may have some free time to try to repro this bug.