Let debugProfilePlatformChannels work in release mode

When it comes to startup profiling, it is very helpful to look at platform channels. debugProfilePlatformChannels today only works in debug and profile mode. Unfortunately, using profile mode is less accurate for startup profiling, because of the service isolate introducing additional overhead.
This PR allows this toggle to work in release mode. Note that there are two parts to debugProfilePlatformChannels:

Adding timeline events
Logging statistics about platform channels

I also considered adding a separate toggle to limit the scope of this change to the former, but that seems like complexity that we might not need at this time.
Towards #102189
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[ ] I added new tests to check the change I am making, or this PR is test-exempt.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Will it have any effect on performance when disabled? It is disabled by default so I'd only expect a small AOT size increase.

Will it have any effect on performance when disabled? It is disabled by default so I'd only expect a small AOT size increase.

Yep, every platform channel message will now be doing a boolean check against wether it was turned on or not.  That wasn't the case previously because the treeshaker would remove that kRelease check since kRelease is a constant.  I think ideally we want a way to turn this on so it will get built into an app on release and ripped out on release without some flag.

In practice removing a boolean check is probably a micro-optimization and probably wouldn't have any effect on performance as a whole. I remember you mentioned that there were some benchmarks for platform channels though, can you point me to them to verify this?
We can also make profilePlatformChannels const, but that would mean that developers have to patch the framework instead of being able to turn it on within their own Dart code.

In practice removing a boolean check is probably a micro-optimization and probably wouldn't have any effect on performance as a whole. I remember you mentioned that there were some benchmarks for platform channels though, can you point me to them to verify this?
We can also make profilePlatformChannels const, but that would mean that developers have to patch the framework instead of being able to turn it on within their own Dart code.

My take is that anything that happens at greater frequency than the frame rate we should do our best to make it as fast as possible.  Boolean checks aren't as straightforward in dart as they are in other languages because of boxed types.  There are these things called kernel transforms that allow people to change compiled code.  Thats how we rip out asserts.  I'd prefer profilePlatformChannels was a constant that got switched on with a kernel transform.  No need to edit the framework code.  Let me see if i can find where the kernel transforms are, they are in the dart repository.

Here is some build info we could use to turn on features like this: https://github.com/flutter/flutter/blob/52ef9d8827e8a53e880b83d59ea030b58a6b47e7/packages/flutter_tools/lib/src/build_info.dart#L21
Maybe there is already something like this you can already latch onto so you don't need to make a new option.  Jacob would be a good person to ask.

I made the toggle a const instead. I think in the near term, we shouldn't worry too much about usability concerns and having to patch this file to turn it on is an acceptable compromise. In the longer term, this should be something integrated into devtools and we can probably defer a proper way to turn it on at runtime for then. What do you think?

I overwrote google testing because it seems like there's some infrastructure issue. I also checked that there are no other users of debugProfilePlatformChannels in Google3.


[nnx] fix demo notebook

What does this PR do?
Updates demo notebook according to the latest changes.

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

Project coverage is 58.75%. Comparing base (83d118a) to head (585e844).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #3744   +/-   ##
=======================================
  Coverage   58.75%   58.75%
=======================================
  Files         101      101
  Lines       12413    12413
=======================================
  Hits         7293     7293
  Misses       5120     5120

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


[catkin_prepare_release] UnicodeEncodeError: 'ascii' codec can't encode character u'\xf6' in position 1592: ordinal not in range(128)

$ git clone git@github.com:ros-planning/geometric_shapes.git && cd geometric_shapes/
$ catkin_prepare_release
Prepare the source repository for a release.
Repository type: git
Found packages: geometric_shapes
Prepare release of version '0.5.3' [Y/n]?
Trying to push to remote repository (dry run)...
Everything up-to-date
Checking if working copy is clean (no staged changes, no modified files, no untracked files)...
Traceback (most recent call last):
  File "/opt/ros/kinetic/bin/catkin_prepare_release", line 390, in <module>
    main()
  File "/opt/ros/kinetic/bin/catkin_prepare_release", line 319, in main
    update_changelog_sections(missing_changelogs_but_forthcoming, new_version)
  File "/opt/ros/kinetic/lib/python2.7/dist-packages/catkin/package_version.py", line 131, in update_changelog_sections
    f.write(data)
UnicodeEncodeError: 'ascii' codec can't encode character u'\xf6' in position 1592: ordinal not in range(128)
$ git log -n 1
commit 70262c6303f8e1ae348b92db1aca588c61890b6e
$ which  catkin_prepare_release
/opt/ros/kinetic/bin/catkin_prepare_release
$ apt-cache policy ros-kinetic-catkin
ros-kinetic-catkin:
  Installed: 0.7.6-0xenial-20170217-212821-0800
  Candidate: 0.7.6-0xenial-20170217-212821-0800

xf6 in unicode seems to be o + umlaut, which occurs in CHANGELOG.rst.

I second that.
Just encounterd this today with u"\xfc" corresponding to ü.

I faced this issue too and the patch in #902 fixed it, allowing me to release

Addressed by #902.


Avoid Yoda Style on condition

Just saw this little part.

I love this! It's the best pull request ever submitted ;)
:+1:


local playground

I am trying to set up a local installation of the playground.
I guess there are some functionalities like including metaParameters that only work when I spin up a node server.
I tried running this code in the browser code editor:
var makerjs = require('makerjs');

function Heart(r, a2) {

    var a = a2 / 2;

    var a_radians = makerjs.angle.toRadians(a);

    var x = Math.cos(a_radians) * r;
    var y = Math.sin(a_radians) * r;

    var z = makerjs.solvers.solveTriangleASA(90, 2 * r, 90 - a);

    this.paths = {
        arc1: new makerjs.paths.Arc([x , 0], r, -a, 180 - a),
        line1: new makerjs.paths.Line([x * 2, -y], [0, -z + y])
    };

    this.paths.arc2 = makerjs.path.mirror(this.paths.arc1, true, false);
    this.paths.line2 = makerjs.path.mirror(this.paths.line1, true, false);
}

Heart.metaParameters = [
    { title: "radius", type: "range", min: .01, max: 100, step: 1, value: 10 },
    { title: "angle", type: "range", min: 60, max: 120, step: 1, value: 90 }
];

module.exports = Heart;

