google colab

Please help me how to change inference.py to see images in colab?
I get the following error:
_qt.qpa.xcb: could not connect to display
qt.qpa.plugin: Could not load the Qt platform plugin "xcb" in "/usr/local/lib/python3.7/site-packages/cv2/qt/plugins" even though it was found.
This application failed to start because no Qt platform plugin could be initialized. Reinstalling the application may fix this problem.
Available platform plugins are: xcb._

Please try to re-install the opencv-python package.

I did a lot of searching and a lot of people said that qt could not be used in google colab.
((
"From the error message, it's because LabelImg use QT.
QT is a GUI library that can run only on your local machine. You cannot run it in the cloud and then let it open a QT window on your machine.
You may need to find another labeling library that use web-interface instead."
))
Now what do you think I should do to get an answer and be able to execute the following command?

thanks for answer.

command :
!python tools/test.py configs/zsi/48_17/test/zsi/zero-shot-mask-rcnn-BARPN-bbox_mask_sync_bg_decoder.py ./checkpoints/ZSI_48_17.pth --show

Since the visualization is based on mmcv (opencv), which uses qt for show the image. If qt is not supported in google colab, you should modify the visualization code in here and use other tools that do not rely on Qt.


Add database configuration documentation

Adds some documentation related to database configuration.
Addresses #367

This looks great! I left some minor feedback, I'll merge this in once those are fixed. Thank you for opening this!

No problem, I’ll fix them up today.
On Wed, 2 Oct 2019, at 01:25, Loïc Sharma wrote:

This looks great! I left some minor feedback, I'll merge this in once those are fixed. Thank you for opening this!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/loic-sharma/BaGet/pull/368?email_source=notifications&email_token=AAFW24O575GLZDCN5QNZYOLQMPS65A5CNFSM4I4IRT7KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEADE5RY#issuecomment-537284295, or mute the thread https://github.com/notifications/unsubscribe-auth/AAFW24MAWQOWBLCS72B6WNDQMPS65ANCNFSM4I4IRT7A.

Great, thank you! I'll get these deployed to the docs website 😄


fix(vue-app): don't attach catch handler to already loaded component (fixes #5751)

Types of changes

[x] Bug fix (a non-breaking change which fixes an issue)
[ ] New feature (a non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Description
Components are not necessarily lazy-loaded. If user overrides webpack
build options with splitChunks:{layouts: false, pages: false}, for
example, pages will be loaded already on loading app.
I was thinking of handling this in the getPrefetchComponents function already, to not return components that are already loaded, but I don't see a way to find that out until we call Component() and that would trigger prefetching in case of lazy-loaded components.
Resolves: #5751
Checklist:

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly. (PR: #)
[ ] I have added tests to cover my changes (if not applicable, please state why)
[x] All new and existing tests are passing.

Codecov Report

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

@@          Coverage Diff          @@
##             dev   #5752   +/-   ##
=====================================
  Coverage   95.6%   95.6%
=====================================
  Files         82      82
  Lines       2662    2662
  Branches     683     683
=====================================
  Hits        2545    2545
  Misses        98      98
  Partials      19      19

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 f391e5d...9c5db94. Read the comment docs.


Build Error -

╷
Error: [0m Error: [ERROR] Error retrieving container vpc cluster: Request failed with status code: 404, ServerErrorResponse: {"incidentID":"314617be-0c97-4bf4-a76c-15a376ba0085","code":"G0004","description":"The specified cluster could not be found. If applicable, make sure that you target the correct account and resource group.","type":"General","recoveryCLI":"To list the clusters you have access to, run 'ibmcloud ks cluster ls'. To list the resource groups that you have access to, run 'ibmcloud resource groups'. To target the resource group, run 'ibmcloud target -g \u003cresource_group\u003e'."}
│
│   with module.dev_cluster.data.ibm_container_vpc_cluster.config,
│   on .terraform/modules/dev_cluster/main.tf line 238, in data "ibm_container_vpc_cluster" "config":
│  238: data ibm_container_vpc_cluster config {
│


Tidied, so unambiguous with quickstart plugin

When recent-cards plugin and quickstart plugin installed, both will appear in ACP header menu with the same name.
This is confusing, therefore it needs to be tidied.

Thanks :) fixed in a different PR


Split up RapierContext

The RapierContext resource is massive. It has the island manager, broad phase, narrow phase, CCD solver, physics pipeline, query pipeline, integration parameters, rigid bodies, colliders, and joints. Quoting the docs: "The Rapier context, containing all the state of the physics engine." It is also what is used to step the simulation, perform scene queries, query for collision pairs, and control character controllers.
This kind of "god struct" feels almost antithetical to an ECS. Almost everything is handled by one struct with dozens of unrelated responsibilities. This is unidiomatic, bad for composability and modularity, bad for documentation, and bad from a UX standpoint.
What solution would you like?
RapierContext should be split up into several resources. For methods that require access to multiple different resources, system parameters can be used.
For example, the query pipeline could be turned into its own resource. It could be renamed to SceneQuery, or there could be a new system parameter for it, like bevy_xpbd's SpatialQuery. If it was its own system parameter, ray casting could look like this:
fn cast_ray(scene_query: SceneQuery) {
    let ray_pos = Vec3::new(1.0, 2.0, 3.0);
    let ray_dir = Vec3::new(0.0, 1.0, 0.0);
    let max_toi = 4.0;
    let solid = true;
    let filter = QueryFilter::default();

    if let Some((entity, toi)) = scene_query.cast_ray(
        ray_pos, ray_dir, max_toi, solid, filter
    ) {
        let hit_point = ray_pos + ray_dir * toi;
        println!("Entity {:?} hit at point {}", entity, hit_point);
    }
}

I find that IntegrationParameters also doesn't really make sense in RapierContext. It could either be in RapierConfiguration where other global configuration resides, or be its own resource.
Some more internal structures like the island manager, rigid body sets, collider sets, and so on might be fine in the same resource, or split into separate resources.
Plugin architecture?
Personally, I would also like to see Rapier try a more plugin-oriented architecture. It would allow for some nice flexibility and composability, making it more straightforward for third-party alternatives to be used if a specific first-party solution is insufficient, while also providing a logical place for docs.rs documentation. For example, collision detection could be handled by one plugin, and the solver by another. bevy_xpbd has a very plugin-heavy architecture, but even just a few plugins would help break up the monolithic structure.
Of course, this would probably require changes to how the simulation loop in bevy_rapier works, but I don't see why it wouldn't inherently be doable.

https://github.com/dimforge/bevy_rapier/pull/328 suggests taking the route of physics World as entities, that's the best option in my opinion too, which plays well with this PR: add more or split components to customize a physics world.


update rdkit install instructions

https://github.com/rdkit/conda-rdkit/issues/84

Solved by #325.


Possible bug in #unsubscribeFor

Please see this thread for reference: https://github.com/postaljs/postal.js/issues/61.
Thank you for adding unsubscribeFor at my request.  It would seem, though, that I have encountered a bug.  If the predicate against a SubscriptionDefinition is a channel and custom property, unsubscribeFor doesn't always seem to actually unsubscribe.
Consider this code:
/**
 * @method unsubscribeTransients
 * Unsubscribe all transient subscriptions registered with the message bus on the outlet message channel.
 */
Datepicker.prototype.unsubscribeTransients = function () {
    postal.unsubscribeFor({
            channel: this.calendarNavigator.outletMessageChannel,
            residence: this.kTransient
        });
    };

where residence is a custom property added using UnderscoreJS's #extend method on the SubscriptionDefinition.
If I comment out the following line
channel: this.calendarNavigator.outletMessageChannel,

in the predicate, it works fine.
NOT SO FAST!
In the one module where I've used unsubscribeFor, the subscriptions are not actually released from memory (and, consequently, not garbage-collected).
I don't have this problem, though, when I use my previous technique of pushing the subscriptions into an array.  This works great:
/*global define*/
define('mixins/mixin.unregistersubscriptions',
    [
        'underscore'
    ],
    function (
        _
        ) {
        return {
            unregisterSubscriptions: function (subscriptionsArray) {
                var subs = subscriptionsArray || this.subscriptions;

                if (!_.isArray(subs)) {
                    return;
                }

                _.each(subs, function (sub) {
                    sub.unsubscribe();
                    sub.callback = null;
                });

                subs = null;
            }
        };
    });

Just checking in - still having issues here?


Add reporter for Automake

Description
Added a reporter to generate the :test-result: lines expected by Automake's default log compiler.  This will allow projects using Automake to integrate Catch tests.  Users of Automake will need to use an appropriate LOG_DRIVER.  A prototype of the  log driver would be something like
catch-driver.sh.txt

We don't want to add more reporters to the single-include header, so I am going to fix-up the PR a bit.
Otherwise looks pretty good.


Address doc feedback and add information about WebView2 Runtime regkey on 32bit Windows

Small change to address doc issue https://github.com/MicrosoftDocs/edge-developer/issues/974. Looking to merge soon.

Docs Build status updates of commit c29517f:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/webview2/concepts/distribution.md
:bulb:Suggestion
View
Details

microsoft-edge/webview2/index.md
:white_check_mark:Succeeded
View

microsoft-edge/webview2/concepts/distribution.md

Line 67, Column 1: [Suggestion-code-block-indented] Indented code blocks aren't allowed. Use a Markdown code block surrounded by triple backticks (```). NOTE: This Suggestion will become a Warning on 1/29/21.
Line 73, Column 1: [Suggestion-code-block-indented] Indented code blocks aren't allowed. Use a Markdown code block surrounded by triple backticks (```). NOTE: This Suggestion will become a Warning on 1/29/21.
Line 100, Column 1: [Suggestion-code-block-indented] Indented code blocks aren't allowed. Use a Markdown code block surrounded by triple backticks (```). NOTE: This Suggestion will become a Warning on 1/29/21.
Line 106, Column 1: [Suggestion-code-block-indented] Indented code blocks aren't allowed. Use a Markdown code block surrounded by triple backticks (```). NOTE: This Suggestion will become a Warning on 1/29/21.

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Create devcontainer.json

it

@sawyeriii Thanks so much for the contribution but I don't think this is a great time for us to merge this PR.
For one, to add .devcontainer support for devs, we would need to update the contributors guide, or new markdown file linked from the contributors guide and/or FAQ.
Two, we don't have time to prioritize ensuring this developer workflow, and documentation around it, work sufficiently.
If you're experiencing issues setting up the project with the existing documentation, we would be happy to help you once we receive an issue targeting the specific problem you're running into.


Annotation of currencies

Even in scientific papers, we may stumble upon expressions of amounts of money (150$, 3€).
Do we annotate $, €, etc?

Honestly not a clue... we could annotate them with type CURRENCY perhaps and have the automatic transformation based on the exchange rate, but I'm not sure we have done it before.
@kermitt2 what do you think?

yes it makes definitively sense to annotate them with the currency as unit and using a type CURRENCY. So far I didn't see such expression of money in the first batch of training data.

here is an example from halshs-01279855.training.tei.xml:
the French professional football league broadcasting rights over <measure type="value">
<num>one</num></measure> season (<measure type="value"><measure type="CURRENCY"
unit="€">€</measure> <num>668 million</num></measure>)

unit="€" I would write something like unit="euro" instead


Caching response after the download complete

Hi,
First of all thanks for your library, is really useful and easy to use!
We have discovered that when using the URL initializer the response was never cached and the video was downloaded again each time.
This is probably caused by an Apple Bug referenced here: https://forums.developer.apple.com/thread/92119
This Bug says that URLCache, that is the object responsible for cache the URLSession responses, does not work with range requests that are the one used to reproduce videos.
We've modified your library so the response id cached manually after the download has completed, in this way for the URLCache it is a normal response and the next time the video is played it will not be downloaded again.
We had also modified the initializer so it is possible to provide a custom URLSessionConfiguration that is the object that contains the URLCache used, so it is possible to provide an URLCache with custom sizes for example.
Thank You!
Gruppio

Hey,
I'm glad that it's been useful to you.
CachingPlayerItem doesn't perform caching by itself, instead, it delegates this responsibility outside. It's up to a user how they would store loaded data and I think this should stay this way mainly because it's simpler and more flexible. Therefore I'm not sure that an additional caching layer in the form of the URLCache is something that should be added. But I'm open to discussion.

Hi,
The modification will not cache everything, it will just honour the cache response from the server.
If the server responds to do not cache the response, the URLCache will not cache it.
If the server say to cache the video for one hour the URLCache will do that.
Instead without this modification if the server say to cache a video for one hour nothing was happening.
This behaviour is the default for all the data task performed with a URLSession, but because of the bug linked above this was not true for the range request used for the videos.
The Cache rules are fully customizable anyway from the URLSessionConfiguration that is provided in the initializer  :-)

I see your point.
I assume that developers that decide to go with CachingPlayerItem generally want to store the downloaded data somewhere on disk anyway (otherwise they would use AVPlayerItem, instead), so URLCaching will perform  unnecessary additional caching in this case.

I think, that it may help you https://andreygordeev.com/2018/03/31/cache-avplayeritem/

I really agree with @neekeetab that CachingPlayerItem is a fantastic class that should stay SIMPLE.  It's amazing like this!

It would be great to see some sample code that just shows how to save to disk

There is this example, andreygordeev.com/2018/03/31/cache-avplayeritem/ but unfortunately it uses an additional library ("Cache") to do the caching...

As I say, it would be great to have a simple example somewhere of how to just cache to disk simply with no external library!  Cheers
Again this is an awesome class just as it is, thanks!


Wrong version in npmjs.com repository

Hello,
could you please fix version of react-slider in npmjs.com repository?
Current version published there is tagged as 0.7.0 but contains code of older version.

Released new version 0.8.0.


Exception while trying examples

Hi,
I'm trying to run the convnet_mnist.py example and I getting an error:
INFO     SGD: Model contains 127242 parameters.
INFO     SGD: 469 mini-batch gradient updates per epoch.
terminate called after throwing an instance of 'std::runtime_error'
  what():  src/image/img2win.cu:56: invalid device function
[1]    10517 abort (core dumped)  python convnet_mnist.py

I'm not sure this is a problem with deeppy itselft. It's probably a problem with my cuda installation since I had similar problems trying to run caffe. But as I have no experience with cuda at all, do you guys have any pointers of what might be the issue?
I'll totally understand if you just say "it's a cuda problem, solve it with the cuda guys" , but it also might be something you encountered so...

Closed due to inactivity. Feel free to reopen.


Fix use of undefined member m_engine_size (#513).

Fix https://github.com/ROCm/rocRAND/issues/513 .

Looks good to me, waiting for CI checks to pass.

@stanleytsang-amd    can this be landed ?

All tests pass, looks good to me.  Merging.


How to get widget.SendBird.js file in my react app.

In implementing web-widget example in react I am facing a problem . How can I get  widget.SendBird.js in my react app. If I just copy the file then I'm getting a compilation error, in react

Hello @yaakuza . The web-widget sample was built with Vanilla JS and wasn't designed for React. That said, you may be able to wrap it into a React component for usage.


[Feature]: Link that renders a component instead of HTMLAnchor

What is the new or updated feature that you are suggesting?
Right now navigation requires to use anchors with <Link /> or <NavLink />, for some use cases, though, we have interactive UIs where a click on a component triggers navigation instead of an anchor.
Why should this feature be included?
We can currently use useNavigate hook for those scenarios, but we loose the ability to prefetch without adding more code. An exposed component that renders children and handles navigation + prefetch would simplify a lot the usage.
interface ComponentLinkProps {
    to: string;
    prefetch?: PrefetchBehavior;
}

const ComponentLink: React.FC<ComponentLinkProps> = ({ to, prefetch = "none", children, ...props }) => {
    const navigate = useNavigate();
    const href = useHref(to);
    const [shouldPrefetch, prefetchHandlers] = usePrefetchBehavior(prefetch, props);

    return (
        <>
            <span {...prefetchHandlers} {...props} onClick={() => navigate(href)} role="link">
                {children}
            </span>
            {shouldPrefetch ? <PrefetchPageLinks page={href} /> : null}
        </>
    );
};

Navigating with anchor elements is what HTML does. You can wrap anything in an anchor element, or make them look like whatever you would want. Do you have an example, where an interactive UI is not possible with the Link or NavLink component?

And there is the example. #1609
Sorry, that wasn't clear to me.

The correct option is to wrap that component in an anchor tag so it’s clickable
Navigation should be triggered by a link click or form submit only.


Make autocompletion of Markdown files subfolder aware

It would be great if (by config option or default) make the autocompletion aware of markdownfiles in subfolders

Agreed! I hope to get to it in the next couple of days.

@woodworker I have just released v0.0.4 that should fix this issue. Please give it a try and let me know if you run into any other issues!


Minor typo fix for French documentation

Really small fixes 😉

@yannickl can you give this a really quick look and approve? 👀

@Pierstoval while you are here, can you update the intro badges, just like someone did here: https://github.com/carbon-app/carbon/pull/917

Yeah sure 👍


Imported TypeScript enum is undefined in Webpack module object

Bug report
webpack version:  5.74.0
Node.js version: 14.15.1
Operating System: Windows 10 Pro

What is the current behavior?
Exported enums are undefined at runtime.

util_types__WEBPACK_IMPORTED_MODULE_0__ contains no property named SessionViewType. This enum is exported from util.ts and imported into a Vue SFC using import { SessionViewType } from "util/types". If import explicitly includes .ts extension, the issue is resolved. This also affects third-party packages.
If the current behavior is a bug, please provide the steps to reproduce.

What is the expected behavior?
Enums should be present Webpack module objects.

Other relevant information:
Webpack configuration:
const { resolve } = require('path');
const { VueLoaderPlugin } = require('vue-loader');
const HtmlWebpackPlugin = require('html-webpack-plugin');
const polyfillPlugin = require("node-polyfill-webpack-plugin");

module.exports = {
    entry: './src/renderer/entry-point.ts',
    target: "electron-renderer",
    output: {
        path: resolve(__dirname, 'dist', 'renderer'),
        filename: 'renderer.js'
    },
    resolve: {
        alias: {
            css: resolve(__dirname, "src/renderer/css"),
            component: resolve(__dirname, "src/renderer/component"),
            img: resolve(__dirname, "src/renderer/img"),
            util: resolve(__dirname, "src/renderer/util")
        },
        extensions: ['.vue', '.ts', '.js']
    },
    module: {
        rules: [
            {
                test: /\.scss$/,
                use: [
                    'vue-style-loader',
                    'css-loader',
                    'sass-loader',
                ]
            },
            {
                test: /\.ts$/,
                exclude: /(node_modules)/,
                use: [
                    {
                        loader: 'ts-loader',
                        options: {
                            transpileOnly: true,
                            appendTsSuffixTo: [/\.vue$/]
                        }
                    }
                ]
            },
            {
                test: /\.vue$/,
                exclude: /(node_modules)/,
                loader: 'vue-loader'
            },
            {
                test: /\.(png|jpe?g|gif|svg)$/,
                type: "asset/resource"
            }
        ]
    },
    plugins: [
        new VueLoaderPlugin(),
        new HtmlWebpackPlugin(),
        new polyfillPlugin({
            includeAliases: ["path"]
        })
    ],
    node: {
        global: true
    }
};

Please you create small reproducible repo.

As I see from your configuration
If import explicitly includes .ts extension, the issue is resolved

you have .js transpiled version as well? sounds like without extension js version is used

@vankop I should be able to get one up by tomorrow. I found that by downgrading to 5.30.0, enums were no longer undefined.

webpack don't touch enums, please check your typescript configuration and version


Remove usage of  jcenter() from buildSrc module

Kind (Required)
Improvement
Overview (Required)
One of the next steps of #338 and #328.
kotlin-dsl is hosted on JCenter.
After it migrates to mavenCentral , we can replace jcenter()  with mavenCentral()
Links

#328
#338

label is welcom_contribute, but we can't solve this issue until kotlin-dsl is hosted on MavenCentral.

Now kotlin-dsl is available on MavenCentral
https://mvnrepository.com/artifact/org.gradle.kotlin.kotlin-dsl/org.gradle.kotlin.kotlin-dsl.gradle.plugin


feat(rxjs.dev): add api list deprecated filter

Description:
Adds a "Show deprecated" checkbox to optionally filter out deprecated items.
BREAKING CHANGE: N/A
Related issue (if exists): N/A

This a cool idea. @niklas-wortmann or @jakovljevic-mladen can you please have a look?


Keep the solved pieces intact when dragging

Hi, I was wondering if there's a way we can lock the pieces when dragging them?
For example, I have three pieces of the puzzle solved. Dragging one of the pieces will be disconnected from the other two.
This doesn't happen when you drag a piece with solved pieces around it.

Hi @hapiben
There is currently no manual lock mode. When pieces are connected, they will work a single unit as long as they are dragged in a direction that is not open, that is, there is no open connector. Otherwise they will get disconnected. As a consequence, user can control whether the puzzle remains locked or not depending on their gestures, instead of a shortcut or another configuration.
This is behavior is quite core to this library and won't probable change.
However it might be augmented by introducing something some kind of keyboard action, like pressing the shift key. Theoretically, the following method...
  drag(dx, dy, quiet = false) {
    if (Pair.isNull(dx, dy)) return;

    if (this.horizontallyOpenMovement(dx) && this.vericallyOpenMovement(dy)) { // here
      this.disconnect();
      this.translate(dx, dy, quiet);
    } else {
      this.push(dx, dy, quiet);
    }
  }

...could be modified to add an additional condition that forces locking. Perhaps something like this:
  drag(dx, dy, quiet = false) {
    //...

    if (!this.puzzle.locked && this.horizontallyOpenMovement(dx) && this.vericallyOpenMovement(dy)) { // here
      this.disconnect();
      this.translate(dx, dy, quiet);
    } else { //...
    }
  }

And then add a lock()/unlock() method pair to Puzzle, which you may control from UI.

I have left some initial thought here: https://github.com/flbulgarelli/headbreaker/tree/experiment-puzzle-locking, but they will require more work. If you think this is what you need, feel free to contribute :smile:

Hi @hapiben I am closing this for now. Feel free to reopen it if you have any other question.


weird behaviour of 'both' with lenses

I found a case where 'both' returns null, instead of a boolean.

const dataLens = lensProp('data')

// authenticated
const authenticatedLens = lensProp('isAuthenticated') // true, false or null because of API design... :(
const isLoggedIn = pipe(view(authenticatedLens), status => status === true)

// account type
const isProLens = lensProp('isPro')

// BUG
const isPro = both(
  view(authenticatedLens),
  view(isProLens)
)

// NO BUG
const isPro = both(
  isLoggedIn,
  view(isProLens)
)

isPro({
  isPro: false,
  isImport: false,
  isFetching: false,
  isAuthenticated: null,
  data: {
    accountType: 1,
    personalData: { firstName: 'firstName', pseudo: 'pseudo' }
  }
})

I think that 'both' should always return a boolean ?

Hi @Powerplex, thanks for bringing this up!
If you take a look at the docs for both, you can see that the function doesn't coerce the results to a boolean, but instead just uses truthy and falsy.
This behavior is expected, and can actually be useful sometimes.

Oh okay sorry. I was confused when I read (… → Boolean) → (… → Boolean) → (*… → Boolean)
As both methods are supposed to return a boolean, I expected a boolean to be enforced. Do you have an example of when this behaviour can be useful :) ?

If you want very long discussions about this, see #1218 and #1335.


Add GALAH dataset

I'd say it's almost done, I'm just not sure how to process resolution. I asked at ANU.

I'm thinking of providing some util functions, e.g. to extract the spectra fluxes corresponding to three arms of the spectrograph - it might be a bit confusing to see a combined dataset like this.

Added filter indices so that it can be viewed per-arm!

GALAH completed fine. Apogee failed in the build test with the outdated configuration. Going forward with merge.


Installing plugins

Not sure what others think, but for me it is hard to imagine a world where I wouldn't want my project's CLI plugin as a global alias. There is already great documentation in the plugin example of configuring the global alias for your project. It even mentions the future possibility of installing plugin aliases.
There are various ways this could play out, but imho it would be great to be bundled in as part of the larger 'install' command of the CLI. When that 'install' command is run, it additionally will check your current directory to see if its a project (or it can find one of its parent directories is one) and suggest (or automatically setup) the plugin alias globally npm install -g ..
If for whatever reason, rolling it into the main install command would prove difficult, we could simply have a separate install-plugin command.

Makes sense to me 👍 I think we'd just need to look at the local package.json and determine if the bin name isn't and-cli to safely install the project globally.

Published with v1.5.0 - let me know if it's all good 👍 I'll add an issue in the plugin example repo to update the documentation surrounding the global alias


fix: Prevent dashboard with filter_values to cause incompatible indicator

SUMMARY

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TEST PLAN

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

cc @john-bodley @suddjian @ktmud

Codecov Report

Merging #12742 (f4df867) into master (9e58eb8) will decrease coverage by 3.36%.
The diff coverage is 55.55%.

@@            Coverage Diff             @@
##           master   #12742      +/-   ##
==========================================
- Coverage   66.73%   63.37%   -3.37%
==========================================
  Files        1021      488     -533
  Lines       49967    30110   -19857
  Branches     4890        0    -4890
==========================================
- Hits        33347    19081   -14266
+ Misses      16491    11029    -5462
+ Partials      129        0     -129

Flag
Coverage Δ

cypress
?

javascript
?

python
63.37% <55.55%> (-0.63%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

superset/viz.py
59.02% <55.55%> (-0.05%)
:arrow_down:

superset/sql_validators/postgres.py
50.00% <0.00%> (-50.00%)
:arrow_down:

superset/views/database/views.py
62.50% <0.00%> (-25.07%)
:arrow_down:

superset/databases/commands/create.py
83.67% <0.00%> (-8.17%)
:arrow_down:

superset/databases/commands/update.py
85.71% <0.00%> (-8.17%)
:arrow_down:

superset/sql_validators/base.py
93.33% <0.00%> (-6.67%)
:arrow_down:

superset/db_engine_specs/base.py
79.59% <0.00%> (-6.38%)
:arrow_down:

superset/db_engine_specs/sqlite.py
90.62% <0.00%> (-6.25%)
:arrow_down:

superset/db_engine_specs/presto.py
82.25% <0.00%> (-5.85%)
:arrow_down:

superset/views/database/forms.py
83.33% <0.00%> (-5.56%)
:arrow_down:

... and 548 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 9e58eb8...f4df867. Read the comment docs.

Codecov Report

Merging #12742 (f4df867) into master (9e58eb8) will decrease coverage by 3.36%.
The diff coverage is 55.55%.

@@            Coverage Diff             @@
##           master   #12742      +/-   ##
==========================================
- Coverage   66.73%   63.37%   -3.37%
==========================================
  Files        1021      488     -533
  Lines       49967    30110   -19857
  Branches     4890        0    -4890
==========================================
- Hits        33347    19081   -14266
+ Misses      16491    11029    -5462
+ Partials      129        0     -129

Flag
Coverage Δ

cypress
?

javascript
?

python
63.37% <55.55%> (-0.63%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

superset/viz.py
59.02% <55.55%> (-0.05%)
:arrow_down:

superset/sql_validators/postgres.py
50.00% <0.00%> (-50.00%)
:arrow_down:

superset/views/database/views.py
62.50% <0.00%> (-25.07%)
:arrow_down:

superset/databases/commands/create.py
83.67% <0.00%> (-8.17%)
:arrow_down:

superset/databases/commands/update.py
85.71% <0.00%> (-8.17%)
:arrow_down:

superset/sql_validators/base.py
93.33% <0.00%> (-6.67%)
:arrow_down:

superset/db_engine_specs/base.py
79.59% <0.00%> (-6.38%)
:arrow_down:

superset/db_engine_specs/sqlite.py
90.62% <0.00%> (-6.25%)
:arrow_down:

superset/db_engine_specs/presto.py
82.25% <0.00%> (-5.85%)
:arrow_down:

superset/views/database/forms.py
83.33% <0.00%> (-5.56%)
:arrow_down:

... and 548 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 9e58eb8...f4df867. Read the comment docs.

Hi

Hi


Cody Gray: Watch pcbmake.com

Cody Gray requests the watch of the watch_keyword pcbmake\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

Approved by tripleee in Charcoal HQ

Approved by tripleee in Charcoal HQ

pcbmake\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

pcbmake\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.


h2o case study

The H2o case study is missing definitions and won't compile.

I have added  required library files. Thanks!


Fixed paypal button callback function

createOrder is expects to return an order ID, not the whole json

#6


Follow Nexus: support for pontusxdev

This adapts our code for the new pontusxdev support on the Nexus API.
No actual work is done to properly support the new pontusx paratime yet,
just patch up our code enough so that there are no more TypeScript syntax errors.
See #1435 for proper support for the new paratime.

Let's merge the API changes together with the proper support... so, closing this in favor of #1435.


Window object

There is this function: window.callPhantom() that could be used as callback Page -> Phantom, but how to call opened page Phantom -> Page??? If inside webpage exist window.pageIsCalled() function, so how can I access global window object from phantom side?

I don't know what you are asking.

I will try to answer.
There is no window.pageIsCalled().  You should send to the callback handler outObject and then poll on state change of some property.

// get window location when window is loaded
let outObj = page.createOutObject();
outObj.objFromPage = {};
await page.property('onCallback', function(objFromPage, outObj) {
   outObj.objFromPage = objFromPage;
}, outObj);

let objFromPage = await new Promise( (resolve, reject)=>{
  let iid = setInterval(async function() {
     let obj = await page.outObj.property('objFromPage');
    if (Object.keys(obj).length !== 0) {
      clearInterval(iid);
      return resolve(obj);
   }
 })
});

page.evaluate( function() {
   window.onload = function() {
      window.callPhantom({ loc: window.location});
   }
});

There is method called callPhantom(). I can call it whenever I want to phantom make me a screenshot.
<!DOCTYPE html>
<html>
   <head>
      <meta charset="UTF-8">
      <title>animation</title>
      <script type="text/javascript">
         function init(){
            setTimeout(function(){
               window.callPhantom({time: Date.now()});
            }, Math.random() * 1000)
         }
         window.playPageAnimation = function(){
            // play canvas animation
         }
      </script>
   </head>
   <body onload="init();" style="background-color:#ff9900;margin:0px;">
      <canvas id="canvas" width="500" height="400"></canvas>
   </body>
</html>

But what if I want to run some animation on that page and only then take a screenshot?
page.on('onCallback', data => {
   window.playPageAnimation();
   page.render(path.join(screenshot_folder, Date.now() + '.png'));
}

I don't think this is a bug. Issues are for bugs or broken functionalities. I honestly don't know how the inner workings of phantomjs in regards to callPhantom so I cannot help.

Issues are for bugs, where expected behavior is different than actual. For questions or best practices please use other sources like stackoverflow.com.
Please review the guidelines for creating issues before opening new issues in the future.
This message is a saved reply and was posted automatically.


macos is an Unrecognized platform

I have a React-Native project and a React-Native-macOS project in my workspace. The issue is that when I compile the macOS project,  the terminal has an output says that
Error: Unrecognized platform: macos
    at DependencyGraph._getRequestPlatform (/Users/username/workspace/theiOSProgram/node_modules/react-native/packager/src/node-haste/index.js:334:13)
    at load.then (/Users/username/workspace/theiOSProgram/node_modules/react-native/packager/src/node-haste/index.js:294:23)
    at process._tickCallback (internal/process/next_tick.js:103:7)
(node:55925) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 5658): Error: Unrecognized platform: macos

Here some useful information about my environment:

macos version: 10.12.6
Xcode version: 8.3.2 (8E2002)
react-native-cli: 2.0.1
react-native-macos-cli: 2.0.1
react-native-macos: 0.16.0

Thanks!

Solved by remove all node_moudles folders and run npm install instead.
I'm afraid it's a yarn issue.


medium/large apps on Windows nearly unusable, finally found reason: Selectors (mostly)

We've deployed nearly identical code on iOS, Android, and Windows UWP systems and across the board the Windows systems are horrendously slow...
Like, 4s to calculate item position slow:

Scouring the internet for the issue I found ALL THESE OTHER ISSUES
And across the board the devs didn't get much of a reply, and in some cases were simply blown off.
But those dev's had the right of it, using more complex apps besides basic tabs/hello world like the iOffice Conference App will show you how its happening.
Why is IE/Edge Slower?
The first most obvious thing is it doesn't support shadow dom and a few other things so it handles tons of DOM elements very very poorly. Instead of only repainting/adjusting the small changes it triggers an entire reflow.
Gregor Srdic Did a nice write up on it here
So first the layout is thrashing hard (thats why it triggers over and over) But then a single layout calculation takes 4S Compared to 20ms or less in chrome. WTF?
Doing more searching I found this article by Alexei Antipov That matches my issue EXACTLY.
Material in Angular 1 Had the same issue
More googling got me to Angular having the SAME ISSUES back in 2015.. Same looking performance charts.
Their solutions? Remove all :active sudo selectors (link)
And go line by line and strip all Attribute selectors
CSS Lint even has a warning for the attributes
Testing The Theory
So the first thing I did was start using Edge to test my build. It seems most devs use Chrome/FF/Not Edge so they haven't seen and dont really care about the issue.
I then ripped apart main.css. I stripped out ALL the iOS related selectors reducing the file by 30%
I then just did a super broad scrubbing of attribute selectors. (1330!)
Then I ran the build:

WHAAT? I got a 92% reduction from 4,000ms to 342ms just by reducing the file size a bit and removing those selectors.
Further refinement of my app by removing other plugins like flexbox has helped some, and of course I re-wrote a LOT of code to hide/ remove if not needed. I've gotten it down to 100-200ms which I think is as best as I'm going to be able to get it.
What's happening
First, because IE doesn't support Shadow Dom it renders the entire body of content on every layout change. Then with it being super slow at attribute & other selectors it runs through the entire matching stack for EVERY ELEMENT. You can see in my first image it takes 1-2ms PER ELEMENT
[tabsLayout=icon-left] .tab-button,
[tabsLayout=icon-right] .tab-button,
[tabsLayout=icon-start] .tab-button,
[tabsLayout=icon-end] .tab-button {
  -webkit-box-orient: horizontal;
  -webkit-box-direction: normal;
  -webkit-flex-direction: row;
  -ms-flex-direction: row;
  flex-direction: row;
}

[tabsLayout=icon-left] .tab-button .tab-button-icon,
[tabsLayout=icon-start] .tab-button .tab-button-icon {
  padding-right: 8px;
  text-align: right;
  text-align: end;
}

[tabsLayout=icon-right] .tab-button .tab-button-icon,
[tabsLayout=icon-end] .tab-button .tab-button-icon {
  padding-left: 8px;
  text-align: left;
  text-align: start;
  -webkit-box-ordinal-group: 11;
  -webkit-order: 10;
  -ms-flex-order: 10;
  order: 10;
}

Bad Selectors
Ok Great you found it, what's the fix?
Well, one way is to do what Material did and simply stop using attribute selectors, start with unqualified ones and try to get rid of them in general with a preference on classes. This core level fix will immediately see significant Windows performance gains.
As thats super unlikely to happen what Im doing is creating a baseIonic.css and using This Article I've changed my SASS to ignore the npm-modules of ionic.. This way my code gets merged without the offending selectors.
I'm went a bit extreme and removing all of them, not just unqualified ones. So I'm doing a version now where I only adjust the unqualified selectors. There's als an INSANE amount of pseudo selectors that I understand run perfectly fine on Chrome, but are killing larger apps.
I don't have a quick fix right away solution, I'm working through it now.. But I wanted to get the update to the issue out there...

This is a great issue, thank you so much for putting together the data! You're right, it comes down to all the attribute selectors. Good news is that we don't have to use data- attributes for scoped styles to work, that was just a choice at the time. But we can update stencil to use css classes instead.
Thanks again for opening the issue! We'll get this fixed up in stencil and then update ionic. Thanks!

Currently fixed in stencil master. The next release of Ionic will no longer be using data- attributes for polyfilled shadow dom. Thank!


New abstract

New abstract highlighting the questionnaire are not merely questions but points to consider.
Same for new introduction, explaining the context, and "how to use".

Okay, wrong files.


Add Github action workflow templates

Codecov Report

Merging #20 (db64b00) into master (96077d4) will increase coverage by 3.01%.
The diff coverage is 90.69%.

@@            Coverage Diff             @@
##           master      #20      +/-   ##
==========================================
+ Coverage   75.90%   78.92%   +3.01%
==========================================
  Files          11       11
  Lines         220      223       +3
  Branches       20       19       -1
==========================================
+ Hits          167      176       +9
+ Misses         44       40       -4
+ Partials        9        7       -2

Impacted Files
Coverage Δ

mloq/assets/templates/setup.py
85.71% <ø> (ø)

setup.py
0.00% <0.00%> (ø)

mloq/templating.py
82.35% <50.00%> (ø)

mloq/workflows.py
84.61% <71.42%> (+7.69%)
:arrow_up:

mloq/directories.py
78.78% <100.00%> (ø)

mloq/files.py
100.00% <100.00%> (ø)

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 4119393...db64b00. Read the comment docs.


Store: Support downloading CVE Database

User should be able to download the CVE database via the store

One of the Docker captains had an attempt here, so there is some example code that could be used https://gitlab.com/almbrand/dtr-cve-file-updater


Potential Instructional Translations/Edits

Hi.  Attached are notes from a native Japanese QA.   I am merely presenting what I received in Japanese, in order to preserve anything that could be potentially lost in translation.  You might want to scan the contents and decide what, if anything, deems merit.
DP-100_QA.docx

Covered in https://dev.azure.com/ceapex/CEINTL/_workitems/edit/823115


[WIP] Add dissasembler to dissassemble a compiled qobj

Summary
This commit adds a new dissassembler module that has a single
dissasemble function. This is used to take an input qobj and return the
circuits, run_config, and user headers which are stored in it.
Details and comments
Fixes #1952

Still WIP until I add tests covering this

The tests will start passing when #2138 is merged


[RC1] navCtrl.pop() from TabsPage

Short description of the problem:
I have a page with no tabs that when you click on a button pushes a tabsPage (with three tabs) to the navigation controller.
I have a goBack function that is simply this.navCtrl.pop() and I invoke from any of the tabbed pages header. Executing the function leads me to a blank screen.
What behavior are you expecting?
In beta 11 clicking on that button would lead me back to the root page (page with no tabs) but in RC1 it leads to a blank screen.
Steps to reproduce:

this.navCtrl.push(TabsPage) from a non-tabbed page.
this.navCtrl.pop() from a tabbed page to go to root non-tabbed page.

Which Ionic Version? 1.x or 2.x
Ionic 2 RC1
Run ionic info from terminal/cmd prompt: (paste output below)

Your system information:
Cordova CLI: You have been opted out of telemetry. To change this, run: cordova telemetry on.
6.3.1
Gulp version:  CLI version 3.8.11
Gulp local:
Ionic Framework Version: 2.0.0-rc.1
Ionic CLI Version: 2.1.1
Ionic App Lib Version: 2.1.1
Ionic App Scripts Version: 0.0.34
ios-deploy version: 1.9.0
ios-sim version: 5.0.8
OS: Mac OS X Sierra
Node Version: v6.6.0
Xcode version: Xcode 8.0 Build version 8A218a

Expected Behavior in Beta11:

Actual Behavior in RC1

Hello, thanks for opening an issue with us! Would you mind posting a repo or gist that I can use to reproduce this issue?

Hi @jgw96. I would be more than happy to post one. Is there an updated plunker/codepen/jsfiddle that I can use with RC1?

@jgw96 I created this ionic2 rc1 app that demonstrates what I mean. https://github.com/delian7/ionic2-starter-app-pop-tabs-bug

@delian7 each tab has his own navigation stack, so when you do this.nav.pop(), you are popping  the root tab page at that leads to a blank page.
You have to get the parent nav controller and pop from there:
this.nav.parent.pop()

@manucorporat @jgw96 thanks for the help.

@delian7  excuse me, i have the same issue that will make the pop() to a incomplete page.
On the tab page i comment out another two tab item.
And in the tab 1 page, i add a component "a" to show the item list,
and in the root page ,i have an another component "b" in tab root page , it can push a new page "B" to add a new item.
In the Page "B" ,i have a plugin "image picker" and a text area, then  the issue is if i click the  text area,
// whatever i type or not
the pop() will return to the Root Page, but nothing component "b" data  was rendering .


merge release 4.3.0 into master

Our CI for Node 16 and 18 for test release is somewhat broken :thinking:

In order to get our integration tests running for newer node versions (16/18) we will have to merge #107 but since this will be breaking for some users we will have to make a new major release for that.

@Uzlopak @HappyZombies @jorenvandeweyer anyone has time for a review?


Test case for #364

I'm not sure I wrote the test correctly.
The problem is that v4.2.4 broke event.data for invoke.onDone: https://codesandbox.io/s/o46qov4mx9

Tthe test I wrote is passing: https://github.com/davidkpiano/xstate/pull/365/files#diff-237806bee3abe58bbe32e2883de20c91R472
But the v4.3.1 build is failing: https://codesandbox.io/s/w0zqnkz5jl
🤷♂


feat(core):  modify severity of rules dynamically at runtime

Summary
modify severity of rules dynamically at runtime
Related Links
Closes #331

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Update iOS RN header paths

https://github.com/facebook/react-native/commit/e1577df1fd70049ce7f288f91f6e2b18d512ff4d moves header imports to <React/*>. This updates header include path and all imports so build works against react master (soon to be version 0.40.0).
TODO: Update examples when RN 0.40 is out.

Thanks for this PR. 👍
I'll get this merged after 0.40.0 is released


Cannot create admin account

Bug Report
Unable to create first account
Description
Bug Summary:
Cannot create admin account
Steps to Reproduce:
Serve the web ui from the backend
Expected Behavior:
First account gets created
Actual Behavior:
No account is created. I queried the ollama.db file and there are no records
In the web interface:
I get the error:
Something went wrong :/
'AuthModel' object has no attribute 'model_dump'
In the terminal, I get:
INFO:     <IP_ADDRESS>:51664 - "POST /auths/signup HTTP/1.1" 500 Internal Server Error
Environment

Ubuntu 22.04
Firefox

Installation Method
Manual install

@willjones1977, how did you solve the issue?

@valerybugakov, You might want to reinstall all the python dependencies!

Huh, it worked! TY
Curious, why did it fail on the first try? Running these two fixed it for me:
pip uninstall -y -r requirements.txt
pip install -r requirements.txt

@valerybugakov, not entirely sure 😅 Glad it's working for you now!

I lost my admin password and now I can't login. I tried to delete the directory and reinstall all over again but it still remember the old admin account. This is on MacBook Pro M3.
Anything suggestion?


Throw exception if Primitive.modelMatrix changes in 2D or Columbus View

Might as well throw if the primitive has more than one instance too.
"If the model matrix is changed after creation, it only affects primitives with one instance and only in 3D mode"

Hi
I would like to fix this up. Can you guide how to proceed forward? Thanks!!

Hi @manujain.  Thanks for the interest.  Please read CONTRIBUTING.md.  Then check out Primitive.js.  In the update function, you'll want to look at this.modelMatrix, this._modelMatrix, and frameState.mode.

@pjcozzi , I think we need to add throw exception statement in "https://github.com/AnalyticalGraphicsInc/cesium/blob/master/Source/Scene/Primitive.js#L1214" . Am I right?

Yes, in that area of the code, but we only throw if (the matrix changed and (the scene mode is in 2D/CV, or the number of instances is greater than one)).  Please include tests.

And update the reference doc.

It is "else if" statement of "https://github.com/AnalyticalGraphicsInc/cesium/blob/master/Source/Scene/Primitive.js#L1220" -
else if (this._numberOfInstances > 1 || (frameState.mode === SceneMode.COLUMBUS_VIEW || frameState.mode === SceneMode.SCENE2D)) {
throw new DeveloperError('If the model matrix is changed after creation, it only affects primitives with one instance and only in 3D mode');
}
I think this would be correct.

Have you set up a development environment (see here)?  Give it a try, write some tests, and see where it takes you.

I am building the environment. Can you assign this issue to me? As soon as I finish building I'll fix this issue. Can you also suggest some other bugs related to the project "Compass / Navigation Widget" og GSoC 2015. Thanks!

Can you assign this issue to me?

Consider it assigned.

Can you also suggest some other bugs related to the project "Compass / Navigation Widget" og GSoC 2015.

@kring is the best person to help here.

@pjcozzi , can you tell how to write tests and update reference doc?

For example tests, see PrimitiveSpec.js.  This is also the file you would add tests to.
The reference doc should be added to Primitive.js where you made the code change.  For example, see Line 707.

@pjcozzi , two tests are not passing -
updates model matrix for more than one instance in 3D with equal model matrices in 3D only scene
does not transform to world coordinates

Do you know how to use the debugger?  Check out https://developer.chrome.com/devtools

@pjcozzi , the above two tests are now passing.
Now all the tests are passing. I want to ask that the tests -
https://github.com/manujain/cesium/blob/master/Specs/Scene/PrimitiveSpec.js#L514
https://github.com/manujain/cesium/blob/master/Specs/Scene/PrimitiveSpec.js#L537
https://github.com/manujain/cesium/blob/master/Specs/Scene/PrimitiveSpec.js#L563
are passing means that update cannot be done in 2D, columbus view. Then how can we write tests to update the matrix in these specific views?
I've updated the reference doc also. After this I'll make a pull request.

@pjcozzi , issue solved, can you take a look at it?

Thanks!  We will take a look soon; we are at a conference this week.

@pjcozzi , can you suggest me some more beginner issues to work on? Thanks!


Rename blog entries as workaround for content types

As reported by issue #8953, we currently cannot properly handle dots in
the file name for blogs. These dots are causing problems in links for
many pages. This PR renames the blog entries to remove dots in file
names.
Closes: #8953

/approve
/lgtm
@tengqm Looks like there's one merge conflict to resolve.

/lgtm
/approve


[StyleSheet] Use Object.assign to flatten styles and add unit tests

The StyleSheet merging algorithm was modeled after Object.assign and the native spread operator. This diff converts flattenStyle to actually use Object.assign.
Test Plan: Added unit tests: npm test -- flattenStyle

I'm torn with this one. I tried to model the styles as close as possible to the spread operator and Object.assign semantics.
var a = {height: 10};
Object.assign(a, {height: undefined});
a // {height: undefined} :(((((

But, at the same time, it makes things more annoying than they have to. For example a common pattern is
var style = [styles.base, this.props.backgroundColor && {backgroundColor: this.props.backgroundColor}];

instead of just
var style = [styles.base, {backgroundColor: this.props.backgroundColor}];

@sebmarkbage what do you think?

I tried to model the styles as close as possible to the spread operator and Object.assign semantics.

That's a really good point. I also feel it's not very good when there's a difference between undefined because a property's value is actually undefined, and undefined because the property does not exist.
My use case is this:
<TabBar style={[defaultTabBarStyle, {
  height: Platform.OS === 'android' ? 56 : undefined, // let iOS keep the default
}]}

If matching Object.assign is really important, I'd also be OK with defining a Symbol sentinel value like this:
{ height: Platform.OS === 'android' ? 56 : StyleSheet.Inherit }

You can write
<TabBar style={[
  defaultTabBarStyle,
  Platform.OS === 'android' && { height: 56 } // let iOS keep the default
]}

The idea is that you should be able to convert all the arrays by spreads.
[a, [b, c]] ~~~ {...a, ...{...b, ...c}}

The actual code is in a StyleSheet.create expression:
let styles = StyleSheet.create({
  tabBar: {
    backgroundColor: '#fff',
    height: Platform.OS === 'android' ? 56 : undefined,
  },
});

I guess I could write this and I sort of like it:
let styles = StyleSheet.create({
  tabBar: {
    backgroundColor: '#fff',
    ...Platform.OS === 'android' ? {
      height: 56,
    }
  },
});

I'll update this diff to use Object.assign and adjust the unit tests. Thanks for the discussion!

I'll update this diff to use Object.assign and adjust the unit tests.

Do you mind not using Object.assign. This is a very perf-critical piece of code and we actually don't get much benefits of using it but this level of indirection is going to hurt performance.

You're right, Object.assign is 75% slower >:(

I've been working an algorithm that tries to avoid flattenStyle and generalizes it so we can put it in core and use it for various renderers. I hit a number of interesting usability and optimization options.
There are a few variants that we could explore.

I can't see the original PR because GitHub. :( Mind putting it back in a separate branch. Want to make sure I don't do the same mistakes.
We still use the modified Object.assign that is the same as mergeFast in prod?

I changed flattenStyle to do this:
  var result = {};
  for (var i = 0; i < style.length; ++i) {
    var computedStyle = flattenStyle(style[i]);
    Object.assign(result, computedStyle);
  }
 return result;

The "75% slower" result was from running a jsperf test on desktop and mobile Safari 9.

cc @vjeux this diff just adds tests now. No new application code.

@facebook-github-bot shipit


JAVA-2110: Add annotation to control the naming strategy

Travis is failing with exception:
  EntityAnnotationTest.should_work_on_nested_class:59 Compilation produced the following errors:
test/Foo.java:7: error: Unexpected error while writing generated code: java.lang.NullPointerException
  public static class Bar {```

Fixed the failing test, thanks.

Just a general comment, the 3.x mapper had a way to set the default naming strategy (among other options) through MappingConfiguration.
I see on the jira ticket you mention:

Alternatively, we could allow the annotation on entities only. If you want to inherit a common configuration, make all your entities inherit from a common class/interface, and put the annotation there.

I agree that having a top-level interface that entities can extend from to inherit common configuration via Annotations is a good way to go.  I always like when there's one way to configure things as it keeps things simple.  Looking at DefaultEntityFactory.getDefinition i see we have a TODO to support inheriting annotations which will be done as part of JAVA-2111


[WebToolsE2E][Aspire] Clicking the "webhooksclient" endpoint and then logging in to order management failed: "invalid_request".

INSTALL STEPS

Clean machine: Win11 x64 23h2 ENU
Install VS 17.9 P1 from build [34310.174.d17.9]
Install latest 8.0 SDK: 8.0.100
Apply NuGet Feeds
Install Docker Desktop

REPRO STEPS

In the VS, clone the eShop repository and open eShop.Web.slnf
F5 / Ctrl-F5, maybe you will encounter this issue #66
Click Endpoint for 'webhooksclient'
Click 'Log in' button

ACTUAL
Sorry, there was an error: invalid_request
Invalid redirect_uri

EXPECTED
Login successful.

Program.cs,
.WithEnvironmentForServiceBinding("WebhooksWebClient", webhooksClient,)
Add  bindingName: "https"
.WithEnvironmentForServiceBinding("WebhooksWebClient", webhooksClient, bindingName: "https")


Correction in README

Sorry, I forgot to correct that

any update ?

Hey! Sorry for the silence, been busy with exams and the like (almost over though). I found that the new preferences items are causing the list to drop lots of frames so haven't merged just yet. I'll look into optimisations before I merge.

I'm going to close this for now, since we need to figure out how to optimise scrolling.


Can't flash custom partition table without factory app

Hello!
Tried to make firmware with ESP-IDF Prerelease v3.1-beta1 and funded that i can't flash firmware in custom partition without "factory" app.
All seems to be ok and flashing started, but when i removed line with factory app definition i have following message:
"esptool write_flash: error: argument  : Detected overlap at address: 0x1000 for file: /Users/thecashit/Documents/LOOK.in/Firmware/Source/build/bootloader/bootloader.bin"
My app have 2 OTA partitions without factory one and with previous versions of IDF it was successfully flashed.

@thecashit
Fixed in https://github.com/espressif/esp-idf/commit/6b5a15e3da93cd4ef94bee60d89c1e254c79bc21

Ok, thnx!


Logic in public getters is actually executed when instantiating the mock

If you have a class as follows
export class TestClass {
    constructor(private dependency: Dependency) { }
    public get myProperty(): string {
        return this.dependency.getVal();
    }
}

when running
let myMock = mock(TestClass);
this.dependency.getVal() will actually get executed, and fail since this.dependency will be undefined.
Occurs when generating method stubs

Thanks for Your report. i will take a look a this asap.

Fixed in https://github.com/NagRock/ts-mockito/releases/tag/v1.1.3
Keep in mind, that it only fixes getter/setter code execution. Functionality of stubbing getter/setter will be added in future release.

Please let me know if release 1.1.3 fixed yours issue.


envoy: Fix NPDS message definitions

Generate protobuf validation code for Cilium's xDS API:

Install protoc-gen-validate into $PATH into Vagrant VM.
Use protoc-gen-validate to generate validation code when generating Cilium's xDS protocol buffers code.

Fix NPDS protocol message validation:

Remove DirectionNetworkPolicy message.
Relax field validation constraints.
Validate every NetworkPolicy message after conversion before sending it to Envoy, to prevent validation errors within Envoy.

test-me-please

@jrajahalme if the tests pass for this PR, we should be in a good shape re: NPDS message validation, as all policies configured in tests will go through translation and validation on the agent side.

test-me-please

Tests are passing: https://jenkins.cilium.io/job/Cilium-PR-Ginkgo-Tests-Validated/952/

test-me-please

test-me-please

test-me-please

test-me-please


Start testing Normalizr

New test/ dir was added, with some basic tests in it.
New tests:

basic tests of URL_REGEX, EMAIL_REGEX.
more tests can be easily add using the same pattern.

It's suggested to start using continuous integration. A basic config file for TravisCI is also included.

Great PR @feinsteinben,
I'll accept it for sure but I'll wait for #8 so normalizr can have support for Python 2. I also have a couple of comments for your other PR that I'll write eventually ^^
Thank you,
David

As I commented in #9 would be nice if you can change the format of your commit messages. Me and my OCD :wink:

Fixed it :-)

Added several more tests. @davidmogar

Hi @davidmogar ,
I found a wrong test-case and fixed it. The fixed commit is in the same branch as the pull-request.
Sorry for the trouble,
Ben.

Yeah, I suppose it's the one I wrote in #16. I fixed on my local branch. I'll push it now. Otherwise you would have to open a new pull request as this one is already closed ;)


Update get-started.md

Issue #522 - Invalid module names are removed from the table now.

@dariomws306 @tseward
Peer review needed for this PR. Thanks.

I think there are some of them that we can find the article. For example, Windows Defender:
https://docs.microsoft.com/powershell/module/defender/
Hardware Certification:
https://docs.microsoft.com/powershell/module/hardwarecertification/
MsDtc:
https://docs.microsoft.com/powershell/module/msdtc/
Please check the others.

Hi @dariomws306
Thank you for the review.
These modules were commented out on the Windows Server 2012 R2. And there is no content available under these modules. And because of the the comments symbols content is not properly aligned.
Kindly suggest me on the next steps. Really appreciate your time on this.
Thank you.

@sowjanya365, MsDTC and SMISConfig have a module for 2012 R2 and 8.1. I think Windows Defender should has one also, but it doesn't, maybe it should be fixed.
I think we should double check each of them.

Hi @dariomws306
Thank you for your assistance.
But I couldn't find the Windows Defender module listed under 2012 R2 and 8.1. And MsDTS and SMISConfig modules are fixed with the respective links. Thanks.

@dariomws306
Any recommendations?

I think it would be good that we check if we can add Windows Defender module listed under 2012 R2 and 8.1, but it should be a different issue.
If @sowjanya365 has double checked that there aren't more missing 2012 R2/8.1 cmdlets list to add I am ok.
Thanks!

Hi @dariomws306
As per my research, defender module is not available on Windows server 2012 R2/8.1. Please refer to the following articles.
https://social.technet.microsoft.com/Forums/lync/en-US/4851f4f8-7f74-4cc4-ae52-de8c9912f3c9/how-do-you-enable-powershell-defender-module-on-server-2012-r2?forum=winserversecurity
https://docs.microsoft.com/en-us/previous-versions/windows/it-pro/windows-8.1-and-8/dn344918(v=ws.11)#BKMK_WindowsDefender
Thanks.

@sowjanya365, it's true it's not available on 2012. But it's available on 8.1.
Maybe we can avoid to add it to the list.

Thank you for the clarification and appreciate your time, @dariomws306
Hi @e0i
Kindly coordinate to get this PR merged. Thank you.

You're welcome @sowjanya365! Thank you for your patience!

The PR is OK by Trevor.

@AndreaBarr The PR can be merged. Thanks.


Create generic Rancher compose template + upgrade for various apps

Generate a Compose template for:

Hadoop

[ ] Install
[ ] Upgrade

Openstack/Swift

[ ] Install
[ ] Upgrade

ELK

[x] Install
[ ] Upgrade

Jenkins

[ ] Install
[ ] Upgrade

Gluster

[ ] Install
[ ] Upgrade

Mesos

[ ] Install
[ ] Upgrade

Swarm

[ ] Install
[ ] Upgrade

Kubernetes

[x] Install
[ ] Upgrade

Show it can be launched with a few config items
Be able to update from one version to another
Goal is to determine what missing features Rancher needs to implement to provide this end to end solution.

+1 . Distributed storage

Hello, any news on this template ?, could there be a skeletal template at least, so that we can testing.

@lavvy we are still in the process of creating the template. Right now its looking like it will start out as HDFS (Single namenode) cluster and Yarn. Does that seem like it would be useful? What are you planning to try and do with it?

@cloudnautique i had just spinned up hadoop for the first time in rancher yesterday and it was pretty simple to kickstart and use . and it ran very smoothly . i could be able to use its web ui (hue).
i used this image https://hub.docker.com/r/caioquirino/docker-cloudera-quickstart/ .
BUT the only issue was how to add more nodes or scale it in rancher .

yes i think it will be very useful . atleast having a distributed file system service with analytics and processing in the environment or infrastructure

+1 . Mysql or mariadb service

I could see multi-master Percona XtraDB clustering as there's work required to get the cluster setup..
What would you want out of a MySQL/Maria template beyond the options in their normal official images?  Master/slave setup?  Auto failover between them?

Yeah just to keep a generic relational database template for rancher that one can create a database and connect to. It must not be sophisticated but it should have replication and failover, I think that would be cool

Still waiting to start testing hadoop in rancher, I came across this   https://github.com/lresende/docker-yarn-cluster
I think it has both namenode and datanodes as containers

@lavvy, I checked in the first pass of the Hadoop cluster and Yarn. Let us know what you think!

Sorry for late response. I gave it a spin yesterday . That was a cool setup. Though I have not been able to test it for long. But I will want to suggest if the set-up could include hue manager . I think it will enable us navigate hadoop ecosystem better, just like what rancher is to docker

Still testing the hadoop stack for rancher. Trying to decommission some datanodes, When you kill a datanode the rancher service recreates it again.

@lavvy, when you say decommission, do you mean you have added to the blacklist and then manually deleted the node? The scale of the service would need to decrease, otherwise Rancher tries to keep it at the same capacity as you observed.
We need to adjust the behavior in Rancher. What would you consider the ideal flow? Mark the node to be decommissioned, wait for draining, then scale down the service removing that node? Or something else?

Yes, you gat the workflow right. But if you can stop the node in rancher and it appears decommissioned, cool too

https://github.com/2mind/Dockerfiles/tree/master/consul - create consul cluster + webui using metadata
currently working on an adaptation of a ceph cluster
rancher great project
Спасибо ;)

Will track catalog item as separate feature requests.


build: remove "Useless option value" settings from pylintrc

PR summary

Remove unsupported pylint options
Add linting step to Jenkinsfile

PR Checklist
Please make sure that your PR fulfills the following requirements:

[x] The commit message follows the
Angular Commit Message Guidelines.
[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

PR Type

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] New tests
[x] Build/CI related changes
[ ] Documentation content changes
[ ] Other (please describe)

What is the current behavior?
Pylint complains about use of options which are no longer supported
What is the new behavior?
Remove these options
Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

see also https://github.ibm.com/cloudant/sdks/pull/705

closing, see changes at https://github.com/IBM/cloudant-python-sdk/pull/467


Update FerretDB app to the latest version

This issue is a:

[ ] Bug Report
[x] Enhancement suggestion
[ ] Feature request

If the issue concerns an existing Marketplace application, please tag the maintainer of the application (found in manifest.yaml) below:
Enhancement suggestions/feature requests
Currently, we have FerretDB version 1.6.0 installed by Civo marketplace as a newer version 1.7.0 is available, so it needs to be updated to the newer version. We kindly request an update to the latest version of FerretDB on the Civo marketplace to ensure that we have access to the most recent features and improvements.
@AlekSi I request you to please take a look into this.

#507 was created before this issue

#540


Get currently open swipe view

Hello. I'm having problems getting information about which bottom view is currently open (left or right). I need this in method onHandRelease.

try   if (swipeLayout.getDragEdge()== SwipeLayout.DragEdge.Left){}


Multiple stats doesn't work

Describe the bug
If pass additional or multiple stats within StatoscopeWebpackPlugin or cli version, only one of the stats is available in the UI.
To Reproduce
Steps to reproduce the behavior:

run npx statoscope generate --input path/to/stats1.json path/to/stats2.json --output path/to/report.html
try to choose stats in the report or show diff
see that only one of the stats is available

Expected behavior
Both stats1.json and stats2.json are available in the report.
Versions:

Node (for non-browser bug): v14.16.0
Browser (for browser-bug): Chrome 91.0.4472.114
@statoscope/cli: 5.6.2
@statoscope/webpack-plugin: 5.6.2

Thanks for the issue.
Are the input files have the same name (but in different directories)?
For example:
statoscope generate --input dir1/stats.json dir2/stats.json --output path/to/report.html

I've got this behavior only when the input files have the same filename 🤔

Nope, files have different names

Can you click the "Make report" button (on the top-right panel) please?
How many files do you see there?
It should be something like this

There are 2 files as expected

There are 2 files as expected

So you should see the "Choose stats" button

and the two stats on the "Diff" page

Nope, while there are two files on the "Make report" page, I can see only one when trying to choose stats:

Wow :)
That's interesting

I think that I need your help to deal with it :)
Can you please insert this query into the Make Report page and send me the result?
compilations.({hash})

hmm...

After I added some changes to my project and ran statoscope, I could see two stats in the report 👍

I think I've got it - the problem appears when you have two same stats, with the same compilation hash
Compilation hash changes only when a bundle changes.
So I think that there must be two stats in the list with some warn message.
I'll fix it
Again, thanks for the issue )


ADD: augmented ei

Pull Request Test Coverage Report for Build 117

54 of 54 (100.0%)  changed or added relevant lines in 5 files are covered.
7 unchanged lines in 3 files lost coverage.
Overall coverage decreased (-0.03%) to 99.373%

Files with Coverage Reduction
New Missed Lines
%

bayeso/utils/utils_plotting.py
1
99.29%

bayeso/utils/utils_bo.py
2
98.4%

bayeso/gp.py
4
100.0%

Totals

Change from base Build 115:
-0.03%

Covered Lines:
2220

Relevant Lines:
2234

💛  - Coveralls

Pull Request Test Coverage Report for Build 117

54 of 54 (100.0%)  changed or added relevant lines in 5 files are covered.
7 unchanged lines in 3 files lost coverage.
Overall coverage decreased (-0.03%) to 99.373%

Files with Coverage Reduction
New Missed Lines
%

bayeso/utils/utils_plotting.py
1
99.29%

bayeso/utils/utils_bo.py
2
98.4%

bayeso/gp.py
4
100.0%

Totals

Change from base Build 115:
-0.03%

Covered Lines:
2220

Relevant Lines:
2234

💛  - Coveralls


FEAT-#5759: Implement lazy Arrow execution for the HDK engine

What do these changes do?

Reimplemented _union_all(), that used non-lazy arrow-based frames concatenation.

Added functionality to dataframe tree nodes and expressions for arrow execution. The arrow execution is performed if the frame has an arrow table in partitions and all the nodes support the arrow execution. It allows to save on the table import to HDK.

The arrow execution is also performed for the operations, not supported by HDK, for example, bitwise not '~'.

[x] first commit message and PR title follow format outlined here

NOTE:  If you edit the PR title to match this format, you need to add another commit (even if it's empty) or amend your last commit for the CI job that checks the PR title to pick up the new PR title.

[x] passes flake8 modin/ asv_bench/benchmarks scripts/doc_checker.py

[x] passes black --check modin/ asv_bench/benchmarks scripts/doc_checker.py

[x] signed commit with git commit -s

[x] Resolves #5759

[x] tests added and passing

[ ] module layout described at docs/development/architecture.rst is up-to-date

@AndreyPavlenko this pull request is very large, how many logical parts can you split it into?

@AndreyPavlenko this pull request is very large, how many logical parts can you split it into?

There are several logical parts in - modin/experimental/core/execution/native/implementations/hdk_on_native/dataframe/dataframe.py:

New implementation of _union_all(). This is the biggest part in this file, but the new implementation is much smaller than the previous one.
Changes to _execute() and _materialize(), related to the arrow-based execution logic.
Added new methods - isna() and invert(). This part is quite simple.

Although there are a lot of changes in this file, most of the changes are deletions. The code, related to the arrow execution, moved to df_algebra.py.
New methods added to df_algebra.py and expr.py. These methods contains arrow-based execution (partially, the code moved from dataframe.py) and expressions folding.

Why don't we split this pull request into 3 parts in that case?

Why don't we split this pull request into 3 parts in that case?

They depend on each other. A lot of code from dataframe.py moved to df_algebra.py and expr.py. I could move isna() and invert() into a separate PR, but does it make sense? The code is quite simple. Moreover, these methods are used for testing of the main functionality, introduced by this PR.

Although there are a lot of changes in this file, most of the changes are deletions. The code, related to the arrow execution, moved to df_algebra.py.

Moving the code can be put in a separate pull request, right? This will greatly facilitate the review, since there is no need to check the code that has not changed, but is highlighted here.

Moving the code can be put in a separate pull request, right?

The code is not just moved but also changed. For example:
_arrow_row_slice moved to execute_arrow
_arrow_select moved to execute_arrow
_union_all_arrow moved to execute_arrow
The last 2 have been changed significantly.

_arrow_row_slice moved to execute_arrow

Let's start simplifying this pull request with _arrow_row_slice. Its relocation as well as the introduction of a new interface for all classes inherited from DFAlgNode.

Let's start simplifying this pull request with _arrow_row_slice. Its relocation as well as the introduction of a new interface for all classes inherited from DFAlgNode.

Do you think it could simplify review? _arrow_row_slice is almost not changed but just moved to a different file with minimal changes.

Let's start simplifying this pull request with _arrow_row_slice. Its relocation as well as the introduction of a new interface for all classes inherited from DFAlgNode.

Do you think it could simplify review? _arrow_row_slice is almost not changed but just moved to a different file with minimal changes.

Yes, however, this is not the last part that I will ask to be taken out in a separate pull request.
Please also keep in mind that several simple pull requests are easier to accept and merge than one large.

Please also keep in mind that several simple pull requests are easier to accept and merge than one large.

I understand that, but, from the other side, the PR should contain a completed feature. This PR reimplements the arrow-based execution logic.  Obviously, it affects the old implementation of the arrow execution. What can be moved out of this PR are isna() and invert(), but I believe it will not simplify the code review at all. Moreover, these methods are used for testing of the new functionality.


  This is the expected behavior of the build system.

OPS Build status updates of commit 0eb3205:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

project/docfx.json
:white_check_mark:Succeeded

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.


Get jfrog-cli-layout from https://releases.jfrog.io/artifactory/jfrog-cli/

Hello,
I want to make a local repository of jfrog-cli. But can't repeat jfrog-cli-layout to be able to download the last release like
https://releases.jfrog.io/artifactory/jfrog-cli/v2-jf/[RELEASE]/jfrog-cli-linux-amd64/jf use [RELEASE] instead version.
Can you share jfrog-cli-layout?

@Vvlad1slavV
The simplest and most convenient option is to use https://releases.jfrog.io/artifactory/jfrog-cli as the URL for a new "jfrog-cli-remote " remote repository. No specific layout is required in this scenario.
If you need the JFrog CLI binary to be in a local repository, you can use the following Artifact Path Pattern:
[orgPath]/[baseRev]/[module]/jf(.[ext])

Please let me know if this solution works for you.

Thanks a lot!
This option is work.
I tried to follow the documentation https://jfrog.com/help/r/jfrog-rest-apis/retrieve-latest-artifact and the phrase confused me
Both [folderItegRev] and [fileItegRev] have to be defined in the repository layout.


optical density changes

mne/preprocessing/nirs/_optical_density.py
what do these changes you made do:

the original code assumed the data was fnirs_raw (like line 31) and this was giving an error when passing phase data...now that I think about it I don't think we'll ever pass phase data to this now.
I'll change this back to what the original was.

this should be addressed in this PR: https://github.com/kylemath/mne-python/pull/11


Crash when opening same NC4 file from different threads BUT under global mutex

This is an attempt at providing a minimum reproducer for https://github.com/OSGeo/gdal/issues/6253
The attached docker.zip contains a simple Dockerfile building libhdf5 and libnetcdf, and a simple C++ program.
The C++ program loops at creating 2 threads, which open the same NC4 file and one calls nc_inq_varname(). It is to be noted that all calls to the netCDF API are protected by a common mutex , so there's no concurrent access to the netCDF API.
I've tried different versions of hdf5 and netcdf, and compiling hdf5 with or without  --enable-unsupported --enable-threadsafe, but the crash always occur
How to reproduce:
unzip docker.zip
cd docker
docker build -t netcdf_issue .
docker run --rm -it netcdf_issue:latest  /usr/bin/test /alldatatypes.nc
docker run --rm -it netcdf_issue:latest  valgrind /usr/bin/test /alldatatypes.nc || echo "this failed!"

results in
T1 begin
T2 begin
T1 open 65536
T2 open 131072
T1 close 65536
T1 end
T2 begin nc_inq_varname(131072, 15)
this failed!

and under valgrind:
$ docker run --rm -it netcdf_issue:latest  valgrind /usr/bin/test /alldatatypes.nc
[...]
==1== Thread 3:
==1== Invalid read of size 1
==1==    at 0x4F921F8: H5F_addr_decode (in /usr/lib/libhdf5.so.302.0.0)
==1==    by 0x519A1F6: H5VL__native_blob_specific (in /usr/lib/libhdf5.so.302.0.0)
==1==    by 0x51922AC: H5VL_blob_specific (in /usr/lib/libhdf5.so.302.0.0)
==1==    by 0x517F48F: H5T__vlen_disk_isnull (in /usr/lib/libhdf5.so.302.0.0)
==1==    by 0x510387B: H5T__conv_vlen (in /usr/lib/libhdf5.so.302.0.0)
==1==    by 0x50EDD6B: H5T_convert (in /usr/lib/libhdf5.so.302.0.0)
==1==    by 0x4F57159: H5D_get_create_plist (in /usr/lib/libhdf5.so.302.0.0)
==1==    by 0x519B3D4: H5VL__native_dataset_get (in /usr/lib/libhdf5.so.302.0.0)
==1==    by 0x5186A77: H5VL_dataset_get (in /usr/lib/libhdf5.so.302.0.0)
==1==    by 0x4F2772A: H5Dget_create_plist (in /usr/lib/libhdf5.so.302.0.0)
==1==    by 0x48EDCB5: nc4_get_var_meta (in /usr/lib/libnetcdf.so.19.1.0)
==1==    by 0x48EAC09: nc4_hdf5_find_grp_var_att (in /usr/lib/libnetcdf.so.19.1.0)
==1==  Address 0x40 is not stack'd, malloc'd or (recently) free'd
==1==
==1==
==1== Process terminating with default action of signal 11 (SIGSEGV): dumping core
==1==  Access not within mapped region at address 0x40
==1==    at 0x4F921F8: H5F_addr_decode (in /usr/lib/libhdf5.so.302.0.0)
==1==    by 0x519A1F6: H5VL__native_blob_specific (in /usr/lib/libhdf5.so.302.0.0)
==1==    by 0x51922AC: H5VL_blob_specific (in /usr/lib/libhdf5.so.302.0.0)
==1==    by 0x517F48F: H5T__vlen_disk_isnull (in /usr/lib/libhdf5.so.302.0.0)
==1==    by 0x510387B: H5T__conv_vlen (in /usr/lib/libhdf5.so.302.0.0)
==1==    by 0x50EDD6B: H5T_convert (in /usr/lib/libhdf5.so.302.0.0)
==1==    by 0x4F57159: H5D_get_create_plist (in /usr/lib/libhdf5.so.302.0.0)
==1==    by 0x519B3D4: H5VL__native_dataset_get (in /usr/lib/libhdf5.so.302.0.0)
==1==    by 0x5186A77: H5VL_dataset_get (in /usr/lib/libhdf5.so.302.0.0)
==1==    by 0x4F2772A: H5Dget_create_plist (in /usr/lib/libhdf5.so.302.0.0)
==1==    by 0x48EDCB5: nc4_get_var_meta (in /usr/lib/libnetcdf.so.19.1.0)
==1==    by 0x48EAC09: nc4_hdf5_find_grp_var_att (in /usr/lib/libnetcdf.so.19.1.0)
==1==  If you believe this happened as a result of a stack

Dockerfile:
FROM ubuntu:20.04
RUN apt-get update && DEBIAN_FRONTEND=noninteractive apt-get install -y --fix-missing --no-install-recommends g++ make autoconf automake zlib1g-dev wget tar ca-certificates
RUN wget https://github.com/HDFGroup/hdf5/archive/refs/tags/hdf5-1_13_2.tar.gz && \
    tar xvzf hdf5-1_13_2.tar.gz && \
    cd hdf5-hdf5-1_13_2 && \
    #  --enable-unsupported --enable-threadsafe
    ./configure --prefix=/usr --disable-static --disable-tests && \
    make -j$(nproc) install
RUN wget https://github.com/Unidata/netcdf-c/archive/refs/tags/v4.9.0.tar.gz && \
    tar xvzf v4.9.0.tar.gz && \
    cd netcdf-c-4.9.0 && \
    ./configure --prefix=/usr && \
    make -j$(nproc) install
COPY test.cpp /
COPY alldatatypes.nc /
RUN g++ /test.cpp -lnetcdf -lpthread -o /usr/bin/test
RUN DEBIAN_FRONTEND=noninteractive apt-get install -y --fix-missing --no-install-recommends valgrind

test.cpp:
#include <thread>
#include <mutex>
#include <chrono>

#include "netcdf.h"

std::mutex oMutex;

int main(int argc, char* argv[])
{
    for (int i = 0; i < 1000; i++) {
        std::thread t1([argv] {
            int cdfid;
            printf("T1 begin\n");
            {
                std::lock_guard<std::mutex> oLock(oMutex);
                nc_open(argv[1], NC_NOWRITE, &cdfid);
                printf("T1 open %d\n", cdfid);
            }
            std::this_thread::sleep_for(std::chrono::microseconds(1));
            {
                std::lock_guard<std::mutex> oLock(oMutex);
                nc_close(cdfid);
                printf("T1 close %d\n", cdfid);
            }
            printf("T1 end\n");
        });
        std::thread t2([argv] {
            int cdfid2;
            printf("T2 begin\n");
            {
                std::lock_guard<std::mutex> oLock(oMutex);
                nc_open(argv[1], NC_NOWRITE, &cdfid2);
                printf("T2 open %d\n", cdfid2);
            }
            std::this_thread::sleep_for(std::chrono::microseconds(1));
            {
                int nVarId = 0;
                std::lock_guard<std::mutex> oLock(oMutex);
                nc_inq_varid(cdfid2, "string_var", &nVarId);
                char szName[NC_MAX_NAME + 1] = {};
                printf("T2 begin nc_inq_varname(%d, %d)\n", cdfid2, nVarId);
                nc_inq_varname(cdfid2, nVarId, szName);
                printf("T2 end nc_inq_varname(%d, %d)\n", cdfid2, nVarId);
            }
            {
                std::lock_guard<std::mutex> oLock(oMutex);
                nc_close(cdfid2);
                printf("T2 close %d\n", cdfid2);
            }
            printf("T2 end\n");

        });
        t1.join();
        t2.join();
    }
    printf("success!\n");
    return 0;
}

Possibly related, we're seeing a similar crash when opening different files: https://github.com/georust/gdal/issues/299. One difference is that there it seems to overflow the stack while trying to print a stack trace (but that's just a guess, I'm not familiar with the two libraries).

Do you know if you are using pthreads for your mutex?

Do you know if you are using pthreads for your mutex?

using std::thread requires linking the code with -lpthread. Apparently the std::mutex implementation on Linux / gcc also uses pthread_mutex_lock() / pthread_mutex_unlock() underneeth.
cf
$ cat test2.cpp

#include <mutex>

std::mutex x;
int y = 0;
int foo()
{
    std::lock_guard<std::mutex> locker(x);
    return ++y;
}

$ g++ -O2 test2.cpp -c
$ LC_ALL=C objdump -sdwxC test2.o

0000000000000000 <foo()>:
   0:	f3 0f 1e fa          	endbr64
   4:	41 54                	push   %r12
   6:	53                   	push   %rbx
   7:	48 83 ec 08          	sub    $0x8,%rsp
   b:	48 8b 1d 00 00 00 00 	mov    0x0(%rip),%rbx        # 12 <foo()+0x12>	e: R_X86_64_REX_GOTPCRELX	__pthread_key_create-0x4
  12:	48 85 db             	test   %rbx,%rbx
  15:	74 10                	je     27 <foo()+0x27>
  17:	48 8d 3d 00 00 00 00 	lea    0x0(%rip),%rdi        # 1e <foo()+0x1e>	1a: R_X86_64_PC32	x-0x4
  1e:	e8 00 00 00 00       	callq  23 <foo()+0x23>	1f: R_X86_64_PLT32	pthread_mutex_lock-0x4
  23:	85 c0                	test   %eax,%eax
  25:	75 2d                	jne    54 <foo()+0x54>
  27:	8b 05 00 00 00 00    	mov    0x0(%rip),%eax        # 2d <foo()+0x2d>	29: R_X86_64_PC32	y-0x4
  2d:	44 8d 60 01          	lea    0x1(%rax),%r12d
  31:	44 89 25 00 00 00 00 	mov    %r12d,0x0(%rip)        # 38 <foo()+0x38>	34: R_X86_64_PC32	y-0x4
  38:	48 85 db             	test   %rbx,%rbx
  3b:	74 0c                	je     49 <foo()+0x49>
  3d:	48 8d 3d 00 00 00 00 	lea    0x0(%rip),%rdi        # 44 <foo()+0x44>	40: R_X86_64_PC32	x-0x4
  44:	e8 00 00 00 00       	callq  49 <foo()+0x49>	45: R_X86_64_PLT32	pthread_mutex_unlock-0x4
  49:	48 83 c4 08          	add    $0x8,%rsp
  4d:	44 89 e0             	mov    %r12d,%eax
  50:	5b                   	pop    %rbx
  51:	41 5c                	pop    %r12
  53:	c3                   	retq
  54:	89 c7                	mov    %eax,%edi
  56:	e8 00 00 00 00       	callq  5b <x+0x3b>	57: R_X86_64_PLT32	std::__throw_system_error(int)-0x4

To be noted that the original reproducer with GDAL uses a global lock that is a pthread_mutex_t
I seem to have found a workaround on GDAL side in https://github.com/OSGeo/gdal/pull/6311 by re-using the same netcdf handle when sequences like nc_open("sam_file.nc", ....); nc_open("sam_file.nc", ....); are done


[v0.5.0] feat: add new event::Store trait definition

This PR marks the start of the new v0.5.0 API.
The new eventually crate code will live in eventually-v0-5-0, until we're able to decommission the old crate completely.
The focus of this PR is the new event::Store trait, which is a greatly simplified version of the previous eventually::store::EventStore one:

No remove method: removing Domain Events goes against Event Sourcing, and Compensating Actions should be used to "remove" or revert things instead;
No stream_all method: streaming all Domain Events for a specific Event Stream category (or all Domain Events in the Event Store) is a concern belonging to Event Subscriptions, not Event Store;
Two separate errors for append and stream (Store::AppendError, Store::StreamError):
On this point, still unsure if these should be two separate traits instead :shrugs:

Other relevant changes from the previous version:

Events are Messages, and Messages have an internal payload (e.g. the actual Event type) and metadata,
Versions are u64 after the discussion in #187
Public fields are used instead of builders: builders do add a lot of complexity to the code and I'd like generally to avoid them. Maybe there is a better way to enforce immutability of types without the complexity of a full-fledged builder type?

:v:
https://github.com/get-eventually/eventually-rs/projects/1#card-72334587
https://github.com/get-eventually/eventually-rs/projects/1#card-72334528

@danieleades @Acidic9 would you be up for reviewing this chunk of code? 👀

Thanks for reviewing @danieleades ❤
Feel more confident about the PR now, I will go ahead and merge this one 👍🏻


There is no category: "WorkflowRuntime"

Hello,
I tried to deploy ARM template using Azure Blueprint with category specified on this page, but with no luck due to issue:
[ { "code": "BadRequest", "message": "{\r\n "code": "BadRequest",\r\n "message": "Category 'WorkflowRuntime' is not supported."\r\n}" } ]
Is it misunderstanding from my side, or this category is not currently supported? Of course, I'm trying to deploy it not for logic app, but for Azure Service bus.

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

ID: b8f43189-3418-fd30-0528-550157d8366d
Version Independent ID: 049e3b0e-6f10-6985-c719-05fe43fc00b8
Content: Create diagnostic setting in Azure using Resource Manager template
Content Source: articles/azure-monitor/platform/diagnostic-settings-template.md
Service: azure-monitor
Sub-service: ****
GitHub Login: @bwren
Microsoft Alias: bwren

@Darleev
Thanks for the feedback! We are currently investigating and will update you shortly.

Team, my apologies, there was my knowledge gap. I found correct way to fix the issue. For each application diagnostic setting with logs and metric variables should be set accordingly. For Service Bus this is: "OperationalLogs". Thank you!


Handling Reduced Visual Tokens During Inference in LLMs

I’m not sure if the LLM can handle a different number of visual tokens than what was used during training. If N visual tokens are discarded, is there a step to adjust the dimension before feeding them into the LLM?

In their code, redundant visual tokens are pruned inside LLM (decode layer[2, 6, 15, 19]). There is no step to adjust the dimension. After pruned, these rest tokens(hidden_states) will be passed in the next layer.