`
I get this errror:
Uncaught DOMException: Failed to construct 'Worker': Script at 'file:///Users/andreasdaiminger/Desktop/wood/maker_origin.js/playground/worker/render-worker.js' cannot be accessed from origin 'null'.(…)
Please point me in the right direction. Do I have to spin up a node server to make metaParameters work?

No server is required, (other than a simple http static web server). The Worker is purely a browser function. Which browser are you using?

yep. simple http server solves the issue.
thanks


Steps to apply ngx-datatatble in webpack + angular2 project

Hi All,
I tried to add ngx-datatatble in webpack + angular2 project . after adding the required import module in app.module.ts and adding this - import<EMAIL_ADDRESS>in vendor.js, still i am not able to see the datattable added in my page. it shows issue below
If 'ng-template' is an Angular component, then verify that it is part of this module.
If 'ng-template' is a Web Component then add "CUSTOM_ELEMENTS_SCHEMA" to the<EMAIL_ADDRESS>of this component to suppress this message.
Any help on this would be very helpful for me to proceed on my project.

You did not include the module in your project.
Please fill out the form in the future.
Please provide a demo and maybe i can help more.


deps: Downgrade uber fx to 1.18.1

Due to a seemingly small change in https://github.com/uber-go/fx/releases/tag/v1.18.2, the ordering of our modules/invocations/provides now impacts the integration tests and node such that certain components do not work as intended.
An attempt to resolve it was made here: #1204 which did fix the issue, but it is still unclear why. Rather than merge that PR without understanding why the change makes a difference, we would rather downgrade the fx dependency and keep that PR open as an investigation.
This PR should fix the integration tests on CI.
cc @Wondertan

Codecov Report

Merging #1216 (5b59e13) into main (18860df) will decrease coverage by 0.09%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #1216      +/-   ##
==========================================
- Coverage   55.76%   55.66%   -0.10%
==========================================
  Files         161      161
  Lines        9650     9650
==========================================
- Hits         5381     5372       -9
- Misses       3740     3746       +6
- Partials      529      532       +3

Impacted Files
Coverage Δ

header/core/listener.go
52.83% <0.00%> (-5.67%)
:arrow_down:

fraud/bad_encoding.go
63.00% <0.00%> (-3.00%)
:arrow_down:

ipld/get_namespaced_shares.go
88.32% <0.00%> (-2.19%)
:arrow_down:

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

Superseded by #1218


Deploy to Maven Central

All artifacts of each release must be deployed to maven central for maximum availability.

Now available. See release notes and readme


Fix compile errors on GWT 2.8.2

@adrianmsmith thanks for your contribution.
To be honest, I more or less gave up on GWT. If you want to maintain this project, I am happy to hand it over to you. IMHO still the best place would be to bring this directly into the GWT project itself. However, as you already figured out JSR310 is complicated and to bring all features to the client would be a huge pile of work...

Hey, yes I would be happy to take over maintainership of this project. Do you know how one does that? Is there a GitHub feature for that, or we just announce in the README that I am maintaining my repository now?
Could I ask, where did the code code from? Did you write it, or did it come from somewhere else? Are we sure that there are no copyright issues in the code? (That someone comes along at a later date and claims they wrote the code and that it's e.g. commercial)
I think I would not do much more than you - I am using this library in a new project, so if I find bugs or features I need then I'll add them. And I'm happy to review and accept pull requests from anyone. But I probably won't do much more than that. But I figure that's better than nothing.

There are several options:

I can simply transfer this repository to you (a github organization owned by you or directily to your user adrianmsmith).
I grant you admin permissions for this repo
You create a fork of this repo and keep working and maintaining on that fork
You create a new fresh repo whereever you like and copy the code over to it.

I am happy with whatever suits you best. Also I do not care much about the copyright of the code. You may refactor it to a new package namespace, change maven groupId and artifactId, etc. as you like. I can also send you a PGP signed Email confirming that again. My only requirements would be obvious: code should remain open-source and you shall not reject my authorship.
Have fun with it.

Regarding copyright, I was just wondering where the code came from? It doesn't seem to have been originally designed for GWT, so I was wondering if it came from another project? I see that most of the source files have "Copyright (c) 2007-2012, Stephen Colebourne & Michael Nascimento Santos" at the top. I think this source code was derived from https://www.threeten.org/ which seems to have been released under a BSD (not Apache) license (as far as I can tell?) Honestly I'm no expert on this sort of stuff, I was just wondering if you had more information there, and was sure that everything was OK in terms of copyright of the code that already exists.

Ups. Too long ago, ideed. Thanks for taking this serious.
Your observation is perfectly right. I researched my emails and found a correspondence on this I made with Stephen Colebourne:
Hi Jorg,
I have no problems with the threetenbp code being used in this way,
especially from a legal POV - the BSD license was intended for this
kind of use case.

Bear in mind that the code forked from the actual JDK 8 codebase a
while back, and as such there may be subtle differences. In
particular, some interfaces have been converted to abstract classes,
and method references have been converted to constants. The Hijrah
calendar system is completely incompatible and you should not rely on
it as is.

The code in git now is the code to use, not the last released code (in
maven central). The current code passes most of the test cases from
JDK 8, though not every last one. It is almost ready for another
release.

Feel free to send me pull requests for areas where you find
differences with JDK 8, such as the Hijrah calendar system.

thanks, and feel free to ask more questions,
Stephen

On 7 April 2014 21:50, Jörg Hohwiller wrote:
> Hi Stephen,
>
> for Google Web Toolkit we are looking forward to supporting JSR310
> (java.time).
> Therefore we would like to include source code derived from your javax.time
> backport:
> https://github.com/ThreeTen/threetenbp
> with some modifications (package change to java.time, changes for GWT
> compatibility, etc.).
>
> I am already aware of your statement about the namespace:
>
>> JDK8 is using the java.time namespace, not javax.time. As such, there
>> is no way to create a compatible backport without using the bootstrap
>> classpath. Such a backport would also violate the JCP/JSR rules as
>> only compatible implementations can use the namespace.
>
>
> But for GWT we are not providing a backport for Java < 1.8 nor compiled code
> at all but just
> GWT compatible source-code (located in some com/google/emul/java/time
> location) so GWT
> compiler can transform Java 1.8 code using java.time to JavaScript that runs
> in the browser (with some obvious limitations).
>
> Are there any concerns or legal issues we might be missing?
> http://www.gwtproject.org/makinggwtbetter.html#clas
>
> All files will carry some "Copyright YYYY Google Inc." header together with
> a ASL 2.0 License note.
> I do not want to do this without some sort of (informal) confirmation or
> permission.
> Of course we would keep the original author credit in JavaDocs, etc.
> Feel free to forward to other contributors of the ThreeTen project.
>
> Hope to hear from you.
> Thanks
>   Jörg

If you want to have the original mail, please send me an email to hohwille at users.sourceforge.net (see my git commits) so I can forward it to you.
Also I give you permission to change the license from ASL 2.0 to BSD if that helps you in any way.

Ah that all sounds great, thanks for confirming this!
So yeah, I am happy to take over ownership of this project. I think the best approach would be if you transfer it to my account (adrianmsmith)
I have deleted my fork (as the instructions on GitHub state that you can't transfer a repository to someone with a fork.)


M2 Mac , simulator build error: Unsupported Swift architecture

Description
M2 Mac , simulator build error: Unsupported Swift architecture
Reproducing the issue
M2 Mac , simulator build error: Unsupported Swift architecture

Firebase SDK Version
10.23
Xcode Version
15.3
Installation Method
CocoaPods
Firebase Product(s)
Storage
Targeted Platforms
iOS
Relevant Log Output
No response
If using Swift Package Manager, the project's Package.resolved

Expand Package.resolved snippet

Replace this line with the contents of your Package.resolved.

If using CocoaPods, the project's Podfile.lock

Expand Podfile.lock snippet

Replace this line with the contents of your Podfile.lock!

facing the same issue on the M1 machine as well.
Not able to run on the Simulator, but it works fine on the device.

@taimoorsuleman Strange that there are build errors for the x86_64 simulator on an M1 machine. Something might be misconfigured.
@fangyuxi We're not able to reproduce and will likely need more information to help.

@taimoorsuleman Strange that there are build errors for the x86_64 simulator on an M1 machine. Something might be misconfigured.
@fangyuxi We're not able to reproduce and will likely need more information to help.

@paulb777  I am getting x86_64 error after updating to firebase latest version.
in previous version its fine. i checked all relevant setting and looks fine.

I have the same issue, also running on an M2 chip. For us this happens when trying to build our watch OS app. On the M3 Pro of a coworker the same code runs fine.

Apparently apple differentiates between "via iphone" and a regular watch simulator. I just went with the default Xcode simulators. After creating a fresh simulator that's paired with an iphone simulator, everything works fine. I just need to run it on the watchOS simulator (not the via iphone one).


null is not an object (evaluating '_RNBackgroundActionsModule.default.start')

I tried running your example but I get the bellow error when I'm trying to start the backgroud action:
Unhandled promise rejection on start
TypeError: null is not an object ( evaluating
'_RNBackgroundActionsModule.default.start')
Any ideas of how I can solve this?
Suggestion: could you maybe add a more detailed example inside the doc?

@PuskyWasTaken, sorry you have this issue.
After installation, make sure to clean your build folder for iOS or run ./gradlew clean for Android.

I tried everything mentioned in #21
I also tried ./gradlew clean but with no success.

@PuskyWasTaken, could you provide the code that is causing the error?

I did try resetting the cache.
The code:
First, I am using exactly the example code provided.
import BackgroundService from 'react-native-background-actions';

// You can do anything in your task such as network requests, timers and so on,
// as long as it doesn't touch UI. Once your task completes (i.e. the promise is resolved),
// React Native will go into "paused" mode (unless there are other tasks running,
// or there is a foreground app).

let heartBeat = false;
const veryIntensiveTask = async (taskDataArguments) => {
    // Example of an infinite loop task
    const { delay } = taskDataArguments;
    await new Promise(async (resolve) => {
        for (let i = 0; BackgroundService.isRunning(); i++) {
            console.log(i);
            heartBeat = !heartBeat;
            await sleep(delay);
        }
    });
};

const options = {
    taskName: 'Example',
    taskTitle: 'ExampleTask title',
    taskDesc: 'ExampleTask desc',
    taskIcon: {
        name: 'ic_launcher',
        type: 'mipmap',
    },
    color: '#ff00ff',
    parameters: {
        delay: 1000,
    },
};

Here is the call
const HeartbeatClass = () => {
  const imageSize = heartBeat ? 150 : 100;

  React.useEffect(() => {
    const startBackgroundService = async () => {
        await BackgroundService.start(veryIntensiveTask, options)
    };
    startBackgroundService();
  }, []);

  return (
    <View style={styles.container}>
      <View style={styles.view}>
        <Image source={heart} style={{ width: imageSize, height: imageSize }} resizeMode="contain" />
      </View>
      <View style={styles.view}>
      </View>
    </View>
  );
};

@PuskyWasTaken, it looks like an error on the link between the React Native and the module. Could you try to remove and add the library?
yarn remove react-native-background-actions
yarn add react-native-background-actions
And then post your package.json?

I did @Rapsssito.
Below is the package.json (changed extension to .txt so I can upload it)
package.txt

@PuskyWasTaken, are you on iOS or Android?

Android

@PuskyWasTaken, could you try to run the app from Android Studio and check if are still having the issue?

@Rapsssito I will, in a few minutes

@Rapsssito After a whole day of debugging it not works but for the love of GOD..I don't know why. Thank you for the help and quick responses anyway!
For future people that might read this..good luck


chore(master): release 1.4.0

:robot: I have created a release beep boop
1.4.0 (2023-12-13)
Features

Add option afterInitConfig (#1695) (ec2b287)

Bug Fixes

auto stop extension when starting nvim failed (#1703) (67e10ad)
buffer: prefer real path as buffer name (#1699) (eff13f2)

This PR was generated with Release Please. See documentation.

TODO:

cmdline manager
docs about remote development

:robot: Release is at https://github.com/vscode-neovim/vscode-neovim/releases/tag/v1.4.0 :sunflower:


Add Itte by Yorushika

Song Name
Itte
Artist Name
Yorushika
Source
Youtube
Youtube Link
MFysBilc9BeDAvuA
Direct File Link
No response
Song ID
857484
Start Offset [ms]
0
Extra
Created from Auto Nong.
Created for level: 96111010

reject
invalid YouTube link


Trigger handlers when events are fired on child elements

Fix the bug that ujs won't handle events when clicking on the following link
Destroy

It's a mistake by me, which is quite disastrous.

I'm a little worried about the release process. Should I also include the dist file in the commit?

@liudangyi yes, I think having the dist file updated is a good idea since devs using it from github (with npm/yarn and soon with bundler) would be able to get last version without having to wait for a new release

with npm/yarn and soon with bundler

Ugh :confused:
Action Cable works with bundler without needing a committed dist file at all. Can npm/yarn not do that? 😟

@matthewd now that this is a gem probably we can make it work exactly as ActionCable. Right now to use the "dev" version of the assets users should require the full entry path manually:
//= require rails-ujs/src/rails-ujs

That is because having just //= require rails-ujs it will try to load the file referenced in the main key of the package.json/bower.json.
Another idea:
For users of Sprockets + npm/Yarn we can add an index.js to the repository and the require to main entry file on this:
// index.js
//= require './src/rails-ujs'

This will make the //= require 'rails-ujs' work using npm/Yarn + the lib from github.
We can exclude that index.js from the distributed packages so it won't affect to users of the npm/bower package.

One small disadvantage of the approach used by Action Cable is that coffee_script is a hard dependency when you're using it from github or a local path:
If you generate a new app combining --edge (or --dev) and --skip-coffee options
% rails/railties/exe/rails new baz --skip-coffee --edge

You will get an error trying to retrieve application.js asset:
LoadError: cannot load such file -- coffee_script

I guess is not a big deal since it won't happen to final users.


get_anchors.py not working

When I run get_anchors I receive the error:
File "get_anchors.py", line 83, in run_kmeans
anchor_dim = ann_dims.shape[1]
IndexError: tuple index out of range
I am running on my own dataset. I checked the shape of my ann_dims is (227,). ``

I am not really familiar with how the function get_anchors.py is working but I was able to get it to work without any problem on my own dataset.
The problem seems to be coming from here (line 117 to 129):
# run k_mean to find the anchors
annotation_dims = []
    for image in train_imgs:
        cell_w = image['width']/grid_w
        cell_h = image['height']/grid_h

        for obj in image['object']:
            relative_w = (float(obj['xmax']) - float(obj['xmin']))/cell_w
            relatice_h = (float(obj["ymax"]) - float(obj['ymin']))/cell_h
            annotation_dims.append(map(float, (relative_w,relatice_h)))
    annotation_dims = np.array(annotation_dims)

    centroids = run_kmeans(annotation_dims, num_anchors)

and especially on this line (line 126):
annotation_dims.append(map(float, (relative_w,relatice_h)))

Indeed, it should add a tuple containing the relative width and height of the bounding boxes but it seems that in your case this is not what it is added as it has only 1 dimension and not 2.
Can you try to print ann_dims and annotation_dims before being converted to an array, please?

It has to do with the map function in python 3. It returns a map object which is only the first part of the tuple. That's why you're missing the second dimension. You should look up how python 3 handles maps.

@anvenkat09 so you mean the behavior of the map function in python 3 and 2 is different?
@madelinehawkins I ran successfully get_anchors.py with python 2.7, maybe just use a different version of python if you were using python 3 before.

@anvenkat09 Thank you for pointing me in the right direction. It was due to the map function. I was able to resolve this and get it working with python 3 by switching.
annotation_dims.append(map(float, (relative_w,relatice_h)))
to
annotation_dims.append(list(map(float, (relative_w,relatice_h))))
This could be resolved for future use from python 3 users with a simple if else inserted there to check to see which python somebody is using.
@LucasVandroux Thank you for responding and helping as well.


AEUX 0.7.6 fail to export to AE

The file is not exported to the AE.
After sends the file Sketch to the AE, this attached message appears in the AE.
The version of those are all latest and my iOS version is 10.14.6(macOS Mojave)

This should now be fixed in v0.7.8. Let me know if the problem persists.

@adamplouff Thanks for the update but it was not successful for me. I'm using v0.7.8 and cannot send from Sketch to AE anymore. Have tried AE 17.5.1 (build 47) and Sketch 70.1, 70.2 and 70.3 without success. I don't receive any error messages, just see the spinning animation after sending.

@adamplouff Thanks for the update but it was not successful for me. I'm using v0.7.8 and cannot send from Sketch to AE anymore. Have tried AE 17.5.1 (build 47) and Sketch 70.1, 70.2 and 70.3 without success. I don't receive any error messages, just see the spinning animation after sending.


Adding New IQueryable Type

I'm digging into EF Core trying to figure out how to add a new query type to support table hints. I have my own SqlServerQuerySqlGeneratorFactory with VisitTable overridden and ready to receive hints and an extension method similar to IIncludableQueryable that looks like so to bundle them with a given query:
internal static readonly MethodInfo WithHintsMethodInfo =
    typeof(QueryableExtensions).GetTypeInfo().GetDeclaredMethods(nameof(WithHints))
        .Single(info => info.GetGenericArguments().Length == 1 && info.GetParameters()
                            .Any(parameterInfo => parameterInfo.Name == "hints" &&
                                                      parameterInfo.ParameterType != typeof(string)));

public static IQueryable<TEntity> WithHints<TEntity>(this IQueryable<TEntity> query, TableHints hints)
{
    return new HintedQueryable<TEntity>(
        query.Provider is EntityQueryProvider
            ? query.Provider.CreateQuery<TEntity>(
                Expression.Call(
                    instance: null,
                    method: WithHintsMethodInfo.MakeGenericMethod(typeof(TEntity)),
                    arguments: new [] {query.Expression, Expression.Constant(hints)}))
            : query);
}

HintedQueryable looks like so:
public class HintedQueryable<TEntity> : IQueryable<TEntity>
{
    private readonly IQueryable<TEntity> _queryable;

    public HintedQueryable(IQueryable<TEntity> queryable)
    {
        _queryable = queryable;
    }

    public IEnumerator<TEntity> GetEnumerator() => _queryable.GetEnumerator();

    IEnumerator IEnumerable.GetEnumerator() => GetEnumerator();

    public Expression Expression => _queryable.Expression;
    public Type ElementType => _queryable.ElementType;
    public IQueryProvider Provider => _queryable.Provider;
}

This results in a "could not parse expression" exception.
I've been digging through the internals and have gotten to the SqlServerQueryModelVisitor pieces trying to see what I need to add to support my HintedQueryable type. I feel like I'm close to an answer but any insight or a better solution would appreciated. I'm working with the 1.1.2 code.

Take a look at how AsNoTracking and Include are implemented. What you want to do is create a custom ResultOperator; take a look here

Thanks for this. In my investigation, I also saw some registration going on with the Remotion libraries. Is that necessary for something like this as well or will EF auto detect ResultOperators et al? To recap, the process looks like so:

Create IQueryable Type
Create Method for Type
Create ResultOperator
Create ExpressionNode
[Register Type and Node with Remotion] - correct?
Create Custom SQL Generator and Replace EF Core's Generator
Detect Output from ResultOperator
Modify SQL as Necessary

Am I missing anything?

That flow looks roughly correct. However, you don't need a custom IQueryable type, just add the extension method.
To register your new node, you need to call INodeTypeProviderFactory.RegisterMethods - An example of that, for our FromSql node, can be found in RelationalQueryCompilationContextFactory.

I have all this done including the removal of my custom IQueryable type. However, modeling this after Include is giving me a confusing result. If I copy Include's constructor and take a LambdaExpression as my second parameter, I get an exception about not being able to cast a TypedParameterExpression to a LambdaExpression. However, using Include works fine.
For your reference: IncludeExpressionNode
public IncludeExpressionNode(
            MethodCallExpressionParseInfo parseInfo,
            [NotNull] LambdaExpression navigationPropertyPathLambda)
            : base(parseInfo, null, null)

My TableHintExpressionNode
public TableHintExpressionNode(
            MethodCallExpressionParseInfo parseInfo,
            LambdaExpression hints)
            : base(parseInfo, null, null)

In examining the TypedParameterExpression, my custom TableHint container class is passed through fine. Unfortunately, I'm not seeing how to extract the runtime hints from the TypedParameterExpression for it to be useful. Can you explain what's going on?

I was able to massage the parameters both on passing and reception. I eventually settled on a generics based solution so the hints that get passed are directly tied to types therefore reducing the likelihood of "weirdness." However, this feels sort of like a hack as I was hoping to pass values directly rather than having to do type examination and some Activator.CreateInstance wizardry. I think in this case the "hack" is more or less OK. I would still like some input on my above question in case I need to extend EF in the future. Thanks regardless!

The expression node ctor parameter corresponds to the type of argument being passed to the custom operator method. So, for Include, the type is LambdaExpression because the Include method takes a lambda as the argument, I.e. customer => customer.Orders.
Now your operator takes a TableHints object, which if passed in line to the method call, will typically result in a ConstantExpression node in the expression tree. However, if you are seeing TypedParameterExpression, it is likely because of our parameterizer, which aggressively turns constants into parameters in order to reduce query cache fragmentation. The easiest thing to do is likely to just apply the NotParameterizedAttribute to the table hints parameter on the operator method - This will prevent parameterization of the argument.
BTW, what you are doing would be a useful feature for EF. Would you be open to sending a PR? If so, we could further refine things during the PR process.

I did not know that attribute existed. Worked like a charm. I'll work on a PR over the next few evenings.


Rename to "Mesh for Data"

Replace "The Mesh for Data" with "Mesh for Data" in website, docs and code comments.
This will remove the burden of avoiding "the The Mesh for Data ..." in docs.
Note that we are not changing the github repository names nor docker repository names.

Closed by https://github.com/IBM/the-mesh-for-data/pull/201


Add missing closing tag

Hi @nzmacgeek — would you mind merging this in soon? I'm getting errors from AutoPkg on every run as a result of this issue. Thanks!

My apologies for the delay in doing this. Been afk on an extended holiday break!


nvidia GPU support for caps and multiple uuids

This improves nvidia support for multiple uuids per container and fixes
the API to add individual capabilities.
Signed-off-by: Michael Crosby<EMAIL_ADDRESS>

Codecov Report

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

@@            Coverage Diff             @@
##           master    #2506      +/-   ##
==========================================
- Coverage   48.98%   44.76%   -4.23%
==========================================
  Files          85       93       +8
  Lines        7592     9555    +1963
==========================================
+ Hits         3719     4277     +558
- Misses       3183     4585    +1402
- Partials      690      693       +3

Flag
Coverage Δ

#linux
48.98% <ø> (ø)
:arrow_up:

#windows
41.04% <ø> (?)

Impacted Files
Coverage Δ

oci/spec.go
40% <0%> (-10%)
:arrow_down:

snapshots/native/native.go
43.3% <0%> (-10%)
:arrow_down:

metadata/snapshot.go
45.8% <0%> (-8.96%)
:arrow_down:

archive/compression/compression.go
43.93% <0%> (-8.9%)
:arrow_down:

content/local/writer.go
52.63% <0%> (-7.37%)
:arrow_down:

archive/tar.go
43.11% <0%> (-7.04%)
:arrow_down:

metadata/containers.go
47.97% <0%> (-6.62%)
:arrow_down:

metadata/images.go
58.46% <0%> (-4.7%)
:arrow_down:

metadata/buckets.go
56.33% <0%> (-4.6%)
:arrow_down:

metadata/adaptors.go
41.46% <0%> (-4.49%)
:arrow_down:

... and 53 more

Continue to review full report at Codecov.

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


CSS files missing in the archetype 12 and 13

In archetype 12 and 13, referenced css are missing:
27.07.2018 11:07:28.818 WARN [0:0:0:0:0:0:0:1 [1532682440380] GET /libs/granite/core/content/login.html HTTP/1.1] com.adobe.granite.ui.clientlibs.impl.FileBundle Referenced path in /apps/aem63training/components/content/breadcrumb/clientlib/css.txt does not exist: /apps/aem63training/components/content/breadcrumb/clientlib/less/breadcrumb.css

@artika4biz Thank you for the hands up!


CSV data collection

Currently we only save to CSV manually, it would be good to do this automatically after a sweep.

Right now we actually don't "export" any data, only use the default sqlite db that qcodes requires to do data collection on. After a sweep finishes we should automatically export the data in csv format. When we run measurements using the queue, we should automatically save at the end of the queue.

Feature implemented- renamed "origin files" to "exported files" as that makes more sense. Autosaves are made to this directory automatically after every sweep. Tested with Sweep0D (time). Needs to be tested with queue.
I believe we should auto save on the completion of every single sweep, although perhaps we could implement a setting to change this behavior to export on the final sweep of the queue.

Closing issue for now, I will re-open it if we want to add a setting as discussed in previous comment. Right now the philosophy is that more saves is better than few, but we shall see.


Backend Improvements #50 - garotm - more backend work.

https://github.com/fleXRPL/RunOn/issues/50
Summary of Changes
Backend Improvements

Test Coverage Enhancement

Improved overall test coverage from 79% to 96.46%
All modules now at 100% coverage except main.py (83%)
Added comprehensive test suite for authentication and endpoints

Dependency Management

Fixed version compatibility issues in requirements.txt and requirements-dev.txt
Pinned critical test dependencies:pytest==7.4.3
pytest-cov==4.1.0
pytest-asyncio==0.21.1
httpx==0.24.1

Added starlette==0.27.0 for FastAPI compatibility

Authentication Improvements

Enhanced token verification error handling
Added detailed error messages for authentication failures
Improved test coverage for authentication edge cases

Android Build Configuration

Java Environment Setup

Fixed Java version compatibility issues
Configured OpenJDK 17 across all build scripts
Added consistent Java configuration to all Android scripts:

format_and_lint.sh
fix_formatting.sh
dev_setup.sh
run_local.sh

Build Script Improvements

Standardized Java configuration across all scripts
Added proper error handling and exit codes
Improved script output and logging

Testing Infrastructure

Test Framework Updates

Implemented proper async test support
Added mock environment configurations
Enhanced test client setup and configuration

CI/CD Preparation

All tests passing with high coverage
Linting and formatting checks passing
Build scripts ready for CI integration

Next Steps

Plan to achieve 100% coverage in main.py
Remaining uncovered lines (23-24, 40-41) are error handling cases
Will add additional tests in follow-up PR

This PR establishes a solid foundation for the RunOn backend and Android build system, with comprehensive testing and proper environment configuration.

SonarQube

The SonarQube analysis shows:

Quality Gate Passed ✅
0 New issues ✅
94.4% Coverage on New Code ✅
0.0% Duplication on New Code ✅

Your PR summary shows significant improvements:

Test coverage increased from 79% to 96.46%
All modules at 100% coverage except main.py (83%)
Fixed dependency versions and compatibility issues
Enhanced authentication and error handling

The only remaining items are those two uncovered lines in main.py (lines 40-41) which you've already documented as error handling cases to be addressed in a follow-up PR.
Source: PR #51


the free links don't work anymore

the free links don't work anymore
Your input can't be opened:
VLC is unable to open the MRL ' http://pub8.di.fm:80/di_trance'. Check the log for details.
Your input can't be opened:
VLC is unable to open the MRL ' http://pub7.di.fm:80/di_trance'. Check the log for details.
Your input can't be opened:
VLC is unable to open the MRL ' http://pub1.di.fm:80/di_trance'. Check the log for details.
Your input can't be opened:
VLC is unable to open the MRL ' http://pub4.di.fm:80/di_trance'. Check the log for details.
Your input can't be opened:
VLC is unable to open the MRL ' http://pub6.di.fm:80/di_trance'. Check the log for details.
Your input can't be opened:
VLC is unable to open the MRL ' http://pub5.di.fm:80/di_trance'. Check the log for details.
Your input can't be opened:
VLC is unable to open the MRL ' http://pub2.di.fm:80/di_trance'. Check the log for details.

Hi,
Thank you for your bug report.
I don't know if you have seen the notice on the site:

Non-premium listening is unfortunately no longer possible due to changes in regulations / fees. You can still listen for free on the website or with the app, but no longer in the player of your choice.

You can register a trial account and use the listen key to generate a new playlist though.

Bummer about the end of fee streaming like this.  I'm just excited to see that 2 other people are looking at my repo!  Woohoo!

Bummer about the end of fee streaming like this. I'm just excited to see that 2 other people are looking at my repo! Woohoo!

I'm here, +1 :)


Add ReturnTypeWillChange attribute to disable deprecation warnings on PHP 8.1

Hi,
I realize this is a really old version, but it is one of the dependencies in our stack which we want to run on PHP 8.1 (and which otherwise works fine on 8.1), and the deprecation emmited by PHP 8.1 is breaking some of our GraphQL responses.
There may be more of these needed, but I haven't run into them so far.
Would it be possible to tag a quick patch release with this fix?
Thanks!

We no longer support pre-14 versions, see https://github.com/webonyx/graphql-php/commit/b49ffe990f1876e24660c6f2af7296c6dc8ce037


Test that the GPU Poisson solver solution is numerically divergence-free

Until the CPU and GPU Poisson solvers are unified this should give us confidence that the GPU Poisson solver is doing its job.
Found it easier to write the test independently so this supersedes PR #238
Resolves #200

Much relieved to see this test. I'm sad that my old test that used dispatch had some bug, but it's academic at this point since we hope to unify the solvers soon.

I think you just had to permute the indices but yeah, and we can go back and use your test once they're unified.

Oh. I missed the fact that you have to permute both on input and output. Got it.


Create deploy-2012-r3-azure-lcs.md

Docs Build status updates of commit 9d158b2:
:warning: Validation status: warnings

File
Status
Preview URL
Details

dynamicsax2012-technet/a.md
:warning:Warning
View
Details

dynamicsax2012-technet/about-absence-administration.md
:warning:Warning
View
Details

dynamicsax2012-technet/about-attachments.md
:warning:Warning
View
Details

dynamicsax2012-technet/3pl.md
:warning:Warning
View
Details

dynamicsax2012-technet/about-automatic-adjustment-transactions.md
:warning:Warning
View
Details

dynamicsax2012-technet/about-batch-attributes.md
:warning:Warning
View
Details

dynamicsax2012-technet/about-batch-merges-in-inventory.md
:warning:Warning
View
Details

dynamicsax2012-technet/abc-code.md
:warning:Warning
View
Details

dynamicsax2012-technet/about-axd-document-classes.md
:warning:Warning
View
Details

dynamicsax2012-technet/about-analyzing-variances-for-a-completed-production-order.md
:warning:Warning
View
Details

dynamicsax2012-technet/accounts-payable-setup-overview.md
:warning:Warning
View
Details

dynamicsax2012-technet/accounting-policy.md
:warning:Warning
View
Details

dynamicsax2012-technet/accrual-accounting.md
:warning:Warning
View
Details

dynamicsax2012-technet/ach.md
:warning:Warning
View
Details

dynamicsax2012-technet/accounting-supervisor-security-role-ledgeraccountingsupervisor.md
:warning:Warning
View
Details

dynamicsax2012-technet/accounts-payable.md
:warning:Warning
View
Details

dynamicsax2012-technet/accounts-receivable-clerk-security-role-custinvoiceaccountsreceivableclerk.md
:warning:Warning
View
Details

dynamicsax2012-technet/accrued-revenue.md
:warning:Warning
View
Details

dynamicsax2012-technet/accounting-rule.md
:warning:Warning
View
Details

dynamicsax2012-technet/accounting-manager-security-role-ledgeraccountingmanager.md
:warning:Warning
View
Details

dynamicsax2012-technet/activity-occurrence.md
:warning:Warning
View
Details

dynamicsax2012-technet/activity-analysis-report-smmactivityanalyses.md
:warning:Warning
View
Details

dynamicsax2012-technet/actual-cost.md
:warning:Warning
View
Details

dynamicsax2012-technet/accounts-receivable-centralized-payments-clerk-security-role-paymaccountsreceivablecentralpaymclerk.md
:warning:Warning
View
Details

dynamicsax2012-technet/accounting-manager-long-term-solvency-report-accountingmanagerlongtermsolvency.md
:warning:Warning
View
Details

dynamicsax2012-technet/accounts-receivable-public-sector.md
:warning:Warning
View
Details

dynamicsax2012-technet/activity.md
:warning:Warning
View
Details

dynamicsax2012-technet/accounts-payable-roadmap.md
:warning:Warning
View
Details

dynamicsax2012-technet/add-and-maintain-your-personal-contacts.md
:warning:Warning
View
Details

dynamicsax2012-technet/add-a-batch-attribute-to-an-item-for-a-customer.md
:warning:Warning
View
Details

dynamicsax2012-technet/accounts-payable-reports.md
:warning:Warning
View
Details

dynamicsax2012-technet/accounting-manager-profitability-analysis-report-accountingmanagerprofitabilityanalysis.md
:warning:Warning
View
Details

dynamicsax2012-technet/accounting-manager-short-term-solvency-report-accountingmanagershorttermsolvency.md
:warning:Warning
View
Details

dynamicsax2012-technet/acquisition-date.md
:warning:Warning
View
Details

dynamicsax2012-technet/add-and-maintain-your-positions-of-trust.md
:warning:Warning
View
Details

dynamicsax2012-technet/add-a-batch-attribute-group-to-an-item-for-a-customer.md
:warning:Warning
View
Details

dynamicsax2012-technet/accounts-receivable-setup-overview-public-sector.md
:warning:Warning
View
Details

dynamicsax2012-technet/add-a-batch-order-to-a-consolidated-batch-order.md
:warning:Warning
View
Details

dynamicsax2012-technet/add-a-batch-attribute-to-an-item.md
:warning:Warning
View
Details

dynamicsax2012-technet/add-a-batch-attribute-to-an-attribute-group.md
:warning:Warning
View
Details

dynamicsax2012-technet/accounts-receivable-reports.md
:warning:Warning
View
Details

dynamicsax2012-technet/activity-log.md
:warning:Warning
View
Details

dynamicsax2012-technet/active-ingredient.md
:warning:Warning
View
Details

dynamicsax2012-technet/additionalsigninoption-xpath-method-microsoft-dynamics-retail-sharepoint-web-storefront-controls.md
:warning:Warning
View
Details

dynamicsax2012-technet/add-batch-attribute-values-to-an-inventory-batch-for-potency-items.md
:warning:Warning
View
Details

dynamicsax2012-technet/accounts-payable-public-sector.md
:warning:Warning
View
Details

dynamicsax2012-technet/add-values-to-an-enumerate-type-batch-attribute.md
:warning:Warning
View
Details

dynamicsax2012-technet/accounts-receivable-roadmap.md
:warning:Warning
View
Details

dynamicsax2012-technet/addorremovediscountcodesrequest-discountcodesoperation-property-microsoft-dynamics-commerce-runtime-messages.md
:warning:Warning
View
Details

dynamicsax2012-technet/address-parseaddressfromxml-method-microsoft-dynamics-commerce-runtime-datamodel.md
:warning:Warning
View
Details

This comment lists only the first 50 files in the pull request.

dynamicsax2012-technet/a.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/about-absence-administration.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.
Line 52: [Warning] Link 'https://technet.microsoft.com/en-us/library/aa556694(v=ax.60)' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
Line 54: [Warning] Link 'https://technet.microsoft.com/en-us/library/aa554509(v=ax.60)' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
Line 56: [Warning] Link 'https://technet.microsoft.com/en-us/library/aa500373(v=ax.60)' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
Line 58: [Warning] Link 'https://technet.microsoft.com/en-us/library/aa582545(v=ax.60)' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
Line 60: [Warning] Link 'https://technet.microsoft.com/en-us/library/aa618740(v=ax.60)' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
Line 62: [Warning] Link 'https://technet.microsoft.com/en-us/library/aa583231(v=ax.60)' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.

dynamicsax2012-technet/about-attachments.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/3pl.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.
Line 19: [Warning] Link 'https://technet.microsoft.com/en-us/library/hh208669(v=ax.60)' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.

dynamicsax2012-technet/about-automatic-adjustment-transactions.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.
Line 94: [Warning] Link 'https://technet.microsoft.com/en-us/library/aa553205(v=ax.60)' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.

dynamicsax2012-technet/about-batch-attributes.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.
Line 56: [Warning] Link 'https://technet.microsoft.com/en-us/library/hh209255(v=ax.60)' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
Line 58: [Warning] Link 'https://technet.microsoft.com/en-us/library/hh209431(v=ax.60)' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
Line 60: [Warning] Link 'https://technet.microsoft.com/en-us/library/hh208645(v=ax.60)' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
Line 62: [Warning] Link 'https://technet.microsoft.com/en-us/library/hh209393(v=ax.60)' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
Line 64: [Warning] Link 'https://technet.microsoft.com/en-us/library/hh209284(v=ax.60)' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
Line 66: [Warning] Link 'https://technet.microsoft.com/en-us/library/hh227369(v=ax.60)' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
Line 68: [Warning] Link 'https://technet.microsoft.com/en-us/library/aa616001(v=ax.60)' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
Line 70: [Warning] Link 'https://technet.microsoft.com/en-us/library/aa585863(v=ax.60)' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.

dynamicsax2012-technet/about-batch-merges-in-inventory.md

Line 1: [Warning] Link 'https://technet.microsoft.com/en-us/library/aa591648' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/abc-code.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/about-axd-document-classes.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/about-analyzing-variances-for-a-completed-production-order.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/accounts-payable-setup-overview.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/accounting-policy.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/accrual-accounting.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/ach.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/accounting-supervisor-security-role-ledgeraccountingsupervisor.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/accounts-payable.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/accounts-receivable-clerk-security-role-custinvoiceaccountsreceivableclerk.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/accrued-revenue.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/accounting-rule.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/accounting-manager-security-role-ledgeraccountingmanager.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/activity-occurrence.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/activity-analysis-report-smmactivityanalyses.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/actual-cost.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/accounts-receivable-centralized-payments-clerk-security-role-paymaccountsreceivablecentralpaymclerk.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/accounting-manager-long-term-solvency-report-accountingmanagerlongtermsolvency.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/accounts-receivable-public-sector.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/activity.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/accounts-payable-roadmap.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/add-and-maintain-your-personal-contacts.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/add-a-batch-attribute-to-an-item-for-a-customer.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/accounts-payable-reports.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/accounting-manager-profitability-analysis-report-accountingmanagerprofitabilityanalysis.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/accounting-manager-short-term-solvency-report-accountingmanagershorttermsolvency.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/acquisition-date.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/add-and-maintain-your-positions-of-trust.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/add-a-batch-attribute-group-to-an-item-for-a-customer.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/accounts-receivable-setup-overview-public-sector.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/add-a-batch-order-to-a-consolidated-batch-order.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/add-a-batch-attribute-to-an-item.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/add-a-batch-attribute-to-an-attribute-group.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/accounts-receivable-reports.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/activity-log.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/active-ingredient.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/additionalsigninoption-xpath-method-microsoft-dynamics-retail-sharepoint-web-storefront-controls.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/add-batch-attribute-values-to-an-inventory-batch-for-potency-items.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/accounts-payable-public-sector.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/add-values-to-an-enumerate-type-batch-attribute.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/accounts-receivable-roadmap.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/addorremovediscountcodesrequest-discountcodesoperation-property-microsoft-dynamics-commerce-runtime-messages.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/address-parseaddressfromxml-method-microsoft-dynamics-commerce-runtime-datamodel.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

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 1dc4e9e:
:warning: Validation status: warnings

File
Status
Preview URL
Details

dynamicsax2012-technet/about-absence-administration.md
:warning:Warning
View
Details

dynamicsax2012-technet/about-accounting-distributions-and-subledger-journal-entries-for-purchase-orders.md
:warning:Warning
View
Details

dynamicsax2012-technet/3pl.md
:warning:Warning
View
Details

dynamicsax2012-technet/a.md
:warning:Warning
View
Details

dynamicsax2012-technet/about-batch-merges-in-inventory.md
:warning:Warning
View
Details

dynamicsax2012-technet/about-ax-table-classes.md
:warning:Warning
View
Details

dynamicsax2012-technet/about-attachments.md
:warning:Warning
View
Details

dynamicsax2012-technet/about-answer-groups-and-answers.md
:warning:Warning
View
Details

dynamicsax2012-technet/about-audit-cases.md
:warning:Warning
View
Details

dynamicsax2012-technet/about-audit-policy-rules.md
:warning:Warning
View
Details

dynamicsax2012-technet/about-analyzing-costs-for-a-production-order.md
:warning:Warning
View
Details

dynamicsax2012-technet/accounts-receivable-setup-overview-public-sector.md
:warning:Warning
View
Details

dynamicsax2012-technet/accounts-payable-reports.md
:warning:Warning
View
Details

dynamicsax2012-technet/accounting-manager-operational-efficiency-report-accountingmanageroperationalefficiency.md
:warning:Warning
View
Details

dynamicsax2012-technet/accounting-manager-long-term-solvency-report-accountingmanagerlongtermsolvency.md
:warning:Warning
View
Details

dynamicsax2012-technet/accrue-subscription-revenue.md
:warning:Warning
View
Details

dynamicsax2012-technet/accounts-payable.md
:warning:Warning
View
Details

dynamicsax2012-technet/accrual-accounting.md
:warning:Warning
View
Details

dynamicsax2012-technet/accrued-revenue.md
:warning:Warning
View
Details

dynamicsax2012-technet/accounting-manager-short-term-solvency-report-accountingmanagershorttermsolvency.md
:warning:Warning
View
Details

dynamicsax2012-technet/accounts-receivable.md
:warning:Warning
View
Details

dynamicsax2012-technet/accounting-manager-security-role-ledgeraccountingmanager.md
:warning:Warning
View
Details

dynamicsax2012-technet/accounts-payable-public-sector.md
:warning:Warning
View
Details

dynamicsax2012-technet/accounts-receivable-setup-overview.md
:warning:Warning
View
Details

dynamicsax2012-technet/acquisition-date.md
:warning:Warning
View
Details

dynamicsax2012-technet/ach.md
:warning:Warning
View
Details

dynamicsax2012-technet/accounting-rule.md
:warning:Warning
View
Details

dynamicsax2012-technet/accounts-receivable-centralized-payments-clerk-security-role-paymaccountsreceivablecentralpaymclerk.md
:warning:Warning
View
Details

dynamicsax2012-technet/accounts-payable-roadmap.md
:warning:Warning
View
Details

dynamicsax2012-technet/add-a-batch-attribute-to-an-item.md
:warning:Warning
View
Details

dynamicsax2012-technet/activity.md
:warning:Warning
View
Details

dynamicsax2012-technet/actual-cost.md
:warning:Warning
View
Details

dynamicsax2012-technet/accounting-policy.md
:warning:Warning
View
Details

dynamicsax2012-technet/accounts-payable_1.md
:warning:Warning
View
Details

dynamicsax2012-technet/add-a-bom-line-to-a-component.md
:warning:Warning
View
Details

dynamicsax2012-technet/addorremovediscountcodesrequest-constructor-microsoft-dynamics-commerce-runtime-messages_1.md
:warning:Warning
View
Details

dynamicsax2012-technet/accounting-supervisor-security-role-ledgeraccountingsupervisor.md
:warning:Warning
View
Details

dynamicsax2012-technet/add-a-batch-attribute-to-an-attribute-group.md
:warning:Warning
View
Details

dynamicsax2012-technet/activity-analysis-report-smmactivityanalyses.md
:warning:Warning
View
Details

dynamicsax2012-technet/add-and-maintain-your-personal-contacts.md
:warning:Warning
View
Details

dynamicsax2012-technet/accounts-receivable_1.md
:warning:Warning
View
Details

dynamicsax2012-technet/activatedeviceserviceresponse-methods-microsoft-dynamics-commerce-runtime-services-messages.md
:warning:Warning
View
Details

dynamicsax2012-technet/add-values-to-an-enumerate-type-batch-attribute.md
:warning:Warning
View
Details

dynamicsax2012-technet/activatedeviceserviceresponse-constructor-microsoft-dynamics-commerce-runtime-services-messages.md
:warning:Warning
View
Details

dynamicsax2012-technet/accounting-manager-profitability-analysis-report-accountingmanagerprofitabilityanalysis.md
:warning:Warning
View
Details

dynamicsax2012-technet/accounts-receivable-clerk-security-role-custinvoiceaccountsreceivableclerk.md
:warning:Warning
View
Details

dynamicsax2012-technet/accounts-receivable-payments-clerk-security-role-paymaccountsreceivablepaymentsclerk.md
:warning:Warning
View
Details

dynamicsax2012-technet/addorremovediscountcodesrequesthandler-supportedrequesttypes-property-microsoft-dynamics-commerce-runtime-workflow.md
:warning:Warning
View
Details

dynamicsax2012-technet/addorremovediscountcodesrequest-discountcodesoperation-property-microsoft-dynamics-commerce-runtime-messages.md
:warning:Warning
View
Details

dynamicsax2012-technet/add-a-batch-order-to-a-consolidated-batch-order.md
:warning:Warning
View
Details

This comment lists only the first 50 files in the pull request.

dynamicsax2012-technet/about-absence-administration.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.
Line 52: [Warning] Link 'https://technet.microsoft.com/en-us/library/aa556694(v=ax.60)' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
Line 54: [Warning] Link 'https://technet.microsoft.com/en-us/library/aa554509(v=ax.60)' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
Line 56: [Warning] Link 'https://technet.microsoft.com/en-us/library/aa500373(v=ax.60)' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
Line 58: [Warning] Link 'https://technet.microsoft.com/en-us/library/aa582545(v=ax.60)' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
Line 60: [Warning] Link 'https://technet.microsoft.com/en-us/library/aa618740(v=ax.60)' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
Line 62: [Warning] Link 'https://technet.microsoft.com/en-us/library/aa583231(v=ax.60)' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.

dynamicsax2012-technet/about-accounting-distributions-and-subledger-journal-entries-for-purchase-orders.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/3pl.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.
Line 19: [Warning] Link 'https://technet.microsoft.com/en-us/library/hh208669(v=ax.60)' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.

dynamicsax2012-technet/a.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/about-batch-merges-in-inventory.md

Line 1: [Warning] Link 'https://technet.microsoft.com/en-us/library/aa591648' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/about-ax-table-classes.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.
Line 32: [Warning] Link 'https://technet.microsoft.com/en-us/library/gg765455(v=ax.60)' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
Line 83: [Warning] Link 'https://technet.microsoft.com/en-us/library/gg937256(v=ax.60)' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.

dynamicsax2012-technet/about-attachments.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/about-answer-groups-and-answers.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/about-audit-cases.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/about-audit-policy-rules.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/about-analyzing-costs-for-a-production-order.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/accounts-receivable-setup-overview-public-sector.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/accounts-payable-reports.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/accounting-manager-operational-efficiency-report-accountingmanageroperationalefficiency.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/accounting-manager-long-term-solvency-report-accountingmanagerlongtermsolvency.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/accrue-subscription-revenue.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/accounts-payable.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/accrual-accounting.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/accrued-revenue.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/accounting-manager-short-term-solvency-report-accountingmanagershorttermsolvency.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/accounts-receivable.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/accounting-manager-security-role-ledgeraccountingmanager.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/accounts-payable-public-sector.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/accounts-receivable-setup-overview.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/acquisition-date.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/ach.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/accounting-rule.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/accounts-receivable-centralized-payments-clerk-security-role-paymaccountsreceivablecentralpaymclerk.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/accounts-payable-roadmap.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/add-a-batch-attribute-to-an-item.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/activity.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/actual-cost.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/accounting-policy.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/accounts-payable_1.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/add-a-bom-line-to-a-component.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/addorremovediscountcodesrequest-constructor-microsoft-dynamics-commerce-runtime-messages_1.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/accounting-supervisor-security-role-ledgeraccountingsupervisor.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/add-a-batch-attribute-to-an-attribute-group.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/activity-analysis-report-smmactivityanalyses.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/add-and-maintain-your-personal-contacts.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/accounts-receivable_1.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/activatedeviceserviceresponse-methods-microsoft-dynamics-commerce-runtime-services-messages.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/add-values-to-an-enumerate-type-batch-attribute.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/activatedeviceserviceresponse-constructor-microsoft-dynamics-commerce-runtime-services-messages.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/accounting-manager-profitability-analysis-report-accountingmanagerprofitabilityanalysis.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/accounts-receivable-clerk-security-role-custinvoiceaccountsreceivableclerk.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/accounts-receivable-payments-clerk-security-role-paymaccountsreceivablepaymentsclerk.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/addorremovediscountcodesrequesthandler-supportedrequesttypes-property-microsoft-dynamics-commerce-runtime-workflow.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/addorremovediscountcodesrequest-discountcodesoperation-property-microsoft-dynamics-commerce-runtime-messages.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

dynamicsax2012-technet/add-a-batch-order-to-a-consolidated-batch-order.md

Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.

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.


fix rstrip of documentation

libsonata is lucky enough to miss those characters at the end but yes. I will do a similar PR there as well.


[DOC] Updated Readme link to turnstile to the english version

Hey, I really like your project. Just a small thing I noticed and wanted to mention.
In the Readme there is a link to https://www.cloudflare.com/fr-fr/products/turnstile/ and I think it would be more convenient to use https://www.cloudflare.com/products/turnstile/ instead.
Kind regards,
Leo

Hey,
You totally right, will do it now


Multi-arch support (arm64)

Hi,
I would love to use this, but unfortunately all of the container images used are x86_64 only, so that results in exec format error on some of my nodes and I would want to do arm64 runner jobs.

@samip5 Is it the actual runner image: https://github.com/evryfs/github-actions-runner you want to run on arm, or the operator (this repo) itself?

@samip5 Is it the actual runner image: https://github.com/evryfs/github-actions-runner you want to run on arm, or the operator (this repo) itself?

I would like to run both.

this was afair solved in the runner image repo


Include array deserialization causes exception

NEST/Elasticsearch.Net version: (Originally tested on 2.4.5) 2.5.0
Elasticsearch version: 2.4.0
Description of the problem including expected versus actual behavior:
I am trying to deserialize my ES query which has an include field containing an array of strings. The
query is working in Kibana Sense but throws an exception when trying to deserialize. The error occurs when serializing the aggregations section.
Here is the query:
{
  "aggs": {
    "sizes": {
      "terms": {
        "field": "size",
        "size": 20,
        "include": [
          "35",
          "50",
          "70",
          "75",
          "100"
        ]
      }
    }
  }
}

Steps to reproduce:
1.
Elasticsearch.Net.IElasticsearchSerializer.Deserialize<SearchRequest>(new MemoryStream(Encoding.UTF8.GetBytes(query)))

Exception is thrown.

Provide DebugInformation (if relevant):
Exception thrown:

An exception of type 'Newtonsoft.Json.JsonSerializationException' occurred in Newtonsoft.Json.dll but                            was not handled in user code
Additional information: Cannot deserialize the current JSON array (e.g. [1,2,3]) into type 'Nest.TermsIncludeExclude' because the type requires a JSON object (e.g. {"name":"value"}) to deserialize correctly.

Thanks for reporting @Jafferwaffer. TermsIncludeExclude doesn't have a read implementation currently; will add one now 👍

Thank you @russcam


Is there a way to add our own SVGs?

This would be so amazing. I've been poking around in the code trying to get any custom shape to work... turns out I don't know nearly enough about SVGs to make this happen.
I was able to get a snowflake to load but it's way off center. I haven't a clue how to fix it.


Rewrite transform_imu

transform_imu needs to be finished so that it outputs the correct tf transform. Right now the ned->enu transform is good, but there needs to be a tf transform to go along with it.

Changes made and pushed to GitHub. Verified with test bagfile data.


Fixed ping command

It was showing negative numbers...

Array.sample() exists.

Fixed itself when putting it on a VPS


Failed to resolve module specifier "crypto"

I get this error after importing, what's going on, I don't have crypto in my project
error message：Failed to resolve module specifier "crypto"
version：three-custom-shader-material 5.3.8

This comes from the object-hash library used if your framework doesn’t have a polyfill for th NodeJS package crypto
it’s a bug. Will be fixed with the release of v6 this week


Integrate Cadene pretrainedmodels [fixes #142]

Edited create_model() to import models from pretrainedmodels.
Added tests to the same function.

Regarding your warning message when working when pretrainedmodels, please note I couldn't load 'nasnetalarge' with this patch. Moreover, the pretrained 'nasnetamoblie' (which does load) doesn't perform as expected out of the box.
I didn't have much time to look into that, but it looks like these models are WIP.


Fix a template icon

Please fix the template icon with something more professional.
Current icon looks like a student project not something that companies want to use.

I find it amazing that folks can make such requests and statements. Template 10 is all volunteer. Rather than complain about this excellent free resource why not fork and create an icon that you feel would be more appealing to the group. Then the group will have an opportunity to input their support or lack thereof.

would have to have to agree with Jeff...

Plus, I like the icon.

I used a few times this template.
It just introduce noise and chaos to the project, I switch to blank template.
There are too many bugs here.

Then don't use it if it doesn't fit your needs :-) If you have issues or constructive feedback feel free to share it, there's always room for improvements. Otherwise, statements like your are just made to trigger flames and useless discussions.


Run sass-migrator with forward flag

Just testing things out.

@mdo Hello! I've just migrated two projects to Sass modules (a website and a React UI library) and I'm open to help with the migration here.
By the way, it will be a breaking change for SCSS consumers of Bootstrap. I haven't come across any Bootstrap's plans with Sass modules so please forgive me if there is one. Am I right when I say Bootstrap 6 must be released before October 2022 when support for @import is dropped? I'm also trying to figure out a way how to support both @import and Sass modules in v5.x but I cannot imagine that at the moment—and v5 is already on the way…

Yeah I just don't see how we can make the change this late in the game. We're about to launch our beta, so unfortunately I think we need to keep things as they are and make further infrastructure/tooling changes in v6.


Laravel Cypress 0.2.1 Version Upgraded & Also Whole Project upgraded to 8 Version

Laravel Cypress 0.2.1 Version Upgraded & Also Whole Project upgraded to 8 Version

Ok the tests are passing now, thanks for contributing!


RC4 and OpenSSL 3.0.8

Hello OpenSSL,
I have tried to put this question in the<EMAIL_ADDRESS>mailing list but I given up waiting for my request to join it to happen.  Anyway here goes:
I am porting a product from OpenSSL 1.1.1o to OpenSSL 3.0.8 and am trying to remove all deprecated code.  Things have progressed well so far but I’m stuck with RC4 (which I need to keep for backwards compatibility in the product).
Here’s the essence of the old implementation:
             const char* pKey = "Some arbitrary passphrase string";
             const char* pInput = "The quick brown fox jumped over the lazy frog";
             unsigned char* pOutput = new unsigned char[512];
             … etc

             RC4_KEY oRC4Key;
             RC4_set_key(&oRC4Key, iKeyLen, (const unsigned char*)pKey);
             RC4(&oRC4Key, iInputLen, (const unsigned char*)pInput, pOutput);

My new implementation looks like this (note: I have already loaded the “legacy” and “default providers) …
            memset(pOutput, 0, iOutputLen);
            EVP_CIPHER_CTX* pRC4Ctx = EVP_CIPHER_CTX_new();
            if (!pRC4Ctx)
            {
                        printf("EVP_CIPHER_CTX_new() Failed\n");
                        return 1;
            }

            EVP_CIPHER* pRC4 = EVP_CIPHER_fetch(NULL, "RC4", NULL);
            if (!pRC4)
            {
                        printf("EVP_CIPHER_fetch() Failed\n");
                        return 1;
            }

            if (!::EVP_EncryptInit_ex2(pRC4Ctx, pRC4, (const unsigned char*)pKey, NULL, NULL))
            {
                        printf("EVP_EncryptInit_ex2() Failed\n");
                        return 1;
            }

            if (!::EVP_EncryptUpdate(pRC4Ctx, pOutput, &iOutLen, (const unsigned char*)pInput, iInputLen))
            {
                        printf("EVP_EncryptUpdate() Failed\n");
                        return 1;
            }

            ::EVP_EncryptFinal_ex(pRC4Ctx, pOutput + iOutLen, &iTmpLen);
            ::EVP_CIPHER_free(pRC4);
            ::EVP_CIPHER_CTX_free(pRC4Ctx);

I would expect the result of both pieces of code to produce the same cipher text given the same key and input data but this is not the case when the key is as above (i.e. some arbitrary string).  The results are the same if I make the key a 128 bit string (e.g. "0123456789ABCDEF") which makes me think I need to do something with the key before supplying it to EVP_EncryptInit_ex2(…) but I cannot find anything to say what that should be.  Any help would be much appreciated.
Thanks,
Paul

I have tried to put this question in the<EMAIL_ADDRESS>mailing list but I given up waiting for my request to join it to happen.

You need to subscribe to the mailing list via:
https://mta.openssl.org/mailman/listinfo/openssl-users
As for the arbitrary key length support, you need to set the OSSL_CIPHER_PARAM_KEYLEN OSSL_PARAM to set the actual key length, otherwise 128 bits is used by default.
size_t key_length;
OSSL_PARAM params[2]= {
        OSSL_PARAM_END, OSSL_PARAM_END
};

params[0] = OSSL_PARAM_construct_size_t(OSSL_CIPHER_PARAM_KEYLEN, key_length);

EVP_EncryptInit_ex2(........., params)

Thanks t8m,
I have subscribed (twice) at the URL you posted but I've got no response and cannot post.  Maybe I just need to wait longer. Thanks very much for your help, I'll give it a go.
Paul

Unfortunately I still get a different result.  Here's the test code "as is " with the key length parameter included.
	const int iOutputLen = 512;
	unsigned char* pOutput = new unsigned char[iOutputLen];
	memset(pOutput, 0, iOutputLen);
	const char* pInput = "The quick brown fox jumped over the lazy frog";
	const int iInputLen = (int)strlen(pInput);

#define TEST_RC4
#ifdef TEST_RC4
	{
		const char* pKey = "Some arbitrary passphrase string";
		int iKeyLen = (int)strlen(pKey);
		int iOutLen(0), iTmpLen(0);

		// OLD
		RC4_KEY oRC4Key;
		RC4_set_key(&oRC4Key, iKeyLen, (const unsigned char*)pKey);
		RC4(&oRC4Key, iInputLen, (const unsigned char*)pInput, pOutput);
		printf("(OLD) RC4 calculation\n");
		printf_array(pOutput, iInputLen);

		// NEW
		memset(pOutput, 0, iOutputLen);
		EVP_CIPHER_CTX* pRC4Ctx = EVP_CIPHER_CTX_new();
		if (!pRC4Ctx)
		{
			printf("EVP_CIPHER_CTX_new() Failed\n");
			return 1;
		}

		EVP_CIPHER* pRC4 = EVP_CIPHER_fetch(NULL, "RC4", NULL);
		if (!pRC4)
		{
			printf("EVP_CIPHER_fetch() Failed\n");
			return 1;
		}

		size_t iKeyLenParam = (strlen(pKey) * 8);   // I have tried this with both byte and bit length - neither work
		OSSL_PARAM oParams[2] =
		{
			OSSL_PARAM_END, OSSL_PARAM_END
		};
		oParams[0] = OSSL_PARAM_construct_size_t(OSSL_CIPHER_PARAM_KEYLEN, &iKeyLenParam);

		if (!::EVP_EncryptInit_ex2(pRC4Ctx, pRC4, (const unsigned char*)pKey, NULL, oParams))
		{
			printf("EVP_EncryptInit_ex2() Failed\n");
			return 1;
		}

		if (!::EVP_EncryptUpdate(pRC4Ctx, pOutput, &iOutLen, (const unsigned char*)pInput, iInputLen))
		{
			printf("EVP_EncryptUpdate() Failed\n");
			return 1;
		}

		::EVP_EncryptFinal_ex(pRC4Ctx, pOutput + iOutLen, &iTmpLen);
		::EVP_CIPHER_free(pRC4);
		::EVP_CIPHER_CTX_free(pRC4Ctx);
		printf("(NEW) RC4 calculation\n");
		printf_array(pOutput, iOutLen + iTmpLen);
	}
#endif

The printf_array() function simply outputs the supplied buffer as a hex string to the console.  I'd expect the results to be the same but I get:
(OLD) RC4 calculation
4c7352e08f666d73502b513e5556c216d657f74a9d030ea3bb79540051c796ac2334cbabaf3e7eb0ca5131c053
(NEW) RC4 calculation
6688766fb4b0d5242e076989d87fe6eace96b1ab862120daaeb3a652da840fcb4c47cda1c5804bc74d7b3fe37e

I can't see any mistakes so any further thoughts much appreciated.
Paul

Thanks t8m, I have subscribed (twice) at the URL you posted but I've got no response and cannot post. Maybe I just need to wait longer. Thanks very much for your help, I'll give it a go. Paul

Hmm now I tried with gmail e-mail account and the confirmation e-mail ended up in the spam folder. We need to investigate that.

That's weird, you should receive the e-mail with confirmation link in a few minutes at the latest, I've verified that. Perhaps it is dropped by some spam filtering for you or something like that?

I've looked everywhere and can't find an e-mail.  I got one saying I needed to subscribe though (having already done so) but nothing else.

I have a sneaking suspicion that we don't quite treat the params right.  I made a small program from the code example above that I currently run through a debugger, and the keylen given through the OSSL_CIPHER_PARAM_KEYLEN doesn't quite catch on.  It should be 32, but what ends up being passed to the legacy provider's RC4_set_key() call is the key length 16...
I'll get back when I can say something more definitive.

Found it.
https://github.com/openssl/openssl/blob/04e0abc8bb1c24534d16cc930b611ac1d03bc9bf/crypto/evp/evp_enc.c#L237-L244
Line 239-240 passes the default key length (which happens to be 16 a.k.a. 128 bits), which rc4_einit() assumes to be the length of the passed key, no matter what key length is given through the passed params.
It's a bit of a catch-22.
The workaround is a two initialization steps (and oh, the key length should be given in bytes, not bits):
-		size_t iKeyLenParam = (strlen(pKey) * 8);   // I have tried this with both byte and bit length - neither work
+		size_t iKeyLenParam = strlen(pKey);
 		OSSL_PARAM oParams[2] =
 		{
 			OSSL_PARAM_END, OSSL_PARAM_END
 		};
 		oParams[0] = OSSL_PARAM_construct_size_t(OSSL_CIPHER_PARAM_KEYLEN, &iKeyLenParam);
+
+		if (!::EVP_EncryptInit_ex2(pRC4Ctx, pRC4, NULL, NULL, oParams))
+		{
+			printf("EVP_EncryptInit_ex2() with oParams Failed\n");
+			return 1;
+		}

-		if (!::EVP_EncryptInit_ex2(pRC4Ctx, pRC4, (const unsigned char*)pKey, NULL, oParams))
+		if (!::EVP_EncryptInit_ex2(pRC4Ctx, NULL, (const unsigned char*)pKey, NULL, NULL))
 		{
 			printf("EVP_EncryptInit_ex2() Failed\n");
 			return 1;
 		}

Note that this sort of multi-step init isn't new, the Cipher and Digest init functions have been designed for that sort of thing since way back.

Note that this sort of multi-step init isn't new, the Cipher and Digest init functions have been designed for that sort of thing since way back.

Now you mention it I have seen that in the documentation somewhere.  Thanks very much for the help.

Yep, the two stage initialization identified by Richard Levitte does indeed work.


Add Http Service Resource

I tried my best to understand the structure of the project and I successfully add a http service reference and get it working but
it needs clean-up. Also, I'm not sure if I use the correct annotation.
I have couple of questions:

What annotation should I use to expose the http service?
What manifest should look like for http service?
Should we able to determine the http version of the service?
Is there any public API that I should Add?
Which public API should I use for the sample and where to show it in UI?

I will add the tests once the implementation gets approved.
Fixes https://github.com/dotnet/aspire/issues/775

@DamianEdwards @davidfowl Since this is planned to be in Preview2, could I have this PR reviewed please, I have free time on Thursday and Friday and I would like to finish this issue in time.

Can you generate the manifest for the app you modified (we'll likely undo that change before this gets merged).

Looking at this PR I wonder why we even need HttpServiceResource. Maybe WithReference taking a URI is all we need. In the manifest, we would just hard code the url. Thoughts?
cc @mitchdenny @DamianEdwards

Let’s treat this like it’s not a custom resource at all. It’s just a way to reference and environment variable in the format service discovery expects

Let’s treat this like it’s not a custom resource at all. It’s just a way to reference and environment variable in the format service discovery expects

Do you mean like this?
builder.AddProject<Projects.MyFrontend>("frontend")
    .WithReference("petstore", new Uri("https://petstore.swagger.io"));

Let’s treat this like it’s not a custom resource at all. It’s just a way to reference and environment variable in the format service discovery expects

Not sure about this yet. This would introduce overloads of WithReference that take primitives other than other resources, yes? I'm a bit concerned that without using a resource type it will muddy the line of when something should be a resource or not. E.g. how would we represent the different between a reference to an HTTP service vs. a gRPC service, which likely require different configuration by the tool reading the manifest?

Undo the sample changes and this will be good to go.

Merged. @davidfowl do you want to take this in preview 2?

Yep


Add missing dependencies in package.xml

robot_state_publisher and joint_state_publisher are required for spawn_turtlebot.launch, adding them into package.xml
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Following turtlebot3 convention/documentation, we should stick with turtlebot3_simulation

Let's bump the package's patch version with this dependency change:
  <version>1.1.1</version>


Support for the "link" protocol

What happened?
Using<EMAIL_ADDRESS>with the following package.json:
{
  ...
  "dependencies": {
    "app": "link:src",
  }
}

and running yarn rnx-deps-check gives the following error:
error package.json: ENOENT: no such file or directory, open '/.../app/src/package.json'

Affected Package
@rnx-kit/dep-check
Version
1.12.18
Which platforms are you seeing this issue on?

[ ] Android
[ ] iOS
[X] macOS
[ ] Windows

System Information
System:
    OS: macOS 12.3.1
    CPU: (10) arm64 Apple M1 Max
    Memory: 36.46 GB / 64.00 GB
    Shell: 5.8 - /bin/zsh
  Binaries:
    Node: 18.4.0 - /private/var/folders/xt/bb9ddqg51_jfz2rmh0cvzvzm0000gn/T/xfs-4bf73da4/node
    Yarn: 3.2.1 - /private/var/folders/xt/bb9ddqg51_jfz2rmh0cvzvzm0000gn/T/xfs-4bf73da4/yarn
    npm: 8.12.1 - /opt/homebrew/bin/npm
    Watchman: 2<IP_ADDRESS> - /opt/homebrew/bin/watchman
  Managers:
    CocoaPods: 1.11.3 - /opt/homebrew/bin/pod
  SDKs:
    iOS SDK:
      Platforms: DriverKit 21.4, iOS 15.5, macOS 12.3, tvOS 15.4, watchOS 8.5
    Android SDK: Not Found
  IDEs:
    Android Studio: 2021.1 AI-211.7628.21.2111.8309675
    Xcode: 13.4.1/13F100 - /usr/bin/xcodebuild
  Languages:
    Java: 11.0.15 - /usr/bin/javac
  npmPackages:
    @react-native-community/cli: Not Found
    react: Not Found
    react-native: Not Found
    react-native-macos: Not Found
  npmGlobalPackages:
    *react-native*: Not Found

Steps to Reproduce
In any project that uses yarn@2+, add any local dependency with the link protocol and run the deps command.
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Hi @SchlagerKhan, I am unable to reproduce your issue. This is what I did in a separate repo of mine:

Add "@react-native-async-storage/async-storage": "link:../../async-storage"
Run yarn
Run yarn rnx-dep-check

It's only when I add /src to the path above that I got the error you hit. Do you have a minimal repro of this somewhere so I can see the issue for myself? Frankly, it doesn't seem like a bug with dep-check from first impression.


simplify tests

Codecov Report

Merging #339 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master    #339   +/-   ##
======================================
  Coverage    99.2%   99.2%
======================================
  Files          11      11
  Lines         884     884
  Branches       93      93
======================================
  Hits          877     877
  Misses          7       7

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 3e1278f...cb7371c. Read the comment docs.

Thanks for your contribution!


Add pip to conda environment schema

Environment data

VS Code version: 1.25.0
Extension version (available under the Extensions sidebar): 2018.6.0
OS and version: OS X 10.13.5
Python version (& distribution if applicable, e.g. Anaconda): 3.6.6
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): conda
Relevant/affected Python packages and their versions: N/A

Actual behavior
A conda environment.yml file that includes a "pip" definition under "dependencies" does not validate in VSC.  The error is Incorrect type. Expected "string".
Expected  behavior
A conda environment.yml file that includes a "pip" definition under "dependencies" should be valid and not throw errors in VSC.
Steps to reproduce:

Add an environment.yml file to a project, and include a "pip" subsection under "dependencies".

Logs
Output for Python in the Output panel (View→Output, change the drop-down the upper-right of the Output panel to Python)

Output from Console under the Developer Tools panel (toggle Developer Tools on under Help)

An example of a environment.yml file that includes such a definition can be found at https://conda.io/docs/user-guide/tasks/manage-environments.html#create-env-file-manually


pull request for e2e test at 11/12/2016 19:15:10

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

optest/docs/updated_at.md
:white_check_mark:Succeeded

Details

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.


Added Reveal Relative & Coordinator Layout

Hi, sorry i can't accept it. CoordinatorLayout is an external dependency => i added simple API to create custom views


handler: vod-decryption: added public key endpoint & file decryption

Would be good to have a review from @mjh1 as well if you have a chance


Signature Verification Failed

Error Code: 110775B5D101FB36BC6C911BEB774491D9FF06E2: Key expired

Error Code: 110775B5D101FB36BC6C911BEB774491D9FF06E2: Key expired

@LAL1970

Identical issue as #485
Please close this issue.


[bug] Conflict between phantom Wallet and core wallet

Is there an existing issue for this?

[X] I have searched the existing issues

RainbowKit Version
2.1.2
wagmi Version
2.9.5
Current Behavior
when install phantom wallet extension and core wallet extension, rainbow-kit can't detect phantom wallet
Expected Behavior
rainbow-kit can detect phantom wallet
Steps To Reproduce

install phantom wallet extension
install core wallet extension

Link to Minimal Reproducible Example (CodeSandbox, StackBlitz, etc.)
No response
Anything else?
No response

@tronghoangvmo We're working on a fix. Apparently they didn't follow the right approach with EIP-6963 icon property.

@magiziz pls build new version with this bug, we need golive our project with bold phantom wallet and core wallet


First import fails

First import of this package off a clean import fails due to not all dependencies being listed in https://github.com/nespinoza/juliet/blob/763f49c495b36d78e7bb1ae75976a5707287f265/setup.py#L21
From a clean docker container,
$ pip install juliet

>>> import juliet
[...]
ModuleNotFoundError: No module named 'h5py'

after manually installing h5py import works fine. It seems like adding h5py to the install_requires would fix the pip installation

Hi @mileslucas,
Thanks a lot for letting us know about this. It turns out h5py is not really a dependency of juliet, but of radvel (which juliet uses). Recently, radvel removed this from their list of dependencies (see: https://github.com/California-Planet-Search/radvel/commit/117d4e6946c73a9b739108da64a891732f9c669b) so perhaps this is causing the issue.
Could you confirm that installing radvel in a clean import reproduces this error? You would follow the same steps (pip install radevel, then import radvel). If that is the case, we might want to reach out to the radvel developers --- perhaps @bjfultn has some input here as well.
Thanks!
Néstor

thanks for looking into this, it does appear to be isolated to radvel. I'll close this issue and open one over there!


Secondary

Description
Documentation for all the backend is done from my side.

Merging since there seem to be no conflicts